Merged
Conversation
Contributor
|
Hi @abcHubHH Thanks for the submission. One thing on a quick check is that parameters such as sigma_base etc. affect the results so this will be over the limit (see the README for what counts as a parameter). David |
Contributor
Author
|
Hi David, I described the parameters in the README. I hope this helps with your decision. My model uses at most three parameters, and I believe I could reduce this to a single parameter as well. Best Aleh |
Contributor
|
Hi Aleh,
As stated in the README the aim is to match the Figure 1 which has a particular minimum sigma and offset. To get the required accuracy, any model therefore needs two parameters to start with. Your model has a minimum sigma of near-zero, and an offset of zero, so these need to be adjusted. Changing numbers like p_exec or U_scale changes the result so count as parameters (see README).
Thanks,
David
On Jan 6, 2026, at 8:51 AM, abcHubHH ***@***.***> wrote:
[https://avatars.githubusercontent.com/u/183022323?s=20&v=4]abcHubHH left a comment (q-variance/challenge#19)<#19 (comment)>
Hi David, I described the parameters in the README. I hope this helps with your decision. My model uses at most three parameters, and I believe I could reduce this to a single parameter as well. Best Aleh
—
Reply to this email directly, view it on GitHub<#19 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AEBN4KLVEXPYKLNP25L4MP34FO4VTAVCNFSM6AAAAACQOQG3KGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTOMJUG43TINJZHA>.
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
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
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.
Submission: AURA