-
Notifications
You must be signed in to change notification settings - Fork 1.4k
docs: Update otel-diagram.svg as requested #6973
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
base: main
Are you sure you want to change the base?
docs: Update otel-diagram.svg as requested #6973
Conversation
|
thank you @leandro-caracciolo! Can you work with @jpkrohling to get the CLA signed? |
Hi @svrnm . It's done! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the diagram, but the SVG isn't editable by our OSS contributors who don't have access to Adobe software, including myself.
That is, the SVG contains about 4000 lines of CDATA:
<metadata>
<i:aipgfRef id="adobe_illustrator_pgf"/>
<i:aipgf id="adobe_illustrator_pgf" i:pgfEncoding="zstd/base64" i:pgfVersion="24">
<![CDATA[
KLUv/QBYVCQBmoh5RjCgSqaIHncdgxyEG6vqJV7YDUfPDZ57rt+GAwV+HhqtW9gkxVeeB+nYFF6H
lx9PYc5cBDQEWgTywQy+upQHo/S1q8qGvuPrwfhTGkso/ToU9Nc3O7rGN4FfffNH4ASL0p8fRonA
... [for about 4000 more lines]
This isn't editable without use of proprietary software.
@leandrocaracciolo thanks, the CLA is the first step:-) Re @chalin's question: the image you shared on slack is editable, this seems to be a different export, can you double check on that? |
e495393
to
61d493e
Compare
That looks great @leandro-caracciolo ! |
The last time I had a quick check, I noticed that there is still a 3-4K CDATA entry. Can the be removed? Do we know what it is? |
61d493e
to
53db960
Compare
Uh oh!
There was an error while loading. Please reload this page.