Skip to content

fix: Autolog Mechanics#99

Merged
monoxgas merged 4 commits into
mainfrom
fix/autolog-updates
Jul 12, 2025
Merged

fix: Autolog Mechanics#99
monoxgas merged 4 commits into
mainfrom
fix/autolog-updates

Conversation

@monoxgas
Copy link
Copy Markdown
Contributor

@monoxgas monoxgas commented Jul 7, 2025

  • Rather than globally disable autologging by default, targeting some explicit improvements to surrounding behaviors.
  • In general, updating logging mechanics to apply more sane defaults when in comes to task inputs, outputs, params, etc.

Goal is to avoid useless noise being created in the UI as much as possible.

Added:

  • New seems_useful_to_serialize mechanic we can use for autologging task inputs and outputs

Changed:

  • Made logging of task execution metrics an explicit toggle that doesn't change with autologging
  • Applied some sanity checks for task inputs/outputs when autologging is being used.

Removed:

  • Stripped the ability for tasks to have parameters logged to them
    • Not a useful feature for most cases and a foot gun
    • Users might as well just use inputs for tasks
    • Parameters are best suited for the run level IMO

Generated Summary:

  • Simplified the log_param and log_params functions by removing the to parameter, making them focus solely on logging parameters as key-value pairs.
  • Introduced a new helper function, seems_useful_to_serialize, that checks if an object is likely useful for serialization based on its schema.
  • Added a new parameter, log_execution_metrics, to various functions and classes to allow optional tracking of execution metrics, including success rates and run counts.
  • Removed the log_params parameter from the Task class to streamline the logging process and improve clarity.
  • Updated documentation to reflect changes, ensuring consistency and accuracy regarding parameter logging functionality.
  • Consolidated logic in run methods to ensure that input serialization checks are applied based on the log_inputs option.
  • Overall, these modifications enhance code readability and maintainability while introducing useful features for performance tracking.

This summary was generated with ❤️ by rigging

monoxgas added 2 commits July 7, 2025 14:53
…or task inputs when autologging. Don't autolog None outputs. Make execution metric logging explicit.
@dreadnode-renovate-bot dreadnode-renovate-bot Bot added area/docs Changes to documentation and guides type/docs Documentation updates and improvements labels Jul 7, 2025
@monoxgas monoxgas merged commit 4886153 into main Jul 12, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Changes to documentation and guides type/docs Documentation updates and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant