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

Fixed the typo #467

Closed
wants to merge 5 commits into from
Closed

Fixed the typo #467

wants to merge 5 commits into from

Conversation

speco29
Copy link

@speco29 speco29 commented Jan 27, 2025

Fixes #466
I think the image wasn't displaying due to some missed parameters, I also noticed that none of the formulations in in that doc are rendered from the notebook, for now i tried fixing the first formulation, if it's working, please let me know, then I'll change the code for other formulations too;)

fixes pysal#466
I have made few changes, please let me know if the docs still isn't displaying the image properly.
fixes pysal#466
fixed a typo in the code.
fixes pysal#466
A typo is fixed
@speco29
Copy link
Author

speco29 commented Jan 27, 2025

There are multiple files are added to this PR, I apologize for this mess. please ignore the first three, they are added by mistake and when I'm trying to delete them, it's giving me 404 error, I'll fixed it later.

Copy link
Member

@gegen07 gegen07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that you deleted notebooks/p-median.ipynb in your commit.

@speco29
Copy link
Author

speco29 commented Jan 27, 2025

It seems that you deleted notebooks/p-median.ipynb in your commit.

Is it working tho?
should i change the other formulation in that code too?

@jGaboardi
Copy link
Member

It seems that you deleted notebooks/p-median.ipynb in your commit.

Is it working tho? should i change the other formulation in that code too?

The only change here was the removal of notebooks/p-median.ipynb, so nothing has been updated.

@speco29 speco29 closed this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

formulations on docs site not rendering properly
3 participants