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

chore: refactor B02_make_spectra_gFT.py #70

Closed
wants to merge 1 commit into from

Conversation

cpaniaguam
Copy link
Member

@cpaniaguam cpaniaguam commented Jan 18, 2024

Simple refactorings for improved readability

@cpaniaguam cpaniaguam requested a review from kmilo9999 January 18, 2024 17:22
@cpaniaguam cpaniaguam changed the title chore: Refactor code and improve readability chore: refactor B02_make_spectra_gFT.py Jan 18, 2024
@cpaniaguam cpaniaguam mentioned this pull request Jan 18, 2024
Base automatically changed from fix-step2 to feat-step5-intg January 18, 2024 19:04
Base automatically changed from feat-step5-intg to main January 29, 2024 14:27
@kmilo9999
Copy link
Collaborator

@cpaniaguam I wonder if this PR conflicts with #90 . Does the latter already have the changes for B02_make_spectra_gFT.py ?

@cpaniaguam cpaniaguam self-assigned this Feb 6, 2024
@cpaniaguam
Copy link
Member Author

@cpaniaguam I wonder if this PR conflicts with #90 . Does the latter already have the changes for B02_make_spectra_gFT.py ?

@kmilo9999 Yes, it might be conflicting now as this is an older PR and too many changes have been added to the core code base. I will convert it to a draft and revisit whether some work could be salvaged later.

@cpaniaguam cpaniaguam marked this pull request as draft February 6, 2024 16:24
@kmilo9999
Copy link
Collaborator

Old PR. The code in make_spectra has changed.

@kmilo9999 kmilo9999 closed this Feb 12, 2024
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