yes sure!!

On Fri, Aug 28, 2015 at 2:59 PM santiaago <notifications@github.com> wrote:

> you should also test what happen is the user is not found or an error occur
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/taironas/gonawin/commit/622c5ee6d67318d771aff226a15222687ccdc755#commitcomment-12947340>
> .
>


It may be better keep `BaseScrobbler` in `/core/content/`, alongside the `Connector` base class.


Ok, thanks. We'll go for gettempdir() then and close your PR :-)


This change breaks OSX build for me

https://github.com/spesmilo/electrum/issues/1007


ok, reverted, it's becoming too complex at this point and not worth introducing a new named query and deprecating the other one.


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


Nothing understand


I don't remember why exactly I added them. Probably in first version the count with parentheses was on the same line as the kind.


I don't think this would get executed.


Hey there! I'm using this PR in a fork. Just noticed that the `null` here will raise warning in Django because it doesn't make sense for ManyToMany fields. The `blank=True` is sufficient to make the field optional.


@mastercoms is right. This is what the repo.blockserver.org domain pointed
to. Run maven with -U compile to update dependencies.

On Friday, August 28, 2015, mastercoms <notifications@github.com> wrote:

> It doesn't matter. The path to the dependencies will still work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DragonetMC/Dragonet/commit/7475d128acb0640d777c3699bb7cf4e7b68c67dd#commitcomment-12946065>
> .
>

--


Not at the moment.  I'll remove it.


lindo modelo de bolsa.


Ok, thanks for the answers!


remove chatbot_src


Why does one define have "" and another has null?


```xml
<optional>true</optional>
```


On second thought, this seems to not be a good catch :-( To the contrary, this seems to brake correct behavior. My understanding is that on a session resumption, a new *master secret* is derived from the session's original *master secret* and the new Client and Server randoms exchanged in the CLIENT_HELLO and SERVER_HELLO messages ... hence the call to `generateKeys()` with the original *master secret* being used as the *pre-master secret*.


Remove redundant `#include` statements (only `<cmath>` is required).


Adding custom.txt was already part of my fix.  Now it's included twice, which doesn't seem useful.


@bnoordhuis I don't believe this one got reverted, can you confirm it's good to cherry-pick to `v3.x`


make typo


In fact, I think `mappings.h` and `sampling.h` should be merged into one. They have the same purpose: mapping uniform random numbers in [0,1)^N to various domains.


Well now I feel especially silly squashing those commits. I'll pop mine out entirely and re-add yours.


I wonder if this would crash if `reactive-streams` is missing from classpath?


You could simplify this further:

    if os.path.isfile(basepath):
        trace_name = os.path.splitext(basepath)[0]
    else:
        trace_name = os.path.join(basepath, "trace")

    trace_txt = os.path.join(basepath, "trace.txt")
    trace_raw = os.path.join(basepath, "trace.raw.txt")
    trace_dat = os.path.join(basepath, "trace.dat")


The last 2 lines should be indented by 2 chars.


@joaocgreis is this good for a `v3.x` cherry-pick?


We capitalize acronyms: `IORs`.


@mykoz
OK.  Give Emacs a try if you're open to other editors.


Super si to uradio matori... ;)


`(%end)` -> `%(end)`


For further improvement , this should be a parsing exception .

also check : http://grepcode.com/file/repository.grepcode.com/java/ext/com.jetbrains/intellij-idea/11.0/com/intellij/lang/refactoring/NamesValidator.java


@mykoz
Good job!
I also like for command line:
    ls -Glp


I liked these plots... Why did you remove them?


This could also be ```attrib_type.value_count.to_i > 0```


Arf too bad ! Can you point this in the spec ?
If you're right thats means that gnutls does not manage session resumption correctly.


2 chars indentation;-)


Looks like you've made it not fail when these aren't present so could you remove these?


By default everything should be configured.

Have to admit it's been a year since i last edited this project. Maybe on this link someone can give more practical advices : http://www.ownedcore.com/forums/world-of-warcraft/world-of-warcraft-bots-programs/509163-spell-kicker-interrupt-cast-autoit-no-memory-read-any-wow-version.html

If it interrupts instant cast spells, you might have DoNotInterruptPVPSpellWithCastTimeLessThan too small value.

If you set AllowAnyPlayerSpellInterrupt=0, you can specify a white list like in the example you gave.
local SpellNamesCanInterruptOnPlayers = "Polymorph|Fear|Cyclone"
You need to make sure you write the spell names as you can see them in your client !


Same again.


NAK. That's not what I suggested at all: look at other plugins to see how the plugin APIs work (or read the plugin authors guide).

The `add_cmd_output()` method accepts either a string or a list as its argument. There is no need for you to reinvent this wheel yourself.


I would highly prefer this to be next to the method it aliases.


The array initialisation was to preserve the exist behaviour where it was unshifting zero onto the top of the array to make sure the array keys correctly represented the version ordinal numbers.

I had to enable strict mode on array_search as otherwise it was finding 0 (integer zero) when it was searching for (e.g.) 'VersionNewFeature' (i.e. a non-numeric string which gets type cast to zero). As it was now doing a strict search, I made sure all versions were cast to strings (and hence the foreach loop rather than using array_keys), and hence I made sure that version zero was a string too for consistency.

Does that make sense?


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


You need to preserve the generation of `trace.raw.txt` in the case when both `trace.dat` and `trace.txt` are present.  WA generates both the `trace.dat` and `trace.txt` but not the raw trace and it's a workflow that I don't want to break.


Or simply `st += ch`


Ich würde die Labels in einem Array ablegen und damit den Switch sparen:

    private static final String[] COLUMN_LABELS = {" ", "von", "bis", "von", "bis", "Wunschzeit"};
    ...
    public String getColumnName(int col) {
      return COLUMN_LABELS[col];
    }


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


Some better variable name is advised.


@cheevahagadog
To print python version:
$ python --version
$ python -V


Yeah we've discussed this before in a topic on the forum. It seems that the only way that it could work on multiple resources is by using exports and a wrapper class. I will make a tutorial about it soon. It's not the best approach but it seems like the only one.


It's a shame that `Venue`s don't have an associated Pombola `Organisation`, so you could just check if the person has a position associated with the same organization that the venue is associated with. I suspect that's not worth fixing as part of this work, but it might be worth a `FIXME` comment here and / or a ticket.


Und hier statt dem switch ein return String.class für alle Fälle (solange es keine anderen gibt). In solchen Fällen braucht man nicht überzuspezifizieren.


awesome


@cheevahagadog
OK with vim and atom!


Auch hier würde sich ein Array WEEKDAY_NAMES anbieten.


So uh...I kinda thought you'd preserve the `params` line at least :smile:


Fixed in https://github.com/xwiki/xwiki-platform/commit/de0216f77638a236510c2c169affd752e3d36e84


TeamCity pull requests :: netty [Build 140](https://garage.netty.io/teamcity/viewLog.html?buildId=886&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Compilation error: Compiler (new) Build time: 00:02:10


Then you will end up with alias everywhere...matter of taste in the end...


Muss es Database sein oder würde das Interface TableModel ausreichen?


:+1: fixed


@cheevahagadog
Good job!
xargs example is good; explanation could be stronger.


Looks good to mee


yeah missed that


I don't think we want to have self written ActiveModel::Validators.

Since all of them are only used in attrib.rb it should be possible to convert them into validation methods.


in fact this repos can help you a lot - you can use scripts, databases and etc
https://github.com/cooler-SAI/WoWSource434-1

https://github.com/cooler-SAI/WoWSource434

https://github.com/cooler-SAI/TrinityCore434


there needs to be a hard purge for files that are generated by "accident" (e.g. `index.php?lang=de_DE&issn=1234-5678` or `ajax_toc.php?yourmom=yoursister&query=drop%20table`... ).

i like the idea of having toc cache files differentiated by issn so we can purge all files related to one issn when the cron job gets a new issue from the remote API. we can switch back to `"cache/toc-".$issn."-".md5($query).".cache"` once we have a secure and reliable $issn parser function. but as long as this is just $_GET['issn'] it goes through md5() before it gets handed to the operating system.

i wanted to put the purging into `services/getLatestJournals.php` at first because it felt like the right place, but it didnt really fit in there. i'm not invested in its current position, however


Nice!  If it's for low depths I wonder if
return std::min(1 + (depth/(2 * ONE_PLY)),  MAX_SLAVES_PER_SPLITPOINT);
would make sense?


# trying a small code on github

print "Hello World"
print \
"""
.... this is a test....
 .. to see the output.....
"""


It's in my yamlish repository, I just forgot to declare the dependency.


So would this do the trick:

```python

    releases = relationship("ArtistCredit",
                             primaryjoin="Artist.id == ArtistCredit.id",
                             foreign_keys=[Artist.id])
```
I didn't realize the ids between these matched.

Then in `__init__.py`:

```python
F("releases", "releases.ref_count"),
```


Well, I am not sure it's the best way to fix this issue.

Right now doctrine migrations count on the fact that there will be a part of the name of the migration that will be interpretable as a number and use that number to order the migration.
If we now use string we will change the order in which the migrations are going to be treated.

For this version of doctrine migrations, I think that it's a better idea to throw an error if there is no number after Version. Otherwise people are going to start to have weird issue because the order of the migrations won't be consistent.


This is useless


Good Job. thank you. ^^


IMO this description doesn't tell what the package is good for. I'd suggest `short_desc="Collection of scripts interfacing pdfpages and pdfLaTeX"` or something along that line.


`No metrics for you` :laughing:


It looks like it has been already shipped, right?


yea, probably makes sense to move the date one year into the past or somesuch.


Explcitly setting python env is no longer needed. As we maintain compatability with both atm.


@sagarlekar 3ee441e63c818ab353686c2c631dc8c769ae1601


@hackmastera If you want to play around with colors, the Primaries, Accents, and Grays are in the [prototype](http://code4lib.github.io/2016.code4lib.org/prototype/).


I think it might be worth adding a comment that this checks for whether the person has *ever* had such a position, not just had that position on the sitting date. Or you could change the code to do that check, but since this code is definitely an incremental improvement anyway, it's fine to just leave a comment, I think.


Not really, I meant the text as "To start with, lets go buy bananas" etc, but I will update if you want to.


смотрел dmesg?


```
<%= locals.globalNamespace %>.load = loader;

function loader(segmentKey, dwhEndpoint, label) {
  // ...
}
```


since we are inside a `fn` I'd return fast here.


Fixes #115


Since this is an external project, we should keep our diffs to a minimum to avoid pain when rebasing to get newer versions. Therefore we should leave all formatting as it is, and avoid all whitespace-changes and other non-functional changes.


one


How does this even occur? lol


linre


Thanks!


Yes, unfortunately. I may want to expand the test to examine the data, to be sure.


For next major version this could change. Meanwhile current behavior must be supported.


@oksitosin
Ok to nano, spyder and sublime!


whoa


Yep. cordova-lib@5.2.0 is available at NPM


@tomchristie thoughts on leaving this here or should we just mention it in the release notes?


@oksitosin
good job!


@olivermt oh, I didn't get it :) no worries, I'll add a comma after with maybe so it's clearer ok?


No prob, you welcome :)


@rvagg It's not strictly necessary but yes, this commit can go in.


They are 2 chars allready, aren't they?


We should try to keep all Capture-specific code outside of Winium. Can we not get the same values from the outside? Couldn't you rather get the center coordinates of the first element, instead of the entire timeline?


Sorry, I keep meaning to get around to that.


Haha, thanks :)


They are 2 chars allready, aren't they? Those lines are inside a method... or I'm wrong?


I think that in this case just the release notes would be fine.


Example for document file preview using viewerjs and unoconv. Modified files:

    geonode/documents/urls.py
    geonode/documents/views.py
    geonode/documents/templates/documents/document_detail.html
    geonode/documents/templates/documents/document_view_recline.html

Quite straightforward. Uses unoconv (refer to wiki for installation) to convert Microsoft files to pdf or ods which are supported by the viewerjs library.


I have a slight fear about this that there may also be a problem with joint sittings, I should probably note that somewhere as well


I read the email now, and oh–my! Now it is even more confusing that `pycon.dk` is linked from http://www.pycon.org/. Makes no sense in my opinion.


I also tried the quest, the item you are given, does not seem to do anything, beside "casting" action.
Furthermore, you can only select Barada as target, not skulls, as the text says you could select either barada, either skulls (if I am not wrong)


Nitpick: I find that a bit misleading since save is a widely used method


O ExpireDate poderia ser colocado na configuração em miliseconds

```
// config.js
expireDate: (60*60*60), // uma hora

// controllers/github.js
expireDate: new Date() + config.expireDate
```


thanks for explicitly pointing it out that this is the password..


@frrrances Is this the correct way to remove the black underline?


I meant whitespaces, sorry.

So:
```ruby
!attrib_type.value_count ||  # If unlimited values
  (attrib_type.value_count && attrib_type.value_count > 0) ||  # If value_count > 0
```

Otherwhise it looks like both lines don't belong together.


@temasek
Just confirming that now having system with f2fs as file system does work.

Btw @arter97
What option(s) would you set for the system partition? There is now a fastboot mount option.

As system is read-only is there any flag you think could be interesting ?


@tomchristie feedback, please.


*2 whitespaces


@azerupi
> I always was under the impression that GPL was not a very good fit for the movie industry as studios often want to customize tools to work with their closed source tools.

They can use GPL software internally. Everyone does. As an example, many Linux distributions include GPL libraries, and they are used everywhere. You can even write code using GPL, and it isn't contaminated by the GPL unless you distribute it.


Yes you are right. I changed this first to will_save and I reverted by error. I'll commit it again. Thanks


There are not so many aliases I hope. But true we didn't agree on any standard yet in regards of this.


Afaik fastboot affects overall speed including read.
On Aug 28, 2015 10:51 PM, "Maxime Poulain" <notifications@github.com> wrote:

> @temasek <https://github.com/temasek>
> Just confirming that now having system with f2fs as file system does work.
>
> Btw @arter97 <https://github.com/arter97>
> What option(s) would you set for the system partition? There is now a
> fastboot mount option.
>
> As system is read-only is there any flag you think could be interesting ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_bootable_recovery/commit/92a8ab732340c2ac4f04f86f57ceb04b1ee08fb9#commitcomment-12948270>
> .
>


Hello Soby,

Cluster switch is not a supported use case at the moment. Whenever we plan
to support it, we will need some more changes. Until then we want to use
only the A57 cluster.

On Fri, Aug 28, 2015, 2:43 PM soby-mathew <notifications@github.com> wrote:

> @vwadekar <https://github.com/vwadekar>,
> Hi Varun,
> Hope all your use cases are working as intended with this code. Especially
> the cluster switch on T210, since the cpu context changes every time a
> cluster is switched. I will wait for your confirmation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vwadekar/arm-trusted-firmware/commit/d6e77ed3f710930f579a8fccb27707c8017063f1#commitcomment-12943632>
> .
>


Hm in a harm or good way?


http://bedroomproducersblog.com/2013/03/16/easy-midi-free-virtual-midi-keyboard/

Use OSX's "audio midi setup" to create a MIDI bus:
https://sites.google.com/site/mfalab/mac-stuff/how-to-use-the-iac-driver and
point the virtual keyboard's output at it. Moodler can now read from the
virtual keyboard, although the virtual keyboard needs to have focus so you
can't twiddle knobs while holding a key down.

On Thu, Aug 27, 2015 at 10:35 PM Dan Piponi <dpiponi@gmail.com> wrote:

> Hmmm...that's a hard problem to get. It means you've convinced the ui a UI
> element exists when it doesn't. What "scene" did you start with? A blank
> one? (aka saves/startup.hs)
>
>
> On Thu, Aug 27, 2015 at 10:00 PM barnabyrobson <notifications@github.com>
> wrote:
>
>> "connect id9.result -> out.left"
>> moodler: failed to find id9 in a out
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dpiponi/Moodler/commit/5974fb0265cb0d4a7ee46127e7c18d72ea7ee9ce#commitcomment-12940821>
>> .
>>
>


Also, an example usage in the readme could be a good fit


Should we actually be doing asserts like this in `lib/` files?


Hmmm, I am really not that much of an expert in all of this but after having gone through RFC 5246 section 6.3 I am now falling back again to agreeing with your code changes regarding the computation of the key material.
[F.1.4](https://tools.ietf.org/html/rfc5246#appendix-F.1.4) states "When a connection is established by resuming a session, new ClientHello.random and ServerHello.random values are hashed with the session's master_secret."
The new random values are exchanged by means of CLIENT_HELLO and SERVER_HELLO but since no further key exchange messages are sent, it makes sense to assume that no new *master secret* is calculated but instead the original *master secret* is expanded ("hashed with the random values") as described in [6.3](https://tools.ietf.org/html/rfc5246#section-6.3).


Perfect.


Congrats!
No announcement anywhere?


Duh, fastboot can't possible increase IO speed :P
On Aug 28, 2015 10:53 PM, "Maxime Poulain" <notifications@github.com> wrote:

> Hm in a harm or good way?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_bootable_recovery/commit/92a8ab732340c2ac4f04f86f57ceb04b1ee08fb9#commitcomment-12948309>
> .
>


@sosier
Ok with vim and Atom!


@MrKepzie
> Matthieu, as for the studios you mention, they can still make customisation of the software via Python (that’s what most studios do) since loading Python scripts in a GPL software does not make them GPL. If they want to make a complete derivative work of the source code, they would have then either to:
> A) Make it GPL so everyone can benefit of it
> B) Buy to us a commercial license where we allow them to do so

If the derivative work is an internal product, then it does not even require a licence! Lots of huge companies are using great GPL software libraries internally. When your company doesn't sell software, but sells stuff made using a sofware (movies, web services) which don't require distributing software, it is perfectly legal.


@sosier
Great job!


I agree. I'll change it momentarily.


If client by mistake would catch this exception like this:

```c++
try {
...
}
catch(CImgException ex)
{
...
}
```

it will lead to double free of allocated memory for '_message' (because of shallow copy).
This commit implement deep copy of CImgException. *If it is unnecessary, than better to make constructor of copy make private.*


I think you're probably the author of this file.


He posted on a commit
![How dare he](https://38.media.tumblr.com/b60b0c86ac6b177f441891934f3454b9/tumblr_inline_mqs2pnzCrn1qz4rgp.gif)


Fix Readme.md.

**npm** can't find `shm-buffer`:

`npm install shm-buffer`
   to
`npm install node-shm-buffer`


Good :)


try! will throw a runtime error if it fails and I feel like there is a chance that this could fail. Is that true?


Maybe it's fine if we were doing it before...


you can put 16384 here and avoid the next line.
see man strncpy: the resulting string is guaranteed to be nul-terminated


@gvanem Please provide a patch for Windows and we'll merge it.


Any reason this is an anonymous function rather than a member of FileController? The pattern we typically use is to put private functions as members starting with an underscore. This means they can be tested, and also show up named in stack traces, etc. Might need to set module.exports = FileController = ... to be able to then call it from other methods.


Do we need this else clause here?


nit: ident


I guess this need to be tested in an experimental way on the device itself.

Since fastboot seems to be affecting only during the mount process with a potential cost in the performances as you said.

Am I right you think ?


> you can't use relative paths here for file references.

Why not? From what I've seen, it appears as though the testcase can access relative paths.

I can add the files in dynamically as you suggest, but if the declared dependency can't compile (`*.pom`, `*.jar`, `*.pom.sha1`), the simulated build will still fail. (Though, I don't believe that this makes the tests fail.)


did you smell the tasty grilled bananas: ![Image](https://farm3.staticflickr.com/2876/9681688504_b5e8689026_z.jpg)


I did


I tried out CPU suspend use cases and the code you suggested dies not work. I had to change the logic!


I wonder if this should take a `File` rather than a `String` then it can be up to the client to manipulate paths etc.


This may be a solution to my current problem, though. The testcases can't seem to resolve `.pom` files, but can resolve the `.jar` files.


Yes.
But in real honesty, fastboot option is not for us. We can wait a few
tenths of seconds on the initial boot-logo.
fastboot is for use cases where mounting speed is critically important.

On Fri, Aug 28, 2015 at 11:00 PM, Maxime Poulain <notifications@github.com>
wrote:

> I guess this need to be tested in an experimental way on the device itself.
>
> Since fastboot seems to be affecting only during the mount process with a
> potential cost in the performances as you said.
>
> Am I right you think ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_bootable_recovery/commit/92a8ab732340c2ac4f04f86f57ceb04b1ee08fb9#commitcomment-12948462>
> .
>


@gpedro o rate limite + uma hora no caso?


@mstembera, Yes that makes sense. Its probably something to try out eventually depending on how this patch performs at LTC.
But I'm not sure if the gain is coming only from low depths. Even higher depth searches may benefit when they eventually search at sub-trees.
Surprisingly  increasing max_slaves_per_splitpiont at higher depths did not give noticable gain or loss at LTC. Although there is some gain at STC. I don't understand this(it should have had a big impact - for good or bad!)
http://tests.stockfishchess.org/tests/view/55de19a80ebc595da943398d
http://tests.stockfishchess.org/tests/view/55d983460ebc595da94338b2


using relative paths here relies on the assumption that your test vm is executed from the project folder. this works when using eclipse but you don't know how your build is configured and how the tests are invoked in other environments like CI, commandline, etc.


I sure hope I did this as a joke last year.


In theory, yes. I don't really know if their cmd does the same as cmder's so I left it in for now.


Why width 30em and not max-width 30em?


@adbuerger This example might be relevant for your PECAS project


okay


ok, fixing locally


Dear devernay,
Excuse me if I'm wrong, but [here](http://www.cplusplus.com/reference/cstring/strncpy/) I found:
> No null-character is implicitly appended at the end of destination if source is longer than num.


Isn't this string "Ref Updated" defined somewhere?
Can you find and reuse it?


Because I am an amateur :)


@renzms Please outdent this line so that it matches the indentation above and below this block.


I wondered what was going on there, but figured you had it under control :)


> using relative paths here relies on the assumption that your test vm is executed from the project folder. this works when using eclipse but you don't know how your build is configured and how the tests are invoked in other environments like CI, commandline, etc.

Good point. I had not considered this.

> keep the generated pom very simple and the jar does not need contain any logic at all. we just want to ensure it is downloaded correctly. we don't need to test the sha1 files I think.

I test the sha1 files only because Gradle was sending HEAD/GET requests for one at the time. I'll look more into it.


Thx!


não, agora + uma hora. Por que o RateLimit reseta a cada uma hora.


Soby, the loop that you suggested does not work for CPU suspend cases on our internal platform. That's the reason I changed the logic. Do you see any issues with it?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 440](https://teamcity.labkey.org/viewLog.html?buildId=268948&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:29


This `}` bracket is should be outdented once.

```php
if ($this->plugin->options['latest_pro_version'] && version_compare(VERSION, $this->plugin->options['latest_pro_version'], '<')) {
    echo '(<a href="'.esc_attr(add_query_arg(urlencode_deep(array('page' => GLOBAL_NS.'-pro-updater')), self_admin_url('/admin.php'))).'" style="font-weight:bold;">'.__('update available', SLUG_TD).'</a>)'."\n";
} else {
    echo '(<a href="'.esc_attr('https://zencache.com/changelog/').'" target="_blank">'.__('changelog', SLUG_TD).'</a>)'."\n";
}
```


Now I get it ;)


I think this commit should be reverted.
RFC 7540 says that client cannot send PUSH_PROMISE and any attempt to enable push in server initiated SETTINGS is connection error.  Which means effectively server disables reception of push.

```
   A client cannot push.  Thus, servers MUST treat the receipt of a
   PUSH_PROMISE frame as a connection error (Section 5.4.1) of type
   PROTOCOL_ERROR.  Clients MUST reject any attempt to change the
   SETTINGS_ENABLE_PUSH setting to a value other than 0 by treating the
   message as a connection error (Section 5.4.1) of type PROTOCOL_ERROR.
```

So, making enable_push to 1 is even more confusing, assuming "1" means enabling push.


Now one can override `prettyPrintDefault` instead. I wonder at this point whether all customization should go through such a method — this would enable some coverage checking (by moving the default to some extra mixin), but that might be a lot of busywork, so leaving this as-is for now.


I've decided that's no obstacle to merging.


@Khaon

Yup tested /system with f2fs and works as intended with the cm's flash script. Now all my system/data/cache are all F2FS though my /system as f2fs is for testing purpose.


this API is not quite foolproof:
- if content is set but location is null, content will be fetched by location
- if location is set but content is null, the opposite will happen

but what if those 2 are set and do not match each other?


@RIAEvangelist See [v0.31.0 release notes](https://github.com/atom/electron/releases/tag/v0.31.0)


:+1 To highlight the guardian using the overuse of service


@renzms Please fix the indentation here:

```css
.plugin-menu-page-version {
    float: right;
    clear: right;
    min-width: 350px;
    margin: 0.5em 0 0 0;
    text-align: center;
}
```


You're right, thanks. I'll fix that and merge, I'm counting on these issues being the last ones.


Yes, that is unfortunate. But the problem is actually tricky. For example, if we do something like

```swift
let filteredObervable = Observable<String>("foo”).filter { $0.length == 0 }
```

We end up with the observable without a value, so optional is necessary at the moment because it allows users to check whether the value exists. If our type is also optional, like in your example, value ends up being double optional and we should do

if let optionalValue = s.value, value = optionalValue {
 ...
}

to be on to safe side.

I understand this is pain and I’m investigation other options.


@jgillis indeed it is! I will implement in PECas, and see where I get with it. Can you tell me what system this model is representing? I don't get it from the code alone.


Did you change the actual file name or do you just call it .png?


Nie mam bladego pojecia dlaczego to działa, hłe hłe hłe :D


Hehe Did you notice that cyanogen has updated their repo few days after we tracked our own ?

Can't be a coincidence can it be? :P


:heart:


Reviewed-by:


Нет. Да и лень мне прошивать заново. Я уже откатился на сток.


Changed in 6f4383e9b0b2ca84fe24b7370882b423a28d6c1b


References #5


Do you want to create a pull request about this? If that's what you want to do, please include a test that demonstrates why this is needed for iframes.


so you can pass null in api too?


wrapper should be as more human-friendly as possible


Doesn't affect me but just spotted this. Does it need to be a global? Guess there could be other libraries that look for a `DEBUG` variable and this would be overwriting it. :)


How is it submitted now?


Aren't those files skipped anyway?


LOL :D


p comes after f


Thank you


Fun.


why a separate method for this?


@tomtor, I didn't quite understand, I want to split at higher depths also.


Yes
```
        if (readTimeout != null) {
            clientConfig.property(ClientProperties.READ_TIMEOUT, readTimeout);
        }

        if (connectTimeout != null) {
            clientConfig.property(ClientProperties.CONNECT_TIMEOUT, connectTimeout);
        }
```


`or f.mimetype or 'application/octet-stream'` = why?


tenants should be tenets


invisiblek?

On Fri, Aug 28, 2015 at 11:19 PM, Janson Kang <notifications@github.com>
wrote:

> LOL :D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android/commit/9e32b04482c293e1b148784d38a5cc3aefdc70b4#commitcomment-12948834>
> .
>


don't try to seek that one


ehh, yeah I know, sorry, didn't sort it


It very simple, constructor + getters, see updated variant in https://github.com/jenkinsci/docker-plugin/pull/308/


This might help you in the future with merge requests and what not:
http://pcottle.github.io/learnGitBranching/


Please measure the time by using a number of iterations instead of one shot.


Missing `self.sync_all()`.


Merged.

Thanks.


My mistake was trying to merge it using Smartgit's conflicr solver :P


Hey, sorry for not getting back to you sooner!

It's not going to fix the issue completely. Because a few steps further on in the process, in

sphinxcontrib-images/sphinxcontrib_images_lightbox2/lightbox2.py

this happens:

        if node['uri'] in builder.images:
            node['uri'] = os.path.join(builder.imgpath,
                                       builder.images[node['uri']])

os.path.join will use the system's slashes (on Windows that's '\') to join the paths. I sent you a pull request with the change that  made this work on my machine.

Cheers,
Sebastian


Our config-wrappers are builders and they are all not Serializable, before 2.1.0 we had ClientConfig that contains this value that was sent to slave. Now we need Serializable object that will have DockerCmdExecFactoryImpl values. And DockerCmdExecConfig shadow similiraty to ClientConfig as in https://github.com/docker-java/docker-java/wiki#intialize-docker-client-advanced


Use camelcase


The command line is always your best friend :P


this is wrong, empty is boolean, so it should be ```if (FALSE === empty($this->tsConfig))```


I'd like to see the deletion of this commented-out old view code rebased into b5893c, personally, rather than in here. It was effectively deleted there, after all.


*Removed


Hi,

My application worked well after I turned back line 32 in BaseSampleFragment.java file from :
mMapView = new MapView(inflater.getContext(), mResourceProxy);
to :
mMapView = new MapView(inflater.getContext(), 256, mResourceProxy);
I also did that same change at line 82 in MapFragment.java and in every file of the sample directory

Android Studio 1.3.2
java 1.8
SDK API 23 (Android 6.0)
Build Tools version 23.0.0
Targetting SDK 19 (Android 4.4)

Florent


It's called a Silverbox system; basically a mass-sping-damper system with an additional nonlinear term.
I used it for benchmarking and to test newly added features. You may see large speedups when adopting paradigms from this example.


tsk tsk


Some guys had pushed a commit updating the repo to the upstream revision one or two days after this commit.

Then invisiblek seems to have updated their repo


ok


@Khaon

You have direct push access to this repo....!


wa哦


I have mixed feelings about this change as well.  Currently I'm not aware of any front end jobs where we don't do a build (the purpose of this step would be to generate build artifacts anyway, otherwise we can just clone the repo on the server), so I don't think Marc's point is valid.  However, I am in favor of having reasonable defaults and this seems like one of them.  I'll try not to lose sleep over this one, since I think the font-end side is fairly unified on this standard (or at least is headed that way), but there may be other cases where we should favor defaulting versus failure.


You probably forgot to update the supporting tests.


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


Oh I didn't know :D


invisiblek is more of an open guy.
He follows CM forks, custom kernels, etc. I'm already aware I'm under his
radar lol

On Fri, Aug 28, 2015 at 11:30 PM, Janson Kang <notifications@github.com>
wrote:

> @Khaon <https://github.com/Khaon>
>
> You have direct push access to this repo....!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android/commit/9e32b04482c293e1b148784d38a5cc3aefdc70b4#commitcomment-12949077>
> .
>


Here I'm preserving behavior from (I think) @fluffyemily's fix, but before that we just ignored the `&err` result, which wouldn't crash (and that's what I do in `deleteLibraryFolderContents` usage below). I'll switch to that here, too.


^ shh! ;-)


How about
```rust
if let Some(joined) = condition.split("\"").nth(1) {
    prefs.extend(joined.split(",").map(str::to_owned));
}
```


TeamCity pull requests :: netty [Build 139](https://garage.netty.io/teamcity/viewLog.html?buildId=885&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5583, ignored: 21; resource leak detected (new) Build time: 01:16:09


@Ooblioob This is my bad... I was looking at the change out of context and didn't expand the selection to see that it's part of a build-js block.


[dsf](htt;)


They (inpaint/segment) are currently broken in Linux, but do work in Windows.


Why do you actually need to keep the ActiveMQDestination object here?

So far ActiveMQDestination has been dead code. I was actually going to remove it. Why not just use the String here or whatever came from the protocol?

I'm trying to decrease the duplicate state between the server and the protocol manager.


XD


what was wrong with the previous code?


I was thinking that what I would really want to do is to have distinct edge types for distinct arities. Something using dependent types, like:
``` Elm
edges : forall (n,m:Int) . Set (Edge (n,m))
source : forall (n,m:Int) . Edge n m -> Vector n (Vertex)
target : forall (n,m:Int) . Edge n m -> Vector m (Vertex)
```
And that may be useful in order to make the compiler typecheck the arities for us. Anyway, dependent types are not available in Elm.


Typo, commit message should be "Changed name of query for checking pink notes"


@Will-So
For terminal editor, you will need to know one of the terminal editors.  For example, in cloud computing and other environments.


Do you have a recommendation?  The one I selected is as close as I could find.


Wenn man das Spiel selber nicht mehr an macht läuft man da in Fallen rein. Die {1} wird mit der Kategorie ersetz: Zutaten, Waffen, Rüstung usw.
Zahlen kommen da gar nicht vor.


@leomelzer The advantage of sending the JSON is only to have less overhead for large lists (buttons etc.). Another point is that all data is managed by javascript. This could make the step towards loading the data dynamically easier for us. But creating the list via twig/php would also be fine, of course.


Hehe,

I was just kidding, glad that CM updated that.


@alepore generally this one is always called with a number interpolation, did you find any instance where it doesn't?


@Will-So
Great job!


Does it make sense for these to be Color3f plugs?


Single quotes should be used here instead if no string interpolation was used.


@ctrueden example please? I don't see where to do that in the pom.xml.


The caveat should be removed. It should only be used for things pertaining to the installation itself, and not generic messages, as we’re not a discoverability service.


Ruby 1.8 doesn't allow a splat to come in the middle of an argument list, only at the end; can you move this? Thanks!


Microsoft Edge Dev


Can we get this backported to a Doctrine ORM 2.4.x?


Ready. 5 attempts for every n-symbols variant. Hope, it's will be enough to detect free value. Specs were changed according to the model.


This is a good start, but we can do so much more with this.  Clearly this is our first step in that direction, but I'd love to see us be able to call it more like this:
```
job('example') {
    description('some description')
    ...
    multiscm {
        project_repos(['collab', 'cfpb', 'etc'])
        git {
            // Some other utility repo, like jenkins-automation
        }
    }
    steps {
        ...
     }
}
```
By allowing us to inject in the repos associated with the project, we'd be saving teams a lot of copy/paste time and making it more maintainable.  Also, it clearly separates project repos from other utility repos we may need to include.


Altere


:derp: is not an emoji, but should be


@rvagg yes, should be good, I can't foresee any problem.


This is great, the syntax is very clear.  I was hoping to avoid anything that felt magical, this seems like it will work nicely.


@ctreuden here the first  the maths change


DOUBLE-DERP


@ctrueden and here is the second.


Sollte dennoch passen, oder?


is it safer to use express like this . why cant we use express package directly here .i am just curious


I missed this issue, but it happened to be fixed in a subsequent commit, https://github.com/MediaArea/ebml-specification/commit/a07a6993ff3c8ee2f4d5e53abed3a27d58c4fed1. Here I was trying to clarify that the length is about the declared length and not the length of the whole element.


Use math.hypot?


:facepalm: You know, I never thought to double-check this and it always worked.  Seems backwards to me but I can't fight the tide.


@ctrueden same maths change here


aaaaand this is why semicolons are evil


@marcgascon Do you think think this link to my blog post would be helpful or would you find it useful to have a shorter (but possibly less readable) version here in the README?


@ctrueden and here again the same maths change.


You don't anthropomorphize your programming constructs?


This method should imho be rewritten to just be a loop around buildeZObjectWrapper() calls.
Also, it does more than its single-item cousin, as it validates that the location is main...


Easy change!


Should be `\a amount`


I'm merging this commit for master.


This is indeed old data in the shared connection, but it is new data for 'rp2'. What is the correct return value ?


Pierre, the report of report_logging.php works for you? After recent changes this report again doesn't work for me as it was early.

Very thanks for the support.


Just scanning through the sources- isn't that last bit here a possible division by zero as well?


Ein Pflanze gibt nur eine Pilgerkrone ab, da passt „mehrere Gegenstände“ überhaupt nicht, das was wir dann daraus gemacht haben.


This the best mod I have ever keep up the great work. I waited for this mod to be released for a year.


Call it `excess_mana` please.


Humm. AFAIK contour means a closed spline.  You mean spline here, which in curve speak means a sequence of continuous curves.


It's not specifically needed, just for some convenience, like method isQueue(). I'm fine if you can get rid of it.


You might as well make these all be methods on Point().


Mhm


@howardgao  please, don't recreate broker functions on the ProtocolManager. This function already exists on ActiveMQServer::destroyQueue(queueName, serverSession, checkConsumerCount, removeConsumers).

you either pass ServerSession as null if you don't or we refactor the functionality to fit what you need.


pls look at the protocolManager.removeDestination(), it needs to pass a Destination rather than a string.


![sherlock holmes](https://cloud.githubusercontent.com/assets/13589718/9549104/422f75fe-4d72-11e5-98b3-629f8b68788f.jpg)


dude, that is `BSD` :D


I'd be surprised that valgrind runs did not catch this the past years... Or cid became zero somehow or the ConnectionManager is not deleting the cid of ConnInputEndpoints and only of ConnOutputEndpoints ?


Yup! Nice work :)


Pre-Alpha (Android): http://d-h.st/Bh0t (Webversion kommt noch)


@athal7 and @jhawthorn this should read: `[">= 1.0.0.pre", "< 1.2.0"]`, correct? Since there isn't a 1.1 branch for solidus yet?


That means we can't build the HTML page without CSS file right?


ok


You probably need to refactor this method to call Server.destroyQueue and only do this part here as the fireAdvisory is something that really belongs to the protocolManager.


* Grid size unit converter,  use a dropdown list
* Explain analysis area
* Explain IC vs Q figure on two channels


Looks good!


hello,
shouldn't we use dynamic_cast instead of static_cast here to perform a run-time check that the pointer is a derived base of Cobyla ?
Saw this thoughout the new code.

j

http://en.cppreference.com/w/cpp/language/static_cast
http://en.cppreference.com/w/cpp/language/dynamic_cast


Initial commit


Will IModelMapper and IQueryExpressionExpander also be part of hook handlers?


Good point.


Yeah.. but that's after your change only. I actually commented on that method.

you are recreating a lot of functionality from the SErver already on the protocolManager.


Hello. Do you know when this commit is going into a npm version? This fix helped me handling a rare  "Maximum number of allowed connections reached for admin" error that I get from a SSH server.


Why are you changing the mimifier to -compress for Window users?
I have the lastest LESS and less-plugin-clean-css installed on Node.js and I get this error...

"The compress option has been deprecated. We recommend you use a dedicated css minifier, for instance see less-plugin-clean-css."


Thanks. Looks good


closes #119


In Theano, we do not specify anything special for debug, non-debug build and others assert have triggered. So by default, that check will be done. I'm ok to have this enabled in debug build, but currently it will be enabled by default.

Do you have a suggestion to make an easy debug build? If it take more then 15m, I think you can just put this in comment, to let know that this is expected.


Smart!


https://secure.php.net/manual/en/function.mime-content-type.php ?


That means every time I have to use a unique class name. Can I repeat my class name?


lgtm


Deprecated function - and we've noticed that PHP installations don't reliably have access to FileInfo. Good times ...


debug trace?


@mjwitte Please check equations under Proportional Control in Eng. Ref to understand how this control works.


How does this improve coverage? It could be reduced to this:

```jl
for T in (Int8, Int16, Int32)
    @eval promote_rule(::Type{UInt32}, ::Type{$T}) = $(WORD_SIZE == 64 ? Int : UInt)
end
```


This file requires reformatting, it's still have line endings issue


@clintonfernandes :+1:


Apologies I must have screwed up testing in windows.


```js
if (!params.length)
```


change `process` to `method` to be compatible with C14N W3C Recommendation naming


Oh Great! I didn't see it when I post my last comment on the PR but looks great! I will follow it to do my set up and if I find any improvements to the gem I will continue contributing! thanks for updating the doc! :)


couple of comments/questions:

1. non-blocking: but typically you send the button (i.e. sender) as the argument to an action, not the sender's owner. can you pass `sender` instead to follow convention?

2. also non-blocking: might be simpler to use target/action here if all we're doing is invoking a method. that said, using a `UIButton` inside a `UIBarButtonItem` raises the question of which target/action pair to set... I'm guessing on the `UIButton`.


@theuni how can I ignore this in systems without zmq?


@wilsonmichaelc nice! One recommendation is for at least the 'default' snippets for various model types to stick to Cl and V parameterizations rather than ke/V (which is rarely used)

@vjd do we want the defaults to have the logtransformed mu-model-esque parameterizations. We'd also need to dump out the actually param values as secondary parameters and initial estimates would be a bit wonky?

I would suggest keeping the `stparm` structure to:

```
stparm(cl = tvcl*exp(nCl), ....
```

type parameterizations

lastly - I know it is nitpicky, but I would say 30% CV on random effects and 20-30% on residual error would be a more 'standard' default rather than  50% - I would be concerned with sparser models or certain edge cases that it would default to too much into residual error and place less emphasis on the model actually perturbing the fixed/random effects during optimization.


ConventionalEntitySetProvider -> Convention collection or resource provider


Ahh... hadn't spotted the deprecated notice.

Maybe implementing one of the shims from the comments?


Line endings...


отлично!


класс `.active` можем на `<li>` повесить, а спан вообще убрать)


Yeah, probably the best bet there is the mime_content_type shim that just carries a giant array of MIME types. It's not great, but maybe there's something useful on GitHub. And if there isn't, maybe that's something we can provide.


Thanks for reminding me. I have modified it.


отлично, можно заодно было бы отрефакторить (переименовать) классы


ссылку надо было убрать только у `.active`


Document this, should be noted that this enabled `C14N` and thus ignores `pretty_print` and `encoding` arguments.


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

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

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

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

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

```


Отлично!


@lionel-  thank you for checking.  Honestly, I did not think that your indentation changes were related to this problem.

I'm happy to envision it being handled after releasing 15.09.


Thanks.


The first argument can be either a `string` or an `array`.


как вариант можем `div.logo > a.logo-image` переделать в `a.logo`


I did indeed, but fixed them already due to a mail I got from travis CI. :+1:


maybe a topic for our eng/code review meeting today. do we want a standard "save button" control which automatically toggles and updates itself for a given title's saved status?  e.g.

``` swift
class WMFSaveButton : UIButton {
  var savedPages: MWKSavedPages
  var title: MWKTitle
}
```

this would make it easier (and DRY-er) to put save buttons wherever needed and ensure they always update correctly. However, this all of them need to be saving/mutating the same list, which might be desirable (until we move to CoreData) and safe (since all UIButton actions are on the main thread).


@burnsjeremy these methods have been added to the latest version of DrupalDriver. Can you check that you are running the latest version?

Since Behat Drupal Extension depends on DrupalDriver and new features often need changes in both projects, both are always getting new releases on the same day.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.934.a592fa97b49c2d68e8beb7bf1606618b6007394d](http://build.palaso.org/viewLog.html?buildId=62161&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:03:17


This method provides more values in the dict than what was previously here. Is that OK?

For example, topicID and teamsDetailUrl.

The helper method is being used for 3 different views, which is confusing me.


do you need any help with that?


Question: should this check maybe go in referrerGatekeeper itself? We're unlikely to ever get referrers on API calls.


I only found it by accident -- I tried it without checking the docs first and it worked. It's not documented (https://docs.python.org/2/library/imp.html) but it *is* deliberately exported (https://hg.python.org/cpython/file/15c95b7d81dc/Python/import.c#l3269).


I pushed a new commit 88114850db211ee3d6d43e739df40ffca7882fb8 which manages this 2 points, but discussion is still opened :)


Eh. I'll remove it later. Only need the second one.


@burnsjeremy I just checked and the method is indeed implemented for Drupal 7 in the latest version of DrupalDriver, ref https://github.com/jhedstrom/DrupalDriver/blob/master/src/Drupal/Driver/Cores/Drupal7.php#L467


Thanks Marius!

Nice work on the mesh!  I am going to try to get your binaries built under OS X with gcc if I can this weekend.  I was hoping there would be makefiles, but I am happy you open sourced the code, so I am not comPlanning :). Unfortunately, I am not that familiar with eclipse.  I have all the nrf51 SDK 9 examples running under the gcc tool chain on OS X.  I would assume the nrf51822 beacon pca20006 would work, but my initial attempt was not successful.  I'll give it another shot after I figure out how to actually build the fruity mesh binary from scratch.

If I can get all that stuff set up, I will use fruity mesh as a basis for an internal technology workshop in October with ThoughtWorks.  Thanks for all the great work.  Any pointers you might have in addition to what is on the wiki for getting the build environment set up would be appreciated.  If I can get make files working with SDK 9 I will certainly send you a pull request...

Tim

> On Aug 28, 2015, at 5:19 AM, Marius Heil <notifications@github.com> wrote:
>
> Hello, I've moved your comment to the issues and marked it as a question so it will be available for everyone to read: #11
>
> —
> Reply to this email directly or view it on GitHub.
>


なるほど、判断しにくいためimage_urlという名前に変更したのです。
しかし、image_urlというカラム名は適切ではありませんね。urlは`ウェブ ページなどを一意に指定するための規約`ことを意味します。
`image_name`にするといいと思います。


My thinking was that this was really global context that was being passed around, so it was easier to keep it consistent if it is constructed in one place. Some of the values are optional because they don't always apply (you may not have a topic in your current context) but that doesn't matter because the code won't look for them.


Travis can't find 21eb5b79072b147d0a9b7fafca98501e7056c834.patch


@liancheng
This is a repeated text?


:smile:


Switch variable to camelCase !!


Fixed, thanks

On Fri, Aug 28, 2015 at 5:08 PM, Frederik Rietdijk <notifications@github.com
> wrote:

> Travis can't find 21eb5b79072b147d0a9b7fafca98501e7056c834.patch
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/01587f02a2eee51385145d361c77d12925e4ebbb#commitcomment-12949917>
> .
>


текст вынеси в strings.xml


Sort imports


> されなくて今困ってます！


fixed #2


Here's a first and insufficient stab at making our own binding construct `lam`:
```maple
app := proc (rator, rand)
  if rator :: lam(name, anything) then
    eval(op(2,rator), op(1,rator)=rand)
  else
    'procname(_passed)'
  end if
end proc;
```
The good:
```maple
> app(app(lam(x,x),lam(y,y)),lam(x,x));
lam(x, x)
```
The bad:
```maple
> app(app(lam(x,lam(x,x)),1),2);
app(lam(1, 1), 2)
```
The problem is of course that we need to teach Maple `eval` about the binding structure of our `lam`. Jacques, would you mind showing how to do that, perhaps by taking care of the FIXME at the top of `NewSLO.mpl` currently?


The first argument here can be any of the following (they all remove the `build` folder):
* `'build'`
* `'build/'`
* `'build/**'`

===

The following remove everything under `build` directory but not `build` folder itself:
* `'build/*'`
* `'build/**/*'`

===

The following remove all the subfolders but not the `build` folder and non-directory files directly under `build`
* `'build/*/'`
* `'build/**/*/'`


Agreed


Sorry about this silly commit everyone


分かりました。


Change:

```
} if (newPos.bottom >= snapRanges.bottom.min && newPos.bottom <= newPos.bottom.max ) {  ```
```

to:

```
 } if (newPos.bottom >= snapRanges.bottom.min && newPos.bottom <= snapRanges.bottom.max ) {
```

;)

But still north/south does not feel right. I've forgotten to update some of the theme files it seems, so the top margin is 30 by default now (or should be). I'll commit that


I don't think the pybombs way actually works, so I will have to fix that first before merging this. Thanks, though!


Also, the first window you move goes crazy for a split second


ぼくの認識だと、UseはたくさんのPrototypeをもっていて、そのPrototypeがたくさんのThumbnailをもっているのだと思いますが、どう考えてこの記述をしましたか？


@guce what about the case where we want to create a user without generating a secret for it? Maybe check here if (gen_secret)?


Thanks for the explanation, that's a good reason.


huh?


Merging for the height/width part of this. The spacing will be removed by any IDE, and the textures are used if Terraria is unavailable.


<keygen> has been deprecated since it was invented. I don't think we're going to make it go away until we provide a better API.


なるほどです！確認しました。


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


Oh, nevermind, it was 29, which is close enough :P I'll commit the proper ones anyway.


I'd prefer to leave the topic values out because those relate to a particular team (and are not relevant in this view, for example). The others I can see as "course context" or something like that, as they are specific to any particular team--

courseID: this.courseID,
teamsUrl: this.teamsUrl,
teamsDetailUrl: this.teamsDetailUrl,
languages: this.languages,
countries: this.countries,

I find the name of the method very confusing because I thought it was supposed to be about a specific team, not Teams as a feature.

Can you think of a name that is more about settings/metadata for Teams as a feature?


:+1: on this commit


アソシエーションを利用してuser_idをもった形でPrototypeをつくってしまいましょう！


That `@deprecated` javadoc annotation is missing an matching`@java.lang.Deprecated` annotation, and the Error Prone compiler signals this as an error: http://errorprone.info/bugpattern/DepAnn


@guce what does this one solve? Not sure we want to force a swift key type in here, maybe it's the caller's duty?


I had no clue about why should we do it before I read #3975. Can we add some more information here and also document this new feature? @julen what is the right place for it?


"Chrome's V8 Virtual Machine?" We get a lot of new people on the site who don't know what v8 is.


typo in `wtihout`


same here


same typo


Not motor oil, should be olive oil


http://stackoverflow.com/questions/12900016/does-ofstream-close-its-files-automatically
http://stackoverflow.com/questions/748014/do-i-need-to-manually-close-an-ifstream
Требуется руками закрывать fstream только если требуется проверить failbit ну или если планируется переоткрыть другой файл.


I think the current logic is simpler and more efficient, your suggestion makes it compact but adds one more level of check (for the right set among the two), then there's array seek and string manipulation.
An advantage that we have here is, if the default is used (means no preference or .com), google redirects automatically to the country-specific server. The feature is convenient for people who are present in a countr other than their native and want to check out the local google.


+1, I just pulled the HTML out of the old site. We should use the JS button instead.


This variable is unused.


`different` -> `canonicalized`


it was put here for convenience while we make modifications to this page, i have no objection to moving it wherever people would like to see it go.


private set


:+1: on this commit


An alternative is
`expect (view.sles.upper_bound(uint256(2)) == b); ++b;`
`expect (view.sles.upper_bound(uint256(3)) == b); ++b;`
`expect (view.sles.upper_bound(uint256(4)) == b); ++b;`
...


proper name for this class


Haha ooops how did that happen!


This `return` is __essential__ (http://schickling.me/synchronous-tasks-gulp/ https://github.com/gulpjs/gulp/blob/master/docs/recipes/running-tasks-in-series.md). This `return` is to tell gulp that this task is finished (return the stream as the completion hint).

We need to run the `build` related tasks in sequence (avoid the async nature of gulp to run all tasks in parallel).

For the `build:remove` task, we can simply use the `callback` argument, but task `build:copy` is a `gulp.src()` and `gulp.dest` operations manipulating __streams__. In this case, the following __wouldn't__ work:
```JavaScript
gulp.task('build:copy', ['build:remove'], function(cb) {
    gulp.src('app/**/*')
        .pipe(gulp.dest('build/'));
    cb();
});
```

We've gotta to use `return` to handle `stream` tasks here so that the task `build:clear` which depends on it will wait for the completion of task `build:copy`.


Т.к. это описание к классу, то "Class contains" можно опустить.


Nice!


Yippee?


I updated the quac


A ? proper name for this method


Why does this fix the issue?


It is better to specify localhost as other devs will have their own instances and machine names.


proper name for this method


I recommended to @iarna that only the `PATH` and `CWD`, which are the most useful bits of information for end-user troubleshooting, be included at `verbose`. Since both `verbose` and `silly`-level output are included in `npm-debug.log`, on crashes you'll still get both, and this is really just the difference between `-dd` and `-ddd`. Since the `cmd` is the stanza from `package.json`, how often are you going to need to see that?


Так бы и написал "Remove unused import." А то понятие рефакторинга очень широкое и удаление неиспользуемого иморта в канонической книжке Фаулера, скорее всего, нет.


This `cb` argument can be thought of as those tasks who depend on this task.

`del('build', cb)` tells those tasks to wait for the completion of deletion.


boooooo!!


:rainbow: :rainbow:  yay! @sethvargo the vault-ruby PR is coming :)


Instead of that you should override ToString for class Employee. And then you can simply Console.WriteLine(k);


ce serait plus pratique `python` plutôt que `python2`, d'autant que dans les exos France-IOI, *Python* désigne du Python3


What about when you edit from some requirements to no requirements? Should the resolve be run to change the bundles? Otherwise the state is unchanged.


Can this be set separately?


Sorry I answered in the main comments. I'll copy my answer here:
Always in the sunrise-design, as it is the source of truth. The current implementation probably doesn't have it because it is using an old version of sunrise-design.


pdb set_traces are not something I'd like to have in the history just in case someone checks out this commit and runs the tests. It's not a big deal and I'd merge this PR even with it but I do like my history clean.


RIP huehuebrbr


If there is a change from some requirements to no requirements, shouldn't you be able to run resolve?


My commit on site


Ah yes. Good point.


Same as for method S()


Because of the user-declared destructor, `msg` needs user-declared copy members (either defaulted or deleted, whichever makes sense).


aaa


proper name for this method


bbb


Первая строчка слишком очевидная, а вторая говорит нечто, что бесполезно -- все равно такого файла уже нет.

Надо пытаться передать смысл -- а смысл тут, насколько могу судить, в  что это конфигурация приложения.


Don't do this.

First off it will delete the research, again, for anyone who re-researched it.

Secondly, Thatumcraft explicitly looks for researches that begin with "ROD_" to build the wand recipies!

That is why I changed it in the first place.

Please revert this change.


Dans taskgrader/schema_input.json les options sont 'python2' ou 'python3'.  Est-ce que je change pour 'python3'?

> "language": {"type": "string", "description": "Language of the program. Possible values: 'cpp', 'cpp11', 'python', 'python2', 'python3', 'ocaml', 'javascool', 'c', 'java', 'pascal', 'shell'."},


For the body entry I need help with another regular expression matching quickblog db entries.
`$quickblogFilePath = $serendipity['serendipityPath'] . $serendipity['uploadHTTPPath'] . $oldDirFile;`
Plugin isp quickblogs old entry pattern is `<!--quickblog:TheFullPath-->` and quickblogs new entry patterns are either `<!--quickblog:(|[none|plugin|js|_blank]|)TheFullPath-->` merged into this line `WHERE body     REGEXP '(src=|href=|window.open.)(\'|\")(HERE_ALL_THE_OTHERS" . serendipity_db_escape_String($quickblogFilePath) . ")'`
Any ideas? @garvinhicking


Please revert this.


By logic, Search should return relevant Employes, And then , outside the method, you can Show them by relevant method.


I can do that.


Review by @nicolasstucki for pushing on scalajs/master.


The attribute should be called `without_comments`.


The thing is, this class is only useful for DiscoveryProvider implementations which are synchronous and thus don't adhere to the contract.  The inevitable result is that such violating implementations will end up in some discovery chain causing hangs and thread pool problems.  I'd rather see a separate interface for SynchronousDiscoveryProviders whose discover method is interruptible by contract, and returns void.


You can reverse the whole condition so it's more concise:

    if not kw.get('without_comments'):
        d['#comments'] = t.text


hmm oui effectivement le grader est plus précis... autant utiliser 'python3' oui


I'm thinking that this is lock-in to github. We should discuss before using this.


Should not need setConcealed ()


I think you're wrong. If fast_handshake_state_free doesn't deal well with a NULL argument, this test will explode. I do not necessarily think the assertion is necessary though.


I'll give backwards compatibility a quick try.


:+1: on this commit


When we use `gulp.src()` and `gulp.dest()` to copy files, we don't have to specify files in glob patterns (we can just use folder patterns).

The following glob patterns work here:
* `'app/**'`
* `'app/**/'`
* `'app/**/*'`
* `'app/**/*/'`

The following glob patterns don't work properly:
* `'app'`
* `'app/'`
* `'app/*'`
* `'app/*/'`


It would be nice *not* to do this, but the "pack_forget" line in create_plot(), which removes the toolbar, doesn't work in (some?) other backends


3) ,4) not done.


ok, made todo item for it :D


Not new with this P/R, but I find it hard to believe that the current copy members of `queue_type` are correct.  If they are, `= default` them, else `= delete` them.


I rather this be a ```Set``` rather than an ```ArrayList``` so the look up time is O(1) rather than O(n). Also, it should be annotated with ```@Nullable```


Good one.


The global negated pattern in this `del` function has to be wrapped with `()`. Don't know why yet.

In `gulp.src()`, we are able to do this:
```JavaScript
gulp.src(['app/js/**/*.js', '!app/js/**/*.min.js'])
```


What is the `file` and `db` type?


Basic Site for my portfolio.


150v 로바꿔줭..ㅠㅠㅠ


This include does not come from any specified dependency but from a rosidl_typesupport specific package which is not available to rclcpp.


If you look at line 116 in the file you will notice the same thing.


The main reason I did this is because Chrome is actively trying and Firefix is about to.


Какой-то интересный у тебя рефакторинг :)

1) коммит должен был быть атомарным, а это значит что удаление файла `mvc-dispatcher-servlet.xml` тоже должно было быть частью этого коммита. Впрочем, возможно, что ты просто забыл это сделать сразу.

2) в оригинальном файле у тебя был класс `InternalResourceViewResolver`, а тут стал `UrlBasedViewResolver`. Это почему так? Я верю, что оно работает, но интересно чем ты руководствовался когда это изменял? Оно зачем-то же понадобилось?

3) Если не ошибаюсь, там `JstlView` это значение по умолчанию и поэтому `setViewClass()` можно было и не вызывать.

4) Желательно возвращать из метода не имплементацию, а интерфейс. Т.е. метод должен возвращать не конкретный тип `UrlBasedViewResolver`, а интерфейс `ViewResolver`. В будущем меньше кода придется менять, если решишь использовать другой `ViewResolver`.


I though only Natron+plugins was going GPL. Shouldn't OpenFX related stuff be more liberal?


@chneukirchen: rdumpfs 0.1 is in Gentoo now!


@abnero DWMR. Press on! Move as quickly as you can - sharpen your focus - your ability to ignore the distractions.


you reacted really fast! :stuck_out_tongue_closed_eyes: I pushed too much in this commit, so I reset it and pushed just the one file. see https://github.com/zikula/core/commit/bc8ba63ecd812faca906a6ac578f4ea5bbac9510


I did not know about that. I'll make the change and we should probably do it in the other nifs.


Shouldn't this be named `getCachedFiberFutures`? :smile_cat:


Uppps :see_no_evil:


`slesBegin`, `slesEnd`, `slesUpperBound`, `txBegin`, and `txEnd` should really be private or protected. I tried to make them private but I ran into trouble. Perhaps you might succeed where I failed?


uhm i don't remember, there was some `order_number ` confusion on some old version, or maybe it was https://github.com/spree/spree/issues/4792
anyway this only on the `2-4-stable` branch right now, so it's clearly wrong :)


This should be simplified a little bit.

```java
String packageName = resolveInfo.activityInfo.packageName;

if(appsFilter != null && !appsFilter.contains(packageName){
   continue;
}

String title = resolveInfo.loadLabel(manager).toString();
String name = resolveInfo.activityInfo.name;
Drawable drawable = resolveInfo.loadIcon(manager);
appResources.add(new AppAdapter.AppInfo(title, packageName, name, drawable));
```

No need to check the size every time either.


I'll take a look, thanks for the hint.


One of the most important decisions a tech team like ours face when it wants to be successful and to scale properly is how to organize the code. Software architecture can make teams lives easier or harder as they go depending on how well adapted the architecture is for the system needs.


okay!


Should we use here `getProjectIdFromOwnerIdAndProjectName` function?


норм


Да, прошу прощения, неуместный комментарий.


Switched to using btn.default's values and simply desaturating them using LESS' functions.


Redundant line.


Should we use here `getProjectIdFromOwnerIdAndProjectName` function?
Who is creating the project? Is it in the before hook?


После `{` и перед `}` лучше ставить пробелы, для лучшей читаемости. Есть еще вот такой вариант:

```java
@ContextConfiguration(classes = {
    WebAppInit.class,
    WebAppConfig.class
})
```


Should be ok after the fixes.


This is what causes the test to fail by changing the resulting signature.


The use of configure not being mandatory, can you also add a note about PYTHON_PATH when using the Makefile? Can you also update git-core/INSTALL and submit that to the git mailing list?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.475.b3c42dde331516adb7718af60287d128fbd4ec94](http://build.palaso.org/viewLog.html?buildId=62162&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 138 Build time: 00:07:57


I also like getting rid of the trailing line return from `cat()`, but there are two issues:

1. on OS X at least, `writeChar()` seems to add the unicode control character `00` to the end of the text. For most places this doesn't seem to be an issue, but RStudio highlights the added control character in source and console text. Specifying `writeChar(..., eos = NULL)` seems to fix that.

2. `writeChar()` collapses multi-line texts into one line. I think this might be a bigger question to think about for the top-level `write_clip()` command. Should there be a `sep = ` option so you can specify the separator between each element of the character vector being written?


@mcming1989 any chance of hdpi?????????????


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.455.b3c42dde331516adb7718af60287d128fbd4ec94](http://build.palaso.org/viewLog.html?buildId=62163&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3354, ignored: 134 Build time: 00:08:09


Boot Camp for FreeBSD.  :)


вполне отлично) только в `div.item-*` можно вообще без спана)


@flavour
1) When I click edit button, this says Insufficient Privileges even if I am logged in as an Alert Approver (this permission check for update https://github.com/flavour/eden/blob/master/modules/s3/s3crud.py#L830 differs from that of delete https://github.com/flavour/eden/blob/master/modules/s3/s3crud.py#L1004)

2) For test, I commented out the permission check. But after I click edit button, it tries to open the update view in the same page /cap/alert/xx/review (should be redirected to cap/alert/xx/update, no? or am I missing something?) However this does not open the full form. Looks like http://i.imgur.com/bjtueQG.png


Knowing a full command line that is being executed may help somebody not deeply versed with `npm` to find out what's going wrong. See the following note:

https://github.com/sass/node-sass/issues/1012#issuecomment-115715502 by  @jkwuc89

> By adding some additional logging, I discovered that spawn.js is using
> the following command line to start install.js:
> C:\WINDOWS\system32\cmd.exe /c node scripts/install.js

It is not obvious at first sight whether running a shell with `-c` is used (a full-blown command execution, like in UNIX `system()`)  or just an executable is run (`exec()`). Somebody who understands how processes can be launched can immediately figure out what npm is doing when we log all three, without referring to the source code.

If I need to modify the source to figure that out first, it means my logging is not adequate to the job.

Therefore I think we should have:

````js
    log.verbose('lifecycle', logid(pkg, stage), 'PATH:', env[PATH])
    log.verbose('lifecycle', logid(pkg, stage), 'CWD:', wd)
    log.verbose('lifecycle', logid(pkg, stage), 'Spawn:', [sh, shFlag, cmd])
````

I can spend my energy on troubleshooting `PATH` issues/node location/whatever instead of digging into npm internals first.


:+1: on this commit


отлично!


`TestFactories` is a module from the foundation that was temporarily used before you introduced the factory girl gem. You can remove the `include TestFactories` from this and any other spec. Use factory girl or ActiveRecord's `.create!` if you need to setup test data.


Looks like it, although only with STTRACE=model so most users won't hit it.


This is a really good catch.

I think we need to do some more research as to why this is working in TWC, CS, etc but not in your environment. Also, we need to determine if we need a non-unique key/foreign key constraint instead of the primary key for performance.


ок


LM-TXL-00981417:ampersand-sync-localforage mveeck$ grunt
Running "jshint:gruntfile" (jshint) task

✔ No problems

Running "jshint:lib" (jshint) task

✔ No problems

Running "jshint:test" (jshint) task

✔ No problems

Running "karma:build" (karma) task
28 08 2015 17:40:23.249:INFO [framework.browserify]: 585902 bytes written (2.51 seconds)
28 08 2015 17:40:23.251:INFO [framework.browserify]: bundle built
28 08 2015 17:40:23.256:INFO [karma]: Karma v0.13.9 server started at http://localhost:9876/
28 08 2015 17:40:23.260:INFO [launcher]: Starting browser PhantomJS
28 08 2015 17:40:24.994:INFO [PhantomJS 1.9.8 (Mac OS X 0.0.0)]: Connected on socket w0CsgEKvDMoVayo4AAAA with id 76055586
PhantomJS 1.9.8 (Mac OS X 0.0.0) ampersand-rest-collection fetches from localForage FAILED
	timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.
PhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 2 of 8 (1 FAILED) (0 secs / 2.003 secPhantomJS 1.9.8 (Mac OS X 0.0.0) ampersand-rest-collection updates to localForage FAILED
	TypeError: 'undefined' is not an object (evaluating 'collection.get(id).save')
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:17457
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:17420
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:12448
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:105
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:14898
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13229
	    at flush (/var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13513)
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13166
	    at drainQueue (/var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13137)
PhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 3 of 8 (2 FAILED) (0 secs / 2.003 secPhantomJS 1.9.8 (Mac OS X 0.0.0) ampersand-rest-collection removes from localForage FAILED
	TypeError: 'undefined' is not an object (evaluating 'collection.get(id).destroy')
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:17467
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:17420
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:12448
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:105
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:14898
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13229
	    at flush (/var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13513)
	    at /var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13166
	    at drainQueue (/var/folders/xj/jt32jtpn2bvchyvn_c5277f0393v7t/T/e136fc630be1e1ca068acf299451e3ab.browserify:13137)
PhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 4 of 8 (3 FAILED) (0 secs / 2.003 secPhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 5 of 8 (3 FAILED) (0 secs / 2.013 secPhantomJS 1.9.8 (Mac OS X 0.0.0) ampersand-model "before each" hook FAILED
	timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.
PhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 6 of 8 (4 FAILED) (0 secs / 4.014 secPhantomJS 1.9.8 (Mac OS X 0.0.0): Executed 6 of 8 (4 FAILED) (4.103 secs / 4.014 secs)
Warning: Task "karma:build" failed. Use --force to continue.

Aborted due to warnings.

Execution Time (2015-08-28 15:40:18 UTC)
loading tasks  939ms  ▇▇▇▇▇ 9%
karma:build     9.5s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 90%
Total 10.6s


For details see: https://forge.typo3.org/issues/66180


You are right on all comments :D If you can add it in your lib I would be obliged… I just didn’t wanted to open an issue and I thought pull req would be better to showcase my requirement. Awesome work btw!

> On Aug 28, 2015, at 18:32, Kenny Campagna <notifications@github.com> wrote:
>
> I rather this be a Set rather than an ArrayList so the look up time is O(1) rather than O(n). Also, it should be annotated with @Nullable
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hastoukopsaro/BottomSheet/commit/0b4b5a27213311d5a3113ba0773d7042beb8514e#commitcomment-12950416>.
>


Move this private field to the top.


PS. You would need to uncomment line 24 if you decided you wanted to use `TestFactories` instead of Factory Girl. This loads all the files in the `spec/support` directory.


4695_hpwh_equip_seq_num (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


да, это тут совсем лишнее, наше "описание к медиаобъекту" обозначается тегом `figcaption`, никакие заголовки (и другие блочные элементы) там не нужны. Давай поменяем как было.


Улучшить. Хотя бы до `Web Application Initializer.`


Web-Version: http://muxcraft.github.io/MuxApp


no need to call expand()


Closes #2


sdaasd


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.476.b2a34aa64cbab29fd388b403c3a4dcdc2ff20e6e](http://build.palaso.org/viewLog.html?buildId=62164&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 138 Build time: 00:05:06


Да, такое поведение будет более предсказуемо для пользователя, отлично!


Will this simplification get purged or not ? it passed both STC and LTC.


No problem, I will accept the PR and update when I have time later.


A logged-in user is able to delete another user's todo items if they tamper with the id in the URL. How could you protect against this?


`for f in man/*`


OK, that looks better, I'll change.
The coverage shows the 32-bit parts as not covered.
To really get full coverage stats, we'd need to run for each platform (Linux, OSX, Windows) * (32-bit, 64-bit),
and merge the results.


@niko-rodrigue - the central obo build at http://www.berkeleybop.org/ontologies/kisao.owl should resolve after the next run around midnight now the build info is correct, thanks. Note you are free to ignore the berkeley build and take control of the /obo/kisao.owl purl yourself.

Here's the docs (need to be updated to reflect move to GH)
http://obofoundry.github.io/docs/Policy_for_OBO_namespace_and_associated_PURL_requests.html


😱 you removed a blank line, you'll make pep8 cry!


Could you explain, please? I do not understand. What time adjustments? What is monotonic clock?


The FORCE seems strong in this one.


i think there's no empty line before a dicstring ;)


I think you have mixed up 2 loops suggested in previous PR. One was for system system and suggested one was generic for any 'state-id' and 'pwr_lvl'. Anyway, I am not too fussed about it as it does the same thing you have done here.

minor nit:  You can mask the state-ID with 0xff before copying it to a 8 bit array element.


This is literally a ticking time bomb.  I don't have a solution today.  But the problem is that different `time_point`s from different clocks have the same type, and thus can be subtracted from each other.  If two time_points from different clocks are subtracted, this is just a hard-to-read random number generator.

The std::chrono fix for this is to ensure that time_points from different clocks have different types, and subsequently it is a compile time error to subtract them.


Could you clarify if `@user` is needed in this action, it appears unrelated though I may be missing something.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 42](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=126&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


Change https://github.com/sgwilym/react-router-relay/commit/ece84e9c6c6cad81623d9ffb71f051f80bc4aa62#diff-14666fb8c75a6b46504ebc245197f7d0R28 to be just `nextProps` as well.


fits in the same line


Move this constants to Util.py, they're only needed there now.  Some like PLOTTER_DATA or PROFILE_DIR_IPYTHON_V4 are only used in one function, maybe just put them there?


not is_deleted


With the automatic regeneration of files, I think it's okay to put it right into `layouts/css/page-modules/_home.styl`.


wrong import order


@AppWerft gittio sollte das Modul jetzt als ein Modul für beide Plattformen listen. Kompilieren konnte ich noch nicht, das versuch ich am Wochenende hinzubekommen. Danach wird die API zusammengeführt.


Здесь и выше -- добавь пробелов до/после `{}`.


saw approval from JeetKunDoug
at https://github.com/basho/yokozuna/commit/4259c3f19ff2b3d39c83ba49dc563d43ffb98cc7


extra linebreaks


merging basho/yokozuna/hotfix/check/force-index-ok-and-wait-for-kv = 4259c3f1 into borshop-integration-545-hotfix/check/force-index-ok-and-wait-for-kv


basho/yokozuna/hotfix/check/force-index-ok-and-wait-for-kv = 4259c3f1 merged ok, testing candidate = 7cab8db4


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.456.b2a34aa64cbab29fd388b403c3a4dcdc2ff20e6e](http://build.palaso.org/viewLog.html?buildId=62165&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3354, ignored: 134 Build time: 00:10:22


usually we use 4 space indentation


#luci


"(JavaScript) engine"? "Virtual Machine" could be misleading.


This change isnn't correct, perhaps it should be name the method "getPatternMinimumLength" to make it clearer. The "?" stand for at least one char while "*" stands for 0 or more chars - so "*" must be ignored for the minimum length, while "?" counts as 1 and MUST NOT be replaced.


where are we using this? if it's just a requirement of nbconvert i believe it shouldn't be listed here


Shall we move all the ipython specific hackery to a file of its own?  Like `trappy/plotter/ipython.py`.


@mattn, not sure, it costs nothing to add `#` and all the functionality is just on top of it. If we already have numbers, why not to have links in web interface? On moving away from GitHub, they won't stand in the way unless other service parses the same notation, but I doubt one can address this in a generic way and that it's good enough reason (there always be some dead references in the history of any project) for sticking with inconvenient manual matching of patch numbers.


I might be crazy with the pointers, but I would love to see one test case where `s` (not `*s`) change its value after the call to `fast_handshake_state_free `. I could not make it happen.

For this reason, I see this assertion as `tt_ptr_op(NULL, OP_EQ, NULL);`


Ok, makes sense.


수정하였습니다 감사합니다


Is there a `CenterAwareFunctionOp`? Or `CenterAwareHybridOp`? If not, I think the `Computer` part is unnecessary.

Is there a `CenterAwareFooOp` for things/situations not involving neighborhoods?

You could simply call it `CenterAwareOp`. Otherwise I think I prefer `CenterAwareNeighborhoodOp` over `CenterAwareComputerOp`, since it is really about neighborhoods. Yes, I know it's in the same package, but redundancy is useful between package and class names, since the package names are not part of the simple class names present throughout the source code (only on the imports at the top).


New the _concurrencyProperties when declares, and save two null checks.


This would make more sense if it was `{string:function}`.


@vwadekar
The code looks good to me. Probably Achin might take a look in the following week.


I wasn't sure to be honest... Do people want to set them using a color picker? @lucienfostier @hughmacdonald


@braidn 1.1.0 is in pre-release on master. the master branch here is incompatible with 1.0.0


how about `string[]`? or `Array<string>`? (These are the TypeScript notations)


I don't know what I'm doing but I had to change this to `resource "aws_db_security_group" "default" {` to get it to work


감사합니다.
파이콘에서 뵈면 인사드릴게용


4695_hpwh_equip_seq_num (nmerket) - x86_64-MacOS-10.9-clang: OK (1807 of 1807 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)


either or. You can make your own for or use IAHM-COL's as yours

Best
IH-COL

On Thu, Aug 27, 2015 at 3:26 AM, OrangeTiger <notifications@github.com>
wrote:

> So I just figured out how Github works. Have I seen that correctly, that
> you gave me access to the IAHM-COL/tu154b repository? Should I develop in
> this one or do I rather use a own one as a fork of yuriknsk/tu154b?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IAHM-COL/tu154b/commit/6b0cb3135ae7a1df2fa3b7b53c34b1d9afc9f811#commitcomment-12923122>
> .
>


No problem! I switched ihat to pull the latest version from git rather than a release gem.


4695_hpwh_equip_seq_num (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1813 of 1813 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)


Quite different from original values, especially the first 2...
I'm curious how you got these ? Some new method ?


I was using browscap-php's sorting method as inspiration for that:

https://github.com/browscap/browscap-php/blob/master/src/phpbrowscap/Browscap.php#L695

I can change back and test.


Unfortunately not, and I couldn't find an example online. @michelk added the loader. Maybe he does?


@jhawthorn @magnusvk here's an idea for providing backward compatibility -- https://github.com/jordan-brough/solidus/commit/5281969

It's a bit weird having the middle parameter be a deprecated parameter but I can't think of another way of doing that.  (Another point for using keyword arguments over default-value arguments :))  Maybe that's OK since my goal is to remove all Solidus usage of `#update_from_params` anyway and then the whole method will become deprecated instead of just the second parameter?  Thoughts? Other ideas?


should this have a `$` somewhere?


Not sure I understand the first sentence?

But yes, the assertion is probably not useful.


@Voles: Thanks! I don't see much test coverage but everything seems to work just fine. Would be happy to contribute back with a PR. Should I include the rebuild or just the source change?


@yehudasa I'm not sure the key pair only have access but empty secret is a useful case? When create user or subuser the secret was default generate and no need specified --gen-secret para.


Здесь есть важная штука -- в веб-приложении Spring MVC есть два контекста -- root application context и dispatcher servlet context. Причем первый является родительским ко второму и это значит, что бины объявленные в первом видны во втором, но объявленые во втором не видны первому. Ты сейчас так настроил приложение, что у тебя и root и сервлет контексты инициализируются одинаковым конфигом и по факту там одинаковые бины. Смысла при этом в такой конфигурации не много, если вообще есть. Поэтому я предлагаю тебе попробовать сделать следующее: создать отдельный `@Configuration` класс для рутового контекста, который будет пустым и использовать его для инициализации рутового контекста. По идее, все должно продолжить работать также как и сейчас. Заодно и проверишь :)

Кстати, можешь поиграться и с областью видимости между двумя контекстами -- объяви бин в одном контексте и попытайся за-`@Autowired`-ить его в другом.


I realize this `days_left` method was provided, Consider a couple refinements:

- A model spec that tests `days_left` behaves as you want it to
- Introducing a constant to explain the unexplained number `7` (unexplained number aka [magic number](https://eliotsykes.com/magic-numbers))


Thanks, that was a mistake, thanks for heads up!

However, if we remove "var" from here, we would expose React to the global namespace, something which we don't want to do. For that reason, I did the following [changes](https://github.com/liferay/alloy-editor/compare/adb30a4~2...adb30a4).

I will release a new version with this bugfix very soon.

Thanks,


Yaml allows arbitrary nesting of parameters.  Should this be a loop instead of hard coded at 0 and 1?


В любом случае, глянешь, когда будет время


root@shamu:/ # cat /proc/1/smaps
00008000-00060000 r-xp 00000000 00:01 7782       /init
Size:                352 kB
Rss:                 256 kB
Pss:                 134 kB
Shared_Clean:          0 kB
Shared_Dirty:        244 kB
Private_Clean:         0 kB
Private_Dirty:        12 kB
Referenced:          256 kB
Anonymous:             0 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd ex mr mw me dw
Physical Page:
205320 205321 205322 205323 205324 205325 205326 205329
205330 205331 205332 205333 205334 205335 205336 205337
205338 205339 205340 205341 205342 205343 205348 205356
205357 205358 205359 205360 205361 205362 205363 205364
205365 205366 205368 205369 205370 205371 205372 205374
205375 205376 205378 205379 205380 205381 205382 205383
205387 205388 205389 205390 205392 205393 205394 205395
205396 205397 205400 205401 205402 205403 205406 205407

00061000-00063000 r--p 00058000 00:01 7782       /init
Size:                  8 kB
Rss:                   8 kB
Pss:                   4 kB
Shared_Clean:          0 kB
Shared_Dirty:          8 kB
Private_Clean:         0 kB
Private_Dirty:         0 kB
Referenced:            8 kB
Anonymous:             0 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd mr mw me dw ac
Physical Page:
205408 205409

00063000-00065000 rw-p 0005a000 00:01 7782       /init
Size:                  8 kB
Rss:                   8 kB
Pss:                   8 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:         8 kB
Referenced:            8 kB
Anonymous:             8 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd wr mr mw me dw ac
Physical Page:
479103 702081

00065000-00068000 rw-p 00000000 00:00 0          [heap]
Size:                 12 kB
Rss:                  12 kB
Pss:                  12 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:        12 kB
Referenced:           12 kB
Anonymous:            12 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd wr mr mw me ac
Physical Page:
479105 479106 479108

b6400000-b6c00000 rw-p 00000000 00:00 0          [anon:libc_malloc]
Size:               8192 kB
Rss:                 380 kB
Pss:                 380 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:       380 kB
Referenced:          344 kB
Anonymous:           380 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd wr mr mw me ac
Name:           [anon:libc_malloc]
Physical Page:
256750 479099 479100 479101 479102 479104 689988 689989
689990 689991 689992 689993 689995 701776 701946 701964
701981 702063 702066 702080 704965 704973 704974 704976
709684 710054 711326 711743 711754 711759 711766 711767
711768 711770 711773 711774 711776 711780 712302 712303
712309 712333 712344 712345 712346 712358 713176 713180
713193 713195 713196 713197 713213 713268 713276 713279
713281 713284 713287 713292 713948 715158 715480 715629
781333 781334 781335 781343 781350 781359 781360 781361
781363 781364 781365 781366 781367 781368 781370 781371
781373 781409 781410 781411 781412 781413 781414 781415
781416 781417 782209 782210 782211 782212 782333

b6f9e000-b6fa0000 rw-p 00000000 00:00 0
Size:                  8 kB
Rss:                   8 kB
Pss:                   8 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:         8 kB
Referenced:            8 kB
Anonymous:             8 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd wr mr mw me ac
Physical Page:
689997 689998

b6fa0000-b6fc0000 rw-s 00000000 00:0c 11271      /dev/__properties__
Size:                128 kB
Rss:                  36 kB
Pss:                   2 kB
Shared_Clean:          0 kB
Shared_Dirty:         36 kB
Private_Clean:         0 kB
Private_Dirty:         0 kB
Referenced:           36 kB
Anonymous:             0 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd wr sh mr mw me ms
Physical Page:
701862 704977 704978 704979 704980 704981 711752 713190
773568

b6fc0000-b6fc1000 r--p 00000000 00:00 0
Size:                  4 kB
Rss:                   4 kB
Pss:                   4 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:         4 kB
Referenced:            4 kB
Anonymous:             4 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd mr mw me ac
Physical Page:
713042

b6fc1000-b6fc3000 rw-p 00000000 00:00 0
Size:                  8 kB
Rss:                   0 kB
Pss:                   0 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:         0 kB
Referenced:            0 kB
Anonymous:             0 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd wr mr mw me ac

b6fc3000-b6fc4000 r-xp 00000000 00:00 0          [sigpage]
Size:                  4 kB
Rss:                   0 kB
Pss:                   0 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:         0 kB
Referenced:            0 kB
Anonymous:             0 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd ex mr mw me de

be872000-be893000 rw-p 00000000 00:00 0          [stack]
Size:                136 kB
Rss:                  16 kB
Pss:                  16 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:        16 kB
Referenced:           16 kB
Anonymous:            16 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd wr mr mw me gd ac
Physical Page:
479107 689994 689996 781351

ffff0000-ffff1000 r-xp 00000000 00:00 0          [vectors]
Size:                  4 kB
Rss:                   0 kB
Pss:                   0 kB
Shared_Clean:          0 kB
Shared_Dirty:          0 kB
Private_Clean:         0 kB
Private_Dirty:         0 kB
Referenced:            0 kB
Anonymous:             0 kB
AnonHugePages:         0 kB
Swap:                  0 kB
KernelPageSize:        4 kB
MMUPageSize:           4 kB
Locked:                0 kB
VmFlags: rd ex mr me


Why bump php requirement to 5.4?
I know that 5.2 and 5.3 are already EOL, but there is no code in doctrine1 at the moment that needs 5.4 as requirement.
It has to be noted that there are a __lot__ of legacy applications that was made to work for PHP 5.3 and must stay on old unsupported PHP versions.


When we upgrade to VS-2015:
`static constexpr millis nUpdateMS{700}`;


@TobiasL Done.


**awesome!**


Send a PR and i'll merge it.  I'm not sure what tuning one would want/need to do in this part, but if you want something, i'm ok to merge it...


Aw yes, Thank you. I will bump the our Solidus app and see how it goes. Thanks again!


In order to define an empty Graph I would need empty `source` and `target` functions. In Elm, we can not use an `undefined` function:
```Elm
emptygraph : Graph v e
emptygraph =
    { edges = []
    , vetices = []
    , source = undefined
    , target = undefined
    }
```
My idea here is to use a `Dict e v` instead of `e -> v`, but, to avoid problems with the too strict definition of `comparable`, we should define our own **Dict**.

I don't know if I am using Elm as it is supposed to be used. I will try to write a more elegant solution.


I think you can remove https://github.com/cpojer/relay-starter-kit/blob/4d6d1bc8cf4b923b394f38727d759e0901d8b430/server.js#L41-L42, https://github.com/cpojer/relay-starter-kit/blob/4d6d1bc8cf4b923b394f38727d759e0901d8b430/public/index.html#L25-L26, and https://github.com/cpojer/relay-starter-kit/blob/4d6d1bc8cf4b923b394f38727d759e0901d8b430/public/index.html#L14-L22 - no point in including the distributed files since they're all coming in through the `app.js` bundle anyway, so no point serving them up either.

Might make sense for the other Relay examples to do the same - it seems a little odd to me to use Webpack this way.


Don't think this is an good idea to do tbh


Hmm... possibly...


Drop `nargs` or  parse all the notebooks passed.  The latter seems easily doable.


Put it in a directory of its own, like `scripts/`.