Skip to content

Conversation

@Darinochka
Copy link
Collaborator

@Darinochka Darinochka commented Mar 19, 2025

  • добавила аргумент в logging_config log_interval_time
  • не стала добавлять в init так непонятно как юзер сможет настроить этот аргумент, если он будет в init. кажется все аргументы инициализируются в start_run, поэтому я добавила его туда

@Darinochka Darinochka requested review from Samoed and voorhs March 19, 2025 16:40
None, description="List of callbacks to report to. If None, no callbacks will be used"
)
log_interval_time: float = Field(
0.1, description="Sampling interval for the system monitor in seconds for Wandb logger."
Copy link
Member

Choose a reason for hiding this comment

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

Это в чем измеряется?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

in seconds. ну я поставила 0.1 от секунды получаеся

Comment on lines 34 to 45
def start_run(self, run_name: str, dirpath: Path, log_interval_time: float) -> None:
"""Starts a new run and sets the directory for storing logs.
Args:
run_name: Name of the run.
dirpath: Path to the directory where logs will be saved.
log_interval_time: Sampling interval for the system monitor in seconds.
(not utilized in Tensorboard logging).
"""
self.run_name = run_name
self.dirpath = dirpath
self.log_interval_time = log_interval_time
Copy link
Collaborator

Choose a reason for hiding this comment

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

если в тензорборде не испольуется то тогда мб сделать float | None = None

Copy link
Member

Choose a reason for hiding this comment

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

Или просто чтобы логирование могло прнимать доп параметры, а потом каждый логер сам отдельно их обрабатывать

@Darinochka Darinochka merged commit 931bd04 into dev Mar 22, 2025
32 of 35 checks passed
@Darinochka Darinochka deleted the feat/wandb-profiler branch March 22, 2025 11:00
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.

4 participants