You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have 3 different executables: aestimo.py For electron solutions only aestimo_eh.py with using k.p theorem, it can do solutions for holes. aestimo_poisson1d I really do not know much about this. It is some codes translated from a book as I understand.
aestimo_eh.py can be used like aestimo.py, but if you want you can use it as a library. But it is not true for aestimo.py.
So, these things are quite confusing. @robochat and @h-hebal is it possible to write your opinions about the unification of these three files and maintain a simple usage? We can remove cython parts if necessary because our simulations are not time-consuming and pure python can run well.
If you can help to simplify the code, then we can change our working type from "directly commit to master branch" to "create pull-request first". And we can continue with small steps.
The text was updated successfully, but these errors were encountered:
aestimo_poisson1d is used as a library for Poisson and DD solvers (scheme 7 and 8) (not executable file).
Cython option is helpful for creating fast .pyd importing functions library. I will remove duplicated and unused functions and rearrange them gradually.
We have 3 different executables:
aestimo.py For electron solutions only
aestimo_eh.py with using k.p theorem, it can do solutions for holes.
aestimo_poisson1d I really do not know much about this. It is some codes translated from a book as I understand.
aestimo_eh.py can be used like aestimo.py, but if you want you can use it as a library. But it is not true for aestimo.py.
So, these things are quite confusing. @robochat and @h-hebal is it possible to write your opinions about the unification of these three files and maintain a simple usage? We can remove cython parts if necessary because our simulations are not time-consuming and pure python can run well.
If you can help to simplify the code, then we can change our working type from "directly commit to master branch" to "create pull-request first". And we can continue with small steps.
The text was updated successfully, but these errors were encountered: