Greetings!
@dongahn @simoatze @jprotze
This is the similar implementation of the task that I did last time. This is to check if I'm in line with your vision. This happens to be a naive case but I intended to use this if I should continue with the other case as well?

> _input.c_
```
#include <stdio.h>
#define N 1000

int main (int argc, char **argv)
{
  int a;
  int b;
#pragma omp parallel for
{
  for (int i = 0; i < N - 1; i++) {
    a=i;
  }
}

#pragma omp parallel for
 {
  for (int i = 0; i < N - 1; i++) {
    b=i;
  }
 }
  return 0;
}
```

And this is the output I generate
```
==================
WARNING: ThreadSanitizer: data race (pid=16999)
  Write of size 4 at 0x7fff5ddc582c by thread T4:
    #0 .omp_outlined. /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:11:6 (a.out:x86_64+0x100017e6e)
    #1 __kmp_invoke_microtask <null> (libomp.dylib:x86_64+0x42022)

  Previous write of size 4 at 0x7fff5ddc582c by thread T3:
    #0 .omp_outlined. /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:11:6 (a.out:x86_64+0x100017e6e)
    #1 __kmp_invoke_microtask <null> (libomp.dylib:x86_64+0x42022)

  Location is stack of main thread.

  Thread T4 (tid=328802, running) created by main thread at:
    #0 pthread_create <null> (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x74ae)
    #1 __kmp_create_worker <null> (libomp.dylib:x86_64+0x21db7)
    #2 start <null> (libdyld.dylib:x86_64+0x5254)

  Thread T3 (tid=328801, running) created by main thread at:
    #0 pthread_create <null> (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x74ae)
    #1 __kmp_create_worker <null> (libomp.dylib:x86_64+0x21db7)
    #2 start <null> (libdyld.dylib:x86_64+0x5254)

SUMMARY: ThreadSanitizer: data race /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:11:6 in .omp_outlined.
==================
current_report = 0x104503568 : Unique
==================
WARNING: ThreadSanitizer: data race (pid=16999)
  Write of size 4 at 0x7fff5ddc582c by main thread:
   TSAN REPORT SUMMARY - BLAH
SUMMARY: ThreadSanitizer: data race /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:11:6 in .omp_outlined.
==================
current_report = 0x104506ba8 : Duplicate of 0x104503568
==================
WARNING: ThreadSanitizer: data race (pid=16999)
  Write of size 4 at 0x7fff5ddc582c by thread T2:
    TSAN REPORT SUMMARY - BLAH
SUMMARY: ThreadSanitizer: data race /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:11:6 in .omp_outlined.
==================
current_report = 0x10450a1e8 : Duplicate of 0x104503568
==================
WARNING: ThreadSanitizer: data race (pid=16999)
  Write of size 4 at 0x7fff5ddc5828 by main thread:
    #0 .omp_outlined..1 /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:18:6 (a.out:x86_64+0x10001811e)
    #1 __kmp_invoke_microtask <null> (libomp.dylib:x86_64+0x42022)
    #2 start <null> (libdyld.dylib:x86_64+0x5254)

  Previous write of size 4 at 0x7fff5ddc5828 by thread T4:
    #0 .omp_outlined..1 /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:18:6 (a.out:x86_64+0x10001811e)
    #1 __kmp_invoke_microtask <null> (libomp.dylib:x86_64+0x42022)

  Location is stack of main thread.

  Thread T4 (tid=328802, running) created by main thread at:
    #0 pthread_create <null> (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x74ae)
    #1 __kmp_create_worker <null> (libomp.dylib:x86_64+0x21db7)
    #2 start <null> (libdyld.dylib:x86_64+0x5254)

SUMMARY: ThreadSanitizer: data race /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:18:6 in .omp_outlined..1
==================
current_report = 0x104506ba8 : Unique
==================
WARNING: ThreadSanitizer: data race (pid=16999)
  Write of size 4 at 0x7fff5ddc5828 by thread T2:
   TSAN REPORT SUMMARY - BLAH
SUMMARY: ThreadSanitizer: data race /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:18:6 in .omp_outlined..1
==================
current_report = 0x10450a1e8 : Duplicate of 0x104506ba8
==================
WARNING: ThreadSanitizer: data race (pid=16999)
  Write of size 4 at 0x7fff5ddc5828 by thread T3:
   TSAN REPORT SUMMARY - BLAH
SUMMARY: ThreadSanitizer: data race /Users/Sumedh/Software/ARCHER_dev/test_cases/test2.c:18:6 in .omp_outlined..1
==================
current_report = 0x10450d828 : Duplicate of 0x104506ba8
ThreadSanitizer: reported 6 warnings
Abort trap: 6
```

I've edited the output to wherever there was repeated info to ***TSAN REPORT SUMMARY - BLAH*** so as to not waste your time.

Does this look good to you?
I've used C++ STL map to reduce the search time @dongahn .

Should I proceed ahead with the other cases or do you intend to have some other changes as well in the output generated?


Hi, Great job with the package ! Is the code for multi-brokers case complete? I've tryed to use this but I had an error  `LoadError: MethodError: no method matching Kafka.KafkaClient(::TCPSocket, ::Int64, ::Dict{Int64,Type}, ::Dict{Int64,Channel{Any}})` when run the line `kc = KafkaClient(sock, 0, inprogress, results)`. Would you be so kind and tell me where there might be a problem?


@shanmdphd ;) This is a fun package, is it?


So where are we displaying public training ads now?


OK, I don't mind. Although I'd give those magic numbers mnemonic names for better understanding.


我买了电脑了哈哈


OK, thanks. Sorry about that.


你这是用Git hub 开微博的节奏了，哈哈哈


Master branch: new commits are available.


Can we get this fix into 1.7 please?


My idea was to display the private ad only when we don't have a public one (e.g. right now)


I found it via a build failure (see extract below, project path anonymized).

node in this case is v0.10.11

09:46:49 > node --harmony ./node_modules/gulp/bin/gulp.js karma
09:46:49
09:46:49
09:46:49 /project/node_modules/gulp/node_modules/gulp-util/node_modules/through2/node_modules/readable-stream/lib/_stream_writable.js:73
09:46:49 }
09:46:49 ^
09:46:49 SyntaxError: Variable 'WriteReq' has already been declared
09:46:49     at Module._compile (module.js:439:25)
09:46:49     at Object.Module._extensions..js (module.js:474:10)
09:46:49     at Module.load (module.js:356:32)
09:46:49     at Function.Module._load (module.js:312:12)
09:46:49     at Module.require (module.js:364:17)
09:46:49     at require (module.js:380:17)
09:46:49     at Object.<anonymous> (/project/node_modules/gulp/node_modules/gulp-util/node_modules/through2/node_modules/readable-stream/lib/_stream_duplex.js:30:16)
09:46:49     at Module._compile (module.js:456:26)
09:46:49     at Object.Module._extensions..js (module.js:474:10)
09:46:49     at Module.load (module.js:356:32)
09:46:49


Eh, it will leave us with a codebase that doesn't match this repo.  So a pain of changing all of the davit calls for a few hours, or the unforseeable length of pain of having to keep a diverged copy of this code.  I'll take the former for $400 Alex.  I'm more confused on why it didn't throw errors until I merged in the pull requests yesterday.  There may be a thing or two more falling out from this though.  Stay tuned.


TeamCity OTA+ :: libats :: ci [Build 15](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26005&buildTypeId=libats_Ci) is now running


csparse


I have found that once you execute the file there is no removing it. In order to switch back to edge, you not only have to change the default browser, but you must leave edgedeflector in Program Files (86). When I removed the program, clicking on Cortana files does nothing even with edge selected as the default. I can switch back to edge, but must leave the program.


Please make the following changes:

- Name variables and functions relevant to what they are
- Remove too many extra new lines
- Remove commented code
- one space before and after every operator `eg: a=b+c; //should be a = b + c;`
- Make one function to read file and take _filename_ as a parameter in that function


TeamCity OTA+ :: libats :: ci [Build 15](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26005&buildTypeId=libats_Ci) outcome was **SUCCESS**
Summary: Running Build time: 00:00:45


Maybe it's already fixed, but this appears although you had changed the spacing settings in scrutinizer.yml


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.behavior/6/violations
```
plone/behavior/tests.py:70:40: Q000 Remove bad quotes.
plone/behavior/tests.py:70:63: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:26:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:27:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:31:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:32:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:36:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:37:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:41:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:42:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:43:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:47:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:48:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:52:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:53:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:54:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:58:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:59:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:60:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:61:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:65:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:66:15: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:67:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:68:21: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:72:1: C901 'behaviorDirective' is too complex (13)
plone/behavior/metaconfigure.py:84:13: Q000 Remove bad quotes.
plone/behavior/metaconfigure.py:133:17: Q000 Remove bad quotes.
plone/behavior/interfaces.py:88:15: Q000 Remove bad quotes.
plone/behavior/registration.py:9:1: I003 isort expected 1 blank line in imports, found 0
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


The rest is great!


I'm good with the release version, I only installed the development one because I missed the part in the installation guide saying not to pass arguments to the script. You might want to rethink this, though, given that the official line is Python >= 2.6 < 3.0.


See New push.


Any chance that you could add support for the ESP32 architecture?


I implemented aacgm_v2 functionality in the davit version of cnvcoord, so it's possible this is the first time someone has tried calling a function which passes a v2 keyword to cnvcoord which will return the error.


Sorry, I just saw this came after the commit here: https://github.com/chamilo/chamilo-lms/pull/1742#issuecomment-286369594


As we're comparing against symbols `eq?` can be used.


Anything I committed now is tested on a new clone of the main branch of the gshogi repository


What's the value of this test? I'm afraid such tests are too fragile and we better avoid them at all costs


@Gillespie59 can we get a release please? ):


@Abdillah Gak tahu kok tiba2 ada emailku yg ke forward ke github...  😕
Tak delete aja


第一次新增页面


Bluemix toolchain: An unknown tool deployed this commit to [prod](https://dev-console.stage1.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


Bluemix toolchain: An unknown tool deployed this commit to [Prod](https://dev-console.stage1.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name")


Maybe could remove the "Security" section in the README.md. This commit invalidates the result of the review. https://paragonie.com/blog/2017/03/jwt-json-web-tokens-is-bad-standard-that-everyone-should-avoid


Give the database please


https://github.com/nodejs/readable-stream/pull/263, I will publish when the tests are green.


Thanks a lot :)

Could we help get the 2.0.2 release out ?
And/or help maintain this gem ?
Removing deprecations alerts & getting security fixes would be lovely :)


[tests/testthat.R:2:9:](https://github.com/BIMSBbioinfo/ciRcus/blob/34129f8b747149b5c808340ca5dd432dd0f9940b/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(ciRcus)
        ^~~~~~
```
[tests/testthat/test-loadAnnot.R:87:25:](https://github.com/BIMSBbioinfo/ciRcus/blob/34129f8b747149b5c808340ca5dd432dd0f9940b/tests/testthat/test-loadAnnot.R#L87) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                        ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:87:48:](https://github.com/BIMSBbioinfo/ciRcus/blob/34129f8b747149b5c808340ca5dd432dd0f9940b/tests/testthat/test-loadAnnot.R#L87) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                                               ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:88:25:](https://github.com/BIMSBbioinfo/ciRcus/blob/34129f8b747149b5c808340ca5dd432dd0f9940b/tests/testthat/test-loadAnnot.R#L88) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                        ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:88:48:](https://github.com/BIMSBbioinfo/ciRcus/blob/34129f8b747149b5c808340ca5dd432dd0f9940b/tests/testthat/test-loadAnnot.R#L88) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                                               ^~~~~~~
```


Bluemix toolchain: An unknown tool deployed this commit to [prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


![](https://github.trello.services/images/mini-trello-icon.png) [#4002 - JUnit test for method getExp()](https://trello.com/c/JbsnZRiD/22-4002-junit-test-for-method-getexp)


Collaborator: Review for this commit was created. [Link to review](http://d86e7f32.ngrok.io/ui#review:id=1696)


I informed Aokromes that none of the pools seemed to work on 434 before he pushed this. Can someone check to make sure they are working on master?


On Tue, Mar 14, 2017 at 3:19 AM, jzmaddock <notifications@github.com> wrote:

>
>
> On 14/03/2017 03:08, Arkady Shapkin wrote:
> >
> > No, it should be vc1410
> >
>
> That's not the name Boost.Build is using: library names come out with a
> "141" suffix.
>

Is that with the latest master?


File, which contain  scripts for "blog.html", "database.html", "info.html" and "urls.html" soon...


Why this method is deleted?
I'm getting this error,

```
 PHP Fatal error:  Uncaught Symfony\Component\Debug\Exception\UndefinedMethodException: Attempted to call an undefined method named "mapFormV
  alidationGroupsParameters" of class "Sylius\Bundle\PromotionBundle\DependencyInjection\SyliusPromotionExtension". in /var/www/releases/f3b67
  819b609cd2424405fbd85593ffc2907a15c/vendor/sylius/promotion-bundle/DependencyInjection/SyliusPromotionExtension.php:32
```

in my composer.json

```
...............
    "sylius/core-bundle": "dev-master",
    "sylius/channel-bundle": "dev-master",
    "sylius/ui-bundle": "dev-master",
    "sylius/grid-bundle": "dev-master",
    "sylius/resource-bundle": "^1.0.0-beta.1",
    "sylius/payment-bundle": "dev-master",
    "sylius/payum-bundle": "dev-master",
    "sylius/shipping-bundle": "dev-master",
    "symfony/swiftmailer-bundle": "^2.5",
.............
```


there is an update on the OCCU to 2.27.7. But you may already know .... ;-)


Ok just made a fresh clone of e2 now building to check if build ok i'll check to clean aslo vs15000


@pb00068 looks good. Good luck.
I have a couple of other test positions:
8/8/1pk2K2/pR6/P2p4/1P1P4/8/3n4 w - - 11 74 (Ke6 or Rg5 wins, b4 Nc3 leads to a draw)
1k6/p3q2r/PpQp1p2/2pPp3/2P1P1p1/2P3P1/2K3P1/5R2 w - - 8 122 (loss with black against Houdini in TCEC: Kb3 or Rf5)
1q2k3/1Pp1Pp1K/2P2B2/8/8/8/8/8 w - - 0 1 (Kg8 Qa7 Bd4Qb8 Bc5)


This is the way locales work. It's understandable that maintaining locales
which are almost identical but have slight variations would be a tedious
task so I think the best approach to this is to support a way to define a
"base locale" from which we can derive sub-locales. I'm not a maintainer of
moment.js so I have no idea whether this is on the roadmap or not.

Safa Alfulaij <notifications@github.com> schrieb am Di., 14. März 2017,
4:32 nachm.:

> I got it from: https://en.wikipedia.org/wiki/Assyrian_calendar
> I am wondering that if there are no differences between the Arabic
> locales, except for the month names, why don't we just make it one locale
> and set some special variable for choosing the month name by a function or
> parameter? It's easier to maintain the file (Other locales are old and does
> not have the same code as this one.)
> But I'm not really sure if it can be implemented with the current
> structure of the library, or even how can this be used by other software,
> as I have almost zero knowledge about web apps.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/moment/moment/commit/811277e419cc3f4348a89f57ca475d55ead13343#commitcomment-21310603>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AA90GY2gk3txTIUW4nG9F4kCF_PRqQlfks5rlqUDgaJpZM4McC0f>
> .
>


@lassetyr OK thanks, I will try again this evening. Did you had time to have a look at how much work is needed to use the feed I want to use?

----- Mail original -----

De: "lassetyr" <notifications@github.com>
À: "rutebanken/OpenTripPlanner" <OpenTripPlanner@noreply.github.com>
Cc: "Nicolas Derive" <kalon33@ubuntu.com>, "Comment" <comment@noreply.github.com>
Envoyé: Mardi 14 Mars 2017 15:20:36
Objet: Re: [rutebanken/OpenTripPlanner] Added support for SIRI Lite ET-feeds (cc32e37)

This is because of an outdated version of "Jackson". I have upgraded to latest version in OpenTripPlanner

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


TeamCity Platform Services :: Environment Manager 2 :: Package [Build 3.36.0-pre-5-g8c54ef0](http://infra.teamcity.ci.ttldev/viewLog.html?buildId=54218&buildTypeId=PlatformServices_EnvironmentManager2_Package) outcome was **SUCCESS**
Summary: Running Build time: 00:01:00


update


The autobrightness curve doesn't look good in stock, it's a too too dark in the higher levels.


should write as below:
if(empty($this->rooms[$room]))


@dobin the 4.0 codebase (currently being worked on) could eventually need another review, but the test suite scenarios verifying the security constraints of the library weren't removed.

If the problem is what Scott posted there (which is fud), then we can add a test (or find one, since I think we already test the existing conditional) that makes sure that the signer being used must match the signer in the token, and all other algorithms are being ignored.

That's pretty much the test that invalidates the entire article above.


This test uses too much of the CLI compiler's internal API, which would be nice to avoid. Please refactor it so that the compiler is run, the output is captured to a list of strings and then verified:
```
val (output, code) = AbstractCliTest.executeCompilerGrabOutput(K2JVMCompiler(), listOf(
    // ... args, including "-d ...", "-Xplugin ...", etc.
))
```


Are the API going public or for internal use?


Приводи к числу сразу, как получаешь данные.


This means that the element will fade out in 3 sec.


@toanju can you please elaborate a bit more on this commit (also in the commit log)? Did the DPDK build system changed?


added readme new line


Зачем уточнять условие? Здесь получается "во всех остальных случаях", т.е. `else`


> implying I use Python 3

But yes, that looks correct.


Schließt #103


Bluemix toolchain: An unknown tool deployed this commit to [prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


迭代器报错 NoSuchElementException


*Changelog


@ipfans the freshest one, installed with glide

```
import:
- package: github.com/labstack/echo
  version: ~3.0
```


Collaborator: Review for this commit was created. [Link to review](http://d86e7f32.ngrok.io/ui#review:id=1699)


одна сторона, `side_max`


Collaborator: Review for this commit was created. [Link to review](http://d86e7f32.ngrok.io/ui#review:id=1700)


Стоит давать название аргументам более осмысленные. Например `phrase` (фраза) вместо `a`.


Is there any real need to test this particular compiler plugin with daemon? We know that the plugin works correctly with the compiler (ensured by the test above), and we know that the compiler works fine with the daemon. Given how complex it is, I don't think we need this test, unless I'm missing something


(I moved it to `Utilities > Scroll`, let me know if you think this to be a mistake)


для этого кейса есть формула попроще )


Лучше выбрать один стиль использования кавычек для определения строк и использовать приемущественно его.


Не надо несколько раз считать значение корня одного и того же числа. Вычисли один раз и положи в переменную


added another line


Thanks for this - I was banging my head against the wall yesterday trying to figure out what I was doing wrong in order to keep getting to_ids=True on every MD5 I was adding :+1:


@forabi Well, it seems to be the best way!
And looks like it is possible: http://momentjs.com/docs/#/customization (Check out third code block).

```
As of 2.12.0 it is possible to create a locale that inherits from a parent locale.
```


Are you sure we can safely remove previous_sync_ ?

I added it because we need to have a pointer to the previous frame to signal it when the commit is finished.


still not aligning center...


Collaborator: Review for this commit was created. [Link to review](http://d86e7f32.ngrok.io/ui#review:id=1701)


You are correct, because the calling code first uses `mat_cql_export?` that's not currently possible. I was queueing off of an IDE error, because technically all the code paths within `load_mat_cql_exports` don't validate this. How about add we add a comment before `load_mat_cql_exports` indicating it is required to validate `file` by calling `mat_cql_export?` first. Also, the inconsistent naming of `file` and `zip_file` between the 2 methods should probably be revised - although I realize that may require other methods to be updated as well to keep consistency.


Bluemix toolchain: An unknown tool deployed this commit to [Prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name")


Обзывая так переменную я имел ввиду максимальное значение в массиве **sides**. Но side_max правильнее будет. Поменяю


redux


I have not had the time to look at the details, but with both a custom
json-format and a customized id-mapping I'm afraid it will be a substantial
customisation of otp.

On Tue, Mar 14, 2017 at 4:03 PM, Nicolas Derive <notifications@github.com>
wrote:

> @lassetyr OK thanks, I will try again this evening. Did you had time to
> have a look at how much work is needed to use the feed I want to use?
>
> ----- Mail original -----
>
> De: "lassetyr" <notifications@github.com>
> À: "rutebanken/OpenTripPlanner" <OpenTripPlanner@noreply.github.com>
> Cc: "Nicolas Derive" <kalon33@ubuntu.com>, "Comment" <
> comment@noreply.github.com>
> Envoyé: Mardi 14 Mars 2017 15:20:36
> Objet: Re: [rutebanken/OpenTripPlanner] Added support for SIRI Lite
> ET-feeds (cc32e37)
>
>
>
> This is because of an outdated version of "Jackson". I have upgraded to
> latest version in OpenTripPlanner
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub , or mute the thread .
>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/rutebanken/OpenTripPlanner/commit/cc32e379b1024a3935cb46abb972e3c39286e384#commitcomment-21311325>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAEfgNPj2H3Q6ZQWgZa3DRcc-Ds9CW26ks5rlqxegaJpZM4Mbsf6>
> .
>

--
mvh
Lasse Tyrihjell


payload请求局部标识


да, я понял, что это side's max, но при чтении программы можно подумать, что их много )


Above comments about hqmf_path apply here. Not currently nil because we validate with `mat_cql_export?`


Argument 'localFileSystem.findFileByPath(file.getAbsolutePath())' might be null


The namespace does not directly play any significant role to versioning; it's purely just a way of organizing the service code. A convenience of organizing the code this way makes the division of code between versions easy to understand. It also enables type names and route templates to be the same across versions because there are no conflicts.

I suspect that you may have been expecting that the name of the namespace somehow directly influences versioning.  This example doesn't and was added purely as an ASP.NET Core counterpart to the Web API samples; however it should be possible to make that work. You would need a custom **IApiVersionProvider** that generates version information from some part of the namespace.  For ASP.NET Core, you'd have to wire this up via a **IApplicationModelConvention** or **IControllerModelConvention**, which is ultimately how both the default attribute and convention-based API versioning mechanisms inject themselves. I haven't tried it myself, but it _should_ work.


Yea, it's not that at all.  Just figured out this effects all of the mlt calls in davit as well.  So, now to change about 100 files...can someone hit the 'hooray' reaction for me some more.


используй постфиксную форму if


Collaborator: Review for this commit was created. [Link to review](http://d86e7f32.ngrok.io/ui#review:id=1702)


и здесь


TeamCity CI BACKEND :: TEST_CHAIN :: template_ci [Build 382](https://teamcity.forcity.io/viewLog.html?buildId=12973&buildTypeId=TestChain_TemplateCi) outcome was **SUCCESS**
Summary: Tests failed: 1, passed: 2 Build time: 00:00:57


Можно в конце строки добавить `\n`, это будет выводиться как перенос строки.


The SuperDARN RST should not depend on anything institution specific. There
are likely to many changes to our respective local codebases as we move
forward.

On Tue, Mar 14, 2017 at 10:52 AM, Kevin Sterne <notifications@github.com>
wrote:

> Eh, it will leave us with a codebase that doesn't match this repo. So a
> pain of changing all of the davit calls for a few hours, or the
> unforseeable length of pain of having to keep a diverged copy of this code.
> I'll take the former for $400 Alex. I'm more confused on why it didn't
> throw errors until I merged in the pull requests yesterday. There may be a
> thing or two more falling out from this though. Stay tuned.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/SuperDARN/rst/commit/8fb0d8b85799dff58c231be338c6428cdaefd400#commitcomment-21311025>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AC2DP3HRAedjSXjuBVSu_9bawAzvhULhks5rlqmhgaJpZM4Mco4T>
> .
>


попробуй решить через `while` и помни, что у массива можно спрашивать отрицательные индексы :)


Uhm. Did you modify this file by hand? That won't work in that case...


I'm not sure, but the example on README.md is like:
```
$config = new Configuration();
$signer = $config->getSigner(); // Default signer is HMAC SHA256
```
And this commit wanted to change the default to "None"?

Anyway, I think we all agree that security products should be secure by default. If an developer wants to make it insecure, it should involve additional work. Also I think "None" is as useful as TLS "NULL" ciphers. This library should guarantee the integrity of the tokens. If it is not doing this, the library is useless.


![](https://github.trello.services/images/mini-trello-icon.png) [#4001 - Information Expert Pattenr for some class](https://trello.com/c/u0VtK3Ol/21-4001-information-expert-pattenr-for-some-class)


I don't know if it's generically a problem, but I watched someone try to do this at Flutter First Look. :)  They would edit FlutterViewController.h (not realizing it was in a framework) and then be confused when their edits would go away when they ran (presumably we're copying the framework from the artifacts on every run?)

In any case, if this causes regressions we obviously can roll out and either try a different way or decide this was an uncommon fluke.


Still working on some minor issues.


test


You could write a simple recursive script that searches and replaces all
occurrences of these calls.

On Tue, Mar 14, 2017 at 11:21 AM, Kevin Sterne <notifications@github.com>
wrote:

> Yea, it's not that at all. Just figured out this effects all of the mlt
> calls in davit as well. So, now to change about 100 files...can someone hit
> the 'hooray' reaction for me some more.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/SuperDARN/rst/commit/8fb0d8b85799dff58c231be338c6428cdaefd400#commitcomment-21311673>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AC2DP817U3N0fGawaLHPYeK6zzsmkz_Tks5rlrB5gaJpZM4Mco4T>
> .
>


thanks for doing these revdeps @krlmlr. What is this one about?


Maybe not use filenames with spaces in them, please?


test messages


@lassetyr I fixed the stop_id and route_id problem (not matching stopPointRef and LineRef in SIRI lite feed) by replacing them in the GTFS data by the ones used in the real time data, so these ones should match (and DestinationRef should match an existing stop_id). I spent quite some time on it but that way, would it be easy (or easier)? Would that currently work?

----- Mail original -----

De: "lassetyr" <notifications@github.com>
À: "rutebanken/OpenTripPlanner" <OpenTripPlanner@noreply.github.com>
Cc: "Nicolas Derive" <kalon33@ubuntu.com>, "Comment" <comment@noreply.github.com>
Envoyé: Mardi 14 Mars 2017 16:18:12
Objet: Re: [rutebanken/OpenTripPlanner] Added support for SIRI Lite ET-feeds (cc32e37)

I have not had the time to look at the details, but with both a custom
json-format and a customized id-mapping I'm afraid it will be a substantial
customisation of otp.

On Tue, Mar 14, 2017 at 4:03 PM, Nicolas Derive <notifications@github.com>
wrote:

> @lassetyr OK thanks, I will try again this evening. Did you had time to
> have a look at how much work is needed to use the feed I want to use?
>
> ----- Mail original -----
>
> De: "lassetyr" <notifications@github.com>
> À: "rutebanken/OpenTripPlanner" <OpenTripPlanner@noreply.github.com>
> Cc: "Nicolas Derive" <kalon33@ubuntu.com>, "Comment" <
> comment@noreply.github.com>
> Envoyé: Mardi 14 Mars 2017 15:20:36
> Objet: Re: [rutebanken/OpenTripPlanner] Added support for SIRI Lite
> ET-feeds (cc32e37)
>
>
>
> This is because of an outdated version of "Jackson". I have upgraded to
> latest version in OpenTripPlanner
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub , or mute the thread .
>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/rutebanken/OpenTripPlanner/commit/cc32e379b1024a3935cb46abb972e3c39286e384#commitcomment-21311325>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAEfgNPj2H3Q6ZQWgZa3DRcc-Ds9CW26ks5rlqxegaJpZM4Mbsf6>
> .
>

--
mvh
Lasse Tyrihjell

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


## Social Links Updated


@sebbean yea, we noticed that pretty quickly as well.  But we were just able to get a PR merged for fixing this, and some other upgrades for Rails 5 (and 4) support: https://github.com/contentful/contentful_rails/pull/31


Nice


I totally agree here.  If you're reading this as "Kevin wants to change it back", I don't.  Again, for the third time, my confusion is why it just took effect yesterday.  But that's probably not worth it...especially with as many files to change.

It would be a good thing to make script to change all of the text.  But distinguishing between comments that include "mlt" and the function calls may not always be straight forward.  I'll take the human touch to know and decide.


с if читаться будет легче


Because that would lead to even more noise in the invocation.


FYI this class breaks all sorts of explicit and implicit contracts of `VirtualFile` which makes it dangerous to use. For example, it cannot be found in its file system by its path; it cannot be found in its parent's children; its not the parent of its children; its length is not equal to the size of its contents array; etc etc

If you're planning on supporting this plugin in the IDE, maybe we should look into another approach, because I doubt anything will work with such virtual files there (I might be wrong though)


попробуй сделать через `each.with_index`, чтобы обойтись без инициализации счётчика


# #.code


TeamCity CI BACKEND :: TEST_CHAIN :: template_ci [Build 383](https://teamcity.forcity.io/viewLog.html?buildId=12976&buildTypeId=TestChain_TemplateCi) outcome was **SUCCESS**
Summary: Tests failed: 1, passed: 2 Build time: 00:00:51


Why not log the error?


Yes , you are right.


fixed


I had to add this additional empty line in order to be able to parse the file with python-fluent or http://projectfluent.io/play/. Is this expected? @stasm


Believe it or not, I considered that. Unfortunately, in this case, it would have been written as `!( accept?.Any() ?? false )`. The null-propagation approach here would not only result in more instructions, it doesn't really bring any additional clarity or readability to the code.

Thanks for the suggestion though. It's good to know there are people in the community keeping an eye out on my PRs. :)


this commit is totally wrong


`times` позволяет передавать в блок счётчик
```
month.times do |n|
```


Also, this way of using images won't really work for CoyIM - you won't have the "build" directory for most endusers.


по-моему, условие високосности неполное


Bluemix toolchain: An unknown tool deployed this commit to [prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


Bluemix toolchain: An unknown tool deployed this commit to [Prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name")


@dadi11 Fixed it


I have just downloaded the same TDM version, touched window.cpp and compiled, getting the same error:

error: 'MAPVK_VK_TO_CHAR' was not declared in this scope

In fact all files under the include folder are identical in both versions. If I add the definition of MAPVK_VK_TO_CHAR to winuser.h then window,cpp compiles OK.

The 64 bit version of winuser.h does include the definition of MAPVK_VK_TO_CHAR (the file is about 50 % longer than the 32 bit version),


This is not what I wanted to get out of the config being able to support multiple IaaSes.

The goal was to have a single file so the calls to cliaas all look the same: `cliaas -c config.yml replace-vm`. If someone specifies multiple configs they have made an error and it should fail.


Use LightClassBuilder typealias


Did you comment on a specific line? Otherwise I don't understand "this one".


Collaborator: Review for this commit was created. [Link to review](http://d86e7f32.ngrok.io/ui#review:id=1704)


<details>

<summary>No TaskCluster jobs started for this pull request</summary>

```js

The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.
```

</details>


Bluemix [toolchain](https://dev-console.stage1.ng.bluemix.net/devops/toolchains/ffcbdd0e-633e-4c76-b311-f85461109925 "TestToolchain10"): [GitHub](https://github.com/jerome-lanneluc/Delivery-Pipeline "Delivery-Pipeline") deployed this commit to [dev](https://dev-console.stage1.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "US South:user@ibm.com:dev")


I'd point out the same thing. AutoPkg doesn't have a board of representatives and can't speak for the intended use of any of the recipes provided by the community. An organization who _uses_ AutoPkg and any recipes is free to contact FossHub and request an exception.

This still seems like an academic exercise to me. There have been three releases of Audacity in nearly two years. Is it really worth keeping up the code changes required to scrape FossHub reliably for an application which has a track record of getting a minor update once every ~8-12 months?


sorry, I meant the `f-revdep` compared to unqualified revdep. There are three versions in the repo.


get commit.template


I would add these to global gitignore, [like this](http://stackoverflow.com/a/7335487/1348344).

It makes no sense to add all IDE's temp directories to all repositories. With global gitignore you make it once and for all :)


hh


why the extra 45ms?


That helps. But things still aren't working. If I get this to work and write up instructions and send a pull request, will you accept the pull request?

I hold the opinion that having detailed simple instructions on how to get a simple project up and running with a development tool is extremely important. I think your plugin does a lot of nice things and would be used a lot more if all these problems I'm running in to were addressed by both better instructions and fixes to make things simpler. Writing complete instructions is the easiest short term solution and will point out clearly where there is room for simplification and improvements in the basic operation of the plugin or dev environment for the plugin.

My latest problem is that I cannot launch the resulting JNLP file. When run from the command line with `javaws launch.jnlp` in the directory of the JNLP, Java cannot find the libraries because the jars have the `.pack.gz` suffix.


Not the small test, the long test:

```
runtest(
  ["LFUCache","put","put","put","put","put","get","put","get","get","put","get","put","put","put","get","put","get","get","get","get","put","put","get","get","get","put","put","get","put","get","put","get","get","get","put","put","put","get","put","get","get","put","put","get","put","put","put","put","get","put","put","get","put","put","get","put","put","put","put","put","get","put","put","get","put","get","get","get","put","get","get","put","put","put","put","get","put","put","put","put","get","get","get","put","put","put","get","put","put","put","get","put","put","put","get","get","get","put","put","put","put","get","put","put","put","put","put","put","put"]
  ,[[10],[10,13],[3,17],[6,11],[10,5],[9,10],[13],[2,19],[2],[3],[5,25],[8],[9,22],[5,5],[1,30],[11],[9,12],[7],[5],[8],[9],[4,30],[9,3],[9],[10],[10],[6,14],[3,1],[3],[10,11],[8],[2,14],[1],[5],[4],[11,4],[12,24],[5,18],[13],[7,23],[8],[12],[3,27],[2,12],[5],[2,9],[13,4],[8,18],[1,7],[6],[9,29],[8,21],[5],[6,30],[1,12],[10],[4,15],[7,22],[11,26],[8,17],[9,29],[5],[3,4],[11,30],[12],[4,29],[3],[9],[6],[3,4],[1],[10],[3,29],[10,28],[1,20],[11,13],[3],[3,12],[3,8],[10,9],[3,26],[8],[7],[5],[13,17],[2,27],[11,15],[12],[9,19],[2,15],[3,16],[1],[12,17],[9,1],[6,19],[4],[5],[5],[8,1],[11,7],[5,2],[9,28],[1],[2,2],[7,4],[4,22],[7,24],[9,26],[13,28],[11,26]]
  ,[null,null,null,null,null,null,-1,null,19,17,null,-1,null,null,null,-1,null,-1,5,-1,12,null,null,3,5,5,null,null,1,null,-1,null,30,5,30,null,null,null,-1,null,-1,24,null,null,18,null,null,null,null,14,null,null,18,null,null,11,null,null,null,null,null,18,null,null,-1,null,4,29,30,null,12,11,null,null,null,null,29,null,null,null,null,17,-1,18,null,null,null,-1,null,null,null,20,null,null,null,29,18,18,null,null,null,null,20,null,null,null,null,null,null,null]
);

```

This fails, I thought because when my code inserts an item (put) I set the request value to 1 (insert is not a request and specification does not say it should be set with put). When I set the latest request in put as well the test fails though:

  addResult = add(
    this.doublyLinkedList
    ,new CacheUsage(key,1,++this.requestCounter,value)
    ,this.doublyLinkedList
  );


Thanks for fixes. Could create PR next time? I can't comment now anywhere but here :/


`f-` is the branch prefix I use for features. I can find only one remote branch that contains `revdep`.

When doing a revdep check, I compare the results of the current CRAN release (tag v0.5.0) with the current master. Each revdep run results in at least one commit to that branch. Does that help?


Была ошибка. Исправлена в следущих коммитах.


Write the script and print the matches first so you can check. Once you are
happy with your criteria, let it loose.

On Tue, Mar 14, 2017 at 11:26 AM, Kevin Sterne <notifications@github.com>
wrote:

> I totally agree here. If you're reading this as "Kevin wants to change it
> back", I don't. Again, for the third time, my confusion is why it just took
> effect yesterday. But that's probably not worth it...especially with as
> many files to change.
>
> It would be a good thing to make script to change all of the text. But
> distinguishing between comments that include "mlt" and the function calls
> may not always be straight forward. I'll take the human touch to know and
> decide.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/SuperDARN/rst/commit/8fb0d8b85799dff58c231be338c6428cdaefd400#commitcomment-21311769>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AC2DP5GhejZ6vD6Rj8B0ImXcBKi8TMbdks5rlrGvgaJpZM4Mco4T>
> .
>


Oh, do you mean the file names?


Интересный подход, молодец ) Но попробуй решить задачу, итерируя массив с месяцами, без методов.


yup


Initial commit


[tests/testthat.R:2:9:](https://github.com/BIMSBbioinfo/ciRcus/blob/dcef37baf22076ceef8d0ace8a7c3f3d8eb6763b/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(ciRcus)
        ^~~~~~
```
[tests/testthat/test-loadAnnot.R:87:25:](https://github.com/BIMSBbioinfo/ciRcus/blob/dcef37baf22076ceef8d0ace8a7c3f3d8eb6763b/tests/testthat/test-loadAnnot.R#L87) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                        ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:87:48:](https://github.com/BIMSBbioinfo/ciRcus/blob/dcef37baf22076ceef8d0ace8a7c3f3d8eb6763b/tests/testthat/test-loadAnnot.R#L87) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                                               ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:88:25:](https://github.com/BIMSBbioinfo/ciRcus/blob/dcef37baf22076ceef8d0ace8a7c3f3d8eb6763b/tests/testthat/test-loadAnnot.R#L88) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                        ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:88:48:](https://github.com/BIMSBbioinfo/ciRcus/blob/dcef37baf22076ceef8d0ace8a7c3f3d8eb6763b/tests/testthat/test-loadAnnot.R#L88) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                                               ^~~~~~~
```


LightClassDataHolder has a method findDataForClassOrObject(), which is only called on the object returned from createLightClassDataHolderForClassOrObject(), and a method findDataForFacade(), which is only called on the object returned from createLightClassDataHolderForFacade(). Looks like it could be split into two different types.


[tests/testthat.R:2:9:](https://github.com/BIMSBbioinfo/ciRcus/blob/bf128a252b9b5521e7950dc42cbcacdeed4cc563/tests/testthat.R#L2) *style:* **Variable and function names should be all lowercase.**
```r
library(ciRcus)
        ^~~~~~
```
[tests/testthat/test-loadAnnot.R:87:25:](https://github.com/BIMSBbioinfo/ciRcus/blob/bf128a252b9b5521e7950dc42cbcacdeed4cc563/tests/testthat/test-loadAnnot.R#L87) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                        ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:87:48:](https://github.com/BIMSBbioinfo/ciRcus/blob/bf128a252b9b5521e7950dc42cbcacdeed4cc563/tests/testthat/test-loadAnnot.R#L87) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                                               ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:88:25:](https://github.com/BIMSBbioinfo/ciRcus/blob/bf128a252b9b5521e7950dc42cbcacdeed4cc563/tests/testthat/test-loadAnnot.R#L88) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                        ^~~~~~~
```
[tests/testthat/test-loadAnnot.R:88:48:](https://github.com/BIMSBbioinfo/ciRcus/blob/bf128a252b9b5521e7950dc42cbcacdeed4cc563/tests/testthat/test-loadAnnot.R#L88) *style:* **Variable and function names should be all lowercase.**
```r
  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                                               ^~~~~~~
```


* schema updates
* fixed loading window refresh and creation of new popup with same name
* renamed properties
* updated schema defaults
* fixes for no selector specified
* fix redirect current window delay


sum же )


Yep, you're right.  Thanks!


The 'timeout' parameter of create_socket( ) will always be DEFAULT_TIMEOUT_MS, that parameter could be stripped.


Revdep checks are committed on top of each other. The `problems.md` is also copied to `problems-f-revdep.md` if on the `f-revdep` branch, or to `problems-v0.5.0.md` if on the `v0.5.0` tag, and so forth. This allows comparing problems side by side from a checkout, or comparing current problems with the CRAN version with those from the last revdep check run, perhaps months ago.

The logic is implemented in `revdep/diff.sh`.


Commit message is very strange, this commit does not contain any syntax incompatibilities, only you added some tests to skipped list ??


<details>

<summary>No TaskCluster jobs started for this pull request</summary>

```js

The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.
```

</details>


Collaborator: Review for this commit was created. [Link to review](http://d86e7f32.ngrok.io/ui#review:id=1705)


Hi @adam-devapp

We've made some changes to the group sync functionality in Outlook plugin but there aren't any changes in the Outlook API extension. We're waiting on one of our clients to come back to us on a group related functionality.

We plan to package that with this and then do a formal release. But just so you know, this version fixes the following bugs -

> - sync triggered even when an email is not sent to a group
> - account disconnected message pops up even when the Civi account has been disconnected.

Thanks
Priyanka


In practice, the 'timeout' parameter will always be DEFAULT_TIMEOUT_MS which is not user-provided and not zero.


This ugly method name can be made shorter by not repeating "LightClass" in the name. I.e. `createDataHolderForClassOrObject`


Hi done


Bluemix toolchain: An unknown tool deployed this commit to [prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


Bluemix toolchain: An unknown tool deployed this commit to [Prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name")


здесь можно переменные обозвать без префикса product_, из контекста будет понятно, что чьи эти свойства, мы же с одной сущностью работаем )


I guess the issue is making sure it matches things correctly and doesn't pick up something strange.  In the time we've taken to design this script I've gotten a quarter way through the files.


Yes, but we realized that it won't work and we have changed it to be the generated file.


можно обозвать cart, это отразит суть )


I think this `asList()` call looks weird. Better to make `.done()` public and to call it here and below.


Would that help anything for such simple self-contained fixes? I don't see any added value for PR, otherwise I'd create one...


Bluemix [toolchain](https://dev-console.stage1.ng.bluemix.net/devops/toolchains/ffcbdd0e-633e-4c76-b311-f85461109925 "TestToolchain10"): [GitHub](https://github.com/jerome-lanneluc/Delivery-Pipeline "Delivery-Pipeline") deployed this commit to [dev](https://dev-console.stage1.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "US South:user@ibm.com:dev")


https://github.com/orgs/gshslatexintro/people/Qyubey 가 아니라
https://github.com/Qyubey 로 넣어야 할 것 같습니다.


Thanks!


nice, thanks


это же просто months :)


This method seems to be no longer used in the final version of the code in your branch.


Yeah, I think that better fulfills the contract for `EditPostPage`. How should I access the post from `verifyPage()` or are you suggesting passing id from `$urlParameters` to  it via `verify()`?


CBECC-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed (2151 of 2420 tests passed, 0 test warnings)

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


<details>

<summary>No TaskCluster jobs started for this pull request</summary>

```js

The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.
```

</details>


develop (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


In previous commits, I modified this file manually, but this commit uses the generated file.


How to use this? What does it do?


develop (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


I know, it's for later if we want to have a iio_create_context_with_timeout() function.


Bluemix toolchain: An unknown tool deployed this commit to [prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


My bad, I didn't mean to push to the main repo.


\*Esboço\* depois pode até ter uma classe para esses objetos e a retirada de `clickableObjects`,


Bluemix toolchain: An unknown tool deployed this commit to [Prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name")


yeah, I guess this patch can be dropped now.


Changing the URL should never be part of the scenario (we cannot expect from any normal user to figure out that he would have to enter 'en_US' to switch to English). A user who wants to specifically change the language should do so at the bottom of the page with the appropriate button in the footer.
The rest of the problem remains.


Adding responsive elements to the website for the LiverpoolFC webpage.


You can use assertion on the list directly...


`|name, item|`, не надо называть сложные структуры value


which will let you chain the `containsOnly()` call without having to repeat `assertThat`.


Rename this to `KtLazyLightClass` or something else more descriptive?


Not that it's anything important, but the calls to various assertions could be fluently chained.


Hello


Having `sudo: false` is Travis's default


It would be good to keep the blank lines in the gitignore - they make it way easier to see the separate sections.


Well, I guess it doesn't hurt to keep it around for iio_create_context_from_uri_with_timeout( ) :p


For code readability, it's best if the tested code isn't directly inside the `assertThat()` call.


This release looks much nicer and is much simpler to read!  Can't wait to test it.


Is it necessary to use YAML sequence in place where string literal is enogh?


Hej :smile:
Jest to czwarty commit z tym samym komentarzem. W treści komentarza warto umieszczać znaczące informacje, tutaj więcej informacji na ten temat: https://chris.beams.io/posts/git-commit/

Pozdrawiam
Adam


`createLazy()`?


Personally, I'd rather first create the 4 persons and then put them into the array - you'll get to avoid all of that nasty indexing business.

`return new Person[] {bruno, robert, joe, mike};`


<details>

<summary>No TaskCluster jobs started for this pull request</summary>

```js

The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.
```

</details>


Hm. This commit use the same fuction QString::fromLocal8Bit() to convert word to QString as before. What locale do you use?


Rename to `computeRealDelegate()` for consistency with `KtLightMethodImpl`?


Fixed in 07b7288762a7be03bb54f1ee23166057b7860ee8


borren los comments que sino, tira error


Looking at these two `@Test` methods, they are identical except for the session source. Did you consider using `@Parameters`?


This change has moved the checkbox input from the left-hand side of the label to the right hand side of it: is this a gov.uk requirement?

Example before:
![image](https://cloud.githubusercontent.com/assets/2122700/23909449/b76752e2-08ce-11e7-8ec7-c96f5e05135c.png)

Example after:
![image](https://cloud.githubusercontent.com/assets/2122700/23909464/c6359108-08ce-11e7-9937-3239d52c43eb.png)


@ucffool Man, still thinking about this. It's SUPER gross to describe the prerequisite in the feat description. I can't think of anywhere else that is done and it's hugely un-intuitive to be like "Hey, I have Attack Specialization, let me take this feat [7 days later during a session] ... Oh I get +1 to Guard ... umm .... wait, what? I have to have a specific type of Weapon Specialization? Why doesn't it say that under prerequisites?"


This is a bit overkill but it does work!

The updated `SOLOCS` did not work since your last commit. Very likely because you are using your own built libraries. It should be working again when the dependencies tarball is updated. `cshoes.exe: error while loading shared libraries: ?: cannot open shared object file: No such file or directory`


Working Register Code


LOL IDIOT


@fchollet AFAIK, `tensorflow` requirement in install_requires may conflicts with `tensorflow-gpu` (google/seq2seq#29). How if we move `tensorflow` and `tensorflow-gpu` to be in extras_require?


Thank you. It required quite the mental effort.


A single usage should have been introduced in this commit. KtLightClassForInterfaceDefaultImpls#findLightClassData


пропустил :)


Done, hope it is ok now and could be backported into v3


For some reason I can't get this to work, for instance, I change:
```
EGLint* attribList;
EGLint* windowAttribList;
EGLint* contextAttribs;
```
to:
```
EGLint attribList[sizeof(defaultAttributeList) / sizeof(EGLint)];
EGLint windowAttribList[sizeof(defaultWindowAttribs) / sizeof(EGLint)];
EGLint contextAttribs[sizeof(defaultContextAttribs) / sizeof(EGLint)];
```

And I get rid of the malloc and free statements, and it no longer works, I get "eglCreateWindowSurface() returned error 12293" (EGL_BAD_CONFIG)


@ucffool Alright, here we go: https://github.com/openlegend/core-rules/commit/275bc6e14ca50fc101f38cf7f05255f4240e48dd


Bluemix toolchain: An unknown tool deployed this commit to [prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


Bluemix toolchain: An unknown tool deployed this commit to [Prod](http://localhost:3100/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name")


when your mind moves as fast as mine you dont have time for spellcheck


I'm not sure if this code will scale well across multiple nodes... Revert imo


can u please send the answers of problem 5 and problem 6 also i am screwed up with those questions pls help!!!!!!!!


Why would you need to run a subprocess? It's a python script already, you may easily invoke that code directly.


This is also not going to work on os other than linux (mac for example won't work).


Thanks. .notdef isn't forced to be present or in the first position right now, but I don't think we care in this situation. Using an invalid gid to ignore downstream sgtm.

Also I've yet wire the ufo2ft FeatureOTFCompiler, I think we want markFeatureWriter in there.


FTR I'm testing with https://github.com/webdriverextensions/webdriverextensions-maven-plugin but it's not helping that much


I think we shouldn't do this. Instead, we should have explicit dependency notation for depending on a core plugin. We want to move away from a fat distribution and users depending on the whole thing.


I thought that using this in just 2 of 3 tests would just worse the readability.


Can be simplified to
```java
doesNotContain("Small City", "Local City");
```


You might do conditional installation it that's needed


#3 done


Sorry, I didn't bother to try to install the built packages, I just verified it builds correctly.


saveでいいの？


いいです。


This can probably be implemented without touching the real delegate.


Je vais l'enlever. My bad


Thank you Francis for your relevant comment. But Sensiolabs insight don't recommend me to use the twig filter 'raw'.
How to solve this problem cleanly?
Merci de ta réponse @francisbesset :+1:


This too?


I believe you could just use `pyenv install`


Yeah, we'll remove the spaces from the filenames.


this guys only do what they want to do. serve was totally useful for everyone. they just think they want to implement and never ask the users.


Finally 👀


Can we make this part of the travis-ci build? It makes no sense to rebuild twice.

It triggers rebuild of all packages because of different parameters, see  https://travis-ci.org/input-output-hk/cardano-sl/jobs/210962783#L2199


TeamCity Code :: 1. Development :: DDK :: Feature [Build 217](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=17432&buildTypeId=Code_Development_Ddk_Feature) is now running


Ref: https://github.com/cherrypy/cherrypy/commit/34d9d70a1a928bb9707a276a5e30d1350c4911f5


The serve command is back anyway. Locking this.


FYI: this breaks CI https://travis-ci.org/input-output-hk/plutus-prototype/jobs/210132159


Mapping functions should not modify original items. They should create new items with the desired values.

Also, should the values be transalated? or it is done by UI?


here is the output of locale:

LANG=en_US.utf8
LANGUAGE=en_US:en
LC_CTYPE=en_US.UTF-8
LC_NUMERIC=de_DE.UTF-8
LC_TIME=de_DE.UTF-8
LC_COLLATE=en_US.UTF-8
LC_MONETARY=de_DE.UTF-8
LC_MESSAGES=en_US.UTF-8
LC_PAPER=de_DE.UTF-8
LC_NAME=en_US.UTF-8
LC_ADDRESS=en_US.UTF-8
LC_TELEPHONE=en_US.UTF-8
LC_MEASUREMENT=de_DE.UTF-8
LC_IDENTIFICATION=en_US.UTF-8

btw, I have two machines, one desktop one laptop, both running Debian jessie (stable). both have the same situation. I use `git reset --hard HEAD~` to go back some commit changes. After I pass this one, Goldendict behaviour returns to normal.


Extract method `KtLightMethodImpl.convertAll()` and reuse here and in `LightClassDataHolder`? Same for `KtLightFieldImpl`.


a


`    for (range in sectionsEnum) {`


I don't think it is true that the view will always have only `\n`. I think that would depend on the OS. I'm on Windows, and when I open a file that I know has CRLF line endings, GitGutter it telling me that all lines are different.


TeamCity CI BACKEND :: TEST_CHAIN :: template_ci [Build 384](https://teamcity.forcity.io/viewLog.html?buildId=12997&buildTypeId=TestChain_TemplateCi) outcome was **SUCCESS**
Summary: Tests failed: 1, passed: 2 Build time: 00:01:02


I'm curious why did you choose to implement `Enumeration` and not `Iterator`, since the former is somewhat deprecated in the JDK


`RadioButton` should already be handled by `CompoundButton`.  Is that not the case?


(Also moved persistence service initialization to app component, not a sub-component.)


@cbracken @eseidelGoogle this change broke the devicelab too. We're no longer able to delete the repo directory because of the following permissions on some of the iOS files, e.g.:

```
-r-xr-xr-x 1 username staff 45155384 Mar 13 16:18 ios/Flutter/Flutter.framework/Flutter
```


Mantab


`Mode` seems to be a better name for this than `Token`.


Not good to print this to the console, just re-throwing the exception is all we need.


Mmm right now, the `tensor flow` backend is the default backend for Keras 2.0.
Not really a good idea.
My HO on this is that `tensorflow` dependency is fair enough for most of the cases.
If you have any GPU on board, you can previously install `tensorflow-gpu` and so the dep is already satisfied.
Thus, to me, no change here is required.

my2¢


When you need something that will be evaluated only once, do not put it in the middle of a while loop !
You are doing the same mistake again and again...
see line 282  `while ((s = *pl++) != SQ_NONE)`

In your current version.

If white has 2 knights and 1 bishop
your line 313 will be executed 3 times for white

If black has 1 knight and 1 rook
your line 313 will be executed 2 times for black

To fix this, if you want to use scores, move your line 313 in evaluate_init

Or you might want to keep this in material.cpp as before
(just average mg and eg of score and multiply by 16)
This is (re) computed only when new material balances happens, which is more efficient.

So something like this (in material.cpp)
int QueenSet[10] = { -160, 144, 256 , 0, 0, 0, 0, 0, 0};


I'm seeing an `INTERNAL ERROR: Unable to parse built-ins` ([see here](https://ci.appveyor.com/project/andystanton/linter-glsl-g9y85/build/108/job/crd8madth9k6qtf4#L495)) whenever running on [this file](https://github.com/AtomLinter/linter-glsl/blob/62a0432affef953d81b84bd8d53fe7db6d428d7d/spec/fixtures/vert/sample.vert). Commit 946e0a6 doesn't have this issue.


lllllllllllllllllllllllllllllllll


To not sync with other things which might use 1s interval in case they are expensive.


TeamCity Code :: 1. Development :: DDK :: Feature [Build 217](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=17432&buildTypeId=Code_Development_Ddk_Feature) outcome was **SUCCESS**
Summary: Tests passed: 41 Build time: 00:06:39


TeamCity OTA+ :: libats :: ci [Build 16](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26013&buildTypeId=libats_Ci) is now running


Why is this needed? You could just keep `\r`s around, they won't cause any harm (but also filter them below in lines 85 and 92)


TeamCity OTA+ :: libats :: ci [Build 16](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26013&buildTypeId=libats_Ci) outcome was **SUCCESS**
Summary: Running Build time: 00:00:45


Jon Baker


March 14, 2017


ok


cool... ill add that behavior to the tests


@TomasVotruba Ok, thanks.


Bluemix [toolchain](https://console.stage1.ng.bluemix.net/devops/toolchains/d68b2e66-855d-49b2-84e2-40823dbeb47c "jiraPipelineDemo-int"): [Delivery Pipeline](https://console.stage1.ng.bluemix.net/devops/pipelines/1cd1e8cc-18ff-4813-973c-c98f31b05d1b "jiraPipelineDemo-int") deployed this commit to [dev](https://console.stage1.ng.bluemix.net/apps/a913c97d-0d3c-4d14-8eb5-a036aff3b48d "ibm:ys1:us-south:patrick.joy1:dev")


Thanks for your comment, doesn't look as if this is going to be merged in at some point though.
Maybe you followed https://github.com/salemdar/angular2-cookie/issues/37, an updated version of the library has supposedly been published as https://github.com/salemdar/ngx-cookie (haven't had the time to check it out).


I think this should check other properties which are implemented by the lazy light class directly without instantiating the real delegate (e.g. `isConstructor`).


Thanks @jchris 👍


Please add why it was expected to the error message.


pra que colocar na rota que é uma deleção se o método http ja é DELETE?


List<Map<String,String>> list = new ArrayList<Map<String,String>>();
		try{
			if(sql ==null || sql ==""){
				log.info("error: sql is null");
				return list;
			}
			Connection conn = dataSource.getConnection();
			PreparedStatement stmt = conn.prepareStatement(sql);
			ResultSet rs = stmt.executeQuery(sql);
			// 先判断字段个数
			String[] field = null;
			try{
				field = sql.split(" from ")[0].replace("select ", "").split(",");
			}catch(Exception e){
				log.error("sql 分解有异常！！！");
				e.printStackTrace();
			}
			Map<String,String> map;
			if(field !=null){
				while(rs.next()){
					map = new HashMap<String,String>();
					for(int m = 0;m<field.length;m++){
						String key = field[m];
						if(key.contains(" as ")){
							key = key.split(" as ")[1].trim();
						}else{
							key = key.trim();
						}
						map.put(key, rs.getString(key));
					}
					list.add(map);
				}
			}
			return list;

		}catch(Exception e){
			e.printStackTrace();
		}
		return null;


+1 for aesthetics! 😀


pra que colocar na rota que é pra criação de um anúncio se o método http já é o POST?


This is now pretty much like the original. It takes advantage of the new krl-compiler@0.26.4 command line flag `--verify`. With this change, the pre-commit hook still works pretty well with older `krl-compiler` that doesn't recognize that flag, by sending all of `stderr` to `stdout`. Once we think people have updated their `krl-compiler`, we can remove that last part ` 2>&1`


It is not the case for me. I am using a MvxRadioGroup view and the binding on it won't work without this two methods added on the LinkerPleaseInclude file.


poderia ser HttpStatus.NO_CONTENT, ao invés de HttpStatus.OK


`tensorflow` as a requirement is perfectly reasonable. If you already have everything set up and don't want new installs to change your config, you should always use `--no-deps` as a pip flag. This will avoid you many issues in general (beyond Keras).


hello


Indeed. A [merge commit (example)](https://github.com/MariaDB/server/commit/9dc10d5851ab4ff38d81e68858de56fbb1d6743d) would have 2 parents, but GitHub is only showing 1 parent for this commit.


it is not work


Bluemix [toolchain](https://console.stage1.ng.bluemix.net/devops/toolchains/25e762ff-83e6-452c-8ffb-50884c1185c4 "jenkinsGitJiraDemo-int"): [Jenkins](http://9.37.138.183:8080/ "jenkinsGitJiraDemo-int") deployed this commit to [dev](https://console.stage1.ng.bluemix.net/apps/827e124c-667d-4d7d-927d-f6bafc4011f5 "ibm:ys1:us-south:patrick.joy1:dev")


I don't understand these several lines, why not just `return buffer.array()`?


Better to include the toString() of the method here.


We can still do that. I just want to avoid creating too big of a story so we can deliver feature incrementally. Please also see the open issue regarding this fact specifically.


okay!


Changes looking good.


:skull: thank mr skeltal :skull:


Views only apply the line endings when saving and reading and use `\n` internally (This leads to problems with mixed line endings). Do you have the newest version from the master branch or the latest release (which does not contain this fix)?


Are they functional now?


I'm not sure why `encoding = getattr(sys.stdout, 'encoding', 'utf8') or
'utf8'` would not work?

On Mar 14, 2017 3:22 AM, "Peter Badida" <notifications@github.com> wrote:

> @llfkj <https://github.com/llfkj> see #5043 (comment)
> <https://github.com/kivy/kivy/issues/5043#issuecomment-285129357>, I
> mentioned it before. The problem is however not that simple because either
> there'll be an ugly patch in window_sdl2.py , or we'll fix all start.c
> files. I'd go for start.c, but this rises a problem i.e. what encoding will
> be used on each platform. I wonder if something bad can happen if we put it
> only as a variable encoding='utf-8' into start.cs.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/kivy/kivy/commit/3542080fd9e811ceb1cc67bc5e43a62c02854f56#commitcomment-21304470>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABkW_k8edkWJlF-IpglOoIhqz_d6XKqGks5rlkAjgaJpZM4McEbZ>
> .
>


Good point. Still, there is some benefit to code reuse. Perhaps extracting the actual test into a new method (`testWithSource(KieSession)`) and calling that from both tests would be the best compromise.


Add a note to not confuse this argument `__ow_namespace` with the environment variabless `__OW_NAMESPACE` the environment variables is always present and is the namespace of the owner of the webaction.


Bluemix toolchain: An unknown tool deployed this commit to [prod](https://dev-console.stage1.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "w3ibm:prod:us-south:user@ibm.com:prod")


Bluemix toolchain: An unknown tool deployed this commit to [Prod](https://dev-console.stage1.ng.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19 "There was a problem retrieving the space name")


I guess it's too late, since it's already commited & merged. Will do so, if possible.


Are you interested in bringing the reuseaddr changes into "websocket-vi" upstream?


CSS problem


Would it make more sense to link to these via the internet?
`<link rel="stylesheet" href="https://raw.githubusercontent.com/elm-community/elm-datepicker/3.0.0/css/elm-datepicker.css">`


App jumps to details activity seemingly sporadically (unpredictable timing, when lists is scrolled, etc). Additionally, several playerDetail Activities are being created simultaneously, stacked on top of Main Activity. I may have set OnClickListener on wrong widget, or am not implementing it correctly.


Why do we need this for a field? If the fields have the same name and belong to equal classes, then they are equal.


Well my idea is to have consistency, so in different builds of the image the flink user will have the same UID, this is not guaranteed if we don't pin this. I agree about the other point the user name however, so maybe we can just pin both and remove the ENV if you agree.


Yes, please backport.


Thank you for the format, @Robert-Frampton. Should we create a new version with those changes?


@sriram9aj @sagar-rao-1306 : : Please review these checked in files.


Not very clear this sentence "The `__ow_namespace` is not present if the HTTP request did not present an authentication header."

I think what this means is the this the argument `__ow_namespace` will only present, only when the web action is annotated with `require-whisk-auth`

Meaning if I send a http request with an `Authorization` header with value `Carlos is such a fool` this value will not be available in `__ow_namespace` this value the user can find in `__ow_headers["Authorization"] == "Carlos is such a fool`

right?


should we rename it to __ow_user?


made some additional comments


new commit


made a comment


Alternatively, require the "English" module to use the more descriptive `$CHILD_STATUS`

e.g.
```
require 'English'
$CHILD_STATUS.exitstatus.zero?
```


I think we need to add the type to each argument, for example __ow_method is a string (lowecase or uppercase?) __ow_headers is a json object or array, path string (case sensitive)


second commit


Production code converted mysql to work with php5.4


Followup patch in #8766


A program to identify Anagrams in java


Why the "as well as" ? Why not just the simple list.


TeamCity Code :: 1. Development :: CMS :: Features [Build 96](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=17443&buildTypeId=Code_Development_Cms_Features&guest=1) is now running


That's so [July 2002](https://en.wikipedia.org/wiki/Display_resolution#2000s) ! Let's go for the most common display resolution from 2012: `1366 x 768`


TeamCity Code :: 1. Development :: CMS :: Features [Build 96](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=17443&buildTypeId=Code_Development_Cms_Features&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:18


We need an example on how raw body comes in


Fixed by commit https://github.com/abrt/abrt/commit/f7a8844686c7b4ed69db927e6149519c88195ef5


"s/forgo/go without"


Stefano, can you take a look and let me know if you have any thoughts on whether I am on the right track?

https://groups.google.com/d/msg/fishcooking/C3KorRmYmWM/g6MJjhI3BgAJ

Trying to find out why my machines crash on all your dynamic tuning tests.  There does not seem to be any dynamic contempt patches where a crunchy-11 machine has contributed any testing, and that just looks too strange to be a coincidence.


Can I see the code that you have for this? If all you did was get rid of the malloc and free and allocate in the stack like I described above (also with the ```/ sizeof``` which was my mistake), it should be exactly the same.


Re: __first class properties available to the action input.__
We need to be consistence over the entire doc, on how do refer to the input to the action "arguments", "properties", or "parameters".


`ktFile`


Expression bodies here and below?


too much empty lines


The diff just looks like this:
```
-EGLint* attribList;
-EGLint* windowAttribList;
-EGLint* contextAttribs;
+EGLint attribList[sizeof(defaultAttributeList) / sizeof(EGLint)];
+EGLint windowAttribList[sizeof(defaultWindowAttribs) / sizeof(EGLint)];
+EGLint contextAttribs[sizeof(defaultContextAttribs) / sizeof(EGLint)];

 size_t FindIndex( const EGLint a[], size_t size, int value )
 {
@@ -67,11 +67,8 @@ extern DECLSPEC m64p_error VidExtFuncInit()
     surface = EGL_NO_SURFACE;
     context = EGL_NO_CONTEXT;
     display = EGL_NO_DISPLAY;
-    attribList = (EGLint*)malloc(sizeof(defaultAttributeList));
     memcpy(attribList, defaultAttributeList, sizeof(defaultAttributeList));
-    windowAttribList = (EGLint*)malloc(sizeof(defaultWindowAttribs));
     memcpy(windowAttribList, defaultWindowAttribs, sizeof(defaultWindowAttribs));
-    contextAttribs = (EGLint*)malloc(sizeof(defaultContextAttribs));
     memcpy(contextAttribs, defaultContextAttribs, sizeof(defaultContextAttribs));

     if ((display = eglGetDisplay(EGL_DEFAULT_DISPLAY)) == EGL_NO_DISPLAY) {
@@ -100,9 +97,6 @@ extern DECLSPEC m64p_error VidExtFuncQuit()
         eglTerminate(display);
         display = EGL_NO_DISPLAY;
     }
-    free(attribList);
-    free(windowAttribList);
-    free(contextAttribs);
     return M64ERR_SUCCESS;
```


We need a link for "content-type is not binary" to point some list on that states this very clear. maybe at the end of this document.


let move the conversation to issue #51, where it will be less buried


Great, thanks a lot @JeffreyWay.


Fix indentation please :)


"A store resource" is ambiguous. Maybe "Represent an Oregon State Liquor Store as a resource."


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

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


only the first 10


> breaks migration on postgres strongloop/loopback-connector-postgresql#217

Ouch, I did not realise this may be a breaking change :( If we cannot find a reasonable fix for postgres, then we may need to introduce a feature flag allowing users to decide what column type to use.


is beter


Ha, those are free, and worth every penny! Use in good health, but also at your own risk. Those are left over from an earlier 3-phase approach we tried on OpenStudio. We found it made more sense to use the 'air.bsdf' just inboard of the glazing, and sub in the shading device in its place. This way, users can make their glazings as normal, and the shading device is just there when it's deployed. The air BSDF is just a straight matrix that passes everything through unchanged. You probably noticed that we no longer ship all these "blended BSDFs" with OpenStudio, just the air and shading device ones. We add a polygon just inside the window, onto which we apply these BSDFs.


Gewoon eens wat uitproberen


Hi,

Is it a completely black screen, or is there a red box in the image? A red box indicates that at the very least the Lepton is connected, while a completely black screen will have bigger issues.
There isn't really a way to duplicate your error, since it just won't show an image from the start. As simple as it sounds, you might have success uncoupling the wire connecting the Lepton to the  power on your breadboard and letting it rest before plugging it back in. Sometimes the Leptons get out of sync with the internal clock and need to be reset.
Other than that, I can't really do much more to debug. We have gotten the code to work on Raspberry Pi 3's, but I can take a look over the code again to see if anything is off.

Andrew Myers
Developer, Novacoast
m: 925.681.9126
s: www.novacoast.com
e: amyers@novacoast.com

From: YuanGZ <notifications@github.com<mailto:notifications@github.com>>
Reply-To: amyers-novacoast/LeptonModule <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, March 13, 2017 at 6:10 AM
To: amyers-novacoast/LeptonModule <LeptonModule@noreply.github.com<mailto:LeptonModule@noreply.github.com>>
Cc: Andrew Myers <amyers@novacoast.com<mailto:amyers@novacoast.com>>, Author <author@noreply.github.com<mailto:author@noreply.github.com>>
Subject: Re: [amyers-novacoast/LeptonModule] Updated /raspberry_video/LeptonThread to work for Lepton 3. Increased SPI speed and video now correctly assembles 4 segment image from Lepton 3. (2853232)

Hi ,
I download the code raspberry_Video_to work for Lepton 3
https://github.com/amyers-novacoast/LeptonModule
I followed the tutorial
https://learn.sparkfun.com/tutorials/flir-lepton-hookup-guide

But I have only a black screen.
Is there anyone who has already make Lepton 3 work with le RapsberryPi 3 successfully?
Is there anything to pay attention?

Thanks,

-
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/amyers-novacoast/LeptonModule/commit/28532322c61d105f68a5f7b7064d711d1b890369#commitcomment-21289940>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ATgApWhNfwyoz2DFXQTL153mhcvbJvz_ks5rlU5ygaJpZM4MbTTw>.


TeamCity OTA+ :: libats :: ci [Build 17](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26015&buildTypeId=libats_Ci) is now running


Zaboravljeno indeksiranje klase Knjiga u prethodnom commitu.


Make Note bold **Note:**
Also don't state the limitation first.
state fact first, then the why.
**Note:**: All annotations for a single action must be set at the same time this includes when updating one of the annotations (otherwise previous values are removed). This is due to a current limitation on the API and CLI.


TeamCity OTA+ :: libats :: ci [Build 17](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26015&buildTypeId=libats_Ci) outcome was **SUCCESS**
Summary: Running Build time: 00:00:40


why Impl - shouldn't be Imp? https://www.allacronyms.com/IMP/Implementation


You didnt test this change...
In usleep, deadline is calculated by adding 32-bit timer value from tc_get_cv but then it is tested by comparing with 64-bit _timer_get_tick that includes the upper 32-bits.

Here is a code fragment I use to do a quick test of the timer-just run on hardware (like SAMA5D2B-XULT) and verify that the time between the first printout and second printout is close to 10 seconds.  If it hangs without finishing or the time is way off, something is wrong:
        printf("timer test.\n\r");
        uint32_t loopcnt;
#if 1  // test usleep
        for (loopcnt=0; loopcnt < 20000; loopcnt++)
        {
          usleep(500);
        }
#endif
#if 0  // test msleep
        for (loopcnt=0; loopcnt < 200; loopcnt++)
        {
          msleep(50);
        }
#endif
        printf("10 seconds end\r\n");


Wait, what?

No, I meant I just wanted to see if the errors would have happend if he ported that feature back to ABXD. I didn't mean for it to go public. Why did you just do that?


uh

what

no


this is a syntax error no matter how you look at it


If it's precompiled, why do we depend on Go?


We are logging the error in createPIN:
https://github.com/twstrike/coyim/commit/1eda127916b5351dfb52e40d29db4877130a7f76#diff-92e9000e83b598f07f331f3918366569R199

But we can log it again before aborting from the wizard.


why isn't this passed in from the view

why is this using django tags in jade


TeamCity OTA+ :: libats :: ci [Build 18](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26016&buildTypeId=libats_Ci) is now running


If we add them in there then making it slimmer later is a breaking change.


TeamCity OTA+ :: libats :: ci [Build 18](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26016&buildTypeId=libats_Ci) outcome was **SUCCESS**
Summary: Running Build time: 00:00:31


I don't think this is actually TODO.


Master branch: new commits are available.


For reference: https://github.com/lcobucci/jwt/pull/174


Good time of day, I used "Score" for observe values for 2 phases. Yes, thanks, i forget about while loop cycle.


sugared is not a term, that translate to other languages.

Also make it a note

**Note:** While `raw-http` annotation implies `web-export`, we have plans to improve the CLI to provide a command that adds both annotations at once with a different flag.


I'm still seeing five instances of the class "notification_message" in /View that this breaks.


any update since (des mises a jour?)
maybe make a list (recessement) of all skeleton 3d built  in the world?


Take into account that when updating annotations, you need to update all annotations at once, if not all pervious annotations are removed only leaving the annotations `raw-http` set to `false`.


But 90 degrees can't be set because the maximum value prohibits that?


This is default since rpm 4.2. Unless you target EL5 there is no need to specify it here. On the other hand it makes no harm. It is just bloat.


refs #13


Nossa que legal


That's definitely the case and I am aware of it. We have to decide on a case by case basis. I sometimes feel like been too conservative is holding us back from delivering features earlier. In this specific case the breaking change is fairly isolated and will only affect plugin developers but not the actual plugin artifact that has been produced.


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1882 of 1882 tests passed, 0 test warnings)

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


Inicio do Shell Scrpit


Isso eh um ShellScript!!!


nossa que legal


oloco meu muito top !!


Realmente eh bacana!


Entendi


kkk


comentário


TeamCity OTA+ :: libats :: ci [Build 19](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26017&buildTypeId=libats_Ci) is now running


`customTitle !== ''` prevents the user from completely clearing the title field. (#7691)


Inicio de ShellScript


isso e um shell


Nhaaaa


Teendi


TeamCity OTA+ :: libats :: ci [Build 19](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26017&buildTypeId=libats_Ci) outcome was **SUCCESS**
Summary: Running Build time: 00:00:38


I have a fork of this project that does the same thing as the above commit but returns the txt records as a dictionary instead of a string.

This approach makes it unnecessary for user code to parse the text records. It also ensures that the text records are parsed according to the [spec](http://www.zeroconf.org/rendezvous/txtrecords.html).

The details of my commit are [here](https://github.com/makeitnew/react-native-zeroconf/commit/fcc8d11f68cf853d78dda3f134e57612c62c910b).

Please consider using this approach instead.

Thanks!


Muitooo bom


@mkonicek this doesn't seem to be published on npm, is this normal ? Thanks.


Pro


testovoiy repozitoriy


This makes the two location lines (address and location details) misaligned.

# Before:
![image](https://cloud.githubusercontent.com/assets/1965565/23911989/d88191be-08b4-11e7-89dd-a55958829926.png)

# After:
![image](https://cloud.githubusercontent.com/assets/1965565/23912085/0d8f6d54-08b5-11e7-80a4-a4f5797bf126.png)


Ok, to dziala bo `textReplaced` ma 90 znakow czyli polowa to 45. Ale co jesli sie zmieni i bedzie mial 100?
Bedziesz musiala zmienic 45 na 50.
Wiec zamiast pisac na sztywno liczbe mozesz poprostu uzyc `textReplaced.length/2` czyli
`var partOfMotto = textReplaced.substr(0,textReplaced.length/2);`


Bluemix [toolchain](https://dev-console.stage1.ng.bluemix.net/devops/toolchains/192c205f-d201-49b3-812b-fce116421ce5 "weatherapp-toolchain"): [Jenkins](http://localhost:4080/job/WeatherApp/  "Jenkins") deployed this commit to [staging](https://dev-console.stage1.ng.bluemix.net/apps/c33d1df2-f17e-4ef1-8318-0be1d723a9d3 "ibm:ys1:us-south:kuschel@ca.ibm.com:staging")


ansome

> **_[ZICAAA](url)_**


You are right, my point was more to prioritize first the current images than the 'future' images. It is just a question of priorities, because if we fix e.g. the docker-entrypoint today to fix something for the current image we will have probably to port this to the 'future' one.

I agree with your point, I just want to start with this once everything is in good shape and once we have a fix URL for those staging builds so we can have them in automatic reproducible fashion.


CSS for the gallery being added


What's the value of this class? Why not have the `Api` class construct the resources directly?


This adds the JDBC mysql libraries to the git too


Much more clear 👍


This is the simple chat application using localhost 3000.
Initially we have to create the package.json using the kernal (npm init) there you would be setting the ,
{
  "name": "socket-chat-example",
  "version": "0.0.1",
  "description": "",
  "dependencies": {}
}
      Then we would using **Express js** for that we have to install its module through
> npm install --save express ,inorder to intergrate it with socket io we need to install
> npm install --save socket.io....
you could further add index.html and index.js then your chat app is ready to use...


have you not forget hd51 ?


straaasznie Ci uciekla ta consola, ja bym to tak uporzadkowal (bez linijki przerwy i mniejszy odstep):
```
for (var....) {
  console.log(...)
}
```


to kwestia preferencji ale nazwa `x` dla zmiennej w petli jest troche dziwna - z reguly sie stosuje `i`


The following doc(s) have been deployed to the [sshkeystr](https://sshkeystr-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/ssh-keys](https://sshkeystr-static-docs.pantheonsite.io/docs/ssh-keys)
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/e9a4c44835cc1c6a060f894dc55b4d76393bc818/source/docs/assets/css/main.css)


zostaw tylko jedna linijke przerwy na koncu


Программа клиент для поиска модифицированных игр на сервере.


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


This commit makes `#front_page_announcement` overlap with the search bar at some window widths:

![image](https://cloud.githubusercontent.com/assets/1965565/23912466/0c900124-08b6-11e7-8155-d5b548170c1d.png)


strike the `yarn run`, it's already being run. Also it's fine here, but usually you configure Jest to source the `__tests__` folder that way you don't have to repeat this everywhere you invoke Jest


hum, changing would be good and less confusing.


This feels a little clunky and was my primary reason for asking the above. Only one resource type needs access to the `Api` and only for one method. Is there a way to re-structure the relationship between the resource and API classes to avoid needing to pass around an instance of the API? Not sure if worth doing, but worth considering.


no need for sinon, just use Jest's built-in mocks and spies. You can do `const onClick = jest.fn();`


Clobbering variables should be an edge case. You'd have to have the same path param declared twice in the route like `/users/:name/friends/:name`. Wrapping it in `ctxKey` prevents clobbering other packages' variables.


- instead of `cout` use `cerr`
- in the message string use pattern like "[ERROR]  Mesh_1D()  Tried to create dimensionless mesh..."
other example pattern:
- [WARNING]    Mesh_1D::toVTK()  description
-[ATTENTION] Class::methode() description

- Return a meaningful value like `NULL` or `-1` or some other number that will represent an error, so you can handle these from outside.


usun te 2 linijki z poczatku


Will there be an update to the docs for AWS_ECR_CACHE_DIR env var?


First submission on Kaggle


dodaj srednik na koncu


then this would become `expect(onClick).toHaveBeenCalled();`


you should be able to get rid of this since you're using yarn


Access page by "/admin/competition/:competition_id"


K, that's fine. Yeah let's drop the ENVs since we don't actually need to really reference them elsewhere.


see above comment about using Jest's builtins instead


Predicting Land Average Temperatures using Random Forest


My oops :/


I see nothing wrong here. I'll pull your branch and run it to try to determine what is going on. It will have to wait until some time tonight though since I'm at my job right now. Also, is there any reason why you are copying the attributes instead of passing the default attributes directly?


TeamCity Code :: 1. Development :: CMS :: Features [Build 97](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=17453&buildTypeId=Code_Development_Cms_Features&guest=1) is now running


TeamCity Code :: 1. Development :: CMS :: Features [Build 97](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=17453&buildTypeId=Code_Development_Cms_Features&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:16


Oops, this broke the forms previewer. Has this shipped publicly yet?


Is that a bash issue really? Which platform integration does this refer to? Also, "FAILES" has a typo.


> **кома** віддавши


До рук


Тільки перша літера велика


Added the media query to make the gallery responsive.


Ok, I see a problem. If the only diff is what is above, then we are passing the wrong size to ```FindIndex```


Відповідно й тут тільки перша велика


My First submission


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1902 of 1902 tests passed, 0 test warnings)

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


Our full code is:
```
			var methods = type.GetMethods (System.Reflection.BindingFlags.Static | System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Public);
			var method = methods.Single (t => {
				var parameters = t.GetParameters ();

				return t.Name == "RegisterAll"
					    && parameters.Length == 1
					    && parameters [0].ParameterType == typeof (Type[]);
			});
```


Agreed ^


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1284 of 1284 tests passed, 0 test warnings)

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


I think this is better than.


There is however one issue, FLINK_HOME, that one we cannot move without hardcoding it (it is before because If we don't define it before the next ENV the PATH don't get it, what do you prefer for that one ?


Кома


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Search Term: Randm
DB Entry: Random Book
               (Rand)_______
Search Term: Chks
DB Entry: Chakshu Tandon
                      (ksh)_______

Why is 'h' also part of match?


code is not according to doc


do you plan on keeping those defines?


#8 #16


Лучше используй хэш, он как раз создан для быстрого доступа к данным по ключу


Щось типу

> Перегляньте вправи "особистості та їх життєві шляхи"


> The win32-svc.h is missing.

Added in


missing use-case... where is the proposal?


Ah right, that's why I had that one separate in my original version. I still like de-varring user/uid from a conceptual perspective.


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


According to one of the last calls, this concept was discarded


kkkkkkkkkkkk

zika mesmo


i think u have to `defer res.Body.Close()` for all http requests... but lets just use loogo here anyway.


Вашим


go's https client apparently doesn't time out. https://medium.com/@nate510/don-t-use-go-s-default-http-client-4804cb19f779#.wue8sz3ja


[GMods_1.0.0.1.zip](https://github.com/cs015/lm-hab/files/842115/GMods_1.0.0.1.zip) Приложение клиент для поиска модифицированных игр и приложений с full лицензиями. Система Android.
[Lenov_2.0.zip](https://github.com/cs015/lm-hab/files/842116/Lenov_2.0.zip)
Ещё один мобильный клиент для сайта Lenov система Android.


Could you add a test for this error?


That's not true, default.cfg is always exec'd before any other configs:
```----------------------
10431 files in pk3 files
execing default.cfg
execing q3config.cfg
execing autoexec.cfg
[...]
execing server.cfg
```
As you said, the server_example.cfg has a matching value of 2, but if you were to remove the sv_floodprotect line from it, it would be set to 0 because of default.cfg (assuming clean install with no q3config.cfg of course)


go back


Кома


Fixed in 70a445e3fba980c17d9527f6d6ec54c866eb29df.


It's fine I think...the lint command is a bit funny about it.  I prefer your style, but haven't found a way to get the tools to allow it! :)


This is giving me errors


Необхідна


So in which cases in `portability.h` should

   #ifdef _MSC_VER

be replaced by

    #if defined(_MSC_VER) || defined(__MINGW32__)

(and in which cases should the change go the other way)?


Nice!


TeamCity OTA+ :: rvi_sota_server :: ci [Build 2273](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=26021&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 268, ignored: 3 Build time: 00:16:39
### Failed tests
```
APIFunTests.test adding devices: <no details avaliable>

APIFunTests.test searching devices: <no details avaliable>

APIFunTests.test adding packages: <no details avaliable>

APIFunTests.test searching packages: <no details avaliable>

APIFunTests.test adding filters: <no details avaliable>

APIFunTests.test deleting filters: <no details avaliable>

APIFunTests.test searching filters: <no details avaliable>

APIFunTests.test changing filter expressions: <no details avaliable>

APIFunTests.test creating components: <no details avaliable>

APIFunTests.test searching components: <no details avaliable>

APIFunTests.test deleting components: <no details avaliable>

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


How do I use this? I understand it isn't a .jar but I have tried and failed to make it a .jar. i would really appreciate some help, thanks?


@djett41 sorry, it's been a while and I can not remember how we solved this case

You may try https://github.com/risetechnologies/phonegap-plugin-push/tree/tappedFixiOS at your own risk (we added some undocumented features). I think that https://github.com/risetechnologies/phonegap-plugin-push/commit/5b548363498dd06c41443c995a46617af09e3549 was related to this issue, hope it helps


Great work, I did this last week and was going to do a pull request, guess you beat me to it!


The following doc(s) have been deployed to the [wordpressc](https://wordpressc-static-docs.pantheonsite.io/docs) Multidev environment:
- [source/docs/assets/images/wordpress-commandline-child-theme-wp.png](https://github.com/pantheon-systems/documentation/commit/70a309ac8c871a1c8acd312f36d7a83f4c360f29/source/docs/assets/images/wordpress-commandline-child-theme-wp.png)
- [source/docs/assets/images/wordpress-commandline-commit-cf7-to-dev.png](https://github.com/pantheon-systems/documentation/commit/70a309ac8c871a1c8acd312f36d7a83f4c360f29/source/docs/assets/images/wordpress-commandline-commit-cf7-to-dev.png)
- [source/docs/assets/images/wordpress-commandline-install-cf7.png](https://github.com/pantheon-systems/documentation/commit/70a309ac8c871a1c8acd312f36d7a83f4c360f29/source/docs/assets/images/wordpress-commandline-install-cf7.png)
- [source/docs/assets/images/wordpress-commandline-theme-example.png](https://github.com/pantheon-systems/documentation/commit/70a309ac8c871a1c8acd312f36d7a83f4c360f29/source/docs/assets/images/wordpress-commandline-theme-example.png)


I think this breaks henkaku build . I m getting this error since this commit :
henkaku/bootstrap/bootstrap.c:524:8: error: too many arguments to function 'sceAppMgrDestroyOtherApp'


this way


ok, I will fix.


Well, obviously I want a device with the power of LEDE OS and of course, a wifi that works well, but as you known, isn't the case yet. I have three WT3020, all of them have the same problem.

There's no way to use the commercial Ralink driver with LEDE?

Anyway, thank you for the great effort you are doing. I have a clue how difficult can be reverse engineering a driver, so, keep up the good work!


Ок. Может вообще не создавать папки уроков, а все в одном проекте вести?
Создавать ветку для каждого урока. PR?
Вроде так удобнее


I'm trying to follow the single responsibility pattern as much as possible. So the API's responsibility is to make Requests and return Resources, but it isn't responsible for creating the resources.

That and, consider how large the API class already is and then imagine what would happen if there are 6 or 7 resources to create DTO's for.


@RyTekk thanks for the heads-up, will take a look once I'm back on Stock ROM :)


Ah, right 🙂


pls


I'm not entirely sold on this method either. I think i'm more in favor of adding another method to the Api class called `getProductPrices`  which accepts a `Product` resource and does the same API call.


If you want to see where I found this mysterious function, click on the next [link](http://www.maximecolin.fr/blog/symfony/19-le-filtre-raw-de-twig-et-sensiolabs-insight) @francisbesset :smile:.


genome_db_ids are per assembly, so this will break when the next human assembly is released.
You can use `->name eq 'homo_sapiens'` instead


Same as above