| *** tpb has joined #timvideos | 00:00 | |
| -leguin.freenode.net- [freenode-info] channel trolls and no channel staff around to help? please check with freenode support: http://freenode.net/faq.shtml#gettinghelp | 00:00 | |
| *** mithro- has joined #timvideos | 02:04 | |
| *** mithro- has quit IRC | 03:01 | |
| *** hyades has joined #timvideos | 03:08 | |
| *** hyades has quit IRC | 03:41 | |
| mithro | Sewar, ping? | 04:29 |
|---|---|---|
| Sewar | mithro, pong | 04:30 |
| mithro | Sewar, you linked me to the edid-grabber-in-c blog post | 04:30 |
| mithro | but that seems to be from the 19th August? | 04:30 |
| Sewar | oops, same blog title/url | 04:31 |
| Sewar | http://mohammedalhadab.com/ | 04:31 |
| tpb | Title: Mohammed Alhadab (at mohammedalhadab.com) | 04:31 |
| Sewar | mithro, iiie, CarlFK, parx: http://mohammedalhadab.com/gsoc/edid-grabber-in-c-2/ | 04:32 |
| tpb | <http://ln-s.net/-t$B> (at mohammedalhadab.com) | 04:32 |
| mithro | Ah ha | 04:33 |
| mithro | Sewar, I'd recommend using mingw rather then Visual C++ | 04:33 |
| mithro | it's a bit harder to get set up | 04:33 |
| mithro | but then you can cross compile under linux | 04:33 |
| mithro | and you never have to touch windows except for testing | 04:34 |
| Sewar | isn't mingw for use IN windows? how can I compile from linux then? | 04:37 |
| mithro | cygwin is for using Unix in Windows | 04:37 |
| mithro | mingw is a gcc compiler for targeting windows | 04:37 |
| *** hyades has joined #timvideos | 04:38 | |
| Sewar | so mingw can be installed in linux to compile for windows? | 04:39 |
| Sewar | that part is hidden in http://www.mingw.org/Welcome_to_MinGW_org | 04:40 |
| tpb | Title: Welcome to MinGW.org | MinGW (at www.mingw.org) | 04:40 |
| Sewar | "Using MinGW for Cross-Hosted Development" | 04:40 |
| Sewar | mithro: what do you recommend for compiling script (to handle multiple platforms, etc)? make? automake? configure? I never used any before so I have no idea where to look | 04:58 |
| mithro | for something as compilicated as your code, just make | 04:59 |
| Sewar | ok | 05:00 |
| mithro | Sewar, only problem I can see automake helping with would be AMD64 verse i686 stuff | 05:04 |
| Sewar | I see | 05:07 |
| *** iiie has joined #timvideos | 05:26 | |
| *** mithro- has joined #timvideos | 05:38 | |
| *** mithro- has quit IRC | 05:45 | |
| *** mithro- has joined #timvideos | 06:06 | |
| hyades | mithro: mithro- how do I send both python and c coverage reports to coveralls? | 06:20 |
| mithro- | hyades, I wouldn't recommend doing that | 06:21 |
| mithro- | hyades, have two build targets | 06:21 |
| mithro- | one for the C and one for the python | 06:21 |
| hyades | oh fine | 06:21 |
| mithro- | hyades, will explain more when I'm not in a meeting | 06:21 |
| hyades | i got it. | 06:22 |
| *** mithro- has quit IRC | 07:07 | |
| mithro | hyades, okay I have some time to explain further if you want | 07:14 |
| hyades | yeah | 07:26 |
| mithro | okay | 07:27 |
| mithro | you've read up about the build-matrix stuff that travis provides? | 07:28 |
| hyades | i was going through it, but not very clear about it | 07:30 |
| mithro | hyades, so the idea is that you have multiple different environments you want to test against | 07:31 |
| hyades | so the test runs multiple times? | 07:31 |
| mithro | yes | 07:32 |
| mithro | IE for web-animations project I work on at work | 07:32 |
| hyades | my whole build is divided into, first installing and then building | 07:32 |
| mithro | we want to test under the major browsers | 07:32 |
| hyades | sorry installing and then testing | 07:32 |
| mithro | even though we want to run the same tests, we want to run them against Firefox, Chrome, etc | 07:33 |
| mithro | we also want to run the lint and style checks separately | 07:34 |
| mithro | so this is what our travis.yml looks like | 07:34 |
| mithro | https://github.com/mithro/web-animations-js/blob/master/.travis.yml | 07:34 |
| tpb | <http://ln-s.net/-t-g> (at github.com) | 07:34 |
| hyades | so, the install and script are run for each of your browsers? | 07:35 |
| mithro | each one of the lines under matrix: is a different environment we want to run under | 07:35 |
| mithro | we then have a set up script (because it got to complicated to just keep in .travis.yml) which downloads the correct browser | 07:36 |
| mithro | https://github.com/mithro/web-animations-js/blob/master/.travis-setup.sh | 07:36 |
| tpb | <http://ln-s.net/-t-k> (at github.com) | 07:36 |
| mithro | it looks at the environment variables set via the matrix | 07:36 |
| mithro | lastly we have a run script | 07:36 |
| mithro | https://github.com/mithro/web-animations-js/blob/master/.travis-run.sh | 07:36 |
| tpb | <http://ln-s.net/-t-n> (at github.com) | 07:36 |
| mithro | which against looks at the environment and runs different things | 07:37 |
| hyades | yes :) | 07:37 |
| mithro | hyades, so you can see in the run script, if MODE=check we just do the linting, else we run the tests | 07:37 |
| mithro | so you'll probably want something which goes | 07:37 |
| mithro | matrix: | 07:38 |
| mithro | TESTING=api MODE=lint | 07:38 |
| mithro | TESTING=api MODE=test | 07:38 |
| mithro | TESTING=project MODE=lint | 07:38 |
| mithro | TESTING=project MODE=test | 07:38 |
| mithro | (I'm unsure if "project" is a good name - but something like that) | 07:38 |
| mithro | hyades, that make any sense? | 07:39 |
| hyades | yes, got it | 07:39 |
| mithro | hyades, there might not be a lint test for gst-switch core yet | 07:39 |
| mithro | maybe TESTING should be "MODULE" or something? | 07:39 |
| hyades | there is a lint test i guess | 07:40 |
| hyades | mithro: any hint on why i get the TypeError here | 07:41 |
| hyades | https://s3.amazonaws.com/archive.travis-ci.org/jobs/10881245/log.txt | 07:41 |
| hyades | in the end | 07:41 |
| mithro | hyades, so where are we at? | 07:53 |
| hyades | this TypeError doesnt come when I do the same in my quantal VM | 07:54 |
| mithro | hyades, probably an older version of gi.py | 07:54 |
| mithro | hyades, what is our current state, what goals do we have for this week? | 07:54 |
| mithro | hyades, and since we have 2 weeks left, what are we going to consider "success"? | 07:55 |
| hyades | the gcov part, I had edited the c file to generate coverage files when it receives SIGUSR1 | 07:55 |
| hyades | I have a coverage of c files. | 07:59 |
| hyades | currwently | 07:59 |
| hyades | though its not very great | 07:59 |
| hyades | http://paste.ubuntu.com/6054298/ | 07:59 |
| tpb | Title: Ubuntu Pastebin (at paste.ubuntu.com) | 07:59 |
| mithro | hyades, I think the travis environment is Ubuntu Precise | 08:05 |
| hyades | yes | 08:06 |
| hyades | the Gio docs i had found were for 12.10, | 08:06 |
| hyades | http://developer.ubuntu.com/api/ubuntu-12.10/python/Gio-2.0.html | 08:06 |
| tpb | <http://ln-s.net/-rBZ> (at developer.ubuntu.com) | 08:06 |
| hyades | https://docs.google.com/file/d/0B8gIq4778ozjdFFwdTdIOWhIUFk/edit?usp=sharing - html of the coverage | 08:07 |
| tpb | <http://ln-s.net/-t:A> (at docs.google.com) | 08:07 |
| mithro | hyades, can you upload it somewhere I can actually view it? | 08:08 |
| hyades | http://hyades.github.io/gst-switch/gcov/ | 08:10 |
| tpb | Title: LCOV - app.info (at hyades.github.io) | 08:10 |
| hyades | there maybe a better report, I will send you by evening | 08:11 |
| hyades | yes, this report has some problem. Will send you again. | 08:15 |
| mithro | okay | 08:22 |
| mithro | so | 08:22 |
| mithro | <mithro> hyades, what is our current state, what goals do we have for this week? | 08:23 |
| mithro | <mithro> hyades, and since we have 2 weeks left, what are we going to consider "success"? | 08:23 |
| hyades | hm | 08:23 |
| hyades | current state is that you can try out the build using the travis steps atleast on a quantal machine | 08:25 |
| hyades | it will work | 08:25 |
| hyades | this week, should be to improve the coverage of integration test in both the api and c code | 08:27 |
| mithro | but we don't have coverage ending up on coveralls yet? | 08:28 |
| mithro | so first I think we want to get the split travis build and coveralls upload stuff first? | 08:28 |
| hyades | i will do that by today | 08:29 |
| mithro | okay | 08:29 |
| hyades | so coveralls will show two coverage reports? | 08:29 |
| hyades | separate? | 08:30 |
| mithro | yes | 08:56 |
| hyades | ok | 08:59 |
| hyades | what else should be done? | 09:00 |
| hyades | and also "success" would mean what ? | 09:02 |
| mithro | well, that is what we need to figure out | 09:02 |
| mithro | our original goal was to | 09:03 |
| mithro | a) create a Python API for gst-switch | 09:03 |
| mithro | b) Use the Python API to make a reliable test-suite for gst-switch | 09:03 |
| mithro | right? | 09:03 |
| hyades | yes | 09:03 |
| hyades | it will test the major things in gst-switch controllable over dbus | 09:04 |
| mithro | okay | 09:05 |
| mithro | So for a) | 09:06 |
| mithro | I'd like to see the following | 09:06 |
| mithro | (these might all be finished already) | 09:06 |
| mithro | On every change, travis runs | 09:07 |
| mithro | * gst-switch Python API unit test coverage | 09:07 |
| mithro | * gst-switch Python API integration test coverage | 09:07 |
| mithro | * The unit test coverage should be 100% | 09:07 |
| mithro | Anything I missed there? | 09:09 |
| mithro | We should also move towards having a always "green master" | 09:10 |
| hyades | looks fine. (readthedocs maybe also) | 09:10 |
| mithro | hyades, yes - we should be publishing documentation automatically too | 09:10 |
| mithro | hyades, let me put this in a document somewhere :) | 09:10 |
| mithro | hyades, https://docs.google.com/a/mithis.com/document/d/16uyLZXdzyqEN1mYptT4lyrX_53KWXQ90CXr0RD7fmtI/edit#heading=h.riihu2tdval1 | 09:16 |
| tpb | <http://ln-s.net/-tBG> (at docs.google.com) | 09:16 |
| hyades | yes | 09:17 |
| hyades | the lint checker runs when you edit a c file. It checks if the code style is fine. | 09:24 |
| mithro | hyades, I'm not sure what you mean? | 09:25 |
| hyades | there are a couple of scripts in scripts/apps | 09:26 |
| hyades | https://github.com/hyades/gst-switch/blob/python-api/scripts/apps/app_lint.sh | 09:26 |
| tpb | <http://ln-s.net/-tBu> (at github.com) | 09:26 |
| hyades | https://github.com/hyades/gst-switch/blob/python-api/scripts/apps/app_indent.sh | 09:26 |
| tpb | <http://ln-s.net/-tBv> (at github.com) | 09:26 |
| mithro | hyades, how does a person run them? | 09:27 |
| hyades | when you commit code, it runs these tests | 09:28 |
| hyades | it doesnt commit unless you have resolved the issues | 09:28 |
| mithro | hyades, okay | 09:28 |
| mithro | I'm pretty sure you can override that | 09:29 |
| mithro | hyades, how does that look? | 09:31 |
| hyades | Code style okay! | 09:31 |
| hyades | on success :) | 09:31 |
| hyades | https://github.com/hyades/gst-switch/blob/python-api/scripts/hooks/pre-commit | 09:32 |
| tpb | <http://ln-s.net/-tC$> (at github.com) | 09:32 |
| mithro | hyades, can you think of anything which I've missed? | 09:34 |
| mithro | does the second section look okay? | 09:35 |
| hyades | Create a "how to write tests for gst-switch" document | 09:36 |
| hyades | using the API? | 09:36 |
| mithro | Yes | 09:36 |
| hyades | looks good | 09:37 |
| hyades | regarding the master branch | 09:37 |
| hyades | some dbus variables need to be changed for master branch. Same code will work, parameters like the interface, object path should be provided. Defaults wont work. | 09:37 |
| mithro | so, we can redefine what the master branch is | 09:37 |
| mithro | I think it probably makes sense to make master the branch you are working off of | 09:38 |
| hyades | ok | 09:39 |
| mithro | hyades, so I assume we still haven't started the | 09:40 |
| mithro | Tests should exist which exercise the major parts of gst-switch | 09:40 |
| mithro | ? | 09:40 |
| hyades | well, i need clarification here | 09:40 |
| hyades | a code which tests PIP modes tests gst-switch functionality? | 09:41 |
| hyades | same for other methods | 09:41 |
| mithro | hyades, code which tests all the functionality | 09:45 |
| hyades | mithro: "Remove the currently failing, unreliable tests." which tests are these? | 09:55 |
| mithro | hyades, the ones in the tests directory | 09:55 |
| hyades | the one which runs by doing make demo? | 09:56 |
| hyades | last time I tried them, they were working I guess | 09:56 |
| mithro | I don't think they work reliably | 09:58 |
| mithro | hyades, some performance tests would be good too | 10:01 |
| hyades | I have a parameter in most of my integration tests called FACTOR. increasing its value will lead to loading the system. | 10:03 |
| hyades | system and gst-switch | 10:03 |
| mithro | what does that mean? | 10:03 |
| hyades | https://github.com/hyades/gst-switch/blob/python-api/python-api/tests/integrationtests/test_controller.py | 10:04 |
| tpb | <http://ln-s.net/-tCh> (at github.com) | 10:04 |
| mithro | hyades, "FACTOR is badly named" | 10:04 |
| mithro | hyades, it should be something like "number of connections" | 10:04 |
| mithro | There are multiple ways to increase the load | 10:05 |
| mithro | a) increase the number of input streams | 10:05 |
| mithro | b) increase the resolution of the input streams | 10:05 |
| mithro | c) cause lots of PIP transitions | 10:05 |
| hyades | ok | 10:06 |
| hyades | a) and b) can be handled by changing NUM and FACTOR | 10:07 |
| mithro | hyades, you shouldn't have to change anything | 10:07 |
| mithro | hyades, and those are really bad names | 10:07 |
| mithro | hyades, and not really documented? | 10:07 |
| hyades | yes, will change names | 10:07 |
| hyades | but,what should I confirm by doing these tests? | 10:08 |
| mithro | hyades, what do you mean? | 10:08 |
| hyades | cannot do frame comparisons for these tests. It takes lots of time | 10:08 |
| mithro | "torture tests" probably shouldn't run on travis | 10:09 |
| hyades | even on my laptop, each frame comparison takes around 1.5 secs | 10:09 |
| hyades | cannot do it for something like rapidly changing PIP modes | 10:10 |
| mithro | hyades, the comparision should happen *after* the test is finished | 10:10 |
| hyades | okay, fine! | 10:11 |
| mithro | If you change 100 times within 1 second, that is till only a couple of minutes to check the result | 10:11 |
| mithro | hyades, torture tests are about making sure the program still works under heavy load | 10:11 |
| hyades | regarding multiple times a second. The gst-switch-srv crashes if the time between two PIP mode change requests is less than 0.6 secs (0.5 it crashes) | 10:12 |
| hyades | crash = seag fault | 10:12 |
| mithro | hyades, well that is a bug! | 10:12 |
| mithro | hyades, and exactly the type of bug we want to find | 10:13 |
| mithro | hyades, so - your working as expected :) | 10:13 |
| hyades | k | 10:14 |
| mithro | hyades, oh | 10:14 |
| mithro | one bigger thing I only just remebered | 10:14 |
| mithro | hyades, I'd really like to see if we could migrate from the script/stage.sh thing to using gnome's jhbuild | 10:15 |
| mithro | hyades, as I think that would make it more reliable/reproducable | 10:15 |
| hyades | i dont use stage script anymore | 10:15 |
| mithro | and less stuff for us to maintain | 10:15 |
| hyades | just a autogen is enough | 10:15 |
| mithro | hyades, ? | 10:15 |
| mithro | hyades, how do you get the dependencies? | 10:16 |
| mithro | hyades, or is this on travis? | 10:16 |
| hyades | yes it is | 10:16 |
| hyades | in the .travis.yml file | 10:16 |
| hyades | one patch is required | 10:16 |
| hyades | https://github.com/hyades/gst-switch/commit/d7d1b8e6afc1643842fb917260c50d1b2b41daa4 | 10:17 |
| tpb | <http://ln-s.net/-ro3> (at github.com) | 10:17 |
| hyades | this path should be the one specified in the prefix argument to autogen.sh | 10:17 |
| mithro | hyades, both the original and your patch are wrong | 10:18 |
| hyades | yes, i know | 10:18 |
| hyades | i need the prefix there | 10:18 |
| mithro | I don't know automake/configure to give you the right value there | 10:18 |
| mithro | but I'm sure there is a help channel here on IRC that could help you out | 10:19 |
| hyades | ok | 10:19 |
| hyades | so is jh-build still needed? | 10:25 |
| mithro | hyades, yes | 10:28 |
| mithro | hyades, I'm not sure how you are building gst-switch without jh-build? | 10:28 |
| hyades | I am first installing all dependencies, then installing all gstreamer + plugins | 10:29 |
| mithro | by | 10:29 |
| mithro | install all gstreamer + plugins, you mean? | 10:29 |
| mithro | downloading from git and compiling them? | 10:30 |
| hyades | yes | 10:30 |
| mithro | hyades, so you manually reproduced what the stage script use to do? | 10:30 |
| hyades | the stage script doesn't provide the PKG_CONFIG_PATH | 10:31 |
| hyades | the stage script needs to be edited. | 10:32 |
| mithro | hyades, your welcome to edit it | 10:34 |
| mithro | hyades, but I think we should dump the stage script for jh-build - I think that would more reliably build the enivornment we need and is better maintained | 10:36 |
| mithro | I'm going to head home so I can do the washing I didn't do on the weekend. I'll be back in about 30 minutes. | 10:37 |
| hyades | I will look into it :) | 10:37 |
| *** mithro- has joined #timvideos | 11:14 | |
| mithro- | hyades, back now | 11:31 |
| mithro- | but my internet seems stuffed | 11:31 |
| mithro- | hyades, so are you happy you know what to do? | 11:32 |
| hyades | :) | 11:32 |
| *** mithro- has quit IRC | 11:47 | |
| *** mithro- has joined #timvideos | 11:47 | |
| *** mithro- has quit IRC | 11:53 | |
| *** hyades has quit IRC | 12:01 | |
| *** mithro- has joined #timvideos | 12:10 | |
| *** mithro- has quit IRC | 12:51 | |
| *** hyades has joined #timvideos | 12:55 | |
| *** mithro- has joined #timvideos | 13:08 | |
| *** mithro- has quit IRC | 13:39 | |
| *** hyades has quit IRC | 13:40 | |
| *** hyades has joined #timvideos | 14:04 | |
| *** hyades has quit IRC | 14:24 | |
| *** hyades has joined #timvideos | 16:23 | |
| *** hyades has quit IRC | 16:44 | |
| *** hyades has joined #timvideos | 16:55 | |
| hyades | mithro: gcov reports for c files after doing all the tests - http://hyades.github.io/gst-switch/gcov/tools/ | 18:05 |
| tpb | Title: LCOV - out.infotools (at hyades.github.io) | 18:05 |
| hyades | around 70% coverage (Rest of the c files are not executed) | 18:06 |
| hyades | irc logs no longer work? | 19:04 |
| hyades | mithro: the docs will be hosted here http://gst-switch.readthedocs.org/en/latest/ | 21:33 |
| tpb | Title: Welcome to gstswitch’s documentation! gstswitch documentation (at gst-switch.readthedocs.org) | 21:33 |
| mithro | hyades, IRC logs should still work? | 23:48 |
| mithro | hyades, they where still being logged just the website wasn't updating | 23:51 |
| mithro | fixed now | 23:51 |
| mithro | hyades, so which files are not tested? | 23:52 |
Generated by irclog2html.py 2.12.1 by Marius Gedminas - find it at mg.pov.lt!