-
Notifications
You must be signed in to change notification settings - Fork 2
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
Error importing LinearCorePotential #67
Comments
You are very welcome to submit a pull request. We will review it anyway. Thank you for finding this error, and sorry that this issue slipped out of my attention. |
@sannant could you please give me the access rights to submit pull requests? |
you can create a fork on github and push there. (you will need to set your fork as push remote) |
Thanks, I have submitted the pull request. |
@jotelha @sannant However, when I am running a code that calls that function using Is there something that needs to be done for the changes to be reflected when executing using Docker images? |
Hi @sitangshugk95 the containers won't update by themselves, we will build a new one with some future release soon. |
Thanks @jotelha. Could you give me a rough estimate of when you expect to do it? It would help me plan my work accordingly, as I can't proceed unless it is done. Also, since it looks like Docker images is the approach I would have to stick to to use contact.engineering for the near future, what would be a realistic frequency (biweekly/monthly/other) to update the containers based on your workload? I don't think it would be needed very frequently anyway once the basic debugging is done, but I would still like to have that info and inform my advisor. |
Can't give any estimate here, since ContactEngineering is an academic project, and many of the repositories like If you need your own modifications in the container timely, you can bind your local copy of whatever repository you changed into the container instance and override whatever is already present within the container. Have a look at https://github.com/ContactEngineering/ce-container-stack/tree/master/docker/jupyterlab-SurfaceTopography#changes-to-the-container Let's say you want a modified
Hope that helps. |
Thank you for the very detailed answer, so sorry again for my unfamiliarity with Linux.
@jotelha Quick question - could you please tell me how to modify the above command to use it with singularity without root privileges? eg; I generally use |
This question boils down to looking into the singularity documentation, https://docs.sylabs.io/guides/3.0/user-guide/bind_paths_and_mounts.html will tell you to run
but isn't your home directory mounted anyway with singularity? |
@jotelha All I to do run the docker images is type Anyway, I used |
https://github.com/ContactEngineering/Adhesion/blob/master/examples/adhesive_plastic_simulation_cone.ipynb
I think it should be
from Adhesion.Interactions.cutoffs import LinearCorePotential
instead offrom Adhesion.Interactions import LinearCorePotential
I was getting this error:
![image](https://user-images.githubusercontent.com/108830413/204718852-8eca202d-4f2d-481a-9a03-971650466ad0.png)
![image](https://user-images.githubusercontent.com/108830413/204719006-81cca1f0-6b6c-4db3-8e97-20882ca03cee.png)
but changing it fixed the error:
However, I cannot make changes to the function definitions, and I guess that it is the same issue that is causing this error:
![image](https://user-images.githubusercontent.com/108830413/204719209-6ed6dfdd-1dbb-4b74-b4b7-0b8bd4770e23.png)
@sannant Please let me know if this is something that needs to be fixed on your end, or is it something that I am doing wrong?
The text was updated successfully, but these errors were encountered: