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

Would like to be able to do DIC on in-memory images #1

Open
sdh4 opened this issue Mar 6, 2019 · 2 comments
Open

Would like to be able to do DIC on in-memory images #1

sdh4 opened this issue Mar 6, 2019 · 2 comments

Comments

@sdh4
Copy link

sdh4 commented Mar 6, 2019

I would like to be able to do DIC on in-memory images without having to write them to a file (they are not stored in a standard format). I'd be happy to implement the capability and submit a pull request.

Any suggestions on the approach?

One simple option would be to modify the Image2D class to handle both cases (filename, and Array2D, only one of which is valid at any given time.

Another option could be to make the Image2D class a superclass with subclasses for file input and array input.

A third option could be to make the existing Image2D class no longer "final" and make its methods virtual, so that a derived class could be externally defined. In this case the interface functions would need to become virtual methods (or thin layers over virtual methods) so that they will properly call the derived class's methods.

Thoughts?

@sdh4
Copy link
Author

sdh4 commented Mar 20, 2019

Implemented the first option mentioned above (Image2D class handles both cases) and submitted a pull request #2 . Feel free to reject if you don't like this approach.

@sdh4
Copy link
Author

sdh4 commented Oct 16, 2019

.... Happy to help out if you want me to test out changes or propose an abstract class interface. Anything terribly substantive might have to wait until the new year because of current time committments.

I've been using my fork (https://github.com/sdh4/ncorr_2D_cpp ) very successfully for the last six months. It has been working very well and we may be presenting at EM next year!

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

No branches or pull requests

1 participant