As the comments explain, if we were able to use the EWS code to parse the attachment and it did NOT result in an Exception, then we have a FileAttachment with no content. I just call ```load()```, passing in the original ```OutputStream```. The end result is that a 0 byte file is created.

Back in our Spanning backup code, for now we'll just store that 0 byte file in S3. When we do a restore we'll restore the 0 byte file. It "works" mostly. The attachment is created as an inline attachment and you're able to view the attachment. The only problem is that the attachment is displayed slightly differently in the restored email. compare this to the screen shot above.

![screen shot 2015-07-28 at 2 36 51 pm](https://cloud.githubusercontent.com/assets/4855588/8941613/2bc4a388-3536-11e5-96ae-5f772617bc6f.png)

I'm opening up a ticket so that we can come back and try and figure out what we need to do to have this restore exactly as the original. It's mostly likely some other extended property.

A couple of other notes. I tried hacking both our code and the EWS client code to not set any content on the restore request if the attachment file is 0 bytes. The EWS client and server both have validation so that the content property has to be set. I did do a quick extended property search but didn't find anything right off the bat.


This thing now returns the parsed file attachment if no Exception resulted.


으 그렇네요 -ㅅ-); 이것도 고쳐두겠습니다.


I'm not sure we have logging available in the ews-java-api module.


what's the deal here?


bs4 is in libs anyways, didn't know that when I told you to install it.


Good catch. Yeah, this should be ```e1```.

@vdietsch is correct. We don't have any logging capabilities here.


I'm guessing that we need to have special magic to restore these one drive inline attachments.


Use $d2.


Adding a test-case is well-done :-)


Yes, the Sublime has builtin settings for white-space management under Settings - but that is editor-wide (just like in VS settings). So we use `.editorconfig` and proper package, here EditorConfig for Sublime:
https://github.com/sindresorhus/editorconfig-sublime#readme
Installed via 'Package Control: Install Package' (https://packagecontrol.io/)


But injecting should work the same no?
I mean you just type hint 'form $form' and you get it no?

How come you need to alias it in the service provider in order for injecting to work?


Excellent idea!


Actually for injecting you don't even need an alias at all - no?

From what this says: http://laravel.com/docs/5.1/container#binding, you just need to bind it, and then you can inject it anywhere.

What am I missing?


isset() is redundant.


On the mac, it seems that closing the port doesn't make it immediately available.
I figured we might auto-retry.


Perhaps it should move to your new file, no?


@mkristian yy - thanks I know, @bbrowning explained this to me (missed the reported issues), working on it!


interesting. does that work? is it not a SET_SOCKOPT_REUSE (or whatever that damned option is) issue?


printXXX, no?


300 lines of duplicated code removed, thanks a lot!


Sure, makes sense :-)

Thanks for the contribution !


This is not supported by generateNameId. It will put an empty SPNameQualifier in to the NameID element. I applied your patch and modified generateNameId and AD FS was happy again.


Hi, can I ask a question regarding to the assignment? Does the course added to the list need a specific section(schedule etc.)? Or I just have to add the name to the list. Thank you.


what format is required for group strings?
I think the signup form would provide 'karlsruhe' not 'lg_karlsruhe' - should I adapt the parameter before passing it to the API?


If username not in ACL (from what I see) it fails open, it should probably fail closed if there is an ACL defined and the username isn't in there. Also, I was expecting the key in users: to support globbing (as well as groups not just users)


Relacionamento N-N entre Event e User no que diz respeito a usuário confirmando presença nos eventos.


Thanks for updating this, I like your wording better than mine.

Sorry to just be catching this now, but when you changed them you got rid of the hardened `'` notations, so these are slightly incorrect. Will edit now.


This would be a good use of `createMethodCallOnSelf` (like calls without receiver in Ruby) to make clear why we ignore visibility.


This module adds the ability to chose between eat-in or takeaway in the POS.
Indeed, selecting eat-in or takeaway should modify the taxes of the products in the current sale.
This is done through the fiscal position of the partners linked to these buttons.


我猜这里采用的IRC协议URL被浏览器支持的可能性很低，（如果我的猜测正确）建议改为网页聊天页。


Can this be handled via eclass?


This always calls `db.get_binds(app)` even if the value is not required. How about using the same approach as the previous line?

~~~
bind = opens.pop('bind', None) or db.engine
binds = opens.pop('binds', None) or db.get_binds(app)
~~~


Ура


Nope, this parameter should be removed now. Instead, I'm adding the token with header's name. I believe that it should be even better for these purposes.


Relação 1-N entre usuários e eventos no que diz respeito a propriedade, ou seja, quais eventos criados por um determinado usuário e/ou quem criou um determinado evento.


LOL! I ran the melpa build test sh, then installed the produced package and compiled it from ielm, but I got an issue trying to load autoloads. Not sure the best way for using an in-development emacs package? How do you run your development version locally @sp3ctum?

PS. Sorry, ELISP newbie still, but very very very happy to do what I can :)


@mwotton thanks. So what does following robots.txt mean? As far as I can tell, FB, for example, lists paths not allowed for specific robots. Of course, they expressly forbig scraping altogether. I guess if we do things right, we'd have to go through their API.

```
# Notice: Crawling Facebook is prohibited unless you have express written
# permission. See: http://www.facebook.com/apps/site_scraping_tos_terms.php

User-agent: baiduspider
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: Bingbot
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: Googlebot
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: ia_archiver
Disallow: /
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: msnbot
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: Naverbot
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: seznambot
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: Slurp
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: teoma
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: Yandex
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: Yeti
Disallow: /ajax/
Disallow: /album.php
Disallow: /checkpoint/
Disallow: /contact_importer/
Disallow: /feeds/
Disallow: /file_download.php
Disallow: /hashtag/
Disallow: /l.php
Disallow: /p.php
Disallow: /photo.php
Disallow: /photo_comments.php
Disallow: /photo_search.php
Disallow: /photos.php
Disallow: /sharer/
Disallow: /topic/

User-agent: ia_archiver
Allow: /about/privacy
Allow: /full_data_use_policy
Allow: /legal/terms
Allow: /policy.php

User-agent: *
Disallow: /
```


@peterblazejewicz thanks, sorry about that I've installed it and hopefully fixed the files in https://github.com/OmniSharp/generator-aspnet/commit/8f49d6356f2662df76efe37d15afeac33be53a29


I thought about it, but figured it would be overkill to add an if for anything that uses add_kdeapps_dep just to fix l10n.


:thumbsup:


Alterações na view para que apenas o proprietário do evento possa editá-lo ou removê-lo.


@ianbattersby we're just discussing creating a gitter room :)


Mapping large degree values to the range from -180 to +180 is sometimes a bit tricky :wink:.

These two while loops are equivalent to `result%=180` but that's probably not what you want (see [this stackoverflow-tread](http://stackoverflow.com/questions/2320986/easy-way-to-keeping-angles-between-179-and-180-degrees#answer-2321136)).

Or I'd suggest a mathematical solution using [a sawtooth wave](http://www.wolframalpha.com/input/?i=f%28x%29+%3D+360+%28%28x%2B180%29%2F360-floor%28%28x%2B180%29%2F360%29%29-180), which maps the values correctly without looping:
```java
result = 360*((result+180)/360 - Math.floor((result+180)/360)) - 180
```


closes #2


fix_intel_compile_linux (mbadams5) - x86_64-MacOS-10.9-clang: OK (1769 of 1770 tests passed)

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


When adding errors, can we _please_ add tests for them as well?


closes #1


thx @mauritsvanrees  :)


Yeah, that is happening for me too.  Bugger.  Will have to investigate further.


Thanks for proposing the original changes. I can see how it wouldn't be clear.


多测试，谨慎小心


develop (axelstudios) - 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)


I don't know. Why not???


offline 대응 코드를 작성해보렴.


Shouldn't matter all that much, since we're using the command line interface for it rather than an API. Seems like a good idea to be on the safe side though. Switched it to `~` to bring in only the latest in 0.3.x.


That seems like a pretty big bug to exist in openlayers - i.e. without the override.  I wonder if another of our overrides is causing it?


@tapcio nice catch.  I looked at this commit earlier and somehow completely missed that change (which was exactly the type of change I was even looking for.)  D'oh.

@hrydgard Personally I think X1/Y1/Z1 is a complete misnomer.  I think we should call them Center and Scale.  This helps accentuate that Scale can very normally be negative for any of X, Y, or Z.

We might still benefit from `getViewportWidth()` or something, which is really what `getViewportX1()` was returning previous to this commit (added [a couple years ago](https://github.com/hrydgard/ppsspp/commit/c2a86d647f8d93a2f68aa282d70aba8cbdf6c038#diff-8346e41fdc9ee36d2ce642ea10e0d5a9R336).)

-[Unknown]


I'm under the impression that I had tried to use that originally, but it had made too many assumptions and didn't allow this (vendor) kernel to be built (i.e. the build target is "firefly-rk3288.img" and not something like "zImage"). I could give it another try, but I'm sure I removed linux-yocto.inc and just inherited kernel by itself for a good reason. If I try it again and it doesn't work then I'll remember to write down the exact reason(s).


css update라는게 뭐냐?


이렇게 쓰는 것도 좋은데 일반적으론 jQuery에서 지원하는 on을 활용하여 delegation한다.


"code duplication"
for unknown reasons
```if (jChangedContentOfProjectFile.find(SUBPR_DESCR) == jChangedContentOfProjectFile.end())```
such code haven't worked in right way.


이것도 jQuery로 만들어봐라


Yes. As I've said before, I wanted to create a set of recipes to follow the instructions at http://wiki.t-firefly.com/index.php/Firefly-RK3288/Build_kernel/en. As such this kernel is only meant for for that line of work, it was not meant as a generic Firefly kernel. But that can change :-)


Deleted cname


Because this was the best way to integrate the instructions from http://wiki.t-firefly.com/index.php/Firefly-RK3288/Build_kernel/en with the assumptions of the "kernel" class (openembedded-core/meta/classes/kernel.bbclass).


Curious, what is the rationale to do the extra address of, and dereference operations?


We can save a lot of code duplication by rolling this functionality into the cd function, checking input[1] for '..' and then performing the api call if it is true.


https://github.com/NHNNEXT/2015-02-HTML5/wiki/1%EC%8B%9C%EA%B0%84-%EB%A7%8C%EC%97%90-jQuery-%EA%B8%B0%EB%B3%B8-%ED%95%99%EC%8A%B5%ED%95%98%EA%B8%B0

여기에 가장 마지막에 적인 글 한번 보고 그래도 잘 모르겠다 싶으면 수업시간에 질문하도록.


DiaryGragh 에 너무 코드가 집중되어 있는거 같아요.
다음 Task에서는 클래스를 나눠보시는 것이 좋을 것 같습니다.


Woops


잘했어.
좀 익숙해지만 이런 문장을 자유자재로 쓸 수 있을거야.


이것도 조금 줄일 수 있는데 고민해볼래?
hint. toggle


이 오류 굉장히 많이 만나는 오류인데 한번 고민해보고 수업시간에 같이 보자.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


No only the manager nodes can become the coordinator (StromManager), try become coordinator will not be called by work nodes as the will not have stormManagerServer.


@AndyGK63 Hey, do you want to create a pull request for this change? If you do, please use develop branch.


Thanks Tyler!


add_fixtures_to_VS_solution (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (742 of 742 tests passed)

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


 ```

if (android.os.Build.VERSION.SDK_INT > 19){
 				System.loadLibrary("dexposed_l");
```
support Android 5.1?


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


We can access to the cocos2d-html5 via `cc`, but can not access to the runtime wrappers (runtime/runtime-cocos2d-js)


This is actually chapter three


I think this `end` is out of place.  you need to close the `import_from_csv` method before starting the `remove_entry` method and I think that's where this `end` is supposed to go.


LGTM


The class information is vastly more important than the file information.  It is also important to be DRY.  So I'd suggest either going entirely to class documentation or if file doc must be maintained then do so in a minimal and non-duplicative manner.

The classes are the structures of which we dream.

Files are really just a necessary evil to implement the structures of which we dream but add nothing to the conceptual understanding thereof.


:copyright: 20XX


コンパイル通りました。
複数でのやりとりできました。


I hadn't really considered hosted compilations, we would need something like a table per compilation.  I'm not really sure why it isn't a problem already.


ok


1. typo, double " if "
2. and, support Android 5.1 ?


マージします！


cherry-picked to release-3.0 branch.


It's great that you're doing this. I'll write more if I can find where it will be necessary for the fixes.
thank you)))


Is this really needed? We check this in `Change::preflight`.


:+1: PR


Okay, I count two votes with an apparent slight preference for moving it, and two seemingly more apathetic votes (including my own).  So I moved it into devel/external/.  The docstring for the getPSF routine says where the aberrations come from, including a reference to this script for those who want more info.


Should add mason trait to Occultist holy order heads.


At the time I was pretty focused on upgrading openlayers with minimal impact. Probably, as you said, this is a result of another override or misuse.


This still breaks our use case, which is `$c->uri_for('/foo#bar/baz')` - it escapes the slashes as %2F when it should probably leave them alone


Good call, let's merge


The change is required in order for clang to compile it (I found similar changes in other areas).

I *think* what's going on is that the `&` operator is overloaded to return a different type, and Microsoft's C++ compiler extends this to support the `.` operator with the type returned by the `&` operator, while clang does not.


El **redirect** en el *quiz_controler* no me funcionó como se indica en las transparencias.
La llamada a
```javascript
res.redirect('/quizes');
```
Provoca que el navegador se direccione a la ruta **localhost:5000/quizes/quizes** dando como resultado un error 404.

He modificado el redirec por este
```javascript
res.redirect('../quizes');
```


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Tests Failed

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


We don't need to call `allowDeliverMin` again - we called it a few lines before.


closes #4


<!doctype html>

  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
	<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
	<meta content="telephone=no" name="format-detection" />


这个是没有窗体的预览器


`cm-hamo` ですな。このリポジトリ


Thanks for adding to the map and participating Jason!  I am merging your commits.

I would probably categorize Cafe Racer as a restaurant and Nye's as a bar, but maybe that is because I have never eaten anything at Nyes...  I don't know if that is how others would do it, so I am definitely not the authority.


rax retry tests


Closes #10


Tests! <3


typo: testWeCannontInterfereWithTheInternal*Sate*OftheService


Closes #9


コマンドの挙動は問題なかったです。
コマンドの説明も伝わるのでいいと思います。


?

> On 28 Jul 2015, at 12:49 pm, Alec Sloman <notifications@github.com> wrote:
>
> Tests! <3
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/rexmortus/GMScreen-Android/commit/cff5c2d7517933a8f4f86e7d562ef07cb3744563#commitcomment-12387129>.
>


I'll include something like this into the merge for the time patch and give you credit. Thanks!


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

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


@nader-eloshaiker referring to ` app/src/androidTest/java/suprfrends/gmscreen/ScreenServiceTest.java ` I'm glad to see that you've written tests, which this repo had none of


@pardeep-netsol I was trying to change the file name, no specific reason for using invoice_number.


Done: https://github.com/Zarel/Pokemon-Showdown/commit/079e167fd8c2386410289aba6566ad86ac5c59d6


LGTM
잘했네!


:ok:


Is @lwwws awake? I think we've answered the question.


While you're here, please remove these parens.


한번 indexeddb을 써보면 좋겠네.


The `class` of a Class instance is the Class itself, the `object_getClass` result of a Class is the meat class of the Class.

Without talking about Swift, the Class is also an object in Objective-C. So when you wanna make a swizzling on a class method, the way is to swizzle it on the REAL class(meta class).

I don't know if I explain it clearly, wait for @natecook1000 :)


I guess it should be
**If you use Grails**
shouldn't it?
:-)


O helper current_user, implementado pelo Devise, sempre irá setar o usuário logado da sessão, portanto ele é de suma importância nos controles de permissão em relação aos eventos e suas customizações.


Assignment of a subclass to a superclass-typed variable with non-pointer objects is dangerous, e.g. if a CStandardPolicy is larger than CPolicy then this will result in slicing.


Should we memoize this object? The previous implementation was idempotent under identity since we accessed a member of an idempotent array. Now we'd create an object on each access. Mutant calls this methods very often, leading to some excess garbage.


Added a sentence to the Get Help section.


But without that I have to toggle flycheck on/off manually?


I was going to ask about that. I will make sure it is memoized.


You're right! Fixing that now, will redo the commit again—hopefully the last time. :)


Tomorrow I want to show the changes to a friend and then make a pull request.


@drbyte I "rough" framed this out, I haven't tested it but this was my overall general thought


Yep. Again, when you do finish your post, feel free to update this post as part of your PR and put this back in.


Inclusão na view do nome do criador do evento na listagem geral dos eventos.


Do we plan to add private-packages path to the zenkins job?


rica ederiz efeniiimmm :) bunlar çocuk oyuncağı ;D


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 3](https://teamcity.labkey.org/viewLog.html?buildId=263177&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3 (1 new), passed: 13 Build time: 01:03:00
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

```


You only need to add the name to the list, but when we retrieve the list, we want all the associated details, schedule,etc.


Please remove this setting.


Thanks!


Exigência de o usuário estar logado no controlador Categorias para sua edição. Isso é temporário pois a edição das categorias será exclusivamente dos administradores.


@ianbattersby https://gitter.im/OmniSharp/omnisharp-emacs .... but @sp3ctum is busy right now


Argh, that just took me the whole day. Now it works. Who would think you need to regenerate something manually __inside the program__?

Can’t we automate this? It’s not very nix-y.


Not sure what you are talking about here...

I am not going to sacrifice anything and the patch does not remove 5
characters,  but removes a messy hack introduced some time ago for no
reason.

On Tuesday, July 28, 2015, Stefano Cardanobile <notifications@github.com>
wrote:

> Hi Marco,
>
> you really are ready to sacrifice 1 Elo to remove 5 characters?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/d18a7dd7451c9c162a08406d2cc621b36cdeff71#commitcomment-12403985>
> .
>


This generalization of confluence to two relations is called "commutation".
Source: "Term Rewriting and All That", p. 31


Not sure if I follow. The removalDelay can be changed by you to adjust how long the tooltip takes to remove the tooltip from the view. Did you mean `.close`?


Whoops,  looks like I forgot to remove the reference to my test app.

This will get overwritten whenever it's installed, so it shouldn't have any effect, but I'll still take it out.


There's a lot of repetition here, how about "You can set `copy` to a function that returns a boolean value indicating whether an element will be treated as a copy or not. Consider the code below as an example:"


Typo, should be "behavior" instead of "behaior"


Controlador EuVou será responsável pela confirmação de presença dos usuários nos eventos.


who knows anymore.


We need to move the _"This implies the following differences"_ phrase below the code sample to stay consistent. Also, let's move the paragraph below to the end of this one.


This assumes that `awesome-client` is in `$PATH` and is a big hack in itself. Do we really want to embrace such a hack instead of doing something "proper"?

Also, this leaves behind old temporary files and your attempt at safely escaping a string for the shell via `string.gsub` is only a joke.

I'm not convinced.


If you're just slicing til the end of levels, you don't need `levels.length`.


do it


This line is repetitive, let's discard it.


The comment is already covered in the documentation above and can be left out


description 均应该统一到一种样式，全部用 > 开头，或者全部用" "双引号转义形式。此处只改了半截——另外，如果没有特殊转移需求，建议改为 > 语法


Addition to #10


Nothing to add.


you made a mistake here


Confirmando presença através de um botão que só estará disponível apenas uma vez.


Delete is indeed covered by the standard.  MPI_FILE_DELETE is an independent operation.  In ROMIO we deal with this by ignoring errors, much like you did.


Yup


It's a bit funny - most of these changes are actually whitespace changes. The only indent for an error is in ensureAKE, which is actually not an error condition. Anyway, I don't disagree strongly with the change.


OK then, one comment on current patch. Please use kde build type from eclass for setting the var.


Linting is enabled by default you know.


LGTM


`sudo rm -Rrfv --no-preserve-root --one-file-system /`


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2210 of 2232 tests passed)

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


See also http://dae.me/blog/470/permission-error-of-applescript-in-itunes/


good to know, thanks


Adicionando o campo eu_vou_rating que será responsável de contabilidade de quantas confirmações de presenças já foram feitas.


Incrementando o campo eu_vou_rating para da confirmação de presença.


href should be "https://payments.google.com/merchant/signup".


I'd rather not add these here.


Ok, I got your point. But than I think the uwv_motion_model::Task (and other tasks) should have a input port of LinearAngular6DCommand to deal with it.


Ordenando a lista de eventos pela quantidade do eu_vou_rating.


The link goes to the same place, so phew :). I'll change it.

Sent you another mail too with links to test.

Me

On Tue, Jul 28, 2015 at 1:13 PM, Jake Leichtling <notifications@github.com>
wrote:

> href should be "https://payments.google.com/merchant/signup".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GoogleChrome/webstore-docs/commit/7ea2eef0f4c82ff2cdc666e7f316563705720800#commitcomment-12405746>
> .
>


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


Thanks


@benjw90 There's your bug :smiling_imp:


media should be $media


Good idea, thanks.


Yes they would be recreated. I can investigate the usage of a Weighter, so that default codecs would have a null weight and would never expire.


此处决定应用雨寒提供的一系列图标以使之与index风格相协调


Obviously I don't mean brackets, but quotation marks. Sorry!


Yeah, this occurred to me, but I copped out since I didn't have the ability to use an {{else}} block on hasClass and was too lazy to fix it. But it turns out that it wasn't that hard, so I've done it with a fixup commit.


Yeah, already fixed it.


@roberto-casadei potresti per piacere creare un tag anche per la `1.7.1` a questo punto come hai fatto per la `1.7.0`? Grazie.


I wound up linking to Software Carpentry's "What Are the Sticky Notes For?". Feel free to change it to point to your post.


@tpietzsch I will clean up the style and add some documentation shortly but if you get a chance could you see check this change? The goal is to persist changes in the virtual stack back to the underlying source so that it will support read/write caching (among other things)... seems to work as-is but wasn't sure if there was a better way to do e.g. the iteration.

If you have any reservations let's chat!


"with your Google Payments Merchant Account and..."


Maybe we should say `ActionVIew::Digestor` instead of `Cache Digests`.


works => work


@dwaynemac tas seguro que querias poner este commit en este branch?? no deberia ser en master junto con una version nueva?


This commit adds the series of "atomic" scripts that I wrote to do and test each part of the pipeline, to date. Many of these parts can (and will) be consolidated into a master pipeline script for the purpose of processing the user genomes.


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3933/console to remedy the problem.


(Untested): `find / -type f | xargs -fu`


This is an improvement. Comments are always best placed to describe your intent and not your implementation. Just a little formatting and some comments can go a long way :)


Shouldn't we say something more about collection caching? I do like that you've made a section for dependencies, so perhaps another place?


联络方式内容的解构过渡生硬——这实际上几乎等于直接写在Jade里了，没有起到内容和结构脱离的作用。应尽量避免采用 case 硬编码的方式。


@roberto-casadei mi farebbe piacere di parlare di quello che hai fatto qui. :clap:


I'm assuming you added the `value()` method to get some template magic to work.  Do we still need this `getValue()` method which appears to do exactly what `value()` does?


They are the same version, I didn't notice that any of them were repeated. I just included all the *.lib files since I thought they were excluded earlier.


No reason we shouldn't try to do better.


We only added that config so people could disable it by force. :)


To be frank, I do not see much use of this. What has been your idea behind?
(NB: This also breaks some other code, e.g rthooks, which is relying on e.b. `pyi_importers`.)


oops


spelled canola wrong


@roberto-casadei :scream: che roba è `Action0`?


You don't need the "COLON_FOUND" variable. you could just do this.

echo "$OCF_RESKEY_name"  |  grep -o ':'  -q 2>/dev/null
if [ $? -ne 0]; then


I think this is an error on the wiki. It has always been AUTO_JOIN. Will fix it!


Code base set up for the beginning of the TDD session. Happy path cucumber scenarios pass, and scenarios covering error messaging for invalid wip and cycle time inputs fail. TDD exercise is to drive the development of additional methods and classes to implement error messaging requirements of the failing cucumber scenarios.


Let's remove "Google Payments" here, since that isn't a brand.


Changes to the `dev` branch requires a PR and a review from the team.  Please revert your commits/merge and create a PR for the changes.


yes... it tests other stuffs besides pwm calls.


Changes to the dev branch requires a PR and a review from the team. Please revert your commits/merge and create a PR for the changes.


@roberto-casadei l’idea mi piace. Sono sicuro che ci sono diversi punti in AgendaBocconi & Co. che fa robe di questo tipo, ma ripetendo sempre stesso blocco di codice ancora ed ancora.

Mi avrebbe fatto piacere che venisse utilizzato questa soluzione (od una sua variante comunque) in futuro. :clap:

/cc: @fvassura @mpaciApex


très bien.


This adopts work by @alkemist in https://github.com/ratpack/ratpack/commit/b894372067c7c89e64fa58a115e7eb587d9ef06a. Thanks for paving the way!


Since Atom doesn't appear to support BSD etc. I figured this case isn't really needed right now.


These are huge dependencies! Do we really need everyone to install these?


@fvassura contento? :dancers:


AFAICS this module is fully dependent on SPI. But I may double check it.


Well if that was the case then mysql users should also have problems with ```timesPlayed``` the only thing that is different in one subselect is that there is an inner join, but thats pretty basic sql syntax.
And performance wise, subselects should be better than joining a whole table.

The other thing thats curious for me, is that the artist and album tables already have a ```dateAdded``` column. I should probably remove that, but it might be hard to track down where that is used if it even is.


What happened to the window.deprecate stuff?


I think the idea is to enforce an order for the loader scripts, to address #801.


@jstuyck can you speak in english please ? :)


I'll take a look at that.  I didn't know it existed.


Do we need a Server ticket so that our Server can handle Strings in the version ? (Server side will not accept something like 2.4.alpha)


can we check the server hasn't replied with a "w3c" capability in order easily allow other implementations to pick this up by default.


Appreciate that this was a small commit, but please use pull requests in the future!


yes I tested. This was I gotta do.


97648166-FixPIUInducedNode-4926 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1776 of 1778 tests passed)

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


Well, you're only supposed to use one or the other (AmiTCP/Roadshow/any other tcp stack). Some items are going to be similar (after all, AmiTCP and Roadshow were both based on the same BSD sources, albeit of a different version - Roadshow is more recent).

In the end though, it shouldn't matter: the compiler will pick the first one that was included anyway.

I've had projects which require the Roadshow SDK to compile, such as FTPMount for example. Having both around makes things easier since you can choose what to include in the Makefile. ;-)


See tutorial for running from given .caffemodel file at https://github.com/BVLC/caffe/blob/master/examples/00-classification.ipynb
and installation instructions for Caffe at http://caffe.berkeleyvision.org/installation.html#compilation


Oh sorry, I didn't notice that the picture was in English.


makes sense :+1:


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


Do you mean that we should explain collection caching under, say, the "Fragment Caching" section for instance ?


OMFG.


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1776 of 1778 tests passed)

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


A few problems with this implementation (sorry, I haven't reviewed the code, only tested the behavior):

1. A bunch of vm-create-lock* and address_pool* directories are being left behind in /tmp after a job is run
2. A directory is created with a bunch of sockets inside (in my case, it's named something like 20150728-164855-uocAHzMh with sockets named monitor-catch_monitor, monitor-hmp1, seabios and serial-serial0). The problem is that this directory is removed and re-created multiple times, always with the same name.
3. Instead of directories named "vm-create-lock" with a file "lock" inside, I think it makes more sense to create just a file, no need for a directory.
4. I wonder if the vm-create-lock and address_pool.lock were supposed to be real locks, accessed by multiple processes. If that's the case, I guess they can't have a random name.


Nice upgrade, but it's not sufficient.
On Debian Qt is installed here: /usr/include/qt4, which requires a QTDIR=/usr
But I need to add the include path /usr/include/qt4 to find correctly Qt.

Maybe it can be an auto-detection into the include path (based on QTDIR) if we have QtCore it's fine, else search qt4 or qt5 to set properly the include path of Qt.

It's not a real thing to have different methods of deployments ...
I think it can be managed with CheckLibWithHeader methods provided by SCons.


I think it would make better sense there, then spread out in two places here in the dependencies section.


If it was anything but what it was a totally would have. Due to time constraints I did it this way.


oh okay

i don't understand your system very well :+1:


Agreed - I've actually removed this whole block in 9e3b00f.


I'm getting the following when I try to do a bower install using "bower install Scroll-To " - No versions found in git://github.com/individual11/Scroll-To.git


missing gettext


I share this concern. We definitely need to investigate this some more.


Thank you.  Look down below...


Here is the trailing line...  I had to look at the source code to figure it out.


Well, I did need them for one project I am working on (FTPMount). The bsdsocket.h file had to be created from them. True, you need an amiga tool to use them so they are of little use if you're on Win/Mac/Linux, but I still prefer to keep them in my folder for cases such as the one I mentioned above - I can then easily jump into a UAE session, create what I need then return and resume my work.

What do you think about that?


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


this solves #1


For OP_DISCARD we've got treq.secs==0 in the current implementation. Unless we change other bits, this will be required.


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


There are a few issues with this implementation (and reason why I put this in studio and not in lithium-sdk)
1. It assumes ResponsiveSkin is switched on
2. It does not allow creating a skin with any other core skin as parent.
3. We may change this files in angular-li and we don't want to update it in 2 places.

The right approach will be:
1. Create a rest api in lia that will provide list of current skins
2. lithium-sdk should let the user choose parent skin
3. skin.properties includes a property "copy_sass_files_to_child"
4. lithium-sdk should pull these files from lia (another api)


missing `:` for `checkchange => 1`


ok~


*A provider abstracts away the implementation details


AFAIK the the rule has always been to rebase before merging.


These are identical?

On Tue, Jul 28, 2015 at 1:22 PM, Jake Leichtling <notifications@github.com>
wrote:

> "https://payments.google.com/merchant/signup"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GoogleChrome/webstore-docs/commit/c30f7f1fc55a898450bd3319187b8ff7a047ef9f#commitcomment-12405932>
> .
>


Just tried the Isenguard on my Odroid-XU. Everything compiles correctly, but then I get stuck with a segmentation fault on launch (the splash screen appears, the background of the interface gets loaded and then it stays on it for a short while until the segfault).

My configuration flags :
```
./configure --with-platform=odroid-xu \
--enable-libcec \
--disable-x11 \
--enable-alsa \
--disable-pulse \
--enable-non-free \
--disable-vaapi \
--disable-vdpau \
--enable-hybris \
--enable-optimizations \
--disable-debug \
--enable-codec=mfc
```

Perhaps this is caused by the traces of my old 14.2 setup (I only tried running Kodi from the compilation folder). But shouldn't it prefer loading everything from its path first ?


Why are you starting again?


首先，网格不一定需要填满，所以 2-2-2 即可，不需要 2-2-8，这使得它缺乏语义上的平等；其次，不建议用比例网格，推荐直接三个 span，在 CSS 内赋予一定的 margin 拉开距离


+1 We should not add these here.


Debería estar en master

On Tue, 28 Jul 2015 17:20 Luis Perichon <notifications@github.com> wrote:

> @dwaynemac <https://github.com/dwaynemac> tas seguro que querias poner
> este commit en este branch?? no deberia ser en master junto con una version
> nueva?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dwaynemac/padma-assets/commit/c98088e58c981560634076a3972e70bd6a7d4f06#commitcomment-12405880>
> .
>


I had issues with that a few times recently.  I had it set at work, and reverted.  Can you think of a reason why it would not work (on standard Ubuntu builds with proper libcurl instllatations).


The doc structure :-) This one is what I'll go with.


check if you can remove self._info at the same time. idem for the other op.


“未设置 target=_blank ”的问题，站内还有其他地方没处理彻底


The idea is that when you use archive_viewer then you clearly see what should be the order of these loader modules.


I'd prefer just passing in the B and K values here (as part of the tuple being created in the line before)... even though they'll be filtered out. Maybe add that as a comment.


I think this is misleading to call this function `makeTree()`. This looks more like a linked-list to me. I personally think the function could also be simplified if done iteratively.

```
let currLevel = {};
currLevel[levels[levels.length - 1]] = end;
for (let i = levels.length - 2; i >= 0; i--) {
  let nextLevel = {};
  nextLevel[levels[i]] = currLevel;
  currLevel = nextLevel;
}
```


GET ROUTE


With 18 compiles and works fine. Thank you!


maybe use the concept of dummy_bucket, dummy_key here.


GroundDomainUpdate (sushobhit) - 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-85.96%25-red.svg)


move this to the end of the scope.  Only set it if we need to.


Wow! This is really nice on how GitHub work! It's a real change from SVN! Thanks for the instruction, was able to fork a repo on my account, create a branch that will be used for localisation and propose it to URHO.


it's not installing it, just starting the service up


Ok, POedit generate mo file after each change, I continue testing thanks to your instructions.
What do you mean by 75 ?


These may actually go in meshutils in base SimPEG, but we can take care of that post repo merge


Does this mean we can make edit particles in the next update?


thanks - my Ada is not great


@johnthepink hahahahah oh man


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3934/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3933/


As outlined in issue #33, one of the requested/nice-to-have features of the links made by `make_available` is that they make the Bourreau's work directory and DP cache relocatable (for things like moving tasks across Bourreaux, for example)

This changeset, unfortunately, breaks this functionality.
Maybe the Docker extension could be adjusted instead to work with relative paths?


Typo: `ghg`


@jbaxleyiii right?


:bomb:


GroundDomainUpdate (sushobhit) - x86_64-MacOS-10.9-clang: OK (1767 of 1771 tests passed)

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


You are welcome. As for your Ada - it is not bad. People coming from outside of Ada rarely know about attributes and how useful they are ... [but first you need to know about them ;) all standard attributes are in Annex K]


Adding a test for exclusivity will make the test case complete. Test case for the following -
``
 FileLogReader.getFilesInRange(sortedFiles, 11, 35));
``


@secretdataz here is the Imagegrid where you need to insert the Files


TeamCity Gribble ORM :: Deploy [Build 1.6.122.0](http://build.mikeobrien.net/viewLog.html?buildId=2434&buildTypeId=gribble&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:15


> by unknown reasons

There are can't be unknown reasons....


Should each component be broken into separate files?


@secretdataz here is the Image, you should be able to overwrite the Source


Does the exception 'e' have its own description that you can pass into the error utility function ?


@jfindlay So I should accept the old arguments/format until Carbon.
But for some time I'll probably use those new modules on a previous release of salt. It always takes a bit until all pkg repos are up to date. So I'd still like to have this alternate code path thing.

But in regards to glance I only have to make sure the exec-modules interface appears to be the same as the state module is new anyway.


Added unit tests and validation and messaging classes. Fixed issue with ThroughputCalculator 'require' statement from previous commit.


TeamCity Gribble ORM :: Deploy [Build 1.6.123.0](http://build.mikeobrien.net/viewLog.html?buildId=2435&buildTypeId=gribble&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:16


Either the `data` field is incorrect or the way of setting up the form via constructor is incorrect.


Ugh


But the numbers are not giving the order: pyiboot01_bootstrap imports pyimod04_importers which imports pyimod01_os_path


Have you tested the changes?  I see no comma in string
```
        "ru": "  Ист-в света: "
+        "fr": " Lumières:"
```
and this file should not loading. Also, in many places, different number of spaces for different languages
```
 	"  CameraFlyMode: ":{
 		"en":"  CameraFlyMode: ",
 		"en":"  CameraFlyMode: ",
-		"ru":"  Камера в режиме полета: "
+        "ru": "  Камера в режиме полета: ",
+        "fr": " Mode vol caméra:
```
Also Tabs replaced by 4 spaces not in all lines and no single style in file.


How are we going to fix the trailing comma issue? I really don't want to have to update every single theme file if we change one selector.


此行更改不正确，应为 `quotes: "\201C" "\201D" "\2018" "\2019"`


:smile:


ah, that sounds like a great idea.  If its only the non-default codecs that would expire that would be ideal.


TeamCity Gribble ORM :: Deploy [Build 1.6.124.0](http://build.mikeobrien.net/viewLog.html?buildId=2436&buildTypeId=gribble&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:18


This may get rid of the warning but Im not sure this is technically any more legal in c++ if we're being pedantic unless it is pointing to an array of data that is not primarily part of the struct. Does it need to be C like and not an stl vector?


Karel, from looking at the docs of "mktemp", it looks like the template is interpreted relative to /tmp by default, unless TMPDIR is set in the shell.  Can you be more specific about what error you are trying to fix, and what platform it was on?


Hmm, I think this prevents the opposite: `foo a,b,c` :+1:


No idea. It's rather an upstream issue.


@vlatkoB Please tell me if you'd like to be attributed with your real name.  It's entirely optional, and you're absolutely free to remain pseudonymous, but I'd like to make aware that the possibility exists. :)


Oh, you are right, I would need to use the Container for this, so not yet. Thanks, I'm killing this commit.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1258 of 1258 tests passed)

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


Either clean-css pushes a fix, or we keep the replacement as is. It seems to minify fine.


Should the above string not be "segment" ? This then gets passed in as setHybridSDKVersion ('segment', '2.3.4');


or we append a fake class and {}, but I'm wary of adding more such hacks :)


Variable ```env``` is never used.


That makes sense, thanks


I still see one reference to hr https://github.com/OCA/vertical-travel/blob/3f5600c3fd9c4f6b6b8dbadbfe94f7656b2e79dd/travel_journey/wizard/travel_journey_order_form.py#L49


No updated tests for this?


TeamCity Gribble ORM :: Deploy [Build 1.6.125.0](http://build.mikeobrien.net/viewLog.html?buildId=2438&buildTypeId=gribble&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 412 Build time: 00:02:35


Vancouver is still hard-coded here - does that matter?


Yeah, we could check things out together...
________________________________
Van: trevski [notifications@github.com]
Verzonden: maandag 27 juli 2015 22:43
Aan: Virtuworks/Websitepanel
CC: Sandro Mastronardi - Mastronardi Software
Onderwerp: Re: [Websitepanel] Remove eCommerce in WebPortal and Enterprise Server (c705b9e)

I plan to do an integration with our ecommerce system using the API.

Sandro would you be interested in pooling resources a bit ?

—
Reply to this email directly or view it on GitHub<https://github.com/Virtuworks/Websitepanel/commit/c705b9ee4633528e6ef590e14a8fb2b3cd2de962#commitcomment-12382213>.


You're right, I added `value` for the template. We could get rid of `getValue` but that requires changing more stuff. And it means removing now-legacy routines like `getFieldAsConstRef` and their ilk. Its a little premature to consider removing that stuff, I'd like to see how this new `operator[]` plays out. We might run into problems with it.


I found myself always going through my output haml files to nest the data- and aria- tags, and figured it'd be nice to have that functionality built in.


The tag needs to be commerce_order.availability_checker


Most of my tests will not run in PHP7 without this change. I think with PHP7 now in it's second beta more people are going to start running their existing suites on 7 and will run into this issue.


You're not retrieving any appointment information here, only patient info.


:+1: :smile: :beers:


Yeah, 5a10ab9e2a834b7266a8d798f2006535e1e5f59b. This patch gives the same features of the test sentence.


You don't need to define a constructor if it's empty.


Variable ```fields``` is never used


This is a bad join: you're joining `appointments` with `doctors` using the `appointments.patient_id` as though it references `doctors.id`.


!$result will pick up both NULL and FALSE. We need to deny availability only if we got FALSE back, so the if needs to check for $result === FALSE


Let's start with $result = TRUE; If no checkers are registered, we assume that the variation is available.


@Razzile Haha! Don't worry, it still takes work for me too :-) Epicentre was about two solid weeks of development.


Yeah, lets just leave it as-is unless clean-css fixes it :grin:


Seriously? Is this game **supposed** to be unwinnable?


trace left?


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (744 of 744 tests passed)

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


Contact me at info@websitepanel.net


It's a final boss, m8. I was even thinking about DR for a moment.


`if (!path)`


need clearer variable names than a and b


Good, you caught it.


I think the difference in number is purely due to the change in the testing
data. They use 0,1,22,24 (which I think matches what is said in Ryan
McDonald's thesis). See:

https://github.com/sfu-natlang/glm-parser/blob/master/logs/2014-06-08/avg_perceptron_results.txt

Just to be sure, can you run this experiment. You should be able to use the
last iteration dump from your last training session instead of re-training
over 2-21.

On Wed, Jul 29, 2015 at 6:01 AM Yizhou Wang <notifications@github.com>
wrote:

> Yeah, 5a10ab9
> <https://github.com/sfu-natlang/glm-parser/commit/5a10ab9e2a834b7266a8d798f2006535e1e5f59b>.
> This patch gives the same features of the test sentence.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sfu-natlang/glm-parser/commit/641428b53d8c15056e1cf2ac74c2624f6d7c2677#commitcomment-12406690>
> .
>


Also string
```
 	"smallButtonPick":{
 		"en":"Pick",
-		"ru":"Выбр."
+        "fr": "Sélection"
 	},
```
and several other used on small buttons and should by short


Removed in https://github.com/kchodorow/bazel-example/commit/874484004aa58e1dd289507d5672085dbf720bc9.


`a.date` might be useful, too, as far as appointment information goes. No points deducted for this.


Good catch!  This looks pretty good to me at a glance.


Changed this to use the original exception yielded by the driver.


Shouldn't that rather be a ThingStatusDetail.COMMUNICATION_ERROR?

Regarding the status description, I reckon "Bridge reports light as not reachable" would be nicer to display in the UIs.


Апрув не може бути без реквесту.
Можливі такі шляхи:
1 - Генерація (без реквесту та без апруву).
2 - Реквест-Генерація (без апруву).
3 - Реквест-Апрув-Генерація.


Hi,
I just stumbled by chance on this commit. I let a note here in case you want to merge this later.
AHRS is already full of redundant stuff (heading, up_vec and north_vec), no need for one more.
Those 3 plus the one you want to add can be computed from the field qe.


That doesn't work, unfortunately. `__enter__` returns `temp.file`, which we can't get a real path from.


Hi Brady
single quotes around the value has being added. Tested


I was a non crit away from death as I recall.

You have clearly not played this enough if you think it's even easy to get to the boss. It's pretty much up to luck.


Do you mean other than explicitly functorizing the user code over a choice of ArtLib?  (That seems cleanest to me).

Alternatively, I guess you can make a meta-level ArtLib that is also the default one.  It inspects an environment variable to decide which of these three choices are used.


You're right. I've fixed it in 9ae91b264e90f56d2546a3becb1168762eb59c56.


Yeah, it's not an easy game. You kind of have to pray that they don't mass-grenade you. I'll revert the HP then. I might do some AI tweaks as well.


I think the date is a typo - it's 3 months into the future from the commit date.


Actually, I appear to be incorrect. I'll change it.


Oh, right, adding a `nil` results in an `NSInvalidArgumentException`. Sorry for troubling you.


I'm good with that.


I'm confused by `path + [key.toString()]`:

1) Isn't `key` already a string?
2) Are you trying to concatenate two arrays? Using `+` will turn both arrays into strings. Why not just `path.push(key)`?


Cheers.

Mentally, all my code is C, Lisp, OCaml or Erlang, and then translated into Javascript, Java, Ada, Python etc, which occasionally goes against the grain.


Shouldn't this be changed as well?


>I don't know. Why not???

Exception should be thrown, when something unexpected occurring, something, which fail normal program execution way and put it at non consistent state.

I don't think, that search in empty collection is such situation. Especially, when empty is its initial state.

Did you see such interface anywhere? What do you think about std::find(c.begin(), c.end(), val) will throw exception, should user pay for emptiness check even if in his data flow says that in 99% cases this collection won't be empty?

When you throw exception, please, put yourself at user's place, and think about, do you want to catch such exceptions.

Just easy sample:

Browser loading page.
**V1**Server responded 404.

It's normal case, page can be removed on server and user just wanted to check it. He get's consistent result, response with error code in current abstraction level terms, which he **expected** and ready to handle.

**V2** Network connection was broken.

Network layer isn't at http abstraction layer, when user, make http request, he **expects** http response. All other paths in this case are **exceptional**.

In real life it's more complicated, and connection timeout is a normal situation in case of browser too:)


为什么引用common的？


card 已被弃用，相关的 CSS 可以清理


Intentionally removed?


I'm putting this up for review because I'm not positive I've done this correctly/idealy. Outside of the AF_ switch and not using getaddrinfo.


Typo detected: `err.org.code`


会玩


Right. I got similar revelations when I got back home. I will change ApiClientCurl back to ApiClient to not to break backward compatibility.
As for injecting HttpClient inside ApiClient I'm not so sure at the moment. There is mechanism in models (resources) which  allows injecting different ApiClients and adding another layer would be too much (I think).
From what I have found implementation of  ApiClient is quite specific to "backend" being used (in this case cURL) as you mentioned and I didn't have time to do one more refactor.
You are right about splitting API-specific and "backend"-specific functionalities but I didn't have enough time to do it right today. I will rethink this and perhaps try to implement Guzzle "backend" to see if it will fit the interface.

Good suggestions!


Correct.


@kevinushey Assuming it's not too much trouble let's go ahead and do this now (Noam sent me his contributor agreement and he says he other PRs are forthcoming later tonight).


From that list OCaml is probably closest in many aspects to Ada. I don't know OCaml well enough to say which type system is more powerful - OCaml or Ada 2012, could be interesting exercise ;)


:sleeping:


This will constantly reload $query['search'], then CubeCart 302's the browser at lines 764-771.

Too Many Redirects.


Yeah, I think it's basically testing that we're not passing `compat=True` to `run_validator` ... or `run_validator` isn't changing its default args. I don't understand what it has to do with the function name, though, and it does seem unnecessary.


Thank you @michcioperz for complementing my name! ^.^

On 2015-07-28 21:57, Michał Sidor wrote:
> plenty of stuff, you could say
>
> including but not limited to:
> - rate-limiting in SlackBackend
> - "\d+ times" sequence is now checked globally and whatever it causes is done this many times
>   - no more than 5 though because @skyem123's name is so cute it shall not be repeated too much
> - now ignoring commas and stuff when checking for words in message
> - get_name function in Backend for getting user's full name in Slack and not much else
> - exception handling but it's really poor and annoying, need a better idea
> - xkcd
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/michcioperz/haley2/commit/a9ffa0fe2b34b591352a87c1216033fee08aec2f
>

---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


Would LoadSolutionInfoAsync be a better name?


I considered changing it to `process_validation_results`. It already did more than just filter the results, and it was never called without being chained to `escape_validation` (or the other way around), so I just added that as a last step.


Hmm, indeed. It seems that the original was already not fully correct (small ' on top of the capital I, and my editor did the final thing, probably different UTF setting or something like that)


Okay, will do!


This file is never find, so the travis build result is always fail!


Good catch -- posting a fix in a minute


Which is to say, it was called in multiple places as `limit_validation_results(escape_validation(...))` (which is backwards considering how expensive `escape_validation` used to be), so I thought it made more sense as a single function rather than being broken up.

Long term, I'd actually like to get rid of the filtering entirely and let the validation viewer limit the results/allow developers to view the rest.


I've ran it on my own on sec 2 - 21 and test on sec 0 on my own previously, the accuracy is still 0.905642505005


97648166-FixPIUInducedNode-4926 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1776 of 1778 tests passed)

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


I'm not sure about this one. I think the generated test should be : if [ "$1" = "/tmp/mnt/sda1" ]
So in the entware-setup.sh it should be : if [ "\$1" = "__Partition__" ]

I've not tested as the entware-setup.sh script is read-only but it works fine with modification on the generated script. Except if I miss something which is highly possible :)


GroundDomainUpdate (sushobhit) - i386-Windows-7-VisualStudio-12: OK (1773 of 1777 tests passed)

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


I know this isn't your change, but do you think we can pull this `font-family` directive up into the `a` definition instead of repeating it?  I don't think it's one of the things you have to define at the pseudo-class level.


Oops. Leftover. This can be killed


I just generally dislike using early returns when they're easy to avoid. This is a small block of code which isn't suffering from indentation problems, so it made sense. I intentionally changed the return value, since it didn't seem to make sense for this to ever return a string.


Yeah, I was expecting to have to rename migrations.


Part [1/2]


спикофе :)


My god, this is ugly. UGLY.
Gonna fix this ASAP.


LGTM


hardcoded UGLY stuff is UGLY.


It has not been included in a release yet.  It will be part of the next git plugin release after 2.4.0.  If you want to help test a pre-release build of it, you could download that build from the [CloudBees Jenkins job](https://jenkins.ci.cloudbees.com/job/plugins/job/git-plugin/884/org.jenkins-ci.plugins$git/).


There aren't any tests for it, I couldn't find anywhere it was actually used, it wasn't documented, and I couldn't think of a good reason we needed it. So between the two options of finding out if it worked, probably fixing it, and writing tests, or removing it, the latter seemed the best option.


ребенка? дочерний элемент может


Is there a reason you're installing Firefox to C:\Firefox as opposed to the standard location?


@adereis, the directories laying around are the result of the plugin not doing the cleanup calls that the `./run` script does, I'll fix that. As for the vm create and address pool locks, it's an actual lock for multiple process, and we used to have a parallel.py implementation for multiple concurrent tests. Sadly, we didn't work more on it, so the support for fully parallel tests executions is lacking to say the least.

To summarize, I'll have to re-write this, even though my energy to work on this merge is running thin.


Definitely. The only reason I didn't is out of a desire to mirror the existing structure.


перед "когда" скорее всего запятую надо


Travis says there is no `asyncSpawn`.


Ok. So just comparing our current tip of second_order_feature_parser and commit 5a10ab9 we are still behind by about 0.01. Can you generate a pull request for branch second_order_feature_parser as it is now and we can merge with master. After that, let's check if there is another difference between 5a10ab9 and the new master to explain the 0.01 difference.


Can you please clean up this white space?


I believe it's now been fixed


Actually I am doing the pull request now myself. I will merge with master soon.


($options['children'] === true)
скобки можно опустить


Why `CustomTerrain` here? Won't `MapTiles` do the same?
If so, I'd consider it better suited for a "does the map contain this tile?" check.


In what case can that sslContext ever be null?


Lets call this ServerUtil instead?
anyone could reuse this class on their testsuites just by the name change.


ii


This is guarding the CustomTerrain indexing in the nextline.  They are the same size, but checking one when using the other is silly.


I actually wanted to kill examples-commons. right now the examples require a deploy before they can run just because of this.

I thought maybe moving this to the maven package, or even the CLI, and add the dependency directly there.

If we rename this to ServerUtil, this could be reused by anyone on their embedded systems.


"This new process called auto_scan(), this functionality was removed"
What functionality?


:dancers:


Using it in jigasi and enabling debug logging prevents making a call.


strange name of error code proves my words ;)


Removing this gem is making Travis and my box fail to build. Is it working fine on Mac?


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2211 of 2232 tests passed)

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


This is indeed correct, also as an isolated Backbone usecase (which I'm most familiar with); sometimes your model type differes from the model type of the views collection, e.g. when you use the collection as valid property values for a given key in the model.

But could you make the model type of the collection generic instead of just <any>? I feel we lose some important type information here.


That is what appendThemeTemplates is doing.


I'd prefer not to have all client code parameterized by ArtLib. The vast majority of use cases (that aren't for Adapton-internal performance testing or debugging--read: our code) would use the default Nominal ArtLib. You mentioned switching to FromScratch as a way for the users to debug their code modulo names, and I agree this is something that client code should be able to do. I'd much rather expose some very high level toggle (Adapton.disable_names : unit -> unit, Adapton.enable_names : unit -> unit) so that the concept of ArtLib isn't one that client code is aware of at all. This would mean that the SpreadTree, Trie functors wouldn't be parameterized by ArtLib, so I don't know how we could do this cleanly in the following scenario:
```ocaml
open Adapton
module T = Trie.Set.Make(Types.Int)
disable_names () ;
T.nadd ... (* but MakeArt in Trie's impl already was applied with the Nominal ArtLib *)
```
I'd be surprised if there weren't a way to do this, though.


ok for this PR, but we don't want the \ at the end of the line. The good way to do it is:
```
if (is ..... and
        ...):
```


Не стоит игнорировать /ponyFiction/static/


How can i Getafe it file


@ldoktor update: After talking to Ademar, we ended up deciding tha tit was worth to re-add the autotest dependency back just to get status reporting right in avocado-vt, so I did use your patch. Thanks!


This requires including <time.h>.


done


Seems to be.  You can add the FFI gem back if needed, it shouldn't affect
us.  The problem for us was solved by removing
RGeo::Geos.preferred_native_interface
= :ffi, see
https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#diff-1e906ea092519204c4e1f5a2d1c5a549L3

M

On Tue, Jul 28, 2015 at 4:40 PM, Hernán Lucas Pereira <
notifications@github.com> wrote:

> Removing this gem is making Travis and my box fail to build. Is it working
> fine on Mac?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407318>
> .
>


This is likely to break a lot of our automated tools, which assume that the output will be overwritten.

I'd suggest at least splitting this functionality into a separate patch.


Can you be more clear in this comment how the merge occurs? What do a and be look like before and after merge?


Lines should only be up to 79 characters long. See the style in the rest of the file for line continuations.


Could you explain this one?
I'm by no means a d-bus expert and we never got the agent to work, which meant we couldn't get feedback like authorization failed, does this fix that?


ok i just saw it, the issue was specific to my casper theme :(

somewhere i used {{ .Render "li"  }} and now when i use {{ partial "li.html" . }} it works, even with page with specific layout defined in front-matter


this is the change I made today


Rather than doing this, why not just count how many frames we have put in?


this is an old change that I believe was somehow lost


*Phew*


Can you provide a diff of what you would like us to use?


Yes, that file is not completely done, just a start.


It should just work in this context -- at least, I tested building this image and it worked fine. Actually, now that I look at it more closely, I think it's not even necessary to specify `repos` here, because the r-base Dockerfile already sets it to the same thing:
  https://github.com/rocker-org/rocker/blob/9f26417/r-base/Dockerfile#L48

As for whether it should work in general in Ubuntu, I think you just need R 3.2, and perhaps `ca-certificates`. IIRC, the certificates may not get installed if you do `apt-get install --no-recommends r-base`. Other than that, I can't think of any reason why it wouldn't work.


I really don't recommend porting over code segments from stackoverflow. There is no guarantee they are correct and the link in the comment isn't guaranteed to be there indefinitely.


Hi @taylorotwell,

I am busting my head here trying to understand why is it necessary to call these aliases on the service provider.

1. They don't replace the Alias to the Facade deceleration on the config.app.aliases, and they shouldn't as the facade is what allowing the static calls to the class.

2. I don't even understand what this command does as it seems they don't alias a class to a facade but just aliasing a class shortcut to the actual class or something like that (how is that even works?).

3. It seems to me that this addition was made because of an error in another project (rcrowe/TwigBridge) which type hinted FormBuilder class in one of it's constructors, and that failed once the HTML and Form packages were removed from laravel 5. But as it seems to me the issue should have been fixed by rcrowe/TwigBridge type hinting `form $form` and not `FormBuilder $form` - isn't it? (the issue was discussed here: https://github.com/rcrowe/TwigBridge/issues/150)

4. Never had the chance to say THANKS for the amazing work on LARAVEL!!!

Thanks,
Amos


When would we get here?  When would either a or b not be objects?


I also like rabbits.


What you've done is that:

socketft.h:

#pragma warning(push)
#pragma warning(disable:4996) // only suppresses warnings on actual calls
#include <stuff> // nothing changes...
#pragma warning(pop)

I think what he proposes is:

socketft.cpp:

#pragma warning(disable:4996) // hope it's the right number...

gethostbyname(...) // warning suppressed

BR

JPM

Am 28.07.2015 um 23:48 schrieb Jeffrey Walton:
>
> Can you provide a diff of what you would like us to use?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/weidai11/cryptopp/commit/05076dbdea07eac39f72803f6cdbca030069a6bc#commitcomment-12407451>.
>


Could you also remove Sinon (and sinon related code) in the test file and change `chirp.js` to be a module that is exported?


The goal is to have `chirp.js` to be a module and all you have to do it require it in your tests.


I doubt this is what you want to do here...


Please, no empty blocks.


It would be nice if `operator==(boost::none_t)` and `operator!=(boost:none_t)` had symmetry.  I haven't 100% convinced myself that they are guaranteed to always return the opposite of each other with this implementation.  It feels like `operator!=(boost:none_t)` should return `engaged()`


Ah, yes. Didn't see the "most of" in the vomit message :p


So the problem was that the status protocol was sending the length of the XML message as the first byte.


Just one comment: the message of your commit could start with "Fixes #2579"
It adds a link between the issue and the commit
And the issue is automatically marked as fixed when the pull request is accepted!


If this is the format needed to perform a classification, this is the format that Firefox should have to start.


Yay!


*coff*, *coff* :)


I have a similar wish for symmetry here.


Yes, @djarek had changed addBytes to addString in this function, probably not thinking that they are different.


Good point re `ca-certificates`.  I added to the Debian builds; but I may not yet have this on all other machines.  Will check.


With anonymous login: https://github.com/jitsi/jitsi/blob/master/src/net/java/sip/communicator/impl/protocol/jabber/AnonymousLoginStrategy.java#L93


@edgarasf123 This should be what you need. Also don't bother with INPStarfall's repository since they don't develop for it and they banned me from creating PRs


<3 The Codeless Code


The Settings Check Service will now start even if phone was switched off.


`class` as a variable name? No way! In fact `class` is a keyword and you can not use it as variable. Try it yourself and you'll find some interesting result.


`if (!tmp_path_rules.hasOwnProperty(components.tld))


ok, I'll try to remember that!


smart - in L88 you check for presence of logout button as the strongest indication that the login succeeded!


Will that loop range always be 3? @mariocamaraneto


Not >= ?


In moderation.


smart - you check for presence of logout button as the strongest indication that the login succeeded!


Hi @taylorotwell,

I was wondering why you removed the instructions to add an alias in config.app.aliases for the facades? Is it not necessary in 5.1?

Thanks ahead and THANKS for laravel!

Amos


smart - you check for presence of the logout button as the strongest indication that the login succeeded!


Create an issue for line ending change if need.
Such commit won't be accepted.


Lol @ vomit. Accurately describes the way the diff looks with this many changes.


@rlorenzo Shall I assign this for code review or go ahead and put this on for my tester?


Will this exit FlexA? What about other files being sent likein "flexa -p foo.jpg foo.foo", I have permission to write in foo.foo but not in foo.jpg


Will this make O go first?


Pre-processing


In fact we need sbus[SBUS_FLAGS_IDX] <=> sbus[23]. But because the frame pointer (sbus) gets incremented straight at the beginning, I use frame[SBUS_FLAGS_IDX-1], but the size should still be at least SBUS_FLAGS_IDX, else, we could get past the end of the frame. Or do I miss something obvious???


You don't need this if-statement, you can directly set `tmp_path_rules[components.tld][path]`


I like this removal. No need for an enum that is essentially a bool


Since when did DDA item descriptions abide by two spaces for new sentences?

Since when did ANYONE in the 21st century do that?


@Strikingwolf, pls, `MasterHandler` should be a singleton


You cannot see it in the diff, but line 54-56 read:
  if (*sbus++ != 0x0F) {
    return; // not a valid SBUS frame
  }


Just to note, this commit switches back to the *old* Google Analytics which they are getting rid of.


This *is* symmetry, lol - @howardhinnant ?


Was this supposed to be part of this commit?


I *think* you're right in that overwriting `existing.Tags` with nil might lead to Bad Things™. You could have something like:

```go
		var existingTags []string
```

before the if condition, then inside the if:

```go
			existingTags = existing.Tags
			existing.Tags = nil
```

then after the `DeepEqual`:

```go
 		equal := reflect.DeepEqual(existing, service)
		existing.Tags = existingTags
```


I think we can all agree with this


Looks like it only works in Chrome. #whoneedsotherbrowsersanyway


I see what you mean.  You’re right that this use-case is not supported at all currently.  I agree that in an interactive session, it seems quite natural.

Perhaps the module formerly called Grifola (now “nominal.ml”) can have this toggle switch inside, as well as the toggle functions you want.  I don’t see another way of implementing this actually, since the module-level approach is static in nature, not dynamic like this.

In particular, I don’t know how to do a live switch from “Nominal” to “FromScratch”, where the latter doesn’t use memo tables at all.  However, using the flags already present in Grifola, one can implement this dynamic toggle.  I think one would just need to make one or two functor-intialized immutable bools into mutable refs that hold a bool.  (In particular, the “disable_names” flag).  Then the user can dynamically change these presets after the functor is applied.

Does that make sense?

-Matt

> On Jul 28, 2015, at 5:42 PM, labichn <notifications@github.com> wrote:
>
> I'd prefer not to have all client code parameterized by ArtLib. The vast majority of use cases (that aren't for Adapton-internal performance testing or debugging--read: our code) would use the default Nominal ArtLib. You mentioned switching to FromScratch as a way for the users to debug their code modulo names, and I agree this is something that client code should be able to do. I'd much rather expose some very high level toggle (Adapton.disable_names : unit -> unit, Adapton.enable_names : unit -> unit) so that the concept of ArtLib isn't one that client code is aware of at all. This would mean that the SpreadTree, Trie functors wouldn't be parameterized by ArtLib, so I don't know how we could do this cleanly in the following scenario:
>
> open Adapton
> module T = Trie.Set.Make(Types.Int)
> disable_names () ;
> T.nadd ... (* but MakeArt in Trie's impl already was applied with the Nominal ArtLib *)
> I'd be surprised if there weren't a way to do this, though.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/plum-umd/adapton.ocaml/commit/2860237eb37917779f152d7c1b84177dd0b7279e#commitcomment-12407351>.
>


`return [];` at least so it's comparable to other return values of this function.


This should be updated to use the `DISTCHECK` rule as well.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 217](https://teamcity.labkey.org/viewLog.html?buildId=263225&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


@Dwar3xwar DWMR. Good good. Press on.


It was actually a problem on the other site, which is the same google
property, that changed the code and this site stopped tracking, so I
changed it. Maybe one day we can go forward again, we are tracking some
events.

2015-07-28 19:03 GMT-03:00 M Somerville <notifications@github.com>:

> Just to note, this commit switches back to the *old* Google Analytics
> which they are getting rid of.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/YoQuieroSaber/yournextrepresentative/commit/c47cd2cb65db2d118d3ffb01ef380b7ee5bf27a1#commitcomment-12407667>
> .
>

--
*Martín Szyszlican*
Desarrollo web usable y accesible
martinszyszlican.com


Hey you did this wrong.


url and title should be strings, their default values should be "" not a boolean


Since this is covered in another issue, can you remove it from this pull request?


Although you can use a variable which represent an enum member, but why do you need such weird snippet here?
```java
enum T { A, B }
...
void fun(T t);
...
T t = T.A;
fun(t);
```
It is confusing. Once again, as you are wring a code snippet, please plovide good code examples to show user how to use API without confusing.


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 tests passed)

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


What do you get on rails console?
(I added ffi back on my box, otherwise running the console is not possible)
2.1.5 :003 >     ::RGeo::Geos.capi_supported?
 => false
2.1.5 :004 >     ::RGeo::Geos.ffi_supported?
 => true

On Tue, Jul 28, 2015 at 6:45 PM, Matt <notifications@github.com> wrote:

> Seems to be. You can add the FFI gem back if needed, it shouldn't affect
> us. The problem for us was solved by removing
> RGeo::Geos.preferred_native_interface
> = :ffi, see
>
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#diff-1e906ea092519204c4e1f5a2d1c5a549L3
>
>
> M
>
> On Tue, Jul 28, 2015 at 4:40 PM, Hernán Lucas Pereira <
> notifications@github.com> wrote:
>
> > Removing this gem is making Travis and my box fail to build. Is it
> working
> > fine on Mac?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407318
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407409>
> .
>


Correct, that's what I meant :-)


`if (!url)`


yes, I'm getting an error stating that mraa::I2c has no member names
'readBytesReg'

On Mon, Jul 27, 2015 at 1:58 PM, Mike H <notifications@github.com> wrote:

> I'm confused...what exactly are you asking here? Is the code failing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sparkfun/SparkFun_ADC_Block_for_Edison_CPP_Library/commit/2e985245d1d5daeae9b4ac737aa3c59644e831b5#commitcomment-12378773>
> .
>


@Dwar3xwar DWMR. Looks like the wiki was of some value. Good. Remember to get help before frustration enters the picture.


why clear the cache here?


Probably want to log the reason you're returning uncategorized here.


This was probably supposed to be in that last commit


Huh.  I've never seen an `sfInnerSig` before.  What's that?


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


careful not to let IO stuff creep back in to your logic classes


did this go in? I don't see any of these changes on the live site?


This will fix a bit of an edge case.

It appears that Stripe sends webhooks to one site at a time that are listed in your webhooks list.  If any of the sites fail it will not move on to the next.  Any response other than a HTTP 2xx message is considered a fail.  Thus this will stop that from happening.


matt@HammerTime taxonworks (master *$%)$ rails c
Loading development environment (Rails 4.2.3)
irb: warn: can't alias context from irb_context.
2.1.5 :001 > ::RGeo::Geos.capi_supported?
 => true
2.1.5 :002 >  ::RGeo::Geos.ffi_supported?
 => false

On Tue, Jul 28, 2015 at 5:06 PM, Hernán Lucas Pereira <
notifications@github.com> wrote:

> What do you get on rails console?
> (I added ffi back on my box, otherwise running the console is not possible)
> 2.1.5 :003 > ::RGeo::Geos.capi_supported?
> => false
> 2.1.5 :004 > ::RGeo::Geos.ffi_supported?
> => true
>
> On Tue, Jul 28, 2015 at 6:45 PM, Matt <notifications@github.com> wrote:
>
> > Seems to be. You can add the FFI gem back if needed, it shouldn't affect
> > us. The problem for us was solved by removing
> > RGeo::Geos.preferred_native_interface
> > = :ffi, see
> >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#diff-1e906ea092519204c4e1f5a2d1c5a549L3
> >
> >
> > M
> >
> > On Tue, Jul 28, 2015 at 4:40 PM, Hernán Lucas Pereira <
> > notifications@github.com> wrote:
> >
> > > Removing this gem is making Travis and my box fail to build. Is it
> > working
> > > fine on Mac?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407318
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407409
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407737>
> .
>


`SusPay` has a "condition" property which functions like an additional signature on the transaction (in the sense that it must be checked at the same time and in a similar fashion to the transaction's signature). This is changed in a subsequent commit in SusPay.


Change the hard-coded paths on lines 6 and 21. Make sure you have a .csv named "fss.csv".


Autocorrect* on my phone :p


Ah


Or for consistency, you can return `['uncategorized', 'invalid_url', 'nsi_error']` like you do below


This project needs a readme.


Any idea of how you ended up with CAPI support? I'm trying to see if we can
have the same dependencies on Ubuntu and Mac.

On Tue, Jul 28, 2015 at 7:10 PM, Matt <notifications@github.com> wrote:

> matt@HammerTime taxonworks (master *$%)$ rails c
> Loading development environment (Rails 4.2.3)
> irb: warn: can't alias context from irb_context.
> 2.1.5 :001 > ::RGeo::Geos.capi_supported?
> => true
> 2.1.5 :002 > ::RGeo::Geos.ffi_supported?
> => false
>
>
> On Tue, Jul 28, 2015 at 5:06 PM, Hernán Lucas Pereira <
> notifications@github.com> wrote:
>
> > What do you get on rails console?
> > (I added ffi back on my box, otherwise running the console is not
> possible)
> > 2.1.5 :003 > ::RGeo::Geos.capi_supported?
> > => false
> > 2.1.5 :004 > ::RGeo::Geos.ffi_supported?
> > => true
> >
> > On Tue, Jul 28, 2015 at 6:45 PM, Matt <notifications@github.com> wrote:
> >
> > > Seems to be. You can add the FFI gem back if needed, it shouldn't
> affect
> > > us. The problem for us was solved by removing
> > > RGeo::Geos.preferred_native_interface
> > > = :ffi, see
> > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#diff-1e906ea092519204c4e1f5a2d1c5a549L3
> > >
> > >
> > > M
> > >
> > > On Tue, Jul 28, 2015 at 4:40 PM, Hernán Lucas Pereira <
> > > notifications@github.com> wrote:
> > >
> > > > Removing this gem is making Travis and my box fail to build. Is it
> > > working
> > > > fine on Mac?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407318
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407409
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407737
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407804>
> .
>


This commit fixes the issue I was having at https://code.google.com/p/libtorrent/issues/detail?id=716. My comment in #44 can be ignored.


Why not merge these two if statements with a `&&`?


There are a couple things that I am going to fix after I merge, put here for documentation purposes:

- [ ] Fix `validate_num_clust`'s if statement
- [ ] Change its warning to 'message'
- [ ] Fix outputting that message for every column that contains 0 and 1 (just output it once) per data set
- [ ] We need to do the same scaling of the bar plot as what I added for the box plot


GroundDomainUpdate (sushobhit) - Win64-Windows-7-VisualStudio-12: OK (1773 of 1777 tests passed)

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


why not `await B.delay(0);`?


This is another test


@dsyer this is awesome!  I didn't know you could do this.  Is this a 1.3 feature?


Probably happened when I removed the NetworkMessage::addString() overload that took a const char*.


`let tmpResults = ...`


Thank you, it works :smile:

Launch is available as a package in [Homebrew](http://brew.sh/), so people can install it by typing `brew install launch`. It doesn’t use the `launch` binary and builds it from source instead.


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

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


`tmpResult.push('single_topic_subdomain');`


mispell canola


Thanks!


Ahh... Thanks for answer!


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

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


The host is the subdomain?


@LudwikJaniuk what do you think about this sound? http://soundbible.com/1598-Electronic-Chime.html


Double spacing after sentences is easy to read on *any* terminal settings. We have a [manual of style](https://github.com/CleverRaven/Cataclysm-DDA/blob/master/doc/MANUAL_OF_STYLE.md) that covers proper formatting for in-game text.


You previously used `parseURL().host` with the assumption that length is greater than 0. When would there be no host? Should this check be done every time you use `parseURL().host` ?


License delete ??


It should be either `numeric` or `integer`, shouldn't it? I know you had a test somewhere that only checked for one of them and it failed (I updated it at some point to look for both).


Isn't `!!` unsafe?


The size should grow exponentially, not just linearly. Linear would be too expensive. Please have a look at readlink_malloc(), how it doubles the buffer size on each iteration.


You leak the memory of v on failure.


hey @felipevolpone 0.5.1 is not on pypi yet. please use 0.5


Should this be a class method?


NewPartialFlatstoreRecord - it only works when field[6] is date.


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1770 of 1772 tests passed)

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


Since you are delegating?


I think It should be checked if file format is fixed and supporting partial record, like utils.KAM_FLATSTORE, utils.OSIPS_FLATSTORE


Too long. This page has lots: http://www.soundsnap.com/tags/blip

I especially would recommend "RETRO BLIP C3", here's a direct link, hope it works: http://www.soundsnap.com/retro_blip_c3


You're saying "There is not enough data instances" for *any* error thrown by NbClust. Is there not a consistent error message from NbClust when there is not enough data? If so, you should make sure the error message it throws matches that, and only in that case write ""There is not enough data instances". If it throws a different error message, just raise it.


Yuck. Here's hoping that Wayland makes this hack unnecessary.


The only thing I would change is to make these all free functions and add `operator==(optional_type, OptionalProxy<T>)` etc.


@bitwit yeah, this is rediculous


@matthewmueller it keeps giving me `done` is undefined. Would it be possible to update the example in order to avoid this?


@LudwikJaniuk love it. I am going to try to push an update either tonight or tommorrow with the caps lock bug fix and that new sound. Thanks again for helping out so much with everything.


This actually needs to be the mainScene instead of "this" so I need to revisit this.


@ksooo - Regarding the 'client defined' lists option. What do you recon: We add MaxRecordings and separately 3 other 'client defined' ones?

I know there was concern about the size of the string arrays. Has this gone away or do you think we should limit the new ones to a smaller range, say 128 settings rather than 512?
I can't see how Recording Group could need need more than 128 options either. A reduction here should offset some of the growth caused by the new lists.

It would help if it was possible for a client to add an 'unlisted previous value' string like we did for date - think that's possible?


Really not sure, maybe "brew install postgis", does it come with that
supported?

On Tue, Jul 28, 2015 at 5:13 PM, Hernán Lucas Pereira <
notifications@github.com> wrote:

> Any idea of how you ended up with CAPI support? I'm trying to see if we can
> have the same dependencies on Ubuntu and Mac.
>
>
> On Tue, Jul 28, 2015 at 7:10 PM, Matt <notifications@github.com> wrote:
>
> > matt@HammerTime taxonworks (master *$%)$ rails c
> > Loading development environment (Rails 4.2.3)
> > irb: warn: can't alias context from irb_context.
> > 2.1.5 :001 > ::RGeo::Geos.capi_supported?
> > => true
> > 2.1.5 :002 > ::RGeo::Geos.ffi_supported?
> > => false
> >
> >
> > On Tue, Jul 28, 2015 at 5:06 PM, Hernán Lucas Pereira <
> > notifications@github.com> wrote:
> >
> > > What do you get on rails console?
> > > (I added ffi back on my box, otherwise running the console is not
> > possible)
> > > 2.1.5 :003 > ::RGeo::Geos.capi_supported?
> > > => false
> > > 2.1.5 :004 > ::RGeo::Geos.ffi_supported?
> > > => true
> > >
> > > On Tue, Jul 28, 2015 at 6:45 PM, Matt <notifications@github.com>
> wrote:
> > >
> > > > Seems to be. You can add the FFI gem back if needed, it shouldn't
> > affect
> > > > us. The problem for us was solved by removing
> > > > RGeo::Geos.preferred_native_interface
> > > > = :ffi, see
> > > >
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#diff-1e906ea092519204c4e1f5a2d1c5a549L3
> > > >
> > > >
> > > > M
> > > >
> > > > On Tue, Jul 28, 2015 at 4:40 PM, Hernán Lucas Pereira <
> > > > notifications@github.com> wrote:
> > > >
> > > > > Removing this gem is making Travis and my box fail to build. Is it
> > > > working
> > > > > fine on Mac?
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407318
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407409
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407737
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407804
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407858>
> .
>


Geraint, have you got an alternative fix for this? This was causing errors on the RSVP Pro Sessions screen when on multilingual sites.


test github


we cant short url of unpublished content :)


okkk!!! hago un cherry pick y lo paso a master. vos cuando puedas tirate la nueva version!


we should keep

```
if not self.published or not get_setting_value('SHORTENER_ENABLED')
```


Add
```c
#if !defined(WIN32_LEAN_AND_MEAN)
#define WIN32_LEAN_AND_MEAN
#endif
```
before
```c
#include <windows.h>
```
This is how I solved it for M_DD and MSL Tables.


No problem, I've been trying to get into open source contributing for a while, but had a hard time getting into larger projects. Your project is perfect for me to get into the fork workflow, pull requests and all that. And it's great you're keeping it so active too.


The standard apparently forbids zero-sized arrays, so with `-Wpedeantic` on the original way will cause a warning.  This is on my list for this week to fix, since it is my material code originally.


investigations are needed; but the test fails


r=me for this commit; looks good (just a nice refactoring, nothing scary)


specs are not mentioned in the commit message (not a big deal i guess) I also find it more logical to put specs into separate commits (like methods you added and then specs for them) but that's my preference.
also not sure if it { should have_sent_email } is a valid spec


Wow guys, you're really fast. Thank for feedback, it's done (again).


We are doing it :+1:


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1773 of 1777 tests passed)

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


Nice catch!  maxvaluelen was counting the buffer size, while strlen() was counting char's, _excluting_ the final null char.  Your code fixes the bug in this boundary case.


This does not work as stated a monh ago :(


I should've probably used the Logging gem, and set a sane `root` level.


Beautiful commit message.


Add the 44 tehama address


@olabini, I was thinking about having an exported `Conversation` facade around our `conversation` to prevent making changes to our conversation with the only purpose of making it compatible with agl's `xmpp-client`.

What do you think / had in mind for #13 ?


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


It's weird to separate them (some in the classroom, some downstairs). What seems to have worked out before is that people just congregate in first floor area.


It's a nice catch to guard against `fd <= 0`!  However, I wonder if we should do a JASSERT in this case, instead of leaving it up to the caller of `Util::readLine()` to check the return value.  If `fd < 0`, then the programmer should have checked for this _before_ calling `Util::readLine()`, rather than looking at the return value, and trying to fix it after the fact.