Ideally they would access `this.props` and `this.state`, but I don't think there's a way to do it.  We will have to document that you can't use `this.props` and `this.state` directly.  Unfortunately, this is something that happens with mixins.

It's because the only way to hook state changes is with `componentWillUpdate`, which is called before the new `state` and `props` are assigned to `this.state` and `this.props`.  React's design for implementing `observe()` in core involves calling it between `componentWillUpdate` and `render`, after the new state and props are assigned, which is easy for them to do but not something we can do. :(


When considering an unblocked `if` statement (when not using braces for the code block), you should put any statement where execution happens on a tabbed in second line:

```
if ( in_array($tag->name, $excludes) )
	unset($tags[ $key ]);
```

In this way, if there is a syntax error, or procedural error in either the condition or the statement, the PHP error logger will give you an accurate line number for the problem making it easier to find issues.


CBECC (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I was confused because I didn't know you publish 0.7.3.  Now I actually bumped it to 0.7.4 after hearing from you.
Haven't pushed yet.


I already had an admissions template in another branch and pull request. maybe we can compare templates tomorrow to see which one to keep.


maybe the correct version should be 0.5.0 rather than 0.5.26? just askin :)


Of general interest: I had a couple of outstanding reports against that module which I hadn't had a chance to triage, but having done so this evening I've substantially revised the logic, and it appears to work much better with some changes to Squad's logic in 1.0.

https://github.com/toadicus/TweakableEverything/blob/master/TweakableEngineFairings/ModuleTweakableJettison.cs


Congrats and good choice


@esteele That has to be 1.1.b5 which fixes two bugs in the upgrade-step to the new listing-views.


@KradekMFC Thanks for the code location where they buffer messages. I see it now on my own local copy, although it's called this.TxReceipt in mine, but the inner code is the same.

Good to see that they're only reading lengths up to 9999. That means we don't have to do anything special for long messages and that they probably split them on the server-side somehow.

Interesting that you say we have to join the ROOMDATA channel to be given room counts. That hasn't been true in my experience. The client CONSTANTLY receives "FCTYPE_SESSIONSTATE" with an argument specifying the modelid/broadcaterid, and a payload of {rc:X} where rc stands for "room count" and X is the number of people in that model's room, constantly changing. That's received after login, without ever joining the ROOMDATA thing above. I guess I'll try doing your ROOMDATA thing to see what it does, but it sounds like the same thing to me.

Lastly, regarding METRICS: It's actually more complicated than that. The official code tries to grab mobj.php, but if it takes longer than 8 seconds (from memory) or if there's a connection error, it instead sends a special message to the server, and it responds with a MASSIVE blob of JSON sent as Payload in a regular MFC Message instead.

I am going to be taking a look at all of this, implement buffering in my rewritten version of MFCSocket, and will be trying chat-METRICS at a busy time of day with loads of models online, to see how they deal with super long payloads over 9999 bytes. It's possible they've never thought of that scenario in their official code. ;)


Comment 1


Could we throw here in case `.notes` is a string?


PS: That's a really freaking weirdly written loop they're using there... "for (this.receiveBuffer += e.data; ; ) {"... I understand that they're looping forever until all messages (length + payload) have been parsed, but that's a really confusing way of writing it. They should have done "this.receiveBuffer += e.data; while (true) { /* process message chunks */ }" :-P


That can't happen because of the cleanup we do at https://github.com/zotero/zotero/blob/sjk/translation-save/chrome/content/zotero/xpcom/translation/translate.js#L109, although it's arguable we should throw there instead of converting to an array.


Not sure if this should be here or in the repos.


Why did you make this change? I should be able to have a user token with a default project id.


Fixed.


@hsbt , I referred [oxforddictionariesl](http://www.oxforddictionaries.com/definition/english/re-enter?q=reenter) before change.

if you saw reenter [here](http://www.merriam-webster.com/dictionary/reenter), please see Facebook comments on it.


:tada: :cake:


Other than the comments I made, it seems good. Please make sure to test the migrations where you are deleting data on a production backup and make extra sure that it won't delete anything that it shouldn't.


Thanks @brunopulis


Looks good.


the good codename is SM-N915F  (tblte)


OK.


Oh, right. That's ok then.


In theory, this could be null or undefined, which would throw.


ummm how does this work exactly ?


Well, not null, since that would throw above where we try to access `.tags`


Let's use SinglePaneActivity as the container of fragment


:tada:


@julianlam  `margin-left` should be applied to lists, not list items ;)


Your welcome. I should probably have given you kudos for helping me realize I could make a [very similar change](https://github.com/zostay/p6-HTTP-Headers/commit/75fadfe24638b62c021fc4f70c87d18c29de00d8#diff-9f9c4e1c33836102f821ecc50848cc9fL24) back in HTTP::Headers.


`returnValue` is only `false` if error handler is going to be called, right?


@Temptin Take the code with a grain of salt; the mobile source is pushed through a compressor or uglifier so that may be modifying the loop for some reason. For example you can see !1 for false to save a couple chars all over the place.

With respect to room counts, it possible they've changed their method and left the previous one in place. When I originally wrote the code, the room count definitely wasn't part of session state. Session state was already I spammiest msg, it would only be worse with room count.


Done. Thanks.

On Wed, Jun 3, 2015 at 6:40 PM, Nikhil Haas <notifications@github.com>
wrote:

> You should probably set your global git config to properly handle Windows
> line endings
> <https://help.github.com/articles/dealing-with-line-endings/#platform-windows>
> so you're not changing the entire file like this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nsh87/receptormarker/commit/e1e1fbc7d8185e775f2713f5363442cf89981e22#commitcomment-11512828>
> .
>


thanks..


As mentioned above, this is guaranteed to be an array as a property of item, but it's not cleaned up beforehand for tags on notes and attachments, so we should make sure here.


You don't upload it to ACS namespace directly.


And not null


I did intentionally not do this to keep the diff small for now. Will remember to do things like this in the future.


This is available as `Zotero.Libraries.isFilesEditable`, which does basically all of the above.


@toadicus awesome, thanks! Had I but procrastinated one more day.... :P


will this be constant?


I have hear an actual reason why this is bad. The only I've heard so far is "it's blocking", which is not true thanks to [this](https://github.com/MultiMC/MultiMC5/commit/30061fb62f6e3aa9536f43191cf2c84ef7148bfa#diff-cd45d6057bdbb921cdc34e90044f5b16L33).


check for Certificate name null or empty


I am not sure abt the API version


nicely done


The plugin is currently being overhauled and the template is likely out of date. Regardless, the file and line number are just for humans -- I'm 90% sure it doesn't affect translations.


Hello, would like to know why you abandon this bundle that I consider of interest because I am working on a very close topic ... How could we work together to insure to release a state-of-the-art quizBundle (I am currently using the latest Sonata 2.4 package as a target environment for integration) ?


This whole block should ideally be:
```cpp
    if (!settings.bypassProxyForHosts.empty())
        networkAccessManager.setProxyFactory(
            new MyNetworkProxyFactory(proxy, settings.bypassProxyForHosts));
    else
        networkAccessManager.setProxy(proxy);
```


Falando em alterar migration...


Might make sense to declare `QList<QNetworkProxy> originalProxy, noProxy;` and initialize them both in the constructor so no new object needs to be created every time in `MyNetworkProxyFactory::queryProxy`.


Interface of  shap.


if it were me, I'd put his "1000" into a #define somewhere.


What if we give a bonus for our king taking squares from the other king's, king ring, especially where the other king's next move must remove squares that were part of both king rings (and opponent's pawn moves are loosing).


Tá quebrando aqui, isso resolve?

 def up
    add_column :users, :name, :string
 end

def down
  remove_column :users, :name
end


Hi, please remove this line, ``displayName`` is optional param (GUI fallbacks to ``friendlyName`` instead). Thanks.


Given
```json
{
    "key": {}
}
```
then
```c++
auto str = ensureString(root, "key", "");
```
should throw, because it's the wrong type. Saying that ensure* functions should never throw is bad.


Take a look a second time, the code used in ctypes does a check for dll with fname not name (the input of the function) as you have done here.


OK. I'll make another issue about the code style.


I don't know what the right solution is here. Usually I'd say don't bump in PRs but you may have to use a new version in a PR in another repo.


because all caps variable names a generally used for constant values acording to PEP 8 https://www.python.org/dev/peps/pep-0008/#id39


It makes things make sense. Without this, the changes are missing their point entirely.

Note that BaseConfigObject had the same QSaveFile patten, but didn't use FS. That's just weird, considering that it's new with this PR.


support aspectJ


that's why I bumped it and that's less confusing when we have a number of versions dependency lining up.

(Say some PRs depends on 0.7.2 and later PRs depends on 0.7.3)


Idk if it really makes sense to return this. Doesn't it have to be reloaded anyway?


@amagdas it might be a good idea to add a few comments to the code just to explain what it's all about.


@KradekMFC Yeah it's definitely part of the session state, and they've got a good reason for it since they've got "sort by viewers" sorting modes etc, and all sorting is done client-side, so the client _always_ needs to know how many people are in each room. They may have used the ROOMDATA method before, but decided that the information is always needed "so why not send sessionstates for it?".

I discovered this when I was going nuts with some debug code printing all incoming message types, and kept getting hundreds of SESSIONSTATE, almost exclusively with "{rc:X}" payloads, and sometimes with a bit longer payload to signify a camera state change or topic change, and then of course occasionally a very long message (full information) for a model coming online. At that time, I had no idea what the "rc" variable was for and wondered why it kept spamming it to me via SESSIONSTATES. Good times... ;-)

Anyway, I suggest having a look into this and seeing if the roomcounts given by SESSIONSTATE match the ones given by ROOMDATA. Do you mind doing that testing when you have time to look at this? No rush...

I'll personally have a look at the length of chat-mode receipt of METRICS when I've implemented the addition of buffering, and will be reporting back. Probably gonna have to wait until peak hour on friday/saturday to have enough models online to cause a problem, hehe.


I don't like ordered PRs, though :-P


As I said, will keep in mind. Should I apply Resource to more parts of the code base too?

That BaseConfigObject doesn't use it is actually weird and will be fixed.


About that *[ItemVariationDetails]* string in the commit message: It was just a test :wink:. The project is very small right now, but I liked what some developers of React Native are doing; they include the name of the component or section of the project that was affected by the commit and then a message briefly explaining the change (See https://github.com/facebook/react-native/commit/43adb7b02c49c3a05c3d4f884c1165303556e372 for an example.)


@KarishmaBagga, yep, I definitely wanted "metrics" )) I will fix spelling.


Maybe it's worth leaving a note here that we're intentionally not waiting for these to resolve.


Will do, now I'm chasing other bug, a bunch of our tests fail when trying to upgrade to latest code from develop branch.
Most probably I will send another PR soon, will add the message then if it's ok.


It's definitely better than big PR that include dozens of things and too hard to reason about things.

I think Google and other companies definitely has this kind of things too.


You beat me to it!  I made similar changes but I like mine better, can you withdraw this commit and then I will do a push?


Hi, thank you for this patch proposal!  I've been hesitant to restore support for Windows, but this just
might be a practical middle ground I can live with :)

Now, before I merge your pull request there are a few things I'd like you to fix for me:

1. Always put #ifdef et al at column 0, start of line
2. Variable declarations should be placed at the start of the section, not spread out in the code. Separate them from the code with an empty line
3. Squash all commits into a single logical commit, then post a new pull request

Thanks!


But it's the answer! :)


The points i brought up previously imo still stand. This might be a useful if this was a program used by a non-programmer. However this is a library, I think clarification of documentation is all that is needed for this. Then developers who create programs using this library will distribute the freetype dll. Most of the time if your not distributing source code to end users (most of the time) this should be fine.


And why is this bad? I'm actually tempted to remove all the ensure stuff because it's never used.


@sureshr-hasvik

What does this column/attribute asNumber mean?

Is it the IP being representa as a number? If so, why not Long instead of String?


delete


I would say no. The preference should really read "Do not import tags from the web" imo.


I bet there are more places where FS can be used. It could also be merged with some of the utils. Remember that I removed stuff from FS before - because the existing code was more correct (think not following symlinks when deleting folders), or because it was never used.


Because a key existing but having the wrong type means the JSON data is malformed, just as if a required key would be missing.

I just quickly looked and found several places in the MultiMC code base that would use ensure* (the optional variant). If you want I might take a look at applying Json:: to more places.


worth pointing out this change.  It made it more difficult than it needed to be to write examples like `pausable` or `repeat`.  Easier to leave the inputs cold and let the example's `apply` method use `publish` to make them hot if necessary.


@sureshr-hasvik

How big can a IPV6 CIDR be? The columns are varchar 255.

Please, see comments above regarding the as_number thingy.


If there are such places, it's an omission. I think I replaced all `ensure*` with `require*` here. I might have missed some. Although that is kinda hard, because the compiler should've caught it.


Hi @sureshr-hasvik

I went through the PR and added a couple of comments concerning the asNumber attribute and the length of columns.

Cheers,
Wilder


How come this has been changed on master? This should probably only be changed on the actual branch rather than master, as a diverging change. WDYT?


Clicking cancel ended the program but not quit. So I rewrote the whole thing. Took the for loop out of the getStudent function to separate the duties, added toLowerCase to the prompt just to get to it a little faster, and removed most of the comments because they didn't really apply anymore.

Next thing to work on: The "No student found, try again!" functionality!


There are similarly named functions from other sources, but no `ensure*` from Json is used.


I meant as in code that's still using the "raw" QJson* functions.


Looks good :)


wellcome


Yeah I got those two I missed on top in the next commit. Wow those are old.


hope me


[Uploading web应用需求规格说明书.doc…]()


Hey, will do.  Thanks for your patience and thorough instructions.  Much appreciated.


@rmuir Wanna me to create the `plugins-contrib` repo with lang, rivers and transport plugins?


Hey Mike, I followed the video best I could. Went back through and read your post to make since out of it, but still having trouble. I might need assistance in the next couple classes.

Sent from my iPhone

> On Jun 3, 2015, at 6:05 PM, Mike Sinkula <notifications@github.com> wrote:
>
> But, it doesn't look like you used your thumbnail featured image in your site?http://maddyswebnation.com/web170/templates/wordpress/blog/
>
> —
> Reply to this email directly or view it on GitHub.
>


``````
echo "    <option value='" . attr($provid) . "'";


A lot of that code depends on stuff not throwing exceptions, so this new `ensure*` would have its place there.


Wrap it in a try...catch?


test


thx


Not yet anymore ;-)


not working


it was so that i could edit .table's style in myownstyle.css instead of bootstrap.css.


这里应该return port > MIN_PORT && port <= MAX_PORT; 这样才对。


Fixed


attempted but not working


That's an option too.


Folder structure came up during a conversation with @lefthandedfryingpan. The "general" folders in the root directory -- that is, all folders except the project folder, will contain symlinks to the respective files inside each project folder for when someone wants solely to read up on or browse one specific topic.


This script needs to be converted to "new" security model before adding these flags(that should be a separate project).


condition is needed for avoiding the nesting of exception, here custom exception thrown by the validate function should not be added again into @queueProcessorException. It should add the runtime dao Exception while saving the data into @queueProcessorException collection


fixed


why not overwrite it? in production we will most likely be using a minified version of bootstrap, so we should not have to change its contents


@vincentchabannes That test does not compile :/
http://my.cdash.org/viewBuildError.php?buildid=775540


Maybe I understand your opinion.
You face the problem by the theoretical point of view, I face it by the practical point of view.
You are right but I don't think I am wrong.
Let me explain my words with an example:

General theoretical scenario:
--------------------------------
|     plugin     |     type    |
--------------------------------
| radiobutton    |    field    |
--------------------------------
| radiobutton    |   newfield  |
--------------------------------
| select         |    field    |
--------------------------------
| fieldset       |    format   |
--------------------------------

With the code:
$sql = 'SELECT si.plugin, si.type
        FROM {surveypro_item} si
        WHERE si.surveyproid = :surveyproid
        GROUP BY si.plugin';

What the query is supposed to select?
This:
--------------------------------
|     plugin     |     type    |
--------------------------------
| radiobutton    |    field    |
--------------------------------
| select         |    field    |
--------------------------------
| fieldset       |    format   |
--------------------------------

or this:
--------------------------------
|     plugin     |     type    |
--------------------------------
| radiobutton    |   newfield  |
--------------------------------
| select         |    field    |
--------------------------------
| fieldset       |    format   |
--------------------------------
This is your point. Ok, correct.

My way to operate is justified by the semanthic of the "type".
Plugins (what populate forms) in surveypro are ONLY with "type" == "field" or "type" == "format".
Plugins of type "Fields" are: radiobuttons, selects, booleans, textareas...
Plugins of type "Formats" are: fieldset, label, pagebreak...
Let me say that "plugin" is a subset of "type".
This means that I will NEVER find a db table like the one I draw at the beginning because
"radiobutton" can never have "type" = "field" and "type" = "newfield". A radio button is of type "field" ONLY.

So what I will find will be:
--------------------------------
|     plugin     |     type    |
--------------------------------
| radiobutton    |    field    |
--------------------------------
| radiobutton    |    field    |
--------------------------------
| select         |    field    |
--------------------------------
| fieldset       |    format   |
--------------------------------

In front of this table, as far as I know,
$sql = 'SELECT si.plugin, si.type
        FROM {surveypro_item} si
        WHERE si.surveyproid = :surveyproid
        GROUP BY si.plugin';
is perfectly valid and the returned fieldset is:

--------------------------------
|     plugin     |     type    |
--------------------------------
| radiobutton    |    field    |
--------------------------------
| select         |    field    |
--------------------------------
| fieldset       |    format   |
--------------------------------

By the way, by adding si.type, nothing would change!


Thanks.


it should be === 0


I would love to see that! I think it will make things much easier.

My vote is to also change the groupid to be `org.elasticsearch.plugin`, and make the versioning consistent as well. Lets make it as easy as possible on ourselves.


A meta-comment (perhaps a wishlist bug?): it seems that we may have to start using a dictionary for Units, since the instructors can put C++ code (I've seen it), txt files, zip files (seen this), jar files (also saw that) and we may want to grab all those resources for archival purposes. A named tuple doesn't seem to cut it (or, to be honest, I don't know much more than the basic usage about named tuples).


but the difference between how it passed the 15+0.1 test, and how it passed the 60+0.1 test is big, maybe it is wize to try to test with 120+0.1 and see if there is a regression with increasing the TC or not.

15+0.1 : LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
Total: 20149 W: 3830 L: 3707 D: 12612

60+0.1 : LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 45384 W: 7089 L: 7006 D: 31289


+1. That's what I have in mind as well!. I'd consider those plugins as the other ones but in another git repo.


@MoOx Super! Thanks.


What I meant with the comment above is that having `txt_urls`, `cpp_urls` etc., with one for each kind of possible file that exists on earth is not feasible. I hope that I am clearer now.


This code is added to capture the runtime exception (which may occurred while fetching the data from payload) thrown by the any of the function.


If we decide on something like what I propose above, we could/should revist these regular expressions, especially making them have named groups (which are so much cooler than having them named).


方法名是isInvalidPort不是isValidPort 哦


Looks like there's an extra " added to this anchor tag href.


also removed overlay in preparation of using bs's modal


hoorah! :D


finally :tada:


I suggest using parentheses around this ternary operator usage, because I had some problems parsing that as a list comprehension filter and failed until I mentally parsed it another way.


yay !


I've added a note in the documentation explaining the convolution.


Very short summary:
"plugin" is a subset of "type".
so by adding si.type, nothing would change!

The addition of si.type is better from the formal point of view.
I add it. Thanks.


Wild (and unfounded) guess: we may, perhaps, try to extract in the future every kind of file that exists in the page/unit (parsing is relatively costly) and, then, group the files by their types. I hope that this suggestion is a scalable way of achieving what I proposed at the beginning of my comments on this commit.


I'm not sure why skip if JDK 1.7


This looks OK, apart from the confusing list comprehension.


...a-a-and it will not work!


Please rewrite the "requirement" and pay attention about English grammar. E.g. this sample requires connecting wireless network, enabling location service. OR make sure sth is done.


The use of underscores makes this not be easy to use when moved to the appropriate module in the (hopefully) near future.


No, not anymore.


I'm not really sure what's the motivation for this, but the patch looks OK.


I added them to make clear what is left to do


(Changed here: a3f4fe181f16672ff018e6959c5105486d44c657)


In processing the `captureEvent(c)` method is only available using java reflection, here I avoid that by creating a `VideoInterface.java` ext file in JRubyArt 'video' branch. Then by the magic of ruby I re-open `Processing::App` and `Processing::AppGL` in a custom `video` library, and make them both implement the interface. @stevekinney @fluxusfrequency works on linux when I get a release out I will be looking for testers on macosx / windows.  Only slight downside is you should probably include `captureEvent(c)` in video capture sketches whether it is needed or not (also 'camel case' is required for reflection).


For me the build failed without these symlinks. dh_shlibdeps is looking for these and upstream also has the same under $(BUILDDIR)/.libs/ (AFAICR), but not installed. Will re-check or may upstream fixed this since then?


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


you meant 'no yet' right ?


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


This broke answer with Expression manager ... Disallow using {if(CONDITION,"Answer 1","Answer 2")}.


Hummm, your comment about "solves the issue of pages who didn't have youtube videos included" isn't clearer than the code. :) I guess that you meant that we should skip creation/extraction of units if they don't have "interesting" data (videos or other material to download), right?


Congratulations!


LGTM in concept.  Can we have more than 1 person confirm this on bare metal before merge?


```class A < ActiveRecord::Base; self.table_name="dbo.measures"; end; A.all```

```ActiveRecord::ConnectionAdapters::SQLAnywhereException: Table 'dbo.measures' not found: SELECT "dbo.measures".* FROM "dbo.measures"```


Guessing this was a mistake? (I updated it to 8eab1e5f194e186a020b5abf2cd0e409e53222a2 to get us stack traces.)


Suggestion: sort the URLs that you grabbed and verify all of them against a (manually inspected and created) list and then compare the two whenever there are lists to be compared. The code may work today, but it may break in the future when we "fix" the regular expressions to something else.


Ellenőrizni kell, hogy a downloadArea Paper vagy DownloadArea típusú objektum.


The `Zotero.Promise.coroutine(` isn't necessary. Can just be `, function* () {`.


Shouldn't this kept the same, according to soundOptions and controlOptions?


Looks good, apart from the huge HTML files. I suggest two things:

1 - split the fixtures from the test cases in separate commits (with the fixtures coming first, so we don't break git bisect).
2 - Use a shorter, more human friendly names for the fixtures, so that we know what they are testing. The fixtures can/should have their origin registered in the commit messages.


현재 이부분은 페이지수만  출력할 수 있게 구현된 부분인가요?
product를 16개단위로 보내는 부분은 없는거죠?


大屏幕不是黑屏问题


Thanks! With your fix working like a charm!


i've met the same problem,  your solution looks good


For me, the build was failing with them in here :-)
I haven't thoroughly examined all the upstream changes since you sent your original PR to see if they fixed something. How are you doing your builds? I just documented how I'm doing it at https://github.com/andrewpollock/grpc/wiki/Debian-Package-Build


I think this is fairly succinct. Maybe this is OK as a first shot and then people are free to refacter afterwards if any one chooses so.

Most important now is the public api. I Need to review the interface...


Remove this.


I think we should always show this as an exception, and just prepend the message if it's not null. The message generally provides more context (why the exception was expected, what the use case was, etc) but the code throwing the exception usually knows things that the calling code might not know, so we tend to provide both.


I think we should always include this in the exception message, and just prepend the user-supplied message if it's not null. The message generally provides more context (why the exception was expected, what the use case was, etc) but the code throwing the exception usually knows things that the calling code might not know, so we tend to provide both.


:-P https://github.com/VladaHejda/NetteSimpleTranslator/commit/1cd061ee315a09914677b4b0aca95c6b5cdba486


that's because `fname` is the full path, in this case, I'm checking `name` before it goes into the path because there can't be a directory before it before it goes into `glob.glob()`.
I don't want to match the file name part including the directory.

with just 'freetype' aline, I'm matching:
- lib`freetype`-6.dll
- 'freetype'6.dll

if you use 'freetype.dll', you can only search for something like:
- lib`freetype.dll`

so I'm providing many possibilities and a failsafe (/freetype/DLLS/freetype.dll) to load the freetype library.

note: freetype-py's freetype.dll does not have to be up to date...
at best, it just has to be an x86 dll that gets freetype-py working.
it's a failsafe, if you want better, put an up-to-date dll in your CWD or specify a custom directory with the dll before importing freetype.


I'd like to keep this, and put assertions (or logging) there to see if it is ever reached. As I told you before (but have not tested), I think that the Convex Optimization course *may* exercise this part of the code.

We seem to need a code coverage report, like what I use for coursera-dl, so that we know what is being used and what is not.

Suggestion: `py.test -v --cov=edx_dl --cov-report=html .` on the top level directory of the project. A directory named `htmlcov` will be created with the information that you can browse (and click on the files to see colored output). :)


Thanks. :)


alright, I can't argue with that... heh


Looks OK.


Comment


Much simpler :)


ack


Should be fixed now.


(re)ack


Here I disagree.

I prefer to have the videos downloaded from youtube, since there I can control their size (by choosing the appropriate resolution---I usually use -f 135+140 to get 480p videos, as that is a good balance in resolution vs. file size), while videos offered directly on edX may be (are?) only present in one huge or too small size (been there, done that---videos either at 1080p or videos at 360p).


I'm not following? :) What do you mean?


I will merge this, but this is one of the first places that I intend to kill if you don't mind.


OK, I also am going to remove the master -set from it, It is causing confusion to users not understanding that both versions are available in this branch. I think I will wait and do this "one time" when we are ready to release it.


Blindly prepending the user-supplied error message to an internally generated one sounds like a terrible idea. Can you give an example of the kinds of messages that people are supposed to supply s.t. this concatenation doesn't result in gibberish?


happy morning


:+1: Smart refactor


Not a good change, not sure what will be the most proper way to deal with the following issue: we call ```isCurrentPanel``` with _number_ thread id, but if we open conversation view directly (e.g. via ```views/conversation#thread&id=1```), then ```Utils.params``` parse ```1``` as string and ```Navigation.isCurrentPanel('thread', Threads.active.id)``` will be ```false```.

One of the ideas was to use ```json``` string to describe parameters that will also resolve issue with complex navigation arguments we have currently (objects like ```activity: { body: string }``` as argument for the new_message view).


Depends on the point of view. My packaging work is very different or generally the same. I used 'pbuilder create' and then 'DIST="unstable" pbuilder build' for building and then 'lintian --pedantic' to check. For packages in the archive I use 'dupload' to make those available for others.


Looks like ```beforeEnterComposer``` is the right place for it, since ConversationView (when we split it) won't load ```recipients.js```


Great.
I'd like to use this, but cannot find version 1.3.0 in Jcenter where I got the previous versions from.
I couldn't find out which repository this project is deployed to. Where is it?


I guess I was just saying that we tend to lean towards efficiency over code duplication - it's easier to have the 2 write methods calling each other (as done here), but it requires passing an array of 'dummyProcessors' in, which isn't so nice.  Yeah I was planning on taking a look over this on the weekend.


Not sure about event names


This sample requires connectting to wireless network


Yeah that's pretty much what I do. I deviated slightly from my normal environment for documentation simplicity purposes for what I wrote in the wiki. I normally use pdebuild configured to use cowbuilder.


Please, don't.

I'm usually lazy to pass a lot of options and then I see that I forgot the exact URL and, then, I put `--course-list` to remind me of the URLs...

Anyway, since we are talking about command line options, are you really attached to the name being `--course-list`? Intuitively, I would expect the verb to come first as in `--list-courses`, as in "Imperative mode" of verbs, like "computer, do this".

Examples from youtube-dl:

--recode-video
--keep-video
--no-post-overwrites
--embed-subs
--embed-thumbnail
--add-metadata
--list-subs
--prefer-ffmpeg

and so on.


In this PR I've got rid of startup dependency in separated views, so had to remove this dependency with ```setReady``` - would be glad to hear better alternatives!


Right now, I don't know any better way to do this.


Last commit before we presented


This sample requires MotionEvent API support, and connecting to a gamepad(such as XBOX) device


Got it already. Sorry for the noise.
At "https://plugins.gradle.org/m2/" as you've described in the Wiki


This sample requires enabling location service


Why was this changed? Breaks now when projects.py calls repository.get_working_copy() which provides a path=None.


I think always calling ```InboxView.init``` won't hurt loading of the app from notification/activity too much, or I'm wrong (rationale: just didn't want to call ```init``` in two places)? :)


New commit


Well done! Now it's Mr Gärtners turn for the frontend modifications.


Android 4.2+ with Miracast protocol support.


Made this method just to simplify code in startups, but may get rid of ```review from shared peer``` dependency by moving ```on/off pair``` to sms app itself.


this ifdef or some variation is needed when deploying a build to hardware.  Simulator doesn't care.


Amazing, thanks, it seems to be working well!


Ough, what a mess


This is a no-go. We're not to add heavily out-dated versions. Please fix.


We should be careful with `get_page_contents` and `get_page_contents_as_json` because they use the network and the edX people would (justifiably) start to get really mad at us for having automated tests that hit their site.

Furthermore, those tests that depend on network connection can fail even if the code is correct, say, due to lost network connection, downtimes etc.

Those functions/tests that depend on net connection should be tested in a separate module and, perhaps, used with Mock connection objects. This is one topic where I still have some learning to do.


Added Javascript


This comment is not sufficient for me to understand what is missing in the code (read: "what should I fix?").


Looks good, but some clarifications are needed.


Sounds good. You can also come into my office hours.

--

Mike Sinkula
Professional Twelve Year Old
http://www.mikesinkula.com

> On Jun 3, 2015, at 11:00 PM, Maddy H. <notifications@github.com> wrote:
>
> Hey Mike, I followed the video best I could. Went back through and read your post to make since out of it, but still having trouble. I might need assistance in the next couple classes.
>
>
> Sent from my iPhone
>
> > On Jun 3, 2015, at 6:05 PM, Mike Sinkula <notifications@github.com> wrote:
> >
> > But, it doesn't look like you used your thumbnail featured image in your site?http://maddyswebnation.com/web170/templates/wordpress/blog/
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


I love the look and feel of the code, and am impressed by the fact unicode is handled 'manually' - after all the scanner seems to operate on UTF-8 encoded bytes. This might squeeze out some performance, or is there another reason to do this ?

Something I find most interesting is the fact that there seem to be no (but two auto-generated) unit-tests. How was it even possible to develop code that complex without TDD ? Or is there something I am missing ? Looking at my own workflow, I simply have no idea if something is working or not unless a test tells me about it.

Thanks for considering a reply.


the way it looks right now, you want users to compile the dll...
this is generally bad in many cases (for example, another library, glfw appears to, but won't compile correctly on my machine)
I havn't attempted to compile freetype as you've been very kind in supplying a binary :)

I believe it best if a **working** binary is distributed with the extension.
(x86 runs on everything, it doesn't have to be fast or even efficient, it just has to work)
^ let the user deal with finding a better binary

> that directory isn't even going to exist most of the time.

the failsafe ./freetype/DLLS/ directory should be created in the distro, this should not be created by the user.
it should exist 100% of the time, though if a dll is in any of the first 3 path choices, this directory would be overlooked as last priority.

> which over time would cause some repo bloat.

this is the only legit case I see, I think we should leave this up to the base developer.
idk much about repo bloat tbh, but we'll see what he says on this... :/

> This also would probably increase the work needed to do a release of freetype-py.

I don't think so but idk how you guys make releases, so I won't question it :)


also was ich ein bisschen quarkig finde:
[ "$INSTALLATION" = "ffweimar" ] && ... keepsetting
wenn sowieso _immer_ keepsettings gesetzt wird, brauchst du es auch garnicht machen.


you took out the link back to the item, can you add back `{{ item.url_item }}` to the `href=`?

```diff
- <a class="metadata__heading-link" href="{{ item.url_item }}">View complete record.</a>
+ <a class="metadata__heading-link" href="">View complete record.</a>
```


:clap:


cool man


slick - from gabe


Excellent. Thanks. I guess that we can use the `strip_non_ascii_chars` (not named this way) from coursera-dl. I hope that that one is better than this one and it is used all the time and included in our test suite: https://github.com/coursera-dl/coursera/blob/master/coursera/test/test_utils.py#L24-L39


Right.


Mostly performance. My goal was to make the fastest CommonMark parser in
existence. I'm not quite there yet, but making progress.

Yes, it was very much TDD. The main test suite was spec.txt from the
CommonMark project.

Thanks for your interest.

Raph

On Thu, Jun 4, 2015 at 12:02 AM, Sebastian Thiel <notifications@github.com>
wrote:

> I love the look and feel of the code, and am impressed by the fact unicode
> is handled 'manually' - after all the scanner seems to operate on UTF-8
> encoded bytes. This might squeeze out some performance, or is there another
> reason to do this ?
>
> Something I find most interesting is the fact that there seem to be no
> (but two auto-generated) unit-tests. How was it even possible to develop
> code that complex without TDD ? Or is there something I am missing ?
> Looking at my own workflow, I simply have no idea if something is working
> or not unless a test tells me about it.
>
> Thanks for considering a reply.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/pulldown-cmark/commit/dfabea724ad536dcf0231adf05f6f46ddcbded9e#commitcomment-11515276>
> .
>


@gRoussac  Do you mean, "All Captions"? Is this new function to update all captions in the list? In english, you would say, "all captions" not "each captions".


What was the problem here? What didn't work? And what do you mean by a "downloader"? Something that does the job of, say, "wget"? We have a class on coursera-dl that does just this, but I don't like it (especially the retries), but for people using coursera-dl on Windows (not so sure about macosx), they really seem to need it.

I, OTOH, use `aria2c` almost exclusively.


Good.


https://github.com/shminer/LG-F460-Kernel/commit/4d26f6cd00988b71824464aca86b2e60e29b2ffd

Now hotplug will suspend when max sleep core < 2.
If max sleep core >= 2,rq work and hotplug work will not suspend.
Add hotplug resume time for speed up do hotplug queue work.


That's fine if it is being addressed.  Sorry for noise.


What happens if vec.entries[i].len is above 65535?  Such case may happen if the length of a single header value exceeds the value.


You won't believe it, but I was going to suggest *exactly* this after I suggested that previously (when I was in bed, I was thinking about modifications of the code).


modification of the opening stream.
delete of the useless includes


please check MAD and Deriv(salt)


yes on latestt comit e96d1e7cab17e73008b2f4d6ee28fc1f4da27f4e
at line 643 clamav-unofficial-sigs.sh

regards


Seems to be a Rails [issue](https://github.com/rails/rails/issues/19001).

As far as I understand, there is no actual change to the database, but I will track the issue.


We prepend the user supplied message in most of our APIs. See `assertEquals`. Let's please be consistent


Nice. It's actually less code without the extension :)


It seems you reply as fast as your code runs ! Interesting you started off with byte-based scanning right away, I'd probably have shied away from the extra complexity at first and waited for some numbers to see how much faster byte-based operation can possibly be. However, in one of my projects, `json-tools`, I noticed a 30% speedup just by switching from char-based iteration to a byte-based one.

Admittedly I have never seen that spec file, could you provide a link ?

My interest stems from my current involvement in `serde-yaml`, currently my progression speed is quite low, which is alarming especially considering I am 'just' implementing the serializer, which ought to be the simple part.

Thank you


i found it ,you are right


I think we should remove the "_" in the _resource_get method, the "_" means private method but is used in a public context here.


There is indeed an implicit coercion from `bool` to `Type` in most practical cases. If you need to call it explicitly it's called `b2t`


Good point. But I believe optaplanner might want to allow entities to have no possible values for a variable. Or put differently, a variable's value range cannot be empty. Need to think about that.


Ideally this needs to be fixed in wherever the bug in the core parser is.

On Thu, Jun 4, 2015 at 1:57 AM, LnmVolbo <notifications@github.com> wrote:

> Found something interesting on the edits to arcade.lst on this commit.
>
> I compiled this after it was committed and found the two new games did not
> appear in the .exe or the -listxml. I edited the two new game lines,
> removing the tab characters after the game names and before the "//", and
> replaced them with space characters, enough to get the "//" back in
> alignment.
>
> Recompiled it and the new games appeared in the .exe and the -listxml.
>
> Not a big deal, but it looks like whatever is parsing the arcade.lst file
> is expecting a certain kind of white-space to delimit the names. It did not
> like the tab characters.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/4d3ae8b5aeff6f4a396051dd8467077a3c087929#commitcomment-11511699>
> .
>


Can you tell mode about lid resume controlls and what acticating lid_resume_S5 will do?


Would it make more sense to sort at the View layer?


P.S. I'm busy in exams too:smile:

Where are you studying in? I'm in National Taiwan University


Should we compare against another Version object?  Magic number consts inline the code are generally :(


Hmm, as I set it up, you don't have any empty lists, but performing head/tail on singletons (which does nothing) is a bit awkward.

Including 'union' and 'init' seem like good ideas. If we're trying to evolve a list, I'm not sure operations that return singletons are a good idea (Hmm, perhaps including head wasn't a good idea of mine :o. I'm learning just like you >.<;)

Hmm, the problem with swap is that we don't have a way to choose which elements to swap. So it's awkward.

ShiftL just took the first unit and put it at the end, thus 'shifting' all elements to the left.

Btw, I see that you didn't add anything to GenProg's instances. Are you running genprog with different sets of the operations and seeing which work better than others?


```URLResource.create(loader, script)``` does not strip the "uri:classloader:" prefix and the ```inputStream``` method fails and catch below finds something usefull.

but looking at the ```locateExecutable(final Object scriptingContainer)``` it now feels to save to merge my fix. warbler will get the script under any allowed uri-like paths.


I don't see any animation. What I see is that the sidebar closes itself when I open the page… :)


LGTM


Interesting description, lots of something made by why and why? From where? Seems ok, but who made it and why?


I realize that, but it doesn't really answer my question. Why prepending, with a single space? Think about it: if the user types in:

`assertEquals("no content", 204, response.getStatusCode());`

The resulting error message is:

`no content expected:<204> but was:<500>`

instead of something like:

`expected <204> but got <500> // no content`

or:

`expected <204> ("no content") but got <500>`

or literally _anything_ that demarcates the user's message.


yep thats better


Right you are.

Would it be prudent to add a comment in the file header to avoid tab characters immediately after the game name?  Looks like the parser only likes spaces or carriage returns as white-space.

Of course, it would be best to correct the parser.  But a note would be handy in the header in the meantime.

Ya know, it's tempting to leave things as they are and the parser trip over the tab characters.  That way you could add a game and not have it show up.  Might have saved a bit of trouble with that recent monkey business...


General best practice: We should always be explicit on the string Comparison type ; calling string.Compare with a specified StringComparison type.


No, it's just to keep valgrind happy.


测试


I think it was `/var/lib/postgresql`. Where `pg_createcluster` is placing its files?


this is my assignment 4


Maybe `menu` should rather be initialized with `null`?

    /** private: attribute[menu]
     *  ``Ext.ux.ColorMenu``
     */
    menu: null,


https://github.com/jgm/cmark/blob/master/test/spec.txt

Heh, ordinarily I am known for being slow to respond. But I posted this
release with anticipation to see what the response would be :)

I also started with using the character-based iterators, but aside from the
performance characteristics, found that using the Unicode methods was also
not very pleasant. Since everything is based on slices (and the return type
is an offset within a slice), I constantly had to count the number of bytes
in a Unicode character, which was especially frustrating when I knew it was
ASCII. There are possibly better ways to do it, maybe involving an iterator
that scans bytes/chars directly, but this is what I ended up with.

Raph

On Thu, Jun 4, 2015 at 12:15 AM, Sebastian Thiel <notifications@github.com>
wrote:

> It seems you reply as fast as your code runs ! Interesting you started off
> with byte-based scanning right away, I'd probably have shied away from the
> extra complexity at first and waited for some numbers to see how much
> faster byte-based operation can possibly be. However, in one of my
> projects, json-tools, I noticed a 30% speedup just by switching from
> char-based iteration to a byte-based one.
>
> Admittedly I have never seen that spec file, could you provide a link ?
>
> My interest stems from my current involvement in serde-yaml, currently my
> progression speed is quite low, which is alarming especially considering I
> am 'just' implementing the serializer, which ought to be the simple part.
>
> Thank you
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/pulldown-cmark/commit/dfabea724ad536dcf0231adf05f6f46ddcbded9e#commitcomment-11515426>
> .
>


No, we aren't. We are using the upload cert API in ID. Will change the comment accordingly.


I noticed that right after submitting the PR and corrected it in an additional commit.


Thanks!


In the column directly above this there is

     haskell.compiler.jhc            jhc-0.8.2
     haskell.compiler.uhc            uhc-1.1.9.0

so this is somewhat confusing.


ich bin auch dafuer es bei TABS zu belassen. einmal damit angefangen und dann bleibt es auch so.


Okay. thx.


2松岛枫


CONFIG_PFT is needed for crypto. data encryption.


> If I run the same two commands, I get this difference:
```
diff --git a/src/common-opencl.c b/src/common-opencl.c
index c0999b9..5efdcfa 100644
--- a/src/common-opencl.c
+++ b/src/common-opencl.c
@@ -997,12 +997,14 @@ void opencl_build(int sequential_id, char *opts, int save, char *file_name)
>
 				memset(&lock, 0, sizeof(lock));
 				lock.l_type = F_WRLCK;
-				while (fcntl(fileno(file), F_SETLKW, &lock)) {
+				while (fcntl(fileno(file), F_SETLKW, &lock))
+				{
 					if (errno != EINTR)
 						pexit("fcntl(F_WRLCK)");
 				}
 #else
-				while (flock(fileno(file), LOCK_EX)) {
+				while (flock(fileno(file), LOCK_EX))
+				{
 					if (errno != EINTR)
 						pexit("flock(LOCK_EX)");
 				}
```

I have the same problem with @frank-dittrich .
```
$ indent --version
GNU indent 2.2.11
$ astyle --version
Artistic Style Version 2.05.1
```
The 2.05.1 is the latest version of astyle


fsdfasdf


The id changed from graphicsOptions to optionsGraphics, while soundOptions and controlOptions kept the same. I would either write optionsSound and optionsControl or switch the other back to graphicsOptions.


@zufrieden Ok so we simply don't need this plugin at all... A select can be styled with CSS.


:+1: wooohoooo


Delete is a keyword, not all JS engines can handle it. Specifically, ES3 spec seems to forbid it (ES5 being less strict).

Taking into account that this is the core of tsc, it's not good to limit the compatibility so. Compiler is still expected to work in ES3.

Consider renaming to 'remove' and 'removeItem', plese.


Do we need `populateA` and `populateB` (if so, are there better names?)


good


I mean all the hyperlink in forum topic  content are still  hercules.ws
![666](https://cloud.githubusercontent.com/assets/2773211/7979182/b8fb85aa-0acf-11e5-8433-87b6b6bddd7f.png)


App sid update


I just noticed this. Whether we delay the asset creation of not. All shot assets should not be stored within ftrackData. Only the final asset to be used by this publish. ftrackData should only ever contain entries that are relevant to to current publish. Other potential assets on the task are not. Instead of saving them into ftrackData, we should just call task.getAssets() when about to validate assets.


first like should do the bump


Oh yes the element IDs. Yeah, should be changed also. Perhaps the same as the screenIDs now, more simple & uniform. Good eye.


please use 22222


yep create another ticket in 1.1. milestone ;-)

On Thu, Jun 4, 2015 at 9:35 AM, tonifisler <notifications@github.com> wrote:

> @zufrieden <https://github.com/zufrieden> Ok so we simply don't need this
> plugin at all... A select can be styled with CSS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/antistatique/antistatique.net/commit/80f8170a7932b4a5be101ed9c40fc9145ee6e8b4#commitcomment-11515650>
> .
>

--
---
Marc Friederich
marc@antistatique.net

Antistatique <http://antistatique.net/>
rue de Sébeillon 9b
1004 Lausanne
+41 21 623 63 03
www.antistatique.net


You're doing the work twice here. Use `index` instead of `find(selectedPhotoIndices, indexPath.section)!`. Preferably appended right to the else statement like `else if`


this looks a bit weird shouldn't this line end in a `;`


:octocat:


Not entirely sure whether this is the right behaviour. Why is it necessary to use `find()` again here?


@ticket = current_user.tickets.new


Can you fix the indentation so that it was like before please :)


> There's no need for them, a push queue executes instantly after you push it so the same as executing the code then and there.

@Sladewill That's not correct, right? Jobs are only executed when the provider (such as Iron) hits your marshaling URL, so that definitely happens in another thread.


Here's a check page:

http://jsconsole.com/?var%20s%20%3D%20%7B%20delete%3A%20parseFloat%20%7D

Run on webpagetest:
http://www.webpagetest.org/result/150604_VP_CYC/
[http://www.webpagetest.org/results/15/06/04/VP/CYC/1_screen.jpg]


Hi Mikhail,

When are you planning to release this? :)
Thanks!


Why do you need to set the messageID? You should only set the "References" and "In-Reply-To" headers AFAIU.


次はInterfaceを使って分岐をなくしてみましょう


:+1:   but this make test failed, please make sure test pass before push next time :p


Fixed.


Maybe Conflict here, instead of BadRequest


I'm not familiar with Iron.io but I was referring to the Laravel Queues in general as that was what the original comment was about.


どんなサービスを提供するのかの説明などがREADME.mdに記載されていると嬉しいな、と思いました。


Sounds good to me, I'll change this when splitting it one validator and one extractor.


@KozhinM Why is this not:
info.range.end = sidx.size ? (info.range.start + sidx.size) : info.range.start + extraBytes;


好


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


エラー詳細がわからないとシステムリリース後に対応ができなくなるのかな、と思いました。
どんなエラーメッセージが必要かを記載したほうがいいと思いました。


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


@KozhinM This is tricky. Maybe lastBox is incomplete and we didn't get lastBox.size? In such cases also maybe download the next 1500 bytes


Same here, Conflict


I think this should be called `maxSelectionCount` to emphasise that it's ok to select less photos too.


Anyway, the header says the parser is makelist.exe.  The source in makelist.exe is src\build\makelist.py.  Line 31 seems to be the line that checks for the first space after valid game name characters.  Perhaps if line 31 was changed from:

            if c==' ':

to this:

            if c==' ' or c==9:

it might parse tabs better.


test


@raven-chen :+1:


@KozhinM Sure :)


Laravel queue has a driver for push queues, only available if you use Iron.io. The rest of the drivers are there, I get that, but now there's no push queue in Laravel.

The other drivers are not push queues. So I wasn't mentioning the whole queue system, just the push ones.


2 哈哈


So what about any Windows-specific handling code we have? cc @iafan.


Please do not apply it by default. It replaces map01 and map02 and should strictly be optional.


Thankfully that token is already expired. Heh.


Why not set the value to `null`?


But mm11fix.wad is already in the dsda ZIP file that we download anyway.


//0x3 & 0x7 != 0x1, bad system call


maggie change


Could we rename this method to "mson" for an easier use with static import?


@mindfreakthemon
It's gonna be released in 1-2 days. If it really affects you right now, you can build it from 'master':
```
bundle install
bundle exec rake build
vagrant plugin install pkg/vagrant-parallels-*
```


This method should be deleted and the contents being moved up into the proxy method above?


flagという名前ではどのような状態かわからないので，変数名を変えて状態をわかるようにしたほうが良いですね．
is～とかneed～とか

ただ，そもそもフラグをなくすようにしたほうが良いです


If you want push queues, you can still have them. I'm sure someone will make a package. They're just not staying in the core from 5.2+ because they don't get used by enough people.


Rename this method to "msons" for a better naming symmetry?
We would then have mson and msons. Epic! :smile_cat:


Hi Will,
Still got some problems I think, you can see I use "blue" as background-color. But it should be series.color and I don't know how to use it in a .css file. Maybe I shouldn't use ng-class?
I had tried background-color: {{series.active ? series.color : "grey"}}, but it seems don't work.


Why not use `npm` or `bower` to load angular library?
Commiting it to repository isn't a good way to manage your dependencies.


+100 :+1:


:heart_eyes: :heart_eyes: :heart_eyes:


Made my day!


Not ready yet, there's still the jerk. @vidya-ram any ideas to fix the jerk?


hello


@clefranc
Is the log above made with one of the 25°C sensors?
The temperature is still rising, but count says it's falling. The likeliest reason, is a unexpected value of GEBED.
GEBED depends on HEATER_BED_RAW_LO_TEMP > HEATER_BED_RAW_HI_TEMP - and these are calculated in the TEMP_MIN/MAX_ROUTINE() macros. If now BED_MINTEMP is outside the range of the temptable this may result in negative values - elsewhere interpreted as big positive - flipping the direction of GEBED. I'll investigate this.

Test 2 and 8 do not work because: a. There is no jump - when disconcerting. b. There can not be a decreasing temperature while heating and we measure below ambient temperature..

I'm close to giving up on this sensors.


@dodgex do not forget you should change the JavaDoc of the annotations as well.


Thank you ! I ran the spec file and saw it runs 548 tests, failing only one ! To gain trust, I'd recommend making it more obvious how the software is verified - I look for standard rust tests, but running on a spec file is even better thanks to the exhaustiveness. Maybe you could add the spec to the repo and provide a simple makefile that shows how it is run. That way it is at least indirectly documented. In any way, it's a workflow I'd love to adapt in my own efforts as well - currently I work with [this spec](http://www.yaml.org/spec/1.2/spec.html#id2776092), certainly there is a text-only version of it somewhere.

It's interesting you found working with character iterators unpleasant - when scanning through an `str` the `char_indices()` iterator was working well for me.
So far I have made no experiences with working on input-strings though, as everything I have seen or have written was based on iterators yielding chars most of the time. `serde` for instance goes as far to take an input iterator yielding `Result<char, std::io::Error>`, which happens to be the return value of `Read::chars()`. Even though I adore the concept of being 'fully-streamed', I also love the idea of being able to just yield spans to allow indexing the source data directly - something that is not directly possible with streamed input. At the end it seems to come down to what one prefers: minimal memory requirements at the cost of performance (but minimal delay to getting a first result), or the other way around with higher-delay to the first result. Personally I seem to prefer the 'low-memory-low-latency-high-CPU-streaming' approach. But one would have to provide both interfaces to cover all the ground there is.

Still reading ? Wow, I got carried away there, random thoughts :).


4回目


The problem remains when upgrading the melpa package because the original defalias is used in the compilation of the new code.  This is because the expression `(fboundp 'save-mark-and-excursion)` above is true at compile time because it is bound to the defalias defined in the existing package.  As a result the definition is not replaced with the macro.  While I haven't tried it, it would appear that only way to upgrade to the latest version is to remove the existing package and then reinstall it using a fresh emacs.


ファイル名の日付とfrontmatterは同じじゃないとbuile errorになりまする。

> /home/ubuntu/tech.feedforce.jp/vendor/bundle/ruby/2.2.0/gems/middleman-blog-3.5.3/lib/middleman-blog/blog_article.rb:179:in `date': The date in posts/2015-06-02-monitoring-mackerel.html's filename doesn't match the date in its frontmatter (RuntimeError)


Yes please, we have to document the intention (rules) of all the other modules too.


ファイル名も変更必要。
https://circleci.com/gh/feedforce/tech.feedforce.jp/293 で落ちてたのはそういう理由。


It is set by openssl.


I'm afraid this a wrong approach. The logger should be running on another thread, especially when you use saving to file instead of the database. Logging into a file will lag badly with a lot of players.


tagに記号が入った場合も正しく動作しますか?例えば%が入った場合など
urlencodeが必要ではないですか?


Sorry, my mistake. `$strong` is an reference var.


jQuery only selects the first element in $('#myId') query so we had to use this approach as we need all elements with that id.


Yay!


can you change this to @return boolean


a bs4?


@misgeatgit there an extra parenthesis here but I don't think that's the problem.


@misgeatgit could you paste the error message here? Thanks.


Да, у меня тоже не верно сделано/

Вот ход выполнение задачи при её непрерывном "DECAY". Выполнение задачи будет зациклено, так как "static::DELAY_TIME^($jobStats->delay + 1) " всегда не больше 30

- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:10:48
32 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:11:03
46 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:11:13
56 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:11:27
70 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:11:37
80 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:11:51
94 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:12:01
104 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:12:15
118 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:12:25
128 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:12:39
142 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:12:49
152 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:13:03
166 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:13:13
176 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:13:27
190 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:13:37
200 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:13:51
214 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:14:01
224 <- jobStats age
10 <- jobStats delay
12 <- jobStats->id
14 static::DELAY_TIME^($jobStats->delay + 1)
- Error(DECAY)  Email: 111111@yandex.ru Date: 04.06.2015 12:14:15
238 <- jobStats age
14 <- jobStats delay
12 <- jobStats->id
10 static::DELAY_TIME^($jobStats->delay + 1)


Nice deduction there! Well done. :) So yes indeed, you have to remove your old expand-region, restart emacs, and reinstall.


Hi vpisarev, I have some questions about the optimization of sqsum. For WxH images, possible max value of sqsum is WxHx255x255, which is very likely to be larger than int32 (when W & H > 255). In that case, the result of sqsum will be incorrect.


but it was already in the .deb; so I just need to revert last change


This process is still doesn't work. See issue #1


I agree that having these files in the repo is cool for people who want to quickly build F* from sources. On the other hand it's a bit of a pain in the ass for developers to always have these files changing whenever they run `make` (btw, there is no global cross-platform fixpoint to be reached because of different line breaks). So what's a good solution to tell `git` that most of the time we don't want to care about these files changing all the time? I'm by no means a git expert, but here is what I discovered so far:

Adding these files to `.gitignore` doesn't directly help, because `git add`ed files are tracked anyway. I've still done it though as a way of marking the files for the following command:
```
git ls-files --ignored --exclude-standard | xargs git update-index --assume-unchanged
```
It locally disregards changes to these files. More details about `git update-index --assume-unchanged` here: https://stackoverflow.com/questions/1139762/ignore-files-that-have-already-been-committed-to-a-git-repository/11366713#11366713
You can always go back on this with the same command just with `--no-assume-unchanged` instead of `--assume-unchanged`:
```
git ls-files --ignored --exclude-standard | xargs git update-index --no-assume-unchanged
```
More details here: https://stackoverflow.com/questions/17195861/undo-a-git-update-index-assume-unchanged-file

So far this worked well for me locally. Please let everyone know if you have problems with it, or if you have better ways of doing this.


@manish - Should the NUnit NuGet package be in the packages.config file too? It looks like the NUnit dlls were directly referenced instead of using NuGet. Json.NET, for example, is in the packages.config file. Directly referencing will work as long as another project in the solution is using NuGet and has the NUnit NuGet package referenced.


Thanks, I will add it.


:clap: :+1:


I will do it in a separated PR to reduce the complexity of this PR.


I wrote some comments on the original pull request - or are we moving to this place now?
with the *10 removed full deflection on you're RC will probably result in no noticeable yaw_rate command and therefor make no difference if you have it or not...


Tested on Debian 6 and CentOS7.

TL;DR
Debian 6 still uses SysVinit scripts, as well as Debian 7. Debian 8 and CentOS7 prefer systemd scripts, but obviously have backward compatibility. So, to be honest the best solution is to rewrite the scripts altogether as systemd scripts, since Debian and RHEL-based distributions now use it. I see this as a band-aid, which I needed so I can deploy Kairos with Ansible. I'll publish the Ansible playbook as soon as it's finished and pretty, and I could probably try to write the systemd scrips after.

Can you please tell me what are you using to package kairosdb in rpm and deb packages, so I could possibly use the same method as you are when testing.


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


не нужен пустой файл


ok fixed, https://github.com/a-detiste/game-data-packager/commit/1bb22cc183e43c4f05f83b917033c5f47c443907
mm_allup.zip now `provides:` mmfix.*

> anyway

... unless someone already has an unpatched mm.wad & GDP become smart enough to download the smallest file provider: 65K vs 2,7M ...

In a normal run, mm11fix.zip is never downloaded:

```
~/game-data-packager $ ./run compet_n --destination /tmp --no-compress
INFO:game-data-packager:downloading http://localhost/requiem.zip
INFO:game-data-packager:downloading http://localhost/mm_allup.zip
INFO:game-data-packager:downloading http://localhost/class_ep.zip
INFO:game-data-packager:downloading http://localhost/av_new.zip
INFO:game-data-packager:downloading http://localhost/mm2.zip
INFO:game-data-packager:downloading http://localhost/hr.zip
INFO:game-data-packager:generating package doom-compet-n-data
generated "/tmp/doom-compet-n-data_42_all.deb"
it is recommended to also install this game engine: chocolate-doom
```


пустой файл


пустой


congrats


Thanks


dfdgvfd


new line


fvvvcvc


shouldn't we then just delete it?


Oh yes, that function is horrible, I particularly hate the global thing. I will check if I change this soon.


Regarding step 8, can anyone verify that an explicit ``git add .`` is not necessary?


Remember the *delete* your SF project. The GIMP tombstone was how SF justified their hijacking of that project.


The link has not been updated.


cccbfbcxc


Pushing a squashed version of the branch now.


difficulté


Should we think about speed? Or if it should be simplified?

http://jsperf.com/while-vs-map-findindex


maybe a try-with resource here. (nice nail for this hammer)
try(FileOutputStream templFo = new FileOutputStream(templateProp);){
    templFo.write(templateContent.getBytes());
    templFo.flush();
} catch (final IOException e) {
     throw e;
}


Guys, would it be possible not to use `@dim` in your commit messages? Github insists on sending me an email every single time :)


Yes I haven't thought about those other cases, yes, go to my mental TODO list.


It was changed as part of harmonizing the structure of the different WorkingCopy and Subversion classes, but clearly this introduced a bug which was missed by our tests.


It's from Moto :) https://github.com/MotorolaMobilityLLC/kernel-msm/commit/328ce711c0741f59618304da39ac05f599520baf


Good move


Some spelling issues in commit message


Nice.


Maybe put this back to shutdown because it's not required for the test anymore.


1) Shortened class names in fff.renderer for convenience. File names prefixed with "FlameRenderer" have been shorted so that they are prefixed with "Renderer" instead, with the exception of fff.rende.FlameRenderer and fff.renderer.ocl.FlameRendererOpenCL.

2) Created fff.render.BasicCallback as a basic implementation of RendererCallback to print updates and save finished images as PNG files. Modified the fff.demo files to use BasicCallback. This greatly reduces the boilerplate code needed to get an FFF HelloWorld running.

3) Various modifications to the demo files. Other than having them use RendererCallback, the biggest change is that they no longer call System.exit() from within their RendererCallback once the last image is finished. Instead after calling renderer.shutdown() in the main method, they then call renderer.awaitTermination followed by System.out.println("Done").

3) Added several methods methods to TransformAffine, such as translate and rotate, to manipulate the underling affine more easily.

4) Moved the newSierpinsky and newReandom Flame methods into fff.flame.FlameFactory. FlameFactory also provides several more stock fractals such as Barnsley Fern, Dragon Curve variants, etc. These static constructors were moved into a new class since they take as many lines of code as the rest of Flame and do not require access to any of Flame's protected or private members or methods.

5) Corrected bug in FlameRendererOpenCL maxBatchTime calculation.Encapsulated the corrected calculation code in protected method FlameRenderer.calcMaxBatchTime().


TeamCity YaccConstructor :: YC.Core General Build [Build 425](http://teamcity.codebetter.com/viewLog.html?buildId=192691&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 136 Build time: 00:13:34


Agree


Would `&` be better, or just a separate function like `promote_linear`?

I'm not sure I agree with `eachindex(A,B)` returning two iterators. You can already do the `zip` thing manually. What's otherwise lacking is an easy way to generate the "least common denominator" iterator for multiple objects. I don't remember my benchmarks on this, but I seem to remember that incrementing two rather than one is a noticeable performance hit for a tight loop.


debug parts from here: MotorolaMobilityLLC/kernel-msm@328ce71


Thanks!


This is not necessary I think ?


The motivation is that once you have the mp4 video from a cdn its quality commonly is better than the youtube one since it is regularly the source file, however there are courses that have high definition videos e.g. Introduction to Functional Programming (Erik Meijer rocks btw), and I don't know what can we do to filter those, since you end up downloading things like: video1.mp4 (1.2G) !, video1.360.mp4 (140M) video1.720.mp4 (350M). And the problem is there is no convention on the name of the files to  asume among courses.

Any idea for this ?


No, I was referring to pages (in particular in non-edx websites like MIT Pro X or FUN) who only have mp4 version of the videos, the script broke and didn't extract anything.


Need help to remove 'alert_id' column from 'cap_area_location' table as this table is now inline in 'cap_area' table


I see we have already a report loaded in this variable https://github.com/dmwm/CRABServer/blob/master/src/python/TaskWorker/Actions/PostJob.py#L1703, can we use that instead of loading it another time? It might be fine to load the report many time in the worker node, but on the schedd I would like to optimize its loading since we execute thousands of postjobs. Also, the variable G_JOB_REPORT_NAME_NEW already contains the filename (`"job_fjr.%d.%d.json" % (self.job_id, self.crab_retry)`)


er.. Do we need explain how `gorm` works in QOR tutorial ?


I don't get it, can you be more clear please.


@clefranc
Was PIDTEMPBED defined during the log?


I don't think there is a way to do this without jshint complaining or indenting the whole file by 4


Yes the names are huge but I put them like this to have the same URL like form, but yes renaming makes sense, I don't understand point 1.


Need to update all test cases and functions calling this func.


no segmentation fault anymore.


PS is it because the retryjob has not been executed?


Can you tell what caused this regression?


Well we can add it for test value, but believe me I have tested like crazy and I have not found this case once !
Also, it doesn't make sense to put things there when you author in the openedx platform, and I think this code still exists just because it comes from a really early version of the script and the platform.


dian zan
 点   赞


I would have loved to see a comment why you need to skip this test.


how should we now create one without example or instruction?


So why do you add this `meqform.py` file? It looks like you committed `meqsilhouette.py` with the same content already:

https://github.com/ska-sa/rodrigues/commit/cc935c4b1116b78c7abc4e64115b6c1f6eb9e01a


`'exitCode' in report` is the pythonic way of doing this.

Anyway, why only these three exit codes? What if tomorrow we add an exit code in the RetryJob that we need to report to the dashboard? I think this should be something like if we got fatal exception (from https://github.com/dmwm/CRABServer/blob/master/src/python/TaskWorker/Actions/RetryJob.py#L159) we report the exit codes.


I'd rather contact the developer or ask for Google to intervene.


No, once the downloader is ready I will put a switch to choose if you want to download the videos from youtube or from the cdn, the real question is which one is the best default ?
For me the best default is to take the direct mp4 files since regularly they have the highest definition (they are re-encoded in youtube), but I understand your point about the youtube resolutions. Maybe I will let it like it is, and put a --prefer-cdn-videos as an option or something like that.


oh sorry meqform.py is meant to be named meqsilhouette.py

On Thu, Jun 4, 2015 at 10:55 AM, Gijs Molenaar <notifications@github.com>
wrote:

> So why do you add this meqform.py file? It looks like you committed
> meqsilhouette.py with the same content already:
>
> cc935c4
> <https://github.com/ska-sa/rodrigues/commit/cc935c4b1116b78c7abc4e64115b6c1f6eb9e01a>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ska-sa/rodrigues/commit/b949810d09240903b1f5de401a3a4e0d72452380#commitcomment-11516819>
> .
>


This should be on the master - I spent hours trying to find out what was going wrong because of this bug!


apparently, gvfs & libsoup-gnome are missing as dependency (also,  gir-freedesktop  isn't actually necessary) . I don't want to spam, so i'll wait for the commit until i know how to make this a valid template.


The ``postFixup`` is failing, only it's hidden by #7524 (but exposed by changes on gcc-4.9 branch). I was unable to quickly find what's the problem in here – perhaps you can do better...


yeah the content is basically the same just changed a few requirements

On Thu, Jun 4, 2015 at 10:57 AM, Tariq Blecher <tariq.blecher@gmail.com>
wrote:

> oh sorry meqform.py is meant to be named meqsilhouette.py
>
> On Thu, Jun 4, 2015 at 10:55 AM, Gijs Molenaar <notifications@github.com>
> wrote:
>
>> So why do you add this meqform.py file? It looks like you committed
>> meqsilhouette.py with the same content already:
>>
>> cc935c4
>> <https://github.com/ska-sa/rodrigues/commit/cc935c4b1116b78c7abc4e64115b6c1f6eb9e01a>
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ska-sa/rodrigues/commit/b949810d09240903b1f5de401a3a4e0d72452380#commitcomment-11516819>
>> .
>>
>
>


*ТТТ*


in this line https://github.com/dmwm/CRABServer/blob/master/src/python/TaskWorker/Actions/PostJob.py#L1703 it loads G_JOB_REPORT_NAME (which in this particular case is empty file) and I need G_JOB_REPORT_NAME_NEW I can change it, and it will be ready only after RetryJob, and this function is for parse_job_report and we are talking about report to dashboard. I would like to avoid not correct functions.


Lines 103-104 are just prep work, populating the internal btrfsList variable.
Line 105 is exercising this piece of code in commands/btrfs.py, in particular lines 98-99
```
     96     def __str__(self):
     97         retval = ""
     98         for btr in self.btrfsList:
     99             retval += btr.__str__()
    100
    101         return retval
```

Then lines 106-107 is checking for duplicates in the data list:
```
    158         # Check for duplicates in the data list.
    159         if data in self.dataList():
    160             warnings.warn(_("A btrfs volume with the mountpoint %s has already been defined.") % data.label)
    161
```

Do you want me to add comments to the code?

I'm just following the coverage report and adding missing test coverage where feasible (I'm striving for 100% coverage where possible).  So far it isn't much extra work and in some cases reveals a few bugs.


fixed


I would like to keep a bit of control, not sure what will be reported with all other failures, that is why


Question: why the "..."? If the excerpt or meta_description (no words limit?) doesn't exceed the number of words allowed and have a final dot, it'll end with ".…" what's a fault. :/


Well, yes, but what do you prefer:
A) `Message-ID: <1542376457.3.1433408127618.JavaMail.Enygma@Enygma>`
or
B) `Message-ID: <018ce8f6a626fa48d66f22fa9a4b32ae/adfd50b9-cb7e-4081-b467-a3dd026239f5@xwiki>`
when you have
```
References: <018ce8f6a626fa48d66f22fa9a4b32ae@xwiki>
In-Reply-To: <018ce8f6a626fa48d66f22fa9a4b32ae@xwiki>
```
?

I consider option B is clearer (as it contains the "conversationID" in the message-ID), which is a practice others are doing as well. Also, its mostly free in terms of performance impact, so why not do it?


clone https://github.com/Ultimaker/Cura or retrieve only  Cura/resources/settings/fdmprinter.json

git clone https://github.com/Ultimaker/Cura.git

or see http://stackoverflow.com/questions/2466735/how-to-checkout-only-one-file-from-git-repository


if (mtu)?


well this change came from trying to have a better UX experience, if you follow the instructions in the README you will do:
edx-dl .... --course-list

and then you choose the course which for 99% implies typing Up Arrow to have the same expression
and then you copy the URL(s) you want to download
edx-dl .... --course-list URL

in the previous version this does not download the course anyhow.

I mean you are probably right, but there was an intention in this change, I can roll it back in a second if you want. What do you think ?

About the naming yes, it maybe comes like this for coherence with sections, but I will change them in the next PR.

About the naming, yes, it is not the best one, gv


wrong indent


same problems as above


hmm so this doesn't work with 4mb buffer? I think we need to look in to the draws sent to GSDX, fix the cause not the symptoms ;)


single quotes


I can't run the Sample applications: 'Dash-IF'.

The Error message:
TypeError: Cannot set property 'laURL' of undefined
    at ob (angular.js:10673)
    at D.assign (angular.js:16324)
    at $setViewValue (angular.js:19372)
    at angular.js:2426
    at h.$eval (angular.js:12746)
    at h.$apply (angular.js:12612)
    at HTMLInputElement.l (angular.js:2426)
    at HTMLInputElement.jQuery.event.dispatch (jquery-1.10.2.js:5095)
    at HTMLInputElement.elemData.handle (jquery-1.10.2.js:4766)

Maybe it's my fault, please help me to check it.
Thank you for all your assistance.


excessive space


add


i'm not going to accept this pull request. Please open up a new one with the changes only.


the thing to fix is that the headers are empty (see line 33), so there is no session and the get_available_subtitles function will default to ['en'] only. For this we need to login the user, and btw, how do we intend to do this ? are we going to have a test user, and if so how are we going to hide its credentials from the tests ?


k

On Thu, Jun 4, 2015 at 11:05 AM, Gijs Molenaar <notifications@github.com>
wrote:

> i'm not going to accept this pull request. Please open up a new one with
> the changes only.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ska-sa/rodrigues/commit/b949810d09240903b1f5de401a3a4e0d72452380#commitcomment-11516994>
> .
>


listner?


remove parentheses


I have contacted the edx-dl about the possible abuse of the script requests, and in principle they find 20 req/s to be a bit too much, but I am waiting to have a clarification about a good rate-limit to put it in place.


why this if


Hi there,

Is it possible to tag versions on the GitHub [releases page](https://github.com/smithlabcode/preseq/releases)? It's a minor thing, but it makes it easier to be consistent when updating installations.

Cheers,

Phil


add new line


space


The thing is that the download uses for the moment the 'urlretrieve' function of python, so no retries, and no progress indicator, and additionally it has like a low timeout, so I didn't consider it proper to release it to the public like this.


, 10


btw. I have to be honest I hate all the downloaders options of coursera, but I am seeing that we are going to end up with those here too.


@wkeese yes that one is never called automatically and definitely not once per item. I think the use-case is, one edit a "renderitem" and want to get back an item from it like to save it back to the server. The will definitely not happen in a loop on all the items, so I this is not very perforrmance sensitive.


@moollaza added the trigger and test too.


Putting `use Exception;` at the top of the file will avoid having to put the explicit "\" root namespace throughout the code. It also makes it easier to see, in one place, all the namespaces the script hits.


PS2 draw primitive by primitive. GSdx will pack primitive as much as possible. If there is no texture change or hw state change, it would use an infinite vbo.

When you see the native rendering there is nice fog?/particles? effect. This effect doesn't need any texture, it is a fixed color. Therefore it doesn't consume the GS mem, neither flush the current vbo.

So it is possible that 4MB is not enough to store all the vertices (32B each).


See how this is done elsewhere. You don't add it to each parameter


ok you can merge now

On Thu, Jun 4, 2015 at 11:06 AM, Tariq Blecher <tariq.blecher@gmail.com>
wrote:

> k
>
> On Thu, Jun 4, 2015 at 11:05 AM, Gijs Molenaar <notifications@github.com>
> wrote:
>
>> i'm not going to accept this pull request. Please open up a new one with
>> the changes only.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ska-sa/rodrigues/commit/b949810d09240903b1f5de401a3a4e0d72452380#commitcomment-11516994>
>> .
>>
>
>


remove `typically`


??? we can't get it to work on Macs? We lose half our users then


, 10

and why the validation differs with 'nigtag.js'?


`href` ?


Why do people hate sourceforge?


*facedesks* ummmmmmmmmmmmmm


Comments in https://github.com/mysociety/whatdotheyknow-theme/commit/798daf0


Good second effort!  Here are some comments:

- The `MEMORY_BASIC_INFORMATION mbi;` is also a variable decl, so move it up to the other variables.
- Assignments `=` need their surrounding spaces, on line 67
- You've changed and removed the if()-check for the return value of `dladdr()`, why?  You cannot rely on dladd() setting the `&info` when it fails, i.e., you cannot only check `dli_ifname`.


Comments in https://github.com/mysociety/whatdotheyknow-theme/commit/4cba934


Can't seem to scroll the content of the ABOUT page any more. Is this commit the cause?


:+1: :smile:


`if [ $1 = 0 ]` looks like an assignment, shouldn't it be == ?


Is there any reason we limit it to a small amount? is there a performance impact?


Rg


@kaustavha http://helb.github.io/goodbye-sourceforge/


Thank you, it's corrected :)


@kaustavha

Because of that http://www.net-security.org/secworld.php?id=18460 and that http://seclists.org/nmap-dev/2015/q2/194


I think you should use System.getProperty("java.io.tmpdir") instead. It will works fine in any case. Or you can even use File.createTempFile(String prefix, String suffix). No need to worry about the temp folder with this method.


Здесь - правильно


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


`an` -> `a`


Va? 😀

torsdag 4 juni 2015 skrev Sven Dahlstrand <notifications@github.com>:

> [image: :+1:] [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/axelebba/axelebba.github.io/commit/a01ad71ccbfca724ffdd6362a5b1657838b598f1#commitcomment-11517200>
> .
>

--
---

sannalund.se


使用cjktty 编译内核需要哪些选项，我编译完成了,结果是终端显示乱码？


Please rename the 'angular' parameter to 'require_angular' (or something else more descriptive) and make it optional with default value True.


Here we should exit, no need to go further. I think that throwing an exception with a relevant message is the best option.


А здесь не правильно. В данном случае вы не создаёте никакой новый stream, поэтому париться насчёт его корректного закрытия не слудет. Реализация HttpServer сама позаботиться о том, как быть с потоками в request/response объектах.
Осталось ещё 1 место, где это нужно сделать :)


I believe all the tests are failing during the maven build, if using the code in "Master" branch. I also skipped the tests during the build. But the code in "Release" branch builds without error including the tests


We must check if a stream is not null before trying to close it.


:+1:  Boss


How's the system going on with those memcg commits? @mkaluza


is -> has


I know that Super-Circle and Sub-Circle haven't been defined yet at this stage, but nor has been the notion of appointing a Rep Link from a circle to another. Also, since the constitution is most often not read from top to bottom but used as reference, it seems to make everything simpler to spell out Sub-Circle here.


Wrong class name


Jag gillar ju din commit, bra jobbat! ;)


Perfect :)


In fact this change will probably land in Chat 1.2


finally, RIP sourceforge


yes. (but that' s what this should have fixed)


typo in commit message: Similarty -> Similarity


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


Yes ... it would be nicer with smooth animation like when panels collapse/expand.


Sf done


Haha ok! Jag vet!

torsdag 4 juni 2015 skrev Sven Dahlstrand <notifications@github.com>:

> Jag gillar ju din commit, bra jobbat! ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/axelebba/axelebba.github.io/commit/a01ad71ccbfca724ffdd6362a5b1657838b598f1#commitcomment-11517361>
> .
>

--
---

sannalund.se


corrected in local version

2015-06-04 11:27 GMT+02:00 Makman2 <notifications@github.com>:

> typo in commit message: Similarty -> Similarity
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/15c0b4113cd3554792b4abaa7b5b25dad2ea82ae#commitcomment-11517374>
> .
>


Привеольно, но я ожидал увидеть коммит за авторством @Sacha111


результат выборки отличается от
"select name from users join messages on users.id=messages.user_id group by user_id having count(user_id) > 3"


That was more wishful coding, it doesn't work.


# sqlite-tools-android
sqlit-tools是对android进行sqlit操作的一个封装类。目前版本为V0.1,此版本提供如下功能.

1. 通过书写实体类，在实体类上就注解，程序生成表结构.

2. 通过书写sql语句，返回map或list结构表示单条记录或多条记录.

sqlit-tools-android V0.1不能做的事：

1. 不提供多表关系与实体类进行映射的功能，sqlit-tools并不是一个orm框架.

2. 不提供修改表结构功能，如果想要实现，只能自行书写ddl语句.

3. 不提供将查询结果直接转换成实体类的功能.

代码简述:

包说明:

annotation:注解包
    |--Colum:列注解定义
    |--Table:表注解定义
    |--SqliteDataType:列数据类型的常量类
engine:执行引擎
    |--DBHelper:工具类,提供建表,dml,dql操作方法.

test:测试包
MyActivity:demo
说明:大家导入到自己工程里时,只需要annotation和engine即可.其它的类可以不要.注意,该工具只能在android下运行,它不能够在纯javase下使用.


What is the point in this commit? As the AppVersion.java file is identical, why do you need to include another copy?

Please try to improve your commit message (use `git commit --amend` or `git rebase -i`)


I can't reproduce this - what browser are you seeing the overlap in?


Since `i` is iterator for `depends`, can we use it as a key for `initDepends` ? Even in yours examples:
~~~js
$("#child-2").depdrop({
    depends: ['child-1'], // dependent only on child-1
    initDepends: ['parent-1', 'child-1'], // initial ajax loading will be fired first for parent-1 and then child-1
    initialize: true,
    url: '/path/to/child_2_list'
});
~~~

So `initDepends[i]` will be undefined in cases where `depends` is longer than `initDepends`. Otherwise, if `depends` is shorter, it will skip some ids from `initDepends`. Just checked it.


@KarishmaBagga, agreed with both comments above. Fixed at:
https://github.com/MSOpenTech/dash.js/commit/87e2ab38e113bc1d7908db36bf6171bde10bb581


can we have
```
const (
...
)
```


It got infected by the bat virus that's going around every git I commit to.


The debugging and correction of the algo comes later.


@mosra What do you think of these `reinterpret_cast<float*>` uses? `Quaternion` doesn't have a `from(const T *data)` btw, which was the original reason why I am casting directly.

I guess casting directly to the type (`Vector3`/`Quaternion`) would be slightly more readable, right? Or would you recommend a small conversion function?


It's about time that SF ends. :+1:


Nor me – strange!


TeamCity YaccConstructor :: YC.Core General Build [Build 426](http://teamcity.codebetter.com/viewLog.html?buildId=192692&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 136 Build time: 00:14:04


## This is a markdown file


`if address := config["fluentd-address"]; address  != "" {` without the assignement below


###this is tertiary


issues: is `[is=tree-view-file]` the proper way to select this, or can 'tree-view-file' be used? should selectors even be used, or some other feature (i saw some docs mentioning "services")

I also use `.active` as a hack, which assumes that the first click activates the view. is this adequate? again, services?


May be you commit messahe should explicitly says that you replaced `assert` with `chai`. Why `chai` ?


:+1: Awesome


Duplicata


This certainly wasn't my preferred option.

I filed a copyright claim since the developer didn't provide their source code. So google removed their application. But unfortunately that doesn't free up the application id. Google are very clear that you do not have any right to a specific ID, and recommend that you claim your one early. I should have done so years ago :(

Anyway, what's done is done. The play store version has users using the new ID, so I cannot expect them to switch.


The class ContactController has 11 methods. Consider refactoring ContactController to keep number of methods under 10.


```
if !strings.Contains(err.Error(), "missing port in address") {
    return "", 0, "", err
}
host = h
```


Replace fun_status with ret = -1
Add the out section
Initialize all the variables, for example:
int index = 0;
r_dentry *ptr = NULL;


This commit also involved this dead lock issue:

frameworks/wilhelm/src/android/AudioPlayer_to_android.cpp:779: pthread 0x767f5460 (tid 31992) sees object 0x76d19188 was locked by pthread 0x5c3a4550 (tid 30444) at frameworks/wilhelm/src/itf/IObject.c:411


Is 20 a bit small number? Since libsocialcache deletes existing events on each update and we don't go beyond the first 20 by paging, the db contains only 20 most recent retweets. If the next round has a retweet for older tweet than what we have in the db, it will be handled as a new one and delta will be wrong (probably not  very common situation for an average Twitter user even with 20, but there are some heavy duty tweeters).


Great! SourceForge have made some incredibly bad decisions as of recently, it is about time projects started leaving them.


Original: strptime(c(powerconsump2days$date, ...), "%Y-,,,%S") throws error message as below,
Error in `$<-.data.frame`(`*tmp*`, "date.time", value = list(sec = c(NA_real_,  :
  replacement has 5760 rows, data has 2880
Use paste instead of C(): converts its arguments to character strings, and concatenates them.


Sorry to hear that. People should make their own business sometimes.


`err` :smile:


oublié le -group, fixé.


one line like:
```
if err := tmpl.Execute(buf, receiver); err != nil {
    return "", 0, "", err
}
```


@KozhinM Do we need an else here? Looks good overall!


It wasn't really a *problem*, just a tip I picked up from other projects :-)


Same for me, https://ci.appveyor.com/project/andreabedini/python-blosc I have no clue what's going wrong.


Swipeでのdiff確認はすごいなぁ


remove


fixes #19


Yes.  We can make it larger.  What do you think is a good number?  Max supported by the API is 100.
One other issue is: currently, I set "trim_user" to false, because otherwise we don't know the "self-user" screen name (and hence, cannot generate the link to the tweet, which is of form twitter.com/screenname/tweetId).  If we knew the self user screen name, we could trim_user and save a lot of data / processing.  Any ideas?


`package visualparadigm` is a bit strange. The code is authored by you? Could you add a [ASL Licence Header](http://www.apache.org/licenses/LICENSE-2.0.html) to each `.java` file?


can `fluentd` use both tcp and udp as input? if so, should we support this? if so again, the address parsing above could be similar to https://github.com/docker/docker/blob/master/daemon/logger/syslog/syslog.go#L76


this _auto_funcs file is intended for testing purpose. There are several dummy names, like `calc_projection` (LOL).


The array bplikeTerms probably should not have a plugin setting stored within it.


Doesn't LGTM, but okay :crying_cat_face:


LGTM


This is a VERY bad hack. How can we get rid of it?


Also this commit broke your because the `test.js` was still depending on the `assert` module. Because the commit is about changing the test assertion library, the `test.js` should also have been changed.


@TKruzze nice to hear you are happy with application of the GPL to the source.
I wonder, would you like to include a license in your upcoming TK releases? That way it would be more practical to be able to backport your improvements to the Open GApps packages too.


I'd considered removing the bp_like_get_text() function, as it adds an extra later of complexity - one which isnt really the WordPress way either... But the function doesnt complicate things that much, and it is pretty useful to some users. I'll sit on this one for a while..


The modifications let playlist files in de music database be treated as normal music files. They can be distinguished from music files by a different icon (sonata-play icon). When a playlist is chosen by a user (for example by double clicking) the playlist is loaden by/to the mpd. You can fill the playlist file with mp3 files or stream files (beginning with http:// )


ok