Saransh Chopra

Results 248 comments of Saransh Chopra

> @priyanshuone6 @Saransh-cpp could one of you pick this up? basically I am trying to make a plot with a box where you can change a parameter to see how...

@tinosulzer I was able to get it up running with some minor changes. For the part where the plot updates after entering the value, do I need to pass the...

I was following this and the mentioned issue, are you looking to create a new method in Citations class to print citations in a better way? The print method (which...

Yes, after the pulling the branch it works, thank you.

I started working on this issue last year but couldn't complete it. I have pushed the local branch on my fork, it might help - https://github.com/Saransh-cpp/PyBaMM/blob/issue-1751-discharge-capacity-as-x-axis/pybamm/plotting/quick_plot.py

The [deprecat](https://github.com/mjhajharia/deprecat) package (built on top of the `deprecate` package) allows us to put deprecation warnings for arguments - https://github.com/mjhajharia/deprecat#deprecated-kwargs. Not sure how we can deprecate some particular lines of...

> but by and large i think having a central place for documentation is good. It makes discovery easy Yes, it definitely makes the navigation and discoverability easy, but we...

Cross-posting this here from the PR - --- The documentation should show up on the given link, but I think the GH pages are not enabled for `NNlib`? Could you...

I don't have very strong opinions about either documentation style as I haven't used `Metalhead` as a user or developer, but some comments - - I agree that the current...

> It's not just that page. I can search for "LSTMCell" in the Flux docs and it will completely miss the docstring at https://fluxml.ai/Flux.jl/stable/models/layers/#Flux.LSTM. Ah, I had experienced this but...