The way I ended up using bson from pymongo is following:-

The simple json dump function reported the following error when I ran that command:-

(ENV)[mangirish@MangoLap client]$ cm key list --format=json
{'--all': False,
'--dir': '~/.ssh',
'--format': 'json',
'--git': False,
'--help': False,
'--name': None,
'--select': False,
'--source': 'db',
'--ssh': False,
'--username': 'none',
'-f': False,
'-h': False,
'FILENAME': None,
'KEYNAME': None,
'NAME': None,
'add': False,
'default': False,
'delete': False,
'get': False,
'list': True}
ERROR: executing command 'key list --format=json'
======================================================================
datetime.datetime(2015, 9, 16, 18, 15, 22, 185869) is not JSON serializable
======================================================================
Traceback (most recent call last):
File "/home/mangirish/ENV/lib/python2.7/site-packages/cloudmesh_client/shell/cm.py", line 390, in main
cmd.onecmd(user_cmd)
File "/usr/lib64/python2.7/cmd.py", line 221, in onecmd
return func(arg)
File "/home/mangirish/ENV/lib/python2.7/site-packages/cloudmesh_client/shell/command.py", line 39, in new
func(instance, args, arguments)
File "/home/mangirish/ENV/lib/python2.7/site-packages/cloudmesh_client/shell/plugins/KeyCommand.py", line 178, in do_key
print(_print_dict(d, format=arguments['--format']))
File "/home/mangirish/ENV/lib/python2.7/site-packages/cloudmesh_client/shell/plugins/KeyCommand.py", line 120, in _print_dict
return json.dumps(d, indent=4)
File "/usr/lib64/python2.7/json/__init__.py", line 250, in dumps
sort_keys=sort_keys, **kw).encode(obj)
File "/usr/lib64/python2.7/json/encoder.py", line 209, in encode
chunks = list(chunks)
File "/usr/lib64/python2.7/json/encoder.py", line 434, in _iterencode
for chunk in _iterencode_dict(o, _current_indent_level):
File "/usr/lib64/python2.7/json/encoder.py", line 408, in _iterencode_dict
for chunk in chunks:
File "/usr/lib64/python2.7/json/encoder.py", line 408, in _iterencode_dict
for chunk in chunks:
File "/usr/lib64/python2.7/json/encoder.py", line 442, in _iterencode
o = _default(o)
File "/usr/lib64/python2.7/json/encoder.py", line 184, in default
raise TypeError(repr(o) + " is not JSON serializable")
TypeError: datetime.datetime(2015, 9, 16, 18, 15, 22, 185869) is not JSON serializable

I found the following solution on stackoverflow i.e. to use the json_utils in bson that handles datetime serialization for json:-
http://stackoverflow.com/questions/11875770/how-to-overcome-datetime-datetime-not-json-serializable-in-python

At first, I tried installing just the bson module (# pip install bson), but somehow it did not have the json_util.

(ENV)[mangirish@MangoLap ~]$ python
Python 2.7.5 (default, Jun 24 2015, 00:41:19)
[GCC 4.8.3 20140911 (Red Hat 4.8.3-9)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from bson import json_util
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: cannot import name json_util

I even tried installing some previous versions of bson, but hit the same problem.

Then, when I tried the bson shipped with pymongo, it worked fine.

(ENV)[mangirish@MangoLap ~]$ pip install pymongo
Collecting pymongo
Installing collected packages: pymongo
Successfully installed pymongo-3.0.3
(ENV)[mangirish@MangoLap ~]$ python
Python 2.7.5 (default, Jun 24 2015, 00:41:19)
[GCC 4.8.3 20140911 (Red Hat 4.8.3-9)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from bson import json_util
>>>

Hence, I retained pymongo.

Please let me know if I have missed on any better solution.


Maybe


I'd get in the habit of only using single quotes for JavaScript string literals, it'll save you having to escape quotes with a \ in other cases like when inserting a piece of `html` from within your jQuery

    myElement.html('
Hello
')

is much easier than

    myElement.html("
Hello
")

note the backslash as the embedded " would terminate the string hence the need to escape it


It does not tag them - it filters volumes based on what tags a user want to filter them with - tags can be anything (any key value pair we want to set ) - If you do set a name for a volume in AWS console, it automatically gets assigned to the tag "Name"


Oh hmm, there was supposed to be one...didn't end up in this commit somehow. One sec.


Nice. Mondrian almost provides you with a pre-made palette. As much as palette #1 may be technically more accurate, maybe #2 would be easier on a screen, as a white background would be the most authentic to your movement and those colors could be almost too vivid. It's funny, with all these really basic and elemental compositional structures, there's so much potential for cool animation possibilities. I see where you're going with the stencil fonts but maybe a nice bold, geometric sans serif or slab font like Futura or Adelle could work also. I can see all of these really lending themselves nicely to a web page.


There have been reports of the implementation of this change breaking some applications in iOS9 (as opposed to iOS8 and everywhere else) --- caused by https://github.com/WebKit/webkit/commit/52807831aa6d02414054dfc90569b6c7e2a93783

Just FYI, it may not have been totally safe, because authors are doing things like `"Bearer " + <some user defined value>` for authorization headers.


I'll address the `preventDefault` here but the same applies to all functions.  It is more common to use the event argument that jQuery packages up and sends to you rather than the global event object

    $( '.readless"').click(function (e) {
        e.preventDefault();
        /* Your code as before */
    });


Why not convert the results of `list` using `JavaCoverters` or `scala-java8-compat`?

 ```scala
javaFile.list.asScala
```


Why do this?


Ha, nice. There was an empty `patient_overview.html` file in the templates but that's all I saw.


No, it's not a boolean - it's a string. Testing for self._reverse_tags == True is a test to see if a --reverse-tags parameter has been passed in by the user or not.


The client send only the final node, we have to generate the path in the core


Was this causing some problem that we should investigate?


cara, se a vida dele continua a mesma nesse "else if(this.getSorte() > 0 && this.getSorte() < 100){", não concorda comigo que seria mais fácil se ele não existisse?


I think line 11 should be:

```
"../parties/styles/parties.import.less";


Before merging can you fix the commit summary to refer to the correct ticket?


Should be there now!


That is not this change. That would be https://github.com/whatwg/fetch/commit/6c00fe28e7a361d2b7e0dda776ebccfaa4c439a5. The specific semantics of which are under discussion on blink-dev at the moment.


acho que ele tá ganhando vida ao invés de perder


Yeah, it's really hard to read the blame view on github, blame my carelessness for that =)


Original patch was OK ,but this is bad.
Its clear old selection when row is specified.
Its breaking change, I strongly wanting to revert this to original PR.


The design choice here echoes `scala.BigDecimal` /  `java.math.BigDecimal` a little. If we had our time over, we probably would opt _not_ to ship those wrappers, and just provide some extension methods to the  java types.

Removing the implicit conversions does help. Here's a commonly seen gotcha in `BigDecimal`-land that stems from having two types to think about and having the implicits.

```scala
scala> def foo(o: Option[BigDecimal]) = o.isDefined
foo: (o: Option[BigDecimal])Boolean

scala> def bar(i: java.math.BigDecimal) = foo(Option(i))
bar: (i: java.math.BigDecimal)Boolean

scala> bar(null)
java.lang.IllegalArgumentException: null value for BigDecimal
```

But even without the implicits, dealing with one type is still way easier than dealing with two IMO.


I think your not trying hard enough.  ;). Surely you can move the exception handling up as well?


WebKit decided to change this on its own volition, which sparked some changes in the standard. Not the other way around.


Watch spacing


``(filters=filters) === filters`` does it not?


why is this API name?


I was using your example ``Name`` as in name of the tag


instead of the if/else

var watchStatement = devMode ? fs.readFileSync('src/html/watch.html', {encoding: 'utf8'}); : ' '


I guess there's no ``string. to_boolean`` method you can use?


I think renaming `better.files.File` to `better.files.BetterFile` is probably clearer?

For people like me who convert the java one early on to the Scala one, I would then do this:
```scala
import better.files.{BetterFile => File, _}
import java.io.{File => JFile}
```


@Osmose, I'm just curious, but why not do ...

```coffee
else if ... or text == "~#{path.sep}"
```

... instead of ...

```coffee
else if ... or text == "~/"
```

... ?


The Java `list` returns a `String[]` which is not at all what I want. I want `Iterator[File]` so I can flatMap etc over them e.g.

```
def cat(files: File*): Seq[Iterator[Byte]] = files.map(_.bytes)
```


I'm trying to move away from using Progress Dialogs in favor of a circular progress bar built into the CoordinatorLayout itself. If you add a ProgressBar with id `R.id.progress_bar`, then you can just call the `showProgressBar()` and `hideProgressBar()` methods from `AnkiActivity` without showing a dialog


I guess people can still do that currently if they don't like the `better-files` File to be used by default in their codebase:
Instead of `import better.files._`, they would do `import better.files.{File => BetterFile, _}`


This proof probably needs `abstract`.


Yeah, that v3.create drove me nuts, too.


Done


ah, yes. i see it pulled in too many tests. thanks for fixing that.


Thanks!


@esmithayer I added the history object since we'll need access to pieces for "edited by _____ at ______" in the header. I just copied this from the history route for starters. Is there a more elegant way?


Merge


I think i do. Currently fusion (without my patch) works only like `DIAL_STRING_BASED_ON_USERID=false`. An because no one excapt me did not use it I made this option as global. I think about using some way domain specific settings but it can be done in separate PR


That rename sounds like a good idea.

In the NIO APIs, `Files.newDirectoryStream(path)` returns an stream of `Path`-s. It probably doesn't make sense to spend too much time discussing `java.io.File`'s legacy APIs.


não tinha me dado conta, foi no xgh, ja arrumei


Seems like a good time to start naming the routes a little more consistently eh? This is the start of #215


Why do this?


I needed the `form` object in order to grab phone numbers. Is that right?


They'd already been backported with different commits. The conflict was in code, something around the `size_t FParamIndex = 0;` IIRC, but trivial to resolve.


@OpenLocalizationTest come on you idiot!


This looks like it's already merged


If we do a  --reverse-tags we will have to fetch all volumes matching the passed in tags , and then do a difference operation against all the volumes. We can simplify (and optimize) the logic by using actual set operators but we will still need to run get_all_volumes twice.

I don't think it would be a good idea to try to shoehorn the --reverse-tags functionality in just a single get_all_volumes call (and then filter the tags from there) - the reason being that while it may appear you are optimizing codes (by just doing a single call), you will actually suffer a penalty by having to go through each individual volume and then going through each individual tag in that volume to test for a match - if you have 500 volumes and each volume had 5 tags, you would be doing a comparison 2,500 times as compared to the current setup where you would only be doing a comparison 500 times.


Should be contains


I don't think this String is necessary, just use the standard message `R.string.full_sync_confirmation`


done


eewrewr


@retronym : I know way too much about NIO after the past week than I would like to admit. `File.newDirectoryStream` is surprising for simple listing purposes. What it does is essentially `listRecursively` and not only its own children which is what I would expect from `file.list`. Also, even if I put a depth param and use `File.walk(this, maxDepth = 1)` to list direct children, it includes its own path too unlike the legacy `java.io.File` one e.g. listing an empty directory using NIO returns 1 element instead of 0. So, this is one of the few places in this library, I actually use the least surprising legacy `java.io.File#list`.

But, back to the original point. If I want to include an unsurprising `def list: Iterator[File]` to BetterFile (and assume people know/don't know that is `String[] list` in `java.io.File`), I have to essentially either make BetterFile not be automatic implicit from java-File (which is how it is presently) OR call the Scala list something else (e.g. `listFiles`).


Wha....


test


it's `You're a squ/kid now!`, not `You are squ/kid now!`


"Reload" doesn't do anything except re-open the logfile, which under systemd doesn't exist (it logs to stdout, which ends up in the journal).


A pid might be active on multiple cores, opening multiple files at once. Why not key off of {pid, cpu} so that the update/lookup/delete sequence is more resilient and per-cpu.


Why change the `&&` to `||`?
```
if (t.GetConstructor(Type.EmptyTypes) == null && t.IsAbstract)
```
After this change, all class without empty constructor won't be able to be used in lua.


I don't get it - sure you can convert it, but what would be the point of doing so when all you are doing is checking if it was passed in or not (and only doing it once)? We can do:

if bool(self._reverse_tags):

For syntactic sugar lol


And since you're using the map just to pass data from the kprobe to the kretprobe, simply key off of cpu.


how about use AIQ_CPF_PATH


readyState 2 is CLOSING https://developer.mozilla.org/en-US/docs/Web/API/WebSocket#Ready_state_constants


you'd better check if id is an object id with `bson.IsObjectIdHex` before using it.


always wise to limit your result set, you may get skip&limit with query params (with your internal sane limits).


Just a tip: I locate all these legacy sdks in /Applications/LegacySDKs

Then I use this little script (I call it legacysdks.sh)
```
#!/bin/bash
sdkrepo=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/
for sdk in /Applications/LegacySDKs/*.sdk; do
   sudo ln -s $sdk $sdkrepo`basename $sdk`
done
ls -l $sdkrepo
```
After an Xcode update (where all previous SDKs were replaced), I just run the script.

It creates symlinks in /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/ to the SDKs in /Applications/LegacySDKs.

Saves some lengthy copies...


Actually, I'm not sure if either of those optimizations will work, since the open call may migrate cpu in between the start and return.


Forwarding www.gdgkk.info to gdgkk.info


see #748


better to validate your inputs before going to database.


ここだとダメじゃない？

> To invoke strict mode for an entire script, put the exact statement "use strict"; (or 'use strict';) before any other statements.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode


I have installed the latest flask-bootstrap (3.3.5.6) through pip. However, I found this change was not included. As a result, if run the sample_app，it will raise the RuntimeError as well.


And the project where all this code was copied from?


for safe concurrency get session from your db, then close it when you are done.

```
	session := m.GetSession()
	defer session.Close()
        session.DB("").C(collection).Find(bson.M{}).All(&users)
```


Add the announcement :smile:


What?


Oh! I understand. The latest version was released on 24 Jul, and this issue was closed on 31 Jul.


Hi @spcui

Where will print the "errors" you defined ?


No, definitely not!

`volumes = self._conn.get_all_volumes(filters=filters)`

Means we are passing filters to the filters parameter of the get_all_volumes function, which is not equivalent to (and will definitely error out when called as):

`volumes = self._conn.get_all_volumes(filters)`

The get_all_volumes function is defined as:

`def get_all_volumes(self, volume_ids=None, filters=None, dry_run=False):`

(see https://github.com/boto/boto/blob/d6eba4e36c3d9eaf677695879e4f5e9f053834ec/boto/ec2/connection.py)


Cool :bangbang:


Can you update the cron job to run at around 12:30am? The bloomberg numbers don't update until then, so the 5pm still grabs the yesterday's numbers.


typo, s/suported/supported


This should be before `boost/optional` include. Don't pollute global namespace needlessly.


The static methods shouldn't be used in Java EE. Do you have special reason for that?


Aw, shucks. This happens from time to time.


This does work for new users that haven't authenticated with the API yet.  However with a user that already has authenticated with the API, 1 of 2 errors is thrown:
- _FacebookTokenError: This authorization code has been used._
```
FacebookTokenError: This authorization code has been used.
    at Strategy.parseErrorResponse (u:\dev\blog-service\node_modules\passport-facebook\lib\strategy.js:199:12)
    at Strategy.OAuth2Strategy._createOAuthError (u:\dev\blog-service\node_modules\passport-facebook\node_modules\passport-oauth2\lib\strategy.js:345:16)
    at u:\dev\blog-service\node_modules\passport-facebook\node_modules\passport-oauth2\lib\strategy.js:171:43
    at u:\dev\blog-service\node_modules\passport-facebook\node_modules\passport-oauth2\node_modules\oauth\lib\oauth2.js:176:18
    at passBackControl (u:\dev\blog-service\node_modules\passport-facebook\node_modules\passport-oauth2\node_modules\oauth\lib\oauth2.js:123:9)
    at IncomingMessage.<anonymous> (u:\dev\blog-service\node_modules\passport-facebook\node_modules\passport-oauth2\node_modules\oauth\lib\oauth2.js:142:7)
    at IncomingMessage.emit (events.js:129:20)
    at _stream_readable.js:908:16
    at process._tickCallback (node.js:355:11)
```
- _Error: Failed to serialize user into session_
```
Error: Failed to serialize user into session
    at pass (u:\dev\blog-service\node_modules\passport\lib\authenticator.js:271:19)
    at Authenticator.serializeUser (u:\dev\blog-service\node_modules\passport\lib\authenticator.js:289:5)
    at IncomingMessage.req.login.req.logIn (u:\dev\blog-service\node_modules\passport\lib\http\request.js:48:29)
    at Strategy.module.exports.strategy.success (u:\dev\blog-service\node_modules\passport\lib\middleware\authenticate.js:228:13)
    at verified (u:\dev\blog-service\node_modules\passport-facebook\node_modules\passport-oauth2\lib\strategy.js:179:18)
    at Object.<anonymous> (u:\dev\blog-service\routes\index.js:36:13)
    at Object.tryCatcher (u:\dev\blog-service\node_modules\sequelize\node_modules\bluebird\js\main\util.js:26:23)
    at Promise.module.exports.Promise._settlePromiseFromHandler (u:\dev\blog-service\node_modules\sequelize\node_modules\bluebird\js\main\promise.js:501:31)
    at Promise.module.exports.Promise._settlePromiseAt (u:\dev\blog-service\node_modules\sequelize\node_modules\bluebird\js\main\promise.js:577:18)
    at Promise.module.exports.Promise._settlePromises (u:\dev\blog-service\node_modules\sequelize\node_modules\bluebird\js\main\promise.js:693:14)
    at Async._drainQueue (u:\dev\blog-service\node_modules\sequelize\node_modules\bluebird\js\main\async.js:123:16)
    at Async._drainQueues (u:\dev\blog-service\node_modules\sequelize\node_modules\bluebird\js\main\async.js:133:10)
    at Immediate.drainQueues [as _onImmediate] (u:\dev\blog-service\node_modules\sequelize\node_modules\bluebird\js\main\async.js:15:14)
    at processImmediate [as _immediateCallback] (timers.js:367:17)
```


@OpenLocalizationTest here we go!


:disappointed:


Any reason we cannot include the compiler-support for these?


数据库表统一用复数


when i use node to get data form hdfs whith hive cli, i met this problem:
```
stdout: FATAL ERROR in native method: JDWP No transports initialized, jvmtiError=AGENT_ERROR_TRANSPORT_INIT(197)                                                 [63/1856]

stderr: ERROR: transport error 202: bind failed: Address already in use
ERROR: JDWP Transport dt_socket failed to initialize, TRANSPORT_INIT(510)
JDWP exit error AGENT_ERROR_TRANSPORT_INIT(197): No transports initialized [../../../src/share/back/debugInit.c:750]

stdout: FATAL ERROR in native method: JDWP No transports initialized, jvmtiError=AGENT_ERROR_TRANSPORT_INIT(197)
```
I thought that might be is the pid used by node, then transfer it to hive, so I use buffspawn(https://www.npmjs.com/package/buffered-spawn) instead of child_process.exec(), it also can't work. oooooooH


@pranavkm @anurse @davidfowl @victorhurdugaci  This unblocks the travis build but this should be designed to support multiple cases of config. In the latest v3 NuGet.exe they expect `NuGet.config` but in v2 they expected `NuGet.Config`.

@pranavkm did some work to support this for the settings file itself [here](https://github.com/aspnet/dnx/commit/9dca281a9594eec292c0bb502e31535900a23223) but making this do something similar should probably be designed/done by someone more familiar with its call sites.

This is a half-done change :smile:


this didn't work:
```
}).catch(function(error) {
    done(error);
}).then(function(user) {
    done(null, user);
});
```


your singleton implementation looks quite alright. Is there anything wrong with it?


There's just one small issue with this approach - now we have a client library that doesn't know how to close the connections it creates... :-) Not a big issue, just something to be kept in mind.


Versión e-CTR 0.1 estable!


That is exactly my point. Moving the exception handling up and putting it this way as you suggest:

```python
try:
  filters['tag:%s' %f] = eval(filters.pop(f))
  ...
except Exception:
  pass
```

Will not work. `filters['tag:%s' % t]` will not exist if the call to `eval` throws and exception.

The reason you want to run the following code:

```python
filters['tag:%s' % f] = filters.pop(f)
```

Outside the `try... except` block is to set the `filters['tag:%s' % f]` key and value. You then want to attempt `eval` it's contents after, hence:

```python
filters['tag:%s' % f] = filters.pop(f)
try:
  filters['tag:%s' % f] = eval(filters['tag:%s' % f])
```

If the `eval` throws an exception (a non "OR" format tag has been passed), filters['tag:%s' % t] will exists with a valid string value as we want.


This is required to build a working boot.img


Add an ID and a CLASS names.


This is a comment, buddies!


I just want a div that I can toggle the image on in using an attribute.
like:
html:

css:
.check[enabled] { background-image: url("../checked.png") }


Hi, i am going to release 2.1.0 today. You could use this feature after this release.


thanks, corrected


Added 2 fonts 'HelveticaNeueLTCom-Md.ttf', 'HelveticaNeueLTCom-MdIt.ttf'


Thanks for the report. I wasn't able to compile test the changes on anything else but win32 and there our pre-compiled headers screw up the required includes.


This folder will contain everything related to client


プッシュ通知の宛先のみ自分を含めるかたちにしたいため、EipTMessageRead のレコードは現状維持で、recipients のみに追加する方法でもいけますでしょうか。


Tarea


... or both :)


Instead of ignoring them, they should be fixed. I spent quite a bit of time in the past days to fix them and make the build green. Actually, it is usually not much to do - it is just a bit hard to find the real error amongst all the warnings. Searching ": error:" in the Maven output helps.


[]byte("PASS\n")


Hum. @mohierf did you not revert my changes to make the webui compatible with pymongo 2?


fixed


yyy


noted


README FOR THIS BRANCH OF CHANGES:

Apologies to all if this branch looks terrible. I committed the entire changes I made the night before without realizing I could not undo the changes. I will list all the changes I have made to each branch in this comment screen and which files are affected by the changes.

1. Added a sortFile and wordSearch method to sort the file in TextBuddy,java according to alphabetical order and to search for which words are found in which line.
Files Affected:
TextBuddy.class
TextBuddy.java

2. Changed the reading of strings in userCommand method to not use (.contains) but rather be switch cases [Reason: Based on feedback provided]
Files Affected:
TextBuddy.class
TextBuddy.java

3. Changed the methods in TextBuddy.java to reflect the switch cases (i.e. add method appears first followed by delete, search , display, clear, sort and lastly exit) [Reason: Personal organisation of program]
Files Affected:
TextBuddy.class
TextBuddy.java

4. Removed Magic Numbers in TextBuddy.java methods addToFile, deleteLine, searchWord [Reason: Based on feedback provided]
Files Affected:
TextBuddy.class
TextBuddy.java

5. Added three Junit test cases to debug the TextBuddy.java commands [Reason: To debug newly added functions wordSearch and sortFile]
Files Affected:
TestCase1.class
TestCase2.class
TestCase3.class
TestCase1.java
TestCase2.java
TestCase3.java

6. Added a Test Suite for the three tests cases to run all three tests simultaneously [Reason: To debug TextBuddy.java]
Files Affected:
AllTests.class
AllTests.java

I hope this clarifies this mega updates and apologies!
Please feel free to leave comments if you need more info!

Sincerely,
David Chong


If that's so, it's a mistake! `view.$el` shouldn't be equal to `$el`. Well, the issue being in the test app, it has to be fixed. However, `if ($el) $el.html(this.mainView.$el);` is what is done in the full app, and some other modules rely on that. The right fix may just be to remove `$("#tab-content")` line 38.


Merged @vrld s shine


added


That change is elegant... so simple.


Well, it didn't work anyway :)

But besides that, all those Javadoc warnings look not very useful to me. So we did not describe why a certain exception is thrown. Who cares?

I get about 50 such warnings for the core module alone.

I agree that we need better documentation in TC, but I would like to have a stable build until we get there.


not sure this is right, have to call smbc_set_context or the old smb interface is not setup.


I would swap these conditions, put err != nil and all the error handling in the if.  It increases the indent by one, but I think it would be clearer that the touch and exit are the normal case.


ha fixed


This is extremely helpful, thanks!


shoot, yeah, it is. any ideas on how to fix?


The warnings are not critical - they don't make the build fail.

But you're right, they are a bit annoying.

Then, when I look a the code and see many "param" and "throws" declarations that have no description whatsoever, I also wonder why wasting space and adding these at all? They can also be removed without harm. In many cases, I did just that when I fixed the real errors - but only in the vicinity of the errors - not in general. A function "remove useless javadoc" in Eclipse would be nice ;)


`Files.newDirectoryStream(path)` is not recursive.


Renamed `get'` to `resolve` and added a comment


Because I'm dumb and missed that when I first put up the fix. :P Luckily I got it right in https://github.com/Osmose/advanced-open-file/commit/b91e3cda506cea6ad01906fecb33d62c134d9e4e#diff-dbaad42ab7eb5ec513d25b1f050443dfR246.


Addresses code review feedback from @SToub.


One final point: the nice thing about `Path` is that is has a pretty small API, so it leaves lots of room for extension methods.

![image](https://cloud.githubusercontent.com/assets/65551/9953082/bb293e0e-5e20-11e5-98dc-b9350e391f27.png)


But the current build seems to fail only because of this.
Or I cannot see the real error between all the javadoc stuff.


Nice!


Shouldn't this be package main?


You should expand "HS" ro high school in the footer.

Also you should remove the footer.


@luiges90  Military.ArrivingDays这个我也储存在Military表了，可是地图上的时间跟实际时间不一致，让人以为运的兵都消失了


was just about to PR this once again :)


@drewwells Can you be specific as to what project and what parts are copied from?


done, will be visible with the next release


Thanks!


I was searching for that one. Seems I dropped it accidentally or messed up a rebase


;) That's why I said to search for `: error:` (written exactly like that). It will take you directly to the error(s):

```
[ERROR] /home/svc_jenkins/workspace/DKPro Text Classification Framework (GitHub)/dkpro-tc-core/src/main/java/de/tudarmstadt/ukp/dkpro/tc/core/task/uima/ValidityCheckConnectorPost.java:36: error: reference not found
[ERROR] * UIMA analysis engine that is used in the {@link ValidityCheckTask} to test error conditions on
[ERROR] ^
```

Here `ValidityCheckTask` cannot be resolved. Maybe it is not imported. Maybe it comes from a module that cannot even be imported because it would cause a circular dependency. If the latter is the case, you might consider using `{@code ...}` instead - JavaDoc will not complain then, but it will also not be rendered as a link and there will be no warning (ehem error) when somebody does refactor the name and forgets to update this instance.


Whoops! :fearful: Sorry I missed the update! :+1:


Näkyykö jossain?


线程池管理类


> @stoeffel this is RxJS-specific – but I hope it fits here! I should’ve opened a PR to discuss it first. Feel free to revert the commit BTW.

No need for a PR looks good to me. Will have to reed it, now that I have some time :smile:


What is updated in the enrollments? Jobs are rescheduled?


We should be careful about those, though, and only use it in examples such as the one in your commit message...


Returns


It's decoding for a reason. Seems odd to unencode a specific decode? What's the problem with Asian languages?


Add a dot on the end


I thought this would be a pull request and you could review it but NO. . Damn cowboys. ;^)


+FaMail是一个家庭信息交互平台


If UrlEncoded is not added, the url looks like

>http://localhost:9666/thread/日本人9年連続イグ・ノーベル

With UrlEncoded, the url looks like

>http://localhost:9666/thread/%e6%97%a5%e6%9c%ac%e4%ba%ba9%e5%b9%b4%e9%80%a3%e7%b6%9a%e3%82%a4%e3%82%af-%e3%83%8e%e3%83%bc%e3%83%98%e3%83%ab/


Why the `instanceof` check is needed? It can be only `String[]` in case of `@DefaultStringSet`, isn't it?


Canola is the right word


Why don't use bootstrap col-xs-3 instead of inline styling?


No need to copy to `HashSet` here.


this is fine - please re-merge master into your branch and then make a pull request (I converted SftpPoller to new-style yesterday too)


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 653](https://teamcity.labkey.org/viewLog.html?buildId=272886&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:27:32
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridColumnSelector: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

##### there are 23 more failed tests, see build details
```


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Ok, I would try to do it the next time


looks like this shouldn't be hardcoded to avoid future errors...


I need to see if that affects & and if it changes it to & in the url?


Which platform? Keyboard commenting works for me with cmd+/ (SublimeText3 OSX). Can you provide more details?

I do notice the comment-line matching (immediately above, L205) has the wrong name however (though commenting should still work). I'll fix that.


Hey kumajaya, do you have hangouts whereby I can contact you?


Damn.. This is converting! need to see if & changes to & amp ; in the url


comment


True... though to be fair, it was already hardcoded


What do you think about putting it into __unicode_trim()?

sub __unicode_trim {
     my ( $self, $str, $len ) = @_;
     $len -= 3;
     my $gcs = Unicode::GCString->new( $str );
     my $pos = $gcs->pos;
     $gcs->pos( 0 );
     my $cols = 0;
     my $gc;
     while ( defined( $gc = $gcs->next ) ) {
         if ( $len < ( $cols += $gc->columns ) ) {
             my $ret = $gcs->substr( 0, $gcs->pos - 1 );
             $gcs->pos( $pos );
             return $ret->as_string . '...';
         }
     }
     $gcs->pos( $pos );
     return $gcs->as_string . '...';
}

and

sub __unicode_trim {
     my ( $self, $str, $len ) = @_;
     $str = ta_mbtrunc( $str, $len - 3 );
     if ( $str =~ /^(.+)(\e\[0m)\z/ ) {
         $str = $1 . '...' . $2;
     }
     else {
         $str .= '...';
     }
}

Am 17.09.2015 um 19:56 schrieb Stephan Sachse:
>
> ???
>
> $i == $#$marked is the last element in the list. so why the loop?
>
> btw: i do not understand what you are doing here :P looks like this
> stuff should go into __unicode_sprintf()
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kuerbis/Term-Choose_HAE/commit/47c4ec7e6df06d18d382619c6984f2ba11e735ad#commitcomment-13294157>.
>


@progresak - nechal jsem tam par poznamek, jinak mi to prijde dost super. Prejmenoval jsem to z ```messages.de``` na ```keyMap```, pac 1) to je mapa klicu a 2) ```messages.de``` je u nas soubor ```messages``` v nemcine (tj. ```messages.fr``` je franina apod.)


Actually it is just enough to iterate over the array i think. Why you are copying here?


Oops. I use qt5.3 constructs and should make them check release, will
try and fix today.
For now build without GC_WANT_HTTP or update to QT5.3 or higher (I'm on
QT5.4)

Mark


Le flashbag n'apparait pas.


please


hmm


added a tag to my release (sorry, this was the first time I used tags on git)


fracking cofh mods


If only Curse's maven access wasn't terrible...
On Sep 18, 2015 1:53 AM, "hilburn" <notifications@github.com> wrote:

> fracking cofh mods
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Drullkus/ThermalSmeltery/commit/3ac2bbf60b339639b5ca71459c868a62836f593f#commitcomment-13303866>
> .
>


I suggested this plugin, because I though that it should set some rules. But I guess that it only defines other rules.. Am I wrong?


Yes. There are no terrible breaking changes right now, so I you'd like, you
could install the latest master branch directly from github.

On Fri, Sep 18, 2015 at 7:28 AM, E Shijia <notifications@github.com> wrote:

> Oh! I understand. The latest version was released on 24 Jul, and this
> issue was closed on 31 Jul.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mbr/flask-bootstrap/commit/e8f0264a79174fe3c85bfb80a5c961f664ef20f7#commitcomment-13303163>
> .
>


Please avoid print statements. Use the "logging" module instead. This is on the "INFO" level.


The blower fan is activated by gcode after printing the second layer.

Please check your slicing program. If you are using Cura or Slic3r, make sure you are using the correct profile (you can download them from BQ's web site).

It could also be a faulty connection or incorrect assembly on your printer.


I just tried '&' in topic title, and it changes to '-' in url.
![default](https://cloud.githubusercontent.com/assets/340794/9953744/0aefcb7a-5e16-11e5-947f-1ab7c57982bc.PNG)


What is this even


When I am trying to view my datasets, nothing is showing up in the lefthand column at all. Or the righthand for that matter. Everything is loading properly otherwise it seems..


Perfect, that will work then :+1:


已调整注释。同时请大神考虑修改Exception 类,注释掉行24 ：  '-1'      => '系统繁忙，此时请开发者稍候再试'   ；因担心其他耦合未commit ，请大神考虑


Now we have dynamic titles for each subpages.


redirect the right subpages


Fix the display with bootstrap CSS, make them align correctly [Done]


This commit makes magit fail on my Windows PC with the following error:

    Debugger entered--Lisp error: (wrong-type-argument stringp nil)
      string-match("^\\([^ ]+\\) \\(.*\\)" nil)
      (let ((output (magit-rev-format "%h %s" (or branch "HEAD")))) (string-match "^\\([^ ]+\\) \\(.*\\)" output) (let ((string49600 output)) (let ((hash (match-string 1 string49600)) (msg (match-string 2 string49600))) (let* ((it (progn (vector ... ... ... ... nil nil nil nil nil nil nil nil magit-insert-section--parent nil)))) (progn (or (and (vectorp it) (>= ... 14) (memq ... cl-struct-magit-section-tags)) (error "%s accessing a non-%s" (quote magit-section-hidden) (quote magit-section))) (let* ((v it)) (aset v 6 (let ... ...)))) (let ((magit-insert-section--current it) (magit-insert-section--parent it) (magit-insert-section--oldroot (or magit-insert-section--oldroot ...))) (catch (quote cancel-section) (if branch nil (progn ... ...)) (magit-insert (concat ... ... " " ... msg "\n")) (run-hooks (quote magit-insert-section-hook)) (magit-insert-child-count it) (set-marker-insertion-type (progn ... ...) t) (let* (... ... ...) (save-excursion ... ...)) (if (eq it magit-root-section) (magit-section-show it) (progn ... ...))) it)))))
      (let* ((branch (if --cl-rest-- (car-safe (prog1 --cl-rest-- (setq --cl-rest-- (cdr --cl-rest--)))) (magit-get-current-branch)))) (if --cl-rest-- (signal (quote wrong-number-of-arguments) (list (quote magit-insert-head-header) (+ 1 (length --cl-rest--))))) (let ((output (magit-rev-format "%h %s" (or branch "HEAD")))) (string-match "^\\([^ ]+\\) \\(.*\\)" output) (let ((string49600 output)) (let ((hash (match-string 1 string49600)) (msg (match-string 2 string49600))) (let* ((it (progn ...))) (progn (or (and ... ... ...) (error "%s accessing a non-%s" ... ...)) (let* (...) (aset v 6 ...))) (let ((magit-insert-section--current it) (magit-insert-section--parent it) (magit-insert-section--oldroot ...)) (catch (quote cancel-section) (if branch nil ...) (magit-insert ...) (run-hooks ...) (magit-insert-child-count it) (set-marker-insertion-type ... t) (let* ... ...) (if ... ... ...)) it))))))
      magit-insert-head-header()
      funcall(magit-insert-head-header)
      (while (and (setq wrapper (car-safe (prog1 magit-insert-headers-hook (setq magit-insert-headers-hook (cdr magit-insert-headers-hook))))) (= (point) (point-min))) (funcall wrapper))
      (let ((magit-insert-section-hook (cons (quote magit-insert-remaining-headers) (if (listp magit-insert-section-hook) magit-insert-section-hook (list magit-insert-section-hook)))) (magit-insert-headers-hook hooks) wrapper) (while (and (setq wrapper (car-safe (prog1 magit-insert-headers-hook (setq magit-insert-headers-hook (cdr magit-insert-headers-hook))))) (= (point) (point-min))) (funcall wrapper)))
      magit-insert-headers((magit-insert-diff-filter-header magit-insert-head-header magit-insert-upstream-header magit-insert-tags-header))


Fix the scaffold by adding a main index page using the cover example from Bootstrap [Done]


No need to show the old way of writing SQL queries in the view.


aa


Hello absimaldata,

Good luck with your tuning attemps for your first SPSA test :-)

I can't say for others, but one rule of thumb I try to follow when I tune on fishtest with SPSA (I do very few such runs as they are very costly) is using STC of 15s/game, with maximum 10.000 games per parameter in the model, and never more than 40.000 games.

Hope this helps,
Stéphane


Template Inheritance: http://flask.pocoo.org/docs/0.10/patterns/templateinheritance/


This line should not be there. It's based on the output of a prototype implementation which printed this value in kilobytes. The final implementation prints this in megabytes, so no conversion is necessary.


SQL Expression in SQLAlchemy: http://docs.sqlalchemy.org/en/rel_1_0/core/tutorial.html


Why?


I will remove the line and the `KB_SIZE` constant (not needed anymore) and attach the new package to bugzilla. Thanks.


why not just injecting `return url` from the beginning? See the issue at https://github.com/shaarli/Shaarli/issues/308#issuecomment-141366230


I mean, split replies is configurable on the homepage, so the fact that they're not split isn't noticeable unless you configure to not display them...


I use unsigned char because:
- you use 4 time less memory (disk and when the program run in RAM): So you can potentially handle 4 times more images on the same computer.
- accuracy loss compare to float descriptor is not a big deal

If you need to extract float descriptor, just create a new regions SIFT_float_Regions and customize the SIFT_Image_describer to detect dynamically the regions type with dynamic_cast.


@maethor : no I do not think so ... but these changes are probably only in the develop branch. As far as I remember ... the compatibility is broken only because of the `connect=True`parameter when instantiating MongoClient.

The main problem is to be consistent across all the modules that are using MongoDB ...


redirect behavior changed with https://github.com/shaarli/Shaarli/commit/35c2c4db5b5179e571023ab7fa7d3ecc03c85391


No need to compile the patterns every time. it's a heavy operation and should be done only once!


If this code is invoked immediately after shutdown is requested (whether it is fast or slow) condition would be evaluated as
if (srv_fast_shutdown != 0 || n_purged == 0) ... assuming srv_fast_shutdown = 0 first condition would be false so it would be if (false || true) ... so true.

So all shutdown would be treated as fast-shutdown. We are examining n_purged value even w/o setting it once.


Octree (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1873 of 1874 tests passed)

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


you can use "mov esi,esi" insted of "movsxd rsi,esi".
it will make the shell code shorter in 1 byte.
also, the same problem on bind tcp x64 shell code.


We had a bunch of feedback from people who didn't understand what they were
or why they existed. It's confusing unless you're actively posting
site-to-site. It's still a configuration option.

The homepage option is likely to change too - people finding that confusing
as well.

On Fri, Sep 18, 2015 at 12:12 AM, Marcus Povey <notifications@github.com>
wrote:

> I mean, split replies is configurable on the homepage, so the fact that
> they're not split isn't noticeable unless you configure to not display
> them...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/idno/Known/commit/7e8e546bc004002b2f869101291c796e2a1c6832#commitcomment-13304081>
> .
>

--
Ben Werdmuller
<http://goog_1933028737>
benwerd.com | werd.io

+1 (312) 488-9373


you should remove the `_int` after define here. as this will tell the AboutLibraries that it can automatically show up your library if it is there. also remove the classPath. please also do this for the other lib.


Thanks for the fast fix of #244!


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (806 of 806 tests passed)

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


Same as above. If shutdown (fast or normal) it will use the batch_size = 1.


Just adding a line.


Before deployment we always can do squashing migrations
https://docs.djangoproject.com/en/1.8/topics/migrations/#squashing-migrations


Closes #108


This if block may be skipped due to || count % TRX_SYS_RSEGS ... condition.
That would effectively mean n_pages_purges will be 0 when the while loop condition is evaluated.

while (n_pages_purges > 0 ... will be evaluated to false and will cause while loop to exit.


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1879 of 1880 tests passed)

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


Sure, but let's do it in sepparate patchset.


@ccx-testerbot runtests master


Thank you for the request. It has been added to the queue (id: 93). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/93) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Merge this part with the part 2-1 that calls this method.


Nie podstawiaj tego tutaj, prześlij ten obiekt po stworzeniu klasy fragmentu/dialogu.


Ok, thanks! We are trying to decide which representation to use because, either way, there are some changes to make (and new bugs to discover...).


Thos call is trying to truncate rseg so may be we can skip this completely if we running in fast-shutdown.

Note: All the purge activity delayed at shutdown will be carried out when next time we start the server which effectively means initial starting load on restart may see some effect. But I assume that is understood by end-user.


usun komentarz


Puste metody też usun


I understand, good luck


fixed #26


Wszystkie zmienne, które nie są ustawiane z zewnątrz tej klasy powinny być private


Usun tego typu Logi


Print some log here to know why cannot create a new call.


Cyanogenmod ---> slim


No for now, because few years ago, the lib was adapted for prestashop. But we really think about that.


Ditto.


1. Reviewed. Relevant comment open in codebase.
2. Can we have a commit message explaining why we thought of do so and what solution we are going for.
3. Same comments can be added to code-base for clarity.
4. Have we tested this patch using the test-case that brought us to this problem.
 Also, do we plan to run Jenkins build ?


This breaks if the caller is passing the XML as bytes, not unicode, which Jenkins Job Builder does.

```
Traceback (most recent call last):
  File "../jenkins-job-builder/jenkins_jobs/cmd.py", line 331, in <module>
    main()
  File "../jenkins-job-builder/jenkins_jobs/cmd.py", line 172, in main
    execute(options, config)
  File "../jenkins-job-builder/jenkins_jobs/cmd.py", line 314, in execute
    options.names)
  File "/home/dcallagh/work/jenkins-job-builder/jenkins_jobs/builder.py", line 334, in update_job
    self.jenkins.update_job(job.name, job.output())
  File "/home/dcallagh/work/jenkins-job-builder/jenkins_jobs/builder.py", line 125, in update_job
    self.jenkins.reconfig_job(job_name, xml)
  File "/usr/lib/python2.7/site-packages/jenkins/__init__.py", line 621, in reconfig_job
    self.jenkins_open(Request(reconfig_url, config_xml.encode('utf-8'),
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 5175: ordinal not in range(128)
```

The caller *should* be passing the XML as bytes, since XML serialization is defined in terms of bytes and not Unicode codepoints.


@tchaikov . Remove it. Thanks your patience.


@darxriggs Thanks for spotting. I have a fix here: https://github.com/elastic/filebeat/pull/24


take the meat out of the pot and set aside


Yo


careless me :P


save the fluids but throw away the onions and whatever else gets caught in the strainer


go easy with the fluids and add by spoon a little at a time, mix together with the meat and repeat until you have your desired juicyness


Please merged


An humble note of mine: I will probably need help to backport all the recent commits to Classic core when time comes because most of them are beyond my knowledge in C++.


Merged as https://github.com/alexmarsev/sanear/commit/259346dc1afbb6eb37aa41643fa345a6d10e112f, thanks!


fixed #23


I guess OP_RETURN might have some use cases where all outputs have 0 value.  We should ask @emeth about that.  I don't think it would be a good idea to diverge from bitcoin core here.


This is also weird because this normally launches chrome, safari and firefox with each running tests.

Which OS are you running?

> Le 17 sept. 2015 à 21:47, Jiří Procházka <notifications@github.com <mailto:notifications@github.com>> a écrit :
>
> I also think it's good to mention how to start the tests, as when you run karma start it just starts the web server and connecting to it there is no button on anything to start the tests. I had to search Stack Overflow for the run command.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Anchakor/plastiq/commit/34cd284dbc4f1c1e134209b4f83e490c67a6816e#commitcomment-13296339>.
>


Thanks for fixing this!
Regards Hauke


Faut dromir la nuit! :zzz:

Joli en tout cas tu as bien boosté... C'est dommage qu'il y ai pas un moyen simple de bosser en parallèle et discuter en même temps (histoire de pas se marcher sur les pieds... :smile:


TeamCity pull requests :: netty [Build 217](https://garage.netty.io/teamcity/viewLog.html?buildId=1450&buildTypeId=pulls_netty&guest=1) is now running


NOTE: Default in 5.1 libfmas.so: 35,30,110


@jordimassaguerpla I don't understand the purpose of this line, I cannot find `hosts.in` inside of our codebase.

Also I think this can be removed given hostnamectl takes care of updating `/etc/hosts/`; moreover this should be done only when a different hostname is specified via parameter to the `firstboot` script. It should not be executed all the times.


TBH I didn’t :) But I can definitely recommend the [gtor](https://github.com/kriskowal/gtor).


Why do this?


TeamCity pull requests :: netty [Build 217](https://garage.netty.io/teamcity/viewLog.html?buildId=1450&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:29


NOTE: Default in 5.1 libfmas.so: 35,40


Well it's certainly feasible, as you say it's part of the fit. But I'm not convinced it follows that it should therefore be part of the same output image. It is different in kind to the DT coefficients (for one, you need to take its exponential before storing), and is very unlikely to be used as part of the same processing chain. I'd expect people might want to use the b=0 output as input one thing, and the DT for another. Having them be part of the same image forces an additional file split for these cases. So I personally think it would add more confusion than anything...

The other point (although a bit speculative at this stage) is that we were talking with Ben about whether the WLLS fit should be used in the tracking also. We thought the best thing to do would actually be to allow using the DT images directly, and implement proper geodesic interpolation, which would actually be pretty simple (and faster than the current approach). That would then allow _any_ DT image to be used, regardless of how it was produced. It would be convenient to keep the exact same command-line usage as currently, which means we'd need to be able to differentiate between DT and DWI images. Currently this wouldn't be a problem since a 6 volume image can only be a DT image (you'd need at least 7 DWIs to estimate the tensor). Obviously this assumption would no longer be guaranteed if we add an extra volume to the DT image.

On the other hand, I'm not sure the argument above is necessarily going to hold forever: we might eventually remove the ability to track on the DWI directly from `tckgen`, since it currently only implements the most brain-dead approach - and it's not even the same algorithm as what's used by default in `dwi2tensor`...

But regardless, I think my main argument against this is that in practice, people will want to use them separately.


This won't work for pointer-to-, slice-of-, or any other composite of external types.


NOTE: Default in 5.1 libfmas.so: 15


Since we already have a check for `vcall_count >= 2`, It seem there are only two cases here, `1` or `0`. Then `if-else` seems enough.


Hello,
If you ever finish integrating your `startCollapsed` argument, I'd be glad to merge this change into the original repository

Glad my code is helping others


Great! Thank You. (But on my site it works only in expert view, not in normal view. )


NOTE: Default in 5.1 libfmas.so: 0


shouldnt have "PeopleArray" an unserialize function here? so that its wrapped properly inside the class?


shouldnt have "PeopleArray" an serialize function here? so that its wrapped properly inside the class?


This commit seems to be missing in develop.


This should also fix some logical mistakes that were induced by the confusion between count and capacity.


`cid`!!


right :+1:


We returns `-1`, but the AVoiceCall object is created, will it cause any unexpected behavior?


Thanks brotha! :)


OK!


Sure! Let me land at work and I will take a look


This commit is the first step. In the second commit,  I replace `DateTime.DateTimeFormat` by the property.


@jimhester: I've been doing some work on memoise, very similar to what we have done for withr: The memoizer now has the same interface as the memoized function. I came up with the `eval(bquote(...))` pattern, but you used `substitute()` in withr. Is there a reason to prefer one over the other, or are they essentially the same? Thanks for your feedback!


This is there to not run vendored dependencies tests :sweat_smile:


Should this happen only in debug mode?


ping @robertab fint va?


It does, see https://github.com/flarum/core/commit/efca923d307fe94c3a8586ff18ff1d104712d2d7#diff-5b6b1d350c7cbc5af3e75469ec85225bR234 :)


Right, but I was saying that we should put two separate `.gitignore` files: one in the root directory and one in the `iis-workflows` directory. Both of them would define the same exclude pattern:

    /*/target
    /target

This would cover all levels of directories that you mentioned and would exclude `target` directories more precisely than the pattern that you suggested in the comment above. To get 100% precise exclusion, we would have to place `.gitignore` file with

    /target

content in all directories that correspond to Maven projects (i.e. those with `pom.xml`).


Ah, glossed over that. Thank you!


@gavinking : even when running quickFixes in a dirty editor, and even doing that during a full Build, I can't reproduce that. Placing a conditional breakpoint that stops when the `EditedSourceFile` is not equal to the related `ProjectSourceFile`, I never stop in the breakpoint.

So I guess this was happening in very specific scenarios involving concurrency (changes inside a module descriptor that just triggered a full build ?).  Anyway, with the new lightweight synchronization of the CPC typechecking this scenario should *not* happen anymore.

Note that a `ProjectSourceFile` and an `EditedSourceFile` will still be equal, even if the underlying `VirtualFile`s are *not* of the same type (`IFileVirtualFile` vs `SourceCodeVirtualFile`), as soon as the *`path` and `fileName` are the same*, which is the case if they are *constructed with the same source Directory and source-relative path*, which is *normally the case in the CPC for project source files*.


Doesn't this break editing standalone .csx files as well?


1: M111 unrecognized component name: ext


Yes, it seams like it only brings some angular-specific rules.


*Well actually*, another idea: We were planning to add detailed error descriptions to the JSON-API response as part of #118 if debug mode is enabled.

In the frontend, we should only check for the existence of that field (`detail`?); and if it's there, show the button.


Eep!

Thanks for the catch.


例子9：

    （1）任务1：在main()里创建任务1，优先级为1。任务1 运行时，以优先级2 创建任务2。现在任务2 具有最高优先级，所以会立即得到执行。
    （2）任务2 什么也没有做，只是删除自己。可以通过传递NULL 值以vTaskDelete()来删除自己，但是为了纯粹的演示，传递的是任务自己的句柄。
    （3）当任务2 被自己删除之后，任务1 成为最高优先级的任务，所以继续执行，调用vTaskDelay()阻塞一小段时间。
    （4）当任务1 进入阻塞状态后，空闲任务得到执行的机会。空闲任务会释放内核为已删除的任务2 分配的内存。
    （5）任务1 离开阻塞态后，再一次成为就绪态中具有最高优先级的任务，因此会抢占空闲任务。又再一次创建任务2，如此往复。


Codejam template code.


Опять бесполезный комментарий...


Неочевидное название..


ack, will invert.


@flavour This gives me the list of the available sync repo with the check box (as I have used that)...How can I insert the 'Create Repository' link in the custom form? How about having a drop-down with the available repo and 'Create Repository' link as the generic one?


TeamCity pull requests :: netty [Build 218](https://garage.netty.io/teamcity/viewLog.html?buildId=1451&buildTypeId=pulls_netty&guest=1) is now running


Предлагаю все-таки разделить сервисы и дао в отдельные конфиги, а не смешивать в одном. Впрочем, для такого маленького проекта может быть это и не обязательно.


Just pushed a fix, sorry about that !


On 18/09/15 08:06 AM, Richard Eckart de Castilho wrote:
> Instead of ignoring them, they should be fixed. I spent quite a bit of
> time in the past days to fix them and make the build green. Actually, it
> is usually not much to do - it is just a bit hard to find the real error
> amongst all the warnings. Searching ": error:" in the Maven output helps.

You can change your Eclipse settings so that JavaDoc problems are
decorated as warnings or errors (Window > Preferences > Java > Compiler
> Javadoc).  This makes it easy to find them when coding rather than
when compiling.


TeamCity pull requests :: netty [Build 218](https://garage.netty.io/teamcity/viewLog.html?buildId=1451&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:25


Эта строчка не нужна. Предыдущая тоже вызывает вопросы.


Изменения по Spring Security должны были идти в отдельном коммите.


Curious to know others' thoughts, or how it's handled in other packages. If other software requires both, outputting the fit with both makes more sense than outputting separate images and having to concatenate; whereas wanting just _S0_ from the tensor fit is a volume split not entirely dissimilar to pulling _b=0_ images out of a DWI series, which many users will be accustomed to. But if other applications expect 6 volumes, it's best left as-is.

Having the _S0_ encompassed in the image would also be of assistance if a tensor overlay were implemented in `mrview`, allowing more options for glyph scaling. But again, speculative.

> ... whether the WLLS fit should be used in the tracking also. We thought the best thing to do would actually be to allow using the DT images directly, and implement proper geodesic interpolation...

Agreed.

> It would be convenient to keep the exact same command-line usage as currently, which means we'd need to be able to differentiate between DT and DWI images.

Well that assumes you want to be able to support tensor tracking on _both_ pre-calculated tensor estimates _and_ on the raw DWIs, which to me would seem unusual. And even then, you could just check for a valid DW gradient scheme. My earlier thinking was the other way around: If an application is loading 'orientation data' and gets an image with 6 volumes, it doesn't know whether it's a tensor volume or _lmax=2_.

> On the other hand, I'm not sure the argument above is necessarily going to hold forever...

... Oh. You said that already. :-/


Any reason not setting the connect timeout? See #20.


I am a user which uses your source, GUITabPagerViewController on iOS. First of all, I am pleasure for you to make it easy to create tabbed view with this open source. i am writing this email to ask custom usage. I have a view controller that has 3 tabs from GUITabPAgerViewController. As you know, when sliding to right from the third tab, it will bounce back. But I want to make screen change from the third tab to the first tab. How should I make this?
Thank you in advance.
Sincerely.


TeamCity pull requests :: netty [Build 219](https://garage.netty.io/teamcity/viewLog.html?buildId=1452&buildTypeId=pulls_netty&guest=1) is now running


Why do this?


could this work?

https://github.com/xbianonpi/xbian-package-repo/commit/ca77f37a164e47481cb3b52e8add0b54ca93bbc7


TeamCity pull requests :: netty [Build 219](https://garage.netty.io/teamcity/viewLog.html?buildId=1452&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:42


Reviewed


For the islands i think we need to have genlayer.java involved somehow.

Im still trying to do that but after 6 hours screwing around i didnt got far :P

We need to be able to fix that cold to hot shit. or better said we need to use groups like hot cold normal snow.


Looks good for the first iteration.


Hmmm.. I wouldn't say that inheritance is magic ;)


1: M111 unrecognized component name: ext


This commit broke the build.


Ah yes. Well, for now I would suggest using goimports then, as I don't have time to write an ast.Expr walker.


.... нужно все-таки стремиться к атомарным коммитам.

Попробую привести примеры:
1) вам понадобилось откатить этот коммит, т.к. выяснилось, что фильтр для Spring Security все-таки нужен и без него не работает. Но, т.к. в этом коммите содержатся также изменения по бинам, то вы откатите и эти изменения (либо придется откатывать вручную).
2) приложение не работает, вы обнаруживаете коммит, начиная с которого оно перестало работать. Но в коммите куча различных изменений и непонятно какое имменно из них сломало приложение.


The test is fine, but did you want to test `createPurchaseWithPaymentControl` as well (I see there is already a test for the error scenario, but not the failed payment one)?


Изменение не относящееся к коммиту.


It's positions form. For both new and edit position. And using `#try` is to make sure that if it's a new position it returns false instead of nil.


Что это и зачем?


@ndeloof  It may be good to not have the spell checker error on `jenkinsci` and `jenkins` on the image :-)


1307 Send GetResultTemplate test


This has been reverted already, but note that clang adds this flag *only* when linking sandbox ABI code.


this would be nice


Wir müssen schauen, wie wir ohne die beiden Imports auskommen. Per Design ist die Hive-Plug-in-Suite vollständig unabhängig vom EPC-Modeller und vice versa. Dies ist hiermit gebrochen was nicht gewünscht.


Rather use if "host" in mastedr_instance_conn_info: raise exception


Aus den zuvor genannten Gründen gehört diese Klasse eigentlich ins Attributes-Plug-in.


Feel free to update the client version, too


whitespace problem? :stuck_out_tongue_closed_eyes:


Unbedingt getAdapter() anstelle eines hard cast verwenden!


It looks like you have forked from recent master revision and then have overwritten it with some old revision, deleting new tests in folder Language, that were developed during summer. Could you clarify this?


Sorry but, do you think about following description:
2. All machines can communicate with each other. Master node needs to connect the Internet to download the necessary files, while working nodes do not.


Bad code style. Entweder eine neue entsprechende Exception werfen oder zumindest die Exception mittels Logging erfassen.


Fair enough I guess, but my €0.02 would be that this is a UX issue, rather than a config option - e.g. have them visually grouped in the homepage admin or similar, rather than introduce another hidden config option.

This would avoid death by config option scenarios (with related support complexity), as well as avoiding conflating the data model (replies being separate classes on Alice's computer, but conflated on Bob's), which would cause problems should they change their mind (this is not a reversible change) or ever want to export/import from to an install where they're split to one where they're not and vis-versa.

In short, I understand the motive, I just don't think it is safe to do it this way...


I would havwe put this below method - since the JS opening this up seems more natural then


CDMA device could also have LTE capability, it seem we could not just put LTE here.


Ist denn das Prolog-Programm in Fall eines Fehlers an dieser Stelle noch korrekt ausführbar? Wenn nicht, dann wäre eine Exception der bessere Ansatz.


generally like a trailing comma in Python (unlike what we had before) - makes it easier to add new entries & comment old ones


I'm pretty new at github so probably my comment should be placed at a different spot - sorry in advance.

I've tested this hacked library with my Arduino DUE - but due to a different Port layout this did not work.
I changed the way the port mapping is given to the library in this way:
    // in my LPD code, aiming on Pins 4 and 5:
    #define pinmask 0x06000000
    #define pinport PIOC
    const unsigned int nLEDs = 64;
    const int  dataPin  = 4;
    const int  clockPin = 5;
    const int  dataPortPin  = 26; // relatively to port C
    const int  clockPortPin = 25; // relatively to port C
    LPD8806 strip = LPD8806 (nLEDs, dataPin, clockPin);
    ...
    setup ()
    {
          pinport -> PIO_OER  = pinmask; // set pins 4, 5 as OUTPUT (PORTC.26/25)
    }

and I build in a little wrapper so I can decide to use the `showCompileTime` or the slower show procedure:

    void writeToLED ()
    {
        if (FASTWRITE)
             strip.showCompileTime<clockPortPin, dataPortPin>(pinport, pinport);
        else
            strip.show();
     }

Finally, this is one of the changes (not completely written down here) I tried dealing with the library:

    template<unsigned int ClockPin, unsigned int DataPin>
    void TransmitBit(byte& CurrentByte, Pio* ClockRegister, Pio* DataRegister)
    // void TransmitBit (byte& CurrentByte, volatile uint8_t& ClockRegister,
    //                   volatile uint8_t& DataRegister)
    // -> that TransmitBit would not work on DUE
    {
        // Set the data bit
        const byte LED_DATA_MASK = 1 << DataPin;
        if (CurrentByte & 0x80)
        {
            // DataRegister |= LED_DATA_MASK;
            DataRegister -> PIO_SODR = LED_DATA_MASK;
        }
        else
        {
            // DataRegister &= ~LED_DATA_MASK;
            DataRegister -> PIO_CODR = LED_DATA_MASK;
        }

        // Pulse the clock line
        PulseClockLine<ClockPin>(ClockRegister);

        // Advance to the next bit to transmit
        CurrentByte = CurrentByte << 1;
    }

Maybe I've made a mistake dealing with these port manipulations (e.g. I should
not have given pointers to the library) but I've no idea why this does not work.
When I test the `PIO_SODR` and `PIO_CODR` methods with `#define ledmask 0x08000000`
and `#define ledport PIOB`  they work pretty well.

Does anybody have an idea what I might have forgotten or left out?

Best Regards,
Hasuino


I love when a framework handles those tricky cases...


Bitte den Logger der AddonsPlugin-Klasse nutzen.


TeamCity pull requests :: netty [Build 220](https://garage.netty.io/teamcity/viewLog.html?buildId=1453&buildTypeId=pulls_netty&guest=1) is now running


Why have this placeholder repo defined here?


TeamCity pull requests :: netty [Build 220](https://garage.netty.io/teamcity/viewLog.html?buildId=1453&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:37


Something went wrong here.
Poke @stfx


The Create Repo link should just show in the custom form if the user has permissions to create these (which will currently only be for full Admin...extending this should come with some additional template-specific limitations I think to restrict to just FTP)


We could add a function to SGD to (de)serialize boost arrays. But PeopleArray is NOT a class and I didn't want to make it a full class. Its only a typedef to save the tedious writing of the name in absence of C++11 "auto"


初始化


We do support having the Create as an option in the dropdown:
https://github.com/flavour/eden/blob/master/modules/s3/s3widgets.py#L6626

I agree with you that this would be more of a common usecase here, so it makes beter UX to have this be default :)


Ok.


I'm not very expert at pull requests, but i think you have the diff here correctly now


Yeah, using this for a WordPress theme and I can't submit it to the store with short tags in the code :8ball:


@davidfestal well I removed all the places where we were iterating over `Unit`s, so I guess the situation just doesn't arise anymore...


link is renamed to socket, should be `socket_for_port()`


same


@krullgor
>when you pass for an item and someone else rolls for it, the pass massage is displayed again when the rolls are displayed

Iam not sur how it should work in fact. For me it look good. Can you tell me more about how it should work?
>when the timer for need/greed runs out on items from raid bosses, it doesnt disappear, but pass messages are displayed (maybe a backport issue as the timer was increased in later expansions?) works on chests etc

i cannot reproduce :( -> https://youtu.be/atxSYJYXHZs

>when an item in a group is dropped that has a condition that is not fulfilled, it is not replaced by an item that fulfills all conditions(for example when a shaman raid item set piece dropps for alliance)

confirmed i need to look at it.


oops. I may have forgotten to. This is actually resources taken from [salem client](https://github.com/EnderWiggin/Custom-Salem/tree/master/resources/src/gfx/hud) - you can get those 8 files and put in proper folder for now. Will push them as soon as I get home.


@AquinasThomas thank you for your commit! However, I don't completely understand what to do with it.

Do you propose a pull request?

Thankfully,

Robert


can `using namespace boost::asio;` in cpp file


commit 내용 split


Hard to get a consensus apparently. I will keep going with separate src.nix because of easiness for now, until we find a better global solution.


The bugs which I found are :
- Notification Light
- Sound
- Camera
- External SD
- Sim
- Rotation
- Bluetooth


why disable download with ssl check? Its work correct...


second commit


"Idiomatic" is generally a pretty vague term and has a lot to do with gut feeling and perceived aesthetics. :-)


second commit2


`H` -> `HeaderType`


Nos vemos un rato hoy  las 14h?
Quiero mostrarte lo que he logrado estos días.
Le 17 sept. 2015 22:51, "José A." <notifications@github.com> a écrit :

> @damenac <https://github.com/damenac> Time to start writing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/damende/puzzle/commit/df6b4c12bf16987a5cf3b4725e090f2269f36eb4#commitcomment-13297579>
> .
>


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (1879 of 1880 tests passed)

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


fix it !
tulip is work in progress


we might use a separate method `documentByType` or something like that?


:+1: :smile:


this should go to where we register types in general i think.


Thanks for the quick commit!

On a side-note: The current logic seems to do the job nicely. However, have you had a use case where you needed to set the maxReplacementsPerPage to higher than 1? Maybe this could be a boolean rather than an integer? This might make processing a little faster, maybe?


This commit is being reviewed as a part of review issue #63.


You ppl need help


I do not know what "an iterator assignment expression" is. catch/throw is fine for exiting loops.


Turned out to not be so bad, fixed.


Genial,nos vemos a  las 14.00


Missing space after `=`


@flavour this one was for my internal testing...will remove that in PR ;)


Multiple slashes only ever are a problem for string comparisons of directories and for showing them to users because they look ugly. Unix itself does not care about too many of them (just too few :-)).


Hey, quick question: Does this tool work with the Find 5 16GB version?


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1344 of 1344 tests passed)

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


Is it possible to allow you to enact those laws if the current ruler has daughters?


These code changes to address the change in API behavior for vCenter 6.0 (when VM Power On task is null) are good. LGTM. Please add the test details.


I noticed :)

On Fri, Sep 18, 2015 at 8:37 AM, David Nichols <notifications@github.com>
wrote:

> this is fine - please re-merge master into your branch and then make a
> pull request (I converted SftpPoller to new-style yesterday too)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qorelanguage/module-ssh2/commit/264a86f63c3dffc373d3d6395cb912341b80e891#commitcomment-13303681>
> .
>

--
Adrián Lachata

Software Engineer
Qore Technologies s.r.o.


The `.project` file is from virtualenvwrapper: https://virtualenvwrapper.readthedocs.org/en/latest/projects.html

The way virtualenvwrapper uses that file, multiple directories do not make sense and multiple lines would break that. You can already use `.dir-locals.el` to specify virtualenvs for pyvenv in a pyvenv-specific way, can't you?


Added serializations require TESTS.


Hey Paul,
All fair questions. Let me start answering them by telling you why I started this series. I am working on and off on a web-based client for EL (far from complete yet, don't get your hopes up), and many times when I was looking for how certain things got done in the official client, I got frustrated because parts of it are still in spaghetti form. That's where the first refactoring patches originated. As you guessed, I started using clang's scan-build static analysis tool to check for potentially dangerous situations after I encountered some questionable code. I have been trying to remove these warnings, 59 remain, but I believe most of these are false positives, or things like dead assignments which I think make the code more uniform or safer. So I think I'm mostly done with those. What might follow is some small refactoring to untangle things (mainly interface.h, which exports variables from all over the place), perhaps some #include cleanups, but nothing major. Of course, if you'd like me to stop these cleanup patches, or if there's something else you'd like me to take on, just let me know.


It's actually returning a Key, for which bool( key ) is False because the type is Invalid - the same as is returned on the C++ side. I deliberated about this quite a bit and I'm still not sure how happy I am with it. It relates to the question of whether time should be a property of the key, or if keys should just hold the value and then be stored in some sort of map<time,Key>. The main reason for choosing the current approach was `closestKey()` - that needs to return not only the value but also the time, and also needs to have a way of saying "there ain't no closest key". That was fulfilled by including the time in the key and having a concept of invalid key. It hasn't felt too bad in practice so far, but I think it's probably best to consider the Animation API to be a bit provisional at this point.


@sunyonggang : not sure this is the right place for your issue. This project is about launching an OSGi (Felix) container on a node and has nothing to do with NodeJS or Hadoop...


@IIvec: +13 Elo after 500 games happens frequenty with neutral patches, it's just that we as human get easily overexcited by the jumpy nature of random statistics. I've seen some of my "ideas" which had +75 Elo after 500 games on local tests on my computer, and which proved to be a regression here on fishtest on a larger scale :-)

What I typically do before submitting a test to fishtest is that I do lots of small runs (length 300) locally on my computer for each idea, and that I submit on fishtest only the tests that passed with a positive score in four consecutive small local runs. It's time consuming, but I find it the best compromise between personal efficiency and politeness for other users of fishtest.

If you decide to tune on fishtest with SPSA as a last desesperate attempt, btw, please consider using a small number of games (20000) and a fast time control (10s or 15s), because the area of code you are trying to improve has already been well tuned by others, and you'll have already spent about 300.000 games at LTC on it :-)

Hope this helps,
Stéphane


After some days stopped working for me too. I confirm the issue.


Das war sehr schön, lieber Patrick!


`setInterval`에서 `setTimeout`으로 변경.
[참고 : stackoverflow](http://stackoverflow.com/questions/7499060/setinterval-javascript-are-there-known-bugs#answer-7499143)


Do you actually meant _a pagefault at **instruction pointer 0x20** is caused_? I'd expect the fault to be caused by data access not instruction fetch.


Yes, I thought of that. But that problem already existed, only we were masking it under the name of a trivial function.


The correction will be available in the next version.


no worries. how can it compile with two "return" ?


Wow, great job. Thanks!

I have a few things I'd like to get fixed before accepting this though:
- [ ] Even though there's only a little logic here I really want full unit-test coverage, both for finding potential issues, giving support for future refactoring and acting as documentation.
- [ ] I don't have a lot of time to do manual testing of this before shipping it, so I'd want end-to-end tests verifying that it all works as intended.
- [ ] Please use Unix line endings (LF) only.

Do you think you can get this done?


Tryout nested emphasis...

- this is *italic*
- this is *bold*
- this is *italic and **bold italic** right?* (see https://guides.github.com/features/mastering-markdown/ for an example: *You **can** combine them*)


No, just the result of a bad merge of two commits with same change. One of the else if-clauses should be removed.


@pynicolas indeed!


@birdsarah, can you explain this change? `proj4` is a runtime dependency, so the original location of this dependency was correct.


pass message is only displayed when the player clicks pass, not when the rolls are made

i'll test the timer again later today


@vilchik-elena LGTM


`libmono-sqlite4.0-cil` is needed by CmisSync, would you mind changing the script to have the .deb install it automatically if possible? Thanks a lot! :-)


`static const char*`


There is one '%s' too much no ?
And don't you want to indent a bit more on the left ? here you are aligned with the for loop, is it correct ?


Is this right? This is giving me an `Illegal offset type in isset or empty` error, because an array can't be a key in an array. Shouldn't that be `array_get($this->action, 'middleware', [])`?


This is probably the cause of #1529. Please revert.


Why shouldn't be static methods used?


Return an error string.


Octree (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1879 of 1880 tests passed)

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


Имя первым же! Это не школьный журнал ;)


`return "OK";`


Neat!


@laurenwaller check this out :arrow_up: and http://s.wearenext.co.za/putter-school/

I noticed you couldn't open (expand) more than one card, without closing the previous one. I think this fixes that.


@vilchik-elena In calls to firstOf, it would be safer to keep the order of the existing grammar.


My observations:

1)  at the moment it's schizo about the name of the 'Save' button - with it called 'Save' on the button & Create/Update (appropriately) in the action menu. I think I prefer save which is why I haven't 'fixed' the button yet.

2) Save a copy is also not quite right -the button has a pop-up but doesn't save. The action menu one is the reverse

3) One of the biggest improvements in this is that the buttons are already rendered on the criteria form - so you can save without rendering first & the current render-twice when saving  is gone too.(the url issue still not resolved)

4) charts - it turns out view pie chart etc don't work in the 4.4 version or this version (yet) but it's also not possible (in either version) but that aside - I think we do need to go back to charts being a separate drop down because the fact it's a pie chart can be saved (although that *could* be saved as part of the navigation/dashlet url rather than the report) AND the output can be combined with the pdf or print options.


> when an item in a group is dropped that has a condition that is not fulfilled, it is not replaced by an item that fulfills all conditions(for example when a shaman raid item set piece dropps for alliance)

@cyberium : this issue was there before the loot rework (small infos about it there: https://github.com/classicdb/database/issues/227). It needs to be fixed indeed but was not introduced by your nice rework.

> pass message is only displayed when the player clicks pass, not when the rolls are made

I agree: the "passed" message are only displayed once: when the player click on the pass button.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **8/15** |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | **0/13** |
| _XDS | 34/34 |
| _Options | **81/87** |
| _DTVCC | **0/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/93) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


This one is included in bourbon....http://bourbon.io/docs/#clearfix


Is there any CME error for bad command? If there is, we could consider use the CME error. And just `return RESPONSE_STRING;`


Its about messenger application based on android OS.


deleting


Okay, `openInMain` is mostly what it was before, except inner `if`, mainly `throw` when `$el[0] === view.el`.

Changed:
* `HomeView` receives `el` option (not sure on `new`ing it every time);
* `logbook` method does not pass `$('#tab-content')` as `$el` to `openInMain`.


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


Should we reply error if the call is not ACTIVE or HELD?


Amazing!! I wanted to do that but couldn't figure out how. :bow:


See also

http://stackoverflow.com/questions/31720893/how-to-configure-eclipse-to-complain-about-the-same-errors-that-the-java-8-javad


@sqndr Is there a reason to keep bower components in the repo?
Seems not right to me....can be wrong.


hello me


@pynicolas Ok


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 654](https://teamcity.labkey.org/viewLog.html?buildId=272916&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:27:45
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridColumnSelector: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

##### there are 23 more failed tests, see build details
```


LGTM


May you use spaces instead of tabs? That would avoid large indentation

Btw that Entity List window may benefit some extra work, I recall thinking about alpha sort for instance of the entities; When you have maps with a lot of entities and looking for a particular one (like when you did a typo and you have a red print in the console ingame), that would help for instance


Personaje


@vilchik-elena LGTM


NOTE "make clean" needed for this one. Even with that I think I see problems, will investigate.


@gavinking : it's more that that since I've put a breakpoint in a place where I still can have a look at the `EditedSourceFile` during the scenario you described,  in order to explicitly (in the debug session) compare the `EditedSourceFile` and the related `ProjectSourceFile`, and there always were equal.

Anyway, we're done with this issue it seems.


should be private?


@akbertram @leo maybe you have some better message in mind for confidence case


xcvxcvcxv


Назовём этот коммит "орущие курицы"


@pepelsbey есть какие-то правила!? О_о


What about moving this upward instead?


Do you really need this? I always thought that within a template class the class name would automatically be expanded to the full templated name by the compiler.


hello how to  fill in the parameters   georadius methods  for example


You could just call `reset()` here which will make the check and the deletion for you.


Будем готовиться к большому проекту и приучаться к хорошему. Разделю.


Лучше поменять на
$redirect_uri = fn_url("index.php?dispatch=ulogin.login&backurl=$backurl");


Uses deprecated constructor


Is it still working on Python 3.4 ?


r+


Why is the "Siblings" view available only for terminal documents? All documents can have siblings. If we think the "Siblings" view is useful then it should be available for all documents. If it's not useful for non-terminal documents then it's not useful for terminal document either.


Does not need to be public?  Package local?


Just a small thing but would we want to check that $GPG_RECIPIENT was not empty?


Есть подход: если ты в списке, среди многих то ты Петров Иван (сортировка по фамилии, принадлежность к семье и т.п.), если ты редактор, автор, личнсть — то Иван Петров, Аркадий Мильчин, Людмила Чельцова.

![image](https://cloud.githubusercontent.com/assets/105274/9955764/de3880bc-5df3-11e5-9cee-18d05dd17848.png)


Can you remove the bower .bowerrc file, I think it's generally better to keep it to the standard name.
and ignore the bower_components/ we should provide a gulp task that installs the bower dependencies on our envs. It is rather fast now in the newer version with the caching of bower.
This repo stays small and clean and no git conflicts can occur during upgrades of packages.

Can you thus also remove the font awesome lib as well form this pull request?
http://addyosmani.com/blog/checking-in-front-end-dependencies/


@cedricmessiant Hello, why did you comment the validator ?


Since this commit I'm having a strange issue, line 2270 is asserting false and causing the program to abort, but it's really sporadic. Sometimes the app runs for a few seconds, sometimes it's up to a minute, but then there's a call to render before newframe and it breaks at that assertion. It's happening in all the demos too so i checked it wasn't a modification of my own that matched up. Any thoughts?

Also thanks so much for the effort you put into ImGUI. Only just properly got approval from my higher-ups to use it, but this completely deserves financial backing on patreon. You rock :)


HTML лучше вынести в шаблон Smarty в папку
design/themes/responsive/templates/addons и
var/themes_repository/responsive/...


Wouldn't it be better to just read the file list of the tarball instead of extracting it?


Yes, it is, it's just that I updated my env :)


Or I will merge this one and see if we can adress the issues in other issue tickets.


As discussed on leap-discuss, from a functional standpoint, only the creation of the security doc is missing.

As azul pointed out, the security docs needs to be created to restrict access to documents in that DB to the soledad user. The security doc is the same for all created DBs, this is the yaml representation: https://github.com/leapcode/tapicero/blob/master/config/default.yaml#L28

Tapicero sends it using `PUT /user_xzy123/_security` (either if it doesn't exist yet or an override flag has been set) here: https://github.com/leapcode/tapicero/blob/master/lib/tapicero/user_database.rb#L18


If $GPG_USEASYM is only ever going to hold a one word boolean, we don't need to enclose it in quotes - though this may be a style decision.


Hmmm icon is not show.... I dont know why....?


Thanks, fixed


@pepelsbey спасибо, поправил ;)


здесь достаточно /css/bundle.css и далее по файлу


My observations:

1) at the moment it's schizo about the name of the 'Save' button - with it called 'Save' on the button & Create/Update (appropriately) in the action menu. I think I prefer save which is why I haven't 'fixed' the button yet.

2) Save a copy is also not quite right -the button has a pop-up but doesn't save. The action menu one is the reverse

3) One of the biggest improvements in this is that the buttons are already rendered on the criteria form - so you can save without rendering first & the current render-twice when saving is gone too.(the url issue still not resolved)

4) charts - it turns out view pie chart etc don't work in the 4.4 version or this version (yet) but it's also not possible (in either version) but that aside - I think we do need to go back to charts being a separate drop down because the fact it's a pie chart can be saved (although that could be saved as part of the navigation/dashlet url rather than the report) AND the output can be combined with the pdf or print options

https://www.dropbox.com/s/j3rem7slywelg65/Screenshot%202015-09-18%2020.55.11.png?dl=0


Will add in next version. :)


I don't understand the question: When you say it is up to users to make sure things are declared don't you think envAttributes is the way to go? I don't understand what other way of implementing this you might be refering to.....


For the clarity I would like to suggest:
```
payload = build_payload payload, defaults: %i(
  currency commerce_type
  return_url
  cancel_url
  restore_url
  skin
  pre_auth_callback
  post_auth_callback
  transaction_notification
)
```


Ironischerweise hat genau diese Engine die Test-Suite (durch rubocop) fehlschlagen lassen :wink:


You're right.


In GoD that value is still 300, you sure of you range changes ?


LGTM


This commit is being reviewed as a part of review issue #63.


This will result in an error, if this user doesn't exist. I'd argue, that if we rely upon an user account with this name, we should provide one (ie. in a data migration/fixture).

From another perspective, we could just provide a group, and if that group is empty, we just display 'your administrator', else we show a list of possible people to contact.


@Dignifiedquire I started sending the title information also, because for extras the id != title.


I need to cache the data as the are available, not to set it at the end. When I read the file but fail to parse the server URL, I still need to have the file cached. The same for the rest. The former state was messing config loading.


Why Techexpo registration page has the registration form of RC Plane competition ?


I agree with this. I asked myself this question. However we need to fix the large More Actions menu since this would add one more entry there ;)

Another option for all this is the Document Tree rooted in the breadcrumb provided we could have a Tree/Livetable switch there (would be awesome).


/cc @milmazz @eksperimental


How this initialization should be handled?


You can see how this new version will look like here: http://hexdocs.pm/plug/1.0.1/


It's fun to improve Stockfish playing style while using symmetrical info, no? I wonder how many Elo points we can milk with all these ideas :)


I got errors looking like this:
```js
{ keyword: 'additionalProperties',
  dataPath: '[\'wadup\']',
  message: 'should NOT have additional properties' }
```

Notice the dataPath array that looks stringified/JSONifyed. I think it is due to this commit that cast it.errorPath to a string.


tu m'as pris de vitesse!!! saligaud!


Why you use 1000 images per file? I have near 2 million files on site.
And putting 1000 img per file not okay when google understand 50000 per file.


ts


`org.hibernate.ogm.options` has a detailed `package-info.java` documentation but it's not rendered to JavaDoc (see http://docs.jboss.org/hibernate/ogm/5.0/api/org/hibernate/ogm/options/)


**no-loop-func**: Don't make functions within a loop


**func-names**: Missing function expression name.


Hi folks! Beerfish here. I ran into a few problems trying to install the clean WotLK master branch on my Unbuntu system last night. Here's a summary of what happend:
- cmake went fine

- make error:
Making install in .
/bin/bash ../libtool --mode=install /usr/bin/install -c libACE.la '/usr/local/lib'
libtool: install: /usr/bin/install -c .libs/libACE-5.8.3.so /usr/local/lib/libACE-5.8.3.so
/usr/bin/install: cannot create regular file '/usr/local/lib/libACE-5.8.3.so': Permission denied
make[7]: *** [install-libLTLIBRARIES] Error 1
make[6]: *** [install-am] Error 2
make[5]: *** [install-recursive] Error 1
make[4]: *** [install] Error 2
make[3]: *** [install-recursive] Error 1
make[2]: *** [dep/ACE_wrappers/ACE_Project-prefix/src/ACE_Project-stamp/ACE_Project-install] Error 2
make[1]: *** [dep/ACE_wrappers/CMakeFiles/ACE_Project.dir/all] Error 2
make: *** [all] Error 2

- make install error:
CMake Error at src/scriptdev2/cmake_install.cmake:72 (FILE):
file INSTALL cannot find
"/home/mangos/mangos/src/scriptdev2/scriptdev2.conf.dist.in".
Call Stack (most recent call first):
src/cmake_install.cmake:42 (INCLUDE)
cmake_install.cmake:62 (INCLUDE)

- Running make and make install as sudo solved the two errors above.

- After cmake, make and make install I am indeed missing a couple of things, among which the entire run directory.

Create a post about this on the cmangos forums, too: http://cmangos.net/thread-7223.html


The Freemarker version is not in the motherpom project so we just need to make sure to enable the right set of Freemarker features if this project is reused elsewhere.

From Freemarker documentation:
```
Configuration cfg = new Configuration(VERSION_X_Y_Z));
// Where X, Y, Z enables the not-100%-backward-compatible fixes introduced in
// FreeMarker version X.Y.Z  and earlier (see Configuration(Version)).
```


Ups, sorry - der Rubocop passt da ja sehr gut aus ^^


abc


This _might_ work, I'll check in more detail once I got paid. You can try to prepare unit tests to cover all scenarios already if you want to speed up merging by then.


:relaxed:


Could we wrap `(:local && adivsers.empty?)` to denote ordering.


Yep, I confused Shell syntax with Makefile syntax. Thanks for fixing the bug I introduced!


We discussed that yesterday. A group will not work because: What if there are more than one in the group? Which should be used for the email link? Maybe in a later version we replace the quick-made email link by a from. That's the moment we should change to a group.
I agree: We should check if there is an admin to prevent an error.


Does this really warrant a new setting? If so, it should be configurable by using an environment variable.

If we want to store 'constants', I propose to use a dictionary, ie. `OSMAXX_CONSTANTS`, which contains the all the constants, and also move other constants (settings, which are not configurable) to there.


"Modifier 'private' is redundant for enum constructors"


That's why I said it's not a big deal right now.


Parece errado mas ele ta perdendo mesmo, caso contrário não passava nos testes
this.vida = this.vida + (-10)


Yes, same as my XDA account nick.


I am a nice comment


LGTM

I suggest as alternative using the scripts only to get the list of users (as alternative to phase0 script) and then proceed with the other phase* scripts, because they delete more resources.

Anyway I think is good to preserve also the legacy code that deletes the resources for historical purposes.


Hmm that's odd. There's no explicit call to EndFrame() in the demo, and Render() which calls is explicitly does `    if (g.FrameCountEnded != g.FrameCount)
        ImGui::EndFrame();`
I don't know how/why that would happen on the demo code. I'll investigate but I don't think I can right now (maybe tonight). Let me know if you find anything.

Thanks!


haha you know that word, you are a China pro, man!


@cornelius yup definitely a better approach, thanks for pointing it out


Thus, this commit depends on the not-yet-merged wrapper.oeclass in OE-lite/core.


It is not caused by this commit and that's an expected error message.

dataPath is not an array here, it's a JavaScript property access syntax using array index syntax to access property. The problem here is that there is a performance penalty to determine at validation time whether this additional property can be used with `.prop` syntax (i.e. whether it is a valid identifier) or it should be used with `['prop-1']` syntax. It is done during schema compilation for all properties in the schema and it is not done during validation for properties in data.

So if you data is in the variable `data` you can access this additional property via `data['wadup']`.

See my comment here: https://github.com/epoberezkin/ajv/issues/48

The alternative is to use JSON-Path in data paths in error messages.


Thus, this depends on the merged-but-not-released help2man recipe in OE-lite/core.


lol was because this file was accidentally deleted in the previous commit


Why not simply add a bison.oeclass which sets BISON_PKGDATADIR instead of this wrapper approach?


Ping @taylorotwell.


please use a trailing comma (the same applies to lists), so that it is only one line in the diff, next time we add something. I know, JavaScript does this differently, but it is Python ;)


There's a limit on images https://support.google.com/webmasters/answer/178636?hl=en


Oh :(   Looks like I merged your patch to master - back ported to develop now.


What sort of modifications do you have? Is the call to EndFrame() that asserts done from the normal Render() path ?


Alright, I'm taking a look at it myself now but don't think I'll be able to hold up much to your own observation. You fathered the code and can probably read it a lot more thoroughly :p.
If I do see anything though I'll let you know, thanks!


I guess it should be "typo3/cms" not "typo3/cms-core", shouldn't it?


I see it's a named parameter short cut to passing in a particular parameter to the function.  My Python foo is very weak.


@lpl6338236 apparently, you need to resolve the conflicts and push again.


My modifications aren't actually in any of the Render() related stuff, everything there is still synced with the master branch. my modifications are just a few extra UI elements that I've made, but I've just removed them to test using only the provided elements and it still happens