I think it should be in format of `ValueError: in case of a bad fit`


The best way to bump versions on projects is `npm version [major | minor | patch]`, which changes the version in package.json, commits and tags the commit.


Assigning function to a variable looks superfluous for this case
One more hint: I'd suggest you making function to be an implicit parameter:
Function.prototype.partialFunction = function() {....
This looks more elegant, and helps you to avoid compromises in ordering your parameters


LSST coding standards have example with all-uppercase constant, so I would like to keep constant name all-uppercase:
https://confluence.lsstcorp.org/pages/viewpage.action?pageId=16908706#C++Statements-Miscellaneous


This code should be
```
if (i == j) tmpVector[i] =1.0;
else if (i == j-1) tmpVector[i] = -1.0;
```
the other case is superfluous, because your vector already has the right size and is zero-initialized.


```i```


I don't know, I have to see it in practice.


No, removed :+1:


Yes. Packages that provide "doom" (i.e. Provides: doom-engine and register a binary as /usr/games/doom -> /etc/alternatives/doom) have an expected command line interface, of which doomsday arbitrarily diverges.


say ```j```


see above


this is dead code!


This statement binds you to a definite type (i.e. number)


upload/catalog/controller/product/compare.php
'minimum'      => $result['minimum'] > 0 ? $result['minimum'] : 1,
should be
'minimum'      => $product_info['minimum'] > 0 ? $product_info['minimum'] : 1,


This looks good, IMO we should start cherry-picking those patches as every commit generates the need to rebase it...


For large applications I suspect we won't reach termination very often, so if we are running out of memory in your examples this may be a big problem (the number of pending but non-terminiated states should usually outnumber the number of terminiated states). Are you giving the JVM enough memory? You might need to specify some arguments to the JVM to give it more memory... -Xmx16g


we should have a follow-up patch that changes that address to `https://slack.midonet.org`


sans s ? Je fixerais ça à la limite, j'vais pousser un commit sur ta PR :)


This is an error; can you see why?


```const```


say ```const```


I'm not really sure about this phrasing. ML posting is not just for these cases. The main case is for discussion that needs lengthy explanations and references.


fixes amazeui/amazeui-react#19


Changed it the suggested way. Even though I can not get rid of the incrementVersion.


why do you need pointers here? Please get rid of them and just use your ```GeneratorList```


Done.


Evil trailing line!


ok, thanks ;)


Initial commit. Need to understand more why this has to be done.


I don't understand. You have just changed the order in which `connected` event is fired, nothing else. `dynConfiguration.register` is not modified.

> Regardless of the UA being registered previous to disconnection, the UA was registered on connected callback.

Why? it was just in case `dynConfiguration.register` was true...


You're awesome! :)


Acked-by: Lukáš Doktor <ldoktor@redhat.com>


you found the error ;)


Again, great candidate for cherry-pick...


most of the comments are for future changes, I only object to the trailing line. What's the fastest way to see the effect?


Same thing here: Declare your ```tmpPoint``` here where you need it, not someplace else


Removed the line.


@joao-manywho I'd prefer this if it was navigation.classList.contains instead of indexOf


Declare your iterator here, not someplace else


ack, since this is purely `process` related exception


Why do you need a else condition? from L112 it looks like the `args.extents` will automatically fallback to `None` if not specified, right?


This is very very dangerous. You modify the tree structure of the ```pointOrbit``` by inserting a new point while you still have iterators running over it. This is bound to cause trouble!


Need to check storyboard changes before commit. That changes should be discarded.


yep


Sure thing, i will commit something, anything else? Also, have you looked at the different logging implementation?


There is no way that you can leave a ```goto``` in your code :)
Please rewrite without ```goto```!


This will break the entire API of the library. If this is enabled, `jsonpath` will return an object containing `JSONPath, handlers, parser` and API would look like:

    jsonpath.JSONPath.*

Please remove so the API can stay unchanged across node and browsers.


Also, you may not only expect a certain behaviour if you call `doom` from a command line but also if it is called from the `Exec` field of a .desktop file.


@zhiayang nah, both


@jglick why not keep the legacy method + deprecated ?


Note that ```std::vector```s have a ```swap``` function, so there's no need for pointers!


This code was leading to false detection of UTF-8 encoding sometimes, when the source file was actually in ASCII encoding. So, I fixed it.


Not sure that would be helpful? The docstring is for the Gaussian kernel in the sourcefinder. Documenting that it returns a function is more descriptive that saying it returns a lambda, IMO, though I appreciate it's basically the same thing.


I'm wondering why don't we change the name of the `path.py`, it'd safe lots of `as` imports.


ack, this looks much better.


@henkmollema enig idee waarom `jQuery.isEmptyObject(data.data)` geen `true` wordt terwijl de server wel `{"data":{}}` teruggeeft?

- http://api.jquery.com/jQuery.isEmptyObject/


Please move this along with others in the deployment-fragment.xml (just for concistency, we should probably move them to a local faces-config file like this at some point).

Otherwise looks good


Hmmm... I think `meetup` is alphabetically lower than `membership`. `e < m`


all cassandra unit test run now, depending on the unittest config for OnlyIfCanRunSpark, could be merged to upstream/feature/cassandra


Reverted in 9f7d7adb0.


Ah yes, bet you're right. Was fixing these at around 7pm on Friday...


a lambda is an anonymous function which is what it returns. Also Sphinx will eat that syntax, while it will barf on `function`.


me too! :)

```
commit 5a9402199b4eec5b9c5885549a98c7dc9d752277
Author: Gijs Molenaar <gijs@pythonic.nl>
Date:   Fri May 29 17:35:31 2015 +0200

    fix most of the doc warnings
``


Clear now after a chat :)


DId you run `flake8`? I think it will complain about one missing newline.


@b0rn2c0de update your sdk :p


予測通りではあるが文字化けがひどい


@WesselKoorn  dit werkt wel:
http://jsfiddle.net/qf45abnd/


Awesome! Bundled into poi.


Why "wing-damage" here if it ends to immediately break both?


kudos, smartass.


@jdubois I noticed that earlier there was this console log that lets the user know that npm install and bower install is in progress, but after the yeoman 0.20 mig, this seems to be gone, and since there is some delay in npm install firing up, it might be a bit confusing for new users as they might think the process is stuck


its not from there lol


thats a clone of the official linux kernel repos


Yep. The move of `sp = cam_extern_sp` along with the new delegation code (#8) fixes #4.

You are right about the `CAMLlocal`s. The only one I am unsure about is the `cont` in `caml_continue`, which invokes `use_continuation(cont)`. Can promotion invoke a major collection? In that case, we might need `cont` to considered as a root? Anyway, this isn't necessary for #4.

Submitted a PR reverting the changes to `CAMLlocal`s #10. Feel free to merge the pull request.


Fair point! OK, let's switch it to lambda.


yeah i noticed that synology discussion and figured might as well add this


Technically is not a Archbuilder fork...


@DarthAndroid Following a bugreport, I've tried this on a hello-world project and it seems that `maven-compiler-plugin` is run **after** `kotlin-maven-plugin` and so Kotlin symbols are unresolved in Java. Is this the complete configuration or am I doing something wrong? (The original report is [KT-7961](https://youtrack.jetbrains.com/issue/KT-7961).)


ack


Google's repos are still on 3.10.57 @Swapnil133609


Is there same reason to have the same methods for String and for CharSequence? String implements CharSequence as interface.


thanks for correction


Spelling mistake?


How do you get past the 'N' ?


![screenshot from 2015-06-08 16 16 33](https://cloud.githubusercontent.com/assets/10119037/8033204/001a9a48-0dfa-11e5-9449-bedbe41901fa.png)


Maybe better link to repo https://github.com/ngryman/raf.js ?


-Управление клиентов: этот вариант использования позволяет игрокам создавать и обновление
записи, связанные с клиентами, как это и должно обеспечить отслеживание клиентов, так как позволяет
привлекать контактов клиента, если клиент компании.
- Управление заказов: управлять и отслеживать заказы клиентов с печатных отчетов.
- Управление счетов: этот случай обеспечивает создание и обновление счета, а
позволяет отслеживать счета до последнего шага не оплата.
 - Управление продукты: включает в себя создание продуктов или обновлений. Кроме того,
этот вариант использования позволяет просматривать изменения запасов и отчет
Недоступные потребности изделий.
- Мониторинг деятельности: Содержит все панели, которые делают анализ
показатели продаж магазина (например, объем продаж за месяц, число продаж
за клиента, анализ счетов).


Sorry, I had not seen well. I added a comment here by error. Deleted, everything right.


Did you have any reproducable specific issues with that? Installation directory is created with Ansible one task before the path registration task.


:+1: looks cool!


oops, will do!


This is a step in the right direction but I have a number of comments.

1. This appears to be copied and then modified from Geronimo. Having the copy and the modifications in the same commit is far from helpful since it makes it much more difficult to review your changes. A series of small commits (which can be collapsed if necessary) is much better than one big commit. I am currently unclear as to what you have modified from Geronimo and why you have modified it.

2. Geronimo implements the JASPIC 1.0 but the latest version of the JASPIC spec is 1.1 (which is what you should be implementing). The proposed manifest is correct but the API is not.

3. The build file integration looks good (I haven't tested it though).

4. Checkstyle fails with numerous errors.

Each of the following need to be in at least one separate commit:
- copy from geronimo
- checkstyle fixes
- JASPIC 1.1 updates
- each of your changes (each change explained in the commit)

To assist with cross-project merging (so Geronimo can pick up any changes we make and we can pick up any changes Geronimo makes) the copy needs to be an svn copy. It looks pretty straight forward to do this so I'll do it shortly and your updated pull request can be based from there.


/**
c
*/


???
You don't need to compare boolean with boolean


// what should this do


You probably know that after this commit, there's a new warning.
```
In file included from truecrypt_fmt_plug.c:55:0:
pbkdf2_hmac_sha512.h:201:13: warning: ‘pbkdf2_sha512_sse’ defined but not used [-Wunused-function]
 static void pbkdf2_sha512_sse(const unsigned char *K[SSE_GROUP_SZ_SHA512], int KL[SSE_GROUP_SZ_SHA512], unsigned char *S, int SL, int R, unsigned char *out[SSE_GROUP_SZ_SHA512], int outlen, int skip_bytes)
```


Just in case we should make the order of these functions the same as the flowOut function above.


"or" stands for "orphan": symlink to nonexistent file, or non-stat'able file


Good work.
1. IMHO, no needed to calculate timeline(zero hour etc), you can do date_time by paste and strptime command and make plot  ~ date_time (if you do it right, no need label = Thu, Fri, Sat, plot does it automaticaly).
2. You have small mistake on plot 4 - no needed to do a border for the label area. (by parameter: bty="n" )


Is the namespace unset now? Or is it set in the builder?


Why was `createElement` removed? Defined in another level?


Oh yeah, I knew I'd forgot something. I doubt it could be random garbage though since its on app start. thanks :D


Shouldn't we use `Symbol` if we introduce new members? Or stay with _ vars in old files?


Ikke helt færdigt. Der er stadigvæk en != "".

Prøv:
if(obj.getIaActivityCode().equals(testelement.getIaActivityCode()) && !"".equals(obj.getIaSubActivityCode())


space between `if` and `(` (was there in previous commit actually).


space between `if` and `(` again.


对。。。我又来厚脸皮的给自己来评论了，点个赞先！


Even local, I think you can find a better name. As-is, it says "it's a stack, and it's used in ref filtering (because it belongs to ref-filter.c)", but does not say either what it contains nor what part of filtering it's used for.


You need 6 cells of distance to get 200% ATK


Woops, didn't realize you moved it.


d


yep, you can change that if you want


IMHO it would make sense to have settings.local or settings.local_vagrant because there more stuff we add to .production the more changes we will have to do in local and I don't think it is best practice to print some variables to python module. I would keep `cp` but just update paths.

Also, I guess this is a keyboard typo, you want to use `echo` here instead of `cat`.
So if it is up to my, I'm sorry dude but this can not go in. :-1:


well i can add on this i think, i accidently readded this deleted function when merging and i think bene just redeleted it


how do you retrieve cached function?


`string.rep(s,n)` bietet sich hier an


FYI, it should be just "win" (cf: http://www.nunit.org/index.php?p=platform&r=2.5)

Otherwise:
```
Test Name:	GivenXAMLThatUsesLibraryClass_MergedWPFApplicationRunsSuccessfully
Test FullName:	ILRepack.IntegrationTests.WPFScenarios.GivenXAMLThatUsesLibraryClass_MergedWPFApplicationRunsSuccessfully
Test Source:	E:\src\p\il-repack\ILRepack.IntegrationTests\WPFScenarios.cs : line 16
Test Outcome:	Skipped
Test Duration:	0:00:00

Result Message:	Invalid platform name: windows
```

I've fixed it here as well: https://github.com/gluck/il-repack/pull/111


mach die commit message bitte mit `docs: initial version`, weil wir fügen ja kein Feature hinzu. Und `package.json` changes separate comitten


This can crash with a NPE when player.getUniqueID() == null.
Also I am fairly certain that the bug was already fixed.


nice!


docOrId, not docOrIdToId


can you please review your comments, they do not follow the same convention, and the names of the variables to do not always match the name of the param of the function


Refactored user sign up to send authToken as an object in the body. The error I am having while running this test is from the authentication middleware. @barodeur Thoughts?


what is this?


@RTLcoil anywhere were you're using embedded videos


can you please reword the commit message? There is no `update` prefix, see https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#-git-commit-guidelines

use `docs(README): ...`


Update #17


No you're right this was always made to supplement irc-osric, so the change is fair enough. Is this all that is needed though? The same thing happens in Class.pm, Race.pm and every file in the Class and Race directories.


Please remove "Not Reviewed." comment.


...word!


Управление закупок заключается в управлении входящие и исходящие товары в наличии в
ее руководство закупок продукции . Эта диаграмма включает в себя общую функциональность , которая будет представлена в системе в управлении закупок и тогда мы будем подробно схема , чтобы показать полезность каждого прецедента


чисто для пробы.


Should replace this code for selecting section to - (JMSaveReportSection *)rangeSaveReportSection method and make it mo universally by adding SectionType as parameter.


About those "+<<<<<<< Updated upstream" lines added by git:

you have to fetch the current Github code (=upstream) to your local repository before you commit your (local) changes back to Github.


Não colega ajudou


tracking it in https://bugzilla.suse.com/show_bug.cgi?id=933923


Ja, das könnte man da nutzen. Auch wäre es interessant die Uptime schöner darzustellen. Aber möglichst ohne ```sed``` und so. Gabs dafür eine lua-library?


Nice to meet you, Your Name! ;)


How to open the file


line 16 may need correct path:

 +* The different file locks which the shd takes for each of these heals is detailed in *doc/developer
+-guide/afr/afr-locks-evolution.md*


Where the file can open csproj? .sln


ok, ok, but there is no .desktop file with a plain `Exec=doom` in any package


Is this still supposed to be in this merge?


It kind of got dropped while I fiddled with other things but is at the top of today's tasks, so I'll let you know.


.button.button?  This seems odd.


It'll be removed when I push the new version of `mas-rad_core`. PR'd anyway since it doesn't block review


`luci.sys.uptime()`


Relates to de52ade


This will disable chatting as (Console).
ply wil not be valid, and the base function will be not be called.


The LU is checked for UAs prior to the reset via a TUR prior to the reset call, which I guess would also fail for targets with queued UAs.
Should I change both calls to iscsi_testunitready_sync(...) loops?
Also, I don't quite understand "iterate over it until all async events have been consumed.". Do you mean loop until the reset UA has been flagged and cleared?

Thanks for the feedback Ronnie.


> However, a typical synapse ofen deals with the spike by adding the weight to the post neuron, i.e. something like pre='v_post += w'

Um, sure, but in this case you don't get any warning, do you?


Fatal error: Call to undefined method HG\Main::sendToAll() in /home/pm/HGtest/plugins/HG-Fixed/src/HG/Main.php on line 1162


I wanted this if statement to detect when both a and b are `undefined`, and return true if so. Chasing it to what you suggested above seems like it might return true for more than just two instances of `undefined`, wouldn't it? If so, why would it be a good idea to do that, and what cases would it catch?


Done,
Thank you Humble :)


Why is this row commented partially?


I have a request to change from: "TimeToDestination": "Czas do przyjazdu:" on "TimeToDestination": "Czas do celu:". Thanks.


waarom in t nederlands ?


Please remove this unneeded gitignore file.


conf.modules.d/*.conf shouldn't contain a VirtualHost
Is that the idea of REJECT_FORMAT_PATH_TO_CONTAIN_VHOST_FILES logic?


Please move all constants to the separate ProvisionConstatns class. provision.gif used in 2 classes - duplicating.


Can you submit this request to https://github.com/mkoch227/ets2-mobile-route-advisor/pulls?


Refactoring suggestion. Thinking about "gaze_positions" and "pupil_positions", they should be called "gaze_data" and "pupil_data", hum?


null check needed


include initializer_list


size check needed


This is likely to get optimized by the compiler and allows us to do some debug checks. IIRC `setState()` will assert on debug mode if you are not changing from a final state.

Also,  `mayStartParsing()` sounds easier to understand then `parsing.test_and_set(std::memory_order_acquire)`.


generate correct serialVersionUID


You're right. I think I was trying to get around some precedence issues with the way stylesheets are brought into Dough, but it doesn't seem to be necessary now.


I like how the buffering's being done, here; I'm going to copy this for my project, to be kinder to the Riemann server receiving samples.  Wouldn't it be better to pre-allocate the buffer?  Otherwise the runtime's going to have to reallocate the underlying array frequently.


try {
 							createTopologyButtonMouseClicked(evt);
 						} catch (Exception e) {
 							// TODO Auto-generated catch block
 							e.printStackTrace();
 						}

Add message for the Exception


This, and other examples have inconsistent usage of tabs vs spaces.


KK


strlen() + 1, really? The same mistake in ssl example.


Changing the router timeout had some effect btw, thanks for pointing that out.


@lefticus Looks good to me


Also please refactor CreateProperties. It shouldn't work with row30 etc.. Big index value in the name of variable - bad codestyle


what did you change here?


Should probably be "/media/"


The requirement of our project is to sort the data by a songname. However, if the user would like to sort the data by song ID, we can add the following code in this file (src/ParsingJason/AlbumDTO.java):

private int id;
...
public AlbumDTO(@JsonProperty("id") int id,
...
this.id = id;
...
public int getId () {
return id;
}

public void setId(int id) {
this.id = id;
}

If the user does want to sort the data by song id, for loop at the end of our sort file (src/ParsingJason/JacksonUsage.java) will use id instead of song:

for (AlbumDTO id : album) {
System.out.println(id.toString());
}


Should be "front matter", hum?


Das wär ebe nu wichtig, dass es blockierend isch. Ich nime susch wider d 'orginal' erchlärig drii, die isch usem tannenbaum, isch zwar chli n overkill aber ich has gfühl mer tscheggts na recht guet was d idee vomne omega netzwerk isch, isch guet?


Also add link, please.


Why not keep 'sails_mysql' as the db? 'sails_unitTest' is a bit misleading since the same db is used for both unit and integration testing.


Okay that was i mistake @akhilnarang


@lauraluiz Ok, good to know!
Can't wait to work with the *new* system.


I'm jobless in life, so don't get bugged :P


I just removed these changes in the next commit, not needed.


@jorangreef `process.platform` is always 'win32' on Windows, the 32/64 bits discriminator is `process.arch`.


- Управление запасами : этот случай позволяет контролировать входящих и исходящих продуктов и
Обновление акции, если такой продукт поставляется она должна быть удалена
автоматически , как это делает обновление, если продукт приобретен .
- Управление поставщиков : Создание записей поставщиков и
обновить сразу возникает уведомление , чтобы добавить .
- Управление заявок : позволяет отслеживать запросы на покупку , что
может создавать, обновлять и анализировать их .
- Управление счетов : этот случай обеспечивает создание и обновление счета ,
так как позволяет отслеживать счета .


LGTM


Ebből kijön az, hogy $\chi{G_{k+1}} --> Ebből kijön az, hogy $\chi(G_{k+1})


That's exactly the point. :-) When the interface contains `*js.Object` values, then their equality is defined by `===` on their JS values. The bug was not only about `undefined`, but about comparing interfaces in general. E.g. it also applies to:
```go
var i interface{} = js.Global
println(i == js.Global)
```
Maybe the test should be extended and renamed accordingly.


This is the key step. Only check for sudo password if it's baremetal.


Is this really a fix or does it just happen to not crash?  What was wrong with `__wargv`?


I have reread the standard carefully, and I think it is correct to accept "input wire time" ports. Below you can find my reasoning with references to the SV-2012 standard:

* Grammar rules that permit "input wire time" port declaration:

net_port_header ::= [ port_direction ] net_port_type

port_direction ::= input | output | inout | ref

net_port_type ::=
    [ net_type ] data_type_or_implicit
    (...)

net_type ::= supply0 | supply1 | tri | triand | trior | trireg | tri0 | tri1 | uwire | wire | wand | wor

data_type_or_implicit ::=
    data_type
    | implicit_data_type

data_type ::=
    integer_vector_type [ signing ] { packed_dimension }
    | integer_atom_type [ signing ]
    (...)

integer_atom_type ::= byte | shortint | int | longint | integer | time

* Data type

Table 6-8 in point 6.11 describes time as "4-state data type, 64-bit unsigned integer", which sounds to me as equivalent of "logic unsigned [63:0]", as you have also noticed. Port declared with "input wire logic unsigned [63:0]" is accepted by Icarus and works as expected.

Regarding the doubts about changing the meaning of "time" type, here is what Verilog-2005 says:
"Almost all of the data types in the Verilog HDL store all four basic values." (4.1)
"The time variables shall behave the same as a reg of at least 64 bits, with the least significant bit being bit 0. (...) In contrast, integer
variables shall be treated as signed regs with the least significant bit being zero." (4.8)

It seems to me that time is also defined as 4-state data type of 64-bit width, equal to 64-bit unsigned integer in SystemVerilog.

Also, if time is a particular integer type, then there is an example in the standard that uses "input wire integer" (very end of 23.2.2.3 point):

    module mh22 (input wire integer p_a, .p_b(s_b), p_c);
    logic [5:0] s_b;

I am going to leave the final decision up to you, as logic unsigned [63:0] will also work for me. Perhaps the reg_flag should be based on the used standard?


Will change this back to 3 seconds.


parentheses instead of curly braces


rsdtrdtdtrdr


I have code in my Entity:
Doctrine\ORM\Mapping\Entity(readOnly=true)

I get an error:
Doctrine\DBAL\DBALException: An exception occurred while executing 'DELETE FROM table':

I think, you should add
```php
|| $class->isReadOnly
```
to selected condition


U should really use meaningful comments for you commits


well, in my case, I defined a variable I to receive the pulse, and I write pre='I += w', but I do get the warning.


I think this is replaced by board.js


应该是 `JavaScript ` 而不是 `JAVASCRIPT`


Hello @mkbond777,

Setting cart_rule_restriction to 1 prevent two cart rules to be cumulated if they both have cart_rule_restriction to 1 to 1. Indeed, if one cart rule is cumulative (cart_rule_restriction set to 0), it can be cumulated with any cart rule even if the second one has cart_rule_restriction set to 1. This is the default behavior of PrestaShop.

Best regards.


能把 Menu 封装到 Dropdown 里吗，作为 `dropdown-item`，这样只暴露出一个接口。


I see, I didn't realize that was also an issue, and my changes in this PR therefore focused only what was described in #237.

Do you think it's better to keep the scope of this PR as it was (and handle the remaining issues in a follow up), or expand it?


i mean stat like benchmark or something comparing it to the old protocol


@mathuin : I think you deleted this file by mistake ? I'm getting import error when pgd.settings is being imported.


v bnvbnv


menu 和 dropdown 没关系...


That's good news. I'll mention it to ST/Luis for the next version of the tools.


no plural


The point is that we only support Linux, and, even the Python Standard Library has modules that only work on specific platforms in a flat namespace. I could live (better) without it.


They are. Github flow is slow. It blocks deploy until the review happens, which goes against our deploy AS SOON AS HUMANLY POSSIBLE. Branching off stuff increases the ceremony for every change as well, so at least I think it would lead to larger chunks of code to review/deploy.

This works for us, but I can see how it could be a problem for others, depending on your needs or organization. We tolerate some bugs being shipped, but we are also a tight team and bugs are really not that common. It helps with good test coverage too.


This would fix the full bug. Does not sense for me to first just fix a special case for it first.


> I defined a variable I to receive the pulse, and I write pre='I += w'

Are you sure you not doing something more complex than that? If I run the following I don't get any warning:
```Python
G = NeuronGroup(1, '''dv/dt = (-v + I)/(10*ms) : 1
                      I : 1''')
S = Synapses(G, G, 'w : 1', pre='I += w', connect=True)
run(0*ms)
```


@Ood-Tsen, the linked test content does seem to use 'matroska' as the DocType, and the [documentation](http://www.matroska.org/technical/specs/index.html) indicates it is lower-case: "A string that describes the type of document that follows this EBML header. 'matroska' in our case or 'webm' for webm files." Have you found some other sample data where the DocType is not 'webm' or 'matroska'?


When? Of course if this isn't top secret? :)


This should probably actually be removed. PrepItem is a very weird call I added for I Don't even remember what.


Shouldn't this rather go to https://github.com/mono/monodevelop/blob/c27adc3fe836dbd5fd9b975c2f02f079eef2084f/main/src/core/MonoDevelop.Ide/MonoDevelop.Ide.FindInFiles/SearchResultWidget.cs#L72 ?


penghapusan code:
````javascript
fx = require('bespoke-fx');
````

memperbaiki tampilan yang tidak sesuai.


1 round = 1 group of shots, not 1 shot.


else if datum == 0 reload();


是否可以这样呢？

```jsx
var Dropdown = antd.Dropdown;
var Menu = Dropdown.Menu;
var Trigger = Dropdown.Trigger;

<Dropdown>
  <Trigger>
    <button>按钮</button>
  </Trigger>
  <Menu>
    <Menu.Item disabled>disabled</Menu.Item>
    <Menu.Item key="1">第一个菜单项</Menu.Item>
    <Menu.Divider/>
    <Menu.Item key="2">第二个菜单项</Menu.Item>
  </Menu>
</Dropdown>
```


Great to remove this huge mess with line numbers !


this is my first project


Surely we can find something more efficient than string comparisons D:


Perhaps itemStack.info is BulletItemThingy


Ah thanks.


What is this number based on ?


continue statements (any ::labels::) are local scoped for current loop. You can try name them as ::continue::


Why reassign constant to a field  ?


Hey, the best way is order: "viewCount" because will show the best videos!


Don't need a method, could just be a variable set


I liked previous approach more than this. Now even there were any exception during obtain the string from standard input, user will get 'please input a number'.
```rust
let text:Option<String, Err> = reader.read_line().ok();
```
Actually your variant works good for std input, so it doesn't really matter. Just want to mention, code should react on each error separately to provide better user experience (don't make user feel himself guilty while the problem is in code :) ).


I think, this should be moved to tests section. What are the options for testing rust applications?


I don't think this is a proper wording. How about 'give the `namespace` property the same value for all of them`


推荐组件组合，不是一个组件完成所有事情（虽然是封装下）


That's why it's in the loop o.O


I don't know how to force commands.


Oh yeah, that'll work


Why does Name need one?


`info.d` finde empfinde ich als nicht-sprechend und generisch. wie wäre es mit `banner.d`?


This will lead to failure of analysis if I am not mistaking, shouldn't this feature actually prevent this ? (ticket is absolutely unclear about this by the way : http://jira.sonarsource.com/browse/SONARJAVA-1063 only describes the technical issue, not the end user issue).


Test


So start with 0 bullets?


Menu 在 antd 单独里可能没用。


For me this makes it less clear.
 Also, 'namespaces' was intended because technically we're talking about multiple namespaces (network, IPC etc).
This is exactly why I would prefer another name for this feature.


I mean - scoped just like local variables. So no name clashes


不然还要单独开一个 Menu 模块来讲解它的使用。


:+1: good idea! I’ll be happy to switch as soon as it’s there :)


It belongs specifically not in the loop is what I meant.


thinking more, it will probably make sense to base my branch on top of this.


Why are you typing in Dutch?


Make a reload function in this fine and have the command call it.


Better replace Tuple with [Result enum](http://doc.rust-lang.org/1.0.0-alpha/std/result/index.html) – function result handling would be clearer.


So it can load the ItemInfo.


Right.


Makes it more flexible.

(But on some embedded devices with very restricted "heaps" it may cause problems to allocate "large blocks" (even more problems, when they have different sizes). So may be a compiler switch ...)


makes clear, there is no param


Nice :)


See comment above.
```rust
if number.is_ok {
  …
```


Yeah!


It's getting replaced with simple Boolean checks / assignments in a later commit, because they can all happen on the same thread.


Witaj kukubaczku mam mały problem z pańską stroną internetową . Mógłbys mi powiedziec jakie są standardowe passy do konta od razu po zainstalowaniu paczki . Bo nie moge dostać się no panelu , instalacja serwera od podstaw również nie działa ponieważ pisze że serwer nie może się połączyć z bazą a gdy baze podepne nie moge instalować serwera bo jest edytowany config . W którym pliku i gdzie moge znaleźć login oraz hasło do panelu oraz jakie jest standardowo hasło . korzystam z najnowszej wersji :D proszę o szybką odpowiedź...


Why don't you just transferred this repo instead of making a copy ?


Me too! :v No Problem Bro! @akhilnarang


Yes, interface dispatch is considerably slower than class dispatch.  Though we could benchmark to verify - HotSpot might make the ugly stuff go away.


There is still the import order check missing...


Hey, you should not add the content of the API documentation to the Git repository :)


Windows is super wonky. i think you should still include that stuff removed in the previous commit for finding the origin and the branch name because that was useful code.

If you want me to review anything i can do it easily, i'm pretty much exclusively on a Windows machine throughout the day.


Why do you need to `continue`?


Also, i put in a request with the GHfW team to support just sending a project directory as an arg, but i haven't heard from them in a while, i could open another issue here and try to bring it up again.


Cool.

I am thinking on making a shot on that equalsfactory. Wanted to discuss it but can't PM on GitHub.


since this is a 'progress' reporting messages, suffix with " ..."


Yeah, good point. I think that makes a whole lot more sense as well as making this code more generic/repeatable when config changes in memory.


`os.path.join()`


if we add `-v` here, users will know what files are copied over... dunno if it is worth, but something I would like to see when I'm including my own httpd config.


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

VSO State: Done


are you sure this works as you want?

$ ls *.html
index-commercial.html  index-community.html
$ [ -f ./*.html ] && echo "foo"
bash: [: ./index-commercial.html: binary operator expected


If I understand correctly ```if not upsert and ...``` would be more readable.


Tested on:
GB Quad Plus
Golden Interstar XPEED LX2
Golden Interstar XPEED LX3
Miraclebox Premium Twin
TM-Nano-3T COMBO
Vu+ Duo
Vu+ Solo
Vu+ Solo2
Vu+ Solo SE
Vu+ Ultimo


should the new function take into account "metalink" when specified?


slide-up 是默认值么


maybe call it  ```divides```, because you give back a bool and not an int


Fine, bye Hobbesian statement.


I prefer that tests will use assertRaisesRegexp in order to ensure that you got the operational error you expected.


bad style:
* don't use `"` here. use `'`
* don't add space after function name `mlog.debug()`, not `mlog.debug ()`


ha! :)


Sorry to be a pain but you might as well use the link to the issue for more context:
https://github.com/scikit-learn/scikit-learn/issues/4072


Turns out this is a namespacing thing in Python 2, though it sounds like it just works in Python 3: https://github.com/sphinx-doc/sphinx/issues/1034#issuecomment-68581974


@mitya57 Yes, search_language_stemming_code will be rendered at next line.


these methods should not be in builder - maybe create simple Predicate object containing them?


Closes #15.


This code searches player.Items twice for the same instance. First to grab an instance, then to grab it's index.


else player.network.sendmessage("nonono")


Sorry about that.  This is now fixed via @burgerdev's PR (#189)


Rewrite the element assignments using the following method:
```scala
def retrieve(i: Int) = if (remaining <= 0) {
    String21(data, i * 21, 0)
  } else if (remaining <= 21) {
    val str = String21(data, i * 21, remaining.toByte)
    remaining = 0
    str
  } else {
    val str = String21(data, i * 21, 21)
    remaining -= 21
    str
  }
```


I'm thinking the self definition is the only way of defining such a type class, could need some hints on that!


Merged will merge on a daily basis . .


Why not? How would you do? I need to save the old APIs somewhere at some
point...

2015-06-08 14:03 GMT+02:00 Julien Richard-Foy <notifications@github.com>:

> Hey, you should not add the content of the API documentation to the Git
> repository :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pauldijou/jwt-scala/commit/078c1838ec3220406a290b8bf9890d8b5eed5c82#commitcomment-11565629>
> .
>


Why the mixed_underscore-hyphen name?


Sure, I'll fix all of them :)


change to final DBHelper helper = DBHelper.getInstance(getApplicationContext());


fff


Mathod corrected!
I'll give some hints to resolve the exerciSes, sure.


yosa


While this fixes it for now, I would prefer seeing a 404.html or a 404.md on the file system. Configurred cloudfront/s3 to pick one up for my own blog: http://blog.wyrihaximus.net/akjwhg%20foiqu2gh34o%20iq2g2o4iyg%2023i (I admit it needs work)


> The only one I am unsure about is the cont in caml_continue, which invokes use_continuation(cont). Can promotion invoke a major collection? In that case, we might need cont to considered as a root?

Yeah, I'm not sure either. I guess we should either dig through the code or ask Stephen.

> Submitted a PR reverting the changes to CAMLlocals #10. Feel free to merge the pull request.

Thanks.


this one is right


@omab
There is issue here.
If user entry have empty field such as 'email' - `getattr(user, name, None)` give as `u''`. And `current_value is None` will always return False. So we never can change empty user fields.

P.S. A have tested this with twitter auth where twitted not return email and psa create user with username, first/last name and without email.


Thanks for fixing this up, @fishcakez .


But for OVERRIDE_SRCDIR this won't be called.
Is there an alternative?


Bench it yourself. delete cache dir, setup huge game and start local game.
mineshit proto can transfer ~10-20 mbit on local connection.
enet ~300-500mbit.


If you're calling terminal_failures before the parse has completed, you almost certainly shouldn't be. It's just not a meaningful thing to do. If not, I don't understand how further terminal parse failures can get added to the array. Please explain?


Where all those `int` => `number` missed from AT -> TS conversion?


Everything looks good I just have two points about the location the params:

1. DagParams looks more like ClusterContext to me, because it is not shared between clusters (since Karamel supports launching more than one cluster) and the usage is not only by DAG. Therefore I would say it is log-term cluster-runtime params that is better to be directly or indirectly inside the Context.
2. Cluster definition model (model behind the yaml) is the logical definition which is being serialized and stored in the filesystem as well. Hence I would avoid including private infos like password and credentials and will directly store it in the above mentioned context.


IHM et manipulations basiques


Try to keep it flat, so:

RUN apt-get install git -y && mkdir /repos

This will result in a smaller image footprint (less images).


The `ListDiffer` still needs a lot of work, but at least the algorithm works. The rest is more about making it easy to use and figuring out how to implement merging. :sweat_smile:

Regarding your shot at the EqualsFactory, feel free to open a discussion ticket in the issue tracker. Or we could open a HipChat or Slack channel, if you'd prefer that.


I think I got the reason, there are two kinds of neuron in my case, excitatory and inhibitory.
so I wrote like this.
```python
from brian2 import *
G = NeuronGroup(1, '''dv/dt = (-v)/(10*ms) : 1''')
F = NeuronGroup(1, '''dv/dt = (-v)/(10*ms) : 1''')
E = NeuronGroup(1, '''dv/dt = (-v)/(10*ms) : 1''')

S1 = Synapses(E, G, 'w : 1', pre='v += w', connect=True)
S2 = Synapses(F, G, 'w : 1', pre='v -= w', connect=True)
run(0*ms)
```
where F are some inhibitory neurons and E are some excitatory neurons.


Look at my second point in the comments.


>>Last blocking issue for the release now is the issue @AXDOOMER brought up in #520.
>Well, then...

Sorry for being so short.

What I meant to say was that, with this issue out of the way, I think it makes much more sense to discuss topics such as hardware scaling and video recording with the release being released. ;)


testing


again, flatten it, in this case a -p option might be sufficient:

RUN mkdir -p /root/.config/NuGet


Oh right. (－‸ლ)
The first Saturday was picked because it (a) falls in the first two weeks of August and (b) is during the weekend meaning summer workers can also play and (c) Saturdays are good for each time zone -- it is late Friday in America (excellent time) and midday Sunday in the Far East.

Finally, why the first weekend instead of the second weekend of August? The first weekend can fall uncomfortably close to the 15th of August (the assumption of Mary), where Christians (esp. the Orthodox denomination) may travel far for various religious events or to meet family. So I saw the first week(end) of August most fitting for a chess extravaganza.


Maybe not related to the issue, but could something like this help in the future?
http://forum.kodi.tv/showthread.php?tid=228877

I just ask because of the parsing for YouTube-URIs. Maybe this could simplify the support for additional addons in the future without transforming the URL into a call suitable for the plugin (plugin://....).


Can't we use the artifacts instead of doing a build in the image? Even so, flatten it with &&


吱吱


reuse context was a flag for the old OpenGL rendering engine. It was used by the map editor to indicate that the game should render to an existing OpenGL context created by the map editor rather than creating it's own.

Either a message explaining that and saying not implemented, or removing it entirely is probably better than just saying not used.


for me it's working ok..have you done any modification to code or
config.json file.

On Mon, Jun 8, 2015 at 2:17 AM, Wiebe Nieuwenhuis <notifications@github.com>
wrote:

> Tested it but i got some problems.
>
> First i get this error: error [pimatic]: Could not initialize the plugin
> "mysensors": configDef must be an object
>
> and my battery sensor doesn't work anymore.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DheerajKhajuria/pimatic-mysensors/commit/4ea747fdb9c079ffced52ebdb9d5443ae7bcd857#commitcomment-11558615>
> .
>


`job_id` is unicode not a digit; currently fails here; %s should be fine.


Also add a:

RUN apt-get clean && rm -rf /var/lib/apt/lists/* /tmp/*

to clean the image a bit


tag as workaround in commit message.


i need to generate auto mail by clicking on the button in the deshboard and send mail to Advertisers. Can you help me on this.


Welcome, Translation Droid :+1:


@udalov
Please take a look at this commit


This must be brought back!!


Dear @kmeinster, first approach is to get it to work, after that i'll clean it up.


Dont think you intended this


please indent


blablabla


`super+click` is already used by default on OSX for making multiple selections.


Ah... I'm very sorry, I thought we had fixed that before merging... The test is currently incorrect when you use `-=`. The warning should disappear if you use:
```Python
S2 = Synapses(F, G, 'w : 1', pre='v += -w', connect=True)
```
or alternatively `v += w` with negative weights `w`. We'll definitely fix this, this is very confusing.

It's not the point here, but note that for efficiency reasons it is better to have less `NeuronGroup` and `Synapses` objects. You can model excitatory and inhibitory neurons/synapses with a single `NeuronGroup`/`Synapses` object and use subgroups for the connections/parameter settings.


Alleluja!


@korhov the ``readOnly`` flag in the ORM mapping does not mean that the entity cannot be removed (it can) but that it cannot be updated (and so does not need to be checked for changes). It can also be inserted.
A much better name for that feature would have been ``immutable`` but it is unfortunately too late for such change.

thus, you are commenting on a totally unrelated commit (this commit is just about fixing the indentation, not about changing any behavior of the code). Requesting changes should be done by opening an issue (not needed for this change as it would be rejected as wrong)


nice


why the <br> ?


@Imperadeiro98 Please be sure to remove the hard copy text from the jade file when localizing strings.  Thanks!


I know that this is how `boost/assert.hpp` defined these, but wouldn't it be more correct if they are defined as `inline` functions?


why the `<br>` ?


@debonair - Careful with adding `@JsonIgnore` to this - for the POST endpoint, this could be ignored, since it's auto-generated, but for the PUT endpoint, this could be used as the identifying field of which entry to update, and should not be ignored.


On Unity, change the name `Permissions` to `Login Permissions`


In order to bake this recipe, first define the variable DISTRO = "poky-qylp" inside conf/local.conf


Of course, you're right.
That's why I isolated those changes in a separated commit. It's a good practice I'm used to not mixing format and code changes, but improving existing code starting with a cleanup commit following some very-minimalist formatting rules (I deactivated all my IDE custom rules). I know the Jenkins policy is fully free per-plugin on this aspect, which does not forbid to sometimes make the files easier to read.
It is very hard to work with source code mixing tabs and spaces, not being normalized at all, even using VI or EMACS to edit the files.

Anyway, I have no issue with removing that commit from the pull-request, just tell me :) Sorry for the annoyance.


Ok, sorry for the trouble.


I guess i will just detabify the file instead and re-do the change. Looking at it now, i want to get rid of the tabs.


related to #593


Just remove the debug print altogether?


This will definitely have saved me a few troubleshooting sessions, but on the other hand might lead to rebind errors down the line where the new entity version is not compatible with the old one. Might be better if behind an off by default feature, could even turn off the one used presently?  This way entity upgrades will always be explicit, with operator knowledge.
In any case a warn log would be helpful for subsequent troubleshooting.

If we end up loading from another catalog item, then the entity's `catalogItemId` should be updated as well. This will make sure that we don't bounce between versions too much.
  * If the original catalog item re-appears at some point then we will downgrade the entity which could be fatal (for the next rebind)
  * If the original catalog item is not present then on each rebind there will be an upgrade to the latest version, also not ideal


Oh well, I just made the (tabified) pull request.  But, as you say, the change is really minor, so feel free to discard it and just do it yourself…


/*comentario*/


Correct; not safe for it to be `*` though because we're assuming we know what the API is. `>= 3.0.0 < 4.0.0` might be OK.


This can return 0,0. Isn't that a problem? Surely we should return 0,0, false in that case, no?

(Also makes the len check superfluous).


Which database is this that is deleted ?
Is it just the admin/password and maybe a few counters ?
If I switch microsd card and run ./setup.sh will that break anything if I want to switch back to the original Evo card ?


ok, I understand, we can revert to 'namespaces' then.

Do you mean the additional sentence makes it less clear? We may have to rephrase it then. Still, I think we need to clarify that namespaces only span instances on the same host, never between hosts, or did I get that wrong?


But it has imports from the mod so we can't put it there that's why I said
util
On 8 Jun 2015 08:47, "Modmuss50" <notifications@github.com> wrote:

> If its an API it should go in the API package
>
> Sent from my iPhone
>
> > On 8 Jun 2015, at 8:42 am, Gigabit101 <notifications@github.com> wrote:
> >
> > Just put it in util
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TechReborn/TechReborn/commit/601812a10c5b80f906dc61ee2f4ffeaad0b68e0c#commitcomment-11561728>
> .
>


Yes. They will grow over time. Here's the guardian one

https://gist.githubusercontent.com/commuterjoy/0911d6c9215c32325547/raw/730fdb5fffaf90c8e7e95840a00d0935c9299120/gistfile1.txt


Yes, I think "give the namespace property the same value for all of them" is better.


Hahahah "wall-hugger"


In the JIRA you created to track the fix of the Swagger doc issue, can you list the classes where you deleted annotations, like this one? So that we try not to miss some pieces which were previously documented?


should be $TargetConfig...


I considered testing for `self` / `global`, but this seemed cleaner / more generic.

cheers, and thanks for a brilliant lib.


Yes you are right, Sorry about that. I'll merge.


good point. moved to separate class


Okay, I'm just curious about the warning and thanks for your advice :)


test slack integration


Weird formatting


Actually I wanted to bitch about a competing library which is probably not good to make public. :)

Both HipChat and Slack channel is new to me, can pick either.


config that


i think we should clear field when data contains key with null value


1: M111 unrecognized component name: docs


I don't like the word `intra-machine`.


24:1: E402 module level import not at top of file
25:1: F401 're' imported but unused
25:1: E402 module level import not at top of file
27:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION, CFG_BIBFORMAT_TEMPLATES_PATH
27:1: E402 module level import not at top of file
28:6: E126 continuation line over-indented for hanging indent
30:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter should be before from invenio.modules.formatter.config
30:1: E402 module level import not at top of file
57:52: E231 missing whitespace after ':'
58:49: E231 missing whitespace after ':'
59:48: E231 missing whitespace after ':'
65:43: E261 at least two spaces before inline comment
65:44: E262 inline comment should start with '# '
70:52: E231 missing whitespace after ':'
71:49: E231 missing whitespace after ':'
72:48: E231 missing whitespace after ':'
76:5: E303 too many blank lines (3)
80:1: E302 expected 2 blank lines, found 1
80: D401 First line should be in imperative mood ('Return', not 'Returns')
80: D205 1 blank line required between summary line and description (found 0)
109: D401 First line should be in imperative mood ('Return', not 'Returns')
109: D400 First line should end with a period (not 't')
141:5: F841 local variable 'tags' is assigned to but never used
142:53: E261 at least two spaces before inline comment
142:54: E262 inline comment should start with '# '
145:79: E261 at least two spaces before inline comment
145:80: E262 inline comment should start with '# '
147:17: E261 at least two spaces before inline comment
147:18: E262 inline comment should start with '# '
148:52: E261 at least two spaces before inline comment
148:53: E262 inline comment should start with '# '
152:58: E231 missing whitespace after ':'
153:62: E231 missing whitespace after ':'
154:13: B901 blind except: statement
164:1: E302 expected 2 blank lines, found 1
191:41: E231 missing whitespace after ':'
192:45: E231 missing whitespace after ':'
193:45: E231 missing whitespace after ':'
196:5: E303 too many blank lines (2)


45:1: I201 Missing newline before sections or imports.
47:1: I100 Imports statements are in the wrong order. import invenio.legacy.bibformat.dblayer should be before from werkzeug.utils
48:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before import invenio.legacy.bibformat.dblayer
51:1: F401 'CFG_BIBFORMAT_CACHED_FORMATS' imported but unused
51:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused
67:1: F401 'CFG_BIBFORMAT_OUTPUTS_PATH' imported but unused
67:1: F401 'CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION' imported but unused
67:1: F401 'CFG_BIBFORMAT_TEMPLATES_DIR' imported but unused
67:1: F401 'CFG_BIBFORMAT_FORMAT_OUTPUT_EXTENSION' imported but unused
77:27: E261 at least two spaces before inline comment
78:28: E114 indentation is not a multiple of four (comment)
78:28: E116 unexpected indentation (comment)
79:28: E114 indentation is not a multiple of four (comment)
79:28: E116 unexpected indentation (comment)
80:28: E114 indentation is not a multiple of four (comment)
80:28: E116 unexpected indentation (comment)
83:28: E114 indentation is not a multiple of four (comment)
83:28: E116 unexpected indentation (comment)
97:33: E225 missing whitespace around operator
182:1: E731 do not assign a lambda expression, use a def
185: D203 1 blank line required before class docstring (found 0)
189: D401 First line should be in imperative mood ('Return', not 'Returns')
189: D400 First line should end with a period (not 's')
195:60: F821 undefined name 'types'
203: D401 First line should be in imperative mood ('Format', not 'Formats')
203: D400 First line should end with a period (not 'f')
203: D205 1 blank line required between summary line and description (found 0)
250:45: W503 line break before binary operator
250:45: E127 continuation line over-indented for visual indent
266:8: W503 line break before binary operator
393:27: E225 missing whitespace around operator
396:5: E303 too many blank lines (2)
414:82: E225 missing whitespace around operator
435: D103 Missing docstring in public function
451: D401 First line should be in imperative mood ('Return', not 'Returns')
451: D202 No blank lines allowed after function docstring (found 1)
451: D400 First line should end with a period (not 'g')
451: D205 1 blank line required between summary line and description (found 0)
472:64: E261 at least two spaces before inline comment
472:65: E262 inline comment should start with '# '
479:38: E261 at least two spaces before inline comment
479:39: E262 inline comment should start with '# '
480:48: E261 at least two spaces before inline comment
480:49: E262 inline comment should start with '# '
483:24: E127 continuation line over-indented for visual indent
492: D103 Missing docstring in public function
495: D400 First line should end with a period (not 's')
495: D200 One-line docstring should fit on one line with quotes (found 3)
508: D210 No whitespaces allowed surrounding docstring text
508: D400 First line should end with a period (not 'a')
508: D205 1 blank line required between summary line and description (found 0)
593: D401 First line should be in imperative mood ('Filter', not 'Filters')
603: D401 First line should be in imperative mood ('Searche', not 'Searches')
603: D400 First line should end with a period (not 's')
603: D205 1 blank line required between summary line and description (found 0)
629:9: E303 too many blank lines (2)
643:5: E303 too many blank lines (2)
647: D401 First line should be in imperative mood ('Return', not 'Returns')
687:28: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
701:1: E302 expected 2 blank lines, found 1
701: D203 1 blank line required before class docstring (found 0)
701: D204 1 blank line required after class docstring (found 0)
701: D400 First line should end with a period (not 'n')
701: D205 1 blank line required between summary line and description (found 0)
732:15: E261 at least two spaces before inline comment
734: D401 First line should be in imperative mood ('Create', not 'Creates')
766:31: E261 at least two spaces before inline comment
791: D401 First line should be in imperative mood ('Return', not 'Returns')
791: D400 First line should end with a period (not 'e')
806: D401 First line should be in imperative mood ('Return', not 'Returns')
806: D400 First line should end with a period (not 'd')
829: D401 First line should be in imperative mood ('Return', not 'Returns')
829: D400 First line should end with a period (not 'e')
829: D205 1 blank line required between summary line and description (found 0)
867: D401 First line should be in imperative mood ('Return', not 'Returns')
867: D202 No blank lines allowed after function docstring (found 1)
976:36: E127 continuation line over-indented for visual indent
979: D401 First line should be in imperative mood ('Return', not 'Returns')
1036:33: E127 continuation line over-indented for visual indent
1047:60: E251 unexpected spaces around keyword / parameter equals
1049:54: E251 unexpected spaces around keyword / parameter equals
1059:37: E127 continuation line over-indented for visual indent
1070:64: E251 unexpected spaces around keyword / parameter equals
1072:58: E251 unexpected spaces around keyword / parameter equals


4: L102 copyright year is outdated, expected 2015 but got 2014
32:1: F401 'os' imported but unused
33:1: F401 're' imported but unused
34:1: F401 'shutil' imported but unused
36:1: F401 'iteritems' imported but unused
38:1: I100 Imports statements are in the wrong order. from invenio.ext.script should be before from six


1: L101 copyright is missing


27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. import sys should be before import pkg_resources
28:1: I201 Missing newline before sections or imports.
28:1: E402 module level import not at top of file
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: I101 Imported names are in the wrong order. Should be InvenioTestCase, make_test_suite, run_test_suite
33:1: E402 module level import not at top of file
34:1: I101 Imported names are in the wrong order. Should be ImportPathRegistry, ModuleAutoDiscoveryRegistry, PkgResourcesDirDiscoveryRegistry, RegistryProxy
34:1: I100 Imports statements are in the wrong order. from flask_registry should be before from invenio.testsuite
34:1: I201 Missing newline before sections or imports.
34:1: F401 'PkgResourcesDirDiscoveryRegistry' imported but unused
34:1: E402 module level import not at top of file
54:1: E731 do not assign a lambda expression, use a def
105:13: F841 local variable 'unknown_output' is assigned to but never used
161:5: E303 too many blank lines (2)
216:80: E261 at least two spaces before inline comment
216:81: E262 inline comment should start with '# '
234:80: E261 at least two spaces before inline comment
234:81: E262 inline comment should start with '# '
253:1: E302 expected 2 blank lines, found 1
349:60: E203 whitespace before ','
350:55: E203 whitespace before ','
350:60: E203 whitespace before ','
351:55: E203 whitespace before ','
351:60: E203 whitespace before ','
352:55: E203 whitespace before ','
352:60: E203 whitespace before ','
353:55: E203 whitespace before ','
353:60: E203 whitespace before ','
354:55: E203 whitespace before ','
354:60: E203 whitespace before ','
355:55: E203 whitespace before ','
355:60: E203 whitespace before ','
356:55: E203 whitespace before ','
356:60: E203 whitespace before ','
357:60: E203 whitespace before ','
360:55: E203 whitespace before ','
360:60: E203 whitespace before ','
361:55: E203 whitespace before ','
361:60: E203 whitespace before ','
362:55: E203 whitespace before ','
362:60: E203 whitespace before ','
363:55: E203 whitespace before ','
363:60: E203 whitespace before ','
364:55: E203 whitespace before ','
364:60: E203 whitespace before ','
365:55: E203 whitespace before ','
365:60: E203 whitespace before ','
366:55: E203 whitespace before ','
366:60: E203 whitespace before ','
367:55: E203 whitespace before ','
367:60: E203 whitespace before ','
368:55: E203 whitespace before ','
368:60: E203 whitespace before ','
369:55: E203 whitespace before ','
369:60: E203 whitespace before ','
370:55: E203 whitespace before ','
371:55: E203 whitespace before ','
371:60: E203 whitespace before ','
372:55: E203 whitespace before ','
372:60: E203 whitespace before ','
373:55: E203 whitespace before ','
373:60: E203 whitespace before ','
375:55: E203 whitespace before ','
375:60: E203 whitespace before ','
376:55: E203 whitespace before ','
376:60: E203 whitespace before ','
382:1: E302 expected 2 blank lines, found 1
453:9: E303 too many blank lines (2)
664:1: E303 too many blank lines (4)


24:1: F401 're' imported but unused


26:1: F401 'warnings' imported but unused
28:1: F401 'cfg' imported but unused


Will this work?


What error? How come I never had this? Im using v 14


TeamCity Jarvis Framework :: CI [Build 58](http://build.prxm.it:8811/viewLog.html?buildId=2931&buildTypeId=CqrsFramework_Ci) is now running


Possible confusion with HTML's `tr` "table row"


There's a warning saying, marking the "collect" as incorrect (it's red). But in fact IDEA doesn't understand that code correctly, and it compiles...
It's not very important, the expanded lambda does the same


Thanks for your reply :-)
No issues, yes it's more curiosity around the implementation..

In general I'm looking at why pretty much all C# statsd clients are written in a way that is essentially synchronous, whereas implementations in other languages offer a buffered client. Sure, UDP is as light as possible on the network, and `SendTo()` may return immediately and send later; but I was still hoping to find at least one implementation that offers a buffered client that sends every few ms or when its buffer is at capacity. I'm aware it may only really affect corner cases, such as sending stats from a tight loop.

In relation to `SendTo()` above, a buffered client may "afford" to retry or at least check that everything was sent.

Realize this may not be the appropriate place for this type of question - happy to take this to stackoverflow.


I just saw that $2 was another use of other scripts.
Desolated the inconvenience.


@jpolitz please make sure you review this -- mainly the changes to secure-loader, runtime-anf, and runtime-util, since I think those bits have changed in the new module system that (I think) happened since this branch forked from master...


This is correct/intended behavior. If you read the documentation, the section I edited is about running the kotlin compiler before the java compiler; Otherwise, kotlin symbols will be unresolved. If you're getting unresolved kotlin symbols in java, that means that `maven-compiler-plugin` is run **before** `kotlin-maven-plugin` (or something else is not configured correctly); This is the default behavior, and this is what the above section is designed to fix.

There are two ways of fixing this:

1. Move the kotlin compiler to an earlier phase. This is the simple solution for simple projects, but ultimately means you're compiling outside of the compile phase, and may cause issues if you're depending on plugins that run in the phase you used instead.

2. Redefine the java compiler to run after the kotlin compiler. This is more verbose (as the above pull request shows), but results in correct behavior as far as maven conventions (both kotlin and java compilers are running in the `compile` phase.

 The configuration in the pull request was used successfully with Maven 3.2.1 when run from the command line, and I recall it working correctly from within IntelliJ as well. I will try to reproduce the bug on my system when I get home today.


Solved with issue #3


@stefano-k: I'm curious - why did you add this modal flag? It's the very thing that broke the sensor config dialog (https://github.com/mate-desktop/mate-sensors-applet/issues/23) :)


actually I am modifying inputPoint in the A case but not too confident about it.


TeamCity Jarvis Framework :: CI [Build 58](http://build.prxm.it:8811/viewLog.html?buildId=2931&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 105 Build time: 00:04:14


I'm not sure, but it looks like this [broke the build](https://ci-bahmni.thoughtworks.com/go/tab/build/detail/Bahmni_MRS_master/4015/BuildStage/1/BahmniApps)? I noticed this while trying to get the Docker containers up and running:

````
--2015-06-08 12:52:52--  https://ci-bahmni.thoughtworks.com/go/files/Bahmni_MRS_master/Latest/BuildStage/Latest/BahmniApps/bahmniapps.zip
Resolving ci-bahmni.thoughtworks.com (ci-bahmni.thoughtworks.com)... 221.134.198.44
Connecting to ci-bahmni.thoughtworks.com (ci-bahmni.thoughtworks.com)|221.134.198.44|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-06-08 12:52:54 ERROR 404: Not Found.
````


Did you the difference between Greenice and Greenicefrost accounts? :)

2015-06-05 21:12 GMT+03:00 Mrkwtkr <notifications@github.com>:

> Update zh_CN.lang
>
> Updated localization. Mainly corrected numerous mistakes leaved by
> Greenice. (Angry
> I have surveyed many players and translators, they said that your
> translation (@Greenice ) seems like machine translation. You‘d better pay
> attention to the  expression. Last but not least, don't add your name in
> the land file. This loacalization was made by serval translator, your
> behavior is just childish.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Mrkwtkr/Botania/commit/9582a2813311499b8eb7cf75b50b20bb05377639


*did you see...

2015-06-05 21:12 GMT+03:00 Mrkwtkr <notifications@github.com>:

> Update zh_CN.lang
>
> Updated localization. Mainly corrected numerous mistakes leaved by
> Greenice. (Angry
> I have surveyed many players and translators, they said that your
> translation (@Greenice ) seems like machine translation. You‘d better pay
> attention to the  expression. Last but not least, don't add your name in
> the land file. This loacalization was made by serval translator, your
> behavior is just childish.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Mrkwtkr/Botania/commit/9582a2813311499b8eb7cf75b50b20bb05377639


Hi

It's quite simple. Consider your test contains several steps: `Given, And, When, Then, And, But`

Currently, when **any** step fails, the whole test fails. However, typically, all steps after (and including) the `Then` step are independent and represent different expectations of the outcome. The steps before this represent the setup and execution, if any of these fail then the outcome cannot be guaranteed.

In order to continue executing steps after the `Then` step simply add the `ContinueOnFailureAfter` attribute to the assembly.

```
[assembly: ContinueOnErrorAfter(StepType.Then)]
```

You can also override it by applying it to the method or class.

Hope this helps


Check this video if you haven't seen too: https://www.youtube.com/watch?v=Dx05x6BwncU

As far as I know 404020 baseband is not compatible when you have installed 404029 (the IMEI storage container might get corrupted). Also the userspace drivers are currently only supporting 404029.


[FIX]: Unable to link the .js-items-count class with the its version on destroy.js.erb


tr is what it's called in json, iirc.
Plus, there are no tables in minecraft chat, nor do I think there ever will be.
I'm in favor of adding a long name (translate) for this tag.


Thanks for the heads up, the changes are good:

- runtime-anf just exports execThunk, which is a handy thing to provide
- secure-loader has an extra try/catch around the eval, which does a better job of catching syntax errors in handwritten JS modules
- runtime-util's makeModuleReturn is a much cleaner way of doing all the `makeObject`...`provide-plus-types`... stuff at the end of each JS module.


hello

you have snapshot last commit fixed?


How about we avoid ad-hominen attacks and stick to the facts. These are the options for lag:

* Only override methods: this is now illegal (R CMD check fails), so it's no longer an option.

* Make it an S4 generic. Fortunately no one is arguing for this :)

* Make it an S3 generic. The problem with this approach is that there are now two S3 lag generics which may have different semantics and will different arguments. The S3 system identifies generics by name (unlike S4 with uses both name and enclosing package name). That means your method for `stats::lag` will now be selected from `dplyr::lag` - this seems dangerous to me.

* Make it a regular function. This is the approach I've taken because I think it is the least-worst solution.


yeah  :+1:


Sorry, will do it next time. I'll take care of this one.


ok


So in what case VG_SIZE is nil?


Please use ExtSourceCSV.class here. You can then use this variable on lines 70, 107 and130.


![screen shot 2015-06-07 at 19 46 42](https://cloud.githubusercontent.com/assets/5517422/8035446/b8906cae-0de8-11e5-9beb-9a754ca02f4a.png)


Hey, guys,  thank your for your sharing! But your project is based on OpenCV-3.0.0, not 2.4.8!
For the three reasons:
1. OpenCV-2.4.x uses smart pointer Ptr<(params)>, rather than makePtr<>(params);
2. OpenCV-2.4.x uses try_gpu, rather than try_cuda, try_ocl;
3. Include files


Hrm, apparently when I merged horizon onto dataflow, I deleted `makeModuleReturn` from runtime-anf because it apparently was never used... I guess now it's coming back in runtime-util?


I don't understand. I don't like to put declarations in loops or the switch statement, the compiler may always reserve the memory exactly once (in loops that run through the delcaration more than 1 time or in if statements that don't run through the declaration at all) it just looks ambiguous to me.


awesome :+1:


Do we actually need to publish any ports? After all, we are running in the host's network namespace.


I see you moved the interval query files to the main script library

There are 2 problems I see:
1. you forgot to add the to git :)
2. I think it will be better managed as part of the original module. Its a better directory structure and it will be easier to pull bug fixes from the parent repository


That's a good point about buffering offering more options but it'd also introduce more moving parts to the client itself I think. We've been using this in production across many high load systems without problem for a few years now, so I think you're right in it solving edge cases that most systems aren't likely to hit. Always appreciate the comments regardless :) Feel free to bring up on StackOverflow - sounds like you'd like a more general conversation on the topic as well.


yup, changed the wording, feel free to suggest a different one, just want to be clear that grouping does not occur between hosts, even if you scale.


Im sorry. :P I meant to call him.


嗯，搞计算机视觉三维方面的，欢迎关注。


No, still have issues #58


To match the lang string style guide please add a fullstop


Experimental: 64DD IPL support


To match the lang strong style guide please capitalise eg Use Search Options


The Django project database is deleted and recreated using default values when you run setup.sh. Currently, the database contains the admin login, saved Bitcoin address and bandwidth limit.

You can switch microSD but it should already be formatted with similar partition as the original microSD: https://forum.bitnodes.io/t/64-gb-storage-card/19/4


Thanks @larshg his commit is correct.


#17 Resolved


Galera uma coisa q me esquecir de cometar foi q sem quere movi algo do themes e o ccs não pegou ai depois concerto isso tbm


"preffer" => "prefer"


It may make sense to rename the view and update doc because this is not a department invitation in a strict sense of meaning. When user accepts the invitation the system will create a new user, new member/account set the user as an owner and add it to a department.


This really should be done better than that. Maybe with a list of devices, or just through a ticky box in the settings.


OK. Thanks for feedback on this.


"effect" => "affect", "processs" => "process"


Please don't report bugs on Github. Report them on the Buildroot bug tracker, or on the Buildroot mailing list. Thanks!


We should also check for presence of META_LV_NAME. If it already exists, don't try to create it.


:+1: for the fast response!


Tests fail and there may be a problem when merging it back to master so I think this require some tiding up.


For this DPU, only parent should have changed because there was already 2.1.0-SNAPSHOT version of DPU that has never been released; So this update to API should be also released as version 2.1.0.


Would be clearer to put this first `<br>` after the header I think


Actually that META_LV_SIZE is vestige of old code. I think we were using it to determine if META_LV is present or not.

-  if [ "$LVNAME" == "$META_LV_NAME" ]; then
-    META_LV_SIZE=$LVSIZE
-  elif [ "$LVNAME" == "[$META_LV_NAME]" ]; then
-    META_LV_SIZE=$LVSIZE
-  elif [ "$LVNAME" == "$DATA_LV_NAME" ]; then
-    DATA_LV_SIZE=$LVSIZE
-  fi

So agreed that META_LV_SIZE should go away. This should instead be replaced for a check to determine whether metalv is already there or not.

Optionally one can check for META_SIZE too.


(-sum * sum / weight) * weight = -(sum *sum)
Мне кажется ты имел ввиду здесь что-то другое.


Any idea why Travis report this to break LDC ?
The offending assert: https://github.com/ldc-developers/phobos/blob/c56a25fde691cae9a3cc125ae7232dc1dbcd7ced/std/internal/math/biguintcore.d#L99


This doesn't exist. The plugin is still under development, and only exists on my github account at this time.


I read both the wamp 1.0 and CURIE w3 spec and I agree that if no prefix has been defined then it shouldn't convert it. I will make the change and submit a pull request.


Created http://jira.xwiki.org/browse/XWIKI-12163


Fine, take 'you' out of my prior comments and substitute 'dplyr'.

There is an important option here that is being ignored:

 - dplyr shouldn't mask stats::lag at all, avoiding intentionally breaking other code.

There is nothing in NEWS and nothing obvious in the commit log for R-Devel that talks about not allowing overriding of methods.  It is clear that masking the generic, without making dplyr's function a generic, violates the spirit if not the letter of the law as defined by CRAN.

In any event, I believe that it is possible to not break other code by:

- masking the generic for *lag*()
- handle whatever cases/classes *dplyr* wishes to handle in this generic
- properly dispatch on other classes

There is a well-understood methodology to masking generics.  It is 'blessed' by Writing R Extensions.  Different function signatures are allowed, as long as the arguments up to the dots are the same.  Just do that.


Oops. It should also be added to line 650 and 655.


True, this is not necessary. Fixed.


15 :)


Ah, good catch. Fixed (and fixed in unit tests)


@RoySegall
This breaks things. Why did you put it back?


no need for this


This one works good for me. What kind of things it break for you?


Haha, sorry I thought it was funny, but it was inconsistent with the other log messages, so I had to either cut this one or think up jokes for all of the other ones.


also change the console.log


Is this suppose to be Pupils ?


I wondered about that myself.  Let me familiarize myself with the metalink stuff and get back to you .


I didn't copy them - It's just a symlink to the original files. if we pull
a new version it will automatically update.
The issue was that the grunt compilation failed on these files (it didn't
include them for some reason).

I can put some time into this to see if I can solve this without the
symlinks (which are problematic on windows, I know).

On Mon, Jun 8, 2015 at 4:16 PM Yonatan Naor <notifications@github.com>
wrote:

> I see you moved the interval query files to the main script library
>
> There are 2 problems I see:
> 1. you forgot to add the to git :)
> 2. I think it will be better managed as part of the original module. Its a
> better directory structure and it will be easier to pull bug fixes from the
> parent repository
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenBudget/open-budget-frontend/commit/a548f6190eea5f7bc3a974a6a2ecc395f84575b5#commitcomment-11566801>
> .
>


Done in a9f8a7b


Disclaimer: I am a Ruby newbie.

Why is docker_image defined as a symbol ? If I understand correctly, this avoids testing different images in a single serverspec run because the first definition of :docker_image is the only one that is actually stored. Correct ?

Is it a strange idea to try to test multiple docker images in a single serverspec run ?


> One thing that might be possible is to hook draw_if_interactive up to the parameter change call backs that
> live in the Artist class and hope that the draw_idle methods on all of the backends work as promised.

Yes that will work.
Actually I went through the same exercise many years ago or so when I started writing the MacOSX backend; my main motivation in writing this backend was to get the event loop right. From looking at the tkagg backend, the issues are exactly the same there.
Using tkagg as an example:
Basically, you don't need draw_if_interactive(); it should be sufficient to call canvas.draw_idle() instead. It doesn't hurt if it's called more than once because of the self._idle flag in canvas.draw_idle.
This effectively schedules canvas.draw as an idle function with Tkinter. This means that canvas.draw gets called when Python has nothing else to do (to be precise, via PyOS_InputHook inside Tkinter just before Python starts waiting for new user input on stdin). So the drawing operation will be run only once; there will not be any useless intermediate draw operations.

I don't think that we need IPython hooks, the sys.displayhook, the stale flags, or draw_if_interactive at all for this; also this will be transparent to third-party libraries.

Can we explore this option? Unfortunately I was not aware of these recent changes to matplotlib; can we revert these commits so we are all on the same page and can start from a blank slate?


I think we need this as work around for some phones that don't scan well.


Jeszcze ten panel nie gotowy :P
Specjalnie nie dałem zrzutu bazy danych, żeby nie dało się zainstalować :P

Poczekaj aż skończę


just clear the findTimer here


Fails here


Really it is not necessary...


It breaks uploads, it breaks access for site admins. It's a huge change that I have no idea what the implications are.

Why did you change it in the first place? This whole time that class has been extended off of RestfulEntityBase. What did you solve by changing this? Why did you just do something this big without discussing it first? I have no idea what the ramifications of this change are.


Fails here, too


Latest released version was 2.0.0 in UV_Plugins_v2.0.1, or not? But I agree that description is missing for changes made prior to API update (configurable character set)


THis is it.


Brilliant! Thanks @smblott-github :relaxed:


do not assign to constant


感謝補上～


I believe this will break.
My proposal was to return `nil` from the getter, thereby having **no** title.

The default is (IIRC) to use the `name` property, which an activity does not have....


A versão do PHP que tem no servidor não possui a constante PHP_SESSION_NONE, nem a função session_status. Por isso, ao acessar o sistema vcs verão um erro fatal. Fiquei sem bateria no notebook e não pude corrigir, mas ainda hj farei isto.


Static method in the Pupils class which will handle UDFs of any type. Could be broken out into a utility class.


This will break, see my original comment on https://github.com/Azure7111/openproject/commit/e705df65c3f34deb742f5bdb3c7b61337853f69b


Would you mind fixing the mixed tabs and spaces? Thanks!


nitpick: space between parameters


I guess I was unsure that this was the right move, so I wanted to  put the files here to start discussion without affecting `postcss-log-warning`. Since the new plugin will have a different name, won't I need to keep log-warnings where it is and add a deprecation notice, instead of quietly switching locations?


reportlab n'est pas nécessaire puisqu'il est déjà listé parmi les dépendances d'edx-platform.


wait - isn't there already an else to this clause (see line 159)


Some arbitrary value based on statistics on jdk and internal plugins.

This value is low enough to not stall the analysis yet it allows us to cover over 99% of methods.


Only Americans pronounce 'qd' and 'cutie' the same. Just sayin'.


had written it before I had a catch implemented so I was not sure what would happen if there is no catch and wanted to be "save".


what happened to the 'rounding up'?


I didn't realize that `null / 2` would work but sure enough it does.

I think the comment you gave below would be better as it would move the usage of it within the if block. In which case you could just delete this line as it would be undefined, and only true if defined.


getuid?  you mean uuid_generate, right?


Nisarg,
looks like this commit introduced this bug:
https://sourceforge.net/p/phpmyadmin/bugs/4948/

Typing an EXPLAIN statement in the SQL query box does not show the explanation. For example, EXPLAIN SELECT * FROM departments.


Ah I see it now. But the comment is rather obscure.


This doesn't fix #3101


I'm not sure what you mean by 'grouping does not occur between hosts'.

Note that groups of instances HAVE to be scheduled on the same machine, all other combination do not need to be scheduled on the same machine, but still can be scheduled on the same machine.


latest released version in http://maven.eea.sk/artifactory/public/eu/unifiedviews/plugins/uv-t-filesFindAndReplace/ is 2.1.0


Good call, I think I scraped part of the code from an example somewhere. Consider it changed!


yep, i think when trying to satisfy the compiler I changed it to double at some point.


Is this a reasonable number of queries? Anything that we could do to cut it down?


@pmitros , @lamagnifica  , thank you!
- are these events emitted *only* when the CC menu contains multiple languages?
- do the events emit which language has been chosen? (If not, what do these events measure?)

Also, have they undergone Analytics team review? I would be happy to help with validation on stage/prod.

Thank you!
FYI @shnayder


You need to change `display_name` each time you call `ItemFactory.create`, otherwise it'


Fully working version of the Coin Calculator


lets squash this commit and re-commit, if i merge this in, the git log history will expose our CouchDB admin details online or we have to drop this admin account.


As a minor aesthetic thing, if it's "BigInteger" in title-case, seems like it should be "ComplexRoughnum" and "ComplexRational" to match...


This logic is a bit different from the cmake flow.

Cmake checks for uuid_generate & uuid_create without using the uuid library - it's trying them in libc first.   The check for uuid_generate in libuuid should include "uuid.h".


Oke isch gued ha nume denk will snid in dr lernziel isch...


I'm interested to know the history of this project. It appears to be mentioned here:

"Any value, printed by using Microsoft.FSharp.Text.StructuredFormat.Display.any_to_string with the default layout settings"
http://en.wikibooks.org/wiki/F_Sharp_Programming/Input_and_Output

Was it once part of the standard F# libraries?


1: D100 Missing docstring in public module
74: D400 First line should end with a period (not 'e')
74: D205 1 blank line required between summary line and description (found 0)
98: D400 First line should end with a period (not 's')
98: D205 1 blank line required between summary line and description (found 0)
118:9: E266 too many leading '#' for block comment
144:13: E266 too many leading '#' for block comment
247: D103 Missing docstring in public function


3: L102 copyright year is outdated, expected 2015 but got 2013


4: L102 copyright year is outdated, expected 2015 but got 2014
35:1: I100 Imports statements are in the wrong order. import logging should be before import re
37:1: I100 Imports statements are in the wrong order. import os should be before import sys
38:1: I201 Missing newline before sections or imports.
233: D401 First line should be in imperative mood ('Create', not 'Creates')
233: D209 Multi-line docstring closing quotes should be on a separate line
233: D400 First line should end with a period (not 'd')
233: D205 1 blank line required between summary line and description (found 0)
252: D103 Missing docstring in public function
265:2: B901 blind except: statement


24:1: E402 module level import not at top of file
25:1: F401 're' imported but unused
25:1: E402 module level import not at top of file
27:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION, CFG_BIBFORMAT_TEMPLATES_PATH
27:1: E402 module level import not at top of file
28:6: E126 continuation line over-indented for hanging indent
30:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter should be before from invenio.modules.formatter.config
30:1: E402 module level import not at top of file
57:52: E231 missing whitespace after ':'
58:49: E231 missing whitespace after ':'
59:48: E231 missing whitespace after ':'
65:43: E261 at least two spaces before inline comment
65:44: E262 inline comment should start with '# '
70:52: E231 missing whitespace after ':'
71:49: E231 missing whitespace after ':'
72:48: E231 missing whitespace after ':'
76:5: E303 too many blank lines (3)
80:1: E302 expected 2 blank lines, found 1
80: D401 First line should be in imperative mood ('Return', not 'Returns')
80: D205 1 blank line required between summary line and description (found 0)
109: D401 First line should be in imperative mood ('Return', not 'Returns')
109: D400 First line should end with a period (not 't')
141:5: F841 local variable 'tags' is assigned to but never used
142:53: E261 at least two spaces before inline comment
142:54: E262 inline comment should start with '# '
145:79: E261 at least two spaces before inline comment
145:80: E262 inline comment should start with '# '
147:17: E261 at least two spaces before inline comment
147:18: E262 inline comment should start with '# '
148:52: E261 at least two spaces before inline comment
148:53: E262 inline comment should start with '# '
152:58: E231 missing whitespace after ':'
153:62: E231 missing whitespace after ':'
154:13: B901 blind except: statement
164:1: E302 expected 2 blank lines, found 1
191:41: E231 missing whitespace after ':'
192:45: E231 missing whitespace after ':'
193:45: E231 missing whitespace after ':'
196:5: E303 too many blank lines (2)