TeamCity NEventStore :: MongoDB :: Build [Build 54](http://build.prxm.it:8813/viewLog.html?buildId=3402&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:39


yeah we do need to mention these on the debian package

Alejandro
Head of Software
www.kano.me
@alex5imon <https://twitter.com/alex5imon>

On 20 January 2016 at 16:54, Radu Jipa <notifications@github.com> wrote:

> @neue <https://github.com/neue> @alex5imon <https://github.com/alex5imon>
> Do we need to include any license files here or just naming them is
> enough?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KanoComputing/terminal-quest/commit/ae4a07c3c26ce925a6e96eda7360d4d575d42f5d#commitcomment-15563030>
> .
>


Hi Team,

After rereading the instructions, I just realized that I interpreted this instruction wrong "move to the my-favorite-things folder and copy the "media" folder, then, rename it "2015-media."  Instead of copying and renaming the folder I renamed the media folder. Yikes.


Dans les faits on limitera en fonction de ses manches


sent it just as a js doc before, should be pushed now though :)

> On 20 Jan 2016, at 09:34, Rachel Smyth <notifications@github.com> wrote:
>
> Hey! I don't see the playToX function... did you push it up?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/douglaswissett/wdi-fundamentals-rps/commit/b9a28bade68b93dc574cfc52662d11bd40f36b69#commitcomment-15558841>.
>


> Reason for removing unique constraint

Oh oh that's a mistake, fixing..


马上开始学Spring，等我一个月后来支援你，记得写好注释:joy:


Ça serait pas mieux de mettre ça dans des logs ou whatever?


Nope, that was just my editor.
I'll "fix" it.


Commit messages are a little non-descriptive. Why did you make these changes? What changed? Knowing that it contains `date filter updates` is not very helpful. :grinning:


fait et fait


It actually showed up in make schema-diff, but I didn't notice it because of the noise. Changing the label on https://github.com/gratipay/gratipay.com/issues/3897 to DevX+


fait


TeamCity Civic Assembly :: Build backend + frontend [Build 14](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=392&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


host --> server


TeamCity Civic Assembly :: Build all [Build 20](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=393&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Thanks @audev I'll test and try to rebuild with new data before pushing to production. Heads up @nikolai-b


I think this break scripts..

http://i.imgur.com/eh3Joba.png


Oh yeah, thanks!


Casing fixed here: https://github.com/TheOriginalJosh/TypeScript-Angular-Components/commit/edb56681e75919d7196314cd3f9ed257de06244b#diff-5f25b2d7085f090327b6b438d6a88967R13


locked sera remplacé par une vérification de ending_message


Je crois que les lumières dans le menu de Resogun sont correctes.


Ah right, of course.


TeamCity Civic Assembly :: Build all [Build 20](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=393&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2; exit code 1; exit code 123 Build time: 00:00:42


TeamCity Civic Assembly :: Build all [Build 21](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=394&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


These static_asserts should go in the .cpp file to be consistent with the rest of the PAL enums.


@ghedamat Can we have the official command option be `--show-progress` but alias is as `--progress`?


`--save-dev` seems more legit, no need to have the loader at runtime


报一个bug， isInStage 只有一个参数，第一个参数不需要


TeamCity Civic Assembly :: Build all [Build 21](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=394&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2; exit code 1; exit code 123 Build time: 00:00:36


Could you revert the change to package.json - I will update the version numbers myself later on. The rest looks great. Thank you.


For what it's worth, Stroustrup (4th Ed, §20.3.2) recommends against repeating `virtual`. But I disagree with him.


\*sigh\* Right. Thanks for the reminder.


Use commit messages. please.


Is there a reason why it doesn't return "not found"?


pass in a child here (just straight from the AnimatedBuilder config)


Fixed, and force pushed to heroku and github.


Not sure I agree from the perspective that we want to educate people on what is available to them (generally speaking).

For instance, its much more handier to me to be able to open up the NBT constants class and have them all in one spot.


(and use that as a hook to put in documentation explaining why you would want to do that)


Yeah, it's a dll. My other trainers work fine, but this one, I need to freeze the value.


Why not use "new"?


according to other files there should be a line beginning with a #, too


use that here too


The arguments passed to this function are in a different format that is not supported by LumiList, that's why all this handling.
Maybe, if we get the lumiMask directly from the schema (not from the WMTask object), then we could retrieve it in a good format to create a LumiList object.


I must be missing something. Please point me to the line where `assert(geoip_country.open(nil, flag)):close()` is now called.


makeCacheMatrix <- function(x = matrix()) {
  inv <- NULL
  set <- function(y) {
    x <<- y
    inv <<- NULL
  }
  get <- function() x
  setInverse <- function(inverse) inv <<- inverse
  getInverse <- function() inv
  list(set = set,
       get = get,
       setInverse = setInverse,
       getInverse = getInverse)
}

cacheSolve <- function(x, ...) {
  ## Return a matrix that is the inverse of 'x'
  inv <- x$getInverse()
  if (!is.null(inv)) {
    message("getting cached data")
    return(inv)
  }
  mat <- x$get()
  inv <- solve(mat, ...)
  x$setInverse(inv)
  inv
}


I would think it the other way around. but I'm not married to the idea.


lgtm, I am assuming that the change in types are properly handled later on.


Aeson.decode text serait supposé marché direct non? il y a un instance de FromJSON Text


that's awesome :) :+1:


I think there is something wrong in line 21:
+  str.gsub(/[\x00-\x08\x0b\x0c\x0e-\x20]/,'?')

Now every " " (space) is replaced by '?' in the changeset replication files.


i'm sad about how using AnimatedBuilder instead of a Transition widget here means the structure isn't as obvious and it's easier to make the slow variant (forgetting to prebuild the child). I think we should find a better solution, maybe multiple transition variants for the various old AlignTransition clients.


info messages are logged inside the AgentUtil class. These debug messages are part of the debug effort of this function. So let's leave as is.


Yes it does Thanks.


This commit fixes the problem. Also, this error is a warning now (at least as long as there are potential issues).


this commit lgtm


@JustAkan 수정했습니다. 커밋 확인 하시고 테스트 부탁드립니다.


Although not changed in this PR, I notice that TestLog.NUnitLoadError dispalys a generic message, completely ignoring the message that is included in the loadResult. We should look at this separately.


Great, thank you for helping with this issue

@pgleeson


The addition of the '$' works well, except if the unit has some latex formatting -  in which case it crashes. For instance, sometimes you might want to have inverse metres (m$^{-1}$) or something like that, but this won't work with the additional '$' tags; so I'll accept this pull request, but change this line back in another commit.


I am still not sure how the previous code failed to check that


updated bontania or actually removed?


This is pretty good—and definitely an improvement over what was there before! :-) —but it doesn't recover as if the user typed "do". The braces are still treated as a ClosureExpr, which means we used closure rules in parsing (like accepting a parameter list).

It's valid to start an *expression* with a closureExpr, but not a *statement*; we decided this was not allowed a while ago:

````swift
{ print("hello, \($0)") }("Kaylee")
````

and that if you really want that you can use parens. That gives us the freedom to say that `try {` is similarly illegal and probably meant `do {`.


@martinoppitz  can you make a pull-request for variant A?


`getSelectedParticipantIds` (just remove the "s")


A printf would be nicer here, IMHO. Something like: `printf 'SUBSYSTEM=="net", ACTION=="add", DRIVERS=="?*", ATTR{address}=="%s", ATTR{dev_id}=="0x0", ATTR{type}=="1", KERNEL=="eth*", NAME="%s"' "$2" "$1" >> "$UDEVFILE"`


Looks good Charley, just the line 785 that's causes a problem in some edge cases - I'll change this back in the next commit.


Are you sure?
https://github.com/robison/lua-geoip/blob/1d5b0b8d68cbb5a8ec660cfb949e7e905d8c6490/test/test.lua#L304


Ah yes, that is superior. I was unaware of that utils method.


Still, this doesn't explain why should you change existing checks


Why don't build system setup dalvik heap config in build prop. May be a clean build would help...


I don't think throwing from inside a ReleaseHandle override is a good idea.  @stephentoub?


Require


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2089 of 2089 tests passed, 0 test warnings)

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


The problem is that we need to do a data/iterable intersection between the lumi list returned by DBS and the lumi list for this task (which are initially in different formats). If we return a generator, we would need to iterate over each element anyways to get the final result.


Same comment as above.  I'm not sure throwing an exception is a good idea inside a ReleaseHandle.


Agreed, I opened #3900.


How about ?

            info.put(getName() + " Details",
                    fairnessInfo[8] + " (avg: " + fairnessInfo[0] + ", rms: " + fairnessInfo[1] +
                    ", Pmax: " + fairnessInfo[2] + ", Pdev: " + fairnessInfo[3] + ", Perror: " + fairnessInfo[4] + ", Pss: " + fairnessInfo[5] +
                    ", Jain's index: " + fairnessInfo[6] + ", max: " + fairnessInfo[7] + ")");

With the change in testFairness returning a square root of sum of squared penalties (RMS -- root mean square).

    DataForEval[1] = sDoubleFormat.format(Math.sqrt(sumOfSquaredPen));


my test comment on my first commit


里面全是垃圾


@neobuddy89 This commit due to
https://github.com/YuPlayAOSP/android_device_cyanogen_msm8916-common/commit/e80537e8ba13d4dbff032424f8732e50a1afe2f1


Не надо так, каждый будет видеть всех адресатов, нужно отдельные письма отправлять.


(in a future patch) we probably want to make this chainable


@lipusz is it not necessary to test the case of `true true` for a direct member of the parent organization since it uses different logic than the call in line 318 above?


when you find a greg iron vain it is normally countless thousands of blocks spanning dozens of chunks.


Just to confirm, this is the primary github, correct?

Getting notices on that https://github.com/SickRage/sickrage-issues/issues/3#issuecomment-173277859 that its actually changed and thats the new one. I recall seeing that other devs are trying to switch over and don't want to get caught up in drama.


Beautiful! I love using the prompt to get user input for the amount of turns they want to play (or maximum wins), and nice job using parseInt! Wünderbär! :star2:


Jordan, feel free to improve this diagnostic further if you feel so inclined.


Tabs


i added this line


ok ! thanks a lot !


tabs


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.3-beta.3](http://build.prxm.it:8813/viewLog.html?buildId=3403&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:03:10


Could you please use iInstructorsValue to follow the same naming conventions as in the rest of the solver? Prefix instance variables with i (and with s for static variables). Same applies to other instance variables to the InstructorFairnessContext and Instructor classes.


Here is my use case for this this sensor:  On my Aquarium Ive got a Red/Greed LED for a visual indicator.. it changes from green to yellow when its doing something (like filling/draining), it goes to Orange when something minor needs attention (time for maintance)... and then blinking RED when something needs major attention asap (flood detected, temp range out of bounds, etc).. that status LED is reflected in Domoticz with this sensor, and the text shows a quick reason.


maybe this will help https://github.com/faif/python-patterns/blob/master/registry.py


I'll rebase it instead, I think


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - x86_64-MacOS-10.9-clang: OK (2050 of 2051 tests passed, 0 test warnings)

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


Не надо обращаться к доменным сервисам из инфраструктуры. Инфраструктура может работать только с другой инфраструктурой или с Common.


if (вот тут происходит проверка на то, является значение true  или false) то есть нам в целом не обязательно делать приравнивание к true или false, достаточно написать if(a) и если true,  то отработает первое условие и т.д.


Since testapi uses CoreFoundation we will never be able to use it (at least until upstream fixes it). So this change is not going to be temporary.


In this case may be use `QThread::terminate`?


Is it possible to pass [ static, 'isValidItem ] as callable here or do you think wrapping it in anonymous function provides more readability?


TeamCity Civic Assembly :: Build backend + frontend [Build 14](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=392&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 70 (1 new), passed: 1597, ignored: 7 Build time: 00:18:02
### Failed tests
```
Organization tests: Should not create new organization if cancelled: <no details avaliable>

Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

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


Btw, you can save a couple of lines with

    $buildTarget .= " testapi" unless isQt();


this commit lgtm


@Spoonikle Testing, culling non essentials.


This looks good. Going to commit.


"not found" is a construct in the city library, where if the pRecord == null, then the string "not found" is pushed onto the lua_State. asnum.c simply returns a nil if an entry for the query does not exist, since there's no pRecord data structure that needs to be populated.


... commit message... :disappointed:


implementation -> programming?


потестировал на отлично


What's the rationale behind change in this condition?


Use `return fn(error);` so that the main logic for this function isn't inside a conditional.


Good Work


I got my modules but I'm still waiting for accessories to assemble everything (from China).

I got a pretty tall tree in my yard (~50ft), so I should be able to install an antenna up there if I need to.

For ham radio, there is also https://www.reddit.com/r/amateurradio and https://www.reddit.com/r/HamRadio.


No problem, that's a perfectly good reason.


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


доставай это из репозитория


Wouldn't it be better to keep API consistent here?


In case updatePermissionsToShare() is called in all your cases you could just move below the switch-case statement :)


Here a proposal is to not propagate $p_sort_field, $p_sort, $p_dir through the call tree (which are only used here)
As they are all filter parameters, they can be retrieved by accesing current filter properties, by calling current_user_get_bug_filter.

However, there needs some way to know if filter is temporary, and which token id, to set the appropiate parameter in sorting link. This could be done by:
1) passing the temporary nature through the call tree( as is done currently)
2) storing that info inside filter properties.


Yes. That specific line checks to see if the test being executed is against a DB type named "asnum". If not, it executes and attempts to index against p.field[i] (https://github.com/robison/lua-geoip/blob/1d5b0b8d68cbb5a8ec660cfb949e7e905d8c6490/test/test.lua#L305). This check is done so that we don't attempt to index p.field[i] against asnum DBs, since they return only a string and not a data structure.


Окей, спасибо, но я надеюсь справиться с нагрузкой, со временем нет проблем

вторник, 19 января 2016 г. пользователь Igor Alexeenko написал:

> Привет. Кроме проведения лекций, иногда я прихожу в репозитории и немного
> комментирую, так что не пугайтесь оба. Пришел, вот, сказать, что проходить
> два интенсива параллельно будет очень сложно и нужно подготовиться к
> ситуации, что в один момент придется выбрать какой-то один)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pselok/152141-code-and-magick/commit/0d6f7830bad417614267bb37578f205d1afa0c0a#commitcomment-15540722>
> .
>


I was attempting to keep with the spirit of the testing suite and have the tests run iteratively, inline.


LG. I like the unifying of logic


Pose: Position und Orientierung, Quaternion: nur Orientierung (Rotation).


Pose -> pose


This may produce too big output and break Travis.


I only skimmed through the paper, but this really seems like an unneeded
microoptimization... But I guess it doesn't hurt either :)
On Jan 20, 2016 15:18, "Geoffrey De Smet" <notifications@github.com> wrote:

> yes (those that don't rely on filling with null values at least)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/droolsjbpm/optaplanner/commit/132669d233d8f30aae378e795622fd8fc19d5f4c#commitcomment-15558416>
> .
>


needed?


WicketStuff projects do not have very good javadoc documentation anyway.


Good one! Taking it in.


typo: "interested"


DirectION Cosine ...


:ghost:


See L20.


Since there is actual string data being returned, it's conceivable that 'not found' could be a part of said string data in the asnum DB. Returning nil is the safer solution, instead of creating the additional and unnecessary complexity of a data structure solely to house the contents of the returned string data, or "not found" if the record is not found.


Also pay attention to line 314, localhost is used there


Here is some code that probably should be cleaned.
Do not use raw json filter strings as url parameters.
Refactor the tokens get/set to allow multiple filters per user, (and get-by-id)


"software and documentation changes"? since we do have a Documentation updates section


@gabizou ping


ага, зато пройдет завсегда


вокруг знака "=" могут быть пробелы


... that transforms vectors expressed in A to B


Sorry, backing up. This flag indicates that this lookup is known not to affect the interface of the file, for incremental build purposes. Functions are special-cased pretty often, since both the function parameters and the function body use the function as the DeclContext, but only the parameters affect callers. Normal name lookup is conservative here, but a caller might know something is never going to affect the signature of a function even though the DeclContext is a function. This particular case is a little tricky:

- If this code path can be used to look up types for parameters or generic constraints, it *does* affect the interface of the file even when `DC` is a function.
- If this code path can only be used to look up values, it probably does *not* affect the interface of the file when `DC` is a function, because that will only happen within the body. Note that this rule will change when we make default parameters caller-side.
- But since this is in error handling code, it's probably okay to treat the entire lookup as not affecting callers regardless of the DeclContext. Whatever lookup that got us to this point has already been recorded, and since we're on an error path we know that this file needs to be recompiled anyway.

In the future, if you're unsure about one of the incremental dependency flags, please don't cargo-cult them; just leave them all out (or ask me or @cwillmor directly). That's nearly always safe. (The best descriptions of the options are on the ASTNameLookupFlags structure, but even that's rather sparse. Is there a better place to put this information?)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.658.f38fea059c12eb92476dc7ae2fa34f2d5ff97513](http://build.palaso.org/viewLog.html?buildId=68156&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:06:09


Active rotation that transforms vectors expressed in B to A


Please use 'GID_FMT` instead of `%d`


Please use 'UID_FMT` instead of `%d`


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.647.f38fea059c12eb92476dc7ae2fa34f2d5ff97513](http://build.palaso.org/viewLog.html?buildId=68155&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:07:05


Thoughts on ditching this method? It doesn't help naming, typing, or duplication IMO.


Hola,

Tu amigo/a manzorenato te ha enviado un mensaje hace 36h días que aún está sin leer

Tu mensaje será eliminado en las próximas 24 horas.

Si quieres saber qué te escribió manzorenato, haz click aquí: a.iqelite.com/entry/new_message/2fdaa05590c20c604aa2f0ac2d3fda9e1f544b80bfb/320c612f7502e5151587cbea8586d58907b6675b683bebd8ac48258906d9089518283bd6eccc5b6747460a9b6896683b68267ccbeb0cb9060b7cb5151515151515158585f6eb0ccbd67c49060a8587b67b7c45152bd826d60b7cbc320c612f75044d3e22edb35af44b80bfb/?hg=UHhyRYJPG6

Saludos,
 El equipo de IQ Elite

---------------------------------------------------------------------------
Este mensaje estaba destinado a reply@reply.github.com. Si no deseas seguir recibiendo mensajes de IQ Elite, haz click en este enlace: a.iqelite.com/common/unsubscribe/?code=d79e9400&email=reply@reply.github.com.&hg=UHhyRYJPG6 IQ Elite es un servicio con oficina principal en: C/ Gran de Gràcia 15, 2º 1ª, 08012 Barcelona, España.
---------------------------------------------------------------------------


For this comment i didnt account for actually using the token ids to allow multiple temporary filters per user, by actually using the passed id number


This revision is clean.


This would cause the Python interpreter to throw an exception. Are you not seeing this?


FYI that long command: ```sh
sudo docker-compose -f docker-compose.circle.yml run -e ST2_GITURL="https://github.com/stackstorm/st2" -e ST2_GITREV="master" -e ST2PKG_VERSION="1.3dev" -e ST2PKG_RELEASE="999" -e RABBITMQHOST="172.17.0.1" -e POSTGRESHOST="172.17.0.1" -e MONGODBHOST="172.17.0.1" trusty build
```
is for Circle really which has built-in mongo and rabbit and some specific environment.

Locally you can use just simple (default way):
```sh
sudo docker-compose trusty build
```

This will be fully isolated (including containers for mongo and rabbit).


Hola,

Tu amigo/a manzorenato te ha enviado un mensaje hace 36h días que aún está sin leer

Tu mensaje será eliminado en las próximas 24 horas.

Si quieres saber qué te escribió manzorenato, haz click aquí: a.iqelite.com/entry/new_message/2fdaa05590c20c604aa2f0ac2d3fda9e1f544b80bfb/320c612f7502e5151587cbea8586d5851906d67a9bea87483cb897467ccb683ccb7c452b0bd895858825175c5af83a889586d75a9b66d9060b7cb5151515151515158585f665f67af75af9060a8587b67b7c45152bd585f588274c320c612f75044d3e22edb35af44b80bfb/?hg=KshMVYCAJ6

Saludos,
 El equipo de IQ Elite

---------------------------------------------------------------------------
Este mensaje estaba destinado a reply@reply.github.com. Si no deseas seguir recibiendo mensajes de IQ Elite, haz click en este enlace: a.iqelite.com/common/unsubscribe/?code=1fa02401&email=reply@reply.github.com.&hg=KshMVYCAJ6 IQ Elite es un servicio con oficina principal en: C/ Gran de Gràcia 15, 2º 1ª, 08012 Barcelona, España.
---------------------------------------------------------------------------


What's the format? We can always extend LumiList should we need this again.


Good catch. Release branch, thanks @chirag04! Fixed.


А что будет если нам сюда придет реально объект? Мы же по условию задачи не знаем, что нам должно прийти. Могут быть различные варианты. Тогда наш код отработает не корректно. Надо бы поправить.  То есть проверять на массив лучше  не через typeof.


https://github.com/robison/lua-geoip/blob/master/test/test.lua#L97

I was mistaken, I remembered taking a look at this last week and researching the INDEX_CACHE flag. I did not actually remove it.


Okay, then please *also* add a check for it being in the current module.

(We don't want to emit this note on an enum that comes from a package you're using or something. Today that has no source location, but that might not always be true.)


In Lua it is an idiom to `return nil, error_message` to indicate error. There is no need for extra data structures for that. Just return a single value, string, if data is found. Return two values, nil and "not found" if it is not.


13.5/15. Make sure you have the list of files that are in your folder and comment more on what is occurring. Otherwise, great job!


Oui ^^, il reste plus que le rectangle des menu qui sont pas tres bien rendu ^^
Plus les erreur du style qui tourne en boucle
>{rsx::thread} RSX: write_vertex_array_data_to_buffer: frequency is not null (12, index=8)

 ![capture d ecran 2016-01-20 18 26 47](https://cloud.githubusercontent.com/assets/5261759/12457152/611d8ed0-bfa3-11e5-840f-d3a877f1b5e7.png)


I like that you separated the commits between the alphabetize fix and adding severity, but it looks like the alphabetization of the spec side didn't get into the right commit. Mind rebasing to address that?


thats wrong


Should return true here presumably.


@mdtusz might as well.


isn't `$(element).get(0)` equivalent to `element`?


@mdtusz as requested.


Please note the first argument is nil in my question and geoip_country_filename in your answer.


This was missing the leading dash sign


aww, but my plan to fuck gregtech was go botanica :P


`ln -s ../../../../Downloads/arduino-1.6.6 ~/Downloads/arduino-1.6.6` would make this not need a file change


Соответсвенно тут кейс на тип объект так же не корректо отработает. А бить проверку на switch и отдельный if для массива - это немного странно.


double quote should be outside `%}` (also on line below)


Gregtech went first :P


The default case won't happen. The type is either []byte or string (original string or something converted to string).


Seems like `custom_string`s are required


good!


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


TeamCity NEventStore :: MongoDB :: Build [Build 6.0.0-unstable.19](http://build.prxm.it:8813/viewLog.html?buildId=3404&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:48


TeamCity NEventStore :: MongoDB :: Build [Build 6.0.0-unstable.19](http://build.prxm.it:8813/viewLog.html?buildId=3404&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:49


Par contre resogun c'es tbon, mais plus aucun autre jeux marche au dirait, bleach me fait une erreur comme je t'ai dit avec le d3d12buffer naruto crazsh sur le menu au moment d'afficher naruto etrange


The value of 'total' is still zero.


Type 1 is to create a new filter
'temporary' parameter is used so as to save in tokens instead of DB.

What is changed in this page basically is:
Do we have 'temporary' parameter? create new filter, and store it in tokens, not in DB
Do we have 'filter' parameter? retrieve filter from tokens, modify, store in tokens again.


test comment


Current QtWebKit uses WINDOWS implementation instead of QT. Look at Source/WebKit2/Platform/WorkQueue.h, and also see WorkQueueWin in Source/WebKit2/Target.pri

I'm not absolutely sure if we need to do it this way though. I guess it was done this way to simplify implementation of WebKit2 on Windows (the same thing on Mac, where DARWIN implementation is used). But we probably are not going to support WK2 on Windows because its implementation was removed from trunk after Qt departure because AppleWin dropped WK2 support and no other users existed.


what do you mean? went first? Like gregtech reached orgasm first? or you tried removing gregtech first to see what happenes? or somthing even more kinky?


Looks good. I'll check it out when I get home and merge it if there's no issues.


webcams


I edited the ``{scijava.jvm.version}`` property to run the test from the CLI.
Setting the for the tests only sounds like the better option!


XD


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1236.8338a72b500573fe43f504af8dda12a00c094628](http://build.palaso.org/viewLog.html?buildId=68158&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 840, ignored: 7 Build time: 00:08:37


This became necessary after rebasing onto apple/swift `master`. It mirrors 62cdab7 and is required to build the project; see: https://github.com/SwiftAndroid/swift/wiki/Getting-started#building-from-source :+1:


Ah, you're correct. In the original test, it would fail outright in my initial testing. I may be doing something incorrectly.

lua: test/test.lua:80: lua-geoip.db.country error: failed to open database file
stack traceback:
	[C]: in function 'assert'
	test/test.lua:80: in main chunk
	[C]: ?


@simon3z Is this the right way to migrate ?


Seems `/var/sockets/` is not the standard.
It's best to place socket with non-root permissions into directory where program lives or in `/tmp`.
`/run` and `/var/run` is used by root processes.

I suspect `/tmp/st2api.sock` would be best, just near the `mongodb.sock`, see:
```sh
root@ubuntu14:/# find / -name '*.sock'
/run/rpcbind.sock
/tmp/mongodb-27017.sock
/run/docker.sock
```
And this also will simplify life since we won't need to create/chown directory anymore.

Resources:
* http://unix.stackexchange.com/questions/88083/idiomatic-location-for-file-based-sockets-on-debian-systems
* http://stackoverflow.com/questions/7580346/where-to-place-unix-domain-af-unix-sockets-end-points-files


That's ultimately why I removed that from my version.


wtf


@FPtje This is actually the Hammer camera spawn point. info_player_terrorist is meant to be a blacklisted by the creator of the map, but you know, support for all maps and such.


File Formats/Images/UCG Image Format version 3.md

These &emsp are HTML entities indenting the following characters a bit. 3rd level index entries are always a bit more indented. Since markdown didn't allow me to create a good-looking index without using them, I had to use them.

What do you mean? "Your formatting is borked"?


1. Add '()'
2. Remove redundant true/false


Resolution failed! :frowning:


All done.


Agreed with @demhydraz on everything (weird, eh?). Also, it'd be nice if you could link to appropriate Wikipedia pages when using the official terminology such as "x86 words".


It's possible this may be a Univa Grid Engine (8.3.0) specific change, I have not tested it on any other versions of Univa Grid Engine or on any version of Son of Grid Engine.


this stuff should be moved in .lang file isn't it?


There's a bit of unnecessary mutation / data movement here:
- prices set to 0
- prices reassigned to the result of a map
- then that value moved to body.prices
Couldn't be simplified a bit?


TeamCity NEventStore :: MongoDB :: Build [Build 6.0.0-unstable.20](http://build.prxm.it:8813/viewLog.html?buildId=3405&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 76 Build time: 00:02:29


Looks liek we can prepare static string with this generated value
And why we need to use return argument instead of usual return value?


AFAIR, nil means "read files from the default location". Please check if this check works on your system with geoip-database package (or equivalent if you are not on Debian-based) installed and put it back to the code with an appropriate remark in a code comment.


Why 4? Add constants


Can you update this message text to be more user-friendly?

How about simply changing it to "Please login to continue."


It's gone now...


I'll update this. Right now, country.c does not return "not found" - it simply returns lua_error(L). Should I do the same for asnum.c?


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-MacOS-10.9-clang: OK (2051 of 2051 tests passed, 0 test warnings)

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


Please, add test for `operator=`


il manque le ; a la fin ;)


weighs?


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (957 of 957 tests passed, 0 test warnings)

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


I think this should have been:

```js
_
    .chain({})
    .map(foo)
    .filter(bar)
    .value();
```


il manque le ; apres le )


je le reredit il manque le ;


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.3+5](http://build.prxm.it:8813/viewLog.html?buildId=3406&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:02:57


Remove this.


The sha256 is wrong


If you want to avoid "unsatisfiable dependencies" errors, don't list features until conda fixes their dependency resolver. Just add a build and runtime dependency on `msvc_runtime`, nothing else.


Yeah it is. Actually, I took that part from the "Google Brotli" and "Deflate" RFC (which is completely normal under RFCs). As you can read in the abstract, this document defines and **explains**.


It's not. Failures should result in returning false.


Yes, but what if the delivery mechanism for the database files puts them in a non-default location, or the package management does not set a system-wide default location? The test would still fail - it's very specific to system configuration, and not testing the library.


Why is this commented out? I think this is breaking ecommerce 360 reporting, because nothing is saving the campaign cookie anymore.


... as can be seen in / as shown in?


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


It did, please excuse my lack of experience with Lua. I'll revert this line to the previous state.


Well, something does (or did) return "not found", judging from the test code. Can you please look it up?


avec ton dernier commit la ligne a changer pour bleach et naruto me fait toujours pareil :/
![capture d ecran 2016-01-20 18 41 17](https://cloud.githubusercontent.com/assets/5261759/12457559/68aeba78-bfa5-11e5-88ba-cb46fff82a6b.png)


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


You deleted line geodb:query_by_name, and replaced it with geodb:query_by_addr. geodb:query_by_name is not getting tested now. Am I missing something?


hi
tanks for your traying
im run terminal in kali linux 2 x64 on vmWare workstation and
 $ git clone https://github.com/ResurrectionRemix/platform_manifest.git      OK
 $ cd ~/platform_manifest     OK
but when im write ./setup.sh terminal saying
bash:  ./setup.sh: No such file or directory
please help me in my problem.
tanks.:-*


Should we put the dates in? I know we haven't decided if we're adding the 3rd day, but we do have the 2 days at BPH locked up. Maybe we say 3 days? easier to back off that than add a third, I reckon (scheduling-wise for sponsors)


au par contre resogun estr parefait maintenant, sur le menu :)


It is a feature of maxmind's library, not our wrapper (AFAIR), so, it is not up to lua-geoip to decide.


That the domain name property of toothmandental.net be transferred in ownership to the Cantate Dot Co.


si no hay puertos indicados en el deploycmd, se usa por defecto 8080/tcp ?

pero también esta definido como defecto en el flag 'port' del metodo deployFlags():

		cli.StringSliceFlag{
			Name:  "port",
			Value: &cli.StringSlice{"8080/tcp"},
			Usage: "Puerto interno del contenedor a exponer en el Host",
		},

por lo que en este punto esta validación no sería necesaria... estoy en lo correcto? o es bullshit? jaja


I think we should capitalize "source" here


So in order for us to be on the same page - let's consider the line in 3.0 -
```js
elem.style.display === "" && jQuery.css( elem, "display" ) === "none" &&
 jQuery.contains( elem.ownerDocument, elem)
```
And line in 2.x -
```js
elem.style.display === "" && jQuery.css( elem, "display" ) === "none" ||
 !jQuery.contains( elem.ownerDocument, elem)
```

And check out what actually happens in alpha and 2.x for the case like this -
```html
...
<style>.hidden {display: none}</style>
...
<div class="hidden"></div>
<script>
$(".hidden").show();
</script>
...
```

In 3.0, first of all we hit this -
```js
elem.style.display === "" &&
```
No inline display, `true`, because of the `&&` operator, we move on to -
```js
jQuery.css( elem, "display" ) === "none" &&
```
Have computed display, but we **go on**, because of, again, `&&` operator and hit -
```js
jQuery.contains( elem.ownerDocument, elem)
```

Whereas before, in 2.x, we had this -
```js
elem.style.display === "" && jQuery.css( elem, "display" ) === "none" ||
 !jQuery.contains( elem.ownerDocument, elem)
```

So third condition - `!jQuery.contains( elem.ownerDocument, elem)` wasn't executed because of the `||` operator.

Now if call `show()` for the case like this -

```html
...
<div></div>
<script>
$("div").show();
</script>
...
```
with new alpha line -
```js
elem.style.display === "" && jQuery.css( elem, "display" ) === "none" &&
jQuery.contains( elem.ownerDocument, elem)
```

We actually do not execute `jQuery.contains( elem.ownerDocument, elem)`, because we bail on `jQuery.css( elem, "display" ) === "none"`.

Whereas before, in 2.x, for the line -
```js
elem.style.display === "" && jQuery.css( elem, "display" ) === "none" ||
!jQuery.contains( elem.ownerDocument, elem)
```
We actually still go to the `contains()`. Which means case of `show()` call for **hidden** element is now more expensive, whereas case for **shown** element is less expensive.

Sorry to be so verbose and repetitive, i would like to everyone get what are we talking about, if someone wasn't, that is.

In @gibson042 table, that shown in

```
show()
cascade-visible	less work (no contains)
cascade-hidden	more work (contains)
```
So for the case, when user **shouldn't** call `show()` - `cascade-visible	less work (no contains)` we improved performance and for the case when user **should** call `show()` - `cascade-hidden more work (contains)` performance is worse.

That is the first point. Now the second one - @timmywil writes -

> I might argue that supporting disconnected elements means not putting a display on it (it has no effect until attached, and then it might be the wrong display based on the cascade).

Which is not true, because in 3.0 we have -
```js
/*case #1*/$("<div/>").show().appendTo("body")[0].style.display; // ""

// But for the case
/*case #2*/$("<div/>").hide().appendTo("body")[0].style.display; // "none"
```

Whereas before, in 2.x, we had -
```js
/*case #2*/$("<div/>").show().appendTo("body")[0].style.display; // "block"
/*case #2*/$("<div/>").hide().appendTo("body")[0].style.display; // "none"
```

Which means for the case #`2` in 3.0 we expect for the user to know where their want to append newly created element, but for the case #`1` in 3.0 we don't. So in new concept, **detached elements are now considered to be shown**, which is counterintuitive and inconsistent in my opinion.

I believe previous behaviour was correct and expected, since we never seen complaints about it.
So to answer your question @gibson042

> what would you like to see answered or changed here?

I'd like these cases -

```
show()
cascade-visible	less work (no contains)
cascade-hidden	more work (contains)
```

to be reversed. I believe if we do it, behaviour or performance wouldn't suffer, but would be improved.


6 tickets?


Looking at the links @mlocati gave, `$environment->getRecord()` might be a fine replacement for your file_exists calls. I'm :-1: on a global function.


its


:+1: just a couple things I spotted <3


That the web site will be developed within one week from the time of an agreed contract.


What is this?


Like this: https://github.com/ClinicalGraphics/conda-recipes/commit/e5f70444f6da40c14852eb68ab91c4da6ee93850


Hi,

I agree with @fvicente, this commit needs to be reverted. It makes no sense and it breaks the lib.

First the syntax:  '/'. ImgCache.options.localCacheFolder ??
Here it means accessing the property "ImgCache" of the String object '/'.

Then, the method i.e. ImgCache.getCachedFileURL *does* exist on the latest version of API: https://github.com/chrisben/imgcache.js#high-level-api


<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8"/>
<title> Sailor wheel puzzle </title>
<style>
* { padding: 0; margin: 0; }
canvas { background: #eee; display: block; margin: 0 auto; }
</style>
</head>
<body>
<p> Just a regular web page, but with a canvas element.</p>
<canvas id="game" width="640" height="480"> Canvas goes here! </canvas>
<script>
var canvas = document.getElementById('game');
var ctx = canvas.getContext('2d');
var centX = canvas.width/2
var centY = canvas.height/2
prev_state = []
var rule1 = function(spoke_state) {
// TODO: change spoke state according to rules
prev_state = spoke_state; //we need a snap of wheel's previous position to
compare with current state so as to recalculate
spoke_state[Math.floor((Math.random() * num_spokes) + 1)] = 1 -
spoke_state[Math.floor((Math.random() * num_spokes) + 1)];
/*
update = [[1, 0, 0, 0, 0, 0], [1, 1, 0, 0, 0, 0], [1, 0, 1, 0, 0, 0], [1,
0, 0, 1, 0, 0], [1, 0, 0, 0, 1, 0], [1, 0, 0, 0, 0, 1]]
for(i = num_spokes; i > 0; --i){
if(prev_state[i] != spoke_state[i]){
for(j = num_spokes; j > 0; --j{
if(update[i][j] == 1){
spoke_state[j] = 1 - spoke_state[j];
}
if(update[i][j] == -1{
//do something else...
}
}
}
}
*/
return spoke_state;
// we could have a matrix. 6x6 matrix in this case.
// diag elements are 1, and elem [i,j] will be an integer say 1, 0 or -1...
// 0 implies ith spoke has no effect on jth.
// 1 implies toggle, i.e., modifying ith spoke WILL modify jth irrespective
of i's, j's positions.
// -1 could imply a conditional toggle like if I push in ith spoke,
// then jth spoke will be pushed in along with it, but pulling i won't
affect j. And so on...
}
function draw_line(x, y){
ctx.lineWidth = 600/num_spokes; //thickness
ctx.strokeStyle = "red";
ctx.beginPath();
ctx.moveTo(centX, centY);
ctx.lineTo(x,y);
ctx.stroke();
}
var draw_spokes = function (spoke_state) {
for (i = 0; i < num_spokes; i++) {
r = spoke_state[i] === 1 ? 250 : 180
draw_line(centX + r * Math.cos(i * 2 * Math.PI / num_spokes ), centY - r *
Math.sin(i * 2 * Math.PI / num_spokes))
}
}
var draw_wheel = function() {
ctx.fillStyle = "gold";
ctx.lineWidth = 1;
ctx.beginPath();
ctx.arc(centX, centY, 120, 0, 2 * Math.PI);
ctx.closePath();
ctx.fill();
}
function mouse_event (){
//I click a spoke and the state should toggle like...
//curr_spoke_state[clickedSpoke] = 1 - curr_spoke_state[clickedSpoke]
}
draw_frame = function() { // Grand draw at the end to call everything else
ctx.clearRect(0, 0, canvas.width, canvas.height);
draw_spokes(curr_spoke_state);
draw_wheel();
// Animation logic etc. goes here, will have to refer back to mdn breakout
tutorial
//console.log(curr_spoke_state)
curr_spoke_state = rule1(curr_spoke_state);
}
var curr_spoke_state = Array(6).fill(0);
//var curr_spoke_state = [0, 0, 1, 1, 0, 1, 1, 1, 1, 1, 1, 0, 1, 1, 1, 1,
0, 1, 0];
num_spokes = curr_spoke_state.length;
draw_frame();
setInterval(draw_frame, 122); //Currently updating every 122 milliseconds,
we need to change this to 'update on every mouse click read'
//mouse_event();
// So next steps are just to figure out mouse input
</script>
</body>
</html>

On 20 January 2016 at 19:53, nitinmdsu <notifications@github.com> wrote:

> I take it back, for loop was easiest there
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nitinmdsu/sailor-wheel-puzzle-game/commit/8147501b7750425724b88845845f66a3df450f03#commitcomment-15558543>
> .
>


That'll work out juuuust fine, as long as we don't start using custom compiled Python interpreters.


warning?


We do not have anywhere, moved to QuorumPeerTestBase so that it can be reused


111768812_VRFNoLoadOAFlow (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 0 test warnings)

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


Can't wait to give the new scale9Grid property a try later today!


revisa mi ultimo commit en este PR, saque el default del Flag para poder controlarlo mejor en la populacion de ServiceConfig


WARNING?


That the creation of a web site will be made in totality upon the deliverable date but for any additional specification requests at the time of said deliverable date, pending the recreation of a subsequent and original contract for the additional specification requests.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.648.db0a3f374a76d6f924780d8fd02c246e3c6aabea](http://build.palaso.org/viewLog.html?buildId=68171&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:07:16


Handled


You can mark it as pending: https://github.com/Quick/Quick/issues/16. Just change the `context` function with `pending`.


This is not lua-geoip deciding, this is simply not requiring a system to be configured in one very specific and optional manner for a test to pass.


That the annual fees necessary for the continued maintenance of toothmandental.net be entrusted to the Cantate Dot Co., no questions asked.


Ahhh yess.... finallyyyy....


I think the above piece of code is very much local to this test class. so not moving this to utility class


@gdi2290 yes I will. Can you cancel my last pull request. I would Like to distribute a clean version


Any strings like this should be converted into props with default values for now.


If they don't understand something, they can ask me; where's the problem?
Why not?


country.c does not appear to return "not found" - that is exclusively in city.c. The test checks for an error, and if there is an error (and the error result is NOT "not found"), then dump the error + result.


That any further alterations of the aforementioned terms be amedended in full to this, TOOTHMAN-DENTAL Gist, Branch, Commit # 6dfbb2c75e724c68bb654e814b0dda5cb0b0403b and made a private repository.


This is breaking the CDN option if you don't inline Raven.


r+


This is a good start. Keep going :-)


schöne Figure!


Done at a65a09c70e7be8c02927496f369076e102c914b4


Well I'm not sure basil will work


It includes the ltca, pca, ra and the root CA certificates.


sage-pdf-url function was in there twice. Deleted one. Added function for journal of neuroscience.


a nice fix, thanks


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1008.bab9ad24dc6ac370cd6b01ef876ee8b815ba2fe8](http://build.palaso.org/viewLog.html?buildId=68162&buildTypeId=ChorusPrecise64v25Cont) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:19:21


@ebeshero We need to talk about this at our mentors meeting. Tried showing this in class and got confused based on what the tutorial says. @spadafour can clarify further tomorrow.


sólo anoté algo que me pareció redundante cli/deploy.go:108 para revisar


May be verify end of search?


@Mumfrey

Give me an example of this in action.


open source


Great job on these... Right now you are in the lead for the prize!


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.649.090d71a13a2dde4e902bf7a7e5556e4e22e84c0e](http://build.palaso.org/viewLog.html?buildId=68172&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:05:09


Another corner case will be assumption that `www-data` user exists, which is not always the case. As i understand it comes from `nginx`.

I think I need to dig into apache or nginx packaging for this example, to see if we can create this user (ensure) without breaking anything else.

-----

You probably saw that we already create `stanley` user if not exists:
https://github.com/StackStorm/st2-packages/blob/a067cbda23a99cfb9b504c88ae827bdea1b4c11d/packages/st2bundle/debian/preinst#L18-L21
as example.


PlantCompBaseClassAgain (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed (1727 of 2056 tests passed, 0 test warnings)

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


what about making this a global constant?


@Zidane https://github.com/SpongePowered/SpongeCommon/commit/c257fe5d11f95246433c314b7f7a16f43d954178


I initially wanted to use a switch/case setup, but as far as I can tell, that does not work with 2 arguments (playerMove, computerMove). My brain seems to shut down when I try to think of a way to use a single variable switch(something). Can cases be boolean statements? Something like:

switch(winner){
 case ((playerMove=='rock')&&(computerMove=='scissors'):
    winner="player";
    break;
... etc
}


I like...


Ok I tried setting the property, it will work fine from the CLI, but won't work with eclipse. Eclipse does not support multiple classpaths in one project, e.g. see this post: https://gist.github.com/aslakknutsen/9648594#gistcomment-1452399 .

I fear we can't merge the test until SJC goes java8.


and now categoRy


not sure why this is still happening, but I'll merge it anyways. Thanks @borzamircea!


Great, please do that, and squash it into this commit. (Otherwise it won't build.)


Фунцкия должная возвращать строку, а не себя саму.
Например: ```return "Привет!";```


`point` is still non-null after doing this. The `point =` part in the original sets it to null, ensuring there are no refs left. Not a big deal for points, but if it was a `FlxSprite`, maybe.


can be senT


typo. gc_lifeime should be gc_lifetime


It includes the ltca, pca, ra and the root CA certificates.


updated


Oh, it sends it as a string? I just assumed it was sending a single byte. So if it wants to send 203, it literally sends the string `"203\r"` ?


in the following sections.


Anyway code isn't lost. Feel free to also review PR #1122 and #1123
Both PRs are quite invasive on the emitter.


Ahhh perfecto, es que aun no lo he visto!


Agreed if my above comment is correct


Ah, too bad.


please add empty line


empty line


Everything is passing so far, but check to make sure that the tests themselves are correct.


Grr. Both encoding `lookahead` and my other alternative fix sacrifice a not-insignificant amount of performance.


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1025.090d71a13a2dde4e902bf7a7e5556e4e22e84c0e](http://build.palaso.org/viewLog.html?buildId=68163&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:06:01


#nth break after finding one, since all you care about is finding one.
#? why `roll`?


Looking at this a bit further, it appears that if a country DB query is not successful, the returned countrycode (in the test suite) is 0, and the error message is nil, instead of "not found".


remove empty line


Sorry if I was unclear, but this CVE was not assigned to git-fastclone, only git itself. I'm not sure if they would reuse the same CVE for this or not, I think they usually don't.

The rest if the description here looks good.


add empty line


ok


Ideally after users come into picture, getScannedFilelist() should be able to get the scanned movies of the currently logged in user. So better add a todo if we have not added it already. We might forget this later.

Better suggestion would be to create a  getScannedFileList(String user) method and pass in the currently logged in user as one of the arguments.


I would perhaps make the translator comment more explicit and not rely on `compass-direction` being the translation key. E.g. `@"Specifies compass direction using clock position; e.g. at 3 o'clock means to the right (or east)."`


TL;DR: use `[NSString localizedSringWithFormat:@"%@", @(clock_direction)]` or `[@(clock_direction) descriptionWithLocale:[NSLocale currentLocale]]`

Explanation:

* use localized conversion to string, this is important for e.g. Arabic which does not use ASCII digits - see e.g. battery percentage and clock in the status bar [here](https://www.google.cz/url?sa=i&rct=j&q=&esrc=s&source=images&cd=&cad=rja&uact=8&ved=0ahUKEwjCp82N77jKAhVL1xoKHW5nAhoQjBwIBA&url=http%3A%2F%2Fwww.appsvillage.net%2Fuploads%2Fios-9-arabic.png&psig=AFQjCNF3JxDG8-K5x3Dxx6ASxZlaZlzeeQ&ust=1453395695130285); e.g. `@(clock_direction).description` outputs 12 whereas `[@(12) descriptionWithLocale:[NSLocale currentLocale]]` outputs "١٢"
* use `@(clock_direction)` with `%@` format specifier instead of `clock_direction` with `%ld` format specifier, i.e. let the compiler do the work of detecting the exact integer type used and thus avoid any 32 vs. 64 bit surprises

Note: this does not apply to `[NSString stringWithFormat...]` you are using below with `NSString` arguments – strings are not to be localized when used as arguments for `stringWithFormat`, it is only numbers, dates, etc. which are.


This newline seems like it has accidentally made it into this commit and should not be there (but maybe I am missing something and this is intentional).


This is interesting, I thought this is using the localization comment from WMFCompassView, but you are supplying it separately. If that is the case, then you can probably ignore my comment above about better translation comment (perhaps only take a few hints from it for this comment?)


7 cm is the size of the lampshade ?


Created ticket for this https://github.com/jquery/jquery/issues/2854, so we would stop discussing it in this commit


We could have done the same with LinkedIn but no, I wouldn't want that when actually attacking full SHA-1's. However, we could check 128 bits in cmp_one and conditionally test the last 32 bits in cmp_exact. Shouldn't affect performance.


No it isn't :(
Could be pls let me know how to get this out again?


Huh, looks like it's working now. Cool!

On Tue, Jan 19, 2016 at 8:39 PM, Ryan Ozawa <notifications@github.com>
wrote:

> Interesting. Opening the image URL directly works as does 'view image in
> new tab' right-click on the CfH homepage. Not sure why it's not working in
> a standard embed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CodeforHawaii/codeforhawaii.github.io/commit/7e4c7ba13306db1b28fc98ee130f54ebeab89419#commitcomment-15549185>
> .
>

--
Billy Meinke, MEd
@billymeinke <https://twitter.com/billymeinke> / Web
<http://billymeinke.github.io>


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


While at it, please also drop the addition of this stray newline.


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


Ah Perfecto! Con este cambio queda listo el  valor por defecto desde el flag port (Y)


```currentMethod.DeclaringCompilation.SourceAssembly``` can be replaced with ```currentMethod.ContainingAssembly```


What's this rtf business? Real text file please


i like these pretty nice events from you dude <3


hantas, you are a big nub


OK, IMHO this now looks much cleaner. And even works ;-)


Thanks, great catch!  Would you like to submit a PR?  (If you'd prefer not to, I can make this change).  IMO the throughput performance benefits outweigh adding a bit of latency in the flusher thread.  Applications can reduce latency by calling Connection.Flush() explicitly - which immediately flushes the buffer.  What type of latency/throughput settings would you like to see?


PlantCompBaseClassAgain (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed (1727 of 2056 tests passed, 0 test warnings)

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


I'd rather keep the `DataQuery` constants used in one class to avoid any sort of possible duplication (that is still found throughout the implementation).


@enykeev Sorry for not being super helpful, I had several issues with Vagrant networking & Docker,
so I only commented, rather then trying to build/fix the things by myself.

The last one I'm experiencing is: https://github.com/StackStorm/st2-packages/issues/4#issuecomment-173305970

Will continue to dig tomorrow by running everything in AWS instance (to avoid Vagrant).


There are a few linting issues introduced by this commit; just a friendly reminder to lint prior to merging.


Not relevant now since we're dropping this PR, but if I didn't declare `char * opteptr`, I got a linker error in gcc and clang.


No need to review this commit as it is redone in the next pr


That's just so that it can reference the libraries we build in this repo as projects rather than packages


That's default with conf: core


r=michaelwoerister


Respectfully submitted, 20 Jan 2016 ibid


Wil a Future<boolean> help in anyways here?  We wont be even able to identify what turned out to a filed movie. If not needed, lets remove the future.get() rt? Just call a eService.invokeAll();


sure, changed it.


Let's get some documentation here explaining what's happening.

Also, what if the user *has* tiempo / redis, but wants not to use it?


lgtm


Probably not going to be necessary if @Deamon5550 wrote a `JSON` [`DataFormat`](https://github.com/SpongePowered/SpongeAPI/blob/bleeding/src/main/java/org/spongepowered/api/data/persistence/DataFormat.java) as seen in bleeding's [`DataFormats`](https://github.com/SpongePowered/SpongeAPI/blob/bleeding/src/main/java/org/spongepowered/api/data/persistence/DataFormats.java#L32)


Lets use a piped exception handling here? catch (InterruptedException | ExecutionException e)


You mean BaseStandaloneConfiguration?
Isn't that already in its own file?


[R/password_from_file.R:14:35:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/password_from_file.R#L14) *style:* **Commas should always have a space after.**
```r
        comb_char <- sample(c("-","_","$","^","!","?"), 1)
                                  ^
```
[R/password_from_file.R:14:39:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/password_from_file.R#L14) *style:* **Commas should always have a space after.**
```r
        comb_char <- sample(c("-","_","$","^","!","?"), 1)
                                      ^
```
[R/password_from_file.R:14:43:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/password_from_file.R#L14) *style:* **Commas should always have a space after.**
```r
        comb_char <- sample(c("-","_","$","^","!","?"), 1)
                                          ^
```
[R/password_from_file.R:14:47:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/password_from_file.R#L14) *style:* **Commas should always have a space after.**
```r
        comb_char <- sample(c("-","_","$","^","!","?"), 1)
                                              ^
```
[R/password_from_file.R:14:51:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/password_from_file.R#L14) *style:* **Commas should always have a space after.**
```r
        comb_char <- sample(c("-","_","$","^","!","?"), 1)
                                                  ^
```
[R/password_from_file.R:21:15:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/password_from_file.R#L21) *style:* **Commas should always have a space after.**
```r
    paste(sel,sep=comb_char)
              ^
```
[R/password_from_file.R:21:18:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/password_from_file.R#L21) *style:* **Put spaces around all infix operators.**
```r
    paste(sel,sep=comb_char)
                ~^~
```
[R/timing.R:20:25:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L20) *style:* **Commas should always have a space after.**
```r
    assign(paste0("st_",name),proc.time(),parent.frame())
                        ^
```
[R/timing.R:20:31:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L20) *style:* **Commas should always have a space after.**
```r
    assign(paste0("st_",name),proc.time(),parent.frame())
                              ^
```
[R/timing.R:20:43:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L20) *style:* **Commas should always have a space after.**
```r
    assign(paste0("st_",name),proc.time(),parent.frame())
                                          ^
```
[R/timing.R:27:25:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L27) *style:* **Commas should always have a space after.**
```r
    assign(paste0("et_",name),proc.time(),parent.frame())
                        ^
```
[R/timing.R:27:31:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L27) *style:* **Commas should always have a space after.**
```r
    assign(paste0("et_",name),proc.time(),parent.frame())
                              ^
```
[R/timing.R:27:43:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L27) *style:* **Commas should always have a space after.**
```r
    assign(paste0("et_",name),proc.time(),parent.frame())
                                          ^
```
[R/timing.R:49:27:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L49) *style:* **Put spaces around all infix operators.**
```r
    tmp <- eval(parse(text=paste0("et_",name, " - ", "st_",name)))
                         ~^~
```
[R/timing.R:49:41:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L49) *style:* **Commas should always have a space after.**
```r
    tmp <- eval(parse(text=paste0("et_",name, " - ", "st_",name)))
                                        ^
```
[R/timing.R:49:60:](https://github.com/acjackman/acjackmanr/blob/363a6fe306510158b22a61e1ce38b5a3458b380d/R/timing.R#L49) *style:* **Commas should always have a space after.**
```r
    tmp <- eval(parse(text=paste0("et_",name, " - ", "st_",name)))
                                                           ^
```


Dont we need to set the scanstatus as failed in case of exceptions? Otherwise it will always be success rt?


Proposition: The inverse kinematic solver, which is integrated in MoveIt, provides an efficient ... by the robot controller, see Section...


It seems like a helpful warning, but I think the bug might be that it doesn't actually check that there is a const constructor before displaying the warning for a final field? There is no const constructor in my example, and I don't see the code to check for this. (Unless it's implicit somehow.)


Should of been `rickRollFound` -- my bad!


And it's been stuck in my head all day.


TeamCity Civic Assembly :: Build backend + frontend [Build 15](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=397&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


I think I saw a static moviedaoimpl object already declared at the top


and a surrounding object *or* surrounding objects


C++11 allows you to state that a function is being overridden (or is final and may not be overridden).


... as well as collisions with itself, ...


Sounds like a fantastic idea, although I believe if someone was determined they could pull it out of git commits anyway.


updateDupMoviesForUser() . We can add a TODO for the time sake,


@timharbour  the "video-description" div will add redundant markup to collections.  Can't we make it conditional, ie. wrapped in {{description}} handlebars?


target was an example. I wouldn't like to handle this by each build tool. What if we use something like arquillian folder in the root or something like that?


Oh yeah, I forgot to note that this also takes care of issue #309 in the Mac version.
In the Windows version I hope to fix this tomorrow (or maybe even tonight), and after that, I'm going to prepare a new demo build for GameJolt and for SourceForge.


they lack configurable...


I haven't used managed/.NET c++ in quite some time -- though by skimming your code again, I believe the problem is linked to your conditional.

Change,
``` c++
checkBox2->Checked
```

to

``` c++
this->checkBox2->Checked
```

To test the conditional, try message boxes.


Ah, should have looked higher in the code. I'm used to seeing things like `roleFound` in this type of code.


@atifsaddique211f what is pilro? And what is it doing here?


Ok, este ya lo habíamos revisado en el otro branch PAAS-563


I think two extremes - no buffering + disable Nagle's algorithm and then max throughput as now. I am not yet comfortable with the codebase to make a PR, just playing with it.


in the following ...


lgtm :shipit:


Oops, didn't set the title or description


its just a user and password, I will change it to stakater


@TorsteinHonsi  awesome, thank you :) I haven't tested this myself yet but it looks like it will work.


@DavidGoldwasser I was thinking more like this:

    found_thermostat = nil
    thermostat_objects = model.getThermostats
    thermostat_objects.each do |object|
      if object.name.to_s == thermostat
        found_thermostat = object
        break
      end
    end

    if found_thermostat.nil?
      puts "Can't find thermostat in model matching selected name. Won't set thermostat for selected thermal zones."
    end


Nice to know


in general: use ", see Section x" *or* "as detailed in Section x".


yep.


Yes it did!


@ChipSkylark37 this is the "original" repo, however all of the devs that were working on it for the last year have moved over to the new one you just linked. It's up to you which one you want to run, but one of them works perfectly fine, while the other one... well this thread exists for a reason :wink:.


nit: I think it's a bit more idiomatic to have a 'NULL' entry at the end of the array, and then just break out of the loop when you hit NULL


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.162.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68170&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:37:02


Do you want a capital `M` here?  The example on your blog didn't have one, but my instinct is that it should be capitalized.  OTOH, I don't think we should end with a period, so it's not a complete sentence.


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


Don't actually need this, since all you're doing in the method is calling `super`.


EconomizerIntegrationNew (lgu1234) - x86_64-MacOS-10.9-clang: Tests Failed (1669 of 1987 tests passed, 0 test warnings)

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


Could abstract this out a bit to just have one loop and assigning things to the proper `@newItems` or `@newHeroes` array depending on what type of Thang it is, like:

`for [newThangTypeArray, originals] in [[@newItems, itemOriginals], [@newHeroes, heroOriginals]]`


whoops! TODO: check to see if this has been fixed in master..


@atifsaddique211f we shld avoid using word "aurora" anywhere


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.570.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68164&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:19:05


Depends on the issue itself. I'll ask MITRE to look into this.


worth adding a bit more explanation here of why we have to try a bunch of different names (the issue is that platforms are inconsistent about symlinking, right?)


It doesn't matter from Moodle's point of view. Integers are fine.


strictly, it subscribes to messages (or the topic) on which the corresponding ROS node publishes... Just a detail.


haste hier eine newline dazu gemacht?


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


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2089 of 2089 tests passed, 0 test warnings)

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


with a high frequency *or* at a high rate


wunderfull :)


Extra `link =`


Double grr. Testing shows that the original code works just fine when `force_utf32` is true, even when the lookahead is not transcoded. Don't ask me why.


Closes pull request #1.


I will use the stakater java image instead of your private account image, then it will change


sent


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


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


Since the other commit is dysfunctional without this, I would just amend to that and then force push.


Thanks!


BTW `Parser::Source::Buffer` is not compatible with UTF-32LE. If you try `buffer.raw_source = utf32_encoded_string`, it fails with an incompatible encoding error.


Rebased and force pushed - you'll need to be careful when you update

On Wed, 20 Jan 2016 at 11:51 Lilly C. <notifications@github.com> wrote:

> I was getting errors with it when compiling. Now that I have a working
> toolchain, I'll add it back in
>
> On Wed, Jan 20, 2016, 10:33 Eric Wieser <notifications@github.com> wrote:
>
> > Why did you delete the color sensor? Is this just a bad merge/rebase?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/mitchgu/TAMProxy-Firmware/commit/613b013b9397958d34938e619bb175af178fe194#commitcomment-15560478
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mitchgu/TAMProxy-Firmware/commit/613b013b9397958d34938e619bb175af178fe194#commitcomment-15562930>
> .
>


What is the purpose of this function?


Isn't hide called automatically when using router.navigate? https://github.com/codecombat/codecombat/blob/master/app/core/Router.coffee#L170-L170


Whitespace is still all over the place here. Can you reformat this or something?


Weird stuff happening with the HP variables for P1 and P2... says it was determined somewhere else??


Agreed that it isn't intuitive, but orphan removal indeed has no symmetrically opposite operation, at the moment.


Nie może być, ponieważ to nie sprawdza czy obiekty mają takie same pola, tylko to, czy obiekt sprawdzany jest DOKLADNIE tym samym, który był utworzony wczesniej. Paweł w którymś z commitów napisał co trzeba zrobic zeby ten sql connector dzialal. Poszukaj w jego commitach.


Or maybe just leave it be at this point. We'll fix it when we have meaningful changes?


@mbadams5 I like your suggestion. I tried it in the code and it worked. So far, code modification is minimal. In addition, I don't need to worry about future maintenance, since it is a single function only. I think it may be a solution to remove optional arguments in other functions.

@Myoldmopar Are you OK with @mbadams5 's suggestion?


@davidpdrsn Did you ever get it working with Rails 5? I am struggling myself...


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


Maybe I am missing which button this is, but if it is the button on article page in the toolbar at the very bottom on the very right (along with 4 other buttons) which currently says "TOC, button", then it is a button (I am testing with the latest external TestFlight Beta, not with git master).


I'm really confused as to why this was necessary... Don't you have `rl-datetime-pickers in the html template?


That was the whole point of it. I at least like that I don't have to keep track of the numbering of questions and subquestions.


This means you are setting the `accessibilityTraits` to contain `UIAccessibilityTraitHeader` and nothing else (i.e. to remove traits the view had before).

You probably instead wanted to **add** the `UIAccessibilityTraitHeader` to `accessibilityTraits`:

```
accessibilityTraits |= UIAccessibilityTraitHeader
```


Commit message: correct `isAccessible` to `isAccessibilityElement`


Derp, this IS the datetimepicker.


getScannedFileList() method seems to be more like a utility method. It can be made public. Helps in code reuse. I think the better place for this method would be in MovieDAOImpl


Corrige l'erreur dans la phrase:

"La Sud-Morvandelle est une organisation unique mélant à la fois le sport et découverte du terroir et **dla** naturne du Sud-Morvan."


I didn't mean to include this file in this commit, but it was fixed in the "autogenerated" commit that was part of this PR. I think ./hack/verify would have caught this.

https://github.com/mikedanese/kubernetes/commit/fe8ae1a28a18d373331ad2b331b98645aeaa34fa


I did!

On Wed, Jan 20, 2016 at 6:45 AM, dothework <notifications@github.com> wrote:

> @yoojinkimhong <https://github.com/yoojinkimhong> DWMR. You worked thru
> the sandbox maze on Instagram?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yoojinkimhong/blocstagram/commit/cd79ff80226311059d8a75b508bd225deac2bb84#commitcomment-15559084>
> .
>


This should have been a separate commit. Would have been much easier to understand. Making solid commits is a skill that takes a lifetime.. Mine def still need tons of work. When you start reviewing code you start to see how important it is! :grinning:


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.659.db0a3f374a76d6f924780d8fd02c246e3c6aabea](http://build.palaso.org/viewLog.html?buildId=68165&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:06:12


who is Custom and why do we need to check him?:) maybe customCheck?


Wait, increasing the count resets date1 to now?


This note is no longer the case, it only depends on DC_ENABLE_HEART_BEATS.


What does this comment means? Lets discuss


Why was `ng-attr-*` necessary here?


Test


This call in inconveniently placed for another application launching aspen programmatically. It's not possible to invoke the behavior without importing `aspen.__main__` and it's impossible to `import aspen.__main__` without invoking the behavior. At the very least, it would be nice if this behavior could be extracted into a function in a public module and invoked explicitly in the `__name__ == '__main__'` block or implicitly in `serve()`.


Why did this typing change (becoming less restrictive)?


Done


I like the way you did everything inside the splice() method. Normally we could have created variables for each of the arguments, and instead you created a kind-of one line code! Great job.


Sure @orangemocha , will do.


Thats not faster at all.


I think you should be safe to just link shaderc_util. IIRC it will include all of it's transitive dependencies.

shaderc_util tests do not link against glslang etc, but maybe that's because they don't actually use anything from glslang.


Just wondering, why does this commit appear in my repo?


The commit message needs to mention exactly why this helps -- I can't remember, and it would be useful for posterity.


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


Welcome back @justvanrossum! :tada:


Yes. I ended up using custom paper sizes for everything. Done.


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


uv_unref on a non-refed handle should be a nop


Great work! Your code looks clean and efficient. I like the way you printed out each step so you can see what is going kind of step-by-step.


Remove this if not needed


I am so confused also. Did you get notified?


The magic strings here make me uncomfortable, but this is probably not terribly likely to change. Would prefer to see this wrapped into an item list like `partStatus`:

https://github.com/RenovoSolutions/RL21.Application.ServiceEvents/blob/master/source/data/serviceEvent/purchases/partStatus/partStatus.enum.ts


Switch to the now-canonical syntax `array(varchar(x))`


Bad comment (a leftover).


Same here


:+1: The best kind of edits!


Lets make line 3 match line 1


ConfigReader supports comma-separated lists which will resolve to a list for that entry, so you can get rid of all the custom `whatever.split('-')` code if you just use a comma instead of a dash in the config


@sirpkt I had considered that, and am not opposed to the idea, but had a few concerns with such an approach:

1. due to the namespaces object contract, a map of the object is required to have `namespace` to identify the unique namespace identifier of the object. This means that a post to a particular namespace ID endpoint is not really helpful and requires repeated information and validation.
2. If you allow a POST to `/druid/coordinator/v1/namespaces` to define the exact state of all namespaces, then that has an extra risk of state race between different POST calls. Currently there is still some risk in the coordinator due to the asynchronous nature of the audit-able config implementation, but that can be fixed. If the race condition is in the client then that cannot easily be fixed.
3. Having explicit POST / DELETE requirements makes auditing changes more discrete. It makes it less likely that something gets added or deleted on accident.

As such I went with the implementation presented, but am open to discussion.


TeamCity Civic Assembly :: Build backend + frontend [Build 15](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=397&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 70, passed: 1597, ignored: 7 Build time: 00:19:17
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

Media type tests: Should delete media type: <no details avaliable>

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


:trollface: :trollface: :trollface:


Good point.  Open an issue, please.


Maybe I should improve this by not putting the instance of XWikiSelectWidget directly in the DOM element but in a local map.


@atifsaddique211f
* avoid using aurora
* use better name for the docker container


Note that it makes this retro-compatible.


Great job working with validation. Big step!


Seems that I should have commented in https://github.com/skrub-wreckers/TAMProxy-Firmware/commit/613b013b9397958d34938e619bb175af178fe194


I was holding that for later reference, but it's exactly the same as the CUPS code. Done.


Yes.


Possible NPE : getResultForPst() seems to be returning null, but we are not checking whether rs is null. The general rule is that null should not be returned by a  method in case of errors. Always handle error cases with exceptions.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.660.090d71a13a2dde4e902bf7a7e5556e4e22e84c0e](http://build.palaso.org/viewLog.html?buildId=68166&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:06:29


* Location: San Diego, CA


Looks good.


For completeness, I would suggest adding the following line prior to 'swarm manage'
$GOPATH/bin/swarm create


@atifsaddique211f plz use similar naming convention; either use "_" or "-" in the name; better we shouldn't use stakater in the name of sample apps


You can use defaults for this (by passing the `default` parameter to the constructor or by having a DEFAULTS section - see [the docs](https://docs.python.org/2/library/configparser.html)), but it is probably better to handle these in a different way by not having options you don't use.


first


your github commit signature is not set up correctly. Github does not recognize your signature.

![image](https://cloud.githubusercontent.com/assets/6539412/12458917/f8e8f558-bf60-11e5-99b2-e1825edcfa66.png)


Does returning 0 in case of exceptions confuse the caller that he received a proper ID value?


firt


Yeah I've been getting emails all this time. But idk how does this commit even shows up here when it was made entirely in your team's fork.


Ah, turns out shaderc_util was missing the dependency on SPIRV.


Consider separating these equations: x = y; where x = ...


Okay, signed.

On Tue, Jan 19, 2016 at 11:17 AM, Gus Class <notifications@github.com>
wrote:

> Thanks for the pull request, before we can merge it, you will have to sign
> the contributor's license agreement here:
>
> https://cla.developers.google.com/clas
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lavahot/gplus-verifytoken-csharp/commit/9efae554d448ce30b853b3db89bc6ad56aa9bd40#commitcomment-15538829>
> .
>


typo ;-)


Hachja. Danke! :P


oh well, I only leaned on this to avoid gregs evil


:soon: :end:


Is there a reason for mixing "active" transformations (R) and "passive" transforms (C)?


Yeah, it's implicit.  At line 1070 below, we `return null;` in the case where no constant value computation was done for the final variable.  If there are no other errors, this is sufficient, since constant value computation walks the dependency graph starting at things explicitly marked `const`, and the only way a non-const final field can be depended upon is from a const constructor.

However, my mistake was in failing to realize that (a) if there are other errors, then a non-const final field *can* be depended upon in other ways that are bogus, and (b) incremental analysis's dependency tracking doesn't take into account these bogus dependencies, so it doesn't know to re-do constant evaluation when those other errors are fixed.  Both (a) and (b) are contributing to #25526.

Adding an explicit check for a const constructor would probably also fix the bug.  If you decide to go with that approach, make sure that changing a constructor's constness either causes incremental analysis to give up (in which case constant evaluation will be re-triggered for the entire file) or causes incremental analysis to re-trigger constant evaluation for fields in the class.


:eggplant: :two_men_holding_hands:


Please only specify how to obtain R (or C) from q once.


There's a few style inconsistencies. Please make sure that `make lint` passes.


@Zidane summary:

* You can have a mixin which targets an interface, as long as it is an interface itself
* You can add `default` methods to the interface
* You **do not need to use `@Shadow`**, just implement the target interface on your mixin interface
* You cannot use injectors of any type
* You can use soft implements and intrinsic proxies
* You cannot add `private` methods (this won't compile anyway)

A super important point to note:

* The keyword `extends` on an interface **is a lie**, `extends` in the context of an interface is the equivalent of `implements`. This means:
 * 'extending' the target in order to shadow it is perfectly fine
 * the concept of "super" in interfaces is different to classes, an interface's superclass is **always** `Object`, interface inheritance isn't really inheritance it's more just an order of precedence
 * **however**, if in the bizarre situation that some target interface **does** have a default method and you want to invoke it, you can use the *qualified `super`* syntax:

          TargetInterface.super.someDefaultMethod();


@atifsaddique211f plz use images from the stakater repo


ding dong the wicked witch is dead! whats going down sokar?


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


Yeah, I will do that.
What do you consider "official" terminology?
On [this wikipedia page](https://en.wikipedia.org/wiki/Primitive_data_type#Specific_primitive_data_types) it says the name for a 16bit binary number is a "word". I also read the term "word" meaning a 16bit data type very often.
When googling "x86 word data type" you get 450 000 results using google, when googling "word data type" you get about 89 million. (Just a comparison, that doesn't necessarily mean it's the official name)

The term "word" is far more often used for 16bit binary numbers than the term "x86 word"


Awesome! Instead of using the splice method to insert the ellipses you use inserted directly into the array to replace it. Great job.


nice


Excellent! Requirements met. Feel free to dive into some of the javascript challenges: https://github.com/nashville-software-school/front-end-milestones/tree/master/the-static-web/challenges.


informations -> information


I am on nightly dated 2016/01/19 on my OPPO N3
i cant see any option to add my fingerprint


Добре маскирано Руби!


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1237.80504d179debce13db99cc4613283cd9293cf994](http://build.palaso.org/viewLog.html?buildId=68189&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 842, ignored: 7 Build time: 00:06:14


I'm no python expert, so I don't fully understand scopes, but since you change the generated code so that it can be run as a standalone file, 1) you don't need to inherit any globals since the generated code file has all of the imports and whatnot that it needs to. 2) the file no longer ran when called this way since `__name__` != `__main__` here.


We'll need to put a note in our release notes as `copy_price` has traditionally been a target of hacks to customize pricing.

:+1:


PlantCompBaseClassAgain (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed (1721 of 2050 tests passed, 0 test warnings)

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


Override but I want to sync sammy source with caf base.


Scalars are usually not bold. Is there a reason for having it boldfaced?


how about next interpretation?
- add new field "String actionToCheck" to Scenario
- update this field in hasAvailableAction() method
- move withParameters() method to Scenario and abolish this new class ActionParameterValidator


It wasn't @deantares, it was me, in commit 7bdb0a3e3c6bcd7663dd260496c2d68562094b69  . I added all the references of this paper to "gpbot-fitness-entcom.bib", because "genebot.bib" was extremely large and had duplicates and incomplete references, so I added to the gpbot-fitness-entcom ONLY the references used in this paper (and directly extracted from DBLP).


Does this line check whether the movie is part of the latest list of movies from the last scan?
If yes, why do we need MovieAbsPath as a param? Do we not need the user name as one of the args?


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


This could be more concisely expressed with a `for/in` loop:

`for _ in 0..<10`


Leave a line note


This looks good to me. I'm going to merge this now :+1:


hi @desalesouche ,

Not, you only need the repo for boringssl.

Greetings


sent


I am a little lost at this step.  Lets discuss.


Yes, agreed it doesn't really matter, but the new code is easier to read
(not just faster), so it was a quick replace.

2016-01-20 18:23 GMT+01:00 Ondrej Skopek <notifications@github.com>:

> I only skimmed through the paper, but this really seems like an unneeded
> microoptimization... But I guess it doesn't hurt either :)
> On Jan 20, 2016 15:18, "Geoffrey De Smet" <notifications@github.com>
> wrote:
>
> > yes (those that don't rely on filling with null values at least)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/droolsjbpm/optaplanner/commit/132669d233d8f30aae378e795622fd8fc19d5f4c#commitcomment-15558416
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/droolsjbpm/optaplanner/commit/132669d233d8f30aae378e795622fd8fc19d5f4c#commitcomment-15563939>
> .
>

--
With kind regards / Met vriendelijke groeten,
Geoffrey De Smet


Ah I didn't know there was a conscious decision to use `object.assign`. Prior to Babel, I have only used sindresorhus' `object-assign` as a polyfill, now I let Babel take care for it during compile time. More generally, I always go for the most widely used solution, if all things being equal. (113k vs 9.7million downloads per month)


I think this paragraph should be moved to the implementation, shouldn't it?


@bors: r=michaelwoerister 1dc7eb8853f3ff5d8e4f7e4a717c55a7073f5533


I think we should do it the OpenSSL's way, i.e., only use the engine for the lowest level for key derivation.  It should be sufficient for most practical applications.


This could be marked `final` since none of this code was probably written with the expectation of being overriden.


This is true but right now this is a proof of concept so everything goes.


Nice. 14.5/15.


LGTM.


I was thinking the logic for running different binaries for different platforms etc would live in your tools/bazel (eg have a shell script that looks at uname, and then execs the appropriate binary). Seems more flexible than hard-coding a limited subset of the information from the environment into a path or something.


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


durations ... are


Great job Thomas. Nice working code