timeseries returns pct_change which is obsolete #27

New issue

Closed

opened 2026-03-04 20:20:12 +00:00 by fabioth · 3 comments

fabioth commented 2026-03-04 20:20:12 +00:00 (Migrated from gitlab.esvedra.one)

Copy link

pct_change in output is obsolete

`pct_change` in output is obsolete

fabioth commented 2026-03-05 07:25:28 +00:00 (Migrated from gitlab.esvedra.one)

Copy link

changed type from task to issue

changed type from task to issue

gertjan commented 2026-03-05 11:49:48 +00:00 (Migrated from gitlab.esvedra.one)

Copy link

mentioned in commit 0ae1847e08

mentioned in commit 0ae1847e0889cd8c2eff599c41381a6898ffc6b8

project_1_bot_053fc026be1655bf3cd800152d0cdead commented 2026-03-05 11:51:47 +00:00 (Migrated from gitlab.esvedra.one)

Copy link

Fixed in commit 0ae1847. Removed the unused pct_change field from PriceBar model in src/models/timeseries.py. This field was never populated from the database and always returned None. The returns field already provides the actual return values.

Fixed in commit 0ae1847. Removed the unused pct_change field from PriceBar model in src/models/timeseries.py. This field was never populated from the database and always returned None. The returns field already provides the actual return values.

project_1_bot_053fc026be1655bf3cd800152d0cdead (Migrated from gitlab.esvedra.one) closed this issue 2026-03-05 11:51:52 +00:00

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

0.17.2

Labels

Clear labels   

bug

  

documentation

  

duplicate

  

enhancement

  

wontfix

No labels

bug

documentation

duplicate

enhancement

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

gertjan/bodega#27

No description provided.