Skip to content

Last two missing titles#2386

Merged
zklaus merged 2 commits intomainfrom
last_two_missing_titles
Oct 26, 2021
Merged

Last two missing titles#2386
zklaus merged 2 commits intomainfrom
last_two_missing_titles

Conversation

@valeriupredoi
Copy link
Copy Markdown
Contributor

@valeriupredoi valeriupredoi commented Oct 26, 2021

Description


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

Copy link
Copy Markdown
Contributor

@bettina-gier bettina-gier left a comment

Choose a reason for hiding this comment

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

Fine by me - my recipe still won't run due to other issues though ^^

@valeriupredoi
Copy link
Copy Markdown
Contributor Author

cheers @bettina-gier 🍺 Let's fix your recipe next, where is the issue I can look at?

@bettina-gier
Copy link
Copy Markdown
Contributor

It's that int64 thing for the time coordinate in the mmm, reported in #2354 , think Klaus wanted to have a look.

@zklaus zklaus merged commit 49fb820 into main Oct 26, 2021
@zklaus zklaus deleted the last_two_missing_titles branch October 26, 2021 13:15
@valeriupredoi
Copy link
Copy Markdown
Contributor Author

gonna have a looksee too 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants