TeamCity YaccConstructor :: YC.Core General Build [Build 452](http://teamcity.codebetter.com/viewLog.html?buildId=193794&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed (47).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies)   at FSI_0002.Core.commonConfig@178-12.Invoke(Unit arg10@) in Z:\Builds\work\4138ce7c6332a78c\tools\Build.Tools\Core.fsx:line ... Build time: 00:14:11
### Failed tests
```
YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Branched binop or negation.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Branched binop.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Branched multy-digit numbers sum 1.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Branched multy-digit numbers sum 2.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Branched multy-digit numbers with Binop.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Branched multy-digit numbers.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Calc with braces 2.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Complex 0: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Complex branched 1.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Complex branched 2.: <no details avaliable>

YC.FST.AbstractLexing.Tests.Calc+Lexer Calc Fst Tests.Calc. Complex branched 3.: <no details avaliable>

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


@varund7726  you can stop updating this now lol


docstring would be helpful


Docstring would be helpful


![woooo](https://cloud.githubusercontent.com/assets/144458/8283623/2975aed8-18ad-11e5-9f93-cb71f54d12ce.gif)

Yeah... I was gonna update it and then I realize that it still works... basically the guys managing the hosting can't seem to decide whether to use `get.popcorntime.io` or `cdn.popcorntime.`. Both work the same (at least right now), and `get` is what the website shows.

Also, I changed the hash of the update.json, not the one of the .dmg


Fuuuuuuck :-D


Perfect!

You can do this in one line via [.update_all](http://api.rubyonrails.org/classes/ActiveRecord/Relation.html#method-i-update_all):

```ruby
wikis.update_all(private: false)
```

since the method ```wikis``` automatically scopes the wikis to the ones that the particular user instance owns, it will only update the appropriate users.


Its a great toolchain
Try UBERTC 5.1 for rom too
got 50k on antutu


What you really want to do here is set the `leftBarButtonItem` property on the `self.navigationItem` to your `backButton`, and I think you'll see the difference.

You might also want to use the following method to create your button, which will give it a target method to hit when pressed, as well (and is localized/localizable).

```
    backButton = [[UIBarButtonItem alloc] initWithTitle:NSLocalizedString(@"Back", @"Back") style:UIBarButtonItemStyleDone target:self.webView action:@selector(goBack)];
```


@roberthorlings I'm not sure at which point in the process these SAMSamples are supposed to be created (my db doesn't contain any). Is it when editing the study assays / samples?


wound't that be enough if we get separator only ones - maybe even as part of the class as it's defined via system property so it should not change after it was initialized?


Not tested, but looks good to me.


Yeah... :-D

Maybe you should rebase, move the pie chart stuff to another branch, and
keep experimental separate.
When we move to XValue classes we may want to do it to the all of the
charts - and keep convenience function for backwards compatibility... I
don't know about you but I don't have the time for that right now :-)

On Mon, Jun 22, 2015 at 5:10 PM, Philipp Jahoda <notifications@github.com>
wrote:

> Fuuuuuuck :-D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11794714>
> .
>


This test can't have worked for over a year - it will be fixed...


alternatively we could add separator at the end and then remove last separator from the end of the buffer


HI Gokul,

Thanks for all your help till now.

I have attained success in passing weblogs to Kafka. Kafka can read my messages, but when I run storm(to pass the message from Kafka to storm) which is integrated with Kafka, I am getting following error.
Error I am facing is : backtype.storm.util - Async loop died
![screenshot from 2015-06-22 19 31 58](https://cloud.githubusercontent.com/assets/12342549/8283596/d56d74ec-1915-11e5-8e1a-3d72fc788d89.png)

Thanks in advance for your guidance.


I've updated the pull request. I rebased with master, so I think this comment that we're commenting on is no longer in the branch. I'll put some of my comments under the pull request


maybe this logic to collect all org entities into string could be encapsulated into a method as this code is related several times, just for improved readability.


since new fields were added DDL and upgrade scripts for dbs is needed


I am extremely busy right now, this is why I can't do much atm.
There are a lot of big issues that should be addressed before moving to the x-values:
 - multiple bardatasets in combined chart not working (currently, each bar takes a space of 1 on the xaxis, that should not be the case for multiple bardatasets - they should "share" a space of 1)
 - stacked barchart negative values not working (i partly solved that already)


https://github.com/bruno-/vim-alt-mappings
meta key bindings work well for me.


maybe remove All from the name of the method as using both All and QueryFilter is bit confusing, wdyt?


did you pullrequest this ?


Awesome...I figured there was a one line solution, but was looking for a
derivative of update_attributes

On Mon, Jun 22, 2015 at 9:10 AM, Levi Cole <notifications@github.com> wrote:

> Perfect!
>
> You can do this in one line via .update_all
> <http://api.rubyonrails.org/classes/ActiveRecord/Relation.html#method-i-update_all>
> :
>
> wikis.update_all(private: false)
>
> since the method wikis automatically scopes the wikis to the ones that
> the particular user instance owns, it will only update the appropriate
> users.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paulmhenke/bloc_wiki/commit/a7ba806cddc953525f92ae2ea6a3314b52198604#commitcomment-11794723>
> .
>


Schweet!


@Stratoscale/runtime  please review


The disk array is down, @natefoo plans to pay it a visit.


Yeah about multiple bardatasets in combined charts I'm getting issues
opened as well...
And about negative values - I see you've done some work on horizontal
charts but not on vertical charts?

On Mon, Jun 22, 2015 at 5:15 PM, Philipp Jahoda <notifications@github.com>
wrote:

> I am extremely busy right now, this is why I can't do much atm.
> There are a lot of big issues that should be addressed before moving to
> the x-values:
>
>    - multiple bardatasets in combined chart not working (currently, each
>    bar takes a space of 1 on the xaxis, that should not be the case for
>    multiple bardatasets - they should "share" a space of 1)
>    - stacked barchart negative values not working (i partly solved that
>    already)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11794853>
> .
>


@benzonico for this one I have no good solution to easily remove the reverse traversal of the list.


Thanks for this, looks good. just gotta find some time to test it locally and I'll be merging


Here are a few things I would do with this...

I would extract all of the upgrade/downgrade logic into the model. You would then have two methods:
  1. User#updgrade
  2. User#downgrade

This will clean up the controller, and encapsulate all the steps that are required to upgrade the user into two spots.

You don't have to do this right now, but here are the methods that I would extract these things to:

1. User#stripe_customer (retrieves the stripe customer the same way it's done via line 38)
2. User#stripe_subscription (retrieves the subscription the same way it's done on line 39/40 minus the delet)
3. User#downgrade would update the user's role if deleting the users subscription was successful.
4. User#upgrade would create the subscription, and update the role if the subscription was created successfully.

If you want to do this later as extra credit, you and I can pair on this. It requires a bit of rescuing from stripe errors, but I had to do this already at my last job, so I have a good idea of what errors we need to rescue from.


I'm an idiot


You should have left "levels" as it is and instead changed it to "speed-1"


I think a lot of the css customizing features can be removed since template overrides does everything the css injectors were intended to do


If I ditch arj, adding unar support for arj archives is easy; I run it twice on both parts then I concatenate the 2 parts with GDP built-in support. I like having the possibility to use the old tools, but here it costs a bit much...

That's a lot of work just to display
`ARJ32 v 3.10, Copyright (c) 1998-2004, ARJ Software Russia. [28 Mar 2015]`

(on the other hand I don't quite like all the stuff pulled in by unar)


after having reverted to this it should be easy to produce the .dot file of dependencies to see what needs fixing... F.


A coupling between Profile2 and Roster2 all ready exists. Roster2 uses org.sakaiproject.profile2.logic.ProfileConnectionsLogic (L 55, 91, 598)

I added a null check before using the Profile2 SakaiProxy, which is how the ProfileConnectionsLogic is used.

If this is not acceptable, let me know and I will read in the Profile2 properties directly.


@deas I personally tried to include this pagedSearch in a project which had some performance issues on large datalists. This however meant problems since we do some post query processing (removing other nodes that could not be adressed in the query). What problems did you experience? No results at all? Or missing nodes?


Here you go :)


yea, your are right, but I fixed it here 7182ef675dbaa96d874d56434ba650abe055616d
I'll work on changing create.html -> bonecard/create/


adminWoeTypes might be a better name for this.


seems like there is no RUntimeDataServiceEJBImpl for these new methods similar as it was added to CDI impl.


Actually this breaks things, like it required `develop` to be up to date when running `review post`, which is rarely the case since you usually run `review post` before pushing. Will remote this in another commit.


Hahahaha you sneaky little gitch, welldone !


this could be extracted into producer class/method that sets up audit task service completely and makes it available for injection already configured so no post construct would be needed


@bjornfor thanks! Fixed it with a612e397d8e85c2eedb131d558464c6782d03b47


There is a great rails helper that will do this for you! [number_to_currency](http://api.rubyonrails.org/classes/ActionView/Helpers/NumberHelper.html#method-i-number_to_currency).

No need to update the code, just thought you would want to know :smile:


Yes, I have gotten several email requests concerning horizontal stacks and negative values so I have started to do that first. It should not be too hard to adapt that to the "normal" stacked bars.

![chart_age_graph_1](https://cloud.githubusercontent.com/assets/6759734/8284007/e1eeb35e-18fa-11e5-977d-894334e5b6db.gif)


Why is this necessary?

This is causing MSVC build failures -- https://ci.appveyor.com/project/jrfonseca/apitrace/build/65/job/1cmd8fmtjn6wm6ul#L133 -- so it will need to be wrapped inside `#ifdef HAVE_UNISTD_H ... #endif`

Also please submit a patch for https://github.com/google/snappy , otherwise this will most likely get busted next time I upgrade to a newer snappy version.


Fixed in 123b9cd54005fe219cee73787e9b34e40a315859


Obsolete.
Please use salsalt.gross.yuck.tomato.rotten


Unable to add the orders to the database. Will investigate


@nyalldawson I think to have managed to change all the icons to svg ones for add and remove icons in 9838ea3 (sorry for the commits noise)


Використовуйте, будь ласка, коментарі phpdoc для класів та методів


when it was on the bottom, bootstrap overrides the default theme, and this fixes it with me. not sure if there is another way to keep the default theme as it is?


Make no mistake, the layout is taken from http://codepen.io/marcysutton/pen/OPbpKm


But I cannot install through cask, because the hash differs?
The current link isn't a direct link to the DMG, but to a download page (at least for me):
![screen shot 2015-06-22 at 16 22 59](https://cloud.githubusercontent.com/assets/1612390/8284032/046fe1be-18fb-11e5-820a-c0a018e9b8db.png)
So the hash is different for each time I run `brew cask`.
You can see they supply a direct link on the page.


Great catch!!!


Hmmm det burde da ikke være nødvendigt? Kan du ikke få modul8 input ved at trykke `i`? Den burde huske valget.


this essentially loads all tasks from db for given offset/size which can result in empty result if first 10 items do not contain user/group if requesting user. So we certainly need to move the searching into db level otherwise this will be unreliable.

Overall, not sure we should flatten the assignments (business owners and potential owners) and instead join it with these assignments tables. With that in place we can efficiently (and reliably) query via audit tasks.


Для читання набагато зручніше, якщо ви винесете використовувані простори імен у верхню частину коду, в блок "use"


Yes it shouldn't be too hard as it's the same code :-)

On Mon, Jun 22, 2015 at 5:22 PM, Philipp Jahoda <notifications@github.com>
wrote:

> Yes, I have gotten several email requests concerning horizontal stacks and
> negative values so I have started to do that first. It should not be too
> hard to adapt that to the "normal" stacked bars.
>
> [image: chart_age_graph_1]
> <https://cloud.githubusercontent.com/assets/6759734/8284007/e1eeb35e-18fa-11e5-977d-894334e5b6db.gif>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11794993>
> .
>


bravo!


Can you show me some code? jQuery's `.off()` shouldn't be calling `.destroy()` on the slider at all.


Hey man, i double checked core, none of this stuff is there, go to item_mods in core and search usukane, 5 hits.. all lvl 75 gear..


А якщо не знайшлося? ^_^


What's the latest version of the ZendAuthenticator which does not require 5.5?


That's interesting. What's the URL from which you actually can download the file?


He oui, la fonction course_index n'existe pas


@tomaszmueller did you notice that this broke the build? looks like just a matter of fixing the expected strings in 4 of the tests to have uuids instead of ids.


@asterite Thanks! Regarding JSON parsing at compile-time, did you mean this [run](http://crystal-lang.org/api/index.html#http%3A//crystal-lang.org/api/Macros.html%23run%2528filename%252C%2520%252Aargs%2529%2520%253A%2520MacroId-instance-method) ? Because your link refers to the root crystal api.

About `emoji.json`, I use [rake](https://github.com/veelenga/emoji.cr/blob/b2e276d36ed599f26edb1d705e191bfa38d3c5f7/Rakefile) task to load it. I would like to rely on file supported by [gemoji](https://github.com/github/gemoji/blob/master/db/emoji.json) instead of my version in repo. Can you suggest any other useful way to load it before compiling a project ?


AH, this is so close.

Replace the ```<li>``` with the ```content_tag```. The difference here is that you are going to create an ```<li>``` with the ```content_tag```

```erb
<%= content_tag :li, id: "item-#{item}" do %>
  <%= item.name %>
  <div class="pull-right">
    [the destroy link]
  </div>
<% end %>
```

We use the content_tag to wrap the content in an element that has an id that will match the selector in destroy.js.erb. (i.e. ```$("#item-1234")```). You may have seen just the destroy link disappearing, that's because it was the only thing wrapped by the content tag.


Didn't know about tinymce acutely! is it ok to use ckeditor or do I have to change it to tinymce?


This one: http://188.226.210.107/build/Popcorn-Time-0.3.7.2-Mac.dmg


I don't really think this belongs in a colour-change PR, and I don't think it really brings WDTK any closer to other MS branding.


Made the 3 changes based on your feedback (font and color adjustments).  Also added a colored shadow around the images for collections to make them "pop" more.


This doesn't match the return type of `make_list`


While trying to fix the case that you proposed, I discovered that `Target.getFirstTarget()` returns `null` when its target is illegal. The offending piece of code appears to be in [`TargetImpl.isLegal`](https://github.com/magefree/mage/blob/fb313be1b6973845ced0aa609ba2e0e3ba8ca90e/Mage/src/mage/target/TargetImpl.java#L360). How to I get the Permanent/LKI of a target that is now illegal? As currently written, Shapesharer's ability correctly handles the case where the Shapeshifter target is illegal (because the call to `game.getPermanent` returns `null`), but it does not correctly handle the case where the creature to be copied from is illegal.


fixes #743


I think it's not really necessary to have a route atm? Maybe just return ""


I don't think so, `b#` seems like a simpler choice. I just didn't think of it.


gibts hier ne reference zu einem bekannt safari bug?


I already did it. check out the current develop branch. I also upgraded the gem `cancan` to `cancanca


1. There are four kinds of ```response_map```s. They are  ```review_response_map```, ```quiz_response_map```, ```feedback_response_map```, ```metareview_response_map``` and  ```teammate_review_response_map```.
2. There is no ```participant_review_response_map```, and :mortar_board: @efg recommend renaming ```team_review_response_map``` to ```review_response_map```.
3. So one ```assignment_team``` will receive many ```review_response_maps``` and we can access to certain ```response``` via ```map_id``` in ```review_response_map```.


a quick test with run shell "am startservice -a eu.faircode.backpacktrack2.TRACKPOINT" shows it is working. I will do some more testing and update the readme with some usecases.


Huh. I didn't try taking it higher than a div block which I added surrounding the h1. Once that didn't work I figured I was barking up the wrong tree. Nice sleuthing!

Personally, I think you came up with the fix with that one, but I guess if you want to call it teamwork, here's my UUID: 9b561b3e-ef0b-4079-9718-c7e5095ff69c  :-)


We shouldn't be testing for content. Content can change and it does not matter what the front page is


I think that was when I inherited bonecard-code with bonecard.


TeamCity YaccConstructor :: YC.Core General Build [Build 453](http://teamcity.codebetter.com/viewLog.html?buildId=193797&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


I did test everything, but a second eye is always better. My main problem is that I cannot test with Tasker, since I don't use it.


Oh, good catch this!


Should this last } have a semi colon? In the old version and in previous commits before my last merge the final } in the enum had a semi colon ie };


1. must convert this from raw-array to list
2. must provide 7 parameters


We should actually keep at at tests/

I think it would be best to have tests/ mirror the code base. So something like
/tests/client/app_test.py


What if you try this one:

https://cdn.popcorntime.io/build/Popcorn-Time-0.3.7.2-Mac.dmg

If this works for you, as a direct download (i.e. no redirects), then I will update the URL using that one.


Great. I am working on categorize implementation. So it will not effect with #30.


@fgerards Could you give us some more info so we can follow up on this?


fixed! also, we now actually have control over the order in which JS files are included


you should, it is a really powerful tool


test


@lindamartin I'm a dummy in line endings, but just to be sure, are you sure that it should be "\n\r" and not "\r\n"?


And PS - My wife will be very happy that you fixed that issue. So, thank you from her.


should keep this as client.api


That one works! 👍


Introduzione pubblicata


Perfect. Easy peasy.


What I meant to ask was "does it work with unar without the need to manually concatenate the resulting files if you rename the archives before?"


I have used your custom datableconverter to create a custom datasetconverter like this:

    Public Class DataSetConverter Inherits JsonConverter

    Public Overrides Function CanConvert(ByVal objectType As System.Type) As Boolean
        Return GetType(DataSet).IsAssignableFrom(objectType)
    End Function

    Public Overrides Function ReadJson(ByVal reader As Newtonsoft.Json.JsonReader, ByVal       objectType As System.Type, ByVal existingValue As Object, ByVal serializer As  Newtonsoft.Json.JsonSerializer) As Object
        Dim ds As DataSet = New DataSet()
        Dim converter As DataTableConverter = New VSApps.SerializableCtrls.DataTableConverter()
        reader.Read()
        While (reader.TokenType = JsonToken.PropertyName)

            Dim dt As DataTable = DirectCast(converter.ReadJson(reader, GetType(DataTable), Nothing, serializer), DataTable)
            ds.Tables.Add(dt)
            reader.Read()
        End While
        Return ds
    End Function

    Public Overrides Sub WriteJson(ByVal writer As Newtonsoft.Json.JsonWriter, ByVal value As Object, ByVal serializer As Newtonsoft.Json.JsonSerializer)
        Dim dataset As DataSet = DirectCast(value, DataSet)
        Dim resolver As DefaultContractResolver = TryCast(serializer.ContractResolver, DefaultContractResolver)
        Dim converter As DataTableConverter = New DataTableConverter()
        writer.WriteStartObject()
        For Each table As DataTable In dataset.Tables
            writer.WritePropertyName(If(Not resolver.Equals(Nothing), resolver.GetResolvedPropertyName(table.TableName), table.TableName))
            converter.WriteJson(writer, table, serializer)
        Next
        writer.WriteEndObject()
    End Sub End Class

I am using this class in the following code:

           Dim table as DataTable= new DataTable();
            table.Columns.Add(new DataColumn("Column1", typeof(string)))
            table.Columns.Add(new DataColumn("Column2", typeof(int)))
            table.Rows.Add(new Object[] { "Row 1", 5 })
            table.Rows.Add(new Object[] { "Row 2", 2 })
            table.Rows.Add(new Object[] { "Row 3", 7 })
            table.TableName = "mytable"
            Dim dset as DataSet= new DataSet();
            dset.Tables.Add(table);
            Dim serializerSettings as Newtonsoft.Json.JsonSerializerSettings= new Newtonsoft.Json.JsonSerializerSettings()
            serializerSettings.Converters.Add(new DataSetConverter())
            Dim JsonDset as string= JsonConvert.SerializeObject(dset, Formatting.None, serializerSettings)
            Console.Write(jsonDset)
            Dim ds as DataSet= JsonConvert.DeserializeObject<DataSet>(jsonDset, serializerSettings)
            Console.write(jsonData)
The console displays:
_{"mytable":{"TableName":"mytable","Columns":[{"AllowDBNull":true,"AutoIncrement":false,"AutoIncrementSeed":0,"AutoIncrementStep":1,"Caption":"Column1","ColumnName":"Column1","DataType":"String","DateTimeMode":"UnspecifiedLocal","DefaultValue":null,"MaxLength":-1,"Ordinal":0,"ReadOnly":false,"Unique":false},{"AllowDBNull":true,"AutoIncrement":false,"AutoIncrementSeed":0,"AutoIncrementStep":1,"Caption":"Column2","ColumnName":"Column2","DataType":"Int32","DateTimeMode":"UnspecifiedLocal","DefaultValue":null,"MaxLength":-1,"Ordinal":1,"ReadOnly":false,"Unique":false}],"Rows":[["Row 1",5],["Row 2",2],["Row 3",7]],"PrimaryKey":[]}}_

Finally it displays this error. I can not see the cause. Is there any suggestions as regards to the cause of the error.
__A first chance exception of type Error reading JObject from JsonReader. Current JsonReader item is not an object: PropertyName. Path 'mytable', line 1, position 11.__

The debugger highlights:`Dim jObject As JObject = jObject.Load(reader)`


@electroly: Downside of this approrach is that you will have to constantly re-merge when features/fixes are added; can I suggest you integrate via consuming the NuGet package instead

You can always add a bridge class to wrapper CsvReader into your own namespace which then delegates to the NuGet library.


Indeed


alright. no problem.


@moezdhm merci pour les accès root en publique ;)


Nice. Just updated it.


This makes sense, otherwise we making it really easy to execute arbitrary code.


Chevere..!! por alguna razón no puedo colocar el fondo a lo jdialog no entiendo porque creo que no tienen un jpanel xD!


Was ich finden konnte:

http://tech.vg.no/2013/10/02/ios7-bug-shows-white-page-when-getting-304-not-modified-from-server/
https://bugs.webkit.org/show_bug.cgi?id=32829
http://stackoverflow.com/questions/3121301/safari-not-always-rendering-page (sehe gerade, die Antwort ist von @Hirbod :))


so descriptive bro


>Please make this more clear, this boundary definition is very confusing. If I read it correctly, you are over arrays, i.e. you go through exactly two iterations.

Yes, you are right, `sizeof(arrays)/sizeof(*arrays)` can be replaced  with number `2`, just don't like magic numbers.

>I also wonder, wouldn't it be better to use information about length of each array? You call string_array_len() on each array on line 236 so this information is available, you just need to store it.

Yes, I can use it. I will modify my code.


you're save to use <code>~2.4.0</code> as long term support


Would be great if we could add some tests which show and smoke-tests those side effects.


@lindamartin Why two more commits where required (which removing System.lineSeparator() usages)?


Hi @celebdor:

I used your code to create the tunnelzones directly on puppet module.

I realised that, if you can receive the hostname somehow in this code from the remote host, you don't need to create this `ip`-`hostId` map. You just have to filter by `name` when the lists of hosts is retrieved.

Is just a heads up. Keep in mind for future updates.


This is working, but I just noticed that after clicking on a link, the vertical-menu of links does not auto-hide, which means half of the screen is hidden (and basically no screen left on smartphones)


:1234:

:bow: for making code better


Excellente idée de feature ;)


why?
The old version doesn't work anymore.
I've replaced the old dashboard by the new version


Ich frage so blöd, da man ggf. den Fix auf eine bestimmte Safari version eingrenzen sollte, da er sonst eine zukünfitge Browser-Version ohne diesen Bug stark benachteiligt.


I don't know about the tests ignored in OSX, I only added two more tests for the new feature. Besides that I use Linux so I can't test them on my machine. Maybe @andrewgaul has the answer.


I wonder what is the function good for since the fallback function seems to be replacing it? If it's necessary, maybe it's not a bad idea to document where it should be used since now that I look at it, the usage of collection_exists and fallback_collections_exists does not seem to be completely consistent in the current code.


Yes, this looks like a good solution.

I suggest that you use `using Parent::next_layer;` instead of implementing a `next_layer` that simply calls is parent. Likewise for other of the trivial wrapper functions. For this to work, however, you need to move the definition of Parent to the beginning of the class.


The order of the items in `GetBackendProperties` will always be the same. I'm not sure I see what the problem is here.


TeamCity YaccConstructor :: YC.Core General Build [Build 453](http://teamcity.codebetter.com/viewLog.html?buildId=193797&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed. Process finished with exit code UnexpectedError (-100).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies)   at FSI_0002.Core.commonConfig@178-12.Invoke(Unit arg10@) in Z:\Builds\wor... Build time: 00:11:00


TeamCity Xbim :: XbimGeometry :: XbimGeometry [Build 5](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=14&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


Since this function is public, and `mNodes` is a `boost::optional`, should this check for unassigned?


Hi Paul.  My concern was with downstream users of my library who may already be including this library in their projects, as the LumenWorks library appears to be fairly popular.  (I'm pulling the same trick with Newtonsoft's JSON library, which is also extremely popular.)  I've had some difficulty with this in the past; if you create a project that uses libraries A and B, and library A additionally has its own dependency on B, you can have a problem if library A updates to a newer version of library B but your app does not (or vice versa).  I really just want this to be an internal implementation detail of my library that won't cause any trouble for downstream users.

My solution is fairly poor (I wrote a script that will apply the namespace changes so I can pull upstream changes and then re-apply my patch in one step) but I think it will serve my goals.


assert_equal 5, @card.value


El evento 'crash' no tiene parametros? seria genial que tenga parámetros y te pueda decir donde ocurrio el error o alguna información mas detallada.
¿Existe esa posibilidad?


Sure, we could store a copy of `SBackendProperties` in `CPVRGUIInfo` and only update that one whenever `m_iCurrentActiveClient` wraps around to zero again.


unar doesn't detect the .A01 file


assert_equal 'Spades', @card.suit


drop that test


That should be 0.4 nm / 4000 Angstroms, not 400 nm,.


Awesome work. Thanks for the continued additions!


No. And in the future we will likely do away with `mNodes` completely, or change how its used. Its optional so it can be destroyed/reconstructed instead of needing a "reset" function.


Edited


This is a Spring MVC assessment


On 22. 6. 2015 at 07:39:47, Lubos Kardos wrote:
> >Please make this more clear, this boundary definition is very confusing. If
> >I read it correctly, you are over arrays, i.e. you go through exactly two
> >iterations.
> Yes, you are right, `sizeof(arrays)/sizeof(*arrays)` can be replaced  with
> number `2`, just don't like magic numbers.

Well, you have it in the definition of the array so I don't think it would hurt
that much :-)

> >I also wonder, wouldn't it be better to use information about length of
> >each array? You call string_array_len() on each array on line 236 so this
> >information is available, you just need to store it.
> Yes, I can use it. I will modify my code.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lkardos/scl-utils/commit/2300c3c1a176f70b30f681880901922a
> f4726714#commitcomment-11795427


TeamCity Selenium POM :: CI [Build 25](https://teamcity.sponte.uk/viewLog.html?buildId=110&buildTypeId=SeleniumPom_Ci) is now running


Done here 7ccfb72


@pdurbin This looks like a really good change to me! It looks a lot better for our Dataverse.

I will say though, now that I'm looking at it, it might be even better if it showed the full title *and* boldfaced the matching term when applicable. I'm not sure if that's possible, and it's certainly not a hard requirement, but I wanted to mention it. Just having the full titles is a huge improvement by itself for sure, so thanks for that!


Done here 74423d5


TeamCity Selenium POM :: CI [Build 25](https://teamcity.sponte.uk/viewLog.html?buildId=110&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 35 Build time: 00:00:34


Not sure that you changed this, but seems like `<%-title%>` should be wrapped in quotes.


Done here aad1bbf


@mogoodrich It's okay as I am actually working on changing ids to uuids in the tests:)


You need to print to the io.Writer the other messages are printing to.


Was this an accidental push to master? Since the issue in the experimental branch was not fixed by this commit, but by a later commit.


Extra blank line detected.


Does this work?  I didn't realize += was a possibility....


Good fixes, Greg. One final tweak and you'll be done. Clients and managers can be extremely fussy so this is good practice. The red border is a lot of "pop"... just a little bit can go a long way. Trying using a light gray instead of red and increasing the blur radius so it's more subtle. Also, the border is only showing on the top three images, it should be on all six.


New issue here: https://github.com/mikrosimage/avTranscoder/issues/203


So my understanding is that http4s is now within spec regarding header value encoding. If thats the case, we will move ahead with the bugfix release. Let me know if that is not true.
Thanks for the discussion.


which is the minimum version of PyOpenSSL that expose this option? I think we should have that clear before merging the fix.


Die aktuellen Versionen sind auf jeden Fall betroffen, daher kann ich es bisher nicht eingrenzen. Links füge ich hinzu.


This is working, but I just noticed that the "Registrations" and "Disclosures" text labels are over-lapping horizontally. I'd say a simple fix for this is to transform those text labels into native tooltips on hover. Makes sense?


@JKE-be I've also posted this on the help forum. https://www.odoo.com/nl_NL/forum/help-1/question/how-to-add-an-extra-step-in-the-webshop-flow-87088


We could probably somehow build this string in an ansible variable (much how salt does it today) and then the template would only have the single variable.

`KUBE_API_ARGS="{{ kube_api_args }}"`

But if this actually works, nevermind...


nice catch @kevlened. Thanks


Yep, figured you were on it, just wanted to ping you just in case.  Thanks!


WTF!


@jeffrymorris

Sorry, I didn't mean to imply that the IQueryable implementation was
incorrect.  It looked fine to me.  I just felt that using an IQueryable
extension might not be the best approach for that particular function, and
that extending LINQ elsewhere had some merits.  For example, this new
approach can support META in the WHERE clause and selecting only specific
properties of the META object.  Also, once joins are up and working, it
will support specifying which document to request META information from.

I also had some thoughts about how this approach could be used to help
implement joins, which I can do a proof of concept on if you're
interested.  I'm not sure how far along in joins you are.  To me, one of
the big issues is how to map the idea of the "USE KEYS" statement to LINQ
expressions.  In LINQ join statements, you're expected to provide equality
comparisons to act as the join mechanism.  But the key of the document
isn't normally a property of the class we're querying against.  However, we
could use a static function N1Ql.Key(p) which could be recognized by the
join operation and converted to USE KEYS in the N1QL statement.

For example:

from beer in bucket.Queryable<Beer>
> join brewery in bucket.Queryable<Brewery> on N1Ql.Key(brewery) equals
> beer.brewery_id
> select new {beerName = beer.Name, abv = beer.Abv, breweryName =
> brewery.Name}

Could be parsed and converted to:

SELECT beer.name as beerName, beer.abv as abv, brewery.name as breweryName
> FROM `beer-sample` as beer
> INNER JOIN 'beer-sample' as brewery ON KEYS beer.brewery_id

This could also easily support combining multiple fields to get the key by
simply evaluating the expression on the right of the "equals".  And we
could provide support for unit testing that uses the N1Ql.Key method the
same way I did for N1Ql.Meta.

Thoughts?  If I'm wasting my time because you almost have it working
already please let me know.

Thanks,
Brant

On Sun, Jun 21, 2015 at 10:21 PM, Jeffry Morris <notifications@github.com>
wrote:

> Bryant-
>
> The approach I took was the (afaik) the standard way of extending the
> queryable interface. That being said it doesn't mean there are not
> alternative approaches.
>
> I'll take a look tomorrow (fathers day dinner atm) and let you know what I
> think. Also, you can always send pull requests; if they are not accepted
> you can just close them.
>
> Thanks,
>
> Jeff
> On Jun 21, 2015 6:39 PM, "Brant Burnett" <notifications@github.com> wrote:
>
> > @jeffrymorris <https://github.com/jeffrymorris>
> >
> > While I was working on other features in Linq2Couchbase this weekend, I
> > noticed some things that seemed cumbersome with the META keyword
> > implementation. I had an idea for another approach, which I've
> implemented
> > in this commit.
> >
> > I was hoping you could give it a glance and see if you think it's
> > worthwhile. If so I'll send you a pull request. Hopefully it isn't
> tearing
> > up your plans too badly. And sorry to keep blasting emails at you on a
> > Sunday, but I have to work on this in my free time.
> >
> > Thanks,
> > Brant
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/brantburnett/Linq2Couchbase/commit/ca0f142e48ed6ad9f172ca361ca40bb7ea4131ec#commitcomment-11786347
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brantburnett/Linq2Couchbase/commit/ca0f142e48ed6ad9f172ca361ca40bb7ea4131ec#commitcomment-11786534>
> .
>

--
Thanks,
Brant Burnett
CenterEdge Software
(336) 598-5934
http://centeredgesoftware.com


Thanks for your contribution.

Few remarks:
1. homepage url is invalid: http://www.example.com/Slicer/Extensions/PickAndPaintExtension


unar package `Recomends: unrar-free` ; that would explain why it can't process recent RAR files


@kostajh this tag is not working as I expected, nor are the instructions at the top of tags.yml applicable. What am I doing wrong? When running it generates this test error:
```
- ./_site/2015/04/01/building-our-site.html
  *  External link http://egardner.github.io/2015/01/09/asset-management-in-jekyll/ failed: 404 No error
- ./_site/2015/06/18/a-man-a-plan-a-canal-a-savas.html
  *  internally linking to /news/tag/identity/, which does not exist (line 724)
```


Chỗ này sao lại dùng session để lưu status cũ? Lúc tắt trình duyệt thì session bị huỷ mà?!

```java
    String oldStatus = contractOld.getStatus();
    HttpSession session = r.equest.getSession();
```

Làm đúng thì phải thế này: sau khi nhấn nút "Huỷ yêu cầu" (tức là trạng thái đang là "Request cancel"), thì lúc này phải **tính toán** lại trạng thái của hợp đồng.

- Nếu không có card nào gắn với hợp đồng: status = "No card".
- Nếu hợp đồng chưa hết hạn: status = "Ready".
- Nếu hợp đồng hết hạn: status = "Expired".


Thanks for the heads up @dkayiwa.  It looks like although I updated the pom for the distro-referenceapp project to use 1.7-SNAPSHOT of the uicommons module, on devtest01 it's still only version 1.6 that is being installed. Any idea what I am doing wrong?  I upgraded appui the same way, and that appeared to work.


@sapier Looks like a typo. It worked for me, anyway…


@fbudin69500 I will let you integrate when the cosmetic details are sorted out. Thanks


This fixes issue #287


yes.  let me fix that.


good catch


Should be [fixed](https://github.com/parasew/instiki/commit/7e58d11247b210985df67ba2eb087395e34b31af) now. Thanks.


Also, this problem occured which i've never seen before:
https://gist.github.com/Finomnis/3d1eb8bf2198ec306b4d


Actually you could just do something like this:

    status.register(AnyBar(<settings>))


@vilchik-elena indeed you're right. Thanks! Fixed in 5da79bd2cfcf9381b436ad14cf0860b895b31354.


Yep, du har ret… havede glemt at jeg skrev det… bare ignorere dete…
tak
______________
www.jonasfehr.ch

Furesøvej 161
2830 Virum DK

> On 22 Jun 2015, at 16:24, Jonas Jongejan <notifications@github.com> wrote:
>
> Hmmm det burde da ikke være nødvendigt? Kan du ikke få modul8 input ved at trykke i? Den burde huske valget.
>
> —
> Reply to this email directly or view it on GitHub.
>


@sapier Here the 'script' I was writing about is called. The conversion of XSAMPA to ARPA is done online, i.e. you can configure Jasper to use the XSAMPA model and it works out of the box.


Спасибо!


This could be a bit better than shrinking things to death:

```
(defun helm-hide-minibuffer-maybe ()
  (when helm-echo-input-in-header-line
    (let ((ov (make-overlay (point-min) (point-max) nil nil t)))
      (overlay-put ov 'window (selected-window))
      (overlay-put ov 'face (let ((bg-color (face-background 'default nil)))
                              `(:background ,bg-color :foreground ,bg-color)))
      (setq-local cursor-type nil))))

(add-hook 'helm-input-in-header-line-mini-set-up-hook #'helm-hide-minibuffer-maybe)
```

Didn't experience any bad side effects yet.


But it still works fine on 2005 (after reverting that commit of course)


cute penguin


L188-L201 is a very general pattern, if there is a shorter stdlib way to do this.


How about just "Candela Outcomes", I think we'll be adding other functionality at some point.


Omg :( my bad


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

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


Oh, and feel free to make a PR to merge this into i3pystatus:master. We're pretty open to niche things ;)


This is missing commas, resulting in syntax errors.


uh, right, this is qt 4.8. please make a proper bug report.


we generally wouldn't comment each parameter we pass in to a well documented function like `add_meta_box`


pagedSearch returned different total count depending on current page on 5.0.d. Found this with a contact  list of roughly 1500 entries. Still have (production) data in case we want to reproduce/fix pagedSearch. Going back to default search fixed that  for me.


@vilchik-elena because [System.lineSeparator()](http://docs.oracle.com/javase/7/docs/api/java/lang/System.html#lineSeparator()) returns "\n" on UNIX system and "\r\n" on Microsoft Windows systems. When "\n" can still be use don Windows system. I realised it only with the failing Integration Tests.


Uncaught ReferenceError: ABIlITIES is not defined.
Game freezes.


Они к ноутбуку [BigARTM для людей](http://nbviewer.ipython.org/github/bigartm/bigartm-book/blob/master/junk/ideas/BigARTM_For_Humans.ipynb), к сожалению nbviewer не умеет брать их из гитхаба и показывать


Brian, your reasoning and your method sounds fine; I just noticed the new fork and as I'd done a bunch of changes over the weekend wanted to make sure you were aware of the potential problems.

The SqlBulkCopy import stuff is really sweet now; did 9.9 m records in 11 minutes.


In this case we can use .call, so we don't have to wrap the arguments in an array


This generates a compile error, if winsock.h is included later, see #34. @tbeu, how do you handle this in your setup?
Following post suggests to ```#define _WINSOCKAPI_``` before loading windows.h.
http://stackoverflow.com/questions/1372480/c-redefinition-header-files-winsock2-h


elsif @value == :A


Indeed ;) L'export CSV est un peu cabreux sur des gros volumes


left commented code


@chrisarusso did you add a new markdown file for this tag in news/tag?


you can take that comment out IMO.


BTW, why not call the hook "helm-minibuffer-setup-hook"?  A minibuffer setup hook for helm might be useful for other purposes, too.  I should  then be run unconditionally of course if we do that.


Does it work if you check that target.is(document)? I'd prefer that solution in case it works, because it's less specific to what kind of element is focused (even if right now inputs are the only thing that can be focused in the browser I think)


Oops, forgot about that one :P


Why was this changed? Compactness? base16 had a nice GUID-feel to it :-/


Let's keep this as unfrozen for now.


So, it's working okay now? I would appreciate you verifying that there aren't any outstanding issues remaining... I would like to push the next release soon.


Nope. Must be it! Doing so now.


Use `<p>` tags instead of manually specifying where you want breaks please.


This is a Spring MVC assessment


So, there are more ways to display and work with Text than just the `FlxText` object. I was going to split this concept up into: working with `FlxText`, working with Bitmap Text, input text, etc, instead of just one Concept for the `FlxText` object alone...


`_meta_key` isn't very descriptive. I think maybe something like `candela_outcome_guid`. And this value is used multiple times and will be referenced in other code eventually, so it should probably be a constant at the top of the file.


Sorry, I don't understand ... Do you want I rename this commit?


@orangejenny thanks for taking care of this. Forgot to port your help bubble improvements.


nit: "This rule corresponds..."


@lindamartin ok. Then commit message and additional test cases could help you to bring this information


Need to remove this describe block too


Maybe use "Hello, World!" for the text?


The java 8 change also included a mybatis bump with the java 8 version of HikariCP


If we name this comment_count, and have a post_count as well, then we can have separate counts for each.

Other than that, this is great. You are the first student to get to this type of solution on their own! Great job Liz!


To make mybatis java 7 compliant we need to use the java6/7 HikariCP artifact


Content of CPVRClients::m_clientMap may change at runtime, especially between two calls to UpdatebackendCache. This is part of the problem this PR is trying to solve, if I got things right. ;-)

m_iCurrentActiveClient++ does not guarantee to pick the right successor if the map has changed since the last call to UpdateBackendCache. So, strictly speaking, the logic to remeber the last active client by index and to pick the next client by increasing the index does not work under all circumstances.

Just my 2 ct. The new code is wrt to this flaw as good (or bad) as the old code.


:ok:


Can you link to both of these rules when mentioning them (via `call`).


The output here still needs to be escaped so that it's not possible to inject anything into the page. The input cleaning is good, but you should still escape the output, probably just: `esc_attr($data)`
https://codex.wordpress.org/Validating_Sanitizing_and_Escaping_User_Data


How many people still used emacs `<= 24.3`?


Not actually fixed, just adjusted the test to reflect what should be returned. Just got around to working on this again


Little fix


best commit so far


The "fdrv" branch is generated.
Regarding the ISO, maybe we can offer a fwlite and a fwfull  version a la "PAE"
BTW I think is wrong to have this discussion here. I just opened a new "issue" on this https://github.com/puppylinux-woof-CE/woof-CE/issues/558


Thanks!  I too spent many hours typing in BASIC programs in the 80s. :)

It still isn't working quite correctly. For example, in the following program:

```BASIC
_10: LET A$ = "FOO";
_20: A$ = A$ + "BAR";
_30: PRINT A$;
```

For some reason, it's trying to convert something to a double. Perhaps the approach of using conversion operators isn't the right path and it would be better to just add all of the basic operators to variant.

Once that is in place, I think it would be pretty easy to add other types. There's a bool right now to indicate if something is a number, but that could be turned into a type enum.


Looks sensible :)


Hi Vivek

can you please give me an update about the issue above


It would be better to make an `__android_common.soy` file (you'll see other ones like that) and just use the same text for both instead of requiring the user to click a link to figure it out.


I'll check


You would need to update it on every change of the content of CPVRClients::m_clientMap.


This is the same as the `android_binary`, right?  We should put that in the `__android_common.soy` I mentioned above too.


I'm not certain because of my lack of php/wordpress experience, but I think this function is in the global scope, in that case having a generic name like `save_outcomes_meta` isn't very good because it could clash with someone else's code. So you should prefix it with something specific to this plugin like `candela_outcomes_save_meta_value`.
I think it'd be even nicer to make a class out of this plugin and make this reference a method in that class. Then it could keep this name, but it'd still be scoped because it's in a class.


Lamentablemente solo es un callback y no tienen variables de entrada.
Para más información visitar la siguiente url:
https://github.com/remy/nodemon/blob/master/doc/events.md#states


could just use @deck, bc setup is run before each and every test - @deck will be a fresh deck for this test.


Applied thx. strange that histogram didn't show up on the 169... data set.


The worst thing that could happen is that the information from one backend doesn't get displayed until the counter wraps around to zero again. I don't really see this as a problem. Currently it doesn't matter if `m_clientMap` changes between calls to `UpdateBackendCache` anyway.


This looks like pointing to 1.6
https://github.com/openmrs/openmrs-distro-referenceapplication/blob/master/pom.xml

On Mon, Jun 22, 2015 at 5:56 PM, Mark Goodrich <notifications@github.com>
wrote:

> Thanks for the heads up @dkayiwa <https://github.com/dkayiwa>. It looks
> like although I updated the pom for the distro-referenceapp project to use
> 1.7-SNAPSHOT of the uicommons module, on devtest01 it's still only version
> 1.6 that is being installed. Any idea what I am doing wrong? I upgraded
> appui the same way, and that appeared to work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-registrationapp/commit/5d6eba210fea73e2629612bbe15d30c8a0776b26#commitcomment-11795761>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


I like that you're cleaning the input.


is it worth correcting "we someone has" to "someone has" as part of this update? (a very minor nitpick, obviously)


done: https://github.com/combak/ShatteredWorld/commit/49946f0a8069e3d5f7812be414fc02c0ea1a8319


@lestarcdog: You commited *ResponseEvent.java* with Windows line endings `CRLF`. Please convert them to Unix line endings `LF` to avoid further change commits.


sometimes guids have other formats that we may want to consider. `:` are commonly used for example. Maybe look into that a bit. We don't have to support everything because we're choosing them, but it's nice to be a bit flexible.


Unfortunately I think you're right. http4s is certainly doing the right thing with regard to quotes and semicolons, which is a big improvement. If I can figure out a way to get newlines through header parsing that http4s doesn't handle, I'll submit another issue. Thanks.


remove all the `error_log` lines.


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


hey @emiltin (sorry for the late reply, google marked the notification as spam...)

> is the purpose to find route that visits all given waypoints?

yes that's the purpose. still pretty much WIP though  :)


@vilchik-elena indeed. Copy for next time!


don't commit this file.


Huh... I thought I just double-checked that I pushed up my change!  Let me check again...


Why do we need this file to be present in **this** mod, why can't we use it from default? Just add default as a dependency (it it isn't already).


which versions are affected?


> Also, this problem occured which i've never seen before: https://gist.github.com/Finomnis/3d1eb8bf2198ec306b4d

You need to give up the lock before assigning the id. This avoids holding the lock in case the underlying operation suspends. This has nothing to do with this patch, it's a problem in the HPXCL code:
```
--- "a/A:\\Temp\\TortoiseGit\\eve1EE1.tmp\\event-6e69343-left.hpp"
+++ "b/D:\\Devel\\hpxcl\\opencl\\lcos\\event.hpp"
@@ -194,6 +194,7 @@ namespace hpx { namespace opencl { namespace lcos { namespace detail
                 HPX_ASSERT(event_id.get_gid() != naming::invalid_gid);

                 // delete local reference to prevent recursive dependency
+                l.unlock();
                 event_id = hpx::naming::id_type();

                 return false;
```


This is looking great, strong work.


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


OK, this is fixe now in MMSdevel


Per our conversation on IRC that `buff`` may have embedded `NULL` bytes (which will break automated testing repeatability), it might be better do do something like:

```py
>>> program = '''
..: #include <stdio.h>
..: #include <stdlib.h>
..: #include <string.h>
..: #include <sys/mman.h>
..:
..: #define TARGET ((void*)0x11111000)
..:
..: void main() {
..:     char buff[1024];
..:     void* mem = mmap((void*)TARGET,
..:                      0x1000,
..:                      PROT_READ|PROT_WRITE,
..:                      MAP_PRIVATE|MAP_ANONYMOUS,
..:                      0, 0);
..:     if(!mem || mem != TARGET)
..:         exit(1);
..:     mem = (void*) 0x11111110;
..:     memset(mem, 0x41, 0x1000);
..:     scanf("%s", buff);
..:     fprintf(stderr, buff);
..:     write(1, mem, sizeof(int));
..: }'''
}'''
>>> subprocess.check_call(['gcc', ...])
```

Also, we can avoid needing to do any parsing by using `write` and `tube.unpack()`.

```
>>> addr = p.unpack()
```


This looks very confusing. Is it an internal idiom, emulating injected class names?


@kfogel thanks for the extra instructions here.


#187 changed attribute


Why also require the galaxy map scale line to be enabled here?


Thanks for the correction!


YES!

Le lun. 22 juin 2015 16:02, Navid <notifications@github.com> a écrit :

> magic number ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RomainGoussault/Deepov/commit/a22a01b8d91b4b1479feee973f0e2c89a8227b47#commitcomment-11794534>
> .
>


Oh. Yah that totally makes sense. Making me feel stupid again :P
Thanks for noticing.


Aha!  It looks like my change got blown away in a merge--I have
re-commited now.

Mark

On 06/22/2015 11:13 AM, dkayiwa wrote:
> This looks like pointing to 1.6
> https://github.com/openmrs/openmrs-distro-referenceapplication/blob/master/pom.xml
>
> On Mon, Jun 22, 2015 at 5:56 PM, Mark Goodrich <notifications@github.com>
> wrote:
>
> > Thanks for the heads up @dkayiwa <https://github.com/dkayiwa>. It looks
> > like although I updated the pom for the distro-referenceapp project
> to use
> > 1.7-SNAPSHOT of the uicommons module, on devtest01 it's still only
> version
> > 1.6 that is being installed. Any idea what I am doing wrong? I upgraded
> > appui the same way, and that appeared to work.
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/openmrs/openmrs-module-registrationapp/commit/5d6eba210fea73e2629612bbe15d30c8a0776b26#commitcomment-11795761>
> > .
> >
>
>
>
> --
> If we keep uppermost in our minds the unkind and unjust acts of others, we
> shall find it impossible to love them as Christ has loved us; but if our
> thoughts dwell upon the wondrous love and pity of Christ for us, the same
> spirit will flow out to others.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-registrationapp/commit/5d6eba210fea73e2629612bbe15d30c8a0776b26#commitcomment-11796178>.
>


Also, since FlxText colors default to White, this will be hard to see if they run the demo code as-is.
Not sure what the best solution is, maybe we start with the first proof being just "FlxText" and shows a few things about it at once?


@cards = [card_1, card_2]


Yes I remember now, I added that code some while ago to fix a bug with a spell having multiple targets.

But in the case the targeted Shapeshifter gets illegal you have to do nothing. So you have no problem I guess with that.
```
10/1/2007 	If the targeted Shapeshifter becomes an illegal target but the targeted creature doesn't, the ability will resolve but have no effect.
```


I send entity to generate id if it's not a BusinessEntityPage.


Consider:

for (auto const& _ : nodeIDs)


Do we need to add filter type? I figure we want to not include duplicates regardless of type.


I like this :D.


Typo
"This speeded up a lot of the back-office operations!"   ???


Do you need help with rebasing to revert the merge?

On Mon, Jun 22, 2015 at 5:24 PM, Daniel Cohen Gindi <danielgindi@gmail.com>
wrote:

> Yes it shouldn't be too hard as it's the same code :-)
>
> On Mon, Jun 22, 2015 at 5:22 PM, Philipp Jahoda <notifications@github.com>
> wrote:
>
>> Yes, I have gotten several email requests concerning horizontal stacks
>> and negative values so I have started to do that first. It should not be
>> too hard to adapt that to the "normal" stacked bars.
>>
>> [image: chart_age_graph_1]
>> <https://cloud.githubusercontent.com/assets/6759734/8284007/e1eeb35e-18fa-11e5-977d-894334e5b6db.gif>
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11794993>
>> .
>>
>
>


I didn't mean to remove this link, it's re-added in f328102b524c10bddb6f281108fd71d09f008b63


Thanks for the great classes this quarter Mike. Hope you had an awesome
father's day. See you around!
On Jun 21, 2015 2:54 PM, "Mike Sinkula" <notifications@github.com> wrote:

> Mike was here. ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bschwartz757/WEB170/commit/d634801e593f9894698c8b7307428368616c46a7#commitcomment-11785744>
> .
>


We rewrite the view reference for $view not for a businessEntity, businessEntities viewsReferences are rewrite after.


The CI part may be removed. We already discussed about it and it's not clear to me that this is needed anymore. It was necessary to use Buildyard during continuous integration, but with the current Jenkins infrastructure this is not used AFAIK.


src/ripple/server/impl/ServerHandlerImp.h:40:15: warning: private field 'm_jobQueue' is not used [-Wunused-private-field]
    JobQueue& m_jobQueue;
              ^


this is because of compress blocks that would surround these, to make sure it doesn't get excluded


It was both, accidentally pushed to master, Then cherry-picked to experimental after I had already committed something else.


This is an autogenerated file.


Im not exactly sure what you mean? Currently everything is on master - i merged the experimental some days ago


Also, the issues required 2 changes, this only being one of them.


Just formatting changes?


Sorry to pester--what's the best channel to watch for releases?


Renamed **dev_ioctl** to **ioctl** and **dev_read** to **read**.


mmmmmmmmmmmmmmmmmm.... pues ponle uno xD. Haber como queda.


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


Basically so we can put css in header and JS in footer?


Not sure why we really needed a shared client too, but I don't think it's affecting things one way or another.


well, they used to be part of this branch...might have stomped on them myself in the various git maneuvers done to separate out the help icon changes and then merge them into master before this PR


Yes - I mean if you want to revert the merge, because you've merge the
stuff with images in pie chart including x value classes...
You can either revert the changes with a new commit - or rebase, change
history...

On Mon, Jun 22, 2015 at 6:26 PM, Philipp Jahoda <notifications@github.com>
wrote:

> Im not exactly sure what you mean? Currently everything is on master - i
> merged the experimental some days ago
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11796493>
> .
>


First off, this .ino is fantastic. The only sketch of it's type that would work with my Teensy 3.1 straight off the bat.

I'm hoping to restart my scan from 5900 onwards until I can properly capture the correct code to use.

Does anyone know what variables I need to adjust in the code for this to work?

Thanks in advance,

David


see pull request #277


I'd also add that we have already and do as much as we can new integration tests that help to keep master stable (at least it helped me a lot :))


Yeah, the Impulse and Interactive I imported were trying to use C, but I didn't have it. My interactive and impulse should now run at lower frequencies more often.


yes, exactly


Wow, good job! :+1:


Is this led by @jonleung


I've seen many patches on openwrt to make various software compilable to compile with musl.
What a great compiler that needs thousands of fixes to existing code. :-(

I'm not a C-programmer. I only take the source code from the privoxy page and build the package around it.
Please ask privoxy developers or musl developers or musl package maintainer for support.

Thanks Christian


Looks good to me!


Should be.
OK, I test and move into private, also [C++03 one](https://github.com/boostorg/fusion/blob/master/include/boost/fusion/container/list/list.hpp#L51).


@Patabugen As @samsonasik, the 2.4 series is our [LTS](http://framework.zend.com/long-term-support/) series, and if you pin to `~2.4.0`, you'll get updates only in the 2.4.0-2.4.Z range going forward. 2.4 still supports PHP 5.3.23+; we made the switch to 5.5 starting with the 2.5.0 versions.


Hi Joe,

I'm not sure if this is working correctly, to create record with array I had to post

```
{
  "locales": "{en-GB, sk-SK}",
}
```

instead of

```
{
  "locales": ["en-GB", "sk-SK"],
}
```

Was it your intention?

Thanks


I would add an user picture with an arrow to show that users also create yadtshell processes when they work interactively


No, the reason for this change was that base64 is sort of AdX's standard for this value. You will find for example that the adx data transfer files contain the auction ids in websafe-base64, we actually made this change by request of other users so they wouldn't need to decode the value here from base16, and reencode it as base64, for matching data from those adx reports. We also use base64 internally for logs, it's just easier for everyone despite uglier :)


Yeah, I was playing with complete sorting, but am not really convinced. Changing this back everywhere.


Looks good


Awwwwww...


File is corrupted.


You were using the new-styled textures, which still use `default_sand_source_animated.png` even if it doesn't exist anymore.
https://github.com/Ombridride/minetest-minetestforfun-server/commit/a752f6efad6604611cbd76c13ab4c9d011c0ca7a


Just FTR: this is invalid and will not produce a `main.js` file, as `include` values must express absolute paths: http://webpack.github.io/docs/configuration.html#module-loaders


This doesn't change the accuracy, it just causes the supporting objects to be created lazily, instead of in the constructor. If a block position was passed to the constructor then the decimal position is only calculated on request from the int coordinates, and no precision is lost. If the decimal position is given to the constructor, then that is returned by this method, and the block position is computed when requested.


assert @hand.cards.include? new_card


What you describe is the default bootstrap behavior, but I can add a few lines of JS to make the menu collapse when you click on a link: http://stackoverflow.com/questions/21203111/bootstrap-3-collapsed-menu-doesnt-close-on-click


Looks good for me!


Thank you for the explanation! Makes sense.


Migration is made redundant by new scheduler and bb41cc7 removing App specific code.


@osworx You know that's not valid JSON right?


More info regarding this change: https://github.com/js-cookie/js-cookie/issues/42#issuecomment-106140193


We'll announce it on the project/devel mailing lists, as well as IRC/Twitter.


Please group expression at 460..462 using parentheses! It is not immediately clear what it does and whether it does what it intends to do.


@kedzie This plugin is breaking the build, because it is Java 7+. Is there a way to make this work on Java 6? Maybe some older version?


Should we still call it a CropWidget if it is QObject derived?


                                                                                Afair Debian 7 comes with 24.3                                                                                From: reactormonkSent: poniedziałek, 22 czerwca 2015 17:08To: yuutayamada/nim-modeReply To: yuutayamada/nim-modeCc: Jędrzej NowakSubject: Re: [nim-mode] eval-and-compile -> eval-when-compile (edbc699)How many people still used emacs <= 24.3?

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


chmod 700 "$MNTDIR/root/.ssh"
chmod 600 "$MNTDIR/root/.ssh/authorized_keys"


Maybe not, the class represents the vtk boxwidget ... Any name suggestions?


Spaces before and after =, < in for loop please.


I m reading this one with mobile phone, just need look it from my pc. I
answer later
22 июня 2015 г. 19:29 пользователь "perthdave" <notifications@github.com>
написал:

> First off, this .ino is fantastic. The only sketch of it's type that would
> work with my Teensy 3.1 straight off the bat.
>
> I'm hoping to restart my scan from 5900 onwards until I can properly
> capture the correct code to use.
>
> Does anyone know what variables I need to adjust in the code for this to
> work?
>
> Thanks in advance,
>
> David
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/knoy/iCloudHacker/commit/d5c7e6f4c1d5233d43586c42c4934eab6c5efd08#commitcomment-11796562>
> .
>


As above


Looks great, now try a different approach

Try to describe the interaction and rules in terms of Features and Scenarios:

```
Feature: Start Game

*As a* Player
*I want to* start a game
*so that I* can play and have fun

  Scenario: Successfully start game
    *given* I am on the home screen
    *and* a game is not in progress
    *when* I click the `New Game` button
    *then* I should see `Game in Progress`
    *and* all cells should be in a unchecked state

```

Have you ever used this type of user story format, the scenarios are not as important as the user story feature description, it is a good way to break down ux/ui to explicit expectations.

Try a couple, mainly around the user selecting a cell and the possible scenarios, you do have to get detailed with given when then, you can just list out the scenarios as bullets

Feature: ....

Scenario: ...
Scenario: ...
Scenario: ...

Make sense?


This needs to be ``-T`` to match the API settled on above.


indeed :)


Just FTR: this is invalid and will not produce a `main.js` file, as `include` values must express absolute paths: http://webpack.github.io/docs/configuration.html#module-loaders


No. Duff's device?


I know, but this file in particular has been manually edited in many projects.


That seems painful ... Is the report call plugged before each request (before_gateway), or only when we fetch info from Janitor?


Hey man, just a quick question but why is Farmbot being removed?


Amended, thanks for the spot.


Not tested on cuda.  It does work on CPU and OpenCL however.


Not sure if we should be including the built framework bundles. Seems like other Carthage based repos depend on users to run "carthage bootstrap".


great, that'd be perfect!

On Mon, Jun 22, 2015 at 11:37 AM, Alex Kyllo <notifications@github.com>
wrote:

> What you describe is the default bootstrap behavior, but I can add a few
> lines of JS to make the menu collapse when you click on a link:
> http://stackoverflow.com/questions/21203111/bootstrap-3-collapsed-menu-doesnt-close-on-click
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ScienceCommons/api/commit/246e4be7710d3a05a28b3afddb666ebe9a6905da#commitcomment-11796730>
> .
>


:ok: :1234:


Well look the differences for yourself they basically don't show the same info at all.
https://scrutinizer-ci.com/g/doctrine/migrations/
https://coveralls.io/r/doctrine/migrations


sh:nodeShape ist soweit ich verstanden habe dazu gedacht, shapes an einzelne Instanzen, oder eben Knoten zu hängen.


Ah, I see now, sorry.


Forgot, but this closes #62.


@mihailik I did so today, thanks for the heads up.


Ah yes, that was a convenience I added for ourselves from our fork. Makes sense it shouldn't be in the canonical repo. I will fix & resubmit.


Seems like `copyTpl` supports globbing, so could just use that for all the files, so it becomes one line, then rename `_package.json`. We can just be lazy and use `copyTpl` for everything.


Ja, shapes haben eine andere Bedeutung. rdfs und owl ist mehr für Inferenz, Shapes für ein Closed-World szenario - soweit mein momentanes Verständnis.


oops, you're right.  I'll get rid of the new FK constraint too


Now all is well. Thanks @mark for the very fast response! :)

On Mon, Jun 22, 2015 at 6:22 PM, Mark Goodrich <notifications@github.com>
wrote:

> Aha! It looks like my change got blown away in a merge--I have
> re-commited now.
>
> Mark
>
>
> On 06/22/2015 11:13 AM, dkayiwa wrote:
> > This looks like pointing to 1.6
> >
> https://github.com/openmrs/openmrs-distro-referenceapplication/blob/master/pom.xml
> >
> > On Mon, Jun 22, 2015 at 5:56 PM, Mark Goodrich <notifications@github.com
> >
> > wrote:
> >
> > > Thanks for the heads up @dkayiwa <https://github.com/dkayiwa>. It
> looks
> > > like although I updated the pom for the distro-referenceapp project
> > to use
> > > 1.7-SNAPSHOT of the uicommons module, on devtest01 it's still only
> > version
> > > 1.6 that is being installed. Any idea what I am doing wrong? I upgraded
> > > appui the same way, and that appeared to work.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> > <
> https://github.com/openmrs/openmrs-module-registrationapp/commit/5d6eba210fea73e2629612bbe15d30c8a0776b26#commitcomment-11795761
> >
> > > .
> > >
> >
> >
> >
> > --
> > If we keep uppermost in our minds the unkind and unjust acts of others,
> we
> > shall find it impossible to love them as Christ has loved us; but if our
> > thoughts dwell upon the wondrous love and pity of Christ for us, the same
> > spirit will flow out to others.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/openmrs/openmrs-module-registrationapp/commit/5d6eba210fea73e2629612bbe15d30c8a0776b26#commitcomment-11796178
> >.
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-registrationapp/commit/5d6eba210fea73e2629612bbe15d30c8a0776b26#commitcomment-11796394>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


Is that the transactionStatus from the KB transaction (where UNKNOWN does exist) , or the PaymentPluginStatus  (where UNKNOWN does NOT exist)?


At least one of the return examples should be `k8s-guestbook-node-3-container-8dcdfec8` to match the sample inputs.


:+1: Huge thanks for this update!


No, we can figure it out in a follow up, just seemed odd, maybe object as it is now QObject derived and is a general object rather than a widget.


Actually from the libvirt specifications:

```relaxng
<define name="smbios">
    <element name="smbios">
      <attribute name="mode">
        <choice>
          <value>emulate</value>
          <value>host</value>
          <value>sysinfo</value>
        </choice>
      </attribute>
      <empty/>
    </element>
  </define>
```

mode can have several values (`emulate`, `host` or `sysinfo`). You should reflect that as well. It's not linked to `sysinfo` section as if you set `mode` to `host` the `sysinfo` section will be ignored.


Ich würde das so sehen:
    dda:Requirement sh:classShape dda:RequirementShape .


This should be version independent. The only thing is to find a suitable gif image. Not every gif image will give successful results.


I forgot to link this, but this closes #64.


Vielleicht einfacher mit einer regexp? Fehlt aber noch in dem shacl draft.


This looks good :)

Be sure to respect:

```
  <!--
      System information specification:
       Placeholder for system specific informations likes the ones
       contained in the SMBIOS area.
       Only a limited subset of entries can be modified there, so we
       fully enumerate each case here.
       The DMTF spec doesn't specify any string subset, just 0 terminated
       byte strings, but better be safe and restrict at least the names
       to avoid problems with space normalization in attribute values,
       the value is kept as the element body for maximum flexibility.
       A priori we allow only type 0 and type 1 string updates
      -->
  <define name="sysinfo">
    <element name="sysinfo">
      <attribute name="type">
        <value>smbios</value>
      </attribute>
      <interleave>
        <optional>
          <element name="bios">
            <oneOrMore>
              <element name="entry">
                <attribute name="name">
                  <ref name="sysinfo-bios-name"/>
                </attribute>
                <ref name="sysinfo-value"/>
              </element>
            </oneOrMore>
          </element>
        </optional>
        <optional>
          <element name="system">
            <oneOrMore>
              <element name="entry">
                <attribute name="name">
                  <ref name="sysinfo-system-name"/>
                </attribute>
                <ref name="sysinfo-value"/>
              </element>
            </oneOrMore>
          </element>
        </optional>
        <zeroOrMore>
          <element name="baseBoard">
            <oneOrMore>
              <element name="entry">
                <attribute name="name">
                  <ref name="sysinfo-baseBoard-name"/>
                </attribute>
                <ref name="sysinfo-value"/>
              </element>
            </oneOrMore>
          </element>
        </zeroOrMore>
      </interleave>
    </element>
  </define>

  <define name="sysinfo-bios-name">
    <choice>
      <value>vendor</value>
      <value>version</value>
      <value>date</value>
      <value>release</value>
    </choice>
  </define>

  <define name="sysinfo-system-name">
    <choice>
      <value>manufacturer</value>
      <value>product</value>
      <value>version</value>
      <value>serial</value>
      <value>uuid</value>
      <value>sku</value>
      <value>family</value>
    </choice>
  </define>

  <define name="sysinfo-baseBoard-name">
    <choice>
      <value>manufacturer</value>
      <value>product</value>
      <value>version</value>
      <value>serial</value>
      <value>asset</value>
      <value>location</value>
    </choice>
  </define>

  <define name="sysinfo-value">
    <data type="string">
      <param name='pattern'>[a-zA-Z0-9/\-_\. \(\)]+</param>
    </data>
  </define>

  <define name="smbios">
    <element name="smbios">
      <attribute name="mode">
        <choice>
          <value>emulate</value>
          <value>host</value>
          <value>sysinfo</value>
        </choice>
      </attribute>
      <empty/>
    </element>
  </define>
```


Can you provide an example where the bug happens? I would like to write a test for it.


This should be `go-multiplex`


This should also be `go-multiplex`


Damn I didn't notice that but now I remember that I had that on the experimental branch and forgot about it.


Fixed now, thanks again!


@enkore sounds great, will do when I have a time for it


sh:datatype gibt es gar nicht soweit ich weiß. sh:valueType ist für literale gedacht.


Yes I could use some help since the images in the pie-chart should not be on the master branch


Wonder if all functions here should get a ProjectInfo.


when you invert the order of the snapshots he will always pick the later one even of the time of the snapshot is chronologically before the first one - is this intended?


Guter Punkt: maxCount per language braucht man da.


The last line is useless, `npm install` automatically runs prepublish.


Consider renaming this `CordovaProject` - IMO it's more easily searchable online.


No, I am new to Git and I was trying to add just that commit. Bear with me as it is all a bit complicated at the moment.

From: Gecko Web [mailto:notifications@github.com]
Sent: 22 June 2015 16:06
To: Gecko-Web/meta
Cc: Puzbie
Subject: Re: [meta] Add '.idea' ignore rule (43653c5)

Sorry, I don't understand ... Do you want I rename this commit?

—
Reply to this email directly or view it on GitHub <https://github.com/Gecko-Web/meta/commit/43653c5c8fe7e5f9b631a4449e6653ae1fbc5fe2#commitcomment-11796024> .  <https://github.com/notifications/beacon/AGv1yqxh2e14uFdYZjUyedxxGnPcuOa_ks5oWBvpgaJpZM4FIxXX.gif>


Could be the simplest solution to delete everything related to the images, since there is not much valueale code anyway. I would also rethink the design of the XValue class so I think not even that is needed.


Grant me permissions to the repo I'll do it later...

On Mon, Jun 22, 2015 at 6:52 PM, Philipp Jahoda <notifications@github.com>
wrote:

> Yes I could use some help since the images in the pie-chart should not be
> on the master branch
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11797039>
> .
>


Good point.


ok I missed that.  I should have checked more closely


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2163 of 2213 tests passed)

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


Instead of an array. An object of artifacts might be useful. {"ipa" : "PathtoIpa", "app" : "PathToApp"} so and so forth.


Made the shadow around the collection images grey with more blur, added a second border between collections and groups


Spacing.  Should be consistent.

Also the comments could be removed.


The changes looked good.  The only issue is with the tab/whitespace in the documentation.


You can eventually throw away `money-tree` too as it's supported by `BTC::Keychain`.


Further enhancements added to Open/Copy URI word selection. Based on http://tools.ietf.org/html/rfc3986#section-2


Si se puede. Copié los sprites, no tengo un editor decente de imágenes (como podrás apreciar en el splash screen).


why 38429 ?


Or you could reverse commit 129ddcb on master. It will create a reverse
commit...

On Mon, Jun 22, 2015 at 6:53 PM, Philipp Jahoda <notifications@github.com>
wrote:

> Could be the simplest solution to delete everything related to the images,
> since there is not much valueale code anyway. I would also rethink the
> design of the XValue class so I think not even that is needed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11797073>
> .
>


Fix program counter


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2188 of 2213 tests passed)

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


Bzjjzjx


Bzbx


After closer inspection, turns out this is not an injected class name emulation.


great!
can you add this to the changes file? it's a good one for the changelog


OK. Let it be! Yes, it doesn't affect the whole performance and we get the same 5 sec. It is better than 20 sec anyway...
@garyrussell, your turn! How to go ahead with this change?


TeamCity YaccConstructor :: YC.Core General Build [Build 454](http://teamcity.codebetter.com/viewLog.html?buildId=193800&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Not a big deal :) At least you try instead of asking asking and asking again :p


@borshop retry


i won't insist on this now, but: from the style guide:

```
Names for `Either` values start with an `e` (example: `eSession`).
Analogously, `Maybe` value names start with an `m`.
```


Do you think the "new EventStoreStopMojo().executeGoal();" is really necessary? The "stop" goal will always be executed in the "post-integration-test" phase no matter if "start" was successful or has thrown an exception. Therefore I think we can remove the IOException catch block and let the other one throw a MojoExecutionException.


TeamCity YaccConstructor :: YC.Core General Build [Build 454](http://teamcity.codebetter.com/viewLog.html?buildId=193800&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:18


you could also use uri parsing from package `uri-bytestring` here (we are already using that).  that would cover more consistency rules with possibly less code.


TeamCity YaccConstructor :: YC.Core General Build [Build 455](http://teamcity.codebetter.com/viewLog.html?buildId=193802&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Should a "failed" return code really just create a warning in the log?


two times in a row "develop your app...develop an app". Maybe change the second develop to submit, so "...a structure for you to submit an app...".


Maybe make these numbers different so that we can verify the two axes are going the right place?


Yep, i thought about multiline option. And you've made it ;)


TeamCity YaccConstructor :: YC.Core General Build [Build 455](http://teamcity.codebetter.com/viewLog.html?buildId=193802&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:17


![about page](https://cloud.githubusercontent.com/assets/12994345/8286233/fbc31d70-193a-11e5-8f11-69c9cc5b50a2.png)
![Uploading contact page.png…]()
![Uploading home page.png…]()
![Uploading login page.png…]()
![Uploading plan your trip pge.png…]()
![Uploading signup page.png…]()


...that are relevant to the app content...you forgot "the" :)


because it's broken at the moment, any time it tries to move, it throws up a runtime error


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 76](https://teamcity.labkey.org/viewLog.html?buildId=256148&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:15:07
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSUpgradeTest: <no details avaliable>

```


Yup, will fix. Also noticed I've used the wrong `readme.md`.


REGRESSION BUG: UnescapeDataString removes // from UNC paths - it breaks loading Nlog.config from network shares (UNC becomes relative path)


The rest looks pretty good :)


missed in changelog:
Fix green overlay xbox, hdmi-splitter, some older TV


`var tpl = {`


The `Extensions` field is deprecated in favor of `default-extensions`. It does not prevent cabal from picking a version, it's only used to enable extensions in the package's modules. Our (hackage trustees) goal is for releases to either have a valid install plan or for cabal to reject them based on constraints rather than causing a compilation failure. This sometimes lets cabal pick an older version that does compile successfully, and otherwise gives a more direct error message instead of a compilation failure.


Is it necessary to change this signature? This is somewhat a public API, as external extensions can write handlers, so we shouldn't change API if it's not required. I don't think any impl of this would want to do anything with the operationResponse besides call getResponseNode().


Can we could have saved one or two calls to `cs` by using `SBS` in `SsoToken` instead of `ST`.  Didn't count them properly, though.


Why? Possible merge mistake?


License header please.


why isn't the transaction throwing that error?


If configured, it's called before each payment request and (as of https://github.com/killbill/killbill-cybersource-plugin/commit/31154217c0c8d9a0e48b20541ad493fb42013991) as part of each get request (for `UNKNOWN` transactions only).


But that still doesn't fix this issue, right?

```
import "DPI" function string fn1 (input string str1);
   import "DPI" function string fn2 (input string str2);
   import "DPI" function string fn3 (input string str3);
```


Fixes issue #10


"am" => "an"
Typo ;-)


I _thought_ it should have just picked up on it from your local `deploy.rb`...I'll have to tinker with this then


I just realized this affects `vim-mode`'s search input as well. Using `div[is=ex-command-mode-input]` instead of `.command-mode-input` should fix that.


Why moduleWithDependents? I think this is too much, and the user will be surprised by the number of suggested variants.


Oh gross, is yaml interpreting those as arrays of single-item objects? In that case, change to:

      expect_cmd_vel:
        linear: { x: 0.6, y: 0, z: 0 }
        angular: { x: 0, y: 0, z: 0.9 }


Why do you need the extra name filtering here?


ohhh, ya revise el archivo "gulp_watch.js", veo que si entrega la informacion necesaria porque el ambito es uno solo.

Gracias Victor!


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


nitpick but please always order names here by last name and always keep "Copyright (C)"


Ok!
I'm new to Git too, so I just do not really help you... ;)
I think pull requests merge branches. So you can't add only one commit.
Why not accept the pull request directly? Are there any problems in my other changes?


You're right - it should be `UNDEFINED`. I'm still working on an end-to-end test (quite tricky to simulate...).


My type in database is

```
locales text[] DEFAULT ARRAY[]::text[],
```

So I guess it's because it's not jsonb


That's good to know. I haven't been maintaining this plugin much because I use Atom a lot more than vim these days. If you're interested in submitting a PR that somehow adds back these bindings in a way that works with `vim-alt-mappings`, I'd probably merge it.


ordered by last name, just like you did so this is just a reminder


This looks great, nice and simple, thanks.


The comment doesn't seem to match the code.


I happened to be reading some of these writeups and couldn't help but notice... isn't this link in the wrong repo? It's for the 2013 edition of ructf-quals.


Oh that was a debug thing (UI driven design;) thanks


this is wrong, what if another desktop has a different panel size or multiple panels


Ah.. that indentation issue happens if I have the above lines outside a `module`.


@justinmk I just noticed a difference between the two. Not sure if the behavior is clearly documented. Try this:

- `vim -Nu NONE +'set nu|help :b'`
    - `b#` twice will make help split have line numbers unlike `ctrl-^`


module.exports allows that code to be called by any other file that requires that script file, which is pretty awesome. In this case, I didn't actually need to do that.


defaq i do


This only adds a subset of the functionality requested in #2


@rgladwell it now lives on master, so i assume this is a final commit. As i mentioned earlier, you should leave 1.5 in Luna. If you will do that, and this is temporary change, then sorry for spamming. :)


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1160 of 1160 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)


:+1: !


sys is unused


TeamCity YaccConstructor :: YC.Core General Build [Build 456](http://teamcity.codebetter.com/viewLog.html?buildId=193804&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


@kyleswartley I would remove `intern` from the `groupId` and package structure.


@juandlizarazo : Hola Juan, tengo un problema en el cuarto punto, y es que la gráfica de la onda propagada me está dando con un par de diferencias: 1. El patrón es un poco diferente, 2 los colores son muy diferentes.
La primera sospecho que puede ser por la construcción del círculo, la segunda no tengo idea.
Alguna sugerencia?
Gracias :smile:


Why is medium_dark_tabset_background.png bright yellow now?

![screen shot 2015-06-22 at 11 13 34 am](https://cloud.githubusercontent.com/assets/414735/8286543/ca7266ec-18cf-11e5-90b7-4c43fae94acf.png)


https://packages.debian.org/jessie/emacs24 nope. So screw `<= 24.3`.


Well I'm going to be home in about 4 hours, let me know if you want me to
change history and remove that merge into a side branch

On Mon, Jun 22, 2015 at 6:55 PM, Daniel Cohen Gindi <danielgindi@gmail.com>
wrote:

> Or you could reverse commit 129ddcb on master. It will create a reverse
> commit...
>
> On Mon, Jun 22, 2015 at 6:53 PM, Philipp Jahoda <notifications@github.com>
> wrote:
>
>> Could be the simplest solution to delete everything related to the
>> images, since there is not much valueale code anyway. I would also rethink
>> the design of the XValue class so I think not even that is needed.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11797073>
>> .
>>
>
>


sys is unused import


The failsafe plugin or takari testing plugin? Either way I'm not sure.


Another similar problem can happen if you return a card to a private zone (hand, library) and then play a different copy of that card.


Actually, this solution had less to do with trust, and more to do with having a board I know the answer to for testing against so I can be sure that its working. I'm planning on refactoring the code once I know its working.


use `splitMember`?

https://hackage.haskell.org/package/containers-0.5.6.3/docs/Data-Set.html#v:splitMember

meh.  possibly not that big an improvement.


CSS3 Transitions not supported by: IE (6,7,8,9)


So this means the docs are no longer included in the distribution zip? Are Edson and Mark aware of that?


add empty line.


kA was hier los ist...


CSS outline not supported by: IE (6,7)


Because original class may be library, or in other module (different from extension function from where renaming was invoked).

Do you think we should restrict scope by only current module? In this case, result of refactoring will differ depending on invocation place.


Maybei it would be better in general to create the corresponding code like it is done for the Backend-Module where one can reinitialize the index queue instead of using the same methods as document editing (wouldn't be that problematic regarding changes then). On the other hand, I don't think there will be too many changes here after 7.4


This is not the panel size (that is handled by [availableGeometry](http://doc.qt.io/qt-5/qscreen.html#availableGeometry-prop)). This is the height of the window decoration/title bar, which is the same across all desktops. It can be detected using `QApplication::style()->pixelMetric(QStyle::PM_TitleBarHeight)`, but that requires QtWidgets and seems to freeze the compositor.


please don't hanging indent like this, bug that we don't have a lint rule to catch it


Sorry about this faulty commit. The change in BlockGrass.java is erroneous.


Why did you change these?


Comment in Processing.py in line 294 contains BearRunner


Okay, I have added you :-)
What I'd like to have is everything I did so far in the master branch to stay on the master branch, only the piechart-image thing in a separate branch (can be exerimental)


This is not a change, but rather, a reverting of the previous faulty change to BlockGrass.java.


Takari testing plugin.


@kyleswartley As the number of dependencies grow, consider creating properties for the `groupId`, `artifactId`, and `version` in a `<properties>` section in the `pom.xml`.  Then reference the properties in the `<dependencies>` section.  This way you don't have to repeat groupId strings and can change things in one place.


Someone keeps adding it back. Drives me nuts because things like git won't stop complaining about it.


Isn't this useful?


So the problem is that this line is returning nil, I just have no idea why.


True.  It wasn't seeing `self.tableView` that was confusing.  I've seen that elsewhere too.  The confusing part is that I wouldn't know without a few extra clicks that I actually am a TableViewController b/c I'm not directly extending it.  That's where I was thinking it would be nice if the super class could handle that pass off.  Not a huge deal though.


No need to default it to an empty object, extend affords this case, and will degenerate to just shallow copying the object literal you have in mind.  Also by way of passing, realize that there are to paths in xtend:
```
var extend = require('xtend');
var extendInto = require('xtend/mutable');
```
The former does not mutate the first arg, rather shallow copying into a newly allocated empty object.  However if the use case is to create an object literal as the first arg, you may as well use `extendInto` rather than allocate another object and then throw away your literal. Only matters in hot paths... of which this is probably going to be one.


NP :-) I'll do it later and let you know when it's done :-)

On Mon, Jun 22, 2015 at 7:19 PM, Philipp Jahoda <notifications@github.com>
wrote:

> Okay, I have added you :-)
> What I'd like to have is everything I did so far in the master branch to
> stay on the master branch, only the piechart-image thing in a separate
> branch (can be exerimental)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/291685e8e2e4fa08b94e08feb656e3b3a54a585c#commitcomment-11797631>
> .
>


You are right, this is redundant.


Except if you need to access streams. I might have misunderstood @aloubyansky when he told me that he'd rather have the stream handling to be ready in BaseOperationCommand.


Uh, when was this decided?


why is the callback optional? we generally consider it to be an anti-pattern to afford optional callbacks, forcing edges to inject their own explicit noop functions rather than pepper such branching all over the place.


@mperham what you think may be we report about `$CELLULOID_DEBUG`? or it's sidekiq problem? I can not understand until the end this.


Oops. There should be "Members which are extension to the same receiver class"


Ouch. That's insane. Thanks for that info, I will have to change some scripts now...


That makes sense.  I didn't get to that level of testing it so I didn't get a chance to see that.


I'll add in the Copyright part. Do you want most recent year first (my name first) or oldest year first (your name first)?


Please Use English Font Name: "Microsoft Yahei".


I'm confused by `unchecked_insert`. It takes the param by rvalue ref, but doesn't seem to need to. In fact, The last line of the function moves the value into another variable and does nothing to to (to make sure the caller gets a moved-out value on return, I'm guessing). Why not take the var by `const&`?


Finally, if you take a look at how `TChannel#requestOptions` is implemented (and the git history of how we got there), you'll see we chose to simply inline the extension rather than use any extend-foo module due to the hot-path nature of request creation.


@kyleswartley Even though the actual JSON may be using underscores instead of camel casing, I would use camel case for the properties on this POJO (Plain old Java Object)


I suppose that you should use English Variable name.


TeamCity YaccConstructor :: YC.Core General Build [Build 456](http://teamcity.codebetter.com/viewLog.html?buildId=193804&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed. Process finished with exit code UnexpectedError (-100).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies)   at FSI_0002.Core.commonConfig@178-12.Invoke(Unit arg10@) in Z:\Builds\wor... Build time: 00:10:50


nice! :+1:


No PM_TitleBarHeight is the title bar height for child windows on MDI applications, docks and toolbars of the applications not the desktop environment. Qt know nothing about window managers.


bracket in new line


We were talking about it last week on `dev-mdn@`. There was a [voting round](https://etherpad.mozilla.org/voting-for-new-youshoulduse-name) and Discord won.


Besides each style can change that value and you are not guaranteed it will always be 50.


Fix, amend, force push, comment :)


Fixes #217


Why is this a logical `or` rather than a conditional?


The `MetaView` implementation takes ownership. This abstract interface has to accommodate the needs of both derived classes. I agree, its not quite ideal. I have yet to think of a better solution but this is a leap forward compared to the previous code. the `unchecked` line of functions is not meant to be called by users, its needed for the implementation of `MetaView`.


Yep, it's some kind of legacy snippets, needs to be edit.


Why did you remove it again? It was quite good and actually usefull for new contributors? It's so good I might actually add it back in again, although into some doc directory.


Good to know! Thanks dude!


Just for the future:
it is better to replace the hardcoded number with `date('Y')`.


Oops, my boss will not like it, thanks :D


Am besten man löscht mein commit und reversed...


ack


That's going to require an explanation, I don't really know what you mean.


It is the third check for errors in a row.


relates to Issue #56.


Will try it asap, thanks!


怪我咯╭(╯ε╰)╮


Oh.


<h1 style="color:red;">怪我咯╭(╯ε╰)╮<h1>


This breaks the code. `proc` must be set during initialization.


What about the wide character versions?


@Mister-v
&= &
in html and xml


Take a look at this style guide from Google:  http://google.github.io/styleguide/javaguide.html
We may want to look into adding Checkstyle (http://checkstyle.sourceforge.net/) with a basic set of checks to automatically enforce certain standards.


@AdSchellevis brackets missing and indent seems off afterwards. are you sure this correct?


not ruined, probably just gave you a different perspective