thanks for taking the time for this detailed analysis!

So if I understood you correctly (the info here and from the mail you've sent me) to use method 1 (do a world-wide check that will maybe be made impossible in gwt 3.0 and or later) I need to call
```
com.google.gwt.reflect.client.ConstPool.loadConstPool();
```
in my client code, just like calling `GwtReflect.magicClass(Classliteral);` like I do here:
https://github.com/fhcampuswien/atom/blob/using-xapi/atom-client/src/main/java/at/ac/fhcampuswien/atom/client/App.java#L99

If I understood you (and your code) correctly - for this to work all of the subclasses I want would need to be annotated with your `@ReflectionStrategy` annotation like I did it here:
https://github.com/fhcampuswien/atom/blob/using-xapi/atom-core/src/main/java/at/ac/fhcampuswien/atom/shared/domain/DomainObject.java#L108
or to curcumvent this requirement the code that does the world-wide lookup here:
https://github.com/WeTheInternet/gwt-sandbox/blob/xapi-gwt/user/src/com/google/gwt/reflect/rebind/injectors/ConstPoolInjector.java#L72
would need to be modified to check each class if it is a subclass of any class that is annotated with `@ReflectionStrategy(magicSubtypes = true)`
Is that correct?

For the future (gwt 3.0 or later) we would need to generate a list of classes during build time before the gwt compiler runs. I use something like that for the JPA file `persistence.xml` This too needs a list of entities (if they are not all part of the same project). What I use there is the maven plugin com.edugility.jpa-maven-plugin, see:
https://github.com/fhcampuswien/atom/blob/using-xapi/atom-server/pom.xml#L200

Which produces a file `${project.build.directory}/generated-sources/entityClassnames.properties` which is then used replace a variable in the `persistence.xml` file with a list of all entities.


TeamCity TripleA :: UnitTest [Build 53](http://188.177.191.210/viewLog.html?buildId=115&buildTypeId=TripleA_UnitTest&guest=1) is now running


:flushed:


There is already a note before on the README: "Matching supports wildcards (using the python fnmatch module)". Isn't this sufficient?


TeamCity TripleA :: UnitTest [Build 53](http://188.177.191.210/viewLog.html?buildId=115&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 40 Build time: 00:00:27


oops... please maybe ignore this for now, there's some issues around tips now, they have noticed this:
https://github.com/golang/go/issues/13206


commit message: not all elements are concerned by this rework: only `ConfigurableElement`s


@hiberno Thank you for the warning.  Thankfully I have been having success with NixOS on VirtualBox and my little netbook (so far).  If I may ask, what OS did you switch to?


Can you also tell me what the embed keyword does? I couldn't understand how to use it?


@Tommy-Geenexus
Thank you!
I'll update with new version asap.


Gustav are you ok?


```java
       "window.medic_android && " +
           "window.medic_android.handle_back && " +
           "window.medic_android.handle_back()",
```
?


Nice Japanese characters there in line 201!


Для токена нужен индекс


cat


There's code in here to close #28, but apparently there's something broken because blank pages load.

I can't get error checking to work, so it's a guessing game for now.


лишние скобки


update ramdisk


sadasd


There is a small flick from the default info page to the console after `Shift-Cmd-C`. Maybe switch the order of show and set focus here?


Автоматически сгенерированные файлы с тестами и ненужными фикстуры тоже стоит удалять.


these crypto patchs may cause softbrick


Each new site requires the addition of a `www` folder by default, which will include all the contents for that app.


TeamCity TripleA :: UnitTest [Build 54](http://188.177.191.210/viewLog.html?buildId=116&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 54](http://188.177.191.210/viewLog.html?buildId=116&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Compilation error: Software\TripleA\CashRegister\CashRegister.csproj (new); number of tests is too small: 0 (new) Build time: 00:00:04


лишний пробел


You need to be able to have multiple connections associated with the MC, so a synchronized Set is needed


不知道


Good point. I can't see the flick on my system. But i have changed order now, should be better. Thanks.


Файлы с пустыми тестами тоже лучше удалять


Nice!


This should clear out the MC in the Cs, and clear the connection set


TeamCity TripleA :: UnitTest [Build 55](http://188.177.191.210/viewLog.html?buildId=117&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 55](http://188.177.191.210/viewLog.html?buildId=117&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 40 Build time: 00:00:27


Hi,

I'd like to ask if the value of pressure is correct, is it really edge[0] for both?

# Estimate emitter coefficient:
+            pressure = (noleak_df[edge[0]] + noleak_df[edge[0]]) / 2
+            emitter = LEAK_DEMAND / mean([pow(i, 0.5) for i in pressure])
+            g.node[new_node_name]['EMITTER'] = emitter


Да, уникальный индекс


Do a no-op implementation; just "return this" and implement on the class


Subject and CRI should be transferred to MC


Verify Subject and CRI


Wrap with `#ifdef SORA_PLATFORM` to allow compilation on platforms that do not support Sora.


Такой настройки нет, возвращается `NULL`. Из-за возникает ошибка mysql в «прямом эфире». Видимо, надо добавить эту настройку в `common/plugins/ls/config/config.php`.


Implement


You can get that from the MC


OK


Не было в описании задачи, но предложу полезное улучшение: вьюхи стоит перевести на slim. С маркапом на slim работать намного проще, мы (и не только) его используем повсеместно.


Implement the methods


Should be true


slim-lang.org


Тут сейчас генерируются лишние маршруты, для которых нет соответствующих экшнов в контроллерах. Нужно добавить параметр only или except. Например: `resource :users, except: :update`


Actually it does not matter to let you control my laptop remotely.
Just call me if necessary. My email is jyy@pku.edu.cn.
Thanks for your help anyway.


Also add the activation code to ResourceAdapterFactory - transaction support is one configuration parameter


TeamCity TripleA :: UnitTest [Build 56](http://188.177.191.210/viewLog.html?buildId=118&buildTypeId=TripleA_UnitTest&guest=1) is now running


This was a tricky thing to figure out. For the duplicate form workflow
the form attachment is required after the form's ID has been changed
and so when you try to look them up from the DB you can't find them
since you're using the new ID.

Solution was to pre-fetch them to a specific 'cached' attribute and
the check that list first before trying to fetch them from the db.


Меня не пускает из-за этого в свой вопрос


TeamCity TripleA :: UnitTest [Build 56](http://188.177.191.210/viewLog.html?buildId=118&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 40 Build time: 00:00:31


Ok, makes sense.  This is how it is in 3.3, but I can fix it up for Master.

1Hz loop?


It appears that your git client has somehow messed up the encoding of the Russian language files.


Ou comment noyer une classe simplissime sous 100+ lignes de commentaires répétant des choses évidentes.
Le méthodes create et nil peuvent être supprimées.
const unsigned int EMBNULL = std::numeric_limits<unsigned int>::max(); ne compilera pas avec VS2013 je pense.


Лучше так: <%= form_for @user do |f| %>


TeamCity TripleA :: UnitTest [Build 57](http://188.177.191.210/viewLog.html?buildId=119&buildTypeId=TripleA_UnitTest&guest=1) is now running


@pyros2097 embed isn't done yet. Right now, it does nothing. When it's ready, it will allow object inclusion - that is, a field embedded in an object without a pointer indirection, like a C struct inside a C struct (as opposed to a C struct pointer inside a C struct).


TeamCity TripleA :: UnitTest [Build 57](http://188.177.191.210/viewLog.html?buildId=119&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 40 Build time: 00:00:27


`self.token.nil?` тут лишний, в контексте active support `empty?` и `blank?` можно вызывать у nil.

     self.token = generate_token if self.token.blank?


Remove this line.


Beware, you bumped this version from 2.3.1 to 2.5.0 (if really needed, shouldn't this bump be part of #228 instead ?)


```
...
else:
     self.port = port
```


Is it intended to remove javax.servlet api from this module ?

Anyway, I'm wondering if it shouldn't be declared as a provided dependency here rather than compile dependency


Works for me, if the property checking is annoying then do window.medicAndroidHandleBack?


Yes it is a bit of a hack. This fix was already implemented for Linux on the [FileSelectorDialog](https://github.com/mono/monodevelop/blob/67e4cb5edf1fb25a80db5076aaf786489afe49ce/main/src/core/MonoDevelop.Ide/MonoDevelop.Ide.Gui.Dialogs/FileSelectorDialog.cs#L225) so I made a similar change for Mac.

It is broken in master and in 5.10 on the alpha channel. Looks like it has been broken since [2012](https://github.com/mono/monodevelop/commits/8cf92fb).

The problem is that the CompiledAssemblyExtension allows .exe and .dlls to be opened as a solution so that overrides the AssemblyBrowser. I tried removing the `selected = 0` from the first IsSolutionItemFile check but that has a bad side effect where .csproj and .sln use the Source Editor by default.

I am currently looking at the Windows dialog.

Another way to fix this would be to remove the CompiledAssemblyExtension. That would not allow an assembly to be directly opened as a solution which is a breaking change.

I guess another way to fix this would be for the FileViewer and IViewDisplayBinding to be changed. Currently it has a CanUseAsDefault property which is used to determine if it can be selected by default in the file dialog. If there was a method that took the filename `CanUseAsDefault (string fileName)` we could change the SourceEditorDisplayBinding to return false for project and .sln files by using the ProjectService.IsSolution


Like jetty module, I'm wondering if we shouldn't improve this and use provided scope for this dependency rather than compile one


Where are you using this?


Like my comment before, I'm wondering if this servlet-api version bump is a good thing

It may imply code incompatibilities with servlet 2.5 containers which is not (presently at last) willed to my POV (see recent discussion on #220 about this topic)


@santimarrone Esto es lo basico que necesito para que la actualizacion de objetos funcione, si llegas a modificarlo avisame :+1:


I'm mitigated about reusing ${maven.compiler.source} variable here, meaning "we use the same target as the source"

I would keep the initial 1.7 hardcoded value here
But doesn't make a big deal, it is just a thought.


Er.... why did you delete the whole config? We can just set flags on the __global__ region...


That's a good point. I wonder what's worse: failing or looping forever. Thinking about it, I think looping forever makes more sense. Failing would let user know that something is wrong, however, he should see a failing master, or something similar. And it's really the master that's at fault, not the slave.

@bparees @mfojtik: thoughts?


(Я принимаю задание, это так уже, на будущее комменты.)
С кейсом можно было бы ещё вот так поступить: поскольку в руби любое выражение возвращает значение, то можно (и красиво) писать так:
```ruby
m.date =
  case m.date.count('-')
  when 0
    nil
  when 1
    Date.strptime(m.date, '%Y-%m')
  when 2
    Date.strptime(m.date, '%Y-%m-%d')
  else
    #...
  end
```
(А ещё можно было применить красивый, но сомнительный трюк: `m.date = m.date.count('-').zero? ? nil : Date.strptime(m.date, '%Y-%m')` — поскольку дальше по коду нам нужен только месяц, и какого числа вышел фильм, не очень важно ;))


I switched back to Arch Linux. I am quite familiar with the way Arch Linux works and have not really had any serious problems with it. It feels a bit like coming home. The reason for me leaving NixOS for the time being is that I do not have the time to really learn Nix well enough to fix problems I encountered. While I really like the way NixOS is approaching building and maintaining operating systems, I do think that knowing and understanding Nix (the language) really well is essential to power-use it. And I simply do not have enough time to dig that deep into it. :( I hope to come back one day.


By the way, is `fetch` useful, somewhere?
It does not seem to be used.
Plus, this one does nothing.


It doesn't though. Any suggestion?


(Это тоже коммент на будущее) Имелось в виду, конечно, не только этот конкретный sort_by упростить, но и везде в остальных местах, где вызывается простой метод.


TeamCity TripleA :: UnitTest [Build 58](http://188.177.191.210/viewLog.html?buildId=120&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 58](http://188.177.191.210/viewLog.html?buildId=120&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 40 Build time: 00:00:27


Can't we have some reasonable timeout where the container will be restarted if this fails? The restart might give you updated resources/etc.


Нужно добавить валидацию наиличия сообщения и пользователя: `validates :message, :user, presence: true`


having container stuck in 'non-working' state forever without restart is bad


**Functionality - Awesome**
Good job on using scripts for scaffolding/set-up.


you should send an email to `$project->getUser()`


not useful imo :)


commit message: "over*r*riding". Please give more details.


Of `fixed` ;-)


Если имя здесь используется как уникальный логин, то нужно добавить валидацию уникальности


the commit message is not accurate: structure export is performed, but also settings set/get as xml/binary


Why `code` here?


@sebbulon Actually it was configured for `text` attributes, only `ltext` was missing! Now it's fixed. Will release a new version


:+1: Hm. It might've been due to some missing dependency at compile time? Along the same lines of requiring a managed package before `(package-initialize)`.


So, there are no more Updates (here)?
Or what did this README mean?


@santimarrone @Jorch95 Atencion a esto, la entrega se tenia que hacer con la version 1.7 de java, si llegan a tener problemas compilando y les tira algo de compatibilidad puede que sea por esto, cualquier cosa avisen!


whatever. ;)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4708](https://teamcity.mesosphere.io/viewLog.html?buildId=73780&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:18:00
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Ok thanks,
weird you have such issue, PoissonRecon seems to be yet developed with VS.


Если имя используется как логин, то нужно добавить уникальный индекс по этому полю


typo "overRiding" in the commit message.

The commit message could be more verbose.


ah, this got mixed up with my other old commit. Shouldn't be here.


TeamCity TripleA :: UnitTest [Build 59](http://188.177.191.210/viewLog.html?buildId=121&buildTypeId=TripleA_UnitTest&guest=1) is now running


You may not edit this file.


Commit message could be more verbose


nitpick: I'd prefer:

``` {.c++}
auto clean = replace ? [&pRootElement] { pRootElement->clean(); } : []{};
```


Can I ask why it has been removed?


TeamCity TripleA :: UnitTest [Build 59](http://188.177.191.210/viewLog.html?buildId=121&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 40 Build time: 00:00:27


Invalid alias


Для всех API-запросов пользователь в любом случае будет получен из базы и инстанциирован.
Текущий пользователь – это информация, которую стоит сохранить на все время обработки запроса, т.к. она может быть нужна для разных целей.

Поэтому тут стоит сохранять в инстанс-переменную не токен, а пользователя.
Механизм аутентификации обычно общий для всего api, поэтому этот метод стоит вынести из TwitsController в общий контроллер для api, от которого должен быть унаследован TwitsController.


report*


As I can see that you've ran a random test to implement java graphics using a buffer file!!
Great work!!
Next step would be to get a map corresponding to a real world scenario.


I'm not sure why've you done this?


Нужно добавить отображение ошибок валидации на форме, это просто сделать если для рендеринга форм использовать гем https://github.com/bootstrap-ruby/rails-bootstrap-forms


I've found this about versioning:

https://github.com/kubernetes/kubernetes/blob/master/docs/design/versioning.md#upgrades


creo que esto no es correcto


Add a comment why uint64_t is required


Объявлять переменную в конце файла не нужно ни при каких обстоятельствах.


Un grand bravo pour le boulot et la sélection des ressources présentées.
Un immense plaisir de participer a ce MOOC.


retested, tested ACK on OS X.

Thank you.


> Also, this new logic ignores ns.files_or_code if the first element isn't a recognized file name and ns.prevcmd was specified. Is that correct?

Yeah, I think that is correct. If prevcmd is given, it should override trying to source or run any code.


Yes, it's not a really serious problem so I didn't ping you about it on Copter-3.3 (or maybe I didn't notice).. in any case, 1hz loop sounds good to me.


Нужно отображение ошибок валидации на форме


У answer не проставляется user_id?


OK


Т.к. подразумевается, что приложение имеет api для мобилных устройств и страницы для просмотра в броузере, то маршруты для api лучше сделать с префиксом `api`.


And you are right that the logic should be flipped here.


этого больше не существует не парся, я все учел в новой версии. таже теги в виде поведений чтобы их к новостям можно было прилепить!


[Screencast 3] Une introduction à Git et Github


correct


Ok for the first.

I'm not sure with your second comment, as the /authentication endpoint goal is to provide a token.


[Screencast 1] Analyse de données en neurogénétique avec Python


Can we return the `override` specifier?


*Повышает скорость передвижения 1 союзника.


Для этих целей нужно использовать гем  [has_scope](https://github.com/plataformatec/has_scope)


Нет необходимости объявлять `d` выше и присваивать ей значение по-умолчанию.
Можно сделать:
`d || 10.years.ago`

return тут не требуется.


ah. good point. My guess is - yes.


ой я ошибся строчкой, выше для actionUpdateQuestion($id) точна такая же строчка.
и $question->user_id =1 и Yii::$app->user->getId()=1


when you are not authenticated, only POST is allowed


Or rather, updated. which I have done.


Sorry, I forgot to change that.


buffer->used = 0 is exactly what I want here. The idea is that I either return with steal/take or I overwrite the previous written data by setting used to 0. Calling init will set capacity to 0 and data to NULL, which is wrong, since I got memory already allocated.


The old syntax was kept for Webstorm 10 support.
We need to test if the new syntax works now in WS 11.


Sure, just haven't noticed it.


HD without hd? :/


OK


ok.


```php
var_dump(Yii::$app->user->getId(),$question->user_id,  (!Yii::$app->user->getId() !== $question->user_id));die();
int 1
int 1
boolean true
```


As you point out, this pull request does *not* fix the race condition because I don't know how to properly fix it.
However, it does fix the situation where the binding that starts up first destroys the serial port list for all bindings that start subsequently. So the window for a race condition is narrowed down from infinite to finite.
In practice I've been running multiple bindings that access the serial port using the same code as in this PR, and I have never been able to observe the race condition  break the start-up of the binding.
At some point this will need to be fixed properly. A static synchronized method in some "SerialPort" class that lives in a comman package comes to mind. I don't know my way around the build environment well enough to decide where such a class should live and how to access it from within the bindings.
Once a proper fix is available I'll apply it to the bindings that are currently using this band-aid (insteonplm, urtsi, alarmdecoder)


You are right that just looking at this patch on its own might not provide a ton of insight, but it is complete. You really also need to look at how `foreign_shells` is implemented, which allows for arbitrary setup & teardown scripts via the `prevcmd` and `postcmd`.


Also super sorry I have been slow to respond this week!  I have been at a conference, etc.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4709](https://teamcity.mesosphere.io/viewLog.html?buildId=73825&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:08
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


I think we need to get this discussed in the coding style thread, all exciting news should be posted with !!! and we should standardize on this. I am open to a sliding scale of !'s depending on how exciting the news is, but then we need to get into creating a comity to vote how everything rates on the excitingness scale. Maybe that should be split out into it's own issue if we go down that route.


Не упорядоченные импорты


Останется пустым?


Oh, there's really no need to invoke extract_first_word() at all. Just use rvalue, that's all we need.


ddc


This commit is being reviewed as a part of review issue #108.


I think two ! were enough in this case


Stripped out all code and moved it to index_archive.html.

Replaced code with meta redirect only to speed up redirect and eliminate the flash of the old version on the screen prior to redirect.


This commit is being reviewed as a part of review issue #63.


tady bych dal fakt classmap, v testech bych psrko nehrotil


exercices ERRADOOOO


funcao inteiro


Things like this can be done in a nicer way.
change this
```
if isJust table
    then Just $ PrimaryKey (fromJust table) n
    else Nothing
```

into this
```
PrimaryKey <$> table <*> pure n
```

the same goes for all the above cases


not the same thing. it can return both null, which is the abscence of anything, including nothing, and "null" which is just a string.


Yeah, I missed the `return`. Sorry for the noise.


Thanks :)


We probably want camel case here since it's JS.


here for example you would do
```
(,) <$> col1 <*> col2
```


Added all types and features from https://developer.mozilla.org/en-US/docs/Web/CSS/Media_Queries/Using_media_queries#Pseudo-BNF_(for_those_of_you_that_like_that_kind_of_thing)


@fprochazka to je moje lenost, abych nemusel generovat autoload, když mně napadne přidat nějakou další pomocnou třídu


Amy thoughts on wifI config


Could be but I am thinking task for a subclass, the classes as they stand are really basic, you'd want to write your own layer on top for convenience.


Ah. I changed some things from a c++ event which triggers a bp event to blueprint native events.


Didn't know that was a thing. Cool.


this is maybe a bit uglier in the aplicative stile because you would have to use "pure" a lot but still i think its a bit shorter


exercises/mongodb-aula-01-exercicio.md ERRADo


Why don't you want to return `resp`?


The instrument is basic waveforms for now with the goal of including envelope shaping later on.


Руки крюки)


Thanks.


@mart-e I kind of don't get why such a non breaking change is applied in master instead of 9.0
Isn't that a mistake?


this is
```
Column <$> table <*> pure n <*> pure pos .....
```
there might be a nicer way when you have one value in a context and the other ones are pure ... maybe ask @begriffs if he knows a way


Any chance a change like this could be publicized somewhere or point me in the direction it is? I just spent some time trying to figure out why a project that depends on this and a lot of other things went from working on JRE6 to requiring 7. Hint: it was this.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 Total|||||


This commit is being reviewed as a part of review issue #108.


There were some more missing table display properties. Added them from https://developer.mozilla.org/en-US/docs/Web/CSS/display#Syntax


Could you guys start using tags and incorporate those into release streams such as docker? Currently you can only pull latest from the official docker builds.


Наверное, стоит сменить на `!=` :)


I dont know but this releases exists, like shown in example :)


Why not. First I would like to see if Travis is happy with this change.
I can't figure out where the new behaviour comes from.


В Visitlog хранятся переходы на страницу события, ничего общего с "Я пойду" не имеющих. Обрати внимание на таск 50


Oh, Applicative is a thing :)


That's nice, I would take a look later and might instead contribute to yours.
I'm currently quite busy however, and won't be able to work on this.
Otherwise I like a lot lasagne interface and since I wanted to compare theano and TF I thought best way is to write a common interface and just switch the back end. So this repo might not live, since if you started it I'm quite against reinventing the wheel rather than helping out.


Forbidden (#403)
Наверное стоит сменить на `Yii::$app->user->getId() !== $question->user_id` всё же


I leave all the work on this overlay.

My new project`s on bitbucket.


'EventWorkspace' should be 'TableWorkspace' here and two lines down.


modif**ied**


Ah no it's ok. I would have expected that in case of an error, the response is returned with an appropriate status code. But in this case an exception is thrown.


Note to self: webjars-locator goes to JRE7 in version `0.24`


alignment


Done


Can you compile the most recent version into a JAR?? As I had a few errors when I tried


@rodrigopavezi Just curious. What are these 2 files?


I think this commit has nothing changed


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4710](https://teamcity.mesosphere.io/viewLog.html?buildId=73838&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:06
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


@astec plugins should not be included in the core just as the HTTP, Router should not be in the core of Angular


修改一下


+1 thanks to @pravsripad


This creates 2 regressions see https://issues.jboss.org/browse/WINDUP-823


Weird indentation.


I was wondering about whether it is possible for `_G.unparsed` to be `nil` or empty, but I was a bit lazy to dig into the `parser:parse` call to find out.

We should make sure we do some checking here before trying to extract the filename. Otherwise it will blow up when the sile binary is invoked from the commandline without a filename (which is allowed I think, because it drops SILE into a repl)

I just tried that on my system and it gave this error:

```
$ sile
/usr/local/bin/lua: /usr/local/share/sile/core/sile.lua:91: attempt to index field '?' (a nil value)
stack traceback:
        /usr/local/share/sile/core/sile.lua:91: in function 'parseArguments'
        /usr/local/bin/sile:7: in main chunk
        [C]: in ?
```

A simple fix is to copy this line from `sile.in`:

```
if _G.unparsed and _G.unparsed[1] then
  SILE.masterFilename = _G.unparsed[1]:gsub("\\", "/")
end
```

P.S. : [here is the doc][doc] for `std.optparse:parse`:

[doc]:http://rrthomas.github.io/lua-stdlib/classes/std.optparse.html#std.optparse:parse


Смысл фразы ускользает.
Если "Добавляем наше расширение в список доступных расширений", то "Add plugin into `extensions` list."
Если называть расширение плагином, то и в '!!!_PATH_TO_EXTENSION_!!!' тоже нужно plugin.


Comment looks incorrect.


If the score index is non-unique, there is no need for this disambiguation by txid.


horrendoys


Shouldn't we break just after we found the security_token or session_token, so we avoid an extra pass and a strange behavior if the user has both values?


thx for the contribution bud


how do you even

what


Checked against:
http://www.w3.org/TR/css3-transforms/#identity-transform-function

Also reverse-sorted to prevent premature captures.


Does this only work on Rails master/5.0.0?

```
undefined method `enqueued_jobs' for ActiveJob::QueueAdapters::SidekiqAdapter:Class
```


this will not be rendered properly


same here


Typo in commit message: consitency


I think you don't want `EINVAL` here. `man strtol` says:

> In particular, if *nptr is not '\0' but **endptr is '\0' on return, the entire string is valid.

> EINVAL (not in C99) The given base contains an unsupported value.
> ERANGE The resulting value was out of range.
> The implementation may also set errno to EINVAL in case no conversion was performed (no digits seen, and 0 returned).

For "least user surprise" you probably want to declare an extra `char *endptr`, pass that into `strtol` then check `if ( network_timeout == 0 && *endptr != '\0' )`, as I understand it from the man page.


Same as above. Yes, I hate C too.


これはいいですね！


herdyher


Do you really want to call the run 'instrument'?


aaayyy


added it


The method name `have_enqueued_job` clashes with the [rspec-sidekiq](https://github.com/philostler/rspec-sidekiq) gem. This means rspec-rails 3.4.0 and higher are not compatible with rspec-sidekiq.


yes, it does nothing because it does not work against the current ghc-mod. It has been left in as a stub for future when it goes in.


А, тьфу :) Откуда там `!`? :)


here is where you change port


sort(1) ignores whitespace when sorting and DanielG is before DanM. It's counter intuitive but that's what sort(1) does.


Всё с вами ясно :) .... заработались, надо на отдых.


How come we're removing the daemon tags?


insecure_rand() returns a uint32_t. Adding another shifted by 30 places won't change that. You need to cast to int64_t (or CAmount) first.


> Can we use 'cat .mailmap | sort' to get the sorted file?

This is what I did with sort 8.21


I just added the params that ghc-mod exposes


fixed


You have a duplicate parameter... how can this work?


@e-kolpakov The problem was fixed in OC-1021 (#88), so it is possible to use Step Builder without a review step now.


@e-kolpakov IMHO it makes sense to have this logic in one place. It used to be spread out over two functions; this one and `validateXBlock` (see further down the diff), which made the scrolling issue quite tricky to debug. I tried to make the logic easy to follow by adding comments to the lines that contain `if` statements; if you read them from the top (and put "and" in between), they should make it clear where you are at at every nesting level. If you think the comments could be improved somehow to make them more helpful, let me know :)


Right ! My bad.


Ok, i will also split this class.


Splitted.


BTW - the changed function is `getnettotals` according to your screenshot (the commit message says `getnetinfos`).


You are not associating this collaborator with either a wiki or a user. You need to do both.


Good point, I'll fix that (and let the FreeBSD folk know they should do the same).


Yep, I'll reword.


It's logically where the rights belong, so once it became clear that Capsicum was going to stay in FreeBSD for good I think they took the opinion that a clearer data structure was preferable to a small per-FD table occupancy overhead.


You really don't need this, as your collaborators controller only needs the index, create and destroy actions.


Yes, exactly so -- I'll reword.


This is somewhat historical, as an attempt to catch any Capsicum applications from FreeBSD 9.x that weren't correctly updated for FreeBSD 10.x.

In 9.x the rights were just a u64, so apps would do things like CAP_LOOKUP|CAP_WRITE|CAP_READ etc.   When the number of possible rights got too big for a u64, bitwise combination of arbitrary rights isn't possible, and the 5-bit layout allows the code to detect if an app has erroneously tried to do so -- if more than one bit is set in those 5 bits then someone has done CAP_ARRAY0_THING|CAP_ARRAY1_THING.

I'll update the comment to make that intent clearer.  Also, I can't immediately see any code to police this invariant, so I'll check on that (and add an explicit case to the test suite for it).

(BTW, I inherited this layout from the FreeBSD 10.x implementation, and although I also found the bitmask stuff rather peculiar there didn't seem to be a good enough reason to change it.)


That's something odd at the GitHub side -- the base file and git diff are correct.


You don't need this.


This is currently only ever called for CAP_FCNTL and CAP_IOCTL, so idx will always be 0 or 1.  But that's not very maintainer-friendly, so I'll add a BUG_ON for idx being in range.


Thanks again !!!


The various indentations around here confused me.
Do we want
```
{
    e.UpdateScore..
}
```

or like in `set_dirty` above:
```
    { e.UpdateScore..   }
```


Also we're looking into whether it'd be better to just use vanilla Cordova plugins and, instead, help document and catalog them.


Thanks! I was in rush for the #7000... :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4711](https://teamcity.mesosphere.io/viewLog.html?buildId=73852&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:02
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


ayt ayt ayt g


maybe we should add here that value depends on several factors like load, machine parameters (ram, cores, etc.) and write the default value instead of writing question mark.


super amigo. Alex de chile


OK


not a good idea to name classes for system features
name ui element instead


in Test are do not print to output, and dot no exit.
Please take a look at latest code, I fixed this in recent commits.


So you need:

```
@collaborator = @wiki.collaborators.new(user_id: params[:user_id])
```

I know this is a little weird, but collaborators are not your standard object.


Perhaps I should change it to unique, what is the effect of that?
The point is we want the tie-breaker to be the same to help @TheBlueMatt with relay network issues.


this var overrides


On the basis that OSX will be much more common than Windows amongst Docker users, our plan for this guide is that for each command we either make it work on both Linux and OSX or provide OSX-specific alternatives where that is not practical.

Windows users will find that they can get working commands by translating the OSX examples into Windows versions, but they'll have to do that throughout to get the examples to work anyway.


I would redirect to wiki_collaborators_path(@wiki).


Changed in next commits.


You actually don't need this.


http://docs.oracle.com/javase/7/docs/api/java/lang/Override.html says it is supported for Java 1.5.


Agreed about the lack of "This is". The start of the next rendering quantum is the same as the definition given above. I could add a clarification to that point. There was a longish dialogue on the issue thread in which it was convincingly argued that we should define this in terms of a quantity updated after every block render, in terms of the endpoint of that block. I'd rather not revisit all of that if it is not a substantive change.

Also, doesn't the very next sentence address your point about the case where t==0?


This could be something like @collaborator = @wiki.collaborators.find(params[:id]). The relationship is collaborators, not collaborator.


This line needs comma at the end (because of newly entry added after it).

I'm sorry for partial reviews, but you are too fast fixing these minor nits :-))


COuld you try adding something else? And commmiting it?


I think there are 3 styles in use:

According to the style document:
```
int function()
{
    return 4;
}
```

This occurs in some existing code:
```
int function() {
    return 4;
}
```

And only for oneliners:
```
int function() { return 4; }
```


We do not have checkstyle_config.xml enforced over ITs ?
If "no" please create an issue.


question mark in this case represents blank value in docs which I want to fill later after we have some experience with picking correct value on production environment


this version also adds as much of the new styling as possible
it also adds a long list of characters in the Unicode code block, organized by category


Very interesting!


Great code!!!


Опционно. Можно было бы писать string("что-то там"). Есть подозрение, что без не явного преобразования работало бы чуть быстрее.


 http://guides.rubyonrails.org/layouts_and_rendering.html#using-partials
Вот так точнее:)


REKT


Commenting??


Incorrect issue referenced, should be #22.


Oops, i forgotted it. Sorry.


Don't be sorry. I have made sloppy job. Thanks, force-push-fixed.


@e-kolpakov @mtyaka The problem here is that we need to wait not just for one, but for multiple elements. I looked into it for a while but couldn't find a way to do it using Selenium features. The current solution is based on [this](http://stackoverflow.com/a/22712023/1199226). Also, turning animations off doesn't solve the problem completely -- it was rare, but the plot tests sometimes failed before we had scrolling functionality.


what happend here? the changelog states that `ic-ajax` was replaced with `ember-ajax` but apparently this got reverted here?!


Brian, I notice in 3.0.6 that mandatory custom fields are mandatory on creation or closing of an issue, but not an update.  Is that intentional?


( ͡° ͜ʖ ͡°)


(in addition we should update the requirements: nobody will be happy with an old buggy Java 1.5)


А может ли быть v пустым? Если да, получим UB. Если нет - может ASSERT?


GO AWAY


There is nothing interesting about thisw


Thanks...at the very least I added the code from your comment to the patch.


Never mind. @barbieri is right. I'm sorry for this one :/ I will drop this commit.

If you check https://github.com/bagder/curl/blob/master/lib/http.c (function http_output_basic()) you can see that Curl is enconding the user and password.


Да, если есть форма, то она должна работать как надо.
Поэтому, чтобы немного уменьшить объем работы, в задаче уточнялось, что интерфейсы создания сущностей не критичны. Но раз уже форма есть, нужно добавить все необходимое.


Ok, make it unique then. Non-unique means that a find may returns an arbitrary match, and you can use ::equal_range() to find begin and end.


out leaks here. we should free it after this line


fair enough


@jeljaik Master merged onto my branch in this commit. You can start using the API.


OK


1. This needs to at least move the comment to stay with the code it goes with.
2. This doesn't include your other fix for when no file name is specified at all.

Do you want to PR this or shall I adapt mine?


Safe to say this didn't work.


I don't believe you.


Why having a function employee_prepare_head
I think new card / tabs with specific employee information  should be in the user_prepare_head (with tabs visible depending on permission and if user is an employee or not)


Why u hav to be mad, is just game


What do you mean?


computer programming class in grade ten, just trying some things.


The line referenced above sets RBENV_VERSION for the current shell, but it's not exported. You would be invoking `rbenv-version-origin` in a subshell, so RBENV_VERSION would not be set.

As I went about verifying this, I realized that since we need to invoke a subshell anyway, we may as well just query for an exported RBENV_VERSION itself, rather than going through the roundabout version-origin. So I verified that `local env_var="$(echo $RBENV_VERSION)"` and it does. But then I realized that rbenv-bundler-ruby-version *itself* is executed in a subshell from the `which` hook. So RBENV_VERSION is only set in the context of rbenv-bundler-ruby-version when it has been exported by the user. So this commit can be reverted. It works just fine. (I tested locally)


The ngCordova is really great, I use it in all my mobile projects, make the plain Cordova plugins in a Promise way, an awesome helper. So keep it :smile: plz


I'm probably missing something here but, where are R.anim.fab_out and R.anim.fab_in ?


same here `out` leaks


Never trust an IDE :( IntelliJ tolds me to set project language level to 1.6 (not 1.7 as I wrote earlier) for override support on interfaces. Neverless ant compiling does not throw any warnings or errors belong to ovveride annotations what the reason is why many of them already in code.

Compiling for 1.5 could updated to 1.7 in next major release indeed. 1.8 or newer is not so common used in many used production environments.


`out` leaks


@e-kolpakov This is just to ensure that scrolling finishes before the test can do something else. If we don't scroll down manually at all, testing the scrolling behavior is useless because the viewport wouldn't have to change when, e.g., navigating to the next step. I'm not sure if there's anything else we can do to ensure that scrolling finishes?


aren't i getting two different random sets of 32 bits now.  Oh I see, if the first is the same the next will be the same?
I calculated it's got about a 50% chance of generating an accidental duplicate coinbase in the next 3 years at 32 bits of randomness if we run it 1000 times a day...    That should be good enough.  I suppose there are other ways for the tests to false positive anyway.


I believe that some doc is needed: doc/config.rst and libraries/config.default.php.


OPTIMIZATION, BITCH!


Good catch, fixed in https://github.com/cppformat/cppformat/commit/57ba9436a0bdea245772f6b25f3871341236232f. Thanks!


Please feel free to adapt mine and add it to the pull request you created, if you don’t mind.


Done.


Was it necessary before this change? I suggest to remove it in a separate patch. The same applies for the 2nd one below.


fixed


what if you have two handlers with the same method name ?


check your field options, maybe you have marked them required on create form (the `Target Forms` options)


Do these ArgumentExceptions that get thrown include a parameter name?  If so, it'd be nice to also verify it's the expected name by using the Assert.Throws overload that takes the expected name as the first argument.


Right now, the `default` branch is meant to build the nightly, and `version` for specific versions. I can't remember the motivation for it… Should probably merge these, so as not to maintain two branches.

(This caused the nightly builds to not update the nightlies.)


Nice work on this! I think we still want this plugin enabled, though, just without a `parent_registry` parameter (or set to None).


No work!


OK


> it is, but not sure if our fixers support that syntax :(

They do. :)


Oh and silly me.  I guess its guaranteed to not repeat within a test (which is all you care about) if we don't reseed insecure_rand within the test.  I'll remove the high order bits for clarity.


I like "Preservation Network Member ID" as the text displayed on the UI, that's more understandable than just member ID. This would be nice to use as the label in the snapshot properties in DurAdmin.


I think it needs to be "Working with MDHisto Workspaces in Python".


I updated them before. We never got around to restoring the nice phpdoc in our own codebase.


We lost it twice it seems!


Die


No.

(1<<30)*insecure_rand() only has 4 possible outcomes:
* 0x00000000
* 0x40000000
* 0x80000000
* 0xC0000000

You need to cast the 1 or the (1<<30) to int64_t before multiplying with insecure_rand. Multiplication of 2 32 bit integers will not give you a 64-bit integer.


Current best practice:

``` ruby
Post.where(
  title: "This is the unique post!",
  body: "This is where I would put the info about this!"
).first_or_create
```


Good point, will update.


Is that change already obsolete?


Note: prev should've brought a 404?


wow...rough day so far?


Hmmm, I'm not totally sure about nested var though, with var at the top. We might be missing a test case there.


@kotfic This should resolve the issue of config.json being overridden.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4712](https://teamcity.mesosphere.io/viewLog.html?buildId=73872&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:10
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


I'm not sure this is doing what you want it to do.. if I read the code right `self.used_periods` is shorter than `self._current_hist_stack` and you're connecting data values with incorrect period values.


I've tested here and its working fine, there's an edge case that is when the user has both security and session token set. This may happen if your aws credentials is created by a script or other automated tool. Maybe it should issue a warning or just add a message saying what is the key it is using.


@e-kolpakov OK, I'll de-abbreviate :)


Fix the issue #8.


I've just looked on the listing from the jboss cli and it seems that the attribute `path` and `relative-to` were already purged. Is it possible?

```
/subsystem=transactions:read-resource()
{
    "outcome" => "success",
    "result" => {
        "default-timeout" => 300,
        "enable-statistics" => false,
        "enable-tsm-status" => false,
        "hornetq-store-enable-async-io" => false,
        "jdbc-action-store-drop-table" => false,
        "jdbc-action-store-table-prefix" => undefined,
        "jdbc-communication-store-drop-table" => false,
        "jdbc-communication-store-table-prefix" => undefined,
        "jdbc-state-store-drop-table" => false,
        "jdbc-state-store-table-prefix" => undefined,
        "jdbc-store-datasource" => undefined,
        "journal-store-enable-async-io" => false,
        "jts" => false,
        "node-identifier" => "1",
        "object-store-path" => "tx-object-store",
        "object-store-relative-to" => "jboss.server.data.dir",
        "process-id-socket-binding" => undefined,
        "process-id-socket-max-ports" => 10,
        "process-id-uuid" => true,
        "recovery-listener" => false,
        "socket-binding" => "txn-recovery-environment",
        "statistics-enabled" => false,
        "status-socket-binding" => "txn-status-manager",
        "use-hornetq-store" => false,
        "use-jdbc-store" => false,
        "use-journal-store" => false,
        "commit-markable-resource" => undefined,
        "log-store" => {"log-store" => undefined}
    }
}
```


I was wrong. It was needed before and afterwards as I get warnings for unchecked assignments.


Upstream is not fixed. That patch is not sufficient though. On a minimal system I needed python-json and python-zlib as well. I'll push a revised patch upstream.


I disagree with this commit. I can understand the possibility that this may be excessive, but for marketing purposes, it's necessary to provide a user the means to unsubscribe and re-subscribe to a list via their account settings. Sure, they can unsubscribe from one of the emails, but say they want to resubscribe at a later time? Their should be an easy, intuitive way to do this, namely, from their account settings. Also, this feature is still listed in the README:

"List synchronization
Automatically syncs Spree's user list with MailChimp. The user can subscribe/unsubscribe via the registration and **account pages**."

I suggest adding this feature back, or implementing some additional logic to only fire it when the subscription status is changed.


Okay, I'll change the comment slightly:

````
// Enable feature manually if it is also enabled in kernel stack.
//#define CONFIG_INCLUDE_VETH
````


I think you'll only ever want to use the singular form in this context.


`boto.log.debug("Using security token found in environment variable %s" % security_token_name.upper())`


Hi @schelhorn,

It won't be a big change-- the plan is to drop doing eXpress and replace it with Sailfish, we'll create the same combined.tpm file just using the Sailfish values instead.


r+


Yes


and here


hello


the Angular team refactor the Platforms out of the Core just now for the beta release :smile:


Merci d'avoir intégré ma proposition ;)


Why aren't the tests in this TestCompare just additional InlineDatas on the previous two TestCompare and TestCompareIndexed methods?  Is it because this test is verifying the results are less than 0?  I'd suggest just having a normalization function like:
```C#
private static int NormalizeCompare(int i)
{
    return
        i == 0 ? 0 :
        i > 0 ? 1 :
        -1;
}
```
which you can then use in your asserts above like:
```C#
Assert.Equal(expected, NormalizeCompare(string.Compare(...)));
```
and then all of the below cases can be turned into InlineDatas above, e.g.
```C#
[InlineData("A", "B", StringComparison.CurrentCulture, -1)]
```


You can read up at `:help E146` where it details that you can use any single-byte non-alphanumeric character.  I tend to avoid ones that have other metacharacter meaning (such as `*` or `^` or `$`) so I usually end up using `@` or `#` or  `%` or `;` (the advantage of `;` is that it doesn't require using <shift>)


Oh you're right.
I was also doubting the correctness.

I'll also add more tests for compound durations.


Same comments as earlier about the parameter name.  This probably comes up elsewhere as well, but I'll stop commenting on it :smile:


@Snake1999 When you call `if("what the brackets".equals(whatTheFunction(whatTheParam())))`. The left always not null.


No more used.


can you rebase this commit in the commit that changed the code? Otherwise, it make git bisect more difficult.


> Like in this previous comment: a591cd2#commitcomment-14320761

Oh right.

---

Okay, please try out #284.


.Sub() here maybe instead of `*-1`


marvelous work of genius


pls use lower to be safe


Does this mean that our existing socket modules won't no longer function?


Nit: just to make it easier to understand, could you please add parens around ```trimChars.Length == 1 && trimChars[0] == ' '```?


You can stop the `ModTImeFrom` being a pointer if you want by using the `.IsZero()` method to see if it set.  That will make the code quite a bit neater


Ditto


merge this error with the one on line 77. This would be `v < 3007`


I would like to change this part, so it doesn't use it's own config file ('jira.cfg') but shared INI file for all pytest plugins [1]. The workflow gets much smoother.

[1] https://pytest.org/latest/customize.html#inifiles


I see where you mean:

![image](https://cloud.githubusercontent.com/assets/11135921/11120578/c1692e9a-891e-11e5-9645-ea9abe028df9.png)

So on update there is not an option to have it required.  Should we consider another option to require it?


Hello,
Is it possible to contact you directly ? I'm looking for help to interrupts in protected and real mode like https://github.com/nickelsworth/swag/blob/master/tsr/0012.pas

Thanks


:-) :+1:


This fixes the latest issue reported in #11


why this change?


OK


forget that comment, it is ok.


I believe they are the same size, since the values that are written to the histogram stack ```spec_octaves``` are in the same conditional statement and should thus be the same size.
```
+            if(len(specs) != 0):
+                spec_center = specs.mean()
+                spec_octaves.append(spec_center)
+                # save the spectral bin
+                used_periods.append(per_center)
```

This is ObsPy, and a plot made from the data derived from ObsPy through our database prototype. They seem to be identical if we disregard the crooked dimensions in our fig!

![Obspy](http://i.imgur.com/VhZpINa.png)
![Orfeus](http://i.imgur.com/AL1eCBB.png)


looks unused


Also looks unused


IM is missing PNG support on OSX, so the plugin will not work ....


use the same description in both back-end.


yes, that checkbox, remove "required field" and you should be done.


I seem to remember reading (I can't quite remember if the code was handling it) some of this {Key+} syntax in the python module... The python module docs said that Key+ is to put the value of the variable "Key" with some type of special treatment applied to it (like uri escaping).
I remember vaguely that URI::Template had the same behaviour putting the plus sign on the other side of the substitution {+Key}.

I'm saying this off the top of my head, just so you know that maybe it's important to mimick the Key+ functionality for some cases


"... Fields with more levels ..." ?

It should be mentioned that CSV does not normalize the data


why it is in the loop? the original call was different:

https://github.com/mellanox-hpc/ompi-release/commit/74e1de7835de0f538c7ebb669c5dc1732c6ec923#diff-6d8f1c96fe3d8534818b3b2224a92889L866


The problem is that `eval` can't handle expressions over a certain size. It's not a problem of serializing the expression; it's a problem of the expression being too large when it gets to the other side and is deserialized and passed to `eval`. The solution here is to stash the form somewhere and pass a reference to it inside the expression that will be passed to eval. This way the expression itself is bounded in size.


can we publish this to npm so we don't run into this helpers issue anymore? i'm only seeing 6.1.4 on npm


And by 'burger' pattern I definitely meant 'cake' pattern...


@matiasgarciaisaia I think the old and first url is not right. sites resource are not inside institutions resources. Did you check they both work? If so, I would suggest to leave only one to work.


Perhaps you can lean on your Rails backend to compile the cable js for you? How were you building the js before this commit?


combine this with the implode below to reduce the amount of times you have to iterate the data


comment me better


From
https://docs.djangoproject.com/en/1.8/howto/initial-data/#automatically-loading-initial-data-fixtures
:

> Automatically loading initial data fixtures¶
> <https://docs.djangoproject.com/en/1.8/howto/initial-data/#automatically-loading-initial-data-fixtures>
>
> Deprecated since version 1.7:If an application uses migrations, there is
> no automatic loading of fixtures. Since migrations will be required for
> applications in Django 1.9, this behavior is considered deprecated. If you
> want to load initial data for an app, consider doing it in a data
> migration
> <https://docs.djangoproject.com/en/1.8/topics/migrations/#data-migrations>
> .
>

On Thu, Nov 12, 2015 at 6:27 AM, Martin Mahner <notifications@github.com>
wrote:

> Migrations are the wrong place to load an initial set of data. You can
> change data here during migrations, e.g. to update the format or move it.
> But it's quite common that you squash, or even skip migrations.
>
> Better use fixtures for this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AASHE/iss/commit/7306dd952cf5bbb837c76cc71628ceb56652adc3#commitcomment-14352916>
> .
>

--
Robert Erb
Web Application Developer
Association for the Advancement of Sustainability in Higher Education
www.aashe.org | bob.erb@aashe.org | (888) 347-9997 ext 35

--

Connect with us:   Website <http://www.aashe.org/> | Facebook
<https://www.facebook.com/aasheorg> | Twitter
<https://twitter.com/AASHENews> | LinkedIn
<http://www.linkedin.com/company/aashe>

<http://www.aashe.org>
------------------------------
Email Confidentiality Notice
<http://www.aashe.org/email_confidentiality_notice>


I pushed these changes from the car, which still had Sina's login (I think I've fixed that now). I don't object to deletion.


OK


An example would be very helpful


Read this first.


Shouldn't the @return tag return a type instead of the $this reference?


Bad description. My Bad.

It actually concerns: XWIKI-12485: Improve the watchlist icons.
http://jira.xwiki.org/browse/XWIKI-12485


@e-kolpakov I'm not sure. @mtyaka, can you take this one? I don't really have room in my sprint to look into this more, I'm afraid.


It seems like you didn't readd the call to `animatePoints()`, so those animations wouldn't be triggered.


Good summaries so far. Try to also write something about what *you* think. Do you agree with what is written? Why? Do you disagree? Why? How does what you have read relate to your experiences?


This file hasn't been updated to use vscale, so remote this for now and add it later when it's usable


Sebastian Liamaco - SebastianLimaco


this ist in dem context die function in der Schleife


The sha256 sum for RRO-3.2.2-OSX.pkg should be d7d096f5e258a6b1945235aefcdf9c52b3e63daaa515d39305a864f6a3a7b640.


this is what `record.binary.value` has when we save the string 'a' as a binary record using:

- node v0.12.7

```json
{
    "value": {
        "0": 97,
        "length": 1,
        "parent": {
            "0": 0,
            "1": 0,
            "2": 0,
            "3": 4,
            "4": 252,
            "5": 1,
            "6": 98,
            "7": 0,
            "8": 0,
            "9": 0,
            "10": 1,
            "11": 0,
            "12": 0,
            "13": 0,
            "14": 1,
            "15": 97,
            "16": 0,
            "length": 17
        }
    }
}
```

- node v4.2.2

```json
{
    "value": {
        "0": 97
    }
}
```


@davidegrohmann Don't you need to decrement active TXs here?


@friendimaginary Please open an issue. Do not comment on commits.


I can't find any references to test-defines.v, so I think you can remove this one


:-1: maintaining long-lived branches that are never meant to be merged


ahh.. now I remember. This is used for mor1kx-monitor, but we shouldn't include mor1kx-monitor, so this file can be removed


@claudia-lat  Code is in your own repo! :dancers:


I'll give it a try! Thanks.


Alright :monkey_face:


I do want it to be required on update.


Rename this to vscale_tb, or wb_riscscale_tb or something like that. Early versions of orpsoc required testbenches to be called orpsoc_tb, but this no longer the case


Please elaborate a bit more in your summaries. Don't just list the challenges, describe them briefly as well. Also, what do *you* think and how does what you have read relate to your experiences?


Is there a JTAG interface for vscale? If not, this can be removed


Looking good! Great work!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4713](https://teamcity.mesosphere.io/viewLog.html?buildId=73883&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:04
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


А для подключение самого бустрепа лучше всего использовать gem bootstrap-sass.


Limaco Sebastian - SebastianLimaco


Yes in fact in makes much more sense to return a value...
compileExpr return a Rep[T]
and compile returns a Rep[Table[Record]] make more sense

Thanks,


Sadly no.


It would be nice to rename the masters to something else, but not strictly necessary


This means that our test dummy data (fixtures) will come from our development setup, so we need to turn on the API MOCKS when running tests, since it looks for this hard-code data.


Thanks!


Make sure to remove this log function


Well spotted. I think that was left over from an earlier version of the commit. I'll remove that paragraph from the commit message.


@claudia-lat This is the source file for the xsens data capture. It is a modified version of the xsens tutorial.


I would prefer that new systems use wb_bootrom instead of hardcoding instructions in the verilog files


@matt-manywho this one as well


i completely agree John, and in fact that line comes from the PR and in my commit i'm removing it ;)


what is the use case for this code, how & how it's used


Rename to vscale_generic_top or something like that


I don't think this is a bug - draft is not a topic or post. And it's connected to the branch and therefore you can't assign permissions to it.


Interesting thanks! I wouldn't do it anyway. Here we dropped the migrations once, and on the hub already twice. They don't need to loaded automatically either, it could be a one-time manual fixture load upon setup. You do this with the actual ISS data too, or how is this data come in?


Hi `"Joo Wan Ro"`!


Remove the tap and debug interface if they are not used


Possibly. I tried checking out the function, but it leads to some C code and I gave up looking. :)


Что еще:
1. Для фиксации версии ruby нужен файл .ruby-version.
2. [rubocop](https://github.com/bbatsov/rubocop).
3. Тестовое окружение, спеки


See if this can be replaced with wb_bootrom instead


Any reason these aren't defined as defaults in the `__init__()` parameters?


IMHO $resources should get wrapped in a separate angular service for reusability & testability reasons.


I didn't know why png was needed, now I know (and I can comment on it!)

do you use 8bit or 16bit PNG? (povray can do 16 bits png)

fixed in https://github.com/MrKepzie/Natron/commit/8f859220df9800f2601a5a6ccf5017391e9fdda1


Hm, so should it use `copy.copy()` instead? I think that should work fine...


No, that's done by `transactionFinished`, which is gonna be called when we terminate transactions as well.


an fault-tolerance => a fault-tolerant


What is the `CSSTableGenerator` use case?


Was this file regenerated by wb_intercon_gen? It looked like the names in wb_intercon.conf were not updated


consume => consumes


same as wb_intercon.v


Nope...that's a better fix.


from N => from multiple


Najs :)


So the two methods you wrote here are actually changing the title permanently to SPAM. I don't think that was quite the intent, but I suppose it's ok. You probably don't need this one, though, and can just use the other.

Personally, I would make this an instance method which checks the id of the current post and displays either the title or the word SPAM depending on if the id % 5 == 0. I can show you how I would do it on our call today.


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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4714](https://teamcity.mesosphere.io/viewLog.html?buildId=73884&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:15:52
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Do you mean that the flow should set the following boolean values on the calculator?:

* `paid_at_least_8_weeks_of_earnings`
* `paid_less_than_8_weeks_of_earnings`
* `fell_sick_before_payday`

And then these boolean values would be made available via query methods?

Either way, I think I'm happy to merge this as it stands and address this question separately.


np


delete junk


The true mark of sanity :)


aa


Move vlog_tb_utils and wb_bfm to the depend section in [icarus]
Use wb_intercon-1.0 instead of wb-intercon
Remove verilator_tb_utils until there is a proper verilator testbench


Seems like it would be best to use [urljoin](https://docs.python.org/2/library/urlparse.html#urlparse.urljoin).


Remove this section until there is a proper verilator testbench


No, I modified the mor1kx-generic one manually. Should it be generated by wb_intercon_gen?


I'd cast to boolean, also in analogue to setDefaultFilter()


Class AbstractPlugin is missing a docblock.


emitting => emitted


the from -> from the


TeamCity TripleA :: UnitTest [Build 60](http://188.177.191.210/viewLog.html?buildId=122&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 60](http://188.177.191.210/viewLog.html?buildId=122&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Compilation error: Software\TripleA\CashRegister\CashRegister.csproj (new); number of tests is too small: 0 (new) Build time: 00:00:04


ummm none of this is vanilla


nie rób else, jak jest return :)


Hi, can I do the hackathon from here? I cannot come down there at the moment, because I have to be on a call and it was delayed


Checkout that commit and apply this commit to device_nevisp.mk. Or only delete this line;
PRODUCT_PROPERTY_OVERRIDES += \


what's the task to be done?


And, in fact, that was there once but it somehow got lost https://github.com/pocoproject/poco/commit/1d8e75687e4ce644a27205f4b01a2e3be2e8e1fc#diff-cef3ab153a5417905a615b634c62f6a3


You should be able to use luaL_tolstring to coerce any value type into a string here.


Perhaps we could avoid the trailing underscore by being more explicit and using REST API terminology, e.g.
```
def _get_resource(self, res_type, res_id, **kwargs):
```


urljoin has *really* weird behavior when trying to combine a base url with a path, and another path.  i'll take another look.


:green_heart: Excellent work with the correction. It might not seem like much but it provides a lot of clarity.


my instance runs without "management", "deployer", "kar" and "instance". Is minimal intended to show the smallest possible, or any reason for using those additional features? my instance clearly works without them :)


Yes the first function create a list of maps. The second one filter that list on provider.


Small nit: seems you have a typo in this commit message `syncronize` -> `synchronize`


Also manually touching the parent folder might help with any potential etag propagation issues


Thanks for the link, it makes sense now.


idempotentStorager => idempotentStorageManager


Yeah, is annoying when instead of fixing a typo you commit the same typo twice...
@dimkr, what's happening with libre?


Issue #1.


ditto


Since etag propagation seems the cause of the original re-upload issue from what I understood


Is this pluralize? If I have a classname like Users, it won't be /userss ?


@joao-manywho We should be using Array.isArray for these checks


if it has seen .... before a failure => before and after a failure.


raz: nie wiesz czy a[length] istnieje,
dwa: wg interferjsu chyba wszystkie sloty po skopiowanych powinny być nullami


yep. I just choosed that one, because I don't want to fetch the parent too ;)


It default to 8bit, will force 16bit.


Great, thanks!


`scaffold` -> `common`


:+1: thanks!


I would define a constant for that numeric value


@joao-manywho I think this should still be in an array


Is it possible to have an event on update? (PUT/PATCH)


:yellow_heart: We should avoid leaving commented-out code.


of kafka => of a Kafka  [also, make sure Kafka is capitalized everywhere
throughout the document]
in Apex => in an Apex


Issue #8.


@itsjeyd Ok, I'll check if I can turn off jquery animations. Thanks for your help!


for each of the kafka partition => for each of the Kafka partitions


możesz zacząć pomijać słówko "should", lpeiej napisać arrayWillBeCopiedTo


:green_heart: I see what you meant about index access affecting the object. Good work here.


Would remove `ng-click`, add `type="submit"` and use `ng-submit` for submitting the form via keyboard


no need for `as e`?


`$file->getParent()`
 :)


Will do.


Willdo.


:green_heart: Great work getting rid of this. The less code we have to manage, the less likely we run into bugs.


Working on a fix.


space in arguments


:question: I don't know if this was uncommented later on.


That helped me a lot, thanks!


primer test


:yellow_heart: This is just a minor nitpick but I think it should be *return all salons*.


Argh!!!! Need to fix that indeed!


`ember-ajax` will ship with `ember-cli 2.0`, I'm unable to find the source on that but I read this somewhere :)


Hmmm - what does 'directly follows from' mean ?

Edit aims to fullfill [AI from editors call](http://wiki.geneontology.org/index.php/Ontology_meeting_2015-07-23#Implement_MF_patterns_using_directly_regulation_relations.3F):

 "AI: Make 'directly positively regulates' as parent term for directly activiates (MF to MF only) ditto inhibits. :CHRIS:
 AI: Work on reconciled definition for direct regulation.
 Can we abstract the definition sufficiently for RO (not referring directly to MF)?
 Yes.  Chris will add bit to RO def about 1 agent per process and 2 agents interacting."

How about:

p directly negatively regulates q if and only if p and q are processes, and p negatively regulates q via a direct physical interaction between an agent of p and an agent of q.


Yeah I forgot to apply your initial comment. Will fix.


`min="0"` and `required`


issue #61


Some whitespace issues in this that I'll get fixed up for you, but nice work, you're getting the hang of it ;)

-Brad


`required`


najwyższa pora zacząć używać asercji festowych, dodaj bibliotekę do projektu (pom.xml) i zacznij używać assertThat (głównie tu chodzi o porównywanie całych zbiorów)


I don't think it's necessary to retain the rolled up numbers for all these read/write sets. Downstream systems will make it easy to aggregate. To make the data structures clear I think I would name the metrics `neo4j.transaction.{read,write}.{active,committed,rollbacks,terminated}`.


Any reason we're prefixing `logger` here and not in other modules?


Missing OOM check... Also, 41? Please just use memdup() and specify data1 and len as arguments, so that you allocate a precise copy..


It's a transitive dependency brought by `restx-servlet` so I don't think we need it here.


`rolledback` for consistency with the others?


:question: Do we have any tests for the `Role` model and corresponding routes?


Old & inaccurate copy & paste comment on this method.


Yes, we should probably add update here too.


And you're right it should be declared as `provided` in `restx-servlet`


that makes sense, I will open an issue


there was never any expressed need for it, it can be added if needed.


Yeah, this is what I was worried about.

```
In [1]: from urlparse import urljoin

In [2]: urljoin('http://foo.com/bar/', '/baz/junk/')
Out[2]: 'http://foo.com/baz/junk/'

In [3]: urljoin('http://foo.com/bar', 'baz/junk/')
Out[3]: 'http://foo.com/baz/junk/'

In [4]: urljoin('http://foo.com/bar/', 'baz/junk/')
Out[4]: 'http://foo.com/bar/baz/junk/'
```

I _think_ if I ensure a trailing `/` on the base, and strip the leading `/` on the path, it'll do what we want.


suggest rewriting:
The AppMaster creates `K = min(initialPartitionCount, N)` Kafka input operator instances where N is the number of Kafka topic partitions. If K is less than N, the remaining topic partitions are assigned to the K operator instances in round-robin fashion.


REQUEST_METHOD and command only, please.  So no logged in self_char_id should be a "redirect-with-error" condition in the controller method.


Please compare the lengths of the field with what we expect and then pass that length to memcmp().


Agreed


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


Difficult to see what actually changed here due to whitespace issues


cc @mjackson


Hmmm. I think you're right. Wondering why I updated here in the first place...


* Check if the form is in a valid state is missing (form.$valid)
* It is possible to submit the form multiple times (when a save is in progress)


I changed it to AsBits.


Oh, OK, I see what you were doing there.. so it's OK for you if in the binning setup I only keep the bin grid that is actually covered by the psd, I guess?


We really should also return an error for UNIT_ERROR and UNIT_MASKED. In fact, I think we should simply invert the check to "if (u->load_state != UNIT_LOADED)"...


Is there a difference/advantage/convention for comparatives? NULL == list vs. list == NULL


Please change this to CharField.


tested and working


If Babun is only available on Windows, I don't think you need this check.


This kind of changes how that setting was supposed to override the `lis_course_offering_sourcedid` passed in; my assumption was that setting was there so you could test different course info on the course, for convenience. With this new approach, what is the assumption for how to test this?


suggest highlighting literal names like msgProcessedPS, bytesPS, etc. For example: `msgProcessedPS`


this is totally acceptable


In RFC6962, add-chain and add-pre-chain returned the components of an SCT broken down into separate JSON fields.  For 6962-bis, we decided to switch to returning a whole SCT as 1 JSON field.  ISTM that, for consistency, the APIs should return a whole STH as 1 JSON field too.
STH and SCT signatures are calculated over a TLS-encoded structure.  So yes, the client will have to do some TLS re-encoding before they can verify an STH/SCT signature.
I think that returning a whole STH or SCT in 1 JSON field means that we can change the innards of these structures in future without necessarily having to bump the version numbers of the APIs.


nope


crossed this upper bound => exceeds this bound


It's not actually an issue if you do both, but it just saves you time.


Change this to the corresponding dictionary keys in ``ecommerce/settings/local.py``.


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


You can make the loop do this for you:
```
for (size_t txinIndex = 0; txinIndex < tx.vin.size(); txinIndex++) {
    const CTxIn& txin = tx.vin[txinIndex];
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4715](https://teamcity.mesosphere.io/viewLog.html?buildId=73893&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:19:56
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Das plonkt, will er mit index -1 zuegriffe will


Ugh, no difference.  I used to have the run-time variable on the lhs, but the current company coding standard specifies the run-time variable is on the rhs.  I guess I've been infected.


OK


:yellow_heart: We'll need to consolidate the different login processes into one. Users and Salon owners should be able to login via the same process. What would differentiate them would be their access levels.


delete


I forgot to add the changes to Main.f90 to call both Global_Setup_fine and Global_Setup. Anyway if you do this, a weird thing happens. The xx array gets around to Output.f90, but not the x array. In Geometry_fine.f90, the z array is loaded correctly, but the x and y arrays are not.


Note, this actually can obscure some issues so that instead of providing a more useful error message; it simply returns to the main page. On the other hand, there should not be many cases where we throw an exception anyway, and if the user can provide us with their exact process, we should be able to figure out what went wrong.


@reinzor  Why is this `chull.points.length - 1`?


@klappradla Hier den defaulttext nehmen


Translation quality is good enough for development and testing purposes. Still, needs a refactor for reaching a **production ready** status.


shouldn't be part of the changelog then: https://github.com/ember-cli/ember-cli/commit/91bf94a36fd059cf51fb68de59fc6023a35a9e3d#diff-4ac32a78649ca5bdd8e0ba38b7006a1eR25


Для какой версии Ubuntu работает эта инструкция по установки? Я не могу найти `ttf-mscorefonts-installer` для 14.04.


"This is an optional parameter that is useful when the application restarts or started at specific offsets (offsets are explained below)"


Hier bitte auch den defaultText wenn keine Inhalte da sind nachziehen


@hiberno That makes a lot of sense.


No big deal but in theory, could you not have added this include just to the .cpp?

I was thinking if only the .cpp needs it but not the .h, it could compile cleaner/faster if only the .cpp included it and not the .h / all the units including the .h.


sorry, I though it;s i different commit. Still I don't get the `_handle_rc` fnc


Move the declaration into the loop, so it doesn't need to copy (you were right, this line does copy):
```
const CCoins& coins = prevCoins[txinIndex];
```
(The [] is more efficient than using .at, as it doesn't do bounds checking)


I prefer not to, as long as its tests run and are not a PITA to maintain. Despite being released, recent issues show that development is still a bit far from over, and abandoned code may solve new problems. A recent example is IncludesFinderWithRegExp, abandoned months ago, now revamped


:yellow_heart: A thing to note: when you find there are too many `if`s in a code base, the code could be improved with the use of the *Strategy* design pattern.


Done


rezzpartition ?


TeamCity pull requests :: netty [Build 645](https://garage.netty.io/teamcity/viewLog.html?buildId=3230&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5258, ignored: 21 Build time: 00:27:02


:laughing:


Also, my IDE is set up for other company coding standard requirements (that I personally find distasteful, but have to live).

- whitespace between the parens, like: node_create( data )
- lack of whitespace between the keyword and paren, like: for( i = 0; i < ...


emit => emits


This is because fgets includes the '\n' in your string right?


That -MDD is added by some platform recipe that's out of arduino-builder scope. IIRC, it's removed because it makes gcc create files that are not useful for arduino-builder and that clutter the hard drive


You may wish to re-comment this debug output before merging (or not).


Yes, that would not make a difference for us, as long as we always have the same periods with values so we can easily group and add them individually. Thanks!


:ok: :1234:


:question: What is `$rootScope` being used for?


store these => stores them


What value do you foresee using for non-release builds, @will-moore? If you're not going to change this between releases, then this seems excessively exact versus something like figure-1.2


done!

On Thu, Nov 12, 2015 at 9:59 AM Vincent Voyer <notifications@github.com>
wrote:

> cc @mjackson <https://github.com/mjackson>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mjackson/expect/commit/1b446c58fb27ae14bec314fe38e47f0ebb0b8764#commitcomment-14357855>
> .
>


If you read the placeholders names, you'll guess they come from unofficial cores. They are here because that allowed those cores to work flawlessly (warning apart) with arduino-builder


With this ViewSet it is pretty obvious that a record is stored as a json field in the storage backend. I think you mentioned it somewhere in the doc, maybe a good time to remember it?

When we define a Schema we could think there is magic defining the database table, but this is not the case right?


I'm on board with renaming the method, but don't see the reason to try and avoid the trailing underscores.


Exactly right.  "chomp" is a built-in function in perl that does exactly this (because it's such a common thing to do) ... a neat thing about learning different languages is you get to see what kinds of problems they're optimized for solving :)

C can do anything, but it's good to know why a particular tool is used for a particular job ...


Changed. Now `#include <Arduino.h>` is added before calling preprocessor


In that case, can we have a test that sets that configuration setting and ensures that it indeed honors the setting and a content with length one more than that fails?
________________________________
From: John Gardiner Myers [notifications@github.com]
Sent: Wednesday, November 11, 2015 8:10 PM
To: proofpoint/platform
Cc: Parag Shah
Subject: Re: [platform] Fix connection hang when ResponseHandler throws without reading large response. (549bdcd)

This is configurable through the http-client.max-content-length configuration setting for the HttpClient instance. This config value is an absolute limit for JettyHttpClient.executeAsync()--it will throw ResponseTooLargeException if exceeded. For JettyHttpClient.execute() it just buffers that much data before invoking the ResponseHandler and the ResponseHandler can stream the rest. I believe this change only has effect when the response size is larger than the configured setting.

—
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_proofpoint_platform_commit_549bdcdc2c5b63a31a2e3dd2e7235ec608a18cdc-23commitcomment-2D14346109&d=CwMCaQ&c=Vxt5e0Osvvt2gflwSlsJ5DmPGcPvTRKLJyp031rXjhg&r=Zm-paw4L-LBldtJybEWfBLLjvC5axjko2vu-eqkngMI&m=1Bdak2_DdHM3Hd6fRtxPSwHfvAa4RLsGwgC-_dG0sKE&s=7T-m7WnsdwGKU_bQzGZvaKkL0C9uuGxmKP02M-t2DrY&e=>.


Τμήμα κώδικα για τον έλεγχο του 7 segment


array blocking queue => an `ArrayBlockingQueue`


Looks good


So does this Cloudflare.conf file replace the jail.local file, or am I supposed to insert this somewhere in it?


This can probably be calculated outside of the loop?


This commit is being reviewed as a part of review issue #87.


has this been pulled?


This commit is being reviewed as a part of review issue #116.


THX!


rah lalalala


FYI... http://ruby-doc.org/core-2.2.3/Enumerable.html#method-i-each_with_index


Thanks :-) There may be an issue if multiple exploits are exploited where
it would add on " and possible exploit code." multiple times but in my
tests there was only one exploit if it was identified in my test files.

On 12 November 2015 at 14:56, Brad Spengler <notifications@github.com>
wrote:

> Some whitespace issues in this that I'll get fixed up for you, but nice
> work, you're getting the hang of it ;)
>
> -Brad
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kevross33/community-modified/commit/e6b2c1e787b5b03c29f8de5affaa567d7fca2876#commitcomment-14357732>
> .
>


Split this stuff into various `mode_*.go` files and `common.go`...


Check the dependencies in the build.xml file. You shouldn't need to explicitly hit the init target ;-)


1: 文字列はマクロにしないほうがいいかも (保守性の問題)
できれば文字列定数はconst charの文字列に対するポインタなんかを使って定数を書いたほうがいい

2: 特定のfdからreadしてもう一つのfdにwriteするという処理、システムコールの発行に失敗した時の処理はまとめたほうがよいかもしれない
もっと言うならエラーが発生した時の文字列はclose(2)に失敗した時とwrite(2)に失敗してる時の2パターンがあるから"error occured"とだけ言うのではなくてエラーの原因を別々に表示したほうがいいのかも。でも、まあこれは別にいいかな。


I hate being so CMake illiterate ...
If I read this right, I should still be able to manually add and specify different paths for MySQL_LIBRARY_RELEASE and MySQL_LIBRARY_DEBUG,, and this do what I mean (i.e. have later build stages pick up on the difference)?


why?

On 12/11/2015 16:13, Manfred Touron wrote:
>
> :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jeannedhack/programmingChallenges/commit/a48c31e3f7910186ce23a55f2a49b236507ff8a2#commitcomment-14358302>.
>


Thank you for the contribution.  Would you mind adjusting the tabbing to be 4 spaces for consistency across the codebase?  I'll go ahead and merge once that's updated.

Thanks,
Jamon


fixed


Nice work, good to have a strong base :+1:


Looks like key errors are not being handled; we're assuming `_get_objects_by_url()` returns the expected input, but if not, this error raises up through the stack I think...?


You forgot to change the names in the cpp file


I think having a page for each problem so interactions/tweets/etc. can be more organized is a great idea.

j
Julie KP Dunbar, PE
Dunbar Transportation Consulting
24 Laurel Wood Dr.
Bloomington, IL 61704
Office Phone/fax: (309) 661-1767
Mobile: (309) 287-7836
Email: julie@dunbartransportation.com<mailto:julie@dunbartransportation.com>

CONFIDENTIALITY NOTICE-
This message is intended only for the use of the individual or entity to which it is addressed and may contain information that is privileged, confidential and exempt from disclosure under applicable law. If the reader of this message is not the intended recipient, or the employee or agent responsible for delivering the message to the intended recipient, you are hereby notified that any dissemination, distribution or copying of this communication is strictly prohibited. If you have received this message in error, please delete this message from all computers and notify us immediately by return e-mail and/or phone 309-661-1767.

From: Andrew Rohne <arohne@oki.org<mailto:arohne@oki.org>>
Date: Thursday, November 12, 2015 at 6:47 AM
To: okiandrew/analyze-this <reply@reply.github.com<mailto:reply@reply.github.com>>, okiandrew/analyze-this <analyze-this@noreply.github.com<mailto:analyze-this@noreply.github.com>>
Cc: Elizabeth Sall <easall@gmail.com<mailto:easall@gmail.com>>, Julie Dunbar <julie@dunbartransportation.com<mailto:julie@dunbartransportation.com>>, "Veronica O. Davis" <vdavis@nspiregreen.com<mailto:vdavis@nspiregreen.com>>
Subject: RE: [analyze-this] Updated index, added an about page (7c54a63)

Copying Julie and Veronica that were left off of this (not sure they knew to look on Github)

Are you saying that we would have something like a page for each item instead of having them on the main page?  I can then set the pages (or maybe manually upload the pages) on those dates and post tweets?

E.g. “Planning Mystery 1: Corporate Strategies to Reduce Single Occupancy Vehicle Trips #TRBAnalyzeThis http://okiandrew.github.io/analyze-this/” (obviously with a correct link, I just put something there to make sure it was < 140 characters after the URL shortening)

We then have individual pages for each of the mysteries, thus allowing for the comments to be a little more organized?

I like.

Andrew

From: Elizabeth Sall [mailto:notifications@github.com]
Sent: Wednesday, November 11, 2015 1:03 PM
To: okiandrew/analyze-this <analyze-this@noreply.github.com<mailto:analyze-this@noreply.github.com>>
Cc: Andrew Rohne <arohne@oki.org<mailto:arohne@oki.org>>
Subject: Re: [analyze-this] Updated index, added an about page (7c54a63)

Unless you all think it is a dumb idea, let's do the week by week roll out of the planning mysteries starting December 7th, the 14th, 21st.... That way we can have an excuse to tweet about them and then discuss them one by one on TMIP, etc.

That means you can put that schedule and some text about that intent and hold off on actually listing them for now. (unless you all think it is dumb)

—
Reply to this email directly or view it on GitHub<https://github.com/okiandrew/analyze-this/commit/7c54a63b8f76d1cdb05a8d94a116fdd1fe8987b2#commitcomment-14335423>.


:yellow_heart: We'll need to set up a build process to reduce the number of HTTP requests the landing page makes on first load.


Was sticking with the apparent local convention of prefixing module-level globals.  Happy to stop doing it.


pls fix whitespace "foo ? a : b"


This is wrong, `%(mode)s` is a placeholder for the value of `context.workmode`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4717](https://teamcity.mesosphere.io/viewLog.html?buildId=73910&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:20:39
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


The headers have been merged from another branch. I'll be deleting this.


I think the following approach may work.

In `potion_cl_eval` extract the PNProto from the current closure and pass it to something like the following like this `code = potion_source_compileExtend(P, cl, code, (PN)proto_extracted, PN_NIL);`

This is a modified version of `potion_source_compile` that extends the parent environment.

``` c
///\memberof PNSource
/// "compileExtend" method for PNSource, an AST fragment. Typically to add a function definition,
/// but also objects, blocks, ... (Almost everything is a PNClosure)
///\param source  PNProto or PN_NIL of the current closure definition stored in f->source of each block
///\param sig     PNSource signature tree or PN_NIL or PNTuple, parsed via yy_sig()
///\return PNProto a closure
PN potion_source_compileExtend(Potion *P, PN cl, PN self, PN source, PN sig) {
  vPN(Proto) f;
  vPN(Source) t = (struct PNSource *)self;
  vPN(Proto) parent = (struct PNProto*) source;

  switch (t->part) {
    case AST_CODE:
    case AST_BLOCK: break;
    default: return PN_NIL; // TODO: error
  }
  f = PN_ALLOC(PN_TPROTO, struct PNProto);
  f->source = source;
  f->stack = PN_NUM(1);
  f->protos = parent->protos;
  f->paths = parent->paths;
  f->locals = parent->locals;
  f->upvals = parent->upvals;
  f->values = parent->values;
  f->debugs = parent->debugs;
  f->tree = self;
  DBG_c("-- compile --\n");
  f->sig = (sig == PN_NIL ? PN_TUP0() : potion_sig_compile(P, f, sig));
  f->asmb = (PN)potion_asm_new(P);
  f->name = PN_NIL;

  potion_source_asmb(P, f, NULL, 0, t, 0);
  PN_ASM1(OP_RETURN, 0);

  f->localsize = parent->localsize = PN_TUPLE_LEN(f->locals);
  f->upvalsize = parent->upvalsize = PN_TUPLE_LEN(f->upvals);
  f->pathsize = parent->pathsize = PN_TUPLE_LEN(f->paths);
  return (PN)f;
}
```

Note that I don't currently have access to a c compiler, so the above may not compile.

Then we can just use `potion_run` instead of creating `potion_cl_run`

I'm not 100% sure this approach will work, but it avoids modifying the existing closure, only accessing its data (and adding any new variables needed).

BTW, it doesn't look like the PNSource and PNProto stuff is GC safe. That could be a problem if the GC kicks in during eval.


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


This check is done twice, maybe refactor into sub-helper?


That error message comes only when using pedantic errors and is expected. http://stackoverflow.com/questions/27817081/no-compiler-error-on-using-garbage-characters-in-include-statement
That regex finds the header name as expected.
If you have a test case where it doesn't work as expected, please share or open an issue :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4719](https://teamcity.mesosphere.io/viewLog.html?buildId=73913&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:05
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


a bit wrong reason :) SDK expalined me about optional arguments, I didn't noticed that they was optional.


this should probably be conditional, as a racing thread  may have set it to something > last_sent_epoch


This is actually also included in #2517, which un-pins the Node version as well. Have you had a chance to take a look?


hey bro you have done wrong :/


In my view should look like:

```
/vendor
/node_modules
.env
```
I would leave the node_modules precisely because some projects in laravel have this dependency, the example of the spark.


Replaced NULL by 0 on last argument. Is it OK?


I added the GitHub Introduction file for Anthony Carlson


The additional FILEPATHPTR defines here were apparently included with the v2.0 release, must have forgot to git-commit this or something.


ok


I might have a go at implementing this over the weekend, if you don't get round to it first!


ok fixed ty


gobla.h ==================>>>

#ifndef _LINUX_GOBLA_H
#define _LINUX_GOBLA_H

extern int gobla_switch;
extern bool gobla_scr_suspended;

void gobla_setdev(struct input_dev *);

#endif


Thanks!


neki komentar...


For what? there is no use in the view.


When could this occur?


Is this the correct comparison? The function is defined to return a boolean, so I guess that being "not null" means testing for the value `true`.


это у нас ссылки, а не спаны


I would reword this a bit...

> Generates an API definition for the given LoopBack application. Supports **YAML** and **JSON**.


треугольнички где?)


extends from => extends


yeah, it works with `copy.copy()`.


@joeha480 would rather not ignore these "white space" differences in his tests because this way he can better detect weird behavior in Dotify and fix if needed. So I'll change my tests.


space in both sides of outdir, and same in the lines below.


Not sure what this sentence is trying to say, suggest rewriting to clarify.


delete junk


zut mon morceau de code pour raffraichir la liste des ia marche pas...
sur l'interface web


:yellow_heart: We'll need to refactor this later once the single login process has been implemented.


Which command? This section is usually reserved for explaining to developers: how to run the automated testing for this module. Is this text copied from somewhere?


Can we do away with this `lorem ipsum`?


We should also support printing to `stdout`.


consume => consumes
override => overridden


This should never happen. You should assert.


Ja, das sieht mir auch komisch aus, guter Fund!


The MAKE_BIFF line causes a parse error with weidu 238 and also with weidu 238.01 beta.


Add note when to uncomment the block.


Ich hatte das diffuse Gefuehl, dass die Funktion komplex genug ist, um einen eigenen Test zu kriegen. War wohl richtig.


Honestly, I'd rather get rid of this in its entirety...I'm really not a fan of "do this in case we're testing" code.  If we're testing, we just need to pass the value in via POST.


**nit**: please add a gulpfile to this module that uses our standard `.jshintrc` to lint the code. Here is an example: https://github.com/strongloop/loopback/blob/master/.jshintrc

This isn't a blocker for merging the PR though.


This commit erases all the work we've been doing on this branch. I think this is not intended?


We can just extract the protocol first and loop around the all the SocketPort structure and apply there.
Since we do have already a storage for protocol  I think we just need to do a LIST_FOREACH and apply here


might as well do `/json/i`


Sure! Will fix in few hors when I get to the laptop:)
On Thu, Nov 12, 2015 at 17:17 jamon <notifications@github.com> wrote:

> Thank you for the contribution. Would you mind adjusting the tabbing to be
> 4 spaces for consistency across the codebase? I'll go ahead and merge once
> that's updated.
>
> Thanks,
> Jamon
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wtfil/playmusic/commit/f1cb51210c284c4dd744d865d88cb96f9a87f4a9#commitcomment-14358415>
> .
>


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


that way we need to have 2 exceptions - one for invalid configuration, one for invalid fixer itself, both with same parent


Though this is not specifically needed, we MAY implement additional formats.
Would you mind adding a test for this?
See http://json-schema.org/latest/json-schema-validation.html#anchor104


If we can do it with just attribute, sure. I assumed that the attribute approach requires applying it to each individual field, in which case I'd prefer just saying "hey look, this file doesn't follow that rule".


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 4721](https://teamcity.mesosphere.io/viewLog.html?buildId=73915&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:05
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


thx


For some reason I thought the standard `/` substitute was `_`


Is this file FILE already exists?


You'll need a separate wrapper for use in the wallet code.


Angular should be able to set these values by itself. See [here](https://code.angularjs.org/1.2.0/docs/api/ng.$http#cross-site-request-forgery-xsrf-protection) for more details. There shouldn't be a need to add a request interceptor, not pull in a new query library.


Could be?
```php
public function __toString()
{
     return $this->toHtml();
}
```


Hey Sergio,
This was added as part of LPS-41902, it was specifically a check for guest users, that had an edit link - and were able to view the content - so we put an isScheduled check to protect against that.

With the new logic, if the user is signed in, it will fail this check, and still go to the "hasPermission" check below, and if the user does not have the appropriate permissions - they will not be able to view the content.

The issue is that on a Remote Live site, the stagingPermissionUtil will specifically return false when we try to check for UPDATE permission, for any user on the live site - not allowing even admin's to view the scheduled content.


класс `a`?


Why `console.error`?

I would just remove this entirely. I'd rather have the tool be silent if everything went OK.


why dont you move your example data to the json folder rather than creating that file in the unit folder? alternate option is to do what @sigmavirus24 did in some of the tests under unit folder where he added the example data in the same file as the unit test


i'd split the validIndex check into a separate statement, since this seems to be the convention


j'allais push cette correction ;)


same here with the null check


странное значение)


Good idea. Filed #50


same here


this is shit


вообще дикие цифры)


fuck


dfg


well does


Hmmm... this `getApiDef` function is outputting a file?

It should just return the contents of the file. Another function should write the file.


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


Creds to LuckyDjuke for this commit :dancer: