joaopm33 / fundspy Goto Github PK

Put default parameter as none and initializing a empty list inside the scope of the functions. This will make the api uglier, but defining the empty list (mutable object) in the arguments of the function makes it a global variable (initialized only once). This can generate bugs.

Make possible for users to download all needed libraries in one simple call.

notebook with code examples on how to use the functions.

investpy (investing.com) only provides data for the index starting from 2008. CVM data starts in 2005.

Maybe theres a feasable ibovespa free source with data from earlier on.

CodeFactor found an issue: Possible SQL injection vector through string-based query construction.

It's currently on:
fundspy\fundspy.py:255-257
Commit d3e1dfc

Data from the CVM source can be corrected after the first release.

The current database updating method only extracts data with dates after the last database update (log table).

The updating method should extract all available data and then remove duplicate values (same fund, same date), keeping the last (most recent) record.

This will probably make the updating method more expensive in execution time (specially in large .db files), but improve data quality.

• Add a parameter to delimit the start and end dates to the database generator
• Add parameter do target specific funds in the database generator
  These changes will make possible for users with specific data targets to generate smaller databases according to their needs.

CodeFactor found an issue: Try, Except, Pass detected.

It's currently on:
fundspy\fundspy.py:320
Commit d3e1dfc