Line3.isparallel()/__or__ still scale-dependent, like the distance() bug fixed in #224
Mantenedores costumam responder em até 1 dia
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 72/100
Direção de pesquisa
Start at the Line3.isparallel() entry point and trace its callers, including or, ^/isintersecting, and commonperp(). Compare the tolerance handling with the distance() fix in #224; done means the parallel decision no longer changes when Plucker direction vectors are rescaled, while the related closest_to_line() check remains separate.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
#224 fixed Line3.distance() so it no longer depends on the magnitude of the Plucker direction vectors (w), only their direction — it was giving wrong results whenever a line's Plucker coordinates were rescaled by a factor other than 1, and crashing outright on parallel lines.
isparallel() (and hence __or__, ^/isintersecting, and commonperp(), all of which call it) has the same shape of bug: it tests norm(cross(l1.w, l2.w)) < tol * eps using the raw, non-unit direction vectors, rather than unit directions. Two lines with the same true angular separation can cross or miss the parallel threshold depending purely on how their Plucker coordinates happen to be scaled.
closest_to_line() wasn't checked for the same issue — worth a separate look.
Not urgent (no crash, unlike the distance() case), but the same class of bug and probably the same fix shape: normalize to unit vectors before the tolerance comparison.
Cross-ref: #224
- Linguagem predominante
- Python
- Estrelas
- 643
- Forks
- 102
- Merge médio
- 11min
- PRs com merge (30d)
- 1
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de rai-opensource/spatialmath-python
-
documentation
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
rai-opensource/spatialmath-python#235 ·
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
rai-opensource/spatialmath-python#32 · 4 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 66/100
rai-opensource/spatialmath-python#236 ·
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 68/100
rai-opensource/spatialmath-python#234 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 50/100
rai-opensource/spatialmath-python#177 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de rai-opensource/spatialmath-python
Issues semelhantes
-
bug llm translation
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Arkansas 2025 tax is $1.70 high above $100,000 net taxable income ($3,809 + 3.9% rule)Talvez já em andamento @PavelMakarchuk assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
PolicyEngine/policyengine-us#9828 ·
Mantenedores costumam responder em até 2 dias
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
jellyfin/jellyfin-mpv-shim#800 ·
Mantenedores costumam responder em até 1 dia
-
skillfs: one malformed chat-log line aborts the entire skill-usage analysis (skill_usage_from_chat_logs.py)Talvez já em andamento @zjncs assumiu hoje. Abertacomponent:skillfs
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
agentic-os-org/ANOLISA#6116 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
P4: low query
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
jeffknupp/association#336 ·