Skip to content

Computus Powershell Implementation #684

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

Merged
merged 2 commits into from
May 24, 2020

Conversation

dovisutu
Copy link
Contributor

@dovisutu dovisutu commented May 4, 2020

A literal translation of Julia version.

@leios
Copy link
Member

leios commented May 17, 2020

I wasn't quite able to run this (on linux), but I can try again later. If someone on Windows would like to give it a go and make sure the output is the same as other languages, that would be great!

@dovisutu
Copy link
Contributor Author

I'm able to run this locally with the output:

The following are the dates of the Paschal full moon (using Servois notation) and the date of Easter for 2020-2030 AD:
Year    Servois number  Easter
2020    8               April 12
2021    28              April 4
2022    16              April 17
2023    5               April 9
2024    25              March 31
2025    13              April 20
2026    2               April 5
2027    22              March 28
2028    10              April 16
2029    30              April 1
2030    18              April 21

Probably powershell refuses to run script because it's from remote and don't have a sign. Maybe try to copy the script to another .ps1 file and try to run that file?

@berquist berquist added the Implementation This provides an implementation for an algorithm. (Code and maybe md files are edited.) label May 24, 2020
Copy link
Member

@leios leios left a comment

Choose a reason for hiding this comment

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

This produces the right output and looks clean. Like with VimL, I am not going to be pressing style guidelines too heavily here. However, other people can definitely create a PR and fix the code later, if they want.

@leios leios merged commit 370f717 into algorithm-archivists:master May 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Implementation This provides an implementation for an algorithm. (Code and maybe md files are edited.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants