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

Syntax error during make check for Guile 2.27 #162

Closed
oskooi opened this issue Jan 16, 2024 · 3 comments
Closed

Syntax error during make check for Guile 2.27 #162

oskooi opened this issue Jan 16, 2024 · 3 comments
Labels

Comments

@oskooi
Copy link
Collaborator

oskooi commented Jan 16, 2024

When building MPB from source using Guile 2.2.7 (the version installed with the precompiled binary guile-2.2-dev on Ubuntu 22.04), an error occurs during make check:

ice-9/boot-9.scm:222:17: In procedure map1:
Syntax error:
../examples/check.ctl:331:2: definition in expression context, where definitions are not allowed, 
in form (define symeigs-inv (compute-symmetries W w))
@oskooi oskooi added the bug label Jan 16, 2024
@thchr
Copy link
Contributor

thchr commented Jan 16, 2024

This is my mistake. When I was doing #134, I was compiling with Guile 3.0 or 3.1 and wasn't aware/did not remember that this syntax was disallowed on earlier versions.

I have time to look at this Friday this week or Monday next week (but if you need a fix sooner, of course, don't wait).

@thchr
Copy link
Contributor

thchr commented Jan 16, 2024

Okay, seems like the issue was very small: see #163.

I didn't check on Guile 2.27 in practice though, since I don't have that version: I'm assuming that's also not tested what is tested by CI (edit: maybe there is no CI here on the mpb repo - only meep?)

@oskooi oskooi closed this as completed in d767526 Jan 17, 2024
oskooi added a commit that referenced this issue Jan 17, 2024
@oskooi
Copy link
Collaborator Author

oskooi commented Jan 17, 2024

maybe there is no CI here on the mpb repo - only meep?

There used to be CI for the MPB repo using Travis. Since the Python API for MPB is provided by Meep, the Meep CI using Github Actions does include a unit test. However, currently there is no test for the Scheme API for MPB. This is why this error went undetected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants