Rahul Agrawal

Results 19 comments of Rahul Agrawal

Hey @jpfeuffer, It's pointing to the license files present in the repo -> https://github.com/OpenMS/OpenMS/blob/develop/License.txt & https://github.com/OpenMS/OpenMS/blob/develop/ I can see that no change are made in these files from a long...

Hey @mfelsche, This project seems very interesting to me so I will like to work on this as part of the LFX'21 Mentorship program. Thank you.

@jpfeuffer @axelwalter can we merge the PR now?

> Was this all tested anyhow? I think actually the bigger problem are the overloads. They will destroy those nice docstrings anyway. Yes I tested it in my local machine.

- inferPosteriorProbabilities: Earlier -> ![Screenshot from 2022-10-12 21-54-05](https://user-images.githubusercontent.com/41531498/195397781-d3584b61-da5c-458c-beff-589503b2ec3e.png) Now -> ![Screenshot from 2022-10-12 21-54-39](https://user-images.githubusercontent.com/41531498/195397883-d4e7c9cd-bf25-434a-abe5-c88b53c32478.png)

Yes it does. I also added newline after the return statement. ![Screenshot from 2022-10-13 01-58-05](https://user-images.githubusercontent.com/41531498/195441384-c2c4d0f7-e82f-4ff6-98dc-4c7d89f6863a.png)

@jpfeuffer @axelwalter waiting for your review.

Hey @jpfeuffer Is it fine if I raise a single PR fixing the indentation issues mentioned [here](https://github.com/OpenMS/OpenMS/pull/6358#discussion_r1001679714), once the pending PR's are merged?

@TarekAS which metrices did you used to set the Cloudwatch Alarm?

Facing the same issue too.