-
Notifications
You must be signed in to change notification settings - Fork 525
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
Rv kmir milestone 4 #1060
Rv kmir milestone 4 #1060
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ehildenb could you please remove the changes made to the workflows? Please note that the delivery should only include a single delivery file (which you included) but nothing else.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deliveries/rv_kmir-milestone_4.md
Outdated
| **0a.** | License | [BSD-3](https://github.com/runtimeverification/mir-semantics/blob/milestone4-deliverable/LICENSE) | | ||
| **0b.** | Documentation | [kmir CLI instructions](https://github.com/runtimeverification/mir-semantics/blob/milestone4-deliverable/kmir/README.md) | | ||
| **0c.** | Testing and Testing Guide | [Testing Instructions with Docker](https://github.com/runtimeverification/mir-semantics/tree/milestone4-deliverable#running-integration-tests-with-docker) | | ||
| **0d.** | Docker | [Dockerfile](https://github.com/runtimeverification/mir-semantics/blob/milestone4-deliverable/Dockerfile | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| **0d.** | Docker | [Dockerfile](https://github.com/runtimeverification/mir-semantics/blob/milestone4-deliverable/Dockerfile | | |
| **0d.** | Docker | [Dockerfile](https://github.com/runtimeverification/mir-semantics/blob/milestone4-deliverable/Dockerfile] | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this was meant to be )
Hi @takahser, sorry I didn't get any notifications when this PR was created. I can help you out with the evaluation. Docker test run problemsThis is strange to me, because our test runner for CI is the same docker script (link1, link2) and has no problem, and I ran the docker same commands locally on my machine just now to check and they executed as expected. Could you provide some details for the system you are running these on? DocumentationWe are in the process of updating the documentation currently. We can add this into the PR, I have made an issue here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you provide some details for the system you are running these on?
I'm using a macbook pro m2, running macOS 14.1.1
What machine and OS are you using?
We are in the process of updating the documentation currently. We can add this into the PR, I have made an runtimeverification/mir-semantics#271.
Thanks, I've updated the evaluation.
Also, please have a look at the inline comments as well.
d37f79d
to
fbe35f3
Compare
Hi @takahser, just giving you an update. I am running the tests on |
Co-authored-by: S E R A Y A <[email protected]>
@takahser So we have spent a bit of time trying to determine the root of the problem that you experienced, and it appears that there are some behaviours with Docker on Mac Silicon machines that create this issue. We have discussed it at length, and have decided that we will not explicitly support running the test suite through docker directly on Mac Silicon. This does not mean that the tests cannot be run on Mac Silicon however, it just means that the Nix test runner should be used (part of this delivery actually). A PR has been made that explicitly includes Mac Silicon as part of CI to demonstrate that it is supported. The wording of the delivery indicates that docker is intended to be used for CI, and so we feel that all of this is in line with the delivery. If a different machine (not Mac Silicon) is available for you to use then the Docker commands run should work (all of our testing has worked on other machines so far). Happy to discuss anything in more detail if necessary of course. |
@dkcumming okay, I'm going to re-test it on a ubuntu AWS instance in that case. Will keep you posted, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ehildenb I've tested on ubuntu, now the tests pass when using docker. However, I wasn't able to build the project using make build
:
# make build
make: *** No rule to make target 'build'. Stop.
Any idea what could be the issue here?
Update:
I'm not sure if I ran it in the wrong directory, but even when running it in the mir-semantics
directory, it fails:
~/mir-semantics/kmir# make build
poetry install
RuntimeError
The Poetry configuration is invalid:
- Additional properties are not allowed ('group' was unexpected)
at /usr/lib/python3/dist-packages/poetry/core/factory.py:43 in create_poetry
39│ message = ""
40│ for error in check_result["errors"]:
41│ message += " - {}\n".format(error)
42│
→ 43│ raise RuntimeError("The Poetry configuration is invalid:\n" + message)
44│
45│ # Load package
46│ name = local_config["name"]
47│ version = local_config["version"]
make: *** [Makefile:30: poetry-install] Error 1
@takahser Hi, I will help out with this one. Running the commands for poetry means that you must be in the directory of As for the error. First glance is that there is a permissions issue. I can investigate further, but I would like to know the set up you are running this on. Could you provide any details for that? Also could you please run |
@dkcumming sure no problem: ~/mir-semantics/kmir# ls -l
total 80
-rwxrwxr-x 1 root root 6141 Dec 13 03:50 Makefile
-rwxrwxr-x 1 root root 560 Dec 13 03:50 check_k_version.sh
-rwxrwxr-x 1 root root 2460 Dec 13 03:50 doit.sh
-rwxrwxr-x 1 root root 3975 Dec 13 03:50 flake.lock
drwxrwxr-x 2 root root 4096 Dec 13 03:50 k-src
-rwxrwxr-x 1 root root 417 Dec 13 03:50 kbuild.toml
-rwxrwxr-x 1 root root 38036 Dec 13 03:50 poetry.lock
-rwxrwxr-x 1 root root 1080 Dec 13 03:50 pyproject.toml
-rwxrwxr-x 1 root root 185 Dec 13 03:50 set_env.sh
drwxrwxr-x 4 root root 4096 Dec 13 03:50 src
I'm running it on a EC2 AWS instance that runs on ubuntu 22: # lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 22.04.3 LTS
Release: 22.04
Codename: jammy |
@dkcumming right, thanks for pointing out! 👍 |
@takahser and @semuelle this milestone review is taking quite some time now. We are testing the semantics in a CI environment continuously, and have provided instructions and utilities to make it easier to install via Let us know, we would like to move this along so we can focus on next steps with the semantics! |
Hi @dkcumming. As @takahser mentioned, I took over the evaluation. Good news, I don't have a Mac and everything worked basically out of the box. The only thing that's missing is D2, the article. You can add it to the delivery document or share it privately by email. |
Thanks, @dkcumming. Your milestone is hereby accepted. You can find the evaluation notes here. Congratulations on completing the grant, and best of luck going forward! Hope to see you around. |
🪙 Please fill out the invoice form in order to initiate the payment process. Thank you! |
We noticed that this is the last milestone of your project. Congratulations on completing your grant! 🎊 |
I've submitted the invoice form. |
Milestone Delivery Checklist
Link to the application pull request: w3f/Grants-Program#1422 < please fill this in with the PR number of your application.