1: D400 First line should end with a period (not 's')
29:2: E114 indentation is not a multiple of four (comment)
29:2: E116 unexpected indentation (comment)
30:2: E114 indentation is not a multiple of four (comment)
30:2: E116 unexpected indentation (comment)
31:2: E114 indentation is not a multiple of four (comment)
31:2: E116 unexpected indentation (comment)
32:2: E114 indentation is not a multiple of four (comment)
32:2: E116 unexpected indentation (comment)
33:2: E114 indentation is not a multiple of four (comment)
33:2: E116 unexpected indentation (comment)
34:2: E114 indentation is not a multiple of four (comment)
34:2: E116 unexpected indentation (comment)
35:2: E114 indentation is not a multiple of four (comment)
35:2: E116 unexpected indentation (comment)
36:2: E114 indentation is not a multiple of four (comment)
36:2: E116 unexpected indentation (comment)
37:2: E114 indentation is not a multiple of four (comment)
37:2: E116 unexpected indentation (comment)
38:2: E114 indentation is not a multiple of four (comment)
38:2: E116 unexpected indentation (comment)
39:2: E114 indentation is not a multiple of four (comment)
39:2: E116 unexpected indentation (comment)
40:2: E114 indentation is not a multiple of four (comment)
40:2: E116 unexpected indentation (comment)
41:2: E114 indentation is not a multiple of four (comment)
41:2: E116 unexpected indentation (comment)
42:2: E114 indentation is not a multiple of four (comment)
42:2: E116 unexpected indentation (comment)
43:2: E114 indentation is not a multiple of four (comment)
43:2: E116 unexpected indentation (comment)
44:2: E114 indentation is not a multiple of four (comment)
44:2: E116 unexpected indentation (comment)
45:2: E114 indentation is not a multiple of four (comment)
45:2: E116 unexpected indentation (comment)
48:1: I100 Imports statements are in the wrong order. import time should be before from wsgiref.headers
49:1: I100 Imports statements are in the wrong order. import re should be before import time
50:1: I100 Imports statements are in the wrong order. import os should be before import re
51:1: I100 Imports statements are in the wrong order. import cgi should be before import os
52:1: I100 Imports statements are in the wrong order. import cStringIO should be before import cgi
54:1: I101 Imported names are in the wrong order. Should be BuiltinFunctionType, ClassType, ListType, MethodType, TypeType
55:1: I201 Missing newline before sections or imports.
56:1: I101 Imported names are in the wrong order. Should be HTTP_BAD_REQUEST, HTTP_LENGTH_REQUIRED, InvenioWebInterfaceWSGIContentLenghtError, InvenioWebInterfaceWSGIContentMD5Error, InvenioWebInterfaceWSGIContentTypeError, SERVER_RETURN
64:1: E302 expected 2 blank lines, found 1
64: D101 Missing docstring in public class
67:5: E301 expected 1 blank line, found 0
67: D102 Missing docstring in public method
76:1: E731 do not assign a lambda expression, use a def
81: D101 Missing docstring in public class
83: D102 Missing docstring in public method
91:23: E202 whitespace before ')'
104:28: E721 do not compare types, use 'isinstance()'
110:37: W602 deprecated form of raising exception
128:1: E302 expected 2 blank lines, found 1
128: D203 1 blank line required before class docstring (found 0)
128: D400 First line should end with a period (not 't')
128: D205 1 blank line required between summary line and description (found 0)
141: D202 No blank lines allowed after function docstring (found 1)
141: D400 First line should end with a period (not 'n')
141: D205 1 blank line required between summary line and description (found 0)
153: D401 First line should be in imperative mood ('Thi', not 'This')
153: D201 No blank lines allowed before function docstring (found 1)
153: D400 First line should end with a period (not 'e')
153: D205 1 blank line required between summary line and description (found 0)
169:5: E303 too many blank lines (2)
169: D401 First line should be in imperative mood ('Provide', not 'Provides')
169: D201 No blank lines allowed before function docstring (found 1)
169: D202 No blank lines allowed after function docstring (found 1)
169: D400 First line should end with a period (not 'r')
169: D205 1 blank line required between summary line and description (found 0)
196: D102 Missing docstring in public method
198:33: E127 continuation line over-indented for visual indent
218:1: E302 expected 2 blank lines, found 1
236:1: E302 expected 2 blank lines, found 0
236: D401 First line should be in imperative mood ('Set', not 'Sets')
236: D400 First line should end with a period (not 'e')
236: D205 1 blank line required between summary line and description (found 0)
251: D400 First line should end with a period (not 'n')
251: D205 1 blank line required between summary line and description (found 0)
267: D103 Missing docstring in public function
293:1: E302 expected 2 blank lines, found 1
293: D101 Missing docstring in public class
294: D102 Missing docstring in public method
314: D102 Missing docstring in public method
315:48: E251 unexpected spaces around keyword / parameter equals
315:50: E251 unexpected spaces around keyword / parameter equals
329:35: W604 backticks are deprecated, use 'repr()'
331: D102 Missing docstring in public method
333:33: W602 deprecated form of raising exception
342: D102 Missing docstring in public method
345:1: E302 expected 2 blank lines, found 1
345: D210 No whitespaces allowed surrounding docstring text
345: D203 1 blank line required before class docstring (found 0)
345: D204 1 blank line required after class docstring (found 0)
345: D400 First line should end with a period (not 'h')
345: D205 1 blank line required between summary line and description (found 0)
358: D300 Use """triple double quotes""" (found '''-quotes)
364: D102 Missing docstring in public method
367: D102 Missing docstring in public method
369:33: W602 deprecated form of raising exception
375:35: W604 backticks are deprecated, use 'repr()'
377:1: E302 expected 2 blank lines, found 1
377: D101 Missing docstring in public class
379: D102 Missing docstring in public method
383: D102 Missing docstring in public method
393: D102 Missing docstring in public method
397: D102 Missing docstring in public method
401: D102 Missing docstring in public method
405: D102 Missing docstring in public method
409: D102 Missing docstring in public method
413: D102 Missing docstring in public method
417: D102 Missing docstring in public method
421: D102 Missing docstring in public method
425: D102 Missing docstring in public method
429: D102 Missing docstring in public method
433: D102 Missing docstring in public method
438: D101 Missing docstring in public class
440: D102 Missing docstring in public method
462:32: W602 deprecated form of raising exception
491:32: W602 deprecated form of raising exception
497:52: E261 at least two spaces before inline comment
498:13: E266 too many leading '#' for block comment
518:36: E231 missing whitespace after ','
523:52: E201 whitespace after '['
605: D400 First line should end with a period (not 'r')
611: D102 Missing docstring in public method
621: D102 Missing docstring in public method
645:54: E701 multiple statements on one line (colon)
656:54: E701 multiple statements on one line (colon)
667:54: E701 multiple statements on one line (colon)
671: D102 Missing docstring in public method
682: D102 Missing docstring in public method
688: D402 First line should not be the function's "signature"
692: D102 Missing docstring in public method
695: D102 Missing docstring in public method
698: D402 First line should not be the function's "signature"
708: D210 No whitespaces allowed surrounding docstring text
708: D400 First line should end with a period (not 'd')
715: D210 No whitespaces allowed surrounding docstring text
715: D400 First line should end with a period (not 's')
722: D402 First line should not be the function's "signature"
722: D209 Multi-line docstring closing quotes should be on a separate line
722: D400 First line should end with a period (not 'e')
722: D205 1 blank line required between summary line and description (found 0)
727: D102 Missing docstring in public method
733: D102 Missing docstring in public method
737: D202 No blank lines allowed after function docstring (found 1)
758:1: E302 expected 2 blank lines, found 1
758: D202 No blank lines allowed after function docstring (found 1)
758: D400 First line should end with a period (not 'e')
758: D205 1 blank line required between summary line and description (found 0)
779:35: E231 missing whitespace after ','
807:5: E129 visually indented line with same indent as next logical line
822:1: E302 expected 2 blank lines, found 0
822: D205 1 blank line required between summary line and description (found 0)
830:5: E266 too many leading '#' for block comment
839:5: E266 too many leading '#' for block comment
843:5: E266 too many leading '#' for block comment
848:5: E266 too many leading '#' for block comment
853:5: E266 too many leading '#' for block comment
856:5: E266 too many leading '#' for block comment
860:13: E266 too many leading '#' for block comment
861:13: E266 too many leading '#' for block comment
867:13: E266 too many leading '#' for block comment
869:9: E266 too many leading '#' for block comment
873:13: E266 too many leading '#' for block comment
876:9: E266 too many leading '#' for block comment
881:5: E266 too many leading '#' for block comment


24:1: I201 Missing newline before sections or imports.
80: D102 Missing docstring in public method
85: D102 Missing docstring in public method
90: D102 Missing docstring in public method
95: D102 Missing docstring in public method
100: D102 Missing docstring in public method
103: D102 Missing docstring in public method
146: D102 Missing docstring in public method
152:13: B901 blind except: statement
156: D102 Missing docstring in public method
162:13: B901 blind except: statement
207: D102 Missing docstring in public method
259: D102 Missing docstring in public method
275: D102 Missing docstring in public method
281:9: B901 blind except: statement
285: D102 Missing docstring in public method
287:17: W503 line break before binary operator
289: D102 Missing docstring in public method
292: D102 Missing docstring in public method
295: D102 Missing docstring in public method
298: D102 Missing docstring in public method


What is this method for?


Netjes opgelost!


> We should be carefull not to include both "src/inc" and "src/GLideNHQ/inc/" in gliden64 and glideNHQ makefiles in order to avoid conflicts between library versions.

Perfect ;-)


Should be Cordova 4.0.0, referring to the Android version of Cordova


Oui en effet, je l'avais remarqué et totalement oublié de le fixer, n'hésite pas à ouvrir un ticket pour une meilleur visibilité la prochaine fois Doc, et merci pour le signalement ! :)


> We should be carefull not to include both "src/inc" and "src/GLideNHQ/inc/" in gliden64 and glideNHQ makefiles in order to avoid conflicts between library versions.

And perfect again ;-)


Ok :)


Yes, you have to compile boilerpipe yourself. The jar files offered for download do not include the ImageExtractor class.


This test does not need to be INTEGRATION, does it?


Yes, exactly, `int` is converted to `long` automatically, and it's worth having both specializations only when the int is significantly better (that's probably never is you have to convert the int/long to a (java) BigDecimal).


+Norilsk Industrial Institute


@zycbobby  醉了


Done. https://github.com/brawer/fonttools/commit/1e7af519bc65d9d341551ac7f2feedde6f072fd3


That's the tricky one :smile:
Because that calls divMod(long,long) so there is no advantage to have an int version.


@kwwall

I agree with you, in fact this change, which I introduced in this commit was reverted to AccessControlException in the last commit of this pull request. I cannot see the contents of whole pull request now, but I am almost sure that I did this that way in the end.


Copy that (cf. message on Bloc).


gg


Double check views-nya.. Ini hanya namanya saja yg hilang, viewsnya terganti apa tidak?


No list was never able to operate on an array, but it used dstore which has a function getIdentity(). The previous function in List.js was just calling the function of dstore to get the id of an item.

I am not sure the loop you are talking about. For me this function getIdentity is only called in List.js, I just moved the function to concentrate the modifications on delite/Store.js.
The function called by StoreMap itemToRenderItem() is the function getIdentity of dstore, but you are right I can have a better factorization of the code by using this one.

But the function itemToRenderItem() is only called at the initialization of the Store (in processCollection()) so one time by item. And then just in _itemAdded and _itemUpdated on the element chosen when there is an item added or updated.

Can you clarify me where do you think there will have a problem of performances please ?


Changed, but these names don't actually follow a strict scheme.


Did it work? :)


__API Blueprint syntax error on line 184__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Not 100% correct like that. The Themes provide the possibility to add custom Snippets inside Theme folder, but it is no requirement. The section is about adding theme-specific snippets and translations, in order to be able to install snippet-translations when installing a theme.


I wrote it while i was figuring out the easiest way to solve this ticket/problem, i removed it afterwards here: https://github.com/openmrs/openmrs-module-chartsearch/commit/d2f79bbe400966776f55f9618cee64edf5f2a357


Copy that.


ok

On Mon, Jun 1, 2015 at 12:24 PM, k-joseph <notifications@github.com> wrote:

> I wrote it while i was figuring out the easiest way to solve this
> ticket/problem, i removed it afterwards here: openmrs@d2f79bb
> <https://github.com/openmrs/openmrs-module-chartsearch/commit/d2f79bbe400966776f55f9618cee64edf5f2a357>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/k-joseph/openmrs-module-chartsearch/commit/59c15da5f8765fb7579a25f53837e573b0dbdce5#commitcomment-11450271>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


> 1. CSources is for building 'git devel', it's not really versionized, we frequently erase the history for it to keep it small. So referencing some old commit to the CSources is a horrible idea.

Okay, now I understand this better. But whose space you want to save? GitHubs ;) ? `csources` clone is shallow by default (`bootstrap.sh`) and since Git 1.9 you can push from shallow clone, then I see no point of doing that. See:

  https://blog.oio.de/2014/02/06/better-support-for-shallow-clones-in-git-1-9/

> 2. The proper releases ship with the C code that can be used to build them.

Okay, so then nobody is supposed to build out of the Git because of these `csourses`. Then proper warnings should be given, for anyone trying :)


hello


test commit.


This is just guessing :-\

On Mon, Jun 1, 2015 at 4:13 AM, Jeroen Ooms <notifications@github.com>
wrote:

> Do you guys have a sparc machine to test, or is this just guessing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RcppCore/RcppParallel/commit/f3133954aaf10778a04f0848a30a763ed0c366c9#commitcomment-11449361>
> .
>


Right, fixed.

There actually is some support for GATT in version 18. I thought about supporting it. But it's not non-trivial and not an actual requirement AFAIK. So I left it for now.


test comment 2.


Agree. Maybe after the branches are merged?


I'm curious about this one - it seems like before we were doing a terrible job of reporting errors, and now we're not even trying to do it at all. What's the upside?


:smiley:


I  tried to compile the code using mvn by myself ( first using ).but when I using the method getImages(), nothing return.  Can you provided your compiled binary file to me for testing ?


Contains pages of origin book: 320-321, 671-682, 1130-1133, 1227-1262.


@kwwall

Yes, the exception was changed from IOException to AccessControlException, but the code was not compiling and one of the tests (which tests exactly this case) was not adjusted to the change.

The code was not compiling, because there is no one-arg constructor in AccessControlException, and the object was inited using one-arg constructor (because IOException can be inited that way). I made that changes at the beginning without even looking at this particular issue.

And as the issue was in open state and because of the problems I found, I took it and made fixes I thought are proper in this case.


Is it possible to only use android version if-else on this line, as similar as my comments in client_connect?


This behavior was in the previous code. I just reproduced it, without delving into its features. I thought it was important to emit modelRefreshed() signal. But now I see that nobody wants it. So I fixed it.


If you used `Set` instead of `Array` in here, I think you could avoid the `if array.include?()` calls and having to sort the arrays for comparison.


Ah, OK thanks. I didn't see you added something to that branch. I will then correct the test for y diaeresis and merge all our UEB stuff.


off = read_pdu_at(cmd, 0, "ii", &server_if, &service_handle);


nit: permissions


And makes 0b00 a quine at -w 4 -l 3, in both directions!


By doing this, you make someone unable to use this lib with webpack :/
Would you be interested to revert to a simpler require("..../package") ?


@julie-sullivan Brilliant! Thanks!


The specification is not always correct. In such cases, the BlueZ source code takes precedence.


Fixed.


@joeyespo This is the most immediate way to solve it, of course, but also quite redundant since there's a `PatternMatchingEventHandler` supplied with watchdog that does the same thing.

What's important though is that it doesn't stop watchdog from crawling into those subfolders, it's just that you won't get an event triggered. A folder with a virtual environment(s) like `.tox` commonly placed in project's root will effectively render watchdog observer dead due to thousands of files and deep folder structure.

If you can suggest a better way of doing the same thing I'd be sure glad to hear it :)


Unlike the case channel_shared==false, the gradient is accumulated to a temporary scalar variable dsum for  a computational batch. To accumulate a scalar gradient to `slope_diff`, I use `caffe_gpu_add_scalar` here.


TeamCity YaccConstructor :: YC.Core General Build [Build 413](http://teamcity.codebetter.com/viewLog.html?buildId=192489&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Muligens. Men: mer data hjelper ikke hvis vi ikke har noen gode regler (features). Første, to siste og vokal-som-siste er neppe godt nok. Finnes det egentlig noen gode tommelfingerregler for feminine/maskuline navn på norsk? Kanskje jeg bør kontakte språkteigen, eller språkrådet, eller.no


Good point. I removed the cast.


@Araq Just tested again on my Windows VM using msysGit `1.9.5-preview20141217`. It does not show shell scripts with executable flag as modified even when `core.fileMode` is `true`, when such script has UNIX Shebang and `bootstrap.sh` has one. As on msys Shebang presence is magically considered as executable flag.


@hli2020 Unlike unshared case, the gradient is accumulated to a scalar temporary variable `dsum` for a computational batch. To accumulate a gradient to `slope_diff`, I use `caffe_gpu_add_scalar` here.


So it can display obviously.


Its a widget


Which is GUI.


This will call `self.get_flags_to_save()` for each iteration of the comprehension, probably better to pull it out into a variable before the comprehension.


This commit fixes issue #489


For the callbacks, I used the same names as in the Bluedroid structure. So I'd rather prefer to leave this name, even though it doesn't sound well.


I'm sure I did have, because it's not the sort of change I'd just make on a whim. But it's nearly three years ago, so I honestly cannot remember what the reasoning was.


also could use set comprehension


doesn't match struct hal_cmd_gatt_server_send_response here.


Oh, right!


s/transport/transid


Ok, I find a 1.2.2 version in my system package


damam garm


I think I put my comment in the wrong place.  My concern was that renderItemToItem() is called once per item, and then renderItemToItem() calls _getItemById().  Thus _getItemById() is called for each item, and every time it's called, it scans the whole array.   Maybe I misunderstood something?


 I think we should mention the shuffle changes. Also maybe DTS Music Disc
support but I'm not sure. The rest is important for those affected but I
guess it falls into the "Various fixes and improvements".


Seems to be fine, I didn't expect that but, well yeah that might be an issue in the parser.


Io direi il 3 giugno. Massimo massimo se qualcuno deve fare ancora qualcosa
tiriamo fino al 4 ma non oltre.

Io per il mio ruolo potrei anche già fare la consegna, però finchè fate i
vostri ruoli nel frattempo continuo a dare un occhiata per vedere se cè del
codice da migliorare/commentare/fare refactoring

Il giorno 1 giugno 2015 10:49, ghia16 <notifications@github.com> ha scritto:

> Ragazzi ma allora quando la mandiamo la mail con la consegna? oggi io
> controllerei gli stili e li sistemerei ancora e poi devo verificare se
> servono dei pattern da qualche parte
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igorravasi/Memo/commit/b1b8b50618152521f6a12d76bb1597f37d0ecbf2#commitcomment-11449791>
> .
>


Just style but could use dict comprehension


Fixed to match the source code.


Hi Thomas,

To answer your points
  * Yes, my code also did rely on community sorting that is what the second patch fixed.
  * I will add sorting for RouteTargets.

I will check Attribute.__eq__ I may indeed made a mistake there.


@shangguokan I would definitely keep `composer.lock`. As @bocharsky-bw already said it is useful to have the same work environment. Although you only _need_ composer when contributing, I am fine with the wording here: _should_ as in "you should look at and run the tests to see what happens in the example". I like this PR.


-n attribute will avoid comparing empty files


We'll just need to find a way to clear the `_buffer`. Just setting it to `null` doesn't do anything. We need to explicitly tell the garbage collector to free the byte array now. We can't wait till the GC does it's job sometime.


ok igor, io oggi finisco gli stili e lo slider, mi puoi dire per i pattern come siamo messi nel server? ci sono quelli più adatti? grazie


uint8_t addr_type; here based on https://git.kernel.org/cgit/bluetooth/bluez.git/tree/android/hal-msg.h#n1083.


Let's increase accountability through robust task delegation and reporting
Kind Regards
Tshepo Ngaleka
Mobile :+27795983130
Fax :0865161997

-----Original Message-----
From: MondeLionel <notifications@github.com>
Date: Mon, 01 Jun 2015 01:38:08
To: MondeLionel/Mobby-shop<Mobby-shop@noreply.github.com>
Reply-To: MondeLionel/Mobby-shop <reply@reply.github.com>
Cc: ngalekat<ngalekat@gmail.com>
Subject: Re: [Mobby-shop] Proximity (67b4ce0)

haha, At this rate the deadline is not going to be met

---
Reply to this email directly or view it on GitHub:
https://github.com/MondeLionel/Mobby-shop/commit/67b4ce04be2a45f59e892240b054393c0006d535#commitcomment-11449652


test merge


TeamCity YaccConstructor :: YC.Core General Build [Build 413](http://teamcity.codebetter.com/viewLog.html?buildId=192489&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 114 Build time: 00:12:53


TeamCity YaccConstructor :: YC.Core General Build [Build 414](http://teamcity.codebetter.com/viewLog.html?buildId=192490&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


@bparadie That probably because you're looking at the just released 2.0. I updated the lib to use that version now and the face has no smile :-) I looked into using their TTF but for now that means I need to write a lot of custom code so I'll just stick to generating it by myself from SVG for now. It shouldn't be any difference.


@Istador
@JenniferSpry
danke


I didn't check but is it really necessary to include cpufeatures?


Ci


Thanks, and good call - I was wondering about how that evaluation would resolve.


Hi @liammooney thanks for brining that up. We will have to investigate what our options are. Stay tuned for feedback.


this is definitely going to conflict with my PR, can you maybe have a look and merge if you're happy with it? this would just go in the `outgoing.scala` file.


Il server è ok. nel modello invece (element, sequence) forse si può mettere
un flyweight con la flyweight factory. avevo intenzione di guardare oggi
pomeriggio, no problem
Il 01/Giu/2015 11:48, "ghia16" <notifications@github.com> ha scritto:

> ok igor, io oggi finisco gli stili e lo slider, mi puoi dire per i pattern
> come siamo messi nel server? ci sono quelli più adatti? grazie
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igorravasi/Memo/commit/b1b8b50618152521f6a12d76bb1597f37d0ecbf2#commitcomment-11450600>
> .
>


Attribute itself is more an interface / inheritance class and the class which handle Unknown Attribute are GenericAttribute - so all good there - thank you for checking :-)


About peerDependencies, giving you warnings about incompatible versions is the whole point with it. You can still install and use it, but you'll be warned since they are not known to be compatible. I'll look into widening the version range, but dropping it entirely seems wrong to me.


The signal seems to be used in `transferlistfilterswidget.cpp` line 110


@jiru thank you for reviewing this commit ^^, I'v addressed your comments in https://github.com/Tatoeba/tatoeba2/commit/0b74ea370eb1be80f03990eeaa5d9e787be8fa67


in my PR, there is a corresponding JSON conversion spec that would benefit from this test.


You're right in that the way I did it is kind of ugly, but I'm not sure your way is The Right Way either.  Maybe I'll look at using a list comprehension.


This mod is needed?

This commit change to /usr/sbin. I'm pretty sure it's because libvirtd expect it to be in /usr/sbin but i'm not sure. 3f2a58551b57884ebaabb00cec4995b92a50a491


While I get similar paths, please try the patch I mentioned above. If that fixes the problem that's a far better solution to the problem.


Based on the structure defined in hal-msg.h,
{
        ...
        int32_t data_len;
	int32_t mask_len;
	uint8_t data_mask[0]; /* common buffer for data and mask */
};
it doesn't match with the implementation here.


is there lack of intendation maybe?


Looking at the code, RouteTarget inherit from ExtendedCommunity which does implement comparison .. So it should work ( not tested tho ).


After merging with master it is sufficient to do the following to store data in the DB:
```
            GBApplication.getActivityDatabaseHandler().addGBActivitySample(
                    (int) (timestamp.getTimeInMillis() / 1000),
                    GBActivitySample.PROVIDER_MIBAND,
                    intensity,
                    steps,
                    category);
```

I did not push the code because I'd have to create another branch with a merge from master and that could create confusion.


I assume there will be real tests?


Благодарю за ответ. Вот сделал пока в своём репозитории тест
https://github.com/snnwolf/prototype/commit/8e1cc4f155ed6b5e1ae8455b2924754286fecf9d
или тут https://github.com/snnwolf/prototype/blob/8e1cc4f155ed6b5e1ae8455b2924754286fecf9d/test/autocomplete.html
К сожалению не знаю как правильнее сделать всё это. Ошибка вылезает, если сделать окошко браузера с вертикальным скролом и чуть прокрутить вниз.
Если поможете решить проблему нормальным способом буду очень признателен. Мне самому тот костыль не нравиться...


kill me pls


These "new" array operations should be packaged into a separated project.


This produced a lot of "Trying invalid coordinate..." blah blah blah, so kill me pls again


If we are going to support interface name and create_ap pid, then changing "--list-clients <id>" to "--list-clients <ifname>" won't make sense. So, we'll support create_ap pid and keep the "--list-clients <id>" convention.


Are those tests from `ember-cli` generators? Please either fill those up, or do not commit at all.


Hey, thanks for this PR, really appreciate it (and for the nod to error handling!).

You're making zlib a dependency for this module but I don't see that dependency reflected in package.json? Can that be added, please, then I will review and accept the PR when I get back to my PC.


Good remark.

Last night I changed the MInorBehindPawn for a MInorBehindPawn [2][RANK_NB]
according to BISHOP or KNIGHT, and rank from 2 to 6 (the current master stops at rank_4)
I used the TUNE "magic code", and tested only the mid game value and left the end game value to 0. After 2000 games, the values had almost not moved ! Just realised that my ck was way too low (1.25) and I forgot to change it for that array.

At the same time, I started with the initial PSQT values (from master) and came up with the values used in the following test
http://tests.stockfishchess.org/tests/view/556c65b80ebc5940ca5d6f1b which was not good at all.

Will retry locally tonight starting with the +22 PSQT values suggested so far, using the latest master, and retune the MInorBehindPawn as well to see if any pattern comes up.


Minha idéia inicial era não trabalhar com requests e responses mas sim com objetos e operações. As apis rest são assim. Recursos e operações sobre os recursos. Recurso é o token e as operações são: Criar, Cancelar, Enviar Pagamento a partir do Token. Sacou?


Consider `runningThisThread`. Rationale: The names of member functions don't need to repeat the name of the class they belong to.


Why create a new type?


I branched off master which is 3.2.21 instead of the 3.2.16 - I need to fix that...


The `[WIP]` prefix on the git commit summary should probably be removed from this commit.


I know. But the `[WIP]` still aplies, I want to cleanup the message more.


> Need to reduce this commit message to something decent still.


So if 20 files are opened and the time is called on the last one opened, what would happen?


Not sure if you meant to, but the mushrooms you named them for the red mushrooms instead of for the brown. XD


Full stop.


This was evidently a regression caused by the change I had made to support conditional reading. I failed to test that aspect and presumed *read-eval* was still off :(


In fact, I was suspicious about the bonus on RANK_2 for the KNIGHT, but your idea that removing the RANK_1 might let the S(16, 0) converge to a more suitable value for the whole thing is a very good insight.


This comment is confusing. I see this generator returning a 0, or buf.


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (644 of 644 tests passed)

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


Would be clearer to explicitly compare the different components of the value with
`if (wIndex & 0xF0 == REQ_PWR_PORT_A_IO && wIndex & 0x0F < 8) {


Very nice! I didn't know about sync.Once, learn something new every day!

On Mon, 1 Jun 2015 9:55 am Manu Mtz.-Almeida <notifications@github.com>
wrote:

> @joeybloggs <https://github.com/joeybloggs> btw, I just did the Gin
> validation optional and plugable:
> fecde9f
> <https://github.com/gin-gonic/gin/commit/fecde9fed68ad71461e90c5477172d5c7fe6fbf4>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gin-gonic/gin/commit/784d73477d056308e8c6ea666d52048906e48a7c#commitcomment-11454494>
> .
>


I doubt if kernel unable to freeze/unfreeze process without that - kernel/power/process.c can manage this github.com/ChronoMonochrome/Chrono_Kernel/blob/stable/kernel/power/suspend.c#L116


These could all be replaced with `pin_set(PORT_A.gpio[wIndex & 0x7], wValue)`


Hey, I just saw this, feel free to open a pull request and I'll merge it in to my repo (if you want to) :smile:


haha, what is this?


Allora per il resto tutto a posto, per il colore delle scritte ne cercherò un altro, era per non lasciare una stringa nera sbattuta li e boh, domani mi invento qualcosa. Sui pulsanti mi sarebbe piaciuto trovare qualcosa di più ma a sto punto tanto vale stare sul classico, ho creato una classe nuova e rimosso quelle vecchie.
un appunto igor, nel dropdown del login a me non mostra i messaggi, a te funziona tutto?
Riusciamo inoltre a buttare dentro un autorefresh per aggiornare la pagina e far comparire subito il nome utente dopo il login?
Il multy come siamo? Dici che riusciamo a farlo per la consegna finale?


> ```ini
> insert_final_newline = false
> ```

Why?

> ```ini
> trim_trailing_whitespace = false
> ```

This actually makes sense, but then I wonder if we are going to use <br>s anyway.


please 'fixup' this into the previous commit


What  is this for? the include path is wrong dont know why that works for you, it should be
```
#include <glm/gtx/transform.hpp>
```
but is this needed for something?


Is this still necessary now that you found that was VPCS fault? As Bernhard said, now it take longer for a bunch of VMs to start.


Patches added:
Dead or Alive 2 - Hardcore [SCES 50003] (E)
Dead or Alive 2 - Hardcore [SLUS 20071] (U)
Incredibles, The[SLES 52812] (E)
Incredibles, The[SLES 52816] (E)
Incredibles, The - Rise of the Underminer [SLUS 21217] (U)
Neo Contra [SLUS 20961] (U)
Neo Contra [SLES 52510] (E)
NHL 2K8 [SLUS 21632] (U)
Spongebob Squarepants - The Movie [SLES 52985] (E)
Spongebob Squarepants - The Movie [SLES 52986] (E)
Spongebob Squarepants - The Movie [SLES 52895] (E)
World Rally Championship 3 [SCES 51684] (E)


This is a cute shortcut but is doomed to break as soon as we have  PV that is not also a V.  Is doing it "for real" too complicated?


NM, I am stupid


Just had a look at 47994115f1324e88c66806e555252c1e4c173b2e to see what else has to be done on a new release
- CMakeLists.txt: set DYNAMIPS_VERSION_SUB to an empty string
- common/dynamips.c: set sw_version_tag to release date


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2120 of 2202 tests passed)

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


I needed this for one of the rotate functions, for some reason the compiler couldn't find it. I'll try w/ that include


Yeah, for the first time a feature being called as unfinished in an emulator makes no sense to me after that comment. A real "unfinished" feature is something that doesn't function and is commented out with "/*"


develop (lefticus) - x86_64-MacOS-10.9-clang: OK (351 of 351 tests passed)

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


>  I take it, you are using non-msys python?

There are two MSYS pythons, and there's also a non-MSYS python (e.g. download directly). This is attempting to detect the one broken MSYS python, but all others should work.

>  What about cmake? msys or non-msys?

Both should work just fine

> I would encourage you to document the environment required to build msvc target. :-)

I'd prefer to actually just get it to work everywhere, as if it's not working it's definitely a bug! After some more testing it appears the problem here is a little more subtle than I originally thought:

* Of the three pythons, `/usr/bin/python`, `/mingw64/bin/python`, `C:/PythonXX/python`, all work.
* If `sh` runs `/usr/bin/python`, it *does not work*

The problem is that when `sh` runs `/usr/bin/python` with an argument of a unix path, it *does not* auto-translate the argument to Windows path, unlike all other configurations. As to why, I have no idea!

Can you try modifying this check to see if this works for you?

```sh
if /bin/sh -c "$CFG_PYTHON -c 'import sys; print sys.argv[1]' /foo"  | grep '^/'
```


This file is objective-c specific. Why is it in the top level directory instead of in the objective directory?


Raymond, thanks for porting my fix.


To me the units of electrons per pixel for readnoise and darkcurrent is a bit counter intuitive since it is measured in ADU and, at least in the case of darkcurrent, corrected for in units of ADU usually.

That being said, consistency is by far the most important thing here.


cool but then put this test in the apis_routes_spec file, or change this one


LGTM


Where this comes from? if I use it in my config it says undefined variable.


hpwh_residential (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1667 of 1674 tests passed)

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


Is it OK to not verify this for correctness? I'm unsure how and where it can be set (as this will be a public method)


Interesting, I am not very familiar with the EC.  Feel free to revert.


std::make_shared here instead, for consistency?


can we simply move line 57 and 58 to here?


is this relevant?


I see no issues here


It's not necessary and has no effect.


This seems fine


We would still need to make the other changes to the request before submitting it (normalizeRequest, removeSingleHopHeaders, etc). It's duplicate work.


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


ts.get_report_data(2014,2) 执行到300208 返回NaN中断，试了其他几个函数也有这个问题。

300208，恒顺众昇，只打出恒顺众三字，可能是codec问题？

Python 2.7，win7中文版， tushare031


Second readme?


This will close #211 for now.


is the tail being done separately?


I would consider flipping this logic around. Bail if it's != ContentType.handshake (which is just a raise), then put the logic below. This will probably grow with time, which will just push that bottom raise lower and lower (thus making it harder to notice to someone unfamiliar with the code).


> The problem is that when sh runs /usr/bin/python with an argument of a unix path, it does not auto-translate the argument to Windows path, unlike all other configurations. As to why, I have no idea!

I think the above [link](http://www.mingw.org/wiki/Posix_path_conversion) actually explains why: `/usr/bin/python` depends on `msys-2.0.dll`, and is therefore considered a native posix program, which gets the `/c/foo/bar` version.

Do you know, at what stage do the posix paths break cmake?   Can we use `cygpath -w <path>` to convert path explicitly?


We're exposing the `LedgerHolder`? I'm not sure I like this


Unfortunately, Cocoapods gives trouble if the podspec isn't in the root of the repository. From my experience, at least it won't pick up the license file correctly, and it won't let dependent projects use the :git notation to refer to the library. On gRPC I had decided to stop fighting the tool and give it what it wanted: the podspec in the root of the repo.


I had the gradient worked out wrong. This change should make the gradient work correctly.


rax retry tests


I understand what you mean about needing to make sure that the configuration for Vagrant doesn't break existing users. I think you're right that Varnish should be used in the VM and to extend that line of thinking further - I think it would make more sense for the Vagrant environment to be staging or production instead of development. At least - if the goal is to get a Vagrant local environment as close to production as possible. Going that direction would mean that this change (and a couple others) could be avoided but would likely require changes in a couple other places to get it working.


Please split this into two separate commits -- it's confusing to have a bunch of irrelevant formatting changes in a bug fix. The formatting changes can get pushed to master, the bug fix (which is just the addition of `.map(fs.makeQualified(_))` should be the only thing on this branch.


Makes sense - thanks Geoffrey!


How would you feel if after line 110 we defined:

max_procs = 4
min_procs = 2

And used those instead of the straight-integer.


Oi, por enquanto tá tudo :ok:
Tenho medo que, quando comecemos a trabalhar com ASP.NET MVC você já tenha problemas. Antes disso, quando precisar rodar um SQL Server. O que tu podes fazer é configurar um **MySQL** no Linux e tentar utilizar os drivers de conexão do ADO.NET para MySQL.


reason ?: [NSNull null]


We should be ready to go now: https://github.com/GNS3/dynamips/commit/9397fb50dc7989a5aa13347880b05f30959efb41

Thanks for catching these :+1:


As discussed, we need to implement a way to find the number of procs in the system.  If it is less than 4, then we should set the thread count whatever it is and no less than 2.


Alex, good job on your first PR.  We can continue tomorrow and wrap it up.


Subframe will also fire this.  When subframe fires, the frame id is uncertain but not 1. I am not sure how to confirm which frame is main frame which is sub frame, so temporarily removed it.


I need to do a set() and getMutable() on it. Would a couple of proxy functions from LedgerMaster be preferable?


In Mac I have to run the install with SUDO, but then I get the following errors. npm WARN cannot run in wd angular-seed@0.0.0 bower install (wd=/Users/robertlucarini/Documents/Github/fusion-seed-app)
bower ESUDO         Cannot be run with sudo
Please help!


They aren't VNc compile flags, they are standard in RunUO 2.3+ :)


Let's rename `_Connection`. If we ever need more connection implementations again (I hope not), we can specialize again then.


Feel free to make a branch and contribute changes.


This makes me very happy. Just one comment.


Solo yo puedo ser tan bobo como para hacer un commit grande y no correr el jUnit en Todas las clases, ahora lo soluciono


This looks good


See my previous comment about RecordSocket.send() (It's possible I'm being silly now that I think about it though)


clase 1/6/15


@vterron You are welcome! :)


There is a memleak with this, only seems to affect dx11 and OGL, DX9 is fine (although the half screen thing doesnt seem to work properly) and software mode is fine, but with the other two, memory usage slowly creeps up


I see no issues


`cp` vs `mv` -- seems to me you'd rather move, no?


Using Varnish in the Vagrant VM definitely makes sense - I'll see if I can get it working in my fork. I think I can get the ports to work better in the Vagrant environment if I move away from using the development rack environment.

Regarding the rerun commands written at the bottom - would that be something that gets added to the cdo-apps recipes? For example, check for the vagrant_dev environment variable and if so execute those rerun commands to restart the services? I think right now the cdo-apps recipes just install and start the services


jkhjkhjh


Viewed the source, checked the raw files and they appear to be the same.


Ah yep, that'd do it! Unfortunately if MSYS does not translate a Unix path to a Windows path for the python-in-use, it will end up leaking into LLVM's build system as a Unix path, and that path is then emitted in turn to a file which is later read by CMake. CMake then tries to read a file at a Unix path, which definitely doesn't exist, so it chokes.

The specific path taken is:

* [We call `cmake`](https://github.com/rust-lang/rust/blob/a49ae5bd4378cb47fd5cfdcd0c31e04d55373696/configure#L1387-L1389) from MSYS (right?)
* [LLVM calls `llvm-build`](https://github.com/rust-lang/llvm/blob/43fceb0bc06945abcdb11d6db299cc21757b7bdd/CMakeLists.txt#L406-L417) (through cmake)
  * [The path to `llvm-build`](https://github.com/rust-lang/llvm/blob/43fceb0bc06945abcdb11d6db299cc21757b7bdd/CMakeLists.txt#L391) contains `LLVM_MAIN_SRC_DIR`
  * [`LLVM_MAIN_SRC_DIR` is set to `CMAKE_CURRENT_SOURCE_DIR`](https://github.com/rust-lang/llvm/blob/43fceb0bc06945abcdb11d6db299cc21757b7bdd/CMakeLists.txt#L162)
* `llvm-build` attempts to [infer the LLVM source root from `__file__`](https://github.com/rust-lang/llvm/blob/43fceb0bc06945abcdb11d6db299cc21757b7bdd/utils/llvm-build/llvmbuild/main.py#L888-L894).
* This `source_root` is then joined on and used to emit lots more `CMakeLists.txt` files, all of which now contain Unix paths and will fail when run.

I'm not 100% sure, but it seems that the current directory of CMake is Unix-style due to being run from MSYS (maybe?) and this then leaks into python via `__file__`, and eventually leaks into CMake as well.

If there's a way to force windows style paths all the way, then that'd be great! Unfortunately last time I tried that I couldn't figure out a way to do so :\


I like that if I `ls tmp` I see the changelog, making me less likely to make another conflicting tmp file called "changelog.html". (Admittedly an unlikely scenario, but keeping around the extra copy is cheap, so why not?)


Unfortunately doesn't work. I will take a look at it tomorrow, maybe will find something useful :)


Oh,...what the hell


thanks


Might be best to leave it alone for now


The reason I changed this behavior is because in my tests of the Vagrantfile and chef setup the correct bundler version was not available by the point that the cdo-app recipes tried to call bundle install. By putting in the explicit `sudo gem install bundler` command it resolved the bundler version error.

Maybe this difference in behavior is happening because of a difference in the image used for production/staging/testing machines and the trusty64 Vagrant box?


ok，这个问题我知道了，晚上回去修复一下。


I uploaded this model because i wasn't sure at the time wether or not the tail and mane would be created with particles. If it's better for you to have the tail and mane as separate objects, just let me know and I'll change it.


sorry, 之前写的忘记改过来了！


This looks OK


Ah I see - I didn't know that schema:load is problematic in practice. My reason was primarily performance - I also thought it might help some issues I occasionally ran into with migration files in the staging branch and using Windows as a host machine (since I have Vagrant setup to use synced folders instead of a full, fresh checkout of the project repo). db:migrate seems like the right choice and having a full/real checkout happen inside the VM (to avoid Windows filesystem/path issues) seems like the it would be the right way to go.


Can you put a comment here documenting whether this is the number of image pixels, or the number of logical pixels based on some resolution data in the image, or some other value?


Full stop.


sorry for this


Aqui


This should be harmless


Looking so good! I can't wait to play!!!!


Nos commits de Arthur tudo parece estar ok para mim, estudamos pelas mesmas fontes e tiramos dúvidas um do outro


Ah shoot.


This is OK


I've noticed that the invoked method has the following parameters:

function _my_callback_method($field_value, $row_id, $field_info_object, $row_object){

}


Why <CanvasColor> and not <{implemented_as}>?


`Decorator`はdecorateするオブジェクトを保持するのが一般的です。
また、`decorate`は前か後に追加の処理をするのが普通で、インスタンス変数を書き換えたりはしません。
やり直してください。


不要在意我写的那个，^^。当初是写给同事做domain specific 的工作，后来需求没了，也没再继续更新。确实还是postman更好用些！！！


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1159 of 1160 tests passed)

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


Fair enough. If you're concerned about tempfile name collisions, though,
consider using mktemp.


Part one uses faculty input file to create list of faculty members


develop (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (351 of 351 tests passed)

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


Should this be a private variable?


Most of these start with `Libraw` except for `UnspecifiedLibrawException` and a few of them end with `Error` but some don't. We should pick a consistant name for them (looks good overall though).


Thanks for the explanation :)  Could you add these comments on top of the file?


Thanks!! It looks like we might need the same fix in `readbytes` in process.jl. (also `readall` should probably just call `readbytes`, but that's a totally separate issue)


@effron halp


Is assert the best way to do this? It won't really return a very nice error.


latest


mismerge. 9818f16 removes the read_current_timer function.


Fixed syntax error / typo in Player.cpp


what exactly is happening here?


What is this file?


update zh-CN


rax retry tests


hoppingforest? :-)


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2138 of 2213 tests passed)

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


Creio que {$("input", name: "_action_delete").click() } deveria ter sido feito em uma classe auxiliar e acho que há métodos melhores para se achar uma mensagem via JQuery


Why not just ``for expected, test in self.tests.items()``?


Such text


@scoheb , excellent.

Yes, reason is better than failure reason as this is generic plugin, failure reason is used for my case.
Yes, "display inline in build history" is much better than "visible"
Yes, it could be too long, this is one of the reason that I put "display inline in build history" as option. It is up to the user how to use it.

For 'job-level" claim report, it is out of my competence ;-)

rgs/larry


Because that's the style used in the rest of the class!


remove_std_function (lefticus) - x86_64-MacOS-10.9-clang: OK (351 of 351 tests passed)

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


"eh" = é?


Mesma coisa, acho que withConfirm(true) {$("input", name: "_action_delete").click() } != "Tem certeza?" deveria ter sido feito em uma classe auxiliar e não diretamente nos testes


Maybe ``path2magic`` should raise an exception if ``"//" in path``?


Creio que esteja tudo ok


This is way over my head. The code seems sane enough though.


Creio que esteja tudo ok por aqui também


You should already be able to pass `isArgument: true` and it will return the renderer function.


No, my thought was that users would access states by name or by object def, and the internal references would be transparent. The type itself is just a way to encapsulate the two different approaches. Anyway, as long as I can make your state events shim keep working, you don't mind if I blow it away?


I see, thanks for explaining!


creio que esteja tudo ok por aqui


.npmignore is not required if you have .gitignore, npm supports both, but chooses one of those afaik, giving preference to .npmignore.

Reference: https://docs.npmjs.com/misc/developers#keeping-files-out-of-your-package


erro de sintaxe, o certo seria "Scenario: xxxx"


My fault - and it was a dumb idea when I did it.


Cenário de controlador, muito específico


@pweil- reminder - sweep through these and change them all to admission.NewForbidden


<3

On Mon, Jun 1, 2015 at 7:31 PM, Yubo Tian <notifications@github.com> wrote:

> lol
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yubotian/CSA_app/commit/0425abc37f614f7f476d063e8257376f1ff3c24f#commitcomment-11465403>
> .
>


Hmmm... I guess my line of thinking is, if the transition is ignored (because same state, same params, same whatever), that's functionally equivalent to it being a no-op, right? And since the outcome is functionally identical to success (you end up at the state you just said to go to), seems like it shouldn't be a rejection...?

It' kind of an awkward one, but I think there's precedence for considering it successful (provided the above) in other programming constructs/patterns. Users still have `Transition.ignored()` if they need to check. We could even have an `onIgnored` event, but maybe that's excessive...


This looks OK


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@winston Ah thanks!!!


that scheme sounds really awesome. combined with some kind of dynamic microcode for encodnig and decoding it could be like a mutable form of oop, or just a really flexible prorotype-based programming paradigm.

growing lists can be eliminated from concepts because the 4 List<Task> in each concept are given a maximum size.  each Task can be bounded by a maximum size if term complexity is limited.  the termlink and tasklink bags are also maximum size limited.  thus the entire Concept structure seems like it is compatible with such a design and the memory required per concept can be calculated ahead of time.  this is what i mean by it's possible to tile a memory block with Concepts once their size is known.  even if they dont have equal sizes i think your scheme can support it.  for example we could begin with a mini-Concept size and if it exceeds that, replace it with the full size beyond which it can grow no larger.  this would be good for Concepts whose lifespan is more or less too temporary to accumulate the metadata that would fill up its max alotted space.


This seems to skip the shard


なんだこれは？


Esqueceu de colocar "->" nesse step


Agreed. I think I was trying to go for some kind of short-hand, access but, not worth it. Plus, then it easily maps to ES6 + types.


Nesse step tu passa o parâmetro e não usa em lugar algum, e além disso o ponto não é cadastrado no teste. O que era pra ocorrer.


For example I just got the following error in the latest staging code while running db:migrate:
undefined method `id' for nil:NilClass/home/vagrant/development/dashboard/db/migrate/20150423215834_create_channel_tokens.rb:13:in `up'


codigo repetido.


acredito que esteja tudo ok por aqui.


tudo ok aqui, talvez o scenario não tenha funcionado pelo erro de sintaxe detectado na classe .feature


Eu acho que deveria ser separado em dois steps, um para executar a ação de deletar e outro para recusar essa ação.


Looking at it now, I think I had the param order wrong. The intent of the `pick()` call was to lock the return value down so that it only contained keys present in whatever hash of defaults you pass it. I'll fix it up and add some tests.


Ok.


I actually pushed a change that superseded that whole thing.

In another branch, I need to be able to find out what Job I'm currently on... so I now keep a table of the current Job for the current thread id...


Mesmas observações descritas no scenario anterior.


não teria como fazer isso, por que não tinha html para a caixa de pop-up no teste, o comando teria que ser executado em só um step.


Yeah, `transition` should be the One True Parameter™ for all events.


O ponto já está cadastradado no sistema base.


``return re.sub(r'[/@]', lambda m: {'/': '@_', '@': '@@'}[m.group(0)], s)``

(see below for why ``@_``).


@OwnDroid  nooooooooooo


rttf


I think there's a few correct interpretations.  The biggest factor for me is that ignoring a transition will fail to fire any events (even onBefore event), where letting the transition execute as normal (no states entered/exited) allows all the applicable events to still trigger.  I implemented "ignored", but almost feel that letting a no-op transition run through each step would be more ideal, and less corner-casey (I added a hook in stateHandler.onFailure to account for an ignored transition, and update dynamic params if they changed)

The way I ended up implementing it is, the transition's promise is rejected with IGNORED, but the state promise is resolved with the current state.

If we switch around to letting a noop transition


remove_std_function (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (351 of 351 tests passed)

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


Я реально не могу понять откуда внезапно мог взяться бинарник на 90 мегабайт в середине репозитория.
Точнее есть одна идея =)
Походу один раз ./deploy.py а не python deploy.py умудрился проинтерпретировать import sys таким образом
О_о


зато у меня в статистике +1кк строк XD


I can't verify the logic, but the code seems sane.


Those 3 spaces need to be a tab instead (probably why it's failing).


no prob, but I exposed it elsewhere too.

Off the top of my head, I think it's just as the $transition$.to and $transition.from, because I thought it was a better organization (to.params(), to.state(), to.$state(), and likewise for from.*) instead of having trans.params().to and trans.params().from.


@Bouckaert @isabarros

essa inicialização do pretty photo tava dando um erro. Removi e não percebi nenhuma diferença na interface. Tudo bem remover, né?


Now ppapi is supported, this flag should be removed i think.


BTW: I reverted all the state stuff to the old 0.2.x arguments. Above comment no longer relevant


I made a choice and this has been refactored.   Ignore above comment if you're OK with my choices.


Considering that any unconstrained implementation of array swap will work properly with same-sized arrays of different-but-swappable types, I don't think we should constrain the library's implementation to only accept two identical array types.

How about:
```
// swappable_array: strip matching array extents, determine if the result is Swappable
template <class, class>
struct swappable_array : std::false_type {};

template <class T, class U>
    requires requires(T& t, U& u) { swap(t, u); swap(u, t); }
struct swappable_array<T, U> : std::true_type {};

template <class T, class U, std::size_t N>
struct swappable_array<T[N], U[N]> : swappable_array<T, U> {};

template <class T, class U, std::size_t N>
    requires swappable_array<T, U>::value
void swap(T (&t)[N], U (&u)[N])
    noexcept(noexcept(swap(t[0], u[0])));
```
Given `Swappable<A&, int&>` and (`Swappable<array<T, N>&, U(&)N>` iff `Swappable<T&, U&>`) this allows even silly cases like:
```
array<array<A, 4>, 3> a[1][2];
int i[1][2][3][4];
swap(a, i);
```
([This program compiles without error](https://gist.github.com/CaseyCarter/c82e4db76618e34fc253).)


Tem certeza que vai chamar o lambda com o environment `s`? E as novas variáveis e alterações que foram feitas depois do `make-closure`?


Perhaps we should throw a proper exception here


@matthewp Thanks for pointing that out. [This](https://github.com/bitovi/canjs/blob/fix-component-partials/view/scope/scope.js#L197) is what eventually calls read with `executeAnonymousFunctions: true` but we wouldn't want `isArgument: true` for each read. I don't see a good spot to add that option to the read when setting up the component.

On the other hand I don't think you would ever want the value of a renderer function so making `can.compute.read` not invoke those seems like a good fix.


@nateabele here's where I'm also exposing StateReference objs because I thought it was a useful abstraction for callers.


Não usa o parâmetro passado para nada.


Also note that we need to process (only) Unicode paths, so all the strings should be prefixed with ``u``.


I'd prefer to avoid the `enhance` method in new code and use the `_p._moveTo = function()...` pattern for this. The `enhance` method is only a little bit more convenient, but not as versatile.

I've already done rewrites for some prototypes that were built with enhance, because it was blocking me. Especially when it comes to adding an asynchronous API using obtainJS the `enhance` way is a PITA. Pens will need some asynchronicity for adding components, how to do this this is not yet solved, however.


Não deveria.


Mesmas observações descritas no scenario anterior.


@fnoble
Do we need to change libSBP for a new size of the tracking message?


Fixed some stuff in the blackheart fairings.


Thank you.


Perhaps "Keep thee out." instead?


29 seconds ago? That is so last year.


Nesse step tu passa o parâmetro e não usa em lugar algum, e além disso o ponto não é cadastrado no teste. O que era pra ocorrer.


`#pragma clang ...` doesn't seem to work:

```
/home/rosbuild/ci_scripts/workspace/build/test_communication/rosidl_typesupport_connext_cpp/test_communication/dds_connext/PrimitivesServiceResponse_TypeSupport.cpp:12:0: warning: ignoring #pragma clang diagnostic [-Wunknown-pragmas]
 #pragma clang diagnostic ignored "-Wdeprecated-register"
 ^
```

I could either wrap that pragma in `#ifdef __clang__` or I could start with a `#pragma GCC diagnostic ignored "-Wno-unknown-pragmas"`. Thoughts?


Esse `current` tá sendo usado onde?


What you want to do here is:
```js
var otPen = OpenTypePen(),
  , pen = new PointToSegmentPen(otPen)
  ;
```
And then go on :-)


This appears to be in the message


I think maybe we need to declare what is going on here better.
 The issue is that we want to start with an empty folder every time the script is run. So, we check for presence, and then if found we delete it. But the first time the script is run we create the folder because there is no expectation that there should be a folder. So the in-line comment needs to be updated.


(￣y▽￣)~*捂嘴偷笑

在 2015/6/2 9:41, Xingyu.Wang 写道:
> PUB:20150310 FAQ--BSD
>
> @wwy-hust 翻译的真不错！赞一个
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/LCTT/TranslateProject/commit/3ef2c75aaccdbb960782195af57b2a27c3ce84ce


第一次提交终于成功


1440-AWS (nllong) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


That makes me think the original implementation was wrong. Throwing breaks the promise interface, so even though strictly speaking it would be a BC break, unless it breaks actual functionality in some way, I say we roll this back and change the test. Thoughts?


latter


The code seems fine. I don't understand the logic though.


não teria como fazer isso, por que não tinha html para a caixa de pop-up no teste, o comando teria que ser executado em só um step.


o ponto está no sistema base.


This happens synchronously in transitionTo in 0.2.x.  I emulated it in 1.0 by having the compatibility events also happen synchronously (using onBefore)


Hmm maybe also I'll put READMEs in the dirs in case people click into them expecting all the python to be there~


Fixed - see the new Homework2 for an updated comment header! Have to remember to do this for everything from  now on. Thanks again!


im thinking of bringing in the WantsZeroCopy design. i think 'Identifier' is a good name for it and it will be responsibe for lazily generating a serializable representation (byte[], int[], string, whichever), equals, hashCode, compareTo and in addition to this, a callback to the host object for replacing its Identifier with the instance of another Identifier found to be equal (so that an instance == will prevent future comparisons between these objects, gradually deduplicating all identifiers in the system).  additiionally the Identifier would be responsible for pretty printing (PrintWriter, because it should primarily be used in streams) format so as to decouple the compact internal representation from any human-readable form, eliminating the " --> " symbol  representation redundancies i mentioned.


Hi! I would have commented here but it was quicker to slack you but then it got out-of-order so fail oh well~ I finally understood what it was for; makes sense but not something I had ever used; I'm OCD allergic to trailing whitespace; I tend to try to find paragraph'ed layouts.


Let's switch everything to LaTex bwahahaa


Thank you.


H phase still needs to be fixed


ping!


It would be better to place all links in a single rule, no? (see lines 47-51)


gfdsd


Oh you mean why the color is not indented properly? It is a mistake, sorry.


Agree.


I think you should use the button-gradient-hover mixin.


s/craeted/created


I wonder if this will have a knock on effect to other games. I recall Metal Gear Solid 2 used to have issues if you were in 1st person view at the starting boat mission and looked at certain things, the fps would drop quite a bit but you would see no obvious issues. Yet if you looked at other spots the fps would be fine.

Note that I last tested that game many years ago, so could be completely unrelated, but just mentioning since it made me think of it.


![Uploading 用例.png…]()


currently I am working on https://github.com/jruby/jruby/commit/4da7547f08c9168c3a5755441e8f2746d4c7afc6 where I basically started with this commit to backport it. BUT ```URLResource.create(loader, script)``` will not find any resource on the classloader since the script still has the prefix "uri:classloader:"

so I am confused whether I miss something or this commit misses something ?

need some clarification before I merge jruby-1_7 into master !


consider internal iconv_* instead of mb_*


I thought you referred to the style. Anyway, you are right about the single rule for all the links.


Please use :djadmin:`pootle init <init>` so this will actually link to the pootle command


Duplicate, see line 110


If we want consistency, then a.tag as a selector should be placed externally and not nested.
I mean:

```SCSS
a,
a.tag,
.btn-link,
...
```
What do you think?


а можно было бы сделать так
try (Connection connection = ConnectionPool.getConnection()) {...}
finally {
// do nothing with connection
}

это называется try with resources - фишка 7 явы. jvm сама бы позаботилась о закрытии соединения с БД


コミットコメント間違い。正しくは fixed #212 [リソース]NumbersからCSVを直接出力する


Isn't it the case that `0.3-` includes versions greater than or equal to `0.3.0` __and__  pre-releases? From the manual:

````
Suffixing a version with - allows any pre-release versions as well.
````


![Uploading 用例.png…]()
![Uploading 用例.png…]()


^^ I don't like goto but sometime, in some case, it's very convenient ;)


yeah thats definitely post instead of pre


Nice catch. Another workaround, commonly used, is to store a reference the value of `this` you need in a variable before entering the callback function:

``` js
// …
currentSoundFile.setVolume(this.volume);

var that = this; // or var SongPlayer = this;
                 // Usually performed at the top of the current scope.

currentSoundFile.bind('ended', function(e) {
  that.next()
  // …
});


I think there must have been artifacts with casting cost 0 on the battlefield when I tested this. When there are, you can then pay X to target something else, but I can see how this is a kind of chicken and egg scenario.


Придраться не к чему. Оценка 9.
10 было бы, если бы вы настроили логгер для работы с базой


Could change commit message to (needs 80 char wrapping):

```
GitHub API v3 supports some parameters which are not supported in github3.py
for retrieving comments associated with an issue.

Following the implementation of issue_params(), a new method
issue_comment_params() has been added to process the params values.
```

from

```
GitHub API v3 supports some parameters which are not supported in github3.py
to retrieve comments associated to an issue.
Following the implementation of issue_params, a new method
issue_comment_params is created to filter the params values.
```


Comunque ragazzi io ho usato quel blu come colore dominante, se non piace ditelo sinceri e adesso perché almeno lo cambio ed evitiamo brutte figure, mica me la prendo :)


yes. But it should never be necessary to explicitly tag `julia 0.3-` in any new release of any package.


If you put this alias (user logs in with...) to this method, then next time you put this test step to story file your test will only open page, but it will not be able to log in to site despite its obvious from step description.

I think its better to put this alias to next method signingInCheckup. In this case in test for phrase "user logs in with anya_test@yahoo.com and hannah_evergreen1 on main page" test will really try to enter credentials and sign in to site


Looks good.


No rush!.

Interesting project btw!, I assume you'll be using pype32! :P

I'm planning to release a new version of pype32 with some fixes and improvements in the following weeks but I'll wait for you PR to merge the changes.


I prefer `.kind_of?` as I find it a bit more descriptive, but just suggestion


This was the other option I also had in mind but I decided to choose one between the two of them and we would have a conversation about which is preffered (as we do now). I agree with what you say so, I'll do it that way.


FPS is affected by a number of things, generally it depends on how many new textures it's creating, any special fx it's trying to do and how many draw calls are being made.

In the case of baldurs gate, the texture cache isn't noticing that the textures already exist probably due to conversion of formats and so forth so it's constantly making new textures and uploading them to the GPU, which is a slow process, completely negating the point in there being a texture cache, hence the low fps.

if we can fix the mass allocation, that should bring the GS usage down, memory usage and raise the FPS considerably.


This also dropped the test database on my machine.


Ah, I get the distinction now. Thanks.


I think we need to discuss it.
For now enrichment is not just only about adding constraints. It responsible for adding all information (including auxiliary, that will be removed further) necessary for future merge. For example it adds "format: none" which is incorrect format, but necessary for correct merge and reduce. But I'm agree that it is confusing with Snowplow enrichment.
What about supplement or augment?


literówka


@ace2nutzer, @ChronoMonochrome this Synapse/UKM combo is really cool. It is much easier to modify then NT app (it just shell scripts), syntax is a bit odd, but this can be worked out. Synapse has a few drawbacks, but I've already made few feature requests and if I get gradle to work, I'll just fix it myself


Fair point. How about Annotator?


I review the code and the issue 163. But I believe the fix is not decent. The code of postgresql has no such change and the Infection should be avoid. So please see my fix proposal in branch fix_163(https://github.com/jaiminpan/postgres-x2, also ask the pull request). I also know my proposal is a temporary fix because i believe the root cause is not here (The bug here is just a side effect)


That's perfectly ok to call `onSongEnded` from the `PlayerBar.controller`, although I'd say it kind of defeats the purpose of separating that logic from that very controller. I'd keep the call to `next` somewhat "local" to the `SongPlayer` service here, personal preference though.


Perfect.


A może moglibyśmy przenieść to do /js/directives/postDirective.js?


voting


If its possible try not to use waitABit method. Use wait methods for waiting for some action (waitForElement, waitForTextToAppear). In this case your tests will react quickly and more stable


Ora che i messaggi sono tornati neri sta molto meglio!

Il blu nel menu non è male, forse un po più scuro starebbe meglio ma non ne
ho idea
Il 02/Giu/2015 12:08, "ghia16" <notifications@github.com> ha scritto:

> Comunque ragazzi io ho usato quel blu come colore dominante, se non piace
> ditelo sinceri e adesso perché almeno lo cambio ed evitiamo brutte figure,
> mica me la prendo :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igorravasi/Memo/commit/9851d874e4179e98e51d4f33d0bcf2a85e0af19f#commitcomment-11472297>
> .
>


Awesome!!!
So brunch too will show time taken like make does??? As brunch used mka


OK, go on with the changes so that we can open a pull request and get some feedback from the community, too.


Мда тут я точно не подскажу, не занимался пока серверным рендерингом. Да, но штука вцелом классная, попробую все-таки доразобраться со своей проблемой.


Yes, lowp, mediump and highp are all implementation defined (with some minimum limits).
I believe rpi uses a pretty high precision format for all of them.


Good first step, Mr. Bachelli was sugesting to also recognize the types automatically, but some types are really complex.


ten console.log już chyba nie jest potrzebny


Nice :)


So I am implementing these changes to all themes, ok?


@joeyespo `PatternMatchingEventHandler` / `RegexMatchingEventHandler` are sure a way to go if you want to do post-factum matching.

I fail to understand your reasoning though :) Your typical project can't be possibly as deep as an average virtual-env folder. Just looking at a `.tox` folder for one of my projects at random:
```
$ find .tox | wc -l
94582
find .tox -type d | wc -l
4792
$ find .tox -mindepth 13 | wc -l
425
$ find .tox -mindepth 14 | wc -l
129
```
I really doubt you would have a project with 5k nested folders, 100k files, going 15 levels deep?

Now even if we use a pattern matching regex handler, it's still too much stuff to watch. What pytest does is it avoids crawling into `norecursedirs` folders at the *recursion* level, not at the handler level. Now, since watchdog doesn't natively support hooking into observers recursion logic (unless one subclasses those and rewrites part of that logic), the only other way to handle this is to subscribe to everything that is NOT ignored (so if you have folders `.tox`, `a`, `b` and `c`, just subscribe to the latter 3 -- who cares it's just three folders and not five thousand).


Actually it's "5 nonzero elements by row". But 1st and 5th elements are equal, so we store 4 values from each row.


Yes ;)
Op 2 jun. 2015 12:11 schreef "Akhil Narang" <notifications@github.com>:

> Awesome!!!
> So brunch too will show time taken like make does??? As brunch used mka
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OwnROM/android_build/commit/ffa2316b1edb28f088c64d1b6b165b4baa24d3ad#commitcomment-11472375>
> .
>


How does this work? It is possible to add comments in the Sibelius file?


@mathiasrw  **Question**: in many places you replaced constructions like:
```
if (variable === constant) ...
```
to
```
if (constant === variable) ...
```
Can you provide me additional information on these changes:Is this recomendation of bitHound or just restyling?


I think we can make this set -euxo pipefail
No need for separate lines.

As an aside can we fix the commit message here to make it clearer what/why we're making these changes.


Yes, I suppose.


Should check /usr/local/lib64, shouldn't it?


Does "character code of a string literal" make sense? I guess there are no char literals, but it still seems slightly inaccurate.
Also, should it mention that this is a compile-time feature?


Or just include the astyle changes to your next commit.


+1 for a new version


We don't define character literals so I don't think it's inaccurate. You'll only get to see this on single character strings which should be fair.

We could mention the compile-time nature of this. I'm not sure if this is really relevant as far as the API usage is concerned.


Perhaps I'm missing something here, but having the detailed output only when sys.argv < 2 suggests you will see "all exit code responses" but only when you haven't specified a file.
So in essence, telling me what the exit code for 1 and 2 means when its an exit(3) seems pointless?


Hmm, this looks involved. I can comment a little on syntax, but then the whole idea might have a simpler method would be better.


Can be one line:
set -euxo pipefail


Do not need to use parentheses if not initializing variables.


Note the name has now changed to `AxisAlgorithms.jl`


I'll try it and see if it works, thanks, then merge something like this into the main project. It seems that people even use this thing, so might as well try to redo it a bit more properly (no way that I'm going to implement the weird Telegram protocol). A good place to start would be to use the node-irc module.


__API Blueprint syntax error on line 1__:  (Autogenerated message from apiary.io)


I think it is relevant, since the compile-time nature makes it more efficient than a `charCodeAt(0)` call.


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


Think Jesse already mentioned but why not site.yml?


Wow! Awasome one! Much more elegant like this. But is [`bootstrap-fix.scss`](https://github.com/diaspora/diaspora/blob/develop/app/assets/stylesheets/bootstrap-fix.scss) still useful, then ?


@mfojtik was it intentional to drop the `$HOME/` from `$HOME/.local/bin/`?


Closes #14.


I think this looks good apart from above points, after over the shoulder review together.

We looked at tests together, seems like params aren't currently tested elsewhere either.

We looked at raising if validation fails, seems like maybe a design choice to ignore bad parameters. This apparently is the behavior of the GitHub API too.

We looked at documenting the default behavior of the `direction` param. This is currently undocumented in the GitHub API, seems best if we don't document the undocumented behavior to avoid confusion.

Seems like `github3/issues/comment.py` is the best home for `issue_comment_params()`, it doesn't seem to need to live in `__init__.py` like `issue_params()` does.


Typo "OSAD_CONFIG_FULE"


Hi Sven,

- It turns out that there is no need for -DANDROID so we can remove that. (In fact, I already have).
- How does the android flag default to `true` above? Perhaps I don't know how cabal works well enough.
- If I remove -DUSE_DLSYM then when I try to build my game I get (during linking)
    function hs_OpenGLRaw_getProcAddress: error: undefined reference to 'glXGetProcAddressARB'
   Also when I look at the symbols of libGLESv2.so and libGLESv1_CM.so there are no references to any
   symbol with substring 'GetProcAddress'
- I'm using Android 5.0 now but I wanted the game to work on older devices, so I'm using GLES v2


> I'd keep the call to SongPlayer's next function somewhat "local" to the SongPlayer service here; personal preference though.

I would say me too, but I hadn't find an elegant way of doing it.
Using ``` var that = this;``` is an even better option. I hadn't think of it.


Well... shit. :)

How exactly it doesn't work? No sound, cracking sound, desync?

Other things that would be helpful:
1. On the filter property page "Rate Matching" status should be "Yes"
2. I'm also interested in those flags https://github.com/alexmarsev/sanear/blob/master/src/MyPin.cpp#L72 (needs debug build, https://github.com/alexmarsev/mpc-hc/commits/ar is up-to-date)


@mishoje alooo


Why? ;-)


Hi @MasseGuillaume

My configuration is geared towards the latest `nixpkgs` [master branch](https://github.com/NixOS/nixpkgs). This means of course that you can use my configuration with the latest unstable builds too, but you'll have to wait until Hydra updates the channels. I think that the current latest unstable channel update does not yet include the `font-awesome` and `powerline-fonts`. Hopefully the next update will be soon, and then you should not get an error anymore.

If you should use NixOS stable, then you'll have to wait for 15.06 to be released. Then the error message should go away as well.

Hope that helps, and if not, don't hesitate to ask me.


nit: I think we can have different title here?


Why is still 1.7 the latest version on NuGet? Isn't the 1.8 release stable yet no new commits the last two years?


It is not exactly correct. What happen is that GS move data in the memory. The RT on the GPU isn't in sync anymore. Therefore we must
1/ stop and flush rendering (uber slow)
2/ transfer the GPU texture to the local memory (slow)
3/ combine GPU texture and local memory (slow)
4/ reupload a new texture (slow)

On the above testcase it happens 100 times by frame => limited to 10 fps.

Someone need to add more debug message to trace those bad-ass calls. And analyze the movement of data. Potentially (in special case) we can replicate the move from a GPU texture to a GPU texture. In short only update the texture content instead to invalidate everything.

I think texture upload is fast enough with 8 bit textures. It divides by 3 the bandwidth requirements so PCIe can handle it


As pointed out by @koikonom, this might expose the password (which was already the case before this PR, but …). I'll fix that.


Intente agregarselo al validador para que se lo haga a los recursos de Jugador pero no parece funcionar en ese caso.


I think that should be a priority, adding debug logging stuff to the texture cache and the giftag handling so we can dump out a log of everything that happens, it's such a pain trying to mess with directx and gpu monitoring tools, if we can work out what's happening in GSDX it will give us a better idea of workflow.


This also fixed 7 sins buggering out menu when 6x native was used


@zycbobby I changed the logic to judge whether UserStore has been initialized.

`Object.keys(_teams).length > 0` is not the 充分必要条件


Opps :P

Looks like I need a new name for my plugin, I should probably look through
the PocketMine forums and GitHub before I name my plugins next time.

On 1 June 2015 at 10:27, Victor T. <notifications@github.com> wrote:

> http://forums.pocketmine.net/plugins/morecommands.743/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CrazedMiner/PocketMine-MP-Plugins/commit/d688076ce4b4ddc8858076e5d89af7ecb2d103ee#commitcomment-11446287>
> .
>


Cool :) there was probably a couple of games that had bus introduced from the snowblind changes


Fair enough, I have added it. I'm still not sure that people who need this information make the "compile-time feature -> it's faster" connection but it's not like adding it is problematic in any way.


Should've referred to Graylog2/graylog2-web-interface#1066


I fixed the underlying but leaving tempdirs lying around in the mean time. https://github.com/DanielG/ghc-mod/commit/14c097530eb8701086482bf2bd9215a1b5569ed4

Sure single user is the common case but this type of race condition is just embarrassing to let slip into any program ;) and I don't want to exclude the possibility of running ghc-mod on a server in the future.


I guess we wont need these once the view is really split?


we should do absolute but not using HOME var


Kya battameezi hai yeh?


@taconaut confirmed and rolled back, thanks!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.223](http://teamcity.monogame.net/viewLog.html?buildId=7926&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 496, ignored: 6 Build time: 00:19:55


Um, scratch that. No localizations! :o


It can't be linked from the help center, actually; we don't allow navigation from `https://` to `chrome-extension://` (well, I _know_ we don't allow it for `chrome://`, I _suspect_ we don't allow it for `chrome-extension://`).

Regardless of how we get users to this site, I'd prefer to see the `uninstallSelf()` call be done as the result of an explicit user action (in order to avoid being phished into navigating into the page). I wouldn't worry too much about the UI, just `<button>Uninstall "Keep My Opt-Outs"?</button>` would be enough for the moment.


Commit replaced by babd0cde8145533dc2ab176e918efb6e17c4c3c4.


@raz123, I agree that the C convention is to have optional arguments as the trailing ones, but there are precedents in Lua where the option arguments are internal to the argument list (e.g. `table.insert()`).  However, there is also a convention that function callbacks are the last argument, and therefore I feel that the proposed change is preferable.


I have merged your commit in dxva_egl branch and will be merged to master soon


thanks a lot!


Do you want to persist save whole request into Session? We are persisting HttpSession's. Will not cause an exception on our side?


The method extraction was done to make clear why we're doing a String.replace(). Form of documentation


Лайк.


It feels as if this test is a special case of the test added in e70462fe7607855117ccb2712fe94de5294f094b which checks for an up-to-date checksum. I think I'd combine the two assertions into a single test. I think it's confusing to get two failures when the checksum doesn't exist.


@rooque  https://github.com/mi3-dev/proprietary_vendor_xiaomi/tree/lollipop-mr1


@Tubyhes missing documentation


@Bitti09 https://github.com/mi3-dev/proprietary_vendor_xiaomi/tree/lollipop-mr1


`.kind_of?` tests class of instance, I need to test class hierarchy here (`Ptable.kind_of?(Template)` is false...)


Well I already added a dump of gs register, hit/miss of texture cache. I think we miss a couple of debug message around invalidation, at least the move part. Hum, I'm not sure everything work on windows but it must be now.

Giftag might be too verbose at least to debug the HW renderer issue.


A question here: cannot we just use copy_expert here? We may relax the hasattr checks performed there (and just let the pqpath code fail if the file object doesn't have the right attrib). For the rest the two methods look very similar.


@Tubyhes Missing documentation. If this is the best possible interface it would be a good idea to explain its rationale. It seems a bit too much right now. Also please explain what the delegate is in the documentation.


Rows 80 to 84 -- you need to sanitize your input information before you use it especially when you pass it to MySQL.


Hinweis: Der bessere Weg externe Ressourcen einzubinden ist `<link href='//fonts.googleapis.com/css?fa...`. Dadurch werden die Ressourcen mit dem gleichen Protokoll geladen, mit dem auch die Seite geladen wurde.


maybe all these backend functions and constants are better defined in a pair of file of their own? They also have a different naming convention from the rest of the library and pqpath is already fat enough :)


@JulienBreux What kind of impact do these changes made?


remove


@bobthemighty I renamed this parameter, fyi


@Tubyhes this is a public library you know that right?


using of the capitalizeAll function has been added


If the OutputField is re-rendered without the parental form, the event handler will be registered multiple times :/.


По поводу initialValue я бы добавил ешё следующее. Значение initialValue === undefined могло бы передано в функцию осознанно. Для того чтобы не ограничивать разработчика использующего твой API, я бы реализовал сделал следующую проверку: ```arguments.length >= 3``` таким образом ты сможешь отличить ситуацию когда параметр не был передан и был передан но значение равно undefined.


Only the Windows specific characters need replacing so it shouldn't be platform independent.
Making this change might be problematic when the master and slave jenkins instances are running different platforms


Regarding the flag: Cabal flags default to true, see https://www.haskell.org/cabal/users-guide/developing-packages.html#configuration-flags.

Regarding `dlsym`: I remember there are some complications/bugs/etc. regarding dynamic linking OpenGL ES on some platforms. It's somehow expected that `glXGetProcAddress` doesn't work, because Android has no GLX AFAIK, but I thought that `eglGetProcAddress` should work. Perhaps some missing library? Anyway, I'll try to Google around a bit to find out more. And it seems that we have a safe fallback for Android (`dlsym`).

Regarding the OpenGL ES version: Fine with me, but I'd like to make it configurable somehow, with v2 probably being the default. Looking at http://developer.android.com/guide/topics/graphics/opengl.html, even v3 should be fine in theory, but I'm sure that quite a few manufacturers screw it up. The dynamic libraries for the various versions should be `libGLESv1_CM.so`, `libGLESv2.so` and `libGLESv3.so`  if I see it correctly.


@lmy441900 What is your environment and compiler? I'm investigating this issue, but can't reproduce it. CC @MingcongBai


@Tubyhes can you remove the not absolutely necessary NSLog statements? It is reducing performance and generally not considered a good practice: http://www.theonlylars.com/blog/2012/07/03/ditching-nslog-advanced-ios-logging-part-1/


На мой взгляд здесь стоило вместо while использовать цикл for так как количество шагов итерации заранее известно. While же несёт семантику цикла с заранее не известным числом итераций.


使文本框获得焦点


@Tubyhes documentation missing


This look like this after correcting the Indent! :crying_cat_face:
![firefox_screenshot_2015-06-01t20-11-02 352z](https://cloud.githubusercontent.com/assets/7114151/7934389/b3be01b8-0948-11e5-81b3-ce9d89625b5b.png)


@m-kuhn is it possible to shift the left position of 0.2 px either for text or icon? For me the icons are too close to the checkbox. thanks

![image](https://cloud.githubusercontent.com/assets/1374682/7934314/94e74410-0926-11e5-8ffa-5b22de844d70.png)


@Tubyhes  this name is confusing, it is not returning a notification


When the user is not authenticated the correct HTTP code is 401 Not Authenticated.

405 is used to indicate to the user that the method that he used (Get, Post, Put) is not allowed for that URL.


This check is no longer needed?


This is depending on the defaults to be set as expected. The code itself is basically a straight copy from https://github.com/puppetlabs/puppetlabs_spec_helper/blob/master/lib/puppetlabs_spec_helper/module_spec_helper.rb#L22-L44 and is (at least mentally) on its way to rspec-puppet.


Pleases Read.


nice. (you could also try http://momentjs.com/ for time calculations.)


Just saw this....Was wondering why it wasn't working on my local.


@Tubyhes you probably thought long and hard about this but what implications will this have? Maybe it could be better to also generate some feedback to the user that they are using an older iOS version that is not supported? Or to the app?


Supposedly it is mingw-w64.
2015年6月2日 下午6:58于 "Tom Li" <notifications@github.com>写道：

> @lmy441900 <https://github.com/lmy441900> What is your environment and
> compiler? I'm investigating this issue, but can't reproduce it. CC
> @MingcongBai <https://github.com/MingcongBai>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AOSC-Dev/Anthon-Starter/commit/4ab5e04306897a05a46179a7e4e917d905ae7937#commitcomment-11473062>
> .
>


Ok, I think I'm able to change all that to do it on `refreshRendering()` I prepare a new commit with all those changes.

>As a side note, _noAttachedPanels is a confusing name. _numUnattachedPanels would be a bit better, although that's also confusing because "attached" usually just means that the DOM node is attached to the document, not that it has finished being upgraded into a custom element.

I use those names because the property I can test is `attached` or there is another way to know when a element has finished being upgraded into a custom element ?


Well obviously we don't have to have them all on at the same time, perhaps a dev logging section so we can select what things to log, similar to how pcsx2 does it.  sure it'll be huge logs, but im used to looking through like 3gb log files :P


@Tubyhes maybe this could actually better be done in the CSSensor to track whether at least one sensor is enabled and if at least one sensor is enabled we can enable the location provider. It would also mean keeping a counter of nr of enabled sensors somewhere.

Right now it might be confusing that location provider is automatically enabled for some sensors/providers but not for others...


hahaha


Seems not - parent will start with smb:// and will most certainly be invalid utc path.


I'm 99% sure that these while loops causes the Notification section to lag, due to pkt_codabalePropertyNames being called over and over. Reverting this commit, brings back the previous smooth scrolling.


@Tubyhes If you want to remove this it is probably best to remove all references to it, also line above and also the actual setting? Note that this will kill backwards compatibility for some apps probably .. ?


there are no this keys in subscription transaction


Bithound have requested to change comparement with forexample `undefined`, `null` or `0` to be `===` instead of `==` (or `!==`instead of `!=`) - see http://archive.oreilly.com/pub/a/javascript/excerpts/javascript-good-parts/bad-parts.html for why `===` is better.

Changing the code I restyled to a habbit of reducing the risk of accidentally assigning instead of compare by placing the constant at se first part

So - no other reason than style. If you prefer consistency I can leave the order unchanced...


Actually the 'input' event does not have to be registered on the form element, any other parental markup tag works fine. IMHO users will have to add this JavaScript  by themselves, this seems not to be OutputField's responsibility.


thanks mutolisp!


This seems like it'll make the call to unsafe_convert more expensive due to the creation of a GC frame? Maybe that doesn't matter give the cost of the containsnul test?


This line causes the progress to reach beyond the mMax.
I think you can leave the original calculation here, as you've already calculated the target scale before.


Now widget is getting "Invalid order: amount cannot be nil and should be more than 0", most likely it is a String here.
Should be coerced to Numeric in stright-server/orders_controller?


maybe rename this method name to be "post" instead of "do" due to the underlying post call?


为毛HTTPClient大写而endpoint小写？


this is mostly "just in case", or for overall consistency

higher level error handling may send TLS Alerts to the other side on read requests, so a read may return a "blocking write" return value


@Tubyhes just don't do anything when we are not in iOS 8?


Congratulations :)


The block of code in question is JavaScript, not CSS, so contains a syntax error because `auto` isn't a string literal.

After fixing this the map placement and behaviour is incorrect:

![screen shot 2015-06-02 at 12 05 18](https://cloud.githubusercontent.com/assets/4776/7934531/5cd34840-0920-11e5-8933-6977e7c0f8af.png)

This patch also sets the height of the map to the height of its containing element, which causes usability issues. It shouldn't be any taller than the viewport.

Can you describe better the problem that you're trying to fix?


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


@Tubyhes just don't do anything when not in iOS 8?


@STELIX: you'll also need StatusBarClockStyle.java fixup for the CUSTOM_CLOCK_DATE_FORMAT_INDEX const:
https://github.com/TeamBliss-LP/android_packages_apps_Settings/commit/482f13b910466e0101ba1891435d2d2123c5cfd5


@MingcongBai I need more details...Just like the UNIX world, you can't say your environment is **GCC**...


inline 前多了个","


ご指摘ありがとうございます。
既にmayahさんが修正してくれたようで、こちらもありがとうございます。


@Tubyhes names of these functions are a bit misleading probably better to change. Oh and documentation obviously :)


dont see where it is used. if it is default, then prefix the name as defaultEndpoint? also make it const?


I'd recommend tox over make for python testing


I haven't noticed that. The scale calculation doesn't take account of the step and the `setProgress` method called immediately below should prevent the progress being set beyond `mMax`?


Added some documentation too, and a bit of debug.


Thanks - I was trying to stop the map from laying underneath the masthead, which was a result of `position: 'fixed'`

![image](https://cloud.githubusercontent.com/assets/11175760/7934629/505d187e-0921-11e5-9c0e-d12034c0b409.png)


I have written a simple test case, but I find there is no any issue. I tested it under 64-bit Windows 8 with TDM-GCC; Gentoo Linux ~i686 and ~amd64 with GCC 4.9, 5.1, and Clang.


Shouldn't this be ProvisioningTemplate?


No behavior change that I could detect.
Accept for losing the warning that `GetMacOSStatusErrorString()` is deprecated.
I have no specific test for this code or the fix, however, since the condition on line 140 puts us in the right error domain it should be equivalent.  Normally, you would use `GetMacOSStatusErrorString()` for the case that the domain might be something else.


may be final


Tutorial: http://learn.koding.com/guides/getting-started-ruby/


Sure


actually we can use KeyFMap here


Also, to make testing easier:
1. Use shared wasapi
2. Disable peak limiter in the renderer (it adds latency)
3. If you use some fancy usb dac, better switch to onboard realtek crap. Rate matching is in more or less proof of concept state right now, and is very aggressive - some dac's may need bigger latency.


Remove these two routes and use ```form_for(@company, url: {action: form_action}) do |f|``` in ```_form.html.haml```


Well I myself did it in AOSC OS3 with mingw-w64, GCC 4.9.1 if you are
asking... and no, I cannot reproduce this.
2015年6月2日 下午7:13于 "Tom Li" <notifications@github.com>写道：

> @MingcongBai <https://github.com/MingcongBai> I need more details...Just
> like the UNIX world, you can't say your environment is *GCC*...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AOSC-Dev/Anthon-Starter/commit/4ab5e04306897a05a46179a7e4e917d905ae7937#commitcomment-11473364>
> .
>


Just wait for @lmy441900 to explain himself... if so.
2015年6月2日 下午7:17于 "Tom Li" <notifications@github.com>写道：

> I have written a simple test case, but I find there is no any issue. I
> tested it under 64-bit Windows 8 with TDM-GCC; Gentoo Linux ~i686 and
> ~amd64 with GCC 4.9, 5.1, and Clang.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AOSC-Dev/Anthon-Starter/commit/4ab5e04306897a05a46179a7e4e917d905ae7937#commitcomment-11473426>
> .
>


All data points do have a TTL, but I do not think it is necessary or even makes the most sense to store it as a field because it is a calculated value that changes with time; therefore, a static value does not make a whole lot of sense IMO. I need to add javadocs to TTLDataPoint to indicate that it has a very specific purpose, which is setting the TTL of tags using the current TTL of the corresponding data points. Maybe the class name could be something better to indicate that as well.


`WritableSlice` has `getKey`


Recursive call? Actually missing `args` as a parameter.


Hi @jamesfoster , I read your code, but I don't know how to call this API,  Could you give me some advices? Really thanks.


Awesome :+1: Thanks!