Hi Jonney.  Thanks for the helpful information.  I completely agree with the comments in your edit.

Part of the reason I used "Sample AaBbCc 123" instead of the font name is to make it easier to compare fonts.  If different characters are used for each font sample, it is hard to compare things like differences in width, height, spacing, etc.  But if each sample uses identical characters, it is easier to see the exact differences between fonts.

My goal (which is mentioned in the UC comments) is to provide sample text for as many languages as possible.  I am open to using any sample text that you think is best for your languages.  (I just picked the word "Sample" on a whim, so we could even change the English text too, if that's better).

Finding supported languages for each font is difficult.  On Vista or later, I think I can use Uniscribe to do it (https://msdn.microsoft.com/en-us/library/windows/desktop/dd368548%28v=vs.85%29.aspx).  I have no idea what to use on XP.

The hard part will be caching supported language values in a timely fashion, so we don't slow down program start-up.  I hope this is possible.

If you think it's okay, I would like to use a system like the following:
1) For each font, find supported languages and store them at start-up.
2) When drawing the combo box, if a font supports the current user language, render sample text in that language.  (For this, we need to have appropriate sample text for as many languages as possible.)
3) If a font does not support the current user language, render sample text in English.
4) If a font is not language-specifc - symbol fonts, like Wingdings - we can use totally different text, maybe just "abc123" or something simple.

Does that sound okay?


Integer -> int


Besides the integer -> int stuff it looks good :+1:


TeamCity Selenium POM :: CI [Build 14](https://teamcity.sponte.uk/viewLog.html?buildId=76&buildTypeId=SeleniumPom_Ci) is now running


@moyni if we're including interns then James Walker and Lawrence are missing! Would be nice to have them in - this ok?


Remove this message.


TeamCity Selenium POM :: CI [Build 14](https://teamcity.sponte.uk/viewLog.html?buildId=76&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 35 Build time: 00:00:39


Remove this message and branch.


Bonjour,
Depuis hier pm après un pull du dernier commit de la dev je suis positionné au sud de Halifax dans l'ocean carte fond bleu avec ceci dans mon xml à l'ouverture de IGO  <contexte aContexteMenu='true' centre="-7939475.48, 5900311.76" zoom="7" > </contexte>   ?


Stylistic question: Why not drop the variable `ncol_items` altogether and instead directly write `len(lg_items)` (or `len(proxy_artists)` when calling `legend()`? The information does not require an intermediate label as far as I can see.


`dconf-editor` for a gui, `gsettings` for a cli tool. All really solid tools that can show real time updates, descriptions, default values, etc.


@KonstantinKuklin What use case of this feature?


Turbojpeg does provide pkgconfig under Linux.


@thinkyhead I've just uploaded the latest dev, I think this PR introduced a "Wait for user..." bug. The LCD now always shows this waiting message.


I created a comment on this line a few days ago; but to my embarrassment I did that on a fork that was removed in the meantime. @ltworf, you must have received an email with the comment and should be able to recover it here.


Ну в хроме на макбуке ж работает, а так тремя миллионами больше, тремя меньше, пфф.
С наследованием некрасиво получилось, да.
Может так 95c87b752418676ad6904fd50852e4a382b01da6?


Same as above, I commented on this line in a fork which got deleted. Please recover that comment as well, @ltworf.


let's hope we don't have any code that asks for the Context CM explicitly ;) I guess this will allow to find them and make them use the provider...


libusb-1.0.pc provides:

```
includedir=${prefix}/include
Cflags: -I${includedir}/libusb-1.0
```

Use the content properly.


Про мой пример: кончено и body и div должны быть закрыты! Я просто написал только открывающие тэги, чтобы показать, что с body прицеплен module, а к div'у controller. Наверное да, добавлю и закрывающиеся для полной ясности.
А с DatePicker'ом все хорошо, просто angular не успевает инициализироваться к тому моменту, как DatePicker уже хочет его использовать. Если обернуть вызов datepicker в setTimeout - то все будет хорошо. Да и, в JavaScript'е event цепляется ко второму элементу (вызвана функция eq), которого на странице нет!


I removed this since it is not explicitly used and reading the documentation it seems that the 'html5lib' argument in line 65 is enough.


That doesn't seem right :-/
Updating to 5.2.0 I get undefined method determine_default_mailer for all the specs for jobs (which aren't even ActiveJob yet)


nice. dconf-editor does the trick. Interesting to see GNOME move to what's basically Windows registry / regedit.


Nobody is supposed to directly lookup this component and anyway this is a plugin.


Okay, so on 1: The three buttons are `submit` buttons, which means the page will refresh if you don't prevent it from doing so. Try preventing the default action.


Please do not mix tab with space.


Tab and space.


It isn't :) the tests fail in travis CI and locally. I'm also getting that feature(tcp) is an error. Take a look at https://travis-ci.org/cyderize/rust-websocket/builds/63587701 . It was there before my commit, so I'm not sure how/if I broke that.


@emord curious if I might be missing something here. Thoughts?


@thinkyhead Or this one introduced "Wait for user..."


Why is this necessary?  gauche/system.h is included from gauche.h.


+1 LGTM  I thought the upstream charmhelpers repository was kept in launchpad.


Could you please move this to the top of the function.


Can you also add support for OpenCL under Windows from [this commit](https://github.com/xlz/libfreenect2/commit/075cd3b49d0034994aafed6b8ebd98f8930d9c71#diff-0d08428557689efc6958436ee0f0d709)?


Am 22.05.2015 um 10:12 schrieb Alex Vandiver:
> This is not without complexity. While having a /smaller/ cutoff size
> would work in a straightforward way, administrators may also expect that
> they can raise the cutoff and have data moved /back/ into the database
> -- which would require additional code.
>
> I'm not saying that |--all| is a bad idea, merely that it needs
> additional code or additional careful documentation to cover expected
> use cases.
Shure, I had just in mind lowering the value.
I think in ether way, the documentation should be updated regarding this.


Mediathek hochgelade
TODO vormerker != entleiher


tst


Should `serializeInstruction()` use `instruction.canonicalUrl` instead of `instruction.path`?


"Nobody is supposed to directly lookup this component". This is only true recently, when we added the provider.

Then maybe we should remove it as a component and let the provider return a unique instance of it?


I'm not 100% certain about this being called 'error_on_missing_fields' but it felt like it was a bit more specific than the 'strict' idea we were talking about before...  Maybe 'raise_on_missing_fields' would be more exact - what do you think?


>All multi-term does is calling ansi-term.
@tuhdo What insight do you have, to make a statement like this.  Have you looked at the multi-term code to determine this, because that's not what the multi-term documentation says?  On that note, multi-term is much more clearly documented.  The gist of what I get from shell-pop's documentation is that it adds a keybinding for a shell window.  Ok, no big deal, we can add a keybinding for multi-term-dedicated to do the same.  I get that shell-pop seems to do more, but it's NOT a drop-in replacement for multi-term

@syl20bnr , I see on Gitter that people are already confused about this.  It might be worthwhile to vet PRs more judiciously.


Why a different variable? It makes docs complicated.


For number 2, working on getting all the deps installed to look at it. My first guess is that there's no `col-xs-2.5` class. Use the inspector in Chrome to see if the right styles are getting applied.


Here too.


This function was missing the closing brace, which stopped the sounds from playing.


I had forgotten, but don't you want to call the field ID instead of KEY ?


missing spaces around =


Wrap at 80 cols.


+        strError = "Failed to read \"" + _strXmlConfigurationFilePath + "\": " + strError;


vimで問題といてるのか！素晴らしい！


Given new platform support, let's make it `4.2.0.dev0`


If utils.sh does not exist, check_glusterfs script will not work properly as it uses variables defined within it, exit-statuses will be void, if utils.sh is not sourced.

Rather than making a conditional source, you should do a mandatory check of it, and fail the script early and loudly if it's missing.


> Interesting to see GNOME move to what's basically Windows registry / regedit.

Maybe in terms of UI but I wouldn't go that far :P


It's misleading or me. You could use described_class, but it's just a matter of taste. So, it's fine as it is now :)


Throwing a RedisDisconnectedException would provide a useful information and avoid the caller to wait. Then it's up to the user to deal with the exception differently or not.
When building a retry logic you should be looking for AskTimeoutException, so throwing a new exception shouldn't really have a big impact right ? Either you treat it the same way (retry after x secs) or you ignore it.


Broke release tarball for darwin: http://hydra.nixos.org/build/22426141/nixlog/1/raw , I think on line 41.


Please wrap the nested statements.


I am not. Angular has service based on q which we use heavily at nib. I believe (and correct me if I'm wrong) that q was the library that made popular the concept of a promise


I updated the files in response to those comments


This will break event update, because *my_events_path* does not permit params


I had missed we use primitive booleans, not just naturals and integers! Any other type to deal with? Fixed in 46693aa9b36715d6138de3fc8eda18b15024d184.

I think I've also implemented widening correctly by now.


@gforcada Too bad :/ Sometimes it's hard to remember, how "everybody wants Mosaic" :)

Gil Forcada Codinachs kirjoitti pe toukokuuta 22 14:59:43 2015 GMT+0300:
> @datakurre sadly nearly no one is able to attend it, so we are thinking to either change the sprint topic or change the dates... maybe next week we have some time to discuss it a bit more with other Berliners :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/plone/plone.app.tiles/commit/e95db1a0ed212c9db31839951551bac108f347e3#commitcomment-1132591


Fixé


Have you tried to login with it? May be we can do some tests to make sure, but I'm happy with 'too good to be true' for now;)


It could be a bit easier if the `@UseCase` annotation included also `@Component` so we don't have to declare the implementations with `@Named + @UseCase` but that couples `@Usecase`  to Spring.

I've tried with `@Named` on the `@UseCase` but Spring doesn't detect the implementation as a Bean.

But so far I see this solution much better than having to give value to the `@Named` annotation and declaring a constant.


fdsfdsfds


This is not necessary. Just add a config/secrets.yml file to your project
```
cp config/secrets.yml.example config/secrets.yml
```
You should also rebase the latest build of the project into your branch


It looks like you're just comparing the names of the slots--is that right? Do you need to compare the offsets too?

Why do you only compare if `slots_a && slots_b`? (What if, say, `!slots_a && slots_b`? Would you need to return 0 in that case?


Why is this?


Congrats!


(Also, small nitpick, but could you make this function return `bool` instead of `int`?


Last commit :'(


This is breaking lot's of tests due to the **c** before **Rack**


it's hard to test this but I can't see any syntax errors or anything
:+1:


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


change "immediatelly" for "immediately"


make this public please


Example:
The data source descriptor
```js
{
  sourceUri: "jr://foo",
  defaultId: "foo",
  intialQuery: "/foo",
  name: "Foo"
  structure: {
    "nested": {
      structure: {
        "element": {
          structure: {
            "@id": {no_option: true},
            "name": {no_option: true}
          }
        },
        "@id": {no_option: true},
        "name": {no_option: true}
      }
    },
    "@id": {no_option: true},
    "name": {no_option: true}
  }
}
```
And in the (possibly broken) form output
```xml
...
<instance id="foo" src="jr://foo" />
...
<itemset nodeset="instance('foo')/foo/nested/element">
  <label ref="nested/element/name" /> <!-- fully qualified: instance("foo")/foo/nested/element/nested/element/name -->
  <value ref="nested/element/@id" />
</itemset>
...
```

What I would expect instead
```xml
...
<instance id="foo" src="jr://foo" />
...
<itemset nodeset="instance('foo')/foo/nested/element">
  <label ref="name" /> <!-- fully qualified: instance("foo")/foo/nested/element/name -->
  <value ref="@id" />
</itemset>
...
```


Makes sense. Thanks! :+1:


:+1:

dat is vreemd :)


@borshop: retry


yeah, my common mistake, thanks for catching ;)


Yes, you're right.
But do we need to base this on QObject? I would imagine you did it for automatic deletion, but this is a signleton and we manage its deletion anyway.
PS: guiconprovider can base off QObject by itself anyway.


could you make it listen only on lo?


code for LR


@smn do we have to store languages with their name? Is there a reason we can't just store the locale codes?


Super ! Ça marche !


"execute" for "executes"


Thanks - fixed


It actually makes a difference on SNES9X DEMO V 1.16.

On 2015年5月22日 21:06:08 JST, AnotherLife <notifications@github.com> wrote:
>Oh, you added the fsingle-precision-constant flag? Good, it makes a
>huge difference in Pilotwings, no difference on the other games I
>tested.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/173210/snes9xTYL/commit/dc39d3a7e3cf5d012a116f56d5bc7d0622df0fe4#commitcomment-11326012


Strange name - include "mssql" in it instead


Won't work for anyone else


"raises"


That's because it's trying to use nightly features in stable rust. I think Travis has started using the stable channel. I'll have to tell it to use nightly somehow.


I am trying to add new project


Honestly not sure for #3 just from looking at the code. The params look right. Can you make the fields non-optional, or use `pry` to inspect the `params` hash before you save it? That should tell you if the data is getting back to the server at least.


Interesting.  I guess `ClientCache` doesn't use the `load()`/`save()` bits from `fsIndex`?


It wasn't a question to satisfy my curiosity. It was a suggestion to add a comment to your source code for people who'll read it. It's not obvious why you want those keybindings only when vim editing style is activated.

```
; We can be in `holy-mode` and the following keybindings are confusing there
```

One short sentence will relieve reader from going to issue tracker.


Hmm, wondering if we should indicate the *intent* in the parameter name rather than explain what it does. Could it just be called `validate`?

```
def as_template_fields(self, template, validate=False):

:param template:   Template for which the fields will be used.
:param validate:   If True then the fields found will be checked to ensure that all
                   expected fields for the context were found.  If a field is missing
                   then a TankError will be raised.
```


done, thanks.. this bothers me the compiler didn't catch this, nor is there any warning.


TeamCity Selenium POM :: CI [Build 15](https://teamcity.sponte.uk/viewLog.html?buildId=77&buildTypeId=SeleniumPom_Ci) is now running


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


I wouldn't have any sources but I would strongly advise anyone to use it. It cleans up the code (one line for `addEventListener` instead of ~4) and it makes the callback less prone to untested hacking, e.g

```js
var Foo = function(){
  var self = this;
  this.shouldntChange = 5;
  el.addEventListener('click', function(){
    self.shouldntChange = 10; // some sneaky, untested hack
    self.calculate();
  });
};

//...

Foo.prototype.calculate = function(){
  return this.shouldntChange; // uh-oh! this should be 5 but it is now 10
};
```


TeamCity Selenium POM :: CI [Build 15](https://teamcity.sponte.uk/viewLog.html?buildId=77&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 35 Build time: 00:00:35


String jest dość ubogi w stosunku do Twojego wypasionego typu ListSubject, lepiej zrób to jako ListSubject


Nope, it manages its own file IO directly. I think it uses a simpler format.


test comment


This is a breaking condition, should be 0.  Fails when running out of the gate on our cluster.


oi, esta lógica funciona, mas fazer o seletor **PARA CADA** resultado dentro do array albums pode ficar pesado se o array for grande. Invés de fazer 50 consultas fica melhor fazer apenas uma, não? Para isso basta jogar `$('#capas')` para uma variável antes do `forEach`.


TeamCity Selenium POM :: CI [Build 16](https://teamcity.sponte.uk/viewLog.html?buildId=78&buildTypeId=SeleniumPom_Ci) is now running


Is overgekomen uit [ETL](https://github.com/praktijkindex/reflex-etl/blob/master/lib/etl/logistic_model/base.rb#L49-L57).


TeamCity Selenium POM :: CI [Build 16](https://teamcity.sponte.uk/viewLog.html?buildId=78&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 35 Build time: 00:00:35


cuidar indentação e ; no final!


Actually any \ that isn't understood is just removed so \s or similar would just be dropped while \n is kept.


w tym momencie fajnie by bylo zapisać w obiekcie list.get(nrStudenta) że uczęszcza na przedmiot subject.get(nrPrzedmiotu)


wut?


We need to bump pins in `tox.ini`, too.


Dunno, har ikke prøvet at lave kommentar her før - smart! :)

Btw så er jeg taget hjem, magtede ikke at sidde og vente til kl 16. Hvad skal jeg gøre for at få det til at virke?


También agregué para que el usuario pueda eliminar un curso


create doc folder and upload SDN Testbed.PDF


Here you create reference to the service each time you call the method.
What is the point of this singleton?


Nahh but I used the code to make something else! Hopefully by the end of today I'll have the ball physics done


I solve local package problem by my repo in github(github.com/plutoshe) to pass the test
we should replace github.com/plutoshe to github.com/master in master branch.
and I will update the comments in other pr


perfeito @cv


* On 2015-05-22 14:44:44 +0200, Davide Principi wrote:

> Hi @zevv, I'm not sure it works :) It should be tested in the near future.
>
> BTW, would you accept the spec file on the repository root level?

It doesn't bother me, there's so much cruft there from automake and
autoconf that I really wouldn't notice one extra file. I live my life in
/src, and learned to ignore the rest :)

But is upstream the right place for a spec file like this anyway? How
does NethServer usualy manage packages?

--
:wq
^X^Cy^K^X^C^C^C^C


Great idea Jon. I'm adding this as an option in plugin settings though, so people can choose the format they like.


@piscisaureus do you happen to know what caused this regression? Users reported that 0.12.2 was working, but in my tests the same problem is there.


@jesssummerfield Lawrence is in now, James W was at the bottom. Merge if you're happy!


What is the purpose of this Context here?
I think it is better to make it a field (the application context) and use it for all other methods.


What is the purpose of this soundID?
You can remove it.


I like this, but I think you can go even one step further and remove the isChangeExpected method completely.  That method was added only as a way to provide me an easy way to state that the local and remote behaviors were different.  Different local and remote behavior is a bad idea and so I think the next step is to completely delete the isChangeExpected method and its one use.


This should in no way still be  md5   that is  WAY too old and insecure


I wonder if we also should add the udp protocol for μtp connections. I know the old QBtSession did not and I wonder why!?!?


This was a good bugfix :)


What if the user just changes the listening port?


no ale deklarujemy pola piszac typ i nazwe, sugeruje zebys zmienila ta linike na <i>ListSubject mySubjects</i>


Shouldn't we also delete our mappings?


My pull request should make the array fifo, at the expense of slower clearing by key when using a named queue. But I figure if you're using a queue name, you want queue-like access. Would two indexes be better than one? Either way, you'll be searching the fifo array for a value when you clear by key.


Thanks.
You are right. We forgot to push the tag.
Now the tag have been pushed.


Add/delete seem to be only called from webui.cpp. Wouldn't the gui need this too, especially when the user changes the listening port?


Published to http://ui.edev.d2l.com/53_a9cd27a44a918b5886810e853bf0b41f8025d1ed


`PortForwarder();` too.


@beledouxdenis
Alright!


Published to http://ui.edev.d2l.com/54_38fab9b4dcc96d962779fe61858db3772867d367


np


`Session();` too


you need to put the nlopt directory under C:
1. the nlopt directory must contain 2 folders: lib and include
2. then run the following command:
$ make boostincludedir=/c/BoostDir    nloptdir=/c/nlopt


`IconProvider();` too


Danger!
Addition of lots of brands to this area will result in overloading.
I'd rather incorporate fastFT content into its relevant subject areas and test whether there needs to be one place for ALL fastFT content no matter the subject.


yesssss :tada:


Rename this parameter to something more obvious.


@Koc common persisters in fully related to SQL, but if I tried to use another protocol to get data (for example get data from mysql database via plugin HandlerSocket) I can`t do it without some piece of magic and hacks inside doctrine engine.
Current list of persisters are limit data sources which I can use with doctrine, because my own driver must be SQL similar and very hard to use non SQL.


ScannedFoldersModel(); too


I believe this IF condition is wrong.
Why only calculate Cystines when num_aa['C'] is uneven?

The condition that only the even number of C residues add to the extinction coeffiecient is already met by calculating num_aa['C']//2.
The only times when mec_cystines == mec_reduced is for num_aa['C'] = 0 or 1 in which case num_aa['C']//2 = 0

lines 292 to 296 should be replaced by something like this:
mec_cystines = mec_reduced + (num_aa['C']//2)*125

This way you get results that correspond with the ExPASy protparam tool (http://web.expasy.org/protparam/)


@allgaeu Oh ja das wäre cool, wenn du einmal nur die rechte Graphik anzeigen kannst und einmal die rechte mit der linken überlagert.
Ich weiß nicht genau was bei mir das Problem ist, ich habe es eingebundne mit

\begin{figure}
\includestandalone[width=\textwidth]{multi}
\end{figure}

und

im main:

\usepackage{tikz}
\usepackage{standalone}
\usepackage{pgfplots}
\usepgfplotslibrary{groupplots}
\usepgfplotslibrary{units}

allerdings sieht es alles andere so aus wie bei Sebastian, Es ist bunt, nichts ist dort wo es sein soll, also alles ist ein bisschen verschoben, ohne dass ich etwas geändert habe.


TeamCity xBIM :: Xbim Essentials :: Release [Build 47](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1667&buildTypeId=XBIM_XbimEssentials_Release) is now running


Sweet!!! - ty so much, this gives me some leads to run down today.
-ers

On Fri, May 22, 2015 at 8:28 AM, Joseph Mastey <notifications@github.com>
wrote:

> Honestly not sure for #3 just from looking at the code. The params look
> right. Can you make the fields non-optional, or use pry to inspect the
> params hash before you save it? That should tell you if the data is
> getting back to the server at least.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Dwin357/AT/commit/70ca46626cc2f0adf98dc8f38940d5a31fac99a9#commitcomment-11327445>
> .
>


Under Windows: the compiled nlopt toolbox contains 2 folders lib and include, the include folder conatins nlopt.h and nlopt.hpp; the lib folder conatins libnlotp.a and libnlopt.lb, so change (and see below for changing the make command):
INCLUDE_DIRS := -I$(boostincludedir) -I$(nloptincludedir) -I$(includedir)


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


Published to http://ui.edev.d2l.com/55_ac34d2046bd0af0f8c243fc859bc8129a5a1d0bc


I just installed multirust and switched the branch to nightly. That fixed my issues locally.


Enclose distinct statements in parentheses so it's easier to understand, even if you got the operator precedence right.


Looks good


TeamCity xBIM :: Xbim Essentials :: Release [Build 48](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1668&buildTypeId=XBIM_XbimEssentials_Release) is now running


TeamCity xBIM :: Xbim Essentials :: Release [Build 47](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1667&buildTypeId=XBIM_XbimEssentials_Release) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:21


- Added website to "who is using fullPage.js"


The function is currently just copied from CPython without any modifications.
I don't have much knowledge in this area of pyston:
I think the ```__slots__``` comparison should be fine because if the slots aren't exactly the same the function will return false. And the offset should be okay because the slot entries are sorted by name and assigned offsets based on the sorted name so the have the same offsets.
But otherwise I'm really not sure if just comparing that ```tp_basicsize``` is the same is enough for pystons implementation. Maybe we need to be more strict than cpython because we don't store our attributes in a separate dict but instead the are always stored more similar to cpythons ```__slots__``` case.
I would appreciate help and input in this regard and I will add more tests.


TeamCity xBIM :: Xbim Essentials :: Release [Build 48](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1668&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:00:46


1. No specific reason for that, I'll change it.
2. ok ,I'll change that too


Оценка "отлично".


@silverwind you forgot `Reviewed-By:` :) (I've done it too haha)


Json response text file is too long and exposing the division and guids. For this uniTest local string object with "\\escape test" text can be used as : string sampleJson = "{\"d\": {\"Remarks\":\"\\\\escape test\"}}";
ApiResponseCleaner.GetJsonObject(sampleJson);


Did you actually test this to see if it works? What is the logic behind filling a random name?


Published to http://ui.edev.d2l.com/56_bc755dfdf8b163221ce3d59c8434d5cffeb1b30d


If this change removed a row that would be in the last release, it would need to account for any dependent rows in AttributeValue in a way to satisfy the constraints involving Attribute and AttributeMap. But this is not the case thus it is OK to remove this row in this simple way.


Зачем вам здесь версия? Вы её используете?


That requires something a bit more elaborate, I'll do it later.


ok about RIL but what do these changes in Audio bring?


Cohenrence => Coherence ;-)


On 22 May 2015 at 14:50, Jerome Leclanche <notifications@github.com> wrote:

> That requires something a bit more elaborate, I'll do it later.
>

You might want to lift code form ttk1 which does the right thing there I
think

--
Dwayne

*Translate*
+27 12 460 1095 (work)


Oh damn. Does it matter?


yes that would be best to access a single dict. you should be able to import it like a function.


fake password I hope ;-)


use `math.pi` instead


Cohenrence => Coherence


you have fixed the problem. fantastic


I have a bug here. Luckily #36 fixes it... Sorry for adding this bug.


done!


You're outside of the force-push zone anyways, so not really so long as it has PR-URL, was just letting you know. :)


一、学生考勤管理系统概要
1.1项目名称
学生考勤管理系统
1.2项目提出的背景
随着计算机科学技术的发展，数据库技术在Internet中的应用越来越广泛，为广大计算机用户提供了更加周到的服务。高校校园信息化的逐步完善有效地借助网络、数据库等技术提高工作和管理效率。如今针对师生的成绩查询系统、教务管理系统、招生就业系统、BBS、校园网站等系统在各大高校纷纷出现，对全校师生的学习、生活、管理、办公带来了便利。
学生考勤管理系统是学校管理工作中一个比较重要的任务，过多的学生考勤管理工作给学校管理人员带来很大的麻烦。虽然原始的记录方法可以解决问题，但是它的不足之处还是显而易见的。学生考勤管理系统是一个对于学生考勤的管理系统，不仅大大减少了考勤管理人员的大量工作，而且还方便了班级和学校对学生出课率的查询，只要通过计算机登入本系统就能及时修改、更正学生的考勤信息。这样节省了人力物力大大提高了工作效率，能进一步加强高校学风建设，维护正常的教学秩序，给学生创造一个优良的学习环境。
1.3项目的目的和意义
  开发本系统的目的就是要解决目前高校中关于学生请假与上课出勤管理中所面临的一系列问题，目前高校学生上课考勤管理都是以任课老师上课点名，记录学生上课情况，对于时间比较长的请假，需要通过递交请假申请让班主任、院系领导都批准方能生效。这种模式在目前高校管理中暴露了不可避免的弊端:
a.学生请假不方便；
b.任课老师不清楚学生是否请假；
c.学生对自己整个学期的上课出勤情况没有整体的统计信息；
d.班主任对本班学生整个学期的上课出勤情况不易查看；
e.院系领导、学校领导不容易把握学生上课的出勤情况。
开发出高校学生考勤管理系统能有效地提高学校办事效率，实现学生请假及上课出勤信息对任课教师、班主任、院系领导、学校领导透明，使班主任、院系领导及时把握学生的学习情况，及时与学生沟通，提高教学时质量。
1.4国内外研究和应用现状
  21世纪是一个信息化时代，因此学生考勤管理系统在国内外的高校中都有广泛的运用，与其相似的就有企业的上班指纹打卡等系统。学生考勤管理系统的开发条件低，开发周期短、功能结构相对简单。属于自行开发，自行使用。所以国内外对学生考勤管理系统的研究也就不多。因此不存在法律上的版权争议。目前，国内的考勤技术已经趋向成熟。


Still unresolved problem under Windows:
MATLAB mex command under Windows cannot directly link .dll file (libVineCopulaCPP.dll), it needs the .lib file.


> But is upstream the right place for a spec file like this anyway?

no, I think. I was thinking about NethServer only and didn't open a PR.

Moreover, we use ``duc`` to create a graph on the web dashboard of the system. We require the XML output only. So I was trying to compile without GUI libraries.

>  How does NethServer usualy manage packages?

We are attempting to use only RPMs from CentOS and EPEL, and avoid self-compilation.

Having a ``.spec`` file at the repository root is good for building the RPM with mock.


yes, was minimal change I made to make comparison easier


Pull request #119 addresses this issue, thank you for pointing out.


Please check inline comment for "ApiResponseCleanerTest.cs". It is also failing for single '\' in remarks i-e: "Remarks": "\escape test"


Yes, would be better to include libspatiallite if we can


Good luck :+1:


You don't need to initialize m_label.


That works - don't know if you could tell from my previous comment but I didn't have a great opinion either way!


1. Background color should be @admin-blue, not @admin-input-blue
2. The icon on the button should be white when in "on" mode (override "color" property in .mce-ico with @admin-white)
3. I don't think you need to use aria-label... I moved &.mce-active, &.mce-active:hover{...} inside .mce-btn {} and everything worked fine after that.


@tonylotts I did look at it. You can have a look [here](https://github.com/emacsmirror/multi-term/blob/master/multi-term.el#L598). It does not call `ansi-term` directly, but as you see in the link, `multi-term` creates a buffer, set it to `term-mode` and enable `term-char-mode`, similar to [ansi-term](https://github.com/emacs-mirror/emacs/blob/3311ace9c54a50b83a838e2eb7fa9565176e0c4f/lisp/term.el#L4154). In effect, it does the same thing as `ansi-term` and that's why I called it just a `ansi-term` wrapper.

Here is [how multi-term handles sub-jobs](https://github.com/emacsmirror/multi-term/blob/master/multi-term.el#L652). The only relevant call is `(term-quit-subjob)`, the rest belongs to `multi-term` for managing its various terms, which is covered by `shell-pop` already.


В принципе неплохо, но плохо, что вы отказались от использования типа Message. Вы оперируете строками и представлениями абстрактного документа в xml / json без использования своего класса.
Скажем так, не знающему человеку было бы очень тяжело понять, какие данные приходят, какие уходят. Хотя я понимаю, что вам это показалось более простым вариантом - исключить из цепочer "json -> java -> xml" & "xml -> java -> json" java представление данных. Меньше преобразований, меньше работы и т.д. Но, повторюсь, в enterprise приложениях на джава так не делают (исключая те случаи, где присутствует какой-то динамический формат данных).

Оценка "отлично-"


when draggable option is enabled, first call of toggle function set enabled = false!!


test


"due to JIT" I think you mean?

Also I would add to the end "macros which do not try to exclude the effects of compilation"  Its useful to say whats different too.


Do we need the LICENSE and README.md if it's only for testing ?


Nikel !
Il serait judicieux de prévenir l'utilisateur avec un truc du style:

Catégories de ce produit (1 choix minimum):


did you mean to change this indent?


@admin-input-blue looked closer to initial Thomas' design :)


Should it also compare the hashsets?


Thanks for the input.  Note that the current build instructions say to use 4.9 and have a note on using 4.8 for Sparky.

Could you point me at the boot bug?  I used to work on compilers and might be able to help.  The gcc-arm-embedded team are pretty responsive too.


flatten() is deprecated; expand() should be used instead. (ref: http://orientdb.com/docs/last/orientdb.wiki/SQL-Functions.html#flatten)


Put `case All:` first since most likely will be the most hitted State.
Leave `default:` for out-of-range values too.


lijkt ergens verkeerd automatisch gemerged dezelfde change twee keer


Or you could just leave it all in the fifo queue and on pop, keep looking till you find a key with a value still in the store...


Yepp, thanks. I'll tweak that.


Changing this to false will break everyone who is running the application in development mode, but isn't using Vagrant. This theme applies to other changes I've seen as well. Based on your first commit, I would do something like:

```
dashboard_enable_pegasus' => rack_env == :development && ENV['CDO_CHEF_NODE_NAME'] != 'vagrant'


APIResponse_Json_Object_RemarksWithEscapeCharacter.txt can be removed by using local variable.


Thanks! Committed to https://github.com/tshiran/drill/tree/gh-pages.


java 7 умеет сравнивать строки через блок switch


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


Since 0.0-3196 ClojureScript has been specifying Clojure 1.7.0 betas. (See https://github.com/clojure/clojurescript/releases/tag/r3196 .)

I know that, at a certain point, perhaps related to conditional CLJC support, but to be honest, it is not clear, ClojureScript began actually requiring Clojure 1.7.0.

I don't know if 0.0-3211 works properly with 1.6.0.

Does Heroku itself support only specific Clojure versions? Perhaps we should wait until 1.7.0 is released?


convierte esto a Map, inicialízalo en la clase, no en el constructor.


Esta clase debería ser un singleton, busca en internet y cámbialo.


Very nice commit mate! ;)


Espantoso nombre de variable.


В веб консоли мелькают 500 ISE на запрос GET в случае, если не поступили новые сообщения. Явно должно быть что-то другое.
В остальном всё работает и к коду замечаний нет.
"Отлично-"


@backender Fände es im Allgemeinen noch gut wenn wir für die wichtigsten library funktionen wie hier z,B. "getDirectoryContents" noch die imports reinnehmen (wie im Kapitel Socket receive), was meinst du?


아 폴더이름도 바꾸었는데 인식못하네


return "";
eso es menos misterioso


What does `term-char-mode` and `term-line-mode` do? You can read [this section](http://www.gnu.org/software/emacs/manual/html_node/emacs/Term-Mode.html) in the manual. `multi-term` uses `term-char-mode` but with [some added key bindings](https://github.com/emacsmirror/multi-term/blob/master/multi-term.el#L343). It is confusing even for experienced people; you can see [this thread](http://www.reddit.com/r/emacs/comments/2jmpzo/multi_term_is_crazy_this_is_not/). But we can easily add those key bindings to `term-raw-map`. At least we should not change how `C-n/C-p/C-r` works in terminal.


initiial => initial ;-)


sdfkhohoshdpf´s
]fsdfsdfsd


dfsdfs


Dónde está el finally para cerrar ese socket en caso de error?


According to wikipedia, it probably should. http://en.wikipedia.org/wiki/%22Hello,_World!%22_program -- maybe we should tweak the spec.

Thoughts?


y dale con el gu


Pas encore fini, faut finir la class MatiereProf.


I wasn't able to reproduce this issue in a little sample test that i built, but i'm fine with it being moved :+1:


BILLEDE

TEKST

Brug eventuelt det her, men læs op på Bootstraps Grid inde på deres side.
http://getbootstrap.com/examples/grid/

Det er cool ;)


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


Всё супер, но не хвтает офрмленной связи между таблицами. Нужен внешний ключ (Foreign key).
Доделаете - будет "отлично"


Yeah, I think it probably should. I'll open an issue for it, since it's going to require shotgun surgery.


Jeg har skrevet kode i kommentaren, men den bliver ikke vist :S


This will break the Rakefile for everyone who is running in development mode but isn't using Vagrant. You probably need something like:

```
if rack_env?(:development) && !(CDO.chef_managed && ENV['CDO_CHEF_NODE_NAME'] != 'vagrant')
```

(This applies to all of the changes below)

Also, please don't comment out code -- just remove it. Git remembers everything so it's easy to look at the Git history to find code that was removed, but leaving commented-out code in the codebase leaves future developers wondering - "Why was this commented out? Is that a temporary change? Is this something someone was experimenting with and didn't intend to commit? Is this code supposed to be enabled at some future date?" etc.


Thanks!


Danke sehr. Muss dass jetzt nicht auch noch im 2.0.x-branch nachgezogen werden für ein Release?


I agree, it should be a feature test for flexbox, and is an error on mi side to assume that it will be supported on newer versions of IE.

Now, without animation, flexbox is supported on all browsers I tested, except for IE9. But the problem is when I try to animate `flex` property.
On IE `flex` properties are not recognized as animatables.
And now, on Safari I'm having some strange behaviour... I'm able to get the `animationend` event, but the `flex` properties are not changing...


Ok. What about then sending bins, calculating speeds and other stuff which is being done with captured packets? Should it be done independently for each stream or just being considered together ?


I just added you. Thanks.

On Fri, May 22, 2015 at 4:06 AM, T.B. Stan <notifications@github.com> wrote:

> Also, could you please add me as a contributor to the repo? GBY!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gwenf/Jeopardy/commit/c7391b2d96fcb9d8d4b0f4875b9afeb1243cc669#commitcomment-11322607>
> .
>


restored in d5412498b3e21d02468cc5e9eb9c6acb60f31696


On Fri, May 22, 2015 at 06:47:37AM -0700, Shinnok wrote:
> Enclose distinct statements in parentheses so it's easier to understand, even if you got the operator precedence right.

Just a minor note on style: for things like a == b && c == d, it is
preferred to not write parentheses. There is a side effect from
unneeded parentheses: they may hide warnings about assignment in
conditions: "if (a = b && c == d)" gives warning, "if ((a = b) && c ==
d)" does not give the warning. Additional parentheses is a mark for
compiler that assignment is intentional.

Though I guess it is ok to write parentheses to show precedence when
&& and || are in one expression.


Yes but then the condition of the second change can be adjusted. I only really noticed it because the condition of the code that adds the button and of the code that processes the button are different which is odd.


This is the upstream CS line I'm talking about:
https://github.com/apache/cloudstack/blob/master/ui/scripts/accounts.js#L1623


restored in 5a5701b25430166e1e4892aac325d422cf9691c0


needs fos:elastica:populate


Wouldn't `m_value = b;` result in the same thing? If yes, do it in the initialization list instead.


Thanks, corrected and committed to https://github.com/tshiran/drill/tree/gh-pages
[gh-pages a050899]


I don't understand this operator. Can you give a code example?


Thanks! I appreciate that :smile: .


:+1:

Now this file does not contain any logic. Can we remove it? It is added [here](https://github.com/isubit/luggage_resources/blob/master/luggage_resources.module#L37).


Why does this belong in entities shouldn't this be either in rendering or in screens?


@renatocron, I get the use case, I'm not sure I like the change.  The ability to list all keys in a bucket and process them in a semi-streaming fashion is nice.  Why do this when you can easily use list_bucket and just pass back the suitable marker?

Furthermore, if we're going to go down this road why not allow the callback to abort the list of the bucket?


Thanks, corrected--pushed to https://github.com/tshiran/drill/tree/gh-pages: 3cbb00c..a050899  gh-pages -> gh-pages


works.  please review my changes and merge.


Please don't rename functions to slightly different names which mean the same thing. This adds churn and it's not necessary.


@stevendanna yes, unless they specify an API key and settings in their manage.rb.


Published to http://ui.edev.d2l.com/57_6e09d4c5a5262a8c6c396de5d925d05b64a3e892


Wording.


Isn't this overkill? Unless, you're planning to use it more in the near future.
I would choose to define the following 2 classes directly and be done.


Please check this.


internally I would use a better name then `vent`, e.g. `event` (to abstract away which lib is actually used)


this was not correct


Thanks!


Сравнение дат через like - не очень правильный вариант. Через like сравнивают строки. Для работы с датами и временем в SQL есть спец функции


Magistraal Wes! Nu de commit message nog in het Engels :dancers:


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


maybe even use a small proxy object which only publishes the `trigger` and `on` method (like a bridge)


Для представления даты и времени в SQL есть специальные типы данных. Хранить дату/время как строку - плохо.


yes


hehe. remember, the normal case is *musicdb* items. it's for music videos listed in the music library, the info is in the video database. earlier you couldn't delete anything in music library except those music video items.


Если изменить тип данных для даты на соответствующий ему, то такое сравнение не "прокатит".


Changes made and pushed to https://github.com/tshiran/drill/tree/gh-pages.


I can confirm, the issues I had previously, with SimpleSAMLphp and metarefresh timing out when downloading larger XML files, in this case the InCommon federation data, have disappeared with the updated version of the library. This really saved my day. Thanks! =-D


Published to http://ui.edev.d2l.com/58_0b777678726f5a90402eff9ef455dedb03f7ebc4


Hmm,

I when I used it, I just did it to list all, abort was not an feature I was
needing, but it could be implemented.

I put the callback, because is just a bit easy than knowing about it and
controlling the markers.

<http://www.eokoe.com>

Renato Santos
renato.santos@eokoe.com
(11) 3386-8181
www.eokoe.com
b-cam.com.br

On Fri, May 22, 2015 at 11:13 AM, Rusty Conover <notifications@github.com>
wrote:

> @renatocron <https://github.com/renatocron>, I get the use case, I'm not
> sure I like the change. The ability to list all keys in a bucket and
> process them in a semi-streaming fashion is nice. Why do this when you can
> easily use list_bucket and just pass back the suitable marker?
>
> Furthermore, if we're going to go down this road why not allow the
> callback to abort the list of the bucket?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/renatocron/net-amazon-s3/commit/9ba3713f7e8b3a4192ec72834d1133558d841316#commitcomment-11328409>
> .
>


It seems strange that code that lets a user post a single project is in a function called `project_list`.


по заданию подстрокой для поиска является слово "hello" but not "hallo" :) спишу на невнимательность


Looks good to me.


Same here.


Just a comment so you know, when you name a slot in the form:

    on_<object>_<event>()

for example:

    on_btnImport_clicked()

Qt will automatically connect the signal to the slot if it can. So you don't need to explicitly connect it at all :-). This is why you were getting the double firing.


mainly optimization purpose. I am aware of breaking the proplist contract... but it's not needed here :)


А вот здесь нужно вытянуть имена пользователей, вместе с их ID и кол-вом сообщений.
Hint: нужно использовать подзапрос или Join


I propose moving this under a "utils.h" file and renaming the namespace to `utils` so we'll use `utils::fromStdString()` instead.
PS: I know that this will require editing pretty much everything in this PR. So to save you work, I don't mind you doing this as a new commit at the end of the PR.


Оценка "хорошо". Если всё исправите, то будет выше.


Both can be forward declared.


Looks good to merge


In my change this morning, I switched to:
      <version.org.hibernate5>5.0.0.Beta2</version.org.hibernate5>

I'm not sure if 5.0.0.Beta2 is better or worse than latest snapshot.


Hoe zit het eigenlijk met die sensor waardes Henk?


TBH, it's not a very nice solution all round. Better, if it works, would be to let both copies try to set aside and fail if the Prince isn't in play. Similarly with Mining Village.

However, at the moment, I think TR etc. make sure the copied card is in play for the second play.


Yes, that is / was a bug.


I may be missing something but creating an instance variable just to synchronize on seems less simple than just synchronizing on the instance.


sisi ma gueule !


Forget this. I saw that you another commit for this. I'll comment on that instead.


Why dir-local.el must is here?


Lol i changed the password before i checked in that ;)


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


That works for me, changed.


Good call. I did wonder if something like that existed (or, indeed, if a card could be discarded from play other than at turn end).

This is an instance of where I need a triggered-ability framework - Prince wants to be told when its linked card leaves play.


Adjusted!


Daar weet ik even niks op te zeggen Weesel


GuiIconProvider(); too


Misschien dat @Lg0enga nog eens de repository gaat clonen en iets gaat doen.


This doesn't look like it will select the correct menu navigation -- we probably need to do some more work to get dropdowns to work properly with angular.


what


I'm not sure we want to use just the hamburger icon for the collapsable menu here. Maybe we do what we have in the app and use the word menu?


It occurs in one very important case - the turn you actually play the Prince and set the action aside! I'll comment


I originally synchronized here so that tasks couldn't be cancelled mid-tick from its scheduler. I don't believe this logic is preserved in your changes but it might be unnecessarily cautious.


forward declare


Wait! We're only running migrations if initialize is false?


why create an array here?


Is this expected to work with more than one key?


This add postbuild for TARGET "opencv\_**test**\_${name}" inside of ocv\_add\_**accuracy**\_tests macro while [this](https://github.com/MSOpenTech/opencv/commit/c9865593fa7449fdf767ed6ad9d06a37d24ed438#diff-a8bb897ccface5bd5e7eff5548b73f77R970) adds similar action for TARGET "opencv\_**perf**\_${name}" inside of ocv\_add\_**perf**\_tests


You may change the tab or whitespace. If you want to do is, could you provide another PR?
These changes are nothing to do with "recognize Fortran generic type bound procedures".


@Martin-Jung  Do we need `rgdal`  and `sp`?  If you're thinking for examples and such we could put them in `Suggests` so users don't have to have the packages to install this.


Add a comment to say that `sc.Scan()` is to skip the first line because it is a comment?
```
#subsys_name	hierarchy	num_cgroups	enabled
```


@fcsonline you mean swapping the call to super and the logger message?


questa cosa mi piace


We need this to now be uncommented and the following:
putPropertyIfAbsent(pu, properties, AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS, "true");

I don't recall why I commented this property setting out but its very important to be true so that the correct sequence id values are generated for "inserts".


:+1: Awesome


It's a slightly-misleading test step which could probably do with being renamed. "it should be my Play Action phase" just checks to see if the active PendingAction is "play_action".

Checking player.rb:595, the game's phase is set to Play Action.


@UniSabina

Das wir bei mir auch so angezeigt. Es kann gut sein, dass er eine eigene Library oder Klasse oder sonstiges geschrieben hat, welche das schön darstellt.

From: UniSabina
Sent: Friday, May 22, 2015 3:46 PM
To: Fallstudien/Optimalsteuerung
Cc: allgaeu
Subject: Re: [Optimalsteuerung] @UniSabina @allgaeu @schweinekrippe (1ed0006)

@allgaeu Oh ja das wäre cool, wenn du einmal nur die rechte Graphik anzeigen kannst und einmal die rechte mit der linken überlagert.
Ich weiß nicht genau was bei mir das Problem ist, ich habe es eingebundne mit

\begin{figure}
\includestandalone[width=\textwidth]{multi}
\end{figure}

und

im main:

\usepackage{tikz}
\usepackage{standalone}
\usepackage{pgfplots}
\usepgfplotslibrary{groupplots}
\usepgfplotslibrary{units}

allerdings sieht es alles andere so aus wie bei Sebastian, Es ist bunt, nichts ist dort wo es sein soll, also alles ist ein bisschen verschoben, ohne dass ich etwas geändert habe.

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


Likely not. I was just copying the general form in this folder, but an Outline would work well here.


one line


в случае ошибки у вас не происходит ничего.
ни записи в лог, ни в System.out/error

это плохо


Update from original


Looks like maybe you want nested `when` clauses?


Can we revert d52d3ca given this fix?


also causes an issue with the current version of the debian cookbook  [https://github.com/reaktor/chef-debian/issues/16]


That's easy enough to do, but there still needs to be the following changes:
1. Output of `wget` and the `build.sh` script needs to be squelched to keep 'build-stage0' from getting spammy and maintain the look of a routine `butch` installation.
2. The `wget` call needs to be changed from '--no-check-certificate', as that is a GNU-ism that breaks things.
3. `BUTCH_BIN` needs to point to a complete path, the idea is that any script that sources 'build-butch' will get that `BUTCH_BIN` variable returned with a valid path that can then be used by the calling script.
4. `build-stage0` should copy the `butch` binary into place, not `build-butch`. `build-butch` should assume nothing about where the binary will be used.
5. Everything has been properly indented for readability.


This is not guaranteed to work. The order that keys are iterated over may vary from time to time.


Soooo much nicer.


Ah dang. I'm too much used to work in the video library. I'm gonna shut up now ;-)


Might also be a good idea to improve our testing on this. How come the only failure we got was a timezone test on Android!?


This is not guaranteed to work, for the same reason as above.


Cela ajoute également la fonctionnalité pour supprimer des blocs via l'interface.


Оценка "отлично-" за невнятную обработку ошибко и отсутствие вывода отладочной информации/сообщений об ошибках в log или консоль.
// В след раз не комментируйте это.


![trap](http://norayr.arnet.am/files/2015-05-22/oberon_trap.png)
![changes]http://norayr.arnet.am/files/2015-05-22/warning_307.png()


I deleted the commented out tests in my jipijapa change today.


I've replied on main chat.


I spent an hour or two figuring out what was going on. Basically every other message would have the body decoded, every other not.

Have you ever had a bug where it goes away when you add logging? It was one of those cases. I added console.log stuff to try to track down and the problem went away.

Eventually I was able to detect that `isHeader.test` was returning `false` even though the regexp matched. This fix is a bit cargo cult-ish but I hope it's fine.


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


Could be. At the moment the distinction between entities and rendering is fuzzy anyway. Maybe we should merge these packages or make a clear distinction


Good call.


https://www.drupal.org/coding-standards
have you seen this?
please make your code readable


If someone wants to override this in their CSS they can easily do this. The default should be `display: none`


Is this expected to work for more than one key?


Thanks. Will remove it now and close the coverity issue.


@UniSabina

Ich schaue mir das genauer an, sobald ich Zeit habe. Ich muss mich noch etwas um das Automodell kümmern. Ich möchte nicht etwas vorstellen, von was ich keine Ahnung habe. Natürlich erwähne ich nicht genau wie es aussieht, allerdings möchte ich auf nachfragen reagieren können.

From: UniSabina
Sent: Friday, May 22, 2015 3:46 PM
To: Fallstudien/Optimalsteuerung
Cc: allgaeu
Subject: Re: [Optimalsteuerung] @UniSabina @allgaeu @schweinekrippe (1ed0006)

@allgaeu Oh ja das wäre cool, wenn du einmal nur die rechte Graphik anzeigen kannst und einmal die rechte mit der linken überlagert.
Ich weiß nicht genau was bei mir das Problem ist, ich habe es eingebundne mit

\begin{figure}
\includestandalone[width=\textwidth]{multi}
\end{figure}

und

im main:

\usepackage{tikz}
\usepackage{standalone}
\usepackage{pgfplots}
\usepgfplotslibrary{groupplots}
\usepgfplotslibrary{units}

allerdings sieht es alles andere so aus wie bei Sebastian, Es ist bunt, nichts ist dort wo es sein soll, also alles ist ein bisschen verschoben, ohne dass ich etwas geändert habe.

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


...which would be because I coded to the tests.

I'm unsure if this is ironic.


This will only work with jQuery. The plugin was designed such that jQuery was not required.


After I deleted ScannerTests that don't compile, I deleted this class since I had deleted all use of it.  If its still used, you can keep it (depends on what you have to delete to get the tests to compile.


Agreed, this fine grained locking is not super-important here.


I believe it is :)


Just the same but for accuracy tests. Kept initial conception to set target properties inside of corresponding macro. Do you suggest to move this code to external function/macro now?


`initialize` includes `run-migrations`


But I re-introduced this so that, when squashing the commits for the final merge, this line (and the related ones) will not be marked as changed.


Should be `[]` instead of `new Array()`.


Quelques bugs sont encore persistants, notamment avec le système de sauvegarde (Des bugs se produisent lorsque lors d'une même session, on ouvre plusieurs fois un/des projet(s)).

Dans un premier temps, je vais finir de développer toutes les fonctionnalités restantes (à savoir pouvoir ajouter des blocs et des composants via l'interface), afin de pouvoir sortir une version bétâ du logiciel.


Much obliged :+1:


Should be
[
[f_1(D_1),...,f_1(D_N)]
...
[f_J(D_1),...,f_J(D_N)]
]

right?


help property shouldn't be here.


commands property shouldn't be here.


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


iiuiuiu


Err, sorry. I accidentally deleted my comments. Rewriting it here for posterity. But I'm cool with your explanation:
I may be missing something but creating an instance variable just to synchronize on seems less simple than just synchronizing on the instance.


doesn't it break the slurm scripts for the mesocenter or any other place using slurm ?
it should be probably specific to each architecture/machine no ?


XML2JS is used for https://github.com/noodlefrenzy/node-amqp10/blob/master/tools/genDocs.js - but that's a tool I was only really using during initial writing, so I'm fine dropping the dependency (had been considering doing so already, and what better time than a major version bump).


I originally synchronized here so that tasks couldn't be cancelled mid-tick from its scheduler. I don't believe this logic is preserved in your changes but it might be unnecessarily cautious.


I thought about this more... The slot offsets just go in order of the names in the tuple, so I think the other conditions (including those in `equiv_structs`) combined imply that the slot offsets are the same in number and in position. (Checking the tuple is just necessary to make sure the names are the same and in the same order.)

One other thing I just remembered is that in Pyston, we use `attrsoffset` instead of `tp_dictoffset` sometimes. So I think we want to replace `(a->tp_dictoffset == size && b->tp_dictoffset == size)` with

```
(a->tp_dictoffset == size && b->tp_dictoffset == size) || (a->attrsoffset == size && b->attrsoffset == size)
```


I think there's a mistake here (and in my original code). The boolean should be "true" on the first successful cancel, and false on subsequent, redundant cancels: https://github.com/akka/akka/blob/ebc39ef9ab4440b98fe38b4994f8641d48128ee8/akka-actor/src/main/scala/akka/actor/Scheduler.scala#L160-L161


It would fail anyway. If even one lane meets the criteria, then `CheckDependenciesSlow` runs instead of the normal dependency checking to do dependencies for all lanes, effectively breaking all lanes that use dependencies. The old version just wrote a log message; this one fails louder.


We need to check `attrsoffset` is equal here as well.


I am merging this now - awesome collaboration all! Looking forward to seeing more refinements as the team from Pasteur work to add some of the Galaxy datatypes to EDAM. Their progress can be tracked here https://docs.google.com/spreadsheets/d/1vptTpJvQfb5FOtRJOajjftUkL9aZ0eeVdhUPvd9h5wY/edit#gid=0.

The current mapping (galaxy extension to EDAM format) at the time of this merge is https://gist.github.com/jmchilton/0f559033430824f157fe.


`CancellableTask` uses an `AtomicBoolean` for tracking cancellation.  Wouldn't this be sufficient for your purposes?


if you don't provide those things, it will asynchronously figure it out given some routes, and then you'd assert in `onChange`, but you can also just pass it a bag of state as props, primarily for server rendering where you need it to render synchronously. Seems convenient in tests to do the same.

I'm not sure I should even have this example, its only encouraging people to test a component we already test!


Ok gar kein Problem, ich frage mal Sebastian deswegen wieso es bei uns so bunt erscheint.
Zur Not kann ich es immernoch an die Tafel zeichnen


I don't think ownership is actually transferred.  You want `g_clear_pointer(&tempdir, g_free)` here, right?


Why don't you fix this function?
If you don't have an answer because you just import the commit from geany, tell me so.
We have to discuss how we should do. Just importing is nice to track the original tree but as seeing quickly keeping parseGenericDeclaration is not a bad idea.


There's a file lock. See line 190 below.

    cappender.LockingModel = new FileAppender.MinimalLock();


Well spotted. :+1:


No it should not, the number of cores are only changed when we are on atlas.
The default number (16) will apply on any other configurations and the module section won't appear.


Oh, I see.  You mean mid-tick here:

```java
        if (!head.isCancelled) {
          val task = head.task
          task.runnable.run()
          task.interval match {
            case Some(interval) => tasks += CancellableTask(task.copy(delay = task.delay + interval))
            case None =>
          }
        }
```


Daar heb ik een hard hoofd in @henkmollema. Echter, iemand moet ook de pannekoeken bakken.


You got the logic a bit wrong from the original.
If `if (!QFile::exists(path))` returns false it should also `return path;` and not `return IconProvider::getIconPath(iconId);`


Hey, why not merge #7 and add Contributors section somewhere? Would be more "professional" :wink:


Get rid of this `delete`. I'm surprised it's not a syntax error.


:smile:


We don't need this:-P


maybe it is not required


Which semantics would you need for your use cases?  Is it relevant/meaningful to prevent mid-tick cancellation?


颜色 改成黄色  会比较好


An R function that can compute and cache the inversion of a matrix


@millerdev Ah. Crap. The autocomplete choices aren't handling the case when you select a nested element.

Easiest way to fix that is on selecting something from the dropdown, find it in the nested structure and populate autcomplete from there.


Scala. Ну ок. На жабке написал реализацию `@user` через плагины.


perfect! Yepp, I never really liked "strict", I think IMO `validate` probably makes the most sense out of the three options we talked about so far :)


can be local now?


![Batman approves](http://media.giphy.com/media/5DQdk5oZzNgGc/giphy.gif)


Maybe you want to use a more stable link, such as https://github.com/git/git/blob/v2.4.1/Documentation/SubmittingPatches#L4-7 or some such?


You beat me to it!

I believe we can remove it, but I see that there is a `drupal_add_css` in the `luggage_resources.module` that references this file and wanted to do some more investigative work.


TeamCity xBIM :: Xbim Geometry :: Release [Build 62](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1669&buildTypeId=XBIM_XbimGeometry_Release) is now running


Why remove the autoload cookies?


This is fine.  When we have a million tests I believe there's a way to select which ones you want based on Java annotations.


Обычно log4j.props кладётся в src/main/resources
WEB-INF/classes/ служит для других целей


See #4


Nothing more awesome will happen today, this is it <3


ok, either way MDK tools are the only ones we'll be testing against so we've removed the old RVDS as it isn't likely to work as-is.


Thank you for the test (just 2 very small inline comments)


This works?  I thought the name of the class was BoardTests.scala.
(Sorry, I don't have time to test right now.)


Ship it!


Because it was removed. Red means removed, green means added.


Is it needed / best-practice to free the buffer before you exit? I would have thought that the resources that you assigned are freed anyway.


Experience.


Your choice I guess; true enough, the JDK has no equivalent to Preconditions.check{State,Argument}(), nor printf-formatted strings.


Absolut, sehr wichtig di verwendete libs z ergründe. Good catch!

> On 22.05.2015, at 16:06, Lorenz Wolf <notifications@github.com> wrote:
>
> @backender Fände es im Allgemeinen noch gut wenn wir für die wichtigsten library funktionen wie hier z,B. "getDirectoryContents" noch die imports reinnehmen (wie im Kapitel Socket receive), was meinst du?
>
> —
> Reply to this email directly or view it on GitHub.
>


No; that is the transpose of what it does.


Should throw an error instead of returning null, no?


haha :+1:


Yeah, I'm down with that.


This kind of test is useful, but does not belong in the suite being run by the autotester.  There should be near zero chance of getting a random failure.  If it ever fails, the thresholds are off by several orders of magnitude.

There are a few permanent solutions to this problem:
- Turn them into soft failures: print a message, or loop until it passes.  If it keeps happening someone will notice eventually.
- Pre-select a bunch of seeds and assert that the output matches the expected - this will check that the algorithm isn't accidentally changed without the possibility of failure.
- Disable running these tests by default


Because I just blindly reverted the whole patch (see #6) rather than cherry-pick. I'll fix that.


This check should either be elided or throw a TypeError.


Nice work! I like that we only have to change the version number once. I will update the build docs.


You should still remove the indent ;)


@wonzhq maybe i am completely wrong and missing something. but following is what i have in my mind, i am using the some of the C++11 syntax to simplify the for loop, but the idea is that we just need to check the shared part of the single op and pipelined op using the loop.
```patch
diff --git a/src/test/osd/RadosModel.h b/src/test/osd/RadosModel.h
index f9c0f9b..b814c59 100644
--- a/src/test/osd/RadosModel.h
+++ b/src/test/osd/RadosModel.h
@@ -1066,19 +1066,33 @@ public:

   void _finish(CallbackInfo *info)
   {
-    context->state_lock.Lock();
+    Mutex::Locker l(context->state_lock);
     assert(!done);
+    assert(waiting_on);
+    if (--waiting_on) {
+      return;
+    }
+
     context->oid_in_use.erase(oid);
     context->oid_not_in_use.insert(oid);
-    assert(completion->is_complete());
-    uint64_t version = completion->get_version64();
-    if (int err = completion->get_return_value()) {
-      if (!(err == -ENOENT && old_value.deleted())) {
-       cerr << num << ": Error: oid " << oid << " read returned error code "
-            << err << std::endl;
-       assert(0);
+    for (auto completion : completions) {
+      assert(completion->is_complete());
+      if (int err = completion->get_return_value()) {
+       if (!(err == -ENOENT && old_value.deleted())) {
+         cerr << num << ": Error: oid " << oid << " read returned error code "
+              << err << std::endl;
+         assert(0);
+       }
       }
-    } else {
+
+      uint64_t version = completion->get_version64();
+      if (version != old_value.version) {
+       cerr << num << ": oid " << oid << " version is " << version
+            << " and expected " << old_value.version << std::endl;
+       assert(version == old_value.version);
+      }
+    }
+    {
       map<string, bufferlist>::iterator iter = xattrs.find("_header");
       bufferlist headerbl;
       if (iter == xattrs.end()) {
@@ -1107,7 +1121,7 @@ public:
               << ", expected " << old_value.most_recent() << std::endl;
          context->errors++;
        }
-       if (!old_value.check(result)) {
+       for (auto result : results) {
          cerr << num << ": oid " << oid << " contents " << to_check << " corrupt" << std::endl;
          context->errors++;
        }
@@ -1138,11 +1152,6 @@ public:
             << " and old is " << old_value.attrs.size() << std::endl;
        assert(xattrs.size() == old_value.attrs.size());
       }
-      if (version != old_value.version) {
-       cerr << num << ": oid " << oid << " version is " << version
-            << " and expected " << old_value.version << std::endl;
-       assert(version == old_value.version);
-      }
       for (map<string, ContDesc>::iterator iter = old_value.attrs.begin();
           iter != old_value.attrs.end();
           ++iter) {
@@ -1188,9 +1197,11 @@ public:
        }
       }
     }
+    for (auto completion : completions) {
+      completion->release();
+    }
     context->kick();
     done = true;
-    context->state_lock.Unlock();
   }

   bool finished()
```


There's no need to remove this.


A handy little function for `rpmostree-util.h` might be:
````
gboolean _rpmostree_mkdtemp (const char  *template,
                             char       **out_tmpdir,
                             int         *out_tmpdir_dfd,  /* allow-none */
                             GError     **error)
````


Oh wait, did I revert the wrong patch?


So J == N?


@nvbn: Top notch! :+1: Thanks!


I'm on it :) I'll have time to port them in about 10 hours.


Revert back as Travis is not using Python3 for system site packages


It doesn't cause a problem on the main site but I'm rebuilding the ios phonegap app and need to pull out xdomain and other config. That's when I noticed it. I will send another pull request to inline. I already made the change but will make sure it works. Thanks.


Seems to be fine.

We can also add method `getState` so we're able to get array of `__state` too


ah shoot I see the typo now. Good reading.


Klopt.. Je kan altijd nog http://www.inholland.nl/Opleidingen/Facility+Management/Voltijd/ volgens @Lg0enga


thx i'll try


The sign of the PID command is inverted here!. See #40. This method is clearly not exercised by the unit tests.


这注释能不能更随意?!!!


Ooops. Missed that. Will do.


Ok, revert reverted, I'll use my brain next time I revert a patch :)


This pattern seems a bit wild. What value is it adding here?


(yours is wrong too)


Would be nice to say that you can unsubscribe at any time (which is true given they are using mailchimp?)


Better to compare against `compress_into_pos_notpos.__name__` rather than string literal?


`vent` seems to be used everywhere, I don't know why.


Сань, закинь мне лабу мою 6-ую, которую я у тебя на компе сделал. А то не
кайф переделывать


Not really, I was just being cautious. Don't think it should be a problem.


Rather than check the exact count, check that it's > 0. The actual hitcounts aren't exact and tend not to be stable over time. I've considered reducing output to three states: {<none>, 0, 1}, but haven't done so because there's still some value in approximate values.

For an example of, have a look at the hitcounts on line 83 vs 84 here: [quiver/lib/src/iterables/merge.dart:83](https://coveralls.io/builds/2591099/source?filename=lib%2Fsrc%2Fiterables%2Fmerge.dart#L83)


It'll ship itself as soon as Travis gives it the thumbs-up :)


yeah I don't think this change will cause problems. I had originally hoisted it out of the function since it seemed benign. But obviously that is not the case :disappointed:


fix pushed.


The `Channel` type should probably go in `src/lib/channel.sjs`.


Yes; I will add a note as a TODO. I am planning on using has_attr later. COmment with note pushed.


OK


Consider adding a test for an uncoverable line, e.g. line 28


Что-то у меня в браузере ваши странички с ошибками не отобразились.


С учётом этого оценка "хорошо+"


javaSSSSSScript ?? :whale: :whale: :whale2: :whale: :whale2:


I feel that this method cohesion is strange.  Its doing both the RequestParser init and arg parsing in one call.  Why not have two separate calls?


Should we use test classes here?


Likewise here, re: test classes


This is a breaking change for our automation, you can't assume it will be run from a relative path!


I feel this is a better approach. I had them split. This function is a one stop shop that takes the POST request, parses the arguments, and returns an object. I feel there is no reason to split this into two calls.


final del archivo


I'm confused.  This comment says `encoder_kwargs` is a list but up in L36, you comment that its supposed to be dict and usually in python kwargs is a dict.  Which is it?


can you release latest version with django 1.8 and py3 support?


this won't work on Linux unless CMAKE_INSTALL_PREFIX is set to /usr, because CMAKE_INSTALL_PREFIX defaults to /usr/local and usually those executables are not installed on /usr/local/bin


It'd be more readable to use the replynames here instead of the numerics.


Maybe:
`@Deprecated('See packageRoot. Removal in 0.7.0.')`


Sorry; the comment is outdated. This PR changed it from a list to a dict. Pushed comment fix.


That may be. But I don't know. I thought about that, but I wasn't sure, so I wrote l.clear().
I try to implement your suggestion later.


Given the followup commit reverting to 0.6.3+1, drop this one.


This update should make upgrading to io.js unnecessary for now. I talked with slayer and he implemented some support for legacy systems running node.js 10 - 12


Should be Write a media player using QtAV quite easily. Please notice the typo in "Write"


(Y)


looks like this change was unintended.


I would rather create a wrapper around XmlDoc with 3 constructors.


are you sure this really search share/xsessions under the prefix?


Looks like this change was unintended.


Like I think this is the lie


@h-kataria Captured Dialog name prior to session cleanup. Ready for review.


Ya I need to rebase the fork...


Stacking classes on "row" elements could have unintended consequences. Can we pull the img-logos down one level into a wrapping div or something?


# makeCacheMatrix creates a list containing a function to


line 2 long


oh I see. `vent` is not the name of the underlying lib but a name of your own var.

then it doesn't matter...


Testing the exact hitcounts makes this fairly brittle over time.


@bnorthan if you have time that's awesome, but truly there's no rush. The most important thing is that you've been contributing, which is fantastic :)


`recipe[apt]` should be first in the list because it updates the list of packages available on the system which should happen before we try to install any.


We should only be using columns as direct descendants of row elements. Putting the h3 here could cause issues for our responsive design. You'll see what I mean if you shrink this to mobile size, it just doesn't work correctly. I'd recommend pulling it above the row element.


>  Setting this to zero prints all statement durations.

http://www.postgresql.org/docs/9.1/static/runtime-config-logging.html#GUC-LOG-MIN-DURATION-STATEMENT


These paths will be machine-specific.


было бы очень крутым наличие аргументации и пожеланий.


Just `[Composer](https://composer.org)`


Personal preference, but I find the more verbose method-y syntax easier to read, e.g.:

```
validates_numericality_of :tool_width, only_integer: true, greater_than: 0
```

Up to you though


Why did you do this?


Usable is vague. Extract has a specific connotation in PHP. getCompletedRows() perhaps?


copy paste, can't it be done in mkXmlDoc ?


Remove (see note above).


copy paste AGAIN. Please factorize.


Looks like you're moving commonlib to a commit that's not available, which is causing the CI failures.


@JiggyPete pulled me up on this the other day, and suggested that I might want to consider [this sort of approach.](https://github.com/moneyadviceservice/rad/blob/d25d4a8b2dfcd4b85db46e92d40baec869c0840b/spec/features/principal_completes_firm_questionnaire_spec.rb#L55)

(I haven't done it yet)


yup ; Ive suggested before that this should be externalizable, so that this change doesnt get accidentally merged in (we all do it in dev).  Can we pretty please externalize the "kubectl" variable?


Yeah I ended up doing that in https://github.com/moneyadviceservice/frontend/commit/8d048b8e14fa65c4c8005a1a42c0f2fd79b72587


I :heart: `is_expected`


Instead of setting TERM, this should set:
ENV DEBIAN_FRONTEND="noninteractive"


This is the lie


It looks like that will open and close the file for each write (and contradicting your commit message "Hopefully better performance. The current logger opens and closes the file for each line; log4net shouldn't need to.").


Perhaps just roll this into a single RUN with the following statement then?


Is there a reason we can't do `subject { create(:partner) }`?


I would also add set -u (exit if eval undefined variable) to enhance runtime checking/safety.


So in addition to rendering server side errors when JS is turned off, Justin set up client side error validation which happens on the fly.


This line contains a typo / potential bug. You should revert this change back to `eZInputValidator::STATE_INVALID;`


I think this can only complicate things with no added value.
What's the reason for it?


Calling upon Code Complete (:P), there's a section on "Design at the Routine Level" and there's a point about communicational cohesion.  I think this method is an example of this and should be split up.

I'll let you make the judgement call.


wrong alignment.


Gotta ask...why is there a `None` here?


ja hesch eigetli recht :) strub dasses kein java fehler git^^


I'd like to roll with this. That is because I think a lot of the transformation code can be done as a keyword to "add_argument" that takes a callable, which I plan on doing later. But I like this better than split.


I'm not too fussed tbh, just tried to keep it consistent with the other validation lines.


I always hate doing this extraction of wild strings to locales stuff. I salute you


`==` is not classy when you write CS.


Just while we get the testing sorted out, so that Travis isn't running broken builds on all the branches simply because the setup is broken.

The idea was to seclude it to a travis branch, iterate on the travis configuration + integration tests, then merge it with master and re-enable across all branches.


@piscisaureus, don't worry about it, we figured it out: https://github.com/joyent/node/issues/25348#issuecomment-104688866

Thanks for this fix!


Because "None" is a kernel argument for the SVM. It is an acceptable argument.


Thanks! My habit of committing with `-a` also added the local Gemfile changes. I've force pushed the intended commit.


Good call. Shave a few lines :+1:


okay sorry, ha nur de code ufm tablet im bunker agluegt gha^^ ned au na asana


Creating a default service account (the actual create call) can take a few seconds. (I've seen up to 6). That's longer than the test wait time. I suspect this is related to the rate limitter. I've bumped the time up and fixed gofmt.


Consider making a single logout endpoint that works with both ajax and non-ajax requests. Django has something you can use in views to differentiate between these requests (See: https://docs.djangoproject.com/en/1.8/ref/request-response/#django.http.HttpRequest.is_ajax).


Indentation is a bit off in this section,


Remove uri, cb params.


@hinerm He already did it, actually: #129! :trophy:


Yeah Faker is pretty cool. I just didn't want to include a whole separate gem when `sequence` worked fine here. But down the line, it may be useful.


<b>Для потомков</b> =)
Дело оказалось не в таймауте.
Просто ветка со статическими примерами в тот момент была отпочкована от мастер-ветки с недоработанным датапикером.
После мерджа ветки с актуальным мастером всё стало ОК.

Свежий пример статического (безангулярного) датапикета - http://lexnekr.github.io/UCA-JS-UI/examples/DatePiker-static.html.html

коммит - https://github.com/lexnekr/UCA-JS-UI/commit/cc6bd86e46d2cefdf3eeccb4de0b235c718b4e51

Вопрос закрыт.


It's just one line shorter than writing

if kernel:
    assert kernel in _SVM_KERNELS


(groan)


This sort of stuff is like washing the dishes to me. Hate it, but gotta be done.


Btw, my commonlib pull is now at https://github.com/mysociety/commonlib/pull/38


hmm joine tuet mer ja eigetli nur uf DB level und döt bruchts denn ebe d id.. weiss ned gnau was du meinsch mit joine uf java level. hibernate luegt ja denn ebe uf db level wiener das chan joine und du chasch nocher in java dich dur dobjekt hangle ohni dass du muesch öppis joine.. hesch denn eifach es signature objekt inere person z.b. und chasch denn döt ganz normal drufzuegriffe ohni dass na öpis musch joine.. velech luegemer sus das mal zeme ah


Things like valgrind will complain about leaks if you don't.  This probably isn't the greatest way to do this, but it was minimally invasive at least.


this is tuning of #2


Individual feedbacks don't seem to have the same final, notFinal functionality as the pre-exisiting types. Probably makes sense to include this.


How about creating an option `"none"` instead?


I think `window.require` is still available? We set it to `System.amdRequire.bind(System)` is JS later steps. If it's not there anymore, it's been overridden by something :unamused:


I seem to have uploaded the wrong version. I'll address this as soon as I can get to my other computer.


So we never needed the `exports` for all of those?


I had check my file /opt/graphite/webapp/graphite/render/functions.py and I have already this line. This is why I don't understand the reason of my issue.


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


> So that Travis isn't running broken builds on all the branches simply because the setup is broken.

This seems to me as to much detailed optimization. Moreover it's temporary state.


Since the last commit moved Config to DI, ie, `getInstance()` should use `StaticContainer`, setSingletonInstance had no effect anymore.


On one hand, it's a bit simpler. On the other hand, the reason it's structured this way is so with a teensy bit of further effort, we can fan out isolates and generate this stuff in parallel. I'm fine with this change in principal, though eventually I'd like to see if we gain anything from spinning off isolates.


I don't. It occurs to me that I should understand the return value better, and then return something more useful than the hash reference it does return. Private messages have a permalink just like any other type of comment, so, I might as well return the permalink ID like the post/comment functions.

I'm new to unit testing too, so, I suppose I'll test the return values and what happens if you put in various wrong information? Sorry to be so ignorant of these things.


I would probably rename this one to ExpectedValueStates


@jv110 is better like that (what you added was wrong located) (I'm saying bad text position)


For whatever reason this bothers you; I've pushed a fix.


hooooooot


Since we're working at the syllable level (at least for vowels), the endings will need to include the diphthong in the nominative ους


Thanks for the patch!


added


So, this change is a pure dead-code removal. The if [ 0 = 1 ] block is obviously dead code. As I mention in the commit message, the block in nova_start isn't doing what you think here, either. For whatever reason, list-migrations doesn't list... well, migrations :/ It only shows resizes.

Look for a separate email where I argue against pacemaker handling migrations at all. This specific patch, however, is pure non-functional code cleanup.


added too


It's true — just thought this pattern was a bit full on. Maybe Dough should supply one. Anyway, not a big issue.


Yes, looks more of a ctrl-C/ctrl-V flipped sign, deeply sorry.
#41, definitely.


We need to include both letters of the diphthong in the endings, so the nominative ending should be αυς


It wasn't wrongly located, just hardcoded, I was trying to find how to get screen size


ja uf java level isch scho nöd würkli joine. Han eifach nöd gwüsst gha wiemer denn mit hql uf dvariable vu denä objekt gaht. Wennmer ds prinzip aber mal verstande hät isches eigetli zimli klar. Hans ez ja usekriegt und es funktioniert,


This is the ppt on slicer coordinate systems

http://www.na-mic.org/Wiki/images/3/3f/Coordinate_Systems_Demystified.ppt


don't feel like we need the parenthetical here.


I don't think we need the 'regardless' portion here either.


hmmm....


Yep, noticed that later on — definitely a good change. Though I think the standard for test classes follows `.t-[a-z\-]*` — see https://www.moneyadviceservice.org.uk/en/styleguide/css


Neat! :+1:


Too long between writing the code and the commit message :) Would you like me to update the commit message?


Same comment as lcov formatter.


Might want to add a travis CI badge to the readme.


[It's your lucky day!](https://github.com/moneyadviceservice/frontend/blob/cecc52f086589f197a75b804c3ebe067b6c2d62a/Gemfile#L89)

Again, though, I've not researched it myself yet!


I like the technical explaination on how it work, but I also would like to see explaination of what the example parameter do. i.e "when user type a pizza recipe, the list of ingredient of choice is automatically populated via a custom REST api"


misses a newline :)


nope, this is generally considered "safe"


@andrew I ran optipng over all the images that get used:
https://github.com/librariesio/pictogram/commit/b1f8a422ac7f5253bac51a5ae55070f80a333e27

and added the settings I used to the npm srcripts: https://github.com/librariesio/pictogram/commit/4ff7434491fbf85ba49aed5e4c648630af3a038d

I deliberately left the files that are marked have the `.orig` in the filename as that is the exact image that was plucked from the internet.

Can you drop a note on what settings you used. I played around with the optipng settings to try and get the best results vs processing time.

Obvs this would all be whizzier as a spritesheet... sounds like a job for picto. I'm all for keeping the repo as a resource of individual icons, and then compiling out optimised assets for libraries.


You seem to use 4 spaces indentation. That's against what your `.editorconfig` states.


Hi thnx for the pull request.
Never knew the command list need backticks.
Do they?


@willmot  depending what it should hold of couse. If it can have HTML, then yes - this is the only thing you can do.


> I'm not keen on just forcing every table from MyISAM.

Yeah I've been thinking about this and I *may* need some table to stay in MyISAM.

So just leave it as is ;)


Because `None` is not really valid kernel type.  Thank you.


I didn't even notice it! Good eyes


these should use `esc_attr()` as they don't do it internally.


It runs all test cases without changing both pom.xml files.
Check instruction on TacocoRunner branch.


`ERR_Copy_Paste` :grinning:


Ah, I forgot we had a styleguide :smile:. Cheers


Could you elaborate a little on your concern?  I've got unit tests that verify it, so my instinct is not to pollute the happy-path with unnecessary work, since that will just slow things down.


Unless I missed something, `name` will be uninitialized when put into `std::strncat`. If this works, it's most likely due to your debug code and/or compiler setting everything to 0 before.

Wouldn't it be better to use `std::strncpy()` instead? Or how about using `std::string` right away, using `name` as the starting iterator and `name + size` as the end iterator (there's a constructor for that)?


Oops, the danger of accidentally pressing Ctrl+A in vim, which increases a number by one (B2SAFE -> B3SAFE)


Michael Grays branch has corrected this!


This seems very repetitive: perhaps it could be configured once if it applies everywhere. However, it's also not clear to me why it's necessary that the controller knows that it's smoke testing or not, nor why this matters outside smoke test runs.


Should `Assert.notNull(rel)`


Wondering if we raise the exception again will make it caught by reactjs?


a ver gato


@jensens released ;)
tnx for the support :+1:


Name the function


Do you need to provide a specific source, or is it enough to use the mirrors?


I extracted this at one point to debug. Should be inlined back in.


Eventuell lieber den StringBuilder verwenden


No new


Understood.


Formatierung noch Regeln


yeah! thx for for fixing it!


name the function


Can we add a check here to make sure the security groups are getting parsed correctly and that the values in the list are what we expect?


Neat :)


Are there other places where the hostname might be set and thus need to be checked?  I'm thinking of systemd-specific things here.


yeah, turns out they do need that. adding back


who knows! i *think* it's just jspm magically managing it. goign with that till i see some errors :wink:


The gradle version is not updated yet. I spent quite a few hours to track down this bug but found you already fixed it two month ago... Could you please update the gradle for this fix? Thanks!


@arthurvr big thanks, was writing node-modules for the first time, so will fix now. Also, i will be glad if you can  to review others :+1:


Adding required library would be appreciated.
The list is:
- libz
- libc++
- Security.framework
- SystemConfiguration.framework


can you try something like that?
```
--- a/lib/network.py
+++ b/lib/network.py
@@ -434,7 +434,7 @@ class Network(util.DaemonThread):
     def process_response(self, i, response):
         # the id comes from the daemon or the network proxy
         _id = response.get('id')
-        if _id is not None:
+        if i == self.interface and _id is not None:
             self.unanswered_requests.pop(_id)
         method = response['method']
         if method == 'blockchain.address.subscribe':

```


ah. ok. I’ll fix it.
On May 22, 2015, at 8:32 AM, Scott Fleischman <notifications@github.com<mailto:notifications@github.com>> wrote:

We need to include both letters of the diphthong in the endings, so the nominative ending should be αυς

—
Reply to this email directly or view it on GitHub<https://github.com/scott-fleischman/greek-grammar/commit/3f99323a28a78c2e59377968093110e9022cd926#commitcomment-11330239>.


Can you update the CHANGELOG ?


I agree.  This was the old algorithm just shuffled around a bit to accommodate my new code.  What I'd like to do ideally is add HMAC support to my new evp lib and then switch the whole thing over to use that, but we're kind of in diminishing returns.  My thinking was to PR this upstream with the comment that if they like it, I'd refine it along those lines.  If this lives on as just our code, we're not using HMAC now, so it's effectively dead code.

Unless you feel really strongly, I'd prefer to defer this for now.


So, as it stands, the compound and reaction dumps can now be generated directly from the master files, but there's a few things left to cover:

1) Use of compartment-free reactions (I forgot, in part, that we're generalizing reactions)
2) Addition of Cyc pathways from plantdefault database (will print these to another directory before integration)
3) Update of reaction balancing based on changes to compounds in recent curation of compounds ( change in protonation means that quite a few reactions will have protons added or taken away in order to properly balance reactions, and we will have the additional issue of changes in charge, but I'll address those when I get to it).


nice! how does one build for GLES2 though? cmake complains: https://bpaste.net/show/eb8bd33587ce on my platform


This was the existing algorithm.  (Trying not to rock the boat too much...)


I like that you started out with a declarative approach first and added apex code later.  This was perhaps by accident but it is considered best practices.  Usually non-salesforce developers fall into the trap of implementing everything with apex rather than leveraging the platform.


document method


All other data is distributed randomly in the select target goal group.
All of it random.  As such, being random, it's possible for a set of
chunks to be hosted on the local server(via single-drive mount points,
or with a raid0, which is what we used).  But, without my option, this
random gets thrown own, and the local server gets used.

Now, the other nodes on the network, that are not local, will still do
random reads from each of the copies of a chunk, so the local node won't
be hit all the time.  However, if a local reader tries to read the local
copies, the random function is not used, and the local is preferred.
Even if there is no bandwidth available for more I/O.

Until such time as actual real I/O feedback is added to the system,
doing it random would seem to have a better balance.

Additionally, using first-gen infiniband, we can get 700MB/s with
IPoIB.  This is not even using fancy verbs/rdma, which using perf test
stuff can get to 1100MB/s.  When you combine all the other nodes and all
the other spindles available in the cluster, you can easily saturate
read/write on a requested set of randomly placed chunks.  Preferring
local, means you are limited to the I/O capacity of the local drive(or
raid0).

Even using gigabit for the storage network, the set of remote nodes with
random chunk placement would have more spindles available, and more
total I/O operators per sec.

On 05/22/2015 01:43 AM, Dmitry Smirnov wrote:
>
> To me reading from local chunkserver seems like a reasonable default.
> You should not be having any issues with that unless you have an
> abnormal data concentration.
>
> Normally with many chunkservers each individual chunkserver hosts just
> a fraction of data in the cluster. Random access (and random
> distribution of data) makes it improbable to local chunkserver to have
> much of the accessed data. That's what we observe in our network.
> We also have a server with ~15 HDDs allocated to LizardFS -- at the
> moment this server have a copy of every chunk in the cluster. Local
> access also not causing any problems on this server because data is
> randomly distributed between 15 local HDDs and number of local
> (rotational) disks compensate for their individual slowness (just like
> number of chunkservers helps to distribute load between them).
>
> On local gigabit network reading from local chunkservers reduces
> network congestion and makes access faster in most cases. We do not
> have a 10 GB dedicated backbone or anything like that so prioritising
> local chunkservers works fairly well.
>
> Besides I reckon it is not a parallel read from local chunkserver
> causing problems for you (at least not directly). As I've mentioned
> local reads will be naturally distributed unless you have all (or
> most) of your data on local HDD and very few chunkservers.
> Any distributed parallel network storage needs more than few nodes to
> work effectively. I suspect that the real problem might be that
> overwriting data takes longer than necessary when local chunkserver is
> under load.
> This could be a problem for mixing slow/fast HDDs and/or slow/fast
> nodes as well -- I've opened #270
> <https://github.com/lizardfs/lizardfs/issues/270> and #271
> <https://github.com/lizardfs/lizardfs/issues/271> to address this problem.
>
> I'll do some testing with and without this option to get more insights.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Brainfood-com/lizardfs/commit/44851769216155336d06d045d858fc03d5084472#commitcomment-11321618>.
>


Do you think there is a way to make this somehow configurable from the environment so wouldn't have to modify the code for every deployment?


you might consider changing to

public Id LoggedInUserId { get; private set; }

So other classes can't change the LoggedInUserId


done and done


done


What do you think happens if you pass a full object to your serialisation strategy?


Public should be lowercase.  Also the variable names should be either camelCase or PascalCase but not both.


Why is there a `nextTick`?


That should be ``addWidget``.


das isch ned prod sondern allgemein bi normal deploy oder prod..


ditto


Why did we ignore this?
```
http://0.0.0.0:8888/static/css/main.css Failed to load resource: the server responded
with a status of 404 (NOT FOUND)
```


It is generally considered best practice to set SeeAllData to false to decouple the test case from the development org.  Also, it is also best practice (though it can be problematic) to set OnInstall to true


das füehrt denn dezue dass du das da usfüehrsch aber bem normale grunt und grunt serve wird jetzt alles vom copy job gmacht. also au dis neue localdeploy..


cool, thanks for the fix.


Only production code should go between startTest and endTest, but good that you are using startTest and stopTest :)


GLES2 is for mobile platforms. Android uses its own format of makefile. I don't know how cmake can help you.


Yes, it's a percentage of the scene diameter.


I'm going to make this release 0.6.4 since we did change some API.


Heh, interesting read.  (I went to school with Nate btw)

Since we're not using this existing algorithm I'm inclined to defer it for now, but lets talk later today.


yep, done in https://github.com/scaleway/image-app-torrents/commit/a4096c3d3f0ebadecca51a5a45529a7ac0ae2348


For future reference, Travis has a "Restart Build" button you can use for this kind of thing :wink:


has gfixt


better name for test


Why are the fees mutable?  I see that the fees updating function has been made const, but that doesn't seem motivating, unless every function that uses `deprecatedUpdateCachedFees` is also made const.  I recommend going "all const" or backing off on the mutable fees.

Agreed that the fees look like a separable class.


I don't think this can ever fail


Update 200: Grats Grz3s :+1:


asserts should go after the Test.stopTest


high level _view_?


indentation


true, thanks!


Do you like SOQL?


cool


Just starting reading the next commit.  Never mind! ;-)


all you need to do to integrate everything here is write `initQueueArray(n)` here


Newline missing at the end of the file.


This one is not needed as the code inside the module is not using any classes from there.


after this whole for loop, add
```
    printf("\n\n");
    printToScreen(x,n);
```
to see the printed results


Good catch. I'll refactor the code a bit to make sure that test runs in all cases.


me


Default doc type is html 4, so why did you change to html5?  (just curious)


If `ELSE` is used as an argument with `Run Keyword If`, the reserved keyword `Else` won't be executed at all.


Test it out with small numbers

```
int MAX = 15;
int MIN = 5;
int STEP = 5;
```

what can you notice on the results? is there anything amiss?


Tell me if there are any more questions


`Run Keyword If`, not `Run Keyword`.


Attention to comments.  Important.  Good :)


From looking at the source, you're right. I was expecting it to do something more advanced like POSIX locking, but it doesn't...


Don't be bitchy or no google summer of code for you :P


Why remove?  Can this be null?  If yes, how does your app behave?


We don't currently use average right? Are you adding it just because it's easy to do and in case we want it in the future?


I need your email address to forward them.


I'll harden these.


Building to test now.  I think I ran into this before when we were bringing up LP...


ah, good point.  sorry.


All fixed


You are searching for: `array_map`.


dsadsa


ack


Ок, не возражаю) В дб - исправлю на днях


Please add these tests:

* stats are changed after a bid matches
* stats are changed after an ask matches
* stats are not changed if there's no match
* stats are updated correctly after 3 successful trades (min, max, start, end, avreage, etc.)

You can copy and paste from other tests most of the code.


no longer working


@magcius This should be `Architecture: any` since it compiles binaries.


Oops, I overlooked this before. We don't want an assert here, if the assertion fails then we'll have a traceback in the state return. Can you please remove this or change it so that it returns an error message here? Something like this:

```python
elif ret['changes']:
    # comments are already provided where the change was detected
    ret['comment'] = 'Some meaningful error message'
    return ret
```


It is ensured that `$value` cannot contain single quotes?


I don't like this phrasing. It makes me uncomfortable. Can't think of a suggestion though... :confused:


:+1: except for my previous comment.


ok


You do realize that %zu is not universally supported


I struggled with this giving/given semantics a little bit.  The difference is one you are given, the other you are giving.  I felt we were giving the name, not given the name.