Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Morphotope tutorial#712

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
u3ks wants to merge5 commits intopysal:main
base:main
Choose a base branch
Loading
fromu3ks:morphotopes_examples
Open

Conversation

@u3ks
Copy link
Collaborator

@u3ksu3ks commentedOct 5, 2025

The notebook has:

  • same place as the clustering tutorial and same variables
  • Overture data
  • Neatnet
  • ETC + adaptive limit + postprocessing
  • No spatial lag
  • S3
  • Dealing with noise

Anything missing ? maybe itd be good to cache the overture data, because it takes 4-5 minutes just reading it ...

@codecov
Copy link

codecovbot commentedOct 5, 2025
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.3%. Comparing base (4037c70) to head (04fa370).
⚠️ Report is 126 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@##            main    #712     +/-   ##=======================================+ Coverage   97.4%   98.3%   +0.9%=======================================  Files         26      26               Lines       4328    4195    -133     =======================================- Hits        4214    4123     -91+ Misses       114      72     -42

see 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@martinfleis
Copy link
Member

Can we do OSM data and osmnx as we do elsewhere in the guide? Any reason to use Overture?

@u3ks
Copy link
CollaboratorAuthor

u3ks commentedOct 5, 2025
edited
Loading

the only reason was to be consistent with urban_taxonomy, but changing to osmnx shouldnt make a difference really. We'll have to drop/change the tunnel and street type filtering though

@martinfleis
Copy link
Member

That is fine, we don't need to be 1:1 with the paper here. The other is not either.

@u3ks
Copy link
CollaboratorAuthor

u3ks commentedNov 2, 2025

I changed the data to OSM. The morphotopes use the same params as the paper and look good. The hierarchy is a bit meh, but i think its ok for a tutorial.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@u3ks@martinfleis

[8]ページ先頭

©2009-2025 Movatter.jp