As far as I see, we need this dependency only for ParsingResource.java file, right? However, even if we move the file to test/resources Javadoc generation tries to load the file and throws an error, so let's leave the dependency in the pom.xml file. As the ApiFest Doclet is a tool used to generate mappings (docs) files, it is not a constantly running service, it's not a problem. Just change the scope of the dependency to test.


@Crupuk this needs to be indented


@jamestiffen I agree. This solution has only been presented to make it easy to get things going. Once we have a proper strategy in place and we know the names of all the required entry points. We will then firm up the security here to lock this down.


Wrong usage of bool. You are the author of the function but (censored). Reverted.


Missing trailing space. You are breaking things. Reverted.


@nigelatwell care to expand on specific security issues please?


What happens if you also define the method
	virtual const WCHAR* GetFilterDescription() const = 0;

You can use that to validate whether the previous method succeeded.  It should return a string containing name of the selected codec description.

Otherwise it looks correct to me..Perhaps it's something to do with the string format (WCHAR vs UTF-8 or something)?

That's all I have now until I look into it in more detail.


TeamCity ChameleonForms :: Continuous Integration [Build 246](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3614&buildTypeId=bt20) is now running


Please, add Javadoc to the class (in that case Javadoc at the class level only is enough).


I think I decided for NULL or 0 (no modification given) according to this description in the mzTab specification: "If protein level modifications are not reported, a “null” MUST be used. If protein level modifications are reported but not present on a given protein, a “0” MUST be reported."


I am from Turkey


ı am 17


ı am a city fan


Well, I didn't look at the code because the vast amount of changes, that's
why I asked.

On Tue, Jun 16, 2015 at 9:00 AM, Ivan Zaera <notifications@github.com>
wrote:

> I'm not sure we want to merge this. It breaks compatibility with the
> previous versions and given the lack of resources (namely time and people)
> I cannot guarantee that it will do more good than harm.
>
> The main idea was to provide a consistent initialization for algorithms so
> that given the name and the initialization parameters we could unambigously
> know what the cipher will be doing.
>
> Also it makes cipher initial states read-only so that whenever you reset
> an algorithm it returns to a well known repeatable state and also it makes
> (de)serialization of algorithms possible.
>
> But as I said, it is risky to merge it now and I don't see enough benefit
> given the current state of cipher (library and users I mean).
>
> Nevertheless if you think it makes sense, we can try to merge it...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/izaera/cipher/commit/b63be4ec7e8897f11e96b933467f448430ece9a8#commitcomment-11695750>
> .
>


Currently there are none. But if ever needed to lock anything down we can do so.


if If data is stored on "bpy.types.Scene" - this data will be stored in the scene. Always. You can even test.
- Copy uvs, and save.
- close and reopen blender.
- try to paste uvs.
UVs will be pasted.

AddonPreference is stored in addon cache. When we press Ctrl+U - it will be saved in addon.


`$('#myDatePicker').datepicker('destroy')` actually works whereas `$('#myDatePicker').destroy()` doesn't, maybe we can mix our both suggestion until we know it is solved jquery-ui side? We are close to the release so we should be safe...
- [ ] overriding DatePicker#destroy
- [ ] surrounding calls (general & specific) with `try/catch { console.log }`.


<%= mail_to 'partner@ginius.ru' %> не нужно?


Yay!


You can do it!


That's a very good idea, redirecting at once to the edit page was confusing


Because you'd only need the token within the scope of the `AuthManagement` service, I wouldn't store it on the `$rootScope`, but rather as a private variable (just like `ref` and `auth` on lines 230-231). Feels more secure.

Or maybe you wanted to re-use the token somewhere else?


Changed in 5a1217e40193c8884155ccaf415091d326ddb52a for `master` & 1d052bdbe621ca111bab3314604dc682d900d716 for `compat`.


I don't have any problem with logging in on GarenaTH with this commit reverted.


! Note: I forgot to account for data sets which do not have any proteins with less than 3 spectra.
Probably need sth like this:
if (length(singles) >0){
pos.pep <- pos.all[-singles]  }
else pos.pep <- pos.all


Dummy commit comment to test github comment/commit APIs


ok


It had two purposes: one was to test error handling when a URI cannot be accessed, and the other was to allow me to test using one server to proxy a thing on another. This is now close to working properly, but there is a problem around identifying the full domain for this server as a means to re-use web socket connections between any pair of servers.  os.hostname() on akira.w3.org reports "akira" and not "akira.w3.org".


@VioletGiraffe May I undo this part of the change?


Sorry guys. My bad. Wrong project. I have just implemented basic CORS in the search project. I thought you were referring to that. (...pay more attention in class...pay more attention in class...pay more attention in class.....)


oops?


@Bakual Per the PR comments this line should be (updated for the current JApplicationWeb API) `$app->redirect(JRoute::('index.php?option=com_users&view=login', false), 302);`.  You've changed the redirect code.


This error was printed, when it didnt have to be printed.


I'm working on a similar feature and took a look at this code.
Isn't this query ambiguous? Isn't it possible that we actually hide different embeddable than you're planning to hide? I think that combination of `page_id` and `embeddable_id` isn't necessarily unique and we might be missing `embeddable_type` here.

I can see that we have the same line used in embeddable removal code where consequences are more serious.

Obviously we have extremely low chances of getting into troubles because of that, but I would consider fixing it anyway if it's not too hard.

Sorry if I miss sth and it's a false alarm.


querytype doesn't seem to be used anywhere, can this be removed?


:blush:


Dummy commit comment number 2 to test github comment/commit APIs


@robrix what do you think of this: https://github.com/antitypical/Result/pull/66


Already fixed.


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0+014](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3614&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3003 Build time: 0:0:0


please add the file.


hlint may suggest to drop the `l`.


Shouldn't composer be available in the PHP base image?


This is for handling the `{:error, :stale}` tuple that is created in the `Connection`. But I think we can indeed raise directly in `Connection` and get rid of this.


how does the intro section from proxy reach the a3 api docs now?


Argh. Sorry.


@Syphurith rebasing an existing database could be insufficient. segments should be correctly created if you load the file into IDA from scratch (specify the correct load address at the loading dialog).


Disse variable kan ikke være `static`.

Hvis du oppdaterer testen `test_it_handles_multiple_objects()` til å teste noe mer en antall patcher, så vil du se at patchene er identiske..


We test for `app/Console/cake`, then link `${HOME}/app/Console/cake` ... this breaks when the current working dir != $HOME


Copied from here: https://developer.mozilla.org/en-US/docs/Web/Guide/User_experience/Using_the_Page_Visibility_API


It's true it should, granted the example from the documentation. I'll play with the code asap, trying to figure it out.


Yes.

The count is not used. Maybe returning the count is misleading.


what is that ??? a tabulation ?


Oh, I see :)


Minor comment on style: for consistency, write white space between the closing parenthesis and const. Sorry for being annoying :-)


BTW it's weird to be manually symlinking to get "cake" in the PATH... and this also assumes that `${HOME}/bin` is in the PATH...


I'd rather remove the installed files than the entire directories. One never knows what the user put in there …


We could auto generate this using a template parameter.


That's a lot of duplication :)


Indent properly, and I would leave out the braces for control flow blocks where all branches are one-liners.


The `HasDocs` instance in`Thentos.Backend.Api.Docs` will add the intro section to all APIs that have `ServiceProxy` as the fallthrough route. Or maybe I'm misunderstanding your question?


What's this for?


In my opinion we should be less aggressive with unghosting from dialogs, using the  Unghost threshold control and doing it after some time, not at every page change. Then we should remember to reset Unghost everytime we call MainWindow Unghost function.


Всегда было интересно, какая политика насчет .gitiginore. Допустим, я использую какой-то тулз, который создает левые директории, а ты им не пользуешься. Я добавляю эти директории в .gitignore, а ты не понимаешь о чем вообще речь, пока я тебе про этот тулз не рассказал. Все так и живут?

venv тогда тоже стоит туда добавить?


`all` is the default target, hence `make` would be enough. But that's just a minor nitpick.


Rack mountable PDU and UPS gear use this, I've got an off brand PDU right here which does this. The other protocol it supports is proprietary.


`.git/info/exclude`
http://stackoverflow.com/questions/1753070/git-ignore-files-only-locally


pProperties is no longer used in this scope after these changes are applied, please remove if superfluous.


:heart:


I think this is why the autosuggestions are enabled by default, instead of loading the values from localstorage?


It looks weird.


Sure. That’s why I replied :-P.

2015-06-16 14:42 GMT+02:00 Steven Roose <notifications@github.com>:

> Well, I didn't look at the code because the vast amount of changes, that's
> why I asked.
>
> On Tue, Jun 16, 2015 at 9:00 AM, Ivan Zaera <notifications@github.com>
> wrote:
>
> > I'm not sure we want to merge this. It breaks compatibility with the
> > previous versions and given the lack of resources (namely time and
> people)
> > I cannot guarantee that it will do more good than harm.
> >
> > The main idea was to provide a consistent initialization for algorithms
> so
> > that given the name and the initialization parameters we could
> unambigously
> > know what the cipher will be doing.
> >
> > Also it makes cipher initial states read-only so that whenever you reset
> > an algorithm it returns to a well known repeatable state and also it
> makes
> > (de)serialization of algorithms possible.
> >
> > But as I said, it is risky to merge it now and I don't see enough benefit
> > given the current state of cipher (library and users I mean).
> >
> > Nevertheless if you think it makes sense, we can try to merge it...
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/izaera/cipher/commit/b63be4ec7e8897f11e96b933467f448430ece9a8#commitcomment-11695750
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/izaera/cipher/commit/b63be4ec7e8897f11e96b933467f448430ece9a8#commitcomment-11701656>
> .
>


forgot tbh


Ok, it's documented above as:
```
  * - 'php' - Uses settings defined in your php.ini.
  * - 'cake' - Saves session files in CakePHP's /tmp directory.
  * - 'database' - Uses CakePHP's database sessions.
  * - 'cache' - Use the Cache class to save sessions.
```

Why are we moving away from php.ini to use tmp dir?
If we save sessions in a tmp dir, then the pod with this app is not scalable → each replica get it's own session storage...


Well I couldn't get it to compile otherwise, even after doing setupDecompWorkspace.

> On Jun 16, 2015, at 1:16 AM, Chris Sanders <notifications@github.com> wrote:
>
> This was copied over from when it WAS in SpongeCommon. .channel() is a Forge add method.
>
> I could remove the at and go back to using channel() but yeah.
>
> —
> Reply to this email directly or view it on GitHub.
>


The driver does support passing keyword instead of a map. We can convert this.


we do not expect linear correlations for all features (already known for absolute intensity - error correlations), thus we chose Spearman to provide a measure of correlation. Generally I think correlation plots are more informative for some features than the correlation values.


Only that it adds a moving part to the mechanism. It would need to be imported and isn't currently. Could add it.


I didn't notice it at first, but actually it's quite simple. The `then` function expects a callback (function) *definition* as its argument, not an actual *call*. The callback will be called for you, if it makes sense at some point.

You could keep your `broadCastEvent` helper function, while still wrapping it an anonymous function, but jshint may not like that (function definition within a loop). The recommended way for fixing that is to use a callback generator. Here's an example: https://jslinterrors.com/dont-make-functions-within-a-loop (btw jslinterrors.com is a very good resource).


Sadly, that's true; The only way I found that wouldn't make it look strange is a way that causes crazy glitches with the bots' ai. I'll work on it today, but for now, there it stands.


You're supposed to test Network calls and JSON parsing of the HTTPS Response.


>venv тогда тоже стоит туда добавить?
Холливарный на самом деле вопрос. Некторые делают так, что venv это как бы часть проекта: т.е. приложение знает, где именно лежит питон. В таком слуаче да, venv добавлять надо. С другой стороны, можно сказать, что приложению пофигу, venv или не venv. Те, например, на сервере системный питон, а локально venv. В таком случае надо добавлять в `/info/exclude`,  если у тебя venv лежит около проекта.

Я пользуюсь pyenv (https://github.com/yyuu/pyenv), так что у меня питоны лежат всегда отдельно от проектов.


Thanks


Does it make sense to do that? The base image referenced by the Dockerfile isn't real, so it would fail if we didn't mock docker.


it's first run, for sure it can be optimized, before change, leave as is for some time and wait for user feedback.

UnGhost() function si trigger, no need to remember anything.


This change increases memory usage quite a lot. Because the max number of channels is 2^16, this line creates a list of 65536 ints which on my 64-bit python3.4 consumes 2.3 MB of memory:
``self.channels_ids = list(range(1, amqp_constants.MAX_CHANNELS + 1))``

It's replicated for every channel, so a server running many connections and channels can easily use tens or hundreds of megabytes just for keeping track of free channel ids. Which will cause more problems than running into the channel id limit for the vast majority of usage patterns out there.

I'd recommend reverting to the old logic of incrementing the next channel id by 1, and when hitting MAX_CHANNELS scanning the connection's used channels for free identifiers.

Some more efficient algorithms must exist, because they're needed for allocating PIDs, file descriptors, port numbers etc. by operating systems.


This looks wrong. Why would setting `DEBUG` *remove* `--no-cache`?


Ok, makes sense. Better save than sorry.


Good point. I'll fix it...


@Toxaris Could you explain this? I have some intuition about what flatmap means for parsers, but I don't understand what the `<|> asum` part is doing.


En noe sirkular anmodning om å ikke gjøre dette?


今天测试中发现，如果一个文件用户没有关闭，就删除了。
由于一直没有调用MarkBlockStable，所以这个块一直不会被实际删除，导致磁盘空间用尽。
这里我建议不要在nameserver额外处理，直接将块删除，回收资源。
那么用户后面的写和close都失败，用户自己去处理这个失败。


Please add a `launch_dns` helper function. Having to add `$WEAVEDNS_ARGS` to every single dns launch is tedious and error prone.


I got it working... thanks

On Tue, Jun 16, 2015 at 9:25 AM, ilausuch <notifications@github.com> wrote:

> It still working for me. What is your request?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ilausuch/ArrestDB/commit/2fd016607043358b49ec0a6436799991ce644cba#commitcomment-11701361>
> .
>


Shouldn't the patch be making it so that HAVE_NETINET_IF_ETHER isn't defined?


Any particular reason you are using x16 instead of x9 .. x15 that don't have special semantics? I know that it probably won't matter in this particular case, but you should typically avoid x16/x17 in non-leaf functions, since they may be clobbered by a function call if the linker decides to jump through a PLT or veneer.


@livingincode DEMR. Looks good. Good work.


OK, thanks for your great explanation!!

By the way, why don't you move global variable to cpuv_properties?
If you use this value, you wrote this line `from . import cpuv_properties`.
IMO, global variable should put together one module(namespace).


Hi @joostkremers

It seems the closing paren is misplaced here. So if `pandoc-use-async` is `t`, pandoc will be called twice: asynchronously then synchronously.


Yes this is supposed to be on a branch and not on the the Soomla Master branch


Meant to say DWMR.


OK, I'm going to fix and rebase.


This is usually something like:

```python

```


Of course, i am happy to do this.


OK. I agree. Removing and rebase.


Hm, no I don't. From the latest version of tdigest:
```
from tdigest import TDigest
t = TDigest(0.01, 5000)
t.batch_update([0,1,2,3,4,5,6,7,8,9])
t.percentile(0) #returns 0.0
```


Unfortunately I did not have this information available for my data sets, would be definitely interesting!


remove blank space


Hum...we also have 0.6.2 in production, but I also see that we had version 0.6.0 at one point.
```
$ ./fits.sh -v
0.6.2
````


Oh, I didn't notice this.
I will also refactor another feature.

> It would also be good to paste edge seams.
Maybe, this will be one of the next subject about this plugin.


@albertdev,
With this change we always switch back to the NORMAL mode as opposed to the last mode. This broke the temporary command line mode (`A-O`) which has the convenience of a non-braking undo history (as opposed to `C-O:`).
Do you remember the change was made in such way?


In my opinion, our tests target a specific converter (e.g. ProvToolbox's provconvert, Prov Python's prov-convert), not comparators. The comparators are provided as utilities for the test framework to check for equivalence between files of various formats. A comparator for XML or RDF, for example, may not even need to understand PROV concepts.


Done with the tests!


Here, the availability zone is going to come back as `us-east-1a`, but the region is what we want:  `us-east-1`. We need to add some code here to trim the trailing letter.


Yes sure.


This image looks fine in all other DICOM Viewers I tried. The problem is
with Cornerstone viewer only. But it was OK with earlier version of J2K
library. I don't know exactly where is the problem, it could be with
Cornerstone also.

On Tue, Jun 16, 2015 at 5:59 PM, Jean Francois Pambrun <
notifications@github.com> wrote:

> This doesn't look like compression artifacts. With the new commit, the
> codec returns a unsigned array when the codestream is unsigned. Are you
> doing any arithmetic that could cause overflows or underflows? I don't
> remember how the VOI transform is implemented, maybe the issue is there?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11701413>
> .
>


The build hangs if the input json has no template['spec']['output']['to'].


Here are some more details to help debugging:
```
In [4]: t
Out[4]: <T-Digest: n=10, centroids=10>

In [5]: t.C
Out[5]: FastRBTree({0.0: <Centroid: mean=0.00000000, count=1>, 1.0: <Centroid: mean=1.00000000, count=1>, 2.0: <Centroid: mean=2.00000000, count=1>, 3.0: <Centroid: mean=3.00000000, count=1>, 4.0: <Centroid: mean=4.00000000, count=1>, 5.0: <Centroid: mean=5.00000000, count=1>, 6.0: <Centroid: mean=6.00000000, count=1>, 7.0: <Centroid: mean=7.00000000, count=1>, 8.0: <Centroid: mean=8.00000000, count=1>, 9.0: <Centroid: mean=9.00000000, count=1>})

In [6]: t.percentile(0.5)
Out[6]: 4.5
```


We shouldn't need to use different environment variables for different database backends (`MYSQL_SERVICE_HOST`, `DATABASE_SERVICE_HOST`)

And we should parameterize the "DATABASE" service name, so it should be something like `getenv(getenv("DATABASE_SERVICE_NAME")."_SERVICE_HOST")` (or whatever is the proper PHP code here).


What is this for?


I was able to understand what this explanation means only by reading the code below.  Please try to explain the special cases in more detail.


I'm not too sure about the difference between 2 and 3. Can you give an example?


Yes, this was a bug. Thanks.


Am I reading this wrong, or there is a missing else branch (for the second `checkAccess` method call) in the above if for the case when `right == Right.PROGRAM`? Otherwise, if `right == Right.PROGRAM`, there are 2 identical `checkAccess` calls.

In other words, the code should be:
```
if (right == Right.PROGRAM) {
  checkAccess(right, getCurrentUser(right, null), null);
} else {
  checkAccess(right, getCurrentEntity());
}
```

Or better yet (considering the other changes in this commit):
```
if (right == Right.PROGRAM) {
  checkAccess(right, null);
} else {
  checkAccess(right, getCurrentEntity());
}
```
The second version is basically the same as the
```
checkAccess(right, right == Right.PROGRAM ? null : getCurrentEntity());
```
 line.


This seems like a strange place to be aliasing jQuery. Can you explain why this needs to be done here?


//hello world

printf("hello world!");


Or simply `_payload.status < 200`.


I am not quite sure if I understand your question right. So far,  input of iPQF should be absolute ions/spectra intensities and can then be transformed to relative ion intensities by the 'ratio calculation' function ( I have not found any option for relative intensitiy calculation in MSnbase) in order to estimate corresponding protein relative intensities/ ratios.


Yeah, I had just added a duplicate of that in the earlier commit :)


Can you describe why the parameter is necessary on each pageview call?


@sfrisk We should probably try to use the commit message style guide.


Why won't you keep the new codebase on Github available for all these great people who are trying to contribute? It is counter-productive.


So awesome.

![image](https://cloud.githubusercontent.com/assets/213371/8183889/a02ad5dc-1400-11e5-9143-7506e9191f3e.png)


Because with Turbolinks, the location is miscalculated by Google Analytics.


Use .isEmpty().


@bmneely DWMR. :+1: Press on.


Yes the second part should be in a else.


Naresh(925879)
Navya(926393)


Is this backward compatible with Rails 3?


Could we add a new event there and let user choose to which one to subscribe? Lets say we're monitoring a continuously-written log file which is never closed - this will fail to do it.


use constant


Hi Tor,

Thanks for getting back to this. I have removed the comment by accident, a poor mistake on my part. I will set that straight so it doesn't cause confusion on this issue.

The only change in this commit is the removal of line 37, the "word-break: break-all;" statement.
Like you and Lorenz, I am no expert on this, so the approach I tried was purely by experimenting with the statements. I'd be happy if a real CSS expert can take a look at this problem.

Because I'm no expert the only way I could find a fix for this issue was by removing the break-all statement because it causes firefox to break words between any character instead of breaking in grammatically correct places only. I think this should definitely be avoided because it hampers readability (see screenshot in the original issue).
You said you rely on break-all to make sure no horizontal scrollbars appeared. So after I removed the break-all I made sure that it didn't cause horizontal bars to appear, and as far as I can tell it doesn't. So do you really rely on break-all for not showing scroll bars?

In conclusion, removing the break-all statement causes Firefox on my machine to break words correctly and it does not cause horizonal scrollbars to appear, nor does it cause any side effects in Chrome. In my opinion the break-all statement is not required and does more harm than good.

If you disagree, that's totally ok with me and I'll remove the PR and move on to another bug :)

Thanks!

Michael


In practice, we don't loose anything and you can still `enable|disable|install` the extensions in most cases, even if the source dir was purged.


I thought I had updated the comment, not sure why it decided to not work. Bah.


Since anything < 100 isn't valid - nice!


OK.
Why don't you prepare `debug_print` function?
For example...
```
DEBUG = True

def debug_print(s):
    if DEBUG:
        print(s)
```
you may use `logging` module as an alternative idea.


I understand. Unfortunately, I can't reproduce your issue and it is very
difficult to debug without a problematic J2k code-stream. You can always
use an earlier version until the issue is fixed.

On Tue, Jun 16, 2015 at 9:14 AM idmpacs <notifications@github.com> wrote:

> This image looks fine in all other DICOM Viewers I tried. The problem is
> with Cornerstone viewer only. But it was OK with earlier version of J2K
> library. I don't know exactly where is the problem, it could be with
> Cornerstone also.
>
> On Tue, Jun 16, 2015 at 5:59 PM, Jean Francois Pambrun <
> notifications@github.com> wrote:
>
> > This doesn't look like compression artifacts. With the new commit, the
> > codec returns a unsigned array when the codestream is unsigned. Are you
> > doing any arithmetic that could cause overflows or underflows? I don't
> > remember how the VOI transform is implemented, maybe the issue is there?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11701413
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702304>
> .
>


So on initial configuration the keyboard works as it should but as soon as I exit out of a game its as if it just totally forgets that it is configured and no longer works in emulationstation or retroarch but the configurations are still there both in emulationstation and retroarch. Also when I configure it on its own through the start menu it doesn't seem to recognise anything has changed where configuring it on boot (while the keyboard is the only thing plugged in) at least works once until you exit out of a game. Its kinda weird, are you seeing the same things with your build?


I'll do...


I don't know why you'd doing this


I tested this version in flight, it seems to improve attitude control. For example, there are less roll / pitch oscillations in descent.
I'll test the latest commit of this branch, and i hope i'll be able to fly more to confirm the benefit of improving motor mixing. Anyone else did some flight test?


I'm encouraged that it looks correct

I did try GetFilterDescription but it gave me a lot of errors, but maybe that is a clue so...

[ComImport, System.Security.SuppressUnmanagedCodeSecurity,
         Guid("91E373C5-C386-457b-AF06-2ECBECCD3E12"),
         InterfaceType(ComInterfaceType.InterfaceIsIUnknown)]
        public interface IVideoForWindowsFilter
        {
            [PreserveSig]
            void InitFilterByDescription(string desc);
            [PreserveSig]
            string GetFilterDescription();
        }

....
   IVideoForWindowsFilter ivfwf = pVFMWrapper as IVideoForWindowsFilter;

   if (ivfwf != null)
   {
       ivfwf.InitFilterByDescription("Lagarith Lossless Codec");

       string s = ivfwf.GetFilterDescription(); // errors on this line
   }

....

A first chance exception of type 'System.Runtime.InteropServices.MarshalDirectiveException' occurred in mscorlib.dll
Critical error detected c0000374
First-chance exception at 0x77B35624 (ntdll.dll) in graphtest.exe: 0xC0000374: A heap has been corrupted (parameters: 0x77B51378).

Unhandled Exception: System.Runtime.InteropServices.MarshalDirectiveException: Marshaler restriction: Excessively long string.
   at System.StubHelpers.StubHelpers.CheckStringLength(UInt32 length)
   at System.StubHelpers.BSTRMarshaler.ConvertToManaged(IntPtr bstr)
   at graphtest.Form1.IVideoForWindowsFilter.GetFilterDescription()
   at graphtest.Form1.BuildGraph(IGraphBuilder pGraph, String srcFilel) in d:\Documents\Visual Studio 2013\Projects\graphtest\graphtest\Form1.cs:line 243
   at graphtest.Form1.button1_Click(Object sender, EventArgs e) in d:\Documents\Visual Studio 2013\Projects\graphtest\graphtest\Form1.cs:line 55

I'm googling the errors to see if I can get a lead. I think maybe it is the return types as you suggested or I have to Marshal something or something.


This is just a made up Guid. It doesn't even have to be a valid Guid, any truthy value will work.
I introduced a change in the manage page to only look up a poll if a manageId was given.


yes, feature names are hard-coded according to the mzTab names (provided in the PSM/Peptide section in their specification file). I am happy for suggestions to allow more flexibility on given feature names.


makes sense to me.


Yes, this simply tells Google Analytics to manually track *this* location.


Ex. If you drop your current php src dir `rm -r $PHPBREW_HOME/build/$PHPBREW_PHP` and try to install an extension `phpbrew ext install github:c9s/cssmin.git` it will be successful and `tree $PHPBREW_HOME/build/$PHPBREW_PHP -L 2` will show:

```
~/.phpbrew/build/php-<version>
`-- ext
    `-- cssmin

2 directories, 0 files


On ie10/ie11 we emulate touch using pointer events. ie11 in gwt gives you the gecko1_8 permutation which will not work in windows phone. Hence forcing ie11 to provide the same user agent string as ie10 allows the ie10 permutation to be used.

I will look at possibly removing this and adding the logic in the copy we have of the UserAgentPropertyGenerator to  return the ie10 permutation for the ie11 user agent string.

When I started the windows phone 8 work I only had to worry about ie10. With windows phone 8.1 I then had to start worrying about ie11 as well. I don't think the gecko1_8 permutation will cut it for mobile ie11 especially with all the ie specific changes I have had to add.


I assume this doesn't affect mage heads. Will T1s get headgear re-activated at some point? Most or possibly all of the races I've done have had their itemsets defined so as to account for some T1 headgear, but I can certainly do a sweep through the older stuff to make sure there's hats and bareheaded options defined for T1.


see https://github.com/phpbrew/phpbrew/issues/523


I would prefer to have a separate configuration file for each converter. For each converter, we will need to specify which representation it accepts as inputs and which it produces as output. Other than that, it is normally expected to pass all the test cases.
We might want an option to specify which test cases to skip, if there is a known issue that cannot be addressed immediately and we don't want the whole testing to fail.

For a particular test case, it might not have a valid, say, XML representation. The fact that we don't have testcase_nnn.xml in the test documents may be sufficient to skip any XML related conversion for that test case. I don't think we need to provide explicit configuration for this.

In addition, as I mentioned above, we will know which comparator to use for which representation. So it needs not to be part of a test case's specification.


I believe the API from cornerstone to the JPEG2000 decoder has not changed,
it would be good to swap back to the older one to determine if the issue is
in the JPEG2000 library or cornerstone.

On Tue, Jun 16, 2015 at 8:24 AM, Jean Francois Pambrun <
notifications@github.com> wrote:

> I understand. Unfortunately, I can't reproduce your issue and it is very
> difficult to debug without a problematic J2k code-stream. You can always
> use an earlier version until the issue is fixed.
>
>
> On Tue, Jun 16, 2015 at 9:14 AM idmpacs <notifications@github.com> wrote:
>
> > This image looks fine in all other DICOM Viewers I tried. The problem is
> > with Cornerstone viewer only. But it was OK with earlier version of J2K
> > library. I don't know exactly where is the problem, it could be with
> > Cornerstone also.
> >
> > On Tue, Jun 16, 2015 at 5:59 PM, Jean Francois Pambrun <
> > notifications@github.com> wrote:
> >
> > > This doesn't look like compression artifacts. With the new commit, the
> > > codec returns a unsigned array when the codestream is unsigned. Are you
> > > doing any arithmetic that could cause overflows or underflows? I don't
> > > remember how the VOI transform is implemented, maybe the issue is
> there?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11701413
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702304
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702555>
> .
>


@rupl Submit a PR :+1:


I have no objection to that.


this line helped me see faces in 3d. It was more visually usable at that debug.


Small optimization?: It seems `for o in other_edges` works correctly.


True. Done.


Currently it does affect mage heads. Once I rewrite MageGenerator it won't.

Now, I did not know about the need to make the mage generator support headgear for T1. Are there other things like that?

Also, the sidhe tier 2 mounted mages had the problem of not getting a mount, right? That was probably because the tier 3 mage did not have a mount and the program was too dumb to give tier 2 a mount when deriving an unit from tier 3.

I think the headgear thing can be fixed by just removing the hardcoded hat removal and adding empty hats with tags for tier 1 for races without support. Beards/hair should probably have a similar treatment instead of hardcoded stuff. The mount thing, if it is what I described, is ludicrously easy to fix as well.


I didn't even think of parametising over the fixup kind - this is 200 times neater!


Reporting "Ran 1 test..." is not ideal, but it is OK. The test framework always can output extra logging for each test case it runs.

However, I think it is important to know all the cases that fail (and for which pair(s) of representations). The knowledge may provide us clues what the issue is. Hence, stopping the whole test suite when once case fails is not OK.


why did you remove this one?


Yes. This goes back towards making it part of the callback state you mentioned earlier. good point.


How would you like the call to be triggered?


Yeah, perhaps something like a Marshal attribute?

[MarshalAs(UnmanagedType.LPWStr)]

Definitely is looking like a string conversion issue.

Andrew Griffiths
Glasgow, UK

RenderHeads Ltd
Email: andrew@renderheads.com
Phone: +44 (0)7845059318
Web: http://www.renderheads.com

On 16 June 2015 at 14:25, TimDPayne <notifications@github.com> wrote:

> I'm encouraged that it looks correct
>
> I did try GetFilterDescription but it gave me a lot of errors, but maybe
> that is a clue so...
>
> [ComImport, System.Security.SuppressUnmanagedCodeSecurity,
> Guid("91E373C5-C386-457b-AF06-2ECBECCD3E12"),
> InterfaceType(ComInterfaceType.InterfaceIsIUnknown)]
> public interface IVideoForWindowsFilter
> {
> [PreserveSig]
> void InitFilterByDescription(string desc);
> [PreserveSig]
> string GetFilterDescription();
> }
>
> ....
> IVideoForWindowsFilter ivfwf = pVFMWrapper as IVideoForWindowsFilter;
>
> if (ivfwf != null)
> {
> ivfwf.InitFilterByDescription("Lagarith Lossless Codec");
>
>    string s = ivfwf.GetFilterDescription(); // errors on this line
>
> }
>
> ....
>
> A first chance exception of type
> 'System.Runtime.InteropServices.MarshalDirectiveException' occurred in
> mscorlib.dll
> Critical error detected c0000374
> First-chance exception at 0x77B35624 (ntdll.dll) in graphtest.exe:
> 0xC0000374: A heap has been corrupted (parameters: 0x77B51378).
>
> Unhandled Exception:
> System.Runtime.InteropServices.MarshalDirectiveException: Marshaler
> restriction: Excessively long string.
> at System.StubHelpers.StubHelpers.CheckStringLength(UInt32 length)
> at System.StubHelpers.BSTRMarshaler.ConvertToManaged(IntPtr bstr)
> at graphtest.Form1.IVideoForWindowsFilter.GetFilterDescription()
> at graphtest.Form1.BuildGraph(IGraphBuilder pGraph, String srcFilel) in
> d:\Documents\Visual Studio 2013\Projects\graphtest\graphtest\Form1.cs:line
> 243
> at graphtest.Form1.button1_Click(Object sender, EventArgs e) in
> d:\Documents\Visual Studio 2013\Projects\graphtest\graphtest\Form1.cs:line
> 55
>
> I'm googling the errors to see if I can get a lead. I think maybe it is
> the return types as you suggested or I have to Marshal something or
> something.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/RenderHeads/DirectShow-vfwFilter/commit/a6df0965cd5b339b4e8bc44c29d0d950949fc1f1#commitcomment-11702584>
> .
>


The content on that page says "you can find how to configure your DNT settings by following one of these links:". I don't see how that helps to configure browser, especially for Gravatar. For me this doesn't clarify if DNT works for Gravatar, and clarification better be done in separate paragraph.


Nice!


I think the conversion to long should be done by the API automatically. Does this fail if you don't do the long?


it will not be cool as i remove items from the "other_edges" list.
L311


Since ManagedCefBrowserAdapter is a ref class I don't think this is strictly necessary, but is good C++ style since it might be required for the native non-ref classes in managed C++. (or so I'm guessing)


This appears to be enforced in IdentityConfig.cs
```
manager.UserValidator = new UserValidator<ApplicationUser>(manager)
 {
    AllowOnlyAlphanumericUserNames = false,
    RequireUniqueEmail = true
};
```


Is this rename a good idea? `pred` and `succ` from `Prelude` are quite standard names.


oh, sorry.
I'm looking only print statement.
Then, why don't you write L257 as ...
```py
if DEBUG:
    shared_face.select = True
```


whoops... another blond moment there... thanks for catching this!


editor config?


It almost certainly does overestimate `n_calls`. I'd love to do a better job of this, but I haven't come up with a better solution yet. Basically, this makes `n_calls` a conservative estimate (and the number is quite random in practice) -- but the effect in my testing is that the final timing numbers are very consistent.

I'm going to explore doing a local window regression, which should start to address the issue you're raising.


Maybe we should follow the Parsec naming conventions and use [`anyChar`](http://hackage.haskell.org/package/parsec/docs/Text-Parsec-Char.html#v:anyChar) and [`satisfy`](hackage.haskell.org/package/parsec/docs/Text-Parsec-Char.html#v:satisfy) to avoid name clashes with the predefined `any` and `pred` functions?


Unify naming: Checksum instead of CheckSum


Shouldn't this be `NHTM.methodPut`?


@lellky Tjena! repot är publikt - du får unwatcha det själv. jag kan inte ta bort på min sida. Ha en fin sommar!


Oops! Thanks for letting me know. Should be fixed now in the latest commit.


@Arthur2e5 did you really? I am trying to fix those problems now.


OK, I will use x9 instead.


i did not use real debug before. i guess this will be ok.


Once connection is established the local port is always assigned randomly from availablepool: net.ipv4.ip_local_port_range.
Assert with 0 is failing if the connection is established.


```
var categoria = db.Categorias.Find(cat);
pontuacao.Categoria = categoria;
```


* meaningless $csv and $aiv names
* missing class properties for above variables (please do it after changing names to something more useful)


This change appears good but unrelated.


@todo - create new domain exception instead of generic exception


I'm not sure - there might not be a good way to do it, so I wanted the example to see how ugly it would be


@todo - domain exception


right, thx


I want to use my own translation and this way it's easier to maintain


to modify 播音


For our purpose, we accept provided comparators as authoritative (and correct).

They may well need their test cases, but this is out of scope.


`anyChar` in our case would accept any token, not only chars. I just renamed `mfilter` to `pred` to be more reminiscent of semantic predicates. Maybe we could also have a binary operator `p ? \t -> ...` instead. Is this valid Haskell?


This needs to check against `'test'`.


@mnin the latest release of rack-track actually has a compatibility issue with rails 3, it's discussed here https://github.com/railslove/rack-tracker/pull/41
I added a pull request to have travis test against multiple Gemfiles https://github.com/railslove/rack-tracker/pull/43 so that we notice such compatibility issues in the future. But it doesn't make sense to merge it before #41 resolve the rails 3 compatibility issue.


I tested the last code from git repository, and I continue with the same problem, the feeds from vimeo don't show anything.


#pasfan


I'm not really familiar with `C++` templates, would it be possible to template some of the boiler plate stuff for `IFrame` and `IBrowser`? Then we could use it for all the callbacks.


What does "in the future" mean? After consuming at least one token? But then `pure 42` should be dead, even though it is not bisimular to `empty`.


Det lages en serialVersionUID automagisk, som er mer enn god nok for vårt formål. Dersom du tar over det ansvaret selv, ved å deklarere en, så har du ansvaret for å oppdatere den når den trenger oppdateres (når et Java-serialisert objekt av denne klassen ikke lenger lar seg serialisere likt). Det er noe du slett ikke har lyst til å forsøke deg på.


when there is a single quote in a string ```'```, please use a double quote around ```"``` so that the .po file will be without ```\'```.

Should be
```php
    $this->Session->setFlash(__("ShortCode couldn't be archived."));
```


yes, that's right. Feel free to cherrypick to master :)


What is the intended meaning of  `True` and `False` here? I expect this property to be undecidable (for the `>>=` and `pred` combinators), so the obvious meaning `True = alive` and `False = not alive` cannot work.


I exect `False` here, see above.


I just don't know how to get it to show the content of a linked table?? I
get only the ID value.

"Farm_Baked_Goods_baked_goods_EN": "~7~9~10~",

Any thoughts?

On Tue, Jun 16, 2015 at 10:09 AM, Blair Cox <wblaircox@gmail.com> wrote:

> I got it working... thanks
>
> On Tue, Jun 16, 2015 at 9:25 AM, ilausuch <notifications@github.com>
> wrote:
>
>> It still working for me. What is your request?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ilausuch/ArrestDB/commit/2fd016607043358b49ec0a6436799991ce644cba#commitcomment-11701361>
>> .
>>
>
>


Why?


Sorry my mistake. Thanks for the catch.


I just took a look around and wasn't able to find any documentation about Gravatar supporting DNT - Twitter is a supporter, but I had trouble finding a specific guide to enable it.

As for Twitter the best informational link is the one I included in the request.

So, what would be the best approach the cover all the other social media/analytic services that don't support DNT?  Here is a current list that I found: http://donottrack.us/implementations

If need be I can add a paragraph specifically addressing Gravatar.


Right. It works without explicit long() conversion as well. Fixed.


Again use double quote around this message and change the ```couldn\'t``` into ```couldn't```


If p is dead we continue with q. So this parser is only dead if q is dead


Remove constructor, it will be empty.


Yep, I saw it was changed in the PR and according to https://en.wikipedia.org/wiki/HTTP_302 I understood the 303/307 redirect codes are actually meant to replace the 302. So I thought it's fine to use 303 here.
Do you think it's wrong?


There are no requirements for checksum implementation, why forcing such variables for future implementations? Remove whole constructor until there are requirements.


I don't like it either but I kept is consistent with the existing code.


validate($checksum). validate($checksumToValidate) is redundant information in variable name.


That would not rule out other issues. It could still be a type mismatch now
that J2k returns a Uint16 when the code-stream is unsigned. It could be
that the j2k code-stream is offseted to be unsigned and resigned after
decompression with re-scale/intersect. I could also be the VOI transform on
unsigned data.

The problem appears only in the darkest regions suggesting leading me to
believe it is a type issue.

On Tue, Jun 16, 2015 at 9:29 AM Chris Hafey <notifications@github.com>
wrote:

> I believe the API from cornerstone to the JPEG2000 decoder has not changed,
> it would be good to swap back to the older one to determine if the issue is
> in the JPEG2000 library or cornerstone.
>
> On Tue, Jun 16, 2015 at 8:24 AM, Jean Francois Pambrun <
> notifications@github.com> wrote:
>
> > I understand. Unfortunately, I can't reproduce your issue and it is very
> > difficult to debug without a problematic J2k code-stream. You can always
> > use an earlier version until the issue is fixed.
> >
> >
> > On Tue, Jun 16, 2015 at 9:14 AM idmpacs <notifications@github.com>
> wrote:
> >
> > > This image looks fine in all other DICOM Viewers I tried. The problem
> is
> > > with Cornerstone viewer only. But it was OK with earlier version of J2K
> > > library. I don't know exactly where is the problem, it could be with
> > > Cornerstone also.
> > >
> > > On Tue, Jun 16, 2015 at 5:59 PM, Jean Francois Pambrun <
> > > notifications@github.com> wrote:
> > >
> > > > This doesn't look like compression artifacts. With the new commit,
> the
> > > > codec returns a unsigned array when the codestream is unsigned. Are
> you
> > > > doing any arithmetic that could cause overflows or underflows? I
> don't
> > > > remember how the VOI transform is implemented, maybe the issue is
> > there?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11701413
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702304
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702555
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702649>
> .
>


-n?


Probably, bisimilar to 'empty' is not the right explanation here. Maybe bisimilar to empty at the next step would be more accurate. For `pure` see 57670b8e78c333d5fd721db719e2eed7139a2e6c.


I'm pretty sure the mount thing was exactly that. T3s with mounts were fine, T2s floated in midair a lot until I waded in and made some messy changes, and at this point I wanna say it was just the T2s erratically getting quasi-random overlays - I'm pretty sure they got them when the T3 mount had an overlay and the T2 didn't, but I got extremely distracted with content and never went back to confirm that supposition.

I can't think of any other things unmentioned with mages outside of hats and hair. Based on some nations I generated last night it looks like #needs still isn't being enforced all the time, as sacred Colossi/cavemen/sleeper-lites (i.e., "highmen") with fancy animal pelts and #needs helmet leatherhood and #needs shirt no_shirt will end up with assorted helmets instead of just hoods - I didn't see any shirts, though, so that's better than how it is in the release version where they wear shirts under their gigantic furs. I also saw a few elite shirts generated on advanced humans, so the release-version issue with those not generating most of the time looks to be resolved as well.

Oh - on the "very easy to fix" front, heroes should probably be #gcost 0 instead of their current expensive-upkeep incarnation. Ideally, a tag for recruitment filters to keep them from generating with chaosrecruitment, etc. would probably be good for them as well.


validate($checksum). validate($authInfoToValidate) is redundant information in variable name.


Why you need this additional variable?


i'm ok with this rule set, but we should make sure that the error message reflects what went wrong.  instead of a boolean, these functions should all return the error message.  the caller can then match for `Just msg` instead of testing for `True`, and in the error case can use the error message from the validation function.


Excellent suggestions, will do, thanks Nik.


But `empty <*> p` is bisimular to `empty` for all p.


LGTM. Take the "--" out of the title.


This commit breaks polymorphism: When called on an `ClassificationDataSet` (as shown in the tutorials) it no longer returns `ClassificationDataSet`s but `SupervisedDataSet`s.

See discussion here: http://stackoverflow.com/questions/27887936/attributeerror-using-pybrain-splitwithportion-object-type-changed/30869317#30869317

And pull request here: https://github.com/pybrain/pybrain/pull/164


One thing to note (not related to this PR): Use of `window.contextVars` is discouraged in reusable modules such as `contribManager`. Reliance on globals complicates reusability and testing. So we should probably add a TODO that says to remove the dependency on `window.contextVars`.

However, using `window.contextVars.node.urls.api` is slightly better than using `nodeApiUrl` because it's explicitly that the variable comes from the global scope.


why not:
```scala
private[sql] val ordering = Ordering by address
```


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


should be private, don't pollute public API


Just going off what's in the PR comments.  If the code is an intentional change (which it was in the code) then so be it.


what are semantic predicates?

What's wrong with `satisfy`?


if you know how to please squash all your commit and force push to this branch, so we have one nice atomic commit. If you don't know how to do that, never mind :) too much effort to learn


_A foolish consistency is the hobgoblin of little minds,..._
R.W.E.


Deals with requesting algolia behind a proxy, based on environment variable.


i guess now we could just get rid of that last inline arbitrary instance as well...


Wrong context.


These two tests (metadataHasChecksum, metadataHasCorrectChecksum) are also implemented in more specific checkSumIsSetCorrectly test. Maybe it could be sane to remove these tests in favour of checkSumIsSetCorrectly?


#576 similar to this one.


I am wondering whether using Python 3.x precludes the testing of Python script prov-convert in Python 2.x.

There might be a more elegant solution, but these seem to address the same issue:
- http://eli.thegreenplace.net/2014/04/02/dynamically-generating-python-test-cases
- https://gist.github.com/patkujawa-wf/1f569d245bbfc73f83a3


"Semantic predicates" is just a term used in some parser literature. `satisfy` is also ok -- I don't have too much of an opinion here, as long as I do not have to remember, where to prefix with `m`.


Leave a line note


в этом случае  $auth всегда равна false даже если присвоен access_token


Closed #11


I'm pretty sure that at least your specific #needs problem is fixed as of last few commits. I moved from sacred generator's own equipping code to unitgen's one now that I can use filters to make special items more common. Old code did not check whether unit already had an item in the slot so it just overwrote the #needs stuff sometimes. The unitgen code should check first.

#chaosrecruitment and stuff can now probably be handled a lot more easily than before by just adding tags notforheroes or something to filters and giving a filter with suitable themeinc to all heroes. Logically #gcost *0 should work for gcost if it works like it should (ie multipliers handled last for all commands). I'm gonna put up an issue about that so I remember to change it later.


will do


When we perform a screen unghost operation with UnGhost, we also reset the Unghost counter. Reason: because it is the threshold. If we dont clear the counter after every UnGhost operation, it will happen randomly that two screen unghost operation will happen in less than 2 seconds.


We have no experience with setting up Jenkin. Would you be able to advise us on this?
We can set up a VM for this.


For default values for iPQF, I would suggest: low.support.filter =  FALSE, ratio.calc="sum", method.combine = FALSE


That test is checking if you will get the value you have previously set. It's testing setter and getter, together - so name 'metadataCanSetSource' is not describing what it does.

Also, there is inconsistency in tests naming. "metadataHasCorrectTimestamp" does the same as "metadataCanSetSource" and yet they have totally different names, none of them describing properly what they do. They're not checking correctness of values, or checking whether you can set value. It tests if you can retrieve the value you have previously set.


Maybe a pair like `consume` and `canConsume` symmetric to `hasNext` and `next` would be more appropriate.


Just looked at the image and it looks more like an image display pipeline
issue more than a JPEG 2000 issue.  The cornerstone image display pipeline
is dependent on correct values for minPixelValue and maxPixelValue - I have
seen issues like this when these values are not right.  An off by one error
will cause this (e.g. setting minPixelValue to 0 when it should be -1)

Chris

On Tue, Jun 16, 2015 at 8:52 AM, Jean Francois Pambrun <
notifications@github.com> wrote:

> That would not rule out other issues. It could still be a type mismatch now
> that J2k returns a Uint16 when the code-stream is unsigned. It could be
> that the j2k code-stream is offseted to be unsigned and resigned after
> decompression with re-scale/intersect. I could also be the VOI transform on
> unsigned data.
>
> The problem appears only in the darkest regions suggesting leading me to
> believe it is a type issue.
>
> On Tue, Jun 16, 2015 at 9:29 AM Chris Hafey <notifications@github.com>
> wrote:
>
>
> > I believe the API from cornerstone to the JPEG2000 decoder has not
> changed,
> > it would be good to swap back to the older one to determine if the issue
> is
> > in the JPEG2000 library or cornerstone.
> >
> > On Tue, Jun 16, 2015 at 8:24 AM, Jean Francois Pambrun <
> > notifications@github.com> wrote:
> >
> > > I understand. Unfortunately, I can't reproduce your issue and it is
> very
> > > difficult to debug without a problematic J2k code-stream. You can
> always
> > > use an earlier version until the issue is fixed.
> > >
> > >
> > > On Tue, Jun 16, 2015 at 9:14 AM idmpacs <notifications@github.com>
> > wrote:
> > >
> > > > This image looks fine in all other DICOM Viewers I tried. The problem
> > is
> > > > with Cornerstone viewer only. But it was OK with earlier version of
> J2K
> > > > library. I don't know exactly where is the problem, it could be with
> > > > Cornerstone also.
> > > >
> > > > On Tue, Jun 16, 2015 at 5:59 PM, Jean Francois Pambrun <
> > > > notifications@github.com> wrote:
> > > >
> > > > > This doesn't look like compression artifacts. With the new commit,
> > the
> > > > > codec returns a unsigned array when the codestream is unsigned. Are
> > you
> > > > > doing any arithmetic that could cause overflows or underflows? I
> > don't
> > > > > remember how the VOI transform is implemented, maybe the issue is
> > > there?
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11701413
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702304
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702555
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11702649
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OHIF/image-JPEG2000/commit/d6c4f548aef5fdb153c673503e1e52fa4a017d64#commitcomment-11703149>
> .
>


Cool .All fixed in latest commit now.


With this update, I am now getting failures from the ovs bridge.
Changing line 53 of cluster/vagrant/provision-network.sh from:
```bash
ovs-vsctl set bridge ${OVS_SWITCH} protocols=OpenFlow13
```
to:
```bash
ovs-vsctl set bridge ${OVS_SWITCH} protocols=OpenFlow10,OpenFlow11,OpenFlow12,OpenFlow13
```

Seems to correct the issue.  I can submit a pull request if appropriate.


I think you have a bug here... pop requires an index, i.e. integer, but you supply the actual element?


I don't think so, 'available_for=content_view' implies 'library' repos, specifying an environment other than library doesn't really make sense


I don't understand why we need to put each GUI element in its own box...


> Hello. I'm facing an issue because of this addition.

@leandro If you can provide more information about the problem as it exists in the code, such as a test case showing the problem, open a new issue on our issue tracker.


详情见：http://www.cnblogs.com/jerrylead/archive/2011/04/18/2020209.html


No, I'm sorry, I've been quite busy covering the WNBR in my town, editing and publishing the pics. I can check it right now. Give me a few minutes. What's your research about by the way? If you want check this app I made to research the distribution of efforts over time at a segment https://ptdrow.shinyapps.io/SegmentApp/


Crash should be fixed.
Issue is that if breaks the render to texture test.
I'm looking at the scaleoffset code, in render to texture test the output is scaled to 20% in a direction and 33% in the other one, so the result is "correct". I wonder if we miss a scaling operation in RSX emulation...


@belerophon According to the AWS documentation, ProgressEvent.PART_COMPLETED_EVENT_CODE was replaced by ProgressEventType.TRANSFER_PART_COMPLETED_EVENT. Is there any special reason to use REQUEST_BYTE_TRANSFER_EVENT, here?


This is not right.

You get {"array":[]} with this


시간은 비교해봤더니 비슷했고, 영어 문법적으로 좀더 말이 되는 방향으로 한거.
게다가 `Status#inside`라는 스코프를 만들 필요 없이 `Group has_many :members_statuses`라는 존재하는 association을 이용하면 되기 때문에 코드상으로도 더 쉬움.


Great! Sounds good


for me unghosting moving map at fixed interval are not needed and can to be removed.
already done in my own "build  for flight"


`target_link_libraries` does not guarantee that proto library is compiled firstly, at least when building in parallel like `make -j4` error always occur on my machine.


lol sorry XD


좋은 지적인데, 이게 내가 전에 말했던 속도 저하의 문제였어. action_goal을 거쳐서 status로 가면 "이 Goal 안에 있는 모든 Status"같은 쿼리를 날릴 때 더 느려지길래.


@robertmesserle  This breaks consistent button styles inside a `layout="column"`, button takes full witdh while anchor does not.


The main thing that concern me is the different behaviours in Python 2.x and 3.x with respect to handling strings. I am simply uncertain that we have the same results in both environments. However, if there is a good reason to believe that it is the case, I am happy with having the integration tests running in a single environment, which is Python 3.4 as you suggested.


rspec 깔고 나서 모델을 rails generate할 때 자동 생성되는 것


ㅇㅋ 감안하겠음. 좋은 지적이다.


Ok now fixed msibuild.cmd I think you can accept it.


@rocapal removed cvBlobs checking and linked to the internal cvBlbobs


jmikola/geojson where is this being used ?


Can you add a test for an enumeration that is written out? You can use the OpaqueDataType.None value. I believe that is the only extension value in the code. That is how I found this bug :)


It would have been better if you had cherry-picked as in all probabilities it comes from my repo


This one should the render to texture test while terraria and voodoo still correct.

	// Scale
	scaleOffsetMat[0] *= (float&)methodRegisters[NV4097_SET_VIEWPORT_SCALE + (0x4 * 0)] / (m_surface_clip_w / 2.f);
	scaleOffsetMat[5] *= (float&)methodRegisters[NV4097_SET_VIEWPORT_SCALE + (0x4 * 1)] / (m_surface_clip_h / 2.f);
	scaleOffsetMat[10] = (float&)methodRegisters[NV4097_SET_VIEWPORT_SCALE + (0x4 * 2)];

	// Offset
	scaleOffsetMat[3] = (float&)methodRegisters[NV4097_SET_VIEWPORT_OFFSET + (0x4 * 0)] - (m_surface_clip_w / 2.f);
	scaleOffsetMat[7] = -((float&)methodRegisters[NV4097_SET_VIEWPORT_OFFSET + (0x4 * 1)] - (m_surface_clip_h / 2.f));
	scaleOffsetMat[11] = (float&)methodRegisters[NV4097_SET_VIEWPORT_OFFSET + (0x4 * 2)];

	scaleOffsetMat[3] /= m_surface_clip_w / 2.f;
	scaleOffsetMat[7] /= m_surface_clip_h / 2.f;


After this is merged we will need another minor release with this bug fix.


Same goes here as well ( It would have been better if you had cherry-picked as in all probabilities it comes from my repo )


We can ignore the above if we decide to go with Python 3.4.


There are two ways that `p >>= f` can accept a word:

(1) If the next token is consumed by p, there are no results right now (`results = empty`), and we can defer handling of the `>>= f` to after we consumed the next token (`consume = \t -> consume p t >>= f`).

(2) If no more tokens are consumed by `p`, we have to call `f` on all current results of `p` (`fmap f (results p)`) and to combine all of the resulting parsers with `<|>` ([`asum`](https://hackage.haskell.org/package/base/docs/Data-Foldable.html#v:asum)).

Since we want to support both options at the same time, we combine the corresponding code snippets with `<|>`.


i check. I did that because when you move to multimaps with terrain on, or even without terrain, the ghost effect is clearly visible.


没有东西哇？


"Sentiment Classifier for Restaurants Dataset" create some unique meaningful commit messages !


If hashtag==false, this will throw an exception.

Thanks!


You should investigate using composer :smiley:
as well as letting you install third party libraries easily it also can generate autoloaders for you with a neat JSON config file: https://github.com/tomverran/robots/blob/master/composer.json


I found this bug few minutes ago...


composer is not part of the php image cause we dont have composer pkgs for the scl php-55.


Hello! :)


Check the sti-php assamble script


these should not be grouped cause of different top level package


Is it worth having this function inline?


hahahah @spastorino I know, it wasn't supposed to be commited, still working on this :smile:


now, unghosting is made when you move to Multimap, so no need to wait xx second for have clear screen
i think it's better like that.


I would use same styles as for the _non-lazy_ images. See `.usertext-image` - basically only:
```
margin: 7px 0;
max-height: 500px;
```


Should we warn if the user sets the pe_affinity variable on OS X (i.e., because it won't do anything)?


Gracias Pedro :)


Incorporated Johannes Feustel's patch from https://review.typo3.org/#/c/36623/


Why did you change this? Testing for dictionary membership (the former method) is typically better than broadly catching an IndexError. Usually if you can test for a specific expected condition, that's better than catching an Exception.


Guess who's back!


@joshlopes No idea. Was already here.


Thanks for the code snippet.
By the way I get a lot of flickering in Terraria, I suspect it's because of the way the command buffer is handled but I'm not sure.


@Rivsen go to fix our VPS, please~


I've just ran the test on my local machine using fits 0.6.0 and got the same result.

```
<identification status=\"CONFLICT\">\r\n
  <identity format=\"Plain text\" mimetype=\"text/plain\" toolname=\"FITS\" toolversion=\"0.6.0\">\r\n
    <tool toolname=\"Jhove\" toolversion=\"1.5\" />\r\n
  </identity>\r\n
  <identity format=\"Rich Text Format\" mimetype=\"text/rtf\" toolname=\"FITS\" toolversion=\"0.6.0\">\r\n
    <tool toolname=\"ffident\" toolversion=\"0.2\" />\r\n
  </identity>\r\n
</identification>\r\n
```


STI adds `${HOME}/bin/` to the PATH, thats why I'm symlinking it there. Why `working dir != $HOME` shouldn't equal ? Thats the purpose of it -> https://github.com/jhadvig/sti-base/blob/master/Dockerfile#L58
It would be probably for the best to prefix them all with ${HOME}, or am I missing some black magic with $CONTEXT_DIR ? :)


SUBSHELLS DON'T BREAK OUTSIDE LOOPS

```Bash
for i in 1 2 3 4 5; do if ((i==3)); then (p=1; break;); fi; done
echo stopped at $i and p=$p.
# stopped at 5 and p=
```


ackj


Thinking about this a bit more, you might want a bit more detailed help message for this variable.  In the MPI community, we have found a LOT of confusion about affinity from end users.  E.g., physical vs. logical processor IDs, mapping physical IDs to actual cores (and/or hyperthreads), the non-contiguous nature of (some) vendor physical IDs, etc.  Is it worthwhile to use a logical numbering system, somewhat akin to (or even directly utilizing) hwloc?  E.g., hwloc supports syntax like "hwloc-bind socket:0.core:3", which means all hyperthreads on the 4th core on the 1st processor package (using logical numbering, not physical numbering).


ack


Could you please be more specific in commit message, for example Fix comment in gadget-import? Please also add a libusbg: example: prefix before subject and a signed-off-by line.


For now, I don't know yet.
I made it available on the `$rootScope`, because I thought I might need elsewhere. Wasn't sure where I would need my token.
It'll change later.


Im already generating `CAKEPHP_SECRET_KEY` env var which hashes both of this values:
https://github.com/jhadvig/cakephp-ex/blob/initial/lib/openshift.php#L27


@jeljaik @naveenoid   you can check the files under r_foot folder for the taxel position serialization of the foot


Could we add more detail to the error message, e.g. mentioning the modName and what the values of the two hashes are that aren't matching - that makes it much easier to debug what's wrong when something is wrong.


Don't think it's enough - I believe we'll need the Issuer Key Hash for both the x509_entry and precent_entry_V2 (to bind issuer to precertificate/x.509 cert).
@benlaurie can you confirm?


@dmethvin I don't think it's a bug, because it's clear it's a feature implemented by the committed line above. The issue is that my script is sending data via POST, but then it gets automatically changed to GET because of that line and that breaks my page. I really need to send the data via POST and process the response as 'script'.


Hi @FernetMenta when can't reach that code path when ff/rw:ing because check is set to false on line 1830, since m_dvdPlayerVideo->GetCurrentPts() returns NOPTS when play speed isn't normal. This can lead to "strange" timestamps getting displayed. E.g.:
![screenshot000](https://cloud.githubusercontent.com/assets/2150851/8185213/41f1d162-1443-11e5-8c28-354ba1069a51.png)

But without that check the frame output is stuttering/jumpy.
btw seeking backwards to zero shows some strange behaviour too.


Yep i did fix the flickering previously in master build when factorize the RSX code but DH said it is not the right way to do so reverted it .


Yes, I have a hard time with the doc about it (and some other things not so clear).


This is still kind of an odd design, that I'm not entirely comfortable with, but I think it will work. As long as we don't continue adding more things to this form of configuration...


I had permission problem with the default `php` value. Based on the conversation regarding session saving, I will look up for the some hack that when the application is using DB it will take `database` value, else `cake`


Lol, didn't realise @ would ping another user xD


Copy that.
I'll take a look when the rest work.


CSS Masks not supported by: IE (10,11), Firefox (37,38), Chrome (42,43), Safari (8,7.1), Opera (28,29), iOS Safari (8,8.1-8.3), IE Mobile (10,11)


Heyo :)
Sorry didn't realise a @ would ping another user!


could another alternative be to write `query/4` with a default for options?:
    def query(pid, bucket, query, options \\ []) when is_pid(pid), do: search(pid, bucket, query, options)


Btw , do you have Disgaea 3 installed? The one uses same engine as Guided Fate Paradox but it crashes at startup.


Agree, I probably forgot to change those names


In #19, I wrote lasers would work in places with high light (7+) and warmth levels (warm biomes).


I like this. This is a clean way to compute the band bounds and have them available to an code that's using the bands.


@VictorJSV, quizás podrías alinear las propiedades para cada clase

```stylus
.icon
	-webkit-font-smoothing antialiased
	display                inline-block
	font-family            "iconFonts"
	font-size              22px
	font-style             normal
	font-variant           normal
	font-weight            normal
	line-height            1
	text-decoration        inherit
	text-transform         none
	vertical-align         middle
```


@VictorJSV alinear url


what ?


Please add libusbg: prefix and some verb to subject for example
"libusbg: Fix typo in macro name CREATE_ATTR_STRING". Please also don't forget about singoff. Moreover, usually git commit logs use imperative form (Fix a bug) instead of (Fixed bug). For more details please refer to some commit log tutorial for example:
http://bit.ly/goodcommitmessages


updated read me


This needs some more explanation. Why would such a thing happen? Why should we support such a setup? Why doesn't allowing the machine-id to be uninstantiated break everything? :)


Reverted, see commit 59932d16afcc749b003cee8252d74435cd7fcb81 for details.


Well, as I said, I would prefer `satisfy`, and maybe `anyToken` if you don't like `anyChar`. Or maybe `anything`. I'm just not happy with shadowing `Prelude` definitions with something completely different.


I'm concerned that there is both a player and a user model with the same exact attributes. I think we only need one or the other.

Also you set the root route to static_page#index, but that doesn't link to anything because you haven't generated a static_page controller with an index action. I would also suggest the using the plural version static_pages#index and static_pages controller to conform to Rails conventions.


xxxxxxxxxxxxxxxxxxxxxxxxxxxxx


Hahahaha :D


I guess this could be nicely combined to:
      return log_error_errno(EOPNOTSUPP, "--private [...]");
That way you get a nicer journal entry (and save a couple lines of code).


Can we try to write down in a paragraph or two which inefficiency we are trying to optimize for, and what the intended result is.


zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz


> при этом ХП > 1 (проигрыш дуэли/боя/смерть)

В поле герои без проблем расхаживают с 1 хп. Так что стоит добавить проверку типа `blablabla.value > (ui_data.isFighting ? 1 : 0)`.


wrong action name.
url/shorten url is triggered and send to yourls-api.php


It's just fun to receive this kinds of notifications from time to time :P No need to say sorry :D Good luck on your game! :)


Django handles the management of static images for us (like uploading to S3), so there's a bit of extra syntax involved.

Images can actually be added to GitHub, in the **manifestos/static/manifestos/images** folder.

Then the image tag should look like this:

<img class="img-responsive" src="{% static 'manifestos/images/DMA.png' %}" id="dma" />

Django will insert the appropriate **src** link automatically.


hi dude ...i respect your work .....

this commit every one can find in almost all kernel developers source .....my friend if i add this in my kernel it doesn't  mean that i have taken from you only or some one xyz only .....i can say many developers made same commit on kernel source example tarun ...now i give it to your name ....what should i do if he asks for his name ....?  this only my point .....

better we discuss this in whatsap ....just shoot me a message in my number friend ..+919900243069

have a nice day


ack


I think this commit may have broken something, we're seeing a lot of diffs where `if` is highlighted in the middle of words (not as a keyword).


I'd rather tie this functionality directly to the `AsyncItemProcessor` implementation.  I realize that may feel a bit restrictive, but my fear is that by adding this new interface, we run the risk of confusing users on if a regular processor should implement `ItemProcessListenerSupport` (which it should not).


Can you give a short explanation why you use 64 as parameter? In general, I would replace the "64" by a descriptive constexpr/enum that is declared in a central configuration place like ``kernel/configuration.h`` or in the translation table if arch specific.


I am testing, the screen is perfect of course but that’s because we do unghosting everytime we change a page.
it is very slow now. Maybe we can have a user choice among: no unghosting, light unghosting, heavy unghosting.
The problem is: when we are switching pages we really dont care to unghost, because we are going for example from map page to multimap4.
I think there are no doubts that now the software is twice slower..

From: Bruno de Lacheisserie
Sent: Tuesday, June 16, 2015 4:15 PM
To: brunotl/LK8000
Cc: Paolo Ventafridda
Subject: Re: [LK8000] UnGhost eInk screen when : (0425726)

now, unghosting is made when you move to Multimap, so no need to wait xx second for have clear screen
i think it's better like that.

—
Reply to this email directly or view it on GitHub.


This listen is way too big, you should put the case bodies into separate methods.


approved


Tanks for spotting.


instead of `cin` please consider using
```c++
int main(int argc, char **argv){
if(argc == 3)
{
// retrieve numbers a from argv
}
// ...
return 0;
}
```


For me this actually breaks them (as in they look completely black now).
Debian sid with gtk 3.16.4-2


Let's Go Baby!


@igrekster
What do you mean with a non-**braking** undo history? Also, is this feature in Vim? It seems I can't find anything about it...

I believe this change was really for fixing #554 like the commit message says, running a command from within the Visual modes would also try to execute the command from that previous mode. Visual mode isn't meant for this though, it clobbers the selection before executing anything. Executing things in Normal mode seemed more natural, so that's what I went with.

Can you give some examples what you use this for and how this preserves history?


:+1: just in case commenting :smile:


You can take a look on my git for TWRP : Ascend P6, Ascend P7 & G620S - > for TWRP P8 , should not differ too much from P6 or P7 !

( I have not yet a P8, and I'm not sure that I'll buy one )

Anyway, good luck !


is `feed == parse`?


In fact , the buffers' and tags' format can be represented by a block of memory, filled by data line by line.
{
-some common lines
-tag0
-tag1
...
-end tag
}

and each tag is a structure like
{
-some common lines
-request param0
-request param1
....
-respond param0
-respond param1
...
}

I didn't get it how to use a zero-length array representation. Could you please refer any sample ?

In fact, vc is the rpi's GPU. The mailbox is a interface for arm and vc communication. Also, it's vc who do the boot at the very beginning. So the defines are for the bcm mailbox interface How would you suggest to name it? I've found others may use something like bcm2835_mbox__xxxxxx_tag.


How about?
    gray = app.image.to_cv_grayscale()


`feed` is `parse` without `results`. It returns a parser


Why is this the correct answer? I expect that `many empty` is bisimular to `empty`, so it should be as dead or alive, shouldn't it?


Looks good to me.  Thanks!


:+1: why not sending a pull request?


This query become redundant.


No worries, I totally understand, thank you for your reply :) I saw your Shiny app, but what I'm looking for in *Strava* data is a bit different. If you want to hear more I'm happy to tell you about it but would prefer to do it via PM (its a project in progress). Can't seem to reach you via Twitter direct message, any other way I can reach you more privately than here?


Yes Alex. I was just working around with the features and was going to do exactly the same after final selection of features.


Really strange as I'm also on Debian/sid! Which wm?


Yes, see the comment on https://github.com/Toxaris/coalgebraic-parsing/pull/1#discussion-diff-32517736


TODO Update: 1 test in t7004 failing, fixed the others. There was a small whitespace which I missed out causing a lot of tests to fail.


Awesome! Det tänkte jag inte ens på.

Trevlig sommar till er på TV4! (Och ursäkta spammet)

/Lellky
On Jun 16, 2015 3:40 PM, "Kristian Saebdal" <notifications@github.com>
wrote:

> @lellky <https://github.com/lellky> Tjena! repot är publikt - du får
> unwatcha det själv. jag kan inte ta bort på min sida. Ha en fin sommar!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TV4/video-metadata-api/commit/1906b3a82a696a937e7a619538c6b95ce7c9d275#commitcomment-11702863>
> .
>


KDE


Okay...that's why i have said it would have been better ( It's logical since bulk of commits ( hotplug,fsync,module param,etc,etc. ) come from my kernel and hence i am of the view this also should have been ). Anways nothing much to talk on this as you say you didn't take it from my repo


CMake configuration looks good.
With "DOCS_AUTO" flag docs build can be run with DOCS_AUTO=OFF (default) which will not run through ConfigureMgr.


Line 60, this should be a variable of new RoutableCollectionViews()s. This is likely why the clicks aren't rendering


ack


Can you provide a codepen to demonstrate this?

On Tue, Jun 16, 2015 at 7:09 AM, Peter Leibiger <notifications@github.com>
wrote:

> @robertmesserle <https://github.com/robertmesserle> This breaks
> consistent button styles inside a layout="column", button takes full
> witdh while anchor does not.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/cc07e63dc5a37b5b220691c3cf8e4b3d08bd2500#commitcomment-11703513>
> .
>


@tarelli why did you remove this 'if'? populateSimulationTree can be called per aspect, right?


reack


Apart from the things @alexanderpanchenko  mentioned: You shouldn't commit IDE or project specific files like ".idea/uiDesigner.xml". Have a look at [gitignore](http://git-scm.com/docs/gitignore).


I would suggest to use git2r instead of the command line git program:

```r
> repo <- repository("~/R/animint")
> head(repo)@name
[1] "panel_backgrounds"
```


The number of reqest and respond for each tag are different, some of them don't need to have request parameter. And each buffer can have multip tags so that they are proccessed at the same time. How do you think a better way to create a buffer?


Testing manual comment addition


@dodobas , Instead of adjective "slovenski" (should be lower cased and followed by "jezik"=language) I'd recommend the use of "Slovenščina" as that is the standalone name of the language (also slightly less likely to be confused with Slovak (or "slovensky" which should be "slovensky yazyk" in full)
cc: @skalca


you should fix the other place too.


Thank you, it was nice speaking with you :D


If you have gmail invite me to the chat: pedrodvf@gmail.com
The problem right now with the code is in the way it is checking how many data it is receiving per page. The idea is to stop requesting for pages as soon as the program sees that the last pages retrieved has less entries than the max per page, being that the last page. The thing is that this is checked with the length(content(req), wich for the leaderboard doesn't seems to represent the number of entries retrieved per page. So instead of returning the number 200 for the first page it is returning 5 and the program stops getting anymore pages.


No I don't have it atm, I'm downloading it so that I can test too.


CSS Masks not supported by: IE (10,11), Firefox (37,38), Chrome (42,43), Safari (8,7.1), Opera (28,29), iOS Safari (8,8.1-8.3), IE Mobile (10,11)


still png me friend i need some help & suggestion from you ....if you free please shoot a message


I'm using gnome-shell. maybe the issue... but if this is the case it will be quite difficult to please anyone!


(You could delete the TODO comment if it's done and other comments)


yeah I know. but when compiling php extensions, some extensions require the php header files or even require header files from other extensions.  if we don't have the source file, the compilation would likely fail.


macports also uses this approach (when compiling any extension, download the php source first)


There's probably a way to not hard-code it using res-auto namespace.


Assume we have parsers `a_i` that have the shape `token 'x' <|> ...` and another parser with `many (token 'y')`. Parsing the input `y^n` causes i*(n-1) parser applications that are unnecessary, since `empty <|> p` is bisimilar to `p`.


Why do you change "ignoreNotFound" to false?


I think the check code should be moved into the install subcommand.


Looks good.


but that would mean we have to move `AsyncItemProcessor` to `spring-batch-core` to prevent circular dependencies.
would you accept that?


commento test


Yes, I was about to suggest that because `disable|enable` definitely don't need the check.


reack


sdvdwv


agree


vewvewv


even the rest is useless you will never have a NameError


Without me having to read all the code, could you explain why this patch was removed/changed/whatever in the first place in upstream?


Maybe the repository was not checked out to a branch at the start of the function. In that case this code would fail.

Instead of attempting to store the branch name (which will not work when the repos is not checked out to a branch), I would suggest storing the commit object:

```r
old.commit <- head(repo)
on.exit(expr=git2r::checkout(old.commit))
```


The reference line tool is giving following error from last few updates
Cannot read property 'start' of undefined


I agree, let's revert the commit and move the check code into ext install command?


should probably revert this and move it to `hihat` - devs should be encouraged to use [errorify](https://github.com/zertosh/errorify) which already does this


sure ^^


Awesome! Does this need to happen at the other atom plugins too?


ack


Its fixed I forgot to push the tag.


hardcoded subfield 035 by mistake


@alexanderdean - it looks like if multiple queries fail, only the last error will be recorded in the status array. Should we change it so that all query errors are recorded and returned?


Never mind - just noticed the `break` statement.


현재 작업하고 있는 프로젝트에서 template 코드를 가져와서 dom을 생성하는 부분이 있는데, data가 많아서 each문 돌리고, template 코드 복제하고, 숨기고 코드가 상당히 지저분한데.. 그냥 map쓰면 될걸... 내일 적용해야지


Feel free I guess :).


They also need to [switch](https://github.com/atom/emissary) from `emissary` to `event-kit`.


It almost looks like at one point we configured FITS to handle this as indicated in slide 35 on this presentation http://www.aserl.org/wp-content/uploads/2013/04/FITS_ASERL2013.pdf but I cannot find proof of this. The configuration on the production server does not seem to have any tweaks different from the defaults.


This is for #1439  (there are still warnings left, but this cleans up a bunch)


Quedaron muy bien las nuevas imágenes de fondo!


> Feel free I guess :).

I never used atom for longer than 5 minutes so am not really the right one lol :p Pretty cool they have this integrated already though. Are they just using io.js or some kind of transpiler?


Programs to demonstrate management of variance in exploited ecosystems with critical thresholds


We need to update the comments also


Babel, notice the first line.


Fixed everything :D Will push


Thanks for naming it "Visibl governance" !


@Temptin Thanks for the feedback. I like the tool so far. I tried it for the first time on Windows yesterday and wanted to see if I could run it on Linux. It was quite easy to get it running :)

> Is it building a 32 or 64 bit binary, or maybe it automatically chooses based on the compiler's system?

I think CMake figures this out by itself. I ran it on a 64-bit machine, so it built a 64-bit binary.

> What packages does the user need on their system to run this process? I'd like to provide a list of necessary base packages in the readme, such as "sudo apt-get install cmake make gcc"

Unfortunately, I'm not sure about this. I think cmake, make and gcc come pre-installed with Ubuntu. I can check this at some point and let you know. Boost should be the only additional requirement. This can be installed with sudo apt-get install libboost-all-dev
> Why are you specifying "Boost 1.46" in the CMakeList requirements? It will work with 1.46, but the preferred version is 1.58 or higher. Any special reasons you chose a lower one?

This was the version I had installed. I'll change it to 1.58

> I notice that you're including the "system" subcomponent of Boost, but we're not including any of those headers anywhere in the source code, so why is that one on the list?

I didn't look much into the code actually. The 'system' subcomponent was listed in build_macosx.sh so I included it here as well. I'll check to see if it's actually required

> Remember to also include the subcomponents that give us "boost/utility.hpp" and "boost/tokenizer.hpp" since it seems like those are missing from the list right now.

ok

> Do we really need to specify all of those "include_directories"? All #include statements in the source files already use relative paths such as "../libs/sqlite3.h", so they shouldn't need it. The only 3rd party include directory that's actually necessary is the location of boost headers.

Again, I probably should've looked at the code before doing this :)

> Instead of giving people a list of "On Linux, type all these commands," the preferred method is a "_gccbuild/build_linux.sh" script which simplifies the process and does all of that. It's especially important to have a single rebuild command when developing and doing rapid changes and lots of recompilation. ;-)

Good idea. I will do this as well

> To keep the directory structure clean, I'd probably put CMakeLists.txt in _gccbuild/linux/ and make the build take place in a subfolder of that as well. Those details don't matter right now... when merging your changes, I'll figure something out that cleans up the structure of the _gccbuild folder and cleanly separates it into subfolders for OS X and Linux.

ok

> And lastly, and most importantly: How is SkypeExport working on Linux? Any issues? I know that Skype on Linux is still version 4.3 which means that it's a slightly older database format (not using any of the new features), but I wrote SkypeExport on Skype 4/5 (can't remember which) for Mac, so it should work perfectly. Please do some test exports and look to see if all file transfers, edited messages, /me emotes, emoticons etc show up correctly.

I tested it on a relatively new Skype account with just two contacts and it ran without problems and I was able to see the conversations. I don't have file transfers, edited messages, /me (as far as I can remember). I'll check emoticons at some point. I can test these as I use them on Skype.

I'll make some of the changes you mentioned in the next couple of days and get back to you. Thanks!


Hmm, do you have a code example when this happens?


haha, yeah :) Neat!


I forgot v.setSelected(false);


<!DOCTYPE html>

	<meta charset="utf-8">
	<meta http-equiv="X-UA-Compatible" content="IE=edge">
	<meta name="viewport" content="width=device-width, initial-scale=1.0,maximum-scale=1.0,user-scalable=no">

<button id="setVar">setVar</button>
<button id="showVar">showVar</button>
<input class="foo">
<input class="foo">

<button id="setVar2">setVar</button>
<button id="showVar2">showVar</button>
<input class="foo2">
<input class="foo2">


Is this patch a temporary one or not?


Ooh sorry. I thought I got rid of all the user stuff. I'll fix this stuff later today and message you when it's done.

Sent from my iPhone

> On Jun 16, 2015, at 07:27, NoahFinberg <notifications@github.com> wrote:
>
> I'm concerned that there is both a player and a user model with the same exact attributes. I think we only need one or the other.
>
> Also you set the root route to static_page#index, but that doesn't link to anything because you haven't generated a static_page controller with an index action. I would also suggest the using the plural version static_pages#index and static_pages controller to conform to Rails conventions.
>
> —
> Reply to this email directly or view it on GitHub.
>


De ce ai folosit aici protected?


https://github.com/D-Programming-Language/phobos/pull/3406/files

Grep for "uniform". Notice how **if** is highlighted. In other occurrences the syntax highlighting is completely broken, e.g. the unittests don't have any highlighting (not sure if it's related though).


I should be done as below,
v.setSelected(position == i);
invokeTabTextChangeNotify(v, i, position == i);


@yannisklonatos
I had a different design in my AST for the Year-extraction. Is this the right way to adapt it?


@skalk I assume that you have this extra class to ensure alignment in ``Page_allocator``. I think it should receive a documentation header that shortly explains this. In general, I wonder why there is the distinction between ``Genode::Page_allocator::Allocator`` and ``Genode::Page_allocator_base`` (as far as I can see, no other class inherits from this interface)? I would keep class nesting and inheritance at a minimum, move ``Genode::Page_allocator::Page`` to ``Genode::Page_placeholder`` or so, merge the class ``Genode::Page_allocator::Allocator`` into ``Genode::Page_allocator_base`` and let ``Page_allocator::_alloc`` be of type ``Genode::Page_allocator_base``. I think this would make code less bulky and more comprehensible. What's your opinion on that?


Eu nu stiu ce reprezinta valorile 1 si 1 din linia asta. Daca ai fi facut cum am specificat eu intr-un comentariu in legatura cu semnatura metodei de pe Employee AddNewSalaryHistory ar fi fost foarte clear. AddNewSalaryHistory ar trebui sa primeasca o valoare a noului salariu si in interior sa creeze un SalaryHistory pe baza valorii trimise ca parametru.


Are you looking in correct repository? This is for the TextMate bundle.


Yes, the app/Http/Middleware/Authenticate.php (ships with 5.1) remembers the requested url and redirects after login.  However, the two middleware files I wrote don't remember the url because they only redirect if the user does not have access to the requested url.


Removed .fa class


CSS Masks not supported by: IE (10,11), Firefox (37,38), Chrome (42,43), Safari (8,7.1), Opera (28,29), iOS Safari (8,8.1-8.3), IE Mobile (10,11)


CSS Appearance not supported by: IE (10,11), IE Mobile (10,11)


@skalk I try to provide a short documentation header for every class in base-hw.


Is it a typo?


> Are you looking in correct repository? This is for the TextMate bundle.

This submodule is included in the github linguist (https://github.com/github/linguist) which is used for syntax highlighting on github.


Для `parseInt` рекомендуется всегда указывать основание системы счисления вторым параметром.

*Always specify this parameter to eliminate reader confusion and to guarantee predictable behavior. Different implementations produce different results when a radix is not specified.*
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/parseInt


Hi dbongs,

Could you please confirm if you are joining the meeting. I am already on the meeting. Thanks.

Thanks & Regards
Vivekjyoti Pramanik


Hmmm... libusbg user should not relay on errno values but use values returned from API funcitons. Internally libusbg uses usbg_translate_error() to do some translation from errno values to USBG_ERROR_*. If you see this error it could be a good idea to add this to usbg_translate_error() because now we return USBG_ERROR_OTHER_ERROR for it. So here I agree with Pawel and I don't see how this patch should fix anything.


> Is this patch a temporary one or not?

See the commit message: it's a temporary backport of an upstream Skia commit that will be part of the Skia tree from M45 on.


Was it tested also on Linux?
Make sure to test it on the test machine (not just win - win).


Oh, I had no idea. I thought it was basically only me that used D with TextMate. Now this adds a bit more pressure. I'll have a look.


Degeaba ai facut acest SalaryEventArgs daca nu il si folosesti.
public event EventHandler NewSalaryAdded; din clasa Employee ar trebui sa fie un event EventHandler<SalaryEventArgs> asta ca sa poti folosi detaliile pe care le pui pe argumentele eventului
https://msdn.microsoft.com/en-us/library/db0etb8x(v=vs.110).aspx


neat :dancer:


As requested, I created a new service_call_implementation branch and undid the
changes in master related to method calls.

On Monday 15 June 2015 21:16:43 Julius Pfrommer wrote:
> `master -> methods -> dev`
>
> Merging methods to dev is the same as merging master to dev.
> And you will get both features faster into master as they are easier to
> review.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/acplt/open62541/commit/4717f4f3057a25f31df6ab79b3ee2117aa
> 8a3c37#commitcomment-11687481


Note, this has been lifted out of the for expression to fix a nasty bug when extracting nested maps. Previously, it would fail, I think because `JObject(fields) <- json` would act recursively, which caused problems later on. Very subtle and tricky to work out what was going on.


@skalk Less nesting would even reduce LOC here and simplify indentation ;-)


Check that player != null otherwise return 0.
Such things can happen if player concede while abilities resolves (most in multiplayer because the game has not ended then often).


De ce sunt campurile astea private? Daca prinzi un eveniment cu argumente de tipul asta, nu vei putea sa vezi niciodata detaliile din argument


The version 0.6-2 is for R-3.2.0 not for R-3.1.2 (sorry :)


Comment best topmost of the implementing lines.


Nice speaking to you too! :)


I recommend having a look at gitignore for idea [1].

Cheers,
Laura

[1]:   https://github.com/hsz/idea-gitignore

On 06/16/2015 04:44 PM, Uli Fahrer wrote:
>
> Apart from the things @alexanderpanchenko
> <https://github.com/alexanderpanchenko> mentioned: You shouldn't
> commit IDE or project specific files like ".idea/uiDesigner.xml". Have
> a look at gitignore <http://git-scm.com/docs/gitignore>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tudarmstadt-lt/sentiment/commit/7b36cf472d2128004491b74cd0c12b087ad5aeb8#commitcomment-11704376>.
>


https://github.com/yiisoft/yii2/blob/master/docs/internals/translation-workflow.md
```
The framework has two types of messages: exceptions that are intended to developer and are never translated and messages that are actually visible to the end user such as validation errors.
```


What about overloaded symbols? I.e. a getter that has the same name as a method?


Thanks. Fixed by 585438d.


next lvl: one function to rule them all!
![http://media.giphy.com/media/OGHg7GLgbqNYk/giphy.gif](http://media.giphy.com/media/OGHg7GLgbqNYk/giphy.gif)


In continuare clasa asta nu este generica.


I wonder if we should test the Cluster.protocol_version == max supported before connect, and that same var dialed down to the proper version after.


@tpetr @tejasmanohar is @extraRenderData a no-op?


Good point -- what about this parameter conforming to the more traditional ruby `available_for?` style?


Can lazy vals be overloaded?

```
trait A{ def foo: Int = 1 ; lazy val foo: Double = 2}
<console>:7: error: method foo is defined twice
  conflicting symbols both originated in file '<console>'
       trait A{ def foo: Int = 1 ; lazy val foo: Double = 2}
                                            ^
```


डॉ कस्तूरी श्रीवल्ली, राष्ट्रीय कृषि विस्तार प्रबंध संस्थान (मैनेज), हैदराबाद, भारत।  ksrivally@manage.gov.in


:+1: for ending newline!


Care este rostul acestei propietati?


Using _global-rainbow-delimiters-mode_ causes an error as it is no longer supported.


Add line break


this breaks the client-side pagination for the s3 logs, need to pass in the `data` var that calls `@getRenderData()` and not `@collection.models` directly.


Otherwise LGTM


Add Line break after Line No. 7


Is iPerf mesh working now?


Just for understanding: Why have you remove the alt_fpga_gp*() function calls?


That was the point where ARM hang, right?


added `slime` to `my-packages`.


Great! But have a look at the commit message before pushing :-)


Apologies - I did not mean to commit the change in style. I can modify it and set up another push request if it's too annoying a change.


Please give more details why the hang happend and how this [FIX] solves that!


This file is under iperfmesh folder, so why does it have hping?


@laura-dietz, interesting find! otherwise just

make a copy of the idea files
```
git rm -r --cached <your directory>
```

copy the files back


Cam la ce te-ai gandit cand ai modificat aceasta metoda? Unde este angajatul in acest context?


CSS Masks not supported by: IE (10,11), Firefox (37,38), Chrome (42,43), Safari (8,7.1), Opera (28,29), iOS Safari (8,8.1-8.3), IE Mobile (10,11)


CSS Appearance not supported by: IE (10,11), IE Mobile (10,11)


i think i just screwed up the merge here -- feel free to adjust this to work for you


forEach?


Dashboardul nu este implementat corect. Analizeaza din nou cerintele temei si refa aceasta clasa.


be aware that ```at``` is the guarded array index operator, i.e. it checks whether the index you ask for is less than the array size.
It doesn't make much sense to use the safe version for one lookup, and the unsafe one for the other.
The unsafe version for ```this``` is ```(*this)[i]``` or ```this->operator[](i)```


I found the problem. I'll try to find out the best way to fix this.


Add Line Break