Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

37 learn offer #56

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

37 learn offer #56

wants to merge 19 commits into from

Conversation

edchapman88
Copy link
Collaborator

No description provided.

sgreenbury and others added 19 commits June 28, 2023 10:41
@sgreenbury
Analysis package set-up, initial notebook
d6e01fe
@sgreenbury
Add min inventory levels for offer adjustment
0fb8c72
@sgreenbury
Merge branch 'main' into 48-analysis
21fd91c
@sgreenbury
Initial offer refactor
7923590
@sgreenbury
Revert to variants without data
61e3b8c
@sgreenbury
Use polars instead of pandas for analysis
81b7c38
@sgreenbury
Format notebook
13c4e8a
@sgreenbury
Merge branch '51-trade-min-inventory' into 48-analysis
89071cd
@sgreenbury
Add time and positions to history, serialize more efficiently
71097bb
@sgreenbury
Add trades to history
aa4ebf6
@sgreenbury
Add plot of locs visited
d336191
@edchapman88
trades determined using posted offers set by setOffer actions
2eb722b
@edchapman88
easier config settings
8dffe0a
@edchapman88
Merge branch '48-analysis' into 37-learn-offer
f15aff1
@edchapman88
pandas analysis
b5c02a2
@edchapman88
merge 52-model-load
b2620f4
@edchapman88
action sampling bug fix
234c55a
@edchapman88
disable history skipping
fbf82f0
@sgreenbury
Merge branch 'main' into 37-learn-offer
719c3ca
@edchapman88 edchapman88 linked an issue Sep 15, 2023 that may be closed by this pull request
Make the offer process learned (rather than rules-based) #37
Open
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@sgreenbury
Copy link
Collaborator

  • will take a look at the analysis notebook in a bit more detail and serializing the None differently so we can use polars!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make the offer process learned (rather than rules-based)
2 participants
@edchapman88 @sgreenbury