Меня смущает использование термина "БЭМ-сущность" в определении Модификатора. То, что модификатор это одна из БЭМ-сущностей есть в следующем определении. Определения блока или элемента не прибегают к использованию этого термина, хотя тоже являются БЭМ-сущностями.
Термины "свойство" или "признак" больше подходят для передачи смысла модификатора. Термин "БЭМ-сущность" указывает лишь на отношение к некой группе.


Why are you using `new FormValidation.CheckMethod(…) { … }.check()` when you could just move the code into `doCheckVersion(…)` directly, similar to the `null` check?


Done

Bobby Brennan
bobby.brennan@gmail.com

On Sun, Jun 14, 2015 at 11:40 PM, Ivan Goncharov <notifications@github.com>
wrote:

> Minor version Y (x.Y.z | x > 0) MUST be incremented if new, backwards
> compatible functionality is introduced to the public API.
> So it should be '1.1.0'
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lucybot/api-spec-converter/commit/97c4a9fc558170398f2305c3656de2aee3f8a41f#commitcomment-11671509>
> .
>


Looks good to me.


@DeicPro didnt think of that, thats awesome! :)


Is the ie8 shim available on the CDN?


makes sense, done

Bobby Brennan
bobby.brennan@gmail.com

On Sun, Jun 14, 2015 at 10:05 PM, Ivan Goncharov <notifications@github.com>
wrote:

> It could be bad you exposing base type for user as fully valid type.
> So it could be used even as argument to CLI.
> Can you expose it as separate variable?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lucybot/api-spec-converter/commit/f85825518fa1de4ce32b5d52a52c709c2b2e131b#commitcomment-11671185>
> .
>


@emmenko Because some of these are no longer a single command and we want them to stay readable


I am confused by this change. Unless I am mistaken, we are using the libunwind from www.nongnu.org for FreeBSD like we do for Linux. And that unwinding library supports context pointers. So why are we blocking them here?


Still unrealistically hoping you one day return and resolve some of the issues I reported. :(


Also works: ```console.log("Fetching: %s", evt.target);```
#goodtoknow


@Bertk Could you provide a PR please to merge and test this C++11 extensions.


wow


I read all the detailed information, thanks for that. The extrange thing is that we are using other cookies in our requests without problems (don't sure what's the path used for JESSIONID and why it doesn't work).

Disabling cookie forwarding in requests will break the compatibilitty of several users and i would prefer to send the cookies by default and include a config option in ujiCrypto.conf in order to completely disable them if you find this mandatory.


Sorry to bug you again, but that line was commented out, previously.


Hi Lamyra, I would suggest to look at this tutorial: https://github.com/node-opcua/node-opcua-htmlpanel

Our Approach is specified to our usecase: Showcase Mi5 (http://projectmi5.com/documentation/hmi-software-architecture/) You would need to make a lot of changes in order to get just a website and a variable.
Best regards


_listerner_ is not very explicit about what it does. Something like ```function requestLoaded``` would be more obvious to me.


This can be removed whenever a new version of crytonite is released. See vincenthz/cryptonite#12.


Cool


As discussed in #64, I've reverted the master branch to the latest stable version, would you please consider this reversion for these new commits.


Is this still used / needed? @corot  @jihoonl
I'm currently updating the yujin navi stack repo..


This is actually a question I have. Should we semi-column the end of functions?


thx @Aaike


Right


https://github.com/ros-planning/navigation/pull/340 has been merged. So it is not necessary anymore.


@jonthornton you reverted the `stopScrollPropagation`feature erroneously (I think) in this commit.


nice


Which is reason of removing this line Cowboy's issue or S3 client support?


@srpearce The POST request expects ONLY usage_id, we should remove `It can also include other parameters` because it does not accept any other parameter.


Да, да как раз я хотел поместить в хеадер и случайно забыл убрать лишний тег


asdadsdsa


There's already a private keyword on Line 66


So, maybe `getScrollTop ` is no needed? Just get `pageYOffset ` when you need? Minus one function.


Which is reason of modifying this line Cowboy's issue or S3 client support or other?


Does normalize use vendor-prefixes to target only specific browser? Because autoprefixer tends to strip them away.


Хватит с меня ограничений. Пора убирать их нафиг!


and this


I don't have a strong opinion about that, it only hurts my integrated JSHint :)


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 547](http://build.prxm.it:8811/viewLog.html?buildId=3012&buildTypeId=Jarvis_DocumentStore_CI) is now running


@ndousse
Don't you think we could then get ride of the local variable rc_check ?

blame for missing parameter comment in the struct !! :)


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 547](http://build.prxm.it:8811/viewLog.html?buildId=3012&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:41


"edited"


Agreed, fixed in https://github.com/scopyleft/PJLRenseignement/commit/54277ace02a4652559c0ef9f0620fa628f3448f2


hmmm


still hmmm


A unicast on this channel _directed at this specific peer_ seems to warrant an error or at least debug message.


I said you. Don't make difference if you make changes here.


Is it only because of the repetition of `--compilers js:babel/register --recursive` ?


Yes thats similar to why I changed the myriadState from Zookeeper to State.
There were two reasons, one since there is an appropriate abstraction State
why not use it and the MyriadState should not care where its being actually
stored; and two I wanted to use Memory State storage for Unit testing
instead of zookeeper so it was necessary to make the change.

On Sun, Jun 14, 2015 at 11:10 PM, Kannan Rajah <notifications@github.com>
wrote:

> Suggest calling this as stateStore instead of state. I don't know why
> Mesos chose that name because we are storing state in a state store such as
> zookeeper.
>
> I had to make this change to inject mock implementation for unit testing.
> Please take a look at this: mesos#112
> <https://github.com/mesos/myriad/pull/112>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pdread100/myriad/commit/1303126c1baab84677b4d845bfb588ef2c7ba451#commitcomment-11672867>
> .
>


[11:00] user@user-VirtualBox ~ > docker run --rm -it globidocker/term-picker-release item1 item2 item3  [127]
Unable to find image 'globidocker/term-picker-release:latest' locally
latest: Pulling from globidocker/term-picker-release

cf2616975b4a: Pull complete
6ce2e90b0bc7: Pull complete
8c2e06607696: Pull complete
5720a883b190: Pull complete
db0990e151fd: Already exists
Digest: sha256:1e49c819e15aad802afbb567545f091426594e46db44ec734e8003c9969cbc0e
Status: Downloaded newer image for globidocker/term-picker-release:latest
FATA[0004] Error response from daemon: Cannot start container f47fe52b1d8baa93ec7178705479717880abc2eafa9946e09a127e7b4967ecd7: [8] System error: mountpoint for devices not found


Change this copyright to you


Set yourself as the owner of this file.


il ne manque pas un `}` ici ?


ah, ça marche maintenant, avec les deux commandes


Just 2 very minor comments. I'll merge this patch as soon as you will apply them


굿굿


@chrisseaton Should both comments be updated?


:+1: :dancer:


essaie de définir une largeur pour `.openclassroom` et `.livementor` ! Pense aux pourcentages, peut-être :)


Thanks. :)


还有人吗？我在zinnia官网下载不了这些库和头文件呢


merge


As we dont need to inject the reactor for testability (as we do in amp based libs), IMO it would be best to rely on the global one, so we dont need to mix up this API with technically required objects.


Can I ask how you changed the variables throughout the script and incorporated the update? The diff look so clean. I tried my hand at updating the variables manually (finding and replacing 'f' with 'h' and so on) and had quite a bit of trouble.


Note that you can hack the template string a bit to handle i18n! http://jaysoo.ca/2014/03/20/i18n-with-es6-template-strings/ #goodtoknow


@kangaroo I'm off to the hospital. Can you expedite and explain this one? Pretty please? :)


Maybe add a oldname='phone_prefix' to ease migration.


rick wel zeggen wat ge doet dankje :palm_tree:


Yes this causes a parser error - should not of been merged.


哈哈


Are you sure this is the only usage?


It's a new field (1 week ago) :)

On 15/06/15 11:33, Thibault Delavallée wrote:
>
> Maybe add a oldname='phone_prefix' to ease migration.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/odoo/odoo/commit/237564626e97fb0a553732a2fcf5d3cfafd44c3f#commitcomment-11676371>.
>


Crazy R&D !


Well… we'll probably anyway just Generator-ify everything here later… so no need for Reactor then.


> It sounds like minetest.show_formspec(...) is not what I want since I want to reuse my node's callbacks.

Speaking about the example I gave, I usually just call ```default.sort_inv(player,formname,fields,pos)``` inside ```on_receive_fields``` of a node.
I.e. I always use global function (```default.sort_inv(player,formname,fields,pos)```) and every node ends up calling that function anyway.
That way I'm able to use node's ```on_receive_fields``` as a wrapper for ```default.sort_inv(player,formname,fields,pos)``` and always know that every if clause I've used will be checked regardless of the source of the field:

```
        on_receive_fields = function(pos, formname, fields, sender)
           if sender and sender:is_player() then
              default.sort_inv(sender,formname,fields)
           end
        end,
```

PS: I couldn't figure it out how to fake on_right_click with that func mentioned in wiki.
Could you elaborate?


why does the async branch depend on constants of `OldProc` aka `Symfony/Process`?


@racodond ? can we copy whats needed from the parent class?


Will do - right now we have issues with RGUI hanging completely on entering (or not accepting input). i had to revert to older codebase. see https://github.com/petrockblog/RetroPie-Setup/issues/863#issuecomment-111917463


Bear in mind that the intended use here is backward compatibility. Now, it would be *unusual* that backward compatibility works as expected when it entails dropping unicast messages, but we cannot rule out situations where it might. So we certainly shouldn't error, and we don't really want to spam the logs either.


What did you want to express with "D"? :)


not so bad


I will open a ticket with retroarch about this today - have been bisecting


Posssible alternate word to 'Dummy': 'Surrogate'


This needs to be done if the write fails too.


I don't see my snowcone anywhere -.-


goedzo rick +1 goeie titel


This is a JSON representation of a workflow ?


`mods1`, and therefore this line of code, is now dead.

Removing the line also makes the `mdef` parameter dead.

The method could then be renamed to `forceAnnotations` or something similar.

Perhaps it belongs more with the Symbol than in Typer.


这里是留言测试


@joeri mooi he


poep


dsfffdsfdsf


fdsfdsfdsfdsf


拼错了，导致图像显示不出来。。。应该是hackpad-privacy.png


Please remember to cleanup the package also from the for-15.05 branch.

argp-standalone is currently in two places in CC15.05 context, as it was added to trunk libs just before the branching:
https://dev.openwrt.org/browser/branches/chaos_calmer/package/libs/argp-standalone
https://github.com/openwrt/packages/tree/for-15.05/libs/argp-standalone


Hey @sureshr-hasvik ,

If that's the case, Perhaps before you doing the work of counting the IPs used by the VPC, NetUtils, etc, you could simple check if the zone is IPv6 enabled. You already have the DataCenterVO, it should be trivial.

From my point of view, deciding to do A or B based on a return being NULL, is not really nice. If in the future NULL is returned for another reason, it will drag the user to a path that is not really desired.


change it to "consult and follow..", comply feels so harsh


Thank you! I just got back to Hitchlog after one year. This was a very simple but important hotfix :) I just merged your commit, now you're a contributor to Hitchlog as well :+1: . Once all tests pass I will push it to master and deploy it so that these damn flags get displayed correctly :satisfied:


There's actually a context processor to select settings that should be added to the context, so you don't have to explictly add them to each view: https://github.com/mysociety/yournextmp-popit/blob/master/mysite/context_processors.py


PRs coming from master will not be accepted, as we've had trouble in the past with such PRs


Oops, you're right. Well spotted. I usually don't use `WeakReference` and instead rely on the caller to properly deregister from the controller/presenter.


Instead of this, all others are fixed. Can you please explain this?


@DiamondBond @hoholee12 i  will give rw permmisions to fast contribute :)


There is no `cprgFork` in cryptonite (yet: vincenthz/cryptonite#15), so this creates a new rng. That's why it adds the MonadIO constraint to `upgrade` and `upgradeE` below.


Do not merge this yet it needs some more work


Directive naming changed


Sure that this shouldn't be 'value   => "https://${server}:${port}/",' instead?


This can be removed as soon as a new crytonite version is released: vincenthz/cryptonite#12.


Spello, should this be olive


LGTM


Thanx for you answer.
I work with modbus tcp/ip not by opcua  i have developped all functions but the problem it's how have my hmi from browser


流程图太复杂了。对没有工科背景的志工来说，会很痛苦


> * Added config `subdef_ele` in conf/battle/items.conf to change bSubDefEle behavior.

that log is invalid.


done. test if can write


@cconcolato, according to the spec it must be `referenced_size` instead of `reference_size`.


Great. Have you already done some plots already @simonepigazzini ?


It might be better to get the site name from get_current_site from [the sites framework](https://docs.djangoproject.com/en/1.7/ref/contrib/sites/)...


what about the null check?


fails all the time with .oh-my-zsh/lib/misc.zsh:3: parse error near `then'


I did mean HTMLElement, document.body in the end is an HTMLElement, but I meant native APIs in general


fixed, thanks


It seemed implied that this was part of his configuration, along with the below. This is what confused me.


yes, I've written to you in skype...anyway a pyROOT is coming...


As much as the geek in me thinks it's cool to hide the binaries in source, I have to agree with Bertrand's email that it doesn't really put us in a better position, so I'd be in favor of a revert. I think it would be better to get the RAT exclusion mechanism working and exclude that way.


Added a space before the `]];` and it works for me.


_external-scripts.json_ ? Seems like above, when you were talking about files, you were using italics. Maybe `external-scripts.json`?


... through the GUI (under configuration) or you can...


`same`


GUI


Why are the other paths relative but this one not? *shrugs*


Can you keep the DOI please --- one click is better than a google search :-).


Me too! :^)


Windows.
Three prerequisites?


same here. Seems like there's a pattern for the `create` methods, to not use the most specific `Id` class?


You should check these files exist before attempting to open them.


@windkit I've understood and just remembered we took a measure of s3-clients as follows:
* https://github.com/leo-project/leo_gateway/blob/develop/include/leo_http.hrl#L59-L66


two different arrows. wat.


first proofread.


Hi
I am trying to build biber 2.1 for Debian, and lot of the tests fail due to differences in the sortinithash entries.

Are these values depending on *which* Unicode::Collate you are using? There are two availables, one from core perl, and one from CPAN (1.12).

Do you have a suggestion how to circumvent these problems, as I don't think they are serious.

Thanks


isn't this precondition guaranteed, based upon how the `Id.Flow` object is instantiated/constructed?


why global?


Saudades los loros nakonu ho natureza nia furak


请问这个变量名为什么是这样？


btw does it make sense to just use `Id.Program` instead of `ProgramRecord`?
Seems like both contain the same information (just one is completely flat - Strings, while other is embedded objects).


Yet another reply on the first thread


+++ Nick Wellnhofer [Jun 15 15 16:28 ]:
>   We should consider to use a binary search instead of a gperf hash for
>   entities. This would reduce the size of the binary considerably.

I'm open to considering it.  How would it work?  Is there a
way to generate code for the underlying data structure from
a simple list like our gperf file?

What about run-time performance?  Would it be a performance
hit relative to the hash table?  (Probably not a big deal if
it's slower, since entity lookups are not all that common,
but I'm still curious.)


just do a `namespaceClient.list().for`?


![console](https://cloud.githubusercontent.com/assets/7091609/8173328/17609e48-13a2-11e5-89b4-aacb05dd7912.png)
This is my console after runing the command, I'm using W8.1

What version are you using?

Another way to add this is go to: `Control Panel-> System and Security-> System -> Advanced system settings->Enviroment variables (under Advance Tab)` and edit the Path variable and add the IDE Path to the end


I think truncation of streams should be done in the TestManager#clear method.
The pattern here appears to be that we just check for existence of data, and do the clearing up in TestManager#clear.
Also, take a look on develop; I refactored this code a little bit.


Why do you clone the parameter and then assign the original to the clone? Why don't you just change the original?


Might need to check if we have any remaining VPP features in-flight that need to get merged first for a customer release.  I think we had some pending.  Otherwise, lgtm.


Wenn wir ganz genau sind müsste bei solchen Abkürzungen ja ein halbes Leerzeichen gesetzt werden. ;)


Usage examples : https://wiki.multitheftauto.com/wiki/User:Ccw


1667_SketchUpInspectorNewZone (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: OK (2196 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-99.23%25-yellow.svg)


This seems more inaccurate than what was here previously. "Full colors" is ambiguous -- the original only ever displayed 16 colors max, so this insinuates the default is to not display the proper image. Having "skip" in the message is misleading as well, and it's also not entirely true. (Isn't the undithering performed by blending two colors together? It's not skipping a step, it's doing something entirely different.)


@gyaresu
I'm sorry I have done no test before I merge your PR.

I think leave it as `__dirname` is better.
I would not force users write more program line.
If we change to `require path` library, users must write a line `var path = require('path');`
It is only a line, but I think users who is a beginner of Node.js may think it is troublesome.


You can make this shorter:
if ((paramType.GetFinalPointee() ?? paramType).Desugar().TryGetClass(out @class))


WiX is also used by a tool that is embedding the MSI transforms: msibuild.cmd. Building the installer with this patchset applied doesn't work for me because of that.


todo: take these three lines out.


So you can call it as `modulename` or `modulename.registerWith`.


@sgiehl In `getLanguage` for `French` this represents the sum of all French languages. There could be some `French - Belgium` (`fr-be`) and `French - Switzerland` (`fr-ch`). In this case, the segment should include all those language code.

Either
 * remove the `segment` from `getLanguage`
 * or better, if possible, rewrite it to include all language codes eg. `<segment>languageCode=@fr</segment>`


You may want to specify this via sauce args too: https://docs.saucelabs.com/reference/test-configuration/#maximum-test-duration

You can pass that via (the poorly named) [`browserOptions`](https://github.com/Polymer/web-component-tester/blob/master/runner/config.js#L68) object in WCT


The dramatic frame-rate improvement is mostly because some heavy debugging code was turned off; not an actual improvement technically. I temporarily turned off the portal rendering stubs which probably increased rendering speed by a few percent as well.

There were some changes which keep the threads from stopping and waiting as much; this should increase CPU usage as wait states aren't generally considered CPU usage. The more fluid activity of the threads means the GPU is being given more work to do. Ideally the GPU should be at 100% since there is no framerate cap on the rendering thread. (: But its queue is probably going dry so the GPU sits and waits for more work. This fits with the increase in GPU usage when the game is paused since the gameplay thread is also paused at this point, which frees up a lot of thread locks so the rendering thread has more opportunity to keep the GPU busy. This is strong evidence that the java side is the bottleneck on your hardware.

There seems to be something fishy happening with memory usage. TRCL is set up to not use more than 768MB of heap ram.  Java may allocate that whole chunk of memory even if the program isn't using all of it. On top of that, the JVM will use a bit more as there are also off-heap resources. But the weird thing is that I am also getting >1GB usage from the OS, which implies heavy off-heap usage or maybe even a memory leak on the native end.

I am getting POD vanishing as well but it has always been a result of the program crashing. Are you getting it following normal exits?


Consider
```
std::uint32_t
ConsensusImp::validationTimestamp (std::uint32 vt)
```
and have callers pass the return value of `NetworkOPs::getNetworkTimeNC ()` Then we can remove `netops_` from here.


the `segment` value is correct for `getLanguageCode` since each row represents a specific language.


No problem, thanks for the feedback!


This line is terrible!


This breaks the Vagrantfile if the Vagrantfile is not also updated to point to the new salt location.


invalid space between hostname and port.


See: https://github.com/memex-explorer/memex-explorer/blob/master/Vagrantfile#L47


done


This broke the case that I fixed in [this commit](https://github.com/core-plot/core-plot/commit/2d2acf7bd7fe1209fc7391fdd33c3490aa75075c)

After tapping a chart and interacting, other cells in collection view/table view are no longer selectable.


I'll revert those parts, but is there anything else we can do about the compiler warnings?


Instead of *Hotfix* I would say something like: *Update dependency to python-irc>=11.0 (backward incompatible change)". Also, for the sake of English typography (and because I'm already leaving a comment), there is no space before a colon (``:``) in English.


Hey,

Yeah, I need to give you the image file. I don't know where to upload it.

Matt Applegate, Ph.D.
Assistant Professor of English
Molloy College
http://mapplega.com

> On Jun 15, 2015, at 10:33 AM, Graham Higgins <notifications@github.com> wrote:
>
> I don't think this will work - let's go over static file hosting.
>
> —
> Reply to this email directly or view it on GitHub.
>


We're still doing `b` prefixes in 2015 ?


In the future, this is a good time to use interactive rebase to fixup a previous commit.  Don't sweat it on this one.


get


`if (m_journal.trace) m_journal.trace <<`...


Okay, I see that you fixed this in another commit.  Don't worry about it.


#1
RemoveTimestamps strips timing headers from a subtitles file.
RemoveItalics strips html elements like <i> from text a text file.


`void ConsensusImp::takePosition (int seq, SHAMap const& position)`


DesignDaysGridView (evanweaver) - Win64-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)


We're paying for a default construction when we don't have to. This should use `emplace` not `operator[]`.


재승이의 요구사항을 들어주기 위해 ClientThread와 PushThread를 수정하였으며,

명령어 잘못 입력으로 인한 오류를 못잡아내는 경우가 있어 수정함.
그부분은 Client.java 부분의

417   				else
418  +				{
419   					System.out.println("올바르지 않은 명령입니다.");
420  +					continue;
421  +				}


This revision is clean.


This comment in aubio.mk refers to these two lines:

```make
    $(if $(BUILD_SHARED),                         \
        mv '$(PREFIX)/$(TARGET)/lib/libaubio-4.dll' '$(PREFIX)/$(TARGET)/bin')
```

If they don't apply to jack, you can remove these few lines of comments.


Cool!  Though you should probably also apply this change to [Protocol17](https://github.com/ORelio/Minecraft-Console-Client/blob/Indev/MinecraftClient/Protocol/Handlers/Protocol17.cs), as #45 is also experienced there.  (Protocol16 doesn't need a fix as sending packet 255 is the proper behavior for that protocol version.)


well, this sure fixed the issue. :)


nvm I just refactored it to always return a value


CrippleMonster should use canUseItem instead of canUseAbility shouldn't it?


Can you clarify the purpose of "testing"?


Addresses issue ganglia/monitor-core#227


Sorry to nit-pick, but PEP8 says avoid extra blank lines if possible.


I've tried it and just show a page like this :

%PDF-1.4 %���� 3 0 obj <> /Contents 4 0 R>> endobj 4 0 obj <> stream x�=�� �0E3�+ΨK��hg��ķ��Z��u��M2]�������!��[c4�!������ؒ�� .........

Can you tell me why so ?
Sorry for bad english..


:+1:
Good Job.
If you can add as well Mobs eggs


We can not use global objects which require global construction. You will need to change this to be a `const char *const` or something similar. Same goes for all the other static arrays at the top of this file AFAICT.


`brew cask update` is just an alias for `brew update` :kissing_heart:


I tried to upload it in wordpress as a plug in. Well I customized the JS and uploaded it and the thing still looks the same.

Not sure how it works.

Sent from my phone,
Don Savoie

> On Jun 15, 2015, at 5:26 AM, Winner Crespo <notifications@github.com> wrote:
>
> Hi @donsavoie,
>
> Where did you try to upload this plug-in as a zip file?
>
> As it says in the readme, this is a Javascript plug-in to change look of the Google Translate plugin after you have it added to your website.
>
> If you have any question don't hesitate to ask, thanks!
>
> —
> Reply to this email directly or view it on GitHub.
>


One thing you missed: basedpc should be used in the DPS upgrade section as well.


Is there a race condition now that ProcessMessage() doesn't block for the batch to be sent? The condition I'm thinking of is when sendBatch() hasn't yet received the new outBatch and assigned it to the output, and ProcessMessage() appends the message to the old slice that's already been sent. Very slim chances, but it wouldn't have happened before when calling sendBatch() directly.


I'll take old+concrete over new+vague :)

I actually recall the debugging I did from looking at that stack trace.


No interpolation of the regex from the CLI, either remove the // construct, or use match() fn from the native library - this will correctly evaluate our string as a regex.


改行が足らないようです。


If you're inclined, you have a lot of commits for adding documentation between Jun 14th and 15th. You can "fixup" all of those together in an interactive rebase down to one commit, then move that commit to the end of your history. Same thing with all your modifications on the DESCRIPTION file.


A fallback would be nice, but an explicit global variable, repository-specific setting, or cmdLet param to override is probably cleaner.

In the meantime, I've implemented the following [Url Rewrite](http://www.iis.net/downloads/microsoft/url-rewrite) rule in my Nuget.Server's web.config as a workaround:

	<rule name="PsGet_Query">
		<match url="nuget/Packages\(Id='([^']*)'\)" />
		<action type="Rewrite" url="nuget/Packages()?$filter=tolower(Id)+eq+'{R:1}'&orderby=Id" />
		<conditions />
	</rule>


I'm going to make some comments on GitHub, though, so before you force push after an interactive rebase check them out, otherwise your rebase will change the commit history and my comments will be lost hah!


Oh right, too focused on clicking upgrades. Thanks :)


Wrap here


インデントはスペース2で統一したほうが良いと思います。


Are all these supposed to be part of the PR?


You'll probably want to add "busybox mount -o remount,ro /system" so your not leaving your system vulnerable as read/write to hackers...


改行が足りないようです。


This wasn't just for test resources, but also for in a reactor build only building to compile.


空行がありますが、上のコメントアウト共に必要でしょうか。


I've tried to address as many of the noted points as possible. Have another play with it.


I'd recommend breaking this into multiple lines to make it easier to follow. It could be a sub-shell tho if you want to avoid making the source accessible afterwards.


We are publishing the shim now. Though it will probably change a bit in the near future. Right now we're including the entire es5-sham, and we'll probably be more selective about what we include instead.

http://vjs.zencdn.net/ie8/1.1.0/videojs-ie8.min.js


I guess the full stop here should really be a semi colon, but that's a minor grammar nit.


For backend code I'd actually prefer to `require('lodash')`. It's pretty likely that some dependencies already depend on the full library and with deduping (default behavior with npm@^3) we can then get rid of any additional version on the harddrive :)


dwa


more stuff


I'm sneaking this one in here. Is this controversial?


why server_name set to dav.mailcow.de ?


@levicole i'm a bit confused as to what I'm suppose to call here for the form to render.


Точно, про торговцев с их 200% забыл. Привычка проверять входные параметры :)


Is there much reason to set both `num1` and `@num1` variables?


I suppose it needs some 'adjustments' to work on Wordpress, but I think you can try something like [this!].

I hope it helps you!

[this!]: http://wordpress.stackexchange.com/questions/556/what-is-the-preferred-way-to-add-custom-javascript-files-to-the-site


Looks pretty good. :+1:


hey @f2prateek, what was the original intention for this?


@levicole this obviously isn't the way to render items already associated with a user and generates the following error:
```bash
Started GET "/" for ::1 at 2015-06-15 19:55:51 -0500
  User Load (0.1ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = ?  ORDER BY "users"."id" ASC LIMIT 1  [["id", 2]]
Processing by UsersController#show as HTML
  Rendered items/_item.html.erb (2.1ms)
  Rendered users/show.html.erb within layouts/application (3.1ms)
Completed 500 Internal Server Error in 7ms

ActionView::Template::Error ('nil' is not an ActiveModel-compatible object. It must implement :to_partial_path.):
    1: <br>
    2: <h3>To-do items:</h3>
    3:
    4: <%= render @items %>
  app/views/items/_item.html.erb:4:in `_app_views_items__item_html_erb__2432554498490135449_70194675810960'
  app/views/users/show.html.erb:7:in `_app_views_users_show_html_erb___2387829553406314504_70194675109040'

  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/_source.erb (7.5ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/_trace.html.erb (2.8ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/_request_and_response.html.erb (0.7ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/template_error.html.erb within rescues/layout (51.9ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/_markup.html.erb (0.3ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/style.css.erb within layouts/inlined_string (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/_inner_console_markup.html.erb within layouts/inlined_string (0.3ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/_prompt_box_markup.html.erb within layouts/inlined_string (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/console.js.erb within layouts/javascript (39.1ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/main.js.erb within layouts/javascript (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/error_page.js.erb within layouts/javascript (0.4ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.2/lib/web_console/templates/index.html.erb (86.3ms)
```


Hotlinking the logo's probably not a great idea: there's no guarantee things stay the same on omeka.org with that type of thing into the future. The easier option is probably to just omit the logo.


*facepalm*


`BUILD_SHARED_LIBS` is the standard option for controlling it.
http://www.cmake.org/cmake/help/v3.0/variable/BUILD_SHARED_LIBS.html


Will this have an effect? The page should be closed before each test in _recreateWebPage, and the page renderer is a singleton, so I would think the memory would be released anyway.


legacy problem for repetitive vars. fixed.


Lol did they need to be xml or plist files?


Add some non-space whitespace


Gotcha, hadn't seen that yet.  I added this here before the return true, let me know if that's not exactly what you're talking about.


`${TurboJPEG_LIBRARIES_STATIC}` is not referenced in CMakeLists.txt, so no static support for it is implemented?

Also, you don't have to duplicate `find_library` calls. `LIST(INSERT CMAKE_FIND_LIBRARY_SUFFIXES 0 ".a")` can control its behavior and make it prefer static library.


## This is a markdown file


good work!


I always have V-Sync enforced in the driver settings because I find it actually quite stupid when any 3D application simply passes stuff to the GPU at the maximum possible rate. Because that's completely unnecessary and gains nothing in terms of performance or anything. It causes unnecessary heating of the GPU. The screen runs at 60 Hz so it won't go any smoother than that anyway…

But now: Maybe my current hardware setup is kinda weird or what but after disabling enforced v-sync I got even lower (!) GPU usage than before. Now it starts out at around 38% and barely reaches 60%. The only exception is pause of course. Don't know what's going on. It still runs smooth and without tearing but it shouldn't as soon as it runs faster than 60 FPS.

As for the vanishing POD list: I cannot recreate the bug at the moment no matter how often I purposely crash the game by temporarily overwriting a POD's internal offset list with random garbage. I'm not sure if it happened on crashes or not.

Whatever, I have to go to bed now. It's 3:07 AM over here and I don't even know what I just wrote :-\


Use `SliceUtf8.trim()` instead of Java `trim()` so the behavior is consistent


No idea, but XML is easier to read from. I have no idea what plist files really do...


Stray line break here.


This homepage stuff (here along with the home-page variable below) links to the domain root. That won't always be appropriate, especially for sites located in subdirectories.

If this links, it should go to either the Omeka root or the "request" root for the OAI endpoint. Or, simply omitting the link could be a workable option.


@levicole removing this line allows me to render users#show without generating an error, but I'm not sure how to move past this part


```s/Buzilla/Bugzilla/``` .


Finished with the undefined error and indents!


"OAI-PMH Repository" is probably a better choice here and in the top bar for the title, just for consistency with the rest of the plugin.


sourceforge :scream:


> Точно, про торговцев с их 200% забыл.

У торговцев 200% праны, а не здоровья.

> Привычка проверять входные параметры :)

Хорошая привычка, думаю. Особенно в серьезных коммерческих проектах. Но смысл проверки в этом случае не ясен. В случае "неподходящих" параметров, то есть _ошибочных_, нужно не `false` возвращать, а кидать иксепшон (`throw "Bad input data";`). Случай-то как раз исключительный.
Хотя даже это излишне: все методы всех объектов аддона покрыты трайкетчем (с помощью модуля `ui_trycatcher`), так что в случае непредвиденного поведения оно само отловится.


Lol I know they're property lists
http://stackoverflow.com/questions/1722608/property-list-vs-xml-in-iphone


I guess this is just an unrelated visual refinement?


Sounds like they're for storing settings, and not data.


It's recommended to specifically call close to release memory in some blog posts and it doesn't hurt. We were calling it before on success anyway.


Who knows, but they have xml now if they need it.


If we want to keep these comments, I'd add a comment to explain what they do


Do we need the commented lines below?


related to https://github.com/TeamMentor/TM_4_0_Design/issues/581


Lol that they do. If they need the plist its just as easy to make.


Great, tests passed: just a couple of notes on comments, but then I think it's good to merge.

Thanks!


Is this related to the rest of the commit? What's the purpose of this setting on the document?


@yogthos just wanted to say thanks for all the work you put in on this any many other projects!


rax retry tests


Well, the python script is right there, if they want to change what it's doing, it should be pretty easy.


If you're using W7 `setx /M PATH "%PATH%;<your-IDE-path>"`

Running cmd as administrator.


@JarrodCTaylor thanks! :)


Lies!! Lol


needs a slight modification so that the retroarch-joyconfig directory is created. Also the hotkeys are commented by default on the keyboard configuration so even after keyboard configuration they don't work unless uncommented manually (F2- savestate, F4- loadstate, esc-exit, etc.)


Hmm... Interesting bug there.  Since they don't use float2, it doesn't use the proper named constant.  fxc optimized that down to sequential memory access.


We had some customers who preferred using the locale specific dictionary to fill the application info. We've changed it since then to use the default `infoDictionary` and override any values with values from `localizedInfoDictionary`


Hello,

Ah thanks, setting to null would be better! I am very unfamiliar with the project still.

Anonymous charge points are useful on the project I'm working on (SolarNetwork), where the charge point may not be accessible to the OCPP server, but can still receive instructions via other means (from SolarNetwork). In this scenario, we'd like to still be able to send OCPP transaction data to the OCPP server, but rely on SolarNetwork for management tasks.


We should add support for https here.


So good


Much code


Very commenting


shut up

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

> Very commenting
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uminac/host-checker/commit/da6048d31dc994bae9fe545c64c18b1dfe53bc61#commitcomment-11693234>
> .
>

--
Christopher J. Umina
chris@uminac.com
781 354 0535


I guess Qemu has to be included as a part of BuiltKit...


1. Add Vote model `rails g model vote value:integer user:belongs_to post:belongs_to` `rake db:migrate`
2. Update Post, Vote, and User model to reflect association
3. Pass Vote validation spec and Post instance method spec
4. Implement voting public/private methods in Post
5. add _voter partial
6. Update /application.scss
7. Update post#show to include _voter
8. Update topics#show to include _voter
9. Update routes
10. Create VotesController with appropriate vote methods
11. Add VotePolicy
12. Add rank to Posts  `rails g migration AddRankToPosts rank:float` `rake db:migrate`
13. Update Vote and Post models
14. Update seeds `rake db:reset`


`to_camel_case` is no longer used, so I removed it.


I'd change it to `/conjur/authz` for consistency with the forwarder.


@thanhdut DWMR. Good work here. Press on.


This doesn't make sense – the tab is part of the format, the actual input line is everything *after* the tab. The leading tab does not end up in the line data that this function returns.


I believe you can access helpers directly in the template, I don't believe you need to put them into the stash:
http://mojolicio.us/perldoc/Mojolicious/Guides/Rendering#Helpers


@thanhdut DWMR. :+1:


*Website hosted on Heroku


Turning `proxy_ssl_verify` on only when `certificate_path` is set makes it silently not verify certs if it isn't. I think turning verification off should have a separate, explicit flag, and show a warning.


send as a PR and I will merge


Was this supposed to be `.channel()`?


This line doesn't run as expected and generates an error in firefox.
Should be:
<pre>
$postPjaxJs = "setTimeout({$this->_gridClientFunc}, 2500);";
</pre>


Hab ich auch überlegt, aber dann kommen wieder die Leute mit dem Apfel auf dem Computer und beschweren sich, dass ihre Leerzeichen als eckige Kästchen dargestellt werden :-P


加了这个没什么卵用啊…………最有用的那个模板函数没加啊


Sorry, you forgot to remove "]" character after "Codcore".


Any idea on when this will be pushed out to npm?


83811490-Fault_DirtyAirFilter (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1669 of 1672 tests passed)

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


ok... lagi gw update..


Newline for new sentence?


@psychoticmeow Added it back. Thx.


Correct me if I'm wrong -- did this change lose the "+2 cost" property of remodel?


I would like to see this merged in. I needed to use the wraparound functionality ($.lightbox.options.wrapAround = true) so I ended up updating my own fork. Much appreciated. Thank you


1度削除した時にもなぜこの項目を追加したのか疑問でした。
ここで定義されるべきものはサーバー側から呼び出されたプロシージャの引数のみで十分のはずです。getMonitoringServerInfoやputHostsなどの引数チェックはサーバー側が行うため、このコミットは不必要ではないですか？
また、仮にプラグイン側でチェックするとしても、この4つだけである理由も分かりません。全て定義するのであればputEventsなどのほかのputは定義しないのでしょうか


The include statement was out of scope of the PHP tags.  Also fixed the improper assignment of the $operatorEmail variable to $email_to.


@eddiemonroe How do I access git@gitlab.com:opencog-bio/bio-data.git ?

Currently directly cloning that repository is not working due to insufficient permissions. I think it should be made a public repository.

Also in order to work on #33 I'll need commit or at least pull request access to that repo (my GitLab username is `ceefour`)


This is fine, but just FYI, there is also [std::make_unique()](http://en.cppreference.com/w/cpp/memory/unique_ptr/make_unique).


teste


这人怎么了…
完全是可以用 ab3 自带 cross 的情况…


not sure what this commit is from....


不用特地写个 build 的啊…


I get an override error if I don't include a call to super.onViewDestroyed(view); Is this now removed?


Can you put a comment somewhere in here giving a high-level description of what's going on?  Not for me, but for the next person that comes along :)


This could use a small description too.


d


s


Maybe worth moving this to be a git subtree?
`git subtree add --prefix=src/lib/dns --squash https://github.com/wahern/dns.git master`


Tuned values are very bad. The W:L rate is  1:2


Thanks @owndroid I was gonna ask you to add this today :P


It just gives some space between the email and the button, they were pushed together previously


Heh I typo that constantly, cheers


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 553](http://build.prxm.it:8811/viewLog.html?buildId=3038&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 238 Build time: 00:07:37


Yeah, thank you. I'll change it in the original repo. (BTW: You didn't create a PR for this, otherwise I would have happily hit the "merge" button. ;)

And I'll ask @juliusv to create a redirect.


I think you shoudn't do `$.scrollTo`here, it's will cause bad user experience.


I think you shoudn't do $.scrollTohere, it's will cause bad user experience.


Please take a look at the API documentation that I started last week:
https://github.com/mapfish/mapfish-print/blob/development/docs/src/main/resources/long-strings/tocApiDesc.html

Basically you document the URL, request type, input and output.


Please keep the part `function(done){` on the same line as the test name when possible (i.e. when the resulting line fits into 80 chars).


:+1: I will move it to `config.yml`


Shouldn't you have '[name="filter_presenter"]' here? We should check input on change event, not div's.


Wait a sec... has already been merged... somehow notifications got sent to different email addresses, so I missed it.


So the javadocs are no longer parsed for API documentation now? They have
to be put in that page?

On Tue, Jun 16, 2015 at 12:03 PM Tobias Sauerwein <notifications@github.com>
wrote:

> Please take a look at the API documentation that I started last week:
>
> https://github.com/mapfish/mapfish-print/blob/development/docs/src/main/resources/long-strings/tocApiDesc.html
>
> Basically you document the URL, request type, input and output.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jesseeichar/mapfish-print/commit/8e2c17f1e419ad951102bc9ba1b7c7565c17ee53#commitcomment-11698802>
> .
>


TeamCity Hello world :: hello-win32-continuous [Build 13-34bde0b8bcf5f7d70a377f9377475c9bb962faf4](http://build.palaso.org/viewLog.html?buildId=58604&buildTypeId=bt29) is now running


TeamCity Hello world :: hello-win32-continuous [Build 13-34bde0b8bcf5f7d70a377f9377475c9bb962faf4](http://build.palaso.org/viewLog.html?buildId=58604&buildTypeId=bt29) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 2 Build time: 00:00:11
### Failed tests
```
HelloLib.Test.dll: HelloLib.Test.TestClass1.TestBonjour_ReturnsBonjour: <no details avaliable>

```


This is ever work for the API docs?


@davixd This variable `BACKUP_PROG_CRYPT_KEY` does not contain a `$` sign. The backup must fail this way...was this on purpose? I'm trying to understand your modification you made.


なおってない！


It did at one point but there was a bug in the parser and I didn't find the
fine to look at it.  But I suppose we can update these docs (at least for
now)

On Tue, Jun 16, 2015 at 12:06 PM Tobias Sauerwein <notifications@github.com>
wrote:

> This is ever work for the API docs?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jesseeichar/mapfish-print/commit/8e2c17f1e419ad951102bc9ba1b7c7565c17ee53#commitcomment-11698857>
> .
>


Good spot.  I must have lost that in a rebase.


This has been fixed.


TeamCity Hello world :: hello-win32-continuous [Build 14-a0a8234dc14451e9bc1205bbf4fc4d64b5e9960d](http://build.palaso.org/viewLog.html?buildId=58605&buildTypeId=bt29) is now running


TeamCity Hello world :: hello-win32-continuous [Build 14-a0a8234dc14451e9bc1205bbf4fc4d64b5e9960d](http://build.palaso.org/viewLog.html?buildId=58605&buildTypeId=bt29) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:00:14


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


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


I see. I would just prefer that if we offer this service via the API, that we make it consistent with the rest of the API. And that would be make it return JSON.


typoをなおしていたと思ったらなおっていなかった…


You can also `return (self.address and self.postcode and self.city and self.sex and self.tel and self.date_of_birth and self.privacy)`

I think that `self.sex and self.privacy` should be next to each other :D


...and I was wondering why tests were not failing for cythoned multidict


Hi,

In Linux version 3.14 Linux/include/net/cfg80211.h introduce into struct
cfg80211_mgmt_tx_params to package parameters together.  The tabs in the
code are there for readability so that there is a top level ifdef < 3.14
{all the old parms} else {the new parms}.  I think if you look at the
source file in my Git you will see this more clearly than in the diff.

hope this helps

Andy

On 15/06/15 18:52, Harshavardhana wrote:
>
> These for me seems like spurious tabs.. why are these tabs necessary?
> can you explain further
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AndyPi/rtl8812AU_8821AU_linux/commit/8c6e9c7e293918fb21193b53c21df7ad1ec34b03#commitcomment-11685719>.
>


I've emailed Christine from the LAA to ask if we can have the entire Kent Members list with email addresses attached. Can't do much until I receive that.


Moving isAvailbel out of for loop


Is it possible to use property instead of global variable?


@brad my bad sorry!


`ao` is not defined - assuming this is a typo.


Done. :)


You can define properties in cpuv_properties.py


Good point, I am renaming it. :)


@patois Thanks very much for the information. i would give it a try. when i have finished importing those symbols i would update and tell you about that. So i would first leave those broken functions as they are.


Please remove commented code for test use


Just the first move


I removed the `enumerate()` later on, since not used.
This code was terrible when I first started, have the menu actions one by one :sob:
Grows nice to be the best Unicode Picker :smile_cat:


test


It's working fine


same as L60.
And please tell me what merits this approach.


Ok, sounds good!  By readline I assume that means cursor keys and history, and maybe tab completion?  Note that if you use "input()" in Python to grab a line then that automatically gives you full readline capabilities (and uPy has this feature as well, even on pyboard).


Ran into this scenario today on IE8 and this pull request solved it. Thanks @danielshorten!


> the @" syntax is valid when within a powershell shell, whereas it's still in a cmd.exe shell at that point

How stupid of me. You're quite right. One line it is, the. I don't think that the caret is a significant improvement, is it?


Ok, good. Only I see several nbproject files in this commit.


no need revise: I want to know why pin_uv is copied. Should I revise other features?


@pthariensflame yes, I did.


@glennsarti, @akollegger I'm going to leave the file name stuff to you.


Trueの場合、 ``select_arm`` によって選択された腕から得られた報酬がゼロの場合には ``register_reward`` の呼び出しを省略できる。
ただし、 ``register_reward`` の呼び出しは  ``select_arm``の結果に対応していなければならない。


Thank you :)


Da gibt es noch keine Migration zu.


Updated, thanks.


it's ok


Would have been nice to do many things :)  But seriously, I wanted to write something more detailed about upip in the annotated tag (like a paragraph at the start) but left it to just a couple of lines in the "unix port" section.  upip deserves more than just a short mention in the README (and it should be short to keep that file concise); it needs it's own mini tutorial, a mention in micropython-lib's README, a section in the docs, probably some mention on micropython.org, etc.

And as for future releases, from now on I'll create a ticket with a few days warning of the release so things can be settled beforehand.


Es ist nur ein Modeländerung. Ich habe nichts im Migration file gefunden, das relevant ist?


That's a bug! `\s++` will always consume the last space, so that `\s++ ` cannot match anything.


This should probably also mention:
- This method is called 3 times (one for the previous, current, next month). (If you're event is showing up three times, make sure you're taking the parameter newMonth into account)
- The fact that newMonth parameter is 1-based instead of the 0-based API of the Java Calendar.


this include is no longer needed: https://github.com/yast/yast-ruby-bindings/blob/master/src/ruby/yast/client.rb
Thats why ancestor is added to have common base for old YaST code on which include file often depends.


Alright. So we need to keep this function.
How about renaming it `sendSuccessfulDeleteJsonResponse`?


Save, or safe?


Alright. So we need to keep this function.
How about renaming it sendSuccessfulDeleteJsonResponse?


this change broke build on drone.io. Is caused because drone.io configured to use jdk 1.7

https://drone.io/github.com/spring-guides/gs-relational-data-access/55


properties will be stored in blend file. If you copy 50000 uvs - all these data will be stored in the blend file. I think Global variable is much better as data will be cleared when blender will be closed.


It should read `')\s++(`, without spaces.


It should also read `)?\s++[`, without spaces.


`safe`, of course, but we're not too picky, just a little :)


looks shaky to me... why not at least log it?


I think you add config to pos.config model and then get it here as follows:

    self.pos.config.dummy_product


It does seem like it's removing them, yes. Follow up discussion at #380.


There is the `x` modifier, so the spaces are ignored and thus fine.


I found this line here http://blender.stackexchange.com/questions/711/how-to-add-a-uv-map-to-a-mesh-using-python
possibly verify() is better when a user changed active uvLayer in EditMode. But i did not test it.


Only user ``sigdev`` may access this environment.


below


@mstembera I was thinking along those lines i.e. + 5 to all values or something similar. Maybe tuning would be better?
@Mindbreaker1 interesting points


ok, sorry :)


this is not really used


This looks cleaner. Another change I would like to see is snake_case instead of PascalCase on methods.


pin_uvs are red points in uv editor which help to unwrap. Here is example https://www.youtube.com/watch?v=SXyIhYcYmoo&feature=youtu.be&t=425
I think it's good to paste them too.
It would also be goo to paste edge seams. But i did not implement it.


No worries, I know it's not so commonly seen but in this case it helps readability a bit :)


the keyboard configuration when configured should uncomment the lines - did you test witha  keyboard and it didnt ? Ill check/confirm though


Parsing plain `id` rows makes the parser a bit more complicated


@mbernat This is the only interesting change, the rest is just boring fixing of what got broken with this change.


Hm. It's not a notification bar IMHO. A notification is ephemeral and usually fades off by itself. This feature is persistent and will only disappear on interaction; that's why I called it "legal-notification" not "notification". I'd love this to have a better fitting and more unique name like "notice-bar" (notice as in *not ephemeral*).


Is there any reason why you decided to enable that snmp?


@patois Orz i can not find NTR-CFW binary of 2.0 via web. Maybe i should first try using the tools..


In case `err` implements `Error` interface, we print much more descriptive output.


This seams to be redundant.


Update.


For now we have no special module type for Kotlin/JavaScript development --- standart Java Module is used. Java Module is recognized as Kotlin/JavaScript by containing kotlin-jslib.jar among dependencies --- similar to Kotlin/JVM, which should contain kotlin-runtime.jar among dependencies.


Still, you are pointing to sysroot but not $PKGDIR.


Well, doesn't this defeat the purpose? I mean the main thread creates a new thread (i.e. AsyncTask) and waits for it to complete (the processing is indeed done in another thread but the main thread still blocks instead of taking care of UI events) and after get() returns then creates another thread (i.e. Thread) to send the request. Maybe a better approach is to just add the this.sipProvider.getNewClientTransaction(r) call in the initial Thread as well. What do you think?


I think line 17 will always clobber line 16, or am I missing something?


want to say mie


@mbernat Actually implemented this to make `ErrNotReleasable` work with the `errs` package. Since the package now uses interfaces, `ErrNotReleasable` now works with the package. The problem was that the descriptive error output containing the list of stories failing deployment was not being printed since the error package was checking for `*errs.Err` and only then printing the error description. And `*common.ErrNotReleasable` is clearly not the same type as `*errs.Err`. With interfaces this all starts to work properly...


Splendid, I will do that from now on, got abit confused from the previous code which standard you used.

Also I'll wait with more commits now until this merge, might be you cannot automerge your branch (searchandreplace) you are currently working on? You are probably more experienced with Github than me when making larger changes concurrently.


This gets pulled out in 4f75abe5fbd480eededf54b70299825145ecda8b


This breaks the build:

```
dh_install --fail-missing
dh_install: usr/share/caja-python/extensions/.placeholder exists in debian/tmp but is not installed to anywhere
dh_install: missing files, aborting
```


wat


KK


Fixed.


GStreamer stuff is dropped in 1.10...


Hi, I've uploaded prebuild exe here.
https://github.com/yosqueoy/ditsnap/blob/master/Release/ditsnap.exe?raw=true


Other repos refactoring this file have been changing the exit code lines to add up the values separately and pass the variable as there is a possibility the addition can go wrong in the exit line like this see example below

e_code=$((test_pass + rubocop + python_linting))

exit $e_code


That's fine, that ensures that some data is present. Might be a little more secure to at least check the type as a string to make sure it's not undefined or empty array etc


> And as for future releases, from now on I'll create a ticket with a few days warning of the release so things can be settled beforehand.

Yes, that's what I meant, thanks.


Can You change your line endings to crlf, please?


@alexander-bobin There is an ``AdminFormBuilder`` that we use in other admin forms, but I didn't like the markup so I didn't use it.  I also re-used the helpers / classes from the front-end - that a good / bad idea?


This makes searching for strings more intuitive. For instance when searching a page with the "page" input field you don't have to type the whole URI anymore.


Thanks, pushed a fix


Also added onkeypress character validation (allowOnly function) to all minigame textboxes


Yes, you are right.  Initially when I did this change, I just needed the example to work, so I was not really paying attention to these details.  The work done in the ClientTxnAsync could be moved to the thread that is started later on.  The only advantage of this solution is to possibly handle error from getNewClientTransaction(), so if this fails, the followup thread will not be started and possibly error could be reported back to UI.  But given now the methods like Register() etc. don't return anything, it may be optimized like you say.


Could you make this a method rather than a constant declaration please?  Ta


@dachary this won't work. what automake (or make) checks is `$$VERBOSE`. so we might want to run

```
make check VERBOSE=true
```
or
```
VERBOSE=true make check
```

i verified my theory by checking the generated Makefile. it is generated by automake 1.14.1.


Done!


What do you mean by graph representation? Draw an image or just list the atoms at bonds?


It's OK for now. We're yet to make a decision on the direction of the admin styling (reuse GDS, go bootsrap or something else) but this leaves us in good position regardless.


Oh god how did we not see this!


:sheep:...


Is it possible to use relative paths here so we don't need to change it between environments?


>Since there is no other mechanism to include firmware in the builds when "huge" kernels >are used, it looks like an "error".

???

```
# download the fw or offer to copy
	tmpfw=/tmp/fw$$
	x=1
	wget -q $FW_URL -O - |\
        sed '/href/!d; /\.tar\./!d; /md5\.txt/d; s/.*href="//; s/".*//' |\
        while read f;do
             [ "$f" ] && echo "$x $f" >> ${tmpfw}
             x=$(($x + 1 ))
        done
    y=`cat ${tmpfw}|wc -l `
    [ "$y" = 0 ] && echo "error, no firmware at that URL" && exit 1
    x=$(($x + $y))
    echo "$x I'll copy in my own." >> ${tmpfw}
    cat ${tmpfw}
    echo -n "Enter a number, 1 to $x:  "
    read fw
    if [ "$fw" -gt "$x" ];then "error, wrong number" && exit
	elif [ "$fw" = "$x" ];then
		echo "once you have manually added firmware to "
		echo "dist/packages/linux_kernel-$kernel_major_version-$package_name_suffix/lib/firmware"
		echo "hit ENTER to continue"
		read firm
	else
		fw_pkg=`grep ^$fw ${tmpfw}`
		fw_pkg=${fw_pkg##* }
		echo "You chose ${fw_pkg}. If that isn't correct change it manually later."
		echo "downloading $FW_URL/${fw_pkg}"
		wget -t0 -c $FW_URL/${fw_pkg} -P dist/packages
		[ $? -ne 0 ] && echo "failed to download ${fw_pkg##* }" && exit 1
		tar -xjf dist/packages/${fw_pkg} -C dist/packages/linux_kernel-$kernel_major_version-$package_name_suffix/lib/
		[ $? -ne 0 ] && echo "failed to unpack ${fw_pkg}" && exit 1
		echo "Successfully extracted ${fw_pkg}."
	fi
```

Maybe copying in the firmware from woof tree should be one of those options too?


:sheep: :sheep: :sheep: @mingo-x


Great, thanks!


@Syphurith https://filetea.me/t1szhFBOxtyRFik4pGMNMpAtw


Yes. Please use the files from the ProvToolbox.
Curating this set of files is a significant effort and I don't see it happening soon.
I think we should start with the files generated by ProvToolbox, put them into a GitHub repo as mentioned above so we can start testing
The test files themselves will be gradually updated over time as needed.


Yep this is not required, removed in ce6b0e6cc0d616cfe1a555be21d1acc8196353b1


Removed in ce6b0e6cc0d616cfe1a555be21d1acc8196353b1.


Please, add mapping.docs.filename as a parameter to the Doclet - additionalJOption in the example pom.xml below. Also, add an explanation that mapping.docs.filename "could be defined in your pom.xml file so they stick to the code" (same as for mapping.filename and mapping.version).


I can do the Python version of prov-compare so you can start testing.


Edited


faire des modifications sur le texte


LGTM.


Shouldn't ```e.preventDefault()``` suffice? Why return false as well?


Hey, hold on there. Now you are assuming that the only architecture that needs cross compiling is armel. Please change the variable, or convert cross-armel as cross-$ARCH.


@MingcongBai cross-armel? in arch.sh I assumed cross-$ARCH


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/3825/


It was introduced in https://github.com/sigmavirus24/github3.py/commit/cefec7b345d7560ce1b80b1121e07d39832a17f7#diff-4ca7f15885d8c65603b4b1a818bda2c4R290
with the if statement.


The prov-compare script will follow the non-zero exit code requirement as suggested above.


Is this actually what it responds with now?


Nice works


Unnecessary for this project, but might as well...


IE 8 -.-


@Icenowy my bad, sorry.


test


This commit is uneccessary if you recommit the big change without the modification of `rlp/encode_test.go`.


//Er du fuld?


Here the error message should contain "cannot create mapping docs file" instead of "mapping file". The same is true for the next error messages as well.


Hi there,
A semi-colon at the end of line 45 should be replaced by a comma :)


It's a very bad idea to remove code when work are in progress, especially when made by another developer. Please ask original developer before removing this kind of code. There are plenty of open and far more important issues to handle than this!


@teg I didn't think so but if you could help to rephrase it then it would be awesome.


Remove this block. It doesn't provide any functionality and enables a race condition.


Will fix it.


@prashanthr05 feel free to update this comment to refer to the right foot cad file, or to remove it.


hm why rollbar isn't catching those errors by default? I guess you don't need rescue clausule  at all


Here the message may contain a message that mode is mandatory. And that makes me think whether the mode should be mandatory - we can use "mapping" mode as a default one.


No worries about this yet - we can add a statement to the UI like "download depends on your internet connection, please be patient"


Why do you change glib2 "TRUE" to glibc "true"?


Nice!


to make sure that the unzipping and download worked correctly, could you also open a database connection and execute a query?

also, this should be an **integration test**, because it needs an internet connection and actually takes some time, which we do not want for the fast unit tests.


I'm pretty sure that full urls for these cases are needed


I now believe the context is different than what I had intermediately thought.


please make a note/card to put all these properties into the configuration mechanism of the SOS/iceland. If you did so already please ignore this comment :-).


+5 or tuning or both all sound like reasonable options to me.  Hope you succeed!


cauta pe net ce inseamna clasa generica. De exemplu  http://www.dotnetperls.com/generic


I do NOT think "abrt-upload-coredump-value" is a good name, because this function can be used with another triple switch widget which would not necessarily be linked to uploading of core files. I suggested something like "abrt-triple-switch-value", did I?


https://msdn.microsoft.com/en-us/library/hh191443.aspx. Tu ai implementat partial


It compiles so there is no bug, however there is something not compatible between gephi and play:

```
% activator clean && activator compile && activator run -jvm-debug 8888
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=256M; support was removed in 8.0
[info] Loading project definition from /home/user/Documents/inf342/AgentsAutonomes/project
[info] Set current project to AgentsAutonomes (in build file:/home/user/Documents/inf342/AgentsAutonomes/)
[success] Total time: 0 s, completed 16-Jun-2015 17:58:04
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=256M; support was removed in 8.0
[info] Loading project definition from /home/user/Documents/inf342/AgentsAutonomes/project
[info] Set current project to AgentsAutonomes (in build file:/home/user/Documents/inf342/AgentsAutonomes/)
[info] Updating {file:/home/user/Documents/inf342/AgentsAutonomes/}root...
[info] Resolving net.contentobjects.jnotify#jnotify;0.94 ...
[info] Done updating.
[info] Compiling 19 Scala sources and 32 Java sources to /home/user/Documents/inf342/AgentsAutonomes/target/scala-2.11/classes...
[warn] warning: Supported source version 'RELEASE_6' from annotation processor 'org.netbeans.modules.openide.filesystems.declmime.MIMEResolverProcessor' less than -source '1.8'
[warn] warning: Supported source version 'RELEASE_6' from annotation processor 'org.netbeans.modules.openide.util.NbBundleProcessor' less than -source '1.8'
[warn] warning: Supported source version 'RELEASE_6' from annotation processor 'org.netbeans.modules.openide.util.ServiceProviderProcessor' less than -source '1.8'
[warn] warning: Supported source version 'RELEASE_6' from annotation processor 'org.netbeans.modules.openide.util.NamedServiceProcessor' less than -source '1.8'
[warn] warning: Supported source version 'RELEASE_6' from annotation processor 'org.netbeans.modules.openide.modules.PatchedPublicProcessor' less than -source '1.8'
[warn] warning: Supported source version 'RELEASE_6' from annotation processor 'org.netbeans.modules.openide.awt.ActionProcessor' less than -source '1.8'
[warn] warning: Supported source version 'RELEASE_6' from annotation processor 'org.netbeans.api.annotations.common.proc.StaticResourceProcessor' less than -source '1.8'
[warn] Note: Some input files use or override a deprecated API.
[warn] Note: Recompile with -Xlint:deprecation for details.
[warn] Note: /home/user/Documents/inf342/AgentsAutonomes/app/Global.java uses unchecked or unsafe operations.
[warn] Note: Recompile with -Xlint:unchecked for details.
[warn] 7 warnings
[success] Total time: 16 s, completed 16-Jun-2015 17:58:27
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=256M; support was removed in 8.0
Listening for transport dt_socket at address: 8888
[info] Loading project definition from /home/user/Documents/inf342/AgentsAutonomes/project
[info] Set current project to AgentsAutonomes (in build file:/home/user/Documents/inf342/AgentsAutonomes/)

--- (Running the application, auto-reloading is enabled) ---

[info] play - Listening for HTTP on /0:0:0:0:0:0:0:0:9000

(Server started, use Ctrl+D to stop and go back to the console...)

SLF4J: The following set of substitute loggers may have been accessed
SLF4J: during the initialization phase. Logging calls during this
SLF4J: phase were not honored. However, subsequent logging calls to these
SLF4J: loggers will work as normally expected.
SLF4J: See also http://www.slf4j.org/codes.html#substituteLogger
SLF4J: org.webjars.WebJarExtractor
[info] application -

Hear me from static Global
[error] application -

! @6mff78d06 - Internal server error, for (GET) [/] ->

play.api.UnexpectedException: Unexpected exception[NoSuchMethodError: com.google.common.base.Objects.firstNonNull(Ljava/lang/Object;Ljava/lang/Object;)Ljava/lang/Object;]
	at play.core.ReloadableApplication$$anonfun$get$1$$anonfun$apply$1$$anonfun$1.apply(ApplicationProvider.scala:170) ~[play_2.11-2.3.9.jar:2.3.9]
	at play.core.ReloadableApplication$$anonfun$get$1$$anonfun$apply$1$$anonfun$1.apply(ApplicationProvider.scala:130) ~[play_2.11-2.3.9.jar:2.3.9]
	at scala.Option.map(Option.scala:145) ~[scala-library-2.11.1.jar:na]
	at play.core.ReloadableApplication$$anonfun$get$1$$anonfun$apply$1.apply(ApplicationProvider.scala:130) ~[play_2.11-2.3.9.jar:2.3.9]
	at play.core.ReloadableApplication$$anonfun$get$1$$anonfun$apply$1.apply(ApplicationProvider.scala:128) ~[play_2.11-2.3.9.jar:2.3.9]
Caused by: java.lang.NoSuchMethodError: com.google.common.base.Objects.firstNonNull(Ljava/lang/Object;Ljava/lang/Object;)Ljava/lang/Object;
	at com.google.common.cache.CacheBuilder.getKeyStrength(CacheBuilder.java:529) ~[guava-16.0.1.jar:na]
	at com.google.common.cache.LocalCache.<init>(LocalCache.java:242) ~[guava-16.0.1.jar:na]
	at com.google.common.cache.LocalCache$LocalLoadingCache.<init>(LocalCache.java:4814) ~[guava-16.0.1.jar:na]
	at com.google.common.cache.CacheBuilder.build(CacheBuilder.java:789) ~[guava-16.0.1.jar:na]
	at com.jolbox.bonecp.BoneCPDataSource.<init>(BoneCPDataSource.java:62) ~[bonecp-0.8.0.RELEASE.jar:na]
```


for the filter part: `grep -v '\*\|master\|dev\|release'`

it might vary per project but it will catch many common conventions


This exception will be thrown when there is no mode set too and the error message is not appropriate in that case.


leave the same number of spaces...


This is weired :/ in my console it returns 0 .. :S


see the comment in activity_remaining_debts.xml.


Yes, here #128, but I not sure why github merged it with previous PR (Spanish Translation)


We're moving to Travis.


Ok, I didn't know about it.
Regards

2015-06-16 13:29 GMT+02:00 Greg Turnquist <notifications@github.com>:

> We're moving to Travis.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/spring-guides/gs-relational-data-access/commit/c45675c244f4a3e6fa95a6eeaad6ec3c09c3bfba#commitcomment-11700322>
> .
>


De ce ai folosit OrderByDescending?


I think mine was a breaking change, shouldn't that jump to 2.0?
I removed `className` and `style` props completely, they will no longer work.
README also needs updating.


Please, add ApiFest project license - "Copyright 2013-2015, ApiFest project..." to all new files. Also, add Javadoc to the class.


Author @Chuck5ta (email unknown)
Sorry.


Testezi undeva asta? Sau te bazezi ca merge? Ai facut cumva teste unitare si stii sigur ca merge?
"Afiseaza toate proiectele pe care e un angajat ordonate dupa data de start" -> Vreau sa vad afisate aceste proiecte undeva


Makes sense, I'll rewrite it.


Good catch! How do we want to handle testing for this?


@capynet help :)

In any case, one more thing to take into account when upgrading the theme to the new design.


Untested !!!


with characters invalid in python argument name like a dot


Well I would like a core dev to have a look at it so, I'll try to generate a test case. Help is appreciated.


Better non-greedy `\s*?`


вроде исправил, не понимаю теперь одного, картинка шире чем ширина
браузера, как поднастроить чтобы всё влезало в окно и было одного размера с
header

On Tue, Jun 16, 2015 at 9:08 AM, Никита Селецкий <notifications@github.com>
wrote:

> здесь должны быть th, вместо td
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aleksejssaburovs92/mernieks/commit/6b0229ce8a06d73ab2b1275e5808605685f90a0e#commitcomment-11695282>
> .
>


Anything that can be done using a variable, should. Though, you're making me second-guess myself now :) Maybe check with @talbs or @frrrances to be sure?


It's nice that you have refactored the code that parses each tag :+1:


Move this class from the test dir structure to the sources dir structure.


I'll blog when the transition is complete. Cheers.


Please, rename to parseDocsSummaryTag (G at the end instead of P ;)) and have a beer on tap :)


Thank you for put sourceforge in the blacklist


`archive.presence || open?`?


Hello,

I didin't commit any print function, Whitch report are you talking about ?


Why don't you pass the cpacsHandle by value as all other tigl funtion do? This makes the API somewhat inconsistent.


Numele de clasele nu ar trebui sa contina verbe. Daca instantiei clasa asta, ai putea spune ca ai un ScrieListaAngajatiInFisier. Ar trebui sa se numeasca ceva de genu ScriitorListaAngajatiInFisier.
https://msdn.microsoft.com/en-us/library/4xhs4564(v=vs.71).aspx


sounds reasonable


`privacy.nil? || privacy == 'open'`


I've changed a type of the defaul_value variable to the int, so that was the reason why I change glib2 "TRUE" to glibc "true".


Fii mai specific cu denumirea de variabile. Ce iti spune tie ca ar fi "line"?


you need to increase the PHP dependency for this


+1 overall. Line should read "...instead *of* boost's...".


thanks @danoli3 !!!


Why there are so many tries?


<br/>
下一阶段进行模块块分割，定制接口


Ok, I'll revert it.


Basically this is because Gephi is built upon Netbeans Platform which currently only properly supports Java 1.6. Shame ~ We can still make an even cleaner way to that graph with a separated project. Hurray \o/


Because exceptions inside of Celluloid actors are not thrown. They just hang the server.


I guess what I meant primarily was the "all-firmware".


Contains 78 zip files and README pdf file


Please, add this annotation to the README file.


/cc @michalmuskala I have added TODOs on this commit. IRC is not working where I am right now but we can discuss them in the detail later. You can use this branch as a reference.


?? an issue with copy/paste ?


Yep


That sounds awesome. Wish I could've seen that live.


I think if you pull the [latest version of vim-webdevicons (>= 4.0.3)](https://github.com/ryanoasis/vim-webdevicons/releases/tag/v0.4.3) some of the rendering issues in gvim should be fixed, if not feel free to open an [issue](https://github.com/ryanoasis/vim-webdevicons/issues) :smile:


oups...
![image](https://cloud.githubusercontent.com/assets/2920150/8182273/1655cbe0-142e-11e5-90cf-dff141dd339d.png)


I think the original author of that code wrote this intentionally. When no devices are specified in the plugin settings, the plugin is supposed to do a single push with `device_iden` set to `None`, which makes the push to be sent to all the devices attached to given API key. After your change, no push will be made to any device if the devices field is empty.


Actually, the name of the file is *really* important: it is not `makefile.txt` but `Makefile`.
Moreover, the tabulation is important too.
And at least, the Makefile does not have to execute your code.


The files are on my laptop


Ah ok, not as interesting, thanks!

LGTM apart from the commit message


please revert this change, the localhost detection on the server side was incomplete. This is fixed. It’s better to use the apiUrl here to be more flexible in production setup.


Only issue I can see is that the site is now configured to run CORS from all locations rather than targeted resources.  Happy for this to be the case in alpha, however may be a security concern going forward?


This commit makes champion select related action unusable.


I agree with @marcusolsson, updated to episode in commit e76e2b815054109156cab9379b903da3294da6c0


Thanks, I've upped it to 5.4.


It is the Log line it seems - so it is ok


@ajroetker I'm on it already: https://github.com/kbarber/puppetlabs-puppetdb/tree/ticket/master/pdb-1523-more-work-on-defaults-for-aio ... https://github.com/kbarber/puppetlabs-puppetdb/commit/a31a183c53dfd116f6eb5b8e92165e57139c84d1


@tchap Uhm, is it necessary to have both `errs.NewError` and `errors.New`? :)


This broke manual installation of the newrelic JAR.  Now, you force us to use the installer which may not always be desired.

https://docs.newrelic.com/docs/agents/java-agent/installation/java-agent-manual-installation

There has to be a way to do the remote_file call and specify a path to download to so that the installer call isn't required.


Find a Title should be Find a title and Search Results should be Search results according to the gds style guide i thought?


this works only with php >= 5.5.

```
Fatal error: Can't use method return value in write context in vendor/doctrine/doctrine-migrations-bundle/Doctrine/Bundle/MigrationsBundle/Command/Helper/DoctrineCommandHelper.php on line 33
```


You can update the date to 2015.


походу понял,туплю по началу :D это же background image, правильно?

2015-06-16 14:36 GMT+03:00 Alexey Saburov <alexeysaburov92@gmail.com>:

> вроде исправил, не понимаю теперь одного, картинка шире чем ширина
> браузера, как поднастроить чтобы всё влезало в окно и было одного размера с
> header
>
> On Tue, Jun 16, 2015 at 9:08 AM, Никита Селецкий <notifications@github.com
> > wrote:
>
>> здесь должны быть th, вместо td
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/aleksejssaburovs92/mernieks/commit/6b0229ce8a06d73ab2b1275e5808605685f90a0e#commitcomment-11695282>
>> .
>>
>
>


I removed it ...
The way I wrote it breaks the `$broadcast`, and I don't know why. And I didn't feel like searching out why, as long as I have other things to do with the authentification.


Please, fix the typo in the name of the variable - one "i" is missing in the endpoint part of the name.


I changed my mind - In case the `archive` is `false` then I will depend on `open?`. Now I remember why I used `nil?` here instead of `present?` (which on any falsey value returns false)


I hope you don't find this too forward but was there a particular reason you decided to disable or any issues you were having?
Each version there have been many fixes [latest is 0.4.3](https://github.com/ryanoasis/vim-webdevicons/releases/tag/v0.4.3)


Is it ok now?


ok, so use `rescue :all` then


`Makefile:2: *** missing separator.  Stop.`


You're welcome.


I think the minimal example is
```julia
f(t::Union(Type{Any}, Type{Integer}), val::Integer) = "integer"
f(t::Union(Type{Any}, Type{Bool}), val::Bool) = "bool"
@asert f(Any, true) == "bool" # fails
```
which, admittedly, does not quite make sense. I'll add a comment and also dig deeper into why this happens.


The `redirectLogin()` (in the service `AuthManagement`) method is not yet working.


Exercism/Codewars/CodeEval can be moved into a new category "Exercises" or
"Practice" category, that could help trim things down. What do you think?

Alex.


Btw. I hate using strings like this. I usually use
```objc
[observed addObserver:observer forKeyPath:NSStringFromSelector(@selector(someSelector)) options:NSKeyValueObservingOptionNew context:nil];
[observed removeObserver:observer forKeyPath:NSStringFromSelector(@selector(someSelector))];
```

However, the key path we want to observe is `"hidden"`, but the `@selector()` trick will either give us `"isHidden"`, `"setHidden:"` or a compiler warning.


How to fix #360 correctly??
I've checked the code in RiotCall but nothing went wrong...
So you don't have any problem while logging in?
TW sucks


As this class is used as a resource to the doclet tests (the code does not need to be compiled), it's better to place it under src/test/resources. Also, the name of the class implies that the class will parse a resource not that it will be parsed. May be "TestResource" or something like that will be more appropriate.


Is it a style convention that you want to be adopted or is it just for this particular case's readability?


This would need to happen in the router, even though the route isn't changing, since the router package also manages the layout.


ouch. Nice catch. What about
<pre>
export VERBOSE = true
</pre>
in this case $$VERBOSE would work because the shell also has it.


Why not get rid of the sign up route too?


Redundant `self` here.


@ADude2 Sounds good. Do you want to make a pull request?


Önce server ardından clientın çalıştırılması gerekir proje sadece o şekilde çalışır.


This is not what you want!
See https://github.com/agiliq/Django-parsley/pull/57.
It should be `<1.8a0` or `<=1.7.999` to use `1.7.x`.

`<=1.7` will install `1.7` (and no point release of it)!


@mbernat `errors` is the built-in package. `New` simply creates an `error` out of the given string...


This branch is redundant. In the `try` block you can check for the value, `raise` if falsy:
```python
if not temp:
    raise KeyError
```
(Not strictly a `KeyError` but you could also raise a `ValueError`  and catch it in the same `except` clause).


Also, why was this done?  (the commit should provide some desc/hint)


Can you explain why you invoice api_groups here?


保留了jquery dom方式，已经采用documentFragment 来进行性能优化


Because I want to cache the value for the next request (sync).


It is best this method ends with question mark if it returns boolean value.


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

VSO State: Done


unnecessary


Besides, you could use double exclamation mark here:

```
def (...)?
  !!members_list(group).find { |m| m["id"] == AppConfig.google.domain_member_id }
end
```


Missing abc.ABCMeta metaclass.


Curious to see what happens if we cull the 6th rank as well. From there it should be clearer how far to ideally restrict the area for a weight increase. Maybe increase weights as a percentage of the ratio of the board removed?


Use ```none``` instead of ```"Null"```


I'm happy you have added unit tests :+1:
IMO that is a nice practice (TDD is even better ;)


How can I handle left joins with this?

On Mon, Jun 15, 2015 at 4:01 PM, ilausuch <notifications@github.com> wrote:

> Ups! I update it as soon as possible
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ilausuch/ArrestDB/commit/2fd016607043358b49ec0a6436799991ce644cba#commitcomment-11687085>
> .
>


I'm currently stuck at work, but I'll jump on it tonight, and add the
recently launched Javascript.com by CodeSchool!


IMHO this should define `:rtype:` in the docsting and body should be just "pass", since it's abstract.


Hi Andrew,
I hope that you are up for a quick question.
I got it all working in GraphEditPlus and got the code and used it to make a working application. I had to add in code to bring up the property page so I could set the wrapper to the codec. Once that was done it displayed the video input
What I really want is to NOT use the property page but to set the VFW value e.g. Lagarith Lossless Codec, programatically.
I tried this in c#:
[ComImport, System.Security.SuppressUnmanagedCodeSecurity,
         Guid("91E373C5-C386-457b-AF06-2ECBECCD3E12"),
         InterfaceType(ComInterfaceType.InterfaceIsIUnknown)]
        public interface IVideoForWindowsFilter
        {
            [PreserveSig]
            void InitFilterByDescription(string desc);
        }

static void BuildGraph(IGraphBuilder pGraph, string srcFilel)
....
   IVideoForWindowsFilter ivfwf = pVFMWrapper as IVideoForWindowsFilter;

   if (ivfwf != null)
   {
       ivfwf.InitFilterByDescription("Lagarith Lossless Codec");
   }

There was no error at this point but latter the graph failed with No combination of intermediate filters could be found to make the connection.

Could you point me in the right direction please.


TeamCity xBIM :: XbimWebUI :: Build [Build 63](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1836&buildTypeId=XBIM_XbimWebUI_Build) is now running


can you explain this problem? Is this a CFPropertyList problem? Can I do something to fix this?


Hm in that case I think this should be in a seperate method called `prepare_sync` or similiar and invoke it there instead of caching inside of cache :)


Use Ctrl+C


Remove the blank between docstring and pass. :)


TeamCity xBIM :: XbimWebUI :: Build [Build 63](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1836&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Compilation error: XbimWebUI.sln (new) Build time: 00:00:48


@scottpurdy this commpit is sort of a workaround, the approach with submetric did not work (some rounding probably happened(?) and I was getting only `0` from this metric. now returning the elapsed time directly works fine! but has a drawback - `sliding window` etc are not inherited from the submetric.

If the fix is not obvious to you, I'd like to leave it as is, and fix in other PR if there is requirement for that functionality.


TeamCity xBIM :: XbimWebUI :: Build [Build 64](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1837&buildTypeId=XBIM_XbimWebUI_Build) is now running


TeamCity xBIM :: XbimWebUI :: Build [Build 64](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1837&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:05


This method should not be exposed publically.
```def _stop_server:```


That version of the library requires Java 8. The ApiFest project is supposed to run with Java 6+ so please change the version.


Image doesn't look right after this update. Lot of white patches in the image.
![image](https://cloud.githubusercontent.com/assets/12911306/8182624/f3d4f7b4-144e-11e5-9d8a-57e94a535c07.png)


As far as I see, this dependency is not used.


@sthoduka This is fantastic! Thanks a _lot_ for working on Linux build support! As I wrote everything in a cross-platform way I had suspected a Linux build would only require a new build script, and this cmake script is looking great!

Some feedback and questions:
* Is it building a 32 or 64 bit binary, or maybe it automatically chooses based on the compiler's system?
* What packages does the user need on their system to run this process? I'd like to provide a list of necessary base packages in the readme, such as "sudo apt-get install cmake make gcc".
* Why are you specifying "Boost 1.46" in the CMakeList requirements? It will _work_ with 1.46, but the preferred version is 1.58 or higher. Any special reasons you chose a lower one?
* Do we really need to specify all of those "include_directories"? All #include statements in the source files already use relative paths such as "../libs/sqlite3.h", so they shouldn't need it. The only 3rd party include directory that's actually necessary is the location of boost headers.
* Instead of giving people a list of "On Linux, type all these commands," the preferred method is a "_gccbuild/build_linux.sh" script which simplifies the process and does all of that.
* To keep the directory structure clean, I'd probably put CMakeLists.txt in _gccbuild/linux/ and make the build take place in a subfolder of that as well. Those details don't matter right now... when merging your changes, I'll figure something out that cleans up the structure of the _gccbuild folder and cleanly separates it into subfolders for OS X and Linux.
* And lastly, and most importantly: How is SkypeExport working on Linux? Any issues? I know that Skype on Linux is still version 4.3 which means that it's a slightly older database format (not using any of the new features), but I wrote SkypeExport on Skype 4/5 (can't remember which) for Mac, so it _should_ work perfectly. Please do some test exports and look to see if all file transfers, edited messages etc show up correctly.

Again, this is fantastic news. Thanks again for taking the time to work on the Linux side of things! You don't have to answer these questions right away. Take your time, and thanks for helping out! :-)


nope, this was a problem on our end (when the plist was written), and this is just the workaround until the problem is fixed in the client producing the bad data. thanks for asking, though!


Do we need -beta version here instead of a final one?


Ok, thanks!


yeah. i thought they are different!


Can you share the dicom file? Can I add it to the test cases?


Adjust class name. The first char should be uppercase.


Tjena,

Kul att få vara med i diskussionen, men eftersom jag inte jobbar på TV4
längre så kanske ni inte vill ha med mig?

Trevlig midsommar dudes and dudettes.

/Lellky

On Tue, Jun 16, 2015 at 1:50 PM, Markus Felldin <notifications@github.com>
wrote:

> I agree with @marcusolsson <https://github.com/marcusolsson>, updated to
> episode in commit e76e2b8
> <https://github.com/TV4/video-metadata-api/commit/e76e2b815054109156cab9379b903da3294da6c0>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TV4/video-metadata-api/commit/1906b3a82a696a937e7a619538c6b95ce7c9d275#commitcomment-11700700>
> .
>

--
_______________________________________

Jonas Fors Lellky, Consultant
+46 (0) 70 499 49 15 | lellky@gmail.com


hmmm, after updating to the new file, I only get this now...

{
    "error": {
        "code": 400,
        "status": "Bad Request"
    }
}

On Mon, Jun 15, 2015 at 4:01 PM, ilausuch <notifications@github.com> wrote:

> Ups! I update it as soon as possible
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ilausuch/ArrestDB/commit/2fd016607043358b49ec0a6436799991ce644cba#commitcomment-11687085>
> .
>


@patois Thanks very much. I've got the file.


Why basket is stored as instance variable, but the actual data dict is passed into save (and fetched in load)?

I would rather make this class stateless and pass in the basket to save and let load return a Basket rather than dict. Then we should have a public API for basket (probably even at OrderSource level) to be serialized/unserialized to/from dict.


The number of spaces doesn't matter now as the view will be changed later anyways after the design is finished.


export PATH=$PATH:/data/data/burrows.apps.busybox/app_busybox; export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/data/data/burrows.apps.busybox/app_busybox


It's not a style conversion.
It's just for readability.

`__END__` isn't used in many Ruby scripts. This is a sample script. It's read by Ruby-GNOME2 users who may not be Ruby expert. It's better that we avoid to use rare Ruby features to understand Ruby-GNOME2 easily.

(Here document is also a rare Ruby feature...?)


This should be called only `if MOCK`


Need this line?
it seems this line is for testing.


Hi,

Hmm that's strange.  Give me some time to look into it and I'll let you
know :)

Cheers,

Andrew Griffiths
Glasgow, UK

RenderHeads Ltd
Email: andrew@renderheads.com
Phone: +44 (0)7845059318
Web: http://www.renderheads.com

On 16 June 2015 at 13:10, TimDPayne <notifications@github.com> wrote:

> Hi Andrew,
> I hope that you are up for a quick question.
> I got it all working in GraphEditPlus and got the code and used it to make
> a working application. I had to add in code to bring up the property page
> so I could set the wrapper to the codec. Once that was done it displayed
> the video input
> What I really want is to NOT use the property page but to set the VFW
> value e.g. Lagarith Lossless Codec, programatically.
> I tried this in c#:
> [ComImport, System.Security.SuppressUnmanagedCodeSecurity,
> Guid("91E373C5-C386-457b-AF06-2ECBECCD3E12"),
> InterfaceType(ComInterfaceType.InterfaceIsIUnknown)]
> public interface IVideoForWindowsFilter
> {
> [PreserveSig]
> void InitFilterByDescription(string desc);
> }
>
> static void BuildGraph(IGraphBuilder pGraph, string srcFilel)
> ....
> IVideoForWindowsFilter ivfwf = pVFMWrapper as IVideoForWindowsFilter;
>
> if (ivfwf != null)
> {
>
> ivfwf.InitFilterByDescription("Lagarith Lossless Codec");
> }
>
> There was no error at this point but latter the graph failed with No
> combination of intermediate filters could be found to make the connection.
>
> Could you point me in the right direction please.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/RenderHeads/DirectShow-vfwFilter/commit/a6df0965cd5b339b4e8bc44c29d0d950949fc1f1#commitcomment-11701111>
> .
>


same as L232


Is there a reason to not using DockerfileParser.lines.setter this time ?


who will generate `$JAVA_HEADER_FILES` ?


Why don't we just make it JSON-serializable? I.e. convert the datetimes, Decimals, etc. to strings in OrderSource.to_dict (or whatever will be the serializing method) and back to correct types in OrderSource.from_dict (or whatever)?


https://github.com/sebfz1/wicket-jquery-ui/blob/master/wicket-jquery-ui/src/main/resources/com/googlecode/wicket/jquery/ui/resource/jquery-ui.js#L5825 (+ 2 occurrences just lines above)


You can but unfortunately it is not anonymized.

On Tue, Jun 16, 2015 at 5:46 PM, Jean Francois Pambrun <
notifications@github.com> wrote:

> Can you share the dicom file? Can I add it to the test cases?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11701208>
> .
>


also L251 L257


IMHO it would be a big pro, if we wouldn't have to use pickle for storing the baskets, since pickling between Python-versions and/or Django-versions might be problematic.


Is this function needed after improvement?


try it still getting error in this line
 windowButtonList.TryGetValue(WindowButtons.Maximize, out btn);


reack


It still working for me. What is your request?


PEP8 говорит  use 4 spaces https://www.python.org/dev/peps/pep-0008/#indentation. Это разумно, так как в питоне не бывает большой вложенности (нету полноценных лирералов для функций потому что)


OK.
by the way, why don't you clear array when operation is done?


These methods touching the storage (load, save, delete, finalize) and the clear_all should be documented. Especially the difference between delete and finalize.

IMHO we should add proper docstrings for all of those and probably document the bigger picture of a basket storage in BasketStorage class docstring and have it linked here somehow.


Fair enough.

A little syntax kink: you may want to get rid of the space between the function's name and the parentheses, `function broadCastEvent(dataEvent)`. And an English typo: the verb is "broadcast", not "broad cast", so you may want to name the function `broadcastEvent`.


This doesn't look like compression artifacts. With the new commit, the codec returns a unsigned array when the codestream is unsigned. Are you doing any arithmetic that could cause overflows or underflows? I don't remember how the VOI transform is implemented, maybe the issue is there?


Hi think the problem has already been highlighted (if I believe google's answers) but it seems that their issue-tracker is currently down.
https://www.google.fr/#q=%24.datepicker[%28%28%22_%22+%2B+options%29+%2B+%22Datepicker%22%29]+is+undefined+site:jqueryui.com


Ahh... I see.
Then, this line should use `verify()`


Please document this. (I know the other settings are not documented, but we shouldn't add more to that documentation TODO-list.)


Do we need this dependency?


Maybe we should make InstancingThreadedCullingMethod switchable ingame to see if threading actually benefits us or if it reduces performance due overhead


Esto es para que ahora soporte los datos de los usuarios que agregaron el registro y el momento de la carga del registro


Also, why separating this with two blank lines even though the rest are separated with one blank line?


If a user wants to paste into many objects - it needs to be stored. For example, to paste uvs to 20 bricks.


@marcbachmann Your refactoring in `exec` broke the ldm tool. I just reverted your changes, you can have a look at it when you're back.


Thanks for the remark.


These can be:

        :field_path  => env_var.valueFrom.try(:fieldRef).try(:fieldPath)

`api_version` in general is to understand the data format and it's not worth persisting it. So basically you can drop it.


Drop `api_version` from here and the rest of the code.


You mean "#break" ? Yes it can be removed.


L257 is for debug. copied faces will be selected. It cam be useful for future development.


Why log file?


It's workable method like "bpy.ops.mesh.select_more()". But with python. Yes, it can be removed.


the memory is contiguous, but the pointer to the first byte is not necessarily aligned to a 4 byte address, which 4 byte types (uint32) require. Intel chips don't require this, but c++ does, and most other architecture will give you a SIGBUS if violated.


Good catch, thx


Yet another way would be to `try/catch` the call to `destroy`... :-/


Looks like someone else has hit issues with this https://github.com/titarenko/OAuth2/issues/73


I guess it is not a bug, but i just found out that, when you build your own version as a parallel install, you have to change this to your own application id, or your AnkiDroidNN will crash when you try to show the about dialog.
(Maybe there is a way to not hard-code the value here. Maybe using that is too much work.)


Om du klarer å forklare meg hvordan serialVersionUID fungerer og deretter hvorfor du har lagt det til ... så komemr du til å ta den bort på egen hånd.


Possibly. My understanding is it isn't a problem as long as we don't have anything of a sensitive nature in the site that we wouldn't want people to access.
My thinking behind adding the header for all requests is to remove the possibility of when a directory changes name or a new one with assets we want to share is added the addition of the header for the specific path is forgotten as it is such an irregular activity.


Humm... I understand your intention.

If data is stored on `bpy.types.Scene`, are all data stored on blend file?
also AddonPreference is stored on blend file?