Closed saketkc closed 2 years ago
Hi Saket, this looks great.
Could you write a short paragraph (one or two sentences with examples or motivation) in each change/addition to the package (when it is not self-explanatory)? This would go into the news section of the README and the NOTES file.
I'll try to have a look at all the detail in this PR fairly soon and then make it part of release 0.3.3 in mid July
Sorry to barge in, but I've been using the v2 model. With the latest changes is there a reason n_cells is fully overridden to 2000 cells? Having the default or otherwise set by the user as it was is fine?
Thanks for pointing it out @cnk113, I will fix it.