1: D400 First line should end with a period (not 'o')
1: D200 One-line docstring should fit on one line with quotes (found 3)
2: L102 copyright year is outdated, expected 2015 but got 2014
23:1: E402 module level import not at top of file
24:1: I100 Imports statements are in the wrong order. import cgi should be before import time
24:1: E402 module level import not at top of file
25:1: E402 module level import not at top of file
26:1: E402 module level import not at top of file
27:1: I100 Imports statements are in the wrong order. import sys should be before import urllib
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. import string should be before import sys
28:1: E402 module level import not at top of file
30:1: E402 module level import not at top of file
31:1: I201 Missing newline before sections or imports.
31:1: E402 module level import not at top of file
32:1: I101 Imported names are in the wrong order. Should be CFG_BASE_URL, CFG_INSPIRE_SITE, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_NAME_INTL, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL, CFG_VERSION, CFG_WEBSTYLE_TEMPLATE_SKIN
32:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.ext.template
32:1: F401 'CFG_SITE_SECURE_URL' imported but unused
32:1: F401 'CFG_SITE_RECORD' imported but unused
32:1: F401 'CFG_VERSION' imported but unused
32:1: F401 'CFG_WEBSTYLE_TEMPLATE_SKIN' imported but unused
32:1: E402 module level import not at top of file
33:6: E126 continuation line over-indented for hanging indent
45:1: I101 Imported names are in the wrong order. Should be gettext_set_language, is_language_rtl, language_list_long
45:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
45:1: F401 'is_language_rtl' imported but unused
45:1: E402 module level import not at top of file
46:1: I101 Imported names are in the wrong order. Should be create_html_link, get_canonical_and_alternates_urls, make_canonical_urlargd
46:1: F401 'get_canonical_and_alternates_urls' imported but unused
46:1: E402 module level import not at top of file
47:30: E126 continuation line over-indented for hanging indent
48:1: I100 Imports statements are in the wrong order. from invenio.utils.date should be before from invenio.utils.url
48:1: F401 'convert_datestruct_to_dategui' imported but unused
48:1: E402 module level import not at top of file
49:31: E126 continuation line over-indented for hanging indent
50:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter should be before from invenio.utils.date
50:1: E402 module level import not at top of file
51:1: E402 module level import not at top of file
52:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.html
52:1: E402 module level import not at top of file
56: D101 Missing docstring in public class
58: D202 No blank lines allowed after function docstring (found 1)
58: D400 First line should end with a period (not 'y')
58: D208 Docstring is over-indented
110: D401 First line should be in imperative mood ('Create', not 'Creates')
110: D400 First line should end with a period (not 'e')
110: D208 Docstring is over-indented
199: D401 First line should be in imperative mood ('Create', not 'Creates')
199: D202 No blank lines allowed after function docstring (found 1)
199: D400 First line should end with a period (not 'r')
199: D208 Docstring is over-indented
260: D401 First line should be in imperative mood ('Create', not 'Creates')
260: D400 First line should end with a period (not 'r')
260: D208 Docstring is over-indented
289: D202 No blank lines allowed after function docstring (found 1)
289: D400 First line should end with a period (not 'e')
289: D205 1 blank line required between summary line and description (found 0)
289: D208 Docstring is over-indented
326:18: E126 continuation line over-indented for hanging indent
328:13: E266 too many leading '#' for block comment
329:13: E266 too many leading '#' for block comment
332: D401 First line should be in imperative mood ('Produce', not 'Produces')
332: D202 No blank lines allowed after function docstring (found 1)
332: D208 Docstring is over-indented
368:13: B901 blind except: statement
368:20: E261 at least two spaces before inline comment
388:21: B901 blind except: statement
390:34: E261 at least two spaces before inline comment
427:17: E126 continuation line over-indented for hanging indent
427:28: E203 whitespace before ':'
430:37: E502 the backslash is redundant between brackets
431:24: E131 continuation line unaligned for hanging indent
433:24: E203 whitespace before ':'
434:24: E203 whitespace before ':'
435:31: E502 the backslash is redundant between brackets
435:28: E203 whitespace before ':'
436:21: E131 continuation line unaligned for hanging indent
437:90: E225 missing whitespace around operator
438:28: E203 whitespace before ':'
439:28: E203 whitespace before ':'
440:28: E203 whitespace before ':'
441:28: E203 whitespace before ':'
442:28: E203 whitespace before ':'
443:28: E203 whitespace before ':'
444:28: E203 whitespace before ':'
445:28: E203 whitespace before ':'
446:29: E203 whitespace before ':'
447:62: E502 the backslash is redundant between brackets
447:28: E203 whitespace before ':'
447:37: E225 missing whitespace around operator
448:34: E131 continuation line unaligned for hanging indent
448:67: E502 the backslash is redundant between brackets
449:34: E131 continuation line unaligned for hanging indent
450:15: E126 continuation line over-indented for hanging indent
453: D401 First line should be in imperative mood ('Print', not 'Prints')
458:53: E251 unexpected spaces around keyword / parameter equals
458:55: E251 unexpected spaces around keyword / parameter equals
458:78: E251 unexpected spaces around keyword / parameter equals
458:80: E251 unexpected spaces around keyword / parameter equals
478:9: F841 local variable '_' is assigned to but never used
511:41: E231 missing whitespace after ':'
512:41: E231 missing whitespace after ':'
513:42: E231 missing whitespace after ':'
516:41: E231 missing whitespace after ':'
517:39: E231 missing whitespace after ':'
518:41: E231 missing whitespace after ':'
519:42: E231 missing whitespace after ':'
528:9: E303 too many blank lines (2)
550:9: E303 too many blank lines (2)
562:30: E231 missing whitespace after ':'
563:38: E231 missing whitespace after ':'
568: D401 First line should be in imperative mood ('Print', not 'Prints')
598:69: E502 the backslash is redundant between brackets
602:41: E231 missing whitespace after ','
612:21: E203 whitespace before ':'
613:19: E203 whitespace before ':'
614:17: E126 continuation line over-indented for hanging indent
623:5: E303 too many blank lines (2)
623: D401 First line should be in imperative mood ('Display', not 'Displays')
623: D400 First line should end with a period (not 'd')
623: D205 1 blank line required between summary line and description (found 0)
623: D208 Docstring is over-indented
641:9: F841 local variable '_' is assigned to but never used
664:9: E122 continuation line missing indentation or outdented
665:9: E122 continuation line missing indentation or outdented
665:13: E231 missing whitespace after ':'
666:9: E122 continuation line missing indentation or outdented
666:16: E231 missing whitespace after ':'
667:9: E122 continuation line missing indentation or outdented
668:9: E122 continuation line missing indentation or outdented
668:18: E231 missing whitespace after ':'
669:9: E122 continuation line missing indentation or outdented
684:21: E203 whitespace before ':'
685:22: E203 whitespace before ':'
686:22: E203 whitespace before ':'
687:21: E203 whitespace before ':'
688:93: E203 whitespace before ':'
692: D401 First line should be in imperative mood ('Produce', not 'Produces')
692: D202 No blank lines allowed after function docstring (found 1)
692: D400 First line should end with a period (not 't')
704:9: F841 local variable '_' is assigned to but never used
708: D401 First line should be in imperative mood ('Return', not 'Returns')
708: D202 No blank lines allowed after function docstring (found 1)


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2013
23:1: I201 Missing newline before sections or imports.
28: D209 Multi-line docstring closing quotes should be on a separate line
28: D202 No blank lines allowed after function docstring (found 1)
37:5: B901 blind except: statement
38:9: F401 'pickle' imported but unused
83: D400 First line should end with a period (not 'y')
97: D103 Missing docstring in public function


32:1: I100 Imports statements are in the wrong order. from itertools should be before import pkg_resources
32:1: F401 'count' imported but unused
36:1: F401 'SchTASK' imported but unused


52:1: I101 Imported names are in the wrong order. Should be current_app, jsonify, request
52:1: I100 Imports statements are in the wrong order. from flask should be before import requests
53:1: I201 Missing newline before sections or imports.
54:1: I201 Missing newline before sections or imports.
56:1: I100 Imports statements are in the wrong order. from invenio.utils.xmlhelpers should be before from lxml.etree


Тут тоже непонятно. Вроде бы класс наследуется от IntegerCalc, поэтому можно подумать, что надо реализовать целочисленное деление. Но тогда уже надо возвращать целое число. Или уже если реализовывать полноценное деление, то проверка на целые числа на вход - лишняя. 5 / 2.5 = 2, например. И вовсе это не ошибка.


I tend to avoid WeakSet for things like this because of performance.

Anyway, sounds good


Performance?


I have the same trouble.


it's cleaner since you don't need starargs


Good point. I'll change it.


Will there be a section that explains what options can be specified and what the default behaviour is?


Instead of listing all possible values, would it be enough to say this?

    a String representation of one of the possible values from the enum org.keycloak.events.EventType

There are quite a few possible events, and it would be a maintenance headache to keep the list up to date.


I don't see the part of the diff where this field `normalize` is added, but I think we should make sure to leave a big comment explaining that this is a (temporary) hack to allow for the fact that, in trans, closure items need to be normalized, but elsewhere they do not.


minor comments, but lgtm!


We are using newlines everywhere for readability, why should we care about the produce html here?


similarly, this comment feels not quite complete (see <https://github.com/jroesch/rust/commit/4156e2a4915b6a593a821e8282c56b8b732b9e49#commitcomment-11881127>)


again <https://github.com/jroesch/rust/commit/4156e2a4915b6a593a821e8282c56b8b732b9e49#commitcomment-11881127>


TEST-7 ASDFASDFASDF


true...


Would this work?
    void errorProcessingEvent(String cause, @Cause Throwable e);


Corrected. BTW I haven't finished doing this part yet.

________________________________
From: Simon Casale Brunet [notifications@github.com]
Sent: 26 June 2015 13:18
To: turnus/turnus
Subject: Re: [turnus] (turnus) extension to add scheduling overhead to the weight (put in (5e9a5d2)

why creating a temporary list? you can use just a counter

—
Reply to this email directly or view it on GitHub<https://github.com/turnus/turnus/commit/5e9a5d23b460fd58b475e33bed9f410079ea08c7#commitcomment-11880070>.


Hi @rati592 I have fixed the demo link and updated the latest version of the player to it. You may find a working demo in http://mailonline.github.io/videojs-vast-vpaid/

Is that what you needed?


@nimrodp do you recall what the error was?

Requiring 0.9.0 and no other version makes it difficult for this gem to coexist with others. A ~> require would be preferred.


is the double dash a typo?


Duplicate.


Yes :-) This runs as an extension to Keycloak, and jboss-logging takes care of sending this to the proper logs. I think there might be some classpath issues if I reference jboss-logging directly from this extension jar (it's not "included" in any war)


아 제가 저런 실수를 했군요;;


Pretty much each usage is in a different maven module, and those listeners are jars put directly as modules in the KC server. So, the fewer dependencies, the better.


why do you prefer-source here? prefer-dist is much faster in most cases.


Ditto


remove these 3 functions `setClientId` `setClientSecret` and `setRefreshToken` and change `setVerifyPurchases` so it accepts a hash with parameters. In that function, check that you have all necessary parameters and if not print an error to the log.


use @pos(f 0)


It's used also only once, so, I don't see an advantage for that.


+1 , as it's passed down to the provider itself.


I thunk this comment is not correct?


I tried to auto-complete the GlobalComponent and Intellij auto imported the wrong one. I'll remove it in a second.


I intend to move medical towards a more high-tech state, after all, this is 2559.


+  return "Hello, World!" "
 Missing semicolon.


+  if (bla    == null) return;
 Use '===' to compare with 'null'.


why is this public?


Yes, perfect.


Joe, we aren't changing the contribution status via the params supplied. This and the code below checks for invalid status & financial type changes in the contribution API.


We don't need to, since we aren't changing stuff related to contributions, just payments.


@LPTK any news?


Hi! Another person has requested that feature. Would you like to create a pull request now? Otherwise I´ll just add it myself soon.


use better variable names instead of `foo`,
in example: `handler`, `iterator`


return the value of array in stead of `console.log`


`cause` and the stack trace will have some words in common, but OK, if you like it like that.


Can you also expose the `Gallery.AzureStorageReadAccessGeoRedundant` setting in the cloud service configuration?


@tchap If it's a chain, it should be a list data structure. Not recursively nested like this. Or do you see any problem with representing all this information by `[(task, error), (task, error), ...]`?


`i++` are enough (instead of ` i += 1`)


That always happened before the update...


I need someone to produce the sprites for these, if possible.


`forEach` is better name for the function


kexec code has been broken down into multiple files in arch/x86/kernel.

crash.c contains code which primarily is needed for crash dump. All the header preparing code kexec file is doing related to crash dump so it makes sense to keep it in crash.c.

I don't like the idea of creating separate file arch/x86/kernel/kexec_file.c. Let us first justify that why code does not fit into existing files. And so far I have not seen that argument.

Yes, I mean keep #ifdef CONFiG_KEXEC_FILE in crash.c. It is that way right now. What's wrong with that.

similarly machine_kexec_64.c contains kexec code for 64bit. I think this file should also contain code for both old and new syscall. Just divide it nicely in 3 sections.

#ifdef KEXEC_CORE
  common code
#endif

#if def CONFIG_KEXEC
   code used by old syscall only
#

# ifdef CONFIG_KEXEC_FILE
  code used by new syscall only
#

Right now machine_kexec_64.c is around 500 lines and it does not seem big enough to warrant moving changes in a new file.


hm... sounds really weird. Plz. put a comment there explaining that `System.out.println` if you insist on that.


Shall we revert this for 1.25, or rather update the translations?


Sorry man


@mbernat Well, it's more like recursive than a chain. No, it doesn't make sense to represent it like that. It works, as you must have notices, in the way that when an `error` is returned (which can be `errs.Err` under the hook), I usually wrap it in `err.Err`, which appends the task description. So no, it's not a chain, it's recursive.


Have done so in subsequent commit.


OK, giving up


Hi @nathanfranklin, I have this almost working.

At the end I have separated KratosCFD and DEMPack as different plugins so one can do

```Python
from simphony.engine import kratosCFD
from simphony.engine import kratosDEM # Or DEMPack
```

The thing is that DEM directory keeps complaining about ```kratosDEM``` not being defined.
I am missing something in the setup.py ?


hahah this happens to me when i try to write up anything at night, lol


@prudhomm I'm not certain for the design of the postprocess.


Can we not disable SSL certificate checking?


> It's possibly a bug and should be made a global variable...

...same as `$firstattachment = true;` on line 841 some lines after (also asked in 821), which both ($postattach and $firstattachment) are used in workPopfetcher() method. But I don't know what that standard attachment mode was originally supposed for. Maybe it was just a copy leftover from workPopfetcher() method. Lets see if that missing set option was the culprit for the OP.


GTM


PLEASE IGNORE THIS FILE!


Change the name from `verifyData` and `verifyDataAsync` to `verifyPurchase` and `verifyPurchaseAsync`.


I would remove the "(probably.... )" part


For one, the extra dependency. By using HttpUrlConnection, I don't need to mess with the classpath. For this specific case, the performance should be similar (or better) than Apache's, as the connection is cached at the JVM level.


Suggest the the public repository instead?


Is this still valid?  It looks right offhand, but I'd be most comfortable if Ian reviewed this.


What's the point of this?
Given we have pagination plugin, should we just paginate it? Should be fairly trivial, as the whole set of data is already there.


return "Hello, World!" !
 Missing semicolon.


if (bla  =  null) return;
 Use '===' to compare with 'null'.


Indeed, I was having a problem that I *thought* that was caused by not explicitly closing it, but the try-with-resources on the reader should, by contract, reflect on the inputStream.


This can be made private right?


Actually this breaks because `lookup` borrows `self` and that borrow is still in scope for the call to `eval_within`. I don't get why it's still in scope, though, since `lookup` should have returned by then.


Again :)


This Factory is required by Keycloak for the listener. It's called when KC server boots, and the provider is then consumed when an event occurs.


@tchap I don't get it :) The query data structure should mimick what you have in `pivotal.Story`, except the fields would be optional. So if you want to update something, just copy it from `pivotal.Story`. If the default JSON encoder doesn't do this for you then it should be thrown out because `story.labels` should never ever be `nil`, it's always an array.


In our jenkins job we use ansible.cfg.example as ansible.cfg, maybe suggest it?


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


ok, the questions:
1. if I call `setVerifyPurchases`, does it mean I turn it on?
2. if so, how can I turn it off?
3. maybe it's better to call it: `configVerifyPurchases`, or `setVerifyPurchasesConfig`?


Personally I do prefer the clarity of declaring the classes only in closely associated header, but ... it a) generates cost  as increased compilation time b) is just an opinion -> so I'm ok with these changes.


"We're going to use the common ... key" -> "For example, consider the following ... key"
The idea is to make it more general and remove internal references


`  return "Hello, World!"`!``
 Missing semicolon.


`  if (bla `=`= null) return;`
 Use '===' to compare with 'null'.


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


GTM :+1:


yeah,it work well in android.


@tchap linguistic corner: "to err" is a verb :) And in any case, I don't like this name since I'm completely lost in your `error`s and `Error`s and `Err`s... :)
But doesn't matter if you're happy with it and don't get confused about what is what.


@mbernat I understand what you mean, but that would probably require to simply pass `map[string]interface{}` into the story edit API call, because there is no other way to make things really optional. It can be unset in the map or `nil` in case it's a struct. And btw `nil` is a valid value for an array anyway :-) I mean, labels can be `nil` even as `[]Label` ;-)


LOL. Nice fix.


I think workarounds have been removed.


fpaste.org? :)


@RomainLG currently we're not allowing this level of customization. I'll mention it to the team, but for now out goal is to keep the CheatSheets as simple as possible. One goal we had for the JSON was to avoid any HTML in the content so we're working purely with plaintext.

We tossed around the idea of allowing Markdown and that may eventually happen, but for now we're keeping the syntax very simple.

Italics and Bold can certainly help, but the cheat sheet should be able to get it's point across without formatting the text.

Sorry you can't use them, but I hope you can understand where we're coming from!


you're right, I'll do


TODO?


still unsure about this, because it is quite brutal


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


`  return "Hello, World!"`
 `                        `
 Missing semicolon.


`  if (bla == null) return;`
 `           ^              `
 Use '===' to compare with 'null'.


I think this is not right ('...example in Confluence...')


done


`  return "Hello, World!"`
 `________________________`
 Missing semicolon.


`  if (bla == null) return;`
 `___________^______________`
 Use '===' to compare with 'null'.


god knows!


Just to explain a bit better: I don't have anything against constants, and I can see me in the future cursing myself for having so many repeated strings, *but*, we'd need to put those strings into a common module. This common module would have only those constants, so, the choice here is between having a module with constants + this module depending on that, or having those strings being repeated. As I don't see those changing that often, I think the "cheaper" is to have them repeated.


Formatting issue?


remove this function


Thanks for the feedback. Upon reflection, I think it would be best to simply pass up more information from the underlying exception as we discussed in the debug code. The message "Can not map empty file" is actually pretty helpful, as it points to the precise problem.


`[$field]` is no longer needed here.
`$this->endtime_html` is enough.


We were planning to do something similar in CitusDB, but @anarazel pointed out that this doesn't work well with prepared statements. I don't know if this is related here, just wanted to mention in case it is related.

In CitusDB we assign the temporary table names at plan time, and prepared statements are planned at prepare time, so multiple execution of same prepared statement will try to create a temporary table with the same name and we will have problems if the transaction isn't committed yet and the temp table already exists:

BEGIN;
PREPARE foo AS SELECT count(*) FROM distributed;
EXECUTE foo;
EXECUTE foo;
COMMIT;


`  return "Hello, World!"`
 `_________________________`
 Missing semicolon.


`  if (bla == null) return;`
 `___________^_______________`
 Use '===' to compare with 'null'.


Yeah I have been restarting the AppVeyor builds to hopefully catch it once when it works.


maybe better `exit $ret` here, no need to discard the exit status, could be useful.


Please check line no 10. This test is failing.


@mbernat It's totally clear to me :-) But if you have a better name, I am listening :-) But it cannot be `Error`...


Couldn't this part be moved above with the "Installation" part, when you talk about the key? It's useful everytime you need a key. Here you could just reference it.


You should replace `$args['start']` with `false`.


there should only be one build, as `master` in a parent for `someBranch`


Good point.

Name the function `configVerifyPurchases `.


Good point. In fact, I was thinking if this shouldn't be somehow configurable with values captured from standalone.xml . Do you know if we have some facility like that? Or should it be added there as system property?


`[field]` is no longer needed.


{SS.config.kana.root} exists?


Have people had great luck with this? So far I've found scss-lint to be scores better, this is the first thing I ripped out of the theme on implementation.


```python
command.scripts.startswith(('open', 'xdg-open', 'gnome-open', 'kde-open'))
```


At least for packstack I would check if workarounds are useful (I think they are not right now)


missing "-": all-in-one


If that's your choice that's ok with me!

I'm new to github and cheatsheets so...
While I have you here, (I don't know how to contact people via PM yet)

I've created an instant answer for imgur and one for reddit. I just want to issue a pull request for the imgur.json... And I'm stuck since yesterday. Do you have any idea?

I thought I was close to succeeding here: https://github.com/RomainLG/zeroclickinfo-goodies/tree/imgur-or/share/goodie/cheat_sheets

Because I have an isolated file. The problem is that I created an orphan branch and it can't merge to the master of DDG.

Any... Advice?


Thank You!


see above


```  return "Hello, World!"
________________________^``` <br/> Missing semicolon.


```  if (bla == null) return;
__________^________________``` <br/> Use '===' to compare with 'null'.


It is to avoid SSL certname matching issues


what's is the use of method "extractUrl" you are using in the code.


Arne,

Ik zal de controles volgende keer zelf nakijken. De 'quote'-fout is door
mij hersteld.
Wil jij voor publicatie zorgdragen ? Ik dacht dat dit, na correcte
validatie, ook automagisch ging, of is dit handmatig.

Alvast dank,

Gerwin

Op 23 juni 2015 06:40 schreef Arne de Laat <notifications@github.com>:

> HTML5 code validatie wordt gefaciliteerd door dit script:
> https://raw.githubusercontent.com/mozilla/html5-lint/master/html5check.py
> Die stuurt de html code naar een website ( https://html5.validator.nu/ ),
> en geeft de resultaten terug.
>
> De verdere code voor de checks (de aanroep en afhandeling) staan in de
> .travis.yml in deze repository:
> https://github.com/shs-geronimo/shs-geronimo/blob/master/.travis.yml
>
> Elke keer als je iets naar GitHub pusht wordt dat script op een travis
> virtual machine gedraaid.
> Al die runs kunnen we hier:
> https://travis-ci.org/shs-geronimo/shs-geronimo/builds terug zien,
> en dus ook zien wat de problemen in de code waren bij elke commit.
> Hier dus de error voor die vorige commit:
> https://travis-ci.org/shs-geronimo/shs-geronimo/builds/67719631#L163
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shs-geronimo/shs-geronimo/commit/d32cc970924b9a36b7abf0eb4f2166efc27ac5f1#commitcomment-11808653>
> .
>


In CitusDB we're planning to divide this function into two functions ```ReportRemoteError``` and ```ReportConnectionError``` to avoid confusion when we're passing a 2nd NULL arg. What do you think?


This is actually not good to keep VERIFY_PURCHASES in sharedPreferences. The moment he will turn it on it will stay on forever. Can you think of a way to make this false by default and only 'true' if the developer sets it in the current run?


```
  return "Hello, World!"
                        ^
``` <br/> Missing semicolon.


```
  if (bla == null) return;
          ^
``` <br/> Use '===' to compare with 'null'.


    statuspage json outfolder > test.json

1. file naming
2. UNIXy
3. more flexible


You can use `$this->endtime()` over here.


```
  return "Hello, World!"
                        ^
```

Missing semicolon.


```
  if (bla == null) return;
          ^
```

Use '===' to compare with 'null'.


We no longer rely on sequelize filters list (which is clearly not working properly).

E.g. See https://github.com/sequelize/sequelize/issues/3275


License =D


  return "Hello, World!":

```
                        ^
Missing semicolon.
```


  if (bla == null) return;:

```
          ^
Use '===' to compare with 'null'.
```


You can skip the WPT_Filters here. They are already applied in `endtime()`.


- Einleitung etwas formatiert, fehler entfernt
- Anhang formatiert
- Bug behoben (äöu im Abbildungsverzeichnis)


added asif in contact


Missing semicolon.:

```
  return "Hello, World!"
                        ^
```


Use '===' to compare with 'null'.:

```
  if (bla == null) return;
          ^
```


ack


### Missing semicolon.

```
  return "Hello, World!"
                        ^
```


### Use '===' to compare with 'null'.

```
  if (bla == null) return;
          ^
```


Understood.


>That won't work for compilers targeting non-x86 cpus. Use CheckCXXCompilerFlag to ensure that the compiler supports it.
I don't have non-x86 cpu. Btw, I just copy/pasted this code from https://github.com/kode54/dumb/blob/master/dumb/cmake/CMakeLists.txt


@quakenul In my opinion sorting by alphabet is a bit more general and everyone knows how to use it and it is easier to find them.
Grouping properties does make sense if they are predefined in which order they should occur.
Also sorting them alphabetically improves GZIP compression be around 3-6%


Am 26.06.2015 um 14:29 schrieb Colomban Wendling:
> Shall we revert this for 1.25, or rather update the translations?

It's a special branch. No need to worry.


get rid of this report and use the one from ReportService


It shows first 20 messages, not last.


> It's a special branch. No need to worry.

Ooooh, sorry, I didn't realize this.


This is not deprecated.


@tchap I have no idea how it works. I see all these err.Error and error.Errs and whatever and I have no idea what it all means and I can't you just use one simple Error structure for everything.

If all those things are different, it would be nice to name them by what they actually do and how they differ. You probably know it inside out but to me it never made any sense.


Where is the other half of this move? I see the deletion here but no corresponding addition in the universalviewer repo. Is there something that hasn't been pushed up yet?

Thanks!


The pin naming was peculiar within this component.

It was an adaptation of JK's pid component, made 4 years later, but appeared to be determined to make every pin and variable, a different name to the pid component for some reason.

Caused me no end of trouble converting it and would be even harder to use in a configuration if familiar with the pid component pins.

The only config in the samples which used it would never have run, it loaded at_pid and then used pid anyway.

Good idea as far as I am concerned.


LGTM


Fixed #18


Fixed #21


Fixed #22


git clone https://github.com/josephgajda/bash-quickies
bash-quickies$ git add ./memory_check.sh
bash-quickies$ git add ./nologin_unity.bash
git checkout master
git push


I believe there is a bug right now when the layout passed to the form builder already has a last row with `removable` set to true. The form builder demo currently has this bug, it has only one row and it's possible to remove it, when it shouldn't be.

This is because we don't fix the value of `removable` for the layout's last row here. We set it to true if it's not going to be the last row anymore, and create last rows with `removable` already set to false, but we also need to set the `removable` attr of existing last rows to false here.

Just adding an `else` clause here with `lastRow.set('removable', false);` should work, but we also need to have a test for this. Can you add this change and a test for it?


Aí!

On Fri, Jun 26, 2015 at 6:09 AM, Caio Gondim <notifications@github.com>
wrote:

> [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/loopinfinito/setup/commit/2cf78d287f014cb028c3dabeac46ce6e7c802de8#commitcomment-11878379>
> .
>

--
Almir Filho
*almirfilho.com <http://almirfilho.com>*
--
LAWS - Laboratory of Advanced Web Systems
*laws.deinf.ufma.br <http://laws.deinf.ufma.br>*


Is this even correct? Last I checked, Libav and FFmpeg both produced the .a files, even with MSVC.


The FFMS2 way is not ifdef soup. It's macros/defines in `ffmscompat.h`.


There's no need for this if, `this._removeConfirmationModal` should always be defined since you create it on the `initializer` method.


consegui!!


Cool, thanks, Glynn. If this is slated for the 1.1 release, then have you got any thoughts for unit testing it?


hello,

I am a beginner and French, and I wonder if the Jetty firmware works with azteeg x3 ?
Can you tell me the demarche to create firmware for azteeg x3 or where I can find documentation.

thank


@mbernat There are only 2 things: `errs.Err` (my interface) and `error` (built-in). `errs.Err` wraps an `error` and adds a task title. `errs.Err` implements `error` as well, though, so it can be wrapped as well. And `errImpl` just implements `errs.Err` and I use it my other functions as the only real implementation of `errs.Err`.


@gupta515 so the url is supposed to be like something like this, "http://www.popgunchaos.com/wp-content/uploads/2011/04/Tick.jpg"; so for the project that I was working on, as a part of which I built this proxy, it was not very  straight forward to get the url out of the servlet request i.e. HttpServletRequest. It was very specific to that project for which I had written the extract url method to get the url from the HttpServletRequest.getQueryString() method.


Can you move the modal creation to another function and call it here? This method got big because of it. Something like `_initRemoveConfirmationModal` would be ok.


you could have used ```fabs``` instead of ```abs```


Do you have a problem if I integrate this in my repo? https://github.com/mgmco/heroku-buildpack-nginx


Fixed issue #1


FFMpeg does not produce .a files with MSVC ;) Actually, I haven't seen any library with MSVC before that produces .a files.


Do you have a problem if I integrate this in [my repo](https://github.com/mgmco/heroku-buildpack-nginx)?


Replace `there is` with `has`


I got error of Bareword "assert_shutdown" not allowed... when I am running the testsuite. How to fix it?


This version of tls doesn't seem to be supported on OSX 10.10, yielding errors like these:

```
[...]
Undefined symbols for architecture x86_64:
  "_TLSv1_2_method", referenced from:
      ssl::SslMethod::to_raw::h35d3304e52e49589fOc in libopenssl-d8a07f4882d58f7a.rlib(openssl-d8a07f4882d58f7a.o)
ld: symbol(s) not found for architecture x86_64
```

Right now I am unable to compile hyper with SSL support on OSX, even though it worked previously.
I am not quite ready to create an issue for this, as I am still trying to pin-point the exact source of the issue.


I am not sure, what you mean by "ifdef soup".

The value is that resampling is made possible with playin ffmpeg. Libav comes usually with avresample but ffmpeg uses swresample. Those API's are pretty close, so the maintanance effort is not that high.


It's best to indicate what the button does instead of the text inside it, since the text may change and people will almost certainly not remember to update this test's name. Replacing `Yes, delete` with `confirmation` would be enough for example.

Also, replace `click` with `clicked`.


Ik moet nog handmatig publiceren (handmatig met één klik..).
Ik was van plan het te publiceren na jouw aanpassing, maar was er in de drukte niet aan toe gekomen.

Je hoeft dus niet zelf te testen, maar gewoon een 1-20 minuten (soms snel, soms lang wachten) na het pushen naar GitHub op Travis kijken.


Better!

But, the problem I see about this approach is that the profile version (1.0, 2.0, 4.0...), being an argument, is not being used. Then the macros SHAMROCK_CHECK_MONO_1_0_GAC_ASSEMBLIES, SHAMROCK_CHECK_MONO_2_0_GAC_ASSEMBLIES etc. don't do what their name say. It would have been better just to remove them, and rename _SHAMROCK_CHECK_MONO_GAC_ASSEMBLIES to SHAMROCK_CHECK_MONO_GAC_ASSEMBLIES.

Also, in the commit message you're just stating the *what*, but the important thing to highlight here is the *why*, I mean, what's the motivation for this change? AFAIU the motivation of this change is that these macros were hardcoding relative paths to the GAC assemblies which were not correct in all distros (case in point: Fedora). I think it's very important to reflect this in particular, and in to reflect always the _why_ in commit messages.


erlang:unique_integer(). not erlang:unqiue_integer().


it's used here:
https://github.com/vedi/android-store/blob/master/SoomlaAndroidStore/src/com/soomla/store/SoomlaStore.java#L528


Found out that hidden `update` parameter here: realm/realm-cocoa#2123


It produces `.lib` import libs for shared libraries, but it most certainly didn't produce `.lib` for static builds. Did this recently change?


Why was this moved to the themes? Now this logic is duped 3 times over... we really need to get this stuff in-check guys -- have smarter defaults and let the themes override only what they need to, please stop doing this.


There is too much music, those who do not have big pc will not be able to play due to lack of memory.
1 music =  ~100MB
There are nearly 1 GB of memory used only for music
In recent weeks I have bad_alloc (with GNU Linux).

And most players mute the music volume.


no more typos, md5s are ok versions too

could be

Fixed typos and MD5 SHA

:smile:


I can't quite remember what you said why you do this, maybe add a comment in the code also? I wonder if there's another way to check this. Do you do this here so that a roundtrip  doXContent() -> fromXContent() returns the same Fuzziness object?


AlloyUI's UI should follow the bootstrap css whenever possible. If liferay's form builder should use a different color/radius/background/font/width/height/float then they should override it on their side. Our buttons should be regular bootstrap buttons. It's best to just remove all this css for the buttons and leave them as plain `btn-primary` and `btn-default`.


Hi Pete! This is mental. Doing this from iOS.


You accidentally removed this attribute's description.


@tchap I thought you disliked bool parameters that change what the function is doing.

Besides, it makes no API sense to put milestone creation into what is essentially a getter operation. So please move it out. You can call that creation code perfectly well outside of this function.


There is no need at all to introduce pointers to yours sets. For the swap below, you can call ```points1.swap(points2);```


@ccd0 I must have accidentally skipped over it when I went to enable it on FoolFuuka too. It should be working properly now.


I also don't know if I agree with this option. We already emit in ES3 appropriately.


Or just squashed into the commits that it fixes.


I'm unsure of this change. It makes sense to only affect a model if it implements `HasPresenter` rather than just applying it everywhere.


@sravfeyn
I don't returning a EMPTY_FIELD because when someone want change value of EMPTY_FIELD then we also should change a string in this line https://github.com/lwyszomi/commcare-hq/commit/7a20fe36888abe8014630c738425572b329c207c#diff-1447108b0f26b538a015e9402945e9cbR45


@cptdanko Thanks for your reply, I will check it in my project.


#7 Forgot I was not in NicholasTD07/SwiftDailyAPI#34


what does this do? :)


В итоге лучше удалить эту строку


We should have a better description here (and on the similar functions on aui-layout-builder-move), it's not clear what this attribute is for. This should explain that this function will be called when remove buttons are clicked, and its return value will determine if row will actually be removed or not.


Saying ```for (const auto& iter : currPoints)``` is much more concise and expressive. You already use c++11 by saying ```auto```, so why write the for-loop the old-fashioned way?


@cptdanko Hi Bhuman, I need one more help from you.
I am not able set the proxy properly, can you guide me with any sample code if you have.


> I am not sure, what you mean by "ifdef soup".

ifdef soup:

```C
#ifdef thing
code1();
#else
code2();
#endif
```

macro way:
**something.h**:
```C
#ifdef thing
#define somefunc(a, b) difffunc(a, b)
#else
#define somefunc(a, b) otherfunc(a, b)
#endif
```

**used.c**:
```C
#include <ffmscompat.h>

somefunc(x, y);
```

See `ffmscompat.h`.

> The value is that resampling is made possible with playin ffmpeg. Libav comes usually with avresample but ffmpeg uses swresample. Those API's are pretty close, so the maintanance effort is not that high.

FFmpeg includes avresample regardless (although disabled by default). Seems kinda silly to me, but ultimately @myrsloik decides.


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


works great but the screen off gestures dont hide in settings on devices without device handler, any ideas?


No, I do not what the client to call unsafe functions (see doxygen).


Have you done `git pull` recently? I don't think this is a problem in the latest commits. If it is, please file an issue.


Oh, I see: you want to erase the current point. Would it not be cheaper to use a queue, or ```std::list```, and save logarithmic query and update time?


@dlonie that will work too!


If we always do the conversion to String, can we make `this.value` a String as well? That way we don't have to convert again in `toQuery()`


This should be a ```NumberType```


Стоит добавить ключ `-h` для вывода справки


LOL, it's not me~


If we convert to String, maybe also return String value here? Or was the reasoning that the user might expect back what he passed in?


Oops, I thought I had rewritten that commit message


Yeh, me too, though apparently it's needed for #74.


also, you are wastefully spending time by initializing ```newPoint``` with zeros, then performing arithmetic on it. It's more efficient to initialize it using ```std::vector<NumberType> newPoint(*iter);```, and then perform arithmetic on the entries


I added it to the uv-moreinforightpanel-module styles.less here: https://github.com/UniversalViewer/universalviewer/commit/bca44dd34c8458d94ee97b3539e7e3efffca731e


never experienced it or got a report, open an issue next time :)


Thanks @pdurbin for the comment. You are right about links, some of our users may not known all of these services. It would very helpful if you can create an issue about this we can discuss and address it quickly:
https://github.com/CenterForOpenScience/osf.io/issues/new

Messages left on the commit will not reach as many people but we check and work with issues daily so that's the best place to add these contributions.


Well, I just tried to find a reference on the web where MSVC produces ".a"-files. I could not find at least one example for this.
You said that MSVC produces lib files for shared-libs and .a files for static libs. That doesn't make much sense form a Microsoft point of view. Either they use propriatary .a-files or .lib-files.

I am sure I can just rename the lib files to a-files and VC will still link them together, but I haven't tested it yet. Still, this is not the standard behaviour for MSVC builds.


Nice, maybe the README could be edited to document this new feature ?


Sorry, which part won't work?


I thought it was. From perlpod, `Links to an absolute URL. For example, L<http://www.perl.org/> or L<The Perl Home Page|http://www.perl.org/>.`


@mbernat Don't entirely love it, but looking forward to all that code duplication I will have to do :-)


Thanks for the clarification. I hadn't understood the subtle nature of this one, and your answer clarified it.

When a "parent" SHA1 (in this case "master") is in the history of a "child" SHA1, remote polling can't detect that the "parent" has already been built because remote polling does not have any way of deducing the relationship between the "parent" and the "child".  Remote polling only sees the "parent" and the "child" as separate items.


My reference is that I am the person who worked on support in FFmpeg/Libav.


oh can you do it please? I gotta run and don't want to mess up something, never did shrinkwrap before


@tchap Obviously there are more than 2 things. You recently had to rename Error to Err, so there's another piece you're not mentioning here.

I still think what you're doing here is wrong. You really want to collect and report a list of errors, so you should create an ErrorList implementing the error interface and return that.


This is one of the things I was worried about with IIabVerifiable. The way you check `instanceof` in SoomlaStore is not something I like to do in code.

Please remove this interface and just add the necessary functions in IIabService.


The reformatting makes it harder to do parallel observation on the patterns, which were viewed one above the other before.  Not a big deal once the patterns have settled down and we're not looking at them by eye to see which ones are likely during debugging.


OK, I got this from the form widget though, that's how it is there.


In fact tremulous mapshots need overBrightBits 1, but some of them are normal…


TeamCity Xbim :: XbimGeometry :: XbimGeometry [Build 8](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=17&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


If there will be a lot of duplication, you can create a higher-level function `createOrGet` or whatever. However, you're calling it three times so far, so that's not any worse than modifying two functions with code that doesn't belong there ;)


If HTTP/2 is to be deployed over TLS, [implementations MUST use TLSv1.2 or higher](http://http2.github.io/http2-spec/#TLSUsage).

That said, hyper currently doesn't use anything provided by this feature and as such doesn't need to include it at all (I commented on this back on the OpenSSL removal PR and thought @seanmonstar fixed it before rolling out the new version).

Another matter entirely is whether hyper should allow even attempting to establish a supposedly TLS-backed HTTP/2 connection while using anything lower than TLSv1.2 (which it does allow right now), since it'd be against the spec...


This is wrong you need to flatter everything, you cannot pass a domain object into a DTO


This and obviously the original comment as well are misleading. The function works with any sqlite library version.

The sqlite library > 3.7.11 is a prerequisite to switch from the foreach-loop (which executes multiple sql inserts) to a single insert statement which inserts multiple lines at once. The later is used by handleMysqlUpsert().


Ok, I didn't see that for the menu bar on OS X, which is system wide.


Для объединения путей лучше использовать `os.path.join`


@caneruguz done! https://github.com/CenterForOpenScience/osf.io/issues/3345 Thanks!


it will not affect the usage of this method, `instanceof` checks if it's supported by the service, but the getter checks if developer turned in on


`detect()` the static method that calls the constructor.


@costin since `execute` can be called for 2 streams concurrently, this either needs to be synchronized or ref-swapped, otherwise this is a race condition that could cause losing tuples


### Missing semicolon.

```
  return "Hello, World!"
                        ^
```


### Use '===' to compare with 'null'.

```
  if (bla == null) return;
          ^
```


Правильнее писать иначе. Присвой результат вычисления условного выражения этой переменной


I'm sure you're aware, but in case you forget, you'll need to remove this before merge.


请问这里为什么要插入local 呢，我在local/share 中没有games 目录，修改之前的那个反而是有的


should be is not typesafe, while === is.


I know. I don't like using that. I only do instanceof when there's no other option and here it actually makes sense that the verification functions will be in IIabService b/c every IIabService has verification service.


Fixed.


no need for finally, the actor system shutdown cleans up anyway.


@harianus Of course not!


@mbernat Renamed to `getOrCreateMilestone...`, don't feel like wasting time on this. It's an internal function anyway. Changed in ae32e38.


Yes, I'm aware of all that. When I say only array I mean pretend that Go doesn't have nil. I'm sure you're using this assumption at lots of place since otherwise you'd have to check for nil every single value you use.
And when I say optional I mean you are allowed to use nil for that value.

So to summarize, you should always have an array for labels and put into the query if you want it changed and leave it null otherwise. And in case encoder doesn't support this, it should be replaced.


I looked at the tests, and this is probably the best thing to do, and in the right place.  Could you put it in a `sys.platform` if context?


Why not Sink.fold?


Логирование не должно затрагивать основной процесс вычисления. Вынеси инициализацию логгера из функций. Имя логгера тоже можно задавать вручную


I prefer the original text without this introductory line...I think the pledge is actually stronger as a pledge and not a contract.


Good Job!


This might've broken the spider aggro... not sure if it's this specifically but they no longer track players.


Может быть, `[2][1:-1]`?


why? now it is a too long line.


Fixing


check!


It's your choice not to use such design pattern, I'll remove the interface, but it does not affect the getter (`getVerifyPurchases`), do it? And I'll need to implement stubs for amazon in this case.


This line may cause Xcode to throw a "terminated due to memory error" message.


this can remain one line


Перед тем, как создавать базу, нарисуй её схему и покажи мне, чтобы можно было обсудить детали


Про `auto_increment` -- да, правильно


buuuhhhhh


There are longer/equally long lines in that file already. I'll fix those too


Can we also keep `code` for backwards compatibility?


Because fuck it


Two failed test cases for this commit:

(1) Is the ordering of symbols in PolyRatFun changed?
```
S x,y;
CF rat;
PolyRatFun rat;
L F = rat(x+y,x-y)+rat(x-y,x+y);
Print;
.end
```
Old answer:
```
   F =
      rat(2*x^2 + 2*y^2,x^2 - y^2);
```
New answer:
```
   F =
      rat(2*y^2 + 2*x^2, - y^2 + x^2);
```

(2) Some compiler changes for interpreting sets?
```
CF H;
S x;
L F = H(2);
id H(?a,x?!{1,0,-1},?b) = H(?a,0,x-sig_(x),?b);
P;
.end
```
New error:
```
foo.frm Line 4 --> Illegal position for {1,0,-1},?b)
```


It's some time since we made this update.  I recall that it was actually taken from a fix on another fork that was being developed at the time.

Basically it fixes the logic of the buffer stuffing when adding a sample.  Note also incorrect buffer size was originally allocated.

I hope this helps you.


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


Ok, I trust you on this.


This is a smart approach! :)


This is wrong, since if you create a subwiki "z" from "x.domain.com" which is you main wiki, the subwiki created will be "z.domain.com", when it should have been "z.x.domain.com".

Also, right now, wiki creation is only done from the main wiki (since that is where WikiManager.CreateWiki is located), so there really is no point in removing any subdomain from the hostname.

So probably the code should be `suffix = "." + hostname;`

Did you have a particular reason for subtracting the subdomain at the time?


Estuvo bien he estado ocupado con exposiciones ayer, asi que espero salir temprano hoy del examen de modelado de procesos laboratorio para hacer los ultimos retoques


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


good


Done


Finally an easy way to get data :smile:


Лучше не `one_struct_info`, а распаковать его сразу по переменным:
`mRNA_tuple, number, name_of_seq = get_...`


I only renamed `Error` to `Err` so that the interface can be embedded. Because you cannot do

```go
type MyError struct {
  errs.Error
}
```

because then you get a conflict with `error`'s `Error() string` method. Because doing `MyError.Error` wouldn't know if you want to call the method on the embedded object (`MyError.Error.Error()`) or just access `MyError.Error`. So I renamed `Error` to `Err` so that you can do

```go
type MyError struct {
  errs.Err
}
```

So no, there are no more pieces.

I am not remaking this into `ErrorList`, because that doesn't make sense. I want a recursive structure, not a list in this sense. I mean, it's already a list, because `Error` basically represents the error list head while its `Err` method returns the tail. But I think that we are spending way too much time on this BS :-)


@mbernat Check the previous comment pls.


It is not legal to use a Subscriber from multiple Publishers, and concurrent notifications are not allowed and an Async publisher needs to provide its own HBR between its thread-jumps so visibility of this synchronous Subscriber should be perfectly fine :)


I guess this could be removed, right?


Can this behaviour of settings the threadlocals be moved to a BaseFormAction?


True


@mbadams5 You've probably already seen it, but the unit tests are failing because of an unexpected preceding comma in the timestamp.


why remove the panic method?


Почему просто не создавать `forna_file = '{}/{}_({}).txt'.format(output_folder, name, origin)`?


In TTT when you start the first game of choosing who and innocent and who is traitor many suffer crash... This occurs on occasion and started after I installed the pointshop, I not report because the crash log is inaccurate but once time I remove the PS and no one had problems during that period...
There are many incompatibility issues with PAC3...


yes, good call


Because I prefer reusable flows, so to break out this from its sources and sinks preserves its behavior :)


@DiamondBond Einstein said it... Humans = stupids


The file doc/TRANSLATORS and many thousands of lines in ts files are deleted with this patch. This a mistake, isn't it?


remove the FIXME


Unhardcode was merged, what is missing from OCA/MQT now? Is there a PR?


As above, I like it better as a description of what we do than as a requirement of what *you* do....maybe like this???

```
The Shoes community follows the "have fun with Shoes" ethic. Our [code of conduct](https://github.com/shoes/shoes4/blob/master/CODE_OF_CONDUCT.md) guides our interactions in codebases, issue trackers, chat rooms, and mailing lists.
```


Because it is not reused. If all logic had to be its own method there'd be methods all over the place :) At least that was my reasoning here.


need to flip image vertically


all the logic is here


No TODOs please. Open a ticket instead.


pick a frame


I use trayhost to display a counter so I needed a way to update the icon. You can merge it.

If I recall correctly, I also had to export Cleanup() 063fee1ea77e6d0079d5d4efd28e065abf9fcf85 and use it to prevent the temp dir from filling up.

![image](https://cloud.githubusercontent.com/assets/24027/8378206/b6adc11a-1be5-11e5-8735-4b9cb5716a0e.png)


convert to internal opencv format


The reasoning for consistency here is the same as consistency in regards of curly brackets or whitespace. Variation breeds doubt.


`only when the path mode is used`
I think what you meant here was:
`only when the path mode is not used (i.e. domain-based)`


color transform (don't remember why I need it)


Is there a special reason why binaries are commited? Isn't it preferential to pull them in using NuGet?


@mbernat I agree that it would be better, but I am probably not writing like hundreds of lines of code just so that I don't have to check for nil here...


to get background - convert picture to white/black.
Then on next iteration algorithm will subtract previous white/black frame with current frame to get motion!


Interestingly, webpack works fine when requiring each JSON files individually (like the above). Although, the below doesn't. Webpack tries to load the entire "cldr-data" content (unnecessarily), which is huge.

```js
cldrData = require( "cldr-data" );

Globalize.load( cldrData( "main/en/ca-gregorian" ), ... );
```

Do you know what needs to be done to get webpack not pre-loading the entire "cldr-data" in my suggested example?


these few steps to remove small non-important movements, like wind blowing etc...


just for fun - ignore it.


drawing rect around moving objects


@mbernat And it's not the same as having `[]Type` set to nil. You can do range over an array that is nil, but you cannot deference a nil pointer without getting a panic. So range over `nil` of `[]Type` is the same as range over `[]` of `[]Type`, so mostly I don't need to be checking for anything...


I will now commit some changes, that will allow a 'stock' package for x86, just misses the last apps.


Unfortunatly not during MACRO evaluation (can access during runtime Catch::libraryVersion)


the zipping occurs three levels down in the `request` library. Our and Nano's methods should work transparently.


Not entirely. We're not building TWRP. Just tracking Omni's bootable_recovery, which has a lot of TWRP essentials in there. We just need these flags to satisfy some pre-reqs. Recovery isn't included in our builds.


Tickets are a good complement, but having to go to the issue tracker for every single section that looks weird is not feasible, useful nor helpful. :)


Thanks for the quick reply !

So far and for some reason, I could successfully run `cargo test` on *solicit* without issues, even with `--features openssl`.

*Hyper*, however, doesn't link anymore with the aforementioned error, apparently due to `solicit/openssl` in its [feature configuration](https://github.com/hyperium/hyper/blob/master/Cargo.toml#L44). If I remove it, everything works fine.

I am not sure if this should be fixed in *solicit*, or in *hyper*. From hyper's perspective, it just wants `solicit/openssl` and expects it to work on any platform.

A quick fix would be the said removal, which I have pushed [as PR](https://github.com/hyperium/hyper/pull/583). Maybe the latter will trigger further discussion.


Actually, this is fix for #139


this object keeps track of No. of obejcts went into/moved out.


@hoholee12 thx u :)
is needed ]; then & true; do? I thought that the original apply was with line jump & next the stuff xD...


lol yup :3


i personally don't like this blue hover

no such color present in eventum current styles.


Inserção do Projeto Completo


this is lib which helps to convert between processing and opencv image formats.


Well, there's two problems there, I suppose :)

The feature here is called `"tls"`, not `"openssl"`, so `"solicit/openssl"` seems to merely pull in the openssl required by this library :)


>What's worrying is that these were seemingly not checked by the checkin tests

DottyBackedInterface is being compiled by integration tests as part of junit test suite.


I put it into a method give it a name basically, reuse is not the only reason to make a new function :) Also, it was much more pleasure to write "panic()" in that section :D (for ordinary error handling I would not have introduced a method, but this was something really last resort)

If you prefer, it can stay inlined, but I will miss "panic".


This will not suffice as it only covers named expansions, consider `ex = '1 + x'`.  You will want `exprs[k] = '({0})'.format(ex)`.


@LPTK Can you investigate the problem?
It produces a match error in the compilation time.
There's a similar error in `HashMapPartitioningTransformer`:
https://github.com/epfldata/dblab/blob/cb1d5dca0269a1e86bd6ca2ecd4795c6cb775a57/lego-compiler/src/main/scala/ch/epfl/data/dblab/legobase/optimization/HashMapPartitioning.scala#L271


Spaces around `*`.


Check the copy/paste mistake in CloseAllProducts. It has "Close All Other Products" in the constructor


this is logic to detect largest contour around moving object.


calculating logic.


fixed crc datatype like suggested in issue #316


Was dealing with issues with btn-group-justified in AdminComponentNav.  You don't need navbar-right-btn-group on the navbar in your screenshot because there is no btn-group present, just a single button.  I've updated AdminComponentNav now, though, to ditch the btn-group-justified altogether:
https://github.com/DeltaSystems/dewdrop/commit/f78d2e054995b6be8d3dda902a879a1b83900ff3
It was a persistent source of little styling issues.


Ok, if you prefer it that way, so be it.


I wonder, could IodineGBA run on the 3DS web browser? (probably not, you'd use up all the script memory)


Hi vtloc,

can you just instruct me how to generate a docx file. I have got some function like addText() and addImage().  I want some more functions like addTable() which gets the input as html, addHeader(),addFooter() are also gets input as html. Main thing to do is that using css either internally or externally.

Please help...


Thanks.


Should this be: return (ResolutionError) cacheValue; ?


And in `solicit/Cargo.toml` it is declared as:
```toml
[dependencies.openssl]
version = "*"
features = ["tlsv1_2", "npn"]
optional = true
```
Does this automatically pull in the `tlsv1_2` and `npn` features ?


x


Should we put this new String constant somewhere else and document why we need it here? Maybe double check if this is really necessary in filter context or if we can simply keep String null in this case. Looking at the code in MultiTermQuery, leaving `null` here will later also map to CONSTANT_SCORE_REWRITE.


Did some investigation on this:

  * Pool.join is fine, because we have both requirement and implementation in place forbidding a host to join a pool with running VM on top of it. In this case, ha_always_run should always stay false.
  * DR import behavior will change a little, but IMHO in a good way. With the current DR implementation, to recover VMs on the 2nd site is a wizard guiding manual process. The user get to choose which VMs to recover/import and which of them to start. With ha_always_run equaling true on some VMs,  a user can encounter surprises when a VM is started autonomously on the backstage during the recover. This might even be contradict with the user's choice. Stripping ha_always_run will actually make the semantics simpler to follow given DR is a manual recovery process.

Last but not least, based on the current import logic in xapi, we always assume the imported VMs are in a dead (non-live, i.e. halted/suspended) status, as a crash-consistent snapshot. Even if the metadata was snapshoted when a VM was running hence powerstate=running, many places in the import logic will reset/treat the VM's power state as halted.  It feels reasonable to let the user decide whether/when to start (or resume) the VM after import.  Autonomous start immediately after the import, on the other hand, will surprise user and take away user's chances of doing any further tweaks before starting/resuming the VM. And it also gives us (xapi) a potential chance to do any further changes ourselves (like reset the flag to true or anything necessary) otherwise even we (xapi) lose the chance (due to the operation of xha).


its all personal preferences bro,
tho this style will effectively prevent you from leaving 'then's and 'fi's everywhere.XD


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


See above, maybe we can avoid introducing new option for rewrite here.


Fortunately XenCenter has not taken advantage of the "relax" I made, so I can just revert part of this changeset to recover the old behavior.


you don't need to delete stuff that's commented out - it could be useful again later, and I would prefer to leave it


@Parbhat @lendl this is a bit confusing.

If they don't supply a map key show a static map of ChrisDev.

If they supply the map key we need to have a static map that links to a dynamic map.

We need to explain this in the documentation.
The map on the contact page need to work like this also.
Wondering if it make more sense to do this with mapbox instead. Will make the cookiecutter automation  less complicated


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


I guess in this case it is quite clear what the code does :) And these TODOs are especially annoying when I know that I won't be able to touch them in 2-4 weeks.


You told me you liked `I18n.t!`!


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


This one gives me trouble: "Fatal error: Can't use method return value in write context"
It seems I have to specify a place where apply_filters can store its output. Can be [$anyfield]. If I omit [$field] entirely the result output is 1.


hm... do you think this should ever be configurable per installation? Can the Bus perhaps run on another host?


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


Sure it does, but those are definitely required by what I want to provide by pulling in openssl here, though... As I say, the spec makes it absolutely clear that compliant agents must use TLS v1.2 so if the platform doesn't support it, it can't really make use of the optional feature `"tls"` exposed here. (And NPN as a stop-gap that most servers support instead of the spec-mandated ALPN.)

Again, though, when we were working on the HTTP/2 feature in hyper, there was a requirement (stemming from servo's requirements, I believe) to allow the HTTP/2 client to use the custom connector machinery from hyper, which includes handling SSL/TLS, so this feature shouldn't even be included there.


@jeromedenis ok with you?


What kind of recursive problems do you mean? Also, what is this a workaround for? I'm still against having it be a hidden directory.


can you reword this commit to `docs(README.md): updated API description` please? I know these commit message guidelines really are confusing ...


Yes... like the other constructor, there's no verification if the device passed actually makes sense. It shows the same behavior as if we passed a non existent "/dev/i2c-200". The behavior will be to fail the first time we try to access it by calling `LinuxI2CDriver::begin()`.

I can change both constructors to fail earlier if the device was not found.


that yes xD... & one question... what do exactly bb generator? I saw it & do not get it lol... & is mother fucking 3 kB xDd


@patriciogonzalezvivo This is really interesting.... how are you making the zoom change? I'm trying to figure out what in the map zoom is changing history - maybe in the hash change, but that's not expected...


Should we remove some or just stop add them?


Examples of looping through a folder and doing something with every file


@DarkDimius Yes, I just verified that. So it was probably a compilation order problem.


Forgot to add braces while adding a line to the if body. Just happened to spot it while stalking.


you don't need to comment out notes - it's better to leave them so that everyone knows who did what...


probably should be a minor? http://semver.org/

Not to be a stickler or anything like that on a Friday :dancers:


LGTM


Thanks


Please leave comments and notes alone - they provide valuable information


@cfitz - no, not a problem. I think I came to the same conclusion. Thanks.


noted

On 6/26/15 8:47 AM, Daniel S. Katz wrote:
> you don't need to delete stuff that's commented out - it could be useful
> again later, and I would prefer to leave it
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielskatz/WSSSPE/commit/fb108913308026d1e3cd4eeaff22f0150aae2f0c#commitcomment-11882698>.
>


Why dark bridge and slime light entries are doubled? They are completely same.


Thanks!


Oh, you meant we will submit to arxiv/JORS with these notes on Monday?

On 6/26/15 8:53 AM, Daniel S. Katz wrote:
> you don't need to comment out notes - it's better to leave them so that
> everyone knows who did what...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielskatz/WSSSPE/commit/7ce0643880c210b142ce90968abe4188b3309a28#commitcomment-11882825>.
>


So the rule for gen1 was already there. But why these two entries for Edison?


The Flickr endpoint doesn't seen to specify these three fields.


In theory maybe, but given the old code does not throw any exception, the new code does not return it either.


make sense - will re-add ;)


Can we test this?


Is this getting reverted as a duplicate?


As a bonus, when you purchase a LifeSpan TR1200i treadmill, you automatically receive a membership to the LifeSpan Fitness Club. This comprehensive online health management program, powered by the Interactive Health Partner, lets you select exercise programs, track and manage your fitness and health results, and receive motivating support as you work toward achieving your goals. The LifeSpan Fitness Club is easy to use: each time you work out on the TR1200i, the treadmill automatically saves your exercise results on your LifeSpan . Because the device is personalized with your age, gender, and weight, not only do you bypass setup screens, you also receive accurate calorie calculations based on your data. Every 20 seconds during your workout, the console will then record your heart rate, minutes, miles, steps, and calories. You'll then have access to a full set of detailed reports and fun facts that you can analyze or share with friends, family, fitness coaches, or physicians.


@etagwerker @mauro-oto
I refactored the admin bar. Let me know if it is ok or not, so I can work on the others. Thanks!


Just wondering what benefits the OpenHashMap has over the Java 5 ConcurrentHashMap or other built-in collections. The util package contains a number of collections that might be better removed now that we build on top of Java 5 since that will save us the maintenance of those fairly complicated classes.


Hi, derekpetersen. I rebuild the MainActivity.java in Android studio. But it came out a warning said that  "This Handler class should be static or leaks might occur Android......", when declaring class handler as followed:
private final Handler handler = new Handler() {
}
What should I do to this warning? Thank You！

private final Handler handler = new Handler() {

		public void handleMessage(Message msg) {
			switch (msg.what) {
			case MESSAGE_DEVICE_NAME:
				Toast.makeText(getApplicationContext(),
						msg.getData().getString(DEVICE_NAME),
						Toast.LENGTH_SHORT).show();
				break;
			case MESSAGE_TOAST:
				Toast.makeText(getApplicationContext(),
						msg.getData().getString(TOAST), Toast.LENGTH_SHORT)
						.show();
				break;
			case MESSAGE_WRITE:
				/* NOP */
				break;
			case MESSAGE_READ:
				byte[] bytes = (byte[]) msg.obj;
				if (bytes[0] == 'G') {
					byte[] temp = new byte[2];
					temp[0] = buttonState.getState();
					temp[1] = '\r';
					send(temp);
				}
				break;

			case MESSAGE_STATE_CHANGE:
				BGState state = (BGState) msg.obj;
				TextView s = (TextView) findViewById(R.id.textView3);
				s.setText(state.toString());
				break;

			default:
				Log.v(TAG, "UNHANDLED");
			}
		}


We can just change the macro definition at the top to turn off all the notes.


Why is this commit not merged into the 1.6 branch yet?


Indentation is wrong.


@Kirek I just merged your contribution, with some changes to implement the option from partition.conf. Please give it a spin and let me know if it satisfies your distro's requirements.


Hi, derekpetersen. I rebuild the MainActivity.java in Android studio. But it came out a warning said that "This Handler class should be static or leaks might occur Android......", when declaring class handler as followed:
private final Handler handler = new Handler() {
}
What should I do to this warning? Thank You！

private final Handler handler = new Handler() {


Test


instead of having boolean flags, you may use a Set<StandardWatchEventKinds>, in this way you may get rid of all switches, also maybe you may use dirrectly the StandardWatchEventKinds not duplicating them with Events


do {} while (0) on both?


Hi, it's reported, please have a look at https://github.com/PrestaShop/PrestaShop/pull/3269
Best regards


Interesting! But a problem with these dangling notes is they take up
spaces and I can't format the document shape accurately.

On 6/26/15 8:57 AM, Daniel S. Katz wrote:
> We can just change the macro definition at the top to turn off all the
> notes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielskatz/WSSSPE/commit/7ce0643880c210b142ce90968abe4188b3309a28#commitcomment-11882925>.
>


36行目単行ifはバグの元凶ですので，括弧で囲ってください．
あと，15行目，テンプレートリテラルでかけるんじゃないですかね（V8だけかも）？
http://js-next.hatenablog.com/entry/2014/11/22/042055


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


I can add a witty comment :)


fgfg


@milipili There are only two ways to accomplish this: an additional program, or by using [COM](https://msdn.microsoft.com/en-us/library/ms679687%28VS.85%29.aspx). I consider an additional program to be the easiest way, by far. I agree that using pipes would be better, but the patch was rejected for being too complex so it won't be helped by adding more complex IPC...!


@cecilia-ombulabs Looks good!


Feels a little awkward to use, the problem here is for modification we would always need to be passing in all credentials but we don't have a way to obtain them all.


TODOs are meant to be annoying so people feel compelled to fix them :)


@Myoldmopar Yeah I know. I forgot to initialize a variable in my unit test and i've already fixed the newline for windows as well.


It is a bit bright. Feel free to adjust the color to something better.


deal


Would be nice if this could be called after the credentials and roles have been set so no intermediate identity ever needs to exist.


I don't think we need to be too concerned with the document shape - JORS will reformat it for us.

If it really matters, you can comment out the notes so they are still in the source but not in the pdf.


Nice, was this tested?


It was not that simple, the following patch fixes it but i think there is a deeper issue:
´´´
diff --git a/rts/Rendering/Models/WorldObjectModelRenderer.cpp b/rts/Rendering/Models/WorldObjectModelRenderer.cpp
index b985086..9d286f6 100644
--- a/rts/Rendering/Models/WorldObjectModelRenderer.cpp
+++ b/rts/Rendering/Models/WorldObjectModelRenderer.cpp
@@ -133,11 +133,11 @@ void IWorldObjectModelRenderer::DelFeature(const CFeature* f)
        FeatureSet& fs = features[TEX_TYPE(f)];

        auto it = std::find_if(fs.begin(), fs.end(), [f](const PDrawFeature& p){ return p.first == f;});
-       assert(it != fs.end());
-
-       *it = fs.back();
-       fs.pop_back();
-       numFeatures -= 1;
+       if (it != fs.end()) {
+               *it = fs.back();
+               fs.pop_back();
+               numFeatures -= 1;
+       }
 }
´´´


I think you can just leave this whole part out (lines 920 - 930).
And on line 941, you can replace `$this->endtime_html[ $field ]` with `$this->endtime()`.


It's a HArray.
On 25 Jun 2015 22:36, "wren gayle romano" <notifications@github.com> wrote:

> Is that a Haskell-list or a Hakaru-list/array? Based on the old Syntax.hs,
> I assumed 'HArray 'HProb... If it's a Haskell-list, then that means
> end-users can't have it's length depend on the results of some Hakaru
> program/expression (only the elements of the list could be)... which
> doesn't seem right...
>
> As for the HNat vs HInt, I set the PrimOp up to be HNat (as we discussed;
> cf., line 388/408). The comment was just because the cut-and-paste of the
> old default definition causes Haskell to infer HInt— though it does
> highlight a limitation of the current coercion machinery (we can only
> sample and then coerce the result, not coerce the distribution itself; this
> is the same as in the old Syntax.hs).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hakaru-dev/hakaru/commit/2f7060edb51ffcdd8bc484ab5cb5a60c11c39f93#commitcomment-11874586>
> .
>


I agree on the int32_t type here.


If we have this Iterator not sure if we should have something with some form of pagination or filtering.  A large realm could be hard to manipulate with the management model and we may not want to force the realm to load it all at once.


Is it possible to merge this commit, but not the previous one?


@chrisdev yes we have to explain in documentation. So which approach we should use this one on footer or  contact page map.


@jakubfiala any chance of documenting these bugs? Happy to work on them myself now.


I was zooming with the track pad... the bug was discovered by John. Seams the zooming was trigger a event that reloads the style.


Nice, thanks! Please submit a pull request :)


Is this going to be more hassle than it is worth?


We could meet half way to insert the Google Notes in section title. Is
it okay?

I am not sure how to reply to a specific change. But the following one
means "research" is appearing thrice in the same sentence. Very clumsy.

"-community by cultivating better, more sustainable software to enable
+community by cultivating better, more sustainable research software to
enable

On 6/26/15 9:03 AM, Daniel S. Katz wrote:
> I don't think we need to be too concerned with the document shape - JORS
> will reformat it for us.
>
> If it really matters, you can comment out the notes so they are still in
> the source but not in the pdf.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielskatz/WSSSPE/commit/7ce0643880c210b142ce90968abe4188b3309a28#commitcomment-11883040>.
>


This got me thinking - is everything involved threadsafe? On the outside things seem to be ok. The ResolveContext is threadsafe by spec. The ResolverImpl contains hardly any state and what it contains seems ok. However, I did not fully look at how doResolve hands pieces of work to the executors. I did see some code in the resolver that didn't seem threadsafe (e.g. non-volatile member) but if these work on entirely different data structures that could be ok.


it has lots of features, tho the main one would be making sure that all applets are executed from busybox.


True. Sent a pr with a cleanup on method names in main.py.
Low priority, can wait until after the release https://github.com/sugarlabs/sugar/pull/543


How is `-d` related to downgrading?


I don't think this is the case anymore - we work well on chrome.


1: M111 unrecognized component name: legacy


29:1: E402 module level import not at top of file
31:1: E402 module level import not at top of file
33:1: E402 module level import not at top of file
34:1: E402 module level import not at top of file
43:1: E402 module level import not at top of file
44:1: E402 module level import not at top of file
46:1: E402 module level import not at top of file
47:1: E402 module level import not at top of file
48:1: E402 module level import not at top of file
51:1: E402 module level import not at top of file
52:1: E402 module level import not at top of file
53:1: E402 module level import not at top of file
54:1: E402 module level import not at top of file
55:1: E402 module level import not at top of file
59:1: E402 module level import not at top of file
61:1: E402 module level import not at top of file
146: D210 No whitespaces allowed surrounding docstring text
398: D210 No whitespaces allowed surrounding docstring text


Ack.


40:1: I100 Imports statements are in the wrong order. from zlib should be before from six.moves


34:1: I100 Imports statements are in the wrong order. from random should be before from invenio.utils.hash


26:1: F401 'CFG_DEVEL_SITE' imported but unused
80: D102 Missing docstring in public method
109: D209 Multi-line docstring closing quotes should be on a separate line
109: D203 1 blank line required before class docstring (found 0)
109: D400 First line should end with a period (not 'f')
109: D205 1 blank line required between summary line and description (found 0)
115: D102 Missing docstring in public method
163: D102 Missing docstring in public method
167:5: E303 too many blank lines (2)
170: D210 No whitespaces allowed surrounding docstring text
170: D209 Multi-line docstring closing quotes should be on a separate line
170: D203 1 blank line required before class docstring (found 0)
170: D400 First line should end with a period (not 'l')
170: D205 1 blank line required between summary line and description (found 0)
198: D102 Missing docstring in public method
222: D102 Missing docstring in public method


23:1: E402 module level import not at top of file
25:1: I100 Imports statements are in the wrong order. import re should be before from datetime
25:1: E402 module level import not at top of file
26:1: E402 module level import not at top of file
27:1: I100 Imports statements are in the wrong order. import copy should be before import zlib
27:1: E402 module level import not at top of file
28:1: E402 module level import not at top of file
29:1: E402 module level import not at top of file
30:1: I100 Imports statements are in the wrong order. import cookielib should be before import urllib2
30:1: E402 module level import not at top of file
31:1: E402 module level import not at top of file
32:1: E402 module level import not at top of file
34:1: E402 module level import not at top of file
35:1: I201 Missing newline before sections or imports.
35:1: E402 module level import not at top of file
37:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.modules
37:1: E402 module level import not at top of file
38:1: E402 module level import not at top of file
39:1: F401 'auto_version_url' imported but unused
39:1: E402 module level import not at top of file
40:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord.xmlmarc2textmarc should be before from invenio.utils.url
40:1: E402 module level import not at top of file
42:1: I101 Imported names are in the wrong order. Should be CFG_BIBCATALOG_SYSTEM, CFG_BIBCATALOG_SYSTEM_RT_URL, CFG_BIBEDIT_AJAX_RESULT_CODES_REV, CFG_BIBEDIT_AUTOCOMPLETE, CFG_BIBEDIT_SHOW_HOLDING_PEN_REMOVED_FIELDS, CFG_SITE_LANG
42:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibrecord.xmlmarc2textmarc
42:1: E402 module level import not at top of file
48:1: I101 Imported names are in the wrong order. Should be deactivate_cache, delete_hp_change, delete_related_holdingpen_changes, get_hp_update_xml, get_info_of_record_revision, get_marcxml_of_record_revision, get_name_tags_all, get_record_last_modification_date, get_record_revision_author, get_record_revisions, get_related_hp_changesets, reserve_record_id
48:1: E402 module level import not at top of file
55:1: I101 Imported names are in the wrong order. Should be InvalidCache, add_record_cnum, cache_exists, cache_expired, can_record_have_physical_copies, create_cache, crossref_process_template, delete_cache, extend_record_with_template, get_affiliation_for_paper, get_bibrecord, get_cache_contents, get_cache_mtime, get_field_templates, get_marcxml_of_revision, get_new_ticket_RT_info, get_record_revision_authors, get_record_revision_timestamps, get_record_template, get_record_templates, get_xml_from_textmarc, latest_record_revision, merge_record_with_template, modify_record_timestamp, record_is_conference, record_locked_by_other_user, record_locked_by_queue, record_locked_by_user_details, record_revision_exists, record_xml_output, replace_references, revision_to_timestamp, save_xml_record, timestamp_to_revision, touch_cache, update_cache_contents
55:1: E402 module level import not at top of file
70:1: I101 Imported names are in the wrong order. Should be create_field, create_record, field_add_subfield, field_get_subfield_values, print_rec, record_add_field, record_add_fields, record_add_subfield_into, record_delete_field, record_delete_subfield_from, record_extract_dois, record_get_field_instances, record_get_field_values, record_get_subfields, record_modify_controlfield, record_modify_subfield, record_move_fields, record_move_subfield, record_order_subfields, record_replace_field, record_strip_controlfields, record_strip_empty_fields, record_strip_empty_volatile_subfields
70:1: E402 module level import not at top of file
81:1: I101 Imported names are in the wrong order. Should be CFG_BIBEDIT_INTERNAL_DOI_PROTECTION_LEVEL, CFG_BIBEDIT_KB_SUBJECTS, CFG_BIBEDIT_PROTECTED_FIELDS, CFG_BIBUPLOAD_INTERNAL_DOI_PATTERN, CFG_CERN_SITE, CFG_INSPIRE_SITE, CFG_SITE_RECORD, CFG_SITE_URL
81:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibrecord
81:1: F401 'CFG_BIBEDIT_PROTECTED_FIELDS' imported but unused
81:1: E402 module level import not at top of file
85:1: I101 Imported names are in the wrong order. Should be guess_primary_collection_of_a_record, perform_request_search, record_exists
85:1: E402 module level import not at top of file
87:1: E402 module level import not at top of file
88:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcatalog.api should be before from invenio.legacy.webuser
88:1: E402 module level import not at top of file
89:1: E402 module level import not at top of file
90:1: E402 module level import not at top of file
91:1: E402 module level import not at top of file
92:1: I101 Imported names are in the wrong order. Should be show_diff, wash_for_xml
92:1: E402 module level import not at top of file
93:1: I100 Imports statements are in the wrong order. from invenio.modules.knowledge.api should be before from invenio.utils.text
93:1: E402 module level import not at top of file
94:6: E126 continuation line over-indented for hanging indent
96:1: I100 Imports statements are in the wrong order. from invenio.legacy.refextract.api should be before from invenio.modules.knowledge.api
96:1: E402 module level import not at top of file
97:36: E126 continuation line over-indented for hanging indent
100:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.refextract.api
100:1: E402 module level import not at top of file
101:1: I101 Imported names are in the wrong order. Should be CrossrefError, get_marcxml_for_doi
101:1: E402 module level import not at top of file
103:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.utils.crossref
103:1: E402 module level import not at top of file
114: D103 Missing docstring in public function
119: D400 First line should end with a period (not ':')
119: D205 1 blank line required between summary line and description (found 0)
126:17: E126 continuation line over-indented for hanging indent
129:22: E203 whitespace before ':'
130:23: E203 whitespace before ':'
131:23: E203 whitespace before ':'
132:28: E203 whitespace before ':'
133:33: E203 whitespace before ':'
134:13: E126 continuation line over-indented for hanging indent
135:23: E203 whitespace before ':'
136:30: E203 whitespace before ':'
138:33: E203 whitespace before ':'
139:22: E203 whitespace before ':'
140:24: E203 whitespace before ':'
141:13: E126 continuation line over-indented for hanging indent
143:1: E302 expected 2 blank lines, found 1
143: D400 First line should end with a period (not 's')
143: D205 1 blank line required between summary line and description (found 0)
157:9: E115 expected an indented block (comment)
159:21: E126 continuation line over-indented for hanging indent
159:27: E203 whitespace before ':'
160:34: E203 whitespace before ':'
161:37: E203 whitespace before ':'
162:26: E203 whitespace before ':'
163:28: E203 whitespace before ':'
164:17: E121 continuation line under-indented for hanging indent
167:21: E126 continuation line over-indented for hanging indent
169:26: E203 whitespace before ':'
170:27: E203 whitespace before ':'
171:27: E203 whitespace before ':'
172:32: E203 whitespace before ':'
173:37: E203 whitespace before ':'
177:1: E302 expected 2 blank lines, found 1
179:11: E221 multiple spaces before operator
196:26: E251 unexpected spaces around keyword / parameter equals
196:34: E251 unexpected spaces around keyword / parameter equals
196:26: E221 multiple spaces before operator
197:25: E251 unexpected spaces around keyword / parameter equals
197:34: E251 unexpected spaces around keyword / parameter equals
197:25: E221 multiple spaces before operator
198:27: E251 unexpected spaces around keyword / parameter equals
198:34: E251 unexpected spaces around keyword / parameter equals
198:27: E221 multiple spaces before operator
199:29: E251 unexpected spaces around keyword / parameter equals
199:34: E251 unexpected spaces around keyword / parameter equals
199:29: E221 multiple spaces before operator
200:24: E251 unexpected spaces around keyword / parameter equals
200:34: E251 unexpected spaces around keyword / parameter equals
200:24: E221 multiple spaces before operator
201:29: E251 unexpected spaces around keyword / parameter equals
201:34: E251 unexpected spaces around keyword / parameter equals
201:29: E221 multiple spaces before operator
202:29: E251 unexpected spaces around keyword / parameter equals
202:34: E251 unexpected spaces around keyword / parameter equals
202:29: E221 multiple spaces before operator
203:32: E251 unexpected spaces around keyword / parameter equals
203:34: E251 unexpected spaces around keyword / parameter equals
204:24: E251 unexpected spaces around keyword / parameter equals
204:34: E251 unexpected spaces around keyword / parameter equals
204:24: E221 multiple spaces before operator
207:5: E303 too many blank lines (2)
241:32: E203 whitespace before ':'
242:33: E203 whitespace before ':'
243:27: E203 whitespace before ':'
243:66: E221 multiple spaces before operator
244:35: E203 whitespace before ':'
255: D210 No whitespaces allowed surrounding docstring text
305:13: F821 undefined name 'bibedit_templates'
306:13: F821 undefined name 'bibedit_templates'
317: D400 First line should end with a period (not 'h')
317: D205 1 blank line required between summary line and description (found 0)
327: D400 First line should end with a period (not 'D')
327: D205 1 blank line required between summary line and description (found 0)
347: D202 No blank lines allowed after function docstring (found 1)
347: D400 First line should end with a period (not 's')
381:17: F821 undefined name 'bibedit_templates'
382:49: E128 continuation line under-indented for visual indent
385:1: E302 expected 2 blank lines, found 1
385: D400 First line should end with a period (not 'r')
385: D205 1 blank line required between summary line and description (found 0)
406: D103 Missing docstring in public function
409:5: B901 blind except: statement
431:9: E128 continuation line under-indented for visual indent
432:9: E128 continuation line under-indented for visual indent
432:9: E129 visually indented line with same indent as next logical line
457:84: E231 missing whitespace after ','
499:1: E302 expected 2 blank lines, found 1
499: D210 No whitespaces allowed surrounding docstring text
499: D400 First line should end with a period (not 's')
549: D400 First line should end with a period (not 'f')
549: D205 1 blank line required between summary line and description (found 0)
582:13: E303 too many blank lines (2)
594: D400 First line should end with a period (not 'r')
594: D205 1 blank line required between summary line and description (found 0)
774:75: E502 the backslash is redundant between brackets
878:25: E127 continuation line over-indented for visual indent
894:53: E261 at least two spaces before inline comment
895:52: E114 indentation is not a multiple of four (comment)
895:52: E116 unexpected indentation (comment)
903:61: E126 continuation line over-indented for hanging indent
910:25: E126 continuation line over-indented for hanging indent
965:17: E128 continuation line under-indented for visual indent
1016: D210 No whitespaces allowed surrounding docstring text
1016: D401 First line should be in imperative mood ('Perform', not 'Performs')
1016: D400 First line should end with a period (not 'h')
1016: D205 1 blank line required between summary line and description (found 0)
1035: D202 No blank lines allowed after function docstring (found 1)
1035: D400 First line should end with a period (not 'g')
1035: D205 1 blank line required between summary line and description (found 0)
1083:13: B901 blind except: statement
1091:13: B901 blind except: statement
1140:21: E128 continuation line under-indented for visual indent
1173:67: E114 indentation is not a multiple of four (comment)
1175:44: E126 continuation line over-indented for hanging indent
1186:44: E261 at least two spaces before inline comment
1188:42: E261 at least two spaces before inline comment
1189:43: E114 indentation is not a multiple of four (comment)
1189:43: E116 unexpected indentation (comment)
1193:17: E128 continuation line under-indented for visual indent
1199:21: E128 continuation line under-indented for visual indent
1204:19: E128 continuation line under-indented for visual indent
1209:13: E128 continuation line under-indented for visual indent
1209:46: E251 unexpected spaces around keyword / parameter equals
1210:13: E122 continuation line missing indentation or outdented
1213:13: E128 continuation line under-indented for visual indent
1219:13: E128 continuation line under-indented for visual indent
1222:42: E203 whitespace before ','
1230:17: E128 continuation line under-indented for visual indent
1237:18: E261 at least two spaces before inline comment
1240:17: E128 continuation line under-indented for visual indent
1255:29: E128 continuation line under-indented for visual indent
1263:33: E128 continuation line under-indented for visual indent
1289: D400 First line should end with a period (not 'e')
1289: D205 1 blank line required between summary line and description (found 0)
1303:9: E116 unexpected indentation (comment)
1376: D200 One-line docstring should fit on one line with quotes (found 3)
1390:21: E128 continuation line under-indented for visual indent
1405:25: E122 continuation line missing indentation or outdented
1406:25: E122 continuation line missing indentation or outdented
1407:25: E122 continuation line missing indentation or outdented
1480:38: E128 continuation line under-indented for visual indent
1500: D401 First line should be in imperative mood ('Add', not 'Adds')
1500: D400 First line should end with a period (not ')')
1500: D205 1 blank line required between summary line and description (found 0)
1516:36: E127 continuation line over-indented for visual indent
1517:36: E127 continuation line over-indented for visual indent
1522: D400 First line should end with a period (not 's')
1539:9: E128 continuation line under-indented for visual indent
1540:13: E128 continuation line under-indented for visual indent
1541:9: E128 continuation line under-indented for visual indent
1546: D400 First line should end with a period (not 'l')
1546: D205 1 blank line required between summary line and description (found 0)
1564:5: B901 blind except: statement
1587: D210 No whitespaces allowed surrounding docstring text
1587: D202 No blank lines allowed after function docstring (found 1)
1587: D400 First line should end with a period (not 'd')
1629:5: B901 blind except: statement
1649: D210 No whitespaces allowed surrounding docstring text
1649: D400 First line should end with a period (not 'g')
1649: D200 One-line docstring should fit on one line with quotes (found 3)
1681: D210 No whitespaces allowed surrounding docstring text
1681: D400 First line should end with a period (not 'F')
1681: D200 One-line docstring should fit on one line with quotes (found 2)
1693: D210 No whitespaces allowed surrounding docstring text
1693: D400 First line should end with a period (not 't')
1693: D200 One-line docstring should fit on one line with quotes (found 2)
1705:13: E126 continuation line over-indented for hanging indent
1710: D210 No whitespaces allowed surrounding docstring text
1710: D400 First line should end with a period (not 'o')
1710: D205 1 blank line required between summary line and description (found 0)
1725:13: E128 continuation line under-indented for visual indent
1726:29: E127 continuation line over-indented for visual indent
1732: D401 First line should be in imperative mood ('Return', not 'Returns')
1732: D400 First line should end with a period (not 't')
1749:26: E128 continuation line under-indented for visual indent
1750:26: E124 closing bracket does not match visual indentation
1765:38: E128 continuation line under-indented for visual indent
1777:5: E303 too many blank lines (2)
1779:1: E266 too many leading '#' for block comment
1781:1: E302 expected 2 blank lines, found 1
1781: D400 First line should end with a period (not 's')
1833: D210 No whitespaces allowed surrounding docstring text
1833: D400 First line should end with a period (not 'e')
1846: D209 Multi-line docstring closing quotes should be on a separate line
1846: D400 First line should end with a period (not 'e')
1846: D205 1 blank line required between summary line and description (found 0)
1859:5: B901 blind except: statement
1866: D210 No whitespaces allowed surrounding docstring text
1866: D400 First line should end with a period (not 'e')
1871: D103 Missing docstring in public function
1908: D103 Missing docstring in public function
1921:45: E131 continuation line unaligned for hanging indent


24:1: E402 module level import not at top of file
25:1: E402 module level import not at top of file
26:1: E402 module level import not at top of file
27:1: I101 Imported names are in the wrong order. Should be CFG_BIBSCHED_LOGDIR, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_PYLIBDIR
27:1: I201 Missing newline before sections or imports.
27:1: E402 module level import not at top of file