ProjectMirador / mirador-annotations

a Mirador 3 plugin that adds annotation creation tools to the user interface
https://mirador-annotations.netlify.app/
38 stars 35 forks source link

cannot install mirador-annotations #75

Open cmcooney opened 1 year ago

cmcooney commented 1 year ago

Dear Mirador -- I've tried two things to get the annotations plug-in to work, neither is working for me:

1) Add the npm module to an instance of mirador, and I get the following error:

npm ERR! code ERESOLVE npm ERR! ERESOLVE unable to resolve dependency tree npm ERR! npm ERR! While resolving: undefined@undefined npm ERR! Found: react@17.0.2 npm ERR! node_modules/react npm ERR! peer react@"^17.0" from mirador-annotations@0.5.0 npm ERR! node_modules/mirador-annotations npm ERR! mirador-annotations@"" from the root project npm ERR! peer react@"^16.8.0 || ^17.0.0" from @material-ui/core@4.12.4 npm ERR! node_modules/@material-ui/core npm ERR! peer @material-ui/core@"^4.9.13" from mirador-annotations@0.5.0 npm ERR! node_modules/mirador-annotations npm ERR! mirador-annotations@"" from the root project npm ERR! peer @material-ui/core@"^4.0.0" from @material-ui/icons@4.11.3 npm ERR! node_modules/@material-ui/icons npm ERR! peer @material-ui/icons@"^4.9.1" from mirador-annotations@0.5.0 npm ERR! node_modules/mirador-annotations npm ERR! mirador-annotations@"" from the root project npm ERR! 1 more (mirador) npm ERR! 2 more (@material-ui/lab, mirador) npm ERR! 3 more (react-dom, @material-ui/icons, @material-ui/lab) npm ERR! npm ERR! Could not resolve dependency: npm ERR! peer react@"^16.8.3" from mirador@3.3.0 npm ERR! node_modules/mirador npm ERR! mirador@"^3.3.0" from the root project npm ERR! peer mirador@"^3.0.0-rc.5" from mirador-annotations@0.5.0 npm ERR! node_modules/mirador-annotations npm ERR! mirador-annotations@"" from the root project

2) Build the module separately by downloading the github repo. But when I run "npm install" to get the dependencies, I get the following error message: npm ERR! code ERESOLVE npm ERR! ERESOLVE unable to resolve dependency tree npm ERR! npm ERR! While resolving: mirador-annotations@0.5.0 npm ERR! Found: @psychobolt/react-paperjs@1.0.3 npm ERR! node_modules/@psychobolt/react-paperjs npm ERR! @psychobolt/react-paperjs@"^1.0.0" from the root project npm ERR! npm ERR! Could not resolve dependency: npm ERR! peer @psychobolt/react-paperjs@"^0.0.58" from @psychobolt/react-paperjs-editor@0.0.14 npm ERR! node_modules/@psychobolt/react-paperjs-editor npm ERR! @psychobolt/react-paperjs-editor@"^0.0.14" from the root project

Can you give me any advice on how to get this running?

Thank you.

Charles Cooney ARTFL Project University of Chicago

dnoneill commented 1 year ago

+1 on this error.

G4me3 commented 1 year ago

+1 on this error.

agentcole commented 1 year ago

+1 on this error.

enriquediaz commented 1 year ago

The source of this issue lies with Mirador itself, which is still using older versions of dependencies such as React in order to build. Developers from Stanford, Harvard, and Leipzig are contributing to updating the core code in Mirador, and this will help resolve the errors you are running into. This is complex work and requires a great deal of time and attention, so the work is still in progress.

That said, the current version of the Mirador master branch includes a number of updates that may already address the errors you're encountering. We need testers to confirm experiences so we can cut a release of the updated code. Please consider trying out the latest build of Mirador and check in with your experiences in the corresponding discussion thread.

agentcole commented 1 year ago

Thank you very much for providing the link to the discussion thread. We've managed to get everything working with React 16.x. However, many plugins we would like to use are still only compatible with the latest stable release.

cmcooney commented 1 year ago

Thank you for the reply. We would be happy to test the latest build and share our experiences -- I don't know if we'd be able to do so until rather late in the summer, however.