Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for configurable indentation length in turtle serialization #2709

Open
dgrtner-ecc opened this issue Sep 11, 2024 · 3 comments
Open
Labels
enhancement Incrementally add new feature

Comments

@dgrtner-ecc
Copy link

Version

5.0.0

Feature

Hi I would like to ask for an extension of the configuration regarding serialization of Turtle (possibly including Trig) s. https://jena.apache.org/documentation/io/rdf-output.html#opt-turtle-trig

The configuration option I propose is the number of characters for the indentation which is currently a constant in WriterConst. It is set to 8 and we would like to configure the serialization (of pretty turtle) to have another indentation.

Thanks,
Denis

Are you interested in contributing a solution yourself?

Perhaps?

@dgrtner-ecc dgrtner-ecc added the enhancement Incrementally add new feature label Sep 11, 2024
@afs
Copy link
Member

afs commented Sep 15, 2024

Hi @dgrtner-ecc ,

That would be good to have; both pretty styles ("wide", the default, and "long" where indent is 2) would benefit with being able to tweak the layout.

"long" style sets it to 2 in Turtle.ShellGraphLong (which really ought to be in WriterConst).

Have you tried it out and got generally reasonable changed output?

@dgrtner-ecc
Copy link
Author

Hi @afs,

yes it is definitely a big difference. This a very low priority issue so depending on your time and mine I could do it myself some time and provide a PR.

@afs
Copy link
Member

afs commented Oct 1, 2024

I quite unlikely to get round to it anytime soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Incrementally add new feature
Projects
None yet
Development

No branches or pull requests

2 participants