the tests
build opensuse13.2->opensuse13.2
analyze opensuse13.2->opensuse13.2
analyze opensuse13.1->opensuse13.1

are failing as they were not run before and thus need work on the test side.

If we remove opensuse13.2 build tests and change the two analyze tests to one
analyze opensuse13.2->opensuse13.1

the tests should be green. I'm currently testing this change.
Other than that LGTM


@arehman199 this was added to enable systemd config only in systemd case, not in sysvinit. If a user enables sysvinit then he'll enable other dependencies which are meant for systemd.


Sorry if I'm a little super nitpicky here, but I think this is not related to the commit message and should happen in a different commit.


This kernel is just for kernel headers so that I don't get compilation errors (due to display, audio HALs etc.). I'm still using the prebuilt kernel from MIUI 5.6.25.


I think the check should be socktype == SOCK_DGRAM.


The changes to this file were needed because I've made the HumanClientApp call the new size/position checking code in CUIWnd::SizeMove() when the app size changes but InGameMenu calculates its own size and position in DoLayout() rather than implementing SizeMove().  My changes move the size/position checking to InGameMenu::SizeMove() and makes this class behave more like other CUIWnds, where SizeMove() calls DoLayout() if necessary and not the other way around.

As for the state variables, I can move those calculations back into DoLayout() although I don't understand how member variables that are set in the constructor (and can be changed later) make dynamic layouts any less possible than having local variables that are recalculated in the same way on every DoLayout() call, unless we're thinking of different things.


http://localhost:3000/meetings/1/messages/new

undefined method `name'でみれないですー＞＜


This fails with java.lang.IllegalStateException: Cannot set more than one test class definition!. Please use "addClass" or "addClasses" method.


With this modification, we can granulate the search
example:
New registration on searchprovider:
```
\OC::$server->getSearch()->registerProvider('\OCA\MyApp'\Search\Provider', array('app' => '{myapp}'));
```
so a single search provider will registerd
with additional SearchProviders:
```
\OC::$server->getSearch()->registerProvider('\OCA\MyApp'\Search\Provider', array('app' => '{myapp}','apps' => array('files')));
```
Now we register the provider of {myapp} and additional the searchprovider of files
If $inApps more than 1 result like ['files','calendar'], only the $inApps provider get loaded without additional options['apps']
so now you can register additional searchprovider on html side like:
```
<div id="searchresults" class="hidden" data-appfilter="calendar,tasks"></div>
```
searchprovider of calendar and tasks get loaded if exists!


Might be a Java habit or something.


Is there a problem with line 35 ?


@nyalldawson can you give some more informations on the failures? It works fine for me.


@arehman199  kindly add similar logic in kernel-config-systemd.bbclass where systemd.cf file is being added in SRC_URI.


I know, but this source work if i add files for msm8916 skum.
I'll publish the source when i added all of the divers. :+1:
What caf tag is this kernel?


Hi @Audryc! You're commenting on just one commit here, so it isn't very clear what you mean by "this". You may want to comment in the GitHub interface or email @reshama directly instead of emailing to the GitHub commenting address you're using here.


I don't think there should be an error printed unless in some verbose mode. Otherwise the user will get errors all the time in distributions which use raw sockets.


I used the template provided, but I like taking the target_dow as an argument. My version of gather_mtime_between has the subs inline but I think this is easier to read. I think your method of finding the start time is complicated though.


Qui ci manca il path.


bence burda hata var


Agree, `function_exists` is the way go... Entirely possible someone might want roll their own `random_bytes` function pre php 7.


should use const Region


Isn't that only needed if modules support is enabled?


I think [there is no need to return anything here](https://www.odoo.com/documentation/8.0/howtos/backend.html#workflows).


LOL


Why not? After all, we cannot ensure any user-defined function of that name works right; which is quite important in this context.


May be there is no reason to quote compiler flags? In my opinion, that is redundant safety.
I agree with renaming AddCflags -> Cflags, thank you.


Changed `ReleaseNotes` into `ListStoriesByRelease` and `GenerateReleaseNotes`.


[Add `self.ensure_one()` at the beginning of this method](https://www.odoo.com/documentation/8.0/reference/guidelines.html#symbols) or change for `@api.one` please.


`required = True`?


добавить комментом пояснение смысла if'а, добавить else с нотификацией нам


@carlosmiranda nevermind, I will change it according to https://github.com/yegor256/takes/pull/356#issuecomment-117995877


I think we should start with a couple of standard properties that are always visible. They just have to be added in the selectedProperties array.


Dropped this function the next commit. Enough to leave this in `ListStoriesByRelease`.


Bumped it.


Yes, we are sorting the stories and sections twice each. That is to make sure these things are sorted. Someone might call `GenerateReleaseNotes`, then change the order, then call `Encode`. We want to be sure things are sorted.


This does not compile for me since KODE::sanitize is not defined:

/home/hcw/git/kode/libkode/enum.cpp:107:65: error: ‘sanitize’ is not a member of ‘KODE::Style’
         retval += QString( " %1_%2 = %3" ).arg( baseName ).arg( Style::sanitize( *it ) ).arg( 1 << value );


Not so unused then... please rename the variable :)


This causes `Conflicting ID 'network_lo'` when I want to assign many addresses to loopback interface.

Example:
```yaml
network:
  interfaces:
    - name: lo
      ipaddr: 127.1.1.1
      netmask: 255.255.255.255
    - name: lo
      ipaddr: 127.1.1.2
      netmask: 255.255.255.255

```


In opencv 3.0.0 is /usr/include/opencv not in CFLAGS anymore. It is necessary add path into include (e.g. #include <opencv/cv.hpp>) or include related module (e.g. #include <imgproc.hpp>) which seems to be recommended.


What type is filenames then?


ideally this would plug into the existing `[declare]` and/or `[import]` infrastructure, where you can already declare dependencies.


one of the main obstacles I see is that we currently don't have a way to check what is already installed (via deken or otherwise).

e.g. if my patch depends on both *zexy* and *smlib*, and it throws a load time error due to missing objects, how can we help the user to not install *zexy* for the 5th time while the really should go for *smlib*.


Thanks! Could you file an issue please? I will fix it this week.


Would this work better is the libPath was split on commas, then use the appropriate item from the resulting list as the pathelement.text value, minus any trailing ')'s?

Something like the following (completely untested) would I think work for most use cases, with a couple of checks that what we've got is valid...

```
libPath = onclickelement.text.split( "," )
if len( libPath ) == 1:
    # Only one element returned, use it
    pathelement = xmltree.SubElement( newelement, "property" )
    pathelement.set( "name", "list" )
    pathelement.text = libPath[ 0 ].replace( ")", "" )
elif len( libPath ) == 2 and libPath[ 1 ].lower.replace( " ", "" ) == "return)"
    # (elementWeWant,return)
    pathelement = xmltree.SubElement( newelement, "property" )
    pathelement.set( "name", "list" )
    pathelement.text = libPath[ 0 ]
elif len( libPath ) == 2:
    # (window,elementWeWant)
    pathelement = xmltree.SubElement( newelement, "property" )
    pathelement.set( "name", "list" )
    pathelement.text = libPath[ 1 ].replace( ")", "" )
elif len( libPath ) == 3:
    # (window,elementWeWant,return)
    pathelement = xmltree.SubElement( newelement, "property" )
    pathelement.set( "name", "list" )
    pathelement.text = libPath[ 2 ]
```


O, that sounds wonderfull! That solves one big problem. Thanks!


Good idea to add the target_dow as an argument.


This is great for those coming from 0.7.2 -> 0.8.2, but not for those going from 0.8.1 -> 0.8.2 :( looks like us in the latter case have to `--fake` again.


After this fixing, laser_scan_matcher (ROS scan_tools packet) doesnt work and throws this error:
http://i.imgur.com/4SyDnIW.jpg


you are right, i was taking into account only the case that the locale passed as default parameter would fail which is totally safe to ignore but not the case where a user passed locale string would not exist. i fixed it with the last commit


`if remote_ip_header_key.present?` tak :smile:


@tchap well, it's not doing anything at all, is it? We only have github for now :)


@tchap ping.


since you're always using relativedelta.relativedelta, it would be cleaner to just import that an use it as relativedelta.


no need to create a temporary string using `run_name`. the later is already a `std::string`.


诗圣杜甫


ditto


On Thu, Jul 2, 2015 at 1:06 PM, Sheng Yu Chiu <notifications@github.com>
wrote:

> TEE_SEServiceHandle isn't defined in kernel. It's defined at libutee, and
> it's part of GlobalPlatform standard.
> We would not use those datatype directly in the kernel space, but use them
> in user space
>
Not sure about that. tee_svc_ta_open_session() does use
TEE_TASessionHandle, defined in tee_api_types.h. In fact I think the handle
types we find in the user API should also be defined on kernel side and we
should make sure it is the same in both environments, to avoid the size
issues. We could make it a uint64_t for instance although GP recommends to
use pointers (to opaque structures). Or we could use uint32_t and use small
integers as object identifiers (like file descriptors).

Anyway changing the type is not what I'm asking for in this patch. What I
am asking is: don't use (struct tee_se_service *h) when h is actually NOT a
pointer to a a tee_se_service structure but to a 32-bit buffer (in user
space). That's all. The current prototype just does not make sense and does
not explain why one needs to use tee_svc_copy_kaddr_to_user32().

The most reasonable option for the time being might be to to use (uint32_t
*h). At least it would make it clear the parameter is the address of a
32-bit handle at that stage.

We have related API in libutee/include/utee_syscalls.h, which is actual
> prototype I think what you want to mention
>
> TEE_Result utee_se_service_open(
>                  TEE_SEServiceHandle *seServiceHandle);
>
> we have many simliar tricks, please refer to
> libutee/include/tee_api_types.h
>
Yes, and this is crappy everywhere IMO :O

Well the utee_prototypes are fine I guess. That's the kernel side that is
wrong everywhere and arguably even more wrong in your API because you are
using pointers to structs when you really have (kernel) pointers to 32-bit
(user) buffers.

BTW, at https://github.com/OP-TEE/optee_os/blob/master/core/tee/se/svc.c#L91
Aren't you copying n kernel pointers to a buffer of n user addresses?

Am I talking nonsense ot do we indeed have a problem of inconsistent
user/kernel ABI as far as object handles go?


Don't really get why we need to mention ResponsderServer, Requester?
e.g.  Given requester sends requests to namespace search:parsers:facets:v1 ->   Given microservice sends n requests to namespace search:parsers:facets:v1

Given responder server responds with '{"result": "hello jbehave"}' -> Given microservice responds with '{"result": "hello jbehave"}' on requests


The cursor is restored exactly to its previous state. If I select a text, the selection is kept.


Already fixed in subsequent commits


I'm beginning to wonder about having "button" classes applied to markup.

Here's an example.

![screen shot 2015-07-02 at 12 46 06](https://cloud.githubusercontent.com/assets/282788/8476363/7d54dcfa-20b8-11e5-94b7-8bd8a95ad321.png)

The unthemed alaveteli presents this as a link, which is quite valid. You may not want the "button" style.

I think what should happen here is that the class on the `a` should be `action-bar__make-request__link`.

Then, if you wanted it to be a button, our `custom.scss` could look like:

```scss
/* _buttons.scss */
@mixin button($color) {
  background-color: $color;
  /* etc… */
}

.button__primary {
  @include button($color_green);
}

.button__secondary {
  @include button($color_grey);
}

/* custom.scss */
@import "_buttons";

.action-bar__make-request__link {
  @extend .button__primary;
}

.action-bar__follow-button__link {
  @extend .button__secondary;
}
```


Testing CLI on Unix


Perfect!


@mbernat Well, it's within the same package, so I don't really care that much. Don't feel really like creating extra structs for every encoder. Markdown and HTML are generating output from a template, the other two are using encoders, so it's different...


@megies Can you try this branch with obspy version < 189fbd6c62970209f6cf0eb574ad6474df9817c1 (May 13 ) which replaced formatSeedLink  to format_seedlink ?


@mbernat Well, the release notes are not posted during `release deploy` if you don't have this module configured ;-)


Markdown lets you do `<http://example.com>` to linkify a URL. Though I think GitHub Flavored Markdown does it even without any markup.


Hi guys! I noticed this impressiv commit and took the time to read through all of the 116 changed files. Nice architecture, kudos for this. I guess the scope was to give the OTRS Appliance an interface for managing the bin/-Scripts and crons? Please correct me if I'm wrong. I have some further questions regarding this change: Will there be a way to call these via the command line (the old fashion way) as well? I know there are a lot of people out there using command line solutions for monitoring these tasks with e.g. nagios or managing cron tasks centralized. As I see it there will be no more exit codes or seperate calls. Do you have to write a log crawler with a matching regular expression for each task after upgrading to OTRS 5? Thanks for your reply.


Seems good!

Do you remember which commands you used? (`rails g …`)


Very cool! Good job on this.


`not` must have only 1 argument, not 3.


@tchap hm, okay. I don't see how it makes sense though, it should be configured automatically along with github code review, lest people forget to do it...


Thanks, @ghuntley.

For things like this, I'm not particularly concerned with the code coverage number.  While it's a good feeling to see a high code coverage number, what's actually important is ensuring that we have good quality tests exercising our code, and the code coverage numbers are just one way to approximate that (unfortunately in many circumstances not a great way).  As such, I don't think we should worry about ensuring that every test project has tests to cover all of the common files it imports; we want to ensure that such common code is properly tested as with the rest of the non-common code, but it doesn't need to be tested in every project just for the sake of boosting the cc values.

For your NotImplemented.ByDesign example, what I'd hope is that any project importing this file would have at least one of the two members invoked, because if the file is being included, some function somewhere is invoking this, and if that code is reachable, we should hopefully be able to write tests for the public surface area that get one of these to be thrown (that may not be true 100% of the time, but hopefully it's true most of the time). Those are tests I would like to see in the test projects.  If the file is included but isn't being used, we should remove it from the project.

We could also consider having a dedicated Common test project, with the goal of ensuring that Common\src code is properly tested, effectively treating each file or set of Common src files as public APIs to be tested, even though they're not public.  We'd want to shoot for a relatively high code coverage number for that project.

In some cases, we could also consider splitting a Common helper file into multiple partial class files, such that only the relevant ones are included into a given project.  While this would help to improve the code coverage number for a given project (by not including code that's not used), the primary purpose would be to highlight exactly what functionality is intended to be used and to make it easier to share just those pieces.

I'm open to other opinions, of course.  This is just my initial reaction.


but no docker in default repos https://openqa.suse.de/tests/96745/modules/docker/steps/3 ?


@tchap Okay, let's keep it this way. But I hope you can see how it's not modular. If we wanted to support 20 other formats, you probably wouldn't want to list them here...


@mbernat Why? It's a different thing. You use one thing for code review. How does that affect how you post release notes?


you've got tones of unused static imports after you commented most formulas


i would recommend to ALWAYS use wxStaticCast instead of C-casts, for
wxObject derived things. A wxStaticCast is not slower than a C-cast in
release mode. In debug mode, it will check if the cast is correct using
wxRTTI. I think it is always a good idea to check things like this.


>
May be there is no reason to quote compiler flags? In my opinion, that is
redundant safety.

You never know what could go wrong in Make next. :) Better escape than
not.

--
Best regards,
Eldar Sh. Abusalimov


this list should be stored into a constant


nome de uma loja. pode chamar de "Kunstee"


test data


@mbernat Well, the only other solution would be to use templates for JSON and YAML as well, which is kinda silly. Or yet another set of structures, which also silly. So let's keep it as it is. I know what you mean, but it's really not worth it right now.


ao clicar nele nao desce o suficiente. colocar mais produtos, tipo 6 por secao, para ficar claro


Хм. Я только что выполнял `npm install`. У меня должно было быть все свежее.


o que acontece?


This should probably be a warning now instead of error?


add a TODO here to explain which alternative should be used when we will have designed it


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


onde conseguiu as imagens? tem que ser imagens que possamos distribuir, comercializar etc. qual a license exata dele? (se for cc, tem varios tipos, qual deles eh? e mesmo sendo cc acho que tem que citar fonte).


Good point... Then what about adding a call to `mcrypt_create_iv` as an additional option/fallback after `openssl_random_pseudo_bytes`?


Всё норм. В генераторах просто глобальный счётчик для переменных и при модификации одного меняется код у всех.


sempre dar nomes reais. nunca lorem ipsum.


Hey, but the baby was tossed with the bathwater: the `$.timezone.new()` calls were removed, too.

Should I fix up my (tiny) fork so it doesn't include the `.say` calls?


@tchap presumably people who will use github reviews will post release notes there as well. Those who don't, won't. Also, we only have one release manager right now, that's why I don't see any point in having all this code that's actually not doing anything useful yet and moreover requires people to do additional work that's not necessary. But whatever, I don't really care.


Shall I add a few lines to the readme about this?


How did you arrive at these new values?


pode ate citar... na hora de implementar falar, mas poxa, eu estou entregando isso pro meu cliente ver se ficou bom e escrevo um nome lixo como esse? ai ele fica sem saber como ficara a tela cm uma categoria cujo nome tem cinco letras tipo "Calça" e uma outra com 5 palavras tipo "Coleção especial de natal 2015"


tudo maiusculo? conferir com designer em 5 secs (thaigo?)


@mbernat Well, you can also leave it disabled, which is what happens when you don't enable it...


This sentence seems to be a copy of the previous one but with something missing at the beginning.


lorem ipsum é o inferno. ele finge ser algo que não é real. o que acontecera com uma descrição real? sera que ela eh mais curta? mais longa? as palavras do portugues tem que tamanho? pode justificar tudo isso pra mostrar que lorem ipsum nao deve ser usado


Да и вообще весь этот гигантский дифф относится к одному генератору. Просто убрал одно ветвление и он меняет цифры шагов.


lembre de citar que tem gente que nao eh a favor do <!-- fimde um X --> <-- inicio de um x_->


commit lgtm


pq  opreço é um botao?


CLI tests pass on Unix


Coloca um endereço. Pode ser Avenida Paulista, 1. São Paulo - SP. melhor


antes tinhau ma search no menu. a direita? ou não? se não, pode colocar :) é legal  e simples!


Right !


Testing HTML-Runners


nome do produto


right !


colocar frases reais em todos os cantos.


Are you sure you're not using the prebuilt kernel? That's pretty impossible since stuff like device tree files and drivers for Redmi 2 aren't here. Anyway, I forgot the CAF tag since I (probably? I forgot) just plucked the latest tag for MSM8916 at that time.


não sou fã de fuzzy nem de cardigan. nem sei o que é fuzzy. nem cardigan. prefiro outras palavras


It's worse than regular Pikachu. And Pikachu (combined) usage doesn't even make PU.


instead of none you could use apache


textos reais bla bla bla. colocar pelo menos 2 paragrafos reais


instead of none you could use nginx


@tchap if this is the case, I see no point in sorting in `GenerateReleaseNotes`. Just in the encoder is enough.


essa tabela eh estranha. em geral eh na vertical e nao na horizontal. entao trocaria.
Colocaria tambem um campo grande, tipo "Modo de lavagem" e diversas regras, para incentivar que a table seja responsiva pq o campo grande vai ficar bizarro.


ok


I think the markup should be in alaveteli core, but with alaveteli-specific class names rather than the `ui-blahblah`.

The js should be in its own file, rather than a script tag.

Probably worth putting the styles in `_locale_switcher.scss`.


We should also move all  the logic out to a rails helper as part of the merge back in to alaveteli


lol hax


You are not testing the second clause of this sentence


That one was in addition the wrong class, since it's `OnlineAccountService`.


@mbernat Well, it takes no time, so I didn't really see any reason not do make sure everything is always sorted, but...


Исправление: 4 этап.


I think this line in particular needs to continue to have reference to `ColdFusion` so that it is more searchable when looking for packages inside Atom. Not everyone will think to search for `CFML`.


So, I'll ask a silly question, just to be clear: This `fn will_change` method will go away after the default actually changes, right?

I originally was going to write that I object to use of future tense in names like this, since the event it is referring to may happen without a corresponding rename (or removal), and then the name is simply confusing.  But AFAICT there should not be any reason to still have the `fn will_change` method around once the change to the defaults occurs, right?


(a similar comment about the use of the future tense applies here; that is, this is probably fine as long as we rename or remove it as appropriate in tandem with the change.)


oh cm12 u silly thing.


You are implementing way ahead of your tests.
Be wary of untested or contrived code ahead.


typo: independent of


@tchap ok, no problem.


In lambda calculus, every lambda has only one argument.


remove all new strings from miuicamera.. they are values.. not translatable...


Unfortunately my ruby /puppet knowledge got a bit hazy, it seems to be only when the module was run via a puppet master Ubuntu 14.04, default puppet-master package.


input data -> input/gradient data


em portugues usamos esse termo? entrar no submarino


frase melhor


nome etc


a cor não bate com a imgem


Did you mean to replace the current test? And not add one for this specific `hbs` case?


typo tuple


this is my new Facebook password tbh


pls no hack


o problema de brincar com isso é que nos usamos mailing lists tambem, como aqui, entao pega mal. nao usaria esse termo


typo, "affect the affect" should be "affect the effect", i think.


CVV separado. nunca vi cvv junto


typo: abstract


ps: cvv é o ultimo campo pq exige alterar o lado do cartao


bandeira eh o primeiro campo


It probably is LNX.LA.3.7.2_rb1


lembrar de sempre ensinar uma coisa por vez, entao so um span. so um button. button com span. etc


are -> be


Can you document what `explicit` is for?


@Parbhat This is Ok but i really prefer that we use make for building the dev environment as per issue #91


s/There also/There are also/


@tchap I almost commented on this line, but for a different reason. Namely, a standard return value should never be an error. When querying for a milestone, it's very much possible that it won't exist, so that `notFound` should be a normal return value and not wrapped in an error and typecased later to distinguish it from real errors.
Suggestions: return either `(milestone, exists, error)` or return `(nil, nil)` when nothing is found.


commit lgtm apart from from some questions and/or nits regarding name choices.


Ah, no, I was changing things trying to find out the issue.


I see you've added that, but it's not used, is it a leftover?


line 189 should be

    $oldRefreshToken->setAccessToken($newAccessToken);


@jspc :+1:  on your suggestion.


It should be staticWithDeviceMapper(name)


確認してみますー


First character of "Raise" should be lower-cased, or first character of when in the comments above should be upper-cased.


typo: function (remove "s")


bj


Does this really spare memory? I once heard a rumor that the Oracle JVM always allocates an int, even if you want byte, and that using byte explicitily will reduce performance due to the needed bit-masking.


Initial persistence.xml configuration warns about "Encountered a deprecated javax.persistence.spi.PersistenceProvider". It is a known issue and there is a nice fix.

http://stackoverflow.com/a/24251993

I'm going to convert persistence.xml into java based config eventually, so I will ignore this warn for now.


@chrisdev yes make is good


priorDriver is not valid in this context. Should be fixed. We should retrieve the name of driver we are calling init function of.


I wouldn't be surprised if range checking every pixel like this makes things *much* slower.  That was our experience during the initial Fortran-to-C conversion.  The range checking function should be called inside a C assert, so that if the code is not compiled in DEBUG mode, it doesn't get run at all.  I don't think there's much point in converting to a Python exception, since the Python programmer (generally) would not be at fault for this. If there is a way to pass in invalid parameters from the Python side that lead to an out of range error, those inputs should be tested or fixed once at the source.

Also, it seems unnecessary to do the out-of-range checking around every call to `set_pixel` and `get_pixel` rather than inside `get_pixel` and `set_pixel` itself.  (And if you don't set a Python exception, you don't need to worry about the extra return value on `get_pixel`).


typo: backward


I still have some changes to make this is just low hanging stuff.


I am debating this. If we detect that devicemapper driver is unsupported, then shouldn't we skip it and move on to next driver, instead of returning error?


I can only see it used once, is it something that you plan on using later? Otherwise just remove it


add to .gitignore the media folders so you don't upload your testing images


I see the missing colon ;)


altul


Also this check should probably come before GetDriver() call. What's the point in doing GetDriver() when we know we are not planning to use it anyway.


u've b33n h4ck


staticWithDeviceMapper() check should happen before initFunc() call, I think.


@dvdhrm thanks for fixing this up.


Info from here: http://zdoom.org/wiki/Patch

ZDoom and some other ports such as Boom allow for textures taller than 256 pixels. This method is achieved by a little trick with the starting offset. If it is a value lesser than the Y position of the last pixel of the previous span in the column, then the starting offset is taken as a value that needs to be added to the previous span's own starting offset. This method, called "tall patches".


>Does this really spare memory?

Of course it does.


@mbernat Yeah, perhaps here it makes more sense...


見れました。
一応db:resetかけてうごかしたんですが、大丈夫でした。
 "mofmof@mofmof.co.jp"
  "hogehogenyaruko"
使ったアカウントはこれです。


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


OK :) Fixed


The extra code can be removed with a C macro define that sets oob_pixel to 0.


suggestion: add the comment "// just_ref will stay the same." here as well, just like you have it in `lifetime-bound-will-change-warning.rs


I think 'When "some" expression is evaluated...' is appropriate because previous sentence does not indicate that the expression is in given expressions.


O'key :) I think with explicit `true` and `false` it's better for beginners to see how `not` works.


Because a malicious package could define their own random bytes and
compromise the security of encryption.

On Thursday, July 2, 2015, userabuser <notifications@github.com> wrote:

> Good point... Then what about adding a call to mcrypt_create_iv as an
> additional option/fallback after openssl_random_pseudo_bytes?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/07ec4460778281a119e786f5defd03256f965532#commitcomment-11973209>
> .
>


What does this give over `import validator.validate`?


commit lgtm apart from nit/suggestion.


The same is true for str_random though.


Here's a good read for you: http://shipilev.net/blog/2014/heapdump-is-a-lie/


great job!

could you please name [localization] all the commits regarding the labels being localized?
sorry to bother, but that way we can create a easier to read map of the work we doing...

fastcolors@gmail.com > you can contact me there, then we maybe find a faster way of communicate.

let's sync the work very often.

I foresee this ready for Isengard!!!!!


Thanks @tiffanytse! Looks fine.


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>SVT Group | Real Estate Group Builders & Promoters in Jaipur</title>
<meta name="Description" content="SVT Group is continuously engaged in building the real estate sector of Jaipur city with all the global standards of quality and luxury living lifestyles.">
<meta name="Keywords" content="Real Estate Group in Jaipur, Real Estate Properties Jaipur, Builders & Promoters in Jaipur, Builders Developers Jaipur, Builders in Jaipur">
<link rel="stylesheet" href="css/svt-style.css" type="text/css" />
<link href="images/fav.png" rel="shortcut icon" />
<link href='http://fonts.googleapis.com/css?family=Lato' rel='stylesheet' type='text/css'>
<meta name="google-site-verification" content="LtO02tmm4Ok_U0i4bB8DpzRr88kmj_lJmnKiL6uKEGg" />

<link rel='stylesheet' id='dt-main-css'  href='silder/main.css?ver=3.1.1' type='text/css' media='all' />
<link rel='stylesheet' id='rs-settings-css'  href='silder/settings.css?rev=4.1.4&ver=3.8.4' type='text/css' media='all' />
<link rel='stylesheet' id='dt-font-h1-skin1-css'  href='http://fonts.googleapis.com/css?family=Open+Sans%3A300&ver=3.8.4' type='text/css' media='all' />
<link rel='stylesheet' id='dt-normalize-css'  href='silder/normalize.css?ver=3.1.1' type='text/css' media='all' />
<link rel='stylesheet' id='dt-custom.less-css'  href='silder/custom-7b7e40f819.css?ver=3.1.1' type='text/css' media='all' />
<link rel='stylesheet' id='rs-captions-css'  href='silder/dynamic-captions.css?rev=4.1.4&ver=3.8.4' type='text/css' media='all' />
<link rel='stylesheet' id='dt-awsome-fonts-css'  href='silder/font-awesome.min.css?ver=3.1.1' type='text/css' media='all' />

   Our Networks

info@svtgroup.co.in

+91-9352555155

Home

About Company

Company Profile

Vision & Mission

Chairman Desk

Awards & Achievements

Milestone

Our Team

Projects

Ongoing Projects

Completed Projects

Upcoming Projects

Customers Zone

Property Buying Tips

Faq’s

NRI Rules Clarity

Vastu Shastra

Stamp Duty Converter

Area Converter

EMI Converter

Clients Speak

Media

 Events

 News

 Press

Careers

Enquiry form

Contact Us

   Destination of Your Dreams

     SVT GROUP

A Project By SVT Group

  ORB Mannat (Rs.3400/-Sq.Yds.)

A Project By SVT Group

  ORB Vastu Vihar(Rs.1700/-Sq.Yds.)

 ORB VALLEY (Rs.2900/- Sq. Yds.)

  ( A Project By SVT Group)

 ORB HIGHWAY  (Rs.4400/- Sq. Yds.)

 A Project By SVT Group

   ORB LOTUS

Coming Soon

 A Project By SVT Group

 ORB HILLS  (Rs.2000/- Sq. Yds.)

A Project By SVT Group

  ORB CITY (Rs.3800/- Sq. Yds.)

Latest Updates

<marquee scrollamount=3> ♦ PRSENTING ORB HIGHWAY (HIGHWAY VILLAS) BASED  ON MODERN ARCHITECTURE,AT THE FASTEST GROWING LOCATION DAYAPURA,ON NIMORIYA ROAD NH-12,TONK ROAD, JAIPUR.
      ♦ UPCOMING ORB LOTUS ON MAIN NH-12, NEAR JDA SCHEME UDAY VIHAR ON NH-12, TONK ROAD,JAIPUR.
      ♦  DIVINE HOMES III , SVT GROUP'S UPCOMING PROJECT IN PRIME LOCATION OF JAIPUR  WITH 1,2,3 BHK UNITS WITH ALL 5 STAR FACILITIES FOR ALL... ..</marquee>

Welcome to SVT Group

Today’s real estate and construction companies must adopt new approaches to address regulatory requirements and financial risks – while meeting the challenges of expanding globally and achieving sustainable growth.

Shri Veer Teja Real Estate & Developers Pvt. Ltd has been established in 2008 and it is ISO 9001:2008 Certified Company. We are in the Real Estate Field from last 6 year Specialized in the development of Township, construction of Residential apartment and management of high-quality, affordable housing communities.

Upcoming Projects

ORB Lotus

Divine Home III

Ongoing Project

  ORB Valley

  ORB highway

  ORB MANNAT

Testimonials

No worries, and tremendous peace of mind being in orb valley (farm house) user friendly envierment  . Convenient Location to everything we need and want....

Divine Homes II is Residential Project Located in the Jagatpura, Jaipur. Like other premium segment properties....

We have booked the property in ORB HIGHWAY, Project location with connected national ...

Great team to work with, will be using again to buy PROPERTY WITH SVT GROUP Jaipur, Excellent Builder....

Quick Links

Home

About Us

Careers

Enquiry form

Clients Speak

Contact Us

corporate Video

contact us

    Shri Veer Teja Real Estate & Developers Pvt. Ltd.

      A-17, Chandra Gupt Nagar, Ground Floor, Jagatpura,

      Jaipur - 302025 (Raj.)

    +91-9772101777, 9352555155

    info@svtgroup.co.in

    marketing@svtgroup.co.in

    svtrealstatedelhi@gmail.com

    Copyright © 2014 SVT Group
    Designed by : Promotech


rubber stamp.  ;)


What does the ":\" operator do? This is the first time I'm seeing this.


string looks too long


Left you a message on Tg, I don't think this is required, as only auth.sendCode could produce migrate error, whereas auth.signIn should not:
https://core.telegram.org/method/auth.sendCode
https://core.telegram.org/method/auth.signIn
But we found the problem elsewhere, pull request coming soon.


Thanks.

You have a point with your concern about the lifecycle support.
Another option is to make it switchable in the configuration file. (So that
you do not have to maintain your own fork)

Currently I'm working my 2 last days before my holidays. I'll try to find
some time tomorrow to merge your outstanding pull request
(JndiConfigurationBridge support #10).
In the next sprint (at work) I'll be focusing on the Angular ui of the
application. So it means that next month I will only have very limited time
left for 'wizard in a box'.

But on the longer term I want to tackle the lifecycle issue (and hopefully
relying on threadpools managed by the hosting application server itself)

Regards

On Wed, Jul 1, 2015 at 5:03 PM, Rémi Alvergnat <notifications@github.com>
wrote:

> Currently, I still use a fork of wizard-in-a-box because of this, i'm not
> really confident about dropping the LifeCycle support from Jetty. But great
> work by the way !
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rvs-fluid-it/wizard-in-a-box/commit/050028bd9ebbce5154164442eef3f93976982958#commitcomment-11955542>
> .
>


heh, glad I spent minimal effort reviewing previous commit.

(rubber stamp encore!)


this too)


I suppose it's just as unlikely that the transformed pixel will be to the left as it is to the right, particularly if the WCS does a 180 degree rotation or something, though the more common case with wcs is usually how this was initially.

I'm concerned that this no longer matches the midpoint calculation below:

```
midpoint[jdim] =
             (delta[1] * segment[0][jdim] - delta[0] * segment[1][jdim]) /
             (delta[1] - delta[0]);
```

Does returning the absolute value *only* at the end of this function resolve the bug?  That seems like the safer and more correct change.  If the sign of each calculation at each step of this inner loop doesn't match, we're bound to get incorrect area calculations.  If the rectangle is merely mirrored, that would give a negative result, and `fabs`-ing it should be enough.


What's the status on a fix for the text?


import loop issues? If so, please comment


I would like you to add two things
- Both of `x==y` and `z==w` call `__nonzero__`, which by default return `True` unless they are None.
- The reason why and returns evaluation result of second term is that both terms return `True`.


This commit looks good to me. I don't think the other variables in https://github.com/kelseyhightower/setup-network-environment are used anywhere else.


LGTM


this line looks too long


this too


If you want to see,fork


this is added to a QLayout so I don't think it's an issue, but I'll do it for peace of mind.


done


and this... the next code block is the same, why not to write this like it, in the same codestyle?


De rien, Tony ! :)


> The extra code can be removed with a C macro define that sets oob_pixel to 0.

Sure -- that's fine, too.  But maybe trigger it off of the standard C DEBUG define, i.e.:

```
#if DEBUG
... declare the result oob_check function
#else
#define oob_check 0
#endif
```


Bonus points for refactor ;)


Tagging @Zarel to update the client.


搞复杂了吧，用这个还得引入 react-router?

react-router 不小


good going buddy.
all the best..
if anything else you need help with,...just ping me..


commit lgtm


Couldn't you just have something like

    cmd = ['virt-sandbox', '-c', 'lxc:///', \
                '--disk=file:disk_image=%s,format=%s' %(disk,format), \
                '/sbin/mkfs.ext3', \
                '/dev/disk/by-tag/disk_image']
?


same here


this is dangerous and error_prone (using a mutable data structure as the default value). If in the future somebody updates the `properties` dict in the method, it will lead to test leakage. Please use `None` and do something like `properties = properties or {}`


good going buddy..all the best..if any help you needed just let me know...


same here... it's just about cosmetic ;)


Comment


Why are we storing the complete list of emails, seems like if the developer wanted the extra emails he/should could choose to do so by using the `onCreateUser`.

My concern is you will be storing them and only a few people will ever really use them.


HTML-Runners test passed.


I got template warnings and Wsign-promo.


bleah; I'd rather you just import `syntax` and write `syntax::abi::Rust` and `syntax::abi::RustIntrinsic` at the two relevant locations below.  (But, I won't block this PR based on that alone...)


LGTM.


ttryrtyrtyrtytyrty


La hoja extremadamente afilada*


@magnumripper please test. If there are some issues with encoding etc, we'll debug together.


Doesn't fit on a single line?


Kyle-  I believe this was needed because we were missing the last field in
one of the lines.

I am trying to re-run all the experiments now and am running into assorted
issues, one of which was needing to fix the csv.  I am leaving the current
harvest point with the older version then will rename the directories when
I get all the experiments done with the latest.

Best, Ellyn

On Wed, Jul 1, 2015 at 7:27 PM, Kyle Wilcox <notifications@github.com>
wrote:

> Was this removal intentional? Looks like good data at first glance.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/USGS-CMG/usgs-cmg-portal/commit/a0eea05f1f83bce3a180b5e2881760e882274086#commitcomment-11964914>
> .
>

--
Ellyn Montgomery, Oceanographer and Data Manager
U.S. Geological Survey
Woods Hole Coastal and Marine Science Center
384 Woods Hole Rd., Woods Hole, MA, 02543-1598
(508) 457-2356


Thanks for the link, this was interesting to read. So essentially virtually all Java teachers are wrong when they say it makes not much sense to use byte vs int. I shouldn't have listened to all those blogs telling that the spared bytes would not be worth the performance impact... Did you notice any speed reductions?


@nursix thank you for review...very constructive suggestions for me...I will incorporate these...Thank you again :)


At this point (and above) `curr` is cached with no lock hold. Could this hint a race condition? I pondered in this question for a while now but came to no definite answer.


Wow, good to know this! Looks exciting! A big :+1: for finding this out!


Then it’s a great update :+1: :+1:


Oh, so very used to push to `origin` (which in this case is my repo)... Sorry.


The command needs to be fetched from the docker image config


Not at the top of the module because of import loop issues? If so please comment


No hay informacion sobre Herramienta disponible*


Вот здесь postdate заменен на postDate, насколько я понимаю, ResultSet - это выборка из базы данных. Поломаться ничего не должно?


Why did you use ```input-group``` class here?


switched to J because of https://github.com/SheetJS/js-xlsx/issues/35 and https://github.com/SheetJS/js-xlsx/issues/135


@esqilin The List isn't necessary anymore. It was used to convert the Page returned by Spring to a list of PostingDTO entities. With the @RestWrapped annotation you can just return the Page object you get from the service.


Oh I like this, this was a good idea you had!  approved.


me gustaria que corrigieras esos errores antes de actualizar


22:1: I101 Imported names are in the wrong order. Should be g, request, url_for
22:1: F401 'request' imported but unused
23:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.
25:1: I100 Imports statements are in the wrong order. from itertools should be before from intbitset
25:1: I201 Missing newline before sections or imports.
27:1: I201 Missing newline before sections or imports.
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from six
30:1: F401 'Collection' imported but unused
132:9: B901 blind except: statement


22:1: I101 Imported names are in the wrong order. Should be ModuleAutoDiscoveryRegistry, RegistryError, RegistryProxy
26:1: I100 Imports statements are in the wrong order. from invenio.ext.registry should be before from werkzeug.utils


43:1: F401 'cStringIO' imported but unused
48:1: F401 'session' imported but unused
51:1: F401 'current_breadcrumbs' imported but unused
54:1: I201 Missing newline before sections or imports.
54:1: F401 'iteritems' imported but unused
55:1: I201 Missing newline before sections or imports.
58:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.local
59:1: I100 Imports statements are in the wrong order. from invenio.base.decorators should be before from invenio.base.globals
59:1: F401 'templated' imported but unused
61:1: F401 'register_template_context_processor' imported but unused
64:1: I101 Imported names are in the wrong order. Should be format_records, get_output_format_content_type
67:1: F401 'facets' imported but unused
73:1: F401 'get_search_query_id' imported but unused
75:1: F401 'wash_search_urlargd' imported but unused


We should host the libraries ourselves to prevent third parties from being able to track our users. This would also fix the mixed protocols. Are there any objections against newer versions of jQuery?


@shingchi 嗯，回收站大多数时候确实没什么作用，但是其实，我认为对于一些人来说，特别是对于那些由于误操作删除掉了自己花费了大量时间写下的文字的人来说，还是有一点价值的。

之所以有回收站，是可以提供给用户一次反悔的机会。

而且，对于博客作者来说这些博客文字无疑是弥足珍贵的财富，这也是为什么WordPress一类的博客系统不仅有回收站功能，而且还有自动保存功能，抑或是自动备份的插件，目的都在于保护好用户的这份财富，不至于毁于一旦。


this should not be required anymore since [commit ec74be5ae1a8f9ca4865abf6d54611473775b11b](https://github.com/apache/marmotta/commit/ec74be5ae1a8f9ca4865abf6d54611473775b11b), artifacts already deployed to apache snapshots


hi Johannes,

thanks for adapting & merging

there is another related issue with desktop notifications in chrome as well, they also always point to the root context, i.e. lead to a 404 when clicking on them.

I didn't search for the point where those URLs are registered though, most probably you are quicker.

I created a ticket for that: https://github.com/codecentric/spring-boot-admin/issues/86


You could use jQuery selectors here (e. g. $( 'category-title' )).


1) There is no observable speed impact here.
`flags` is not used very often (as compared with sending messages over TCP).

2) HotSpot uses native instructions. In other words: there are "int to bool and bool to int" conversions in java virtual machine specification (e.g. i2b), however JIT compiler recognizes common patterns, and it can eliminate non-required int-boolean-int-whatever casts.

E.g. see slides 9-13 here: http://shipilev.net/talks/jvmls-July2013-jcstress.pdf


Our coding conventions recommend to use single quotes.


erererer


Hi @mgruner,

Yes it is a typo, I will fix it in the next hours

((enjoy))
cr


erewrwerewrwe


Please use strict equality operators (i. e. ===)


Этот PR надо будет влить после #663, учитывая правки регистра.


Whoops. I was following the Bootstrap manual. Didn't realise there was only one input in there.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.717.9f6e443c54f17fe6e053345e38e242e87075a82c](http://build.palaso.org/viewLog.html?buildId=59220&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:05:37


@esqilin We use the curly braces also on one liners.


Thx that was super clear, I'll give it a try later today. Would it be cool to make getFirstParent() public then? It's private currently.

https://github.com/openframeworks/openFrameworks/blob/master/libs/openFrameworks/types/ofParameter.h#L192


No problem, @louh


Mmm this is a problem newSize is not the right size either, it can cause the window to be resized to the wrong size, since newSize is the internal shell size. In my system this causes my window to be resized improperly.


peut-être mettre un _() (pour internationaliser)


Nice one, I wanted that a while ago using ptv :)


@emontgomery-usgs , I think @kwilcox is talking about WRIGHTSVILLE being removed.


This is pretty good to know, as it essentially proofs that it is better to use the "right" (most exact) data type of the Java language while programming, instead of relying dumb rules-of-thumb like "simply use int always". Thanks for the infos! :-)


nao parece fazer sentido


Hi @pamval
Please verify our plugin.
We have updated support of our plugin upto Pentaho 5.4


Haha, no, they really are unused! lol


OK


My apologies, I'll revert and change it.


linter didn't pick this up?


22:1: I101 Imported names are in the wrong order. Should be g, request, url_for
22:1: F401 'request' imported but unused
23:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.
25:1: I100 Imports statements are in the wrong order. from itertools should be before from intbitset
25:1: I201 Missing newline before sections or imports.
27:1: I201 Missing newline before sections or imports.
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from six
30:1: F401 'Collection' imported but unused
132:9: B901 blind except: statement


22:1: I101 Imported names are in the wrong order. Should be ModuleAutoDiscoveryRegistry, RegistryError, RegistryProxy
26:1: I100 Imports statements are in the wrong order. from invenio.ext.registry should be before from werkzeug.utils


43:1: F401 'cStringIO' imported but unused
48:1: F401 'session' imported but unused
51:1: F401 'current_breadcrumbs' imported but unused
54:1: I201 Missing newline before sections or imports.
54:1: F401 'iteritems' imported but unused
55:1: I201 Missing newline before sections or imports.
58:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.local
59:1: I100 Imports statements are in the wrong order. from invenio.base.decorators should be before from invenio.base.globals
59:1: F401 'templated' imported but unused
61:1: F401 'register_template_context_processor' imported but unused
64:1: I101 Imported names are in the wrong order. Should be format_records, get_output_format_content_type
67:1: F401 'facets' imported but unused
73:1: F401 'get_search_query_id' imported but unused
75:1: F401 'wash_search_urlargd' imported but unused


You might consider using $( '#errorBox' ).show() here. In this case it actually behaves the same, but whenever you are using other values (like inline), this method will set it to the value defined by CSS.


Ah, but I guess it got added back, because I see it at https://github.com/USGS-CMG/usgs-cmg-portal/commit/6eb8f41c1f47b2fcedbc04fcdbdfdc36cd065738#diff-3a2f5ba2e0bc6f4bf5a51cbac23983b1L50


I tend to prefer to keep the namespaces instead of implicitely importing each component individually these days, but that is fine anyway :)


Thanks for the explanation!


@arranf It might, but then again there are tons of [non-Rock specific] things devs potentially need to know. Do you have a particular wiki page in mind where you want to add this note? (I believe we have things configured to allow users to edit the wiki.)


commit lgtm apart from above `use` rename nit.


Thanks for the explanation! :+1:


Are you dure it did nothing? (no idea what drag_highlight does)


Is it possible to give in the FIXME more advice than what the problem is?


@lkraav ah, i tagged it, but forgot to push with tags :D thanks!


maybe we should fix that before the classes are merged into master?


I'd suggest using a class to identify the missing responses column instead of relying on it being the 7th column


commit lgtm


@anatoly-cmdx @must-not-code мужики, как успехи?)


That is the only change in that structure, right? Why have you moved the complete structure in the file!?


quebrando a abstração? nenhuma garantia?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.429.baa5af84c0cd56190cd885657fbc14e83fcb1fe1](http://build.palaso.org/viewLog.html?buildId=59219&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3227, ignored: 119 Build time: 00:26:22
### Failed tests
```
SIL.Core.Tests.dll: SIL.Tests.Progress.ConsoleProgressTests.LongRunningMethodUsingConsoleHandler_ProducesLog: <no details avaliable>

```


ultra-nit: I think you meant to have a period rather than that first comma.  I.e.: "Create the destination. If we were passed a destination, X. Otherwise, Y."  The way it is currently written, it sounds like you are saying that we create the destination only if we were passed a destination.


There is a rails method called [options_for_select](http://apidock.com/rails/ActionView/Helpers/FormOptionsHelper/options_for_select) that builds the options tag list for you.

Then you'll be able to set shorter option names in the form, and set the current selected value more easily.


again, I think the first comma in the comment here was supposed to be a period.


Cant wait to see more feedback on this!


You can also use jQuery's ´appendTo()´, ´addClass()´ and ´text()´ to inject elements into the DOM. This might also improve readability.


camera is not working for me after this. Is it the same for you?
First it was not even booting but after reverting the bionic commits it boots now but camera FCs.


Thank you! Although, if you figure this out next week while I am gone then this point is moot :)


muito grande. outros bad smells


This could be moved into a function to remove redundancy.


1. ... para palestras, já que estamos ...


@balrajsingh just told me that this might be the issue: we are supposed to send a packet here without waiting for it to finish. It's possible that's we are now deadlocking because the `Lwt_mvar.put q.tx_wnd_update win >>= fun () ->` code below is then not called. @MagnusS or @talex5 can you check if adding an `Lwt.async (fun () ->  q.xmit ~flags ~wnd ~options ~seq rexmit_seg.data)` here instead of blocking could make the test pass again? Thanks!


это я в курсе, там и if ([]) равнозначен if (false), но если уж явно считать count, то лучше явно его и сравнивать


I see that you wrote above that your motivation for this change is that "its more clear this way", but I am curious: What was otherwise happening here that led you to even notice this code?  (Were you surveying all of the uses of `mem::transmute` in `libstd` and this one annoyed you?  Or did you notice particularly bad codegen here? Or ...?)


Si le fichier n'existe pas on l'ouvre avec curl_file_get_contents() ?


perhaps a better name would be `with_level` and make a special argument of `:all` or something bypass the filtering then you can change the select box to use those options (see other comment) then you're not coupling the interface text to the code.


answer não seria uma das alternativas?


Hi Irit, can we make the links to these images relative instead of absolute? Currently this cannot generate a pdf on the docs site due errors with the path. Something like `Images/DocsLogin.png` instead of `/Docs/Images/DocsLogin.png` works fine


LGTM


nice, much more informative


Please switch to camel case.


Ok, maybe we should just pick out the primary email and store that?


The extra whitespace makes the `if` condition more readable IMHO, so it is intentional... I can remove if you don't like.


Yeah I dont get his commit messages either. The majority are pretty hilarious.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.718.0f5bd750f42c687ab26a26ad8b6f44807696a9c3](http://build.palaso.org/viewLog.html?buildId=59224&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:07:24


Strip out the RPM stuff (including the spec file) as it's hopelessly out of date.

Explicitly instruct to build out of source - don't bother with the alternative as its just a bad idea.


Kedar- the requirement is this:admin_user with neutron_admin role will exist in default domain.
The admin_user needs to be added for a given project in a non-default domain


Nützlich


"View view", not "View v". Also, no custom abbreviations please. "courseViewHolder", not "cvHolder". "timetable" is one word, so don't use "time_table" or "timeTable".


Ah ok - I'll check


The charset attribute is only needed if it differs from the one defined in the meta tag.


I guess this commit lgtm ... I am a bit worried that I didn't notice the bug that this is fixing -- makes me worry about whether the intrinsic code is becoming a bit of a mess...


Russ,
I found your repo from your coursera profile.
As one of the CTAs in the course,  I wanted to thank you for your contributions to the discussion fora.  As scored by Coursera,  using sum of the square root of upvotes in your posts,  placed you as the #1 contributor among peers.

Thanks again,

John Raphael

ps - excellent,  very well written, project report


I guess ompi_buffer_detach_f should be updated
(see the comment in the code)


It actually prevented Ember from loading since part of the security patches included upgrading jquery-rails which includes a version of jquery > 1.7.


The project given by project_id is in non default domain, and line 900 adds admin user to this project with neutron_Admin role.


I have the impression self._dropped_files is not properly name as the same thing will happen when importing through the "import button" am I right?


Apart from that OK


Fixed in master using inline editor. Thanks.


nenhuma mensagem de sucesso? nenhum outro campo a ser preenchido?


minor error is resolved in latest commit.
role = v3client.roles.create("neutron_Admin")


Interesting, thanks for the clarification!


try to create


Please use CSS rather than skipping heading levels to reduce the font size.


As I see this over and over again. What about super.setPosition(pos)? also with all the other delegate calls?


apaga todas?


super.setPosition?


Does it mean we will link with -ldl in non-Linux builds?


It did, actually ;o


oder "jemand den ersten Knoten aufstellen"


me too, but better to be consistent with the code around


I'm not sure how a `Polytop` is defined. Can you maybe elaborate a bit more (and potentially find a definition) if it is defined as a list of n-dimensional vertices? Or is this more the definition of a `Polygon` for 2D vertices and in 3D you need actually more information?!


Better to follow the instruction in Jarvis and Shier, page 12, second last paragraph: that is, initialize, say, `g` and then update by `g = gcd(g, value)`. Terminate the algorithm once we have `g = 1`.


Addresses pootlepress/page-builder-for-
canvas#315


Oh, that was a bit too fast but pushing to master seems like fun.


Update structure of repo.


duplicando código


I'm sure this is incorrect. at least it doesn't work for a `Polyeder`, which is a `Polytop` but has different properties.


ich würd schreiben "Zusätzlich gehen die Gesetzgeber auch davon aus" vgl https://de.wiktionary.org/wiki/Gesetzgeber


I don't understand this. How could an update possibly trigger re-download of embeds ?
Wouldn't all embeds be cached already ?
Are caches being explicitly dropped by code ?
See #1727 for what I'm after


This revision is clean.


não está bem verificando o que deveria ser verificado


prefect!!!


Ah, it does actually do something, a 1px black border around the timeline, looks ugly. I'll update the commit message.


Hi Feng
May by i found bug in this commit:

[89195.660176] BUG: unable to handle kernel paging request at ffff8878050402e0
[89195.681071] IP: [<ffffffff81079808>] get_next_timer_interrupt+0x158/0x240
[89195.701879] PGD 193b067 PUD 0
[89195.722149] Oops: 0000 [#1] SMP
[89195.741973] Modules linked in: sg udp_diag unix_diag af_packet_diag sch_hfsc xt_TCPMSS ipt_REJECT nf_reject_ipv4 iptable_filter xt_IMQ xt_dscp xt_DSCP xt_multiport iptable_mangle xt_nat xt_addrtype iptable_nat nf_conntrack_ipv4 nf_defrag_ipv4 nf_nat_ipv4 ip_tables xt_IAND(O) xt_IADN(O) xt_iar80(O) xt_IA(O) ebt_iapa(O) ebt_iabo(O) ebt_iabi(O) xt_IADB(O) IA(O) netconsole hwmon_vid bonding imq pcnet32 tulip dmfe ne2k_pci 8390 b44 ssb natsemi 3c59x fealnx via_rhine sis900 e100 8139cp 8139too dl2k acenic r8169 mii tg3 libphy e1000e(O) e1000 ixgbe(O) vxlan udp_tunnel ip6_udp_tunnel igb(O) ptp pps_core nf_nat_ftp nf_conntrack_ftp nf_nat_pptp nf_conntrack_pptp nf_conntrack_proto_gre nf_nat_proto_gre nf_nat nf_conntrack pppoe pptp pppox gre mcryptd sha1_ssse3 sha1_generic arc4 ecb ppp_mppe ppp_generic slhc
[89195.968798] CPU: 4 PID: 0 Comm: swapper/4 Tainted: G           O    4.0.7 #1
[89195.989472] Hardware name: Supermicro X9DRD-7LN4F(-JBOD)/X9DRD-EF/X9DRD-7LN4F, BIOS 3.0  07/09/2013
[89196.030331] task: ffff88085be8eea0 ti: ffff88085beec000 task.ti: ffff88085beec000
[89196.071258] RIP: 0010:[<ffffffff81079808>]  [<ffffffff81079808>] get_next_timer_interrupt+0x158/0x240
[89196.112633] RSP: 0018:ffff88085f283e98  EFLAGS: 00010083
[89196.133296] RAX: ffff8878050402c8 RBX: 00000001408789f9 RCX: 0000000100878a01
[89196.154114] RDX: ffff88085b6090d8 RSI: 0000000000000001 RDI: 000000000100878a
[89196.174587] RBP: ffff88085b608000 R08: 000000000000000a R09: 000000000000000a
[89196.194569] R10: ffff88085b609038 R11: ffff88085f283e98 R12: 00000001008789fa
[89196.214207] R13: ffff88085f283eb8 R14: 0000000000000040 R15: 0000000100878a7a
[89196.233479] FS:  0000000000000000(0000) GS:ffff88085f280000(0000) knlGS:0000000000000000
[89196.271292] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[89196.290342] CR2: ffff8878050402e0 CR3: 000000007c989000 CR4: 00000000001406e0
[89196.309527] Stack:
[89196.328587]  ffff88085b609038 ffff88085b609438 ffff88085b609838 ffff88085b609c38
[89196.367147]  0000000000000001 0000510f73d99a80 00000001008789fa 0000510f73d9bca6
[89196.405620]  ffff88085f28be00 ffff88085f28d4a0 0000002f77794000 ffffffff81087db0
[89196.444018] Call Trace:
[89196.462567]  <IRQ>
[89196.462803]  [<ffffffff81087db0>] ? tick_nohz_stop_sched_tick+0x190/0x280
[89196.498867]  [<ffffffff81087fe5>] ? __tick_nohz_idle_enter+0x95/0x160
[89196.516814]  [<ffffffff8103db3d>] ? irq_exit+0xcd/0x120
[89196.534359]  [<ffffffff81027ef7>] ? smp_trace_apic_timer_interrupt+0x67/0xa0
[89196.552048]  [<ffffffff81536f17>] ? apic_timer_interrupt+0x67/0x70
[89196.569446]  <EOI>
[89196.569681]  [<ffffffff8100ae37>] ? mwait_idle+0x57/0x90
[89196.603026]  [<ffffffff810670cf>] ? cpu_startup_entry+0x17f/0x2d0
[89196.619580] Code: 6c 24 20 41 89 f9 41 83 e1 3f 45 89 c8 66 0f 1f 44 00 00 49 63 d0 48 c1 e2 04 4c 01 d2 48 8b 02 48 39 d0 74 24 66 0f 1f 44 00 00 <f6> 40 18 01 75 10 4c 8b 78 10 be 01 00 00 00 49 39 cf 49 0f 48
[89196.670370] RIP  [<ffffffff81079808>] get_next_timer_interrupt+0x158/0x240
[89196.686977]  RSP <ffff88085f283e98>
[89196.703452] CR2: ffff8878050402e0
[89196.719939] ---[ end trace 78a95702fb625015 ]---
[89196.736629] Kernel panic - not syncing: Fatal exception
[89197.813247] Shutting down cpus with NMI
[89197.831535] Kernel Offset: 0x0 from 0xffffffff81000000 (relocation range: 0xffffffff80000000-0xffffffff9fffffff)
[89197.864948] Rebooting in 10 seconds..
[89207.873458] ACPI MEMORY or I/O RESET_REG.


I personally find this use of `if let / else` as an initializing expression hard to read.  I don't know whether we have style-guidelines here.

(Admittedly I suspect some other people would find my preferred alternative:
```rust
let is_datum;
if let ... { is_datum = ...; } else { is_datum = true; }
```
to be just as hard to read, if not harder.)


我把这个依赖删掉。


It is the fold method on a list. Look at Odersky or scaladocs for a list

On Thu 2 Jul, 2015 6:10 pm Sayantan Khan <notifications@github.com> wrote:

> What does the ":\" operator do? This is the first time I'm seeing this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/siddhartha-gadgil/ProvingGround/commit/3717c00cb5942c3a75fd58c4cb86e336a51ef9c5#commitcomment-11973728>
> .
>


problema de idioma


Misspelled canola


commit lgtm


Out of curiosity, is this the commit that changed all the URLs in the Floraverse RSS feed, causing (at least in my reader) every single one of the 323 pages so far to trigger a fresh notification?

If so, could you please avoid doing it again? 😊


yay docs!


`accepted` is no longer a property of `Ledger`. `isAccepted` can simply return `closed()` - all closed ledgers are accepted. All open ledgers are not.


p.s. has there ever been discussion about allowing some intrinsics to be used outside of `unsafe { ... }` ?  It certainly seems like  requiring `unsafe` is overkill for `likely` and `unlikely` ...


This gives a compile error since Map is of type <String, String>


あとで見てみます！ http://tinkerer.me/


Confused why this and the other cast were needed. Because of extern "C" thing?

It was a feature that you could use `sol_vector_get()` without casting.


sorry, my bad ...


С файлом плохое решение. Глянь как я сделал с нотификейшенами, через SharedPreferences


why not just logging the results to make a comparison with the actual filter?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.430.baa5af84c0cd56190cd885657fbc14e83fcb1fe1](http://build.palaso.org/viewLog.html?buildId=59226&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3228, ignored: 119 Build time: 00:06:27


We also need to grant heat_stack_owner role to the admin


Re-reading the commit log this seems to be about avoiding large embeds via ajax. What makes an embed "large" ? We're not talking about the pointed-to resource, are we ?


+2

@bdilly @dorileo @demarchi as a potential future improvement, could we use glob to avoid enumerating the files inside the include dirs (since the directory is just for that), or is it bad for make?


Wouldn't it be better to replace the bankAccount argument with the bankAccount ID? In the current situation you would always need to pass the entire object. This isn't very transparent as this object is generated by another request. If you would save the bankAccount ID to a datasource you would not be able to easily call this function because you would need to recreate the entire object structure.


@ibriano we'll not ship our own modules but we still may have some "third party" modules that could be loaded, since the implementation using dlopen will still be there then it makes sense to keep linking against dl.


Please don't hard-code the GPIB address - please do it like the other instruments


What will be the size on 64 bit system (22 byte)? Don't we need a padding variable at the end of the structure like done in tDllIdentParam?


Then you need loadable modules support. If you don't want to ship our own, you disable those specifically, but disabling module support means there's no way to load any. Not even third party.


Thanks :+1:


There nothing new there. It just make clear the distinction between a schema and a real irn. In schema `?org` is just a sort of param that can match with any valid string. a string like `irn` just match with itself


This is not a pull request yet. I'll wait for the creation of one before reviewing.

Although I can tell you that the fact that <<<<<<< HEAD appears in the commit leads me to think that you did not rebase your repo cleanly. You should rebase, make sure you have the marketplace.xml latest version and then apply your changes.


Put the first entry on the next line.


No need to do it here - do it like the other instruments.
Maybe query the PROLOGIX adaptor and pre-populate the field with ttyUSBX


upps so peinlich ^^


@cmarcelo this is propagated only to shared lib build, if this non-linux build you're concerned about is things like riot and or contiki then it doesn't change too much since we'll never use the shared lib with them but the static one, which will not be affected by this change.


Even when there's one file, put it in a line of its own.


problema de idioma


composer.phar is redundant here.
Can be composer.lock instead


1: M111 unrecognized component name: WIP global


MOST UP TO DATE VERSION THAT WORKS. RUN EYEMODELFITTER.PY


It's not neccessary


+1 with the suggestions above.

Thanks for this. Avoided touching because was afraid of conflicting with other patches.


Player start with Level 1 and Lvl Up to 2 with this here!
Change to -                new Level { LevelIndex=2(instead of 0)	, XPRequired=1	    , EngramPointsEarned=3  },


The reason we need at lest the primary email from this list of emails is that it's the only way to get the info if it's verified or not.


First Commit


@ibriano  I see your point, my explanation was just taking in account the current implementation, as far as I remember right (not checking), disabling modules will not rip out the parts where we do dlopen...

I have proposed a patch set improving the moduleless builds, I can check the aspects discussed here and see how we want to go...


我是这么想的。

业务开发中基本上都要用到路由，而面包屑的状态也是 95% 情况下要和路由绑定的。单独使用会很麻烦，也没啥意义吧。


To make it more general this field could be called "Anzahl der Listen".


Instead of creating a new `vec4` and copying the alpha component, you can just assign to `color.rgb`.


because the actual settings change is used via a `sed` command, I think this open should be in read-only mode? @carolineclark


Indentation looks odd here. Run js-beautifier.


@fjuniorr Passei uma base da despesa, por engano, para a mde_rec. O erro retornado foi "Campo RECEITA_COD possui valores inválidos.", quando o correto seria retornar "Campo RECEITA_COD não encontrado".

A causa disso é que a linha a seguir inseria a coluna RECEITA_COD na tabela, com valores nulos:
```r
base_rec$RECEITA_COD <- format(base_rec$RECEITA_COD, scientific = FALSE)
```


instead of journal: store them into a separate data store (or even database)


That is interesting.  Maybe it is better to just make xbuild work out-of-the-box, remove the makefile?  What do you think?  I am in the process of putting together a whole pile of xUnit tests for the SDK, so will these also work with Linux / Mono?


The initial value can be removed ;) And also the comment


Thank you! That fixed the issue.

From: Vivek Krishnakumar <notifications@github.com<mailto:notifications@github.com>>
Reply-To: Arabidopsis-Information-Portal/PhosPhAt-API <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Wednesday, July 1, 2015 at 5:43 PM
To: Arabidopsis-Information-Portal/PhosPhAt-API <PhosPhAt-API@noreply.github.com<mailto:PhosPhAt-API@noreply.github.com>>
Cc: Ismail Liban <iliban@jcvi.org<mailto:iliban@jcvi.org>>
Subject: Re: [PhosPhAt-API] Fixed metadata (bd8d292)

Hi @iliban<https://github.com/iliban>, I came across your JIRA ticket where you mentioned you encountered an issue when registering your ADAMA adapters and that the likely reason is an issue with the metadata.yml file. Your conclusion is correct.

To fix this issue, you can alter the above highlighted line as shown below, by replacing the slashes (/) with dots (.):

services.experimental_phos_all.main.py

Doing this across the board for all your metadata.yml files will make everything OK.
Hope this helps!

Thank you.
~Vivek

—
Reply to this email directly or view it on GitHub<https://github.com/Arabidopsis-Information-Portal/PhosPhAt-API/commit/bd8d29245cbab52656d99e59aa2031abb3389900#commitcomment-11963192>.


Please move this code to the script tag in head.


sorry @carolineclark , please disregard my previous comment, we are actually writing to the file


Ok then. I'm fine with this matching the current implementation then, but I do think that disabling module support should remove all related code.


sqlite_timeseries (mbadams5) - x86_64-MacOS-10.9-clang: OK (1713 of 1713 tests passed)

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


I don't know why I should provide all these links


ack


very good that you added this, very important, people must not think that they can get good query performance from event streams for arbitrary queries - I hope this is enough to make that clear


It seems we need those casts for C++ compilers even with extern "C" thing.


ok


I don't like waiving the paypal donation option into everyone's face.


The size will 24 on both 32 bit and 64 bit systems.
Due to reordering, a padding of 2 bytes will be only introduced by the compilers on both systems at the end.


@dorileo got it.


Because of using it in a c++ project.
It complains about these casts, that needs to be explicit.


Noticed the following warnings on Linux:

eventkcalintf-circbuf.c:249 : passing argument 4 of ‘circbuf_writeMultipleData’ makes pointer from integer without a cast [enabled by default]
hrestimer-linuxkernel.c:448 : format ‘%lX’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘tTimerHdl’ [-Wformat]
eventkcalintf-circbuf.c:249 : passing argument 4 of ‘circbuf_writeMultipleData’ makes pointer from integer without a cast [enabled by default]
eventucalintf-circbuf.c:381 : passing argument 4 of ‘circbuf_writeMultipleData’ makes pointer from integer without a cast [enabled by default]


Is there a way to set a default service when using "maps"? [I do not grasp this right now](https://github.com/SemanticMediaWiki/SemanticMaps/blob/master/SM_Settings.php#L16)


文学っぽいですね


As it is now, the hare will almost always win. The hare will rest if the random number is less than 80, which is too low. I fixed it locally and I'm trying to figure out how to push it as an update.


Maybe we shouldn't use `user:email` as the default scope for github service as we don't really need that not that we decided to go with using `identity.email` over the primary email for the `email` field.


This is a bad assertion because it breaks gcc's `-Wswitch` by having a `default` case. (Also because the message will be unhelpful. If you want to guard against an "impossible" code path, use something like `assert(!"Unexpected value for light drawing pass")`.)


heh, the classic `if x == true { ... } else { ... }` redundancy, except in the generated code, right?  :)


Hi,

I think that maybe creates a problem with readthedocs… I remember there was a reason we went with the absolutes.

From: Michael Quested <notifications@github.com<mailto:notifications@github.com>>
Reply-To: ARMmbed/documentation_docs <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, 2 July 2015 14:16
To: ARMmbed/documentation_docs <documentation_docs@noreply.github.com<mailto:documentation_docs@noreply.github.com>>
Cc: Admin <irit.arkin@arm.com<mailto:irit.arkin@arm.com>>
Subject: Re: [documentation_docs] Again (868ec75)

Hi Irit, can we make the links to these images relative instead of absolute? Currently this cannot generate a pdf on the docs site due errors with the path. Something like Images/DocsLogin.png instead of /Docs/Images/DocsLogin.png works fine

—
Reply to this email directly or view it on GitHub<https://github.com/ARMmbed/documentation_docs/commit/868ec757a504fb3d8e6c2b02a6b0167f04b66d6c#commitcomment-11974686>.

-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2557590
ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2548782


:+1:  看文件介绍 “Git extension for versioning large files”，放在增强目录下面比较好，新开个目录 “git 增强” ，放在`最佳实践备注`上面吧。 意下如何？


It seems a good idea to me.
Now that files are separated it should be doable.


:+1: looks sweet


Right. only resv variable is added. The structure was moved earlier to avoid multiple pragma definitions. I will move it back.


Yes, that would be ideal. Good to hear that you are using xUnit. I tend to steer away from Visual Studio unit testing and use NUnit mostly. The last time I checked xUnit I found it didn't work well with ReSharper and so reverted to NUnit. However that may have changed and I'd take another look! Cheers.


the block expression changes seem right to me ... i hope they work.

I don't know all the syntax for the codegen test, so I'm just going to rubber-stamp it.


This is already implemented within `logt`


On 32 bit targets this is critical, you convert a 64 bit value to a pointer holding 32 bit addresses!

GCC on Nios gives a warning... "dllkcal.c:366 : cast to pointer from integer of different size" like all other casts do!


I don't think `get_apps_in_domain()` is cached, in which case you should probably store the result instead of calling it in each list comprehension.


Not sure I understand the question. You can set the default in the Maps extension, which will be the default uses by SM unless you set another default there. That is the two lines you linked to.


Okay thanks!


llll


So i guess R path does not solve the issue?


this is a free project


De ce get_. Peste tot e fara, hai sa respectam o conventie


As this method may be overridden by people willing to change this behavior, some javadoc would not harm:

```
A deployment failure has been encountered. You can override this method to customize the behavior. By default it closes the `vertx` instance.
```


I checked and it supports:
MAV_FRAME_LOCAL_NED = 1,
MAV_FRAME_LOCAL_OFFSET_NED = 7,
MAV_FRAME_BODY_NED = 8,
MAV_FRAME_BODY_OFFSET_NED = 9
So yes it makes sense to have this!


ss


thanks. I will fix this


Any reason to delete this ?


Mi se pare putin hackish. Ce ai vrut sa faci aici?


I thought that it was the `Application` trait that had been deprecated in favour of the `App` trait?


If a script error occurs during check, this line will never be executed and the request will be sent to the server. You may consider to use the jQuery submit handler and use ´event.preventDefault()´ to stop this from happening.


Why `INVALID_EXPRESSION` instead of `NO_VALUE`? (You could use `ValidationUtils.rejectIfEmpty()` here.)


Я уже нашёл работу и начал работать, когда Ильдар нашёл время ответить,
что, мол, "извините" ))
Хотя в своём решении я не увидел явных косяков, про которые написал Ильдар
в общем ответе.
Но само задание было интересным. В любом случае хочу походить ещё по
собеседованиям
Сами-то как?

чт, 2 Июл 2015, 16:11, Paul Blaze <notifications@github.com>:

> @anatoly-cmdx <https://github.com/anatoly-cmdx> @must-not-code
> <https://github.com/must-not-code> мужики, как успехи?)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/must-not-code/scraper/commit/cfc42008c70d5583949d7e5b99df17d50b21f213#commitcomment-11974580>
> .
>


there is the book => the book "Livro de Teste" is stored...


never mind I see that was fixed in later commit.


3: L102 copyright year is outdated, expected 2015 but got 2014


22:1: I101 Imported names are in the wrong order. Should be g, request, url_for
22:1: F401 'request' imported but unused
23:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.
25:1: I100 Imports statements are in the wrong order. from itertools should be before from intbitset
25:1: I201 Missing newline before sections or imports.
27:1: I201 Missing newline before sections or imports.
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from six
30:1: F401 'Collection' imported but unused
132:9: B901 blind except: statement


22:1: I101 Imported names are in the wrong order. Should be ModuleAutoDiscoveryRegistry, RegistryError, RegistryProxy
26:1: I100 Imports statements are in the wrong order. from invenio.ext.registry should be before from werkzeug.utils


43:1: F401 'cStringIO' imported but unused
48:1: F401 'session' imported but unused
51:1: F401 'current_breadcrumbs' imported but unused
54:1: I201 Missing newline before sections or imports.
54:1: F401 'iteritems' imported but unused
55:1: I201 Missing newline before sections or imports.
58:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.local
59:1: I100 Imports statements are in the wrong order. from invenio.base.decorators should be before from invenio.base.globals
59:1: F401 'templated' imported but unused
61:1: F401 'register_template_context_processor' imported but unused
64:1: I101 Imported names are in the wrong order. Should be format_records, get_output_format_content_type
67:1: F401 'facets' imported but unused
73:1: F401 'get_search_query_id' imported but unused
75:1: F401 'wash_search_urlargd' imported but unused


didn't think about this implementation, well at least I learned something today. :)


These constants seem arbitrary. Where are they from? Why don't they add up to 1.0?


You should rescue `Redis::BaseConnectionError` instead.


Shouldn't you check `v` against `csp.CLOSED` rather than `ch`? Or am I misunderstanding something?


我把强依赖去除了。https://github.com/ant-design/ant-design/commit/925f811b45e6377157ba882dad6286a24e4ccbc3


:+1: That's exactly what I wanted to know. :) I will quote you on this. :)


Is `C4DP_Last` a valid value for `C4DrawPass`? If not, why doesn't it have the same value is `C4DP_Color`?


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


This looks like it should be `C4DP_Last`.


Should the "HAMEG Instruments,HM8123,5.12" and "HEWLETT-PACKARD,53131A,0,3944" be:
     #define HAMEG_HM8123 "HAMEG Instruments,HM8123,5.12"
and then just use that below?


This tab doesn't belong here.


This would effectively remove existing support in IE8 and 9 for equal height.We could use Modernizr to detect FlexBox support and then run the JavaScript it FlexBox isn't supported. You will also need to test in IE10 because that browser is still in the support window and requires prefix to support it (autoprefixer might handle that).

Also, this is not a small fix. It would need a full suite of unit test to make sure nothing breaks.


/cc @masterbee @pjackson28


the route has nothing to do with your filesystem therefore you don't need all the /home/pi/druckersripte/ ... path stuff.
"/input/<pid>" should be sufficient
Also the sending works by sending a request to http://10.0.0.1 + ROUTE
where route is whatever you have specified  in @app.route("") so fre /input<pid>

you have to send data to http://10.0.0.1/input/1
for pid=1


I've also disabled travis-ci integration for the time being so that contributors don't see the scary red "x" of doom.


Yep,  good


@lgu1234 Please review these changes.  The deleted calculations of CO2 (and GenContam) for the plenum outlet seemed misplaced since ZoneContaminantPredictorCorrector is calculating the CO2 and GenContam concentrations for the plenum zone.  So, I changed this to set the plenum outlet to the plenum zone node condition similar to the way temperature is done.  But I am not completely sure if this is correct.


If you type Fix #20 in the commit message it is automatically closed https://help.github.com/articles/closing-issues-via-commit-messages/


`Get{Red,Green,Blue}Value` already return `unsigned char`, so this mask seems superfluous


@anatoly-cmdx я похоже пропустил основной движняк, пока был в отпуске, поэтому спустя неделю Ильдар написал "извините, уже нашел двух человек"


@b0rn2c0de, @gauravagerwala, @CoderHam: Where is LoginProgress? Also, Where is Pull to Refresh?


saw approval from JeetKunDoug
at https://github.com/basho/yokozuna/commit/47d7448d3af4369bcc80f2d726324b74720ef7dd


merging basho/yokozuna/backport/zl/merge-develop-into-2.1 = 47d7448d into borshop-integration-522-backport/zl/merge-develop-into-2.1


basho/yokozuna/backport/zl/merge-develop-into-2.1 = 47d7448d merged ok, testing candidate = 6f402da3


hmm, is this help emitted through some other path?

That is, are you removing a redundancy?  (Or is there less help output after this commit is in place?)


Nit: s/png/bmp/


I guess my big concern here is whether truncating in this way would lead to non-unique meeting names. That being said I don't know a better solution. Best to get @joel to weigh in on it.

Also, I think you want three dots not two. Otherwise the string is going to be one character longer than MAX_NAME_LENGTH.


`glScissor` takes `GLint` and `GLsizei` arguments, so using a `double` division here is probably not what you want.


**1.** Can you have a look at the [Minimizing Plugin Footprint](https://learn.jquery.com/plugins/basic-plugin-creation/#minimizing-plugin-footprint) section of the jQuery plugin guide and apply the techniques explained there? The end result of calling e.g. `setAxis` should look like this:
```javascript
$('#card').flip('setAxis', 'vertical')
```
This way, we keep only using one slot in the jQuery namespace.

**2.** I like the `callback` parameter you added to `setAxis`. However, on line 172 you are calling the callback directly. This gives 'unpredictable' order issues.

E.g. if we have this code:
```javascript
$('#card').setAxis('vertical', function(){
    alert('World!');
});
alert('Hello');
```

... the order of the `alert`s would be either 'Hello', 'World!' or 'World!', 'Hello', depending on whether the axis needed to be set or not.

If you use `setTimeout` to call the callback function, like this:
```javascript
//If we didnt have to set the axis we can just call back.
if (callback !== undefined){
    setTimeout(callback, 0); // callback will be called async
}
```

... the order becomes predictable. It will always be 'Hello', 'World!'

**3.** Could you please prepare an example of using it (e.g. on [JS fiddle](http://jsfiddle.net/) so I can better review it?


I wouldn't bother with an interface; just make it a public type.

Also, change the constructor to take all the settings as parameters. And move the consts into router.go. And ditch `DefaultTokens` (or `DefaultCapacity`).

Can't comment on the algorithm; ask somebody else.


´$tempName´ is never used again


а для чого нам це якщо ми все грузим на '/' ?


oh, and the method should just be `Wait()`.


`assert type == "BMP"` seems like a reasonable addition.


I think you pointed this out in a comment, but do be cautious about "if x" vs. "if x is not None". I tend to reserve "if x" for actual booleans, and lists/dicts/other containers (for which it tests whether they are empty).


This shouldn't be `long`. If you're storing a RGBA color value, use `uint32_t`.


all 하면 모든 동작에 애니메이션이 걸리니까,
사용자 입장에서도, 성능 이슈면에서도 나쁘지 않을까요.....?


niet erg mooi omdat hierdoor in bijv. de edit control ook deze tekst staat bij (initieel) openen van de popup (terwijl er wel waarden/lagen in de combo staan)


There's no point using `Nillable<long>` here unless you want to handle the `nil` case different from a defaulted `0`.


тому що '/api' це нe static, Все що пiд '/api' будуть REST ендпоiнти.


Promises for the world ;-)


Spaces after opening brackets and before closing brackets are missing.


@dwtj , will we have a single parent pom as well which contains `at-paninij-benchmarks`, `at-paninij-core`, and `at-paninij-examples`?


latest working version, run eyemodelfitter.py


Locale is not active if you use --noacl  hack.


Awesome!


Maybe we should extract `ns-path` to some of the util namespaces to avoid creating deps between middlewares.


一切的开始，感觉不错

今天是7月的第二天，依然的黑色星期四，上证指数一度跌近3800点。就在三周前，所有的人都还是那么疯狂的吹捧着股市，3000点，4000点，5000点，弹指可破，轻而易举，只要钱砸进去都能日进斗金。5月28日（周四）的突然暴跌仿佛打开了一场悲剧的序幕，然而随后周一的暴涨让人们迅速找回高歌猛进的信心。再来即使习大大的生日，随之迎来了最暗无天日的暴跌，打的股民们毫无招架之力。我虽然有幸在5月27日清仓躲过一劫，然后随之的贪欲作祟，为了能够参与打新股，我也在市场开始暴跌时持仓市值2万元左右。好在端午节北京回来以后，在一个号的情况下居然让我打中了红蜻蜓，算是弥补上了连续2周暴跌造成的4000获利回吐。人总是喜欢在偶然性背后寻找必然的关系，就好比红蜻蜓，是我老婆结婚时穿的那双红色婚鞋，婚后就再也没有穿过。
今天红蜻蜓（603116）在上证暴跌3.48%的情况下第四天挂上了涨停板，浮盈16151元（91.228%）。同时，我计划做T的吉鑫科技（601218）连续下挫后在8.21入了一手，这是第一次尝试做T，市场已经恐慌，但是我依然相信后市能够扭转过来，这仿佛已经上升成为一场政治斗争。其实我也不愿意看到全民炒股的局面，虽然我也不是专业的，但也有自己的投资理念，人生投资的第一笔收益过万的操作就是定投的指数基金。市场趋向安全，稳定，才是真正众赢的局面。
跑步，减肥，减脂。被嘲笑一次次的信誓旦旦，今天也是突发奇想，结合今天下午在办公室的聊天内容，觉得至少自己应该做些什么，和周围的环境区别开来，让人耳目一新，那也就只有减肥了，让自己变得不同，变得精神焕发。今天晚上成功连续跑了2.1km。那种感觉简直无法描述，在1公里时身体出现过松懈和疲态，不过坚持过去就感觉步伐轻快很多。尝试轻腹上阵，这样可能还能够跑更长的距离。
一定要坚持下来，相信自己，相信自己一定会拿物质鼓励自己的！


Actually no


You can use `var _ = lodash.runInContext()` too. This will create a pristine lodash instance that can be augmented and extended without modifying the `lodash` package instance itself to avoid potential conflicts.


Sad to see those lines removed...


add space after comma


Why we can not just use EntityDtoMap?


That's what happens when you dig down and really understand the inner workings of everything you're using, it took a lot of learning to get there. This technique is also not compatible with any regular browser. Phonegap only


Referencing to the warnings created e.g. on Nios...

What about using a union here...

````
typedef struct
{
    union
    {
        void*     pEventArg;    ///< Pointer to event argument
        UINT64    placeholder;  ///< Placeholder to achive 32 with 64 bit compatibility
    };
    // ...
} tEvent;
````

.. but I'm not sure about the correct alignment of pEventArg.

What about using uintptr_t?


baixo nível


Btw, using the line might be useful for compilation errors, as you'd use it as an offset from the start of the expression you evaluated.


sqlite_timeseries (mbadams5) - i386-Windows-7-VisualStudio-12: Build Failed

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


Please look at those changes - this is, I think, what the idea with DUs might look like.

You have a DU case as the data and that data is directly used to then set the appropriate option on the buffer.

This works pretty well for setting values. For reading them, I think, one would probably have to pass one of the constants to get the data out? Does this make more or less sense?


Do you see a possibility of data loss on 32 bit systems which will critical for the functionality?

I am not sure if currently there is a generic solution to this problem. I only see following possibility to avoid the compiler errors:

Define a MACRO data type for pointer typecast say `PTR_TYPE` with the standard pointer size of the platform
#if defined(__NIOS__)  ||  defined (__MICROBLAZE__)  ||  defined (__LINUX__)
#define PTR_TYPE   ulong;
#elif defined(__WIN32__)
#define PTR_TYPE ULONG_PTR
#endif

and use this to typecast the eventArg like
`pServFilter = (tDllCalAsndServiceIdFilter*)((PTR_TYPE)pEvent_p->eventArg)`

This really messes the code though. What is your opinion?


Note: As long as we have the ``etoolbox.sty.ltxml``, we can make this branch obselete.


Fixes #62


dobre che go iztri tui


rrthrsthsth


... or instead of placeholder the eventArg as you did....


Done.


arquivos que não deveriam fazer parte do commit


lol


现在还能说得通，可以支持传个配置对 react-router 有照顾，但不能依赖，很多场景都不是单页面，也谈不上 react-router


Thanks for letting me know, I'll look into it


This should be in the update_settings feature spec


Arantor, it's not that I think it's a totally bad idea, also, it seems that it got messed up in the topic, due to Illori and I actually discussing different things.

What I meant, is the post warning for old topics. That should be able to be a board option to set for each board, or not to set for those boards that should have topics replied to, no matter the date. It was a miscommunication between me and Illori.


This commit breaks a few things.
When I open TrackAtlas/TrackAtlas.etp an error is thrown: "(0) : error ERRSEM_VAR_NOT_DEC : Variable MRSPMaxSections : variable not declared"
When opening the SpeedSupervision TrackAtlasTypes are missing (obviously, as it has been relocated by this commit) Shall I correct that?

please advise,
Thorsten


rerewrerwerewrewrewr


I was thinking no. If you put `at-paninij-examples` inside of a parent pom, then `at-paninij-examples`’s pom will no longer be a “clean” example of setting up an @PaniniJ project.

- David

> On Jul 2, 2015, at 9:10 AM, Dalton Mills <notifications@github.com> wrote:
>
> @dwtj <https://github.com/dwtj> , will we have a single parent pom as well which contains at-paninij-benchmarks, at-paninij-core, and at-paninij-examples?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hridesh/panini/commit/d84d8fe051f118b9039dc5240c9f726328a4f11c#commitcomment-11975902>.
>


redirect?


sdfsdfsdfsdfsdfsd


The above two rows could also be written as:
```
next(reader)
```
`next()` will iterate the generator once. This way you can still keep `reader` as a generator.


Are you sure, this won't delete files that are currently being downloaded? Perhaps only files older than an hour?


Nice. This reminds me that our `err-info` util function doesn't add the `:root-ex` to the response. Display the enhanced stacktraces for middleware errors would be nice.


sbt-plugin-test fails in nightly because we forgot to update this line when upgrading to 2.11.7:
https://github.com/scala-js/scala-js/blob/f5f9ef074187201dae48e1309b47c3b2db2a0a38/sbt-plugin-test/referencedCrossProject/build.sbt#L4
It unfortunately passed the PR validation, because the machine on which that test was executed happened to have a locally published version of the Scala.js compiler plugin for 2.11.6.

I tested locally that this passes with the change of that version number. Since it is an internal test, and not something that ends up published anywhere, we'll live with it.


I will change it to only check on non-existent formats. As far as bogus max sizes, there's nothing to do there except fix the limits. We already enforce these checks everywhere else -- this is just a back door that was left open.


dfdfdsfdsfdsfd


this line should be removed


+1 for enumerage!!


first pull request


What is the purpose to have such clone?


RIM station is ignored?


contrib/dualprocshm/include/dualprocshm-winkernel.h:94: new blank line at EOF.


erro fatal


Simeon soon will be ru translate. RIGHT now I on red sea)))
Before 3 days I back


portabilidade


Yep, that's even what the stacktrace buffer does. Because this is only affects the `info` op, it doesn't remove the line from standard messages like errors or exceptions.


Thanks for the tip! I'll fix on my next round of related changes.


ok


Where we can edit these csv files? (WEB-INF/classes/it/geosolutions/geobatch/opensdi/csvingest/utils/withdrawal.properties i suppose) can place them there by default (so create a copy in the web module?


It's very unlikely that the tool will be used that frequently. But in the rare case of two requests at the same time, the name will not be unique.


use assert_true or assert_false please


too much indentation


Same as above, assert_true or assert_false. Raw assert is not following other tests.


same here for assert_true and assert_false


@jdalton Thanks, I'll try that. Should be able to fix https://github.com/sequelize/sequelize/issues/2281


here as well.


The package TrackAtlasTypes has been moved to inside the TrackAtlas project
in an attempt to consolidate the number of SCADE projects a bit.

Normally, as the path has not changed, it should continue to work.

Probably you only have to import the TrackAtlas project via file view and
remove TrackAtlasTypes

On 2 juil. 2015, at 11:19, Thorsten Schulz <notifications@github.com> wrote:

This commit breaks a few things.
When I open TrackAtlas/TrackAtlas.etp an error is thrown: "(0) : error
ERRSEM_VAR_NOT_DEC : Variable MRSPMaxSections : variable not declared"
When opening the SpeedSupervision TrackAtlasTypes are missing (obviously,
as it has been relocated by this commit) Shall I correct that?

please advise,
Thorsten

—
Reply to this email directly or view it on GitHub
<https://github.com/openETCS/modeling/commit/9d7fb06b31460a7960013f8aead23d4d6f20462d#commitcomment-11976097>
.


Maybe using a code of a typical card eg. One Day of Peace (33782437), just like 30459350 chk of Imperial Iron Wall?


The language is actually "de" :), but I guess, the attribute can be omitted.


This change breaks the GUI!

When click the search icon, the menu does not open. Instead:
Chrome and Firefox: all content is wiped inside the lealflet container.
IE11: a page reload is done.

Running version 0.7.3 of leaflet and version 0.7.11 of leaflet-angular.


Why sad? Does it no longer work across NATs?


factor is not used, it should be something like:
`ID_pri;distr;prov;year;mon;dec;commodity;*`
that can be similar to other resources. We can use the first field to identify it (ID_pri). I think it is a good compromise.


This signature seems to be ambiguous in DtoEntityMap context (when dto and entity mapping is reversed)


e
4$$$$


stturuy


ltsdh


Thanks for using my bootani and providing credit @Snuzzo! It was a pleasant notification to receive :)


you meant **properties** file right?


this introduces a hole of 16bits since the next alignment is 32bits. You can use uint32_t here.


This code is not extendable


@sdumitriu Should there be a logger message for when the patient could not be deleted?


I completely forgot that #56 was resolved in this commit as well.


Good !


yep, makes sense. three separate then


2 space indent here as well


Sure, created an issue for that: https://github.com/clojure-emacs/cider-nrepl/issues/225


Now all I have to do is get is readable.


can conflict with other ones i suppose. let's identify it with the first column title something like:
`id_withdrawal`


No need to change in this commit, but I wonder if we should have only "port_name" here or also provide a config (as done for GPIO) to set all the uart parameters. It's better and simpler than offering post-open setter functions.

@zehortigoza any comments?


Does reverting this commit actually solve a bug ?


what
?


no styles declared here, can be removed


Do we need that ";" ?


Вылетел по результатам тестового задания, по причинам «сплошной субъективщины». Только зря месяц прождал.


like this @fernandog ?


If execution of the job readds the job to the queue, it will get reevaluated right away. It will be a busy wait if I understand it correctly.


Do we need to have IO as a separate library? I think we should always provide it.


luis tem razão. uma única tarefa de refactoring poderia ter vários commits associados.


Thanks! These are great improvements.


Not sure I would recommend the second one, it is fine to use a static instance within the scope of one method call, but to use a static shared instance and also return this instance externally with get bounds() is probably not a good idea. Any developer should just be using the two arg getBounds call with their instance, and the property is just always going to have to create a new instance if someone choosing to use it.


minor: I guess you can also create an empty define for that


They should be reals.


It's about time. :D

I can add files to filepack without having to modify filepack.c now. Much cleaner then the old hash check function. :D


It built! Nice! It is awesome to be able to rewrite *correctly* unix applications into node9.
Do you have any implementation of wm/wm? any GUI?


It's still pretty preliminary atm. I've only experimented in Firefox so far, but intend to test in IE10/11 at some point. Also need to make it work happily with different screen sizes and further adjust the demo's CSS. I probably won't try to tackle JS until later on.

**PS:** The commented-out tabbed interface equalization properties aren't working properly. I doubt it'll be possible to get that working solely with flexbox CSS. Btw the existing equalization plugin isn't working properly with the tabbed interface either.


merge in one line


You need more space on the downstairs?


não estou vendo o field sendo encapsulado


new file needs to be listed in Makefile.am