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

Implemented get_fast_buffer_spectrum() for the cseabreeze backend #233

Merged
merged 11 commits into from Feb 21, 2024

Conversation

padalev
Copy link
Contributor

@padalev padalev commented Feb 8, 2024

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.

Copy link
Owner

@ap-- ap-- left a 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 ❤️

src/seabreeze/cseabreeze/c_seabreeze_wrapper.pyx Outdated Show resolved Hide resolved
src/seabreeze/cseabreeze/c_seabreeze_wrapper.pyx Outdated Show resolved Hide resolved
@padalev padalev requested a review from ap-- February 19, 2024 10:37
@ap-- ap-- merged commit 6e75500 into ap--:main Feb 21, 2024
1 check passed
@ap--
Copy link
Owner

ap-- commented Feb 21, 2024

Thank you so much ❤️
I made a few tiny formatting changes and renamed MetadataDataset to SpectrumMetadata.
Also np.uint24 isn't supported by numpy, so I raise an error for now.

I'll try to get out a release very soon!

@ap--
Copy link
Owner

ap-- commented Feb 21, 2024

You can check out the change set here: 39278ef

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.

None yet

2 participants