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.

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

feat: add collect info #233

Merged
merged 31 commits into from
Sep 7, 2024
Merged

feat: add collect info #233

merged 31 commits into from
Sep 7, 2024

Conversation

SunsetWolf
Copy link
Collaborator

@SunsetWolf SunsetWolf commented Aug 29, 2024

Description

Motivation and Context

How Has This Been Tested?

  • Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
  • If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

  1. Pipeline test:
  2. Your own tests:

Types of changes

  • Fix bugs
  • Add new feature
  • Update documentation

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

@@ -27,7 +27,9 @@ Steps to reproduce the behavior:
<!-- A screenshot of the error message or anything shouldn't appear-->

## Environment
<!-- TODO: We may provide a more automatic way to get the information -->

**Note**: User could run `rdagent collect_info` under project directory to get system information and paste them here directly.
Copy link
Contributor

Choose a reason for hiding this comment

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

under project directory?

Copy link
Contributor

Choose a reason for hiding this comment

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

pip install

rdagent/app/cli.py Outdated Show resolved Hide resolved
@@ -37,6 +46,58 @@ def ui(port=80, log_dir="", debug=False):
subprocess.run(cmds)


def find_project_root():
Copy link
Contributor

Choose a reason for hiding this comment

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

find the file via Path(__file__).resolve().parent.parent

["Name of current operating system: ", "system"],
["Processor architecture: ", "machine"],
["System, version, and hardware information: ", "platform"],
["Version number of the system: ", "version"],
Copy link
Contributor

Choose a reason for hiding this comment

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

Docker setting

root_dir = find_project_root()
current_version = get_version(root=root_dir)
logger.info(f"RD-Agent version: {current_version.split('+')[0]}")
package_list = [line for file in root_dir.joinpath("requirements").rglob("*") for line in open(file)]
Copy link
Contributor

Choose a reason for hiding this comment

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

check it works if it is a package installed by pip install

Copy link
Contributor

Choose a reason for hiding this comment

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

maybe pip provides similar feature to get dependency

<!-- TODO: We may provide a more automatic way to get the information -->

**Note**: Users can run `rdagent collect_info` to get system information and paste it directly here.

- RD-Agent version:
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be changed now

SunsetWolf and others added 9 commits September 6, 2024 09:14
* fix package

* fix lint
* update cross validation for kaggle scenario

* CI Issues

* delete useless file

* CI issues
* init a scenario for kaggle feature engineering

* update the readme

* Delete rdagent/app/kaggle_feature/conf.py

* update some pictures

* Delete rdagent/app/kaggle_feature/model.py

* change a photo

* add pics to docs

* update the readme

* update the README

* for a try

* for another try

* change the style of the pictures in readme

* fix a small bug

* update the readme and the docs

* update the docs

* fix a typo

* change a website url

* change some styles

* fix a typo

* change the size of the logo

* change two urls

* Update README.md

* Update README.md

* Update README.md

* Update README.md

---------

Co-authored-by: Linlang <[email protected]>
root_dir = Path(__file__).resolve().parent.parent.parent
current_version = get_version(root=root_dir)
logger.info(f"RD-Agent version: {current_version.split('+')[0]}")
package_list = [
Copy link
Contributor

Choose a reason for hiding this comment

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

update package

logger.info(f"OS of the last run is: {last_image.attrs['Os']}")
else:
logger.info(f"No images.")
containers = client.containers.list(all=True)
Copy link
Contributor

Choose a reason for hiding this comment

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

latest container

else:
logger.info(f"No images.")
containers = client.containers.list(all=True)
last_container = containers[0]
Copy link
Contributor

Choose a reason for hiding this comment

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

index error

@SunsetWolf SunsetWolf merged commit 89f4af9 into main Sep 7, 2024
7 checks passed
@SunsetWolf SunsetWolf deleted the add_collect_info branch September 7, 2024 04:37
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.

6 participants