Add entity geocoding tool#8003
Conversation
- Extracts location entities from NLP-annotated JSON (spaCy/Stanza) - Geocodes GPE, LOC, FAC, and ORG entities using Nominatim - Dual output: GeoJSON for interactive maps + tabular summary - Galaxy OpenLayers integration for map visualization - Configurable Nominatim server (public or self-hosted) - Entity type selection and deduplication - Pure Python with urllib.request (no geopy dependency) - Rate-limited public API or unlimited self-hosted options - Comprehensive tests and documentation - Enables spatial analysis of text corpora Tool: geocode_entities (v1.0.0+galaxy1) Categories: Text Manipulation, Natural Language Processing Citation: OpenStreetMap contributors
- Geocodes named entities from NER JSON to GeoJSON format - Uses Nominatim geocoding service for location resolution - Outputs interactive GeoJSON for map visualization in Galaxy - Works with JSON output from spaCy, Stanza, and CoreNLP NER - Includes entity aggregation and coordinate mapping Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
RZ9082
left a comment
There was a problem hiding this comment.
This PR is adding two different directories with identical content! So please let's start by removing one of them.
Keep only tools/geocode/ to resolve duplicate tools issue
- Update profile from 21.05 to 24.1 - Remove macros.xml and inline version - Fix homepage_url and remote_repository_url to point to IUC repository - Add Galaxy copyright notice to Python script - Add ftype attributes and has_n_rows to test assertions
|
Addressed review issues - removed duplicate tools/geocode_entities/ directory and applied standard IUC fixes |
Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
- Changed version suffix from 1 to 0 (bgruening) - Added requests requirement for Pulsar compatibility (bgruening) - Updated to use JSON asserts for GeoJSON output (bgruening) - Removed version history section from README (bgruening) - Updated homepage_url to point to specific tool directory (RZ9082) - Updated remote_repository_url to point to specific tool directory (RZ9082) Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
- Changed required_files type from 'file' to 'literal' - Added version to requests requirement Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Replaced complex JSON path with simple text assertions for GeoJSON output Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
…etic operators Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Add comment to ensure file is properly recognized by CI linter. Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
- Add required 2 blank lines before function definition - Update comment to trigger clean GitHub Actions environment - Addresses both flake8 and planemo lint CI environment issues Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
|
I filled an issue here: galaxyproject/planemo#1646 |
And I also opened galaxyproject/planemo#1645 |
bgruening
left a comment
There was a problem hiding this comment.
Thanks @ksuderman just two tiny things
|
Since the two referenced Issues are now closed, is this tool good to go? |
…ement, use profile macro
Done. |
…param help, correct input example and ORG mentions, fold README into tool help Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Deployment status: skipped https://github.com/galaxyproject/tools-iuc/actions/runs/29002795920 |
|
Thanks @ksuderman |
|
Deployment status: success https://github.com/galaxyproject/tools-iuc/actions/runs/29002795920 |
|
Thanks so much @ksuderman! |
Summary
Test plan
🤖 Generated with Claude Code