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.

Sign up for GitHub

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

Jump to bottom

Restructure code #5

Closed
cjdoris opened this issue Jun 2, 2020 · 2 comments · Fixed by #12
Closed

Restructure code #5

cjdoris opened this issue Jun 2, 2020 · 2 comments · Fixed by #12
Assignees
@cjdoris

Comments

@cjdoris
Copy link
Owner

cjdoris commented Jun 2, 2020

Separate out the code for Infinite and InfExtended (and the future InfExtendedTime) into separate files, something like:

InfExtended.jl
utils.jl
infinite/
  base.jl
  io.jl
  ...
infextended/
  base.jl
  io.jl
  ...
infextendedtime/
  ...
@cjdoris cjdoris self-assigned this Jun 2, 2020
@cjdoris cjdoris mentioned this issue Jun 2, 2020
Closed
@fchorney fchorney mentioned this issue Jun 4, 2020
Merged
@fchorney
Copy link
Contributor

fchorney commented Jun 5, 2020

Hey, just wondering if you were going to tackle this issue (I assume so since you assigned yourself). I possibly have some time to try and get this done myself, as it would be nice to have for #3.

@cjdoris
Copy link
Owner Author

cjdoris commented Jun 5, 2020

Be my guest!

@fchorney fchorney mentioned this issue Jun 8, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cjdoris cjdoris

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Restructure code, and rename InfExtended to InfExtendedReal fchorney/Infinity.jl
2 participants
@fchorney @cjdoris