Skip to content

DM-51037 : Add ssp_balanced_index, loadBalanceLinkageBundles #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 2, 2025

Conversation

Gerenjie
Copy link
Contributor

No description provided.

@Gerenjie Gerenjie requested a review from enourbakhsh May 22, 2025 22:54
@Gerenjie Gerenjie force-pushed the tickets/DM-51037 branch from 2dfb055 to 91d758a Compare May 23, 2025 20:26
Copy link
Contributor

@enourbakhsh enourbakhsh left a comment

Choose a reason for hiding this comment

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

Consider adding __all__ in the loadBalanceLinkageBundles.py. I have one more comment below.

import astropy.units as u
from datetime import datetime

import lsst.pipe.base.connectionTypes as connTypes
Copy link
Contributor

Choose a reason for hiding this comment

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

You already imported connectionTypes from lsst.pipe.base above, and also have import lsst.pipe.base.connectionTypes as cT below. If there are additional similar cases as well, please organize and consolidate them.

@Gerenjie Gerenjie force-pushed the tickets/DM-51037 branch from d914be8 to 9c6aff2 Compare June 2, 2025 16:07
@Gerenjie Gerenjie merged commit 0e4febb into main Jun 2, 2025
2 checks passed
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.

2 participants