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

DOC: Updating instrument intro list. #105

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

zssherman
Copy link
Collaborator

@zssherman zssherman commented Aug 8, 2024

@AdamTheisen Let me know if I'm missing any of the new ones

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@zssherman
Copy link
Collaborator Author

@AdamTheisen just repinging as i think I originally pinged late on a Friday lol, but do you think once this is merge, this should be good to send to Lexie?

Copy link
Member

@AdamTheisen AdamTheisen left a comment

Choose a reason for hiding this comment

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

@zssherman looks good! Yes, lets merge in and send a link along. I might do one last run through later this week and see if there's anything missing or if we can add a little more content.

@zssherman
Copy link
Collaborator Author

@AdamTheisen Sounds good! I can run through the notebooks as well to make sure everything is good.

@zssherman zssherman merged commit 0f38669 into ARM-Development:main Aug 13, 2024
2 checks passed
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