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

MNT: Cleanup ruff codes #314

Merged
merged 2 commits into from
Jan 12, 2024

Conversation

greglucas
Copy link
Collaborator

Change Summary

Overview

Update the pyproject.toml ruff flags that we are ignoring.

This looks like a large diff, but is largely a reorganization within the IDEX code. There was a large trigger_dict block that was being set in each object, but it was more of a global constant with information. I pulled that outside the scope of the object and index into the NamedTuples later instead. I also updated it so that the "calc_*" functions don't return things now, but rather set the values to the attributes directly, so I changed the name of those to make it clear we are setting things.

@greglucas greglucas added the Ins: IDEX Related to the IDEX instrument label Jan 12, 2024
Copy link
Collaborator

@bourque bourque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Greg! Looks good to me.

We can rework the IDEX code to pull out the unchanging variables
from the objects and keep that separate.
@greglucas
Copy link
Collaborator Author

I saw that the MAPPING global was actually never used. I turned that into an Enum now in a separate following commit if you wanted to check that at all too.

@bourque
Copy link
Collaborator

bourque commented Jan 12, 2024

Looks good!

@greglucas greglucas merged commit d70afd8 into IMAP-Science-Operations-Center:dev Jan 12, 2024
17 checks passed
@greglucas greglucas deleted the ruff-cleanup branch January 12, 2024 22:40
laspsandoval pushed a commit to laspsandoval/imap_processing that referenced this pull request Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ins: IDEX Related to the IDEX instrument
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants