You can use shorthand routes `category` instead of `category_path(category)`


In `seeds.rb` I would recommend using bang version of `create` as you ll get notified on validation error while now you won't.


The commit message does not explain the reason behind changing ``lbs/year`` to ``lbs``, and it broke a test. If ``lbs`` is the correct unit, we can just fix up the test.


Could you also add the appcast right after the `url`?
```ruby
appcast 'https://github.com/tatey/Lighting/releases.atom'
```


It seems you changed much too :) Positive too, same. And thanks for the pat on the back


I'm not sure I understand the article's reasoning for why tabindex > 1 is discouraged:

> It imposes a tab order on the content that bears no resemblance to the expected tab order.

Everything is hidden in our DOM tree, so there is no "expected" tab order.

Then they have an example where they set the tab order to the password first before username.

Is there a more detailed argument about why to avoid it? For instance, it seems performance would be faster by changing tabindex, instead of dynamically rearranging DOM elements.

Also, our current template for the sim HTML is present at https://github.com/phetsims/chipper/blob/master/templates/sim.html, presumably the prototype could use a similar HEAD, and proposals for different ways to structure the HTML should be based off of that.

It's important to note that the entire sim display is created dynamically, so there is no element in the HTML source for it.


Thanks


There is no reason to pull `@videos` and `@categories`, you can reach all videos from a single category in Template as you iterate over the collection. Associations are the key.


The problem is that this allows the downloading of files on the GitLab server's local network, like localhost/127.0.0.1 or other machines on the same network. Could you add a check to make sure the file is actually at the Fogbugz domain?


...de ce guide **ne** seront pas écrits...


const T&


Isn't there a better way of doing this? I presume this creates multiple new files just so that one attribute can be changed on text boxes.

If so, ISTM we would be better creating a region-base resource file with styles...


Make sure this file stays in `.gitignore`. Your username should be confidential and this file is now public. Definitely not for everyone's eyes.


Done.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.523](http://teamcity.monogame.net/viewLog.html?buildId=9789&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Thanks for fixing that, @renestalder. I was trying to learn Grunt and postcss yesterday, and that example was giving me nightmares, hahaha. Hopefully I'd made it myself. I was coming here today to contribute, but I'm glad you've already done it. Cheers! Hugs.


A helper function is probably needed


infoon originally appended site:wikipedia.org, without a space causing odd query results (this was fixed in the previous commit). infun purposefully does not add the space because reimplementing the odd query results was a requested feature.

This should be made more obvious. It would probably be a good idea to write it as "question + queryModifier". Where queryModifier is site:wikipedia.org with or without a space. The duplicate code between the the infoon and infun if statements should also be extracted out.


develop (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


I'm not sure if you need to check `sol_vector_get()` here. (or `get_node()`)


I miss you, bats united


goodbye drone pigeons! live a long life!


si, eso lo genera el visual paradigm, no me deja poner el final, lo hago a mano en el eclipse y punto.


And, as noted on that issue, 200 is expected and correct at that point in the app life cycle.


Need to hide the x (should be &times also) with an aria-hidden=true.


OMG! :+1:


Throwing the UnsupportedEncodingException and returning the default answer makes much more sense. URL also does in fact look like a constant, which should also be changed.

Thank you for these suggestions @omajid!


Yes, I'll fix that.


I guess it's a hackish way to implement some left padding for the action bar title. See the following screenshots of the action bar with and without this change:

**With leading whitespace:**
![title_padded](https://cloud.githubusercontent.com/assets/4244109/9372128/78ccfeda-46f4-11e5-8906-97e7b3706a83.png)

**Without leading whitespace:**
![title_unpadded](https://cloud.githubusercontent.com/assets/4244109/9372134/832fa9d6-46f4-11e5-9ec1-598217de5808.png)


This doesn't quite behave right. When you do this on i3: open terminals A, B, C. Float C. Focus B. Toggle mode, toggle mode. B should be focused.


var að testa Git... Færði create league í nytt template


rax retry tests


a) Any idea how many times this can happen in a 15+0.05 game ?
b) Have you ever tried to reset the stats after each pawn or King move ?


Ah, I see what your getting at here. Should we keep the DownloadService class generic and do the check before using it, or should we create some kind of whitelist in DownloadService?


@cheatiiit You shouldn't submit files such as logs or .jar. Remove them and amend the commit using `git commit --amend`


Yep, this is really a fundamental thing about Open-Closed and abstraction in general: *somebody* has to wire the concrete dependencies into the running system. Usually that's main, or something like it. In gigantic enterprise Java apps that means the dependency injection framework. In your test suite, some test file somewhere is operating as the main. So one option is to put it in whatever the "main" is for your web server.


Fixes #226


Looking forward to playing with these changes, I was about to fork your repo and do it myself. Let me know if you need help. @bevanhunt


Стыдно :)


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-7.21%25-green.svg)


ConfigureTransitionIDDname (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


run script for GROMACS 5.0.4


Interesting. What kind of optimization is this?

Can you perhaps add a build folder as well, somewhere we can put all the *.o files?
So compiling would be faster?


I think a whitelist makes sense for now. It's really only supposed to be used with importers.


Example script of making plots from specific input information.


@mosic added `*.dSYM` here to, it's used for dynamic linkage on the Mac platform.


So - I think this changed occurred because we changed how RNGs where allocated
which changed how they are accessed ...

We can look at it more ...just to confirm ..

thanks,
Chris

----------------------------------------------------------------------------------------------
Christopher D. Carothers

Director, Center for Computational Innovations
Professor, Department of Computer Science
Rensselaer Polytechnic Institute
110 8th Street
Troy, New York 12180-3590

e-mail: chris.carothers@gmail.com
web page: www.cs.rpi.edu/~chrisc
phone: (518) 276-2930
fax: (518) 276-4033
----------------------------------------------------------------------------------------------

________________________________________
From: Justin M. LaPre [notifications@github.com]
Sent: Wednesday, August 19, 2015 6:21 PM
To: carothersc/ROSS
Cc: Carothers, Christopher D.
Subject: [ROSS] Is it possible we missed this for all these years? (45665b4)

Is it possible we missed this for all these years?

@carothersc Can you look at this, Chris?  It was brought up by Jae-Seung at LLNL.

---
Reply to this email directly or view it on GitHub:
https://github.com/carothersc/ROSS/commit/45665b48a649b136f173a47916ab231c6cd92a07


bzw. `@EventHandler`


Is it possible to do something like this from travis?

```docker run --device=/dev/fuse --cap-add=SYS_ADMIN /bin/sh -c "make; make -C test;"```


ConfigureTransitionIDDname (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.95%25-green.svg)


could change this to the interface net.Addr and support udp. :heart_eyes:


Where will PATH_MAX be defined?


Hey @flosell

Any changes to push 1.3.0 to rubygems.org? Last one there seems to be 1.2.5 - https://rubygems.org/gems/marathon-api

Cheers


PRs welcome :wink:


WWD_DOES_NOT_EXIST will equal 1005, while WWD_SUCCESS will equal 0.  Changes above keep wlan_select_antenna_impl() from returning -1 on error.  I'd like to ensure it returns -1 on error if possible.


Thank you!


Vale

2015年8月20日木曜日、JSAURAJ<notifications@github.com>さんは書きました:

> si, eso lo genera el visual paradigm, no me deja poner el final, lo hago a
> mano en el eclipse y punto.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JSAURAJ/AlmacenesElChollo/commit/037a3de092eebd0bd4083e6e6e9c9e25ecb0217b#commitcomment-12797142>
> .
>

--

Antonio Jose


hey, it looks like this causes a small bug for those who _don't_ want tsx support.
existing non-tsx projects fail with
```
Errors occurred during the build.
Errors running builder 'TypeScript Builder' on project 'slate-module-native'.
No enum constant com.palantir.typescript.services.language.JsxEmit.
```


4590-VRF-defrost-power-calculation-problems (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1798 of 1798 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Indeed I hadn't realized.
So since I already had a localeIdentifier field on the Installation class, I just checked with the data browser and things are not looking good...
I don't know if this is the best place to report this, but here is what I have :
 - When browsing the data browser everything looks like locale identifiers are in the form en-US (or fr-FR, etc...). At first glance it looks like you automatically converted underscores into dashs (without telling anyone ?).
 - When looking more closely it seems it's just a UI trick because, when making queries for en-US, I only get objects created after August 5, and when making queries for en_US I only get objects created before August 5. So changing underscores into dash only started on August 5 ?!
 - But event when making queries for en_US (using the data browser or the REST API) objects are all returned with en-US (before and after August 5).
 - And on top of that it seems that values that were not in the en_US format got a special treatment (sometimes there is a @calendar=... at the end e.g : ja_JP@calendar=japanese). From what I can gather, they are unchanged for objects created before August 5 and empty for objects created after...
- Of course my app has always sent and is still sending values in the en_US format, I'm using [[NSLocale currentLocale] localeIdentifier] and the version on the store is using the version 1.7.5 of the SDK.

I'm sorry, I guess you tried to make the change as seamless as possible, but it didn't work... I have cloud code that rely on this value that is now broken, and I'm probably not the only one. So if it's not too much to ask, what exactly did you do ?


And also to the team signature on the Core Team Docs which we use for external PR communication


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/19351/testReport/) for this commit**


This seems to break BC - and lichess :)


@levicole made the change for the db, ran rake db:create, rake db:migrate and rake db:seed


Yes, the logger may be null otherwise.


@danurna I was getting a crash pretty often whenever the data was nil, using the other method doesn't crash


codigo temporario


Wow, that was _fast_ !!
I like that it returns success indicators directly from `produce`, that feels right. Worth changing the interface for I think. It may be unintuitive though that when you disregard the return value, it basically gets stuck there?

I'm also thinking about the rdkafka equivalent which I haven't totally figured out yet. There, the "delivery reports" are really async (they come as callbacks), so it wouldn't be possible to match this interface - the information isn't there yet when you return from `produce`.

Could we define some sort of future/promise object, that we can return from `produce`? (That could also carry any failures occurring in `_send_request`?)


@levicole made these changes but something seems to have broken along the way. Do I also need to update the javascript in my application.js file?


Many thanks.

We need Sky Arts/HD if possible.


The reason I changed it is that the convention is 0 is success, other values are errors. (Sure we could negate the error to ensure it's negative.) I feel it's useful to propagate as much information out rather than swalling the error code and returning a generic -1.


Etc is not very helpful. They need to be added to
* github
* ADT CLA google group
* shared google drive folder
* tracker
* ipm, retro, standup, feedback time calendar invites


@levicole should .save be replaced with the class I was using previously to capture the event?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.523](http://teamcity.monogame.net/viewLog.html?buildId=9789&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:23:58


kkkkkk


@levicole the class in this view is 'url'


Consider just returning the current budget either in a callback or promise (https://docs.angularjs.org/api/ng/service/$q) instead of passing vm. So, in the case of a promise, the call in the controller would be like: dashboardFactory.getLatestBudget().then(function (budget) {vm.currentBudget = budget; vm.openToast();});


@levicole the class in this view is 'sign-up'


oops :) thanks


Am I missing where `connection_failures` is incremented?


Similar comment here about passing vm. The factory becomes less useful when the caller needs to pass its implementation details such as UI updates i.e. openToast.


It's because Python expects to see either `assert expr` or `assert expr, message`.
Apparently when it sees `assert (expr, message)` it interprets `(expr, message)` as a tuple literal which, being non-empty, is considered truthy. I guess that is the narrative that explains the warning we see here.


make sure @d-reinhold is added too


Nope! Great catch!!! I'd originally had it in the if (++connection_failures...) but changed it. Doh'. pushing now.


Pffff, I can't keep up with this pace! Awesome :)
One other place this threshold comes up is in `resolve_event_state`.


I hesitate to even make this comment but a common convention is to declare library modules first i.e. ($mdToast, dashboardFactory). Total nitpick, sorry.


Whoops, sorry - just pushed a fix


TeamCity Xbim :: Xbim Property Sets :: Xbim Property Sets [Build 4](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=110&buildTypeId=Xbim_XbimPropertySets_XbimPropertySets) is now running


TeamCity Xbim :: Xbim Property Sets :: Xbim Property Sets [Build 4](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=110&buildTypeId=Xbim_XbimPropertySets_XbimPropertySets) outcome was **SUCCESS**
Summary: Running Build time: 00:00:29


*En attendant il te suffit de commenter les 3 lignes de debug d'encoding.*

je comprends pas


Hrrrm. Is there a less hacky way we can do that?


98318570_RemoveRadianceCheckbox (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Or more like:

 #if (QT_VERSION >= QT_VERSION_CHECK(5, 5, 0))

then add these QDataStream?


Можно сделать метод нестатическим и передавать в него только `freePlace`. `tilePlace` и так будет внутри у каждой фишки.


TeamCity pull requests :: netty [Build 76](https://garage.netty.io/teamcity/viewLog.html?buildId=551&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5159, ignored: 21 Build time: 00:52:52


Like, I'm ready to play around with 1.0 immediately, let me know!


Можно сначала сделать `places.set()`, а потом вызвать `updateTiles()`, чтобы не обновлять их вручную.


triple eq !==


Cool, that's exactly the kind of thing I've been looking for. I spent ages trying to get this to work. In the end, the ```vm.openToast()``` was used as a hack to force the UI to update.


Since you fixed, do you know if we still need the $timeout?


The speedup here is just correcting a regression from https://github.com/JuliaOpt/JuMP.jl/commit/49e6aa96677785373057c2967e6e6c82c4702402, right? Or are we faster than we were a day ago?


cc #19


Not a big fan of the genric home symbol, mainly because there's nothing that it relates to on the site. Was thinking that the dotted circle could be a stand in logo as it's kind of a target and can also be used as a checkbox.


Won't the LHS of this expression always be false?


Looks good. Does it run?


Apologies for the delay in reply.  One quick question: should we remove the import of PrimMonad?  Or is it used somewhere else in the code?


(I know this is just moving code, but) please delete this comment.  We're definitely using the batch delete API below.


This comment also wants to get gone.


Templated specs!? @jkburges, you're mad!


1) OK, I'll make permanent explicit for spells.

2) I like permanent T/F, vs. a type, because permanent can be layered with other logic, and is easy to code for.

3) I like the target idea.


This could just be `dataCollection = {}`, really.


Okk, mañ lo agrego
El ago 18, 2015 8:26 p.m., "cristiaan3003" <notifications@github.com>
escribió:

> git no esta ignorando los archivos copia que hace linux por ejemplo este
> NuevoDocumento, me creo un NuevoDocumento~ oculto que lo borre para que no
> me lo agregue al repositorio. agregar la regla para que no estos archivos,
> pienso que con un *.*~ seria suficiente.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/loquinho33/IC-FICH-2015/commit/13eeb8297a4b353a0d6086ec8ecf9e8ed48f8184#commitcomment-12773047>
> .
>


可折麽死我了,终于找到了,谢了,赞一个


cleanup edits


Link broken


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-70.9%25-red.svg)


TeamCity pull requests :: netty [Build 58](https://garage.netty.io/teamcity/viewLog.html?buildId=502&buildTypeId=pulls_netty&guest=1) is now running


m to MatrixInverse for clarity


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - Win64-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-70.9%25-red.svg)


i would propose setting `bits = false`


Just fixing a regression. We can probably get even faster than before by going back to ``String[]`` and using ASCIIString most of the time


TeamCity pull requests :: netty [Build 57](https://garage.netty.io/teamcity/viewLog.html?buildId=501&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 4745, ignored: 16; exit code 1 (new) Build time: 00:09:23


oh sorry i thought you were asking what was the value (driver) of the button 12 (mapping). Let me check again, but based on the actually mapping i did, it seems like none of them.


Indeed.  As has been discussed in the issue tracker, this is clearly a heuristic but that said, the idea of "free memory" is a bit wiggly to begin with.  This seems to be the best solution for the time being so I'll add a pull request for it.


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


init


commit comment


Julia string situation is muddled to say the least


I touched up `data/audio/demo/theme.ogg` and `data/audio/demo/dungeon.ogg`, and I also added a new demo: `data/audio/demo/raid.ogg`. Criticism appreciated.


test


debate: idiomatically I'm not sure if I like this being a setter that our owner needs to set.  The react.js "props" mechanism (including componentWillReceiveProps and friends) is brilliant for stuff like this.  Arguably our onArgs() mechanism and our standardized use of EventEmitters is somewhat similar, although with the introduction of "dom_evt", things are getting a bit diverse.  (Although I think there is an argument for keeping the DOM events in the DOM and just morphing them as you're doing without forcing all EventEmitter stuff to happen via DOM dispatchEvent)

A few thoughts:
* Given that data-event and data-prop are aware of their containers (thanks for the explanation of that, btdubs), maybe it wouldn't be insane to have some card-level event-bus using evt?  Maybe even just exposing the card's itself?  It seems like having this folderType thing be a: ```ownerCard.on('folder', (folder) => { ... })``` would be reasonably clear, and the point at which the owner explicit emits things is arguably intuitive for a reader to know they have to go fish.
* The model stuff is also pretty standard here; maybe it knows about the model?
* ... other.


Option 2 does not work for me. I often need to work on some 3rd party modules which are installed under site-packages. So they can be imported easily. At the same time, I have another copy of these modules sitting in a working directory, which is not part of `sys.path`. This setting allows me to modify the code in the working directory without affecting the stable version in site-packages. The modified code only become effective when I explicitly run them in the working directory (because dot takes effect before site-packages). If the dot is to be moved to the end of `sys.path`, it would break my workflow.

So maybe as a temporary solution, we can rename `copy.py` to `pbcopy.py` and `paste.py` to `pbpaste.py` to avoid conflict and the new names are consistent with their OS X counterparts.


@motivatedHamster Tests are running cleanly now.


Today I test other setting.It will work if I set post status as draft even post title value is Chinese.But if I set post status as publish it doesn't work.


can a :goat: nest?


er, dang, I wish I had looked at these more before writing the above.  Maybe what I really want is just for all of these to be explicit methods that indicate that they're expected to directly propagate through to changes in the DOM.  (Did I mention I really like the react.js props mechanism? ;)


haha nerd


Something you can do in cases like this is use `#try` and see what happens:

```ruby
comment.user.try(:name)
```


alternately, I guess we could have a more explicit naming convention (or I just have to get used to what we have) for things like editToolbar.  This toolbar is very dumb on its own and you're just using dom_evt for all the real brains stuff that happens outside of here.  `editToolbarWidget`?


TeamCity ATK :: QA :: Integration Per Branch [Build 1887](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546529&buildTypeId=Atk_IntegrationPerBranch) outcome was **FAILURE**
Summary: Tests failed: 43, passed: 393, ignored: 15, muted: 11; number of failed tests is too large: 43 Build time: 01:58:09
### Failed tests
```
testcases.frame_create_test.FrameCreateTest.test_baseline_rows (Test the baseline number of row without modification is correct_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_delimiter_empty (Test an empty delimiter errors_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_delimiter_mismatched (Test a mismatched delimiter errors_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_delimiter_none (Test a delimiter of None errors_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_frame_create_row_count (Trivial Frame creation_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_frame_delim_colon (Test building a frame with a colon delimiter_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_frame_delim_tab (Test building a frame with a tab delimiter_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_frame_errors (Test error frames are created properly_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_frame_invalid_column (Tests retrieving an invalid column errors_): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_frame_name (Trivial Frame creation with naming variations): <no details avaliable>

testcases.frame_create_test.FrameCreateTest.test_frame_name_special_characters (Frame creation with special characters in the name fails_): <no details avaliable>

##### there are 33 more failed tests, see build details
```


Need to talk on irc before I consider accepting this pull request.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 360](https://teamcity.labkey.org/viewLog.html?buildId=267260&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:18
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


Huh, weird... When I double check it doesn't have ITMs... Gotta investigate this more...

After double checking, I figured it out! When I loaded it after the plugin Guard Dialogue Overhaul, it showed as having 49 ITMs - all of them were SPIKE keywords that were also used in GDO. Hmm... Should probably add a condition about the ITMs and GDO - I'm not really sure how to do that, though... (or just delete the part about the ITMs, since they aren't technically ITMs even though TES5Edit says they are).

As for the rest of my commit... I'm going to go out on a limb and say that it should be okay, seeing as how unlikely this kind of fluke would happen twice. I'll double check the plugins that I have (some of the plugins in this commit I'm not using or am using updated versions of), just to be sure, and if anything else pops up, I'll make another commit. But if you want to delete the whole thing... well, I can't stop you.


rax retry tests


TeamCity pull requests :: netty [Build 59](https://garage.netty.io/teamcity/viewLog.html?buildId=503&buildTypeId=pulls_netty&guest=1) is now running


TeamCity pull requests :: netty [Build 59](https://garage.netty.io/teamcity/viewLog.html?buildId=503&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:53


We need to discuss some elements of this in a private chat.


high-level-nit: should this be converted to be a DOM dispatchEvent-style thing?


you mean we can provide a key like "0?54c-0?5c4-Wireless Controller" as a key when populating the map ? I checked the polyfill too, and I'm not seeing it supporting this


FourPipeBeamAirTerminal (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1802 of 1802 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


TeamCity pull requests :: netty [Build 60](https://garage.netty.io/teamcity/viewLog.html?buildId=504&buildTypeId=pulls_netty&guest=1) is now running


TeamCity pull requests :: netty [Build 58](https://garage.netty.io/teamcity/viewLog.html?buildId=502&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 4745, ignored: 16; exit code 1 Build time: 00:21:30


note: I'm giving this file a lighter level of review since there is a high potential for changes for convoy.  (Or at least, there's good changes we could make with the new way the ListView implementations work in convoy.)


ZoneVentilationDesignFlowRate (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


ZoneVentilationDesignFlowRate (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


ZoneVentilationDesignFlowRate (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


oh, this is where they went without their XXX comment friend.  Can these just be handled via CSS selector on our card's mode?  Or would that run contrary to the shadow DOM possibility?


@cyanogen not sure if you saw, but there was another patchset since this (and that was the one that was submitted).  It would revert the Linaro A15 memcpy though, so not sure if you'd want it.  I don't have any A15 devices to test on.


Pasta errada no repositório


whoops, I think we want a guard here so that we only invoke the function if (node) ?


typo nit: perforam


ZoneVentilationDesignFlowRate (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


I checked again, the indent in line 1179 is correct, but the indent in 1176-1178 are not correct, for some reason, used tab, I will modify it in new check-in.


also grammar nit on the is that precedes it.


コピペしてきてコメントと実装違ってると後で混乱するので確認しておいた方が良いですね（他にもある）


waht


@wushin can you be any more vague than that? :laughing:


shotCountは二重管理になるのでいらないと思います
弾の数が取りたかったら`_bullets.size()`で取れます


ZoneVentilationDesignFlowRate (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


これはメンバーで持たない方が良いですね。ローカル変数で良い


Is this openocd 8 or 9?
Need to double check this with gdb-multiarch on mips. openocd 8.0 needs a change to src/target/mips32.c, it should be  #define MIPS32NUMFPREGS (34)
I believe code was refactored and fixed for openocd 9


Avacalhar as permissões?

On Tue, 18 Aug 2015 14:08 Victor Campos <notifications@github.com> wrote:

> Mas qual a diferença de não rodar como sudo e colocar sudo em todos os
> comandos?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/victorlcampos/install_redmine/commit/73162a7d1a4e65ea78c0da52102bb9a1f11a064d#commitcomment-12765994>
> .
>


default not needed here


can you please teach me how to use your resource into eclipse?


thank your


thank you


remove static variables and move them to plane class


FourPipeBeamAirTerminal (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1802 of 1802 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Have you add the add cookie test?


encapsulate with #ifdef so APM2.6 excludes it - it doesn't fit!


move is_flying and crash detection to new file - we're starting to take up a lot of lines in ArduPlane.cpp


make sure to use #ifdef's to not load this on APM 2.x


Comment edit - "t his" to "this"


If I did something wrong or need to fix a specific thing, please let me know.
I'm new to this and TeoTwawki has been helping me out as much as possible!
Thanks!


Maybe somet=hing like `getCoreleasedPackages()` is a good name for this? The idea is that they're just packagist packages, but they're ones that are "co-releeased"—every time the parent project has a release, this package does as well.

It's almost tautological that a module is co-released iff its version dependency is `self.version`, so I would advocate using that.


@tremainebuchanan and @nick22891 we should be consistent with the location of the toast dialog.  We also shouldn't be hardcoding this value... maybe a global value that can be modified in one place.


Yes, instead of fmt.Fprintf(os.Stderr, ...) for the error case use log.Fatalf


I think so.


Reason for edits:
Line 100 - redundant
105-107 - seems too basic for a prereq
112 and 119-122 - Potion effect spells don't function as buffs; duration changed to compensate.


Please see the included docs/INSTALL.md file


No don't do it.


Oh. That's right.
It certainly doesn't work, but behaviours are being compiled anyway — seems that rebar considers `src` directory as default, and compiles everything in it recursively.


@mekolat I could.


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-7.21%25-green.svg)


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1277 of 1278 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.92%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.13%25-green.svg)


テストにまつわるコードは spec/support 以下にファイルを作るか、rails_helper.rb に書くほうがベターな感じです


Why is `NewUserSchema` being used?  I thought we were gonna do the admin approval after registration


TeamCity pull requests :: netty [Build 60](https://garage.netty.io/teamcity/viewLog.html?buildId=504&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5568, ignored: 21 Build time: 00:34:27


The intention is to replace the toast messages with dialog boxes until we work out how they will be incorporated throughout application in a more standard way.


rax retry tests


ブロック使ってかけます
```
expect { File.exists? ("public#{profile_image.image.url}") }.to be_truthy
```


@matjames007, I'd need to get your thoughts on the admin approval after registration. What's the rationale there? If the admin initiaties the registration process, why would they then need to do another approval?


And MSVC, for qsort_s.


No man... I was thinking that any user could register on the website via some registration link.  However, it is up to the administrator to approve/decline the user.  The user cannot login until after the administrator approves.


The new user schema is in line with the time to live functionality of Mongo.  There must be a strong rational behind adding another step to the user creation flow, hence the use of the new user to hold newly created users until they update default password.


## This is a markdown file


ta pas oublié d'enelever cette ligne

from allocine import allocine

?


ok, so I found the source of confusion

some explanation first :

the ps4 controller has 4 "buttons" in the central area, the `share`  and `options` button on the top, and then what I would call the `touch` pad  (that also serves as a button), and the `PS/home` button.
![image](https://cloud.githubusercontent.com/assets/4033090/9348478/e6e250fe-4669-11e5-9e52-b9985cf4f4bd.png)

taking the standard mapping, as given by Chrome they are mapped as followed :
https://w3c.github.io/gamepad/#gamepadevent-interface#mapping
* `share` : `[8]`
* `share` : `[9]`
* `PS/home` : `[16]`
* `touch` : this one is not defined in the w3c mapping, but reported as `17` by http://luser.github.io/gamepadtest/

in the Firefox mapping I defined, i then actually map the wrong button, and it should be  :
`[ 1, 0, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 14, 15, 16, 17, 12, 13]`

with 13, being replace by 12, and the 13 added as the 17th value


good work...


lol! Yes and no.

Yes, that's the string you will provide.

No, `Map` doesn't work like that. We have to add the code that does what I described.


日本語の方がよいのでは？


Wasn't this fixed eons ago? in commit f74252bb07f7733fdbf4e82dbe8b95f2b33a3631


I think u need to move the *pos != 0 case in c_next()
In the current code, next calls start just for reuse. But now that u have seperate functionailty it makes sense to seperate them !


Move duplicated code into separate function.


エラーメッセージを加えるとよい。


Why the index.php in install folder has been removed? I can't get this in master repo as well


ここはそのまま。


ええええええええええ


TeamCity ATK :: QA :: Integration Per Branch [Build 1888](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546534&buildTypeId=Atk_IntegrationPerBranch) outcome was **SUCCESS**
Summary: Tests passed: 437, ignored: 15, muted: 10 Build time: 02:03:37


`consensus `


OpenCart's integration is on a separate branch. The master branch is a clean copy of VQMod only, available for all systems


Hello, are this module support prestashop 1.6.0.11 ?


I know this was mentioned in the slack group, but I don't want the `TO_EMAIL` to make it's way to the `develop` branch.  Also a rename of `FROM_EMAIL` to `REPLY_TO` email would be preferred.


Sir maduranga good day sir, i have a concern if it is possible to integrate the barcode scanner to your inventory module, thank you.


@vijaydev sorry my commit bit to docrails disappeared. Can you give me commit bit again?


:fearful: That just confused me even more.

Ok, when you give numbers, I only want the numbers reported by http://luser.github.io/gamepadtest/ (I'm already familiar with the standard mapping and the numbers it needs. Let's not go down that path.)

So when you push the Home button, what number is it? It should be `13` according to your mapping; if it's not `13`, your mapping is wrong.

And going back to the original question, which button is `12`, since it is not listed in the mapping? If it's the touch pad, then that answers the whole question.

However, you said the touchpad is `17` ... But that's not possible because I can see from your mapping that D-pad-right is `17`. Right and touch pad cannot both be `17`. That doesn't make sense!


bringing this in to master, will work on development branches under a fork (which I'll figure out how to do next)


Hi @sc1entist
I think there is no smart way to import aar library to eclipse.

Something like this might work:

1. copy [RecyclerTabLayout.java](https://raw.githubusercontent.com/nshmura/RecyclerTabLayout/master/library/src/main/java/com/nshmura/recyclertablayout/RecyclerTabLayout.java) to your source folder.
src/com/nshmura/recyclertablayout/RecyclerTabLayout.java

2. copy [styles.xml](https://raw.githubusercontent.com/nshmura/RecyclerTabLayout/master/library/src/main/res/values/styles.xml) to your resource folder. Or copy [this](https://github.com/nshmura/RecyclerTabLayout/blob/master/library/src/main/res/values/styles.xml#L4-L20) to your styles.xml.
res/values/styles.xml

3. copy [attrs.xml](https://raw.githubusercontent.com/nshmura/RecyclerTabLayout/master/library/src/main/res/values/attrs.xml) to your resource folder. Or copy [this](https://github.com/nshmura/RecyclerTabLayout/blob/master/library/src/main/res/values/attrs.xml#L3-L21) to your attrs.xml.
res/values/attrs.xml

4. import RecyclerView to your project.
http://stackoverflow.com/questions/26492345/importing-cardview-and-recyclerview-android-5-0-in-my-existing-project-eclips


Move duplicated code outside of if, change if condition to negative and assign `v` to default value inside.


The intention is to email the buyer a PDF based report.  Not a link to it.


Removed in upcoming feedback 1 commit.


Looking good. I did somethin similar for a while but it was a bit resource hungry.


Ahh yes the pain in the ass extrapolate button...


16 and 17 are respectivelt reported by the online tester as 12 and 13 ! I'll take more screenshot once back from lunch.

So
12 is button 16 in the mapping
13 has no corresponding mapping, but when using the online tester under chrome (that uses a standard mappin), it's then button 17


Weird, you should have access.

![Uploading Screen Shot 2015-08-19 at 1.33.20 AM.png…]()

Could you check again?


数値直書きよくない。


括弧をつけるとよい。


:unamused: Don't list index numbers. You're confusing me with that. Button you press => number shown in tester. GO!


Chrome
Home : 16
Touchpad: 17

Firefox
Home: 12
Touchpad: 13


Hrm


ok update this


It is something that is heavily copy/paste moved from message_list, with the interface to the component, like the messageClicked part you called out, the new part. Scrolling, loading more messages, adjusting the scroll to match current selected message still work when i test on flame device. Folders with no messages also work as before.


metaconst! ^^


Fontos mysql változások is tartoznak ehhez az adaghoz. Lásd a msql_sample.sql fájlban!


@rafaelfranca  thanks. Now I can edit and delete others comment (see picture), so maybe I have write access (maybe...). I will try push some fix if I find typo or something :)

BTW Now Do I  not belong to Ruby on Rails team ?

![2015-08-19 13 39 09](https://cloud.githubusercontent.com/assets/5356517/9348877/0e368a0e-4678-11e5-95d2-281bbad25171.png)


This cannot happen with my App. The sickrage repo is downloaded to a sub directory and no other app files are placed inside. A fit reset (forceful) cannot delete the start script.

My current App should be up to date, with the exception of a pyopenssl module. I am looking to fix this soon

I don't know the status of QSickRage


Worked with Daniel to get this pull request rebased since there are not
many outstanding changes. Didn't manage to preserve the --author but it is
pushed to master.
On Aug 18, 2015 7:58 PM, "Daniel" <notifications@github.com> wrote:

> ooops I missed this pull request from earlier (!)
> I think this is totally the right direction but I'm worried there are too
> many outstanding diffs atm...it might make them hard to rebase....I think
> for now, I'll wait until everyone's in and has committed what they have---
> the big problem is the CABAC refactor
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielrh/losslessh264/commit/5a34bad54134bb8432344a65f07924619b74aac7#commitcomment-12775124>
> .
>


A copy/paste move from code that did this before. I think the containerNode.addEventListener helps provide the right scope guard for it, but I added an if (node) for good hygiene in upcoming feedback 1 commit.


You are collaborator of this repository only, but not part of the Rails organization. We restricted the organization to only people in one of the officials teams like Core, Committers, Security or Issues.


> The half pixel offset here mimics the default MonoGame/XNA camera source

I see why you did it but I'm still not convinced it's actually needed. The original source code is compensating for differences between OpenGL and DirectX.

            Matrix.CreateOrthographicOffCenter(0, vp.Width, vp.Height, 0, -1, 0, out projection);
    #if !DIRECTX
            // GL requires a half pixel offset to match DX.
            projection.M41 += -0.5f * projection.M11;
            projection.M42 += -0.5f * projection.M22;
    #endif
            Matrix.Multiply(ref _matrix, ref projection, out projection);

In our case, I think the offset compensation has *already happened* and we don't need to do it again.

More importantly, if I setup my camera without any zoom, position or rotation changes on a resolution of 800x480 I'd expect the `GetBoundingFrustum` method to represent a rectangle that's (0, 0, 800, 480) not (0.5, 0.5, 800.5, 480.5) as it's currently doing.

We'll have to check to see if this behaves differently when referencing the different DirectX and OpenGL implementations of MonoGame.


Both fixed in upcoming feedback 1 commit.


OK I see thanks!


merging my pull


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/3950/


I'll just remove this one metadata entry then.


Pulling


not completely sure about `continue' (so restart `tls_read' operation), as the man page said `the caller should call the appropriate function'.


what? they print something else on stdout with that command... gah. Thats probably something that we should report upstream that should go to stderr or something.


The default (Windows) cursor blink rate is actually 530ms :facepunch: but flash might use 500, idk


And I guess Chrome has the standard mapping, already? In that case, yeah, this Firefox mapping needs to be fixed as you described [an hour ago](https://github.com/melonjs/melonJS/commit/2748d0feb23f2d7cf768abf823f7a234963fbc7c#commitcomment-12775609) :smiley:

Sorry for being so much trouble with this! I was just really lost in all the numbers, and wanted to clarify why `12` was originally missing. Thanks!


Just to be certain, are you suggesting adding them to the blacklist in the carbon deamon config? or am I not seeing an option, or misunderstanding an option in mod-graphite


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1802 of 1802 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I see.. wasn't aware of this. Thanks for explaining this. I am temporarily using the async option and added a TODO for it.


bom


No worry, seeing a bunch of numbers and the mapping issues also highly confused me initially :)

And a double-check is always better than a single one (and you did spot an issue)

> On 19 août 2015, at 12:55, Jay Oster <notifications@github.com> wrote:
>
> And I guess Chrome has the standard mapping, already? In that case, yeah, this Firefox mapping needs to be fixed as you described an hour ago
>
> Sorry for being so much trouble with this! I was just really lost in all the numbers, and wanted to clarify why 12 was originally missing. Thanks!
>
> —
> Reply to this email directly or view it on GitHub.
>


bombam


My Name is Chuck Norris and I approve of this commit!


Importing print as a function is not nessary if you are not using the print function methods. print('stuff') works in 2.6+ without the import.


Shouldn't e.g. `t` be wrapped in `<var>`? (and maybe `0` in `<emu-val>`...) Or I guess we should make authors do that in their EMU/EMD input, and this is just a slightly-incomplete example? Disclaimer: I haven't actually checked the spec's rendering.


Still works fine with regard to the DUO2's WoL functionality, but not tested on other boxes.
So now we've added a total delay of 10sec (!!) to cope with problematic boxes and problematic HDD's. I'm sorry to see we have to make such a work-around for issues that IMHO should have been fixed at another level.


looks good to me :shipit:


Oops, this logging statement is useless. It can be deleted.


am using the same version as you tested don't know why i hit this issue.

[root@abc chef-repo]# find / -name cloud

/root/.chefdk/gem/ruby/2.1.0/gems/knife-openstack-1.3.1/lib/chef/knife/cloud
/root/.chefdk/gem/ruby/2.1.0/gems/knife-cloud-1.2.0/lib/chef/knife/cloud

[root@abc chef-repo]# rpm -qa|grep -i chef
chefdk-0.6.2-1.el6.x86_64


Heheh, it's actually really easy if you just focus on the numbers shown in the tester, and then lay out the array in the order that you press the physical buttons.

Introducing the concept of the standard mapping numbers and array indices just convolutes the whole thing. :confounded:


I am not really sure if switch will help us to reduce number of `match` calls. Can you elaborate what you had in mind maybe I am missing something.


3A analyzer runs in different thread with image processor, in order to apply the right parameters / settings on each processed video frame, use timestamp to do synchronization.
-> Attach timestamps on Analyzer result parameters
-> The timestamps come from the input statistica data of Analyzer, and these originally come from input video frames.


What's up with this?


This is the wrong way round.
10 GBP -> NOK should be

10 / 0.75 (GBP -> EUR)
7.5 * 9 (EUR -> NOK)


Yay!


:clap: Nice!


@clinton-hall I think he is using some way outdated sickbeard-tvrage package before the change. either way, he should be ok for now.


Forgot to mention - you will need to rebuild some DB files for some changes to take effect.
1. First delete the playlistfs folder in app/db/
2. Start Volumio
3. Click refresh spop tracklist to rebuild spop tracklist
4. Click import playlists from services to rebuild playlist filesystem


Maybe it makes sense to trigger the kernel stack shutdown in this function already, because the oplk_shutdown only issues a cleanup command.


{{fullscreentesting}} is put in {{onwebkitfullscreenchange}}, only change to full screen successfully, this function is invoked. tct-fullscreen-nonw3c-tests get full pass.


i can not install the Carbon-Forum until i added this to PDO.Log.class.php


This is the final version of SRS document :)


Yes, it should. I was too fast yesterday... :(


fdsf


Correct, they are now ordered by id, and since ids are autoincremented integers, it's the time of registration. I'd like to add a proper datetime field later, but for now we're trying to avoid migrating the database as much as possible.


Wow, alot of changes! If I read this right then I am glad you are teaming up with tyleryperHD


Thanks for working to solve this issue.
Basel


See commit d588403616a33c80d9c4ebf5363c92229d9f260b.


This line is commented because it has a permission error. Any thoughts about this? @pocke


Correct spelling error in test function: `WithInconsistenCounty`
Changed Assert.Fail description to be consistent with `Find_FindsErrorsOutOnWhitespaceIds`
Remove redundant comment (`\\expected`)


This line unlinks or remove the previously selected templates from a specific host.


I'm new to this and being babysitted by @TeoTwawki, if there's any edits you need, lemme know!


Changed method name `Update_UpdatesAddress_ForInconsistentCountry()` since sounds like it will actually update the address even though `Result` has errors.


You're on track here, but this wouldn't work. The problem is this:

`<Springs><Bla /></Springs>`

You'd think Bla would render a Child which receives the context correctly, but what if Bla unknowingly added its own `Springs` wrapper? The second Springs would pass the new context (whether it's overshadowing or a new item on the stack) and the Child would receive/pop the wrong one.

This is basically scope shadowing. I don't know of a sound solution to solve this (beside Lisp hygienic macros, but these need the scope to be analyzable at compile time). You _could_ do `<Springs myContextName="uniqueID" />` but then you'd be better off passing around AnimationConfiguration instances or something.

Which is why I think I'll revert to children function for now.


Yeah, I think that is actually the explanation! Thanks for merging :)


You can use instead:

```elisp
(and golden-ratio-exclude-buffer-regexp
  (cl-loop for r in golden-ratio-exclude-buffer-regexp
           never (string-match r (buffer-name))))
```


@Stryzhevskyi new app version is in Chrome Web Store. Thank you!


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Moved to CSS in upcoming feedback 1 commit, removed the paired xxx comment in edit_controller.js


Thanks, looks good, you can avoid the delq nil by using cl-loop though.


Should probably remove the commented out code


As much as I like readability. I'm not sure how this improves it. The way I see it, it only obscures what's actually going on when setting the base address. If the logic of selecting base address was actually complex, this may have been worth it, but in this case, it just makes me as a reader of the code jump back and forth to get an idea of what's going on.

I would say that you just switched string.IsNullOrWhiteSpace for GenerateHipChatBaseUrl. The former being more general and less domain-specific, but much more common. There's no way a reader will misinterpret it.

DRY also isn't applicable, as the method is only called in one place.


I do not know the cause ...
I'll try to investigate.


@clebettre modify main.go is no necessary


兩個問題：
1、為何這裏需要廣播 REHYRDATE 事件？
2、一般名稱會是 hydrate (含水) 與 dehydrate(脫水)，這裏的 rehydrate 是何意？


Maybe we want to use FileUtils.rm for consistency here ?


Aaaand I didn't read the comment two lines above this. Reminder to myself to clean up the commit message to reflect this.


nice approach


keep clean


Please try to change from `(hosts: [{hostid: host_id}],templates: clear_ids.map{|x| {templateid: x}})` to `(hostids: [host_id], templateids: clear_ids)`


Thank you that comments.


Ref https://www.zabbix.com/documentation/2.2/manual/api/reference/template/massremove


Np.


Do not use strdup, it can lead to memory leak. Use `ngx_pstrdup` or `ngx_copy`.


Bug report is at https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=796061.


How can this work at all? Isn't the function overwriting the wait variable from above?


I know that's why I always add `.gitattributes` > `* text=auto`


LOL musical commit messages :-)


OK, there's not problem this modifies.
I will merge it.


Cool!


This fix is causing a test fail that might be revealing an issue long-masked issue by a old (but valid) file hanging out on s3 with gzip content-encoding.


umm... have we addressed with in ACID (For all Core Bases?)

Please contact me on IRC to arrange this or confirm this has been done.


TeamCity ATK :: Source Code Compile and Test [Build 8609](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545997&buildTypeId=Gao_SourceCodeCompileTest) is now running


It Worked!.. Thank you very much. I will try to create a pull request on this.


TeamCity ATK :: Source Code Compile and Test [Build 8609](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545997&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Exit code 1 (new); org.trustedanalytics.atk:doc-api-examples Build time: 00:00:52


TeamCity ATK :: Source Code Compile and Test [Build 8610](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546048&buildTypeId=Gao_SourceCodeCompileTest) is now running


TeamCity ATK :: source code Package [Build 8238](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=545998&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:05


TeamCity ATK :: source code Package [Build 8239](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546008&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:06


TeamCity ATK :: Source Code Compile and Test [Build 8611](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546168&buildTypeId=Gao_SourceCodeCompileTest) is now running


TeamCity ATK :: source code Package [Build 8240](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546214&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:06


TeamCity ATK :: Source Code Compile and Test [Build 8612](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546216&buildTypeId=Gao_SourceCodeCompileTest) is now running


TeamCity ATK :: QA :: Integration Per Branch [Build 1889](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546215&buildTypeId=Atk_IntegrationPerBranch) outcome was **UNKNOWN**
Summary: Canceled (Snapshot dependency failed to start: ATK :: source code Package (new)) Build time: 00:00:04


TeamCity ATK :: Source Code Compile and Test [Build 8613](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546324&buildTypeId=Gao_SourceCodeCompileTest) is now running


TeamCity ATK :: Source Code Compile and Test [Build 8612](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546216&buildTypeId=Gao_SourceCodeCompileTest) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:11


TeamCity ATK :: Source Code Compile and Test [Build 8614](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546698&buildTypeId=Gao_SourceCodeCompileTest) is now running


Shouldn't count be 10 at some point?


TeamCity ATK :: Source Code Compile and Test [Build 8610](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546048&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Exit code 1 (new); org.trustedanalytics.atk:event-api Build time: 00:00:50


TeamCity ATK :: source code Package [Build 8241](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546049&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:04


run the fixture by:

    cd /var/www/my_geonode
    python manage.py loaddata icraf_dr/fixtures/icraf_groups.json


TeamCity ATK :: Source Code Compile and Test [Build 8611](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546168&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Exit code 1 (new); org.trustedanalytics.atk:doc-api-examples Build time: 00:00:43


TeamCity ATK :: source code Package [Build 8242](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546169&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:04


TeamCity ATK :: Source Code Compile and Test [Build 8613](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546324&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Exit code 1; org.trustedanalytics.atk:event-api Build time: 00:00:42


TeamCity ATK :: source code Package [Build 8243](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546325&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:03


TeamCity ATK :: Source Code Compile and Test [Build 8614](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546698&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Exit code 1 (new); org.trustedanalytics.atk:event-api Build time: 00:00:49


TeamCity ATK :: source code Package [Build 8244](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546217&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:03


factorisable


やｍｌとｙｍｌ...


Huh? is it working?


Thanks for the feedback - I'm going to go ahead with the Validator refactors, and also merge in these tests so long - it would be good to discuss testing on the tech team call this week, so I will add that to the agenda.

To explain my intention in adding these unit tests - I was trying to test the updates in Donations validation from #324, not UtilController - any UtilController work that was included was just to get the tests running error free. The tests were not only added to increase the confidence about the correctness of the final codebase, but also help to make user testing and code review quicker for the specific issue at hand (application of general configs in this case). Rather than the developer having to repeatedly test the API endpoints to validate the code, and then the same process again during code review, these tests make it very clear what should happen, and also explicitly describe what the intentions were. These unit tests helped to pick up on an inconsistency in the BloodPressure validation, where an `||` was needed instead of an `&&`: https://github.com/jembi/bsis/commit/ccb3eaf64ba7b269cb6ed689e73aed217ba8b57b#diff-8beb0b6ccdfa57dc2c60d2cab1a747deR147

The above is stating the obvious, and I think the discussion we need to have is on the best approach to this, not on the need and value of this (I think there's a shared understanding of this), but the point I want to make is that we want to try focus on unit tests at first, rather than integration tests, and that the purpose of these is tests is to help the developer, as well as the code.


what is this


TeamCity ATK :: Source Code Compile and Test [Build 8615](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546706&buildTypeId=Gao_SourceCodeCompileTest) is now running


after this commit:
2015-08-19 07:41:01,461 DEBUG  getShowid: Trying to get IMDB, Addic7ed and OpenSubtitles ID for Tyrant
Traceback (most recent call last):
  File "/home/jezus/autosub/autosub/Scheduler.py", line 21, in __init__
    self.command.run()
  File "/home/jezus/autosub/autosub/checkSub.py", line 90, in run
    showid,a7_id, OsId = Helpers.getShowid(title, UseAddic, UseOpensubtitles)
  File "/home/jezus/autosub/autosub/Helpers.py", line 351, in getShowid
    AddicId = AddicIdMapping    if AddicIdMapping    else AddicId
UnboundLocalError: local variable 'AddicIdMapping' referenced before assignment


TeamCity ATK :: Source Code Compile and Test [Build 8615](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546706&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Exit code 1; org.trustedanalytics.atk:doc-api-examples Build time: 00:00:18


TeamCity ATK :: source code Package [Build 8245](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546707&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:02


ウウウ…


得到


I think it's better to install jasmin through apt-get, just like you're doing for other packages, because jasmin is just a simple requirement like others.


@amittal1417  Can you please run gem list knife* command and send me the output.


Epic, nice work to everyone involved! I know this must have been a mammoth task.


And it is not 'never' but 'thereis' to use in the `cl-loop` sorry.


missing comma after `&pegasus.Product{}`


i want to use thissss!!!


I really like this. Making it more secure :+1:


I'm gonna have to check out the error_check_for() method -- I haven't done that sort of thing before. Super cool.


I would probably call this procedure `anagram?`.


Já percebi que isso está errado e corrigi.


Maybe you wanted to do

```python
try:
    from webcompat import app
except ImportError, err_message:
    raise ImportError('importError: %s\n%s' % (err_message, IMPORT_ERROR))
```

:wrench:


No, not at all. Nice catch :)


this


OK, thanks!
I would suggest in your github main page https://github.com/vqmod/vqmod. The README.md file add the link for the opencart h2 title to easy to spot out.


Or we can even make it slightly better by including the right thing to do with variable substitutions.


"Base" is not very descriptive name for a class :)


I would have removed the class altogether, but I needed it around to reimplement it. It's gone with next commit.


请提个 PR，把这些内容 merge 回 fireball，后续调整我再加


其实都不用PR，你直接 merge upstream 就行了


tests will probably need to be rewritten


gedgf


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3954/console to remedy the problem.


In The Racket [Guide][1] and [Reference][2], `let` forms are written with square brackets, e.g.

```racket
(let ([sorted-string
       (lambda (s)
         ...)])
  ...)
```

[1]: http://docs.racket-lang.org/guide/let.html
[2]: http://docs.racket-lang.org/reference/let.html


@michalmo scheduleOnce will use prevent running the function twice only if it's *the same* function. Here, a new, different inner function is created for every call to update() and will schedule it repeatedly


Pushed this change to our build server. Unfortunately it breaks builds on Altera and Xilinx platforms!


TeamCity ATK :: Source Code Compile and Test [Build 8616](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546725&buildTypeId=Gao_SourceCodeCompileTest) is now running


UI::Base seems ok for me


Also I didn't remember we are still using `cl` and not `cl-lib` so we should use `loop` and not `cl-loop`.
It's time to switch to `cl-lib` though.


TeamCity ATK :: Source Code Compile and Test [Build 8616](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546725&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Exit code 1; org.trustedanalytics.atk:event-api Build time: 00:00:42


TeamCity ATK :: source code Package [Build 8246](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546726&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:04


Why not use the [shorthand][1] notation?

```racket
(define (anagrams-for subject candidates)
  ...)
```

[1]: http://docs.racket-lang.org/guide/define.html#%28part._.Function_.Shorthand%29


@milancurcic Perfect! I know that my `1._R_Pe-4.R_P` was wrong, but without compile it (I am out of office) I was not able to see that `R_Pe` is viewed as a single kind parameter. On the other hand I was not conscious that `1e-4` is a real literal constant and not an expression! Thank you very much!


Good idea to discuss testing in the meeting. Then we can clarify what we each mean by unit tests and what we mean by integration tests. I have a feeling that we're all putting the line in a different place.

I'd like to put my other strong opinion about tests down before anyone makes any assumptions about my position - any test is better than no test. I really feel that how the test is written is not very important (although Brett's point about DBUnit tests being slow is very true).

Daniel, I also agree that the tests help me when I'm writing code - so I mainly add them for selfish reasons :)


Thanks for the updated patch.

:+1:

@lbudai is doing merges these days and he is on holiday as far as I know, but I am sure this gets merged in the coming days.


Not sure if I'm missing something, but I'm a bit confused by what you've said here.

> my intention in adding these unit tests

> we want to try focus on unit tests at first, rather than integration tests

Are you classifying these tests as unit tests? And then are you saying that these are not the types of tests we should be focusing on?

I wouldn't call this a unit test since it isn't testing a single unit of code, but rather the interactions between several classes and the database (See https://en.wikipedia.org/wiki/Unit_testing#Separation_of_interface_from_implementation). Also, for a good summary of the differences between types of tests take a look here http://stackoverflow.com/a/4904533/1230897.


No, it used to be `form.condition` and `args.path` in the builder is the entire path to the form object for this specific field. Something like `schemaForm.form[3]` or `schemaForm.form[2].items[2]` so slapping on a `.condition` should work.

The reason it changed is that the condition is added to each top level DOM node in the template. And the `sf-field` directive is only applied to the first (which maybe is a bad idea... maybe it should be each child. hmmm got to think about that).


'continue' should be fine here.


:sunglasses: This is my favorite way of writing "Hello, World!" in a Scheme-like language.


Right :) since I am a little bit paranoid and I cannot claim to fully understand what `ConfigureAwait(false)` actually does when called on one of Rebus' worker threads, I've added a test that processes 10000 messages in 20 saga instances with up to 20 of them in parallel on one single Rebus thread.

Result: All handlers are run on the worker thread, just as they should.

Conclusion: It works as it should :) you are awesome! Thanks for contributing, I'll release 0.93.0 in a few minutes.


Shouldn't the healer's attack proc handle this logic in an actual OOP way?


What is `hou` and where does it come from?


It's important to note that the "snippet" object that's returned from the YouTube API doesn't include the ID for the associated playlist object.  This information will have to be retrieved using the "contentDetails" query parameter or some other method.


@timler I agree.

Probably best to leave this for now and rather discuss it in the meeting.


Really "little step for man, great step for  guitarixity" ;)  thanks


I guess this will give a notice when version is not an attribute?


貌似 onSelect 和 onCheck 容易让人混淆，先从 antd 的文档中移除了。以后确实有需求再加上好了。@warmbug


this command is wrong, should be `addresult laddername +winner loser 30:21` imo


I don't really see the added value of managing a seperate fact-script while the unless statement is much simpler to use/read and runs the same command...


Strava doesn't mind `Treadmill` here, but Garmin Connect rejects it... *sigh*.


smart!


why does it need this?


@cheeseblubber , you also need to change the stage_collection_spec.rb to test what you have changed. Otherwise it will fail in travis CI.


@duzydook From your code, `1` is `test mode` right?
But from current code, `0` means `live mode`.
So, why you wrote the condition like if `test_mode` is 0. Then, change it to 1 (I think it should change to `2`).
Could you check or confirm this condition again ?

Thanks


are these scopes? Then it is much more efficient to let the sql query do the count, and not ruby.


@DiamondBond great! I'll doing dat now & some nice ideas/fixes xD

Off: Sorry for my month afk, I was a little/few (I don't know when is 'a little' or the other lol) busy xP


TeamCity ATK :: Source Code Compile and Test [Build 8617](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546751&buildTypeId=Gao_SourceCodeCompileTest) is now running


Woot, nice to see progress for you guys!


https://github.com/sequelize/sequelize/issues/4314

The motivation behind this PR is that failing fast is good - no need to bother the db


These are `ActiveRecord::Relation`s. Calling `count` on them changes the `SELECT` statement to a `SELECT COUNT` statement, so we actually let postgresql count the entries.


Thanks.  This will go away soon.


**Проба от сайта


*Проба от сайта*


**Проба от сайта**


it is the `ember-list-item-view` class that should be removed.


@bausmeier - you're absolutely right, these are integration tests, not unit tests - the reason for writing them was to test the validator (unit), but the setup and actual test touches more than just the validator (integration). I've appropriated the term unit test here to refer to the intention of the test, rather than the implementation of it, if that makes sense :)
As suggested, we can leave this discussion for tomorrow's call, but I just wanted to clear that one up.


Oops, fixed. Thanks!


TeamCity ATK :: Source Code Compile and Test [Build 8618](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546766&buildTypeId=Gao_SourceCodeCompileTest) is now running


TeamCity ATK :: Source Code Compile and Test [Build 8619](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546768&buildTypeId=Gao_SourceCodeCompileTest) is now running


TeamCity ATK :: Source Code Compile and Test [Build 8620](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546770&buildTypeId=Gao_SourceCodeCompileTest) is now running


@drtshock my chat looks like this after buying your plugin and putting it in my plugins folder, https://gyazo.com/53ec3ce7c39e977927220af74a0894c7 where can I remove the "< >" and add "[ ]" outside there faction names such as, ([FactionName] playername: message) please respond back.


Теперь нет, т.к. в метод передается сразу нужный тип без кастов и неправильно его поиспользовать не выйдет.


Can you remove this non-portable flag again? If you need it you can always pass CFLAGS when calling `make`.


TeamCity ATK :: Source Code Compile and Test [Build 8621](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546773&buildTypeId=Gao_SourceCodeCompileTest) is now running


Erm, why not just

```c
#ifdef HAVE_GMA
#include <CL/cl_ext.h>
#endif
```
?


You have to check for these headers during CMake configuration otherwise compilation will break.


this stuff should usually go into you [global .gitignore file](https://help.github.com/articles/ignoring-files/#create-a-global-gitignore)


nav_rth_altitude variable is defined as UINT8, which is too small to accommodate practical altitude values in centimeters. Eg., a height value of 20m would be '2000cm', which obviously won't fit into an unsigned char. Changing the definition to UINT16 solves the problem.

btw, not sure if this was the right place to post this comment (I'm still fairly new to github).


> This needs to be reverted

definitely not. it needs to be fixed properly


test


Why is it `err2` and why do you need it at all? You use it once in which case you can just check the error when calling `clEnqueue MakeBuffersResidentAMD`.


why is part of the ontology repeated here, including the detailed description? Wouldn't it suffice to check that the needed proof statuses occur in the ontology?


Super!


how about copying utils.cleanUrl from the old UI?


Add the correct header include and remove that stupid comment.


TeamCity ATK :: Source Code Compile and Test [Build 8622](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546785&buildTypeId=Gao_SourceCodeCompileTest) is now running


Do they increase the code size significantly?
I thought some of these changes also have some good performance characteristics, like `var w = window` means the engine does not have to go up in the scope chains at runtime to the global scope when you use `w` as opposed to when you use `window`.


Cool. Clarification- does this require thresholds to have been bootstrapped, so that thresholdsbootstrap exists? (rather than = "none" like I do, because says it will take about 27 min for my data, for "parametric"). If so, would be good to add an informative error message if it's NULL.


That magic number looks very arbitrary. Make it at least a constant that one can change later.


` tOplkFileTransferChunk` and `tCtrlDataChunk` are similar structures, why we need both of them?


this is just a minimal placeholder for proper integration with splash


Wow, thanks a lot for reviewing the code @K-ballo, I am honored! I promise making a pull-request about this was was in my to-do list :smile: !! But it's been a lot of work last few days... I hope that you don't understand the word "stupid" and "eggs::variant" in the same sentence as an insult, `eggs::variant` is an awesome library, thank you so much for making it :)


right, that's an artifact from before a refactoring


You never make use of these (arbitrary) return values. So, please use a `gboolean` to denote an error condition.


TeamCity ATK :: Source Code Compile and Test [Build 8617](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546751&buildTypeId=Gao_SourceCodeCompileTest) outcome was **SUCCESS**
Summary: Tests passed: 1196, ignored: 6 Build time: 00:18:45


Tell me if we need something more in the demo module.


I think @jonnii is working on that next.


Sorry,
![screen shot 2015-08-18 at 12 18 18 pm](https://cloud.githubusercontent.com/assets/13241574/9350055/e0a1da88-4601-11e5-84ff-d05a2bd9650f.png)
![screen shot 2015-08-18 at 12 21 46 pm](https://cloud.githubusercontent.com/assets/13241574/9350056/e538cebc-4601-11e5-8b95-9a4029a63857.png)


I intentionally do *not* include the OpenCL header in a public header. So please remove them and replace `cl_xxx` arguments with `gpointer`s.


Sounds good. One question though, are you planning to include this change in the 2.4.x stream? I'm asking because this is a binary incompatible change.


@williewillus I think this line is not supposed to be removed.

should be `ingredientMap.addIngredient(NormalizedSimpleStack.getFor(stack), 1);`

Might be a cause for #943


Okay, I will move the members from `tCtrlDataChunk` to `tOplkFileTransferChunk`, having only one structure finally.


+# GitHub


Should be `do gdb $(pwd)/crasher $i`, I believe.


TeamCity ATK :: Source Code Compile and Test [Build 8618](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546766&buildTypeId=Gao_SourceCodeCompileTest) outcome was **FAILURE**
Summary: Tests passed: 1137, ignored: 6; exit code 1 Build time: 00:17:54


TeamCity ATK :: source code Package [Build 8248](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546767&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Snapshot dependency failed: ATK :: Source Code Compile and Test Build time: 00:00:02


TeamCity ATK :: Source Code Compile and Test [Build 8619](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546768&buildTypeId=Gao_SourceCodeCompileTest) outcome was **SUCCESS**
Summary: Tests passed: 1196, ignored: 6 Build time: 00:18:28


Thanks! Fixed at:
https://github.com/w3c/webcomponents/commit/39ecd05e8bb8f77ebf91de6018944d86e2673789


TeamCity ATK :: Source Code Compile and Test [Build 8620](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546770&buildTypeId=Gao_SourceCodeCompileTest) outcome was **SUCCESS**
Summary: Tests passed: 1196, ignored: 6 Build time: 00:18:59


ctrlucal_getFileTransferChunkSize() is only implement for hostif?


What is the status relating to this problem mentioned by @gward ?


It would take much longer to process the ontology. For the tests, we only need a few of the statuses and we need them to be saved the database very fast.


It's getting harder to manage "xcode beta / swift / multiple targets" projects with Cocoapods :S


SP nodes are very rare. How can this have any effect?


No, if there are good reasons it is fine!

Am 18.08.15 um 22:37 schrieb Carlos Rodriguez:
> Hi @mgruner <https://github.com/mgruner>,
>
> If fact was my first option but it didn't look so nice along with other
> checks e.g.
> OTRS Disk Partition (/dev/disk1)
>
> But I will change it anyway :-)
>
> ((enjoy))
> cr
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OTRS/otrs/commit/e781eca24e76b3a6ee797c5751c5ee387fef7141#commitcomment-12770198>.
>

--
Martin Gruner
Senior Developer R&D

OTRS AG
Bahnhofplatz 1a
94315 Straubing

T: +49 (0)6172 681988 0
F: +49 (0)9421 56818 18
I:  www.otrs.com/

Geschäftssitz: Bad Homburg, Amtsgericht: Bad Homburg, HRB 10751,
USt-Nr.: DE256610065
Aufsichtsratsvorsitzender: Burchard Steinbild, Vorstand: André
Mindermann (Vorsitzender), Christopher Kuhn, Sabine Riedel

Grenzenlose Kommunikation mit der OTRS Business Solution™! Bereichern
Sie Ihren IT Support und Kundenservice mit smarten Business Features!
https://www.otrs.com/solutions/otrs-business-solution-on-premise/?lang=de/#OTRS-Business-Solution-Basic-Features


you're mixing quotes again
:D


TeamCity ATK :: Source Code Compile and Test [Build 8621](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546773&buildTypeId=Gao_SourceCodeCompileTest) outcome was **SUCCESS**
Summary: Tests passed: 1212, ignored: 6 Build time: 00:19:21


노답


TeamCity ATK :: source code Package [Build 8251](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546774&buildTypeId=Gao_PackageSourceCode) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:15


Hi Badri,
Please find my comments inline.

Regards,
Krishna

> Date: Fri, 14 Aug 2015 04:31:11 -0700
> From: notifications@github.com
> To: growlin@noreply.github.com
> CC: krishna.moorthy94@gmail.com
> Subject: [growlin] SQL: Merge multiple publication/copy tables into "item" (f0b1cbb)
>
> SQL: Merge multiple publication/copy tables into "item"
>
> This is to fit to the new database structure we decided on in the
> meeting of 2015-08-07. Bascially, each item (it can be a book,
> magazine, article, or anything else with an accession number) has
> its own record in the table. If a particular item type has extra
> properties that need to be stored (eg. magazine date), an extra
> field for that property can be added to the "item" table, ideally
> using varchar2 so that it doesn't take up extra space when not in
> use.Krishna - if the extra property is date, let us add it is as date itself. Let us use varchar2 if the extra property is text or alphanumeric
>
> To avoid displaying multiple results of the same book, if there
> happen to be multiple copies registered in the library, a separate
> view will be created which merges the copies into one record,
> displaying the number of copies existing and (later) available and/or
> borrowed.Krishna - yes
>
> @KrishnaRamamoorthy: could you write the code for the materialized
> view? Code for counting how many are borrowed and how many are
> available, etc. can be added later, but we should at least get the
> main view going.Krishna - how I wish I could write the code!! It is ages(more than a decade) since I wrote code and there will be a learning curve in terms of syntax and having the environment and toolset installed etc to try these out. So I feel it is better I dont do it now. Are you facing challenges in this ? Please feel free to let us know.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/badrihippo/growlin/commit/f0b1cbbf337bf63ce7acaf05bc946750c25f1b07


why is `unknown` greater than `success`? I would expect `success` to be greater, since it indicates that at least there is some result. (Anyway, `success` probably is never output by a prover, but I have seen it in the tests...).


So, now application should handle splitting the firmware file into chunks of some maximum size and copy to kernel using ctrlu_writeFileTransferChunk().

The kernel daemon then handles the chunk reading in the callback for kCtrlWriteFileChunk command right?


Thx, missed to implement it for others, will fix that!


TeamCity ATK :: source code Package [Build 8247](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546765&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:10:10


you would need to fix this comment also in the tests (although I rather would remove it from the tests, which would be more DRY).


Its not working, updated version coming soon


UI stands for "User interface". So what is this, a "base user interface"? This tells me nothing about the class :)
Maybe something like UI::Control or UI::Element ?


Ooops, my bad I forgot to reset it to it's previous state :)
But use it instead of removing it.

Before turning on animations again:
self.enabled = wasEnabled


Denis/Carsten, I tested using different user rights but feel free to revert if there is anything I forgot to take into account.


Right, do you prefer a different approach?


Badri,

Sorry about the delay in response. Ideally we should be using varchar2. I read up and found that MySQL supports varchar and not varchar2 - with the same benefits as varchar2. So considering we are not on Oracle, let us use varchar instead of char if the size is more than 2. If the size is 1 or 2, let us use char. Hope this is clear


I don't think we need any of this bit...?


It stands for base user interface class. All other classes are inherited from it


Thanks @danfuterman that makes sense.


Would be better to use Logger instead of System.out


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1277 of 1278 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.92%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.13%25-green.svg)


修改


I guess this should work in all zones


Yes, I agree. In the previous version of firmware update application, we had to use the ctrlu_shutdownStack() to achieve this.


TeamCity ATK :: Source Code Compile and Test [Build 8622](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546785&buildTypeId=Gao_SourceCodeCompileTest) outcome was **SUCCESS**
Summary: Tests passed: 1196, ignored: 6 Build time: 00:20:40


Probably a good idea.


TeamCity ATK :: source code Package [Build 8249](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546769&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:10:19


nice i was gonna leave worrying about the amended reports until later :+1:


right, thanks!


indeed, the gamepad implementation under Chrome is just easy and super neat :P


TeamCity ATK :: source code Package [Build 8250](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546771&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:10:57


@shadowslasher410
@WrinklyNinja

The problem here is, that you loaded GDO alongside Immersive Weapons within TES5Edit, shadowslasher; whenever you want to clean a mod, you should ONLY load up that specific mod with no other plugins selected in the master/plugin selection window. If doing so, only the specific plugin in question + it's masters will get loaded by TES5Edit, and only then can you identify ITMs, so records inside a plugin that are identical to records in it's master plugins. Those 49 "ITMs" you are seeing when comparing to GDO are not ITMs, as GDO is not a master of Immersive Weapons. In any case, I had a look at two further entries of this commit.

Hothtrooper44_ArmorCompilation.esp
0x2A9F7DED
99 ITMs

Purity.esp
0xA466C02D
2 ITMs

Hothtrooper44_ArmorCompilation.esp is another case of a false report, it has just as Immersive Weapons.esp 0 ITMs. The reported 2 ITMs for Purity.esp on the other hand are correct, I can find these aswell.

I do not have the other plugins within this commit, so I can't test them.


window.scrollTo works, but we dont get the nice animation then :/
Maybe there is something wrong with the animation code. Did it throw any errors ?


Why would you add this?


Change the type in Settings instead of casting


Same as above :)


Okay, I will rework the commit!


or we can just extract the vendor_id (1st one) and the product_id (2nd), conver them to decimal and uses them as key for the hash ? Maybe a bit more CPU intensive, especially if it has to be done at every update function (another good reason maybe to use the connected/disconnected event, so that we can apply the required map at that time)


I removed the verbose descriptions from the factory in 42d7451. We should stick with the factory, however, because it is *much* faster than processing the ontology.


Right, `unknown` should come right after `open`. Fixed with ddb5d38.


Отлично! Вот так в основном мы будем общаться в ближайший месяц.


If there is no Storage provided, disabledRulesStorage will be null!


the storage might be null here as well


Good job translating scss from the checkpoint into css!


Thanks,
I requested the hardcoded https for security reasons
https://www.mantisbt.org/bugs/view.php?id=19932#c51067
and was a bit astonished that you changed it.


same here, the storage could be null.


Missed adapting the event members...

````diff
            if (pEvent->eventArgSize > 0)
-                pEvent->pEventArg = &aRxBuffer_l[sizeof(tEvent)];
+                pEvent->eventArg.pEventArg = &aRxBuffer_l[sizeof(tEvent)];
            else
-                pEvent->pEventArg = NULL;
+                pEvent->eventArg.pEventArg = NULL;
````


Yeah maybe forgot about switch, because you can do it your way with `ifs` too.

But what I really meant is something like this: `m = url.match(/(youtu\.be)|(youtube\.com\/v)|(youtube\.com\/watch\?v=)/)`

Regexp with 3 groups. If first group match you can just do `if m[1]`, if second group match `if m[2]` and of course for the third `if m[3]`. The `m[0]` is matched always general matched value when using groups. Try it out, may be usefeul in the future:)

I believe it works like that.


No, I am okay with it.


OK


I will definitely try it out but do you think it will help us with making the code cleaner?


I tend to prefer overlay over aufs, i.e. if overlay is available we should use it, even when the underlying fs is aufs. overlayfs seems to be most compatible.


Can we avoid this by declaring a lambda function before this block and then call it here followed by return?
Though I am not sure in general if the lambda function is called multiple times in a scope does crc/JIT  still inlines it for each call site in Release configuration (optimization: on) or does the smart thing; the goto effect?


TeamCity ATK :: source code Package [Build 8252](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546786&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:11:02


TeamCity ATK :: QA :: Integration Per Branch [Build 1890](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546842&buildTypeId=Atk_IntegrationPerBranch) is now running


TeamCity ATK :: QA :: Integration Per Branch [Build 1890](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546842&buildTypeId=Atk_IntegrationPerBranch) outcome was **FAILURE**
Summary: Failed to resolve artifact dependency Build time: 00:00:19


Maybe there could be created a default-storage here if no storage is provided.


should be merged


@clebettre this can be private


@clebettre why this comments?


Sorry,
Last login: Tue Aug 18 23:46:59 on console
zzzs-iMac:~ zzz$ cd Gigabyte-GA-Z77X-DSDT-Patch
zzzs-iMac:Gigabyte-GA-Z77X-DSDT-Patch zzz$ sudo ./GA-Z77X.sh --inject-hda
Password:
/System/Library/Extensions/AppleHDA.kext failed to load - (libkern/kext) different version/uuid already loaded; check the system/kernel logs for errors or try kextutil(8).

hdaInjector v0.1 by theracermaster
Heavily based off Piker-Alpha's AppleHDA8Series script
hdacd.plist & XML files by toleda & Mirone
--------------------------------------------------------------------------------
VIA VT2021 (0x11060441) / (285606977) detected. Downloading codec files...
######################################################################## 100.0%
######################################################################## 100.0%
Archive:  /tmp/VT2021.zip
   creating: /tmp/VT2021/
  inflating: /tmp/VT2021/hdacd.plist
 extracting: /tmp/VT2021/layout5.xml.zlib
 extracting: /tmp/VT2021/layout7.xml.zlib
 extracting: /tmp/VT2021/layout9.xml.zlib
  inflating: /tmp/VT2021/Platforms.xml.zlib
AppleHDA2021.kext already exists. Do you want to overwrite it (y/n)? y
Removing directory...
Installing AppleHDA2021.kext...complete.
Triggering a kernel cache rebuild...complete.
zzzs-iMac:Gigabyte-GA-Z77X-DSDT-Patch zzz$ ./GA-Z77X.sh --install-clover
This part of the script needs to be run as root.

Gigabyte GA-Z77X.sh Post-Install Script v1.6 by theracermaster
Updates & Info: https://github.com/theracermaster/Gigabyte-GA-Z77X-DSDT-Patch
--------------------------------------------------------------------------------
Detected motherboard: Gigabyte GA-Z77X-D3H
Script arguments: ./GA-Z77X.sh --install-clover: Installing Clover Bootloader
--------------------------------------------------------------------------------
EFI system partition (disk0s2) mounted at /Volumes/NO NAME.
An existing UEFI bootloader (CLOVER
Microsoft) was found on the EFI system partition.
Do you want to overwrite it (y/n)? y
Removing existing CLOVER
Microsoft UEFI bootloader...

Gigabyte GA-Z77X.sh Post-Install Script v1.6 by theracermaster
Updates & Info: https://github.com/theracermaster/Gigabyte-GA-Z77X-DSDT-Patch
--------------------------------------------------------------------------------
Detected motherboard: Gigabyte GA-Z77X-D3H
Script arguments: ./GA-Z77X.sh --install-clover: Installing Clover Bootloader
--------------------------------------------------------------------------------
Installing kexts to EFI partition:
cp: /Users/zzz/Gigabyte-GA-Z77X-DSDT-Patch/kexts/FakeSMC.kext: No such file or directory
 - Atheros AR8161 [1969:1091] detected, installing AtherosE2200Ethernet.kext...
cp: /Users/zzz/Gigabyte-GA-Z77X-DSDT-Patch/kexts/AtherosE2200Ethernet.kext: No such file or directory
 - Marvell SATA controller detected, installing AHCI_3rdParty_SATA.kext...
cp: /Users/zzz/Gigabyte-GA-Z77X-DSDT-Patch/kexts/AHCI_3rdParty_SATA.kext: No such file or directory
 - PS/2 hardware present, installing VoodooPS2Controller...
cp: /Users/zzz/Gigabyte-GA-Z77X-DSDT-Patch/kexts/VoodooPS2Controller.kext: No such file or directory
cp: org.rehabman.voodoo.driver.Daemon.plist: No such file or directory
cp: VoodooPS2Controller: No such file or directory
Generating SMBIOS data:
 - Product Name: iMac13,2
 - Serial Number: fg0fgN8IdsDNMP
 - MLB Serial Number: C0fg18sd00dfDNJJCB
 - System UUID: 54365456-5624-4523-3424-123456789111
File Doesn't Exist, Will Create: /Volumes/NO
Invalid Arguments

File Doesn't Exist, Will Create: /Volumes/NO
Invalid Arguments

File Doesn't Exist, Will Create: /Volumes/NO
Invalid Arguments

File Doesn't Exist, Will Create: /Volumes/NO
Invalid Arguments

Press enter to continue...

Gigabyte GA-Z77X.sh Post-Install Script v1.6 by theracermaster
Updates & Info: https://github.com/theracermaster/Gigabyte-GA-Z77X-DSDT-Patch
--------------------------------------------------------------------------------
Detected motherboard: Gigabyte GA-Z77X-D3H
Script arguments: ./GA-Z77X.sh --install-clover: Installing Clover Bootloader
--------------------------------------------------------------------------------
Rebuilding kext caches:
kext-dev-mode allowing invalid signature -67030 0xFFFFFFFFFFFEFA2A for kext AppleHDA2021.kext


I don't think it will make it dirty for sure.


@clebettre I will refactor this, to match this cal:
```conf.C().GetSting("db_url")```


should this have another `)` in it?


so file name convention to follow whatever we export? :S feels strange


TeamCity ATK :: Source Code Compile and Test [Build 8623](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546850&buildTypeId=Gao_SourceCodeCompileTest) is now running


TeamCity ATK :: Source Code Compile and Test [Build 8624](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=546856&buildTypeId=Gao_SourceCodeCompileTest) is now running


```css
.foo {
  padding-left: 10px; padding-right: 10px;
}
```

Like this?


Нет, рекурсия тут ни при чем, `{StatementList}` говорит о том, что если стэйтментов в списке не будет, то создастся пустой `StatementList`, а не `null`.


@cpfeiffer I would like to replace the status string with the Enum declared here. But I don't know how to make this enum "point" to string resources. That's why I left it as TODO.


CREATE TABLE "group"
+(
+  "id" INTEGER NOT NULL PRIMARY KEY,
+  "position" INTEGER NOT NULL, "name" VARCHAR(128) NOT NULL,
+  "visible" SMALLINT NOT NULL
+);

Here, can we have name as UserGroup for easy understanding. This will be a table with very rows. So why have an integer field as PK ? Can we not have groupname itself as PK ? This will avoid lookups to bring up the name. I am not clear on the fields position and visible.


Here I would like to have a device-dependent low battery threshold, if you agree that it makes sense.


3A analyzer runs in different thread with image processor, in order to apply the right parameters / settings on each processed video frame, use timestamp to do synchronization.
-> Attach timestamps on Analyzer result parameters
-> The timestamps come from the input statistic data of Analyzer, and these originally come from input video frames.


can view cat'-'


Could you use this syntax

render json: suggestions.map(&:keywords)


@koenpunt Care to explain the reasoning behind this change? Did Sprockets 3 not work without it?


@clebettre rename the package from mongo to db?


CREATE TABLE "publishplace"
+(
+  "id" INTEGER NOT NULL PRIMARY KEY,
+  "name" VARCHAR(512) NOT NULL
+);
Here also we can remove the field id. Change the "name" field to call it place or something more relevant and make that as the PK


is that still compatible with older devices ? i suggest a revert, doesnt improve the perf. on a 960


Thought about this a little and it would be better to split getting EM and EMF. When you create EM from EMF you have to take care of it - close it after test. If EM is provided from context then it is shared EM and don't need to be closed.


good, but the tests have to be updated as well.


looks like you missed a `focused: true` :)


Not familiar with that {80,} notation. Does it match 80+ non-space characters?


So now you store it in the `null` key if not `TL_SCRIPT`. What's the purpose of this?


テストケースの修正は不要でしょうか？


CREATE TABLE "publisher"
+(
+  "id" INTEGER NOT NULL PRIMARY KEY,
+  "name" VARCHAR(256) NOT NULL,
+  "address" TEXT,
+  "imprint_of_id" INTEGER,

1. Name need not be 256. Maybe 128 ?
2. Why e the field names as id and name in every table ? Can we not have more meaningful names as relevant for that table ?
3. Why address is text ? Why not varchar ?
4. What is imprint of id ? I am not clear. Further, it ends up being circular reference.


CREATE TABLE "currency"
+(
+  "id" INTEGER NOT NULL PRIMARY KEY,
+  "name" VARCHAR(32) NOT NULL,
+  "symbol" VARCHAR(4) NOT NULL,
+  "conversion_factor" REAL NOT NULL
+);
1. No need for id. Make the name , curr_name as PK
2. Conversion factor has no significance without a date. We discussed the other day and decided not to have it


That doesn't make any sense to me? How can a minor parse tree change affect the semantics? (Yes, I have a background in PLT) :p


minor: "Move ComponentTesting **to** sonar-db"


コミット文が長くなるのが、問題はないのかどうか、判断できてませんが、
もし、短く済ませるなら、
before applying this patch:
 \* case OFPIT_APPLY_ACTIONS and OFPIT_WRITE_ACTIONS
{
    "1": [
        {
            "actions": [
                "OUTPUT:2",
                "OUTPUT:3"
            ],
        ...  ←★省略
        }
    ]
}
でも良いかなと思いました。


Ouch, I'm sorry to have ignored you – I've been busy moving and working on other projects, and for some reason did not get e-mail notifications.
Regarding your first comment, using `pcl::getTransformation()` makes much more sense, I agree.
Regarding your second comment, this could of course be implemented, but I'm not sure if it's good for the usability of the tool.
By the way, feel free to extend on my code, as I currently don't have the time to work on this...


:facepalm:

Fixed :)


Ah, I forgot that the comma in the assert statement is magic


@mpushkin @ebdrup Hi guys. I've been talking with one of the mongojs maintainers to get mongos implemented. He created an experimental branch which I tested by using the staging db url and credentials, and the implementation might have a bug, but it needs to be tested more. Can you try to setup mongos locally and run the test script in https://github.com/mafintosh/mongojs/issues/241 with and without authentication and post the result in the issue.


I totally forgot about them. Done with 2a4d758.