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

Typo in the physical_optics_propagation function #111

Closed
5 tasks
Lupatran opened this issue Jan 14, 2025 · 2 comments · Fixed by #112
Closed
5 tasks

Typo in the physical_optics_propagation function #111

Lupatran opened this issue Jan 14, 2025 · 2 comments · Fixed by #112
Labels
bug Something isn't working

Comments

@Lupatran
Copy link
Contributor

Problem

Hi!

I found a small issue in the code of zospy/analyses/physicaloptics.py.
At lines 334-335, I noticed this:

elif isinstance(start_surface, int):
        analysis.Settings.StartSurface.SetSurfaceNumber(1)

It seems like it should be:

elif isinstance(start_surface, int):
        analysis.Settings.StartSurface.SetSurfaceNumber(start_surface)

I'm not a great GitHub user, so please let me know if there's a better way to report issues like this.

Thanks ! :)

What version of ZOSPy are you running?

1.3.0

What version of OpticStudio are you running?

not concerned

Which operating system do you use?

not concerned

In which environment do you use ZOSPy?

  • Plain Pythons scripts (not in Spyder)
  • Jupyter notebooks
  • Spyder

In which connection modes does the problem occur?

  • Extension mode
  • Standalone mode

Example code

No response

Output

No response

Additional information

No response

@LucVV LucVV added the bug Something isn't working label Jan 14, 2025
@LucVV
Copy link
Contributor

LucVV commented Jan 14, 2025

Hi @Lupatran,

This indeed seems to be implemented wrong and should be fixed. Can you create a small pull request that fixes it?

@LucVV
Copy link
Contributor

LucVV commented Jan 20, 2025

@Lupatran version 1.3.1 in which you fix is implemented has just been released!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants