Skip to content

Conversation

prateekdesai04
Copy link
Contributor

@prateekdesai04 prateekdesai04 commented Mar 12, 2024

*Issue #, if available:*#3952

Description of changes:
Follow up PR to #3962, this is to create a table of package versions installed and comparing them with previous CI run.
The table is then displayed as a GitHub comment on the PR.
Example: https://github.com/prateekdesai04/autogluon/pull/52#issuecomment-1984495579

Referred official GitHub docs for table creation and formatting: https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/organizing-information-with-tables

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link

Job PR-3972-4787606 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-3972/4787606/index.html

@tonyhoo tonyhoo self-assigned this Mar 13, 2024
for key in "${!prev_packages[@]}" "${!curr_packages[@]}"; do
prev="${key}=${prev_packages[$key]}"
curr="${key}=${curr_packages[$key]}"
if [[ -z ${prev_packages[$key]} ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

For the sake of compressed output, we shall limit the output to only packages with version diff with 3 columns ["package_name", "previous_version", "current_version"]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently we have only 2 columns, do you suggest increasing it to 3?
The readability may be a little difficult or confusing.
The current PR makes it look like this: https://github.com/prateekdesai04/autogluon/pull/52#issuecomment-1984495579

@prateekdesai04 prateekdesai04 merged commit 9df1126 into autogluon:master Mar 18, 2024
ddelange added a commit to ddelange/autogluon that referenced this pull request Mar 21, 2024
…tch-4

* 'master' of https://github.com/awslabs/autogluon: (46 commits)
  [core] move transformers to setup_utils, bump dependency version (autogluon#3984)
  [AutoMM] Fix one lightning upgrade issue (autogluon#3991)
  [CI][Feature] Create a package version table (autogluon#3972)
  [v.1.1][Upgrade] PyTorch 2.1 and CUDA 12.1 upgrade (autogluon#3982)
  [WIP] Code implementation of Conv-LoRA (autogluon#3933)
  [timeseries] Ensure that all metrics handle missing values in the target (autogluon#3966)
  [timeseries] Fix path and device bugs (autogluon#3979)
  [AutoMM]Remove grounding-dino (autogluon#3974)
  [Docs] Update install modules content (autogluon#3976)
  Add note on pd.to_datetime (autogluon#3975)
  [AutoMM] Improve DINO performance (autogluon#3970)
  Minor correction in differ to pick correct environment (autogluon#3968)
  Fix windows python 3.11 issue by removing ray (autogluon#3956)
  [CI][Feature] Package Version Comparator (autogluon#3962)
  [timeseries] Add support for categorical covariates (autogluon#3874)
  [timeseries] Add method for plotting forecasts (autogluon#3889)
  Update conf.py copyright to reflect current year (autogluon#3932)
  [Timeseries][CI]Refactor CI to skip AutoMM and Tabular tests w.r.t timeseries changes (autogluon#3942)
  Fix HPO crash in memory check (autogluon#3931)
  [AutoMM][CI] Capping scikit-learn to avoid HPO test failure (autogluon#3947)
  ...
LennartPurucker pushed a commit to LennartPurucker/autogluon that referenced this pull request Jun 1, 2024
Co-authored-by: Ubuntu <ubuntu@ip-172-31-9-154.us-west-2.compute.internal>
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