Skip to content

Mapa de Nueva York como cabecera del README y figura de paneles en Resultados - #52

Open
R0SEWT wants to merge 1 commit into
developfrom
feature/hero-readme
Open

R0SEWT wants to merge 1 commit into
developfrom
feature/hero-readme

Conversation

@R0SEWT

@R0SEWT R0SEWT commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

Pone en el README las dos figuras que generan los scripts nuevos de tp/scripts/.

Cabecera: Nueva York en cuatro momentos (tp/reports/figuras/mapa.svg, de hero_mapa.py)

mapa

Cada zona de taxi va pintada con el arquetipo que más viajes tiene a esa hora, y la opacidad depende del volumen. Los cuatro momentos (semana 8 AM y 7 PM, fin de semana 3 PM y 11 PM) se eligieron porque en cada uno domina un arquetipo distinto en Manhattan. Los datos son los del visor (tp/app/data/). La línea del pie sale de reports/benchmark_*.json y de spin-casos.tsv, y el script falla si alguna de esas cifras deja de cumplirse.

Resultados: figura de paneles (tp/reports/figuras/hero.svg, de hero.py)

  • A: esquema de una iteración. Los workers toman chunks en cualquier orden y las sumas parciales se guardan por índice.
  • B: speedup frente a P en las cuatro máquinas, con IC 95 %.
  • C: la matriz máquina × P. Las 20 corridas terminan con la inercia 4531798,747970126. El script falla si dos corridas concurrentes difieren.

Cambios en esta PR

  • README.md: el mapa en la cabecera y la figura de paneles antes de la foto del Pixel.
  • tp/CLAUDE.md: los dos scripts en la tabla de archivos clave.

Los scripts y los SVG ya están en develop (f0a2b57, 8694767, fe754f5); esta PR solo los incorpora al README. Los dos SVG traen la tipografía incrustada (Arimo), así que se ven igual dentro de un <img> de GitHub.

Cómo regenerar

cd tp && uv run scripts/hero_mapa.py && uv run scripts/hero.py

Generated by Claude Code

Summary by Sourcery

Showcase the project’s New York taxi analysis and validated performance results directly in the README.

New Features:

  • Add a New York taxi-zone map to the README header.
  • Add a results panel figure covering the iteration mechanism, machine scaling, and consistent clustering outcomes.

Enhancements:

  • Document the scripts that generate the README figures in the project’s key-files table.

Documentation:

  • Update the README with the generated map and results visualization, including accessible descriptions and captions.

…aneles en Resultados

El mapa (tp/reports/figuras/mapa.svg) abre el README; la figura de paneles
(hero.svg) acompaña la tabla de resultados con el escalamiento de las cuatro
máquinas y la inercia idéntica en las 20 corridas. tp/CLAUDE.md lista los dos
scripts que las generan.
@sourcery-ai

sourcery-ai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

La PR integra en el README el mapa de Nueva York y la figura de paneles de resultados, ambos SVG ya generados por scripts existentes, y actualiza tp/CLAUDE.md para documentar esos scripts y sus comprobaciones.

File-Level Changes

Change Details Files
Añadir dos figuras SVG generadas por scripts existentes para mejorar la presentación y documentación visual del proyecto.
  • Incorporar el mapa de Nueva York como cabecera, con texto alternativo descriptivo y ancho completo.
  • Incorporar la figura de paneles de resultados antes de la imagen del Pixel, con leyenda sobre mecanismo, escalamiento e invariancia.
  • Mantener referencias a los scripts generadores y a los datos de origen en la presentación.
README.md
Documentar los nuevos scripts de generación de figuras en la guía interna del proyecto.
  • Añadir hero_mapa.py y hero.py a la tabla de archivos clave, describiendo sus salidas y validaciones.
tp/CLAUDE.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

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.

1 participant