Skip to content

add support for time dim, fix simple bugs#3

Closed
helenqu wants to merge 6 commits intomainfrom
temporal-ae
Closed

add support for time dim, fix simple bugs#3
helenqu wants to merge 6 commits intomainfrom
temporal-ae

Conversation

@helenqu
Copy link
Collaborator

@helenqu helenqu commented Jan 29, 2025

No description provided.

dawgz.schedule(
dawgz.job(
f=partial(get_stats, cfg.dataset, args.samples),
f=partial(get_stats, cfg),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good catch! Introduced in a51de35.

# Config
cfg = compose(
config_file="./configs/train_ae.yaml",
config_file=os.path.join(os.path.dirname(__file__), "configs", "train_ae.yaml"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should not be necessary.

Comment on lines +11 to +12
with warnings.catch_warnings():
warnings.simplefilter("ignore")
Copy link
Collaborator

Choose a reason for hiding this comment

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

The action option was added to catch_warnings in Python 3.11.

https://docs.python.org/3/library/warnings.html#warnings.catch_warnings

@francois-rozet francois-rozet force-pushed the main branch 22 times, most recently from 310613a to 36977d2 Compare February 3, 2025 13:10
@francois-rozet francois-rozet force-pushed the main branch 30 times, most recently from 5b78f60 to ab805dc Compare February 9, 2025 19:54
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