Should also pass the `info_request` as a parameter, rather than relying on instance vars set elsewhere. In the case of an incoming message, I guess we can grab it from the relations?


@miigotu is this ok ?


That's fine with me. I'm just direct installing dokku nowadays anyway. :-)


Okay, then sorry, I'm not going to cherry pick this into 1.8. There's absolutely no reason for this to be done, especially since it's serversided only on not shared between client/server (it appears to run on both sides right now, but I'll serverside it)
If the threads of MC ever mess into each other then something is terribly screwed up, much worse than the terra axe not working


nit: Sa3D -> SA3D


Why would you need that?


Revert to 18.0.0 because newer version are incompatible with VolumeFraction at the moment


Nice re-commenting.


Needed, because these are not under scijava in older fiji versions


Whoops, I must have had `edit` on the brain. I think I'm supposed to be calling `update` there instead?


Channel -> Channels


check this commit.
https://github.com/dorimanx/DORIMANX_LG_STOCK_LP_KERNEL/commit/fe5549a05add3da8fc14986dc4c6421284d518f6
merged from main line.


Thanks for noting. I've fixed this in a833934.


Please, use format args like this:
return fmt.Errorf("Can not read ["%s"] api_authtoken param from config", apiType)


I think you never really save into nsuserdefaults.
1. This folder will result in blahblah/Caches/home ... which means that your userdata will be in blahblah/Caches/home/userdata

2. Here you filter for allowed pathes that you allow for nsuserdefaults:
https://github.com/MrMC/mrmc/blob/master/xbmc/platform/darwin/DarwinNSUserDefaults.mm#L33

Caches/userdata is allowed - but you will never be called with pathes which contain Caches/userdata because your home dir is Caches/home/userdata.

fyi i will implement a proper solution using the CFile loader to encapsulate nsuserdefaults. So we don't need to alter the xbmctinyxml implementation...


Какой-то скудный вовсе получился default.cfg, может его разбавить комментариями различными и примерами?? Да и обновить не плохо было бы с последними функциями.


Hi @sugananth2003 !


Which update are you referring to? I think for those still hanging on to NZBMega, this is a decent solution but I've begun my migration to NZBHydra. Is that where you're going with your "alternative/fresh start"?


nit / style suggestion: I think you can actually this as just: str(self.channel_map)


Ditto on str(self.channel_map)


it's not my comment, it's the comment on the fork you're recommending.


Makes sense, since we are actually updating a resource :+1:


Remove: and 'horizontal'


Sounds great, :+1:


It's beautiful isn't it? Done, ship it.


Remove 'horizontal'


Remove


I thought it would be more clear, but perhaps it is both less clear and less correct.  I withdraw it =)


Subjective, but I find with this many filters its easier to understand what's happening when they're listed one per line


> but that would mean to really talk before...

… you commit stuff like `.bp_note` with a `background-color` that is completely different from all other `input` elements for no apparent reason (to me, at least)? Or `placeholder="seconds"`? (No, that's **not** what `placeholder` is for.) Or ` mediaproperties metadata `? (What is that supposed to mean/communcate?)

Sorry, just noticed those because I checked what you two were even fighting about.


Okay, let me rephrase this, in previous versions, if node was disconnected, we were sitting default display, we accomplish this behaviour by use of `contains()`, now we call that method only to fix one edge bug in FF... to do the opposite.

Now though, it hits `contains()` when element is hidden, whereas before, it would reach this point when element is shown.

Which means, we improved the perf for the case when element supposed to be shown, by worsen the case when element is already shown i.e. incorrect use-case.

So why do we want to change the logic on which never receive any complaints about it in three years by worsen the case for which the `show` method was written?


+1, Already fixed that on mine.


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


At this time it would. I can refactor request-handler.js next to use the new database created.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11681](https://teamcity.mesosphere.io/viewLog.html?buildId=105909&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:04:26


FWIW, also a bit confusing that `check_` has been used for filter actions that block the request,  but this method doesn't do that. This would be better named as `set_internal_review_ivars`.

It looks like `@internal_review_pass_on` is _always_ `==` to `@internal_review`, so I don't see the point of setting it.

```
$ grep -r internal_review_pass_on app/
app/controllers/response_controller.rb:    @internal_review_pass_on = false
app/controllers/response_controller.rb:      @internal_review_pass_on = true
app/views/response/_followup.html.erb:        <% if @internal_review_pass_on %>
app/views/response/followup_preview.html.erb:      <% if @internal_review_pass_on %>
```

We'd then have a nicer filter name (`set_internal_review`).


Forgot to add it, this should fix #41 and #43 btw.


fix #203


I have some considerations on user avatar, especially concerning pubsub (which is not enabled in the server yet by the way). I will write to you soon.


This isn't a bower dependency so not sure I understand


@yellowled Ah, I postponed going though the backend for stuff like that…


@swarzesherz crea un issue por favor, respetando el formato si es posible, declarando:Problema, y Objetivo de la tarea de forma breve. Luego nos puedes mandar un Pull Request. Gracias!


Ohhh regexes I get it. Done in de19e0a


Ok great


Turns out both the local src directory and the teensyduino libraries were already in the system path because of the makefile. I just removed src


I've moved this function out from the `render` function. If this is defined within `render`, JavaScript has to recreate the function in memory every time `render` runs (and `render` runs a lot!). Out here, it's defined once (when the file is loaded up), and then it just runs the function as needed.


I don't really know whether we have any established conventions elsewhere in the app, but since this seems to be a request-halting filter, maybe we should stick with the `check_` convention of this file and call this `check_logged_in`


TeamCity pull requests :: netty [Build 1039](https://garage.netty.io/teamcity/viewLog.html?buildId=4791&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6371, ignored: 24 Build time: 00:42:51


Sure, we could do that (only `CFG_TA_FLOAT_SUPPORT` is needed). But then we'll need to do that or the test will just silently skip floating point tests.


The rest of these changes are about indenting.


ใช่ป่าว


Ugh escape fail in commit message


Ah, I see what you mean now. That's a fair concern but also nothing I haven't started to see with the original. Would it be worth adding an option to the configuration? Default to pillone and anyone else can adjust to Mirabis if they'd like to use that fork?


I've rearranged the way the legend takes its properties, so there's a little less coordinating. More on this in the legend.js file.


These only get pulled in when you're developing the addon locally.  Pulling in this addon to you app does not bring these deps in.


Oops, I should have changed the defaultProps too. That's in the next commit.


It's not really clear to me why are you using the first vertex only.


จุติ สเตตัส 1200 อัพเกิน 400 ไม่ติดสถานะ


To clarify: I'm refering to phpseclib 2.0. The variable name was changed in that version.


This is not being used?


อัพให้จะดีใจมาก


Does it actually successfully inject the code?


I think this will be a separate  PR.  I am going to change nodesPrivacy.js to when I rename nodesPrivacySettingsTreebeard.js and everything in it to something more generic, as it is very usable across applications (it builds node trees for any use whenever you want to change settings on multiple nodes.  I will pull getNodeTree from nodesPrivacy in that PR if its cool by you.

![jazzhands](https://cloud.githubusercontent.com/assets/6232068/12361107/26ce5ea2-bb8a-11e5-980a-eb9c6f424c48.gif)


These `leftBound` `topBound` things are bits you get for free if you're using `BoundedSVG`. It's a bit weird, but the principle is you give your component margins and a width and height, and it calculates all these positions for you and makes them accessible.


Thanks for catching that!


We can just use static_cast instead?


@AnneTee , thanks for the feed back. Something like this?
```
// COMMENTS.
$(document).ready(
    function() {
        // Expandable comment section.

        $('#js-expander-trigger').click(function() {
            enableCommentForm(this);
        });

        $('#comment-count').click(function() {
            enableCommentForm(this);
        });

```


See: https://github.com/cabrerahector/wordpress-popular-posts/pull/98


The above isn't working for me. Something in the syntax of "this", perhaps?


Is this change necessary? `StreamOutput` should be available without the full namespace, it's `use`d at the beginning of the file.


So instead of having an array for `colours` and an array for `textLabels`, we'll instead lean on one array of `legendItems`, which contain objects that have all the properties each label needs. This also means there's no coordinating, and you won't easily get your colors and labels out of sync.


Minor - please leave a blank line after the fields.


unack, "Make run_tests.py work with virtualenv " isn't a very good description of the change you're doing IMO. More like `Catch ImportError for site` or something like that?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11682](https://teamcity.mesosphere.io/viewLog.html?buildId=105920&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:45


replaced


Sounds good. Can you reticket though, so we don't lose it?

![giphy](https://cloud.githubusercontent.com/assets/134455/12361170/93ef658a-bb8a-11e5-9700-cfb70e2a04d9.gif)


I DID IT FOR YOU @tomreese!!!


Grr, thanks! I can't actually access the page above. The error is:
/mt-dev/xrd/xrootd/src/XrdFileCache/XrdFileCacheFactory.cc:500:38: error: no member named 'st_mtim' in 'stat'
                  accessTime = fstat.st_mtim.tv_sec;

I'll change this to st_mtime, like it's done elsewhere in the code.


ouch :) it did work originally, then I guess we screwed it up adding 'home' later.


Please add a comment line explaining the reason for using type on Unix amd64 but lvType in other cases  (preferably with an example).


Oh weird, this commit was actually by me, so I'm guessing this was ported as Mike's cherry-pick from one of the live instances @waltz? In any case, heads up that while this seems to be correct, it really needs some test coverage to be verified.


You could of at least made it have a option to summon 1-5 or something.


Why can I not see it on Atom?

On Fri, Jan 15, 2016 at 12:43 PM, mancusop <notifications@github.com> wrote:

> @Samana04 <https://github.com/Samana04> Added this - need you to review
> and proof and/or suggest a better structure
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mancusop/UoGd7-Training/commit/283b182005728a568e7bdcd13b4d9fe16648dd05#commitcomment-15480187>
> .
>

--
*Sincerely,*

Saman Asif
Undergraduate B.A.Sc
University of Guelph
Guelph, ON  N1G 2W1
asifs@mail.uoguelph.ca
(519)-824-4120 ext: 52869


I'd do think the import should stay on the top, you could maybe just do `getsitepackages = lambda *args: []` in the except clause, that should simplify it.

Also it definitely needs at least a short comment to explain why youre doing this.


Great, Torben. Now I'm waiting for the next release which would include #44 and #42 resolved.


@dkayiwa is this required for the standalone to work?


Each item comes back with a group containing one rect and one text. For simplicity, I'm using a translate on group to assign their y positions instead of assigning them separately.


Какой-то скудный вовсе получился default.cfg, может его разбавить комментариями различными и примерами?? Да и обновить не плохо было бы с последними функциями.
Я конечно понимаю, что есть README, но -- дублирование системы повышает надежность...

Типа вот такого???
```
# nodename -- условное название ноды.
# нода (node) -- это станция на которой поинт получил авторизацию
nodename station13

# editor -- команда вызова текстового редактора
editor nano

# color_theme или theme -- имя цветовой схемы из директории themes/ без расширения
# если тема не будет указана, то прилипнет тема согласно настройкам терминала
theme default

# nosplash - указание этой опции отключает splash screen при старте клиента

# to - имя, по которому клиент будет определять сообщения для помещения в карбонку
# карбонка (carbonarea) -- место где будут сообщения с указаным упоминанием имени
to John Doe

# node - адрес для работы с нодой
node http://spline.rooker.ru/ii/ii-point.php?q=/

# auth - строка авторизации для отправки сообщений на ноду
# по умолчанию авторизации нет, для получения авторизации свяжитесь с админом ноды
auth no

# echo - название эхоконференции и описание
# эхолист (список доступных конференций) -- можно взять у администратора ноды
# либо в случае со station13 посмотреть тут http://spline.rooker.ru/ii/
echo ii.14 Обсуждение вопросов, связанных с ii

# archive - название и описание эхоконференции в архиве
# (архивные эхоконференции доступны только для чтения и не синхронизируются с нодой).

nodename mira
node http://ii-net.tk/ii/ii-point.php?q=/
auth no
editor vim
to Jonnie Doe

# эхолист для станции mira
echo pipe.2032 Межсетевая болталка

# в данном примере caesium.cfg приведены настройки для работы с двумя нодами
```


LGTM.


Serversiding it may also fix the issue and would be a welcome fix, as well. Do you have a reference for the Minecraft concurrency model?


something's wrong with this factory... need help!


@7andahalf  , I did some enhancements to your code. Try to understand it. From now on try to follow object oriented approach.


Ah- that was in the commit as I had it.. I'll take that out... thanks.


Sorry @flowersw you're right - please disregard


:smile:


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1504 of 1504 tests passed, 0 test warnings)

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


looks like you have a typo


Hi @matiasbeckerle , in this branch I have added client static files, koclient.js has been generated with _Browserify_.

The first commit (578d10334733bf2d3ef4c1a159a778ce4a0dd192) corresponds to the source code from 34f176404400c3d262fec03a23135761dee7e5b1, when the library rfc6902.js were being referenced as external script file in koclient.html.

The second commit (a8b76b511d16fea9cd8b7638f8ec1b32837c357a), this one, is after [replace rfc6902 reference by a reference to a Node package](https://github.com/andresmoschini/board-poc/commit/098237c125dfc3ba19108d35e6cc8c7f69d7998e#diff-15567d757b2e1b77b6990f39dafe4460R3) and corresponds to source code 098237c125dfc3ba19108d35e6cc8c7f69d7998e

As you can see, at the end, koclient.js includes rfc6902.js sources, in the same way than with my own modules like common/utils, common/model and view-model.

I hope that this brach be useful to you, if you are interested we can talk on Monday about it, I think that Browserify is great and will be useful for us in different scenarios, not only with _isomorphic_ applications.


This is the thing we really want to avoid. Components are supposed to be "generic", in that you could theoretically apply them to an arbitrary chart with (in this case) an arbitrary number of categories. But this way, the legend must have four items. (Also, if you need to change something in the way they're drawn, or there's a bug, you'll have to edit four lines individually, instead of one. In general, try to avoid repetitive code for this reason.)


void Time_idiv_int(sf::Time& left, sf::Int64 right)
void Time_idiv_float(sf::Time& left, float right)
do nothing to change the output "& left" parameter sf::Time& left ... one solution is to use the "/=" operator as follows in the two offending functions;

void Time_idiv_int(sf::Time& left, sf::Int64 right)
{
	left /= right;
}

void Time_idiv_float(sf::Time& left, float right)
{
	left /= right;
}


I _think_ that rather than being removed, this should call [`get_last_public_outgoing_event`](https://github.com/mysociety/alaveteli/commit/bc743d9fc8c8f740f37b91cbe374c6ae20b10619).


I'm not even saying it needs to be changed back or whatever. Most of our users probably never even get to see those anyway. `¯\_(ツ)_/¯`

Just everyone calm down and don't get offended just because someone dares to touch your code. None of this is set in stone, it can always be changed or rolled back. Might even be that someone actually improves your code. This is a team effort after all.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


With the project dwindling, I can also get on board with simply not bothering at all.


Actually, it *is* an includer, but not on a filesystem.  But I'll let this pass.  :-)


On Tue, Jan 12, 2016 at 4:30 PM, Mathieu Stefani <notifications@github.com>
wrote:

> Definitely more work but probably
> the cleanest (I think this is what you wanted since the beginning).
>
> Yes, basically, every-time the AugmentationLoop calls the toAugmentors,
> now, it should invoke a special AugmentorInterface member function. So
> that the HttpAugmentorInterface will send the message through http where
> as the ZmqAugmentorInterface will send it over zeromq through the
> ZmqNamedClientBus. It's exactly the same scheme as the BidderInterface.
>
> So, basically, the AugmentorInterface should have a sendAugmentMessage
> pure virtual member function that both ZmqAugmentorInterface and
> HttpAugmentorInterface will override. Then, the AugmentationLoop will
> call augmentorInterface->sendAugmentMessage(auctionId.toString(), ...).
>
> Does it make sense ?
>

Yes it does, I just pushed the changes, let me know.

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Motrixi/rtbkit-public/commit/ddc27da16d547825bd94c1a34560df4d235b0ac0#commitcomment-15406054>
> .
>


Could also look for a `playbook.yml` located within `$project_root` and the user could modify the role lists etc as long as they change the paths to point to the drupal-vm subdirectory.


CFile might be a better vector interception. But you do have to remember NSUserDefaults size limits, not everything can go in there and you must strip the spaces/linefeeds off the xml or the used size explodes. This was easy to do inside xbmctinyxml implementation intercept.


Why defined(_TARGET_AMD64_ ) in the condition?

the logic inside 'else' block is dealing with SIMD types, shouldn't

#if FEATURE_MULTIREG_STRUCT_ARGS && defined(FEATURE_SIMD)

be sufficient?


DIdn't knew about the sha256sum, learning new things every day!


Does this has to be put into the header?


I made it more generic because I thought it read better and required less thinking. Strictly speaking we are gossiping about stuff that isn't "response to queries" as we're gossiping SCTs we get from SCT Feedback - which is not a response to a query to a log.  But I don't feel strongly about this change.


I think we need to also check to make sure the options field is set, otherwise we get an error as desccribe in [Issue 17](https://github.com/roundhouse/FormBuilder-2-Craft-CMS/issues/17)

```
$options = isset($field->settings['options']) ? $field->settings['options'] : '';
```


Actually, I just realized that I jumped too soon!  The SHA has to be [regenerated locally as per the Gradle instructions](https://docs.gradle.org/current/userguide/gradle_wrapper.html).


this.voltageControl.


Was this change intentional?


I get a spec failure on this commit:

```
Failures:

  1) ResponseController when submitting a followup message logged in as the request owner when not sent preview or reedit params displays an error if the request has been closed to new responses
     Failure/Error: expect(deliveries.size).to eq(0)

       expected: 0
            got: 1

       (compared using ==)
     # ./spec/controllers/response_controller_spec.rb:193:in `block (5 levels) in <top (required)>'

Finished in 13.48 seconds (files took 26.06 seconds to load)
21 examples, 1 failure

Failed examples:

rspec ./spec/controllers/response_controller_spec.rb:184 # ResponseController when submitting a followup message logged in as the request owner when not sent preview or reedit params displays an error if the request has been closed to new responses

Randomized with seed 14950

```


Sure.


TeamCity pull requests :: netty [Build 1041](https://garage.netty.io/teamcity/viewLog.html?buildId=4793&buildTypeId=pulls_netty&guest=1) is now running


Ah, well since it's napoleon that does the actual parsing, I think it should be fine then. Wasn't aware that was an alternative.


thanks for the quick fix @bonii-xx :)


It is a modified DLT lstm, so I would call it platoon_example_lstm or something like that


00-HTML-CSS-basics/01-HTML/01-The basics of html/styles.css

Minor: always try to use #CCCCCC instead of #CCC, IE old versions do not like abbreviated 3-digit hexadecimal shorthand color values.


Can you add the job name there?


Maybe, than just `import site` at the top and than use `hasattr(site, 'getsitepackages')` condition?


This is O(n) but it doesn't have to be. Make `current_trace` a set?


I think this check was missing from the previous commit.


On the topic of size limits, you can store an Object in NSUserDefaults... have you ever thought about compressing the data using gzip in addition to removing the whitespace?


You're welcome. I guess.


I debated about the condition here, as we currently don't have other targets with these two conditions - but this would not necessarily apply as-is for other targets. So I opted for the more conservative approach of conditioning it on AMD64 as well.


Can you either not shadow `name_to_class` or explicitly use the one in the enclosing scope?


it used to be a set up until the point I started writing the error message. when it turned out that having the actual loop is quite nice


That difference pre-existed this refactoring, but I will add a comment.


第一次从电脑中同步到这里来


`description` should be a separate annotation?


Make it a dict then: class -> encountered index. You can extract the list from that in case of error.


Thanks!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11683](https://teamcity.mesosphere.io/viewLog.html?buildId=105934&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:04:40


nitpick: `circular dependency` rather than `infinite loop`


oh cool! so that's how you get it to accept multiple args. I gave up after passing in a vector. I should remember to try using multiple args like this in analogous situations.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1523 of 1523 tests passed, 0 test warnings)

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


Just curious, do we need to support this case (I want to use a shortcut AND pass some extra targets)? Vs. simply overriding.


Added in this commit, prep linux-next for next build, remove duplicate doc files


I'd still do the lambda thing I think because this way you don't have to change the code below. So instead of all usages (which is only one in this case but might grow), work around it at the import


Done.


Haven't thought of it that way


I find it incoherent between `TypeParameter` and `SqlType`. Maybe `SqlTypeParameter`? I don't feel strongly about this. But I would choose `SqlTypeParameter`.


`Boolean` need to be all CAPS


@boushley - right,  I think we want to use the two files no-external.debug and external.debug instead now and eliminate the debug file that has no reliable source map?  If this works I can do the work to update the samples like you just did for the new all.debug ;)


well spotted - thanks :)


I would add `        // ...and so on for each native container type` here as well


Done


rapá, cê tem qi currigi essa coisa do seu userneime, cê botou anderscore e os programadô não encontra seu répo!


To make that point perfectly clear: There are no two parties that need to not get offended here. This is just about Ian having to accept that his code can and will be changed, especially if its UI and naming is completely not understandable. More so when prior critique in that direction gets ignored.

That sounds like starting a fight, but that is not what that shall be. This is just something that he finally has to learn. I'm not doing the egg-shell dance around Ian again and again.


@RReverser Thanks, see https://github.com/shapesecurity/shift-parser-js/issues/259. Fixed in https://github.com/shapesecurity/shift-parser-js/pull/260.


I don't want to go there. Either I can simplify the message "Encountered self-reference in dependency chain of blah" or just accept O(n). This function will be called rarely and we are not using python for efficiency anyway


How about abstracting this logic in a routine lvaArgType()?  In fact we already have such a routine for Windows Amd64.  Looks like instead of extending it for Unix amd64, we have duplicated quite a bit of logic.  I see the same duplication of code happening even in genJmpMethod().


Should this be

```julia
pi_epsilon1 *= kernelpdf(epsilon1[i], d)
```


Note to self that I got to here while reviewing on Friday evening


Should this be

```julia
pi_epsilon0 *=  kernelpdf(tune.epsilon[i], d)
```


Appreciate it, that explains the strange concurrency issues I was having. Looks like I was trying to metaphorically hammer a nail with a screwdriver.

I will properly make the static block swapper maps server-sided only.


This revision is clean.


Yep, this just confirmed it when I updated to the `2.10` version to get the new SHA.  :siren:

![screen shot 2016-01-15 at 12 41 02 pm](https://cloud.githubusercontent.com/assets/989928/12361713/5189528c-bb85-11e5-8909-0f935a6e0b23.png)


Good catch. Done


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11684](https://teamcity.mesosphere.io/viewLog.html?buildId=105944&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:16


Is File::Temp core?


I would do 2,4


This method needs LOTS of refactoring, and I consciously decided to defer that to a future refactoring change, rather than conflating refactoring changes with functional changes.


Just realized as I was backporting to v3.0 that the message/error could change per version.  I'll update this to state that it'll return an error.


we can replace RHS with regType?


Dope commit bro mind if I fork this?


nit: indentation


Done


It should already be there: http://search.maven.org/#artifactdetails%7Ccom.tapstream%7Crollbar-logback%7C0.1.2%7Cjar


> Why?

I'm taking the time to belabor this point not because it is important in this one case, but because I think it will be valuable in the long run if the team agrees on this.

Basically this discussion is about the Robustness Principle, which states "Be conservative in what you do, be liberal in what you accept from others". This principle originated in protocol design, where following it helps facilitate extensibility: if you allow any value for a field in a packet, then later versions of the protocol can use that field without breaking backwards compatibility.

If we try to apply this principle to programming language design, we get Javascript (without linters). The reason we use linters is the reason why the robustness principle is ill-advised here: flexibility creates complexity and complexity creates bugs (see here https://www.destroyallsoftware.com/talks/wat).

I think everyone would agree that it takes more code to check for 4 literals than 2. In this simple case it isn't a big deal, but if we were to follow the _principle_ of extending flexibility maximally to all non-ambiguous cases, we would be forced to also accept "yes", "no", "on", "off", "enable", "disable", "✓", "✗" etc, all possible casings of these, translations of these into every language, and so on. If we draw an arbitrary line somewhere before accepting all of those, then we are violating the principle, but if we actually implement that we are wasting time and creating complexity that will likely lead to bugs.

Making things as strict as possible minimizes the amount of special cases and alternate code paths, which reduces cyclomatic complexity and code size, ultimately increasing productivity.


See @svanharmelen https://github.com/go-validator/validator/pull/43
TLDR: it wasn't "working" before it was just skipping calling the `nonzero` validator when it saw the struct. So if you had anything other than `nonzero` there it wouldn't have worked :P


Fix bugs. Don't do design and fancy things now.


Since `_cha.getCurrentHp() == 1` is fairly infrequent why don't we consolidate the check into one line, something like:
```
} else if (_cha.getCurrentHp() == 1 && _cha instanceof L1PcInstance) {
    damage = 0;
}
...
```
Even if we reject that for other reasons, we probably shouldn't set `damage = 0` then set it back to if the target is an NPC since the actual application of damage (after resistance, hard skin, etc) could round that down to 0 and have the NPC survive (I think--been awhile since I've been there--ignore if you've tested already).


Not sure if we want to mint a new object rather than mutate the current one. I see the logic but not sure it's the IO::All way.

Now the IO::All way is up for change and discussion, but I'd like to keep this change in line with current.

I'll think on it. You can push that test file to a 'temp' branch on my repo.


make this a static method and remove the self parameter?


Can you please indicate what sort of testing was done to show that this is an improvement over the Denver memmove routine used on AOSP? In my testing, using bionic's memmove benchmarks, I couldn't find anything conclusive.

Before
BM_string_memmove/8               100000000         12   657.29 MiB/s
BM_string_memmove/64              100000000         17  3667.40 MiB/s
BM_string_memmove/512              50000000         37 13729.53 MiB/s
BM_string_memmove/1K               20000000         99 10300.25 MiB/s
BM_string_memmove/8K                5000000        409 20021.83 MiB/s
BM_string_memmove/16K               2000000        923 17739.73 MiB/s
BM_string_memmove/32K                500000       3156 10379.72 MiB/s
BM_string_memmove/64K                500000       6252 10481.72 MiB/s

After
BM_string_memmove/8               100000000         14   560.43 MiB/s
BM_string_memmove/64              100000000         11  5782.22 MiB/s
BM_string_memmove/512             100000000         26 18984.04 MiB/s
BM_string_memmove/1K               50000000         56 18105.63 MiB/s
BM_string_memmove/8K                5000000        471 17384.01 MiB/s
BM_string_memmove/16K               2000000        973 16829.21 MiB/s
BM_string_memmove/32K               1000000       2429 13489.39 MiB/s
BM_string_memmove/64K                500000       4726 13866.15 MiB/s

Also, not sure which sizes are used most often (or how to profile that).


Minor - since you are at it, this comment line is not aligned with the rest of the lines in that block.


Recommendation:

```rust
pub type ExampleResult = Result<(), Box<Any + Send>>;
pub type ExampleFn = Box<FnBox(Arc<Mutex<WorldState>>) -> ExampleResult + Send + 'static>;

// ...

examples: Vec<ExampleFn>,
```


I would recommend we reference `CertiPay.Common.Testing` for our unit testing needs instead of the Visual Studio one (though I'm sure that was your next step).


:byefelicia:


We should add a test for this (if it isn't there already).


🍴 you bro!


I actually was just adding a stub for the project


replace `utils.isSubdir` -> `utils.isSubpath` in Repositories.py


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


fgfdfdhdfhfd


hfdhfdhfdhfh


it seems that there is a new image here

http://ftp.sliim-projects.eu/boxes/kali-2.0-amd64.box
http://ftp.sliim-projects.eu/boxes/kali-2.0-light-amd64.box

please see https://github.com/Sliim/pentest-env


Not sure whether we will be testing ARM32.  Please add a comment line the reason for this change.


That works.

We should also remove `dash.all.debug.js` from the Grunt `copy` task.

Unless you want to, I can pretty easily create the PR that fixes this.


Not yet added effect of the lethice piercing on imps enc rate in this area :(


Done


@michaelficarra /offtop Hmm, is this some kind of Github bug? Otherwise what is says doesn't make any sense. ![screenshot-github com 2016-01-15 18-48-18](https://cloud.githubusercontent.com/assets/557590/12361907/bb17cc32-bbb8-11e5-899b-02b02a2d9a51.png)


Maybe a tool like [autoenv](https://github.com/kennethreitz/autoenv) would allow us to use a `.env` file for local / Heroku environment variables? I'm going to look and see if there is a recommended way to do this with Python. Rails apps and Hubot, for instance, use `.env` files for configuration, which prevents having to modify your `.bash_profile`.


Not really - the semantics here are subtlely different than getRegType, because it deals with how the incoming register arg is STORED, not how it is PASSED. Again, this whole method needs refactoring so I think it should stay as-is until then.


kjjkkjjkkjjkjkjhkjj


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11685](https://teamcity.mesosphere.io/viewLog.html?buildId=105945&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:35


Done. Also changed atom -> box


Just simplify the message then, because we'll have hundreds of classes soon and O(n^2) for no reason is not going to work.


That's correct: the value would depend on the type of `Value`, but only signed types are used.  For the two places where we call `get_range` without providing defaults, these are the desired values.


Cheers!

Heh, I just clicked your name and it was who I thought it was (thought the pic was familiar). Hope your break's been going well! :P


missing timestamp at line beginnings. Please add back:
"[" . date("H:i:s", $now) . "] " to line 199


we should use a prefix for all commant that we are reserved for platoon. Rename platoon_get_job_uid


idem


Yeah, looks like it.


Nice. Done


`http://support.aerofs.com.com/` should be `http://support.aerofs.com/`


I'm on it. Will happen in a short while.


If BlockEntity belongs class Entity then I can use the Entity functions (ride and other)


FIXED vmalloc build problem


HALF A THOUSAND COMMITS!


Which problems do you mean? If you'd like absolutely the same behavior as `echo`, the command should have been `printf '%s\n' "${kak_opt_clang_errors}"` and not `printf '%s' "${kak_opt_clang_errors}"`.

Using `echo` is really dangerous (POSIX doesn't forbid additional arguments for `echo` and on different POSIX systems, `echo` has widely varying arguments which might easily appear in `kak_opt_clang_errors`).


/Class should be /class


Ugh correct silly mistake as I am rushing at all levels.   Please do if you have time ;)  Thanks @boushley


This case is not changed, it is merely folded into the code below that does general handling of MULTREG_STRUCTS (and avoids making an implicit assumption that the next slot uses REG_NEXT). This method needs MANY more comments, and I don't think it is useful to attempt to fully articulate what's going on until it is cleaned up and refactored.


Schedule.html should be schedule.html


This now uses recipe_engine/step, but the example later when adding unit tests still uses just 'step', presumably that one needs an update too?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11686](https://teamcity.mesosphere.io/viewLog.html?buildId=105949&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:05


I think I'd prefer `getsitepackages = lambda: []` since it's way shorter and somehow better readable to me and easier to see that it's a workaround replacement.

@Makman2 @Uran198 what is your opinion on this?


Ha, quite. I finally got down to learning Forge again, though documentation is still a little spread out and random. Looks like I confused another game's threading model with Forge's. Anyway, back to trying to fix things - commit comments are not the place for idle chatter.


and more importantly why


This indentation is a bit weird?


@mikemcquaid what is the `cat` command being used for??
If I remove it, it works fine... but with `cat` and then pipe `brew...` I have to hit a key to continue...


I find it strange that this exception is associated with interop. What stops non interop code (unsafe, pointers etc.) from throwing this exception?


whoops! :) don't think this should be there? Sorry my comments which is why i didn't make a pull request


group should be grouparg, right?


Does it make sense to include obsolete members in a new contract?


the => The

it pains me to pick this nit, but I will


Its ok, I removed it with the last push


@fastest963 very nice! Thanks for the great support and super fast fix!


Indeed, thanks @byronmccollum. Will fix.


Seems ok to me at first glance


Why do we need this short cut and what are we saving? can't we simply return the correct type handle instead of SIMDFloatHandle always?  If some one is relying two SIMD types being the same by doing type handle comparison, it would not be correct since the base type could be different.


It works in both MSVC 2013 and 2015


@wfenchel This is a good question. I have put in in meantime only in search because I believe it is the most important one and also because for me it isn't yet clear what  role the puts in qsearch play in this context. As far as I remember they act only on the static eval not the value, right?


unrelated but cleaning up from previous commit.


Test fix in bdc009e.


Yup, good catch.


Yes, leaving it as a todo is fine.


From what I gather, I think this function should only be called on clients for local explosives. `FUNC(setPosition)` calls `setVectorDirAndUp` which seems to be global.


OK, fair enough, I'll drop it. I also realised that it means that `--allow-newer` doesn't really work with `HTTP`, which makes it a practical annoyance as well as a theoretical mistake.


Don't you also need logic in when connecting to the database to construct a DSN appropriate to the driver? They might have different syntaxes.


PDF


[modelo Canvas.pdf](https://github.com/edmundo3912/Ingeneria-del-Software/files/92363/modelo.Canvas.pdf)


Since I called `Underscore()` with nothing, the `_wrapped` property should be undefined. `_wrapped` should equal whatever argument I pass into `Underscore()` I believe.


What about this?
These are implementations for when a type parameter binds to ~~a concrete type with~~ a specific native container type


I think it's the tabs setting in the Github viewer. Here's how it look in my editor:

<img width="536" alt="screen shot 2016-01-15 at 2 02 22 pm" src="https://cloud.githubusercontent.com/assets/1163840/12362401/1dddf1de-bb91-11e5-8210-17fa29a3f33e.png">

Let me know if there is a preferred indentation style.


could we have used // to fix the deprecation warning instead? We used that elsewhere and this is clearner I think.

But if it won't I won't block this PR for this.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11687](https://teamcity.mesosphere.io/viewLog.html?buildId=105953&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:06


I shouldnt leave commit descriptions like that ...
I comment like i push. thats the i are :)


What this fix?


Looks good! However we need to do two extra things here:
 1. Make sure that this is not a hard requirement. In a similar away with 840834def55473dd51cb54ba97d0b57c934d71fe .
 2. Add the package to the requirements list


shouldn't you be using key_xyc instead of key?


This is needed because previously we did not require type handles for SIMD types beyond the importer, so SIMD type handles were not always propagated. I am of the opinion that we should NEVER require them post-rationalize. However, as that is not currently the case, this approach was taken to avoid an even more massive set of code changes.


instead of calling `HostedCluster.StopAllSilos();`, please call `base.Dispose()` (after the call to delete azure queues)


LOL, I forgot to commit here the most important change. Instead of this here you have some part of my actual work on coloring algorithms. Fix for swapping is here: https://github.com/buddhi1980/mandelbulber2/commit/411b565ff102debfb1fa691d57ebd05ed4a3650e


TeamCity pull requests :: netty [Build 1040](https://garage.netty.io/teamcity/viewLog.html?buildId=4792&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6372, ignored: 23; resource leak detected (new) Build time: 01:23:25


TeamCity pull requests :: netty [Build 1042](https://garage.netty.io/teamcity/viewLog.html?buildId=4794&buildTypeId=pulls_netty&guest=1) is now running


@antograssiot Why do we have `3.0` and `2.0` in quotes?  Just curious.


this is only a test


Hi @chuckwired
Are you sure this DragonRise joypad has the same product ID of this? --> https://github.com/libretro/retroarch-joypad-autoconfig/blob/master/udev/DragonRise_Inc.___Generic___USB__Joystick__.cfg

It's messing up with my joystick, take a look at https://github.com/libretro/retroarch-joypad-autoconfig/issues/139


For number 1 I missed that. Thank you for remember it to me.
For number 2 it has been done.


TeamCity pull requests :: netty [Build 1041](https://garage.netty.io/teamcity/viewLog.html?buildId=4793&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6372, ignored: 23 Build time: 00:42:05


Fixed:
Braces were added for all single line if statements. Verified new files for the coding conventions.


ack


@rodrigopavezi Curious, any idea why this was previously referencing `RiseVision.Image`? I don't see any changes to the src (non dist/ files) ...


wouldn't this REM prevent from running TesterInternal tests in the meantime?


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1507 of 1507 tests passed, 0 test warnings)

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


ok cool but you need to cater for the situation when the .mirthless file does not exist within the users home directory as it crashes the game upon starting. I made changes to mirthless file, check it out please when you have the time.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11688](https://teamcity.mesosphere.io/viewLog.html?buildId=105955&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:04:01


Is there a point?
It's not seen by the user ever, and it cannot be launched twice on the same port.


Done. This could probably do with some sanity checking on the metadata and also verification that the input ambisonic metadata keys are valid. Given that the audio metadata is hard-coded in gui.py I think it's ok for now. Let me know what you think.


@sils1297 although you don't follow pep8 in your tests, I wrote it this way because of pep8 E731.
It's more explicit and readable in my opinion.
The benefit of lambda expressions over explicit def is that they can be part of larger expressions.


I probably could have mocked one of their providers, but I figured if its working for one, then we assume its working for all of them so I just picked the easiest to test.  Along with the fact that its not this code's responsibility to verify all their providers work


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11689](https://teamcity.mesosphere.io/viewLog.html?buildId=105957&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:38


nice cleanup


Is this nee


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11690](https://teamcity.mesosphere.io/viewLog.html?buildId=105959&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:09


What is all this shit for. I see Katana. Okay. ImagicalMine, more mess for you.


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


I'm not sure what makes that better.  I did just clean it up a bit though.  I've moved the version bump functionality to its own function, and improved the variable naming and comments.


коммиты пока еще зашкварные, "только для чтения"


Sure, good idea, I'll add --no-rdoc etc. in both places we install gems.

On Fri, Jan 15, 2016 at 7:29 AM, Joseph D. Cohen <notifications@github.com>
wrote:

> maybe: sudo gem install bundle --no-rdoc --no-ri
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pellaea/developer-startup/commit/b2086cded06bbc23139bcc59ac6e8e6e85c1bacc#commitcomment-15477016>
> .
>


Explosives that are placed in the editor would be local to the server, won't they?


Great job, dude!


сессий сильно много получается, даже тесты валятся


We should use `Done` here instead of `Receiving`, since we have read the complete response body.


I changed it because we're submitting more than SCTs, we're submitting SCTs and certificate chains; and the sanity checks we define operate on the certificate(s) as well as the SCTs.


Hi Jameo,
I have not used this library in a while, but I think the correct syntax to use the run_continuously function is similar to this:
schedule.every().day.at("10:30").do(job)
cease = schedule.run_continuously()

The scheduler should then call run_pending on its own, you do not need to call it explicitly in a loop.
To stop the scheduler, you can do:
cease.set()

I used pretty much the same instructions in my personal project. If you want to see what it looks like, you can find the relevant code here: https://github.com/JeSuisAbraham/dorsalfunbot/blob/master/cafe.py
Note that I used a dedicated scheduler rather than the default scheduler but that should not be necessary.


Files must end with a new line.  See [Style Guidelines, General Rule, #5](https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md#general-rules)


### Update
Will need to update optimizer/Dockerfile as well if aspnetbase's image name is changed to _aspnet:1.0.0-rc1-update1_


Should this be  varTypeIsSIMD(src)


Good catch. Thanks!


This was an existing problem in the code.


Issue #1 this is where it needs to be changed @PurdueAgComm/designers


nice cleanup here as well


Sure.


[Edmundo_SolanoR_modelo Canvas.pdf](https://github.com/edmundo3912/Ingeneria-del-Software/files/92378/Edmundo_SolanoR_modelo.Canvas.pdf)


Is this the correct way of using multiple abbreviations? Will using min for multiple units cause a problem?


Reviewed and good to go


Same here: Is this the correct way of using multiple abbreviations? Will using sec for multiple units cause a problem?


This is missing the TRACE value.


Problem with "CMS" is that I'm always thinking "Content Management System"


It took me a bit to get what is going on in this section, can you add a comment summary at the start of the conditional explaining the overall logic?


I recently converted an app from Rails 3.2 using RailsAdmin and CanCanCan to Rails 4.2 with RailsAdminPundit. After the conversion, our dashboard was taking over two minutes to load, due to `select *` type queries to four of our tables that each had more than 500,000 rows in it. I tracked the slowdown due to the `.all` above. Locally I've removed the `.all` and I no longer see the long queries and my dashboard appears to work.

I'd like to submit a PR that will both solve our slowness issue but not interfere with Rails 4.1. However, I don't see how the comment for this commit applies. If it's obvious to anyone reading this, please let me know. Otherwise I'll do some experimentation.


You're referring to "When an HTTPS client connects to an HTTPS server, the client receives a set of SCTs as part of the TLS handshake, as part of the leaf certificate, in an OCSP response, or via a TLS extension."?  How about:

When an HTTPS client connects to an HTTPS server, the client receives a set of SCTs as part of the TLS handshake -- either as part of the leaf certificate, in an OCSP response, or via a TLS extension.


Sorry... I recommend to hold on on this one!!!


There are so much DragonRise out there.


Better to use <pre>``` js</pre> for code examples so you get syntax highlighting.


Done!


nit: why is this not just calling `this(new[] { buff }` instead?


It's not an improvement compared to the bionic ones, it's an improvement over the default arch/arm ones.


Tole vrni, da ti bo prevedlo poročilo.


For context: I managed to include this commit in #463 without even realizing it. Apparently pushing to my fork when #463 was open resulted in that pull request being updated to include this commit.


It's used for a Bash heredoc. If you can file a reproducible issue I'm happy to revert this.


Well, we don't need a live-skel package at all. We should revert that. Since live-session never ends up on the installed system and sometimes user need to change stuff, it would be better to keep it as it is. This is an overlay and should be used as such.


This is a codeaurora commit, ask Qualcomm :)


There is a real website CRITTERBOB.COM .  Please invent a different URL for a nonexistent domain.


There's a regression on staging against master for quite a few darwin builds (transitively): http://hydra.nixos.org/build/30554365/nixlog/1/raw


Let's use the [JSDoc comment format](http://usejsdoc.org/), so that this documentation can be extracted and formatted later.

For an example of the toolchain we're using and what the output looks like, go to `five-bells-shared` and run `npm run jsdoc`. It uses JSDoc 3 to render: https://github.com/interledger/five-bells-shared/blob/master/package.json#L14


I mean, I don't know if this exact commit is the cause.


Unrelated change to drop the `#to_str` here?


Should be portable to Windows, but haven't tested it.


Ditto, dropped `#to_str`.


¿Por que se me saca del authors.txt? Siendo que el 99% de los commits del proyecto son míos.


You need to add a trailing space after each line, it only "worked" because it was the last line of the file
Also, it failed to build:
http://hastebin.com/buyosuwago


done


I'm not fluent enough with what's considered proper around here. I, personally, do not mind having several components in a file (after all, you don't have to export it, if it is "private")


I added it to the top so that it showed that this letter was in response to `dear-github` but not discrediting each other and that both groups are thankful for GitHub and all they do for us.


The situation I would like to get to is where we never retain raw response data in addition to the parsed representation.

* For revalidation, we should retain the etag and modified date only.
* For checking if something changed, we should look at `response.notModified` only.

Then the invariant for `response.data` will be that it is null if and only if `response.notModified` is true.

As an further goal, I'd like to introduce a variant type for `Response`, with variants for "normal response", "not found", "not modified", and "error". The body would be accessible only for the first of those four.


The `1` here was to ssh into the first result in the list, same as without the filtering. This allows you to filter by name, then ssh into the nth server of the filtered list.
The next line, 43, was making that connection.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1507 of 1507 tests passed, 0 test warnings)

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


Hmm. I'm not having that same issue. For me it builds just fine.


Got it, sorry, adding it back..


Same Typekit as the main Mule site, since we've got the same fonts here. Will work on `mulegallery.com` and any subdomains.


sorry if my docs didn't make that clear :frowning:


leave it. that's not critical :)


I've never seen this syntax before. Is this correct?


It would be a little more realistic you determine the errorCode first.
The responseCode is left 0 if there was an errorCode != 0 (since your
simulation could not connect to simulated server), and otherwise
pick a random responseCode as you are doing now.


Борец с очепятками в readme не дремлет 😊


Required at least by ```c_rehash```


what does std::move do here?


Reformatted Mailchimp form a little bit for a few things:

* More clear markup order (because I am anal and I like form fields to coëxist in harmony whenever possible)
* Slightly bigger and more readable form elements using Proxima Nova rather than the default Helvetica
* Slightly nicer responsive display using `flex`


ceil() always outputs a float value


Yeah, dropped it when moving around since it does nothing.  I can push that back/new PR/different commit.


I figured we'd want Twitter in here. Right?


Looks good to me.


Variables for re-used values


reduce_redimensions (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


No, because in future we could have other known-sized struct types that are not SIMD, but which could be referenced with a GT_LCL_FLD.


There were two classes that did this, and now there is one. Bada-bing, bada-:boom:.


This looks like a typo to me...


arm-linux-androideabi-ld: warning: unwinding may not work because EXIDX input section 24 of arch/arm/common/built-in.o is not in EXIDX output section this fixes


```
lfe-cowboy-examples @ ca02f66f1ddba8b15fec9f6bad2bf241a8e06a5b
ERLANG_MK_VERSION = 2.0.0-pre.2-60-g1e6fc7b
deps/lfe @ d7bc80da04aedd0a607f17d8d149eba363872b4b
deps/lfe.mk @ d4432fac6837f22ebaf8c1b1c903ff1b94f6c7dc
```
```
$ cat /usr/local/lib/erlang/releases/RELEASES
```
```erlang
[{release,"Erlang/OTP","18","7.2.1",
          [{kernel,"4.1.1","/usr/local/Cellar/erlang/18.2.1/lib/erlang/lib/kernel-4.1.1"},
           {stdlib,"2.7","/usr/local/Cellar/erlang/18.2.1/lib/erlang/lib/stdlib-2.7"},
           {sasl,"2.6.1","/usr/local/Cellar/erlang/18.2.1/lib/erlang/lib/sasl-2.6.1"}],
          permanent}].
```


It is there of course, but what I hear from ledvinap and dominic not part of the coding style to keep it safe.
Will remove it when I make a pr to cf


This was an accidental revert, re-reverted later on.


cuuuuuute


Bit design and fancy things are cool
But everything is crashing
'oh, look this programm has an function to disable logging to increase lag! And it crashes! So much GOOD.'


time... twice?


check this patch
https://github.com/dorimanx/DORIMANX_LG_STOCK_LP_KERNEL/commit/fdc10cc6ef3256332de5272e973128a12a5b08d5


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11691](https://teamcity.mesosphere.io/viewLog.html?buildId=105962&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:04:16


du solltest die _4 durch eine _5 ersetzen. ;)


Please revert this!!
It's stupid!
(Or add timestamps back and change imagicalmine to debug again and so on!)


Cool!  So the CPP11 usage is really pretty simple, even in contrib?  I gave
up on contrib with VS2008 out of lack of time, but very excited to see your
work here.

On Fri, Jan 15, 2016 at 12:49 PM Patrick Snape <notifications@github.com>
wrote:

> Update Windows scripts (steal @msarahan work)
>
> This now supports building using @msarahan script and now I can build
> using opencv_contrib for Python 2.7, 3.4 and 3.5
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/menpo/conda-opencv3/commit/1af4b349346eac1ee50b1b91b16e29e1ad963160
>


I'd advise including a code comment to the effect that "The URL's containing spaces
in this list are 'malformed' (not conforming to RFC 2396), but Crittercism will report them
anyway."
Also, since you are testing a URL's having query strings, you should test a correct URL
having a query string in addition to testing an incorrect URL syntax.


WTF Is this..


This letter is certainly a reaction to `dear-github`, but it is in itself independent. This one expresses something I wanted to say, and people are signing in order at the bottom. Happy that you guys endorse this letter as well.


ach dreck


Ok.


Yes, so you should'nt set their position on clients I think. I'd add a test like this:

```sqf
if (local _x) then {
```


[LAB1 Revised.pdf](https://github.com/wirelessglab/Infonet/files/92401/LAB1.Revised.pdf)

As per your instructions after lab1 submission, here is the PDF file.


I didn't do this for a couple of reasons.
1. Text editors and many other tools will work better if the file has a .py extension (they will know what kind of file it is)
2. I thought the taxonomy/ directory might one day contain static resources or other objects that were not cgi scripts, in which case it is unsafe to make the whole directory a cgi-bin directory
3. If you've added the directory to turn something into a cgi-bin directory, you might as well add an alias - i.e. if you're already modifying the apache config, it doesn't make any difference whether you add one new directive, or two

I guess I don't understand the issue or why this change confers any advantage.

If it's just for local testing, why not use the taxonomy/cgi-bin/browse.py path instead of the taxonomy/browse path?

This should also be unnecessary, since apache will add the extension if it's missing via the content negotiation feature (this works for .html, may require some additional configuration to make it work for .py ... not sure if it works for cgi directories though)

Another option is to make a little shell script (browse) that just invokes the python script. The question here is what is the current directory when the shell script runs i.e. can it address the python script.

Another option would be to add a symbolic link from browse.py to browse. (this requires the right apache Options AllowSymLinks or whatever for that directory)


Realized it was redundant...I'm about to change it.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11692](https://teamcity.mesosphere.io/viewLog.html?buildId=105973&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:03:40


no, IT'S UPDATED REVERT THIS


It's my commit, and I haven't got a clue how it applies either. Removing `.all` should be fine here.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11693](https://teamcity.mesosphere.io/viewLog.html?buildId=105974&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:30


Why  do we need constructor on such class to be public
(may be for spring injection...)


Yes it is. Simple way to add two arrays.


`containsMetaParameter`


Possible but then NSUserDefaults dump becomes unreadable :)
I'd have to see the size differences to see if it gains anything.


on all the tests I've done if I have

```shell
brew bundle --file=- <<EOF
tap 'caskroom/cask'
tap 'homebrew/services'
tap 'homebrew/versions'
EOF
```
everything works fine

but if I add a `cat` to the first line

```shell
cat | brew bundle --file=- <<EOF
tap 'caskroom/cask'
tap 'homebrew/services'
tap 'homebrew/versions'
EOF
```

It get stuck waiting on the `cat` process

I tried also running that code directly in the terminal, and I get the same behaviour....
You don't get that???

I don't know with the tests pass and you don't seem to see that..


Nit: let's add a newline above this line, just like in the file above this one.


You are right. Thanks.


oh yes this is unused, dead code of my first iteration I will clean that up


Hey @IstvanD!

Thanks again, your fix is excellent! There was a border case where it failed: if one moves an instance to the column of the first instance, then the start date would change:

![first-day-of-recurrence](https://cloud.githubusercontent.com/assets/156064/12363079/726048f6-bba5-11e5-8368-9d5cea02ec51.gif)

This happened because the offset is correctly calculated in relation to the edited instance, but `getFirstCalendarBookingInstance()` expects to receive the currently first instance of the recurring event. Since we changed to apply the offset to the current instance, then `getFirstCalendarBookingInstance()` got crazy.

Now I adjusted the offset inside `getOffset()` itself. I have to confess that this was quite complicated, I caught myself scratching my head for a long time...

Anyway, thanks again, it would take too much more time to fix it without your fix.


nothing ;-)


reverse the `if`


Why can't we have this member in FunctionCallWrapper directly?


Don't put style directly in the html, add it in the CSS. Personally, I don't like the left menu touching the header, the margin was intentional, remove the margin-top in the CSS.


We need some handling here for the case where the setting is not set; it's an optional setting.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11696](https://teamcity.mesosphere.io/viewLog.html?buildId=105982&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:04:04


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11697](https://teamcity.mesosphere.io/viewLog.html?buildId=105983&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 207 Build time: 00:03:49


command to check memory from terminal is:
free -k<shows in KB> ,-m<shows in MB> , -g<shows in GB>


I was thinking that once we start implementing the mongo-only index annotations processing (needed on top of JPA)
we could decide what's the best option.

For now, I went with `GridDialect`, because it was available code from my first iteration.


This should only be for the regexp stuff, `$LOAD_VARS` is still better than `$:`


most are complete urls to avoid redirects

On Fri, Jan 15, 2016 at 9:06 AM, Brian Murray <notifications@github.com>
wrote:

> This looks fine thanks. Are most links on the site complete urls or just
> relative references? If we were to build a testing server the complete urls
> (with the full domain name) wouldn't work out well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ethroes/Website/commit/260f2973b014c1cc0beaacd4239f3de9b801d99f#commitcomment-15479281>
> .
>

--
Brandon Murray


ah yes. good call


I think we need it.
Its installed in the live layer, and provides some autostart stuff, such as localized xdg desktop-item links.
https://github.com/manjaro/desktop-settings/tree/master/shared/live

But I think /root can go, useless, and we should only copy .bashrc in /root.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

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


This change is not needed.  It breaks signals for `LinePlot`.


lel np sweetie xD


Done


@bberenz should this be part of `style.css`?


E sim, o Mooshak dá Wrong Answer. Outra vez. :disappointed:


Unnecessary as we're not directly passing the args to the commands pattern.


TeamCity pull requests :: netty [Build 1043](https://garage.netty.io/teamcity/viewLog.html?buildId=4796&buildTypeId=pulls_netty&guest=1) is now running


Yes. I just need to test a little bit more to be sure it is working ok...

Would you like to test Pulsar on Android L? If so, please send me an email. I actually need someone who can test this build.


unnecessary `else`


To be honest I forgot there was one. But yes, 786878d5dc88fa003b8ca410bb22ddf73cc93a89


yo, @tedsta. Remember that you can rebase to reword the commit messages :)


Following commit made me understand the new way. It is good like this.


`user_defined_value` only works in the context of mustache templates


by returning `false` here, the `InfoWindow` will automatically be shown.  I know of one person who wants to create their own custom `InfoWindow` and disable the built-in `InfoWindow`.

@see [MapView.OnMarkerClickListener](https://www.mapbox.com/android-sdk/api/2.2.0/com/mapbox/mapboxsdk/views/MapView.OnMarkerClickListener.html)

```
Returns:  If true the listener has consumed the event and the info window will not be shown.
```

Those users wishing to create their own marker-popup will want this to return `true`.


I'm surprised that the `#` in `user_defined_values.#user_agent` works, I thought I strip those out during parsing.


Should probably be `'should render a list item with the correct name for each watched repo'`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11695](https://teamcity.mesosphere.io/viewLog.html?buildId=105981&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:37


I removed downloading from the asmple app because the API changed substantially in how it supports downloading. I'm not yet sure if and how it is practical to use the current GTL library and/or GTMSessionFetcher for it. I was trying to update the app rapidly for the newer API but had not devoted time to rewriting those portions.

I'm sure it's still possible to download and export using the newer Drive API, but unfortunately for right now you'll need to figure out appropriate code for that based on the generic Drive documentation.


I literally just cloned the repo, went to the upload dir and ran "make run".


detached is a hook and therefore it should be a top level method not a method inside the `methods` attributes. This is like http://vuejs.org/guide/components.html#activate_Hook


I don't think I use `make run`. Will try and report back.


`st[sfSequence] = 0` also works


Is there any other way you know of to add the `pasv_address` without manually changing the file? Since without that address being set the ftp servers would not function correctly.


'center_changed' should not be an event. It is already a two way property. if someone wants to know when the center changed he should use $watch instead


it was in 2.0.0 version of the Cashier module.


There is a typo - competators -> competitors


Y'all are much better at responding lol


Same result. No error and it starts the app successfully and opens up an Erlang shell.


you mean <br>?


Why this empty line?


A penas subimos la carpeta y página principal, pero aún no subimos todo
Profesor en eso andamos.

2016-01-14 19:43 GMT-06:00 ffranjavierutez <notifications@github.com>:

> Excelente que sean los primeros de los 3 grupos, solo un favor pasen su
> carpeta dentro de la de *practicaGeneral* porfis.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/desdeapps3fjlrutez/practicasGit5B/commit/3615489c1c074cf5338a0b0e32514f87b9fe0db2#commitcomment-15465602>
> .
>


Utilizei `new StringBuilder();` para evitar o loop que o `sb.setLength(0);` faz com todos caracteres que estiverem no StringBuilder


Do not remove EOLs


Ok, enterado!

El 14 de enero de 2016, 20:01, AleHdzRuiz <notifications@github.com>
escribió:

> A penas subimos la carpeta y página principal, pero aún no subimos todo
> Profesor en eso andamos.
>
> 2016-01-14 19:43 GMT-06:00 ffranjavierutez <notifications@github.com>:
>
> > Excelente que sean los primeros de los 3 grupos, solo un favor pasen su
> > carpeta dentro de la de *practicaGeneral* porfis.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/desdeapps3fjlrutez/practicasGit5B/commit/3615489c1c074cf5338a0b0e32514f87b9fe0db2#commitcomment-15465602
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/desdeapps3fjlrutez/practicasGit5B/commit/3615489c1c074cf5338a0b0e32514f87b9fe0db2#commitcomment-15465749>
> .
>

--

[image: UTEZ] <http://www.licfjlr.tel/>

Lic. Francisco Javier López Riojas / Profesor
franciscolopez@utez.edu.mx / 777-332-61-69

UTEZ
(777)3-19-79-58
www.licfjlr.tel


You did not regenerate your eclipse settings.


To match the spec, should this be called "ambisonic_normalization"?


I see the code to download files was removed. Is this no longer a part of the API? What's the correct way to download a GTLDriveFile now? Thanks.


Remove 'horizontal': 1


Minor nit: I don't see how a macro is better than calling a function directly.


Oops, forgot to log out of the team laptop again, this commit was also by CursedNeon/Brian S.


For future-proofing / posterity, should these also be grabbed from audio_metadata (and filled in)?


@tj Why was this reverted? Should #3 be reopened? It's definitely a feature I would like.


O padrão correto é sem hífen, mas vou deixar passar dessa vez :ok_hand:


This is a change, not a fix.


hi. nice job.!!

i see that your gpctoc.py file hides internal gpc code that can be seen with gpcdump.py file.
is there a way to use gpctoc.py and see all the code behind ? not just the user code but the code added by the compiler as well. thank you.


Sorry @moderna but you've deactivated "Issues", so I need to use this one here to write you. You're plugin does not work with Cordova iOS 4+. Could you please update and also check for WKWebView Support? Thanks!


nit: You can have just 1 newline between each of the methods.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11629](https://teamcity.mesosphere.io/viewLog.html?buildId=105661&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:03


I think it makes it easier to distinguish the changing parts and associate them with each other. Otherwise, one needs to consider all arguments.


Disregard this, mesa needs the xa flag


For command line argument parsing (in the `grasp_planning.cpp` example), consider looking at the Boost.Program_options module:

http://www.boost.org/doc/libs/1_55_0/doc/html/program_options.html

It may shorten your code quite a bit.


Fix for https://github.com/bbc/microbit-extras/issues/2693


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11627](https://teamcity.mesosphere.io/viewLog.html?buildId=105658&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:08:23


It makes it hard to debug, for example, you can't hover over scriptcontext value. Local macro are not aesthetically nice to me. I will leave it to you to decide though.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11628](https://teamcity.mesosphere.io/viewLog.html?buildId=105660&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:08:34


Ah yep, re-opened. I threw it in last minute but the stack frame depth was only correct for a single case. We'd have to increment an int as it goes through calls from `pkg.go -> logger.go -> entry.go`. I'm not sure it's really worth the trouble haha, I usually log in top-level code and delegate errors when possible, so the location quickly becomes less useful. If you have duplicate messages then maybe, but it's pretty easy to avoid that.


Collections.sort() is in-place, right? The buffer we pass in to this function is the buffer used in the percentile reservoir, which will be modified in another module. We want to make a copy so that the buffer stays sorted.

This doesn't make a big difference, but I liked the `sortedCopy()` function as a one-liner that might also be a tad more efficient. Happy to switch.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2083 of 2087 tests passed, 0 test warnings)

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


当俩点相同或者距离接近的时候,计算出的浮点数字比1.0大(计算机无法精确保存该值),所以该值参数给acos会导致最终结果是一个NAN ^_^


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 203](https://teamcity.labkey.org/viewLog.html?buildId=296866&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:06:33


So basically you reverted the ascii and some other minor stuff... Ok


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


done


ack


gr8 b8, m8


I prefer the original comment in the design doc, which describes the behavior a taint should be. And, better not to point out a taint is a part of NodeSpec here, since it will probably used on other resource types.


Esto hay que borrar


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 324](http://ci.infinispan.org/viewLog.html?buildId=34030&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for 3f30df33b14952e088f946f3268a40c1260597ea
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:09:34


Poner nombres a los EditText


I think the current DUP is correct, and the operation you (@condret) are refering to should have a different name. We can name it NUM. like this:
```
1,rax,=,rax,NUM,rax,1,+,+
```
the result here should be 1 + 1 aka 2


Fix-allocation-for-feneSolarDelaySeq (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1925 of 2051 tests passed, 0 test warnings)

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


Is it duplicated here? `The taint key to be applied to a node`

Dedicated node is just one use case of taint, if no things to describe here, you can just leave the comment as: `// Required.`


Poner nombre al boton


Oh, sorry for being super mad last night; I just get kind of frustrated when I have to re-fix bugs I've already fixed.


Well, I have an O(M*log(N)) solution working...


There's a way of making automatic hyperlinks come out in Doxygen so that you can link directly to another method or class... this page https://www.stack.nl/~dimitri/doxygen/manual/autolink.html might be useful


Just tested 1.4 and 1.7, it doesn't launch the app, perhaps I don't have steam installed?


What exactly have you done for get_oauth_token.php?
scopes -> scope
plural -> singular
Would you please explain me a bit?


Doesn't seem to compile because of "sdmcWriteSafe" for some reason... I'm using all the latest tools and libraries, if that matters.


``new HashSet<>()``


right! i forgot to delele the deprecated plugin.


... is an index


``new HashSet<>``


hello @edsilfer, what code requires you to make minSdkVersion = 18. what's stopping it from being 16 for example?


empd (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2083 of 2087 tests passed, 0 test warnings)

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


Hi Julian,

I'm not quite sure what you mean?

Thanks,

David

Julian Gardner wrote:
>
> Hmmm.
>
> If( 0)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LADSoft/OrangeC/commit/75112bb7c3aa16b1af10efac7ad4a02bc3b4da19#commitcomment-15443292>.
>


The code in the line 72 and 73 is working in the local system , but in the guest it isn't....tried to search but couldn't find why.... may be because the ubuntu version in the vagrant is old 12.0


also, valueIndices


storeIndices


@Stinkfist0  We can see if we can gradually make the scripts converge more in their functionality and options. The windows build script is definitely more versatile right now and more suitable for end-users. The *nix script is basically what I use for the official IfcOpenShell.org builds to which I applied your formatting colors.

I will check that macro again. IIRC it is definitely necessary for older version of OCC/OCE, but is probably obsolete now.


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 204](https://teamcity.labkey.org/viewLog.html?buildId=296940&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:06


include -y in this command


I deleted test.py and Hello.py since we obviously don't need those files.


hlkhh


This may seem like a lot of deletions, however these are all generated files in the bin/obj folder that should never be checked in. I updated the .gitignore so they aren't checked in again in the future.


<title>NATASHA WALL</title>
NATASHA WALL

1B Systems Design Engineering

n5wall@uwaterloo.ca

289.689.7536

Skills Summary


valueIndices


also, ``new HashSet<>()``


please include " -y " inn line 73


Correction: The actual call should be
```python
plant = models.Plant.for_slot(self.ID, raise_if_not_found=False)
if plant:
...
```


In the end, updated algorithm to simply
* stick with the Sieve of Eratosthenes, however, changed prime from list to dict
* then iterate through entire range from 0..highest value inputed
* check if prime[value] already exists, indicate that it's a prime, if so, add it to current_total[value]

Then at the end, just output current_total[value]

This gave O(N).  The previous algorithms actually was pretty close to O(N) (maybe O(2N)) as well due to memoize-ation...the problem was in those attempts, the inner operations used " in list", which I've realized is abdominably slow compared to a simple try catch.  i.e.

try:
    if primes[value]:
        curtotal = curtotal + value
    else:
        pass

is immeasureably faster than

if value in primes:
    curtotal = curtotal + value

when dealing with large sets!!!


... is an index


empd (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I'm curious why there isn't an attempt to `find_by_id` for a room. I'm trying to do something with a channel name using `lita-slack`, but the `room_object` for a `response` doesn't have the right name unless you `Lita::Room.find_by_id` with the room id. Sort of feels like Lita would provide this (or at least try). Does this make sense? Is this an opportunity to contribute? Thanks for you time @jimmycuadra!


Well, shit - now it's broken on py35...


Seemsgood.jpg

Maybe add a line to the tldr showing the use of `devroute` also?


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1506 of 1506 tests passed, 0 test warnings)

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


Yes using "if value in primes" would obviate the need for try..except.
*BUT*, using in is abdominably slow!!!
The highest scored suggestion from http://stackoverflow.com/questions/7571635/fastest-way-to-check-if-a-value-exist-in-a-list is flat our _wrong_, can't trust the interwebs


develop (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1505 of 1505 tests passed, 0 test warnings)

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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11630](https://teamcity.mesosphere.io/viewLog.html?buildId=105678&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:13


Changed


test phone call


Should we write new metadata only if createOrUpdate returns true?


Here's a more concrete example:

```ruby
[1] pry(#<Lita::Handlers::Lolwat>)> response.room
=> #<Lita::Room:0x007fd131242b60 @id="C024FA2V9", @metadata={}, @name="C024FA2V9">
```
Notice that if you "find" the room, you get better details:
```ruby
[2] pry(#<Lita::Handlers::Lolwat>)> Lita::Room.find_by_id(response.room.id)
=> #<Lita::Room:0x007fd130009570
 @id="C024FA2V9",
 @metadata={"name"=>"playground"},
 @name="playground">
```


Nah, that's what this bit of state is for:

https://github.com/davidbanham/devrouter/commit/d70678f49e76809ad0e65f0dbf2da7e704d6f6e2#diff-1ae34f5cb3c9c536753872f513ce97a1R57


Maybe it has something to do with confusing a byte-string with a unicode string?  I would expect the bindings to convert a python string into a std::string and a python byte-string into a char*.  The test looks right to me, ie, it's using a string literal, but maybe Windows is doing something whacky?

Do you get the same problem with Python 3?


@daniceexi,  thanks for spending time to review security doc.   For this one comment, I tried to add space between ``xdsh``,  ``xdcp``... but once after ``make html``, all space were ignored by compiler automatically.  Then the doc was put as such compact style.


커맨를 추가해 주세요.


Oh yeah. For some reason I thought that was reset when it reconnected, but that makes no sense at all.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (955 of 955 tests passed, 0 test warnings)

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


If colon is used as a separator for key and value, then we should re-use `MetadataDataset.KEYVALUE_SEPARATOR` here.


You're worng.


コメントテスト


develop (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1505 of 1505 tests passed, 0 test warnings)

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


Please put functionality into pure data functions/classes where possible, that greatly reduces the amount of mocking you need to do in testing.

I'd suggest a split something like this:

```
  if (!canApply) return;
  current_time = this.getCurrentDate();
  granular_timestamp = getEffectiveDate(current_time, granularity);
  new_sql = replaceCurrentTimestamp(old_sql, replacement_timestamp);
```

This keeps the dashboard/widget specific code separate, and also isolates the current date into a single location. That'll also make it easier to apply granularity to user-specified end dates in the future.


这个修改应该是写进 0.12.0 的 changelog 吧。。。毕竟是不兼容升级。


Por que a diferença de nomenclatura?

True, False ...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11631](https://teamcity.mesosphere.io/viewLog.html?buildId=105680&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:03:40


I included the 2 random fields to make the tests pass, if you remove those it will fail!


Do you know what this is for? I commented it out and everything still built locally on MSVC.


Same thing. Commented this out and everything built with MSVC. Any idea?


geography (macumber) - i386-Windows-7-VisualStudio-12: OK (2350 of 2385 tests passed, 0 test warnings)

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


I added some descriptions in my PR as well, but these are better.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11632](https://teamcity.mesosphere.io/viewLog.html?buildId=105681&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:58


@xDragonZ download 1.8, create an issue with what the log file says.


ready


Fixed in https://github.com/CulturePlex/Sylva/commit/0c2274835f0b3b8c4949868778a19744478f2708


Agreed, I think this is good enough for now; I'm happy with adding more rigor in future commits. I'll open an issue for that.


Gotcha. Thanks :)


I'll try again tomorrow.


this have to get changed


Closes #23


nice


Alright i'll do my best.


The latest [tagged release](https://github.com/jasonmit/ember-cli-moment-shim/releases) on GitHub is 0.6.2 - on NPM it is 0.7.3.


Work on issue #11


:+1: I've got a Debian virtual machine and I'm gonna try it on there too.


Perhaps add after line 105
```c
				else if (options.verbosity > VERB_DEFAULT)
					fprintf(stderr, "Section [%s] overridden by john-local.conf\n", name);
```


@foghina - any plans to support this in ViewPager as well?


falta un enter al final


Which Android API version is being used to compile by the buildsystem?


Is len() needed here? Why not just test the intersection for truth?


It converts String subclasses like ActiveSupport::SafeBuffer to a String: #281. No test accompanied that change or this would have been flagged as a regression.


Oh, does that mean [this blueprint](https://github.com/pk4media/ember-cli-moment-transform/blob/9022ef20c9bcb743512ce4fc4c11b90c4de025ee/blueprints/ember-cli-moment-transform/index.js) of the ember-cli-moment-transform addon is doing something it shouldn't do then?


Still not happy with you @popey456963 for this pathetic attempt at a commit message. I'd have thought you'd learnt by now what happens when someone commits with a message like that. They get their code tar balled and then pushed back to the code base.
:stuck_out_tongue_winking_eye:


Since you are already using LumiLists, you might be able to combine a lot of the logic of what you're doing here already.


reduce_redimensions (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1574 of 2088 tests passed, 0 test warnings)

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


Yes

--
sent from a rotary phone, pardon my brevity
On Jan 15, 2016 10:42 AM, "Ingy döt Net" <notifications@github.com> wrote:

> Is File::Temp core?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frioux/io-all-pm/commit/a6360aa178e529745903b84f02d564af9d8ab36b#commitcomment-15481419>
> .
>


@leoniv упс - спасибо

пользуясь случаем скажу
* 2 года назад я начинал именно с wire протокола - все проклял: контекст нужно было хранить между вызовами, поэтому мы перешли на полноценный 1С, только реализуем gherkin3
* скрипты управления мы наращиваем в рамках https://github.com/EvilBeaver/oscript-library/tree/develop/src/v8runner, пробовали на ruby и python - это получается слобоподдерживаемо

это я про ваши репозитории. Делюсь так сказать нашими "граблями"


Should also fix #1608


If that's the case then that's a bug, it's supposed to create the directory at runtime if it doesn't exist. At least it should be easy to reproduce. I'll write a fix.


each recipe has it's own config with the respective configuration:
https://github.com/libretro/libretro-super/blob/master/recipes/android/cores-android-armv7-ndk.conf

In this case as you can tell, r10e


not that I care, but what motivates this change per se?


The whole sentence was a train-wreck. Re-written.


Why Double? Not double?


not to be confused with Jenkins 2.0 :-)


Is this subst variable used anywhere?  If not, it should be dropped.


Q.denodeify?


i haven't added
```
cb.setButtonDrawable(getResources().getDrawable(R.drawable.check_box_selector));
cb.setPadding(32, 16, 4, 16);
```
in XML,as we are creating checkboxes dynamically ,hence we need to apply these to each dynamically created objects.


nice style change faggot


Although, having said that... be867c3


fs-extra-promise?


Really? `(.:!)` was for that case; modifying `(.:?)` will break much more code. it already made people use `aeson-compat` as there is no operator capturing old behaviour i.e. there is no way matching `Null -> Nothing`


no


Ok, tagging and uploading afterwards...


reduce_redimensions (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (955 of 956 tests passed, 0 test warnings)

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


Typo: "msg_msg" should be "max_msg".

Could you change this to something like "Error: Message exceeds provider's maximum message size.  This limitation will be corrected in a future release."


This is a full LaTeX document.  Not the assignement.


Missing changes to man page and bash completions.


@JanSchulz Ok, cool. No hurry.


return fs.writeFileAsync?


You're right, it's there. My bad. For some reason my repo browser wasn't showing it, but it fetches just fine.


Thank you for the quick turnaround. I've submitted a PR.


What subst variable?  `RPATH_LDFLAGS`?  It's used about 5 inches below this dialog box...


I mean the sysroot that the NDK is using as target API. I'm guessing it is targeting an android Android API version higher than v19?

Apparently they removed timeb.h in API's higher than 19. I'll have a look at the PUAE source...


A brief comment that `MessagesProcessed()` is thread-safe, while `Messages()` is not would be useful. Not blocking me from merging this PR :)


Read the commit message.


Hi,

It's better but there's still a bug if the dashboard windows is over the toolbar, for example if the 'x' overlaps the options tools, closing the dashboard will open the option dialog box.

Regards
Didier


Could be, I compiled that toolchain a loooong time ago and I think I should redo based on a lower common denominator.


Makes sense. Changed it to `Set`.


Done.


as a reminder, newer clojures have `some?` which would allow `(some? #(= element %) seq)`


Indeed!


where the fuck did my commit message go ._.


gen_gaussian can be replaced with scipy.stats.norm.


Ненавижу MonoDevelop


Why does this commit removes `bcm2708-rpi-zero.dtb`?
It breaks the usb gadget:
https://github.com/raspberrypi/linux/pull/1239#issuecomment-170220201


pulling down final


Will do.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11698](https://teamcity.mesosphere.io/viewLog.html?buildId=105996&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:03:45


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11699](https://teamcity.mesosphere.io/viewLog.html?buildId=105997&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:04:00


@Nadahar @valib it's close already, I would remove the line in the middle though since that goes without saying, and we can bold the "warning". I have attempted to establish a convention in the tooltips of having the sections "Default" and "Notes", but we can add a "warning" to the convention too so it looks like this:

`Default: Auto detect
Notes: You can find the list of supported character encodings at the ICU4J page.
Warning: If you set the character encoding manually, any subtitles that use other character encodings will not be displayed correctly.`

I like using `<br>` because by default, Java makes the tooltips span the width of the screen before it breaks, whereas I think it looks better to have it break at sentences or sections.


@RubenVerborgh enig idee waarom dit niet wil centreren?


Will probably need to do this with Octokat for private data repositories


Actually stride and start is of no use as we want to track each frame. I want to remove these parameters. Or the default can be set as 1 for working on all frames.


Hey Elod,

I'm still learning the ropes with the Github. I reverted this and added more comments to the new commit. The only thing I really meant to do was remove the clocking constraint in the xdc files because it was overriding my higher level constraints.


BTW, this code isn't in the bungeecord codebase, it was a joke by Lax.
The handshake procedure is [well defined](http://wiki.vg/Protocol#Login).
Don't worry I was fooled by it too :)


Can you confirm that this fix a current test? It seem ok to me.


The Error type will already add the colon after the description.


It seem I prefer much longer explanation then you, but as it would fail, it is fine.


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 211](https://teamcity.labkey.org/viewLog.html?buildId=297127&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:24


No I only use fitEllipse. minEnclosingCircle is used in line 71 just to get rid of small noises and get the biggest contour which corresponds to the pupil.


Use `flask.views.http_method_funcs`.


Damn, I'm at a conference right now but I'll take a look when I get some time, or cc @pikajude @gridaphobe if they have some spare cycles right now.


I have now changed it as percentage of the patch region.


RIP Bowie!


After this, I'm getting on PhantomJS,
AttributeError: 'NoneType' object has no attribute 'close'
How can I fix?


TeamCity pull requests :: netty [Build 1042](https://garage.netty.io/teamcity/viewLog.html?buildId=4794&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6372, ignored: 23 Build time: 01:23:03


Ah, righto.


Why not do this regardless of type (e.g. around L626)? Presumably language is useful for captions as well as just audio. It would be fairly random for video, but if it's there you may as well include it.


Ya that is a better way of implementing. Thanks.


`Type.values()` returns an array of constants defined in `Type` enum. I want to construct a list of `PrestoPrivilege` instances as opposed to a list of enum constants. I don't think I can do that without using a `Builder`. BTW, I have changed this code further to rename `PrestoPrivilege` class to `PrivilegeNode` class. Also, instead of defining an enum, the new class instead has a local variable of type `Privilege`. Will push a new commit where you can see the changes.


Anyways we are using uniformly picked points so we wont be using this approach now.


Build a map of constructors before methods are processed.


Suggest:
        for (auto const& c : connections)
            c->close(code,reason);


ok, brown paper bag: forget to increment the version string and had to push the tag twice...... I think I should add a release.md file... sight...

sdist currently uploading, binary builds in progress...


Done.


It does not give the same result as the python code otherwise.


In some circumstances


TeamCity pull requests :: netty [Build 1043](https://garage.netty.io/teamcity/viewLog.html?buildId=4796&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6372, ignored: 23 Build time: 00:41:58


@JanSchulz Yeah. No worries. That's an excellent idea!


The function tcflush **discards** all characters in the specified queues, however we want to make sure everything is written (flushed) in these two functions you changed.
To quote the glibc documentation:
> It is unfortunate that this function is named tcflush, because the
> term “flush” is normally used for quite another operation
> —waiting until all output is transmitted—and using it for discarding
> input or output would be confusing.


Yup, you want `addAddonToProject` https://github.com/stefanpenner/ember-moment/blob/master/blueprints/ember-moment/index.js#L11


Looks good.


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


@Chocobo1 Did you mean to leave this as **(l, r)** rather than **(left, right)**?


Thats a good point. Done.


Hm, shouldn't we just ignore those, and not treat them as invalid?


Ya agreed. Will change that.


Oh, that is no good.  Switched that back.


I find it weird to call `.type` twice and assign it to two different variables given `type()` does not make a copy after all.


LGTM.


ur an fagit


This makes my life super easy. I am trying to build my first SMD project ever with your open designs. Thank you very much for having added the BOMs!


Oh, I thought `AC_SUBST` puts it in `config.h`, at which point it is potentially visible to programs.


Proposing we roll back this change - would like to see devoid cards inline with their similar casting cost cards and not at the top. http://strawpoll.me/6546366/r


Ya that is why I have 'tt' variable in the code. Sorry for the naming convention :P. If you put tt=30 then it save one image per 30 frames. Or we can give a very high value if we don't want to save any image altogether.


This commit requires a memcached flush in addition to a regenerate_and_update.sh run.