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

Upgrade to python 3.8? #32

Open
capitanevs opened this issue Dec 7, 2020 · 2 comments
Open

Upgrade to python 3.8? #32

capitanevs opened this issue Dec 7, 2020 · 2 comments
Labels
STAND BY featured topic right now.

Comments

@capitanevs
Copy link
Collaborator

Here I report some (apparently) weird behaviors I caught while writing the code.

In OpticsNumerical

Left is wrong, right is ok

immagine
Yellow = rx = ok
grey = sx = wrong
immagine
immagine

I have no ideas (perhaps try is very strict and it catches warning, too?).

Action: I revert to RX version.

@capitanevs
Copy link
Collaborator Author

LX: wrong
RX: Correct
immagine

Screenshot of the error, with LX
immagine

Comment:
it must be some stupid "boundary error" (points are 10000, returned are 9999)

In any case, if I geld XProp I should also take care of what is happening to other vectors (which will be made up of \tilde(N) points and not N).

@capitanevs capitanevs added the warning useful to know label Dec 7, 2020
@aljosahafner aljosahafner changed the title notes: attempts in moving to python 2.8 Upgrade to python 3.8? Jan 22, 2021
@aljosahafner aljosahafner added the help wanted Extra attention is needed label Jan 22, 2021
@capitanevs capitanevs removed the help wanted Extra attention is needed label Mar 8, 2021
@capitanevs
Copy link
Collaborator Author

We decided that this is not a priority as OASYS is stuck to a previous version of python.

Compativility with Python> 3.7 COULD BE RELEVANT when using LibWiser only, but this is not a priority now.

@capitanevs capitanevs added STAND BY featured topic right now. and removed warning useful to know labels Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
STAND BY featured topic right now.
Projects
None yet
Development

No branches or pull requests

2 participants