Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate Remora versioning issue in notebook #34

Open
jackVanish opened this issue Dec 18, 2024 · 4 comments
Open

Investigate Remora versioning issue in notebook #34

jackVanish opened this issue Dec 18, 2024 · 4 comments
Assignees

Comments

@jackVanish
Copy link

The remora R notebook seems to run into an issue wherein it uses a version of Remora older than the code currently on the HEAD commit of the master branch. I have encountered this before but I do not recall how I fixed it- I thought that the rm ls() call at the start of the notebook would clear it but I seem to still be having problems.

Either way, this needs to be investigated and fixed. I expect to be handling this but I have also added @jdpye as an assignee in case he feels like he has time and availability to poke his nose into this while he's trying to install the notebook.

@jdpye
Copy link
Member

jdpye commented Dec 18, 2024

image

it's not 'run and done' but i'm looking at some errors that popped up when installing remora. The order of dependency appears to be installing glatos, surimi, remora from GitHub.

The dependencies issue i'm hitting from my local installation are about nanoparquet and having to compile it on Mac:

using SDK: ‘MacOSX13.1.sdk’
arm64-apple-darwin20.0.0-clang++ -std=gnu++17 -I"/Users/pye/miniconda3/envs/ipyutils-R/lib/R/include" -DNDEBUG -Ithrift -I. -Izstd/include  -D_FORTIFY_SOURCE=2 -isystem /Users/pye/miniconda3/envs/ipyutils-R/include -mmacosx-version-min=11.0 -mmacosx-version-min=11.0 -I/Users/pye/miniconda3/envs/ipyutils-R/include   -DR_NO_REMAP -fPIC  -ftree-vectorize -fPIC -fstack-protector-strong -O2 -pipe -stdlib=libc++ -fvisibility-inlines-hidden -fmessage-length=0 -isystem /Users/pye/miniconda3/envs/ipyutils-R/include -fdebug-prefix-map=/Users/runner/miniforge3/conda-bld/r-base-split_1728291248336/work=/usr/local/src/conda/r-base-4.3.3 -fdebug-prefix-map=/Users/pye/miniconda3/envs/ipyutils-R=/usr/local/src/conda-prefix  -c rwrapper.cpp -o rwrapper.o
/bin/sh: arm64-apple-darwin20.0.0-clang++: command not found
make: *** [rwrapper.o] Error 127
ERROR: compilation failed for package ‘nanoparquet’

@jdpye
Copy link
Member

jdpye commented Dec 18, 2024

nanoparquet is a relatively recent dependency of rerddap, not sure if pinning to a lesser version is the complete answer but it may be a way to sidestep this stopper.

@jdpye
Copy link
Member

jdpye commented Dec 18, 2024

Building up the environment.yml or the DESCRIPTION file to accurately portray these new remora dependencies will possibly help us get this versioning issue figured out as well as build the remora environment cleanly for more users.
see:
https://stackoverflow.com/a/41017200

for ways to declare remote repositories such as github repos as sources for certain packages in the DESCRIPTION file.

@jdpye
Copy link
Member

jdpye commented Dec 18, 2024

For posterity: the environment.yml file in question is over in the ipython-utilities repository on GitLab, in the remora_notebook branch. https://gitlab.oceantrack.org/otn-partner-nodes/ipython-utilities/-/tree/remora_notebook?ref_type=heads

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

No branches or pull requests

2 participants