labofsahil/github-event-dataset-2012 huggingface dataset:
Interesting technique, but probably the result is not as fast as if you just let e.g. `n_threads.x` be the warp or half-warp size (32 or 16) and then just cut `n_threads.y` down to fit.  (But In that case you would want to make `i` the inner loop variable.) The goal would be to set up contiguous reads  and writes of the size preferred by the hardware.
Yes, that makes a lot of sense. Thanks :) I've added the change.
Is this width=800 height=800 really necessary? Looks fugly.
Note use of the bizstrap "form-actions" class which is built into bootstrap
Okay, above diff broke test :)

Let me mull this one over.

Personal note:

- Migrations is not always needed
- So does Extensions
- Investigate whether we can load them only when request is in /admin prefix?
I'd suggest doing a find/replace on "data set" -> dataset.  the docs are consistent between the two
The wiki entry probably using tabs.
So I suppose this is gonna be added back?
remove the word useless
It's from https://github.com/haf/gollum/blob/feature/gitcode/test/test_markup.rb

Will have to check to make sure it works on 1.8.7.
for windows. seems we could only copy from one folder each time
This is all moved to string_intern.h, along with some other helper functions that should not be exported and should not pollute the global namespace.
Also, `path` is a really common variable name that's stupid to spend on an import.
So two (or more) Homebrew processes can't try to install the same formula concurrently, or uninstall deps of something that is being installed, or link/unlink at the same time, etc.
This has been as-is for a long time. In that case we just manually specify another build type (or fix the Eigen build system as this should not break it).
Aye but won't this happen for all future versions of the CLT (unless we have a version somewhere)?
On 29 Aug 2012, at 4:30 AM, Jack Nagel wrote:

> Sucks that it has to be run as root just to dump the help text

Indeed it does.

I’ll make the necessary changes now and commit.

Then I’ll start looking at an elegant way to test it for later update to the formula file.
wenn in einer solchen Tabelle eine Warnung und/oder Error drinne steht sollte im table title das auch ersichtlich sein ohne aufklappen.. das seh ich auch so.

Das sollte aber bei SQL und EPs so sein.

Vllt macht es auch sinn diese firebug table in einer separaten klasse dann zu realisieren...?
this line is unnecessary?
I'm not talking about our debugging output in qml. I 'm talking about normal qml  thrown errors. You'll never see them if vibra is on. But don't resubmit the pull request for this, I'll update it this time.
Can you include the abstract-avoiding behavior in the javadoc?
If we just create one file, I'm OK with that.  Thanks.
Yes, it seems a better solution. And I don't like to call `replaceAll()` and `format()` every time `getName()` is invoked. I will fix this, too.
Need to update the message to FileViewController.setFileSelectionFocus.
Let's use some more generic wording like this: `"For more information, see <a class=\"clickable-link\" data-href=\"{0}\">Troubleshooting Live Development connection errors</a>."` -- that way we don't have to deal with the fact that it might or might not be a wiki page in the future, might have info for things other than Chrome, etc.
Done. I also fixed tests for error message wording change.
There are a couple problems with dropping this code in here:

* The var commandID is not actually a command ID (the Command.id field) -- it's an internal MenuItem id (the MenuItem.id field).  Should be renamed.
* There's some unneeded code here:
    * the DIVIDER case is unneeded (it won't work anyway since divider IDs are all uniquely generated)
    * there's no need to go through menuItemMap since `this.id + "-" + command` is the same thing as `menuItemMap[this.id + "-" + command].id`.
* This is somewhat in conflict with the code in Menu._getRelativeMenuItem(), which maps a command ID to a MenuItem in a much more roundabout way.  Could you create a new utility function (e.g. "Menu._menuItemForCommandID()") that is called by both these methods?
Re doSequentially(): you can still attach listeners to each individual Deferred before returning it from beginProcessItem()... and then either push each doc onto a list as they get resolved, or add them at the right index if you want to keep the order the same.  See the code around checkOneFile() DocumentManager for an example of the latter.
Why aren't you passing previous editor here?
I'm not sure I know what you mean. Would mouseenter be a better event to listen to?
Another option is to move the call to _fireSelectionChanged() into the _handleDocumentSelectionChange() function to make the event binding cleaner,
If you leave it as a function, it'd be nice to rename it `get_labels` so it's more obviously callable.
Me, too. Will be changed.
I wouldn't recommend throwing an exception if you're planning to catch it immediately in the same method. You could just as easily do what you need to without the exception.
What is the goal of the interface if we're stuck to its implementation ?
this could be written on a single line with String concatenation (no need to `format` something so trivial)
I hope that is OK now...
*This view logs ...*, be consistent ...
This is *awesome*.
Top 5 lines should be aligned with target.
add a link to irc://irc.mozilla.org/popcorn
These abbreviated param names are unreadable. Either spell them out fully, or name them "unused".

I think the stub itself is fine, but it would help if you briefly describe at a high level what this stub is doing, and why it's necessary.
What is being returned from form_valid()?
technically, some BIO cleanup could be done, but since the process is about to go away, it's not really needed.
You can just add this to localResources
I'm not sure I understand why this is needed
The thing is nowadays spammers that want to spam your forum will suceed whether you allowed unlogged user collaboration or not. If you take stackoverflow, they allow such interaction with a simple captcha protection.

But in my opinion forums are outdated anyway, I think you should have focused on a Stackoverflow clone instead of yet another forum. Even if I think the purpose of this project is to show good coding practices instead of having using it.
But in this context secpolicy_vnode_setids_setgids should not check if the current user can create a setgid of the requested gid? (checking vap->va_gid, which should be different for example when we are creating a file inside a setguid directory? See zpl_vap_init in zpl_inode.c)
The current user is always a groupmember of the current gid, so secpolicy_vnode_setids_setgids is always going to return 0.
I've tried to exploit this,unsuccessfully. I *think* that Linux VFS shields against this problem, as of now.
what this # EOF mean?
I would expect adding default parameter to katello_config_value (if its not present already) so the code would became:

    katello_config_value("foreman_web_workers", katello_process_count(0.4))

Otherwise ACK
No, this one is fine.  Declarations with initializers are still declarations.
You've not replied to my question here? I would like to avoid the need to run the make target.
getpwuid stuff is not supported on Window...
I haven't gotten in the habit of `@property.setter` but I will, that looks nicer.
Likewise, this should be a `DistutilsFileError`.
/cc @Vagabond
See #2065
Why is index generation going through name(). It seems like this should be done as part of buildIndex() instead of modifying name().  name() gets called 100's of times per request.  We should be running as little code as possible there.
I didn't read the function code earlier just saw the docblock. An empty array would mean no value being set for destination directory so this change does make sense.
tell me what level of php-cs-fixer are you applying?,  our === doctrine?
Ik vind van niet; maar het staat op de manier dat jij zegt in de stijlregels :p
Single line comments do not start with a capital letter.
A `save_a_copy(newname)` maybe with a version that have a configurable template that take parameter like (`%{oldname}, `%{date}` ....).
This technically isn't valid ics, ATTACH is only valid on a VEVENT or similar.
Not sure how *that* happened... Fixed!
yes, the main title is translated in the FAQ page, I will remove the first argument
oh sweet!
@peterbe I mean I have the work done in one of my branches. I'm waiting for this one to go before I push my changes, to free you from a complex merge. (That branch is a lot bigger than mine, it will be much simpler for me to rebase. )
This is odd. Does this HAVE to be done *after* the `with` block? Surely the same result, in terms of Python, is achieved from doing:
```
with ...:
    try:
       result = stuff()
       connection.commit()
       return result
    except:
        ...
        raise
```

That flow looks more readable to me. Could be entirely subjective but it still makes me think why the return statement is after the with block. As if it matters.
there is a `{% trans %}`. @ednapiranha just likes to have localizers translate tags.
Are these the following (in terms of the ontology)?

* ModelBijectiveMapping
* ModelExpansiveMapping
* FaithfulMapping

BTW do these have to be integers, or can we (at least on the Ruby side) use enum types for clarity and safety?
I switched it to `andalso`.
Right. Shit, I haven't looked at this code in ages. Going to merge & deploy. Thanks!
IMO, you should wrap it in a CachedReader using an ArrayCache when building the default reader. It does not make any assumption about the setup (PHP always allow you to create an array) and has better performances as it does not read the same annotations several times (people wanting to use the ApcCache to cache them across requests would of course need to pass the reader themselves)
I thought I good get rid of the && with an high level language :) At least with the 'and' your are not likely to put a '&' by mistake... But if it's standard thanks for the tips :+1:
fixed in everzet/symfony#3
Agreed, I change this ...
Missing space after `(bool)`
Ah right, forgot this method accepts also objects... =)
getParent could be null at this point
Fancy unifying this with its demorganized equivalent above?
这个item确实不太需要。我也越发觉得我写的像是一个入门文档了，汗，这真不是我的初衷呀。
我尽量把论文需要的那些基本知识放进去，别的略高级深入的用法，再看情况处理吧。
It was logging an error every time it timed out, so basically every 5 seconds (or whatever the timeout was). This solution didn't do that and seems to make sense.
It is necessary for the PLASMASHOP_VERSION macro which is added to the decompiled output.
"flashes" sind werte, die fuer einen Request lang in der Session gespeichert werden. Sie sind also vor allem fuer Fehlermeldungen etc. gemacht. Das FOSUserBundle nutzt sie genau dafuer. Mit jQueryUI zeige ich sie hier in einem Dialog an. Liesse sich also auch fuer alle eigenen Meldungen wiederverwenden. Dazu einfach

$request->getSession()->setFlash('key', 'value');

ausfuehren.
Maybe expose a PROJ4_VERSION here. I don't think a regex is needed here:

```
>>> cartopy._crs.PROJ4_RELEASE.split(',')[0].rsplit(' ', 1)[1]
'4.8.0'
```

or simply:

```
>>> cartopy._crs.PROJ4_RELEASE.split(',')[0][5:]
'4.8.0'
```

Maybe even provide an easy way to get the major, minor and point release numbers too: 

```
>>> PROJ_VERSION_TUPLE = cartopy._crs.PROJ4_RELEASE.split(',')[0][5:].split('.')
['4', '8', '0']
```

Finally, I would probably wrap this in a try except in case there are some development practices in the proj4 community of having non integer releases for development branches...
This bit isn't necessary, but causes no harm.
Right. :ok_hand:
`value === void 0 || value === null` can be changed to simply `value == null`
Woah - actually just realized that this logic is in the wrong place entirely. This will also apply to single selects (adding default text instead of giving you an empty search field).
I had to think for a second to get it but I like it, I'm just trying to think if there's anything that will break it? As long as all the keys in the hash map directly to parameters in the Resource, this will work and it's flexible enough as we make improvements to the lxc_container Resource.

I'm not sure about the container.has_key?(:action), that seems like a lot of meta in the hash and I'd just as soon leave actions out of the hash.
This problem was fixed in maven-compiler-plugin which we upgraded recently. so fork is no longer needed
This resets the run, not the entire job.
Maybe add a sentence about what happens if one calls `playable()` on a non-linked track?
We should not use `br` for creating visual space that should be done with `margin` or `padding`, BTW this tag accepts autoclose
nit: promtps -> prompts
settins ->settings
Can you turn 100 into a constant and add the comment relative to bug 8080231 next to it instead of duplicating it.
It would be nice to add a name to this function as well :)
This removal is causing issues:
* the contact creation `+` button gets on the left of the toolbar instead of the right
* the `Done` button on the contact creation page overlaps the `x` button on the left of the toolbar instead of being on the right
Also remove this line ... let SettingsListener update UI
Ok, I'll refine it or implement the layout of Clock according the latest spec.
???

L171 - 174 should be simply `return this.alarmList[id] || null;`
Yes, but that is another PR :D, let's keep working step by step, it will be too much changes.
`return null;`
I don't think this is good for the following localization.
May need to consult l10n expert, like Kaze.
Should be
// XXX: Before https://bugzilla.mozilla.org/show_bug.cgi?id=816452 is confirmed .....
I use window.parent.FakeAlarmManager.snoozeHandler().
Then I got "JavaScript Error: "TypeError: window.parent.FakeAlarmManager is undefined" "

And I try to use window.opener.FakeAlarmManager.snoozeHandler()
Then I got "JavaScript Error: "TypeError: window.opener is null" ".
I'm not sure what I missed the opener page.
pTotal -> currentTotal

And congrats for yesterday result!
Please remove dead code.  Or if there is a reason to do that, make a comment.
This means that if a hardware back button exists, it will no longer navigate the browser back. @lco is this the desired behaviour?
This doesn't look really good for l10n purposes, but we can follow up with the l10n commits
Try to follow pep8 style rules in your code, e.g. http://www.python.org/dev/peps/pep-0008/

You can run the "pep8" test on datasets.py, e.g. go to the directory where "datasets.py" is stored, run:
      % pep8 datasets.py
and then fix it until no violations are detected.
we should eventually generalize the first parameter to this to just (id), since we could also pass teacher or student
Unfortunately this code doesn't work if user hasn't posted anything.

I think that the real issue is in counting user posts -- post in hold shouldn't affect users post count.
On 09.06.2012 [09:52:24 -0700], Lucas Meneghel Rodrigues wrote:
> > @@ -74,7 +84,11 @@ def machine_install(self, profile=None, timeout=None):
> >          @param profile: Profile name inside the install server database.
> >          """
> >          server_info = get_install_server_info()
> > -        if server_info.get('xmlrpc_url', None) is not None:
> > +        if install_server_is_configured():
> > +            if profile is None:
> > +                profile = self.profile
> > +            if profile is None or profile == 'Do_not_install':
> > +                return
> 
> The later. Pull this patchset, rebase and fold it together and let's
> produce a whole patchset for upstream inclusion. We might need to
> repeat the process a few times, but that's OK.

Great, I will do that and put in the unit tests at the appropriate
spots.

-- 
Nishanth Aravamudan <nacc@us.ibm.com>
IBM Linux Technology Center
The cnfProject != null I believe upon review is not required.
dup log.msg
I personally prefer using dict.get instead of this kind of if/else. Removing if/else makes the control flow simpler to me ... but maybe others disagree:
   ss_repository = cb_info.get('repository', repository)
Technically, I think it's faster as well, since the dict lookup only occurs once (rather than twice for the 'in' and then the 'get')
Did you test this in Safari? I recall needing the *. at the beginning of the string if there is not a subdomain.
Should use static here?
Missing empty line here
%u expects a 32bit destination
filter return true means `DO NOT CONTINU`
you check for project description while rendering issue description. its logic issue :). I'll fix it
It's the lowercase x in 2X.
doesn't look like you do this elsewhere, so are the counter threads no longer in the management set?
Probably want a SizedQUeue here
This only exists in two functions, one is `getItemWorkerForWindow` which also creates a worker if one doesn't exist and `destroyItemWorkerForWindow`.
typo: tranIStion => tranSItion
I think the browserid router process is where we'd want to round, not in frontend code nor in kpi servers.

make sense?
transitionNoPassword has a test, sadly this does not.
good point, yeah.  OK.  I moved BATCH_NAMES
I don't think this should be moved; it's there in case something like ``jsonapi.dumps(msg)`` in ``reserialize_reply()`` fails.
Did something slightly different because this generates a random directed graph.
Don't know yet if options should be case sensitive or not, camelCase or underscore_separated...
We have that rule throughout the code that setter & accessor methods go together, and setter goes first.  So why this small CS fix is bad?
I'm not sure but I think that this function is for manipulate URI parts and not filesystem paths. So in this case you must use slashes
I see now that code we added elsewhere does use apr_strnatcasecmp, so yes, we should have used that here, too.  Thanks for spotting this.
yes, but for other methods in this class in doc block we have a full path. Again it is better for IDE. For example, PHPStorm has a great modue for static code analyzing. But when we use relative path in doc block it doesn't understand type of return value.
you should make the relation extra_lazy instead of lazy (which is unfortunately the default for BC reasons as extra_lazy is a doctrine 2.1 feature), to avoid loading all the collection when you only need one page.
@jeffhandley why?
ほげ++
The semantics seem a little strange. Also, I'm not sure this is the right way to approach this functionality.
please fix the CS. The visibility is missing.
We should go with Erlang philosophy here. If the argument is a binary, we return a binary. If a char list, we return a char list.
EMailAddressValidatorTest
Same remark for later regarding service tag
it's code and terminology from the SCOOP code. i just added a few custom entries.
i'll try to get these back into the SCOOP code, to get rid of it altogether
Mike, without being very insistent (but answering an earlier request from Pete) - here's an example of what I was talking about: http://en.wikipedia.org/wiki/Representational_state_transfer#RESTful_web_services (which in turn quotes Sam Ruby's book http://shop.oreilly.com/product/9780596529260.do).
If you indent this by 4 spaces, it will show a code block
This test is needed because the config file may overwrite it to set it to None.  This is necessary because the configuration file is stored in a location which is not writable by all users, and build users are isolated and do not necessarily have a home directory.

In addition, the sanity is likely to be tested at install time, and it would be best to avoid doing it for each package compilation.
1. @zacc's back!
2. Is there a preference (maybe for performance reasons) to always start css selectors with an ID? If so I can add one, but both of those selectors work as is. The one on line 13 was copied directly from the pervious version, and the one on line 14 is just looking for any ```source``` tags which follow ```video``` tags.
No... its used in the map right below this function definition.
Done, thnx!
Same reason as above
Where do we delete days from the trades?
^ yep
1) That catalog does not seem to reload the actual page when loading items. I have not looked very closely, but if the page does not fire the `ZoteroItemUpdated` event (http://www.zotero.org/support/dev/exposing_metadata#force_zotero_to_refresh_metadata), translators might not work very well on that system.

2) It looks like the library serves MARC data, which Zotero can import very well. It might be worth trying to use that data instead of scraping the page directly.
Anyone know if there is a good way to MathJax the NUM_1 here, and not the "is"? Also, if you could remove the extra spaces here, that would be great.
Woah. I was joking. I didn't realize these actually existed! TIL.
yes. work in progress.
Replace exit with return.
All manufacturer logos will have the naming convention (format {vendor id}_sm.gif  ). Thanks!
This should probably search for `$EDITOR`, `$VISUAL`, and maybe even `$BERKSHELF_EDITOR`
I changed it to n3_temp.close. There is an interaction where if a file is still opened via Tempfile and you try to read from it (i.e., curl -T file), the file contents will be empty. Tempfile.close or Tempfile.rewind will ensure the seek is at 0 when the file is opened or read from again.
I still don't understand why "unique" isn't last, then.
I think this could interfere with backwards compatibility, as above its align: right,

Also people could be depending on the floating and width,
It's preferable not to include trailing commas here.
Of course my preference would go out to #1, but #2 I will be happy with as well..
graph::Adapter --- и чего же он адаптер?
что за данные?
Right now the split is:

* Page = URL
* Template = On-disk File

So the same template, could be used in multiple pages depending on Proxies.

I think I have a bad Java-y streak where I use classes so it's easier to differentiate stuff in debugging. Been doing Clojure recently and re-evaluating that. Maybe data is just data and we add methods to work with it :)
i'd prefer:

if (!file.exists())
    return;

if (file.size() == 0 || file.size() > FILE_MAX_SIZE)
    return;

that way it's clear what conditions you're checking, and adding more of them won't turn that if () into something more massive over time
This one should be indented one step to the left since it keeps track on which article you have handled. It should increment even if we did not create an directory item.
Not sure you would like me to fix that. Should I create a scss file in the image gem with the fix?
Note: Once the config administration settles down a bit, we should probably have the default locale coming from there as well.
Indentation here and in the elseif
Same thing here w/ the `stream` variable. should be `writeStream`
In your own words, you can mention generally that there are several useful improvements for vocabularies.
Can you be more specific about your use case where inNodes is a NodeList. When I wrote this method, I assumed the input would always be the result of getDistributedNodes(). Does that method return a NodeList under native?
Are all of these used?
Hmmm. JS is vaag. Ik ga ff kijken hoe dat werkt
маленькая поправочка.почему бы не использовать do while,вместо простого while.в этом случае не нужен будет лишний повторяющийся оператор на строке 22. это же относится и к следующему while и повторяющемуся оператору на строке 30
В задании сказано, что метод mutate получает переменную типа int
Вы ничего не сэкономили - каждый раз возвращаете новый объект. Используйте спецификатор static.

(Если захотите хранить NSArray в статической переменной - обратите внимание на метод класса - (void) initialize. Но здесь лучше использовать обычный массив C)
Был добавлен класс для гена (т.к. это отдельная сущость), и категория printer для печати клетки. Функция  mutate для клетки мутирует клетку, mutate для гена - мутирует отдельный  ген. 
Функция mutate для клетки  работает следующим образом:
- делается копия массива генов
- мутируется ген
- мутировавший ген удаляется из массива, чтобы не быть выбранным снова
Поскольку ген - это объект и в массивах хранится ссылка на него, то нет разницы из какого массива его мутировать. Ссылка везде показывает на тот же самый ген.

То, что расстояние Хэмминга может быть одинаковым после мутации, - это нормально, вот пример мутации (0,1) :
AT-GC => AT-TC, расстояние осталось таким же (2)

То, что программа выдаёт одинаковый результат, вызвано деталями реализации функции rand(). Можно в начале программы менять "затравку" генератора случайных чисел (например, от времени)
Лыжы не едут)
Долго думал - не додумал, подскажите при каких условия будет неверное значение.
Вот код для теста:
```        for (int proc=0; proc<100; proc++) {
            int a = DNA_LENGHT * proc / 100;    //Моя реализация
            int b = round(DNA_LENGHT * proc / 100);    //с раунд
            int c = (float)DNA_LENGHT / 100 * proc;    // подсмотрел у l0rda
            NSLog(@"%i, %i, %i", a, b, c);
        }```
Выдает одинаковые значения для диапазона процентов от 0 до 100/
У вас, похоже, массив changedPositions инициализируется пустой, видимо, надо initWithCapacity:maxItemsNumber. А в цикле while добавлять к нему элементы с индексом position. Иначе получается, что в цикл while никогда не будет ни одного вхождения, потому как в нем нет элементов.
Please use internationalization here : https://github.com/MarkUsProject/Markus/wiki/Internationalization
The Mac buildInstaller.sh assumes it's run from installer/mac -- see the last lines of build_mac.sh.
Ah sorry about that, didn't pay close enough attention to your style. Will do.
Filed as #160.
Touché ! Even if for now I have not exported any password for the plugins I use. I will edit my pull request later to allow what you suggest.
Thanks again for watching my back! 
On a less mositive note, this also means I have to review all the potentially wrongly chosen IntPtr in the NativeMethods type :)
There is no need for the recursive step. Please remove it and remove the "haveJson" function.
no need for those to be on the same line, please put them on separate lines.
I stopped using this, it can be removed.
as above, this should be T extends OpenmrsMetadata
Wow, I didn't even notice that, you are right.
Thanks for the help.

The 2 commits were squashed.

When it comes to the parenthesis, I just followed the style just few lines above, at line 165, in the same file. :)

I have been using play frameworks for a year and love it. And I am happy to contribute to them :)
There should be a blank line between the previous line and method definitions.
I am having trouble fully testing logNormal distribution. Does anyone have better ideas?
@WouterJ  done, thanks!
We might need a small note here to make sure that it's clear that `my_command` is the "name" used inside the `MyCommand` below.
`totalEntries`
Yeah. I actually just wanted to use `each_line` but then there is the 1.8.7 (uses `each`) vs 1.9.x compatibility issue. So I used `\n`. But I think you're right.
The example should possibly avoid blocking, as users generally mimic the documentation.
Nope, it only includes those items which are constructed and ready at the point when the default dispatcher is created; settings were the last omission.
Точки в конце фраз.
Síp, me quedó de la implementación anterior. Ya lo arreglo. Gracias.
Keep in mind these functions need to be generic - useful to anyone who wants to add subscriptions, not just for one particular test.

This function should add multiple subscriptions, not limited to one.

something like

```lisp
(defn add-substo-activation-key
  "Add subscriptions to activation key."
  [name subscriptions]
  (navigate :named-activation-key-page {:activation-key-name name})
  (browser click :available-subscriptions)
  (doseq [subscription subscriptions]
    (browser click (locators/subscription-checkbox subscription)))
  (browser click :add-substo-act-key)
  (notification/check-for-success))
```
je sais mais c'est un update à partir de 3.7 donc on ne doit pas reformater pour éviter que le patch concerne trop de lignes. 
enfin si j'ai bien compris mais rien n'est moins sûr... 





----- Mail original -----

De: "Nicolas Le Goff" <notifications@github.com> 
À: "alchemy-fr/Phraseanet" <Phraseanet@noreply.github.com> 
Cc: "jygaulier" <gaulier@alchemy.fr> 
Envoyé: Mardi 30 Octobre 2012 16:56:09 
Objet: Re: [Phraseanet] 3.7 sig ign (#256) 


In www/xmlhttp/deletecandidates.j.php: >      } catch (Exception $e) {
>          continue;
>      }
>  
> -    $domth = $databox->get_dom_thesaurus();
> -    $domct = $databox->get_dom_cterms();
> -
> -    if ( ! $domth || ! $domct)
> +    if ( ! $tsbas[$ksbas]['domct']) { 
Unary operator are no more wrapped by spaces 
Check your formatting options in netbeans 
if ( ! $tsbas [ $ksbas ][ 'domct' ]) { 
— 
Reply to this email directly or view it on GitHub .
Installing Whisper and Carbon via PIP introduces a bug where whisper is unable to create files (unless the pip package has been updated very recently).

https://github.com/graphite-project/whisper/issues/12
Ok, I didn't know this feature. Thanks for the doc ;)
Sorry! Fixed in #1201.
Right. Also, we're only doing the calculation once that way. I'm not sure if the python interpreter is clever enough to optimise it out as a constant for us. It's microseconds so probably no worth it for solely that reason. The readability will improve, though.
Why the translation?
indentation doesn't actually exist in french. And it wasn't Google Translate.
i would rather use the word "indentation" or "retrait" as you used in this document rather than "espacement"
No changes here except sorted lines.
You don't need the case statement, this can be simply:

    ensure => $ensure. 

ensure => present will result a file because the content attribute, ensure => absent will behave as expected.

Please refactor and update puppet-rspec as well.
Why all the spacing between the variables here.
"put it _IN_ the game data files directory"
Sorry, read a bit more and realized that KML takes more timezones than just UTC, my bad. I'm not sure what to think now...
Templates should probably not have display:none anymore, as they are already in comments.
please also everywhere limit line width to 80 chars
@chillu Yep, I'll extract it somehow once the final solution is developed when I will be doing the next rc squash.
It should be if  (row) {
Default db value might be 1 - but clearly thats no always set on upgrades.
So many users WILL have form_id set to 0 in the db. You need to support
this.
I'm fairly sure that logic elsewhere just assumes 0 means default form.

On Fri, Nov 30, 2012 at 1:02 AM, Angela Oduor <notifications@github.com>wrote:

> In application/helpers/download.php:
>
> > +		if (count($incidents) > 0)
> > +		{	
> > +			foreach ($incidents as $incident)
> > +			{
> > +				// Start Individual report
> > +				$writer->startElement('report');
> > +								
> > +				// Generate report map
> > +				$report_element_map = xml::generate_element_attribute_map($incident, $report_map);
> > +				
> > +				// Form this incident belongs to?
> > +				$form = ORM::factory('form')->find($incident->form_id);
> > +				$form_name = $form->loaded ? $form->form_title : '';
> > +				
> > +				// Add it to report element map
> > +				$report_element_map['attributes']['form_id'] = $form_name;
>
> Hey @rjmackay <https://github.com/rjmackay>, just tested out with the
> zombie reports. The one's with blank form_name attributes are those with
> the form_id set to 0. If i'm not wrong, that default value for that in the
> db should ALWAYS be set to 1
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ushahidi/Ushahidi_Web/pull/944/files#r2263838>.
>
>



-- 
Robbie Mackay

Software Developer, External Projects
Ushahidi Inc
m: +64 27 576 2243
e: robbie@ushahidi.com
skype: robbie.mackay
default() will throw an ArgumentError because the lambda is expecting exactly 1 argument; it's basically equivalent behavior to `def default(value)`
That would be OK with me.

Where do we go with signals and name spaces, that is, other programming issues that aren't tutorial in nature?

It sounds like we're on the same page, so feel free to pick a solution here, or tell me to do it and I will, tomorrow.
可以練習在 `bootstrap-config.scss` 裡面設定這些 default 的東西  
變數可參考這裡  
https://github.com/anjlab/bootstrap-rails/blob/master/vendor/twitter/scss/variables.scss
Are you sure ``'ord'`` has the correct type as "string" is not a sub-type of "object<callback>"?
This needs to be changed as `$values` can be only in similar format to the `create()` method.
I'm just afraid that this can result in unexpected behavior in case of differences between gcc and clang, when someone uses clang with code that is intended to be compiled with gcc. But maybe you're right about this, especially if your have compilation errors (though I never get ones caused by gcc-specific features). Still this worth mentioning in the documentation (and the default value must be changed there to omit diverging of the documentation with the code).
That's really nice piece of code :P. I don't use Python, and I was looking for a way to do that like ruby.
Du hast natürlich recht! Thx für den Hinweis
changed to Action instead of explicit delegate.
Thanks - this has been fixed.
So we could say something like: The following tags always take a full stop: the title, @params, @return @note @deprecated  ... but not @since or @link maybe unless you really want full stops all over the place...  I think the rule could be stated as tags with labels, descriptions or comments take a full stop at the end of the label, description or comment. Tags that only have   a value with no additional information do not.
@LouisLandry Done - moved it back into the Syslog class https://github.com/AmyStephen/joomla-platform/commit/8eb5ad5b1b48ea29ae9a05f85ec3f58c962e86c0#diff-7

I assume it's still okay to abstract out the others? (I did not change back the other two.)
Looks cool though :)
Take these out?
Je vois que tu rajoutes à plusieurs endroits le "user_fullname_lower". Quel est la raison? J'ai l'impression que tu ne le ré-utilise pas ailleurs, donc je ne vois pas trop l'intérêt.
Why have you removed `ul`? The idea is that we keep semantic meaning of tags. Using divs everywhere is discouraged. You can always style `ul`s as you want, same as divs.
And how can i write this.. My english is not very good
OK, use [$.each](http://api.jquery.com/jQuery.each/) here.
(You should write only when verbosity flag was set.)
Why are you not rendering this with `{% include "form/form.html" %}`? You can move `id="email_confirmation"` to something else.
Good!

`DEFAULT_IMAGE_PATH` -> `DEFAULT_USER_IMAGE_PATH`
You don't need to do this. If you just call parent's class method, then you simply don't define anything in this class. And only parent's class method will be called.
omg, I didn't. Arrrrgh
What kind of information? <form action="Arranging users">
what does the source flag do?
I think it would be better to delete the invite key, rather than saying they accepted, which is not actually the case.
`createValidatorChain `
Don't need this class
Why does this function exist if saved() can do the same thing?
you may want to remove your user name like: 
git clone url git@github.com:<user_name>/playing_cards.git
looks nice. Thanks.
Dodalem kolejna linijke nieortograficznie ::)
useCapture is not optional since very long, or is it?
It's static? That means that this is effectively a global variable, right? Why? Shouldn't this be specific to each NgxRewriteOptions instance?
Something odd is going on here
Again, these two should be arguments to the setup function. You need to add the help function. For example, see `prompt -h adam2`.
What do you propose that has_constant do? I started with this.

hasconst : bool or None
    If None, the RHS matrix is automatically checked for a constant. If True, k_constant = 1, const_idx = [] and no checking is done. If False, k_constant = 0, const_idx = [] and no checking is done.
Not sure if this still works with -O compiled python, since then the try is empty. I never tried this.
I prefer postfixing qualifiers

endog_all  or just endog ordata
Если честно, непонятно что именно там может быть помимо https://uploadcare.com/documentation/reference/basic/cdn.html
И как оно тут может быть использовано. Для более детальной проверки переданных параметров?

Тут вообще есть проблема с тем, что параметры, заданные в фильтре для  проверки нужно сначала распарсить, а потом заново собрать. Поэтому пока проверок в обоих случаях минимум, а строка переданная фильтру напрямую вставляется в url.
Doesn't `$openvpn::params` have to be inherited for this to work?
Just curious: Does this have anything to do with this patch? (By all means keep it in)
Hi guys,

As far as naming goes, I realized at some point that we needed to make a distinction between URLs that were pointing to webpages and URLs that pointed to more resources on the API. I chose to use "URL" to note that it's pointing to a displayable... thing that you can use directly with your end-user, such as a link to a product detail page or an image that you can display. "Products" are named as such to try to hint that it's linking to more abstract "resources", not webpages. 

Sorry for the confusion, but hopefully you can appreciate the desire for a distinction.
There's a bunch of siblingNav-related stuff here that really should be in toura.UI that probably just didn't get moved over in the split. This is likely causing some of the as yet unresolved siblingNav bugs, so it's worth taking care of.
- g
Do we even want non-public functions to be doxyfied?  The user can't call them, so do we want to pollute the dox with these?  I might favor just converting to `//` and remove the `@brief`.  i.e. just make them a normal comment.
This is what caused Jane's problem; this line got removed from master.
Why the hell is db/schema deleted?
groovy's "[:]" map produces a LinkedHashMap. Perhaps better to use that, so we get more deterministic behaviour (e.g. on toString etc)?
maybe make this one atC1/  (append trailing slash) to ensure works with and without that slash
I was surprised at this, because "then I should see" is used everywhere.  Changing the xpath expression to use the current node as a search scope seems like a good idea, I am just surprised (and happy) it didn't cause other existing features to fail.
Sure, that's not unreasonable. But it's also not wrong to consider the native `Array::toString` as part of the language, which would make it just as "explicit" as any of the language's other behaviours.
Another hardcoded port.
I understand south's benefits, but the initial migration shouldn't have columns that the original app didn't have...
Hi Alexis,

i'm still thinking about going the predicate route, but didn't have the opportunity to get more into the details due to other obligations. Thanks for providing feedback, i hope to get back working on this soon.

Cheers,
Andreas.
I first thought that the reason for all this parsing back and forth was to gently replace. However, it's not, so you could just do:

```
failure_url += '?' not in failure_url and '?' or '&'
failure_url += 'bid_login_failed=1'
```
What's the motivation for wrapping the credentials in an object?
The logger will include its own timestamp. Also, "Cassadra" has a typo.
Probably a good idea--want to make that change?
o nome correto seria test_user_login_inactive
Why not keep the old logic of using `"-"` rather than `"nobody"` and `"system"`?
I don't understand why it can't be:

    settings.LANGUAGES.get(lang, '')

Are there cases where settings.LANGUAGES.get(lang) can return an empty string?
I think the problem right now is if you want to support a new board you have to make changes in 20 different little places.  Hardcoded VID/PID (and only one at a time) being the least of the problems.  Adding a hardware page, making the bootloader know about it, etc.  If we start supporting 10 different boards we should try and make the code specific to each be grouped together instead of all throughout the code.

Would you be happier if we make the plugin singleton :D

As to things being done dynamically that could also be done and would be worth it.  In this case based on some external files (xml ?) the plugin could create and register different IBoards so that is certainly not exclusive with this IMO.
Does AuthValue exists?
I did expirement with define_method, but was unable to get it to work. This is working fine with no observable side effects.  Sorry, but I feel like I've already spent too much time on this refactor so I would really like to leave it as is.
Название пакета со строчной должно быть.
ga.push(0L);
Why did you changed this from Class to function ?
Because the acceptance tests fire up a Ray UI, and because they aren't really needed to be run every time, I disable them by default and force the use of an environment variable to enable them.
Why is this Paris Time? I'm in New York. Just because symfony is some garbage french invention doesn't mean I shouldn't be allowed to eat my freedom fries.

In other words, can you clarify why you specified a date here instead of trying to get system timezone instead?
s/manifste/manifest
In OO parlance, what I wrote seems more like programming to an interface to me.
It's best to be correct when setting Copyrights.
I mean doesn't apply `opts[:class]` to class option.
Maybe make this a separate test named `test_perimeter_neighbourhood`?
Typo: representations
@GaelVaroquaux I should be careful if I become even more picky than you ;)
This method doesn't depend on the object state so it can changed to a function.
I think this calls for a utility function that checks if a parameter is >0 and raises a ValueError otherwise.
тут возможно лучше

     $this->categories[] = $category->getId();

как ты считаешь?
чому саме @PreAuthorize, а не @Secure?
This looks to be an array of multiple objects, as such the member should be plural such as childObjects_
As of @SpiffyJr's open PR, we can use `is_string` here
Can you add [`implements InitProviderInterface`](https://github.com/zendframework/zf2/blob/master/library/Zend/ModuleManager/Feature/InitProviderInterface.php) and add the `/** {@inheritDoc} */` docblock as of the other methods as well? :)
Very nice idea to point to the unit tests in source code! I'll use that trick in future.

But since this is a novelty, please spend some more words on it:

`Sample code is at` -> `A fully working example can be found in the source code, in the testsuite. See ...o.f.sd.SpatialIndexingTest`
I don't think this is correct if end==cur
Not sure this applies to all numerics (e.g., packed ints).
Still problems with -ve leaks here I think..

The temporary 'index' table, doesn't link the rows, so neither leaving index[x] set, nor clearing them helps..
The rows are still owned by rowsToSort..

You'd either need to clear the rowsToSort member that corresponds to the index[x] - which you can't easily do, as you don't know it's position.

Or, as I said in previous commit: 
>In this case prob. easiest to link the row returned.
Please add a space after the comma in the argument list
These aren't consistently named with the "currencies" object in the CreateAccountPage.java page. They need naming consistency. Also, this value when passed through to the server will be an enumerated int, so I'd recommend using an array of currencies (You can decide how the enumeration works) so that they can be easily accessed without large else-if blocks.
Tu parles de quoi exactement ?? Commenter le bouton, ou bien les stickers, ou bien tout ??
We don't need this anymore and it was causing extra padding on the bottom
Covered by index.isValid, I believe.
@derekhiggins I just learned something. There is actually a parameter called binary. After reading the code, it looks like it does the exact same thing as start. That is a little confusing.
It has to be required somewhere.  Some of the mixins get pulled in by a few big require-factory files like ```lib/msf/core/exploit/mixins.rb```.  Now that I think about it, this and the PhpExe mixins probably both belong in there.
Rex also has a Base64 decoding function called "decode_base64()"... is there any particular reason why you're using Ruby's Base64.decode64() instead of ours?
It's really not a good idea to swallow all errors like this.
This assignment and comparison business will drive men mad. I know, I've done it.
does the 8-connectivity make a large difference compared to 4-connectivity?
The construction of the transform matrix would be much clearer if created all at once:
```python
transform = np.array([[a0, -b0, a1],
                      [b0,  a0, b1],
                      [ 0,   0,  1]])
```
It looks like we initialize `children` in the prototype constructor, so you could initialize `bindings` there, too.
These are here too as the learn branch branched off of the master in my repository. The master in my repository has this updated and a pull request has been issued about this (#2).
There is no `autoconf` in the main repo, so this will cause undefined behavior. Cross-repo dependencies and dependencies between formulae in Homebrew-alt are currently not supported.
I guess there's no reason that we couldn't always use image.src, I guess I just exposed the id in case someone wanted it, but since "less is more" you're probably correct.  I'll change it.
We usually say `image is required.`, not `must be defined`.
A more descriptive name would help for people like me that don't read carefully.
@mramato it's OK to make this tweak in master since I just merged this.
If something is likely to be used by more than one layer in the stack, then I suppose it should be in `Specs`.
The variable name `mv` for "model view" no longer makes sense, right?

Are you sure about this change.  In my OpenGlobe RTC [implementation](https://github.com/virtualglobebook/OpenGlobe/blob/master/Source/Examples/Chapter05/Jitter/RelativeToCenter/RelativeToCenter.cs), I use the model-view matrix.  This is also how Deron [described](http://blogs.agi.com/insight3d/index.php/2008/09/03/precisions-precisions/) it.

@kring changes here will affect your terrain code.
OK that is up to you, @kring, etc.
音量设定应该和音效设定放到一起。
This should be `doesNotExist(..)`, i.e. without the `s` at the end
Dunno, do you why?
Why is this here? This isn't needed for this new feature.
Поехало табулирование (табы+пробелы?). Остальное все норм.
Does this class need all the _ names? Since everyone that is using this is in the same library, I don't think it buys you much.
`oob` is the default, you can leave this out (and the contstant)
Dismiss changes in this file. It is just the compiled version
I like this, very readable
A minifier would do that for us, and a `var` per-line is arguably as readable but easier to expand upon later (you don't have to change which declaration ends in a semicolon.
Double check that :P
Make sure this isn't the path for backoff. Backoff *is* shared between accounts on the same server.
And killed since thincloud-test provides this.
Yeah, any modified rows count as 1 deletion and 1 addition.
This assignment has been commented out in ajax_backend.html, so removal here is dependent on removal there.
@schmittjoh Seems like one param is missing =)
Should add SB to the description?
Registering the bundle in the Kernel isn't specific to Composer, is it? The old `bin/vendors` method doesn't automate this step IIRC. I think we can do without it.
maybe. at the very beginning there was prepare post data for each message. anyway. yes
The following function, 'load', is really only temporary for us to see some sample data.  Why don't we add a comment so that others won't wonder what this magic is about?
3 times! MUST BE REFACTORED
Expected result shouldn't be required.
Let's remove the `_pad_t` alias. It's a redundant indirection and the linux header uses `short` and `int` too.
Shouldn't the hl7Output have some sort of "error" string in it if there was an exception?
Hi, Gary.
IMHO: to catch more generic Exception and then check what we are expecting, allow to end up with
 more informative failure about a test.
In this case if Exception isn't `BeanDefinitionParsingException`, test fails with `AssertionError` 
on the problematic line of test class.
Otherwise, if we catch exactly `BeanDefinitionParsingException` and the Exception will something else, 
we end up with stackTrace of that unexpected Exception. And it might be hard to determone what's goiing on.