-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature noise lf pulse #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make the minor changes and take a look at the changes in TdsModel, which I believe are not necessary.
TdsModel.h
Outdated
@@ -297,6 +297,9 @@ class TdsModel | |||
void getSectionFlow(int sectionIndex, double &inflow, double &outflow); | |||
double getSectionPressure(int sectionIndex); | |||
|
|||
void calcNoiseSample(NoiseSource* s, double ampThreshold); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does this need to be public?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was to be able to use the noise source function already implemented, but it turns out that this function is too complex for what we need so I found it simpler to just generate random numbers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, then I'd say just make it private again.
TdsModel.h
Outdated
@@ -297,6 +297,9 @@ class TdsModel | |||
void getSectionFlow(int sectionIndex, double &inflow, double &outflow); | |||
double getSectionPressure(int sectionIndex); | |||
|
|||
void calcNoiseSample(NoiseSource* s, double ampThreshold); | |||
void incrementPosition() { position++; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this function needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as before, this was to use the noise function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, so we can just remove it, then.
No description provided.