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

Randomize p0 for decomposition #109

Merged
merged 2 commits into from
Feb 3, 2020
Merged

Randomize p0 for decomposition #109

merged 2 commits into from
Feb 3, 2020

Conversation

fzeiser
Copy link
Collaborator

@fzeiser fzeiser commented Jan 31, 2020

Closes #100. Now we can choose between either a CT-like to BSFG-like initiallization (in addition to providing p0's directly). By default, we choose a BSFG-like initial value. This is because we often find a solution that looks CT-like; so by starting with he competitor model, we are unlikely to have received the CT-like result just due to a local minium.

In addition, the input values are by default randomized for each run (currently an initial value betwen 1/5 and 5* the proposed value of the model(s)).

Closes #100. Now we can choose between either a CT-like to BSFG-like initiallization (in addition to providing p0's directly). By default, we choose a BSFG-like initial value. This is because we often find a solution that looks CT-like; so by starting with he competitor model, we are unlikely to have received the CT-like result just due to a local minium.

In addition, the input values are by default randomized for each run (currently an initial value betwen 1/5 and 5* the proposed value of the model(s)).
@fzeiser
Copy link
Collaborator Author

fzeiser commented Jan 31, 2020

As for now there is a problem with the choice of the boundaries for the differential evolution, now that we have randomized the input conditions. They should be chosen depending on how the nld / gsf look like before normalization. I get one case now that deviates quite a bit from the others, and with the current automatically set boundaries is lies at ~1/1000 of the absolute values of the rest.

@fzeiser
Copy link
Collaborator Author

fzeiser commented Jan 31, 2020

Rerunning the notebook might already have solved the problem...

- Extending the bounds was necessary after chaning the initial conditions
- Added "parabola" as a 3rd option in extractor
@fzeiser fzeiser merged commit 77dec9d into master Feb 3, 2020
@fzeiser fzeiser deleted the dev/randomize_p0 branch June 19, 2020 10:12
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 this pull request may close these issues.

Compare different choices of p0 for decomposition
1 participant