Thes entities couldn't be jsx but plain js. It is incorrectly refer to some component but it isn't.


TeamCity ArachNGIN :: Build [Build master-1.0.1-48-gd652bd1](https://tc.phome.cz/viewLog.html?buildId=18&buildTypeId=ArachNGIN_Build&guest=1) outcome was **FAILURE**
Summary: Unexpected error: java.lang.IllegalArgumentException: Argument for @NotNull parameter 'artifactsPaths' of jetbrains/buildServer/agent/impl/artifacts/ArtifactsBuilder.setArtifactsPaths must not be null (new) Build time: 00:00:53


TeamCity Gitversioner :: Build [Build 8](https://tc.phome.cz/viewLog.html?buildId=20&buildTypeId=Gitversioner_Build) is now running


TeamCity Gitversioner :: Build [Build master-1.0.1-4-g8ad3c7d](https://tc.phome.cz/viewLog.html?buildId=20&buildTypeId=Gitversioner_Build) outcome was **FAILURE**
Summary: Unexpected error: java.lang.IllegalArgumentException: Argument for @NotNull parameter 'artifactsPaths' of jetbrains/buildServer/agent/impl/artifacts/ArtifactsBuilder.setArtifactsPaths must not be null (new) Build time: 00:00:09


特殊化・・・そういやそんな手があったな、うーん、汚い。

``dxle``名前空間に``noexcept_arithmetic_class``とかいう空の構造体を追加して、それを継承しているclassを使えるようにすれば、例外投げるような糞仕様のクラスを使おうとする輩が減ったり・・・しないよなぁ、うーむ。

というか基本演算ごときに例外をつかう神経が理解できない・・・。


Please can you revert this change? I believe this code was written for readability, not for its conciseness.


Empty span should be non pair TAG.


@demoneaux I'm facing an issue with using `gem`. Will try this on Ubuntu and let you know in a while.


Try to turn on Atom Indent display to help indent code properly.


Instead of ```let``` lets use ```const``` if it is possible. (More immutable way)


TeamCity ArachNGIN :: Build [Build 6](https://tc.phome.cz/viewLog.html?buildId=22&buildTypeId=ArachNGIN_Build&guest=1) is now running


I added a test.


TeamCity ArachNGIN :: Build [Build master-1.0.1-48-gd652bd1](https://tc.phome.cz/viewLog.html?buildId=22&buildTypeId=ArachNGIN_Build&guest=1) outcome was **FAILURE**
Summary: Jcifs.smb.SmbException: The filename, directory name, or volume label syntax is incorrect. (new) Build time: 00:00:27


![screenshot_1](https://cloud.githubusercontent.com/assets/16239080/12551590/abd00516-c37c-11e5-8bbc-58512b593005.jpg)
![screenshot_2](https://cloud.githubusercontent.com/assets/16239080/12551591/abd407e2-c37c-11e5-862d-a805b697d911.jpg)
We have great density for ULF and VLF, because there is no preproccesing.(
(second image is for corrected data from article)


I believe it would,

By mounting /var/lib as tmpfs this will effectively block access to the actual on disk folder having the same "perceived" result as deleting the entire content of /var/lib at every boot-up, it is for this same reason the /var/log contents need rebuilding during each reboot.

The only way this might work is to rename the original so a to have a proxy folder eg /var/_lib/ and use rsync to maintain a disc copy of the tmpfs version and load that disc copy at each boot. I had suggested this method to use with /var/log as the minimal file size and count due to using logrotate made it a feasible option.

However I think this may well be too much to manage with /var/lib.

Paul


TeamCity ArachNGIN :: Build [Build 7](https://tc.phome.cz/viewLog.html?buildId=24&buildTypeId=ArachNGIN_Build&guest=1) is now running


What is the use of the above setUuid?


TeamCity ArachNGIN :: Build [Build master-1.0.1-48-gd652bd1](https://tc.phome.cz/viewLog.html?buildId=24&buildTypeId=ArachNGIN_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


This is not necessery, if you use ```onKeyDown={() => this.handleSubmit()}``` down.


@isidorn if this ends up to be the same color in sidebar and panel, I suggest to introduce compositePart.css and reuse colors for sidebar and panel in there (maybe there is more)?


@hyprstack never style using ID's! Add a class and use that instead. ID's have a very high specifity which makes them difficult to handle when there are many components.


Interesting. I just learnt that people prefer using uppercase `L` to denote a long instead of lowercase `l` because `l` can be confused with 1.


nevermind.. meu packagist estava na v1.0-dev ao invés da dev-master


TeamCity Gitversioner :: Build [Build 9](https://tc.phome.cz/viewLog.html?buildId=26&buildTypeId=Gitversioner_Build) is now running


TeamCity Gitversioner :: Build [Build master-1.0.1-4-g8ad3c7d](https://tc.phome.cz/viewLog.html?buildId=26&buildTypeId=Gitversioner_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:16


This is hard to read, since the end of the or (||) statement is on another line. It's best to keep either all statements on the same line (when very small), or each part of the main logical statement on a separate line (on cases like this). For example:

```javascript
if (cols.length === 1 &&
  (!cols[0].get('value') || !cols[0].get('value').get('fields').length)) {
```

It would be even more readable if there was a function with this `!cols[0].get('value') || !cols[0].get('value').get('fields').length > 0` logic, since it's used on 2 different places here, and is very big to be inline. Something like `_isColumnEmpty`, then the code would be much clearer:

```javascript
if (cols.length > 1 || !this.isColumnEmpty(cols[0])) { // line 158
```
```javascript
if (cols.length === 1 && this.isColumnEmpty(cols[0])) { // line 169
```


In future, lets look at css-modules. It is cool feature for react-like apps.


You mispelled canola!


Nno, this isn't spelled out explicitly in the regmap file.  Some additional information is in the PDFs for individual RS-422 daughterboards such as 7i69man.pdf but not all of the above information is there—particularly the bug in sserial roms version 43 and lower (i.e., all of them in the wild)


Why not just use `validity.valid` ?


'shuffle' is a verb, please find a more suitable name for this object.


I renamed the macro to ALOGE_ERRNO_NUM. It means that the caller can specify the errno code explicitly, instead of using what's stored in errno.


Il manque un test pour tout ceci


Removing the date is not the right approach. Here is some guidance from the Free Software Foundation: https://www.gnu.org/prep/maintain/html_node/Copyright-Notices.html

Ref PR #1696


We don't need any of these vendor prefixes. Our grunt task 'autoprefixr' automatically produces the correct vendor prefixes for us. So, in this example: `display: flex;` will work everywhere without us having to worry about it.

https://github.com/postcss/autoprefixer


TeamCity Gitversioner :: Build [Build 10](https://tc.phome.cz/viewLog.html?buildId=28&buildTypeId=Gitversioner_Build) is now running


Again, remove ID and replace with a class


TeamCity Gitversioner :: Build [Build master-1.0.1-4-g8ad3c7d](https://tc.phome.cz/viewLog.html?buildId=28&buildTypeId=Gitversioner_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:16


oops, should have been "each kind of remote error"


TeamCity ArachNGIN :: Build [Build 8](https://tc.phome.cz/viewLog.html?buildId=30&buildTypeId=ArachNGIN_Build&guest=1) is now running


A comment has been lost here which would be a shame as it helps understand the code.


TeamCity ArachNGIN :: Build [Build master-1.0.1-48-gd652bd1](https://tc.phome.cz/viewLog.html?buildId=30&buildTypeId=ArachNGIN_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:26


the patches aren't committed


What is the use of setUuid above?


I think this change should be reverted because the comment is lost and because there might be more conditions added later.


This is what I told about in email. In new flux application is recommended to not use muttable JS methods like ```push, pop, shift, unshift, splice``` & recommend to use immutable ```concat, slice``` and spread operators.


Can you change this test's description to make it clear that this will only happen for the next to last row?

Also, it's not exactly true that this logic is making the row go back to its initial state. The form builder could have already been started with a filled form with fields, so removing a field would not return to any previous state, it would just remove the row. It's best to say that instead in the description then, that the row will be removed, not that it will go back to the initial state. It will be clearer and more correct.


Is there a really good reason why we're using 'div' or 'elements' in the class name? Classnames are much better when they describe what the component does rather than what it is. Maybe a rethink on this one?


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


Can you also add a test which confirms that two non hibernate managed objects of the same type are equal if they have the same uuid?


Cool I see this was done, but not the above changes. I have marked the above two changes with a comment.


Not sure if this will actually do anything will it? Think you mean `input[type=checkbox]`


This is something like REDUCER. In react REDUX way it is better to not change state inside the reducer, but reducer will return new changed state (if should be changed). Otherwise return undefined (if no changes).


This has been fixed.


Again I think that this is overkill


slbaseopts  =- is used as a template, to write your first machines file... cool right?


This looks quite brittle to me - isn't this going to break as soon as someone changes the paragraph markup only slightly?


Slim looks very nice & easy for small projects ;)


Еще раз посмотри на разметку.
https://htmlacademy.ru/courses/67


Go with everything on this side.


Its working with jasmine 2.
But it seems to have some indexing issue as Name of suite 1 doesn't appear and test of suite 1 appears under suite 2.Also total test executed is 1 less than total number of tests executed.


It breaks things… :-/


Please revert the changes in this file (DonationConstraintChecker) also. It's much better the way it was previously.


@methodik Oopss, my bad! Will change that.


helo I hv some question hope u can help...one guy own my money give me one MacBook pro retina 13 ,but give no password I don know what can do ?can u help pls?


Please revert these changes from DonorConstraintChecker.


Ok. Noted. Was unaware of that.


Fair.


Docstring for conflict ?


@methodik probably.


Not so convinced about these changes being better either (TestBatchConstraintChecker)


Still to be done - Add the default() handler for the menu option case switch and add a input to set file content when using the writeFile() function.


that is breaking the support to build against a remote repo, no?


TeamCity Civic Assembly :: Build backend + frontend [Build 23](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=430&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 1722, ignored: 12 Build time: 00:21:15
### Failed tests
```
User Management: Should submit a form for user creation: <no details avaliable>

```


> To update the list of year numbers, add each year in which you have made nontrivial changes to the package.

This is crazy. I'm pretty sure no developer really wants to keep the list of years up to date when touching a file. And the fact that they recommend doing it for the whole project as soon as you make any significant change to any file just proves this. I think it's safe to assume that every year contains at least one such change.

---

Besides that, I'm pretty sure the FSF recommends this mainly because they want code to be as free as possible. I think what should matter here is whether the Flask project has any disadvantages by:

- not putting years there or
- simply putting "firstyear - currentyear" everywhere and updating it yearly


trailing `return reason` ?


The guide id is a string with numbers and letters, is'n it


Ok pour moi :)


Which makes the compiler produce an is-always-true error.


Are you sure that we must to have two ```;;``` here?


This test is hard to follow now, since it's also checking for the behavior you've just added to the code, but the description doesn't say anything about rows being removed, we only know about that because we understand how the code will work. Before this change it was better, since it was checking that the new form builder had the 2 rows passed to it. But now it's very complex.

Tests should be clear to people reading them, as they also act as documentation for the code's behavior. It's hard to maintain tests when you don't know why they're expecting what they're expecting.

It'd be best to keep this test simple as before, having the first row have content, so that it keeps the same purpose as before: checking that received attributes are correctly being passed to the form builder.

If you think it's important to have a test checking that this case of passing empty rows in the constructor also works for the next to last logic you've just added, then there should either be a new test case with a suitable description for this, or this test's description should be changed accordingly.


Can you remember why we did this? AFICT BottleDaemon is vendored anyway?


@appleguy This change breaks all truncations since ASTextKitTailTruncater calls [self _truncate] in it's initializer.

I think this change relates to #1033, #865.


Hey Stian, I think that's a mistake on my part. Thanks for looking at this.


In methods setcatapult and setintake the parameter names cannot be numbers. Each parameter name has to include at least one character before a digit.

P. S. Im probably going to miss tonight because I have surgery today.


HEY GUYS THIS GIVES AND TAKES MONEY BUT ONLY IN ALL CAPS


``getParameterCount()`` is not available in Java 7.


We need to research how extract variable value.
```
jint local_variables_count;
    jvmtiLocalVariableEntry* local_variables;
    error = jvmti->GetLocalVariableTable(method, &local_variables_count, &local_variables);

    if(error == JVMTI_ERROR_NONE) {
        jvmtiLocalVariableEntry* entry = local_variables;
        for(int i = 0; i < local_variables_count; i++, entry++) {
            printf("Local variable name:%s\n", entry->name);
            printf("Local variable signature:%s\n", entry->signature);
            printf("Local variable slot:%d\n", entry->slot);

            char* variable_signature = entry->signature;
            jint slot = entry->slot;
            jint depth = 0;
            char* variable_name = entry->name;

            switch (variable_signature[0]) {
                case 'Z':   /* boolean */
                    jint boolean_value;
                    jvmti->GetLocalInt(thread, depth, slot, &boolean_value);
                    printf("Local variable value:%d\n", boolean_value);
                    break;

                case 'C':   /* char */
                    jint char_value;
                    jvmti->GetLocalInt(thread, depth, slot, &char_value);
                    printf("Local variable value:%c\n", (char)char_value);
                    break;

                case 'S':   /* short */
                    jint short_value;
                    jvmti->GetLocalInt(thread, depth, slot, &short_value);
                    printf("Local variable value:%d\n", short_value);
                    break;

                case 'B':   /* byte */
                    jint byte_value;
                    jvmti->GetLocalInt(thread, depth, slot, &byte_value);
                    printf("Local variable value:%d\n", byte_value);
                    break;

                case 'I':   /* int */
                    jint int_value;
                    jvmti->GetLocalInt(thread, depth, slot, &int_value);
                    printf("Local variable value:%d\n", int_value);
                    break;

                case 'J':   /* long */
                    jlong long_value;
                    jvmti->GetLocalLong(thread, depth, slot, &long_value);
                    printf("Local variable value:%ld\n", long_value);
                    break;

                case 'F':   /* float */
                    jfloat float_value;
                    jvmti->GetLocalFloat(thread, depth, slot, &float_value);
                    printf("Local variable value:%f\n", float_value);
                    break;

                case 'D':   /* double */
                    jdouble double_value;
                    jvmti->GetLocalDouble(thread, depth, slot, &double_value);
                    printf("Local variable value:%lf\n", double_value);
                    break;
                case 'L':   /* Object */
                    jobject object_value; // The object returned by object_value is a JNI local reference and must be managed.
                    jvmti->GetLocalObject(thread, depth, slot, &object_value);

                    // TODO extract value
                    break;
            }
        }
    }
```


this was caused by the `--system-site-packages` virtualenv setting on teamcity


@jannyHou please never commit changes like this and ccf52f0 directly to master, always create a pull request - even if it does not need code review.

See also #767 where the user is asking us to run `mocha` directly.


Hi
Just found your work! Fantastic! Thanks!

My 2 cents:
in order to group similarly named functions together, and to make code completion easier
rename from `fast_array_unique` to `array_unique_fast`


where I learn that `learned` is an acceptable form in English :)


I don't understand this change.  SynchronousExecutor() is less performant than a ThreadPoolExecutor and except for MySQL could lead to a deadlock.


Good point. It seems _target platform_ is one among the possibly many _variant axis_ and in the end we want all _variant coordinates_ to be included in the report. Sounds fair?


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


I think that We have to check if type is interface or abstract class due to VM crash


:cry:


TODO can be removed?


FYI:
Writing 1 to /proc/pid/reclaim reclaims only file pages.
Writing 2 to /proc/pid/reclaim reclaims only anonymous pages.
Writing 3 to /proc/pid/reclaim reclaims all pages from target process.


I would consider "emulating" `input` here (e.g. `listener({type: 'input'})`).


ah, ok, let me fix the teamcity template again, since it should be working now.


This commit (ec90228) looks good to me.


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


TODO: iterate over and create Method object
```
jmethodID **methods;
jint method_count;
```


"Completly observed cases" son evaluaciones hechas. Si un elemento de una fila y columna dada es "NA", esto no se contaría. Hay una función específica para esto y es "complete.cases"


Could you please reuse already parsed scripts from `js_content` variable (https://github.com/forslund/pycaching/blob/55e634f5bfe42e60f43c8cb1b3e808cc2495baab/pycaching/cache.py#L625)?


Todavía no he llegado a frijoles_negros_para_impresionar.md


How about changing 0 to '0'? Stupid idea?
It helps me to understand how getNthPrevCFromInputFile works.


So where do we draw the line then? Some want MS.Owin - some Autofac. I don't think this will happen.


Nah, it is just a nice to have. This will make your code a little bit cleaner. Leave it like it is right now


Se te fue el  `ccafs_pr`


@johanneswuerbach This looks fishy. Did you mean `return !item.passed || item.failed;` or do  you really want to keep all items in case the `result.failed` is true?!


This change should be reverted. The method is to be used outside of the class. It would be better to remove it completely, but I don't see what harm it is doing in a test class.


Ya mismo lo corrijo!


I just realized that we are not enforcing 2.6 compatibility. Once this PR is merged, I will create another one to fix the 2.7-isms.


Humm missed =)


Would be better to just delete this class if it's not being used. This intellij optimization is just weird.


I don't think `extend_from_slice` applies to `&str`.


^^The above would be in addition to your change.


I'll reattach the correct template in a second.


This is incorrect.  Just because ``dirtyBits`` has DIRTY_BIT_READONLY set *does not* mean that we will necessarily call ``setReadOnly()`` to reset.  However, your code change will always report that we reset read-only.  We only set ``resetBits`` to indicate what was *actually* reset, and then log that.


Something is missing here in the format string ?


Где потерялся элемент с картинкой Welcome to the gorgeous Sedona?


Inserting before the end is the same, you can just always use the code in the else case below.
(Verified that it works correctly for std::list)

You don't really have to change this though.


Please revert this back to public.


commited and added all wordpress files + installed wordpress + added woocommerce + added vagrantfile + added


This one should also remain public


This method should also remain as public.


Please, use getting a reference in case of usage iterators.
It look really cool:
```
ValueType value = (*it).second;
value.a = ...;
value.b(...);
if(value.c.valid()) ...\
```


shouldn't return `false` here ?


Good catch! Mind sending a PR?


Думаю, что это тянет на заголовок страницы.
И опять пересмотри все спаны в разметке, уместно ли их применение.


could be changed to `'\0'` if you want, sure (or `'\n'` or pretty much anything but alphanumeric characters).
`0` is the same as `'\0'` and I'm used to that, but as it's indeed supposed to be a character I can very well use the character escape.


@Slaals I get a syntax error:

    [Mon Jan 25 14:22:38 2016] [error] [client 127.0.0.1] PHP Parse error:  syntax error, unexpected T_STRING in /var/www/steatite/metadata.php(39) : eval()'d code on line 1

Did you have to change something to your `php.ini` to get it working?


Petit problème dans ce commit <a....></a>


will fix! thanks for the pointer!


IT's good practice to use the bag case at first.
It's hard to scroll 50 lines to what going on.
```
if(err1) {
  warn();
  return bad_err1;
}
if(err2) {
 ...
}
common_logics();
```
It also help to avoid the followng case:
```
if(err1) {
  if(err2) {
    if(err3) [
      if(err4) {
         ...
}
warn2();
}
warn1()
}


Hmmm. Not sure about this one. I would chose to either just remove the method and use the class attribute `component` directly inside this class instead of the getter, OR (if the `component` attribute is meant to be accessible outside of the `ComponentViewModel` just revert the change so the method is once again public. The reason for the hesitation is that I'm not sure what the original intention was, but I suspect the getter and setter for Component should both be public. For some reason Intellij left the setComponent method even though it isn't used.


true! will fix!


Hmm, do you have local changes on your machine (uploaded to the server) which hasn't been pushed to the public repo?


Ah, você tem razão, nesses casos a gente usa o [Array.filter](http://yuilibrary.com/yui/docs/api/classes/Array.html#method_filter) do YUI, mas pelo visto dessa vez esqueci e usei o filter diretamente. Vou atualizar, valeu por avisar!


흠.. 그렇다면 숨기는 것이 낫겠네요..


Shall we just delete this class if it's not being used anywhere instead of making it private?


delete image


Done in 4da7d96933cbf0521a7098b78208cabab6bec668


this is apache doc, should either be [official php doc](http://php.net/manual/en/install.fpm.php), and/or official nginx doc: https://www.nginx.com/resources/wiki/start/topics/examples/phpfcgi/


put this field in the end to avoid a hole in the struct


Will add that sentence!


Please revert all these changes in `FileUploadUtils`


> 基本演算ごときに例外をつかう

例えば無限の値を扱える整数クラスとかはメモリの確保を必要とするため例外を投げざるを得ないです。
~~はたしてそれとpoint_cを組み合わせて使う人がいるのかという話はありますが。~~


I mean the [st terminal](st.suckless.org), there can be a patch for it, I am not sure if the base16 theme is done.


done


Please make these three methods in CustomDateFormatter public again.


pussyole


Top-level .gitignore wildcards without file extensions are particularly risky.


SDNN


Isn't this commit against CM contributing policy, as per the copyright header change?


Alex... tog du bort "Lägg till" knappen? :facepunch:


I think that this class and the `GeneralConfigUpdater` don't seem to be used. This particular change is OK, but I would suggest the better solution would be to delete both. @danfuterman not sure what your opinion is on this one?


there was no need of ccfras variable


The iframe onload function will be triggered out of a digest cycle.
For consistency, I chose `setTimeout`.


This is unreadible code. Please introduce local named variables.
As a rule of thumb I never have multiple []-operations on the same thing in one line - I always introduce named variables so that it is clear for example that lines[....] is a line and that line[...] is an endpoint of the line (??).


@bpasero makes sense, I'll introduce the compositePart.css


`;` missing here...


else is redundant


I actually think people should rebase their commits before posting a PR, so that they notice merge conflicts early and deal with them. Of course, they should also test their stuff after rebasing...

But given that "current git master" is implies a race condition I will change it to "very recent git master".


This should not be here I think


Add Mono Support


Hi @tabfugnic, sorry I missed this earlier. We were intentionally keeping this version at 2.2.x to maintain compatibility with the latest version available of Traveling Ruby: https://github.com/phusion/traveling-ruby/blob/master/RUBY_VERSIONS.txt, which is still only at 2.2.2.


Ok.


avoid mixing infix with dot-notation function calls, we use the other style on L61-63


.map(doc2source) should work


Can you explain what kind of read you are talking about? Or where it is used?


Please, use `const` in case we are not going to change a variable.


Well, I wanted to give another try instead of just giving up.
However we can enter in a infinite loop if I keep returning true.
I think I should return false indeed.


The `logWithFormat` is already including the location and function name, no need to including it on the message as well


Hi there, may I ask you why deauth() is called two times (one by esp timer and one in loop) ?
It randomly crashes execution with wdt reset.
Thanks in advance


shouldn't be a `continue` here (for the `EAGAIN` case)


@benel That's weird. I didn't change anything in php.ini.

I've just tested and it's working, could it be caused by PHP version?


@keszybz you are aware this reverts your own patch e5d7bce12d95b38e592a2561b49a42d033cc82c1?


Interaction hud now only looks nice on doors, you should try all basewars entities to see what i mean.


Hi Clement,

I have a question regarding the name of this file.
Since you actually create DD4hep's Tube, and also it's later translated to Geant's G4Tubs, can this be called a SimpleTube for the consistency? What do you think?


You could go a little further and define a .privateMailboxesForCurrentUser() which takes the username from session if defined, no?


might be nicer to do `def withOptionalId(id: Option[String])` in an `implicit class IndexDefinition`. Then it would be:

```
elastic4sindex.
    .into(indexName -> doctype)
    .doc(in)
    .withOptionalId(id)
```


Это можно оформить одной картинкой, но она должна быть выше по коду.
Если останется время и желание, попробуем оформить с помощью svg отдельными элементами.


Done => #750


nice one


Hmmm. Maybe. I wanted to give the control back to soletta.
If we have more that this callback will be called again.


Not clear - is it secnods, milli, nano?


ok sory but emuCR.com said decaf is avaible


Please rename `orientation` to `orientation_idx` and introduce a local variable `orientation = shuffle[0]->orientations[orientation_idx]`.

The same holds for the functions `tryInsertFirst`, `tryInsertLast` and `tryInsertMiddle`. (these are not yet present in this commit).


@NixOS/nixpkgs-committers ping


Especially in those functions the code will become a lot more readable.


yesssssssssssssss


fixed above by defaulting `$KUBELET_BIND_HOST`


sure


new list is superset of old list


fixed above by defaulting `$API_BIND_HOST`


functionally equivalent due to the envar setup


instead of logging again, i think we want a bool.  otherwise for the create and the truncate cases (see above) we'll log the fnode twice, which is a waste


No need to check count == 1. We only get to this place when it is 1...


the break here will make call `sol_platform_inform_locale_changed` and i'm afraid this is not what you want.


TL;DR: use `version` attribute besides `name` on derivations for future compatibility.


Где форма поиска?


@joecaintic DWMR. This is good. Have you decided on what you work on after the ng-intro project you're almost done with?


these are needed for now for the legacy transaction encoding.. let's drop this commit


I am easily confused by Maven. But shouldn't this be `neo4j-management`?


Indeed.


For the Leaflet HTML part, can you use the slightly-different code from our first assignment?

```
    <!--left panel-->
    <div style="position: absolute; left: 0px; width: 400px; top: 0; bottom: 0;"></div>
    <!--map-->
    <div id="map" style="position: absolute; right: 0; left: 400px; top: 0; bottom: 0;"></div>
```


:unlock:


The dout(30) is a hidden if (should_log(..)) {, so the second block will only execute when logging is turned way way up.  I think we should drop this commit.


having problem in uploading my session 3 assignment

On Mon, Jan 25, 2016 at 7:20 PM, Ishu Mishra <ishu.mishra97@gmail.com>
wrote:

>
>
> On Mon, Jan 25, 2016 at 4:24 PM, Ishu Mishra <ishu.mishra97@gmail.com>
> wrote:
>
>> actually respected sir i have written there bottom of page that please do
>> correct this i am not getting this problem.
>>
>> On Mon, Jan 25, 2016 at 3:08 PM, AcadGildProgrammers <
>> notifications@github.com> wrote:
>>
>>> ishu4.java is incomplete. Please complete it.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/coderishu/java-assignment2/commit/13d8c1103d71ae8239348ce3aedc8620a3b2dae1#commitcomment-15644919>
>>> .
>>>
>>
>>
>


Please, let's try to minimize indentation of blocks here, and merge these two if checks into one:

```
if (c == '@' && warn_rootfs) {
…
```


Counting Game


TeamCity pull requests :: netty [Build 1103](https://garage.netty.io/teamcity/viewLog.html?buildId=5044&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6389, ignored: 23 Build time: 00:43:16


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


Could be the `state` value which is an int


hmmm  I see it too, i thought that was normal, you can see commits from all repos that is forked from your own.

 I'll do a pull request and see if that resolve the problem..?!!?


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

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


problem in creating layout design on android studio as i am quite good
working on eclipse.

On Mon, Jan 25, 2016 at 7:22 PM, Ishu Mishra <ishu.mishra97@gmail.com>
wrote:

>
>
> On Mon, Jan 25, 2016 at 7:20 PM, Ishu Mishra <ishu.mishra97@gmail.com>
> wrote:
>
>> having problem in uploading my session 3 assignment
>>
>> On Mon, Jan 25, 2016 at 7:20 PM, Ishu Mishra <ishu.mishra97@gmail.com>
>> wrote:
>>
>>>
>>>
>>> On Mon, Jan 25, 2016 at 4:24 PM, Ishu Mishra <ishu.mishra97@gmail.com>
>>> wrote:
>>>
>>>> actually respected sir i have written there bottom of page that please
>>>> do correct this i am not getting this problem.
>>>>
>>>> On Mon, Jan 25, 2016 at 3:08 PM, AcadGildProgrammers <
>>>> notifications@github.com> wrote:
>>>>
>>>>> ishu4.java is incomplete. Please complete it.
>>>>>
>>>>> —
>>>>> Reply to this email directly or view it on GitHub
>>>>> <https://github.com/coderishu/java-assignment2/commit/13d8c1103d71ae8239348ce3aedc8620a3b2dae1#commitcomment-15644919>
>>>>> .
>>>>>
>>>>
>>>>
>>>
>>
>


nice abstraction


a) `isinstance(x, list)`
b) `len(x) == 1`
c) `len(x[0]) > 0`


Even though procfs_file_alloca() plus file_same() isn't too much code of its own, I think I'd prefer if this check would move into a generic function in src/basic/process-util.c, that's called "pid_from_same_root_fs()" or so, that returns an int, that is either negative on error, or 1 if the process is from the same root fs or 0 if not... I think this would make the code more readable...


i think this should go below, by io_since_flush.set(1).


`# test_command_config.rb`


Begrijp ik correct dat _ALLEEN_ de volgende wijzigingen hebben plaatsgevonden in deze commit:
* level0.lvl en test.lvl zijn verwijderd
* `MAX_SCALE_FACTOR` (incl gebruik) is vervangen door `MAX_COMPRESSION_FACTOR` en `MAX_EXPANSION_FACTOR`?


Assignment looks good! My only additional change is that the formatting is challenging to read. I would add additional line breaks and more clear headings/structure. Take a look at my recent PR to see how I am approaching format and let me know what you think.


I can add `MailboxQuery.builder(session)` that takes the users, but mixing with mailbox selection add an unrelated choice that is IMO better served by `.privateUserMailboxes()` without argument. WDYT ?


This should be fine; have you tried instantiating an object with a mock `application` and `request` parameter?


Hello Sir,
I download udemydl windows latest version. but when i try open it. its automatic close.. i give you screen shoot link: https://gyazo.com/bb49e5b205b6b564cfe6b55012b5e078
How do i use it please help me..


Are you still working on this?  If not: can you document what is missing here?


very elaborate config :-)


ss


Hmm, I think this should say: `"Process " PID_FMT " (%s) has been been marked to be excluded from killing. It is running from the root file system, and thus likely to block re-mounting of the root file system to read-only. Please consider moving it into an initrd file system instead."` or something along these lines.


</p>


1000. commit


Hmm, I do think this will never hit anyone without a IDN domain... So remove nasty in title... :)


Not. Currently is an integer. Is the id of guide persisted in atheneum.
Perhaps we should change that

El lun., ene. 25, 2016 10:14, Federico Scarpa <notifications@github.com>
escribió:

> The guide id is a string with numbers and letters, is'n it
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mumuki/mumuki-classroom/commit/96a3246f41fc0c51a598114e62654e23f05cc494#commitcomment-15649109>
> .
>


展示列表


Sicher. In unseren Breitengraden sind jedoch die meisten Leute gewöhnt dass links der Montag steht und rechts der Sonntag.
Wir könnten eine Option draus machen. Oder du kannst es auch einfach reverten.


Not sure to understand your answer. The current user is already in session.getUser().getUsername(). So my point was exactly yo add a .privateMailboxesForCurrentUser() method without argument.


I was wrong. thank you.


ssss


or just call the common harness?  `test_harness_common(tests, id)`

It skips the show test if you don't have a `:show_pattern` key in `tests`.


Yep, thanks for the answer, I tested it in the meantime and forgot to reply here...


Why? We are shutting down anyway, there is no need to do extra cleanup


4096 is higher than 1024, which is the default max page size


You are better off with storing the subscription id in a file and read it, than creating it all the time


You are doing this without optimistic concurrency, without checking if the room exists on the server side.
This will likely overwrite the room


When gun process crashes, than shotgun crashes too. Maybe there should be a try-catch clause in shotgun init function? Than, I think, this ``after`` clause can be removed.
BTW Thanks for comment !


Ok, I think we agree, my point is `ForCurrentUser` is not a good choice, it's not obvious what is a *currentUser* for a builder.


@Sunilkumarr correct spelling mDisplayName


ouais je vérifierai, effectivement vu que c'est dans un autre module y'a moyen que ce soit undefined...


explained personally. You were right here :)


with 3.2 patch?


谷歌开发者工具中的这些样式是浏览器默认样式？
```
input, textarea, keygen, select, button {
    text-rendering: auto;
    color: initial;
    letter-spacing: normal;
    word-spacing: normal;
    text-transform: none;
    text-indent: 0px;
    text-shadow: none;
    display: inline-block;
    text-align: start;
    margin: 0em 0em 0em 0em;
    font: 11px BlinkMacSystemFont;
}
```


This should run only when `sudo` is needed, or it should at least not fail when `sudo` is not present.


Consider also `EINTR` here


yes with 3.2 patch


Do the fsck task hunks belong in this commit?

Also, in `resize_device`, it looks like we never resize the luks format -- how does that work?


This doesn't work with Deepin! When I remove desktop declaration, autologin doesn't work...


Prepare new commit


如果是默认样式的话，那默认就是`display: inline-block;`


I think the second replacement is required, at least on TC2, to have a channel name without spaces on it.
Fir example, on TC2 I'm getting these channels enumerated:

```sh
01:56:20  DEBUG   :    EnergyMeter - Enabling channels {'kinds': ['energy'], 'sites': ['A7 Jcore', 'A15 Jcore']}
01:56:20  INFO    :    EnergyMeter - Channels selected for energy sampling:
[CHAN(vexpress_energy/energy1, A7 Jcore_energy), CHAN(vexpress_energy/energy1, A15 Jcore_energy)]
```

The second replacement is requited to translate: " A7 Jcore_energy" into " A7_Jcore".
Note sure that the space in the name could produce some errors, but I think to remember I've added the second replace cause I've got an error at certain point when using labels with spaces in between.

Can you please check that in Juno we get a label name without spaces?
In that case, it it's not a big issue for Juno, I would suggest to remove that patch from the branch to merge.


There is no reason to use finalizer here, you don't have anything unmanaged.


Shouldn't this block be indented so that it is only run if `device.resizable` is `True`?


Refer to our [JavaScript Style Guide](https://imqssoftware.atlassian.net/wiki/display/AR/JavaScript+Style+Guide) regarding comment formatting. Also, note the continuous blank lines below.


asasas


TeamCity Gitversioner :: Build [Build 11](https://tc.phome.cz/viewLog.html?buildId=31&buildTypeId=Gitversioner_Build) is now running


bla bla comment


Same answer like there: https://github.com/PolymerLabs/crisper/issues/30#issuecomment-174517327


I have read your readme file. And i am trying to run it but it has some errors.Tell me the steps that i should follow.I also installed libsvm 3.20.


Hi Ana,

good point, I could change that when re-shuffling the code later this week.

Cheers,
Clement


TeamCity Gitversioner :: Build [Build master-1.0.1-5-ge6169c9](https://tc.phome.cz/viewLog.html?buildId=31&buildTypeId=Gitversioner_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:42


TeamCity Gitversioner :: Build [Build 12](https://tc.phome.cz/viewLog.html?buildId=32&buildTypeId=Gitversioner_Build) is now running


sam jestes first try


@IggHub DWMR. Checkpoint #21 is good. Press ahead.


1. Can you describe in comments what each test is doing  ?

2. Can you also guard code such as logger.info and logger.debug with logger.isInfoEnabled or logger.isDebugEnabled as this could become a perf hit ?


TeamCity Gitversioner :: Build [Build master-1.0.1-5-ge6169c9](https://tc.phome.cz/viewLog.html?buildId=32&buildTypeId=Gitversioner_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:15


@Sunilkumarr check the correct sentence, need or needed?


nice changes


cool!


Any chance of getting this thing fully open sourced, since "It's dead jim"?


I'm cool with adding this as it is but a couple of comments.

I think you should add your name to the copyright. *You* did the work.  Add your name to THANKS.

Be warned that due to technical difficulties, unless MELPA has been improved, the new debugger won't  be seen until one of those folks makes a change.

I think the approach where you make this a separate github repo doesn't have that problem, I think.

That said, if you don't want to modify the commits and just go with this, let me know.

If you don't want to make a separate repo, but instead would prefer to be a collaborator in this repo, also let me know.

And most importantly, thanks for undertaking this.


other nice changes


@Sunilkumarr lowercase shouldShow...


missed 'set bit after method executed'. reverted and committed change


Do you think test-writers need to override the `sudo_cmd`? I think if they already knew they need a special handling, they should call the full command using `cmd`. If they just want it to be executed as superuser, then they should just set `sudo=True` flag. I would like to avoid over-complicate API...


fix


@Sunilkumarr how about only this:
return ActivityCompat.shouldShowRequestPermissionRationale(activity,
                 Manifest.permission.WRITE_EXTERNAL_STORAGE)
                || ActivityCompat.shouldShowRequestPermissionRationale(activity,
                Manifest.permission.READ_EXTERNAL_STORAGE);


Fixes #4


This seems to break a few of our add-ons. When have a parameter where we expect `UserInfo`. That's not compatible anymore and thus causes an error..


test


Merge CI


Doesn't this break using print() in Python 2.x?


reverted already see https://github.com/nitincchauhan/HikariCP/commit/5f851a4cfb823e639ec9e776e715367fe03bf2b8#commitcomment-15396819


awesome :)


Hmmm. The only change I made was removing the ::. I uploaded the
change to the server. Do I have to push them to the public? How do I
do that?

David

Quoting Derek Braid <notifications@github.com>:

> Hmm, do you have local changes on your machine (uploaded to the
> server) which hasn't been pushed to the public repo?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/DeBraid/puck/commit/de1909d3ae22fdb1bf805fae188504f44e1c6632#commitcomment-15649408


Type hint against \Concrete\Core\User\UserInfo?


grrr :( ...reverted too and committed.


Since lastRecordId and firstRecordId remains constant for each iteration, one might want to cache this calculation. The performance hit would be minimal (if existant) doing the calculation as you are doing, but it remains good practice to avoid unnecessary repetition of instruction cycles.


Fix FRENCH translations +
Added missing translations


1. Use `title[/ethernet/i]` since the other `when` statements use that syntax.

2. This reminds me that we still need a `puppet resource` technique to clean up ethernets, since we can't use 'ensure=absent' on them. Maybe we could do this:

```
def interface_default_cleanup(agent, intf, stepinfo='interface clean')
  step "TestStep :: #{stepinfo}" do
    cmd = get_namespace_cmd(agent, PUPPET_BINPATH +
      "resource cisco_command_config command='default interface #{intf}'"
    logger.info("  * #{stepinfo} Setting interface '#{title}' to default state")
    on(agent, cmd_str, acceptable_exit_codes: [0])
  end
end

```


@adrianq why is this needed?


in python, there's nothing wrong with calling this defaultTimeout (the same as the named arg you call shortly thereafter


@Sunilkumarr Dispaly -> Display


yup, this is fine


meanfabsSD and stdSD - what is it? I don't find these features in papers. Are they really useful?


posixpath is used intentionally even on Windows, due to a bug in ntpath (which serves as os.path on Windows):
os.path.join("C:", "example.txt.vvv") == "C:example.txt.vvv" instead of the expected "C:\example.txt.vvv", which will then be expanded using cmd.exe to "<current directory>\example.txt.vvv" because of some stupid MS-DOS compatibility hack. So running "python teslacrack.py C:" will appear to decrypt the entire C: drive, but in reality will be confined to the directory you started teslacrack.py from.


very good


IIRC previous version avoided `sudo` in case uid was `0`. I can't find it here. I liked that, it'd avoid the need to tune the `sudoers` when running jobs as root.


It seems that `-php` argument is a recent feature that is not available on Ubuntu 12.04 package.
I am going to refactor the code to use [exif-read-data](http://php.net/manual/fr/function.exif-read-data.php) instead.


Good catch. Thank you.


Greet commit I have been thinking of this for a while.
Now If the jasper reports were updated for this new column, even better.


112331505_SurfaceHeightRange (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2050 of 2050 tests passed, 0 test warnings)

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


我总是在最深的绝望里遇见最美丽的惊喜


I think this is changing behavior here.
Before in the MONODROID case, stdout/stderr were initialized to a UnexceptionalStreamWriter before being passed to LogcatTextWriter.

Now they're not initialized at this point from what I can see.


Use compass helpers. xt


@dtenenba Just a suggestion, as Jim H. pointed out in [this comment](https://github.com/b-long/BBS/commit/4066b9e3dc67a4da7b74fed45fa3c13933fbb44d#commitcomment-14874093) it's best to prefer `str.format()` over `'foo %s' % bar` substitution.


Use compass helpers


features.append(['outliers20', np.sum(np.fabs(diffs - 1) > 0.2) / float(diffs.shape[0])]) - it is more optimal?


"RSA" is important here, as only RSA uses a semiprime modulus. (Moduli used in other areas of mathematics are typically either primes or prime powers.)


TeamCity Torrentizer :: Build [Build 2](https://tc.phome.cz/viewLog.html?buildId=35&buildTypeId=Torrentizer_Build&guest=1) is now running


TeamCity Torrentizer :: Build [Build master-0.9.0-27-g7c0b96b](https://tc.phome.cz/viewLog.html?buildId=35&buildTypeId=Torrentizer_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:52


I'll ask the person reporting it to join this discussion, as I don't really know exactly what he did...


I think it should have more generic name. It shouldn't contain context in which the class is being used and if it does I would try style it in the feed.


You may want to show a suitable message on failure of an AJAX request - in this case, the previously populated `this.records` will be drawn again


这个方法好像没什么用


We might best start by support on the nix side, i.e. it should only try to parse version out of the name if the version attribute isn't defined.


:+1:  looks good to me


Can this go into the MSE player?


1. ScriptingManager has some logic around task invocation. Should the same logic be applied here?
2. Should I add specific methods/copy the structure of ScriptRunners for all the cases they support?


Please use the same format as org.infinispan.configuration.cache.SecurityActions


This change causes https scheme detection in Server#getURI to incorrectly identify https connections as simply http (getURI looks for the "SSL-" prefix).


@tarelli I tried by mistake to load a model with the old format (previous to ecore) and no error was displayed, just the load spinner.


异步设置为false，不然数据获取不了


I do not tolerate including sources from outside. Please copy them into the repository. Otherwise the game will not work offline. If you copy them, make sure they comply with the license and get proper attribution.


Interesting project.
It is not really usable without a license though... Which one have you chosen?


Please put the CSS into a new file.


Will this work also in Python 2.7? Looks like Python 3-specific features to me.


I'll rollback whitespaces changes in the file in the final version


I would prefer to use the a list of the constants her to not confuse beginners eg.
```python
for int_time in ["TSL2591_INTEGRATIONTIME_100MS", "TSL2591_INTEGRATIONTIME_200MS" ...]
```


same here


怎么转换数据，好头痛


Did you create these images? If not, we may run into legal problems when putting them under a creative commons license. Please prove, you created them.


Also there should be realization of 'NN' + str(bound)


This is a fix for RT4149.


it's cool


Yes, `iter()` it [works also in python-2](https://docs.python.org/2/library/functions.html#iter),
although I don't remember if i tested this specific commit under py-2 (in linux).

But never mind, I have canceled the unslurping because it corrupted decrypted-files when Ctrl-C the job.


112331505_SurfaceHeightRange (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

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


* Cache results, screen capture as 2 appendix of interesting cases

* Page for guide, step by step introduction

* Provide more detailed information in the email

* Default value initially on the map

* ~~Spectrum efficiency~~


The above 3 statements are often duplicated - maybe have your Grid class manage record deletion as well as all html manipulations.


이분탐색


But the important stuff is read+ decrypt, and then write-file,
because as it is currently, it also corrupts the decrypted files (they are empty) if abrupt-interrupted.


112331505_SurfaceHeightRange (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2088 tests passed, 0 test warnings)

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


Děkuji, upravím


Dík, upravím


Actually, this commit is related with issue #1.


Shall we add EXIF and IU manifest files as well over here?


not used


I think the example value is misleading. Could you please a more realistic one, e.g. `/r/example/`?


Поведение по умолчанию ещё хорошо бы затестировать.


This is potentially dangerous, in case an encrypted file has since been replaced by some other file (e.g. Firefox prefs.js will be regenerated on startup if not found, this will unconditionally overwrite the recreated file with the old one). At the very least, backup the file to be overwritten.


reviewed


I am not actually sure. That is a good question. It should be easy to check.


meanmeanA -> SDA


dík, upravím


> I can remove this commit, if the maintainers don't want to merge it, though.

I am perfectly happy with this commit getting merged in. It just didn't seem to make sense in this PR to me.


This is the reason. `Typeable m` is an alternate spelling of `KnownVariant ('DUnit m)`.


Will do it (in the afternoon :-)


Jelikož to celkově funguje úplně jinak, tak bych se na zpětnou kompabilitu vykašlal a vydal pak novou major verzi bez deprecated metod..


Feels dirty being unable to mix optionals and streams elegantly, we may add a stream helper library.
Then we could "lift" optional to streams and use flatmap instead of the weird filter + map.
eg.
.flatMap(o -> o.isPresent() ? Stream.of(o.get()) : Stream.empty()).
I've already stumbled upon similar pattern in other part of our code (though cannot remember exactly where!).
StreamsHelper, StreamsUtils, ... ?


You used tabs instead of spaces (here and in some other places). Please fix.


Already caught by Preconditions.checkState(base != null || username != null);
no?


Any reason not to call `string:to_upper(A)` here?


UNIX nomenclature is to use --dry-run for an option like this, -n sounds like it stands for "numeric".


I don't think there is one for this, especially not with the platform-specific behavior.  Most of those APIs are meant to work with CMake's internal forward-slash path representation.


This should be fixed in all instances of Adhocracy.ts, not just here.


This is not correct. The language should have access only to arguments for itself; not to arguments for other languages.


I should add that it might be possible to eliminate this, (possibly with a performance boost?) if you de-associated the `Dimensional` data family from the `KnownVariant` class. Not taking a position on whether that's a good idea or not.


@americool Happy to see you thinking about the user experience here. You'll do far more UI/UX thinking when you work on rails.


can we order the resources in this file so they are together? i.e.  '/conferences/{accesscode}':


TIL - didn't know POSITIVE_INFINITY was a thing.


more robust way:
if os.name == 'nt'

I've seen Python 2.7 installations where platform.system() didn't exist.


Should the callback run if fail() is called? If not, we should return here.


something like this for example:
```php
namespace Concrete\Package\MeschUserManagement\Service;

use UserInfo;

class User
{
    /** @var UserInfo */
    private $ui;

    public function __construct(UserInfo $ui = null)
    {
        $this->ui = $ui;
    }
```


If I understand you correctly, that is true.
The env gets populated only with the arguments from the registered mime types, see: https://github.com/graalvm/truffle/compare/master...smarr:feature/add-arguments-to-polyglot-engine#diff-f26317b7976a72f565cbf562059cc574R836


in Murukesan14
bounds = [50]
step = [60000]
and
step_for_hr = [600000]


Oh, my bad, I just noticed the commit message on the last one. I guess it's all camel-case all the time.


i am compiling the 3.2 kernel with 3.2 patch.


A copyright *notice* must include the **first year of publication**.
> U.S. law [no longer](http://www.copyright.gov/circs/circ03.pdf) requires the use of a copyright *notice*

[]()

> The Copyright Office [also says that](http://www.contentious.com/2007/01/07/copyright-notice-is-the-year-really-necessary) any work of intellectual property is **automatically considered copyrighted from the moment it is created.**

So the year is not required at all.


The description is a bit confusing to me. Maybe something like "This is the path you get redirected to when visiting /. This is especially relevant for cases in which cameFrom is not set, e.g. account activation."


I think this is a configuration error: If I write `1277.0.0.1` in the config , this should be the case, right?  So for now I'd say some reasonable pick from base's exception hierarchy should be thrown?


![mind blown](http://media.giphy.com/media/EldfH1VJdbrwY/giphy.gif)


@JohnnyR1co не стоит светить ключи в открытом доступе


This commit prevents using TAPi18n setting ROOT_URL like http://domain.com/subfolder
I do not have the knowledge to understand why only in this point is present the condition isCordova, but maybe we can change it a bit to permit case like mine


In fact it is privateUserMailboxes which throw, so you can rename this method and remove the .build() in the next line.


Extract to separate file.


yes. agree


I see that was done with the 9300-RADES and 9300-8EDM devices, but this commit also removed four rows of GE-IP devices, starting at line 48


Choose better name. Should say what it does.


@americool You might force users to type in "DELETE EVERYTHING" before the final execution of the data. Might even want to consider marking the records for later deletion - just in case...


Yep, I was waiting for this commit to be tagged haha :) Saves 20 lines of code on our end :)


This is a comment.   Testing 1,2,3


You need to update the doc (`dev/time_event.rst`) to reflect those changes.


typo: "propmpt"


Also, what about `localhost`?  Or host names only available from dns?  (We didn't discuss this and it didn't work before, but it would be nice if it would.)


Any reason not to call `periodic_call` for robots?


this stroke should be commented


It seems that table numbers are assigned manually?
I hope it could be automated, and I don't get two tables far apart when I have two projects.


Why do you change EventDateUtc from DateTimeOffset format to DateTime format ?
And those who do not use MySQL ?
We have the choice : modify our production database or not upgrading.
It is a good work


Actually, more than just variables. Added code to populate new field added to bioscale models tables in db.


If it is required then I have no problem with it. My thought was that if it wasn't strictly required it would have been better to remove the constraint from the manual for clarity.

> On 25 jan. 2016, at 09:31, Douglas McClean <notifications@github.com> wrote:
>
> I should add that it might be possible to eliminate this, (possibly with a performance boost?) if you de-associated the Dimensional data family from the KnownVariant class. Not taking a position on whether that's a good idea or not.
>
> —
> Reply to this email directly or view it on GitHub.
>


Typo: "Anf"

Also, almost surely admin rights will be required when working from a drive root, unless you're running on XP. Vista and up has tightened permissions on drive roots.


Apache approach: http://www.apache.org/legal/src-headers.html

They have a single file that contains the copyright statements in the base directory and do maintain the years.


definitely a good idea,
although I would call this function toHoursBasis(...)
(additional note: adding this function increases the code size by 8 byte,
but it's definitely worth it)


done.


아하 고맙습니당!


You can write a common ajax function that does the bulky setup of an XML http request (to avoid doing this too many times): A simple Get function that takes a url and callback function as parameters. JQuery provides a similar simplification of ajax requests and it might be useful to try both.


Hmm, the reason I changed it was because `nextProps.items.length` was a loose check, and the currentIndex would reset every time the component receives new props. One way we can solve this is maybe instead of using `componentWillReceiveProps`, you can just call `slideToIndex` when you change your `items`. What do you think?


don't find two last features in papers


Wir lassen diesen heretischen Commit :)


Where is RMSSD?


additional supported extensions: '.zzz', '.abc', '.xyz', '.aaa'


I probably miss something here, but when `must_call` is false, I don't see how you increment `_nb_call` => `must_call` is not going to become true.


I had changed this from

    else if (program.args[0] instanceof program.Command) {


I'm not 100%, but that was the first fix i did to achieve unicode support.
If it works as you described then definetely i have to revisit this point.

By the way, I have built a number of test infected files to check the decryptor, that i can push.


Looks like a good fix to me.


I don't think so, if base is not null and username is null, you pass precondition and this test but if both are != null it throws.


typo: "ransomeware"


 +Para agregar Y/O registrar cambios:
+	git add <filename>
+	git add .
+
+Para hacer commit o etiquetar  cambios:
+	git commit -m "Commit message"
+
+Para enviar estos cambios al repositorio remoto:
+	git push origin master
+
+Para actualizar repositorio local al commit más nuevo:
+	git pull
+
+Para revisar lo añadido al repositorio podemos ejecutar el siguiente comando:
+	git status


@methodik I've made all the changes recommended.


So use the copy of the device that is returned by the BasicTests function instead of getting it a second time then mucking with the community, a second time.


nop, you looks tired (:


Session should be final :-)


Only '.vvv' is "v8" or "2.2.0", '.ccc' is "v7" or "2.1.0" with a slightly different key scheduling (doesn't affect decryption using TeslaCrack, as only the Bitcoin keys are handled differently).


I do not understand this snippet of code. `_nb_call` is the number of time the component has been called since what? why do you reset it to 1 when `_nb_call == `_frequency`? Some comments would be helpful to better understand the logic.


This line is needed for the `\\?\` trick - does not work with bytes!

In general print does not have problem with either bytes or u_strings.
In cygwin-py2 that i just checked, no prob.


Is there a nice way to move parsing of the `allowed_ips` config field to `Thentos.Config`?  That would make errors occur at start time.


ooops, ok, need a coffee.


Well, probably not. Last time I checked 1) you need to register and login to download the code (or send them your public key for git access). 2) their cmake script is still a bit of a hack. It is autogenerated and doesn't allow for much configuration at all.


Any reasons for `_time_before_next_call` instead of the simpler `_component_period`?


hm... I'm not sure why I wrote the code this way,
but for optimization reasons It should be:
 retu /= 1000000/TIMER_INTERRUPT_PERIOD_MICROSECONDS
             /3600/TIMER_SLOW_INTERRUPT_INTERVAL;


THD::set_time() is also called by THD::init_for_queries() below, so this one seem redundant.


yep :/


ouch!


Nice ;-)


"wait until _server_ is fully initialized" ?


Maybe worth explaining that this is where the flaw lies, as the ECDH shared secret is simply multiplied with the AES key instead of using a proper cipher. TeslaCrypt 3 ('.xxx', '.ttt' and '.micro') switched to AESing the session key with the shared secret instead, which is why this tool can no longer decrypt it.


You need to remove those 2 lines


Couldn't the above two be combined into one?
````java
ByteBuf[] nativeBuffers = Arrays.stream(buffers)
		.map(b -> ((NettyDataBuffer) b).getNativeBuffer())
		.toArray(ByteBuf[]::new);
````


I suggest removing the logs, except in the case an exception is catched.


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


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


replace rs with row which is way more descriptive.

You can do it with alt + f6


This should have a prefix based on the database name, so that multiple instances on the same host don't end up sharing the same Elasticsearch index, e.g. in Pombola this is [done by using the database name as a prefix](https://github.com/mysociety/pombola/blob/54911eab77396cef91c558fb065dca569dd1bd98/pombola/settings/base.py#L283-L295) (though it should probably include the database host as well to be sure)


yes, definitely a better place  for the STATIC_ASSERT


> ooops, ok, need a coffee.

is like `assertThat(true).isTrue()`

just saying


la méthode parse n'est pas encore ajoutée dans l'interface. Un commit a été réalisé, l'issue scru sera corrigée lors de la prochaine alpha? https://github.com/open-orchestra/open-orchestra-bbcode-bundle/commit/d3d8ecd18a8c2b7da83e1d831ef28331168b483e


feed ~~them~~ into *an* existing factoring tool


OK! :)


You may want to move this line to the bottom of the code sample.
Btw, `set_speed` should be a method of `Environment`, shouldn't it?


It probably makes more sense, but originally it was like this -- if device is not resizable and format is, resize_device would resize the format.


TeamCity pull requests :: netty [Build 1104](https://garage.netty.io/teamcity/viewLog.html?buildId=5045&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1801, ignored: 4 Build time: 00:02:14


It seems like a `class`. It has constructor (it's `init` now), states. I prefer to create a class `TestContextStorage`.
It's used by `TestRunner` only. So, I suppose we can create an instance of `TestContextStorage` in the `TestRunner` constructor. We should add the `sesstionId` argument to it.


@mattnworb @dflemstr For some reason making HeliosClient implement `Closeable` makes the tests fail...


but in this case (to be consistent) the STATIC_ASSERT(ANALOG_AMP(1.0) == ANALOG_CHARGE(1.0));
should  go to AnalogInputs::getCharge()


pycryptodome comment is outdated here, as pip will always fetch the right version (back when I was using pycrypto instead of pycryptodome, this was important, as pycrypto binaries for Windows weren't available in the official repositories, instead requiring 3rd-party URLs to be specified)


You should not unactivate and then activate a SIEVE script as it let a window for a user without SIEVE scripts.

Start activating sieve script id needed (that means if name.equals("") ) and then unactivate the old script.

Extract such queries in submethod to put names on them, which will highly improve readability...


do we need to add the same sort of behavior for IU manifest and EXIF?


Is looping here necessary? `CompositeByteBuf` seems to have `addComponents`.


Unfotunately, there is no alias `firefox`, only `ff` (it's also mentioned [below](https://github.com/VasilyStrelyaev/testcafe/commit/bd9244feb430709922621e7c88889d64b0b42577#diff-679a3d497074db631b0e722a5db01a5eR84)). So the example will not work, it may confuse someone if they didn't read the entire docs but only code examples.


We need to better explain `fixed timestep`: does it mean 'every simulation loop takes the same amount of (real, physical) time'? or 'every simulation loop correspond to the same amount of *simulated* time (and in that case, how can I set the amount of simulated time per loop? -> `fps` is confusing: are those 'frames per physical-time seconds'? why are we talking about 'frames' when we really mean 'loop'?)


I wonder if we need `;` at the end of NULL lines like the others? Maybe you can try that and see if it works?


@amolmkhedkar eventually, yes. But I'd prefer to wait for a ticket that describes the work.


Is the program model desired schema to follow while writing code?


Do we still need this line?


hi now i try to rebuild
kernel 3.2.76
and redownload patch for 3.2 from your git :
https://raw.githubusercontent.com/gfreewind/linuximq-HighPerf/master/kernel/v3.x/linux-3.2-imq.diff
and the same error :

net/core/skbuff.c: In function 'skb_release_head_state':
net/core/skbuff.c:387:15: warning: cast from pointer to integer of
different size [-Wpointer-to-int-cast]
     "%08x\n", (unsigned int)skb->cb_next);
               ^
net/core/skbuff.c: In function '__copy_skb_header':
net/core/skbuff.c:546:2: error: implicit declaration of function
'skb_copy_stored_cb' [-Werror=implicit-function-declaration]
  skb_copy_stored_cb(new, old);
  ^
net/core/skbuff.c: In function 'skb_init':
net/core/skbuff.c:2916:14: error: invalid application of 'sizeof' to
incomplete type 'struct skb_cb_table'
       sizeof(struct skb_cb_table),
              ^
net/core/skbuff.c: At top level:
net/core/skbuff.c:3240:13: warning: conflicting types for
'skb_copy_stored_cb'
 static void skb_copy_stored_cb(struct sk_buff *new, const struct sk_buff
*__old)
             ^
net/core/skbuff.c:3240:13: error: static declaration of
'skb_copy_stored_cb' follows non-static declaration
net/core/skbuff.c:546:2: note: previous implicit declaration of
'skb_copy_stored_cb' was here
  skb_copy_stored_cb(new, old);
  ^
net/core/skbuff.c:3240:13: warning: 'skb_copy_stored_cb' defined but not
used [-Wunused-function]
 static void skb_copy_stored_cb(struct sk_buff *new, const struct sk_buff
*__old)
             ^
  CC      net/core/ethtool.o
cc1: some warnings being treated as errors
make[5]: *** [net/core/skbuff.o] Error 1
make[5]: *** Waiting for unfinished jobs....

m.

On Mon, Jan 25, 2016 at 4:34 PM, Gao Feng <notifications@github.com> wrote:

> i am compiling the 3.2 kernel with 3.2 patch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gfreewind/linuximq-HighPerf/commit/7ff578c40cebd1d640472a38c2e31133e785e830#commitcomment-15650821>
> .
>


Whoops `ApplicationL`.


Incorrect, OOXML and ODF files both have "PK" as their file magic (as they are both XML-in-ZIP formats). "ODF" as a file magic number AFAIK doesn't exist.


`logic | physics`: the use of `|` seems to mean `either the logic loop or the physics loop`. Is it what you mean? I suspect it is `the speed of the logic and physics loop`.


same comment


This method should not be 80 line long !!!!

Look at myself, I'm a limited human getting confused, and posting wrong PR comments as it gets complicated to read it !

You should extract things using ctrl + alt + m in order to make it more descriptive.

You can so put labels on your methods like :

 - activateNewSieveScript
 - deActivateOldSieveScript
 - ensureTargetDoesNotExists

The logic will be far easier to understand, don't you think ?


it was `defferedError` because it was in the context of client-server intercourse. Now we can name it `testError` or `fatalError`


would you please stop commenting the commits ! It's not a subreddit ! it's annoying we get notifications expecting comments/improvements on the code not some random stuff !


I don't think this is a good idea:
getIout() and getVout() has a uint16_t type,
so the result is also uint16_t (you are losing some data)
you could however rewrite this to:
uint32_t P = 1UL * getIout() * getVout();
but in my opinion the form:
uint32_t P = getIout();
P *= getVout();
is more convincing (the result type is clear)


Nice ! Just get the rename operation easier to read and it will be OK for me.


@zzzsochi вот здесь services вместо resources


missing space


Consider adding comments in the code section as well.


@vchirkov Please remove `debugger;`


@khoozhiyan1997 https://github.com/ide/Synchronized


move that line to the bottom. I really think that `set_speed` is a very confusing name. `speed` should be only used for 'things that move'. What about `set_simulator_frequency`? (and since `20` and `20` are the default values, I think we should omit them from this example)


ano, to jsem chtěl.. Strana Heureky tam vypisuje něco jako "Chybi email nebo API klic"


`bundle update` won't run until devise is updated to 3.5. So unless this and 4 other commits are squashed into one, we have to wait until cef1fa09b9eca65 for an updated `Gemfile.lock`


Change the name to more meaningful, for example SCryptPasswordEncoder


needs to be `logger.debug` ;-)


@vchirkov It would be nice if `angular.isUndefined` would be defined as `isUndefined = angular.isUndefined` to fit general code style.


holy high!


Can you please elaborate on moving asset files to new location where curation_concerns now expects them?
I was not able to understand what it meant.


To someone discovering the doc, the `morse_sync` tool is a mystery: "what is it? why do I need it? is it something I must launch separately? where can I download it? ...raaah! MORSE is too complex!"... Is it really useful to always force the users to explicitly invoke it? Can't we simply spawn it if the simulator main frequency is above 60Hz?


what is the 'full stack of software'?


Can you explain this in a bit more detail - what part of hubspot-connection relies on the existing interface / what fails if this isn't done?


let's add to `TestRunner` getter (or instance field) for `nextStep` and `testError` and get it from there


We don't need vendor prefixes!


Missing newline at end of file? :wink:


Can we remove the empty blank lines?


IMHO I think this scenario should become *scenario 2*


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1248.42bd35c19567f77d498eecc317728030b1b04686](http://build.palaso.org/viewLog.html?buildId=68399&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 846, ignored: 7 Build time: 00:07:08


@methodik you're looking at the wrong commit. I've removed all that 2 hours ago


@dirkx I would tend to opt for (b) as (a) might be a bit off consensus as we until now had no version members at all. @sgillies thanks a bunch for taking the ball, and kicking it into the field ;-)


I think the check for lowercased event names can be removed. The warning should never show up, thanks to this code: https://twitter.com/foomanNZ/status/689924329065164800 - configured observers will be converted to lowercase while loading. Magento installations where this warning will show, will also have no problem dispatching or observing the events in mixed case (should be warned that their installation is unpatched, but I don't think this command is responsible for doing so).


Why should the FileSet accessor name be changed?
Does it mean that this object can only consist of technical metadata?


The `Program` you see here was created using `raikls generate curation_concerns:work Program`. It creates "boiler plate" code for creating `Program` objects through the CurationConcerns UI.

A `Program` for WGBH, is an object that has descriptive metadata, and can have files attached. Looking at the [IU diagram] (https://docs.google.com/drawings/d/13Ir6VVweuVHNfLX2NhW6790TpZXU4eHRUaHyhIrKlC0/edit), you would probably want to do the same thing for the box labeled "pcdm:Object (Work)". But you would need to come up with a name for it. I'm not sure what that would be. But once you come up with it, you can generate the same code with `rails generate curation_concerns:work YOUR_MODEL_NAME_HERE`


> Do the fsck task hunks belong in this commit?

It doesn't. I made a mistake during rebase. This should go to "Move resize related methods from FS to DeviceFormat".

> Also, in `resize_device`, it looks like we never resize the luks format -- how does that work?

The luks format is resized during the luks device resize. `LUKSDevice.resize()` actually just calls `self.slave.format.do_resize()`.


Shall we add sane default to reduce the overhead of settings all required fields of MailboxQuery for recurrent queries (like get all user mailboxes) ?
Assuming for instance, that the base is stable, the path delimiter has a default value, ... We may have a call like that:
MailboxQuery.forAllMailboxesFrom(username).build()


I'm not referring to bytes vs u_strings, rather, Python 2.x uses "print whatever" instead of "print(whatever)" as the syntax for print by default, print_function enables using print as a function even in Python 2.x


This sentence: `the minimal frequency of the set_speed level` is not very clear. What is a `set_speed` level?
Then you say that the value is automatically tuned for our time factor. So do we need to set it, or is it automatic?

`time factor` sounds bizarre. `time multiplier` maybe?


same as before, should move at the bottom


you are reading an old version, we added exactly that in later commits


u r silly lose r '(


I would add: `By definition, the simulation time will [...]`


:+1: beat me to it! nicely done.


I CRIE


Balls.


Our internet isn't very good at work today and CodeMirror (or whatever GitHub uses) was lagging.


this looks strange a little. As I see the `nextStep` var is obsolete now. For top we get it from the storage, for iframes it's always `0`. So let's move it out here and define it in runner constructor (they are will be different in `Runner` and `IFrameRunner`


Nope, that's an unused variable that was left behind when we introduced the format filter.


Sorry, if it was too harsh. To be honest I don't agree that in-line comment formatting consistency is a critical part of coding conventions. If it was, we also should put up rules about where to comment and how, because that is much more critical to readability then the question of /* vs. //

If you insist, I will change it, but in return consider picking up #9


I would rephrase as: `You want a faster-than-real-time simulation (typically to accelerate a long running experiment), but you only need to read out sensor values at low frequency.`


removed


@benel `-php` tells exiftool to export metadata in a php array. It's easier to get the result in the program since it's programmed in php. There is no manipulation to do after extraction. You can find all the options [here](http://www.sno.phy.queensu.ca/~phil/exiftool/exiftool_pod.html).

If this option is an issue, it cool be possible de get rid of it, but we would have to write some code to process the returned data since it won't be readable for php (the result would be a plain text in String type).


defined as new variable


Sorry. The position of "...props" will overwrite some property before. I find this bug when I am using ant-design. Really sorry about that. I am too confident about this PR to make such a stupid mistake.

Please forgive me.


Aahh...well, all print-statements were substitued with logging-statements - so this does not apply any more.


Two hours is was it was meant to be! 20 hours might be a bit too much :)


> it's super nice to have the ability to test RiakCS on travis moving forward

nitpick: we're technically not testing Riak CS on travis. We're testing a different (moto) implementation of the S3 REST API, which may or may not be equivalent to the Riak CS implementation. For example, one difference is that Riak CS does proper authentication with `aws_access_key_id` and `aws_secret_access_key`, but moto does not.


Neat, but does the nogame even load? :P


No longer having a constructor, but a factory function instead, that will return a deferred that will fire once the instance is built


We no longer have that in the innit


Is there any WGBH specific feature in this overrided class?
Do I need to override the local copy of search builder class for including IU specific features?


I would rename `fps` in `base_simulator_frequency`.
Then the sentence becomes:
`The actual frequency of the simulator main loop is ``base_simulator_frequency * accelerated_by``. If this frequency is above 60Hz, the situation is similar to the one in Scenario 4.


Making the order in the docs and in the actual code match


True, GitHub says so...is that required? Other example scripts don't have that either.


factory method


Maybe change this to just "Acquia"?


Well it's an untested assumption for now.  I did however come up with some logic which might be close to how it's implemented in the hardware:
w=c=0, n=num;
while (n) {
  if (!w) {p += c; w=wl, c=cl;}
  if (c >= 0) unpack(p++); else fill(p++); // probably actually implemented with a flag that's set on c hitting 0 and cleared on w hitting 0
  w--, c--, n--;
}


Ahem, c > 0, not c >= 0


How about `adapters.first { it.itemViewType == viewType }.onCreateViewHolder(parent)` instead of loop, conditional and throwing exception?


Tottaly missread the old text.

Then `we should add to PK` also all `.xlsx`, '.docx` etc that are zip-archives as well.


I understood your test case, in the ExecutorServiceExecutionStrategy class, because resolveField method run into recursive calling itself for deep fetching child fields(default DFS), parent field node hold on first thread, and recursive calling child field, which got second thread, but first thread didn't give back to thread pool.


No, this English was incorrect for this file.


I am not sure I got it right. Can `exchange` have "nil" values?


My hope is that Riak CS implements a superset of the S3 API implemented by moto, so hopefully most things that work with moto will also work with Riak CS. The best way to verify this is to run tests against both, or at least Riak CS, when running tests locally.


Probably should have consistent whitespace at the start of the `map` block here.


Don't start activities from adapters.


Comma after applications


Whats your thoughts on "Turbocharge your devlpoment and testing process with Angular Multimocks"


Das ist gut. Ja.


Titre II/Chapitre Ier - Environnement ouvert.md

Amendement 704 (Luc Belot)


missing words? can be done through


This can 'be' through


Des corrections ont été apportées.
La structure du projet va bientôt être revue.
# TODO
- //VBO Preférez l'interface List plutôt que ArrayList
- Internationalisation


Comma after engineer


Sous-Amendement 873 du Gouvernement


I would move this line just before `set_time_scale` and add the following comment:
`The base frequency of the simulator is 2Hz: seen from outisde, all the sensor/actuator updates will happen at 2Hz. But we accelerate the simulated time by 20: one physical, real second is equivalent to 20 seconds in the simulated world, and the physic simulation, for instance, is going to be 20 time faster.`


This second sentence is missing something to make it make sense


If you define the lookup table from within the lookup function, wouldn't it be inlined into a big switch-case? (this way lg2_lookup[] wouldn't be a global anymore)


Может лучше возвращать ```<NOT AUTHENTIFICATED>``` ?


Thank you for this very important piece of documentation! I've made plenty of suggestion, but I do like a lot the overall document.


'is going to talk about' - probably reads better


Avoid adding unnecessary comments. If someone wants to find, who added a bug in code, they will use git blame.


'backend' - maintain consistency


An issue is here!


I don't think the string constants are really necessary in JS-land, literals are fine.


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


@SimonWoolf By "this logic" do you mean the logic that decides whether it needs to do a server request for the time offset? If so, I don't see how `getTimestamp()` can do this without becoming asynchronous itself.

I understand why my previous solution with `Rest.prototype.serverTimeOffset` was not a good one, but the way I'd solve this would be to put the logic in `Auth.prototype.authorise()`. Then if `serverTimeOffset === null` I would call `Rest.time()`, passing it a callback where it would check for `token.expires > this.getTimestamp()`.

It seems I haven't understood how you'd like this solved.


Don't use underscore in names.


missing word? would see when


Comma after development


"against different responses" sounds a bit strange?

suggestions:
"with different responses"
"against all the different responses.."

not sure


TeamCity Gitautocommit :: Build [Build 1](https://tc.phome.cz/viewLog.html?buildId=37&buildTypeId=Gitautocommit_Build&guest=1) is now running


an -> a


Considering that, can we say following?
* In the chain, there has to be at least 2 levels (`x.y.z`).
* where at least one of those has to be a function call.
* just `x.y` would not be affected, even if both x and y are function calls (i.e. `x().y()`)

I am not sure, if above is correct and also accepted by others.


I think enabled should be True by default.


This is good!
In the future make sure to reflect changes in both the source .cpp and header .h files.
Declarations go in the header, implementations and initializations go in the source.


Referencing MainActivity. Code not reusable.


or 'going to explain' ?


TeamCity Gitautocommit :: Build [Build 1](https://tc.phome.cz/viewLog.html?buildId=37&buildTypeId=Gitautocommit_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:50


Again local test seems promising (W  155 - L 133 - D 397   Elo +10,65),
but fishtest is another world.


Comma after Multimocks


Some existing hubspot-contacts tests use the HubspotClientError("msg", "request-id") form, so they fail if run against this version of hubspot-connection. This seemed the easiest fix, but I'm open to other ideas.


Is the ellipsis character not used in Mexico?


TeamCity Gitautocommit :: Build [Build 2](https://tc.phome.cz/viewLog.html?buildId=39&buildTypeId=Gitautocommit_Build&guest=1) is now running


and -> an


TeamCity Gitautocommit :: Build [Build 2](https://tc.phome.cz/viewLog.html?buildId=39&buildTypeId=Gitautocommit_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


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


:+1: :smiling_imp:


Remove unnecessary secondary constructor. May call primary constructor instead.


Thank you so much Anton !!!


I think it should be "Events List" and not "Event List"


Your backend


Should better leave`.xyz`  by default outside the list, commented out maybe,
since there are [programs that using it](https://en.wikipedia.org/wiki/XYZ_file_format).


Summary of irc discussion: parsing IPs is in `IO`, so it would complicate things in `Thentos.Config`, that's why it happens here.


Nevermind, github presentation made me feel it was waaaaaaaaay longer...


You have a full stop on the end here, but not on others. Standardise on one or the other?


These numeric constants are available via

    var ScrollViewConsts = require('NativeModules').UIManager.RCTScrollView.Constants;
    var ScrollViewDecelerationRateNormal = ScrollViewConsts && RCTScrollViewConsts.DecelerationRate.Normal;
    var ScrollViewDecelerationRateFast = ScrollViewConsts && RCTScrollViewConsts.DecelerationRate.Fast;

That's preferable to hard-coding them.


required to


Integer? Lose the quotes


s/now/not/


Which packages?


nit: this is one-space too far indented.


should this be indented? bullet/number point?


Ну дык опечатался. Бывает.


OK, sorry. I was adding comments on the fly


Since 1998.  However, my experience is primarily with sequential coding.  R is object oriented and a completely different ball game.  I struggle with it.

From: Johnny Ocampo [mailto:notifications@github.com]
Sent: Monday, January 25, 2016 7:53 AM
To: DRavaglia/ProgrammingAssignment2
Cc: Ravaglia, Daniel C
Subject: Re: [ProgrammingAssignment2] Update cachematrix.R (9c73ccf)

It is good to hear from you Dan. It can be a bit of a challenge at first to use Git bash alone with Git Hub since one is not constantly keeping version control on documents, but, once you use it twice or more times then it becomes fun to work with. I like the fact you provided a test case for the programming assignment. Out of the four individuals I had to grade their assignments, you were the only one that provided the test case. How long have you been programming for?

—
Reply to this email directly or view it on GitHub<https://github.com/DRavaglia/ProgrammingAssignment2/commit/9c73ccf4b5b63e5ebb5b1cbf46beeeec1240029a#commitcomment-15648750>.

________________________________

This email is intended solely for the recipient. It may contain privileged, proprietary or confidential information or material. If you are not the intended recipient, please delete this email and any attachments and notify the sender of the error.


you -> your


need to do 'an' appropriate level


Credit goes to @derekprior! https://github.com/thoughtbot/dotfiles/commits/master/bin/bundler-search


I'm not sure scaling is the correct term to use. It's more making the application do more things with rest endpoints. :palm_tree:


Handle errors.


In case I completely missed it; I noticed the trace is currently "coloured" in greyscale - is it possible to get more colours by setting the term variable differently?

-original message-
Subject: Re: [usercorn] add some -32 suffixed linux syscalls (abe9225)
From: Ryan Hileman <notifications@github.com>
Date: 25.01.2016 13:08

You sure? I get that error address at the end of every busybox armv5l command but the command still works.

For me it prints this on the line before (tested on 125d90e64ad917b7ccd20b14c65295e133311b9a): `whoami: unknown uid 501`

---
Reply to this email directly or view it on GitHub:
https://github.com/lunixbochs/usercorn/commit/abe922513ae0238f6314f3a6a6147ffa8c258c71#commitcomment-15647992


TeamCity Gitautocommit :: Build [Build 3](https://tc.phome.cz/viewLog.html?buildId=40&buildTypeId=Gitautocommit_Build&guest=1) is now running


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


Unsubscribe when necessary.


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


TeamCity Gitautocommit :: Build [Build master-1.0.0-66-g4a71f26](https://tc.phome.cz/viewLog.html?buildId=40&buildTypeId=Gitautocommit_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:28


TeamCity Gitautocommit :: Build [Build 4](https://tc.phome.cz/viewLog.html?buildId=41&buildTypeId=Gitautocommit_Build&guest=1) is now running


Uh sorry about all the mode changes.
fkn git


Awesome thanks!


This looks like trailing whitespaces… Do you use editorconfig?!


TeamCity Gitautocommit :: Build [Build master-1.0.0-66-g4a71f26](https://tc.phome.cz/viewLog.html?buildId=41&buildTypeId=Gitautocommit_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


Initial


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


Why is it necessary to extract into a local function?


true, but it's better than skipping the riak tests on travis

On Mon, Jan 25, 2016 at 10:14 AM, Daniel Miller <notifications@github.com>
wrote:

> My hope is that Riak CS implements a superset of the S3 API implemented by
> moto, so hopefully most things that work with moto will also work with Riak
> CS. The best way to verify this is to run tests against both, or at least
> Riak CS, when running tests locally.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/9f1ef4b438c0afa7cfe3f806ffc1198543e201d2#commitcomment-15651854>
> .
>


Glad it wasn't me who broke the build :dancers:


This seems like a good thing to do, I was just curious about what the change broke.


This is not necessary for the current bot API, since `/cmd@botname` will only be received by `botname`.


I don't think we're just formatting, we also unwrap it. So I would call it `unwrapJson` and potentially leave the `nospaces` call inline.


With Multimocks, delays ... Maybe be constisten with the proj name. i.e. With Angular Multimocks, delays


Hi Nicklas,

FYI: http://tests.stockfishchess.org/tests/view/56a094f40ebc591002ad9bf7
showed that already condition depth8 - 5 is probably a regression for single-thread.
This means that if your 3-threads test passes (I really hope so), you will subsequently have a hard time to pass the regression test on single thread. regards G.


@timler Looking at the names of the classes `GeneralConfigFile` and `GeneralConfigUpdater`, my assumption is that these relate to the second part of https://github.com/jembi/bsis/issues/313#issuecomment-125494383 (PR #315) - where Duma added a way to bootstrap general config values during compilation.
@debonair - can you comment on those classes - are they in use?


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


@dainis I'm afraid these tests need some adjustment when I fix the logic to avoid reporting empty values..


Quote it.


Whoa


Thanks!


Great, a good start.


missing break


I'm not in favor of a "version" property or in versioning the GeoJSON spec – stability of the format over time has been one of its best features – but somebody had to propose some text and so I did.

I'd like to prevent other parties from layering versioning extensions on top of GeoJSON and defining a versioning property and freezing it seems like it might help, but establishing a versioning mechanism also seems like a sink for time and energy. Is this a best practice for JSON based formats that GeoJSON needs to follow? What formats would we be following?


Sometimes I really feel like adding spaces to abbs build tmpdirs.


`!isset()`?


Is this a new requirement?  I don't remember having to include this java recipe in the past?


Good point, so perhaps this should be removed, given that "attempts" here speaks to a specific method, rather than the dictionary binding code.


Hab das Request-Problem jetzt mal einfach gelöst. So lange eine Scraping-Klasse eh nur innerhalb eines Landes scraped, reicht es ja, wenn lediglich die erste Station geo(de)kodiert wird. Dann sparen wir uns eine PostGIS-Datenbank + eine Länderkürzel-Validation


112331505_SurfaceHeightRange (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

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


Is this part of the PR?


:facepalm:


I don't get this title :s


*your


assertion comparing AMP and CHARGE is still in AnalogInputs::getCharge


Need debug in here or should this be pulled?


А можете адресовать, так как я не понимаю английский.


Awesome.  Thanks Matt!


Your solution would work well. But I think it is not the best way.

If we provide the submit button, we would not need to use an event handler by using JavaScript code.
It would also be no need to provide a button for each of the select box controls.
In order to resolve this issue more properly, I think we should modify the PHP code together with the template files.
As a reference case, there is a comment bar of the Geeklog comment system.

I'm going to tackle this issue.


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


Think of ... doesn't sound right ? :S


Ja klingt super, es denn der unwahrscheinliche Fall tritt ein und die erste Station ist eine Teststation in Zentralasien... aber das kann man ja beim Testen ausschließen.


nevermind, they were moved from doSlowInterrupt, so it is still there. I will move it.


make sense, I'll remove the sentence starting with "Attempts"


yay! I was hoping those were available somewhere.


default scenario, a ...


Also changed file name refrences from settings to settings.json


How about mentioning keyrings.alt package in this error message?


Nope, doesn't change a thing. But maybe the fact that the PyNumberMethods structure doesn't have the same number of fields in 3.4 and 3.5 is a clue :-)
Sure, it has two more fields with 3.5, but I'd say they get to NULL unless not initialized. So, this should either be reverted, or the last two fields should be conditional on Python 3.5.


Hmm... I see the PR has been merged already; will do a new one then.


s/test_is_ceftification_enabled/test_is_certification_enabled


Isn't `mapgen_air` used in older versions? Maybe we should add it to `legacy.lua` to maintain backwards compatibility?


for (String fieldName : futures.keySet()) {
                System.out.println(Thread.currentThread().getName()+":"+fieldName+":"+futures.get(fieldName));
                ExecutionResult executionResult = futures.get(fieldName).get();

                results.put(fieldName, executionResult != null ? executionResult.getData() : null);
 }

root cause by recursive waiting executing as below:

main:hero:java.util.concurrent.FutureTask@4b5a5ed1
pool-1-thread-1:id:java.util.concurrent.FutureTask@39749523
pool-1-thread-1:name:java.util.concurrent.FutureTask@d29af38
pool-1-thread-1:friends:java.util.concurrent.FutureTask@16be82d3
pool-1-thread-2:name:java.util.concurrent.FutureTask@78abdaca


If not,


Missing jsdoc


I am testing this now and it looks good - the message needs adjusting again to refer to "widgets" rather than "rules".


Thanks, @volkan, the regression should be fixed now.


`host` and `port` should be assigned on their own lines to increase readability.


Took WP out of login page


And back in firewall_rules.php you can make the same correction of:
s/now/not/


Aah, we also have problems at the office. It is the worst, being a web developer and all.


I ran into an issue when not including this recipe:

```
invoke-rc.d: initscript jenkins, action "start" failed.
       dpkg: error processing package jenkins (--configure):
        subprocess installed post-installation script returned error exit status 7
       Errors were encountered while processing:
        jenkins
       E: Sub-process /usr/bin/dpkg returned an error code (1)
       ---- End output of apt-get -q -y install jenkins=1.645 ----
       Ran apt-get -q -y install jenkins=1.645 returned 100
```

Running this command again shows that Jenkins was installed and is runnable.  However, kitchen will never succeed with this.  I found a related issue about Jenkins having a bug when installing:

https://issues.jenkins-ci.org/browse/JENKINS-31814

I then attempted to pin kitchen to an older version of Jenkins to try to fix the issue, but the jenkins['master']['version'] attribute is actually totally worthless when doing a package installation because the Jenkins apt repository has been broken with this issue for years:

https://issues.jenkins-ci.org/browse/INFRA-92

I started down the path of making a change to the Jenkins cookbook to do a manual .deb download and a dpkg install when a version is specified.  I did a kitchen test without a version specified, and noticed the Jenkins cookbook succeeds in kitchen with the latest version...

After a bunch of digging around I noticed that all of the tests run jenkins::java before installing from apt.  Walah, I added the dependency to sous_chef and kitchen started succeeding.  I really don't like the fact that this wasn't needed before, probably still isn't needed in the scheme of things to run it, but I do want kitchen passing.

I'm not sure why this works, because I could not find any logs or extra information on why the Jenkins initscript was borked.  I figure that if the Jenkins cookbook is using it all over, it's probably a safe bet to use it.


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


It doesn't replace the for loop. Just needed the conditional for the new stuff on L428-431.


leaving for now, until I can work out how not to break the call to `RequestMailer.alert_overdue_requests`


gostaria de receber oposições de como melhorar este exemplo. Obrigado!


That is a later problem. Let see if this is a gain with low hash pressure. 0 ELO-patches ends up like that often with [0,X].


Modificado el script de inicio del servicio


I disagree with this, only badge-danger???


fixed


thanks. removed.