Yes, this is not right. Thanks for catching this.

Deleting mirror pod is a costly operation, since it'd need to contact the apiserver. We should fix the `getFullNameMaps` (and change its name) to return a list of `podFullName`s for this function to delete. I will upload a PR.


The wiki is wrong, I'll fix that. See:

https://github.com/galaxyproject/galaxy/blob/dev/config/job_conf.xml.sample_advanced#L462

- `registered_user_concurrent_jobs` - number of jobs a registered user can have active across all destinations
- `anonymous_user_concurrent_jobs` - number of jobs an unregistered user can have across all destinations
- `destination_user_concurrent_jobs` - number of jobs a user can have at the specified destination id, or across all destinations in a tag.
- `destination_total_concurrent_jobs` - sum of jobs by all users that can be active at the specified destination id or across all destinations in a tag


If you put a link to your name you need to add the URL at the end of the file.


Can we have one rewrite the world commit and then your new commit?  This is a mess.


I changed this to use the std::string it's ultimately assigned to and reserve() the space.


Weird, looks like a compiler bug. But this is a good workaround.


IMHO it was a good choice to un-inline those methods.

Interestingly, even though the file didn't change much except for white space, the difference was too much for github.  github refused to give me a diff view.


Javadoc please.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 12](http://teamcity.otenv.com/viewLog.html?buildId=1257380&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 12](http://teamcity.otenv.com/viewLog.html?buildId=1257380&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


Yes, that hasn't changed. Just the Angular $http request timeout.


Why not:
```Javascript
const BINDING='_binding';
const KEY_ID='_keyId';
for(var i = 0; i<length; i++) {
  this[BINDING+i] = this._createBinding(bindings[i]);
  this[KEY_ID+i] = this[BINDING+i].key.id;
  this.eventEmitterAccessors[i] = this._createEventEmitterAccessors(this.[BINDING+i]);
  if (length > 10) {
    throw 'Maximum number of directives per element has been reached.';
  }
}
```
Would this cause performance drop?


Nice solution!


@natefoo, amazing, thank you.


This one has 20000 games instead of 2000.


Good find about the non-portablenenss, though it probably needs to be %td to be portable. LL implies 64 bits; the Rasbperry Pi is a 32Bit, so %lx is what it needs to be there.


Delegate in Cocoa land is usually an object with a protocol.
I find this somewhat confusing...

What do you think if we change this to say (some ideas for naming):
`registerCancelledObserverWithBlock:`
`registerCancellationObserverWithBlock:`
`registerCancellationObserverBlock:`

Also - what if we would `typedef` a block that we are using here?
I know this is a simple void block, but creating this type on a local stack is a pain-in-the-... and typedef would help here.


Cosa hai pulito? (Oltre il readme riuppato)


Este es el bueno


Read message from file


This has been addressed in the [css-in-js](https://github.com/callemall/material-ui/tree/css-in-js) branch.


I was leaving a place for a log message, which I've just added.


I think `yield` is okay here for [the same reason I described below](https://github.com/bocoup/test262/commit/60d21a9e3e89b56a44e67ed8688e41f4e80c8a8f#commitcomment-10741927).


"Marathon" leads to crashes


For some reason this still doesn't work.

Here's the link.

http://emberjs.com/api/classes/Ember.View.html#toc_event-names


@cgwalters

Looks good. How about minor alteration. I wanted to highlight that for production use, dm.thinpooldev
is only good option.

logrus.Warnf("WARNING:Usage of  --storage-opt dm.datadev or dm.metadatadev is strongly discouraged for production use. Setup an LVM thin pool and  use --storage-opt dm.thinpooldev  instead.")


Try this to confirm that it fixes the problem:

```
diff --git a/pirate-get.py b/pirate-get.py
index 5bf5c50..7f4740c 100755
--- a/pirate-get.py
+++ b/pirate-get.py
@@ -668,6 +668,7 @@ def main():
         url = mags[int(choice)][0]

         if args.transmission or config.getboolean('Misc', 'transmission'):
+            url = '; echo aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa'
             subprocess.call(['transmission-remote', '--add', url], shell=False)
             subprocess.call(['transmission-remote', '-l'])

```

subprocess.call puts each of the arguments directly into the argv of the program it calls instead of invoking a shell.


not comma separated... should be able to specify -f multiple times so map to a util.StringSet

This was right in the original PR:
https://github.com/openshift/origin/blob/master/pkg/cmd/experimental/bundlesecret/bundle-secret.go#L70


Thanks... Although I've tried many different ideas, all my successful patches have been related to evaluate threats :)
Even if this test fails, I will try a few experiments around this idea... lets see how it goes...


I'm not sure using `std::string` is legal here either, actually.  I haven't been able to get a straight answer from google on that yet (lots of articles about what C++11 might do in this regard, but I haven't seen one that says what it does), but I think `std::string` can be implemented using copy-on-write, and that might get in the way of being able to safely write to its internal buffer.  Using `std::vector<char>` with `resize()` rather than `reserve()` up-front is probably the safest option.


blergh. thanks


ok, done.


Oh, oops. I see what you mean. That's okay I think because command is passed on the command line or from the config file and is not controlled by the pirate bay mirror.


git commit options


Nevermind. The issue is the % url part. Yeah, that's bad. I don't think there's a simple solution to that.


[Found it](https://github.com/bocoup/test262/blob/60d21a9e3e89b56a44e67ed8688e41f4e80c8a8f/test/language/arrow-function/syntax/arrowparameters-cover-formalparameters-arguments.js) (sorry)


Update: I just went ahead and did this in 3ed47565059b45d1cd9c64d2461671328f37dc36


I took this phrasing from the [`std::seed_seq` page on cppreference.com](http://en.cppreference.com/w/cpp/numeric/random/seed_seq).


This test is missing assertions


Git commit options


```\&3``` and ```3``` are different things.
We really want ```\&3``` here.


Read message from file.


`preferred.getPartitioningColumns().isPresent()` implies `hasPartitioningRequirements`, so, technically, you don't need to check both.

In any case, I'd add a `unpartitioned()` to PreferredProperties that returns true if partitioning requirements have been specified and the partitioning columns are empty


fixup


I found that it was hard to justify the use of `std::seed_seq` without using the term "entropy". I imagine a beginner would just think "Well does `std::random_device` already give me an unpredictable seed? Why do I need multiple of them?"


rax retry tests


Psst! Try-with-resources!


Reuse and edit message from specified commit


Assuming sha256 is secure, combining by XOR or truncating is the same (just as any *uniform* mapping to 160 bits, i.e. mapping where all 160-bit values are created from the same amounts of 256-bit values).


try again


Minor nit: The `Flags.` prefix is not necessary, and I'd put this val inside the def of `synthesizeCompanionMethod`, unless you intend to use it somewhere else.


Psst! Pre-Java7 compatibility!


Noted. We could do a combination of both.


Same thing here. We really want ```\&1```.


add two items: "ec2:CreateSnapshot", "ec2:DeleteSnapshot"


Add signed-off-by:Use specified template file.


Gabe Aproves


Ta uma bosta


Yeah and I can't find any indication that the host is used.

The 'id' is stored by [`getUserFromId(id)`](https://github.com/nandub/hubot-irc/blob/master/src/irc.coffee#L107)

This is called by createUser(channel, from) in https://github.com/nandub/hubot-irc/blob/master/src/irc.coffee#L112

Which could be called by the join listener at https://github.com/nandub/hubot-irc/blob/master/src/irc.coffee#L296

Thus the irc.js emit at the JOIN in https://github.com/martynsmith/node-irc/blob/master/lib/irc.js#L444 will pass message.args[0], message.nick which looks equivilant to channel and nick (or who)


Ah, I didn't quite understand your original intent. Now that I do, I can work it back in.


I'd write "companion" instead of "companion object", since this could be a companion class too.
Also typo: "dissapeared" should be "disappeared".


I have added following line in my ckeditor config file but it not work and I have apache not found error
config.filebrowserBrowseUrl = 'Barryvdh\Elfinder\ElfinderController@showCKeditor4';

when i set the url to 'vendor/barryvdh/laravel-elfinder/src/ElfinderController@showCKeditor4' i have a laravel error :

NotFoundHttpException in RouteCollection.php line 145:


This test is equivalent to another test in this patch: [`language/arrow-function/syntax/arrowparameters-bindingidentifier-concisebody-assignmentexpression.js`](https://github.com/bocoup/test262/commit/60d21a9e3e89b56a44e67ed8688e41f4e80c8a8f#diff-6e481a28254c972ad3401d4c48fd6cf8R1)


Every time you release a new project with Java 6 support, a puppy dies.


If the test runner is doing its job, an explicit directive shouldn't be necessary


good catch


So, the thing to do would be use the std::vector temporarily, (allocate the size, make the call to get the name) and then copy the contents into the std::string?


Maybe something like this?
```
void writeYes(int fd) {
  in rc  __attribute__((__unused__)) rc = write(fd, "Yes", 3);
}
```
Also write Note, the rest with attributes.


Well, considering it's something so small and simple, I might as well make it as compatible as I can.


I wonder if a corresponding `[noStrict]` test would be a good idea--one where the arrow function itself opts in to strict mode using a directive.


this is not enough: it only modifies this instance, but not the clones.
The same change should be applied to pfMetDQMAnalyzer and pfMetT1DQMAnalyzer


Oh, you are relying on that function to do both square and non-square, huh? Alright.


I am a little bit wary messing with the cassettes so I just wanted to confirm that this is what I need to change in the original cassettes?


Why not 2.1.6?


So in conclusion message.nick -> who -> from -> id.

I don't know why they bother storing 'user' and a `user.name`.


If this is the only place that this function is used. Could also inline the logic instead of creating the extra function.


It looks strange for me that there is two almost same value that is AVERAGE and empty string.  In some cases "ULTIMATEE" will be AVERAGE too, without warnings.

You expect fixed number of results. Why not to do it fixed?
May be make function for each TYPE and return something from list of constants?  Move constant handle to helper and make logic more simple.

```
#grades.get(get_ai_tag_grade(tag_list, "WEAPONS"), 1.0)
grades[AVERAGE] = 1.0
grades[get_ai_weapon_grade(tag_list)]
```


All sections of parameter must be specified.


does not matter either way and was not sure if travis supports it yet :D


The addition of the javascript function on lines 204-222 was originally performed on an outdated version of this file. No changes were erased.


Since you already depend on Boost, can you just do
```
std::string hostname = boost::asio::ip::host_name();
```
for simplicity?

(It may well be a bad idea to rely on Boost at this level -- I dunno -- but it would at least sidestep the debate here.)


Fair :D


If we're doing this we should probably do it in all the xTo* functions post-rename.


Sorry for answering so late. My target is to be able to calculate back the required income before tax, for a given income after tax. Before you go into the piecewise linear function the income before tax is reduced by the costs you have to get the income. The whole system is full of rules to round up and down to full euros. If I want to get the inverse function of all this, I need to be able to inverse also floor and ceil and practically they are, if you use intervals.


```
     <collection name="outputs_1" type="list" label="ReadGroup  BAM set (forward)">
       <discover_datasets pattern="(?P<name>.*)\_1\.fq?"" format="fastq" directory="outputs"/>
     </collection>
     <collection name="outputs_2" type="list" label="ReadGroup BAM set (reverse)">
       <discover_datasets pattern="(?P<name>.*)\_2\.fq?"" format="fastq" directory="outputs"/>
     </collection>
```


I'm not too keen on the magic `34`.  Might that be better as `pk.size() + 1`?


Yeah, if you need it in a `std::string`, probably best to have the C function write into a vector's internal buffer, then use `std::string`'s constructor that takes iterators to construct it from the vector.


instead of `--since=30.days`, it might be better to use `--not [upstream]` to filter the set of commits to only commits on the branch.


Home

News

Contact

About

<!DOCTYPE html>

SOCIAL NETWORK

NEWS FEED

PROFILES

ABOUT

SOCIAL MEDIA

WELCOME TO THE WORLD'S MOST VAST SOCIAL NETWORKING SYSTEM WHICH CONNECTS YOU AROUND THE GLOBE.
SOCIAL MEDIA IS THE WORLD'S FASTEST GROWING MEDIA.

ONLINE SOCIAL MEDIA IS THE GROWING DAY BY DAY IT PROVIDES THE FORUM TO INTERACT WITH DIFFERENT
PEOPLE AT THE SAME TIME.IT MAKES THE LIFE MUCH MORE EASIER AND ENTERTAINING.

Copyright © W3Schools.com

<!DOCTYPE html>

Scute baby


Thanks for __your__ fix!


This mod is still experimental and not ready for use online yet.


drop this and just put the directory syncs in the xTo* functions?


This definitely works. It seems like we could do something where the price is only added impact if the order is not limit, e.g.:

```
price = event.price
if order.limit:
    # check price_worse and return
else:
   # do simulated_price logic
   price += simulated_impact
```

This jumped out at me because the simulated_impact is set to the integer 0, though it is added to a float.


This only became set by passing ```-DperformRelease=true``` to maven; using ```-Prelease``` was not enough.


Function name should not contain underscores.
Also there is already a `jointTransform` method in util that takes a RigidBody (or really anything that has a field called `floating`) and the coordinate vector for that body, and returns the homogeneous transform. If you set `floating` to 2, then it returns exactly this (i.e. `jointTransform(struct('floating', 2), xyzquat)` is the same as `xyz_quat_2_tform(xyzquat)`). If the intended use is to call this on a coordinate vector corresponding to a quat parameterized floating joint of a rigid body then you should probably use that instead.


That I lost,
Is there anything else I should check?


Two small comments, but neither are blocking. /signoff


This strongly suggests another test which might be valuable:

```js
var plusOne = v => {
  v + 1;
};

assert.sameValue(plusOne(1), undefined);
```

`test/lang/arrow-function/this-isnt-ruby.js`


Have an idea to make this function bigger.


This is using the same code as the two 'Triumph of' enchantments. To me, this shouldn't be able to check for ties, but it does. I'm missing some of the logic involved.

It's only ever checking for the highest, how does this manage to get the correct player for ties? Is it just a handy fluke that it's always checking the opponent's permanents first?


1570-EnergyPlus-8-3 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


1570-EnergyPlus-8-3 (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

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


Having no min-height feels slightly more natural to me

no min-height
![image](https://cloud.githubusercontent.com/assets/1035299/7167178/4688d024-e3b3-11e4-8fcc-ffbb76d6c5a8.png)
vs  min-height: 60px;
![image](https://cloud.githubusercontent.com/assets/1035299/7167186/5f389622-e3b3-11e4-911f-0dccb7d59756.png)


1570-EnergyPlus-8-3 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


whoa. I saw the weird glyph and then when I came back it is now gone. :confused:


Michael,

This patch fixed the takeoff-to-waypoint transition issue on my flying wing perfectly. Great job, and thank you!


Wrap this for readability


This commit uses tabs for indentation whereas the whole file being modified uses spaces for indentation.


1570-EnergyPlus-8-3 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


any perf impact on this branch?

On Wed, Apr 15, 2015 at 3:06 PM, Eddie Hebert <notifications@github.com>
wrote:

> Two small comments, but neither are blocking. /signoff
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/quantopian/zipline/commit/7d37ecf1bd8e8e63a57b20271520dd7200ed75c4#commitcomment-10743056>
> .
>


thats my bad T_T


This test is equivalent to another test in this patch: [`language/arrow-function/syntax/arrowparameters-bindingidentifier-concisebody-assignmentexpression.js`](https://github.com/bocoup/test262/commit/60d21a9e3e89b56a44e67ed8688e41f4e80c8a8f#diff-6e481a28254c972ad3401d4c48fd6cf8R1)


Inline this variable


@jfkirk and @brianpfink are going to run it through performance regression before merging.


Going nicely.


You should be able to replace the lambda with node.getOutputSymbols()::indexOf


@rjaquino can you review this?


This should not be `static`, right?


This overload is needed too


Ah, did some reading up on join vs resolve after I saw this yesterday. Good catch!


`boost::asio` isn't on our whitelist of approved Boost libraries, but it's not on the blacklist either (and I don't remember the state of things in the middle).  In fact, I think it was added after the last time we made a modification to either of those lists.


Formatting is broken


This *should* be neglible since it only adds a cost during the execution of a trade, and then only iff it is a limit order.

(And just remembered that we need to cover more order types.)


If I were to be picky about quotes, I'd do double quotes everywhere.


I thought of that as well. But in that case the test should just check for the class name. And in that case it's even more clear that the test doesn't add much value.


>in groovy it's better to: ['blah', 'blah'] in e.errorMessages
`in` can be used to check if single ellement in collection.


sad switch :(


Do you need a return statement here?


Ok, my script to add header apparently removed the UTF-8 BOM. :stuck_out_tongue_closed_eyes:


This is really weird. Zips don't use commas so doing an underscore seems wrong.


I did not even know about this. Not sure it is more intent revealing to me, but probably is to newbies.


Removed.


I've actually ran into weirdness with rescue at the method level vs using begin. I can't remember now what it was though...


Ah, very good. I'll add this.


```java
Collections.sort(shards, comparing(ShardMetadata::getDataSize).reversed());


MainActivity.java
in line no. 809,1003 , please replace "editor.appl()" to "editor.commit()" and check if any crash happens.


Sample config.json file, my note @2 is not actual anymore taking into account file format and consistency reason (on Android this works the same)
```
{
    “windows”: {
         “debug”: {
                     “packageCertificateKeyFile”: “foo.pfx”,
                     “authorName”: “hello”
        },
        “release”: {
                     “packageCertificateKeyFile”: “bar.pfx”,
                     “authorName”: “kitty”
        },
    }
}

```


See https://github.com/JuliaLang/julia/issues/10838.


1570-EnergyPlus-8-3 (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Haha. Crazy the stuff that sneaks in on PRs sometimes. Who would ever do semicolons in ruby...


It took me some digging to find the policy on Boost -- I didn't realise it existed in that level of detail. For future reference, it's here: https://dev.lsstcorp.org/trac/wiki/TCT/BoostUsageProposal. I'm not aware of a version on Confluence; it should be probably be migrated.

The policy on libraries it doesn't call out by name is that they can be used after "design review". In modern usage, I guess that would mean an RFC. I think you should stick to using a vector here!


This feels less obvious. I'd rather see hash, query or something more full than changing the letters.


FIX 1. Correction of nodename Debtor_FindByName.
FIX 2. Correction of nodename DebtorHandle.
FIX 3. Error in search results with a single search result.


Why fail? I see no benefit for this.


Remove newlines


Check out Dialog.showError static method, I've already made a helper method for this


No, please squash it all together


Hello, thank you for replying.

 $amz=new AmazonFeedList("myStore");

what's the myStore means?

I didn't set any $store array in amazon-config.php file.

How should I defind?

Thanks.

On Wed, Apr 15, 2015 at 11:21 AM, Thomas Hernandez <notifications@github.com
> wrote:

> Double-check your config setup. Did you make sure to put the store
> settings in amazon-config.php and not amazon-config.default.php? Are your
> store's settings correctly indexed under the same store name in the $store
> array? If you have more than one store defined in your config, are you
> passing the correct store name to the Amazon object when you initialize it?
>
> If you still have trouble, try var_dumping the object to see what the
> property storeName is set to. It should be an array key in the $store
> array.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boostsup/phpAmazonMWS/commit/2d7627ab25cabcd5c7ae5a6ddd2a2b684ff2c42b#commitcomment-10742186>
> .
>

--
God bless you!
Joseph


comments


producer can be final.


Changed `$this->...` to `$product->...`


You mean myStore is my store name?

Like this one: Faith-Hope-Love?

I just need to replace myStore with Faith-Hope-Love?

Thanks.

On Wed, Apr 15, 2015 at 12:20 PM, Joseph zhu <zhubujun314@gmail.com> wrote:

> Hello, thank you for replying.
>
>  $amz=new AmazonFeedList("myStore");
>
> what's the myStore means?
>
> I didn't set any $store array in amazon-config.php file.
>
> How should I defind?
>
> Thanks.
>
> On Wed, Apr 15, 2015 at 11:21 AM, Thomas Hernandez <
> notifications@github.com> wrote:
>
>> Double-check your config setup. Did you make sure to put the store
>> settings in amazon-config.php and not amazon-config.default.php? Are
>> your store's settings correctly indexed under the same store name in the
>> $store array? If you have more than one store defined in your config,
>> are you passing the correct store name to the Amazon object when you
>> initialize it?
>>
>> If you still have trouble, try var_dumping the object to see what the
>> property storeName is set to. It should be an array key in the $store
>> array.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/boostsup/phpAmazonMWS/commit/2d7627ab25cabcd5c7ae5a6ddd2a2b684ff2c42b#commitcomment-10742186>
>> .
>>
>
>
>
> --
> God bless you!
> Joseph
>

--
God bless you!
Joseph


so... progress or process?


hier ist die datei


The multiline ones look worse to me like this than with the brackets.


nÃO GOSTEI DESTA ALTERAÇÃO


Test Comment


Did you submit this yet?


I prefer no space. Feels harder to read with more spaces around brackets. I look at brackets like parenthesis. We don't do `( query: ... )` so why do `{ title: ... }`.


Closes #3


Whoa, does this work? I would have expected you to have to write

```
if .Some(service.name) == lastSelectedDeviceName
```

Maybe that's not necessary because `NSNetService.name` is an implicitly unwrapped optional? Hm.


```
via AC_METADATA_URL environment variable
   ^ the
```


Please use Closer in the following pattern as suggested here http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/io/Closer.html:

```
 Closer closer = Closer.create();
   try {
     InputStream in = closer.register(openInputStream());
     OutputStream out = closer.register(openOutputStream());
     // do stuff
   } catch (Throwable e) {
     // ensure that any checked exception types other than IOException that could be thrown are
     // provided here, e.g. throw closer.rethrow(e, CheckedException.class);
     throw closer.rethrow(e);
   } finally {
     closer.close();
   }
```


fixer output? it may lead to thinking that this will be handler for output of single fixer like `strict`.
maybe `output` or sth like that?


I think things were getting reversed in some way that wasn't working anymore.


Hi, Paul,
Thanks for this very good work to improve ipal. This change will make the ipal interface look much more like the quiz interface, since it now has the standard question bank. In prior versions of ipal, I had used the standard question bank, but replaced it with my own version when the changes in Moodle 2.8 made it more complicated to use the standard question bank.
     The quiz module has now moved the list of questions from the mdl_quiz table (field questions) over to a new table, mdl_quiz_slots (as you probably know). In order to make the backup and restore functions in ipal work more smoothly using the Moodle utilities, I plan to introduce a new ipal table, mdl_ipal_slots (if the standard "mdl_" prefix is used with tables). This will impact a few of the changes that you made. I would like to involve you in this discussion, but it might be better to continue this conversation by email. If this is agreeable for you, my email address is junkinwf@eckerd.edu.
     Again, many thanks for this good improvement to ipal. As you can see I have merged your changes with the _MOODLE_28_STABLE branch and as soon as I get the backup and restore utilities working the way that I want them to work, I will modify the description to indicate that this version works with both Moodle 2.8 and Moodle 2.7.


`Closeable.close()` should allow throwing `IOException`.


no need for both methods ;)


`this.closer`.


Please remove this joke


There is another screenshoots file under the activityspy folder which included emulator image!


I suggest to replace `static_cast<SocketCanIface*>` to `static_cast<const SocketCanIface*>`, as the current implementation may break in future.


I think it's safe to simplify this test by removing both constructor definitions and this assertion.

Alternatively, you could re-define `B`'s constructor to invoke `super` from an immediately-invoked arrow function (similar to what you're doing in `B#incrementer` already). This would increase coverage over the behavior of `super` within arrow functions (i.e. "invoked as a function" and "used in a method invocation"). In that case, maybe a new test file is in order.


I would suggest just using `gobblin.metrics`.


Docstring? I'm not sure what this is supposed to be doing.


I was trying several different approaches to figure out what the compiler warning was about. Originally I thought it was to do with uint32_t not getting defined, but I see you actually included it in gpio.h so these lines can actually be removed.


It's fine to hard-code it for now, but let's discuss how to get rid of this and can you also add a `TODO`?


It probably is a compiler bug, g++/gcc doesn't complain bug clang/llvm does.


This is the line that I am referring to


Marcel

I have had a look at the code doing the call.  I am not sure that the change should be to that.  I think it is to make sure that the functions being called are properly defined.  There may be some SFINAE ambiguities which mean that the wrong routine is getting called.

Thank you for your help.

John

________________________________
From: Fletcher, John P
Sent: 15 April 2015 09:35
To: boostorg/phoenix
Subject: RE: [phoenix] lambda_tests1b1r and 1b2r updated for MSVC 12 and 14 (be6cf5c)

Marcel

Thank you for that.  As that is a change to phoenix core it may well have some other consequences.  I will do some experiments.

Thanks again

John

________________________________
From: Marcel Raad [notifications@github.com]
Sent: 15 April 2015 09:30
To: boostorg/phoenix
Cc: Fletcher, John P
Subject: Re: [phoenix] lambda_tests1b1r and 1b2r updated for MSVC 12 and 14 (be6cf5c)

You're welcome, thanks for the update!

Just one additional note: the offending call to boost::result_of is the one in phoenix/core/detail/call.hpp for call_impl::result_type. If this is changed to boost::tr1_result_of and BOOST_PHOENIX_DONT_USE_PREPROCESSED_FILES is defined, both tests compile.

�
Reply to this email directly or view it on GitHub<https://github.com/boostorg/phoenix/commit/be6cf5c307d76bec6156115aafb67377ecc49ef8#commitcomment-10730901>.


that's not how you do it ya @Daliaemad  check the tips or ask someone on the group.


By default, anchors should already get `cursor: pointer`. What problem is this solving?


Hmm... faz sentido. A implementação original é conforme as necessidades da PagPop. Para nós, não pode ser autorizada E capturada; são situações mutuamente exclusivas. Mas para Stonez, faz mais sentido seguir a lógica da Stone.


So far, Cardlayout,creating an account, reading data from a file, writing data to a file, and modifying logs(add or delete) are done.
But, they are not any exception handling yet. Also, the look of GUI needs to be changed.


I see


No the truth table was right before. There is simply an extra case here.

The fact is the truth table has a number of cases that cannot actually happen revolving around when `leftValue` or `rightValue` are 0 and the result is not.

Both of these are sufficient I think:

    If (leftValue <> 0 AndAlso ResultValue / leftValue <> rightValue) OrElse
       (rightValue <> 0 AndAlso ResultValue / rightValue <> leftValue) Then

    Dim s = (leftValue < 0 AndAlso rightValue < 0) OrElse (leftValue > 0 AndAlso rightValue > 0)
    If (ResultValue <= 0 AndAlso s) OrElse
       (rightValue <> 0 AndAlso ResultValue / rightValue <> leftValue) Then

The former has a different truth table for conditions where `ResultValue <> 0 And rightValue = 0` (all not possible states at this point of this method) and the latter is the same truth table.


content


Extra `s`, should be 'maxShardSizeInBytes`


I've noticed that the learnyounode solutions use `==` instead of `===`, and I'm not sure why, but it's best practice to   use `===`.

Reason being `===` checks that the two values being compared are both the same type and equal to each other. `==` converts the types to match, then checks their equality. The way the conversions happen are not so straight forward and can lead to unexpected bugs, which is why the `===` operator is usually recommended.


Can you explain what this is?


OK


I added that code in, but now I'm getting

```
galaxy.tools.toolbox.base ERROR 2015-04-15 19:26:22,990 Failed to load potential tool /tools_import/pcap_tools/biobambam_split.xml.
Traceback (most recent call last):
  File "/galaxy-central/lib/galaxy/tools/toolbox/base.py", line 772, in quick_load
    tool = self.load_tool( tool_file )
  File "/galaxy-central/lib/galaxy/tools/toolbox/base.py", line 806, in load_tool
    tool = self.create_tool( config_file=config_file, repository_id=repository_id, guid=guid, **kwds )
  File "/galaxy-central/lib/galaxy/tools/__init__.py", line 116, in create_tool
    tool_source = get_tool_source( config_file, getattr( self.app.config, "enable_beta_tool_formats", False ) )
  File "/galaxy-central/lib/galaxy/tools/parser/factory.py", line 26, in get_tool_source
    tree = load_tool_xml(config_file)
  File "/galaxy-central/lib/galaxy/tools/loader.py", line 11, in load_tool
    tree = raw_tool_xml_tree(path)
  File "/galaxy-central/lib/galaxy/tools/loader.py", line 43, in raw_tool_xml_tree
    tree = _parse_xml(path)
  File "/galaxy-central/lib/galaxy/tools/loader.py", line 231, in _parse_xml
    tree = ElementTree.parse(fname)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1182, in parse
    tree.parse(source, parser)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 656, in parse
    parser.feed(data)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
    self._raiseerror(v)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
    raise err
ParseError: not well-formed (invalid token): line 15, column 37
```

Are question marks allowed?


I agree %td makes more sense here.


Should this be at the end of the loop?


This approach allows for false positives (since there is no expected assertion count). You *could* ensure that the `catch` block was executed by using a counter variable, but `assert.throws` is more direct/readable (plus, it lets you specify exactly what type of Error you expect).


Hi @paulpilone, thank you for your answer and explanation about what exactly you want to accomplish with this feature. I think we should try to change your idea to something more comfortable in iOS, I think about to have only `duration`, and I think it will be more comfortable as parameter for extended `[ setProgress:animated:duration:]` method.

We can leave `setProgress:animated:` as it is, and it will use default constants `AnimationChangeTimeStep` and `AnimationChangeTimeDuration` when `animated` set to `YES`. For new method we can still use `AnimationChangeTimeStep` constant for timer and value update, but keep passed to new method parameter in some `_customAnimationChangeTimeDuration` and reset it on update finished.

What do you think about this? Is this implementation fits your countdown timer, or you need also timestep changeable?

Andrew


Yes - but < needs to be < an > needs to be >.


nodestime=600 must be set on both opponents: you are wasting results, even after you have been warned test is bogus


This is your commit. Is it needed?


Here's what colors look like at the moment:
![Lamp](http://i.imgur.com/zAUeeVq.png)

Also, I don't know scala to save my life...
Is it `Colors(15-m).c`?  or `Colors(16-m).c`?


Good, now the command is behaving like I expected it to.


Add parens


Instead of using

#if IOWIN32_USING_WINRT_API > 0

I think you should use

#if IOWIN32_USING_WINRT_API


	$ scripts/avocado multiplex examples/mux-selftest-advanced.yaml --tree --attr
	Config file tree structure:

		      /-intel
		     |
		  /cpu--amd
		 |   |
	       /hw    \-arm
	      |  |
	      |  |     /-scsi
	      |   \disk
	      |        \-virtio
	      |
	      |       /-fedora
	      |      |
	      |      |     /-6
	- -virt      |-RHEL
	      |-distro     \-7
	      |      |
	      |      |--gentoo
	      |      |
	      |       \-mint
	      |
	      |-env--prod
	      |
	       \absolutely-fresh--new_node


Yes


We have been eventing None for things that aren't set instead of the empty string. Can you change that?


asdf | asdf
--- | ---
asdf | jjjj
jjj | hhhh

* 22
* 33

asdf | asdf
--- | ---
Content from cell 1 | Content from cell 2
Content in the first column | Content in the second column


It seems impossible for this condition to ever be true given the check below.


No, updating it


Make sure you do your CHOREs!


Despite the fact that Zubax coding style recommends to use `#pragma once`, libuavcan should avoid it as it has to be cross-platform (this pragma is not part of any C++ standard). It should be replaced with conventional include guards, [as has been done here](https://github.com/UAVCAN/uavcan/commit/eb982bdd40e3b47bf436cec5c91aab5ac836b1d1).

For this file, proper include guard macro would be `UAVCAN_TRANSPORT_CAN_ACCEPTANCE_FILTER_CONFIGURATOR_HPP_INCLUDED`.


Typo: Service, not Servise.


I think you have a typo here. It should be _cancelFocusTimeout


OK


Why I lifted them - I do not want them to be recomputed for every function application.
So having it out of synthesizeCompanionMethod is intentional.


swizec, these move commits will be rebased and cleeaned up. I just had to push them, so i could debug the emails sending on app engine. the PR deos not include these yet.


Why you didn't accept my pull request? I added these rules with it.


Tak w ogóle, to piszę się "physics" a nie "phisics"... chyba że to francuski, to pardon!


после scandir ты получаешь массив всех файлов в переменную - ок. Тогда получается, что все файлы в твоей папке могут рассматриваться как страницы..(попробуй залогинившись, в ручную поменять ?page=logout). Если проверяешь на возможные страницы, то сами страницы должны быть вынесенны в отдельное место..например папку pages. И вообщем с 2-х ф-ций - это можно упростить до одной строки- проверять напрямую существует ли файл(страница) в папке(pages) ф-цией file_exists()


I think it’s because `lastSelectedDeviceName` is also an optional?


Don't you think it could be without nested loops? Like
```
    Object.keys(base).forEach(function(item) {
        numberGadgets += base[item].length?
    })
```
?


Yes (I meant to update this and the other yield param test after we discussed the _Cover..._ implications earlier today)


Sure


Unfortunately this introduced a bug.


This is not required, as this class doesn't use `uavcan::Array<>` anymore. All occurrences of `FilterConfig` can be replaced with `CanFilterConfig`.


I'll go with registerCancellationObserverWithBlock


@Tjatse whats your thought about this? The close event was never triggered :/ and was creating leak and never closed file descriptor


Name this `rowValues`


Clever.


This doesn't adhere to the coding style, plus there's an extra semicolon at the end of the constructor definition.

```c++
explicit CanAcceptanceFilterConfigurator(INode& node)
    : node_(node)
    , map_configs_(node.getAllocator())
{ }
```


OK


Note. At line 780 we do

    if ((DOWNLOAD_ENABLED != 'true') ||
     $stock_values->fields['product_is_always_free_shipping'] == 2 ||    (!$stock_values->fields['products_attributes_filename']) ) {

so the query should probably include *product_is_always_free_shipping* and *products_attributes_filename*


Regarding the std::string issue, this is the one I am aware of: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=21334 (see in particular comments 45 and 47). TL;DR it isn't legal for std::string to be implemented with COW in C++11. Note though that the fix for this will only land in GCC 5. Clang doesn't use COW for strings from what I remember, so it's not an issue there.


Yes, in the example scripts, "myStore" is the store name. This should match what you put in the `$store` array, as mentioned in the examples' comments. The default config file has "YourAmazonStore" in the places where you put your store's name.

And remember, if you only have one store in your config, you don't need to pass the store name when you initialize the object. It will automatically detect the one store and use those settings.


This is susceptible to false positives: if the engine *always* sets `new.target` to `undefined` for arrow functions, the test will pass.

Instead of testing the return value (and relying on the behavior of `instanceof`), it might be better to increment one of two counters based on the value of this expression. Then you're execution/validation would look like:

```js
F();
assert.sameValue(functionInvocationCount, 1, 'Function invocation');
new F();
assert.sameValue(newInvocationCount, 1, '`new` invocation');
```

but you could be even more direct by defining two separate functions to use for each assertion (maybe separate files, even.. who knows. we all live files here)


This is the same on local worlds.


I see, fair enough.


Thank you


This file doesn't use any assertions, so this inclusion can be safely removed.


@rj0495 Only if you use colors. It'll be fixed in the next build D:


Why not `const double NegativeZeror = -0.0` ?


не Access denied ,а хедер с ошибкой 404(page not found)


Jag vill ha en peer-review @palka285!


Is there any advantage of using this method to animate instead of using sprites? Sprites sound like an easier solution


Ok, that works.
It seems like its producing the two datasets collections.
But both are named 'BioBamBam Split on data 2' (its not catching the labels) and neither are select-able by the BWA-Mem tool. It simply says that now fastq dataset collections are avalible.


Ok cool. What is the official way to report Snapshot bugs?


Coding style violation - brackets are required (per MISRA C++ 2008). You probably should run uncrustify with [this config](https://github.com/Zubax/zubax_style_guide/blob/master/uncrustify.cfg).


Ok... why won't this remove the image that I created in the before block???

```Shell
....
An error occurred in an `after(:context)` hook.
  NameError: undefined local variable or method `image' for #<RSpec::ExampleGroups::Dockerfile after(:context) hook>
  occurred at /opt/boxen/rbenv/versions/2.1.0/lib/ruby/gems/2.1.0/gems/rspec-expectations-3.2.1/lib/rspec/matchers.rb:926:in `method_missing'

Finished in 32.91 seconds (files took 0.32936 seconds to load)
4 examples, 0 failures
```


@rj0495 Mojira, for example :)


These methods don't update the uncompressed size.


OK, for a moment I thought you were removing `get` entirely from the `AvocadoParams` class. phew.


Should be a double so 0.5, not 0.5f


Ok i have a large list so should I just email them to you or Add them to Mojira individually ?


This is a good feature


Random generation with puzzle-ish gameplay sounds like a recipe for disaster to me. I can't imagine how it could work unless we significantly overhaul the original concept (which would be fine by me).


Thanks for pointing this out. I added the change manually because we use Jade templating files.

Fixed in 1d60870


Why are you calling this function without arguments?


в пред.ревью я говорил, в емайле, в разделе общее, под одним пунктом я сказал, что при возврате на страницу index.php (если я был залогинен) меня должно перекидывать обратно, т.к технически я не разлогинился. Для этого, сессию надо включать сразу и в этом файле и делать соотвествующие проверки.


Originally I compiled locally (which failed unsurprisingly), then I cross-compiled (which succeeded).

However I've successfully compiled it on the pi without librt. As far as I can see all the thread related functions only use libpthread.


Also, Sorry for going off topic.


I'm confused about the process, here. Are we not using pull requests? It seems this was committed to the sdo-gozer branch directly, or did I miss a pull request where this was discussed?


The name thing sounds like a bug - I'll look into it.

Once the job is still running - you should switch to select two datasets in bwa-mem and then click the third icon to map over dataset collections for both inputs.


OK, equality test


Adding this call has added about 2.5 seconds of overhead to every call of our IKServer using the convex hull model of Atlas. We're going to have to take this out or radically speed up the `getTerrainPoints` function


return role.Has...


return role.HasKeyAccess(key, writeAccess)


no newlines :(


I see, make sure the `repr` and `str` methods are not generating exceptions.


This method provides smoother and more versatile movements. However if the action is as simple as running, you'd be fine with ~10-20 sprites and it would be smooth enough.


I guess this `const_cast` now can be safely removed at last.


```go
list := rw.Read
if write {
    list = rw.Write
}
```


This seems redundant given the contents of [` test/language/arrow-function/lexical-new.target.js`](https://github.com/bocoup/test262/commit/60d21a9e3e89b56a44e67ed8688e41f4e80c8a8f#diff-555b62da5ae1642af3c7edf6f404295eR1). This version relies on additional irrelevant semantics (object literal definition, method invocation), so I suggest removing it in favor of the other.


I very much doubt that this will be accepted :smile:


Static import


Why `this.Select.type` instead of `this.type` ?


Please document all these exported names - Client is a fairly generic name.


Add `@VisibleForTesting`


In jQuery/cheerio, you can loop over a result set using `.each` (https://github.com/cheeriojs/cheerio#each-functionindex-element-) which is a bit more concise and quicker to read than using for loops. Your code could look something like this:

```
permissions.each(function(i, element) {
    permissionsArray.push($(element).text());
});
```

But since `element` is very broad it would be clearer if we named `element` something more memorable like `td` or `permission`:

```
permissions.each(function(i, permission) {
    permissionsArray.push($(permission).text());
});
```

But there's another handy method in jQuery/cheerio called `map` (https://github.com/cheeriojs/cheerio#map-functionindex-element-) which iterates over a result set and returns an array that you build. The docs explain it better than I can, so I'll show you how you can modify your code to use `map`:

```
var permissionsArray = $('tr td:first-child').map(function(i, permission) {
    return $(permission).text();
});
```
What the above code will do, is find all `td td:first-child` elements, iterate over each one and returns its text, then `map` pushes them all into a new array which in this case you're saving as a variable in `permissionsArray`.

Caveat: I haven't tested the code snippets I wrote so they may be buggy. Refer to the docs for proper syntax, but hopefully the examples should help you see how you can use `.each` and `.map` in practice.


"This seems redundant" -> "This test file seems redundant"


1 параметр зачем ?, он у тебя описан в файле db.php. Лишний во всех ф-циях для работы с бд.


Smaller changes are sometimes made directly to release branches.


Also, why does this need to be an interface?


https://github.com/boostsup/phpAmazonMWS/blob/stable/amazon-config.default.php

You mean I need to add $store['YourAmazonStore']['myStore'] = 'xxxxx';?

Thanks.

On Wed, Apr 15, 2015 at 12:38 PM, Thomas Hernandez <notifications@github.com
> wrote:

> Yes, in the example scripts, "myStore" is the store name. This should
> match what you put in the $store array, as mentioned in the examples'
> comments. The default config file has "YourAmazonStore" in the places where
> you put your store's name.
>
> And remember, if you only have one store in your config, you don't need to
> pass the store name when you initialize the object. It will automatically
> detect the one store and use those settings.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boostsup/phpAmazonMWS/commit/2d7627ab25cabcd5c7ae5a6ddd2a2b684ff2c42b#commitcomment-10743644>
> .
>

--
God bless you!
Joseph


Hi,

sorry it was obviously not intentional, I stopped it.

2015-04-15 21:32 GMT+02:00 Marco Costalba <notifications@github.com>:

> nodestime=600 must be set on both opponents: you are wasting results, even
> after you have been warned test is bogus
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/Stefano80/Stockfish/commit/c24f70c3e3b62f403c31ca60a50b6ecc935b03d6#commitcomment-10743535>
> .
>


Could would you use `.map` here? You might also find `.attr` useful here.


What if `document.getElementById(addName1)` do not return anything?


The loop below will overrun the array if `filter_array_size > MaxCanAcceptanceFilters`. I suggest to add a explicit check with `UAVCAN_ASSERT(0)` and `return -ErrLogic`.


@mikedn Thanks, missed this. Updated locally and will be included in next commit.


I agree the description could also reflect that change...


Move this above `isFull`


lol


mmh, probably the pthread_setschedparam() is already covered in libpthread. I might have played with raw sched_setscheduler() before. So yep, maybe this is not needed - will try when I am at home.


The return type is `void`, right? (this will require a return type from this block of `id`.


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


merging basho/riak-ruby-client/bk-j308-typed-bucket-get-loses-it = 13af94ad into borshop-integration-215-bk-j308-typed-bucket-get-loses-it


basho/riak-ruby-client/bk-j308-typed-bucket-get-loses-it = 13af94ad merged ok, testing candidate = a9a9ef3a


This won't typecheck, because
``` Haskell
map :: (a -> b) -> [a] -> [b]
a :: Action :: World -> (Boid -> Boid)
map a :: [World] -> [Boid -> Boid]
```
You want to apply `w` to `a` first, and then map the result over `w`.
``` Haskell
a w :: (Boid -> Boid)
map (a w) :: [Boid] -> [Boid]
update a w = map (a w) w
```


I'm going to give this a tentative :+1: under the assumption that you've tested it.  We don't have a really useful test suite right now.

Also for things like this, I tend to look for similar code in yum/dnf as a reference point.  It looks like the closest analogue would be `def handle_upgrade` in `dnf/dnf/base.py` ?


This description is incorrect


Looks good to me.


Oops, yep


It looks like this might create a retain cycle.
E.g. Source -(strong)-> token, token -(strong)-> source.


@mikedn The code already existed as such and I left it as is. I will update locally and confirm that there are no issues with making this change.


Definitely a bug with labeling - https://github.com/galaxyproject/galaxy/commit/a0012b2df1c257b1cfecc1ff22569700c83cce01.


If it's in the stdlib why copy it here? Is it a go1.3 vs go1.4 thing? If so, document that.


Nit: `observerBlock` or `cancellationObserverBlock`


Можно так разделять PHP с html, но это не красиво и легко можно запутаться.
Существует [альтернативный синтаксис](http://php.net/manual/ru/control-structures.alternative-syntax.php) c помощью которого такие конструкции заметно легче читаются и код выглядит чище.


Page in biulding


I don't really have a code style guidelines, but not a big fan of things squished together like this.  Need me some white space!


How about "Comments, typically from users." ?


sgtm if you'd like to change it -


@NobodysNightmare great comment!


The CancellationTokenSource doesn't retain any CancellationToken instances - it always returns a new one.
So the only real strong reference should be Token -> Source


`.type_info()` is only for compatibility and keeping old signatures. `std::type_info` has many issues and there's no way to extend that class and fix them. Comparing `value_type()` with `.type_info()` will result in comparison of `std::type_info`.

`boost::typeindex::type_id()` returns some type_index, that contains all the workarounds and fixes, has additional overloads and comparison operators with traditional `std::type_info`.


This change should break workflow tests.  That's ok, but we'll need to eventually change how workflow constructs json which it submits to shell-command.


Wrong import. Make sure these are excluded in your IDE.


Oww...right... I thought there could be only 1 cancellation token for a given CancellationTokenSource.
Is there an upside to this flow? Compared to sharing a single instance of CancellationToken?


Okay... How often is this function called?! I mean, shouldn't `updateCollisionElements()` only handle changes in the collision geometry?


I think this can just be `compactionInterval`


:+1:  :))


Nit: Move bracket to the previous line ;)


bad. typically, only class names have capitalization, method names begin with a lowercaze letter and is camelcased.


quotes should go here to for consistency with screen readers


ditto


"How often to check for local shards that need compaction"


lgtm!


gg


I also removed the redundant brackets on line 12. Haskell functions are right-associative by default, and my linter was yelling at me


Nit: No need for `nil` check on token - just use `if (token.cancellationRequested)` - it's going to return `NO` if `token` is nil.


Please update this to `json_encode` the message, rather than construction JSON by hand.


@dmzaytsev Use the full name `RsWithCookie.CVALUE_PTRN`


@dmzaytsev Use the full name `RsWithCookie.CNAME_PTRN`


Why final?


It's called every time a new problem is set up from our user interface, so the overhead for us is pretty substantial.


Is this email ok @tehsis ?


Oops, sorry. What bug?


Does this really need to call other FCALL functions? I haven't looked too closely at what FCALL_CONTRACT involves but it seems safe to just call sinf/cosf directly.


See PR #1633.


Darn you, IE. Still, I could still sneak something under a .png.


Is it necessary to clear out `Fragment` here? It might surprise someone who constructs a url like this:

```
var url URL
url.Scheme = "http"
url.Host = "example.com"
url.Fragment = "page3"
httpUtil.NewResourceLocation("foo")
```


get_surface_element_at will have already iterated over the none surface elements. Same goes for below.


The VM_NEXT_OPCODE is redundant now.


Looks good as far as I can tell. I haven't followed the madVR issue so I cannot really tell.

I have https://github.com/mpc-hc/mpc-hc/compare/develop...Underground78:df872f5 (which needs some extra updating, I will probably skip the last commit for tonight).


Nit-pick: I think your indentation is off by one space in each line, or?


You're passing in a pointer, but then you do a copy. Why not pass a `url.URL` by value in the first place?


лишний файл. мог бы просто добавить обычную кнопку и проверять isset' ом в index.php


I think that a lot of the stuff that this commit does is better moved to the Boid module. See f5df4eb5ee1d68f9bc10a3ba8a4b787f9be56654 for an example of this.


Thanks walters. Yep, it worked fine in my local testing, and it's about to get a bunch of testing by the Fedora QA people -- https://bugzilla.redhat.com/show_bug.cgi?id=1211991 is a blocker bug in Fedora :)


fast-forwarding master to auto = 7dfb509b


Consider a named constant (e.g. `revokedManifestSequence`) for self-documenting code.


t1 is exactly Strength at (j, 0, i) = 1 - (Distance to center) / (Brush radius)


@thadchai done with my review. checkpoint complete.


more readable form. also merged lines 847-853 into this line


No problem, love the project! ReSharper told me all this anyway ;)
On Apr 14, 2015 9:42 PM, Derrick <notifications@github.com> wrote:Ha. No idea. Probably a remnant of some debug code that I removed.
Definitely can be removed. Thanks!

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


@thadchai done with my review. this was well done.


Create folder


gdfgdgfdgf


conversion from Y to rawHeights is:
rawHeights (0-65535) = Y (0-1024) * 64


@junenam what happened here???


I am having merge conflicts due to this file


@thadchai love the long press to share.


@assaron Are we going to do selective mount for different workspaces? I do think so if we are going to have independent fs for each command.


Can you please do this for Data.Graph.Inductive.Tree as well?

Also, please put the `deriving Generic` bit on the next line and put parens around `Generic`.


where is this located


thank you very much for the info. Actually we can wait a few more time before getting this in, because there is still one TODO to complete the correct setting.


@thadchai this view doesn't have a NavigationBar that will show this BarButton. Consider using a UIButton instead and placing it in a fixed position - upper right corner?


The denominator c = 64 here is removed in a later fix:
https://github.com/SkylinesSimsfirehouse/SkylinesTerraform/commit/a39c2cbddd69d4b9a5b9aab85544b3b424c57ca2


I used it to test the scrolling aspect of my game, but I have yet to make the player ship appear not moving, even when using the same value to move it against the scrolling view.

Based on learning from the link below, the View has a flipped coordinate system, moving in negative direction relative to the game window. Therefore the most I can do is use that value for passive player movement unless I can better solve the problem.
http://dev.my-gate.net/2012/06/using-sfview/


How would I use this? I don't really understand `set opfunc`.


We define `seq` as `sfSequence` which is a 32-bit type. Would we be better off with `std::uint32_t` here or with 64-bits on the wire format?


rewriting the method with Vector3.Dot(). It acts in the same way as the original code.


These were moreover grammatical and or speaking points I wanted to correct. I suppose for the most part these changes weren't that necessary.


Here's how i imagine it http://imgur.com/yJhBzv1


See https://github.com/jcsalomon/smarttabs for the rationale for my changes above (I didn't do much, emacs did it for me). Your code only looked good at a tab width of (I think) 4, while my whitespace changes make it look good at *any* tab width.


thanks for the catch. I've removed that comment.


whart art ok do int


IRC doesn't have user IDs, which is where we run.


It doesn't seem like all these should be required. For example, why are google picker, google cloud sql, debuglet controller, or bigquery api required?


I recently received the following support request for gridspree. Is this similar to the problems you're experiencing?

----------------------

Hello,

I’m interested in using grid spree in a side project. However when I try to add a new spreadsheet I get this error:

400. That’s an error.

Error: origin_mismatch

Application: Gridspree

You can email the developer of this application at: team@gridspree.io

Request Details
immediate=false
response_type=token
scope=https://www.googleapis.com/auth/drive.readonly
redirect_uri=postmessage
proxy=oauth2relay679258998
state=640160653|0.3665328653
origin=http://gridspree.io
include_granted_scopes=true
client_id=976732173475-gph3msg050oaba4k7u12c0ilcjvt94u1.apps.googleusercontent.com
authuser=0
That’s all we know.

Thanks!


This is a new function that converts target view (aka world) coordinates into coordinates based on window pixels. withinWindow came in very handy because I do not have to worry about the entities disappearing when gameView happens to be scrolled.


@gabebw this creates an operator pending mapping, just like `d` or `c`. So you use it a `S` followed by a text object or motion, e.g. `Si"` to replace inside the quotes (assuming you're in a quoted string), or `Si)` for inside parens. `opfunc` is the magic that lets you create operator pending maps


If any of the options is disabled, setAllOptions will be cancelled immediately. This operation will fail due to a option is disabled.


Let's merge as these changes, but consider this likely case:

```python
@node_processor('table')
        def if_table(node):
            return {}
        ...

@node_processor('ul', 'ol')
        def if_list(node):
        ...

strat = TABULAR + (if_table, if_list) # exception will be thrown by if_list
```

How would address that exception?


Why did you use x_position - 1 and x_position + 1 instead of +/- x/y_scope? Just curious


This new function is quite useful since I can compare rectangular bounds of entities against the cells.
Although I have used it against all entities, I discovered a strange behavior that the player ship seems to disappear when hit by "invisible enemies" .


null is different from empty string. "" is an empty string but not null.


I got the point. I'm always using tab width 4 and haven't paid attention to the misalignment under other tab widths.

> On Apr 15, 2015, at 10:11 AM, magnum <notifications@github.com> wrote:
>
> See https://github.com/jcsalomon/smarttabs <https://github.com/jcsalomon/smarttabs> for the rationale for my changes above (I didn't do much, emacs did it for me). Your code only looked good at a tab width of (I think) 4, while my whitespace changes make it look good at any tab width.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/magnumripper/JohnTheRipper/commit/ed9506cb8cc187f692189dcd267480a92ba96c79#commitcomment-10727469>.
>


UpdateArea area bounds only need to be <s>2</s> cells wider than the area modified.
Later I found they only need to 1 cell wider.
74c2634944ce9c49d207bfa3fb6de07775430ca7


Thanks. Could the edit page go on a separate edit_exceptions page? That could be linked to at the top of list_exceptions (similar to what's done in profile.php), as well as in the menu. That way the delete form isn't included on every page of the exceptions list.


Instead of leaving it as "" may be define a constant like NO_COMPRESSION_CODEC and use that in the code.


rax retry tests


:+1: can this be a separate PR?


This commit doesn't work, so I reverted it later at https://github.com/SkylinesSimsfirehouse/SkylinesTerraform/commit/a1360bf5eabf27b2456511bd65ebae63026dc6b5


It was going to be comparing intRects based on game window coordinates before going into collision functions. Sadly, I encountered a strange bug that caused the player to disappear from an invisible bullet or enemy, which caused me to revert it back to global coordinates.

When I tested the game, the game ran faster to closely 60FPS, but when multiple objects collided with explosion entities, the game slowed down to around 30FPS again.


@rodricios is it safe enough, or do we have to make a "compiled" version, e.g.:

```python
class Processors():
    def __init__(self):
        self.mapping = {}

    def register(self, *tags):
        def decorator(fn):
            for tag in tags:
                self.mapping[tag] = fn
            return fn
        return decorator

    def process(self, nodes):
        for item in nodes:
            if item.tag in self.mapping:
                yield self.mapping[item.tag](item)
                continue
            yield item
```


I didn't find this file on my computer. Did you include it by mistake?


Yeah!


I like this improvement.
However, the unit test generated from
```
ember g serializer application
```
is no good because it assumes I have a model called 'application'.


These new icons never get re-added to anything, do they?  Should you have a placeholder bin to place these in?


All of these were never used outside of `RippleAddress.cpp` and the `enum` could simply be moved into `RippleAddress` and out of the header files.

We now use `VER_NODE_PUBLIC` in `UniqueNodeList.cpp` and `Manifest.cpp`. Is there any way we can avoid this (though _not_ at the cost of bloating `RippleAddress` any more)?

(Sidenote: should we make this `enum` explicitly be `std::uint8_t`?)


changed default bush sizes to multiples of 8 and, in a later commit, the increments https://github.com/SkylinesSimsfirehouse/SkylinesTerraform/commit/aed59bf6838fa1bbe14e39d40b671330412db653
This is easier to work with.


Minor nit: How can I tell which validator the ephemeral key is associated with? Would it not be better to have an `ephemeral_keys` array, that contains the list of all ephemeral keys with a link back to the ephemeral key's master identity?


This looks good. How would this get added within the pipeline?


Nitpick : Is it possible to undo changes like these. The original comment format looked better.


WAS_NONCREATURE_ARTIFACT is rather niche, why not do the check in groovy?


same changes as in https://github.com/SkylinesSimsfirehouse/SkylinesTerraform/commit/4fc794b0d24ce52fdc094140df75880018909d2f


```
STRATEGY + (processor.process,)
```


merged the two methods, so only one call.


The numbers here are reversed: you're ignoring `seq` because it's not newer than `old_seq`


`ActiveRecord::Rollback` is necessary because `@order_details = order.add(product, quantity, note: note)` will change state even when a validation errors occurs.


Why not `unpackManifest (s)` and let it worry about the details?


this new tool behavior is explained with this sketch: http://i.imgur.com/OuV9gvE.png
it worked better on uneven terrain.


these codes are not changed as it might seem.


This is a better implementation in my opinion. I was exploring a similar approach with "pruners".

As an aside, have you looked at this book? [A.I. - A Modern Approach by Peter Norvig](http://51lica.com/wp-content/uploads/2012/05/Artificial-Intelligence-A-Modern-Approach-3rd-Edition.pdf)? ctrl-f "pruning"


This didn't show up when I searched for mmrCache in rooms.js while doing https://github.com/Zarel/Pokemon-Showdown/commit/59597bc7106fe12c80e8b45bb382d58f44b69a43 :|


@Ricardo1991 I like what you did. Can you submit a pull request in SVG format?


@PalladiaMors The best way is to submit your change as a pull request. See https://help.github.com/articles/editing-files-in-another-user-s-repository/ especially steps 6 to 8. That way we get a notification with all the necessary information from github and we can act on it easily. With comments, there is quite a bit of manual work to do to get the fix in.


Not really. tsd_exit_key will call dtor which will free poolname as I mentioned in the comment.
But if we are going to use tsd_set(key, NULL) as you mentioned. We probably want to keep it explicit.


L7-8は`@yield('body')`と書くことができます！
`@section ~ @show`は他のviewで継承する要素を入れておくための文法です。
http://forumsarchive.laravel.io/viewtopic.php?id=13147


* Fixed issue #270: Clone fails: bash: X.X.X.X: command not found
 * Fixed issue #568: push using ssh private key with a password fails the first time
 * Fixed issue #577: Commit warning "svn:externals"
 * Fixed issue #586: Ampersand in log description underlines next character
 * Fixed issue #323: GIT SVN dcommit --rmdir
 * Fix stash list can't show stash at reflog default
 * Fixed issue #603: Columns not sorted in Compare revisions dialog box
 * Fixed issue #598: Background color of log messages changed when mouse is over tag or branch indicator
 * Fixed issue #616: Problem encoding Cyrillic characters in Author name
 * Fixed issue #612: "Copy all information to clipboard" copies incorrect header
 * Fixed issue #602: Rebase shows overlapping branches
 * Fixed issue #542: Difference between line endings in compared files (external Diff Viewer)
 * Fixed issue #468: Non-ASCII characters in user info aren't stored correctly
 * Fixed issue #593: Fetch and push fails with TortoiseGit but succeeds with git bash


Fix pushed! Changed n to m :)


why do we need node_id in the request ?


This should probably have been tagged as "skip ci", whoops. Need to remember that. Sorry for wasting Travis' time and electricity :/


Moar hardcode!


Yours asserts there is an error. In 5.0 there should be. In 4.2, there is currently, but there shouldn't be.

In the master branch there is an error, and there should be. However, when I originally did this against the 4.2 stable branch, I was asked to put it on master instead, and let them backport it.

Your test should stay in the master branch, as it is good to have. But it should not be in this pull request, because they are conflicting goals: This PR deprecates the behavior rather than removes it.


I think it should happen the other way around. I would imagine that DIC would be kept inside of the ElggApplication that would we the most general, non-singleton thing to have


I'm not aware of what I've done replacing this try-finally block with the single line from the finally block.
Please tell me what I'm doing wrong here.


Sounds good to me.


Have you test the performance of HoughLinesP (openCL version)? I find it only 2 times faster than CPU version...


nb


this.m_mouseRayValid = (!this.m_toolController.IsInsideUI && Cursor.visible) as defined in OnToolLateUpdate()


i would also checkarg that partitionBy contains all of prePartitionedInputs


Fixed: 7e090dc4809bab77fceb20cfabab2c430592f1f3


this probably needs to be canonicalized too


Why not compress this:

    // Only insert key if it hasn't been revoked
    if (seq != std::size_t (-1))
        unl.insertEphemeralKey (m->signingKey, iter->second.comment);


Using the same ceil() and floor() method to get brush bounds as in ApplyBrush()


Creating a new overlay call (not sure how it works)
OverlayEffect is going to be called multiple times, so I save it here


Nope, it's needed.  :(  It shouldn't be though, I'll have to fix that...


1 cell wider than brush bounds because I'm gonna also highlight the points just outside the brush bounds


在阅读帖子的情况下，这个logo很碍眼：
![2015-04-15 10 45 44](https://cloud.githubusercontent.com/assets/72378/7151447/9f09df42-e35c-11e4-9511-a25e2a2e1b32.png)


I did not -- neat.  But in this case the issue is that we get `[String: AnyObject]`, which is not `Equatable`.  However, `NDictionary` does do the right thing.  Weird, eh?


Do we need to send node_id as part of the request ? The request is already being sent to one specific node via this line of code : { rpcOps.sendAndReceive(nodeId,...}  right ?


My RCP Visit


if just outside the brush bounds, paint color3 (gray).
If inside the brush bounds, paint color1 (color dependent on brush strength: 0.1, 0.2 is red, 0.3 orange, 0.4, 0.5 yellow)


yeah that's not right, I reverted this and just changed the forcesync that was already there.


What about zfs_fsyncer_key?
There's this ```tsd_set(zfs_fsyncer_key, (void *)(fsync_cnt - 1));``` in ```zfs_log_write```,
Does it will always reduce to 0?


First block for square brush, second block for circular brush

if just outside the brush bounds, paint color3 (gray).
If inside the brush bounds, paint color1 (color dependent on brush strength: 0.1, 0.2 is red, 0.3 orange, 0.4, 0.5 yellow)


Пожалуйста, называй переменные более понятными именами. Пусть они будут даже длинными.
`i`, `t` - названия не оч :-1:
`myAwesomeCounterForPlayingSounds` - более менее :+1:

На досуге почитай коды больших пректов, написанные профессиональными программистами. Там в принципе по названиям переменных можно определить, что делает код.


some modified


Replacing the quad overlay with the square bitmap brush later in https://github.com/SkylinesSimsfirehouse/SkylinesTerraform/commit/9e8316a8c1f3d26ab78787827a0594f66034252e


Не могу понять, как это работает :sob:


I am sorry for the waste, I deleted the test. For some unclear reasons I
did not get any email for some hours, so I did not realize that I am
wasting resources.

I hope it is fine to you if I resubmit the test, this time with the correct
setting.

2015-04-15 21:47 GMT+02:00 Stefano Cardanobile <
stefano.cardanobile@gmail.com>:

> Hi,
>
> sorry it was obviously not intentional, I stopped it.
>
>
>
> 2015-04-15 21:32 GMT+02:00 Marco Costalba <notifications@github.com>:
>
>> nodestime=600 must be set on both opponents: you are wasting results,
>> even after you have been warned test is bogus
>>
>> --
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Stefano80/Stockfish/commit/c24f70c3e3b62f403c31ca60a50b6ecc935b03d6#commitcomment-10743535>
>> .
>>
>
>


It mankes me wonedr hwo I manaced to write tho documhentatino withaen tpyos.


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


export let


Thanks - I really appreciate that, once we have a bunch of contributors that care we'have mostly fixed our problem :)


Make this a member of the handler with a default


Hi Alec


I wonder how useful this function is if you already have a `url.URL`; it seems like I could get most of the desired functionality by just doing: `url.Path = url.Path.Join(url.path, id)`


Use a wait group instead


`widgets` should be named `pairs` since it is a list of pairs of (key, widget).


LGTM


This should be `length pairs`.


This spurred me to make bigger changes than anticipated. I pushed a new commit; please take a look.


'products_attributes_filename' can not come from the query in line 773 as it is being pulled from TABLE_PRODUCTS_ATTRIBUTES and that is done with  $stock_query_raw at line 757
product_is_always_free_shipping can come form both queries ($stock_query_raw line 757 and  $stock_values line 773), so yes this one should be added.

I'll make a new pull request.


How is this different then using New File>Cocoa Class

Then picking Subclass of XCTestCast with Lang of Swift?


Where is your variable declaration for randomNum1 and randomNum2? What I am seeing, unless I'm missing a section of your code, is that these are global declarations.


Not fond of the name here, since this function does not make a manifest as defined by `struct Manifest {...}`.  What are you making and, perhaps more importantly, why?  Either a better name of a comment (or both) is in order here I think.  Yeah, it's just a test, but someone may have to maintain it.


Por que dessa linha?


Doesn't look like its returning nil


BAAF


It should be completely safe to call `sin` and `cos` directly. However, if any changes were made in the future to `sin` or `cos` (for whatever reason), the same changes would also need to happen to `SinCos`. Having it call the FCALLS instead should eliminate the need for that.

Also, on platforms that support it `SinCos` should end up as an intrinsic call instead of an FCALL anyways (intrinsincs are not part of the PR yet).


Doesn't


[I think it's Haskell convention to use lists for this type of thing.](http://stackoverflow.com/questions/7556573/why-is-there-no-built-in-set-data-type-in-haskell)
There is also [Data.Set](https://downloads.haskell.org/~ghc/7.8.2/docs/html/libraries/containers-0.5.5.1/Data-Set.html), but that seems like overkill for this.


Your code does exactly the same as mine.


Put these near the constructor for each object


I would really prefer for git commits to include *why* the change was made, not just "what".  It allows someone later to understand the rationale in case something went wrong.  It helps communicate the change in release notes.

In this case you could link to https://git.fedorahosted.org/cgit/fedora-atomic.git/commit/?id=77aa1764323789e73c1441f4f0a3a2003534540e
which in turn links to more information.


No, I mean that you replace "YourAmazonStore" with the name of your store. You should end up with something like this:
```php
$store['Faith-Hope-Love']['merchantId'] = '____________';
$store['Faith-Hope-Love']['marketplaceId'] = '____________';
$store['Faith-Hope-Love']['keyId'] = '____________';
$store['Faith-Hope-Love']['secretKey'] = '____________';
```

Also, I forgot to mention that, until the next version, you must give the store name to each Amazon object if you are using the code in the `stable` branch. You can omit the store name when initializing an object if you are using the `master` branch.


"a the"


Probably a bad request


Добрый. Татьяна проверила ваши исправления и всё же решила оценку за эту лабу не менять :(


This will crash if `widgets == []`. There might be other spots in this new implementation that are not robust to that condition, too.  My preference would be to avoid this by pattern-matching on the case where it's empty so that we have
```
insertMultipleIntoList _ [] _ = return ()
```


nothing in this file??


Thanks for committing.


@oeddyo fast work!

:)


Bad request


You, Vinnie, and Nik clearly get what a `Manifest` is.  I haven't figured it out yet.  It might be nice to have a paragraph comment describing what a `Manifest` represents and what purpose it serves in the system.


Aside from my comments above, this looks okay, although this code was tricky enough before the change so I wouldn't be shocked if it's hard to get right here, too. If your testing has convinced you that there aren't any unhandled edge cases (aside from the one I mentioned), then I'd be willing to merge it after you address the comments.

Thank you!


Depends on whether we need to use "if" block twice or repeat four statements of code in the two branches of "if".  Since there is not much saving I am going with later approach.


Please make sure no conflict before submit...


Should we pass `connection=None` to the bucket, to ensure that the override is being used?


Ф-ции не имеют никакого отношения к файлу db.php. В файле db.php должно быть только соединение, как и раньше. а ф-ции должны быть в отдельном файле или файлах, и эти файлы должны быть еще в отдельной папке (т.к это отдельный слой приложения)


This commit seems to introduce a dependency to a file called ReturnVisitor.py, which is not part of the commit?


No semi-colon needed when closing an if or any loop statement.


It avoids a reference cycle :)
TBH I just implemented it this way since that's how the C# version is implemented.


Good catch! Just updated...


Hey @adityasharmacs, I've somewhat abandoned the Code for San Jose repo and am working from https://github.com/howdoicomputer/calidrought .

I also don't think I'll be attending the Code for San Jose meetings anymore. I will be in the Slack channel all day every day until I... don't want to be in there anymore. I will be attending the Code for San Francisco brigade meetings from now on.

But, more pertinent to this thread, the repo I'm working out of will have a wiki page by EOD on architecture since it's somewhat solidified now.


Ditto for these tests.


What is the purpose of this?


just do it


Оценка за 8-9 задания: "отлично-"
От меня просьба: пожалуйста сделайте ещё 1 шаг в сторону улучшения качества commit message' ей. Можете попробовать добавлять какие-нибудь метки типа "[Lab7] blah-blah-blah". Это не критично, но здорово упростит понимание истории коммитов ;)


There is Q_OS_IOS already and u->uc_mcontext->__ss.__pc may work in addr_cache_exception


:grin:

I would vote for using a single instance of `CancellationToken`, since there doesn't seem to be an advantage over it.
After you get a token - it is automatically retained in the caller stack, so it won't go away even if `CancellationTokenSource` is gone. So I think using a `weak` reference here from token to source + single instance of `token` that is strongly owned by the source is probably the way to go, otherwise it is slightly inconsistent with TaskSource and Task.

cc @grantland - thoughts?


see notes above about passing in a pointer, then copying it.


Esperando resposta!


Oh right, thank you, I'll add it to my check workflow.


Hmm... Is there a rationale to use `-rm ...` instead of `rm -f ...`?


Main HTML structured file


The function should be documented (its purpose).


feel free to make pull-requests


Shouldn't the relationship be the other way around? Since the CancellationTokenSource really contains all the logic it should be retained as long the CancellationToken is in use.


I have a story about this change.

I just updated aiohttp to the latest master and all aiocouchdb tests fall with all the same reason:

```
  File "/home/kxepal/projects/aiocouchdb/aiocouchdb/tests/test_client.py", line 162, in test_decode_json_body
    result = yield from resp.json()
  File "/home/kxepal/projects/aiohttp/aiohttp/client.py", line 947, in json
    return loads(self._content.decode(self._get_encoding(encoding)))
  File "/home/kxepal/projects/aiohttp/aiohttp/client.py", line 922, in _get_encoding
    encoding = chardet.detect(self._content)['encoding']
  File "/usr/local/opt/virtualenv/aiocouchdb/lib/python3.3/site-packages/chardet-2.3.0-py3.3.egg/chardet/__init__.py", line 25, in detect
    raise ValueError('Expected a bytes object, not a unicode object')
ValueError: Expected a bytes object, not a unicode object
```

WTF I said and looked in chardet.detect method:

```
def detect(aBuf):
    if ((version_info < (3, 0) and isinstance(aBuf, unicode)) or
            (version_info >= (3, 0) and not isinstance(aBuf, bytes))):
        raise ValueError('Expected a bytes object, not a unicode object')
```

Oh, really - everything not a bytes is unicode. Fine. The problem is that actually I use [bytearray](https://github.com/aio-libs/aiocouchdb/blob/master/aiocouchdb/client.py#L140-L148) with aiohttp.FlowControlChunksQueue since single `_content.read()` call as what ClientResponse does will return me only a single chunk.

May be I was not right to use FlowControlChunksQueue way by default for all resources, even for resources which doesn't respond with chunked transfer. But that's another question on which I should think about. Previously, I didn't never use chardet with aiohttp since all my data goes with utf-8 all the time.

The point of this story is that chardet may surprise you with the exception. I'll send them a fix shortly.


In relation to this code change, can you please remove login details from Enigma Plugin?
If you select the \<ENIGMA2_LIST\> the key="DEF" will contain the passwort in plain readabel text:

Structure looks like:
\<ENIGMA2_LIST\>
		\<ENIGMA2 name="VU_Duo2" state="on" sets=........"\>
			\<INT key="CHANGED" value=""/\>
			\<INT key="DEF" value="\<IP address of receiver\> \<port\> \<polling intervalvalue\> \<username\> \<passwort in plain text"/\>
			\<INT key="INTERVAL" value="100"/\>
.......
		\</ENIGMA2\>
\</ENIGMA2_LIST\>


Will display: "Array cannot be converted to a callable." Unless it is an array object implementing an `__toString()`.
Is this intended?


Sorry, debugging remnant, I have it cleaned up in the version that I forgot to push. Pushing now.


Will trigger a notice if the callback is not an array as you removed the `is_array()` check above.


Not really, first time ever I noticed that 'force' ignores nonexisting files


Upstream issue: https://github.com/zurb/foundation-apps/issues/575


You presumably copy the headers out of a concern for someone mutating the original ones. But are you worried about mutation of the other reference type members? Eg, `Cookies`.


Not sure if we should really overload JACK_PROMISCUOUS_SERVER for this. Can't we have a separate variable?


@steinachim - Sorry if the name is misleading, `updateCollisionElements()` updates the element-to-world transform of the collision elements. This is inner-loop code that runs every time `doKinematics` or `manipulatorDynamics` is called. Could we move the call to `getTerrainPoints` into `addCollisionElement`?


@TD-4242 Since you set EnableRemotecommands to 1 (which is not default) I think LogRemoteCommands should also be 1. These were the settings in the original template in zabbix cookbook: https://github.com/laradji/zabbix/blob/master/templates/default/zabbix_agentd.conf.erb


Already at top on [updated library](https://github.com/logical-and/php-oauth)


I'm getting this on fresh Mamgi install with 1.9.0.1

![screen shot 2015-04-15 at 3 12 32 pm](https://cloud.githubusercontent.com/assets/2286106/7168364/76860dae-e382-11e4-9867-2d89ac5f8a23.png)


@tawdle You were correct that it was not this commit specifically but it is still very much related.
The commit where it starts happening is https://github.com/mochajs/mocha/commit/1430c2b5102e9d355119a643e9b2be2e76e91d1f

If the following is replaced in your:
```var proc = spawn(args.execPath, args, { stdio: 'inherit' });```
with:
```var proc = spawn(process.argv[0], args, { stdio: 'inherit' });```
everything seems to work as expected.

Where args[0] is the absolute path to **node** and process.argv[0] is just `node`.

The issue seems to be with using the absolute path for **node**.

The result in the first case is a <body> with no results:
```html
<body><div id="coverage"><h1 id="overview">Coverage</h1><div id="menu"><li><a href="#overview">overview</a></li><a id="logo" href="http://mochajs.org/">m</a></div><div id="stats" class="terrible"><div class="percentage">0%</div><div class="sloc">0</div><div class="hits">0</div><div class="misses">0</div></div><div id="files"></div></div></body>
```

Where in the second case I get a full <body> as expected:
```html
<body><div id="coverage"><h1 id="overview">Coverage</h1><div id="menu"><li><a href="#overview">overview</a></li><li><span class="cov high">100</span><a href="OMITED"><span class="dirname">OMITED</span><span class="basename">index.js</span></a></li><a id="logo" href="http://mochajs.org/">m</a></div><div id="stats" class="high"><div class="percentage">100%</div><div class="sloc">6</div><div class="hits">6</div><div class="misses">0</div></div><div id="files"><div class="file"><h2 id="OMITED">OMITED</h2><div id="stats" class="high"><div class="percentage">100%</div><div class="sloc">6</div><div class="hits">6</div><div class="misses">0</div></div><table id="source"><thead><tr><th>Line</th><th>Hits</th><th>Source</th></tr></thead><tbody><tr class="hit"><td class="line">1</td><td class="hits">1</td><td class="source">var Cube = module.exports = function(width) {</td></tr><tr class="hit"><td class="line">2</td><td class="hits">1</td><td class="source">  this.width = width;</td></tr><tr><td class="line">3</td><td class="hits"></td><td class="source">}</td></tr><tr><td class="line">4</td><td class="hits"></td><td class="source"> </td></tr><tr class="hit"><td class="line">5</td><td class="hits">1</td><td class="source">Cube.prototype.getVolume = function() {</td></tr><tr class="hit"><td class="line">6</td><td class="hits">1</td><td class="source">  return this.width*this.width*this.width;</td></tr><tr><td class="line">7</td><td class="hits"></td><td class="source">}</td></tr><tr><td class="line">8</td><td class="hits"></td><td class="source"> </td></tr><tr class="hit"><td class="line">9</td><td class="hits">1</td><td class="source">Cube.prototype.getSurfaceArea = function() {</td></tr><tr class="hit"><td class="line">10</td><td class="hits">1</td><td class="source">  return this.width*this.width*6;</td></tr><tr><td class="line">11</td><td class="hits"></td><td class="source">}</td></tr><tr><td class="line">12</td><td class="hits"></td><td class="source"> </td></tr></tbody></table></div></div></div></body>
```

It should maybe be noted that I am running mocha as an npm script with `npm run cover`


Is this intentional? There's already such definition in https://github.com/embox/embox/blob/master/src/include/util/array.h#L147-L148
Extbld limitation m.b.?


This seems good to me, but someone else who is more familiar with the spec should review as well.


Porque em y a velocidade é zero.


If that's the case - then you can run into the case where `CancellationToken` is gone, but the `Source` is still there and is referenced - which would be quite crazy.

Though - right... you won't be able to register an observer - since the logic and ownership is contained inside the Source.
On the other hand - if we would move the control logic inside `CancellationToken` an make it private (the same as with Task) - it would eliminate this possibility.

Say a `CancellationTokenSource` is a public interface for interacting with `CancellationToken` (same as TaskCompletionSource is for Task).


I like this idea,
For example, if Black King g8, and pawns f7 g7 f6 g6,
at first sight, Black is "safe" ! (and I see this once in a while)
But Black can be in deep trouble if White has doubled rook on h file.
In those cases, the double pawn acts like a coffin, it takes time to move them away,
and Black King may suffocate, with no quick escape.

So this bonus can help in some precise cases.
I never studied those shelter calculations, and neither your patch...so I dare to ask:

Double pawn can help, but they must come from somewhere,
is there are some penalty in those shelter calculations if a pawn is missing on some files ?

Or simply, if the double is isolated...not much added safety value here.


Any purpose for 7.0? Does build finishes with success on 7.0?


Is this logging still necessary?


Addresses #237.


Yes, I think you're right - sorry for being unclear, I didn't mean that the result_of call itself should be changed. I only noticed that boost::result_of appeared multiple times in MSVC's extremely long template instatiation messages and tried to find out which one gets the wrong result with the combination of BOOST_RESULT_OF_USE_DECLTYPE and BOOST_NO_SFINAE_EXPR.

I just noticed that the tests are still failing - I think this is because the BOOST_MSVC macro is checked in the tests before config.hpp is included. MSVC's own macro would be _MSC_VER.

Marcel


I'm fine with that. would you like me to update the commit message on this one?


It can't, the super method doesn't allow it.


Russ and I just noticed exactly that. The fact that it's called inside doKinematics is almost certainly the problem.


Eh, I think this was supposed to be `points = points - (server["clients"] - 16)`.


Just wanted to note that comparing something with `true` or `false` is slower then just `if (condition) then`

This is because now it has to check things twice if I remember correctly (@madmaxoft explained it to me when I just started writing Lua)

Instead you can use `if (PlayerFrozen[PlayerName]) then` or `if (not PlayerFrozen[PlayerName]) then`


@rdeits and i are resolving this locally.

On Apr 15, 2015, at 4:16 PM, Andres Valenzuela <notifications@github.com> wrote:

> @steinachim - Sorry if the name is misleading, updateCollisionElements() updates the element-to-world transform of the collision elements. This is inner-loop code that runs every time doKinematics or manipulatorDynamics is called. Could we move the call to getTerrainPoints into addCollisionElement?
>
> —
> Reply to this email directly or view it on GitHub.
>


see above


I kind of noticed that. Adding some now.


did you mean && ?


yes, httpd meant to build on host systems too, and ARRAY_SIZE isn't in any standard header =(


```
C:\work\test>clever init my-test.prj backend -S -f --verbose
Preparing for installation...
-----------------------------
args             = ["backend"]
--force          = true
project          = "my-test.prj"
projectFolder    = "C:\work\test\my-test.prj"
singleSeed       = true
installFrontend  = false
installBackend   = true
-----------------------------
  ├── Creating project installation path...
  ├── Creating installation path C:\work\test\my-test.prj...
  └── Preparing seed installation list...
Installing Backend...
Downloading and extracting node-seed...
  ├── Creating local configuration file config/local.json...
  ├── Installing NPM modules...
Error: spawn npm ENOENT
Status @ 5s |   Installation Progress: [▒▒▒▒▒-----]  50 %   ?   Installing: Back
end   ?   Step: Installing NPM modules...   ?   Step Time: 2s   ?
C:\work\test>
```


Why do everyone ignore C99 `for` loop variable declaration? :sob:


Ah, I see...

On Wed, Apr 15, 2015 at 11:20 PM, Anton Kozlov <notifications@github.com>
wrote:

> yes, httpd meant to build on host systems too, and ARRAY_SIZE isn't in any
> standard header =(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/embox/embox/commit/17a0dcf6c9337fedda5b086e1ce06d6daa11636e#commitcomment-10744451>
> .
>

--
Best regards,
Eldar Sh. Abusalimov


Misspelled 'canola'


Maybe a better name is "AddQueryValues" - there's no real merging in the sense that no reconciliation happens when there are duplicate query values.


I don't think it makes sense to be able to register a block without a source. For instance

    BFCancellationTokenSource *cts = [BFCancellationTokenSource cancellationTokenSource];
    BFTask *task = [self myAsyncMethod:cts.token];
    // ...
    BFTask *task2 = [self myAsyncMethod2:cts.token];
    // ...
    [cts cancel];

It seems to me the source *should* be retained throughout the async operation since it is the controller, whereas the token is the observer.


OK, it's bad that `ScheduledReporter` overrides `close` and suppress the `IOException`.


And the token only needs to exist for as long as the asyncMethod is running


If a pawn is missing, this will have an effect on the stormfactor in the other direction - and of course if  rooks are doubled on the h-file then safe checks override everything.

Your point about the g7/g6 axis is why I was sceptical about the SPSA values, which seem to value the doubled g-pawn more highly. But one has to ask.. if it is a bad idea, why does SPSA move the value up?

The original thought which inspired the patch was the not uncommon formation of f7 g7 h7 + f6 after an exchange on f6. The extra f6 pawn blocks the long diagonal very securely.

It will be rather disquieting if the SPSA values fail, when the original guessed values passed...


http://stackoverflow.com/questions/24174500/is-cancellationtokensource-mandatory-when-cancellationtoken-is-requested

> Cancellation works where one side has the means to notify cancellation (CancellationTokenSource) and the other side needs to observe that and get cancelled (with a CancellationToken)


One thing to consider is that if we have an id tied to a project and not a slug, we can associate multiple slugs to a single project. This can be useful if I'm working on pgd/protein_geometry_database/pgd_core/etc a fuzzy search would be better, but it at least gives us options going forward.


Right. But it will be - since you have it on your local stack.
The thing here is that you can get a source publicly from a Token - so it is required to pass it around anyway.
Since default reference is strong one - it won't be gone as long as someone has a reference to it.


Or C, my native language :-)


i hit this problem too


fast-forwarding master to borshop-integration-215-bk-j308-typed-bucket-get-loses-it = a9a9ef3a


Maybe make this a generic object. Or think of a way to be able to apply this to anything in codalab. A user, Competition, worksheet forum.


Whoops


also chardet may return None


@TD-4242 Seems this now requires me to set server and active_servers information in two places.

Thinking attributes should be...
``
default['zabbix']['agent']['conf']['Server']                = node['zabbix']['agent']['servers'].join(',')
default['zabbix']['agent']['conf']['ServerActive']      = node['zabbix']['agent']['servers_active'].join(',')
```
and that both these attributes go back to empty lists.
```
default['zabbix']['agent']['servers']               = []
default['zabbix']['agent']['servers_active']    = []
```

This forces me to have to set only two attributes instead of 4.


(Mistyped: you can not get a source publicly from a token)


Okz..So can I still contribute to the project or it can;t be done...Just wanted to know from you.I found out the project is good.So that's why got interested in doing somethings and starting off with it.


Actually, the correction should be AND rather than OR. If in update mode then the kernels that can be run must be missing from the config. The regex further restricts that set.


@sschuberth my indentation matches the rest of the file (tab + 2 spaces, vs. 10 spaces in @dscho 's version). Looks like github displays tab as 7 spaces in diffs (i.e. if preceded by '+').


Whoops, okay, I really thought that was inferrable. This is just me really not quite getting Haskell syntax.


yep, they were doing that at previous sprint review :smile:


Fair enough.


The Debian people don't like hardcoded /dev/shm/, because this complicates moving shm somewhere else. And indeed it's under /run/shm these days.

The usual approach is to rely on shm_foo() or shmctl().

But wouldn't it be easier to use the correct values in the sem_open() calls?


Thanks for the clarification!


This seems more like a `Strategy` or something but it is probably not necessary for me to continue quibbling over names in this manner.


It is shortly described in the following common/gid.h


add a last_post date for easy sorting. That is updated on Post submission.


Do the commented out lines have value?  I'd be inclined to remove them.


I agree -- I was going off of the whole `Agent :: Perception -> Action` thing we were talking about (but with `Update` instead of `Action`).
But yeah, `Strategy` makes more sense for this.


Check if all the tests exists for the suiteResult


This has nothing to do with #161


true if all the tests were found, false otherwise


You need a reference to a source (and its intended to be a strong one) to be able to cancel - but you don't need to keep track of token.

Say with Tasks - if you loose a reference to a TaskCompletionSource - you won't be able to finish/cancel that task since there is no way to publicly access a source from a Task.


That was a mistake ;)


Changed in 79a2c67d80eda6d2075000904cfa33abeda41dfc.


What is the purpose of this doing this instead of doing what you did on line 394?


What version of Django are you using?


This page is really only used when you don't have JS, so it doesn't actually matter that much.


`DWORD_MAX` would look better probably.


oh, None and everything-is-a-unicode conspiracy. love this library (:


U forgot to delete the branch for this


Right, good call. Silly mistake


Wydaje mi się, że raczej dużymi literami.

```THUMB_WIDTH = 100```


Nate, that line is for unit test. It is designed to trigger our testing suite and is not actually yielded in the results. Those tags will be orphan tags when the parser hits them, or at least they should. I am still parsing down 1995 with the new changes it should be done with in the next hour or so.


I agree. I just did a search and found out that this test is added by this ticket https://issues.liferay.com/browse/LPS-53601. I think the test is needed for the Registry API. I can talk to Peter Fellwock to see if it's really necessary to have a dummy validator class for the test.


The extension of this file should be `.yml` rather than `.rb`


Yes, that's normal, the variable is set by the maven-release-plugin when running `mvn release:perform`.


Looks good. About P016 commit t is up to you, I didn't follow this ticket closely.


an other -> another


I was just opening that ticket to make sure the images and information was saved somewhere reliable.


is it rand() is the same as rnd() when it comes to randomness? i saw @cydh change this to rnd()...


OK, that looks good.  I tested it and it works too.  Actually, I tested virtual and the hook version.  I also tested the failure case of the default, but I can't actually test the success version because I don't have a machine that talks regular Marlin.  Kinda why I went the other way since it was easier to see how much I'd touched the main line.

In my proposal, I made _detectPort into detectPort to indicate ok to call by the hook?  I'm just learning python but that seems to be the convention.  Would you rather plugins not call back into that one and avoid the contract?  It turns out it doesn't work for me anyway, so I'm ok either way.

I think the #TODO is unnecessary since the default is always last and it can't actually get out of that loop with a None serial_obj without throwing anyway.

This version only has two possible values for the error that shows in the UI: Failed to autodetect serial port. and Failed to open serial port, permissions correct? and the user has to dive into the logs to figure out anything else.  My current plugin doesn't support auto detect of baudrate, it'd be nice to let it shine the message through into that same spot.  I could sink the Error event and handle some UI myself, but perhaps overkill?

I've updated the branch with your code.


Filling the field with a blank string is the same as not filling it in at all, so you could leave this line out.


This revision is clean.


Game_Map::WrapX/Y and Game_Map::RoundX/Y do the same.