Conversation
…f multiple runs to extract response new file: yambopy/nl/sin_analysis.py => class for analysis of multiple runs with monochromatic signal to extract response
modified: yambopy/__init__.py=> freqmix_analysis added new file: yambopy/nl/freqmix_analysis.py=> analysis of two monochromatic signals (sum_frequencies) modified: yambopy/nl/nl_analysis.py=> changes to further generalise modified: yambopy/nl/sin_analysis.py=> changes as consequence of freqmix_analysis addition
modified: yambopy/nl/nl_analysis.py modified: yambopy/nl/sin_analysis.py changes result from debug on a test case.
modified: yambopy/nl/freqmix_analysis.py Changed header of output files
modified: yambopy/nl/freqmix_analysis.py modified: yambopy/nl/nl_analysis.py modified: yambopy/nl/sin_analysis.py Added an extra lstsq solver based on the scipy lstsq optimizer which takes initial values
modified: yambopy/nl/freqmix_analysis.py modified: yambopy/nl/nl_analysis.py coded the spike_correction function even though it is not used so far
new file: yambopy/nl/Xn_from_signal_diagrams.md started a doc of the code suite for nl analysis ). use the md that can be then converted in whatevers
new file: yambopy/nl/pulse_analysis.py Started the new class for pulse analysiss
modified: yambopy/nl/external_efield.py modified: yambopy/nl/freqmix_analysis.py modified: yambopy/nl/pulse_analysis.py modified: yambopy/nl/sin_analysis.py finished to code the pulse analysis and small chenges to the other files. pulse analysis to be tested.
modified: yambopy/nl/Xn_from_signal_diagrams.md Added a brief theory section to the documentation
deleted: yambopy/nl/Xn_from_signal_diagrams.md new file: yambopy/nl/Xn_from_signal_doc.md First draft of documentation completed. Name of documentation file changed.
modified: yambopy/nl/Xn_from_signal_doc.md Minor edits
more minor edits
minor edit: fixed crossed references
minor edits
modified: yambopy/dbs/nldb.py modified: yambopy/nl/external_efield.py modified: yambopy/nl/freqmix_analysis.py modified: yambopy/nl/nl_analysis.py modified: yambopy/nl/pulse_analysis.py Corrected small bugs and added minor features
Changed default for starting sampling time to T0 - sigma.
modified: yambopy/nl/sin_analysis.py added run info at the end of output files
Added debug mode to hide comments
… nl-restructuring
modified: yambopy/nl/sin_analysis.py modified: yambopy/nl/sum_frequencies.py Minor changes, corrections and additions
myrta
added
documentation
Contributor
|
I run tests, I think there a problem in the sign of Xhi^0, probably tests should be updated? |
Member
|
Hi all, for now, I would leave these developments out of the new pseudorelease 0.7.0, is that ok? Do you want this PR added to the |
modified: yambopy/nl/freqmix_analysis.py modified: yambopy/nl/nl_analysis.py modified: yambopy/nl/pulse_analysis.py modified: yambopy/nl/sin_analysis.py Moved get_sampling as common method in nl_analysis.py and created the abstract method set_sampling to get the specific range/sampling number
extra code deleted
Upadated documentation to be consistent with changes introduced in the code
attacc
reviewed
Mar 6, 2026
Contributor
attacc
left a comment
attacc
left a comment
There was a problem hiding this comment.
Branch tested and perfectly working.
It also fixes a but in the sum-frequency extrapolation
Member
|
Dear all, thank you for all the developments! I am merging in the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I create the$P$ and current $J$ .
Xn_from_signalabstract python class and a set of derived classes for extracting nonlinear susceptibilities and conductivities from the macroscopic time-dependent polarizationThis is the start for rationalizing the nl part which has grown following need. Code which initiallly was translated from fortran ypp, was copied to a new file and modified to add new functionalities since the structure was very similar. This change is an attempt to highlight/force that structure even though keeping a file for each functionality.
Documentation is drafted in in Xn_from_signal_doc.md