typo in README file was fixed in https://github.com/simonjenga/GoEuroTest/commit/65ec60d16d530ec1dd4ca31a69d81d0c1fe3a180


Only in the directory, which is empty at this time (this is the basefiles package).

Files added to the directory later will be handled by the recursive fixup code in c2598511a4


And please note that, only in the case of requiredDuringScheduling of anti-affinity the topology-key should be limited to node level, there's no limitation for preferredDuringScheduling.


++ Updates are working well.


Why did you drop semicolons?


fixes typo in https://github.com/simonjenga/GoEuroTest/commit/8b9c19eef7d8db9e54266a12c80f7dacf0ee548d


и add src/ плохое название - никто не знает что именно ты добавила и зачем.


Yes, they've been updated


ты убил мой старенький 286Пентиум таким количеством библиотек.


This change has no effect; I assume you meant `{matches: "<all_urls>", js: ["context/extension_page.js"]}`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15429](https://teamcity.mesosphere.io/viewLog.html?buildId=132836&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:08:05


Agreed.


ack


oh sorry , I guess maybe you use other text editor.

Sent from Outlook Mobile<https://aka.ms/qtex0l>

On Thu, Feb 11, 2016 at 6:52 AM -0800, "Glaucia Del-Rio" <notifications@github.com<mailto:notifications@github.com>> wrote:

In Spider the shebang is different from Atom (# -- coding: utf-8 --)

-
Reply to this email directly or view it on GitHub<https://github.com/biolprogramming/assignment-6/commit/87b5f09623454e6535bf2bbf7a8d9a3427cf36cf#commitcomment-16034626>.


When SDA publishes, it should move the license from being a Preference to being a direct statement about the Aggregation. (The user requests a license as a Preference, the publishing repository accepts it (or changes it - which would probably require some out-of-band approval process), and then asserts it. See https://sead.ncsa.illinois.edu/ndsrepository/api/researchobjects/tag:sead-data.net,2015:RO_PX4CfyMHAU5ELvEs8YOLaA/meta/oremap.jsonld.txt as an example.


deleted


Does this work? Would you mind creating a pull request? I'd be glad to add it in the project.


Nope.


this was wrong in my test. Register this defer hook right after we sucessfully track so we clean up the track on error.


The idea of vendor tags should remain in there with a TODO to make sure everyone reading it is aware of this missing piece which must be addressed before these changes are implemented in practice.


Almost there thanks to @ercolanelli-leo


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15428](https://teamcity.mesosphere.io/viewLog.html?buildId=132835&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:09:09


Good suggestion @shabanovd


ad *WARNING: Modality not found*, Modality-toolkit and ModalityWorkshop are installed.


They're not _needed_ but rather so a standard is followed for spacing/parens.


don't set now, use 1984 or whatever as the time so it's easier to be certain that keybase is picking up the debug clock.


Activate could be called twice (once on takeoff, once when entering the flying state) so scripts would be able to decide based on altitude, but I really prefer clean code. If you need it, a workaround is to make Activate spawn a thread that sleeps for a second or two (or whatever the expected climb-time is) and then enables weapons.


this is as it was in the snooze test..


get rid of this when your test works, but it was visually convenient for me


As I said:

build b762
```
java.lang.NoClassDefFoundError: fr/xephi/authme/libs/google/common/io/Files
        at fr.xephi.authme.settings.NewSetting.readWelcomeMessage(NewSetting.java:223) ~[?:?]
        at fr.xephi.authme.settings.NewSetting.validateAndLoadOptions(NewSetting.java:188) ~[?:?]
        at fr.xephi.authme.settings.NewSetting.<init>(NewSetting.java:52) ~[?:?]
        at fr.xephi.authme.AuthMe.createNewSetting(AuthMe.java:451) ~[?:?]
        at fr.xephi.authme.AuthMe.onEnable(AuthMe.java:216) ~[?:?]
```


This should be changes to use `java.nio.file` classes, and at the same time, `Files.copy` would seem sensible


now set the flakey bit


Typo on Imagery


you weren't waiting long enough for the 24-hour soft snooze to have worn off.  I fixed that for you. this was the main bug in your test (aside from snoozing hard failures)


I agree with @shabanovd. Should also switch to `java.nio.file`


I see what you did there


ack


Typo on posted


I think you still need the #!/usr/bin/env python and as far as I know generally is consistent within all programs/editors... I am not sure if its points off or not if you don't have it.


ya looks like it, but pub seems to handle this just fine.


this is a test


"To enhace interoperability" should probably be "To enhance interoperability". Thanks!


Agree with @shabanovd


`testcode <p>`


I'm guessing that it's the `width` and `height` in the call to `aes()` in ggplot that cause this. Maybe they have been deprecated in ggplot v >2.0.0, I haven't used oncoprint after updating. Maybe you could try installing an older version of ggplot?


As the commit says, you can still copy on demand, if it's necessary for your team's deployment settings.

The old copy script is flawed and broke. That's why it's better not to run it (also because it doesn't add values in most cases). I can

-------See my comment in another thread--------

It's more standard to require CUDA installed on the target machines. No toolkit that I know ships with CUDA runtime. Caffe Linux build also doesn't copy these files.

What about cuDNN?

I suggest not running this script in the pre-build. We can still leave the script for those who want to run on demand.


commentaire de la quatrième ligne du fichier autre-fichier.txt


it's data -> its data


si porque quieria compartirlo antes, aver que opinas de mis creaciones jajajja


:/ Right, i'll fix this asap


Bah!


Are you sure? The tests use e.g. `BiorthWaveletTransform` and call it, that would raise an error if Jan-Olov was not installed?


Prefix doesn't store the old value as suffix increment does to return it after. I'm doing it out of habit more than anything. The performance is probably void in a case like this one.


https://github.com/spring/spring/blob/develop/rts/Sim/Units/Unit.cpp#L2320
activate won't be called twice without interfering with even more stuff

since we're in a feature freeze, I'd much prefer to keep it to the previous state, where activate happens on flying and deactivate on landing


Import


I've just noticed this was added to `Node` when it already has a `__copy__` function declared above. Now, the other `__copy__` function probably doesn't work but it really should have been removed as there's no way to know which one will be called.


Thanks, fixed in https://github.com/everypolitician/everypolitician-popolo/commit/e4afe671e6b88bc095a911e00cf2152aaed5d51f


I don't like them.


si pones aprueba este poin pon te vadar error ya o simple mente la ball solo repetira los mismo movimientos asi que tienes que hacer que cuando choque contra la raqueta tiene que actualizarse eso es lo jodido de este juego el resto es facil


The whole backwards compatibility arguments thingy will be removed in future, so we can just keep that there for now.


Old scripts will work fine. And still work while devs are using 'single item' ingredients in GTTweaker's methods.
When you try to use the ore dict in recipes, the script will add a recipe only with the first ore dict entry. And it's working so but it's incorrect behavior.
So, the short answer: cancellation is not necessary.
Also you can merge this pull after I finish changes relating to #4

PS: what the IDE are you using?


Thanks for the patch. Please introduce an intermediate variable like ``$fullSettings`` or something to avoid ``$settings = $settings[..]``.


The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "MAY", and "OPTIONAL" in this document are to be interpreted as described in RFC 2119.

"Try to" is not part of them and MUST NOT appear in the recommendations.


Uh... OK, I'm getting a bit tired of git doing this when I try to make a branch. I did not (mean to) push all this into master.

I suppose it doesn't really hurt to have it in here, though.


Hmm, maybe qualify that because, honestly, normal people don't know that the Åland even have a Lagting; but brilliantly we have a website that makes it clear :-D   so how about

"...Lagting (which is the legislature of the Åland islands, available as JSON data from the [EveryPolitician page for Åland](http://everypolitician.org/aland/)."


This comment is a lie. This file is only loaded in the background page.


replace the “shops at olde colonial place” with the Marlborough Savings Bank in Sudbury, MA Photos


I don't have access to the repo where the discussion of this took place (woothemes/woocommerce-shipping-fedex#24), but just FYI, this statement is not true for all carriers: "You wouldn’t ship to Guam for the same cost as the US."

For USPS, the US Territories _are_ typically eligible for domestic services and domestic rates. So if I'm in New York and I want to use a USPS flat rate service, it costs the same to Guam as it does to New Jersey.

Further, most USPS APIs require that the territory be listed as a state and that the country be listed as US.

So I would agree this change makes sense for _some_ carriers, but it makes things more complicated for working with USPS.


Eh, yes, something like that.

The idea is just to **force** this to be the first JS loaded on every page which uses the *content-script-loader*.


Feel free to PR a better wording: SHALL etc


@pynicolas See what I've done on this 9bae3ccbe073a8e75dd4628a093bdd39c0f959e7


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15430](https://teamcity.mesosphere.io/viewLog.html?buildId=132844&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:09:08


pourquoi ?


Note that it's also easy to parse these times with Java 8's [`java.time.Instant#parse(CharSequence)`](https://docs.oracle.com/javase/8/docs/api/java/time/Instant.html#parse-java.lang.CharSequence).


Maybe something like `# Still need a/b_slice in case it is a list of indices`?

As I recall, there were some discussions at some point about how disallowing lists of indices would make some things simpler, especially for some backends. I can't find an issue for it though -- is it worth opening up one? I have made the argument in the past that anything you can do with lists of indices you can instead do with multiple connections. Another option would be to convert lists of indices to multiple connections in the builder, or something of that nature.


Should we add support for generator in Theano for updates? If this is useful in Python 3, we can make an issue about this.


You can copy [this](https://raw.githubusercontent.com/zer0n/caffe/master/windows/CommonSettings.props.example) and [this](https://raw.githubusercontent.com/zer0n/caffe/master/windows/CommonSettings.props.example2) to http://text-compare.com/ for a quick diff.


Well, looks like it is while linking the KDSME app, don't know how "external" you find that ;)

```
...
cd /work/build/kdsme_build/src/app && /usr/bin/cmake -E cmake_link_script CMakeFiles/kdstatemachineeditor.dir/link.txt --verbose=1
/opt/rh/devtoolset-3/root/usr/bin/c++    -std=c++0x -Wno-deprecated -Wextra -Woverloaded-virtual -Winit-self -Wmissing-include-dirs -Wunused -Wno-div-by-zero -Wundef -Wswitch -Wpointer-arith -Wmissing-noreturn -Werror=return-type -Wunused-but-set-variable -Wlogical-op -Wsizeof-pointer-memaccess -Wreorder -O3 -DNDEBUG    CMakeFiles/kdstatemachineeditor.dir/mainwindow.cpp.o CMakeFiles/kdstatemachineeditor.dir/main.cpp.o CMakeFiles/kdstatemachineeditor.dir/kdstatemachineeditor_automoc.cpp.o  -o ../../bin/kdstatemachineeditor -rdynamic /work/build/kdsme_qt5_install/lib/libQt5QuickWidgets.so.5.5.1 ../../lib/libkdstatemachineeditor_core.so.1.0.0 ../../lib/libkdstatemachineeditor_view.so.1.0.0 /work/build/kdsme_qt5_install/lib/libQt5QuickWidgets.so.5.5.1 ../../lib/libkdstatemachineeditor_core.so.1.0.0 /work/build/kdsme_qt5_install/lib/libQt5Quick.so.5.5.1 /work/build/kdsme_qt5_install/lib/libQt5Qml.so.5.5.1 /work/build/kdsme_qt5_install/lib/libQt5Network.so.5.5.1 /work/build/kdsme_qt5_install/lib/libQt5Widgets.so.5.5.1 /work/build/kdsme_qt5_install/lib/libQt5Gui.so.5.5.1 /work/build/kdsme_qt5_install/lib/libQt5Core.so.5.5.1 -Wl,-rpath,/work/build/kdsme_qt5_install/lib:/work/build/kdsme_build/lib
/opt/rh/devtoolset-3/root/usr/libexec/gcc/x86_64-redhat-linux/4.9.1/ld: warning: libxdot.so.4, needed by /work/build/graphviz/lib/libgvc.so.6, not found (try using -rpath or -rpath-link)
/opt/rh/devtoolset-3/root/usr/libexec/gcc/x86_64-redhat-linux/4.9.1/ld: warning: libcdt.so.5, needed by /work/build/graphviz/lib/libgvc.so.6, not found (try using -rpath or -rpath-link)
/opt/rh/devtoolset-3/root/usr/libexec/gcc/x86_64-redhat-linux/4.9.1/ld: warning: libpathplan.so.4, needed by /work/build/graphviz/lib/libgvc.so.6, not found (try using -rpath or -rpath-link)
/work/build/graphviz/lib/libcgraph.so.6: undefined reference to `dtview'
/work/build/graphviz/lib/libcgraph.so.6: undefined reference to `dtextract'
/work/build/graphviz/lib/libgvc.so.6: undefined reference to `dtflatten'
...
```


lgtm


What was the reason for commenting this out?


Yep good point. Added your suggested text in https://github.com/everypolitician/everypolitician-popolo/commit/79581390550d55e1fd7cb8d2ebfe2903ac5d2d41.


pathon code


Ok not ajax but use web.ajax and insert file in dom ;)


This is Main function. Entry point for app


Je pensais le commenter ensuite


Thought I'd see something really cool but once again your just weird


haha ok... pas de pb


ASCII Art !


OK, simpler, whenever possible, is much preferred to me.


Removed underscores.


> from a packaging perspective this allows me to not list OpenAL as a package dependency (similar to fluidsynth)

...and it's from **packaging** perspective exactly where this approach is wrong


Tbh, addressing the copy functions I think is worth a separate PR, they're in a mess and I would like to kill with fire the ```__getstate__``` and ```__setstate__``` functions which have mysterious behaviour at best imo.


Ah, missed that one, ok should be fixed now. I had the impression it was possible to "git pull" from the Quarks gui but indeed now I can't find that option. @crucialfelix was this ever implemented ?


I would squash this into the base-hw riscv commit as the intermediate state of having a riscv startup lib but no use for it makes no sense.


The problem is, the whole fighter branch got merged into master, not just this one commit.

I recommend hard resetting master to 788a054ca05ef349dfb111b630f8c7dd6db216e9 (latest commit in master not belonging to the merged fighter branch). Shall I do this?


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


Let's open an issue. It would certainly make things simpler, and allow us to get rid of the SlicedCopy operator. But it might make model description harder (but we could have a helper function or convert indices to connections somewhere), and it might also make performance worse by requiring separate sets of operations for each connection (though we've talked many times about adding an optimizer, which could look for things like this and combine them again at the operator level or something). It might be worth discussing anyway.

For now, I'll just add the comment.


what do you mean?


We should make this configurable.


Sounds good. I'll open the issue.


I would squash this into the base-hw riscv commit as the intermediate state of having a hw_riscv builddir but no use for it makes no sense.


This commit is missing the parboiled fix:
```
def contentTypeLookup(fileName:String) = {
  import org.parboiled2._
  import org.parboiled2.CharPredicate._
  class ExtensionParser(val input: ParserInput) extends Parser {
    def FileName = rule { NotExtension ~ Extension ~ EOI }
    def NotExtension = rule {oneOrMore(oneOrMore(Visible -- '.') ~ '.') }
    def Extension = rule { capture(oneOrMore(Visible -- '.')) ~> (s => mimeTypeMap.get(s)) }
  }
  new ExtensionParser(fileName).FileName.run().getOrElse(None)
}
```


@nle-odoo then my comment is "did you just add field in stable version?" (even if not stored, should not happen, potentially is conflicting with a community module, especially something as generic as 'name')


Could you please use them nevertheless? The [original author](https://github.com/felixge) of this library, who also maintains a pretty popular Node style guide, [recommends using them](https://github.com/felixge/node-style-guide#use-semicolons). If that's possible, it would be nice to follow it :smile:


(c) 2015-2016


Jeff! Save us. Apply https://github.com/grails-plugins/grails-quartz/pull/51 :)


i would refactor this into its own method


again (c) years


missing (c)


again missing (c)


https://amisr.asf.alaska.edu/PKR/DASC/RAW/
this was the url, first I used searchsite() then download()


On 11.Feb 2016, at 16:12 , Miguel Negr�o <notifications@github.com> wrote:

> Ah, missed that one, ok should be fixed now. I had the impression it was possible to "git pull" from the Quarks gui but indeed now I can't find that option. @crucialfelix was this ever implemented ?

I remember the discussion, but I wasn�t sure of its outcome.
At least here it definitely doesn�t pull via the gui.
Got the update now, thanks a lot!


Okay, maybe someday Bikini will have a "real" lexer + parser + ast into code... Maybe using the llvm tools as a binding, who knows :). I agree this iteration can't go much further, as I said a long ago.


Looks ok. Did you just update to CS6? Looks like the MSBuild rewriter is not running. If you use .Net Reflector the ExpressionField attribute should have been filled with an string argument.

From:

> [ExpressionField]

To

> [ExpressionField("ToStringExpression")]

Is this happening only on this entity?

Von meinem iPhone gesendet

> Am 11 feb 2016 um 13:28 schrieb Marcel Numeijer <notifications@github.com>:
>
> static readonly Expression<Func<CategoryEntity, string>> ToStringExpression = e => e.Name;
> [ExpressionField]
> public override string ToString()
> {
> return ToStringExpression.Evaluate(this);
> }
>
>
> Van: Olmo [mailto:notifications@github.com]
> Verzonden: donderdag 11 februari 2016 12:40
> Aan: signumsoftware/framework
> CC: Marcel Numeijer
> Onderwerp: Re: [framework] add missing [ExpressionField] (a7e3699)
>
>
> There are no changes. Can you paste the ToString of the entity you're trying to retrieve Lites from?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/signumsoftware/framework/commit/a7e369966d99dc8a6fc36eb7a0b7227d50223f30#commitcomment-16030405>.
> —
> Reply to this email directly or view it on GitHub.
>


В дальнейшем в пагинаторе, так же как в хлебных крошках, нужно будет убрать ссылку у активного элемента.


One of the best commit I've seen in a while


Do we really need this? I didn't see the value in the abstraction because you don't need the content type in the consumer of a message.


and so on ... ;)


@AKindyakov в конце концов это все должно записаться в бд, если данные обновились с последнего запуска


merge


lol hehe


I prefer `IMessageSerializer` because you don't serialize the queue but the message.


(c) and signature missing


if the list got excluded in the json it deletes it entirely from the database (yey, protobuf!)


You can prevent this from happening at least in the main repo if you make a personal fork and maintain your feature branches there. In case that happens again, only the master branch in your fork would be affected and has to be reset. Much less trouble :wink:


copy paste :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15431](https://teamcity.mesosphere.io/viewLog.html?buildId=132855&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:44


Accidental, I was having local build issues with it in.


Agreed, updated. :rocket:


If we have the a serializer, we should use him instead of the direct function call.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15432](https://teamcity.mesosphere.io/viewLog.html?buildId=132856&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:14


I kept it out of my commits this whole time, and then fucked up on the last one...


@AKindyakov класс! блин, яж знал что есть чтото подобное, тока не было времени, тыкался в обычный try/catch


I would say either headers for no mk file or for all.


What's that for?


See above...


I think there's an argument for a different config area for "game rules"
like this. There's no reason to keep a magic value in a local variable
inside a global function just because some day we may change it to an
algorithm.

On Thu, Feb 11, 2016 at 9:20 AM, Roy R. <notifications@github.com> wrote:

> If we were going to turn the number into a constant, we could put it into
> ./deploy/derived_constants.php (tracked constants file). But I think if we
> ever change it, we're going to want to change it to not be a linear
> function, but a log function, and then a constant won't be useful anyway.
> Moving the "5" to a different location would just have to be undone in that
> view, and that 5 isn't used anywhere else either.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BitLucid/ninjawars/commit/26e193640e0476e8e9ae923405a1b7b53e9f9a94#commitcomment-16033731>
> .
>


again copy paste


Still having some issues with the ~~strike-through~~ and *italics* it seems. Not sure if it is allowed on the README.md and only available through comments or if I am just doing something wrong. either way time to move along.


Да нет, она то и делает, обновляет данные. А то что возвращает, дак это для последующей проверки, что данные обновились, или нет, чтоб вести учет


Can you explain what you mean here?

`webcompat.app.config['TESTING'] = False` means fixtures are not used, and you get the actual response. Or do you want to test the fixture response?


You have used a `collections.defaultdict(set)` and made this easier.


Again (c) and signature missing...


I think this is not needed:
> 31.5. importlib – The implementation of import¶
>
> New in version 3.1.

and we support python >= 3.3

https://docs.python.org/3/library/importlib.html


Maybe also the locations for the plugins?


Точно, почитаю про эту штуку обязательно. Почемуж не выкинуло с ошибкой интересно


This isn't a correct change, because (1) now the function declaration in tsk_img.h no longer matches the function definition, (2) `ext_img_info` is required to be castable to a `TSK_IMG_INFO*`, and (3) there is no purpose to passing two pointers which MUST be equal.

What build error did you have?


The align within the the columns does not seem to be working from within the README.md might be good to investigate later.


I didn't notice this one in the issue tracker over at TC. Do you have a link perhaps? I'm not sure how it should work exactly. I've been looking at some spell values to see if there is anything to indicate what type of behavior they should have but can't.

Currently they don't enter combat until a player moves. Meaning if I stay still and autoattack, the creature will also just stand still and do nothing. It's almost like we need a combat pulse on summon, but then this might not even select random targets either.


"could be" meaning theoretically, and activate-on-takeoff is not a feature.

I'll go along with SAMT activate on flying/deactivate on landing, but *only* for 101.


Если сумки перенесены ниже, при их открытии, надпись с кол-ом ошибок перекрывала фрейм.


@etirelli I've send a PR to fix this: https://github.com/droolsjbpm/drools/pull/641


*not initial import. created new class, "Exercise01"


nda bila login giman cara instalasinya ?


Slight naming preference for `getIncludedLayerNames` or similar, `includeLayers` sounds like an action is being performed.


Same comment as above.


why if I may ask?


Just at the current master pls. There have been some changes since then.


I added a few formatting ideas and tried to work with them.. It helped re-solidify my basic git understanding at least. :smiley:


disctrict удалил
Вообще весь этот код отчасти сборка из кусков. Но суть осталась сутью, это именно тот код которым, надеюсь еще не долго, я оперирую. Со всеми этими сомнительными заглушками, и тупыми решениями, типо брать вложеный ключ из словаря через ексепшн)


Removed


Pushed form incorrect branch - reverted.


That's weird... It's not in the file I pasted it from.
I used the web editor to do this commit and it added loads of blank lines?!


I was just being facetious. I would maybe include your border on the bottom edge.


I would have thought application/x-vtk as well.


@AKindyakov Пасибо еще раз. Буду должен )


In the event of file reloads.. idk.


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


The workaround would create a mismatch between activate sound event and unit animation, and if activate is called twice, it would play the sound twice.


Do You Have Server !


This can also be a shared example

```ruby
# ideally...
it_behaves_like 'a record with a uuid as a primary key'
```


It should be optional.


@fjammes, I do not really know how do we install client-only mariadb, if this is something that is controlled by mariadb makefiles then it's probably worth notifying them of this unexpected behavior. I guess you know more about it so I leave it to you to talk to mariadb guys.


Would love to do that on many more places...


This is likely failing some tests as `include_resources_param` on line 407 could be `None` which can't be `split(',')`


better: "if (slen > 0)"...

Also, maybe this is something where we should add memcpy_safe(), similar to how we did it with qsort_safe()?


Students/30421/Alb V. Corina Bianca/Gin Rummy - final/app/src/main/java/com/example/corina/ginrummy/CardsSetup/CardPiles.java

If it models one single card pile then it should be `CardPile` not `CardPiles`


Students/30421/Alb V. Corina Bianca/Gin Rummy - final/app/src/main/java/com/example/corina/ginrummy/CardsSetup/CardsArrayCost.java

Same here on using ploural


Mmm.. no fix https://github.com/L2J/L2J_Server/issues/205 :(


lgtm


It needs to be 37 because that's the lowest version of MCPE supported.


looks like this fixed the problem I was seeing with certain "VEVO" youtube videos that did not provide m4a format.


man page addition is missing for this, no?


I think
`padding: 0px 5px 0px 5px;`
looks nicer for the dropdown button, but maybe you had a 0 padding on purpose and I just don't get it?


The `Play` class looks slightly too big, however good job otherwise


`return` missing?


Fix for #21


It's hard to tell because the code moved, but I changed this from `hasPermission` to `hasAllPermissions` like we discussed.


looks great otherwise


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15433](https://teamcity.mesosphere.io/viewLog.html?buildId=132871&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:05:16


You're right - I'm going to add that eventually. I just committed my WIP edits because I have to study for exams.


You're correct. The intent was to simply put the declaration of img_info at the top of tsk_img_open_external() method. I accidentally put the definition of img_info as one of input parameters into the tsk_img_open_external() method. I will change this.


I also changed the output of this error message: `req.query.doc_ids` is a string. In previous code we had `req.query.doc_ids[0]` like it was an array.


죄송해요~~~ 이것저것 하다가 실수 했네요.


tidak isa logi


Sounds like a good idea. I won't touch this for now then. In general I also wonder about combining `Node` and `GangaObject` into one since nothing else ever inherits from `Node`. Either that or have a much cleaner division between the two.


Fixed in 5a6ad54.


lgtm.


Thanks! I'll add in the notices and error handling next, then move on to the other functionality.


ah sure :D I can also create a pull request so you have less work!


@nirvdrum


Do we need to reset? After removing a few preunlocked items, the extra stuff shouldn't actually affect the game. The feature was well enough received that it seems likely to get integrated eventually anyway.

I'm not sure what constitutes a "fork" as you describe it, but if it decides that I meant to push to the main repository master instead of a branch, then won't it still push to master regardless of whether there are other repositories in other places?


lgtm


@metanet: could you please describe an idea behind this fix? why the old code was hanging?


Cool, thanks.


Looks good to me. Small but very useful addition. +1


Yay :+1:


If no more questions, the prev button is disable, so user can't reverse back to earlier questions.


I will add author and CreatedAt after this pull request


@noahpryor awesome! I wonder if we should refactor Hangar to have drivers for multiple factory backends (FactoryGirl, Fabricate). Would you be interested in doing a PR for that?

I do think you should keep me on the copyright.


write just:
if (mQuestionBank.prevMoreQuestions() )
Don't need to compare since the method returns a Boolean.


Not a good idea to hardcode the value 3. Use questionList.size().


The link works great, we just don't want to put it in the NetLogo help menu right now since NetLogo-Users is already available to our users. This gives us the ability to turn it on easily in future releases without any code changes.


suggested desert


Use a nested linear layout for two buttons.


my name is ali


We do these kinds of checks quite often actually, i guess in an excercise to write code that doesn't react to allergic to signedness of integer variables...


Oh, my bad; I read that as they were all three required to hit the top-if-block.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15434](https://teamcity.mesosphere.io/viewLog.html?buildId=132872&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:16


ack


>what the IDE are you using?

Ahm, yep, IntelliJ, thanx .gitignore :)


reack


Doesn't work


LGTM


2016 by CPqD


or any update what so ever?


I understand the const :)  The comment says that the output bytes are preceded by a length; so presumably somewhere something's doing the equivalent of Marshal.WriteInt32(data.Length, data, 0) (not that that method exists)... but I didn't see anything doing that work.

I think I found it in a different PR, which probably means that here it's just assumed to exist?


Hmmmmmmmmm


Again, feel free.


coding style... not line break between function type and name. no space between function name and opening (, no newline before opening {...


Yes, the error message looks good to me.


coding style... no {} around single-line if blocks...


5420-OSCM-Docs (mitchute) - x86_64-MacOS-10.9-clang: OK (2078 of 2078 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

![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)


Hi Chririla,

I think I can probably explain what's going on. The $substr expression is currently only designed to work correctly on ASCII strings, not UTF-8 strings, see docs here: https://docs.mongodb.org/manual/reference/operator/aggregation/substr/#exp._S_substr

When $substr is passed a string encoded in UTF-8, it interprets the 'start' and 'length' inputs in terms of number of bytes, even though some UTF-8 characters require multiple bytes to represent one character. This can result in an output that is no longer valid UTF-8. See below for an example in the mongo shell:
```
> var x = '\uD834\uDF06'
> x
𝌆
> x.length
2
> x[0]
// won't render here, it's an invalid UTF-8 character
```
The error you saw means that somewhere in your data there is unicode character that is represented by multiple bytes, and your $substr computation created an invalid UTF-8 character. I hope this helps!


Why not just use brew to install mongodb? I think the benefits of keeping the mongo install localized to this directory are minimal. Upgrades and maintenance would be automatically handled.


Refactoring #3

Method extraction
To make it easier to read


add indentation


この判定方法でもいいのですが、純粋に`$_FILES['image']` の中に `type` という配列があるので、この中の値で反映してやればいいと思います。
具体的には、画像(jpg/gif/png)の場合であれば `$_FILES['image']['type']` が image/jpeg か image/png か image/gif になるので、

`elseif (!in_array($_FILES['image']['type'], array('image/jpeg', 'image/png', 'image/gif')))`

で判定できます。


:+1: for clear variable names


```
def require_admin
return if current_user.present? && current_user.admin?
flash....
redirect_to root_path
end
```


no file closing anymore?


@gcagle1 I'm not sure why the code you reviewed was missing all of these things...


If these are ready to pull request to ygopro-scripts, then feel free to do so.


Not related to your changes, but I realize that `taskId.mesosTaskId` is mainly used to pass a MesosProtos.TaskId to the schedulerDriver – wouldn't it make sense to push that into an own layer that does the conversion so we don't need to do this in various places?
(Besides, one could argue this kind of conversion belongs into a Serializer object.)

Irrelevant for this PR, but please share your thoughts on this.


この内容は、34行目より下の直下に入れてやるのが適当であると思料します。


Refactoring #3

Restruction
To make it easier to read
Remove unneeded data wrapper "DataPair"


@natemccurdy moved to Atom as an editor.  This no longer be a thing.  All the same, thanks for the PR! :metal:


Buggy - stops tracking player position shortly after being online.


Sounds okay.

(We should really introduce a 80/90 char limit per line in the book as well, since the diff and github ui is not usable otherwise)


Nitpicking! This is the order Ubuntu/OSX/Windows appear in on the download page.


I believe additional commit is required:

```
[ERROR] [pool-5-thread-1] org.red5.server.scope.Scope - Could not start scope Scope [name=chat, path=/default, type=APPLICATION, autoStart=true, creationTime=1455205579069, depth=1, enabled=true, running=false]
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'webSocketScopeDefault' defined in ServletContext resource [/WEB-INF/red5-web.xml]: Invocation of init method failed; nested exception is java.lang.NullPointerException
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1578) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:545) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:482) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:306) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:302) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.context.support.AbstractApplicationContext.getBean(AbstractApplicationContext.java:1054) [spring-context-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.red5.server.Context.getBean(Context.java:413) ~[red5-server.jar:na]
	at org.red5.net.websocket.WebSocketScopeManager.addApplication(WebSocketScopeManager.java:77) ~[websocket-1.4.jar:1.4]
	at org.red5.demos.chat.Application.appStart(Application.java:36) ~[chat-1.0.jar:na]
	at org.red5.server.adapter.MultiThreadedApplicationAdapter.start(MultiThreadedApplicationAdapter.java:401) ~[red5-server.jar:1.0.7-SNAPSHOT]
	at org.red5.server.scope.Scope.start(Scope.java:1255) [red5-server-common-1.0.7-SNAPSHOT.jar:na]
	at org.red5.server.scope.Scope$ConcurrentScopeSet.add(Scope.java:1483) [red5-server-common-1.0.7-SNAPSHOT.jar:na]
	at org.red5.server.scope.Scope.addChildScope(Scope.java:212) [red5-server-common-1.0.7-SNAPSHOT.jar:na]
	at org.red5.server.scope.Scope.init(Scope.java:983) [red5-server-common-1.0.7-SNAPSHOT.jar:na]
	at org.red5.server.scope.WebScope.register(WebScope.java:232) [red5-server.jar:na]
	at org.red5.server.scope.WebScope.afterPropertiesSet(WebScope.java:113) [red5-server.jar:na]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.invokeInitMethods(AbstractAutowireCapableBeanFactory.java:1637) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1574) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:545) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:482) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:306) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:302) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:772) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:839) [spring-context-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:538) [spring-context-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.red5.server.tomcat.TomcatLoader$1.run(TomcatLoader.java:495) [tomcatplugin-1.12.jar:na]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [na:1.8.0_66]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [na:1.8.0_66]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [na:1.8.0_66]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [na:1.8.0_66]
	at java.lang.Thread.run(Thread.java:745) [na:1.8.0_66]
Caused by: java.lang.NullPointerException: null
	at org.red5.net.websocket.WebSocketScopeManager.addApplication(WebSocketScopeManager.java:70) ~[websocket-1.4.jar:1.4]
	at org.red5.net.websocket.WebSocketScope.register(WebSocketScope.java:67) ~[websocket-1.4.jar:1.4]
	at org.red5.net.websocket.WebSocketScope.afterPropertiesSet(WebSocketScope.java:49) ~[websocket-1.4.jar:1.4]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.invokeInitMethods(AbstractAutowireCapableBeanFactory.java:1637) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1574) [spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
	... 34 common frames omitted

```


ack


добавь internal к объявлению класса.


not sure it would even be an improvement, but you could consider only loading these if you're going to display a map


(probably more trouble than it's worth)


hi

uint8 sp_rate = abs(skill_get_sp_rate(skill_id, skill_lv);  / /you forgot " )" after skill_lv)


**Nitpick - Code Quality**
This block is not properly indented. It can be indented like this
```
scope(:by_route,
  lambda do |from, to|
    where("departure_id = ? AND arrival_id = ?", from, to)
  end
)
```


Is tube::Message not working?   We should fix it, rather than eliminate it.


unack until @AbdealiJK answers the question


Sure. Might want to compare the phrasing to your current phrasing as well - see if anything can be improved.


`SUPPORT_URL = 'SET-ME-PLEASE'`


画像アップロードしたものを保存するときのファイル名を `$title` としていますが、できればアップロードしたときのファイル名でそのまま保存してやるのが適当だと思います。その場合、
`move_uploaded_file($_FILES['image']['tmp_name'], './album/'.$_FILES['image']['name']);`
となります。

また、そうすると画像のファイル名を保存するカラム(imageカラム)がpostsに必要になるので、DB構造を
```
create table posts(
 id int primary key auto_increment,
 name varchar(32) not null ,
 title varchar(32) not null ,
 image varchar(255) not null ,
 created_at datetime not null
);
```
としてやるのがいいでしょう。

もちろん画像表示の部分も
`<img src="./album/<?php echo $post['image']; ?>" width="100" height="100" />`
とします。


nit but if this is user facing could use a translation


@giogonzo i didn't find out an easiest way to do it


sweet. :skull:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15435](https://teamcity.mesosphere.io/viewLog.html?buildId=132873&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:08


TBD = to be determinate (à déterminer)


Move this to the core context processor so that it is available to all views.


remove all except one, meaningful naming for state, ad controller


This is awesome! Good job Alex!


is this a side effect? There is a method of pushing an object into an array in which return the updated array?


How do you plan to embed the close icon here?


:sparkles:


you probably need to add `--all="true"` as well to make sure all artifacts get copied


if we're using this in more than one place should maybe be in settings (though I realize it's my fault that it's not already there)


copy can be changed, this is just an example


It's perfect for the Q1 to get error msg1, when you turn off the 'enable_contant_folding' option. What I'm not clear is which part of your code will detect and raise the error.  Looks to me UnsupportedOperatorsVisitor only checks Select/Orderby/GroupBy; it does not check Where clause.  Can you point me to the code that raise the error msg1 for Q1 when option is off?  If we have such logic, then looks good to me. +1

{code}
Q1:
select * from dfs.D:\drill\files\testData where dir0 = maxdir('dfs','D:\drill\files\testData') and columns[2] = 'Q2'
{code}

msg1:
{code}
java.lang.UnsupportedOperationException) The partition explorer interface can only be used in functions that can be evaluated at planning time. Make sure that the planner.enable_constant_folding configuration option is set to true.
{code}


追記ですが、テーブルに値を入れる時ももちろん、
```
$sql = "insert into posts (name, title, image, created_at) values
            (:name, :title, :image, now())";
$stmt = $dbh->prepare($sql);
$stmt->bindParam(":image", $image);
```
とします。


@sils1297 are you talking about `_close_std()` ?


hah, next commit ;)


Whoops, yeah chat app is broken, just remove it until I can fix it


Hello ptrbrtz,

Im using your razor's firmware and, for my personal use, i changed your firmware (output datas) to receive AccX, AccY, AccZ, Yaw, Pitch, Roll, MagnX, magnY, magnZ in one line with 5Hz sampling. And i have some nan-issue on yaw,pitch,roll after 20h of acquisition.

Do you have an idea for what razor work fine for some hours and send nan after ? And i don't know if i must modify type "int" to "int16_t" as Jewang. I use Beaglebone Black (debian) Rx/Tx pin for communication.
If you want i can send you the piece of code that i modified for output.
(Sorry for my english)

Lot of thanks for help and firmeware.

Regards, Christophe.


I'm very pleased with your patch, as I was just beginning the same work. One comment - if you set the ssl key to false, it still creates the entries, as it is just checking for the existence of the key.

I have created a variant https://github.com/dalesit/sensu-puppet/tree/cluster_config which is a hybrid of the two approaches. It specifically has a Boolean rabbitmq_cluster, and has the option of rabbitmq_cluster_hosts which is an array of hosts (utilising the same config for all) or you can use rabbitmq_cluster_custom which is an array of hashes and is essentially your option.

Depending upon which option is selected, it uses one of two templates. The vagrant test configures using one method for sensu-server, and the other for sensu-client.

Would appreciate your thoughts.


Ah. i see what you mean. No this is a bug. I deleted it by mistake.


@adrianmoisey :smiley:


bad_since + ENTRY_GUARD_REMOVE_AFTER not always 30 days


Missing a spec for when `PublicBody#followup_bad_reason` returns `"unexpected followupable inconsistency"`. Also note that in this case, the current behaviour is to `raise`, which I think we should maintain.


- static final loggers
- final class with private constructor
- EnumMap


Is that the same meaning? Isn't the default Array[Any, 0, default) ??


I'm curious, does this parse the "$20.00" correctly?


Facts module [checks](https://github.com/theforeman/smart-proxy/blob/develop/modules/facts/facts_plugin.rb#L8-L14) for facter on startup, but we do not have it in Gemfile so check fails and facts module is disabled.


fixed in 37344ec. thank you.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 155](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=779&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:41:39


All four of these PNSE's need messages (not just comments), unless they're believed to be completely unreachable on the relevant platform (the exception having come before this point).

If this is unreachable, add a Debug.Fail.  If it's reachable, add a message.


The `setUpClass()` code is copy&pasted into the three test classes.  Why don't you make `Executor` inherit from `unittest.TestCase` and this classes inherit from `Executor`?  They could specify their specific things (the path to ther `tests_conf_file`, the name) as parameters to the `Executor` constructor.


remove_obsolete_entry_guards
`chosen_on_date + guard_lifetime` not always 30 days


don't copy other people's work! do it yourself!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15437](https://teamcity.mesosphere.io/viewLog.html?buildId=132875&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:03


@ProfYaffle, Your opinion please?


Message, not just comment.


@ProfYaffle, here too? :) Thanks.


/fixed see #174


Hey Chris,
thanks for verifying your work again. When I reviewed the Layer_5output file, I was quite surprised, that I only got 5 decoded strings like LAIMER PLATZ from a 2 Minute recording. I thought , that I would get the whole Station ID table as shown in your README.md file. Could you also upload your Layer_5output file for a direct comparation ?

Yes I saved the demodulated subcarrier data in a file and then put that through the decoder via the darc_prototype.py. So the gnuradio blocks, which are located in the darc_testbench (layer2_sync and layer2_format) are subfunctions of the gnuradioblock "layer2" and "layer2_sync + layer2_format + python darc_udp.py" = python darc_prototype.py" ?

I tried to use the darc_udp.py, but it didn't work, as there is no bitstream coming out of the L2 decoding block. I verified it with the following setup:

1.) Random bitstream to UDP sink (gnuradio) -> wireshark (looking on loopback device 127.0.0.1) receives bitstream
2.) Random bitstream to L2 decoder to UDP sink (gnuradio) -> wireshark (looking on loopback device 127.0.0.1) does not receives bitstream.

Klaus


This assert says that the decrypted data can't be the same size as the encrypted data.  I assume that the != was supposed to be ==?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15436](https://teamcity.mesosphere.io/viewLog.html?buildId=132874&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:05:37


I used Github's in-page editor. Apparently it trims whitespace.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15438](https://teamcity.mesosphere.io/viewLog.html?buildId=132876&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:14


Thanks, looks much better now :-)


I see, but i think it should, now I get warnings about wrong parameter type when i'm typehinting against EntityManagerInterface and passing it to this method


@SilverFire I propose a fix for maintain compatibility [here](https://github.com/yiisoft/yii2/issues/9953#issuecomment-182604550) maybe can help


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (982 of 982 tests passed, 0 test warnings)

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


gj!


s /`guard value`/`entry guards number`/


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 1](https://136.243.151.173:4433/viewLog.html?buildId=732&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 1](https://136.243.151.173:4433/viewLog.html?buildId=732&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:00:05


gecos, not gcos


same here, gecos not gcos, according to getpwnam's doco, and what was there before.


**Nitpick - Code Quality**
When breaking an argument list, place each argument on its own line.

You can check out andela's code review guideline to see how this can be done better.
https://github.com/andela/code-review-guidelines/blob/master/style/ruby/sample.rb


@norkunas indeed. Can't do much about it tho, because `AbstractIdGenerator` cannot be changed in 2.x


Nice find!
In Ember it's preferred to use the run loop instead of setTimeout directly, `Ember.run.next` does exactly what you did here.


Andy, `format` parameter is the one to determine whether it is ordered by rows or columns. I think that parameter should be a moose enum, but I will not change it in this PR. `x, y` however is used to denote for function `y=f(x)`. They could be either row or column index.


А маржин зачем убрали? :-)


auhm, can't we incidentally hide some real error?

If this is not the case, +1


jajajja si que se porto el tio enves de madaros a estudias jaja pues parece que con lo que has puesto ahi sera para un 7 como minimo pero que se supone que hace


could maybe also consider subclassing `ReportColumn` with `LocationColumn` (and then I think you wouldn't have to change any other code if you just made a `location_column` property that preserves this behavior).

That would keep this from bubbling all the way up to the top of the `ReportConfiguration` data model.

Not sure if that is harder than it sounds though.


On 11/02/16 16:58, David Precious wrote:
>
> gecos, not gcos
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PerlDancer/Dancer2-Plugin-Auth-Extensible/commit/29066f9ae5e8049c18937f168619383b34bd3384#commitcomment-16036676>.
>
Sticky 'e' key - need a new keyboard.


Ich denke, die sollte besser als Child von App, nicht als child von Object-edit eingehangen werden. Sonst muss jede Komponente, die Messages anzeigen will, die gesondert aufführen.


When option if off, exception is generated in on the fly created class ProjectorGen0 in method doSetup:
work1 = (incoming).getContext().getPartitionExplorer();
When we are trying to get partition explorers from FragmentContext:
{code}
  @Override
  public PartitionExplorer getPartitionExplorer() {
    throw new UnsupportedOperationException(String.format("The partition explorer interface can only be used " +
        "in functions that can be evaluated at planning time. Make sure that the %s configuration " +
        "option is set to true.", PlannerSettings.CONSTANT_FOLDING.getOptionName()));
  }
{code}


finding another value requires looping through the rest of array, its runtime complexity is O(n^2).
try hash table later.


    why

because event handling should be symmetric: more predictable behavior, less chance of state conflicts. events that are each other's logical inverse (activate/deactivate) should also be coupled to logically inverse states.

    a mismatch between activate sound event and unit animation

no.

    if activate is called twice, it would play the sound twice.

the engine would take care of that, but it ain't happening anyway.


had no time, unfortunately, to move chat application to tomcat-websockets :(


What is this?


reack


Yeah no problem, we have to make sure it's compatible and extendable before
switching it over anyway.

On Thu, Feb 11, 2016, 11:01 Maxim Solodovnik <notifications@github.com>
wrote:

> had no time, unfortunately, to move chat application to tomcat-websockets
> :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Red5/red5-server/commit/1ffdb393f9751c36082a5453491a5cac42c774b0#commitcomment-16036789>
> .
>


I didn't do the actual processing in the `Launcher.cpp` Instead, I modified the `ActuateDartCommand::Execute` to do the processing and added a `int`method to retrieve the value of the potentiometer in `Launcher.cpp` also not forgetting to declare it in `Launcher.h`.


test


Ok, Il'l do it


True, will update.


Should the bot remove the `[PATCH]`?


Не перебарщивай с дроблением коммитов:)


очень плохое именование - 1) ты ничего не добавляла, а редактировала уже существующий файл 2) название ничего не говорит о логических изменениях в файле.


Unless I'm missing something, doesn't this lose some of the metrics?  current_connections, global_lock_hold_time and the cache hit/miss metrics jump out as I look over the list we have mapped


опять-таки вместо такого файла надо удалить все лишние и глючащие файлы


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 2](https://136.243.151.173:4433/viewLog.html?buildId=733&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 2](https://136.243.151.173:4433/viewLog.html?buildId=733&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:01


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2116 of 2116 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

![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)


"SAMT activate on flying/deactivate after landing" I agreed to revert to, not the rest.


I had a similar comment yesterday. The `[PATCH]` stuff is fugly so I vote to remove it. If we agree on what words we'll be using I can have the bot do a search/replace pretty easily.

- [PATCH]
- [MINOR]
- [MAJOR]

?


You can do :
```
make([]*flow.Flow, 0, len(s.flows))
```


Thanks. Fixed it in b9cce00. :)


Зачем тебе этот массив?


this probably looks really stupid


"events that are each other's logical inverse (activate/deactivate) should also be coupled to logically inverse states."
not to logically inverse states, but to logically inverse changes:
```
             Landed
        4/            \1
    Landing          Takeoff
        3\            /2
             Flying
```
currently activate happens on 2 and deactivate happens on 3
you changed it so activate happens on 1 and deactivate happens on 4


Gestiona tareas, te registras y puedes crear tareas de una duracion en la cual puedes añadir otros usuarios indicando el numero de horas y labor que van a hacer. Es una mini aplicacion.


Еще раз: посмотри как выполняется чтение JSON из файла в документации библиотеки


what does this line do? looks unnecessary to me.


Refactoring #3

Signature change
Parameters changed
To make it easier to read and more generic


I think we'd prefer to be explicit about the import's rather than doing a `.*` -- what do you think?


@Pilchie Turns out it doesn't matter -- this gets removed later and doesn't exist in the final diff. @CyrusNajmabadi Can you squash away the existence of permissions-related stuff?


sorry, just so the subsequent commit...


hover has the same issues, and it's a logical change which definitely isn't a bugfix


Neat! However, we now primarily use the s0i,s1i representation based on the Vocabulary object and the resulting embedding matrix.  So we should serialize s0i, s1i, y, vocab instead, right?

Could you please do that rather than serializing the XA, XB representation?  Maybe it'd be best placed as methods of the Vocabulary class.

Also, please use a little more descriptive commit messages. :)


I know that some apps are pretty sensitive to dispatch overhead, and that's what the entrypoint rewriting is meant to address.


I will refactor the whole thing anyway soon.


You don't need to do this just use {{ loop.index }}


A dumb question: What is the benefit of this change? Since sensors report hundreds time per sec, I think that we should optimize the report function.


help me please . You can say , how i can make pocketmine.phar in src.zip?


See line 53. The function will not be executed currently, because it's only executed if not null. I don't know if it's safe to change that.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1368](http://teamcity.monogame.net/viewLog.html?buildId=14777&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: ... Develop (Mac) Build time: 00:00:00


sabes que compilador se utiliza para PHP quiero probar tu proyecto :palm_tree:


This is not related to PocketMine plugins.
Anyway, you can refer to https://github.com/PEMapModder/pmwatch#create-a-phar-from-source


Is it necessary to initialize them here? Registers like ip and sp are initialized on startup anyway and I assume for most of the remaining registers it is not relevant which value they have.


hmm, a better way to put it since some state changes are bidirectional:

Before:
```
Deactivated

             Landed
         /            \
    Landing          Takeoff
         \            /
------------------------------
             Flying

Activated
```

After:
```
Deactivated

             Landed
------------------------------
         /            \
    Landing          Takeoff
         \            /
             Flying

Activated
```


@faheus what is the next step here? python mqtt to be added?


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 3](https://136.243.151.173:4433/viewLog.html?buildId=734&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


Ohhhh dang, this is slick!  Thanks @adamwathan


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 3](https://136.243.151.173:4433/viewLog.html?buildId=734&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:00:05


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 4](https://136.243.151.173:4433/viewLog.html?buildId=735&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 4](https://136.243.151.173:4433/viewLog.html?buildId=735&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:00:05


Wouldn't that result in download of the huge map file for all?


One of those times when you spend 30 minutes reading contextual code, trying to understand why `timeout` was **removed** from the test state when it's just started to be used by the runner. Then you realize your brain fooled you and `timeout` was actually being added to the test state. Sigh.

This looks good!


My skype handle is chunky.dunk, should be a profile picture of me when
searching.

On Thu, Feb 11, 2016 at 9:39 AM, mariahcann1995 <notifications@github.com>
wrote:

> How about a skype call tonight?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16034253>
> .
>


Documentation uses npm start and you remove it here :(


This will break the activities tab, which checks for the `isError` value.


Likewise.


Yes. Very ungood :-)


@nf071590 @DaleMcGrew Does this make sense? Any questions or steps that are not correct? I can do a screencast that can be added if need be


> Do we need to reset?

If you can commit some quick adjustments so that master isn't broken, then no, not necessarily. It's just a bit messy... usually you'd maybe want to clean up your feature branch (squash commits, maybe rebase) before you merge.

Anyway, your call. If you prefer not to reset, then we don't reset.

> The reason this even happened was that I was going to make Fighters4 since it didn't want to cleanly rebase against master without adding / subtracting mystery commits.

Hm, strange, because the branch *did* get rebased before it has been merged. Hopefully nothing got messed up, because if what got merged is the result of an unsuccessful rebase attempt, then what you have in master now is not a clean merge of the fighter branch. To be sure just build and run current master, if everything works the merge should be ok (otherwise I'd expect build errors/crashes).

> I'm not sure what constitutes a "fork" as you describe it

A clone of the main repo. With guthub repos, you usually keep such a forked clone in your personal account. Creating one is easy, just go to the repo you want to fork and press the "Fork" button in the upper right. This will automatically create a clone of that repo in your account.

Then you just have to tell your git client that there is an additional "remote" of the repo (don't know how to do that with TortoiseGit though, but I could probably give you the required command line statements), and you're set.

> but if it decides that I meant to push to the main repository master instead of a branch, then won't it still push to master regardless of whether there are other repositories in other places?

Well, theoretically yes, but if you make your fork the default remote (usually called "origin"), then your git client will most likely push to the master branch there. You'd have to deliberatly specify the main repo (usually called "upstream") if you want to push your commits to it.


Yep, will do


Please branch out and not push directly into dev unless specified. :+1:


If 5080 is still present I'll fix it some other way


**Nitpick - Code Quality**
Ruby's string class has a method `empty?` to check the availability of a value.

So, you'll have something like:
```
if params[:date].empty?
```


Or are these for parsing, not serialising?


move calc to directive code


Yay!


events no longer being spammed *is* a bugfix.


Hi Nichole. Looks great! You might want to pull the changes that I made from upstream to deal with subtracting background. If you don't do that, the growth rates will not quite be correct (as Phoebe & I discovered yesterday). To pull changes from my repository into yours, use: https://help.github.com/articles/syncing-a-fork/


I also thought of doing it by reflection.
I think we'll add `getApplicationContext()` instead of `getActivity()`. Would that be okay ?
This is because PApplet won't always be inside an Activity now.


k, I replied before you edited "(since it reintroduces 5080)" in


Para tirarlo necesitas un servidor Apache con PHP (yo usaba el XAMPP, te instala el Apache junto con el PHP) y luego una BBDD de MySQL. El script de SQL viene dentro.


yeah, I was about to say that 2 isn't the only transition where activate happens, and 3 isn't the only transition where deactivate happens.


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 5](https://136.243.151.173:4433/viewLog.html?buildId=736&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 5](https://136.243.151.173:4433/viewLog.html?buildId=736&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:02


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 6](https://136.243.151.173:4433/viewLog.html?buildId=737&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


Is this outer div necessary?


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 6](https://136.243.151.173:4433/viewLog.html?buildId=737&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:02


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 7](https://136.243.151.173:4433/viewLog.html?buildId=738&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 7](https://136.243.151.173:4433/viewLog.html?buildId=738&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:02


@faheus  I question 'the last one wins'
This is nondeterminisic. Please discuss other conflict resolution or conflict avoidance strategies.


my point is that both decisions are logical


As far as I can see, this file isn't necessary.


Probably not any more.


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


I thinks `selection()` and `select(stock)` should both be preface with `searchCtrl.`


I can't tell what you've changed. I thought I was using Unix line endings already.


You could change `func ExecCmds(t *testing.T, cmds []Cmd)` to `func ExecCmds(t *testing.T, cmds ...Cmd)` to be able to reuse it in `testTopology` of `tests/topology_test.go`


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 8](https://136.243.151.173:4433/viewLog.html?buildId=739&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


it's a local variable m8, it'd be out of scope on a reload unless you made it global


I've already pushed a few more commits that make things work as though fighters didn't exist in the code, mainly be removing them and Flak from preunlocked items and starting fleets.

I think what happened today was I rebased against master (by hand, manually resolving conflicts in a couple project files), then pushed, thinking it would push the new branch, but had it left with master as the remote branch instead of blank, so it decided to push to master instead of pushing a new branch on github.


On 0.13.2 I just get a Outdated error also in the console


AFAIK Thread_reg_id isn't necessary anymore.


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 8](https://136.243.151.173:4433/viewLog.html?buildId=739&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:18


Given that change, it looks like `searchCtrl.selection() !== stock.code` could become a method on the `searchCtrl` that looks like `searchCtrl.darkClass(stock)` or something.


is that just for clarity?


would error on legacy pages since `page` is not within the context for them


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 9](https://136.243.151.173:4433/viewLog.html?buildId=740&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 9](https://136.243.151.173:4433/viewLog.html?buildId=740&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


I agree, explicit is better than doing the wildcard, that must've been done automatically by IntelliJ with its 'optimize imports' feature. All I intended to do in this class is change how it addresses the resource bundle since I moved the test resources. Good catch


TeamCity Azmq :: Build (Linux) [Build 159](https://136.243.151.173:4433/viewLog.html?buildId=741&buildTypeId=Azmq_BuildLinux&guest=1) is now running


surprising? methods is* usually return a boolean and are side-effect free.

what about a simple assertNotNull?


It is mostly for clarity, yes. Before, in the directive, that is what the functions were, but now they're actually functions on the search controller.


not quite because one allows more state conflicts than the other.


how?


That was my thought exactly, but when I aligned it like this, rubocop complained. :by_route and lambda are parameters of the scope method and thus should be aligned when they span more than one line. A more appropriate indentaion would be
```
scope(
  :by_route,
  lambda do |from, to|
    where("departure_id = ? AND arrival_id = ?", from, to)
  end
)
```


Be careful, this leaves the token, while previous code did not... the token is for one use only, that is why it should be deleted here


@richaagarwal  we still need the `image()` as wagtail needs a way to render wagtail uploaded images. so instead if created a conditional for legacy pages & wagtail pages.


your "landing on a moving carrier" example.


Sure, I've tested them pretty well at this point.


qualifiedNames?


you need to create a 6.0 branch from cm-13.0
this branch is waaaay behind


it's ''HELLO''


TeamCity Azmq :: Build (Linux) [Build 159](https://136.243.151.173:4433/viewLog.html?buildId=741&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:01:29


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 10](https://136.243.151.173:4433/viewLog.html?buildId=742&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) is now running


Is there any reason to keep only: [] here?


TeamCity Azmq :: Build (Linux) - gcc-4.8 [Build 10](https://136.243.151.173:4433/viewLog.html?buildId=742&buildTypeId=Azmq_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:00:05


@jimmynotjim full width check for browse page


what about subpackages?


updated legacy pages to account for hero changes


assertNotNull is a junit method.


@jimmynotjim Browse page specific full width font check


**Nitpick - Code Quality**
There should be a new line before this line. Appropriately adding new lines helps in code readability.


Thank you! I have cherry-picked this commit to master now.


we can have our own assertNotNull then


nill → nil


... and other 32-bit platforms :)


Good catch John, thank you !


Dude, you switched them....


datetimepicker


Sorry, I guess I should have merged - I got confused by the other commit and thought it comes *before* this one!


If hw_riscv is able to  run a generic test like printf already, we should remove this.


**Nitpick - Code Quality**
There should be a new line after this `if else` block. Appropriately adding new lines helps in code readability.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15439](https://teamcity.mesosphere.io/viewLog.html?buildId=132881&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:15


Nitpicking: Please put spaces around =.


Invalid reason format.


Before it was a string, wasn’t it?


fast-forwarding riak_ts-develop-1.2 to borshop-integration-1347-at-remove_claimant_call = 6278716c


This fixes the wrong powered bug?


Any reason to remove this?


Welp, we need to delete this entire repo and begin from scratch due to sensitive information being present that we cannot cherry-pick for deletion.  Will take care of this when I return.


We cannot have two types with the same name in a package.


You also need to bump pkgrel.


And whats up with the lever under an block powering


Yes and `isNotNull` rethrows an exception.


Run php compile.php --out $OUT ? - where is this?


Reason doesn't function with _/stfu all [reason]_


It would be nice to somehow be able not to use sm, but change it to md or xs even.


We should document somewhere how to set things up - where to get the skipthoughts module, where to download a precomputed model, or how to compute one (and we should publish one, if you give me a Metacentrum path I can put it up somewhere).


Is there no plugin for Jekyll which translates usernames into gh links automatically?


Also we need to start forking/branching this shit for mix-ups like this rather than commit-pushing to master like a rookie.


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 1](https://136.243.151.173:4433/viewLog.html?buildId=743&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) is now running


Do we need to add a test for blank data?


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 1](https://136.243.151.173:4433/viewLog.html?buildId=743&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:02


ok, got rid of the "pipe and filter" approach.


Make a fix for this


Also; Instead of illuminate/html I updated to laravelcollective/html.


AFAIK, Cpu_lazy_state isn't not needed anymore.


Frankly, it also appears that we need to ensure that our .gitignore file is accurate for every clone/pull.


I think we could create a username to GH link script and set it as a pre commit hook


............. The tests passed because it was inlined?...............


What does this do?  make python think the default encoding is utf-8 for everything?  that seems great (ie. like python3)!


fixed earlier


0.37.2 was just released today if you'd like to keep up-to-date


`StringUtils.join()` already supports sub-arrays. ArrayUtils is redundant here.


And you need also check for `pod.Spec.Affinity.PodAntiAffinity.RequiredDuringSchedulingIgnoredDuringExecution`


A usermode...


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 2](https://136.243.151.173:4433/viewLog.html?buildId=744&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) is now running


Can drop this commit now that most of the stuff it was originally doing has been ported to other commits


How is increasing the time it requires to wait until after going to produce less test flakiness?


:100: thanks dude!


I see, although it's still a bit strange. Usually, when doing a rebase, the checked out branch is the one which gets rebased, not the one rebased against. A subsequent push should go to the remote branch that is tracked by the rebased branch (which will usually fail unless you force push, which is why you have to do exactly that after a successful rebase), or create a new remote branch.

But maybe TortoiseGit does some things a bit weirdly...

Anyway, one thing that won't work now is building on OSX and Linux, as the Xcode project and cmake files most likely will have to be updated accordingly (adding Fighter.h/cpp to common as far as I saw). I'll try to get around to that ASAP.


Talked to @jklein24. He's going to work on making the doc available (either the doc itself or we'll copy most of it into a new doc), hopefully within a day or two.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15440](https://teamcity.mesosphere.io/viewLog.html?buildId=132883&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 225 Build time: 00:07:13
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 2](https://136.243.151.173:4433/viewLog.html?buildId=744&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:17


TeamCity Ppconsul :: Build (Linux) [Build 100](https://136.243.151.173:4433/viewLog.html?buildId=745&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


Good point - I'll make that change.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15442](https://teamcity.mesosphere.io/viewLog.html?buildId=132890&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 227 Build time: 00:04:04


way better. Thank to @FrancescoCioria :+1:


It seems that in_fault can be removed.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15441](https://teamcity.mesosphere.io/viewLog.html?buildId=132885&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:45


5420-OSCM-Docs (mitchute) - x86_64-MacOS-10.9-clang: OK (2078 of 2078 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

![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)


Should we rename this, too?


Seems to be complaining about a missing second argument to `rebase`, which should be inferred to be the current branch. Rather than redirecting the output and ignoring the error, I believe this can be fixed by adding `#{remote_ref}` after `origin/#{remote_ref}`


:crown:


Gemnasium lied to me then! (I mainly upgrade to avoid the "dependencies out of date" badge)

Thanks for the heads up!


It seems that translation_added can be removed.


Your assumptions about the block structure is correct. The layer2 block consists of the two blocks layer2_sync and layer2_format.
darc_udp.py does not do layer2 processing (it assumes it is done in gnuradio), whereas darc_prototype.py does layer2, therefore it is not allowed to do it in gnuradio.

Putting random bitstream into the L2 decoder won't output anything, as it will probably never detect a valid L2 packet from random data.

If you get
Laimer Platz, Fuerstenried West, Krokusstrasse, two times Lochham
when you put in the samples I uploaded for you, then everything is working fine.
The end stations of a bus line are only broadcasted once a day, so this is as much data as you can expect from 2 minutes recording. In between, service data and the actual ETA times are broadcasted in L3 and L4

The table in the README file was mainly built by walking around and looking at the signs at the bus stops, not by the files I gave you :)
We did receivings of 24h length and more, but we directly demodulated and dissected the received signal, so I think we only have the layer5 output of these sessions (somewhere...).


of course – it's a simple one-liner function :)


TeamCity Ppconsul :: Build (Linux) [Build 100](https://136.243.151.173:4433/viewLog.html?buildId=745&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 00:02:34


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 3](https://136.243.151.173:4433/viewLog.html?buildId=746&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) is now running


looks great Shannon, thanks for the review!


I've reverted the `TaskSummary` field names in https://github.com/apache/brooklyn-server/pull/19


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1369](http://teamcity.monogame.net/viewLog.html?buildId=14781&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 583, ignored: 6 Build time: 00:17:14
### Failed tests
```
MonoGameTests.exe: MonoGame.Tests.Visual.BlendStateTest.VisualTests: <no details avaliable>

```


Yeah, I think you jacked up the wrong value :)


SucceedsWithin does a tight loop. On a very busy box, we may not get to it quickly enough. Allowing more time for it to run helps (the important part is the timeout for SucceedsWithin, not so much the actual duration)


I don't think this `version` should be changed.


Nice!

Don't worry too much about my old work it was ugly and messy.

Mindeunix's configs are awesome but I like the tyrannical-based approach better.
I'll drop wirefu for now but I think I'll continue to just try and make your old config work until the new version comes out.


TeamCity Ppconsul :: Build (Linux) - gcc-4.8 [Build 3](https://136.243.151.173:4433/viewLog.html?buildId=746&buildTypeId=Ppconsul_BuildLinuxGcc48&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 00:01:03


Really new to programming, what is the stdio.h header used for?


It seems that flush_tlb_by_pid can be removed.


Cool, I'm not sure what I did with the includes is best, so feel free to play around with those.


so my sublime text cleans up white space on save and that made this one hard to read. Magic is on lines 1041 - 1043


Oh, I see, you're just making the test 10x slower and thus less likely to fail. LGTM


OK


# Misc
*.properties
*.MF


Cool.  LGTM


@andy-armstrong my bad on this one.  You actually want "Content" to be the aria-label here, not Main.  I spaced on that one.


yeah. I'm generally not a fan of increasing timeouts just for tests, but 1ms and 100ms is pretty short, and that is the right way to test this function.


The li looks fine.


ok


Ok


It seems that prepare_proceeding can be removed.


I'm a little confused here. map_tokens() with ndim=1 returns bag-of-words embedding vector of the whole sentence, with ndim=2 returns list of per-token embeddings.  But with skipthought, we are generating a whole sentence, is that right?  So that'd be what's returned with ndim=1, but with ndim=2, what do we return?


I understand it correctly, is no longer required to specify the attribute of agro and threats?


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


Yes, but one can also use StartMoving() and StopMoving() to get info about the landed state, minus some minor exceptions such as movement due to shockwaves. #5080 can also be worked around with a widget issuing fly state to units who are guarding, although that's probably not desirable in games with limited fuel.

Why not just check if unit has commands before calling Deactivate?


Could we get some boostrap information here? :-)


@hwmaier why

`$data = @yaml_parse("---\n" . $data . "\n...");`?

`$data = @yaml_parse($data);` works fine for me.

Just asking if you remind why it's adding `---` that.


@sergiocano91
Revisa el código


@chuchurrio
invitate a algo


It seems that this can be removed.


`isVisualNullIncluded`


Ok we need an explanation here how these tests are supposed to work! the workflow.

This test should be started from the root directory or called by another one from the root directory


we generally avoid line breaks between return types and function names in systemd... see coding_style


@jesusAlbarregas
Ok


@springjools that's (more or less) how it's fixed in the latest commit


And this.


spurious newline


@LeeMcNeil this includes the execution time of all commands. Plugins running SQL queries need to be doing it asynchronously.


If it can't be fixed in git just an explanation why we do this then


TeamCity Azmq :: Build (Windows) [Build 185](https://136.243.151.173:4433/viewLog.html?buildId=748&buildTypeId=Azmq_Build&guest=1) is now running


`isNullIdIncluded`


TeamCity Azmq :: Build (Windows) [Build 185](https://136.243.151.173:4433/viewLog.html?buildId=748&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:00:11


**Nitpick - Code Quality**
You should always adopt the **Skinny Controllers, Skinny Models** pattern. This should be moved to the model or a `poro` class if the need arises.

More on **Skinny Controllers, Skinny Models** pattern https://robots.thoughtbot.com/skinny-controllers-skinny-models


in C functions without parameters take a `(void)` parameter, not an empty `()`. That is a C++ism, which does something different in C...


`getVisualSeparator`


same here...


Was something that was just copied from the above file modified (MarkdownFile.php), and here is not actually needed?


`isNameNullIncluded`


If we are intending to run this on a given X distro or host we must make sure that the ssh-agent is already running or loaded, and perhaps after the test remote the key


What would I do with that json string then ? validate method expects an objectnode instance.
Building a valid ObjectNode that would later throw when deserialized seemed too complex for that purpose, so I chose to revert to using a mocking lib.


@chuchurrio
hola que tal


definitely works, all tests pass.


And this.


not sure i follow what this does, but shouldn't the description parameter be "const"?


@marioalguima
hoy no, mañana


upstream reload doesn't use this method yet, it was added to prevent forking on reload


Now this option is more a preference of the actual matcher than a user setting. What about removing it and always clear the descriptors?


The jekyll-mentions plugin is limited and kind-of broken. Names can only be in lowercase, and two  tags are created:
@sphinxc0re


this->viewports->at


Patch LGTM; simplifies the test a lot, which implies less maintenance in future: fewer ways to go wrong


LGTM


Great :)

My electric counter does not send the "PAPP" label, so Domoticz was not able to read the usage current.


Can we make this interval longer  like a week.


ok, sorry for the noise


I don't like this idea too much - it seems to me it'd be better to have a consistent API where we always work with a list of tokens, and when we use skipthoughts (perhaps *in addition* to per-token embeddings!), we could ' '.join(qtext) back.

That is, if skipthoughts.encode() can't take a list of tokens on its own, which is kind of weird. Do they split by spaces, or do more tokenization?  (For example, I errorneously re-tokenized using nltk, which caused '<NUM>' in source dataset to be split to three tokens '<', 'NUM', '>' in our internal repr.)


renamed.
For the record : Seen === NOT IsUnread, leading to lots of confusing namings and conditions expressions !


The views are redundant.


window.opener code broke it in some cases, but others still worked


Thank you for the review @agcolom.


invitate a algo no?


- Full visibility into all server metrics
- Billing: When Server Monitoring is deployed alongside Application Monitoring (APM), New Relic bills per server instead of per container (cheaper).


why `int32_t`? Use `int` for simplicity or `std::size_t` as a proper size type.


enga no seas rata


agreed, should be another dedicated test fixture ideally. removed


que se que tienes dinero cabron


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15443](https://teamcity.mesosphere.io/viewLog.html?buildId=132891&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:08:39


:dancers:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15445](https://teamcity.mesosphere.io/viewLog.html?buildId=132897&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:03:52


I was pretty sure I tested this, too... :/


now it looks like a pretty bar chart


Move to .cc file, remove unneeded includes, space after `switch`.


`getNameSeparator`


`getIdSeparator`


Why is verbose set to true in our implementation? Is this needed to get all the audit data?


`getTypeNameSeparator`


```
cd MyCustomPocketMineBuildDirectory
# if you run `ls`, you should see a src directory here
php compile.php --out MyPm.phar
# now a MyPm.phar will be created
```


You could create a script if you want, but creating a link manually about once in a month isn't too hard I think.


remove `std::string`


Thanks for the response. I think this may be it. I will post the outcome after we modify out code.


Can you also check that this will get placed in the right position in the broader list?


Me aburro...


https://github.com/rdfjs/rdfjs.org/wiki/Relevant-Specs
https://web-payments.org/specs/source/ld-signatures/


good catch. The name was pasted unchanged from a previous PR comment!


looks fine to me, @overtron @jportwood any comments/concerns?


**Nitpick - Code Quality**
There should be a new line after this block.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15444](https://teamcity.mesosphere.io/viewLog.html?buildId=132892&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:28


Technically another race since multiple goroutines are setting a variable. Please set this once in the Connector function.


Why int32_t?


you say that like the inlining pass has some kind of rational behavior


I don't know when the Logstash folks will review the change. Hopefully within a week or two.


@petebacondarwin, since `.url()` includes the hash, shouldn't we ignore it ?
Isn't the router actually only interested in path + search ?


Couldn't we use one-time binding for `$router` to spare a few watchers ?


**Nitpick - Code Quality**
There should be a new line before this block.


I don't believe the vertex depth and pitch are just `m_index` like in texture. They *might* be `(m_index * 8)` like the other vertex ones.

My reasoning is
https://github.com/RPCS3/rpcs3/blob/791070c15811a3ebcd6bd1b9be9ebf2a34f18c27/rpcs3/Emu/RSX/GCM.h#L788

`NV4097_SET_VERTEX_TEXTURE_CONTROL3` evaluates to 580 (base 10), and
`NV4097_SET_VERTEX_TEXTURE_FILTER` evaluates to 581,
so as you move 'up' in textures, you end up grabbing the wrong registers

The Texture depth and pitch seem to be different as they leave a 'space' of 16 between
`NV4097_SET_TEXTURE_CONTROL3 = 1552,`
`NV4097_SET_VERTEX_DATA2F_M =  1568`


And as in the build script we should also add the possibility to pass ldflags so some tests can succeed and probably have to build fleet before


it happen for every create action against each resource. disabling of this code lead issues. please verify properly before deploy this code.


LGTM


What if you moved this up to the body? that way the border goes around the entire page, and you can add additional margins in there.


test Test Tect


When this fix will be delivered? What version?


This got renamed to `fetchMapService`.


Thank you Rob for putting together this process. Can we discuss it Saturday morning before enforcing it? I'm right in the middle of getting all of the basic pages built and checked in for the Feb 23rd MVP and would prefer to work out the kinks when we can work together.


Hay que tratar de mantener la confianza en la gente. Y si alguien se sale igual se elimina y en el control de versiones queda registrado igual que estuvo y hasta cuando.


Space before 128, 80-chars limit.


scope test aussi


Rob looks good to me, I don't know much about this process yet. I can follow it and then let you know what I think. Do you know how to fork a repo from git on terminal?


Muy bien, tiene sentido


test data


Hmm what you can do probably here is add a small config that shares definition then source it from both build test and so on... ?


Test


Done.


Please check what other sfm code uses as ID type. `int32_t` is literally used nowhere.


we do not need to constrain zeppelin to use only wss b/c it does
(location.protocol === 'https:' ? 'wss:' : 'ws:';)


Cool. Makes sense. Updated.


Esqueci desse detalhe, hahaha!


The code does not comply to PEP8.

```
<Result object(id=0xbe3c228f2df1494bb22b665852a67976, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmp6i7nqu18/test.py', line=2, column=None) at 0x7fa6795c4b70>, end=<SourcePosition object(file='/tmp/tmp6i7nqu18/test.py', line=2, column=None) at 0x7fa6795c4a90>) at 0x7fa6795c4908>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7fa67961c908>
```


The code does not comply to PEP8.

```
<Result object(id=0x75a6d9e49cae4d0ea06c9510b739c933, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmp6i7nqu18/test.py', line=4, column=None) at 0x7fa6795a8668>, end=<SourcePosition object(file='/tmp/tmp6i7nqu18/test.py', line=4, column=None) at 0x7fa6795a8438>) at 0x7fa6795a8710>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7fa67a22fb38>
```


The code does not comply to PEP8.

```
<Result object(id=0x3df2a654c7a0453cbceabfaca68e7d66, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmp6i7nqu18/test.py', line=6, column=None) at 0x7fa6795c4b70>, end=<SourcePosition object(file='/tmp/tmp6i7nqu18/test.py', line=6, column=None) at 0x7fa6795c4eb8>) at 0x7fa6795c44a8>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7fa6795c4400>
```


haha :D


Um pequeno detalhe :P


What is this include for?


Please remove if not needed.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15446](https://teamcity.mesosphere.io/viewLog.html?buildId=132900&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:03:38


Funky alignment ;-)


any reason for all these reverts?

statusbar lag?


It's in nightlies of master and will be in 2.14


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1370](http://teamcity.monogame.net/viewLog.html?buildId=14785&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:20:37


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


Merged here, this time for real.


@cra0zy & @leezer3 - I made that list based on the people I who made positive comments here.  If @thefiddler  gives you more administrative privileges than you want, don't use them - isn't that how a community project should work? I'm tempted to say everyone who regularly contributes code should be an admin - but that may be toooo chaotic.

@leezer3 - I hear you about not being  happy commenting on impacts non-obvious pull requests may have on something I haven't thought of -- that's why I said I'm not sure if I'm worthy to be a full admin b/c I don't understand the full system, just certain parts - but someone has to do it.  The reason I asked @thefiddler for  at least push permission to assist with pull requests is b/c it's actually more dangerous to have too few people w/ privileges & then if they quit  like @Frassle or are busy at school/work like @thefiddler  then the project gets frozen :(

Remember github has ways to roll back after a bad commit & unit tests can help test code quickly.  http://www.opentk.com/doc/chapter/0 has the simplest possible OpenTK 'game' - we can put something like that in an NUnit test project w/ the nuget NunitAdapter & it will run in Linux to test if the graphics code is working (which is actually kind of like the Examples programs that already exist) & simple things like the math classes can have traditional unit tests w/ assert statements.

I agree w/ @ilexp about being more comfortable trying to fix the OpenTK brand than trying to start a new one.   Remember, even if we did a radical break away & called this "SuperTK" to something, we'd still have the existing bugs in the source code & they have to be dealt w/ either way, so I think it's better to stick w/ "OpenTK" & rehabilitate its reputation as we kill off the bugs


Isn't the hot soft for search deprecated?  (It's not listed on the main site anymore)

Also, there _is_ relevance2, but I'm not sure if it would make sense to include that.


I'll fix it for the next release


Can you use `~ExhaustiveMatching (void) override = default;` in the header?


OK, fair enough.


@joshuaknox hmm okay, I will take a look, thanks for letting me know!


We use  hash_from_dict(self.inplace_pattern) to hash that already in the Elemwise class. To be sure it work with older python version, I would reuse that.


@crionez: This is the missing fix for the clang build.


FWIW, I'd have to inspect the backup to tell what version of Modality* I had on my disk, but
for a test I removed all four quarks (all Modality, UnitTesting) and reinstalled *ModalityTests* only.
Now this is what I got in the post window, which wasn't the case before:
```
-> a QuarksGui
Installing ModalityTests
Installing Modality-toolkit
Adding path: /Users/mz/Library/Application Support/SuperCollider/downloaded-quarks/Modality-toolkit
Modality-toolkit installed
Installing UnitTesting
Adding path: /Users/mz/Library/Application Support/SuperCollider/downloaded-quarks/UnitTesting
UnitTesting installed
Adding path: /Users/mz/Library/Application Support/SuperCollider/downloaded-quarks/ModalityTests
ModalityTests installed
```


Is this doing the right thing?


Oh, yeah, it is a bad flag then.  We should be asking `empty?`  The goal was to catch error if we had issues w/ the lookup service due to our API key being a free acount.


All well and good to add a deprecated notice, but what is recommended instead?


Merged here.


We should probably print a helpful message here, something like:`fprint(stderr, "Could not resolve a path for the system libGL.so library\n")`


I wrote some crappy non working code on Tuestday. It lays on the openfabbster pi.
Lessons learned: don't code if you are sick, its not funny.
Will work on it if im up and running again.


Yep!

On 11 February 2016 at 08:52, Georgios Kalpakas <notifications@github.com>
wrote:

> Couldn't we use one-time binding for $router to spare a few watchers ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/petebacondarwin/ng1-component-router-demo/commit/f2cf47116095dab2092c57d295228edb508719b0#commitcomment-16038529>
> .
>


5420-OSCM-Docs (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1534 of 1534 tests passed, 0 test warnings)

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