Some new changes incoming :). This fixes the last things from the original issue.
GENERAL
I ran the automatic styling using the styler package. Styling seems to confuse GitHub's diffing, so a bit hard to see :. Sorry about that.
DESCRIPTION
Thanks so much for adding me as an author! That's awesome 😊 ! Realised that you'd only listed yourself as creator, so I added an author tag to your name too as done here
Added a full stop to the Description to make the styling happy.
nls_multstart.R
Added start_upper and start_lower arguments.
Set a warning about the limits being missing instead of a prompt, since it was a little finnicky with its being two vectors. Though actually I think I solved this issue in the end, so I suppose this could be added again. UPDATE: fixed this so that it's like it was.
Changed the colname/name differentiation for data.frames or environments. Data.frames are special kinds of lists, so it works just the same but with no special check. And then it means that people can input lists of data too if they so desire.
README
I updated the README file input arguments. However, when I output to .md, it gets errors about using Helvetica since I'm on Windows. So maybe you should just knit the README.Rmd again. Also, I got strange issues about stuff being deprecated when it called on broom::augment. Though actually, thinking about it now, it might be that I should update broom. In any case, I added that warnings shouldn't be displayed in those code blocks.
Summary
This is the last of the initial things I mentioned in the issue. I'll try implementing this in my package next :). Excited to see what comes of it!
Some new changes incoming :). This fixes the last things from the original issue.
GENERAL
DESCRIPTION
nls_multstart.R
README
I updated the README file input arguments. However, when I output to .md, it gets errors about using Helvetica since I'm on Windows. So maybe you should just knit the README.Rmd again. Also, I got strange issues about stuff being deprecated when it called on broom::augment. Though actually, thinking about it now, it might be that I should update broom. In any case, I added that warnings shouldn't be displayed in those code blocks.
Summary
This is the last of the initial things I mentioned in the issue. I'll try implementing this in my package next :). Excited to see what comes of it!