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

Fix error geo.py #12

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix error geo.py #12

wants to merge 2 commits into from

Conversation

FLomb
Copy link

@FLomb FLomb commented Oct 23, 2024

There seemed to be a typo in line 92, which was just breaking the code whenever trying to run

FLomb and others added 2 commits October 23, 2024 14:15
There seems to be a typo in line 92
@FLomb
Copy link
Author

FLomb commented Oct 23, 2024

The typo was also later in the file. Fixed both with the second commit

@sjpfenninger
Copy link
Member

Can you add more explanation? Does this raise an exception under certain circumstances? I haven't seen any issues with this code.

@FLomb
Copy link
Author

FLomb commented Oct 23, 2024

In the two lines I have edited, there was a multiplication sign (*) appearing out of nowhere before a list: df[*["html"] * len(df.columns.names)]. The first * was leading to a code-breaking error due to invalid syntax whenever trying to run calligraph.

@sjpfenninger
Copy link
Member

Which Python version?

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

Successfully merging this pull request may close these issues.

2 participants