No jak nie ma powiązania jak widać, że ustawienie limit zmieniło nazwę na debtors_limit. Dzięki temu w jednej sekcji można trzymać wszystkie ustawienia.


This code seems duplicated. What about extract method?


Okay, I got lost amongst all the IFs and did not see this one:

![image](https://cloud.githubusercontent.com/assets/5129209/8848986/423f052a-313c-11e5-89a6-ddc5e856588b.png)

Indeed it's a rare case, since it depends on information coming from the database, which would probably be caused by some weird inconsistent error. Due to that, I wouldn't just return null, but instead throw an exception, log it and proceed. Why that approach? If the same code gets called by someone else, in another place, and the person doesn't check the "null" return, it will cause an NPE. Having the exception would make the person know what could happen.

That's just a preference and doesn't require you to go and change your implementation

Could you apply the code style to the code and get the curly brackets in place? It can be a separate commit, no need to squash.

Cheers,
Wilder


I don't mind repeating this line two times. I think it's better than creating a new method. Anyway, I'm open to listen other peoples's opinions.


Hola Diego.
Esto se habrá cambiado por error?


I agree it's annoying. It also steals your back button (PR #572) making accidental triggers reeally annoying.


Did you mean to push this to the new branch? :P


Woooo

!blocker You have to fix this and that!


And there is a backup file before any changes.

$pfsense_version=preg_replace("/\s/","",file_get_contents("/etc/version"));
if (!file_exists('/root/'.$pfsense_version.'.captiveportal.inc.backup')) {
-		copy ($cp_file,'/root/'.$pfsense_version.'.captiveportal.inc.backup');


OK, so speed>type-strictness (within reason) going forward?

I'll commit speed improvements to a new feature branch - and I expect you to object loudly (or just raise concerns) if, at any point, you feel I'm going too far.  OK?


dkf


이걸 왜바꿔요? (테스트)


Seriously?

!blocker And this is even more broken...


match parent가 뭔가요?


Well as far as I understand the `status_strings` map in which every postprocessor saves its state currently contains one string per postprocessor. Since everything I serialize is essentially part of that postprocessor I dump it all in that string. I could split things up under different keys, but it would only make things more complicated. Also the only reason for creating this map is (as far as I can see), that we are able to add or remove postprocessors between restarts, but I do not think it makes sense to implement the same functionality for parts of the tracer postprocessor e.g. for tracer outputs. You do not typically want to switch output formats of tracers between checkpoints ... the only thing you might want to change is the output timing, but that is covered by this patch already.

That being sad, if someone has an application for switching output plugins during restarts or also wants to serialize the property manager to be able to switch tracer properties during restarts, it would be relatively easy to implement that ... I just did not see the need for it at the moment, and we can add it later (as far as I remember we do not guarantee compatibility of old restart files with new ASPECT versions).


match_parent를 왜 써요?


I've waited for 100.0 to become default before pushing drastic changes.


잘 모르겠어요


match_parent를 왜 쓰나요


sounds good! You can also make multiple braches for independent stuff so we can select/reject seperatly.


부모와 넓이를 같게 만들기 위해서 입니다.


a demo project
use greenDao、EventBus


Ben this is awesome! Thank you so much!! Especially appreciated the comment about beautiful stars :laughing:


이렇게 댓글을 남기면 선생님이 답을 해주나보네요


신기하네요


Confirm that this fixes #8391


would be also great to have a performance suite in place to verify the improvment


its default since a few minutes :)


maybe use file selector dialog ( so probably some kind of button to select file that as response call this UI method and fill input field )? see https://doc.opensuse.org/projects/YaST/openSUSE11.3/tdg/AskForExistingFile.html


Are there any benchmarking tools/services you know of, that might help?
I've so far only used naïve timers and console.log(), etc... which aren't always very productive...


@bydga can you please rebase your code on the current master? I've switched the URL scheme from hashbang `#/` to HTML5 mode (See 3807ecf1b802d02627dad9b6d5a2366bd1cf8df1 and [app.js](https://github.com/kwk/docker-registry-frontend/blob/master/app/scripts/app.js)). I think it should work if you write

```javascript
$window.location.href = 'repositories';
```

But please double check that it works.


Put the file to pastebin.com and post the link right here ;)


이 부분이 회색으로 alt+enter 를 눌러서 'lambda' 로 고치고 실행 하니 오류가 나오네요...
Error:(41, 41) error: lambda expressions are not supported in -source 1.7
(use -source 8 or higher to enable lambda expressions)

이건 뭘 바꿔야 하나요??
그리고 'lambda' 를 써야 하나요????


Where do you get a warning? What does the warning message say?


Yes, I removed it.


Sorry for being out of the loop :P I'll push the commits today.


Other than my comments this LGTM


lgtm


@mbernat This is basically the only change here, to use this function from the salsaflow repo, the rest is just moving stuff around.


Should probably use /^x-rincon-queue:/ to be picky.

queueInUse would probably also make sense to return the same value for any member of the group, since it is a group-property (as opposed to volume and mute), meaning that row 850:

queueInUse: this.state.queueInUse

should be

queueInUse.this.coordinator.state.queueInUse

What was your user-case for this property? I'm just thinking if there would be future requests for identifying other kind of states, and having this as a multi-value property instead (I'm thinking radio, queue, line-in etc).


`console.time` might be handy in the first place.

http://benchmarkjs.com/ seems a good choice since it's from the author of lodash and also used by lodash.


From wich device have they been imported ?


As these properties are dynamically retrieved each time they are needed it reflects better to changes to other modules and configuration.

This also enables the init method to return to being empty.


same as above


Made this function more generic so that it can be used to search in both body and title.


I'm terribly sorry for the push to master, but we needed this since stuff was broken


@RandomAltThing Well there's nothing wrong with it... Yours just saves the config twice


No `fromJust` in production code, please.  It's safe here, but it's harder to see that than using pattern matching.


if only changing the VERSION is not working somewhere then it is a bug :)


![new mockup 1](https://cloud.githubusercontent.com/assets/9396988/8849599/a20916a0-3149-11e5-87b0-395d1829acff.png)


TeamCity xBIM :: Xbim Essentials :: Release [Build 61](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2123&buildTypeId=XBIM_XbimEssentials_Release) is now running


>The countries.nerd.dk service is used to look up countries.

It seems that this is not true. Apparently, outdated documentation.
You should not worry about the rest now. In our case, we have either an ISO country code, or an empty string if the IP is not found in the database.


TeamCity xBIM :: Xbim Essentials :: Release [Build 61](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2123&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:01:01


@jkarni didn't you say that `port` should always be set here?


Maybe add a comment stating what this is for, e.g. "if the divider value must itself be divided before being written to the divider register, we must ensure we don't have any bits set that would be lost in doing so"


It doesn't work for me. Backdrop doesn't move along with the dialog.


No need to update the pull request, I'm already working on merging it!

I just wanted to make sure that `output_includeScroller` can be removed, or were there plans for this option?


`/docs/styleguide.md` line 80.


Might be worth using `Config.getbool` for added certainty that you'll get a boolean back - should only matter if someone inadvertantly uses 1 or 0.


Regarding name - once I propagating new name all around the code (to make EtcdHelper private), I think EtcdStorage is much better - just "Storage" will cause to many name conflicts.


The initial problem statement said that this should be true by default.


in this case I think the username should be passed in, we only want to use the credentials to create credentials not for things like this


we would get this for free if we used types from `uri-bytestring` to store the uri, not only to parse it.  but i don't insist on changing that.


greater or equal to start*


I'd add on the end of this something like "(ie. for if the effective divider value is the value written to the register multiplied by some constant)"

Just so it's clear what the difference between the shift and div fields are.


Not needed.


It is not the same event.
One is "show" and the other is "shown" :)

That for pointing out that i should be using vm..Fixed.

Preselecting the note name is good to have usability thingy. If user wants to give a new name, he can directly type it, without having to go and select the text field.(Reducing the number of mouse clicks).
Instead of having DOM manipulation in controller(my mistake earlier), i have moved that to directive.

Let me know ur thoughts.


It looks like 6 line of codesmell. Moreover it's more than half of visual space of method. This implies there are probably some differences, which is false assumption after redundant time of reading, because it's just 2 words in message.

I think demo should demo present best practices more then anything. I had to unlearn same similar bad practices when starting on demo of another MVC framework.

Despite of this, I respect your opinion to keep that way. Do you see any advantages for user in this approach?


This way of setting `SYNC_LOCAL_DATA` seems quite brittle - we don't really know what other calls to `MySociety::Config` might be made by the code being tests. Maybe better to use `expects(:get).with("SYNC_LOCAL_DATA").returns(false)` so you know what you're setting the value for and that it's not affecting anything else.


yes, I'm also sure there was another element in the chain for sure, but fact was that some users have reported that and when I tested I got it like in the first try.

I was for two weeks in a discussion with popcorn and his colleague about but in RPI FW. then when all failed , I started git bisect.


ok... I will update this


Nice, weiß schon wofür ich des auch brauchen kann :P


Might be worth an assertion here that the value is true - so that we know we're actually changing it.


But then we'd have to use `fromJust` everywhere, and all the extra validation wouldn't be reflected in the type.


FYI I just made a quick coverage check:

```
homebrew/apache: 45.45%
homebrew/binary: 89.47%
homebrew/completion: 100.00%
homebrew/devel-only: 75.00%
homebrew/dupes: 91.89%
homebrew/fuse: 80.77%
homebrew/games: 79.52%
homebrew/head-only: 65.52%
homebrew/nginx: 0.00%
homebrew/php: 37.80%
homebrew/python: 71.43%
homebrew/science: 87.66%
homebrew/tex: 100.00%
homebrew/versions: 92.27%
homebrew/x11: 90.00%
```


Fixed in wp 4.2.3:
https://core.trac.wordpress.org/ticket/32305

we can:
1) totally revert this code and remove the images
2) make a check on the wp version.

IMHO we should remove it, it's not a real issue :)


Fixed.


These tests all rely on the later change to `is_imported?` made in https://github.com/mysociety/foi-register/commit/406622e70b71272c5904c3256e305ed8fff292e3.


udat


We need a link to the website in author section


@barryvdh - please provide more information and details in a new issue submission.


much better, there is just some notes, but in general I think it is ready to move to yast organization and adding to opensuse


What's with the 2-space indents julen?


Adapted for Shamu and removing Input Boost isn't enough to put your name on the driver and as the MODULE_AUTHOR
You didn't even give credit to @showp1984 for writing it at all


Not sure if this stuff is supposed to be in there... https://travis-ci.org/FRosner/spawncamping-dds/builds/72259380


It does not matter. just want to create items and blocks. the craft can be in the same JS


Looks good to me.


Slava,

Indeed.  I do apologize, I had been awake too long working on .eslintrc.  When I realized that they were different formats, I made the necessary changes.  I don't usually just copy a config file & use it, that's ultimately why I put the comments in.. for the credits.

When I hit a couple of my JSON dotfiles with comments I had been awake for too long and all wrapped up in the .eslintrc (which I just happened to be debugging now, lol).  You may note that I changed .jscsrc to .jscs.json, it is more explicit and works nicer with the highlighters.

I published my skeleton & dot files a bit earlier than I would have liked (they've only been up like 70 hours, thanks for checking them out!), but I had to make room in my  private repos.  I do hope you'll forgive me. :)

With that said, there are plenty of bugs left in my dot files & skeleton, they are all newly created.  Maybe check them out next week, things should be tightened up by Monday.


> why python and VS

@KostyaSha because someone may well choose to use [`contextify`](https://github.com/brianmcd/contextify/wiki/Windows-Installation-Guide) as has happened in at least one plugin.

And yes - I had several WTF moments - some javascript needs a C compiler...


this stuff should go out of this PR, it's unrelated to the new completion. the idea is to give some hints about what to import when referring to an unknown type. the impl is quite fragile.


Replace magic number with a constant


should go out as well (see above)


Just installed (almost) all `homebrew/nginx`, we’re at 98% now.


OK, thank you :)
file:///C:/Users/Windows7/Desktop/sk.yml


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


@retronym do you remember what prompted this change?


What happens when I am in the JTA case (config wise) but no JTA transaction is surrounding the call?


Wouldn't it make more sense to highlight ```-behaviour``` attribute instead?


Isn't this exactly the same as h2? Also, why have it be a method on keyManagementContext? Not better to have it as a free floating function?


It is the internal knockback velocity accounting to prevent false positives, due to the latest MC versions having the knockback calculated on client side. The method should probably be named assumeKnockBack(Velocity) or similar.


Closes #1


@tdhock could you give a correct example of test `counts of chunks downloaded or not at first`? I have no idea which chunks should be downloaded or not at first.


The comment above is good :-). Even better is to write down these formulas, in the module docstring.


Closes #6


First thing first!


It looks to me like you found a bug in our `TribalScents_Sniffs_Arrays_ArrayDeclarationSniff`
https://github.com/moderntribe/TribalScents/blob/master/Sniffs/Arrays/ArrayDeclarationSniff.php#L389

line 35 should have a trailing comma.

This isn't really an "unusual style" though.  As you can see from the sniff code I cited above, we borrowed that almost in it's entirety from Squiz.  It is also shared in the WordPress sniffs:

https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/blob/develop/WordPress/Sniffs/Arrays/ArrayDeclarationSniff.php#L635

Additionally, it is explicitly optional in Pear standards, though shown in the example: http://pear.php.net/manual/en/standards.arrays.php

The main argument for keeping a trailing comma in arrays is that future code changes have more isolated change sets.  For example, if you add a new entry to the end of an array, the diff will only include that line as you will not need to add the comma to previous last line.


Those display names are great, but I think that they belong to another commit. The contextualize reactor for isomorphic js is experimental, and I'd like to be able to revert it easily if necessary.


в гемах тянуть отладочные гемы не стоит. из релиза надо будет убрать


I thought about this. Ultimately, I changed it (in the first commit) from one to 1. Outside this section, I changed all single-digit numbers to numerals, but in Time to conduct, I standardized everything as numerals. My thinking was that this:

##Time to conduct
Three to eight hours

is a lot harder to scan than this:

#Time to conduct
3–8 hours

And with such a short field, we ought to optimize for scanning. Of course, not all of them are quite so easy, as with this one where we change units from minutes to hours. In which case, we might prefer to go the other way? And certainly we have some that have caveats or "Part A takes X minutes and part B takes Y minutes" constructions. So I'm not set on this and open to being turned around on it. But that's how I ended up there.


Hey, there are no tests that exercise these error conditions. Please make sure to add tests for errors going forward.


the reason i took these out (and/or put them in example) was because the lines within master we not consistent. if this has changed, ok, but need to wrap testing around teh whole feed (minus maybe the dga feed, might be too big to check in?)


Why is this being changed?  It wasn't changed in the last release commit, right?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.801.12e7cfdd81181862bac0de9a10bc069321efde51](http://build.palaso.org/viewLog.html?buildId=59945&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 393, ignored: 3 Build time: 00:04:19


remove


I could if you want, yes.  I didn't for one reason because I think its location is more logical now (as it's next to the `newToken()` and `deleteToken()`), but I can indeed move it back if you prefer.


Thanks!


test scope


<img width="1429" alt="zrzut ekranu 2015-07-23 14 14 45" src="https://cloud.githubusercontent.com/assets/948674/8850078/4c1a9970-3145-11e5-90a5-88febda6825e.png">

Current game UI


I'm not sure to be honest.

It sounds to me like most of that work is very generic DOM manipulation. So not really Flip's business...
I'm fearing that if we add too much fluff Flip will lose that it's a small, focused library. You could also just do:

```javascript
$('#card').find('#oldface').replaceWith('<div id="newface">The new face</div>');
```

Then again.... Maybe I'm thinking a bit too much inside the box.... Why does Flip have to be about cards? We can flip more than that... Thinking about your suggestion I'm suddenly reminded of a very cool feature that James Bond has on his car. He can flip his license plate around. To enable this, his license plate is triangular, with three sides (faces):

http://www.instructables.com/id/Super-Spy-License-Plate-Swap-Mechanism/

Maybe we could do something like that?

> i'm not sure how hard it'd be to implement

At the moment that's an important consideration I think. If this takes dozens of lines of code then maybe we should leave it out... If it can be done in just a few lines than I think we should do it.


Ah, because it's Oracle which isn't read and re-written.  Ignore previous.


I see. The reason is enough.


shit


LGTM. Thank you.


Согласен


my computer at home looks like it does fonts slightly different. I think that is the only change


:+1: Sorry, I thought I'd caught all of the R&P references. I found one other I'd missed and have updated both (will be adding to the branch shortly).

To your larger question, yes, I linked cross-references whenever I found them. I may have missed a few, but I am less worried about that than other issues (except in this case, where the language is so standard across pages that it seems important to do it the same way). We can add more as we find them.


Yes sure :)


TeamCity xBIM :: Xbim Essentials :: Release [Build 62](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2124&buildTypeId=XBIM_XbimEssentials_Release) is now running


Info übernommen von https://wincrypt.org/


Caught it in the second commit. Sorry, I hadn't thought through how weird this PR might end up with two big commits on top of each other.


TeamCity xBIM :: Xbim Essentials :: Release [Build 62](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2124&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:01:34


This update seems to break Vagrant when provisioning calling salt-bootstrap.sh (as it's not called with the -P flag)


These are actually two objects you might call "database" here: DatabaseConnection (which the library's user sees) and Database (which the library's user does not). Both DatabaseConnection and PreparedStatement have an Rc<Database>. So, although the PreparedStatement can outlive the DatabaseConnection, it will _not_ outlive the Database. By the time sqlite3_close is called, all the sqlite3_stmts will have been finalized.


Looking over the configuration, I want to use `~` instead of leaving the values empty.  For one, it's clearer that we didn't forget to put something in. Second, using a nil character means that the configuration is equivalent (character-wise) between the flow style and the tree style.  Finally, http://www.yamllint.com suggests that this is a "canonical" representation.


I think it'll be a good example of voter usage. Maybe better move this logic to voter?


Most were actually formatted with ```-``` so I tried to get all unordered lists formatted that way on the second commit. I did miss a few and will add those fixes shortly.


See above.


@vsavkin does this commit fix the injection of async services (as in an injectable that returns a promise with a factory)? Because the `@InjectPromise` has been removed as described [here](https://github.com/angular/angular/issues/2813#issuecomment-118601763).


Made changes for SPARK-<4557 #>


you do the same transformation again and again. (stream.flatmap(blabla))
You should put a moduleTables() method into CassandraModule (same for moduleIndex())


So this is supposed to be ["release-1.0"]. I realize it's confusing but that's what "release_id" means - it's not a database ID, but a short name of the release


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.802.e108370d26c91c6d2add7d093acebb7607ad3199](http://build.palaso.org/viewLog.html?buildId=59946&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 393, ignored: 3 Build time: 00:02:47


Well... Long story short I think your suggestion is actually a good idea... But only if we can implement it within a few lines of code, because Flip should remain a tiny library.


I keep it simple - just assume the strings for the "releases" are release_ids, do not put them into a hash. It makes things unnecessarily complex


This is missing documentation on "release" query parameter - which is a product release ID by which to query


After seeing how the Ruby YAML writer outputs it's content, disregard previous comment.


здесь может упасть теоретически при парсе, лучше добавить хандлер и написать тесты, в том числе и на наличие 'Desc'


This looks good for merging.


And this should be ["release-1.0", "release-2.0"]


this commit leads to big problems with roles.

1. simple roles without groups don't work anymore (schema type: [String]) when you want to toggle a role on a user
2. if you use roles with groups (object, blackbox:true for the roles field), adding users will work, but removing won't as "adminRemoveUserToRole" doesn't provide Roles.GLOBAL_GROUP for the group permission to be removed. worse: "make first user admin after meteor reset" broken as it uses simple roles without groups.

suggestion:
1. generally distinguish if Roles.GLOBAL_GROUP should be used at all depending on the user scheme's roles type ([String] or Object).
2. add "Roles.removeUsersFromRoles _id, role, Roles.GLOBAL_GROUP" in complex roles model (with grouping)
3. update the "make first user admin" logic to support complex roles model (with grouping)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.803.21d566b1eee0de51d86b7660219c3abe607e82fc](http://build.palaso.org/viewLog.html?buildId=59947&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:35


```%s```.

Also, why not report progress inside buildSources call?


OK, works fine with the `pointer-set`. AFAICS the GCC `hash-set` doesn't hash string contents either, it just uses the pointer as a hash.


Formatting.


I think we should get in the habit of using these colorful branch names consistently, they give the project some desperately needed pizazz.


I wasn't happy about this change. There are somethings I'll investigate to hopefully remove this XBoxism


why is this not commented out?


Am Donnerstag, 15. Januar 2015 07:21:39 UTC+1 schrieb oroulet:
>
> You already can do this by using DataValue ( see test_high-level.py ). But
> this patch  will make it easier to discover and use... BTW maybe we can
> automate this... By fetching type first.... Must think about it ;-)
>
> Are there other things you are missing or that do not work?
>

Hi there,

as beginner I am playing with the server.py  and the UaExpert Viewer. While
setting values on server side are working fine, the time sourceTimeStamp
is  obviously not updated when just doing a  .set_value.

     # now adding some object to our addresse space
     myobject = objects.add_object(idx, "NewObject")
     mystr = myobject.add_variable(idx, "MyString", 'Hallo')

     ....
              mystr.set_value( 'some data')

How are variant (and SetValue) can be used to also getting the
sourceTimeStamp  to work? I don't understand how to link variants and
variables.
Can you please give me a hint?

in model_variable.cpp I found
-----------------------------------------------
    void Variable::SetValue(const Variant& value)
    {
      DataValue data(value);
      data.SetSourceTimestamp(OpcUa::DateTime::Current());
      SetValue(data);
    }

Thanks in advance,

 Udo


:rage4: where are the curly braces?


It won't work if you call this method twice


ImmutableList.of()


Why should it be? I need it in the `strongly_regular_graph` function.


Use one processor which checks ```isTests``` inside - wouldn't it be more concise?


it cannot do copy paste :(


`sub` is not used anywhere.


no longer works


llala


We should decide what macro we will use for OW for ARM architecture.
From existing logic it should be __ARM__ for any ARM sub-architecture and __ARM_...__ for specific sub-architecture. But I am not familiare with ARM sub-architectures.


@ayushagarwal95 Fixed in last commit..


LGTM.


idea is to have InputField and next to it button, that open dialog if needed, so you can copy&paste to input field or use dialog


It would be some DOM manipulation, but also some treading around the specifics (the CSS and transforms) of flip. So I don't think anyone who hasn't read and understood the source will be able to do something like this.

I agree the possibilities extend beyond simple cards, this is why I prefer to call them "flip elements".

This seems fair, I think if we focus on avoiding code duplication and move some stuff to functions we can avoid bloating flip.


Yes, my first implementation used Django's cache. But that serialized the value before putting it into the cache, which was breaking things. And it adds additional serialize/deserialize overhead on cache access (didn't measure the performance diff). I was a little surprised that Django doesn't have a cache backend that does exactly what this does.

Aside: I wonder if serialized model objects have a larger or smaller memory footprint than their deserialized counterpart?


For running 1-5 this is the expected and actual output:

**Expected:**

```
1
2
3 fizz
4
5 buzz
```

**Actual:**

```
1
2
3
fizz
4
5
buzz
```

- What needs to change for the actual to match the expected?
- The requirement was to if it was evenly divisible by `3` and `5`, not `15`.
- `console.log` should be directly next to the parenthesis when _executing_ a method or function. In a definition it's fine for there to be a space, but we haven't covered that yet.

Objectives:
- Make this code so it works without the modulus of 15
- Instead of 4 console.logs, try and use 1
- Fix console.log's
- Fix missing curly braces


Yep, that's why I made it as an option — enabled by default but can be turned off if performance is an issue.

Sent from my iPhone

> On 22 Jul 2015, at 19:42, Andrea Bogazzi <notifications@github.com> wrote:
>
> i can see the difference as well on my android screen.
> 5 inches at 1920x1080
>
> —
> Reply to this email directly or view it on GitHub.
>


Thanks for letting me know @JorgeFerrer :)


Identical to:

```php
$this->_version_major = $ver_mj ?: 0;
```


Wait does this mean it's done?


I currently don't know how to add a crafting recipe. Also, there's no JS in these mods. But I learned by copying what they did. And before you know it, I knew how to add custom blocks and items! :D you can also contact me on Twitter at @skullkeeper94


Detection of issue numbers in commit messages seems wrong if the project has a prefix.


I incorrectly thought this pom was used as the parent for plugins/pom.xml - but I'm wrong on this count :-/


@bos: Nice change!

Can you next time follow the github workflow, so create a branch, create a pull request, wait till TravisCI builds it, then wait a little so that other people involved may comment, then merge?

haskell-mode is a growing project and we try to have some procedures, for example your commit happened to break a build https://travis-ci.org/haskell/haskell-mode/builds/72236081. It is not the end of the world but could have been prevented.

And your change will not be mentioned in Month in Haskell Mode because it lacks a corresponding pull request.

Thanks!


yes, thanks, fixed in  ca35d81


It's a class attribute. `cls` is `Role` in this case, since `get_cache` is a class method.


Real divisor is effectively UBRR+1, and U2X "doubles the clock" (halves a pre-divisor 16->8 really, read the datasheet for more) meaning you need 2x the real divisor for the same clock when U2X enabled. Now when the real divisor is even, it can be represented without U2X with no change of speed since it can be integer divided by 2 with no remainder. Real divisor is even when brr+1 is even. Which means brr is odd, which means LSB set. Now it also happens to be that doing the conversion from UBRR value to real divisor and back isnt needed, you can just discard the LSB with shift right (>>1) and the result will come out the same: eg. imagine ubrr 123 + 1 = 124 = real-divisor-with-U2X. 124 / 2 = 62 = real-divisor-without-U2X. Thus UBRR value (without U2X) 61. But 123 >> 1 is also 61 (/2 is 61.5 but integer divide loses that .5). Thus you can do what the code does and gain better noise resistance and clock recovery when the baud rate happens to be nice.

Why am i explaining basic math again...


r+


Adding feedback here for you to have it:

For running 1-5 this is the expected and actual output:

For running 1-5 this is the expected and actual output:

**Expected:**

```
1
2
3 fizz
4
5 buzz
```

**Actual:**

```
1
2
3
fizz
4
5
buzz
```

- What needs to change for the actual to match the expected?
- Syntax for curly braces:

**Expected:**
```
if (i % 3 === 0) {
/* Do something here */
}
```

**Actual:**
```
if (i % 3 === 0) {
/* Do something here */}
```

Objectives:
- Make this code so it works without the `i % 5 !== 0` and `i % 3 !== 0`.
- Instead of 4 console.logs, try and use 1
- Fix curly braces


Also, the 'commits' menu item seems to be missing


I've seen a pattern in events that I quite like whereby the event is broken up into a namespace & the type of event.

<namespace>:<event>

So in this case it could be:

'login:success'
'login:failed'
'login:timeout'

Just an interesting approach I thought I'd point out.


@fuzzyami @doodyparizada @bolshoy please review. there isn't a pr for reasons you don't really care about


TeamCity Hyrise :: Unit Tests [Build 86](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=94&buildTypeId=Hyrise_UnitTests) is now running


The `__ARMEL__` macro is the "Linux"-ish way of indicating an ARM platform, similar to the `__AXP__` macro for Alpha.  I personally don't care what we use, but it might be convenient to stay close to GCC conventions for compatibility.

Sub-architectures with ARM will get confusing fast (Raspberry Pi's CPU, for example, is an ARMv6, but it also contains floating point hardware that isn't part of the ARMv6 standard...).  However, if we're focusing first on using ARM strictly as a host platform (in contrast to a target platform), I don't think we should worry about sub-architectures just yet.  Let's just get ARMv6 hosted tools (preferably without "hard-float" to improve compatibility) building and releasing first.


Things:

- [ ] make this into `defcustom`
- [ ] change the name to `haskell-process-load-or-reload-prompt`
- [ ] move to `haskell-customize.el`


changed email address


@avivl as well


Didn't realise Angular provided an isArray method. Might look into what other gems are on that global object!


change email address


What's the empty dependancy about? Does that have a meaning?


What is this `message` for?


Sure. It was just bugging me what that "!" meant. I assumed that you may wanted to know.


What is HCPO?


TeamCity Hyrise :: Unit Tests [Build 86](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=94&buildTypeId=Hyrise_UnitTests) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:04:49


They should, though I didn't explicitly test them.


Note


Will pepyatka break here if accessKeyId is not defined in config?


How about adding

def display(self, cmd):
             subprocess.check_call("/usr/bin/echo \"%s\" % cmd, env=self.cmd_env, shell=True)

And then calling that lots of times, to simplify code


This will make external layout files not be updated if changed?


Hi, I've just read on twitter that you are interested in user's opinions about new readme. Today is my first day looking up redux. I didn't read old readme so I could be a good readme tester :smiley:
*comment:*
Probably, there could be a React.render in this hello world example. I'm really a little confused about good architecture in react apps generally and looking for some good and simple example. It's ok for me how createSore() works, but my next question is: *what should I do with `store.getState()` further*?


Why not pass the `$options->mtime` in this case?


Add project to rep


normally, that should work. What can cause different result are different version of some packages. Be sure to use those version: flake8==2.3 pep8==1.6.2 pyflakes==0.8.1

this should make less difference.


This change has broken vagrant installations with Ubuntu and now requires an argument override in order to pass the -P for pip installation. If it is no longer possible to install salt stack with the setup script on ubuntu without pip surely the -P flag should just be assumed rather than the installation throwing an error?


This file isn't required, we've it already in our `core-base` MIBE.


Thanks for your input.
- I'll add the '^' ;-)
- didn't know that it was a group property, I will update that.

As to the multi-value property. Good point. What would be a good name for that?
 - "playbackSource" or just "source"? Any suggestion?
 - and the possible values would be "radio", "queue" & "line-in"?

My usecase is pretty simple. On the queue view of my app I just want to display if it is in use or not.

cheers


What's `update_config` for?


Oops, looks like this one made it into the master branch...


Yes, not enough.
So better compare both codes and make comment.


Typo Finanncial


It's possible. OK


Wait for confirmation by administrator


lol. That doesn't makes sense xD copy the content of the file into pastebin.com (textfield). And then submit it to get the link, lol.


don't you want `.get("config", "")`?


It's better use: oper->NameStr())


I believe this was already fixed in the omnibus installed by Rafal

https://github.com/conjurinc/omnibus-conjur/commit/1f7afb747b06896790ccbe5af401b23340a0417b

Just want to make sure we're not tripping over ourselves


Here is the error "local variable 'version' referenced before assignment" #41 isn't it?


`item.name` here, if not equal to `item.bosh_blob_name` may lead to errors because BOSH expects the folders inside the archives to have specific names. Probably we need to replace all the `item.name` values with the `item.bosh_blob_name` values and remove the `bosh_blob_name` property at all.


New password sent to your email!


update_config is a synonym for `change_config`.

------------------------
Alex Lomov
*Altoros* — Cloud Foundry deployment, training and integration
*Twitter:* @code1n <https://twitter.com/code1n> *GitHub:* @allomov
<https://gist.github.com/allomov>

On Thu, Jul 23, 2015 at 3:54 PM, Lev <notifications@github.com> wrote:

> What's update_config for?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Altoros/power-bosh-jumpbox-bootstrap/commit/100a7245c5d113c6012a913b67af6d8b17080c3e#commitcomment-12320061>
> .
>


Yes, it will be more concise.


@bocharsky-bw the good thing about the current example is that shows that you not always need a voter. A simple method in your entity will do the trick sometimes. However, it's true that we need to create a voter to showcase them. Maybe we can add a feature to edit comments and use a voter to check if the user is the author of the comment being edited.


i'd wondered about that. I believe `-` is not defined outside of the `[...]` but it won't hurt to escape it.


Wait for email confirmation


> And yes - I had several WTF moments - some javascript needs a C compiler...

C, gem, and etc. For example imagemin and etc libraries calls binaries if this binary absent, then they are downloading and building it! (Welcome to win/lin compiling hell).
This is web-development! :dizzy_face:


Fixed in 6b9f4c8058292facbef3cfa2fca7b1cf927fa2e3.


You're duplicating a lot of function definitions here, from nodeInLink() to addHolderContent(). Any reason why?


Hi, thanks! Redux is not React-specific, but we'll definitely have a hello world example in react-redux repository.


asdfasdfadsf


Hola Gustavo,                          Si definitivamente fue cambiado por error. En breve subo los cambios.

      De: Gustavo Muro <notifications@github.com>
 Para: ciaa/Firmware <Firmware@noreply.github.com>
CC: diegoVom <diego_vommaro@yahoo.com.ar>
 Enviado: Jueves, 23 de julio, 2015 11:30:35
 Asunto: Re: [Firmware] #293 Coding start (fbfc1f1)

Hola Diego.
Esto se habrá cambiado por error?—
Reply to this email directly or view it on GitHub.


Looks fine to me


I like idea to edit comment using voter :+1:


@64FF00 How?


Yes, it's.

We can look for a common place as this is used in multiple files.


I thought it's -mdb at the end.


this should be: `git clone https://github.com/LMBD/lmdb.git`


deleted file


Finally did. See further commits in PR


01: Plugin saves default config.yml when ```$this->saveDefaultConfig();``` is called
02: config.yml is saved again when ```$this->saveResource("config.yml", false);``` gets called


@psimyn This fixes the source-map error we see in react-router.


Check should be r > 0.


@64FF00 How come? I set it to ```php false ``` for a reason


@RandomAltThing Take a look at https://github.com/PocketMine/PocketMine-MP/blob/master/src/pocketmine/plugin/PluginBase.php dude ._.
```
public function saveDefaultConfig(){
		if(!file_exists($this->configFile)){
			$this->saveResource("config.yml", false);
		}
	}
```


@bolshoy you should've pushed to this branch instead of master, and then merge it. now i can't merge the statsd branch without having to merging all the other branches into statsd first (cors, newrelic, gzip, whatever else).

we're starting to have too many unmerged branches and its hard to maintain them all in an orderly manner. we should probably talk to manu and strive to merge them upstream.


Are you sure you must create a new CSVParser for each line?


I think that's an implementation detail. I'd still call this just "oracle". Whether it's an old or a new version doesn't matter too much, both should be reported as "oracle".


@RandomAltThing Config file only gets replaced when ```$this->saveResource("config.yml", **TRUE**);``` is called


No "Signed-Off-by:" please.


I noticed other appearances of -dbm in the readme - maybe scan the whole thing.


LOL


@RandomAltThing No it's not funny at all... seriously... :'(


@64FF00 How can I save and load stuff likes
```yaml
messages.yml
```
?


Не мог бы в кратце рассказать, что имено было поправлено?


The host address is specified when creating the virtual host so its not needed here. What could be used here is a shortcut to whether the connection is secure. Would that be a `lazy val secure: Option[Boolean]`? I don't think we should make assumptions about whether a connection is secure or not.


Yea good call i will merge that!

On Thursday, July 23, 2015, Justin <notifications@github.com> wrote:

> rock on. i wasn't aware of html_body. Should we use html_title for
> xml.title and xml.description?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SalesLoft/release-ninja/commit/7143d3243bdaa6cbf4a64d90b6f51d9a72e6b7f9#commitcomment-12318320>
> .
>

--

  Steve Bussey •SalesLoft

•Software Engineer

e steve.bussey@salesloft.com w salesloft.com
<http://salesloft.com/?v=1&utm_expid=76406993-1.E5ncJhwyTAW9uGaR1WdBTA.1?utm_source=Email-Signature-Rescue&utm_medium=Email-Signature&utm_campaign=Email-Signature-Rescue>

a 3423 Piedmont Rd NE, Atlanta, GA 30305

[image: Facebook] <https://www.facebook.com/SalesLoft> [image: Twitter]
<https://twitter.com/SalesLoft> [image: Linkedin]
<https://www.linkedin.com/company/salesloft> [image: wordpress]
<http://blog.salesloft.com/blog/> [image:
http://salesloft.com/prospector/#video]
<http://salesloft.com/prospector/#video>  [image: Software Engineer Logo]
<http://salesloft.com/?v=1&utm_expid=76406993-1.E5ncJhwyTAW9uGaR1WdBTA.1?utm_source=Email-Signature-Rescue&utm_medium=Email-Signature&utm_campaign=Email-Signature-Rescue>


xDDDDDDDDDDDDDD :D
LINK - http://pastebin.com/NZkWDnuu
Now it is hopefully right :D


https://github.com/javelinanddart/bricked-flo/commit/8c9f949a2ed577a852191bbf750a3a62e56aa81f
Most of these changes are trivial, there is some original work though I have to admit


Библиотека не должна падать, а должна выбрасывать исключения или возвращать результаты.


imho, ignore file width for such cases. Readability should be more critical rather than blind checkstyling.


Probably [keep track of the exact iteration count]...? (That's at least how I'd implement it if I needed to...)

Sent from my iPad

On Jul 23, 2015, at 07:19, Simon Warta <notifications@github.com<mailto:notifications@github.com>> wrote:

Okay given a user encrypted a disc in 1s PBKDF. How does he decrypt it? Run 1s-PBKDF again? Is the iteration count stored somewhere?

Don't get me wrong – I am just trying to understand the use case.

—
Reply to this email directly or view it on GitHub<https://github.com/randombit/botan/commit/30b0e3c88e94ba04c1843798f7ac74a008e01d9b#commitcomment-12318551>.


(been lurking, this caching stuff has been a longtime interest of mine for HQ)

I got burned pretty bad trying classmethod or instance level in-memory caching with invalidation (but admittedly, I did something really stupid with it in terms of never invalidating it). I think the 60 second timeout is a good compromise, but there still exists the corner case that if two or three successive requests hit all 3 different django web-workers, then these will all be cache misses. I think django doesn't implement this in-memory method because caching usually is necessary once multi-webworker scalability falls short and some sort of shared model (using memcached or redis shared across all workers) is necessary.

As such, the serialization/deserialiation and network costs of usiing redis are so negligible vs. hitting the database. I thought this PR was just using Danny's quick-cache of "avoid hitting DB again during lifecycle of request" which is an appropriate thing to even use to avoid hitting redis/memcached


@RandomAltThing Place a .yml file in the resources folder and use
```$this->plugin->saveResource("custom_config.yml");
$this->customCfg = new Config(PluginBase::getDataFolder() . "custom_config.yml", Config::YAML, [
        ]);```


:dancers:


is this branch even relevant? there's another cors middleware branch. which one is the one we use?


`haskell-check-paint-overlay`


As you appear to be working on unicode issues (great work, btw!), and to save me asking after you do a PR, do you mind looking at line 700 whilst you're on this - it's doing exactly the same as your new code but it's decoding the string whilst your new code is encoding - only one can be correct, and I don't have a system setup to test unicode stuff atm. Cheers.


I notice that there are quite a few unimplemented methods in there that look to be reasonably straightforward, would you like those added too?

Not quite sure what there is to test for them, as the wrapper is pretty thin.


Can fix that, no problem.


Me too!


this branch has the gzip and newrelic branch merged in it as well, it's a mess


NO


set_current_person do not need to put here,

it is a before action in application controller


This commit reverts the commit 86eb438f (Updated lua 5.2 compatibility) and removes the Lua 5.2+ support again. Was that intentional?


@64FF00 Thanks. I have been wondering this for my CrateKeys plugin :/ :+1:


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.804.28a30f8a323fec23ac55b6e311ee38a6d79dd8fe](http://build.palaso.org/viewLog.html?buildId=59952&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:04:12


@avital awesome <3


I think it will be useful to display on the top the item on which the `M-o` action is about the happen. It will be especially useful when doing actions like file deletions.

An idea is to display something like:

```
<selected element>
<binding1>: <docstring1>
<binding2>: <docstring2>
```

In my case, it would be something like:

```
~/temp/temp.txt
o: default
x: delete
```


You fetch [here](https://github.com/Altoros/power-bosh-jumpbox-bootstrap/blob/100a7245c5d113c6012a913b67af6d8b17080c3e/roles/binaries-builder/tasks/binaries-builder.yml#L15) files that tend to have following names `{{item.name}}.tar.gz`.

And [here](https://github.com/Altoros/power-bosh-jumpbox-bootstrap/blob/100a7245c5d113c6012a913b67af6d8b17080c3e/roles/binaries-builder/files/scripts/update-configs.sh#L17) you expect this tarballs to have names built from `bosh_blob_name` [[link](https://github.com/Altoros/power-bosh-jumpbox-bootstrap/blob/06f008a77741d183eac44ebe7c7931d63f9bc04c/roles/binaries-builder/tasks/binaries-builder.yml#L32)].

This will fail in cases when `item.name != bosh_blob_name`.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.805.6a1229f2e3e59324d7ec7e6cc7cd7a52fcf35830](http://build.palaso.org/viewLog.html?buildId=59953&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:03:29


Missing space after the "."


Also this fix allowed me to pass this steps.


Also can `C-g` after `M-o` be used to gracefully exit back to the ivy state from where `M-o` was called in the first place?

Right now it says: `ivy-dispatching-done:  ^G is not bound`


The kernel calls this "unicast flood", shouldn't the option be "UnicastFlood=" too?


https://trello.com/c/1TgzNKYD/59-circle-c-to-r

去除瀏覽器報錯


To be fair, it's also by far the most complex chanceinc there is.


@tkQubo Polished by applying your feedbacks 336bb87b5371ff857883fc5eb1da7bff62c72f3f Thanks a lot :100:


s/allowed/allowing?


See issue #4.


duplicate


s/is/if?


@brasmusson this seems to have broken the release process:

```
$ mvn --batch-mode -P release-sign-artifacts release:prepare -DautoVersionSubmodules=true -DdevelopmentVersion=1.2.5-SNAPSHOT
[INFO] Scanning for projects...
[ERROR] [ERROR] Project 'info.cukes:java-wicket-main:1.2.4-SNAPSHOT' is duplicated in the reactor @
[ERROR] Project 'info.cukes:java-wicket-main:1.2.4-SNAPSHOT' is duplicated in the reactor -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DuplicateProjectException
```


You probably don't want to store your Kadira API keys in a place where other people can see them.


Excellent move, runtime dependency on a spec folder :)


Why is everyone obsessed with making my crate keys plugin? Lol


sure, sorry about the confusion. here is a version of the data viz that renders correctly http://bl.ocks.org/tdhock/raw/5e1e0fbcb37e6d2072b1/

and this is the version of animint that was used to make it
https://gist.github.com/tdhock/5e1e0fbcb37e6d2072b1/raw/482ebb3a7083e5e61a128a8d0c0bd902242effa9/animint_variable_value.tar.gz


thx


It works like a charm :-). You are an angel!


Driveby: camelCase?


Ditto: camelCase


not that it matters, but these macros come from a later commit, right?


And here


nice!


Fixed on commit 879d63c8c8347cbf3e3ce430817ecd6f5955e66e


@XFusionios **Excuse me?**


@caijun now the test includes a vector of expected number of downloaded chunks for each geom. Does this help?


I'm not even sure I understand why we need to release the examples. I always drop them from sonatype anyway before syncing to central.


@ccoenen gem `devkit` is not RubyInstaller's DevKit but something else.

There is also the conditional about the dependency on Windows platform which is unnecessary (isn't this gem Windows-only) :smile:

Cheers.


I'm porting the PC version, Mystery Crates


Your one is probably shit


Yes, I think that the default method hashes the pointer, and not the contents.  As for the same pointer for identical strings, in this case the answer is yes but for the wrong reasons.  The frontend should guarantee that identical modules have the same pointer. :)


what's the command to run after VERSION update? ... `mvn verify` did not work for me


saturating add doesn't seem right, feels like checked add would be right?


Ahahahahahahahahahahhahah

.....

Ahahahahahahahahahahahahahahahahahahahahahahahahahahhahahahahahahahahahhahahahahahahahahahahahahahahahahahahahahahhahahajahahahahahahahahahahahahahahahahaha+ahahahahahahahshxhahhdbfjfnfjd xjd sksnsjs


can we add an assert that unsafe count >= 1 here?


Django does have such a cache:
https://docs.djangoproject.com/en/1.8/topics/cache/#local-memory-caching


ugh it's annoying that we have to track unsafety state twice


pre-existing, obviously


Yeah mine is shit compared to the real thing. But at least I know what saveResource does


I agree - your fix allows us to pass this step. But the issue is that BOSH won't be able to compile these packages.


No widgets APIs exposed by view


Ha ha!! Good point @tboerger. Has this bitten you?


Thanks, updated.


Hey @IshmaelBelghazi this is good that you added a warning so the user can know if the algorithm did not reach the desired optimality threshold.

As we discussed at our last meeting can you please create a Pull Request so we can have an online discussion about your glm code?


Good catch (it was because I was rewriting sh test).


@@ -48,6 +48,129 @@ You have successfully disabled the application.


Done.


Constructor injection in order for easier instantiation with mocks in tests.


Reuse DefaultErrorCallback() from uberfire.


This commit breaks the existing codes, so I'm merging this PR except this commit


applied code review comments


+### Packages
+
+

+
Package Name	Description

+

+
com.andrew.apollo	 Music Player

+
com.android.apps.tag

+
com.android.backupconfirm	 Restores Google settings

+
com.android.browser	 Stock Browser

+
com.android.calendar	 Stock Calendar

+
com.android.camera2	 Stock Camera

+
com.android.captiveportallogin

+
com.android.cellbroadcastreceiver	 Alert Broadcasts

+
com.android.development	 Development App

+
com.android.dreams.basic	 Screensaver

+
com.android.dreams.phototable

+
com.android.email	 Stock Email

+
com.android.exchange

+
com.android.facelock

+
com.android.galaxy4

+
com.android.gallery3d

+
com.android.htmlviewer

+
com.android.inputdevices

+
com.android.inputmethod.latin

+
com.android.magicsmoke	 Live Wallpaper

+
com.android.managedprovisioning	 Google Workplace

+
com.android.mms	 Stock Messaging

+
com.android.moto.appdirectedsms

+
com.android.musicfx	 Audio Equalizer

+
com.android.musicvis	 Live Wallpapers

+
com.android.noisefield	 Live Wallpaper

+
com.android.pacprocessor

+
com.android.phasebeam	 Wallpaper

+
com.android.printspooler

+
com.android.providers.downloads.ui	 Downloads

+
com.android.providers.partnerbookmarks

+
com.android.providers.userdictionary	 Dictionary for Keyboard

+
com.android.proxyhandler

+
com.android.quicksearchbox	 Search Box

+
com.android.sdm.plugins.connmo

+
com.android.sdm.plugins.dcmo

+
com.android.sdm.plugins.diagmon

+
com.android.sdm.plugins.sprintdm

+
com.android.smspush	 Tethering connectivity

+
com.android.soundrecorder	 Sound Recroder

+
com.android.sprint.hiddenmenuapp

+
com.android.sprint.lifetimedata

+
com.android.videoeditor	 Movie Studio

+
com.android.voicedialer	 Voice Search

+
com.android.vpndialogs	 VPN System

+
com.android.wallpaper.holospiral	 Live Wallpaper

+
com.android.wallpaper.livepicker	 Wallpaper Picker

+
com.android.wallpapercropper	 Wallpaper Cropper

+
com.bel.android.dspmanager	 Audio Equalizer

+
com.cyanogenmod.account	 Account Manager

+
com.cyanogenmod.CMWallpapers	 Cyanogen Wallpapers

+
com.cyanogenmod.eleven

+
com.cyanogenmod.filemanager	 File Manager

+
com.cyanogenmod.lockclock	 Lock Screen Widget

+
com.cyanogenmod.trebuchet	 Launcher

+
com.cyanogenmod.wallpapers	 Wallpapers

+
com.facebook.katana	 Facebook

+
com.google.android.androidforwork

+
com.google.android.apps.books	 Books

+
com.google.android.apps.cloudprint	 Cloud Print

+
com.google.android.apps.currents	 Currents

+
com.google.android.apps.docs	 Drive

+
com.google.android.apps.docs.editors.docs	 Docs

+
com.google.android.apps.docs.editors.sheets	 Sheets

+
com.google.android.apps.docs.editors.slides	 Slides

+
com.google.android.apps.enterprise.dmagent

+
com.google.android.apps.fitness	 Fit

+
com.google.android.apps.gcs

+
com.google.android.apps.inputmethod.hindi	 Hindi Keyboard

+
com.google.android.apps.photos

+
com.google.android.apps.plus	 Google+

+
com.google.android.apps.walletnfcrel	 Google Wallet

+
com.google.android.email	 Stock Email

+
com.google.android.gm.exchange

+
com.google.android.googlequicksearchbox	 Search Box

+
com.google.android.inputmethod.japanese	 Japanese Keyboard

+
com.google.android.inputmethod.korean	 Korean Keyboard

+
com.google.android.inputmethod.pinyin

+
com.google.android.inputmethod.piyin

+
com.google.android.marvin.talkback	 Accessibility

+
com.google.android.play.games	 Play Games

+
com.google.android.street	 Street View

+
com.google.android.tag

+
com.google.android.talk	 Hangouts

+
com.google.android.videos	 Play Movies & TV

+
com.google.android.webview	 Webview

+
com.google.android.youtube	 Youtube

+
com.google.earth	 Earth

+
com.motorola.android.buacontactadapter

+
com.motorola.entitlement

+
com.motorola.motocit

+
com.motorola.motosignature.app

+
com.motorola.service.ims

+
com.motorola.triggerenroll

+
com.motorola.triggertrainingservice

+
com.qualcomm.atfwd

+
com.qualcomm.qti.rcsbootstraputil

+
com.qualcomm.qti.rcsimsbootstraputil

+
com.qualcomm.timeservice

+
com.svox.pico	 Text to Speech

+
com.tmobile.themechooser	 Theme Chooser

+
com.tmobile.thememanager	 Theme Manager

+
com.verizo.permissions.appdirectedsms

+
com.verizon.omadm

+
com.verizon.permissions.appdirectedsms

+
com.vzw.apnservice

+
jackpal.androidterm	 Terminal Emulator

+
net.cactii.flash2	 Flash Light

+
org.codeaurora.ims

+
org.cyanogenmod.audiofx	 Audio Equalizer

+
org.cyanogenmod.launcher.home	 Launcher

+
org.cyanogenmod.theme.chooser	 Theme Chooser

+
org.cyanogenmod.themes.provider	 Themes Provider

+
org.cyanogenmod.voiceplus	 SMS through Google Voice

+
org.cyanogenmod.wallpapers.photophase	 Live Wallpaper

+
org.whispersystems.whisperpush	 Secure SMS integration

+

+


can't we move this check into the helper, rather than repeating it twice?


Simplify view logic: instance id / process id text are never enabled -they're just read only, so this can be done in view's init method. The rest was just deduplicated.


$name = $obsah_stranky->find('span[itemprop=name]', 0)->plaintext;


Awesome, thanks! :+1:


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.806.95062e0968bf8038fdc50f2a200649fb7f8201ff](http://build.palaso.org/viewLog.html?buildId=59954&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:40


@yihui 不急不急，正在看第一版，借期一年。馆藏1本，下位童鞋得有耐心。:grinning:
BTW, 新版图书已经上线: [Dynamic Documents with R and Knitr, Second Edition](http://book.douban.com/subject/24654324/).


Why should I use react-redux? You should add it to current and future readme. BTW, why? :smile:


That's a fix for an a bug in the presentation compiler, but it needs a test. I'll try to dig up the issue.


Yes it has bitten us. our RPMs split out the testsuite from the runtime package.


@zzpianoman We found out that the real patch that fix this issue for your device is:
https://github.com/CyanogenMod/android_hardware_ti_omap4/commit/a512755c3fd3f0aef1e4eb6ead5d21c7ea85834c

Unfortunately part of that code won't work for our device, as you can read in commit's comments.

The good news is, we finally have our camera libs source code:
https://github.com/pauldmps/vendor_st-ericsson_u8500/blob/5c3c8f4491a9996429c52ecc946bf76d3c4fc5a8/multimedia/video/components/h264enc/proxy_ddep/src/H264Enc_Port.cpp

Any eventual help to our cause would be appreciated, because we're not experienced devs.


"Note that each process will have its own private cache instance, which means no cross-process caching is possible. This obviously also means the local memory cache isn’t particularly memory-efficient, so it’s probably not a good choice for production environments. It’s nice for development."


typo


@Mrbrianojee I've just released [v0.7](https://github.com/raymondwanyoike/django-intensedebate/releases/tag/0.7) and [v1.0](https://github.com/raymondwanyoike/django-intensedebate/releases/tag/1.0). Both support Django 1.8. Use v0.7 if you're coming from v0.6.


I know - don't know how it got there and already removed it.


Is that the "full blown namespace"?


I'm not 100% happy of the way this is done, but I had to overcome some specific parts of this ambush:
- the ambush NPCs are not unique to this even sincer they are also used dozen of times as trash NPCs within the same instance. So this limited the ways to script them
- the NPCs are supposed to spawn at each end of the bridge, move to the center (or to the player if available) and then have random waypoint around the center. However, when setting random movement after a spline movement, the MovementGenerator allows random movement around the spawn point, making the NPCs return to it. This should not be the case here. The only way (to my knowledge) to work around this was to use DBScripts + Waypoint to have random movement around one of the WP. This is only possible here because no other NPCs of the same entry use WP and they are not summoned elsewhere (at least for now...)

I'm quite open to a better approach if someone has it. But at least, this script does the job.


$obj is not a good practice


Ich würde es bevorzugen, wenn doch private Nachrichten frei zwischen Usern ausgetauscht werden können. Bleibt die Frage: Private Nachrichten im Rahmen eines Spiels?
- Entweder blendet der Client die privaten Nachrichten unterhalb des Spielfelds ein, sofern die Nachricht vom Gegenspieler kommt
- Oder man definiert nochmal separate Befehle, um im Spiel zu kommunizieren (bzw. macht diese Änderung rückgängig und definiert neue Befehle für private Nachrichten)


Probably not, thanks for noticing. As for the support for the module, LuaDist will soon use LuaRocks rockspec modules and only libraries and special modules will be in the LuaDist repository. This alt-getopt repo will soon be obsolete.


Where is this file?


Ok, I did that -- azure + ec2 are happy with negotiate auth working correctly as well.


yes it is :)


https://adaptlearning.atlassian.net/browse/ABU-969
updated in component here https://github.com/adaptlearning/adapt-contrib-narrative/pull/132
current icon font size is too large for alternative layout


I'm not sure if this should be SD2's job to tell world DB about movement/DBscripts content, but at least you get the whole idea behind the patch this way. Feel free to remove it if needed.


We currently support back to ember-data 1.0-beta15. This will only work on ember-data 1.13 and later.


Could make this clearer, have two separate tests. One for strike bonus and one for spare bonus


Similarly, this will only work on ember-data 1.13 and later.


Lol :)


Comment out the following lines (do not set to zero) in order to use the current.


I don't really follow what's happening here :( but I trust you! :)


wow!


This is a major change to the change watcher, far outside the scope of checking the rev to avoid unnecessary reloads. This will make it immediately load any record it encounters into the store. We had an extended discussion on #50 about why this is a bad idea for general use.


Wrong call. Use following:

    void  LoadTomvizExtensionsBehavior::onServerLoaded(pqServer* server)
    {
    ...
    vtkSMProxyDefinitionManager* defnMgr = server->proxyManager()->GetProxyDefinitionManager();
    defnMgr->LoadConfigurationXMLFromString(tomvizXML);
    }

I think `LoadConfigurationXML` on `vtkSMProxyDefinitionManager` should be renamed to `LoadServerManagerConfigurationXML` to avoid ambiguity with the GUI configuration XMLs that `pqApplicationCore` loads.


Ah oh dear. Can you send us a PR to move it into lib? We can get it out in the next release for you.


Thank you for sharing your thinking! It makes sense to me how you have it changed here, I was just getting confused between the commit order and couldn't figure out what the final verdict overall was.


That's a bit ugly since you'll end up hiding the container again and again for each subfield. It's the easiest way though  so I guess it's an accepable drawback. The other/cleaner option would be setting the data attrs on the `.i-checkbox` container and adapt the JS to handle this case.


Well, you've got my blessing!  :)


TeamCity Undertow :: Core [Build 1398](http://teamcity.cafe-babe.org/viewLog.html?buildId=18165&buildTypeId=bt35) is now running


actualizacion


Actually... the JS change might be pretty small.

Right now we have

    field.prop('disabled', !active).closest('.form-group').toggle(active);

Something like this mighty do the job:

```
field.closest('.form-group').toggle(active);
if (!field.is(':input')) {
    field = field.find(':input');
}
field.prop('disabled', !active);
```


could extract into separate method. isAStrike(i) {}


Similarly, `hasDirtyAttributes` is only available on 1.13 and later.


gah. why oh why are keywords still not their own tokens!


At a glance it feels like some of this code is duplicated in other parts of the connector ... correct me if i'm wrong ... can we black box as much of it as possible into a method and then reuse?


could be isADoubleStrike(i)


Same for spares


Yes. FileLocator understands vtl files so this works. I first need to open a FileLocator to get the version to pass to run_file. For vtl files without a version it should be enough to load the pipeline and check if it is valid. This makes the test pretty fast, the slowest being a terminator example.


sdlskfsdjsdflsfdkfdsk  slsldkf


Then we will need to write a better scripts.


Can use reduce instead of a loop


LGTM (as far as I can tell).


I think this should be concerned whether it executes on the node...


Used the following instructions https://gist.github.com/garethrees/096cd19f785143983524


wrong import order once again

pycharm protip: always check your imports after auto-generating one


fixing!


I think you can remove these deprecated warning as the whole class is deprecated.


no need for that empty lien


Some parts of this patch could have been done in a easiest way or by ACID/DB but they are ground work for coming scripts also in the Grim Guzzler.
This is also why I'm pushing this patch while the next are not complete: having some early feedback would reduce the amount of rework if changes are required.

Coming changes (also in the Grim Guzzler):
- Hurley Blackbreath
- Survey patrol when one of the patron is killed
- Plugger Sparring
- Mistress Nagmara and Potion of Love quest (maybe, depending on data since this was removed)

Note that ACID for NPC 9502 (Phalanx) should be dropped with this patch.


Why did you add asynchronicity here?


`attachments = ...` - you never use more than one source now


don't call `.all()` if you iterate over it immediately. it just adds overhead to materialize a list


Some conversation points with this change:

1. We are not evaluating parameter expressions. This means that `value: data.csv` will be included while `data: !r file.path("a","b.csv")` will not. We could capture this if we `knitr:::resolve_params`; any concerns with evaluating these expressions (yet again)?
2. We are treating these resources as non-web.
3. We are only processing parameter values if they have `input: file`; we could process all `character` data.


@XFusionios Lol


What we design now can be later used for OW compilers if we design ARM code generator and ARM architecture will be really fully supported.
We should not break standard OW rules and we should follow existing one if it is possible.
Generally OW architecture macros were derived from MS compilers.
By example in the past we were migrating \_\_86\_\_ and \_\_I86\_\_ macros to \_M\_I86 macro overall.
For sub-architecture we can define by example \_\_ARM6\_\_ or can define value for \_\_ARM\_\_ macro which represent sub-architecture or both.


@istufin Mainly this $DROPDOWN_CONTAINER and


why do you need this? Shouldn't the stack only ever return Gap::PERIPHERAL for an S110?


`https` has worked for me in the past, and still works as far as I am aware. It could be because I have read and write access to that repo though.


fixed via 1fd4c7ea3e9e0184a3bd6c1aaace0778f6eb6ce5

sorry guys.


that `getattr()` call is quite ugly for two reasons:

- it's huge (thanks to the great legacy naming)
- you ALWAYS run `att.folder.linked_object.getOwner().getAdjustedStartDate`, too, since `getattr()` is just a function and thus does not have short-circuit evaluation

assuming it's for subcontributions, why not add a helper function like this to the file?

```python
def _get_start_date(obj):
    if isinstance(obj, SubContribution):
        return obj.getContribution().getAdjustedStartDate()
    else:
        return obj.getAdjustedStartDate()
```


Oh shit... i want to change but still there ...


And these DROP_TRIGGER AND TOP_LEVEL_LIST, basically anything involved with the old dropdown


Ah.  Makes sense.  Thanks.


if the form validation is taking care of it, this check is not really necessary


@djindjic I'm not sure I understand the question. Redux is a state management library. If you want to use it with React components, you need to use react-redux to bind them together.


We also want to include attachments from stuff inside that session. So you need to get attachments linked to a session/contribution/subcontribution and then check if their linked object is the session or inside the session.


is there any way to test that no intermediate is produced? guess not.


same here - form validation should take care of it


form valication again ;)


or at least not without dirty hacks that read the return pointer


without `.all()`


helper function


not very pretty. use `obj=FormDefaults(filter_type='all')` in the form constructor


s/str/unicode/


This wasn't the case in 0.12, but I think we had good reasons to split React-specific part. Its API is completely independent of Redux core and is not as good as we want so we need to keep iterating there. Much easier with a separate package. See discussion here: https://github.com/gaearon/redux/issues/230

You don't *have to* use it. But it's handy for connecting React components to Redux. :-)


return this.rollScore + this.bonus;


I would add a check if file permissions allow rename, or maybe check if rename fails.
Otherwise we still have to display the silly old message. As the config file is in webroot, it would possibly fail quite often as several providers do not allow write access to webroot for scripts.


Just as a heads-up up front: I will continue to change the signature of these functions. More patches to come, so you may be best off sticking with last week's version of the repository for the moment until I've arrived at the final interface.

My mental model is that these `InternalData` structures are ones that are filled once at the beginning in `get_data()` and then just read from in `fill_fe_values()`. That is, they hold information that can be pre-computed. This works because we call `fill_fe_values()` once for every cell, but there is no information we typically want to store on one cell that has any meaning on the following cell.

I don't know what you want to store in the `InternalData` object in your case. I suspect that it is scratch arrays etc., for computations *within* `fill_fe_values` and possibly the functions you call from there. I thought about this a bit, and my suggestion would be to mark the member variables you want to use as scratch arrays as `mutable`, so that you can write into them even though the `InternalData` object is `const`.


It's no longer present in the library.
The data is at https://github.com/unicode-cldr/cldr-core/blob/master/supplemental/telephoneCodeData.json, you can copy the mapping to your own class, or use a more feature complete library such as https://github.com/giggsey/libphonenumber-for-php


I'm not sure if you sent that to the right person?

On Tue, Jul 7, 2015 at 10:05 AM, jince7 <notifications@github.com> wrote:

> the guest list challenge code is wrong. Could I please have the correct
> code
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GeekTrainer/Introduction-Programming-Python/commit/a710c9a9309be1a67f23f9e04e6cfe47a74e92f6#commitcomment-12040087>
> .
>


Plus you often cannot change permissions on webroot.


`day_number` sounds a bit very verbose. i'd call it `offset` or even `n` (it's quite clear what it does based on the context)


Baslangic tarihi: 23.07.2015


"Specific days" might sounds better here


I agree, we shouldn't break any standards.  However, let's also not jump into this too quickly.  I'm not overly familiar with ARM, so we should look at some compilers that have good support for ARM to see what they are doing.  ARM is a complicated architecture simply because it supports both big- and little-endian operation, has a large variety of architecture versions (including with and without floating point units), and comes in 32- and 64-bit variants.  It's nearly as messy a family as x86/AMD64.  We should take these steps slowly before leaping in.

This simple patch allows boostrapping Open Watcom using either GCC or CLANG on ARM, so I don't see much reason to modify it.  We could maybe use `__ARM__` in this commit without any problems.  Again, though, this patch doesn't address even constructing ARM-hosted tools.  We should open another issue for support of ARM as a host.


those fields should also have `DataRequired()`


you aren't using the ID anywhere


Was there some purpose to this change?


I'd exclude choices with nothing to select


Let me make sure that the `__ARM__` macro works fine with GCC, and I'll change this code again to use it instead.


these conditions can probably go away and you can use `{{ form_rows(form) }}` to render all the field when you adapt the `filter_type` field to only include choices that actually make sense


you are not assigning anything. if all you do is call some functions you don't need to wrap it


Some more refactoring is ncessary (not to demotivate you):
* Please rename upload.sh to upload_sources.sh, to make more clear what it does.
* Make sure all the indent alignments are correct in the functions (like getinfo and upload)
* Only use capitalized variable names for (static) global values
* 'dpis' is a property, so I would expect this to be part of getapkproperties
* Don't let functions inter-depent on each other's variables assignment if it can be avoided and is unsuspected (e.g. of the 'get' function you can expect it)
* Don't use -a on commit, because it will add a lot more, very possibly unwanted, changes to the commit. Try to figure out a way to do a 'git rm $oldapk' somehow to delete the apk that will be replaced. (Hint: I think you can use 'git rm' on the foldername that the old apk would have been in).

So, in short:
* get rid of getinfo and put a line 'apk=...' directly in the for-loop
* pass $apk then as a paremeter to getapkproperties
* use within getapkproperties "$1" as the apk-variable
* also use the parameter and $1 with the upload function.


I really like the new documentation structure. Very good job @gaearon ! I am so excited to start working with redux and react for the first time. In the future, when I know what I am talking about, I will contribute too.


I'd always heck <= 0, rather than == 0, just in case something is a signed or so... After all you write "greater than zero" in the comment, rather than "equals zero"... But this of course doesn't matter.


Looks fine.


I was thinking about this earlier today.  It may be what you meant by "read the return pointer", but nonetheless, I was wondering if we could extract the size of the function's stack frame, that would be enough (i.e. if I then added a case that did `box ReallyBigArray` ...)


Steve, can you modify this to `elsif update`

that way it only complains IF they send an update that we don't understand?


Looks good!


@michaelcahill, if `wiredtiger_open` sets the flags, and we don't re-set them in future calls, would that be safe?

I agree configuration is a cleaner solution, but we'd have to add configuration for things like `WT_BTREE_NO_CHECKPOINT` (OK, we'll want a configuration for that anyway at some time in the future), and `WT_BTREE_LAS_FILE` (much less interesting from a configuration point of view).


Looks good to me.


The worst part is Coverity wasn't even complaining about this very line. Here a `PVOID` is being passed as `%p` - perfectly okay and Coverity was happy too. The CID only included the other three lines.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.807.e6580880bfaee8ed602a838dc6b0bd4b68738b85](http://build.palaso.org/viewLog.html?buildId=59955&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:35


Should be a "<" rather than an != no?


@benrudolph They should show up now, as of https://github.com/dimagi/commcare-hq/pull/7199.  At least they do for me.  Ymmv, so try it out locally - I linked to a few resources about making them show up in that PR.


Yeah, and the post-check for overflows should be there too.


Good idea to separate frame from the game


Hmm, given that we use this loop here twice, and if I am looking right in a subsequent patch even once more, we should probably turn this loop into a call of its own?


Hello,

thanks for your fork.
Could you add the dependencies, version and so on please.


yeah I think this was just an artifact of my workflow in how I broke up the commits, and this leaked a bit early.


I tried the LocMemCache backend. It serializes objects before storing them in the cache, which was causing related objects to be re-queried every time an object was pulled from the cache. I subsequently [worked around django relationship naiveté](https://github.com/dimagi/django-prbac/blob/1c457e08b73fcea2925e375728714122322835f6/django_prbac/models.py#L92-L96), so maybe serialization wouldn't break things anymore?

@dmyung, thanks for the input. Very helpful.
> there still exists the corner case that if two or three successive requests hit all 3 different django web-workers, then these will all be cache misses.

Indeed. However, the number of DB hits should be at maximum 2 per cache period per web worker (although, they are larger queries than before), which I think is much better than the 10s or 100s we had previously. This caching implementation is built on the (maybe naive) assumption that HQ has more than one request per minute on average. If not, maybe we should increase the timeout? Profiling after this goes live will be necessary to convince me that it is actually making a positive impact :)


Does this line make sense when we have the sine wave dataset?
I got an error (x_mask is 2D tensor, while h[d] is 3D tensor

Btw, when we have a mask that is not 1 everywhere, are we sure that we are keeping the correct last value of the state for carrying the hidden_state?


I don't think this is a real problem. It would only present itself when using the two modes  simulteanously. As the same class is used, it currently cannot assume to be session scoped or else the standalone mode would fail on concurrent access. For this to work, the XPageApp would have to check if it has been put in session and act in consequence. It would then be simpler to just use the session directly and not use instance fields.

> Le 23 juil. 2015 à 12:50, Jon Harper <notifications@github.com> a écrit :
>
> StandaloneAppService is updated to not use the instance stored in the entry, which
> might break apps relying on xpage not being per session and using the
> StandaloneAppService. This can be dealt with by using a spring managed
> xpageapplication with singleton scope, if necessary.
>
> If you want the current behavior (independant beans between Portal.jsp?page= and RunStandloneApp?page=), you would have to create 2 beans, one with the scope session for Portal.jsp, one with the scope singleton for RunStandaloneApp. But the code will always get the bean named 'entry.getPluginName( ) + ".xpage." + entry.getId( )' so it won't work.
>
> I don't know if this this feature (might even be called a bug ??) is used, but I'm stating it so that we make a conscious decision to drop it.
>
> —
> Reply to this email directly or view it on GitHub.
>


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 196](https://teamcity.labkey.org/viewLog.html?buildId=262244&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 16, passed: 3 Build time: 00:06:56
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

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

org.labkey.test.tests.CDSTest.verifyFilterDisplays: <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.verifyHomePage: <no details avaliable>

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

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

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

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

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

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


Ok, thanks! :smiley:


Hello @rande.
Is this commit included in the next 2.3 release ?

Cheers


conn->is_closed() returns 1 only once. Need to fix that. Rewrite it like it is in debug.pike.


Thanks for the reply. I use a very similar data structure to fe_poly_tensor, although I don't make use of cached variables, just shape_values, shape_grads, etc..  so using mutable is probably the obvious the answer to this issue. I'll move my version to last week until you've finished and make the changes then.


Hmm, why this last hunk?


is this variable used?


If we have a `caml_is_marked(obj)` function which returns true if `obj` is marked, would conditional marking just be

    if (caml_is_marked(obj))
      caml_darken(val,0);


Will this be part of the QA/cleanup task?


will do.  (i won't get into my theories of how this code ended up this way...)


If we're going to use Bootstrap, we'll want to save it as a dependency:

```sh
npm install bootstrap --save
```


It is https://gitter.im/scala/scala?at=55734a28813c577e1cf57fef, extension methods on `Array[Int]` not included in the completion results.


i got this warning in the FE when i placed the plugin on a page.
the message is a warning that actions are public..


I'll give `checked_add` and `checked_sub` composed with `expect` a shot, okay.


maybe we can make `.get` always synchronous, and introduce `.fetch`, which always returns a promise, fetches the data from server, and updates local cache. Still not happy with all this, I think we can do better. I'm also not happy that `.get` and `.set` look so alike, but one is synchronous, and on is a async :/


@chenghao-intel , will this cause twice evaluation of `expr` on the same input line? why not eval once and store in a tmp val? Did I miss something?


Yes but it throws me an error saying "Use array as string", I believe its because you're using original Html helper instead of *yours*


Ah, das ist noch ein guter Hinweis. Tritt vermutlich vor allem dann auf, wenn man kein Passwort eingegeben hat am Anfang, oder? Muss ich mir mal anschauen.


i know, it burns my eyes every time I work on this bit of code in the parser.


Better turn this loop into a function of its own, see comment on earlier commit...


yes, good points.


Does the `col` class do anything, or should we be using something like `col-md-12`?


@esoergel This commit removed a Hindi translations for custom reports in up_nrhm domain.


No, that was caused by an earlier change (#4345). However I'm considering allowing shadowing `using Base` (see #12183).


Hi @Th3Mouk.

Mope, 2.4 only.


if(phase == kLoss) {

} else {
  src= .....(... , phase);
  Tensor<cpu, 2> dst;
   if (phase == kPositive)
      dst = ... data_
   else
      dat =...hid_sample_

}


this is a mistake; we want revXXX represent our releases, independent of sigar version


@michaelobriena What was the intention behind this in the first place?


look ok


Perfectly, thanks!


Why is this `sudo` necessary? All the files under this directory should be created by the same user that the script runs as. We should avoid sudo when possible.


Diese klasse soll die Function MD5StringHash ablösen um im Konverter und Brute-force andere Verschlüsselung Methoden neben MD5 benutzen zu können!


Yes, you are right. I was in a rush to get something out the door for our own needs. Some of the code was basically pulled out of the `automigrate` function. That function could now be refactored to use the new `setupIndices` function to handle creating the indexes.


Okay, I believe things are the way they should be now. Please let me know your thoughts.
-Max


devrait être `field` au lieu de `fied`.


Why not just change `titles.keys()[0]` to `list(descriptions.keys())[0]`?


nit: older formatting was easier to read -- my first thought was 'why did he remove the call to collect?'


again


Typo - Episode ;)


Activation key has been sent


Looks good but where's the bibliography.


Could you put this example (lines 37 to 44) as a docstring for the function excel_table_generator ?
See cn_extract_data.py for examples of docstrings


Thanks! fixed now


`query` au lieu de `guery`


ok, already pushed to upstream


why is the org.hpccsystems prefix removed? Is this a functional change, or is it just a style change?


mushrooms need to be generated as elf rings


did you mean to add `figaro` here?


Could you post an issue with the code which is throwing this error? Because I did not have it when I tested the code I gave you on your last issue.


You're welcome , I'm always glad I help! :)


Maybe we chould add different method to create guest user?
Something like userService.registerGuest(username).
Also please rename username to userName


As mentioned in the commit it's temporary before porting over branch.c to ref-filter. If you remember it was the same with tag.c, this will be removed with the next patch.


The same here, maybe we should have different method to login guest?


@JNPA : Is this really needed? In case you are not sure do a quick test of removing this details from the web-fragment and test if your application is still working. Thank you!


/search/ should be replaced by: {% url 'wagtailsearch_search' %}
Using the django url tag makes the code a bit more maintainable in case urls change in the future.  (Although I don't really see the search url changing, it is a good practice to follow).


You could go as far as making each list item a form rather than a link, but I doubt that gives us any real benefit: http://docs.wagtail.io/en/v1.0b1/search/searching.html#default-page-search


.project must me removed from gitignore


I'll see what I can do about it :)


Found the reason: The capture clause was introduced in OpenMP 3.1. GCC v4.7 as minimum requirement.


You forgot to edit the `REQUIRES_devel`.


Please do not delete this file. Eclipse needs it and it's nice to be versioned, thank you!


Strange.
The duplicate cookies are most likely a consequence of `ignore_discard` and `ignore_expires` in the cookie save/load calls.


pierwszy


Nit: this XXX is kind of pointless, unless you do it in a later commit, in which case I'd prefer you do `TODO` so that I know that tidy passes, you must have removed it.


Ignore could be a bit. will change


It's not clear why this is needed. Could you explain a little more?


abrrev needs to a value (size).
verbose either take a single or double input.
detached perhaps could be a bit.


Maybe we should continue using name like guestUser instead of temporary? Just to know that this param related to guest user. Or, maybe renam eguest user to temporary user (as it done in Tain )))


AssignmentOne Second Day work


nit: reference monomorphization in comment


The same here, temporary = guest


No mention of the generic source function usage. Better mention the design goal and the docker side of it


I didn't do that cause the ref-filter.h only part doesn't make sense without the inclusion of branch.c changes. But if you think that's better, I'll do that :) @chriscool


TeamCity DNN JavaScript Libraries :: CI [Build 53](http://teamcity.engagesoftware.com/viewLog.html?buildId=321&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 53](http://teamcity.engagesoftware.com/viewLog.html?buildId=321&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:19


Update to official v3.12 libs ?


Ok thx @Soullivaneuh, so it's for soon :yum:


:)
You should add the same for all items in this readme))


Same here, mention why we need this function rather than giving details on what it does on the docker side


We cannot drop this configuration burden on our users. Please do:

- [ ] select a sensible keybinding in accordance with Emacs conventions
- [ ] bind them in `interactive-haskell-mode-map`


You need not to submit test script/code.


I did a fix for this in the http-api:

https://github.com/jishi/node-sonos-http-api/commit/90151468c8010fc2b3010741c359794f7ad6a3a5

I'm not sure if this should be part of the discovery, and perhaps it should be handled the same way as I did there, meaning a round-robin distribution of players to even out the load (all players will accept the albumArtURI). This becomes more relevant now that it also returns high-res album art for some music services. The players are very limited in capacity, according to Sonos it will only use up to 4 threads for http requests which is also shared with UPnP control signals like volume, play, pause etc.

Why did you need this in the discovery package? I could argue that if you implement the sonos-discovery package in your application, you could probably also handle prepending it with a relevant IP. The HTTP API is a different ball park, since you will run that from a different context.


這個是自已寫的還是從網上檔的？出處？


TeamCity DNN JavaScript Libraries :: CI [Build 54](http://teamcity.engagesoftware.com/viewLog.html?buildId=322&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 54](http://teamcity.engagesoftware.com/viewLog.html?buildId=322&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:6


Please squash this one with the commit where you introduce this functions. It is not a good thing to leave in history such development details.


This sounds very counter-intuitive. I would expect “is *exactly* `Nothing`” to be true for less cases than “is (implicit assumption: *in some way*) `Nothing`”.


don't put <scope>test<scope> here, it should be define in submodules


Not intentionally--thanks for pointing it out.  I guess I'll backport that to the v1.0.x branch.
I don't think it's a super important fix.  Is this something you've encountered yourself?  No one ever reported this bug to me before I found it myself.


The `-P` flag can be defaulted to true because the vast majority of sysadmins would hate for nom system packages to be installed without explicit consent.

We are aware of your problem and SaltStack is trying to make the necessary packages available to minimize this issue.


Please quash this with the previous commits.


Looks like the guy who did this PR slipped it passed me:

https://github.com/nraboy/ng-cordova-oauth/blob/master/src/oauth.js#L1553

I will see about fixing it tonight.

Thanks,


Closes #76


I later updated this to be 9.0.1.0-SNAPSHOT.  Sorry I forgot after yesterday's release excitement.


So basically don't need it before any/all actions?


Jut eszembe: még decemberben csináltam próbaképp egy ffmpeg 2.5.2-t magamnak.
Mindössze 5 csomaghoz kellett patch.


Hallo bedankt voor de bijdrage. Bij nadere inspectie toch wat opmerkingen:

* waarom niet het patroon gevolgd van de andere VIEWs? Die doen allemaal een FROM *actueel VIEW (ipv FROM brontabel bijv `gebruiksdoelverblijfsobject`). (Aha zie nu waarom (`status` en `geom_valid` missen in `verblijfsobjectgebruiksdoel` tabel))

* onduidelijk hoe/waarom de begindatumtijdvakgeldigheid wordt gebruikt (meerdere keren en zowel met 'now' als LOCALTIMESTAMP)

Ik zou de VIEWs als volgt hebben opgesteld (ben er ook weer een beetje uit):

    DROP VIEW IF EXISTS verblijfsobjectgebruiksdoelactueel;
    CREATE VIEW verblijfsobjectgebruiksdoelactueel AS
        SELECT vog.gid,
                vog.identificatie,
                vog.aanduidingrecordinactief,
                vog.aanduidingrecordcorrectie,
                vog.begindatumtijdvakgeldigheid,
                vog.einddatumtijdvakgeldigheid,
                vog.gebruiksdoelverblijfsobject,
                vog.status,
                vog.geom_valid
        FROM verblijfsobjectgebruiksdoel as vog
      WHERE
        vog.begindatumtijdvakgeldigheid <= LOCALTIMESTAMP
        AND (vog.einddatumtijdvakgeldigheid is NULL OR vog.einddatumtijdvakgeldigheid >= LOCALTIMESTAMP)
        AND vog.aanduidingrecordinactief = FALSE
        AND (vog.geom_valid is NULL OR vog.geom_valid = TRUE);

    DROP VIEW IF EXISTS verblijfsobjectgebruiksdoelactueelbestaand;
    CREATE VIEW verblijfsobjectgebruiksdoelactueelbestaand AS
        SELECT vog.gid,
                vog.identificatie,
                vog.aanduidingrecordinactief,
                vog.aanduidingrecordcorrectie,
                vog.begindatumtijdvakgeldigheid,
                vog.einddatumtijdvakgeldigheid,
                vog.gebruiksdoelverblijfsobject
        FROM verblijfsobjectgebruiksdoelactueel as vog
      WHERE
        vog.status <> 'Niet gerealiseerd verblijfsobject'
        AND vog.status  <> 'Verblijfsobject ingetrokken'
        AND vog.status  <> 'Verblijfsobject gevormd';

Dat is simpeler en dan klopt `verblijfsobjectgebruiksdoelactueel` ook. Echter daarvoor missen we  `status` en `geom_valid` uit de verblijfsobjectgebruiksdoel tabel. Die zouden we simpelweg kunnen toevoegen, alleen is dit Py code wijziging maar hoort in feite bij de filter die je met de VIEWs legt.

Zou (`status` en `geom_valid`) namelijk m.i. ook bij andere (2) multiattr tabel-VIEWs zoals `verblijfsobjectpand` en `adresseerbaarobjectnevenadres` moeten.

Hoop dat je dit kunt volgen (en ik dit juist heb). Anders doen we toch maar de merge en optimaliseren we met aparte issue.


why spaces vs tabs? has this changed now? I can do either, I'm just curious...
https://github.com/openframeworks/openFrameworks/wiki/oF-code-style


Is there any particular justification for the PHP 5.4 dependency? Many projects use PHP 5.3, and this would be useful for those projects. Was this decision made arbitrarily or are PHP 5.4 features actually used?


Use `string-prefix-p` instead.


@skullkeeper94 lol still working on that lel


aking it the best job they’ve ever had at a design studio*, one* they’ve always wanted to work for.


Why 13? This needs a comment. Or better some code restructuring so that magic number is not needed.


works only on java 8


Some suggested text to add?

Application virtualization is software technology that packages or encapsulates application software from the underlying operating system on which it is executed. A fully virtualized application is not installed in the traditional sense, although it is still executed as if it were.


Huh? The type `Sequence<Nothing>` has no instances. Therefore it is exactly the same type as `Nothing`.


you can inline the version


I've already *had* ....


7.23
Now i'm starting studying again


> How about `createSystemField`?

How is that an improvement over `CreateFieldInSystem`? First of all, I prefer to name functions in a way that similar functions, or functions that belong to a group/category of functions, start with a common prefix, in this case `CreateField`. By naming `CreateFieldInSystem` the way I did, it conveys the idea that it's a variant, or specialization, of the more general `CreateField` function.

It also conveys better the idea that I'm creating a field within a system. `CreateSystemField` isn't as clear IMO, it sounds more like specifiying a certain type of field (a "system field") - which is probably why you'd prefer it, but I still think that's misleading, because of the reasons already repeatedly cited.

Well, neither of us is a native speaker, maybe one of our English native speakers can comment on that...

> Adding dogstrings together with function will save a lot of time.

Can't argue with that. I admit I'm dragging my feet here - I've to go over the entire server side Python interface anyway, a couple of functions more won't make much of a difference, and I don't enjoy writing comments/docstrings etc. very much, so I'm using this (probably lame) excuse to put it off... :smirk:


@austinmcorso this change was required to clean up old diffs. without it the diffs just kept stacking up. You can pull in the previous commit hash if you need to use the old version. I should start doing tagged releases for sure.


I don't like having to write code to cleanup the environment


When users log in via ITSI but do not yet have an account in MMW, they are redirected to the ITSI sign up page `/sign-up/itsi/tstudent2/Terence/Student?next=/project/1/clone`. The URL has embedded username, first and last name fields pulled from ITSI to pre-populate the sign up form, and also a `next` URL to go to once sign up is successful. The value of `next` is [parsed and handled within Backbone](https://github.com/WikiWatershed/model-my-watershed/blob/develop/src/mmw/js/src/user/views.js#L380-L392) so that we don't do a full round-trip / page refresh, since in most cases (when we're redirecting to `/` or to a project `/project/1/scenario/2`) we don't need to. However, in this case, since all the cloning is handled by the server side, we _must_ make a round-trip. Thus, we have a special handler for `/project/XX/clone` which sends the current URL to the server and lets that handle it.


group mute and group volume should probably continue sending different data since you might want to be notified about all players in zone or the actual group state.

I have been thinking about moving the convertToSimple logic to the HTTP API since it is actually used for preparing JSON serializable data, not to be used internally. There might be some discrepancies on all events though, since this has grown organically.


#1 会社概要を取得し、特色・事業に合わせて業種情報を抽出。
CompanyProfileに登録する


Yes, but if I want to check whether `Sequence<Nothing>` is empty, I now have two methods that sound like they could do that: `isNothing` and `isExactlyNothing`. And by their name, I would guess that

- `isNothing` would return `true`, since `Sequence<Nothing>` is `Nothing`, but
- `isExactlyNothing` would return `false`, since `Sequence<Nothing>` is not literally identical to `Nothing` (`instanceof NothingType`)

which is the exact *opposite* of what the methods actually do.

Of course that’s not a problem if you’re the only one working on this code ;)


Dear Olivier,

there is a small logical bug in `operator>()`. Statement `x > y` is not a negation of the statement `x < y`; `x` and `y` can be equal.

Cheers,
Igor


some comments here wouldn't hurt, the names are long but not entirely obvious


I hope so! ;-)


what -> that


i really don't like this bitwise XOR on bool use you keep doing... gaaah...

if you'd place != here, then it's far less cryptic, and it actually doesn't pretend to be a bitwise operation! And all at the cost of just one more character! ;-)


don't -> doesn't


We have run into an issue where this change was cleaning up all screenshots, including baselines. In other words running PhantomCSS would just re-establish a new set of baselines every time. Will be swinging back to this in a bit to investigate.


Nit: I prefer to have all three arms indented or none :) but do what you feel.


Dude, I wanted to make a change that didn't impact _all the other projects_ under `ceylon/`!


Please prefix variables with VERSIONED_COLUMN_ , and COLUMN_


beurk, we don't have "fail" for that ?