This is not going to be OK. It's fine to check and print them but failing CI for just this is already becoming a mess.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 123](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1128&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:45


- It gives you authentication and *authorization* management out of the box. (It's important to mention authorization since we're talking about Laravel 5)
    - One of the things that makes Laravel special, is that it gives you a set of tools aside from the framework. Like Homestead, Envoyer, Valet, etc.


@vinothkr Can we please make this part of `ClientSettings`?


To further validate this interpretation, I reverted to *before* the commit, and made some more tests, and got results similar (even a bit higher) to the ones of the second test-set (several times I had 95 seconds uptime and 19 garbage collections for the PS Scavenge GC). Seems to me this mainly depends on the immediate machine state.


We can, but most crawlers do them by default. You cannot rely on it in the
wild. Thought this client is going to be used only for crawlers, so we need
to see if it has more usecases than that

On Thu, Jul 7, 2016 at 5:26 PM, Ashwanth Kumar <notifications@github.com>
wrote:

> @vinothkr <https://github.com/vinothkr> Can we please make this part of
> ClientSettings?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/ind9/bubblewrap/commit/17a2e3b99209d9581a23caefc59d330d0ca2c1fa#commitcomment-18156498>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAWbU0CfNAhWPlssSn7uXRQMr39oi9Kjks5qTOl7gaJpZM4JHAVU>
> .
>


not sure what \\: does


@ilovezfs If CI fails, then those formulae have missing direct deps.


Tu aici ai demonstrat cum se utilizează [String.Contains](https://msdn.microsoft.com/en-us/library/dy85x1sa(v=vs.110).aspx), dar pe noi ne interesa [Enumerable.Contains](https://msdn.microsoft.com/en-us/library/bb352880(v=vs.100).aspx).
Ai putea, spre exemplu să faci metode care să ia o listă de Band și să întoarcă True sau False dacă există sau nu există trupe din England în listă.


xlt


Yes for sure it helps. Yes I did understand that as Mann group pointed out that one should calculate their incorporation for each label alone. so I understand it and it is clear. Actually they published a paper in 2006 nature protocol and I just read it entitled "A practical recipe for stable isotope labeling by amino
acids in cell culture (SILAC)" it is stating the strategy.

You said "The red line indicates a incorporation rate of 95%. If your densities max higher than that your incorporation is sufficient" means it is 0.95 , so if the maximum density (in the little table) is for example 0.4 for lys and 0.42 for arg and 0.4 for total. It means that the labelling was not fully completed? is this something to do with light ?
or do you mean the tail of gaussian distribution should be higher than 0.95?

Cheers,
Mohammad


I'll work on that.


remove console translate alert


Do we have any guidelines on when the `errorType` has the `xError` suffix? The other errors I've seen so far all use it... `VersionMismatchError`, `NotFoundError`, `RequestEntityTooLargeError`, `UnsupportedMediaTypeError`, etc.


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 14](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7107&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) is now running


xlt 1142 and 1148. remove console.


Please include all relevant files in the commit (node modules, index.html etc..) ready for running on NodeJS.


I'm not thrilled about this change either, for the record. This is going to lead to some ridiculous formulae on things with ties into the GTK tree, for example, and there's no actual breakage currently because the deps _are_ part of the recursive tree.

It feels like an unhelpful thing to contributors to just fail CI hard over.


see the commit comment - I was not able to run this script on the server.


Aici nu le-ai sortat după nimic. Poti folosi orice algoritm de sortare clasic.


#11 fix


We're using paragonie/random_compat as a fallback


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 14](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7107&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) outcome was **SUCCESS**
Summary: Running Build time: 00:01:43


BTW, the merge is also wrong has Wro4j can't compile anymore

```
Jul 07, 2016 1:59:41 PM com.google.javascript.jscomp.LoggerErrorManager println
SEVERE: /catalog/js/GnAdminModule.js:43: ERROR - Parse error. ')' expected
    }]);
     ^

2016-07-07 13:59:41,785 ERROR [geonetwork.wro4j] - Error occurred during a wro4j request handling
ro.isdc.wro.WroRuntimeException: Compilation has errors: [JSC_PARSE_ERROR. Parse error. ')' expected at /catalog/js/GnAdminModule.js line 43 : 5]
```

I'm fixing those issues.


ok if translation of "submit" won't mess with processing


I use MXE_GCC_THREADS := 'posix --disable-sjlj-exceptions --with-dwarf2' in my settings.mk

Which then leads to

```
checking whether madvise is declared... no
checking for struct tms... no
checking for clock_t... yes
checking for F_SETLKW... no
checking if mkdir takes one argument... yes
/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config.gcc: line 1627: test: too many arguments
Using `/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/i386.c' for machine-specific logic.
Using `/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/i386.md' as machine description file.
Using the following target machine macro files:
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/vxworks-dummy.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/i386.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/unix.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/bsd.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/gas.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/dbxcoff.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/cygming.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/mingw32.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/mingw-w64.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/i386/mingw-stdint.h
	/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/initfini-array.h
Using `/Users/aj/dev/mxe/tmp-gcc-host-i686-w64-mingw32.shared/gcc-4.9.3/gcc/config/ i386/xm-mingw32.h' as host machine macro file.
Using host-mingw32.o for host machine hooks.
posix --disable-sjlj-exceptions --with-dwarf2 is an unknown thread package
make[3]: *** [configure-gcc] Error 1
make[2]: *** [all] Error 2
make[1]: *** [build-only-gcc-host_i686-w64-mingw32.shared] Error 2

```


perfect


So ... to resume, I am trying to follow your suggestions. Are the following two functions correct ?

    // convert single codepoint to UTF-16 encoding
    codePointToUTF16 =
      function(codePoint) {
        // out of range, use replacement character
        if (codePoint < 0 || codePoint > 0x10ffff) {
          return '\\ufffd';
        }
        // javascript strings are UTF-16 encoded
        if (codePoint < 0x10000) {
          lowHex = '000' + codePoint.toString(16);
          return '\\u' + lowHex.substr(lowHex.length - 4);
        }
        // supplementary high + low surrogates
        return
          '\\u' + ((codePoint - 0x10000) >> 0x0a) + 0xd800 +
          '\\u' + ((codePoint - 0x10000) % 0x400) + 0xdc00;
      },

    // convert single codepoint to string
    stringFromCodePoint = String.fromCodePoint ||
      function (codePoint) {
        if (codePoint < 0x10000) {
          return String.fromCharCode(codePoint);
        }
        return String.fromCharCode(
          ((codePoint - 0x10000) >> 0x0a) + 0xd800,
          ((codePoint - 0x10000) % 0x400) + 0xdc00);
      },

From what I understood, these are the basic fundamental functions needed to replace the current "convertEscapes" and "unescapeIdentifier" helpers. Did I understand you correctly ?

I am trying to keep the code free of .slice(-4) and .map() operations for backward compatibility with old browsers so I did some minor change to what was suggested above.


Fixed in #243


xla


@avit I'm not sure but I don't plan to support Rails 3.2 for this gem.

Sorry, but please choose a version that works for you as I'm focused on shipping the final 0.5.x version and bumping the minor to support Rails 5.

The code must go on.


se putea cu Lambda Linq cu 2 "SelectMany", primul penru lista de albume, al 2lea pentru lista de litere dintr-un album (obținută după Split).


fixed #11


Yes by definition they have direct linkage. But does this check if one of their declared dependencies directly depends on the same? If so, it's not particularly important that it be declared again further down the tree.

More importantly, to "fix" what is not actually broken is going to require the supposedly "broken" formulae to themselves pass CI, and many do not for *unrelated* reasons (audit failures, missing tests), which then causes everything else to be blocked waiting on those fixes. Or we can just start merging things that are red.


FYI, the reason we need to check the direct deps (although they are indeed included in dependency tree as secondary or deeper deps) is it will help us to prevent breakage from formula updates. `brew test-bot` will only run test for direct reverse deps. As result, for any formula who has undeclared deps will fail to be checked whenever its deps get updates.


Hi! Was reading through this PR.
Looks like `checkNotNull` is supposed to be `checkArgument`.


should this default to the user's default format for dates?


remove comments


"submit" ok to translate?


same question


I have to assume it is fine...


Yay! Because i Don't know nos to compile phars


This should be in the Creature spec above. Putting this here has no effect.


@rhero think the original one was correct.


That would be fixed if we turned uses --recursive back on, which we should.


一个巨大的前端资源清单。


Com package?
Maven setup missing source/target and encoding.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 403](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7110&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


HTML and LaTeX are treated quite differently by the makefile. I can confirm that there is currently no toc in the generated PDF.


Don't let others catch you like that. :P


translate?


xlt?


🔦 Reviewing


✅ Approved


great, thanks a lot Ben!


I think, _at the very least_, this breaking your everyday PR isn't helpful to contributors. It's a fairly opaque system. There's no hints as to:

* Why are they possibly undeclared?
* Is this actual breakage?
* Does this need a revision bump to fix?
* Do I just fix this by adding the dependency or?

That may be obvious to us, but it won't be to everyone. We've already barely documented this `linkage` command & have seen it confuse people even _before_ this change, and now we've introduced a mandatory check that is going to ensure a lot more people see it a lot more often, and have no idea what to do with it.


I find that warnings achieve very little and rarely put warnings in recommended configurations.
Sure, having duplicates for instance will not create errors, but in the context of ESLint settings, that would warrant an error level IMO.

I would also specify every rule, even if just to set it to `off`. That makes it easier to copy-paste all the rules for those who would prefer to do it that way.


Please change the license from MIT to Copyright (c) 2016 Ganesh Pawar


I think it only matters if you build your database for the first time. In that case one gets a "Cannot add foreign key constraint" error from sqlalchemy.

In detail:

> sqlalchemy.exc.DatabaseError: (mysql.connector.errors.DatabaseError) 1215 (HY000): Cannot add foreign key constraint [SQL: '\nCREATE TABLE `MatchParticipant` (\n\t`championId` INTEGER, \n\t`highestAchievedSeasonTier` VARCHAR(30), \n\t`participantId` INTEGER, \n\t`spell1Id` INTEGER, \n\t`spell2Id` INTEGER, \n\t`teamId` INTEGER, \n\t_id INTEGER NOT NULL AUTO_INCREMENT, \n\t_match_id INTEGER, \n\tPRIMARY KEY (_id), \n\tFOREIGN KEY(_match_id) REFERENCES `MatchDetail` (`matchId`) ON DELETE CASCADE\n)\n\n']


translate


I just changed the way I created the recommended config in one of my plugins. I now put them all in the rule meta docs field, then loop over each. https://github.com/jfmengels/eslint-plugin-lodash-fp/commit/97a04abc91b6447b2823f5df4fa9b23b4e97904f I think I'll use something like this in my other plugins too. I have tests that make sure that all rules appear in the recommended config.

The same system could be adapted to create multiple configs (recommended, errors, warnings...)


maybe call the field mean_absolute_error to make it similar to mean_residual_deviance?


testing


TeamCity OTA+ :: rvi_sota_server :: ci [Build 403](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7110&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 177, ignored: 3 Build time: 00:05:08
### Failed tests
```
org.genivi.sota.resolver.test.VehiclesResourceWordSpec.Vin resource should delete a VIN and its installComponents: <no details avaliable>

org.genivi.sota.resolver.test.PackagesResourcePropSpec.create a new resource on PUT request: <no details avaliable>

```


the max is not useful as mana should be always > 0
you could also use the function `maxBy` (http://www.scala-lang.org/api/current/#scala.collection.immutable.Vector)
player.getHouses.maxBy(_.mana).mana


Liquidbounce has 2 devs and here's their email addresses:

senkju4@gmail.com
Marco.MC0902@gmail.com


Turning off removeClippedSubviews doesn't solve my issue #8607 , only solve the problem described in the comment.
I've reverted this commit and the issue #8607 went away


`--recursive` is a temporary hack as it will run lots of unnecessary checks. By design, formula should declare all deps which it directly links to.

Also this change will help us to prevent bottles linking to incorrect formula, which may happen to be installed in CI.

I understand in short term this will cause some trouble in maintaining, as it exposes a problem we just ignored in past. But this is indeed a correct step to help us make homebrew more robust and fixes the long term problem that some formulae fail to be bumped during deps update.


can this be removed?


Remove mysql database credentials.


manque une dep, suffit de regarder


It seems we need to solve a CSS resource issue before a Plone 5 compatible release. Currently icons do not show in portlet. May be related to this: https://github.com/collective/eea.annotator/pull/14#issuecomment-229506924


@DomT4
> Do I just fix this by adding the dependency?

This is most common fix. Unless some dependency are clearly out of nowhere(which means formula accidentally picks up some formula happening to be installed on CI).


i believe it there is no "s"
max-width: fit-content;


This change broke other code which calls this method by using an instance of ilLanguage. There are currently many mixed usages (3 static calls, many instance calls) of this method.
Furthermore some unit tests are broken now.


▶ Review started


extra hash mark here


thanks, now would erase


ok to delete


Why you do this?! :P


commented out --> ?delete


Poți simplifica:
```csharp
return bandsList.Where(item => item.Country == "England").Sum(item => item.Albums.Count());
```
sau chiar
```csharp
return bandsList.Sum(item => item.Country == "England"?item.Albums.Count():0);
```


Your call :P


that would be great!


TeamCity OTA+ :: rvi_sota_server :: ci [Build 404](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7119&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Yeah I'm in. I will do my best to contribute.

2016-07-06 0:12 GMT+02:00 Jan Hudec <notifications@github.com>:

> Regarding both YASim and JSBsim, I've actually tried a couple of years
> back and the lessons were:
>
>    - The interface differences are *pain* to handle. YASim has no stopped
>    state for turbines, different fuel handling, different weight handling (and
>    that part is just silly — YASim itself has fully configurable property
>    names, it is just the fuel and weight dialog that uses different ones if
>    the model is YASim).
>    - I didn't like the model anyway. It was better than the JSBSim one,
>    but slats apparently don't do anything, flap efficiency and drag can't be
>    configured and I would have liked to be able to simulate coffin corner. And
>    then there was the issue of different models. Basically all A319s should
>    have the same drag, lift etc., but different thrust, but because YASim
>    solves the model at startup it is not possible to achieve.
>
> So I would stay with JSBSim.
>
> For tuning the model, it would help to have something to draw graphs of
> lift, drag, thrust etc. for some conditions in a similar way to what the
> yasim command-line tool does. Basically I'd need something that would get
> attitude/relative wind, altitude, speed and control parameter values, made
> sure the engine values are as if that condition was stable for a while, and
> then read the forces and moments. I would then run it for various
> conditions to get useful graphs. I think it should be possible with the
> stand-alone jsbsim, but I didn't understand the documentation for it. So I
> would welcome some help with that.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/FGMEMBERS/A320-family/commit/5f3e7cbe944e759555e63df17107062bb68848b4#commitcomment-18133552>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ARQSK-NeyZhTl5hqIsGT7ODbf-degqyGks5qStbfgaJpZM4JCcxN>
> .
>


Can you change this to something like:

Precalculate blocks are executed in the question/outcome method first, i.e. before the question/outcome template is rendered. This means that values calculated in precalculate blocks can be accessed in the template.


- Vagrant is not for running commands from the terminal.
- The `hosts` file exists in all Operating systems, not just MacOS.
- Talk about reloading the VM every time you change the Homestead.yaml config file.
- Since Homestead installs Nginx by default. It allows you to run as many applications as you want!


Kann hier einfach rausgenommen werden, dass auch zwei andere Studenten gewählt werden müssen?


Not that I know of, no.


So both of those don't work for me now, and I have to do ```git clone https://github.com/palantir/atlasdb.git``` (which is the one from the github [landing page](https://github.com/palantir/atlasdb)). I have no idea why the first one doesn't work for me.


This actually might be the first time that our UnitTests found a new problem. This is a good day =)


I know Rails 5 requires _at least_ 2.2.2, but we could use [2.2.5](https://www.ruby-lang.org/en/news/2016/04/26/ruby-2-2-5-released/) to use the latest 2.2.x release. What do you think?


This does not explain our expectations, as it reads `name to point should return an error`, while what we really want is `name to point should respond with Not Found in case of failed name resolution`


E ok tema, cu observațiile punctuale.


perfect


> This is most common fix.

I'm aware, but I can guarantee not everyone is, and our absolute minimal documentation of this command & changes to it are frustrating for people who are here every day & perplexing to newer or more infrequent contributors.


Wo ist da was rausgenommen? Der Prüfungsausschuss hat nur ein studentisches Mitglied.


Why is this necessary?


Can you change this to something like:

Precalculate blocks are executed before the user response has been input and parsed, the user response is not available in precalculate blocks, because it has not been received at the point the block is executed.


I assume none of the lines above need to be translated...


git@github.com:palantir/atlasdb.git works for me.


Done to here.  Placeholder.


i want to drink tea and bread


@DomT4 May I ask what do you think we can do to improve from this aspect, i.e. helping contributors to understand how to fix formulae which are broken in this area?


http://bot.brew.sh/job/Homebrew%20Core%20Pull%20Requests/4481/version=el_capitan/console

```
==> brew linkage --test webkitgtk
==> FAILED
No broken dylib links
Possible undeclared dependencies:
  atk
  cairo
  fontconfig
  freetype
  gdk-pixbuf
  gettext
  glib
  harfbuzz
  icu4c
  jpeg
  libpng
  pango
  sqlite
```
This PR has nothing to do with that linkage "problem" in webkitgtk and so it should not be a blocker to merging this PR. So unless you want to start merging things that are red, this is not going to work.

Even if webkitgtk were one of the subjects of this PR because it needed a revision bump (which this time it does not but sometimes it does), the linkage "problem" would still be a totally unrelated issue, so again not a good reason to fail CI.


I sense a "redo from scratch, this time TDD" approaching :dancers:


I am missing the `psnfss` package here that is required to fulfill various font requirements for the most recent `jupyter nbconvert` PDF output (asking for the palatino font etc). Adding the `psnfss` solved that issue for me. Could you please add it in your next conda release?


Just want to link to my library [react-native-tachyons](https://github.com/fab1an/react-native-tachyons) here, which includes this constant and similar styling as well.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 404](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7119&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:08:03


Thanks it works now.
Badly I think this introduce a new bug.
when calling node->Clone(), cloned RigidBody2D properties are not correct.

For example when trying to clone a body with BT_DYNAMICS, it return a BT_STATIC (default value)


Oh, I get it. Because :8ball: told you so. 42 is the answer man.


@olivierdalang I merged your django fork for view_permission backport 1.9.x and was facing `AttributeError: 'Profile' object has no attribute 'id'` error on inline models while accessing admin. I think 'pk' should be used instead of `id`. It fixed the error though.


@xu-cheng If the PR were actually a version bump for webkitgtk, it might be an appropriate time to fail CI, but I don't see how randomly tying the issue to totally unrelated pull requests is a good way to implement this kind of a change.


@snicolet, this is a nice idea. I hope you intend to move this into king safety. I guess it logically belongs there.
Good luck!


It the line 21 is removed then the line at 23 should be removed too. It is referencing a non existent variable. The fd.close() should be removed.


Thanks for testing - I see you have created a ticket for that - should be fixed now on master...


My first idea was to derive a class based on bitstream like cBitstreamEP3. It would need its own readBits function that is almost identical to the base readBits only with the addition of the EP3 removal check.
For the sake of speed, I would go for that duplication.
Now comes my issue: If the derived class does not define readGolombUE and a call to cBitstreamEP3::readGolombUE is made, will the base class function call its own original readBits or will the version of cBitstreamEP3 be called?
If not, the derived class would have to have its own identical readGolombxx funtions. That's what I meant with duplication.

I also don't like the if statements if only either of the paths is taken for a certain object, because it might/will mean that the queue has to be emptied depending on what the branch prediction did. But it must happen either in the EP3 or non-EP3 case. Do you have a trick in mind?


I'm with @xu-cheng here; `uses --recursive` never managed to catch this type of thing reliably but `brew linkage` does.

> Do I just fix this by adding the dependency or?

I think this is the case.

I definitely agree the documentation and messaging for this should be improved a lot and it should be something that's called by e.g. `brew test` or `brew audit` to contributors have a chance to catch it before then.


In your example, we will miss `webkitgtk`, whenever any of its undeclared deps get version bump.


> I use MXE_GCC_THREADS := 'posix --disable-sjlj-exceptions --with-dwarf2' in my settings.mk

I see, remove the quotes. `make` will pass that though as a single string.

Note also, this isn't heavily tested - it only really works from a clean build. I haven't had a chance to dig deeper.


Hi Mohammed,

if your maximum density is around 0.4-0.5 it means that your H/L Ratios are mainly around 1. That means that you have as much light amino acids as heavy ones incorporated. A Silac experiment with these incorporation levels would not be successful. You need a much more complete labelling. As stated previously the commonly used threshold is at 95% (max density 0.95).
The nature protocols paper give some tips on how to perform the experiment in detail (like for example the recommended number of cell doubling) to gain higher incorporation.
Maybe you can send me the plot to:

b.hessling@zmbh.uni-heidelberg.de

Best,
Bernd.


It's worth noting we break the build similarly for new `brew audit` (not `--strict`) rules but in those cases it's obvious how to fix the issues and we tell people to run `brew audit` before submitting PRs.


@francescosalvadore please take a look at the *developing* branch that I have pushed: I have made some design modifications, I would like to know your opinions. The log of those commits are here reported for the sake of clearness:

### Improve (I hope) classes design:

+ descriptions strings are now allocatable;
+ abstract classes provide `free` methods to free dynamic memory;
+ init/output methods becomes subroutine: I do not like *impure* function, and it could be nice to have error code optional;
+ concrete classes provide `set` method;
+ abstract field require now an `associate_mesh`: it has been pulled-out from `init` method in order to able to reuse the code also into other methods, e.g. into `setters` methods.
+ Split sources, 1 class => 1 file: just to make for my simple mind more clear the hierarchy of classes.


If you want to flatten the dependency tree, that's fine, but it should be tied to PRs that are version bumps of the specific formula, not revision bumps, and not entirely unreleated PRs.


gfgfdg


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


Points 1 and 2 refer to the same thing. Both are talking about the `SmartAnswer::InvalidResponse` exception. I don't think it's good idea to start a bullet point with "This".


@chrishaid I think this could be HUGE!  Excited to talk it through later.


Nice, thanks


o que é deixar o código identado?

Date: Thu, 7 Jul 2016 04:07:17 -0700
From: notifications@github.com
To: TrabalhoSemana4@noreply.github.com
CC: nataliamirandaa@outlook.com; mention@noreply.github.com
Subject: Re: [UnicariocaDev/TrabalhoSemana4] commit4 (cd10c30)

@NataliaMiranda Está ok seu código mas o projeto está confuso.

Tem duas Classes com o mesmo nome.

Faltou deixar o código identado.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


It's not flattening the dependency tree unless there is direct linkage in which case we need to declare a dependency for many reasons.


holy crap I just nuked a zillion lines of code by using this ref.


Se não foi deletado uma letra


you have define what stock


TeamCity OTA+ :: rvi_sota_server :: ci [Build 405](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7125&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


can you release a new version on hex?


Whoops forgot to make the itemblocks and register them, as it is atm it will not work :P


* This should be considerably better documented. The entire command, not just this feature. Let's not pretend this is an entirely dev-facing command any more, we've exposed it to anyone who submits a PR regardless of how new or ever-present they are to Homebrew.
* That document should detail some of the failures, _what they mean_ & how to fix them.
* In an ideal world, this wouldn't be triggered outside of a similar mechanism to `brew audit --strict`, which we already encourage users to run locally before submitting PRs. This will reduce the already noticeable number of PRs being mysteriously tanked after a user has already tested/built/audited a PR locally.


Точно надо?


Indeed... I should only display this when --force option is in use.


в самом конце лучше вставить рендер шаблона, который по GET возращается


TeamCity OTA+ :: rvi_sota_server :: ci [Build 405](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7125&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 178, ignored: 3 Build time: 00:05:43
### Failed tests
```
org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should be able to assign exisiting filters to existing packages: <no details avaliable>

```


удаляй принты


> I could produce the evoked files with a python gist I suppose, but I don't see the difference.

It's probably better for reproducibility to do everthing in Python but it's not strictly necessary. Why add the `.ave` file to the repo? If it's to be able to reproduce the file, a Python script would be better.


Fixes #367


@flagos I suspect you're right and I should be using "path" here, but the logic needs adjustment then.


Ok, see PR #403 . I was wrong when I said that `COQDOC_OPTIONS` was used for latex. #403 corrects this.


I'd probably call this Function and not Foo. :)


@leobiscassi
Please, look at test/load_population directory. There are all files necessary to execute the protpred-Gromacs-Test_load_population binary. Please, you need to update the installation path at configuracao.conf file.


Can this return error?  If so, should we watch for an error and then re-attempt delivery again later?


всегда делай редирект и тд по имени, не по url-запросу


isn't that obsolete after my fix there: 30fe924e2594b13567287c67d274382d9f980cf8
your check will never fail, because mine would fail first?


@tonyouyang LGTM


Yup, I was taking baby steps to get it to compile on my machine.  Future versions will use [CHECK_TYPE_SIZE](https://cmake.org/cmake/help/v3.0/module/CheckTypeSize.html) for setting `SF_COUNT_MAX`.  It may be better to set `TYPEOF_SF_COUNT_T` in the source code.


@MikeMcQuaid http://bot.brew.sh/job/Homebrew%20Core%20Pull%20Requests/4481/version=el_capitan/console

The fact that webkitgtk has undeclared dependencies is not a reason to fail a webp version bump PR. The webkitgtk test passes and the linkage of webkitgtk to webp is fine:
```
$ brew cat webkitgtk|grep webp
  depends_on "webp"
```

If instead the PR were for a version bump of webkitgtk itself, then by all means, go ahead and fail CI for webkitgtk until the undeclared dependencies are added.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#eSkoEv7JTmicTsqO1bXh5g


TeamCity OTA+ :: rvi_sota_server :: ci [Build 406](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7137&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


sentite libre de modificarlo y aparecer como colaborador de este gran proyecto jajaja


Merge here. To see the last merge commit, you now need to look at the tag version : https://github.com/MinetestForFun/server-minetestforfun-hungry_games/releases


It doesn't. Support for global chunk options of any kind is still TODO (but will be straightforward).


REFACTOR use get_or_create()
==> REFACTOR use Models.objects.create()


Keep ligne return


This method does not seem to allow `None` as described in its docs here.


no bang


Ah, super cool idea! I am torn a little, because I like being able to put relevant comments in the config source, and that gets lost if it comes from the meta... but then again, then the docs can automagically keep up with the shared states of the rule ("This rule is included with:"...)

I will mull it over. It's probably a good time, since a major version is nigh.


agile ruby on rails evrone.com


no empty line here


use () en declaration


too many empty lines


Hi, Jonas.  Your change in particles_radius breaks our hourly auto-tests.  The offending line is line 531, in which k1 and k2 are not initialized at all when used.  Could you please check this implementation and see what the right behavior here should be?  Thanks.


Merge here, we will now use tags to inform about the latest merge : https://github.com/MinetestForFun/server-minetestforfun-skyblock/releases


saw approval from jvoegele
at https://github.com/basho/riak_kv/commit/583b520cf5869cb8aa82812bf04529908c471aef


merging basho/riak_kv/update-to-bitcask-2.0.3 = 583b520c into borshop-integration-1434-update-to-bitcask-2.0.3


basho/riak_kv/update-to-bitcask-2.0.3 = 583b520c merged ok, testing candidate = 8fc4fa14


rm planned


tournées


Okay, great!

On Thu, Jul 7, 2016 at 8:58 AM, Jonathan <notifications@github.com> wrote:

> It doesn't. Support for global chunk options of any kind is still TODO
> (but will be straightforward).
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/rstudio/rstudio/commit/a6363a05468d45e60d506e5243fce07ac36dbc6c#commitcomment-18157245>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAGXx-03sOBPmLG1nVa4fobEx_i2ZMdZks5qTPf7gaJpZM4JG7D_>
> .
>


Merge here, we will now use tags to inform about the latest merge : https://github.com/MinetestForFun/xorhub-serverscontents/releases/


테스트 코멘트


@erikd Some of these could be determined in source as well.  Is there a reason there are multiple defines for things with similar meaning?  I'm mostly referencing the Windows defines here, since we could just use [some pre-defined compiler macros in their place](https://sourceforge.net/p/predef/wiki/Home/).


Actually, the Arch repos are up to 6.1.1, so work out of the box from pacman


> If instead the PR were for a version bump of webkitgtk itself, then by all means, go ahead and fail CI for webkitgtk until the undeclared dependencies are added.

Sorry, I misunderstood earlier: I think I agree with this. Yes, I don't think this should be failing for dependents. If they fail to declare that backwards dependency then that formula will break on upgrade without a `revision` in the `test` block so the `linkage` check is redundant unless we were able to e.g. run it on every formula.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 406](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7137&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:07:16


t


Merge here, we will now use tags to inform about the latest merge : https://github.com/MinetestForFun/server-minetestforfun/tags


Foo/Function can be any function with any return value. So I am not sure how to handle the return value.
Semantically speaking, in case of adding to the POLog, if the message was not added for some reason (I don't know if this could happen) and as it was removed from the Queue, it can be lost forever if we don't handle this error. yeah maybe by re-attempting the delivery later.


更新文件


上次可能沒講清楚，我們這邊不是要限時，而是要縮時。

我會改一下piWebCam那邊，你好好讀一下，我會把opencv那邊的盡量做掉。讓你看比較好讀的python code。用我寫好的library，不要用os裡面的指令去開。

會在屬性那邊設定 :  frame_rate = 20，frame_rate就是攝影機一秒鐘會讀多少張，一般來說是20張。
參考faceDetect那邊，固定去取cam調回來的frame，間隔每20張把frame寫進videoWriter裡面。

然後上傳整個video，不是一張一張傳，不然會發現影像非常多，很可怕!

未來再用臉部偵測程式一張一張取出來。

Chun


I've solely been using tab-indentation, so I'm not really sure how this got all messed up. I'll have to look at the file once I get home.


Merge here, we will now use tags to inform about the latest merge : https://github.com/MinetestForFun/server-minetestforfun-creative/releases/tag/v2.0.0


Hi,

it did not break the autotest when I ran it before checking it. I will
check it now.
Kind regards,

Jonas

2016-07-07 15:02 GMT+02:00 Chao-Chin Yang <notifications@github.com>:

> Hi, Jonas. Your change in particles_radius breaks our hourly auto-tests.
> The offending line is line 531, in which k1 and k2 are not initialized at
> all when used. Could you please check this implementation and see what the
> right behavior here should be? Thanks.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pencil-code/pencil-code/commit/6be3496926572575255d3308102cef26b997f262#commitcomment-18157282>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ALLCvyStP__eGOAFBTVf7o-xU7qd-Klsks5qTPjLgaJpZM4JHDcJ>
> .
>


I think the main place where there's ambiguity is when a formula revision bump ("`foo`: revision for `bar`") is required, does it suddenly become relevant that `bar` has unrelated undeclared dependencies? I would say absolutely not. Whether a particular webp upgrade does or doesn't require a webkitgtk revision bump is determined by factors having to do with how much webp is changing or not changing and has nothing to do with unrelated webkitgtk undeclared dependencies.

So I think the main places to enforce this are at formula authorship time and formula VERSION (not revision) upgrade time.


s/relevant that bar/relevant that foo/


@MusikAnimal take note of some spacing fixes I made here.


I can't believe you deleted my great bio line!!


Thank you! I'll be adding more about her wealth in the summary (managed to miss that in the first draft).


Yep, I was able to compile without this addition as well.  I mostly did this change so I could test the pre-commit hooks on my Mac, since I ran into a problem with setting it up on my first commit.  But it also serves a consistency purpose.


Why not using latest IMS BSP with vendor/codeaurora/telephony/ims ?


话说System.out.println()在这里是有效的吗？
其实还有另外一种更科学的日志，代码如下：
```Java
private Logger logger = LogManager.getLogger();
logger.info("日志内容");
```


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


== Commit message ==
Addition of warning message if there is no intersection between PPI network and mutation profile


第二次更新文件


@erikd Currently working on implementing the [necessary platform checks](http://www.vtk.org/Wiki/CMake_HowToDoPlatformChecks).


I only found instance calls in chatroom test cases. Can you give some examples?


- We can use the variables directive to add environment variables.


FEATURE #1


I use 4 spaces. It doesn't matter how wide the tabs are in your editor, idgaf. I'll let you know if your lines are too long during PR review.


`sbt run` should be enough. It automatically does `nativeLink` and package is not necessary unless you want to get a jar.


Yeah we discussed that with @NathanW2 - that would be a preferable way to go. That will just need more work as we would need to add some kind of extra identifiers (everywhere) on what has changed - or have some diff for stored XML data to automatically identify the topic...


I am on it :)

Be aware that what I am going to try is to compile libfec using qemu for different architectures.

That will have two effects: I can properly compile libfec, and - more interesting - I can save the output of `gen_cc_sds_tal`and `gen_cc_sds_tab`in order to reuse them in a cross compile environment.

So, just a small description in the README, won't be enough.
I will add into my fork specific `ccsds_tal.c` and `ccsds_tab.c` for every architecture I can compile with qemu.

It's worth to mention that it's also possible that `ccsds_tal.c` and `ccsds_tab.c` will be the same for every architecture, that depends on the reason why the author of libfec decided to generate the tables instead of manually compile them.


If I remember right upgrade Spring-boot was creating problem in quickstarts.


Try using database to store the data and to display the same. This is very childish.


könnte man auch später noch ändern.


It's not a typo. It's the new null-conditional operator in C#6. The equivalent would be

```csharp
if (partyOfRequestedGroupJoinLeader != null)
{
    partyOfRequestedGroupJoinLeader.AddJoinRequest(ownParty);
}
```


Great!

Alternatively, one could generate the tables using an interpreted language (python, perl, whatever, up to you)

mpb


Reverted my last merge commit


... it was in the wrong if-clause. I'm sorry. It should be fixed now,
samples/most-modules succeeds now.
Kind regards and thanks for the hint,

Jonas

2016-07-07 15:02 GMT+02:00 Chao-Chin Yang <notifications@github.com>:

> Hi, Jonas. Your change in particles_radius breaks our hourly auto-tests.
> The offending line is line 531, in which k1 and k2 are not initialized at
> all when used. Could you please check this implementation and see what the
> right behavior here should be? Thanks.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pencil-code/pencil-code/commit/6be3496926572575255d3308102cef26b997f262#commitcomment-18157282>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ALLCvyStP__eGOAFBTVf7o-xU7qd-Klsks5qTPjLgaJpZM4JHDcJ>
> .
>


Beautiful Angel Dreams


Create gh-pages branch via GitHub


kann man nochmal hervorheben


TeamCity OTA+ :: rvi_sota_server :: ci [Build 407](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7153&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


why?
you are backtracking several FGMEMBERS commits with this revert.


While we're here, I just noticed by chance that line 886 of `modules/EmailTemplates/EmailTemplate.php` reads `Exceptin` instead of `Exception`.

@samus-aran please let me know if you prefer for me to make an Issue for these corrections, or if the comment on the commit is sufficient...


really? thought this removed only the old things still in my folder after not pulling for a long time....


It caused problems (tyresmoke.nas) when retracting gear


ああーぅぅああぅあぅあー！！
ここに、` onchange={ ctrl.onimage } ` が足りてない！！！申し訳ないです！！！


on the contrary: reverting like this is getting the repository back to the older state, and ignoring multiple changes by @Umoxfo, and a livery by me.

If you don't mind, we'll need to test a few things, then (I hope you can help me, and I can use git to merge correctly)

1. over this commit b54fa419a2c502e6e8772a6b914db37ade976e81, do you see the problem of the gears you mentioned?

`git checkout b54fa419 -b gears-test`


I made it non-static again.


You should not have put the URL in the commit message in the first place, just put it here in a comment.
Now it will be in the source code forever...


That's what was expected.
XML for this task sucks anyway ;)


I instantly killed it xD


Nice, thanks a lot for all this!!!


I've done
git checkout b54fa419 -b gears-test
and it doesn't work (fg slows hardly down when retracting gear and putting out many many warnings like these:
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119
Nasal runtime error: vector index 3 out of bounds (size: 3)
  at /usr/share/games/flightgear/Aircraft/E-jet-family/Nasal/tyresmoke.nas, line 53
  called from: /usr/share/games/flightgear/Nasal/globals.nas, line 119


caused by


During an outage


Fixes #2


@neworderofjamie - you just want my input on the stuff in nengo_spinnaker, right?


should either return true directly or at least break the loop here


TeamCity OTA+ :: rvi_sota_server :: ci [Build 407](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7153&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:08:18


s/whether/if


@dpgeorge : FYI. Perhaps a bit hacky, but as the aim was to minimize memory usage, I decided to start with reusing a single array buffer object for both dupterm read and write, instead of introducing a separate one for write.


@lineflyer that was an issue for a single cache only. No need to fix on release.


Bold in the cold. I'm almost inclined to chat with you.


I think that `concat` collect all buffers and call `cb` with the result but it's don't work


how to do it properly? (keep working tyresmoke.nas but get back the other new commits)


OK
That means that there is a bug with the tyresmoke recently introduced.

I say, removing lots of changes over 1 bug is probably the non-subtle way to go (your revert).
Instead, I preffer we restore the newer code, and I'll open an issue and notify @Umoxfo  that it need addressing.

Sounds good?
@IAHM-COL


odod ? Seriously ?


If the directory exists but the file doesn't the warning is logged and the file is not created.


Update Venue


The quickstarts has been upgraded to 1.3.x already


@D-ECHO @Umoxfo
Well... step No. 1.
I reverted D-ECHO merge revert. And the code is back over 7ede041e4c621ef22130
(after D-ECHO did the Correct OHP Knob)

Secondly, I am creating an issue over the tyresmoke. I'll try identifying when the issue was introduced.


https://github.com/Homebrew/brew/blob/master/Library/Homebrew/dev-cmd/test-bot.rb#L619-L624

That seems like it should only fail if there are broken dylib.

Also, it seems test-bot never runs `brew linkage --test` for the formula that actually IS the subject of the PR:
```
==> brew doctor
==> brew --env
==> brew config
==> brew tests --official-cmd-taps
==> brew tests --no-compat
==> brew readall --syntax
==> brew uses webp
==> brew fetch --retry webp --build-bottle
==> brew uninstall --force webp
==> brew install --build-bottle --verbose webp
==> brew audit webp
==> brew bottle --verbose --json webp
==> brew test webp --verbose
==> brew linkage --test gd
==> brew test --verbose gd
==> brew linkage --test mapnik
==> brew linkage --test sdl2_image
==> brew linkage --test sdl_image
==> brew linkage --test webkitgtk
==> brew test --verbose webkitgtk
==> brew linkage --test pillow
==> brew test --verbose pillow
==> brew linkage --test openimageio
==> brew test --verbose openimageio
==> brew uninstall --force webp
```

And it does not cause audit failure even with `--strict`:
```
iMac-TMP:local joe$ brew linkage --test webp
No broken dylib links
Possible undeclared dependencies:
  giflib
  libtiff
iMac-TMP:local joe$ echo $?
1
iMac-TMP:local joe$ brew audit --strict webp
iMac-TMP:local joe$ echo $?
0
```

So unless I'm missing something, test-bot currently doesn't catch it for the particular formula actually being test-bot'd.


Update Participants


Agreed. Must have missed it while committing.
Thanks for pointing it out @octospark


The mile-high, I've-not-really-looked-at-your-code, response is that I don't like that this non-core feature has changed nengo_spinnaker. My intent for robot specific stuff was that you'd provide a new set of Node build methods rather than modify existing ones.

e.g.,

```python
class MyRobotActuator(nengo.Node):
    ....

@Model.builders.register(MyRobotActuator)
def build_my_robot_actuator(...):
    ...

@Model.sink_getters.register(MyRobotActuator)
def build_my_robot_actuator_input(...):
    ...

@Model.source_getters.register(MyRobotActuator)
def build_my_robot_actuator_output(...):
    ...
```

can all live outside `nengo_spinnaker`. However, I may be missing what you're trying to do?


Old version results in:
"Error: Error: logged start point not found"


@erikd You have the sources in a folder for a reason.  It modularizes the code, so it's better to build the module separate from the other code and then link to it later.  It also makes it easy to find and add related code in CMake files.


> The idea was to return a proplist, so that you can then get it by name.

That's where the `N` from `lists:keyfind(Key, N, TupleList) -> Tuple | false` comes in handy:

```erlang
lists:keyfind(CommandName, #mongoose_command.name, ListOfCommands)
```

Unless `mongoose_command/0` has to be an opaque type and the record underneath hidden.


In the end this was not a good idea because we have Velocity scripts that expect ``$doc`` instance to be shared between them. E.g.:

    {{velocity}}
    $doc.setTitle('foo')
    {{/velocity}}

    {{velocity}}
    $doc.title
    {{/velocity}}

And with this change each Velocity script gets a different ``$doc`` instance.


commit 345b681


更新截图


@D-ECHO

This commit reverted one step of changes over tyresmoke.nas only

Do you mind checking whether the bug persists on this branch?
`git pull`
`git checkout -t origin/tyresmoke-bug`

cc, @Umoxfo
@IAHM-COL


cpac_install.sh assumes that default nipype pip install is 0.12 right now- it doesn't look like they've put 0.12 on Pypi yet so check this before release.


@ysbaddaden Wouldn't you need to use [`CryptGenRandom`](https://msdn.microsoft.com/en-us/library/windows/desktop/aa379942(v=vs.85).aspx)? `RtlGenRandom` doesn't say it's cryptographically secure, but `CryptGenRandom` is.


OK, I just dug through the source and histories for GCC and newlib and to be very specific:

It's the responsibility of `libc` to implement these intrinsics, and newlib added support for memclr, memset and memmove on Feb 17th 2015 (commits 72ba8b107a891d6b4094f1fc74b548062dd2ead7 and 554f33c48d069a0cbd1748c87ee9d4f4e22f8c01 in the newlib repo -- git://sourceware.org/git/newlib-cygwin.git).

These changes are present since version 2.3 of newlib (tagged as newlib-2_3_0 in the repo),

Each package repository packages their own combination of GCC and newlib. For example, no version of Ubuntu or Debian includes a version of newlib > 2.2, while Arch linux includes version 2.4.

Our friends at gcc-arm-none-eabi (the one hosted at launchpad and maintained mostly by ARM employees) have included it since at least 4.9-2015-q2-update release (they use commits to make the versions and the newlib commit for 4.9 q1 doesn't exist in the newlib repo anymore).


OMG by complete accident I was reading this code yesterday, after googling around to see if I could find a copy of your thesis pdf, and I saw this and thought "ha, yeah, a temporary file race condition, classic". BUT THEN! Twitter handed me this, this morning: https://github.com/chrislgarry/Apollo-11/blob/master/LUNAR_LANDING_GUIDANCE_EQUATIONS.s#L179 and I made the connection.

Paul, did you... read the lunar lander source code years and years ago? That is *seriously* cool.


WTF! 💥


Couldn't this be package visible? I want to keep the public API surface as small as possible.


@xu-cheng should the `linkage --test` perhaps be split into `linkage --test-broken` and `linkage --test-undeclared`?


Nice dude... fist bump 👊


I open #470, which enable linkage checks for `brew audit`. However, I would still prefer to check linkage in dependents, as otherwise, many of them won't be checked until they have updates.


@kirbyfan64 please see https://blogs.msdn.microsoft.com/michael_howard/2005/01/14/cryptographically-secure-random-number-on-windows-without-using-cryptoapi/ (as linked in libsodium source code).


siunitx 는 이제 모두 mathmode 이므로 inter-unit-product 를 $ \cdot $ 에서 \cdot 으로 변경.

https://github.com/gshslatexintro/examples/commit/e07db45a8668986b5441d19f6d7cb3ede984fa29 참조.


@raztechs I've removed this file because the code does not work in production


Now that you mention it, we haven't uploaded the new build to https://meteor.com/install yet.

When you get a chance, can you try [this installer](https://www.dropbox.com/s/83sk68wfz5kr1sz/InstallMeteor.exe?dl=0)? If it still says Meteor is "already installed," you may have to remove `AppData\Local\.meteor` first.


Ok. What makes a user get an instructor global role? How do I get my course_role instructors to be able to create exercises and workouts?


OK. I've added a new fix-up commit to make it clear I'm talking about the Rails environments: 421c6e66bb3aaf3e8bb49cf3911482d3a1f67e77.


Unfortunately WifiMulti causes delays when connection is lost and renewed...


@LegNeato so basically, this was an intentional decision.  There was a good deal of discussion about it, too, but the decision was as Jonas describes it.

I'm curious what brings you to taskcluster-auth -- I haven't heard from you in ages!


This is going to attempt to read the key `None` from `site_overrides` whenever `site_code` isn't set. It might be less surprising to check both `if site_overrides and site_code is not None` in the previous line.


Change to our default header licence


えかん


Not quite sure what I was thinking when I wrote that commit message :/


This old one is no longer in the proto-file and can now be deleted. I've removed it for you in be1af8684bfd94dd3ed2c20420d68ac27f33330c.


Good deal, this is a huge usability improvement :+1:


"Le" -> "La variable"


Manque "contour"?


Current http_parser Makefile is a mess, so I created patch from these PRs: https://github.com/nodejs/http-parser/pull/321 &  https://github.com/nodejs/http-parser/pull/294. I want to be consistent with naming on all platforms so I would not have any troubles (at least with this Makefile). Everything else I could fix in `http_parser.mk`.


>However, I would still prefer to keeping to check linkage in dependents, as otherwise, many of them won't be checked until they have updates.

Why not? We could make that a "project" of its own to address it proactively and directly rather than waiting for formula version upgrades, instead of tying it randomly to unrelated PRs with burden shifting.


First comment


あか


The terraform one dont work atm and the fix will be pushed soon.


*stream-based


This solution seems(!) to be slower than the sequential version


test


I've addressed this in a fix-up commit: 9b67cefd0f7763e6a48621548a19397b2db2bf1b.


Add licence


Change to our licence


licence


WhiteSpace ( 1 more tab on abobe 6 lines)
Are you sure, they consume all 3 of them if you have all 3?


numero de  serie deve ser index


header


to our licence


header with licence


Vorsicht damit! Es gibt files, die fur den vim buffer viel zu aufwendig sind...


That's some pretty lame variable naming there Eddie?

if (bool == true)


You seem to have ended up committing your home directory into the repo! Have a look at `format.every.sketch.sh`.


ok


header, licence


Wouldn't this be needed in 2.0.4 too?


I can't believe somebody got the joke!

I'm pretty sure that I didn't read the source code spontaneously; I think I might have seen the comment on Lambda the Ultimate: http://lambda-the-ultimate.org/node/3522


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 412](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1129&buildTypeId=XbimEssentials_XbimEssentials) is now running


This might be useful in 2.0.4 too, not sure though it'd merge cleanly?


You forgot adding `libavahi-client-dev` in the `apt-get install` line (right before this one).


if it already merged create an issue, if it an open pull request comment on the commit


Relevant to: MPAuthor.Stores Management Pack - 1.0.0.36


it doesn't. If you feel like it, you can make a PR against the 2.0.4 branch.


Thank you!!!!


@bluetom I am not sure, what the purpose of changing the context is.  I think system context is not correct in that case. Maybe that is better a better solution:

$context = $cm->context;
line 768 in lib.php


改变


Hi. You must put here field name(maybe **tbl.{$descendantId}**), but you are putting here the value, so we have error of data type. So, it can change count of order of params in line 64. Check it, please


fast-forwarding develop-2.2 to borshop-integration-1434-update-to-bitcask-2.0.3 = 8fc4fa14


Remove


Types


这是为啥


**SUCCESS** - TeamCity HotRod C# Client :: Pull requests monitor - WIN_S2012 [Build 255](http://ci.infinispan.org/viewLog.html?buildId=40882&buildTypeId=HotRodCSharp_PullRequestsMonitorWinS2012&guest=1) for 0b271aab41ba4fb5ae8f573842111c6952250ca6
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:13:52


What does Add = TRUE do?

 Sent from   the future (https://itunes.apple.com/app/apple-store/id922793622?pt=814382&mt=8&ct=future)

>
> On Jul 7, 2016 at 7:40 AM,  <Andrew Martin (mailto:notifications@github.com)>  wrote:
>
>
>
>
> holy crap I just nuked a zillion lines of code by using this ref.
>
>
>
> —
>  You are receiving this because you were mentioned.
>  Reply to this email directly,   view it on GitHub (https://github.com/almartin82/mapvizieR/commit/eb54ab5e1d4bfd63f07ea2240fea1c3db69e3e2f#commitcomment-18157045), or   mute the thread (https://github.com/notifications/unsubscribe/AA7c_8caGvxvpYI1fJM9eVjoViwuRaIaks5qTPPYgaJpZM4JHCUC).
>
>
>
>
>
>
>


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 412](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1129&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:07:46


'../src/Popup'  这里的 web demo 指向了 rn 文件，是否搞错了？


출력에 대한 포맷은 Recoder의 책임이고 주사위 타입의 책임은 FraudDice 클래스이기 때문에 문자열 상수 네이밍 룰을 지켜주시는 것이 좋습니다.

단순히 문자열 상수에 대한 변수 이름을 짓는 측면에서 특수 기호를 빼는 이유도 있지만
이보다 더 중요한 것은 객체들 간의 책임 분리라는 점을 고려하시는 것을 추천드립니다.


Tried with 6.x.x and 5.x.x and didn't work, had to fall back to 0.10
This doesn't force anything, it's only informational about what version of node we should use


I agree. I have been using Mint KDE for years. Now I will have to choose between Cinnamon or Mate. Which one did you go with??? Also which kernel are you running?  Thanks for the input I am gonna give the latest Mint a go. Thanks again.


예 맞습니다. 기본적인 getter, setter, toString은 주석달지 않으셔도 됩니다.
서비스 로직 또는 프로젝트의 요구 사항을 포함하는 메소드라면 주석으로 로직 설명을 넣는 것이 좋습니다.


本当に27年度であってる？


Merge sha 8fc4fa14 is stale.


merging basho/riak_kv/update-to-bitcask-2.0.3 = 583b520c into borshop-integration-1434-update-to-bitcask-2.0.3


제가 syntax라는 표현을 써서 헷갈리셨나 봅니다. 틀린 표현 이었을 수도 있겠네요.
메소드 주석은 아래와 같이 원형 위에 작성합니다.

/*
* 1~6 사이의 무작위 정수를 만든후 주사위의 value에 그 값을 대입합니다.
*/
public void roll() {
...
}


We dont need that many clients. 5 is more than enough. We can even run with just 3.


@NateWr, `$jqueryUI` (here and below) is set but never assigned to the template for use. (And it appears not to be used in the template either.)


first time


Modified at: https://github.com/zetxek/jorani/commit/99f2a1c8fe872e3b006d0eeeec9a460ed4f2dcb1


We should be using bulk index instead of normal index. Maybe we can have both tests separate to compare, but I would expect people to use bulkindex if they want to load a lot of docs.


- I think you should also talk about URL parameters to match incoming requests. Or you can talk about it later when creating resource controllers.


TODO: Create an interface for TrustedComputer entity and rename current TrustedComputerInterface


I would iterate much more. Not sure how long is it taking right now, but I would use something like 10.


Jim, this logic is a model method, right?


create testing files


test


Helpful resources:
- https://docs.python.org/2/library/2to3.html
- https://docs.python.org/2/library/types.html
- https://docs.python.org/3/library/functools.html#functools.cmp_to_key
- https://docs.python.org/3/howto/sorting.html#sortinghowto


Whoops! Spelled canola wrong!


f


Hi @jan-hudec

> Basically I'd need something that would get attitude/relative wind, altitude, speed and control parameter values, made sure the engine values are as if that condition was stable for a while, and then read the forces and moments.

This sounds really interesting to me and a challenge I'd love to tackle.
Quick question: do you want to be able to get this values while running FG? as in, could I do some nasal wrapping that pipes properties into a text file and then we grab that file?


The sleep idea is not bad. Maybe we should do it in the bulk request and not at the end of each iteration. Depending on the ES cluster performance, it will kill the server and we do not implement a back-off. So, I would actually sleep a 2s after each bulk request. This test is not about the ES cluster stress, but about the client stress. We want to run it long enough to be able to spot memory issues.


Whitespace error?


It wouldn't use Pair<String, FilePath> here.


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


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/5124/testReport/) for this commit**


It seems like the ::encode can go outside of the if/else?


Looks great!


That line should be removed (and the one below) should be removed. jQueryUI is not used on the frontend. I can try to get to this later tonight but it may not happen before I go...


Should probably be derr


This doesn't necessarily mean every GBC game will work properly. Currently, GBE+ doesn't send the data fast enough for the fastest supported link speed (64KB/s) but it will work with some of the slower ones (Army Men 2 works, for example).


Oh, I think I see, ignore previous


will this init still be called when we do not run tests?


No problem, I'll get it.


Instead of those two lines, please use `rabbit_ct_broker_helpers:get_node_config(Config, 0, tcp_port_stomp)`.


Why is this marked mutable?


it appears this PR has removed all the current icons. I'm going to revert this.


Thanks, I thought there must be a better way.


For some reason it won't work with docker in action example. No idea what the problem is, but it cant bind.


More generally, we can't stick all of the crcs in the hinfo.  I'd like to stripe the set of crcs over bounded size omap entries.


You should use the hostname from the configuration too, you can get it with:
```erlang
Hostname = ?config(rmq_hostname, Config),
```


@kenichi @digitalextremist can we do something like this for `extension` ? :smiley:


Whitespace.


which command do i use to dump the BIOS?
Thank you for your quick reply


test


日了狗了！


Los métodos deben de estar en ingles !


@bernhard-herzog I have added the fields ```import_source``` and ```import_time``` to the automatic fields. I wasn't sure about the n-m-tables, it might be more  elegant to remove the fields from there


Agent must have a feedback for his verification


If the checkbox is selected and the `students` array includes the ID, then remove the student from the array?


Do you continue this Core fix ?


Yes, but wouldn't it be useful? I believe the alternative is setting it up in CyberSource directly?


@fpietrosanti does this fits your sysadmin taste?

@NSkelsey


So the '<' worked in #if? Awesome.


test comment


problem fixed.


Near as I can tell. https://github.com/cake-build/cake/issues/1032


This seems wrong.  p appears to start at 0 regardless of old_size.


already boot?


The previous link text failed WCAG (Web Content Accessibility Guidelines) [Success Criteria 2.4.4](http://www.w3.org/TR/UNDERSTANDING-WCAG20/navigation-mechanisms-refs.html).


The first time I ran it, it reinstalled fine, despite the fact that I hadn't uninstalled the previous version or removed `.meteor`.

After reinstallation, I deleted `.meteor` and ran it again and it said it was already installed.

I then uninstalled Meteor via Control Panel (which worked fine) and ran the installer which worked successfully.

But, seems like the old behavior?  Oh and it's not signed, but I understand why.

Anyway, the only reason I asked in the first place is I somehow got into a situation where I had no `.meteor` folder, could NOT uninstall, and the installer thought it was installed.  I had to go through Registry Editor (like a boss).


update now


ip packet이 아니라면? eh의 frame_type을 확인하는 로직이 들어 가야 함.


<?xml version="1.0" encoding="utf-8"?>
<MOML>
 <APPLICATIONINFO>
  <START url="start.xml" />
  <ORIENTATION type="defaultPortrait" />
 </APPLICATIONINFO>
</MOML>


`through` expect to receive functions, not streams, check https://github.com/maxogden/mississippi#through

Callback function passed to `concat` would be called with single argument, `cb` on line 16 would be always undefined. Check https://github.com/maxogden/mississippi#concat


Whitespace error


commit ac75e83


uh는 언제 쓰임?


tcp header의 sport랑 dport 찍는 코드가 안 보임.


fixed


应该还是2， 否则第二、三、、、次发布tag时， 可能解析不到comment参数


Markdown incompetence...


Okey, but what dit you change ? Is it now CR or CRLF , because it needs to be CRLF , it is a windows file.
Else, please change the pull request without the changes in the visual studio part, then i will do them


fixed indentation


Amazing!


issues fixed.


Polish -> English :-)


This should be:
`<div class="col-xs-12 col-sm-4">`

I thought I had fixed that is it was working locally but doesn't seem to be fixed in the commit, sorry!

You can see it doesn't work on a desktop when you are viewing at a Mobile size and changing this fixes it.

Cheers, Chris


for (i = 0; i < header->caplen; i++)


printf("%.2x ", pkt_data[i]]);


This is surely very childish! Will try putting Data Base once I acquire the required backend skills!


what are thooose? XD


to English or remove


Sounds like ffmax argument should be removed from this static function,
or we should have changed only parent_len type.


changed `get_option` to `get_any_option` to allow time-dependency to be generalised for all constraints with possible time dependency.


text="hi"


please use  mini synchronized ,method synchronized is so Coarse－grained


<BUTTON id="btn2" themeId="listButton">
<LABEL themeId="listItemText" text="Camera"/>
<LABEL themeId="listItemArrow"/>
 </BUTTON>
 <BUTTON id="btn12" themeId="listButton>
<LABEL themeId="listItemText" text="Uncategorized..."/>
<LABEL themeId="listItemArrow"/>
 </BUTTON>


Yes, look:
https://wiki.sa-mp.com/wiki/Keywords:Directives#.23if


do you have this automated? It's a fascinating concept


packet debugging을 쉽게 하기 위해 "port 80"으로 filter를 거는 것은 상관 없음. 하지만 packet_handler에서는 "port 80"이라는 filter를 믿지 말고 filter에 걸리지 않는 패킷도 들어 올 수 있다라고 생각하고 코딩할 것. 즉 "ip 맞나? tcp 맞나?"라고 판단하는 로직이 packet_handler에 코드 레벨로 들어 가야 함.


th는 사용이 안되는 것 같은데?


`listed` typo


But, we can enforce this function adhere to an interface -- for instance return 'ok' or '{error, Reason}'.


**Fixed Bug 1279463** - [windows] Two download buttons displayed in release notes footer


K!


Thanks @Zooke for reporting.


Improved `Get ` methods to skip over blank lines to get the last cells - before it just took the last one within a continuous chunk.


Good job!


I would call this `--dry-run`; it's more conventional to uses `-` in long option names than `_`.


あ


▶ Review started


✅ Approved


Send a PR for this?


Cool, that helped! Seems to be `xs` in most places but you might've missed this one.. no biggie!

Also note that the release notes for all releases are now at the bottom, they could contain important information.

We should split up the actual release notes and the upgrade instructions into two fields.. but that's a bigger task, would need to go in and change the content for each release then.. :o


The location in the PATH is inside of the string join...


Wasn't this the real reason why 4 bytes of the DATA out stage were lost?  This pops an additional 4 bytes from the queue if the length was 8 byte, e.g. a setup packet received.


Which commits are you referring to?  How do I use github to find the
graph you are showing?

Barb

> MKLau <mailto:notifications@github.com>
> July 6, 2016 at 11:42 AM
>
> @tfjmp <https://github.com/tfjmp> @blernermhc
> <https://github.com/blernermhc>
>
> Looking over the diffs, I think we might want to re-do these commits
> as commits to *development* and rollback *master* to avoid conflicts
> when merging *development* into *master*.
>
> Here's the current state of the branches:
>
> screen shot 2016-07-06 at 11 40 34 am
> <https://cloud.githubusercontent.com/assets/5342857/16624172/ae4ef1d6-436e-11e6-8c59-6f061d380fcf.png>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/End-to-end-provenance/RDataTracker/commit/790080e861ef8d841bb99693db4ddd183c3b8508#commitcomment-18144068>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AFLado5BOJgCSCVfJbLaLn9R6Vivbvguks5qS8zYgaJpZM4JGLtj>.
>

--
<http://www.getpostbox.com>Barbara Lerner
Associate Professor
Computer Science Department
Mount Holyoke College

<http://www.getpostbox.com><http://www.getpostbox.com>


Yes I agree, I thought the same as there is quite a bit of repetition, still it's a good improvement and something we can look to improve when we have a bit more spare time, thanks for your help merging in and doing the code cleaning.


A test comment


@bellbind / @Joris-van-der-Wel
I just pushed the changes to the WIP ... still only for testing the new functions and their correctness.
I left in older functions for debugging purpose, renamed as "convertEscapes2" and "unescapeIdentifier2".
For the same reason I also exported temporarily "Dom.codePointToUTF16()" and "Dom.stringFromCodePoint()".
Need suggestions, feedback on implementation and possible comments correction where you think is adequate.


There's a better method that doesn't involve changing the component

you can set iconLeft/iconRight as following
``` javascript
iconLeft={((field)=>{
  if(field.valid){
    return <Icon name='my-icon' />
  }
})(this.refs.myform.refs.myfield)}
```

Here it is


A slightly better fix in renderer.php would be:

`html_writer::tag($tagname, $contents);`
in this example:

`$output .= html_writer::tag('span', get_string('groupby', 'mod_newsletter'));`


About the interface: let's say I have `category` and `action` specified. How can I get the command I'm looking for? There is only filtering by category in `list/2` function introduced


I've just checked: also the SPSA tuning session was perfectly normal. I reset the priority to 0, but if you have any idea, I will check it.


Muppet


Ooh, I like that. I'll have to modify the script to work with that.


check out https://github.com/DatabaseCleaner/database_cleaner and https://github.com/thoughtbot/factory_girl as well


@triceo Turns out it was possible to get rid of this line in the solver config etc ;) It's indeed much nicer now to build a hello world etc.


Commit: 790080e861ef8d841bb99693db4ddd183c3b8508 (the one the comment is attached to).

To see this graph, on github, on a repository page, on the tabs bellow the repository name (e.g. End-toEnd-Provenance/ RDataTacker) click on the `Graphs` tab, then on the set of tabs that will appear showing various graphs select `Network`.


It only very recently (post TP5) made it into Windows. It's on the list to hook up in docker before RTM


Very minor: I think a better commit subject would be something like "Remove redundant conditions from British Virgin Islands outcome template" instead of "Update outcome for British Virgin Islands".


Only appears when "clean network" is selected


can we include the QAE.env?


I misread Web IDL, I thought only classes could be made [nullable](https://www.w3.org/TR/WebIDL/#idl-nullable-type). :)

If a platform exposes a timeout in its API, I would expect that setting the timeout at the WebApi level would manage the timeout at the platform API level accordingly.

I do agree that we still have one case left, which is what if the platform has an hardcoded and/or non-modifiable timeout? In this case, this could/should still throw the timeout error as if it was thrown by the WebApi. However, this would mean the developer can't rely on the WebApi for timeout mechanism. In any cases, whether there is or isn't a field to change the timeout on this kind of platforms does not change a thing.


ok


(C) or ©
Fix your template :)


Yep, will do.


No worries, thanks for the contribution!!


已看


(C) or ©


See test/16.binarraybuffer.js

```js
// run this test when Uint8Array is available
var describeSkip = HAS_UINT8ARRAY ? describe : describe.skip;

describeSkip(TITLE, function() {
```

This is good to tell testers that those tests exist but are skipped by a reason.

`if (usemap) { it(...) }` will just remove those tests when Map is not available.


Huh, an extruder is_used if `machine_nozzle_size > 0`?


probably wanna move this line to after the next error check


The first eye of github 。。。。


I see you are using the X-Accel-Redirect. Awesome! I am looking forward to hearing how this is working out.


Is it possible to introduce some generic interface instead of LRUMap to allow using of custom cache implementations like Guava Cache, Caffeine, etc. ?


▶ Review started


En c++ recomiendan usar ++i por cuestiones de rendimiento, ya que al hacer i++ se genera una copia temporal de la variable en cada incremento para guardar el valor antes del incremento y poder usarlo.


✅ Approved


I guess the idea was that having nodes send their data to a different machine was fairly generically useful. I can see that implementing this in classes derived from Node could be a better idea though. Did you see my email? I'm still stuck on that issue.


What's that? :+1:


I just noticed this commit will make new bugs.

Both HeldItemIndex (hotbarIndex) and HeldItemSlot (the inventory slot related to the hotbarIndex) need to be set before sending the held item but setHeldItemIndex() will send the held item before setHeldItemSlot() sets the new related slot.

So, this commit will set the HeldItemSlot before setting the new hotbarIndex and this will cause a new bug:

HOW TO REPRODUCE THE NEW BUG:

- Delete the "players" folder
-  Start the server
- Join the server with a friend and make sure you are in GameMode 0
- Your friend should be not too far from you
- Your hotbarIndex should be 0 , change it to a new random one and remember the random one
- Break a block and pick up the dropped item
- Your friend can see your held item: air and this is right because the picked up item is in the hotbarIndex 0
- Change your hotbarIndex from the random one to 0 (where the picked up item should be)
- Your friend can't see the picked up item :/
- Now change hotbarIndex to the random one you choose before
- You are holding air but your friend will see the picked up item

I think setHeldItemIndex() shouldn't send the MobEquipmentPacket and sendHeldItem() should be called after both setHeldItemIndex() and setHeldItemSlot() are called.

Hope someone will understand my explanation and if needed i will make a pull request


Hi Barb, I find the Network to be a great way to get a handle on where all the branches are at and trace back to divergences.

My thinking here is that this code that got merged into master directly (and not into development) might create merge conflicts for us later when we merge development into master.

I should be able to wrangle this commit into development and roll back master prior to merging development into master.

To go back to Thomas' initial question though, is there any reason that that merge needed to go straight into master? Like, do you or does someone else need it for something that they're doing?


also, shouldn't this function return the whole `module.exports`, otherwise how do you access `plugins`?


The best way to install pyinstaller is within your Python virtual environment, IMO. If you do not have a virtual environment, you can install it system wide.

    pip install pyinstaller

You do not need to edit or overwrite pyinstaller's own files. You can put the modified hooks in you project, and pass command line options to the pyinstaller command so it can find them when building your application. In my case, I created a `pyinstaller-hooks` directory in my project and generated the spec file with the following command:

    pyinstaller --onefile --windowed \
                --additional-hooks-dir pyinstaller-hooks \
                --runtime-hook pyinstaller-hooks/pyi_rth__tkinter.py my_app.py

Inside that folder I placed both the modified `hook-_tkinter.py` and `pyi_rth__tkinter.py` files.

I believe you can also edit the spec file and put the `hookspath` and `runtime_hooks` options in the Analysis object, in case generating a new spec file is not an option. This is the relevant part.

    a = Analysis(['my_app.py'],
                 ( … snip … )
                 hookspath=['pyinstaller-hooks'],
                 runtime_hooks=['pyinstaller-hooks/pyi_rth__tkinter.py'],
                 ( … snip … )


[R/calc_nltt_stats.R:41:1:](https://github.com/richelbilderbeek/Cer2016/blob/c7532b39aa6538e8499cdaa10bedcdbbb9ce7cb0/R/calc_nltt_stats.R#L41) *style:* **lines should not be more than 80 characters.**
```r
  diff_nltts$nltt <- abs(diff_nltts$nltt - rep(phylogeny_nltts$nltt, length(others)))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/calc_nltt_stats.R:42:1:](https://github.com/richelbilderbeek/Cer2016/blob/c7532b39aa6538e8499cdaa10bedcdbbb9ce7cb0/R/calc_nltt_stats.R#L42) *style:* **lines should not be more than 80 characters.**
```r
  nltt_stats <- dplyr::summarize(dplyr::group_by(diff_nltts, id), nltt_stat = mean(nltt))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/calc_nltt_stats.R:42:3:](https://github.com/richelbilderbeek/Cer2016/blob/c7532b39aa6538e8499cdaa10bedcdbbb9ce7cb0/R/calc_nltt_stats.R#L42) *warning:* **no visible binding for global variable ‘nltt’, Did you mean 'next'?**
```r
  nltt_stats <- dplyr::summarize(dplyr::group_by(diff_nltts, id), nltt_stat = mean(nltt))
  ^~~~
```
[R/calc_nltt_stats.R:42:62:](https://github.com/richelbilderbeek/Cer2016/blob/c7532b39aa6538e8499cdaa10bedcdbbb9ce7cb0/R/calc_nltt_stats.R#L42) *warning:* **no visible binding for global variable ‘id’, Did you mean 'if'?**
```r
  nltt_stats <- dplyr::summarize(dplyr::group_by(diff_nltts, id), nltt_stat = mean(nltt))
                                                             ^~
```


The strings benchmarks do not appear to be particularly thorough. The total "test" content of `BaseBenchmarks/src/string/StringBenchmarks.jl` is:
```jl
str = join(samerand('a':'d', 10^4))

SUITE["replace"] = @benchmarkable replace($str, "a", "b")
SUITE["join"] = @benchmarkable join($str, $str)
```

It would be great to have more complete benchmarks, especially given all the churn in the world of strings and some reports (which I can't find right now) of performance regressions.


Woot!


hongouts 始终无法拨打电话


why have you removed this?


Matthew, you may want to delete the commit all together rather than revert (it leads to a "cleaner" history)


@shorowit I think it's correct (correct in the sense of matching sim.py/energyplus.py). Look at the `_processInteriorShadingSchedule` definition in sim.py.

Also, a search on "window_shade_multiplier" in sim.py and energyplus.py shows that it is never used there either.

cc @ejhw


It seems plausible to me that template expressions should have any type 'a that's needed: they could even have type Bot, since execution cannot progress past a template.


#1 #3 fixed


I was just going to give them any type that's needed.


Why is this necessary? Could just add the current cards to the setcode


I would rename this to `NO_DEMAND`.


likewise, I would rename this to `DEMAND`.


Are you referring to #1390 ?


Yup, great. Ditto for synthesis, below?


Yep.


Thx @alex40724


Fixed https://github.com/geonetwork/core-geonetwork/commit/4f734fb55bd9250bfa83b837cb934632ade0fd2b


whoops, forgot to rename column here


Ah I see, fixed column name here!


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#K_FZUkYeRu-sbQcYJoro2w


@riastradh-probcomp suggest to merge in this commit into your feature branch 20160624-riastradh-cgpm (and fixing anything I might have broke!). I ran ./check.sh and it seems that nothing complained because of this commit.


Not I ...


closed #26


No. I'm trying to avoid bumping up "milestone" number, but I still need to publish artifacts, so I made up an alphabet. X stands for experimental.


The following doc(s) have been deployed to the [ataylormea](https://ataylormea-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/alternate-docroot](https://ataylormea-static-docs.pantheonsite.io/docs/alternate-docroot)


is_display_on is a function, not a variable.


ok


oups, sorry guy !
@antonylesuisse


TeamCity OTA+ :: rvi_sota_server :: ci [Build 408](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7195&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


oh god i mean 4 spaces please dont use 4 tabs


This is different. I think we want that in other languages too, that means that it's XWikiScriptContextInitializer job should take care of not creating a new instance of api.Document if the document did not changed in the context.


could just as well use `site.data.nav.main`, whichever we like more...


done in 190e667


Wow! Such commit! Very amaze!


Excellent thanks.  Good idea.

> On Jul 7, 2016, at 7:25 AM, Eric Rasche <notifications@github.com> wrote:
>
> Remote user proxy
>
> cc @nathandunn built a small dockerfile for remote_user proxying
> Should help with my/your/our testing of remote_user things.
>
> I'm working up a branch of docker-apollo which supports remote user
>
> ---
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub:
> https://github.com/erasche/docker-recipes/commit/17d5e6ae56134ae7e8d79d34a1ca0123347ad560


just testing if the `|contains`-filter also works with strings.


Solved: sending the request headers first (PATCH Bad Request fixed)


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#fRRUYEU4R7OcbIMj5h6_tQ


why not just use `std::unique_ptr<size_t[]>`?


@joekiller I had to update check, similar to how it's done in `gopacket` because the previous version caused panic when running tests on Mac.


Can you please make it a strict "<"? The performance would be better on the edge case where the record has 256 fields.


the comment is uncorrect,it should be switching database from mysql to oracle.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 408](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7195&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 176, ignored: 3 Build time: 00:09:20
### Failed tests
```
org.genivi.sota.core.PackageResourceWordSpec.Package resource should list resources on GET request: <no details avaliable>

org.genivi.sota.resolver.test.FirmwareResourceSpec.Firmware resource should be able to accept installed software updates: <no details avaliable>

org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should be able to assign exisiting filters to existing packages: <no details avaliable>

```


transformative


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#acDLMIUiRsabWw6b2Bgw1A


Please take another look


"...and darkness came over all the land" :sos:


@jlowin I'm new to the airflow project, so forgive me if this is out of line but, are these changes compatible with the normal (as of 1.7.1.3) way of using executors and plugins, or is this part of a larger effort that isn't ready yet?  Running off of master seems to break things in `airflow/executors/__init__.py`.  I'm getting an error saying my plugin executor is not supported, and I've traced it back to this.  Thanks!


LGTM!


@ajithcj  thanks! Indeed this could be moved to evaluate_king_safety() if we want to:
• either at the end of evaluate_king_safety() to keep the same bench,
• or at the beginning of evaluate_king_safety() before "Main king safety evaluation", with the unexpected property that this would change the `attackUnits` count because of the dependency of attackUnits to the midgame value of the king shelter and enemy pawns storm score.


For Mac compat, I believe this will need to be `PrintingUtilities.getPrinterId()`


if you want to remove a node which sets itself in the on_destruct function in the node definition, minetest.remove_node isn't strong enough, then you need to use vmanip


does work, but the maximum zoom level is a bit too low
i.e. in a 45 min scenario I cannot zoom further than 5 seconds graduations (see attached screenshot) - I'd like to be able to work at least 5 times more precisely... is that doable ? maybe the max zoom level could be settable in the user prefs ?
![Uploading zooom.png…]()


yes this commit will assign a wrong hotbarIndex but i don't think have something to do with 1390


TeamCity OTA+ :: rvi_sota_server :: ci [Build 409](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7203&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


<img width="768" alt="zoooooom" src="https://cloud.githubusercontent.com/assets/113982/16659716/988368c0-446b-11e6-9a53-97aeba0864e0.png">


hmm. I'll look into it.


Ah, it was probably only used for DOE-2 then. @joseph-robertson I recommend we remove window_shade_multiplier code from the measure.


Issues with getting the jquery timeline plug in to work


Nit: This is true because we only use this api when computing invoice but from an api point of view, this is a bit broken. Keeping your change probably still make sense (i think we care more about optimized code that cleanliness of internal apis). I guess this is a matter of semantics too (or naming of the api).


Nice, good luck for LTC!


I'm not 100% sure what this is supposed to do but it breaks tests
```
testExecAsWithGroups(org.fao.geonet.services.user.UserListIntegrationTest)  Time elapsed: 0.205 sec  <<< ERROR!
java.lang.NullPointerException: null
	at org.fao.geonet.listeners.security.CheckUsername.check(CheckUsername.java:45)
	at org.fao.geonet.listeners.security.CheckUsernameCreation.onApplicationEvent(CheckUsernameCreation.java:43)

```

Fixing it by adding null checks. Is it fine for you ?


This part seems a bit weird to me. I can see that you overriding this method in `ServletHttpHandlerAdapter`, but I still think there should be a nicer way to solve this.


It is ok


Looks good overall!


TeamCity OTA+ :: rvi_sota_server :: ci [Build 409](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7203&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests passed: 73, ignored: 2; compilation error: Scala compiler in Test (new) Build time: 00:06:49


What happens before the save is picked? e.g. client is still at main menu?

Should this be an Optional?


@youzipi arrow functions only have inferred function names, and can't have an explicit syntactic name.


Is it easy to past a fail log?


うえーい


@chrisroos: Thanks. I created a [Trello card][1] to capture this.

[1]: https://trello.com/c/dMX2wO9d


это лишнее. может быть и null


@cro are you sure this works as expected? I get a `BadStatusLine: ''` exception after a few requests. Do you get this?


смешно. надо было еще на украинском написать   =)))


> Liquidbounce has 2 devs and here are their email addresses:
>
> senkju4@gmail.com
> Marco.MC0902@gmail.com (Lead Developer)

I saw that. I will try contacting Marco.MC0902@gmail.com once Google sends me the final "Delivery Failed" notice.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 410](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7209&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


описания пусть всем плагинам напишут менеджеры


Decomposed on separate modules and made code mooore clear.
Added error handling and comments.


When will changes be published to npm?  Last time this repo was published a month ago.


Hi,
We've tried to provision 60 servers into Azure ARM using the latest version of puppetlabs-azure module, with bootstrapping using extensions. This doesn't seems to work as expected. The module doesn't even recognize paramter called extensions. We'd prompt appreciate help on this, thanks!

Error: Could not retrieve catalog from remote server: Error 400 on SERVER: no parameter
named 'extensions' at /etc/puppetlabs/code/environments/feature_win_provisioning/modules/provisioning/manifests/azure/windows/arm_vm_devtest.pp:13 on Azure_vm[azw-prov-001] at
/etc/puppetlabs/code/environments/feature_win_provisioning/modules/provisioning/manifests/azure/windows/arm_vm_devtest.pp:13 on node azw-test-001
Warning: Not using cache on failed catalog

) {
  azure_vm { $title :
    ensure               => $ensure,
    location             => $location,
    subnet_name          => $subnet_name,
    image                => $image,
    size                 => $size,
    user                 => $user,
    password             => $password,
    virtual_network_name => $virtual_network_name,
    resource_group       => $resource_group,
    extensions           => {
      'CustomScriptExtension' => {
        'auto_upgrade_minor_version' => true,
        'publisher'                  => 'Microsoft.Compute',
        'type'                       => 'CustomScriptExtension',
        'type_handler_version'       => '1.7',
        'settings'                   => {
          'commandToExecute' => 'powershell.exe -ExecutionPolicy Unrestricted -File bootstrap_v2.ps1',
          'fileUris'         => ['https://someblob.blob.core.windows.net/scripts/Bootstrap_v2.ps1']
        },
      },
    },
  }
}


> It would be great to have more complete benchmarks

Both strings and [IO](https://github.com/JuliaCI/BaseBenchmarks.jl/blob/master/src/io/IOBenchmarks.jl) are woefully under-tested by our existing suite, especially compared to our indexing/linalg benchmarks.


player is better than pl


here!


I'd put this into params rule ...


Protip: `$(function() { ... })` is a shortcut for `$(document).ready(function() { ... })`


@supern 这里是不对的


This is strange 😁


> The contract for it is any repo returned by the store goes through it.

I don't disagree, but can you please point me to where you found that contract written?


I am not having any more luck with pushing the cps conversion through while only depending on the length.


I think the update of official cards should conflict with pre-release cards, so cards.cdb shouldn't include any unofficial data.

see https://github.com/Fluorohydride/ygopro-core/commit/7058e371fcfb77f1da6c4b69fefc1dc76cee8bd2


Did you create this organization?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 413](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1130&buildTypeId=XbimEssentials_XbimEssentials) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 410](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7209&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:08:18


/cc @jcredding, see #26 for reference.


I'll plan to take a look at it when I get in to google in like 15 minutes.


```
debian@beaglebone:~$ dmesg | grep bone
[    0.000000] Kernel command line: console=ttyO0,115200n8 bone_capemgr.enable_partno=am33_pruss_rproc root=/dev/mmcblk0p1 ro rootfstype=ext4 rootwait coherent_pool=1M quiet cape_universal=enable
[    2.261788] bone_capemgr bone_capemgr: Baseboard: 'A335BNLT,,BBG115081143'
[    2.261823] bone_capemgr bone_capemgr: compatible-baseboard=ti,beaglebone-black - #slots=4
[    2.300671] bone_capemgr bone_capemgr: slot #0: No cape found
[    2.344630] bone_capemgr bone_capemgr: slot #1: No cape found
[    2.388630] bone_capemgr bone_capemgr: slot #2: No cape found
[    2.432630] bone_capemgr bone_capemgr: slot #3: No cape found
[    2.438443] bone_capemgr bone_capemgr: enabled_partno PARTNO 'am33_pruss_rproc' VER 'N/A' PR '0'
[    2.438456] bone_capemgr bone_capemgr: slot #4: override
[    2.438469] bone_capemgr bone_capemgr: Using override eeprom data at slot 4
[    2.438483] bone_capemgr bone_capemgr: slot #4: 'Override Board Name,00A0,Override Manuf,am33_pruss_rproc'
[    2.438814] bone_capemgr bone_capemgr: initialized OK.
[    2.447185] bone_capemgr bone_capemgr: slot #4: dtbo 'am33_pruss_rproc-00A0.dtbo' loaded; overlay id #0
[   10.548396] LUN: removable read only file: /var/cache/doc-beaglebonegreen-getting-started/beaglebone-getting-started-2016-04-20.img
```

&

```
debian@beaglebone:~$ dmesg | grep pru
[    0.000000] Kernel command line: console=ttyO0,115200n8 bone_capemgr.enable_partno=am33_pruss_rproc root=/dev/mmcblk0p1 ro rootfstype=ext4 rootwait coherent_pool=1M quiet cape_universal=enable
[    2.438443] bone_capemgr bone_capemgr: enabled_partno PARTNO 'am33_pruss_rproc' VER 'N/A' PR '0'
[    2.438483] bone_capemgr bone_capemgr: slot #4: 'Override Board Name,00A0,Override Manuf,am33_pruss_rproc'
[    2.447185] bone_capemgr bone_capemgr: slot #4: dtbo 'am33_pruss_rproc-00A0.dtbo' loaded; overlay id #0
[    3.003327] ti-pruss 4a300000.pruss: platform data (reset configuration information) missing
```


see #2727


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 413](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1130&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:03:34


what is the effect if this change?


Isn't there a msbuild task that does create dirs?

```<MakeDir Directories="$(OutputDirectory)"/>```


I think this is unnecessary. Microbuild does this already (there is a checkbox for cleaning in the web ui).


I wonder if the problem here is that the install file was not executable. My understanding of debhelper and dh-exec is that when the config file is executable, then debhelper will execute it rather than reading it. From dh-exec(1) - dh-exec is a simple program, meant to be used as the interpreter for executable debhelper config files. Notice the executable part. Also from debhelper(7) in the `DEBHELPER CONFIG FILES` section - If you prefer power and complexity, you can make the file executable, and write a program that outputs whatever content is appropriate for a given situation.

My guess is that debhelper handled this in the normal way with dh_install and it just did the wrong thing.

BTW, if you want to go this way, maybe you can drop dh-exec from Build-Depends.


Should probably be a config agnostic setting.


Is it wise to have all those underscores? Do the conflict otherwise?


TeamCity Units.NET :: CI [Build 127](http://teamcity.anj.no/viewLog.html?buildId=636&buildTypeId=UnitsNet_CI) is now running


Good


I'm unsure - I generated these automatically from `/usr/src/sys/kern/syscalls.master` with

```python
for line in master:
    tokens = line.split(' ')
    id = int(tokens[0])
    name = ' '.join(tokens[5:]).split('(')[0] if len(tokens) > 4 else tokens[-1]
    print id + '\t' + name
```


TeamCity Units.NET :: CI [Build 127](http://teamcity.anj.no/viewLog.html?buildId=636&buildTypeId=UnitsNet_CI) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 729; artifacts size (bytes) is too small: 10953454 (new) Build time: 0:0:32
### Failed tests
```
UnitsNet.Tests.dll: UnitsNet.Tests.CustomCode.DensityTests.DensityTestsBase.ConversionRoundTrip: <no details avaliable>

UnitsNet.Tests.dll: UnitsNet.Tests.CustomCode.DensityTests.DensityTestsBase.FromValueAndUnit: <no details avaliable>

```


Which behaves differently on Cygwin.


@Kofotsmordarn Säger den som döper sina commits til "GUI wohooooo mera GUI ahhhh nom nom asdf massor av GUI mega GUI on the road"


Not sure if there is much value in extracting a bunch of tasks in a separate project if the name of the project doesn't tell anything about what it does (```SetupStep1```, ```SetupStep2```). It would be imo better to just include the content here.


@gkiko მგონი მაინც არ ბრუნდება CORS ჰედერები, http://docs.airconditioningswitch.apiary.io/#reference/0/air-conditioning-status/get-air-conditioning-status?console=1
აი აქ production url-ზე რესურსის ქოლის მერე რესფონსის ჰედერებში არ წერია.


数列的に処理
自分的には好き


これ好き


Is that the loadDDG branch you are referring to?  Isn't it already
reverted?  (It was accidental.)

> Thomas Pasquier <mailto:notifications@github.com>
> July 7, 2016 at 11:22 AM
>
> Matthew, you may want to delete the commit all together rather than
> revert (it leads to a "cleaner" history)
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/End-to-end-provenance/RDataTracker/commit/790080e861ef8d841bb99693db4ddd183c3b8508#commitcomment-18160165>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AFLadrBYkNKUd3cBCjjQtqwWUOMZCG2kks5qTRnLgaJpZM4JGLtj>.
>

--
<http://www.getpostbox.com>Barbara Lerner
Associate Professor
Computer Science Department
Mount Holyoke College

<http://www.getpostbox.com><http://www.getpostbox.com>


მაგას დაანებე ცოტა ხანი. ერორზე გადის


ok


Not sure why my commits are not showing up on the profile page