Closed janash closed 9 months ago
Merging #765 (0325110) into main (23c4103) will decrease coverage by
0.08%
. Report is 22 commits behind head on main. The diff coverage is5.00%
.:exclamation: Current head 0325110 differs from pull request most recent head fee87f3. Consider uploading reports for the commit fee87f3 to get more accurate results
Looks good! Thanks!
This PR makes some edits to the
compile_values
method and adds a new dataset method calledget_properties_df
.The following changes are made to
compile_values
:compile_values
now always returns a multi-level index where the specification is the top leve index.compile_values
now has an option to unpack results from the callable, allowing more than one dataframe column to be built at a time.entry_name
may be a string or list of strings, but now also has a default value in case the user doesn't provide it.Example use:
The function
get_properties_df
is added to allow easier compilation of record properties into a dataframe. The function takes a list of properties and returns a multi-level index dataframe similar to the type returned fromcompile_values
. Under the hood, the function usescompile_values
accessing theproperties
using.get
in case property doesn't exist for a particular specification. Additionally, before the df is returned to the user any columns containing allnan
are dropped.Example use (continuing from example above)