Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

FB-005: per-day context fields are actually per period [trap]

Open
#24 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
55/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
python
Domain
backend

Research direction

Read context.py around lines 97 and 218, then trace how tick dates and period-level fields are built and exposed to strategies and trades. Check the existing context tests for day boundaries; done means per-day tick index and date are available on ctx, and the affected docstrings describe their actual scope.

Written by the indexing model from the issue text.

Description

bug

Severity: trap · Source: BUGS_FASTBT.md (2026-10-09)

  • ctx.tick_index, total_ticks ("today's clock", context.py:218), is_last_tick and ticks_remaining span the whole period. A "first tick of the day" check (tick_index == 0) fires only on the period's first date.
  • ctx.trade_date, strategy.trade_date and every trade.trade_date are the period's first date.
  • DayStartContext.get_spot() ("first available tick of the day", context.py:97) returns the period's first bar.

Workaround: derive the date from the tick and keep your own per-day index.
Fix: add ctx.day_tick_index and ctx.date, and correct the docstrings.

Dominant language
Python
Stars
28
Forks
17
Avg merge
8h 50m
Merged PRs (30d)
2

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from uberdeveloper/fastbt

All issues in uberdeveloper/fastbt

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.