Muito bom Luan!!!

Está commitando com o dx-pointz??!

Se tiver com vontade: Precisamos criar a opção de facade no Repository.

Por exemplo (aproximadamente):

yawp.save(obj, facade)

yawp.query(Person.class).facade(Facade.class)

Busca por Id também...


@newyorkpizza does this seem alright to you?


look,  it's ok !!!


@BestOwl What's this for?


类似于网盘的东西


You can also use the context manager `with` for this purpose.
```
    with open('sherlock.txt', r) as f:
         f.readlines()
````

After the code in the indented block has run, the file will be closed.


merge master


你居然在GitHub上……


reverted, let's track a better solution at https://github.com/mapnik/mapnik/issues/2952


たすけて


あきらめろ


You're useless.


Generally it's pretty rude to remove the first author's name instead of just adding your's in addition to mine.

But hey, open source!


you might consider using len() of words in the for loop here rather than the if statement below to determine when you should stop


HELLO


Fix for issue #22 Favicon not displaying


93469234-DemandResponse2 (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1170 of 1173 tests passed)

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


I'd say "and is intended for use with tanh activations."


Which typo ...


I am not sure to see why... ```filePath``` should not be null at this point.


Forgot to add the prefix [EN_FIXT]


Is this new map file something that we generate automatically somehow or do i have to manually go in and edit it?


My bad, I saw filePath and filePathInDB as the same thing.


Edited the api documentation to be inline with our current models


I realize I'm probably not doing this properly either, but I would be much happier if we used a proper library for this.

http://hackage.haskell.org/package/http-types-0.8.6/docs/Network-HTTP-Types-URI.html#t:Query comes to mind but I'm open to alternatives. Has the advantage of already being a dependency.


Xuất phát từ đam mê cùng sự phát triển thị trường công nghệ như hiện nay, chúng tôi (Lâm Apple) những con người nhiệt huyết đã và đang khai thác những tiềm năng của công nghệ Internet, tri thức và nội lực sáng tạo để tạo nên những sản phẩm và dịch vụ chất lượng cao phục vụ tận nơi cho Khách hàng.

Nguồn hàng chúng tôi luôn bảo đảm và giá chúng tôi đưa ra luôn thấp hơn thị trường do nhập trực tiếp không qua thương lái và được kiểm tra rất kỹ trước khi đến tay quý khách hàng. Sự hài lòng của quý khách hàng chính là niềm vinh hạnh cho cửa hàng Lâm Apple.


Note that removing this part does not seem fair. Extracting the set bits to an array is a common operation. But I agree we might want to benchmark iterations separately.


Oh crap, I just realized that you're specifying 2 workers. That won't work with the default free plan. Can you get away with just 1 worker? That would let heaven be a free app on Heroku. If not, you're going to need to specify `standard-1x` as the size, but it's going to fail until we fix some things in an underlying API.


I appreciate the comments. Part of the reason I didn't send it is that I felt some bits of it were a little gross, and I wanted to sleep on it. :)

My plan is to move them all into `path.c`, and just do away with the static ones. I'll see how it turns out.


Shouldn't the function return the brush like tags?


I just tested with 100 residues

```python
def split_by_residues(struct, start=0, stop=-1, stride=1):
    '''split Structure into different residue
    TODO : test
    '''
    n_residues = len(struct.residues)

    if stop < 0:
        stop = n_residues + stop

    for i in range(start, stop, stride):
        j = ':' + str(i + 1)
        yield struct[j]

def split_by_residue_JS(struct, start=0, stop=-1, stride=1):
    '''split Structure into different residue
    TODO : test
    '''
    n_residues = len(struct.residues)

    if stop < 0:
        stop = n_residues + stop

    for i in range(start, stop, stride):
        yield struct[i, :]
```

Result: your code a bit slower.

```python
In [44]: p
Out[44]: <AmberParm 481 atoms; 100 residues; 488 bonds; parametrized>

In [45]: %timeit list(split_by_residues(p))
1 loops, best of 3: 2.27 s per loop

In [46]: %timeit list(split_by_residue_(p))
split_by_residue_2   split_by_residue_JS

In [46]: %timeit list(split_by_residue_JS(p))
1 loops, best of 3: 2.45 s per loop
```


Não precisa alterar o nome de tudo. Dá pra adicionar um texto depois que será exibido no formulario. Tipo assim:

 <%= f.label :password, "senha" %>


What is renderNonBreaking and what is the use case?


It is truly redundant! this code is in [bonescript-demo.js](https://github.com/ehab93/bone101/blob/gh-pages/static/bonescript-demo.js), but when I include this script some other scripts got broken, don't know why?!


Yeah, not very important though


i would issue at least some warning to logs in this case


Hi


hi


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Não precisa alterar o nome de todos os label's. Dá pra adicionar um texto depois que será exibido no formulario. Tipo assim:

<%= f.label :password, "SENHA", %>

Isso gera algo tipo

<label>SENHA</label>


I have a weak preference for using the same label here as we use inside commands/install.py, so that it's more clear to someone reading the bits of code separately that they're referring to the same thing


Whitespace


ÐµÐºÑ‚Ñ€Ð¸Ñ ÜÞéÝëÙ


1712-AvailabilityManagers (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


sorry, this was unintentional. I'll fix it. Still, I don't see why it fails.


We've had 2.7 issues before  but with with bs4, not supporting 2.6 means most android builds so would be an unpopular option. Disabling subs with we have an error importing pycaption is probably safer. Unfortunately I don't think we can add this logic to the settings.xml file, unless we can make the pycaption module  not install if python isn't pressent.


Regarding the signal declarations:
>> „7. In signal type, "E5a" is "E5aQ", right? because "E5aI" exists.“

> I See your Problem. I study the Galileo Dokumentation, but i am not able to understand the meaning of
> the Band-declarations till the last conseqences.
> So i would change „E5a“ to „E5aQ“ or should i remove „E5al“? Please have a look at the whole list,
> there arer other unclear signal_types.

Well, I am afraid different receiver manufacturers also might have differing opinions on this. Basically, "E5a" is either ambiguous, because not more specific and only telling about the used band (or subband), or means that both "E5aI+E5aQ" are included. "E5aI" and "E5aQ" are both code modulations used in the "E5a" subband. See http://www.navipedia.net/index.php/Galileo_Signal_Plan#Galileo_E5_Band for details. I don't see a way that we can further restrict it. So actually, I think, all must be allowed, "E5aI, E5aQ, E5a, E5bI, E5bQ, E5b, ..."


No, it shouldn't, so we have a bug. I will have a look at it.


do you mean `getenv`


Closes #3.


Still not working?


looks like `build-docker.sh → docker-build.sh` but git didn't treat it as a rename


__Original text:__
> where people more talented than I can readily suggest improvements.

__suggest:__
 donde existen personas con mayor talento que fácilmente pueden sugerir mejoras. => donde personas más talentosas que yo,  pueden fácilmente sugerir mejoras.


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Add the text "using the "FOR" loop." in the exercise


ah ok . no worries after explain . thanks merged again


UMAD?!


Лучше называть методы глаголами - `initializeProject()` и т.п.
Ну и с маленькой буквы.


We'll have to see why this can't be 1800000


Мне кажется, эти пять строчек тоже надо завернуть в метод прожект конфигуратора, а здесь оставить только единственный вызов этого метода.


We now have test patterns for a whole stack of sequences. This changed the length of any sequence involving `echoCR`, which causes the test patterns to not match.


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


Replace with

		regulator-state-mem {
			regulator-on-in-suspend;
		};


I _think_ this is a mistake.  The concept of eMMC reset is a general one and (I think) baked into the MMC core.  dw_mmc doesn't yet support this concept, but it might one day.  When it does, it will be harder for dw_mmc to do what it needs if this is specified as a power sequence, right?

See MMC_CAP_HW_RESET.


The word **Unices** is the plural of Unix.  It's possible check this on [Wikipedia](https://es.wikipedia.org/wiki/Unix)


No será mejor

pero debido al interés mostrado, parece que vale la pena usar Github, donde existen personas más talentosas, que fácilmente pueden sugerir mejoras.


Awesome, thank you sir!


Sorry for that! :)


why can't we do uninstall by one single command?


Is `efuse {}` not supported upstream?


Hi! Basically, if len(term) is inside an expression it can overflow because the whole expression is treated as a C expression (I discussed the matter here http://comments.gmane.org/gmane.comp.python.cython.user/12703). Obviously this holds if and only if marshal.py is cythonized and cython tries to optimize that expression.


Happens, no problem. :)


`vdd_logic: pwm-regulator` will eventually percolate in.


Probably don't include Bluetooth here.

On Marvell modules, which only use SDIO and no UART, there is no connection here.

On Broadcom modules, which use SDIO for WiFi and UART for BT then this pin only controls BT.

...thus this pin has nothing to do with SDIO and doesn't belong in the SDIO power sequence.  I believe at some point Broadcom talked about implementing it as an "rfkill" driver, but I have no idea if that's right.  The whole BT over UART is strange since it has a huge userspace component...


No, I do mean ~getconf~; that's what Tramp docs say and is consistent with the behavior that I'm seeing.  On GNU/LInux that is /bin:/usr/bin, and not the remote user's PATH, which is why I think this may be worth noting.


r+


saw approval from MonsieurNicolas
at https://github.com/graydon/stellar-core/commit/499000b57e2ec60e0feca785b8256c645dc46dd1


merging graydon/stellar-core/bug-257-collapse-duplicate-entries-in-buckets = 499000b5 into auto


graydon/stellar-core/bug-257-collapse-duplicate-entries-in-buckets = 499000b5 merged ok, testing candidate = afc86541


*soLo* => solo


Oops, shouldn't you be returning `result` here?


@tconst according to the spec, a better regex here would be `/\A[A-Z]?\d{9,10}\z/`. I don't think all 26 letters are actually used, but I can't find a definitive answer to that for now. Regardless, the spec for this field is `[Optional Letter, Number, or Padding] + [Tax ID]` so restricting the prefix to letters vs. _anything_ would be a good idea.


Hello rethink-kmaroney,
I am tying to clone the baxter simulator but I do not have permission.
Do you think you can add me to the repository, and enable me to clone it.
My email is AlexAlvTrin@gmail.com
You can contact me and I can give you the serial number of the Baxter, I am currently working at the University of Nevada, Reno  Robotics Research Laboratory.

Thank you


I verified this with python-driver version 2.1.1, 2.1.4, 2.5.1, Python 2.7.9, 2.7.10, Cython 0.21.2, 0.22.1


Give it more of an option I guess...


bad merge: the any here should be void


(because we now enforce return types are present on non-void methods)


Give it more of an option I guess??


что тут вызывается?


Wouldn't it be simpler to just wrap FSL melodic's output in a single `iframe` and only tweak `img` and `href` URLs?
Full HTML parsing seems a bit beyond a viewer's scope.


Need to add more HTML!!!! To busy trying to make DDoS stronger!!!!


What about `[a, b].min`?


The english version seems a bit more general than the german version. Probably we could try this:

The subject may under no circumstances be used in systems of any kind that assist the execution of death penalty.


IDK???


oh, I totally misread all that. I didn't notice the nested if. thanks :).


Upon further consideration, for stricter adherence to the spec, this would be better: `/\A[A-Z\d\s]{1}?\d{9}\z/`


Why is it necessary to use an NSInvocation to init the alert view here? Why not call it normally?


There is a **VERY IMPORTANT BUG HERE**. You renamed `lovely_touching` to `init` which is like SO MUCH LESS FUN.


An example would *greatly* help in understanding what this function does. `key` also seems optional (`... if key == "" or key.nil? ...` wouldn't it be nicer to have `append_string_to_link link, string, key = nil`?


This seems redundant if there's already an error logged inside RCTAlertView() anyway.


FIFA Football 2005 no works, Fix it please


A right (I thought parenthesis were also missing :-)


Addresses Issue #125


At the very least, this static URL should be a constant defined at the top of the file.


ok


The `initWithTitle:message:delegate:cancelButtonTitle:otherButtonTitles:` method is flagged as `NS_EXTENSION_UNAVAILABLE_IOS`, so if there's a direct invocation of it Xcode won't let you link into an extension target.

And since you need the return value, `performSelector:` won't do it.  Pretty ugly hack though..


I didn't changed them i just commited what west already done in the RR repo


The word **UNIX** is an operative system name so it doesn't have plural, but letting wikipedia ruling us I can accept **Unices**, although I don't like the term.


@pgdaniel what is `@address[:address]`? Is it an array-like thing? Looks like they were already calling `take(2)` on it before the change, which means it had to have multiple items...

Your change would act on the 2 items in the array (i.e. `@address[:address]` and `@address[:address_2]`), right?

But if `@address[:address]` is already an array-like thing, when you do the take 2 it'll give you like `[['address line 1', 'address line 2', 'address line 3', ...], 'apt #9']`. Is that what you're expecting?


Wouldn't it be simpler to do `raw(@userfile.modified_file_content(...).join)`? `modified_file_content` already returns an array...


Very grateful that the intents on the NewUserLoginActivity work!


I'd only store the base URL. You can use this variable to then generate the URL to store an answer and the URL to display a question.
Example:
BallotBaseURL = "http://www.adreas.ifi.uzh.ch/ballot/".

Also, please move this variable into the scope of the ballot portal adapter:
hcomp.ballot.baseURL <-- this should be the right name


Ah, I see. Sorry, I should've read the description more closely. Seems reasonable.


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


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


ConfigFactory not needed here. You can use the PortalBuilder, it does this automatically.


@kenkouot Will you please code-review this? Liz helped me this. Thanks.


Replace with the _URL_, not the actual value. The URL should be "hcomp.ballot.baseURL"


Is it possible to move this commit to a different PR?


eventually, this test should fail because it lacks an input..


nice catch. Thanks


ola


__Original text:__
> and ctrl-u to delete all the way back to the start of the line.

__Change:__
desde => hasta

Note:  **ctrl-u** delete from right to left, because the direction the word is `hasta`.


I'd consider using a member variable to store all of these answers. This way you can reuse the object and ask it to return different answers when needed - and define the kinds of answers you like using the constructor


Calling exit() is actually wrong.  What if there are other classes that need such a thing? The exit() call should be removed and the method renamed.


performSelector actually returns an object, so that wouldn't be a problem. The issue is that it can only take a maximum of two params, and ARC expects the return value to be autoreleased, which init isn't. You could use objc_msgSend directly, but NSInvocation seems safer.


no need to use the configfactory here. It will crash if the entry in the log isn't present. If you don't need the URL in the Test class, just initialize it with something that makes sense. http://andreas.ifi.uzh.ch:9000 for example


was this wrong before? the argument to new was a ComponentAnnotation not a ComponentArgs


rename to DAOTest. (Better to stick to your naming conventions that you've introduced with your base class "DAO"


Engraçado que não alterei esses arquivos. Vieram do commit de Rogério, fiz merge e apareceram aqui como que eu tinha alterado kkkk


Typo


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


972124-2NewDetailedIceStorage (RKStrand) - x86_64-MacOS-10.9-clang: OK (1667 of 1670 tests passed)

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


Yeah, sorry; I knew I did that for a reason :smile: It was the number or arguments, not the return value.

I didn't really consider the ARC angle either, so that's a happy accident


Keep the word `script`, remove `guiones`


    cf_log_err_cs(cs, "Failed loading module rlm_%s from file %s", module->name, dl_get_pathname(module));

That's worse.  If there's no dladdr(), then it prints NULL, which is unhelpful.

The current code is fine.


guiines = > script


Implementation in tictactoe.cpp :
![capture](https://cloud.githubusercontent.com/assets/8801647/8556621/374e247a-24c6-11e5-8676-ce0036b3ebf5.PNG)


Real name here, please.


Well, we have the pokemondb.js file in chat-plugins. That shouldn't interfere with merging with Main should it? If not, in the future I will just add commands into that file to avoid issues. I removed all the Scizbot scripts from repo and they will not need adding in as I can host it my self now via a .io domain.


guión => script


It is now done, fab25503716f6497273dda9b01b0e6e3f04f9bc9


Make sure entryDistribution is not null.


Please put real data here, and into Source field above too.


This all looks good. Please also set the right visibility of the classes (especially in the main source folder, not just "test"). Most classes don't need to be publicly visible and just clutter the environment. Almost all classes can be package-visible. You can define this using private[PACKAGENAME] MYCLASSNAME.. in scala.

That means:
1. move everything that's related to the portal adapter to
"hcomp.ballot"-package
2. put "private[ballot]" in front of almost all classes except for the PortalAdapter itself to make them package visible


Likewise.


and... it could be exiting successfully. Imagine a scenario in which ardupilot is well behaved: it process a SIGSTOP/SIGINT sent by init, "deinitialize" everybody and exit. It should exit with 0, not 1.


Wow! Such commit. Very upload.


`log.error`


Boilerplate.


Hier ist ein Komma zuviel hinter "html5"


good!


Yeah, agreed, the error is unneccessary


Some of these should probably be static inline or just static.


Shouldn't be native if it's not something originating from Debian.


"Rhein-Main" würde ich ungern als Stadt durchgehen lassen ;)


Impressive. :+1:


The cake is a lie...


‘Initial release’ is something more appropriate.


This is worded a little academically. How about this to replaces the whole Channels section:

> Channels are namespaces for events. **Currently, all events that you write get routed to channel ID 0.** In the future we may support multiple ingest channels, or channels for computed values.


Acepted


Thanks!


Also here it is possible that no entry will be found. Validate the entry pointer.


#100
the other number was a mistake


(in general) Please add some logging.


okay, but the code here looks crazy, if I'm right, it creates 2 new strings (`integer` and `string`) just to copy from one (`integer`) to the other (`string`) and then free the first (`integer`)??


You haven't read this, have you? ;)


Updated the QnA proof of concept in order to add different outputs based on whether or not data contained in the agreement was subject to FERPA or HIPPA regs. Still need to go back through and
1) say when FERPA and HIPPA are triggered during Q(1.1.1.1.1.1.1)
2) will need to update the specific language that will be output in DOC(1.1.1.1.1.1.1.1), DOC (1.1.1.1.1.1.1.2), and DOC(1.1.1.1.1.1.1.3)
3) create DataPurpose variables under each of the 4 types of contract that assign the unique DataPurpose to the text in the document.


Isn't ingest-$ENV obsolete now? Should we go through and change them? Maybe that should be a separate PR.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.321](http://teamcity.monogame.net/viewLog.html?buildId=8565&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 501, ignored: 6 Build time: 00:24:28


I would say that from long-term perspective switching to sqllocaldb is a good solution but for upcoming release with VS2015 support is too risky.

I will propose some simple solution and will see how it will work.


__Original text:__
> Know basic `awk` and `sed` for simple data munging

- Keep  word `sencillo` without the `s`
- __Suggestion:__
Conozca aspectos básicos de `awk` and `sed`


- Removed CSS and JS

- Remove realpath (incompatible with phar)

- Remove tests (now pointless)
- Add phar generation


Um this is wrong.  I will delete it and fix it


Does Elemental now use 1 based indexing?


"Garbage collector thread" rather than "event"


Aw... I thought 'native' is than upstream contains 'debian' folder. It should be quilt?


style inconsistency: `!= NULL` here but not in prev hunk


parentheses????


okay


@jamiealbrecht I started a Google doc so we can work on this language together. Maybe we can add in a bulleted list that keeps our ask straight forward.
```
When applying, tell us more about you by:
* Adding a link to your GitHub profile, design portfolio, blah blah
* What role would you like to be considered for? Take a look at our [positions](link to positions)
* Don't see a role that suits you? Tells us what you want to do!
```
Here you go: https://docs.google.com/a/gsa.gov/document/d/1szZRSgD3boBVz645tGmLw3MvU11J3Flgv7nkwP5i46E/edit?usp=sharing


Can you add a paragraph to the README about how to use this


worked as a single command for me: sudo make uninstall


Yep, I've deleted it. Should I restore it?


:+1: Happy you like it! :)


summin' aint right here


This will loop trough the file's contents ~35 times if all options are set. Maybe you could refactor `set_option_in_design_file_content` to take an array of options instead?


:+1: danke!


Close #2306


What is the `AtomicBoolean` being queried?


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


@BishopWolf  Great Job!!! Pretty good revision.

I added some comments.  Please, could check and do the changes?

Thank you very much


This converts the single-digit drive number [0-9] to an ASCII string without the overhead of a sprintf.


Siktir

On Monday, June 15, 2015, thilankaat <notifications@github.com> wrote:

> My icloud account locked bcz of security reasons. I dnt remember recovery
> email or security questions. But I know my previous password. I called
> apple. But they will not activate my account. Can u help me.
> thilankatsene@yahoo.com
> <javascript:_e(%7B%7D,'cvml','thilankatsene@yahoo.com');>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Pr0x13/iDict/commit/bf14ee82774b6e7c1f6c3139aed668612c5e4c93#commitcomment-11673761>
> .
>


the translation of `scripts` is `guiones`, ver [Wikipedia](https://es.wikipedia.org/wiki/Guion#Inform.C3.A1tica)


Part of #2314


Right now the feature item style isn't a mandatory field so we'll probably want to add checks to this and only add the styles if it is set.  Maybe set a default style if it isn't set?


This will loop through the entire contents of the file ~25 times. Maybe `get_option_value_from_design_file_content` could be refactored to return a hash of which options are set in the file?


Is that all of it?  great :)


La sugerencia está mejor


done


Awesome! Thank you. :)


ship it


Properly closing DIR objects is recommended by ChaN for future compatibility.


should I touch the licence text?


PHP? So you don't like [Meteor](https://www.meteor.com/) huh?


Could you provide a complete list of those formats?


@tmat The eventual goal is to turn this into a core-clr deployment for the NuGet.


Added some debug printf statements.  Seems aapt has the same issue here as in add_sources.sh:

```
appversionname="$(aapt dump badging "$appversionfile" 2>/dev/null | grep "versionName" | awk '{print $4}' | sed s/versionName=// | sed "s/'//g")"
```

does not like: "**/cygdrive/c/**SANDBOX/git/opengapps/sources/all/app/com.google.android.apps.magazines/14/nodpi/2015051304.apk"

but: "/SANDBOX/git/opengapps/sources/all/app/com.google.android.apps.magazines/14/nodpi/2015051304.apk"

works fine...


Probably off topic. But looking at the new style for writing routers (sort of embedded routers), I'm starting to think that we should probably introduce an alias for `/>` as `apply`. Just to compare:

Before
```scala
def uploadImage(db: PetstoreDb): Router[RequestReader[String]] =
   Post / "pet" / long / "uploadImage" /> { petId: Long =>
     fileUpload("file").embedFlatMap { upload =>
       db.addImage(petId, upload.get())
     }
   }
```

After
```scala
def uploadImage(db: PetstoreDb): Router[RequestReader[String]] =
   post("pet" / long / "uploadImage") { petId: Long =>
     fileUpload("file").embedFlatMap { upload =>
       db.addImage(petId, upload.get())
     }
   }
```

It's just less noisy. Any thoughts?


:+1: perfect!


1712-AvailabilityManagers (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Ah yes, my bad. Was having an issue with comments when I merged stuff yesterday and this made its way in as part of what I thought was the fix :P


estupendo:
que encontré útiles al trabajar en Linux


Test


This looks like something the Intro to Java kids would write. I know it's not a code smell in this context, but darn if it doesn't look like one!


Super!

donde existen personas más talentosas, que fácilmente pueden sugerir mejoras.


Basic ServerApp that logs the file to socket on 4444 port is implemented. Simple console test client to listen to it is implemented. Should be further tested and improved.


I'm just concerned with slowing down the build by copying hundred files.


Removed all comments.


I see several areas where it can be used, almost anytime you have "(in <area>) you can have a location check as well.


This is called GPL-2+ ;)


Is it intended for the to shell expands `$HOME`? If so, doesn't this assume a home directory is available?


The FromZone should not be the battlefield but wherever it comes from, since it never hits the battlefield. In the case of tokens, there is no FromZone.


Why? It's not checked anywhere anymore. Yes, this makes this commit a tx-breaking change, but we want to get this added, no?


... No.


Minor nitpick; why not `self.respond_to? "job_template_goes_to_vm?" && self.job_template_goes_to_vm?` instead of nested `if`s?


even though I agree with you, let's keep the original word used by the author __Unices__


OK, je comprends. Mais tu ne penses pas que ce serait mieux d'offrir des URLs courtes, et d'effectuer le soulignement à partir d'un objet JavaScript défini programmatiquement, en parsant la tables des matières ? C"est un peu plus compliqué, mais ça doit être possible.


I'm not a fan of the "to_regex" part of this method. It doesn't actually return a regex, it returns a "glob pattern" string.


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Did this help?


Well, this isn't the license text, this is license notice, usually verbatim as you put it into your code. Specifically, here # stuff is added by debmake or whatever you used to generate this file.


changed:

```
appversionfile="$(find "$SOURCES/$arch/" -iname "*.apk" -ipath "*/$appname/$sdk/$dpi/*" | head -n 1)"
```

to:

```
appversionfile="$(find "$SOURCES/$arch/" -iname "*.apk" -ipath "*/$appname/$sdk/$dpi/*" | head -n 1 | sed "s/\/cygdrive\/c//g")"
```

and all is fine, I know this is specific to my use, but I am also reaching the limits of my shell scripting abilities!


i forgot to compare and merge so i will remove this in my next commit. I think so it breaks the project.


Should begin with lower case


dLocation.matches used to require (until you just fixed it) l@, but that didn't work with "notable" (the old verbage used in events to denote a location... because.. I don't know why you did that), so I had to write around the limitations, checking for "notable" first and then using .valueof (which supported notables, I think) to check to see if I had a valid location without the need for l@.


When I changing it to 'quilt' it starts demanding 'original tarball' and I have no 'original tarball'. What I should but here and how it should be build than?


Thanks!


Maybe only use that hard-coded path as a fallback if `mnc2nii` isn't in the `PATH`.


Maybe we could remove `if value.present?`.
It is unexpected that the `value` is nil


Ok


Should probably be max length of 254, which is what the spec has.


I can't comment below, but on line [2505](https://github.com/vhda/ctags/commit/8273dfc1c8892856395bcf8c424209a7aeffc0e4#diff-129bea13ed3162d166a5eab00811f319R2505) `parseProgramUnit (token);` is called a second time, this isn't what you want.

IMO, just do
```C
rescanReason rescan = RESCAN_NONE;
...
parseProgramUnit (token);
if (FreeSourceFormFound && ! FreeSourceForm)
{
    ...
    rescan = RESCAN_FAILED;
}


Minor nitpick; EOL whitespace


Perhaps a link to a page explaining what `MOZ_LANGPACK_CONTRIBUTORS` is would be helpful for n00bs like me?


The problem is that this uses System.currentTimeMillis() and that only updates every few milliseconds (for performance reasons) so it's very possible you'll get the exact same time when you call System.currentTimeMillis() multiple times in succession even if they are called milliseconds apart.


then add the location checking into runInCheck, the problem is, most location checks are using "on location" or "at location".  Or, I can just delete runAtCheck and use tryLocation.


I think you need to look at the line you're commenting on.


This change and the one above both have the same behaviour; `cb_error "..." unless ...` but use completely different syntax; `() ||` vs `or`


How can I do that?


Mostly done, check my comments as well to solve conflicting parts.


>.>   nevermind...


Do they know what `<=>` line does? I would point it out as another operator


972124-2NewDetailedIceStorage (RKStrand) - i386-Windows-7-VisualStudio-12: OK (1673 of 1676 tests passed)

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


Look at the commit message


ya, if not a list it's obviously a blah


I'll stop arguing about a chuck of code that will be deleted :)


I would try and find a different examples of sort that does something that `.sort` wouldn't do anyways by default. Maybe sorting an array of strings but sorting them case insensitive so `s.upcase` kinda deal?


A heredoc would make the command *much* easier to read and understand.


here you now have a potential inifinite loop and need to break out of it.


You don't want to use `lstrip` for removing the `#define` because the argument to `lstrip` is the set of characters to remove. For example, a line like `#defines blah` would pass the `startswith` test above, and then leave the `s` at the beginning.

While I think in the long term we should look to using something like [parsimonious](https://github.com/erikrose/parsimonious), in the short term I'd use a slice to chop off `define` like `line[:7]` instead of `lstrip`.


grab the image and s3


s3 please


yes but the string number 363 is corrected


@hyi I'm not sure we need to customize these views right now.  If you can update this code to use HttpResponseNotFound and HttpResponseForbidden as mentioned above, I will look at the code once it's been merged to develop.  If the 403 and 404 pages need to be updated, I will create an issue and open a PR to address the problems.


you can simplify this to
```val tokenTypes=token.zipWithIndex.map( (x, i) => new TokenType(i, x, 0))```


These function should document their assumptions. Notably, `sed_design_file` assumes `old_value` and `new_value` do not contain commas.


I'm a little unclear what `entities` means here.


Hah. This is all temporary hacks to support the features @mdrbhatti is working on for GSoC:

* Exception classes / basic inheritance (inheritance part coming soon).
* Multithreading module.

I hacked it together quickly, as the representation of Python objects is going to change soon to be more compatible with CPython. Thus, this code is somewhat temporary.

Perhaps I should have used a ternary? *shrugs*


Doesn't this add the contents of the shell variable `line` to the file?


Oh man I'm soooo excited to see how good it will be! :smile:
To be honest, I'm learning C by coding here and there and it's pretty
helpful for my future projects. :)
And yes, no problem, I'll refrain from committing.


Ahh, now I see what you mean. In that case I guess we do need a plugin like

```
--insert <string> at <index>
```

In this case

```
--insert '\n' at -1
```

What do you think @stoeffel ?


This seems unrelated?

If you use GitHub for Mac, it makes it super easy to split the changes from your working tree across multiple commits. I , as well as future developers, would really appreciate this.


From a database perspective, UUIDs should be stored as BINARY(16) instead of VARCHAR.  :)


Be careful with the `==` operator - it's good practice to use the strict equals instead (`===`). The difference between them is complicated (http://stackoverflow.com/questions/523643/difference-between-and-in-javascript), but the general rule is that strict equals (`===`) checks both value and type equality of the two operands (ie: that they're the same type and that they have the same value). Regular equals (`==`) converts the types if they don't match in hard to remember ways and can introduce bugs that are hard to find.


Might be worth adding a bug for this, that way people know they aren't crazy if they run into it, and we can implement the fix if they comment on the bug.


These could use some more specific names, like `relative_file_path` and `locale_file_path`.


Yes, you're right.

But, the word `guión` also have another meaning outside computer; this could be confusing, `script` is a word general accepted and is relate with programming.  Keeping the word in english could transmit the idea to everyone, in this case. IMHO

look the translation of the word in this site:
[Mozilla](https://developer.mozilla.org/es/docs/Tutorial_de_XUL/Instalando_Scripts)

What do you thing?


Nice! I like that you're saving `$(this)` into a variable, both for faster re-use and for cleaner looking code.


To be honest, I don't know anything about licensing, so I put the one that was the most permissive but yeah, I let you put back GPLv2+ since we're forced to stick with it.


Hacky, but it seems fine for now.


Not sure about the hardcoded string. If there's a better place for it, let me know.


Thanks.


#### :white_check_mark: Awesome
Good strategy using img responsive, your site looks good.


Sorry about that, I think I changed it when I was experimenting on stout because it wasn't recognizing CUDA 7.0 on there. But I got it resolved.


oh yeah i also reoriented the STL oops


#### :white_check_mark: Awesome
Using media queries in CSS, pretty advanced stuff, great job. `@media only`.


I will delete it with the entire parsing library as soon as I replaced everything.


Should probably confirm you got the right #j and #inc as well.


Thank you!


I recommend indenting the code blocks in the if brackets.
```java
if(condition) {
dox()
}
```
to
```java
if(condition) {
    dox()
}
```


hmm, when looking at Dougs https://groups.google.com/a/chromium.org/forum/#!msg/chromium-os-reviews/e66AUDR2FGc/bbJW8ZplRtYJ it looks like the mapping is suspend-mem-disables -> regulator-off-in-suspend?


Could you quickly mention your reasons (just out of curiosity)? If not, that's also fine... :)


I think you should look into using cygwin without that sandbox mode? Because this completely not in line with normal *nix behaviour, that it would not have access to that path :-/


Dropping the slot is not a good idea here without slotmove.


This shall not happen so assert or really critical error report.


also, you probably should do something to break out of the loop before the real EOF.  Maybe fake EOF when `FreeSourceFormFound && ! FreeSourceForm`?


Same as the other comment.


Yes. The meaning of field in this context is "table". However, the name of the field is contained in a `FieldFragment` object, which may contain more information about it (database and source expression). I will probably rename it as soon as I rename the fragments.


Be careful here - if `url` does not exist or if the array resulting from the `split` doesn't return at least two items, this line may throw an error. Safer code might look like this:

```
var url = tr.children().eq(2).children().attr('href'); // this will always return a string, whether empty or otherwise, so it's safe to split
var urlSplit = url.split('.');
var filetype = '';

if (urlSplit.length > 1) {
  filetype = urlSplit[1];
}
```

This will check that `urlSplit` has at least two items in it, so that indexing for the second item (`[1]`) will not throw an error.


This is specific mmc power-sequence driver for emmcs ... see https://www.kernel.org/doc/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.txt . So it looks like it is meant to be used that way.


The JavaScript looks great! Good indenting, you've got all your `var`s and semi colons. Good work!


Why is the new argument to buildModule a "flag" that is named "priv"?  Does this really have the semantics
of bool?  If the input should be a Flag, then what would it mean to pass a value other than FLAG_PRIVATE?
Should that be reported as an error?

This thought is separate from the application of FLAG_PRIVATE to the resulting module which I feel I understand.


This isn't used anywhere.


It could be worth saving the result of `url.split('.')` into a variable so you can avoid splitting the string more than once. See my note in the first commit for how indexing the split may be a problem, though, and how to avoid it!


Could you give an example of a subtype test that returned `false` before and will return `true` after this commit?


So the `line-height: 1` is what fixes IE? If so, should we be applying it inside of one of the mixins?


`lines` is an odd name, `inc_file` or something would make it clear this isn't a list of lines but an open file. Same goes for the dumping function.


`ActiveModel::Model` [includes `ActiveModel::Validations`](https://github.com/rails/rails/blob/master/activemodel/lib/active_model/model.rb#L61) uses ActiveSupport::Concern to  [extend `ActiveModel::Callbacks`] (https://github.com/rails/rails/blob/4cc5917dce83fb841ac78b250ef3d1024168eb1e/activemodel/lib/active_model/validations.rb#L43)


```
{% highlight java linenos %}
ModelCluster bestCluster = new ModelCluster(cluster);
double bestBalance = evaluateBalance(bestCluster);
double currentBalance;

for (int i = 0; i < iterations; i++) {
    // fork the model cluster and generate a random relocation operation
    ModelCluster currentCluster = bestCluster.generateRelocationOperation();
    if (currentCluster == bestCluster) {
        // no valid relocation operations are possible
        return bestCluster;
    }

    // see if the new model cluster's balance is better than the best we've found so far
	currentBalance = evaluateBalance(currentCluster);
    if (currentBalance < bestBalance) {
        bestCluster = currentCluster;
        bestBalance = currentBalance;
    }

    // if we have generated enough relocations, return the best cluster we've found
    if (bestCluster.getGeneratedRelocatedOperations().size() >= cluster_concurrent_rebalance) {
        return bestCluster;
	}
}

// hopefully the number of iterations was sufficient to find a better cluster
return bestCluster;
{% endhighlight %}
```


MMM, in that case lets keep `script`


no yet. Last series is from mid-june. Base problem is that there is an eeprom framework nearly complete (hoping for 4.3) which also includes efuses.


I'm not sure, but api connection errors are handled by method ```OnConnectionFailed()``` implemented below


Ok, all done!


@ColinDKelley is the `:` a typo here?


This issue was the cause for #2


Do we want to drop it also in the rest of the code?


The indentation seems a but off here but I may be wrong


My thinking in this regards was a bit forward-looking: when we deal with classes and records, we may decide that privacy should be a trinary concept (private, "protected", and public) instead of a binary one.  Though that won't affect the building of modules (as "protected" is only really applicable in the case of inheritance), it seemed reasonable to have a guiding/unified system in place rather than one that varies depending on the use of private.


No, that's not the reason. The ``modified`` is not even set on the row:

```bash
[info] - should query back the record expecting a modified date *** FAILED ***
[info]   java.util.NoSuchElementException: None.get
[info]   at scala.None$.get(Option.scala:347)
[info]   at scala.None$.get(Option.scala:345)
[info]   at org.scalarelational.h2.modular.ModularSpec$$anonfun$1$$anonfun$apply$mcV$sp$18$$anonfun$apply$mcV$sp$19.apply$mcV$sp(ModularSpec.scala:193)
[...]
```


Good! :)


Looks like progress!  Do you want people to test this out?


Also, these dev strings should be documented somewhere. I'd prefer to name them something like, 'wikia-dev'.


Ok. As long as that's documented somewhere.


It also keeps us thinking of FLAG_PRIVATE as the only way to denote privacy, lessening the chance of divergent handling of the situation.


@BishopWolf You are amazing!!!.

The corrections are great @jlevy, do you like to merge this?


duplicate from 2 lines above


Small comments otherwise LGTM


That said, I would be willing to consider a boolean in this case :)


Perfect :)


Something wacky happens when there is more than one tumour for a donor. Check results. Only a handful, so won't change overall trend by much, but STILL!


There is still some issues I have to work out. But yeah. Progress ^^


Si tu as envie de t'amuser en jQuery... ;-)

Ceci dit, un préfixe court peut être une alternative. Ca permet aussi de
ranger les images knitr dans des sous-répertoires. Et pour naviguer dans
les fichiers, un tri alphabétique reproduit au moins la classification en
section.

 Joseph

Le 7 juillet 2015 22:52, François <notifications@github.com> a écrit :

> OK, je comprends. Mais tu ne penses pas que ce serait mieux d'offrir des
> URLs courtes, et d'effectuer le soulignement à partir d'un objet JavaScript
> défini programmatiquement, en parsant la tables des matières ? C"est un peu
> plus compliqué, mais ça doit être possible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/larmarange/analyse-R/commit/c2b0066fffbaac6a1800babecdcb811e814f0835#commitcomment-12048319>
> .
>


comment is outdated, should say: `if the error is a "no support" error, then don't set lastErr`


Related: #35


waiting for the release guys !


This package doesn't need to be disabled as it is not packaged.


Commenttt


Thanks!


That would be awesome. I can run this text in QnA but can't save it there or change the output to html.


Fits into one line.


NB : il me semble important d'être uniforme entre tirets haut ou tirets bas
dans les noms de fichier. Il faudra faire un choix et éviter tout mix des
deux.

 Joseph

Le 7 juillet 2015 23:25, Joseph <joseph@larmarange.net> a écrit :

> Si tu as envie de t'amuser en jQuery... ;-)
>
> Ceci dit, un préfixe court peut être une alternative. Ca permet aussi de
> ranger les images knitr dans des sous-répertoires. Et pour naviguer dans
> les fichiers, un tri alphabétique reproduit au moins la classification en
> section.
>
>  Joseph
>
> Le 7 juillet 2015 22:52, François <notifications@github.com> a écrit :
>
>> OK, je comprends. Mais tu ne penses pas que ce serait mieux d'offrir des
>> URLs courtes, et d'effectuer le soulignement à partir d'un objet JavaScript
>> défini programmatiquement, en parsant la tables des matières ? C"est un peu
>> plus compliqué, mais ça doit être possible.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/larmarange/analyse-R/commit/c2b0066fffbaac6a1800babecdcb811e814f0835#commitcomment-12048319>
>> .
>>
>
>


That was part of the original code when trashBox was introduced.
But you're right, we should review this as it appears to not being doing anything relevant!


@kbenne thoughts?


Good catch!
I did see that originally and meant to update it later. Apparently "later" never happened! :P


Evdev maybe enabled by default? The majority of desktop systems should have this installed.


Nope it didn't. Anyway that commit isn't related to our ste hardware.


972124-2NewDetailedIceStorage (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (1673 of 1676 tests passed)

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


> Si tu as envie de t'amuser en jQuery... ;-)

Je vais y réfléchir :P

> NB : il me semble important d'être uniforme entre tirets haut ou tirets bas
dans les noms de fichier. Il faudra faire un choix et éviter tout mix des
deux.

Vu que la racine du site utilise un tiret, ce serait plus cohérent d'avoir des tirets partout, je pense…


Why does this not reference an external plan from the plan table?


If I did not miss anything, yes.
But that's why I asked for masatake's help running fuzz testing ;)


Have you planned converting this to cake3 plugin? I'd like to see it happen.
Maybe you could also continue this feature as a Helper so it does not get in a way of actual responsibility of view classes.
Then it could be easily loaded at any view without ruining purpose of views.


Thanks, of course a bug.


hab ich gemacht aber von Torbens PC


Unfortunately we have to revert this commit, because TWRP does not play nice with the compressed PNGs :-/


I would prefer fixing the warning rather than disabling it. I've spent quite some time cleaning warnings for the amd64 build and I would not like to re-introduce them for ARM.


Alright. As much as I would like to license all code I write to the MIT license, unfortunately most licenses don't have any exceptions or ways to change licenses after the fact.

I will change it in my big commit later today (hopefully, if everything goes OK).


The same here.


Okay, it's probably fine then.


Neat, I didn't know about GetAddOnDependencies


You're probably right about that.
Let me review to be sure and I'll update it according to your suggestion (or equivalent).


If it does get packaged though, it would need to be handled here?


Please remove the space at the end.


`grunt dev` isn't a valid task so shouldn't it just be 'grunt` to trigger the default task?


Bugs like that are pretty embarassing because they've been there for who knows how long


This logic is hacky and possibly wrong. It's there so that a pull by digest on a v1 endpoint falls back without setting `lastErr`.

Ideally in the case where `err.NoSupport` is true, `lastErr` should only be set if there were only v1 endpoints.

/cc @dmcgowan


I'm seeing a lot of repeated code here. Perhaps it'd be useful to add a module (maybe something like `pontoon.base.formats.inc` that lets you parse and output `inc` files? Maybe have some small `IncEntity` class that holds the key and string, and some methods to convert between them.

A really rough example of what I'm talking about:
```python
with open(path) as f:
    inc_entities = parse_inc_file(f)
for inc_entity in inc_entities:
    pontoon_entity = get_entity(inc_entity.key)
    inc_entity.string = pontoon_entity.translation_for(locale)
with open(path, 'w') as f:
    write_inc_file(inc_entities)
```


I was thinking to replace the constant 5 by the debuff duration provided by low layer.

Instead of
```lua
core:AddMsg("LITTLEB", string.format(self.L["LITTLE BOMB on %s"], unit:GetName()) , 5, "RunAway", "Blue")
```
We could try
```lua
core:AddMsg("LITTLEB", self.L["LITTLE BOMB on %s"]:format(unit:GetName()) , fTimeRemaining, "RunAway", "Blue")
```


The parameter should be `int|string $id`


We don't need to check attrs, talloc is sane and does nothing if the pointer is NULL.


Thanks, man. :smile:

I know C enough, I suppose. :laughing:

The Lua code is really confusing in some places though. :-1:


Thanks.
How did the varint overflow mentioned here manifest? I'm not seeing it in
integration tests -- just wondering if it's lack of coverage, or something
different in my stack.

On Tue, Jul 7, 2015 at 3:21 PM, Stefano Ortolani <notifications@github.com>
wrote:

> I verified this with python-driver version 2.1.1, 2.1.4, 2.5.1, Python
> 2.7.9, 2.7.10, Cython 0.21.2, 0.22.1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ostefano/python-driver/commit/2de0d613af6b1e52f3d0a657c1c1ca9f8e525aa3#commitcomment-12047703>
> .
>


This is why.  If you redefine Addon = Dominos, then this basically redefines Dominos.CreateMenu for everything after this file is loaded.


Instead of `array` the type of the parameter can be `int[]|string[]`.


Thanks for the additional detail.


Return type should be `int[]|string[]`.


No, we're just using `top` and `margin-top` to center. Before, we were using `line-height`.

With something like a `div`, the mixin is all you need. When you're trying to center an icon or a piece of text, I set `line-height: 1;` so that the height of the icon is equal to its font size. You could set `line-height` to another arbitrary value but that can get confusing.


You would be doing toString on actual objects here. I guess it's the authors problem is they include the wrong data here so it doesn't matter.


You bet - I will bring this to our Boston Brigade legal hackers meetup
tonight and try to get some user feedback. Also - see new issue  #14
requesting info on schedule for this group/activity.  Thanks!

   _ _ _ _ _ _ _ _ _ _ _ _ _ _
   |   Dazza Greenwood, JD
   |   CIVICS.com, Founder & Principal
   |   MIT Media Lab, Visiting Scientist
   |     Vmail: 617.500.3644
   |     Email: dazza@CIVICS.com
   |     Biz: http://CIVICS.com
   |     MIT: https://law.MIT.edu
   |     Me: DazzaGreenwood.com
   |     Twitter: @DazzaGreenwood
   |     Google+: google.com/+DazzaGreenwood
   |     LinkedIn: linkedin.com/in/DazzaGreenwood
   |     GitHub: github.com/DazzaGreenwood/Interface
   |     Postal: P.O. Box 425845 Cambridge, MA  02142
   | _ _ _ _ _ _ _ _ _ _ _ _ _ _

On Tue, Jul 7, 2015 at 5:26 PM, bryangw1 <notifications@github.com> wrote:

> That would be awesome. I can run this text in QnA but can't save it there
> or change the output to html.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UMKC-Law/DataSharingAgreement/commit/36dcce78ada14cad879641b3f8ff252d5ff16e37#commitcomment-12048947>
> .
>


Screwed up big time here!
Fixing...


I found it here: http://wow.gamepedia.com/World_of_Warcraft_API#AddOn_Functions

:)


LGTM


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


Habe es nochmal angepasst. :)

> Am 07.07.2015 um 22:37 schrieb Florian Brandel <notifications@github.com>:
>
> Hier ist ein Komma zuviel hinter "html5"
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/dflick/freelancer-rhein-main-data/commit/fd210ac234efed0039ee9922049b38447f98bba9#commitcomment-12047994>.
>


TeamCity DNN JavaScript Libraries :: CI [Build 11](http://teamcity.engagesoftware.com/viewLog.html?buildId=279&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **FAILURE**
Summary: Failed to start build Build time: 0:0:44


This was required to suppress the warning at https://github.com/dotnet/coreclr/blob/master/src/vm/arm/stubs.cpp#L370 . It seems again that the Clang preprocessor is stricter.


Sorry folks - the 2.4.x branch isn't a big priority for us ATM.
It'll come in due course.


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


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


Perhaps we could still cache the op instance, but call `op.setIn(getInput())` or similar to update it. Of course, there are threading issues here either way...

> Maybe a new HistogramFeatureSet op should be created for each region, which would also likely fix the issue?

That is the general approach with OPS, unless you are doing something pixelwise (e.g., `map` over an `IterableInterval` or some such).


Changes 2 and 3 look very good to me. Change 1 merits some further discussion probably.


This was so when rebasing later to combine the commits this would not be modified, this change was unintentional initially.


yes, that was an error.


Sure, it is a separate SHA. But why?


Can I use it for HTC Glacier?


My favorite language is the C# because it's pretty easy to code in it. But C++ is a real pain in the ass :(


I'm excluding marshal at this point with the supposition that it didn't have a lot of opportunity for optimization. Did you find otherwise?


Guter Vorschlag! Dann würde ich sie "amtlicherGemeindeschlüssel" und "regionalschlüssel" nennen. Ich denke wir sollten die Begriffe nicht ins englische übersetzen, da es feststehende Begriffe sind.


Ok, no problem. Thanks again.


Gotcha. Thanks for clarifying.

On Tue, Jul 7, 2015 at 5:36 PM, Josh Rubenoff <notifications@github.com>
wrote:

> No, we're just using top and margin-top to center. Before, we were using
> line-height.
>
> With something like a div, the mixin is all you need. When you're trying
> to center an icon or a piece of text, I set line-height: 1; so that the
> height of the icon is equal to its font size. You could set line-height
> to another arbitrary value but that can get confusing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dobtco/dvl-core/commit/635628f3454392a9465dcda4a20a26cec3b0b393#commitcomment-12049141>
> .
>

--
Adam Becker
(951) 9-BECKER
@AdamJacobBecker


You're fetching individual entities from the database one at a time. It'd be better if you just fetched them all at once and then searched the returned list for each entity:

```python
entities = Entity.objects.get(resource=resource, obsolete=False)
entity = first_or_none(entities, lambda e: e.key == key)
```

(I made up `first_or_none` because there's no built-in function in Python that finds the first item that matches a test in a list and returns None if nothing is found. There's a few ways to implement it online if you search for something like "python first item match find list".)


Yes, clang is stricter. It can be fixed by adding spaces around the ##.


rax retry tests


@sandimas2688 , perfect catch! Thanks!


I don't have precise numbers right now but in my tests there was some gain.


I don't mind C++ or C#, or regular C for that matter. I hate Java with a passion, though. :see_no_evil:


@0xc0170, does this solve this comment?


i like this new shadow however!


This was more of a proof of concept pulling the data from a nightscout server.  Someone else in the NightScout Facebook group used the alerts, like an SMS message to get it working during a workout.  The Garmin API doesn't let one communicate with the internet while an activity(e.g. Running/Cycling) is active.

Using the Nightscout notifications seems to be the best approach currently.


Last bit of information: I've managed to detect the bug by running the standard test suite included in the driver (decimal tests if I remember correctly).


Why `+=1`?


@borisbstyle I can test in a bit.

I'm not a fan of using a global dT vs. passing it as an arg to pid.c through the function pointer as I have shown before.

You need to fix in filter.h
from:
float filterApplyPt1(float input, filterStatePt1_t* state, uint8_t f_cut, float dt);
to:
float filterApplyPt1(float input, filterStatePt1_t *filter, uint8_t f_cut, float dT);


will explain <=> but not sure about using sort to do something sort doesn't
normally do. First time we're talking about comparing strings and the idea
that one string can be greater than or less than another based on position
in alphabet. that is confusing enough without introducing idea that string
starting with capital letter is  greater than string with lower case letter
that come before it in the alphabet and other cases. what do you think?

On Tue, Jul 7, 2015 at 4:59 PM, Joe Burgess <notifications@github.com>
wrote:

> I would try and find a different examples of sort that does something that
> .sort wouldn't do anyways by default. Maybe sorting an array of strings
> but sorting them case insensitive so s.upcase kinda deal?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/learn-co-curriculum/sorting-readme/commit/83f9b5a79d1159f27a94bbd1af98180144fe0fbd#commitcomment-12048457>
> .
>

--
Sophie DeBenedetto


Need to remove those lines


ovo


Ha! Fat heads!


Hi Sarrala. Thanks for the support.

I was actually thinking about how is the best way to put it into a plugin. I don't have a tremendous amount of experience in creating cake3 plugins that modify core behavior, however, your suggestion of putting into a helper is very interesting.

I'm glad you're interested in seeing it happen, I will probably have a plugin ready some time by the end of this week. Any tips would be appreciated. :smile:


No, honestly IMO Java is okay.


if uyouı want enable doze .must me same like this
https://github.com/ResurrectionRemix-Devices/android_device_lge_g3-common/commit/f33b737a22be2e53ebe3b77e6dd5a9648e832b70

otherwise doze wont work as expected


Yeah. Using present? also prevent to pass an emtpy string.


Thanks. I see it. Was previously running integration, and not unit tests.

On Tue, Jul 7, 2015 at 4:48 PM, Stefano Ortolani <notifications@github.com>
wrote:

> Last bit of information: I've managed to detect the bug by running the
> standard test suite included in the driver (decimal tests if I remember
> correctly).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ostefano/python-driver/commit/2de0d613af6b1e52f3d0a657c1c1ca9f8e525aa3#commitcomment-12049397>
> .
>


@selimnairb If this does not create a problem for REST API, I suggest to keep it as is for now. I think changing the code to use HttpResponseForbidden needs to be done together with customized 404 page to make the error user friendly on HydroShare interface. In any case, if there is any change needed on this, a new issue needs to be created since this has nothing to do with this pull request or issues being addressed by this pull request.


Yeah. We could also throw. I'm going to leave this as-is for now, since I'm unsure what the best course of action is WRT Dart.


(and not compiling marshal)

On Tue, Jul 7, 2015 at 4:53 PM, Adam Holmberg <adam.holmberg@datastax.com>
wrote:

> Thanks. I see it. Was previously running integration, and not unit tests.
>
> On Tue, Jul 7, 2015 at 4:48 PM, Stefano Ortolani <notifications@github.com
> > wrote:
>
>> Last bit of information: I've managed to detect the bug by running the
>> standard test suite included in the driver (decimal tests if I remember
>> correctly).
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ostefano/python-driver/commit/2de0d613af6b1e52f3d0a657c1c1ca9f8e525aa3#commitcomment-12049397>
>> .
>>
>
>


One is tempted to propose that the API defined by buildModule() should be independent of the use of flags attached
to ModuleSymbols.  So one might argue that the new argument should be either a bool or, maybe even better, by
a dedicated enum that might currently have just 2 values.

Separately, I am not personally sold on the broad use of FLAGS to implement attributes but recognize that flags
are used pretty widely and that I don't have a good handle on the conventions for using flags.  One problem with
flags is that they typically imply "logical or" rather than "distinct"; in this case you posit that there might be a day when a module is precisely one-of { Public, Protected, Private }.  There isn't a good way to recognize that it would be invalid for a module to have two or more of these flags set.  But I assume there is a good reason to rely on a flag for this situation rather than adding a new property to Module that is defined by a bool/enum.


Maybe use a constant for this like `ROOT_SEGMENT_FLAG`.

Same goes for parent and currentLevel flags.


as opposed to `++` ? Idk. Personal preference.


yes you are right about filter.h

I don't know, but I think dT can be used much more often through the code. You don't have to keep passing it all the time.


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Dunno if you noticed, but this is not yet live on your blog :)


Could use https://docs.python.org/2/library/string.html#formatexamples instead of template and avoid the {$tool thing. I think that format just needs {}


When doing ClassLoader magic like I do in `ceylon.build` (file `ceylon/build/tasks/GatewayClassLoader.java`) the anonymous `Enumeration` class distorts the classloader for the Ant wrapper in ceylon.build.

This pull request enables ceylon.build calling Ant calling ceylon-plugin of ceylon-ant.jar

Otherwise the following error occurs:
```
ceylon.build.tasks.ant.AntBuildException "Error running Ceylon plugin tool (an exception was thrown, run ant with -v parameter to see the exception)"
        at ceylon.build.tasks.ant.Gateway.buildAntException$priv$(Gateway.ceylon:59)
        at ceylon.build.tasks.ant.Gateway.invoke$canonical$(Gateway.ceylon:96)
        at ceylon.build.tasks.ant.Gateway.invoke(Gateway.ceylon:88)
        at ceylon.build.tasks.ant.AntProject.execute$canonical$(AntProject.ceylon:91)
        at ceylon.build.tasks.ant.AntProject.execute(AntProject.ceylon:86)
        at build.scripts_.scripts(build.ceylon:243)
        at com.redhat.ceylon.compiler.java.runtime.metamodel.AppliedFunction.$call$(AppliedFunction.java:251)
        at com.redhat.ceylon.compiler.java.Util.apply(Util.java:1371)
        at com.redhat.ceylon.compiler.java.runtime.metamodel.Metamodel.apply(Metamodel.java:1646)
        at com.redhat.ceylon.compiler.java.runtime.metamodel.AppliedFunction.apply(AppliedFunction.java:403)
        at com.redhat.ceylon.compiler.java.runtime.metamodel.AppliedFunction.apply(AppliedFunction.java:394)
        at ceylon.build.runner.functionModelToFunction_$1.$call$(goal-annotations.ceylon:87)
        at ceylon.build.engine.executeTask_.executeTask(execution.ceylon:66)
        at ceylon.build.engine.executeGoal_.executeGoal(execution.ceylon:57)
        at ceylon.build.engine.runGoals_.runGoals(execution.ceylon:32)
        at ceylon.build.engine.processGoals_.processGoals(build.ceylon:43)
        at ceylon.build.engine.runEngine_.runEngine(build.ceylon:16)
        at ceylon.build.runner.start_.start(run.ceylon:45)
        at ceylon.build.runner.run_.run(run.ceylon:27)
        at ceylon.build.runner.run_.main(run.ceylon)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at ceylon.modules.api.runtime.SecurityActions.invokeRunInternal(SecurityActions.java:57)
        at ceylon.modules.api.runtime.SecurityActions.invokeRun(SecurityActions.java:48)
        at ceylon.modules.api.runtime.AbstractRuntime.invokeRun(AbstractRuntime.java:110)
        at ceylon.modules.api.runtime.AbstractRuntime.execute(AbstractRuntime.java:176)
        at ceylon.modules.api.runtime.AbstractRuntime.execute(AbstractRuntime.java:160)
        at ceylon.modules.Main.execute(Main.java:69)
        at ceylon.modules.Main.main(Main.java:42)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.jboss.modules.Module.run(Module.java:312)
        at org.jboss.modules.Main.main(Main.java:460)
        at ceylon.modules.bootstrap.CeylonRunTool.run(CeylonRunTool.java:245)
        at com.redhat.ceylon.common.tools.CeylonTool.run(CeylonTool.java:390)
        at com.redhat.ceylon.common.tools.CeylonTool.execute(CeylonTool.java:327)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at com.redhat.ceylon.launcher.Launcher.runInJava7Checked(Launcher.java:114)
        at com.redhat.ceylon.launcher.Launcher.run(Launcher.java:41)
        at com.redhat.ceylon.launcher.Launcher.run(Launcher.java:34)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at com.redhat.ceylon.launcher.Bootstrap.run(Bootstrap.java:55)
        at com.redhat.ceylon.launcher.Bootstrap.main(Bootstrap.java:32)
Caused by: ceylon.build.tasks.ant.sealed.SealedAntBuildException: Error running Ceylon plugin tool (an exception was thrown, run ant with -v parameter to see th
e exception)
        at ceylon.build.tasks.ant.sealed.SealedAnt.execute(SealedAnt.java:146)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at ceylon.build.tasks.ant.GatewayCaller.invoke(GatewayCaller.java:65)
        at ceylon.build.tasks.ant.Gateway.invoke$canonical$(Gateway.ceylon:94)
        ... 51 more
Caused by: Error running Ceylon plugin tool (an exception was thrown, run ant with -v parameter to see the exception)
        at com.redhat.ceylon.ant.CeylonAntTask.executeCommandline(CeylonAntTask.java:228)
        at com.redhat.ceylon.ant.CeylonAntTask.execute(CeylonAntTask.java:173)
        at ceylon.build.tasks.ant.sealed.SealedAnt.execute(SealedAnt.java:144)
        ... 57 more
Caused by: java.lang.NoClassDefFoundError: com/redhat/ceylon/launcher/CeylonClassLoader$1
        at com.redhat.ceylon.launcher.CeylonClassLoader.getResources(CeylonClassLoader.java:225)
        at com.redhat.ceylon.common.tool.ServiceToolLoader.getServiceMeta(ServiceToolLoader.java:36)
        at com.redhat.ceylon.common.tool.ServiceToolLoader.toolClassNames(ServiceToolLoader.java:71)
        at com.redhat.ceylon.common.tool.ToolLoader.getToolClassName(ToolLoader.java:60)
        at com.redhat.ceylon.common.tools.CeylonToolLoader.getToolClassName(CeylonToolLoader.java:40)
        at com.redhat.ceylon.common.tool.ToolLoader.loadToolModelMemoised(ToolLoader.java:110)
        at com.redhat.ceylon.common.tool.ToolLoader.loadToolModel(ToolLoader.java:103)
        at com.redhat.ceylon.common.tools.CeylonTool.getToolModel(CeylonTool.java:488)
        at com.redhat.ceylon.common.tools.CeylonTool.execute(CeylonTool.java:323)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at com.redhat.ceylon.launcher.Launcher.runInJava7Checked(Launcher.java:114)
        at com.redhat.ceylon.ant.CeylonAntTask.executeCommandline(CeylonAntTask.java:216)
        ... 59 more
```


One thing I've noticed a lot in Pontoon is this pattern. Django actually lets you traverse relationships to make these kinds of lookups easier:

```python
translation = entity.translation_set.filter(locale=locale, approved=True).latest('date').first()
if translation is not None:
    # blah blah
```

You could also optimize this by selecting all the translations for the locale with entities belonging to the given resource, and then filtering manually in Python. The call to the DB would maybe look something like:

```python
all_translations = Translation.objects.filter(entity__resource=resource, locale=locale, approved=True)
```

The code for that is more complex, though. A possibly easier but ultimately more complex solution would involve [prefetch_related](https://docs.djangoproject.com/en/1.8/ref/models/querysets/#prefetch-related):

```python
entities = Entity.objects.get(resource=resource, obsolete=False).prefetch_related('translations')
# Get an Entity instance from the entities list
translations = entity.translation_set.all() # No extra query necessary here!
```

I'd suggest reading the prefetch_related docs closely if you decide to try this. If you do go that route, I'd suggest adding a method to the `Entity` class called something like `get_latest_translation` that calls `self.translation_set_all()` and does the filtering in Python. That way the code here ends up just being `entities[0].get_latest_translation()`.


Sorry for the big text in the commit. I thought it was the comment area.


It's nice that you don't need different behavior here anymore :)


Double quotes aren't necessary here, better to be consistent and stick to single quotes.


Hey @lvca, you may have not seen this. There's a TODO here that wasn't mean to end up in the docs, let me know what I should write here so I can update this section.


Комментарий


No, but the Julia interface does (I convert within the calls).


Does this actually need to check __FreeBSD__?

If any platform had a sys/queue.h we probably want to be alerted.


You're right, thanks!

When dealing with padding I decided to not use OpenPACE:
https://github.com/frankmorgner/vsmartcard/blob/master/npa/src/iso-sm.c#L106-138


good job


ty


okay merge please!


lol


@TomHAnderson This is a BC break!!!!! This either needs to be reverted, or tagged in a new **MAJOR** version, with indications of how to update your code accordingly. Please see #199 for an example of the sort of problems it causes.


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


[Biggest BTC Transaction (size = 999657 bytes)](https://blockchain.info/tx/bb41a757f405890fb0f5856228e23b715702d714d59bf2b1feb70d8b2b4e3e08)


I think this sentence again raises the question asked by the reviewers of why we don't just prove bounds on mixing directly. I think we explain this nicely in the new intro but need something more here (my abstract didn't really do this either). What do you think about something direct, like:

The structure of the graph induced by tree rearrangement operations is an important determinant of the mixing properties of MCMC.
Indeed, topology changes, and SPR rearrangements in particular, have been identified as the main limiting factor of Phylogenetic MCMC algorithms.
As mixing properties of phylogenetic MCMC algorithms are difficult to characterize analytically, we were motivated to take the different strategy of characterizing the structure of the \emph{rSPR graph} in greater detail.


Most of the other code (creature choice, seats, ...) work like that so It doesn't sound weird to me


Can we change this to '%s' so we don't miss whitespace?


not the hardest ^^


I like this sentence, but isn't it a bit strange to note two lines above that the structure of the graph is important and then say here that we found significant structure? Maybe change "indicating" to "confirming"?


<script src="js/app.js"></script>

The "app.js" should match the file name.  So in my project I changed to the following:

<script src="js/application.js"></script>


Make sure the lib file names match.  On my system, I had to use the following.
    <script src="js/libs/jquery-1.11.2.min.js"></script>
    <script src="js/libs/handlebars-v1.3.0.js"></script>
    <script src="js/libs/ember.js"></script>
    <script src="js/libs/ember-data.js"></script>


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Why don't we just test for something on window.branch. I mean, if someone has something other than the branch sdk stored at window.branch, that's going to be an issue for them anyway. I also added a console error message.


and do this before the dom manipulation below, so that the test above will be falsey right away


Make sure the lib file names match. On my system, I had to use the following.

    <script src="js/libs/jquery-1.11.2.min.js"></script>
    <script src="js/libs/handlebars-v1.3.0.js"></script>
    <script src="js/libs/ember.js"></script>
    <script src="js/libs/ember-data.js"></script>


On 2015-07-07 18:05, rgetz wrote:
> Can we change this to '%s' so we don't miss whitespace?

Done.


Good point. I reworked the effect.


yup thats correct


Yeah, you must be right. After all, tiles being dug have a fullness decreasing on that scale. Nevermind. :)


rax retry tests


Using a static field to compare against getClass will fail across multiple compilations in the same JVM.  This occurred while testing the OsTrustedChecker in Checker Framework inference on GetClassTest.java in framework tests.


@TomHAnderson told you earlier about the downside of not having dedicated factories for REST services built from this abstract factories, and that you could not inject dependencies. By removing an instance of the service manager, it's even worse! Why not leaving it? Or proposing a PR and discuss about it?!


shouldn't be 'getRequestAdapter().untouch(page);' ?


I'd do this as `this._options` to prevent leaks.


changed file name


asdf


`instanceof Array` vs `Array.isArray` ?


Love this. Incorporating now.


Hi sudosurootdev!
Probably you didn't receive my message on your website, in that case I wonder if you can provide me some experimental commit/diagnostic advise about this issue involving all ST-Ericsson NovaThor hardware with any lollipop rom (except jb and kitkat):

https://github.com/zzpianoman/android_frameworks_av/commit/4dd20cf8df10e3f71897b01b0b4849b93ad8d6c0

I suggest you to read all comments there to give you a full picture of the situation.

Many devs gave up on this, but I'd like to have your personal opinion / assumption about the origin of the issue because actually it's unclear. We just know that every recorded video produces green lines, without any trace of debug.

Repository in question is here:
https://github.com/hastalafiesta/android_frameworks_av/blob/cm-12.1/media/libstagefright/ACodec.cpp


Please update only English.xlf. Other files will be updated later from Transifex site


I kind of liked the horizontal bars of "*" to help more obviously delineate where one turn ended and the next started, so I'm likely to add that back to one of the handlers.


@afd or constant?


This is GLES 3.0. Not sure if we support it yet.


nit: should this not be `RCTScrollViewManager`?


@adlai - I did test, but very lightly.... it does use the proper values in each variable... Full disclosure though, I am not a programmer, just a code hobbyist at best.  I'd have to go back and look again for sure, but I believe max_mix format is mixdepth, satoshibalance, ['0, 12345678', '1, 12435678', '1, 12345678'] etc, so max(mix_balance) is from 10 integers, the mix depths being the other five values ["md0", bal"].  So, the first value in the mix depth, which is what is being used in utxofulllist[max_mix], and this is replaced during each for loop with the next highest balance, rinse and repeat.

I could be way off... would really appreciate any further insight.  Thank!


Is this reachable? I'd expect this to come before `case LMFInvokeDynamic(lmf) =>` as `case indy: InvokeDynamicInsnNode if destinationClass == calleeDeclarationClass =>`


I think the same could be said for a lot of our flags - for instance, FLAG_CONST and FLAG_PARAM.  The high prevalence is to my mind an argument *for* using flags, because the number of fields on Symbols would grow to be very large indeed if all of distinct properties were placed on the type itself rather than stored in the flags field.  In our debug output, we'd end up with a type filling most of the screen rather than one with the most relevant storage information that can still be quickly queried for the property we care about in the moment.  Yes, to function properly this system does rely on us as the developers being disciplined about when we add certain flags that should be exclusive, but I think the parser handles that fairly well and a developer who is familiar with the language would be cognizant of these exclusivities.


Would be great to have failing cases in tests and tests for the case with both exceptions (`allExcept: ['exports', 'expressions']`).


Should we attempt to stuff this artifact into the database?


Yea, I was wondering about that.  Have not had a chance to try compiling yet.  Really the lines can be removed entirely (on ES) if that is what is necessary.  It will still work but take a sizable performance hit, but if somebody is building for ES they probably wouldn't use this anyway.


:-) nice new feature - this is relevant for option 2 - makes sense!


2014 -> 2015?


Fixed: https://github.com/nikic/php-ast/commit/12b5d4ff4568aa5662fe9fe78a8f266e1da11e80 Also enabled strict types while at it ;)


description update


@afd ```if data not in ["little", "big"]:```


de acuerdo ing.


updaing -> updating


@jkeiser shouldn't this say it's no longer supported in Chef 13? That's what the Resource one has. Also 12.4.0 seems a bit late to start saying something is deprecated and will be removed in Chef 12.


Is there any reason to keep all the commented out code around?


A commit for your systemUI FC:
https://github.com/sudosurootdev/frameworks_base/commit/c109747d38da29a1936fc5ed26ff26a52954cd29


@afd please document this (and also the new fields)


Yea I did that, but the compile within my app still failed. No worries. I'll explore when I have the time.


[Image of Norwegian share error](http://d.pr/i/18b8w+)

These changes should fix the issue with weird share text coming up on Norwegian.


I dunno...is `element` descriptive enough of a variable name??? :)


I wonder if you could use [trac-admin](http://trac.edgewall.org/wiki/TracAdmin#FullCommandReference) `config set` for cleaner editing of the trac.ini file?


That's what I meant when I said I got a lot from these conversations =)


Superb job!  I'm glad you challenged yourself with the extra credit work too.  And nice, clear code comments.   Really solid coding!

The only advice I have is that I found there is a built-in punctuation string in the string module:

import string
print string.punctuation

I got this by googling "python remove punctuation" and the first link (StackOverflow as usual) talks about this:
http://stackoverflow.com/questions/265960/best-way-to-strip-punctuation-from-a-string-in-python

It's generally a good idea to look for built-in Python constructs first and use those if you find them.  In this case it brings two advantages: it has one additional punctuation character, and the program might work for other languages which use different punctuation sets (although I haven't verified this).

Again, great job!


Should:
- [ ] Extract SignIn Component to own module
- [ ] Pass to Modal Component


It has been removed.


Why is this check not in visitAndVerify?

I am wondering if we can reuse code between visitAndVerify and visitAndChange.
Maybe call visitAndVerify here and let it return after.
This can then just write after.


I don't like the last three lines here, as they sound very similar to our original abstract and the response to that was "meh". I think a big part of this is that it is not clear to non-specialists why these results are interesting, which is why I tried to be more specific.

My ending was (or, rather, was intended to be):
1. Simulation shows that access time and curvature are connected
2. We identified difficult to traverse regions via negative curvature
3. We characterized the change in degree and used it to bound curvature
4. These results and bounds suggest that the bottlenecks we identified occur due to large degree changes.

Any particular reason why you dropped 2 and 4? The flatness theorem is interesting but it may not appear so at first.


Test comment Ben


@mairatma, @zenorocha reported that when having ids the bind doesn't work. Could you add a test case for that?


Hi Ashes999, sorry that I noticed your message so late.
I plan to add documentations to the repository in the future.
Right now you can show image this way:
https://gist.github.com/Moon4u/5e5bef71904552938175

Keep in mind that I'm making mruby-sdl2-image into a separate gem so if you want to try this be sure to include the mruby-sdl2-image gem as well.


@pHspec, nested JSON doesn't always have the same properties for each sub object which makes it difficult to represent as a single CSV row. That's why this only deals with flat JSON.


Is dry-run a better name for this flag?


ok I see ! Gonna fix that tomorrow morning.


I don't know what's wrong, just for your information: my gcc does not generate any machine codes from this line of source code. All is ok if I force cast 0x80 to char:

```c
/* CFLAGS=-std=c99 -DDEBUG -g3 -ggdb -gdwarf-4 -O0 -fno-inline -Wall -Werror -Wno-pointer-sign */
if (data[len] != (char) 0x80)
```

```bash
~$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.8/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.8.4-2ubuntu1~14.04' --with-bugurl=file:///usr/share/doc/gcc-4.8/README.Bugs --enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.8 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.8 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-libmudflap --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.8-amd64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.8-amd64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.8-amd64 --with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 4.8.4 (Ubuntu 4.8.4-2ubuntu1~14.04)
```


any implications of this?


I'll come back through this an update copyright for 2015 and both of us to authors yeah?


An -> A


Well, look who it is :). This is actually the `element` that the tooltip is being added to.


Do these need to be in a conditional block? I saw that in the main csproj file.

```
<ItemGroup Condition=" '$(TargetFrameworkVersion)' == 'v4.0' ">
```


Sorry to butt in, but what repository's which branch are you guys having this discussion on? I am really confused...


You don't need to change core behavior to make it work, actually you should not change core behavior if it's not absolutely required (bugs in core methods, during pr merge wait).

Or if you really really can't do something without changes in core but I can't easily imagine anything.

It's not uncommon to rip off core functionality and move it to plugins just to keep core small, consistent and efficient.

Basic thing with plugins is that they are not different from your application components like Controllers, Views, Models, Behaviors, Templates, ... only real difference is that they are easily reuable and can be distributed as separate packages.
You'll load them just as you would load your own application stuff, only adding YourPluginName in front.
See directory structure example here: http://book.cakephp.org/3.0/en/plugins.html#creating-your-own-plugins

About how this kind of functionality should be done, I'll suggest moving your View into Helper plugin.

Why?

I'm suggesting this because of fact that your original approach would ruin everything when, for example, using AjaxView or XmlView derived from core View. Or any other view that can't consistenly use javascripts scripts in its output.
Also, javascript loading has nothing to do with view layout and template controlling.

Another why?

Helpers are easily loaded through single function call from your own (application, not plugin) view class by doing, (plugin users would add this):
```php
$this->loadHelper('YourPluginName.YourHelperName');
```

Then it could be used in templates, you could for example add into your layout template file this to tell app about where you want scripts to appear (plugin users would also do this):
```php
$this->YourHelperName->autoloadJS();
```

And if you'll require, you could also add Component as middleware between user's application code and your helper. However, Helpers alone provides following callbacks which I think would be more than enough:

 * - `beforeRender(Event $event, $viewFile)` - beforeRender is called before the view file is rendered.
 * - `afterRender(Event $event, $viewFile)` - afterRender is called after the view file is rendered
 *   but before the layout has been rendered.
 * - beforeLayout(Event $event, $layoutFile)` - beforeLayout is called before the layout is rendered.
 * - `afterLayout(Event $event, $layoutFile)` - afterLayout is called after the layout has rendered.
 * - `beforeRenderFile(Event $event, $viewFile)` - Called before any view fragment is rendered.
 * - `afterRenderFile(Event $event, $viewFile, $content)` - Called after any view fragment is rendered.


it's good to always check user input when taking parameters:

    try:
        filename = sys.argv[1]
        factor = float(sys.argv[2])
    except IndexError:
        print("Need filename and binning factor: ")
        sys.exit()


Yes.  The Microsoft.Bcl.Async package is a requirement for this to compile in .net 4.0.  Those features are built-in for .net 4.5 and later, so that package isn't required.


super legit


Thanks Andrey - for now, let's leave it as the "sandwich" with introductory paper synthesizing prior literature and meta-analytic paper synthesizing the new, coordinated IALSA results.


"is being" is misleading.  The v0.5.0 no longer uses powerstrip.


IndexError because if the user doesn't give enough arguments, then sys.argv will be a shorter list than expected

in general, when doing try/except clauses, you have to know what error the code will try to throw ahead of time and use that to control the behavior


Very nice!


this was supposed to be misspelled


Sorry it was my mistake, I got a minor HTML thing that prevented one of the events to be triggered.


Actually... Looks like this command isn't working properly...


So I feel much better now :+1:


Thanks @kaarthiksundar


Using += in both of these locations means we add again all of the previous errors (saved in nerrorsOfAllPreviousCheckers).  In SourceChecker.typeProcess (which is called at the end of this method) we compare against errsOnLastExit to determine if the Javac Compiler ran into compilation errors.  Counting these errors multiple times was causing this check to pass when in fact we ran into compilation errors.  We then typecheck code that failed compilation leading the checker framework to throw a runtime exception.


Why is int32 wraparound relevant? I think that's independent of whether the
opcode signatures are mode dependent (which I would argue against).

On Tue, Jul 7, 2015 at 9:11 PM, Luke Wagner <notifications@github.com>
wrote:

> If there are separate opcodes, wouldn't that imply that you could use
> int32-indexed loads even with a 64-bit heap? While this certainly could
> work, I was thinking it might complicate internal heap optimizations just
> by adding another case to consider (in particular, concerning wraparound of
> int32). That's the nice thing about having the signature of these ops be
> depending on the heap size declaration.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WebAssembly/design/commit/57bdfbe7d8beef86bccf5af4854dfb1f6968651d#commitcomment-12046492>
> .
>


Well, I agree that there is an exclusivity issue with many flags and so am unsure how it become so popular.

I'd be tempted to argue that it would be better if certain symbols did indeed have a large number of fields as
that would make it clear that something has gone hay-wire.  Falling back on all of these ambiguous flags hides
the implicit complexity.

But again I think I'd like to separate 2 distinct issues; (1) what's the appropriate API for buildModule() (2) what's
the preferred implementation for determining the visibility of a Module?  My immediate focus is probably (1) as
the issue of representing a lot of stuff by flags is pretty wide-spread.

Mike


thx, shoulda done this long ago.
line - 147 is wrong though --- should be **IOPlatformUUID**

`
ioreg -rd1 -c IOPlatformExpertDevice | awk '/IOPlatformUUID/ { split($0, line, "\""); printf("%s\n", line[4]); }'
`


I believe it's safe.


Anyway, I like the idea of the fullness being handled by the tile. But if it bothers you, we can change...


Volvo makes great cars, but their ability to pay attention to small things like this are kinda lacking.


again, I would wrap this in a try/except clause. so adding to the above,

    except IOError:
        print("could not find the following file: %s" % (filename))


Wäre es nicht besser, eine Dictionary-Eigenschaft zu haben, die Bezeichner und Wert von (zusätzlichen) IDs wie diesen erfasst? Das würde einerseits die Notwendigkeit eines deutschen Bezeichners eliminieren, andererseits aber auch genug Flexibilität bieten, um die verschiedenen regionalen Systeme zu unterstützen.

Man müsste in diesem Falle natürlich die Spec dahingehend informativ erweitern, welche Form von IDs gemeint sind. Da bieten sich durchaus als Beispiele die in Deutschland verwendeten an.


It's seemingly a lot more error prone now than before this update.


I am getting the following error: `Uncaught TypeError: Cannot read property 'onItemDelete' of undefined`


I've updated the PR without these changes.


looks like a good `change`!


:hamburger:


in general, you want to only run the following if the file is executed in standalone mode (python filename.py <args>)

to do so, add this check:

    if __name__ == "__main__":
        <all your code>


Hm, we start with all ebuilds having SLOT=0, slot move that to 4, fine, but what about 9999 which in the next commit is changed to SLOT=5?


No, each package has a has local setup file expanded from: https://github.com/ament/ament_package/tree/master/ament_package/template/package_level

This local setup file will source all the environment hooks of this package.


@gitster Sorry, this is jk/safer-git-path on peff/git, which I'm planning to send to the list today or tomorrow. You can access the commit via `git/git` because the objects are shared, and we don't enforce reachability for arbitrary sha1s via the web interface.

Did this end up in your notifications somehow? I'm wondering if you have a "watch" on `peff/git`. We don't usually discuss here (I usually send to the list and we do review there), so this would probably be the first time it would come up.


true indeed


yeah, wtf, right? i guess it's a leftover of a misunderstanding, can be trashed :)


> Anyway, I like the idea of the fullness being handled by the tile. But if it bothers you, we can change...

Nope, it's fine to me. :)


Humm, there should be a way to handle this much better (environment variables?)


Actually this adds caching for 9 days - is this needed? - Why not just rely on the caching of the IE (maybe also set INTERNET_FLAG_CACHE_IF_NET_FAIL in HttpOpenRequest)? - or at least only cache for 1 or two days?


Good to add this since it was missing, but what do these do anyway?


Actually I didn't change slots here, in tree is SLOT=0 but 4.9999 already 4, just added new package with 4 as well.


Hi Nathan. I think perhaps all of the changes to this file in this commit are unnecessary because of all the type aliases we've added, the complete implementation of MSSQLDatabase#getDataTypeMaxParameters(String) and the new XML Type we added. What do you think?


Done: https://github.com/chalk/supports-color/commit/fff7edef9ed076cb3c9f2e283025cadd93a745f0


@robyurkowski I just added additional caching around conditional blocks.


Ah, okay.  I bet that your problem is that you're attempting to replace a column that isn't being supplied. You presume that the "modified" column is being supplied in the update, but you're not actually supplying it when you call it in line 179.  You need to filter it out if it exists and then add it so you can handle both situations....like my example I sent you. :-p


@matheusportela: The reason behind the 25 magic number. :P
For some reason, the current rgb dummy video has its fps info corrupted (or at least OpenCV thinks so).
The video itself works fine, but if the loop rate is left unchecked, and turns out to be invalid, the node won't run.


Also, if you go to settings->config vars in your Heroku app, there's a predefined `DATABASE_URL` from Heroku's free tier that seems to point to a PostgreSQL database somewhere in Amazon, i.e:

`postgres://user:pass@ec2-54-217-202-199.eu-west-1.compute.amazonaws.com:5432/dc948134kjfy`

From what I've seen with a quick look... perhaps we could use that database instead of running it in the same dyno of our Django webapp?


Separar la conexion a la BD, y la identificacion de usuarios, filtrar los datos del formulario


Please add a `.gitignore` to this repo ignoring all files that end with a `~`, otherwise the diffs/PR get cluttered.


Comments are fine, here, too.


I think I mentioned it in your blog post in the comments (awaiting moderation):

http://www.lohitaksh.me/index.php/2015/06/23/bugs-bugs-and-more-bugs/

But I'm going to do it here again, this repo is great reference/inspiration for our usecase with Heroku:

https://github.com/lightning-viz/lightning


Should we add our names to the MIT license?


I think by this point in the code you should be able to just do 'instant-clean'? Shouldn't the sourcing that happens in check dolfin set up the paths correctly?


Can we we have GPU defaulted to true, and display defaulted to false?

I ran this thing all night in CPU mode on accident :dancers:


Make sure that - (Object) latest_pages_build(repo, options = {}) can get it right now that you can get some sleep 💤 😊


It probably needs to be 4.8.3.


Can you please include the three missing figures?


You will be sad to learn this is not the only place where this occurs; however, it is not prevalent and your arguments make sense to me so I will change it.


انتخاب به عنوان محصول پستی


Try and avoid single character variable names. As programs become larger, it can become difficult to debug when you have no idea what "d" means.


ingeniero agregue los casos de uso


Hey jlin -- PR's have a one button 'revert' for commiters that we can use here.


This is not a good idea: it is going to stop you from having multiple sets of imgCheckboxes on a page. See https://github.com/jcuenod/imgCheckbox/commit/e2beb474a9d745c9e465b0c86c948cc551d612b5#diff-a60b5656604b18886c310c8092f2e98fR109 (use $wrapperelement instead).

Thanks for the work though, I have merged many of your improvements.


`error` is a non-standard method and will throw if `native-or-bluebird` returns a native promise.


[![Food vessel gui](http://api.artsmia.org/images/830/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=830)


[![Dagger-axe ge](http://api.artsmia.org/images/835/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=835)


[![Dagger-axe ge](http://api.artsmia.org/images/836/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=836)


[![Food vessel fang-ding](http://api.artsmia.org/images/837/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=837)