PVLib ForecastModel class

[aashishyadavally]

@Anirudh-Kakarlapudi , @Jayant1234 :

Considering we are using, from einstein.models import pvlib, which in turn uses 'from pvlib.forecast import ForecastModel' - the problem is that pvlib currently says the ForecastModel API that we are importing will need maintenance, as they are planning to move the whole API into an io.

Thus, each time we run the module, the ForecastModel warning pops up. To avoid this, I was thinking I will move the from einstein.models import pvlib import into the --model=pvlib switch-case construct in the __main__.py script, which will ensure that the warning pops up only when the user chooses to run the PVLib models through einstein.

How do you think?

[Jayant1234]

It will definitely improve the project's professional packaging. Nice consideration.

[aashishyadavally]

Sure. Will go ahead and do that! Thanks!