Skip to content

Feature/biomarker from correlation#89

Open
GonzzaG wants to merge 10 commits intov5.0.0from
feature/biomarker-from-correlation
Open

Feature/biomarker from correlation#89
GonzzaG wants to merge 10 commits intov5.0.0from
feature/biomarker-from-correlation

Conversation

@GonzzaG
Copy link
Collaborator

@GonzzaG GonzzaG commented Feb 20, 2026

La mayoria de cambios en src/frontend/static/frontend/src/components/pipeline/experiment-result/ResultTableControlForm.tsx son por correxion de indexacion en el archivo


// Styles
import { StopExperimentButton } from '../pipeline/all-experiments-view/StopExperimentButton'
import { DeleteButton } from '../common/DeleteButton'
Copy link
Member

Choose a reason for hiding this comment

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

No debería borrarse este archivo porque van a romperse un montón de estilos! Si los necesitás también en el panel de GEMs lo que podés hacer es importarlo también en el componente que necesites!

Create Biomarker
</Button>

{/* Modal con opciones */}
Copy link
Member

Choose a reason for hiding this comment

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

Todo debe ser en inglés


return (
<>
{/* Botón que abre el modal */}
Copy link
Member

Choose a reason for hiding this comment

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

Inglés


.no-margin-right-form-field * {
margin-right: 0 !important;
white-space: nowrap;
Copy link
Member

Choose a reason for hiding this comment

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

Esto debería quedar como estaba antes. Si necesitás el efecto de ellipsis, hay una clase que se llama .ellipsis en base.css. Usar esa donde haga falta

path('biomarker', views.biomarker, name='biomarker'),
path('differential-expression', views.differential_expression, name='differential_expression')
path('gene-symbols', views.GeneSymbols.as_view(), name='gene_symbols'),
path('gene-symbols-finder', views.GeneSymbols.as_view(), name='gene_symbols_finder'),
Copy link
Member

Choose a reason for hiding this comment

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

Eliminar todas estas URLS nuevas, ya están definidas en su propia app. Si se usan los nombres con las que se definieron en su lugar ya deberían funcionar perfecto

'threshold_to_consider_ordinal': settings.THRESHOLD_ORDINAL
}
)
class GeneSymbols(APIView):
Copy link
Member

Choose a reason for hiding this comment

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

Por qué están todas estas nuevas APIs? Ya existen en las apps de Django correspondientes y se pueden llamar desde el frontend a través de su URL. Eliminar de acá (a menos que haya una nueva, en ese caso avisarme para analizar donde deberían definirse. La app frontend solo se utiliza para definir vistas que devuelven HTML)

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