These are chat archives for bjz/gfx-rs

16th
Jul 2014
Dzmitry Malyshau
@kvark
Jul 16 2014 01:14
@csherratt agreed. @bjz - we need to make a list of points to advertise gfx ;)
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:16
Yeah I agree
marketing is important
I will make an issue
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:27
@csherratt @kvark #107 and #108 made!
Dzmitry Malyshau
@kvark
Jul 16 2014 01:37
@bjz awesome! one thing though, you created T-discussion, and I believe T-question was supposed to be the same thing. Perhaps, we can remove one and rename the other?
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:37
ahh
dunno :/
maybe we could just remove question
or discussion
Dzmitry Malyshau
@kvark
Jul 16 2014 01:37
Also, any chance of answering my comments on #106?
brendanzab @bjz doesn't know
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:38
@kvark I am working on it now!
@kvark sorry - was pretty focused on work today
Dzmitry Malyshau
@kvark
Jul 16 2014 01:38
np, just want to make sure I'm not blocking you
I removed T-question, so this is resolved
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:42
What is the meaning of FloatPrecision?
does it lock to 64bit?
Dzmitry Malyshau
@kvark
Jul 16 2014 01:43
GL does, yes
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:44
what is the behaviour of the others?
Dzmitry Malyshau
@kvark
Jul 16 2014 01:45
others? there is the default and the precision. Precision is meant for attributes that are declared as double on GLSL side. Everything else is declared as float
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:46
so F16, F32, F64 all convert to floats for use on the hardware?
Dzmitry Malyshau
@kvark
Jul 16 2014 01:46
yes, GLSL sees them as floats
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:51
@ozkriff ohai!
Brendan Zabarauskas
@brendanzab
Jul 16 2014 01:57
@kvark rebased #106
arrrgh travis why you do this
Dzmitry Malyshau
@kvark
Jul 16 2014 02:02
one last (minorish) concern...
Corey Richardson
@cmr
Jul 16 2014 02:04
Gaaaahhhh gitter wasn't sending me new messages :(
Dzmitry Malyshau
@kvark
Jul 16 2014 02:04
@cmr ouch. @trevorah ^^^
Brendan Zabarauskas
@brendanzab
Jul 16 2014 02:04
yeah, I find you need to refresh every day at least
brendanzab @bjz just has a pinned tab open in firefox
Brendan Zabarauskas
@brendanzab
Jul 16 2014 02:05
I like the little activity indicators
and the OS messages it gives me
Dzmitry Malyshau
@kvark
Jul 16 2014 02:06
I just close the tab whenever I leave.
Brendan Zabarauskas
@brendanzab
Jul 16 2014 02:06
(integrates with notification center)
Corey Richardson
@cmr
Jul 16 2014 02:12
@bjz let's talk here :P
Brendan Zabarauskas
@brendanzab
Jul 16 2014 02:13
@cmr my covert marketing strategy
:D
ok, heading home. bye!
Corey Richardson
@cmr
Jul 16 2014 02:15
I hope to have travis working again by the time you get home
Corey Richardson
@cmr
Jul 16 2014 02:40
@bjz see the fix-travis branch, that seems to do it.
I guess the PATH was just messed up.
Corey Richardson
@cmr
Jul 16 2014 02:48
Still having some problems with the ordering of the sending of the ESC key and the program itself it seems....
Dzmitry Malyshau
@kvark
Jul 16 2014 02:51
@cmr weird, I haven't seen anything wrong with it after that last patch. What do you get examply?
Dzmitry Malyshau
@kvark
Jul 16 2014 02:53
hmm... need to check it locally here
Dzmitry Malyshau
@kvark
Jul 16 2014 03:13
@cmr could you look at that please? https://gist.github.com/kvark/adb5c7b725fcf95cab87
I think it's confused by Err inside an Err arm and expects the wrong type
d'oh, sorry, my bad - name collision
Corey Richardson
@cmr
Jul 16 2014 03:25
@kvark your problem is resolved?
Dzmitry Malyshau
@kvark
Jul 16 2014 03:26
yeah, it was dumb enough :)
Dzmitry Malyshau
@kvark
Jul 16 2014 03:49
r? #109
Corey Richardson
@cmr
Jul 16 2014 03:49
Reviewing.
Dzmitry Malyshau
@kvark
Jul 16 2014 03:55
hmm... Travis commands work fine for me locally. Must have something to do with the headless mode. Can we get Xorg log?
Corey Richardson
@cmr
Jul 16 2014 03:55
probably, cat /var/log/Xorg.*.log ?
Dzmitry Malyshau
@kvark
Jul 16 2014 04:00
I know what the problem is. xdotool exited before the app even started
Corey Richardson
@cmr
Jul 16 2014 04:00
shouldn't the sleep prevent that?
Dzmitry Malyshau
@kvark
Jul 16 2014 04:00
perhaps, travis ignored & at the end? Maybe it needs special treatment
Corey Richardson
@cmr
Jul 16 2014 04:00
maybe :(
Dzmitry Malyshau
@kvark
Jul 16 2014 04:01

The command "sh -c "sleep 5; xdotool key --window \"$(xdotool search --name Hello)\" Escape"" exited with 0.

That is reported before we start the example

Corey Richardson
@cmr
Jul 16 2014 04:01
weeeird.
it has worked before
its the only way tests have been passing up to this point
Dzmitry Malyshau
@kvark
Jul 16 2014 04:02
don't know what to tell, the current message is clear...
Dzmitry Malyshau
@kvark
Jul 16 2014 04:07
I'm off for today :zzz:
Brendan Zabarauskas
@brendanzab
Jul 16 2014 04:51
@cmr still not working?
Corey Richardson
@cmr
Jul 16 2014 04:56
@bjz I have no idea.
@bjz the -E to sudo fixed the rustup thing
but xdotool seems to be doing weird things
Brendan Zabarauskas
@brendanzab
Jul 16 2014 04:57
do you want to just disable it for now?
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:05
@cmr can we disable the headless stuff for now?
Corey Richardson
@cmr
Jul 16 2014 06:08
@bjz yes
that seems reasonable
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:11
@cmr could you submit a PR?
then I can merge my other thing
Corey Richardson
@cmr
Jul 16 2014 06:14
@bjz fix-travis branch
r?
oops pushed wrong
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:14
is it fixed?
oh
Corey Richardson
@cmr
Jul 16 2014 06:14
should be!
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:14
yeah, didn't see it ->
Corey Richardson
@cmr
Jul 16 2014 06:14
travis will tell...
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:14
ah, I see
Corey Richardson
@cmr
Jul 16 2014 06:14
see -->
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:15
I can cherry pick it onto my PR
Corey Richardson
@cmr
Jul 16 2014 06:15
ok
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:17
@cmr hmm - errored
Corey Richardson
@cmr
Jul 16 2014 06:19
I doin't know what your branch is doing but it doesn't have my changes.
since it isn't running sudo -E
Brendan Zabarauskas
@brendanzab
Jul 16 2014 06:19
no ->
oooh
wait
Corey Richardson
@cmr
Jul 16 2014 06:19
My branch passed :P
Andy Trevorah
@trevorah
Jul 16 2014 09:51
@cmr sorry about the new messages not reaching you. We’ve just made some changes to gitter’s relatime messaging that should fix that, but if you need to refresh again, then let us know
Andrey Lesnikov
@ozkriff
Jul 16 2014 11:02
@kvark , @bjz hi :) gfx-rs looks interesting. I will try to use it in Marauder later.
Dzmitry Malyshau
@kvark
Jul 16 2014 12:19
@cmr updated #109, notice the waffle icon
@bjz #106 needs a rebase...
Brendan Zabarauskas
@brendanzab
Jul 16 2014 15:10
@kvark good to merge #106?
@kvark got waffle pinned now :)
Dzmitry Malyshau
@kvark
Jul 16 2014 15:14
@bjz travis too busy to build it?
Dzmitry Malyshau
@kvark
Jul 16 2014 15:20
These pins look to be all over the place.... "build | passing" and "GITTER | bjz/gfx-rs" are entirely opposite styles. We should either name the services explicitly (i.e. "TravisCI | Passing"), or not mention them in favor of simple names (i.e. "chat | <number of people in the room>").
Brendan Zabarauskas
@brendanzab
Jul 16 2014 15:21
Dzmitry Malyshau
@kvark
Jul 16 2014 15:21
watching it...
fingers crossed
uh oh a warning
Brendan Zabarauskas
@brendanzab
Jul 16 2014 15:24
\o/
yeah...
I reckon fix it in another commit
it's my fault
Dzmitry Malyshau
@kvark
Jul 16 2014 15:25
@bjz Thanks!
ouch, no I need a rebase :)
Brendan Zabarauskas
@brendanzab
Jul 16 2014 15:26
I have had a cursory look at #109 and didn't really understand it :P
I'll look again
Dzmitry Malyshau
@kvark
Jul 16 2014 15:26
well perhaps I'll fix the readme pins with the merge then, as long as we can agree on a scheme
@bjz #109 brings 2 things:
  1. Draw errors are carefully processed in draw, and a Result is returned to the client
  2. Resource loading errors are kept in the queue for the user to fetch and iterate
The Dispatcher thing is there for convenience, it handles everything we receive from the device (resources and errors). Renderer used to have "resource-related methods implementations" (or something similar) impl section, which is now simply a impl Dispatcher
Brendan Zabarauskas
@brendanzab
Jul 16 2014 15:51
@kvark Awesome, thanks for the info
Brendan Zabarauskas
@brendanzab
Jul 16 2014 16:00
@kvark unfortunately I will have to look after work
Dzmitry Malyshau
@kvark
Jul 16 2014 19:38
@cmr how is texturing going so far?
Dzmitry Malyshau
@kvark
Jul 16 2014 20:33
@cmr btw, I was wrong about Output Merger. It does include depth-stencil state. So what we have now is combining D3D rasterizer and OM states. I'm rethinking it now.