Conversation
…a correlation experiment
|
|
||
| // Styles | ||
| import { StopExperimentButton } from '../pipeline/all-experiments-view/StopExperimentButton' | ||
| import { DeleteButton } from '../common/DeleteButton' |
There was a problem hiding this comment.
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 */} |
|
|
||
| return ( | ||
| <> | ||
| {/* Botón que abre el modal */} |
|
|
||
| .no-margin-right-form-field * { | ||
| margin-right: 0 !important; | ||
| white-space: nowrap; |
There was a problem hiding this comment.
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
src/frontend/urls.py
Outdated
| 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'), |
There was a problem hiding this comment.
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
src/frontend/views.py
Outdated
| 'threshold_to_consider_ordinal': settings.THRESHOLD_ORDINAL | ||
| } | ||
| ) | ||
| class GeneSymbols(APIView): |
There was a problem hiding this comment.
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)
… into feature/biomarker-from-correlation
| from django.contrib.auth.decorators import login_required | ||
| from django.conf import settings | ||
|
|
||
| from rest_framework.views import APIView |
There was a problem hiding this comment.
Hay un millón de importaciones nuevas pero nada de código nuevo abajo. Así que estimo que no se usan! Usar PyCharm para corregir los warnings que aparezcan y eliminar todas las importaciones innecesarias
2 participants
La mayoria de cambios en src/frontend/static/frontend/src/components/pipeline/experiment-result/ResultTableControlForm.tsx son por correxion de indexacion en el archivo