Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

✨ Export trades from hyperopt/backtest result #188

Conversation

raftersvk
Copy link
Contributor

No description provided.

@Rikj000 Rikj000 changed the title Export trades from hyperopt/backtest result ✨ Export trades from hyperopt/backtest result Nov 8, 2021
@Rikj000 Rikj000 self-requested a review November 8, 2021 23:24
@Rikj000 Rikj000 added Feature - New New feature or pull request Needs Testing This feature, improvement or bugfix still is in need of testing Planned Planned feature, improvement or bugfix (not being worked on yet) labels Nov 8, 2021
@Rikj000
Copy link
Owner

Rikj000 commented Nov 8, 2021

@raftersvk could you please add some example result files to help visualize what this PR will do when implemented into mgm-hurry? 😄

@Rikj000
Copy link
Owner

Rikj000 commented Nov 8, 2021

Also for mgm-hurry implementation,
the export_csv() function can act as a example/template to implement these new functions 🙂

@Rikj000 Rikj000 marked this pull request as draft December 16, 2021 17:17
@raftersvk raftersvk closed this Jan 2, 2022
@raftersvk raftersvk deleted the Export-trades-from-Hypertop/Backtest branch January 2, 2022 14:48
@Rikj000
Copy link
Owner

Rikj000 commented Jan 2, 2022

@raftersvk why did you close this PR? 🤔
It is still marked as planned?

@raftersvk raftersvk restored the Export-trades-from-Hypertop/Backtest branch January 5, 2022 20:17
@raftersvk
Copy link
Contributor Author

sorry mistake while doing some cleanup in my dev branch

@raftersvk raftersvk reopened this Jan 5, 2022
@raftersvk
Copy link
Contributor Author

Here is an example of result file with all detailed trades from a given epoch.
It helps understanding where from and when the profit or drawdown occurs instead of having only macro and cumulative indicators.
this is only for "research purposes" and therefore not sure it will benefit to most of the users ;)

backtest-result-2021-11-12_09-46-24_trades.csv

@Rikj000
Copy link
Owner

Rikj000 commented Jan 5, 2022

Thank you for re-opening the PR & for uploading the example results file @raftersvk 🦾
I do truly see the usefulness of this feature though,
I think it truly can come in handy to get a deeper comprehension about our test results! 😄

However I'll keep this PR a draft until you, me or another collaborator has the time to implement it in mgm-hurry

@Rikj000 Rikj000 added this to the v1.0.0 milestone Jan 17, 2022
@raftersvk raftersvk closed this Jan 20, 2022
@raftersvk raftersvk deleted the Export-trades-from-Hypertop/Backtest branch January 20, 2022 17:35
@Rikj000 Rikj000 restored the Export-trades-from-Hypertop/Backtest branch January 20, 2022 18:05
@Rikj000 Rikj000 reopened this Jan 20, 2022
@Rikj000 Rikj000 closed this Jan 20, 2022
@Rikj000 Rikj000 deleted the Export-trades-from-Hypertop/Backtest branch January 20, 2022 18:06
@Rikj000
Copy link
Owner

Rikj000 commented Jan 20, 2022

See PR: #245

@Rikj000 Rikj000 removed Planned Planned feature, improvement or bugfix (not being worked on yet) Needs Testing This feature, improvement or bugfix still is in need of testing labels Jan 20, 2022
@Rikj000 Rikj000 linked an issue Jan 20, 2022 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature - New New feature or pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ Implement mgm-hurry export_trades
2 participants