Skip to content

Clean up print statements and maybe use logging package #1050

@rickecon

Description

@rickecon

We should explore cleaning up what gets printing during an OG-Core run and how it gets printed.

  1. Currently, the steady-state GE loop errors show up as np.float64(). We should format these output to show fewer decimals and to not show the np.float.
  2. Copilot gave a [nitpick] suggestion in PR Add comprehensive Dask performance benchmark tests #1049 to "Consider using logging instead of print statements for better control over output in test environments. This would allow filtering of debug output during test runs." Change line 186 in test_real_txfunc_benchmarks.py from print(f"Generated {len(data)} valid tax records for year {year}")tologging.info(f"Generated {len(data)} valid tax records for year {year}")`.
  3. Revisit the other output during a compete run and decide what is necessary and desirable.

cc: @jdebacker

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions