Okay, I will try
Thanks

> On 29 Jul 2015, at 15:11, Hervé BREDIN <notifications@github.com> wrote:
>
> I don't think this is fixing the loading problem.
> We might need to completely rewrite the video player directive to support partial video playing.
>
> I would suggest to add a function setCurrentTime(currentTime, callback) that would use the callback only when the video player really is able to read the video starting at currentTime.
>
> Or a function preloadTimerange(startTime, endTime, callback) that would do the same when the player really has loaded the complete [startTime, endTime] range.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Show-mee/camomile-web-frontend/commit/215ae4bb805133dea8e60b130158b5b480ff43a6#commitcomment-12419810>.
>


In the ddf-parent pom the maven-javadoc-plugin has skip=true but in these api modules that is overridden by skip=false so the javadocs *should* be generated.


@bjornfor np, now you know... that happens when a package has some custom builds and then ends up adding -lfoo without any -L to the .la files. So it may happen also for other basic packages that don't use pkg-config.


Yeah, agree - was irritating me, too.


The version number is totally freeform in upstream OSTree.  And in Fedora it appears like `22.$serial` e.g. `22.0, 22.1, 22.2...`.  So maybe just `\S+` ?


@alandekok

I Believe that this is redundant, because we just have the below code.

``
 149 #ifndef HAVE_PTHREAD_H
 150 #  define pthread_mutex_lock(_x)
 151 #  define pthread_mutex_unlock(_x)
 152 #endif
```


would be fun to have a category on `NSDateFormatter` that returns a `MTLValueTransformer`, since all this does is wrap the method call in a block.


fun => easier to reuse


Problem is, BSD's and darwin have a BSD make as default.


Could you please detail a little bit more your use case? Why can't you use the ```sonar.properties.erb``` template? Anything missing?


@vcunat yes that package adds index.theme which triggers the gtk-update-icon-cache from system build which creates a correct index-theme.cache.


Hello,

please have a look at the output of:

use Data::Dumper; warn Dumper($adp); warn Dumper($pd); warn Dumper($sum);

--8<-
$VAR1 = {
          '0' => '32'
        };
$VAR1 = {
          'Inquiry Data' => 'SEAGATE ST9300653SS     YS076XN0B6CL',
          'Shield Counter' => ' 0',
          'Non Coerced Size' => '278.896 GB [0x22dcb25c Sectors]',
          'Port\'s Linkspeed' => 'Unknown',
          'SAS Address(0)' => '0x5000c500473a8731',
          'Locked' => 'Unlocked',
          'FDE Capable' => 'Not Capable',
          'Predictive Failure Count' => ' 0',
          'Link Speed' => '6.0Gb/s',
          'Media Error Count' => ' 0',
          'PD Type' => 'SAS',
          'Enclosure Device ID' => '32',
          'Connected Port Number' => '2(path0)',
          'Slot Number' => ' 0',
          'Drive is formatted for PI information' => 'No',
          'Device Speed' => '6.0Gb/s',
          'Media Type' => 'Hard Disk Device',
          'Port status' => 'Active',
          'PI Eligibility' => 'No',
          'Foreign State' => 'None',
          'Sequence Number' => ' 2',
          'Drive has flagged a S.M.A.R.T alert ' => 'No',
          'Drive Temperature ' => '27C (80.60 F)',
          'Device Firmware Level' => 'YS07',
          'Coerced Size' => '278.875 GB [0x22dc0000 Sectors]',
          'FDE Enable' => 'Disable',
          'Other Error Count' => ' 0',
          'Drive\'s position' => 'DiskGroup: 0, Span: 0, Arm: 0',
          'Successful diagnostics completion on ' => 'N/A',
          'SAS Address(1)' => '0x0',
          'Sector Size' => ' 0',
          'Needs EKM Attention' => 'No',
          'Secured' => 'Unsecured',
          'WWN' => '5000C500473A8730',
          'Firmware state' => 'Online, Spun Up',
          'Enclosure position' => 'N/A',
          'Last Predictive Failure Event Seq Number' => ' 0',
          'Raw Size' => '279.396 GB [0x22ecb25c Sectors]',
          'Device Id' => ' 0',
          'PI' => 'No PI'
        };
$VAR1 = {
          'Disk Type' => 'SAS,Hard Disk Device',
          'encl_pos' => '0',
          'Power State' => 'Active',
          'Capacity' => '278.875 GB',
          'Product Id' => 'ST9300653SS',
          'State' => 'Online',
          'encl_id' => 0,
          'Vendor Id' => 'SEAGATE',
          'slot' => '0'
        };
--8<--

As you can see, some of the (somewhat) numeric values returned by megacli do have a leading space.

The comparison itself relates to those vars:

$sum->{encl_id} => 0
$adp->{$sum->{encl_id}} => '32'
$pd->{'Enclosure Device ID'} => '32'

$sum->{encl_pos} => '0'
$pd->{'Enclosure position'} => 'N/A'

$sum->{slot} => '0'
$pd->{'Slot Number'} => ' 0'

The version of megacli is:

--8<--
megacli -v

      MegaCLI SAS RAID Management Tool  Ver 8.07.14 Dec 16, 2013

    (c)Copyright 2013, LSI Corporation, All Rights Reserved.

Exit Code: 0x00
--8<--

Best regards


this should inherit from our `WMFApiResponseSerializer` to get error-checking business logic.


This is the copy of the target framebuffer used for shader blending (the "fbo tex".)  To make sure that we don't end up with bugs or errors deleting, it's unbound when no longer needed.  Maybe deleting it can be skipped.

-[Unknown]


ui was the hardest part: https://github.com/JetBrains/imperative-nodejs/commit/bb62fb7491a42356a238126992e1eb2ec5acff53


Thanks for the comments.

Since the changes were related with heterogeneous support I included them in a common commit. So the commit includes:
* Structure changes for 64 bit pointer handling (tEvent and tTimerHdl) and changes in the code to support these changes.
* Structure padding to maintain equal size across platforms.
* Reordering of the structure variables to avoid implicit padding.


This is only tested on darwin and cygwin, not freebsd. They have no strchrnul


https://github.com/kripken/emscripten/blob/667dcd241886fdb878e248d95d8e03abb09c80b7/src/library_async.js#L322 it might need an extra level of callback.


you might want to use `util::Dir` for that, i'm not sure if thats the best path representation at the moment^^


And needs to be, because it also deals with stencil-related blend state.

-[Unknown]


You want to call out the columns you want to fill with the following values. Here, you'd want `INSERT INTO appointments(date, time, patient_id, doctor_id)` if we go by the first row you specify.


Commit this


The API has changed:
now the second arg gives the maximal order until which the fit is done,
whereas previously it was the total number of moments in the final tail
(including the known ones)

e.g

suppose you know moments -1 and 0
previously, if you gave "4" as second argument, you would have computed
moments 1 and 2
now, giving "4" will compute moments 1, 2, 3 and 4

Thomas

Le 29/07/2015 14:37, Igor Krivenko a écrit :
>
> Dear Michel,
> |single_site_bethe| test of cthyb is failing after this commit (yes,
> it involves tail fitting). What is to be changed there?
> Cheers, Igor.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TRIQS/triqs/commit/6ca1348630957abe8d84077930284e198e0f39dd#commitcomment-12419092>.
>


side note, i've been thinking about having fetchers be generic by doing something along these lines. not sure if it's possible in AFNetworking to have each request have specific response serializer parameters. maybe using accepts or content type? at the very least, we can have a "factory" for response serializers that return a class-specific serializer.


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

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


onDataChange 方法现在官方上公布的插件，用了没反映，是不是源代码没有更新


troll startzone + orgrimmar = todo


Dang...that is weird. I just tried again, I changed nothing, just ran the same command and it installed with no issues. My bower is 1.4.1 like you said. No worries now...just odd :)


IMO `untrack` doesn't sufficiently convey the true intent: cancelation


	<?php
	$num1=$_POST["num1"];
	$num1=$_POST["num2"];
	$suma=$num1+$num2;
	?>

</form>

<?php if ($num1&&$num2)
             echo"La suma de los numeros ingresados es:$suma";
             else
         if (!$null&&!$num2)
         	echo"Ingrese los numeros por favor";
    ?>


i would omit the manual creation of the data ingestion status preferring the automatic one done by the 'populate-db' command. the SearchTestCase.test_data_status fails because the test json has changed. we should instead update the MD5 hash in this test


Jim:
It works with the line commented out because if nothing is set the library
will set it to :capi if supported by the system, to :ffi if CAPI is not
available, and to nil if none interface is available. Not Travis nor my box
(Ubuntu 12.04) seem to work with CAPI, so geos-ffi is used. All
geographic_item_spec tests are still green here.

On Wed, Jul 29, 2015 at 9:57 AM, James Tucker <notifications@github.com>
wrote:

> Hernán:
> What do you think is the reason for the need for FFI support under
> Ubuntu/Travis? Also, if it is needed, why can the line
> ‘RGeo::Geos.prefered_native_interface = :ffi’ be commented out (in
> ./config/initializers/gis.rb:3) and your process still succeedes, WRT the
> geographic_item_spec tests? Do you know of any guidelines for choosing one
> or the other?
>
> I know that we originally used the ffi-geos gem to accommodate the use of
> Windows dlls, and, until yesterday, just never removed it. However, wether
> because of new, more detailed tests [‘eq(P16_ON_A)’], or because now
> intersects returns an object made with a different factory
> (@projected_factory) that the one from which it came, our comparisons fail
> (spec/models/geographic_item_spec.rb:263)
>
> _____________________________
> James Tucker, Programmer
> Species File Group,
> Illinois Natural History Survey,
> Prairie Research Institute at the University of Illinois, Urbana-Champaign.
> TuckerJD@UIUC.Edu<mailto:TuckerJD@UIUC.Edu>
> 217-244-9416
>
>
> From: Matt <notifications@github.com<mailto:notifications@github.com>>
> Reply-To: SpeciesFileGroup/taxonworks <reply@reply.github.com<mailto:
> reply@reply.github.com>>
> Date: Tuesday, July 28, 2015 at 17:28
> To: SpeciesFileGroup/taxonworks <taxonworks@noreply.github.com<mailto:
> taxonworks@noreply.github.com>>
> Subject: Re: [taxonworks] Remove dependencies to FFI, resolve remaining
> geographic_item problems. (e9ab17f)
>
> Really not sure, maybe "brew install postgis", does it come with that
> supported?
>
>
>
> On Tue, Jul 28, 2015 at 5:13 PM, Hernán Lucas Pereira <
> notifications@github.com<mailto:notifications@github.com>> wrote:
>
> > Any idea of how you ended up with CAPI support? I'm trying to see if we
> can
> > have the same dependencies on Ubuntu and Mac.
> >
> >
> > On Tue, Jul 28, 2015 at 7:10 PM, Matt <notifications@github.com<mailto:
> notifications@github.com>> wrote:
> >
> > > matt@HammerTime taxonworks (master *$%)$ rails c
> > > Loading development environment (Rails 4.2.3)
> > > irb: warn: can't alias context from irb_context.
> > > 2.1.5 :001 > ::RGeo::Geos.capi_supported?
> > > => true
> > > 2.1.5 :002 > ::RGeo::Geos.ffi_supported?
> > > => false
> > >
> > >
> > > On Tue, Jul 28, 2015 at 5:06 PM, Hernán Lucas Pereira <
> > > notifications@github.com<mailto:notifications@github.com>> wrote:
> > >
> > > > What do you get on rails console?
> > > > (I added ffi back on my box, otherwise running the console is not
> > > possible)
> > > > 2.1.5 :003 > ::RGeo::Geos.capi_supported?
> > > > => false
> > > > 2.1.5 :004 > ::RGeo::Geos.ffi_supported?
> > > > => true
> > > >
> > > > On Tue, Jul 28, 2015 at 6:45 PM, Matt <notifications@github.com
> <mailto:notifications@github.com>>
> > wrote:
> > > >
> > > > > Seems to be. You can add the FFI gem back if needed, it shouldn't
> > > affect
> > > > > us. The problem for us was solved by removing
> > > > > RGeo::Geos.preferred_native_interface
> > > > > = :ffi, see
> > > > >
> > > > >
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#diff-1e906ea092519204c4e1f5a2d1c5a549L3
> > > > >
> > > > >
> > > > > M
> > > > >
> > > > > On Tue, Jul 28, 2015 at 4:40 PM, Hernán Lucas Pereira <
> > > > > notifications@github.com<mailto:notifications@github.com>> wrote:
> > > > >
> > > > > > Removing this gem is making Travis and my box fail to build. Is
> it
> > > > > working
> > > > > > fine on Mac?
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > > >
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407318
> > > > > >
> > > > > > .
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407409
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407737
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407804
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407858
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub<
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_SpeciesFileGroup_taxonworks_commit_e9ab17f393d8905dd4ca6199ba7609b0e5174d52-23commitcomment-2D12408116&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=87FiFUsLQn0J2UnE2s49rk_Ml7AdI6AGU3DZpzKZL1o&m=3L5s078sXB4ZRAzGc7AKPnN-bE81QdX4mN2hs2V7v_s&s=TtTNc8OrtWIoOGZrpejrmAhJ-J-ssDFDzZnAXwRvNtM&e=
> >.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12419498>
> .
>


Could we somehow make this dependent on the total number of angles so that existing users won't have to change their deconvolution macros? I.e. change the signature to take `nAngles` and `nTimepoints` as additional arguments and then use something like `"spim_TL" + decimalFormat(nTimepoints) + "_Angle" + decimalFormat(nAngles) + ".ome.tiff"` where the `decimalFormat()` function does something like

```java
private static String decimalFormat(int maxPlusOne) {
    int n = maxPlusOne < 101 ? (maxPlusOne < 11 ? 1 : 2) : (int)(1 + Math.log10(maxPlusOne - 1));
    return "%0" + n + "d";
}
```


use projectPath variable


Ca serait pas mieux de revert ca et mettre dans la doc/config par default le nom du role complet (ROLE_EL_CNIL2 etc) ?


Might have been a bower hiccup. Glad you got it working, and best of luck with your project!


<form method="post" name="suma" action="suma.php">
	Ingrese el numero 1:<input type="text" name="num1">

	Ingrese el numero 2:<input type="text" name="num2">

	<input type="submit" value="Enviar">
	<input type="submit" value="Borrar">


I know this code is clumsy - but I couldn't come up with something better.
If you don't like it one of the students has to take over.
At least it works. In principle this should clean all $$a subfields.
I couldn't figure out how to set up a unit_test for this convoluted change.

Browsing through records "year:2012->2016" ( I don't think old records are affected) and checking ["084__a", "6531_a", "65017a", "695__a", "980__a"] shows it's mostly subjects and keywords:
980: 2
084: 5
695: 7
653: 58
650: 155
I don't know how often this should run and whether more fields should be checked.
Would it make sense to check the full record on rare occasions?

If we clean keywords (6531_a, 695__a) catalogers could add them 'quick and dirty'.

I can't debug the processing with rules.cfg; the real bibcheck workflow doesn't work at DESY.
Should it run only on records that have the listed fields?


@vcunat Exactly. It works very well on my machine :)


Thanks!


That might be more of a pain then going all the way now. You can still build all your plugins as Func's, but anything that uses a plugin should be expecting an interface. This will make it much easier to upgrade one of your plugins to a struct in the in future. Just my 2c


itself.


folders:


I love your new comments...


its comment


hey


@JedWatson this is because of event pooling and calling onTap synchronously (or calling e.persist()) also fixes this problem.
See https://github.com/facebook/react/issues/3657#issuecomment-125681594

I opened my initial PR because of this issue


I know, but I don't care

On Wed, Jul 29, 2015 at 8:47 AM, Dennis S Lebhardt jr. <
notifications@github.com> wrote:

> When adding CandyDevices repos you can omit the CandyDevices from name and
> make remote cd instead of github
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CandyDevices/local_manifests/commit/570b6f9d79fddb8d84ade526c744656256b98360#commitcomment-12420631>
> .
>


Rather doing this here, pass the full thing to Template and make some change in template! :smile:


<b>Suggestion</b>: Adding nested lists


Thanks!! This make the game playable for more testing! :) +1


Hmm. Why not leave the commit message as `git merge` suggested? I was a bit thrown when reading the commit message of one of the merged ones, repeated as commit message of the merge commit...


I'll say the same about your next pull request.
On Jul 29, 2015 10:03 AM, "Paul Keith" <notifications@github.com> wrote:

> I know, but I don't care
>
> On Wed, Jul 29, 2015 at 8:47 AM, Dennis S Lebhardt jr. <
> notifications@github.com> wrote:
>
> > When adding CandyDevices repos you can omit the CandyDevices from name
> and
> > make remote cd instead of github
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/CandyDevices/local_manifests/commit/570b6f9d79fddb8d84ade526c744656256b98360#commitcomment-12420631
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CandyDevices/local_manifests/commit/570b6f9d79fddb8d84ade526c744656256b98360#commitcomment-12421002>
> .
>


Thanks for the notice.
Honestly, I don't really notice a difference and have just been following the general kanon about the flag. CM sets it to A53 as well (see Tomato (Yureka)).


5051_fix_local_win_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1777 of 1777 tests passed)

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


да, давай


This folder must never be modified. Here the engine stores  runtime data like process definitions, and jars for the classpath. it also contains all the default spring files and properties files


should be: @param fields: list of marc tags


same for `WMFArticleResponseSerializer`


Everything
configuration.


transliterate("Čižmy".downcase) => transliterate("Čižmy") => "Cizmy" <---> Problem
transliterate("Čižmy").downcase => "Cizmy".downcase => "cizmy" <---> Fix


Looking at that URL I have 3 response:

1. Good to have some sources for the algorithm that we could track down or mention (  CSound source code, Stilson/Smith CCRMA paper., Timo Tossavainen (?) version )

2. The "resonance" parameter seems to be what we have in this filter, but we have it labelled as Q.  Something doesn't smell right here...

3. I don't doubt that the instability is better, but I don't understand why it is better?


check length


> thanks for the input! Could you motivate why you prefer raw strings in generated code? Preventing code injection security issues? Misinterpretation of string escapes (like "\n" getting interpreted an extra time)? I cannot readily see any cases where things like '"${getterName}"..' would be affected. That said, it's certainly likely that we should use raw strings in generated code, but making the rationale explicit would we helpful.

This is to get around escaping issues in the generated code. The specific issue I ran into was we have a getter called `$` (to align with polymer js). In the generated code this becomes `"$"` which is not a valid string as the `$` should be escaped. The raw string method is a simple and most effective way I to fix these sorts of issues.

> On the type annotation changes: IntelliJ says "No problems found in Dart code", make check in 'tool' shows no errors nor warnings, and yes, it does run on the package bots (all green). But in general it seems to be cases where the current type annotation relies on an implicit downcast (one side of assignability), e.g., from Iterable<String> to List<String>. It should of course be Iterable<String> such that we avoid this unnecessary downcast. It would have been nice to have an option on the tools saying "No, we didn't mean those downcasts, please detect them and tell me". Who told you, by the way, dev_compiler? ;)

I also did not get any editor warnings, but when I actually ran the tests (in checked mode) I got errors.

> The ElementAnnotation vs. ElementAnnotationImpl case is different: ElementAnnotation does not have an evaluationResult (the analyzer presumably will be updated such that access to evaluation results does not imply dependencies on classes named ..Impl, but it isn't there yet), so we are forced to downcast. In fact, all occurrences of ElementAnnotation in 'transformer_implementation.dart' could be changed to ElementAnnotationImpl, but in the end we would just be moving the downcast around: element.metadata has type List<ElementAnnotation> at the front end of the data flow, and we rely on getting an evaluationResult so we must have an ElementAnnotationImpl at the other end. With your change I actually get a warning at line 623, so at least the argument type in the function literal on lines 622-626 must be kept at ..Impl.

Good point, updated.


if this should never be `nil`, enforce it with a designated initializer and assert there


shut up tim, all the other manifests have CandyDevices there

On Wed, Jul 29, 2015 at 9:04 AM, Tim Klein <notifications@github.com> wrote:

> I'll say the same about your next pull request.
> On Jul 29, 2015 10:03 AM, "Paul Keith" <notifications@github.com> wrote:
>
> > I know, but I don't care
> >
> > On Wed, Jul 29, 2015 at 8:47 AM, Dennis S Lebhardt jr. <
> > notifications@github.com> wrote:
> >
> > > When adding CandyDevices repos you can omit the CandyDevices from name
> > and
> > > make remote cd instead of github
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/CandyDevices/local_manifests/commit/570b6f9d79fddb8d84ade526c744656256b98360#commitcomment-12420631
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/CandyDevices/local_manifests/commit/570b6f9d79fddb8d84ade526c744656256b98360#commitcomment-12421002
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CandyDevices/local_manifests/commit/570b6f9d79fddb8d84ade526c744656256b98360#commitcomment-12421034>
> .
>


server.


The owner's $location is not necessarily the node's location in the production (and also on our docker as far as i understood). The owner-server knows the node's uri from the configuration files. How should we pass this info to the owner-client?


here.


Not mine


You can change the platform configuration in the `bonita-platform-community-custom.properties` file by uncommenting lines, and replace services implementations by adding them in the `bonita-platform-custom.xml`.


This path is also often used to save figures, so it's not completely obsolete


Here you specify which edition


This is just a(n) (anomalous) value for `date`, with no value for time. You wanted a date string, e.g., `'20150801'`, followed by a time string, e.g., '3pm'.


@zml2008

Can you go through and do this across the entire API (if this wasn't the last of them).


remove semicolon


tenants.


i was actually hoping we could use the commented out `NSURLComponents` method above. I have some refactoring of that in the DTCoreText stuff I'm doing if you want to wait until then


5051_fix_local_win_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


move comment to line above method


@amareesh here is the context / logger issue I am talking about...


These look like artifacts from a previous exercise.


Default files that are copied when a new tenant is created.  You can change the new tenant configuration in the `bonita-tenant-community-custom.properties` by uncommenting lines, and replace service implementations by adding them in the `bonita-tenants-custom.xml`.


I'd suggest asking Barnoy for a logo with a transparent background.


Sure, just a reminder to myself. Although I'll expand the comment to say that.


details.


shut down.


fyi @DanielePucci


missing `wmf_` prefix


@richmahn that usage makes perfect sense: you're using it as an installation. In that case it doesn't really matter whether you use submodules or not (although for consistency and ease of managing contributions to plugins I would say keeping them separate is better).

**But** in this usage it doesn't make sense to graft in the Dokuwiki commit history and @superdav42 should get a heads up: whatever you want to keep in extra branches for reference is fine but when making a Dokuwiki upgrade the entire changeset between last used version and whatever version you are installing should be one commit (`merge --squash` if you like, but only 1 commit) for the Dokuwiki code update then a series of commits for whatever modifications have to be made to the installation to get it working.


La oveja negra vuelve a ser rebelde


Thats c/c++ style, this has to be valid python and this is the syntax for a choice

(False case, True case)[test]


Content
**Must not be modified**.


La oveja negra es muy necia


is the 'string-scrub' gem missing by any chance?


Why are you splitting the sponsors to multiple lines? Showing all of them at the same line will make sure it'd look good on every device depending on the screen size and resolution.


I think a more general solution would be to use a `next_node_calculation` block here. This would cover the situation where you need to save the response from the first question on the calculator (which is something I've run into elsewhere).


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


height attribute is missing, and could affect page load time.


improve variable name, maybe filesParamterValue


#include <iso646.h>

And, or, not, etc are part of the c++ language, even if VS does not recognize them as tokens.

> On 29 Jul 2015, at 13:52, $B'4'e'b'R'Q'_'`'S(B $B'!'_'U'b'V'[(B <notifications@github.com> wrote:
>
> What is not? My VS can't compile this code.
>
> $B!=(B
> Reply to this email directly or view it on GitHub.
>


Here they are, the drawing functions. I know that would come in handy sometime :)


Fixed in ecdc980. Thx!


Not a d-bus expert either, but apparently the Agent goes in the same bus as the rest of the API, which is the system bus.
This change allowed me to successfully use the Agent [here](https://github.com/pcarranzav/connman-simplified/blob/master/lib/WiFi.js#L251) (the PR for that one is coming after this one is merged :) )


They are used by XMPP client
2015年7月29日 上午7:41于 Ola Bini <notifications@github.com>写道：Are these exported variables actually used outside of otr? If not, I think we shouldn't expose them.

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


@d-led `scrub` in this case is ruby method, see [ruby doc.](http://ruby-doc.org/core-2.1.6/String.html#method-i-scrub)


Awesome updates Ryan.

I am having an issue with the new modal link to the parent/child templates though. Modal is blank and I am getting this error:
```
Uncaught Error: Syntax error, unrecognized expression: a#_tab_family&modal=1
```


Sorry guys. I'm using Java 8 which does not report use of non-final variables in an inner scope. My bad... :(

----- Original Message -----

> @bbrodt this commit introduced compilation errors. I fixed them with de593e5

> Please do run mvn clean install -Dfull before pushing any changes.

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


Is it actual? (since LowMemoryWatcher removed)


Not implemented yet, something for the future.


In my opinion to do things properly we should go even further. Currently "Angle" means both angle and XY position. For example, if I record several XY tiles, they are still called Angle0, Angle1... A more transparent file name would look something like that:
spim_TL0{i}_Pos0{j}_Angle{k}, or even Pos_00{x}_00{y}.
But this would require more changes to the code - all functions that now take angleIndex as parameter would also have to take positionIndex(s). This is something I could try to do, but is it wanted? This way the length of the filename would change for everybody, also people who don't do tiling. Another option: make the "Pos" appear in the filename only if there is more than one position.
Yet another point: it's easier to work with data when they are not all called spim_, but the user can change the prefix (disadvantage: length of filename varies), or at least they are called spim{i} using some counter (then length at least stays constant).
In my opinion as a user the current naming scheme is a pain, but I'm not the only user.


@alawvt Anne - we'll leave this commit in here. These are changes that were done on the amandafrench/dspace and the VTUL/vtechworks repositories recently, including some fixes I took from the main DSpace project, and work on the VTMirage2 theme.  A commit in git is a snapshot of the whole project, and not just the modified files, so these changes are what was different from your out of date snapshot when we did the late pull.

Normally you'll pull in changes from the remote system before you make changes on your local system, and this won't happen again. We'll have to make sure that the importance of doing a git pull before starting on changes is in the documentation.

I'm still thinking that we may eventually want to switch to a model where we create a branch for each feature that we add or change, even small ones, and then initiate pull requests, so that we have a chance to review exceptional cases like this one.


Sick


Ugh, that's ugly but I guess better than what's currently on there.


Thank you for this. This has been getting to me for a long time.


Usage example can be found at https://github.com/SemanticMediaWiki/SemanticCite/commit/24cd046c1d3d270aec8cae2f02c7007f46542baa.


The purpose of this exercise is to acquire the `doctor_id` and `patient_id` values using `SELECT` statements. Suppose someone were to go in and increase all the `id` values in your `doctors` and `patients` tables by 1000. Would these statements still work?


Senhores,
Não  consegui instalar.
Descompactei no Raiz e transferi os arquivos, porém, não funcionou, ....no SIstema/Configurações dá página não encontrada - 404.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.850.f2e429259134d750b197659e4d7123d13da746e2](http://build.palaso.org/viewLog.html?buildId=60163&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:02:50


Nice. That's the kind of stuff (initialization) I was talking about when speaking about AngularJS' initializer. I just checked and services actually don't have built-in support for bootstrapping (like, automatically running an `init` function when a service is used in a controller and that controller is booted), so having an explicit call to an `init` method, the way you did, is the way to go. Good job.


Not me. Please @paskl47 we need a quick summary in the commit message!


The only change I expect to see here is the added module output, other changes seem random.


@jackfranklin please can you check again?

fyi @benmiller169 !


I think "idea-repl" would be a better name.


I was being sarcastic :stuck_out_tongue_closed_eyes:


+# khgsf
+lakjgsetdmcbsgfakspddjddvdvdnd
+La oveja negra era muy rebelde
y aun mas


To be consistent with what? dynamic_cast is used elsewhere in the scene view node code (I was following the pattern from RestoreScene), I seem to recall that the safe down cast caused problems at one point, but the git blame/svn annotate is masking the original due to a name change on the SnapshotScene variable.


If `offset < 0` then you are substracting it here.


Improve return, isn't well use two returns for best practices


use constant


@hlee @TreyE
Do we need a new param or could we set value in options hash itself?


We should use var == true so it's uniform over all types and it makes it more readable.

${(' checked="checked"', '')[sickbeard.LAUNCH_BROWSER == True]}
${(' selected="selected"', '')[sickbeard.DEFAULT_PAGE == 'news']}


Also it should be put under "module" group.


Make it version 1


fix_intel_compile_linux (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1777 of 1777 tests passed)

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


"util" module exports kotlin-runtime so there's no need to specify it here.


lack comments


Yes. I'm not sure that it is completely removed, that's why doublechecking is suggested here.


`deviceNotFound` -> `deviceSizeAliasNotFound`


@sagarlekar , @TreyE

Check there, the f object sometime can not provide the dob. That's the reason, I added another params for such situation.

That's also the reason, we have bug ticket. while update dependents, the saved dob will not display. Let me know, if you have better way to solve this problem.


Avoid using Style tag in header and keep it to CSS file only


This is a syntax error.  It needs to be enclosed in () or have the \ put back, right?


That's some interesting code in htcondor (redefining types and function pointers for things in our headers and libraries)

 The 7.9 package has not been moved to our stable releases yet, so it's not announced. I can grudgingly revert that change.


At first I hadn't enough things to do to require a real initialization. But now I begin to have a bit more, and it'll be even more later.

I didn't really think about the best way to do it. I just wrap them together in a function. I might call it later asynchronously.


Very helpful to get an idea for the concerns and mindset of the XDG folks.  Based on this, I've done an updated implementation in 1ce3968f9a8f36dabf82b419104536530ec8c5ac.


Do not use inverted comparisons there possible.


Thanks @ubuntudroid ~ you rock ^^


I think that breaking compatibility - it is not good. Now I, and everybody who uses ash, should replace config, but actually **nothing** changed.

Why not just use something like as following:
```go
base := args["--host"].(string)
if !strings.HasPrefix("http") {
	base = "http://" + base
}
```


use constant


missing break;


Sorry looks like it might have been a js caching issue. Seems to work great now.

Just wondering whether or not the links to templates from inside the modal should be disabled - if someone were to open a second or third nested modal which edited the templates that are from the main parent browser window you might end up with some issues if the main window was saved again after the modals are closed.


If you run the seeds file multiple times, does it add the post and the comment multiple times as well? I think it will here.


The original was how it's pronounced in Greek based on http://www.pronouncenames.com/pronounce/Hygieia, but anglicize version of this is Hi-gee-ya so merging the pull request


> Shouldn't this be reverted

It seems it should.
The commit introduces hardcoded strings.
https://www.mantisbt.org/bugs/view.php?id=19984


make a comment commit


@jamesreggio I know this code is old, nevertheless is highly related with [this PR]( https://github.com/stripe/jquery.payment/issues/161). If you want to support zepto as well, why cannot just use this line

```
$ = jQuery or Zepto or $
```

Ultimately, [Backbone.js does something similar](https://github.com/jashkenas/backbone/blob/f55b4a22458ad01a8246bc79081858c8a2e872dc/backbone.js#L31).


¿por qué hace falta incluir main.h?


Adrian, I haven't bumped the version up to 2.6.11 just yet, so you'll need
to do a Modules > Refresh, to pick up the changes to the JS files and such.
Once you do that, the error should go away.

On Wed, Jul 29, 2015 at 10:13 AM, Adrian Jones <notifications@github.com>
wrote:

> Awesome updates Ryan.
>
> I am having an issue with the new modal link to the parent/child templates
> though. Modal is blank and I am getting this error:
>
> Uncaught Error: Syntax error, unrecognized expression: a#_tab_family&modal=1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ryancramerdesign/ProcessWire/commit/f7222229ccbfefc3576ad0074eb31ddf4f27764b#commitcomment-12421307>
> .
>


ignore the commit message


This line is very long, for best practice reduce this line.


`ON a.doctor_id = d.id;`

Most of the way there.


yes, error. I just pushed new one.


`viewportQuery` -> `deviceName`?


Inverse comparison used to make 'landscape' default value in case of args[1] === undefined, or string that's not equal to 'landscape' or 'portrait'.


what doc are you reading that uses these defines?  Those don't look right in any way :)


That would also explain it, as I have been using VS2013


Why's that? Does it have a different name in VS2010? This should be enabled! At least make it conditional for VS > 2010


Make this Retina compatible


Just for verification, the only difference between this commit and the commit previous to the bulkedit change is the bulkedit configuration.

git diff 3a86f8312794cb91da8655cfa94 f6848308a4f9985b3eb25cbd5
diff --git a/dspace/config/modules/bulkedit.cfg b/dspace/config/modules/bulkedit.cfg
index aea7013..1550bf3 100644
--- a/dspace/config/modules/bulkedit.cfg
+++ b/dspace/config/modules/bulkedit.cfg
@@ -16,7 +16,7 @@

 # A hard limit of the number of items allowed to be edited in one go in the UI
 # (does not apply to the command line version)
-gui-item-limit = 1000
+# gui-item-limit = 20

 # Metadata elements to exclude when exporting via the user interfaces, or when using the
 # command line version and not using the -a (all) option.


(Note that my worry is mitigated a bit by the fact that Roadshow is an opt-in feature)


5051_fix_local_win_test (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1777 of 1777 tests passed)

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


Great! I didn't we had loaders in FA :) changed it: https://github.com/meurio/hub-client/commit/c468d2cb5a39a0f2f329ef310a78472a242655ef


Estaría bueno definir un name space. Algo así como:
TimTickTock_init()
TimTickTock_start()
TimTickTock_stop()
Te parece?


tsuika


Hi and thank you.

It would be great to have the ability to set different directions for `source` and `target`.

I want to set `right` on `source` and `left` on target, so horizontally aligned graph looks good.
I want to set `bottom` on source and `top` on target, so vertically aligned graph looks good.

Thank you.


safsdfad


`JSON.stringify(err)` can be taken out. That said, I'm fine with leaving these in as it makes debugging test failures a lot easier


The name of the field and of the corresponding flag seem misleading here. If I understand correctly, they manage repl output not input. Imo it is justified to give it a more specific name, say "repl.ideMode".


use `OSAtomic` here, can be called from any thread


Wow fail on the commit message...guess I'll copy it down here so markdown actually does something.
-------
Coordinates and multi-dimensional arrays are STUPID.
I got my x and y mixed up with my row and col

```
var matrix = [
    [1, 2, 3, 4, 5],
    [1, 2, 3, 4, 5],
    [1, 2, 3, 4, 5],
    [1, 2, 3, 4, 5],
    [1, 2, 3, 4, 5]
]
```

A muli-dimensional for loop would look like this
```
for (var a = 0; a < 5; a++) {
    for (var b = 0; b < 5; b++) {

    }
}
```

a would be the row number (y) and b would be the column number (x)

So doing `matrix[2][3]` is actually the coords (3, 2) where x = 3 and y = 2 (a = 2 and b = 3)

A little confusing lol...


Why use H3 and not H1?


Early returns / guard clauses are usually a good thing.
Why was this one removed?


go


Got it. When things get more complex we'll have to change :)


The plan is to switch GCC4 out for GCC5, correct?


fssafdssd


Header should not wrap all of the content. Learn when to use header, section, div, aside, footer, nav


Looks like the file is Idno/**Core**/Time.php but it declares namespace Idno/**Common**/Time.


No me queda claro para qué está esto :(


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


1: M111 unrecognized component name: legacy


1: D100 Missing docstring in public module
25:1: I100 Imports statements are in the wrong order. from invenio.utils.datacite should be before from werkzeug
61: D102 Missing docstring in public method
64: D102 Missing docstring in public method
77: D102 Missing docstring in public method
81: D102 Missing docstring in public method
113: D102 Missing docstring in public method
125: D102 Missing docstring in public method
151: D103 Missing docstring in public function
180: D103 Missing docstring in public function
248: D103 Missing docstring in public function
283: D103 Missing docstring in public function


36: D204 1 blank line required after class docstring (found 0)
45: D102 Missing docstring in public method
51: D102 Missing docstring in public method
78: D102 Missing docstring in public method
83: D102 Missing docstring in public method
109: D102 Missing docstring in public method
121: D101 Missing docstring in public class
122: D102 Missing docstring in public method
126: D103 Missing docstring in public function
186: D203 1 blank line required before class docstring (found 0)
186: D204 1 blank line required after class docstring (found 0)
186: D400 First line should end with a period (not 'I')
186: D200 One-line docstring should fit on one line with quotes (found 3)
190: D400 First line should end with a period (not '2')
190: D200 One-line docstring should fit on one line with quotes (found 3)
215: D102 Missing docstring in public method
228: D203 1 blank line required before class docstring (found 0)
228: D204 1 blank line required after class docstring (found 0)
228: D400 First line should end with a period (not 'I')
228: D200 One-line docstring should fit on one line with quotes (found 3)
232: D400 First line should end with a period (not '2')
232: D200 One-line docstring should fit on one line with quotes (found 3)
250: D102 Missing docstring in public method
262: D203 1 blank line required before class docstring (found 0)
262: D204 1 blank line required after class docstring (found 0)
262: D200 One-line docstring should fit on one line with quotes (found 3)
266: D102 Missing docstring in public method
273: D102 Missing docstring in public method
285: D203 1 blank line required before class docstring (found 0)
285: D204 1 blank line required after class docstring (found 0)
285: D200 One-line docstring should fit on one line with quotes (found 3)
289: D102 Missing docstring in public method


1: M111 unrecognized component name: docs


24:1: F403 'from idutils import *' used; unable to detect undefined names


The logic of evaluating isReadInputFromTerminal field should be 1. executed outside of try block. 2. be visually separated from creating a console reader (for example by putting some empty lines or extracting to a separate method)


23:1: I201 Missing newline before sections or imports.
24:1: I101 Imported names are in the wrong order. Should be MutableMapping, MutableSequence
24:1: I100 Imports statements are in the wrong order. from collections should be before import six
24:1: I201 Missing newline before sections or imports.
26:1: I101 Imported names are in the wrong order. Should be AndOp, DoubleQuotedValue, EmptyQuery, Keyword, KeywordOp, NotOp, OrOp, RangeOp, RegexValue, SingleQuotedValue, Value, ValueQuery
77: D102 Missing docstring in public method
83: D102 Missing docstring in public method
86:5: F999 redefinition of unused 'visit' from line 82
87: D102 Missing docstring in public method
90:5: F999 redefinition of unused 'visit' from line 86
91: D102 Missing docstring in public method
94:5: F999 redefinition of unused 'visit' from line 90
95: D102 Missing docstring in public method
102:5: F999 redefinition of unused 'visit' from line 94
103: D102 Missing docstring in public method
106:5: F999 redefinition of unused 'visit' from line 102
107: D102 Missing docstring in public method
110:5: F999 redefinition of unused 'visit' from line 106
111: D102 Missing docstring in public method
114:5: F999 redefinition of unused 'visit' from line 110
115: D102 Missing docstring in public method
118:5: F999 redefinition of unused 'visit' from line 114
119: D102 Missing docstring in public method
122:5: F999 redefinition of unused 'visit' from line 118
123: D102 Missing docstring in public method
126:5: F999 redefinition of unused 'visit' from line 122
127: D102 Missing docstring in public method
130:5: F999 redefinition of unused 'visit' from line 126
131: D102 Missing docstring in public method


26:1: I100 Imports statements are in the wrong order. from invenio_query_parser.ast should be before import six
26:1: I201 Missing newline before sections or imports.
31:1: I100 Imports statements are in the wrong order. from invenio.utils.memoise should be before from invenio_query_parser.visitor
35: D401 First line should be in imperative mood ('Return', not 'Returns')
78: D102 Missing docstring in public method
84: D102 Missing docstring in public method
87:5: F999 redefinition of unused 'visit' from line 83
88: D102 Missing docstring in public method
91:5: F999 redefinition of unused 'visit' from line 87
92: D102 Missing docstring in public method
95:5: F999 redefinition of unused 'visit' from line 91
96: D102 Missing docstring in public method
103:5: F999 redefinition of unused 'visit' from line 95
104: D102 Missing docstring in public method
107:5: F999 redefinition of unused 'visit' from line 103
108: D102 Missing docstring in public method
111:5: F999 redefinition of unused 'visit' from line 107
112: D102 Missing docstring in public method
115:5: F999 redefinition of unused 'visit' from line 111
116: D102 Missing docstring in public method
119:5: F999 redefinition of unused 'visit' from line 115
120: D102 Missing docstring in public method
123:5: F999 redefinition of unused 'visit' from line 119
124: D102 Missing docstring in public method
127:5: F999 redefinition of unused 'visit' from line 123
128: D102 Missing docstring in public method
131:5: F999 redefinition of unused 'visit' from line 127
132: D102 Missing docstring in public method


This won't work. pt.x should (almost?) always be < mapWidth -> negative value in unsigned value.
Better convert to int first.


1: M190 line is too long (51 > 50)


4: L102 copyright year is outdated, expected 2015 but got 2014
84:1: E402 module level import not at top of file
85:1: E402 module level import not at top of file


25:1: I100 Imports statements are in the wrong order. import logging should be before from datetime
31:1: I101 Imported names are in the wrong order. Should be ImportPathRegistry, RegistryProxy
31:1: I201 Missing newline before sections or imports.
32:1: I201 Missing newline before sections or imports.
33:1: I100 Imports statements are in the wrong order. from invenio.ext.sqlalchemy should be before from sqlalchemy
33:1: I201 Missing newline before sections or imports.
36:1: I100 Imports statements are in the wrong order. from .logging should be before from .models
37:1: I100 Imports statements are in the wrong order. from .checks should be before from .logging


merging pull request #100


24:1: E402 module level import not at top of file
25:1: E402 module level import not at top of file
26:1: E402 module level import not at top of file
27:1: I101 Imported names are in the wrong order. Should be CFG_BIBSCHED_LOGDIR, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_PYLIBDIR
27:1: I201 Missing newline before sections or imports.
27:1: E402 module level import not at top of file


1: D210 No whitespaces allowed surrounding docstring text
27:1: I101 Imported names are in the wrong order. Should be CFG_SITE_ADMIN_EMAIL, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.base.i18n
29:1: F401 'config' imported but unused
29:1: E402 module level import not at top of file
144:16: E126 continuation line over-indented for hanging indent
191:15: E126 continuation line over-indented for hanging indent
194:1: E402 module level import not at top of file
225:17: E203 whitespace before ':'
226:20: E203 whitespace before ':'
227:18: E203 whitespace before ':'
228:22: E203 whitespace before ':'
229:20: E203 whitespace before ':'
230:23: E203 whitespace before ':'
231:19: E203 whitespace before ':'
232:20: E203 whitespace before ':'
233:21: E203 whitespace before ':'
234:37: E203 whitespace before ':'
248:4: E131 continuation line unaligned for hanging indent
277:1: W391 blank line at end of file


24:1: I100 Imports statements are in the wrong order. import cStringIO should be before import tempfile
27:1: I100 Imports statements are in the wrong order. import stat should be before import time
28:1: I100 Imports statements are in the wrong order. import shutil should be before import stat
30:1: I101 Imported names are in the wrong order. Should be ImportPathRegistry, PkgResourcesDirDiscoveryRegistry, RegistryProxy
32:1: I101 Imported names are in the wrong order. Should be InvenioTestCase, make_test_suite, nottest, run_test_suite
40:1: E731 do not assign a lambda expression, use a def
125:5: E303 too many blank lines (2)
138:15: F821 undefined name 'reader'
186:9: B901 blind except: statement
196:9: B901 blind except: statement
213:9: B901 blind except: statement
240:13: B901 blind except: statement
258:13: B901 blind except: statement
273:13: B901 blind except: statement


Why not just return `size` here?


sdfasdfs


Yep, strongly disagree. Early returns & minimizing nesting levels are good.


:dart:


creo que esta variable podría ser local a la función que la utiliza


Out of curiosity, why do we use date in the recipe name? Isn't version not enough?


Got any example code that uses this?  Only reason not to merge this outright is that increases the memory usage of the usb layer for everyone.


Since in `resize` we use just the first size, why we not throwing away all viewports except the first here?


should be `'checkchange' => true`


Learn when to use `br`


> Another option: make the "Pos" appear in the filename only if there is more than one position.

I like that!

> Yet another point: it's easier to work with data when they are not all called spim_, but the user can change the prefix (disadvantage: length of filename varies), or at least they are called spim[i] using some counter (then length at least stays constant).
> In my opinion as a user the current naming scheme is a pain, but I'm not the only user.

I think that is a very valid point. If you feel up for it, how about a new text box for said prefix (default: `spim_`)? That way, we stay backwards-compatible but also support the biologist users much, much better?


It'll be released whenever Gradle stops being a derp and lets me build it


31:1: I201 Missing newline before sections or imports.
32:1: I201 Missing newline before sections or imports.
34:1: I100 Imports statements are in the wrong order. from invenio.ext.sqlalchemy should be before from sqlalchemy


I guess it works, but there are warnings in the current code that are not trivial to fix.
For me (VS 2010) these are:
9>..\..\..\src\CatapultStone.cpp(48): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\GameWorldViewer.cpp(43): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\ai\AIPlayerJH.cpp(53): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\buildings\nobBaseWarehouse.cpp(70): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\buildings\noBuildingSite.cpp(89): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\nodeObjs\noDisappearingEnvObject.cpp(53): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\nodeObjs\noGrainfield.cpp(48): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\nodeObjs\noShip.cpp(88): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet
9>..\..\..\src\nodeObjs\noSkeleton.cpp(42): warning C4355: this': wird in Initialisierungslisten für Basisklasse verwendet


Esta bueno hacer commits separados del código y de los archivos keil :)


`console.log` can be taken out


lol. copy and past.


You could do

```
  def project do
    [
      ...
      preferred_cli_env: [
        docs: :docs
      ],
    ]
  end
```


Doh thanks!


We are passing string '' to this parameter to get all possible devices for playground. Also, you can get multiple devices for single value of parameter, so it looks like ```query``` more than ```name```, because query means uniqueness.


hmm, why remove `font-awesome.css` by default?


update: or, if it's only once per _launch_, just use `dispatch_once`. we can revisit resetting safely later


1 => true


Hey, this broke deployments for Rugen. Why didn’t this go through a PR?


Emm, and the `default path` is.. ? Is there a hyperlink for the made screenshot?


This code is not easy to understand (there are actually 3 cases). Extract it into a method "getPrompt".


Can you create a group/content item/discussion/.. with a displayName set to a user's email address and assert they don't return in the response


Learn when to use `<button>`.
Use button image from sketch file


We are getting list of all devices to fill comboboxes in playground.


Are you sure that ___am.jar___ and ___socket___ don't lead to some FNs (False Negatives)? I think that also Malware can call am and also tries to use sockets. And we want to log this behaviour.


it could be a problem if using something like `wiredep` :(


Same as below.


I think it's time to create separate files for each action in the `src/`. E.g. `src/open.js`, `src/resize.js. And make `index.js` just an API hub.


Why the extra `$userVoice` service and why the leading `$` in its name? Would it be better to just make a provider for the UserVoice service and configure that directly?


This should not be `<h4>`. Figure out what is should be.


chapter 27 done


If we're installing the coala using the version it becomes redundant to check the version with VERSION - So, the next 2 lines can be removed


The purpose of this exercise is to acquire the `doctor_id` and `patient_id` values using `SELECT` statements. Suppose someone were to go in and increase all the `id` values in your `doctors` and `patients` tables by 1000. Would these statements still work?


Oui tu as raison, je sais pas pourquoi je pensais que le ROLE_ était juste ajouté côté client par le javascript, mais pas du tout en fait. Il y a le ROLE_ dans le header.

Je modifie ça du coup


oh, yea, indeed.. just too easy to copy things in c++


But that is a difficult task to divide between FPs (Logmessages cased by droidBot) and FNs. We should do that later with some evaluation.


this has not been necessary for more years than, even I, have been alive!


It's should be named `getDeviceSize`. And I think, since we will have `resize` in the separate file it should go there instead of utils.


This should not be `<button>`


I don't see it like that. E.g. catapult stone can be fixed by putting the event initialization into the constructors body. So let's better fix those and disallow compilation with warnings for better safety (otherwise one would miss a potential bug)


PROVENANCE!


we're actually using `i` here in a few places .. `each_with_index`?


Remove one level of indentation for this class


Exactly, now we have to add special config in in overrides section of `bower.json` to have FontAwesome included properly...

```json
"overrides": {
  "font-awesome": {
    "main": [
      "./css/font-awesome.css",
      "./fonts/*"
    ]
  }
}
```

But, previously it worked OOTB...


Yeah dude I was bored last night.


remove this php closeing tag


see also https://github.com/facebook/react/issues/4514


well it assures coala can be executed and the version number is uploaded
accordingly which was a bug we had before

2015-07-29 16:38 GMT+02:00 AbdealiJK <notifications@github.com>:

> If we're installing the coala using the version it becomes redundant to
> check the version with VERSION - So, the next 2 lines can be removed
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/8e4d62615c28ff3b6645911637c464410e074cb1#commitcomment-12422015>
> .
>


please remove spaces (see also rest of files in this commit)


Hmm. Seems to me that it would be simpler to do this when the names of the agents/monomers themselves are extracted, as this would eliminate the need to do the replacement in every assembly method where names are used.


That is quite an exotic use for enum class. Imo putting constants into KotlinConsoleRunner class would be better.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.851.5539648ab21afdf467507ee2f9673ab2d1115f00](http://build.palaso.org/viewLog.html?buildId=60164&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:02:41


since you guys are on windows is the no-line return an issue?


Give this a try...

`<!--[if lt IE 9]>
        <script src="./../bower_components/html5shiv/dist/html5shiv.js"></script>
        <script src="./../bower_components/respond/dest/respond.min.js"></script>
    <![endif]-->
    <!-- endbuild -->`


It's a style preference. I do use `===` when checking for `0`, `undefined`, or `null`, but otherwise, `==` works just fine. I like Angus Crolls' idea that style shouldn't be overly dogmatic https://speakerdeck.com/anguscroll/stop-being-perfect


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

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


Nah, github did that for me when i saved it.


Added some extra error logging as well. It was unclear where it went wrong when it fell back to `compile()`


being really pedantic, but sizeof(provenance_data_struct) would be nicer here - If someone decides for some reason to change provenance_data to a pointer, the size will change to the sizeof the pointer which is not what you mean..


Or wait. Perhaps it should be `logger.warn()` instead?


5051_fix_local_win_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (743 of 743 tests passed)

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


I'll have a look during the next few days. I think this is the kind of
thing I can manage.


You are also expected to remove the Remindable related functions.


BLAM FART...as Junie would say :8ball:


That's a good start, but some improvements are still required.


1. At the moment the system assumes that all `CycleCourseGroup` has `ProgramConclusion`. In fact, in the future the `ProgramConclusion` can be a candidate to replace `CycleType` so until it is replaced, it is necessary to configure all `CycleCourseGroup` with the corresponding `ProgramConclusion`.

2. The `ProgramConclusion` in this method from this line on,  is only used to get its description, so it assumes that all `ProgramConclusion` attached to all `CycleCourseGroup` share the same description, which replaces the degree type official name. I must say this is a somewhat shady, this should be solved with an official degree type name or by including the degree type name as part of the degree name itself, but for the moment we have to stick with this solution.


Maybe it would make more sense with (map (splicett (length G0)) GX) for GX in the conclusion.


"vf" property should be private and have a more meaningful name, say "consoleFile". It is rarely worth it to sacrifice clarity for brevity.


You should make this suggestion in the original PR. This is a straight merge from PR done by thenickdude


i just checked #6780 and #6227
all due to `wiredep` workflow..

it could be a problem if write both `less/scss` and `css` in main, when work with `less/scss` and `wiredep` workflow..

hmm...


Yes it should. Updated.


Change the label value for:
"Se souvenir de moi"


The screenshot image opens automatically after the action. Do we need to show hyperlink in this case?


@JamesDeFabia  looks good.


Put this on one line for better readability.


In the future, please use english

For reference on commit messages: https://github.com/tecknoworks/guidelines/wiki/Writing-good-commit-messages


Versão 15


Some corections requiered.


make the example as a comment when it comes to production environment


Ok, I probably told that it is better to use document.textLength. But if you need the text itself there is indeed no difference, you can use command.length() and inline "document" variable.


Info


ERROR


combine both phones to 1 image


Good catch, see PR #66. Didn't do exactly as you said, because the internal TextBox's setText and setValue does different things under the hood.


should be `textual_group_diagnostics`, you removed the trailing s


robocop should be restricted to dev and test environment.

This way, we don't need to install gems we don't use on production


Use better naming convention. Learn more about CSS naming convention.


@SiebelsTim Thanks! I have a diff out for that now.


I'd be curious to know if this is enough to keep any mention of webpack-stats out of the js on the production build. In theory the UglifyPlugin should remove it.


default path is ```test/playground/screenshots/{{browser.id}}.jpg```. There is no link to the screenshot, it's opening in image viewer on a server after capturing.


I will kill you. *If my lawyer is not fast enough.*

*You* listed *my* email address.

This will cost you. A lot.

Remove it immediately.


varExists is a cheetah keyword

in mako this needs to be:

% if not header is UNDEFINED:


yes these warnings do occur on vs2010.

so you can change the code (its better then, since this in initializer list is dangerous) and we can readd the WX again


escaping?


Hey @homebysix. One thing that's incomplete about this is that it shows groups which are not scoped in any policies. But groups may also be scoped in Configuration Profiles. Are there any other things that can be scoped to a group?

Also, this is completely ignoring MobileDeviceGroups!

I'm starting to wonder if it might make sense to instead of a `report` verb, having a bunch of subverbs like `--packages`, `--computergroups`, `--scripts`, `--mobiledevicegroups`, and a wildcard `--all`.


Learn when to use `<aside>`


saw approval from JeetKunDoug
at https://github.com/basho/riak_kv/commit/e56e471c46d025ba34022fdcd757cf0b783133f6


merging basho/riak_kv/bugfix/fd/RIAK-1850 = e56e471c into borshop-integration-1153-bugfix/fd/RIAK-1850


basho/riak_kv/bugfix/fd/RIAK-1850 = e56e471c merged ok, testing candidate = 635d45b2


can getMesh return const VboMesh*?


Do not keep the defined job here because anybody who uses the template will have to remove the 3 lines (and he/she most certainly will).

Please use the original comments instead, as they are descriptive and point the user in the correct direction


-1 for using this macro. Makes code less direct putting statements inside the parentheses, and it doesn't really allow adding a more detailed debug message. Later we need to tell what's wrong with the JSON file.


I would prefer to use the `classes` parameter on the tab for this


Rich, sorry that I wasn't clear here - you need to add the additional underscore to the file after the "V1.0.0.6.1" portion of the file name: V1.0.0.6.1__schema-create_laertes.sql.


yes I think so, it's used only to iterate through label in `labels.cpp`


missing `to_a`


errorNotification function is nice, but you probably should distinguish between environment failures like "module not found" or "sdk isn't configured" and programming errors. In case of latter you should report errors using Logger infrastructure in Idea so we would actually know if someone experiences this problem.


Tara Rm Pum


Minor: I would take this comment off.


Since start/end timestamps are not needed to be injectable (unlike say NotificationService). It is best to just have these as public fields and not in constructor args. But they can be initialized in constructor.


I'm not sure. But currently I have no idea avoiding these FNs.

On Wed, Jul 29, 2015 at 10:39 PM, hannoL <notifications@github.com> wrote:

> But that is a difficult task to divide between FPs (Logmessages cased by
> droidBot) and FNs. We should do that later with some evaluation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lynnlyc/droidbot/commit/6ae90265d8cba41a9bbbb797718c9ef4f0516ec9#commitcomment-12422025>
> .
>


Please move metric_fu under development/test group. Don't need it on production.


This comment seems wrong. `_free_entry` is called in other situations.


This line is very odd. KotlinConsoleKeeper should report an error itself, doesn't it?


Fixed https://github.com/OmgImAlexis/SickRage/commit/e7449c8dc14148049ac09a65a85997ced069fe88


Does this return a string type? We are trying to get rid of returning *any* because we should know what it is we are returning.


the comment in a new line to let the `{` alone?


Yogurt is spelled wrong in mechanical resolution 1. Not trying to be mean, but I'm sure it would have come up eventually. Also, I am working on an attack pose. Where would I be able to upload that on this repository?


`i < nbColumns`


The property should private. HistoryMove class can be private inside KtConsoleListener (but that's a matter of taste)


So in order to synchronise the receiving and sending of data with multiple clients and server, an identifier is sent in the HTTP header so the server can identify a client's receiving and sending streams, and can bind a TCP connection to it appropriately.

The resolvedId in the server is the same as the clientId in the client. If you were to remove it (with care), you would end up with a server which can only serve 1 client, and no more than 1 as it won't be able to identify the sending and receiving connections between multiple clients.

If anything is stripping out HTTP headers (such as an HTTP proxy), it could also be causing issues with the tunnel.


finally!


天马行空，未测试的 code
数据库需要运行在有配置了数据源的 web 容器中。若需要单独运行，请重新编写 Manager 内获取连接的方法。


Minor: I would take this comment off.

When the functions are small and internal, a descriptive name should be enough to explain what it does. In this case, I think `_dup_json_str` is good enough, but it's fine to rename it so that you can get rid of the comment.


I think `QString::fromUtf8` is not need.


should be a val


I submitted this, but use the wrong email setting...


Does this function accept nulls for any particular reason?


Yes


In the previous version, we wouldn't try to load defaults when explicitly passing a path, but init now seems to do this regardless. Is that intentional?


Ah, ok. So this is more for "future proofing" once the `Forwarded` RFC makes its way through.


One Idea could be:
* We know when and which droitbot insert an Event.
* If we synchronise the timestamps of droidbox, we can mark FNs in the DroidBox.json with a tag as "cased by droidBot". In a graph they can also be shown in a different colour.

Another idea: If we can improve the droidbox image itself, so that it shows us, which process and threat has called current log-message, we can divide not only different threats of the malware. We also can differ droitBot.

But I think that is a issue for later. ;-)


https://github.com/yiisoft/yii2/commit/26e5b021a8904febe8dc3d8748777cde2dcf94a3#diff-74b1030c748d9da559378bf10fba0078R188

я так и не понял кому такое понадобилось


Minor typo: "Get the...".


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.407.e0464310577963d7486c032649f401baeb36f7f0](http://build.palaso.org/viewLog.html?buildId=60167&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3318, ignored: 130 Build time: 00:10:06
### Failed tests
```
Palaso.Tests.dll: Palaso.Tests.WritingSystems.DefaultKeyboardDefinitionIClonableGenericTests.CloneCopiesAllNeededMembers: <no details avaliable>

```


I'm not yet sure they are so Handy :(
I use them because I spend time figuring how to. But they don't add much.
They might be replaced by self-made sprites.


Keep the calloc so you don't need to set `entry->id` and `entry->type`.


lame.
1/10
would not push.
:sunglasses:


I've replaced all `MakeMyIsland` strings in b218e001de59ae24a0f40eada87fdb79e8f95525.


Minor: it seems strange that these two calls follow in this order. It feels natural when you setup things first and the actually run the thing.


good


Also note that this doesn't yet eliminate the error, because an error is still triggered on Monomer instantiation during assembly (not just Rule instantiation).

I've made a test in a branch, issue_19_bp_hyphens.


git update-index --assume-unchanged baystation12.int


I don't think that "writeback" is a word, is it?


Is it worth using `rails-api` here - https://github.com/rails-api/rails-api.  We've used this on a few other apps (eg content-store, url-arbiter).  Makes things a little lighter.


Why don't you check if parent.footer.uuid is null before copying it?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.852.5539648ab21afdf467507ee2f9673ab2d1115f00](http://build.palaso.org/viewLog.html?buildId=60169&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:04:36


"use to debug repl process"


The current master of govuk-rails-app-template adds the combined jenkins.sh instead of these 2 scripts.  Probably worth switching to that.


What iexactly "cutting some trash from path" means?


TeamCity FubarDev.AfA :: Test-Build für Pull-Requests [Build 1.0.1](https://build.fubar-dev.com/viewLog.html?buildId=44&buildTypeId=FubarDevAfA_TestBuildFRPullRequests&guest=1) is now running


Not necessarily. ```DefaultController.DefaultSimulation.stateInitial()``` asks the human player. This assignment makes the Human the player with more authority (going first and replaying decisions) and it forces them to be O, which there was no restriction on.


TeamCity FubarDev.AfA :: Test-Build für Pull-Requests [Build 1.0.1](https://build.fubar-dev.com/viewLog.html?buildId=44&buildTypeId=FubarDevAfA_TestBuildFRPullRequests&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 10 Build time: 00:00:17


@drdanz why this?
we'd need to have the name of this port unvaried since we rely on it from within iCubStartup...


yea, I caught it after I had made the other changes


What is the expected result here?


would it be worth saving everything into savedInstanceState instead?


unless there's a specific reason, it'd be better to change it back


ターゲットテーブルを識別する箇所、３箇所にバラついてしまったので、DRYにしたほうがいいかも。。


doesn't match always returns an array even if it doesn't match anything, in this case accessing the second element in the array will cause an exception


not used since https://github.com/GoogleCloudPlatform/kubernetes/pull/5219
Plus we have -c now


beforehand, this function usually returned `''` when not found.


Why don't you just clone travis-cache?  That would condense this to one line.


I think this should be `return this.withPathParameters(null, descriptors);`


also avoid irrational second time `_do_fetch()`.


Fails if data is None (so new_args and new_kwargs never get assigned to)


I don't think these comments clarify anything.


I think they should return it back: #142


I like the clear separation of configuration from the service. But yeah, either or works.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.853.5539648ab21afdf467507ee2f9673ab2d1115f00](http://build.palaso.org/viewLog.html?buildId=60173&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:09:52


:+1: it's much better this way.


Cryptic variable name.


Thanks for the explanation Hernán.  Seems we're all OK here.

On Wed, Jul 29, 2015 at 8:59 AM, Hernán Lucas Pereira <
notifications@github.com> wrote:

> Jim:
> It works with the line commented out because if nothing is set the library
> will set it to :capi if supported by the system, to :ffi if CAPI is not
> available, and to nil if none interface is available. Not Travis nor my box
> (Ubuntu 12.04) seem to work with CAPI, so geos-ffi is used. All
> geographic_item_spec tests are still green here.
>
> On Wed, Jul 29, 2015 at 9:57 AM, James Tucker <notifications@github.com>
> wrote:
>
>
> > Hernán:
> > What do you think is the reason for the need for FFI support under
> > Ubuntu/Travis? Also, if it is needed, why can the line
> > ‘RGeo::Geos.prefered_native_interface = :ffi’ be commented out (in
> > ./config/initializers/gis.rb:3) and your process still succeedes, WRT the
> > geographic_item_spec tests? Do you know of any guidelines for choosing
> one
> > or the other?
> >
> > I know that we originally used the ffi-geos gem to accommodate the use of
> > Windows dlls, and, until yesterday, just never removed it. However,
> wether
> > because of new, more detailed tests [‘eq(P16_ON_A)’], or because now
> > intersects returns an object made with a different factory
> > (@projected_factory) that the one from which it came, our comparisons
> fail
> > (spec/models/geographic_item_spec.rb:263)
> >
> > _____________________________
> > James Tucker, Programmer
> > Species File Group,
> > Illinois Natural History Survey,
> > Prairie Research Institute at the University of Illinois,
> Urbana-Champaign.
> > TuckerJD@UIUC.Edu<mailto:TuckerJD@UIUC.Edu>
> > 217-244-9416
> >
> >
> > From: Matt <notifications@github.com<mailto:notifications@github.com>>
> > Reply-To: SpeciesFileGroup/taxonworks <reply@reply.github.com<mailto:
> > reply@reply.github.com>>
> > Date: Tuesday, July 28, 2015 at 17:28
> > To: SpeciesFileGroup/taxonworks <taxonworks@noreply.github.com<mailto:
> > taxonworks@noreply.github.com>>
> > Subject: Re: [taxonworks] Remove dependencies to FFI, resolve remaining
> > geographic_item problems. (e9ab17f)
> >
> > Really not sure, maybe "brew install postgis", does it come with that
> > supported?
> >
> >
> >
> > On Tue, Jul 28, 2015 at 5:13 PM, Hernán Lucas Pereira <
> > notifications@github.com<mailto:notifications@github.com>> wrote:
> >
> > > Any idea of how you ended up with CAPI support? I'm trying to see if we
> > can
> > > have the same dependencies on Ubuntu and Mac.
> > >
> > >
> > > On Tue, Jul 28, 2015 at 7:10 PM, Matt <notifications@github.com
> <mailto:
> > notifications@github.com>> wrote:
> > >
> > > > matt@HammerTime taxonworks (master *$%)$ rails c
> > > > Loading development environment (Rails 4.2.3)
> > > > irb: warn: can't alias context from irb_context.
> > > > 2.1.5 :001 > ::RGeo::Geos.capi_supported?
> > > > => true
> > > > 2.1.5 :002 > ::RGeo::Geos.ffi_supported?
> > > > => false
> > > >
> > > >
> > > > On Tue, Jul 28, 2015 at 5:06 PM, Hernán Lucas Pereira <
> > > > notifications@github.com<mailto:notifications@github.com>> wrote:
> > > >
> > > > > What do you get on rails console?
> > > > > (I added ffi back on my box, otherwise running the console is not
> > > > possible)
> > > > > 2.1.5 :003 > ::RGeo::Geos.capi_supported?
> > > > > => false
> > > > > 2.1.5 :004 > ::RGeo::Geos.ffi_supported?
> > > > > => true
> > > > >
> > > > > On Tue, Jul 28, 2015 at 6:45 PM, Matt <notifications@github.com
> > <mailto:notifications@github.com>>
> > > wrote:
> > > > >
> > > > > > Seems to be. You can add the FFI gem back if needed, it shouldn't
> > > > affect
> > > > > > us. The problem for us was solved by removing
> > > > > > RGeo::Geos.preferred_native_interface
> > > > > > = :ffi, see
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#diff-1e906ea092519204c4e1f5a2d1c5a549L3
> > > > > >
> > > > > >
> > > > > > M
> > > > > >
> > > > > > On Tue, Jul 28, 2015 at 4:40 PM, Hernán Lucas Pereira <
> > > > > > notifications@github.com<mailto:notifications@github.com>>
> wrote:
> > > > > >
> > > > > > > Removing this gem is making Travis and my box fail to build. Is
> > it
> > > > > > working
> > > > > > > fine on Mac?
> > > > > > >
> > > > > > > —
> > > > > > > Reply to this email directly or view it on GitHub
> > > > > > > <
> > > > > >
> > > > >
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407318
> > > > > > >
> > > > > > > .
> > > > > > >
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > > >
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407409
> > > > > >
> > > > > > .
> > > > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407737
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407804
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12407858
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub<
> >
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_SpeciesFileGroup_taxonworks_commit_e9ab17f393d8905dd4ca6199ba7609b0e5174d52-23commitcomment-2D12408116&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=87FiFUsLQn0J2UnE2s49rk_Ml7AdI6AGU3DZpzKZL1o&m=3L5s078sXB4ZRAzGc7AKPnN-bE81QdX4mN2hs2V7v_s&s=TtTNc8OrtWIoOGZrpejrmAhJ-J-ssDFDzZnAXwRvNtM&e=
> > >.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12419498
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/e9ab17f393d8905dd4ca6199ba7609b0e5174d52#commitcomment-12420913>
> .
>


This line is no longer needed. You are not using OCCI here :-)


`dynamicDampingFactor` is not  a property of `OrbitControls`.


Thanks. I still don't want to `eval` it, though.


Why do you override hashCode and equals here?


Indeed it should. Thank you.


Double jump is not necessary here


Btw. you should run `make` to update `README.md` instead of modifying it by hand.


Minimax runs the risk of returning occupied board slots otherwise (boards could be bound to moves that are no longer valid). ```twoSolversTie``` get's stuck in an infinite loop without it.


yea


This change shouldn't be needed - convert_disk_image() already returns a no-op if the disk is already in the target format:

```
    curr_format, curr_subformat = get_disk_format(file_path)

    if curr_format == new_format and curr_subformat == new_subformat:
        logger.info("Disk image {0} is already in '{1}' format - "
                    "no conversion required."
                    .format(file_path,
                            (new_format if not new_subformat else
                             (new_format + "," + new_subformat))))
        return file_path
```


Thanks Allan. I went through the function extract_currdev(), just before returning it calls
env_setenv("currdev", EV_VOLATILE, i386_fmtdev(&new_currdev), i386_setcurrdev, env_nounset);
It seems like, env_setenv("curdev") sets correct value for environment variable curdev.


while I've had a github account for a while, I'm still new to git, thanks for the tip and I think I will do that


Yes, exactly. It was already accepted as RFC, we just need folks to start using it. Which should happen any time after 2020. :P


Thanks Allan. I changed the code.


aha!


GITHUB DID NOTHING WRONG


On not wanting to eval it; that's in the original code. So...


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2235


what is this all about?


Oh, wait, all is good.


Yeah, sorry, my mistake.


"os" is a common enough abbreviation but the name of the variable could easily be more descriptive, say "processInput", "processInputStream" or "processInputOS".


Hi Przemek,

Made my attempt with `tget.prg`: https://github.com/vszakats/harbour-core/commit/9749878869b2e29ac2f4006956a9bc843ad402f5
(and a subsequent bugfix)

`HB_SOCKEX` looks like a good choice 👍🏻

As for the prefix, I'd say let's leave it at `hb_U*()`. Reason is, I'm very distracted now to make long-term decision about this, for sure `hb_E*()` doesn't look good, maybe `hb_W*()` _may_ be better, but I don't feel it so much better that it'd be worth the hassle.


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

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


Compare this before duplicating the string, using `sol_json_token_str_eq`.

Put an "if (ignore) continue" before the if to make clear that we don't care about the other values being read by the json macro loop.

The block of comment above seems misplaced. Maybe put it either near the place you set ignore or the place you check ignore below.

Minor: maybe `duplicate` is a better name than `ignore`. Not sure.

Minor: consider calling


I was just glancing at the pr but sizeof(rec.u32)) != sizeof(rec.u32) looks like a constant.


I like the idea of avoiding unnecessary file copies when just editing the OVF descriptor, however I'm not a huge fan of this hack (the idea is that you set COT_WORKINDIR_OVERRIDE to be the same as output_dir?)

A better approach might be to do copy-on-demand for files instead - so if the input directory and output directory are the same, and it's an OVF for input and output (no untar/retar required) then files aren't copied at all.

Similarly, if the disk files aren't modified, then maybe we can copy those files directly from input to output rather than copying from input to working and then again from working to output.


Hmmm, strange.


This change does not belong in this commit.


Read up on enabling spring for rspec and consider following the setup instructions: https://github.com/eliotsykes/rspec-rails-examples#enable-spring-for-rspec


@roberto-casadei che pigrizia! :disappointed: intendevo una release come si vede, con release notes e *.nupkg scaricabile... :wink:


@dretrex These 4 files had been replaced in an old commit, they weren't in my *.pro file but still in the folder.    They had been replaced by IDEAS13ElementGeDetector and IDEASKetekDetector related classes.


Should these be using === to literally check for 0 and 1 or is == okay?


@roberto-casadei continuo a non capire che cos'è `Action0` anziché `Action`...


Okay awesome. Was this because of the issue I posted about not being able to see the middle slot?


addressed in #3642


It's cosmetic but this union is already localy defined above in a function. Could this be factorized ?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.408.07732cdf2f0395bd08d662d35fa6913753bed11f](http://build.palaso.org/viewLog.html?buildId=60176&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3319, ignored: 130 Build time: 00:06:53


please implement WIT comments


no check on the write return code here ?


Note: this might be wrong if ptr is a prefix of filename. E.g. ptr="luck" vs. filename="lu".

It seems since you are doing the `strlen` anyway, better simply use `strcmp`.


I don't think the first idea can avoid FNs, because the droidbox logs
generated (when droidbot inserts an Event) might be from a background
service or something.
Thus I prefer the second idea. :)

On Wed, Jul 29, 2015 at 10:58 PM, hannoL <notifications@github.com> wrote:

> One Idea could be:
>
>    - We know when and which droitbot insert an Event.
>    - If we synchronise the timestamps of droidbox, we can mark FNs in the
>    DroidBox.json with a tag as "cased by droidBot". In a graph they can also
>    be shown in a different colour.
>
> Another idea: If we can improve the droidbox image itself, so that it
> shows us, which process and threat has called current log-message, we can
> divide not only different threats of the malware. We also can differ
> droitBot.
>
> But I think that is a issue for later. ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lynnlyc/droidbot/commit/6ae90265d8cba41a9bbbb797718c9ef4f0516ec9#commitcomment-12422661>
> .
>


Commented out code.


Sweet, thanks @sebflipper .  I've been very delinquent on documenting these docker images.


=== is php/js, not python xD
If you want to be more pythonic and follow pep8 you can do:
${(' selected="selected"', '')[sickbeard.POSTER_SORTDIR]}
and
${(' selected="selected"', '')[not sickbeard.POSTER_SORTDIR]}


> I'm having trouble mocking up the callback function

you could do that with http://sinonjs.org/


This comes from `ss := strings.Split(line, "@@")` but it's possible to have `@@` in "function prototype" on the line with line numbers.

For instance the heuristics that git uses for "function prototype" fails on T-SQL code and `@@` prefix is often used for global variables there.


Should catch a specific exception


This is causing my builds to fail. Apparently my compiler doesn't like calling it with an unsigned int and then an unsigned long int.

I'm not sure if that would change the functionality here, but it does resolve if you change 0u to 0ul.


Thanks. I changed all of them.


Could this exception be made specific?


Oh sure, good!


remove the result summary and result detail function


Felt like this was redundant to repeat since the selectors share the same names as the message bundle keys


Why do we need two ways?


This is logic is unclear from this code. Please, think how it can be refactored so this edge cases will be obvious from the code.


Also need to fix the "Release Tag" column


nit "the the"


This patch is ignoring the return value of the function is define below. Maybe checking that everything work could
potentially help to diagnose pesky hardware bugs.


Oh, lol. I moved from PHP to node.js and javascript. Still trying to wrap my head around Python, this is the first time I've touched it.

I may change them once I've got home done.
Still want to get most of the files completed.

We could always do a find and replace to remove == True. That's mainly why I put it at the end instead of the start.


Rats, I knew there was something wrong. Thanks for spotting it!