Feature add jsonargparse support

When using the LightningCLI feature that uses jsonargparse the parameters are not captured automatically. I have attached a file with the code to reproduce (discussed with @bmartinn). lightning_cli.zip

[mauvilsa]

@eugen-ajechiloae-clearml certainly. Feel free to tag me if you want me to review.

[jkhenning]

Hey @thepycoder!

Hi, I would like to try and take this up 😃

🙏 👍

Since jsonargparse is so similar to argparse, should I try and add support for it to the existing argparse bind or create a new, specific bind for it just like what was done for click?Since jsonargparse is so similar to argparse, should I try and add support for it to the existing argparse bind or create a new, specific bind for it just like what was done for click?

Well, it depends - if you think you can get a clean implementation that will be much simpler, we won’t object 🙂 - but I think the reason the argparse binding looks much more complex is because it is - there are much more things to handle and edge-cases (and bugs) that were addressed - personally I would start by trying to figure out the similarities and differences, to get a better understanding of what can be reused and/or extended.

Also the code in argparse is barely commented, so someone to guide the way just a little would be much appreciated! 😃

Sure 🙂 - we’ll be super happy to help, explain and guide. You’re also welcome to ping us in the Slack clearml-dev channel (for quicker answers to simple questions - I still thing larger discussions are better off done here, for reference-sake)