This is how `accoutrement-fonts` expects it to be written, for the sake of parsing each variant individually. The previous was a typo. In order to write it that way, we'd have to inspect strings and split them on any space character.


Ok, rescheduled


Ah, makes sense.


i wonder if it's feasible to just check current action for that


Maybe this program runs like this..
```
class login
	Naver.Login()
	Save login data to Program.cs

class Form1
	Get id pw from Program.cs
	Navaer.Login()	(Checking & timer running!!)
	TimeTick()
		Naver.GetNotiCount
		Check noti value changed
			Set value
	/* Can not understand... */
	IF Logout click
		new Login(false)
		this.Dipose() 	//
	/* ... */
```

Try these.
1.  ``` Logout ``` has something wrong.. (Checking error)
2.  some account can login.
3. it shows ``` logged in ``` when I send wrong password. (``` Login ``` has wrong)
4. use ``` Cookie ``` when you try to connect ``` "http://my.naver.com/" ```.
  It's not good way login every check. (winhttp get / set cookie)
5. I suggest use ``` HttpWebRequest ``` insted ``` winhttp ```

p.s Check ``` "https://nid.naver.com/nidlogin.login" ``` again!


Hahaha fixed this yesterday but didn't push :-/

No idea how it got there though... seems my Git gymnastics are still a little sloppy.


Thats exactly what I had intended to try.


The trigger should use the stack. You are allowed to remove counters before it resolves to prevent the win. Reference: "the ability will do nothing if Helix Pinnacle has lost enough counters such that it has fewer than 100 counters on it when the ability resolves." -- http://gatherer.wizards.com/Pages/Card/Details.aspx?multiverseid=150992


An odd change (likewise the .csproj) for a commit that only talks about switching to FluentAssertions…


@portante, i think that macro way will be better


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/19f74838f28065588b653703c05b378c0f7ebbca


merging MonsieurNicolas/stellar-core/maintenanceStartup = 19f74838 into auto


MonsieurNicolas/stellar-core/maintenanceStartup = 19f74838 merged ok, testing candidate = 0ef06651


you should try to rebase when possible to avoid merge commits being added to our hx.


asfafdasdfasdf


Why "All" if it's only going to be one (or zero) subscription?


The close match / double matchCost thing is a little specific for an EditDistance class.


Not sure what was the reason for slice in the first place but anyways, revealed interesting behavior
```javascript
var x = [];
x['01'] = '1';
x['02'] = '2';
x['3'] = '3';
console.log(x) // [3: "3", "1", "2"]
console.log(x.slice()) // [3: "3"]
````


I fixed this in 9b0d2cc8d6fe1f471edd9ee94392416f39269edd. The fix was to add a use for the generic enums in the test-case. Note that generally it would be preferable to sort out failing test cases pre-commit :-)


This uses the stack as the condition needs to be checked again when it resolves.
Reference: "Both conditions need to be true for Barren Glory's ability to trigger, and they both need to be true when the ability resolves or it will do nothing. " -- http://gatherer.wizards.com/Pages/Card/Details.aspx?multiverseid=136048

Reverted the change in https://github.com/magarena/magarena/commit/0af8637fdc476f57623b096fcdc635c283516f11


@benichu Is this necessary since we define the Ruby version in the Gemfile?


Replace "dummy" with "unused"


I Installed windows7 sp1 32bit to the virtual machine and installed visual studio 2015 community and succeeded to build the proxydomo.
![1](https://cloud.githubusercontent.com/assets/647810/11733260/feb51368-9fee-11e5-910d-ae579597b5b4.png)
![2](https://cloud.githubusercontent.com/assets/647810/11733261/fec08bda-9fee-11e5-9e43-4cad298bb537.png)

C:\Program Files\Windows Kits\10\Include\10.0.10240.0\ucrt
Does stdio.h and ctype.h exist in?
![3](https://cloud.githubusercontent.com/assets/647810/11733400/3c8cb500-9ff0-11e5-96a0-31d5714f73a5.png)
Include directory
C:\Program Files\Windows Kits\10\Include\10.0.10240.0\ucrt
Does it exist?


I think you can only have two {} arguments in  LOGGER.debug statement, I think you should be using String.format()


Huh?


Okay, more philosophy stuff: do you think beauty is subjective? :)

Beauty is subjective for a single person, but it is *statistically objective* for general human-beings. It means people can create predict the feeling for beauty with some common pattern. This is how Hollywood works, this is how aethetic works, and this is why machine learning works, fundamentally.

So is readability. It is not subjective, it is "statistically objective", and it is reasonable.

Okay, 300-line limit is indeed questionable. Why not 299 lines, or 301 lines? But there has to be a hard non-negotiable limit to make this thing work. What you are really looking for I guess, are ways to refactor and break inter-dependencies when things grow bigger, and keep it under the hard limit.

The way I do it is to add function pointers in the main builder/formatter, like this:
https://github.com/e8vm/e8vm/blob/master/g8/builder.go#L37

There might be better ways and you are free to propose, but loosing the line limit is not the way to go. 300 line is already pretty long, it is close to the limit of the code that a person can handle in an one-hour software engineer interview. An algorithm that needs 300-lines of code to implement is definitely a pretty hard one (almost has to be something like a balanced tree, I remembered my AVL implementation was 300 lines in C++). Remember that the users we are facing and trying to attract are very junior programmers.


1947-SizingZone (kbenne) - i386-Windows-7-VisualStudio-12: OK (2337 of 2373 tests passed)

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


It seems that tuning is appropriate. Set this to priority -2 too.


Looks good to me! But Matt just suggested to use absolute value for both the total and each column here.


OK ,cool


Willard, I fixed the image issue, now it looks neat!, pull this branch, test and let me know..


ㅋㅋ


야 이건 아니지


For your extended thought:

then the code should classify items into different categories, or build tables that represent routing in form of data rather than a large switch (which is essentially also a routing table behind the curtain).

Also, we should keep G language simple, so that that won't happen in the future. But it is a valid concern for other scenarios (like when building a code generator for x86).


Remove Link and create a way to pass it as an input parameter


remove these comments, and fix the bug.


rename me!


Must use port 8080.


Help! I'm trapped in a terrible name!


fix the way you provide the .mp4 to the script. This only works on your local machine.


Thank you😎

On Wed, Dec 9, 2015 at 10:32 PM, Adarsh Pandit <notifications@github.com>
wrote:

> Yes!
>
> On Tuesday, December 8, 2015, pangperry <notifications@github.com> wrote:
>
> > For typos in pushed commits messages (like jeystream, above), would you
> > leave them/it alone until it's time to rebase?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/pangperry/solitaire_cipher/commit/442826b55d1ae3097f861dfbfd4a27377ebf7cf5#commitcomment-14864740
> >
> > .
> >
>
>
> --
>
>
> --
> @adarshp <http://www.twitter.com/adarshp>
> 313.454.1515 (mobile)
> See when I am free: http://bit.ly/adarshcal
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pangperry/solitaire_cipher/commit/442826b55d1ae3097f861dfbfd4a27377ebf7cf5#commitcomment-14892257>
> .
>


fix the way you provide the .mp3 to the script. This only works on your local machine.
 Add a line note


Hello. Which milestone build is this in?


Do you mean the location of the footer? There isn't an offset that makes sense in this context. I guess I could stringify the options object.


用户注册修改


Remove this file.


sick


#4994-ControllerWaterCoil-TemperatureAndHumidityRatio-control-active-when-it-shouldn't-be (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1967 of 1983 tests passed)

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


Remove the .classpath.


Remove the .factorypath.


Remove file .project.


I was confused with RecordReader. Yes. Stringifying the options could be useful.


Missing Javadoc


ffff


Should probably be `NeuralNetwork[L]`


@adrian-prantl Thanks!


It's there. Line #630 & 726 in my second commit.


It's not a problem at all, it just requires me writing (really copying) more code. I can make it work for any types you want, we just have to decide what those are. MP12 used errors terms over \Z, which is why I only implemented it for Int64.


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/96176118


合并本地分支


Good catch.


What was the error? I’d assume it was unhappy because `expect(x).to.exist` is in theory not doing anything, only relying on the side effect of checking a property (i.e. it’s not calling a method and the value of the property isn’t assigned to anything), which I’m guessing is the same reason you changed lines like `expect(JSON.parse.called).to.be.true`?

If that’s the issue, using `expect(data).to.have.property("service_code")` would also mitigate it and not require you to use `should`. Beyond that, @bensheldon is the main maintainer for this project, so I’d leave further matters of style to his decision :)


@yuhow, since you are having questions in the note, how about creating issues in this repository so that we can discuss them?  If necessary, we commit new changes and write equations in the notes (unfortunately GitHub markdown github/markup#274 doesn't show LaTeX in pages).

For the quick question discussion, please try to upload iPython notebook.  You can easily create equations in iPython notebook and let GitHub show it, e.g., https://github.com/yungyuc/cythonup/blob/master/module/interact_with_c/lax.ipynb .

cc @tai271828


(This code is orthogonal to MP12, because it's just computing norms in cyclotomics.)  I suppose most applications will have already discretized the error, though, so if it's a lot more code to support Double then don't bother.


this won't pick up the variable if it's set in `config-file`, though. Anyway, i don't feel comfortable merging this commit until we have tests for it, since `append_hash` would now have 5 different ways to be set, and i'd want to make sure we're doing the order of preference right between the command-line flag and `config-file`.


Ignoring you :)
FormattedMessage generates a span but if you put a span inside an option, browser does not like it ;)


:gb:


@yuhow , I would like to see the notes that you are confident about in this document.  I think you should give the notes a proper title other than "Yuhow's Note", which is not informative.  For those you are not sure about (the questions), note them in the notes.


To be honest, I forgot I was testing this.


```R
> d1 <- data_frame(t=seq(as.POSIXct("2015-12-01", tz="UTC"), length.out=2, by="days"), x=1:2)
> d1
Source: local data frame [2 x 2]

           t     x
      (time) (int)
1 2015-12-01     1
2 2015-12-02     2
> d2 <- d1 %>% inner_join(d1, by = "t")
> d2
Source: local data frame [2 x 3]

           t   x.x   x.y
      (time) (int) (int)
1 2015-12-01     1     1
2 2015-12-02     2     2
```


Testing


Linhas devem ter um limite de 80 caracteres. É boa prática e faz parte do estilo de programação escolhido.


Can you explain what the naming convention is here for `auth__form__input` ? Is the idea to use double-underscore in place of a space in the selector? Should we prescribe this in the `css.md` docs?

Also, is there any reason we couldn't just use the full selector, spaces and all, in the string?

```sass
z-index: z-index( 'root', '.auth__form .form-fieldset input' );
```


For an invocation of the form foo(6, a + b), the trees for "6" and for "a + b" are visited and added by the normal walk. I think Aleksey is asking if those trees will appear twice in the result due to being added in the normal walk and by visiting the newly-added argument nodes.

Certainly we don't want them added twice. A comment explaining what's going on, including an explanation of why no duplication occurs, is definitely in order.


I think the check for DateTime is redundant if we check for a Date.. a DateTime is a Date

2.2.2 :004 > DateTime.new.is_a?(Date)
 => true


ops, using $state.go


Borrowed from here https://github.com/Sanqui/pokered-randomizer/commit/25afac9abb0dfe088eb002e523dd6433efe65c49

I'd been noticing that some trainers were spinning in place when they weren't supposed to, and this was occasionally causing them to walk through walls and see you, or making them avoidable when they shouldn't be. This seems to have taken care of it in initial tests, anyway.


Random thought: source & target are weird names for these. Maybe "input" and "candidate" or something along those lines?


i think its to print the html page but not sure tell me more plz ?


i removed those useless link button because i made them wih html but they may be needed ? springrts url may change ??


nit typo: unique


One of the niceties of VB.


Maybe just don't even consider names longer than 256 characters? What's the longest public name in mscorlib/system/the compilers/github/...?


sorry about all ths favicon link ^^ but we are now aviable on chrome windows 8 and 10 and all standard browsers !


@danielinux, Done for sixlowpan_iphc_tf, should probably happen for sixlowpan_iphc_tf_undo as well. Something for the train later today.


For future reference: it's a package that contains only third-party binary files, simply rebuilding it is never required.


new magnify system ( wanted to start from 0 to learn so i took a new one ) and actually old was ugly and too much useless stuff.


@doutriaux1 can you wrap this long line?


Php code still here for release version !


you really need to tel me why those php links !


@mathiasertl: I think you have **Issues** disabled on this project.  I can't figure out how to open an issue.
                       I believe that @jaraco wants you do do a release of Fabric3.


styles.css not finished sorry.


Done!


scheduleService, subjectService - are more clear names for service variables


I've heard of it but also not witnessed it. Can always make the check strategy pluggable in the future I guess.


Think it should be multiplied, not added, yeah.


Totally agree with you @remicollet. Never version your composer.lock in a library or you can get non relevant results in your CI platform


picture of OWASP -> picture from OWASP

talking about security, the threat agents in the picture

motive of the attacker, what he or she is trying to achieve (joko yksikössä tai monikossa hyökkääjät ja motiivit, mutta ei hyökkääjä ja heidän motiivinsa)


debug code ;)


Here should be ${PACKAGENAME}. It cause some problems in Android 5


partly done with automatisation -> partly automated. Tyylillisesti siistimpi


This needs to be:
for (; kv_pair != NULL; kv_pair = strtok(NULL, "&,\r\n")) {

(added comma as key/value seperator as that's what I see on our gateways)


Hello,

we have implemented this code below but not able to see the banner ad.

we are able to see the interstitial ad but what we want is banner ad at bottom center.

also how to get the Device Hash for ad.

Code :
var ad_units = {
			ios : {
				banner:"421303174734325_442748962589746",
				interstitial:"421303174734325_443106955887280",
                nativeAd:"726719434140206_777151705763645"
			},
			android : {
				banner:"421303174734325_442748962589746",
				interstitial:"421303174734325_443106955887280",
                nativeAd:"726719434140206_777151705763645"
			}
		};

		// select the right Ad Id according to platform
		var adid = (/(android)/i.test(navigator.userAgent)) ? ad_units.android : ad_units.ios;

		// set your hashed device id if testing on device (optional)
		if(FacebookAds) FacebookAds.setOptions({
		    isTesting: true,
		    deviceHash: 'fdc5f2914ce1d0642d7311b901489253cacbf80e'
		});

if(FacebookAds) FacebookAds.createBanner( {
		    adId: adid.interstitial,
		    position:FacebookAds.AD_POSITION.BOTTOM_CENTER,
		    autoShow:true} );


Extra empty line detected at class body beginning.


Extra empty line detected at class body end.


Unused method argument - `object`. If it's necessary, use `_` or `_object` as an argument name to indicate that it won't be used. You can also write as `map_record(*)` if you want the method to accept any arguments but don't care about them.


Use `fail` instead of `raise` to signal exceptions.


the source code jos on myös the application configuration.
look for troubles -> look for problems?


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


Line is too long. [104/100]


Unused method argument - `attributes`. If it's necessary, use `_` or `_attributes` as an argument name to indicate that it won't be used. You can also write as `map_entity_attributes(*)` if you want the method to accept any arguments but don't care about them.


Unused method argument - `attributes`. If it's necessary, use `_` or `_attributes` as an argument name to indicate that it won't be used. You can also write as `map_record_attributes(*)` if you want the method to accept any arguments but don't care about them.


Unused method argument - `attributes`. If it's necessary, use `_` or `_attributes` as an argument name to indicate that it won't be used. You can also write as `unmap_record_attributes(*)` if you want the method to accept any arguments but don't care about them.


確認しました！


Using function to check it.


harded to test -> harder to test

it is easier to leave holes for attackes in code of bad quality -> bad code is more likely to contain security holes? (Ainakin attackes kannattaa korjata attackers)

As you might now the the security -> As you might know, the security


Breaking there and returning mapped after skipping the else is identical functionality, it was just a semantics change for linting purposes.  I didn't look into the logic itself, and if you see some improvements that could be made, go for it. :+1:


If you suggest that we keep the names simplifyTokenList1 and simplifyTokenList2 that is ok. But the new Tokenizer::simplifyTokens() needs a different name.


It was actually meant to not make a decision in TravellingSalesman depending on how it's used. If `element` would have a pointer type, that is making a decision depending on how it's used, since it assumes that the element is harder to copy than `intptr_t`. Right now the choice is open. E can be a pointer type or a reference type or a value type. The correct usage would be to use `TravellingSalesman<int>` for types that are lighter than `intptr_t` and to use `TravellingSalesman<Object*>` for types that are heavier than `intptr_t`. This causes a performance dip for neither of the two options. If a pointer to `E` is always used, it causes a performance dip for light types (which are precisely what are used in `LineOrderOptimizer`).


@NateWr this was set to a small value because the grid scrolling feature calculates the best height in JS for the grid based on its elements and on the scrolling element scroll height. If there are no grid elements, the 150px are enough to display the grid.

The scrolling grid height was designed to respect the items per page setting, so that's why the grids with this feature must have a dynamic height.


"how your system is setup." -> "how your system has been set up". (set up on kaksi eri sanaa jos se on verbi).
Toien vaihtoehto olisi muuttaa koko lause esimerkiksi näin: "These tools will test your system's setup." jolloin "setup" ei olekaan verbi.

"installation is done by best practices" -> "installation follows best practices" tms. "installation conforms to known best practices".


1995-InducedTerminalRemove (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2336 of 2373 tests passed)

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


![screen shot 2015-12-11 at 2 08 25 am](https://cloud.githubusercontent.com/assets/2508142/11741204/33d2669c-9fac-11e5-8412-b10658b970ce.png)


Thanks!


directly the security
acting against the law if you scan targets without their permission.


(
    JIRA(...),
    basic...,
)


is in the core of security, the performance... could use to amplify his or her DOS attack.
(tietääkö lukija mikä on DOS?)


1. If edit  savings item -> delete old image from os
2. Add signin with google+, facebook
3. Add flash


No. The `PluginManager` calls `Promise.resolve(object[method]).then`


We must find another solution for this. It's better to put it in options, because some users need with time other without time.


is in the host or in the application

is needed thus you - tästä puuttuu nyt joku välimerkki. Puolipiste? Väliviiva?


You read the spec correctly. And what I said about fillBuffer() was a wrong assumption indeed. However what I said about there never being spaces at the beginning of the buffer is true (nextTranslatedRow takes care of that, e.g. at L186). So, while there can be spaces at the end of lines, and while this is "wrong" according to the spec, actually it doesn't matter because you can't see spaces at the end (maybe I should remove that line from the spec?), and spaces at the end are clipped when they exceed the limit (see L191).

My problem with trim() is that it's a somewhat higher level convenience function that deals with white space, which is exactly what we're trying to do but in a controlled and custom way. trim() seems to aggressive and it simply won't work what the spaces are blank braille patterns. Also, the trim() line won't solve everything, it would have to be repeated e.g. at lines L177 and L184.

If I were you I would just change the tests, maybe add a note somewhere about the "imperfection" in regard to the spec.


This is going to be topology dependent, we can't just hardcode it in the ATT setup


jira = (
    JIRA(options=dict(server=self.url, verify=self.verify),
    basic_auth=basic_auth, validate=True, max_retries=1)
)


tighlty -> tightly


If you need to "query" the photographer, you can use Dictionary here.
After you processed all the data, turn the Dictionary into Array.
Then it can use less `append` and `removeObject` on photographers.


2 things one you have a typo

```
mv ${D}/usr/local/hbb-browser/lib/hbbtv-ssl-$OPENSSL_VERSION.app ${D}/usr/local/hbb-browser/lib/hbbtv.app
```

it should be

```
mv ${D}/usr/local/hbb-browser/lib/hbbtv-ssl-${OPENSSL_VERSION}.app ${D}/usr/local/hbb-browser/lib/hbbtv.app
```

but second tuples only have precompiled libs for ssl 1.0.2a, oe-a use 1.0.2d, so this commit will still cause build failure, as "hbbtv-ssl-1.0.2d.app" does not exist in the zip.


Alliances -> Alliance's


Thanks, updated!


trisashley@gmail.com kthxbye


[PR wanted]


let logger format string.. use , instead of %   logger.error("string %s", ex)


disturdbed -> disturbed
harded -> harder
Example how one could use it from -> Example for using it from ? Tyylillisesti ehkä parempi?


`usingBlock` is suitable for method signature, but not for a variable name.
You may use `usingBlock block: (UIImage?) -> Void` instead.
Or just use `completion: (UIImage?) -> Void`, which will be easier to understand.


Instance field and constructor parameter are typed differently. Not inherently wrong, but I'd say in this case they ought to be the same.


there are several jscripts for Javascript-Load-Image. Which one is needed? Also, how do I include it, just in the html as normal?


This commit has not yet been pushed to the master.


comments is good thing instead -> comments is a good thing. Joku välimerkki, vaikka ihan piste ennen instead sanaa.


Hi Dima - will add assertion, also this test is not green yet - i will push
the working (green) test later.

On Fri, Dec 11, 2015 at 10:36 AM, d-konovalov <notifications@github.com>
wrote:

> Where is expectation? (assertion)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EquinoxBlack/automation/commit/a9acc80061ff8ef573a0c071be495cd4e1f26d7c#commitcomment-14919770>
> .
>


Besides, will it be downloaded twice if `downloadImage(url: NSURL, usingBlock: (UIImage?) -> Void)` called faster than you download an image?
Think about what have you done in image filter.


@MarcusWalther: we are heavily depending on the prompt availability of nighty builds for a critical project. If you cannot fix this by today, would you mind rolling back the commit until you have a solution for this problem?
Thanks a lot!


I'd say pipeline is a special case, it is making a chain of actions (that's easier to see at the start of line), but `<>` is rather a continuation of single action – building a string. :)


keeps at track -> tracks

metrics instead välistä puuttunee joku välimerkki.


continous -> continuous.


I know this, but I wanted to be extra safe. Here's my old commit https://github.com/farmdve/unicorn/commit/d378ce1d1c042b607f7b250c65150fcbe2ea7ef7


Try to use `URL` instead of `url`, which is common in Apple docs.


@ullrichp
Wenn du artifactId verwendest, steht da am Ende `silo_schema`, das ist quasi ne Konstante.
rootArtifactId wäre hier angebracht


The funny thing is that Python 3.0.1 does not provide function `_Py_fopen` and this code simply does not compile.


yes, i konw. i used to use dictionary. but i want to try something new. so i used array


LGTM


yeah good job.this was annoying


maybe better use connction_ok or something like this instead of ok


What is the new thing??


Great, cheers


The return type ought to include `null`.


Ping @vezzi @Galithil


Is it required to store the User in a serialised manner?


It's hard to see what it was there for. Why would we be adding events to the start of the timeline, but not paginating?


All you need is to ask (dudu-dudu-dudu)
All you need is to ask


This ought to be `Locale`, rather than `string`.


What is the use of the above code?


I didn't want to change the original code more than necessary


We should only use fingerprints - key IDs are deprecated and dangerous.


Nein, dort steht dann something_schema ... probiers aus ;)


typo


like array.filter(), implement the Equatable protocol, etc


ok


What's the rationale behind having a VO for default locale and having an array of strings for available locales (rather than VOs, possibly in a set)?


Looking good for this particular scenario. I am just wondering whether one should create helper classes like `GetDeclaredFieldAction` similar to Hibernate Validator. I guess it comes down to, whether we think this is a once off fix just for this class or whether the whole ORM code base needs to take security manager restrictions into consideration. I am a bit surprised that this has never been discussed before. I am as @sebersole a bit confused on how this particular class is causing a problem now, but not any of the other zillion of places where ORM is doing reflection.


Habe ich, und da stand silo_schema


Derive Big business 大小寫這樣沒問題嗎


Yes, I agree that the trailing spaces don't matter much. I think that the spec could say something like "...collapsible spaces at the end of line boxes MAY be removed".

You're also right about trim(), so I ditched it and changed the tests accordingly.


Appveyor might be an option  http://www.appveyor.com/


You were not the abuser .. not at all.  I don't care if other people rename the modules or claim authorship or whatever ... .  Everyone is free in doing whatever they want with my scripts .. .  There is no support on them eitherway, so ;-).

And you're welcome .. :-)  glad to be able to help at least one person ;-)


`"PT-2M0.00000001S"` is a correct representation of `ofSeconds(-120, 10)`.
Are you not confusing it with `"PT-2M-0.00000001S"` from `ofSeconds(-120, -10)`? Note the second `-`.


there is no opal_getenv function


The different methods used in VB.Net for global storage is by declaring the same in Application , Session , Global asax, Modules etc. Which method should we follow?


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


fix #109


Чотко, только помним, что `id` мы пишем в виде CamelCase. Это делается дл того чтобы мы визуально различали названия классов и идентификаторов.

Исправлять не нужно, но запомнить стоить :). С таким образом `quest` превратится в `Quest`, `country` в `Country`.


um... it is just slightly different from objective-c `NSMutableArray` and `isEqual`.
I think you may try this on another thing.
Less code, less bug.
Try to use `filter()` and `Equatable` on another situation.


side menu with controller


there is few layers of header-files i am not sure PACKAGE is workable here


agree


LGTM


i dont know, :smile:


Всё ок, только помним, что `id` мы пишем в виде CamelCase. Это делается для того чтобы мы визуально различали названия классов и идентификаторов.

Исправлять не нужно, но запомнить стоить :). С таким образом `text` превратится в `Text`, `email` в `Email`.


This is `null` initially.


realize -> understand
is it clear?


我沒有仔細檢查，抱歉
目前已回退並且重新修正


Good catch


Илья, а кто сказал, что можно не закрывать теги?
Я уточню, я не вижу ни одного закрывающего `</tr>`.


Thanks for the clarification, ill try to read the code better new time. Im a nub, but i should have spotted this. My first thought was "OH noes, 40 sec response times in the ap again..."


We already have a user defined Response class in our website and this one has been extensively used. When trying to call the Response object, this Class is getting referenced.


Fundamentally fine. Personally I' move the "in some cases", so say "In some cases the OMPT API may allow a tool to ..." (If you want to leave the "in some cases" where it is, it should have commas before and after it...


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


Wow, those files had some bad spelling. Nice update!


Неплохо, но `<p>` вот тут
```html
<td><p>Question:</p></td>
```
как бэ лишний. В нём здесь нет смысла и необходимости, а вот написать `<label>` вместо него и завязать с соответствующими control'ами будет найс.


to cache which photographer did user select.
if the user selects the same photographer, then do not need to clear the cache. else clear cache.
just want to release the memory


@mstajdohar, wouldn't it be better to use [gzopen](https://github.com/genialis/resolwe-bio/blob/39fc5adf4fb5000117c8b1ad3e194c0dddd31bca/resolwe_bio/tools/utils.py#L9)?


Would probably be worth adding a test to make sure it actually still starts up.


You're right, I'll fix that in the next release as I won't publish a patch just for a typo in the CHANGELOG ;)


Export


Also, will there be a jQuery clash with cropit?


I'm not familiar with this code at all, is `__path.join(".")` correct or should it be `__path.join("_")`?


> older PHP.

Haha. All PHP 5.x. ;)


Is the function name correct ?


The Liches, Lich and the other one, can't think if the name right now. Begins with 'R'?

Sent from my iPhone

On 11 Dec 2015, at 07:33, Melvin Zhang <notifications@github.com> wrote:

> then confused myself after the last win/lose game triggers were 'Whenever'
>
> which cards are these?
>
> —
> Reply to this email directly or view it on GitHub.
>


:-1:  You should not just remove this. Maybe change it (or even avoid adding new aliases to already defined ones)


Does the accumulated error ever get so high as to be of order `intg.dtmin`?


почему эти поля публичные? это значит, что ты их в любой момент можешь снаружи поменять и целостность запроса может нарушиться. Или есть какая-то причина?


Does not exist in 7.1 branch.


Как мы и говорили **`id`** - это уникальный идентификатор элемента на странице, а это значит, что **не может** на странице быть **2 одинаковых `id`**,а у нас их мати рiдна.
А также, `id` должен быть написан в _CamelCase_ стиле и там не подразумевается никаких ` `, `-`, и `_` также не всегда желателен.

> Исправляем


Is that relevant ?


Add a comment to be able to make a push test


timeout не работает, т.к. функция вызывается мгновенно =(


для документирования методов лучше всего придерживаться принятых методологий, в частности, посмотри PHPDoc


Pretty sure it's correct. We use ExternalConnection quite a bit and it's
working fine with the change i submitted.

On Fri, Dec 11, 2015 at 11:38 AM Simon Krajewski <notifications@github.com>
wrote:

> I'm not familiar with this code at all, is __path.join(".") correct or
> should it be __path.join("_")?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/flipflophhj/haxe/commit/6c71b8be129779bbdb3e30ae5ec9d53c23522320#commitcomment-14921958>
> .
>


Примеры работы с методом можно вынести в readme.md


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


XD


Найс, только `id` пишется в _CamelCase_.
И `<label>` для остальных элементов тоже можно применить. Не только для `radio` и `checkbox`.


Everything related to the Protocol Message is on the charge of the developer, we shouldn"t interact with it unless for serialize/unserialize on send/receive


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


it will not be downloaded twice actually.
in the same view controller, the individual image will only be downloaded once, unless the user press back and force very quickly.


Какой смысл дополнительно делать LIKE, NOT LIKE, если по умолчанию делается то же самое.


Лимит с двумя аргументами работает только в mysql. По RFC должно быть limit и offset. Лучше заменить на 2 метода, limit и offset.


yes, i know. but i think the purpose of this project is to get familiar with swift.


ParseFromString is the function which unserialize protobuff object.


Fixed in c9fa33410d3a40e4dde0a33b993d91263580e8bd
I added some tests to check for fill generators on points and line layers to avoid such issues in the future.


SHit


ok, so it should be private


OOOOh!


Конфиги вынеси в отдельный файл с примером вызовов


В библиотеке этого не делают


No way


Why not just use the `image_tag` helper?


哈哈，刚哥好:smile:


yes it is. A logical wrapper to manage Protocol buffer need to allow user to add data to their package and for that they need to get this pointer.


I think this need to be a field.


```python
import numpy as np

T = np.arange(0,10,0.01)
t = 0.0
for i, tt in enumerate(T):
    if not i%100:
        print(i, tt-t)

    t+=0.01
```

Gives:
```
0 0.0
100 -6.66133814775e-16
200 -1.33226762955e-15
300 1.99840144433e-14
400 4.13002965161e-14
500 6.2172489379e-14
600 8.34887714518e-14
700 1.04805053525e-13
800 1.26121335597e-13
900 1.4743761767e-13
```
Accumulates to more than dtmin in less than 300 iterations!


But ImageDownloader has no responsibility to know that it will download once, so think about their responsibilities to make less error to the caller object.


if it is private, the user can only serialize object and wont be able to unserialize string to get data :/


@glock45 Shouldn't the source files be at the root of `Sources/`? Seems like it's the canonical practice so far.


This indexOf trick is genius!  If it had returned Option then this wouldn't have been possible, -1 return codes FTW!


This is not there anymore.


remove leading space


Why do you need these 3 pages?


ㄎ ㄎ


This appears to be doing two jobs. Might be worth splitting it into two methods and renaming to help make a little more sense as to what the method does?


Again, why do you need these 3 pages?


Thanks!
I will play with this and report back once the next release is out.


Maybe update or add test for this new part of the code


yes ,you are wright!!!


You can keep confident to `NSCache` which will auto clear if it receives memory warning.
Unless the rare move done by user, no need to clear the cache.


Many thanks!


Кпача теперь обновляется по фокусу в поле капчи и фокусу в `textarea`.


Co za rzeczy są @mikolajswiatek ? jakiś skończony issue? :D


The test must not know the rename URL. This needs to go in a page object, if one is not available already.


метод save исходя из названия должен сохранять данные, а у тебя судя по всему делает все подряд.
назови тогда его execute или типа того или подумай, как лучше сделать


That was quick! I'm just starting out with nix and love the enthusiasm of all the contributors!


Okay.  There will still be an error but it should be in line with everything else in the code.  Feel free to submit a pull request.


I see, but this is guaranteed and honoured by the API, so it is unnecessary.

Thanks.

On Dec 11, 2015 6:18 PM, "farmdve" <notifications@github.com> wrote:
>
> I know this, but I wanted to be extra safe. Here's my old commit
farmdve@d378ce1
>
> —
> Reply to this email directly or view it on GitHub.


Same, the test must not know how to locate the terminal checkbox and the submit button. This needs to go in a page object, if one if not available already.


[uploader] documenting Tandem-specific notes on checklist


This method returning only the root category, only one job. But I will extract this to a class and then I will decorate on the controller.


確認しました！


( ﾟдﾟ )


![Good job, bro](http://mincore.c9x.org/gamil.png)


Or you could use method extract into a private one to keep things simple? Up to you.


Also reverted the map type to "normal".


According to the Javadoc for `Duration.toString`, the sign of hour, minute and second should always be the same. I filed a bug against the JDK but have not received a bug ID yet.


Why do you need this page?


Why do you need these 2 pages? (A and A.B)


Again, why do you need this page?


Опечатка


I don't have to much experience with reviews on github but this looks like tabs and space being mixed in?


Again, why do you need these two pages? (A and A.B)


I think you need to update the Serialization.h for writeSIL. It still contains the bool in the declaration.

"error: out-of-line definition of 'writeSIL' does not match any declaration in 'swift::serialization::Serializer'
void Serializer::writeSIL(const SILModule *SILMod) {"


@clochix s/string/String/ ?


The test must not know the rename URL. This needs to go in a page object, if one doesn't exist already.


`createSyncable()` is a bit heavyweight for everyday use. Like Java EE heavyweight.
So, I insist on the `new Model` sugar.

`super()` call issue is resolvable; all it needs is to submit the init op, all `this.fields` can be initialized later.


I think I also fixed this in my PR


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


yes there's a conflict now :cry:


The test must not call any ``findElement*``. This needs to go in a page object, if one doesn't exist already.


Same, the test must not call any ``findElement*``. This needs to go in a page object, if one doesn't exist already.


i'll try to resolve


lol @hak86 , you need patience wait for team RR agree release MM.


You can fix this locally by: `heroku git:remote -a ci-pipeline`. We often assume you have that setup so we don't have to put that everywhere in the scripts.


Not yet run


It's just 4 more spaces than before :)


I missed that part of the doc. Do you have a link to that part?


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


In any folder add OneLastActivity file  ????


Ah I didn't know that


I thought your PR only did console.log and not actually ran the plugin: https://github.com/transloadit/uppy/commit/b43657adf33c108043f80d34cae257d4b9ba5fa9


you should check here that e.NativeErrorCode == (int) SocketError.OperationNotSupported


I think you are right :) ( according to "Creating a Library Package" at https://swift.org/package-manager/#example-usage )


yeah i only fixed it in the sence that it was sourcing correctly and could show the uppy objects. i mean it's more like we have an open pr that touches these same files which is unfortunate. but i resolved the conflicts now in the pr by using your version


:-) Those with less enthusiasm are less visible around here.


Why do you use reflection to get the aspect types?


自動生成されたテスト用のファイルを消去しました。


I think this commit has increased memory usage (probably when Firth regression is used), though I am not sure whether or not it constitutes a memory leak. valgrind and intel inspector_xe don't seems to detect anything at first glance, but firth regression in particular may be worth thinking about.

Another possibility is memory fragmentation, as the usage does appear to plateau


ой. исправила


なにもないよ


:thumbsup: thanks


Would you please include this patch in a new release?

Current version (0.0.7) does not include this patch yet.


null: false の削除
と
avatarカラム（string型）の追加を行いました。


you mean without config files? as you can see from the code there is already lenience towards the files not even existing


ino pak kon dg. file asli proje java sabt kardam . nabayad toosh .py bashe ke .


Jeszcze nie, mam problem z zapisem do bazy (sypie mi datą) i wyświetleniem tylko jednego pracownika dat, nie wszystkich (i trochę gui jest zrobionego)


Didn't Jake said no Hungarian notation?


Done in commit f390b1129bb711d41bd08d9d14c7a8ecb6338c80 (I did not managed to squash).


Can you try this:

    1. pod deintegrate
    2. pod cache clean --all
    3. pod install


I agree on the repeat part. But... I am passing to {{dataMatcherFor}} what I expect, that is, the order in which cells are written should depend on the order of the assays in the result...


use SetState for vibration or just print something?


So the humans 'get' me


But then it does not work with Windows 7 anymore. I have only Xinput1_3.dll


Suppose this should be /jobs/manual/delete


<img width="960" alt="2015-12-11 8 17 09" src="https://cloud.githubusercontent.com/assets/15645784/11742551/307c260c-a044-11e5-97cc-33735bac185b.png">

T.T


TeamCity pull requests :: netty [Build 840](https://garage.netty.io/teamcity/viewLog.html?buildId=3969&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4801, ignored: 21 Build time: 00:36:32


Hi Rianne,

Raw will never be used, that's correct.
Besides log2values, there can be z-scores, or log2ratios (in case of
multiple nodes, and log fold change is calculated based on a 'baseline'
node.), though we don't have this implemented yet.

Janneke Schoots

www.thehyve.nl
email janneke@thehyve.nl
office +31(0)30 700 9713
us cell +1 617 869 9256
skype jannekevdp

the hyve

2015-12-10 17:35 GMT+01:00 RianneJ <notifications@github.com>:

> can we always assume measurements contain log2 values and not the raw?
> From algorithms specification document "Z-scores will be calculated on the
> fly on log2 values "
> "Use 9 decimal points precision."
> No clamping of z-scores is needed (from -2.5 to 2.5 as done right now in
> advanced analyses tab), right @jannekevdp <https://github.com/jannekevdp>
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thehyve/heim-SmartR/commit/5f75a2ea9efd6c5a1a3fea79ee3cbd59aa7d3e19#commitcomment-14904677>
> .
>


looks good, will merge


Ah, I got it! I'll fix it soon!


Hello,

Would you consider putting some of your Forth projects on GitHub?

Thanks!


There's also the file `.git/info/exclude`, where your personal excluded files should go. `.gitignore` is for the public list theoretically useful for everybody (and visible by everybody), `.git/info/exclude` is your personal list (not visible by anyboby else).


This could be shortened to return just

``` ruby
@queue.empty?
```

since `Array#empty?` already returns a boolean


Ah, I got it! Just remove the line `34` and the function from `36-43` and your errors will gone! I'll fix it soon!


Add an empty line, and how about move init function to the end of this file?


Denke mal ein Download Button sollte schon vorhanden sein, immer vom beschränktesten User ausgehen


I don't think that making the getters and setters private is a good idea, even if they aren't currently used outside of the class.


исправила


I think this paragraph could be merged into the previous one with something like:

...and the Cast Web SDK. The SDK was already available for web pages in Chrome desktop (with the Google Cast extension installed) and also on iOS. On Android controlling Cast devices was only possible for native apps.

I think it's confusing that the current paragraph says about desktop and Android native apps but then repeats the information but adds Chrome iOS to the list of supported platforms and native iOS apps.


We have uploaded our  jar file


The commit message is empty. Could you write an explanation on why these changes had to be made to solve the problem?


We have uploaded our output files


'string' in fact, sorry, just fixed it.


Should this change to match new test path?


This is likely only compatible with 1.11 (since it will make the model protected), be sure to update `requires_foreman` to suit.


Gut gemacht Taner!


completant  expresionUnaria i expresionMultiplicativa


Released v0.0.8.
https://rubygems.org/gems/fluent-plugin-ec2-metadata/versions/0.0.8


Shouldn't Lich's last ability "When Lich is put into a graveyard from the battlefield, you lose the game." also be a trigger and goes onto the stack? It has the standard trigger template, eg "When X is put into a graveyard from the battlefield, <do something>".


This is the plugin I’m trying to use: https://www.npmjs.com/package/postcss-svg#usage


Danke, Caro!


Thanks!


Suggestions:
s/web app/mobile web app
s/from you phone/
s/using your phone (or tablet or laptop)/using the web app/


I expected :rainbow: :>


I don't see why "you lose the game" should be any different from say "destroy target nonblack creature" from Chime of the Night's "When SN is put into a graveyard from the battlefield, destroy target nonblack creature."


Environment:the filestore choose Newstore.
In function NewStore::fiemap ,variable offset may be equal to variable start,so m[start] value is 0.
the problem was recorded in the osd log:
2015-12-10 18:41:43.914146 7f85584ed700 20 newstore(/var/lib/ceph/osd/ceph-11) fiemap 0~4194304 size 4194304
2015-12-10 18:41:43.914148 7f85584ed700 20 newstore(/var/lib/ceph/osd/ceph-11) fiemap get fiemap entry, off = 0 len=0
2015-12-10 18:41:43.914149 7f85584ed700 30 newstore(/var/lib/ceph/osd/ceph-11) fiemap get overlay, off = 4128768 len=65536
2015-12-10 18:41:43.914151 7f85584ed700 20 newstore(/var/lib/ceph/osd/ceph-11) fiemap get fiemap entry, off = 4128768 len=65536
2015-12-10 18:41:43.914155 7f85584ed700 20 newstore(/var/lib/ceph/osd/ceph-11) fiemap 4194304~0 size = 0 ({0=0,4128768=65536})
2015-12-10 18:41:43.937608 7f8561d00700 30 osd.11 22593 heartbeat_dispatch 0x7f859c51c800
2015-12-10 18:41:43.937614 7f8563503700 30 osd.11 22593 heartbeat_dispatch 0x7f859c51fc00
2015-12-10 18:41:44.021043 7f8563503700 30 osd.11 22593 heartbeat_dispatch 0x7f8596f03400
2015-12-10 18:41:44.021045 7f8561d00700 30 osd.11 22593 heartbeat_dispatch 0x7f8596f01400
2015-12-10 18:41:44.069612 7f85584ed700 -1 ./include/interval_set.h: In function 'void interval_set<T>::insert(T, T) [with T = long unsigned int]' thread 7f85584ed700 time 2015-12-10 18:41:43.914870
./include/interval_set.h: 338: FAILED assert(len > 0)

ceph version 9.2.0 (bb2ecea240f3a1d525bcb35670cb07bd1f0ca299)
 1: (ceph::__ceph_assert_fail(char const*, char const*, int, char const*)+0x85) [0x7f8580b9ebe5]
 2: (interval_set<unsigned long>::insert(unsigned long, unsigned long)+0x27c) [0x7f858080fc1c]
 3: (ReplicatedBackend::build_push_op(ObjectRecoveryInfo const&, ObjectRecoveryProgress const&, ObjectRecoveryProgress*, PushOp*, object_stat_sum_t*, bool)+0x1122) [0x7f85809bd372]
 4: (ReplicatedBackend::prep_push(std::shared_ptr<ObjectContext>, hobject_t const&, pg_shard_t, eversion_t, interval_set<unsigned long>&, std::map<hobject_t, interval_set<unsigned long>, hobject_t::BitwiseComparator, std::allocator<std::pair<hobject_t const, interval_set<unsigned long> > > >&, PushOp*, bool)+0x486) [0x7f85809c2716]
 5: (ReplicatedBackend::prep_push_to_replica(std::shared_ptr<ObjectContext>, hobject_t const&, pg_shard_t, PushOp*, bool)+0x1a4) [0x7f85809c2bb4]
 6: (ReplicatedBackend::start_pushes(hobject_t const&, std::shared_ptr<ObjectContext>, ReplicatedBackend::RPGHandle*)+0x1c1) [0x7f85809c37f1]
 7: (ReplicatedBackend::recover_object(hobject_t const&, eversion_t, std::shared_ptr<ObjectContext>, std::shared_ptr<ObjectContext>, PGBackend::RecoveryHandle*)+0xf8) [0x7f85809c39b8]
 8: (ReplicatedPG::prep_backfill_object_push(hobject_t, eversion_t, std::shared_ptr<ObjectContext>, std::vector<pg_shard_t, std::allocator<pg_shard_t> >, PGBackend::RecoveryHandle*)+0x45d) [0x7f858079157d]
 9: (ReplicatedPG::recover_backfill(int, ThreadPool::TPHandle&, bool*)+0x19d9) [0x7f85807d3959]
 10: (ReplicatedPG::start_recovery_ops(int, ThreadPool::TPHandle&, int*)+0x8bc) [0x7f85807d672c]
 11: (OSD::do_recovery(PG*, ThreadPool::TPHandle&)+0x356) [0x7f8580605e76]
 12: (OSD::RecoveryWQ::_process(PG*, ThreadPool::TPHandle&)+0x27) [0x7f858064e827]
 13: (ThreadPool::worker(ThreadPool::WorkThread*)+0xa76) [0x7f8580b901a6]
 14: (ThreadPool::WorkThread::entry()+0x10) [0x7f8580b91070]
 15: (()+0x7df5) [0x7f857ec34df5]
 16: (clone()+0x6d) [0x7f857d4dd1ad]
 NOTE: a copy of the executable, or `objdump -rdS <executable>` is needed to interpret this.


`q_line` would be fine as a local variable. There's no compelling reason here to make it an instance variable.


LLR: 2.96 (-2.94,2.94) [0.00,5.00]
Total: 6026 W: 1047 L: 901 D: 4078
state: accepted, tc	5+0.1, threads	7


The commit message is also empty here, as in #289, can you explain the issue and how you solved it?


хорошо. исправлю. у меня как раз вопрос возник почему он красным выделен


@ivank Decripting.


DELETE


Missed this the first pass. Missing 's' in nosetests


Excellent use of PORO class here. Good job.


typo. issue


I don't think we need to check that (also in all other occurances) since mHistoryDeviceAdapter will never be null after init() which is called in onCreate()


it is very good tool...i wish you to add option for insert fake AP essid manually because if i want to hack hidden network , the AP essid is null


+1 for returning meta information for paginated request.


Other than two minor typos. This looks good. Thanks!


should never be null since mHistoryDeviceAdapter is assigned in init() and before the instance is subscribed to the sensor facade.


Excellent Job using a PORO to handle pagination. Looks good and clean.


Article à placer Livre IV, Titre I, Chapitre unique, Section 7, Sous-section 3


We need to see but I'd rather lock an extra variable


thanks, just fixed this.


sure @mrhainavn generally I update the rom after release of changes log, by the way i am checking for new device and in the list the new xiaomi redmi note 3 but don't know if the device tree of redime note 2 are same for 3. :+1:


You're right of course, I added a note in the README about that. I'll have to change that in the future.

By the way, I did not expect you to see this because the project is so old :D I don't think you need to merge this code because I only played with it, I'm not going to use it I think. I was just exploring what's available for Windows game development.


When using "phpunit/phpunit": "4.8.19|^5.0.0" in Ubuntu 14.04 with PHP 5.5.9, Composer download the phpunit version (5.1.3) that requires PHP 5.6. I am not sure if this is a Composer bug or what.


Could be, but then please add the result into the output so we know why it failed...


@X140Yu just tested this in landscape mode, and there is some issues, not correct working!


good job!


If rook is discovered piece then it is a discovered piece on diagonal and the sliding piece is either bishop or queen (it cannot be discovered piece on rank/file because in that case it would have already given a check), therefore any of this discovered piece rook move will open discovered check. That is why I am eliminating here discovered piece rooks. The same is true for bishops. This is a chess question, so I might be wrong, so please check.


?? New test path?

Sent from my phone.

> On Dec 11, 2015, at 06:24, Aaron D. Marasco <notifications@github.com> wrote:
>
> Should this change to match new test path?
>
> —
> Reply to this email directly or view it on GitHub.
>


Polecam CommonJS w pliku `lib/TabManager.js`.
Nie będziesz miał tyle wcięć ;)


"The hours, minutes and seconds will all have the same sign."


Why we need this method, what it should do?


s/cast to a Cast application (registered with Google)/present a Cast [Receiver Application] (https://developers.google.com/cast/docs/receiver_apps) using the Cast Web SDK.
s/to enable the Presentation API to be used from Chrome on Android with/to enable the Presentation API to be used without the Cast SDK (on desktop and Android) to present any web page to a Cast device without registration with Google. The page using the API directly will work on


Can we change that to "that is to be sorted" since it doesn't have feelings?


I've only put desktop and Android there since I'm not aware about plans for iOS


Comment.


That should do it!


LGTM


tag <option> has no enclosing </option>


We can leave this condition out: if the `children` array is empty, the loop body won't be executed anyway.

If `nil`s were accepted for `children` then we'd need a `nil` check here. But to avoid that you could imagine forcing an empty array in the initialiser if no children or `nil` are passed in:

``` ruby
def initialize(payload, children=[])
  @payload = payload
  @children = children ||= []
end
```


why we need special styles for one component?


You are right we don't need moveChecksOpponentKing anymore (and there are other functions which we don't need). I will do this cleanup once I will setup perft for current moveGeneration and eliminate the bugs.


Jo, stimmt - sry.


Изменения отступов делай отдельным коммитом, пожалуйста.


Comment. Rather return by value.


107952866_correctSurfaceType (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2338 of 2373 tests passed)

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


Extra information:
New sprites have been added that Mike drew up and the headboard scene has been reconfigured. The new sprites are in the res folder.


An alternative way to create this method is to make it an instance method. In this case, you won't need to pass in an instance of User as a parameter. You will be able to use the method like this
```
 user = User.find(1)
jwt_token_hash = user.generate_jwt_token


Could we replace these two methods by a single getter returning VectrSeqDouble?
Put just comment on this idea, what changes in reactions and transport models it requires.


Adruino and esp8266 bits


synchronize around this if since otherwise (hypothetically) the model might already be removed or added by the time you reach the synchronization block. Also it saves some vertical space :)


Thanks @burningice2866 for pointing it out


See comments, good otherwise


Ah; now this time I can  plead a reason: there is no documentation for these fields on the API page; I deduce them only from the response structure.  The fields that have Javadoc are the ones that _are_ documented. I  don't know what mem_dev_total means; for example.


указание типа давай вообще уберём


Please, cancel this commit, test was right.


I failed to get these units... if one set altitude to `2`, it'll be actually setting to `2000m`? No way to set to `750m`?


Do we still need these getters if it is provided by mesh (Balance cleanup pull)?


Byłbym zrobił na CommonJS, tylko nie wiem, jak to potem serwować przeglądarce. Docelowo będzie ładowane przez require.js - test jest na Node, bo to moja natywna platforma.
Raczej nie każdy będzie chciał przechodzić na webmake O.o


Oczywiście "webpack".
Ustawiasz w konfiguracji "libraryTarget: umd" i po zabawie ;)

Pozdrawiam,
Piotr Kowalski

W dniu 11 grudnia 2015 12:47 użytkownik Robert Kawecki <
notifications@github.com> napisał:

> Byłbym zrobił na CommonJS, tylko nie wiem, jak to potem serwować
> przeglądarce. Docelowo będzie ładowane przez require.js - test jest na
> Node, bo to moja natywna platforma.
> Raczej nie każdy będzie chciał przechodzić na webmake O.o
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rkaw92/tabs/commit/a02ad47fafe4a919448d91c86cbafa8b27763f07#commitcomment-14923129>
> .
>


I'd make this two specs so that the descriptions are clearer:

- It interprets single line breaks as br tags
- It interprets double line breaks as p tags


dz nts


Isn't  this kind of getter part of the EquationBase interface?


The add seemed to contain 8 "new" files, dunno where they came from or went. GitShell strangeness


TeamCity Civic Assembly :: Build backend + frontend [Build 75](http://54.229.176.143/tc/viewLog.html?buildId=169&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Subtle error introduced: the code does g_free (uri) all over the place... freeing a const gchar* is not valid (code does not build)


Comprehensive test cases for creating a bucketlist. Handled multiple scenarios. Keep doing that.


I happened to see your fork by looking at my github start page. I haven't noticed that the mksyscall script is now written in go instead of perl, if i did probably just used it. I like the CamelCase names in your version however, I was somehow unaware of this style when I first did xinput.go, but now I know they are the standard for constants.

I use this project only in my profiler (joyster). They weren't updated a long time ago, because they just work for me.


You can do this once:
```js
var quote = (function() {
   var div = document.createElement('div');

   div.style.backgroundImage = 'url(http://example.com/img.jpg)';

   return div.style.backgroundImage.match(/url\((.*)http:\/\/example.com\/img.jpg/)[1];
})();

function getExpected (url) {
     return 'url(' + quote + url + quote + ')';
}
```


Do not understand meaning of this class.


There's no `get_mode`? One has to try all modes?


I think there is a bug: https://github.com/jrief/django-admin-sortable2/issues/79
Older solution wasn't perfect, but relative url was always right.


Looks like you misspelled "canola"!


Good luck at LTC!


Gahh, uri should have remained gchar* -- want me to fix it up? Sorry about the breakage.


Thanks


This should be guaranteed by limits set in the declaration of the input array.


Thank you, Maxim!


This case is an exact duplicate of the one that is being removed.


/* pourquoi pas utiliser la formule du barycentre qu'on a vu en cours? t'as juste à ajouter deux double d et d2, ça donnerait ça :
x =(x*d +xSecond*d2) / (d+d2)
y= (y*d + ySecond*d2) / (d+d2)

et donc tu incrémentes d et d2 en fonction du timer, sachant que d2= 1-d  si je ne me trompes pas */


I checked the commit history and there was no documented reason why this test case should be run twice.


Typo


TeamCity Civic Assembly :: Build backend + frontend [Build 75](http://54.229.176.143/tc/viewLog.html?buildId=169&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 9 Build time: 00:08:49
### Failed tests
```
Blogs tests: Should add new blogs: <no details avaliable>

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

```


I suppose that also states that it is NUL terminated, right?


O. Dzięki. Chętnie wywalę taki np. Index do osobnego pliku, żeby się nie mieszał z TabManagerem.


hehe


Isn't `bzero` deprecated? Shouldn't you be using `memset`?


Makefile requires use of tabs (not spaces)


You can use items.count as against items.all.length since items.all.length is more expensive. This is because items.all.length attempts to grab all available item in the db that has a bucketlist_id of the first bucketlist of the given user.


ok, that was my though too.

no worries, I'll just switch it back and commit it directly. Things
like this can happen.


I think its time to update the Drupal core version aswell, there is security vulnerabilities in both `7.38` and `7.40`.

Some of the patches mentioned in this README is also a part of the newest (`7.41` at the time of writing) version.


top_controllerってなに？

基本的にモデル一つに対して、一つのコントローラをつくるべきです！！


Yes, correct. I left a comment in the original PR #246. This is an Android project and the variable is inside a class that extends `android.app.Activity`. It would be logical to think that standard Android [field naming conventions](https://source.android.com/source/code-style.html#follow-field-naming-conventions) apply. If this is not the case for this project due to a conscious decision by the project owners, then that is totally fine. Would it be a good idea to state this explicitly (e.g. in a coding style guide) to make other developers aware of the altered rules?


@sjoelund : Can you please start the Linux Nightly Job again? I think I have fixed the issue this morning.


This is the Rails version, not the Ruby version.


Interesting, never seen "cascade" before.


wouldn't that create something like
<LETMEGOOGLE>+a+b+c ? we don't want the first + directly after the LETMEGOOGLE


TeamCity pull requests :: netty [Build 841](https://garage.netty.io/teamcity/viewLog.html?buildId=3970&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4801, ignored: 21 Build time: 01:07:53


Hi, die ganzen Plugins müssen jetzt im Iframe eingebunden werden und das div flotplaceholder an die passende Stelle in der iframe.js


Right.


Agreed!


Comprehensive test suite for updating a bucketlist item. +1


We upgraded phpunit because we switched to the new Travis CI Trusty environment (see #373) and wanted to keep close to the version they have (5.0.9 currently) so that unit tests and coverage checks work the same locally and on Travis.

PHPUnit should only be required for development. On a deployment server you can do `composer install --no-dev` which should work even if you have an older PHP version.

We use Ubuntu 12.04 and 14.04 machines for development, but with [updated PHP packages](http://www.dev-metal.com/install-setup-php-5-6-ubuntu-14-04-lts/) from a PPA.


najss


Yes, but this way you're just testing that the order is consistent within the result, rather than 1) testing that the order is the correct one in absolute terms, which I think would be that the assays are ordered by assay id (and which you don't test at all) and 2) that the assays are the expected ones (which you test in a separate assertion in all the tests). It would be easier (no need for separate assertions), not to mention clearer (because you wouldn't be partially testing the result against itself), if you passed the matcher producer the expected assays, in the correct order.


Es wird wahrscheinlich irgendwann auch int_source und anderes geben und so fand ich es irgendwie übersichtlicher. Ist aber möglicherweise unnötig.


GETLEVELBBYNAME() ON NULL HELP TO FIX


@adamwen829 @imcaffrey   review 的时候一眼扫过了。。很多问题正在慢慢改~~ 十分感谢


Did it work out well to update all gems in one go? I think a more common step-by-step style is to first update all dependencies, then update Ruby/Rails. So make the change easy, then make the easy change.


This commit is related to #4, not #5. :confused:


Wonder if other VPN worked though...


Das hier setzt natürlich voraus, dass die injizierten Funktionen nur bei Tests benutzt werden. Würdest du die inject- und reset-Funktionen lieber nach test.c (oder so) verschieben?


hi Micha,

you could improve the readme and send a pull request :)


Yes. We can make this one as general placeholder color.


I was annoyed by a one star rating from Vietnam, so I have just blocked the device. I was tempted to exclude Vietnam and Android 5.0, but I don't want to let other people be victim of some bad guys.


Maybe a line saying what it does – not clear to me. Is it whether Rails loads everything up front?


wew chira thogloif


RDD?


Aha, README-driven.


I like the PR :-)


Yep, check `docs/TODO.md` in the recently updated `experimental` branch.

Cheers


没有意义的注释，不如不要，个人建议。


尽量避免MyXXX这类命名，命名需要有意义。


OK... I see... Then I have to treat them differently... THANKS!!!


TeamCity Jarvis Framework :: CI-GitFlow [Build 69](http://build.prxm.it:8811/viewLog.html?buildId=5158&buildTypeId=CqrsFramework_CiGitFlow) is now running


well keep it up


Could you do `AddinsMRUList.this` ?


The CP utility copies the contents a source file to a target file. You do this my typing "cp -> then the source file -> then the target file"


Cool! Thanks @DigitalFlow  :+1:


Maybe just `">=1.2.0-rc"` for brevity? :bowtie:


没用的注释代码，建议直接删掉。


dns_resource_key_match_cname_or_dname() can return negative on error, this needs to be handled.


parens after be `be(nil)`


hi Ingo,
thanks for the pull request.
code looks fine, I'll run some tests and give you feedback afterwards.


Search client (Firms JS supporting Solr index search) was updated to match last changes performed on index


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


I think this needs the corresponding call on the client to pull out the clientInfo (you most likely are aware of this already but just pointing out in case you weren't)


这里真的还有TODO吗？事情干完了TODO就Remove掉。


这个TAG的值，建议全大写，且最好与当前的class有关。


Hmm... That's very strange. I don't remember I do it explicitly. It seems an issue.


cc @BLangendorf for review.


Very good and comprehensive unit tests for your models. +1 on that.


仔细考虑下，真的需要这个JPG String吗？


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.1.11](http://build.prxm.it:8811/viewLog.html?buildId=5158&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 136, ignored: 8 Build time: 00:04:47
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.DomainTests.StringValueMongoTestsFlatMapper: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.EngineTests.BsonIdentitySerializationTests: <no details avaliable>

```


TeamCity Jarvis Framework :: CI-GitFlow [Build 70](http://build.prxm.it:8811/viewLog.html?buildId=5159&buildTypeId=CqrsFramework_CiGitFlow) is now running


其他有的地方是带m, 这里又不带，如果选择了一种风格，建议保持统一。


I wonder if these should really be treated as regular disk i/o stats..
The time they take is likely to be network..

@ghalliday - Perhaps should either be different category or also register net stats for these remote ops. ?


Is this the timeout we documented in README? If it's important enough to doc, maybe make it a constant so it stands out?


Then you could also more easily document the unit.


Possibly should into a different header, e.g. jstring.hpp


这里的init感觉很奇怪，总之是不好的设计。


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-unstable.4+4](http://build.prxm.it:8811/viewLog.html?buildId=5159&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 136, ignored: 8 Build time: 00:03:32
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.DomainTests.StringValueMongoTestsFlatMapper: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.EngineTests.BsonIdentitySerializationTests: <no details avaliable>

```


I'm so proud of you, @Convery :smile:


Having this many positional arguments is a bit of a smell – maybe pass the attributes object instead?


I simply updated the dirs.props file and then ran a build using the old/new flags as had been previously recommended. No search and replace.


Maybe make "ping me" a link somewhere?


A fix we can trust!


missing related help messages (above)


Ha! You never said it was your "hi" :)


Would be nice to have it as a local, to avoid 3 instances of the expression.


命名有点随意，不像一个方法名。尽量还是保持含义的同时取一个cool的名字。


I disagree with the main presumption that it's better to harm users who are able to run spring (forcing no shaders on **all** intel cards, even those who deal with them OK) than harming those who can't (assume the drivers say the truth).

So I have no issue with blanket flags, I just prefer the opposite default value.


@sjoelund : Thank you for starting the job. The job failed again, but I think the current issue is not related to my commits.


I'd recommend that you don't specify `idleOnly == true`. If a user is executing an addin that uses Shiny or running a Shiny app this will cause the incremental work to wait on termination of the app.

I'd also recommend that you use the other variation of scheduleIncrementalWork that enables you to specify a initial execution period up to a certain threshold -- i.e. how long am I willing to tolerate blocking the R process up front. To me it seems like it would be acceptable to block for a full second trying to index as much as we can. This will give you "instant" responsiveness for most cases but gracefully degrade for the unbounded packages case.


TeamCity pull requests :: netty [Build 842](https://garage.netty.io/teamcity/viewLog.html?buildId=3983&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:08:46


Я предполагаю запуск питоном 2.6-2.7 (что надо бы указать, конечно, в первой строке). А решить что именно?


Ага)


No meu caso não funcionou eu uso o magento 1.9.2.1

Ao verificar pelo inspetor do chrome aparece o erro no console

Failed to load resource: the server responded with a status of 404 (Not Found)

http://rockmezz.com.br/autocompletar/ajax/endereco/

preciso alterar a url no final  Consulta AJAX no final do arquivo billing.phtml

  var url = '<?php echo $this->getUrl('autocompletar/ajax/endereco') ?>';

Pode me ajudar?


Maybe we'll avoid using images as resources?


如果是想表达一个状态，是否可以用mImageViewRecycled?


Minor: Might be nice to wrap this third-party dependency.


太多TODO，看着挺乱，如果已经搞定的事就remove掉TODO


方法名尽量有意义。


use getElement().getStyle().setVisibility(...) instead of manipulating the whole attribute


I take it from the bits used by these, that only 1 of these can be ORd ?
For these definitions that makes sense, might there be others later, where you'd want >1 modifier and therefore should use non-overlapping bits for these?


Da der Downloadbutton ab einer gewissen Bildgröße nicht mehr zuverlässig funktioniert, sollte man den Benutzer lieber darauf konditionieren mit Rechtsklick zu speichern. Solange dieses Problem besteht, ist es in meinen Augen kein fertiges Feature. Lösungsansätze wären das Triggern der "Save As..."-Eigenschaft der Canvas oder das Skalieren der eingefügten Bilder.


Да я даже не знаю, как сформулировать. Постоянно каракули проскакивают в разных местах. Я просто какого-то принципа не понимаю похоже


В смысле, "не хватает проверки"? Ты про то, что не проверить на реальном боте? Во-первых, проверить можно -- на тестовом, а во-вторых, тут же речь просто о том, что функция выдаёт.


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


观察者-observer这段文字，出现了好几次，尽量避免重复。


@abalabahaha @khionu I'm not disagreeing. I was trying to point out to @SimonSchick and @Guichaguri that this is for **library developers** and not **bot makers**


haha yes :) PR? ;-)


use com.google.gwt.dom.client.Element#getId and com.google.gwt.dom.client.Element#setId methods


Should we force CFG_TA_FLOAT_SUPPORT to n in case this is not CFG_ARM32_$(sm)?


The LTC-test is something I'm really scare about, so thank you for the good wishes!
Yhanks to all users offering their machines which make this tests feasible.


TeamCity pull requests :: netty [Build 843](https://garage.netty.io/teamcity/viewLog.html?buildId=3984&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:51


picky: but expected a method called 'mergeStats' to perhaps be merging to sets of collected stats, not a IFileIO into stats.
Perhaps mergeFileIOStats would be better method name?


好好玩~


I recommend to use strip method instead of chomp.


У тебя не винда случайно?


please explain cloned_line.gsub!(/\\"/, 'U'), docs is not matching


Before reassigning to `m_sceneGraph->m_attributesRenderable` here, we need to do a `deferReferenceRemoval( m_sceneGraph->m_attributesRenderable )`. The update we're doing is threaded, but we can't destroy GL resources in just any old thread, so the `deferReferenceRemoval()` utility keeps a pool of discarded stuff to be actually deleted on the main thread.

For the same reason we need to update `SceneGraph::clear()` to do a `deferReferenceRemoval( m_attributesRenderable )` too.


Its saves the data from the relations, right?


So, you removed index_ below ? Is it intentional ?


No, that is due to the server running out of disk space :)


please use defaultAttendee here as well:

  record.data.attendee[0] = defaultAttendee[0];


умничка, все правильно сделала!  маленькое замечание по стилю - не нужно делать отступ в коде для циклов.
твердая пятерка)


尽量避免字符串拼接，用%s方式，stringformat


在这里聊天，真心涨姿势，哈哈。


避免Action1这种命名。


つながって一番最初に {friends : Array} が流れてきてゴミがDOMに追加されるみたい


Это уберу


thug :V


Wait I'm a collaborator :3 :3 :3


Спасибо!!!! Вот оно счастье!!!! *YAHOO*

11 декабря 2015, 14:41:14, от "Sergey Strizhak" < notifications@github.com >:

умничка, все правильно сделала! маленькое замечание по стилю - не нужно делать отступ в коде для циклов.
твердая пятерка) —
Reply to this email directly or view it on GitHub .


Maybe inform that `timestamp` is the number of seconds relative to Epoch.


во-во


尽量避免业内没有形成广泛共识的简写，比如这里的stu


TeamCity Civic Assembly :: Build backend + frontend [Build 76](http://54.229.176.143/tc/viewLog.html?buildId=171&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


```js
var acceptJsdoc = [];
if(accepts.length>0) {
  accepts.map(function(accept) {
    acceptJsdoc.push(...);
  });
}
```

`Array.prototype.map` returns the mapped values. If you don't need the return values, then you should use `Array.prototype.forEach` instead.

However, in this case, you should use `map` - here the usual solution:

```js
var acceptJsdoc = accepts.map(function(accept) {
   return ...;
});
```


After the update of Linux,we will get a "wlan: disagrees about version of symbol" even a "Expection Stack" with this commit( https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/fs/proc?h=linux-3.10.y&id=cf6c05a77c5ef41bd72f45a4a008724ffd393668 ),then the wifi cannot work,so we must revert that commit and disable module version check.


Same comment here.


For time variables, we should use 's64' rather than 'u64'.


&&后面少个空格，建议开启IDE格式化功能，自动格式化这些细节。


Consolidating branches


这里就不可能为null了吧，多余的判断没有必要。


Looks like this is not a static method (https://github.com/doctrine/phpcr-odm/compare/1.2.6...1.3.0#diff-21d202058a853ef0902f2a7f4c84e2d7R41).

Should I create a fix PR for this?


Wouldn't this rather be "*Trying to extend overridden method*" or so?


@vlj j'ai enfin réussie, ça ma pris un long moment mais bon enfin, et ça marche parfaitement, testé sur plusieurs jeux aucun probleme.
Je voulais faire la PR, mais ce qui m'agace c'est que j'ai du désactivé 2 truc tu voulais rien faire, mème si y'en a 1 j'ai remplacé par le bon, il veut rien savoir.


Why the `data`? Isn't the callback enough? Any way, other callbacks usually return a handle - that is used to delete it then, why is this different?


Nicht viele will nur das Frame voll bekommen, muss 8 Byte voll machen...


Why not a dedicated spell class? I think that'd unclutter a lot this kind of code


不理解这里为啥要try catch, 如果有特殊理由，建议注释说明。


Same comments about 'system_clock_monitor' applies here.


@jeanlaurent Could you make sure the ITs don't trigger bugsnag reports?


Sinnvoll?


I like your renamings to `LocallyApproximableConstraint` and `GPConstraint`.


＋前后建议保留空格，还是之前的建议，开启IDE格式化。


I just wish there'd be a way to skip the text 😩


@vlj pour cette ligne j'ai droit a cette erreur
>1>Emu\Cell\PPULLVMRecompiler.cpp(70): error C2660: 'llvm::createTargetTransformInfoWrapperPass' : la fonction ne prend pas 0 arguments


Ухх, да, слышал я про проблемы питона на винде (особенно с кодировками) -- лучше с ними и не сталкиваться) Я бы на твоём месте на сервере разработку вёл (как я), либо просто в виртуалке.


Yes, but it does not specify anything on the nanoseconds there.


@vlj et pour celle la j'ai ça :
>1>Emu\Cell\PPULLVMRecompiler.cpp(81): error C2664: 'void llvm::legacy::FunctionPassManager::add(llvm::Pass *)' : impossible de convertir l'argument 1 de 'llvm::LoopInfo *' en 'llvm::Pass *'
1>  Emu\Cell\PPULLVMRecompiler.cpp(81): note: Les types pointés n'ont aucun rapport entre eux ; conversion nécessitant reinterpret_cast, cast de style C ou cast de style fonction


See #2


no, ubuntu 14.04 only


merging from patch 2


A monster and a player share similar properties like "HP", "MP". What do you think if we abstract these out on a new class?


`R.string` at the rescue mate :+1:


Right, I'll fix that in EN strings.


Maybe add link, to providers, because I'm not sure online is worldwidely known.


Hi,
thanks for this nice script!
just a quick comment: the python script does not seem to have any `-s`  option, only the `README.md` file changed in this commit.
thanks!


I think that regardless if it's logically impossible, all arm32 and arm64 specific source files should be explicitly checked with $(CFG_ARM32_$(sm)) and $(CFG_ARM64_$(sm)) respectively.

So in this file it wouldn't make a difference, but we could do:
```
CFG_TA_FLOAT_SUPPORT ?= $(CFG_ARM32_$(sm))
```
in `mk/config.mk` if you'd like.


Hey Chema, no everything works as expected ;)


done


we might need to add format='js' here


Since this is an SMP test, LTC seems not as useful as just a STC test with more cores. But ok higher depth result might be interesting.


Shouldn't this override the existing file `_posts/de/beginner/0200-12-23-id-editor.md`? Now the menu on the left hand side is duplicated: http://learnosm.org/de/beginner/id-editor/


TeamCity Civic Assembly :: Build backend + frontend [Build 76](http://54.229.176.143/tc/viewLog.html?buildId=171&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 133, ignored: 4 Build time: 00:11:33
### Failed tests
```
Basic Events CRUD Actions: Should Create an Event: <no details avaliable>

Basic Events CRUD Actions: Should Edit an Event: <no details avaliable>

Basic Events CRUD Actions: Should Delete an Event: <no details avaliable>

```


Looks good to me.


Now I see, you are right.


A cluster may have two possible orientations or four. Clusters in the skin typically have four orientations and clusters in the infill typically have a mix. This is why. A typical cluster (in the skin at least) looks like this:

    ──────────
    ──────────
    ──────────
    ──────────
    ──────────
    ──────────

This cluster can be traversed in four possible ways (orientations), like so:

    >────────┐  <────────┐  ┌────────<  ┌────────>
    ┌────────┘  ┌────────┘  └────────┐  └────────┐
    └────────┐  └────────┐  ┌────────┘  ┌────────┘
    ┌────────┘  ┌────────┘  └────────┐  └────────┐
    └────────┐  └────────┐  ┌────────┘  ┌────────┘
    <────────┘  >────────┘  └────────>  └────────<

In the infill, clusters are often a bit different since both endpoints of neighbouring lines are not near. Usually only one line gets put in a cluster. This is by design, since that produces more efficient paths since it can put lines in the `/` direction next to lines in the `\` direction (those two can never be clustered since at most one endpoint is near to the other line). So there you get clusters with only one line:

    ──────────

For this case, two pairs of orientations collapse. There are only two possible orientations:

    >────────>  <────────<

I've changed the way this is stored in `Waypoint` now. I agree that it's better to keep the start and end point together. It is now stored in an `std::pair`. This actually makes the code a lot more self-documenting since such a pair can be called an "orientation".


Anstatt eines neuen Branchs für das FS15 kannst du auch ein Tag FS14 beim letzten Commit von mir machen und am master Branch weiterarbeiten - was dir/euch lieber ist.


I would better to be consistent with the other parts of the code and use `s` instead of `states`.


LGTM


А можно какую-то адаптивную вставку сделать вначале (кроссплатформенную типа). Я вчера что-то такое читал и даже пробовал, но остановился уже на каком-то решении, которое сработало для меня.


What is this code ?


это вопрос


These will eventually be called within the service methods as well.


What is it doing … ?


yes!! docstrings! great commit


I don't think "location" should be in the example.  It's much better to calculate it by geoip when needed than to expect an administrator to put it in by hand.


Why have you stopped developing this?


I did some editing on windows and didn't pay attention to line endings. We will have to be more careful in the future, this caused an issue where because of the \r's this file was not executable with `sudo` on ubuntu.


Maybe a slight comment on, what is the purpose of this Driver would be helpful.


`vm_config_default.yml` doesn't need to be hidden?


does this means the upcomming RaspBian package need special code... or maybe both Debian & RspBian could always both use SDL2, that would make things simpler


This is needed for the environment variables to work:

```yml
addons:
  apt:
    packages:
      - gcc-4.8
      - g++-4.8
```


A change on `cpp/mcr.c` is to `#define` `__CHAR_UNSIGNED__` for `limits.h` on Linux.


OK


```python
# ../addons/source-python/plugins/testing/testing.py

# ============================================================================
# >> IMPORTS
# ============================================================================
# Source.Python Imports
#   Colors
from colors import BLUE
from colors import RED
#   Effects
from effects.base import TempEntity
from effects.hooks import TempEntityPreHook
#   Engines
from engines.precache import Model
#   Events
from events import Event
#   Players
from players.entity import Player
from players.helpers import index_from_userid

from engines.precache import Model

# ============================================================================
# >> GAME EVENTS
# ============================================================================
@Event('player_jump')
def on_player_jump(game_event):
    """Game event fired when a player is jumping."""
    # Get the player who jumped...
    player = Player(index_from_userid(game_event.get_int('userid')))

    # Get a BeamRingPoint temp entity...
    temp_entity = TempEntity('BeamRingPoint', life_time=1)

    # Set some values after initialization...
    temp_entity.end_width = 10
    temp_entity.center = player.origin
    temp_entity.end_radius = 750
    temp_entity.start_width = 5
    temp_entity.alpha = 255
    temp_entity.color = RED
    temp_entity.model = Model('materials/sprites/laserbeam.vmt')
    temp_entity.frame_rate = 256

    # Create the beam ring point effect...
    temp_entity.create()

# ============================================================================
# >> HOOKS
# ============================================================================
@TempEntityPreHook('BeamRingPoint')
def pre_beam_ring_point(temp_entity, recipient_filter):
    """Called before a BeamRingPoint get created."""
    # Is the temp entity color already blue?
    if temp_entity.color == BLUE:
        return

    # Don't allow any color except blue...
    temp_entity.color = BLUE
```


Thanks @tjenkinson :)


@ intels
you need to differ intel (windows) drivers & mesa (that's changed here):

intel drivers:
1. suck hell
2. aren't updated
suck hell and black terrain bug (3-5years ago) is the reason why GLSL is disabled for those. Also intel _never_ updates drivers for old gpus, so when something is broken it will always be. So if at all those bugs are fixed with new gpu gens, but that needs testing ... (also current gpu detection in spring doesn't allow to differ between gens).

mesa drivers:
1. suck hell sometimes
2. _are_ updated often
Also we fixed a lot incompatibilities in spring, and the shaders drivers in mesa are always under heavy development so bugs are fixed likely. So it's worth testing from time to time, if the reasons for disabling stuff for those, have been fixed. And that's what is done here.


More things I don't understand xD


The problem is, the `Image` doesn't know what to add to.  This example only has the `Screen` container that things are being added to, but there will be multiple containers so a `.add` method on the `Component` wouldn't really work.


@JayWood Sweet! :)


Thank you for the re-indentation but...

Can't you do this in a cleaning patch ?


Hello


@HappyHarm thank you for contributing a Dutch file


Hi, Sorry for the late reply.

I will open a PR, but please don't merge yet as this was done in a hurry from a functional POV only, i intend to add more stuff (like tests, update dependencies, etc).

Thanks,
Marin


I would prefer to have sound inside core instead of widget,
Widget directory should only be for widgets.


I agree, and I will be more mindful of the contributor document going forward.


Better not to format line here. With such approach the line formatting is split into two. Partially here and partially in the log_message_loop_thread_win. Better to have it one place.


По поводу events['values'][0]['id']. Оно же эксепшн выдаст? Я просто не помню


I don't have strong feelings. This wasn't cleared up by using the ABC?


Thanks — I've updated this with your suggestion. Still kept an 'In other words ...' sentence to reiterate.


It doesn't look like it would be nearly as flexible. Besides, I need to be able to assign a `Double` or a `ReadChannel[Double]` like I do with `Sub`.


No worries!  Users could simply run `gravity.sh` again and it will be resolved.  This was just a quick fix.

Congrats on completing finals!


Surround class with empty lines. Do not indent class with spaces.


looks very good.


No, `join` combines each element of a sequence with the given string.

    ➜  ~  python3
    >>> "+".join(['a', 'b', 'c'])
    'a+b+c'

One case not handled is the empty search however -- it should perhaps just not do anything in this case, or taunt?


yes please! help to debug all the station things and shipping will be super useful.

cc @Dignifiedquire


indent


@viswanathamsantosh  Please consider this  moving to the README.md file!


btw have a look at this for GPU detection:
http://feedback.wildfiregames.com/report/opengl/feature/GL_SHADING_LANGUAGE_VERSION_ARB

(ignore, the ARB in the name, it's actually GL_SHADING_LANGUAGE_VERSION)


フォルダ削除もそうだけど、
今後コントローラーやモデルを作った時に、
今のままだと同じようなファイルが自動で生成されてしまうので、
application.rbに自動生成されないような記述を施してください。


Could you please maintain some code style? space after if, no space after opening brace, no space before closing brace.

I'd propose throwing an exception when `is_scalar($html) === FALSE && $html !== NULL`, and then casting $html to string every time, although this would be a BC break.


Puedes poner las coordenadas justo al final? Así es mucho más fácil para el proceso de generación del JSON de datos

Gracias!


Global/static variables are difficult to understand and quite often make future changes more difficult to make. I have removed them in favour of `this.accepts` & co in this commit.


I guess the goal of this patch is to add yirl-sound into core.
Thing is, your adding the include, but the use should still manually initialize
the sound library's manually, so you should made game.c initialize your library's too.


Corrigido em 92acc7a


@binky123: I think, in case the scheduled LTC will succeed, a test with more cores will follow anyway,
since otherwise several instructions around line 419 wouldn't  be tested at all ;-)


sorry, I'm still confused. The comment says `# the approximated constraints are at the front of list`, but the slice above seems to take the end of the list. Is there a difference between `posyapproxs` and "the approximated constraints"?


Oh nooooo.... Stupid!!!


s/required/require/


add empty line


It should. I will fix it in EN sources (DONE)


Thanks,the data is fixed,I forgot


I think that nowadays we document each enum value as well...


We got other content to put on readme since we are moving to
github.com/mozilla !
On Dec 11, 2015 8:09 AM, "Abhiram Ravikumar" <notifications@github.com>
wrote:

> @viswanathamsantosh <https://github.com/viswanathamsantosh> Please
> consider this moving to the README.md file!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abhi12ravi/fsa-training/commit/db5881d97aadaad2d3eb483f42fbe2766e1799c0#commitcomment-14924502>
> .
>


```` if (entity.x < (768 - entity._frame.centerX) ) {


this method seems to have now disappeared


great


Thanks. You may want to replace "native Android and iOS Cast apps" with "Android and iOS Cast SDKs from the native apps"?
Cast apps to me refers to the receiver apps (for native apps with Cast support a better term would be Cast-enabled apps). Also it seems clearer to say that native apps have their own SDKs vs the Cast Web SDK referred to in the beginning of the sentence?


great, thanks


This should be inside game.c

Because we have only one sound library's we can hardcode the
sound initialization.


sum(Vector==0);
% nie byłoby prościej i szybciej?


@Zoschi, how would you refactor this to look attractive? It looks like butt now.


ok


Yes. I should make that clearer.


@malirod, the main purpose is to complete log entry with `__LINE__`, `__FILE__` and `__FUNCTION__` here and pass it to `log_message_loop_thread` as single `std::string`. Otherwise we should add a lot of fields to `LogMessage` struct and I don't like this way


I don´t think this step is neccecary. robots.txt is selected from a dev/prod version via a route since ff7ea0e393635acd2bb7dd2d76da7e55f61885ef if the enviroment is set to 'production' and not 'local' or 'staging'


[CommCareRestorer.java](https://github.com/dimagi/commcare/blob/master/application/src/org/commcare/util/CommCareRestorer.java#L474) sets OTA restore GET params


It's problem to add 3 members? Currently the formatting is split. I see this as the drawback.


I think the basic idea is a good one. Generally, we often know the type parameters before the type. I think this needs to be generalized to all completers, though, not just the ones coming from namers.


This is the worst Travis config option, since they look at the value of this setting for the branch you are actually on, so you can add e.g. "only 1.0.x" to the 1.0.x branch and it will still run. Confuses the heck out of me.


to be consistent with the rest of the naming changes s/notification/alerts/ here


```erlang
{leo_storage,
     [{snmp_agent,"./snmp/snmpa_storage_0/LEO-STORAGE"},
      {queue_dir,"./work/queue"},
      {log_dir,"./log/erlang"},
      {log_level,1},
      {mdcr_stacking_procs,1},
      {mdcr_req_timeout,30000},
      {mdcr_stacking_timeout,30},
      {mdcr_size_of_stacked_objs,33554432},
      {stacking_timeout,5},
      {size_of_stacked_objs,33554432},
      {grp_level_2,[]},
      {mq_interval_between_batch_procs_reg,500},
      {mq_interval_between_batch_procs_max,3000},
      {mq_num_of_batch_process_reg,1600},
      {mq_num_of_batch_process_max,3000},
      {num_of_mq_procs,8},
      {mq_backend_db,bitcask},
      {threshold_active_size_ratio,60},
      {warn_active_size_ratio,70},
      {auto_compaction_interval,3600},
      {auto_compaction_parallel_procs,1},
      {auto_compaction_enabled,false},
      {dir_cont_expiration_time,250},
      {dir_cont_buffer_size,524288},
      {dir_db_path,"./avs/directory/"},
      {dir_db_name,leveldb},
      {dir_db_num_of_procs,1},
      {min_object_size_for_ec,262144},
      {erasure_code_lib,vandrs}, %% for the erasure-coding support
```


Ah, perfect. Then I will delete this part of the readme and we will just have to make sure that the portal is deployed with env set to "staging" until we release it officially.


This dose not compile with php7.


@viswanathamsantosh  Please consider moving this to README.md
If me move it to github.com/mozilla - this will be all the more important. Please look at [PDF.js](https://github.com/mozilla/pdf.js), one of Mozilla's most popular projects on GitHub for some inspiration!

cc @fayetandog


off


Last welcome page.
![screenshot_2015-12-11-10-22-19](https://cloud.githubusercontent.com/assets/5032371/11744712/5da5bbf0-9ff1-11e5-9ed2-b09912346934.png)


https://github.com/spring/spring/blob/51e30888cd02f81d8a38b9546b8e5994f5933660/rts/Rendering/GlobalRendering.cpp#L180

Now kindly come up with a robust way of identifying "those who deal with them OK", *without* assuming a driver tells the truth.


* It's just a minor thing but could you please position all dialogs at the screen center instead of the top left corner?

* The game crashes when attempting to load the 2nd TV level. Something menu related. Log: www.wdlmaster.de/TRCL/log20.html (scroll down to the very end)


Wonderful, thanks @jelledevleeschouwer - The module looks very good now. If only we could climb up a bit on unit test coverage I would definitely consider it for inclusion...


This imports not add map and interval to Observable prototype chain

Use this:
```
import 'rxjs/add/operators/map';
import 'rxjs/add/observable/interval'
```
but this files are out of Rx.d.ts :(

```
require(rxjs/add/operators/map');
require('rxjs/add/observable/interval)'
```


Color is math now


ok, I've changed `self.assertEqual(p.sub(x.varkeys["x"][0], 3), 9)` to `self.assertEqual(p.sub(x.key, 3), 9)` in a commit that I'll be pushing shortly; let me know if that was a bad idea.


hi


The coordinate system is adjustable per cmd basis and a few of them assume some "default" like global relative to home but others don't and must be instructed using MAV_FRAME, I'm normalizing it all over to always assume the same value - in that case global relative to home.


I don't think we should be passing this in, as ultimately we want to pass a empty map here by default and there's no matching field in an alert rule.


Would need to support this not being in the request (for existing apps)


Hi
On 11 Dec 2015 4:58 pm, "pAyDaAr" <notifications@github.com> wrote:

> hi
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SEEDTEAM/TeleSeed/commit/7e18b409606acab528ef1a122bfabe9927b9e5a1#commitcomment-14924863>
> .
>


should make this optinoal


TeamCity Civic Assembly :: Build backend + frontend [Build 77](http://54.229.176.143/tc/viewLog.html?buildId=173&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Yes?
On 11 Dec 2015 4:59 pm, "pAyDaAr" <notifications@github.com> wrote:

> :|
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SEEDTEAM/TeleSeed/commit/7e18b409606acab528ef1a122bfabe9927b9e5a1#commitcomment-14924881>
> .
>


+RDEPEND


Got it.

Let me ask one more stupid question, if you don't mind.
I saw a v3.10.87 merge commit in the history of this repo, (60ffbd7 Merge tag 'v3.10.87' into tw-5.0.x-lentislte) how did you manage the conflicts and some random commits that cause bootloop in G906?

Thanks in advance.


1) Should be added RDEPEND
2) Regenerate manifest


No, not now - at least, the user will normally want to test for a specific mode, like, I can't just takeoff in another mode except guided mode, so, are we in guided mode? no? well so set mode... but ok, a ```get_mode()``` call sounds fair...


if you pass `include_remote=False` to `get_apps_in_domain` you shouldn't need the type check


this looks like a good fix, thanks


1) Regenerate Manifest
2) Check RDEPENDS


I think that one should possibly be deleted.  I think strlen on a constant string is already constant folded.


Został tylko zapis do bazy, wieczorem zrobię go...


It's really early, so forgive the dumb question -- but why aren't we escaping all instances of `<>&`?


Remove this


not return value for main?


Ooo, that slipped through. It's not a restore param. I'll remove.


You know it's old code when your name is literally IN the code. ;-)


Modify repository name and add class util


I don't care as long as we keep the examples in sync with the object definitions ;)
(cf. https://github.com/ocpi/ocpi/blob/master/mod_locations_and_evses.md#41-location-object)


Isn't checking in the webhook URL a problem, as anyone could use/abuse it?


arghhhhh !!!111!!


1) Remove src_install from ebuild
2) Regenerate manifest


I can't think of any reason why - they are all distinct functions applied to the fields.  I use the scaled variant mask as an index into an array in various situations.


Agreed. [`drupal.make`](https://github.com/ding2/ding2/blob/master/drupal.make) also references 7.41.

Even if we updated it this part of the documentation easily becomes obsolete in general. I think we should change it so someting more generic.


Может быть, в описаниях коммитов стоит писать название кодера к которому относится? Их количество разрастается, возрастает и шанс запутаться, да и в ленте новостей читать удобней.
Этот выглядел бы ```sh.coder fix: variables; highlighting file.ext (if ext == word)```


This line has to be exist!
if not then we have exception:
PHP Fatal Error – yii\base\ErrorException
Class 'c006\utility\migration\controllers\Object' not found


"without assuming a driver tells the truth"
That's what I think should change, not the detection.

Everyone who's able to play spring without a fuss, should be able to play without a fuss.

Handling the issues with older cards is a better situation (same as we're dealing with https://springrts.com/phpbb/viewtopic.php?f=11&t=32986).


In a later pull request this has become a template function.  Possibly it should be renamed mergeStatsFromSource.


This line has to be exist!
if not then we have exception:
PHP Fatal Error – yii\base\ErrorException
Class 'c006\utility\migration\controllers\Controller' not found


note : Check les remédiations bash