ebi-gene-expression-group / atlas-web-bulk

Bulk Expression Atlas web application
Apache License 2.0
0 stars 1 forks source link

The results are not appearing for some of the base experiments in bulk #171

Closed ke4 closed 6 months ago

ke4 commented 7 months ago

Example experiment in staging: https://wwwdev.ebi.ac.uk/gxa/experiments/E-MTAB-6592/Results

Here is the error log from the JS Console:

The above error occurred in the <AnatomogramSvg> component:
    in AnatomogramSvg (created by Anatomogram)
    in div (created by Anatomogram)
    in Anatomogram (created by mapProps(Anatomogram))
    in mapProps(Anatomogram) (created by defaultProps(mapProps(Anatomogram)))
    in defaultProps(mapProps(Anatomogram)) (created by defaultProps(defaultProps(mapProps(Anatomogram))))
    in defaultProps(defaultProps(mapProps(Anatomogram))) (created by onlyUpdateForPropTypes(defaultProps(defaultProps(mapProps(Anatomogram)))))
    in onlyUpdateForPropTypes(defaultProps(defaultProps(mapProps(Anatomogram)))) (created by withPropsOnChange(onlyUpdateForPropTypes(defaultProps(defaultProps(mapProps(Anatomogram))))))
    in withPropsOnChange(onlyUpdateForPropTypes(defaultProps(defaultProps(mapProps(Anatomogram))))) (created by defaultProps(withPropsOnChange(onlyUpdateForPropTypes(defaultProps(defaultProps(mapProps(Anatomogram)))))))
    in defaultProps(withPropsOnChange(onlyUpdateForPropTypes(defaultProps(defaultProps(mapProps(Anatomogram)))))) (created by DebouncedContainer)
    in DebouncedContainer (created by _HeatmapWithControls)
    in div (created by _HeatmapWithControls)
    in div (created by _HeatmapWithControls)
    in div (created by _HeatmapWithControls)
    in _HeatmapWithControls (created by Uncontrolled(_HeatmapWithControls))
    in Uncontrolled(_HeatmapWithControls) (at uncontrollable.js:128)
    in ForwardRef (created by HeatmapWithControlsContainer)
    in HeatmapWithControlsContainer (created by ChartContainer)
    in div (created by ChartContainer)
    in div (created by ChartContainer)
    in ChartContainer (created by Container)
    in div (created by Container)
    in Container (created by ContainerLoader)
    in ContainerLoader (created by Refetch.connect(ContainerLoader))
    in Refetch.connect(ContainerLoader) (created by ExpressionAtlasHeatmap)
    in div (created by ExpressionAtlasHeatmap)
    in ExpressionAtlasHeatmap (created by Main)
    in div (created by Main)
    in div (created by Main)
    in Main (created by Context.Consumer)
    in withRouter(Main)
    in Unknown (created by Context.Consumer)
    in Route (created by ExperimentContainerRouter)
    in Switch (created by ExperimentContainerRouter)
    in div (created by ExperimentContainerRouter)
    in Router (created by BrowserRouter)
    in BrowserRouter (created by ExperimentContainerRouter)
    in ExperimentContainerRouter

Consider adding an error boundary to your tree to customize error handling behavior.
Visit https://fb.me/react-error-boundaries to learn more about error boundaries.
ke4 commented 6 months ago

I did some research today. It looks like that we have a repo: atlas-experiment that is published to npm with this name: @ebi-gene-expression-group/expression-atlas-experiment-page. That has the ExperimentContainerRouter component. I still have to dig deeper to find what exactly is the issue with this bug.

ke4 commented 6 months ago

The issues were the following:

What I have done to enable the bulk release:

This branch of bulk was built on my local successfully with the following manual steps:

  1. Manually build the UI: ./compile-front-end-packages.sh -iu
  2. Manually build the backend: ./gradlew :app:war
  3. Use this script to startup the web app on your local environment: ./build-and-deploy-webapp.sh -n
ke4 commented 6 months ago

I also deployed the locally built WAR file into:

ke4 commented 6 months ago

Next steps:

ke4 commented 6 months ago

A compatible develop branch has been created to be able to merge the hot-fix branch (without the experiment design refactor): https://github.com/ebi-gene-expression-group/atlas-web-bulk/tree/hotfix/without_exp_design. This is the new develop branch: adc31c62a20dfa004e7f3c5052808b1b318704e5

ke4 commented 6 months ago

I have created another another ticket for the dependency update: Update bulk with the latest webpack and irs dependencies. I am going to close this ticket as the original problem has been solved.