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

Definition of dirichlet series #626

Merged
merged 6 commits into from May 22, 2023

Conversation

VictorBlanchi
Copy link
Contributor

I have cleaned the file dirichlet-products-species-of-types-in-subuniverse.lagda.md.
I have defined Dirichlet series for species of finite inhabited types and for species of types in subuniverses.
I have also defined the product for each of these series

…verse.lagda.md. I have Dirichlet series for species of finite inhabited types and for species of types in subuniverses. I also defined the product for each of these series
Copy link
Collaborator

@fredrik-bakke fredrik-bakke left a comment

Choose a reason for hiding this comment

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

Just some comments on spelling 🙂

@VictorBlanchi
Copy link
Contributor Author

Thank you for the review :)

@fredrik-bakke
Copy link
Collaborator

Alright, I don't have anything more to add at this point. I'll wait and see if Egbert has anything to add, but otherwise, it's good to merge I think.

@VictorBlanchi
Copy link
Contributor Author

Nice, thank you :) .
I was wondering, I want to open a new PR, but it is based a bit on this one. Should I wait until this one is merge, or should I just push all the new change in this PR ?

@fredrik-bakke
Copy link
Collaborator

fredrik-bakke commented May 22, 2023

Thanks for the question. Ideally, you would branch from your current PR branch, and then rebase onto agda-unimath/master after this PR is merged, I think. However, this is tricky to do(read: I mess this up every time) so I'll do you a favour and merge this PR now.

@VictorBlanchi
Copy link
Contributor Author

Thank you so much :)

@fredrik-bakke fredrik-bakke merged commit 31cff22 into UniMath:master May 22, 2023
3 checks passed
@EgbertRijke
Copy link
Collaborator

Excellent! I agree with the merge. Good work!

@EgbertRijke
Copy link
Collaborator

This PR suffered from #595

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.

None yet

3 participants