Skip to content

Conversation

@lerit
Copy link
Contributor

@lerit lerit commented Aug 25, 2022

position is a subproperty of _value,cash and now_account_value belongs to _value.position
maybe error since:#1120

position is s sub attribute of _value
error since commit(id:89972f6c6f9fa629b4f74093d4ba1e93c9f7a5e5)
@you-n-g
Copy link
Collaborator

you-n-g commented Aug 29, 2022

Thanks for your contributions!

Please fix the CI error.

@you-n-g you-n-g requested a review from SunsetWolf August 29, 2022 09:59
@you-n-g
Copy link
Collaborator

you-n-g commented Aug 29, 2022

Thanks for your contribution.
This reflex another flaw of our CI that it can't detect the errors in the report.
So I think it would be better to add such a check in our CI. (It may be some command like jupyter nbconvert workflow_by_code.ipynb)

Would you like to contribute to this feature in this PR?

Otherwise, @SunsetWolf will help with this feature.

@lerit
Copy link
Contributor Author

lerit commented Aug 30, 2022

I have fix the CI error.
@you-n-g

@you-n-g
Copy link
Collaborator

you-n-g commented Nov 8, 2022

It looks great.
Thanks for your contribution.

@you-n-g you-n-g added the bug Something isn't working label Dec 9, 2022
qianyun210603 pushed a commit to qianyun210603/qlib that referenced this pull request Mar 23, 2023
* fix position access error

position is s sub attribute of _value
error since commit(id:cf48cc94d0422e6354a08765b4176552442987d5)

* lint with blank
qianyun210603 pushed a commit to qianyun210603/qlib that referenced this pull request Mar 23, 2023
* fix position access error

position is s sub attribute of _value
error since commit(id:89972f6c6f9fa629b4f74093d4ba1e93c9f7a5e5)

* lint with blank
you-n-g added a commit to you-n-g/qlib that referenced this pull request Oct 31, 2025
* add simple rag mcp

* add rag_agent in expGen v2

* add conf config for research rag

* fix CI

* refactor: move context7 and rag config files to new conf modules

* make rag agent general

* fix CI

---------

Co-authored-by: Young <afe.young@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants