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

Adding plugins for doing processing of differential phase contrast datasets #134

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

magnunor
Copy link
Contributor

@magnunor magnunor commented Apr 17, 2017

Adding plugins for doing processing of differential phase contrast datasets.

The plugins made in #122, #123, #124, #125 and #126 has been combined into a single file.

  • Add unit tests
  • Make the record functionality work
  • Better documentation
  • Fix comments from last pull requests

@magnunor
Copy link
Contributor Author

Any good examples on how to make the recording functionality work?

@vidartf
Copy link
Member

vidartf commented Apr 17, 2017

Looking at the code, I'm not entirely sure how to best record as code: Almost all your entry points start off with select_x_signals which is hard to record well as a macro code. Maybe it is best to simply leave it out, and have the user rely on recorded actions?

@magnunor
Copy link
Contributor Author

Added unit tests for the processing itself

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