Hey @russellbstephens
Sorry it's taken me so long to take a look at this (like... really long)
It's a great feature idea! I'm a bit worried about the implementation though. For something like hiding the remaining attempts I think we should provide an explicit API that tells the controller to hide or show the remaining attempts string. A method like:
    @property (nonatomic, readonly) BOOL shouldShowAttempts;
    - (void)lockScreenShouldShowRemainingAttemps:(BOOL)shouldShowAttemps;

Implicitly implying it using the string length could lead to potential bugs. Did you want to implement it as described and resubmit?

Cheers mate,

Aron


```c++
explicit AtomicSymbol(const std::map<std::string, std::string> &param) {
 for (std::map<std::string, std::string>::iterator it = param.begin(); it != param.end(); ++it) {
    this->SetParam(it->first.c_str(), it->second.c_str());
   }
}
```
The map here.


Did the vga666 overlay get left out? I see it in the commit comment and I see the [changes in the kernel](https://github.com/raspberrypi/linux/commit/79d83fc55240eb7413f50556fac3d94be862b3ff), but it appears vga666-overlay.dtb didn't make it into the commit.


Thanks,
Reverted in CyanogenMod https://github.com/CyanogenMod/android_kernel_cyanogen_msm8916/commit/24b1b598bc6f6958467581aba94dcd00b0f4004f


That's why I wanted the handle, to set up DoF RigidBodies, etc..., for each
of the cloned molecules.

If it's better to keep it buried, I'm writing some classes and functions
for playing with homo-oligomers that may be a better alternative?

On Wed, Jul 8, 2015 at 5:39 PM, Charles Greenberg <notifications@github.com>
wrote:

> maybe a better thing is to just return the molecule when you call
> create_clone() but then have an error when the user tries to change
> anything. I guess that's also good because you may want the Molecule handle
> for other classes like Degrees of Freedom.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salilab/pmi/commit/52ae303b9943599c4c070552e2c5b06786676a88#commitcomment-12074600>
> .
>

--

Daniel Saltzberg
Post-doctoral Scholar
University of California at San Francisco
Lab of Andrej Sali (www.salilab.org)

T: 415.514.4258

*Mailing Address:*
UCSF MC 2552,
Mission Bay, Byers Hall
1700 4th Street, Suite 503B
San Francisco, CA 94158-2330

saltzberg@salilab.org
ds229@bu.edu


Misspelled "canola"


Oh no...still getting used to CL git. Just uploaded via the webapp, and will head to office hours this week to get more comfortable uploading in CL.

Thanks!


Yeah, now I'm thinking it's good to keep it exposed but (somehow) disable add_representation() and add_structure().

What are your plans for homo-oligomers? Maybe move that discussion to a github issue...


Would first cut with just strupr() strlwr() be adequate for now?


OK I've changed it on my development branch.
After playing with it a bit I also think it's better to split the `NOTES_DATA` URI with the fields and tags into separate URIs like `notes/#/fields`, `notes/#/tags`, `models/#/fields`, `models/#/templates`, etc. The two disadvantages of combining them into the same table like we currently do are:

1) The user of the API has to "demultiplex" them, which is a bit of an unnecessary burden. As we expand the Provider, I can imagine users of it will start to get frustrated if they have to do this for a bunch of tables.

2) It's kind of unintuitive and clunky expanding the DataColumns class, and I don't want to have the situation where some tables are multiplexed and others aren't, so even though it probably involves slightly more code, I think we should just always split the sub-tables into separate URIs.

I think now (that we're thinking about moving to beta) is a good time to revisit these the design decisions, as once the beta is released we won't really be able to change the API again, ever, other than expanding it or providing alternative URIs with a new version.


Read the UsersManual.pdf to get started :)


Somehow we have caused a new test failure here independent of the change in the sequence. Somehow one of the triggers is hanging off the end of a sequence, and we don't catch it until we read the data back in.


Thanks can you make the change when merging?
On Tue, Jul 7, 2015 at 9:45 AM NielsCharlier <notifications@github.com>
wrote:

> This should probably change to six as well
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jodygarnett/geoserver/commit/141dd558490c52d9385428d973d953ffce5f5d41#commitcomment-12039631>
> .
>
--
--
Jody Garnett


j


This commit seems to introduce a bunch of bugs...`c3.getType()` and `c3.getBlob()` both require a `columnIndex` parameter, and in method `bytesToString`, there are several classes that are not yet been imported and several constants that are not yet been defined...These bugs should be fixed, otherwise it is quite inconvenient for new users...


 - general
  - debugMode
  - (cache) longLength
  - (cache) shortLength


test


i hate your skill its too good for me, by the way, what language is this?


A little Freudian slip?


I made some recent changes to port_platform.h so that this should not be needed... can you take a look?


no min_ver?


@wlky
FYI I'll be submitting a PR over the next week or two which will implement some breaking changes to the ContentProvider, and add a higher level API that developers can use. It would be great if you'll have time this month to comment on that PR and implement the changes. This should be the first and only non-backwards-compatible change to the ContentProvider.


So the file is only written once per day??


@nicolasnoble - I'd've expected WSAStartup to be called via grpc_iomgr_platform_init

any reason why this is too late?


Duh, thanks.


Closes ianbrown78/rubydesk#1


So far, functions to create homo-oligomers of size N and a class to build
ambiguous restraints among a set of copies.

Eventually, I'd like to construct a system where there are multiple states,
each with a different oligomer composition (monomer, dimer, trimer, etc...)
and weights for each state.  The application of this is for protein
aggregation, where the composition of the samples in each dataset are
heterogeneous and likely different among the different experiments.

On Wed, Jul 8, 2015 at 6:02 PM, Charles Greenberg <notifications@github.com>
wrote:

> Yeah, now I'm thinking it's good to keep it exposed but (somehow) disable
> add_representation() and add_structure().
>
> What are your plans for homo-oligomers? Maybe move that discussion to a
> github issue...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salilab/pmi/commit/52ae303b9943599c4c070552e2c5b06786676a88#commitcomment-12074802>
> .
>

--

Daniel Saltzberg
Post-doctoral Scholar
University of California at San Francisco
Lab of Andrej Sali (www.salilab.org)

T: 415.514.4258

*Mailing Address:*
UCSF MC 2552,
Mission Bay, Byers Hall
1700 4th Street, Suite 503B
San Francisco, CA 94158-2330

saltzberg@salilab.org
ds229@bu.edu


Fair question. Probably should go with something more like:

try:
from salt.cli import daemons
except ImportError:
daemons = None

Yeah?


whoa, i didn't know you could mention people in your commit messages! that's cool.


Yeah, definitely not in general. It's true for containers, I think. Not something that needs to be solved now.


Ooh shit, you're right, I have a wrong conditional. And yes, probably this would be the method name, or... ```tryToEnsureOriginCanBeObtained```


Not yet. I am really tired today. So I will watch the video class online.
 On Jul 8, 2015 5:50 PM, "Sinan Ozdemir" <notifications@github.com> wrote:

> I wasn't able to find a one pager for your project idea, did you submit
> one yet?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zhhyun2/SF_DAT_15_WORK/commit/e5e3dac7727527f0163dc1a1db50153d6ff4b359#commitcomment-12074720>
> .
>


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 122](https://teamcity.labkey.org/viewLog.html?buildId=259881&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 23 Build time: 00:33:13
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

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

```


Closes ianbrown78/rubydesk#3


const char* lzPath?


OK, that method name seems okay to me


Remove this line.


FLAT?


I hope you feel better and feel free to stop by office hours to a catch up session :)


"FLAT" or "UNICODE"?


Thanks very much.☺I will.
 On Jul 8, 2015 6:21 PM, "Sinan Ozdemir" <notifications@github.com> wrote:

> I hope you feel better and feel free to stop by office hours to a catch up
> session :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zhhyun2/SF_DAT_15_WORK/commit/e5e3dac7727527f0163dc1a1db50153d6ff4b359#commitcomment-12074988>
> .
>


`scm`だとDSLと重複するので別のに変える


Well, I mean, if it's necessary, make it exceptional, sure.  I haven't looked.


Good


I use windows because the computers I bought, for the price i paid, with the processor (i7) and memory i wanted don't yet (or did not at the time) have Linux systems available as-built.

It's also more Plug-and-play.  I personally had fits with most of the Linux distros I used.  I like Linux, but it's not so user friendly.  For now, I don't have time to mess with changing OS's.. And I can't believe running a VM is more efficient either.

  The issue we are dealing with here is actually something of a  Linux "flaw" which is convenient for your purposes.. I.e. Linux can "delete" (remove) files that are in use. http://man7.org/linux/man-pages/man3/remove.3.html  ... because of this "feature" of Linux, you may actually have the same "error" and not ever know it because Linux is letting you delete a file that is actually in use.... the file _must_ be in use somehow.  Not sure why; I am going to keep researching this.


Mendoza


This is a solution to the hackerrank problem [Detect HTML Attributes](https://www.hackerrank.com/challenges/html-attributes/submissions/code/12416529)


I also removed the `cmake_minimum_required` as it doesn't look like the right place to be


In addition, could also do:
```
s/affiliations for users/
s/a user needs to be associated/users need to be associated/
s/her W3C ID/their W3C IDs/
```


This is definitely better!

> but it seemed to me that creating a Notifier module like you suggested was too tightly coupled with these particular errors/warnings.

Honestly, the coupling of my suggestion doesn't bother me at all; my suggestion was to define the modules inline in this file.  They are intended for this exact use.  Concretely, one of the benefits of the polymorphic approach is that the conditional gets evaluated once (in order to pick which implementation) rather than every time a stub or message expectation is added.  That said...that's going to be absolutely unnoticeable here, given that the user is probably only mocking or stubbing a small number of methods (often just 1 or 2!).  And I'm realizing that the solution I suggested would make it more difficult to support the `RSpec::Mocks::ExampleMethods#allow_message_expectations_on_nil` API.

So let's go with what you've got here!


not sure. I'll do it anyway.


BaseLayer.preOut won't work due to line 313 (due to shape of weights and input). But GravesLSTM.preOut() certainly does need dropout/connect as per BaseLayer.preOut()
https://github.com/deeplearning4j/deeplearning4j/blob/master/deeplearning4j-core/src/main/java/org/deeplearning4j/nn/layers/BaseLayer.java#L313


@rouault

This is not complete. Need to debug this error.
[http://paste.ubuntu.com/11845218/]


I don't think this one should be deprecated since it is used just below.


doesn't this conflict with the definition on line 13?
It looks like `def detail(request, question_id):` is identical to me


97647908-DocIssues4615 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (647 of 647 tests passed)

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


In LSTM: applied on the memory cell contents, then multiplied by output gate activation (output gate activation is in range [0,1]). It's not like a standard FF network (or even vanilla RNN) where the non-linearity is the last thing to be applied.


can this be published as a new version of perftools.rb?


This will return the A value and exit the method, so the n and E never get reported.


@psytronx DWMR (done with my review). Looks good. Press on!


Trying to understand how your commit message correlates with the code changes...

Unless you did something in the sqlite database that is invisible?


I'm following the design for ConvolutionLayerFactory, PretrainLayerFactory, LSTMLayerFactory etc which do it this way. Not sure what you are getting at?
https://github.com/deeplearning4j/deeplearning4j/tree/master/deeplearning4j-core/src/main/java/org/deeplearning4j/nn/layers/factory


So this is how to make a new `kinetics` entry *and* simultaneously link it to a given reaction `r1`?
I guess this is appropriate because a `kinetics` entry will never change to a different reaction, nor be associated with more than one reaction. It will belong to `r1` from the moment it is created.


I was under the impression that execAndReturn was in-place. That was the reasoning. I'll have to double check that.


This will only return the `self.pub_date`, and probably not as a unicode object.


```
$ echo -n ABC93e41c6e | md5sum
ceb0356a2272e428b350903ab4e0974e *-
```

```
processing this expression: dynamic=md5(uc($p).$s),debug

The expression checks out as valid
push
app_p_uc
app_sh
f5h
Expression=dynamic=md5(uc($p).$s)
Flag=MGF_FLAT_BUFFERS
Flag=MGF_SALTED
Flag=MGF_PASSWORD_UPCASE
Test=@dynamic=md5(uc($p).$s)@ceb0356a2272e428b350903ab4e0974e$93e41c6e:abc
Test=@dynamic=md5(uc($p).$s)@7304cdc9467148dfe105f192a7bd6b18$20911b9b:john
Test=@dynamic=md5(uc($p).$s)@c4f43f07ea48ea25a1118756244f3083$36bc7ce9:passweird
SaltLen=-32
Func=DynamicFunc__clean_input_kwik
Func=DynamicFunc__append_salt
Func=DynamicFunc__MD5_crypt_input1_to_output1_FINAL
MaxInputLenX86=110
MaxInputLen=110
```


Nice Work!  Everything looks great!


Compatibility.


Note, it is all or nothing in dynamic.  ``` md5($p.uc($p).$s) ```  is not not legal in dynamic


Sorry about the inconvenience, but the Egit plugin of Eclipse seems not to be working all right sometimes :joy:


you could do
`len(killings[killings.year == 2015])` in order to return an int instead of returning a tuple then getting the first element.


It seemed that MC version 1.5.2- is not working all right with resources, though. It is the problem of  the directory .minecraft/versions/ , and I'm just working on it.


+1 that'll work for me, unfortunately the only thing I think that used the X-Registry-Config at the moment is the buildImage, but this works! Thanks!!!


yeah I know it make sense. start trying frameworks today.


very nice :)


I don't particularly mind having the files listed in order in two places, if it works. Is that too annoying? I prefer an explicit list to using file names for order. The globals won't be changing much so I think it's more of a problem for the other files.


Only minor version changes are supposed to go in the changelog, ie only new features.


beautiful solution!


Sure, I'll do that.


No need to pluralize; that's a singular "their" I'm using.


Looks much better in my opinion!


Cordial saludo ing Ivan:
Con las disculpas del caso sobre las observaciones de la entrega, por favor le suplico que me pueda a  volver a revisar los avances ya que tuve algunos percanses  en el manejo de git hub lo cual lo tenia de forma digital pero no en la nube de git.


but... then should I wrap all callback in a method ? I didn't understand that, what did you mean about put the name?


@CurlyMoo

you got it almost correctly. the /etc/apt/sources.list can't be
```
deb http://mirrordirector.raspbian.org/raspbian/ wheezy main non-free
deb http://ftp.us.debian.org/debian jessie main non-free
deb http://http.debian.net/debian jessie main non-free
```

but

```
deb http://mirrordirector.raspbian.org/raspbian/ jessie main non-free
```


d


ok


Shouldn't this flag be part of ALL SIMD code?  it is a flag for a generic interface .....


You don't need a system pre-installed with Linux. It's easy to install yourself. That being said, no, VMs are plenty efficient. Many Intel and AMD processors feature virtualization acceleration to help make VMs as fast as possible. More often than not, it's more than enough, especially for Minecraft servers. Factor in the fact that Java 1.8 tends to be a bit faster under Linux, and also that a light distro such as Ubuntu Server (not the lightest, but pretty good) has very little overhead, it'll be fine. I guarantee you it'll be 50 times better than dealing with Windows bugs all the time.

As for plug-and-play, it 100% depends on the hardware (computer itself, peripherals, etc.) you're using. In my own experience, I actually find Windows to be quite the opposite - almost everything requires me to install some sort of silly driver. Hell, I just installed Windows 7 on a laptop recently and I couldn't even use the ethernet port let alone wifi or even USB disks. I would have no clue how to get drivers on there if it weren't for the fact that I had a Linux partition (that I had just freshly installed and had wifi, ethernet, and USB drives working out-of-box), and I had used that to copy the files over. Once I got that going, I couldn't even use my USB audio card with it without more drivers. All of this worked out-of-box on Linux. Obviously this varies between hardware, but I certainly wouldn't say that one or the other is more plug-and-play. Just depends on stuff. :P

As for the last paragraph, this is *far* from a flaw in Linux. Having file locks like this cause severe issues with servers. It's a *serious* hindrance, especially in a server environment, and is MOST CERTAINLY NOT something you want for one. It's an exploit vector (if software in a controlled environments manages to simply open one file read-only, it's locked forever and other important processes can't handle it). It is likely to cause instability and random issues and can take forever to resolve, such as these issues. It's *never* been an issue in Linux, and there's no such thing as having issues with this sort of stuff. If a file is indeed being read when it is deleted, it is physically kept on disk until the file is closed in which the free space is regained. In crucial scenarios where there's not enough space, I believe the kernel will just force close it and .read()/.write() operations on it will cause it to throw a perfectly reasonable error which programs can easily catch (and should catch).


@tgoyne I believe the RealmSwift project has to have the same workaround for now. I was unable to build without it.


Can i compile the examples in visual studio 2013 using this cmakefile?


killings['streetaddress'].fillna("Unknown", inplace = True)

Got to use inplace.


Another, cleaner way could be `len(killings[killings.year == 2015])`


The functionality provided by this line is provided by `utilities/polly_common.cmake`, however I can't include it without running `poly_init` to set the name of the toolchain. Can I move this to inside `utilities/polly_init.cmake`?


First way is great, don't need the second.


As I did not used to work with git, it was a way to keep a tracked of what I changed. I removed it.


@leonkyr  why do we need this ?


@alicetragedy is there a way to indent? like this:

![screen shot 2015-07-09 at 12 00 25](https://cloud.githubusercontent.com/assets/11146524/8592761/2f586b1e-2632-11e5-8703-592c081f311d.png)


Yeah it's really motivating to get all the collaboration! Looks like the Java 8 upload failed BTW


On my side I think that it's better to embed this information in a view thanks to inheritance.
Since here you "break" the existing API.
It would be better to make a new View_Sensor that derive from View or a new derived Intrinsic.
I mean sensor width is a optional value, that is not always set.


One could almost say this demonstrates a danger of cherry picking commits from a branch, when only the head of the branch has been tested with a degree of thoroughness.


Detect Duplicate form submission using struts saveToken


I don't move `if (!string) return {args: {}, params}` to line 53. because I need params,I can't get them at line 53.


Change to something more generic? Not all containers are French


better to not include files such as these in version control


Přidej sem a na ř. 263 prosím pletený závorky. Líp se to čte a tenhle způsob nikde nepoužívám, tak ať to je konzistentní. Díky


@bmanth60 I forgot about them. Yes, it's a possible source of errors.
@SamFiltered you're right with type hinting. I was following symfony-like pattern. Associative array gives you advantage when you're using the provided API, because you see which parameter is what. It is also a built-in type, so it would help you when testing.

It just helps you on higher level. In first example you see what is what, even when someone isn't using good variable names.

```php
$task = new Task(array(
   'id' => $a,
   'content' => $b,
   'created_at' => $c
));
```

In this example you have to dig into documentation to know what is what.

```php
$task = new Task($a, $b, $c);
```


@F3PiX Could you add a star here, too?


Just a note, in rust, the fact that something is a type is indicated by capital letter at the beginning, thus IMO it is redundant to add the suffix `Type` to it.


oops "adding title" without title tag.. :-)


I use generators instead promises. thanks for your help, @mdibaiee.


add container to test graph rendering in conditions close to a real usage


> There are tests already which use range faceting, which for some reason passed originally. With this change I could make this test happy again (WebShopTest) but two others are failing as I am seeing now. Still looking into it.

You are right.

>  I was confused because the test explicitly use STRING as encoding. Was it an error in the test or is it something that should continue to work?

In this case it was.


I'm afraid so - I'm using trusty, which is the current Ubuntu LTS, supported till 2019, and this version of GNAT is the default there too.


it's an update or a twkea of the selectbox ?


@F3PiX Brrrr needs to be inside the _ symbols since it's about your taste in cheese haha ;)


> Is a self test failure good enough?

Yeah sure. I doubt anyone will try to use it anyway. I haven't seen such construct ever.


By default, will auto be a ref or a copy of the original map?


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: js/ajax.js


Do the webgl renderer test only if the browser is webgl-capable
Copy-pasted webgl detection code from `sigma.renderers.def`, maybe we should put in in sigma.utils


Unhandled cases:
```erlang
bar(X) ->
  [A || A <- [94]] = X,
  xxxx ! A = X.
```


clean the container the dirty way


Identation should be uniform. It's isn't uniform on this line.


+1 on this, can this get pulled in.  I have applied this change manually and it fixes my borked Plex updates too


reorgonized the imports to include the .svg.renderers imports and looks like the imports in the examples


Are Lieke and Carsten linked? Do we want that also with the coaches' names above?


There is a small typo in `populate_if_emtpy` :)


Also if you set absolute path `return '/comments/'...` this generates `http://comments/...` GET request. Not sure if this is intended ...


Updated the offset of the second gradient stop so the linear gradient is visible


Is this correct?


The easiest way to do this, @roosdebildt, would be to put the fun facts inside <blockquote> tags. It will give you this style: http://getbootstrap.com/css/#type-blockquotes which is not perfect, but it works for what you want, I think.


Just a suggestion, the `message_type` field is not a type, perhaps `message_content`? Or just `payload`?


No, it's wrong. Thanks. :)


For some reason I was working with an old version I guess and I had to make it running. I cloned the link you sent to me: https://github.com/cms-jet/JetMETAnalysis is that correct?


> But overall, this collector shouldn't be doing this. The published version shouldn't be determined from the working file, but rather from other published files.

This is what I was aiming for. It's collecting highest version from version folders in the published directory. So basically just figuring out what the previouslu highest version of a publish was for this asset.

I have this as a Collector so it's easy to replace the *versioning* implementation without having to recreate the full integrator. This also means other Plug-ins can acces this version information, if they need to.

> Actually, where is os.walk walking? How are you controlling the current working directory here?

Good catch. Consider this version branch experimental / pseudo-code. I had little time this morning and did a quick draft. Still wanted to share te concept I was going for to start discussion. ;) Anyway, it should walk the `publish` directory so it retrieves the highest previous version from previous publishes!

Also the regex pattern was out of the top of my head. Will have to double check that one as well. ;)


Mybe call it PrimitiveSetUtils


rename appropriatelly


Haha! Hablo en inglés porque no es facil saber quien habla español en la internet~
Cuando tengo un poco más de tiempo, voy a traducir lo que puedo en la aplicación.


@AlbertoSONIC Not a problem if you're using snprintf/sprintf. If else, you could just put a string `"Selected Theme:     <  0  > "`and add theme to the '0' character, no sprintf.


Ok, i forgot that m-s-a have any translations and po files.
I will change the text to feedback page usage.


@AlbertoSONIC Isn't it mean boot into sysnand. Okey not a problem really.


Love the fact that we're getting rid of the type tag!


Oops. You're right. auto is only deducing the type per default, not whether as a reference or not, AFAIK:
http://en.cppreference.com/w/cpp/language/template_argument_deduction#Other_contexts

Thanks. I'll fix that now. Using `auto&`


The `alt` attribute is supposed to be always set. That's part of the (unwritten) contract of image nodes. If no attribute is specified, it should be set to the basename of the image file with underscore and hyphen converted to spaces.


toFastutilHashSet


@lukeboorman, @gregoire-pointeau:
Could you please manually run these two commands in PGAdmin:
`CREATE INDEX visualdata_instance_time ON visualdata (instance, time);`
and
`CREATE INDEX proprioceptivedata_instance_time ON proprioceptivedata (instance, time);`

This should massively increase the speed for `triggerStreaming` etc., as in you will get a much higher frame rate.


"symfomny" :P


一些文件路径或者SVN提交日志中可能包含【空格】，如果不用“”包起来，可能会出现异常


To clarify, the basename without the file extension. We have a convenience method for this in Asciidoctor (as of 1.5.3). You would write

```
(Helpers.basename target, true).tr('_-', ' ')
```

I haven't decided whether to make the `tr` part of the `basename` API.


Perfect! So @F3PiX can you do it like this on all the fun-facts? Just copy it, no understanding required :p

<blockquote> _Fun Fact #1: Psst… don’t tell, but “Kaaskoppen” or “Cheese Heads” is a not so friendly nickname for Dutchies._ </blockquote>


...and even if it isn't set, I'm pretty sure that an interpolated nil value ends up being an empty string.


How does this work if you have switchable keyboard layouts in your desktop environment on the host?  Which one will be placed into the VM - the currently active one?


The assert fail message looks out of sync with the condition that would lead us to reach here. In other words, even if the values are '1,2' and '1' but the global property key is neither 'test1' nor 'test2', we would still be here. So can you change this message?


Thanks.


What exception is getting caught here? Is it worth logging that something went wrong?


@GrahamCampbell well, they broke BC, I'm entitled to typo's :dancer:


svn1.7版本之前，工作副本的每个目录（包括子目录）下都有.svn文件夹
svn1.7及后续版本，只有工作副本的根目录下有.svn文件夹


why are you using your "regular" ssh key for vagrant, too?


how are you doing.


Apparently it did not help with the Travis issue :-(


why should nested virtualization be explicitly disabled?


what's this for?


Thanks!


@Ombridride The hitbox is still the same, however the display mesh now has fewer triangles (went from 2(triangles)*6(faces)*2(boxes) = 24 to 2(triangles)*6(square faces)+3(triangles)*2(stairs sides) = 18) and better overall fillrate (smaller faces/triangles, i.e. don't draw parts that will eventually get overwritten in the graphics buffer, such as the top face of the lower step which will be half-hidden by the upper step).


I return params as empty object instead no return params for check params in lib/index.js:169 and lib/index.js:170


@angelsoffury Nitpick: Please indent the code here as well. Also test this if works without any issues. In Ruby, Rails by default we follow 2 space indentation.


wow ...great changes


Looks great @anandbagmar. I'm currently learning more about how to automate Mixpanel Metric implementation and GA. I've seen that this currently only supports GA out of those two. My background isn't tech, how do I know if this fits for us or what to do exactly with this?


この行問題ないでしょうか？


awesome, really love the use of a static method for this.  Each `Kinect` object being a specific device.


Pourquoi pas récupéré le prix de l'arme, divisé par 10 ?


Yes, agreed, it bugged me for a while that. Yes the name is interesting and wrong, we need to come up with something better, perhaps user_Data or something makes sense.


Could you move prototypes of `saiki_*()`, which should not be public to anonymou namespace in field.cc?


More specifically, I've been thinking about a specific scheme for bounds-checking hoisting with `load_mem_with_offset`.  Let's say you're on 32-bit so you compile:
```
load_mem_with_offset_i32(base, 4)
load_mem_with_offset_i32(base, 8)
```
into
```
check_i32(base)
load_i32(HEAP+base+4)
load_i32(HEAP+base+8)
```
Where `base` is an `int32` and `check_i32` is an internal op that ensures `base` is in bounds.  For this to be valid, a single guard page needs to be placed after the heap (so that if `base` is in-bounds but `base+4` is out-of-bounds, the code will safely fault).  However, if the heap length is > `UINT32_MAX` (accessible via `int64` loads), I can't place a guard page since since there is valid memory.  This is fixable in various ways, but it adds complexity and chance for bugs.  I expect this isn't the only heap optimization strategy with corner cases like this.


エラー説明文は、os_error because dog command is not executable
あたりでしょうか。 unknownではない気がします


ca marche !


It renders a string, but making sure that there won't be any line break.


Ca règle le probleme du cut aussi :3 et y a la "decoy" sur CSGO, molotov, etc incgrenade.


Looks good :+1:


yes, it could. I don't know what we'd do with it, but it would be at least consistent.


Great. :)


do we really need to create a disable button CSS style why not use ng-disabled ?


I just tested it on official. target sends move again after stun, indeed. But doesn't send ForceRun\WalkTo


Можно так
```ruby
[(:role_id if can? :update_users_role, User),
(:password if can? :update_users_password, User)].compact
```


Sorry it was mistake


+1 or rather +82 :-)


@Dani723 @vanthome How about naming all input action attributes like "doSomethingAction"?

```javascript
{{my-comp changeRangeAction="doStuff"}}
```

Inspired by https://github.com/balinterdi/ember-cli-star-rating#usage


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


기업들 추가 마쳤고 해당 로고 이미지들은 메일로 보내드렸어요~ 잘 되어야 할텐데! :ㅇ


An index with this size should be a "no go". (See http://www.sql-server-performance.com/2007/covering-indexes/ e.g.)


It's going back up due to an error on my end :)


TeamCity Jarvis Framework :: CI-GitFlow [Build 21](http://build.prxm.it:8811/viewLog.html?buildId=3430&buildTypeId=CqrsFramework_CiGitFlow) is now running


Just for the future, the right way is to use <code>@Issue</code> annotation


для таких длинных условий используй if ... else ... end,   a ? b : c только для простых выражений.


Okay, I'll.
I'll change to the functions that already exist there.

On Thu, Jul 9, 2015 at 5:37 AM, ngeiswei <notifications@github.com> wrote:

> @arleyristar <https://github.com/arleyristar> don't hesitate to have a
> look at
> https://github.com/opencog/cogutils/blob/master/opencog/util/numeric.h
> there might be functions there you may need, for instance the function
> bound, a functional variant of check_bounds.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arleyristar/moses/commit/2957ffb8d469f97eac18614ece84281f0138f88a#commitcomment-12079542>
> .
>


Why not just return ansistr?


ditto


и тогда вообще можно сделать так:
return lot.paid if @selected_lots_initial.nil?  (blank?)


Mistakes were made.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.1.0-NugetNewStructure+1](http://build.prxm.it:8811/viewLog.html?buildId=3430&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 110 Build time: 00:02:54


Congratulations.


Well, then no choice. Unfortunately I'm on Debian Sid, which uses gnat 4.9 right now (and will be on 5.x once gcc undergoes transition 5.x). Additionally we can't use Ada 2012 features (and pre- conditions would be nice way to do many FIXMEs in just a few lines).

Well then - I will have to rely on you for checking if it builds with gnat 4.6 as I don't have enough free disk space for qemu image.


If i have a global property who value should have space, wouldn't this clear it?


do u need this controller?


delete this file if you don't need.


Please merge this commit into master.


alignment is missing


この処理で async = false のルートが存在しなければテスト追加要
（⇒他のテストケースで通過しているようなので現状のままでＯＫ）


We could use `ProtocolV1Features` here. Is that a layer violation? @dpw thoughts?


ok, I had not seen it was an enum and not an enum class. Changing that may represent many work as we would need to define enums in different namespaces and we would loose scoped improvements. It may not be that good to use them everywhere after all.


well done


Мне не нравится такая конструкция.
Лучше сделать как то так:
```
def selected_lots_for_initial(lot)
   Lots::Base......
end

def какой то метод
  @selected_lots_for_initial = selected_lots_for_initial(lot)
end
```
с next_lot так же


I think we should also remove the old include/db/mysql.inc.php file and all other (select) pointers to it and switch to use mysqli by default!
Why? Since the mysqli or PDO is the absolut recommended way these days. And we have a requirement for PHP 5.3+ and MySQL 5+ too. But Upgraders would still use the old API without ever noticing! This is what happened to me. I did not really notice and care until I switched manually with 2.0. That is something what we should take care of!


@mrehanali Well done :+1:


LOL, nice picture. I guess the site will be behind https. Loading resources on non-secure channels will result mixed-content warning (yellow triangle in the ssl lock) in most modern browsers.


Simplify the widget markup.


https://github.com/biplovbhandari/eden/commit/d8393b4f07013f2b775442de29f2ac8fce58b2f5
please review this


@flavour please review this...I tried to incorporate your suggestions...


тебе кажется это условие читаемым?????


Yeah, I think it's more coincidence that they happen to be the same, and at the moment we can remove V1 support simply by removing some lines in `protocol.go`.


All of these ebuilds get a 403 error response when they try to fetch the package.


Separate comment post action as a backend file, so that if someday you remove the `manager` folder, your visitors will not be able to post a comment anymore.


это зачем и где используется?


Changing the fetch URLs to https:// will provide a bit of extra security.


и это тоже непонятно зачем


This is bad bad bad, checking this sort of stuff should happen in the check_pong.


No :)

mysql is perfectly valid until PHP 6.0. mysqli is already the default for serendipity 2.0, btw.


Thanks for the PR. If anyone have time, I think those 3 blocks should be refactored into either putting all 3 regex checks + settings into one loop, or making it a function. We now have 3 almost identical blocks of code.


@pveentjer: This test has some flaws and it seems directly pushed to the `origin/master` which makes it impossible to review before merged in.

- There's a `TransactionImpl.TRANSACTION_EXISTS` flag which checks nested `tx.begin()` calls. In some tests here, `commit()` or `rollback()` is not called at the end . Because flag is not cleared properly, `TRANSACTION_EXISTS` flag remains always set and that makes some other tests fail, like; `TransactionImplTest` and `MultiMapTestsFrom2X`.

- Most of the tests here are not testing the intended case. I mean they are failing before the expected failure case.

For example;

```java
@Test(expected = TransactionNotActiveException.class)
public void commitTransaction_whenAlreadyCommitted() {
    TransactionContext transactionContext = hz.newTransactionContext();
    transactionContext.commitTransaction();

    transactionContext.commitTransaction();
}
```
Test above expects that second `commitTransaction()` call after original commit should fail. But here test fails on the first `commitTransaction()`, because according to the implementation a transaction cannot be committed before it's started. Correct test method should be:

```java
@Test(expected = TransactionNotActiveException.class)
public void commitTransaction_whenAlreadyCommitted() {
    TransactionContext transactionContext = hz.newTransactionContext();
    transactionContext.beginTransaction();
    transactionContext.commitTransaction();

    transactionContext.commitTransaction();
}
```


What we could do is if the "mysql" layer is selected but not available to try to load mysqli instead...


Why?


 * Apply the new `tidy` method in **MTE** and **HTE** plugin.
 * Add keyboard shortcut for _table_ and _jump link_ button.


Allow multiple custom field scope _selection_. Use checkbox instead of selectbox!


Add _file_ custom field.


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


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


It triggers whenever the directional (arrows) inputs are changed AND other keys, kinda like OnPlayerKeyStateChange but it also triggers for changes in updown & leftright key states.

include was also not yet updated for these new publics.

This version of plugin for some reason won't compile in linux yet - it's being worked on, perhaps @ikkentim could help out? (gcc doesn't find #include <sampgdk/eventhandler.h> when NO_EVENT_HANDLER is undefined)


priyanka repo


Official should send a walk packet as well if the target is actually moving, how else is it supposed to stop?


@WloBeb
Thanks for the addon.
Two small things,
I can't edit the code myself now, ```if (charSettingsList[charNamesList[idx]].general.active) ```
I think will be better as ```if (charSettingsList[charName].general.active) ```  you get the char name from the ```.forEach```
And did you try using the event element in the on change function ?  I made the call to onchange functions to pass the triggering element as second parameter, so something like
```
onchange: function(newValue, elm) {
+                if (newValue) {
+                    $(elm.target).parents('.ui-accordion-content').prev().removeClass('unactive');
+                } else {
+                    $(elm.target).parents('.ui-accordion-content').prev().addClass('unactive');
+                }
+            }
```
should work but I don't remember trying, or testing.


Since you are not fetching order fresh from the database, how could it have become null? In other words, what is the use of the above assert?


With this you can use :

    exec(cmd, opts).on('ready', function() { console.log('connection ready'})


Whts this..???


Case insensitive.


These DB changes are being performed based on real test results not a theoretical best practice.  We are monitoring the before and after results to ensure that we are optimizing for performance under a range of load conditions.


Thanks mate. Is the Java 6 version okay? It seems undersized compared to the others


That looks good.


ok :+1:


Link to the contribute guide which has a detailed walkthrough ...


Did you just forget to add the corresponding should annotations on the service? https://wiki.openmrs.org/display/docs/Generate+Test+Case+Plugin


@legaultpierre why is this line commented out? Is this intentional?


Good improvement.  Merged.


I think these data joins need an identity key to allow animation.


Allow custom math text with the third parameter:
…
~~~ .php
echo Guardian::math(1, 10, array(
    1 => 'one',
    2 => 'two',
    3 => 'three',
    4 => 'four',
    …
    …
    '+' => ' plus ',
    '-' => ' minus '
));
~~~


TeamCity Jarvis Framework :: CI-GitFlow [Build 22](http://build.prxm.it:8811/viewLog.html?buildId=3432&buildTypeId=CqrsFramework_CiGitFlow) is now running


the old behavior allows suspension of the current action, start a new action, and resume the current one when new action finishes

it ok to change


This seems to break installations, where apache2 is already installed because that prevents libapache2-mod-fcgid from installation because httpd-cgi is already fullfilled.
Any idea how this could be solved debian style? Otherwise i just revert that httpd-cgi part.


Shouldn't that be a capital "A"?


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.0.2-beta.1+1](http://build.prxm.it:8811/viewLog.html?buildId=3432&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 109 Build time: 00:02:06


It's deprecated on 5.1.


I can then override the code in my theme as follows:

```php
/**
 *  Override the markup for the related posts.
 **/
function aamhc_related_posts_content($str, $related_posts) {
  if( count( $related_posts ) ){

        $h = min( count( $related_posts ), cprp_get_settings( 'number_of_posts' ) );
        $str = '<div class="row row--you-may-also-like">
                  <div class="columns small-12">
                    <h4 class="page-also-like">You may also like</h4>
                  </div>
                </div>';

        for( $i = 0; $i < $h; $i++ ) {

          $thumb = '';
          $link = get_permalink( $related_posts[ $i ]->ID );

          $str .= '<article class="you-may-like column small-12 medium-4 home-grid">' .
                    '<div class="page-grid-body eq-height-div">' .
                      '<a href="' . $link . '">' . get_the_post_thumbnail( $related_posts[ $i ]->ID, 'recent-feature' ) . '</a>' .
                      '<a href="' . get_the_permalink($related_posts[ $i ]->ID) . '"><h2>' . get_the_title($related_posts[ $i ]->ID) . '</h2></a>' .
                      '<p>' . wp_trim_words( strip_shortcodes( $related_posts[ $i ]->post_content ) ) . '... <span class="excerpt-read-more"><a href="' . get_the_permalink($related_posts[ $i ]->ID) . '">» Read More</a></span></p>' .
                    '</div>' .
                '</article>';
        }
  }
  return $str;
}
add_filter('cprp_content', 'aamhc_related_posts_content', 10, 2);
```


Yes I will try to change that.


This doesn't make it fullscreen for me, but it tries to download a file...


Extra blank line detected.


Use 2 (not 0) spaces for indenting an expression spanning multiple lines.


Pass `&:stringify_nested_symbolic_keys` as an argument to `map` instead of a block.


Align the elements of an array literal if they span more than one line.


Space missing after comma.


yes, std::map stores key value pair using a sorted search tree, so looping through it means sorted by keys


I would prefer `Do not` and include `Use refinements`


Great, will merge


Keep in mind: that is a very generic rule for such a specific component.


Do we actually give a strong recommendation to use mysqli in whatever flavour with 2.0+ for upgraders?
And that a (manual) switch (in the backends config) is very easy and not really making any other trouble than using a better API? :)
When I did this I just tried and was glad it did without any further issues.


I saw that too... :laughing:


I put it there as a placeholder, in case there's some specific action to perform in that case.


:+1: Nice!


We really need to get the tests back working. After that recent refactoring the elliptic eval wouldn't compile for me, and the error seemed really obscure. I was pretty lucky that Tobias ran into that same problem earlier.


Angular 1 (both js and dart versions) use function.apply to invoke the pipes (a.k.a filters). I believe for most pipes the overhead should be negligible compared to work load of the pipe itself. The benefit is that number, type and default values of the arguments is much easier to follow in the code. Assuming that number of arguments is small (which is a reasonable assumption for pipes) I agree with you that removing function.apply is good enough solution.


Hi,

Does it throw any error upon failure? And with the above change what would be the default value??


IMO, setting mysqli as the default on installation is as strong as a recommendation it gets ;)

For the 2.1 release we could add a upgrader routine to switch to mysqli automatically. When it makes trouble, people can switch back to mysql through the config file; I wouldn't remove the mysql layer just yet.


Having one operation instead of two (or three) makes it much easier to write the transformation code - there are fewer cases to consider.

> if I'm reading your transformation code correctly

You are.

>  What's the benefit of non-invertible ops if you already have invertible ones in place?

There are other operations that might be interesting. One would be a union operation on sets. Say the document is a set, and the operation is "add A, B, and C" to it. This is itself not invertible because given a set you can't simply remove A, B, and C because you don't know whether or not the elements were a member beforehand. Another might be "pop from the end of an array", which isn't invertible because you can't push something back onto the end of the array unless you know what it is that you popped.


OK, I guess I buy that. =)


@PMickael could you please explain what this commit does? You broke current master tests.


> For the 2.1 release we could add a upgrader routine to switch to mysqli automatically

That was the route I had in mind.


Why do You want to hide the `$metadata$` field in constructor function?


This is a new feature in DNN 8.  You can (and should) separate your SqlDataProvider scripts into separate files.  DNN will order them according to the fourth set of digits and then execute them in order.  This feature allows you to upgrade from one build to another.


就是觉得设置链接颜色这种事应该由用户自己来做。


Extra blank line detected.


Use 2 (not 0) spaces for indenting an expression spanning multiple lines.


Assignment Branch Condition size for run_test is too high. [20.12/20]


Pass `&:stringify_nested_symbolic_keys` as an argument to `map` instead of a block.


Align the elements of an array literal if they span more than one line.


Space missing after comma.


Align the parameters of a method call if they span more than one line.


Line is too long. [123/120]


Line is too long. [125/120]


Use `size` instead of `count`.


No worries @sukima. (1) was definitely a mistake. Fixing (2) doesn't resolve the issue. I'd agree with (3) too!


Why do you transform `com` to `double`?


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


You need to also check it in `save(…)` logic as POST data is never to be trusted.


@johnelse This exception is when list.assoc throws Not_found exception - I saw this during host boot as part of dbsync


Noticed that this should be `need net`


Official sends a walk after stun ended, it doesn't sends a stop. "how else is it supposed to stop?" Stunned targets don't walk?


You can remove the comment now :3


For more accurate


Please only use `%` with two types of arguments: a `dict` and a `tuple`.


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


Do you ignore created,modified and effecitve dates for Brains?


done


why not set the flag here?


Look, i'll pull it and redo it and see if that makes a difference


I don't see why this needs to be a message.


I do not recommend hard coding the messages. I would rather you use Context.getMessageSourceService().getMessage() and pass in the message key.


Ditto, this does not have to be a message. You're rendering the entire form and can provide better feedback in the template than in these messages.


Ditto.


it should be better than without the index, but is still far from being optimal


Same as above regarding the use of Context.getMessageSourceService().getMessage


Eh... yea. Nvm.


Why not setting the same table as WP?
For example:
`$user_permissions['capabilities'] = $user->allcaps;`


why are you not using existing view?


@pveentjer Also we found out that, because of this test somehow ``HazelcastConnectionImplTest `` fails on our sonar build.  When we run sonar build with this test, we get ``org.mockito.exceptions.misusing.MissingMethodInvocationException`` from ``HazelcastConnectionImplTest ``  under JCA module. When we ignore all the tests in this class, sonar build is green. Right now, we did not figure out how this test effects our sonar build, because other builds are fine.

Here is the exception we are getting :

https://hazelcast-l337.ci.cloudbees.com/view/Sonar/job/Hazelcast-3.x-sonar/com.hazelcast$hazelcast-jca/692/testReport/junit/com.hazelcast.jca/HazelcastConnectionImplTest/getList/

Also here is our custom sonar build which runs jacoco on a branch without this test :
https://hazelcast-l337.ci.cloudbees.com/view/Sonar/job/hazelcast-3x-sonar-mustafa/


@jushe
But the return type of **pow()** is `double`, according to the **promote rules**, the result will be the `double`


OK, I'd forgotten that conversation we had. Leave it with me.


Debug info? :p


lol.


FYI: it is a behaviour Change, not to update LastModifiedOnDate,


Nothing wrong with it as uncompressed the app weighs the same as the others (111MB). Not sure why its more compressed, but could be the program i use to .dmg the Java7/8 releases and what it uses to compress it.


cool, thank you


Debug info too? :p


Ok, done in https://github.com/neovim/neovim/commit/00c2d72633486aba826a6f4a7b9c1e9225572e8b.


Not a problem; I'm much more interested in removing *deprecated* Ada features rather than avoiding new features, as the motivation is to keep the code usable; Ada evolves much more slowly than C, but C code can become uncompilable after 15 years or so with changes in standards and tooling. WORDS was written from 1993 -> 2007 in what seems conservative Ada (Whitaker was indirectly one of the Ada creators), so hopefully the situation isn't going to take too much effort to resolve.


bitte keine magic zahlen. (SWT)


It reminds me some good time investigating in order to find the fastest solution.. https://hal-enpc.archives-ouvertes.fr/hal-00769267/


Does this fix your issue? It seems to be the same code as used in redhat guest which atomic inherits.


Sure. No further comment ;). Except is there a special reason you put the fieldset inside the article and not after it?

> Op 9 jul. 2015, om 13:41 heeft Wolfgang Thomas <notifications@github.com> het volgende geschreven:
>
> Move the "Comments" box back to the bottom of the page. Fixes #57
> @cornae I hope you agree with this change, I think we're moving back to your original proposal....
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/euphorie/osha.oira/commit/be80f15036070e764b06d258396fc9d16d35541c


@mk01
I noticed that you take care of the .dtb files, that's good.
Did you already make the kernel dtb ready, so these .dtb files are loaded automatically during boot?
I made this already by modifying the hooks.d/pre-deb.d/cp-kernel-sym file:
```
diff --git a/hooks.d/pre-deb.d/cp-kernel-sym b/hooks.d/pre-deb.d/cp-kernel-sym
index 72fd839..3bee026 100644
--- a/hooks.d/pre-deb.d/cp-kernel-sym
+++ b/hooks.d/pre-deb.d/cp-kernel-sym
@@ -11,7 +11,13 @@ case $config_platform_branch in
         ;;
     rpi*)
         cp ../build/$config_platform_branch/working/System.map ./boot-new/System.map-$kver
-        cp ../build/$config_platform_branch/working/arch/arm/boot/zImage ./boot-new/kernel.img-$kver
+        #cp ../build/$config_platform_branch/working/arch/arm/boot/zImage ./boot-new/kernel.img-$kver
+        if [ -x ../build/$config_platform_branch/working/scripts/mkknlimg ]; then
+           ../build/$config_platform_branch/working/scripts/mkknlimg --dtok ../build/$config_platform_branch/working/arch/arm/boot/zImage ./boot-new/kernel.img-$kver
+        else
+            echo "Warning: can not add DT-trailer to kernel image"
+            cp ../build/$config_platform_branch/working/arch/arm/boot/zImage ./boot-new/kernel.img-$kver
+        fi
         ;;
     *)
         ;;
```


Why do we need `docker_image`? Looks like it's mandatory when we trying submit jobs, does't it?


The meteor command is available on PATH though...

On 9 July 2015 00:47:59 CEST, Tony Tamplin <notifications@github.com> wrote:
>I'm going to assume it's the default install location for the `curl
>https://install.meteor.com/ | sh` command
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/meteor-gazelle/meteor-gazelle/commit/6c09eaafae97eb944245875a1b6fd174a5e8011b#commitcomment-12073260

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Я бы изменил название переменной, например teacher_ids


Можно вынести из цикла


ah, good point


A number of things:
1) you should get and close the error stream at the end of the validate().
2) before you get the stream, first get all the other fields (headers, response code, etc). Closing the stream will cause the other elements used in the construction of the error to fail
3) you should wrap the getErrorStream() in a try-with-resource block.
4) also add a catch block if closing the stream fails (eg: server side closed first)
5) I struggle with getNetStorageType() in the first place. It shouldn't be in this layer and should be part of the factory constructor phase. That aside, the signing and validation should use the appropriate sign() and validate() components instead of duplicating the business logic. Currently if the error was caused by time drift you will not know.


Thanks for that :+1:


Please don't export permissions ... remove them from features.


There's `adamantium` gem which provides that + other handy things like `memoize` of method return values and "flat" mode for non-deep-freezing


I would agree but this is going to be read by someone using Windows so... maybe we could rephrase that :D


Почему бы не брать просто ай-ди из списка по очереди ?


yep thats what I thought you beat me to it, I have a slight cleanup Ill push it to your branch in a sec


typo


replace startAsTring != 'undefined' with startAsString != undefined.

Simply, you can have if(startAsString) { too.


My tip would be to use Sys_Mkdir instead and if necessary add a Sys_ function that checks if dir exists (if such functions doesn't exist already). Implementations on windows and posix systems may vary.


awesome... I think this could also be a nicer place to handle the work I did in #78 (i.e. an extra case to strip out the page url prefix if it's in there already)


Hi Yury,

Thanks for your pull request. Indeed better to be compatible with older php versions. I merged it into the branch and will be in the next release.

Regards,
Rik
Adyen


it would be better to rename this to something like playlists or playlistResource. It is a resource object which has CRUD methods.


Изменение касается только последнего метода. Затронуто всё содержимое из за смены кодировки на "UTF-8".


why u do dis?


I checked the code as today we send every time the record version and never negative value, so this part can be removed.


>I assume you didn't try compiling it on Mac :) ...

True ... compiling with clang 3.6 on Linux (only). No VS, no Xcode, no compromise ... :).

Works like a charm with Qt5.5, while Qt5.4 produced the exact same TONS of warnings (originating from the `Q_OBJECT` macro), that you saw on OSX/Clang. Those should be fixed.

I respect your decision. I'd just not recommend it and rather write a bug report for each and every warning to get it *really* solved!


Yeah, thanks, I know about it. But in such a simple case as this gem, `adamantium` seems to be an overkill. It adds to much complexity to objects IMO.


Please report this upstream: https://github.com/h5bp/html5-boilerplate/blob/master/src/css/main.css#L233


Thank you for this! Even if it is nearly a year old; it still saves lives! :+1: :smile:


Try check content of fragment_main.xml for existing the next line (in ListView control):
`android:id="@+id/listview_forecast"`


The extension uses grunt to build parts of it, which is why I could not publish the extension directly via vcs/git. Please correct me if I am wrong.


Přidám, také preferuji použití { a }, ale zde to bylo napsáno takto, tak jsem nechtěl provádět příliš velkou změnu.


I think it is exactly it. Thanks for the reference. It is faster than asking networkx for the connected components and more importantly uses much less memory. Networkx uses Bread First Search which should be linear on the number of edges, but the constant seems too high.

Challenge: keep tracks at a global scale when not all of them fit on RAM.


begin
	select * from Employee left join Location using (EmpId)
union
	select EmpId, EmpName, EmpLoc from Employee right join Location using(EmpId);
end


I'd avoid `defineProperty` when it possible because AFIK it's not fast in JS VMs


@Electroid yo, I'll fix this. I can't assign things since I'm not a map dev, but take this as self assignation


Hmm


Máš pravdu :smile: . Jak jsem to psal, tak jsem si toho vůbec nevšiml, odkudsi jsem to převzal a jak jsi to upravoval tak mě to trklo.


ボタンタップ系のメソッド名は以下のように、何がされたから呼ばれるメソッドか、わかるような名前にしましょう
saveButtonTapped


The only really big change was between 83 and 95. Later on 05 introduced new keyword: interface and slightly changed rules about limited types [wasn't an issue in WORDS, in fact change was so obscure that I only once had experience problems caused by it - when "porting" code written in 85 to Ada 2005, resolving it took only 4 hours (for a scale of problem)] and Ada 2012 introduced another keyword: "some". As far as "problematic" changes only new keywords would be problem here.

Another thing is that WORDS uses "with Text_IO" (and similar) which is illegal in Ada 95 and later (due to introduction of child package and making all library units be children packages of Ada package) and may cause compile errors on compilers other than gnat which accept this form for compatibility. Resolving this one is not hard or time consuming - only bothersome.

As for evolution speed - it seems that Ada is actually evolving much faster than C (but slower than C++... which to me looks like is taking all possible features w/o doing any analysis on consequences). In fact we are starting to fight uphill battle against halting problem (with introduction of contracts and type invariants... next standard will almost certainly introduce loop invariants). The difference is that Ada community in general is trying to be backward compatible - doing best to minimize compatibility breakage.
Funny thing is that C code can become uncompilable just with change in tooling.


啧啧啧 都开始研究 react了，向大神学习？


Ok- removed debug code for now, will keep in mind next time.
Thanks.


SQL keywords should be in capital case.


After looking at Opus_Search_Result_Match I found out that getServerDateModified returns an Opus_Date object that was created with the value of "server_date_modified" from the Solr index. I think there is potential for confusion, but I can see value in the new approach. Just need to be careful here.


「霸占」是固有的漢語詞彙，也是該詞彙最初的字形（「佔」字後起，分化自「占」），收於《重編國語辭典》（修訂本）、《漢語大詞典》等官修辭書。雖與「OpenCC標準」不一致，但不應貿然刪除。維持原來的0詞頻即可。


クラス内で指定する"array"みたいな識別子の文字列は定数で定義しています

static NSString *const saveStringKey = @"array";


+1, Understood. Will take care from next commits


I don't mind submitting the reports, but my wrappers have 0 negative effects and 1 positive effect.
By the way, there was a bug report for the `Q_OBJECT` OS X warning on day 1 of 5.5 release, and it was fixed on the same day. But we still have to wait until 5.5.1 release in SEPTEMBER to actually get the fix. Go Digia!


i should have said Ada "loses backwards compatibility" more slowly than C, rather than "evolves faster".


anyway - please file tickets for the code hygiene changes you foresee as being necessary, so others can get an idea of where things might go


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


Because the next boss with the Highest HP shouln't be the current boss (which is the uNextBoss)


1. AtomicBoolean shutdownRequested is unused. Please remove it
2. Fix indentation of scnGenerator variable.
3. Inject scnGenerator instead of instantiating HBaseSCNGenerator directly. This will enable the producer to use any other alternative implementation.
4. Please rename HBaseSCNGenerator to something that conveys the SCN generation logic better like MonotonicSequenceSCNGenerator or some other relevant name.
5. Please add copyright notice to HBaseSCNGenerator (or its renamed class)
6. Please consistently use SCN in uppercase because it is an acronym - do not use Scn or scn.
7. Rename generateSepSnc() method - maybe you meant generateSEPSCN()?
8. The class comments in HBaseSCNGenerator refers to some earlier implementation - there can be none because this is a new class you are writing. So please limit the commentary to just what this class does and what shortcomings it addresses with using timestamp based SCNs - no reference needed for any "previous" implementation.
9. Please fix code indenting in HBaseEventProducer, empty lines in HBaseSCNGenerator.
10. Might be a good idea to add inline comment in HBaseSCNGenerator why localSCN of -1 or 0 is returned as-is.


We have too here because carbine sucks at managing the Laser in hardmode.


actually, the whole test needes to have indentation fixed :)


@wires : I think new API is ready! Deploy! :floppy_disk:


The no-chdir options gives worse results. My lib folder is copied inside this OS X application (dist/SickRage.app/Contents/Resources/lib/python2.7/lib/). With the relative path it look for the file with the PATH settings from py2app but now you explicitly say the lib folder is located at the parent folder of sickbeard folder. But this is inside a zipped folder so it can use that location.

For the moment I can revert your changes but I don't get why it has to be an absolute path location.


Loudrob the argument here then is "You did not get permission from Disney and you agreed to the terms and conditions when you played toontown now to use third party software or to modify the client in any way." All these sources are still respectfully the rights of Disney. You can say I didn't get permission but when the source was public not private it is there to be viewed and modified. Now if Disney were to be in contact saying no permission which I doubt as people like toontown rewritten with over 4000 active players is still going, I doubt they would stop this source code from been reused. The end of the day the toontown house source was branched off of the early leaked toontown rewritten source, and i bet you don't have proof that you could use their source to carry on your development. Correct???

I'm not here to bicker I did this in my spare time just messing around and improving what the end of development of toontown house made to firstly integrate widescreen  support and other features i have enjoyed that i have seen around.

Credits have been given where necessary and that's all i can do


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.738.11545726e6ee61091b0e5fe7a7b00a0a65ddb972](http://build.palaso.org/viewLog.html?buildId=59493&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:03:33


ok fine, that's where I was confused
I was unsure whether their links were dynamic or not, will fix it ASAP, thanks @Electrex


有什麼用呢


Ah, yeah, sorry missed that


yeah, i am good with this change =).


format line


@eliasdc if you don't move the lib folder, and use the --no-chdir, you have no issues. You are making this more difficult than necessary.

If you have a better way for me to do it, please explain.


Le nom du group n'est pas correct


format


empty line


This addresses the last point I made in the PR comments


You need some spacing here (we usually put it at the end of the lines)


大陸繁體標準這麼用。
而且八股文中與OpenCC規定用字有別的詞彙相當多，比如「布」表「佈置」、「發佈」義（頒布、公布）。

若是人工刪根本就刪不完，而且刪除這些本就正確的詞也不符合語言文字習慣。

正確的做法應該是讓髙頻字詞使用OpenCC標準，同時兼容其他正確的（或者既有的）詞形。

注：大陸、臺灣均用「頒布」「公布」，OpenCC標準中某些字不符合兩岸任何一地人民的使用習慣，若是連低頻詞都沒有而需要自己造的話就太不人性化了。


I've tried following the versioning of osrm in the cookbook metadata.
But you're right, an hint in the README wouldn't hurt.
I'm currently a little over capacity, so I can't really implement much on my own. If you have an improvement, feel free to file a pull-request!
I'll take care of updating and testing this cookbook using the current osrm version shortly.


it issue #9 not #8


works for me!


Thanks, I didn't notice that ;)


zusätzliche Verbesserung bei Auswahl der Wartungsvertragsnr. bei nachträglichen Serviceleistungen


Can you get an SSL certificate for visualcrypt.com and use server redirections, so these URLs won't need to be changed in the future?


It's a size thing....


change in 7/9


Not sure if you are aware of https://github.com/OpenRA/Chocolatey


好。
（但是大陸也用…能這麼算嘛？你可別想糊弄我）


o_O :+1:


fails when I pass 0


This is embarrassing...


Absolutely nothing, gd != imagemagick


Légyszíves töröld ezt a sort


```
git pull --rebase
```


Makes source code checkers freak out, but in such technical environment a little bit of emotion ain't derogatory. :)


I totally already fixed this! Twice!


それそれ（笑）改行2回しないと、前の行の続きなのか表の始まりなのか区別できないってところだな。


// Procedure with FullJoin. (MySql doesn't support full join commant.)
begin
	select * from Employee left join Location using (EmpId)
union
	select EmpId, EmpName, EmpLoc from Employee right join Location using(EmpId);
end


First let me clearify what I'm trying to do:
* With py2app I bundle all dependencies of SickRage and SickRage itself into a OS X application SickRage.app like an *.exe file in Windows.
* The script to run SickRage is located at SickRage.app/Contents/MacOS/sickbeard which is the python script to start SickRage.
* The dependencies and lib folder are automatically copied to SickRage.app/Contents/Resources/lib/python2.7/lib/
* The rest of SickRage is compressed and bundled in SickRage.app/Contents/Resources/lib/python2.7/site-packages.zip
* the site-packages.zip and Resource folder (which has a lib folder) are automatically added to the build path which is why it used to work.

What broke my current build:
* You look-up the location of the lib folder based on the file location of scene_exceptions.py. Normally a lib folder is searched on the class/build path.

So my question is: why do you prefer looking up the lib folders location based on the location of a certain file instead of the way it was before?


I know it's not a pattern that's used now, but you can use $(NULL):
```
    pdns.service.in \
    $(NULL)
```

That way you avoid changing the last (existing) line when you add a new line.


HTTP Server OS X/Server/Controllers/UserController.swift

Oh, I forgot to explain why I ditched Foundation! It's to make possible that the framework compiles for Linux when the Swift 2 compiler is released. I'm using only C libraries that are available in OS X, iOS and Linux as well.


Did you just forget to include "sorted" or "ordered" in the above should?


I was not, thanks for bringing it to my attention.

I've been cleaning up existing dormant packages and converting them to automatic packages so they stay updated.  My builder scrapes the github latest release and automatically builds and pushes a new choco nupkg if the version changes.

There hasn't been much activity with the chocolatey package ... how do you want to proceed?


This will only activate the service within the environment of the nagios server itself.
So lets say you have 3 environments, _test_, _beta_, _prod_ and your nagios server is living in _prod_.
You specify one service within the databag and say that it should be active in _test_.
This will never be active, only if you also create a nagios server within _test_. (but this might be just what you want)

Now I want to have the service also within _prod_, so you need to specify an other environment.
So you might want to make this more intuitive by having the option to specify more than one environment (use array or something)

But I think that it's better to update the readme and remove this option and work with the node['nagios']['multi_environment_monitoring'] option. Here you specify the environments that you want to monitor, so it gives you the possibility to exclude environments.


This is now similar to what I think Paul added to devel when I started working on this.


Thanks!


I don't like that one tbh. WriteFully sounds like this writes a full suspension bike (abbrev. fully ;)). Though i have no suggestion on how to rename it. Imo it seems a bit like a big change to an API just to squash warnings.


Our repository is pretty much unmaintained. Automation is very much appreciated.


asd


Are you sure that those added "> 0" checks don't alter behavior here?


Hello


close #4


close #5


Резюме — письменный или устный доклад, содержащий краткое (итоговое) изложение какой-либо информации


close #8


вывод )


fix #9


close #10


我說的是「大陸繁體標準」，即印刷繁體書時的用字標準。
「佔」在《異體字整理表》中作爲異體字被廢除了，所以印繁體書（除了字典）的時候不會出現他。

大陸的簡化字文字系統通常不在我的討論笵圍之內（因爲沒有可比較性）。

臺灣方面同時用「占」和「佔」表示佔用，詳見《重編國語辭典》「占」條下的辨析：
```
占與佔二字於ㄓㄢˋ音時用法全同，如：「占用」與「佔用」，「占據」與「佔據」。今據法律統一用語表以「占」為主，「佔」謹注明同「占」。
```

說白了就是「占」字表「佔據」義更古更正統，且現代兩地政府都規定使用它。「佔」字是爲了分化其ㄓㄢˋ音而造的，但始終沒有取代掉「占」字。
OpenCC用字的修訂原則是「能分則不合」，所以喜用這種分化字義的後起字。但是這不能抹殺分化前的字才是「主流用字」的事實。


:before_save не работал, вообще не вызывал кол бек.

когда я поставил вместо него, :before_validation, и дебажил  create!
то выбивало ошибки, что либо last_name or first_name is blank , то есть presence не работал..пришлось его заменить..


Should `Compat` be updated to reflect where those functions actually are?


close #3


This leaks. The result of strdup needs to be freed.


This revision is clean.


Should be "save" instead of enter key in the future


On Thu, 9 Jul 2015 02:59:19 -0700, Török Edwin wrote:
>
> On 07/08/2015 02:26 AM, Christophe Troestler wrote:
> > Shouldn't a dependency be required /iff/ the lib/exec is built? In case a section build is conditional, the dependencies should be optional (if the default property is false) — the install field is still true but the lib/exec won't be installed anyway.
>
> You are right, the `Install` field has nothing to do with build dependencies.
>
> There might be a situation where the `Install` field should be taken into consideration: build vs runtime dependencies for libraries.  If a library (and its reverse deps) are not installed, it should only be a build dependency.

I have added that.  The behavior may depend on flags but using their default value is probably good enough for most cases.


Of course I'm going to add OS X and iOS only features to the HTTP framework through Foundation if the developer is targeting these platforms. But I think most people will choose Linux, because most of all cloud services use this platform. And rewriting some foundation libraries, besides making them available to Linux, is a very good programming exercise.


1. This is not the proper way to set it up. SR is not a module/package.
2. The scene exceptions are not a package, and cannot be imported.
3. There is no "buildpath/classpath"

https://github.com/shotgunsoftware/python-api/wiki/Packaging-an-application-with-py2app-(or-py2exe)


Strange, since I tested all combinations locally and they compiled for me. Regarding the test: most of them should work at the moment, and those that do not will be fixed by the refactoring. It is clear, however, that I will not work on those in the next four weeks.

The `test_linearelliptic_cg_...` are supposed to be in working shape for quite some time now, though.


Eran voluntarias y yo demasiado vago para hacerlas. :P


s/which applies/applying/


@Electrex  any clue of any other hosting provider which can track download count?
And if possible FTP upload
AFH is the only one that does it, but then no direct links :/


Remove however, remove would. Be directive and short

    It requires you to use an XA transaction.


I am not understand the relation between XA transaction and the second part of the sentence "so by default messages...". It looks like an entirely different setting.


Hibernate ORM


could rename it to `is_tracked`?


`to_unicode(session.getTitle())` (all the new code, forms, etc use unicode but the legacy stuff uses utf8 strings)


@muzaffar21  @shabir
Kindly avoid checking in generated files ( .class files)
Use `git rm` to remove these files


Neither have I, but now that we have dyna being able to be 'programmed' by anyone that can type in an expression, I am sure that this will be tried in some way, lol

When I document uc($p) I will list these behaviors, so that it is a 'defined' failure.


what `eval $(/sbin/blkid /dev/block/mmcblk0p11 | /sbin/busybox cut -c 24-); /sbin/busybox echo $TYPE` gives?


why enumerate if you don't use the index?


Best I can do is construct the relative path from __file__ instead, after that its up to you.


defaults to false


The constants are now in the `Mmap` module and the plain `mmap`/`munmap` wrappers that work with pointers are gone (JuliaLang/julia#11280). It might actually work to put the entire `Mmap` module here, but I probably won't get around to doing that until I try to port code to the new interface.


It's not a big deal to me, I was just mentioning that it isn't strictly following [semver](http://semver.org/).


Ups, forgot about removing enumerate.


error handling or exception handling?


Thanks Matthias, those are very helpful. I added them to my working format doc <https://github.com/vadmium/python-altium/blob/library/format.md#sheet>. I guess the corresponding SHEETSTYLE numbers follow the same pattern.


The last executable file of the web server.


I think this would be much more readable:
```
dates = {unicode(contribution.getStartDate().date()) for contribution in self._conf.getContributionList()}
return sorted(zip(dates, dates))
```


I haven't released the HTTP framework yet. It doesn't even have a name yet, haha. That's why I still don't have a README explaining all these things. Actually, I was quite surprise you found this project so quickly. I'm glad you did it. As you said is very nice to exchange ideas.


The sentence reads like a tautology. Paraphrasing a bit: "option if you need work stored within the same tx however it will be done in the same tx" :)


Is `RouteHandler` handler needed here?


This is a setter method (It sets the value for the members right?)
use `setData()`


Then consider it done.  :)

I do like how you setup a repo for the packages. Do you want to give me push rights to that repo so I can automatically keep it updated? The source files for the builds are at https://github.com/dtgm/chocolatey-packages/tree/master/automatic/_output/openra


Actually... `return [d, format_date(d, 'short') for d in sorted(dates)]`


Done.


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


name it remove_extensions if it can remove multiple.


self should be aligned with `form`


By default the column anntoation names are shown even if the rownames are turned off and if the legend is also shown it looks rather nice. However, the column annotation names have different behaviour. All in all I must say that the behaviour of the annotation titles is a bit of a mess right now:) But it works for most common use cases. If you have good ideas to make it more consistent let me know. Probably should add some parameters to turn the names explicitly on or off, but I have not gotten around to do that.


@topherfangio - What about other vendor browser prefixes?

```js
$(element).css({
    "webkitTransform":"",
    "MozTransform":"",
    "msTransform":"",
    "OTransform":"",
    "transform":""
});
```

@see [Coding Vendor Prefixes with Javascript](http://www.developerdrive.com/2012/03/coding-vendor-prefixes-with-javascript/)


I think we can improve this section by using the use cases for tx I describe in section two of http://in.relation.to/2015/07/09/hibernate-search-jms-transaction/


Maybe a `tutum.yml`?


Don't use capitalized letters for variables/identifiers. Full caps are usually used for constants.
Read more about java identifiers.


Still no subfolders for the contributions..


Added you to https://github.com/orgs/OpenRA/teams/packagers which should give you the necessary rights. Maybe use git submodules here? I don't know best practices for NuGet/Chocolatey though.


In this commit 'getopt' is replaced by 'argparse' for the command line argument.


I suggest that rather than modifying the current instance it returns a new Certificate instance after:
* Copying the _asn1_cert to a temp variable.
* Filtering out the extensions from this temp variable.
* Returning a new Certificate instance made from the DER encoding of this temp variable.

An immutable class is significantly easier to test and use than one with state that affects all methods.


The `docker-composer.yml` file will do for a starting point.


You mean subcontributions?


I think I'm doing some sketchy stuff. I'll try to look into what I'm trying to do and try it better :-). I know the lib folder can be included in site-packages.zip but I changed this because txt files were not supported in the zip. But maybe I can work around that.

Thanks for your help and explanation. If i succeed in a clean and good way of building and updating a OS X build I'll be glad :) All this work because I wanted to add a tray icon like the one from CouchPotato and an Application icon.


This sort of change is a bit less than ideal
```
+        if (stats.n_tweets == 0) {
+            output << et.name << ": " << et.stats.n_tweets << "\t("
+                   << 0 << "% of total tweets)\n";
+            }
+        else {
+            output << et.name << ": " << et.stats.n_tweets << "\t("
                << 100*et.stats.n_tweets / (double) stats.n_tweets << "% of total tweets)\n";
+        }
```
because it is doubling the amount of places that must be updated in the future.
I prefer introducing a helper variable:
```
double tweet_ratio = (stats.n_tweets == 0 ? 0.0 : et.stats.n_tweets / (double) stats.n_tweets);
output << et.name << ": " << et.stats.n_tweets << "\t(" << 100*tweet_ratio << "% of total tweets)\n";
```


No... the folder structure was like this:

date/hhmm_contrib/folder/attachment


I think the javadoc ought to be "foo", "debug"


Good to see this code works for both ```uc($p)``` and ```lc($p)```

$ ../run/john -test=0 -form='dynamic=md5(uc($p).$s)'
Testing: dynamic=md5(uc($p).$s) [128/128 XOP 4x2]... PASS

```
$ ../run/john -test=0 -form='dynamic=md5(lc($p).$s)'
Testing: dynamic=md5(lc($p).$s) [128/128 XOP 4x2]... PASS

$ ../run/john -test=0 -form='dynamic=md5(lc($p).$s)' --list=format-all-details | grep Case
Invalid options combination or duplicate option: "-test=0"

$ ../run/john -form='dynamic=md5(lc($p).$s)' --list=format-all-details | grep Case
 Case sensitive                      no
```


It's OK to return an ASN.1 representation for a caller to modify. However, returning a reference to the internal object itself means the caller could modify it and break the class in interesting way. Instead, return a copy of this object (may be doable by re-decoding self.to_der())


Thanks, in that case could you replace the `_` on line 44 with `Not_found` ? This will prevent other exceptions from getting thrown away.


UniRx.SystemReactive.csproj の方に合わせて。
あっち、 UniRx.SystemReactive にしたのよね？こっちもそれに。


this might need an item in pending release notes.


I'll look for one. ;)


Good catch, thanks!


これ、改行コードどうなってる？
UniRxは基本がLFだけども。


(1) use braces to split over lines
(2) see comment above on returning a reference to a private field of the class.


Fixed both.


Sorry : I've merged in the wrong way.


Fixed.


Ok, I was thinking it would be easiest to move the auto source files for this package over there so you could update the nuspec info as you see fit and my builder will be able to immediately integrate it....not so much an issue with openra, but openra-playtest you seem to have more frequent releases.

fyi, here's my commit of the build files I mean 3167c3e5f286d6cde5b029281d6b7a7d84a42e32


Exactly this commit, didn't change that much, only made behaviour of external entities more logical, current documentation still applies.

At the release time however, i will through all commits and update relevant parts of the documentation if necessary.


Don't call private methods - either make it public or extract to a helper function to be used by this code and the Certificate class.


But why are the prototypes are still there in constr_SET.h? Wouldn't it be better to drop them too?


I think this should be disabled by default with `nFullRbfActivationTime = 0;`


Apart from my comment, fixed the rest.


I totally agree, I like the exaggerated square version.  Let's put it up!


Shouldn't it be the variable `undefined` instead of a string?


Sounds like a plan. I guess some documentation at [README.md](https://github.com/OpenRA/Chocolatey/blob/master/README.md) might not hurt as no one isn't really aware of this at the moment. Thanks by the way!


If my understanding is correct the two bullet points in the commit note describe unrelated changes, so ideally this commit would be split into two.


This large if/else clause is a smell that each of these blocks belongs in a method of its own.
(1) define is_precertificate that will simply check for the CT_POISON or CT_EMBEDDED_SCT_LIST OIDs. That'll make it clear what this condition is.
(2) Put all precertificate-related code in one function.
(3) Put all X.509-related code in a separate function.


This allows you to use your own or others presets from like npm right? I don't think we had any docs on that?


This should definitely fix the ROM switching bug.


I am seen this error:
```
/usr/sbin/biosdevname --policy=all_ethN -i bash: line 2: /usr/sbin/biosdevname: No such file or directory
```

Will do some more testing and let you know how it goes.


@strootman You mean, if you were keeping data in the */usr/share/elasticsearch/data* directory? How can it be conflicting?


to run phpt tests: pear run-tests tests/
RunRerunTest.phpt depends on RunNormalTest.phpt


i am not sure about this, this is by-design a virtual method. and is implemented by its child classes, `FileStore`, for instance, and is used by `PG::_has_removal_flag()` via the `ObjectStore` interface.


Ojo que en el copy paste te quedo el ] en lugar de la comma


Thank you very much to the difficult conclusion that.


And what was wrong with the old functions
1) the Standard doesn't define the encodeing of SET
or
2) the Standard defines the encoding, but the implementation was buggy
?


@alex-ab please consider for merge. Upstream issue is TUD-OS/morbo#4


Why was this done? I just started using this. Is it being moved to another repository or what?


@Wafer-Li  You are right, I forgot it yet.


@bensechrist here's the `ScheduleExpression` support, as requested


Don't use this kind of separating comments here


<img width="151" alt="screen shot 2015-07-09 at 8 38 44 am" src="https://cloud.githubusercontent.com/assets/11030914/8595787/1ea9420c-2619-11e5-8144-72ce27bc2c4a.png">
Hi,
I don't why, but in my mac OS X 10.10.3, Scratch 2.0 Plug-in For LM can not find the scratch 2.0. And all program scratch with leap motion can not  use, and all button of leap motion in Scratch 2.0 displayed in gray.
Can you help me ?
Thanks.
Li


We don't need this wrap scopes here, since our Gulpfile is small and clear already.


Built-in node packages should be at the top of the requires list


And this same section.


two syntax remarks:
- @threads and @started_threads variables, while having similare names, have dissimilar contents: the first one contains threads objects, the second contains thread identifiers. This is confusing and error_prone
- I'm umconfortable with the construct used in lines 225-227 (NetDiscovery) and 174-176 (NetInventory) as I'm still trying to figure what it does exactly, after more than 30s of reflexion. I'd prefer a simpler construct, using an explicit loop for instance, rather than embedding a grep construct inside another one.


tooltipParse


Ditto :)


Thanks! Will test this out asap.


What is it you are trying to do here?


@inikulin Is the Gulpfile structure ok (with //------, scopes)? We use it to separate building logical blocks.


@JoeCooper We have moved Fragging support to a new repository: https://github.com/MvvmCross/MvvmCross-AndroidSupport


Top level tasks should be at the end of the Gulpfiles


Denodeify exec here and use promise


Amazing work!

The size going from 1.9k to 4.4k is quite a lot so your aims to make a  ~35% smaller version is welcome :) Hope you can nail that!

I saw the size go to from 4.2k to 4.1k with the `-c` option. What does it actually do?

btw: there is a redundant `var` on [this line](https://github.com/riot/riot/blob/1d361fbfd2d53438cddf5e76942ad3a6f81753b0/lib/browser/tmpl.js#L369).

I think we should put templating on a separate `tmpl` folder with a README.md file to reduce documentation on the JS file. What do you think?


These structures hadn't been released yet, the methods are min_ver'ed


Thanks for the quick reply. Big sigh of relief.


I think that is more cleaner if you create a admin object and use admin.token, admin.password, etc...


The same, with user object


Do we really need this method extraction here, I think it's small enougth to keep it inline


В прошлом комменте, было сказано, переделать этот блок используя методы: "find_or_initialize" or "find_or_create"
http://stackoverflow.com/questions/18747062/rails-create-or-update-magic

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


Version 1.0.11 (which has this support) should be hitting Maven Central later this morning.


Using Windows Powershell. Sorry if the description looks bad.


Yo borre la GeneralList porque creo que no tiene sentido, era un wrapper del arraylist y no le encontre sentido, implmentalo con arraylist


It looks like [openra-playtest](https://chocolatey.org/packages/openra-playtest) hasn't been created on choco gallery yet. openra-playtest could just be integrated into openra so it would be something like `openra-20150628-playtest`  See https://chocolatey.org/packages/dd

Any hypen+text designates a prerelease build so playtest = beta = alpha = prerelease

Prerelease packages are not moderated but require `--prerelease` argument passed to choco

so to install latest stable: `choco install openra -y`

to install latest playtest: `choco install openra -pre -y`

Or would you prefer having a separate package? so `choco install openra-playtest -y`

Is it possible to install stable and beta side by side?  That might be a good reason to have separate packages...


There are some new methods + new mixin, I think, we should bump minor version instead of patch


NO. Use simple comments instead, e.g.:

```js
// Windows natives
...
// Mac natives
```


agree


I'll find someone who can.
I split the diffs on github. What is the file on the left vs the file on the right ?
left = old
right = new ?


> Is it possible to install stable and beta side by side?

It is not officially supported, but it often works.


I think I know a way to make it work for you, gimme a sec.


TeamCity Jarvis Framework :: CI-GitFlow [Build 23](http://build.prxm.it:8811/viewLog.html?buildId=3437&buildTypeId=CqrsFramework_CiGitFlow) is now running


I'm using https://github.com/OpenRA/OpenRA/releases/latest to get the latest stable.

Do you know if there is a similar URL to get the latest prerelease? Otherwise it looks like I could parse https://github.com/OpenRA/OpenRA/tags for `playtest-`


Can we remove the above line ending as advised on the ticket?


Kann man auch eine Response ganz ohne URLs verwenden? Fallen dann onAnswer und onHangup weg? Da scheint es auch keinen Test für zu geben.


Our website needs manual updates at https://github.com/OpenRA/OpenRAWeb/blob/master/lib/openra.rb Not sure if that helps you.


I meant tests as in buildbot. If you fixed the tests you broke that's nice and all, but we're not seeing that atm ;-P


As mentioned in another comment, trying to use perlcritic again to be "best practice" burned me ... so it's better that I use my own judgment in the future.

Ps. the error messages from TravisCI certainly were crazy looking.  I had trouble believing that a shipped core module wouldn't "just work".  Live and learn.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.1.0-unstable.10+10](http://build.prxm.it:8811/viewLog.html?buildId=3437&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 110 Build time: 00:02:20


Haha no worries :)


Can you give this a try?  I don't think React will bind a function in that way.


Intentional :p


@shadeven We use the other Preconditions for that

```!java
Preconditions.checkNotNull(name, "Table name cannot be null.");
Preconditions.checkArgument(!name.isEmpty(), "Table name cannot be empty.");
```


Obviously you want to avoid them. Nobody wants their users getting mad at the app when it crashes.


:+1: thanks


Btw, not totally related to the discussion here (or might be), the rpi2-jessie target vanished from `apt.xbian.org`.

Its dist packages are empty (e.g. http://apt.xbian.org/dists/stable/rpi2-jessie/binary-armhf/Packages)
The pool doesn't seem to have any `rpi2-jessie` directory on it (although it has an `rpi-jessie` that doesn't seem to be updated since February).


Related to #15


I assume you meant Status rather than Statis...


this should work. I currently do not have the time anymore to check but I'll report when I do.


Typo here.


This should go in your Application creation:


NICE!!!

> On Jul 8, 2015, at 10:00 PM, Adam Schwartz <notifications@github.com> wrote:
>
> Add first run tour @zackbloom
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/EagerIO/JSON.is/commit/26379b727d3dc40f2cb16173059993e7c643c052


I like the proposal, but please fix the typo.


Do you mean just Shell constants or to use the same constants in both Shell and Python code? The former doesn't sound overly useful. The latter could be done by having a .py file that happens to be valid Shell syntax, e.g.

```python
INIT_TYPE_SYSV="sysv"
INIT_TYPE_UPSTART="upstart"
```

would do the job, but flake8 would complain about missing spaces. Alternatively we could maintain the constants in two separate files (Bash + Python) or auto-generate both versions from a common definition. I'm not sure, though, if this would actually be better or just a different kind of bad.


If you want it to be disabled in debug mode (and I'm sure you want) use this instead:


it's not necessarily an event. it might be a session/contribution/subcontribution


It's changed, the problem was passing the right URL. That's solved now. Thanks!


bmnnm,


there's an improved one in indico.util.fs now - better use that one. also, `attachment.file.filename` is already sanitized. it's only titles where you still need to use it (with the element's id as a fallback)


Give it a better name, such as `expected_size`


Nope, not too annoying. After talking about it, I agree that listing the files explicitly is the way to go.


Again as my previous comment how sure are you that var1 will accommodate "trusted." + argv[2] ? you need to dynamically allocate memory to var1 using calloc() or asprintf() and later free at the end.


Resolves #11


The folder structure is still far away from the old one... *please* have a look at the structure used in ConferencePacker. I'm sure it can be done with much less code and in a cleaner way than it's done int here, but the folder structure generated in there was pretty good...


meaningful variable name please what is var1 ? what does it mean???


This identifier is still not [standard-compliant](http://stackoverflow.com/questions/228783/what-are-the-rules-about-using-an-underscore-in-a-c-identifier#answer-228797 "What are the rules about using an underscore in a C++ identifier?").
How do you think about to choose really unique names at such source code places?


Thank you very much.i got it!


I think you're better off doing something like:
```py
for i in range(5):
    game.player2.give(WISP).play()
    expected_size -= 3
    assert len(game.player1.deck) == expected_size
    assert len(game.player2.deck) == 25
```


LGTM


I see some things that could be improved:
* MonsterWorker/PCWorker/NPCWorker delegates appear to be wrappers around a Dictionary but with locking.  That means the dictionary access is multi threaded?  But I'm pretty sure they're only accessed on one thread; there's even a check that if the scanning is still happening that it should not try scanning.
** The auto-property should be replaced with Lazy<T>, this also ensures thread safety.. but may also be un-needed.  If you create a new dictionary in the static ctor, that should void the need of lazy - it's *going* to be created upon running the app, most likely very quickly.  Lazy doesn't appear to save much here.
** These classes entirely may just be null and void anyway, if they're a wrapper around a dictionary to ensure thread safety, just use a ConcurrentDictionary.  When you call the Raise methods, create a new dictionary there so none of the raise methods can mess with your existing dictionary.

* When loading up the existing entries, like so:
`var currentMonsterEntries = MonsterWorkerDelegate.GetNPCEntities()
                                                                             .Select(kvp => kvp.Key)
                                                                             .ToList();`
The method GetNPCEntities creates a new dictionary, you then take all the keys and create a new list.. This could be done more efficiently if the dictionary was owned by the ActorWorker class.. you can then obtain the current list by doing:
`var currentMonsterEntries = _monsterEntries.Keys;
_monsterEntries = new ConcurrentDictionary<uint, ActorEntity>();
// lots of code here
                                        case Actor.Type.Monster:
                                            if (currentMonsterEntries.Contains(entry.ID))
                                            {
                                                currentMonsterEntries.Remove(entry.ID);
                                            }
                                            else
                                            {
                                                newMonsterEntries.Add(entry.ID);
                                            }
                                            _monsterEntries[entry.ID] = entry;
                                            break;
`

But I'm also not sure a ConcurrentDictionary is required.. or any locking at all.  That's a lot of overhead, and this method is the only thing that accesses it, right?  Outside of 3rd parties, which should have their own dictionary instance anyway (but the way the events work now, each 3rd party has to share the same one.. that's another issue).

Sorry if this post is confusing, it's just a stream of consciousness.  But the main point to get is I think a ton of the locking and wrappers are extra overhead that isn't helping.


Can you please test it with an event that has:

- Material attached to the event
- Sessions (w/ material attached)
- Contributions (w/ material attached)
- Contributions inside sessions (w/ material attached)
- Subcontributions inside these contributions (w/ material attached)

I think the folder structure you are generating is still more basic than the one we had before...


Sorry about that, please use the lastest version instead :+1:


Resolves: #33


@hackedbellini
I noticed this too. Not only rpi2-jessie, a lot more. Maybe @mk01 rearranges the pools


Is it possible for us to do a lazy load here (load once from file if it doesn't already exist, otherwise read from memory)? I think that makes more sense.


should have expression to check this, "should(Array).containDeep([values])", also you can use ".not.containDeep()"


@Electrex found, sourceforge :)
http://downloads.sourceforge.net/project/ownrom/Sprout/OwnKernel/OwnKernel_sprout-5.1.zip?r=&ts=1436448994&use_mirror=nchc


containDeep()


UniRx.* に変更。


comment


line note


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.739.e123fca6a80984066f05e4c3d86b9082afbd1285](http://build.palaso.org/viewLog.html?buildId=59503&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:03:11


I think having a space here will still associate the roxygen block correctly, but to be sure it is best to remove it


your program returns 0 for success or failure! Please fix this.
Plus where have you freed attr_name ? memory leak!


maybe a "-" here missing ?


You could cache the value:
```ruby
def self.load_schema
  @schema_properties ||= JSON.parse(File.read(self.json_schema))['properties']
end
```