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

Inverse/Hyperbolic Trig Functions and NCL Receipt #69

Merged
merged 3 commits into from
Jul 3, 2024

Conversation

cyschneck
Copy link
Contributor

PR Summary

Inverse and Hyperbolic trig functions. To be added to existing trig functions notebook

Related Tickets & Documents

Closes #68

PR Checklist

General

  • PR includes a summary of changes
  • Link relevant issues, make one if none exist

@cyschneck cyschneck added ncl receipt Receipt files are small files with little to no narrative content ncl entry NCL entries are pages that explain specifically how to achieve something that was possible in NCL labels Jul 1, 2024
@cyschneck cyschneck self-assigned this Jul 1, 2024
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

github-actions bot commented Jul 1, 2024

Meowdy! See your PR preview:
🔍 Git commit SHA: 053ac79
✅ Deployment Preview URL: https://NCAR.github.io/geocat-applications/_preview/69

@cyschneck cyschneck marked this pull request as ready for review July 2, 2024 16:25
Copy link
Collaborator

@kafitzgerald kafitzgerald left a comment

Choose a reason for hiding this comment

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

Looks good other than the notebook diffs and I don't think that's a problem for now - just a nice to have for later.

I did notice the assertion statements don't seem to show outputs (or maybe I missed something). The diffs are visible and the important part is that they're there to run so I think it's all good.

@@ -76,7 +80,7 @@
},
{
"cell_type": "code",
"execution_count": 1,
"execution_count": 47,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you know if we've thought about any ways to filter these types of metadata changes out?

It l looks like we might be able to use nbconvert (see here).

Other options might look like moving to MyST MD or using jupytext.

I'll log this as an issue.

Copy link
Member

@anissa111 anissa111 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for this!

@cyschneck cyschneck merged commit e841684 into NCAR:main Jul 3, 2024
2 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 3, 2024
@cyschneck cyschneck deleted the trig_add_68 branch July 3, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ncl entry NCL entries are pages that explain specifically how to achieve something that was possible in NCL ncl receipt Receipt files are small files with little to no narrative content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inverse/Hyperbolic Trig Functions and NCL Receipts
3 participants