Hmm possibly. It was in the div instead of the image element. Seems to be
working correctly now.

Thanks!

On Tue, Mar 3, 2015 at 2:40 PM, Pavel <notifications@github.com> wrote:

> was divclass
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MAPC/myschoolcommute/commit/c8db28da742e087699e5c0336ee691dd0961f797#commitcomment-10014579>
> .
>

--
Matt Gardner
M.S. Urban and Regional Policy Candidate
School of Public Policy and Urban Affairs
Northeastern University


Should LN be referenced here?


you may also want to look into fissle/fissile


Very well articulated!


If you start new arena season in config. All players get earned all PVP mounts, and pvp titles.


Bedankt, dit ziet er interessant uit! Ook goede aanvullingen voor de README. Wie Jabref gebruikt om de Bibtex databank te beheren moet nog een aantal instellingen aanpassen, dit zal ik nog best wat uitzoeken en in de README opnemen voordat ik deze verbeteringen integreer in het sjabloon...


ack


Probably should have bumped that


I actually have no idea how that space was removed. I can't find it in the
commit history.

On Tue, Mar 3, 2015 at 3:33 PM, Matt Gardner <gardner.w@husky.neu.edu>
wrote:

> Hmm possibly. It was in the div instead of the image element. Seems to be
> working correctly now.
>
> Thanks!
>
> On Tue, Mar 3, 2015 at 2:40 PM, Pavel <notifications@github.com> wrote:
>
>> was divclass
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/MAPC/myschoolcommute/commit/c8db28da742e087699e5c0336ee691dd0961f797#commitcomment-10014579>
>> .
>>
>
>
>
> --
> Matt Gardner
> M.S. Urban and Regional Policy Candidate
> School of Public Policy and Urban Affairs
> Northeastern University
>

--
Matt Gardner
M.S. Urban and Regional Policy Candidate
School of Public Policy and Urban Affairs
Northeastern University


TeamCity Akka.NET :: Akka.NET PR Build [Build 94](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=146&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll, D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Tests\bin\Release\Akka.Tests.dll ... Build time: 00:09:11
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliverySpec.GuaranteedDelivery_must_redeliver_lost_messages_after_restart: <no details avaliable>

Akka.Tests.dll: Akka.Tests.Routing.TailChoppingSpec.Tail_chopping_router_must_throw_exception_if_no_result_will_arrive_within_the_given_time: <no details avaliable>

Akka.Tests.dll: Akka.Tests.Actor.InboxSpec.Inbox_have_a_default_and_custom_timeouts: <no details avaliable>

```


Nope, looks good to me!


oh removed, its in this commit:
+    <divclass="span2 footer-span">


I see now. Good catch. Thanks, @demiurg.


This change breaks the intel 13 compiler:
http://hermione.cct.lsu.edu/builders/hpx_intel_13_boost_1_54_debian_x86_64_debug/builds/289/steps/build_unit_tests_errors/logs/stdio


May I modify the ms sql server migration scripts? I'd like to split them into the table creations and data inserts, as well as make them idempotent (by checking for table existing before creation, and deleting from tables before inserting (in he event the insert script is rerun it won't insert data twice).


The value is brought in via the env variables, so it must be written in
during the setup phase. @migurski, help?


Ah yes. Thank you!


maybe we should change the naming of this, is_disabled, or is_final_state or...


Important update. Will help you avoid errors in Javascript file you develop.


No, Windows batch files do not require blank line at the EOF


Sure - I can do on Oracle too, just hesitant to do stuff like that when I can't test it.


> Noep, lukes gud 2 meh

I am sure it does :)


I don't know how, but I think this silently reverts my commits 67c219cdecc18e628f20cbe1e43c09308c2e720a and 67c219cdecc18e628f20cbe1e43c09308c2e720a. Was this force-committed or force-merged? I don't see anything in the diff here that would indicate a reversion, but if you look at [interface.md at this commit](https://raw.githubusercontent.com/curateteaching/digitalpedagogy/e8a9b46588d453cf56b59e146e67fddc90a4cc8a/keywords/interface.md) and compare it to [interface.md at the previous commit](https://raw.githubusercontent.com/curateteaching/digitalpedagogy/e44d09fee3db6b31bbd193901bcfd85f19544977/keywords/interface.md), you'll see that the leading spaces and quotation mark fixes have been reverted. I'll see if I can just revert this commit on our fork.


@rizar This commit broke the master branch on Travis; I think you forgot to rename a bunch of `feedbacker` keyword arguments in tests.


ack, see #351.


This is in #9


Don't be afraid to fix the root problem: if we only ever expect a double value, why not just change the type signature on the command? Then the compiler can worry about the type and you won't need all the if clauses below :-)


na das ist ja der kommentar des jahrhunderts! cpt kreativ am werk ! :D


SHOTS FIRED


This is also in #9.


siehste jetzt können wir das hier diskutieren bevor ich das in den master merge, so ist das eigentlich gedacht


ack


You should use Prototype's [`addClassName`](http://api.prototypejs.org/dom/Element/addClassName/) and [`removeClassName`](http://api.prototypejs.org/dom/Element/removeClassName/) methods, since they're safer than string manipulation.


Agreed with all those.

And we are going to need one for the Section Learning Objectives block also (but do NOT add ost-reading-discard).

I will make a branch and add that in.


Mmhh sorry, maybe I let my comment on the wrong commit. Seems to be related to 07c3ee5dd1ddc1b091fc0700249d5d5def6ca7cd or 9e3566d1dcd0e9248f0f77f146224c1514c02d72.


/cast minimize cross-section self


Anthony, you may also want to look into shutting your god damn mouth, whore.


Missing a dot :)


Premier commit pour creer le projet sous github.


And "its" -> "it's"


Good catch. In an email from Kristina Grammatico on 2/11/2015 9:21 AM she ask to call it

     HPCC Systems® from LexisNexis®


So how does it differ from `CVec` then? There's `as_mut_slice` still. Why not just throw away `Unique` if you don't agree with a need for it. I believe that only an immutable slice kind can be constructed from a shared pointer. There shouldn't be anything to free (or at least where's the use case?) and you can't mutate the data. The users are free to lie about uniqueness if they wish like I did in `rgtk` so `Unique` is not a real barrier when using `CVec`, it just makes one think about the implications of sharing the pointer and make an explicit promise.


I thought you would like that since those are your own words


And what if the value is an Integer? We silently ignore the value and condemn the API user / me / your future self to hours of frustration trying to figure out why the value never changes but there's no error? :sob:

If you can't figure out a way to get the compiler to prevent the problem at compile time, never hesitate to throw an exception if your assumptions turn out to be wrong at runtime: your future self will thank you!


thats a comment, dude, no official documentation... will fix ;)


Changed it from boilerplate to sample. “Boilerplate” implies everything it contains will be required for a project, when it should be more of a kitchen sink-type demonstration. Also, starting projects from another git repo is kind of messy because it requires deleting and re-initing git. We can provide an easier starting point for blank-slate projects.


75127368_DCFSEC (Nigusse) - x86_64-MacOS-10.9-clang: Tests Failed

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


One AU is defined as the mean Earth-Sun distance.  Your comment "If date is not set, mean value
for 1AU used" is slightly confusing to me.  Why not change it to "if date is not set, the Sun-Earth distance is set to 1 AU".


fix #782


No problem, I'll take care of the sql side testing.  And also, I don't think i'll split it into different files, just 1 migration script for the feature works for me. I'll push up a change in a few minutes.


We should require `v0.12.x` minimum since that’s the latest. This’ll require a bit of testing and upgrading across the board, unfortunately.


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1129 of 1130 tests passed)

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


You win this round, bond... But only because of the <3


A newline too much^^


The real benefit is that somebody else could do:

```sh
pip install Harold
harold
```

As opposed to:

```sh
git clone https://github.com/speakerbug/Harold
cd Harold
pip install -r requirements.txt
python Harold/main.py
```

Which makes distribution and running easier


Your comment: "Default=None implies mean Sun-Earth distance used" - do you mean that 1 AU is used?  In that case why not change to "Default=None implies that the Sun-Earth distance is set to 1 AU"


Finalizing the URL needs JP/Ed/Dante. I will work on that and include in next branch.


TODO: mention a useful change, and link to the [online docs](http://developer.marquee.by/marquee-static-sdk/) for more detailed examples.


todos meus exercicios estão dando certo como mando para vc analisar


Checked out the code, and it passes on my MacBook Air. +1 Added some comments on the docstrings.


ack


That's great to hear, I'll be sure to check it out!


I don't think this will work. If `db_ipv4.time_zone_by_addr` cannot resolve the IP (because it's an IPv6 ip, or otherwise formatted), `socket.error` bubbles up to this level. You need to handle that exception instead of checking for `None`.


shouldn't that be a "." (dot) inside the fallback?
```python
bear_dirs = path_list(section.get("bear_dirs", "."))
```


No, I trust your research, this is good.


Hum... I see your point. I like having a difference between "fake" vector and "followed" vector but I think the CSlice destructor is too much like you said. I was wondering if it was a good idea but finally I don't think so.


Despite being its own package, I'd still add a prefix to this variable.


It might be a good idea to adjust the prefix for all of these variables. Mainly because there may be another role that involves Oracle (hoping not) and we wouldn't want them to collide.


@openstaxalina @kjdav - Can one of you ask Helene what we decided? I do think that we decided on "note", but I can't remember.


I'd probably use `lineinfile` to manage this within `/etc/environment` vs. a script in `/etc/profile.d`.


Should be `~> 1.4.2`, since `<= 2.1.2` is still vulnerable.


A Test-ben? mert a metódusban az elején van.

2015. március 3. 21:00 thomastrinn írta, <notifications@github.com>:

> Ennek azért illene az elején lennie.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ViktorSarkozi/StringCalculator/commit/d56c26505ae264e37e6c54e83838128e2679fdcb#commitcomment-10014931>
> .
>


ah, @mistic100  I see your point.

I have a question though:  If the dependencies are bundled with the minified version, then what if a different library requires the same dependency but in a different version? Wouldn't this be especially bad in the case of the jQuery namespace?

See, what I'm doing for development is I have a "debug-main.js" where all the code is source so that I can step debug into plugins when something goes wrong. I will make a "production-main" once all my functionality is built which will use the minimized versions. Am I doing it wrong?

BTW, thank you for this plugin.  :-) It's super useful for representing a particular kind of nested query I need to do through a graph database.

P.S. It's unclear to me from the documentation what the validation callback should return. I assumed it should return false, which causes an error elsewhere that src.replace() will throw a null pointer...  But now when I think about it, maybe my validation callback should be returning a string with a "{0}" in it?


I don't see rejected as a final state. I'm thinking from rejected it can go back in open or closed, and also from accepted it can go to closed. Need to figure out what to do with INACTIVE.


shouldParseGenericOBO(org.coode.owlapi.obo12.LoadCellTestCase)  Time elapsed: 3.633 sec  <<< ERROR!
org.semanticweb.owlapi.model.OWLRuntimeException: org.xml.sax.SAXNotRecognizedException: Property 'entityExpansionLimit' is not recognized.
	at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.getProperty(AbstractSAXParser.java:2063)
	at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.setProperty(SAXParserImpl.java:580)
	at com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl.setProperty(SAXParserImpl.java:309)
	at org.semanticweb.owlapi.util.SAXParsers.initParserWithOWLAPIStandards(SAXParsers.java:60)


sin comments :D


@bors: retry


Dehát osztható 5-el, 7-el és van benne 3-as is, mindegyik teljesül :)


My recollection is example for these. I think the rules was if it has "example" in the name it gets to be example.


@dgcliff I tried a 'my_communities' route but couldn't get that to work so for now its just employees/:id/communities


Sorry, running `nose2 -s tests/bricks` is a dangerous habit.


sdlfij


And here the fallback shouldn't be `false`?
```python
self.conf_writer = ConfWriter(str(default_section.get("save",
                                                      "false")))
```


no, bears dirs are either specified or it lies in the default bear dir


Maybe worth keeping a mention in the comments about the name of non-WFirst-specific function.  That's more optional for the other two, since I don't know of any other detectors that have recip failure or IPC.  But non-linearity is a common feature of pretty much all devices, so the generic version will be worth mentioning in some demo if not necessarily here.


Thanks, I'll delete this branch.


okay just matched what you deleted in the config^^


@goodengineer fijate de poner esto en el .gitignore

# Intellij project files
*.iml
*.ipr
*.iws
.idea/
/libraries/*

# Log Files
*.log
.gradle
/local.properties
/.idea/workspace.xml
.DS_Store
.gradle
/local.properties
/.idea/workspace.xml
.DS_Store
# built application files
*.apk
*.ap_

# files for the dex VM
*.dex

# Java class files
*.class

# generated files
bin/
gen/

*.bin


@openstaxalina Can you ask Ed to confirm/deny this?


Good catch ! (essai de commentaire dans le code depuis github)


it is probably better to use 'and' instead of nested if statements. Its easier to read and faster as well


It seems I accidentally wrote the message in Finnish. :D


ack


__API Blueprint syntax error on line 2288__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


no, we're falling back here because save is a non-bool string. Maybe the default value doesn't make sense at all because it's never used.


Your number pattern looks good to me, actually!


+1 Looks good


@dgcliff I looked for a helper method for the link creation and found this but .path was not a method I could apply. Can you point me in the right direction?
<code>
%article.drs-item{data: { href: item.path,  "drsitem" => item.pid  }}
</code>


saw approval from Ms2ger
at https://github.com/Adenilson/servo/commit/a2fea4c093c06bc2ecb4c0cc75b3a96bee344eaa


merging Adenilson/servo/cleanupWarning01 = a2fea4c0 into auto


Adenilson/servo/cleanupWarning01 = a2fea4c0 merged ok, testing candidate = 621150db


yeah wondered already, but then it's fine


las variables estáticas por convención les ponemos "s" al comienzo. En tu caso seria sContext


What is the clash here?


Finalize is dangerous and is very unrecommended in most code styles. And this also caused the build error. Why to use it?


Sorry if I'm being too hard on you, it's just this is supposed to be a very critical piece of FFI that needs to be thought through very strictly so we don't let down an unknown number of people who use this. ;)


Este String ponelo como variable Final y Static :dancer:


ok, then take the new commits ;)


Ack


@mhaskel what's the reasoning behind this? Did Puppet Labs stop using `tests/` ?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1126 of 1126 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)


Meant to say fixes #415, addresses #406


En general los comments los sacamos, solo dejamos los comments que son necesarios para aclarar algo o cosas que no queden muy clara en el codigo.
![delete](https://cloud.githubusercontent.com/assets/494001/6472025/27f880c0-c1ce-11e4-9a1b-5e11ba50854a.gif)


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


ahaha, it seems world missed great author :) . agreed, I will revisit it.


No problem, it's important to have a well thought lib and having a second opinion is important. I'll make a PR with my changes and you'll tell me if it fits better to your vision.


r+


@kjdav But what if there is only one Practice Problem? I think that CAN happen. Either way, this will be up to the XML team or authors to do correctly, so I don't think we need to worry about it.


This is highly suspicious


Lo que hacemos con todos los items de la UI los creamos como variables privadas y creamos un metodo que sea setUI donde se realiza el binding de todas las variables con su respectivo compenente.

Despues generalmente tenemos otro metodo setListeners donde a cada item de la UI se le asignan los listeners.


@kjdav We need both. virtual-physics is what this book calls this feature. But simulations in general are an important book feature and so ost-interactive describes that part.


that why there are two minified versions, normal and standalone... choose the one that suit your architecture

the validation callback must return true (success) or an array, the first element of the array is the identifier of the lang.errors code, optional other elements are used to replace {0}, {1}, etc in the error string


Este comment solo lo podes dejar si es una feature que no anda en la API por un problema de ellos, y cuando lo solucionen e solo descomentar. Me explico ?


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


Yes, note.


89224666-FixNANProblems-Issue4557 (rraustad) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


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


@vmdanilov Could you please explain this change? I thought the conditional was necessary for backwards compatibility with 10.9?


you should simply put the instance.tradeIn or tradeFor values here, all the logic should be done in to_internal here you should rapidly get stuff.


Per Ed, this should be a note.  This will save us from having to suppress numbering in the PDF or web view, and keeps with prior practice of making book features like this into notes.


should do this in to_internal


Umm, isn't this assert testing the boolean value of a MonoEQConstraint?


@kjdav What is this second comment about?


Now I've pushed a commit with pattern literals and desugaring of patternmatching functions to lambda+case expression. The mergeCase function may need a rewrite but it works.

Right now the pipeline is broken but I will look at that tomorrow.


c'est OK sur testing :)
prévu quand sur neptune2?


should this be `!==`?


Why include gabi++?


Yes, I think so, just make sure that it's backwards compatible. For other patterns though, I think we can just leave it up to administrator to make it case sensitive or insensitive as they desire


Yup. :D


Added more detail above -- just that this needed to be changed from sample-problems to worked-examples


@mobileoverlord you need this too. We need to translate reference serial types to the underlying serial representation. It is "integer" in Postgres and I think in TDS too.


It is, that's where I was referencing it from in the original fix. That's set up in the Heroku app as a config var called GA_WEBSITE_URL. In our case that's set to https://bouldercolorado.gov. The url is coming from the realtime Google API call, which seems to match what gets displayed in GA as well (they only show the truncated domain). I'm checking in Google Analytics to see if there's a setting that controls whether that url is truncated or fully qualified.


not reviewing this :)) boring tests are boring


This seems like it should break the specs [here](https://github.com/causes/overcommit/blob/483a3062a645f0d2368542211b5098f457518de2/spec/overcommit/configuration_spec.rb#L165) and [here](https://github.com/causes/overcommit/blob/483a3062a645f0d2368542211b5098f457518de2/spec/overcommit/configuration_spec.rb#L173), but it doesn't. Why?


Great, will work with this in the morning. Really appreciate the before/after examples!


mardi,


El nombre del commit es genial :-D Parece que esto tiene buena pinta, ¡buen trabajo!


This information, besides that is is non-sense, should be added by the FederationManager, not by each adapter.


thanks


@trommel I built an application with a similar idea as yours in terms of the "Schedule Manager/Creator" in Swift. My courses were hard coded, but I'm sure you could figure out how to set `UITextField`'s to the cells. https://github.com/istx25/schedule


@dnehls : maybe the adapter manager could add default (from the config) gps coordinate if an adapter does not specify coordinates?


Hi Garrett,
Just curious as to why I earned a 10/40 on the week 5 homework assignment.  Was there something I did incorrectly or missed on the assignment? Thanks!
Chad


Ping @pstenstrm


@adam ?


Agreed, I will fix prior to merging.


cool ... merci


@trommel I already have implemented the block rotation, but it's quite a simple application. If you'd like to use it, go for it.


why don't you use directly the CommentSerializer ?


It seems that many of the schematic symbols will not load as beaglebone-cape-cache.lib was not committed to the repo. Would you consider adding it?


ooops


Looping over indexLabels would be cleaner. This was my quick and dirty solution to fix the problem.


Change all sets and maps to unordered!


everything looks good :shipit:


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


I'd rather suggest a different way of hiding svg that works everywhere.


Can't see evrything ;)


Short tag <defs/> should also work


Doc https://docs.chef.io/config_rb_knife.html says:

> The sub-directory for cookbooks on the chef-client. This value can be a string or an array of file system locations, processed in the specified order. The *last* cookbook is considered to override local modifications.

When a user creates a cookbook, I wonder why `knife solo cookbook create xyz` creates `xyz` in `cookbooks`?


`i2cRead` is the continuous read, `i2cReadOnce` is the single read—no `mode` here


never said that. We're adding . to the path list that doesn't remove others.


This is a much needed commit. I even think we can get it committed separately.


Nope. This is basically equivalent to saying `!empty($query) || $query === '0'`. I switched all `!empty()` calls to `!= null` so that the string `'0'` works correctly.


missed while you did a rebase?


I think the `semantic_linkbacks_commentdata` filter should be enough to filter the commentdata... What do you think?


apparently, thx :)


If you want to consolidate enqueue calls, I would do that in a separate, private abstraction. This is too risky if i2cRead ever needs to modify the arguments


just noticed something; shouldn't this have the armor check like human/species does too?


I think we have only one `<defs/>` in resultSvg so `.first()` is not needed.


Then why need system imports stand before these three lines?


"Nugget" lulz.


Type definitions for polyglot.js https://github.com/airbnb/polyglot.js as requested in #2409


I don't think that is the creating user. The creating user is who has the `creating_user` role associated with it.


This patch breaks logging to find differences between repository branches. I found that getting a "long" format log worked correctly, but the "short" format log was wrong. I'm not certain, but the short format looked somewhat like the log of the *common* patches, not the difference. Reverting the patch solves the issue in my case. I am using magit version 20150124.930, and git version 2.3.0.


Yeah i added how in the test-inline-svg.html


yes, maybe, but then the federation manager needs to subscribe to new adapter instance events. Otherwise it wouldn't always know which adapter instances exist.


oh, ignore twitter profile images


Can svg have multiple defs inside?


*Thats* codestyle only ;)


yes sure!


Эти переменные лучше вынести в конфиг файл


Yes, indeed, it piles up, I didn't check carefully.

Easy workaround would be to either delete these lines from the skeleton, or not to re-add them when we copy from the temp file to $NFS_SYSCONFIG.


should we use `HasRecording()` instead and get the recording within the if.


I'll change that...
pushing a fixup commit again...


@rbf : `value="true"` has disappeared so now the browser send `on` instead of `true`.


Nice! ;)


What if there are the same ids inside different files?


@dnehls Isn't it nice that we've a message bus for this? ;)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/917
success: http://build.servo.org/builders/linux2/builds/915
success: http://build.servo.org/builders/android/builds/953
success: http://build.servo.org/builders/mac1/builds/1132
success: http://build.servo.org/builders/mac2/builds/1132
success: http://build.servo.org/builders/gonk/builds/103


:clap:


QwtUpdate (axelstudios) - 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-87.48%25-red.svg)


I am curious about the naming: avi? avy? avi- prefixed stuff in avy-jump.el?


NE koristit post/get i sl sirove varijable.
Citaj ovo:
http://docs.phalconphp.com/en/latest/reference/request.html


There's two times the 'current' var definition..


Sve funkcije, varijable i sto ne pisat na engleskom jeziku.


I'd do `pass` before the comments so make it valid python code.


Yeah, that'll be good I think.


No, the channel could change in theory in the meantime between those two calls, and the only thing it does in `CPVRChannel` is checking if the recording is set, so the same check as this one


You are still building this string before the precondition. The precondition check is expected to always be successful. So this will be constructed but never hopefully never used. So it's a waste of CPU cycles. Can you just use a simple string as the message?


Engleski jezik i tu. U bazi imena polja i sve na engleskom molim.


`knife cookbook create` you mean, right? https://github.com/chef/chef/blob/master/lib/chef/knife/cookbook_create.rb#L70 shows it using `.first`. Seems like a bit of ambiguity/disagreement in the docs.


Have you considered setting up a PR against Lela for this? We've run into issue with comparisons to image blurred views which required a threshold be applied.


ok


why do you do `is` instead of is? (And why not above?) I'd rather do is than `is` in all those cases. (Looks weird otherwise...)


Because I am not an experienced programmer and the word sounded right. Thanks!


Vidjeti kako koristiti phalcon forme a ne pisat rucno forme.
http://docs.phalconphp.com/en/latest/reference/forms.html


fast-forwarding master to auto = 621150db


This one looks good to me.


what does @include size(10px) do? Add 10px height and width I assume?  Is that a SCSS mixin or default feature?


It is generally better to make sure that preconditions cost almost nothing. So use a simple string constant as the error message.


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1124 of 1124 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.59%25-green.svg)


Just reading up on how to accomplish this and came across this: https://github.com/jfinkels/flask-restless/issues/124

Github itself doesn't even allow a trailing slash and issues a 404.

I'm fine with using my custom logic for the proxies, just thought it would be an easier out of the box experience if it didn't add them.  Thanks for the response.


I think this might need to include `{shallow:true}` ?


Since `intersect` returns a node ID or `null`, should this be `return nodeId !== null` (since `null != false` and `null !== false` are both true)?


I'd do `*args, **kwargs` instead of `[params...]` because thats how variable argument lists are realized in python. If you don't know about this, try google and learn because it's essential :)


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - x86_64-MacOS-10.9-clang: Tests Failed

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


Ah, yea, this is a good catch. I've just committed a better fix to MimeKit. Thanks!


Don't see why BatchNode needs to duplicate updateGeometricState() code.
Why not just do:
```
@Override
public void updateGeometricState() {
    super.updateGeometricState();
    if (!children.isEmpty()) {
        for (Batch batch : batches.getArray()) {
            if (batch.needMeshUpdate) {
                batch.geometry.updateModelBound();
                batch.geometry.updateWorldBound();
                batch.needMeshUpdate = false;
            }
        }
    }
}
```


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (625 of 625 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-3.94%25-green.svg)


Have you considered creating a PR against Lela for this change? We've seen issues with comparing images with Blur effects which are eased by adjusting the pixel threshold.


Dude, from all of us to you - thanks!!!

On Tue, Mar 3, 2015 at 1:23 PM, Jeffrey Stedfast <notifications@github.com>
wrote:

> Ah, yea, this is a good catch. I've just committed a better fix to
> MimeKit. Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nachocove/MimeKit/commit/bac5b83fa6aae1c269697a15c57c7350a3f69b9e#commitcomment-10016855>
> .
>


your id renamer already renamed them!


saw approval from metajack
at https://github.com/iamdanfox/servo/commit/522b0ea3b97a67af3eef782274c14f58c6b4399a


merging iamdanfox/servo/replace-uint-with-usize = 522b0ea3 into auto


n/m


iamdanfox/servo/replace-uint-with-usize = 522b0ea3 merged ok, testing candidate = 0ff54a97


I agree. I actually resolved my comment before you wrote this, so not sure why it still displayed for you.


@ezpuzz you are correct, it would be nice to have it automatically.


btw, now that I look at this code again, same thing can happen here, got to fix this one too


dont block move its used by "system" like PW matchmaking


Ah good catch. I didn't even think about the combination of versions. Will update. Thanks!


No more changes after this? :computer:


Still confused about how this works -- can we add a couple very simple comparisons of usage with and without `replace=True`?


Why is this a TimeoutException? Should this be an InvalidOperation exception, i.e. trying to send something to a non-existent queue?


@jdaudier That's not default - so I missed it. It's in MojoTech helpers in vendors code:
https://github.com/marionettejs/marionettejs.com/blob/master/src/stylesheets/vendor/_mo.scss


if this is intended as a short overview, its good. If it's intended as a template it's not because of all the code I'd have to delete every time I'd want to write a test. My template is:

```python
import sys
sys.path.insert(0, ".")
import unittest

class Test(unittest.Test):
    pass

if __name__ == '__main__':
    unittest.main(verbosity=2)
```


On Tue, Mar 3, 2015 at 4:16 PM, Pieter Hintjens <notifications@github.com>
wrote:

> Pull requests are always worth it.
>
> I don't think there's an _unsigned anywhere (I can't see it).
>
I meant $(class.name)_$(name)_number, of course. Sorry for the confusion. I
was just trying to explaining my original motivation - i don't want to make
the impression that i am arguing nor negotiating here :)

I perhaps jumped to conclusion too quickly - i looked at _number method,
which returns uint64_t (i think) and deducted that it would be a good idea
(i saw it as a oportunity for excercise) to make my first PR ever and play
with gsl a bit :P What i did not realize is what you wrote. Plus while i
was playing football today, it clicked with me. Not only zhash_t has void
pointers/zhash_freefn for individual key:items, which you even pointed out,
but we could very possibly do a variant struct/class like
libvariant::Variant has, which is a much better solution than trying to
reimplement subset of json functionality (and very poorly, while we are at
it :) ). But even Variant is probably an overkill here, i am getting slowly
to the conclusion that: Since both parties must know the app "domain"
(otherwise the data being transferred would be meaningless to them) in
other words what keys to fill in/expect and what values should be there, if
the producer simply stores (void *) <whatever int type, or string here> and
set the freefn accordingly, the consumer of that data should be perfectly
able to retrieve those data even without any metadata attached to
individual hash items.

> The string/number type system is old and works for simple config cases. it
> does not work for encoding data, nor APIs, so we've developed a more robust
> one in zsock_send/recv and zproto.
>
> The V2 codec does not provide any API for working with hash table values.
> It's redundant; we have that in zhash already, or can add it on top. The
> sheer size of the V1 API was a serious problem for something meant to be
> "simple and fast".
>
> Now to your question. Sending a hash is a decent idea. I'd call it
> "headers" maybe, to match the HTTP sense. It's extensible and
> self-describing. This is what it was made for. However I'd write my own
> methods to work with the hash, and leave that out of zproto.
>
> As for parsing complex data and getting complex results, IMO it's better
> to return 0/-1 and then provide extended error information via another
> class method, e.g. _error (). The trouble with variable error return codes
> is that it makes applications very fragile. So we simply don't do that.
>
I would like to thank you that you took time and patience to explain these
issues to me.

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/karolhrdina/zproto/commit/a2569707b2b8f4df17a2927564a8add05ca3bc6d#commitcomment-10008712>
> .
>


89224666-FixNANProblems-Issue4557 (rraustad) - 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)


I put it in, because where I'm checking it, I don't think we want to include the relations...  but maybe, or it doesn't matter? I'm fried now, so let me know what you think.


`avy` + `interactive` = `avi`.

I didn't like the name `avy-jump-move-line`, so I renamed it to `avi-move-line` and then everything
else from there.


by the way, you don't mention the last two lines anywhere. It's just the way we execute our tests.


great explanation


no prob!


Зачем ты тут делаешь angular.extend? Если у тебя все на одной странице и один контроллер байдинг и так будет работать!


FYI this is really hard to read on white backgrounds. Can you revert this and try normal? I don't think its good to set "white" or "black." Or maybe this should just be user defined?
![screenshot 2015-03-03 13 26 52](https://cloud.githubusercontent.com/assets/2273987/6472688/1467534e-c1a9-11e4-9a2d-c2638c054ac9.png)

@DepthDeluxe


bless you @bryanfriedman


:dancer: i made a lib a long time ago to help with this kind of stuff it is called mo

https://github.com/mojotech/mo#documentation-index

it is awesome


I do not rename ids of the elements inside svg files.


Using the real time API Explorer on the bottom of this page I was able to confirm that the rt:pagePath dimension is returning just the page path without the full domain prepended.

https://developers.google.com/analytics/devguides/reporting/realtime/v3/reference/data/realtime/get#examples


First wanted to use CommunicationException but this requires to add
reference to servicemodel which is an overkill. The closest I could came up
with is TimeoutException. Switching to IOE then


Hahaha. Been doing this as I run across ugly KB articles. There are more to be sure... ;-)


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2195 of 2212 tests passed)

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


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


Ignore my earlier ramblings, community_path(yoursolrdocumenthere) should work fine.


Also did you mean to add the auto vim back in? I don't think you want this.


sorry i mixed things with svgmin.

same ids in different files was already a bug, i guess


I wasn't able to fix this with `git revert e8a9b46`, but I was able to fix this by interactively rebasing the branch with `git rebase -i`, and deleting this commit. I'll try to submit a pull request with this rebase.


I only see Sweeps. No Substitutions section. huh!?!?


I hope you didn't have to count the number of columns "by hand" :P


Dude that's awesome! I can basically close up shop on the Isotope themes now :wink: Good stuff.


already fixed in 02eec1cecd1fa74e0d8d2fc5d31ed16964626015


yeah it definitely was a bug


Вместо пустого объекта лучше создать пустой объект пользователя со всеми полями { name: '', email:'' ... }


Is this a "fix" for https://github.com/pfefferle/wordpress-semantic-linkbacks/issues/19


this is a comment


no me gusta


Speaking about all of this reminds me of something I forgot to tell you... look at EspeakPrinterTest.py.

If you define a `skip_test()` method like this:

```python
def skip_test():
    try:
        subprocess.Popen(['espeak'])
        return False
    except OSError:
        return "eSpeak is not installed."
```

This is intended to use if e.g. dependencies are missing.


Вот в этом ифе начинает вызываться транкейшн... прямо в конструкторе GeometryPair... я завтра за питон сяду, постарайся поднять это место, если найдёшь время...


Here's how we did it for another piece of content:

<content>
<figure id="id1165015354178"><media id="yt-media" display="block" alt="This video lecture is designed as a supplement to Chapter 1 of Collaborative Statistics (col10522) by Barbara Illowsky and Susan Dean. In this chapter, students can expect to learn how to recognize and differentiate between key terms in statistics, apply various types of sampling methods to data collections, and create and interpret frequency tables."><iframe width="425" height="344" src="http://www.youtube.com/embed/viXNnZO8X9k"/></media></figure></content>


Let's avoid the word "lame" here. Just "emits a warning" is descriptive enough.

Thanks for not filling up the console with logspam by the way :)


Since it is checked with the `is` operator and not with `==`, I thought that would make it clearer. But if it confuses you my intention failed :O


ehh forgot...


Does this seriously just fix an issue that goes back to the 2-series? Damn.


oh and `skip_test()` is a nice feature, good that you mention it (didn't know that)


there's another skipping mechanism: https://docs.python.org/3.4/library/unittest.html#skipping-tests-and-expected-failures

This is intended for other python versions.

The difference is:
 * skipping by unittest library will skip a tiny test function and will not appear as skipped when executing run_tests.py, thus will not fail travis!
 * skipping by the `skip_test` function is done by our run_tests script and will appear as skipped. If the test gets skipped on travis, travis will fail because thats meant for dependency based skipping. If dependencies are missing on travis that is considered an error in travis.yml


Two comments:
1) the test should not segfault even if the open returns false. Was this fixed?
2) what is the test that was failing?


Should add; this.packages = ""; Packages can potential be null.


Эта сточка повторяется ее можно было бы вынести вне таба, а в функции nextStep добавит условие что если таб последний выполнить регистрацию. И да, как я уже упомянул пользователя тут можно не передавать


Thank you!


This code is awful.


EL string metelo en static final


To šlo do nav.less.


I learned something new.
@mistic100 Thanks


porque? https://github.com/golang/go/wiki/CodeReviewComments#initialisms


Nice feature from `unittest` to use attributes, but a test is intended to be executed. I won't mention the attributing method, I think only the `skip_test()` function is worth mentioning^^


La activity no tiene Menu
![duro](https://cloud.githubusercontent.com/assets/494001/6472918/7b9e262a-c1d4-11e4-855a-0932d5bc64f7.gif)


This is probably a pretty big change. You need to make the changes everywhere I noted, plus maybe a few others. Also, you need to rename a couple of files. You might actually be better off redoing the devise stuff in a different branch and discarding this. Sorry - that sucks I know. But it might be easier than trying to hunt down all of the places that it is wrong.

C

> On Mar 3, 2015, at 3:01 PM, neema122 <notifications@github.com> wrote:
>
> I made the changes but I get the same error on the page.
>
> On Tue, Mar 3, 2015 at 9:47 AM, Christopher Beck <notifications@github.com>
> wrote:
>
> > Should be users
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/neema122/blocitoff/commit/98207ca95b6ffe6fabfe46d377f70ef848fdbe3b#commitcomment-10009698>
> > .
> >
> —
> Reply to this email directly or view it on GitHub <https://github.com/neema122/blocitoff/commit/98207ca95b6ffe6fabfe46d377f70ef848fdbe3b#commitcomment-10014950>.
>


I have fixed the typo and pushed it separately.


delete!


I think both should be mentioned because you might want to have tests specific for python version (-> need unittest). So it's important to highlight the differences. May be a note so people can just not read it if they don't care.


Naughty of me, I should have done a local build. It fails because the property name is wrong.


Вот, вижу изменил на правильный вариант


Esta activity no tiene Menu :dancer:


Fixed the mining error message as well.


If we expect to eventually support format 1, maybe we should choose another name.


some things


Mas expresividad en als variables!


Removed this part.
- This should be changed server side -


oh, `skip_test` is expected to either return False or a reason as string. (Never true.)


Fixes #8


Looks like MySQL will do a table scan, but even if we check only one field or check no fields, it looks like MySQL prefers table scan (but it appears MySQL doesn't actually scan the whole table, just row by row until it reaches the limit).

```
mysql> EXPLAIN SELECT * FROM tasks WHERE type IS NULL OR cluster_template_name IS NULL OR user_id IS NULL OR tenant_id IS NULL LIMIT 10;
+----+-------------+-------+------+---------------+------+---------+------+--------+-------------+
| id | select_type | table | type | possible_keys | key  | key_len | ref  | rows   | Extra       |
+----+-------------+-------+------+---------------+------+---------+------+--------+-------------+
|  1 | SIMPLE      | tasks | ALL  | NULL          | NULL | NULL    | NULL | 125685 | Using where |
+----+-------------+-------+------+---------------+------+---------+------+--------+-------------+
1 row in set (0.00 sec)

mysql> EXPLAIN SELECT * FROM tasks LIMIT 1;
+----+-------------+-------+------+---------------+------+---------+------+--------+-------+
| id | select_type | table | type | possible_keys | key  | key_len | ref  | rows   | Extra |
+----+-------------+-------+------+---------------+------+---------+------+--------+-------+
|  1 | SIMPLE      | tasks | ALL  | NULL          | NULL | NULL    | NULL | 125685 | NULL  |
+----+-------------+-------+------+---------------+------+---------+------+--------+-------+
1 row in set (0.00 sec)

mysql> EXPLAIN SELECT * FROM tasks WHERE type IS NULL LIMIT 10;
+----+-------------+-------+------+---------------+------+---------+------+--------+-------------+
| id | select_type | table | type | possible_keys | key  | key_len | ref  | rows   | Extra       |
+----+-------------+-------+------+---------------+------+---------+------+--------+-------------+
|  1 | SIMPLE      | tasks | ALL  | NULL          | NULL | NULL    | NULL | 125685 | Using where |
+----+-------------+-------+------+---------------+------+---------+------+--------+-------------+
1 row in set (0.00 sec)
```


* `std::bad_alloc` if memory allocation fails
* whatever the "default initialization" of T might throw


Jsem přidal čárky, protože se mi chtělo.


This was committed by mistake. I will remove this file.


Can this be `srcPattern = ['src/**/*.js', 'test/**/*.js']` ?


Jsem smazal řádek, protože v něm nic nebylo.


You're reading my mind. Thank you!


Added a line to the story


tbh I don't remember. At some point it was different but it seems now it's not anymore.


meshServer is no longer a constructer. It is a factory which returns an LB app.


if no game exists


Tohle celý šlo do piče.


Missing trailing comma


By convention, user-defined classes should be Uppercased. That simplifies reading since it's obvious when an object is being constructed.


Delete de las cases que no se usan


T's copy constructor is not called here. All in all, this throws in the same conditions as the default constructor.


Delete de las clases que no se usan


This is a super annoying case for the dataset as the protocols people use for generating a validation set from the "full_train" set are complicated. You can't reproduce the kind of validation set in the literature using start and stop, because they involve sampling non-uniformly from both the 'train' and 'extra' sets.


Probably shouldn't change this line in the tests since it isn't related to your changes


Please change the webhook_uri too, since in some cases, it has username/password info.


Yah, I'll add a comment


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


function designed to generate random fillename for tmp. file containing new zonefile.


Sure, I'm adding it.


The service create was working fine but profile download and some other pieces were not working due to lack of the ServerService class. Changing from ServerService to plain Service would be an API rev.


Si el 1 esta en todos los request metelo en el BASE URL :D


Just state `elements_(rhs.elements_)`, likewise in the two lines below.


In trigger thread you made an announcement to 1.8.1.280 and I made an update immediatly.
Now you changed .280 to .281 but my installation isn't able to see the update - it stays on .280
Is this an issue?


@Rochet2, Thats right! :+1:


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


No, it is a cache :)

Clear the cache folder.


Si no lo usas borralo


Надо не только подсвечивать инрут, но и выводить текст


@sam-github yes, subclassing is the intention. That way only the hooks that are required need to be implemented and the rest of the functions can be left as the defaults.

@rmg The distinction is mainly for hooks (due to rest actions) vs events (needed for testing)


Yes, that should be fixed


Thanks Jesse!


ah, crap.. Good catch.


Deallocating `elements_` is redundant -- this is a constructor, we have no existing data.


next release i'll update


Hi,
Thank you for the patch. You are right about it. We need an action not a filter here. I will add that. We need an action  to allow any future module to clean their act for the wall galleries( The gallery delete action will always fire, just providing another hook so that it can be specifically targeted(same applies for the Id, I was wrong there in returning Boolean instead of id.)


No funciona con las Relative Layouts


Nuestra nomenclatura de nombres diferente.


@bartvm This brings up a pretty fundamental question we need to address. How does Fuel decide to handle datasets which are too big to be practical for in-memory use (SVHN is borderline), but aren't distributed in any format that allows lazy out-of-core access? Pylearn2's approach was to make you run some preprocessing scripts to get it into a format that could be loaded in such a fashion. Feels icky but I don't see a more elegant solution.


This test is going to use a hell of a lot of memory. A lower bound of about 12 gigs if I'm not mistaken...


Eventually, I would like to replace the `git` package here with the respective `mingw-w64-${_arch}-git` one, of course... :grinning:


looks good to me :+1:


Generalmente ponemos todas las cosas de layout primero y luego los otros atributos.


[Here](https://github.com/GuillaumeGomez/c_vec-rs/tree/cslice-evolution) are my updates. What do you think ?


Realised this was missing during irc conversation:

>16:28:48	mbalmer	ok, I will go through all this again.  I think we maybe also need to check the trace file in :finish().
> 16:29:35	daurnimator	mbalmer: yeah good idea. PQfinish calls PQuntrace for us. but yes, in my patch I don't remove it from the uservalue on :finish getting called
> 16:29:51	daurnimator	I can fix that now if you want?


r=larsbergstrom


You are right, thanks! Good catch!


Luke, the trigger is working with several records inserted at the same time, let me know if you have a good review of this.


The undefined offsets are after or before the revert ?


Did you forget to remove this line ?


В coffee так можно не делать, чтобы передать область видимости можно использовать жирную стрелку => почитай!


..but I guess when we get to the last batches, MySQL will have to scan lots of rows, which is probably what you meant.


can we set this to localhost as the default


No usamos menu


It would make a lot of sense to install only the *Git for Windows SDK (MinGW 32-bit)* for the 32-bit `msys2-runtime`, don't you think? I wonder how we can ask for it in this here Javascript code...


I'd make this

request.session.get("email").map { email =>
  if (Global.whitelist contains email)
    email
  else
    ""
}


`std::bad_alloc` on allocation failure can be thrown here as well.


What's the reason for this change?


all tests pass:
success: http://build.servo.org/builders/linux1/builds/918
success: http://build.servo.org/builders/linux2/builds/916
success: http://build.servo.org/builders/android/builds/954
success: http://build.servo.org/builders/mac1/builds/1133
success: http://build.servo.org/builders/mac2/builds/1133
success: http://build.servo.org/builders/gonk/builds/104


This fail with the message: `{ code: 'InternalError', message: '"string" == "object"' }`


@goodengineer fijate los comments, cualquier cosa decime .D


* Po przejrzeniu kodu nie zrozumiałe dla mnie to co dzieje się w `lib/dispatchers/local/dispatcher-local.js`
, `lib/chip-types/service.js` oraz `lib/chip-types/chip-manager.js` myślę że do przyswojenia podczas tworzenia dokumentacji.
* Pozostałe zmiany dla REST w `base-chips/use/channel.www_server.js` oraz te w `base-chips/define/service.user_manager.js` jak dla mnie są zrozumiałe.


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 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-3.69%25-green.svg)


`std::bad_alloc` forgotten again.


use "" instead


request.session.get("redirect_to") getOrElse routes.Application.index.absoluteURL()


И да в coffee не принято писать this вместо этого используй @


> Pointing me to the standard really doesn't answer my question.

Your comment was incompatible with CSV format standard (suggesting I could split two types of delimited rows with the same `csv` object). That's why I referred you to the standard. Unless you suggested that I should change the delimiter mid-parsing and have CSV do the splitting for me, in which case I have to tell you that this is not part of the standard (the standard only mentions comma as column delimiter). Though `csv` library allows using a different delimiter than comma, it doesn't allow using multiple delimiters in the same CSV file. It just doesn't support that (have a look [here](https://docs.python.org/2/library/csv.html)).

> a) what it is that you are trying to store

Metadata about the list. This is something that Citizen Lab folks can add to the file. We talked about it [here](https://docs.google.com/document/d/1PX9L5Loqkq94sbUTpv0zQrJ4OIkkOBDpKCaHdzkDTp4/edit). I, Phillipa, Masashi, Jakub, and Adam all talked about this and exchanged comments to arrive at that point. I mentioned it a few times during our meetings and asked everyone to read and comment on it. You could read that and leave your comments there at the time. Since we all agreed upon this format, I went ahead and implemented it.

>  b) why your dual use of CSV/ key-value pairs in comments is an appropriate solution.

It was easier for people to add to their lists. It made more sense than using something more sophisticated (e.g. using CSV fileds with a special format). Again, we talked about this on that document and in our in-person meetings.
Additionally, to support this, I only had to write ~10 lines of code, which is roughly the same amount that I would have written if we decided to use CSV format for comments.

The rest of your questions are just redundant. You are reiterating the same question.


Para ser mas claro si queres podes hacer is Empty.


A destructor might throw, though by default all destructors are noexcept since C++11... Don't know what will happen here. Such specialization won't compile, I suppose.


Proponuję nie wrzucać do changelogu każdej najdrobniejszej czynności, lecz zbiorczo jednym wpisem opisać to co zostało zrobione w czasie całej pracy.


Before
Le 3 mars 2015 22:53, "Jocelyn Fournier" <notifications@github.com> a écrit
:

> The undefined offsets are after or before the revert ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PrestaShop/PrestaShop/commit/27e4841246ad0df00b65d6d3fcbadc50576db18d#commitcomment-10017422>
> .
>


fast-forwarding master to auto = 0ff54a97


how does this work?


Which one? A cache on server?
I cleared browser-cache and used another browser normally not used but both say "RainLoop is up to date".

The previous update to .280 ended with a message "repository not reachable" (or something like this), but after reloading the "About" page, everything seemed to be ok.

Further info: I use the owncloud-app of rainloop.

Should I delete the .280 folder in rainloop "v" dir?


@Globegitter did you want to revert this name change?


Такие значения лучше вынести в конфиг файл


Para tener el ID del record
http://stackoverflow.com/questions/2848056/how-to-get-a-row-id-from-a-cursor


Missing check of that `size_+1` won't overflow.


can you merge this with run_instance.sh, make it default to dev, and then update /start.sh to use prod mode instead?


Ok, this makes sense :) I will take a look at what could cause your counter issue


When you say "node" do you mean *use nodes*, *definition nodes*, *assignment nodes* (if distinct from definition nodes), or all nodes that refer to a symbol?


This breaks on systems with grep older than 2.5, which is what added the `--color` flag.  Unfortunately, Git for Windows ships with grep 2.4.2.


Try-catch is not needed here at all, I suppose.


Can you make the camera a public property of the Item Spawner? This will allow us to set the camera in any given scene.


as discussed, lets update tasks cluster by cluster, that way it will use the index on clusterid when looking up tasks.


rax retry tests


Again, try-catch which simply rethrows is redundant...


saw approval from Ms2ger
at https://github.com/jdm/servo/commit/9a3d58032d53dce1627b2e443c131d45e2d3687d


merging jdm/servo/pageload = 9a3d5803 into auto


jdm/servo/pageload = 9a3d5803 merged ok, testing candidate = 13a3a48e


Maybe just `boost::throw_exception(std::out_of_range (...));`?


Same here, plus broken indentation ;)


>
>
>
>    1. But there are 2 rather wild-looking chdir() calls in
>    vte_get_working_directory() - I think the code is supposed to just find the
>    shell's working directory but not change Geany's working directory
>    (probably caused by copy/pasting without thinking what it causes in Geany).
>
>  I was about to remove those, but I then wanted to check if our code was
> OK for uncommon OSes and import changes from xfce4-terminal
> <http://git.xfce.org/apps/xfce4-terminal/tree/terminal/terminal-screen.c#n1880>,
> but I then stumbled upon this link <http://unix.stackexchange.com/a/94365>
> trying to find a better way of getting the CWD (as e.g.
> */compat/linux/proc/%d/cwd* doesn't exist in the FreeBSD 10.1 VM images
> <https://www.freebsd.org/where.html>). If I believe this guy, reading the
> link doesn't work on Solaris, and I'm way too lazy to try and find/install
> a Solaris installation to check.
>
> So, as they should actually be harmless (it chdirs back)
>
Ah, I've misread what it does and haven't noticed it cd's back - then
there's no problem.

By the way /proc is missing on OS X so this part doesn't work. I'd have to
dig into mostly undocumented sysctl() calls and since it behaves quite sane
now (always starts in home directory), I'm a bit lazy to do this now.


@Frymasterspeck I have pushed an update with a note so it was there.  I also changed the logo to match what you and Alex agree the temporary logo should be.  When it comes to the note I wrote the following.

Note: Column height is determined by the number of categories, not by dollar amount. While each block size represents its respective value, space is added in between all the blocks to make even the smallest funds visible.


@MarkEWaite ok, thanks for investigating on this. Can you also please run git log with the same format string and paste here to compare the actual output with expected ISO format ?


> Should I delete the .280 folder in rainloop "v" folder?

No.

RainLoop cache folder is:
```
./data/_data_xxx/_default_/cache/
```


@davideast - could you look at this commit and let me know what I should change before submitting a PR?

Here are the issues I had:

1) I could not figure out how to use a template for the Component that has a `<content></content>` tag to allow the user to specify their own menu content instead of hard-coding the content in the template. I could see the children getting distributed into the content element but the decorator selector did not match them when I would have expected it to match. Seeing as their is no documentation and no examples doing this, I left it unsolved but if you have pointers, this would be my preferred mechanism for this.
2) There is no support in the current gulpfile for external templates (url instead of inline), so I left the entire template in as an inline
3) I could not figure out how to extract the decorator into a separate file (so it could theoretically be re-used by multiple components). When I did this, I got a weird error about something not being a function. I was not sure whether I was doing something wrong or simply trying to do something that is not yet supported. If you would like me to show you the code I tried, let me know.


Hola

Pudiste compilarlo ?  Prove con Ant, pero me arroja variados errores.


Nitpick: Should that read "defined"?


Just a thought - I think it would be worthwhile to have a setting for RTE fields to limit the image fields that show up when you go to upload an image. I have situations with one image field that is dedicated for a specific purpose (with images called directly in the template file) and then a more general images field for storing images for embedding into RTE textareas. I think users could easily get confused, especially if the specific purpose images field happens to be listed first.


`operator[]` is usually made non-bounds-checking; only `at()` does so... Don't know if such variation is acceptable.


Oh gosh.


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-MacOS-10.9-clang: Tests Failed

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


side effecting function should go in `Execution.from` :smile:


Это действие лишнее, ангуляр использует двунаправленный байндинг, ты же не зря прописываешь ng-model в инпуте?! Твоя переменная $scope.transporter будет изменятся автоматически, как только ты изменяешь данные в инпуте


Hmm? The move assignment operator is misssing ;)


*A commit that only JS pros will understand. Line 65 will shock you!*


This was a typo that was accidentally introduced in commit: [0d98faaca](https://github.com/hep-gc/shoal/commit/0d98faaca16d8ce461e7bdc667c1c580ad4b5ad2)

I never detected it as an error until i was testing recently on different flavours of linux.


Already removed merge conflicts.
Committed settings produce already a quite nice video.


It looks like we will need to do a polling wait here prior to deletion.


Let's make this 100% on mobile; they'll just stack one on top of the other and people will be like "god it takes forever to scroll past these hats"


Latest compiled code has this issue:
```
$ REDECLIPSE_BRANCH=inplace REDECLIPSE_HOME=~/.redeclipse/ ./redeclipse.sh
cannot determine game version, please ensure 'config/version.cfg' properly loaded
```


Just state `if (this != &rhs)`.


"managed" in the commit message should read "managed by the runtime".


Should we do it inside the processor instead, to validate it at the Java API level?


Whoops sorry I did not. Thank you for noticing.


Is there a need for transition here?


`std::copy()`?


No recordaba como insertar... (buscar un orm para trabajar en android yo no voy a estar en esta vaina)
http://elbauldelprogramador.com/programacion-android-insertando/


If this code is not necessary, why you didn't delete it?


Capacity assignment is missing.


 #include "DebugDraw.h"

    Write
    Preview


:clap:


dont need to wrap in try catch.  If you need to log something when there is an exception, the caller should do it instead of logging here


**merged** #??


Hmm, the popped element actually must be destroyed. I've forgot that.


my bad, I should have cleaned it up


Nit: s/Picutres/Pictures


nope : P


![Uploading adoberfp.dll . . .]()
![Uploading AdobeXMP.dll . . .]()
![Uploading AGM.dll . . .]()
![Uploading AGMGPUOptIn.ini . . .]()
![Uploading ahclient.dll . . .]()
![Uploading arh.exe . . .]()
![Uploading AXE8SharedExpat.dll . . .]()
![Uploading AXSLE.dll . . .]()
![Uploading BIB.dll . . .]()
![Uploading BIBUtils.dll . . .]()
![Uploading ccme_asym.dll . . .]()
![Uploading ccme_base.dll . . .]()
![Uploading ccme_base_non_fips.dll . . .]()
![Uploading ccme_ecc.dll . . .]()
![Uploading ccme_ecdrbg.dll . . .]()
![Uploading CoolType.dll . . .]()
![Uploading cryptocme.dll . . .]()
![Uploading cryptocme.sig . . .]()
![Uploading Eula.exe . . .]()
![Uploading ExtendScript.dll . . .]()
![Uploading icucnv40.dll . . .]()
![Uploading icudt40.dll . . .]()
![Uploading JP2KLib.dll . . .]()
![Uploading logsession.dll . . .]()
![Uploading LogTransport2.exe . . .]()
![Uploading NPSWF32.dll . . .]()
![Uploading Onix32.dll . . .]()
![Uploading PDFPrevHndlr.dll . . .]()
![Uploading PDFSigQFormalRep.pdf . . .]()
![Uploading pe.dll . . .]()
![Uploading pmd.cer . . .]()
![Uploading ProtectedModeWhitelistConfig.txt . . .]()
![Uploading reader_sl.exe . . .]()
![Uploading rt3d.dll . . .]()
![Uploading RTC.der . . .]()
![Uploading ScCore.dll . . .]()
![Uploading sqlite.dll . . .]()
![Uploading ViewerPS.dll . . .]()
![Uploading wow_helper.exe . . .]()


Any reason error still needs to be declared at the top of the method rather than declaring it here?


TeamCity Akka.NET :: Akka.NET PR Build [Build 95](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=147&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


This one threw me for a loop sometimes.  I never quite put my finger on it. Thanks for nailing it down!


Can we make sure that we have something in the git log for things like this.


I can't tell if I've provided what you need...does http://cnx.org/eip-help/flash help?


6,40 Mo la fenêtre met un peu de temps à charger.


some tests failed:
failure: http://build.servo.org/builders/gonk/builds/105


Thank you LibreOffice :)


You added the media queries to your CSS file, but didn¹t put any CSS inside
of them to actually do anything at those screen resolutions.

- Garrett

From:  chporter2 <notifications@github.com>
Reply-To:  chporter2/cPorter-Site
<reply+002616ec33cc1d46f6886f57986d0387f9896983d70ebfe492cf00000001110de4bc9
2a163ce0098d767@reply.github.com>
Date:  Tuesday, March 3, 2015 at 4:08 PM
To:  chporter2/cPorter-Site <cPorter-Site@noreply.github.com>
Cc:  Garrett Wills <garrett.wills09@gmail.com>
Subject:  Re: [cPorter-Site] Added Design Assets (8bafc38)

Hi Garrett,
Just curious as to why I earned a 10/40 on the week 5 homework assignment.
Was there something I did incorrectly or missed on the assignment? Thanks!
Chad

‹
Reply to this email directly or view it on GitHub
<https://github.com/chporter2/cPorter-Site/commit/8bafc38e0728e1aef167f66104
4028db8144306a#commitcomment-10016615> .


Nevidím moc smysl přidávat verze PHP a rovnou je hodit i do allow_failures. Třeba ta 5.7 teoreticky začne fungovat až to Filip opraví v Kdyby/Doctrine, ale ostatní ne.

Nevím jestli jsi koukal, ale testování na PHP 5.3 a 5.4 jsem asi před půlrokem [odebral](https://github.com/o5/grido/commit/ca9c3f564cb0e12003665e9cfe2993d56e77b99a). Je tam i nalinkovaná diskuze, kde je trocha historie o [hackování](https://github.com/o5/grido/blob/0e36dbf8398e9587c1bc7d2b2449fc2a6878b807/tests/Grido/DataSources/TestCase.php#L38-L40) testů. Možná by stálo za to prozkoumat, zda by ta minifikace HTML před assertem, co tam napovídá pepa, ty problémy nevyřešila.


Yeah i was debatting... I think having a check at the API (JAXRS) makes sense anyway.
Another layer would be straight in the java APIs, and then another alternative is in the processor directly... I am not sure, we can review it.


@cedricbu https://github.com/ClusterLabs/resource-agents/pull/581 i addressed the issue here.


Also `std::bad_alloc` and object's destructor.


Thanks, after clearing cache rainloop showed me, that there is an update to .181. available and it starts the update.
But after a while: "Cannot access the repository at the moment.".

Reloading the admin panel and .181 is installed with "RainLoop is up to date."

Seems to be an issue with repository. I never got this error ater an upgrade before.

Another question: I am right to be able to delete older versions in "v" folder?


<inherited home from generations ago>


This should be using Stop:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/kubectl/stop.go#L66

not creating its own (wrong) version of this. If this routine can't be used as a library routine, kubectl needs to be refactored.


I will attempt it. My initial test was to run the same job again.

The second time I ran the job (a multi-configuration job), it moved the error message from the flyweight executor's log to the logs in the jobs in the various configurations.  The error message disappeared from the flyweight job.

The third time I ran the job, it resolved the error message from most of the configurations.  A few (one or two) failed, but most succeeded.

The fourth time I ran the job, the error message was resolved in all configurations.

I'll attempt to print changelog entries from all the changes in the repository I was using, in case the problem is somehow specific to the changelog of that repository.


Ad 1301


Try-catch which just rethrows is redundant...


Thanks :)
Le 3 mars 2015 22:59, "Jocelyn Fournier" <notifications@github.com> a écrit
:

> Ok, this makes sense :) I will take a look at what could cause your
> counter issue
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PrestaShop/PrestaShop/commit/27e4841246ad0df00b65d6d3fcbadc50576db18d#commitcomment-10017573>
> .
>


Sorry if it's a dumb question, but you can explain why we are having to special case ScriptCs.Contracts here?


I've added a drop shadow to the index page and have committed the change to the markup.


Для корректной работы с уролм 3 и выше степени вложенности путь должен начинаться с /


Cheater!! :-)


saw approval from larsbergstrom
at https://github.com/glennw/servo/commit/86baef2cc0acfee669b5ffad7eca9ac10a59ad6f


merging glennw/servo/remove-iframe-layers = 86baef2c into auto


glennw/servo/remove-iframe-layers = 86baef2c merged ok, testing candidate = e19fbc68


Good to be merged.


@ghoneycutt yeah, we're favoring `examples/` to `tests/`. There's been a lot of confusion when we ask contributors to update tests, and examples is more accurate of what we're looking for for files under this directory.


If `null` is passed in to the `options` parameter (as it is from https://github.com/webpack/webpack/blob/6173bb83bad2b07e8f9f965f19f1b777c652b2fb/lib/WebpackOptionsApply.js#L191), then an exception is thrown because this if statement evaluates to false and the `else` block is executed.


Is there a reason to include any code at all after the `throw`?


das sieht gut aus.


surprisingly, yes. usually the `this` context is defined by the context that called the function so it seems that `$parent` would indeed be `this`. however i think knockout is doing something funky with the context when it gets called and the `this` context is the `formLink` object if i don't bind it.


> Another question: I am right to be able to delete older versions in "v" folder?

Yes, you can do it if you want.


I guess this better since all the other members are static so it make things consistent.

Purely out of interest, is there any other reason other than consistency? Does non-static cause any problems?


we need virtual-physics to be listed here under Visual-only (right?)


Было бы хорошо, да наверное и правильно чтобы урл был /signup/shipper или /signup/transport-provider и если ты заметил у них у обоих есть стейт signup, надо воспользоваться фишками ui-route и использовать наследование


Good question... Thinking about it, slightly radical proposal which might make things a lot simpler on the long term: What if we simply expect all datasets to be in HDF5 by default, and I mean including MNIST, CIFAR-10, etc. That's what HDF5 was created for after all. In this case it would be easy to let the user decide whether to keep things in memory or out-of-core (which is a choice that really should be available anyway for all datasets 8-80GB). We would then provide a series of preprocessing scripts, which could take the form of something nice and simple like:
```bash
fuel-convert --mnist  # Finds MNIST files in the current directory and outputs mnist.hdf5
fuel-convert --cifar10  # Etc.
```
The MNIST dataset wrapper itself could now be as beautifully simple as:

```python
class MNIST(HDF5DatasetFile):
    file = 'mnist.hdf5'
    load_into_memory = True
```
`HDF5DatasetFile` could automatically support the `start` and `stop` arguments, a new `load_into_memory` argument, a `flatten` argument, etc. `train` and `test` could be groups in the HDF5 file, and hence also automatically detected.

The downside is that the user is required to run a preprocessing script, but maybe that's not all bad. Some files are easy to read, but for datasets that are distributed as raw images, inefficient text files, etc. you'll probably want to do this preprocessing anyway. For `binarized_mnist.py` @vdumoulin already ran into similar issues which he resolved by trying to read a NumPy file if it exists, but falling back to the original, slow text file if it doesn't exist. Also, all this preprocessing (like switching around axes as is happening in this file) would only be done once, making scripts start up faster.


I could reproduce the same error locally.


Kevin,

Can you help to take a look at this why this jQuery code doesn't work here? I guess it might be because of JS loading sequence?

http://www.steamdev.com/zclip/

Maybe we can try a package here https://github.com/pix/meteor-jquery-clipboard
but it's not in atmosphere.

-Bryan


Already fixed in another commit.


When this occurs, will it be obvious by cross referencing with other messages which package is missing? If loglevel is set to debug then I guess we have the 'Finding package' message immediately above, but how about when the log level is higher, e.g. INFO which is the default? Perhaps we should include the package ID in this warning message?


@bezoerb fyi. Cut a new release. Would love to land streams wrapper support in the next one, but we have time.


Hi Viktor, just discovered this issue too....I have applied your fix and it works :) thanks...You have just saved hours of debugging :)


\o/


Má to smysl podobně jako u hhvm/hhvm-nightly, pokud se dlouhodobě prokáží jako fční, lze je z failures vyhodit. PHP 7.0 je WIP, takže je to otázka dopředné kompatibility.

Koukal, nevidím moc smysl v testování renderu, o ten tomuto balíčku vůbec nejde (mezera nic nezůsobí). Raději testuji jednotlivé objekty. Navíc to testuje spíše Nette samotné než tento balíček.


Omg jak tohle může projít testy? :) Vždyť tak jsem to zkoušel a neprocházelo. Navíc si tam měl new PropertyAccessor(TRUE, TRUE); a pak ten sukřík $name = '[' . $name . ']';.


what's all the XML that you've committed here?


@kjdav if you search "formative assessment" in this commit you will see that @philschatz replaced it with Practice Problems.  Is Check Your Understanding part of Practice Problems?


Sooo.. I thought about this. The issue is (from my tests) that every single time the developer calls vendor:publish this would get re-published. Since Laravel has no other way to detect if the migration already is copied over.


Could you try to get the latest version of the code, give it a try, then revert only the changes on Manufacturer.php (back to version c0a4fe2) and tell me if everything is back to normal.


@BenExile modify email address - jump into skeleton app dir

git config user.name "Your Name"
git config user.email "you@example.com"


TeamCity Akka.NET :: Akka.NET PR Build [Build 95](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=147&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:10:43
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


Toto je třída v Gridu, ne v Symfony.


> nevidím moc smysl v testování renderu

To je dobrý point no.


docstring refers to an int, but bools are used


Ok. I think the version should be removed from composer.json anyways. If my understanding of the composer docs is correct.


\> Tells players to use `/setplayer`
\> Actual command is `/addplayer`


przeczytaj mój poprzedni komentarz - tym sposobem zapełnisz changelog bardzo szybko i pobijesz wszelkie rekordy dorabiając tylko jedną funkcję od lms-a. Nie od tego jest changelog. Przecież masz git log, a w changelogu opisujemy zbiorczo całą spójną zmianę jednym wpisem.


After 1.6.0.13, I just revert manufacturer.php to get produc counter
working. It's stange because I get correct numbers in php errors.
Le 3 mars 2015 23:19, "Jocelyn Fournier" <notifications@github.com> a écrit
:

> Could you try to get the latest version of the code, give it a try, then
> revert only the changes on Manufacturer.php (back to version c0a4fe2
> <https://github.com/PrestaShop/PrestaShop/commit/c0a4fe2e18f0a29c7831f86cc90165a963aaef01>)
> and tell me if everything is back to normal.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PrestaShop/PrestaShop/commit/27e4841246ad0df00b65d6d3fcbadc50576db18d#commitcomment-10017927>
> .
>


This might fix the problem @pmbohm but I don't think this is a satisfactory solution. We can't just fire an unrelated event because this has a high potential to cause bugs in far-away and unrelated code.

Is there any other way we can get the behaviour we need? We obviously trigger a 'temporalextentloaded' event from here and that would seem to be a much more appropriate event to listen for.


@permcody This does not work. You can also throw in some random text instead of the function and it still doesn't work.


Random equals sign? :P


@philschatz I think we also need links-to-physics.  @kjdav on the design template it is "Links to Physics" (plural...not that there is necessarily more than one link per feature).  Above you referenced "link-to-physics" so I think we should be careful about whether the tag should be singular or plural.


:+1: Mesh renderer/shader is boss!


Just `announce_auth_failure;` is fine.


the wavelets look awesome


> nevidím moc smysl v testování renderu, navíc to testuje spíše Nette samotné než tento balíček

To jsou dobré pointy no...

> PHP 7.0 je WIP, takže je to otázka dopředné kompatibility

To mi moc není jasné. Koukám do logů Travisu jen když zařve, takže to dopřednou kompatibilitu stejnak neřeší. Nebo?


Dropped the code in, and it seems to be working great.

Will test more over the next day or two, but so far, i think its good to go.


This one was pretty trivial, since there were so many inner loops. In this one, I simply keep everything in flat format, then within crypt_all, marshal over to mixed layout, do the crypts, and when done marshal back to flat format. That means almost NO changes to the jtr method formats, other than crypt_all.

Speed is probably only affected about .1% so who cares.  The code is much simpler this way.


you need to add `, :integer` for this to be a valid `change` migration


And with the latest version, at line 210 if you change
```php
			if (count($counts))
				foreach ($manufacturers as $key => $manufacturer)
					if (isset($counts[(int)$manufacturer['id_manufacturer']]))
						$manufacturers[$key]['nb_products'] = $counts[(int)$manufacturer['id_manufacturer']];
					else
						$manufacturers[$key]['nb_products'] = 0;
```
by
```php
			if (count($counts))
				foreach ($manufacturers as $key => $manufacturer)
					$manufacturers[$key]['nb_products'] = $counts[(int)$manufacturer['id_manufacturer']];
```
does it change anything ?


You might want to make this message a little more clear, with regard to the input format.


Awesome!


Nice!  Yes.  Can you see that `st_init_table` (whatever that is) is really doing most of the work?  Everything else is about handling various arguments and preparing the hash for initialization, but whatever `st_init_table` returns is the thing that's actually storing the information.

Try to reason through how getting and setting values work.


@alanshaw: less has some stuff changed and you need to add clean-css or its plugin.


That's a late lunch


Do you see a chance how I can help to debug the error message or should I ignore it for now.
Rainloop seems to work as expected.

And one last question about the "v" folder: Is there always (atomatically) the last version used and if I want to revert to an older version I have just to delete or rename the later (last) versions, which I don't want to use?


Nice work @powersa! :+1:


Since this is a public method, would it make more sense to do
```ruby
def usage_limit
  promotion.per_code_usage_limit
end
```
?  If we did want to add this at the promotion_code level in the future we'd probably call it `usage_limit` right?


Jo jasně jsem slepej.


Yes, this creates a two-element array named `args` that contains elements of type `VALUE`.


Can this be done without using !important?


je pense qu'une musique type gameboy sera plus raisonnable ;-)


At least generateJson needs to run before the other 2, so that should probably run first.


> To mi moc není jasné. Koukám do logů Travisu jen když zařve, takže to dopřednou kompatibilitu stejnak neřeší. Nebo?

Pokud bych používal hhvm/PHP 7.0, tak bych na Travis mrknul tak jako tak.

Nevím jestli budeš hlídat release date PHP 7.0 a testovat balíček hned po jeho vydání, případně omezovat composer na <7.0. Tímto ti tato práce a stresy odpadají.


@reitblatt you might as well merge this, right?


As for now, there is no better way to do it, apparently: https://github.com/moment/moment/issues/778


And notes: https://github.com/addyosmani/critical/releases/tag/v0.5.5 Please feel free to edit as needed :)

cc @sindresorhus as an fyi too


@mattharris5 Can you take a look at this? This is Kevin Li's ( @efawe  ) first contribution to DreamSIS. It is the basics of an implementation of generic tags for mentors. For example, someone could mark "Steering" as a tag, or "CCRA," with details about that in the title attribute of the content tag, and are accessible on mouseover.


`rb_hash_foreach` might be returning something, but if it is, we're not doing anything with the return value.  Judging by this call, I'd guess that `rb_hash_foreach` is declared as returning `void`, meaning it doesn't return _anything_.  In C (and other languages) it's possible to have a function that doesn't return anything.  This is different than returning `NULL` or the equivalent.

If `rb_hash_foreach` has a `void` return type then trying to assign the return value of `rb_hash_foreach` to a variable would actually result in a compiler error.


@hufnagel this patch doesn't apply cleanly anymore, this is the head now:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMRuntime/Scripts/SetupCMSSWPset.py#L542


This isn't a fix, more of a hack. Since it was working fine before, something I changed must have broken it.  Please don't include this change.


You should check if there isn't a grade and say "Not Graded" or something.


@razzeee got your pm on kodi forum but i wasnt notified about it so letting you know here that i left a reply, notifications from kodi forum seem a little hit or miss, sorry for additional traffic


again?


Hi ppat,

It was a save action from my IDE. I'll remove all of them

Respect to null safe check. It's very dificult to identify which parameter
is not defined simply by the NullPointerException trace. Maybe a error log
can help in this case. I included this check cause the test trident
topology fails if no "rabbitmq.requeueOnFail" parameter set. It's really
necesary define all the pameters ? May some defaults of then can simplify
the API.

What do you think about ?

2015-03-02 23:14 GMT+01:00 ppat <notifications@github.com>:

> I'll mention this only here but this applies for every else in this commit
> as well as other commits in this pull request.
>
> Please remove the final qualifer on all local variables. During the early
> years in Java, there was increase of using final as a qualifier on local
> variables as a defensive mechanism. I believe that is now considered a bad
> pattern, at least I do. I would remove the use of it except where it
> actually makes sense. Instead of adding the final qualifier, I would prefer
> keeping the code blocks smaller to avoid making the mistakes that would
> necessitate this kind of defensive coding.
>
> Similarly, this was intentionally written to blow up and throw
> NullPointerException when used in a manner that required you to provide
> defaultIfNull checks. The idea here is also to avoid defensive coding and
> instead provide a better API so that the null checks are not needed in the
> first place. In this particular case, what's more important is what creates
> this maps (ProducerConfigBuilder... ConsumerConfigBuilder...), as that's
> the API that is exposed to the user. If the API that is exposed to the user
> is allowing the user to write code that creates this kind of problems, then
> that's exposing a major flaw in those APIs and we should fix it there
> rather than defend against it here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frascuchon/storm-rabbitmq/commit/bffc8ffaefc04ee595995d1453b35517bba0d877#commitcomment-9995344>
> .
>


> should I ignore it for now.

Yes.

> if I want to revert to an older version I have just to delete or rename the later (last) versions,
> which I don't want to use?

To revert to an older version (x.x.x) just copy a file (with replace):
```
./rainloop/v/x.x.x/index.php.root -> ./index.php
```


The dev down the hall reported that they cold not get the jetty test
deployment to work with RI enabled. There you go.

On Tue, Mar 3, 2015 at 4:34 PM, ajs6f <notifications@github.com> wrote:

> Does this seriously just fix an issue that goes back to the 2-series? Damn.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/barmintor/fcrepo/commit/c14763c13403f8fd8d790e8577576b1fbcb0da04#commitcomment-10017059>
> .
>


We initially set `args[1]` to `Qnil`, which is the C representation of Ruby's `nil`.  Presumably `args[1]` is now either `Qtrue` or `Qfalse`, which are the C representations of Ruby's `true` and `false`, respectively.

Where was this change made?  How did it happen?


Já pak ale stejnak postrádám význam třídy Grido\PropertyAccessors\SymfonyPropertyAccessor. Měl bych asi raději jednu metodu typu $grid->getProperty() která by s vytvořila objekt (+ zacachovala) a použila to
```php
if (is_array($object)) {
    $name = '[' . $name . ']';
}

```

Ale chápu, že vám to asi bude smrdět prasárnou co? :)


yeah, I fixed it last night and they made changes again over night.


Right on, makes perfect sense to me and the confusion is definitely there as this convention was added before there were any testing frameworks like rspec-puppet or beaker. The module docs[1] should be updated to reflect this. This could be a breaking change to people that have automation around grabbing a module and running `puppet apply tests/init.pp` or otherwise interacting with these file(s).

[1] - https://docs.puppetlabs.com/puppet/latest/reference/modules_fundamentals.html#module-layout


crazy


Sometimes I run across code that needs to be fixed while I am adding new functionality as in this case but I do see your point.  I'll be sure to try to organize my commits better going forward.


TeamCity Akka.NET :: Akka.NET PR Build [Build 96](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=148&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


J'en ai de deux minutes qui font 2 mo je les mettrai quand j'aurais le temps


What does `rb_hash_foreach` do?  What does each argument mean?  What is `key_i`?


I didn't see any references to `this` in these constructors, so there's no need to disable these warnings.


Damn I need to read everything not the first line... I see now the conditional attribute skip. This is indeed important^^


Guess it's too late now but IMO the name refactoring should be a separate commit


Fixed


Ahh - I see:
> define('APP_VERSION', '1.8.1.281'); :smirk:

Thanks and have a good night.


"Jolly good looking fellow-me-lad" (LGTM)


Thanks :)


Why use heroku instead of elasticbeanstalk?


cf. https://github.com/dobtco/knowledge_base/commit/4af3a7565926d5873196a080a8d9e77c1097f5f2#commitcomment-9959275


Sounds good!


Why do you think it is better to remove Gemfile.lock ?
In my opinion it gives an interesting clue on the last level used to develop the gem (but it might be bad practice, reason for my question).


The problem is handling all popular platforms, not one particular target.


89445170CoilVsSpacePeak_ExampleFiles (wfbuhl) - x86_64-MacOS-10.9-clang: Tests Failed

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


Thanks for reporting this typo. This is already fixed in the task_records branch. This branch will be merged into master and released by the end of this week.


Hehe w sumie racja :-). OK, będę pamiętał.


dont need to try catch just to log an exception and then throw it again.  Instead, leave the logging to the caller.


Why do you think it is better not to follow Gemfile.lock ?
Is it bad practive to commit it ?
I thought it was an interesting information to know which gem version were used during dev.
Just asking to understand your point of view.


Also, judging from the name, I wouldn't expect `rb_hash_foreach ` to do any kind of mapping.  If it were doing that, I'd expect it to have something like `map` in its name.  Instead, I'd expect `rb_hash_foreach` to be a way of doing something _for each_ entry in the hash table.

That said, I'd look at other contexts in which `rb_hash_foreach` was used to see if that helped me make any sense of it.  I'd look for contexts that were simple and that I knew I understood.  I'd find those contexts by looking at the map of Ruby methods to C functions at the bottom for methods I was familiar with and then look at their corresponding C implementations.

Working under the assumption that `rb_hash_foreach ` is a way to do _something_ for each entry in the hash table, I would first look at the methods which were most likely to translate into a single call to `rb_hash_foreach ` without much else.  If I found an example like that, I'd be able to see `rb_hash_foreach` in use without any extra noise.

I'd guess that basic iteration methods like `Hash#each`, `Hash#each_value`, `Hash#each_key`, `Hash#each_pair`, etc.  are the methods most likely to translate into a simple `rb_hash_foreach` call.


@openstaxalina I am not sure if I know what is important to change here. Will check back with you.


TeamCity Akka.NET :: Akka.NET PR Build [Build 96](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=148&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:09:31
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.PersistentActorSpec.PersistentActor_should_be_able_to_opt_out_from_stashing_messages_until_all_events_has_been_processed: <no details avaliable>

Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


But still an attacker could still send a header with a low height (e.g height 10) during the initial block download having a current date, so that the node is fingerprinted for 1 month. Correct?
We could insert in line 3365 something like:
CBlockIndex* pcheckpoint = Checkpoints::GetLastCheckpoint();
if (mi->second->nHeight < pcheckpoint->nHeight)
  send = false;
else
 ....


Zauważyłem dopiero Twój drugi komentarz.


Dear both I beleieve 'nb_products' need to be set for the template and retro compatibility too I guess.

Regards


Možná bych to vzal postupně, toto mergnul a pak vytvořil PR. Těch změn už je dost.


<img src="http://replygif.net/i/542.gif">


Check Your Understanding is practice-concepts (NOT practice-problems). But from my reading of the doc, what Phil did is correct. Can you double check to make sure you agree with my reading?

See:

+

+  <title>Check Your Understanding</title>
+  <exercise class="os-exercise">
+    <problem>
+      ...
+      <para>[exercise]</para>
+    </problem>
+  </exercise>


OK -- Will change to the iframe example you gave.


ASMLYTICS_APP_ID will need to be set based on the code: https://assembly.com/gamamia/metrics/snippet


Are you using `toString(part)` instead of `part.toString()` because an `ident` could be `null`?


@sellout: a very small first step. I eyeballed the annotations and I'm not yet convinced that they're useful at all, but if I can come up with a convenient way to visualize them maybe we can look at some examples together and figure out where to start.


Since you're creating a `std::string` here and when passed to the `SociConfig` constructor, why not specify `d` as a `std::vector<std::pair<std::string, const char*>>` (or `std::string, std::string`)?

Test efficiency isn't our highest priority, but we still need to keep it in mind, and I think this will make the test a little clearer as well.


see issue #227


Sharding is certainly part of it but the main motivation for trying elasticsearch was to deal with more complex structures such as those that would be handle via normalization and joins in a relational database. The primary example in the gazetteer is to deal with the many-to-many relationship between names and locations. If we were to group all of the locations that have the same name into a single composite record, the total record count would drop by 30-40% and the number of records that have to be merged in the client (after the matching) would drop by the same amount.
The current Solr based implementation is very efficient and will probably remain the basis for the OpenSextant tagger for the foreseeable future. We have some other use cases (focused on managing large networks of interrelated words.phrases and concepts) where the complex structure handling in ES may be of benefit.

Also ignore that interval tree stuff. That was just me messing around with other data structures.


We have removed that line now. Thanks for the suggestion.


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2191 of 2212 tests passed)

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


Done in 8ab1621cafb426f83450b30c42c6cb691038f627


Done in 8ab1621cafb426f83450b30c42c6cb691038f627.  Changed map name to negatedShortHand and added a comment.


Hi Gregory,

Actually the original code before my fix was not checking if $counts[(int)$manufacturer['id_manufacturer']] was set before trying to make the  $manufacturers[$key]['nb_products'] = $counts[(int)$manufacturer['id_manufacturer']]; assign, hence the PHP Notice.
I've changed my isset() test to a array_key_exists() test to see if it changes anything.


With automatic reconnection, the rabbitmq connection nature is of type
AutorecoveringConnection.

I have done many reconection tests in other projects (it's cause i think is
interesting this configuration parameter) but not directly with this
component, and all results were correct. You can take a look at the
AutorecoveringConnection implementation by if you saw any objection to this
improvement

Best regards!

2015-03-02 23:18 GMT+01:00 ppat <notifications@github.com>:

> Also please make sure to keep formatting consistent with the rest of the
> project.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frascuchon/storm-rabbitmq/commit/c92e7047703d356a3adbff34ff2b14e22ab7a575#commitcomment-9995388>
> .
>


I still think that the best solution would be to revert this commit in PHP 7 as well. The missing type hint does not cause any trouble.


Would it make sense to use `toString(requestArguments[i+1])` here to protect against `null`?


Nothing so clever as that. Just using it because that's how it's called below. I realize now that it shouldn't be possible for it to be anything but a buffer in this case.


Hmm... I'm not sure I'm completely wedded to HDF5 to that degree (various solutions seem to have a lot of overhead that I don't fully understand the source of... have you experienced that at all?). But it's one option to consider for sure...


I think we discovered this bug during the ECOOP tutorial. We still have it, right?


Comments have been moved to the appropriate blocks of code that they refer to.  Done in 8ab1621cafb426f83450b30c42c6cb691038f627


No, it's not this change, it's the missing explicit return type for the lambda at line 386 above. I'll commit a fix asap.


This ignores the range of the controlling player (read the java doc :-) )
You need something like this:
for(Permanent permanent : game.getBattlefield().getActivePermanents(new FilterCreaturePermanent(), source.getControllerId(), source.getSourceId(), game)) {
You always have to take care of multiplayer games.


yes, I suppose. Or the protection against null could be entirely removed with asserts about the length of the message, which have known requirements. It's not a valid message if any of these fields are null.


wow you are best


run with `make test`


@scottpurdy : @rcrowder pointed out we need to fix default c++ constructor, tested that it does not affect TP10X in nupuc repo.


This is a typo. Fix?


Comments added throughout this section of code detailing the process and changed name of variable from `invert` to `negateOperator`.  Done in 8ab1621cafb426f83450b30c42c6cb691038f627


@openstaxalina "formative assessment" became 'practice-concepts' and the label for it is "Check your Understanding". Current version in here looks correct to me. The robust content should be CHANGED to reflect this. W&N agreed to it, but it was probably after we got the content.


This is incorrect like that.


@retiv42 Changes to Button.java on line 39 caused the ready button graphic to draw smaller than it should, because you set the default behavior of set text to automatically resize button to the text size.


Thanks minrk. I really appreciate all this work.