#	From Group 1

##	Things you did well:

*	You did a wonderful job in vectorizing the code after using profiling tools to detect
	the computation bottlenect. In specific, it is great to see that
	*#pragma ivdep* works well in certain loops. And we find your discussion of reasons why
	this annotation does not work in certain loops very helpful.
	Moreover, you give clear and well-formatted
	numerical results and plot of your scaling study in vectorizing your code.

*	You give a clear and reasonable parallelization scheme.	It seems that you have a lot of
	good ideas in parallelizing the code and had a paragraph of discussion in great details.
	We learned quite a lot from your reasoning.

##	Things you should try:

*	We think that there might be some room for improvement in vectorizing the code by
	playing with various compiler flags. It seems that you did not exploit the use
	of compiler flags, yet they can sometimes offer computational boost. Also,
	I am not sure if you have taken a look at the vectorization report. It contains
	useful information of why certain attempts of vectorization fail.

*	It would be great to see your progression in parallelization. Your motivation and argument
	in using parallization is in great details. We look forward to seeing more details and
	numerical results of your implemented domain decomposition.

*	In the parallelization part of the report it would be better to see some discussions
	about weak and strong scaling.
	Spicifically, even when there is no numerical results available for plot, it is beneficial
	to talk about the potential dependencies in both scaling studies.

*	Professor Bindel's implementation in C code had a computational boost. It would be great to
	see you reconstructing your C++ code based on C code.


Here?


In der editorconfig kann man dafür übrigens "trim_trailing_whitespace = true" setzen.


I also use the idea of

```
point estimate + (something that depends on `alpha` and the shape of the sampling distribution) * SE
```
then computing `qnorm(alpha/2)` reinforces both the `alpha` part and the shape part. The move to `qt(alpha/2, df)` should only further reinforce this.


@mmurooka
なぜこれが必要なのか（robot_self_filterがaptで入らないから）をコメント書いておくといいと思います。


where is in-my-city.jade file?


yep from the headers in a PO file

atm this means you cant import non-po files

which is the next issue i need to address


aca faltan muchos comentarios, pero además me parece que estas funciones deberian ser macros...


That makes sense. I also created methods for hot and grayscale palettes, because they can be use for other applications in the future.


Can you release a new version? :)


Done!


Проще, но так как со стилями еще не силен решил попрактиковать)


No, it allows minor enhancements inside the release (by adding extra fields at the end of the structure)


Yes


I looked into this some more and I don't believe you can use self.load(). PostgreSQL has to re-connect each time you want to access a different database unlike MySQL. Looks like the solution I came up with in the create_database function is the same solution you found in select_database().


It would increase flash usage on 9X!


mejor seria #if CIAAPLATFORM_BIGENDIAN == CIAA_ENABLE o asi. comparar con 0 ademßas es peligroso ya que un macro no definido es por defecto 0.


Not exist, i commented this line because this, jade error.


aha! :)


@bai this commit seems to broke rubygems.org travis builds. I checked that you use ethon and ffi gems inside of rubygems.org search2 branch with this commit. I'm not sure what is the solution but @dwradcliffe used elasticsearch-model and elasticsearch-rails gems in search branch without updating this repo.


Просто в некоем роде это не правильно.. хм теги не прописало которые наводил как пример.."<em>"      и "<li>"


Sorry in advance...but I would make these instance variables again, so that they go against `INSTANCE` instead of the class. The way it is right now, you have a static method that's loading values from a file and storing them in static variables, which means that `INSTANCE` serves no real purpose. The fields themselves are not actually associated with the instance in any way (so `INSTANCE` is blank). I would keep the static method to load the properties from the file, but have them be loaded into instance fields on `INSTANCE`. Static variables are also subject to different rules with respect to classloading, so in a scenario where you reloaded this class within the JVM, you might run into some issues (like in a webapp). My Java is a little rusty though...this is my only suggestion though! Other than that, LGTM - I would just stick with instance fields.


el codigo se ve mucho mejor sip... ahora ya si lo hiciese lo abstraira todavia más con un

TEST_SETVERSION(config_buffer, version) y el macro del test se deberia ocupar del resto.. pero bueno, segui adelante y luego lo podes ver que preferis...


@timwangdev Why should this be broken? It works for me.


:wave:


top


Meant to say "Fixes #3 and #1" issues.


falta un enter.


deberiamos charlar en firmware que queremos y como sean las librerias genericas. me parece el enfoce que le estan dando es muy especifico.


no entiendo por que el copyright de todos si los commits los haces solo vos Pablo. pero como prefieran.


What about https://github.com/dhananjay92/arewefastyet/commit/9b2233#diff-c78c912ff4dafaea903e386ebd2f36d9R27 ?


esto esta mal, falta una mascara o un cast a uint8_t, ademas en la delcaracion me parecio ver uint8_t * config_buffer


question : how did this not make doctest fail ?


Third example usage within TravisCI https://github.com/peterjc/pico_galaxy/commit/6d20c30fd4821397e6da5c109c668a80e214234a and https://travis-ci.org/peterjc/pico_galaxy/builds/85548438

Much like the previous examples, this uses ``planemo dependency_script`` on several ``tool_dependencies.xml files`` to install binaries then used to test the tool wrappers.

Fourth example usage within TravisCI https://github.com/biopython/galaxy_packages/commit/91c4d21e959ac19abbb270ff0bd9897ce31ad2f0 and https://travis-ci.org/biopython/galaxy_packages/builds/86869381

This example is a little different to the previous ones. Here the repository only contains Galaxy dependency packages, and ``planemo dependency_script`` is used essentially for minimal validation of the ``tool_dependencies.xml`` files (and to check their download URLs are working).


This would create duplicate cursors for series that have more than one tag selected in the group by. Might be a substantial overhead on pathological data sets.


If that's the case, we'd have to add the name of each downloaded file to a list, remove them, and then try rmdir().
If there are other stuff belonging to the user, the directory won't be removed.


Won't look right and is not square


6 in one hand 1/2 dozen in the other… its temp code for excluding sections that we aren't currently displaying.


First, I would like to thank you. It's hard to get access to OS X for me and I didn't notice these abnomalies on OS X.

I spotted an error in line 452 of your patch set:
+	local os_lang = os.getenv("LANG") .. os.getenv("LC_ALL") -- one of both should exist

The comment is true, but if one of the values is NIL, then the VLC Lyrics Finder script crashes.
lua debug: [Lyrics Finder] Activating
lua warning: Error while running script /home/geoffrey/.local/share/vlc/lua/extensions/lyricsfinder.lua, function activate(): ...eoffrey/.local/share/vlc/lua/extensions/lyricsfinder.lua:455: attempt to concatenate a nil value
lua error: Could not activate extension!
lua debug: Deactivating 'Lyrics Finder'
lua debug: [Lyrics Finder] Deactivated
lua debug: Extension thread end: 'Lyrics Finder'

This is standard Lua behaviour. Please store them in seperate variables and check if they're nil before assigning them to the correct variable (use "if"). You can't compare to nil either, so if a variable is not nil, you check it like this:

if VARIABLE then
  --the variable is not null
end


maybe collect these into a collection?


it looks pretty much like mine. good job!


書きました．


```didsabled```?


this should say URIs for points-of-contact -- mailto is supported, right?


In kernel `4.1.%` at least one new DTB has been added (for the official Raspberry touch screen)
Maybe we can add some thing like :
>> KERNEL_DEVICETREE_append = " overlay/rpi-ft5406-overlay.dts


Yes, you're right, this belongs to `platform/galileo`


should also say URI


hadoop provides FtpFileSystem, S3FileSystem which are extension of FileSystem and any of those can be used based on the scheme in the path.
When there isn't any scheme then default fs is used which on a hadoop cluster is hdfs.


Symfony 2.8 changed index to block_prefixes


:clap:


Ah, gotcha.


Regarding the set, you implemented, you should still put the type of the elements into an template, allowing the use of the set also for e.g. Generators, other sets, groups...

Then the class can be quite useful for many different, tasks, however there may still be cases (when the subsets are sufficiently small relative to the whole set in which my implementation makes sence).

Thus I  suggest you to continue your implementation, I will rename my class and continue as well and finally we can use both.


Where should I place it?


your last plot looks like mine. I also plotted bar with x axis of each partition and y axis of efforts. good job!


This change does not mean that.

This one: https://github.com/munki/munki/commit/9e0deb754083c671f8005f05e7ec44fbe001b790 , back in July means that.

softwareupdate in 10.11 refuses to use file:// sucatalog URLs. So the change back in July removed Munki's use of those for 10.11

softwareupdate in 10.11 _really, really_ hates changing the CatalogURL at all. If you change the CatalogURL, it seems to take softwareupdate minutes (hours?) to be happy again. Today's change removes yet one more catalog change.

See https://github.com/munki/munki/issues/511 for further discussion on this (and you participated in that discussion...)


I don't think we need your `.project` and `.cproject` in the repo.


@ahugeat asked to make them stop when they're close enough. So they stop if there is less than the half of their width between them and the character. We'll see if we keep it or change it later.


OK


good explanations!


the top one is the same as mine. You did more than enough!


good! the same as mine!


Conflict tested


This is our number of tries, just like a timeout. I'll change this code to a clearer one.


looks good!


most important part is missing ;)


Sorry I just got to Pittsburgh

On Thu, Oct 22, 2015, 1:22 PM Doug Breitbart <notifications@github.com>
wrote:

> Conflict tested
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Peeragogy/peeragogy-workbook/commit/c6892184293ea97ee5e945be878cb13a64146084#commitcomment-13933260>
> .
>


the solution could be simpler, and e ... why I did not use a map or fast pointer ....


That function only returns the origin data include rotation however it could be modified to return the internal ID but it shouldn't return -1


@sirmordred Do you have to make something else to the sources?
Because i'm gonna leave my computer compiling tomorrow.
Let me know ;)


@nabildanial: Thank you for your tests. So with regard to NPS all versions seem to be competitive.


I haven't known the task "takeOrdered". it seems you did take advantage of it well!


amazing fix!


These look fine, but my review has exposed a new issue concerning redundant elements.


Just ignore the last one (micropilot). The cheapest one they have is $1500.


Since this is getting pulled from an NSDictionary, we can't guarantee that it will be there for nullability annotation purposes.

However, it should be there since all sections should be loaded before querying for cells. Thats why its an asset.


is this pushed to NPM yet?  I've been downloading the latest and it still is saying <=3.5.0 in the package.json


I'm having a problem with no quotation marks around the $base_url variable on build, I think. Will try to fix.


resssoollveed


Personally, I would put a doc comment for this function.


good! I think zipWithIndex and join are the tricks that we need to use for this problem.


Possibly, but older clients rely on that naming for the INFO message from the server, etc.


@nabildanial Thanks for testing. what about time to depth?


move this up one line


Here you can find some basic exercises in Python, hope you'll enjoy!


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


Не совсем понял вторую часть


Should be "Method not supported: " + method.

Is it not possible to write the code so that it doesn't have to check the method? Just let it pass through?


Maybe this should call `exit 1` rather than return?


sentences are really funny! good job!


My understanding was that these links were intended to other working groups—if that isn't the case, then this can certainly be changed back.


I can only imagine your "Go away stupid files i don't need u"


yep, that was the intention :smile:


Jeżeli  coś zostało poprawione w moich materiałach i trafi do mnie jako pull request to tak, są za to punkty z aktywności. Opis commita powinien informować o tym jakie zmiany wprowadzono w ramach commita.


Не суть... )
P.s. писал эти теги в 1 комментарии то их не отображает без спец кавычек


Jesus, this is just like, another plugin


Simple reproducer is to try to execute ppc machine on x86 using TCG.


`// Extended raycast function returns internal pointer of mapobject class instead of modelid`

That's exactly what you wrote in the include. So it wasn't just misinterpreted. Since I thought it was actually returning the index I was sure it needed to be `-1` because `0` is a valid index. I'll undo this commit until changes are made (if I figure out how to return the index, which I'm guessing a loop will have to do, I'll do it).


May want to verify `$tmp` is set here, you don't want to `rm -rf` the wrong thing by accident.


Hi !

In firefox the input file overflows and is arranged with the height: auto

The example is as follows :

![overflow_input_file_firefox](https://cloud.githubusercontent.com/assets/14580377/10672981/51e77fb2-78a8-11e5-8e51-fda4c169d244.PNG)


Понял)


It's copy&paste from "mmio" version. Well it's the same as when you use OVMF on x86 :wink:


@mgracer48 when will toontown world be released i really want to play :(((( :+1:


I would use `-o` option to curl rather than redirecting


1911-8-4-Update (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


This makes the behavior match the documentation. The docs say that the javadoc-opts key can have a :java-source-paths key that defaults to the project's :java-source-paths key. This lets you javadoc some Java code without building it. Which is useful at least for the test files I added to the project, but in general when you might want to javadoc some code that is different from Java code you want to build.


@proxycase Do you really like pizza?


Just removing this commit in my current project's build.sh for now. Error message was:
`Parse error: syntax error, unexpected ':' in /Users/toto/Sites/ri/sites/default/settings.php on line 619`


did you include each root node to compute "touched nodes"?


I tested it. You can also look at leiningen-core.classpath/get-classpath. It doesn't add :java-source-paths to the classpath.


I seen it, he really does


Yeah that comment should be removed it was experimental


Ok I will check it

On 22 October 2015 at 12:54, smillier <notifications@github.com> wrote:

> I've changed it and re-routed everything. Can you make a quick review ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smillier/HydraOBD/commit/3b8d6e3dc79e87b6639be43fa314d9909061f4a3#commitcomment-13923130>
> .
>


ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ왜 여기서 대화하는거얔ㅋㅋㅋㅋㅋ


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 1985](https://teamcity.mesosphere.io/viewLog.html?buildId=62918&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:14


This commit stack overflows in debug mode. Maybe this line ?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 1986](https://teamcity.mesosphere.io/viewLog.html?buildId=62921&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


@mkonicek I meant change that URL back to how it was before but make the protocol be git+https:// instead of git://. But I see that you made it use the shorthand for github repos which is even better (then it's npm's job to figure out the right protocol).


This pull request can be closed as this fix is not needed


1. What is the 1423 - 1427 (lines) block supposed to do?
2. What is setting the placeholder to ```none``` supposed to represent?


Makes sense. Cleaner and more separated than before.


It's not used. I used it while debugging, and it might prove useful later, so we should keep it.


I think that would cause the script to exit the shell (since this script is meant to be sourced).  Because of this, lines before the declaration of errexit (https://github.com/Bioconductor/bioc-cm/blob/fdca6eaf673c4113e7c793e37c7083752cd0988f/setup-environment.sh#L32) need to call `exit 1` or `return 1` explicitly.

Of course, now I'm realizing that sourcing this with Bash 3.x will do just that : https://github.com/Bioconductor/bioc-cm/blob/fdca6eaf673c4113e7c793e37c7083752cd0988f/setup-environment.sh#L6-L20 .  So, this needs to be fixed.


@mauriel133  As @hyi pointed out the checking for 'discoverable' is not necessary since if a resource is 'public' it is also 'discoverable'.


ok, will do


do you mean change the width and height of each item in the listView? That is controlled by the view you inflate in your adapter, the size you set there will be the actual size


Now just looked at the code in release (0.3.0) which is working fine with Safari 5.1.10:
there is a completely different mechanism to determine 'version' via 'storage'.

Maybe just make quick fix "cannot access Google translate server" for release (0.3.0) for legacy Safari?


I think I'll just use setUserPointer for the mapObject index. Then instead of returning the getUserIndex (which you stored the model in) the exended functions would return the getUserPointer (or -1 if it's not added to the manager). Your thoughts?


Ah, I agree.  Thanks!


Not sure the assert is super useful but I can add it. For the nop insertion, good catch, they should both happen *after* the prepare code. I didn't test the deallocate path as lldb doesn't seem to free the memory it allocates :)


Yeah. Find and replace being weird


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1947 of 1956 tests passed)

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


Thank you, as will.


Moved because it looks cleaner (keep everything related to Thaddius grouped).


Can you please change the variable name to be DE_ALGORITHM like -

public static final String DE_ALGORITHM = "DE";

and change its references?


Couchbase Server communicates on many different ports...

Also, it is generally not supported that the cluster nodes be placed behind any NAT.


> Moved because it looks cleaner (keep everything related to Thaddius grouped).


good work with accumulator! i didn't use it like what you did.. but, I come to understand how it works


What a stupid change


You are right...


ok, my mistake


Ah, another good catch.  Thanks Jim! :)


@JeromSar Scrolled straight past that ~_~


Sure, I remember that discussion. Then I guess I'm curious under 10.11 what circumstance the CatalogURL was getting set/changed by Munki if `SoftwareUpdateServerURL` wasn't set in the prefs?


does that work now?


Thanks @adammacias


@andyjansson Promise itself do not give you a async if you didn't use callbacks here


You should link to where you got this, I presume https://gist.github.com/pkuczynski/8665367

I also find it basically unreadable, but I guess it is fine to leave asis.


Two levels is one level too many. Not exaggerating. Refactor into a readable method that explains what's happening.


Looks good to me. @hyi needs to OK these changes.


I would prefer to only put stuff on context that absolutely needs to be there, e.g. `<Link>`s need `history`. If people need `params` further down in their component hierarchy, they can pass them as props from their route component, no?


Into SQLTemplates?


sry, fix in the next version (in booth places)


It's all in the code there for review...


You definitely are. Anyway:

> How would _you_ restructure it?


You should just call it `FEVOR`. :-)


Perhaps localhost.localdomain would have been better?  Does that work?


this is messing up the spoon date validate


Awesome thanks! :)
On 21 Oct 2015 12:31, "Viyat Bhalodia" <notifications@github.com> wrote:

> The final table schema and description:
>
> [image: index_db]
> <https://cloud.githubusercontent.com/assets/4557897/10634075/d5311212-780c-11e5-84be-867ff098673c.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/owtf/owtf/commit/3aa0722a962b74d3165c2e658565145309c74a28#commitcomment-13897717>
> .
>


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


you shouldn't have uploaded this private key..


@nilac8991 the following 5 kernel commits should be merged before compiling new build
1-https://github.com/LegacyXperia/android_kernel_semc_msm7x30/commit/0947f944d00c7cc0bfd3ee938b9e5a6292fa877c
2-https://github.com/LegacyXperia/android_kernel_semc_msm7x30/commit/af8d3fa67d3d52bcb708a63e08a65fdc60977ecd
3-https://github.com/LegacyXperia/android_kernel_semc_msm7x30/commit/164a6e7495c7877398432444db31b2a240f5bd03
4-https://github.com/LegacyXperia/android_kernel_semc_msm7x30/commit/40afc1fcec20ef68d3502ab0e1ac2673769a15a7
5-https://github.com/LegacyXperia/android_kernel_semc_msm7x30/commit/690082a69bbad75b047079e0f5ffcc9818a0fb59

and display HAL should be updated with upstream commits too https://github.com/msm7x30/android_hardware_qcom_display/commits/cm-12.1-caf-7x30

thats all bro ;)


Nice!


We obviously don't want 2D mode on, as this thing could be going outdoors, but we probably want to measure Z velocity, and make sure its covariance is pretty small. We know the Jackal can't have instantaneous Z velocity, so we can be confident that a 0 measurement is valid. Changes in Z will come from pitch angles changing and linear motion. In any case, as it is now, I would think that if you watch the covariances, both Z velocity and Z position take off pretty rapidly.


@andyjansson Async is not multithread


I really should, @ckhroulev . I really should.


ı have some net connection issues ATM thats why ı couldnt cherry-pick them.. :)


rAEdme => rEAdme ;)


I think you gave me a good concept of how to use accumulator for BFS. Thanks!


I don't like the fact that we need to install `gnuplot` into each container. It'd be better to have `deps_to_ninja.py` just write the data file and then have the user run `gnuplot` themselves if they want/need to.


Spring Security secures all HTTP endpoints with basic authentication; these preperties change default values.
By default, username is *user* and password is randomized, printed at INFO level at the start of the application.

Ref: https://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-security.html


What happens if `self.json` doesn't have a key of `rename`? Do we want to provide a default or not even run the async call if the key doesn't exist? If rename is optional or it's always going to be there maybe a comment or toss in a safe default?

I try to strategically use `self.json['rename']` style over `self.json.get('rename', default)` when I want the KeyError to happen if something goes haywire.

Just $0.000002


Awesome


irl-bpt@rangat.org


Thank you for that!


it seems you missed to use accumulator for this problem.


Should be `'revert'`, not `'changes'`.


Yep, I am sure that the wildcard is only allowed at the beginning and end of the dynamicField name. I tried actually running it and solr crashed with that error message. :)


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


both places


Thx, found out myself now https://github.com/andi34/android_device_samsung_golden/commit/76814b1c430e21bacb1499aadeae8b74f8f34ae5


Mmm, I think if the item's not in the store the behavior should be to just exit and return undefined. I believe that's consistent with how other methods to it.


your code looks time-cumsuming. how long did it take for each run?


Dang it, I usually don't forget that.  Yes, based on:
- http://stackoverflow.com/a/21189044/320399
- https://gist.github.com/pkuczynski/8665367

Will add that now.


Use `DSUtils.forOwn(object, cb)` for maximum browser compatibility.


I added a note so that the authors of this part can decide.


Here is one clunky way to let `Day` depend on `Month`: https://github.com/fthomas/refined/blob/master/core/shared/src/test/scala/eu/timepit/refined/examples/MonthDay.scala

Note that this is for demonstration only. I do not recommend that for Monocle. :-)


Addressed review comments.


should be 5.0, right?


good! you did connected problem for Problem 5! Thanks for letting me understand it!


Why are these checks needed?


ack


Hi Hillary, we don't want to actually save the user at this point - we just want to show this page to make sure all is well before we save them. Otherwise this is no different than create...

You can also remove lines 34 - 39


Hi Giovanni!

Giovanni Blu Mitolo wrote on 20.10.2015 2:35:
> Hi there does it work with 8Mhz with this setup???

Yes it does. However I've not tested it thoroughly enough, also I've not tried achieving higher speeds and connecting
more than 2 devices. I've just noticed that after downclocking MCUs at both sides from 16 to 8 MHz the PJON library
stopped working and tried dealing this issue in a silliest way - increase timings :) My project transmits only couple of
bytes once in several seconds so I'm really OK with observed performance now. I've not investigated the cause of the
observed behavior since after downclocking all delay()s are still accurate and PJON code _shoud_have_been_ working fine
with default timings...

> Nice idea to add the setter for the ID, why don't you open a pull request for that on PJON repository?

OK, will do this a bit later. You can also grab my code if you're in hurry :)


I am Virgo so I am great to see this.


it should return 'config' here in case we add stuff to the defaultConfig later


yes, I'm sorry for making this mistake over and over. I'll have to write this word on my monitor to remember it's written BOTH.


Thanks! It remains a lot to do if we want a clean code ...


`'\n'`?


NOPE


this closed issue #3


:eyes:


Maybe your change was only the trigger but not the real cause for the problems on buildbot, looking at https://dev.openwrt.org/changeset/47245


totes intentional


I'm not sure why it took me three passes to notice this, but you can refer to `INSTANCE` in the body of this method (and throughout the class), you don't need to assign it's value to another reference.

![iu](https://cloud.githubusercontent.com/assets/642870/10673842/f67a4330-78c5-11e5-9774-3627a3722c7b.jpeg)


Is "unsigned" really good style? I always use "unsigned int", but that's just personal preference.


Only one change had to occur to satisfy the new requirement :smile:


What is this pull request?


Way past nitpicking since there's only one usage of this class, but I'd be tempted to put a
```python
@property
def auth(self):
    raise NotImplementedError()
```

just in case someone uses this bad boy elsewhere. It'll break either way but at least it'll be obvious as to why.


extra space!


Nice find. Thanks for this.


Can we use the `aes_string` instead please :)

Thanks!


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


This looks alot nicer


Since the set_pub_citation method returns an array, the accumulator array should be replaced with the return value of the function:
```ruby
to_field 'pub_citation_display' do |record, accumulator|
  pub_info = set_pub_citation(record)
  accumulator.replace(pub_info)
end
```


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


comment


I don't know why it took 3 passes for me to get what you were communicating.

![Spidey](http://www.memes.at/faces/spiderpman.jpg)


Couldn't you make this default to an empty string too, which would make it default to the name of the method?


@gloman -- Thank you.


fix indentation - mixing tabs and spaces is a nightmare :)


I don't think we need to visit expressions.


I missed the last one too, but I'll be at the November meetup. Wanna get there an hour or 2 early and hack / hangout for a bit?


Can you move this file back. It's not supposed to be deleted


What is up with this change?  (removal of </td>)


Np i will handle it


ARHG, wait how does my test work?


Hello, there is a ;#3F3F3F pasted in the code. What do you think of the version switch for this plugin : only 2.8 or do you think this should handle more options ?


For both devices right?


not needed - as they should be in credentials


@Pottus Perhaps something like this?

8e903fc71049741a4097c2c876115a0b5d3d6917


likewise - should be in credentials only


This was the file that changed in the merge @ericnewcomer


good job


This is deleted code, sillyhead.


good color choice


Can you remove the hack that is in `testMotif`?
https://github.com/QuackenbushLab/pandaR/issues/6


@peetucket can you re-add this `begin/rescue`?


Fair enough, I try to always use built in methods so I don't have to learn every frameworks way of doing things. Maybe a polyfill?


should be set from the credentials object instead.


oh crap, I need coffee >.>


ditto


Will the call to `_this.previous(_resourceName, _id)` blow up anyway if its not in the store?


lexis - improper indent


Looks good!


Hi

I read documentation and apply setting with your instruction.

but i define in session.py in livestreamer folder.

If i define in session.py its make problem or i should create config file.

Thanks for your help


looks right


is it ok to have user and password passed in as null other servers ? (should be but just checking)


This link is broken.


Bug!
maxIdx can be zero, subtracting 1 is not meaningful, and comparing (in max) as size_t is a bug.
What does Rng::discrete(0, -1) do?
Or actually, maybe subtracting 1 is an algorithmic bug?


> Also, you can just stick this base_url code right into the settings_additions.php file, since that gets copied into settings.php by this script

@wxactly I had this thought initially too, but I'm not sure you can dynamically pull the profile name reliably (i.e. can't assume it will match the repo/drupal root name) from `settings_additions.php`.

You could, of course, count on the user to update the `$base_url` within `settings_addition.php` manually, but that would defeat the purpose of putting it in in the first place.


@jamesob I think PRs without merge commits are preferred. You may try `git rebase bitcoin/master` (or `git rebase origin/master`).


Actually sv_autoaway global variable, not teamplay.


Also, best practice set references to `null` if you don't need them anymore, especially good for custom `dispose` methods ;)


nope, returns `undefined`


Is there a reason why you stick to kW/m**2 instead of W/m**2 (which is a SI Unit) ?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 1990](https://teamcity.mesosphere.io/viewLog.html?buildId=62946&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:13


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 1991](https://teamcity.mesosphere.io/viewLog.html?buildId=62947&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


:coffee:


Oof. Does this work in npm 3?


Filed [bug 1217523](https://bugzilla.mozilla.org/show_bug.cgi?id=1217523).


It looks dangerous to count backwards with an unsigned type. This is correct, but one could argue that it sits there waiting for someone to introduce the overflow bug.


prof non riesco a inserire un immagine su github...come devo fare??

2015-10-09 10:00 GMT+02:00 Antonio Pierro <notifications@github.com>:

> Per vedere il tuo sito online, occorre che il nome del tuo repository sia
> nicola-antonelli.github.io e non nicola-antonelli
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nicola-antonelli/nicola-antonelli/commit/74a5b05811697c671b4888e06a83c33cd0fd941c#commitcomment-13680631>
> .
>


@simon816 Missing license header


@rubenbaetens No reason at all, it just came from the formula's I used.


Ah, thanks for the tip @MarcoFalke! I had only ever done `git rebase -i HEAD~n`. This seems to have worked out well.


It makes sense that numberOfClasses is an unsigned int, since this is the type that represents labels. On some systems (x86 with segmented memory) size_t could be a 16bit integer and unsigned int a 32bit integer. Then there could be a data set with 10^6 classes, then then what? Sure, not really realistic, but still wrong.


color : sure because of the background I just saw the images, ok. Can you fix the #3F3F3F ?


Typo: $grouo_opt -> $group_opt


For me a rule of thumb is to associate the pronunciation to each meaning -> a booth is a stand in a conference, and the double 'o' has that characteristic long sound. 'both' is for a pair of things, the 'o' is short. Always when I see booth I remember of a couple of people in a stand distributing T-shirts. Maybe this helps :)


Are you in some qq group I know?
I encounter the DBIOAction of DBIOAction so .
The format nearly kill me :cry:


This revision is clean.


Wait - does coinToss return unsigned int or bool? Should be the latter, and then the original version is more descriptive.


@saeedalipoor could you deploy this fix to npm?


Can you find a way of fixing it in the fixtures addon without altering unity itself? The majority of our users don't use the fixtures option and wouldn't love to have all their output double-spaced.

Thanks!

Mark


Well done @ribeirojr !


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


This behavior is a little less ideal than it was before because it makes a new ``IN: scratchpad`` and you can't just edit the error like the old way?


Yes of course, but we are speaking to #3F3F3F to comment ?


If you already throw in min/max_element - can't the other loops be replaces with std::copy? :)


- мені теж сподобався фінальний результат)))
- о, це була головна проблема)) спочатку коли створювались 2 компоненти, все управління падало на останній, бо локальні змінні ловили останній рут елемент. Я тоді причепив до хендлера addItem() поточні локальні змінні як аргументи і далі по цепочці передавав їх в інші функції. Хоча так, можна було зробити одну велику функцію в яку передати змінні і звідти тягнути)
- дивився на неї, але ради 100 рядків коду не хотів бавитись з тим.
- дякую :)


Hi,

I have a few questions about the goptions example code, examples_test.go,

- Could you add the demo code showing how to handle the subcommand? E.g., when the program is passed with parameter `execute --command 'test'`, how the program passes the control to the Subcommand handling? Printing out the passed command would be sufficient.

- I was trying to make the `options` variable a global one, not a local variable for the `main` function, but I was unable to. How can I do that?

I know you are drafting goptions v3 right now, but not sure when it will be out, so in the mean time, could you help me with above please?

Thanks


"Imagem legal" boa


Whoops, you're totally right.


can we get a fix for the gradle issue?


same comment as above - should be unsigned int


Hm, we should discuss what we want to use (start_time or start_date) and use that consistently everywhere in the libraries and backend.


Bị lỗi hình như chưa commit maker mới


Stimmt, allerdings ist diese Einstellung für Markdown-Dateien nicht zu empfehlen, da man mit zwei Leerzeichen am Ende einer Zeile einen Zeilenumbruch einfügen kann (siehe: https://daringfireball.net/projects/markdown/syntax#p).


This is super cool :+1:


Hey @suntong :)

Regarding your first question, see [the example for the `Verb` type in the docs](https://godoc.org/github.com/voxelbrain/goptions#Verbs).

Regarding your second question: I am not sure what problem you are running into.

```Go
var (
  options = struct {
    // ...
  }{
    // ...
  }
)

func main() {
  goptions.ParseAndFail()
}
```


why use ```write```? By the way, what is ```w```


looks good for me.


Can you try don't remove it and remove tux


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


i don#t think so. the first loop is a transform operation, the second involves a cast, so also transform. even if it compiled, the implicit cast would yet again transform into a warning


Nice refactoring, thanks for that!


This is ugly... did you maybe do a search&replace "int" -> "std::size_t"?
I'd be surprised if VC was happy with that ;)


Doesn't it return a path?


Oops, auth_login_pgm_domain(init_t)() should not be here. My fault.


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


well. glad to hear from ya, and you never know, maybe one day you will cpme back. in rest long life to you


This is wrong on Pi. We don't have X11 but we do have EGL.
This stops include of "egl/WinSystemEGL.h" in WindowingFactory.h


should this be in ErrorChecker?  seems like that should detect all the errors and avoid calling ES if required params aren't present


Hi! Your modifications are I think a good strategy to solve the problem:

The actual PJON's communication speed bottleneck is function execution time. The master now, is near to the limit where, execution time is more then the available time to read and write bits correctly. For this reason reducing also slighly BIT_WIDTH will produce a steep decrease of communication accuracy.

What you did is to give to the MCU twice as time to compute bits and bitbang, I think it is correct and there are not more elegant ways to obtain the same result.

Can you please run the SpeedTest and NetworkAnalysis and let me know results?

I am thinking to set conditionally (depending on cpu frequency) timing values. This contribution you did enables compatibility to tons of devices : ) I think it is right for you to do the pull request! : )

But I should before understand and test how to correctly switch timing definition depending on cpu frequency constant.

Thank you again for your support.


II think that here we correct for xmin, xmax and ymin, ymax and not the values that are sent to this function.


It also does not allow us to make a build because of duplicate rules.


search&replace again


search&replace again, you need a better tool :)


You accidentally deleted your second migration!


again...


Perfekt, danke!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 1998](https://teamcity.mesosphere.io/viewLog.html?buildId=62966&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:20


one more


see above


create_table :gametracks


search&replace


commit


HBIRE_vector_take2 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1958 of 1960 tests passed)

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


Ксения, у Дмитрия @dmitryvodop скопировали?


Check above: "var w=window.open();", w is a separate new window. What else other than write would you use?


Did you mean Jeff W. Kester?


Help text?  Maybe too obvious.


maxIdx can not be 0 unless m_pop.size() is 0, which is wrong anyways. As the population is set in init(), there is always at least one pareto front. if the pareto front has only one element, discrete(0,0) will return 0 which is correct behavior.


Good progress


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (882 of 882 tests passed)

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


Awesome. This fixed my deleted pics to reappear upon reboot issue.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2000](https://teamcity.mesosphere.io/viewLog.html?buildId=62968&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:16


Hi Lee, version update looks nice. I'd take this to upstream as well.


On second thought, maybe the separation is correct. The cube is so close to the viewer, I sometimes have trouble converging it at first.


Agree


Obrigado Roberto,
tens algum sistema já rodando para eu dar uma olhada, o api pelo que li tbm pode ser utilizado para outras operações fiscais, isso confere?


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


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


Debateable - why should the range be couples to the size of objects in the language? I see no reason here to prefer size_t over unsigned int.


quando ele disse isso??


autogenerated


Oh, yes. Thanks again. Fixed in new commit - [2aa745](https://github.com/dhananjay92/arewefastyet/commit/2aa745b45dc89eb1f4f34b97a6e01961869bac99)


also autogenerated


The question is: why??? Who have I pushed this stuff? ... It's me!? Really? OK, I'm going to fix it now...


é isso aí mano =D


Oh, good point. Yep, this should be included with the other error checker routines and messages.


Why isn't this using the group_segment_size in the dispatch packet? Should this also use an undef initializer


No problem, sorry for the spam today!


I do prefer _initialBufferArray. The same for methods. What do you think?


I meant Options bit 1 (XOPT is used in ACPIBacklight.kext -- this feature not implemented there).


This path flip flops in-between yum and apt based operating systems, need to handle this better.


@alexsandro-xpt it's fixed in PR #555


Not only does `require(mosaic)` takes care of attaching `dplyr`, it also also does the gymnastics needed to have both flavors of `do()` available without needing to specify one of them manually.  So `do()` will be `mosaic::do()`, which will call `dplyr::do()` when appropriate.  Best of both worlds for everyone -- except the package maintainer ;-)


i have thought about this as well. My argument is consistency with everything that represents a size. It does not harm that numberOfClasses is larger as you are right that the number if restricted by unsigned int. Please note that "number of classes" is not the same as "maximum label" even though you can derive it from that. It also prevents errors like putting accidentally classes as a class label as this will (At least on VC++) create a warning of non-matching types.


@Daniil-Osokin, у нас был курс Кирилла по инструментам разработки на 3-ем курсе. Никто ни у кого не копировал.


Most of the commit looks good, but I left a few comments. At least the search&replace bugs must be fixes.


I lost my code reading ability in an unfortunate accident. :(


#1 and #2 were shown to Anton and approved


The return type can't be null, right? Perhaps better:

```
SynthesizedInteractiveInitializerMethod submissionInitializer;
if (!hasTrailingExpression && ((submissionInitializer = method as SynthesizedInteractiveInitializerMethod)) != null)
```


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


HUEHUE, no Shopping.


the original version would transform into

    label = (unsigned int)Rng::discrete( 0, 1 );

is that still better?


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


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


Nit: parens not needed.


This will cause a runtime error if callback isn't defined, e.g.

```
> if (!callback) {console.log('a');}
ReferenceError: callback is not defined
    at repl:1:6
    at REPLServer.self.eval (repl.js:110:21)
    at repl.js:249:20
    at REPLServer.self.eval (repl.js:122:7)
    at Interface.<anonymous> (repl.js:239:12)
    at Interface.EventEmitter.emit (events.js:95:17)
    at Interface._onLine (readline.js:202:10)
    at Interface._line (readline.js:531:8)
    at Interface._ttyWrite (readline.js:760:14)
    at ReadStream.onkeypress (readline.js:99:10)
```

Needs to be
` if (typeof callback == 'undefined') {`


@gitam-kavya
I don't think we will ever have a json for images stored in our DB. The parameter `json` and this condition may not be required.


this is atuomatically resolved into a float, unlike i/2


just a thought... but shouldn't the secure tokens be stored on you local machine only and not in the git repo?


Any reason these filtered terms need to be in the gitignored settings file? Might be better to include them in the repo in some fashion.


i will fix that


We needed this section for UWP to work.


Probably something like ${libdir} or ${base_libdir}. But testing or review of the glibc build/configure system is needed to determine which of them is the right one.


Should remove this comment


Sorry I mistakenly commit the code. Please revert. Thanks


It is better to split each class up and actually implement the functionality, than it is to commit empty classes with stub TODO messages.

<review


Should remove this console log


definitely a fan of only passing down the specific props needed :+1:


> Error message was: `Parse error: syntax error, unexpected ':' in /Users/toto/Sites/ri/sites/default/settings.php on line 619`

@jamidwyer I wasn't able to replicate this. I use BASH, but tested with ZSH and did not see this issue with either.
I tested a build without an install and another with and install; neither gave me an issue.


Sorry about the whitespace problems. My editor kept freaking out.


This is not needed. Result property waits.


@Paridhi25
take care of the length of line here. Might cause PEP-8 failure


body {
background-color:
#b0c4de;
}


Show value in the lable


Sorry, compulsive urge for consistent punctuation got the better of me (standard ellipsis is 3 periods).


Delete


There are two issues with this. First, it doesn't recognize c1 files. Second, the regex is a bit broken: we need \\. to get a backslash and period in the actual regex, and $ should be replaced by \\' (end of file name).


Also, edited small parts of the storyline, I guess it's a bit bland.


yes man ;)


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


@Paridhi25
This might cause PEP-8 issues. Verify and ignore if it doesn't


lol


ahh, não sei então :/


Почему выполнил-то?


LOL.

By the way, why did you call it `checkers`. There are Python packages named or with `checkers` in their name. That will lead to a lot of confusion.

Really nice project.


assuming that the first field of 'in' will always be the family, i think something like this here would be more 'readable':
af = ((struct packed_inet *)in)->af;


Testing Comments :-)


Please consider splitting up each major change into one commit.  It's difficult reviewing your work when you bundle so many features in one commit.  Your work is good, so present it more clearly!

<review


@lsegal Could you comment on the failing test in `yardoc_task_spec.rb`? It's this one:

```ruby
    it "does not invoke stats" do
      expect(@yard_stats).not_to receive(:run)
      @yardoc.statistics = true
      YARD::Rake::YardocTask.new do |t|
      end
      run
      expect(@yardoc.statistics).to be true
    end
```

The test seems to imply that if you don't set `stats_options` in your Rake task, it shouldn't display stats. But that is completely false. `YardocTask#define` calls into `YARD::CLI::Yardoc#run`, which shows stats here:

```run
        if !list && statistics && log.level < Logger::ERROR
          Registry.load_all
          log.enter_level(Logger::ERROR) do
            Stats.new(false).run(*args)
          end
        end
```

The spec didn't hit that before, simply because a different spec had set `log.level` to `Logger::ERROR` ("quiet mode") and left it there.

Perhaps that spec is just based on a false premise and should be deleted?


I'll add a verify here.


@jjallaire you may also want to bump required version here:

https://github.com/rstudio/rsconnect/blob/master/R/bundle.R#L388:L395


I don't actually know if this can be null, but the old code (in ```RewriteScriptInitializer```) did a null check, so I kept it.


This is to make something similar to:
FAST_XXms{
     SHIFT_XXms{}
}
Right?

Can be usefull to add 2 DHT on the same phase but call at different times to avoid problems with the library.

Thanks


Potentially, your n_len could be the same as an error result from read() when implicitly casting from signed to unsigned int


Perhaps one . would suffice :)


@cattata1056 my name is iplayksp


The tcp functionality of this commit is already in my master branch, the udp bit needs some consideration, as I don't like the double read.


I've tended to do `self.class.` in places like this, just so there's one less thing to be out of sync, but there are probably arguments the other way? Doesn't matter. Merging.


LEL


free pictures!


@stramel could you explain why you are using `prefixSelector` method with a regex here?
Sorry, I didn't get it.


I am this father who suggested to add please at the front.


Why did checkstyle trigger this as an error?  If it required removal, it might have been mislabeled.

<review, organization


free pictures!
![15038229](https://cloud.githubusercontent.com/assets/15038229/10675206/1e6b25b2-78bc-11e5-8ea0-00d747432ab1.jpg)


Yeah, I'm talking about the string. "Not Following" is for me unclear and can be easily confused with "Ignoring" (I understand that ignore is an action performed by the user, when "Not following" is just the default mode, when the discussion is not followed or ignored, but heh, I'm a experienced user, I don't think everyone will get it).

Concerning "Mentioning", you're right, it refers to an expression, my bad. Maybe "Default" can be a solution? It indicates that's the default state of a discussion (when not followed/ignored), and the (`not_following_text`) describes what's the meaning of that behavior.

For information, I already added the "only" sense on the French Language Pack.


This boolean does not say whether the music is muted, but whether the key is pressed down. Name it accordingly.


So just to be clear, there's still no way to cause a "focused" cell (not in edit mode, as in it's been canceled) to enter edit mode with a single click, right? I can't for the life of me find a way out of the box.


I am not certain whether common is the right folder name. [May use "build"?](https://gist.github.com/davidfowl/ed7564297c61fe9ab814)


how about 'just do yourself' aka globie's slogan?


Unfortunately, it also bugs me when things that are not a complete sentence end in a single period.


ok @iplayksp


@Paridhi25 @akawtia1
Take care of `business_image` field in models.py. Use `models.ImageField()`. Refer the following 2 commits made over product_images section and change accordingly.

https://github.com/Vishal0203/UrbanUrn/commit/24cfeb13c017c299f08957e0bf88d0447ca4da68

https://github.com/Vishal0203/UrbanUrn/commit/e5b2bb3423aa8477f9603e2357803a3b95ae7a82


What if this key is not present in the properties? No default value is passed now. The getBoolean method will try ```Boolean.parseBoolean(null)``` which will yield unexpected results.


This might still be leaking an allocation for the returned C string but it's not really clear from the v8 docs.  Would have to check the impl and see where the memory came from.


the conversion from analog reading to distance is not correct


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1010.6ab71a2479374f5f930aa76dcd85eb9aaa6fd242](http://build.palaso.org/viewLog.html?buildId=64741&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 679, ignored: 6 Build time: 00:05:39


1911-8-4-Update (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


That's accidental. Restoring...


Image Updater might be a better name unless you see this script being used to get images the first time.


Do not forget to tag commits!

<review


also it needs to check for NULL in case the conversion failed.


ouch, zaraz zerknę


now I am happy :)


What is this wait for?


0 additions and 0 deletions.


Poisé, foi naquele dia que fomos no BK, ele tomou um .gitignore daquela menina


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


Since there is fileip cluster now, I'm playing with the thought to have a `validate` match type that will route the metrics to any cluster when validation fails.  This will also allow me to define (by regex?) how the metric should be validated (I'm doing so looking at influxdb's annotated metric format which I'd like to support).


@ChuckPierce If you look at the commit bar above, you can see it's only in `master` at the moment. It'll be in either `0.4.11-RC5` or `0.4.11`, whichever comes out first. For now, just do `npm install c3#master`.


I have implemented Pawel's wait. But I think I have found a bug at that occasion. and this time, in wicked, apparently :-( .


i think you only changed the interfaces here


Haha, well, because I originally developed this for Google internally and there wasn't a project named Checkers internally. And honestly, there are some packages with checkers in their name, but not many that seem widely used, so (hopefully) there won't be too much conflict.

The Checkers name was because in the original incarnation of this project (which has seen *many* iterations), each function was what I called a checker...because they were "checking" something. I know. Super clever. It is also a short, easy-to-type, English package name, and that goes a long way.

Lastly, THANKS!! Let me know if you start using it for anything. I'm happy to support it, add [reasonable] features to it, and answer any questions you may have trying to use it. I just put it out into the world because there was no reason not to and with the hopes that someone like you may find it useful. I personally like writing Checkers tests (of course I'm biased) because I like the control over what I want to do with my tests in terms of organizing them, running them, etc.


Wow, super fast response, THANKS.

Just FTA, I've put my code and its running result at
https://github.com/suntong/lang/blob/master/lang/Go/src/sys/CommandLineGoptions.go


:balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon: :balloon:


Updated the source code:
http://aldostools.org/temp/test/webMAN_MOD_1.43.12.rar

- Added proper support for auto-play /BDISO and /DVDISO
- Auto-play option can be disabled temporarily holding L2 while mounting (formerly was L2 or R2)
- Added option to disable temporarily [online] tag holding R2 while mounting.
 * If the [auto] tag is not used, holding L2 while mounting will enable temporarily the auto-play option.
 * If the [online] tags is not used, holding R2 while mounting will the remove syscalls.
- Added option to pause/continue/toggle rsx. e.g. /browser.ps3$rsx_toggle
- Screenshot of XMB now supports optional path. e.g. /browser.ps3$screenshot_xmb/dev_hdd0/xmb.bmp
- File Manager now will show <Play>, <BDV> or <DVD> on *dev_bdvd* with a link to play the disc
- /delete.ps3?uninstall now also removes the files of VSH Menu for webMAN and custom combos
- Reduced file size of some VSH Menu backgrounds


I'm not sure about building critique in this way into the adventure. It then becomes more like the player is playing at being mikey going around the gallery. Maybe we want that but it seems a bit didactic for a game. Perhaps after `you realize` it should then be a text box that the user has to fill to continue. And that stuff doesnt save or anything. ?


my_capture might not be the best name for this method. It doesn't communicate clearly the purpose (to me)


Narrowing to requests with messages included definitely makes sense. Thanks for the fix !


My virgin seal was broken last night...I wish I could say how much enjoyment it feels but I think its still a bit of a painful process. I bought Bitcoins and now own a wallet, I transferred some of these funds (0.28530 btc) to another wallet. I kept the default transaction fee of 0.08 btc which still seems pretty expensive in retrospective none the less...I chose this because I thought it would allow a fast transaction within the blockchain and worth someones cpu's processing power. But no funds and no joy. HELP me out please?


Shouldn't we perhaps just remove it then?


Thanks, I'll try. The problem is I learned it incorrectly long time ago and every since then I repeat the same mistake. I'll do my best...


a=capsules              a1,b1,c1,c1,d1
b=structural
c=fual
d=engines


The list will not contain both integers and strings.

the list will look like this
("T","h","i","s","i","s","a","n","e","x","a","m","p","l","e")

The Nodes of that list will be defined something like this:
class ListNode:
    def __init__(self, prev, next):
        self.prev = prev
        self.next = next

and that means, that any letter in the list is connected to its previous and following item, "h" is connected to the left to "T" and to the right to "i". There is no number anywhere. The number of occurences will not be considered in this linked list, because it doesnt matter. this is a separate task of the program.

The class will look for a Node that looks like [aeiouAEIOU] and check if the left and right item of this node are not a vowel = [^aeiouAEIOU]. We could also add something like that: If a node is a vowel and if the item to the right is another vowel, the next item will be checked and if that is a consonant, the core of that syllable consists of two vowels. So the word "could" would be counted as one syllable.
so far i dont know how to ensure that also consonant clusters like "ld" will be found and counted as the end of a syllable. we must remember to assure that word borders are not ignored, so maybe a whitespace character has to be its own node in the list and we write that whenever the next item is a whitespace character, the stop whenever the word ends.
The list should probably look more like this:

("\b","T","h","i","s","\b","\b","i","s","\b","\b","a","n","\b","\b","e","x","a","m","p","l","e","\b")

Start: "\b","T","h","i","s","\b", Stop
found "i" and before that "T","h" and after that "s" between word borders
= one syllable (cvc)


Instead of this if:
```
Settings.setBoolean("PLAY_MUSIC", !Settings.getBoolean("PLAY_MUSIC", false));
Launcher.playMusic(Settings.getBoolean("PLAY_MUSIC"));
```

This fixes the following problems:
- behaviour is undefined if the key "PLAY_MUSIC" does not exist in the properties, because no default is passed
- it is easier to understand


@Paridhi25 @akawtia1
I wrote a decorator for this check. You might want to consider using that. The name of the decorator in `validators.py` is `check_business_or_super`. Checks if the authenticated user is a superuser, staff or business user


Not a huge fan of this.  We do need to decided on a style guide. If this is merged as is we will have propTypes defined in 3 different ways in the project!

I am a fan of using what Is in the propTypes.es6.js file because it gives maximum readability and is easier to type. The component files should only contain a component definition, really any methods/variables broken out should probably be moved elsewhere.  propTypes are more like meta data about a component so its nice to define them apart from all the behavior.


This is not necessary and makes diffs unpleasant to understand in the long term.


Now you can get mad and do pew-pew

```java
        new MetricsStreamService(
            new CompositeStream(
                new AggregatorStream(new JmxMetricAggregator("org.ametiste.redgreen")),
                new ThreadPoolStream(
                    new AggregatorStream(new StatsDMetricAggregator(statsDClient))
                ),
                new MetaMetricsStream(
                    new AggregatorStream(new NullMetricsAggregator()),
                    new AggregatorStream(new JmxMetricAggregator("META.NULL"))
                )
            )
        );
```


Test comment


esimene


@sferik The redefinition of callback_url was deleted from OmniAuth::Strategies::OAuth2 while it was used in the strategy to get redirect_uri, which should be the same redirect_uri which we sent to facebook without params (code, etc). That's why the callback_url was redefined. Right now omniauth-facebook login, and omniauth-google login don't work.

The fastest fix would be to create a new function in the strategy: redirect_uri, and define it as a "full_host + script_name + callback_path", and then use it in build_access_token and request_phase methods (instead of callback_url).


If you use the suggestion above, this method is not needed.


( ͡° ͜ʖ ͡°)


Yes it is, have a try and let me know.

From Mobile.


@Daniil-Osokin, во-первых,  pull-request Дмитрия был сделан позже чем мой.
Во-вторых, потому что на титульных листах пишут "Выполнил: студент ..." вне зависимости от пола. По крайней мере нас так учили.
Скопировала у себя из README по devtools-course1.
https://github.com/KsenyaKochanova/devtools-course1/blob/master/code/ksenya-kochanova/README.md


@clebergnu what do you think about `shell=True` here? I might even have a usecase for it:

`avocado run "/bin/echo Compiling qemu; cd \"$QEMU_GIT\" && make"`

On the other hand it's a bit dangerous to allow people to execute custom scripts without writing them... (on the other hand they can write the script and execute it, so it's not that different)


fork pull


what's the binary flag for?


Ти пиши або так <meta content="text/html" charset="utf-8"> або так <meta content="text/html; charset=utf-8">
але ж не так <meta content="text/html; charset="utf-8">


Massive reformatting for no special reason, in most cases lines are broken and readability is worse.


Mobile


Is it possible to use Jquery or prototype?


Aha, I didn't think about that. nice catch.
I was referring to https://github.com/TrinityCore/TrinityCore/blob/6.x/src/server/game/Entities/Transport/Transport.cpp#L629 blizz doesn't do this^ anymore


Checked this solution with omniauth-facebook. It works. Although we should check how callback_url was used by all the gems...


I can think of three possible methods to do that:
1. to always insert a newline after a `.` or `TEST(...,...)` call. This is simply adding `UNITY_OUTPUT_CHAR('\n');` at [line 93 of unity_fixture.c](https://github.com/ThrowTheSwitch/Unity/blob/master/extras/fixture/src/unity_fixture.c#L93). However, this makes non-verbose tests take up multiple lines, which somewhat defeats the purpose of the `.Verbose` option.
2. Have some global state attached to `UNITY_OUTPUT_CHAR()` which records the previous character inserted. If the previous character wasn't a newline, `putchar` (or whatever the user overrides `putchar` to) a newline as well. This still affects `unity.c`, but doesn't unnecessarily double-space output.
3. Only output a `.` in `UnityTestRunner` on completion of the test, if there were no errors, instead of the beginning. I like this one the best since it's extremely easy and is localized to `fixtures`.

Which would be appropriate? I've only started using Unity recently (as you can see, I wasn't even sure what was part of fixtures and what was mainline), so I apologize if my first attempt was a bit naive.


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1941 of 1950 tests passed)

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


ну а чого воно тоді проходить валідацію :)


probably `return` here, if we know that the succeeding code will inevitably fall over.


Cool ! worked for me upgrading 2.0.0->2.0.1 and just starting fresh


Is this for *planned* future use!? Terrifying, if so any way to get some info on this?


@kippandrew By running R CMD check I just noticed that we have an undeclared plyr dependency. If it's straightforward it would be great to not drag in plyr for this one function. Is there an easy way around this?


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 21](https://teamcity.labkey.org/viewLog.html?buildId=279649&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:29


Welcome Marvin!


Awesome . . absolutely silly that what you wrote didn't work, though, but maybe they have their reasons.


```happy to be here```


if you don't have X11 this block is not relevant. what is the issue?


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


I think `product.url` and `product.url.url`, both are holding same values and `product.url` is pointing to exactly what we have in our DB. So we should consider using that.


@benmccann just published as 1.0.3


@nrel-bot-3 You are clearly not Marvin, since you are way too happy...


Зачем тогда исправила? Ребята, не будете копировать, не будет вопросов. Надеюсь, Дмитрий исправит название у себя. А "МО ЭВМ" ещё актуальное название?


Ha. No, but you can do quantum gremlin in master/ now. I added this logo cause its cool and I like collecting logos on Gremlin. By the days end I my talk abstract at GraphDay will be online which discuss Quantum Gremlin. Finally, in about a month or so, I will have an academic article on it out.


Even robots are enthusiastic on their first day.


thanks!


Burnnnnn...


Ok, TY.

2015-10-22 21:06 GMT+02:00 Dario Di Maio <notifications@github.com>:

> Yes it is, have a try and let me know.
>
> From Mobile.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/souliss/souliss/commit/f255edba5716aff34e9bb644ad59397f94f8d6ef#commitcomment-13935648>
> .
>


Could this check be moved out of both branches to above the 'if' ?


Understandable - not programming extra complexity that you don't need. Do you create the PF arrays manually to pass it through now?  I like the idea behind this and am trying to think of a way to include this functionality with full support for advanced functions as well... But I would also need to change the infix to postfix method so that it doesn't touch functions I would think.  Basically, create an "unaltered" PF array based on the infix.

All that being said - if you have thoughts on how to do this or an interest in working on making it and keeping it efficient, they would always be well-received. ^^


hum this is already in master right?


I think running EnergyPlus CI tests all night and day would be FAR beneath Marvin's superior intellect and thus make him even more depressed and bored than usual...


@Kagamine
You have renamed the wrong filename in templates. I assume it should be `templates/class.cs`


This comment seems inconsistent with the implementation.
with the implementation I would think you would write
foo(a, b, spread(others))


This is all abandoned now anyway :)


Note that it isn't supported in dartium either


perhaps keep these in their own library rather than making this a part.
when ready for dart:js users, we can re-export form js.dart


Be consistent with whitespace. Don't write `) {` and later `){`.


This seems to break the app.


Oh good.


please remove the additional new line


the green LED is already on that pin


@sirmordred

All done man :+1:

I included latest changes also from audio and media caf... If it's the last time let's do it with style :D


@coke: I don’t think that Perl should take the stance of agreeing with a lone [blog post](https://tedclancy.wordpress.com/2015/06/03/which-unicode-character-should-represent-the-english-apostrophe-and-why-the-unicode-committee-is-very-wrong/) over the Unicode Consortium and most writing published on the web and in print (well, excluding the even more predominant U+0027!). The main argument of the blog post is that everyone is doing it wrong because it’s hard for programmers to parse, but the fact is that natural languages are ambiguous and natural language processing is hard. Fortunately for us, both Perl 5 and 6 provide the best built-in Unicode support among programming languages, and provide the ability to handle Unicode text segmentation including inter-word punctuation (and many languages do allow various punctuation marks within words, which are not considered alphabetic letters). The solution is to improve the tools to parse language, not how people use and write language.

**The Unicode Standard 8.0, §6.2 General Punctuation: Apostrophes**
> ***Letter Apostrophe.*** U+02BC MODIFIER LETTER APOSTROPHE is preferred where the apostrophe
is to represent a modifier letter (for example, in transliterations to indicate a glottal
stop). In the latter case, it is also referred to as a *letter apostrophe*.
>
> ***Punctuation Apostrophe.*** U+2019 RIGHT SINGLE QUOTATION MARK is preferred where the
character is to represent a punctuation mark, as for contractions: “We’ve been here before.”
In this latter case, U+2019 is also referred to as a *punctuation apostrophe*.

**The Unicode Standard 8.0, Code Charts: General Punctuation**
> `2019` ’ RIGHT SINGLE QUOTATION MARK
> • this is the preferred character to use for apostrophe

**The Unicode Standard 8.0, Code Charts: Spacing Modifier Letters**
> `02BC` ʼ MODIFIER LETTER APOSTROPHE
> • `2019` ’ is the preferred character for a punctuation apostrophe


Regarding line 150 of install.md. User passwords are notoriously insecure, even when they're told to "feel free to make a good long one." They should be generating one with KeePassX or Diceware.


Okay, done!

On Thu, Oct 22, 2015 at 2:52 PM, Andy Kipp <notifications@github.com> wrote:

> @jjallaire <https://github.com/jjallaire> you may also want to bump
> required version here:
>
> https://github.com/rstudio/rsconnect/blob/master/R/bundle.R#L388:L395
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rsconnect/commit/96f5ef70161cc739e4af46d21a82008e07b46606#commitcomment-13935304>
> .
>


Looks like a test UUID was accidentally committed.


Turns out it's a custom toolchain issue. Everything was setup properly, but the UBER linker doesn't like the SGX binaries in MM anymore… Anyway, all is well after reverting to the official Android EABI prebuilts.


Shouldn't you check if ```typeof data[key] === 'object'```?


После того как закомитила первую версию посмотрела как сделано у остальных ребят, после этого поправила кафедру и выполнил->выполнила, закомитила отредактированный вариант.
Что касается МО ЭВМ - нет, не актуальное.


Yay! Recursion.


Wuhu 0.9 is comming. Nice! Device adding per UI?


Typo.


@Zedron once again, NearTeleport only handles teleports on the same map (in context of transports this would be from grom'gol to undercity)


Minor, of course, but bad format. Need whitespaces around `=`


sdfksdnfk


Another thing to decide for the style guide.

I was in favor of this style until I realized in other places the variable is only being instantiated/assigned in the if statement, it looks clearer and is effectively the same thing as having a bunch of undefined variable intitializations. either way when the final jsx is rendered its going to be asking for something that is `undefined`

what I'm referring to is:
```javascript
if (prop) {  // prop is falsy
 let x = (
    <div></div>
  );
}

return (
  <div>
    {x}   // x is undefined
  </div>
)
```
vs

```javascript
let x;
if (prop) {
  x = (<div></div>);
}

return (
  <div>
    {x}  // x is still undefined
  </div>
)
```
vs

```javascript
let x = prop ? <div></div> : null;

return (
  <div>
    {x}  // x is `null`
  </div>
)
```

what does everyone think?

I think putting anything that is conditionally included in the return from render  on the lines above the return is clearer.

Also prefer the if statement but without instantiating an undefined var


That ```#define HAS_EGL``` is only in affect when X11 **is available**. But on Pi this is not the case. EGL is there but without X11 and EGL required without having X11. Same for other platform with EGL and without X11 (such as iMX6). That is the issue as I understand it: having (and requiring) EGL without X11


I really hate this spacing standard that jams together all the closing parens so you can't communicate subtle nuance by varying the spacing to show groupings better.

It irks me that it has become such a commonplace thing that now automated tools enforce it and people have been conditioned to think of it as "correct".


Thank you so much man.. ;)
Yeah all repos are ready now...
And last time yeah... :) goodbye samsung msm7x30 family :(


new files


Ja by bym za tym, by nie robic zrzutow narazie a narysowac schemat blokowy, (nawet w paincie) jak poszczegolne rzeczy ze soba dzialajo. Tak pogladowo,a w oprciu o te wersje alpha zrobimy beta uml itd


I find this test as an overhead, but it is a good practice for mind :smile:
Thank you for polishing!

You can merge it with fix for that my concerns.
Meanwhile I'm applying your change to another PR on the matter.


So why was this off by one month?


remove the additional new lines in this file as well


Maybe you missed the [Temporary Commit] message. This is still a WIP


new commit


Which do you use? Eclipse CDT is a living hell, but teaching these whitespace rules was refreshingly easy.


а можно не менять имена переменных


It that aligned with Production/Staging CND links?
https://github.com/aspnet/Templates/blob/203329d66cf0c732f2baefcba165122d1c8b751e/src/Rules/StarterWeb/IndividualAuth/Views/Shared/_ValidationScriptsPartial.cshtml#L10
That is: will be shipping the same version?


Não tenho nada que possa ser disponibilizado são sistemas interno e não web
(são intranets)
Mas existem muitos usuários usando a API, com ou sem adaptações para seu
uso particular. Inclusive vários ERPs WEB.

Mas isso é melhor perguntar para o grupo

Não sei bem ao que você se refere quando diz outras operações fiscais, mas
o sistema em suas versões tem NFe, NFCe, CTe, e MDFe

Roberto


It is a mystery to me why it would work the way you describe. I don't see where in that function the starting layer should have any effect on what areas are combined.

Can you explain why the starting index should be a multiple of infill_sparse_combine?


This sounds like we are recording the shard, rather than operations on the shard. Call this `ShardOperationRecorder` maybe?


not testing `load_tagged_results`


>I find this test as an overhead, but it is a good practice for mind

I agree it was unnecessary with your change, but now there is conditional logic I wanted to check coverage.


You changed prog_char_t to char on AP_HAL/Scheduler.h, why not already change in here too?
And get rid of typedef prog_char_t


This commit now makes sense since we've committed (heh) to changing the meaning of the DownloadFile permission as of 4.2.1. Also, testing is taking place of the bug I reported above at #2648.


I agree the pull request is large, besides the changes were made over 2 weeks ago so merging the pull request maybe a problem

However, the change required the module to support a minimum of OpenMRS 1.9.9.
On talk, it was decided to make the module so. After this the resources in api-1.8 and api-1.9 clashed because they both built to OpenMRS 1.9.9.
This meant all the resources in api-1.8, api-1.9 had to be changed to resolve this conflict by giving the correct 'order's.
ie. resources in api-1.9 had to have precendence over resources in api-1.8, so resources in api-1.9 had to have a lower order than resources in api-1.8.

There were a few tweaks to the tests as well to get them to run.
(Incidentally the tests only run if a version of OpenMRS 2.0.0 has been built locally, currently on my machine building OpenMRS 2.0.0-SNAPSHOT wouldn't work.)

Finally, the actually change could be made which was to update all 'DAO's to use DbSessionFactoy instead of SessionFactory.


Just figured out a way: https://github.com/rstudio/rsconnect/commit/c3a92264edcc475361884356d82c8ee08b5462a6


HBIRE_vector_take2 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1952 of 1954 tests passed)

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


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


Why call it ```fix...``` rather than for examlpe ```change...```?


No need to spend O(n m) time on this -- O(n + m) time should be sufficient.  Make a dictionary mapping the constrained row/column keys to their constrained values (O(n)) -- at the same time, check for duplicate or nonsensical constraints (O(n)).  Then go through the query to find which ones are in the constraints (O(m)), and ignore those.


    Y_prime = list(Y)

No need for a deep copy.


Should be "Active Support", see http://guides.rubyonrails.org/api_documentation_guidelines.html#wording :heart:.


Что будет с точки зрения REST, если вы будете игнорировать все поля, кроме rating?


We ought to name `logmeanexp` and use it in both cases here.


That rule is weird. Both expressions (i / 2 and 5) are clearly integer valued (in contrast to float), one of them is happily converted, the other not. Just because 5 is a constant? How about 5 / 2? How about i? The only sensible behaviors seem to accept all or none. Anyway...


Logování by mělo být společný. Jen bych ho trochu upravil. (Jméno modulu do řádky s logem, název geoMop.log ... ) To že se potkávají dvě logování je nepříjemné. V communicatorech (JobScheduleru) má logování trochu jiný smysl, než ve zbytku applikace (sebrat informace o běhu na vzdálených PC a umožnit tak doladit program pro kterékoliv prostředí a to i v případě omezeného přístupu) To logování funguje jen tam kde je importnutý logging?  Ty jsi ho dával do všech souborů?

V applikaci nemá logování tak jak je udělaný v communicatorech význam. To co je však nepříjemný je, že applikace komunikuje přes communicator a tudíž má logování taky. Ǐ když na druhou stranu logger sebere i chyby, které doputovali přes ssh spojení a to je dobré.

Takže z toho vyplývá:
- mimo communicatory budeme používat Tomášovo řešení, ale musíme ho nějak upravit a oddělit od communicatoru. Kdyby si přišel na to jak používat dva loggery, bylo by to dobré.
- Honzo, mimo communicatory bych logování omezil na logování chyb, jak to udělal tomáš. Koukal jsem že loguješ i události, které můžou vzniknout jen v applikaci, navíc s vyšším levelem než je potřeba. To bych omezil. Potom se na to kouknem.


У нас институт :)


Hi -- I had a quick question ( I was trying to find the proper place to contact you -- sorry ) -- I noticed that Chrome no longer supports NPAPI -- I'm relatively new to java programming -- not sure if your plugin makes use of this or not? The reason I ask is that I can successfully authenticate to my test crowd instance  from firefox, but not from chrome. :(

Any tips appreciated -- understand that you are busy, and you are under no obligation to respond. Wanted to thank you for the awesome plugin all the same!!


No problem, Stackoverflow just convinced me that they are the same type. So it is just for the style guide.


Uh, well, All those `#x` got recognized as issue references.
By the way it appears that you are using different indentions.


oh, I only looked at the comment below "egl detected, don't use glx'. I overlooked that this block defines HAS_EGL


Ohh... that went in by mistake. I had the intention of removing prog_char_t, but realized it would be too much trouble for this PR.


Nice!

On Thu, Oct 22, 2015 at 12:35 PM, Daniel McDonald <notifications@github.com>
wrote:

> ability to restrict cpu count, thanks @mortonjt
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/wasade/American-Gut/commit/6bcd6ed74fb59fc9e879e5d0df96081d1902189a
>


Interesting. Was this not needed in java/pom.xml at all?


this is in our style guide? :o


Change them back if you would prefer.  It's been "correct" ever since I learned to code in C#.  I try not to do the same thing when dealing with strings, but leading and trailing spaces like that don't look right to me at all.

For me, the automated tool implements what I already do.  You can call it "conditioned" if you like, but really any standard that isn't directly enforced by the language is on equal footing.  I review every change that code-maid makes, and I don't commit those changes that take away useful formatting.


Sure, there is always one front. If it consists of a single element (dominating all other, depending on the problem that's not so improbable after random initialization) then the index of that guy is zero. And yes, the resulting call should be discrete(0, 0), that's why I asked whether the "-1" could actually express the wrong logic.


Проверь как на сайте отображается. У меня что-то все в кучу.


Unfortunally yeah :(


What's the explanation for why the last parts are missing? I thought it might have been dropping default values, but some of them are 0 and some are 1.


@popcornmix @smallint could either of you send me a pr? i only have a tablet here


Мне кажется, нужно ссылки в описание задания вставить


nit: close paren on new line (you don't need to fix this; this commit is still strictly better)


Hmmm. It seems this bears some thinking about.

Perhaps it's just an English thing? To me, the difference between "not following" and "ignoring" seems quite clear. I don't think experience as a user has much to do with it; we use these terms in everyday English, as well. For example:

- If you're not following a conversation, you're not *actively* paying attention to it.
- If you're ignoring a conversation, you are *actively* not paying attention to it.

As you said, the latter is an action performed by the user. The former is of course the default, but I think @tobscure chose to use the term "Not Following" to emphasize the contrast between "Following" and "Not Following" as the two *main* states. That is:

- "Following" is either ON or OFF.
- (Or you can ignore.)

I'm wondering if perhaps *ignorer* means something closer to "not paying attention" in French than in English. It could be a case of cognate term that has taken on a difference emphasis. If so, I can see where that could be a problem. Of course, you're free to translate the terms however you think best.


```aes_string``` doesn't work for me. I get ``` no visible global function definition for ‘aes_string’```


Fine - anyway, the case where the difference matters is constructed and unrealistic.


Nit:  It's not clear from this comment why this is needed. Could you specify that this is to allow benchmark-specific flags?


@ajithcj @alwey
Some interesting formulas:
1) 5 * log(1 + log(1 + idx)) coincides with my formula on threads 1-7, and with ajithcj's formula on threads 14-20.

2) 3 * sqrt(log(1 + idx)) is more close to alwey's formula, keeping (0,2,3), and adding additionalDepth on higher threads moderately.

3) 3.1  * fabs(sin(0.785 * idx)) is altering 0, 2, 3, 2, 0, 2, 3, 2, 0, ...

@ajithcj : if your test passes STC, could you please start LTC with formula 1)? Or simply retest STC? There is no need to avoid proven (0,2,3) on the first three threads because there are many combinations using this, like the three formulas proposed above.

I hope to be helpful here,
Ivan


@Chima1707 it better we let the caller catch this exception and handle it. that is lets leave out the `.catch()` block


Implement  `rollback()` for `UpdatablePageSource`


Yes, since it expresses intent. The conversion bool -> {0, 1} is really ugly, you just got used to it too much.


Yes, that makes sense, too.
May be it would be good to add `INT-3859` as a comment to this test...


The passphrase for encrypting the transfer device must be
diceword-generated, as it must be typed out on 2 machines.

I'm beginning to streamline in-person trainings by saying at the very
beginning: you will require 5 diceword-generated passphrase. (For journos;
admins might need 1 or 2 more-- I haven't counted yet...) I recommend
adding that info to the docs, too.
On Oct 22, 2015 15:22, "Noah Vesely" <notifications@github.com> wrote:

> Regarding line 150 of install.md. User passwords are notoriously
> insecure, even when they're told to "feel free to make a good long one."
> They should be generating one with KeePassX or Diceware.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/freedomofpress/securedrop/commit/1109f86c8d46f5a593bac9a4b6330fd3c9b1cd80#commitcomment-13935961>
> .
>


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


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1420 of 1420 tests passed)

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


This closes issue SoftwareEngineeringToolDemos/ICSE-2013-MCT #5.


The problem was that middle_from and middle_to were ***on*** the boundary.

But your solution was to move those points, instead of using new points and moving them to the location off the boundary.

Now you are using the moved points @191 and moving them outside again, which is now harder, because they were moved already - chances are now higher that the `max_moveInside_distance2` will not be reached.

Note that I did use such a schema at line 189:  `Point from_outside = middle_from;`
Then from_outside is a copy of middle_from, and from_outside is moved instead of middle from.


Nit: specify that the value returned is `BENCHMARK_FLAGS`, and why.


We should turn this file into a admin.wsgi.in file and let this file be an output of the configure script. We should avoid hardcoding paths for files that can change location based on path, have a look at the .service.in files for an example on how to do this.


Shouldn't this be static?


This could potentially raise `FileNotFoundError` (`IOError` in Python 3.2).
For safety reasons I would suggest to wrap the `os.remove` into another try-catch:
```python
try:
   os.remove(...)
except FileNotFoundError:
   pass
```


Лучше не таблицей


I am generally not convinced that is needed - `git blame` is your friend.


Also add this check to `deleteRows` so that we can fail early.


I am. The only issue is index.html that messes with root of page overriding
the root url. Thinking about it, a basic reorganise of hymn files will fix
this.
On Tue, 20 Oct 2015 at 13:24 Addy Osmani <notifications@github.com> wrote:

> I wonder if app.use('/..', express.static(__dirname + '/../../dist'));
> would be of any help here. You appear to be trying to virtualise anything
> in root back to /../../dist/. I may be missing something.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/addyosmani/app-shell/commit/7c8ebb81565c48700d6d34eb282a8c9937f75335#commitcomment-13872999>
> .
>


Not a big deal, but try to align all slashes in the same column


This closes issue SoftwareEngineeringToolDemos/ICSE-2013-MCT #6


OK. That's fine, too.


So freetds 0.95 is unstable and causing segfaults with mutlipul result sets with mssql.  This is not an issue with ports because they use freetds 0.91.  Downgrading via brew edit freetds and replacing the 0.95 formula with 0.91 resolves the issue.


If this is the only place we use the coverage bool, could you pass a path_prefix string rather than a bool?


You need to add a line like this: https://github.com/QuackenbushLab/pandaR/blob/master/R/plotting.R#L14-L20 to the documentation when used (instead of aes though put aes_string) and call `devtools`' `document()` function so it is imported.


This is fine for now, but I would probably use another description and package name here, calling it "integration with Apache" seems a bit misleading, this ought to be the default way to run it. There's no need to fix this right away but food for thought before we go live with this.


java convention == camelCase


@ccx-testerbot runtests master


I guess you have a line ending problem because some files have all its lines modified. Check the autocrlf flag of GIT.


Example to cause segfault:

```
<?php
require_once("dbinfo.php");
//putenv('FREETDSCONF=/opt/local/etc/freetds/freetds.conf');

$pdo = new PDO("dblib:host=" . DB_HOST . ";dbname=" . DB_NAME . ';charset=UTF-8', DB_USER, DB_PASS);

$multipleresults = "SELECT 1; SELECT 2; SELECT 3;";

$stmt = $pdo->prepare($multipleresults);

print "Executing SQL: '{$multipleresults}'\n";
$stmt->execute();

print "Looping through results. Should segfault on nextRowset\n";
do {
    print_r($stmt->fetchAll());
} while ($stmt->nextRowset());

print "Complete.\n";
```


Thank you for the request. It has been added to the queue (id: 127). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/127) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


@ajithcj : maybe the best solution is to try all formulas on STC first, because LTC is so expensive here. And we expect higher elo difference on STC, as it was the case with pre_merge testings.


https://github.com/search?l=ruby&q=%22callback_url%22+%22OmniAuth%3A%3AStrategies%3A%3AOAuth2%22&ref=searchresults&type=Code&utf8=%E2%9C%93

321 files...

The biggest problem could be with these projects, which redefined build_access_token
https://github.com/search?utf8=%E2%9C%93&q=%22callback_url%22+%22OmniAuth%3A%3AStrategies%3A%3AOAuth2%22+build_access_token&type=Code&ref=searchresults

For example basecamp:
https://github.com/Verano/omniauth-basecamp/blob/9bcb88e6472891236a7591ec5086e88654e52aa2/lib/omniauth/strategies/basecamp.rb
```
:redirect_uri => callback_url,
```

From these projects which didn't redefined build_access_token, the most usual case is that they used callback_url in same rudimentary way:
https://github.com/tkengo/omniauth-dropbox-oauth2/blob/eb7d19cff04b4046f8bba44903674e9340684e1e/lib/omniauth/strategies/dropbox_oauth2.rb
```
  def callback_url
    if @authorization_code_from_signed_request
      ''
    else
      options[:callback_url] || super
  end
```
and few of the projects use @authorization_code_from_signed_request or options[:callback_url]. So it won't change their work. But still some of the projects use these params...


`uname -m` would be more appropriate; prevents e.g. matching "64" in the hostname or kernel version.


This is not testing anything. Can you read the values out and check for existence/non-existence?


<!DOCTYPE html>


Same


@mfonville Yeah, it's apart of the Play Store beta for Google Now.


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


also bro ım still not sure which branch that we should choose...
because ı found that stable branches(like stable-yogpn4) dont include newer upstream commits...
cm maintainers are merging newer commits into cm-12.1 branch directly...
so probably we should still use cm-12.1 branch for our builds but as i said ım not %100 sure :)


I'm going to revert this as it's breaking fresh installs.


去看看 BFC是什么 flex布局


Fixed


Do we have rules about method ordering?


I'm going to revert above as it's breaking fresh installs.


Even you don't want to add a Manager class instead, the functionality looks highly like one :)


According to this: http://drumcoder.co.uk/blog/2010/nov/12/apache-environment-variables-and-mod_wsgi/
We could use SetEnv to set the configuration file from the fleet-commander-apache.conf file which is already using .in substitutions.


Shouldn't the user be queried for a location to save the report? I.E using a [`FileChooser`](http://docs.oracle.com/javase/8/javafx/api/javafx/stage/FileChooser.html)


any reason to use `#[]` here?


border-radius 其实目前可以直接用 这题主要是让你熟悉margin


That's really cool - I'll make a point of getting the first Gremlin article read before then.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 759](https://teamcity.labkey.org/viewLog.html?buildId=279651&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 20, passed: 240, muted: 23 Build time: 00:42:32
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

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


Could you fill in one or two strings into the dict so we know we actually pickled the file correctly?


html   不要出现空格 用css的 padding margin去实现效果


each argument on its own line is the prevailing style on the native side, and the most common on the js side. just `grep "function.*([^)]*$"` for examples.


github's interface for moves is :(


sink.flush?


不要用  id选择器


nope, if there's an exception here then the test should legitimately fail


去看下 怎样设置字体比较规范


the fact that a dict goes out (and not a list) is enough


Ship it!
```
                                   (`-,-,
                                   ('(_,( )
                                    _   `_'
                                 __|_|__|_|_
                               _|___________|__
                              |o o o o o o o o/   hjw
                             ~'`~'`~'`~'`~'`~'`~
```


Align ``options`` with ``sorted``.


testing it together with tag_results


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2010](https://teamcity.mesosphere.io/viewLog.html?buildId=63028&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21