Spencer Smith

Results 226 comments of Spencer Smith

Thanks for the update @cd155. Eventually, the initial values should be added as inputs in the SRS. We should also read these values from a file, rather than hard-code them...

@samm82 does the change in #3610 break the generated code, or are you saying that you made an additional change that broke the code? How integrated is the code generation...

Thanks for the explanation @samm82. The fact that the IMs (actually the requirements) aren't connected to the code generator is a long-term thing to fix, but for now at least,...

@samm82, yes it looks like the code you provided would work to get both theta_1 and theta_2. It took me a long time to confirm that though because the code...

@daijingz you might not be looking in the right folder. You should be able to find the correct paper in the [Papers folder](https://github.com/JacquesCarette/Drasil/tree/master/Papers).

@harmanpreet-sagar, thank you for creating this issue. Yes, we definitely should look into this and generate the Jupyter notebook without the extra spaces.

I feel like underscores are okay in proper nouns. The name of the software might have underscores, and that would be a proper noun. I feel like underscores go in...

Looking at [IM:heatEInWtr](https://jacquescarette.github.io/Drasil/examples/swhs/SRS/srs/SWHS_SRS.html#Sec:IMs) the equation for Ew shows Ew(t). For the IM, we could definite Ew for an instant of time, but as of right now it is a mix...

I'm definitely in favour of as much automation as possible. If scraping the equation for the description field is working already, I think you are right that we can also...

Once we get `flake8` working, we could look at `pylint`. From some ad hoc checks on the Drasil code, it looks like `pylint` uncovers more areas for investigation than `flake8`.