After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 756608 - docs: Reduce the size of slides, 3.8MB is too much
docs: Reduce the size of slides, 3.8MB is too much
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gstreamer-vaapi
unspecified
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks: 758397
 
 
Reported: 2015-10-15 06:29 UTC by sreerenj
Modified: 2016-10-31 14:23 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description sreerenj 2015-10-15 06:29:36 UTC
I have a feeling that, it is bit inappropriate to add all html-css-image (3.8 MB) stuffs in the gstreamer-vaapi source. I would rather prefer to have a simple text pdf which should only include the contents of the slides, so that we can think about including this in distribution tar-ball too.
Comment 1 Víctor Manuel Jáquez Leal 2015-10-15 10:38:53 UTC
You can export the slide to PDF, using Chrome: 

https://github.com/hakimel/reveal.js#pdf-export

But the output is 22MB size
Comment 2 sreerenj 2015-10-15 13:58:36 UTC
May be a stripped-down version?,,,We can even omit  pages under "Challenges". Also without any images..
IMHO, all the pages under the sessions "VA-API VIDEO ACCELERATION" and "WHAT IS GSTREAMER-VAAPI?" should be enough as a reference document..
Comment 3 Víctor Manuel Jáquez Leal 2015-10-19 11:39:08 UTC
I have been thinking on this all the weekend. I feel bad because I made a mistake pushing this without asking for comments.

I see two possible solutions:

1\ trim the reveal.js to it's minimal required to render the slides
2\ remove all reveal.js and use the makefile to clone a specific version of reveal.js (like a git submodule)

Either way, all these 3.8MB are already in repository history :( and will be downloaded by the clients, unless we do some surgery in the git tree, which is dangerous.


(In reply to sreerenj from comment #2)
> May be a stripped-down version?,,,We can even omit  pages under
> "Challenges". Also without any images..
> IMHO, all the pages under the sessions "VA-API VIDEO ACCELERATION" and "WHAT
> IS GSTREAMER-VAAPI?" should be enough as a reference document..

The slides themselves are in index.html, and it doesn't use almost any space, the problem is the reveal.js code, theme and fonts.
Comment 4 sreerenj 2015-10-19 12:53:17 UTC
I think you can do,

-- git rebase -i 20baf4c39d0c9296ad001fe09636aa5d96c07b29^
-- remove the commit "docs: slides from GStreamer Conference 2015"
-- save&quit
-- git push -f

Any issue with this???
Comment 5 Víctor Manuel Jáquez Leal 2015-10-19 13:11:02 UTC
(In reply to sreerenj from comment #4)
> I think you can do,
> 
> -- git rebase -i 20baf4c39d0c9296ad001fe09636aa5d96c07b29^
> -- remove the commit "docs: slides from GStreamer Conference 2015"
> -- save&quit
> -- git push -f
> 
> Any issue with this???

I thinks it is a bit risky: who has pull/clone recently the repo, will find broken their work-flow. A simple pull won't work.

If we do it, it would be done asap, to avoid a bigger history rewrite.

Should I do it?
Comment 6 sreerenj 2015-10-19 13:14:47 UTC
(In reply to Víctor Manuel Jáquez Leal from comment #5)
> (In reply to sreerenj from comment #4)
> > I think you can do,
> > 
> > -- git rebase -i 20baf4c39d0c9296ad001fe09636aa5d96c07b29^
> > -- remove the commit "docs: slides from GStreamer Conference 2015"
> > -- save&quit
> > -- git push -f
> > 
> > Any issue with this???
> 
> I thinks it is a bit risky: who has pull/clone recently the repo, will find
> broken their work-flow. A simple pull won't work.
> 

Thats right, but I am ready to take that risk :)

> If we do it, it would be done asap, to avoid a bigger history rewrite.
> 
Exactly !

> Should I do it?

Yes
Comment 7 Víctor Manuel Jáquez Leal 2015-10-19 14:22:58 UTC
done... 

(fingers crossed)
Comment 8 Víctor Manuel Jáquez Leal 2015-10-28 18:14:35 UTC
I have published them here:

http://people.igalia.com/vjaquez/talks/gstvaapi-201509

I was thinking the option of only pushing the index.html and the important images. reveal.js might be loaded remotely (we need to check that option) or clone the sources when running make.
Comment 9 sreerenj 2016-03-24 16:54:16 UTC
Moving to Product:GStreamer, Component:gstreamer-vaapi
Comment 10 Tim-Philipp Müller 2016-03-24 23:52:18 UTC
If you have slides in PDF format they can also be put here fwiw: https://gstreamer.freedesktop.org/data/events/gstreamer-conference/2015/
Comment 11 Víctor Manuel Jáquez Leal 2016-03-28 14:59:52 UTC
(In reply to Tim-Philipp Müller from comment #10)
> If you have slides in PDF format they can also be put here fwiw:
> https://gstreamer.freedesktop.org/data/events/gstreamer-conference/2015/

Done
Comment 12 Tim-Philipp Müller 2016-06-02 11:25:21 UTC
I think we can close this then?

aiui the large docs have been removed from git and published elsewhere. Anything left to do?
Comment 13 Víctor Manuel Jáquez Leal 2016-06-02 11:48:06 UTC
(In reply to Tim-Philipp Müller from comment #12)
> I think we can close this then?
> 
> aiui the large docs have been removed from git and published elsewhere.
> Anything left to do?

Yes, let's close it. Thanks Tim.