Skip to content

fix: fix combined_factors_df.pkl not loading in docker#697

Merged
you-n-g merged 3 commits intomainfrom
fix_load_combined_factors_df_error
Mar 18, 2025
Merged

fix: fix combined_factors_df.pkl not loading in docker#697
you-n-g merged 3 commits intomainfrom
fix_load_combined_factors_df_error

Conversation

@SunsetWolf
Copy link
Collaborator

@SunsetWolf SunsetWolf commented Mar 18, 2025

Description

Motivation and Context

How Has This Been Tested?

  • If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

  1. Your own tests:

Types of changes

  • Fix bugs
  • Add new feature
  • Update documentation

📚 Documentation preview 📚: https://RDAgent--697.org.readthedocs.build/en/697/

@SunsetWolf SunsetWolf changed the title fix: Fix combined_factors_df.pkl not loading in docker fix: fix combined_factors_df.pkl not loading in docker Mar 18, 2025
combined_factors = combined_factors.loc[:, ~combined_factors.columns.duplicated(keep="last")]
new_columns = pd.MultiIndex.from_product([["feature"], combined_factors.columns])
combined_factors.columns = new_columns
# Due to the rdagent and qlib docker image in the numpy version of the difference,
Copy link
Contributor

Choose a reason for hiding this comment

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

raise code error here.
image

@you-n-g you-n-g merged commit 3984b99 into main Mar 18, 2025
8 checks passed
@you-n-g you-n-g deleted the fix_load_combined_factors_df_error branch March 18, 2025 10:36
Jensen246 pushed a commit to Jensen246/RD-Agent that referenced this pull request Jun 6, 2025
* Fix combined_factors_df.pkl not loading in docker

* change combined factors file type

* change qlib commit id
licong01-cloud pushed a commit to licong01-cloud/RD-Agent that referenced this pull request Dec 13, 2025
* Fix combined_factors_df.pkl not loading in docker

* change combined factors file type

* change qlib commit id
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.

2 participants