I have an "unfinished" package listed here and I think it is fine! I think you do a great work listing them. And I agree this list can motivate the community to contribute to the repos.

Particularly, I use to use git tepositories others than github when I dont want my code to be visible. Bitbucket is the best one I have tried so far.

Best,

Charles

________________________________________
From: SVAKSHA [notifications@github.com]
Sent: Monday, February 01, 2016 5:43 PM
To: svaksha/Julia.jl
Subject: Re: [Julia.jl] @iamed2 : WikiSort.jl (a884fe9)

On Mon, Feb 1, 2016 at 3:35 PM, Tom Breloff <notifications@github.com> wrote:
>
> Sorry to hijack this thread, but I wonder if it would be reasonable to add some sort of metadata to each listing which includes whether a package falls into various categories:
>
> Production-ready
> Tested
> Actively maintained
> Abandoned
> Experimental
> A "throw-away toy repo"
>

This would be a nice to have list but who are we to judge? Unless the
package owner/creator tags their repos, it is just guesswork and
probably premature to assume what status the package is in. I list
some packages that were developed in ver0.01 only because its freely
available and someone else may want to continue with the work if it
aligns with their research or work.

> It may be reasonable to let the community compile this data (possibly a google docs form or sheet?) by asking on julia-users for assistance in filling in the blanks for repos that they have personal experience with.
>

Perhaps we could try an experiment - instead of googledoc ask the
community to add a {tag} near the package name and submit a PR?

Not sure atm, but the more I think, I realize that owners/creators or
core-dev maintainers are the best persons to tag the status of their
package as only they know what works and what does'nt - especially
true for lesser used packages. That is ofcourse just my personal
opinion. fee free to disagree. IMO, the danger with crowd-sourcing is
: What if a community member decides 'foobar' package is unstable as
it didnt work for them or 'not maintained' if they refused to add
requested features?

Community feedback is a good thing if they file bugs and so on, but
lets not forget that there is a risk of some package owners being
offended with the community tagging the status of their package
negatively. If you know what I mean? :-) I usually keep METADATA as
the final authority for stable or actively maintained packages but
even that changes frequently. I think the easiest way is to have
METADATA find a way to effectively tag packages, else it is hard to
know the original author intends - example, some packages on
metadata.jl fail for the latest Julia version, so how do we label
those?

> I find that this repo is such an enormous help in searching out new functionality, but that it can take a long time to evaluate which packages are worth looking at. Also I too have a couple repos on here that are in various states of usefulness, and it can feel strange to have them on here without context.
>

Thank you for the kind words. I understand what you mean. I guess that
comes with the new language and its growing pains. I would love to
find a solution to the problem too but I dont have the perfect answer.
I am more worried that if we start being aggressive about tagging
packages, then some folks will not want to be listed at all as their
package is still 'unfinished'. That would be sad, dont you think?. I
want to encourage folks not discourage them from showing off their
unfinished julia code on github.

/zero paise worth.
SVAKSHA ॥ http://about.me/svaksha ॥

—
Reply to this email directly or view it on GitHub<https://github.com/svaksha/Julia.jl/commit/a884fe9e921d57b87d85e970c2f57b8f21025641#commitcomment-15804236>.


Will this class (and others situated in Queries module) be recognized as part of a private API because of presence of `# @api private` in other files?


proposed rewording:
If the data file is in hd5 format, any calculated quality result will be added to file with corresponding tag.
If the data file format supports only raw data, the quality results will be stored in a separate file with a name corresponding to the data file.
The experiment user will receive visual feedback on the data quality.


> Since a witness will gain income but a committee member will not?

indeed


Right, it will do the tasks 1.2 and 1.3 when the data is available.
Will do the task 1.1 when the experiment is about to start.


This point here is rather to make sure that the experiment initial setup is complete. But, the setup data may be corrupted (incorrect entry, etc.).
Propose to reword 1.1:
to verify experiment setup (i.e. all required setup data is entered and the values are acceptable)


Agreed; I'll do the changes.


Which games does this help with? Asking out of interest.


@CaptainElliott Because it doesn't require any information from the LoadedMap object, nor is it long enough to justify being put on a separate line (like the message var).


"hastimer" and "hastimerschedule" are not used anywhere else, so it should be save to do this


I'm not entirely sure about that one. If it were the `DefPath` we were querying here, I'd say this should not be a read, since any change to it would make it a "new" item to the tracking system anyway. In case of the path, which is almost the same but not quite, I have to think about it.


ok - I merged the above code onto my curator branch


Why is a POST used to simply read details?
I would have thought as REST-like API would make more sense?


@mattheworiordan After reading your comments I believe I can write a much shorter test that checks only for what we want and does not involve explicit timeouts. Will get back to you shortly.


111411352_ATDeprecateFields (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2093 of 2093 tests passed, 0 test warnings)

 * 549 tests had: AUD diffs.

![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)


you can upstream anything that is not related to visual studio projects itself (simply do a PR). The visual studio support was held in my repo till now because there is no upstream maintainer. Please at least provide a link to one of your repos inside the built windows package - so we know which one has the most current Visual Studio support in it.


i.e. GET is more typically used for reading Data


I'm not sure this really does what you're after.
How will you tell whether a user has the CAP Publisher role just from a list of IDs?
(You most certainly shouldn't hardcode that any specific ID is a specific role!)


@mk01
IMO not longer necessary - amazingly, we worked at the same problem :smile:


Can you rename this to something more descriptive than virtual_attribute?


steps to technology


agreed with moving this to the base security class. maybe we need a 'default instance' that applies to the most common case (quote=USD, multiplier=1, pip size = 0.01 (one penny?) For example, with the 'wildcard' support, we'll need to be able to return a `SymbolProperties` instance matching a key say 'equity/usa/*[symbol]' - These symbol properties will certainly be needed for options support which is next my list, so we may need to move this to the base now. It looks like the commits are separated enough, if need be we can extract it as it's own pull request. Let me know what you think is the best course of action here.


Ah, from the UUID...ok...but why do you need the ID then?


> -More consistency on the price_per_kb fee, and more explanation/thoughts on the different values used for these kind of fee.

There is no consistency possible. Operations that allow you to put arbitrary data on the chain have a higher per_kbyte fee then those that do not.

> -Current BTS value for every fee applied.

Could do that for the current price, sure. I would also like to add another paragraph that compares the old and new fee structure in USD and percentage change.

> Avoid fractions: although a $1 fee makes some sense, a same fee of 251.12321 BTS makes no sense when showing on the fee schedule.

so either you get convenience in USD amounts and not in BTS amounts, or vise versa. Since most people do understand USD denoted values and many businesses have approach me about this "issue", I though targeting a USD value makes more sense. Also, except for the crypto crowd. The goal is to have end users that pay their fee in USD (or other currencies, but not BTS).


So, are the other images (not comment avatars) also pixel measurements? I would think that these should be percentages so that we have a design that responds to the size of the device screen. Thoughts?


This seem pretty cumbersome to me.
Why not just do a lookup where you login as the user & then read current.auth.user from that?
I wouldn't send the username/password across...


This test was only built when `FMT_PEDANTIC` was used. I don't see the point to check with default flags (i.e., less noisy and picky) when stricter flags have been requested explicitly.

I generally advise to only define the absolutely needed flags in the build system and delegate the responsibility to specify the desired (i.e., additional or different) flags to the user. If the user is interested in test results for the default flags then he should build and test the library in this configuration.

The same is already true if any different combination of flags is desired. One simply defines them as `CMAKE_CXX_FLAGS` and builds the default configuration (`FMT_PEDANTIC=Off`).

However, it is your project and if you insist on keeping the test then I will obviously respect that decision.


indentation


s/boards/toolchain/ ?

Any reason why not to pass that flag to other toolchains as well?


I'll look over this later I've been extremely busy the last 2 weeks


FYI, @mpost Finished all the left translation for en-zhcn at https://crowdin.com/translate/plusonelabs-calendar-widget/20/en-zhcn please help check and hope release the new version soon.


I would prefer if we null check `SymbolProperties` in the constructor to avoid this. Also, looking at this statement makes me think it's possible for `SymbolProperties.QuoteCurrency != QuoteCurrency.Symbol` - I fear this could be a dangerous inconsistent state for the application. Maybe we can add a check in the constructor to enforce the `SymbolProperties` and `QuoteCurrency` are aligned.


@isidorn fyi there is functions in node to join 2 paths: path.join()


111411352_ATDeprecateFields (Nigusse) - x86_64-MacOS-10.9-clang: OK (2055 of 2055 tests passed, 0 test warnings)

 * 546 tests had: AUD diffs.

![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)


Remember to always add your ``!DOCTYPE`` declaration (further info [here](http://www.w3schools.com/tags/tag_doctype.asp)).


BTW, `select()` can be called as `select(li)` and this is the only reason I need to add this method to find li index.

The is nothing in docs about this optional param to `select()`. I wonder what's the original use case for this public API `select(li)` was? Or maybe it is more like a side-effect of using this method internally?


I mentioned before maybe we just need a 'default' symbol properties to avoid embedding these default values into the CFD class


looks great. Have you thought to contribute as a PR?


Are you sure they are f4m links? This page seems to say they are m3u8s.
http://www.bbc.co.uk/blogs/internet/entries/d031b300-721a-4df5-8506-723303f2887e?comments_page=12&initial_page_size=20&filter=none&sortBy=Created&sortOrder=Descending

like post #92:
http://a.files.bbci.co.uk/media/live/manifesto/audio/simulcast/hls/uk/sbr_high/ak/bbc_radio_one.m3u8


Hm is there reason why data_error ignored?
I want to catch it (to identify this is plain text or deflated data). Before this commit i do:

```crystal
class Zlib::MaybeInflate
  def read(slice : Slice(UInt8))
    if @crashed
      if @remain.not_nil! > 0
        size = {slice.size, @remain.not_nil!}.min
        slice.copy_from(@pointer.not_nil!, size)
        @remain = @remain.not_nil! - size
        @pointer = @pointer.not_nil! + size
        size
      else
        @input.read(slice)
      end
    else
      previous_def(slice)
    end
  rescue Zlib::Error
    @crashed = true
    @remain = @stream.avail_in + @stream.total_in
    @pointer = @buf.to_unsafe
    size = {slice.size, @remain.not_nil!}.min
    slice.copy_from(@pointer.not_nil!, size)
    @remain = @remain.not_nil! - size
    @pointer = @pointer.not_nil! + size
    size
  end
end
```

and if it just plain text, i get plain text from IO.


Maybe a stupid question, but can we reference to (ss-3)?
The comment at the beginning of the iterative deepening loop implies only (ss-2) to (ss+2) is possible.


Thanks for your input. We will have a look.


just a heads up, this may cause issues if `config.DataTimeZone != config.ExchangeTimeZone`


Personally I prefer much smaller data sent via JSON to speed things up & reduce charges/congestion.
So:
* Why include email when you already know this?
* Do you really need the first_name/last_name?
* Use keys which are single letters rather than full words (yes, slightly harder to debug, but to me this is well worthwhile when transmitting over the wire)


"UNIQUE NOT NULL" when _authenticator_ is "NOT NULL DEFAULT 'Database'" elsewhere?


Why was this needed? Seems odd that we would have a public circular object graph:
`Security.Holding` and `SecurityHolding.Security`
Can this be avoided?


Shouldn't this just say

```go
creds = []credentials.Provider{&credentials.StaticProvider{Value: v}}
```

As we don't want it falling back to anything else if those credentials didn't work?


111411352_ATDeprecateFields (mjwitte) - 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)


Links `@AGaliuzov` and `@anosach-luxoft` should be here?


This worked then? :)


Even though those are pixel measurements, those are not static sized.
So let's assume we write CSS that displays specfic image as 50x50 in iphone and that displays it as 100x100 in tablet. Then 50x50 image will come from imgix if device is iphone and 100x100 will be loaded from imgix if device is tablet.
In other words, size of image is calculated automatically so it depends on how it's shown when imgix is not used.


So, methods are not tracked separately from their container?


The delegate isn't hooked up yet - that happens 2 lines below.


-------------------------------------------------------WRONG COMMIT-------------------------------------------------------


Now that you mention it, there is one instance that I pass repositories to ```dedup_array```. But most of the existing code already assumes that repos don't contain spaces and I don't think this edge case will ever come up in practice.


@XhmikosR Why not test on `0.12`?


@michaelwoerister hmm, good point. You probably don't need a read here.


Still don't see why this would be in Auth.
Converting person_id to pe_id is PR not Auth


The title "Isotime" seems to be incorrect. Also, do you think you could create a screenshot or something similar on your repo you could then link to from here? It would make your example more attractive, IMO.


Ouch, I did it in a hurry copying some code from C ([here](http://www.zlib.net/zlib_how.html)) and, of course, didn't pay attention to that. They should be separated by commas. Good catch! I'll fix it, please let me know later if I did it right.


yes it should be documented i the package but I also think we should move this repo to the organization to have it more visible. I cloned to my personal because I did a quick and dirty test to see if it would build.


@michaelwoerister

> So, methods are not tracked separately from their container?

Not right now, no. I'm trying to keep things at the item resolution for simplicity. But I guess sooner or later we will want to drill a bit further down. Though I'm not sure how much difference it'll make in practice, given that all things in the impl are always in the same module etc.


Indeed.


Looks like we may have lost the `SecurityKey` change here


Intentional - these are not methods of the WMFExploreSectionController protocol - these are methods of the WMFBaseExploreSectionController class that require implementation by subclasses.


These seem to be location_ids...so better to make that explicit
- I would even have that in the arg names (less need to read the details then as obvious)


@bpasero thanks - pushed! Now the code is much nicer without string magic.


@djkirby Yep, I need a image browser , and I will add propTypes validation, Thank you~


@ToDo: Method which uses Shapely


If we call the args location_id1 & location_id2, I'd be tempted to call the rows location1 & location2


Why are you reading level/parent?
Seem unused & I can think of no reason to use them for this


nice!


Try to avoid using IDs if the element is not unique in the page (in this specific case seems like this selector could be reused).
Also, don't use inline styles, since it'll be difficult to track styles and even more important is the fact that inline styling is difficult to override (due to specificity's mechanics). You should define all required styling attributes in your CSS file.
Here are some links that could be useful:
* [CSS specificity](https://developer.mozilla.org/en-US/docs/Web/CSS/Specificity) (it has more useful links at the bottom)
* [Writing efficient CSS](https://developer.mozilla.org/en-US/docs/Web/Guide/CSS/Writing_efficient_CSS)


aclocal: couldn't open directory `m4': No such file or directory

this is on osx ... where is this dir supposed to be?


This ultimately called out to [`nanosleep`](http://linux.die.net/man/2/nanosleep), the Emscripten implementation of which is just a busy `while` loop (see also https://github.com/kripken/emscripten/issues/1256). We'll give that time back to the browser instead.


Awesome work @modocache!


I documented the path to this commit yesterday so if you change this we need to re-upload


Right, missing an if check on this line.


Even if we don't have any extra time to spare, we need to give *some* time back to the browser, or else the user's UI will become unresponsive.


I added a hash function in IRGenOptions. This makes it more obvious to add a new option there.


Although using ``onclick`` is not incorrect, it's a bad practice, since we want to keep our layers separated (markup, styling and behavior). Use ``jQuery``'s events handlers instead.


Thanks for the thoughtful comments @svaksha.  I see your points and agree.  (I especially agree that it would be nice if METADATA had some of this curation)  Also thanks for taking the time to compile this repo!

I suppose it does make sense for only the package authors to be allowed to update a "status".  I'm happy to be the guinea pig and add a status for my packages with a PR.  I wonder... what do you think would be a useful format?

With an example, I'm sure the rest of the community would be happy to contribute.  We could just start an umbrella issue here and let package authors comment with the desired status, then compile into a PR when time allows.


Sure, send a patch to the Tegra mailing list for review. Best CC myself and the author of the RSA patches (see git log).


does it make sense to name it `TimerHasConflict`, `TimerIsActive`, `TimerHasError`?


@shama: isn't 0.12 superseded by 4 which is LTS? 0.10 makes sense to test as the minimum supported version, 4 as the LTS and 5 as the latest one.


:+1: ok, let me know!

these libs (were as described in the commit message) from the nexus stock image.


thought more about this, maybe it's best we keep it as simple as possible for users to edit. csv's are certainly easier to add a new line to, so no action here, just thinking out loud


This is a convention we have to separate standard and 3rd partly libraries in the import


Umm. See CODING_STYLE: functions shall either log about all errors or none (in which case the caller has to take care of doing so). This function should probably log about all errors, hence you need a `return log_oom()` here.


neds to log the error.


If we're doing this then I would check path not just parent
i.e. match if we're looking at lower levels than L1...

Do you see this as an important usecase?
I guess for the CAP usecase it could be, since one of the locations will be an Lx in the subscription, whilst the other will most likely be a Polygon.

ok, so let's generalise it:
if location1.level or location2.level:
    if location_id2 in location1.path.split("/") or location_id1 in location2.path.split("/"):
        match
    else:
        query


Could you please verify that everything works even when the ReadyToRun image is renamed from .ni.dll to .dll (ie overwrites the original IL image)?


LGTM


Why did you make a fork when you have write access anyway? xD


See discussion on the pull request.

On Mon, Feb 1, 2016 at 10:59 AM, Jonathan Wiedemann <
notifications@github.com> wrote:

> I have the same message, and I get segfault when I start to draw anything.
> Compil of commit just before run ok.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FreeCAD/FreeCAD/commit/8219102dbef87ece2100ea408542831a5b27242a#commitcomment-15802755>
> .
>


Disagree.. this is an element that is meant to encourage adoption of LTM. If the LTM is in the neighbourhood of $97 and the cost of the Asset can be 80% less if they become a LTM, then if they want to create one or even a few, they maybe motivated to consider a LTM prior to creating an Asset.

I say we raise this to $57.


needs to log.

(also, afair udev_device_new_from_devnum() will also generate an error if the device does not exist currently, right? if so, this should really return errno (where the call leaves its error afaik), instead of always -ENOMEM).


Right, if XSI strerror_r fails, the returned pointer will most likely be invalid (~0) and this will read to a crash. However, I didn't create this problem. I am to blame for suppressing a valid warning though.

I'll re-fix this.


The name `pl061_gpio_device_t` implies this is a typedef when it's a struct. Remove the _t. Also, the name implies this represents a gpio device when in fact it represents all the pl061 devices. Perhaps `pl061_devices?

Actually, using a struct doesn't add much except some name scoping; I would just use global vars (but that's only personal preference).


needs log_oom()


I really sorry for this, but I just noticed one more anomaly - you're using `{}` for blocks. Can you make a final change and make it `do - end`?

Don't forget to rebase later.


log error


$this->logger is set when class gets instantiated, and there's no setter for $this->logger, but yea, checking for logger first is probably safer.


Forgot the namespace ( `xmlns:ti="http://chs.harvard.edu/xmlns/cts"` )


Also, facelock libs cause settings to crash too. But it's not libs problem I think, it's some kind of permission problem. Needs to be tested too.


log the error, please.


No, no, no!
Just because the BBOXes intersect does NOT mean that the locations do.
Only the other way around.
i.e. if the BBOXes do NOT intersect then they do NOT intersect


`__cts__.xml` file in work directory should start at the ti:work node, as explained here http://capitains.github.io/pages/guidelines.html


I would put this check AFTER the simple level/path & BBOX checks...as these work independently of the spatial DB


And... all the tests still pass that used to pass! (Though a few of the tests seem to take a long time.) Any chance for tests of disintegration?

And does "queue" mean FIFO?


That is actually pretty clever. Can we maybe have an error check and proper error message here?


There's no need for the then.  Also this should be platform_family and there should be an else since as it stands this will break the functionality on non-RHEL platforms.


ah no - that error is caused due to the makefile.am change in this commit ... mhhh


please do this for both block and char devices... there's no reason to do this only for block devices, is there?


Which you even have implemented!


So we use Shapely if we have PostGIS & nothing if not!?
Surelty we use Shapely if we do NOT have PostGIS & we use PostGIS if we can...


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


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

Offending files: libraries/Config.php


It's necessary because if AUTHORIZENET_LOG_FILE is set to '', file_put_contents() will generate a warning.


@ranjitmindteck  please mind on what you are pushing


btw, i'd really write this as `if (asprintf(…) < 0) return log_oom()` rather than first cast this to void and then check the ptr...


#define DEG_TO_RAD      ((float)M_PI / 180.0f)
 #define RAD_TO_DEG      (180.0f / (float)M_PI)
As we are using float no?


Hi, #IFDEF is super easy. You use #DEFINE or the -D compiler flag to set and #IFDEF ("if defined") to check for it. Look for example at DWMs code: http://git.suckless.org/dwm/tree/dwm.c#n39 in line 39 is the #IFDEF. http://git.suckless.org/dwm/tree/config.mk#n15 has the -D flag. Documentation for it is here: http://gcc.gnu.org/onlinedocs/cpp/Ifdef.html


@jterps08 it is in separate branch and [this PR](https://github.com/PowerlineApp/powerline-server/pull/32) is merged already


Hm, wir haben aber schon länger noch folgende ids.
243 Dargul
244 Gul


Oops! :)


You are right this is a bug. Bad copy paste


I would rather embed the key in a vars file, to use Ansible vault. This way, you could also use a lookup to read a file if you really want to have it in a file.


I think a better description is:
"Whether CAP alerts have workflow for ACKnowledgement & Checklists"
i.e. Focus on the user workflow here not the implementation?

Imagine this text being a Help field for the setting in a Web-based configuration tool...


the only reason why we cast the result of asprintf() to void, and checked `t` rather than the return value of the function was to be able to do all OOM checks in this if orgy the same way. But if you move the OOM check into the inner block, then you might really better check the return value of asprintf() directly...


Added unity ads and it works fine now. Thanks.


yeah -- I just haven't had time to go fix it yet - probably later this week


I ran into the same linker problems - linking librbd_internal.la before librbd_api.la fixed them, and there's no need for $(LIBRBD) or $(LIBCOMMON) below


Should delete this line


Why not spell-out SOP in the help text?
Standard Operating Procedure


This is a pretty ugly loop to search for the index into the array; it can be simply calculated as `gpio_offset / GPIOS_PER_PL061`. Also, as I said above, I think it's better to use a direct index into the array in the interface.


What does this do? :v


well, choose: either make the called function log about all errors, are the calling function. You did a mix now. I'd suggest to make the called function log about all errors (as discussed above), and hence remove the log message here. Really, we should always see exactly one error message for each error, not two.


I think it's better if we make this optional. A commandline option to choose which file to create (or both). Starter should be able to get devs started with docker even if they don't want to run their apps on Cloud 66.


book.rb looks better now .... library still facing issues with lend method


Do the instructions really come at the top-level of the checklist?
I would imagine them more at the level of each individual entry...?


Perhaps verify that the built table is of the type expected?  Ie. assert tag == table.table.tableTag?


I think if the arguments to this function are validated properly with asserts, you can make this function void.


label should match comment - I think comment is right here: plural


Nothing xD Just adds a line


yeah - it probably should be - thanks for pointing that out


actually, both sd_journal_add_match() invocations need to be check for errors. I understand this was already broken before, but this should really be fixed if you touch the code anyway...


Really?
I would have thought these would always be Lx: no need for a Map or Polygons here


For info Glyn - phpmqtt_input.php is still using 'nodes' on master & stable.


Can assert this.


I would therefore also make the fieldname the same too


I think I agree with everything you say here....

But, I do think it's worth keeping the hn model in the table, just to
show what's going on there (though with some mention that we don't think
it's a very good model).

Definitely worth adding something about model selection, particularly
due to the small delta-AIC.

If you add this as an issue I can take a shot at it, otherwise feel free
to edit it yourself :)

On 01/02/2016 09:00, Laura Marshall wrote:
> @dill <https://github.com/dill>
>
> We say that the hn model should be discarded based on fit but it is then
> included in the model comparison table... is this confusing? There is
> also no real guidance here on model selection (we do not explicitly say
> model x shoud be selected)... although the AIC is smallest for
> amakihi.hr.obs.mas, some would argue that a difference of less than 2
> does not really justify the inclusion of the additional covariate. Are
> we simply saying minimum AIC wins? Are there any other considerations?
> Should we point them at further discussion on the topic?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dill/distance-package-paper/commit/46ee9807641a46bb6e538089e1d7200a5a49c557#commitcomment-15799577>.
>


It's not best practice to use `@variables` in tests. Instance variables are not guaranteed to be cleared between tests, which polutes your tests. This is the purpose of `let` (plus it's easier to read and refactor). I recommend that you always use `let` over `@instance` in all cases possible! Remember http://www.betterspecs.org


Also better to put globals at the top of the file


cheee


Wow how helpful!
The helptext is *exactly* the same as the label!
Sorry, but this is terrible UX: either add something useful or remove the helptext completely...


111411352_ATDeprecateFields (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (961 of 961 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.97%25-green.svg)


Looks like it is when I generate the docs


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)
- [ ] Program is different than [Task 4](task-4) (1 pt)

## Task 6

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (3 pts)


Same as above


Can you change these 4 lines to the following to allow Ruby to better optimize this:

mocks = mock_headers.map {|mock| File.basename(mock)}


You scared me! For a second, I thought it was THAT [Trump](https://en.wikipedia.org/wiki/Donald_Trump) :P


Seems cumbersome to me.
How about just 'Checklist Template' not 'Event Checklist Template'?


This is a regression.
Following vars cannot be used after the ;


Seems cumbersome to me.
How about just 'Checklist' not 'Event Checklist'?


Acknowledge


I think this would be generally useful, BUT I worry about the amount of
space it might take to explain what mads is in the paper...

On 01/02/2016 11:03, Laura Marshall wrote:
> @dill <https://github.com/dill> @erex <https://github.com/erex>
> @lenthomas <https://github.com/lenthomas>
> I feel like it would be really nice to get mads working with Distance so
> we can say they interface too, how nice would it be? I reckon 1 - 2 days
> work... worth it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dill/distance-package-paper/commit/521e2780be8490a5124a1504537be5067021b813#commitcomment-15802934>.
>


Thanks!


Is there are place where we can ask OpenTK related questions, like:
How can we have AnyCPU build that provides its own 32-bit and 64-bit versions of the OpenAL dll and not require that the OS has them or
How do we have gamepad vibration support without SDL2 (or even, how do we port the code over?)


I think this could be a compile time assert. At least an assert instead of a runtime check anyway.


@XhmikosR 0.12 and 4 are completely different versions and more importantly completely different versions of V8. I want to test on all the versions we plan on supporting. 0.12 probably doesn't have as many users as 0.10 or 4 but a significant enough to test to at least know if we broke something in that version. Much easier to see the commit where it broke rather than chasing down a user bug report.


assert


Do you 'ACK the checklist'?
Doesn't seem right to me.

I see 2 separate steps:
* ACK the Alert
* Complete the Checklist

I'm not certain, but I can imagine that some deployments might wish to have ACK but not use checklists.
(I can also imagine people using Checklists completely outside of Alerts, but I've seen nothing to prevent that yet)


It shouldn't be necessary; it's a temporary workaround until we can fully free comprehensions from dependence on type inference.


Do we know what's going on here? Is there something that the AppVeyor folks could do on their end to improve reliability?


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


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


That's the HLS source, see the part in the URL after "simulcast". HLS uses m3u8 playlists.

iPlayer on computers uses HDS with f4m-playlists. No idea, if Kodi can cope with them at all.


redundant braces


Is it possible to add this?

https://github.com/anarkia1976/AK-Angler/commit/1b9ecef4a47ea0e9fae94ee43f0e16f8807e2536

Thanks!!


TeamCity pull requests :: netty [Build 1166](https://garage.netty.io/teamcity/viewLog.html?buildId=5295&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:32


Maybe also use `empty()`?


Das war nicht "tröpfchenweise", sondern erstmal für **alle** core Plugins. Aber bislang hat sich auch keiner daran gestört, dass das überall anders war. Es geht mir dabei auch um die Beispielhaftigkeit für künftige Plugins. Für additional_plugins habe ich auch einiges in der Pipe. Aber wirklich machbar ist das nur nach und nach.
> dass wir einfach megaviel Code haben der die braces anders setzt,

Meine Erfahrung ist da anders. Es geht bei diesen rund **31** core Plugins zB bei der `"function introspect(&$propbag) {"` um gerade einmal `(4 hits in 4 files)` gegenüber einer 2.0.3, die daraufhin von mir geändert wurden. Das ist ein Beipiel für meine geschilderte mehrheitliche Erfahrung damit. Ich war daraufhin einfach nur konsequent. Abgesehen davon habe ich wirklich **mehr** als nur das getan. Und in 2 tranchen (für sidebar und event) commited. Packt man solche multi-file, multi-code changes in nur einen commit, sieht man hier auf Github gar nichts, da HTML ausgepackt und gehighlighted limitiert ist. Und dann kann man auch keine Fehler mehr erkennen.


> How can we have AnyCPU build that provides its own 32-bit and 64-bit versions of the OpenAL dll and not require that the OS has them

I assume you need this for Linux, setup folders lib/ and lib64/, add bash script that checks if the system is 32 or 64 bit and then just override the lib path with one of those.

> How do we have gamepad vibration support without SDL2 (or even, how do we port the code over?)

OpenTK does not support it, you would need to write the code yourself.


I guess stopping criterion should not depend on the number of datapoint. Otherwise, by increasing the number of datapoint the accuracy of models decrease.


well the rebase was erased when pushed :) I will try again


Ringfree PBX does not run independent MySQL instances.  Removing the section of the init script that tries to start it.


thank you, already looks very good!


Not that it matters much for an implicit, but I think you mean `javaFromScalaFunction` (or `scalaToJavaFunction`).


Taking it out broke our templates.  I'm not sure how this will affect others.


Added you as a collaborator, try now.


This will crash if resource is not an ActiveRecord Relation ( and response_should_be_filtered? is true ). Maybe there's a better way to do this than claiming a parameter (:ids) that might have other uses?

I don't actually have this use case right now, so no big deal.


I know it doesn't fit the current CAP workflow as well, but to me the checklists are more suitable for incident_type_id/incident_id than Events.
I wonder if it might be a good idea to create an Incident for each location, linked to the wider Event...each Incident is then managed by the local team who fill out their checklist.
My only concern with this is that some locations might not actually experience anything which could be called an Incident...in which case it might be slightly wrong. Worth discussing with Nuwan perhaps.


no need for that loop parameter, that is taken care by the `sustain`.


@tyrasd, I did this in 84c98da


don't do the queryset inside the the list comprehension for some reason django evaluates the query everytime in the cycle executing n queries when n is the number of registers, instead do something like

`
release_qs = Release.objects.filter(available=True).order_by('-release_date')
releases = [release.version for release in release_qs]
`


What??


nice


Not just Linux but Windows as well. I can provide a 32bit target with the 32bit dll and it'll work on 32 and 64-bit Windows but there wont be any advantages on 64-bit like the increased address space.

So for Linux I only have to change the path in the OpenTK.dll.config?


This is for the specific Event/Incident not the type (am sure I gave that feedback before!)
Ideally both fields with a filtered lookup on the 2nd, although we don't ever envisage creating these manually through UI in the current usecase...would always come through system...so the filtering part could be @ToDo for when such a usecase does come up.


Esta música já está introduzida, mas com um titulo ligeiramente diferente "Seguindo a Jesus" com o id 28: https://github.com/psalterio/repository/blob/master/songs/pt/seguindo_a_jesus_28.tex

O melhor é apagar esta com id 233 e transferir os acordes para a outra. Se achares necessário pode-se mudar o titulo da 28 para "Estou seguindo". O ideal seria encontrar o titulo original.


You don't need to grab the user in the controller, if you just use `current_user` within the view, it will return you that user record


@rouson What did you have in mind? The closest thing I can think of is [cloc](https://github.com/AlDanial/cloc) which can count the total number of lines of code (declarations and executable statements, ignores whitespace and comments, AFAIK.

I think what we really need to get going on is source code documentation with [FORD](https://github.com/cmacmackin) and maybe Doxygen. (I say "maybe Doxygen" because FORD can be used for non-fortran code, but it doesn't perform any analysis of the code itself---Doxygen may be better for C because of this, even though I prefer FORD's output.)

Also, at some point I think FORD will include metrics for SLOC or maybe comments, executable statements, and other non-executable statements. See the discussion [here](https://github.com/cmacmackin/ford/issues/98). Also, I think there is [planned support instability indices for various modules/classes, etc.](https://github.com/cmacmackin/ford/issues/99) If you have input, please go add it to those FORD issues.


But no good idea for coding
next time, fix the error more flexible


This is another ugly loop. Again, the index into the array is simply `gpio / GPIOS_PER_PL061`.


It doesn't actually try the other credentials providers if the first one returns something.

But I've just noticed that I slightly broke the anonymous case when using `region = "other-v2-signature"`.


Instead of doing this `each` iteration, just do `<% = current_user.email %>`


This function can be void.


What a fine feature that is, thanks for pointing that out. AFAIK this wasn't mentioned in the docs. I only took a short look at the builds overview and thought it ran some tests over the tag.


Nice! great coding skills!


What I had in mind (and what I think the posterior on MAF approximates) was not truncating: `.filter(d -> d <= 0.5)` but rather reflecting:  `.map(d -> d < 0.5 ? d : 1 - d)`.  Apologies for not making that clear!


Just mention its been tested against 3.6-EA+ through 3.6


It's not considered a good practice to explicitly query the database in the view. Consider moving this logic into a MutualFund instance method, that way you can call `mf.industry_counts` anywhere in your code.


Hi,

It seems that M_PI has disappeared .


We should have a larger discussion about this, but for component-writers there needs to be a clear set of rules / expectations.  When tracing inputs to outputs, there should never be a discrepancy between xforms.  I will submit a list I have regarding these rules in a new issue ticket.


Accept this.  Your repo is broken.


fast-forwarding 1.7 to borshop-integration-229-im/bugfix/bitcask-file-process-leak = 9fbd9b4b


@samuelklee One last thing and you're done, no need to send back to me,


Just add a note here how TestDoNothingRegistrator requires explicit consul setup, and to read the source code before running


I guess here is missing a class for div?


NB I think this is important not just for separation, but because the checklist can take time to action and fill-out...yet we want to receive the ACKs ASAP to at least know that the message has been received OK.
So I would create a cap_alert_ack table as a component of the Alert...it would look similarly to the event_checklist: location_id, datetime, ACKed by


Actually I'm not sure you need a function at all since you can just do `pl061_dev.reg_base[gpio / GPIOS_PER_PL061]` and let the caller do any assertions.


Your Task 5 program prints "none" after the five words. To avoid that you can change the "print(go_state())" statement to a simple function call "go_state()"


In addition, you are just looking for one record, so if you did want to use the above strategy you should use `.first` so you only have one user record, or `.find_by(id: current_user.id)`. Right now if you run `.class` on `@user` it will return a `ActiveRecord::Relation`, but when you click `first` it returns a User instance.

This might be worth a read: http://guides.rubyonrails.org/active_record_querying.html


I agree.. This was just the first step in getting it to compile :). So I will give it a shot getting libgmp on windows too although then users have to manually configure the path to libgmp. Might not be too bad if the documentation is fine but I always prefer to reduce dependencies (mainly on windows) since paths are not very standard :)


Sometimes you don't want a message to disappear.  For example, if your components fail to load or the build fails, you do not want to return to a ready-state.  You are in a fail state and need to take action to rectify, thus the message should remain until the next action.


What do we get from suffixing everything except `sequence` with an underscore in the `detail` namespace?


Hi Sicheng-zhu! I also printed hello world, but using Python! Printing "hello world" in java looks more complicated.


Use `find` instead of `position` here to be clearer?


Otherwise it looks weird ;)
![screenshot from 2016-02-01 18-37-26](https://cloud.githubusercontent.com/assets/177257/12725435/f64948aa-c912-11e5-89c5-13d71896032c.png)


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (1 pt)
- [ ] Program is different than [Task 4](task-4) (1 pt)

## Task 6

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program functions as expected (3 pts)


Cherry picked


@glennmatthews @chburket : I have committed the latest changes.


ACKed by /could/ be just the created_by field, but I'd prefer it separate:
Imagine the case where the person doing the ACK has no access to the system & so radios in to the EOC to do the ACK. EOC staff creates the ACK in the system but they put in the real person doing the ACK not the name of the EOC staff member doing the data entry...


thanks fs44e1a


cherry-picked.


@geek so.. we've already changed the lib for handling prompts once, what's compelling about switching to vorpal?


The idea is that there is a single name of the rig and "guide" is only appended when building the guide rig.  This is just shoring up a convention already in the code that had some inconsistencies.


remove this nil?


Unfortunately the URL returned by builtQuery.generateKeenUrl('') is not compatible with keen explorer (even though it is compatible with keen API)


I've got something working but I need a better list of the local channels.


Delete transaction id checking, which is not needed for this release.


May be add pointer check for NULL? What may be the reason to add null pointer ?


Isn't it a concern that the String parts can be arbitrarily wrong?


looks correct. For simplicity, you could try:

```
phase = LFSaw.ar(sawrate, 1).range(begin, end)
```


112796613_DesicDehum (mjwitte) - x86_64-MacOS-10.9-clang: OK (2051 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-100.0%25-green.svg)


I am very wary of these wrappers inside loops.
I would much prefer the API to be bulk.
Why query all the filter_ids then lookup the filters one by one?
A single JOIN query could look these up in a single query


#113 fixes this, thanks for fixing this so fast @genuss


Keep validations near the top of your class, before your methods.


Best not to store scale unless explicitly needed.  Perhaps this is a flag on a guide control in the future if needed.


Looks good we will apply after testing (but it is a bit busy here for a few weeks). We would prefer to have pull request against the develop branch since all internal development is also done against that branch.


nit: could pull the `mmio_write_8()` call outside the if/else condition.


Made folders


On Windows 4.10.2 it works, but the Mac has still a problem with:
"Mac and Linux should work as well but requires compiling the sqlite3 binaries and editing the build script so that the compiler can find the correct libraries to link to."

After compile sqlite3 sources i have:
libsqlite3.0.dylib
libsqlite3.a
libsqlite3.dylib
libsqlite3.la
libsqlite3.lai
sqlite3.o

Help me to compile sqlite3 lib (SQLite3.Mac.lib) for mac from source. After fix i will give to your fork commit request with instructions (+ build script for mac).


function can be void


untested ;)
location_id undefined here...this should now be locations


My only comment is that it is hard to tell the substantive changes from the pep-8 stuff and visual cleanup...


Since now we have `if` statements in the `switch`, it won't compile without `return nil` :(

```
etcdserver/v3demo_server.go:127: missing return at end of function
```


When you destroy the user, what happens to the user's mutual funds?


Are we always sure that `temp` can't be null pointer?


Fixes #12


Not ideal by any means, but this was a simple way to allow us to inherit multiple formatting for some controls.  It would be great if there was maybe a tagging / property system we could use instead.


I investigated Outrun Online "greenish" hue in car selection screen when I found the issue.


Please CC @diracdeltas for the https everywhere changes.
@diracdeltas please review.


All of this stuff should probably be condensed into a for loop to make it easier to change in the future. Have some array of strings called $seasons or something that has ['housingWinter2016' => 'Winter 2016', 'housingSpring2016' => 'Spring 2016', ...] and iterate through that array to push stuff to $housingTimes.


Can you use rebase in the future instead of merging and creating extra commits?


Function should return `unsigned int`


Needed for dynamic ports.  Ideally, any connections on ports that are deleted become "disabled", but still connected so that toggling a dynamic port accidentally doesn't break a connection.


Thank you sir!


Can we handle non-strict hierarchies here?
Remember PH has one, so it's well within our scope for the current project.
I think this needs 2 parts:
* Don't nest the if L2 is not None inside the if L1 is not None (we may have an L2 direct to the L0)
* Have the parent check include both the list of L1 location_ids & the list of L2 location_ids


Fixes #24


P {margin-top:0;margin-bottom:0;}

Yes, some tests are next.  I don't know why the tests would now take longer than before, I don't think I have changed anything that would have affected that.

Hmm, "queue" was wrong.  I was thinking too far ahead... it's a stack of actions; I mean to implement a queue-of-stacks of plans.

From: Chung-chieh Shan [notifications@github.com]
Sent: February 1, 2016 12:15
To: hakaru-dev/hakaru
Cc: Carette, Jacques
Subject: Re: [hakaru] First draft of a Maple-side disintegrator. Main entry point is "disint", (8bdb2d5)

And... all the tests still pass that used to pass! (Though a few of the tests seem to take a long time.) Any chance for tests of disintegration?
And does "queue" mean FIFO?
�
Reply to this email directly or
view it on GitHub.


Updated pod files


kk I definitely tried doing that but it wasn't working.  Would the current_user.id need to be in <%= %> ?


My god man, you're right! D: Would you care to make a PR?


Here too this wrapper should not be inside a loop.
1st query within that is:
area_location_rows = db(ltable.alert_id == alert_id).select(ltable.location_id)
This will be identical for every iteration of the loop, so really should be done outside the loop.
Breaking things up into 'helpers' like this may make logical sense but they can very quickly mask massive inefficiencies in the running of the code


wat


May be add const iterator = obs_vec.end()? I can be wrong but don't we calculate obs_vec.end() twice?


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


на моё письмо почему нет реакции видимой? ответь на мыло, а не тут.


Great job!


Creation of a custom url session, NSURLSessionConfiguration can be used for set different values compared to the NSURLSession sharedSession, like configured caching rules


This is the fundamental part, once it is created a sessionConfiguration, we need to add the super mock protocols. The method is implemented in the extension of NSURLSessionConfiguration.
This method need to be mentioned and added to the readMe file and in the "How to use" instructions.


@Hypnotoad90 Not really. It just moves every file individually.


This prints just an empty line, yeah?


Please reformat the code with 2 spaces identention and no tabs


Are you referring to all of [those](http://www.bbc.co.uk/radio/stations)?


typo in comment :)


broken


The way I like to write it is keeping all variable declaration for the class inside the constructor


check if a recorded Response is already existing and it give it back.


Agreed.
I'll look into this separately from this PR.


картинку можно занести в бэкграунд в css
так даже лучше будет


Record the response (it will actually save the satis and the header fields)


`#include "application_manager/event_engine/event_dispatcher_impl.h"` should be on the first place before all other includes


ok!


Yeah, it's a newline, because otherwise we write on the same line as "x out of y test suites processed"


Magic number 2. Use a macro and a comment to explain how the value is read.


Good idea - will aim to do this as a future improvement!


Considering I manually edited both files, I'd be surprised if such a
thing had snuck in under the radar. I just reviewed both files, tabs are
present in both versions.

Cheers,
Andrew

On 2/1/2016 6:56 AM, Andrew Wilson wrote:
>
> It looks like there's no tab following the COST:0.1 in the Silver entry.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PCGen/pcgen/commit/c91db1d9df77c3e75ddb54680eabf9bcd7f892b8#commitcomment-15800980>.
>


Haha ok.

On Monday, February 1, 2016, Francis Ricci <notifications@github.com> wrote:

> Yeah, it's a newline, because otherwise we write on the same line as "x
> out of y test suites processed"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fjricci/ds2/commit/f8d7ba4bf651640ebb5b4467252f5af899f99ea8#commitcomment-15806223>
> .
>

--
Stephane Sezer


It's necessary to exclude venv out from repo by modifying .gitignore.

> Git requires only pure code in dense and terse state
A.K.


This method return the URL of the file for the mocked NSData linked in the Mock.plist


`#include "application_manager/event_engine/event_observer.h"` should be on the first place before all other includes in file event_observer.cc


Hi @empyrical!

This change is not Qt5 specific but C++ standard specific (98 vs 11).
"Strong enums" are not accessible with GCC with default standard (before c++11). The fix is to use transitional syntax. It has been checked with Qt 5.5.1.

Moreover I know that this code does not compile with Qt 5.3 and 5.4.

I am working on a patch now.

Cheers,
Mateusz


This method return the URL of the mocked Response for the request linked in the Mock.plist


So this one won't support a callable for 'how'?  I thought that was a powerful feature in other 'how' arguments elsewhere.


Shouldn't it be an `INVOICE` event, since the external charge is on a new invoice?


This file should probably be added in this commit.


nerds и since 2004 лучше в один спрайт


File event_observer.cc should have the following lines at the end:
`} // namespace application_manager `
`} // namespace event_engine`


эти картинки ещё и ссылки


Need to assign a value to it.


this is my first commit


Actually when you click preview, github gives you a json syntax check.


Your task 1 program prints a comma after the 50. Not a big deal, but I tinkered with your program so that it wouldn't do that. Here's what I came up with.

for i in range(0,51):
    if i%5==0 and i!=50:
        print(i, end=", ")
    elif i == 50:
        print(str(i))


Det är bra att använda:
 ```
if __name__ == "__main__":
```
efter funktions och klass definitioner. Det gör att funktionerna kan återanvändas från andra skript och dyl. Nu om någon skulle importera `makeCSV.py` från en annan fil skulle main funktionen köras. I detta fallet spelar det ju inte så stor roll kanske.


Record the NSData for the request


Record the Response Headers as NSData for the request


If you open a PR for this, I'll merge.


I started there, thanks, but the f4m names are quite different and inconsistent. There is a good list here: http://forums.slimdevices.com/showthread.php?103208-Available-HLS-Streams


import java.util.*;

public class Sieve{

  public static void main(String[] s)
  {
     int n = 2000000;
     long start = System.currentTimeMillis();
     BitSet b = new BitSet(n + 1);
     int count = 0;
     int i;
     for (i = 2; i <= n; i++)
      b.set(i);
    i = 2;
    while (i * i <= n)
    {
      if (b.get(i))
      {
        count++;
        int k = 2 * i;
        while (k <= n)
        {
          b.clear(k);
          k += i;
        }
       }
       i++;
     }
     while (i <= n)
     {
       if (b.get(i)) count++;
       i++;
     }

     long end = System.currentTimeMillis();
     System.out.println(count + "primes");
     System.out.println(( end - start) + " milliseconds");
   }
 }


I thought we already had such a lookup table defined.  Can we share them?  Or e.g. have two, one for reduce operators like these and one for unary ones, each named to indicate the difference?


That's tricky to do here while handling missingness in a robust way. Handling just the standard numpy reductions seemed like a good stopgap for now.


We already have C abstraction for file-like objects - it's called streams. So, there should be strong reasons why this is needed.


I disagree strongly about this.  I made it centralized so that it can change later.


That won't work since test_basic_ops import config too.  I haven't designed the test support environment to allow running with pygpu not installed.


I don't understand why you're writing `BIT(offset)` as the value when in `pl061_get_value()` you're only checking for a non-zero value. Couldn't you actually write `GPIO_LEVEL_HIGH` or `GPIO_LEVEL_LOW` as the value and return the register value directly in `pl061_get_value()`.


Actually not, but I agree it is not intuitive. If the variables are already set then requests.get() will use the environment when proxies=None is passed. Thus the system is consistent. We only want to pass proxies to requests.get() if we set them in all other cases the default behavior of the requests module applies.


Please submit this to uzlib upstream, then fix can be imported from it.


здесь какой-нибудь h2, у h1 суть озаглавить, быть в начале


Could you clarify please?


If you really want to do this make another test file that doesn't import anything and try to load your pickle from there.


I'll make a small edit to link this to the definition of "presentation is disabled" above.


great!


Börja filen med `#!/usr/bin/env python` och kör `chmod +x makeCSV.py` så kan man köra skriptet direkt med `./makeCSV.py` vilket är lite trevligt :)


It doesn't really describe what this project is for. But no document in tree currently do either, so I guess I could actually write one. It's not priority for me, though.


How so?  Does it just entail specifying that users should handle NaNs just like nansum, etc., for these functions?  Or are you not sure it will stay like that?


Special thanks for the suggestion @maxdev1 :)


We only take a lock if we have a handle?


The code behavior is consistent considering the requests will use the environment variables if they are configured. We do need to deal with existing setting as a user can call registercloudguest in a shell and there the settings could exist.

The consistency implies knowledge of the behavior of requests, and thus can be confusing. I'll add a comment.


read the mock of the response for the request. it goest to read the file for the response


This is the test content of this repo


I'm not sure what lookup table you're talking about, this is the only one for numpy ufunc reductions. And as I said below, this is hard to do while robustly handling missingness.


It is used by anaconda, but that seems to be a workaround for a udev bug since we should never have to poke udev directly to make it notice a change. I will remove it.


These were something I used just to help confirm I had expected 1:1 mapping of request and rgwop.  They don't really do anything, so should probably be removed.


Your Task 2 program technically does not call another function from within main(). Also, it prints numbers one below the other without commas (should be the same as task 1 program output).


Comment on commit msg: This is a bit too brief. You should at least reference the PL061 TRM here (ARM DDI 0190B is the version I'm looking at).


Just to be forward compatible, could you check if the status is committed first? This would ensure the code works if we introduce a new status one day.


Added bithound badge.
You need to


Excelente!


They'd have to be `nan` sparing, and take the `axis` parameter. In general, this whole method is a stopgap hack for now. I'm going to remove dynd shortly, and cleanup how all of this works. Will greatly simplify operating on aggregates.


I see we're waiting to navigate until `trackEvent` is finished. What sort of delay does that introduce?


We will actually need to remove blank=True as well, as we won't need it anymore.


@CaptainElliott i did it because of what @ryanwse said, it doesn't require any information, and it just occupies one line less of code


@TomyLobo call it what you will. I'm not that concerned with the name lol.

I should mention, there are some edge cases with this I thought up last night, so when I get around to it tonight I plan to rewrite this to lay "structural" blocks and then come back, figure out block dependencies, and place the remaining blocks only after their dependencies have been met, or until we're in a cycle, and the dependencies are unmeetable. That should fix the edge cases, and then I can start on a supervisor.


That's a perfect list. Basically, you just need a similar list like channel_list in ListLive.

If radio makes it to the master branch, I think I'll have to break the code up into several files including one to manage just these kind of lists, one for all radio stuff, one for all TV stuff and one for everything that's common/main.


Do we need two `then`s?


Hi

The diff on GitHub is very deceiving, it looks like it's not there.

I won't trust it in future.

andrew

On 1 February 2016 at 17:52, Andrew Maitland <notifications@github.com>
wrote:

> Considering I manually edited both files, I'd be surprised if such a
> thing had snuck in under the radar. I just reviewed both files, tabs are
> present in both versions.
>
> Cheers,
> Andrew
>
> On 2/1/2016 6:56 AM, Andrew Wilson wrote:
> >
> > It looks like there's no tab following the COST:0.1 in the Silver entry.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/PCGen/pcgen/commit/c91db1d9df77c3e75ddb54680eabf9bcd7f892b8#commitcomment-15800980
> >.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PCGen/pcgen/commit/c91db1d9df77c3e75ddb54680eabf9bcd7f892b8#commitcomment-15806232>
> .
>


I can’t control this on the server sadly. I have my browser set to default to utf-8 but I’m not sure if that’s the default. But thanks for pointing this out.

> On Feb 1, 2016, at 10:45 AM, Tomek Mańko <notifications@github.com> wrote:
>
> @CoralineAda <https://github.com/CoralineAda> : also unrelated to this exact commit, but I'll think it's worth mentioning (and don't want to raise an issue for something's that probably in the server configuration, not the repo proper):
> Documents returns by links to the markdown versions lack charset=utf-8 in the Content-Type header resulting in gibberish with non-ASCII characters (especially noticeable with Japanese translation <http://contributor-covenant.org/version/1/3/0/ja/code_of_conduct.md> for example). HTML and plaintext versions have the charset in the header specified appropriately.
>
> You might want to take a look at that.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/ContributorCovenant/contributor_covenant/commit/f3e1e6cffc397f4f198cb4a1c465c6169f489428#commitcomment-15804308>.
>


I think the favicon got blown out.


Buena!


MediaFileSet.Duration already takes care of returning a valid duration even if there are no files in the fileset.
So you should use project.Description.FileSet.Duration instead


Detta är bra sätt att öppna resurser på så att de automatiskt stängs som de ska


We should never catch throwables, cause it can even produce OutOfMemoryError or another errors which have not to be handled


Nah, it just felt easier to read to me.


@tstreble clearly a typo here.  I've fixed this in the version pushed to dev-recipe branch but you might want to update this on your own branches....


I just checked, and the other one is _interpolate_lookup, in https://github.com/bokeh/datashader/blob/master/datashader/transfer_functions.py#L92 .  The two lookup tables are not overlapping, and thus shouldn't be one object.  But then it's clear that this one shouldn't be named generically like _ufunc_lookup, because it's not simply a mechanism for looking up ufuncs by string name, but a list of the specific ufuncs that are relevant for this application.  Could either be _nanaware_reductions (if named by what is allowed to be there, which makes sense to me), or _flatten_ufuncs (if named by the specific method that's making use of them).


Нельзя проверять форму до того как загружена картинка.


why couldn't this and its corollary in timeFloorAction go into baseExpression?


I miss the part of the weekly retrospective we have scheduled to monitor, evaluate and adjust our own process. Perhaps this can be added under: "A reflection: what is necessary to make this type of structure work in this context, and to what extent did you do that. To what extent did we adapt our strategy during the way."
We schedule a weekly time-boxed session in which we evaluate and adjust our process based on the individual input of the group members. This session is based on the retrospective meeting from scrum.


The note above is not relevant anymore can you delete it please?


Ok, sounds good.


auth token is using for user session, so it shouldn't be generated in constructor


Will the rpl_set_downward_link(0) be set somewhere else so that the is no risk of ending up believing that the link is valid but it is not?


Andrew,

Lol -
Yeah, I use the diff on github just for broad review. For complicated
items, I end up fetching the branch from the user and using my diff tool
which is a lot better visually. Though I do see why multi-line code is
sooooo much more preferable. I use the wrap feature just to keep my sanity.

For post merge, you can use the log and do a diff on that as well.

Cheers,
Andrew

On 2/1/2016 10:06 AM, Andrew Wilson wrote:
> Hi
>
> The diff on GitHub is very deceiving, it looks like it's not there.
>
> I won't trust it in future.
>
> andrew
>
> On 1 February 2016 at 17:52, Andrew Maitland <notifications@github.com>
> wrote:
>
> > Considering I manually edited both files, I'd be surprised if such a
> > thing had snuck in under the radar. I just reviewed both files, tabs are
> > present in both versions.
> >
> > Cheers,
> > Andrew
> >
> > On 2/1/2016 6:56 AM, Andrew Wilson wrote:
> > >
> > > It looks like there's no tab following the COST:0.1 in the Silver
> entry.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/PCGen/pcgen/commit/c91db1d9df77c3e75ddb54680eabf9bcd7f892b8#commitcomment-15800980
> > >.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/PCGen/pcgen/commit/c91db1d9df77c3e75ddb54680eabf9bcd7f892b8#commitcomment-15806232>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PCGen/pcgen/commit/c91db1d9df77c3e75ddb54680eabf9bcd7f892b8#commitcomment-15806611>.
>


The following comment added in Agda.cabal

> -- parsec is only for Agda.TypeChecking.SizedTypes.Parser, which is not included

is doubly confuse because Agda.TypeChecking.SizedTypes.Parser

1.  is included in Agda.cabal and

2. doesn't use parsec (because its code was commented-out).

@andreasabel why not move out Agda.TypeChecking.SizedTypes.Main/Parser from the Agda source tree and put them in something like src/size-constraint-solver?

To prevent bit-rotting, we can test this stand-alone program from the Makefile and Travis.


What was the verdict?


why should we use selenuim asserts? Common junit's one is enough


In fn_Setup_Sation_Service.sqf  we should make that addAction localized


But did the test work previously? Is this an inference regression? See https://github.com/JuliaLang/julia/issues/14883 which might be related.

(A few words in the commit message would avoid the need for this kind of discussion.)


I believe in python regex `/` is no special character, no need to escape it^^


Is this redirect necessary?


this shouldn't be in default config, we need to create another one each app, or create a bash/cmd script which is replace it on deploy


Any chance to make this non-polling? Frequent wakeups prevent the CPU from going to sleep which increases power consumption. We haven't spent any effort on that front in GPDB, but we have done something in later PostgreSQL release, and I'm sure this will come up for GPDB sooner or later too. So would be nice if we could avoid tight polling loops like this.

So: is there a way to interrupt the blocking Queue.get() call in Python, so that we could keep the timeout relatively long? One idea is to add a dummy Command to the queue.


From a quick read of the TRM, I think I see what you're doing here now but it definitely needs some comments.


Good job! We added a few more comments that should be easy to fix. Then we'll merge your PR.

We will take care of merging ourselves the latest `master` into the HR branches because we made quite a few changes that may conflict with what you did. We 'll let you know when you can merge back on your side.

:+1:


It worked!


since we select new console, " $select_console 'user-console';", the needle "ssh-loging-prompt" doesn't match anymore, because console is now cleaned, before wasn't.

so new needle has to be added for the test to.


one of the main pros of using PlayFramework is non blocking i/o, servelts does not support this. Need a discussion on this case


Just for the record, this commit also has changed a behavior that Atom will not contain a bundled `node` anymore.


@vozhyk- dunno.  Can you provide a `diff`?


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 102](https://teamcity.labkey.org/viewLog.html?buildId=300293&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:14:36


Please update the indentation for the next two lines.


👌👀👌👀👌👀👌👀👌👀 good shit go౦ԁ sHit👌 thats ✔ some good👌👌shit right👌👌there (self.copypasta)
inviato 7 mesi fa da Notpoe779
👌👀👌👀👌👀👌👀👌👀 good shit go౦ԁ sHit👌 thats ✔ some good👌👌shit right👌👌there👌👌👌 right✔there ✔✔if i do ƽaү so my self 💯 i say so 💯 thats what im talking about right there right there (chorus: ʳᶦᵍʰᵗ ᵗʰᵉʳᵉ) mMMMMᎷМ💯 👌👌 👌НO0ОଠOOOOOОଠଠOoooᵒᵒᵒᵒᵒᵒᵒᵒᵒ👌 👌👌 👌 💯 👌 👀 👀 👀 👌👌Good shit


That one wasn't mine, but thanks for noting so I can take the opportunity to fix it while I'm moving stuff around.


Lol, woo! I would get rid of the comment so we don't have to think about it ever again!


And I should change this back as it breaks apm.


Duh, hold on…


Tror du har en liten bugg här. Om `row[0] != format_frase(row[0])` och Värdet av `row[0]` förekommer mer än en gång så kommer enbart sista värdet sparas.


Yes, we have some links within our code that redirect the user to `/docs`. Although we don't actually have a URL that maps to that because we always do `/docs/{{version}}`, the redirect will send us to the `docs` state which when the version is absent it will just use the latest.


thx bb


I guess this is just to know that it is not configured? It is assumed to be a "temporary" variable set by the dao seqno counter whenever that specific instance is under and waits for the ACK. But I guess this will make it more visible when getting an ACK while not having made a request.


It would be nice to add 16 and 32 bit per pixel support. Perhaps something like this: https://gist.github.com/RichieSams/091c3c06b7c679ec208d


Ah, gotcha.


The issue as it presents is `bad option: --harmony_collections`.


Yes it's not, just tested


unack


111411352_ATDeprecateFields (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1512 of 1512 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)


I had been thinking about this for a while, and AFAIK there are more needed to that than this:

- a bunch of places hardcode paths to `module_init_tools`s modprobe, which doesn't support XZ-compressed modules IIRC
- AFAICT this leaves the uncompressed modules un-stripped. Something like https://github.com/dezgeg/nixpkgs/commit/76926a6ba0f6bf94d3839f657e10f1e2d0bb9fc9 should fix that.


yup since if you don't do the `=` it isn't output to the view? If its just `<% %>` then it is typically for calculations or iterations.  In addition the user could not be signed in, you can check that by doing `user_signed_in?` which returns `true` if user is signed in and `false` if not.  You can see that in the [devise documentation](https://github.com/plataformatec/devise#controller-filters-and-helpers)


Technically if it's < 1 it should be an error, as the C standard says (argc < 1) is impossible.

Though (argc < 1) may be possible if `int` is something like a 64-bit type, and your `int32_t argc` usage of POSIX `<stdint.h>` incorrectly truncates `int` to `int32_t`.  But otherwise this should never happen.


Thank you for noticing it
I corrected it in the original branch.


On line 104, you set `button` to the button for Exercise #1.  You never updated it to refer to the button in Exercise #2.


Or also maybe something like:

```rust
args.iter().all(|i| i.to_str() != Some("--target"))
```


Patch atom and remove it. This is a deprecated flag in nodejs.


oh this wouldn't work, you don't need to do a User lookup, can just do `@user = current_user` or just call the `current_user` in the view


Language codes are designed so that you can drop from a region specific to region neutral code. e.g. `en-US` can drop to `en` by removing the `-US`

To avoid the looping, can't we just try the region specific code and then degrade to a region neutral code?


After removal of the flag this shows up...

```
Error: /usr/share/atom/resources/app/apm/node_modules/git-utils/build/Release/git.node: undefined symbol: _ZN2v82V817IsGlobalNearDeathEPPNS_8internal6ObjectE
    at Error (native)
    at Object.Module._extensions..node (module.js:440:18)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at Object.<anonymous> (/usr/share/atom/resources/app/apm/node_modules/git-utils/lib/git.js:8:16)
    at Object.<anonymous> (/usr/share/atom/resources/app/apm/node_modules/git-utils/lib/git.js:371:4)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
```


method name doesn't correspond to it's logic, so for next one


Demangle it, will you?


I was going to comment on this, but I realized that you had written unit tests, and thus had probably caught it yourself. :smirk:
BTW - you spelled 'optoin' here if you care...


Euh, beurk ?


`v8::V8::IsGlobalNearDeath(v8::internal::Object**)` hmm. Checking all sources.


Now that I think about it, webp has been added ti the images format list, so it should be included as well (minor edit)


Removing the need to type 'typename [...]::type' all the time.
Like the std::[...]_t aliases.


Done in d801ea9735ae29a31e73b40ea2df0e5a41a4661a


а) to_s избыточен, он и так вызовется, когда ты что угодно в строку таким образом вставляешь
б) но лучше не to_s, а inspect, потому что если вызвать без параметров, то с to_s ты получишь ошибку "Unknown name: ", а с inspect — "Unknown name: nil"


if it can be called or by something else (isnt, negate) add the function and also add to expression.pgjs (same as plyql)


`sort_by(&sort_block)` ;)


Register part of Modal


i just dont wan't to duplicate this everywhere :P


That looks like a private function. What did atom guys do to their bundled V8?


Very nice! I've been meaning to do that at some point, but never had the time to do so. Well done!


here it is using the context `c` as the source of the timezone which in resolved at evaluation time


Or equivalently `!args.iter().any(|i| i.to_str() == Some("--target"))`.


Weird.  I must not have gotten this error on my end for some reason, so I never noticed.


:tada:


Checking `v8.h` history.


We make an excellent bike-shedding team :+1:


Wouldn't this also need the key as well? ie: 'Entrust' => Zizaco\Entrust\EntrustFacade::class,


@mitza-oci Thanks for pointing this out, fixed


Hard to read this - create random value 50 and 250 milliseconds before making a second request.


> That looks like a private function. What did atom guys do to their bundled V8?

AFAIK, yes.


@bpcoxjr don't forget to extend event emmiter


If you like I can add you to email support rotation so you can see how scary it really is when feeds disappear on users. But I think the checkboxes are hiding what's really going on. The dialog is called Mute Sites, so when you check a site, it should be muted, which is not happening here. And if we invert the logic, then some folks will think the opposite. Better to be explicit.

I'd be open to a better word than Options, but there should only be a single menu. I'd be OK with putting the settings cog in the right, next to Done, and have that be the menu (like it is elsewhere), and put a Cancel button in the top left. In fact, that would be much nicer IMO.


Mais tu vas dupliquer `const util = require('util');` partout à la place, qui est moins explicite que `const expect = require('chai').expect`.


Checked this out, the root comes from the API =( If possible, we need to find a smart solution for this to avoid handling Throwables


Omkastad ordning på `subdirs` och `dirs` tror jag


nifty.


* "There's only a throttle delta which is assigned to both of them." - Throttle delta means "a change in throttle value". If 1 is passed to throttle delta, the value increases with each tick, if -1 is passed, the value decreases with each tick. The keys, since they are strictly 0 and 1, need an offset of -.5 and a scalar of 2, then their value routed to throttle delta so that they incrementally adjust the throttle instead of set it to maximum and minimum with each press and release. This becomes important because the orignial games routed the gamepad trigger directly to throttle (not delta) so this complexity is to make everything come together in the same system.

I'm thinking of getting rid of throttle delta and replacing it with a user-specifiable setting function: "absolute" or "cumulative" where "absolute" sets the control target to whatever the controller is reporting whereas "cumulative" increments or decrements the control target by that value.


if you like, you can leave out "BG", we can see it in the commit history.


Why is there matplotlib code in the file that's supposed to handle the sockets?


I'm not sure. I did the same for all of the names that store original values for the purposes of passing to a change default_namedtuple constructor.


I think the performance improvements are related to JavaScript, not DOM rendering. I vote for switching it back.

(Also, another issue I found this morning, when you get to the bottom of a story and pull up, the background is now white instead of the dark grey as before.)


Hmm so @demmer if `periodicLiveRead = true` and `to=:now:`, it will still do live?


c'est vrai remarque, de toute facon je vais remplacer mocha par :
https://github.com/sindresorhus/ava ce qui resoudra le probleme a la source
!

2016-02-01 10:28 GMT-08:00 Olivier MIEL <notifications@github.com>:

> Mais tu vas dupliquer const util = require('util'); partout à la place,
> qui est moins explicite que const expect = require('chai').expect.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dstendardi/monolithic-repository/commit/16f3e0afc0fe8d538a7e6b303e07396e167e000e#commitcomment-15807263>
> .
>

--
David Stendardi


Detta blir troligen fel om inte alla filer ligger under `directory`. Om du ersätter med `dir` (vilket jag tror är det du vill) så måste du komma ihåg att också lägga till ett `os.sep` (`/` i unix världen).


the break was for testing puropse. I just wanted to check the flow for one message at that time. will remove it.


I agree you don't want to add a macro to platform.mk. But you could provide a macro in this header to "register" the max number of GPIOs, which then then calculates MAX_GPIO_DEVICES. e.g.

    #define REGISTER_MAX_GPIO(x) const unsigned int max_gpio_devices = \
        ((x) + (GPIOS_PER_PL061 - 1)) / GPIOS_PER_PL061

Then just call this macro in your platform source code


doesn't need a separate class


The code fix the alert, but the MinifyX stop working because the cache files under `/core/cache/minifyx/` can't load properly.


It would be nice to have TestInfoNegative alongside.


Okay, I'll revert it.  Sorry for the hassles!


This would be really useful as a separate plugin. Would be cool if you notified me in IRC when you start working on this.


LGTM. I have a couple questions (mostly to satisfy my own curiosity) about what we're going to be _doing_ with the hostname data itself, but I realize that isn't part of the scope of this commit. I'll probably hold those off for a future PR. :)


Alright. #1637.


Hey guys, it's good to see you're jumping into implementation. Just as a word of help, having long lines of hard-coded 'example' states can get you into trouble later on, especially if there's some tiny bug hidden inside. It might help to code your serializer early on to help with programmatic return values, test cases, etc.


Check the libraries too, will you?


Dans le dashboard on avait un peu plus d'information que la liste des numéros. On avait aussi les informations de l'utilisateur. C'est correct qu'on ne l'aie plus, on pourra la remettre dans la page d'accueil de l'espace utilisateur.


Bara för att jag är lite anal och stör mig på detta: `frase` => `phrase`


my mistake - thanks @igaponov


Je mettrais `^/$` pour que l'accueil éditeur soit directement dans `/editeur`


Last seen in node 3.14.x


Got it - makes sense to me. Thanks Alex!


No worries, I've been wanting to switch to WKWebKit for a while. There's a ticket sitting around somewhere for it. I didn't realize it would introduce all sorts of chaos, though.


I believe the node bundled is of 0.12.x branch.


TODO: Il faudra aussi vérifier les permissions de l'utilisateur


Meant to say V8…


You may want to place this to `/bin` instead and use `wrapProgram`.


Why is `LD_LIBRARY_PATH` needed?


As the core folder can be moved out of the webroot as well, MinifyX should not be writing its cache files there if they need to be loadable from the web.


Damn right.


@nirzar please review the snapshot images here. the red border represents the save button hit area


dumb


TODO: est-ce qu'on conserve cet url ou on nomme abonnements-individuels ?


Nasty typo: `Spekaer`?


Sure. I’ve updated the PR accordingly.


Dude, si no saps que posar al missatge del commit, deixa el change-log, no posis porcades... x)


Not necessary in this PR, but we should have tests with a secure MockClient (tlsconfig.ClientDefault).


Then it will never match for our node...


Listo! Cambiado el nombre del script a **zenity.sh** como lo indicado.
Tienes razón respecto al comentario, solo copie y pegue por la hora y el sueño ya realizó las modificaciones de la descripción en cada scripts!


Bien noté


also doesn't need to be a separate class


necessary*


I'd suggest that we keep the same formatting as the above call, copy lines 1558-1561 and just change the 1 to 2 so we can more easily see the difference in the two calls.


@edolstra in case @globin's point wasn't clear from the icon, you negated this check :smile:


do you have a suggestion how to document the format of the returned JSON object?


get_all_stations() == sapphire.Network().station_numbers()

https://github.com/HiSPARC/sapphire/blob/master/sapphire/api.py#L318


done


Could we add comments above this line

```
        #
        # Get the component property, 2- Description string
        #
```

Also add a similar one above line 1558 so we can better read this code.

```        #
        # Get the component property, 1- Current firmware version
        #
```


`RGWLib() : fe(nullptr), fec(nullptr), olog(nullptr), store(nullptr) {}`


Rest LGTM.


Oh, never mind, @aszlig fixed it


Git is not the place for binary files


that gives a compilation error because of the outer exception also being called 'e'


Can someone tell me how to fix the 1440p error when building?


A little nit-picky, but I would even say we should write this out "component_string" or "component_name" instead of comp.  Just to understand slightly better


What do you recommend move it where and what is going to happen with the cache folders inside it not only the MinifyX one?
Thank you Mark.


@mattbenjamin yeah, ignore this comment.  `lookup_fh(parent, name, RGWFileHandle::FLAG_LOCK);` will make sure we're locked


`async` methods shouldn't be declared `void` (unless absolutely necessary).


I am a nice comment


I am glad you made a pull request, however, you could just have add "/xterm-termite" on line 14, as mkdir -p would create all directories in the given path.
Thank you for pointing that I forgot this directory.


the laptop browser blacklists based on host + path, but blacklisting by just host is probably ok.


> You can use the skipif decorator for particular tests in the python source itself; you
> might consider doing that instead to keep the skipping logic closer to the tests that
> are skipped.

Doing this declaratively so test to be skipped is marked with `@pytest.mark.winskip` - the logic just looks for the cmd line argument and ensures the system is windows before skipping. Let me know you had something else in mind.


fixed


The change is that inference is now deciding not to run on this code.


Skriptet kommer alltid att ge ett fel eftersom du inte skickar med parametrarna till `make_untranslated_list`


Ce type de modifications, je la comprend parfaitement, mais c'est typiquement une méthode pour ralentir Matlab. Je pense qu'il faut faire comme j'ai fait (dans le cas ou l'on est dans des grosses matrices) car tu as besoin de pré-allouer surtout quand tu sais ce que tu fais.
Mais si ça te déranges de revenir en arrière, je peux laisser comme tel mais il va il y avoir plein d'autres endroits ou j'aurais la même logique (matricielle).


are the tests related to metadata changes?


Bien noté


Changes to service and stack metadata look good.


OK looking at this again it seems I got confused, I mistook this variable for a lollipop but it isn't supposed to be is that right?


The enumeration name should be `WD_COLOR_INDEX`. These names are formed by systematically translating the MS API enumeration from camel case into All-caps snake Case. So separate words (like "Index") are separated by an underscore. The designation `WdColorIndex` should only be used to refer to the MS API enumeration; within python-docx it's always `WD_COLOR_INDEX`.


LGTM


This conversion is unneeded and can be removed.


Yay


Until the decision is made to change the entire code-base to use point-free style, things need to stay consistent in the separate request style.


Added command line option since I don't want to assume tests are always skipped. These are skipped when using docker containers to run postgresQL on the Mac - haven't yet gotten the windows mount working in docker container. The idea being for our current test setup we can skip w/ cmd line argument.


This is only a variable to keep track of the sequence number that this instance is currently using for an "outstanding" DAO transmission - until the ACK comes back. It is always overwritten by the next "send".


Extract creation of `Function` to private method.


Missing copyright notice.


Consisten with the comment above, the `wdYellow` should be `YELLOW`. There is a little bit of interpretation required, in this case to drop the `wd` suffix. Basically the idea is to make it read as naturally as possible in Python, where it will always appear in dotted notation with `WD_COLOR_INDEX`, e.g. `WD_COLOR_INDEX.YELLOW`.


the URL -> the attribute


Looks good, don't see any failures.


Thanks


@wei170 you shouldn't commit .class files to the repo; add `*.class` to the `.gitignore` file


Not sure this merge commit was necessary.


I've been watching this for a while, and definitely a :+1:!


Any reason in particular for replacing that one test, instead of adding a test case?


Don't get the meaning of the sentence.


in a the


As stated in [https://forum.minetest.net/viewtopic.php?f=6&t=12748](this forum thread), I noticed that there is a 2 seconds interval in which inventory changes are shown on the client. I now found the source of these 2 seconds:
m_nothing_to_send_pause_timer is set to 2.0 every time there are no changes to make, but this timer is not resetted when SetBlockNotSent or setBlocksNotSent are invoked. So in worst case, the changed block will be sent over 2 seconds too late.
Please notice that I am a C++ newbie and if I did anything wrong, please let me know. I don't know how threading works and what would happen the functions are called in different threads, this maybe could cause inconsistencies.
This will probably only be a temporary solution until [https://github.com/minetest/minetest/pull/3166](this pull request) gets merged.


lgtm


associated with this item type.


This is a terrible fix and tells nothing about what went wrong.

This is why PRs.


I'd actually suggest that we do this string manipulation in the callback in the call to the $sessdata->{ipmisession} so that the `%comp_string`` is translated and saved.

Instead of re-using ``_on_receive_version`` callback, define another callback function that will convert the ASCII to alpha

```
    sub _on_receive_component_name {

    }
```

Then call it...

``callback=>\&_on_receive_component_name``


show -> shown


и если не сложно то добавь еще в %prep

`find . -name '*.cpp' -or -name '*.hpp' -or -name '*.h' | xargs chmod 644`

это исправит 14 warnings W: spurious-executable-perm


The enumeration, either the MS one or the python-docx one, is a mapping of names (e.g. wdYellow or YELLOW) to an integer (not a string value). The mapping to a string is part of the XML schema. You can add that string mapping if you want, just put it separately. I don't usually document the name to number mapping since that's readily available at the link you provide above, but you are of course welcome to add it if you like :)


Great! Thanks.


Just update the hashes in the `abjadbooktools` tests. They fail because the `abjad-book` output files are named after the hash of their contents. If the contents change, the file name changes. This is going to be a normal and expected failure pattern, much like the failure pattern we see with the function counting tests in `systemtools`.


There is a parser control configuration (see end of language file). So the @@ROWCOUNT was intended to be highlighted as @@ as prefix and ROWCOUNT carrying the actual link. The part of the keyword in this list is also what is used to generate the documentation links (which is necessary for some language files).


What we had done before was check the encodings and fallback entirely if any page wasn't dictionary encoded.  That seems like the only viable solution because you don't want to resort to scanning the data to satisfy the filter.

I'm not sure if this is the best approach, but it's what we were using:
https://github.com/zhenxiao/parquet-mr/blob/dictionary/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ParquetFileReader.java#L580


@smarterclayton reasons why this is bare?


Ha estat es puto lluc


sorry I missed that one.  Thanks for putting back in.


Are these checks necessary? Would a hpm file component ever by at version 0?

I wonder if we could figure out the component name here or if we need to hard code "BOOT", "APP", "BIOS"


Ets retrassat...


You also need to read the first line of the file, parse it, and then initialize the serializer with the set of headers sothat it can properly implement this check for new or missing fields:
https://github.com/juttle/juttle/blob/master/lib/adapters/serializers/csv.js#L32


@mattbenjamin the original code didn't have that, maybe we should do it now, or just see if we need to call init().


Also see [Issue 130](http://github.com/weidai11/cryptopp/issues/130).


This seems like a bogus change.


@Krivega Yeah I got this now. I had to run `renderAllItems` after the `filterer` condition has changed. It was just a bit misleading that collection-view has two ways of not showing the items: 1) through not adding them into DOM, 2) through setting CSS display: hidden style.

I guess this is part of the optimization, so we render items for a first time only when it's needed, and we don't render them again on changing `filterer` conditions if there are in DOM, but just using CSS.


It appears that that would do it.


Yes I'm aware, this will be changed once we merge the MQTT dev branch.
Testing the new PHP MQTT lib with the new MQTT tree structure seems to be
working well. Did you mange to get the php mqtt dev branch to work? We will
have to add the MQTT php library install nodes into the docs install before
the merge.

On 1 February 2016 at 17:21, Paul Reed <notifications@github.com> wrote:

> For info Glyn - phpmqtt_input.php is still using 'nodes' on master &
> stable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/emoncms/emoncms/commit/cb9b24eaf290cd3d79b175c05b173a1690e25c9b#commitcomment-15805369>
> .
>

--
Glyn Hudson

http://megni.co.uk
http://adventuresplusnorthwales.blogspot.com


I think when I rebased I fudged this up as pattern is only accepted with grok.


I think the fee is reasonable.. even cheap.. I could see this be $1 even.


this red looks wierd


ok. worked!


Sure!


You should remove any trailing whitespace (blank lines) at the end of the file. After these changes this looks pretty good, I think it's ready to be committed and we can proceed to the acceptance tests. You'll need to get it on a separate branch (`feature/highlight`) and squash the commits so it's just the one, with the standard naming: `docs: document font.highlight_color analysis`.


ох, да. забыл :)


That one also looks wrong to me now that I look at it again...

Moving to `dao_output_target_seq` was necessary though, else the DAO sequence number would not be always initialized.


OK I'll update the PR


Oder Schlamm? Darunter wir der Schlamm eingeordnet. Einge andere Nachrichten erzählen von Ogern und Schleimen, die angreifen. Und es gibt den Sumpfschleim... Alles auf Schlamm oder alles auf Schleim ändern?


Could you elaborate the use case of host ip? If you're talking about the host ip in `api.PodStatus`, it's going to be deprecated. https://github.com/kubernetes/kubernetes/issues/6558#issuecomment-178098244


FYI, if you import cursor_value from bayeslite.util, you can just do:

    table_count = cursor_value(count_cursor)


Oh, you already did the same thing I was going to propose.


whats the heroku url???? lol


Two very tiny nitpicks.
- `blogID` could mean the property on the `Blog` model. Maybe say specify its the `NSManagedObjectID`?  Same for the other places the error is logged.
- `existingObjectWithID:error:` can return nil if there is no match and won't necessarily throw an error.  Maybe the wording could be tweaked so its not specifically an error, or perhaps add a conditional specifically for the error?


Unfortunately, this creates a boot-lock:

19:44 < MaStr--> jup, there is a script error
 [    1.190000] Freeing unused kernel memory: 216k freed
 /etc/preinit: /lib/preinit/99_10_failsafe_login: line 23: syntax error: unexpected newline (expecting ")")
 [    2.350000] Kernel panic - not syncing: Attempted to kill init!

you slipped the order of \|  , it should be |\


We should name the property `highlight_color` rather than `highlight`. Long story, and basically a judgement call, but the gist is that highlight_color names the value assigned and that highlight by itself could be a boolean. Just a readability thing. Note the MS API calls it HighlightColorIndex, which is along the same line, just too long and index is too implementation dependent for my taste :)


En fait on peut tout simplement l'appeler « Espace utilisaeurs »


When assembling queries from strings, you should use bql_quote_name for the names of any tables in columns, in case they have, e.g., spaces or other funny characters in them:

    from bayeslite import bql_quote_name

    bdb.execute('SELECT COUNT(*) FROM {}'.format(bql_quote_name(table)))


Hi, Thanks for all comments, I didn't know about this default delegate: registerDelegateDefault
Nice feature :) I still have to force AB to compile on my system, I think this is because of gcc compiler
I use, Microsoft compilers seem to be more clever, and probably I need to put some include in proper file
in my project to make it work.