Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| """ | ||
| from alembic import op | ||
| import sqlalchemy as sa |
Check warning
Code scanning / Prospector (reported by Codacy)
Unable to import 'sqlalchemy' (import-error) Warning
| flux_type: Annotated[ | ||
| FluxType, | ||
| typer.Option( | ||
| "--flux-type", help="Flux type for the flux standard star catalog." |
Check warning
Code scanning / Prospector (reported by Codacy)
undefined name 'flux' (F821) Warning
| flux_type: Annotated[ | ||
| FluxType, | ||
| typer.Option( | ||
| "--flux-type", help="Flux type for the flux standard star catalog." |
Check warning
Code scanning / Prospector (reported by Codacy)
syntax error in forward annotation 'Flux type for the flux standard star catalog.' (F722) Warning
| flux_type: Annotated[ | ||
| FluxType, | ||
| typer.Option( | ||
| "--flux-type", help="Flux type for the flux standard star catalog." |
Check warning
Code scanning / Prospector (reported by Codacy)
undefined name 'flux' (F821) Warning
There was a problem hiding this comment.
Pylintpython3 (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
There was a problem hiding this comment.
Pylint (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
There was a problem hiding this comment.
Remark-lint (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
| @@ -0,0 +1,50 @@ | |||
| """add total fluxes | |||
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Module name "20250505-162126_fc8ea63b8c9d_add_total_fluxes" doesn't conform to snake_case naming style Warning
|
|
||
| """ | ||
| from alembic import op | ||
| import sqlalchemy as sa |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
third party import "import sqlalchemy as sa" should be placed before "from alembic import op" Warning
|
|
||
|
|
||
| # revision identifiers, used by Alembic. | ||
| revision = 'fc8ea63b8c9d' |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Constant name "revision" doesn't conform to UPPER_CASE naming style Warning
|
|
||
| # revision identifiers, used by Alembic. | ||
| revision = 'fc8ea63b8c9d' | ||
| down_revision = '5b40ed8dcd48' |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Constant name "down_revision" doesn't conform to UPPER_CASE naming style Warning
| # revision identifiers, used by Alembic. | ||
| revision = 'fc8ea63b8c9d' | ||
| down_revision = '5b40ed8dcd48' | ||
| branch_labels = None |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Constant name "branch_labels" doesn't conform to UPPER_CASE naming style Warning
| op.add_column('target', sa.Column('total_flux_j', sa.Float(), nullable=True, comment='J band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_g', sa.Float(), nullable=True, comment='Error in g-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_r', sa.Float(), nullable=True, comment='Error in r-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_i', sa.Float(), nullable=True, comment='Error in i-band total flux (nJy)')) |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Line too long (131/100) Warning
| op.add_column('target', sa.Column('total_flux_error_g', sa.Float(), nullable=True, comment='Error in g-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_r', sa.Float(), nullable=True, comment='Error in r-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_i', sa.Float(), nullable=True, comment='Error in i-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_z', sa.Float(), nullable=True, comment='Error in z-band total flux (nJy)')) |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Line too long (131/100) Warning
| op.add_column('target', sa.Column('total_flux_error_r', sa.Float(), nullable=True, comment='Error in r-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_i', sa.Float(), nullable=True, comment='Error in i-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_z', sa.Float(), nullable=True, comment='Error in z-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_y', sa.Float(), nullable=True, comment='Error in y-band total flux (nJy)')) |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Line too long (131/100) Warning
| op.add_column('target', sa.Column('total_flux_error_i', sa.Float(), nullable=True, comment='Error in i-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_z', sa.Float(), nullable=True, comment='Error in z-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_y', sa.Float(), nullable=True, comment='Error in y-band total flux (nJy)')) | ||
| op.add_column('target', sa.Column('total_flux_error_j', sa.Float(), nullable=True, comment='Error in J band total flux (nJy)')) |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Line too long (131/100) Warning
| # ### end Alembic commands ### | ||
|
|
||
|
|
||
| def downgrade(): |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Missing function or method docstring Warning
1 participant
total_flux_{g,r,i,z,y,j}andtotal_flux_error_{g,r,i,z,y,j}are added.--flux-typeoption for inserting target lists.