Ahhhhh, I sorta thought it would be something like that if statement,
but I was trying to add it to the ArrayList<string> as an if-else or
something.....anything....and was failing horribly. Also, I was not
returning a new ArrayList.

Which brings me to ctl-shift-f, Thank the gods and you for cluing me
into that. Holy fuck that saves some major headaches after smashing
the shit out of my little bits of code then trying to get it all back
to tidy.

Also, I looked EVERYWHERE in the toolbar and could not find the
download javadoc section. Would not have looked in Maven even if i
found it.

Thanks for that as well.

CB...

On 9/11/15, jasonandersen <notifications@github.com> wrote:
> Also? The next time you're connected to decent interwebs, you can download
> Javadocs and source files. Right mouse click on the mahfugger project in
> Eclipse, go to Maven and Download JavaDocs and Download Sources. That will
> grab all the Javadocs and source code for all the Maven dependencies in the
> project.
>
> For Java classes, try the suggestions from this link:
> http://stackoverflow.com/questions/22642230/how-to-use-java-8-docs-in-eclipse
>
> <img width="579" alt="screen shot 2015-09-11 at 7 53 10 am"
> src="https://cloud.githubusercontent.com/assets/1189392/9817880/5ed1bc7e-585a-11e5-8f16-69f3efbaa3d7.png">
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jasonandersen/mahfugger/commit/e8b45d4f9c24569549907ba403a97aad9abf6d6a#commitcomment-13184896


Hi @carloslancha , What types are in the map? Please see the data attribute of <name>app-view-entry</name> in liferay-ui.tld as an example. Thanks. Jim


What happens with the following?

```javascript
var user1_ref = NAMESPACE['id'](1);
var user2_ref = NAMESPACE['id'](1);
user2_ref.close();
// What is the state of _all_ids here? What should it be?
```


Ah, fixed in 0eb2c6a21cdee576a4f6b48c132dacc2e80ac79e . Perfect.


@Nelspike @vanthome can we maybe accept an array of images instead of this many fallback images? to try them one by one?


What is the scope of `_all_ids`? Is this a good place for it?


Returning 'void' is a valid c++ statement. I favored taking this approach rather than having 2 separate lines with the return on the separate line as it looks cleaner. If the return tyoe ever changes the compiler will catch it.


Will Remove.  Needs to got to dev-wip1 branch instead.


Where is the end of this block? What happens where `NAMESPACE` is already defined by another included package, but does not yet contain our `id` "constructor"?


GAYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYY


@zorg93: I'm wondering, what was the reason to replace the ScriptHook "worldGetAll..." calls with actual low-level memory access code?


Je ne comprends pas.
En 0.85 justement je n'ai pas changé le nom qui est toujours $usenotepadrights.
Je viens de corriger pour la 0.90 l'appel de ce paramètre dans #102


ZONE was here!!!!!!!!!!!!!!!!!!!!!


I'll just fix this so we only convert once.


cade o executavel?


Damn, that's a good commit.


sarna?


@kench We're thinking that this move of the debs directory broke the build process (#57). Would this be better as a cp -r?


EXECUTÁVEL?


do the hokey pokey -smokey


cachorrinho da minina?


Hoffentlich schon gelöst mit mein ganzen Bird Commits.
Der letzt war https://github.com/FreifunkBremen/ansible/commit/dcfb4c6e0e25ce44faa2b2b77b0d865e5c0d26dd


DA MENINA?!?!?!?


Scrubbing out HTML from MODS display as per #816


mata o au au!!!


Yes, they do.


this breaks building uuidgen for :host
uuidgen is used during image generation.


rave?


Means I can do a position/model check before adding them to the entity pool and handle list


MATA A SARNENTA


RAVE!!!!!!!???


Try to avoid multiple acquire release by making sure thread.searching is set to false only if it could not latejoin.
Otherwise keep searching = true. This makes sure it is almost impossible for split() and idle_loop() to enter race conditions.
This is functionally equal to the minsplitdepth2_max_slaves7 patch except that the new logic should hopefully make splitting and latejoining much faster.


Corner-cases? Never heard of it...
![disorders-you-ve-never-heard-of68009075-aug-6-2012-1-600x400](https://cloud.githubusercontent.com/assets/1192958/9824398/2fb25266-58d5-11e5-9872-84953555194b.jpg)


Made a mistake, sorry!


MINHA SARNA?!?!?


QUEBROU... quebro a sarnenta


I noticed in the original report that the depth was up around 10,000. Is that level of ambition warranted (or practical, or useful)?


Adicionando os comentários.


I'm not saying that it's invalid or unsafe, just that it's less than optimally readable because it suggests a return value where there is none. It gives people more work to understand the code, for a mere line of code saved. Personally, I don't think the saved space is worth it.


This is great! If you have some time, explore the JavaScript `switch` statement for another way to do this.

Tiny comment: instead of using `string` as the argument name here, perhaps something like `operation` or anything more explicit would be better here.


develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Please see here: https://github.com/docker/distribution/blob/master/registry/storage/driver/base/base.go#L1. The extra embed shields the caller from being able to change `Driver.Base`.


:+1: Tiny comment, I would not use `string` as an argument or variable name because it could be confusing. Since string is a word that represents a JavaScript type, I'd rather use `str` or `inputString` for example.


Uninitialized values!
Any primitive type will be initialized with trash unless it's declared as global variable.
This happens because allocator doesn't erase memory before giving it to someone. It's someone's responsibility to clear acquired memory.
In the very beginning stack memory is filled with zeros. That's why global objects are initialized with zeros by default.


QUEBRO MEU CACHORRO


While this is a good solution, can you think of a simpler solution that does not modify `string`?


TIC

TIC is a cross platform scripting language, witch is mostly targeted for ti 84 and Casio calculators. It will also be able to run on your computer so that you will be able to test you scripts on your computer and also write and test them on your calculator


Fair enough. There are a few other code sites where I have done this. I'll patch them up.


It would be possible to merge this and previous line if **Node** has better construction options.


Oh my bad... Thanks BM!


4385-CondenserLoop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1324 of 1324 tests passed)

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


Please follow the style guidelines for Go comments. The comment should start with the name of the identifier. In this case, the sentence should start with "driver".


const Node* root


is this the only thing that is needed in this file? seems like the prior pattern was there to not stomp on things that are pre-existing


The custom model should be able to inherit the options Object from g5Model.

There are several base properties on the g5Model that need to be inherited.

```js
this.opts = assign({
    interval: 40000,
    enablePolling: true,
    path: ''
}, opts);

this.instance = false;
this.dataCache = {};
this.dataFetch = null;
```

You are correct though, in this case the custom model would inherit the properties from the g5Model however it would not be extended properly with the opts instance of the custom model.

I will update the documentation to include the opts Object.


// Allow the location of the ini file to be specified via a --config command line arg
You have outdated comment in the code :)


I think the tags should be included for the other MUST criteria.


Laravel follows semver, only offset so that semver major is laravel minor, and semver minor, is laravel patch, so it's ok to make BC breaks in patch releases. I'm not saying everyone should follow semver, because that's just silly. It doesn't make sense for everyone. :)

Yeh, I get what you're saying. My argument would just be to not bother with contracts if you don't think people will use them, or maybe mark them with `@internal` if you want to exclude them from the public api, thus telling people, if they want to use it, take extra care when upgrading.


Although, since there's only one field to serialize (remote_url), we wouldn't get an awful lot more structure... All the errors would be on the same field, only distinguishable by the message I attached to the error.


Looks good.


I'll take your word on it, seems perverse.


@danielpadula este es el tipo de chart en la estructura hacia atrás, quizá tengamos que cambiarlo. Donde saltaba el error?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 593](https://teamcity.labkey.org/viewLog.html?buildId=271584&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:21:56
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridColumnSelector: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudies: <no details avaliable>

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


yes, I came to the same conclusion. enough work for another day :)


81211464-LinkInfiltrationAndReturnFlows (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1865 of 1866 tests passed)

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


Course Projec1


81211464-LinkInfiltrationAndReturnFlows (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


:+1: Good! What happens if you pass something other than a number to your function?


Right now, yes. As I commented below, I'll move all the logic past 40 into an asynchronous call - which will have the benefit of not 500ing you.


"inputColumns" is strange naming for an output handle. Maybe just "columns"?


Great! Additionally, I'd like you to explore the usage of the `switch` statement to get the same thing done.


I may be wrong, but I think glyphs aren't drawn at the offset position which is why I asked about it. If you take a look at the image in #495, glyphs are tightly packed.


`toString()` should be at the end


retest this please


This field didn't change, leave it in the original position to minimize changes


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2306 of 2342 tests passed)

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


good catch!


Don't say -1 here.


Ok, the resulting functionality is close enough. In most cases it'll do the same thing (it does not use PATH). Renaming.


could this cause redirect loops when the OAuth provider redirects to the register page?


change indentation to be consistent with the rest of the file


This is just a hint to make to know if the generated artifacts are newer than their dependencies, since doxygen will always generate it all at once we don't need to specify every single file, also specifying the directory will not make sense cause between builds the folder will remain there - previously we were removing the folder in the end, now we don't do that anymore and the files remain there and are generated only if ```doxygen_resources``` are newer than ```doxygen_generated```.


Fibaro Wall Plug, now using multisensor for power reading

What give it for standard user ?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1328 of 1328 tests passed)

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


Seems like we were doing a None check on value before, do we still need that?


This method seems to only be called from one place, and it isn't clear why the type manager would sometimes be null from that callsite.

Maybe make it an `Optional`?

Invert this check and use `checkArgument`


Yeah bad style here... I'm using the computer at school without compiler and access to forge/mojang site... Sorry


Hmm, shouldn't we go even futher and remove unit-files from /etc which overrides stuff from /usr?


"structs" -> "struct"


This wrapping seems unecessary


Descriptive variable name might be nice. :P


Same about wrapping


Possibly add a comment if this maps to a particular version? e.g., # grpc beta release


`"null"` and `null` are two different things. One is a string with the word "null" and the other is a special JavaScript value that represents the absence of an object. In this case you could simply return `undefined` by doing `return;`.


Urgh, right. `m4` strips the first pair of square brackets. Will fix it.


Ok thanks, I'll keep that in mind from now on.


typo


Probably we should set up travis in this repository as well :wink:


why don't we need do pip install from the requirements file anymore?


Wrong name: `mapTypeInfo`


what is *this* magic?!


we still need a ```Requires: python3 >= 3.4``` since we're distributing python scripts.


Thank's for updating the doku!


For equality/inequality comparisons please always use the strict operators, that is, `===` and `!==`. The one you used here is a lenient operator, and it does not always compare things in a straightforward manner. For example, while `0 == ""` is `true`, `0 === ""` is definitely `false`.


unused import


Seems like some sort of caching, `getattr` doesn't appear to be hit when doing `os.stat`... ugh


Can you add a comment here to describe the intent of the log message?


The main 'pip install' does it already, not sure why I had it before...


Same here?


... yes, this would be good to fix


clean up this and below


This is good. I like the fact that you are doing defensive programming, always checking for error cases rather than simply complying to the input you are given.

One thing we will learn later on is about `throw`/`catch` in order to signal and handle errors in programs. If you learn about these things, you will be able to `throw` a real error rather than `return` a string with an error message in it.


Initialize maxSpatialLayers to 0 to use WebRTC defaults.  Only set it to 1 if simulcastUseCustomValues is true.


The code should not be commented out


Use `new ArrayList<>()`


Try to make your function `return` its value rather than log it to the console.


Fixed.


This should also work:

`function reverse(string) {`
`    var a = "";`
`    var b = string.length - 1;`
`    while (b >= 0) {`
`        a = a + string[b];`
`        b = b - 1;`
`    }`
    `return a;`
`}`


Make public


Please avoid using `string` and `number` as an argument names because it can be confusing. Even though JavaScript lets you do it, I would rather use `str` or `theString` or `inputString` to avoid mixing it up with the value type "string".


Fixed


do we really need to explicitly depend on lib?


Put in utils package?


This is the old syntax and discouraged now. Instead:

"except face.AbortionError as e:"

(from PEP 8, https://www.python.org/dev/peps/pep-0008/#id42 see the section that begins `When binding caught exceptions to a name, prefer the explicit name binding syntax added in Python 2.6:`)


Why adding one more param `all_pass`? Just call `__check_all_conditions(check_names)` and `not __check_all_conditions(check_names)`!


Wow this is so great! Thank you, @timaschew. Kudos to you!


Yeeee....sss, it's very tricky. :-)

In this particular case you don't want to do that. You actually do want to do `RecordImmatureSilo` **before** calling `NotifyListeners`, since:
a) `RecordImmatureSilo` changes the `immatureSilos` collection
b) `NotifyListeners` will cause the listeners to go back and ask `GetQueues` and the result of `GetQueues` depends on `immatureSilos`.

I would go ahead and claim, based solely on my subjective and individual opinion, that lazy evaluation of operations with external effects is dangerous and should be avoided for any statefull computation (and this class is sure statefull and `RecordImmatureSilo` definitely has external side effects - it modifies `immatureSilos` collection).


`return self.WrapperObject()` is enough.


Traducción


nImgs in line 37 is not defined anywhere. nTest seems to be intended.


This does not work as expected... It's returning me the same string I pass it as input.


It's still in master?! 👨🏻


NaN will appear


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Good!


Interesting.


I'd suggest something like:

``` js
var paths = {
  rooms: '/rooms/',
  messages: '/messages/',
  members: '/members/'
};
```


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1606


Mutable argument (class)


93653360-ResetVarsAtEndOfWarmup (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (775 of 775 tests passed)

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


Temos um bug aqui, vamos corrigi-lo.


Probably a good idea to use the word "Error" in the message itself, may be confusing with just the error code.

Also, know that `console.log` is good for debugging, but shouldn't be used in production code unless you provide a polyfill for older browsers and some mobile ones:

``` js
if(!window.console){ window.console = {log: function(){} }; }
```


@anvaka maybe I need to debug it again,  but I was persistently getting `node.data` being undefined until after the event.
Perhaps it was something else.


Is there a reason you decided to use $timeout here? I think it would still work without delaying the execution.


let's go by object here...


Matthias:
Rule number one do not mess with other peoples copyright:
https://github.com/outscale/packetgraph/commit/da837d0dd7380c33ed0be54eb301ddd677f5170e#diff-4d9e988ab9104df0b28d7f7db894c6d5R51

You can add your own on vtep.c since you have written a good part of it but it's unfair to just move thing around and change copyright.


You could probably just do `return <RoomNode name={room.name}/>;`


Did not see this program (patients/message_links.php) included with this commit


Fixed, thanks!


Boa noite professor, o Sr. poderia dar uma olhada no meu.


@chrisdone `stack-mode-back-enqueue` is undefined, probably you mean `stack-mode-back-queue`?


Hello @cheevahagadog (Nathan) :
cc:  @jonhanke  @julialintern

Great job on these exercises.  The examples you provided are very good and comprehensive!
Q7 - check last two functions

Answers for Q5 (datetime):

a)  937
b)  513
c)  7850


I wonder whether `asComponent` could be a good name:

``` js
this.state.rooms.map(asComponent);
```

And actually, I wonder whether this piece of code should be extracted to `RoomNode`, and made available as a property of the class itself, renamed `RoomNode.fromObject` for instance:

``` js
this.state.rooms.map(RoomNode.fromObject)
```

Dunno. Just thinking out loud.


Please let me know if you find the root cause.


Perfect. You got Flux right!


Ic.

I didn't look deeper into that.


wtf


Perhaps this should be `alda.test-helpers`?


Ok, I got it.


fast-forwarding develop to borshop-integration-1246-bugfix/update-src-ct-on-copy = 4f47671c


php配置文件路径


I think the idea of setting a flag to stop the modal from opening is a good idea, but this isn't the right flag for it. $scope.isSaveInProgress is set to true when an actual save is happening (when the 'Save Draft' button in the modal is clicked). The flag changes the button to say 'Saving...' until the backend replies with whether or not the exploration was saved successfully.

Consider creating a new flag specifically to stop the modal from opening a second time.


Travis-kun is sure bit picky with these :laughing:
the current 19.2, is it Webstore already or just the Dev?


Pay attention on convention.


Why do you need the `Global attribute` (`$` sign)?


This should be updated to use the new flag. However, I'm not sure the button actually needs to be disabled. You will need to remember to re-enable the button if the modal closes or if a save happens. Perhaps we can just avoid opening the modal a second time in saveChanges() rather than disabling the button.


ack


`(self.name.downcase.start_with?("import"))? true:false`
Could be better with:
`(self.name.downcase.start_with?("import"))?`


why do we have TWO strings that are the same?


Ditto.


We shouldn't be concerned with $scope.isSaveInProgress. Instead, we should use a separate variable. The code currently handles $scope.isSaveInProgress after the modal is closed and the user requests a save.


unlike most empty things in js, an empty array is truthy


如果能在loop（）里面随时关闭所有schdeule task就好了。
其中你提到Sch.DeleteTask(任务ID) 删除任务没太看懂，将addtask任务ID的数据形式是int还是char？


```
[modify] Edit file product.rb
Add method description_v1 for nice display
AnhNV
```
===>
```
[modify] Add method description_v1 for nice display
```


컨텐츠 문서 하위 폴더로 이동


imho it could be its own language so we lessen the workaround in terms of extending english, etc


Unfortunately, because of our options SNAFU, we can't specify a settings module for tests, so have to dynamically change it here.


Hello @oksitosin  :
cc:  @jonhanke  @julialintern

Great job on these exercises.  The examples you provided are very good and comprehensive, especially for Q1-Q4!

Answers for Q5 (datetime):

a)  937
b)  513
c)  7850


```
if name.eql?""
			name ="(Unknown)"
		else
			name
		end
```
===>
`name.eql?""?"(Unknown)":name`


Is this always unavailable, or just on the build server?


Just on the build server. I deleted my local copy of the video to test the test.


@thiagofm This is a workaround for the issue I posted at  https://github.com/memcached-manager/memcached-manager/issues/9

This workaround works for me with Rails 4.2 but I don't know if it breaks anything, that's why I didn't make a pull request.


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1830 of 1840 tests passed)

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


what does miscellaneous mean in this case? I;m thinking about trailing arguments as inl
<pre>
task pro:foo +bar myprogram -f example.txt
</pre>
the "pro:foo +bar" being $FILTER, "-f example.txt" being trailing arguments..?


Yep, now that Genesis 2.2.2 is out, I can focus on pushing out the update for the sample child theme.


Sgtm.


Just trying it out.  If the user is on an Admin or settings screen or when the main is not displaying a room - the tabs still works; but the fly-ins are just blanks :smile:


hallo hallo tschuss tschuss


I'm not sure what the repercussions will be of this for Rails 3 users (if there are any).  I haven't looked at this code in years and am not interested in maintaining it. Would you mind forking the gem?


Nit: Should say "if the least required number of choices"


Where is input file?


I was like (# ﾟДﾟ) when Travis was complaining my new translation broke the building.


Good_Input?
Is Good_output?


`Good.new("Book of life",10,10)`
===>
`Good.new("Book of life", 10, 10)`


okay. thank you :)


This is good for once we get to versioned files, but part of the RFC acceptance criteria is that existing asset stores work without migration / renaming. So we need an optional (default on?) way to say "use the regular asset (i.e. no hash) filename".


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Unless I'm mistaken, I think you forgot to insert the comment. ;)


It isn't obvious just from code docs or naming how AssetContainer differs from AssetStore


Don't like that AssetContainer and AssetStore both extend AssetWriter - same method signature isn't the same thing as same type.


Probably could be in SilverStripe\Filesystem\StorageAdaptor or something


The problem is that for Cordova Android v4 they are now recommending a new url whitelisting scheme which involves specifying a CSP meta tag. This made many people realize that some plugins where still using the deprecated sendJavascript and evalJs in Android and iOS respectively which result in an eval() call in Javascript which in turn violates any reasonably strict content security policy (no one wants to explicitly allow eval() in their scripts).

Are you planning to address this on iOS also? What's the timeline on this? This is a fairly high-priority issue for me so I'd be happy to help in case you need it.

On Fri, Sep 11, 2015 at 4:56 AM, Tim Lancina <notifications@github.com>
wrote:

> Ah I didn't realize sendJavascript was blocking for anyone.  Didn't merge this in because it wasn't that well tested and everything seemed to be fine with the old (albeit deprecated) solution. I'll take a look tomorrow about getting this in.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/driftyco/ionic-plugin-keyboard/commit/0a1165af7d4aa248ddbd218b60a96961090d2eb4#commitcomment-13173870


I think this is a complicated way to achieve an 'else if' context. You could remove the '&& !(numChoices < minAllowedCount)' part, move this if block under the block checking for 'numChoices < minAllowedCount' and make it an 'else if' statement instead. Once done, I think the comment should either be omitted or updated to describe why only one of the two errors can show up.


我日


okay. thank you :dancer:


Last decent layout


```
if name.strip.length == 0
			name = "Unknown"
		end
		name
```
Could better with:
`name.strip.length == 0 ? "Unknown" : name`


What is the harm of putting `calc_d_tick` inside `calc_d_homer_tick`?  I moved it there in [Shorten calc_d_tick](/dxx-rebirth/dxx-rebirth/commit/d4c2152009620d4ffce3adecdc24e6a0e4c42b1c) because it presented a nice tail-call optimization opportunity, and it looked like the code would get the same result either way.  They both depend on `FrameTime`, but that is not updated between the two calls, so they can be called in either order.


Sure, a legacy_filename config perhaps?


I think that the line "target_path": "device/samsung/qcom/common" mistake, gix on qcom-common


fix*


Hello @sosier
cc:  @jonhanke  @julialintern

Great job on these exercises.  The examples you provided are very good and comprehensive!
Q4 - Wow, that table in markdown is impressive!

Answers for Q5 (datetime):

a)  937
b)  513
c)  7850


`filter_gender` => `filter_people_by_gender`


Professor, Regra atualizada


Check this commit guide line: https://github.com/AsianTechInc/AST-ruby-code-review/issues/6


Fluent for any reason? to match stringbuilder?


it's fixed now just by the version number XD
Travis-kun wanted `19.2.0` instead of `19.2`


Why do you need `Global attribute` (`$` sign)?


Implementación de Software/03 - Construcción de Software/3.2 - Scripts SQL/bdkarisma.sql

Se borró por la ciencia.


Se elimino un espacio y está linea subió una


Revisado


Yeah that's exactly what I've tried. StanHeaders and rstan seem to install
fine but I still get an error about inv_Phi not being found when trying to
build rstanarm.

On Thu, Sep 10, 2015 at 7:59 PM, bgoodri <notifications@github.com> wrote:

> Probably just do a
>
> git checkout develop
> git pull -ff
>
> in the rstan clone and then R CMD INSTALL StanHeaders/
>
> On Thu, Sep 10, 2015 at 7:56 PM, Jonah Gabry <notifications@github.com>
> wrote:
>
> > @bgoodri <https://github.com/bgoodri> What do I have to do to install
> the
> > versions of rstan/StanHeaders with inv_Phi working?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstanarm/commit/133c7e355b9b58d2ba4843b746799a37e5f6090d#commitcomment-13172770
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/133c7e355b9b58d2ba4843b746799a37e5f6090d#commitcomment-13172795>
> .
>


```
if !(is_free_tax?)
			number_tax += 10
		end
		if is_inport?
			number_tax += 5
		end
```
===>
```
number_tax += 10 unless is_free_tax?
number_tax += 5 if is_inport?
```


93653360-ResetVarsAtEndOfWarmup (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1308 of 1308 tests passed)

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


thanks for the link fix @machikoyasuda


For context: we discussed offline different ways of reorganizing the tests to better illustrate the behavior of the errors. In the process, we realized that two errors would always be issued in the case of numChoices < minSelectionCount. To address this, we decided to update the validator to only allow one of the errors to be outputted at one time. This also allows them to be tested in isolation (which addresses the original comment in this thread).


professor o meu estar com algum erro?.


That's right, but we should avoid the `!important` clause, this will prevent to overwrite the font from any place, so maybe we should find another way to overwrite the font definition, maybe setting specificity to the selector.


Professor, Regra atualizada com a 21 que é a minha


I didn't know how to make one to sourceforge...
On Sep 10, 2015 10:36 AM, "Holger Joest" <notifications@github.com> wrote:

> Do you intend to create a pull request?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kedzie/autotools-maven-plugin/commit/d9cb0f2d7b9d1e54c3ed7e2e2765da4ba3e8f872#commitcomment-13162874>
> .
>


`filter_age` => `filter_age_lager_than_18`
`filter_age`` => `filter_age_less_than_18`
18 should be in a `CONSTANT`


啪啪


Making the confused jokes is not a joke but not making any jokes is not fun here. :3


Hmmm, this is interesting. I only test redlock in node v0.12 and v4.0 now, but I see that they made breaking changes to the "events" standard library since v0.10, which is what I assume you're running. I'll get this fixed and add v0.10 to travis.


I had tested this case, but when i removed '$' sign, my code will take a error: "  undefined local variable or method " on all methods having this variable.
I think that if  it don't use '$' sign, i can't use it within all my methods.  I had tried it.


I think not for now, we will tell them this when they get to the frameworks topics, maybe there would be a nice idea :B hahahaha


Yeah. Don't feel super strongly about it though.


Pay attention at `coding convention`


Also if you only provide one format please add a text source inside the `<audio>` or `<video>` tags, so when the browser do not support this kind of format the text will be helpful. :)


So, try to find a OOP solution.


GroundDomainUpdate (mmatts) - x86_64-MacOS-10.9-clang: OK (1840 of 1845 tests passed)

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


Consider this: http://ruby-doc.org/core-2.2.0/Array.html#method-i-index


```
		tax_free = 0
		if is_free_tax? == false
			tax_free += 10
		end
		if is_import_tax? == true
			tax_free += 5
		end
		tax_free
```
===>
```
number_tax += 10 unless is_free_tax?
number_tax += 5 if is_inport?
```


That hasn't happened. Thanks for letting me know! I'm looking into better ways to deal with cases like this as deleting things off req is not ideal. I'll use this as a case for researching an improved solution.


Travis-kun is kuudere today, not tsun?


`is_free_tax? == false` => `is_free_tax?`


Перевыложил


:+1: Great! As a challenge, can you try to do this exercise without any kind of loop?


`function reverse(word) {`
`    var a = "";`
`    var b = word.length - 1;`
`    while (b >= 0) {`
`        a = a + word[b];`
`        b = b - 1;`
`    }`
    `return a;`
`}`


@zamar, I've submitted a new test to really speedup splitting and latejoing(if it works!).  I think it should also reduce any hang scenario:
http://tests.stockfishchess.org/tests/view/55f32bce0ebc5976a2d6db5f
If this doesn't work, then we have to rely on @Louiszulli's tests to narrow down on what causes the hang.


Ah! I forgot about this one.

case "add":
return numberOne + numberTwo,
break,

etc, etc...


this module depends on http_client which depends on curl, so we should explicitly Requires in it here.


Tu veux que je remplace "alors, arrêtez de..." par "ensuite, vous pouvez arrêter de..., pousser..." ?


=(. The FIXME is fine, but let's at least not raise a generic Exception... create a new class for this file, something generic like TODOException is fine with me.


Dans l'exemple que tu donnes, est-ce que l'environnement `verse*` sert à quelque chose ? Je n'ai pas l'impression…


Does this work with non-grpc update streams?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 594](https://teamcity.labkey.org/viewLog.html?buildId=271589&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 31, passed: 9 Build time: 00:23:38
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

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


No, as OAuth provider always adds a parameter ?oauth=(...)


Thank you!


@hvelarde eu não removi o iw.debug poque achei que alguém poderia estar utilizando. Mas se você acha melhor remover o iw.debug, vá em frente :smile:


outside the scope of this change, but it's bad that we have to manually close all these. this should really be a context manager that handles the closing for us.

`with blah as replica_conn` is much nicer syntax.


same as for curl dependency here.


requires libudev.


C-s C-l jumps to the last window by default. So maybe this shortcut isn't needed?


Corrigé : cf36fbbd9c4e212b1c5d5d06e92b50ca3ea43a22


yes it's part of the API (already a few close in this fine, a few were missing)


Agreed.


requires dbus?


You could have solved this with a single line:

    return [charactersArray valueForKeyPath:@"favorite drink"];


I once did this with a file containing a DB password. no bueno.


all_pass changes the method behavior, as __check_all_conditions used in Wait and WaitNot with different behavior, we requres a key to control it.


Woohoo!


cut&paste of old code, I'll fix the source too.


drop a comment here on why we're disabling doc package.


It differs in return **boolean** value! So why not revert the value after the return? :)


Read the README.


You raise a good point. Perhaps Manifests and Documents are different but both useful -- manifests can have a model, but not documents, but in Solr a manifest is a field on a document.

Perhaps you can come up with a non-database model that treats a Solr document like a traditional DB model.


Agree, fixed.


Fine, then. Tested and looking good, +1.


Initial commit


Just read up on this. Sounds cool Let us know if you get it to work!


Thanks!

Dave

> On Sep 11, 2015, at 3:47 PM, Scott Chamberlain <notifications@github.com> wrote:
>
> added @davharris as author
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ropensci/traits/commit/a321c21e95ac2a8a494dde45005f27e89324b278


@ButchersBoy why not here also `CurrentUICulture`?


maybe ```%{name}-%{release}``` ? or maybe introduce a new variable just to expand it elsewhere (cause release is **v**1_beta4)?


yeah, I see it in master... check line 12...


patch looks good, just minor comments.


Fixed.


Pour info, j'ai mis en place des tests pour l'analyse du chordpro et la conversion en LaTeX. Une manière de vérifier si ce n'est pas cassé est de lancer `python3 -m unittest` à la racine du projet.

Les tests sont dans le répertoire [patacrep/songs/chordpro/test](https://github.com/patacrep/patacrep/tree/cf36fbbd9c4e212b1c5d5d06e92b50ca3ea43a22/patacrep/songs/chordpro/test).


`child processes`? Also, it's only Facter 3.0 versions that were affected by this.


This has to be changed, probably overwritten in the app javascript if possible?


i integrated the enable and disable features into the apartment scene in my previous commit


Si c'était ça le sens originel de la phrase, alors la formulation en anglais aurait été "ready to be pushed" à mon avis, tu ne crois pas ?


<div main>
</div>


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: Build Failed

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


You're right


Should this be a REGRESSION FIX? Only affected Facter 3.


Nice password


@josevalim the only downside I see is that we call `PO.dump/1` here and later we call that again. I'm not worried about performance but I want to hear what you think first. I did this for simplicity, and btw I'm checking equality of dumped contents instead of `%PO{}` structs because structs can be different for silly reasons like a different `:po_source_line` in a translation. Wdyt?


I would avoid branching in this case; treat no 'q' parameter as an error, but don't have a separate case for the "main" functionality. This helps reduce the levels of nested blocks, and you don't end up with incredibly large if statements for the correct (i.e., non-error condition) behaviour.

So:
```
class SearchView(generics.GenericAPIView):
    def get(self, request, *args, **kwargs):
        q = request.GET.get('q', None)

        if not q:
            return Response({'error': 'Did not provide query (q).'},
                            status=status.HTTP_400_BAD_REQUEST)

        return Response({'q': request.GET.get('q')})
```


fixes #2 most probably


This one's tricky. We also introduced a new feature, reporting all the bound IP addresses in `networking.<interface>.{bindings,bindings6}`, but we didn't file a separate `New Feature` ticket. @peterhuene


cream


This doesn't seem like the right ticket? Also, this is tricky as we're not really claiming AIX support yet. @kylog


It is best practice to name function parameters with descriptive names which are not variable types. For instance, "value" would be a good replacement for "string".


@navv1234 this fixes the test problems, as described
@CaseCarter Do you have any idea, why this commit compiles & runs fine, but reverting it lead to the build failing as seen here https://travis-ci.org/simulationcraft/simc/builds/79903225

It looks like something very weird is going on with timespan_t


Well, but on the other call site, it is a list with multiple elements (one per user that hands in together).

I'll leave this as it is for now, maybe change it when (if) I later update that part of the code to communicate with Discourse.


81211464-LinkInfiltrationAndReturnFlows (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (796 of 796 tests passed)

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


Should be version 1.2.4.


What would happen in the case of firstCharacter(12345) ?


is the `n` argument required here?


OK. I will begin doing that from now on. Thanks.


Week 1
Tip 1: Be comfortable...!!Be cool
#Thought of Software Engineering course is about coding but It is very Interesting and challenging.
#Activity done in class. To create a game (candy crush),we can  design a game or even better.
This helps me to understand that user and developer are different.
Week 2
Tip 2:Think broad but do as specific [precise] as possible
#Life cycle of Software
#Readings of Gladden G.R."Stop the life-cycle I want to get off".
Week 3
#Software requirements
#Importance of case-diagrams.


What would happen in the case of lastCharacter(12345) ?


Oh wow! There is so much defensive programming going on in here! I like that :)

Tiny comment: I wouldn't use the identifier `string` as an argument name to my functions. It sounds confusing even though it's not a reserved word. Perhaps just `str` or `inputString`.


Since you already checked with `isNan`, do you really need `Number(...)` here?


I'm fine with opening a new feature ticket describing the change to the `networking` fact to report all address bindings on all network interfaces.  FACT-1150 was still fixed as described.


change class name to ArithmaticOperations


the actual change is on line 708


I would get the "undefined".


Should this perhaps be in pkg/api/unversioned somewhere?


Individual project:
Name: Exam portal
I'm Interested to do exam portal. In this portal all the lectures can upload the quiz papers or any online instant contents to the students. So that every student of the Institute can register in this portal. The questions will be randomized to minimize the copy cases. The quiz papers can be uploaded many days before the exam but students can view it in a particular time slot decided by the professor. This portal is done by using Intranet. The clubs within the campus can use this portal to much extent.


Got it.


If you change this to a regular DRF view, then you can probably get rid of the POST handler altogether.

(Also, assuming you change this to the Manifest view)


Will do - I just slapped this together in a couple minutes to get the response going, but normally I would structure it in the way you suggest.


I would get "undefined"


That's fine -- just trying to be helpful :)


is `Number(...)` really needed here?


I see.


Nice use of the `switch` statement here. However, note that if you are `return`ing from inside a `switch`, you do not need to add a `break`. The `return` will break out of the `switch` immediately.


EvapCoolingSizingFix (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1853 of 1855 tests passed)

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


Yay :)


nice re-use of functions here :)


Вы все поняли правильно, по отдельности, но как-то неправильно соединили в общую картину :) Почитайте еще, статьи какие на хабре или даже хакере.


Maybe use: https://github.com/webpack/css-loader?


Like the class config. Was thinking of similar like a DOM config value so that a config variable could have children.


From workshop exercise: "The indexing of number should start at 0."
The returned value of tellMeTheCharacter("Hello", 0) should be "H".


Yes i do, otherwise, if i add "2" + "10" i'll get "210" rather than 12.


Ack.


time & fmt can be in the same block-- thanks for noticing this was in the wrong spot.


I saw it on official fedora repo .specs, I'll double check.


Ack, I'll do it a grep to check for others.


Fair.


это было раньше, заглушка


Ok.


>Но получается что если мы будем отображать такие строки в браузере, нам нужно при выводе их экранировать.

Да, все так и есть, нужно перед отдачей пользователю их экранировать.

>И мой вопрос был про использование JavaScript для экранирования символов прямо на странице.

Вы не сможете с помощью JavaScript экранировать код на JavaScript :) Поэтому только на серверной стороне и только с помощью Java.


Thanks a lot.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2307 of 2342 tests passed)

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


Thanks @kruus, fixed in https://github.com/facebook/fbcunn/pull/83


Why stop there why not just factory reset entire /etc?
 Ask yourself this if an administrator is already in the state he needs to reset his entire type unit files he's already in the state he needs to reset entire /etc right?


I've opened [FACT-1233](https://tickets.puppetlabs.com/browse/FACT-1233) to track the new feature.  It needs to be added to the docs.


Right now, I'm using ReactCSS to inline everything because I'm horrible. Since I'm using Material UI, I don't think I need Foundation or any other CSS framework. Also, I don't even want to attempt using Webpack in a Meteor project.

https://github.com/keokilee/grader/blob/teacher-login/client/components/teachers/login.jsx


Also updated how buildTimeSeries handles field names


Correct :)


It is best practice to name function parameters with descriptive names which are not variable types. For instance, "value" would be a good replacement for "string".


This got merged already, I became late. :) But a further observation, if you allow: if it is important to run the tasks got from ``RecordImmatureSilo(silo)`` before ``NotifyListeners().Ignore();``, shouldn't they be awaited before ``if (!isStarting)``? From this piece -- I haven't looked into ``RecordImmatureSilo`` I get the impression hot tasks are returned, but they might not have run to completion and hence done what they should do.

Anyway, the collection is lazy until materialized, but it can be materialized. This alternative formulation may appease someone's eye, hence putting forth a suggestion. :)


This is my only question in the whole PR. I think we want to get to the point where the unversioned types are registered a single time, and added to swagger in a single place, instead of re-registering them in every version. I imagine things break if you leave this line out? If so, we can handle this in a separate PR?


this scope orders the list by time created and takes in input of time. The arguments are where the time is less than time( in this case I put Post.ordered_by _reverse_created_at(Time.zone.now) which is the current time of the zone.
Unfortunately the time created for all the post are the same ( in our case). I ll chage it to order ('title ASC)


Good! Could you simplify this function if I told you that I don't need separators between the strings?


Thanks Phil, it's nice to have a second pair of eyes spotting these things!
On Sep 11, 2015 3:29 PM, "Philip Durbin" <notifications@github.com> wrote:

> Ah, fixed in 0eb2c6a
> <https://github.com/izahn/workshops/commit/0eb2c6a21cdee576a4f6b48c132dacc2e80ac79e>
> . Perfect.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/izahn/workshops/commit/74b459c12dc9c0514e49402982e2f75e55024ac5#commitcomment-13190266>
> .
>


While this is logically correct, in this case I would prefer to test for `i >= 0` rather than `i > -1`. It's clearer what you are trying to do that way :)


101884090--NewVenting-Availability-Schedule (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1857 of 1857 tests passed)

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


@aweiksnar I just noticed this request firing every time I viewed a discussion.  Can we defer these requests until moderation is open?  It's a pretty heavy request to be running on every page load.  It also fires even if the user doesn't have access to that panel.


It's building fine for me.

On Fri, Sep 11, 2015, 04:44 lethalman <notifications@github.com> wrote:

> Broke i686 build.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/4f001836856498717f5216eaa9f1c86334c708d7#commitcomment-13180863>
> .
>


Good recursive usage, we did not see this topic yet I think. :100:


`routerset_contains` is failing because `options->ExcludeExitNodes` was not assigned to NULL but pointed to a invalid address
This makes me doubting whether `tor_malloc_zero` is enough to prepare a struct which has pointer properties.


Yep, it got removed at the top of the file.


Sweetness. For the future: maybe we should save a hash of the tar.gz on archive, so we can check for integrity when unarchiving.


Yeah good idea - I'll open up an issue so I don't forget


![tblgestores- benjamin](https://cloud.githubusercontent.com/assets/14224454/9825745/0f29347c-589b-11e5-8a4d-2317411d14ed.PNG)


This should probably be reverted since this old version of qemu is
vulnerable to a number of exploits and the segfaults were likely caused by
the build machines themselves.

On Mon, Sep 7, 2015, 03:22 Domen Kožar <notifications@github.com> wrote:

> Reverted in master and release-15.09
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/0e0e3c0c08b77a64fc934795dc5d9647fbd58d42#commitcomment-13093974>
> .
>


for the dtype, always use the output dtype. When both inputs aren't of the same dtype, as this is elemwise op, the output will be the upcast. This is already avaiable in the output dtype. it is same to use the output dtype, but not the outputs[0].shape, as this keep the output variable in the graph.

idem elsewhere.


Зачем сканировать весь базовый пакет в котором еще куча всего? Выдели сущности в отдельный подпакет и сканируй только его.


So would I :D

Anyway, my new avatar is cool, isn't it?
Yeah, I love puns XP


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

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


Зачем update? У нас liquibase все должен подготовить. Я использую validate обычно.


really? I remember doing that earlier... thanks


oh maybe that was templates....


:cake:


That last sentence should be a margin note.

Also, I'm not 100% happy that the docs for `require/typed` (a pretty basic TR thing) now have to dive deep into units. Maybe just have a stub here, and put the bulk of the docs in the units section?


I know, and I appreciate it. I'm pretty sure you taught me this exact technique a few months ago... I was trying to communicate I hadn't forgotten/disregarded it - apologies if it came off as dismissive!


@shockone If your able to find good sources about the correct behavior of scrolling margins - mind to share it? My scrolling margin implementation is all wrong atm and it is hard to grasp from xterm sources, it is scattered all over their buffer handling.


Желательно возвращать интерфейс. Например, `PlatformTransactionManager` в данном случае.


There is no ⊊ operation defined by the standard, but you can of course implement one. It's meaning should be subset-not-equal instead of interior.

I like the ⪽ symbol for interior, although this symbol is missing in most fonts.

[entire] is an interior set to itself (as is [empty]), so the check `b.lo < a.lo && a.hi < b.hi` must be improved. The test generator should give test cases for all special combinations with empty and unbound intervals.


no problem ;)


Bug type : Objectives take 10 seconds or more to pop up
Location : Tutorial island
Reproduction : Right click on an npc whose task requires you to, it will take approximately 10 seconds until the task pops in chat.
Additional notes : There is no lag present on either ends, so this must mean there is some kind of code that is making the quests longer to complete.


                                                                                Ok                                                                                Sent from my BlackBerry 10 smartphone.                                                                                From: rajatjindal83Sent: Saturday 12 September 2015 01:54To: sulabh1201/Java-ProjectsReply To: sulabh1201/Java-ProjectsSubject: Re: [Java-Projects] Second Commit (fec3a80)change class name to ArithmaticOperations

—Reply to this email directly or view it on GitHub.


Well, these are more than just test beds but actually used in production. I
see no reason why this is any different from chromiumDev or ghcHead. While
these are not stable releases the codebase passes through quite rigorous CI
and is on par with the quality of chromiumDev. Unlike s significant number
of other git expressions in the repo, this is also updated regularly.

On Thu, Sep 3, 2015, 02:17 Eelco Dolstra <notifications@github.com> wrote:

> I don't think such test beds belong in Nixpkgs. You can easily keep
> development versions in your own tree.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/05d7fb205474843e62694a15539fcf97a7f76a8b#commitcomment-13042596>
> .
>


Really?


I installed uuid-dev on Ubuntu and the build continues with out issue


double space


It seems we need it, it stays.


Отлично, что ты в коммите все подробно описал. Благодаря этому сразу видно, что это неправильный фикс. Надо разобраться почему у тебя такое происходит -- liquibase не должен пытаться выполнять свои миграции несколько раз. Возможно, что дело как раз в том самом `update`-режиме Hibernate, который может выполняется до liquibase-а. Если это так, то нужно разбираться с приоритетами исполнения ну и убрать update режим.


do you you have an idea of how to do this in a nicer way? i.e., loading up each of the labels and combining them into a single object? this is super ugly...


Ahh okay
What did you mean by "shouldn't we just avoid having"?


This ties in very closely to the "problem" slide, I wonder if they could be combined into a sort of "but reviewing online means you miss out on..." slide. I think the point about recording decisions with reasons afterwards might also work well as part of this.


all right


Haha xD I noticed. The picture is much better :D


I checked one by one, but if you want to double check go ahead.


Done


I honestly cannot remember


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1328 of 1328 tests passed)

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


previously it was doing a bit too much


be sure to actually test the builds.  Could well be that some required packages were in the gold image and not installed by the salt tasks.


FYI, this change is now live in chef.

Although, do we need our own copy of this file at all, after https://github.com/chef/omnibus-software/commit/a714be43a973a3b07f5cf7ef23f2c43e670e3536 ?


What happens if I do: addition("a",2); ?


Yes, understood, and I saw.  Didn't want to make any assumptions on your intent.  Just noting the failure point with that scenario.  I think the customization great, glad to see it as an option.  Thanks @gbabula.


Hmm, a string drawn with al_draw_glyph and al_get_glyph_advance /should/ look the same as the same string drawn with al_draw_text, provided the font wasn't loaded with kerning disabled... But now that you mention it I'm not 100% sure that this is always the case for ttf text. BMP text should be fine now after the bugfix I made.... Apart from fixing your new suggestions I'll make sutre this is the case.


What happens if I do multiplication ("a", 2)?


Yes, but ````roomsize```` can be modulated, and thus can be a UGen. Then ````roomsize + 1```` will be wrong. Here an example:

````GVerb.ar(roomsize: MouseX.kr(1, 5))````

If your cursor happens to be on the left end of the screen, you will not be able to change the size to 5 while it's running, because maxroomsize was initialised on the basis of what that value happened to be at that moment.


#359 changed the code to eliminate direct recursion, but that could have other consequences. With version 3.4.0, people have the option to use deferred updates to do something similar. We haven't had any other complaints about stack overflows, so I don't feel too concerned.


Issue#4425BlankTubeLength (wfbuhl) - x86_64-MacOS-10.9-clang: OK (1838 of 1855 tests passed)

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


Use return instead of console.log


FYI dit is een 302, moet 301 zijn. Opgemerkt door marketing en fixed ondertussen ...


решения не нашёл как объединить, в массив запихнуть не прокатывает


This is a perfectly reasonable to do this but if you want to collapse it to a one-liner, `mkdir -p` has the nice property of both creating any missing parent directories *and* not raising an error if the target directory already exists.


Thanks for catching that Terrence, appreciate it.


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1857 of 1857 tests passed)

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


Oui, c'est ce que je propose. Ici "read" est la forme prétérit. La deuxième partie de la phrase fait bizarre en français.


Replaces tabs with spaces


Thanks. More good stuff to come....

Sent from my mobile device (I'll be brief)

> On Sep 11, 2015, at 4:27 PM, Sean Óg Crudden <notifications@github.com> wrote:
>
> Like the class config. Was thinking of similar like a DOM config value so that a config variable could have children.
>
> —
> Reply to this email directly or view it on GitHub.
>


Thanks!


"allows only specification of"
s/, v/. V/


not a big deal today, since the 0.4.5 has the latest instrumentation code.
however it will be a hassle to keep this updated.


s/in the//1
Mention it's only a limitation of the typed version.


non, c'est une élision comme "prêt à porter" pour un vêtement. Mais, ça marche mal en français.


Любые операции, которые изменяют данные не должны привязаны к GET-запросам. Для них есть POST/PUT/DELETE.

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


s/Racket/untyped Racket/


Also, I don't think that sentence is necessary.


:cake:


If you do keep it, thought, s@/@.@


It is most certainly not in master: https://github.com/balderdashy/sails/blob/master/lib/hooks/userhooks/index.js


This doesn't follow the order in the file.

It will require to create an other resource in pfconfig while parsing the config as the hash is not honouring the order in the file


Это конечно хорошо, что ты даже предусмотрел возможность спрашивать перед удалением, но вот беда -- если у пользователя отключен JavaScript или не поддерживается, то удалить он никак не сможет. Наверное, таких людей совсем мало, но все-таки хорошо бы и о них не забывать.


Bug type : Banks are not working correctly
Location : So far, first city where you first spawn
Reproduction : Right click an enderchest, instead of popping up your 9 slot storage unit, it keeps asking you to upgrade bank slots. There is absolutely no way of opening the bank
Additional notes : None


s/toplevel/top level/


Ok thanks for the link, I'll follow up over there. We can revert my PR if
you'd like

On Fri, Sep 11, 2015 at 2:29 PM, Cody Stoltman <notifications@github.com>
wrote:

> I'd rather focus on this
> <https://github.com/balderdashy/waterline/pull/981> as a solution if you
> have the time. Adding types here one by one and implementing polyfills in
> all the adapters isn't scaleable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/balderdashy/waterline/commit/7b1729321aaded7c6bfef177189cb13a219bf774#commitcomment-13189136>
> .
>


Новый функционал это хорошо, но ты отклонился от плана. А ведь мог записать это к себе в TODO и вернуться к этому потом ;-)


> if it is important to run the tasks got from `RecordImmatureSilo(silo)` before `NotifyListeners().Ignore();`

Yes! It is very important. This is part of the whole logic of this change. To answer why it is important I have to explain the whole logic. It goes like this:
1) When silo joins, we put it in the logical "immature" state.
2) It takes 2 min to mature.
3) Immature silo serves his own queues but does not help with serving other queues.
4) "his own queues” notion is defined via what we call “ideal distribution”. ideal distribution is how the queues will be balanced across silos if all silos are alive.
5) Immediately after silo B joins we have to notify the listeners on silo A, so silo A can update its current set of queues which takes into account the queues now taken by B. If we don’t do that, A may still be serving the queue that belongs to B.
6) After B matures (2 min later) we have to notify the listeners on A again, so they have to check the queue distribution again. It could be that now silo B will also help with other queues, so my queues may change.

The optimistic case is that by those 2 min (configurable of course) ALL silos will join, so every silo will immediately grab its own “ideal distribution” and thus no silo will grab queues that don’t belong to it, and we will have zero redundant queue re-assignments. But if something goes wrong, some silo does not come up in time, anything goes wrong, we still maintain the invariant: “every queue is assigned to exactly one silo”.

It is definitely a non-trivial algorithm and we worked seem time to devise and polish it.


the above is going to cause an issue as below, work around might be to set $PROJECT to OpenELEC

21:47:21 632.986816 T:1710584896   ERROR: CCurlFile::Open failed with code 404 for http://addons.openelec.tv/6.0/Slice/arm/addons.xml.md5
21:47:21 632.987244 T:1710584896   ERROR: Open - failed to open source <http://addons.openelec.tv/6.0/Slice/arm/addons.xml.md5>
21:47:21 632.987671 T:1710584896   ERROR: Failed to fetch checksum for directory listing http://addons.openelec.tv/6.0/Slice/arm/addons.xml for repository repository.openelec.tv.


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


`@racket[init-depend]`


Types don't result from evaluation.


Про метод `remove()` ты в курсе? Т.е. такой delete более оптимален, но просто хочу убедиться, что ты и альтернативный способ тоже знаешь.


This changes the second dark background color in solarized to be quite bright, and doesn't match the actual color scheme.


I like that idea!


Good commenting. FYI we put doxygen blocks with the function definition in the source file rather than with the declaration in the header file. Our doxygen configuration looks in all the .c files under lib, so public APIs (without \internal) will show up in the generated documentation. We don't generate \internal documentation so that's just for source readability, and formatting codes aren't necessary.


I couldn't quite figure out the best way to word that sentence. Does "the type of the result of executing the body of the unit" sound any better? It still seems rather wordy and awkward.


Explain in which cases this static info would be there. Also, this general explanation is pretty low-level. How do the `define-unit` docs explain this? Do they mention static info at all? You should probably shoot for an explanation at a similar level.


Bug type : Innkeeper is not working correctly
Location : So far, first city
Reproduction : Right click an Inn keeper, it keeps saying I can't afford to pay 1000 gems for an unknown item. No menu is shown either.
Additional notes : None


eu não quero fazer as coisas de meu jeito, simplesmente estou te falando o que eu acho; se todo mundo concorda, removemos e problema resolvido.


102941444_CorrectBSDFs (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


s/Since/Because/


couvrions


s/toplevel/top-level/


Great stuff - I'll merge this in for the next release asap


I skipped the conversion to JSON format and went back to parse the XML with jquery, because the converter had problems with the correct JSON key naming format. JS hell ...


Also, this is not how "however" is used. Use a different connector, and split into two sentences.


space after if


I keep that in mind. Till know, lets keep it like this. Thanks anyway!


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

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


Doesn't this ignore/re-introduce the x,y ordering issue?


idem


I might be confusing how glyphs are layed out on the sheet with how you're rendering them... I'll have to take a look.


same here. x,y ordering issue?


Там есть какой-то более ООП способ, когда ты просто находишь сущность и потом на ней вызываешь setter-ы и все. Ты в курсе про это? А то у меня закрадывается подозрение, что ты при переходе с SQL ищешь аналоги SQL-я в JPA, но использование JPQL это дополнительный путь для гибкости, а основной это ООП подход и работа с объектами (ведь это ORM).


Updated version number from package.


No se supone que int a secas puede ser de 16 o 32 según la plataforma? Yo para una variable de ese estilo preferiría dejar que el compilador elija la que mas le convengas, nunca toma un valor mayor a 32.


What exactly was the bug about? I have't experienced any issues with the navigation in my apps running 6392


P.S. А, уже все увидел в следующих коммитах :)


Updated in the develop branch and will get merged in next week.

https://github.com/MajorLeagueBaseball/g5-component/commit/10aba44c93a4e088571facfbba6a2c06c559674d


l'environnement


Reindented and nothing else.

Emacs re-indented the document.

There are no other changes.


"the type of the last expression of the body"
Expressions have types.


create another instance of matcher (matcher = Pattern.compile(RegExp.STARTING_TAG_REGEX).matcher(TEMPLATE_HTML4.substring(stringIndex)); )


If you have time, `git checkout -- shells/firefox/src/contentScript.js`


...branches et vous avez votre équipe...


Course Project1


complex stuff you got there


dude seriously
those aren't the same names


not required, import*ing util.py brings along its libraries


v2 coming. I figured a -bin package would be too much too -- I'm staying for now with lib, devel and modules.


This is correct as written. The transform is applied to a DPX file.


This is a great answer, but beware that `String.prototype.repeat` is part of JavaScript 6. It will not work in many current JavaScript interpreters.

Can you re-do this one manually please?

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/repeat


s/if an byte/if a byte/
or maybe
**Check if two bytes are equal.**
or
**Check if a byte is equal to another.**


bravo for the succint answer :) one thing i would check for is if the input is actually a string. otherwise if you call `.split` on it it will throw an error.


you missed an @SideOnly here and it's going to crash the server whenever handleJump is called


LGTM - just a minor comment.


I cannot pretend to understand this generalized answer!

Can you please re-do this exercise for integers only? I'd like you to do it with a loop first, and then without a loop, using recursion.


IMO, this is just as clear `bool end_start = lnum < 0;`

If you disagree that's fine; I don't want to waste time bikeshedding ;)


While this is OK, you can do much simpler than that :) Can you re-do this exercise without using any comparisons?


I don't think we should filter out warnings in this layer. The hosting API should do so.


@VincentDondain - Minor: We could have this both of these properties set in the GetDefaultCreateProjectDirectorySetting method instead of here.


@bunzli en que casos se usaba el `toHaveBeenCalledWith`?


:shipit:


81211464-LinkInfiltrationAndReturnFlows (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1332 of 1332 tests passed)

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


Ovo je zbilja važna izmjena...


Unfortunately there are no functions in this commit :(


@VincentDondain - Should we rename this to be consistent with the IsCreateProjectDirectoryInsideSolutionDirectoryEnabled property that exists already? Maybe IsCreateProjectDirectoryInsideSolutionDirectoryVisible? We do not refer to CheckBox anywhere else in this class.

Everything else looks good.


Delete comments.


Looks good, please sign the CLA here - https://cla.developers.google.com/clas - and I'll merge and test your change.


this is actually caused by the side effect of SchemaPath.getCompoundPath(). see DRILL-3767

once DRILL-3767 is merged, we'll need to revert back this line of code


en el caso de que se quiera comprobar que el método se llamó con ciertos parámetros, no en el momento de la definición del spy

`expect(FinancialMetricsSrv.UF).toHaveBeenCalledWith('2015-01-01');`

Si estuvieramos revisando que estamos pidiendo el valor de la UF para una fecha dada


use sqlite3 as default.


Actually I did that in my commit cee9b6fdcbae559cef75b01fdf2a4fbfca1d2e6c

I differentiate first between integer and non integer (as well as positive and non positive), then I apply the look.

I'll do the recursion too (but I find it less elegant).


Good. Please put this inside a function that returns a value.

Tiny comment: please always use the `===` strict equality operator. Using the `==` lenient equality operator is error-prone because the rules are weird.


@dmick yes, I checked them a week ago when I wrote this. I will test again before merging


gitognore!! :fire:


Here outputs examples with `docker 1.9.0-dev` and `docker-compose 1.4.0/docker 1.8.2`.

#### `docker 1.9.0-dev`
###### `docker ps`
```
$ docker ps
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS               NAMES
35b0c726d725        php:7-apache        "apache2-foreground"     11 seconds ago      Up 10 seconds       80/tcp              php7-apache
1859e0ae4239        redis               "/entrypoint.sh redis"   11 seconds ago      Up 11 seconds       6379/tcp            redis2
ea5e492e9416        redis               "/entrypoint.sh redis"   12 seconds ago      Up 11 seconds       6379/tcp            redis
f83b20683f72        redis               "/entrypoint.sh redis"   12 seconds ago      Up 11 seconds       6379/tcp            sentinel
```
###### `docker ps --no-trunc`
```
$ docker ps --no-trunc
CONTAINER ID                                                       IMAGE               COMMAND                         CREATED             STATUS              PORTS               NAMES
35b0c726d72521645c2b715fd227f429f0e46213109da07e69a64e061c812c15   php:7-apache        "apache2-foreground"            19 seconds ago      Up 18 seconds       80/tcp              php7-apache
1859e0ae4239f54caa56a75ed2b9fa3a0b1b7149ce2deb73407ba427569b8b63   redis               "/entrypoint.sh redis-server"   19 seconds ago      Up 18 seconds       6379/tcp            php7-apache/redis2,redis2
ea5e492e94164132276d3fda630de069b37d11544ffcb5541df5f29637ba3539   redis               "/entrypoint.sh redis-server"   20 seconds ago      Up 18 seconds       6379/tcp            php7-apache/redis,redis
f83b20683f72a038846bb23179eab709c6e226c752e5d146cde0e8ae93229877   redis               "/entrypoint.sh redis-server"   20 seconds ago      Up 19 seconds       6379/tcp            redis/sentinel,redis2/sentinel,sentinel
```

###### Autocompletion **with** `${line[${begin[NAMES]},${end[NAMES]}]}%%,*}`
```
$ docker stop
 -- containers --
35b0c726d725  php7-apache                                          --       2 minutes, php:7-apache
f83b20683f72  ea5e492e9416  1859e0ae4239  sentinel  redis2  redis  --       2 minutes, redis
```

###### Autocompletion **without** `${line[${begin[NAMES]},${end[NAMES]}]}%%,*}`
```
$ docker stop
 -- containers --
35b0c726d725  php7-apache                                                                             --       4 minutes, php:7-apache
f83b20683f72  ea5e492e9416  1859e0ae4239  sentinel  sentinel  sentinel  redis2  redis2  redis  redis  --       4 minutes, redis
```

#### `docker-compose 1.4.0/docker 1.8.2`
###### `docker ps`
```
$ docker ps
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS               NAMES
de3b46d74a13        php:7-apache        "apache2-foreground"     3 seconds ago       Up 2 seconds        80/tcp              app_php7-apache_1
7c44699c2b36        redis               "/entrypoint.sh redis"   3 seconds ago       Up 3 seconds        6379/tcp            app_redis2_1
49f70c6dde94        redis               "/entrypoint.sh redis"   3 seconds ago       Up 3 seconds        6379/tcp            app_redis_1
bf02493ff474        redis               "/entrypoint.sh redis"   4 seconds ago       Up 3 seconds        6379/tcp            app_sentinel_1
```

###### `docker ps --no-trunc`
```
$ docker ps --no-trunc
CONTAINER ID                                                       IMAGE               COMMAND                         CREATED             STATUS              PORTS               NAMES
de3b46d74a13a6215ab7014c380f4d816939c955fd36966423d456be1f8b027b   php:7-apache        "apache2-foreground"            40 seconds ago      Up 40 seconds       80/tcp              app_php7-apache_1
7c44699c2b36559fdbbd8acc54aed76e64fa4d3873d0e06ffa0bd02ca2d28a29   redis               "/entrypoint.sh redis-server"   40 seconds ago      Up 40 seconds       6379/tcp            app_php7-apache_1/app_redis2_1,app_php7-apache_1/redis2,app_php7-apache_1/redis2_1,app_redis2_1
49f70c6dde9473c7c7b10cf557683b9955850bb125a08ee5e43c619aee028b1f   redis               "/entrypoint.sh redis-server"   40 seconds ago      Up 40 seconds       6379/tcp            app_php7-apache_1/app_redis_1,app_php7-apache_1/redis,app_php7-apache_1/redis_1,app_redis_1
bf02493ff47478285cefcc893de0c163769d1a7fe7ba1dd715124b4413d42ef8   redis               "/entrypoint.sh redis-server"   41 seconds ago      Up 40 seconds       6379/tcp            app_redis2_1/app_sentinel_1,app_redis2_1/sentinel,app_redis2_1/sentinel_1,app_redis_1/app_sentinel_1,app_redis_1/sentinel,app_redis_1/sentinel_1,app_sentinel_1
```

###### Autocompletion **with** `${line[${begin[NAMES]},${end[NAMES]}]}%%,*}`
```
$ docker stop
 -- containers --
app_php7-apache_1  de3b46d74a13                                                         --       2 minutes, php:7-apache
app_sentinel_1     app_redis2_1  bf02493ff474  49f70c6dde94  7c44699c2b36  app_redis_1  --       2 minutes, redis
```

###### Autocompletion **without** `${line[${begin[NAMES]},${end[NAMES]}]}%%,*}`
```
$ docker stop
 -- containers --
app_php7-apache_1  de3b46d74a13                                                                --  About a minute, php:7-apache
app_redis2_1       app_redis2_1    bf02493ff474  49f70c6dde94  7c44699c2b36  redis2_1  redis2
app_sentinel_1     app_redis_1     app_redis_1   sentinel_1    sentinel      redis_1   redis
app_sentinel_1     app_sentinel_1  sentinel_1    sentinel                                      --  About a minute, redis
```


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


:cry:

Thanks for everyone who worked on the PaperSpigot 1.7.10 project... :+1:

(I need to update to 1.8 fast...) :neutral_face:


no need to set db file for postgres; ditto for the postgres change in test-db-driver.js


removed.


call initDbWatcher before reconnect, essentially right after intDatabase.


Is this indentation necessary?


Course Project1


Without going into too much detail, it is considered bad practice to iterate over an array using the `for...in` construct. This kind of loop can iterate over keys that you were not expecting. For arrays it's always better to use a `while` or regular `for` loop.


done


What were you trying to accomplish with this line?


Got me all excited haha, you need help with anything?


How do you use it? Gulp? Could you please show me the code of the task?


add a comment how `attrs` is structured: a map with even indices keys and odd indices values.
Create issue for ts2dart


@hvelarde descobri que o pexpect não é instalado por causa da versão do setuptools:

https://github.com/ipython/ipython/issues/8806

O cara do ipython falou que não vai resolver isso. Então  as opções são:

* Pinar o setuptools para uma versão maior
* Deixar como esta, com o pexpect instalado explicitamente
* Remover o iw.debug


When I have a huge array like this, I usually break it down with one element per line. It is more readable that way. Here is an example:

```javascript
var quotes = [
    "hello",
    "world",
    "blabla"
];
```


The same as above.


@vivekkiran Looks good, before we can merge the change, you'll need to sign the CLA here and put in your GitHub username:

https://cla.developers.google.com/clas


type `directives`


Can you do better than `0, 10` here?


Bug type : Wooden staffs so far depleting the whole energy bar when used
Location : Wooden staff
Reproduction : Right click a wooden staff when your energy is at 100%
Additional notes : It used to be 2/3 energy per use, so this is definitely a bug.


Perfect! I'd like you to explore the `switch` statement to see how you can get to the answer in a different way.


interesting feature :+1:


Nice answer! I wasn't expecting anyone to come up with this!


Duals can be returned through the MPB interface


good job!


Whats is a bit confusing is the fact that ttf fonts have all sorts of different sizes defined for glyphs. you have the size of the glyph itself, it's offset, it's advance (for RTL writing). And additionally there are kerning pairs that (normally) reduce the advance size. All these are different.

 On the sheet you have, you probably use the bitmap in the font cache, however, the advance is unrelated to that. For example, a letter i or l in a sanserif font will have a very narrow bitmap in the cache, but it's advance may be larger than it's size+offset to make the font more legible.

I checked it though and there is a bug in my current PR, somehow kerning isn't applied when it should be. I'll look into it further.


Course Project1


Remove the if condition and call setMaxLimitation for all cases.


done


Excellent. Calling a function from inside itself is called recursion. It's one way to get at the answer. Can you show me another way to do this without recursion?


Are all these comments necessary?


maybe keep all of these as properties with values, and remove the getter/setters?


Even without the reduced costs as well? I saw this check for both being present:

https://github.com/JuliaOpt/JuMP.jl/blob/562862ad0118985abf63227d8f9dd3005756767e/src/nlp.jl#L585


I like this, but I think it probably needs to live in a separate file and be sourced by each test.  I'm a big fan of the fact that right now, an individual test can actually be executed directly, which is really handy for debugging (and makes it more obvious where this `retry` function comes from).


I think of the hooks as forming a pipeline (which is rather a mixed metaphor).


Still don't like double underscores. The forward double underscores mean "private" method and trigger special mangling of the method's name (ClassName__privateMethodName). The technique used mostly in API libs. Leave just one underscore to mark it as an "internal" method. It's quite enough here.


use `MapWrapper.createFromPairs`


s/currentely/currently


Use also for `ChangeDetectionStrategy`


or even just as a `retry.sh` that gets run each place instead of sourcing a function


Why did you break this line?


When you do it, write sth like this:
validates :content, length: { maximum: 140 },
                             presence: true


C90 dice que int es del tamaño del registro del procesador y minino 16
bits. No habla de máximo.

Mañana miro el código pero en general prefiero también que el programador
decida que tipo quiere.

Saludos
Mariano
Am 11.09.2015 23:01 schrieb "Esteban Volentini" <notifications@github.com>:

> No se supone que int a secas puede ser de 16 o 32 según la plataforma? Yo
> para una variable de ese estilo preferiría dejar que el compilador elija la
> que mas le convengas, nunca toma un valor mayor a 32.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/76905f6ba96a6f70fe06d75315cfdf408b1cb304#commitcomment-13191875>
> .
>


Spell Check!


@jerch, OK, I will if I find one.


check whether `ts2dart` has an issue for this


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (789 of 789 tests passed)

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


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


@tstachl Hahaha. I just now saw this.


4967-SZRH-spm (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1841 of 1855 tests passed)

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


Course Project1


umut commenti


using SSL


Be patient. I am currently working on a major update. In the development version the link is working. I will merge develop and master branches soon (including and updated version of the link/vignette)

Tim


wow! 1.0 already :-)!


I'll concede that this is necessary only if the package wants to export a concrete type. We've had this for the other packages, but I am not sure why that was done originally. We may actually be able to remove the double wrapping.


Bloated code. What's in there now is leaner and does the same job.


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


You forgot one.

```patch
--- liblxqt-9999/lxqtpower/lxqtpower.h.orig     2015-09-11 23:28:45.600645372 +0200
+++ liblxqt-9999/lxqtpower/lxqtpower.h  2015-09-11 23:30:57.566150385 +0200
@@ -103,7 +103,7 @@

 private:
     QList<PowerProvider*> mProviders;
-    LxQt::ScreenSaver mScreenSaver;
+    LXQt::ScreenSaver mScreenSaver;
     QEventLoop mLoop;
 };
```


FYI I made a class for this. You can just add `class="yellow-font"` to any tag to make the font yello. eg in this case you can add it to the like this: `<h3 class="yellow-font">`.

This way if we ever need to change the yellow color we don't need to track down all the instances, we can just change the color in the CSS class.


@danielmarbach what this change is for?


Fixed.


Done. Made it 403.


:clap:


Hi @ThomasBurleson!

Was there a test added with this change?


Ansioso pra colocar isso no front =)


It has `css` but it doesn't have `js`


Ideally, the iteration numbers could be passed as an argument like `new Preset(100, 1000, 10000)` since some tests might require higher or lower numbers of iterations to get good measurements.


Course Project1


@tkelman - does this commit mean that it’s been backported and will be available in 0.4? (How to tell whether it's been merged?)


Suggest dropping -webkit-transform at the least, since that only applies to versions so old nobody will be using them. These days it's just bloat.

-ms-transform is only needed in < IE9, but <IE9 has zero support for CSS3 anyway so it still won't do any good. Just use the W3C syntax.

http://caniuse.com/#search=transform


Homework: Introduction to Programming
This document defines homework assignments from the “C# Basics“ Course @ Software University. Please submit as homework a single zip / rar / 7z archive holding the solutions (source code only) of all below described problems.
Problem 1.	Play with Visual Studio
Familiarize yourself with Microsoft Visual Studio (if you already have it installed) or install Visual Studio (or Visual Studio Express) at your laptop or home computer. Search in Internet for the correct download link: http://google.com/search?q=download+visual+studio. You do not have to submit anything in your homework for this problem. Start Visual Studio and play with it. Create a simple C# program (console application), compile and run it.
Problem 2.	Blank Solution in Visual Studio
Create a blank solution in Visual Studio called “Intro-Programming-Homework”. This solution will hold all your homework projects, code and files. For each problem (exercises) add a separate project with self-descriptive name like “Hello-World” and “Print-Your-Name”. You do not have to submit anything in your homework for this problem.
Problem 3.	Play with MSDN Library
Play with Microsoft Developer Network (MSDN) Library Documentation. You may find it online at http://msdn.microsoft.com/library.
•	Find information about Console.WriteLine() method in MSDN.
•	Find information about the Console class.
•	Find information about the class keyword.
You do not have to submit anything in your homework for this problem.
Problem 4.	Hello World
Create, compile and run a “Hello C#” console application. Ensure you have named the application well (e.g. “”HelloCSharp”). You should submit the Visual Studio project holding the HelloCSharp class as part of your homework.
Expected Output
Hello C#!

Problem 5.	Print Your Name
Modify the previous application to print your name. Ensure you have named the application well (e.g. “PrintMyName”). You should submit a separate project Visual Studio project holding the PrintMyName class as part of your homework.
Expected Output
My name is Bob.

Problem 6.	Print Numbers
Write a program to print the numbers 1, 101 and 1001, each at a separate line. Name the program correctly. You should submit in your homework the Visual Studio project holding the source code of the PrintNumbers program.
Expected Output
1
101
1001

Problem 7.	Print First and Last Name
Create console application that prints your first and last name, each at a separate line.
Expected Output
Bob Dylan

Problem 8.	Square Root
Create a console application that calculates and prints the square root of the number 12345. Find in Internet “how to calculate square root in C#”.
Expected Output
111.1080555…

Problem 9.	Print a Sequence
Write a program that prints the first 10 members of the sequence: 2, -3, 4, -5, 6, -7, ...
Expected Output
2,-3,4,-5,6,-7,8,-9,10,-11

Problem 10.	Reformat C# Code
Reformat the following C# code to make it readable according to the C# best practices for code formatting. Change the casing of the identifiers in the code (e.g. use PascalCase for the class name):
HorribleCode.cs
using

System;

class hoRRiblEcoDe
{
    static
     void

        Main()
    {
        Console.

    WriteLine("Hi, I am horribly formatted program"
); Console.
      WriteLine("Numbers and squares:")
; for (int i = 0;
i < 10;
i++)
        {
            Console.WriteLine(i +
                " --> " + i
                *
                i);
        }
    }
}
Problem 11.	Programming Languages
Perform a research (e.g. in Google or Wikipedia) and provide a short list with information about the most popular programming languages. How similar are they to C#? How do they differ from C#? Write in a text file called “programming-languages.txt” at least five languages along with 2-3 sentences about each of them. Use English.
Problem 12.	Development Environments
Perform a research (e.g. in Google or Wikipedia) and provide a short list with popular development environments (IDEs) like Visual Studio. Write in a text file called “list-of-IDEs.txt” at least five IDEs along with 2-3 sentences about each of them. Use English.
Problem 13.	C# and .NET Differences
Describe the difference between C# and .NET Framework in 2-3 sentences. Write your description in a text file called “csharp-and-dot-net-framework.txt”. Use English.
Problem 14.	* Current Date and Time
Create a console application that prints the current date and time. Find in Internet how.
Expected Output
19.5.2015 г. 12:02:55 ч.

Problem 15.	* Age after 10 Years
Write a program to read your birthday from the console and print how old you are now and how old you will be after 10 years.
Input	Output
12.04.1991	Now: 24
After 10 years: 34

Problem 16.	* Print Long Sequence
Write a program that prints the first 1000 members of the sequence: 2, -3, 4, -5, 6, -7, … You might need to learn how to use loops in C# (search in Internet).
Expected Output
2,-3,4,-5,6,-7,8,-9,10,-11,12,-13,14,-15,16,-17…998,-999,1000,-1001

Problem 17.	* Play with the Debugger in Visual Studio
Write a program that prints at the console the numbers from 1 to 1000, each at a separate line. You might need to learn how to use loops (search in Internet). Set a breakpoint in the line, which prints the next number in the Visual Studio code editor. Run the program through the debugger using the [F5] key. When the debugger stops at the breakpoint trace the code execution with [F10] key. You do not have to submit anything in your homework for this problem. Just play with the debugger to learn how it works.


Since you're linking to google docs you could just remove the 2nd param of the lect() function so there's no dead links.


don't forget to call the callback on all the error returns in initDbWatcher.


Good idea ! :)


@millpond care to explain?


Seems to be the `tell` and `ask` functions. Which definitely should have been here....they were on s.b.e.


103122604-ClimaticDataSummaryBug (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1857 of 1857 tests passed)

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


Not _yet_. But getting closer!


Perfect! Can you read about the `switch` statement and find out a different way to do this exercise?


The first two tests here are exactly the same.


you can use PostgreSQL.prootype.tableEscaped
 https://github.com/strongloop/loopback-connector-postgresql/blob/master/lib/postgresql.js#L326


Perfect! This is a great solution. Calling a function from inside itself is referred to as recursion. It is great for some problems. As a challenge, can you re-do this exercise without recursion, using a loop?


Comparison of psmops_veres_nn747 (MOPS) and mafsso (neos_s3m_747_allObs.txt).

Objects found in MOPS sim are all in mafsso. 524 objects. 558 objects are in mafsso, so there are some missing in MOPS. This number can vary depending how mafsso defines the magnitude and visibility. I checked those  - on average, the missing ones are only 0.02 mag away from the limiting magnitude, so perhaps the discrepancy in filter transformations (see below) could cause this. All missing detections were inside of the FOV (ra and dec).
Attached plot shows discrepancy in RA and DEC in arcsec between matching detections. Average dRA=0.62 +/- 1.18", dDec=0.35 +/- 1.04 "
Discrepancies in colors. Night 747 only has 2 filters (r,z) and I expected that we will use fixed V-filter terms (average S+C). However, this is what mafsso has:
filters	avg(dmagcolor)	std(dmagcolor)
r 	-0.239 +/- 0.03
z	-0.383 +/-
![mops_catsim_comp1](https://cloud.githubusercontent.com/assets/13923689/9827125/1cc50b12-5894-11e5-8b24-4df39276abba.jpg)
0.04

MOPS has r: -0.213
                  z: -0.349
With no dispersion
Attached is histrogram of magnitude difference. This is MOPS magnitude (already in filter and with trailing loss implemented) minus mafsso (magFilter + dmagTrail). I am not sure if these are the correct values.

![Uploading MOPS_Catsim_comp1.jpg…]()

![mops_catsim_comp2](https://cloud.githubusercontent.com/assets/13923689/9827128/228e7a6a-5894-11e5-86ea-cb7ce5c3bd80.jpg)


I think that should be "The Scala forums do not allow..." assuming the "forums" is a group term referring to more than one. ;-). Perhaps even better would be a more explicit "Job postings are not allowed on the Scala forums and are subject to be removed." or something like that?


Awesome!


@brianharvey Wait this is somewhat wrong -- this has ask and tell in both sense and control palettes. (Same code though.) So, only one set should stay.


@glima,
 * 1_beta3 (*changelog* contains this info, which is the info used by dpkg-buildpackage);
 * chrpath has to be removed, indeed;
 * -bin x -devel: Makes sense. I remember I got an issue while testing this, need to revisit the issue;
 * Yep, I need to add a line to *rules* to fetch the submodule. I did that at certain point, but I rewrote this file when I learned about override and forgot about duktape =/ ;
 * Yes, I'll split the packages. This was just an import of the packages already published at (new url) https://launchpad.net/~solettaproject/+archive/ubuntu/packages.


Sorry.  I will remove it tonight.  Thanks


That sounds wrong.  Why can't you just leave the themes/ directory as a sibling to dijit/?

At some point I hope you add theming for other components, like dojox/charting and gridx etc.  At that point it wouldn't make any sense for the theme to be inside dijit.


Use named params?


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-MacOS-10.9-clang: Tests Failed

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


Can you explain? I dont your comment.


Curious why removing the ability to using src as a directory path was removed to make it more flexible. It seems less flexible.


No thanks,
I already fixed the "missing mobile network issue",
was pretty easy.
And my implementation of obtaining the wifi name was to complicated,
(carrier name, too, but it was working fine the complicated way, but why
complicated), I found a much easier way, so also the wifi name will be shown as expected now.

@NL-BlackDragon
I didn`t got a Hangouts,
even if the problems are solved,
maybe you`ll need it later, who knows
my Hangouts "mail": xxmrhyde2@gmail.com,
but don`t email me, I use the mail address for Hangouts only.


After testing with the test driver, I found that kerning does work, but there was a bug in ex_ttf where I mistakenly didn't apply kerning, when I wanted to that confused me. With the test drive and the "test font glyph match" series in tests_fonts.ini I could see that string drawn with al_draw_glyph and al_get_glyph_advance DO exactly look the same now.

So I push a new commit that fixes that bug and tries to take care of your comments.


101917310_waterCoilHumRatFalseWarning (Nigusse) - x86_64-MacOS-10.9-clang: Build Failed

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


While this certainly works to some extend I'd like to know what @grahamdumpleton thinks about this. IIRC he has a blog post advicing against this pattern.


this is final


You can force HTTP retrieval with a context like so:
```
function file_get_contents_http($path) {
    $opts = array(
        'http' => array(
            'method'=>"GET",
        )
    );

    $context = stream_context_create($opts);
    $result = @file_get_contents($path, false, $context);
    return $result;
}
```
Taken from examples people posted on the manual page: http://php.net/manual/en/function.file-get-contents.php

Please note: This is still not very safe.  You are essentially hosting an open lame box.


This one is just to get rid of the  "attempt to re-open an already-closed object: SQLiteDatabase" error that occurs if you
1.open the app
2. push the back button (app closes)
3. re-open the app again (with closed db access and trying to query the db for unread post count)

I'm sorry for the multiple pull-requests. I'm used to SVN and new to GIT


I'm running into this now after bumping the version of Angular Material and it it's been difficult to debug. I wish the error message addressed the resolution to avoid getting into this quandary.


There's actually already a pull request for dgrid. https://github.com/SitePen/dgrid/pull/1174

That being said, this struck me as odd as well - not necessarily because I expect the theme in the themes repo to encompass more than just dijit, but because if we were expecting it to be located under dijit, I'd expect us to...just add it to dijit directly.  Otherwise, I expected this repository to behave as its own package, essentially.

I suppose that does make me wonder why we're PR'ing it directly into dgrid but not directly into dijit...


I just put back the double wrapping, shall I remove it again?


> > I think the comma here can be dropped.
>
> I'm not crazy about it either, but this is to be consistent with other prose around setters in the spec.

OK, agreed that consistency should win.


Why have the static block for just initializing these constants? You can do that inline.

The negative side of this approach is that it makes refactoring a bit harder.


Great :)


Oh god, it's the great pyramid of timesync again. If we add another table we'll have to change both this code and nearly identical code in the tests. Maybe we should refactor them to use one function?