Improper coding style here. I understand that uncrustify doesn't handle this correctly, it just has to be fixed manually.


Probably add a test in which the file has children.


Include must run as non-root


re: "{0}" - this is only for Python 3.0, 3.1+ supports "{}".  I imagine we're (eventually) going to support 3.3?

I'll revert to TypeError, that seems like good logic to follow.


What other plugins might have to be restricted?   What security risks are involved with pods being able to arbitrarily ask for other types?


Debug output can be replaced with `UAVCAN_TRACE()` (include `uavcan/debug.hpp`)


`num < 0`


Driver errors are kind of expected, so assertion should be removed.


test


@cristiandley did you check that this works for both xls and xlsx? what if someone wants to write the file in xls format for older excel versions?


hmm this sleep isn't blocking?


@Nigusse @mjwitte The red text at 11:40 UTC here: https://status.github.com/messages.  It caused all our bots (and probably many other things around the globe) to go crazy.  Anyway, we went ahead and cleaned up any stray old builds, ran system updates and reboots, and got them going again.  Results are coming in pretty rapidly since the restarts, so I'd assume these will start also.


Why `_handler`? :)


Making dev up to date


This is what I am doing now; but I thought as this API is exposed in previous version, it is good to have it here as well because the user case I have is not unique, Rails generator also would update configuration files for many commands it have.


because I forgot to rename


Is the off-by-one indent here intentional?


This is a new coding style, not seen elsewhere in the source code. I don't want to get into a big refactoring and huge diffs adopting this style. Such rewrites make it hard to compare different versions of the source code.


what do you mean? I actively only test the beginning of the AST because its so much unneeded data


@maniac103 I have the Git talent of "I figured out how to use the GitHub web interface, unfortunately. I can't cherry pick commits or rebase things or solve merge conflicts. (One of the reasons I try to make sure I fork things before the change I don't want is committed to master.) I have a slight loathing for how overcomplicated version control software is.

Also I actually made all of the changes manually reading from your commits rather than just applying the commit, as I wasn't sure what would and wouldn't work correctly. (Credit to you, refreshingly all of the API logic seems compatible between the master and material forks, material makeovers have a nasty tendency to be rampantly incompatible code-wise with the previous version.)

Arguably, putting it on GitHub at all is largely irrelevant, as I don't intend to publish or publicize the release. It's just for personal use. Once the license PR is addressed I'm going to revise the readme text on there and be a bit more clear that none of the code is mine, and people should look at the original for source authors or something similar.


typo


I mean the file currently being tested has only one level. I dont know if its necessary but would make the tescase look complete


Yes, I hope so xD


Transifex has not been abandoned, or did you mean something else?

Now I realized the problem: transifex still had the old (source) strings, so you couldn't actually translate them. I tried to configure transifex such that it would always use the original strings directly from github without me needing to push them manually.

But apparently that does not work. So for now, I pushed the current source strings to transifex, so that you can translate them.

And I thought the whole thing would be easy :-)


"**the** file pointer" sounds somehow better, IMHO.


ss


ah I took the next lie away because thats a very long one. IMO this suffices for a debug utility...


Changed in commit https://github.com/drf5n/drf5n-public/commit/91facedb709d5727ca622edc3c538c78d230b73d


cool


I'll wait for name change


The order of cleaning up should be:
1 - curl_multi_remove_handle before any easy handles are cleaned up
2 - curl_easy_cleanup can now be called independently since the easy handle is no longer connected to the multi handle
3 - curl_multi_cleanup should be called when all easy handles are removed

source: http://curl.haxx.se/libcurl/c/curl_multi_cleanup.html

Tested on iOS, Android and Windows.


@Areidz That's what matters. BTW it's nice to know about you again, Rigle :)


Tried the fix and got an exception:

org.objectstyle.wolips.eomodeler.core.model.EOModelException: There was more than one SQL generator factory defined.
	at org.objectstyle.wolips.eomodeler.core.sql.IEOSQLGeneratorFactory$Utility.sqlGeneratorFactory(IEOSQLGeneratorFactory.java:39)
	at org.objectstyle.wolips.eomodeler.actions.GenerateSQLDialog.generateSql(GenerateSQLDialog.java:297)
	at org.objectstyle.wolips.eomodeler.actions.GenerateSQLDialog$1.run(GenerateSQLDialog.java:279)
	at java.lang.Thread.run(Thread.java:745)


I added the next line for you, split it up.


> I have also thought about the EXPECT_EQ(golden_startup_values, dumpMixxxCos()) test, but I think probably not significant enough compared to the work. On addition it is a maintenance trap since the test will fail on every CO change.

I disagree because such a simple test would not have caught the widgetstack bug.  We should write this test, as well as have tests for ControlObject setup.  Yes, we should have to update the test every time we add a new control object that exists at startup.  That way we know that startup is working correctly.

As I said before, more work on this topic should go in master, not 1.12


OK, I tried it again, this time by using the URL to the "raw" content of strings.xml:
https://raw.githubusercontent.com/Freeyourgadget/Gadgetbridge/master/app/src/main/res/values/strings.xml


@fritsch: On startup when it can't find the language that is configured it tries to find an addon (from all the available ones i.e. installed and installable from the repository) whose name matches the value of the setting. If it finds one it downloads and installs it. If that fails or if it doesn't find one it falls back to English. This has been tested by myself, @MartijnKaijser and some other people. The only real issue is for people without an internet connection.


I don't see that in the actual code, so I'm not sure what that is about.

________________________________
From: Michael Heuer (NMDP) [notifications@github.com]
Sent: Thursday, May 07, 2015 2:28 PM
To: mhalagan-nmdp/pipeline
Subject: Re: [pipeline] changes. (2355e38)

this is a commit of conflict flags?

—
Reply to this email directly or view it on GitHub<https://github.com/mhalagan-nmdp/pipeline/commit/2355e381b9f51c88431cb0806d9ce23d3cd97ff3#commitcomment-11094710>.


I didn't change that code, so I'm not sure why it's acting like I did.

The process_fastq.bash in the update-report branch is the same as the process_fastq.bash in the nmdp-bioinformatics/pipeline/master branch.

________________________________
From: Michael Heuer (NMDP) [notifications@github.com]
Sent: Thursday, May 07, 2015 2:29 PM
To: mhalagan-nmdp/pipeline
Subject: Re: [pipeline] changes. (2355e38)

this file shouldn't have been touched by changes to the validation code

—
Reply to this email directly or view it on GitHub<https://github.com/mhalagan-nmdp/pipeline/commit/2355e381b9f51c88431cb0806d9ce23d3cd97ff3#commitcomment-11094714>.


Good catch, I'm fine with the code as well. I would like to read over the paper one more time, but will not have a chance till after data mining final. After that I think we will be done.


OK ... good idea.


Hmmm, sorry, I am too deranged, caught a bad cold. I will try and revisit this later.


You can read HTTP headers so much better than I do! Thanks :)


You've forgotten the 'packages_to_branch_out' in the versions array. See line 587


getNextPage returns a non-empty page or a null page if the page source is finished. So this doesn't need to be a loop.


Can you change that to "Blizzard Entertainment (contact tvandijck@blizzard.com)"
I don't want to take credit for things I didn't do, but just merged into our internal repository...


I've uploaded the vast majority of our final submission to the drive.

Remaining things to do are add the latex source and pdf once it's
finalized.

Also the data for two of the parameter tuning tests is missing. If you have
it throw them in that folder preferably with the image and mathematica
notebook files as well as I have done for the others.

Other than that I believe we are done.
On May 7, 2015 3:48 PM, "Akiira" <notifications@github.com> wrote:

> Good catch, I'm fine with the code as well. I would like to read over the
> paper one more time, but will not have a chance till after data mining
> final. After that I think we will be done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Akiira/Comp-524-program/commit/9dc311d3d380c8f62112e30ebb2b2cf452054e66#commitcomment-11095036>
> .
>


![rubocop jordane](http://staff.osuosl.org/~voigte/jordane-rubocop.jpg)


@Montellese: Oki, thx very much for verifying.

2015-05-07 21:48 GMT+02:00 Sascha Montellese <notifications@github.com>:

> @fritsch <https://github.com/fritsch>: On startup when it can't find the
> language that is configured it tries to find an addon (from all the
> available ones i.e. installed and installable from the repository) whose
> name matches the value of the setting. If it finds one it downloads and
> installs it. If that fails or if it doesn't find one it falls back to
> English. This has been tested by myself, @MartijnKaijser
> <https://github.com/MartijnKaijser> and some other people. The only real
> issue is for people without an internet connection.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/xbmc/commit/ad8e5cad5c16d25fddd4452a4dc2afc510e61739#commitcomment-11095024>
> .
>

--
                   Key-ID:     0x1A995A9B
                   keyserver: pgp.mit.edu
==============================================================
Fingerprint: 4606 DA19 EC2E 9A0B 0157  C81B DA07 CF63 1A99 5A9B


seems like the enumerated list of allowed volume plugins would be more flexible, but I don't know if that's compatible with how volumeplugins are implemented


`&` should be escaped?


Sad panda. Lost my user name on this commit


Yep it works for xls and xlsx, js-xlsx is a merge of both.

Mmmm the second question i dont know how to anwser.


escape `&`


ack


I can't usefully review the details of making a gem, but I am curious why the development dependencies are in street_address.gemspec and not in Gemfile.


why is there so complex code. Why wouldn't we just find first branch in list on main page and go there?


or `buf.writeAll(string, ',');`


Paylaşım için teşekkürler


We should have a test for the widget stack.
A stack that catches the widget stack bug with a golden CO sample is very hard to write, because it is skin depending and hardware depending.
And yes we have some annoying issues, right now. But once we have fixed the bugs there is a very low risk for a regression. In this case its better to have focused test that guarantees a specific behavior, instead a brute forth test that fails any time we touch the code.


LOL.  Tower autocorrected `neume.` to `gnome.`.


On Vero, this is osmc@vero:/sys/devices/soc0/soc.1/20e0000.hdmi_video


rax retry tests


You know what, forget about this, I've just noticed `activesupport` is already included, I totally missed the first time, But then we can create a PR out of this without making changes ;)


Ah, cool!


Should stripping be done as part of line1 and line2?


I'll bet this logic can be simplified to something like

    [line1, line2].select{ |l| !l.empty? }.join(', ")


Thanks for the explanation, it's a bit more clear now.

> there's no way to tell what type of field a custom field is

Not sure why the type of the field would help in this situation. See my next question where I try to guess the answer.

> regardless of the order of the values submitted, JIRA is either always appending the new value, or it's auto-sorting the values

Do you refer to  the fact that setting the custom field type would allow predictable sorting?
Even in case of predictable sorting, there will be no warranty that a new value is inserted at the beginning or the end, unless we follow a convention for the new values (alphabetical/numerical sorting). This would mean we need to distort a bit the desired values by prefixing them with e.g. additional numbers.

How do you currently guarantee `APPEND`? From what I understand you managed to get back either a sorted array or an array with the new element appended.  How did you manage to obtain these 2 different behaviors?
Not sure if Jira stores the insertion order at all (I remember I saw unexpectedly sorted labels on the Jira web UI as well), but I can imagine that when querying the values it sorts them and I am not sure if this can be prevented. Wouldn't this mean that we can only guarantee `ADD` (so no promises on the order of values)?


This change looks like "remove remember me box". It appears to have been deleted not hidden.


Had to comment out all the iptables work that I put in.

This is the 0.0.1 release and will install strongswan and allow you to do some certificate generation.

The problem is, VPN configurations are often complex, it makes it very hard to 'charm' a VPN.


Ben,

Not sure I handled broadcast_not_unicast correctly of for that matter any of this does what you expected.


@rynkwn Shouldn't it be this:
```ruby
to_buy = min(in_stock, max_purchasable - tickets_owned)
```


fixes #2


@BambooIV 从mirom那里的patch改过来的，修复cpu1不能设置频率和调节器，这个可以用。。


markdown语法写成这屌样。。


new special note


a curious dilemma: i would have more time to support embedding for real, if i weren't spending so much time creating things that need embedding


我来了


we could expend the test purpose to track onPageLoadStopped, onPageLoadStarted, onProgressChanged called times.


Totally just figured out another source of my branch coordination issues. The code from the branch you are on when you create a new branch will be copied to the new branch. I knew this, but for some reason it was not in the forefront of my mind when creating new branches. There had to be some instances where I created a new branch for a lesson while not being on the proper base branch.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Tuesday, May 05, 2015 8:31 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] add author-post association, updated seeds, modify posts/index view, add scope to Post model (99bc441)

This means you don't have to wait for me to review the checkpoint before you merge your lesson branch into master. You can if I manage to review the checkpoint fast. The important thing is that before you start working on your next lesson, all the changes from the previous one should be in master already.

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/99bc4413300772434f33ec4a7e9109ab4819e1f0#commitcomment-11043441> .  <https://github.com/notifications/beacon/AKgq0PztU2qsCtaQvf2PyhitSsNMJvxnks5oGK91gaJpZM4EQOMH.gif>


## This is a markdown file


using /etc/ceph/ceph.conf as default.


This doesn't look correct to me.


I can not change the value of struct in the list.Value.
I think it just is the temporary solution to fix the issue.  You will rewirte the LRU cache to provide the better performance.


helloworld


System.out.println("fly with me!);


LastVersion


No comment on this commit!


When launching KSP, KeepFit is complaining that GameData/Timmers is no longer the correct place for KeepFit to be installed.


Spacing


Ditto


:snake:


> The idea here is that is the double-tap point is within 40px around the user location dot when in tracking mode, to not change the center of the map and thus not alter tracking mode.

OK, but in practice, the map would soon (within a fraction of a second) recenter on the user dot due to the next location update.

> Now, unless the user double-taps on the exact pixel occupied by the user dot, user tracking mode will be lost.

Actually, the comparison is between the user dot location and the intended zoom-in position. It’s testing whether we’ve set [`zoomInPoint = userPoint`](https://github.com/mapbox/mapbox-gl-native/commit/5f21bbb15f35cac2eb7fa3c1c540ec30e8900de3#diff-f315cd6ead3270a0818a2dcfb84bd99bR1089), which will still happen anywhere within the 40-pixel margin.


Thank you! I will fix this in few hours


Thanks  a  lot  :  )


Yes, I will keep CREATE VERTEX and CREATE EDGE.
I woould like to use CREATE GRAPH primaryli in constructions like:
```sql
    CREATE GRAPH FROM GEXF("mygraph.gefx");
```


오타
캐스 -> 캐시


@sawasa1 Thanks


통일할 생각이죠?
저장소 클래스 -> Repository 클래스


Nice


The browser default behavior would move the cursor outside the text editor (e.g. going to the URL for safari). I guess this should be prevented??


Shouldn't this be moved to the write alphabetical place?


Just copy the value out and put it back in after it has been modified.

Alternatively, do the TODO. This just involved pulling out the relevant code from container/list. Then you can specialise the element and we get the improved performance.


Which version of Yosemite are you running, and can you provide your hardware profile as well? I'm working on making the repository more comprehensive to support the many combinations of Yosemite versions and hardware specifications. The more details the better I'll be equipped to fine tune the post-installer.


So apparently you finally saw it too!
![image](https://cloud.githubusercontent.com/assets/129756/7507599/1a48fa78-f440-11e4-8092-96c0b1a85661.png)


Me parece sensacional esto :+1:
le agrége una linea   <link rel="stylesheet" href="css/styler.css" type="text/css" media="all" />
dentro del encabezado html para que no pasara algo que me sucedia la cargarlo,


Haven't found a way to set / alter the value of GSR in a unit instance, seems like GPG didn't provide such a function.

At the moment this only affects SCU / ACU which isn't patrolling that often. There is a GuardRadius bp-value, maybe that could be used in conjuction with GuardScanRadius to get the desired behavior.


赞


In the story, Shannon add lines as following:
     copy api.html to api-<previous version>.html
     update api.html#versions: version links for current and past.
I will leave a note to confirm with Shannon.


can you push the pin image to master?


Hi drmyersii,

I have the same problem like you had, only that I am using a different vagrant box, namely the 3rd most popular box "homestead". (https://atlas.hashicorp.com/boxes/search?utm_source=vagrantcloud.com&vagrantcloud=1)

Since I just started using vagrant, I wonder I you have an easy solution for homestead?

I also opened a stackoverflow question here, because homestead on github does not allow the openning of issues:

http://stackoverflow.com/questions/30090691/configuring-homestead-to-work-for-windows-users

thanks.


sure, do we have CC, CLI and CF release version now? and still waiting for Shannon's reply.


hmmm idk if its worth it


really?


fthf


gek weruh,, web e apik e design e meh ta conto ah


The change log is update as following:

Update Service Binding section: GUID of the application that you want to bind your service to, will be included when users bind applications to service instances, and will not be included when users create service keys.


修改user的数据类型 更新相关操作


That was all @marshallnorman :smile:


I'd like to use "access", will update.


:-D


Yes, both are endpoints to show the detail pages.


Ok! Rsrsrs...


Maintainer is pumping out nightlies... says they work


I/Adreno  (  122): ioctl code 0x40140931 (IOCTL_KGSL_TIMESTAMP_EVENT) failed: errno 22 Invalid argument

How to fix it ?


@jedberg r?


We'll need to find a proper solution later anyway to be able to make it available on the demo (so that we can link to it in the developer documentation, but not on every Piwik install.


Yeah - same for i4. It's a typo. It will fallback to 1.0 for pixel ratio - so won't harm for now.

ping @mk01


title and proper encoding


Wow that's late at night...


Hehel no worries :)


The problem is that `c` now contains a mixture of items that can be safely changed on-the-fly, and items that require a server restart. And we don't really know which are which. My approach was to do the `c` refactor item-by-item so tests could be run and we would be able to more easily find regressions. Also, for efficiency, the Redis config will actually live in several different dicts so we can only call a refresh on the ones that are relevant to a current action; this also simplifies configuration changes. There is no need to use `__getattribute__` because everything going into the config variables already lives in a dict - that dict just happens to do some voodoo to stay in sync with the version in Redis. The existing config loading will keep loading the existing globals (now `c`) until we have moved every place in the code referring to `c` to instead refer to the appropriate "new" dict.

It's not the end of the world, but the merge is now a huge pain and I'm probably going to find it takes less time to start over. I can copy/paste some parts of my parsing and loading code, but the part that backfills to the old stuff is pretty much a goner, as is the places I started testing changing items to point to the new dict. Plus I hate to have seen you do a bunch of work that's just going to have to be redone - we could have avoided this if I'd known this was being worked on before it was done, and we could have had this conversation then.


Heh, yeah, the learning curve is steep with git. One could use this as a learning exercise though, it's worth it :)


Obviously you have to revisit this code to use the appropriate email address. How are you keeping track of such TODO issues. I might expect to see some sort of code comment here at least:
```
# TODO: change hard-coded email address to use the real sponsor's email
```


Do these wood recipes actually work? Last time we did them in this script, they didn't load (despite everything else in the script loading)


@laszlomiklosik `ADD` is likely a much better guarantee, because what you've seen supports what I saw (but didn't investigate further), which is the inherent re-sorting of values in JIRA. I'll update the the label and help text to reflect this.


I had already fixed this one in your PR.  Sorry I didn't mention it in my comments.


Changing it to build using an array is a great technique.

I would like to see `to_s` tested as thoroughly as `line1` and `line2`.  Perhaps assume `to_s` is `line1 + ', ' + line2` unless explicitly specified?

Mod that, and the inline comments, it looks fine.


Do we really want these to be craftable with only palm planks? What happens if all their tools break while in the Erebus? These should at least be Ore Dictionary so all types of wood can be used.


Ah yes, I sometimes forget about empty() being so limited before PHP 5.5... Just committed a fix. Thanks for the head's up!


This is breaking the toy. The drawwaypoint no longer prints waypoints right of middle screen.


oops!


* Replaced some magic numbers in grid zoom with values based on cell dimension variables. Still needs to compute grid spacing based on arbitrary zoom or else put in limits on zoom.


Isn't this inverting the logic so that now this exception is raised when `pull-server-side-config=true` ?


@laszlomiklosik re: type, the type of a field would tell us if JIRA is accepting only 1 value in the submitted array of values (standard text field), or if it accepts *all* of the values (labels field). My logic is specifically built to deal with the multi-answer (labels). For text field, I believe I tested, and JIRA accepted only the first entry in the array each time.

So for anyone to get any append/prepend type of functionality (to keep expanding their text field value), that's where we'll need to implement variable expansion. That's outside the scope of what I need right now (and I don't have a scenario where I'd truly need that ability yet), so I'll wait until it's desired before looking at implementing :-)


I'd love that.


no stress


Ah! My apologies. I was using a strange fork of lemonbar that was broken. All is well now. Thanks


What does this imply about when it is ok to change the contents of the buffer that was input to  the AsyncMethod? Is it immediately after the call to AsyncMethod or only after the call to Finish ?


thanks @snak3ater :)


Why de we need two separate methods Topics.goToBranchPage()  and Branches.gotoBranch(String branchTitle). Why won't use only one Branches.gotoBranch(String branchTitle)?


Oh sorry I missed that!


I am bored with excuses why we shouldn't write tests.  I am also tired of fixing regressions.  You made the same arguments about the enginebuffer endtoend tests, and I wrote them in a weekend and they have been useful.  In the case of the rubberband and soundtouch tests, we had to disable them because they were dependent on the library version, but that was also good information to learn.

We need both types of tests: unit tests for specific behavior and end-to-end regression tests that test the whole stack. We have had a specific problem with first-run behavior degrading, so it's time we wrote code to test the first-run use-case specifically.


I synchronized translations between transifex and github, so everything should be fine now. Manual pushes of the source files should not be necessary anymore either.

German translation is up at 100% again.


@laszlomiklosik I think I did that to be defensive, and to also avoid having to do a null check later. Not sure why I set size to 0, believe I was thinking of ArrayList that initializes with size of 10 by default.


Hi Sheng,
  I noticed that have fixed it on my local version but have not uploaded it
on github yet.
Will do it soon.

Sandeep.

On Thu, May 7, 2015 at 12:37 AM, Sheng Guo <notifications@github.com> wrote:

> It seems that all processors will have the same m_load_prefix, because
> m_load_prefix is broadcasted when dmrginp is broadcasted.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sanshar/Block/commit/b8d27b69e97750cadcf773615a5a772aad7d4ddf#commitcomment-11076574>
> .
>


byebug är en bra gem som alla borde läsa på om, hjälper vid debugging!


This is a good change, but the commit message "VS2015 needs noexcept(false) on ~once_temp_type" puzzled me quite a bit. It suggests that noexcept was not present at all and this commit adds it.


This code seems repetitive. Can you make it shorter? Is there a possible loop?


Thx I opened a PR to fix that.


I believe I did this as grass uses a custom renderer, so I just directly copied everything we do from some other custom-rendering block. The inability to place beds, torches and for monsters to spawn is definitely not intentional, so it can be removed if necessary.


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.5%25-red.svg)


@nfeske, @chelmuth: please add to staging


Moving those preprocessor checks to soci-platform.h looks good to me, but I'm curious why renaming `SOCI_HAVE_CPP11` to `SOCI_HAVE_LAMBDA`? Do you expect more `SOCI_HAVE_<C++ FEATURE>` defines?


I think they are copied from a git diff and the preceding `-`'s have to be removed.


This looks good to me.


Agreed on removing that from docstring and replacing with a comment.

As far as prematurely optimizing the code/this being alarming, I kind of agree that it seems slightly scary to me? But, it does seem like a perf win, and this seems kind of reasonable? I was surprised when I got an error doing some stuff that wasn't the schema error I expected, and this seems like something Claypoole (a library I went to a talk about at Clojure/West; https://libraries.io/clojars/com.climate%2Fclaypoole) also does.


> I am bored with excuses why we shouldn't write tests.

Thats not the point here, it is the point HOW. And excuse me, I have never argued against the engine buffer test.


saw approval from christophermancini
at https://github.com/basho/riak-ruby-client/commit/72ae9f89d0a13b51088bbba7d7ce5ff7f0793ee0


merging basho/riak-ruby-client/bk-j338-synchronous-yz-index = 72ae9f89 into borshop-integration-219-bk-j338-synchronous-yz-index


basho/riak-ruby-client/bk-j338-synchronous-yz-index = 72ae9f89 merged ok, testing candidate = e6c8d9fb


LGTM


:shipit:


LGTM,


RRRREEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEE


Nope. They won't be erased even if they should, but when the player moves, those are then refreshed and removed.

I have to check that the there is a refresh between moving and looking, though.


Made Db property protected. See:
https://msdn.microsoft.com/en-us/library/ba0a1yw2.aspx


Thanks, @djarecka. I agree that some of the learning objectives need to be rephrased. There are a few things to consider though:

- learning objectives should be verbs in infinitive. The idea is that they complete a sentence like "After this topic, learners will be able to ...". So we use "Restore old versions of files" instead of "Restoring old version of files".
- you don't need to modify the html files, maintainers do that. This allows to keep the html consistent and free of conflicts (we use some trickery to get pretty pages.) To get rid of the html files in your commit, you can do this (courtesy of @wking):

```
$ git checkout objectives  # get on your feature branch
$ git checkout HEAD^ -- *.html  # clobber with the parent's HTML
$ git commit -a --amend -v  # squash the old HTML into your commit
$ git push --force origin objectives
```

where `origin` is the alias of your remote repo.

After doing that, you'll need to rebase your branch, because we have merged some stuff in the meantime. To do this you have to:

```
$ git checkout gh-pages # get on your gh-pages branch
$ git pull upstream gh-pages # get the last version from the SWC repo
$ git checkout objectives  # get on your feature branch
$ git rebase gh-pages # put your changes on top of the lastest version from the SWC repo
$ git push --force origin objectives
````
You may have to fix a couple of conflicts, and I'm assuming that you aliased the SWC repo as `upstream`. (You can check the aliases for remotes doing `git remote -v`.)

After all this, you would have updated your `objectives` branch (and therefore your PR) so it's free of html and good to merge.

If you think this is too much of a mess, you could also make a new PR starting from scratch and I'll drop this one. But it could be good practice if you don't know how to do it.


Fuck you


FH's git only accepts OCG rules. This modification is for the TCG and will be rejected.


Currently, just decomposition is implemented. Putrefaction is a different process (that leads to decomposition). Yes it would be nicer, but it would also mean a bigger commit.


You mean I can just ignore transifex likd before ;) ?


I don't like the dump and restore method used here. This causes a downtime on the wp_users table. Sure it's small, but it's still there.

Let's do wp user create and wp user delete instead.


This would introduce at least one new enumerated type. Not sure I want that. These values would also be hardcoded and compiled.

Maybe.


does anyone use this anymore? if not can we delete it?


Fixed in 2f975516cf38ec6eee9200e304ea6af04fee6c21


Not you, but we should move these animation constants into AppConstants and try to be a little consistent with them in the app.


In this URL you can find que different descriptors Tags allocated by MPEG for  MPEG, ATSC, ARIB, DVB and SCTE standards:

http://www.etherguidesystems.com/help/sdos/mpeg/semantics/mpeg-2/descriptor__loop.aspx


alphabetical ordering off here


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.88](http://teamcity.monogame.net/viewLog.html?buildId=7045&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed: 2 Build time: 00:00:00


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)


Closes #951


Is there a reason to have this block outside of the UIView.animateWithDuration call? I find it can make things harder to read when I have to jump back and forth.


Yes, that should work equally well. :)


Heh. Since reader is attached to the urlbar, this actually moves at 2*dy, eh? I think I'd put a comment about that in here, since it isn't immediately apparent.


all tests pass:
success: http://buildbot.bos1/builders/ruby-client-test/builds/330


Yes it did install OK, I haven't trialled it but looks fine.

Thank you for creating it.

On 8 May 2015 at 02:59, Ayoub ELHAMDANI <notifications@github.com> wrote:

> Hello,
> Did you test this module on Amazon Linux Server?
> thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apitalent/puppet-openswan/commit/89bf751a74a82757c4e3c05d5f92dcb2496c1d45#commitcomment-11089073>
> .
>

--
Lindsay Parker
Cloud Infrastructure Architect
Mobile: 021 5995 40

---
API Talent, 150 Featherston Street, Wellington
Telephone  04 979 9631 :: Email hello@apitalent.co.nz
www.apitalent.co.nz


Modified sentence  as jalle19 commented it was a odd sentence, this is my attempt to improve that. Suggestions welcome.


Modified sentence as jalle19 commented it was a odd sentence, this is my attempt to improve that. Suggestions welcome.


We'd like the database to reject a record that cross refs an item with a bid on another item. Can that be accomplished with this design?


SO f**king awesome!!!!!!! LOL~


Nit; Maybe its github, but curly-brace on the same line as the function.


Adressed comments for this iteration.


Is this available in Nuget?


в смысле, без добавления в конец списка (x ++ [1])?
а могут ли списки быть не отсортированы в лексикографическом порядке внутри главного списка?


i had to update the symlink to `../../../mysql/migrations.sql` in order to make it run properly


Never found one that works like this, it will return the return and the cancel. So you can `require(['requestAnimatinFrame'], function(raf){  raf.on(function(){ /*somecode*/ }); })` I use RAF on every project.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.89](http://teamcity.monogame.net/viewLog.html?buildId=7049&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed: 2 Build time: 00:00:00


Is it necessary to set @topic and @post?  Aren't those set in the controller action?


Here too, you should set this in the controller action.  But, even if it's here, the embedded ruby shouldn't be left open.


This looks like a good start! Here's a quick class outline that I had written down while we were discussing the design of this class that you can refer to if you're not sure how to continue:

- "trivia question" class -- represents a single question
  - Has a publically-gettable string for the question text
  - Has a publically-gettable set of the available answers
  - Has some knowledge of the correct answer
- "trivia set" class -- represents the current set of trivia questions that is being asked
  - Keeps track of an array or list of current questions in the set
  - Has a publically-gettable question object for the current question
  - Has a method to submit a user's answer
    - Compares the given answer to the current question's correct answer
    - If it's correct increments a counter of the number of correct questions in the set
    - Sets the "current question" variable to the next one in the set (or "null" if it was the last question)
  - "trivia" class -- supplies basic tracking of overall trivia
    - Keeps a bank of available questions (that haven't been put into a set yet)
    - Has a method that will return a trivia set with the supplied number of questions

I hope that my notes help -- if they aren't clear and you aren't sure how to go about implementing something, feel free to ask for input. I think one thing that might be helpful for you is to just write the empty class skeletons with empty method declarations so that you know what code you need to fill in later on.


What's the reason for using ```any?```? This method actually checks the first item to see if it's ```nil``` or ```false```. Is that meaningful? All we really want to know is that there are some. How about ```empty?```.


inlining and flat are orthogonal. It makes sense to keep them separate. You can already do what you want using `--inline` flag: `sea clp --inline --step=fsmall in.c -o out.c`


Woow! It worked not sure how I did this.
Super excited! Go Git!


Errm, but our documents don't use explicit crossrefs. We *want* the implicit crossrefs. Agree the warnings are noise, but this is not the right fix.


Hmm but this had more to do with poor SVG quality no? Invalid SVG's and stuff


Looking good!


Please stop that. It is unfair and leads to nothing.

> Testing this is reasonable, I think we can it make petty reliable if we use a square wave test signal that matches the buffer size.

> Once we have decided how it should be, we are able to write a test that verifies the result and keeps the behavior for later changes.


In the interest of making it easy to use the ReasonFlags object I'm not sure that this is a good approach. Once you actually want to decode a reason from the actual CRL this class can't really be used because a revocation can only have exactly one reason. So you would need to have another class capturing the revocation reason which doesn't have any relationship with this ReasonFlags class.

I wonder if it might be better to have a class where each instance represents exactly one reason and then the ReasonFlags class is just a collector of multiple of those.


Fair enough.


83811490-Fault_DirtyAirFilter (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1138 of 1138 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.78%25-green.svg)


Rather than print the calibration, it should return it. Anything returned from the REPL will be printed, but returning the value makes it useful programmatically.


Since the reader mode toolbar is moving twice as fast, I wonder if this is correct? i.e. would a fixed position header stick to the reader mode toolbar as it moves? I don't think we have that anyway, so it probably doesn't matter (but the fact that you took the readermode bar into account for the maxInset made me wonder).


so we need to manually track the JS files at two places?


Maybe ``published`` instead of ``awarded`` here?


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


Okay, done!


You removed the ``extra`` stuff from the template here, but you still pull it out of the ``msg['msg']`` a few lines up.  Is this a typo?


See also https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=784694


Yeah, but dependencies shouldn't be mirrored through a class. Just require them when you need them IMO.


fast-forwarding master to borshop-integration-219-bk-j338-synchronous-yz-index = e6c8d9fb


Awesome, thanks guys!


Oops, disabled in 6b34433bb0e23e0b8880c1d7aaf271af0e31f38c again. Sorry for that :-(


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

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


Wow noce, i would put in ccw.trace :smile:


There's a couple of odd things about this method (introduced before this commit):

1. The naming is a little confusing. It says `storeLog` which sounds like it takes an entire log and stores it somewhere, but it actually takes a `logMessage` and prints it somewhere. Maybe something like `log(String message)` or `storeLogMessage(String message)` makes more sense? The `.println` bit however will still be confusing if you keep the word `store`.
2. What if you want to log somewhere else, such as a file in the system?

Now for the actual changes in this commit... again, `storeLog` sounds like something different, but even if it was named `log(message)` it seems like you're also truncating the string before "storing". And there's some confusing logic there that makes me think this is a different thing that should have its own name. Why are strings up to 5000 chars safe, but 5001 gets truncated to 200? there's some important piece of functionality hidden there. From the commit message I take you're somehow passing images as string to your logs, and it's messing up the output. So your method to detect images is by checking if the message is longer than 5000... that right there is the unnamed piece of logic that should be more evident. Just consider that checking on length seems a bit arbitrary and could have unexpected consequences (what happens when you do want to log very, very long messages?). Why not prevent image logging in the first place?


There is one other place in the cvmfs_server where repository name is not known and in this case "-L" is always given - just in case the redirects are needed. In order to be consistent, I guess that redirect flag ("-L") should be removed or it should be added here also?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.90](http://teamcity.monogame.net/viewLog.html?buildId=7053&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:24


Hello,
im unable to submit "Unit Tests” through the roadmaps Github repo.
would this be do to my completeing that assigment previously? the completed
test is in my github account.
Michael

On Thu, May 7, 2015 at 3:29 PM, dothework <notifications@github.com> wrote:

> @mikeloz2015 <https://github.com/mikeloz2015> done with my review.
> checkpoint complete. press on!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikeloz2015/ios-exercises/commit/fdf464265fc1699ce1b3c5f351925154c120ee29#commitcomment-11094716>
> .
>


is that correct?


Instead of guessing, why don't you declare an int, set it to 200, 100, whatever, and tune it?


It's too bad it "fixed" the bug.

The problem is that during an install a subvolume won't be created unless its volume is already mounted. But mountsCache won't believe that the parent volume is mounted unless it is given the proper subvolspec. When doing a BTRFS install, it appears that the btrfs format for the parent is given no subvolspec so failure during subvolume creation occurs. Essentially, I think subvolspec is assigned correctly during discovery, but not during creation.


@rfay Just to be clear this isn't form altering this is actually removing the 'block' choices throughout the site, including VBO etc. Is this expected behavious?? Or do you just want them removed from the cancellation form?


@danielegobbetti
The FCs should be fixed.


Hi. So long as the commit was pushed to your repo we are good to go. The issue, I suspect, is simply that you'd already submitted the checkpoint. That will be a non issue shortly. ☺
Press on!

Sent from Outlook for iOS

On Thu, May 7, 2015 at 1:39 PM -0700, "mikeloz2015" <notifications@github.com> wrote:

Hello,

im unable to submit "Unit Tests” through the roadmaps Github repo.

would this be do to my completeing that assigment previously? the completed

test is in my github account.

Michael

On Thu, May 7, 2015 at 3:29 PM, dothework <notifications@github.com> wrote:

> @mikeloz2015 <https://github.com/mikeloz2015> done with my review.

> checkpoint complete. press on!

>

> —

> Reply to this email directly or view it on GitHub

> <https://github.com/mikeloz2015/ios-exercises/commit/fdf464265fc1699ce1b3c5f351925154c120ee29#commitcomment-11094716>

> .

>

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


:sparkles: Thanks @kevinsawicki .


excellent.
Cheers!

On Thu, May 7, 2015 at 4:42 PM, dothework <notifications@github.com> wrote:

> Hi. So long as the commit was pushed to your repo we are good to go. The
> issue, I suspect, is simply that you'd already submitted the checkpoint.
> That will be a non issue shortly. [image: ☺]
> Press on!
>
> Sent from Outlook for iOS
>
>
>
>
> On Thu, May 7, 2015 at 1:39 PM -0700, "mikeloz2015" <
> notifications@github.com> wrote:
>
>
>
>
>
>
>
>
>
>
> Hello,
>
> im unable to submit "Unit Tests” through the roadmaps Github repo.
>
> would this be do to my completeing that assigment previously? the completed
>
> test is in my github account.
>
> Michael
>
>
>
> On Thu, May 7, 2015 at 3:29 PM, dothework <notifications@github.com>
> wrote:
>
>
>
> > @mikeloz2015 <https://github.com/mikeloz2015> done with my review.
>
> > checkpoint complete. press on!
>
> >
>
> > —
>
> > Reply to this email directly or view it on GitHub
>
> > <
> https://github.com/mikeloz2015/ios-exercises/commit/fdf464265fc1699ce1b3c5f351925154c120ee29#commitcomment-11094716
> >
>
>
> > .
>
> >
>
>
>
>
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikeloz2015/ios-exercises/commit/fdf464265fc1699ce1b3c5f351925154c120ee29#commitcomment-11095884>
> .
>


Is the prepeding in IE 10+ also fixed? I don't think so, because IE handles the code using msSaveOrOpenBlob which isn't affected by changes on blob inside this functions. It is required to modify the blob before that function is called I guess


Both fixed (technically, they were not off-by-ones, though): https://github.com/git-for-windows/build-extra/commit/c902278c6b883cf319a7af819a00dd4d638acf57


Fucking politics. Ok, I'll address this.


The travis CLI formats it like this.
Yes, should be correct.


this is wrong... the python version is managed by tox. this change will cause travis to build the same thing three times.


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


This code searches bids twice: ```bids.maximum``` and ```bids.find_by_amount``` which is unnecessary. The method ```high_bid``` should do the only search and return a ```bid``` instance not a number. Then code ```high_bid.amount``` would be more natural. You want ```bid``` objects to be comparable by amount. So why not ```include Comparable``` in the ```Bid``` class and implement the spaceship operator ```<=>```?
```
# in class Bid
def <=>(other)
  @amount <=> other.amount
end
...
# in class Item
def high_bid
  bids.max
end
```
and no need for the ```winning_bid``` method any more,


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2182 of 2200 tests passed)

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


Authorization vs authentication... in this case a user is authorized to proceed with the request as long as he's authenticated. The naming seems a little confusing... might want to have an `Authorizer` decide based on the `Authenticator` output (instead of the authenticator knowing about authorization).

```
public isAuthorized
authenticate(user, pass)


(oops nevermind the bit of code that I wrote as part of my last comment about authorization vs authentication)


@MaicoLeberle @Mallku2 Ahora me doy cuenta que dejé el path de Flex y Bison que utilizo yo (es porque utilizo las últimas versiones que bajo con el package manager y no las que vienen por defecto que están outdated). Cualquier cosa, si no les compila vean de cambiar tanto las vars LEX como YACC y las LDFLAGS y CPPFLAGS que se adapten a su entorno. Perdón por eso, se me pasó jaja.


> The problem is that c now contains a mixture of items that can be safely changed on-the-fly, and items that require a server restart. And we don't really know which are which.

True, but the end goal is to have literally EVERY option in ``c`` be something that can be changed.  And we're not very far from that now; it will take a reasonably small amount of effort to fix the last few holdouts.  The main reason I didn't finish that is that MAGStock didn't need it and we were on a deadline.

> My approach was to do the c refactor item-by-item so tests could be run and we would be able to more easily find regressions. Also, for efficiency, the Redis config will actually live in several different dicts so we can only call a refresh on the ones that are relevant to a current action; this also simplifies configuration changes. There is no need to use __getattribute__ because everything going into the config variables already lives in a dict - that dict just happens to do some voodoo to stay in sync with the version in Redis. The existing config loading will keep loading the existing globals (now c) until we have moved every place in the code referring to c to instead refer to the appropriate "new" dict.

Ah, interesting.  That was something I didn't understand from either the discussions we'd had or from the existing commits.

I guess I have two main questions about that architecture choice then:

1) How long would it take to do a full sync when a config change happens?  I would expect config changes to happen relatively rarely, since they're something that a human is doing and thus it's not like there will be 10 of them happening every second through some automated process.  So even if it takes literally several seconds to do a sync, is that not okay?  And I can't imagine that it does take several seconds, in which case is that level of complexity required?

2) Am I understanding you correctly that you envision splitting ``c`` into a number of different redis-backed dicts?  If so then I'm dubious about that from an API standpoint.  Right now one of the nice things about the unified ``c`` global is that the programmer doesn't have to know or remember which of several different config objects hold the option they're referencing.  Even if we decided to go with the architecture choice of having several different redis-backed dicts for the purposes of efficiency, my preference would still be to just define a ``__getattribute__`` for ``c`` and then have it check and dispatch to the appropriate redis-backed dict.  That way we could still just say ``c.ATTENDEE_BADGE`` and ``c.PAYMENT_METHODS`` and whatever else without needing to know or care which bucket we've put the options in for the sake of efficiency.  (Again, it's possible that I'm missing and/or misunderstanding something.)


yep!  nice idea


Because SPSA is an algorithm to find a local optimum of a continuous function, but the ordering is not a continuous function of the factor.


At a risk of sounding arrogant... This is 2015. Everyone is using `utf8`. Why do you care about BOM exactly?

Why should anyone waste time supporting `notepad.exe` when it's painfully obvious Microsoft couldn't care less about this piece of... 'software'? The more it breaks, the less people will use it, the easier our lives will be.

I do not want to see libraries adding undocumented random binary junk in their output. This is an invitation for trouble.


Oh, awesome, thank you @quarzoliquido for the spanish translation! :smile:

BTW, I'm sorry for not making this clear, but the main file to edit is `index.md`.

This project uses [Jekyll](http://jekyllrb.com/) to generate the HTML from `index.md`. I will make sure to update the README to make that clear.

All you have to do is edit the `index.md` and then use `jekyll build` to generate the website.


Ya están arriba los cambios


+1; looks good to me.


That makes a lot of sense.  My suggestion reveals I don't understand that part of the code.


That's really helpful! I wasn't sure if we were going with the "question set" idea or if trivia was supposed to just send individual questions, so this clears things up. I'll get more work done on this later this week.

--Sonia

> On May 7, 2015, at 13:29, Wasabi Fan <notifications@github.com> wrote:
>
> This looks like a good start! Here's a quick class outline that I had written down while we were discussing the design of this class that you can refer to if you're not sure how to continue:
>
> "trivia question" class -- represents a single question
> Has a publically-gettable string for the question text
> Has a publically-gettable set of the available answers
> Has some knowledge of the correct answer
> "trivia set" class -- represents the current set of trivia questions that is being asked
> Keeps track of an array or list of current questions in the set
> Has a publically-gettable question object for the current question
> Has a method to submit a user's answer
> Compares the given answer to the current question's correct answer
> If it's correct increments a counter of the number of correct questions in the set
> Sets the "current question" variable to the next one in the set (or "null" if it was the last question)
> "trivia" class -- supplies basic tracking of overall trivia
> Keeps a bank of available questions (that haven't been put into a set yet)
> Has a method that will return a trivia set with the supplied number of questions
> I hope that my notes help -- if they aren't clear and you aren't sure how to go about implementing something, feel free to ask for input. I think one thing that might be helpful for you is to just write the empty class skeletons with empty method declarations so that you know what code you need to fill in later on.
>
> ―
> Reply to this email directly or view it on GitHub.
>


LGTM


@yankejustin (https://github.com/yankejustin/xRAT/commit/b59cee60258b99bc4a656b29f5373f1de2f999e3#commitcomment-11086977) very cool, I'm looking forward to it. :)


Cool. Like it.


It was originally `MP_OBJ_IS_OBJ` specifically to catch both native and user defined types.  Hashing needs to be efficient, hence why a lot of types are handled directly in this function.  For everything else, it delegated to `__hash__`, whether that be in a native table or user type.

So it should be reverted.


Dessa tester gick inte igenom!!


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1138 of 1138 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)


||die!


@vineetgarc no problem, have now posted upstream:  http://lists.uclibc.org/pipermail/uclibc/2015-May/048934.html


do you need to pass the file path of the current file to check if there's an `etag` match? unless I'm misunderstanding and currentFile is actually an etagFile and not the Authenticator.java file?


LGTM, except I feel queasy with writing "URL's"...is that actually the right way of doing it?


O.K., sounds good. I had the feeling after we met on Friday that I had confused you more than I had cleared things up ;)


Whoops, thanks for fixing this!


it is one of the oddities of `Gemfile` and `.gemspec`;

`Gemfile` is for gem dependency management in your project, and `gemspec` is for defining the gem.  So by including dependencies in the `gemspec` file, it will pull in other dependencies when folks download the gem.  However, `add_development_dependency` doesn't do that...it is only for local use.

The 'proper' behavior is, for gems, to use `gemspec`.  The downside is that it isn't *near* as flexible as the `gemfile`, esp. for things like non-released dependencies, MRI vs JRUBY dependencies, etc.


> Authentication = login + password (who you are)
Authorization = permissions (what you are allowed to do)

the etag match doesn't seem to have anything to do with authentication or authorization?


@sta this line causes "Use of unassigned local variable 'data'" error


Are you making a valid assumption about the order of Auction records?


@sta same here - causes "Use of unassigned local variable 'data'" error


90179052-SetupEMSActuatorMissingDuplicates (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1650 of 1650 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)


Wonky commit message... fumble fingers.


the `PutPostResponse` already stores the filepath, you could use `protected` in the variable instead and get rid of this line


It might be better to create an alias w/ deprecation notice, rather than change the class name? @tractorcow

Changing the class name *could* break modules, if they're referencing this class.


I think I should just take this method out of this and put it into its own class, making this method just a static method


obviously ignore this


@trown
EL7 has pexpect-2.3 so Requires: python-pexpect should be replaced with:

%if 0%{?fedora}
Requires: python-pexpect
%else
Requires: pexpect
%endif


PHPCS: Expected 1 space after closing brace; newline found


ignore this file


Loaded Fugue icons, you can skip this commit :D


So for our code I actually would like to propose that via a config we throw an error if there is no name specified... what do you think?


@jnugent, I don't think this template file was ever added. Could you check your repo to see if you have it?


When I do `:*`, nvim prints `E20: Mark not set`. Is this to be expected as opposed to it not being recognized as mentioned here?


Won't be long before there are records in the database for more than one auction and then this code will erroneously include the winning bids from any auction. Seems to me that an auction has many winning bids, but that relationship is missing from your model.


Kids, don't do it at home :) There's always another way to do it :)


General comment #1:

I understand BDD is to cover all global cfg for a blank installation, correct? And you seems to defined one BDD per data item?

Right now if cover: system message, global properties/environment variables, jdk and email notification. But what about the following global properties:
- 	# of executors
- Label
- Usage
- Quiet period
.....
Global properties:
- Disable Xvnc execution on this node
- Tool location
.....

General comment #2: in jdk and env var you verify that new value are use in job when AM leave cluster, i.e. the BDD focus on HS using proper data when becoming AM, which is good. But shall we also check that data is replicated to HS even if AM still active, i.e. replication successful? Maybe it does not matter if the only use case where HS need to have proper value is when is take over (which so far it is)

JDK BDD:
- If a job is configured to use JDK1 on AM, and you delete JDK1 and install JDK2 on AM, shouldn't you also need to reconfigure the job on AM to now use JDK2? Otherwise the job will fail, cannot find the deleted jdk.
- Do you consider replication of all possible scenarios/data associated with JDK i.e. "Install automatically" check/uncheck, use installers?
- Given you don't specifically verify the global cfg replication, is there a risks that the operation to add/update or remove JDK that is NOT use in a job is not cover?


how do you test this changed?


If you have better, i'll take it ! ;)


sounds good


impressive coding skills.


Need change " {{app_label ...}}" to " {{opts.app_config.verbose_name ...}}" https://docs.djangoproject.com/en/1.8/ref/applications/#for-application-authors


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.91](http://teamcity.monogame.net/viewLog.html?buildId=7057&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:20:15


you're testing the string output of the response here.. be careful because if for some reason the `respond` method changes, it will break all of your tests, you could test against the response object and ensure that the string is built correctly where you test `respond()`


Working on it


Yo, why you versioning compiled code?


GJSlint : HarvestController.js is now longer than 80 characters


This is as it was before (remember, `*` isn't in `'cpoptions'` by default). `:*` is equivalent to `:'<,'>`, which gives `E20` if you haven't used Visual mode before.


:+1: I was going to suggest this in the previous commit


> True, but the end goal is to have literally EVERY option in c be something that can be changed. And we're not very far from that now; it will take a reasonably small amount of effort to fix the last few holdouts. The main reason I didn't finish that is that MAGStock didn't need it and we were on a deadline.

Would you be able to factor out every instance you're not confident can be changed on the fly into a separate variable? Maybe c_static or something?

1. The recommended method per the DurableDict documentation is to call a sync at the beginning of every request. If nothing has changed, this is an extremely fast O(1) lookup. If anything has changed, then the longer - but still O(1) - lookup of the entire dict is made from Redis and parsed into the Python native dict structure. This way, the check for sync doesn't happen on every single lookup, only once per request.

2. I primarily want to do this to separate out different types of Redis-backed data. I envision, eventually, "core configuration" items being in one dict, "plugin specific" ones being in a different one. Those two could be combined and both use `c`, I don't see that as a major deal though I would prefer to have a more detailed and hierarchical naming convention for config items as it's kinda Wild West right now. There's no real reason those couldn't all live within `c`; my concern is just that putting them all in the same dict makes it easy for us to continue throwing globals in there with no real enforcement of naming conventions or specification of which part of the system that configuration is actually for.

I do think that clearly distinct TYPES of configuration data, though, should be explicitly referenced by the programmer. For instance, the dict that holds "this is what we call a BADGE for this event", or the dict that holds "this is our text of our bottom-of-page disclaimer for this event" or the dict that holds "these are the email templates for this event". That's more a future phase, though, so not too relevant at the moment.


dont you think it is wrong way? maybe just uncomment one line in .vimrc.before (I tried, it works...)

...and at alll - that is not the problem. Have you found out what is the real problem?


The following will cause the entity to be dirtied if it has a collection reference that was set to null or replaced by a different collection, regardless of whether the entity is versioned.

Dirtying the entity ensures that the EntityEntry loaded state is updated by an EntityUpdateAction. I've verified that if the entity is not versioned and there are no other dirty properties in the entity, EntityUpdateAction will not issue an UPDATE statement.


Not a biggie but in the future it's nice to have the ticket number in the commit log to make cherry-picking clearer


:+1:  @jalle19 had commented, this sentence was odd, this hopes to make it better. I only wanted to correct the spelling but I think this is a better improvement. Suggestions for better wording welcome.


this looks like an cob spec's acceptance test, if this class had been test driven you'd probably end up with smaller steps, such as `returnsEmptyStringWhenNoQueryParams`, `decodesOneParam`, `decodesTwoOrMoreParams`, etc... the problem of not having unit tests is that if something breaks you won't be able to pinpoint what it is as easily, your only test will break for different reasons


@skrew It's there https://github.com/DyCI/dyci-main/commit/469167bb1bc821d1eaaaf4ea10a268cec0743651


Would it be better to just update the values rather than changing the keys throughout en.yml? We could break people who are referencing the old keys. They would also have to be changed in the non-english yml files too. @tractorcow


Why not `while (flag)` ?


This code needs a TODO comment because it must be revisited to replace "Forrest" so that the appropriate sponsor is used.


Typo "Cultor"


Just casually banning 4.653138834498368145776998455562e+613 IP's with those double *'s there, completely pointless and damn stupid.


With `while (flag)` (see previous comment), this is no longer necessary.


I don't often think it makes sense to commit a change just to fix whitespace (versus fixing it when you have to change the code) but I think that this is OK.


The locking is indeed meant for all operations.
Here I tried out the diagonal lines:
![qml3webvfx](https://cloud.githubusercontent.com/assets/9467/7526062/f8e1455a-f50e-11e4-8255-004bf5682921.png)
It's hard to find a suitable color within SystemPalette


(value & CultureTypes.UserCustomCulture) != 0
please :)


Yes sir,  works like a charm. Thanks a mil loud.


alias is the only way this is going into `3.2`. The only time we can remove classes is next in `4.0`...


Why do you have a flag per thread? Since you're changing the value of all flags at once, a single flag shared for all thread would work (and be more efficient).


lifespan
noun
1. the longest period over which the life of any organism or species may extend, according to the available biological knowledge concerning it.
2. the longevity of an individual.

The meat is dead already. It decomposes after a certain period of time.


what?? didnt get it, anyways I guess would be nice to remove public from
version control imho

On Thu, May 7, 2015 at 11:13 PM, Gabriele Cirulli <notifications@github.com>
wrote:

> Koral used GitHub pages, I copy pasted the Koral thing
>
>
>
> > On 07 May 2015, at 23:09, Vasco Fernandes <notifications@github.com>
> wrote:
> >
> > Yo, why you versioning compiled code?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scum/scum-frontend/commit/808ec7dfca318045ad025bc28a2ed42921a39653#commitcomment-11096416>
> .
>


I agree. Nobody sees these kets in the CMS. You can leave a comment:

```
# @TODO: rename these before 4.0
```


Merging


90179052-SetupEMSActuatorMissingDuplicates (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (626 of 626 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-5.26%25-green.svg)


Good


btw this gulpfile is larger than gymia's backend :P


You fixed both the escape issue and the arrows with a one-line change, and kept omnicomplete?

If so, could you post how at the original [bug](https://github.com/spf13/spf13-vim/issues/567)?

There are several workarounds proposed there, but no root cause or solution yet - the bug is still open.  This was impacting my daily work, and I was not getting any great benefit from omni_complete, so I killed it so that I could move on.

When a solution is implemented, I'll update from spf-13-vim and remove this customization.


Will this be merged to master?


Excel will display wrong characters on some cases like ç, à, á and other latin characters if BOM isn't used


the 'aauth_' part is not really needed because its in the aauth lang file


Should also be attributed to ceylon/ceylon-ide-eclipse#1341


I see, I missed the note above under `*cpo-star*`. I should probably get off for now :-)


What is this for?


No javadoc? Shame upon you :)


Hello Dr. Brown,

I'm glad my talk didn't put you to sleep. Thanks for the kind words. I just
posted "Exercise 6" to my GitHub.

Best wishes,
Vallmer

Vallmer Edward Jordan II
Ph.D. Student
Bridges to the Doctorate Fellow

Batzer Lab (Laboratory of Comparative Genomics)
Department of Biological Sciences
A653 Life Sciences Building
Louisiana State University
Baton Rouge, LA, 70803 USA

On Tue, May 5, 2015 at 5:34 PM, Jeremy Brown <notifications@github.com>
wrote:

> Vallmer, how far did you get on code related to trees? I don't see
> anything in your repo related to "Exercise 6".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vallmer/Computational-Phylogenetics-Scripts/commit/c7aced45e1ce036142534198bda8d1ed20bac88b#commitcomment-11054402>
> .
>


Just a typo :(


Great!


From Codeigniter documentation:
Note: It's a good practice to use a common prefix for all messages in a given file to avoid collisions with similarly named items in other files. For example, if you are creating error messages you might prefix them with error_

I use multiple lang-files at once and have run into issues in the past.


Mistake. Oops


Hola, Podrias ayudar con la estructura de la base de datos el sitio web parese no estar en servicio http://vighiapremier.com, si la tienes seria bueno agregarla o facilitarla.  Gracias.


Oh, ok.

I guess, you will remove it with some of your upcoming PR-s with changes to `build.gradle`?
I mean, I'm lazy and don't want to fix it myself. :)


No Mac 10.10? It has been 7 months since it shipped...


Camelize?


Cited evidence on arrow functions being broken in iojs/io.js#1655 for anyone seeking further information.


Yes it's planned, unless @gbrail will do it himself in `master` :)


@pzich - no problem :smile:


Why do you have 10.10 here and not above?


How do you get from a specific auction to all of its winning bids?


pretty sure you fixed this code in a later commit ("it" becomes invalid when calling "erase") - squash those together instead of having commits with buggy code


Why both mac and mac64? Do we have 32-bit testers?


How about an example?

I suggest you check the code you use to generate XLSX files first. Those are supposed to be XML and I've never seen any problems in XML regarding utf8.


@ironfroggy We are using dublin core metadata. However, the underlying data models to support dublin core metadata has been changed. Initially we were using a third-party module that allowed us to store only one value for given metadata element. The underlying data model for that was key/value. Due to the need for each metadata element to have a number of sub elements, we had to define data models for each of the dublin core metadata terms. Only recently, I removed reference to the old 'dublincore' submodule in code. However, now I see there still exists some references. If you are asking why the 'dublincore' submodule still exists in the project I think Michael probably forgot to remove it. If resource filtering needs to filter on metadata then it needs to use the newer metadata models. The metadata we see on resource landing page comes from these models. Not sure if I answered your question.


There is a setFlowStepStrategy where we can put a FlowStepStrategy that sets a conf to the JobConf .  We can set "mapred.name" but that wouldn't change the "step name" that it appends to that name.  Also the .dot file wouldn't work.  However, for the production version we can push a change to cascading that reads from a specific config to set the stepName and then set it in a FlowStepStrategy.


bypass in line 24:
datetime.date.today instead of datetime.date.today()


oh ok i havent issues with that


The problem is that the zoom code was assuming the the closing tiddler was being displayed when in fact it could be hidden.

When hidden (note that another tiddler in the list will have display set to block):
*removing the tiddler from the list with animation (not really part of the problem as it is hidden)
*displaying the next tiddler in the list (changing display to block and animating).  - this can cause two tiddlers to now have there display style attribute set to display one on top of the other.

The fix is just to remove a hidden closing tiddler.


Made changes for this in https://github.com/tbaik/java-server/commit/8b8e69e1105351191cc8062e242ce815e0984562
it's now log(String message) and we log images without its bytearray body separately.


ı still dont understand because ı toggled torch through dev/ledflash path many times and ım still able to take pictures with flash, no broken camera nor 3rd party cam apps...


ResortedSurfaces (amirroth) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


approved


a MultiPolygon example might be helpful here to clarify things.


oooooooooooo


As far as keyword vs. booleans for true/false, the only thing I wonder is what is best for whatever's going to be consuming this, unless we want to coerce these to booleans before including in the http response JSON. But looking at the commentary on PE-8269, it seems like for load balancers it's not really going to matter either way, so long as a predictable string is found. So I'll change this to be all keywords.


Is this alwyas going to work if there's more than one auction in the database?


Oh yeah, I just remembered that IE will only use msSaveOrOpenBlob anyways. I'll look into fixing that as well today


More lines == more power (???)

anyway I'll remove the stuff

> On 07 May 2015, at 23:18, Vasco Fernandes <notifications@github.com> wrote:
>
> what?? didnt get it, anyways I guess would be nice to remove public from
> version control imho
>
> On Thu, May 7, 2015 at 11:13 PM, Gabriele Cirulli <notifications@github.com>
> wrote:
>
> > Koral used GitHub pages, I copy pasted the Koral thing
> >
> >
> >
> > > On 07 May 2015, at 23:09, Vasco Fernandes <notifications@github.com>
> > wrote:
> > >
> > > Yo, why you versioning compiled code?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/scum/scum-frontend/commit/808ec7dfca318045ad025bc28a2ed42921a39653#commitcomment-11096416>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


> At a risk of sounding arrogant... This is 2015. Everyone is using utf8. Why do you care about BOM exactly?

Because Chrome still doesn't support the charset parameter for Blobs, unlike Firefox. This is literally the only thing I can do to hint to Chrome that a file is UTF-8.


We can't remove any classes in 3.2. We can have classes with deprecation notice and just not use them, though.


I made one individual commit for each your comment of the previous pr. The commit corresponding that particular one is this one https://github.com/gmarceau/stellar-core/commit/57b64e604aba191c70dc2d6b29dabbfeafc63b0c


Maybe break this out into provider / source?  (For cases where the provider is doing so on behalf of another party.)


You don't need to touch en.yml... this is updated from the source code automatically as a part of the release process.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1138 of 1138 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)


Format?  Or is GeoTIFF assumed (not a bad thing)?


No need for the statsd lib any more. Can be removed


Looks good to me


statsd lib not required. This can be removed.


indentation off by one


Arrow function?


> You fixed both the escape issue and the arrows with a one-line change, and kept omnicomplete?
Not yet. I try to find what is up...
> If so, could you post how at the original bug?
Yep, I will do it when I got it. But... I noticed that this commit is not the best. Firstly, this is local (in my opinion it SHOULDN'T be edited, even in repo...), secondly the same effect when uncomment one line in .vimrc.before (global workaround for this bug, isn't it?)
> There are several workarounds proposed there, but no root cause or solution yet - the bug is still open. This was impacting my daily work, and I was not getting any great benefit from omni_complete, so I killed it so that I could move on.
OK, i know, but now you probably know what I suggest.
> When a solution is implemented, I'll update from spf-13-vim and remove this customization.
Yep.


Do we need the _benchmark_ prefix?


I added `code` formatting to all the literal values. I can undo it if you prefer.


tufClient?


why a generic `interface{}`?


why not `failOk := false` and getting rid of the comment.

And even if not, you should do `var ok, failOK bool // false by default`


![screen shot 2015-05-08 at 7 37 32 am](https://cloud.githubusercontent.com/assets/386336/7526494/5a18f516-f555-11e4-97e7-c4a7be1a33c6.png)


it's how the existing distribution configuration works.


@juhovh any reason you didn't merge this yet into master? I have a POC for this now and it works reliable (needed to add some methods for injecting time and totaltime to our player but it now shows the correct values in kodi - hopefully those injection methods will be accepted by the team).


Resolves: #18


@ChrisWren: this is what I meant. Maybe it's worth adding an option for this or include it as warnings.


yep


thank you


есть в планах пофиксить хтмл скан ?


@tlunter hi, would you push v1.21.4 tag to GitHub?


Return an error and a result?


Updated Italian Translation


you are assuming that mProposedSetTrackers only contains txset for the ledger that just closed - which I think is incorrect - there are some edge cases where externalize could be called out of sync with triggernext. It's safer to just index on ledger sequence number (triggernext is only called once per ledger)


Let's get a new name for this variable.


@pfalcon do you have a micropython-lib/struct implementation in the pipeline?  Lots of things no longer run under unix because struct doesn't exist anymore.


The variable name `failOK` makes this logic hard to read. At least for someone with very limited mental abilities, like me.


squash the commit that removes it to avoid the churn


failOK is true in this instance so we want to report the error but proceed.


sorry for all the confusion. I used to have a etag variable in authenticator plus this method but I forgot to rename it once I changed that.


would `failureIsOK` be better for you? :-P


So with this commit, does it just feed the game your port file? Or will it generate the user port correctly now?


Discussion is at https://github.com/juhovh/shairplay/issues/14 and I had problems with the values, but that might have been because of timestamp missing. Not sure if I should merge this to master or move it to the experimental branch I have with NTP and all. I have to rewrite it there anyway, because callbacks are now moved to a separate outout thread instead of raop_rtp.


Used my secondary github account to reply apparently...


assuming you need this because of some obscure bug in this version of SCP (that I hope doesn't happen in the new one) - file an issue to re-introduce the assert with the new SCP version in case the merge is too clever


- fichier a utiliser aussi avec overall_header ! attention a bien verifier.


does this change belong in the previous commit that updated test cases?


this commit should be squashed in the commit that introduced the change in the first place


rax retry tests


``requireNotNull`` for outputBuffers


Just FYI, we should be able to remove that floating patch by [using the RHEL developer toolset on our CentOS Jenkins agent](https://github.com/joyent/node-infra/issues/4). Hopefully someone will be able to do that work soon.


It buys us not having to include Storage.strings in l10n stuff.


Can we make it run on ARM? :wink:


Well i only use those values for current time and duration atm - it doesn't matter if they differ from the timestamps for this use case. Because of the ntp branch stuff will be much more work to support in kodi (and take longer for getting it approved) i would find it pretty cool if you could add the progress callback as is to master as well so we have the "full feature set" with dacp and progress in kodi before adding the ntp stuff at a later point. Of course its up to you really :)


I don't know if anybody will find this using Google, but here's the error this patch fixes:
PHP Fatal error: Call-time pass-by-reference has been removed in pg2mysql.inc.php on line 133


The code as it stands faithfully reproduces the behavior that existed prior to the pretching code, see here
* https://github.com/stellar/stellar-core/blob/a0e66eb1eddcfcda6e3b78b25ca9fd69297a9f5d/src/herder/HerderImpl.cpp#L1205
and here
* https://github.com/stellar/stellar-core/blob/a0e66eb1eddcfcda6e3b78b25ca9fd69297a9f5d/src/herder/HerderImpl.cpp#L815.

so I would prefer to address this issue in a separate pr.


This file is probably not needed @hhutch


I don't understand this change:
* this is reintroducing something you undid it 3 commits ago
* only difference it seems is that it does the async behavior every 100 operations instead of every operation
does it mean that we now have a 1% chance of triggering the bug that was triggered with the async callback?
If so, why is this safe?


This is awesome, really nice work!

Staring at these hard-coded messages got me thinking: we should probably use json schema (or something similar) to validate the messages on both sides of the postmessage API. Not important right now, but maybe on the road to Nightly


Looks good to me.


Long story.... but Mozmill still lacks one missing feature afair before we can fully make use of 10.10 on production.


why does mesos.interface pull 3.0 protobuf? doest it not pin the dep version?


WSHP_SHR_GreaterThan_1 (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1644 of 1650 tests passed)

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


agreed. opened: https://github.com/stellar/stellar-core/issues/480


I'm going to remove the empty-popup packet from the prototype in the next few days, probably good to just remove it here too.


That exists for testing the current feature set of Mozmill with 10.10. Once all is working we could upgrade the machines in production. But given my limited time I'm not sure if this will happen. Maybe we do it when replacing those jobs with Firefox UI tests.


Weird that I didn't have it pushed, but then I have lot of code lying around... Pushed to both repo and pypi.


this test is rather slow so i keep changing depending on what i need. this hunk is just churn.


We should move this to ``HashGenerator`` so that it is not duplicated


pas de problème :)


Ah, yeah... I noticed it didn't really do anything, thus the `if (false)`. I think I just copypasta'd that in from a gist also


No need for `else`.


This needs a better localization comment.


well no: before it would just re-download it from some other peer at validation time, while the new code will just crash


deleteRowsInGameByGameName.php needs to also delete all teams that are associated with that game.
same for gameNameAndPatch but we won't use that.
Thanks


To avoid waiting for user input when running test .. what about checking if testing is enabled.

```
if not (background.GetScalarType()==vtk.VTK_SHORT and gestureInput.GetScalarType()==vtk.VTK_SHORT):
  logging.warning("GrowCut .... ")
  if slicer.app.commandOptions().testingEnabled:
    return
  result = qt.QMessageBox.quest ...
  if result != qt.QMessageBox.Ok:
    [...]
```

else:

```


Hi @mgruner,

Thanks for info, it is much better now. To be honest we though of this, and Sanjin suggested me that we should add IDs however I wanted to make Selenium test without changing frontend modules.
Actually it is the best solution, and it is good that you added IDs.

Regards
Zoran


Yeah just found 1 drawable lol.


I had to take the long way around to figure out whether or not to add the root node to `fluidElements` below, since `element.matches()` still has only partial support. Any thoughts on a better way to do this?


mesos.interface specifies:
```
'install_requires': [ 'protobuf>=2.5.0,<3' ],
```

But apparently `3.0.0-alpha-2` is considered `<3`.


Thanks for catching!


TeamCity Azmq :: Build (OSX) [Build 74](https://78.46.71.115/viewLog.html?buildId=467&buildTypeId=Azmq_BuildOsx&guest=1) is now running


There are two bug fixes
* `resetReadIdle` while the event is in the queue
* `self = shared_from_this`.
That makes it solid. The magic `100` skip is for performance.


TeamCity Azmq :: Build (Linux) [Build 90](https://78.46.71.115/viewLog.html?buildId=465&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 116](https://78.46.71.115/viewLog.html?buildId=466&buildTypeId=Azmq_Build&guest=1) is now running


WSHP_SHR_GreaterThan_1 (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1644 of 1650 tests passed)

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


This fixes issue #7


This will truncate to whole seconds. Use `Double(self) / 1000`.


Actually, better yet, use

```
    return NSDate(timeIntervalSince1970: millisecondsToDecimalSeconds(self))
```


what about `allowsFailure` or `failureAllowed`, or `strictMode` ?


TeamCity Azmq :: Build (OSX) [Build 74](https://78.46.71.115/viewLog.html?buildId=467&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:01:25


This...is not a fork of spf-13-vim.  This is my personal dotfiles repo.

I'm not trying to contribute to spf-13-vim with this change, I'm just customizing my own personal environment.  This repo just exists to help me maintain the same environment across multiple hosts.


TeamCity Azmq :: Build (Linux) [Build 90](https://78.46.71.115/viewLog.html?buildId=465&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:01:54


lets see if this will cause it to send me ( @tmlink ) email


TeamCity Azmq :: Build (Windows) [Build 116](https://78.46.71.115/viewLog.html?buildId=466&buildTypeId=Azmq_Build&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:03


Can we make a bower component out of it? It seams wrong to include it on skeleton by default.


Yes, but still, this is something that we shouldn't use for production now until we know more about it.


Thanks!


Yes, don't know another way of doing this, unfortunately :-1:


Also removed the manage-2525d-styles utility project as it really belongs in a separate branch


Ty callbacky v npmInstall a bowerInstall jsou kandidáti na refactoring (DRY), nemám teď nápad na elegantnější řešení :)


why not COPY it there directly? like you do on the next line for ovf-env?


@josevalim This change breaks deriving on `Access` because there is no `Access.Any`. Phoenix is one the projects that breaks.


The new code will provide the value from the the `ItemFetcher`'s lru cache, which is tuned to be longer-lasting that what used to be provided by the other nodes.

To be clear I totally agree that it is an issue and that it needs fixing. I'm simply saying that the fix belongs in a different pr since it's not regression


LGTM :+1:


@mojodna We're going to assume RGB GeoTiff to start and maybe expand through further discussion. @lossyrob should we add it to the README?


if you do
```make
binary: $(BINDIR)/$(BIN)
$(BINDIR)/$(BIN): docker-extension/*.go
 ...
```
make can actually check whether the go files changed and run the build only if it has to...

Feel free to ignore this comment, as I don't think it will need much speedup...


do not review this commit, please read the PR message.


Is there any worry with overwriting the existing file (which could contain v4 links)?


> Would you be able to factor out every instance you're not confident can be changed on the fly into a separate variable? Maybe c_static or something?

I just looked through the list of all config variables and I don't see anything that we couldn't theoretically change on the fly.  There are a few variables which at the moment can't be changed on the fly because of how we're currently using them, but like I said it wouldn't be much of a refactor at this point to fix it so that the last few holdouts would behave properly with an on-the-fly change.  I've just made a ticket specifically for that, since it's related to the Redis refactor but is a good idea in general: https://github.com/magfest/ubersystem/issues/1103


Why you have to explicitly check if func is not undefined? What would the case it is undefined


Yes it looks nice :) I have not known that some of that fields could be triggered as the submit directly, but it is good options. I know that it works in some forms, and we use it for certain input fields. Nevertheless, it is good that there are IDs for Update button for another test.

Step by step we are coming to the best solution :)

Regards
Zoran


90179052-SetupEMSActuatorMissingDuplicates (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1139 of 1139 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.82%25-green.svg)


<yy0409b>
レビューありがとうございます。ではメソッド名は、掛け算=multiplication の意味から"mul"に直したいと思いますがよろしいでしょうか。


That's what you want to do @bmmpxf .


This comment is no longer true then(?) since we also aggregate the type from yield expression in the case of generator


I think the point is: inserts and updates return ints (counts or ids), and queries return cursors. So these two things both have specific types.

I do see the point of having either return `Void`, but I think we'll probably address that when we switch to `Deferred`.


But yes, these two can share an implementation.


I agree with the +2 ..

For the triangle the idea is to have an isosceles triangle where the bottom line is half the width of a fieldsize and if we want to center it in the field, the line starts at + fieldSize / 4 and ends at  + 3 * fieldSize / 4. I'm not sure what names I should use for this any suggestions?


Should be fix factoids :)


Did you manage to run it?


alright, although I might remove all this entirely. this was for testing.


Did this fix your problem? should i merge this back into my repo?


@ruifcardoso [I have fixed it for real now](https://github.com/eligrey/FileSaver.js/commit/babc6d9d8fc60e667ddeafb7a7d3b844ce761ab5), sorry about the incorrect commit message.


OK.

I recall updating some of these (ages ago), so that they returned something, but it looks like I must have missed one.

I presume you are picking these up because you've increased your warning level.

Actually, I'll post another issue about other changes to remove warnings


The parser is a little bit lenient in parsing yield expressions. In correct ES6 code, it is impossible for a yield expression to not have a func, you're right. But the parser recognizes that "yield" followed by an identifier is always a yield expression, even outside of a yield context. So that is the case, func will not exist.


Your error is here you have a if block outside of a method.  just remove this if block.


Yes, good catch. I should augment the comment to account for yield expressions too. Or should I just remove it? Not sure it is helpful.


@stevage ah thanks - no let's keep that's an improvement.


93906584_FluidCoolersTwoSpeed (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1650 of 1650 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)


In **MemoryRecordUnit.pas** at line **2185** there should be this:
```if (customtype<>nil) and customtype.scriptNeedsRealAddress then```


We ref() all slices contained in the message, so it's ok to unref() them
after AsyncMethod, but only safe to change them after the completion of
Finish().

On Thu, May 7, 2015 at 1:06 PM Vijay Pai <notifications@github.com> wrote:

> What does this imply about when it is ok to change the contents of the
> buffer that was input to the AsyncMethod? Is it immediately after the call
> to AsyncMethod or only after the call to Finish ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ctiller/grpc/commit/3676b38183c2d288c6daef1c0f3fd9b08f3927bc#commitcomment-11095322>
> .
>


This is very familiar-sounding, but it looks like the HTML spec now does this kind of thing with the magic words "in parallel": https://html.spec.whatwg.org/#in-parallel

The most common incantation seems to be "Return, but continue running these steps in parallel" although usually returning a promise or object.


Not sure if latexml.sty is the right place for such, but...

I assume you've actually tried this out? on something non-trivial? I'd suspect that stuff involving packages like babel (maybe) or tikz (more likely) are going to fail.


either assert or don't check if string is empty. Don't no-op silently.


Cuentame las novedades de vighia que hace como 2 años que no sé nada de ese
proyecto. La base de datos la teniamos alojada en un servidor propio y una
copia en un vps que contrató vighia pero lo botaron por falta de pago como
a los 2 meses.

El 7 de mayo de 2015, 16:27, pmercadohn <notifications@github.com> escribió:

> Hola, Podrias ayudar con la estructura de la base de datos el sitio web
> parese no estar en servicio http://vighiapremier.com, si la tienes seria
> bueno agregarla o facilitarla. Gracias.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mapizeta/vighia/commit/60160a09c53c7e9a57b90c4e8617caca45419b6b#commitcomment-11096621>
> .
>

--
Se despide atentamente de usted.

Manuel Pizarro Mendoza
Ingeniero en Computación e Informática

Optimiza Ltda.

*Temuco*
*fono: 56-9-61303568*


The key is to preload latexml.sty last, after all non-trivial packages have finished loading.

It's not perfect, but it goes a long way and fills in some very large security holes.


In the helsana projects i use the uncompressed files and when i do the prod build i replace the main.js file by the uglified version...


should it rather return the filename (`$self->{SSL_options}->{SSL_ca_file}`) here?

It looks like the %ssl_args is filled with 1 and later overridden with the values in `$self`, but that's a little bit odd code and isn't consistent with other return values in this method.


Interestingly, string concatenation (`+`) is faster than `sprintf`.  ;)


RadianceRefactor (macumber) - i386-Windows-7-VisualStudio-12: OK (2182 of 2200 tests passed)

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


TeamCity Azmq :: Build (OSX) [Build 75](https://78.46.71.115/viewLog.html?buildId=469&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 91](https://78.46.71.115/viewLog.html?buildId=468&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 117](https://78.46.71.115/viewLog.html?buildId=470&buildTypeId=Azmq_Build&guest=1) is now running


ResortedSurfaces (amirroth) - 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)


TeamCity Azmq :: Build (OSX) [Build 75](https://78.46.71.115/viewLog.html?buildId=469&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:51


remove this


TeamCity Azmq :: Build (Linux) [Build 91](https://78.46.71.115/viewLog.html?buildId=468&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:01:03


n,n,m,mn,m,mnm,n


The perfect solution would have a clear distinction between user-invoked IO and package-invoked IO, so that the complex cases succeed without any issue, even after latexml.sty gets loaded.


this should be return and yield  expression


ah or maybe this could help us: https://github.com/jrburke/almond


Missing newline at end of file.


Why fuck Etherpad?


Whitespace up to the indentation level isn't "trailing whitespace" and shouldn't be removed.  (Code folding in certain editors requires even blank lines to match the surrounding scope.)


Same missing newline.


Yes


Hello this is a test line.


Good catch


Straight assignment of the exception object to the Document instance will likely not work.  Some effort will need to be made to save the traceback (stack trace) and exception class (`marrow.package` should help resolving the exception class to the import path for it) and then reconstruct the original exception (printing the original traceback) when anything attempts to access the result of the task.


I think you want the JobConf

https://github.com/twitter/scalding/blob/d19e656974bfca00615a5084db0f5409bc3fade1/scalding-core/src/main/scala/com/twitter/scalding/reducer_estimation/Common.scala#L74


This change is incompatible with my app.

```php
		$url = 'https://api.twitter.com/1.1/search/tweets.json';
		$getfield = '?q=#'.$hashtag;
		$requestMethod = 'GET';
		$twitter = new TwitterAPIExchange($settings);
		$response = $twitter->setGetfield($getfield)
			->buildOauth($url, $requestMethod)
			->performRequest();
```

With the `rawurlencode` addition, the hashtag's pound symbol was double escaped. And I received the following error:
```json
{"errors":[{"code":32,"message":"Could not authenticate you."}]}
```

b91a4af15b73fd359b8a2f3b9f0a1574fc836848 seems stable.


can you schedule at sprt(-1,4) instead (and sprt(0,5) for ltc if it passes) ?

to be fair to your patch we need to reduce the resolution a bit to detect the extra elo (if any) of the capture condition.


The README is pretty clear about avoiding "pickle" wherever possible.  Where this is used (exception storage) see the comment above for the values to store.  On "unpacking" the exception it should be reinstantiated using the stored class reference and *args saved from the original.  Pickle *might* be the only option, here, though.  It's worth testing.


TeamCity Azmq :: Build (Windows) [Build 117](https://78.46.71.115/viewLog.html?buildId=470&buildTypeId=Azmq_Build&guest=1) outcome was **FAILURE**
Summary: Tests passed: 7; exit code 8 (new); failed to parse xml report (new) Build time: 00:09:23


- fichier a check


Hi Team,

I am facing issues with select option group .

Version 0.0.5 is not supporting option group.I have gone through the releases and downloaded
or
node_modules/.bin/bower install angular-ui-select2#0.0.5
it is supporting only options
 repeatOption = tElm.find('option[ng-repeat], option[data-ng-repeat]');

Here it is missing option group..
But in master branch is supporting option groups.

Could you please release one more version  form  mater branch .. then we can download or install through browser.

Thanks,
r@m.


- fichier a verifier


@mhansen1 Eigen has operator overloads to do matrix math, you could've just written
```
CMat = UMat->transpose() * RsideMat;
```


Resolves #388.