Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include plot of Principal Strains #35

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

talledodiego
Copy link

With this PR I included a few features:

  • add the possibility to plot principal strains as contour
  • add the possibility to add title to the plots
  • add the possibility to specify the output resolution and the size of the figure
  • add the len function to the Fields object returning the number of frames saved (useful for instance to save all pictures in a loop after doing the dic analysis).

I worked over the dev branch because I wanted to use the implemented csv exporting and save_path which were not available in master branch.

With this PR I included a few features:
- add the possibility to plot principal strains as contour
- add the possibility to add title to the plots
- add the possibility to specify the output resolution and the size of the figure
- add the len function to the Fields object returning the number of frames saved (useful for instance to save all pictures in a loop after doing the dic analysis).

I worked over the dev branch because I wanted to use the implemented csv exporting and save_path which were not available in master branch.
@PolymerGuy
Copy link
Owner

Thanks for the nice new features! I'm a bit busy at the moment but will review the code as soon as possible.

@talledodiego
Copy link
Author

Thanks for the nice new features! I'm a bit busy at the moment but will review the code as soon as possible.

I perfectly understand. Thanks for your effort.
Let me know if I need to change something.

Diego

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants