C'est dans la définition des accords en LaTeX:
```
  \cover
  %verse
  \gtab{Cm}{3:X02210}
  \gtab{Fm}{1:022000}
  \gtab{E&}{6:X02220}
  \gtab{G}{3:022100}
  %chorus
  \gtab*{Fm*}{8:X02210}
  \gtab{B&}{6:022100}
  \gtab{C}{8:022100}
  \gtab*{Cm*}{8:022000}
```
Mais mon script sépare les accords et le contenu de la chanson, du coup les commentaires se retrouvent orphelins...
Je vais corriger à la main (il y a sûrement d'autres petits soucis de ce type un peu partout)


(Apologies for dropping such a big refactor on this pullreq at this late a stage of review. I half considered keeping this commit for the upcoming rdkafka-producer pullreq, but the problem is it touches so many lines of the module that it would be hell to keep it mergeable on that other branch. In any case, there are no functional changes here, it's all just moving lines around.)


cool... I merge this to the master and remove this branch


There appears to be some confusion about whether gestAge has a `valueQuantity` (as in the original code) or a `valueString` as in this change. Can we clarify the expectations?


Rather than commenting out dead code, please remove.


->
```python
attr.connect("query_tooltip_markup", self.show_mark_tooltip)
```


Why is this part of `writeln` instead of redefining `write` itself? It doesn't look like anything earlier calls `write` instead of `writeln`, but if anyone ever adds something that does it'll skip the output stream.


why was this removed?


````
08-19 16:36:56INFO[hpotato.core.plugins.base] Opening url: get http://api.themoviedb.org/3/movie/7270?api_key=xxx&append_to_response=alternative_titles, data: []
08-19 16:36:56INFO[providers.info.themoviedb] Found: [u'Les associ\xe9s (2009)', u'Once a Thief (1991)', u'Matchstick Men (2003)']
08-19 16:37:53INFO[hpotato.core.plugins.base] Opening url: get https://api.couchpota.to/search/Les+associ%C3%A9s/?limit=5, data: []
08-19 16:37:53INFO[hpotato.core.plugins.base] Opening url: get http://www.omdbapi.com/?type=movie&y=&t=Les+associ%C3%A9s, data: []
08-19 16:37:53INFO[hpotato.core.plugins.base] Opening url: get http://api.themoviedb.org/3/search/movie?api_key=xxx&search_type=ngram&query=Les+associ%C3%A9s, data: []
08-19 16:37:54INFO[providers.info.themoviedb] Found: [u'Les associ\xe9s (2009)', u'Once a Thief (1991)', u'Matchstick Men (2003)']
08-19 16:37:54INFO[ie.providers.info.omdbapi] Found: Les associĂŠs (2009)
08-19 16:38:03INFO[hpotato.core.plugins.base] Opening url: get https://api.couchpota.to/ismovie/tt0325805/?adding=1, data: []
08-19 16:38:03INFO[hpotato.core.plugins.base] Opening url: get https://api.couchpota.to/info/tt0325805/, data: []
08-19 16:38:03INFO[hpotato.core.plugins.base] Opening url: get http://www.omdbapi.com/?type=movie&i=tt0325805, data: []
08-19 16:38:03INFO[hpotato.core.plugins.base] Opening url: get http://api.themoviedb.org/3/movie/tt0325805?api_key=xxx&append_to_response=alternative_titles, data: []
08-19 16:38:03INFO[ie.providers.info.omdbapi] Found: Matchstick Men (2003)
08-19 16:38:03INFO[ie.providers.info.omdbapi] Found: Matchstick Men (2003)
08-19 16:38:03INFO[hpotato.core.plugins.base] Opening url: get https://image.tmdb.org/t/p/w154/vRjT0WdjQbbIpYSG5f520BAS2dn.jpg, data: []
08-19 16:38:04INFO[hpotato.core.plugins.base] Opening url: get https://api.couchpota.to/eta/tt0325805/, data: []
08-19 16:38:04INFO[core.media.movie.searcher] Search for Les AssociĂŠs in 1080p ignoring ETA
08-19 16:38:04INFO[hpotato.core.plugins.base] Opening url: post https://www.t411.io/users/login/, data: ['url', 'login', 'password', 'remember']
08-19 16:38:05ERROR[   couchpotato.core.event] Error in event "provider.search.torrent.movie", that wasn't caught: Traceback (most recent call last):
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/event.py", line 15, in runHandler
    return handler(*args, **kwargs)
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/media/_base/providers/base.py", line 220, in search
    self._searchOnTitle(title, media, quality, results)
  File "/Users/maxcore/Library/Application Support/CouchPotato/custom_plugins/t411/main.py", line 74, in _searchOnTitle
    log.debug('encoding du terminal stdin : ' + sys.stdin.encoding)
TypeError: cannot concatenate 'str' and 'NoneType' objects
encoding=UTF-8 debug=False args=[] app_dir=/Users/maxcore/Downloads/CouchPotatoServer-master data_dir=/Users/maxcore/Library/Application Support/CouchPotato desktop=None options=Namespace(config_file='/Users/maxcore/Library/Application Support/CouchPotato/settings.conf', console_log=False, daemon=False, data_dir=None, debug=False, pid_file='/Users/maxcore/Library/Application Support/CouchPotato/couchpotato.pid', quiet=False)
08-19 16:38:05INFO[core.media.movie.searcher] Search for Les AssociĂŠs in 720p ignoring ETA
08-19 16:38:05ERROR[   couchpotato.core.event] Error in event "provider.search.torrent.movie", that wasn't caught: Traceback (most recent call last):
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/event.py", line 15, in runHandler
    return handler(*args, **kwargs)
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/media/_base/providers/base.py", line 220, in search
    self._searchOnTitle(title, media, quality, results)
  File "/Users/maxcore/Library/Application Support/CouchPotato/custom_plugins/t411/main.py", line 74, in _searchOnTitle
    log.debug('encoding du terminal stdin : ' + sys.stdin.encoding)
TypeError: cannot concatenate 'str' and 'NoneType' objects
encoding=UTF-8 debug=False args=[] app_dir=/Users/maxcore/Downloads/CouchPotatoServer-master data_dir=/Users/maxcore/Library/Application Support/CouchPotato desktop=None options=Namespace(config_file='/Users/maxcore/Library/Application Support/CouchPotato/settings.conf', console_log=False, daemon=False, data_dir=None, debug=False, pid_file='/Users/maxcore/Library/Application Support/CouchPotato/couchpotato.pid', quiet=False)
08-19 16:38:05INFO[core.media.movie.searcher] Search for Les AssociĂŠs in BR-Rip ignoring ETA
08-19 16:38:05ERROR[   couchpotato.core.event] Error in event "provider.search.torrent.movie", that wasn't caught: Traceback (most recent call last):
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/event.py", line 15, in runHandler
    return handler(*args, **kwargs)
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/media/_base/providers/base.py", line 220, in search
    self._searchOnTitle(title, media, quality, results)
  File "/Users/maxcore/Library/Application Support/CouchPotato/custom_plugins/t411/main.py", line 74, in _searchOnTitle
    log.debug('encoding du terminal stdin : ' + sys.stdin.encoding)
TypeError: cannot concatenate 'str' and 'NoneType' objects
encoding=UTF-8 debug=False args=[] app_dir=/Users/maxcore/Downloads/CouchPotatoServer-master data_dir=/Users/maxcore/Library/Application Support/CouchPotato desktop=None options=Namespace(config_file='/Users/maxcore/Library/Application Support/CouchPotato/settings.conf', console_log=False, daemon=False, data_dir=None, debug=False, pid_file='/Users/maxcore/Library/Application Support/CouchPotato/couchpotato.pid', quiet=False)
08-19 16:38:05INFO[core.media.movie.searcher] Search for Les AssociĂŠs in DVD-Rip ignoring ETA
08-19 16:38:05ERROR[   couchpotato.core.event] Error in event "provider.search.torrent.movie", that wasn't caught: Traceback (most recent call last):
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/event.py", line 15, in runHandler
    return handler(*args, **kwargs)
  File "/Users/maxcore/Downloads/CouchPotatoServer-master/couchpotato/core/media/_base/providers/base.py", line 220, in search
    self._searchOnTitle(title, media, quality, results)
  File "/Users/maxcore/Library/Application Support/CouchPotato/custom_plugins/t411/main.py", line 74, in _searchOnTitle
    log.debug('encoding du terminal stdin : ' + sys.stdin.encoding)
TypeError: cannot concatenate 'str' and 'NoneType' objects
encoding=UTF-8 debug=False args=[] app_dir=/Users/maxcore/Downloads/CouchPotatoServer-master data_dir=/Users/maxcore/Library/Application Support/CouchPotato desktop=None options=Namespace(config_file='/Users/maxcore/Library/Application Support/CouchPotato/settings.conf', console_log=False, daemon=False, data_dir=None, debug=False, pid_file='/Users/maxcore/Library/Application Support/CouchPotato/couchpotato.pid', quiet=False)
`````

jai ca comme erreur


The `__future__` magic might look scary to beginners just getting started with Python or astronomy — could you switch this up to two `print` statements, one doing `ra` and then the next doing `dec`, so that the parens do the same thing in Python 2 and 3? Then we'd be 3-compatible without looking scary. Thanks, I look forward to merging this — good work!


I'm confused about what this means: does this remove the parental view from the app? (If so, the change should be reversed.)


do you need code below?


Capitalization. In most instances its "Head of Security"


This is the function tied to the zoom event, so when you try to remove `this.zoom` it won't be able to find that event


How's this? https://github.com/jonsterling/slamdata/commit/03470661872c6d91e8770d35ffb9d37fa6f3b283


@MartijnR Does this fix a bug?
Also let me ask here, although unrelated, in your blog post (installing enketo on ubuntu for production) you don't copy enketo's **enketo-main.rdb** to anywhere. It's not necessary is it?


Good. The best :D


Will you spin a new tarball with the missing files included? 1.0.8 is not buildable


Red Hat Enterprise Linux Server release 6.7 (Santiago)


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


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


Jesse,
    Was there a reason that the global switch was removed here?

it seems that this should have been: if (p.task instanceof FlyweightTask && Jenkins.FLYWEIGHT_SUPPORT)


fixes #301


So we could probably do this, right?

    if (!response.rows[i].cdb_columnnames.indexOf('the_geom')) {

    }


@vlad-ignatov can you clarify whether this is still relevant? Probably this reference should be removed (rather than commented out). But in any case, there's no reason comment it.


What does async actually do?


This change needs a comment -- did `jQuery.extend` fix a bug here?


what is this function suppose to do? ```loadModuleModule...```?


To make it easier to use I would introduce `get_mark_tooltip(mark)` and `set_mark_tooltip(mark, tooltip_text)` that invoke `setattr` and `getattr` like you do, because using these built-ins is a little hacky (even it's completely legal and okay, but you dynamically generate/access an attribute).


@oliverguenther FYI. Please check.


It seems this commit removed the Fly Weight support switch.  Was that intentional?


again seems to be duplicated


:+1: I knew we had a helper for it somewhere!


@overtron there seem to be some dot files in the commit. Should those have been gitignored?


Create subtitle


@pgavlin PTAL


94006714-MultipleTowers (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1801 of 1801 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 ATK :: source code Package [Build 8267](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547557&buildTypeId=Gao_PackageSourceCode) outcome was **SUCCESS**
Summary: Running Build time: 00:13:32


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


Better to be explicit about what is being imported?


Except that there's this condition now that the current phase must be in the symbols valid set.


we could make a rotation with alpha beta gamma delta angle
and a CRotationMatrix44
with rotation logic from
https://en.wikipedia.org/wiki/Rotations_in_4-dimensional_Euclidean_space,
or we can use quaternion properties to realize a rotation:
https://en.wikipedia.org/wiki/Quaternions_and_spatial_rotation

Any opinions?


still working on routes "user/:user_id/cards/:id"


also, console


Same thing here with explicit imports.


We want to compute only 1 time the random number per block to place, not per potential ore to place per block. Whether we're using weight and ratio, the configuration values can't be directly used, we need to do a pre-computation pass to define thresholds for each possible blocks in each stratum. Hence, in my opinion, the ratio vs weight doesn't change much on implementation side.

From user point of view, defining weight lacks fine control, unless you use big numbers.
More importantly, when I assemble my modpack, I care about how many block of that ore can be found per chunk, I don't care about weight. With weight, I need to check all others ores to find the actual ratio of that one in the chunk. I've the same problem with Bukkit plugins doing casino/random bonuses, using weight is a nightmare to configure.
Interestingly, you like the idea that, when adding a new ore block, it scales everything, while I consider it a nightmare for balancing the game. It might be better to implement both approach in the end.

We could add a stratum wide scaling option to adjust all ores globally up or down, it's easy to add through I don't see much use cases for it (you need more than simple scaling to adjust difficulty).

In my mind, there's no hardcoded category. It's the structure xml that can reference any category by name in its different layers of construction. Hence, I think I've missed your point here :)

Metadata is mandatory for WarpDrive gases, ThermalExpansion ores, GregTech blocks, UndergoundBiomes stones... Using ItemStack to code them sounds like an overkill of memory. A block is static, so it's just a 'pointer' to it, while an ItemStack is a whole new object consuming memory for many properties while we only use 1 nibble in there.
A block+metadata by itself is useless to us, we need the weight/ratio with it, so we might as well have our custom class to hold them together.


Quick an dirty, I didn't bother loading permitted params from a private/protected method.


this is a cache resync job that is running, to add any new tenants or users into the map.


thanks !!


I despise these nested types


This actually would not even be run as err won't be nil !! But yes, I should fix this.


I think `hint` is misleading here, I *think* its the page number within the directory.


They would be the same. Unless and until the gophercloud changes


I guess most of them should be "const char *"


5110_stdbaropress_sealevel (nmerket) - x86_64-MacOS-10.9-clang: OK (1773 of 1792 tests passed)

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


@benoitg I wrote out your example using a new dist of raven.js (of master) on jsbin:

http://jsbin.com/qadatonofa/edit?html
http://output.jsbin.com/qadatonofa

Again, it seems to work fine for me. Can you take another look?

Also, re the `require` call – are you using this example with Browserify?


Ese no es el Fix, eso es algo que agregue que no estaba. El fix esta en este commit https://github.com/datal-org/datal/commit/cbbaa28cc3df4a39dad97b3b1eff2d2e98cd5b91


maybe better to minimize C code and do formatting on go side?


np, I'm publishing garbage versions bumps all the time! my workflow is a bit too efficient and I stumble sometimes


Yeah that should work, but do we not need to test for the `the_geom_webmercator` field as well? Or does that have it covered.  That's slick, I'm assuming it's faster this way than testing the string equality?


What is the significance of the sentence? It doesn't feel like it fits in the paragraph. Is it even needed?


Build summary: https://gist.github.com/rocco408/4801bd54a97fa22eb122


On some micro-benchmarks, this was a 25% perf gain.


Esto no nos dio ya problemas en el proyecto de pawnshop ??


this should be:
```javascript
.on('zoom', this.zoom)
```


Do you remove the mark again?


Por favor prueben ese fix y me dicen. Yo estuve cambiando JS sin problemas todo el dia de hoy con el tema de la visualizaciones


sure. that makes sense


No, it doesn't fix a bug. I tried to set up a script for automated backups and ran into issues with permissions/ownership when trying to run this cronjob. The only way I finally managed to do this was to change the group of the db files, which will allow you to run a cronjob as enketo user (that copies the enketo-main.rdb).

When installing enketo the enketo-main.rdb should be generated automatically (when the first redis save is made, i.e. after launching your first form).


We need a <object /> element side by side with shell so we can place derelicts in asteroid fields or inside a moon.
Attributes would includes relative position, chance of occurrence, random offset on each axis, schematic name.

Schematic element would include it's file location (ideally a worldedit format file), with a list of id vs block name to convert it. Each block may have optional blocks (like fillers) so reinforced stone is obsidian if IC2 isn't present. We could encourage using only vanilla blocks in the schematic for simplicity.
For initial release, schematic would be hardcoded existing derelict.


mamilton daño el programa!!


the problem with that is, this doesn't even work. it needs to pass parens to the this.zoom function, and can only do that by adding the lines of code below


What does this give us?
A few lines below you left the same thing unchanged.


The values are defined here in properties file.  some of the properties are re-defined in test case properties. Does it needs to be defined in both places?


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2203 of 2245 tests passed)

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


Spot


can we replace the bullets with numbers?


Bizarre, sys.stdin.encoding devrait retourner un encoding du genre FR_UTF8, pas None.
Bon c'est pas trop grave j'avais mis ces debug pour maîtriser mon environnement, y en a plus besoin, je les virerai au prochain commit.
En attendant il te suffit de commenter les 3 lignes de debug d'encoding.


correct, but none of the energy weapons have any capitalization, not even the captain's


Added the SRS that is needed for the QA fundamentals team project.


:+1: yay! :)


Me :p


In Soledad's case, all u1db tests passed and the exception remains there to avoid mistakes. I removed to get persistence out of document and ensure that it is done by caller.
I didn't knew about leap_mx usage of this class... In this case I will review and re-run leap_mx tests to make sure that it is not broken...
I think that if this dependency is there we should have tests or create something to decouple them. Like api.py on soledad.client: Something that we can change the underlying implementation without breaking leap_mx.


This only caches the first 256 vTable entries. The memory usage goes up somewhat, since we have a large mostly sparse array of size: `last classId to be loaded * 256`. I chose the `256` number arbitrarily, we can reduce this.

This caching scheme allows the compiler to emit `flatLinkedVTableMap[classId << 8 + vTableIndex]` which on some micro-benchmarks is nearly a 25% gain.


It makes the `run` function return a `Promise`, and that `Promise` executes in its own `Fiber`. The `Promise` takes care of catching errors (so the `try`-`catch` block is no longer necessary) and we no longer have to create and `.run()` a `Fiber` explicitly. In other words, the new code is functionally equivalent to the old code!


not sure what this is, diregard


if we aren't using this, then disregard


why not tests againts 404 ?


ZoneVentilationDesignFlowRate (kbenne) - i386-Windows-7-VisualStudio-12: OK (2222 of 2242 tests passed)

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


Make sense. Will drop the except


:+1: I'm still amazed the neat solution to flatten. I think it's a good representative for things that initially it seems are not possible in this kind of languages.

typeof + type inference seems to be a good clue for the extra mile here.

:tada:


Just a quick fix to make the test compile/link on 386 linux, which also matches the 64 bit linux version. The line below you mention is for djgpp, I don't know if that needs the change also, so I didn't do that.

Sent from Samsung tablet

-------- Original message --------
From Stas Sergeev <notifications@github.com>
Date: 19/08/2015  21:58  (GMT+00:00)
To andrewbird/dosemu2 <dosemu2@noreply.github.com>
Cc Andrew Bird <ajb@spheresystems.co.uk>
Subject Re: [dosemu2] Test: reorder linker flags for i386 (74991f5)

What does this give us?
A few lines below you left the same thing unchanged.

—
Reply to this email directly or view it on GitHub.


Ion thought that `fp`, `sp`, and `pc` were doubles and was producing terrible floating point arithmetic code. This was discovered using Jit Inspector, and using the `-D` flag in the jsshell which dumps out Ion LIR.


develop (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)


8 here is confusing
I guess you should start from checking signature
if it matches you can assume it's definetly generated by you and size is correct and you can just check time


if id is not provided it will be nil by default, so you don't need to include this. Notice in my Build class I don't include Title in default options


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

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


I would probably set the default to 1. When I make an Item I generally intend at least 1 to be used.


Looks much better.   How can we add the DB Visualizer (which is a weird static link)?


Oops. Sure..


Using `|0` everywhere helps Ion remove overflow checks. Probably not a huge deal on x86 but sill, every bit helps.

In this patch, I also deduplicate some operations. We don't really care about types anymore, so `LALOAD` is identical to `DALOAD`.


@ryoknk i never released the version with label as stable, but the current beta is working in a production store running Prestashop 1.6.0.14.

You can download it from https://github.com/Exadra37/cashondelivery/releases/tag/1.1.0.0-beta.1-exadra37.com


If we end up in dictionary mode anywhere, we are hosed.


i get this. i reported on the forums too:
Membership Level 	Start Date 	           Date Modified 	End Date                  	Level Cost
Premium 	             October 28, 2014 	October 27, 2014 	November 30, -0001 	$0.00 now.


maybe .ebuild demands USE_DOTNET=""${EBUILD_FRAMEWORK}"" ?


But the 64bit version is not compiled by
default, and 32bit version is _always_ compiled,
even on a 64bit host.
That said, it compiles well for me, and I am really
surprised such a change is ever needed.
It would be nice if you extend the log with a bit
of explanations, the error msg you see etc.


Should the template have a `.cfg.j2` extension or is that just a convention? (But even then it might be better to follow the convention)


to be honest I don't like this part but for now I still think about the possible ways of improvements here


101280704_LowTempRad (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (751 of 751 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-7.25%25-green.svg)


@derekcicerone , Schimpf just asked me to yell at you.


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


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


As discussed, it would be better to have this on trappy.  Just making a note here 😄


I think the ID shouldn't have a number restriction. If we're validating IDs, then we mind as well have a list of every single ID in the game, and compare it to that. This may be something we add later, but right now our application is API call free.


I don't think we should validate types in a ill typed language. It's part of what we accept by using one. I would however verify this is 0 or higher.


LGTM. Merging this commit the old way


Missed that, will fix


These are actually public fields, I'm not sure we have any specific guidelines for them


hehe, you found me out :)


Can we add an 'else' part with an error here please, just in case :-)


Can we remove this debug message?


Not sure if we should leave this debug message here...


FYI, I have submitted PR #647 to that effect.


4590-VRF-defrost-power-calculation-problems (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1275 of 1275 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.96%25-green.svg)


@LorenzMeier probably Eigen may change this in the mean time, given that it does the allocations dynamically. I will apply the migration on it, test it, and let you know.


Ok, makes sense.
I'll check how this interacts with my scripts.


Not only this. The added methods to Document makes this necessary, but refactoring is not over on this point as the commit message.
As soon as persistence and logic are separated on this file we can move it to a module and leave Document simple again. :)


for me it is better to see what exactly ebuild demands. The selected version will be printed in the next line of log, so you I be able to compare


Can you add a block comment here that explains what is going on and how this validates that the right behaviour is occuring


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


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


empd (jdwoods21) - i386-Windows-7-VisualStudio-12: OK (1795 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-99.89%25-yellow.svg)


`\n`


:+1: * 2


This looks really cool! I wonder if there be a good way to automate satisfying protocols since there seems to be a lot of repeats, genetic code now.


That's correct, however bumping a major or minor means resetting the lower
version levels back to 0. So 2.8.6 becomes 3.0.0 and 1.0.8 becomes 1.1.0
On Wed, Aug 19, 2015 at 16:54 Nikhil Mashettiwar <notifications@github.com>
wrote:

> My understanding was changing a function signature is supposed to bump
> major version.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/docusign/DocuSign-Node-Client/commit/f5157dda71763cbeb4596fe0619cb38bcce035e1#commitcomment-12788191>
> .
>
--
Cheers,
CJ


Minor nitpick: still missing a period at the end of the comment there :P


:wave: hi code


deffer free?


Found the stray bar character, see my latest commit


@Juan33650 VAMOS CON LA GRID CARAJO!


@wushuang2012  I am not sure if you are done or are still working on this. If you are done, did you test this on Meteor?
Please test and let me know when you make changes in the future, as figuring this out took me time. Thanks for the quick change.


I have no idea how this works! ;)


@embooglement


I don't see an obvious way to correct this, I /think/ the guarding code needs to go in the caller of flushMemoBuffer somewhere....


I would suggest changing line 11 to:

Everyday chat is in `#general`, where every new member is added by default. There are also other channels such as `#hackathons` and even some for student year groups, `#year1` through `#year4` for Years 1 through 4 respectively.


same boilerplate repeated 4 times. function?


Yes, better yet if it was  HOST_OSX


Change the commit message to say "Define", present tense imperative.


I'll fix that.


Brakuje mi html5 :-1:  Jebac IE


Thank you! fixed in fb46ee69b8e6375597c68e54b5a390108cc584bd


looks like you fixed close here


If you're not going to do anything based on the `KeyError`, I don't see any reason to not just use `.get`.


last part doesnt do anything but should not have been included in this commit


D: http://geradormemes.com/media/created/o08bb7.jpg


why import


Extra whitespace @embooglement


:penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin: :penguin:


This seems quite good.
Do you pass the wine tests ?

Have you weighted that 'hook' approach versus Mesa calling a function that would return the result ?
The advantage of the latter is that if wine needs to do some special thing in the future to get it work correctly, it can more easily if it is being pinged about the status. Just to give an example on wayland you don't treat events unless you explictly read the list of received events. There could be some similar thing in wine in the future who knows.
But even if you opt for your hook approach, have you looked whether a 'volatile' keywork should be added to the variable ?


btw you can chain the .start() call in the same line as getFlowManager()


5110_stdbaropress_sealevel (nmerket) - x86_64-MacOS-10.9-clang: OK (1774 of 1793 tests passed)

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


Read


The whole function needs to move class to the line
.... classmethod(...)


Enable


didn't work - still broken on production.


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


Still causing an issue


Hi ph87, the PR looks good, however we would like to have test case(s) to cover your new logic. Would you mind to add test case and update your PR again? Thanks in advance!


if submenu not in submenus and label not in labels:


call the variable what it is.. e.g. menu_names


If reverting remember this are also applied to 4.8 and 5.0
ebda45e  cb7c053


I'm assuming this got fixed during the refactor, but this should be health not max health right?


loop thru   .items()
for key, value in menu_names.items():


Ah yes oops. That's my mistake.


True


I would translate it little different:

Łał! Czy to nie było niesamowite? Możesz teraz podejrzeć całą konwersację. Dotknij poza powiadomieniem lub naciśnij symbol fajki, aby zamknąć powiadomienie.


Should we have a desugar here to handle typedef'd array types?


TeamCity pull requests :: netty [Build 70](https://garage.netty.io/teamcity/viewLog.html?buildId=538&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4699, ignored: 16; resource leak detected (new) Build time: 00:22:02
### Failed tests
```
io.netty.handler.codec.http2.Http2ConnectionRoundtripTest.noMoreStreamIdsShouldSendGoAway: <no details avaliable>

```


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


If there is a chance this may be used in more than one thread, the wrapper
should have a a static critical section due to curl_global_init and curl_global_cleanup.


curl_easy_init() can fail, it may be useful to know if that happened.


The print function needs to be imported as I use its keyword arguments to print output to stderr (check out line 105)


curl_global_cleanup needs to be called even if curl_easy_init fails


:+1: -- my only minor objection is that I'm not sure it's necessary to bring in numpy. If you want to generate a pareto variate, you can use random.uniform and scale it appropriately (exponentiate).


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.919.1745f65008ae93f190504ac094a0e8966a6c6579](http://build.palaso.org/viewLog.html?buildId=61578&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:02:48


likely a mistake ?


I was going to add a rich text persistent tile that isn't configured to be insertable into the layout but can still be used with layouts so you're not forced into using field tiles.


No timeout here, may hang forever if it establishes a connection. (Establish connection timeout is default 300 seconds though).


This vector of size 0 is used as if it had a non-zero size at line 3887 below. Crash is pretty obvious

```
#0  0x00000000004a9ca5 in kaldi::nnet2::Convolutional1dComponent::Propagate (this=0x7ffcf8b39b20, in_info=..., out_info=..., in=...,
    out=0x7ffcf8b38ba0) at nnet-component.cc:3887
3887            column_map[index] = p * patch_step_ + s * patch_stride_ + d;
(gdb) bt
#0  0x00000000004a9ca5 in kaldi::nnet2::Convolutional1dComponent::Propagate (this=0x7ffcf8b39b20, in_info=..., out_info=..., in=...,
    out=0x7ffcf8b38ba0) at nnet-component.cc:3887
#1  0x00000000004884ec in kaldi::nnet2::Component::Propagate (this=0x7ffcf8b39b20, in_info=..., out_info=..., in=..., out=0x7ffcf8b38ba0)
    at ../nnet2/nnet-component.h:214
#2  0x0000000000480fae in kaldi::nnet2::UnitTestGenericComponentInternal (component=..., in_info=..., out_info=...)
    at nnet-component-test.cc:46
#3  0x0000000000482591 in kaldi::nnet2::UnitTestGenericComponentInternal (component=...) at nnet-component-test.cc:222
#4  0x0000000000483421 in kaldi::nnet2::UnitTestConvolutional1dComponent () at nnet-component-test.cc:393
#5  0x0000000000487884 in main () at nnet-component-test.cc:893
(gdb) l
3882
3883      // build-up a column selection map
3884      for (int32 p = 0, index = 0; p < num_patches; p++) {
3885        for (int32 s = 0; s < num_splice; s++) {
3886            for (int32 d = 0; d < patch_dim_; d++, index++) {
3887            column_map[index] = p * patch_step_ + s * patch_stride_ + d;
3888          }
3889        }
3890      }
3891      patches.CopyCols(in, column_map);
(gdb) p column_map
$1 = std::vector of length 0, capacity 0
```


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


I did not intend that, thank you.


"The present group version has been bumped to 2 as the new function ID3DPresent_RegisterWindowHook is required."
probably needs to be changed in the commit message.

Beside about commit messages, please always add an empty line between the commit title and the explanation messages. Somehow else git rebase is really disturbed and for all your commits I have to add the empty line manually to get it ok.


1761_add_zonehvac (kbenne) - 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)


It fills up the logs with useless info on any loop that was polling.  Removing it instantly increased the readability of a lot of our runs.  I would like to leave it out.


Your library has 2,973 downloads in a month. Assuming anyone else is sending requests at an average rate of 10 requests per minute, and 1 in 10 of these requests are going into zombie state, with no reply from Parse, 4,281,120 requests are hitting Parse and not timing out consuming precious blocked resources for a lot of time. Just a thought :-D


Might it be useful to log that subnets == 0? Something is probably wrong. Since we log "Tor IPs download succeeded" regardless, we may mislead the user.


:+1: #Inspect2016 :smile:

Glad you git it figured out!


This commit seems to break anonymous login in as `_logOnDetails.account_name` is null and so `doLogin()` is never called during or after the sentry file handling.


@datakurre  Or maybe "selectedLayout" and "customLayout"? I don't think the terminology we're using here is optimal either.


Comma before "cokolwiek robisz".

Last sentece I would translate like this: "Dotknij przycisk "rozszerz", aby zobaczyć, co się stanie :)"


just override the effect in a spellscript :)


Probably the compiler is smart enough to not create a copy when returning std::string and assigning it to a const reference. (In my small test I even had to modify the string when assigning it to 'std::string kind' to trigger an actual copy).

The getString() cannot (officially) return a string reference as the empty string that might be returned is  a local temporary..
With the previous version I was just sure that no copy will be made :)


I guess I was lazy. p.a.blocks.utils.resolveResource seems like the correct way to go!


You could just get rid of the sysStart and sysEnd assignments and just do them directly to the result object


Addressing your comments in #12926


There should not be a space after the sentence


Thanks- I just committed a fix.
Dan

On Wed, Aug 19, 2015 at 2:36 PM, Kirill Katsnelson <notifications@github.com
> wrote:

> This vector of size 0 is used as if it had a non-zero size at line 3887
> below. Crash is pretty obvious
>
> #0  0x00000000004a9ca5 in kaldi::nnet2::Convolutional1dComponent::Propagate (this=0x7ffcf8b39b20, in_info=..., out_info=..., in=...,
>     out=0x7ffcf8b38ba0) at nnet-component.cc:3887
> 3887            column_map[index] = p * patch_step_ + s * patch_stride_ + d;
> (gdb) bt
> #0  0x00000000004a9ca5 in kaldi::nnet2::Convolutional1dComponent::Propagate (this=0x7ffcf8b39b20, in_info=..., out_info=..., in=...,
>     out=0x7ffcf8b38ba0) at nnet-component.cc:3887
> #1  0x00000000004884ec in kaldi::nnet2::Component::Propagate (this=0x7ffcf8b39b20, in_info=..., out_info=..., in=..., out=0x7ffcf8b38ba0)
>     at ../nnet2/nnet-component.h:214
> #2  0x0000000000480fae in kaldi::nnet2::UnitTestGenericComponentInternal (component=..., in_info=..., out_info=...)
>     at nnet-component-test.cc:46
> #3  0x0000000000482591 in kaldi::nnet2::UnitTestGenericComponentInternal (component=...) at nnet-component-test.cc:222
> #4  0x0000000000483421 in kaldi::nnet2::UnitTestConvolutional1dComponent () at nnet-component-test.cc:393
> #5  0x0000000000487884 in main () at nnet-component-test.cc:893
> (gdb) l
> 3882
> 3883      // build-up a column selection map
> 3884      for (int32 p = 0, index = 0; p < num_patches; p++) {
> 3885        for (int32 s = 0; s < num_splice; s++) {
> 3886            for (int32 d = 0; d < patch_dim_; d++, index++) {
> 3887            column_map[index] = p * patch_step_ + s * patch_stride_ + d;
> 3888          }
> 3889        }
> 3890      }
> 3891      patches.CopyCols(in, column_map);
> (gdb) p column_map
> $1 = std::vector of length 0, capacity 0
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5526c218f988018472378fb9b375950826802fdd#commitcomment-12795296>
> .
>


(Y)


This should have a corresponding test case in  test-mi.


101280704_LowTempRad (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1784 of 1799 tests passed)

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


Should the right side be `(p * patch_step_ + s) * patch_stride_ + d`? Or maybe even  `(p * num_splice + s) * patch_dim_ + d`. I am not sure I understand the intent.


Gonna write out an example of the Product side of things to make sure I understand the intent of the model

* Product
  * id: 1
  * name: "Hockey Stick"

* SKU
  * id: 1
  * product_id: 1
  * name: "KIDS_STICK_54"
  * id: 2
  * product_id: 1
  * name: "ADULT_STICK_89"

* OptionType
  * id: 1
  * name: "Color"

* OptionValue
  * option_type_id: 1
  * value: "Red"
  * option_type_id: 1
  * value: "Blue"

Does that look right?


I agree with pawelkw. "Motywy" means "Themes". "Schematy kolorów" means "Color schemes". In default strings.xml there's "Themes" so it should stay "Motywy".


Are brackets needed for licenseName?


develop (kbenne) - 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)


If you use a function as a model, then this is different. The model is only like a getter, there is no setter. This you have to do it manually, using something like `ng-change`, like in the example.


Yeah, it's expecting a list for some reason.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 36](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=104&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 03:01:33


try https://github.com/nZEDb/nZEDb/commit/efbdc7564c0c90597049ad17db5122235a116d8d


I think "popup options" means popup window options, not popup menu. "Menu kontekstowe" means "popu menu". So i think it should be translated to "ustawienia wyskakującego okienka".


The code as it is is plausible, at least.
I'm a bit confused as to how this code worked for Xingyu before, though.
Dan

On Wed, Aug 19, 2015 at 2:44 PM, Kirill Katsnelson <notifications@github.com
> wrote:

> Should the right side be (p * patch_step_ + s) * patch_stride_ + d? Or
> maybe even (p * num_splice + s) * patch_dim_ + d. I am not sure I
> understand the intent.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5526c218f988018472378fb9b375950826802fdd#commitcomment-12795457>
> .
>


Thanks for replying.  For whatever reason, using ng-change causes an error where my data does not fill in, but ng-click works.  I just can't figure out how to get anything back that specifies whether or not the box is checked.

Sent from my iPhone

On Aug 19, 2015, at 4:46 PM, "Adrian Ber" <notifications@github.com<mailto:notifications@github.com>> wrote:

If you use a function as a model, then this is different. The model is only like a getter, there is no setter. This you have to do it manually, using something like ng-change, like in the example.

—
Reply to this email directly or view it on GitHub<https://github.com/vitalets/checklist-model/commit/52e4be0801f180a41c37725b25a554f8c22278a2#commitcomment-12795484>.


@alankay82, this change is wrong because you are calling startup() on the button widgets even though they aren't yet attached to the DOM.   Also IIRC startup() isn't necessary for either Toolbar or Button.


On second thought, you are right - the `||` should  be `&&` since both conditions should evaluate to true in order the column name to be pushed to the array.


Totally understood on the experimentation front.

The change from the setters to updateDom*() methods completely satisfies my concerns.  Although I do love the "react.js as a functional rendering of state" thing, my reference to props here was mainly that it's a case where from the standardized idiom it's 100% clear that we're passing state to a sub-widget that is responsible for reflecting that state but also has no ownership/control of that state.

And the updateDom* stuff makes this clear in the naming convention.  Just like seeming DOM calls on a property with an "elem" or "node" in it is usually pretty clear.  (Or grabbing anything on-demand via querySelector/getElementById/etc.)  Or having widget on stuff, etc.

I had forgotten that custom elements did get the attributeChangedCallback that makes them like XBL.  That would definitely be a workable idiom.

Attempting to boil down my request to one sentence: I would like it to be clear from context whether a line of code is saving state, performing program logic, or updating the DOM.  I don't deeply care about how that is accomplished; I know you have good design style :)

(What I mean by performing program logic is that I prefer the idiom of a method call versus magic setters in all cases.  Especially now that we have destructuring assignment that could hide the invocation of a magic setter amongst other state writes.)


Can you create a plunkr or jsfiddle example?


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


It should stay as it was. "Aktualizacja koloru ikony QKSMS" - it's good translation from original text.


I will try.  Thanks

Sent from my iPhone

On Aug 19, 2015, at 4:51 PM, "Adrian Ber" <notifications@github.com<mailto:notifications@github.com>> wrote:

Can you create a plunkr or jsfiddle example?

—
Reply to this email directly or view it on GitHub<https://github.com/vitalets/checklist-model/commit/52e4be0801f180a41c37725b25a554f8c22278a2#commitcomment-12795575>.


@brianlball FYI, new design doc for Static Pressure Reset


Someone was complaining about lack of peer review of this feature on the mailing list, so I took a look at it. Now you have one more.

It looks fine. I just did some nitpicking.

If the unittest framework allows, I think it would be nice to have test that checks that InitTorIPGroups is not called if any of the conditions we don't want it in exist (disableipprio, !fListen, proxy for ipv4 _or_ ipv6 _or_ tor)


Oh and I forgot:
Commit messages should try respect the 80 char per line maximum policy


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


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


I would remove word "tak".


Thanks @boq
My IDE did the opposite. I'll change the style.


s/this.//


Nice one


"Pokoloruj wysłane"


Pokoloruj odebrane


@samsonasik Because you specifically reported you were getting a 405. Additionally, getting a 404 would be dependent on having the FinalHandler in place; I just need to see what is returned from invoking the application instance.


Curious. Why define an alias for "struct"?
What does it achieve?


this fixed it


Considering this changed entirely over the last few days, yes. You could've easily checked on master to see what it looks like now instead of commenting on a massive commit from 10 days ago.


leap_mx usage of this file is quite simple; :D
https://github.com/leapcode/leap_mx/blob/develop/src/leap/mx/mail_receiver.py

Looks good.


Please omit "this." here and everywhere else.

It's only used when there's ambiguity e.g. in a constructor.


I am not following you here.

Please be more clear, with some extra explanation.

Noam

________________________________
From: Vineet Gupta <notifications@github.com>
Sent: Wednesday, August 19, 2015 7:14 AM
To: EZchip/linux
Cc: Noam Camus
Subject: Re: [linux] ARC: Adjust cpuinfo to work on sparsed mask (a95f79a)

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 !

-
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/a95f79acf3d7457b0da6f304f5dd8775eeebf7a9#commitcomment-12775643>.


OK good.   After you update that it should be looking good.  But one other question:  You are supporting three types of queries:

1. a function
2. a hash like `{foo: 1, bar: 2}`
3. a dstore/Filter object like `{type: "ne", args: [ ... ]}`

There's also nesting.    My question is, can type (1) and (2) ever appear nested inside of (3)?   If not, then the code to handle (1) and (2) should be pulled out of the recursive _compileQuery() method and just be run on the type level args.query object.


This is used by the current indexer?

If yes, we should probably not touch it but copy the method for v2


Thanks

I will apply all your comments.

Can I rebase?

________________________________
From: Vineet Gupta <notifications@github.com>
Sent: Wednesday, August 19, 2015 3:53 PM
To: EZchip/linux
Cc: Noam Camus
Subject: Re: [linux] ARC: Set vmalloc size from configuration (c3eb0e3)

Scratch my prev comment !
Instead of hard coding 0x2000_0000, lets hard code 0x6000_0000 (i.e. TASK_SIZE), and derive USER_KERNEL_GUTTER from it (as it is used id pgalloc etc)

-
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/c3eb0e353970e5e225c0469da81b1ea194ce4c1f#commitcomment-12783244>.


spoke to soon.. it now runs, without an error but goes into a loop.. take a look at screenshot
![2015-08-19 17_59_33-mtputty multi-tabbed putty](https://cloud.githubusercontent.com/assets/13871539/9370157/20f5f67e-469c-11e5-8cfe-7b102273a8cd.png)


"Pokoloruj odebrane"


Yes, I think we should allow `pitch` to be set and retrieved outside of the camera, just like zoom, etc. `MGLMapCamera` will be powerful, but probably a bit more verbose than `mapView.pitch = 45`.


Это константы лучше хранить в одном месте. Лучше в `OrderConfigurator`. Там просто сделать их `public final`, это ок.


Si. Es útil tener una grid. Estuvimos conversando y la simplificaremos. Más
aún. Mañana la empujó.
El 19/08/2015 18:21, "Gastón Avila" <notifications@github.com> escribió:

> @Juan33650 <https://github.com/Juan33650> VAMOS CON LA GRID CARAJO!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/datal-org/datal/commit/26a5e3c38646f3aedb478d7405711111074297ea#commitcomment-12795036>
> .
>


nit: shouldn't this check that refCount is greater than 0?


My documentation comments are really bad, btw — should be improved.


I couldn't find that text in the app, but in original there is "color the background" not "background color" so I think it should be "Pokoloruj tła dymków wiadomości przychodzących".


Wow good catch!


@sbenesventuretech Do you have any idea how to make this function correctly? I'm trying to get the currently selected tags/categories in an existing blog post to prepopulate as tags.


nope, it does not get same error, but now it just goes into a loop, take a look at the screenshot.
![2015-08-19 17_59_33-mtputty multi-tabbed putty](https://cloud.githubusercontent.com/assets/13871539/9370214/89a15150-469c-11e5-9f05-dd26bbbe75e0.png)


I am never able to get Plunkers to work because I am new to Plunker.  But the code is here:
http://plnkr.co/edit/69OeErHUzJEuB0k12H9x?p=preview

Anything assistance is greatly appreciated!

Beryl Small
IT / Programmer
985.629.5934 (D)
800.892.8179 (O)
985.892.8114 (F)
http://www.pecsafety.com/
This e-mail is confidential and is intended solely for the use of the recipient.  If you are not the intended recipient, be advised that you have received this e-mail in error and that any use, dissemination, forwarding, printing, or copying of this e-mail is strictly prohibited.

From: Adrian Ber [mailto:notifications@github.com]
Sent: Wednesday, August 19, 2015 4:52 PM
To: vitalets/checklist-model <checklist-model@noreply.github.com>
Cc: Beryl Small <Beryl@pecsafety.com>
Subject: Re: [checklist-model] function as model fix and test case (52e4be0)

Can you create a plunkr or jsfiddle example?

—
Reply to this email directly or view it on GitHub<https://github.com/vitalets/checklist-model/commit/52e4be0801f180a41c37725b25a554f8c22278a2#commitcomment-12795575>.


I have Parallels Desktop 10.2.2. and creating it works fine. What kind of docker support is Parallels 11 providing that 10 is not?


newb question: if the ISE is thrown below, won't this object still be in an inconsistent state?  Other threads or locations that had references to this AbstractReferenceCounted object will still be able to use it, even though it is corrupted.


Leaving this comment here since there's nowhere to actually open issues for these plugins.

This plugin DOES NOT work with iPad devices in iOS8 with landscape mode.  It causes the viewport to have both incorrect height and width in either landscape or portrait mode.


101280704_LowTempRad (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1275 of 1276 tests passed)

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


Мне кажется, лучше тут убрать `-1`, а вызывать `isFreePlaceNearby(tilePlace - 1, freePlace - 1)` выше, чтобы `-1` были везде в одном месте.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 366](https://teamcity.labkey.org/viewLog.html?buildId=267488&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:52
### 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>

```


"dla każdej rozmowy" instead of "w każdej rozmowie". That counter is shown in main screen, not in the conversation. And also I would prefer "liczba" instead of "ilość".


Well when you are right, you are right, I will not change author cause I commited this. If you want I can tag him or add him to title.


Веселись :smile:


ConfigureTransitionIDDname (Myoldmopar) - 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)


test!!!!!!!!!!!


Hello


Jesse, the updated W3C ARIA document you reference in your intro comment is a working draft from 2009. The updated 2103 working draft is here: http://www.w3.org/TR/wai-aria-practices/#keyboard.


Or maybe "Pokaż dymek"? "Powiadomienie" is "notification". When I see "notification", I'm thinking about status bar notification rather than toast.


1761_add_zonehvac (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2204 of 2256 tests passed)

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


Hi just updated to 0.94 but this setting are not in the Advanced tab
can you push this too?
thanks


I don't get what this does but ok :)


Ah, I did not realize, thank you for clarifying!


I don't like dead code in the codebase, take it out


No worries. What I would do is maybe revert this commit and commit it again with the authorship in place or like you said, tag him in the commit message so if someone was to cherry pick this, @bigrushdog would get the credit he deserves :-)


Why not simply use [[NSLocale currentLocale] localeIdentifier] ?
The only difference is that sometimes there is a @calendar=... at the end.


(The important things)


OMG my English is terrible, probably because I AM English to be fair. Can I blame my Cherry MX Red switches? They aren't the best for typing on.


ok, now it got 200 like shown in #76


nit: methods named get typically return something.  Considering picking a different name.


`http://localhost.evil.com` isn't a local domain.


Methods with doc strings, yay!


We don't want this. Its only purpose is consuming memory.


`//evil.com` isn't local.


ConfigureTransitionIDDname (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: 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)


We want this condition instead.


Any tips or suggestions \cc @sokra?


Hello


Thank you @JaredReisinger. I will have a look at this.


This is not thread safe: http://www.bing.com/search?q=static+local+variables+c%2b%2b+thread+safety

You should use ConfigDWORD to cache the value instead:

```
static ConfigDWORD injectionEnabled;
if (injectionEnabled.val(CLRConfig::INTERNAL_ThreadSuspendInjection) != 0)
{
   ...
```


cannae spell canolie


can you verify that this is still a problem? if it is, it looks like we can optimize it:

```java
Instrumentation instrumentation = this.getInstrumentation();
Context context = instrumentation.getTargetContext();
// Wait for the application context to exist (to avoid an Android bug)
// http://stackoverflow.com/questions/6516441/why-does-androidtestcase-getcontext-getapplicationcontext-return-null
while (context == null
    || context.getApplicationContext() == null) {
  Thread.sleep(100);
  context = instrumentation.getTargetContext();
}
```


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1272 of 1272 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)


Also still an issue here?


I share your general paranoia around getters and setters. I have traditionally avoided them because of the side effects submarines. In the edit_toolbar case I was playing with matching the traditional style of the construct: DOM elements have getter/setter style in this fashion. Trying to be sure I was not stuck in my old crusty ways. But then DOM elements have weird horrible stuff around attribute/property reflection too, so maybe emulating their traditional style is not so desirable. :)

I will update edit_toolbar to use the `updateDomX: function(value)` convention instead of `set X(value)`. I might still play around with the property/attribute combo getter/setter thing just out of curiosity. Although it will be a side project, just started having flashbacks from the weirdness in attributes vs properties from Dojo and jQuery library code as I do the experiment.


We should release any instance variables that we're holding on `BaseActivityInstrumentationTestCase2` since there's a bug where Android unit tests classes live way too long and basically leak all their instance variables.


Issues can be reported in JIRA for all cordova core plugins .
https://issues.apache.org/jira/issues/?jql=project%20%3D%20CB%20AND%20component%20%3D%20"Plugin%20Statusbar"


you can use `buffer_receive.data`, it is a `void*` then you shouldn`t need to cast.


This is the first steps in implementing support of the right click menu (#45). I still want to add a setting to specify the behavior when a selection is made on the right click menu. The options should be:
* Replace only the selected word
* Replace all instances of selected word on current page (current implementation)


Afaics the api doesn't provide continuation for wbgetclaims, so I don't think missing support for continuation should be an issue here.


FYI, Chrome currently does not fire these events after reloading the page. Beware of that behavior.

Also, we do not want to allow users to remap "standard" mappings. Remapping are for normalizing all gamepads to the standard mapping. It's not for arbitrarily changing button actions; The `bindGamepad` `bindKey` etc. family is purpose designed for that. #724


1761_add_zonehvac (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)


done.


`super.tearDown()` should go at the end of `tearDown()`


I don't think it makes sense to transfer ownership of the buffer to the rawHandler callback here.  I feel like you should need a good reason to transfer ownership to a callee through a function argument, and a very good reason to do so through a raw pointer.  Also, none of the places that this gets used actually bothers to delete this...


Same RE releasing any instance variables we have


This should be just ```__tablename__ = 'category_rating_link'```


Yes, but as soon as they try to release it, they'll get the same exception.  This works the same as in Netty.


make a wrapper macro for this function, then it will do the sizeof() for you, avoiding mistakes.


Done.


nit: empty newline at EOF


Unless this is some very new syntax, I don't think you need to add ```Base.metadata``` to the ```__tablename__```. Did you see this in the docs somewhere?


Do we even need to implement our own `onCreate` and `onDestroy`?


Is this file used anywhere?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 13](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=108&buildTypeId=XbimEssentials_XbimEssentials) is now running


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 13](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=108&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:00:49


you need to use a temporary variable, `st_size` is actually `off_t`, 64bits if you compile with `-D_FILE_OFFSET_BITS=64` (large file) on 32bits machine. In this case the result will be wrong.


r=jedmccaleb


98318570_RemoveRadianceCheckbox (macumber) - 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)


ConfigureTransitionIDDname (Myoldmopar) - 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-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-7.21%25-green.svg)


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/2f87cfc337e9f5a1264ec3fe1b649a37cce67869


merging graydon/stellar-core/yet-more-prepared-statements = 2f87cfc3 into auto


graydon/stellar-core/yet-more-prepared-statements = 2f87cfc3 merged ok, testing candidate = 8a1bc71c


Fix the error above you may need to move this class definition to the top of the file. It should be above any other table definitions that depend on its class name or ```__tablename__```.


/sys/devices/f9927000.i2c/i2c-5/5-0020/wake_gesture_enabled

default is 0, set it to 1


this will generate a prime 32 bits long...

use one of the `goodGroupParameters` from `tlslite/mathtls.py`


I kept moving them around, but let me redo it to match and re run it.

> On Aug 19, 2015, at 6:25 PM, brianthelion <notifications@github.com> wrote:
>
> Fix the error above you may need to move this class definition to the top of the file. It should be above any other table definitions that depend on its class name or __tablename__.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/amarie004/ndb/commit/1a6d2d48b725bf702219fa6b14fb20bcca045aa6#commitcomment-12796235>.
>


Is the only difference between the `if` and the `else if` blocks supposed to be `foonMatch` and `funMatch`? The code makes it quite hard to figure that out.

Case in point: I can't tell if the `question += "site:wikipedia.org";` line missing a space before `site` is a bug or intentional.


add some documentation to this function in the .h, it already changed behavior and we must document that, say that it will append to buffer starting from `buf->used`, if users want they must `sol_buffer_reset()` before calling it.


Hello


Yeah, I added them on purpose ... was useful as I was working on locking and watching log files to convince myself that the server was doing things in the order I wanted.  But I don't mind terribly if they get deleted.  Also they might make more sense at TRACE since, as you pointed out, borrows happen all the time.


I'm going to add the typography styles back in but they still need to be addressed


https://github.com/cfpp2p/transmission/pull/3#issuecomment-131260424

https://trac.transmissionbt.com/browser/branches/2.7x/libtransmission


@andresgutierrez done https://github.com/phalcon/website/pull/298


done.


woah! didn't know that was a thing!


https://github.com/cfpp2p/transmission/pull/2#issuecomment-131215066

https://trac.transmissionbt.com/browser/branches/2.7x/libtransmission


Both are interesting, try them and see. But first you intelligent young men
have to discover why I can not get an addition constant for z.w to work. It
does work in OpenCL but either i am missing a typo or I do not know enough
c++ to find the bug.

BTW, Another thing I could not do, was change the parameters names to
CVector4 for the UI, so that i could have spinbox4 and slider4.

Cheers

On Thu, Aug 20, 2015 at 8:49 AM, Sebastian Jennen <notifications@github.com>
wrote:

> we could make a rotation with alpha beta gamma delta angle
> and a CRotationMatrix44
> with rotation logic from
> https://en.wikipedia.org/wiki/Rotations_in_4-dimensional_Euclidean_space,
> or we can use quaternion properties to realize a rotation:
> https://en.wikipedia.org/wiki/Quaternions_and_spatial_rotation
>
> Any opinions?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddhi1980/mandelbulber2/commit/7ed64240c4e721f2b151305f0b3739428b553d48#commitcomment-12794466>
> .
>


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


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


maybe add `sol_buffer_new()` and `sol_buffer_free()` to be used in this case?


http://docs.sqlalchemy.org/en/latest/orm/basic_relationships.html#many-to-many
<img width="772" alt="many" src="https://cloud.githubusercontent.com/assets/8523645/9370701/3843dd74-46a0-11e5-99a5-2f24f8622f5d.png">


how does that differ from setting an incorrect `pskIdentity` or `psk`?


Shouldn't this put python2 first? Some Linux distros are going to make the default python Python 3.


`read` will cause a shadow warning in some old compilers (i guess coverity) because it's the function name... and they don't differentiate different symbols for a local integer and the global function.


>Is the only difference between the `if` and the `else if` blocks supposed to be `foonMatch` and `funMatch`? The code makes it quite hard to figure that out.

If that is the case, why are the following three lines duplicated in each block?  Could reduce the amount of duplicated code here.


Check this sol_vector_append()


If you are trying to handle bad/malicious input, this isn't a good idea. What happens is:

1. Exception is printed
2. Code continues executing after the `catch` block. The `searchTerm` that couldn't be encoded successfully is used-as in the `URL`. If there's any malicious text in there, it's not escaped any more. Anything can happen, really.

It would be safer to throw a `UnsupportedEncodingException` from this method and let the caller decide to not search for a term that can't be encoded correctly.

Btw, `URL` looks like a constant. It's actually computed using a variable, so it's not a constant and shouldn't be `CAPITALIZED`.


reloaded base.py with link classes to the top.

new error:
```
======================================================================
ERROR: notssdb.test.test.test1
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/Users/ack/code/venv/NotssDB/notssdb/test/test.py", line 14, in test1
    u1 = api.create_user('dance', 'cats', 'Jen', 'C', 'dancegirl@aol.com')
  File "/Users/ack/code/venv/NotssDB/notssdb/api/object.py", line 35, in create_user
    new_user = User(username, password, firstname, lastname, email)
  File "<string>", line 2, in __init__
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/instrumentation.py", line 347, in _new_state_if_none
    state = self._state_constructor(instance, self)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/util/langhelpers.py", line 747, in __get__
    obj.__dict__[self.__name__] = result = self.fget(obj)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/instrumentation.py", line 177, in _state_constructor
    self.dispatch.first_init(self, self.class_)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/event/attr.py", line 258, in __call__
    fn(*args, **kw)
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/mapper.py", line 2792, in _event_on_first_init
    configure_mappers()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/mapper.py", line 2688, in configure_mappers
    mapper._post_configure_properties()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/mapper.py", line 1710, in _post_configure_properties
    prop.init()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/interfaces.py", line 183, in init
    self.do_init()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/relationships.py", line 1616, in do_init
    self._process_dependent_arguments()
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/orm/relationships.py", line 1641, in _process_dependent_arguments
    setattr(self, attr, attr_value())
  File "/usr/local/lib/python2.7/site-packages/sqlalchemy/ext/declarative/clsregistry.py", line 293, in __call__
    (self.prop.parent, self.arg, n.args[0], self.cls)
InvalidRequestError: When initializing mapper Mapper|Element|elements, expression 'category_element_link' failed to locate a name ("name 'category_element_link' is not defined"). If this is a class name, consider adding this relationship() to the <class 'notssdb.model.base.Element'> class after both dependent classes have been defined.
-------------------- >> begin captured stdout << ---------------------

```


And even though Mypy makes sure it's Python 2, I don't know of any major OSs that have Python 2 without the `python2` binary, so it's more likely to hit the first time.


4590-VRF-defrost-power-calculation-problems (Myoldmopar) - i386-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)


Hello


Hope this works per http://joeytat.com/post/ios/8.-cocoapods-specs-dependency-conflict-jie-jue-ban-fa


Where can I find yang_parser.py that has this issue fixed?
Thanks a lot....


:clap:


:+1: with minor


TeamCity pull requests :: netty [Build 74](https://garage.netty.io/teamcity/viewLog.html?buildId=548&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 4373, ignored: 16; compilation error: Compiler (new) Build time: 00:08:23


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


`IED`


TeamCity ATK :: QA :: Integration Per Branch [Build 1899](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547564&buildTypeId=Atk_IntegrationPerBranch) outcome was **SUCCESS**
Summary: Tests failed: 1 (1 new), passed: 438, ignored: 15, muted: 8 Build time: 02:02:17


How many gerbers would a gerber gerber if a gerber could lightninghoof?


https://github.com/cfpp2p/transmission/pull/2#issuecomment-131215066

https://trac.transmissionbt.com/browser/branches/2.7x/libtransmission


I just installed the latest changes and was very happy to find this toggle! This is of huge value.


nit: you don't need to use wildcard import here


twelveteen


`static` is often used to indicate purely functional code. `static` says: this method does not access any instance variables, it only manipulates inputs. Having  a piece of code that is purely-functional is nice to have, because purely functional code has no side-effects. `static` doesn't really imply no-side-effects, but it's a step in the right direction.


Analise


empd (jdwoods21) - Win64-Windows-7-VisualStudio-12: OK (1795 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-99.89%25-yellow.svg)


Hello


Assignment에 대한 변경은 별도의 커밋으로 빼는게 좋아 보입니다.


Hi again, Jesse, I'm not sure if there is some confusion between a head tag and a header tag in the prototype. I think you might want something more like this:
<!DOCTYPE html>

		<meta charset="utf-8">
		<title>Simulation Title</title>

Simulation Content Title

			version


https://github.com/olear/Natron/commit/378e58ec4d2a71219d5d582e5da2519bdabc17eb, ok?

btw, on Linux the fontconfig cache is triggered regardless of this commit, due to the qt font setup on startup. That should be fixed, since the font cache update run before splashscreen.


This is the best README I've ever seen


TeamCity pull requests :: netty [Build 72](https://garage.netty.io/teamcity/viewLog.html?buildId=546&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5568, ignored: 21 Build time: 01:09:48


valign is depreicated in HTML5 - http://www.w3schools.com/tags/att_td_valign.asp

Use css - http://www.w3schools.com/cssref/pr_pos_vertical-align.asp


1761_add_zonehvac (kbenne) - i386-Windows-7-VisualStudio-12: OK (2223 of 2256 tests passed)

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


bodyAds should be "body-ads" snake case your class names.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/76370753


snake case your id names.


Missing curly braces.


Don't write inline styles. This will make it harder to locate them later. Just commit and write a class in the head or an external css file.


:hankey:


spiffy


Done.


I'm not sure if it's just the commit, but none of your content is indented.


The reremoved labels are used on:

https://doc.owncloud.org/server/8.1/admin_manual/configuration_server/security_setup_warnings.html

and the links are now broken.


feel free to add to it, my mouse doesnt have anymore buttons, and im too lazy to look it up.


We explicitly need this format aka `en-US` to be consistent across Android/iOS/OSX/.NET platforms.
`localeIdentifier` by default gives `en_US` (underscore instead of dash).


This is for www.itrepo.com


1761_add_zonehvac (kbenne) - Win64-Windows-7-VisualStudio-12: 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)


101280704_LowTempRad (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1784 of 1799 tests passed)

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


Derp. This was on the wrong branch.
Dowsn't matter now I guess.


Add more! It's easy! With `node` installed:

```shellsession
git clone git@github.com:kemitchell/american-legal-archaisms archaisms
cd archaisms
./add-word "comes now"
git push
```
No need to PR. I'll review, if ever, when I push new versions to npm.


`serverName=""` will send an empty SNI, use `None`


Thank you so much for this neat little feature @Mazda-- , we love you bro <3


fast-forwarding master to auto = 8a1bc71c


Why remove the hint about open_basedir? Thats mostly the main problem for the message about /dev/urandom in https://doc.owncloud.org/server/8.1/admin_manual/configuration_server/security_setup_warnings.html

and should stay in here from my PoV.


the number should reflect the pull request number


since both the DH part and RSA part is shared with other ciphersuites, moving it to some separate method would make the code cleaner


So this is why I'm pulling my hair out.


TeamCity ATK :: QA :: Integration Per Branch [Build 1900](https://ubit-teamcity-iag.intel.com/viewLog.html?buildId=547558&buildTypeId=Atk_IntegrationPerBranch) outcome was **SUCCESS**
Summary: Tests failed: 1 (1 new), passed: 436, ignored: 15, muted: 10 Build time: 02:04:08


I think someone (me :facepunch:)  forgot to add sol_ptr_vector_clear() for args.json_files. Add here and in the other proper places.


rm, mkdir, cp, chown は実は `os` とか `shutil` パッケージに機能がありますので探してみてください。


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1795 of 1797 tests passed)

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


again, this can be shared with RSA key exchange


Думаю, стоит упростить:
```java
return ((abs(tileColumn - freePlaceColumn) == 1) && (tileRow == freePlaceRow))
       || ((abs(tileRow - freePlaceRow) == 1) && (tileColumn == freePlaceColumn))
```


@EventListener fehlt


CMessageMessage? jk


You deleted comment?


Fixed in https://github.com/ElunaLuaEngine/Eluna/commit/2379c00711d04991ac057ef49ae25416d49daeb1


:+1: with comments


No doubt that this creates better consistency in the naming. However, I question whether, or not, it would be better to move in the opposite direction, replacing Z_MIN_PROBE (and all of its variations) with simply PROBE.


empd (nmerket) - 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)


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 14](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=109&buildTypeId=XbimEssentials_XbimEssentials) is now running


What about this one?


Yeah I read your comment different when I was mobile and then I got back to the computer and realized you were saying what I was saying so didn't want to beat a dead horse...

Anyways, keep shit going man! I see you've changed a lot from the last time I talked to you haha


It's not, the current indexer uses the "collections_create" method above on line 116.  This was another reason why I changed it from create_collection b/c it was confusing me.  :smile:


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 14](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=109&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:00:57


Nice work. Good to see a full game sample :+1:


This needs to be a gamepad `id`. Arbitrarily remapping gamepads based on their index is not going to work out. The `id` is the only way to distinguish which mapping needs to be used, and with multiple Ouya gamepads (just a generic example) they all have the dame `id` but different each lives in a different `index`. That makes this function signature pretty hard to deal with.


amazing stuff. thanks @eviltrout! Also, not sure if you've seen the question on implementing resource instead of routes -> https://github.com/team-melbourne-rgsoc2015/interface-testing/pull/2


You can remove this since we're using symbol just once inside generate_node_type_assignments()


http://puu.sh/jHu5s/1bf3e0ac5a.png
http://puu.sh/jHu7D/45ef91caae.png


ok thanks


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


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


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


After this refactor, default parameter of TrimPrefix() won't be needed anymore. (to be removed whenever we break API)


:+1: with comments


yes, this is still an issue.


This is better accomplished in [logback.xml](https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/main/resources/logback.xml)

In production, [logging of fonts](https://github.com/MarginallyClever/Makelangelo/blob/a67f00b1d5f32113a4d230b98b79505fc1bacd10/java/src/main/java/com/marginallyclever/makelangelo/MainGUI.java#L135) will never happen because we use [Logger#info(String, Object)](http://www.slf4j.org/api/org/slf4j/Logger.html#info(java.lang.String, java.lang.Object)) and logging is set to [error](https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/main/resources/logback.xml#L15-L17). You are most likely seeing it in development due to [logback-test.xml](https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/test/resources/logback-test.xml#L13-L15) which doesn't make it into the release.


@ryantheleach Those are `Career`s, not `Profession`s.


kk


In this case we only need to set `activity = null` right?


`checkNotNull`?


This is added in next commit


Hi Jesse,
I see that the tabindex attribute is used a lot. I'm not sure you need to use it so extensively.
1. links and form elements get focus by default, so generally don't need tabindex on those elements.
2. tabindex = "-1" makes an element can programmatically focusable, but not made focusable to the keyboard.
3. tabindex = "1" or greater should be avoided.
This brief article explains these uses nicely.
http://www.paciellogroup.com/blog/2014/08/using-the-tabindex-attribute/


Yes... but why do the work when it's not needed?  I built the list of fonts
there as a test.  I plan to move it into the "your message here" dialog so
users can choose the font they'd like.

On Wed, Aug 19, 2015 at 4:06 PM, Peter Colapietro <notifications@github.com>
wrote:

> This is better accomplished in logback.xml
> <https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/main/resources/logback.xml>
>
> In production, logging of fonts
> <https://github.com/MarginallyClever/Makelangelo/blob/a67f00b1d5f32113a4d230b98b79505fc1bacd10/java/src/main/java/com/marginallyclever/makelangelo/MainGUI.java#L135>
> will never happen because we use Logger#info(String, Object)
> <http://www.slf4j.org/api/org/slf4j/Logger.html#info(java.lang.String,%20java.lang.Object)>
> and logging is set to error
> <https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/main/resources/logback.xml#L15-L17>.
> You are most likely seeing it in development due to logback-test.xml
> <https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/test/resources/logback-test.xml#L13-L15>
> which doesn't make it into the release.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MarginallyClever/Makelangelo/commit/a67f00b1d5f32113a4d230b98b79505fc1bacd10#commitcomment-12796842>
> .
>

--
Dan Royer :: Marginally Clever <http://www.marginallyclevergroup.com/> ::
Raising Robot Literacy :: +1.604.916.2281


So what's wrong with default `operator=`?


We don't, I just copy paste this from the old repo:)


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2203 of 2256 tests passed)

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


are you available on gchat?  I'm hacking on the code right now.

On Wed, Aug 19, 2015 at 4:08 PM, Dan Royer <dan@marginallyclever.com> wrote:

> Yes... but why do the work when it's not needed?  I built the list of
> fonts there as a test.  I plan to move it into the "your message here"
> dialog so users can choose the font they'd like.
>
> On Wed, Aug 19, 2015 at 4:06 PM, Peter Colapietro <
> notifications@github.com> wrote:
>
>> This is better accomplished in logback.xml
>> <https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/main/resources/logback.xml>
>>
>> In production, logging of fonts
>> <https://github.com/MarginallyClever/Makelangelo/blob/a67f00b1d5f32113a4d230b98b79505fc1bacd10/java/src/main/java/com/marginallyclever/makelangelo/MainGUI.java#L135>
>> will never happen because we use Logger#info(String, Object)
>> <http://www.slf4j.org/api/org/slf4j/Logger.html#info(java.lang.String,%20java.lang.Object)>
>> and logging is set to error
>> <https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/main/resources/logback.xml#L15-L17>.
>> You are most likely seeing it in development due to logback-test.xml
>> <https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/test/resources/logback-test.xml#L13-L15>
>> which doesn't make it into the release.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/MarginallyClever/Makelangelo/commit/a67f00b1d5f32113a4d230b98b79505fc1bacd10#commitcomment-12796842>
>> .
>>
>
>
>
> --
> Dan Royer :: Marginally Clever <http://www.marginallyclevergroup.com/> ::
> Raising Robot Literacy :: +1.604.916.2281
>

--
Dan Royer :: Marginally Clever <http://www.marginallyclevergroup.com/> ::
Raising Robot Literacy :: +1.604.916.2281


I'd say to put it before `sol_shutdown()`, just because `sol_init()` is before `sol_buffer_init()`


yup!


get rid of this extra line and we're golden


Why do you need cloning back and forth here at all?


`testGetItemViewWithCustomLayout `


It was [fixed when merged](https://github.com/SpongePowered/SpongeAPI/commit/0b1cf1f77c82adb2bb345efd97247eb91260165d#diff-c94a2b7359796a94a693e89c08e6dfacL39)...


tags too


No, will delete this.


Nice catch, will delete this.


Floating-point number의 (동등) 비교 연산은 간단하지 않습니다. 아래의 링크들을 천천히 읽어보기 바랍니다.

http://stackoverflow.com/questions/17333/most-effective-way-for-float-and-double-comparison

http://floating-point-gui.de/errors/comparison/


I'm hesitant to allow downloading of arbitrary files in light of https://www.gnu.gl/blog/Posts/multiple-vulnerabilities-in-pocket/.


Both items probably got lost in all the copy-pasting and rearranging. You're welcome to put them back, or I'll do it tomorrow.


파일 끝에 줄바꿈 문자가 또 빠졌네요 :)


You are correct -- thanks for the suggestion. Updated the logic to try `python2` first (on the `hack` branch).


Yes, I learnt a lot... Anyways I do not approve your ways of how you spread all around... We wanted to talk with you but you kept spreading shit about us and obviously you have more followers than us so you would be winning... I remember you wanted us to refund donations, well you acted like police. i did not like that and I do not. Start with unblocking me then :dancers:


nit: newline at EOF


oops we forgot to update version in plugin_info


Aren't we #3 ? #1/#2 would be the xbox itself or the x360 emu :P


@Wackerbarth Some can argue that you don't know what kind of probe will eventually emerge from the future. Personally I think `Z_PROBE` is appropriate, the `MIN_` is perhaps overkill. But for the record, the first filament sensor was labelled `FILAMENT_SENSOR`, it is in fact a filament diameter sensor, later, another filament sensor appeared and was labelled `FILAMENT_RUNOUT_SENSOR`, you see the pattern?


Use the real column name in the error (since it is fixed for the test)


mejor aun con `static final`. Las constantes es comun escribirlas en mayusculas


if we go this route we should get at least an AWS alternative here.

@justinsb


This is not necessary with `expectedExceptions`


Inline this


Hm, why OnUserRaw() doesn't get tags?


Lo mismo que antes


On[Unknown]UserRawMessage()


Yeah when I said to keep shit going, I wasn't implying that I wanted to be BFF's or anything haha I meant it as a pat on the back and saying good job.

Nor was my comment meant to resurface old shit. What's done is done and like I said, it seems you changed a lot so that's a positive.


I originally had a check in here to check against the file size. I ended up taking it out since it still has to download the file to the cache first. Should I put that back in?


develop (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)


should we handle TB/Ti or does GCE's API expect TB to be expressed as 1024GB ?


"Guide de style par subjectif pour Angular par john papa" is not french.
What do you think of : Guide de style subjectif pour des équipes qui utilisent Angular par john papa"


Trust me, I spent literally a whole day trying to script it. I tried overriding and tried different methods (BeforeHit, AfterHit, EffectLaunch.... etc), then I got frustrated and hackfixed xd


Check this `sol_vector_get()`


Despite being net more salt, I like this.

@zmerlynn @markturansky


This route does not seem to be good. Show by definition should require an `:id` rather than having it optional. I recommend using `resources :categories, only: [;show]`, looks like a less code as well.


I would use `= image_tag @video.large_cover_url` - it reads better.


This case should be throwing a BadRequestException or an UnprocessableEntityException, certainly not a Fatal Error. It is a client error, and anything in the 5xx range indicates a problem with the API, not the request.