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

lightfield sensor and source class #113

Open
StefanThoene opened this issue Nov 16, 2022 · 5 comments
Open

lightfield sensor and source class #113

StefanThoene opened this issue Nov 16, 2022 · 5 comments
Assignees

Comments

@StefanThoene
Copy link
Collaborator

StefanThoene commented Nov 16, 2022

create light field sensor and source classes

@StefanThoene
Copy link
Collaborator Author

@BenoitHeraud I created this issue so you can link your work to it

@StefanThoene StefanThoene changed the title lightfield sensor and soruce class lightfield sensor and source class Nov 16, 2022
@BenoitHeraud
Copy link
Collaborator

@pluAtAnsys

@BenoitHeraud
Copy link
Collaborator

@StefanThoene @pluAtAnsys I have written new methods in LightFieldSensor class:

  • set_type
  • set sampling
    I also created LocalMeshing class with inside:
  • set_faces
  • set_Sag
  • set_Step

I written it in Lighfield sensor and source branch/speos_sensor.py on my local computer.
I don't think it's the right place to have it → need to be reviewed with one of you :)

@pluAtAnsys
Copy link
Collaborator

@BenoitHeraud
I refactor the LocalMeshing into separate class
I add speos_source class module, please follow them to add your corresponding methods

@BenoitHeraud
Copy link
Collaborator

BenoitHeraud commented Dec 5, 2022

Hi @pluAtAnsys @StefanThoene I have created a .py file, in application section, on my local GitHub. Should I do a push in order to create it into Optical Automation Github? (it is still work in progress, not functional for the moment)
image

@BenoitHeraud BenoitHeraud linked a pull request Dec 6, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants