I just manually merged. Here was the plip file where the tests were running. https://github.com/plone/buildout.coredev/blob/5.0/plips/plip-extensible-fc.cfg

This isn't a new feature. This was a blocker for plone 5 from a regression that people are not able to provide their own folder contents actions.


Dan,
at this point, PMA_SQP_getAliasesFromQuery() became dead code so it would be better to get rid of it.


I do not understand your questions completely, but yes you should create a new stdatomic.h file. And then simply put it into one of the folders: /usr/local/include or /usr/include


лучше создать абстракцию для адаптеров. Отдельная функция-кноструктор Adapter с методом sendText в ней


第一次使用这个东西，希望自己成长的越来越快


revert


Длина массива вычисляется каждый раз в условии.
Wrong: for(var i = 0; i < args.length; i++)
Correct: for(var i = 0, length = args.length; i < length; i++)


use toArray without parameters.
check first if size is 0


Here you say "*minimum* PAOS version", but the actual code has an exact comparison? I assume that there are no newer versions available, but I still suggest dropping the "minimum" part. That way the documentation doesn't have to be updated if a new version shows up.


адаптеры не подтянуться, если я буду подключать модуль через глобальную переменную.


некорректно используешь require.js; Зависимости должны указываться в массиве первым параметром функции define: define(['path1', 'path2'], function(adapter1, adapter2){}

для реализации подключения твоего модуля как CommonJS или глобальной переменной, погугли про umd (universal module definition)


Could you use Ansible handlers instead of shell for this?

https://docs.ansible.com/playbooks_intro.html


адаптеры принадлежат логгеру, поэтому лучше их занести в прототип


I hope this version solves the test issues. mg doesn't produce traps anymore when stdin or stdout are not tty's


I've tried that one before, and it fails in some cases, so I favored the predictable approach here. I didn't spend more than an hour on it or so back then, so it is quite possibly solvable with some effort.


не вижу реализации WebApi endpoint


Is this to get the latest mono nightly build? E.g. could it be mono 4.1.0.78 in one day and mono 4.1.0.89 in the next day?


mh, it works well now.. but when running unrarall as cronjob, it doesn't extract my stuff and doesn't return an error message.. someone know how to fix this?


стоит вынесли сразу готовые logLever в прототип, чтобы не создавать их всё время при создании кадого логгера. А так ты каждый раз создаёшь logLevel и перезатираешь уже созданные методы в прототипе


@k-okada
Thank you, I know it.
But I need to confirm whether jsk_perception nodelet could be made correctly.
I will send this pull request after that.


Длина массива вычисляется каждый раз в условии.
**Wrong**: for(var i = 0; i < args.length; i++)
**Correct**: for(var i = 0, length = args.length; i < length; i++)


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


Thanks!


не вижу реализации заданий: 5,6,7(кастомный метод логирования, а не кастомный уровень),8,9,10,11,12


@simison, you are correct.  I added it when I noticed nav dropdown was not working.  Resolved by https://github.com/Gym/mean/commit/141cde157803b0d0d640bffda259f77376d1d860


Thanks for it. It help me lot


I don't think this cast is required?


It's probably not worth the effort anyway. Most people will follow the keymap.


Hello.

While using OSX I've never encountered such a dialog (Accept Connections), but accepting connections is obviously normal for KeePass/KeePassX with HTTP plugin.

Please ensure you don't have any stale KeePass/KeePassX process.
In case you were previously running KeePass using mono, you might have stale mono processes running.


so a change log entry explaining this would helped to avoid confusion,
right?
Am 07.07.2015 13:44 schrieb "Nathan Van Gheem" <notifications@github.com>:

> I just manually merged. Here was the plip file where the tests were
> running.
> https://github.com/plone/buildout.coredev/blob/5.0/plips/plip-extensible-fc.cfg
>
> This isn't a new feature. This was a blocker for plone 5 from a regression
> that people are not able to provide their own folder contents actions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plone/plone.app.content/commit/336a39447cda97fe78c3cf6f60861ea7631b60f7#commitcomment-12037230>
> .
>


@NobodysNightmare any more thoughts on this?


@jcwilk This is how we reason against `let` - https://github.com/thoughtbot/guides/issues/322#issuecomment-86574235


You're right, votes don't need blockchains, just a list.


I'm going to merge it after #70


มายคาฟ1.0.0


If kcp is installed, then base-devel is already there too (default for all stable KaOS ISO releases).


yes please. I believe I cherry picked and pushed directly already, hope I didn't break anything. Haven't figured out the proper way of doing this, I'm afraid.


An alternate place to put these is wcf.targets at the root.  I already started putting the names of other commonly used folders there.  Not required, just an alternative.


おおなるほど！


Eugene,

Thanks for your fast answer.

I think I'm not explained myself well:
1) The dialog (accept connections) appears any time I run the app. Why OSX doesn't know to keep this setting after the first time I allow the connection such in every other app
2) I never ran Keepass using mono, I even don't have mono. This doen't happend if I install the master banch of keepassx

Any other idea?

Thanks again


I guess we should use `log.error` here since that handles Twisted failure objects nicely?


You skipped here :) https://github.com/enygma/modler/blob/a4863f2906138a0f639b07e9cc19cf31cdb7c8fb/src/Modler/Model.php#L226


foreseesは、うまく日本語にできなさそうなので私もこんなんも考えました。いただきます


空の開業はやめた方がいいです。

クラス名はxxxxxにしてください。


@lime62 Good


Looks all good to me. :+1:


Fixed in v2.1.1


Реализация избыточная. Можно обойтись без пересоздания анонимной функции.


**this** здесь не обязателен, профита от него нет.


Look at the commit message above, which references the underlying ticket: http://bugs.jqueryui.com/ticket/5860


불필요한 주석은 남기지 맙시다


#3 Association de plusieurs contacts avec une compagnie


提交过后再次进行添加


first time


why?


Иногда может пригодиться фильтрация по индексам, было бы здорово передавать в callback ещё и индекс.


You have always spelling mistakes Hahahaha


Sorry, I forgot to invert the value :
```javascript
angle = -(angle % 360);
```

I don't know if angle can take huge values, but if it's the case, the while loop will slow down drastically the process. here is an example : https://jsfiddle.net/dnejhgco/


I hate you....


when I see this maybe it make sense to add `fetch` method to Term that operates on params?


test


> Thanks! About PR:s, I think we can handle this on one branch - unless
> something needs a special review!?

Of course, and you'll notice that most of my commits *are* on that
single branch. It's just that extra bit of changing the logic that I
wanted you to see before merging. That loop might have been on purpose
(e.g. as a preparation for future implementation).


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

VSO State: Done


의도는 이해가 가지만 OnelevelEventListener라는 이름은 이상합니다.

이 리스너가 watch해야 하는 이벤트는 이벤트 리스너가 watch해야 하는 이벤트와는 다르지 않나요?


Thanks for this - just synced the update I made when we changed the random
number on lottery night.

On Mon, Jul 6, 2015 at 3:54 PM, rgknight <notifications@github.com> wrote:

> Can you confirm that this is the seed that you used in the lottery?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rgknight/charter-lottery/commit/a18fcf7ca2f3aeea4c307aabcbde9a22880a5ce0#commitcomment-12024365>
> .
>

--
*Jill Kahane*
*Data Analyst*
UP Education Network
90 Canal Street, Suite 600
Boston, MA 02114
Mobile Phone: (508) 517-9668
upeducationnetwork.org <http://www.unlocking-potential.org/>


yes, that would be nice.


Для реализации рекурсии лучше использовать именованные функциональные выражения (данный кейс не безопасен)


Nice to see you're adding a readme file! Some pointers:

* Common agreement is that the readme.md file is in all caps. (README.md)
* https://help.github.com/articles/markdown-basics  -> learn these, make sure that you include h1,2 etc at least


Okay, message chains, not block chains. Just semantics. The whole point of using chains is to conserve a chronological order, necessary for moderation. If blockchains can't achieve this, how do you solve timing issues?


Currently it only has the date of the time I build it.
Need to improve that so it is updated when a new mono weekly is available.


This line was probably lefr from earlier... taskDao is not used/needed in this function any more...


I'm not sure what the difference is in this context. Aren't both the same thing?


Unfortunately setAttribute gives an error in RStudio.  I am pretty sure there is an easy workaround.  Should have something in a little bit.


To think about this problem i need to break it down into smaller parts to be able to create solutions for each process. For example:
* If an empty string is given, it will output an empty sequence.
*if jobs abc are given, it will output abc in random order.
*if jobs abc are given but b has dependency c, then it will output 3 jibs with b with dependency c.
*if jobs abcdef are given and f comes before c(1st) c comes before b(2nd) a before d(3rd) and b before e (4th) - output with all 6 jobs and dependencies given.
*if abc is given but c comes before c it cannot work and must throw an error as c cannot depend on itself.
*And if abcdef are given but a has no dependency, b has c, c has f, d has a, e has no dependency then f has d, an error must be thrown as jobs can't have circular dependency which I'm assuming is a gap in between dependencies in a sequence.

I'm trying to plan this more logically and work out best way of solving this.
I'm think it will be best to create an array for jobs, an array for dependencies and then an array for output at this moment,


Hi


There is a changelog entry.

But yes, I should have made the PR nonetheless. I apologize. I guess I was just in "get it done" mode--it was my code, my regression, it was tested...

Is it policy, even for small changes, that you need a PR for everything?


>In general, Babel currently uses a message-chain only in places where authors are expected to actively avoid forking of the chain.

Mind explaining this further?


@ophian Sorry, I don't know about that specifically, only that "SET NAMES" works on the installs I use it. Haven't used mysqli_set_charset() yet.


@ophian Good catch! That's a bug :-)


https://gist.github.com/Wonsebolatan/1a36a743d5c2ee472cda

https://github.com/Wonsebolatan/stdatomic/blob/master/stdatomic.h

does this count either of this count as making a stdatomic.h file?


Ah sorry, this is not the right line/function -- I meant the same code in the function _getTasksToRun (line 342) :-\


euh pourquoi vous parler du mot réfrigérant dire qu'il n'existe plus dans la trad actuel.... faut regarder les fichier présent plutot que les vieux commit qui date du 13 avril.
a l'heure actuel c'est
<MustPlaceVentWithFreeSpaces>Les deux parties doivent être exposés à l'air pour qu'elle passe à travers.</MustPlaceVentWithFreeSpaces>


I dunno, should yes really be the default?


Spederiva,

Seems like you have enabled OSX firewall, which is probably off by default.
The following article might be helpful:
http://robcottingham.ca/2008/03/os-x-applications-constantly-asking-permission-to-accept-incoming-connections-heres-a-fix/


Looks good to me.


壁を無視して移動できる敵の実装 #317


Comment should read "Should include FW_DESC_SIZE (116kB)


Comment should read "EEPROM storage area (@116kB)"


It's moved to parameters.yml. Actually it is a generated FB test user.


We're on the same page then! :)

The loop was a consequence of (you) using the dialog for opening multiple files. So I guess it was indeed a preparation for the future. But it is better to use the single file dialog for now!


I just figured out. Since we are changing from 4f to 5f so maxjetflavor =5 . Am I right?


Well, you need to put the file into a directory in the $PATH for the compiler to acknowledge the file.


If Alice bans Eve, what happens if Eve fakes the timestamp of her posts to avoid her last few posts from being banned? The chain of posts attempt to solve this.


You should not initialize hook variables this way.  It's considered a bug, since hook variables are allowed to be set before being `defvar`d, and in that case `defcustom` won't change the variable value, meaning that the default hooks aren't present.

See http://lists.gnu.org/archive/html/emacs-devel/2015-02/msg01271.html


Collection. I meant collection.


Why is a chronological order needed for moderation? Or any of the items you posted below?

What are you trying to solve?


From what I can see from their docs pycaption requires python 2.7, there are still some android builds using 2.6. I'm tempted to just disable subtitles on older versions.


Actually we need to fix the README to acknowledge what we're doing.


lgtm


Hi,

I got a bit further, here is a diff of my modifications:

```diff
 diff --git a/pkgs/development/interpreters/ruby/bundler-head.nix b/pkgs/development/interpreters/ruby/bundler-head.nix
index 6edc010..2087485 100644
--- a/pkgs/development/interpreters/ruby/bundler-head.nix
+++ b/pkgs/development/interpreters/ruby/bundler-head.nix
@@ -5,7 +5,7 @@ buildRubyGem {
   src = fetchgit {
     url = "https://github.com/bundler/bundler.git";
     rev = "a2343c9eabf5403d8ffcbca4dea33d18a60fc157";
-    sha256 = "11pbg71bhp66nyhwk0cpbcn1fhp4amv8sr2vx8dw83dcnqqafn92";
+    sha256 = "1vzm21fc37w89psxfk2fzi64zyb3gyhzr9smd6jk2r2kvgp6d38b";
     leaveDotGit = true;
   };
   dontPatchShebangs = true;
diff --git a/pkgs/tools/security/metasploit/default.nix b/pkgs/tools/security/metasploit/default.nix
index d34fce7..5c6b8b4 100644
--- a/pkgs/tools/security/metasploit/default.nix
+++ b/pkgs/tools/security/metasploit/default.nix
@@ -1,9 +1,8 @@
-{ stdenv, lib, bundlerEnv, ruby }:
+{ stdenv, lib, bundlerEnv }:

 bundlerEnv {
   name = "metasploit-4.11.3";

-  inherit ruby;
   gemfile = ./Gemfile;
   lockfile = ./Gemfile.lock;
   gemset = ./gemset.nix;
diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix
index 7dc78d7..c3f8b02 100644
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -2212,8 +2212,8 @@ let

   mscgen = callPackage ../tools/graphics/mscgen { };

-  msf = builderDefsPackage (import ../tools/security/metasploit/default.nix) {
-    inherit ruby bundlerEnv makeWrapper;
+  msf = callPackage ../tools/security/metasploit/default.nix {
+    inherit stdenv lib bundlerEnv;
   };

   mssys = callPackage ../tools/misc/mssys { };
```

However, I have a build error when nix tries to build metasploit:

```bash
$ nix-build  -A pkgs.msf
these derivations will be built:
  /nix/store/ac6945wzg7r2vrhdav0id4l7q5hd66kz-gemset.rb.drv
  /nix/store/3i2dxvcp8709kshpmg2y3zly4gjccd7d-sources.rb.drv
  /nix/store/pmpyf8sph338fbkb4ss9i82crn7k8pka-purifiedLockfile.drv
  /nix/store/4zig2765zhxcqy8049gvwk9szhwh4qr1-purifiedLockfile.drv
  /nix/store/5dyyk39ihnxfx5hsridhxhmq9m5fn9s2-metasploit-4.11.3.drv
building path(s) ‘/nix/store/n1jcgj4j6kp59vfbv1zf1w315ln8gygd-gemset.rb’
building path(s) ‘/nix/store/x2hkhz6718jr8a7ccyks6hh9qbx40zvc-sources.rb’
building path(s) ‘/nix/store/ka6fkgm11wjqipncmh9im2997mp5yvxr-purifiedLockfile’
building path(s) ‘/nix/store/3xqahmnn0pjxi7r5kglzgvyp13xi9286-purifiedLockfile’
building path(s) ‘/nix/store/rc00j0amkk7b1lwx7ij9b2ib3r66wgmd-metasploit-4.11.3’, ‘/nix/store/xg035831il0f06sbxlvlvhjc8zcgg8nx-metasploit-4.11.3-bundle’
installing
running pre-install script for ffi
There are no gemspecs at
/nix/store/xg035831il0f06sbxlvlvhjc8zcgg8nx-metasploit-4.11.3-bundle.
builder for ‘/nix/store/5dyyk39ihnxfx5hsridhxhmq9m5fn9s2-metasploit-4.11.3.drv’ failed with exit code 15
error: build of ‘/nix/store/5dyyk39ihnxfx5hsridhxhmq9m5fn9s2-metasploit-4.11.3.drv’ failed
```

(Note that I have never packaged ruby based packages up until now, so I might have missed something related to that…).

I’ll continue to see if I can move forward, and try to understand how `builderDefsPackage` works.


Why did you add an empty file bro!


Won't that signal an error if `(magit-file-at-point)` returns `nil`?  `expand-file-name` doesn't handle `nil`, does it?


That Leonid knows better, but I feel ok if we copy that setting to every style. To be honest I do think that all what displayed over the map should be style configurable, even POI, transport stops and I would go that way. It requires sometimes copy pasting between styles, but all features on the map belong to rendering style, that what I feel right now


That's better, on idea the escape was a diamond question mark, very weird.


I haven't added enum for pack flags yet. And I have no clue about possible values. And official packet read function only checks first bit. So I assumed that this would enough for starters.


this is actually fixes for prestashop 1.5


This even seems a better idea

```css
@import "syntax-variables";
.term2 {
  background: @syntax-background-color;
}
```

since it avoid weird color difference when the terminal is not the exact same size of the pane (since we round px to avoid line out of the pane viewport)


thanks eugene...this post is very old and I already did something similar...

do you have mac and you don't see this problem?


On Jul 7,  4:40am, notifications@github.com (=?UTF-8?B?RnJpZG9sw61uIFBva29ybsO9?=) wrote:
-- Subject: Re: [tcsh] avoid gcc-5 optimization malloc + memset = calloc (Fri

| This is optimized because of copy propagation. There should be used volatile variable as suggested in the original discussion:
| http://mx.gw.com/pipermail/tcsh-bugs/2015-May/000944.html

Fixed, thanks!

christos


http://i.imgur.com/0mFzwap.png

Using latest AMD Catalyst Driver 15.200.1023.7

Compiling for test it...


Реализация не рабочая.


@jsanda I had not paid attention to this: could we log the message at trace level? Or at least debug? It fills the logs and it's only useful when you're debugging.


Thanks again ^.^ Offsets for 20216:

    send_2=0x290B6F
    receive=0x28EFA8

:)


True, I missed that. Thanks


If Alace bans Eve, any rendering software will remove all posts by Eve as soon as that ban comes in.  Which I think is an adequate solution to the problem. Your idea above doesn't seem viable, as far as I can tell.

See the 3rd paragraph under 'Posts' for a very similar situation; "The user might try to post after his right to post was rejected"


Yo te lo he corregido para el curso de MiriadaX. Perfecto! ;)


Definitely, I'll get it updated. Great spot, thanks Mark!


hehe


hhe


I see.
Unfortunately, currently I don't have OSX.
But last time I was using it with old reincarnation of this plugin and OSX 10.10.2 there was not such a problem.


data_len is not fixed by T and f_data?


test


Commented out xeng vector accumulator as it was running a loop.


tt


ttt


ttttt


the story continues here:

http://www.smarttrade.co.com/en/daily-reviews


Alice and Eve live in opposite corners of the world. Eve posts a nasty message. Actually, a lot of nasty messages and posts; she's spamming the board. Alice bans Eve. Eve uses rogue nodes to fake the timestamp of her posts. Eve keeps spamming more nasty messages until Alice's ban reaches her node. Having a chain with chronological order would allow Alice to define a single cutout point where Eve's messages won't reach the board.


All changes made to config file


typo, but it won't affect here


Should be changed to `element.removeAttr("data-click");`.


Ever you compile or me. If you manage to compile it could you send me it so i can finally test on here. @SpaHost


@varund7726 so it works for both variants like this?
else i will modify the app, to just read the part before _


The design specifies that each board has a "ownership-chain" Which contains things like CreateBoard, AssignModerator, BanUser, etc.

This has to be a chain because it has to be sorted and because reordering things can lead to different results.

The platform itself doesn't make sure this is a chain, it will just reject a chain that doesn't make sense. As such it is the moderators responsibility to not send out new events before their nodes are synced. As that might cause a fork.


Why T_elem is 10e-3?


It's just me being a tad stupid, was thinking of 7.0 but wrote 5.7 :-)


Для чего здесь вызов produce?


should we add clock_stub for other cpus and other cluster? could you explain why that's not needed?


Ok.

In the current design individual posts and votes and replies are only tied to the board and the ownership-chain.
As such two posts have zero relation to each other and this means that two people on different parts of the globe can submit a post simultaneously and this works just fine.

You agree about that?


Let's suppose that Chuck posts a message, and Alice bans chuck, but Chuck's previous message was important for the conversation. How does Alice ban Chuck without removing his last message, that might have not been propagated to the whole network?


(y)


Hola,

"asset's values" ya lo corrigió Fergu en la sección 4.3.2. Se cambió por "values of the assets". Por favor, si ves que ahí está corregido, lo puedes corregir en los demás directamente. Se ahorra bastante trabajo así. Gracias.


The structure is much better. I think the document could use an introduction and goals section at the beginning to make the expected result clear at the beginning.  These could be added later though.

LGTM (with miminar's typo corrections)


This was not meant to show up in youtube-dl's issue tracker, but GitHub makes it very hard to avoid doing so


That's outdated. You can either submit this as is and I'll adapt or you could rephrase this section to use `@EnableRabbit`. The feature and code are very similar so if you could use the same structure that'd help.

Either way, make sure to sign the CLA when you submit the PR


Those changes are result of automate refactoring. I haven't checked them much. It's looks wrong, indeed. How about checking defender skill Id?


In general, looks good to me. That's awesome, thanks!


Shouldn't we keep this content as resource file rather than inline to the source code?


Also removed object-assign


linearFold используется не только для работы с числами.


Is this if statement required? We always push_back above the while loop, so there should always be one in the vector.


Oh, I see. So we use a chain only for moderator and board status, and let messages sort out themselves?


Nope didn't tried, but I haven't noticed anything but 0 in combat dumps.


`auto const& prev`?


Crap. Cant use a proper IDE at work :P Will fix.


Cool, I am just trying to understand the docker ecosystem.

I have been reading on it lately and one thing I noticed is that Dockerfile is meant to be immutable meaning that it will always get the same bytes whenever you build it.

 - Did I understand that part correct?
 - If so, wouldn't this be against that?


почему тут используешь не строгое равенство?


change made by reporobot


adasd sad from githum


to clarify: using addUnconfirmedUserId with a user id provided from the backend, not the rest client, may be safe.  but the action api should be safe no matter which function is called in which context.

come to think of it, that may be asking a bit much.

to summarize: at least the exception should be thrown as intended if user already exists, and the action should be documented: "it is only safe to call this with a user id that is not provided by the rest client, but by the backend."

in principle, this is the kind of policy that lio is designed to handle automatically.  but i am ok with leaving this as a FIXME for now.


mr.pleaze@askmen.com
www.1foreverfreedom.org


@steida personally requested that I add the `.` here :smile:


The 2.4.2.2 branch is inactive. Please don't push new commits into this branch.


**Рекомендация** старайся всегда определять блок даже в случаях когда используется одна инструкция в теле (сбережешь себя от ошибок)


So what am I supposed to do?

```elisp
(defcustom hook nil ...)
(add-hook 'hook 'function2)
(add-hook 'hook 'function1)
```

? Now the option is always modified, even if the user did not modify it. But I suppose if that's the way one is supposed to do it.


This isn't covered by your changes, but looking over the algorithm I wonder if we need to cover the case where we have:

 * SEG_MOVETO
 * SEG_CLOSE

or

 * SEG_MOVETO
 * SEG_MOVETO


this test doesn't need a docstring


Rather than calling the higher dimensional types "cubical", which will just lead to endless confusion, what about calling them 'cartesian types'?


Please merge this commit into the master branch.


Why? I think that's a perfect example for the usage of multiline strings. :)


We are 1FOREVER FREEDOM'everywhere


не хватает ```else```


`aria-label` must be localized. Prefix it with `label:`.


Done in https://github.com/geneura-papers/2014-2015-COMCOM-MUSES_vs_SotA_BYOD_Systems/commit/1647431815e0f6db426cd12dfcd34dcc64f11cbc ...


如何检测现在lua代码里面是否有全局变量呢？请补充下


you're not testing what happens after the hero power is played - check the opponent hero's health before and after.

you should also test the player's new hero's health is 8.

Additionally, what you could do is test this with a warrior hero (by passing WARRIOR, WARRIOR to prepare_game) and kill majordomo on your own turn instead of with a Pyroblast by giving the player an assassinate or something. Before destroying majordomo, play your hero power, check it's no longer playable, then play the assassinate, check you have 0 armor, then see if your new hero power is playable once more. Play it, check for the results.


If the above change is implemented, this would then become something like:

```
if (am_is_valid_paos_request(r))
```

This `#ifdef HAVE_ECP`-block would then become the only place in this function where anything special is done for ECP-requests, which I think makes the logic easier to follow.


Yep, I can see how ulong would be a problem on x64. But, is that right? I think it might be interpreting the handle values as pointer addresses. Or does it work? I'd be curious to see a code sample.


@machisuji this patch should also be applied on `release/4.2`.


この仮実装まではとても教科書通りの TDD を実践できていてとてもよいです。
で、この次のコミットでいきなり突然大正 1 年のテストケースを追加して、その次でまた仮実装をやっていたんですけど、ここはできれば

- 明治 2 年のテストケースを追加 [Red]
- 明白な実装をしてテストをパス [Green]
- テストケースのリファクタリング[Refactoring]
  - 不要なケースを削除でもよいし、パラメタライズド・テスト（※）にするのでもよい

という風にしたほうがよかったですね。

※パラメタライズド・テストについてはこのあたりを参照。
http://mike-neck.hatenadiary.com/entry/2015/05/06/081646
http://tech-sketch.jp/2014/03/rspec-tips-%E3%83%91%E3%83%A9%E3%83%A1%E3%82%BF%E3%83%A9%E3%82%A4%E3%82%BA%E3%83%89%E3%83%86%E3%82%B9%E3%83%88%E3%81%A7%E3%83%86%E3%82%B9%E3%83%88%E3%82%B3%E3%83%BC%E3%83%89%E3%82%92%E3%82%B7.html

要はテストケースは共通で使って、可変の部分だけパラメータとして渡しましょうっていう感じです。RSpec の場合は gem を別途導入しないとできないんですけどテスティングフレームワークによってはデフォルトでサポートしています。
素の RSpec でも工夫次第で同じようなことができます。以下例。

```ruby
describe Year do
  describe "#to_jp_str" do
    let(:year_obj) { Year.new(year) }
    subject { year_obj.to_jp_str }

    describe "1869年" do
      let(:year) { 1869 }
      let(:expected) { "明治1年" }
      it { expect(subject).to eq expected }
    end

    describe "1911年" do
      let(:year) { 1911 }
      let(:expected) { "明治44年" }
      it { expect(subject).to eq expected }
    end

    describe "1912年" do
      let(:year) { 1912 }
      let(:expected) { "大正1年" }
      it { expect(subject).to eq expected }
    end

    # 以下略
  end
end
```

パラメタライズというほど綺麗にはならないですけど、`describe` や `context` のネストと `let` をうまく使うと各テストケースの重要なところだけを注目させることはできます。
ちなみに明治1年と44年をテストしてるのはここが境界値だからですね。


When editing the entry with Vim editor, you can use markdown syntax


@teoljungberg Cool, makes sense. I'd argue that trading making it a bit easier to read a specific example and making it robust enough that it could possibly be used again for a different purpose while losing the behavioral meaning of contexts (in the above example, the `context` might as well be part of the `it` docstring) and making it more difficult to visually trace the distinguishing difference of the context (which when using `let` is a huge help while doing code reviews... reviewing the more verbose-but-easier-to-read form is very tiring and error prone, every example is largely starting from scratch rather than just making a tiny tweak to what you've already read) would sometimes fit the bill but it seems very heavy handed to declare it outright "bad" without properly outlining what you're giving up in the choice.

Why not at least properly use `let` in the "bad" example so people can see a real comparison? To me at least, the whole point of rspec (at least in simple, everyday usage) is to nimbly draw behavior out of the `context` blocks thereby tightly aligning the behavior to the structure. It seems like this approach would be more appropriately fitted to the philosophies of test unit or mini test which use the same arguments as in that issue.

I'm sure you've been down this road many times though, so whatever works for you guys :beers: Hope all's well @gylaz!


good catch: it was a leftover


If the superclass's implementation of equals() uses **getClass()** test then sublass objects and superclass objects will never compare equal. This means that you can not ask a HashSet of subclass object whether it contains an equal superclass object. The answer would always be **no**.


We can **not** override superclass's equals() method implementation if superclass's equals() uses **instanceof** operator to compare type. It allows for a slice comparison of sub- and superclass objects but it will always lead to violating equals() contract as we can see in the console. When we are extending some superclass we are responsible for checking it's equals method implementation. If superclass's equals() is using instanceof operator we can not ovveride equals method in a sub-class. Additionaly designer of a superclass should actually make sure that we can not do this by declaring his equals() method as **final**.


Indeed, thanks for noticing. I have pushed a fix.


Duh, this should belong to defender, I messed up. I just combined every possible downed flag, reversed from client into single number. But In context of attacker, this should be completely different set of flags ...
Thanks


> надо бы создать функцию `earlyInit`

А если юзверь отключил скрипт кнопкой на панели? Тогда `earlyInit` будет мешать, меняя нативные конфиги борды. А щоб проверять (Cfg.disabled = 1), нужно инициализировать конфиг скрипта (что занимает много времени).
Не.


I think it would be better t = 0:ts:T-ts but it's not critic.


I've renamed `iconButton` to `sg-icon-button`. Please use it for icon buttons only. The  `iconButton` class from the above *text* buttons should be removed.


Done in https://github.com/sunny/devise-track_locale/commit/7abcc4d0677dc4fc452c05e68bdb812ec85e0f20 ! \o/


Apologies for that crappy test. I'll make a better one when I'm back to my laptop


Okay, I'll remove it now.


this is slightly misleading: it doesn't explain that `postconf` checks for AES-NI and will switch it off even if `modernity` is enabled (thus there is no way atm to compile AES-NI support for nocrypto on a host without  aes instructions). but in the end it shouldn't matter too much :) -- great work! :dancers:


Should I write "<summary>lol, dunno</summary>"? Name is right, purpose unknown. I can only guess and do more reversing/research.


Yay!


I would use a bbox strategy instead `ol.loadingstrategy.bbox(extent, resolution)`


null password always fail


n_samples = floor(fs*T)


これは程度問題もあるしこの程度なら大丈夫だとは思うんですけど、基本的にテストコードは宣言的であればあるほどよいはずで、いかなるロジックも入らないのが理想です。
なんですがここではテスト対象のオブジェクトを使った計算（単なる引き算ではあるけど）をやっているので厳密にはよくないです。しかもここで使ってる 1869, 1912, 1926, 1989 はマジックナンバーになっちゃってますね。


Let's reiterate on issue #1340 but I still feel not to open too much functionality of configuring it, rather keeping different styles and preconfigure it for user.


If a moderator wants to delete only some replies, then use the delete-reply message. If the moderators bans a person, that persons messages are no longer shown.

 I understand what you are trying to suggest, but the thing you are trying to do doesn't work without a centralized server.
The reason for this is that you are guaranteed to get conflicts and branches when people everywhere can just post messages with a parent.  If you somehow solve that, you will end up with a really really slow system that does a planet-wide mutex or something. It can't work.

This is why Bitcoin was so revolutionary because it solved this unsolvable issue. But I don't think that miners and blocktimes of maybe 10 minutes makes any sense.


Oi! This text should not be dropped completely as it’s needed for explanation, right? It’s also not the same text I mentioned in https://github.com/owncloud/core/issues/17437 – I only meant to drop the word »3rdparty«, and the extra HTML container element around the whole block. ;)


My gut is we could just leave it as `title: 'Groups',`


That is what I do, but if you're unsure you'd better ask on emacs-devel.  I'm not sure I understood your next to last sentence, though.  I mean, of course the option is modified, but you want the default value to be there, don't you?  And for sure no user expects to actually _clear_ the default value by calling `add-hook` before Magit is loaded—which is what happens now.


Problem is that right now we're setting both action's skill ids to the skill used by the attacker, and various parts of our code depends on that. Officially the skill id of the target action seems to be the active skill of the target, or CombatMastery if there is no active skill. I guess we'll need an AttackerSkillId property or something.


これは TDD といより設計の問題ですがここは文字列返すんじゃなくて例外をスローするほうがいいんじゃないですかね、たぶん。


Hmmm I must have mixed it up with another text as this one doesn't have "3rdparty".
I'll recheck


> 大丈夫だろうか

あまりよくなくて、その答えは https://github.com/bukuro0312/TDD_study/commit/e3ef5ea554a0d407ff0e1abd7eb7f42df2d64297#commitcomment-12038280 に書いた通りになります！


I left them intentionally, to make fewer breaking changes at once. Because this would mess @xeroplz pull request and potentially others. Skills can be fixed gradually.


:) muchas gracias!!!


That sounds like the best thing to do, indeed. It is what the current 'readme.md' explains to do.

One message chain per board.
I call it the 'board ownership message chain' for clarity.


So you don't know what this and the other options are for? Well, obviously you can't comment them in that case, I thought you knew^^"


Собсно, я реализовал, щас выложу чисто чтоб ты глянул. Тут аще пришлось вот этот костыль выносить в отдельную `beforeDOMLoad` функцию выносить, ибо мы не используем напрямую `localStorage`, а проверяем его на костыльность.

```js
function(beforeDOMLoad) {
	try {
		locStorage = window.localStorage;
		sesStorage = window.sessionStorage;
		sesStorage['__de-test'] = 1;
	} catch(e) {
		if(typeof unsafeWindow !== 'undefined') {
			locStorage = unsafeWindow.localStorage;
			sesStorage = unsafeWindow.sessionStorage;
		}
	}
	if(!(locStorage && (typeof locStorage === 'object') && sesStorage)) {
		console.log('WEBSTORAGE ERROR: please, enable webstorage!');
		return null;
	}
	return true;
}
```


You're right ^ fix is on its way!


そしてこれ RSpec 実行時に `--format doc` オプションつけて実行したときの実行結果どうなりますか？
仕様を表してるといえますかね？確認してみてください。


Rather than change this to an explicit list of files to check (which won't catch new added files), use the `#noqa` comment at the end of a line to ignore specific lines/issues.

`../uvcdat/Packages/xmgrace/Lib/__init__.py:1:1: F403 'from xmgrace import *' used; unable to detect undefined names`

Or even better, just import the objects that make up this module's public interface explicitly (e.g. rather than using the `from module import *` syntax in `__init__.py`, just import the specific bits that need to be exposed). That's how this warning is supposed to be fixed -- `import *` is considered bad practice in general.


I'm a bit sceptical about this. Basically, you are grabbing query parameters from requests that were not necessarily aimed at mod_auth_mellon. E.g. a request for `https://example.org/ListComponents?IsPassive=y` will fail because this code tries to parse it as a mod_auth_mellon parameter.

Do you have a need for sending passive authentication requests with the ECP binding at the moment? If not, I suggest simply dropping this piece of code at the moment, and if the need arrives one can add a different method of signaling it. (E.g. a request header or a specific endpoint for starting ECP authentication.)


I meant that the option would show up as "modified" even if neither the user nor a third-party package touched the value.

So far I just assumed that users were expected to ensure that default value was already established before attempting to modify it.

```elisp
(with-eval-after-load 'foobar (add-hook 'foobar-hook 'my-additional-stuff))
```


This approach is so much neater splitting the large module up into classes


@xeroplz 's PRs need quite a bit of fixing still anyway, don't worry too much about it.


@flippmoke - probably but I'd keep changes to minimum for now. Current change works around `agg_conv_clip_polygon` returning  `line_to(prev_move_to)`  followed by `close_path` where close_path is sufficient.


Should this be `"PAOS"`?


>I haven't added enum for pack flags yet

Isn't CombatActionType the flags enum? o,o


@valib Aaaagggrrrhhhh, read last post only :-1:
I forgot about FastStart flavor which I have all the time set to $True :+1: with combination of this patch.
Now I understand faststart had different behavior for MEncoder [0] and FFmpeg [2500].
Thx


opcache has to store this array in SHM, and to do this, it should know its size. See https://github.com/php/php-src/pull/1393/files#diff-5f65684ab44e2bf79164ab4115b185e9R240


Done.


Hi6220 is special case, all cpus only use one clock source. So just need add clock node for cpu0.


OK


Add tests for assertions:

```
with self.assertRaises(ValueError):
    SBV(-500, 2)
```


обычно директиву use strict объявляют в самой функции, а не в начале файла. Если объявляешь в начале файла, не забывай, что могут быть потом проблемы при склеивании скриптов (если один из скриптов содержит ошибки, которые исправляет директива use strict).


погугли про umd (universal module definition). твой модуль будет работать только как AMD, но не как CommonJs или глобальная переменная.


@j3k0 I need your opinion on this. I removed service version number from redis prefixes, but I am not sure whether I should also remove versions from apps that token belongs to. In other words, my question is does token stays the same when app is updated?

1. `alice` stores token for `chess/v1`;
2. `alice` update her chess to `chess/v2`;
3. will token `alice:chess/v1` work for `alice:chess/v2` app or those should be separate?


Add test on equality of SBV and BV for positive numbers.

```SBV(10, 16) == BV(10, 16)```


стоит реализовать через XmlHttpRequest


@mestebangutierrez, which is the actual path of the harvester? :)


Кстати, а ведь можно читать конфиг и до загрузки DOM'а. Как раз сэкономим время на инициализации и визуально скрипт будет выполняться быстрее.


does this break every time sent by the client, even if client and servers are in same timezone, instead of the old behavior causing issues only if client was on a different timezone ? For example creating calendar events


каждый раз будешь затирать всё содержимое #main. Возможно, пропустил + (+=message)


@hoholee12 the last modified date isn't secure because is the build date, busybox binary can be build today & can be older version like 1.1.2... I don't get version in the name and parse it lol


точка с запятой не нужна


If we can sort the results in mongo. We could dramatically speed up the results. We would only have to find 10 results at a time having offsets of multiples of ten for each page.

Example:
```
db.bios.find().sort( { name: 1 } ).limit( 5 )
db.bios.find().limit( 5 ).sort( { name: 1 } )
```


Great!

One really surprising thing to keep in mind:

Abjad implements many things as Python properties (with the @property decorator). This makes sense because so many of the attributes of stuff in musical score are derived from the positions at which things sit (relative to time or relative to the nesting of containers). But it turns out that calling ``hasattr(expr.foo)`` will cause ``expr.foo`` to evaluate completely in order to return a true / false value to ``hasattr()``. This is of course fine, except in the case that ``expr.foo`` takes a significant amount of time to evaluate. The best-practice guidelines for using Python properties tell us that properties should preferably never take a significant amount of time to evaluate (ie, that properties should always be computationally "thin"), and over the years we've gone through the codebase and systematically changed complex properties to methods. But there probably are a few places remaining in the codebase that feature "fat" properties where ``hasattr()`` might be problematic.

The use of ``hasattr()`` in this commit is great! Just some background lore here on our evolution towards the recommend best practice in property implementation.


See 6533ee7c5407b0e358d9db7f7500c4aa168247aa and earlier comment.


Sorry, I misread the flow. It's not a big issue here, but creating the object every time a file is added is not necessary. $this->zip can be instantiated in the constructor or an initialization method. The method ´zip()´ or something like ´addFile()´ could be called for every new file added to the archive. Closing the zip archive and dumping the temporary files can then be cleaned up right before the response is being sent to the client.


On Monday 06 July 2015 11:07 PM, Azhaguselvan SP wrote:
> shouldn't this be part of the fabfile.py so that we can shell into the
> remote machine and publish this ?

You are right. Let me fix it.

Thank You.
--
Senthil Kumaran S
http://www.stylesen.org/
http://www.sasenthilkumaran.com/


Yeah, I noticed that in elf ranged and made a ugly hack to change it to defender skill or basic combat. That was kinda surprise for me. In official action is combatant, so it's natural to have appropriate skill id set.
I don't understand part about AttackerSkillId, is there so much code that expects defender skill id to match attacker?


Зацени.
Плюс: избавились от `window.location.reload`
Минус: скрипт всегда правит конфиг борды.


пропущена точка с запятой


Thanks! The single quotes are great because we always try to use single quotes first and then use double (and even triple quotes) on an as-needed basis in expressions that already feature a pair of single quotes.


For my reference, the default frag size is this:

    curl http://localhost:8080/api/admin/settings/:SearchHighlightFragmentSize -X PUT -d 100


Thank you man


Seems like a bug. I think this must look as following:
```c
check(padded->data[m_len] != 0x00, "Invalid padding");
```


@flexiondotorg: the absence of slash here completely broke GTK+3 build - and I couldn't figure out what's going on for about 5 minutes :laughing:


A space would be helpful here I think.


FYI I think SHEETSTYLE=1 is A3 (metric paper size). If these figures are the paper sizes in mils, A3 would probably be about 16553 × 11705 mils, although I found the actual drawing area in Altium is 15500 × 11500 mils.


The main issue I have with "FIXME" is that it implies that there is something that _must_ be fixed at that location. E.g. "FIXME: We need to deal with the client specifying the foo-parameter here".

From your description in the email this sounds more like something that may be nice to have at some point, but which has no immediate requirement.


This commit message slipped away during this branches merge, it should have been as follows:

fix(PopupMenu): refactor update header entries and it's handler


Совсем нет абстракции и ооп. Подумай, как можно организовать свой код, используя функции-конструкторы и прототипы.


:smile:


Do you mean new line?


Is it not better to have the docs here:
https://wiki.jenkins-ci.org/display/JENKINS/Delivery+Pipeline+Plugin
?


So the addition to setup.py means that we intend the ``six`` module as a new dependency?

I had to use ``sudo pip install --upgrade six`` for everything to work with this one. So if we intend the new dependency we should probably mail the list and let everyone know.


Не вижу реализации заданий: 5, 7, 9, 10, 11, 12


Have you tested this? You seem to referencing into a 5-element array with a satoshi-denominated index.


А, ліл.Он и так бы его всегда правил.
Что-то я не соображаю. `if(Cfg.disabled)` находится гораздо позже `var formEl = Initialization(checkDomains);`

```js
	var formEl = Initialization(checkDomains);
	if(!formEl) {
		return;
	}
	new Logger().log('Init');
	var str = yield* getStored('DESU_Exclude');
	if(str && str.includes(aib.dm)) {
		return;
	}
	excludeList = str || '';
	yield* readCfg();
	new Logger().log('Config loading');
	if(Cfg.disabled) {
		addPanel(formEl);
		scriptCSS();
		return;
	}
```


If the superclass's implementation of equals() uses **getClass()** test then sublass objects and superclass objects will never compare equal. This means that you can not ask a HashSet of subclass object whether it contains an equal superclass object. The answer would always be **no**.


Okay, then. Scrap my commit then. Also, I only just saw your part about syncing. Your solution is much more elegant. Just a question, where are the user status (banning, approved submitters) changes going to be stored? A separate chain, I suppose?


First Commitment


Если из `readCfg` убрать коррекцию времени и проверку на обновления, то его можно перенести в `beforeDOMLoad`.


you should keep all tags lowercase.


👍 great!


Thinking out loud..

If we change the major version number, it means a major non-backward compatible change was made.

 * Only clients supporting `chess/v2` should receive notifications concerning `chess/v2`.
 * Clients supporting both `chess/v1` and `chess/v2` should register their token for both.
 * If newer client drops support for `chess/v1`, they should NOT receive `chess/v1` notifications.

End result: **yes, it's separate**.


The skill training  code was written under the assumption that the combat action's skill id is the id of the skill used by the attacker. That was easy and made sense at the time. We simply had no idea why the target action even had a skill id, and when it seemingly always was `CombatMastery` we set it to the actual skill used instead^^"

I've just realized that we actually already have an `AttackerSkillId` property xD Though it was added for a different reason and isn't used consistently.


Shouldn't this be:
``long refs = AtomicDecrement(&m_refs);``
Reading m_refs directly seems unsafe.


what's DOCSTYPE? there's a missing piece here, I'd suggest googling "doctype mdn" and picking the first result you see and look at the example of how they use doctype and read more about it.


I think this commented code can be removed


Okay, rolling this back.


Правда тогда скрипт будет считывать конфиг на всех страницах. Смысл расположения `readCfg` после `Initialization` в том, что если скрипт не обнаружил поддерживаемую борду, конфиг считываться не будет.


what is the "head" tag for? Why shouldn't you have this kind of content in it?

Also, what's wrong with putting the "body" tag in the head tag?


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 570](http://build.prxm.it:8811/viewLog.html?buildId=3376&buildTypeId=Jarvis_DocumentStore_CI) is now running


Сейчас напишу патч для переноса `readCfg`.


Iterator객체 보다는 Iterable Interface를 parameter로 사용하는 것이 좀 더 확장성이 있을 것 같습니다.


Check argument values; at least there must not be negative.


Right.


Может, все-таки лучше откатиться?
beforeDOMLoad выполняется на всех страницах, выполняя try-catch с костылями localStorage и записывая `sesStorage['__de-test'] = 1;`


Yes I know. Debug is still on going. I think it remains a bad case but I don't know if I can fix it. Otherwise I will add an userhack to disable texture shuffling....


@vcunat Is this a good idea (given the discussion at https://github.com/NixOS/nixpkgs/pull/8369)?


String.format


all selectors that are defined by the HTML specification (ie, not your own selectors for id and classes) should be lowercase.

Also, every `property: value` should be on a new line, like this for the h1:

```
h1 {
  font-size: 38px;
  font-family: Impact, sans-serif; /* <= you didn't use a selector in yours, you should fix that through out your code */
  color: a44400;
  border-bottom: thin black solid;
}
```


А, опять таки, он и так везде выполнялся! Проверка на БОРДУ у нас тоже позже происходит


All hex colors should start with a #, so here: `#a44400`


Because code style, slowness and complexity.

I'll handle it soon, thanks! Also i should change repo style of these `npm-pkg-*` repos.


"Firewall Notification
Your access has been blocked by firewall policy 80.
If you have any further concerns, please contact your network administrator for more information."
message to [http://www.gknw.de/mirror/curl/win32/] and[http://www.gknw.de/mirror/curl/win32/old_releases/]
and it's not by my Firewall !


Need a `font-family` property name for the `Impact, sans-serif`.


hum, exact. I didn't think to this scenario


This line was fixing the flake8 error: `lib/galaxy/__init__.py:95:1: F401 'galaxy' imported but unused` .

Unfortunately this is breaking Galaxy datatypes loading with a ton of errors like:

```
galaxy.datatypes.registry ERROR 2015-07-07 11:19:12,332 Error importing datatype module galaxy.datatypes.binary: 'module' object has no attribute 'galaxy'
Traceback (most recent call last):
  File "/usr/users/ga002/soranzon/software/bgruening_galaxy/lib/galaxy/datatypes/registry.py", line 204, in load_datatypes
    module = getattr( module, mod )
AttributeError: 'module' object has no attribute 'galaxy'
```

@dannon: suggestions? Should we add a `# noqa` at the end of this line instead?


Update T4 Version to 2.4.0


tab (or 2 spaces) for indentation.


Check parameters, if needs a {}


You're right. That's why I want to fix it :smile:


don't forget the # in your hex code: `color: ff7a1b`


you see...all what you confused  is solved....
yes ..I do...


이 에러로그는 문제의 본질을 표현하지 못하고 있습니다.
주입 오류라는 단어는 모호해서, 무엇때문에 에러인지를 전달하지 못합니다.
Enum type을 사용하는데 default로 흘러든다는 것은 eventType이 null인 케이스일텐데요.
이 경우 null을 리턴하는 것도 이상하다고 생각합니다.

데이터 중에 잘못된 값이 있는 것이 문제라면 에러를 발생시키고 동작을 멈추는 것이 더 좋을 것 같습니다.


I'm not sure if it's allowed to add a coma after last property value. But I think there should not be any coma, it looks weird.


missing semi-colon, make sure `property: value` pairs are on new lines.


The issue is that max_amount is currently:
``static const unsigned int max_amounts = 4; // Must be at least 4.``
Without this change I could presumably set it to 5 and have sensible behaviour.
After this change, m_amount[4] would be uninitialised.


indentation, otherwise, :+1:


This is not the only place where this check is performed, I would extract a method and avoid repetition.


indentation, otherwise :+1:


I happen to disagree, but I don't think that our opinion has any relevance whatsoever given an authoritative statement by the Emacs maintainer.  Besides, this feature has been in Emacs since ages and won't go away.

And I think that not even Magit has the standing to work against Emacs upstream.


doSelect -> select면 충분할 것 같습니다.


Yep, this was a clear bug.


Just comment inline and I will fix the things. Otherwise I will maintain my own fork.

Thanks for the nice module, btw! :smile:


코드를 주석화해서 남겨두지 맙시다.
코드는 git이 기억합니다.


@nyalldawson unfortunately I've not found any issue and cannot reproduce the test failure myself. I'm just pretty sure that the failure in the test is caused by QUERY_STRING environment variable not set in:
 https://github.com/qgis/QGIS/blob/master/src/server/qgsserver.cpp#L463


Just curious, why did you create a `index2.html`? you should port this code over to `index.html`.


yup import * kills the protection :smile: its like free for all...


nice one nir! hehe wa lagi sud ang whatsbeef.net nir?


@Oksisane So you did create multiple files for each instruction.  I really wanted to stay away from that.


@Ordval this is not the place to report any issues.


k_car_prn, k_car_nh, k_car_data and k_car_data_NH are not necessary


Ok then I think it will be harder to resolve it than just switching to OnUnitCreated. :-(


Add examples of proxy and target file management parameter use


pack이라는 이름은 이상하다고 생각합니다.

EventInfo정도면 괜찮을 것 같은데요, 이미 사용하고 있나요?


@lindamartin what is `sslr-core` dependency for?


Huh?

> `./configure --enable-modernity` enables non-portable code
if the build machine supports it.

>
> It defaults to `enable`, ...


This transducer can return nil, compose with filter? `(comp (map (...)) (filter some?))`


Enum이라는 suffix가 반드시 필요한가요?
java는 강력한 타입 체계를 가지고 있는 언어인 만큼, 이 값이 enum이라는 것을 알아차리는데는 변수명의 도움은 필요없다고 생각합니다.


phi0_if is not used after this line


You should delete this file. In the terminal, in your project directory, run:

  git rm css

This will remove the file from your code base _and_ from your git history. You shouldn't need this file at all


oh, I overread that conditional. sorry, stupid me


```
final Collection<MultiDeBruijnVertex> danglingHeads = vertexSet().stream()
        .filter(v -> inDegreeOf(v) == 0 && ! isRefSource(v))
        .collect(Collectors.toList());
```


* LAFFAGE Clément


Where's your closing tag for all of these `li` opening tags?


eventInfo에서 eventPack을 얻을 수 있는 메서드를 만드는 것이 좋을 것 같습니다.
지금 상태로는 eventInfo가 eventQueue를 가지고 있고, 그 큐에서 하나를 꺼내와야 한다는 사실까지 알아야 합니다.


Anchor's look good. Nice work on those.


It will look better if shift will be calculated in one place, then you can use "shift + '%'" as return value in the first case


kk :+1:


第二次添加


where's the opening `ul` that's wrapping this batch of `li` tags? Do you need `li` here? What's an alternative tag you could use (`<br></br>` is not a tag, it should be a void tag (where it self closes, like `<br />` but you can remove `<br />`)


readme 都是能跑的


I am sorry, but ".so" is not always followed with ".", some libraries simply ends with ".so".


I mean, by all means, suggest clearer wording if you think it's needed.


@jeremeamia After this commit, and subsequent commits to remove the version check. How can I use Aws\Credentials\CredentialProvider::env() as the 'credentials' instead of passing the array containing key & secret?

My goal is to use the assigned IAM Role on an EC2 elastic beanstalk instance.


Good use of table, but there's an element that should wrap `th` tags. We'll dig into this deeper tonight.


I don't know which is correct.


I expected this call to happen within `startGame`, since this is the message at the start of the game.


maybe just in between of `*`s?


Why 4 lists? Isn't the message already in the lintResults list?


Is it needed to set background-size to cover here?


익명 내부클래스를 이용한 callback의 구현은 java의 몹시 나쁜 부분입니다.
closure를 사용해야 하는경우가 아니라면, 내부 클래스로 정의하여 사용해도 좋지 않을까요?

이 코멘트는 어느정도 취향에 좌우되는 부분이니만큼, 단순한 의견으로만 받아들여주면 좋겠습니다.


Also, what was the reasoning behind using the `strong` tag in the `h1`? Why might you not need it?


Say what?


Sounds reasonable.


I note that c4ed7b80c5153bd06d082bf8c6f2f3247a959b05 appeared to allow this.


Why two getters?


magic word는 상수로 분리합시다.


It would be simpler:
phi_prn1 = (phi0_prn1) + (f_prn*(t+code_delay))

where code_delay = the delay in seconds rounded to a multiple of ts


> You shouldn't need to explicitly destroy the bean now as it handles it's own destruction

Good to know.

> Sorry Max for the oversight (I feel the pain when code breaks because usage wasn't checked thoroughly).

Thanks for the sympathy, but this time wasn't so bad though. :-)


Maybe a note at the top that every word actually matters? :smile:


This was fixed elsewhere and is corrected on HEAD


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 570](http://build.prxm.it:8811/viewLog.html?buildId=3376&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 290 Build time: 00:13:50


Fix sent. Thx


这个是不是都可以评论?


Si peu !


we haven't covered it (we'll cover it tonight), but what is an alternative you could use for this kinda data instead of an unordered list?


Nice @isaquediasm :rocket:


i like that you like pizza this much. it's a shame to lowercase it and remove the apostrophe. :smile:


Please change this to `"-framework=" <> fwk`, see the `nub` call on line 296. There should be one option per element in the returned list.


phi0_prn1 is not used after this line


交流展示平台


OK :). Added.


@isaquediasm só lembrando que não devemos fazer commits direto na branch `master`, o ideal é sempre passar pela revisão de algum dos @IonicBrazil/moderadores /o/


왜 굳이 string으로 변환하나요?


@hodgestar Ready for review


always put spaces between your `property: value` instead of `property:value`


Good idea. There is another copy of this code that could be removed in AiWander::fastForward.


space between selector and curly brace.


Used multiple lists as it would help to separate the warnings ,info and errors ,which were mixed in lintResults.


Além disso seria ótimo adicionar um comentário relevante ao commit ao invés de `Update README.md` :smiley:


> Why don't we enforce this process from xbian-update?

couldn't agree more :+1:


It would be simpler:
phi_nh = (phi0_nh) + (f_nh*(t+code_delay))

where code_delay = the delay in seconds rounded to a multiple of ts


Oh, that one is already fixed. Diffs are confusing... my bad.


简约,人性化


> Thanks for the nice module, btw!

yea but.. hence.. the name, these names sucks. lol.

I cant say you what to fix cuz i have totally another in mind and with other modules. And if i have that time to comment how and what to use to implement it...


The presence of these counters for the debugging message is what is preventing you from extend the stream construct to take over this second for loop. This is also true for the recoverDanglingHead code...

Would you be inclined to use a 2-length final int array to keep both counts instead?
Perhaps you can find a more elegant solution on a already existing "IntCounter" class that would allow to keep a updatable counter on a final variable, or create your own if none exists; the class would be quite trivial, so I don't see much harm in doing that.


It would be simpler:
phi_prn1 = (phi0_prn1) + (f_prn*(t+code_delay))

where code_delay = the delay in seconds rounded to a multiple of ts


phi0_nh is not used after this line


Check parameters?


이러한 용도로 toString을 override하는 것은 적절하지 않다고 생각합니다.

eventStateString이라는 필드 대신, code라는 이름의 필드를 사용해도 괜찮을 것 같은데요.

그리고 toString을 오버라이드 하는 대신 getCode라는 메서드를 구현하는 것을 추천합니다.


Hi @graphific - I think the `#` and `!` characters swapped in the hashbang here


{} x 3


It would be simpler:
phi_data = (phi0_data) + (f_data*(t+code_delay))

where code_delay = the delay in seconds rounded to a multiple of ts


Any particular reason that tolerance parameter was removed? I believe the original intent was to prevent actors crowding on pathgrid points.


Will remove the unused one.


Can't you just reuse the same list and iterate it?


phi0_data is not used after this line


I stephen have make change to kazeem repo


data_len has to be set in bits (at 50 bps rate)


Enum에 코드값들이 모두 들어가 있는 것 같은데요.
코드값의 용도는 무엇인가요?

Enum은 기본적으로 ValueOf라는 스태틱 메서드를 통해 값을 찾을 수 있는데요.
이름을 그냥 사용하지 않고 코드를 사용해야만 하는 이유가 있을까요?


A singing account.. is that some AI-thing?


Ouch. Sorry about that. That is a function of by [editor crashing and puking in the file I was trying to open](https://github.com/junegunn/fzf/issues/206). I'll rebase and get that out of there.


Sure can be done.


层次有点混乱，先继续往后做笔记，我们上课串讲之后你再增补


Which change fixed the issue?


You need to add a code_delay = the delay in seconds rounded to a multiple of ts


큐에 넣는 동작을 표현하는 단어는 insert나 add를 사용하는 것이 적절하다고 생각합니다.
inject라는 이름은 낮설군요


Again the mustache tags were replaced with hard-coded values


you mean ![](http://sd.keepcalm-o-matic.co.uk/i/keep-calm-and-read-carefully-8.png)


Can you merge this if and the following one into one expression?

```
if (X && !Y) return false;
if (X && !Z) return false;
return true
```
to
```
return !(X && (!Y || !Z));
 // better
return !X || (Y && Z)
```


we talking about CombatActionPack


FYI with this change, '/etc/init.d/hekad' is no longer getting created on Ubuntu 14.04


blah?


이 로직은 큐를 소비하지 않고 있는 것으로 보입니다.
큐를 소비하지 않고 순회만 하는 이유가 있을까요?


Hi Arndt.  Thanks for posting the ov7670 driver.
I am inspired to do the same thing with the
Beaglebone Black.
later................dd


@phbcanada If customData is required, jsdocs should not tag the param as optional.


Will you also try the inverse.. i.e. where an activated King is vital to escort/defend advanced Pawns?  https://groups.google.com/forum/?fromgroups=#!searchin/fishcooking/escort/fishcooking/vuoEKP2yOCE/qhVtYiIhilUJ


Typo: should be SELECT, here and below.


이 switch문의 용도를 잘 모르겠습니다.
show로 getCurrentPlace의 결과값을 전달하면 안될까요?


What are fileType 1 and 2?


Hi Arndt.  I am taking your drivers and implementing the same for the
BeagleboneBlack.
Thanks for sharing them...............dd

On 7/5/2015 at 12:00 PM, "Arndt Jenssen"  wrote:

	Hi Elecmania, this project was developed with Eclipse CDT ..

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


Oh hell yeah!

The background image of all READMEs ever!


@riteshchandawar @pboyer Luke requested it as he is investigating the root cause.


@phbcanada jsdocs - shouldn't this be trayId to match param name?


parseMap이라는 이름으로 충분할 것 같습니다.


임시값이죠?


Oh, right. Sorry, confused it.


much better :+1:


@phbcanada Error message method name incorrect


Indeed. It was fix-in-a-hurry.


Similar to js, we recommend that your styling live in the css file.  HTML should be used for structure only.


I think you tried to explain that in your commit message, but why don't we use the shiny new DSL here too?


now this is fast. thanks!


Created is the only thing that can not insert a row into an existing pagination. We don't use auto increment IDs, but rather UUIDs. So after discussing what would make most sense, created makes more sense than a random UUID without any predictable sort order.


이 부분은 재검토가 필요할 것 같습니다.


This is not working. Can we back this out.

Here is the errors that happen:

```
Testing: xsha512, Mac OS X 10.7 [SHA512 128/128 XOP 2x]... (4xOMP) PASS

Testing: ZIP, WinZip [PBKDF2-SHA1 128/128 XOP 4x]... (4xOMP) PASS

Setting of real/effective user Id to 0/0 failed

FATAL: Could not load /lib/modules/2.6.32-042stab108.3/modules.dep: No such file or directory

Error! Fail to load fglrx kernel module! Maybe you can switch to root user to load kernel module directly

Setting of real/effective user Id to 0/0 failed

FATAL: Could not load /lib/modules/2.6.32-042stab108.3/modules.dep: No such file or directory

Error! Fail to load fglrx kernel module! Maybe you can switch to root user to load kernel module directly

Device 0: AMD Opteron(tm) Processor 6376

Testing: sha1crypt-opencl, (NetBSD) [PBKDF1-SHA1 OpenCL 4x]... PASS

```

NOTE, we may have to sudo run john ??


But more generally, this query doesn't make any sense. faviconSites joins a site ID to a favicon ID; there's no URL to test. Take a look at the pull request references in the bug to see Wes's existing proposed approach.


왜 필드를 각각 데이터에서 읽어야할까요?
객체로의 deserialize가 불가능한가요?


for each를 사용하지 않는 이유가 있을까요?


This should probably change to six as well


Is there a difference to not using a `do` block and instead just performing the assignments after finding the element?

Just trying to understand the purpose of the `do` block here in general...


delete


We have a `NotificationConstants` file now.


Within a few time, you will be able to code new functionalities ^^


Commit comment says history, code does remote tabs!


@rnewman Okay, Thank you very much.


I think this should still declare what it throws (AddressFormatException).


Instead why not initialize `currentPlotMover` to `null`, and lower down:
```js
function onSaveBefore(data) {
    if (currentPlotMover) {
        currentPlotMover.onSaveBefore(data);
    }
}


Have you upgraded all meshes that display error logs ?


Not sure where the comments for this change disappeared: even with a rebase they are usually still visible in the original ticket.

This test was introduced to verify 1db47f4535e82af881de1466c194d0211aaebdfa. It was required to ensure the old, m2e-android APK builder executed after the Google ADT builder so it does appear this is not required.

Are we sure we no longer need the old ordering?


15: M102 unrecognized bullet/signature


{} x 4


na i doubt its working


rank_of(ksq) or... relative_rank(Us, ksq) ?


this timer was created, but the payload was commented out...


find better name.


usually GET comes first ;)


just "package" might be better


not needed, it's always for an event


@phbcanada Error message method name incorrect


that reordering doesn't make much sense. `c` comes before `v`


LGTM


Shouldn't this be `date`, instead of the `Date` object?


I'd really use https://docs.python.org/2/library/zipfile.html directly  instead of this legacy thing


The name of this test is slightly confusing: just this ensure that the Maven builder is the last builder to execute?


for the attachment i'd use `attachment.file.filename` and for the folder i'd use `secure_filename(attachment.folder.title) or unicode(folder.id)` (since `secure_filename` may return an empty string if there are only invalid characters)


Good catch - unfortunately, Duration is [defined as](http://golang.org/pkg/time/#Duration) `type Duration int64`, so Sleep will happily accept an int.

More unfortunately, this int is then interpreted as a duration in nanoseconds.  We should probably fix that ;)


I looked deeper into it and there seems to be a lot of places where `action.Flags` compared to different sets like `CombatActionType.Defended`. I think `action.Is` should be same as `Has` (except is None, lol). And all direct comparisons should be changed to `Is` or bit checks?


Well, the intention here was always to use the "preferred" service provider for that platform (upstart), so this is "as designed". In general I try to consider SysV a fallback when there's not a more modern init available.

Definitely don't want to break expectations though... If this is an inconvenience, we can move this logic to an attribute so it can be overridden? Let me know what you think.


This does seem to make sense reading your explanation, the code just makes it look like there is duplicated effort.


GIT didnt foreseen comments on comments so i comment them together and start each comment with a  reference to commenter and his comment in apostrophe.

to okurz-iis:
„@frankheinen you should have provided this as a proposal in your personal repository.“

I dont agree with that. We can discuss via GIT but the changes dont need to be made in such a complicated manner.
Dirk Kowaleski directed me to make the last decisions on open Points, so i can just update direkt.
If you dont like that discuss with him.

to okurz-iis:
„I don't see a necessity to change our current implementation including what we proposed for RTKLIB as your changes don't seem to provide any benefit.“

I Think you has to.( Dirk Kowaleski: „Otherwise you work against the project request. Thats not allowed“ ).

to okurz-iis:
„You should have kept the diff easy to read. „

For this time i used an automatic formatter, because of that earlier comment:
„Fraunhofer-IIS commented on b0e801c on Feb 24
The alignment seems to be broken. Did you mix tabs and spaces?“

I Think that GIT will we able to handle the diff better on the next updates because i will produce the comming updates with that tool too.

to okurz-iis:
„For now we should rather allow more than less, e.g. keep the time status options“

I Think you mean in addition to the new options.
„UNKNOWN“: Do you can think of a system that use an unknown timebase.
I could think of „None“ as an option or „Other“.
„FINESTEERING“ dont know what it mean. Please explain.

to okurz-iis:
„pseudorange is definitely a single word“

No, if you follow the link you provided you will read „is the pseudo distance
„. It is a distance. So it is a Number not an Integer and it is measured in a distanceunit. I choosed Meter.

to sjko and okurz-iis:
"1. Subframes 1,2, and 3 with hex-codes are not included. " and "With your reasoning regarding the subframe I don't agree. As you mentioned at first, both should be allowed".

I Made it rather clear i thought.But I do a last try.
The O in OGRP means "Open". It is intended that all possible users should be able to provide and consume data. It is necessary that all participants can communicate without extra regulations. At least for the Main information this should be ensured.
For Main data there could be only one possible way, otherwise one System delivers in that way and the receiver expect the other way. That won't work.
It only could be fixed when all senders or all receiver has to implement all ways.
And that means unnecessary extra implementations.
So it is clear that only one Version of Ephemeris Data should exist.
In the beginning of the Projekt we decided that all Data should be clear readable. For me that is enough argument.
I have two GNSS Receiver from different Manufacturers in Test. For Testscenarios i implemented the OGRP Interface in both. Both deliver on their propietary interface only the decoded version of the Ephemeis Data. So if we want to be able to use other commercial Receivers than there even is no other way.
If you not agree than please discuss it with Dirk Kowalewski.

 To sjko:
„2. Time_stamp: It is better to separate in week number and seconds a week.“
„3. GPS week number was set at 1.1.1980 to match up with UTC.“

The Timestamp in general is not a GPS Time, but a System time. So i think it is better to use the timerepresentaion of the systemtime in Linuxsystems.
For the „timestamp_*“ in the channel_measurements you are right. I will change that.

 To sjko:
„4. Does 'timestamp_xxx' mean signal transmission time corresponding to each satellite?“

Yes! I think i See your Problem with that. There appear a lot of duplicate information!
But the Ephemeris Data shouldnt appear in every cycle, so remove them to Ephemeris Data wont work.

One Solution may be to create a new Message „Epoche“ with general Data of the whole Epoche. And made the measurements as part of the Epoche.
But should we use one Epoche for all GNSS Data or one Epoch per GNSS Type. In Either case it would be a big change for processing and producing Data.

 To sjko:
„5. Carrier ranges are not needed in channel measurement section“
„6. Code-phase is a similar case with carrier-range as long as not providing at every code epoch“.

Your right i will create a separate Messagetype „measurement_info“. And move them

 To sjko and okurz-iis::
„7. In signal type, "E5a" is "E5aQ", right? because "E5aI" exists.“
"L2CA" does not exist. Do you mean L2C?

I See your Problem. I study the Galileo Dokumentation, but i am not able to understand the meaning of the Band-declarations till the last conseqences.
So i would change „E5a“ to „E5aQ“ or should i remove „E5al“? Please have a look at the whole list, there arer other unclear signal_types.
The scheme is not clear because it is a mix of different documentations. Please provide a Schema and a better List.

To sjko:
„8. WGS84 defines in Geodetic coordinates, especially for ECEF. Change it into LLH.“

I See that point too. Can you provide a list of common and usefull coordsystems along with desciption of its units

While working at the WEBinterface i has the need for another additional Message
„satellite_info“. It will hold calculated information of the satellites.


ajuste


It seems it still needs to be updated to the external repo.


most of this class belongs into the mixin since the public part will use the same form etc


Hi Juan,

what's the purpose of this branch?

Thanks,
Dario.


Yeah, i am pretty sure. m2e-android now relies on the ADT builders, after the ADT APK builder is done, we cannot add anything to the build actually.


Is that really correct?  I mean, my namespace could actually have `:doc` metadata instead of the positional, optional docstring.  Does that still work in these cases?


@futureimperfect Good thinking about the $EDITOR env variable, although mine isn't set. I know Sublime Text uses `subl`, but I've had trouble making that work reliably from within scripts, so I use `open -a` instead. Feel free to make a PR if you have a better way that works reliably, though.


this might be the only line that changes depending on where you do it (different WPs)

so you could simply use `wp = WP...` in the classes using the mixin and move this method to the mixin, too


just


I dont understand the changes in the uart dt node.. an extra space?

And the commit subject.


@oliward Could you take a look at this commit for me as I don't know where or what I should do next.


i do not see any files in this course. am i missing something?


Except the APK builder. So the expected order is:

other builders < maven builder < APK builder.

This is for testing the first relation. The previous test is for the second relation.


I am not sure, but this checking for "last" seems useless, I mean it must be always true, isn't it?


Sigh...


`linked_object=self._conf` - no need to specify all those Nones manually.

and hm, do we really want to specify only top-level folders? maybe we need to think about how to do it... showing folders of contributions etc would be confusing because you might have the same name in many places. on the other hand, what if you have "slides" folders in many contributions and just want these?


singular? that's not what i meant... i meant "session" instead of "sessions"...


{} x 3


i see it.
I hope they will put it soon in the Codiad Market :smile:


Backporting my changes? :) Why don't you go with Cowrie? :) http://github.com/micheloosterhof/cowrie


Check arguments?


some old objects actually have non-numeric ids here :/

in the legacy code only event/category ids are now guaranteed to be numeric


I suggest moving this asynPrint down to line 2919 - just before we actually call H5Pset_istore_k() with the calculated (and checked) value. Otherwise we print a value which is potentially modified before using it...


You only have 6 outputs here!


you have to adjust png as well


Tests like this make me feel a whole lot happier. Those previous tests with partial stubbing were making my completely broken logic pass.


don't pass a lambda to map. in this case you could simply do `map(str, folder_ids)`.

but folder ids are ints - why convert them to strings at all?!


Скорее всего можно убрать в общий `try/catch`. И подумать как вообще хэндлить эксепшн в этом случае.


Разве это должен быть не `Xtext Nature`?


we have sugar for this one, too: `Attachment,find(..., _join=AttachmentFolder)`


Я бы написал наоборот:
`iExecutionEnvironment.getId().equals("JavaSE-1.8")`

Вообще `"JavaSE-1.8"` можно еще вынести в какую-нибудь константу.


Тоже надо вынести в константу.


`inputStream`

Я так понимаю, именно его нужно будет использовать, чтобы скопировать в эдитор модель.


As this is a custom set of finders in the user model (there's also `find_by_{api,rss}_key` etc.) with more than just a select wrapper, it doesn't really belong into this PR IMHO.


:laughing:


don't you want to cast to `Date` here?


Is this a typo? Does it really exists?


To much spam for logs :P


`session_ids = map(unicode, session_ids)`


This block is about the return value, not about the side effect, isn't it?

I'd say the `{ }` were correct.


Хочется разнести все это по методам: создание проекта, конфигурирование проекта, открытие эдитора модели, открытие эдитора пятнашек и т.п. И вынести в отдельный класс.


oops! fixed. thanks for catching that


Как-то так. Правда запустить `earlyInit` после `readCfg` не всегда получится, поскольку иногда чтение конфигов асинхронное и может так получиться, что конфиг будет прочтён после выполнения встроенных скриптов. Пока что не стал делать.


Seems that you could merge the code for generateCigarAgainstUpwardsRefrencePath and generate...DownwardsReferencePath, what are the differences?


enable to step with one leg


{} x 2


The attempt to prevent recursion backfires. Not only won't this prevent recursion, it will prevent backup delivery for every packet that passes here after the first one (when more than one packet gets delivered after the connection is closed, only the first packet gets handled by the backup deliverer). This causes https://igniterealtime.org/issues/browse/OF-903


Right, that is what I meant.


Can you be able to put fb_finfo inside VSCREENINFO? Doesn't it crash? They are different structures... Shouldn't it be fb_vinfo? You are modifying it, and also later fetching it again...


as noted on IRC, 290px seems to do the trick here.


you should also filter by link_type


cast to bool to compare with triple = ?!?  uhmm look quite too much work .. should be the same to simply do

`if ($var)`... don't you think?


You can combine this and the previous method using lambdas.


don't coerce to int here (same for contributions)


I will change all the Serial.print("static") to F("static") on this branch, Im sorry, I not controll very well Github yet... :P


why not use `form_rows()`? it also accepts `widget_attrs`...


Работает ведь только для трехзначного числа.
Для любого числа в пределах integer вариант с циклом:
var a, b, c: integer;
begin
  writeln ('Введите число '); readln(a);
  while a>0 do begin
    b:= a mod 10;
    c:= c+b;
    a:= a div 10;
    end;
   writeln (c);
end.


yes :) because we have our changes in our fork, and I think it is easier to cherry-pick your changes (I can choose which ones I like, though most of it gets to our fork)

our last change is integration with Virustotal


usually we only use a `SubmitField` if you have multiple submit buttons and want to know which one was used to submit the form


again, I'd say `{ }` was correct...


This seems dangerous. The entire point of evacuation is for the new instance to come up and be in the RUNNING state (and hence routable) before the evacuating instance is destroyed. Was this change in response to a specific test failure?


me i prefert to use FriendlyName since u can change it in "sevice--general"
if we use ProfileName we need to add the access to it somme where in the menu
and not everyone use profile


Hi @shelbys, I am trying to understand the reasoning behind this change (together with 741fb9f9). Could you please explain what issue is this fixing?

Since you made this change, swagger-ui has moved the image to CSS (see https://github.com/swagger-api/swagger-ui/commit/f8991b), thus it's rather difficult to build the URL dynamically.


Why not touch `format.json` too?


It's not an open source project without badges


the guest list challenge code is wrong. Could I please have the correct code


Ack. Thanks.


Die Verwendung von Abkürzungen ist für Properties nicht unbedingt empfehlenswert, insbesondere wenn sie ziemlich unbekannte Begriffe abkürzen. Aus dem Grund wird inzwischen die Benennung `foaf:mbox` auch von ursprünglichen Namensgebern für ungünstig gehalten. `ags` sehe ich heute auch kritisch.


`do ... end`?


Ack.


I saw that, interesting. I may port that one back from yours :) Stay in touch!


{} x 2.


your result above gets ignored.


hmm okay makes sense. I thought that because a) the comment was Ubuntu <= 15.04 and b) hekad didn't get started on my box that it might be a bug.

turns out that heka wasn't started because my heka configuration file was getting permissions errors. It attempts to create some log files in /var/log/heka and also some cache files for ElasticSearch in /var/cache/hekad/ but is failing because of the user https://github.com/nathwill/chef-hekad/commit/31ebe24a683028e3dc5661e8994fa76a08ee42da#diff-f85f866f4b835eac862f3f06d42d4a97R13


I found an issue in that change:
$INFO[Control.GetLabel(29)] shows different information according to the current visible view like Timeline-View or other


s/scale/scales/


profile_name is fine, just as long as it's clear in the field name that it's not the email address or uid that they're using to login with  :)


this could be `maybe def env <$> lookupEnv env`!


thanks fergie


Seems that you could use streams to construct the path but elements are inserted an the beginning; is there a collector that reverses the order?


@paulvi

Thank you for this update!
Yes, please share :)
I appreciate it.


I actually like explicit return statements.

If you asked me I would even write `return` in most of the cases... This might be my view alone, not sure...


simply doing `if ($var)` should be the same of casting to bool and then check explicitly against false ... don't you think?


Agreed.  :)


when I catch up with you, I'll be able to send pull requests, but it will not be anytime soon


I do not really get what you want to test in the ScalingCalculatorTest vs ScalingPropositionTest.


Without this structure i'm not able to install with cordova add plugin


Seriously ? True it's better in this case, to avoid broken secure links (as on the Prestashop forum...)


let's keep it like this for now; I think we refresh when entering the view as a side effect of calling clear() and I don't want that we call the same functions too much... likely we should throttle these functions a little more but this is for later.


it's not called "apply", right? What's the difference in scope between the this test and the ScalingCalculatorTest?


Second time lucky, @casey-collab


You can use streams for this I think.


Check on arguments?


So the parts the `sed` calls are replacing is based on the defaults in https://github.com/travis-ci/worker/blob/sol-pkgs-134/utils/pkg/templates/default_env_travis_enterprise and https://github.com/travis-ci/worker/blob/sol-pkgs-134/utils/pkg/templates/default_env_travis_worker, which end up in the packages on package cloud

Do you want those changes to be the defaults in the files in the packages?


On 7.7.2015 15:53, aveshagarwal wrote:
> It seems it still needs to be updated to the external repo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/miminar/atomic-enterprise-training/commit/108fde211fc129dc39727181dde77af69bd9733a#commitcomment-12039793>.
>
Ahh, this commit got there by an accident.


Hi,

Sorry but SSL can cover all the pages, or the order funnel only.

Regards


Strange, I don't know when I modified that.


js uses camelcase


`.hide()`


what is difference between session.config.platform ?


I'm not doing any kind of `clear_settings` when pushing to the index, so if the user defined `typoTolerance: false` in a previous push, it will stay as `false`. I thought reverting to a sane default would be what users expect.

(I actually got bit by that myself in my tests ;) )

But maybe if we go this way we need to be more exhaustive in the settings we're pushing.


please use snake_case


`is`?!?!?!


That's great, I've merged the previous commit, most (or all) should be yet
in place.

Dario.


see http://stackoverflow.com/questions/2854655/command-to-escape-a-string-in-bash


```
{% trans -%}
    The access...
{%- endtrans %}
```


why not put the class on the `<strong>` or style the `<span>`?


@jonmifsud @jurajkapsz thanks guys! Ping me for any problems.


If v is int, v.replace will fail with error: "AttributeError: 'int' object has no attribute 'replace'". This happens f.ex. when mount -module is given arguments "dump": 0, "passno": 2. The problem is preventable by quoting the numbers ie. making those strings.


Yes I saw this, this is why I opened a new branch, to remind me finish this job.


I think you mean all- __t__ san


However, ZipFileHandler uses `zipfile.ZipFile` internally.


`.children('option:selected')`


Add contributor's user profile to each PR.


IMHO this is a bit too generic for the global form util... the `aclIfProtected` ~~is~~ was quite generic and thus suitable for other objects we'll add in the future that use protection


I get the error:   File "rmgpy/molecule/molecule.py", line 1530, in rmgpy.molecule.molecule.Molecule.toRDKitMol (build/pyrex/rmgpy/molecule/molecule.c:27418)
Boost.Python.ArgumentError: Python argument types in
    rdkit.Chem.rdmolops.RemoveHs(Mol)
did not match C++ signature:
    RemoveHs(RDKit::ROMol mol, bool implicitOnly=False)

Is this because my RDKit is out of date?  I use RDKit_2013_06_1


This actually makes the line too long for rubocop (89 while max is 80)


We create the stdatomic.h file in the /usr/local/include
In that directory we have the polarssl and the OpenV2G we have successfully install
If i understand well from the error message there is something missing between the file we created and the files exists on the lib/gcc/arm-linux-gnueabihf/4.9/include.

Error messages: http://pastebin.com/dGeCzR4c


Any reasons why not load modules on centos?


yes, and it's a rather useless legacy wrapper that adds some useless clutter (indico_file.dat was a python 2.6 workaround for a bug where empty folders were not added properly)


Does this require a change to MPlot? My version here (up to date as of yesterday) doesn't show a MPlotDataPositionTool::units() method.


Why kmerSize is not declared within the ```for``` head? is never used outside the loop.


@benvanik @GamingDamned Confirmed, it runs with latest AMD Driver on W10

P.D. Error to load libturbobobadger.lib


yes, we only remove the `:doc` metadata if the positional docstring is provided.


Since generate_node_specs() return is useless please change it to void


Sorry, right, I should have been more clear on this.  Each individual issue would continue to have its own unique uuid, but each known type of issue would continue to have a single unique identifier that would stay consistent across all known instances of it being reported.  It could be a CVE number (CVE-2000-0211), MS number (MS00-013), uuid (fd257ec9-5d74-4a16-a6a2-c492ef03162a), or plaintext ("no-csp-header"), just as long as it is consistent across plugins.

If you have three different plugins reporting that a site is vulnerable to Heartbleed, you should be able to suppress all of them at once for a site, since they're all actually reporting the same issue.  If they all report it using "CVE-2014-0160" as their classifier, then that is easy to do.


Must be. The RDKit documentation doesn't say when that `sanitize=Bool` option was introduced, only that it currently exists.


ACK with comments


ah jeez, yeah, that makes sense. we switched to running heka under its own user a while back, and apparently that never happened with the sysv script? i know when we went from the heka-running-as-root version of the hekad cookbook to the heka-running-as-heka version, we ended up having to do a permissions hotfix on those directories in our wrapper, though I'm struggling to find the exact code we used at the moment...

We came up with something based on [this comment](https://tickets.opscode.com/browse/CHEF-690?focusedCommentId=41441&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-41441) at the time. It's a little nutty, so definitely tread lightly and test thoroughly if you decide to go that way.


> That's outdated. You can either submit this as is and I'll adapt or you could rephrase this section to use @EnableRabbit. The feature and code are very similar so if you could use the same structure that'd help.

Sorry I had not followed the @EnableRabbit usage. Would you mind making the necessary adjustments?

> Either way, make sure to sign the CLA when you submit the PR

I already submitted the PR and signed the CLA. I need to send the CLA number?


top


I was asking @nh13 but I think it is ok now.


Is this a perfect oportunity to use stream's flatMap?


@oliward Also this doesn't seem to work, can you help me with where I am going wrong ?


Remove this line?


:joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy: :joy:


Probably a lot faster and simpler than running up another service and attempting to integrate with it.


arguably this comment wanted to make a point...

Was that an automatic removal by rubocop? Would not have thought that it removes comments on its own...


Erf...
The only important condition is:
if ($relative_protocol)
			$base = '//'.($ssl && $this->ssl_enable ? $shop->domain_ssl : $shop->domain);
		else
			$base = (($ssl && $this->ssl_enable) ? 'https://'.$shop->domain_ssl : 'http://'.$shop->domain);

So, if you force to null, cms pages will never be in https...


Opa, perdão @juarezpaf, não me atentei pra esse aspecto! Terei mais cuidado na próxima. :)


А точно вот это надо?


`do ... end`?


I have a problem with that, model.Name is null
And I can't get IdRecruiter


missing spaces inside `do ... end`


Why? Without the coercion, form will not validate.


Is it a list or a dict?


Может быть `must join files` ?


"german" -> German!


@nttaylor @kfogel It worked for me using Couch yesterday, but I'll merge in master and get the branch working with Postgresql for Karl.  Then we can merge to master.


Are you on 2015_03_1?


Right, filter by attachments that are not links.


Да, иначе ниже будет `false` превращаться в строку `false`.


Without that, `get_objhect_from_args` will fail.


Fixed this in ab597e


Ah, great.


Да, исправлю в следующем PR, спасибо.


Think so. Whatever was on the RDKit anaconda channel a few weeks ago


Close  #4


You shouldn't use that at all since you only deal with events here...


TeamCity OsmSharp :: GitHub - OsmSharp [Build 1037](http://build.osmsharp.com:8080/viewLog.html?buildId=5052&buildTypeId=bt16) is now running


Maybe you are still passing ints instead of strings in `choices`?


it was the parent commit (https://github.com/open-multinet/playground-rspecs-ontology/commit/d8192f716528e1535fb35c1335923244b44c4150#diff-ce696e114e021943cff2c2a96737de8cR2), but its fixed in 238844a6ac0056ad2e70fa5ee717a06c5fc74f0b


Checking for browsers is generally A Bad Thing (TM) as it will likely cause this to break in the future when browsers may differ. Really, we should check for features (or lack of). What was the bug that was being fixed?


We're all good. I am going to take it over. Thanks again!


Is this placement an issue ?


понял, спасибо


Well sorry not according to our tests.

Regards


오... 멋지네요. 헤로쿠에 올려주세요~ 직접 사용해보고 싶습니다 ㅎㅎ


Hello John!

Our primary argument against `let` is the difficulty in reading and debugging a test. Say, your last example fails -- and I'm the one debugging it, and I've never seen the code before. I look at:
```rb
expect(subject).to eql(false)
```
"Okay, what is subject?"
I look 14 (or so) lines up.
"Ah, it's `policy.allowed?`. Okay, how was `policy` created and with what attributes?"
I look a few more lines up and see what `allowed_report_ids` were used, but then I need to jump down to the context of the given example to see what `user_report_id` is. I've done a lot of jumping around just to discover what the test fixture (or test context) is, in order to try and debug it. If the example is more complex, or `context`s are more deeply nested, that increases amount of jumping and the cognitive load (of state to keep in one's head).

In the example that thoughtbot recommends. The varying test fixture is right there close to the expectation. If needed, I only need to make one jump into `build_policy` to figure out what it's doing -- but that is almost the same as a factory.

> I've found it to be invaluable in "proving" a context does what it says it does

I don't see how thoughtbot's example doesn't prove a context does what it says. `context` is semantics to help describe what is encapsulated in the block.

> Why not at least properly use `let` in the "bad" example so people can see a real comparison?

This is a good point. We'd welcome a PR :smile:.


the badges feels very big for error flooding in the range between 1-99.
The thing that gives me 1519 errors in a row on my system is avahi, but I feel something else is broken there.
If we set
```
float: right;
```
there instead of the width, will that cause performance issues?


why ?


In this commit there is a missing comma. Please run `make up-to-date-std-lib` from your Agda repos (master and maint) for fixing the problem.


Niṭ: m_tabDd => m_tabId in commit message.


TeamCity Jarvis Framework :: CI-GitFlow [Build 20](http://build.prxm.it:8811/viewLog.html?buildId=3383&buildTypeId=CqrsFramework_CiGitFlow) is now running


oops


Sorry , Can I ask an question?
I use you code to build mtrace and run the mtrace-linux , there are some error like "mtrace_entry_register: mtrace_host_addr failed (XXX)" and "bad mtrace_entry_host type  abort()".
Finally , I find out what lead these problem.  If add "double timestamp" into the "struct mtrace_entry_header",something will go wrong,but I do not know why. Can you tell me why you can success or how Can I fix these?


fixed #31


Why change from .lower(). I thought maybe it shouldn't crash out if someone uses "pgen list Targets"


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


@MoOx that works perfectly! thanks


Изменения касаются лишь строки 518. На 87 строку ругался codesniffer.


MediaFire huh :stuck_out_tongue:

Would love to see patches tied to github releases (like I said before on reddit :wink: )

But anything's better than SF! :grin:


Yeah, generally the more I can do dynamically at query time the better I feel. I think the plugin should focus on overriding what is really related to it but I would not set any query expansions/approximation parameters.


It seems that it uses the wrong stdatomic.h still, you could try to replace it or change in the include order in the path. (Remember to backup the old stdatomic.h in any case)


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.1.0-repo_concurrent_saga+1](http://build.prxm.it:8811/viewLog.html?buildId=3383&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests passed: 111; exit code 1 (new) Build time: 00:02:24


@mcorkum I like the way you think


那么怎么判断补语呢？


Corrected #1


Cool. I will push that into the plugin.


Arg :+1: I'm used to use 100 instead of 80 since a few years (even 120 in Java) because my screens are wider. You can definitely rollback to 80 chars, fair enough :)


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.4.0.1037](http://build.osmsharp.com:8080/viewLog.html?buildId=5052&buildTypeId=bt16) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 541 Build time: 0:0:0
### Failed tests
```
OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.Serialization.RoutingSerializationTests.RoutingSerialization: Test(s) failed. System.OutOfMemoryException : Exception of type 'System.OutOfMemoryException' was thrown.
   at System.IO.MemoryStream.set_Capacity(Int32 value)
   at System.IO.MemoryStream.EnsureCapacity(Int32 value)

```


I don't know if this is a useful comment


:hankey: 程式碼不 merge，修正之後再重發 PR


补语的词类最终没有写出来啊？


Returning an actual `response_promise` here is misleading. We need a return type to indicate that this has been properly forwarded in typed actors, but we must make sure users don't actually use this return type for anything else.

Either we "misuse" the promise for the sake of type checking (what this code basically does) or we introduce a new type that models this. The "clean and sane" solution is the latter, so I'd go with that.

This function could return a new `delegated<...>` (empty) struct that passes `detail::ctm_cmp` in the same way `typed_response_promise` does. The name also makes the intent clear: this is just telling the runtime of CAF that the typed actor has delegated the response to someone else.


This will probably break things for most people who need to upgrade.  I guess it had to be done sometime


loopback-explorer handles multi-line values OOTB and joins them in the Swagger spec, see https://github.com/strongloop/loopback-explorer/pull/59. I think it's safe to leave this change out. @shelbys thoughts?


@Eolia from what I see in code CMS will be SSL if force ssl is enabled because condition in getBaseLink


Looks good Mooneer, thanks!


proyecto final


Bonus points for omitting the Array brackets (`[]`) ;)


Yeah, I had a similar thought, but I'm really hesitant to write versions that replace rather than append, just because it's a weird departure from most of the other strbuf functions. There's also no strbuf version of `mkpath`, though we could add one (TBH, I find `mkpath` kind of gross in the first place; it is basically string formatting, but it will magically eat "./" from the beginning of a path.  Which it seems like only a tiny minority of callers care about. I think its main use is that it predates strbuf and `xstrfmt`, and its static buffers let you avoid dealing with memory management).


in getBaseLink:
$ssl && $this->ssl_enable  <- 2 conditions

not only $ssl...
but if $ssl is forced to null or false, theses pages will never be in secure mode.


Surely there is a better way to do that…


SQL import check by travis is the feature of the decade!


No problem! My test Arduino running this code (w/the leap second test code
I talked about before) is apparently stuck on 14:03 UTC though, I wonder
why.

On Tue, Jul 7, 2015 at 7:39 AM, Dan Quigley <notifications@github.com>
wrote:

> Looks good Mooneer, thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tmiw/arduino-ntpd/commit/0940a8a5d50cd9ce021760ab9da4f6079de4ad45#commitcomment-12040880>
> .
>


@turboevil Да не за что) Админку будете пилить?


you probably want hem ordered by "age"


Can you post on the forums with specific examples?  I cannot duplicate this issue.


What means Geotrek-admin > V2 ?
Actual version is V0.34.
Which is the minimum needed ?
Thanks.


@sokra could you please cut a new release for this? thanks!


I re-added the first sentence now; you're right that that's important.

It doesn't seem important to me to discuss the SESE property here.


Тогда зачем это все?


@SpaHost sorry about that - just pushed a fix to remove that error.


lgtm :+1:


It makes sense to me; at the time, this function was not intended to be used elsewhere, and I was just copying the old behavior.


oh yes indeed, i'm on it


Thanks :)


Why you do that?!!


:santa:


you said you stole 960, then `jewels_in_bag` is `100?`


Yes, ideally if a file defines a class, it shouldn't perform other logic as well. I've moved that to the constructor, but maybe it would be better in the sign() method?