37:1: I201 Missing newline before sections or imports.
145: D102 Missing docstring in public method
154: D400 First line should end with a period (not 'n')
154: D205 1 blank line required between summary line and description (found 0)
243: D102 Missing docstring in public method
283: D400 First line should end with a period (not 'w')
283: D205 1 blank line required between summary line and description (found 0)
296: D400 First line should end with a period (not 'e')
296: D205 1 blank line required between summary line and description (found 0)
311: D400 First line should end with a period (not 'e')
311: D205 1 blank line required between summary line and description (found 0)
319: D400 First line should end with a period (not 'e')
319: D205 1 blank line required between summary line and description (found 0)
330: D400 First line should end with a period (not 'd')
330: D205 1 blank line required between summary line and description (found 0)
352: D102 Missing docstring in public method
356: D102 Missing docstring in public method
368: D400 First line should end with a period (not 'd')
391: D400 First line should end with a period (not 'n')
391: D200 One-line docstring should fit on one line with quotes (found 3)
398: D400 First line should end with a period (not 't')
398: D200 One-line docstring should fit on one line with quotes (found 3)
405: D400 First line should end with a period (not 'e')
405: D200 One-line docstring should fit on one line with quotes (found 3)
412: D102 Missing docstring in public method
434: D102 Missing docstring in public method
442: D102 Missing docstring in public method
446: D210 No whitespaces allowed surrounding docstring text
446: D400 First line should end with a period (not ')')
451: D210 No whitespaces allowed surrounding docstring text
451: D400 First line should end with a period (not 'd')
458: D210 No whitespaces allowed surrounding docstring text
458: D400 First line should end with a period (not 't')
474:17: W503 line break before binary operator
488: D102 Missing docstring in public method
502: D102 Missing docstring in public method
523: D210 No whitespaces allowed surrounding docstring text
523: D400 First line should end with a period (not 's')
529: D102 Missing docstring in public method
536: D210 No whitespaces allowed surrounding docstring text
536: D400 First line should end with a period (not 's')
541: D210 No whitespaces allowed surrounding docstring text
541: D400 First line should end with a period (not 's')
546: D210 No whitespaces allowed surrounding docstring text
546: D400 First line should end with a period (not 's')
551: D210 No whitespaces allowed surrounding docstring text
551: D400 First line should end with a period (not 's')
556: D400 First line should end with a period (not 'r')
556: D205 1 blank line required between summary line and description (found 0)
569: D400 First line should end with a period (not 'e')
614: D102 Missing docstring in public method
619: D102 Missing docstring in public method
630: D102 Missing docstring in public method
637: D102 Missing docstring in public method
643: D102 Missing docstring in public method
649: D102 Missing docstring in public method
654: D102 Missing docstring in public method
661: D210 No whitespaces allowed surrounding docstring text
661: D400 First line should end with a period (not 'e')
665: D210 No whitespaces allowed surrounding docstring text
665: D400 First line should end with a period (not 'e')
669: D210 No whitespaces allowed surrounding docstring text
669: D400 First line should end with a period (not 'e')
673: D210 No whitespaces allowed surrounding docstring text
673: D400 First line should end with a period (not 'e')
678: D203 1 blank line required before class docstring (found 0)
678: D204 1 blank line required after class docstring (found 0)
678: D400 First line should end with a period (not 'r')
678: D205 1 blank line required between summary line and description (found 0)
684: D102 Missing docstring in public method
689: D200 One-line docstring should fit on one line with quotes (found 3)
705: D102 Missing docstring in public method
710: D210 No whitespaces allowed surrounding docstring text
710: D400 First line should end with a period (not 'n')
718: D210 No whitespaces allowed surrounding docstring text
718: D400 First line should end with a period (not 'n')
722: D210 No whitespaces allowed surrounding docstring text
722: D400 First line should end with a period (not 'n')
728: D200 One-line docstring should fit on one line with quotes (found 3)
734: D400 First line should end with a period (not 's')
734: D200 One-line docstring should fit on one line with quotes (found 3)
747: D203 1 blank line required before class docstring (found 0)
747: D204 1 blank line required after class docstring (found 0)
747: D400 First line should end with a period (not 'm')
747: D200 One-line docstring should fit on one line with quotes (found 3)
751: D102 Missing docstring in public method
762: D102 Missing docstring in public method
770: D102 Missing docstring in public method
781: D102 Missing docstring in public method
784: D102 Missing docstring in public method
787: D102 Missing docstring in public method
794: D200 One-line docstring should fit on one line with quotes (found 3)
800: D200 One-line docstring should fit on one line with quotes (found 3)
810: D200 One-line docstring should fit on one line with quotes (found 3)
873:13: W503 line break before binary operator
943: D400 First line should end with a period (not 's')
951:9: E731 do not assign a lambda expression, use a def
966: D203 1 blank line required before class docstring (found 0)
966: D204 1 blank line required after class docstring (found 0)
966: D400 First line should end with a period (not 't')
973: D102 Missing docstring in public method
995: D102 Missing docstring in public method
999: D102 Missing docstring in public method
1003: D102 Missing docstring in public method
1008: D102 Missing docstring in public method
1012: D102 Missing docstring in public method
1016: D102 Missing docstring in public method
1023: D400 First line should end with a period (not 'd')
1043: D400 First line should end with a period (not 't')
1043: D200 One-line docstring should fit on one line with quotes (found 3)
1066: D400 First line should end with a period (not 't')
1066: D200 One-line docstring should fit on one line with quotes (found 3)
1093: D200 One-line docstring should fit on one line with quotes (found 3)
1107: D400 First line should end with a period (not 't')
1107: D200 One-line docstring should fit on one line with quotes (found 3)
1124: D400 First line should end with a period (not 'n')
1124: D200 One-line docstring should fit on one line with quotes (found 3)
1147: D400 First line should end with a period (not 'w')
1175: D400 First line should end with a period (not ')')
1175: D200 One-line docstring should fit on one line with quotes (found 3)
1188: D200 One-line docstring should fit on one line with quotes (found 3)
1201: D400 First line should end with a period (not 'e')
1201: D205 1 blank line required between summary line and description (found 0)
1208: D400 First line should end with a period (not 'p')
1208: D200 One-line docstring should fit on one line with quotes (found 3)
1215: D400 First line should end with a period (not 'r')
1215: D200 One-line docstring should fit on one line with quotes (found 3)
1232: D400 First line should end with a period (not 't')
1232: D200 One-line docstring should fit on one line with quotes (found 3)
1240: D400 First line should end with a period (not 'd')
1240: D200 One-line docstring should fit on one line with quotes (found 3)
1258: D200 One-line docstring should fit on one line with quotes (found 3)
1267: D400 First line should end with a period (not 'e')
1284: D400 First line should end with a period (not 'm')
1284: D205 1 blank line required between summary line and description (found 0)
1300: D200 One-line docstring should fit on one line with quotes (found 3)
1311: D102 Missing docstring in public method
1317: D102 Missing docstring in public method
1323: D102 Missing docstring in public method
1337: D102 Missing docstring in public method
1350: D200 One-line docstring should fit on one line with quotes (found 3)
1377: D102 Missing docstring in public method
1436: D102 Missing docstring in public method
1481: D102 Missing docstring in public method
1489: D102 Missing docstring in public method
1499: D102 Missing docstring in public method
1508: D102 Missing docstring in public method
1511: D102 Missing docstring in public method
1515: D102 Missing docstring in public method
1519: D102 Missing docstring in public method
1539: D102 Missing docstring in public method
1547: D102 Missing docstring in public method
1555: D102 Missing docstring in public method
1561: D102 Missing docstring in public method


According to google it's spelled as "Programmatically".  (There are a few cases of this word in this PR)


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/3f0981f5ee76b3fb879dd7466d250a3929df22c8


merging basho/riak_cs/bugfix/wrong-time-format-description-on-storage-schedule-2 = 3f0981f5 into borshop-integration-1153-bugfix/wrong-time-format-description-on-storage-schedule-2


basho/riak_cs/bugfix/wrong-time-format-description-on-storage-schedule-2 = 3f0981f5 merged ok, testing candidate = a52df803


Hey Felix,
I was wondering if I should put another if statment here to handle the unlikly case that for some reson it fails to update the database and change the user role to premium after it has already charged the user's paypal account?

How would you handle this?
Would you put another if statment and refund the users charge if the database update was not successful?

Thanks,
Blayne


The virtual network "%s" does not contain a subnet with name "%s"


Seems odd that vaapi is being pulled into the v4l2 interface layer.  Looks like there was a lot of design effort to keep it separate.


I think we should keep the stuff in graphics/, that's source material that might be useful later.


I think we don't have to check for "if (!vNet.Subnets) {" here, the below utils.findFirstCaseIgnore(subnetList, {Name: subnetName}); should return null if 'vNet.Subnets' is null. Please check


It's "attribution" ;P


Please, fix this branch ASAP.
Make sure Travis and AppVeyor passes.


Ah, typo


can you add an apply that also takes 1 string, which is a common case. Same for the read side.


I think these are the reactions that mapped to functional roles


Added issue #591.


Ah, then that number does sound about right. With the expansion of GPRs
from KEGG, I thought we were around 8,000.

On Thu, May 28, 2015 at 7:06 PM, janakagithub <notifications@github.com>
wrote:

> I think these are the reactions that mapped to functional roles
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/1153d1db4f02eaab25232a4c02303602e917b5b1#commitcomment-11418486>
> .
>


LOL yes :D it was late ;)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.189](http://teamcity.monogame.net/viewLog.html?buildId=7740&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:22:27


p закрывается div'ом


just checking, can we avoid above for loop by using utils.indexOfCaseIgnore which will return null in 3 cases

1. when vNet.Subnets is null
2. when subnet with name not in the subnet collection
3. when vNet.Subnets is an empty array

if subnet found, it will return the index.

due to 1 we can avoid above 'if (subnetList && subnetList.length > 0) {' as well


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1185


Leave a line note


hi


OH SHI~ SOURCE 2~~~


HAC uses this method?


Yes, RFC3986 is saying username should be encoded:
>   authority     = [ userinfo "@" ] host [ ":" port ]
>   userinfo      = *( unreserved / pct-encoded / sub-delims / ":" )
...
>   pct-encoded   = "%" HEXDIG HEXDIG
>
>   unreserved    = ALPHA / DIGIT / "-" / "." / "_" / "~"
>   reserved      = gen-delims / sub-delims
>   gen-delims    = ":" / "/" / "?" / "#" / "[" / "]" / "@"
>   sub-delims    = "!" / "$" / "&" / "'" / "(" / ")"
>                 / "*" / "+" / "," / ";" / "="
I'm thinking we could add support for encoding the username/password.  However I think we'll need some rule to resolve ambiguities when the colon is not encoded--such as user:name:password@ (is the username/password "user:name"/"password" or "user"/"name:password"?).  Maybe just go by the first unencoded ':' character as the boundary and let the remainder be the password?

Thanks for the quick feedback--this is great!


Note: The correct PR is #5326.


i think we can get rid of line 125-134 by using

return utils.findFirstCaseIgnore(vNet.Subnets, {Name: subnetName});

please check.


Oh I like the syntax ``{% url ':foo' %}``. That's nice and explicit. I'd think maybe it should only look in the current namespace though.


Why are these bouncer == true checks all over the place? You should try to have one thing be responsible for performing these checks, then not have to worry about them all over the place. It makes it difficult to read/understand what each function is doing because these steal your attention.

Think about what would happen if all of a sudden there was some configuration option where this had a second check that was needed. Then you'd have to go throughout this entire program and make it say like "if (bouncer == true && otherCheck == false)" - having to change a lot of code when one requirement changes is a good sign that you need to rethink the approach.


Yes, see https://github.com/MFSiNC/HAC/blob/f1f8159f1ce27c175215a545fbecebec34a2d0b9/v13/HeX%27s%20AntiCheat/lua/autorun/client/nuke_config_client.lua#L135


Security tip - from an authentication stand point, this might be a bit suspect. You probably don't want a system that reveals if the user name is correct before having to enter the password.


> Please note in case you are writing Less rules outside of `theme.less`: overriding variables will only work if they are defined in a less file that is imported by the main `theme.less`.

@mnapoli now that https://github.com/leafo/lessphp/issues/302 was fixed in lessphp maybe this could be removed/ outdated?


There are typos in the two references above the ones you added (it should be Kaus instead of Kauss). It would be great if you could fix that for us :)


High-five, David.


Tons of duplicate in here, you could have something that was like "FileAccessor.cs" that has a "Create(string path)" - you'd pass all these in and be done. Or maybe just like a "ReadText(path)" is a starting point for reuse.

string ReadText(string path)
{
       //check if exists and then create if needed
      //read all the text and return
}


I can easily revert the change you are asking, but I need you to elaborate on why the moved threads need to be there. When a thread creator needs to go to his thread, he can easily go through the database of his content. Very rarely does a member visit his own thread by browsing the forums, even more so that the member in question is unaware of the potential of the thread being moved.

It causes more confusion to the other members when they see a thread containing 0 posts, regardless of whether it has moved or not. Also, with the code that was originally used, the moved badge did not function in the first place.


Do these need to be static or should these be local variables? You only want to define class level/static variables for things that need to live forever, otherwise they should just be local to the method.


GroundDomainUpdate (sushobhit) - x86_64-MacOS-10.9-clang: OK (1665 of 1668 tests passed)

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


Update


I think there should be an option on the Split that will remove empty entries, then you wouldn't need this at all I think? I'm not 100% sure what the intent is with this method.


it still works with >= 0.4 and < 0.6 though, no?


Na verdade esse 'push' foi pra reverter uma alteração que eu tinha enviado em BuildConfig.groovy antes do monitor me informar isso que você falou.


I'm sure this has meaning to you / it's some sort of encryption, but without a comment of any sort, it would be impossible for another developer to say whether this is correct or try to fix any bugs associated with it.


Can you change this to 'characterized' so that we are consistent within the manual?


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/bb2938307cbbdf481377ece40d4aa0ac30015804/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/bb2938307cbbdf481377ece40d4aa0ac30015804/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/bb2938307cbbdf481377ece40d4aa0ac30015804/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/bb2938307cbbdf481377ece40d4aa0ac30015804/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/bb2938307cbbdf481377ece40d4aa0ac30015804/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Why revert this?


r+


saw approval from jedmccaleb
at https://github.com/jedmccaleb/stellar-core/commit/56ccfa8e99ae1ae27e718bc6f5d7309aab507f80


merging jedmccaleb/stellar-core/fetch = 56ccfa8e into auto


jedmccaleb/stellar-core/fetch = 56ccfa8e merged ok, testing candidate = 0c3f8cf8


eh true. i'll put that back


I would move this to policy.go.


what are you doing?


change this too


Begin business logic - AccountManager.Delete(name)


I left i comment on the PR too, but can't we do this with DownloadUpdateRpmDiff, and only print the diff output without the daemon. That way dbus clients can still know what is happening on the system.


End Business Logic


Also, this query function is very simplistic compared to https://github.com/SitePen/dstore/blob/master/SimpleQuery.js.  It just does exact match, rather than less than, contains, etc.   Is that intentional?


Begin business logic:
AccountManager.UpdateAccountName(name, newUser)


End business logic


decQIDState( qid, null) wasn't actually used anywhere for clearing qid from qidstates so the change is safe.
If we'll need to have this functionality it can be done in a different way


Just FYI -- this line is a no-op.  In the IPython terminal, it printed the `__repr__` of the return value (which was `True`) -- I just used it as a quick sanity check.


Having so much code in one class just isn't really a scaleable solution. I'd probably argue that it doesn't even work for this program, which is pretty straight forward in most ways. At the basics, I'd say you have to start thinking about at least separating your UI and your business logic.


I added a couple examples in here where I think you should be delegating the actions to another class that is responsible for all the management that is happening.


Instead of magic numbers, consider an enumeration.

```
var UserStatus = {Active: 2, Pending: 1, Inactive: 0}
// later
if( this._activeUsers[user] ){
  return UserStatus.Active;
}
// etc
```


Totally!

PRs are always welcome!

> On May 28, 2015, at 4:58 PM, bdaigle <notifications@github.com> wrote:
>
> Yes, it seems to work fine after I fixed a small typo. Just opened a Pull Request.
>
> I've also added some additional functionality in my fork including support for the "tsv" option using "gcalcli search" and a "--detail_email" option to display the event creator's email. Let me know if I should open Pull Requests for either of these commits.
>
> —
> Reply to this email directly or view it on GitHub.
>


Once you capture the business logic (all the account management) in its own classes, then you can work to continue to capture your data storage (your file based access) into a FileAccessor class. Then you would have your "UI" class here, your business class (AccountManager), and your data access class (FileAccessor). You really should always try to separate out your thoughts along these lines when you're designing software.


The condition statement has been modified; It is at the next commit.


@cyrilf I've removed the previous release / tag and republished a new one with the proper version in `bower.json` should be good now - thanks for the head's up.


Looks OK to me.  Thanks.


I checked zint, For Micro QR Code and QRCode,they both wrote +20:
void micro_setup_grid(unsigned char* grid, int size){
//...
   grid[(8 * size) + 8] += 20;}

Please verify again whether zint made mistake. Thanks.


I had to resist the strong temptation to not refactor the `case` to improve the clarify of these codes. Since there are no tests and I am uncertain who else calls [line 51](https://github.com/ActiveState/aok/blob/6ceb7c46c3286122495b6ee66ed0ea5dee863c5e/controllers/groups_controller.rb#L51), I'll leave the `case` as-is for now, unless anyone else knows where else this is used. Trying to write tests for these controllers is just a pain because of its coupling with Sinatra.


I can confirm that this fixes the issue.


looking at this I realize my suggested change was incorrect as strings in ruby don't directly include enumerable. The above code will work for 1 through 9, but will return false for something like `12` since `.all?` is returning items in the array so you're actually comparing `12` to `1` through `9`. Another way to do it and closer to what I meant to say originally would be `how_many.each_char.all? {|num| numbers.include?(num) }`

Sorry!


what does smeggingsmegger have against json?


@kanaka

Oops! That's right...
I'll add it and rebase.  Thanks!


CHANGES, not README.


Oh, sorry.

I thought of it when I was messing around with RunString in C++


この修正は誤りです


fast-forwarding develop to borshop-integration-1153-bugfix/wrong-time-format-description-on-storage-schedule-2 = a52df803


very nice sidebar ʕº̫͡ºʔ


Since the problem is very similar to what I was encountering with the other Coverity issue, it seemed reasonable to me to revert the change and tackle the fix for both of them at the same time.  Now, tackling them both doesn't mean I have to revert the change entirely - I guess I was operating under the thought that the connection makes it fall under the "coverity error" priority and reverting the change would allow it to live there instead of under the "pgi bug" priority, since the problem I was encountering with the other Coverity error was with gcc.  Thus it would cause less problems for everyone else and yet still remain on my radar (as the remaining coverity errors are).  But since we aren't running testing for the next few nights, I can also wait until Monday to see if I can fix it then.  Thoughts?


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/64505366


I'm pretty sure the C.Water tag should stay in place since the UOP does edit water heights for correction.


Whoops, must have missed that, was supposed to be off by default. Thanks.


86139842-AddAutoSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1160 of 1160 tests passed)

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


voltando


fast-forwarding master to auto = 0c3f8cf8


Yes, just forgot to take it out when copying and pasting after doing the same thing you did in IPython.


@cxqclz 不知道xml你用什么工具打开，我建议用一些编辑工具（如notepad++），然后预言选择xml，这样的话在翻译的时候就不会忘记加一些“]"导致文档结构出问题


1) Added Sendgrid to Heroku
```
heroku addons:add sendgrid:starter
```
2) Create a file in config/initializers named setup_mail.rb and add Sendgrid information.
3) Use Figaro to set ENV variables. Update secrets.yml and application.yml. Add to gitignore.
```
figaro install
heroku config:get SENDGRID_USERNAME
heroku config:get SENDGRID_PASSWORD
```
4) Devise provides an object named current_user that represents the user who is currently signed-in. Add it to application.html.erb.
5) Update registrations#new, sessions#new, and welcome#index form styling.
6) Add `before_action` to permit names in ApplicationController
7) Add ActionMailer's "silent" errors to be less silent.


Is this commit really necessary to backport? Since we are no longer accepting translations for the 2.4 branch it will mean languages that are currently 100% translated will have a rogue string in English


Thanks for the fix @siniStar7boy.


Thanks. Will test tomorrow.

For the record, which browser and game did you see the RangeError on?


Are you opening and closing the file on every request? I think that is extremely bad. I don't know exactly how nginx logs to the files on disk but very probably not by doing so


Chromium 42 (x64 Linux), Counterfeit Monkey.  No idea what actually controls the maximum.

I got 16384 from [this stackoverflow answer](http://stackoverflow.com/a/22747272/17875) that tested various browsers; all of them allowed at least 32767 arguments.  (Curiously, the author gets two different results for the same version of Chrome.)


1) Create a new migration to associate Users and Posts `rails g migration AddUserToPosts user_id:integer`
2) Index foreign keys:  index foreign keys. Create an index on user_id by adding an add_index line in timestamp_add_user_to_posts.rb.
3) Migrate changes `rake db:migrate`
4) Update Post model so it belongs to User
5) Update User model is it has Posts
6) Update seeds.rb
7) `rake db:reset`
8) Update create method in PostsController.
9) Add author information and scope to posts#index.
10) Add default scope to Post model.


The readme's been overwritten, you might want to fix that


Thank you SO much!


I guess it could be useful for some people though...


I was aware of that and didn't think it was a big deal.

Now that I think about it, I think this is also a bug fix since I'm pretty sure creating a filtered deck on the desktop client that uses this sort order will crash on AnkiDroid if you try to rebuild on that (I'll test it later, maybe it's already been reported somewhere).


Can we get a regression for this?


![screenshot from 2015-05-29 03 04 31](https://cloud.githubusercontent.com/assets/1211455/7874432/e9877960-05af-11e5-8d46-3ba4018f79d1.png)


Jumped ?_?


1. Add role to User table `rails g migration AddRoleToUsers role:string`
2. Migrate database `rake db:migrate`
3. Add role methods to User model.
4. Include Pundit in ApplicationController
5. Add ApplicationPolicy class `rails g pundit:install`
6. Create PostPolicy class
7. Add error handler for Pundit in ApplicationController
8. Update PostsController to authorize appropriately.
9. Restrict access to links in posts#index
10. Update posts#show


Yes. It's hard to wrapper libva in decoder interface. take
    virtual Decode_Status getOutput(unsigned long draw, int64_t *timeStamp
        , int drawX, int drawY, int drawWidth, int drawHeight, bool draining = false
        , int frameX = -1, int frameY = -1, int frameWidth = -1, int frameHeight = -1) = 0;
for example.
It just a wrapper for X11 version of vaPutsurface. But there are so 3 version of vaPutSurface. X11, android and wayland. If we want support those platform, we need condition compile the decoder interface.
So It's better if we can output VASurfaceID. Customer use it as he wish. Android use android version vaPutSurface. Wayland use Wayland version of vaPutSurface. It's no need condition compile in interface level. and it's not add complex to customer, because getOutput is just a simple wrapper for vaPutSurface.


Yeah, [there was one](https://code.google.com/p/ankidroid/issues/detail?id=1967).


Well thank you for doing this!

Yes I think it is "safe" to assume first non encoded ":" will do as separator. URL Encoding always shouldn't be a problem though - as URL encoding an already encoded string does nothing right?


hi @davidwadden , it's should be `install-from-tar` not `install_from_tar`.


hi @davidwadden , `install-lattice-coordinator` is not exist, is it insteaded of `install-brain` ?


hi @davidwadden ,should be `install-cell`.


This matches the format of the existing less file so the usage is consistent.

cc @wjdp


This change makes no sense...why it work?!


hi @davidwadden ,should be `install-from-tar`.


The original author just submitted a correction, so we're all good. Thanks for volunteering!


@nathanielks ready to merge when you are


ok.


見ていてとても気持ちいいです！

ありがとうございます。


GroundDomainUpdate (sushobhit) - i386-Windows-7-VisualStudio-12: OK (1671 of 1674 tests passed)

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


Don't you hate when you make a spelling error in a commit log.


sweet change bro


Config won't cause that much lag, thus I want people to configure it anytime they want even if it's running.


1. Add Topic model `rails g model topic name:string public:boolean description:text`
2. Edit Topic migration file: set the public attribute to true by default.
3. Run migration `rake db:migrate`
4. Associate Topics to Posts `rails g migration AddTopicToPosts topic_id:integer:index`
5. Update Post and Topic model to reflect the association.
6. Update seeds `rake db:reset`
7. Add Topics resources to Routes
8. Add TopicsController `rails g controller topics index new show edit`
9. Update TopicsController
10. Update topics#index
11. Add TopicPolicy class
12. Update topics#new
13. Update topics#show
14. Update topics#edit
15. Update Routes to nest posts within topics
16. Refactor posts#show to update with topics
17. Refactor PostsController wrt topics
18. Refactor posts#new, posts#edit
19. Refactor Application layout


DOPE


因爲原本「質」只有zhì一个讀音，而詞典壓縮工具把可以自動注音的「人質」給刪除了。
現在在25809行，「質」多了一个zhí音，會對八股文中的「人質」、「典質」等詞進行自動注音。

要在詞典中加入「質」的zhí音進行自動注音的話，就必須同時把之前被詞典壓縮工具刪掉的只能讀去聲的「人質」、「典質」等詞找回來手動加上zhì音。


1. Deploy to production `git push heroku master` `heroku run rake db:migrate`
2. Set ENV variable in Heroku using Figaro `figaro heroku:set -e production`
3. Configure Sendgrid for Production


1. Fix error I did not catch in last commit.


Hail @bsagal, slayer of [dependency chain](https://gist.github.com/piedar/e81e8905d01e4cde581c)s!


rax retry tests


Global variables were moved to separate file global.js.
Now any module can use this vars by require() them.


This is my first go with github. Starting code for co2 fluxes.


Yes, It's a problem. We will remove other display except NATIVE_DISPLAY_VA. Because LIBVA_DRIVER_NAME select and DisplayType select is very head ache thing. Again if we want support anrdoid or walyand we need add new display type. And core library(ie decoder, encoder, vpp) do not care about it X11 dipslay or drm handle. It Just need a VADisplay to decode or encode. So we only accept VADisplay in future. Let client(ie, v4l2decoder or yami decoder to choose the display or driver name.
I will move LIBVA_DRIVER_NAME related code to v4l2decoder


OK.


boa


The better solution would be to inject the GrainFactory into this class. In fact, the "old grain factory" should be deprecated because it encourages writing non-testable code.


Parse and store json-ld data move to corresponded modules. Later I convert them to Flux.


OK


실행결과

쉘> python blog/cralwer.py
crawling ...
[1994157] 구글 인박스, 공개로 전환
[1994128] SKT, 하루 6시간 데이터 부담 없는 'band 타임프리' 출시
[1994102] 통신료 인하 무한경쟁 예고… 제4이통사·요금신고제 도입
[1994101] "삼성, 접는 스마트폰 '프로젝트 밸리' 개발중"
[1994076] 구글, 모바일 검색광고 매출 75%는 애플 덕분
[1994023] Google Photos 발표
[1994022] Google, Project Brillo & Weave 발표
[1994021] 구글, 안드로이드 M 공개
[1994002] 갤럭시 알파, 안드로이드 5.1.1 벤치 포착
[1993984] 파이널 판타지 아기토, Windows 10 용으로 출시 예정
[1993968] Apple 의 Jeff Williams씨, WWDC 2015에서 Watch Native SDK를 발표할 것이라 발언
[1993926] 인텔 스카이레이크 i7-6700k는 4790k 보다 4~8% 향상에 불과해
[1993846] 미래부 장관 "통신비, OECD 평균이하…단통법 효과"
[1993788] 애플 iOS9, 구글 나우 닮은 '프로액티브' 추가하나?
[1993786] '통신 요금인가제' 24년만에 신고제로 전환…어떻게 달라지나?
[1993783] 갤럭시노트5용 '기어 VR', 홀로그램·프로젝터 탑재 가능성
[1993779] 레노버, 프로젝터 활용한 '가상 키보드' 컨셉 스마트폰 선보여
[1993774] 하반기 출시 쉐보레 스파크에 애플 카플레이 탑재된다
[1993720] 루머) 소니, 전혀 새로운 시스템의 미러리스 카메라 준비중
[1993716] 루머) 소니 APS-C타입의 신형 플래그십 미러리스 준비중? (A6xxx or A7xxx)
[1993714] 소니 XPERIA Z3+, 수동 카메라 지원
[1993713] 레노버, ThinkPad 10 2세대 공개
[1993689] Verizon, LG G4 및 G Pad X 출시 예정
[1993682] 샤프, 아쿠오스 크리스탈 인도네시아 출시
[1993680] KDDI, 한국행 관광객 대상 LTE 라우터 대여 개시
[1993675] 파나소닉 Lumix CM1, 안드로이드 롤리팝 업데이트 개시
[1993673] NTT 도코모, 2017년 블랙베리 서비스 종료 예정
[1993637] 법원 “무단 사이트 미러링은 위법”…임의 데이터 수집에 경종
[1993514] 배설물로 질주하는 버스, 최고 속도는?
[1993508] 기본료 폐지, 이통사 7조 수익감소…"비현실적 포퓰리즘 공약"


1. Refactor Application layout.
2. Refactor registrations#edit
3. Modify routes for Users
4. Create UsersController
5. Refactor PostsController


if(preg_match("/\r\nContent-Length: ?(\d+)\r\n/", $http_string, $match))


@guanglinlv thanks for pointing those out.  i was in a hurry and don't have a test setup for openstack

i'll correct it tomorrow


@rafaelfranca you will want to revert this commit as it doesn't work as intended.  Since popen is being used ruby tried to execute '/usr/bin/git credential-osxkeychain' as a command and not evaluating it like the backtick or system calls do.  In order for this to work the git command along with 'credential-osxkeychain' added to the ARGV list would need to be used.  Since boxen installs its own git there honestly isn't any value in "making this better".

https://github.com/boxen/our-boxen/blob/master/script/boxen-git-credential#L47


As a result this is what happens

```shell
/opt/boxen/repo/.bundle/ruby/2.0.0/gems/open4-1.3.4/lib/open4.rb:37:in `exec': No such file or directory - /usr/bin/git credential-osxkeychain (Errno::ENOENT)
```

Its hard to see whats going on but that is how popen works.


Yes. It's needed. Someone(v4l2decode applicaiton ) will call the v4l2decoder without set the drm fd. We do not want break it.I will check other place to invoke this function.


After adding this extension point, the big endian include path is also added automatically. To void other side-effect, I will add big endian compiler into plugin.xml


Do you think we should have `IBrowser` extension methods for some of these?


1. Add validation to Post model
2. Update posts#edit and posts#new to create a form partial
3. Add form errors method to ApplicationHelper


新版本已经修复了这个问题


1. Add markdown support for topics#show, posts#show
2. Add markdown to html method in ApplicationHelper


Close #526.


Why was this necessary?  The `pip install .` should've installed all the requirements.


I managed to partially test both versions and I can confirm that they both restore the graph structure, but they differ in what they do with the edges (src,dst) when dst falls outside src's function. The version I propose doesn't include those edges, ivg's version does. The problem is that ivg's behavior is desired in certain cases, such as the flowgraph plugin.  A new optional boolean argument, say cross_connect, is required to specify what to do with such edges.


The right solution seems to be to add GrainFactory to IProviderRuntime instead. And yes, the plan is to get rid of the codegen'ed factories fairly soon.


Events should be in React views. React has VirtualDom. Change components DOM from out side(jquery) could be break React render.

This logic should be moved to corresponded components.


ahahahahahaahah, I loved it!


I'm not sure what the regression test would look like. Right now, just being able to parse IR and geometry optimization jobs at all would fix this; maybe it will become clearer to me as I finish more attributes.


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1671 of 1674 tests passed)

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


What are your thoughts on the name of this variable?  Do you thing name and author are confusing?  Could name be more descriptive?


Moved to separate module. Good way to use bootstrap http://react-bootstrap.github.io/


I legit just used this tutorial because I hadn't removed the remotes after fixing my merge conflicts. Awesome tutorial :+1:


GroundDomainUpdate (sushobhit) - i386-Windows-7-VisualStudio-12: OK (1613 of 1639 tests passed)

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


What are your thoughts on how this could be tested?  Are you aware of any patterns that allow the 'real' thing to be substituted?


render function should not return false. Better null.


We aren't using these names in the UI.


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


A couple typos to fix for your final. Line 5 'has' is repeated. Line 7 'accountto' is missing a space. Line 8, 'speakeer' with the extra e! I'm glad you're on twitter. :+1:


It used to because it bound to the `inserted` event. I don't think it does anymore.


well, actually new IR handles all this issues correctly. And it has `call` abstraction that allows one to move at desired direction, either intra-procedurally or inter-procedurally.
So to speak, I have a strive to delete this function from BAP altogether... and maybe even remove the dependency from ocamlgraph, at least in the interface


Template could be loaded without request.


start learn git
![desert](https://cloud.githubusercontent.com/assets/12641420/7874795/15dd1e16-05e7-11e5-82b0-f4e810a4e376.jpg)


At least strip away debug.


TeamCity YaccConstructor :: YC.Core General Build [Build 405](http://teamcity.codebetter.com/viewLog.html?buildId=192245&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


TeamCity YaccConstructor :: YC.Core General Build [Build 406](http://teamcity.codebetter.com/viewLog.html?buildId=192246&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


I think that archlinux guys don't strip kernel objects at all.


Digging that chocolate covered bacon reference my friend. Had a great time as well. Interested to see how the steam is holding up in week 4...... :+1:


Data is allready loaded from json-ld. It stored in variables. Not need to load one more time same page for get same data.


Ha, well, a lot of this diff is from me renaming getbufptr to getbufstart.

The main thing though is that the nasty linear search that skipscreenlines did in refresh_bounds is gone, and that search now happens when the scroll position is updated in set_scroll.  So now we get one linear search when the text is scrolled, instead of many linear searches whenever anything looks up the start/end of the window.


I went with the `IWebBrowser` extensions to keep the breaking changes a minimum, could also include ones on `IBrowser` and reference those in here.


OK. I misunderstood a little. This is great :)


OK


Does not really matter, but removing the '# >>files' line should be in the "Remove spectacle cruft" commit.


I went back and looked at your tutorial this morning when I first started on media queries in Vertical Timeline. Awesome write up! :+1:


The solr field is "reader" not "readers"


hola


esta bueno


That's what I though too, but we can figure it out. In one of my first implementations I was keeping a global file handler, but that's unable to detect file failures. So when it fails, nothing will be logged. I will check the nginx source code.


bueniimo


`withContext` is deprecated in React 0.13-alpha. Any idea for a replacement?


TeamCity YaccConstructor :: YC.Core General Build [Build 405](http://teamcity.codebetter.com/viewLog.html?buildId=192245&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 116 Build time: 00:12:37


TeamCity YaccConstructor :: YC.Core General Build [Build 406](http://teamcity.codebetter.com/viewLog.html?buildId=192246&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 116 Build time: 00:12:44


@chiggs For my use case, we make extensive use of verilog and vhdl generate statements.
And there can be multiple levels of verilog/vhdl instances i.e verilog->vhdl->verilog->vhdl->verilog

Having the cocotb iteration support these rtl language features and support crossing mixed language boundaries, would be very useful for us.

Even initially having the ability to reference instances within generate statements would be extremely useful.
I'm currently having to use a workaround as per #248 to get access to signal handles within generated instances.


This code is not so clear and readable.
It should be written like this
player.y += ((player.y + deltaY) >= 0 || (player.y + deltaY) <= player.gameHeight) ? deltaY : 0
player.x += ((player.x + deltaX) >= 0 || (player.x + deltaX) <= player.gameWidth) ? deltaX : 0


@chrisseaton Was there a reason you were lowering these to begin with?  I want to make sure I don't break anything.  But at the same time, lowering introduces the possibility of only writing half a word on 64-bit machines.

The case where this came up is `TimeVal`.  Both `:tv_sec` and `:tv_usec` are longs on Linux and I had a case where values greater than 32-bits were being written.  It looks like on Mac one of the fields is a long  and the other is an int, according to the Rubinius config.

In any event, I can't readily test this since I don't have a Mac.  Please give it a glance over.


As always never gonna improve by patting your own back. At least write the commit message sensibily if you aren't sensible enough.


I wasn't aware that Lua needs special treatment. Will push in a sec.


s/contains/contain/


Goed


Is that to check that getResource() still works after serializing and unserializing? If so, that's a different test method.


:cake:


Doh! You are right :-)


Ok, I still don't have time for a complete answer, but in summary, I strongly advise not to expand Kint functionality for such a trivial and custom application in general. Is there a real reason to change the display (I'm not even talking about creating a whole new decorator class just to remove a couple of lines)?

All you need is this simple function:
```php
function l($msg)
{
    if ( !Kint::enabled() ) return;

    // note, that this way of adding aliases to Kint might be discontinued without further notice
    if (!in_array(Kint::$aliases['functions'],'l')) {
        Kint::$aliases['functions'][] = 'l';
    }

    ~$output = Kint::dump($msg);
    file_put_contents('log.txt',$output, FILE_APPEND);
}
```

aren't you from Latvia BTW :)


Thats a great change ya made there now. Good job buddy!


That would be "testResourceBecomesStaleWhenFileIsDeleted"


Field int_255 sudah dihapus, jika masih ada cache di lokal jangan digunakan.


jesus


Reported as https://github.com/travis-ci/travis-ci/issues/3977


I think `update` should copy the markdown from the previous instance, not reparse and rerender the whole thing.
If I understand how virtual-dom's widget API works, a new instance is created on each render, and update is called on the new instance with the previous instance as the first argument.


I don't remember details now but I was not able to achieve the same by only
 overriding EventStoreSerializer. Just wanted to point you to my solution
in case you have problems with "default" behaviour.


Was just about to report this...


a "if" statement seems to me a simpler approach than "elif", though it works :)


if they are not existing, should they be created for the tests and deleted with the cleanup?
or should it be rejected if they are missing?


@t3hnar, do you mean the fork isn't necessary?


I suggest that `EnterLeave` receive the `MenuButton` (later an abstract interface representing everything with enter/leave) instead of a C++ object for two reasons:
  1. avoiding to pass C++ objects around as soon as possible
  2. If other organisms need to react to `MenuButton.mouse_enter/leave`, they would require access to `e1` which is a local variable, inaccessible, in `MenuButton`.

Here, you could have only `MenuButton` on the interface (say, `parent`), move the events declarations to `MenuButton`, and test/emit with the following:

    var _GUI_Component* gui = await ON_POINTER_ENTER until (gui == &menu.mb_ptr);
    emit menu.mouse_enter;


I can compile it this way. How can I check the "it works for me"?


So gerade mal den Test gemacht mit add.png
Hier aus dem Master:
(master) 733B ----------> 579 B (-21)
(überarbeite) 1,2KB---->227 B

Anfangsgröße vom überarbeiteten ist so groß, da Photoshop in PNG24 abspeichert


This matter is up to devloper.
Max password length may have a restriction. It depends on the way it password value is stored.
If you use encryption (not hashing) for the password field (or other security key, which input should be masked) the length of encrypted text depends on length of the source text. If you will to save the encryption result in `varchar(255)`, you will have to set input maxlength restriction.

I suppose the change can remain. If developer wishes to make his program inconsistent - it is his choice. He will set 'maxlegth' on password field anyway.


Yes.


if the only reason to separate `PersistentRepr` data from `payload`, then YES it is not needed.


This class is abstract but hasn't any abstract methods. Probably you forgot to mark DoStep method as abstract.


we first have to check whether python is installed, which is not generally the case on windows. Something like:
if [ x`which python` = "x" ]; then #python is not installed


will do in the issue:
https://github.com/sphereio/sphere-product-sync/issues/8


Why you use enumerations?
Try to use: DiskModels.SingleOrDefault(it => it.DriveName == driveName)


1: M111 unrecognized component name: legacy


46:1: I100 Imports statements are in the wrong order. from MySQLdb should be before from invenio.utils.datastructures
48:1: I100 Imports statements are in the wrong order. from thread should be before from MySQLdb


1: D400 First line should end with a period (not 's')
22:1: E402 module level import not at top of file
23:1: I100 Imports statements are in the wrong order. import os should be before from six.moves
23:1: I201 Missing newline before sections or imports.
23:1: E402 module level import not at top of file
24:1: E402 module level import not at top of file
25:1: E402 module level import not at top of file
26:1: I100 Imports statements are in the wrong order. import tempfile should be before import time
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 tempfile
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. import datetime should be before import sys
28:1: E402 module level import not at top of file
33:1: I101 Imported names are in the wrong order. Should be abort, url_for
33:1: E402 module level import not at top of file
34:1: I201 Missing newline before sections or imports.
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
36:1: I201 Missing newline before sections or imports.
36:1: E402 module level import not at top of file
38:1: I100 Imports statements are in the wrong order. from invenio.config should be before from six
38:1: E402 module level import not at top of file
39:6: E126 continuation line over-indented for hanging indent
64:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.config
64:1: E402 module level import not at top of file
65:1: E402 module level import not at top of file
66:1: E402 module level import not at top of file
67:1: E402 module level import not at top of file
68:1: E402 module level import not at top of file
69:1: I101 Imported names are in the wrong order. Should be get_all_restricted_recids, get_record, record_exists, search_unit_in_bibxxx
69:1: E402 module level import not at top of file
71:1: E402 module level import not at top of file
72:1: E402 module level import not at top of file
73:1: E402 module level import not at top of file
74:1: I101 Imported names are in the wrong order. Should be EscapedXMLString, X
74:1: E402 module level import not at top of file
77:25: E203 whitespace before ':'
78:25: E203 whitespace before ':'
79:25: E203 whitespace before ':'
84:25: E203 whitespace before ':'
88:25: E203 whitespace before ':'
106: D205 1 blank line required between summary line and description (found 0)
118: D400 First line should end with a period (not 'r')
118: D200 One-line docstring should fit on one line with quotes (found 3)
134:9: E266 too many leading '#' for block comment
135:9: E266 too many leading '#' for block comment
146:1: E302 expected 2 blank lines, found 1
146: D202 No blank lines allowed after function docstring (found 1)
146: D400 First line should end with a period (not 'r')
146: D200 One-line docstring should fit on one line with quotes (found 3)
168:1: E302 expected 2 blank lines, found 1
168: D200 One-line docstring should fit on one line with quotes (found 3)
178:1: E302 expected 2 blank lines, found 1
178: D401 First line should be in imperative mood ('Get', not 'Gets')
178: D202 No blank lines allowed after function docstring (found 1)
178: D200 One-line docstring should fit on one line with quotes (found 3)
186:9: E221 multiple spaces before operator
191:1: E302 expected 2 blank lines, found 1
191: D401 First line should be in imperative mood ('Return', not 'Returns')
191: D205 1 blank line required between summary line and description (found 0)
201:1: E302 expected 2 blank lines, found 1
201: D400 First line should end with a period (not 'e')
201: D205 1 blank line required between summary line and description (found 0)
211:1: E302 expected 2 blank lines, found 1
211: D400 First line should end with a period (not 'e')
211: D205 1 blank line required between summary line and description (found 0)
221:1: E302 expected 2 blank lines, found 1
221: D209 Multi-line docstring closing quotes should be on a separate line
221: D202 No blank lines allowed after function docstring (found 1)
221: D400 First line should end with a period (not 'r')
221: D205 1 blank line required between summary line and description (found 0)
230:1: E302 expected 2 blank lines, found 1
230: D400 First line should end with a period (not 'e')
230: D205 1 blank line required between summary line and description (found 0)
246:1: E302 expected 2 blank lines, found 1
246: D400 First line should end with a period (not 't')
246: D205 1 blank line required between summary line and description (found 0)
278:85: E261 at least two spaces before inline comment
278:86: E262 inline comment should start with '# '
283:1: E302 expected 2 blank lines, found 1
283: D200 One-line docstring should fit on one line with quotes (found 3)
288:5: E266 too many leading '#' for block comment
289:5: E266 too many leading '#' for block comment
296:9: E116 unexpected indentation (comment)
297:13: E116 unexpected indentation (comment)
298:17: E116 unexpected indentation (comment)
299:13: E116 unexpected indentation (comment)
300:17: E116 unexpected indentation (comment)
301:13: E116 unexpected indentation (comment)
302:17: E116 unexpected indentation (comment)
303:13: E116 unexpected indentation (comment)
304:17: E116 unexpected indentation (comment)
305:13: E116 unexpected indentation (comment)
306:17: E116 unexpected indentation (comment)
308:9: E116 unexpected indentation (comment)
309:13: E116 unexpected indentation (comment)
310:17: E116 unexpected indentation (comment)
311:13: E116 unexpected indentation (comment)
312:17: E116 unexpected indentation (comment)
313:13: E116 unexpected indentation (comment)
314:17: E116 unexpected indentation (comment)
316:1: E302 expected 2 blank lines, found 1
316: D401 First line should be in imperative mood ('Print', not 'Prints')
316: D202 No blank lines allowed after function docstring (found 1)
333:37: E713 test for membership should be 'not in'
334:13: E266 too many leading '#' for block comment
335:13: E266 too many leading '#' for block comment
349:5: E266 too many leading '#' for block comment
396:1: E302 expected 2 blank lines, found 1
396: D401 First line should be in imperative mood ('Generate', not 'Generates')
396: D202 No blank lines allowed after function docstring (found 1)
415:1: E302 expected 2 blank lines, found 1
415: D401 First line should be in imperative mood ('Generate', not 'Generates')
415: D202 No blank lines allowed after function docstring (found 1)
440:30: E261 at least two spaces before inline comment
446:9: E266 too many leading '#' for block comment
447:9: E266 too many leading '#' for block comment
470:9: E266 too many leading '#' for block comment
475:1: E302 expected 2 blank lines, found 1
475: D401 First line should be in imperative mood ('List', not 'Lists')
475: D202 No blank lines allowed after function docstring (found 1)
475: D200 One-line docstring should fit on one line with quotes (found 3)
496: D401 First line should be in imperative mood ('Return', not 'Returns')
496: D202 No blank lines allowed after function docstring (found 1)
512:12: E127 continuation line over-indented for visual indent
521:1: E303 too many blank lines (3)
521: D401 First line should be in imperative mood ('Generate', not 'Generates')
521: D202 No blank lines allowed after function docstring (found 1)
521: D208 Docstring is over-indented
545:17: E128 continuation line under-indented for visual indent
546:17: E128 continuation line under-indented for visual indent
547:17: E128 continuation line under-indented for visual indent
548:17: E128 continuation line under-indented for visual indent
549:17: E128 continuation line under-indented for visual indent
565:1: E302 expected 2 blank lines, found 1
565: D200 One-line docstring should fit on one line with quotes (found 3)
571:1: E302 expected 2 blank lines, found 1
571: D200 One-line docstring should fit on one line with quotes (found 3)
579:1: E302 expected 2 blank lines, found 1
579: D401 First line should be in imperative mood ('Generate', not 'Generates')
584:1: E302 expected 2 blank lines, found 1
584: D401 First line should be in imperative mood ('Generate', not 'Generates')
588:1: E302 expected 2 blank lines, found 1
588: D401 First line should be in imperative mood ('Return', not 'Returns')
588: D209 Multi-line docstring closing quotes should be on a separate line
588: D400 First line should end with a period (not 'd')
588: D205 1 blank line required between summary line and description (found 0)
603: D401 First line should be in imperative mood ('Return', not 'Returns')
603: D400 First line should end with a period (not 'C')
603: D200 One-line docstring should fit on one line with quotes (found 3)
617: D210 No whitespaces allowed surrounding docstring text
631:29: E261 at least two spaces before inline comment
631:30: E262 inline comment should start with '# '
638:31: E261 at least two spaces before inline comment
638:32: E262 inline comment should start with '# '
652:1: E302 expected 2 blank lines, found 1
652: D401 First line should be in imperative mood ('Return', not 'Returns')
652: D200 One-line docstring should fit on one line with quotes (found 3)
673:1: E302 expected 2 blank lines, found 1
673: D401 First line should be in imperative mood ('Generate', not 'Generates')
679:1: E302 expected 2 blank lines, found 1
679: D400 First line should end with a period (not 'e')
679: D205 1 blank line required between summary line and description (found 0)
692:1: E302 expected 2 blank lines, found 1
692: D200 One-line docstring should fit on one line with quotes (found 3)
712: D200 One-line docstring should fit on one line with quotes (found 3)
727:31: F841 local variable 'e' is assigned to but never used
731:1: E302 expected 2 blank lines, found 1
731: D200 One-line docstring should fit on one line with quotes (found 3)
740:5: E266 too many leading '#' for block comment
745:5: E266 too many leading '#' for block comment
756:9: E266 too many leading '#' for block comment
760:9: E266 too many leading '#' for block comment
765:1: E302 expected 2 blank lines, found 1
765: D400 First line should end with a period (not 's')
765: D205 1 blank line required between summary line and description (found 0)
772:5: E266 too many leading '#' for block comment
777:39: E261 at least two spaces before inline comment
777:40: E262 inline comment should start with '# '
783:5: E266 too many leading '#' for block comment
787:5: E266 too many leading '#' for block comment
789:12: E713 test for membership should be 'not in'
790:16: E126 continuation line over-indented for hanging indent
793:18: E261 at least two spaces before inline comment
795:5: E266 too many leading '#' for block comment
797:12: E127 continuation line over-indented for visual indent
804:5: E266 too many leading '#' for block comment
806:12: E127 continuation line over-indented for visual indent
814:12: E127 continuation line over-indented for visual indent
827:12: E127 continuation line over-indented for visual indent
828:12: E127 continuation line over-indented for visual indent
832:5: E266 too many leading '#' for block comment
834:12: E127 continuation line over-indented for visual indent
835:36: E261 at least two spaces before inline comment
839:5: E266 too many leading '#' for block comment
841:12: E127 continuation line over-indented for visual indent
845:12: E127 continuation line over-indented for visual indent
848:5: E266 too many leading '#' for block comment
850:12: E127 continuation line over-indented for visual indent
853:5: E266 too many leading '#' for block comment
855:12: E127 continuation line over-indented for visual indent
860:1: E302 expected 2 blank lines, found 1
860: D401 First line should be in imperative mood ('Run', not 'Runs')
860: D400 First line should end with a period (not 'k')
860: D200 One-line docstring should fit on one line with quotes (found 3)
866:90: E203 whitespace before ':'
866:91: E231 missing whitespace after ':'


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
33:1: E402 module level import not at top of file
34:1: E402 module level import not at top of file
37:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS, CFG_BASE_URL, CFG_BIBFORMAT_HIDDEN_TAGS, CFG_BIBINDEX_CHARS_PUNCTUATION, CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS, CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS, CFG_BIBSORT_BUCKETS, CFG_BIBSORT_ENABLED, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE, CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE, CFG_CERN_SITE, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_LOGDIR, CFG_OAI_ID_FIELD, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_URL, CFG_SOLR_URL, CFG_WEBSEARCH_DEF_RECORDS_IN_GROUPS, CFG_WEBSEARCH_DETAILED_META_FORMAT, CFG_WEBSEARCH_DISPLAY_NEAREST_TERMS, CFG_WEBSEARCH_FIELDS_CONVERT, CFG_WEBSEARCH_FULLTEXT_SNIPPETS, CFG_WEBSEARCH_IDXPAIRS_EXACT_SEARCH, CFG_WEBSEARCH_IDXPAIRS_FIELDS, CFG_WEBSEARCH_PREV_NEXT_HIT_LIMIT, CFG_WEBSEARCH_SYNONYM_KBRS, CFG_WEBSEARCH_VIEWRESTRCOLL_POLICY, CFG_WEBSEARCH_WILDCARD_LIMIT, CFG_WEBSEARCH_WILDCARD_LIMIT, CFG_XAPIAN_ENABLED
37:1: F999 redefinition of unused 'CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE' from line 37
37:1: F999 redefinition of unused 'CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG' from line 37
37:1: F999 redefinition of unused 'CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS' from line 37
37:1: F999 redefinition of unused 'CFG_LOGDIR' from line 37
37:1: F999 redefinition of unused 'CFG_WEBSEARCH_WILDCARD_LIMIT' from line 37
37:1: F401 'CFG_XAPIAN_ENABLED' imported but unused
37:1: F401 'CFG_BIBINDEX_CHARS_PUNCTUATION' imported but unused
37:1: F401 'CFG_OAI_ID_FIELD' imported but unused
37:1: F401 'CFG_WEBSEARCH_VIEWRESTRCOLL_POLICY' imported but unused
37:1: F401 'CFG_WEBSEARCH_SYNONYM_KBRS' imported but unused
37:1: F401 'CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE' imported but unused
37:1: F401 'CFG_WEBSEARCH_DISPLAY_NEAREST_TERMS' imported but unused
37:1: F401 'CFG_WEBSEARCH_FIELDS_CONVERT' imported but unused
37:1: F401 'CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS' imported but unused
37:1: F401 'CFG_SITE_RECORD' imported but unused
37:1: F401 'CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS' imported but unused
37:1: F401 'CFG_INSPIRE_SITE' imported but unused
37:1: F401 'CFG_LOGDIR' imported but unused
37:1: F401 'CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG' imported but unused
37:1: F401 'CFG_WEBSEARCH_PREV_NEXT_HIT_LIMIT' imported but unused
37:1: F401 'CFG_BASE_URL' imported but unused
37:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused
37:1: F401 'CFG_WEBSEARCH_WILDCARD_LIMIT' imported but unused
37:1: F401 'CFG_WEBSEARCH_DETAILED_META_FORMAT' imported but unused
37:1: F401 'CFG_BIBSORT_BUCKETS' imported but unused
37:1: F401 'CFG_WEBSEARCH_FULLTEXT_SNIPPETS' imported but unused
37:1: F401 'CFG_SOLR_URL' imported but unused
37:1: F401 'CFG_WEBSEARCH_IDXPAIRS_EXACT_SEARCH' imported but unused
37:1: F401 'CFG_WEBSEARCH_IDXPAIRS_FIELDS' imported but unused
37:1: E402 module level import not at top of file
38:6: E126 continuation line over-indented for hanging indent
81:1: E402 module level import not at top of file
82:1: F401 'get_most_popular_field_values' imported but unused
82:1: E402 module level import not at top of file
83:1: I101 Imported names are in the wrong order. Should be InvenioWebSearchCitedbyLimitError, InvenioWebSearchReferstoLimitError, InvenioWebSearchUnknownCollectionError, InvenioWebSearchWildcardLimitError
83:1: I100 Imports statements are in the wrong order. from invenio.modules.search.errors should be before from invenio.modules.search.utils
83:1: F401 'InvenioWebSearchUnknownCollectionError' imported but unused
83:1: F401 'InvenioWebSearchReferstoLimitError' imported but unused
83:1: F401 'InvenioWebSearchWildcardLimitError' imported but unused
83:1: F401 'InvenioWebSearchCitedbyLimitError' imported but unused
83:1: E402 module level import not at top of file
84:6: E126 continuation line over-indented for hanging indent
88:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.modules.search.errors
88:1: F401 'get_fieldvalues_alephseq_like' imported but unused
88:1: E402 module level import not at top of file
90:1: E402 module level import not at top of file
91:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from .utils
91:1: I201 Missing newline before sections or imports.
91:1: F401 'create_record' imported but unused
91:1: F401 'record_xml_output' imported but unused
91:1: E402 module level import not at top of file
92:1: I101 Imported names are in the wrong order. Should be get_bibrank_methods, is_method_valid, rank_by_citations, rank_records
92:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.record_sorter should be before from invenio.legacy.bibrecord
92:1: F401 'rank_by_citations' imported but unused
92:1: F401 'get_bibrank_methods' imported but unused
92:1: E402 module level import not at top of file
97:1: I101 Imported names are in the wrong order. Should be calculate_reading_similarity_list, register_page_view_event
97:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.downloads_similarity should be before from invenio.legacy.bibrank.record_sorter
97:1: F401 'register_page_view_event' imported but unused
97:1: F401 'calculate_reading_similarity_list' imported but unused
97:1: E402 module level import not at top of file
98:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_stemmer should be before from invenio.legacy.bibrank.downloads_similarity
98:1: F401 'stem' imported but unused
98:1: E402 module level import not at top of file
99:1: F401 'BibIndexDefaultTokenizer' imported but unused
99:1: E402 module level import not at top of file
100:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_utils should be before from invenio.modules.indexer.tokenizers.BibIndexDefaultTokenizer
100:1: F401 'author_name_requires_phrase_search' imported but unused
100:1: F401 'get_field_tags' imported but unused
100:1: E402 module level import not at top of file
102:1: I101 Imported names are in the wrong order. Should be lower_index_term, wash_author_name, wash_index_term
102:1: F401 'wash_index_term' imported but unused
102:1: F401 'wash_author_name' imported but unused
102:1: F401 'lower_index_term' imported but unused
102:1: E402 module level import not at top of file
103:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_config should be before from invenio.legacy.bibindex.engine_washer
103:1: F401 'CFG_BIBINDEX_SYNONYM_MATCH_TYPE' imported but unused
103:1: E402 module level import not at top of file
104:1: F401 'create_download_history_graph_and_box' imported but unused
104:1: E402 module level import not at top of file
105:1: F401 'DataCacher' imported but unused
105:1: E402 module level import not at top of file
106:1: I101 Imported names are in the wrong order. Should be perform_external_collection_search, print_external_results_overview
106:1: F401 'perform_external_collection_search' imported but unused
106:1: F401 'print_external_results_overview' imported but unused
106:1: E402 module level import not at top of file
107:1: E402 module level import not at top of file
108:1: I101 Imported names are in the wrong order. Should be CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS, CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_USERIDS_IN_TAGS, CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS, CFG_ACC_GRANT_VIEWER_RIGHTS_TO_USERIDS_IN_TAGS, VIEWRESTRCOLL
108:1: F401 'CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused
108:1: F401 'CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_USERIDS_IN_TAGS' imported but unused
108:1: F401 'CFG_ACC_GRANT_VIEWER_RIGHTS_TO_USERIDS_IN_TAGS' imported but unused
108:1: F401 'CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused
108:1: E402 module level import not at top of file
114:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch.adminlib should be before from invenio.modules.access.local_config
114:1: F401 'get_detailed_page_tabs' imported but unused
114:1: F401 'get_detailed_page_tabs_counts' imported but unused
114:1: E402 module level import not at top of file
115:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.websearch.adminlib
115:1: I201 Missing newline before sections or imports.
115:1: E402 module level import not at top of file
116:1: I201 Missing newline before sections or imports.
116:1: F401 'InvenioDbQueryWildcardLimitError' imported but unused
116:1: E402 module level import not at top of file
117:1: F401 'deserialize_via_marshal' imported but unused
117:1: E402 module level import not at top of file
118:1: I100 Imports statements are in the wrong order. from invenio.modules.access.engine should be before from invenio.utils.serializers
118:1: F401 'acc_authorize_action' imported but unused
118:1: E402 module level import not at top of file
119:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.modules.access.engine
119:1: E402 module level import not at top of file
120:1: I101 Imported names are in the wrong order. Should be encode_for_xml, strip_accents, wash_for_utf8
120:1: F401 'encode_for_xml' imported but unused
120:1: F401 'wash_for_utf8' imported but unused
120:1: F401 'strip_accents' imported but unused
120:1: E402 module level import not at top of file
121:1: I100 Imports statements are in the wrong order. from invenio.legacy should be before from invenio.utils.text
121:1: E402 module level import not at top of file
123:1: E402 module level import not at top of file
126:1: I101 Imported names are in the wrong order. Should be calculate_cited_by_list, calculate_co_cited_with_list, get_cited_by_list, get_citedby_hitset, get_citers_log, get_records_with_num_cites, get_refers_to_list, get_refersto_hitset
126:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_searcher should be before import invenio.legacy.template
126:1: F401 'get_citedby_hitset' imported but unused
126:1: F401 'get_records_with_num_cites' imported but unused
126:1: F401 'calculate_cited_by_list' imported but unused
126:1: F401 'get_refersto_hitset' imported but unused
126:1: F401 'get_cited_by_list' imported but unused
126:1: F401 'get_citers_log' imported but unused
126:1: F401 'calculate_co_cited_with_list' imported but unused
126:1: F401 'get_refers_to_list' imported but unused
126:1: E402 module level import not at top of file
131:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_grapher should be before from invenio.legacy.bibrank.citation_searcher
131:1: F401 'create_citation_history_graph_and_box' imported but unused
131:1: E402 module level import not at top of file
132:1: I101 Imported names are in the wrong order. Should be get_self_cited_by, get_self_cited_by_list, get_self_refers_to_list
132:1: F401 'get_self_refers_to_list' imported but unused
132:1: F401 'get_self_cited_by' imported but unused
132:1: F401 'get_self_cited_by_list' imported but unused
132:1: E402 module level import not at top of file
133:55: E126 continuation line over-indented for hanging indent
136:1: I101 Imported names are in the wrong order. Should be get_table_update_time, run_sql, run_sql_with_limit, wash_table_column_name
136:1: F401 'wash_table_column_name' imported but unused
136:1: F401 'get_table_update_time' imported but unused
136:1: F401 'run_sql_with_limit' imported but unused
136:1: E402 module level import not at top of file
138:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid
138:1: F401 'collect_user_info' imported but unused
138:1: E402 module level import not at top of file
139:1: I101 Imported names are in the wrong order. Should be create_error_box, pagefooteronly, pageheaderonly, write_warning
139:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.legacy.webuser
139:1: F401 'pagefooteronly' imported but unused
139:1: F401 'create_error_box' imported but unused
139:1: F401 'write_warning' imported but unused
139:1: F401 'pageheaderonly' imported but unused
139:1: E402 module level import not at top of file
140:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.legacy.webpage
140:1: E402 module level import not at top of file
142:1: F401 'apache' imported but unused
142:1: E402 module level import not at top of file
144:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch_external_collections should be before from invenio.utils
144:1: F401 'do_calculate_hosted_collections_results' imported but unused
144:1: F401 'calculate_hosted_collections_results' imported but unused
144:1: E402 module level import not at top of file
145:1: F401 'CFG_EXTERNAL_COLLECTION_MAXRESULTS' imported but unused
145:1: E402 module level import not at top of file
147:1: F401 'DatabaseError' imported but unused
147:1: E402 module level import not at top of file
152:22: E203 whitespace before ':'
152:14: E225 missing whitespace around operator
153:24: E203 whitespace before ':'
154:24: E203 whitespace before ':'
155:28: E203 whitespace before ':'
156:30: E203 whitespace before ':'
157:24: E203 whitespace before ':'
158:23: E203 whitespace before ':'
159:29: E203 whitespace before ':'
160:26: E203 whitespace before ':'
161:33: E203 whitespace before ':'
162:30: E203 whitespace before ':'
163:30: E203 whitespace before ':'
164:30: E203 whitespace before ':'
165:30: E203 whitespace before ':'
166:30: E203 whitespace before ':'
167:30: E203 whitespace before ':'
168:41: E703 statement ends with a semicolon
170:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from sqlalchemy.exc
170:1: E402 module level import not at top of file
171:1: F401 'collection_restricted_p' imported but unused
171:1: E402 module level import not at top of file
172:1: F401 'restricted_collection_cache' imported but unused
172:1: E402 module level import not at top of file
173:1: F401 'get_permitted_restricted_collections' imported but unused
173:1: E402 module level import not at top of file
174:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.utils
174:1: F401 'get_all_restricted_recids' imported but unused
174:1: E402 module level import not at top of file
177:1: F401 'check_user_can_view_record' imported but unused
177:1: E402 module level import not at top of file
179:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.records.access
179:1: E402 module level import not at top of file
180:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.collections.cache
180:1: E402 module level import not at top of file
181:1: F401 'get_field_i18nname' imported but unused
181:1: E402 module level import not at top of file
183:1: I100 Imports statements are in the wrong order. from invenio.modules.indexer.models should be before from invenio.modules.search.cache
183:1: E402 module level import not at top of file
189: D401 First line should be in imperative mood ('Create', not 'Creates')
189: D202 No blank lines allowed after function docstring (found 1)
189: D400 First line should end with a period (not 'n')
189: D205 1 blank line required between summary line and description (found 0)
197:33: E261 at least two spaces before inline comment
206:1: F401 'get_synonym_terms' imported but unused
206:1: E402 module level import not at top of file
208:1: F403 'from invenio.modules.search.washers import *' used; unable to detect undefined names
208:1: E402 module level import not at top of file
211: D401 First line should be in imperative mood ('Return', not 'Returns')
211: D300 Use """triple double quotes""" (found '-quotes)
232:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.washers
232:1: F401 'get_collection_allchildren' imported but unused
232:1: E402 module level import not at top of file
235: D401 First line should be in imperative mood ('Return', not 'Returns')
235: D202 No blank lines allowed after function docstring (found 1)
238:5: E266 too many leading '#' for block comment
242:5: E266 too many leading '#' for block comment
243:5: E266 too many leading '#' for block comment
245:34: E261 at least two spaces before inline comment
262:13: B901 blind except: statement
267:9: E266 too many leading '#' for block comment
279:9: E266 too many leading '#' for block comment
284:17: E266 too many leading '#' for block comment
293: D400 First line should end with a period (not 'o')
293: D205 1 blank line required between summary line and description (found 0)
293: D208 Docstring is over-indented
338:1: F403 'from invenio.modules.search.searchext.engines.native import *' used; unable to detect undefined names
338:1: E402 module level import not at top of file
341: D209 Multi-line docstring closing quotes should be on a separate line
341: D400 First line should end with a period (not 'y')
341: D205 1 blank line required between summary line and description (found 0)
341: D208 Docstring is over-indented
354:38: E203 whitespace before ','
385:1: E302 expected 2 blank lines, found 0
385: D209 Multi-line docstring closing quotes should be on a separate line
385: D400 First line should end with a period (not 'g')
385: D205 1 blank line required between summary line and description (found 0)
385: D208 Docstring is over-indented
414: D209 Multi-line docstring closing quotes should be on a separate line
414: D205 1 blank line required between summary line and description (found 0)
426:1: I100 Imports statements are in the wrong order. from invenio.modules.search.models should be before from invenio.modules.search.searchext.engines.native
426:1: E402 module level import not at top of file
431: D210 No whitespaces allowed surrounding docstring text
431: D400 First line should end with a period (not 'h')
431: D205 1 blank line required between summary line and description (found 0)
449: D401 First line should be in imperative mood ('I', not 'Is')
449: D400 First line should end with a period (not '?')
460:1: E302 expected 2 blank lines, found 1
460: D205 1 blank line required between summary line and description (found 0)
460: D208 Docstring is over-indented
467:1: I101 Imported names are in the wrong order. Should be CACHE_SORTED_DATA, SORTING_METHODS
467:1: F401 'CACHE_SORTED_DATA' imported but unused
467:1: E402 module level import not at top of file
470: D205 1 blank line required between summary line and description (found 0)
470: D208 Docstring is over-indented
484:20: E127 continuation line over-indented for visual indent
486:25: E126 continuation line over-indented for hanging indent
537: D103 Missing docstring in public function
547: D401 First line should be in imperative mood ('calculate', not 'calculates')
547: D209 Multi-line docstring closing quotes should be on a separate line
547: D202 No blank lines allowed after function docstring (found 1)
547: D400 First line should end with a period (not 'e')
547: D205 1 blank line required between summary line and description (found 0)
558:20: E261 at least two spaces before inline comment
562:17: E261 at least two spaces before inline comment
589:1: E302 expected 2 blank lines, found 1
607: D210 No whitespaces allowed surrounding docstring text
607: D400 First line should end with a period (not 'n')
613:25: E261 at least two spaces before inline comment
614:13: E222 multiple spaces after operator
637:10: E261 at least two spaces before inline comment
654:1: E266 too many leading '#' for block comment
656: D103 Missing docstring in public function
657:25: E128 continuation line under-indented for visual indent
658:25: E128 continuation line under-indented for visual indent
659:25: E128 continuation line under-indented for visual indent
660:25: E128 continuation line under-indented for visual indent
677: D401 First line should be in imperative mood ('Set', not 'Sets')
677: D202 No blank lines allowed after function docstring (found 1)
677: D400 First line should end with a period (not 'l')
677: D200 One-line docstring should fit on one line with quotes (found 3)
678:23: E128 continuation line under-indented for visual indent
679:23: E128 continuation line under-indented for visual indent
680:23: E128 continuation line under-indented for visual indent
681:23: E128 continuation line under-indented for visual indent
682:23: E128 continuation line under-indented for visual indent
720:9: B901 blind except: statement


24:1: I100 Imports statements are in the wrong order. from invenio.modules.search.api should be before from six
24:1: F401 'Query' imported but unused
36:19: F821 undefined name 'SearchEngine'
43:32: E711 comparison to None should be 'if cond is not None:'


Happy Birthday :p :D


36: D202 No blank lines allowed after function docstring (found 1)
60:17: B901 blind except: statement


4: L102 copyright year is outdated, expected 2015 but got 2014
26:1: I201 Missing newline before sections or imports.
28:1: I100 Imports statements are in the wrong order. from flask should be before import six
29:1: I201 Missing newline before sections or imports.
31:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils
32:1: I101 Imported names are in the wrong order. Should be CFG_BIBRANK_SHOW_CITATION_LINKS, CFG_CERN_SITE, CFG_INSPIRE_SITE


24:1: I100 Imports statements are in the wrong order. from flask_login should be before import pypeg2
25:1: I201 Missing newline before sections or imports.


56:1: I201 Missing newline before sections or imports.
57:1: I201 Missing newline before sections or imports.
60:1: I100 Imports statements are in the wrong order. from invenio.base.decorators should be before from werkzeug.local
215:5: B901 blind except: statement
283:9: E731 do not assign a lambda expression, use a def


@Neotmiren - Yes, since our code is based on that commit I would imagine it has


Hi,
try open http://www.watterott.com/ and ee 403 error.
Possible to buy this pcb (empty or mounted) ?

Thanks in advance
Mitek


with `EventStoreSerializer` you'll get `PersistentRepr -> EventData` and `EventData -> PersistentRepr`, so you can separate payload and metadata without changes to core code of the library.


26:1: I201 Missing newline before sections or imports.
28:1: I100 Imports statements are in the wrong order. from flask should be before import six
29:1: I201 Missing newline before sections or imports.
31:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils
32:1: I101 Imported names are in the wrong order. Should be CFG_BIBRANK_SHOW_CITATION_LINKS, CFG_CERN_SITE, CFG_INSPIRE_SITE


make changes


I waited for this one quite a while! Thank you so much! <3


想对作者说一句。从4年前开始使用goagent。goagent帮助自己了解很多墙外世界。很感谢作者的付出。


This change seems reasonable (once the logic is fixed), but I think it needs a couple of other corresponding changes to keep things working as expected, and as documented. The [installation script documentation](http://alaveteli.org/docs/installing/script/) says that this will install a site in development mode by default - I think we should update the instructions on that page so that they show the `--development` flag being used, and then give instructions for what to do if you want to install in production mode. Our own instructions for updating the AMI also now need to be updated to pass the `--development` flag, as the [documentation for the AMI](http://alaveteli.org/docs/installing/ami/) also says it runs in development mode by default.


There are a few syntax issues here. $DEVELOPMENT_INSTALL isn't passed to this script, so isn't defined at this point. Also, this logic seems inverted - if we got the `DEVELOPMENT_INSTALL` flag, then `STAGING_SITE` should be 1, not 0.


This trailing stuff is a bit confusing. I think `) >>= fun () ->` on a new line is easier to read.


Just ensure that they exist


Another mismatch one between hal-msg.h(uint8_t) and hal-ipc-api.txt (4 bytes)...
Which one should we follow in this kind of situation?


I've added DockerConfig.Endpoint validation, as this is the only used in origin for now.


here you could try this import and if it fails you could try importing shapely (`from shapely.geometry import Point, LineString`), if both import fail you can raise an `ImportError` with a nice message that suggests to either install shapely or django, for an example of the see https://github.com/djangonauts/django-rest-framework-gis/blob/master/rest_framework_gis/filters.py#L14-L20


DITTO.


Why do not way this code to overrided ToString() method in your curHddModel class?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.195](http://teamcity.monogame.net/viewLog.html?buildId=7784&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 496, ignored: 6 Build time: 00:22:20


I do not see any sense of having this method. If you like to have logging, please implement extra class Logger with ILogger interface and call this logger instead of asking the DataBase class for log to console.


in place of `request` and `format` put the parameters needed internally


This part could probably be improved upon, but it is completely valid and should work everywhere.


OK, I couldn't get it to compile that way.


Our website is working: http://www.downforeveryoneorjustme.com/watterott.com
You can buy the PCB here:
http://www.watterott.com/en/HDMI-RGB-adapter-with-ATmega32U4-touch-controller


We usually do not use `<br>` to make indents.


@pawelkaczor the problem you could face that Serializer bound for class name, and you don't want to use your for example `JsonSerializer` for `PersistentRepr` with payload from not your domain (akka sharding uses PersistentRepr as well)
To resolve this issue you just match on payload type and may fall back to default protobuf serializer, etc.


this can be an empty dict:

`params = {}`


Some null check?
Mainly for testing but we never know.


Fix mal den Typo hier im Titel. Ich finde auch wir sollten den Titel ändern, aber es sollte das Thema schon besser ausgedrückt werden.


Pero usa `toString()` hombre! :joy:


@Alectradar  it was already there


Die Backslashes sind Absicht, das sind geschützte Spaces, ansonsten denkt TeX das wäre ein Satzende


Handle empty string e.g.``` basedir ="";```


#1 but we are not triggering the org.dy.lint.scala-linter's error ... why?


Its exported just before running `install-as-user` https://github.com/mysociety/alaveteli/blob/rails-3-develop/script/site-specific-install.sh#L192-L193. If its not defined it uses the else.

Oops, quite right about the ordering.


Fix formatting


I usually prefer ```List<String> subsList = new ArrayList<String>();```


Running this script only outputs three of four servers. Is this a problem client-side or server-side?


@KozhinM Can we get into a recursive loop here in case no sidx is found (re: malformed segment) ?


@ahjohannessen here is a screenshot on how I'm using it right now: http://prntscr.com/7ap4iq


You're tracking FWB from CM @Buda1996
:O
Then what customizations will you'll have?


I think it is probably better if we add this to the config.yaml rather than teaching charmers bad habits!


`ELEVATION_DEFAULT_SAMPLING` can be a function parameter with a default value (`50`).

This parameter tells the distance between each point for which we want to get the elevation from the Google API.


OK Jay, thanks. If you can help me to do it!


already did it, we're good...


All right. Is it ok to use a dependency for this (For example: https://www.npmjs.com/package/component-clone), or should the module be left dependency-free?


You're right, but wow that's confusing! Maybe this needs a bit more thought to ensure the script _is_ suitable for a non-development install.


Primer commit de la aplicación. Generado con express-generator siguiendo las instrucciones del curso.


este fichero se te ha colado también en el commit, antes de hacer el commit es interesante ver que ficheros entran


Hi Varun, Some coding style nits have been flagged by our CI. Could you at least fix the errors?

```
Stacktrace

  CHECKING STYLE
WARNING: Do not use whitespace before Signed-off-by:
#23:
    Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>

WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#26:
new file mode 100644

WARNING: line over 80 characters
#530: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:123:
+	tegra_fc_write_32(FLOWCTRL_L2_FLUSH_CONTROL, !!MPIDR_AFFLVL1_VAL(mpidr));

WARNING: line over 80 characters
#550: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:143:
+	tegra_fc_write_32(FLOWCTRL_L2_FLUSH_CONTROL, read_midr() == CORTEX_A53_MIDR);

ERROR: do not use C99 // comments
#635: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:228:
+		; // wait till value reaches EVP_BPMP_RESET_VECTOR

ERROR: trailing statements should be on next line
#640: FILE: plat/nvidia/tegra/common/drivers/flowctrl/flowctrl.c:233:
+	while (val > mmio_read_32(TEGRA_TMRUS_BASE));

WARNING: line over 80 characters
#720: FILE: plat/nvidia/tegra/common/drivers/memctrl/memctrl.c:67:
+	(void)tegra_mc_read_32(MC_SMMU_CONFIG_0);	/* read to flush writes */

WARNING: line over 80 characters
#724: FILE: plat/nvidia/tegra/common/drivers/memctrl/memctrl.c:71:
+	tegra_mc_write_32(MC_SMMU_CONFIG_0, MC_SMMU_CONFIG_0_SMMU_ENABLE_ENABLE);

WARNING: line over 80 characters
#725: FILE: plat/nvidia/tegra/common/drivers/memctrl/memctrl.c:72:
+	(void)tegra_mc_read_32(MC_SMMU_CONFIG_0);	/* read to flush writes */

ERROR: do not use C99 // comments
#866: FILE: plat/nvidia/tegra/common/drivers/pmc/pmc.c:116:
+	reg |= 0x10;		// restart

WARNING: externs should be avoided in .c files
#929: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:51:
+extern unsigned long __RO_START__;

WARNING: externs should be avoided in .c files
#930: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:52:
+extern unsigned long __RO_END__;

WARNING: externs should be avoided in .c files
#931: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:53:
+extern unsigned long __BL31_END__;

WARNING: externs should be avoided in .c files
#934: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:56:
+extern unsigned long __COHERENT_RAM_START__;

WARNING: externs should be avoided in .c files
#935: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:57:
+extern unsigned long __COHERENT_RAM_END__;

WARNING: externs should be avoided in .c files
#938: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:60:
+extern uint64_t tegra_bl31_phys_base;

WARNING: externs should be avoided in .c files
#971: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:93:
+extern uint64_t ns_image_entrypoint;

WARNING: line over 80 characters
#1025: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:147:
+		plat_bl31_params_from_bl2.tzdram_size = plat_params->tzdram_size;

WARNING: line over 80 characters
#1026: FILE: plat/nvidia/tegra/common/tegra_bl31_setup.c:148:
+		plat_bl31_params_from_bl2.bl32_params = plat_params->bl32_params;

WARNING: externs should be avoided in .c files
#1327: FILE: plat/nvidia/tegra/common/tegra_pm.c:46:
+extern uint64_t tegra_bl31_phys_base;

WARNING: externs should be avoided in .c files
#1328: FILE: plat/nvidia/tegra/common/tegra_pm.c:47:
+extern uint64_t sec_entry_point[PLATFORM_CORE_COUNT];

WARNING: please, no space before tabs
#1838: FILE: plat/nvidia/tegra/include/drivers/memctrl.h:50:
+^I (TLB_FLUSH_VA_MATCH_ALL | ^I\$

WARNING: line over 80 characters
#1987: FILE: plat/nvidia/tegra/include/plat_macros.S:41:
+	.asciz "gicd_ispendr regs (Offsets 0x200 - 0x278)\n Offset:\t\t\tvalue\n"

WARNING: do not add new typedefs
#2276: FILE: plat/nvidia/tegra/include/tegra_private.h:37:
+typedef struct plat_params_from_bl2 {

ERROR: do not use C99 // comments
#2542: FILE: plat/nvidia/tegra/soc/t210/plat_secondary.c:40:
+#define SB_AA64_RESET_LOW		0x30	// width = 31:0

ERROR: do not use C99 // comments
#2543: FILE: plat/nvidia/tegra/soc/t210/plat_secondary.c:41:
+#define SB_AA64_RESET_HI		0x34	// width = 11:0

WARNING: externs should be avoided in .c files
#2545: FILE: plat/nvidia/tegra/soc/t210/plat_secondary.c:43:
+extern void tegra_secure_entrypoint(void);

WARNING: Do not use whitespace before Signed-off-by:
#2708:
    Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>

WARNING: Duplicate signature
#2708:
    Signed-off-by: Varun Wadekar <vwadekar@nvidia.com>

total: 5 errors, 24 warnings, 2796 lines checked

NOTE: Ignored message types: COMPLEX_MACRO GERRIT_CHANGE_ID GIT_COMMIT_ID

Your patch has style problems, please review.

If any of these errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
```


Can rootDir be null?


la ruta mejor que sea relativa


Where should that be applied in this diff?


Why not direct using %Ecto.Query.JoinExpr{on: %Ecto.Query.QueryExpr{expr: true, params: []}.... ?


No need. Since we do not use yaml, it does not even break the every commit should compile -rule.


I would prefer not deal with akka types used for persistence and still be able to use my payload type for eventtype + store metadata from my envelope. I suppose I need to experiment in order to identify possible painpoints :)


this can be removed, the number of samples (points requested) will be determined automatically by calculating the length of the path and diving by the `sampling`param


I would assume this should be covered by the `!baseDirFile.isDirectory())` check but I'll double check.


también se ha colado este fichero en el commit


Изменил. Теперь массив url


OK.


watch out for this, unfortunately the google elevation API can return the elevation for a maximum of 512 points at each request.


The same as hier %Ecto.Query.QueryExpr{}


Can you show json document as well? Thanks.

I spent a couple of hours trying to get rid of fork. So I'm almost sure I
need it :) I know its a pity I can not give concrete explanation now. I
remember the logic of serialization/deserialization was not so obvious.


also need to quote `$DEVELOPMENT_INSTALL`


Ah, I hadn't seen the significance of the export. However, when `$DEVELOPMENT_INSTALL` is undefined, the script produces the warning "[: =: unary operator expected", as the syntax effectively expands to

    if [ = true ]

so you should quote the `$DEVELOPMENT_INSTALL` variable:

     if [ "$DEVELOPMENT_INSTALL" = true ]; then


for {inner, extern} <- [left: "left_join", right: "right_join", full_join] do
  def build(query, %{unquote(extern) => join_arr}), do: build(query, unquote(inner), join_arr)
end

Something like it, saves much place


I tried removing lib and behaviour doesn't change. I'll remove 'lib/', thanks :)


WTF? Why on earth was this code included in the first place? Connecting over HTTP to get random numbers??


The advantage is to warn the developer that he adds something to the context and called it `view` but it is not a Django REST framework style view instance. He should not do this but either use such a view instance or rename the context object. The warning is important at this point of the code because the code silently ignores this misconfiguration and assumes that the developer does not want to generate a username what might be wrong.


this is mostly ok, just return `OrderedDict` in place of `Response`


Well, when I run curl on my local machine I only get four servers, so I'm assuming it is server-side.


Wenn ihr auf "view Raw" geht könnt ihr die Datei downloaden und in eurem Excel anschauen.


```json
{
  "sequenceNr": 7,
  "persistenceId": "GameTable-bfxoiebwyg3ytxcv",
  "deleted": false,
  "payloadClass": "com.xxx.xxx.GameTableActor$Event",
  "payload": {
    "type": "TableLeft",
    "value": {
      "userId": "8xn0vbo1469noeam",
      "kicked": false,
      "timestamp": "2015-05-29T10:37:58.37Z"
    }
  }
}
```

so it is all together payload and `PersistentRepr`, but separation can be done on `EventStoreSerializer` level in `PersistentRepr -> EventData` and `EventData -> PersistentRepr`


I think i got it: duplicate of https://github.com/sbesson/bioformats/commit/c4ad1896e26a11290da029ccfe6a94e45e033f3f#commitcomment-11424485


Why is this line commented out? This is necessary - at least for me. Any reasons for this?


This would be a client side issue somehow. I'm not sure why, I'm looking into it.


:exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation:
WTF? Am I missing something or will this result in XSS when intuitively used, e.g. `{$comments->text|nl2br}`
:exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation: :skull: :exclamation:


Hast du vielleicht ne Ahnung, warum mir der font nicht angezeigt wird?


Which line/variable does this apply to?


it's confusing to set Bold, Underline and Strike through Set* methods and Italic through set property accessor.
A unified design would makes things more clear.


Overall if that separation feature is everyone want, I can make it to be configurable and put in to the plugin sources...


I decided to use a linkify list group from the Bootstrap components, which meant that I had to use a div.


clause


@jamesotron not that I want to turn this into a big pros/cons thread, but I think @matthewrudy has a point.

I'd say we all agree that `I should have confidence that the CI service is building exactly the code that I specified when I committed` .. which actually focuses the question on *what exactly did I specify*?

Since for gems, the `Gemfile.lock` is not part of the specification that a gem user receives or uses at runtime, it is kind of irrelevent. All that matters is what the gemspec says. So that implies the only relevant CI build would be against the gemspec, not Gemfile.lock.

And I'd +1 "Ideally I'd say travis should re-run whenever new versions of your dependencies are released and track against them." .. then next we can ask for full matrix regression testing a la appraisal (but automated);-)

OTOH, I do commit Gemfile.lock for apps and other code where Gemfile.lock is actually part of the deployment (i.e. where it is actually playing the role of runtime dependency specification).


Interesting @t3hnar :) thx for screenshot. Would it not make sense to keep akka persistence messages hidden from custom eventstore serializer hook-in methods. Or am I not seeing something obvious?
Main priority is custom serializer + proper eventtype and ability to store my metadata without worrying about akka message types like persistent repr and similar.


why not add a setPdo method ?


By the way do you mind changing the name `getResource` to `getPdo` ?


join_models |> Enum.map(&String.to_atom/1) |> Enum.find_index(&(model == &1))


Why not something like `fields_with_types[field]` ?


# Final Version
* Major Bug Fixed.
* All 17 tests passed

# TODO
* Check answer.txt if the name, UID, and email are correct.

# Improvement can be done
* Comments


TeamCity xBIM :: XbimWebUI :: Build [Build 58](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1771&buildTypeId=XBIM_XbimWebUI_Build) is now running


@torchtrust Apart from the typeform tests, these three tests also fail:

{"input": "Ææ", "expected": "⠠⠁⠠⠘⠖⠑⠁⠘⠖⠑", "received": "⠠⠁⠘⠖⠑⠁⠘⠖⠑"}
{"input": "Õõ", "expected": "⠠⠘⠻⠕⠘⠻⠕", "received": "⠄⡳⠭⠴⠴⠙⠢⠄⠘⠻⠕"}
{"input": "ÿ", "expected": "⠘⠒⠽", "received": "⠰⠘⠒⠽"}

You think it's a problem with the test data or the table? You can view all your changes combined here: https://github.com/liblouis/liblouis/compare/83102fea36485fde1f38b5a88a6968170a4b7fa4.


TeamCity xBIM :: XbimWebUI :: Build [Build 58](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1771&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:21


@ahjohannessen I don't know business needs of all users, thus `EventData -> PersistentRepr` & `PersistentRepr -> EventData` gives the most flexibility. Also `PersistentRepr` is the API of akka persistence used by other plugins...
So you could end up using same `custom` serializer with different plugins. Next akka version will provide ability to configure plugin on per actor basis.


Just mark file with readonly attribute and VS won't change it.