Skip to content

Research for forcasting approaches#63

Merged
BytecodeBrewer merged 2 commits into
BytecodeBrewer:developfrom
KartavyaDikshit:fix-issue-43
Jun 21, 2026
Merged

Research for forcasting approaches#63
BytecodeBrewer merged 2 commits into
BytecodeBrewer:developfrom
KartavyaDikshit:fix-issue-43

Conversation

@KartavyaDikshit

@KartavyaDikshit KartavyaDikshit commented Jun 21, 2026

Copy link
Copy Markdown

What changed?

  • add research for first forecasting approach

Related issue

Closes #43

Tests

  • I ran pytest
  • Existing tests pass
  • I added or updated tests where necessary
  • Not needed because this only affects documentation or repository setup

Checklist

  • The change is focused
  • No secrets or API keys are included
  • Documentation was updated if needed
  • CI is expected to pass

Notes

It violates the scope of the isssue as well as the contributing.md
@BytecodeBrewer BytecodeBrewer marked this pull request as draft June 21, 2026 08:04
@BytecodeBrewer BytecodeBrewer self-requested a review June 21, 2026 08:04
@BytecodeBrewer BytecodeBrewer changed the base branch from main to develop June 21, 2026 08:07
@BytecodeBrewer BytecodeBrewer changed the title Fix for issue #43 Research for forcasting approaches Jun 21, 2026
@BytecodeBrewer BytecodeBrewer marked this pull request as ready for review June 21, 2026 08:17

@BytecodeBrewer BytecodeBrewer left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KartavyaDikshit I fixed it on my side.

The research fulfills the acceptance criteria, so I removed the out-of-scope automation script and retargeted the PR to develop.

Thanks for your contribution! Just please don't work around the contributing guidelines next time. XD

@BytecodeBrewer BytecodeBrewer merged commit 8d6cbe0 into BytecodeBrewer:develop Jun 21, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Research first forecasting approach for market time series

2 participants