Please see my earlier comment:

Please rewrite this as follows:

``` go
f, err := GetCurrentAclFactory()
if err != nil {
  return nil, err
}
return f.New(entries), nil
```

This is the expected way to indent error flows in Go. See the internal Google guidelines.


As said in https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Math/round

The Math.round() function returns the value of a number rounded to the nearest integer. But we need a float.

Math.float() doesn't exist. You give 2 params to Math.round(x), javascript ingnores second param.


That is not really what I meant. I was hoping we could not have to expose a setter on `System`.


s/Expect/Expected/


For what it's worth, I preferred `Freya.memo`. Why do we need `Freya.Memo.wrap`? I may have missed some need for additional memoization primitives.


`test-user` for consistency with `test-repo`?


> that’s

Wrong contraction, should be `that was`


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


Looks good! I might change the "Once your community decides what to work on, write it down!" part since that may not be relevant for the fellows right now. But is there somewhere else where we could highlight that the contributors should be part of roadmaps as they are revisited? Maybe just at the end of that first paragraph


TRUE and FALSE are reserved words in C++. So, they cause compile errors when used in schemas.


so this function was basically taken from https://github.com/opencontainers/runc/blob/master/spec.go#L637


Bit of a memory test for you, @qqshka - have had a request to hide mm2 messages between matches and wondered if this change was in response to a similar situation?

We've got issue at the moment that ezquake's cl_fakename will cause all mm2 to be shown, which stops the player from being able to use say_team to communicate privately.

I'm thinking of changing this to include all in-game mm2 in mvd, but exclude all mm2 during standby/countdown.  Would that conflict with the original reason for this change?


updated


Ah makes sense. Java has the same issue with the lowercase versions.


nice feature!1! Must have on any server!


Take a look at this... This code could overwrite previous data


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


@OwlFonk Why remove the dist folder? Some peeps want it: https://github.com/smashingboxes/OwlCarousel2/issues/1136


Thanks for this by the way. :+1:


$awayTteam spelling mistake?


Is Markdown formatting preferable to HTML tags?


i like this, although is view actually an eventemitter on master?


This function sounds like feature creep to me. User can already do this with "write_files". This just duplicates code that can break. Do we really need this?


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


@IonicaBizau yay you are using sweet alerts!!! hehe :+1: :heart_eyes:


Did you manage to get this done?


It's also a mono-tasker function. Why is this in lib? how many modules need this?


I think this might need a version test because urxvt in slacko (9.20) does accept `-hold`. May have to go through the urxvt changelog to find when they added it.


Again, are Markdown ticks preferred over italics?


Can you please change g (Green) paramter to 0.0 as well.


И всё-таки -- зачем генерируемые файлы (`cmd/fmcount/types/type_ffjson.go`) держать в git'e?


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


Nope. I just didn't wanna delete then it in case we needed it for anything. I'll get rid of it.


That... is a really good question. I only ran unit tests, I didn't actually test the debug plugin.


Would also be useful to include a comment about what the default behavior is on single- and multi-core devices since it's not clear what the default is.


I concur that it's worth keeping. There's only been one "complain" to date
- give it some time and see if more people object.


your welcome, we had our own timeout apocalypse, so glad to get that in :D


@WhiteNeo- any specific reason for this? Due to this [Chrome uses its lowlevel tab switcher](https://code.google.com/p/chromium/issues/detail?id=525745) and one needs to set `ro.config.low_ram=false` again to re-enable the regular one.


Another test: Tap on the VERY SAME street
- near a house number: Resulting town assignment shown is correct (I have not seen it wrong yet)
- near where no house is: Street is shown from the WRONG town, in a large number of cases

See 2 screenshots attached: "Au-Haidhausen" is the correct part of Munich, while "Dachau" is wrong (suburb many miles away).
![untitled1](https://cloud.githubusercontent.com/assets/845735/11486748/5776ad7c-97bb-11e5-8a1c-eda21f7e6bde.png)
![untitled2](https://cloud.githubusercontent.com/assets/845735/11486750/5b87dcec-97bb-11e5-9e16-f04f1ad04e07.png)


Is Chrome looking to support more than one SourceBuffer per MediaSource? It would be excellent to have audio and video separately!


103643052-FiniteDifferenceFluxOutput (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1962 of 1972 tests passed)

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


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


need to verify that a filename can contain dots in this pattern.


accidentally submitted more than I meant to


correct syntax of these variables' content is "http://www.com:123/"

the "http://" part is needed to make many applications work.


Hello,
The script will automatically install it on your wordpress setup.
Just config the parameters in the installer/main script! Let me know if you have questions.
> On Nov 30, 2015, at 4:48 PM, YasserGomaa <notifications@github.com> wrote:
>
> How Can i run these please :)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/abiusx/WP-SQLI-LAB/commit/d0b2668e3c94e6fc4761b277f7ed4ba05fdb43fa#commitcomment-14694248>.
>


@PEMapModder  i have a question about the pocketmine .phar files... i am really good with plugins but.... how do you make the pocketmine files into a phar after its made a zip? thxs


_forward + _right * u + _up * v  - the same as on the line above, can be reused


There's a typo for "REnault"


can you please tell me how can i contact you directly through FB or whatsapp or anything else i tried to contact you through researchgate as iam a researcher in a masterdegree


@RamiroV ramiro esto lo usas vos? Porque este sp lo cagaste con esas líneas y devuelve cualquier cosa probando algunos valores. Si vos lo usas se puede arreglar pero con juan lo usabamos y lo cagaste, fijate antes de rancear sp ajenos.


mailto:me@abiusx.com

> On Nov 30, 2015, at 5:43 PM, YasserGomaa <notifications@github.com> wrote:
>
> can you please tell me how can i contact you directly through FB or whatsapp or anything else i tried to contact you through researchgate as iam a researcher in a masterdegree
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/abiusx/WP-SQLI-LAB/commit/d0b2668e3c94e6fc4761b277f7ed4ba05fdb43fa#commitcomment-14695409>.
>


"Invalid characters defined for field name %s"


The id here was not changed. It needs to be ```id=pretty-print-registries``` instead.


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


Hi, Segment Team!
I would catch events "page", with emitters but, the page event is not catched.
We can't catched event with `this.on('initialize',` but with `ready` the event is catched.
Why not replaces "on" through "ready"
```
this.ready(function(){
  self._parseQuery(window.location.search);
});
```
It is not reliable?


OpenGL tracks a per-thread current context in thread local storage (did I mention OpenGL is a horrible API) so tracking a global context like this doesn't match GL's per-thread model.

I think you can remove the gputop_current_wctx, and in gputop-server.c, you can get ahold of a context something like:

```c
pthread_rwlock_rdlock(&gputop_gl_lock);
if (gputop_gl_contexts->len) {
    struct winsys_context **contexts = gputop_gl_contexts->data;
    struct winsys_context *wctx = contexts[0];

    //Optionally print a warning if gputop_gl_contexts->len > 1, since we'll ignore those to start with and assume the simpler/common case.

    gputop_list_for_each(query, &wctx->queries, link) {
        // fill in feature description
    }
}
pthread_rwlock_unlock(&gputop_gl_lock);
```


does length compute sqrt? suggestion : lengthSquared


why all upper? I do not believe we use all upper for class names, even if they are final.


Makes sense


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


If this is possible, I prefer
("The field type was never set for the DDM form field with the field name " + fieldName)


hmmm


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


I think there is a small typo here: s/NOOL/BOOL/ (?)


With some complex tables, the ```tableElem.rows``` property would return an empty ```<HtmlCollection>``` in Internet Explorer. Gathering the first ```<tr>``` skirts the issue. No issues in Chrome, FF, IE.


"Invalid available locales set for the property \"" +
propertyName + "\" of field name " + fieldName);


Please correct indentation throughout.


insert the-- "...the property..."


저런꿈틀자결하시오/


so now which script should i run first of all i downloaded the whole folder then i tried to open the wordpress folder site which told me that i have a DB error " Error establishing a database connection " i noted that the DB is "wp38-4" which i couldnt find anywhere so is there anything iam missing


Please either decide to update Overview.html with each change, or a separate change.

Personally, I'd prefer if Overview.html was re-generated each commit.


I think you may be able to drop these. I think the first is a left-over from an earlier version and you're now getting to the list of queries in a context object within gputop-server.c

The comment above covers gputop_current_wctx.

for gputop_gl_available_n_queries, it may be fine to use `gputop_list_length(&wctx->queries);` in gputop-server.c which shouldn't be performance critical during initialization.


Indentation/wrapping is wrong.


It's not. This commit was just to get Travis to pass, and I was planning to do what you said later tonight.


Is this correct? Did the CFRG formally adopt this? I was under the impression it was still an I-D


add ifctrl-osx.o here seems causing some problem for later ".objdeps.mk: $(OBJS:%.o=%.c)", the make seems stopped because to find ifctrl-osx.c


Invalid characters were used in the field name?

"Invalid characters entered for field name " +


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


Правильней будет сдалть наследование


I am deleting blue


I like "must be set" instead of "should be set"


Лучше разносить каждый класс по отдельному файлу


I'm not sure why you made these changes. They seem to be a step backwards in maintainability - that is, it's easier to remember JWK rather than the assigned RFC number.

I would prefer not to make these changes.


Add a comment explaining why this is needed.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7305](https://teamcity.mesosphere.io/viewLog.html?buildId=82885&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:07


Можно было сдлеать так, что название класса собираеться динамически


Every MSE-enabled browser already supports more than one SourceBuffer per MediaSource.  Audio and video are already and have always been in separate SourceBuffers in Shaka.  What isn't supported by browsers in general is more than one audio or more than one video SourceBuffer.


Needs a ':'


Not valid in Python


These changes should arguably be in a separate commit and w/ their own justification. I'm not a fan that you combined JWK renaming with removing a bunch of (informative) references.


"Invalid column size, column size must be positive and "


Should there be list of processes?
scsynth.exe
ruby interpreter (CUI)  (possibly)


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


Nice catch


Сейчас появился hard code. Смысл в том, что мы делаем универльный интерфейс
`Parser.new(file_name, extname).data.each ....`


Define it where it's first assigned: `var zoom = zoomSteps.ElementAt(nextIndex);`.


This is for issue #3 @hubyeti


"The default locale for the DDM form layout's page title is not the same " + "as the DDM form layout's default locale"


Можно сделать hash из:
`json: JsonParser`


Hi Hadrian,

Could you please correct isDigit method, ASCII 57 (number 9) should be included.
It causes issues during leader election. Sequence 0009 is considered smaller than 0008, which is wrong.

Thanks in advance,
Denys


-1 to this change. This isn't entirely accurate going into PR now, is it?

I'd also like to make a distinction here - editorial notes *should* be kept in the editor's draft - I don't think merging into the text is the 'right' way to resolve the need for the final documents to be editorial note free.


Not sure I understand this one. When is this thrown?


У нас сейчас есть сильная зависимость класса парсера к полям в БД
Лучше вставлять сразу все поля и сделать обработку ошибок


-1; this requires better treatment than what is proposed, and is certainly not normative.


How about a test for this? @ppaeps @captn3m0


Fixed.


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


-1

This was intentional (and a matter of great contention in W3C). Let's not fight this on GitHub


Сложный метод, можно выделить несколько более мелких


-1

Including this text normatively is inappropriate.


You should change it to https://addons.mozilla.org/firefox/addon/twitch-now/ so it doesn't default AMO's language to Russian.


Should only need to take a read lock here (`pthread_rwlock_rdlock(&gputop_gl_lock);`)

The length should be read with this lock held + maybe use `gputop_list_length()`


Можно использывать `attr_accessor`


Wrong update, direct javascript modification. Please ignore.


Test comment


-1

This is inappropriate to add to a normative section.


Код вышел сложный, есть идеи как упросить?


Unnecessary to be a separate commit


тут все отлично


Nice profile page m5


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


I want to change the colors so they correspond to editor logos,
and hybrid is in the middle of the two,
Insert state color changed to resolve the conflict,
and proposed hollow for visual mode (suits every theme imo).

![emacsicon](https://cloud.githubusercontent.com/assets/15216366/11487256/19365b18-97be-11e5-970d-34b0b89baf9b.png)


Классы лучше вынести в отдельные файлы
1 класс на 1 файл


We haven't defined a workmode, but I'm not a big fan of merging merge commits :( It'd be great to rebase so we can avoid the merge.


Correct update on coffee source code + test.


every deprecated method should have a javadoc explaining what to use instead.


Seems inappropriate to make these changes


They are actually both BACK in the same repo now, so you could share again :-)


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


I think `.begin()` and `.end()` are more common for this, no?


@jozic I really like your approach!!!!
![image](https://cloud.githubusercontent.com/assets/2567525/11487303/e4723c7e-979c-11e5-8419-48f3768ab770.png)

LGTM @ivantopo WDYT?


likewise here


The `However` below is contrasting something that agrees with its point. I would rewrite the above and below:

>[...] can’t always do without.
>
>For a creation-focused app like minutes.io, where there is no sharing of data between users, networking is largely optional. Depending on the feature set and scope, there’s a whole lot of useful measures you can implement to make sure that your users and customers have a good experience with your app, whether they’re online or not.


Looks good! Is this for performance, readability or bugfixes?


Not sure, I assume it's an anti-corruption tactic or something like that.


I haven't had the time yet. The only part remaining was getting the last filter to render directly to the OutputRenderTarget right? That shouldn't be too difficult, but it's also not that important, maybe we should just log it as a "todo", for now?

Something rather more important would be to get the AudioChains to use Filters. That might be a bit problematic since the main processing is done by this "bool Process(Audio input, Audio output)" method, since "output" needs to be allocated beforehand. It's not really a good fit for this "IFilter Process(IFilter input)" method.

It is, however, a good fit for the "Render" function of IFilter class. So maybe it would be a more natural fit to make the AudioChains into Filters, since they seem to operate on a somewhat lower level. Then you just need a wrapper (like StaticChain) to make them into a RenderChain, one difficulty with this is that they can only be used as type parameters if they have a parameterless constructor, which could be problematic.


It's minor but would you mind renaming this to proxy? I do think proxy_url is a better name but I want it to be consistent with how api_endpoint is named. Otherwise, I'm happy to merge this.


It would be safe to use `gputop_list_for_each()` here since you aren't modifying the list.


10 seconds seems kinda long; was someone holding a lock for 10 seconds during a test or something?


По тестам все отлично
Нужно еще покрыть сам парсер


registration


is that the new version that fixes `Math.random()`?


Yep! :)


a valid ACL factory


During these changes you seem to have changed the >= to a >
was this intentional?


No idea, this is a missing 0  (a typo) in the actual V8 library I'm using from brew.


The semi colon here terminated the statement early:
https://github.com/lairdesign/live.byba.co.uk/commit/8881b0f360ffd46d492227d4ee7b98981c96577b#diff-8a99459567830f770f1a7fdeb5bc1ea1L1008

It was at the end to show the end of the statement but its not actually required, as long as there's a line break javascipt is ok:
https://www.codecademy.com/forum_questions/507f6dd09266b70200000d7e#answer-508000efde2d860200004bbe

It is a good idea to use them though and I do a lot. :)


Could this be removed?


Danke!


Hopefully this addresses #4.


Feel free to open a PR


I will start by bumping the version requirement for cmake to 2.8.12.


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


adds HP


In my opinion we should now get rid of the tricky
    Time.elapsed() > Time.available()
condition, I'd prefer if we reduced Time.available accordingly when we are going to play an easy move and had a variabile that tracks whether an easy move has been played last time! Who cares if that means three more lines of code...


no, it's more like trying to perform admin commands on the database while the instance is running (operations like vacuum)


Mind changing the indention to meet the Ruby style guide approach? https://github.com/bbatsov/ruby-style-guide#indent-when-to-case


Please add a comment or two at the top, perhaps explaining why this unit test must not fail (required by other modules?)


Verify that this is the expected error message:

```Bash
$ python ut_repyv2api_netstat_check.py
Traceback (most recent call last):
  File "ut_repyv2api_netstat_check.py", line 9, in <module>
    raise Exception('Netstat failed internally ' + str(e))
Exception: Netstat failed internally [Errno 2] No such file or directory
```


Is the 'setter' or `System` your primary concern?

I think we need some sort of setter, the value needs to be transferred from the static text in the file to the runtime.

I think we should remove our bootstrap `System` in favor of `$traceurRuntime`.  Let it have the non-standard loader and version stuff.  Esp. since we don't even know if `System` will be.


If you leave this TODO here - when are you going to address it?


I think this needs to be switched back, see https://github.com/geoblacklight/geoblacklight/commit/ffc09b36946d36211a8057f914a5489a80a6a4b5


for double checking programmer errors, maybe slightly better to use `assert(q != NULL);` instead of quietly adding a `NULL` ptr to the array of queries and crashing later.

in other projects, I'm used to having separate macros like `c_return_val_if_fail(q == NULL, NULL);` for this kind of check, but haven't added anything like that so far to gputop.


add log Exception


r+


Mind removing the whitespace changes?


don't care


Hi Michael,

David Madl implemented most of this change, so he may {want|have} to chime
in on the details, especially if I'm misrepresenting things here. We are
currently working towards reducing the dependency of Moses components on
static data, in hopes that Moses components can eventually be instantiated
even outside of the Moses Monolith.

In essence, this change only moves the specification "OOV feature enabled"
from the global space (via moses.ini) into the local space (via the
configuration line (oov-feature=1) within moses.ini, which can be supplied
to the constructor even when StaticData is not available globally). So if
you have any legacy moses.ini files that do have

[lmodel-enable-oov]
1

you'll have to rewrite them into

WhateverLM ... oov-feature=1 ...

The abstract LM class LanguageModel sets a boolean member
m_enableOOVFeature during construction and sets the feature count to 2
(instead of 1) if m_enableOOVFeature is true. The (virtual) default
implementation of LanguageModel::EvaluateInIsolation(...) provides the OOV
feature and counts OOVs.

LM sub-class implementations that override this virtual function must take
responsibility for this themselves. This is unchanged from the way things
were before. According to 'find moses/LM -name '*.h' | xargs grep
EvaluateInIsolation', none of the current implementations in the core Moses
code repository actually do override EvaluateInIsolation().

LDHT (whatever that is, it seems to be related to RANDLM, which in my
opinion is primarily of historical interest) will need some fixing up (it
resets m_enableOOVFeature to false but doesn't update the
m_numScoreComponents), but I'm inclined not to cross that bridge before I
get there.

Feature functions are now registered in the respective FF factory after
they have been constructed (instead of FeatureFunction::FeatureFunction()),
because the number of feature values a FF provides is now unknown until the
FF has been constructed. That's a purely technical change (with some
benefits for future modularization).

So bottom line, except for the need to enable oov-feature=1 individually on
each LM feature in moses.ini (instead of the [lmodel-enable-oov]
specification), nothing should have changed from the user perspective.

I hope I got it right and I hope this helps

 - Uli

On Mon, Nov 30, 2015 at 9:25 PM, Michael Denkowski <notifications@github.com
> wrote:

> Does this mean there's a generally usable LM OOV feature or do individual
> LMs have to implement it? Does the KenLM interface support it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14693766>
> .
>

--
Ulrich Germann
Senior Researcher
School of Informatics
University of Edinburgh


After performing your npm and bower install, you must run the following command for the json-editor to work:
bower install angular-json-editor --save.


Fixed, thanks.


Its done..!


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1436 of 1436 tests passed)

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


@journeyman9 You should use actual fuel consumption here.


This probably isn't good to leave free


Gute Frage... Ohne Bindestrich würde es mir auch besser gefallen.
Umhüllung verformen klingt für mich nicht so gut.


Technically speaking we don't have to, but it's a good practice to keep the major & maybe minor the same :dart:


Ok, got it. Thanks.


@alexcjohnson anything else?


This now changes when telemetry happens. Prior to now, if someone `require`'d taco-cli and invoked runWithArgs (which invokes runWithParsedArgs) then it would not produce any telemetry for taco-specific commands. By moving the telemetry into here it now will. Is that intentional?

Looking closer, since it won't run the telemetry.init code (unless the caller does so) it won't actually send any telemetry and this is fine.


Hello mr Abbas i already emailed you thx for your response :)


Well, I know that we don't need to, that's why we have 1.2.0 and 1.2.1. You always have a choice :wink:


It seems like you took care of the comment here on line #55.


Fixed in https://github.com/JuliaOpt/SCS.jl/commit/89a41ddd56d2bbfc827bfeef40dd0f3bbc04fcc5. We can update METADATA if it becomes an issue.


@alexcjohnson was it intentional that `newPlot` didn't return a promise? Seems like a bug to me.


saw approval from graydon
at https://github.com/MonsieurNicolas/stellar-core/commit/9b7aa0cead7c367240f8282c243bf7830b3e7d58


merging MonsieurNicolas/stellar-core/scpHistory = 9b7aa0ce into auto


MonsieurNicolas/stellar-core/scpHistory = 9b7aa0ce merged ok, testing candidate = a1a17a91


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


I should mention this doesn't actually work. It copies the list in reverse. But since I can't get the program to compile, there's not a whole lot of reason to actually write something that would work.


I hit an error here.  needs to be command.extend([...])?

~~~~
Traceback (most recent call last):
  File "./cropgtk.py", line 330, in <module>
    app.run()
  File "./cropgtk.py", line 284, in run
    command.extend('-copy', 'all', '-crop', cropspec,'-outfile', target, image_name)
TypeError: extend() takes exactly one argument (7 given)
~~~~


I think that the error handling here still needs improvement. We want to distinguish different cases
- a partition key is given that is not even in the working set. That is an illegal argument
- a partition key is given that has expired. That is an illegal state, or an illegal argument, as you prefer.
- a partition key that was never in-progress. That is also an illegal argument.

The error message should clearly indicate, which of the three cases apply.


/cc @chrisroos


Diese Änderungen habe ich alle irgendwann Anfang November gemacht, und jetzt endlich die Zeit gefunden, das  zusammenzuführen - ich kann mich leider nicht mehr so genau erinnern, ob der Modus da schon so war oder nicht.
Außerdem habe ich die 0.91-Version mit dem trunk abgeglichen. Vielleicht war das hier auch bloß eine 'Übernahme' aus der Altversion... War keine Absicht!


I guess you want it to break if the vault item doesn't exist.


Forgot to note this tweak came from Brad Spengler, excuser moi.


Can you please change the other occurrences in the file as well?

The past tense is required here because it's probably a case where the pronoun is dropped. I.e. read the sentence more like this: "[I] expected Init to fail, but got ..."


This doesn't make sense where there are nested gold annotations. As I've said before. I also don't think that calling this "weak" is sufficiently descriptive and it should be renamed to specify character level.

But at least you've given me a file to rehome my fix-spans technology real soon now.


Ach Mist, das war auch neu?...


Regarding nested annotations, this is true of TAC, and certainly of TAC outputs. This tool should produce an error/warning where such duplicate annotations are produced by the conversion.


@bpostlethwaite is there better way to handle 404?


(And hence the advantage of pull requests even on small projects)


Erinnert an deformiert... Stimmt. Gargoyles und der Glöckner von Notre Dame...


Seems a little convoluted.

What's ok:
number + number
character + anything
string + anything

Example error message:
The operator + is undefined for the types: char, boolean


xdr_get only has a constructor that takes the addresses of the begin,end (excluded).
So I would have to do something like `xdr::xdr_get g1(&*envBytes.begin(), &*envBytes.end())` but `*envBytes.end()` is an illegal operation (read past last element), so the code then has to be written as `xdr::xdr_get g1(&*body.begin(), &*(body.end()-1)+1)` which is worst.


Looks like Gurobi was left out?


Danke!


it disappears without a trace? Not even a log message? Or better, add the removed partitions to the consumer result.


Remade the reflection method inside the player class, since it was producing an error.


change path to /var/log/


I was imagining that we would just have a generated file (called version.js) that exported the version:

```
export default %VERSION%
```

where we replace version in a make rule.


Will tag in a second wave
On Nov 30, 2015 6:19 PM, "Iain Dunning" <notifications@github.com> wrote:

> Looks like Gurobi was left out?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaLang/METADATA.jl/commit/2e73754fcea4afb113ee496915a689f6b67401d1#commitcomment-14696161>
> .
>


I'm not a fan of introducing this code and then changing it in the same pull request.


Great job again!


The first case appropriately throws an IllegalArgumentException:
```
throw new IllegalArgumentException("PartitionKey not found: " + partitionKey);
```
The second case:
```
throw new IllegalStateException(String.format("Attempting to operate on an expired partition: %s",
                                                    consumablePartition.getPartitionKey()));
```
The third case:
```
throw new IllegalStateException(String.format("Partition not in progress: %s",
                                                    consumablePartition.getPartitionKey()));
```


Personally, I would prefer the spelling "TableACL". In any case, we should agree on one version and use this consistently throughout the file.

Other variations which I saw were:
- "tableAcl"
- "Table ACL"

Can you please make all of these occurrences consistent in a spelling of your choice?


these have no effect here.


You don't set the Image to anything?


Nothing I can think of - its good that we have some useful error text here. Though do we want to handle other errors as well? What if the user is sandboxed and has no access to the network?

Also should we use the Plotly notifier as well as write to the console?


OMG! :O


(3D ist auch irgendwie redundant...)


I still need it in develop


Each course has (hopefully) only one book. And each role is only in 1 course. So by the role in that case.


Done


Resume


I liked the informality, but:
`Init` is called, and then `Poll` in a loop. When the TTL expires, `Poll` terminates, and `Cleanup` is called. Each iteration of this cycle generates an event and sends it to the Collector.


we really probably should have had `-ci` in this name. makes it clearer that this is using CI-latest k8s and not the latest stable release (like most other gke jobs).


`alt` isn't the same thing as `title`. Did you mean to remove the attribute when you removed the wrapping link tag?


Removed it for now.
Reason: It's an additional complexity for the user (maintaining order), for minor performance improvement.


The `NewRoutine` function [takes x, y, z where x/y/z may be args?] and returns a [pointer to?] a new khealth Routine.


Isn't a 404 "not found"?  Maybe a 409 or a 412 would be better?


What do you think of renaming this build event to "START_BUILDING"? Just "START" isn't clear (if someone hasn't read the comments on this class and is just using auto-complete in IntelliJ) whether or not it's starting the entire build (from before build preparation) or starting the running of the meat of the build itself.

"START_BUILDING" is also slightly misleading, because the whole thing is a build, but I feel it's more specific than "START".


Controlling greenroom is doable, but will take further work, and I ain't specifically interested in this. Maybe in the future.


@lambrojos good catch! Clearly a leftover from word by word translation.


lgtm


:+1: it was not intentional.


Hey @ScottDowne, it makes great sense, good work :+1:


s/should/must/ && s/routines/Routines/g && s/collector/Collector/

"To wire Routines to a Collector implementation, follow this general pattern:" maybe cleaner phrasing?


Curious why this color is replaced but in other places you've added `-2` variants.


Thanks for the explanation!  It looks like KenLM doesn't support
oov-feature=1, though it does track the number of OOVs in part of the
code.  I may look at implementing this when I have some time.

Best,
Michael

On Mon, Nov 30, 2015 at 6:11 PM, Ulrich Germann <notifications@github.com>
wrote:

> Hi Michael,
>
> David Madl implemented most of this change, so he may {want|have} to chime
> in on the details, especially if I'm misrepresenting things here. We are
> currently working towards reducing the dependency of Moses components on
> static data, in hopes that Moses components can eventually be instantiated
> even outside of the Moses Monolith.
>
> In essence, this change only moves the specification "OOV feature enabled"
> from the global space (via moses.ini) into the local space (via the
> configuration line (oov-feature=1) within moses.ini, which can be supplied
> to the constructor even when StaticData is not available globally). So if
> you have any legacy moses.ini files that do have
>
> [lmodel-enable-oov]
> 1
>
> you'll have to rewrite them into
>
> WhateverLM ... oov-feature=1 ...
>
> The abstract LM class LanguageModel sets a boolean member
> m_enableOOVFeature during construction and sets the feature count to 2
> (instead of 1) if m_enableOOVFeature is true. The (virtual) default
> implementation of LanguageModel::EvaluateInIsolation(...) provides the OOV
> feature and counts OOVs.
>
> LM sub-class implementations that override this virtual function must take
> responsibility for this themselves. This is unchanged from the way things
> were before. According to 'find moses/LM -name '*.h' | xargs grep
> EvaluateInIsolation', none of the current implementations in the core Moses
> code repository actually do override EvaluateInIsolation().
>
> LDHT (whatever that is, it seems to be related to RANDLM, which in my
> opinion is primarily of historical interest) will need some fixing up (it
> resets m_enableOOVFeature to false but doesn't update the
> m_numScoreComponents), but I'm inclined not to cross that bridge before I
> get there.
>
> Feature functions are now registered in the respective FF factory after
> they have been constructed (instead of FeatureFunction::FeatureFunction()),
> because the number of feature values a FF provides is now unknown until the
> FF has been constructed. That's a purely technical change (with some
> benefits for future modularization).
>
> So bottom line, except for the need to enable oov-feature=1 individually on
> each LM feature in moses.ini (instead of the [lmodel-enable-oov]
> specification), nothing should have changed from the user perspective.
>
> I hope I got it right and I hope this helps
>
> - Uli
>
>
> On Mon, Nov 30, 2015 at 9:25 PM, Michael Denkowski <
> notifications@github.com
> > wrote:
>
> > Does this mean there's a generally usable LM OOV feature or do individual
> > LMs have to implement it? Does the KenLM interface support it?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14693766
> >
> > .
> >
>
>
>
> --
> Ulrich Germann
> Senior Researcher
> School of Informatics
> University of Edinburgh
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14695969>
> .
>


s/compatbile/compatible/


don't think so... only geo and mathjax have any async AFAIK


This must be a formatting error, right?


This patch seems wrong to me: 'i', 'nchar' and 'k' are all signed integers; the only risk of overflow is in 'k+10', and your patch does not address this issue. Introducing an unsigned loop index just creates signed/unsigned clashes here.


@sleevi because you don't think we're yet ready for PR?


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


Ugh. Hopefully this won't change again.


Collectors expose Prometheus-compatible status endpoints and metrics, providing readymade infrastructure to aggregate statistics from a set of canary pods, designed specifically to exercise Kubernetes cluster resources.


I added a `LOG.warn` for partitions that are dropped/discarded.


this should be left `this.regions =  this.regions || {};`


/cc @chrisroos


It seems like we should init `n_gl_queries` while we have the `gputop_gl_lock` where we iterate the queries.

maybe this api could take a &features pointer to fill in both `gl_queries` and `n_gl_queries` ?

I also get that this is little pedantic. as it stands we know `gputop_gl_available_n_queries` is a global initialized once, and we've documented we're assuming all contexts have the same number of queries, which should be fine.

With some of my other comments about using `gputop_list_length(&wctx->queries);` I suppose that would reduce where we make that assumption, in case we ever want to handle corner cases where it's not true in the future + avoids a global.

I'm not sure a.t.m if I have a strong opinion about this, so hopefully you can make something from my rambling :-)


Formatting error


s/Use/Use the/


This is actually necessary. I'm merging this all in now however, so I'll put it back in place.


what can we do to get this to do The Right Thing (tm) upstream?


@colm2 Feel free to add anything you think appropriate


Kubernetes (whereas "routines", being used in the more general sense, is ok no capitalized here)


I think maybe this should take a `&features` pointer so it can reference `features->n_gl_queries` instead of referring to `gputop_gl_available_n_queries`


"package" is repeated. Delete one.


I'll be creating a PR soon for an update


I don't think this "false" does anything, does it?


Is there a reason for why you are using a fixed value vs. the commented out tick_handler time?


Oops, accidentally committed a test value. Will fix.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7316](https://teamcity.mesosphere.io/viewLog.html?buildId=82917&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:43


I think this api will just need a GL query ID instead of perf_metric_set, period_exponent, overwrite + aggregation_period. The GL query ID will be one of the gl_query->id values forwarded as part of the features.


Ah, that's fine.


Hahaha.


**`cmd/rcscheduler`** (missing `c`, use monospace)


I liked all the changes you have made but this, I had something similar to this in my first attempt, but then I decided to use a datagrid to keep it more organized, similar to a table.

![image](https://cloud.githubusercontent.com/assets/10853349/11488068/56af3c60-9789-11e5-8759-ece12227c937.png)


**`pkg/collectors`** *(monospace literal string of relative file path)


This patch introduces a signed/unsigned clash in the compare. 'i' and 'nlist' are int, and the counter is stepped by increments of one. If the compiler croaks here, you need a new compiler. Sorry.


**`cmd/`** (mono font for file path)


I did report it a while ago and it was assigned for a while and then went unassigned again.  https://bugs.launchpad.net/python-swiftclient/+bug/1511813


Well, yes, that may be so, but it quiets gcc's -Wstrict-overflow warnings.
If there's ever going to be a hope of compiling with -Werror,
either the warnings need to be quieted or the -Wstrict-overflow flag
needs to be disabled, either globally or for this file.
I'll concede that this one came from me personally...


I thought this was supposed to be healthd_board_mode_charger_set_backlight ?


**`pkg/routines/`**


Asserts won't work in a shipping version of the app. What if you had a static array of permitted run loop modes and checked for membership in that array?

```
NSString *runLoopMode = _runLoopMode;
if (![@[NSDefaultRunLoopMode, NSRunLoopCommonModes] containsObject:runLoopMode]) {
    // pick default run loop made based on processor count
}
return runLoopMode;
```


Tinha um motivo para essa função retornar 1 caso o movimento fosse inválido. Tá comentado no começo da função.


User cases:
    - EMI user: enters Assembly_number, revision, and format (creation of format & RegEx)
    - Operating user : enters sale_order, scans serials_number


alguém mais quer opinar ?


VHDL não é sensitive case


I think the language should be `erb` here.


@colhom Noticed some more spots and tried to polish them.


The cast is benign (because the 'rp->datalen' is known to be strictly positive here) but that's OK for me.


Again, it ought to be a goal to be able to build with -Werror.
Either the warnings under -Wstrict-overflow needs to be dealt with, one way or the other,
or the warning flag needs to be disabled, either globally or for this file.


Ah yeah, I totally agree. I'll try to think of a better name. Some other ideas I just had are `START_SUBJOBS` or `START_COMMANDS` (to reference the "commands" section in the yaml).


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


Hmm I can't remember what we were going to do with AudioScript's Initialize and Reset methods. Any changes we make we'll still need to account the things they do - i.e. load audio kernel and disposing unmanaged resources.

"Output" is no different to ITextureOutput right? I think it doesn't need to be allocated beforehand, at least not in the way it is currently implemented. It can be allocated in the same way as how textures are allocated. When "Process" is called, IFilter's Output would already allocated would it? The way I see it, it is no different than render script's textures?


Sure. that could work.
We could add an exception too. It is expecting a dynamic value anyway.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/94053702


@lassoan could you take a look at this and see if you've updated your version with these fixes yet?


@wseltzer No, because the changes were made to the .html and not the source XML, so this is more or less a no-op, as the next time we re-generate the HTML, it will be readded.


Wow you guys seem to be having fun! Hello from CSULB!


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


The point is that the result of sizeof() is a size_t, and rp->datalen is signed, so
without the cast you would get a warning for signed/unsigned comparison.
But, yes, the cast is benign, but it also gets rid of a warning when compiled with
the -Wsign-compare warning flag.


Do you recall why we made this asynchronous? Because the client can't get the build results until the artifacts have completed being created and aggregated, which I believe the async_postbuild_tasks does.

As you know, it'd be so much prettier if we could just move the contents of that function inside here and trigger the FSM event directly.


Why the MySQLVersion is a required property? It should be optional . We should default it to 5.6 so that the new change in the resource does not break existing configuration scripts or composite resources (such as the ones used in WordPress)


This is the best change to make. USE IT!!


David! You're alive!


We should default the version to 5.6 so that the new change in the resource does not break existing configuration scripts or composite resources that depend on this resources (such as resources used in WordPress)


Thanks for this Joshua, it all seems to make sense. One thing before I merge, do you have an example of it working?


Seems to be a missing `;`.


Weird, but without this I see lots of errors like the following when trying to materialize the relationship :
```
ex(75)> Repo.all( from p in Product,  join: category in assoc(p, :category) )
** (UndefinedFunctionError) undefined function: Category.__schema__/1 (module Category is not ava
ilable)
    (ecto) lib/ecto/query/planner.ex:124: Ecto.Query.Planner.prepare/3
    (ecto) lib/ecto/query/planner.ex:66: Ecto.Query.Planner.query/4
    (ecto) lib/ecto/repo/queryable.ex:91: Ecto.Repo.Queryable.execute/5
    (ecto) lib/ecto/repo/queryable.ex:15: Ecto.Repo.Queryable.all/4
```


Good catch.


103643052-FiniteDifferenceFluxOutput (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1962 of 1972 tests passed)

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


Technically "Process" doesn't allocate anything (usually), the filter's "Render" method does.

The more I look at it the more I'm convinced it would be easier to make the current "AudioChains" into Filters. Moving the Initialize and Reset methods to the Filter would also fit.


C'est ne pas intraduisible :monkey:


MantisBT plugin overview:
https://www.mantisbt.org/wiki/doku.php/mantisbt:plugins_overview


Mmh. Nach Anleitung funktioniert es bei mir. Über den Sinn bin ich mir nicht ganz im klaren...

Ich konnte damit gleichzeitig mehrere rechteckige Bitmaps exportieren, und die wurden automatisch zu einer HTML-Seite zusammengesetzt, wo sie zueinander in gleicher Entfernung stehen wie im SVG (stehen dann natürlich auch als Dateien einzeln zur Verfügung).
Wann braucht man das als Webdesigner?... (die HTML-Datei hat zumindest den Titel 'Web Layout Testing'...) In Inkscape kann man das doch auch schon hin- und herschieben...
(für den 'Mehrfach-Bitmap-Export' finde ich das allerdings ganz praktisch... Die html- + css-Dateien kann man ja wegschmeißen...)

Web-Guillotine? Wer die Guillotine kennt, kann damit dann vielleicht was anfangen... im Analogieschluss. (Internet-Hackebeil wär' doch auch nett ;) )


@sleevi Happy to redo changes to XML so you can regenerate properly (apologies, though you were using Respec.js), with each change in a separate pull request. Or if you want, you can simply commit the changes that you believe are correct to get through PR (see pull request comment)  just ping me when you are happy with doc. Any preference?

Regardless, I can do the staging at W3.org/TR for Proposed Recommendation from the generated HTML - but we need the substance of the pull request, if not the precise changes, addressed.


fast-forwarding master to auto = a1a17a91


Why is value state being duplicated in each form field instead of being passed down?

This will make it awkward should you ever need to update a field's value from outside itself.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 71](https://teamcity.labkey.org/viewLog.html?buildId=288393&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 17 (3 new), passed: 30 Build time: 01:15:05
### Failed tests
```
org.labkey.test.tests.CDSTest.testLearnAboutStudiesSearch: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

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

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

##### there are 7 more failed tests, see build details
```


And there was another typo (too late at night typing) in the motorcycle stuff.


Move ``buffer.containsKey(..)`` check here so that we don't have to worry about NPE if we reuse this method.


Hmm... You mean against a possible memory leak?
Don't think so.
It could be that it is there to make sure that the usermap can forget names. E.g. if you want to clear the usermap file and do a reload of Essentials. It could be a use-case.


Since we're now +36sec to TAI and earth will probably not speed up again, this is ok. The proper solution would be to make either 'sys_tai' signed or leapsec table entries unsigned. I don't expect trouble here, so we might as well cast.


What is a reenter state? Also, I don't know what triggers the FSM to get this error event. Are we assuming that the subjob queue, slave teardown (and such) has properly been taken care of at this point?


From Spring documentation Logback - logback-spring.xml, logback-spring.groovy, logback.xml or logback.groovy
as I see logback not supports .properties configuration, but we can specify some properties for logging in spring application properties file.
https://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-logging.html


Why you reverted it ? It didn't worked ?


Honesty due to how apps are often paired with specific versions of MySQL it makes more sense to require that the user specifically tell the resource what version of MySQL they intend on using instead of making the assumption that a certain version is popular, and if other dependent resources make the assumption that this module assumes X version is popular they need to stop that and fix it to be perfectly frank, its bad practice.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7322](https://teamcity.mesosphere.io/viewLog.html?buildId=82926&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:08


Would we still build the filter chain in Update() though?

The other thing to keep in mind is, Update() only gets called when input format changes - i.e. when you open a new media file. It doesn't get called when you change something in Script Chain for example.


Here, instead of using ```len(onsets)``` which is 85 or 86, I think we should use 240 which is the number of TRs.


@insin thats fair -- this is incomplete and theres a good chance we'll need to change that yet


Forgot to add to commit log... Added ability to read archived MTL files.


Again a benign cast for a compiler that does not do good analysis.

The string formatting can be done this way, but I want to point out that the original code used static constant strings. If a compiler warns about that it's analysis is incomplete.


@LuigiBlood Alright, thanks LuigiBlood! Your work is always exciting. :+1:


PS. instead of "cendari:nerd"  CendariProperies.NERD can be used.


Well, however we do it for the RenderScripts seems to work, so why not just do that?


in line comment


This file path looks invalid based on where you put the files in this commit.


I know this is a pre-existing bug, but the status can also be in BuildState.PREPARED.


Disculpen , me equivoque en el nombre, es el estandar 7408 el de la AND, lo necesito para construir el circuito final, por ello es hice la expansión de puertos de una vez.

https://es.wikibooks.org/wiki/Circuito_integrado_7408


The way audio script works is very different from render scripts, but let me see if it's possible to make it mimic its counterpart's behaviour.


@MarcoFalke Yes, it is cleaner to merge a fix when possible. (In fact, it would be nice if all bugfix commits were based directly on the commit introducing the bug...)


Thank you @kellijohnson !


I think the logic that we wanted to be async was creating the artifact archive file. We didn't want to hold API requests open while tarring up something huge.

We could still make that cleaner probably though by using Tornado's async patterns.


Moved comments here to the right place at https://github.com/coreos/khealth/pull/12/files


Hey David,
Seems fix not in a right place


Thanks for this @shyam334
I will review and get it in soon.


Ild suggest removing the stored jar in your cache, this has happened to me
before but not enough to be a problem.

On 30 November 2015 at 05:35, Yuri Z. <notifications@github.com> wrote:

> I get this error when trying to run it:
> gradle buildServerJar
> Incremental java compilation is an incubating feature.
>
> FAILURE: Build failed with an exception.
>
>    - What went wrong: Task 'buildServerJar' not found in root project
>    'wave'.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wisebaldone/incubator-wave/commit/7d8609e77cf5b623bc056efa94f27adaaa57e215#commitcomment-14670105>
> .
>


This should cause the exception:

```
using System;
using System.Collections;
using System.Collections.Immutable;
using System.Linq;
using System.Reflection;
using System.Reflection.Emit;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            var assembly = AppDomain.CurrentDomain.DefineDynamicAssembly(new AssemblyName("dynamic_assembly"), AssemblyBuilderAccess.RunAndCollect);
            var module = assembly.DefineDynamicModule("dynamic");
            var typeBuilder = module.DefineType("Dummy");
            typeBuilder.DefineDefaultConstructor(MethodAttributes.Public);
            var dummType = typeBuilder.CreateType();
            var createMethod = typeof(ImmutableList).GetMethods().Where(m => m.Name == nameof(ImmutableList.Create) && m.GetParameters().Length == 0).Single().MakeGenericMethod(dummType);
            var list = (IEnumerable)createMethod.Invoke(null, null);
            var addMethod = list.GetType().GetMethod("Add");
            list = (IEnumerable)addMethod.Invoke(list, new object[] { Activator.CreateInstance(dummType) });
            // Will crash when invoking the enumerator
            foreach (var item in list)
                Console.WriteLine(item);
        }
    }
}

```


Great, I'm glad we're all in agreement here.


6.2.2 vs 6.3.1 (note the white box in 6.2.2)


This broke a lot of games.  :disappointed:


Let's remove the dep (make it a dev dep for the example) and bump the version to `0.2.0`


Doh! Wrong repo. These are just meaningless changes that Xcode made.


should make this private. if we ever wanna push a detail view controller, all we would need to set is the school and then simply push it. so its better to put the network request in viewDidLoad or something, so we dont have to manually call it. because from another person's perspective, all the work theyd need to do is pass the school to the VC and expect it to automatically handle the proper displaying and stuff


I've discovered today that in combination with some of the css this broke explorer and possibly safari iOS.

I'll be revising this tonight.


Also improved statistics, added possibility of sorting.


So anyone using this in a project will also need to include the styles package (if they want nice looking forms)?


Hmm... Guess not. You want me to PR the reversion?

It's really a bummer that there's not a great way to see what happened in a failed login. Do you have any tricks for digging into login issues?


Ah no, it's fine with these mods. I flashed the wrong branch, sorry!  :P

I'm just wondering about the 60ms thing. I still think we need to reconsider that maybe? What did you base the value on?

Regarding the 40m thing, it's been beaten out pretty much on drones-discuss. Several people have test flown and come to the consensus that the lidar is ineffective above 40m AGL. Especially outdoors.


> You want me to PR the reversion?
No worries, I just reverted it.

> Do you have any tricks for digging into login issues?
Not off the top of my head, but I'll take a look.


You committed the Xcode project.


This should be `upstream.name`.


There are several issues:

1) We should decide the title of this file. Now the title of the document is "A review of implementation DNS over port 80/443". confirm ?  The name of the file is "draft-shane-dns-over-http-00"， confirm? because we use HTTP(s) port but not require only "over HTTP", right ? how we name the file without confusion;
2) You are the first author and editor, OK? I found it was changed by me by mistake;
3) I will ask paul to view it before we upload it this week;


It looks like we'd want to poke at the dom here: https://github.com/simon-weber/gmusicapi/blob/7ccf62c177953c8143bb3687f1a940b8eec1577b/gmusicapi/session.py#L118.


This is too much code, just do:
```
NSString* upstreamBranchName = [upstream isKindOfClass:GCRemoteBranch.class] ? [(GCRemoteBranch*)upstream branchName] : upstreamFullName;
```
```


So falsey is exactly like truthy. Did we want this?


Needs to be tested further, algorithm seems to run very fast


Add more excersise


Write document


My recomendation is to use #my-div-hidden instead here, also I'd use fadeOut or fadeIn instead of a delay and a hide after it


Weird. And that would change based on the Travis infrastructure? I believe it, but I presume the only way to try it out is to commit something with extra logging?

Though it's possible I can replicate this on an EC2 instance.


Functions must be stored into a variable, like this

var clearTextarea = function () {
};


Sorry, still seeing Greek xlation problem:
```
/vagrant/mushroom-observer$ ruby -v
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]
/vagrant/mushroom-observer$ rake test test/controllers/translation_controller_test.rb test_edit_translation_form_post_save_greek
...
[Coveralls] Using SimpleCov's 'rails' settings.
Started with run options -n test_edit_translation_form_post_save_greek --seed 58891
...
1 tests, 7 assertions, 0 failures, 0 errors, 0 skips

/vagrant/mushroom-observer$ rake
Started with run options --seed 44752
...
 FAIL["test_edit_translation_form_post_save_greek", TranslationControllerTest, 9.842702666006517]
 test_edit_translation_form_post_save_greek#TranslationControllerTest (9.84s)
        Expected: "one"
          Actual: "uno"
        test/controllers/translation_controller_test.rb:243:in `block in test_edit_translation_form_post_save_greek'
        app/models/language_exporter.rb:51:in `alt_locales_path'
        test/general_extensions.rb:75:in `use_test_locales'
        test/controllers/translation_controller_test.rb:239:in `test_edit_translation_form_post_save_greek'
```


was for testing purposes, i was just using ../out and created a few dummy directories


I'd recommend to split this into different functions, maybe generate something as name. Just a thought


1. I'm not sure if it has to be or not... can add `sudo chown` to find out.
2. The script can be modified to change `PLEX_MEDIA_SERVER_USER=plex` in the default file `/etc/default/plexmediaserver` in order to change to run as current user (`sed -i`). I'm not sure if there would be any advantages.
3. If the user doesn't want to automatically update plex when a new release is out, yes.


> I presume the only way to try it out is to commit something with extra logging?

Yeah, unless you can replicate it (I'm not sure what exactly travis uses behind the scenes). I'm hesitant to add a bunch of logging since I don't want to accidentally log the test account's credentials.


On UDCO:
![image](https://cloud.githubusercontent.com/assets/11859381/11462979/e46108f4-96e9-11e5-8a11-d7d192895dbb.png)


Another thing, the values with zero in them wont be tuned, I would like them to be tuned as well, so please define a custom range for them. Specially in the blocked by pawn.


I'll be updating the copyright with logic to stay consistent with the current year.


:+1: after personally experiencing having some of those mixed in a long JSON string.


It pushed those changes again, there seems to be something wrong with your git tree.


(y)


Hi, sorry for the delay, this is great, thanks Nicole!

I found what seems to be a bug:

![image](https://cloud.githubusercontent.com/assets/164714/11462962/acac6b4c-96e9-11e5-9414-3b063d40d16d.png)

Appears not to accept 0 as a number...

When I set it as a string, it still seems to work, but that's because Neo's being permissive, rather than Cycli, right?

When I then run a query, with 4 query parameters, it seems to work, but I get this warning:

![image](https://cloud.githubusercontent.com/assets/164714/11463007/31a320e8-96ea-11e5-97fc-c220d212e66a.png)

The data seems to come back correctly but thought it might be worth reporting ...

Thanks!


try-with-resources?


@vuntz, `storage_available` and `cloud_available` are being used in https://github.com/crowbar/crowbar-core/blob/master/chef/cookbooks/provisioner/recipes/update_nodes.rb#L297 to tweak the autoyast profile for storage-only installs. What's the right way to check this, now that those attributes are gone?


You'll not be able to get this properties since this are into a private scope. Hashmap means that you have to put some properties as value but just putting them as:

this.title = title;
this.genre = genre;


This is OK if you use the approach that I've recently posted in the previous comment


开源的项目


aqy


did I made a mistake in using this? :O I'd really want to know :) this would mean most of my other code is broken x3


Looks good! I think you skipped the drawable section though.


@NielsCharlier does this mean we have a regression in locking until this TODO is complete?


@gaocegege 这是材料的作业....cad制图，老师让用cpp, 但是due是今天晚上.... 我环境配置完估计就已经due了.... 新坑应该还是那个forum search, 爬虫的那个。除了爬出国的论坛准备爬下github, 然后做一个更好用的搜索(现在没有拼写纠错太要命了.... 每次都是google gh里的东西)


`self.encode()` only ever returns type `bytes`. Can you explain to me why this should be necessary? If this fixes a real bug I'd like to incorporate it upstream.


remove


You won't be using the drawables in HW 3 thought so I'll mark it as complete as long as you get HW 3 done too.


@NielsCharlier as for roles, does this mean we have not locking until this is ported and the TODO is complete?


Jamtaba 2.0.7 update with vstscanner fix


LOL this is cm-12.1 not cm-13.0


ping @kyledrake


:+1:, Thanks for chasing this Mike.


Goog


@NielsCharlier some brackets might improve legibility of these nested ternary operators.


Why are all commits named initial commits?
Give better names and more frequent, modular commits.

https://sethrobertson.github.io/GitBestPractices/
http://chris.beams.io/posts/git-commit/


Hello x)
Nice idea removing the nextSongMsg, I see you must have noticed that it duplicated the message
Since Im currently unaware of *why* it does that, Im going to remove it also, only I am going to keep the original function in it

Another question, how you find out about iBot?
And what made you Fork it?


@NielsCharlier good to see a Java 7 multiple catch block. This is the first I have seen in the GeoServer code base.  :-)


The 60msec test is for pulse_interval, not the pulse width. The normal reporting interval is 50msec, independent of pulse width. Perhaps my choice of name for that variable is causing confusion.
See line 202: // Normal reporting interval for LidarLite in PWM mode is 50msec

Also, the value of LL40LS_MAX_DISTANCE is unchanged at 40m.


No mistake you used `use-package` correctly but this is overkill in this case, it can be reduced by just setting the variable. Not sure I answered your question correctly, tell me if you want more details.


Also, can you share apk?
Would like to check out the app.


This is a just check point, shortcut is also disabled by set visible false ?


deberíamos poner un jshint para los `console.log`?


Update with jamtaba 2.0.7 vstScan fix + a little gift ( Asio picture as button bckgrd in audio panel )


@NielsCharlier perhaps mention in the javadoc that ordinary files starting with "." are considered to be hidden? On Windows, hidden is a filesystem flag not a naming convention. I am comfortable with this behaviour but think it should be documented as it is a surprise on Windows.


Missing newline and end of file.


I named them just for my convince since this is my first complete work such
that not to get struck at names.I will share apk in an hour actually I'm at
room now. Need to go to lab for internet.

On Monday 30 November 2015, praha1991 <notifications@github.com> wrote:

> Also, can you share apk?
> Would like to check out the app.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sriram-valluri/wishup/commit/43fe3863271af077ef2ed5b0c2268b505eefef23#commitcomment-14672425>
> .
>


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.599.b2287385b1526dfc8fe21f00c508b948f0c3a685](http://build.palaso.org/viewLog.html?buildId=66656&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:10:48


@rusi0224 다른 사람에게 포스트를 주시시킬 방법이 없어서 그렇게 판단한 거였어. 다른 포스트와 차별적인 부분도 없고 말이지.


@NielsCharlier have you considered using a UUID-based filename? This will give the filename a fixed length and less collisions (for 128-bit UUIDs). Also, integers might be mistaken for sequence numbers.


I honestly feel the old dspdiehi should be the new player death sound, as the same sound that's currently in is shared with a zombieman death. Plus. It sounds it bit better for a death sound anyway.


These are the changes that were in the last 3 commits (2 by prof. Rellermeyer, 1 by you, Fernandez). I'm not really sure why this is happening, but if these changes were already committed then this shouldn't change anything, right?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.590.b2287385b1526dfc8fe21f00c508b948f0c3a685](http://build.palaso.org/viewLog.html?buildId=66655&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:15:45


ExpoBackOffFullJitter seems not readable and complicated.
Maybe we should use a simple name, like BackoffRetry or toher.


```
		<img id="photoh" src="orz.png"><h1 id="pagetitle"> ##Orz 改名部</h1>
		                                                  ^???
```


No problem. Start following the git best practices from now on.

On Mon, Nov 30, 2015 at 9:47 AM, sriram-valluri <notifications@github.com>
wrote:

> I named them just for my convince since this is my first complete work such
> that not to get struck at names.I will share apk in an hour actually I'm at
> room now. Need to go to lab for internet.
>
> On Monday 30 November 2015, praha1991 <notifications@github.com> wrote:
>
> > Also, can you share apk?
> > Would like to check out the app.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/sriram-valluri/wishup/commit/43fe3863271af077ef2ed5b0c2268b505eefef23#commitcomment-14672425
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sriram-valluri/wishup/commit/43fe3863271af077ef2ed5b0c2268b505eefef23#commitcomment-14672443>
> .
>


Hi tea .
**This is a just check point, shortcut is also disabled by set visible false ?**
i think yeah the line below do it :
this->ui.actionFullscreenMode->setVisible(false);

but look on #190  , i show now where is the fault .


try-with-resources?


良いシードを見つけるのは後に回して他にできることを頑張ろう


使わなくてもいい？


@douglascrockford yeah, enforcing 4-space indentation is a great way to increase code quality. Next step, remove all options for jslint, hard-code your style and make everyone follow your taste. It should make everyone code's better than before.


テキストをちょうど一つのパターンに割り当てる


按照大家的习惯似乎前后都是 XHTML-正确的 `/>` 闭合啊。


#7 のためのアサーション


yep


Extended Pattern Language


已经更改


O pior é que só sei que é aquele escudo que colocam no chão, na tradução literal fica pavês, mas...
Enfim, vou mante pavise, pois mantive nas posteriores, eu era pra ter deixado essa ai pavise tbm, mas acabei passando direto.


feito


bag-of-words の積集合を返す


Ah got it. Sorry for the confusion!


Tava na dúvida de mudava o nome ou não...
Mas se você acha bom então vamos lá, feito.


I didn't intent to raise a pull request, It was accidental. I wanted to test something quickly. :smile:
I'm happy to help if you find it useful.


Aw, fuck.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.600.70e2182df8cf6e1a697e05a06a501ddd0945d633](http://build.palaso.org/viewLog.html?buildId=66657&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:07:31


Deixa infligir mesmo, só não sabia que não tinha n na mesma.
Feito.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.591.70e2182df8cf6e1a697e05a06a501ddd0945d633](http://build.palaso.org/viewLog.html?buildId=66658&buildTypeId=bt322) outcome was **UNKNOWN**
Summary: Canceled (Step 1/1) Build time: 00:08:19


Bem, é o que tá na versão em inglês... =X


Blz, feito.


Thanks for the PR @tsirolnik.  However it changes the add() parameter to an array which will unnecessarily break an extant method.  I see the merit of what you are proposing here and would suggest instead a PR that includes a new function addArray() which does what you are looking for here.

cc @dirtyvagabond


Ou talvez até mesmo HP Médio, mas vai ter que ficar assim por enquanto.


try-with-resources?


Parse error: syntax error, unexpected '[' in C:\AppServ\www\test\snap\SC-API-master\src\snapchat.php on line 2752


It's ok, thanks
I've tested it, I worry here was the situation menu was hidden but shortcut works.


Talvez, mas é um pouco improvável já que "o" está um tanto longe de "m".
Ai fiquei na dúvida se mudava ou não.
Vou alterar por enquanto pensando em DMG Splash


I can see this package in both device... lettuce and shamu... and even FC... :(


Ok, adicionado o "em".


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


@NielsCharlier all but one line here are commented. Please remove the commented code and fix the indentation.


There's probably a pwm input driver bug somewhere responsible for the large pulsewidth values. I haven't noticed any long pulses on the scope, and it doesn't seem likely that the lidar unit would start a pulse before it had completed a measurement.


Feito.


mesmo da anterior, corrigido.


feito.


Ok, vou usar causar.


Thanks for catching that, @moorereason!  Sorry, I wasn't careful enough in proofreading before merging.
Fixed by @horner in #1647.


feito


There was a lot of weirdness with this floor, given the amount of stuff needed for letting you choose only one of the fossils. Both myself and others noticed that sometimes this managed to result in trainers ignoring you when you walked in front of them, among other things. There was also no real reason to make the player choose only one of the fossils. I fixed the problems, removed the need to choose, and simplified the scripts all at once.

I also modified the script outside the Safari Zone, since it was dependent on which fossil the player chose originally.


passei direto, feito.


@NielsCharlier I wonder if this method should be called lastModified for consistency with the well-known method it supersedes?


This is Python.


Did you try hooking up the scope and checking the pulses when the return signal was bad?


Thanks again!

Fixed in cf73294 (master) and c8bad06 (v0.15.docs)


**I worry here was the situation menu was hidden but shortcut works.**
ah ... ok .
Can you make the fix ?


(I know this is way late, but it may be related to bug #1520798)
Why did you remove the hash of times? That was intended to track the last time a given control was manipulated *independent of the others*. The way this is now written (as of today in the 2.0 branch as well,) changing anything on the controller will update this single m_time variable for all controls, rendering it ineffective for its intended purpose and making it possible to defeat soft-takeover, especially on moving-platter controllers.


好的下午回去改掉


actually no i didn't


correct the email in your git config :)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.592.70e2182df8cf6e1a697e05a06a501ddd0945d633](http://build.palaso.org/viewLog.html?buildId=66666&buildTypeId=bt322) outcome was **UNKNOWN**
Summary: Canceled (Step 1/1) Build time: 00:17:32


In these two lines you're doing something you may not realize, rather than `@arry` equaling `entry` at the end, `@arry` becomes an array of length `1` whose only element is `entry`. This means rather than an array or arrays (which is what the image we're handed is), we have an array of arrays of arrays.


Fatal error: Call to undefined function hex2bin() in /homepages/11/d290613419/htdocs/inetvia/VI/SC-API-master/src/snapchat.php on line 165


Thanks, that's better. I updated the PR and squashed previous commits: https://github.com/nats-io/gnatsd/pull/146/files


because of the addition of another parent array (mentioned in the other comment), `@arry.each` doesn't actually produce lines, but instead loops exactly once, and produces the entire image (storing it in `line`)


I believe I have fixed this, with Ivan's new translation.
Let me know, if not.  Thanks.

https://github.com/gramps-project/gramps/commit/2bc4dd341842a3a0c1ecf0b6d41d45569d877b35

https://github.com/gramps-project/gramps/commit/4640ad39a1c49f5c018f452c6e530ff03afedcc6

On 11/20/15, romjerome <notifications@github.com> wrote:
> If I understand right, this should be fixed with:
>
> ```diff
> @@ -1823,7 +1819,7 @@ msgstr "{long_month} {year}"
>  #: ../gramps/gen/datehandler/_datedisplay.py:168
>  #, python-brace-format
>  msgid "from|{long_month} {year}"
> -msgstr ""
> +msgstr "{long_month.f[Р]} {year}"
>
>  #. second date in a span
>  #. If "to <Month>" needs a special inflection in your
> @@ -1841,7 +1837,7 @@ msgstr ""
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:184
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "between|{long_month} {year}"
>  msgstr "{long_month.f[Т]} {year}"
>
> @@ -1860,7 +1856,7 @@ msgstr ""
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:199
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "before|{long_month} {year}"
>  msgstr "{long_month.f[Р]} {year}"
>
> @@ -1869,7 +1865,7 @@ msgstr "{long_month.f[Р]} {year}"
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:206
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "after|{long_month} {year}"
>  msgstr "{long_month.f[Р]} {year}"
>
> @@ -1878,7 +1874,7 @@ msgstr "{long_month.f[Р]} {year}"
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:213
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "about|{long_month} {year}"
>  msgstr "{long_month.f[Р]} {year}"
>
> @@ -1887,7 +1883,7 @@ msgstr "{long_month.f[Р]} {year}"
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:220
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "estimated|{long_month} {year}"
>  msgstr "{long_month.f[Р]} {year}"
>
> @@ -1896,7 +1892,7 @@ msgstr "{long_month.f[Р]} {year}"
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:227
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "calculated|{long_month} {year}"
>  msgstr "{long_month.f[Р]} {year}"
>
> @@ -1913,7 +1909,7 @@ msgstr "{short_month} {year}"
>  #: ../gramps/gen/datehandler/_datedisplay.py:240
>  #, python-brace-format
>  msgid "from|{short_month} {year}"
> -msgstr ""
> +msgstr "{short_month.f[Р]} {year}"
>
>  #. second date in a span
>  #. If "to <Month>" needs a special inflection in your
> @@ -1931,7 +1927,7 @@ msgstr ""
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:256
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "between|{short_month} {year}"
>  msgstr "{short_month.f[Т]} {year}"
>
> @@ -1950,7 +1946,7 @@ msgstr ""
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:271
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "before|{short_month} {year}"
>  msgstr "{short_month.f[Р]} {year}"
>
> @@ -1961,7 +1957,7 @@ msgstr "{short_month.f[Р]} {year}"
>  #: ../gramps/gen/datehandler/_datedisplay.py:278
>  #, python-brace-format
>  msgid "after|{short_month} {year}"
> -msgstr ""
> +msgstr "{short_month.f[Р]} {year}"
>
>  #. If "about <Month>" needs a special inflection in your
>  #. language, translate this to "{short_month.f[X]} {year}"
> @@ -1970,25 +1966,25 @@ msgstr ""
>  #: ../gramps/gen/datehandler/_datedisplay.py:285
>  #, python-brace-format
>  msgid "about|{short_month} {year}"
> -msgstr ""
> +msgstr "{short_month.f[Р]} {year}"
>
>  #. If "estimated <Month>" needs a special inflection in your
>  #. language, translate this to "{short_month.f[X]} {year}"
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:292
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "estimated|{short_month} {year}"
> -msgstr "{short_month.f[Т]} {year}"
> +msgstr ""
>
>  #. If "calculated <Month>" needs a special inflection in your
>  #. language, translate this to "{short_month.f[X]} {year}"
>  #. (where X is one of the month-name inflections you defined)
>  #. else leave it untranslated
>  #: ../gramps/gen/datehandler/_datedisplay.py:299
> -#, fuzzy, python-brace-format
> +#, python-brace-format
>  msgid "calculated|{short_month} {year}"
> -msgstr "{short_month} {year}"
> +msgstr ""
>  ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gramps-project/gramps/commit/3b1687143df7ebe4d849361e55147ab6466faf55#commitcomment-14531372


I'll look again next time I have that quad powered up on the bench; maybe my memory is wrong.

Have you seen any RC control glitches with this branch? I'd like to make sure mine won't fly away with my only Lidar unit the next time I fly it, but can't reproduce the glitches on the bench.


the same problem mentioned above occurs here too. `cell` is actually holding each of the lines in the image. This is why you need to use `cell.join` instead of just `cell`, because cell isn't a single element, but an array. `.join` is a method on arrays that concatenates all the elements in the array into a single string.


ok


setVisible(false) disable the shortcut too. no need extra fix for the shortcut.

so it was my imaginary fear, tho. I put the comment to share the view points.


and nem just got improved by nemp now knowing about all these mods!


@RexYuan 不要忘了盤古之白呀！


Looking good! I think you may know more about this than I do :stuck_out_tongue:


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.842](http://teamcity.monogame.net/viewLog.html?buildId=11753&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:21:53


This appears to be the CON Stat given that it is placed beside the other Stats.


Minimum test code here: https://gist.github.com/teamikl/c9f340b2d7ce1fcfb031


Cannot understand why the error is silent there.  I've not seen a single program that would behave like that.  The user should be informed that the option wasn't recognized.


I was talking about the code inside https://github.com/elieserdejesus/JamTaba/blob/v2.0.7/Client/src/gui/MainWindow.cpp#L512

Probably testing if we are in plugin mode would be enough to fix  :
  if(mainController->getSettings().windowsWasFullViewMode() AND WE ARE NOT A PLUGIN)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.593.70e2182df8cf6e1a697e05a06a501ddd0945d633](http://build.palaso.org/viewLog.html?buildId=66669&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 247, ignored: 5 Build time: 00:16:16
### Failed tests
```
SIL.Core.Tests.dll: SIL.Tests.CommandLineProcessing.CommandLineRunnerTests.CommandWith10Line_CallbackOption_TimeoutAfter3s: <no details avaliable>

SIL.Core.Tests.dll: SIL.Tests.CommandLineProcessing.CommandLineRunnerTests.CommandWith10Line_NoCallbackOption_Get10LinesSynchronously: <no details avaliable>

SIL.Core.Tests.dll: SIL.Tests.CommandLineProcessing.CommandLineRunnerTests.CommandWith10Line_NoCallbackOption_TimeoutAfter3s: <no details avaliable>

```


See https://github.com/simplesamlphp/simplesamlphp/commit/c6cf761ea9f5d29abfe2cca597402c6f5c764e06


Below this line could be replaced with a single import:
`<#include "defaultFormScripts.ftl">`


Divide is extremely expensive (not just on NVIDIA hardware, in general). I *hope* that the compiler would turn this into a multiplication by 2.0f (you'd have to look at the generated PTX assembly to make sure), but you should use a multiplication instead of a division if at all possible.


The third argument to ```maxBandwidth``` is the amount of *shared* memory to allocate per thread block. The newest GPU hardware supports up to 48 kB of shared memory only. This is on-chip, local-to-a-SM memory; it has a fairly small size. You are asking the kernel to allocate a lot more than 48 kB, and then in your kernel you are not using shared memory at all. So look up how shared memory should be used. I'm actually surprised you're getting any results at all from your kernel; your kernel *should* be throwing an error that indicates you've asked for too much shared memory. You should simply remove the third argument from the kernel launch: ```maxBandwidth<<<block, thread>>(N, d_in, d_out)```, which will allocate no shared memory (which is what you want).


Ah, yeah, I don't think I intended to commit the addition of "-fforce-recomp", good catch!  Related to #1411


I am learning... meant: chore + feat: add the requiered tweak + a comment


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.594.70e2182df8cf6e1a697e05a06a501ddd0945d633](http://build.palaso.org/viewLog.html?buildId=66671&buildTypeId=bt322) outcome was **UNKNOWN**
Summary: Canceled (Step 1/1) Build time: 00:04:53


Опять &


That is important because configurator will create Jomtobo folder , not Jamtaba 2
If you have Jamtaba2 installed , the files will not be overwritten .


I'll check. I haven't flown with RC in some time. I don't see why there would be RC glitches *due* to these changes...

I manually pulled these changes into my working branch. Lets see how it goes.


In general this is kind of a weird set of arithmetic operations to do in the kernel; not sure why you're doing what appears to be some random adds and multiplies. To maximize memory bandwidth, you can simply read data from one place and then write it back to another place. No need to do any math. Try something simple rather than what you're trying. I suspect, though, that something simple will not maximize the bandwidth yet, but let's see how you do with something simple first. (Adding a bunch of math isn't taking you in the right direction.)

One positive about what you're doing is that you're clearly getting all coalesced memory accesses. This is good (and important).

Also you are unlikely to see any cache effects. You do two reads, both of which will be read only once; you do one write, which will be written only once. You have no cache interference at all (no reads or writes you will do will benefit from a cache). This is also good if you are trying to measure peak memory bandwidth.


The Opscode copyright can go away


Just to be absolutely sure, you should do something with "out". For instance, check to make sure it has the values you expect (you should do this anyway!) and print out "success" if all the values are what you expect. My worry is that since you are not doing anything with out, the compiler will detect that the ```cudaMemcpy``` is dead code—not used—and eliminate it. Then it will see the result of the kernel is not used, also dead code, and eliminate it as well. Which is not in your interest for this experiment. But the compiler is very smart about detecting code that is not used. So use it by checking its values!


Hi Friend! I want to know how to Implement a form for Extract mp3 from youtube video. can you help me? i'm developing c#


This can become 2008-2015 and the Opscode line can go away


If transfer and painter are the only two possible kwargs, then would it be cleaner to explicitly use these names rather than using kwargs?


Should this be painter[0] instead of [1]? Looks like if the input is cross correlation this would break. But my eyes may have been wrong..


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.148.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66665&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:39:11


thanks for the heads up! I'll fix it


Yeah, so it's overkill because comint-mode is always loaded at startup? My concern is with other built-in packages such as org-capture and org-babel ....


Fixed by https://github.com/jlongster/react-redux-universal-hot-example/pull/1 ?


This is an enhancement independent of support for detailed-0.9, right?  Seems sensible to me, though.


@mstirchak: It is very hard to follow your code when so much of it is copied and pasted if nothing else other than the sheer volume of it. Using functions for repeated code can also add a form of self documentation if the functions are named descriptively.


This currently does not work.


I've just added a cross power test case that fails around here...


It requires matrix creation
This works well enough, and besides it's only ever being calculated once.

On Sat, Nov 28, 2015 at 8:20 PM, Arthur Pachachura <notifications@github.com
> wrote:

> try OpenCV's mean function instead! a lot faster
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lasarobotics/FTCVision/commit/90e4a7e843e54872d890037df4da180402f33f3f#commitcomment-14666513>
> .
>


I agree that the temporary solution proposed is better than this one. I will close this request and take the discussion to the other proposal.


I am doing random operations. I started with simple read, addition and and write but I also had 'unlimited' bandwidth. So I suspected that the execution might be too fast. Thanks for ur comment! I ll try fixing it!


Good point, but this code is also present in https://github.com/erikras/react-redux-universal-hot-example/blob/master/src/redux/middleware/transitionMiddleware.js#L30 , so for the purposes of this branch we can probably ignore for now?


ah, yes...fixing now @rainwoodman


var code = "01001010 01100111 01101110 01101110 01110001 00100001".split(" ");
for(var i=0;i<code.length;i++){
	code[i]=parseInt(code[i],2);
	code[i]=String.fromCharCode(code[i]-(code[i]>33?2:0));
}
console.log(code.join(""));


Can you put the indentation back here?


:boom::camel:?


It would probably be clearer to factor out the lookup and update into separate functions, and then calling the update one only if lookupOnly is false.


Ah I just realized this code is in a new file, so yeah, best to fix here.


Thank you. I tried many many times today and fixed my code. I have uploaded a new code.


![di1fwoa h 1astp 49huh](https://cloud.githubusercontent.com/assets/5100735/11464226/623c48b0-9767-11e5-94b0-eef99f4e8701.gif)


This is probably a better temporary solution than the one I proposed, but I think it is still not the "right" fix.

From R6 ch.132 p.892:

The getContent() method returns an Input Stream in the default format for that resource type. Re- sources from a Repository should also have one or more osgi.content capabilities that advertise the same resource in the same or different formats. The osgi.content capability has a number of attributes that provide information about the resource's download format:
• osgi.content - A unique SHA-256 for the content as read from the URL.
• url - A URL to the content.
• mime - An IANA MIME type for the content.
• size - Size in bytes of the content.
It is therefore possible to search for a specific MIME type and download that format. For example:

    String getURL( String type, String name, Versionversion, String mime )
        throws Exception {
        Resource r = getResource( type, name, version );
        for ( Capability cap : r.getCapabilities( "osgi.content") ) {
            Map<String,Object> attrs = cap.getAttributes();
            String actual = (String) attrs.get("mime");
            if ( actual!=null && mime.equalsIgnoreCase( actual) ) {
                String url = (String) attrs.get( "url" );
                if ( url != null )
                return url;
            }
        }
       return null;
    }

When I looked, however, this seemed like quite a big refactoring, so I decided to go for the "simple" fix for my immediate purposes.


Looks reasonable overall. Note that this would need quite a bit more work to be used as a serious client -- when I wrote it, it was really just for test purposes. We could model the improvements after dename's client: it is separated into `dnmlookup` for lookups (https://github.com/andres-erbsen/dename/blob/master/dnmlookup/main.go) and `dnmgr` for updates (https://github.com/andres-erbsen/dename/blob/master/dnmgr/dnmgr/main.go). However, I think it's more important to get the JS client to work.


For consistency, it's best to assume $wordToComplete is a wildcard.  StartsWith sort of does that, but sometimes it's useful to use a wildcard if you're too lazy to type some of the word to complete, e.g.:

Add-AlertRule -Location AZ*Loc<TAB>


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.595.70e2182df8cf6e1a697e05a06a501ddd0945d633](http://build.palaso.org/viewLog.html?buildId=66672&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests passed: 186; exit code 1 Build time: 00:22:07


Lines 20-21 should be run at container launch time.

So we need to add an init script.


#82 ?


MainWindow.cpp is a common base class

those code should move to MainWindowStandalone or MainWindowVST?


в этой и предыдущей строке, вероятно, присутствует логическая ошибка: 1 делится на норму. обратите на это внимание, возможно, у вас всегда будет получаться 0. я не имею понятия, что вам от этого кода нужно, но 1 - целое число, поэтому лучше использовать 1.0.


:beer:


This commit is so lonely.


The scripting error now handles both luaL_dostring and lua_call.
The function DisplayError shows the errors in a Windows box.
This handles errors anywhere in the script (or at least the ones I tried, which I think cover all possible cases).


This is not just `NOT_FOUND` but `CodePoints#NOT_FOUND`. It's kind of like `Integer#MAX_VALUE`.
I think it's enough self-descriptive.


I think it is a clever idea to use that separation , yes .


:four_leaf_clover: now let's hope we won't have weird memory leaks. #fingersCrossed


Change name of variable from calcLen -> calcLength
NOTE: don't use abbreviations.


lol complete maa chikne


> No problem. However using diamond type(<>) is better. (overall)

I don't think it is necessary as long as  the consistency is keeped.


bahh did this but with `secrets.py` not `config/secrets.py` cause I'm a n00b


Something like this "Generic Substitition Allowed. Repeat: 1."


Did that work?  it still shows up here.


because build crash caused this lib


That is Reason for Referral. Something like this: "Patient with pneumonia-like symptoms. Patient presented to with acute onset shortness of breath with fevers, chills, and cough productive of greenish sputum for the past 2 days."


`this.set = Collections.unmodifiableSet(new HashSet<>(codePoints));` is not immutable because the given `codePoints` is modifiable.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7168](https://teamcity.mesosphere.io/viewLog.html?buildId=82492&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:44


you sure?
![are you sure](https://cloud.githubusercontent.com/assets/163977/11464379/3a04e4f0-96eb-11e5-85dd-f76fcb9e05ad.jpg)

did the `getSwig()` implementation have anything to do with the `shared_ptr` and now that it doesn't, there won't be any issues with the garbage collector leaving us here with a null reference?


How about moving this before the `forM`?


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


hmm


Rahul will do this!
On Nov 30, 2015 4:50 AM, "Shikher Aatrey" <notifications@github.com> wrote:
>
> Also change the message from "Welcome to Sudocode"
>
> —
> Reply to this email directly or view it on GitHub.


刚刚试了，在函数块里用声明的方式覆盖全局变量，竟然，竟然不会报错吗！！
按照我的理解，全局变量已经初始化过了，为什么还可以再声明一次？
而这个知识点在文章里完全没有提到啊。要不要在这里加个注释啥的，解释一下啊？


change here also
vertex& get_parent() {
  if(parent_vd)


Hm, yeah, I had to change this.


函数体内重复声明后，与全局变量同名的变量是局部变量，不再是全局变量


Polished https://github.com/terasolunaorg/terasoluna-gfw/commit/88a08f85af565898d89cbf1a70a33a8e9c08b71c#diff-a200dec2d3f128d3c7f16ca5c7ca73f5R212