Comments (3)
Thanks for the detailed report! This sounds like an issue at a deeper layer than puget; the print width is passed directly to fipp, and the only interaction that I can think of that is implicated by the colorization is the use of :pass
nodes to avoid counting color markup against the document width. I'm not sure why this would show up only at higher widths, unless the behavior of rrb-vector somehow alters at larger sizes.
from puget.
Some brief experimentation seems to indicate that this is related to the size of the data structure passed to fipp. Inspecting the print document being rendered showed that puget was doing some (very slightly) suboptimal behavior by passing a no-op [:align ()]
structure inside empty collections. Removing this did not fix the problem, but it did raise the :width
limit at which the error occurs by the same amount as the number of empty collections in your bad-data
sample, which seems significant.
from puget.
I also did some digging, and I agree that it doesn't seem likely that the bug is in puget. I've filed the following issue for fipp: brandonbloom/fipp#53
I have a hunch that the bug is actually in in clojure.core.rrb-vector. However, my attempts to produce a rrb-vector-only test case that triggers the bug have been unsuccessful.
from puget.
Related Issues (20)
- cannot work with core.cache HOT 2
- non-global-extensibility HOT 10
- sort-keys missing from 0.9.0 HOT 3
- Lazy seqs don't print in 0.9.0 HOT 3
- ClojureScript support? HOT 18
- Can you document how to customize colorschemes in puget. HOT 2
- Error on load HOT 3
- Print namespaced maps using #: HOT 7
- Sorted collections are re-sorted HOT 4
- Puget doesn't respect overriden clojure.core/print-method for records HOT 1
- colors start at beginning of line instead of beginning of element HOT 4
- README code coverage badge is stale
- Java object cprint does not match README screenshot HOT 1
- :seq-limit for vectors? HOT 2
- Unidiomatic representation of namespaced maps HOT 1
- GraalVM binary HOT 1
- Migrate references to java.util.TimeZone to java.time.* HOT 1
- Metadata doesn't respect current `:align`
- Broken links need a fix
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from puget.