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
Implemented get_fast_buffer_spectrum() for the cseabreeze backend #233
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for your contribution @padalev 🙏
I made two comments to the PR. Once they are addressed, we should be good to merge this new feature ❤️
for more information, see https://pre-commit.ci
…ed now and tested again with spectrometer.
for more information, see https://pre-commit.ci
Thank you so much ❤️ I'll try to get out a release very soon! |
You can check out the change set here: 39278ef |
I implemented the get_fast_buffer_spectrum() method for the cseabreeze backend.
For the data retrievel the length of a raw spectrum must be known so I also added the _raw_spectrum_length() property.
The buffered data is returned as a list of namedtuples including the metadata and intensity information.
I could not find the right place for the documentation in the github repository, so I did not add any documentation appart from the basic code commenting.