Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Refactor Aggregate class to Summary for better clarity and consistency
Browse files Browse the repository at this point in the history
rcboufleur committed May 27, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1 parent 60f3d1d commit dea6f5f
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -3,7 +3,7 @@
import numpy


class Aggregate:
class Summary:
"""
A class that performs various transformations on a list or numpy
array of values.
4 changes: 2 additions & 2 deletions python/lsst/consdb/efd_transform/transform.py
Original file line number Diff line number Diff line change
@@ -7,7 +7,7 @@
import pandas
from dao.butler import ButlerDao
from dao.exposure_efd import ExposureEfdDao
from efd_transform.aggregate import Aggregate
from efd_transform.summary import Summary
from lsst.daf.butler import Butler

# from sqlalchemy import Engine
@@ -173,7 +173,7 @@ def proccess_column_value(

values = self.concatenate_arrays(topics_values)

column_value = Aggregate(values).apply(transform_function)
column_value = Summary(values).apply(transform_function)

return column_value

0 comments on commit dea6f5f

Please sign in to comment.