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

NSE calculation #914

Closed
wants to merge 43 commits into from
Closed

NSE calculation #914

wants to merge 43 commits into from

Conversation

nicola
Copy link
Contributor

@nicola nicola commented Apr 23, 2020

To run:

$ ./orient/bin/orient web --system=src/orient/nse.orient --port=8000
$ obs serve 8081 src/orient/viz

@nicola
Copy link
Contributor Author

nicola commented May 12, 2020

This PR is ready to merge on my end.

@sternhenri
Copy link
Contributor

Have a reviewer who should look at this @nicola ?

@nicola
Copy link
Contributor Author

nicola commented May 16, 2020

It could be @porcuquine or @irenegia maybe?

@nicola
Copy link
Contributor Author

nicola commented Jul 7, 2020

We should just merge it in, this does not have any impact on the spec.

@daviddias daviddias added the scope: content Scope: Editing content of the spec label Sep 8, 2020
@yiannisbot
Copy link
Collaborator

@nicola can you advise what we do with this PR? As far as I can see it does not affect the spec doc itself, so we can just merge it if it's still relevant.

@nicola
Copy link
Contributor Author

nicola commented Nov 6, 2020

yes we should merge it! I still use this code every now and then. It's the go to resource for NSE uber-calc

@yiannisbot
Copy link
Collaborator

Thanks @nicola!

@hugomrdias can you take care of merging this PR please?

@hugomrdias
Copy link
Contributor

this is old stuff @yiannisbot it uses orient

@nicola
Copy link
Contributor Author

nicola commented Nov 9, 2020

It's ok to make a new repo for these models

@ianconsolata
Copy link
Collaborator

This is more than 3 years old now -- is this PR still relevant, or can I close it?

@vmx
Copy link
Contributor

vmx commented Jul 25, 2023

I'll close this one as NSE never hit mainnet and the code is preserved in this PR, it doesn't need to be on the main branch. In case anyone disagrees, feel free to re-open.

@vmx vmx closed this Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: content Scope: Editing content of the spec
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants