-
Notifications
You must be signed in to change notification settings - Fork 95
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
Validation script modifications #37
Comments
What was the change you made? |
I suspect ... I changed too many things when I tried it the last time. For this script above:
eV = 1240.0 / nm
eV = eV + 3 / 2 * kB * T # Assumes 3 dimensional degrees of freedom
nm = 1240.0 / eV
p1 = dist.lookup(nm) Comparison below:
|
By the way - I truly believe in the capability of PVTrace. I might do some pull requests with new additions if you are open to it! I like how clean and easy to understand it is. |
What features are you thinking of adding? I would certainly consider any pull requests. You want to make a proposal first so that it's definitely something I want to include in the project. |
Thanks for preparing the script above and the table! Makes it very clear. In fact I was half way along doing this myself. This is why I isolated this on its own branch. |
Sure! I will think about it a bit more - I will run the proposal by you. So far, I feel that the ability to import meshes would be the most welcome addition. I see that you have already created an issue about that. Would be great if we can import STL meshes. btw, feel free to use the script above as a demo script in the next version if you'd like. |
Thanks! Those examples went under my radar. I will take a look. |
The notebook includes comparison of other LSC models: * ICL Raytrace Model * ICL 3D Flux Model * ECN Raytrace Model The notebook is standalone and will generate a plot which compares pvtrace results against the other models. I also include Validation.png which contains the results of a long running simulation (1000 rays, 40 runs) which brings the uncertainty does to below +- 1%, and includes errors bars. Fixes #37
Validation notebook has been added! https://github.com/danieljfarrell/pvtrace/blob/dev/lsc-device/examples/Validation.ipynb Thanks for your input in getting this completed. |
See #37 |
The notebook includes comparison of other LSC models: * ICL Raytrace Model * ICL 3D Flux Model * ECN Raytrace Model The notebook is standalone and will generate a plot which compares pvtrace results against the other models. I also include Validation.png which contains the results of a long running simulation (1000 rays, 40 runs) which brings the uncertainty does to below +- 1%, and includes errors bars. Fixes #37
https://core.ac.uk/download/pdf/76989828.pdf validating against fig 3.21
The text was updated successfully, but these errors were encountered: