That should be fine for now — I recorrected it on my end already.

> On Feb 22, 2016, at 11:21 AM, stephanripke <notifications@github.com> wrote:
>
> ok,
> did accept the pull request.
> is that enough or do you need a new complete version?
>
> On Mon, Feb 22, 2016 at 5:00 PM, jigold <notifications@github.com> wrote:
>
> > @stephanripke <https://github.com/stephanripke> -- can you please add
> > these fixes to your latest version (or accept the pull request)? I
> > encountered this same problem when re-QCing PsychChip data this weekend.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Nealelab/ricopili/commit/2cb6d4a1e991fcad51fd9974694c2997ce0c599e#commitcomment-16244200>
> > .
> >
>
>
>
> --
> Stephan Ripke, M.D., Ph.D.
> Analytic and Translational Genetics Unit
> MGH Simches Research Center
> 185 Cambridge Street - CPZN6818
> Boston, MA 02114
> USA
>
> sripke@broadinstitute.org
> ripke@atgu.mgh.harvard.edu
> —
> Reply to this email directly or view it on GitHub <https://github.com/Nealelab/ricopili/commit/2cb6d4a1e991fcad51fd9974694c2997ce0c599e#commitcomment-16244857>.
>


Cool :+1:


Running `cabal haddock` will now generate a .html file with all documentation.


> @ralphtheninja I saw these are no longer used in node-gyp, and I made the same change in node-ninja so at some future date we can remove these lines of code IMO.

:+1:


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.29](http://build.clear-measure.com/viewLog.html?buildId=5067&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 84 Build time: 00:02:21


1. Перемести проект в папку с соответствующим названием ("CleanCode" например)
2. Поправь чтение/запись (через коллекции или массивы), чтобы файл log.txt из задания работал.

В остальном хорошо, задание 2.1 принято


@hintjens We need to update `node-gyp` to new major though, right?


ресурсы надо закрывать.


Donde esta la bibleoteca


Ese codigo se me hace familiar...... -___-


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


@Lukas-W this throws the following error on my machine...

```
../cmake/build_mingw64.sh: 17: ../cmake/build_mingw64.sh: Bad substitution
```

I believe you've used `bash` syntax in an `sh` file.


we could just file another difficulty/low bug for this though. Can you do that?


Как правильно организовать тест на наличие/отсутствие ссылки на странице?
RSpec считает ссылки, генерируемые хелперами `course_path(course)` и `user_course_path(cser, course)` одинаковыми. Т.е. проверка засчитывает строку `"/courses/{id}"` равной строке `"/user/{user_id}/courses/{id}"`.


the file needs renaming though, this is not a python file ;)


Well, this could just be a race, so "pending" fits better. But considering this is just for display purposes, races do not matter, and this is indeed more intuitive.

For the record: "unmanaged" means that networkd has processed the link and decided that its configuration says not to manage it; "pending" means that the link has appeared, but networkd has not yet gathered enough information to decide whether or not it has a matching .network file.


w ten sposób, jak wybuchnie jedna z cech, to nie będziesz wiedział która – będziesz tylko wiedział, że wybuchło

lepiej to rozbić na kilka assertów, czyli:

```
def assert_valid_video(video):
    assertTrue(video.title is str);
    assertTrue(video.title is not '');
    // itd, w ten sposób dokładnie wiesz, która reguła się wywaliła
```


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.30](http://build.clear-measure.com/viewLog.html?buildId=5073&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


Table, not title


Задание 1 принято

Коммит все-таки стоило назвать "Update status.txt"


Cleaned up a bit..


I was surprised by this as well.


TeamCity Civic Assembly :: Build all [Build 70](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1306&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new); exit code 123 (new)) Build time: 00:03:01


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


object нужен только при super


I'm having issues with this change, If i take it out my app compiles fine without it.

The imports all go through the above plugin "expression imported var" instead of this one

Can't pin down exactly what breaks, but it's causing some of my imports to be undefined.

What is it enhancing?


regexp and string are the same parameter


I think I would prefer this to be nil and to panic if version is requested without it being set. I realize panicing is sort of nuclear, but I think it would strongly communicate to any reader that you really shouldn't screw it up. Also add a comment indicating how this eventually gets set.


не верные отступы у товаров.
У 4-го должен быть справа отрицательный отступ чтобы он уместился в строке (все из-за паддингов которые нужны для наведения)


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.30](http://build.clear-measure.com/viewLog.html?buildId=5073&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85 Build time: 00:02:46


should we be using the ignore tag?


Done.


skoro wiemy, że tytuł nie może być pusty, to umocnij model

* niech `VideoSignature` sprawdzi to w konstruktorze i rzuci wyjątkiem – w ten sposób nie będzie się dało stworzyć nieprawidłowego obiektu
* przetestuj jednostkowo `VideoSignature`, żeby potwierdzić takie zachowanie klasy
* a ponieważ powyższe będziesz miał już „pewne”, to w tym teście wystarczy tylko, że sprawdzisz, że `search` zwraca `VideoSignature`


this throws : chamiloInstalattion//app/cache , with doble slash (//)


PLUGINS
No remove please


DON'T REMOVE APIS!!!!!!!!!!!!!!!!!!


Mine carts are WIP


Having less node is good. Having simpler c code is also good.

In this case, as need already need to support int32 in the c code for backward compatibility, I would go by making the cast in the c code... But I'm not super strict on this.


ta sama idea: https://github.com/Glenpl/LYTM/commit/7ec65e85ff5ceb0069fb1896f1a3dbe3ed59ee69#commitcomment-16245043

tylko tutaj masz wybór: albo odpalać w konstruktorze `str()` albo rzucać wyjątkiem – czyli filtrowanie albo walidacja – ale nie dopuszczać złego stanu dla obiektu


Тень при наведении светлее. Плюс при наведении меняй z-index чтобы другие товары не перекрывали тот на который навелись.


fixed the typo.  I think it makes sense to leave both symbol and graphic in the example.


No styles here? and instead of explicit value, use the bootstrap variables.
``` @media (min-width: @screen-md-min) ```


This will actually need to be a reference to the base key, since monitors will trigger quota/rate limit overages webhooks etc and the consumer will want to know what key is doing the damage, we shouldn't expect consumers to have to decode the api-specific key...


Changed this to forEachDataExtensionSegment


I don't understand this. There already is a sandbox and we already compiled the stuff (in the previous steps). Is it yet another sandbox and yet another `cabal install`?


I don't see the need for this.


Fixed in https://github.com/rails/rails/commit/b2d1de2697f70239e2e80d93c0d728cd8509eae9


Сердечко неправильно выравнено


Yes, so when node-gyp hits 4.0.0...


Don't panic. Even impossible conditions are supposed to be a returned err. And it's pretty easy to send in a nil HTTP client.

If you're dead set on not sending an err back on NewCertSource, create a new func called NewCertSourceWithClient and then pass an empty client if you get a nil one. That sort of implicit functionality I don't like in the main function, but I think it's OK in a helper as long as its documented.


For the reference, this was documented by @est31 here: https://github.com/godotengine/godot-docs/pull/10


Also above two orange images, lets give them more descriptive names instead of just orange and orange1.
Give something like ```bg-{section name}```


Typo Retruns


Why you call this API ... I cant find this in PocketMine ... Genisis ... ImagicalMine ...


Ширина немного меньше. Плюс текст копирайта не должен переноситься на новую строку. (2016)


thank you for doing this


And it have already add back


TeamCity Civic Assembly :: Build all [Build 71](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1308&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 166 (1 new), passed: 6, ignored: 4 Build time: 00:05:47
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1456158591657: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

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


Yeah, these are magic numbers. Make them consts.


every file should end with an empty new line at the end. Just a coding convention. Thats why GIT is adding the red flag here.


more ?


nit: I think this is actually gErr if you go by the capitlization scheme.


Indenting is off here.


proponuję precyzyjniej `html_title` albo `head_title` – bo samo `title` to dosyć popularne słowo i trudno grepowalne


слово download тут не уместно.


Also, this line cause several fatal errors in all the platform like this in create sessions

Fatal error: require(): Failed opening required '/var/www/chamilo110x//app/cache/__CG__ChamiloUserBundleEntityUser.php'

doctrine is trying to find the entities in app/cache


Great thinking!


嗯～這老師有教過了,今天有搞懂了.


Стоило бы добавить jar которую использовал (с учетом того что она не из 3 предложеных нами)
Пользователю не ясно, откуда берется файл, при его отсутствии приложение падает с FileNotFoundException (касается всех IOException).

Задание 2.1 принято


See https://circleci.com/gh/cockroachdb/cockroach/13114 for the test failure that motivated this change - basically, there is a race here between `serve` trying to send on `sl.l.connc` and `Serve` closing that channel. There is no guarantee that this `select` will favour the closed channel `sl.l.donec`.


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


Redundant class ```services`` on both containers. Keep it only on one. May be keep on ```<main``` only.


100th commit, congratulation!


on plugins they are used.


typo. should be "unexpected".


Are you writing any styles for ```col-left```? If not, then you should remove this.


Let me know offline if you do find evidence of that in the standard library; I've never seen it before but it's not a bad suggestion.

Possibly better: make them named consts (here) and pass them in. But yeah, right now you shouldn't stick with these nakedly.


Isn't required for backward compatibility given that we do not add any upcasting node anymore?


Why the rename? Seems like the previous naming was correct.


Here's all of the events:

```
[58](#<RSpec::ExampleGroups::OutgoingMessage::SmtpMessageIds::AResentMessage>)> InfoRequestEvent.all
=> [#<InfoRequestEvent id: 900, info_request_id: 101, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 1\n", created_at: "2007-10-12 01:56:58", described_state: nil, calculated_state: nil, last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 1, comment_id: nil>,
 #<InfoRequestEvent id: 901, info_request_id: 103, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 2\n", created_at: "2007-10-14 10:41:12", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: "2007-10-14 10:41:12", incoming_message_id: nil, outgoing_message_id: 2, comment_id: nil>,
 #<InfoRequestEvent id: 902, info_request_id: 101, event_type: "response", params_yaml: "--- \n:incoming_message_id: 1\n", created_at: "2007-11-13 18:09:20", described_state: nil, calculated_state: nil, last_described_at: nil, incoming_message_id: 1, outgoing_message_id: nil, comment_id: nil>,
 #<InfoRequestEvent id: 903, info_request_id: 101, event_type: "comment", params_yaml: "--- \n:comment_id: 1\n", created_at: "2008-08-12 23:05:12", described_state: nil, calculated_state: nil, last_described_at: nil, incoming_message_id: nil, outgoing_message_id: nil, comment_id: 1>,
 #<InfoRequestEvent id: 904, info_request_id: 104, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 3\n", created_at: "2011-10-12 01:56:58", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 3, comment_id: nil>,
 #<InfoRequestEvent id: 905, info_request_id: 105, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 4\n", created_at: "2006-01-12 01:56:58", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 4, comment_id: nil>,
 #<InfoRequestEvent id: 906, info_request_id: 105, event_type: "response", params_yaml: "--- \n:incoming_message_id: 2\n", created_at: "2007-11-13 18:00:20", described_state: "successful", calculated_state: "successful", last_described_at: nil, incoming_message_id: 2, outgoing_message_id: nil, comment_id: nil>,
 #<InfoRequestEvent id: 907, info_request_id: 106, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 5\n", created_at: "2006-01-12 01:56:58", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 5, comment_id: nil>,
 #<InfoRequestEvent id: 908, info_request_id: 106, event_type: "response", params_yaml: "--- \n:incoming_message_id: 3\n", created_at: "2007-11-13 18:09:20", described_state: "successful", calculated_state: "successful", last_described_at: nil, incoming_message_id: 3, outgoing_message_id: nil, comment_id: nil>,
 #<InfoRequestEvent id: 909, info_request_id: 105, event_type: "comment", params_yaml: "--- \n:comment_id: 2\n", created_at: "2008-08-12 12:05:12", described_state: nil, calculated_state: nil, last_described_at: nil, incoming_message_id: nil, outgoing_message_id: nil, comment_id: 2>,
 #<InfoRequestEvent id: 910, info_request_id: 107, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 6\n", created_at: "2001-01-02 01:23:45", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 6, comment_id: nil>,
 #<InfoRequestEvent id: 911, info_request_id: 107, event_type: "response", params_yaml: "--- \n:incoming_message_id: 4\n", created_at: "2001-01-03 01:23:45", described_state: "successful", calculated_state: "successful", last_described_at: nil, incoming_message_id: 4, outgoing_message_id: nil, comment_id: nil>,
 #<InfoRequestEvent id: 912, info_request_id: 108, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 7\n", created_at: "2001-01-02 02:23:45", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 7, comment_id: nil>,
 #<InfoRequestEvent id: 913, info_request_id: 108, event_type: "response", params_yaml: "--- \n:incoming_message_id: 5\n", created_at: "2001-01-03 02:23:45", described_state: "successful", calculated_state: "successful", last_described_at: nil, incoming_message_id: 5, outgoing_message_id: nil, comment_id: nil>,
 #<InfoRequestEvent id: 914, info_request_id: 109, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 8\n", created_at: "2009-01-02 02:23:45", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 8, comment_id: nil>,
 #<InfoRequestEvent id: 915, info_request_id: 110, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 9\n", created_at: "2009-01-03 02:23:45", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 9, comment_id: nil>,
 #<InfoRequestEvent id: 916, info_request_id: 111, event_type: "sent", params_yaml: "--- \n:outgoing_message_id: 10\n", created_at: "2016-02-22 15:50:40", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 10, comment_id: nil>,
 #<InfoRequestEvent id: 936, info_request_id: 124, event_type: "resent", params_yaml: "---\n:email: request@example.com\n:outgoing_message_i...", created_at: "2016-02-22 15:50:45", described_state: "waiting_response", calculated_state: nil, last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 27, comment_id: nil>,
 #<InfoRequestEvent id: 937, info_request_id: 124, event_type: "resent", params_yaml: "---\n:email: request@example.com\n:outgoing_message_i...", created_at: "2016-02-22 15:50:45", described_state: "waiting_response", calculated_state: nil, last_described_at: nil, incoming_message_id: nil, outgoing_message_id: 27, comment_id: nil>,
 #<InfoRequestEvent id: 934, info_request_id: 124, event_type: "sent", params_yaml: "---\n:email: test@example.com\n:outgoing_message_id: ...", created_at: "2016-02-22 15:50:45", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: "2016-02-22 15:50:45", incoming_message_id: nil, outgoing_message_id: 26, comment_id: nil>,
 #<InfoRequestEvent id: 935, info_request_id: 124, event_type: "sent", params_yaml: "---\n:email: test@example.com\n:outgoing_message_id: ...", created_at: "2016-02-22 15:50:45", described_state: "waiting_response", calculated_state: "waiting_response", last_described_at: "2016-02-22 15:50:45", incoming_message_id: nil, outgoing_message_id: 27, comment_id: nil>]
```


Коммит стоило бы назвать "Update status.txt"
Задание 1 принято


Cf. http://valve.github.io/blog/2014/07/04/from-object-to-functional-immutability/ , section on "String immutability and Ruby"; unclean design.


@scottpurdy Can you comment in @chetan51's absence?


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


Remove col-right if not used for styling.


@vilchik-elena yes, so what is returned is not always the block which is returned at the first line. It may be clearer to not return anything.


viewController的fighterArray....
為了要跟FighterTableViewController.h的 fightersArray區別....


нет, просто будь внимательнее:)


HAHAHAHAH!!!


Need to mention https://github.com/chef-cookbooks/php/pull/80


8 columns and offset for 2, didn't understand the purpose of these here??


Once you introduced the `mCheaterArray`, the `mIsCheater` no longer plays any useful role and should be eliminated.


This is made for a university project, I prefer to mantain it as simple and clear as possible, also using some redundant constraint or equalities if necessary. Redundancy is not a problem in this case, anyway I'm really thanksful for your suggestions, they can be usefull for future works and to improve my Prolog programming skills.


@vilchik-elena Indeed, that may simplify some other pieces of code.


Этот коммит стоило бы назвать "Move java files to 'up1' package"


{string|array|number}


Great thank you!


nice catch


Instead of having full width columns, these should be new container classes instead.


valgrind want his file to have a correct +x flag, you may force by chmod a+x $SWITCH_BUILD_ROOT/switch


Same as above.


Ha, I wonder if this is the issue:

> but starting with Rails 4 first always used a default ORDER BY (rails/rails #5153) because good databases (PostgreSQL :) ) will optimize a query without an explicit order, which means calling User.first may return different users every time, while User.order(:id).first may not.
>
> – http://featurebranch.com/the-two-biggest-bugs-fixed-in-rails-4-1/


The `mCheaterKeyArray` is the first part of an elaborate workaround for using the `putBoolean` and `getBoolean` methods of the `Bundle` class when `putBooleanArray` and `getBooleanArray` would have been more appropriate.


@pynicolas indeed!


I already talked to him, calm down and check other commits before!


Looks like this file doesn't exist. Also can you rename your gif file to include no spaces to avoid any problems with finding the file? :)


Any reason for enclosing image in ```span``` tags??


Are you testing in production?


@PeratX What exactly this fixes?


LGTM


Ok, rebased to master.

Notes on testing:
- Revert the highestAvailable commit
- As this changes the default location of config_dir, please delete HKCU\Programs\OpenVPN-GUI  (or just the config_dir key) before testing it. The HKLM keys set during service install are ok (i.e., point to the system wide location.)
- Start the GUI  and configs in both system-wide config_dir and users config dir (if any) will appear in the menu. If the user config_dir does not exist it will be created on first use as OpenVPN\config in user's home
- Add some config files (anything.ovpn will do) to the either location and see all appear in the menu
- Add a config with same name to both locations and see only one appears in the menu (duplicate warning will be shown only if the GUI is restarted)


Very cool :-D

I've made comments in-line, I like the approach, as it retains the initial session identity throughout the request which means that per-api policies are only used for specific scenarios where they apply.

There's a lot of feedback below though - sorry about that :-/ It's mainly down to how deeply policies go into other components of Tyk, with a duplication approach like this, where a base Identity is kept throughout and a copied api-specific one is referenced where needed, the touchpoints need to be managed. If we used the "identify and replace" method originally proposed, then these effects would trickle through (mostly, although I'm sure we'd have a whole set of new issues)

Deep breath...

With the additional lookup in `applyRateLimits` we are adding another redis roundtrip to every request through the rate limiter, since we need to fetch the per-api session.

Also, this approach of having a separate session that we check, means that we lose ability to check the context version of the session 9which is nice and fast, since it's in-memory), and would need to round-trip constantly, i.e. the scenarios where policies (currently) are applied are:

- Rate limiting (covered here)
- Quotas (covered here)
- Access rights (on a [per-api & version]((https://github.com/TykTechnologies/tyk/blob/develop/middleware_access_rights.go)) basis)
- [Granular access](https://github.com/TykTechnologies/tyk/blob/develop/middleware_granular_access.go) - these are method and path specific access rights

For the last two, we'd need to do two more redis lookups for the right session in order to grab the correct session object. The reason why by default we look up the session object in the context is pure speed, we wan't to avoid touching redis.

My suggestion would be:

1. Write a helper method that does the per-api session fetching from redis (`GetAPISpecificSessionStateForThisSession(s * SessionState, r *http.Request) SessionState`) - and use in `(k *RateLimitAndQuotaCheck) ProcessRequest` just so that logic is isolated
2. Write another helper method that will check the current in-context session object for a valid per-api session key (`GetSessionKeyForPerPolicySession(s * SessionState) string`) we can then use this to easily do context lookups
3. Make the function you created in (1) also write this session object to the context: `go context.Set(r, apiSessionKey, thisSessionState)` this way, we can easily override the other policy-specific elements lookup for non-synchronised middleware like access control and granular access control by just getting an api-specific key from the existing session, then grabbing it from the context so we don't need to round-trip to Redis each time.

With that, we get the full benefit of per-API policies with little code change :-)

There is another potential thing to think about, by binding per-API policies to a token, and not to a policy, you can't "upgrade" a token simply by swapping it's one policy reference, we'd need to change it's per-api policy references. I guess this isn't so bad, but it does mean that we can't manage thousands of keys simply by changing one policy

So a bundle of Bronze, Silver, Silver policy plan for three APIs, needs to be changed per token, instead of just having a BronzeSilverSilver policy, in which you change the per-API links and then all holders of this policy automatically get the benefit of the new policies, I'm still trying to work out if this is a real concern, since you could just modify the per-API-policies, but then you might affect other users of that policy - so it depends on the level of abstraction/flexibility needed. The latter is more future proof, but making my head spin.

Not a biggie though, I think we could abstract that out further later if need be.

The last thing worth considering is getting rid of these tokens somehow. If we delete the base key, we also need to kill off all the api-specific tokens too, otherwise they will pollute redis (they might also cause an issue down the line as they are actually valid tokens that might pass an auth lookup if they could be guessed. This will only need to be done in the API [here for regular keys](https://github.com/TykTechnologies/tyk/blob/develop/api.go#L360) and [here for direct hashed access](https://github.com/TykTechnologies/tyk/blob/develop/api.go#L402), I don't think anything else deletes tokens.

*phew!* this is much deeper than I expected this would go...

:-D


@hellboy81 why you comment here?


Сделано хорошо и аккуратно, жаль что реализован лишь функционал на 4 балла.
Подозрительно похоже на реализацию Анастасии Дец.

Задание 2.1 принято.


@joaquimrocha this is ready for your review again. Thanks for checking it so closely the last times!

I did this as a revert commit, instead of rebasing, simply because it was easier (I'm expecting a bunch of conflicts if I try rewriting history to avoid the original commit), but I can rebase it away if you prefer.


при вводе не числа появится NumberFormatException и программа упадет с ошибкой.


Electric-Service-Refactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1562 of 1563 tests passed, 0 test warnings)

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


已修正.


@Pub4Game fixes that when you join you can't see your items, except when you touch a block/edit armor.


+1 for creating a repository for V2 data
.. and to force-push a diet to original repo


Подозрительно похоже на реализацию Анастасии Хапаевой.


Making these deleted functions public would result in a better diagnostic message if someone should try to use them. Better get an error saying that you're trying to call a deleted function than a function visibility error.


This is python code. If you check what is done with this input, I don't think it will change anything. It do a loop to know the total size and it is used in a reshape. In both case, I think that it work in all cases for any [u]int* inputs. So I would not add this cast.


	public TileGrid(int[][] newMap){
		map = new Tile[20][15];
		for (int i = 0; i < map.length; i++){
			for (int j = 0; j < map[i].length; j++){
				if ( newMap[j][i] == 0)
					 map[i][j] = new Tile (i * 63.9f, j * 64f, 63.9f, 64.1f, TileType.Grass);
				else
					map[i][j] = new Tile (i * 63.9f, j * 64f, 63.9f, 64.1f, TileType.Dirt);
			}
		}
	} Fix!


Done!


Although not explicitly mentioned in the book, notice that rotating the device after pressing the cheat button clears the display that shows what the correct answer is. That seems like it would have been a good thing to fix too. The value you retrieve here from the `savedInstanceState` could also be used to decide whether to redisplay the answer.


Good point :)


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

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


This is not equivalent in the slightest.

![](http://i.imgur.com/FHsI3he.png)

Unless there's a compilation error, there will always be an artifact, and hence a download.


Задание 1 проигнорировано.
Работа с гитом зачтена.


@ see  http://www.webtoolkit.info/


'The features (e.g. attributes or functions) of...'


@srobertson no wheel on https://pypi.python.org/pypi/codd though...


@Chleboir-CZ Hej? Doporučoval bych někdy číst diff


same here


and here


I would argue that this is pretty necessary, given that these are cutting-edge builds designed for developers/people who are experimenting, who are the kind of people that would want to know exactly what changed. Plus, as I mentioned on one of your other commits, build status `!=` download being available.


i removed the wait_idle depcrated api.


here test should fail, no soft_fail


Я постараюсь
22 февр. 2016 г. 7:38 PM пользователь "Tatiyana Domanova" <
notifications@github.com> написал:

> нет, просто будь внимательнее:)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EugeneSayko/webChat/commit/5068ac3a9fe26264b7741ddcdcb1c60d78f225d8#commitcomment-16245335>
> .
>


@PocketKiller thx


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


Makes sense, will do.


The type of `mIsCheater` would more appropriately be `boolean` rather than `Boolean`. As a `boolean`, it would always be either `true` or `false` and would be initialized to `false`, which is appropriate. Whereas as a `Boolean`, it can have any of three kinds of value: `null`, a reference to an object encapsulating `true`, or a reference to an object encapsulating `false`. It gets initialized to `null`. To figure out whether `null` will work requires a fair bit of sophisticated examination of the rest of the program. Using `boolean` would have been both simpler and more efficient.


at some point we should move "objectType" into an explicit CPPML alternative. This will take up less space on disk.


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2140 of 2142 tests passed, 8 test warnings)

 * 6 tests had: AUD diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 2 tests had: ERR diffs.

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


I'm not sure I understand the "notify" addition here, you're actually load/unloading, right?


Is this different than `Promise.delay`?


yeah

On Mon, Feb 22, 2016 at 1:07 AM, porn <notifications@github.com> wrote:

> Hey! Isn't it 2016 already? =)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stormpath/stormpath-sdk-python/commit/f57110cf0eb212102e17a5552dec68569a125205#commitcomment-16234552>
> .
>


```py
>>> 1024 == int(Bytes('1KiB'))
True
```

and

```py
>>> Bytes('1024B') == Bytes('1KiB')
True
```

You can even do:
```py
>>> Bytes('1GiB') % Bytes('1000MiB')
'24MiB'
>>> Bytes('1GiB') / Bytes('1MiB')
1024
>>> Bytes('1GiB') / 1024
'1MiB'
```

I have put quite some effort into getting the units to match at all times.
You also always get the highest possible unit:
```py
>>> str(Bytes('1MiB')*1024*1024)
'1TiB'
```


Ресурсы надо закрывать. Логирование должно быть информативным. Задание 1 заканчивалось просьбой переместить проект во внутреннюю папку (сделай это обязательно).

Задание 2.1 принято.


Noooooo... :P


Probably not; I didn't know about that function. I'll change it.


is this a max size? or a chunk size?


They don't have the same behaviour right now, but maybe they should. I actually had to fix a couple of these kinds of `_.identity` violations once I turned this on for one of our projects.
shouldn't be to hard to implement.


Не ясно что произошло в коммите с сообщением "status". Постарайся быть информативнее.
Задание 1 принято


But, the execution is triggered through Cache


A couple of minor comments, but otherwise looks good.


Why do you move it as static ? What if it depends on a property (such as, if i want or not to activate this listener through a `$this->enabled = true` ? Having it static here limits things, DYT ?


@jrgm 0.56.2 is here if there is time to deploy it.


 при вводе какого числа?

>Понедельник, 22 февраля 2016, 8:43 -08:00 от Tatiyana Domanova <notifications@github.com>:
>
>при вводе не числа появится NumberFormatException и программа упадет с ошибкой.
>—
>Reply to this email directly or  view it on GitHub .


آقا سلام علیکم استاد کار واجبی باهات دارم ایمیل زدم ظاهرا خرابه ایدی تلگرام داری ایدی من اینه @hussainmir
و لطفا به من ایمیل بزن بهت نیاز داریم shm379@live.com
اینم سایت gs-co.ir


The following doc(s) have been deployed to the [primergpat](http://primergpat-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/redirect-incoming-requests](http://primergpat-static-docs.pantheon.io/docs/articles/sites/code/redirect-incoming-requests)


Thank you, Thomas. I'll probably just hold out till the next version.


just leave a message.


schemas++


Nothing to say


This file really ought to be called "CompilerMapKey.hppml"


Looks like with the balance reset, it's not testing for that bug anymore. "There I fixed it." :)

Might be fine, probably the ledger tests are a better place to check for that issue. If that's the decision, we should just take out the TODO here.


Задание 1 сделано наполовину (без status и log), зачтено.
Задание 2.1 сделано отлично.


OK, my understanding is process_start_time_seconds and process_cpu_seconds_total is supported on corefx, and makes sense on Windows and Linux. Suggest we move the rest to a windows specific namespace.


Because of the way Jekyll detects posts, the file name [*must* start with a date in the form of `YYYY-MM-DD`](http://jekyllrb.com/docs/posts/#creating-post-files).


function - 171frames
thread - 174frames (approx)
Remove comment - 164frames
Get next/prev pixels only if green - 49 to 55frames
Check right/below only - 41frames
Commenting out if's - 30frames


You assumption that it never worked without it is false. I had it working in many config without this and it was working pretty well. So the problem is only in restricted cases.

I have no objection to this, but can you describe in more detail the environment where you have this problem?


this went wrong i guess i will just start from beginning undo all


All universal mode tests should have asserts for the connector's balance, because universal has a failure mode where the connector loses money. We should check for that.


yes,

There are no longer any public keys for `vagrant` and `root` included in the AMI. `cloud-init` takes care of it now.


Is there a case where this isn't a string? I don't think we actually want to set undefined if it isn't a string.


Файл лога можно не включать в отслеживаемые файлы.
Задание 1 и 2.1 приняты (умничка).


I see


Thinking of using the faster (messyier) green screen removal for the live camera view, and then running the image through the more intensive green screen removal function once a picture has been taken


you could also do

```
return std::string(this->tagName()).substr(/*chop off 'Memoizable' prefix*/)
```


I was more wondering why the layout had to be renamed.


It was so that we could put the generators inside a `generators` folder (at least says Yeoman docs). Maybe it's not needed. I forgot that it might have side-effects with npm.


Although not mentioned in the book, you ought to have noticed that rotating the device after displaying the answer not only cleared the result but also cleared the display itself. It would have been good to use the value you retrieve from the `savedInstanceState` to also redisplay the answer if appropriate.


Старайся никогда не использовать абсолютных путей. Тем более что пользователь не знает откуда ты будешь загружать сообщения.


> But if we just leave a info._oracle['client'].auth_token here doing nothing, it will not make sense to anyone who looks at this part of the code.

That's what comments are for :-)


Sometime you'll have to show me what this report is supposed to look like, and we can figure out how to put it on the site.


The overall approach is "creative", but this part no more than others :)


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 34.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -31,5 +31,5 @@

 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
-    (good_file1,good_file2,),
+    (good_file1, good_file2,),
     (bad_file1, bad_file2,))
```


In that case, I don't think even having a MessageId class makes sense; I'll replace it with a function that wraps the uuid generation.


Hi @mcatanzaro , no problem! I'd prefer however not to have a reversion for a commit that comes in the same PR. I think that to delete the original commit or revert it, the conflicts should be similar (you did have conflicts to revert this, right?). (It wasn't clear to me if you know this, but in case you don't, you can delete commits from *rebase --interactive* and I can help if you need)


**Shortlog of HEAD commit is too long.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


Had to bump the version to get the wheel to upload

On Mon, Feb 22, 2016 at 8:48 AM Thomas Grainger <notifications@github.com>
wrote:

> @srobertson <https://github.com/srobertson> no wheel on
> https://pypi.python.org/pypi/codd though...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/trivio/codd/commit/64d91cc833bded9e395fc633d603032b14a071ce#commitcomment-16245670>
> .
>


Умница.


I've moved the changes to `add_log_handler` to the original place where the name get's changed. Also when `add_log_handler` is used after the rename, it's already used according to the new interface.


looks solid!


//My team, 2ez, are creating a cashier's POS. It will be able to keep inventory, keep track of the items that are currently out of stock and in stock, QR code compatible, and calculate totals and subtotals for a customer.


@gribozavr There is a small problem with this, because this `clang` invocations do not take any auto-linking into account. And this results in some test failures when my pre-specialisations PR lands.

As a workaround, we could just invoke `swiftc` in the previous line with `-O` or we could add `-lswiftSwiftOnoneSupport` to the Swift command-line. I'd prefer the `-O` solution, if it works for you.

What do you think?


Si vous prenez exemple sur ce qui se fait, les fichiers de templates possèdent généralement deux extensions : le type de destination + le type de source.

E.g.:
* `.html.erb`
* `.html.haml`
* `.css.scss`
* `.css.less`
* `.js.coffee`
* `.js.jsx`
* `.json.jbuilder`
* `.xml.builder`
* etc.

Voir: http://guides.rubyonrails.org/command_line.html#rails-generate

Prenez exemple sur ce qui se fait.


我可以在github上面的留言可以做分類嗎？積了太多留言有時候會漏看而且難查找......


The new string or `null`.


If you're using Beta, v1.7.0-beta0 will include this, and should be released in a week. The stable release, v1.7.0, should be out two weeks later.


Great, many thanks for the fix!


test


LGTM, u can go ahead and merge


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


such hax, lol


minionSocket*


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


This nested destructuring and assignment is fascinating.

Could this be simplified to:

```js
let {result = defaults} = options;
```

And then ultimately return the `result` instead of mutating the supplied `options`?


How/where is this one used?


Remove?


@mirkoklukas See @cogmission's comment here https://github.com/numenta/nupic/pull/2755#issuecomment-187267227

I think you are looking at this commit in isolation when you should be looking the entire PR's diff.


I agree, this code should just link using `swiftc`, too.


если в консоли вместо числа ввести напримем "qwe" то оно не сможет распарсить его как Integer и упадет.


Thanks, I'll let you know when I have more content that might interest you.


missing `;`


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


There was a bug in this commit, which is fixed by https://github.com/webpack/webpack/commit/65e801487cff82428340cacfcb7092ed7e73fbd8.

Could you check if it is already fixed in the latest version? Elsewise I need a piece of code which is broken to investigate..

---

The change fixes the `this` scope for imported functions.

Example:

``` js
// module.js
"use strict";
export function test() {
  this === undefined
}

// call.js
import * as x from "./module.js";

x.test(); // scope in test must be undefined and not x
```


Should be "indicated by `#isolation_level`".


Kok aneh wkwkwk


Nice ad-hoc test framework. :)

I think it might be worth using Mocha (as in `co-mocha`), before we slowly reproduce all of its features. Doesn't have to be in this PR, but it wouldn't be very difficult to switch over.

On a related note - the setup and test phases should be much more separated I think. Perhaps even to a point where the outside interface is `integration setup && integration test`


Can you make this to a for loop, too, to be consistent?


asa


Doesn't the unlock need to happen before the free? Otherwise I think there's a race condition here; a different thread/process may have allocated this memory in the mean time - and it may not want it locked for the short duration between lines 100 and 105.


Should say more about this. I haven't thought a ton about what a phantom read means for an RDF datastore.


Also, instead of `options || {}`, could the function parameter use a default value?

```js
const mergeDefaults = (options={}) => {
```


Taaac...

www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
☎ (+39) 340-9090490

2016-02-22 17:15 GMT+01:00 Federico Monaco <notifications@github.com>:

> e la madona (alla Pozzetto)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16244681>
> .
>


Yes, at least this parameters do not work


Unsolicited advice from a nitpicky bitslinger:

Naming variables with descriptive names helps a lot for code readability
Code is often written once, but read multiple times. You may have known exactly what s1 meant _when you wrote it_ but when I read it now, I have no idea what the intent is. You may not remember next week, even though you wrote it.

This nitpick brought to you by experience!


Good. Any new opinions on how to get round-trips to work? Is hacking on ultrajson the way to go here?


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


Je propose :
[str(entry.recent_food)] ! Encore ? Ma bouche me fait mal.
et
Du calme langue ! Fait semblant d'être allergique aux [str(entry.recent_food)].


Ciao,
hai un'oretta ora?

Federico

http://www.federicomonaco.com

just married online http://www.cvpido.com

2016-02-22 18:05 GMT+01:00 Fabrizio Terzi <notifications@github.com>:

> Taaac...
>
>
> www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
> ☎ (+39) 340-9090490
>
>
>
>
> 2016-02-22 17:15 GMT+01:00 Federico Monaco <notifications@github.com>:
>
> > e la madona (alla Pozzetto)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16244681
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246133>
> .
>


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/11//violations
```
plone/recipe/codeanalysis/__init__.py:172:13: T001 print function found.
plone/recipe/codeanalysis/__init__.py:187:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:196:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:244:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:59:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:66:13: T001 print function found.
plone/recipe/codeanalysis/tests/test_csslint.py:24:0: P103 other string does contain unindexed parameters
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Calling `_processingEngine.AreTasksPending();` in the parent thread might fix it, but I would prefer something more explicit, like adding a new method `IProcessingEngine.Initialize()` that should always be called from the parent thread.


Hi


sgtm, process_windows is probably best


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


Electric-Service-Refactor (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2113 of 2113 tests passed, 3 test warnings)

 * 3 tests had: AUD diffs.

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


Nice stuff, can you pls update CHANGELOG too?


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


Was there any reason why it was not using `swiftc`? I though that it is using `clang` probably because it wants to link object files in a specific order, i.e. `swift_begin.o` first, then `CXTest`, then `swift_end.o`. But if we can use `swiftc`, it would be even better and would fix all problems, because `swiftc` would consider auto-linking if required.

Can you commit a quick fix for it?


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


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


Ah. I updated this morning to 2.1.0-beta.1 - you've been busy though, the latest version is fine again.

Cheers


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


si :)

www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
☎ (+39) 340-9090490

2016-02-22 18:07 GMT+01:00 Federico Monaco <notifications@github.com>:

> Ciao,
> hai un'oretta ora?
>
> Federico
>
> http://www.federicomonaco.com
>
> just married online http://www.cvpido.com
>
> 2016-02-22 18:05 GMT+01:00 Fabrizio Terzi <notifications@github.com>:
>
> > Taaac...
> >
> >
> > www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
> > ☎ (+39) 340-9090490
> >
> >
> >
> >
> > 2016-02-22 17:15 GMT+01:00 Federico Monaco <notifications@github.com>:
> >
> > > e la madona (alla Pozzetto)
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16244681
> > >
> > > .
> >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246133
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246195>
> .
>


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


I think this can just be `ring/wrap-params` without the anonymous fn


We used latest Cordova Purchase plugin 4.0.0. And Now.
After the Inapp purchase through testflight using internal tester,

And if you loose the focus from the app (back to home page/ open any other app )

Then  it is asking for the credentials  to "Sign In to iTunes Store" only for one time.
 after providing the credentials once,
Then it is not asking for the credentials continuously for that InApp purchase . (This is little better. )

Is there any better solution than this ?


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


I'm not sure we need this. Can you explain what this does?


Nice catch!


I think it's the variable name in the API that Kevin has no control of.


This change is flawed as the inner continue should probably continue the outer for while it continues the inner one instead.


@ofnlut instead of LiftMech here should be an instance of liftmech passed down by butttons or triggers in OI or robot. This was the same issue with shifter.

Look at these for reference:
https://github.com/Firebird1915/Bobpy/blob/master/oi.py#L28
https://github.com/Firebird1915/Bobpy/blob/master/commands/speed_toggle.py#L5


@Sunilkumarr  previewThumbnail = FileUtils.getPreview(filePath, reqWidth, reqHeight, ApplozicSetting.getInstance(getActivity()).isImageCompressionEnabled());


I tried looking for you and it showed 2 different people with that same email address and when I went to start a conversation with them it said neither had Google Hangouts. :\

My email is csimmonswork@gmail.com <- always on Hangouts


Created bug #790 for this problem.


What are the expected values for `parameter.schema.$ref`?


Graph of your move importance function (and ratio1 ) tells  at which tc to expect problems. I just looked at it, and thought that at sudden death problems are likely


This cannot be an ok(), it has to be something like:
`return Response.status(Response.Status.BAD_REQUEST).entity(e.getMessage()).build();`


This seems to require to be constantly injected...


>  It's possible that these are .net metrics rather than process metrics, so all may need to be prefixed by the name of the VM if they're coming from there rather than the OS

The metrics coming from .net are already prefixed with "dotnet". Are you happy with that convention? Or move to "process_dotnet"?


[style] Minor nit, missing space before `{`.  At the moment the spl has a ton of other style issue but we can at least avoid adding new ones.


same


Thanks. We missed that. Will merge as soon as I'm free. Seems there's some renewed interest in this little exercise that we did a few years back.


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

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


From #3474


This is incorrect, the command *is* executed by `/bin/sh -c` (see `ClangShellCommand::executeExternalCommand`). Are you seeing something which makes you think that is incorrect?


@joaquimrocha Rebasing will always lead to more conflicts, and conflicts that are harder to solve, because they come one step at a time, lacking the context of later commits. But it almost always leads to nicer history; I'll do it. :)


Hi Mr Antonio Lignan
I m trying to implement Elliptic Curve cryptography algorithm such as ECDH and ECDSA in Contiki Rime Stack in multihop executing in Cooja simulator but first I need to know..whether they implemented any security algorithm in Contiki Rime stack or not..I already tried the contiki-tls-dtls-master which is not working.I think its not working for skymote..I tried several codes but still not working..the normal FILE is working as we use CFS in contiki OS..So pls help me with this question.
Thank you in advanced.


This doesn't strip units. Some newb gonna try `1600px/900px` and it'll die.


J'approuve totalement !


http://beta.brlcad.org/ is this the new one?


ora ti chiamo...
;P

http://www.federicomonaco.com

just married online http://www.cvpido.com

2016-02-22 18:11 GMT+01:00 Fabrizio Terzi <notifications@github.com>:

> si :)
>
>
> www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
> ☎ (+39) 340-9090490
>
>
>
>
> 2016-02-22 18:07 GMT+01:00 Federico Monaco <notifications@github.com>:
>
> > Ciao,
> > hai un'oretta ora?
> >
> > Federico
> >
> > http://www.federicomonaco.com
> >
> > just married online http://www.cvpido.com
> >
> > 2016-02-22 18:05 GMT+01:00 Fabrizio Terzi <notifications@github.com>:
> >
> > > Taaac...
> > >
> > >
> > > www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
> > > ☎ (+39) 340-9090490
> > >
> > >
> > >
> > >
> > > 2016-02-22 17:15 GMT+01:00 Federico Monaco <notifications@github.com>:
> > >
> > > > e la madona (alla Pozzetto)
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16244681
> > > >
> > > > .
> > >
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246133
> > >
> > > .
> >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246195
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246328>
> .
>


Cool. Thanks for the merge.


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1002 of 1002 tests passed, 0 test warnings)

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


Didn't even know i committed this, realised after why you did it. Forgot about all the Sass quirks :D


Ok, added a test to ledger: https://github.com/interledger/five-bells-ledger/pull/161


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - 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.svg)


I suspect the lines referencing foo were added in testing?


Although not mentioned in the book, note that rotating the device after showing the answer not only clears the result but also the display itself. As such, it would be desirable to use the value you get back from the `savedInstanceState` to also redisplay the answer if appropriate.


Saved door groups seems to have messed up after this change?


The right target here is not the (mostly deprecated) Num (Big_int) package, but instead ZArith (https://forge.ocamlcore.org/projects/zarith). But so far, I've had trouble getting ZArith installed.

Charlie, perhaps you'll have more luck.

If I remember correctly, people report about 3x or 4x (?) slowdown int->zarith. Probably we'll get at least a 2x improvement vs. Coq Z, with the added benefit that extracting to zarith is still sound :-)


If you had 3 layers, strongest at the top:
default key
non default key
default key

The top most layer would never write out anything and would not be correct

Sent from my iPhone

> On Feb 22, 2016, at 6:32 AM, Matteo Spinelli <notifications@github.com> wrote:
>
> why did we do this?
>
> ―
> Reply to this email directly or view it on GitHub.
>


process_ is for OS-level metrics, anything coming from dotnet should be dotnet_ (or mono_ etc. if it's VM implementation specific)


da


Would a better fix just be to have _sslContext not be readonly?


The `KEY_INDEX_ARRAY` is the first aspect of an elaborate work-around for using `putBoolean` and `getBoolean` where `putBooleanArray` and `getBooleanArray` would have been more appropriate.


Just to clarify- I was under the impression that the profile model had been removed?


I have, thanks!


personnellement j'ai une préférence pour le 3ème choix


Is this not assignable as is?


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2108 of 2110 tests passed, 8 test warnings)

 * 6 tests had: AUD diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 2 tests had: ERR diffs.

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


Done. Change these fields to `int`, and zero them out every time `update_odometry()` is called. It's very, very unlikely it's going to overflow even with the smaller type. That's what the code this is based off of does, which I forgot to factor in this version.


code cleanup because we're not using labview smartdashboard


@vtjnash @JeffBezanson If you can pick up this updated version for https://github.com/JuliaLang/julia/pull/15176, it makes those constants scalars instead of arrays, as requested, and also adds test cases that don't depend on the string type hash function.


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


`swiftc` would take care of begin/end files, so that would even simplify things further.

Sorry, I don't have time now to prepare a fix.


Probably shouldn't be skipped


missing doc for @ private formProvider and urlParams


gitmate will complain about this


Can you clarify where the documentation recommends this? I found that this line caused the JSON-schema validator to fail: http://jsonschemalint.com/draft4/#

See also "The "$schema" keyword":  http://json-schema.org/latest/json-schema-core.html#anchor23

Not trying to nitpick- I'd like to be sure that any schema edits I make also adhere to the correct spec in the future. Thanks.


as long as this is placed at the end of all of the CSS then it's fine, otherwise you might consider:

```
:last-child:last-child {}
```
yes, that actually works!

Or less clunky
```
[class]:last-child {} // all last children with a class
:not([class]):last-child {} // All last children without a class
```


mark null as code -> `null`


Sorry, asi jsem neuložil změnu. Bude napraveno.


merge


It is skipped because its failing -- I don't want to break CI until this bug is fixed. Though incidentally, CI is broken for this branch (I'm not sure why yet).


why did you revert my changes from 0b7eb9b01


`helpfully` is the correct spelling


This cannot be an ok(), it has to be something like:
`return Response.status(Response.Status.NOT_FOUND).entity(e.getMessage()).build();`


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2108 of 2110 tests passed, 8 test warnings)

 * 6 tests had: AUD diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 2 tests had: ERR diffs.

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


Le troisième est bien si tu n'a que des petites fonctions de ce genre
S'il y a de grosses fonctions, il vaut mieux partir sur les protos je pense


This cannot be an ok(), it has to be something like:
`return Response.status(Response.Status.BAD_REQUEST).entity(e.getMessage()).build();`.


\* The game actually does this itself, but it doesn't call any binds.


w


This cannot be an ok(), it has to be something like:
`return Response.status(Response.Status.NOT_FOUND).entity(e.getMessage()).build();`.


I believe `isObjectOrArrayOrEnum` is no longer needed here.


**Code Quality - Critical**

Why do we have this block of  code repeated in 3 blade files? :open_mouth:

Why do we have 2 master blade files? Please make the code DRY, we should have only one master blade file that every blade file extends.


So you're saying just leave these methods on the `this.props`?  There are no passed values being set on state here.

`this.onSwipeLeft = props.onSwipeLeft || noop;` is set once in the constructor so it still maintains a single source of truth.


I guess you mean `throw new LoggingConfigurationException("Invalid package exception");`?


"Proof"


**Code Quality**

A `.ico` file is an image/icon file. It should be in an image directory. Why do you have it in a css directory?


dashboard language ?


Applying an order on the query seems to fix the issue:

```diff
diff --git a/app/models/outgoing_message.rb b/app/models/outgoing_message.rb
index c4bf337..3a01a0f 100644
--- a/app/models/outgoing_message.rb
+++ b/app/models/outgoing_message.rb
@@ -213,7 +213,10 @@ class OutgoingMessage < ActiveRecord::Base
   #
   # Returns an Array
   def smtp_message_ids
-    info_request_events.map { |event| event.params[:smtp_message_id] }.compact
+    info_request_events.
+      order('created_at ASC').
+        map { |event| event.params[:smtp_message_id] }.
+          compact
   end
```


I guess you mean `throw new LoggingConfigurationException("Invalid level exception");`?


This is an issue. Given a uuid `1fa5...` mysql matches records with id=1 and uuid=`1fa5...`. Rails sort results by id. id=1 won. :boom:


number format


Will do.  Excuse my ignorance, but could you give a hint on how to run tests individually?  This site indicates that I can just name a test: https://github.com/cython/cython/wiki/HackerGuide#getting-started.  However, I've been grepping for about 10 minutes now, and can't find a .py file in the tests that runs the various cpp tests, either individually or in combo?

For example I'd like to somehow test tests/run/cpp_smart_ptr.pyx and/or all tests/run/libcpp_all.pyx on their own.


date format


this should read ul (UL, short for unordered list), not ui (UI).


are you ok with committing your api keys?


dashboard language


This, on the other hand, is an example of the anti-pattern you're describing. https://github.com/project-gustave/gustave-client/blob/c54e645d4eee69e823d4a545ecd72058b8a5da29/components/recommendations-scene/edge/index.js#L22-L24


This can be just `return Response.ok().build();`.


This code looks great and you've clearly beautified it for better readability. I would spellcheck your work before you commit and check out your unordered list. It should be ul, not ui.


http://free-technology-guild.github.io/Learning-City-Point/#pages/about.html

www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
☎ (+39) 340-9090490

2016-02-22 18:19 GMT+01:00 Federico Monaco <notifications@github.com>:

> ora ti chiamo...
> ;P
>
> http://www.federicomonaco.com
>
> just married online http://www.cvpido.com
>
> 2016-02-22 18:11 GMT+01:00 Fabrizio Terzi <notifications@github.com>:
>
> > si :)
> >
> >
> > www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
> > ☎ (+39) 340-9090490
> >
> >
> >
> >
> > 2016-02-22 18:07 GMT+01:00 Federico Monaco <notifications@github.com>:
> >
> > > Ciao,
> > > hai un'oretta ora?
> > >
> > > Federico
> > >
> > > http://www.federicomonaco.com
> > >
> > > just married online http://www.cvpido.com
> > >
> > > 2016-02-22 18:05 GMT+01:00 Fabrizio Terzi <notifications@github.com>:
> > >
> > > > Taaac...
> > > >
> > > >
> > > > www.fabrizioterzi.eu <http://fabrizioterzi.eu/>
> > > > ☎ (+39) 340-9090490
> > > >
> > > >
> > > >
> > > >
> > > > 2016-02-22 17:15 GMT+01:00 Federico Monaco <notifications@github.com
> >:
> > > >
> > > > > e la madona (alla Pozzetto)
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16244681
> > > > >
> > > > > .
> > > >
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246133
> > > >
> > > > .
> > >
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246195
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246328
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PUNTOZERO/puntozero.github.io/commit/12f7d529d7533505517c5e190eb8cc7f31bdfffd#commitcomment-16246543>
> .
>


Good points. I would be happy to support having one version of gensys throughout the julia world.

To do that we need to resolve points your points 1-3 and get the NYFRB on board. Would you be willing to collaborate to make that happen?

I'm not sure what you mean by your last question. The test included in the fed's DSGE.jl package just pulls out those matrices as they were computed in matlab using Chris' original gensys code and compares them to the Julia output.  Is that what you were wondering?


Seems to be a mixup of types. filters is not of type EINTERLACEMETHOD.


You can remove lines 34-36.


@eamonnmag This is how I'm working around the problem we were having the other day. It's a little hackish, but it does the job.


Just the README to be followed by the second commit with the entire app files


Tested and working.  Thanks.


The mkl in Anaconda has this problem.  Also the one in Canopy.

I have personally never observed Theano successfully linking with mkl with any of the free options.  I also did not try any of the commercial options, but I see no reason why they should be significantly different.


Again, you are not throwing any of these exceptions (here and below). Please, fix!


...Here as well...


Let's make these changes on the original change lists before they are merged instead of here.


or map[charm.URL]bool

(which is faster FWIW)


I mean :oden:


Cool, thanks @jacquerie. I got around my problem through being able to supply the user instead of assuming presence of a current user. It helped to split up the code a bit too.


The script has used a combination of `swiftc` and `clang` from the very beginning: https://github.com/apple/swift-corelibs-xctest/commit/f740cdbb560da6b76fd20a0853a79ecb4ebe32dd It may be worth asking @parkera if he had any particular intent here.

I'd be very much in favor of simply invoking `swiftc`!


Make sure text tracks do not keep rendering in iOS. Maybe they need to be disabled first.


Perhaps `T, R extends ObservableInput<T>`?


I see an edit I have to make; also want to do another revision on the intro, now that I am reading it here.


upload java project from lab 2


Let's fix this double quote in this change list before merging.


You can define this as a constant, no need to re-instantiate a new `List` each time.


Same.


I may be wrong, but doesn't `JsonConvert.DeserializeObject` use the constructor for responses? Or does it honor `DataMemberAttribute`? I don't use the data contract attributes, and haven't for a few years, but I seem to recall only DataContractSerializer supporting the attributes.

The spec makes it sound like there's a need for custom serialization logic (if DataMember properties with private setter isn't honored).


I agree with @ChrisDBrown.  There's something that doesn't seem quite right about this commit.  Perhaps you misspelled something?


Meh. A bit.

The point of these code cleaners is to make a good faith effort to save you from yourself, until we get to the point in PHP where we can reliably just try things and see if there are exceptions. Whenever it's a choice between disallowing valid code or letting you write potentially invalid code, we have to err on the side of letting you do something invalid.


Close #166


That does seem like an odd place for it. Woops!

Can definitely delete this.


"Beautify" this please for better readability. Too much white space. Other than that, great job!


Strange, I can confirm that it doesn't work on my Windows 7 and 10 machine, but it did work on another one (not sure which version). isxdl is so stupid, time to get rid of that. Why can't Inno Setup download files in the first place?


The following doc(s) have been deployed to the [945httpho](http://945httpho-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/configuring-settings-php](http://945httpho-static-docs.pantheon.io/docs/articles/drupal/configuring-settings-php)
- [docs/articles/drupal/drupal8](http://945httpho-static-docs.pantheon.io/docs/articles/drupal/drupal8)


All right, noted.


can you choose more meaningfull names .... TextBox, Feld ....textBox of which content? Whats the field about? .....

pls, change to lowercase after the "_"

and may you can choose english names?


Also, might be good to have some of that variadic signature generation magic of yours here.


now there's one too much, put your cursor here, press backspace *once*, save!


Theses keys are not valid anymore ;-)


Actually, i will be moving to resolving types as well.  The work to just get the assembly path was a first step approximation back when we were trying to make the RC.  For RTM though i am going to want to verify the type exists in the assembly in the context of the project we're working in.


Feel free to remove these // TODO comments.


good to merge now, but could you capitalize to "Curation tool"?


same for "Platform"


Array<Array<Object>> the key is a string but the value can be anything, since * is out of the equation we are stuck with `Object`


Brilliant :stuck_out_tongue_winking_eye:


Don't `meta_*` options have to be in the first pattern entry?


That said, if we can be reasonably certain about not getting false positives, we can add self, parent and static checks back in. That's just a lot more intense because it requires knowing more about inheritance, what else is also defined in this class, etc. It's not impossible, just requires more static analysis than I wanted to knock out as a bug fix before leaving for work on a Monday :P


code undefined -> `undefined`


this looks good, but please beautify it. Too much white space.


what is this package meant to be?
there is no code for it, the alpine jazz hands builder looks for `varnish-header` and it's not there so it fails.


LGTM


Now I'm conflicted. I'd like to keep compatiblility for the modules, as keeping two differnt versions of each one would be a pain. (There are already four additional modules for my addon that are not in the stock collection.)

On the other hand, I'd really like to see how you would simplify this. I could stand to learn how to do that.

And if you'd like to save some more time, the front page modules have also been done and are available as an addon.


Could use first 3 of song title to get it a bit smaller and maybe filter if default author is set to defaults :D


This will also prevent the removal of non stratux files from the root directory


{Array\<Array\<Object\>\>|undefined}    and code undefined -> `undefined`


for some reason the base16 schemes aren't showing up nice in ubuntu's terminal. Zenburn is still great though...


Shouldn't this be the other way around? Otherwise the value could have changed after we read the version.


Um...think you made a typo.


Rats.  You're right.


if its not a final property pls use camalcase _menubar1_menu1 -> _menuBar1Menu1
may you find more meaningfull names .... what is Menu1 -> options, info, exit, save, open etc ....


Array\<Array\<String\>\>


Better now?


Minor wordsmithing to get around removing "application hang". Technically, the application may not hang depending on what the app is doing.

> The application must call fi_trywait and obtain a return value of FI_SUCESS prior to blocking on a native wait object. Failure to do so may result in the wait object not being signaled, and the application not observing the desired events.


[edittable.txt](https://github.com/partha20/Formvalidation/files/141096/edittable.txt)
[edits.txt](https://github.com/partha20/Formvalidation/files/141095/edits.txt)
edit and delete table


@parkera Hi Tony, is there any good reason for using `clang` instead of `swiftc`?


We should test if these state changes are cumulative


would it make sense to use :ship: (`:ship:`), :thumbsup: (`:thumbsup:`) and :shipit:(`:squirrel:` or `:shipit:`) here instead of their unicode equivalents since we're specifically talking about PR comments and they'll support the GitHub emojis?


CSV


do we really need width and height?
can choose more meaningfull names
x and y of what?
don't forget the leading "_"


Interesting way to make both class and instance methods! Trying to think of some DRYer way to do this other than having to define the class methods on each subclass (class methods are inheritable) but so far I don't see one...


TODO do this concurrently?


Fine by me. I'll merge and update the POT as well. Thanks!


indentation could be better


I'd say that if you have to ask, you probably don't need to know :). We can always improve later if it comes up.


indentation.


It would be good if there was a space between `:` and `OSX`


They implement the mkl differently... I had them working. On which OS do
you have see this problem?

On Mon, Feb 22, 2016 at 12:36 PM, abergeron <notifications@github.com>
wrote:

> The mkl in Anaconda has this problem. Also the one in Canopy.
>
> I have personally never observed Theano successfully linking with mkl with
> any of the free options. I also did not try any of the commercial options,
> but I see no reason why they should be significantly different.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abergeron/Theano/commit/f2100b06e1fc39f3f916484bec094d543063f0b7#commitcomment-16247005>
> .
>


I'm not totally OK with all the edits, for instance we now have normative language in notes, and a lowercase "required" has sneaked in.


initMap function is defined, but it seems like its never called. If its somewhere else, its not immediately obvious so comment would be helpful here.


End time must be 23:59:59 when the filter is run or data result gets empty for last day in the time frame.

Example:

Filtering data from 10/01/2016 to 10/02/2016 does not contain purchases from 10/02/2016 without that fix. (This should be save as long as it is never be tried to filter by time)


you need to use connect method before this


Cant see anything here, aliedowell. Commit and sync, please.


if you are working on side projects, using jquery is better than rewriting these yourself. $.get should be some thing, but with fewer lines of code.


Indexes here are 0-based, and we use 1-based indexes in streams.
How about `uint64_t first_index, uint64_t last_index` and allow `first_index == last_index` meaning range with one element?


you don't need "this."
pls format you code


Utils.addArgs


code undefined -> `undefined`


aaah now i see, its a placeholder, ok


Requires https://github.com/sv-esk/df-structures/commit/892285f7892b0435884be8924eed8a8d17926e6e


Managing return values when the input is an array of fids can be tricky. Even if one of the FIDs results in the application having to poll, the app might not be able to tell which one to poll on.

It might be that restricting this to one FID will result in simpler looking code, and a simpler implementation of try wait in the provider (eliminating loops). Might be important if wait objects are more lightweight than the current FD model.


I just mean multiple tests rather than one. So that I'm sure I don't change
anything when rewriting certain lines.
Would this gensys available only in DSGE or as a package that DSGE
would import?

On Monday, February 22, 2016, Spencer Lyon <notifications@github.com> wrote:

> Good points. I would be happy to support having one version of gensys
> throughout the julia world.
>
> To do that we need to resolve points your points 1-3 and get the NYFRB on
> board. Would you be willing to collaborate to make that happen?
>
> I'm not sure what you mean by your last question. The test included in the
> fed's DSGE.jl package just pulls out those matrices as they were computed
> in matlab using Chris' original gensys code and compares them to the Julia
> output. Is that what you were wondering?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/matthieugomez/HJBFiniteDifference.jl/commit/99a46282a8387788791fed17a998fd2c4f687e25#commitcomment-16246940>
> .
>


I'd rather not use LMR types for that either. We should just get a path and open it using MetadataReader. A couple of reasons - you can't get Roslyn symbols from LMR types. Second, long term goal is to deprecate LMR, so let's not add yet another usage.


Hi Alex,
I am trying to hide sheets through openxlsx. I use it to read an existing Excel template, put data into it and save it, using openxlsx. After saving the sheets that were invisible in the template become visible. I am not so familiar with Github, but I understand this commit addresses sheet visibility. Is it possible with the CRAN version of openxlsx to hide a sheet? If not, how can I do it? Thanks,
Pedro


I can live with that in HEAD. I can even live with that in 1.0.2g for the UEFI build, since we're paying enough attention on upgrades.
But for osslsigncode and any other hypothetical users of `PKCS7_verify()` in OpenSSL 1.0.2 this doesn't actually fix the regression, that their code *used* to work in 1.0.2d and now it doesn't work any more unless they pass in a new flag that didn't previously exist in this stable branch.


code null and undefined -> `null`  `undefined`


J=wiener2(invBW); %Eliminem soroll.


@ description


I had a couple other tests at some point, but I don’t remember where I put them.

I think it makes sense to have this gensys be somewhere else because multiple projects would rely on it.

// Spencer

On February 22, 2016 at 12:48:40 PM, Matthieu Gomez (notifications@github.com) wrote:

I just mean multiple tests rather than one. So that I'm sure I don't change
anything when rewriting certain lines.
Would this gensys available only in DSGE or as a package that DSGE
would import?

On Monday, February 22, 2016, Spencer Lyon <notifications@github.com> wrote:

> Good points. I would be happy to support having one version of gensys
> throughout the julia world.
>
> To do that we need to resolve points your points 1-3 and get the NYFRB on
> board. Would you be willing to collaborate to make that happen?
>
> I'm not sure what you mean by your last question. The test included in the
> fed's DSGE.jl package just pulls out those matrices as they were computed
> in matlab using Chris' original gensys code and compares them to the Julia
> output. Is that what you were wondering?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/matthieugomez/HJBFiniteDifference.jl/commit/99a46282a8387788791fed17a998fd2c4f687e25#commitcomment-16246940>
> .
>
—
Reply to this email directly or view it on GitHub.


I don't recall actually, let's put it back and see what happens.


Should I change it to "NODUAL" for both master and 1.0.2 and invert the flag?


`flat_map` flattens the result of the map, rather than the enumerable:

```ruby
# mta_ids.map…
message. exim_mail_server_logs.map(&:class)
# => [ActiveRecord::Relation, ActiveRecord::Relation]

# mta_ids.flat_map…
message. exim_mail_server_logs.map(&:class)
# => [MailServerLog(id: integer, mail_server_log_done_id: integer, info_request_id: integer, order: integer, line: text, created_at: datetime, updated_at: datetime), MailServerLog(id: integer, mail_server_log_done_id: integer, info_request_id: integer, order: integer, line: text, created_at: datetime, updated_at: datetime), MailServerLog(id: integer, mail_server_log_done_id: integer, info_request_id: integer, order: integer, line: text, created_at: datetime, updated_at: datetime), MailServerLog(id: integer, mail_server_log_done_id: integer, info_request_id: integer, order: integer, line: text, created_at: datetime, updated_at: datetime)]
```

Its more like doing `blah.map{}.flatten`


@thiagofigueiro Want to be helpful? Re-write it in Go! :trollface:


I am having issue with the firmware after this commit.  Ergodox Infinity on Win 7, the HID devices are constantly going into a conflict mode then resetting 10 seconds later.  The keyboard works fine (see layer screen updates etc) but there is no keystrokes sent to the PC.


@legoboy0215 use `if()`


Проверка


Oh "this.{x} should be used for state that has nothing to do with a render"


hier müssen wir überlegen ob es sinnvoll wäre die ActionListener nicht anonym zu erzeugen, dann haben wir die Möglichkeit sinnvoll namen zu vergeben, sowas wie createAndSetField(...)


well, i feel like that it adds too much information that people can just check by going to the build.


die methode kannst du komplett löschen


@Kymation the basis of my thought is to use a similar approach as is done at the "social bookmarks" module.  This has an extra page in admin for add/edit/delete the submodules (which make up the main module).  It's basically the same as you have done...just a little bit cleaner.  Keep an eye on this Branch - as you have already done some work in this area I woud be grateful for any pointers if I run into problems. Thanks.


Hmm, seems like windows is being slow (as usual). I'll see if I can
reproduce this evening.

On Mon, Feb 22, 2016, 09:52 p-callaghan <notifications@github.com> wrote:

> I am having issue with the firmware after this commit. Ergodox Infinity on
> Win 7, the HID devices are constantly going into a conflict mode then
> resetting 10 seconds later. The keyboard works fine (see layer screen
> updates etc) but there is no keystrokes sent to the PC.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kiibohd/controller/commit/46765e85c57e19b89754f4f9c26c8f18b35148e5#commitcomment-16247459>
> .
>


Ok, then I guess we can change the download to an X if it failed


In it's current state the  centroiding software can be compiled using either the g++ or clang++ compilers.

clang++ -std=c++11 -pthread Source.cpp
or
g++ -std=c++11 -pthread Source.cpp


Agree it's too generic, but not sure adding `::ss::` would make it better.


This was bugging my mind and I couldnt focus on anything until I did something about it. I wasnt able to test it in a proper environment since I was at work and needed to do the changes very quickly.


Good point. I'm a big fan of ranges being `from, inclusive` to `to, exclusive`. Perhaps just add `1` to both ends?


:+1: b077b17


This was already mentioned on irc asking someone to take care of it p:  Know Aokromes was around when asked p:


Can we use Array<Array<string>> ?


what's the purpose of this `asModifiable`? Why is it needed? Seems like the tuple is discarded.


```patch
diff --git a/avocado/core/job.py b/avocado/core/job.py
index 9c785be..3f2857c 100644
--- a/avocado/core/job.py
+++ b/avocado/core/job.py
@@ -118,8 +118,8 @@ class Job(object):

         if self.show_job_log:
             if not self.silent:
-                output.add_log_handler(_TEST_LOGGER.name)
-                output.add_log_handler('')
+                output.add_log_handler(_TEST_LOGGER.name, level=logging.DEBUG)
+                output.add_log_handler('', level=logging.DEBUG)
                 _TEST_LOGGER.setLevel(self.loglevel)
                 _TEST_LOGGER.propagate = False

```


They aren't real keys. I mangled the actual keys with random characters and left them as an example.  The ip is fake too.

> On Feb 22, 2016, at 12:33 PM, Jason McFarland <notifications@github.com> wrote:
>
> are you ok with committing your api keys?
>
> —
> Reply to this email directly or view it on GitHub.
>


\</p\>


What's done so far, bear in mind *untested* !

Addition of new cfgm module to add the link into the modules sidebox in admin.
Update the main navbar content module to pull in submodules.
Update the main navbar template file to pull in getBlocks

which leaves:

make a bunch of submodules that plug into the oscTemplate->getblocks

I'll make 1 or 2 today/tomorrow...for testing...


Isn't this the same like 2 lines below?


As long as the chosen limits are part of the `$ticks` array, they should've been respected correctly. 0038b8ec8c3f59358445ea46bbfe1b2e1ae412b5 will force axis min/max to be exactly what the user has chosen.


Handing you some code deletion opportunities back ;)


I'm making progress with new code and I hope that we'll get rid of move_importance completely.


Fixed even..

:)


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


the three version is in HEAD branch, please check


cool!

guess it requires the change suggested in https://github.com/larshp/abapGit-Plugins/issues/3 to work?

will take a closer look when I have time


acho que o tipo deveria ser `[Vaga]` (e não Vagas), pq representa uma array de elementos únicos (Vaga), já o nome da property continua `vagas`, pq é exatamente isso que vai ter lá. Você faz o parse aqui quase do mesmo jeito que vc fez no outro comentário:
```
self.vagas = json["vagas"].flatMap{ Vaga(json: $0) }
```
Mas antes teria que mudar seu JSON de:
```
"vagas": [{
				"01": false,
				"02": true,
				"03": false,
```
para:
```
"vagas": [
                  { "id": "1", "occupied": true },
                  { "id": "2", "occupied": false },
                  { "id": "3", "occupied": true },
```
e também mudar o modelo para:
```
struct Vaga {
    let occupied:  Bool
    let id: String

    init?(json: JSON) {
        guard let occupied = json["occupied"]?.boolValue, id = json["id"]?.stringValue else { return nil }
        self.occupied = occupied
        self.id = id
}
```

Obs: o código pode ter erros de compilação, já que eu não escrevi no Xcódigo :stuck_out_tongue:


In my tests GameOver event always results in GameOver scene in RPG_RT. In EasyRPG it returns now to the map.


I didnt know I did all that :D


ujson is the fastest thing out there by far. Hacking it seems to be the way to go.


Favorite commit I've ever read!


What do the tests show?

On Wed, Feb 17, 2016 at 12:14 PM, Henry Herbol <notifications@github.com>
wrote:

> Ah, that makes sense. I'll see if I can upload the xyz test reactions and
> a self-contained unit test script soon. That way you can just run the
> script, and get back results on all the methods to see if things are
> changing or not. I'll do 2 test methods though, one full (1 to 2 days of
> simulations) and one quick (just CNH or something like that).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jminuse/clancelot/commit/886ee02fd54627b6b3122324d6331e3ca0e96b15#commitcomment-16153342>
> .
>


FYI - Missing semicolon.


That's exactly what I did: I copied the approach used in the Social Bookmarks module. No cfgm module needed. Compare the structure and you'll see what I did.

bm_product_social_bookmarks.php => cm_modular_navbar.php
social_bookmarks/<any module file> => content/navbar/<any module file>


I was thinking checking the limit here but not passing in the limit_msg -- instead raise a ValueError and then add the try: except in constant_op.py, filling in the exception message there.  Right @keveman ?


Hmmm seems like this would do the trick (see the last example in the accepted answer):
http://stackoverflow.com/questions/1251352/ruby-inherit-code-that-works-with-class-variables

However, it's causing my brain to explode...


@baltie...
Also I think we have an issue with these union types definitions :
```typescript
    /**
     * The X axis or category axis. Normally this is the horizontal axis, though if the chart is inverted this is the
     * vertical axis. In case of multiple axes, the xAxis node is an array of configuration objects.
     */
    xAxis?: HighchartsAxisOptions | HighchartsAxisOptions[];
    /**
     * The Y axis or value axis. Normally this is the vertical axis, though if the chart is inverted this is the
     * horizontal axis. In case of multiple axes, the yAxis node is an array of configuration objects.
     */
    yAxis?: HighchartsAxisOptions | HighchartsAxisOptions[];
```

See this piece of sample TS code that does not compile : http://www.typescriptlang.org/Playground#src=interface%20A%20%7B%0D%0A%09a%3A%20number%3B%0D%0A%7D%0D%0A%0D%0Ainterface%20B%20%7B%0D%0A%09b%3A%20A%20%7C%20A%5B%5D%3B%0D%0A%7D%0D%0A%0D%0Alet%20foo%3A%20B%3B%0D%0A%0D%0Afoo.b.a%20%3D%205%3B%0D%0A%0D%0A%0D%0A


ww


this should be done in the appropriate commits before, right?


If props are changed externally they may get out of sync with internal this.onSwipeLeft...

Better pattern I think is...

```
...
defaultProps = {
  onSwipeLeft: () => {}
}

render() {
  ...
 this.props.onSwipeLeft();
}
```


unack


If `swiftc` is the preferred approach now then we should go ahead and use it. I think we just used `clang` originally because we reused an approach from Foundation. However, there we had C files to content with - and that's not the case here.


@delarosatrevin , does this "SpiderWfMDA" protocol even exist? :)


foo.


ack


Yes, could be an option. And let's have the chosen convention documented :)


Thanks, pushed with your suggestion.


So, I just tried to replace the `clang` linking line with this:

`run("{0} -emit-library {1}/XCTest.o -o {1}/libXCTest.so -L{2}/lib/swift/linux/ -lswiftGlibc -lswiftCore -lm".format(swiftc, build_dir, swift_build_dir, arch))`

It seems to work for me.

But as you can see, I removed the two `-Wl` options, because somehow when I tried to prefix them with `-Xlinker`, swiftc would invoke `clang` with those options still prefixed with `-Xlinker` which would result in an error. But even without these options it seems to work fine.

If you think it is OK, would you commit the change or should I?


python runtests.py cpp_smart_ptr libcpp_all

On Mon, Feb 22, 2016 at 9:33 AM, Kevin R. Thornton <notifications@github.com
> wrote:

> Will do. Excuse my ignorance, but could you give a hint on how to run
> tests individually? This site indicates that I can just name a test:
> https://github.com/cython/cython/wiki/HackerGuide#getting-started.
> However, I've been grepping for about 10 minutes now, and can't find a .py
> file in the tests that runs the various cpp tests, either individually or
> in combo?
>
> For example I'd like to somehow test tests/run/cpp_smart_ptr.pyx and/or
> all tests/run/libcpp_all.pyx on their own.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cython/cython/commit/767fce81073e7f96c6e9dc95906d96795e61cc32#commitcomment-16246919>
> .
>


@parkera Even if there is C content (object files), `swiftc` can handle them.


lgtm, but I'm not familiar with how oc patch works (or any gotchas associated with its use)


Electric-Service-Refactor (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2114 of 2119 tests passed, 548 test warnings)

 * 553 tests had: AUD diffs.
 * 483 tests had: RDD diffs.
 * 1 test had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


In my implementation, I abort immediately on the first failure.  An application can use a poll set in conjunction with this call to determine which FIDs to poll.

But, an alternative is to move this routine from the fabric to the base fid class (or use fi_control) and push the iteration into the app.


Well yes, but it's not like I could think of anything other than "it works now" or like "div class changes" (since it was such a small change) but anyone could see that just by looking at the commit. Plus it's also funneled into an issue, so people will know why the changes are occurring, as a contribution towards the greater goal of the PR.

I get what you mean though, so I'll keep it in mind. I usually keep them normal(ish) if they aren't a "wait why in the world does this do/fix/cause this?"


I think this should come with a doctest example, really


T'as pas l'impression qu'il y a le même changement à effectuer une ligne au dessus ? XD


Is this pointer ever being deleted? I suggest for new code we should switch to the C++11 pointer types.


Zero-based indexes in Persister would not require further documentation :-)


评论就评论了，还删啥。。。 我只是突然发现可以一次动两格就那么写了
# kick your butt


ack though ti needs changing due to https://github.com/coala-analyzer/coala/commit/b1754b5c749ca87533f97f864db55ed0deb4276b


If you want to update the Foundation build script to use swiftc instead of clang, then put up a pull request there too.


@swiftix The `-Llib/swift/linux` part is not needed, `swiftc` implies that.


unack


Can you email me these tests if you find them ? I'm going to have a stab at
it this afternoon.

On Monday, February 22, 2016, Spencer Lyon <notifications@github.com> wrote:

> I had a couple other tests at some point, but I don’t remember where I put
> them.
>
> I think it makes sense to have this gensys be somewhere else because
> multiple projects would rely on it.
>
>
>
>
> // Spencer
>
> On February 22, 2016 at 12:48:40 PM, Matthieu Gomez (
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>) wrote:
>
> I just mean multiple tests rather than one. So that I'm sure I don't change
> anything when rewriting certain lines.
> Would this gensys available only in DSGE or as a package that DSGE
> would import?
>
> On Monday, February 22, 2016, Spencer Lyon <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
> > Good points. I would be happy to support having one version of gensys
> > throughout the julia world.
> >
> > To do that we need to resolve points your points 1-3 and get the NYFRB on
> > board. Would you be willing to collaborate to make that happen?
> >
> > I'm not sure what you mean by your last question. The test included in
> the
> > fed's DSGE.jl package just pulls out those matrices as they were computed
> > in matlab using Chris' original gensys code and compares them to the
> Julia
> > output. Is that what you were wondering?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/matthieugomez/HJBFiniteDifference.jl/commit/99a46282a8387788791fed17a998fd2c4f687e25#commitcomment-16246940
> >
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/matthieugomez/HJBFiniteDifference.jl/commit/99a46282a8387788791fed17a998fd2c4f687e25#commitcomment-16247385>
> .
>


PTAL now :)


For now? Yes.


I like this statement. Can this be moved up?


I got it. May I change it now, then?


End time must be 23:59:59 when the data filter is used or data result gets empty for end day in the time frame. (This should be save as long as it is never be tried to filter by time)

Example:

Filtering data from 10/01/2016 to 10/02/2016 does not contain purchases from 10/02/2016

![Alt text](https://monosnap.com/file/uEHbN0wwZemFsReSSwZKWOIZkQc4X6.png)