FlexMeasures / flexmeasures / 10781652022

77%

Build Type

push

github

Committed by web-flow

Commit Message

Speed up assets and accounts pages (#988)

* Testing doing 1 single request instead of one request per account. This relies on the fact that the index of the internal API already take care of auth. Running agains the simulation database, it takes 52s to load the asset page on average.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* check permissions

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: harmless, but redundant plus

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: no dumb children

Signed-off-by: F.N. Claessen <felix@seita.nl>

* feature: create Asset objects faster by selecting from db, while still using internal API

Signed-off-by: F.N. Claessen <felix@seita.nl>

* style: black

Signed-off-by: F.N. Claessen <felix@seita.nl>

* add public assets to the call and fix tests

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add future

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add `all_accessible` to get all the accessible assets by a certain account

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* Improve docstring.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* raise if the account is provided and all_accessible=True

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add owner, sensors and generic_asset_type to Asset schema

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* avoid dumping all the children fields (e.g. child_assets). this avoid recursive calls of GenericAssetSchema

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add number_of_assets and number_of_users  properties to Account class

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add pagination

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add annotations

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add number of records

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: ... (continued)

Run Details

6168 of 10008 branches covered (61.63%)

Branch coverage included in aggregate %.

85 of 117 new or added lines in 11 files covered. (72.65%)

7 existing lines in 2 files now uncovered.

7810 of 10754 relevant lines covered (72.62%)

5.75 hits per line