Skip to content

feat: added carboncode for full pipeline with logging#173

Closed
Darinochka wants to merge 3 commits intodevfrom
feat/code-carbon
Closed

feat: added carboncode for full pipeline with logging#173
Darinochka wants to merge 3 commits intodevfrom
feat/code-carbon

Conversation

@Darinochka
Copy link
Copy Markdown
Collaborator

@Darinochka Darinochka commented Mar 25, 2025

Comment on lines +169 to +172
if tracker is not None:
emissions_data = tracker.stop_task()
emissions_data_dict = json.loads(emissions_data.toJSON())
context.callback_handler.log_emissions(emissions_data_dict)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Мб логировать в end_run?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

как-то нелогинчо получается, в end_run придется делать wandb.init и завершать таску + передавать условие-аргумент track_emissions и emissions_data

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Можно просто залогировать там дополнительно, без создания отдельного рана

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

возможно действительно стоит как-то обойтись имеющимися методами у колбеков, а то интерфейсы нагружаются

Comment on lines +129 to +138
self.wandb.init(
project=self.project_name,
group=self.group,
name="emissions",
config=emissions_data,
settings=self.wandb.Settings(x_stats_sampling_interval=self.log_interval_time),
)

self.wandb.log(emissions_data)
self.wandb.finish()
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

я посмотрел wandb логи которые ты прикрепила, мне кажется надо бы как-то отделить эти логи в отдельную подгруппу, чтобы они не смешивались с основными логами

tracker = None
if context.logging_config.track_emissions:
tracker = OfflineEmissionsTracker(
country_iso_code=context.logging_config.country_iso_code or "USA",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

а нету russia?)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Мб просто на EmissionTracker с интернетом?

@voorhs
Copy link
Copy Markdown
Collaborator

voorhs commented Mar 26, 2025

предложение: передавать emissions в log_final_metrics и распределять их по разным группам чартов

@Samoed
Copy link
Copy Markdown
Member

Samoed commented Mar 26, 2025

Мб еще каждый ран логировать тоже, чтобы смотреть что самое ресурсоемкое? Думаю будет интересно посомотреть

@Darinochka
Copy link
Copy Markdown
Collaborator Author

Мб еще каждый ран логировать тоже, чтобы смотреть что самое ресурсоемкое? Думаю будет интересно посомотреть

я пробовала так сделать, но оказалось что у нас ран создается только в objective при подчсете метрик (может я чего не поняла). как будто время обучения (если оно есть, иначе просто функция .fit) не логируется. это не так?

@Darinochka
Copy link
Copy Markdown
Collaborator Author

т.е логируется только эта часть

        context.callback_handler.start_module(module_name=module_name, num=self._counter, module_kwargs=config)

        self._logger.debug("Scoring %s module...", module_name)
        all_metrics = module.score(context, metrics=self.metrics)
        target_metric = all_metrics[self.target_metric]

        context.callback_handler.log_metrics(all_metrics)
        context.callback_handler.end_module()

поэтому я подумала как будто логировать весь пайплайн лучше, больше функций будут захвачены

@Darinochka Darinochka closed this Mar 30, 2025
@Darinochka Darinochka deleted the feat/code-carbon branch March 30, 2025 12:29
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.

3 participants