The best way to handle this problem is that when we are trying to mark the element as -1, we first check if it is 0, if it is, we do nothing.


Git n00b here, what do you want from me?


TODO: if object try to create duplicate set it name+#1


@BrotherJing student pack 靖


YES!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! :+1:


デフォルトのままじゃifとかelseとか多すぎて読む気にもなれなかったのですが、選択ソートを導入したら読めるようになりました。


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


Este campo debe ser una fecha. Perdón por no especificar.


Este campo es fecha :(


Estoy casi seguro de que esto está escrito mal.


That makes sense. In that case, big :+1: :smile:


> Fuzzy 该去掉的赶快去掉，别以后忘了到时候一看“写这么好开 Fuzzy 逗我呢”。

有一些我是故意的，只是做个标记的作用，最后审查的时候去掉即是。


How does it work


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2434


¿Probaste esto, Marco? ¿Funciona? No estoy muy seguro de que exista el tipo de atributo `float` en Ember. Verifica, por favor.


Miss follow up problem.


dumb :+1+ Jaja.


Yep, agreed. I'll add it to the roadmap for  it :)


Why using 2 times a `if(_instance == null)` ?


Hi @jef-n,

Yes, that definitely helps, also on GCC where the compile time for the file was reasonable (even on older Mac at ~ 30s).

However, with Clang and Release or RelWithDebInfo builds, the compile time, when texts have changed or a with a clean build, is still very long, as reported. Any qualms with adding the [patch noted here](https://gist.github.com/dakcarto/0cc443708487d205254a) to reduce optimization on that one file?


Due to the fact that seems an issue that we could face in other implementation in the future, maybe should be fixed at component-icon level, not here.
What do you think?


solve part of #57


Here's a new link to the article:  http://johnwilger.com/software%20development/teams/workflow/git/2011/01/08/production-release-workflow-with-git.html

I would no longer recommend the described workflow for a team with more than two long-lived, parallel tasks.  We found that having long-lived branches discouraged refactoring, because more edits ==> higher risk of merge conflicts.  Also, we occasionally found that two pairs would refactor the same shared code in different ways.

Also, I haven't been able to get the 'webby' ruby gem to work for several years, which means I haven't been able to update the site.  At some point I started a branch to port to Middleman, but never finished it.


It may not be worth it (yet), but that's typically the kind of logic that could be extracted to a more "generic" component (say, `TogglingButton`) and `FactionSwitch` would extend that custom component instead of `React.Component`, making the toggling button a reusable unit.


Why ```input_``` and not ```input```? Simple names are good.


This should probably be `/api/metrics/?format=json` because the response comes back in human-readable HTML right now, rather than a JSON representation of the metric POST data with `created` & etc added. We're not using the response right now, but it could be handy at some point


hay buen intento en uso de métodos
- el código debiera estar más ordenado, revisar tema de las vcaliaciones
- los nombres de ciertos métodos no son muy apropiedos dato1????
-


What I mean is that all elements have a `contentEditable` JavaScript property which we can check for. But... it is indeed a special case because its attribute form is `contenteditable`, not `content-editable`.


Congrats too :)


@prettyxw 去掉的话，应该是 visibility: none 吧，等下看看。


This says 'phony target', but that means [something specific to make](https://www.gnu.org/software/make/manual/html_node/Phony-Targets.html). Does this need to be included in `.PHONY`? If not, can you rephrase the comment?


这个 .idea 是什么？


Is there a suggestion on how to use this on templates with existing business names? It works when I change a current business name but does not show existing ones. I assume because they are not stored in the properties in the manner required/added.
Also, is there an on/off toggle for this planned where on shows business names and off shows the regular name? If not, I may want to add one.


之前隐藏掉 scrollbar 主要是 sidebar 里头的 toc 太长的情况下，会出来一个滚动条，非常难看，所以干脆隐藏掉了。


So I just checked this one out and looked at the files and you should switch all your indenting to tabs :)
Line endings are fine!
But in the diff above you change some lines with a 4 space indent to have a 5 space indent etc


There's no comments in this whole program. I'd encourage you to comment your code. You might say "No one else ever reads my code". You don't necessarily comment your code for anyone else. You comment your code so Stupid You 6 months later can understand what Smart You 6 months before actually coded up. I would start with one line per function, what does that function do?


IntelliJ IDEA 开发工具的项目配置文件。另外我觉得对于稍微大一点的网站，jekyll 每次重新生成网站得速度非常慢。


probably ```i``` and ```j``` instead of ```iteration``` and ```iteration_``` would be more idiomatic.


rechange your local changes to default before commiting


@YuryBY hadn't caught up on my emails and wanted the comment in the pull request so please disregard the dupe on the pr, this sounds good, thanks


I strongly believe this should be limited to some sort of beginner mode. It's hugely distracting.


Era "03-…" jajaj pero lo arreglé ya en nuevo commit


So for 'arm64' it is also necessary to include 'arm'


Pourrais tu me donner ton kik ? plz


@deads2k the default reactor breaks my tests here, and when I try to use Prepend it panics with an index out of range. Can you have a look at that?


x86_64 also needs 'x86' (and 'arm' and 'all')


now does not raise error with use not called


LGTM, this one is ready to merge I think.


@luca020400 ah youre right, but seems the sensor did not work for cm_kernel_base, is LPM is supported in cm kernel based?


@prettyxw 速度确实是比较糟糕，不过如果不是重新生成的话就还好，另外，liquid 这个比较慢，所以不用太多复杂的 liquid 还是可以接受的。.idea 这个单独提一个 PR 吧，那个 scrollbar 还在考虑怎么处理。


@ShonM We could, but we'll wait to see if that's a common problem. Have you run into issues with this?


**no-unused-vars**: codeResolver is defined but never used


**require-jsdoc/require-jsdoc**: Missing @jsdoc for _resolveValueFromPicklistOrCodeResolver


That was effortless! ;)


The specific version mentioned for `pygit2` was for its compatibility with `libgit2`. (travis-ci failed last time)


Actually, `pTeX` like engine is not yet supported in our package. So we should ignore them, maybe say
```TeX
\ExplSyntaxOn
\bool_if:nF
  {
       \sys_if_engine_pdftex_p:
    || \sys_if_engine_xetex_p:
    || \sys_if_engine_luatex_p:
  }
  {
    \msg_log:n { Not-supported-engine! }
    \tex_endinput:D
  }
\ExplSyntaxOff
```
before loading `ctex`.


Man page of XZ says that it needs up to 674MiB RAM for compression on -9e.


You haven't actually registered this simple module with the mapper?


Not yet, but I am certain that I will. Rails projects with PHP files in them, Node.js projects with PHP files in those too. Happy to wait and see what the community says :)


no it isn't


Oh, this post should go to the translate tool, but here it is valid as well.


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


Well, if you are going to remove transfers between backends, it is probably better to forbid mixing backends altogether.


Indeed, it's ~701 when converted from MiB to MB


Create a folder.
Copy the BestPracticesReview.ps1 and .bat to the directory
create a txt file with the instance names :
Instance1
Server\Instance2
click the .bat

* - The conenction to SQL Server is made by windows authentication and you need to be SA


Create a folder.
Copy the BestPracticesReview.ps1 and .bat to the directory
create a txt file with the instance names :
Instance1
Server\Instance2
click the .bat


Oh my. SV_HEAD() adds this `;` also.
Interesting.


OK.


typo: PRESENTN


Would it be reasonable to add a warning, until this is fixed?


This was misguided, I didn't read the doc carefully.  Please change to `msg.data_start`.  This can only make a difference if the client is lagging behind the server significantly, but still it's incorrect now.


Кров ворожу... отойді я
І лани і гори —
В сем’ї вольній, новій,


:+1: Excellent API


Fixed


The Main class is temporary. It won't exist in future builds, and it is therefore not necessary to have it create an instance of itself.


Aha, I was just confused by KB instead of MB :)


OK, I guess it was just to reduce the time needed to run the tests.


Ce fichier est-il utilisé ? il semble faire doublon avec le fichier de même nom dans le repertoire au dessus...


Exactly!


we should specify a version here :) otherwise it will always pull the latest (even if the latest is a new major breaking version) :)


Right, you've got it now.


you should leave out the -g -O2 and -DNDEBUG flags.  cmake adds different flags for these depending on the CMAKE_BUILD_TYPE

for -D flags, put them in add_definitions() instead
for other flags that belong in both C_FLAGS and CXX_FLAGS, you can use add_compile_options()

OS_LINUX and ROCKSDB_PLATFORM_POSIX could be detected, but setting them here is probably ok for now

HAVE_JEMALLOC should be detected with a find_package

ROCKSDB_ATOMIC_PRESENT and ROCKSDB_FALLOCATE_PRESENT are both detected below, take them out of this list


Our users will be beginners. This is based on feedback from teachers.

Sent from my Windows Phone
________________________________
From: Michał Moskal<mailto:notifications@github.com>
Sent: 10/1/2015 5:00 PM
To: Microsoft/TouchDevelop<mailto:TouchDevelop@noreply.github.com>
Cc: Peli de Halleux<mailto:jhalleux@microsoft.com>
Subject: Re: [TouchDevelop] automatically select first line of code when entering an action (bd6e43e)

I strongly believe this should be limited to some sort of beginner mode. It's hugely distracting.

—
Reply to this email directly or view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fMicrosoft%2fTouchDevelop%2fcommit%2fbd6e43e6abcbe8ba505d0f65aab651ededb908d8%23commitcomment-13544990&data=01%7c01%7cjhalleux%40microsoft.com%7ccf218b9d711b4070c6ea08d2ca79700f%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=NnvplE%2bdAA%2bUXnb5Qkd9TsyQdDHqoiQlC8rEy97jiHo%3d>.


We should use/recommend flake8 and make #noqa the standard.


I think we could do the matching for --shallow as part of one of the 'case' statements, and then do a for loop over all arguments, just like we do in https://github.com/opengapps/opengapps/blob/master/report_sources.sh line 49.
What do you think of that idea?


The commit message is missing:
1. A single line, followed by a blank line, so that GitHub picks it up as a subject
2. The AERGO issue key followed by `#comment` to pick up the comments in JIRA


generated files should go in the build directory, not in the source tree


This can trigger a CPU copy before the transfer.
Wouldn't it be better that `pygpu_fromhostdata` reads the data in its original layout, but then writes it C-contiguous in GPU memory? Is it feasible?


This is good when the literal is inserted manually, but when there is a default value of a function it's super annoying. Again, this should be beginner-only.


Sketchy walk-around, at least it compiles.
Probably version with component (_browserPanel_) and Callback should be used.


Extra s on file name


Congrats !

There was no need to compute Bishop contact check, and now, no need for
Rook contact check !

And for the record, from April 17, removing both QCC and RCC did not worked.
http://tests.stockfishchess.org/tests/view/5530afcd0ebc596367ee9a08
But this thing is ...ALIVE, so this conclusion might have changed too.

Meanwhile, another patch is increasing scores for distant safe checks.
Can we safely merge those 2 patches together ?


lgtm


:angry:


The tricky bit was in the visitor_actions_test_visitor_actions_info() hook, because at that time we didn't know the nid yet.

Updated the approach to force the newly created node's nid at b69197a35d5a6c52c8ef1cb38823be1d7341fa84.


asaa


if you put these three strings in else block it would be easier to read.
if(true){
return ...// end of story
}
code that is after closing brackets will only run in else case.
In my opinion it will increase readablitity as you don't need to concentrate your attention in the code that is enclosed in if statement.

if you would not have "return" statement inside if this would be perfectly fine.

@dlyash What do you think about it?


This tutorial was transferred from a tutorial I created for a group project


This might be the reason why during the solo upgrade the device partition gets all the data on the failed encryption and subsequent reset wipe.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 82](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=259&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Compilation error: Xbim.Geometry.Engine\Xbim.Geometry.Engine.vcxproj (new) Build time: 00:15:02


Cool will test it.


Fixed.
Thanks for idea, I considered something similar but hesitated :-)


Удачи, братан!


currentlayer and printing resets when a print starts.
Second print will work.


Без комментариев.


TeamCity pull requests :: netty [Build 352](https://garage.netty.io/teamcity/viewLog.html?buildId=1934&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5616, ignored: 21 Build time: 00:37:11


Can you explain what is the logic behind this date-based comparison?


I can disable I when the string is non-empty.

Sent from my Windows Phone
________________________________
From: Michał Moskal<mailto:notifications@github.com>
Sent: 10/1/2015 5:17 PM
To: Microsoft/TouchDevelop<mailto:TouchDevelop@noreply.github.com>
Cc: Peli de Halleux<mailto:jhalleux@microsoft.com>
Subject: Re: [TouchDevelop] immediately open string literal editor when inserting (2285d7d)

This is good when the literal is inserted manually, but when there is a default value of a function it's super annoying. Again, this should be beginner-only.

—
Reply to this email directly or view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fMicrosoft%2fTouchDevelop%2fcommit%2f2285d7d8ec64c39c47a57823c776ac5b8a508e67%23commitcomment-13545431&data=01%7c01%7cjhalleux%40microsoft.com%7c215c8eae44fa43f1eee508d2ca7bcc8e%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=YCzgZXFZSJz3Ee5x4Sj9VoSAbwxIICihVDwaZBvnEA8%3d>.


@tommypeps

Thanks for contributing!

From my understanding, iOS9 unlike iOS8 and below doesn't update the content inset of UIScrollView when the scrollview was added programatically instead of using storyboards.

Based on that this code does not work for all scenarios and should be added manually case by case.

I have not found a good solution for the case we added the table view programatically,
do you?

Regards


We're actually interested in documents that don't have a blob holder here, right? I'll change this in my cleanup code.


As I said in the PR, I install a hook on LdrpCallInitRoutine() or the relevant nt loader code to cover all Windows XP to Windows 10, x86 and x64 platforms in order to Refresh Modules before the dll entry point. The current implementation Refreshes the Modules after the dll entry point which in effect does not detect any allocations made on dll entry point.

In this respect VLD would not catch any allocations made in DllMain()


ah.. no wonder it doesnt work


To make normal qtcreator work with appstudio

Copy what is under:
``` C:\Users\XXX\Applications\ArcGIS\AppStudio\QtCreator\bin\qml\ArcGIS ```
over to your Qt installation's \qml\ArcGIS folder, ie:
``` C:\Qt\5.5\msvc2013_64\qml\ArcGIS ```

then:
```
import ArcGIS.AppFramework 1.0
import ArcGIS.AppFramework.Controls 1.0
import ArcGIS.AppFramework.Runtime 1.0

//import ArcGIS.Runtime 10.26
//import ArcGIS.Runtime.Toolkit.Controls 1.0
```

Make sure you change it to Release instead of Debug

:octocat:


sudo should be done for *coreos_pkg_home_bin* owner


Apparently I'm blind O_O Sorry.


I don't understand why pylint thought the patch method didn't exist.  It clearly does, as the test passes.


What is the semantics of `cls` and `type_`? Is it described somewhere?


Thanks Ben!!!


Also, should we pass `cls` to `self.__set__()` instead of `None` a couple of lines below?


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


@dg You're wasting my time, I'm wasting your time :disappointed:

1) recursion leads to more practical call stack in real-word use-cases.
2) recursion requires less code
3) recursion does not use goto => the code with recursion is more readable (using `while` will require even more code)
4) recursion was good enough => no reason to change it
5) recursion is closer to where I would like to be in 3.0 (no special case for ForwardResponse)


Juste par curiosité personnel comme je ne m'y connais pas, à quoi correspond ce "hostel77" ?


Mm, right. No problem, I just got confused.


Ah ok.  I have very little Cython experience, so I wasn't sure if it was copying the pointer or if it was going past that.  None of my testing showed this operating incorrectly.

Valgrind reported memory leaks here, and from what I can tell, it makes sense.  Malloc is used on the libemu side (in the C, not in the Cython), but there is no free call.

I'll drop this PR.


Will you fix the KB/MB typo then for the PR?


Too, I love :pizza: :1234: , and :+1:.


Probably a good idea, but strings are often empty by default. If you're not beginner there is good chance you will want to use a local variable and not a string literal.


Sigo practicando


Спасибо)


This line is not necessary, since you already cover arm64 in the block below


These logs may be useful for debugging purposes, perhaps use `java.util.logging` rather than remove them to preserve it.  `FINE` level for this should be fine.


Use `OptionalLong`


Interesting bug. Fix LGTM.


There should be a newline before `@return` in Javadoc


Hmm, this is ugly, but it seems there is no native way, like `ReflectionClass::isAnonymous()`. :(


You should remove the *) block, and instead put the decleration of modules before the for-loop.
Because if one doesn't give any arguments, the 'argument' function gets never called.


Yes, I think so.


Does that mean that all cached C code has to be regenerated?


Change this to put in the message in the `ValueCompareException` make `ValueCompareException` take in the left and right objects.  Note store only the classes not the object as the object `left` and `right` are not guaranteed to be `Serializable`.


Turns out our documentation is incorrect, and `trimcolor` is the proper name for this parameter.


Should that be fixed in the old back-end as well?


I compiled using the Visual Studio community 2015 and the way the new compiler work , it generate quie a few false positives with some Antivirus software. Compiling the same code the old way with VS 2013 fixed the issue.


Actually there is ReflectionClass::isAnonymous(), but it is undocumented (yet): https://github.com/php/php-src/blob/8557e8323a33288dd75eaafc4fb26a295351698a/ext/reflection/php_reflection.c#L3950


consider using configure_file() instead of a shell script

the script relies on git, so you probably want to find_package(Git) to make sure it's installed, then use ${GIT_EXECUTABLE} instead of just 'git'


This seems odd to write a null here. Why not just return an empty result set?


> In my opinion, we could have a 'one step after' scope variable, in this case inside window scope, to store the multiple expensiveResources we may have, AND, to check the duplicity of IDs when calling the object constructor

> ... this could be a functional use for the _all_ids collection, but in a very different implementation that you made on the original code

Keep in mind that the original code was seriously flawed (albeit by design, as part of the exercise). Care to take a stab at a working implementation that will pass my test above?


Can you write a docstring for that decorator?


AFAIR strings like "t" or "tty" do happen /sys/class/tty/console/active nevertheless using 4 instead len would avoid possible future console devices named "t" or "tty" (last one is reserved and can never be a console device)


Name this `rowCount`


Why not change 'Item %s variation #%s ' to 'Item %s variation %s ' just like you did for simple products?

'Item #%s' => 'Item %s'


Add parens


In Radium, you cannot mix longhand and shorthand properties in the same style object.


It gets better: #2880


Fails with this pull? https://github.com/kubernetes/kubernetes/pull/14664


Not sure if just a line note will be seen, hence this duplicate comment.


Nice!


"ordinalPosition is invalid"


Please not that this one is old, but it is still the most recent leanback version currently available.


I thought I picked it downstream (or something similar) already.


Actually, we don't need ordinal position anymore


Should be using escaped translation functions eg `esc_html_e`


> and it will become final result returned by stamp.

This may be inaccurate if a subsequent initializer also returns a different instance.

> Stamp specs will not support async as of now.

That's silly. You can still do async. You just need to handle promises appropriately in your stamp initializer. This is userland support vs spec support.


Dont create a new suffix field. Just remove the final of the default suffix and instead of setCustomSuffix simply call it setSuffix - also remove the bolean flag then which is no longer needed. Ill merge it than - thank you


Is there anything I can do to help release this fix to npm?


Opps, fixed


AFAICR there are not more devices at least asI had written the kernel API for /proc/consoles but may changed in meanwhile.


Sure, fixed.


Shrink wraps all the objects in the stage - can regulate what to show via removeChild and addChild via the switch statements checking for the state number equal to the public constants from the Config.ts class - to cycle through each state appropriately from the choices made (e.g The Dragon's decision tree)


I'm not ecstatic about what you have done here.  See issue #12.


I think the issue here is the `leaveDotGit`, as mentioned in https://github.com/NixOS/nixpkgs/issues/8567.


You know that this ramdisk is unsed right ?


indent


If keypress == ERR should not return output. It should simply return to the top of the key press loop. See issue #12


Isn't this reordering an issue since the `gem.executables` line refers to `gem.files`?


Really? I didn't noticed this abnormal drains :/


Error


this is the same search that phyx uses


Je le mettrais directement sous le titre, que ceux qui connaissent la NPI n'aient pas à lire le paragraphe précédent. Mais du coup, plutôt que de déplacer ce bloc, tu pourrais en ajouter un, moins explicite :

L'objectif est de créer une calculatrice en notation polonaise inversée. Vous devrez gérer les opérateurs ... avec pour opérandes des nombres réels.


jjj


Jenkins requests fail if you send a parameter that doesn't exist; so you have to update the Jenkins slaves first.

Its probably very much worth converting everything to a single parameter and sending a json blob to get around having to update the jenkins slaves...


My design here mimics the way tempting engines sanitize strings before appending them to an HTML document. There don't seem to be any guidelines for sanitizing generated eval code but this a conservative and battle-tested concept. Checking local variables (like `i` and `j`) is probably overkill.


Je chipote, mais dans le bloc information précédent, `2` et `4` sont des entiers, pas des réels. Permet-on à l'utilisateur d'entrer `2` au lieu de `2.0` (en Python, un `float(chaine)` règle la question, mais peut-être n'est-ce pas le cas dans d'autres langages) ?


> This is userland support vs spec support.

Yeah. That's why I said "Stamp specs", but not just "Stamps". ;)


`CJK` package has installed some CJK fonts, we can use it instead of `zhmetrics`:
```TeX
\ExplSyntaxOn
\sys_if_engine_pdftex:TF
  {
    \tl_const:Nn \CJKrmdefault { gbsn }
    \cs_new_eq:NN \CJKsfdefault \CJKrmdefault
    \cs_new_eq:NN \CJKttdefault \CJKrmdefault
    \usepackage[UTF8, zhmap = false, fontset = none]{ctex}
  }
  { \usepackage[fontset = fandol]{ctex} }
\ExplSyntaxOff
```


Although it's not a big deal, I think ```if (Math.abs(leng1 - leng2) > 1) {``` looks better.


find_package(Thread REQUIRED), and replace pthread with ${CMAKE_THREAD_LIBS_INIT}


*division


"only supports"


No, that is not possible.


"It does not support generic row-by-row deletion."


N'y a-t-il pas un nom pour désigner cette syntaxe ?


This is why we have the "pro" mode - not to annoy ever so slightly more advanced users.


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


This seems like a weird comment to put in the Hive connector. It should be elsewhere, like the in SPI documentation


Hmm... I'm not clear what this is doing. Isn't it just changing the one node that happens to be the child after the newly-reordered node? What about others?

Consider starting with this:

    childA       (id=1)
    childB       (id=2)
    childC       (id=3)
    childB[2]    (id=4)
    childB[3]    (id=5)
    childD       (id=6)
    childE       (id=7)
    childB[4]    (id=8)

and reordering `childB[4]` to before `childC`. The result should be:

    childA       (id=1)
    childB       (id=2)
    childB[2]    (id=8)
    childC       (id=3)
    childB[3]    (id=4)
    childB[4]    (id=5)
    childD       (id=6)
    childE       (id=7)

Notice how the `reorderBefore` node is ID 3, but the SNS (e.g., path) of nodes with ID 4 and 5 were changed. How does the above code help this?


alteração caminho


This is duplicated, remove it


Shouldn't we check for any of PEEK|CLAIM|DISCARD here?


Isn't this reordering an issue since the `gem.executables` line refers to `gem.files`?


I probably should have made it `appRoot + '/mainApp'`, right? At the time I made this commit I had little experience with node, and was in the habit of creating paths in Python using `os.path.join`. But apparently [slashes are fairly platform-agnostic in node](http://stackoverflow.com/a/13650454/427339), so `path.join` isn't necessary. Is that what you were referring to?

http://stackoverflow.com/a/13650454/427339


Do you prevent the use of FI_DISCARD alone?

"FI_DISCARD : This flag must be used in conjunction with either FI_PEEK or FI_CLAIM."


this include and the one for FindBZip2 shouldn't be needed, since they come from /usr/share/cmake


```java
// it is too expensive to determine the exact number of deleted rows


Don't we want to use JSON params here instead of `URI.encode_www_form`?


Put the comment above the line. We avoid end-of-line comments.


If the request has keys set to be arrays, (`ip[]=8.8.8.8` in the url) this will output `[object Object]` in the Request body. The solution is to `JSON.stringify` the data as is done on line 49. The change is simple: `JSON.stringify(request.transformed.data, undefined, 1) +` ...but I don't know if that would lead to breaking other output.


When will the tactic be proved sound? We need to get a stable release which means eliminating anything that isn't complete.


I created a partial called header_small which is referenced in the single post/portfolio/page views. Right now it's just a duplicate of the big header but can be styled separately to create a small header.  Also, though I created partials for header and nav, I later discovered that the nav is embedded in the header so I call that partial from the header partial (no need to put in each view).


your commit doesn't add any files to cmake/modules/. did you forget to commit them, or is this module path just not used?


This make any php inside of served js's stop working, any ideas why? At least with Tinymce, it reproduces urls with explicit php on it.


Thanks for the review! Fixed this in #22. Once that gets merged, I will rebase this on top of it.


Post_type view still needs some consideration and thinking...


^^^ @artgon this is what you wanted (no need for `_ <- monadic-val`)


In this case the parent process does nothing but waiting on the child, all other had already happen in the successing child process


Does `NoSuchObjectException` always mean the table doesn't exist? What is thrown if the partition does not exist?


See also comments at https://github.com/NixOS/nixpkgs/commit/9cea5bcf2cb180638e2e5e47692e57387fcbb051


Miss follow up question.


Wrap the "Currently" to next line


I was wondering if it would make sense to extend support for all these Jvm** attributes across all platforms.

Given that these are some of the more common options most people would use, we could use them - not really instead - but in precedence to the standard java_options, so that we could achieve consistency across all platforms?


This should have a `fail("expected exception")` after it


Почему english? Это для первого варианта?
Я так понимаю, что у нас почти не будет англоязычных постов. Да и мы вовсе на них не расчитываем


Catch the proper exception type


It doesn't work yet. Scope is that it should tell you what floor you're on if you tell it what floor you're on.


@arq5x @brentp Does bcolz need to be locked on 0.10.0 or are later versions allowed? I was doing some speculative work on an all conda gemini build and running into build problems because conda installs the latest 0.11.0. Do I need to work around it, or is this just a typo?


The intent was to allow both. Tho I'm tempted to just use Selector for both. One of the potential use cases was bootstrapping components from external to addon deployment.


Name the exception parameter `e`


none of your calls to find_package() use the REQUIRED flag. some of the dependencies do look optional, but it's hard to tell


I guess these were added for ease of testing?


```
if (v % 2 != 0) oddNum++;
```


Quite reasonable, I think.  I was thinking that people might borrow `kind()` from a kind they required but I think that would be lazy and bad practice, as you pointed out.

```
var kind = Button.kind;

kind({
    kind: ProgressBar
});
```


the stats showed the phone awake but the panel is off


"git push
qqa" wtf


... don't if it is allowed to call exit() if "prctl(PR_SET_PDEATHSIG, sig)" fails


I agree uint8_t is better, to avoid problems with literals >= 0x80


[Pop Singers Works.txt](https://github.com/jeannemcham/Master-Branch/files/6162/Pop.Singers.Works.txt)

This is my text file.


make pull req.


Oh. =)


Yes, in the python documentation: https://docs.python.org/2/howto/descriptor.html#descriptor-protocol


I now see, we miss 'basename' as one of the identifiers of coreutils.
It might also be smart to add 'coreutils' also as an identifier.


What if s1 and s2 are the same?


THANK YOU! I just ran into this issue today. I'll sync now...


You should comment this line (require "pry"; binding.pry) , no?


ome codes in the level 1 what i remember

v = Zombie.find(3)
v.destroy

v = Zombie.find(3)
v.name = "jose heriberto"
v.save

Zombie.limit(10)

Zombie.all

Zombie.create(:name => "jose heriberto", :vayard => "algun menesaje")

i go in the level 2
http://railsforzombies.org/levels/2

insert the validate in the model example
class Tweet < ActiveRecord::Base

  validates_presence_of :status
 validates :status, :presence => true

end

For enlazar tables use the same logic like cakephp belongs_to : zombie
and in the table "model_id"

validate bod in model presence and unique the name
validates :name, :uniqueness => true, :presence => true

create a relashionship where zombies should have one weapon
class Weapon < ActiveRecord::Base
   belongs_to :zombie
end
class Zombie < ActiveRecord::Base
   has_one :weapon
end

using realtions chips good find all the weapon's ash
Weapon.find(:all, :conditions => { :zombie_id => 1 })

quantity of weapon's ash
Weapon.find(:count, :conditions => { :zombie_id => 1 })

Using views

posted by <%= tweet.zombie.name %>

link_to tweet.zombie.name, zombie_path(zombie.name)

link_to @tweet.zombie.name, @tweet.zombie, :confirm => "are you sure"

Tweet.all.each do |tweet|
 tweet.status
end

or equals to
tweets = Tweet.all
tweets.each do |tweet|
  tweet.name
end

if tweets.empty?
 puts no found tweets
end

if zombie.tweets.count < =  2 then
end

##Edit and delete links
link_to tweet.zombie.name, tweet.zombie
link_to "edit", edit_tweet_path(tweet)

link_to "link text",

link_to zombie.name, zombie , :style =>"color:#fff;"

exercises
truncate to display only 10 caracters
  truncate(zombie.graveyard, :length => 10)

#Imprime SMART ZOMBIE si tiene mas de dos tweets
<% zombies = Zombie.all %>

  <% zombies.each do |zombie| %>

      <%= zombie.name %>
      <% if zombie.tweets.count >= 2 then %>
      SMART ZOMBIE
      <% end%>

  <% end %>

###link to edit zombie
<% zombies = Zombie.all %>

  <% zombies.each do |zombie| %>

      <%= link_to zombie.name, { :controller => 'zombies', :action =>
'edit', :id => zombie.id } %>

  <% end %>

###Using controllers
into def we can call a diferent view

example
def show
 @tweet =  Tweet.find(params[:id])
 render :action => 'status'
end

status.html.erb
<%= @tweet.name %>

use xml or json or html

inside de controller

def show
 @tweet = Tweet.fin(params[:id])
 respond_to do |format|
  format.html
  format.xml {render :xml =>@tweet}
  format.json {render :json =>@tweet}
 end
end

add some autorizacion for the person has tweet
class tweet
 before_filter :get_tweet, only => [:edit, update, :destroy]
 before_filter :check_auth, onlye =>
 [:edit, update, :destroy]
def check_auth
 if session[:zombie_id]!= @tweet.zombie_id
  flash[:notice] = "sorry, you can't edit this tweet"
  redirect_to tweets_path
 end
end

def get_tweet
 @tweet =  Tweet.find(params[:id])
end

def destroy
end

def edit
end

def update
end

end

##In the layout a notice for show message to user
<% if flash[:notice] %>

<%= flash[:notice] %>

<% end %>
--------------------------------------------------------------------
##Add a before filter which calls a method to check if a zombie has no
##tweets. Redirect to zombies_path if they dont have tweets. Only on show.

class ZombiesController < ApplicationController
  before_filter :find_zombie

  def show
    render :action => :show
  end

  def find_zombie
    @zombie = Zombie.find params[:id]
    @tweets =  Tweet.find(:all, :conditions => { :zombie_id =>
@zombie.id })
    if(@tweets.empty?)
      redirect_to zombies_path
    end
  end
end

------------------------------------------------------
##Routing in rails

llamdo resfull resources
we need define routes in config/routes.rb

for example localhost/tweets/new

in routes.rb
resources:tweets
match 'new_tweet' => "Tweets#new"
el primer de match es el path rapido y el segundo es a donde realmente
esta apuntado que es el controller#action

now if you run localhost/new_tweet will show the form to create a nuew

if we wan redirect some action for example the before action we want redirect to list of tweets
we need to write in routes.rb
match 'new_tweet' => redirect ('/tweets')

if we want show an action from localhost/ we need
root :to => "Tweets#index"
------------------------------------------
#routes params
if params[:zipcode]
  @tweets = Tweet.where(:zipcode => params[:zipcode])
else
  @tweets = Tweet.all
end

In routes for send params we will call

match 'local_tweets/:zipcode' => 'Tweets#index'
and to we can specify other name

match 'loca_tweets/:zipcode'=> 'Tweets#index', :as =>'local_tweets'

y the link to call that action will
 <%= link_to "Tweets in 32828",
local_tweets_path(32828) %>

That seem to call link is the match +underscore+ path
------------------------------------------------
find all tweet from one zombie

if params[:name]
  @zombie = zombie.where(:name=>params[:name]).first
  @tweets = @zombie.tweets
end

--------------------------------------------

Create a named route. It should generate a path like
'/zombies/:name' where :name is a parameter, and points to the index
action in ZombiesController. Name the route 'graveyard'

RailsForZombies::Application.routes.draw do
  match 'zombies/:name' => 'Zombies#index' , :as => "graveyard"
  #match ':name' => 'Zombie#index', :as => 'graveyard'
  #match 'zombies/:name',:to=> 'Zombies#index', :as => 'graveyard'
end

---------------------------------------------

finished the course thanks


We miss a check for "java"


PodStatus is an aggregate of multiple containers. These targets are singular. The combination of liveness and readiness just looks like: Alive,NotReady


There's a misunderstanding.
You should compare ```original.length()``` and ```result.length()```.


And for "md5sum"


And "unzip"


And "zip"


Do not use this, please!


This will also require updates to a few unit tests that may not have expected a message


Sou eu Auris! Add ae


As of this commit in bcolz: https://github.com/Blosc/bcolz/commit/530a39d793126147fa23ddbe96851e27b60f250d
the problem should be fixed once the next version of bcolz is released. bcolz 0.11 required mock but didn't add it as a dep so it could be that adding mock as a dependency will fix the issue.


Oh shit, I thought we missed some commands, but they are ofc handled below :-P
sorry


`forceMove()`


No, you probably got it wrong.
Those are only special cases.
Dependencies that are usual command verified as is.


@aertmann this is a syntax error, an additional ")" was mistakenly added here. grunt build doesn't run anymore. :)


Good catch!


Tengo una pregunta, cuando terminemos los 5 niveles se da algun tipo de certificacion o solo la aprobacion de curso?


I think you want != null in the conditional. If filters is null, then you want to assign it a new HashMap.


Yep. The hope is to soon be supplying most of this data just via jobstep_details to Changes client; this is only needed because some builds are still not on changes-client. There's also a diff waiting for review that provides a JSON dict through Jenkins and optionally exposes them all as env variables automatically, and we'll go that way if the changes-client stuff isn't happening in the shorter term.


Yes that would be great. Have a look in rock-core/base-cmake it should be possible by extending the rock test macros in Rock.cmake
Also it would be necessary to add lcov as osdep in the rock package set.


So only the 'basename' has to be added (and maybe 'coreutils' too) and then I will merge it.


Yes


We simply cannot have `synchronized` here. We had that previously in `OutOfOrderSequence` and performance benchmarks, micro and macro showed massive regressions. Replace with `volatile` access at most.


Yep it should; was fixed in b6c1f1d1629371910cadb3f4be15ec0eb6842b43


Same point about `synchronized` here as above.


> >    and it will become final result returned by stamp.

> This may be inaccurate if a subsequent initializer also returns a different instance.

"The object instance being produced by the stamp. If the initializer returns a new object, it replaces the instance for subsequent initializer calls and will become final result returned by stamp unless any following initializer replaces it again."


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 83](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=260&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Compilation error: Xbim.Geometry.Engine\Xbim.Geometry.Engine.vcxproj Build time: 00:14:27


> The old recruiting practice says, "Never hire a developer who computes the factorial using Recursion" ;)

and haskell programmers would be homeless :P


Also here a ")" too much was added!


It's possible you might not have to hack it in manually soon, if this gets merged: https://github.com/smealum/ctrulib/pull/176


@tim-sueberkrueb Ah ok, I add the p tag because without it looks bad on mobiles X)
Need to figure this out


Fichier materiel.cs modifié et à finir (accesseurs), écriture du load à finir également (voir pour les collections pour l'insertion dans les ComboBox), quelques modifs graphiques et ajout d'une confirmation de suppression


This is i the wrong section (project instead of workflow)


hahahha great point @RamonHossein !! :bow:


can't we use unordered header items instead?


Ok, understood ;)


Не, ну вы там уже совсем обленились. Скоро будете писать commit message: "qwertydduyftdduiofvuyghcjdbsvgdfgdshj".


Or here: https://docs.python.org/2/reference/datamodel.html#object.__get__


I'm also seeing it rebuild this apparently unconditionally. Very annoying.


@remram44 we probably need to explain how to update the path to the unpacked experiment, if the user needs to do so.


We shouldn't ever use a $_GET value like this, due to security vulnerability. Can you fix up so that it uses the standard approach for using form values in postProcess functions?


thanks really


Ah good call. I think I lazily copied it from somewhere else.


Technically, yes.  But since nothing uses the context mechanism outside of tests for now, there would be no difference.


@remram44 is there an installer just for the reprounzip package in VisTrails?


This doesn't work, because the -f should be on tag, not pull https://github.com/OpenDataServices/resource-projects-etl/commit/43a411284891017a7ffd7ef64df41242c5f578ab


This is "quick hack" to make it work in some real use case. What is needed is ordering of all matching routes. I have a concept, will implement soon.


this should inherit from ApplicationController, instead, so you don't have to redefine render_404 below (it's already defined in Sufia).


all of these redirect calls should probably add a moved_permanently status to tell browsers and Google et al to flush these URLs out of their index, if they've been crawled by anyone. By default these will just be 302 Found temporary redirects

```ruby
redirect_to "/files/#{file.id}", status: :moved_permanently
```


Oh and also, it doesn't build right now when you mix contexts and COp, so there would be no cached code to rebuild, I think.


Instead of throwing an exception and then immediately catching it, this could more cleanly be written as

```ruby
def item
  render_404 ActiveRecord::RecordNotFound unless is_uuid
  file = find_item
  redirect_to "/files/#{file.id}"
end
```


See issue #10


same with this:

```ruby
    render_404 ActiveRecord::RecordNotFound unless is_uuid && is_datastream
     file = find_item
     redirect_to "/downloads/#{file.id}"
```


```ruby
redirect_to "/downloads/#{file.id}", status: :moved_permanently
```


same as above


sdjfhdjsf


```ruby
redirect_to "/collections/#{file.id}", status: :moved_permanently
```


You have too many commits and I cannot find a better place to comment.

1. Miss follow up problem.
4. Correct, but you don't need to consider odd/even length.
8. Your solution require extra O(n^2) space.
9. Please do not use "goto". It's not allowed in most modern programming style.


Aucune idée, faut demander à @firm1 la référence ;)


Miss follow up problem.


Gotcha, thanks. I'll wait for a new point version with this fix. Not a big rush, but all-conda gemini looks good and I'll work on adding in missing dependencies. Hopefully this'll make the install much easier long term.


Maybe...

Ok, I'll do it.


ok


@steevesv can you review this as well? thanks!


Да, но иногда в русских постах встречаются слова, написанные латиницей - названия песен, групп, книг и т.д. Такие слова не стоит игнорировать


You can get rid of this method, it's [defined in Sufia](https://github.com/projecthydra/sufia/blob/master/app/controllers/concerns/sufia/controller.rb#L15-L18)


yeah I got the code from that pull request anyway :).


This is important: let's make this change for all the redirects before merging. We definitely want Google to catch up on our changes.


`chine` → `China`

`follow up` → `follow-up`


Great solution!


The standard getAttribute call should work for this.  Selenium falls back to getting the property value if attribute value is not defined.


Fixing and commiting.


"static" is meaningless, enums only become known if you include the actual definition in another source file.

I am quite confused on what name system we actually use (see also below); I treat enum values like C constants, and use all-upper-case (with a prefix, but I am perhaps too much C-oriented). I am fine with another system, but see below.

The enum name is somewhat like a type, I would use class-name names.

Finally, negative values in enums are officially not supported afaik, as it is non-trivial how large the underlying integer type is. C++11 has a solution, I heard about being able to specify the size of enum values, but no idea how to do that. It may be easier to either remove all numbers, and let the enum decide on their value, or to define integer constants or so.


Can we consider alternative names for the network component?


Yeah, I only later understood it properly.
Still we have to fix 'basename' and 'coreutils' though (to make it consistent/nice) and then it is all good to go.


Was machst du junge?
Du kannst auch color.green und color.red


2 lines down, we use a different style of constant names.


Now should works. I restore the missing char _ (why my cleanup script deleted it?! Boh)
__pattern__ must be a simple string or pyload's parse_url routine goes crazy... XD


Два языка указать можно ведь? Это и будет решением нашей проблемы?


I agree. I don't think I'll worry about it now, but I'm definitely thinking of which components I can generalize and reuse elsewhere in the future.


Oh I've never seen .ONE before. Cool.


Since author profiles won't be part of  HydraNorth, we should return a 410 Gone code for these.


A note about the `%PROGRAMDATA%\Git\config` file would be helpful here, considering it's the `FILES` section of `git help config`


:) of course, you have any preferences or just "network"?


The `-MP` documentation in GCC says that it adds a fake target for each dependency other than the main file.  It looks like this line doesn't have the same behavior.  I'm not sure if it makes much difference here or how much trouble it would be to filter out the "main" file in this case.


I'm not sure I follow all of this, but it seems that you're creating a temporary table that will be used later in the reporting. We had spoken about having some default values for contents of columns, but this seems to be hardcoding them rather than just providing a browser default value on the column tabs with 'IN (22,23,24)'. Shouldn't these three fields be set up as defaults in buildForm stage, and then the actual values selected be used in building this query?


That sounds the best. @ericelliott I belive we should use that in the specs. It's much more clear.


This is the first time I see enumtype::enumvalue, so it's unlikely this is compliant to code style.

I don't mind it, but it is not common. It's also not really needed, as an enum value does have the enum as its type already.


it does not matter. The main thing is to download and install in the off.


Let's reuse the `obj.pointer` rect here, instead of creating garbage all the time.

See: https://github.com/melonjs/melonJS/blob/d627b5210b5dedb0434c92deb768330c67521794/src/input/pointer.js#L410-L421


fixes issue #2


Fixed.


c3819228597d88bba2a7b3b776f1299d5a03d9e8

That fixed all related problem, workaround seems to work back to 10.6 too.


How about "The object instance being produced by the stamp. If the initializer returns a new object, it replaces the instance."


Опечатка onion


this is a duplicate, removing


Noted.  Is 0x0 != 0 the only test failure?


Perhaps move the declarations at 360/361 closer to here as well?


it is now


Isn't this supposed to be based on a criteria around the name of the fieldset containing the fields including 'Survey'?


teste


+ some BCDEDIT mess around because of Kaspersky antivirus didnt like it. But officially the fix will come from them as it was an issue with their scanning engine.


(2) - with that logic in mind - using the changedState function appropriately cycles through the states at the leisure of other factors being at play (changing the state number for a reason - and deciding the state according to the switch statement), then calling the stateFunction() to call the state as a (main) function.


Have you seen that you can run parallel flows with iperf3?
Look at the -P option


We won't be able to use Sufia's default 404 method for that, so it will need to be

```ruby
 render template: '/error/404', layout: "error", formats: [:html], status: :gone
```


lol, oops.


Your code would be more readable and maintainable if you create a variable and pass that instead of running the query in the with statement


Why were changes were made to map1?


Hmmh. Yeah that shouldn't work should it? :)
Thanks for pointing this out.


looks like we've hardcoded these values again. :(


Why is this necessary?


Hey! I looked at your solution to problem #1 with finding the sum of multiple of 3 or 5. I approached the solution the same way.
Did you notice what happens when you put var sum = 0 inside the for loop? I'm trying to figure out why the solution doesn't work when sum = 0 is in the for loop but before the if statement.


it's wrong, you should trim spaces and one (last) CR+LF


hardcoded again :(


Just small improvements to make it look nicer :)
Changes a few tiles, and added an tile animation on the castle.


That's too short and vague. It doesn't answer the questions the reader will ask the first.

Let's make specs clear rather than short. :)


Just remove, you cannot #undef a C/C++ function, the pre-processor runs before the C/C++ compiler


I was going to try it for myself, but something is broken...

```
parasyte@Turkatron:~/projects/melonJS$ grunt serve
node.js:720
    var cwd = process.cwd();
                      ^
Error: ENOENT, no such file or directory
    at Error (native)
    at Function.startup.resolveArgv0 (node.js:720:23)
    at startup (node.js:63:13)
    at node.js:814:3
```


Add some spacing around the < ?


Rahul,

You are not using new label in a query, does it mean that your dir0 is a column in the directory you are querying ?

Vicky.


needs so much spacing :)


See https://github.com/angular/angular/pull/4435#issuecomment-144789359 the reasoning.


The test passes in develop, should eb OK to merge to master.

Thanks,

Aparna

On 09/02/15 08:02, Marshall Clow wrote:
> Merge pull request #22 from akumta/patch-1
>
> Update for Boost ticket #11594 - @akumta , please watch the test bots and ping me in a week or so to merge to master.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/boostorg/mpl/commit/53d9af7e8f2da32e6687b6896a023eeaeef200b4


As of Java7, any concatenation done with raw strings is automatically converted to StringBuilder operations during compilation.


this is ironic


Oh nice one ;p


This revision is clean.


Hmm, isn't that called "calloc" ?


(3) - For now in the Initial function - set the state number to a default - then call the changeState() function to call the menu state function on the first frame.


Ooooh, my shell was in a bad state from when I broke my local repo (testing the `gh-pages` patch). Fixed it now. :)


Because i check if the instance is null, then i lock it, but before the lock and while checking, other thread could have modified the instance, so i check again after i enter the lock


Port here is the port on the ExternalServiceTarget Host.

Service depends on ExternalServiceTarget, not the other way around. So the ServicePort.TargetPort maps to ExternalServiceTargetPort.Name (if string) or ExternalServiceTargetPort.Port (if int)


Lots of demagicizing, very nice :)


I believe 선점 means 'Loot' .. It has a similar meaning to 'spoils' or 'prize'.

It makes sense in this context.


Thank you for the comment and references.
I think so, too :smile:


You probably need to export the card container service from the card container module so that it will be visible outside of the library from the module.


I think my attempt at a longer description is the reason the confusion came up in the first place.


Will disable in pro.

Sent from my Windows Phone
________________________________
From: Michał Moskal<mailto:notifications@github.com>
Sent: 10/1/2015 5:50 PM
To: Microsoft/TouchDevelop<mailto:TouchDevelop@noreply.github.com>
Cc: Peli de Halleux<mailto:jhalleux@microsoft.com>
Subject: Re: [TouchDevelop] automatically select first line of code when entering an action (bd6e43e)

This is why we have the "pro" mode - not to annoy ever so slightly more advanced users.

—
Reply to this email directly or view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fMicrosoft%2fTouchDevelop%2fcommit%2fbd6e43e6abcbe8ba505d0f65aab651ededb908d8%23commitcomment-13546064&data=01%7c01%7cjhalleux%40microsoft.com%7c7a5ed6bbe7ab4ad3d52908d2ca806f01%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=B0uFJJIsBkVlcvD5jyJX5VFF8M8yz7Inv1MP3grssOE%3d>.


Vicky,

That is correct. I want dir0 to be treated as a column in the data itself.

- Rahul


Not sure i can, me.pointer just gives the coordinates of the primary pointer, and we could have multiple one when using multi-touch?


Oui, je sais pas ce que j'avais ce matin, j'avais cru comprendre qu'on pouvais également utiliser courriel pour désigner l'adrese e-mail, mais en revérifiant mes sources, il semblerait que non.

Les choix possibles sont :


@obiot The updated maps looks great! There's just one thing I would change in map2: use the same background ratios that are used in map1. As I recall, the mountains use 0.5, and the clouds use 0.25?


Do you want to make a Merge Request for this?


I was planning to anyway create a resuable object for this one, just wanted to have it working first before looking at possible optimization


Exzellent. :-)


It works! :D


I did update map2 to match map1, but i guess i missed those ones :)


선점 means Loot or Prize.

Preemption is wrong as it's similar to gaining a monetary advantage before others, colluding.. But Loot or Prize makes sense because you're gaining a reward.


I had the ratios backwards. But yeah! I noticed the mountains were not scrolling with parallax.


Do you also like the changes i made to the first map?


Will disable in pro.

Sent from my Windows Phone
________________________________
From: Michał Moskal<mailto:notifications@github.com>
Sent: 10/1/2015 5:35 PM
To: Microsoft/TouchDevelop<mailto:TouchDevelop@noreply.github.com>
Cc: Peli de Halleux<mailto:jhalleux@microsoft.com>
Subject: Re: [TouchDevelop] immediately open string literal editor when inserting (2285d7d)

Probably a good idea, but strings are often empty by default. If you're not beginner there is good chance you will want to use a local variable and not a string literal.

—
Reply to this email directly or view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fMicrosoft%2fTouchDevelop%2fcommit%2f2285d7d8ec64c39c47a57823c776ac5b8a508e67%23commitcomment-13545761&data=01%7c01%7cjhalleux%40microsoft.com%7c30db06ddb0ef40222fac08d2ca7e61c2%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=wnd07NHA0BAJxFzIrDjXhmLN6JrpakwtcBqfXpsE%2fd4%3d>.


Yes.


Definitely did not know that. I will test that out.


I'd like to keep validation of id, so 'id' property make any sense. As the 'string' id should be supported the possible values should be: 'uuid4', 'string', 'int' ('couse why not? :) ) any other value should default to string with $log.warn.


@mgjv feel free to look at this more recent update I had made that didn't make it into the PR (73).


cls.is(Module, butNot = Scala2x)


cool, just making sure I understand.


Please, forget the xdebug recursion limit. I mentioned it only as a hypothetical limit on a hypothetical Jan's question how would things change if application will run a week?

Recursion was not problem here because was shallow.

Better said, it was not problem **only** because was shallow.

In other words: recursion here was not used correctly.


didn't work ¯\\\_(ツ)\_/¯


@aliaspider, One thought just came to mind. You might want to detect if the system is a 2DS before turning the screen on and off, since that console only has one screen. If I'm not mistaken, turning off either "screen" on that console will result in BOTH turning off.


Heh, true.  But it's spec not tutorial, perhaps explicit description is better?

But the short version seems also ok.


With this framework - you now have to make the other states and - remembering the purpose for this scene = new createjs.Container(); in the state functions - we call stage.removeChild(scene) (in the changedState() function in game.ts (?)) to wipe away the objects (labels and button, etc.) of that scene, and the stage.addChild(scene) in the state function (when called via the changedState() function in game.ts(?)) will refill the canvas (which is the stage, but also meaning changing the scene of the stage, in terms of the current state) with the appropriate objects pertaining to the state (menu, play, or over) that gets switched to.


This one is actually no a mistake, as I wanted new objects to be presented separately (collection.html:44-53)


@jklontz this should do it.


CFGU_GetModelNintendo2DS should do the trick.


Original was: "Using the Stackato VM makes no changes to the underlying host operating system. If you wish to remove it, simply select the VM in your hypervisor and select Remove... or Delete... as appropriate."

I like the original better. This was added because of some weird requirement from some partner that our docs had to have uninstall instructions. Can't access the original bug, but:

https://github.com/hpcloud/stackato-doc-internal/commit/c8bab84e95bf32d6d335554e97dbc3e1a9e8af83


@dexX7 I had to add this because I used `Collections.singletonMap(toAddress, amount)` to create `outputs`. I was just trying to simplify the code, but because the singletonMap is immutable, I found this issue.


This might need to be (timeInterval / window) + 1? I.e. if we're given a point at t=0 and t=5 with a window of 5, we'd probably want to have 2 points?


fixes issue #5


[**@desiderantes**](https://github.com/desiderantes) commented on [1 Oct 2015, 19:29 GMT+2](https://github.com/desiderantes/venom/commit/666bbf1ec61910276a8993bdddb9c172f13ea7bc#commitcomment-13546864 "2015-10-01T17:29:03Z - Replied by Github Comment Enhancer"):
> Because i check if the instance is null, then i lock it, but before the lock and while checking, other thread could have modified the instance, so i check again after i enter the lock

Why not locking first ?


this fixes #154


Thank you. :+1: Appreciate this.


Huh, that was a simple fix. :D

Travis is still failing, but for different reasons now.


mDNS hostnames will have to be updated to new product name / URL throughout the docs. Do a search and replace on all "stackato-xxxx.local" strings. Consider using a variable.


Please keep PR separated: one issue = one PR


Because locking wastes more resources than branching (ifs), and you'd be locking every time you need an instance, which is wasteful


Good call, wrote some tests in what will be the ember-cli build: https://github.com/Addepar/ember-charts/pull/102


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


A tiny thing, but to match the style elsewhere in YNR's templates, the `{% if ... %}` and `{% endif %}` should be outdented by two spaces. (This is consistent with [this advice](http://mysociety.github.io/coding-standards.html#indentation-and-whitespace), even though I realise I've been misinterpreting it!)


maybe cache the result instead of calling t.performance_data.toJSON() twice?


I think C++11's enum class is the way to go here. Unlike C enums they are scoped like I did below, and the values are typed.


I can't repro this on my mac with Chrome 45. Do you have any add-ins installed? The test that's being used is a pretty standard test (and, to be honest, not one I've modified). Does `'ontouchstart' in window` return `true` or `false` when in Incognito mode?


Much better. Thanks @JoergFechner, @nineinchnick.


This is the style for C++11 enum classes. If I change the type above it will be valid.


Miss follow up problem.


@fprochazka „therefore it makes sense to have it above them in execution stack.“ totally agree.

But do not confuse execution (or whatever) stack with "call stack". Surely you don't want to say that it thing B would never happen if the original thing A not be executed, therefore thing A should be in callstack of thing B. This way no code works.


I'd prefer to keep to the logic used in this part of the code, Don't post-process flagsets but write (further up)

    MODULE_PKL -> Module | Lazy,

instead of just

    MODULE_PKL -> Module,


Yes, same behavior before and after this commit.


:soon:


Did it actually work? I figured this could get around the weird type problem by letting it infer whatever the constant actually resolves to. Not sure if it did the trick because my build works fine either way.

I can't understand what's going on with Travis. I inspect the builds and every test passes, then returns 1 at the end and says it failed ???


good idea!


Hi. If the variable is set inside the loop, it will be set again at 0 each time the loop goes around again. Setting it outside ensures that the sum will accumulate.


"to connect with" or "to connect using" the CLI client


All of rnc.cpp uses a different naming convention than the rest of CorsixTH. I was on the fence as to whether to be consistent with the file or the rest of the application. I will replace ```rncSignature``` with ```rnc_signature```


ah, seems logic. Sorry for the stupid question x)


por favor remueve esa linea twbs/bootsrap


Can you believe we missed this for over 2 years?


Some of the Argentine elections do use party-list-based PR systems, but I think we should sort that out with Martín after merging this.


Too many blank lines.


mcote used to want me to use list comprehensions when possible. What do you think of?

retval = {'framework': self._framework,
                'suites': [suite.toJSON() for suite in self._suites]}


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


Why remove this? Not sure I understand.


Yep! It looks more modern now.


All of our other engines use ruby 2.2.2 and I had some issues installing `rbx-2.2.3`. 2.2.2 seems better to me for consistency, but not sure if there was a reason we chose 2.2.3 initially.


Check out the Javadoc. http://docs.oracle.com/javase/8/docs/api/java/math/BigInteger.html


Por favor borra el código de prueba


s/mit/


I did not understand why you removed the wireme call.


This comment makes a lot more sense now, thanks.


No biggie. It's simply Ctrl+A, Ctrl+I to fix an entire file's indentation in Eclipse, no human thinking needed.


El código al parecer está bien, borra lo que te pedí y después lo reviso de nuevo ;)


No problem. I really appreciate that you took the time to improve my code.


I'm still reading, but I wonder about the benefit of these add/get methods. Wouldn't a regular attribute be ok? There doesn't appear to be any additional processing occurring which would require methods.


What jobs did you look at? The neko / haxe dev job of this build for example failed: https://travis-ci.org/HaxeFlixel/flixel-ui/jobs/67250367#L231-L235


did you know `'object' === typeof null`? Is that OK in this instance?


For example we can change Application::$requests to array of pairs $request & $response, it might be helpful.


You mean move load_extra's declaration/definition above the declarations for luaopen_sdl_*? I could do that. It would probably be even better to make load_extra a lamba function once we drop VS2013.


OK, since the worst case scenario is 11 seconds, this is fine.


@rauhryan couldn't get fresh vm's to install the gems without versioning them - let me know if this mucks with HB:E or anything


Also I investigated both compression and log compation. Offsets are weird in both cases, but the Marshal treatments should work.

Compression: Kafka compresses N messages into a single offset, so you still have sequential offsets. You just no longer have the ability to request a specific message -- now it's message "sets".

Log compaction: Kafka never renumbers messages or reuses offsets, and if you request an offset that has been removed the broker will automatically fast forward you to the next available offset.


@tenderlove You just became the **TOP #1 Rails Contributor** with this commit. How does that feel? :crown:


Besides, not only the demos that are built are relevant - they're just the most visible because they have colored output.

For example, there's also a coverage test (basically imports and compiles every single flixel class) which is failing right now:

```
/home/travis/haxe/lib/flixel-addons/git/flixel/addons/display/FlxExtendedSprite.hx:746: characters 6-27 : Unknown identifier : mouseStopDragCallback
/home/travis/haxe/lib/flixel-addons/git/flixel/addons/display/FlxExtendedSprite.hx:748: characters 3-24 : Unknown identifier : mouseStopDragCallback
/home/travis/haxe/lib/flixel-addons/git/flixel/addons/display/FlxExtendedSprite.hx:748: characters 31-37 : Unknown identifier : mouseX
/home/travis/haxe/lib/flixel-addons/git/flixel/addons/display/FlxExtendedSprite.hx:748: characters 39-45 : Unknown identifier : mouseY
```


`for !c.Terminated()`

?


Please provide a FULL explanation on the WHY not the WHAT.


OK


ditto list comprehensions?


why u put semicollon?


I did not quite understand: Why do Dotty modules not always require an outer pointer?


On line 121 above you check if the claim in this map is nil, but if one is nil you'll crash here. So you should probably check both places or neither.


Yes, I vote to remove it.


Good catch, man!

From: Troy Topnik [mailto:notifications@github.com]
Sent: Thursday, October 01, 2015 10:44 AM
To: hphelion/documentation
Cc: Khmelnitsky, Mike
Subject: Re: [documentation] Finished Quick Start (391e34e)

mDNS hostnames will have to be updated to new product name / URL throughout the docs. Do a search and replace on all "stackato-xxxx.local" strings. Consider using a variable.

—
Reply to this email directly or view it on GitHub<https://github.com/hphelion/documentation/commit/391e34ecf8125c50853e7a5558220fcecb2f49b5#commitcomment-13547104>.


Doesn't this lead to an error if there is no rental with that name. Also what's the parameter for


Ok. Cool, this explanation would be better to add to commit description or in code.
Do you have test for DllMain allocation?


ok thanks for the tip.
I'll add that as soon as I can get confirmation about the current behavior on 2DS.


btw, this is BC break. I have overridden processRequest method in my project.


>framework/resources/Datasources/metadata_caching/ctas_auto.ddl
+alter session set `planner.memory.max_query_memory_per_node` = 8147483648;

You are increasing memory used for sort operation here.
What is the minimum requirement for the hardware configuration (4 nodes, 8 nodes ...) for this suite to execute within reasonable time?
Will this test suite run on a single node with the default drill settings ?
I feel it's time to start labeling test suites that require cluster .... Maybe we should put a README with these details before we come up with the solution. Just a thought.


ty


This works the same if this if is on line 242 I think but is easier to read?


No, it's fine


sure, I don't remember why we have distributed artifacts in zip archives for darwin in the past (looking at fleet, specifically)


Agreed


It seems odd to do this in the critical path of consuming. Is there a reason not to make this a background goroutine like `manageClaims`?


I hope I didn't misunderstand how the "container" property was gonna be used, but if I did, then I'm sorry! :(
This is also a little buggy, because sometimes the player will be hurt when they stomp an enemy that has a second form. I will investigate this.
I also made a few changes to death effects.


Dotty modules are always locally instantiated. Either inside the class that contains the module, or inside the default method of trait.


Yes, but for now the only know work around is to keep updating the hash...


Ok, I'll remove it from 2.3


How did we get out of the for loop on 239 with a `nil` message?


Yeah, the jenkins configuration was already in place; the use site further down in the build scripts had a problem though.


En ce qui concerne ton dernier point, j'ai dû modifier car cette clé était utilisée pour plusieurs clés. Elles correspondaient toutes à "adresse de courriel", sauf une fois (sur la page des notifications). J'ai donc édité en conséquence. :)


If the `claim.messages` chan is empty, this for loop will busy loop forever while holding the read lock on the Consumer. Won't that cause the Consumer to go braindead since write locks cannot preempt?


The flags are checked for validity before gnix_recv


Broken link, should be "/standard/words" I believe.


If switching to ORM (#15) instead of hand-crafted SQL is probable enough to be reviewed and potentially accepted, please postpone merging this PR, as it is essentially part of the ORM change.


I see. So we need something like 10 rects (in a `Map`, keyed by pointer id?) that get reused. That would be satisfactory, IMHO.


Comments.


Yes. Ill correct that


Well check this out for instance:
https://travis-ci.org/HaxeFlixel/flixel-ui/jobs/83138375

It says it's "failing" but there's nothing in there that indicates a failure.


Some of the scripts in the Datasources/metadata_caching folder are used to generate large datasets used in Advanced/metadata_caching suites. These scripts will not be run during test execution. I just added them here as a reference if someone wants to understand how the data is generated.


Yes, that is a much cleaner condition.


deleted some old backup files


I don't have any experience with route53, so I will file an issue on it after this PR lands.

I will update `externalDSName`


i just did ^_^


Pretty sure you can just use list1.equals(list2) to compare two List objects.

From https://docs.oracle.com/javase/8/docs/api/java/util/List.html#equals-java.lang.Object-:

> Compares the specified object with this list for equality. Returns true if and only if the specified object is also a list, both lists have the same size, and all corresponding pairs of elements in the two lists are equal. In other words, **two lists are defined to be equal if they contain the same elements in the same order**.


warning to match the recent patches I have for s1s2/webapp ?


Oh I see now, it's just not in red...


nit: You can construct the Uint8Array [inline](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Uint8Array#Examples):

```ks
new Uint8Array([0x3C])


At this point we might not be claimed anymore, since `c.claimed` doesn't use the lock. Is that ok?


typo: publi~~s~~SubscribeChannel


Shouldn't you pass in `absentKeys`?


Fixing on merge, BTW...


it looks like the errors come because the test code might have FLX_NO_MOUSE defined?


Test case is here:
https://gist.github.com/dbrosoft/81a013dbcba44b5cd471#file-intersectionbug-java


Ah, I should, good catch


Fail here with a runtime exception instead of eating the exception. It should never happen though.


LGTM


I think you don't need this one.


Not yet. Need to work on that part a bit more.


Yeah, probably. There are actually two coverage tests - one has almost no flixel conditionals defined, the other one almost all of them, just to see that it works with and without a conditional. This still leaves room for weird combinations of conditionals to cause errors, but the vast majority should be detected by this.


This doesn't run forever if the coordination topic is brand new and empty does it?


Is this for the dataflow product, or not? I can run it, but it gives me `close because of CloseReason[1007,Illegal UTF-8 Sequence]`. Also I'd like to deploy to a dataflow process that never quits, for message grabbing.

Is this the point of this example? How would I deploy to Dataflow?


Isn't this check redundant, in light of the one above?


So do we have real errors in travis I need to fix, or is it just the fault of the FLX_NO_MOUSE false positive?


@taylorotwell is not recommended to install the homestead composer package anymore?


Please remove these two lines, they are not needed anymore. There is always history if we want to restore them.


Fails on Travis-CI: https://travis-ci.org/xp-framework/core/jobs/82889506


Have you tried this setting with a custom background too?


SELECT test_row(1, 2).col0 works in current code and it does not go through this code path. This method is called when visitUnnest or visitValues.


Let's try the short version and expand only if it causes confusion. =)


I'm fine with the change, but is there a reason you're installing rubinius instead of MRI?


What's the need for `(unsigned long)0 - 1`? Why not simply `(unsigned long)-1`?


And if we do change this, the Dockerfile should change as well: the two should follow each other.


Not sure how the FLX_NO_MOUSE thing is a false positive? Not flixel-ui related though.

Afaik, everything else is fine with this fix (though I think it still needs to be merged into dev?)


The generated css and js includes should not be committed, or what do you think, @121052? I'd suggest actually generating all files into a separate build directory from grunt so that it doesn't modify the sources, but that's not part of this PR of course.


This reverts the FPGA submodule to back to a commit from 2/18/15. Is that intentional?


Won't this cause large spikes at the beginning? If the data is nil,nil,10000,10001,10003,10004 then I think you'll get a big spike to 10000 followed by 1,2,1. If you're graphing that data then that "2" is going to be lost in the noise because the 10000 is going to dominate the graph. What if you waited for 2 good values before running the derivative?


Thanks everyone for keep wasting my time.

> Better said, it was not problem only because was shallow.

If a recursion is not tail-optimized (BTW this recursion could be tail-optimized but PHP does not do such optimizations AFAIK) it must always be shallow. Saying that "it was not problem only because was shallow" and concluding from that "recursion here was not used correctly" is equivalent to saying "every recursion in PHP is used incorrectly".

For example `IContainer::getComponent()` uses recursion but because we can safely assume that component tree will not be very deep, it is OK.


@racardoso @gut, That is right, the index is invalid here.


Oh, so we do need to fix that error? Make that class still work if FLX_NO_MOUSE is defined?

Anyways, guess I'll merge this in if it's "working"...


In the non-versionable case, the stored rendition should be found/reused only when it is "current", i.e., its last-modified date is equal to or more recent than its source document's last-modified date. OTOH, when the source document is more recent than its stored rendition, the rendition needs to be recomputed.

By default, the source document's "dc:modified" property is being used for this comparison. In the use case whereby a user wants to store the zipTreeExport rendition of a folder, the "dc:modified" property of the In reality, the "dc:modified" property of a folder for example does represent the last modification date of all the children in its tree.


Ah, thanks for catching that.


No. It hasn’t been for a while.

On Thu, Oct 1, 2015 at 1:14 PM, Gonzalo De Spirito
<notifications@github.com> wrote:

> @taylorotwell is not recommended to install the homestead composer package anymore?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/laravel/docs/commit/48eecaca99e5938ee9a37abb97755ac45c6efc79#commitcomment-13547673


So there would be a separate package that contained all the commands? Where would that live? I don't care too much, and since you called this a nit I would prefer not to go down this path.


why warning?


you're correct, but this is also not a long-running process so this likely wouldn't have caused an issue anyways


Very good point. Committed at https://github.com/openmrs/openmrs-module-xforms/commit/cc74e56a5ca549c4c9b4f8e58e1fcef397545b68


Hot dang! Keep up the hard work man! So cool to see this project is still lit c:


## This is a markdown file


TeamCity pull requests :: netty [Build 353](https://garage.netty.io/teamcity/viewLog.html?buildId=1935&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5616, ignored: 21 Build time: 00:37:58


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


this seems to lead to abuse of log levels.


As Artenus reaches a stable state, I will try to have less commits like this which are stuffed with changes.


Very nice


Why are buttons in the opposite order now?


Sure. It would be interesting to know either way what happens.


yep


Could we call Save button `Update`?


ok, will add them


Did this actually change?  If not, I propose leaving the original giant line in.


wtf?!


Ok, we can make them configurable later. Will remove configurability for now


Maybe change "the diagnostic file" to "each diagnostic file"?  That matches the language above and is also more clear about how it works.


wtf - wtf ?!
))


ACID part:
http://paste2.org/DD2zb3yj


same old WTF


Yes it will be remover on next commit


On second thought. I don't want to diverge too far from the scripts. They are seemingly configurable, but nothing is actually wired up to make them configurable through cloudformation. I think it's harmless for now.


Please use the GREEDY_REALLOC macro rather than the function directly.


atts atts atta atts


Yes this will be The next thing i will do


No. This was supposed to mean "for either VisTrails (the application) or ReproUnzip (the program as well, one of the all-included installers for Mac or Windows that bundle Python, reprounzip and plugins)".

Changing to "reprounzip package" is wrong in this context.


sure.


Remove ending period?


@ifandelse Please take a look on your implementation once more (https://github.com/voronianski/flux-comparison/tree/master/lux) because currently **your example doesn't work at all**, it throws such error in console:

```bash
Uncaught Error: There is no action named 'getAllProducts'
```


@evilhamster I see what you mean about right hand artist names missing album list mode (no need for tag example). No idea what is causing this, where does the skin get the data from? Another bit of Kodi to explore. Any suggestions?


What i mean the parameter name is confusing between `attributes` and `attachments`


I'm silly... don't worry about favicons for a mobile app 😝


maybe BatchControllerConf ?


This sentence (fragment?) is confusing.  How do you pass HTMLMediaElement elements to decodeAudioData?


done


We try to avoid doing zero and non-zero checks via boolean checks, to avoid confusion, see CODING_STYLE. It's OK to use boolean checks for pointers and actual booleans though.


yep


I think you need to initialize this again in PerfTest.setup_job and PerfTest.teardown_job. teardown_job would be sufficient if we didn't have an abnormal termination due to an exception, but to be safe I would do both I think.


Extracting CourseOverview here breaks call stack dependency for cms and lms: this method is used in course creation or rerun, but results in calling some lms functions (I found it trying to change course image default: it resulted in some missing settings from lms) in CourseOverview._create_from_course. Here is relevant part of my trace:

cms/djangoapps/contentstore/views/course.py line 259 in course_handler
      return _create_or_rerun_course(request)
common/djangoapps/util/json_request.py line 27 in parse_json_into_request
      return view_function(request, *args, **kwargs)
cms/djangoapps/contentstore/views/course.py line 709 in _create_or_rerun_course
      return _create_new_course(request, org, course, run, fields)
cms/djangoapps/contentstore/views/course.py line 790 in _create_new_course
      new_course = create_new_course_in_store(store_for_new_course, request.user, org, number, run, fields)
cms/djangoapps/contentstore/views/course.py line 821 in create_new_course_in_store
      initialize_permissions(new_course.id, user)
cms/djangoapps/contentstore/utils.py line 52 in initialize_permissions
      CourseEnrollment.enroll(user_who_created_course, course_key)
common/djangoapps/student/models.py line 1072 in enroll
      course = CourseOverview.get_from_id(course_key)
openedx/core/djangoapps/content/course_overviews/models.py line 228 in get_from_id
      return course_overview or cls._load_from_module_store(course_id)
openedx/core/djangoapps/content/course_overviews/models.py line 175 in _load_from_module_store
      course_overview = cls._create_from_course(course)
openedx/core/djangoapps/content/course_overviews/models.py line 128 in _create_from_course
      course_image_url=course_image_url(course),
lms/djangoapps/courseware/courses.py line 133 in course_image_url (here comes an undefined setting error).

Do you have any ideas how to fix this? I'm not familiar with CourseOverview model.


Not necessary?


Maybe some service would be better for this ?


They can, but I feel like one option per line is more readable. It also means that new config options added show up as a new line, rather than a changed line.


Spec tests should help against such confusion.


s/test that/tests that/


Used to be - this file can go away now.


I'd prefer to solve this in the future. We really need to consider ACLs, too


+1 great changes, especially leading them to the D8 issue queue for current planning. Thanks @nataliejeremy


I'm not sure what the best values are here but having them set explicitly seems better than relying on defaults which can change when we update okhttp.


a tak na prawdę, to fix #35 - pomyłka była przy commicie


Maybe I'm blind, but I just  3 changes to one part. Changing from inchs to mills, it seems you're starting to love the SI ;)


Why do we need the map anyway? Wouldn't it totally suffice to track the PIDs of the forked clients in a "Set" object?


Yeah, it would still have a spike at the beginning.  If we waited, what would we use for the values where some are nil?  `nil`?


Искре така като комитваш в Гитхуб после ние не можем да си свалим кода нормално а трябва да го растартираме всеки път, не отдели ли време да разбереш как става. Мисля че е време! Казах ти за лесния вариант с Вижуал студио


Generally, you want to invoke _exit() in forked, but not exec()ed child processes... so, stick to that here...


- You should gather some statistics about success/fail import products
- Web interface to view running jobs, statistics and ability to start/stop/edit(schedule) job


I'm not too worried about this. It really isn't any different, imho. This cloud-config will live in a template in our S3 account, so if you have an issue with pulling things over the public internet from us, you won't get to the point of nit-picking where the install-controller.sh script lives.


This had me thinking that maybe .kind is the wrong name to offer to kinds for extension, maybe `extend` would be more meaningful, and keep people from reusing `kind` ?


Any idea when we can expect to see this in a release version ?
This change is in 4.2.1  latest release is 4.0.4


Yeah, I was thinking `nil` until 2 real values were available.


Well it's a simple string on the only port of the Directory module at the top, but this could be mentioned.


The broader point is that we really shouldn't have to write our own template parser. There's a template package built into golang that we could use. However, in the interest of not delaying this code landing and solving real problems for folks, I would like to revisit this later.


With this change, retest TICKET 7282 passed.
I was now able to set and change a test suite customer field via xmlrpc.
Thanks Luiko


Wow, that used not to make any sense indeed.
LGTM.


Whoops, I was treating this like a select where you needed a default to break out if no cases matched. Will remove.


LGTM


just commited

On Thu, Oct 1, 2015 at 8:38 PM, Luiko Czub <notifications@github.com> wrote:

> With this change, retest TICKET 7282 passed.
> I was now able to set and change a test suite customer field via xmlrpc.
> Thanks Luiko
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TestLinkOpenSourceTRMS/testlink-code/commit/d4dd54009f9d92ae8ecee853ae41b893f7a90a8f#commitcomment-13548155>
> .
>


That's fair. We'll open all UDP egress.


Yep I'll get it published when I get back to the office

On Thursday, October 1, 2015, Daniel Hwang <notifications@github.com> wrote:

> Is there anything I can do to help release this fix to npm?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chuckmo/grunt-ssh/commit/f5ce3623f4e0d4cd6f80ba56077c87345e8c641c#commitcomment-13545924>
> .
>

--
Sent from Gmail Mobile


and just tested


Well, still, we don't work this way in systemd, we don't wait arbitrary amounts of time. Wait properly for the event in this case, which is the process exiting, and then maybe put a timeout on it (sigtimedwait for example in this case), but don't wait for no reason for a fixed time if what you actually want to wait for is something else...


GREAT!!!
Thanks for your help

On Thu, Oct 1, 2015 at 8:40 PM, Luiko Czub <notifications@github.com> wrote:

> and just tested
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TestLinkOpenSourceTRMS/testlink-code/commit/d4dd54009f9d92ae8ecee853ae41b893f7a90a8f#commitcomment-13548194>
> .
>


@h2o64 Nope, I assumed that the ramdisk folder contained the ramdisk


$scope.collection.factory is undefined


got it


Keeping the `Assert (option)` part would be useful, as `option` is required to be non-NULL in this function.


ack


Uid PWD does not work anymore.


ok, will expire at 1/2 the time.


Sorry about that…I did not notice I guess.

May I make up and fix it now and waive the penalty?

Thank you for your patience and help!

Best wishes,
Qi
> On Oct 1, 2015, at 1:11 PM, Fei Yiran <notifications@github.com> wrote:
>
> Miss follow up problem.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/CrystalQi/CCAssignment/commit/0613f97baf27a7bf5f849d4b4e4db604b972ee05#commitcomment-13546501>.
>


resuelto ;)


Така има строга зависимост мисля че не е добре
става еднотипно в даден момент


This is in bytes... Is 8KB significantly different from the default of 0 for us? I feel like all of our files will be much bigger than that so this will always spool to disk anyway. I think hundreds of KB or 1MB might be more reasonable. I don't have any other comments on this PR though. Regardless of what you decide on the max_size here, :+1: merge it.


In inits and union
```
 inits' [] = [[]]
+inits' (x:xs) = reverse' (tails' (x:xs))
```
If you don't use pattern matching `(x:xs)` then just use `x`

```
 inits' [] = [[]]
+inits' x = reverse' (tails' x)
```


Inspired by https://github.com/MarkRabey/SideNav/commit/f0d1865dc367720998d281082262e80c07c34ee1 thanks @markrabey!


add it our module


must be public because it's used as parameter in interface method, well I am not strictly against removing this, but could be helpful in case we will need to add more params in future, or it can nencapsulate all of the params?


inject the dependency in the chat controller


Alternatively, If either prev or cur is `nil`, could also just append `nil`.  That would avoid the initial spike too.    If you had data like `nil,nil,10000,10001,nil,10003,10004`, the `nil, 100003` values would get converted to `nil`.  Using `fill(previous)` might work for that data though.


@jmcphers Could you review this? I noticed that toc_depth wasn't supported for html_document in the UI (but was for pdf_document). Since this has worked from day one with html_document I'm not sure if there was a reason it wasn't included that I'm not aware of. Also, the default value in the package is 3 however in our JSON definition it was 2. I also recently added toc support for MS Word so I've not marked it as a transferrable option.

Anyway, these changes may not be appropriate based on considerations I'm not aware of. Let me know what you think.


ha, getting into Interceptors and all, #okhttpMaster


got it


So I've tested this. `getAttribute('paused')` returns null. It doesn't seem to work on chrome or webOS.

https://selenium.googlecode.com/svn/trunk/docs/api/rb/Selenium/WebDriver/Element.html#attribute-instance_method

According to the docs it should work, but maybe it will only work for the listed properties.

As for right now I'll keep investigating.


I'll figure this out with @robszumski


That's better, cool!


Do you think this function can be improved to make it more readable?


To me it seems good idea ta have them public just to let the public know they exists. For now these constants are used in GoodDataHttpClient and LoginSStRetrievalStrategy, so if we don't want to be known, it can be as you recommend.


Ah right! I haven't look on the second change in the line. Factory to resource change is right ;)


Adding 3rd party libraries to your repo should always be tracked in a separate commit so there is a clear distinction between configuration effort + downloaded files.

Like we discussed today, the `/dist` (build) folder should not be tracked in your repo since it gets compiled from our `/app` (source) directory. Also, when you use grunt copy to start copying the third party libraries from `/node_modules`, you can eliminate all of these from being tracked in the repo.


侧滑菜单, 可基于此思路或者直接将该控件导入你的工程中进行使用


Adds support for #47.


Looks great so far. Be sure to mention heavy use of Bootstrap to accomplish this.


probably I had them used more times at some state of development ;-) can be inlined now...


sure


My bad, seems like I looked at particular commit.


This isn't actually reusing the peer config!  It's creating a new instance every time the function is called.  It would need to be created during `newServer` and stored there as `s.peerConfig`.  Then where this function is currently being called, they would just access the stored struct pointer.


The **or** operator should be bitwise `|` and not logic `||`. Please change both.


3rd party deps are okay here since it’s a relatively lightweight, and well-encapsulated commit. :+1:


I *may* have gone a little overboard and documented all the functions in that file.  But I did this one too.


need a comment explaining the hardfork situation here (and below, for DSCOs as well)


Not sure about tracking that `.grunt` folder.


Oh yeah, I could see Question Answer Fields too.  I guess I was thinking
Add Tags : Taggings (which is what's produced by add_tags_uri)
::
Add Question Responses : Answers (which is what's produced by add_question_responses_uri)

Could go either way on it though.


J'aimerai bien faire un cherry-pick de ce commit en prod, mais je ne voudrai pas faire d'erreur.
Tu confirmes @yaf que c'est comme cela qu'il faut procéder ?

```
git checkout production
git cherry-pick <commit>
deploy.sh
```


ditto with stuffing the println into the panic


`fill(previous)` works pretty well for missing data in the middle. I like your suggestion, too. In that case, if you want to know where the missing data is, then you can query without `fill(previous)`, but if you don't want to miss the `10001` -> `10003` transition then you can use `fill(previous)`.


@anatolykazantsev I find it easier to discover new features and remember where they are if they are always in front of me. If it's on one page, but not on another I will never find it out. For example, how will the use know that Expr. Order is available if they never add anything to cart? Normally they don't, so we need to show it to them anyway, then they click and find out.


Can this send a message to the user when they exceed the limit?


Thank you for your effort! :smile:


You misspelled "daaaaaaaang"


Yeah and if we go the Answer way, then we should change the whole helper
function.
lets discuss too

On Thu, Oct 1, 2015 at 2:54 PM, Shai Sachs <notifications@github.com> wrote:

> Oh yeah, I could see Question Answer Fields too. I guess I was thinking
> Add Tags : Taggings (which is what's produced by add_tags_uri)
> ::
> Add Question Responses : Answers (which is what's produced by
> add_question_responses_uri)
>
> Could go either way on it though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joshco/osdi-docs/commit/6a2c0db588cdecc873890b34af7264ce2780008a#commitcomment-13548515>
> .
>

--

*Josh Co*hen  joshco@gmail.com | @joshco | LinkedIn
<http://linkedin.com/in/joshrcohen/> | GitHub <https://github.com/joshco>
  | OSDI opensupporter.org <http://opensupporter.org/>
"If [OSDI <http://opensupporter.org/>] succeeds, it could usher in the next
chapter in political technology." - Washington Post <http://wapo.st/1hYXX7O>

Join the #Day1 campaign and help stop bullying.

Engaging in isolating gossip, out of view of the target is bullying too.
Don’t do it.


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Text.RegularExpressions;
using System.Threading.Tasks;

namespace _3_ExtractEmails
{
    class Program
    {
        static void Main(string[] args)
        {
            string text = Console.ReadLine();
            // Innos             @"(?<=\s|^)([^\W_][\w.-]*[^\W_]|[^\W_])@[^\W_](([a-zA-Z-]*[a-zA-Z]+|[a-zA-Z]*)\.([a-zA-Z]+[a-zA-Z-]*|[a-zA-Z]*))+[^\W_]\b";
            string patternMeil = @"(?<=\s|^)([a-z0-9]+(?:[_.-][a-z0-9]+)*@[a-z]+\-?[a-z]+(?:\.[a-z]+)+)\b";
            Regex regexMail = new Regex(patternMeil);
            MatchCollection matches = regexMail.Matches(text);
            Console.WriteLine("Found {0} matches", matches.Count);
            foreach (Match name in matches)
            {
                Console.WriteLine(name.Groups[0]);
            }
        }
    }
}


one


comment


Hoofdletter > Saldo mag niet negatief worden


I think it is better to mention! Thanks!


That was my first instinct by `extend()` already exists for MixinSupport so we'd have to find another word. Rather than further confuse things, using `kind()`, which devs already know, seemed easiest.


Hoofdletter : Aantal liters....


TeamCity pull requests :: netty [Build 355](https://garage.netty.io/teamcity/viewLog.html?buildId=1937&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5616, ignored: 21 Build time: 00:37:31


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


testing


Oh, I thought you meant the VisTrails package. Let's use *reprounzip* (component) to be consistent with the documentation.


TeamCity pull requests :: netty [Build 354](https://garage.netty.io/teamcity/viewLog.html?buildId=1936&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5616, ignored: 21 Build time: 01:11:56


TeamCity pull requests :: netty-tcnative [Build 19](https://garage.netty.io/teamcity/viewLog.html?buildId=1944&buildTypeId=pulls_netty_tcnative&guest=1) is now running


As soon as I merge this in we'll have a place to style that page specifically. I put a couple little things in there just to make it work, plus added the links @matthewp mentioned.
https://github.com/donejs/donejs/pull/135


Maybe https://github.com/Unihedro/lila/blob/d4334918c88b74bc55106c102c84425b8efe38bf/public/javascripts/big.js#L476 needs to be changed too.


How is this different other than before? The only difference I can see is that we are now no longer skipping the logic if `curTime == itr->second->expireTime`. Also since the iterator is incremented in all code paths, just like before, couldnt we just move that to the for loop?


TeamCity pull requests :: netty-tcnative [Build 19](https://garage.netty.io/teamcity/viewLog.html?buildId=1944&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:50


Disagree with this as it can lead to gnarly bugs.


Likely just an oversight; this change looks good to me. At some point we might consider auto-generating this JSON data via an RPC that reverse-engineers it from the function parameters in `rmarkdown` so that it's always right, although that's enough work that we'd have to have considerable motivation for doing it.


The script should check if the preview window is actually visible. Otherwise you could not change the instances in the preview view as "PreviewPanel" is always set.


From that comment, its possible you also fixed an entirely separate crash related to traversing server in a multiple map server config. I kept seeing the crash dump show the wrong zone, and nobody could figure out how the heck that was possible. The only thing players triggering it had in common was they were multi boxing and crossing map servers.

I'll let you know if you killed 2 birds with one stone as soon as I can get updated and test if.


Yeah, I think this is static enough that we don't need to auto-generate.

On Thu, Oct 1, 2015 at 3:02 PM, Jonathan <notifications@github.com> wrote:

> Likely just an oversight; this change looks good to me. At some point we
> might consider auto-generating this JSON data via an RPC that
> reverse-engineers it from the function parameters in rmarkdown so that
> it's always right, although that's enough work that we'd have to have
> considerable motivation for doing it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rstudio/commit/b53b1fc5607dd22fd79568e3cd5c60d73272669d#commitcomment-13548693>
> .
>


This looks like a pretty worthwhile feature. I'm curious about what use case you encountered where you needed it? Should we roll this into the core of osc.js?


on the `Response` constructor, think we should have a null check?
if for some reason the `r` passed is null, everything else will start throwing NPEs left and right.


Synchronize as a safety net?


It's a dumb implementation, and I'm not even sure it will work with future rules. But it correctly applies that one rule we defined!


TeamCity pull requests :: netty [Build 356](https://garage.netty.io/teamcity/viewLog.html?buildId=1942&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 3600, ignored: 12; compilation error: Compiler (new) Build time: 00:03:47