Keep cool, jahto. :-)

I think they had this dilemma years ago discussed too as you can also see on the extensive manual section about permissions.

Yesterday in bath tube - current system is a pure permission system. What you want is a permission and deny system. But for this the project roles permission fields would need a 3 status behavior (allow, deny and unset (use global rights of the current user))
And it creates IMHO even more problems with adding peoples to the correct roles working together.
The fact is, you can technically accomplish ALL your prefered user restrictions with the current permission settings.


Is it not `.Primary.Damage`?


@jacshfr Wait, we are supposed to be using a new GTM container for this work now, correct?


oluuuum direk kendi clouduna atıyo img dosyalarını burası bi harika lannnn


Also, it would probably be better if his was disabled by default so the Stunstick's name isn't "Bash baton" for everyone.


Kanka onu clone ettin sanırım peki desktop üzerinden commit edebiliyormusun projeye bende sadece yapılan değişiklikler gözüküyor !!! OMGOMG whatanawesomesitethankyouzuckerbergforfb


ha! Yeah, I still haven't understood everything and I'm watching/reading tutorials on OOP.
I'll get there... :)

Thanks for the heads-up, fixed.


umm alex what are you doing?


Ah, I see, yes avoiding use-package entirely would be beyond what the keyword modular could do.


LAN Bİ DE HER DEĞİŞİKLİK OLDUĞUNDA BENİM EMAİLİME NOREPLY MESAJI MI ATACAK BUNLAR AMK ??!?!?!BİR!!!=!1!=!=!=!=!


whoops, I didn't mean to push that bit.

2015-03-26 21:15 GMT+01:00 Bo Anderson <notifications@github.com>:

> Also, it would probably be better if his was disabled by default so the
> Stunstick's name isn't "Bash baton" for everyone.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FPtje/darkrpmodification/commit/cce3983730a7a3928527b34fae5952b71363a3f3#commitcomment-10420474>
> .
>


That's bad english, but I actually meant what I thought I wrote - that is we *should* probably continue flagging conflicted files as needed, but we need to be aware of that in the puller. That's the next step of the evolution here. :)


For **unrestricted files**, avoid calling the permissionService for "Permission.DownloadFile".

Instead check:
- if the dataset version is released
- If the user has Dataset level permissions that allow File Download


I get a schema parse error when adding an aufs pool even though it is optional.  With the regex and the maskRe on the text field, you shouldn't be able to enter a bad value anyway.


Now featured here:
https://github.com/stars/ForthStar


Package restore was added, the rest wasn't. What parts would you like added?


Uppercase D.


Can you do a PR?


sharp eyes you have, can I borrow them?


I confirm it works on windows (MinGW). However, old c style cast throws error. You need to do:
```
renderWindow->getCustomAttribute("WINDOW", static_cast<void*>(&hwnd));
HINSTANCE hInst = static_cast<HINSTANCE>(GetModuleHandle(nullptr));
SetClassLong(hwnd, GCL_HICON, reinterpret_cast<LONG>(LoadIcon(hInst, MAKEINTRESOURCE(IDI_ICON1))));
```


Markdown file


"I thought MS still ran makecatalogs" -- it might, but both MunkiServer and Simian store their catalogs in a database, so I can't be sure how that database is generated/populated. In any case, I can't be completely sure that 1) makecatalogs is being used, and 2) that people are using an updated makecatalogs. (I can easily see an admin updating their Munki clients, but forgetting to update the Munki admin tools on some dedicated machine they use to manage their Munki repo)


Did you start to get that error when you added the regex? That's really weird! No I know, it's more a case of never trusting external input.


np, I'll see if I can do it; thanks for the inspiration ;-)


@emeijering

>also because it's not fully clear to me what the implications are in terms of maintenance

The [fiji contribution requirements](http://imagej.net/Fiji_contribution_requirements) page is intended to answer this question. For one, there's the required MAJOR version compliance of projects. Also, when you want to upload a new version of your project, you would need to ensure it's compatible with the current state of Fiji's dependencies - e.g. by updating to the latest pom-fiji.

Anyway, I think that page could use some updating. I'll give it another pass tomorrow, which I hope will help you (and future contributors) make an informed decision. Let me know if you think of any other specific questions you'd like answered in the mean time!


That file should be a symlink to the real `hyper_1` music, because I don’t want to have the same file in the repository twice. Maybe Git on Windows doesn’t create NTFS symlinks as it should do? With Windows XP out of support, there shouldn’t be any Windows system that does not support NTFS symlinks, so I’d be surprised if Git doesn’t support that. Symlink creation on Windows is really easy:

~~~~~~~~~~~~ c
#define WINVER=value-for-windows-vista-which-i-dont-remember-right-now
#include <windows.h>
//...
CreateSymbolicLinkW(source, target, flags)
~~~~~~~~~~~~

If Git doesn’t do that, I’d regard that as a bug in Git. For now, see what Git gave you in `data/music/game/menu.ogg`. If it isn’t an OGG file (which is pretty likely), replace it with a copy of the symlink target, which is `data/music/land/hyper_1.ogg`. And file a bug report against Git. O_o

Vale,
Quintus


Yeah, it doesn't need to be a big note, just that I think a small mention would help some people since I've seen the question come up enough times. (Maybe there's a better place for it than this particular page, though.)


This is a build made for a hackathon. Used to increase productivity of the THR import


Class names should be [CamelCase](http://en.wikipedia.org/wiki/CamelCase).

See [Google's Java Style](https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s5-naming) for more information. Recommend reading through this as there's a lot of good stuff in there.


I get the error no matter what.  aufs shouldn't be a problem because it defaults to 4g.  I don't have the regex correct on the line.

"mlimit": { "type": "string", "format": "regex", "pattern": "/^\d+(m|g|%)$/", "optional": true },

I think I need something else here.


I'd recommend coming up with more intuitive variable names for these. Ex: `addButton`, `subtractButton`... This will make your code much more readable.

> There are only two hard things in Computer Science: cache invalidation and __naming things__.


why not [self getPersonOutOfAddressBook]


Ok


Again, variable naming. The variable name should be self descriptive of what it contains.


thats why maykrosoft


Here are corrections for several of the `STVar` special  members.


wtf


Since this package is visible from the Python interpreter, we actually *should* provide an `__init__.py`.
It should contain a proper package docstring (instead of the comment) and it should raise an `ImportError` (or do something that causes such an exception) when imported, saying that actually the module `soundfile` is the one that should be `import`ed.


Yes. Xunit 2 changed their packaging story to have a separate msbuild runner package.


Same as above for wildcard imports


It sucks when the polyfill doesn't actually work.


Is there a reason why these were introduced and maybe some examples on why they are useful, thx


Converted & into html char


The commit you're looking for is: aab376b


Mispelled.


Is 72 histogram driven?  Just curious.


some tests failed:
failure: http://build.servo.org/builders/gonk/builds/350


It's really important to get the tabs and spaces consistent and to keep it clean. It's one of the simplest things we can do as professional programmers to keep the code well aligned. Different languages have different conventions for indenting. Ruby for example likes things to be indented with 2 spaces. Whatever you choose, all files for that project must be consistent. Google around for the convention for the language you're working on.

It looks like [Google Java Style](https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s4.2-block-indentation) recommends using 2 spaces. Any editor worth its salt should have a configuration to help keep things consistent.


Oh, maybe we should just revert 7925516d13. The performance benefit is pretty much nonexistent.


Commit


А ты когда замержишь это в мастер?
Кстати брать argmax при поиск параметров по гриду при использование bootstrap (или еще какой-нибудь стохастической оптимизации) не самая хорошая идея, там может быть достаточно большая дисперсия в зависимости от seed'a. Лучше сохранять полный лог поиска, а потом его уже анализировать (вдргу изменение параметров менят 5 знак только случайным образом). Заодно понятно, куда дальше сдивагать список параметров для поиска.


Hi thank you for the suggestion.
I think the angle bug should be fixed right now.
Working on making the UI better...


VSHPWH (shenb) - x86_64-MacOS-10.9-clang: Build Failed

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


I would love to add this functionality. Unfortunately, I don't see a hook in xunit that allows this behavior. It would be great if IDisposable.Dispose would have a flag telling us that an exception has gone I handled since we could then print extra diagnostics but it does not. I found a hacky workaround for this that we can explore but do let me know if you find a supported approach for doing this.


Ah, right, thanks.


I looked at the code in OMV and he escapes it so I think it would end up like this:
```
"mlimit": { "type": "string", "format": "regex", "pattern": "\/^\d+(m|g|%)$\/", "optional": true },
```
I hardly think the $ would need escaping?

Sorry I'm not more of a help now. I currently have no VM setup in a way that I could test it directly.


Maybe use a switch statement here?


saw approval from jdm
at https://github.com/leavengood/servo/commit/0d0be2aa1dd330d53607498d25d398211c44a50c


merging leavengood/servo/fix-alphabetic-key-codes = 0d0be2aa into auto


leavengood/servo/fix-alphabetic-key-codes = 0d0be2aa merged ok, testing candidate = 07e271b4


This method is getting really long. Any way you can break it down into smaller functions?


Heh. I actually wanted to do that and then I couldn't remember the name of the extension.


8.3-Release-WrapUp (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


@indirect You may be interested in [this comment](https://github.com/rubygems/rubygems/issues/1207#issuecomment-86702317).


This method can be moved to miq_request_workflow.rb and reduced to the following once we setup the proper subclass preloading in miq_request_workflow:
```ruby
MiqRequestWorkflow.descendants.flat_map(&:encrypted_options_fields).uniq
```

Thanks @Fryguy for the assistance on this one.


The ASCII cat in

```el
(defhydra hydra-rectangle (:body-pre (rectangle-mark-mode 1)
                           :color pink
                           :post (deactivate-mark))
  "

  ^_p_^     _d_   delete   _s_tring          |\\     _,,,--,,_
_b_   _f_   _k_   cut      _r_eset           /,`.-'`'   ._  \-;;,_
  ^_n_^     _w_   copy     e_x_change       |,4-  ) )_   .;.(  `'-'
^^^^        _y_   paste    _e_xtend        '---''(_/._)-'(_\_)
"
```
causes error because now `,` is part of the regexp :P

```
Debugger entered--Lisp error: (error "Unrecognized key: _,,,--,,_")
```

Time for the cat to go.


yes it should be. nice catch. pr?


@marcin-wosinek


I meant to rename it.
Thanks for pointing out the mistake, Mark!
Should be fixed now.

David

On Thu, Mar 26, 2015 at 12:51 PM, Mark Harrison <notifications@github.com>
wrote:

> Did you mean to delete radtrack/ui/LaserInterface.py or just rename it
> like radtrack/ui/BunchInterface.py?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/radiasoft/radtrack/commit/43a267b49bcff5aeb07af517d51557a41a3d483a#commitcomment-10419100>
> .
>


Same error.

Error #2002:
exception 'OMVException' with message 'Invalid method parameter: Schema parse error (4).' in /usr/share/php/openmediavault/rpcservice.inc:187
Stack trace:
#0 /usr/share/openmediavault/engined/rpc/unionfilesystems.inc(219): OMVRpcServiceAbstract->validateMethodParams(Array, '{\n             ...')
#1 [internal function]: OMVRpcServiceUnionFilesystems->set(Array, Array)
#2 /usr/share/php/openmediavault/rpcservice.inc(125): call_user_func_array(Array, Array)
#3 /usr/share/php/openmediavault/rpc.inc(79): OMVRpcServiceAbstract->callMethod('set', Array, Array)
#4 /usr/sbin/omv-engined(500): OMVRpc::exec('UnionFilesystem...', 'set', Array, Array, 1)
#5 {main}


also, `()` isnt needed


return is also not needed


Thanks! That's waaaaay better!


I think this should be in the DB not in a separate file.


Ah, I see that you agree!


we moeten nadenken of we wel een thread willen hier, denk ik

Is het een probleem als de GUI geblokkeerd is zolang het inlogproces loopt? Vgl met een terminal die ook niet interactief is zolang `kotnetcli` output toont.


:+1: Jeg lavede bare en copy/paste fra deres opslag. Jeg hacker måske et bedre system til dem i morgen(ish).


`list_type v_;`
(so you can change `list_type` in only one place)


I fixed it here anyway in c9f7f54c296ccce086c9df8fb04f2f743150c770.


Probably. I keep forgetting `chat-plugins` exists.


@szul I just saw this -- don't want you to think I didn't see this. Will respond tonight or first thing in the AM.


> As soon as we start replacing app elements by native kotnetcli code, we should be very vigilant not to break stuff. That comes down to changing only one part every time, so we can easily roll back.

I agree indeed. As far as I understood, for now we only have a conflict with `mechanize` and `BeautifulSoup4`? I would suggest to drop `BeautifulSoup4` then and use regular expressions as you indicated above.


:yum:
virtual void 	setPayload (boost::shared_ptr< Payload > payload)
boost::shared_ptr< Payload > 	getPayload () const
virtual void 	handleResponse (boost::shared_ptr< Payload >, boost::shared_ptr< ErrorPayload >)=0


Oh, sorry to be unclear! I didn't mean student cheating.

I just meant that I started using git several years ago for collaboration on production websites. As a front-end developer my first impression of GitHub was that it was sort of a dumbed-down version of git or just git GUI.

But now I see it's much more and I'm ready to drink the Kool-Aid.

I plan to use GitHub for student communication next semester!

Let's keep in touch,
-Ben

> On Mar 26, 2015, at 11:52 AM, Jason Costello <notifications@github.com> wrote:
>
> I'm curious, did you feel that it was cheating because the students can see each others code in an open workflow? I could see how from a professor's perspective, that'd be a barrier. When a student finds employment at a startup or software-based company, the same open workflow becomes the basis for self-driven learning and collaboration\at the speed of light.
>
> Thank you so much for attending. Please, feel free to contact if you have any ideas or questions regarding GitHub or an open design workflow. I hope to lead more intermediate workshops in the near future.
>
>
>
> \
> Reply to this email directly or view it on GitHub.
>


there's still

Mar 26 16:22:28 schoolserver.lan httpd[17945]: Invalid command 'WSGIPythonPath', perhaps misspelled or defined by a module not included in the server configuration
Mar 26 16:22:28 schoolserver.lan systemd[1]: httpd.service: control process exited, code=exited status=1
Mar 26 16:22:28 sc


Actually, this isn't possible: if the package cannot be imported, the docstring cannot be accessed ... so let's *not* raise an `ImportError`.


f20 fresh install


By tapping continue, "you" are indicating.....


8.3-Release-WrapUp (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


I am pretty sure that syncthing does not support multiple query strings of the same type.


Multilingual sites could 'divert' SQL to using alternative db tables or the parser into using alternative documents/resources according to the selected language (implementation is left as an exercise for others!).


`camelCase` all your variables. This would be `inputNumber`. Also, no need to abbreviate unless you get a really long variable name. Use your best judgement.


Да, надо добавить логирование, пожалуй. Я, правда, хотел сделать gridsearch многопоточным, а у нас везде всё просто в stdout льётся и в этом месте будут проблемы.


Could you make comment on why it is safe to use checkExpressionCached with contextualMapper in the spread?


Did you see my patch replacing the $db->fetchAllArray($sql)
by a $sql->FetchRow() loop?

Have you tested your huge sql with big tasks size? Does it start to use temporary tables when running the sql? What performance differences between old and your new whole-in-one?
Can you show us the explain from postgresql and maybe make some comments to the output please?
I will now do permission overview pages first , so be a bit patient with me testing your stuff.


Many functions had undefined parameters in the doc strings. Fixed.


@Entropy512 Hi and sorry for post here but any chance to add rhine to the omnirom project?


Right, i overlooked it. Maybe we can check the length of EditLog for this item and if it is empty, treat this as new object? I cannot think about more clear sign of new item.
> 26 марта 2015 г., в 22:53, Harut <notifications@github.com> написал(а):
>
> Not exactly correct. When creating language version, item.id is not None, but the type should be still "create"
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/SlivTime/iktomi-cms/commit/de0cb8b5b24ec58757d3323c9e8f26395705acfd#commitcomment-10420089>.
>


Also, don't forget that order matters as syncthing is generally stupid, so destination dir has to be scanned before the source directory.


This constructor is getting really big, and has a lot of responsibility. Think about what this function is doing and try to `encapsulate` each of those tasks into a separate function.


Updated with your changes


That is correct. Only the upper left corner should be rounded.


@jacshfr Looks like the firing trigger for this event in GTM is looking for `newsletter-interaction` instead of `newsletter-registration`.


Maybe this should say 'is not yet supported'


Informational (not a request for a change).  I recently used the following style in my own code.  I'm still deciding whether I like it or not:
`constexpr defaultObject_t {} defaultObject{};`


This is the parent class to everything below, be sure to indented everything below this properly.


I guess we can remove it. The note on how to switch usage to other stuff (in the deprecation warning) should be moved to changelog in that case.


good catch :)


This is perfect README content :) I would add it to the README file you created in the root of this repo.


VSE NORMAL POSONI


I changed the RS485 address and while the project builds the LEDS no longer blink and the BT network "MySpa" does not appear on my phone.Going to try redoing this project with a different starting version. one that builds, and programs the device correctly.


1: M111 unrecognized component name: oauthclient


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


1: M111 unrecognized component name: kwalitee


1: D400 First line should end with a period (not 'e')
1: D205 1 blank line required between summary line and description (found 0)
59:1: I101 Imported names are in the wrong order. Should be CFG_PYDEV_DEBUG, CFG_REMOTE_DEBUGGER_ENABLED, CFG_REMOTE_DEBUGGER_IMPORT, CFG_REMOTE_DEBUGGER_NAME, CFG_REMOTE_DEBUGGER_PYDEV_PATHS, CFG_REMOTE_DEBUGGER_PYDEV_REMOTE_IP, CFG_REMOTE_DEBUGGER_PYDEV_REMOTE_PORT, CFG_REMOTE_DEBUGGER_TYPE, CFG_REMOTE_DEBUGGER_WINPDB_PASSWORD, CFG_REMOTE_DEBUGGER_WSGI_RELOAD
59:1: E402 module level import not at top of file
68:1: I100 Imports statements are in the wrong order. from invenio should be before from .config
68:1: E402 module level import not at top of file
69:1: I100 Imports statements are in the wrong order. import os should be before from invenio
69:1: I201 Missing newline before sections or imports.
69:1: E402 module level import not at top of file
70:1: I100 Imports statements are in the wrong order. import glob should be before import os
70:1: E402 module level import not at top of file
71:1: E402 module level import not at top of file
72:1: I100 Imports statements are in the wrong order. import sys should be before import traceback
72:1: E402 module level import not at top of file
73:1: I201 Missing newline before sections or imports.
73:1: E402 module level import not at top of file
76: D103 Missing docstring in public function
89:1: E303 too many blank lines (3)
98:1: E303 too many blank lines (3)
111: D202 No blank lines allowed after function docstring (found 1)
111: D400 First line should end with a period (not 't')
117:19: E222 multiple spaces after operator
130: D400 First line should end with a period (not 'y')
130: D205 1 blank line required between summary line and description (found 0)
142: D401 First line should be in imperative mood ('Return', not 'Returns')
142: D400 First line should end with a period (not 'r')
142: D205 1 blank line required between summary line and description (found 0)
164:5: E303 too many blank lines (2)
168:1: E302 expected 2 blank lines, found 1
168: D401 First line should be in imperative mood ('Parse', not 'Parses')
168: D400 First line should end with a period (not 'v')
168: D205 1 blank line required between summary line and description (found 0)
182:13: B901 blind except: statement
194: D400 First line should end with a period (not 'e')
194: D205 1 blank line required between summary line and description (found 0)
202:5: F821 undefined name 'rpdb2'
206:1: E303 too many blank lines (3)
206: D400 First line should end with a period (not 's')
206: D200 One-line docstring should fit on one line with quotes (found 3)
211:5: F821 undefined name 'rpdb2'
215:1: E303 too many blank lines (3)
215: D202 No blank lines allowed after function docstring (found 1)
215: D400 First line should end with a period (not 'e')
215: D205 1 blank line required between summary line and description (found 0)
236:9: F821 undefined name 'putils'
237:20: F821 undefined name 'pydevd'
238:13: F821 undefined name 'pydevd'
244:16: F821 undefined name 'putils'
244:56: E261 at least two spaces before inline comment
244:57: E262 inline comment should start with '# '
245:32: F821 undefined name 'putils'
246:18: F821 undefined name 'putils'
246:57: E261 at least two spaces before inline comment
247:32: F821 undefined name 'putils'
271:5: E303 too many blank lines (3)
276:5: E303 too many blank lines (2)
278:37: F821 undefined name 'pydevd'
278:61: F821 undefined name 'pydevd'
279:9: F821 undefined name 'pydevd'
282:5: F821 undefined name 'pydevd'
282:5: E303 too many blank lines (2)
288:5: F821 undefined name 'pydevd'
294:1: W391 blank line at end of file


As this is the main point of entry for Java apps. Maybe it would be cleaner to move this to line 29, after your imports but before the SimpleCalculatorApp class declaration? Having it in the middle of the class feels a but buried.


@pkolaczk  asked me to format it better. So, I though that it would better just to revert it:)


ok


RecyclerView has a bug. Its is currently only appearing on the bottom half of the Screen when the transition from MainActivity to KingdomFragment is made.


No problem. :)


tested and working


That one's a good question...  We had togari_gpe up and running using CM's
repos as a base.  I need to talk to the crew about what to do about the
rhine family, specifically if we want to just completely swithc to Sony
AOSP.

Feel free to idle in #omni - most of the guys doing Sony work are active
there this week.  :)

On Thu, Mar 26, 2015 at 4:44 PM, David Viteri <notifications@github.com>
wrote:

> @Entropy512 <https://github.com/Entropy512> Hi and sorry for post here
> but any chance to add rhine to the omnirom project?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/omnirom/android_device_sony_shinano-common/commit/1ff08fee9b798b454bbac81bc623dcf1206b43e4#commitcomment-10420961>
> .
>


What do you think of using constants for the index, such that you do not duplicate them?


I added this to the U1 L1 Accessing Snap page.


It seems you have to do some weird escaping. Does this work?
```
"mlimit": { "type": "string", "format": "regex", "pattern": "\\/^\\\d+(m|g|%)$\\/", "optional": true },
```


tnx, fixed it in the last version. (assuming I read the nzbget repo properly)


Super le nom de l'image


thanks for your answer


git clone https://github.com/plusjade/jekyll-bootstrap.git ayie-media.github.com
cd ayie-media.github.com
git remote set-url origin git@github.com:ayie-media/ayie-media.github.com.git
git push origin master


I think the reschedule should happen in cases where we are fairly certain that the issue will be resolved by retrying later... e.g. a timeout which could be because of network issues. And we can progressively add other issues to this list as well discover them. I am afraid to reschedule (without a degree of certainty) for all exceptions once we get to the point that we start adding data to the datastore.


Do you want to limit the visibility as well?


Done


That's probably it. You should contact Yoav to correct the URL.

If there's the case, email me and I will modify your grade.


Good to hear that :)


I think the array indexes are quite intuitive, not vey intuitive but quite intuitive.


You could relax the type argument of the `Class<T>` parameter to `Class<? extends T>`.


Works :)


It was true, otherwise I wouldn't have written it :smile:

Is it still true? Unknown.

That was from the original version of hoverIntent (2006/2007). It's possible that was for IE6... or IE5.5 (when the hoverIntent precursor was written in plain, non-jQuery JavaScript). I vaguely remember there being an issue with IE and passing `e` straight into the setTimeout (lines 97 and 104).

I'm pretty sure the test is simple/obvious... like remove that statement, change the few instances of `ev` to `e` below and then try using hoverIntent in IE6. If the browser complains about `ev` not being defined (in compare/delay functions) then it's still an issue. I do not have access to IE6, so I can't test this.

I would argue that as long as we're supporting jQuery 1.x (which supports IE6), then we should keep the copy in place unless someone can definitely test it and report that it's not an issue.


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


1: D400 First line should end with a period (not 'e')
1: D205 1 blank line required between summary line and description (found 0)
59:1: I101 Imported names are in the wrong order. Should be CFG_PYDEV_DEBUG, CFG_REMOTE_DEBUGGER_ENABLED, CFG_REMOTE_DEBUGGER_IMPORT, CFG_REMOTE_DEBUGGER_NAME, CFG_REMOTE_DEBUGGER_PYDEV_PATHS, CFG_REMOTE_DEBUGGER_PYDEV_REMOTE_IP, CFG_REMOTE_DEBUGGER_PYDEV_REMOTE_PORT, CFG_REMOTE_DEBUGGER_TYPE, CFG_REMOTE_DEBUGGER_WINPDB_PASSWORD, CFG_REMOTE_DEBUGGER_WSGI_RELOAD
59:1: E402 module level import not at top of file
68:1: I100 Imports statements are in the wrong order. from invenio should be before from .config
68:1: E402 module level import not at top of file
69:1: I100 Imports statements are in the wrong order. import os should be before from invenio
69:1: I201 Missing newline before sections or imports.
69:1: E402 module level import not at top of file
70:1: I100 Imports statements are in the wrong order. import glob should be before import os
70:1: E402 module level import not at top of file
71:1: E402 module level import not at top of file
72:1: I100 Imports statements are in the wrong order. import sys should be before import traceback
72:1: E402 module level import not at top of file
73:1: I201 Missing newline before sections or imports.
73:1: E402 module level import not at top of file
76: D103 Missing docstring in public function
89:1: E303 too many blank lines (3)
98:1: E303 too many blank lines (3)
111: D202 No blank lines allowed after function docstring (found 1)
111: D400 First line should end with a period (not 't')
117:19: E222 multiple spaces after operator
130: D400 First line should end with a period (not 'y')
130: D205 1 blank line required between summary line and description (found 0)
142: D401 First line should be in imperative mood ('Return', not 'Returns')
142: D400 First line should end with a period (not 'r')
142: D205 1 blank line required between summary line and description (found 0)
164:5: E303 too many blank lines (2)
168:1: E302 expected 2 blank lines, found 1
168: D401 First line should be in imperative mood ('Parse', not 'Parses')
168: D400 First line should end with a period (not 'v')
168: D205 1 blank line required between summary line and description (found 0)
182:13: B901 blind except: statement
194: D400 First line should end with a period (not 'e')
194: D205 1 blank line required between summary line and description (found 0)
202:5: F821 undefined name 'rpdb2'
206:1: E303 too many blank lines (3)
206: D400 First line should end with a period (not 's')
206: D200 One-line docstring should fit on one line with quotes (found 3)
211:5: F821 undefined name 'rpdb2'
215:1: E303 too many blank lines (3)
215: D202 No blank lines allowed after function docstring (found 1)
215: D400 First line should end with a period (not 'e')
215: D205 1 blank line required between summary line and description (found 0)
236:9: F821 undefined name 'putils'
237:20: F821 undefined name 'pydevd'
238:13: F821 undefined name 'pydevd'
244:16: F821 undefined name 'putils'
244:56: E261 at least two spaces before inline comment
244:57: E262 inline comment should start with '# '
245:32: F821 undefined name 'putils'
246:18: F821 undefined name 'putils'
246:57: E261 at least two spaces before inline comment
247:32: F821 undefined name 'putils'
271:5: E303 too many blank lines (3)
276:5: E303 too many blank lines (2)
278:37: F821 undefined name 'pydevd'
278:61: F821 undefined name 'pydevd'
279:9: F821 undefined name 'pydevd'
282:5: F821 undefined name 'pydevd'
282:5: E303 too many blank lines (2)
288:5: F821 undefined name 'pydevd'
294:1: W391 blank line at end of file


28:1: I101 Imported names are in the wrong order. Should be BlueprintAutoDiscoveryRegistry, ConfigurationRegistry, ExtensionRegistry, PackageRegistry, Registry
31:1: I201 Missing newline before sections or imports.
32:1: I201 Missing newline before sections or imports.
34:1: I101 Imported names are in the wrong order. Should be unicodifier, with_app_context
263:9: B901 blind except: statement


86: D202 No blank lines allowed after function docstring (found 1)
86: D400 First line should end with a period (not 'd')
86: D205 1 blank line required between summary line and description (found 0)
127:59: W604 backticks are deprecated, use 'repr()'
131: D203 1 blank line required before class docstring (found 0)
131: D204 1 blank line required after class docstring (found 0)
131: D200 One-line docstring should fit on one line with quotes (found 3)
138: D203 1 blank line required before class docstring (found 0)
138: D400 First line should end with a period (not 'f')
138: D205 1 blank line required between summary line and description (found 0)
151: D400 First line should end with a period (not 'e')
159:31: E261 at least two spaces before inline comment
171: D210 No whitespaces allowed surrounding docstring text
171: None: None
180: D210 No whitespaces allowed surrounding docstring text
180: None: None
180: D209 Multi-line docstring closing quotes should be on a separate line
180: D202 No blank lines allowed after function docstring (found 1)
180: D400 First line should end with a period (not 'f')
180: D205 1 blank line required between summary line and description (found 0)
212:26: E127 continuation line over-indented for visual indent
220:13: E266 too many leading '#' for block comment
221:13: E266 too many leading '#' for block comment
222:13: E266 too many leading '#' for block comment
223:13: E266 too many leading '#' for block comment
247: D210 No whitespaces allowed surrounding docstring text
247: None: None
247: D202 No blank lines allowed after function docstring (found 1)
247: D400 First line should end with a period (not 'y')
265: D210 No whitespaces allowed surrounding docstring text
265: None: None
265: D209 Multi-line docstring closing quotes should be on a separate line
265: D202 No blank lines allowed after function docstring (found 1)
265: D400 First line should end with a period (not 's')
265: D205 1 blank line required between summary line and description (found 0)
269: D210 No whitespaces allowed surrounding docstring text
269: D401 First line should be in imperative mood ('Thi', not 'This')
269: None: None
277:13: E266 too many leading '#' for block comment
300:13: E266 too many leading '#' for block comment
305:13: E266 too many leading '#' for block comment
306:13: E266 too many leading '#' for block comment
307:13: E266 too many leading '#' for block comment
313:13: E266 too many leading '#' for block comment
328:17: E266 too many leading '#' for block comment
333:13: E266 too many leading '#' for block comment
337:17: E266 too many leading '#' for block comment
338:17: E266 too many leading '#' for block comment
339:17: E266 too many leading '#' for block comment
352: D202 No blank lines allowed after function docstring (found 1)
352: D400 First line should end with a period (not 'n')
352: D205 1 blank line required between summary line and description (found 0)
422:13: B901 blind except: statement


return max(h1, h2) + 1; (???)


this is beautiful


This is great, but can you add code to migrate people's existing settings. Should just be a matter of another `if` at around line 79 of `ProjectItemSettings`.

:+1:


r+

Nice fix.


That's certainly a very valid argument. My main concern is the data-loss (I've grown to fear this term thoroughly) situation an unforeseen exception can lead us to. As long as we make sure these kind of cases are at least properly propagated and urgently handled by other channels (error mail list?) we should be fine.


No this is not a fix..


Great! Lots of slashes haha! We've got our solution and I hope everyone should be satisfied!


Y U passing pointer by reference?


Please use square brackets for consistency


Use UTF, bro


gosh how did slip that in? I never use these curly ones ... .


Wrap any statements after ````if```` in ````{ ... }```` block, please


Not sure if it will work, but can you try marking these up with `[DisplayName()]` and/or `[Description()]` so they show in the properties dialog as "Do not run", "Before build" and "After build".


I think it should be `if IS_PYTHON3 and isinstance(obj, (int, float))`


Is this not an error?


1.5 is in the repo.  Works on my VM and my system at home.


What is that magic constant **110**?


cs


Generating working code is a good thing I guess but the project is about more than that. How much of the existing pipeline was used to make this work (or more importantly, how much was changed/replaced without discussing with the group)? The project is a team effort and we all need to be part of and understand the code so we can build it together and write the report together. Kudos for the hard work though.


@leahpadgett spot on Leah.


@leahpadgett down with my review. checkpoint complete.


should have tagged it before the change with traits, but you should better upgrade to 2.4.0 anyways. ORM 2.5 is RC-1


@GrahamCampbell same here lol


*yay*


> True. The only case this would happen would be if an ability such as Friend Guard or Solid Rock would make a 1-2hp hit into a (0-1), right? I think that would return 0 without having a Sub.

If I remember correctly, Gen 6 (and I think 5?) can round damage down to 0, so if you have really low Atk and foe has really high Def or something, it can happen normally.

The classic examples in past gens are things like Endure when you're already at 1hp, or False Swipe against 1hp.

> Anyway, it's better to have this bug than the previous bug.

Isn't SubPunch a really common thing? I can't imagine previous bug being even more common.

`lastAttackedBy` in general is dumb. Marty, do you have a better idea of how that sort of thing is represented in-game?


Part of courage is simple *consistency*.

**Peggy Noonan**


Oh well, he had a good run.


Your fix is much slower, though. I'd say revert 7925516 and c9f7f54.


For all models? Do you have any backtrace of the segfauls?


It's really hard to read without ````{...}```` block


@jbrooksuk @GrahamCampbell Are we ready to send the PR to master form L5?


It works great, the big test map loads in ~5s now instead of ~40s previously :)


> I confirm it works on windows (MinGW). However, old c style cast throws error. You need to do:

Will be done now. :)

@hwoarangmy Can I ask one last test?
`SET(OD_SOURCEFILES ${OD_SOURCEFILES} dist/icon.rc)`
This line is not 100% accurate. It should be:
`SET(OD_SOURCEFILES ${OD_SOURCEFILES} ${CMAKE_SOURCE_DIR}/dist/icon.rc)`

I'll fix it now as well.


You don't need to use ````typedef```` in C++


Can you provide the URDF that is causing this behaviour ?


Timestamps are now converted to UTC when stored, and parsed as UTC when fetched.

When an absolute timestamp is requested, it's returned in UTC for the time being, disregarding player's settings and the server's local timezone. However, that doesn't matter a lot right now, since IIRC the only place where an absolute timestamp is shown is the ban page, Carbon handles relative timestamps nicely and datetime form controls are not affected. I'm planning to make the player see the time in their own timezone, though.


I meant the 0 damage is better.


This did not appear to work.  I think the error is in fact queued but it never wakes up my thread which is blocking on `fi_eq_sread()`.  My thread gets stuck in `dlist_wait_avail()` on the main list for the EQ, but only the error list actually got signaled.


Not very cross-platforming command


Okay.


Github is advising adding new line at the end of the file. So do I.


Double-check me on this, but I think we can now do this: https://github.com/HowardHinnant/rippled/commit/8fdbd7ce372b6a3a713f008dd2dfc8f26f226520
It passes tests for me.


VSHPWH (shenb) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1150 of 1150 tests passed)

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


@tim-moody, hmm, I am testing in F18


You will also need to update the tests over at https://github.com/mozilla-b2g/gaia/blob/master/apps/findmydevice/test/unit/findmydevice_commands_test.js#L111,L112 .


````"\n"```` -> ````endl````


it's fine to drop the `@strictfp`, we aren't bothering with that in UI code.

I don't think you need the `private val` either, `editorArea` is in scope throughout the class even if you don't declare it as a `val`.


Yes, it is a standard sbt plugin. Every time a file in `src/assets/javascripts` gets changed, the target script file is automatically rebuilt using the sbt build system. There are some issues though. Mainly I'm still little confused by the documentation about *how* exactly the build process works and where the plugin should output the result. I consider this a temporary solution. ES6 with JSX is likely to get quite popular (ES6 is after all standardized) so I expect some more solid solution to emerge soon.

At this point the biggest problem is **that it is slow**. It is a good enough solution to get the final build but it is close to being unusable during the development phase. It takes 3-4 seconds to recompile every time something changes and at this point it's only few lines of my own code with react included. It might be necessary to run webpack separately along the sbt and let it watch the changes (in which case webpack is supposed to be able to take advantage of some in-memory caching and therefore it should be significantly faster).

Regarding the style tool: I don't think we'll share much of the JavaScript code base. But webpack gives us freedom to use any tool available. So we could pick one.


you can replace the combination of `isInstanceOf` and `asInstanceOf` with a single pattern match.


not very important, but we fixed it in some places so maybe we wanna be consistent: this leaks the allocations before it in case assertion fails.  So maybe allocate things after this point.


I did it myself.


There's your problem...

You miss a curly brace at the end (you opened 2 namespaces, ripple + RPC but only closed one of them).


I mean changing an index in one place and then having to remember that you
need to change it in another.

On Thu, Mar 26, 2015 at 11:59 PM, Yousef Hamza <notifications@github.com>
wrote:

> I think the array indexes are quite intuitive, not vey intuitive but quite
> intuitive.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yousefhamza/openmrs-core/commit/98415052f35e429015487f1a1a482543b2275d71#commitcomment-10421229>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


This was done so that a blank "Filter" config wouldn't cause a validation error to appear (since saving the config even with the error simply results in saving `"*"` anyway).

However, that causes the tests to fail.


Done


Please take this repository down.


Important Note : I've taken almost all of the new functions from https://github.com/kaspth/tokie. All of them can be used for improving signing of cookies as I 've shown.

 I've shown that the Tokie and current implementation can be merged to support inbuilt signing of expiry and purpose of cookies. User can generate cookies just like before, but now he can also add a purpose of the cookie with the options as shown below:

cookies.signed[:name] = {:value => "John", :expires => 1.day.from_now, :for => "Registration"}

As you can see from the code, these three parameters are also signed inside the cookie in JWT format. This is just to give an idea on how we can proceed to add these features to the Cookie Implementation of Rails. This has many bugs and problems in it. e.g If the user doesn't give an expiry, that would cookie should prevail until the browser session. For now, I have assigned the default expiry of the cookie to be 1 day.

Any suggestions and comments are welcome :) .


`break` statements immediately after returns are unreachable and can be removed.


did it force to re generate wem's in any case? Just  curious


Don't add gitignore to the project, please. Everyone have their own gitignore.


Edited the config manually, works!


Nice job! Love it.


maybe `errors->len > 0`?


@MissChocoe Yes, I would think so


Doesn't look like one form, it's two different inputs, FS handles what should be done, users don't need to know what goes where, as long as it works.


I really don't get this check


Done.


can `output` really be `NULL`?  If it really can, then probably `errors` can also, and so has to be checked before dereferencing it.


isn't this kind of a code smell? We'll have to add this in a lot of our authorization methods.


You're right, I thought I could easily modify ST to watch for changes in multiple files but this doesn't look too trivial.
Actually, even if I'd reverse the order, changes are high the puller will be too slow to reuse the existing blocks because the addition&deletion happens in a single batch to the db. Correct?


You can return ````nullptr```` for clarity


I've inserted the URL of the matrix.org instance of the pusher.


@jerpelea @Entropy512

checking at ramdisk by Sony I can see only this

chown root radio /proc/cmdline


What are you doing in this loop? If ````n != nullptr && n->left() == nullptr```` then ````n```` doesn't have predecessor


Sorry, I should have reviewed this commit better before pushing it out (I was touching the code again after not looking at it for a few weeks). The two *Task classes were integrated into GcmRegistrationManager and are now no longer needed. I have removed them.


- Inventory of currently available APIs and data on number of downloads, current applications created using the APIs, etc


I've removed this class as explained below.


Wrong


Same here. I don't get what are you doing...


hey I've added the style.css however, it doesnt show up so im going to add it here


.info{

	background: -webkit-linear-gradient(90deg, #D1FFFF 10%, #75FFFF 90%); /* Chrome 10+, Saf5.1+ */
	background:    -moz-linear-gradient(90deg, #D1FFFF 10%, #75FFFF 90%); /* FF3.6+ */
	background:     -ms-linear-gradient(90deg, #D1FFFF 10%, #75FFFF 90%); /* IE10 */
	background:      -o-linear-gradient(90deg, #D1FFFF 10%, #75FFFF 90%); /* Opera 11.10+ */
	background:         linear-gradient(90deg, #D1FFFF 10%, #75FFFF 90%); /* W3C */
	color:brown;
	margin:30px;
	padding:15px;
	border:4px solid brown;
	border-radius:5px;
	box-shadow:3px 3px 5px #B5D7B5;
}

.info img{
	box-shadow:4px 4px 5px brown;
	border-radius:5px;
}


Как минимум, вот тут у тебя ошибка.


The same command could probably be used in all OSes, I don't see why we wouldn't want to quote on non-Windows.  If the filename contains a double quote it'll be wrong anyway, so it doesn't matter, and if it contains spaces (way more likely) I'd say we need them, don't we?


*chances are high* I think is not exactly true.
If you manage to perform both scans in 5 seconds you might be lucky enough for both actions to propagate in a single index exchange. I guess the ideal scenario is to change the way the scan works to accept multiple arguments, and to make sure we hold the lock throughout all scans so that no index despatch happens in the meanwhile.


Not the best name for variable that hold minimum...


Sorry about this merge commit, I forgot to pull...


got it with both coman and bigman, as loaded from idynutils (idynutils::iDyn3Model())


сброс лишний


Wrong file name and wrong location. Now fixed.


This commit were updated to add new improvements over the code. See 684a99c10678fdb2b42695237e139fbaf5e03951 for more recently changes.


Android and Maemo? I don't think that we need this file


No, we want to support arrays of those chips, which have an arbitrary data width.


databits is already defined by PhySettings, don't duplicate it.


This breaks being able to use policy_scope with activerecord associations that may be empty, is that by design?

    policy_scope(@user.posts)      # as shown to be used in views in the readme

the blank? call evaluates the activerecord relation and throws an error if it is empty.  lines 35 and 37 will have the same effect


а протокол вообще здесь обязателен?


I've also removed this class, it wasn't doing anything anyway. Sorry.


Small typo in the commit message, "ixed"


I tested it, and it does not work.
Please check your change.


What does this `.js` do here?


Yes you did it with something in mind, I understand. I trully hoped I wouldn't have had to release alpha with that crap looking UI. I've gotten remarks about how ugly the interface is to tell you the truth... It's not you, not me, not any of us separated that made it ugly, but all of us together. Those buttons, they are space taking (and I remember another topic where you guys were discussing about saving height in details view). I have made remarks here on Github about how we feel the UI looking that way. Nothing changed... Look at the misery in the comments tab... It's the same... So as nothing moved, because you were working on something else all this time, I decided to change it myself. Now, that I actually changed it, to something more agreable looking, less place taking and working as well, you come and bug me about you had something in mind for 1.1? Dude, we are releasing 1.0 first... And I WILL NOT release until the UI is somehow decent.


Ah, I see! Ha, yeah I think some devs use all our services straight from the command line and don't utilize the site all that much.

> I plan to use GitHub for student communication next semester!

It's great to hear this. Which classes do you teach? I'll stay in touch. Let me know if there's any way I can help.


Can we finally get rid of the C version of flterm?


Use the return values of these .pipe calls somehow?


Can you start creating releases with the CHANGELOG details in them after you have done an `apm publish`: https://github.com/atom-community/autocomplete-plus/releases?


Sorry, it is not the problem.


@mattleff Why? To align with the top bar? I personally think it looks horribly unbalanced.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1169
success: http://build.servo.org/builders/linux2/builds/1167
success: http://build.servo.org/builders/android/builds/1203
success: http://build.servo.org/builders/mac1/builds/1382
success: http://build.servo.org/builders/mac2/builds/1385
success: http://build.servo.org/builders/gonk/builds/351


I could agree on something like that, but for example the "Releases" were in the "Misc Git news" in edition 1 and it is not very clear that they happen outside or inside the mailing list especially since libgit2 releases for example are not announced on the git mailing list while git releases and tig releases are.

So if we want to cover all the releases together in a "Releases" section where should we put this section?
And there might be other news that are announced on the mailing list but that we want to cover very shortly along with news from outside the list.


Resyncing a preexisting writeit instance with a popit instance.
This only provides the backend for resyncing. But the frontend would probably look something like this.

In somewhere like ```nuntium/templates/nuntium/profiles/writeitinstance_and_popit_relations.html``` in the
```{% for record in relations %}``` you would have to add something like following:
```
<a class="resync" href="{% url 'resync-from-popit' slug=record.writeitinstance.slug popit_api_pk=record.popitapiinstance.pk %}">resync</a>
```

and at the bottom of the file:

```
<script type="text/javascript">
$(function(){
  $('.resync').click(function(event){
    event.preventDefault();
    var url = $(event.target).attr('href');
    $.post(url, function(data){
      location.reload();
    })

  })
})
</script>
```


It looks way better with markdown


Hey Atif, thanks for your questions and comments! I'm starting an email thread to discuss our implementation shortly!


The index are used explicitly in the getter and setter methods only, I doubt those change frequently if not at all, except in case of refactoring -like this case-.


code review done!


If Yoda-speak we're avoiding, then write `tu_style == 'classic'`.


@khadijaibrahim @samaasallam


fast-forwarding master to auto = 07e271b4


You need to explain why the change is being made. Simply put it out there makes little sense.


Hi the url is updated, it should be playable right now in app, even though I need to implement drag n drop or improve ui to make it more playable...


More specifically, the commit message needs to have an explanation of why this is being done.


Why is that? Consider, for example, a multisigning service that multisigns for accounts after SMS verification. Why can't that service account both be a signer on my account and a signer on an account that signs for my account?


Also what should we do if someone sends a pull request with an interesting summary of a discussion that happened on the Git For Windows mailing list? Where do we put that?


saw approval from glennw
at https://github.com/pcwalton/servo/commit/d99aa1a295f3f808959ff46fa4df289a35496812


merging pcwalton/servo/root-flow-color-reform = d99aa1a2 into auto


pcwalton/servo/root-flow-color-reform = d99aa1a2 merged ok, testing candidate = 3e05221e


@jemmybutton not quite sure why adding the .csv to the interactionFields url helped resolve the issue you saw. I think it is related to the server setting the proper content type (text/csv) that way.


omglolnoob du har skrivit fel ju


This is a bit confusing. It looks like this replaces one check with two checks. Can you elaborate a bit why this is faster. Maybe leave a comment so this won't get reverted?


Uhm nej? Det ska stå så där?


See https://github.com/facebook/stetho/issues/59#issuecomment-75679958.  I don't recommend using these range wildcards.  I was mistaken in Stetho that it is required to satisfy the dependency resolver.


Yeah, in this situation comment wasn't really necessary actually.


I added the specialized tech node to try to fix the contract appearing. I think your right about needing its own requirement node, but sleep got the better of me so I had to leave it sit. Anyway, seems like you got it fixed so I will go back to the master version.


iter -> *


It would be nice to fix that. These kinds of inconsistencies will cause bad behavior in the new fee code.


Important Note : I've taken almost all of the new functions from https://github.com/kaspth/tokie. All of them can be used for improving signing of cookies as I 've shown.

I've shown that the Tokie and current implementation can be merged to support inbuilt signing of expiry and purpose of cookies. User can generate cookies just like before, but now he can also add a purpose of the cookie with the options as shown below:

cookies.signed[:name] = {:value => "John", :expires => 1.day.from_now, :for => "Registration"}

As you can see from the code, these three parameters are also signed inside the cookie in JWT format. This is just to give an idea on how we can proceed to add these features to the Cookie Implementation of Rails. This has many bugs and problems in it. e.g If the user doesn't give an expiry, that would cookie should prevail until the browser session. For now, I have assigned the default expiry of the cookie to be 1 day.

Also, when user wants to access the signed cookie to which it has given some purpose, then he can do that as follows:

cookies.signed[:name] = {:value => "John", :expires => 1.day.from_now, :for => "Registration"}  #=> Base 64 Encoded JWT token
cookies.signed["name", {:for => "Registration"}]           # => "John"

Any suggestions and comments are welcome :) .


Ca aurait pas été mieux de rajouter picture dans hull :stuck_out_tongue_closed_eyes:


What is the rationale for this change? I prefer not to change the format of description strings without a good reason.


Nice catch


Was this test color suppose to be committed?


AFAICT no code is using ``seed_to_data`` outside of sdram.h, so it should not be declared here and should be made ``static``.


Merge commit w tym projekcie, serio? LOOOL Bez kitu poprawilo mi to humor;p


@ericbn , oops, did I screw this up? Apologies!


> Can't make it work due to the issue with permalink not picking up date from the front matter.

Yep, that's on the roadmap for 3. Just not coded up yet.

> 3) We need pagination for posts and collections but in different folders. Say, /blog/page/:num, /updates/page/:num and newsletters/page/:num

Probably won't get to this before 3.0, so if you have time to fix it up, :+1: I have an idea sketched out in https://github.com/jekyll/jekyll-paginate/pull/22


Oh just spotted that running
```
git grep -i "hornet"
```

Shows some more code that likely could be migrated too.
And there is code comments that refer to the old jboss issue tracker. Not sure what the best approach is there.

eg such as a comment as
```
// that was happening under https://issues.jboss.org/browse/HORNETQ-988
```

But there is some more which should be fixed such as
```
docs/quickstart-guide/en/running.md:             11:05:10,848 INFO  [org.apache.activemq.core.server] HQ221001: ActiveMQ Server version 2.5.0.SNAPSHOT (Wild Hornet, 125) [e32ae252-52ee-11e4-a716-7785dc3013a3
docs/user-manual/en/management.md:    {"timestamp":1422019706,"status":200,"request":{"mbean":"org.apache.activemq:module=Core,type=Server","attribute":"Version","type":"read"},"value":"6.0.0.SNAPSHOT (Activ
etc/checkstyle.xml:      <!-- developed at https://github.com/hornetq/hornetq-checkstyle-checks -->
etc/checkstyle.xml:      <module name="org.hornetq.checks.annotation.RequiredAnnotation">
```


> If I remember correctly, Gen 6 (and I think 5?) can round damage down to 0, so if you have really low Atk and foe has really high Def or something, it can happen normally.

Only Gen 5 can round damage down to 0, and only because the final modifier (stuff like Reflect and Multiscale) comes after rounding up to 1, and then it rounds half down from there. In Gen 6 they definitely moved the final modifier to before the rounding up to 1 part, because V4 and I tested early in XY and a burned level 1 Joltik's Bug Bite couldn't do 0 damage to a level 100 Talonflame behind Reflect. But yeah, like you said stuff like Focus Sash Shedinja and similar things can still result in 0 damage.

> `lastAttackedBy` in general is dumb. Marty, do you have a better idea of how that sort of thing is represented in-game?

I would imagine each Pokemon keeps track of which other Pokemon have hit it throughout a turn. [Pokécheck seems to have a similar idea](http://vsplayer.pokecheck.org/?src) (search for `hurtby`)


Keep this integer, floating point is imprecise.


hmmm ... if we are hardcoding the 3 classes maybe simply pass in the 3 values instead of an array, its a bit clearer then that there is no flexibility


Ahmen vafan har du gjort!?!?!! Så kan du ju inte bara skriva!!


How are dynamic IDs a thing then if MC doesn't store them? Does it store everything as a string and make up IDs at (each and every) runtime?


so that humans get me


ditto


Check for null


This is still using the old astropy.config API.  It should probably be updated.  (I didn't see any other instances of the old API in the actual astroquery code).


I just used the official GitHub for Windows client in a different Win 8.1 Pro x64 computer to clone the `devel` branch - instead of creating symlinks, the client download the two OGG files and they contained just the plain text displayed in the commit.

I, too, see this as a bug in Git, and I'll try and see whether it's been reported already.


No, I haven't seen your patch yet and I haven't tested with big tasks sizes yet. Let's put this on hold for the next 1-2 weeks. I've got currently somewhat limited development environment and possibilities, must rely solely on my ages old travel laptop running windows. Water damage at home, my apartments' floor taken away, been staying at my mother for a while. When I get back home to my desktop running linux I've got both postgresql and mysql and anything else needed available for testing almost anything and can also create really huge databases. For now, I'd just want you to check if I had the nested join syntax right for mysql this time. In the future, I will test myself.


lhtm, thanks


Too bad, this commit is already obsoleted by commit 9d255591f5286b2f91f9fb7427d86957ec96cbbc!


I think it should be better to use the Policy Pattern in this case:

```ruby
class FacebookPolicy < Struct.new(:auth)
  def first_name
    auth['info']['first_name']
  end
end
```

Then in the `OmniauthCallbackController` :

```ruby
policy = "#{auth_hash['provider']}_policy".classify.constantize.new(auth)
```


No it's fine! :dancer:


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3511/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3510/


```java
return Maps.transformValues(logging.getAllLevels(), Object::toString);


Is there some reason we are not doing LTC tuning of these tables?  Just start with whatever you got.  Also, the assumption of symmetry kingside to queenside may not be valid for pawns and knights.


Try: ```System.Convert.ToBase64String(byte)```


If you reach here... for sure you are not bootstrapping :smiley:


That is really awesome! My Ruby knowledge is very limited so I can't really make a PR for this. But I'll be waiting for that feature in order to switch to Jekyll.


If @alexmarsev isn't around we might go with your solution. I will have to go soonish and won't be able to push tomorrow so today would be good I think.


It would make sense to use the same hashed+encoded value for the mutex name.


Great find, love the work. This helps a ton! Will look at this tomorrow and study how this can help speed along the integration of the ui-connections branch.


tested and working


ah yes, this will be it then: #206


Sure, sorry about that. I didnt realize you were doing that.


Thanks.  Fixed with f45f28bbe19d70b4a497ba3da00b2ac8bf97e75b


What was the point in renaming yourself?..


@parkr: is this normal to not show up at all with v2.5.3? With 3.0.0-beta2 I get an error thus I ended up wrapping the highlight tag in a raw tag.

The problem are `{{ }}`.


Looks great man. Thanks for combing over that. I really appreciate it.


fixes #328


@Ravie do you have an executable that we can use?


This may not be a good idea -- there really are a finite number of these.


argh, fixed via 29c07254c48f84f4b6e2aec5b4d0c03018d83138


Fine with me. We can revert it later if needed.

About LAV, this commit https://github.com/Nevcairiel/LAVFilters/commit/93ea8fe574565530818c6073d72898c284215bcd fixes #5333 so mention it in changelog.


Couldn't include file 'extensions\net.lua' (File not found) (@lua/includes/init.lua (line 10))


Is this the way this should happen? 580 translations thrown out because it is incomplete?


Oh, I'm definitely keeping it in until I can show that it's not an issue. It does sound like it might be something that was fixed once jQuery was brought into the mix, though it should definitely be verified first. Now to download a VM with IE6 on it...


Agreed


Any particular reason?


Check which is our permission now and in case of different permissions check what nexus has
If is the same pr


did you install iiab?  I didn’t.  It uses wsgi, so maybe that doesn’t get installed if you only install pathagar.

From: Gonzalo Odiard
Sent: Thursday, March 26, 2015 5:14 PM
To: godiard/xsce
Cc: Tim Moody
Subject: Re: [xsce] Pathagar: update version of django-sendfile (b6571a6)

@tim-moody, hmm, I am testing in F18

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


Ca c'est pas moi, c'est la correction automatique de syntaxe d'eclipse. Je devrais pas l'utiliser finalement ou bien la configurer. Mais à part ca Eclipse est bien configuré maintenant.


True, but the named pipe file name is case-insensitive, is it not? Sure, this may be an academic distinction, but I think shorter file name isn't really a goal either.


I've learned that this actually won't matter because we should not allow constructs like this anyway. See #2519


lrlgsroh


shame on me; fix on the way


It's not an error because ES6 takes argument list resulting from spreading the iterator, and then packages it into an array that is called s inside the function foo.


Couldn't include file 'extensions\net.lua' (File not found) (@lua/includes/init.lua (line 10))
Back to require and work again


send.txt was not updated (it should have been done in https://github.com/garrynewman/garrysmod/pull/798)


byoman-vu


Closes #14451


This should fix a NullReferenceException I am getting when this executes in my WorkerRole.


`{"sql": {"slow": x, "very-slow": y, "very-very-slow": z}}`?


without return coffee generate following js:

```js
(function() {
  'use strict';

  /**
    * @ngdoc function
    * @name gruntApp.controller:MainCtrl
    * @description
    * # MainCtrl
    * Controller of the gruntApp
   */
  angular.module('gruntApp').controller('MainCtrl', function() {
    return this.awesomeThings = ['HTML5 Boilerplate', 'AngularJS', 'Karma'];
  });

}).call(this);
```
which breaks controller. Besides `return` I have no idea how to address it.


This fixes the lang problem but sending the email. I think my local is not setup to send emails so what we discussed about the system not sending emails is a non issue I think. I will keep you posted


Do you have a reference for these new values by any chance? Feel free to include URLs in the source code.


thanks :+1:


As a workaround, we can of course just commit this file twice, but somehow this doesn’t appear correct to me...

Vale,
Quintus


I think the prevalent usage of fstop is something like f/4.5 where either the fancy f symbol or the letter f in lower case is used.  Whether one prefers f/5.0 or f/5 is up for debate.  That said, I probably should have kept this change out since it is fairly subjective.


Actually I think I prefer this formatting. Nice. Could you add this to the other F-number tag description please? Here's [an example](https://github.com/drewnoakes/metadata-extractor-images/blob/master/jpg/metadata/apple%20iphone%204.jpg.txt#L13).


What for?


yum install mod_wsgi fixed it.  maybe add a task for that


nbits is the number of bits of a module, we can have more than one module (ex Mixxeo has 2 MT46V32M16).
nbits is needed for simulation where we don't pass pads to the PHY but only the base module and the number of modules we have. (I don't want to mess with fake pads in the sim platform)


@iSWORD Cabinet hasn't been open source for months, the public repo was taken down and this one is very behind.


Why `force_text` and not `six.text_type` here? (Sorry if the question was asked before.)


``__str__`` (Python 3) or ``__unicode__`` (Python 2)


`list(APPEND` supports passing multiple elements at once. I think we should make use of that here. Same below.


Could you file a bug to use metadata reader instead of Reflection here?


Why the question mark? "Fin" with optional "d"?


I teach two tech classes in MFA. Mobile Web Technology and GDS Responsove Web. I am also writing additional classes now that will use git. I can begin with GitHub in all my GDS classes.

> On Mar 26, 2015, at 2:32 PM, Jason Costello <notifications@github.com> wrote:
>
> Ah, I see! Ha, yeah I think some devs use all our services straight from the command line and don't utilize the site all that much.
>
> I plan to use GitHub for student communication next semester!
>
> It's great to hear this. Which classes do you teach? I'll stay in touch. Let me know if there's any way I can help.
>
> \
> Reply to this email directly or view it on GitHub.
>


This seems to make no sense. The same variable is overwritten multiple times.


Does this mean, Z3 can be used in commercial products without buying a licence?


Why not use `git pull --rebase` ? ;)


That's not a very interesting parameter considering that the same module often exists in different data widths with all other characteristics being equal.


Right, I forgot to edit the commit log.


Yes, of course


The positions are not "blizzlike"?


```ts
// It is usually not safe to call checkExpressionCached if we can be contextually typing.
// You can tell that we are contextually typing because of the contextualMapper parameter.
// While it is true that a spread element can have a contextual type, it does not do anything
// with this type. It is neither affected by it, nor does it propagate it to its operand.
// So the fact that contextualMapper is passed is not important, because the operand of a spread
// element is not contextually typed.
```


Doesn't seem like a good idea to load the assembly just to show a display name of the reference. Could we just return _fullPath here?


@samaasallam  @khadijaibrahim


If the decorator also wrapped the `__str__` or `__unicode__` method with `safe_text`, that would be more dry.

In the current state of the code, safe classes need both `@html_safe` and `mark_safe` in their `__str__` or `__unicode__` method.


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/1170
failure: http://build.servo.org/builders/mac2/builds/1386


It'll strip whitespace, they said. It'll be fine, they said.


I can't see nothing related to cmdline in AOSP devices


8.3-Release-WrapUp (kbenne) - x86_64-MacOS-10.9-clang: OK (1668 of 1670 tests passed)

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


Oops, I need glasses! I wrote this in the previous comments :
"cdrtools devell 3.01a28" instead "cdrtools devel 3.01a28",
"tlide" instead of "tilde", and
"Gitgub" instead of "GitHub".


Ah oui il va falloir configurer ça ... parce que je ne vais pas accepter des diffs de ce genre :) Sinon ça va être invivable ...


I guess I need to buy you guys some beers :-)


The layout was done when we started in January 2015.
It is not the fault of the three of us.
I know the same as you that many parts are quite ugly, but they had also problems before.

I started a litte bit to make it  a bit more responsible, but then all the internal very old never fixed issues got our attention which must be addressed first.

Don't understand me wrong, I'm glad if someone takes care of layout stuff again.

And yeah, the comment section of details.view needs overhaul. There are many possibilities how to layout that and I currently have no favourite layout. Do it like github? Do it like old flyspray blue theme? Do it like an iPhone chat? Any ideas? Screenshots?


> I prefer Apache 2.0 these days

Haha!
As long as it can be redistributed along with commercial products I don't see any problem.


Passing the geometry settings plus the pads as it was done before is a better solution as it does not contain redundant information (the number of data lines).


Hey @desarrolla2, I haven't been using showRSS anymore, so havn't found the need for myself to write a test case. Last time I checked, this class was working for showrss feeds. It's kind of based on EzRSSAggregator.


Can you give me your test?

I had tried a simple test by commenting out fi_shutdown on the client side for cmatose test, and it worked fine. I can try with your test.


The navigation does not seem to work.  If I click on one of the cities it generates this URL:
http://localhost:5000/http//oakland.localhost:5000/

I tried changing
--- a/app.rb
+++ b/app.rb
@@ -23,7 +23,7 @@ class OpenDisclosureApp < Sinatra::Application

   helpers do
     def subdomain_url(subdomain, path)
-      scheme = "http#{'s' if request.secure?}"
+      scheme = "http#{'s' if request.secure?}:"

       if request.port == (request.secure? ? 443 : 80)
         "#{scheme}//#{subdomain}.#{request.host}#{path}"

That gets a better URL but its not found.


I think it looks better this way, in any case, it follows the same style as for .msg files (lines 40-50)


Je pense que je vais pas l'utiliser, je fais faire attention "a la main".


No, it took 4 hours... watching youtube videos, getting the position, looking for the gos and npcs.
If you have some blizzlike positions for the spawns, update it :-)


Ah yes, you are right. Pipe names are not case sensitive. OK, leave it the way it is (although it irks me a little that we create the string one way and then strip out the dashes)


Nice!


This does not work, it should be `\u00e4`.


When I first read this I was a bit confused about what schemas are being listed.  Maybe change this to `List the available system schemas`


We could change both to something like this:

    list(APPEND _dds_idl_files
      "${_dds_idl_path}/Sample${name}Request_.idl"
      "${_dds_idl_path}/Sample${name}Response_.idl"
      "${_dds_idl_path}/Sample${name}Response_.idl"
      "${_dds_idl_path}/WriteSample${name}Response_.idl"
    )

For the case below with the many repetitions it might even help if we reduce them using some for loop?


// Copyright 2014 Citra Emulator Project
Should be 2015 since these new files were created in 2015 :)


To be pendantic, you're closing the RPC namespace on this line and the ripple one on the next.

:thought_balloon: Wouldn't it be awesome if there was some automated tool to add such standard stuff (include guards, comments where namespaces end...) to C++ code?


@HarryRybacki Ignore this pull. I closed it. It was for debugging purposes of the front end code. The second pull request I put in fixes the JSON structure to play nice with the D3 expectations. I added comments throughout so you can probably see my train of thought.


related. This is generated code. My changes cause this space to be created.


:D


It seems that @olee shouldn't change `persistenceTimer`'s null check into `persistenceTimerTask`'s. That caused crash unfortunately, just like that in #1367...


If you initialized this as an array, then you won't need to check to make sure it's an array later.

```php
$_last_query_log = [];
```


Problem happens when activate the auto updater. And these updates are already hand-applied.
How I can get the hash to bypass them?


Oh, I got it. I need to signal the EQ event list fd.
Let me update the patch.


good point.


Agreed.

CC @tmeschter as he owns this API


It should be safe to remove the ``$lastLog &&`` and just use ``is_array``.

If you initialize the variable as an array, you can remove the ``is_array`` too.


Currently does not work. When ran comes up with this error.

$ Dim init
./Dim: line 176: unexpected EOF while looking for matching `)'
./Dim: line 182: syntax error: unexpected end of file

But that 'Should' be an easy fix.


on my machines, `i == int(float(i))` for all integers up to 1e9


several of these models are now subsets of other models so I think that's okay.


Ah, it's not Copy due to `Extension` http://hyperium.github.io/hyper/hyper/method/enum.Method.html. It's probably still cheap to clone in the general case but going to leave it for now.


return color[0] + color[1] - 0.5;


return alpha[0] + alpha[1] - 0.5;


What happens when munee.php is in /someproject and a request is received for /someproject/images/someproject.jpg? Problems! stripos === 0 and substr to the rescue :)


ojo que olvide el evento agregar grilla


I think `dlopen_e` has a deprecation. I was mostly trying to make things work quickly, not caring as much about deprecation warnings on 0.4. Is the warning coming from master or the most recent tag, and on 0.3 or 0.4, and which overwritten binding is it complaining about? I did need to delete the `DL_LOAD_PATH = Base.DL_LOAD_PATH` line below to fix a warning on 0.3, but that's not tagged yet.


Should also test usage of std::next and std::advance, because that's what was originally complaining about non-satisfaction of the ForwardIterator concept.


If you need a simple test, even just connecting to an invalid (or non-listening) address using `simple/fi_msg` from fabtests will cause the issue to occur.


please always return Transaction Response from Manager!!!!


Good point :).  Submit a PR to fix it? :)  Brownie points if you submit tests as well.


add `@NotNull`


OK


Ack


As stated you don't need to reapply it.


(I always comment my closing namespace braces, despite pressure from an unnamed team leader not to do so.  :-))


Not yet, it works but all features are not yet implemented (cmdline, initrd and gdb)


Isn't it standard practice to only write `__str__` methods and let `@python_2_unicode_compatible` do the rest?
That's why I didn't mention `__unicode__`.


typeof what?


this is my first stab at updating @sidewalkballet 's blog to reference the arcgis open data site and new smart mapping capability and rewriting in markdown for our geodev hackerlabs in PDX and Seattle in a couple weeks.

still needs more work (and embedded screenshots.)


It might be useful to explain what we are looking for in solutions. Is ten scenarios attempted, with five correct better than four attempted four correct? Also, are ten correct but inelegant solutions better than eight elegant ones?


you could revert the text-right to pull-right to fix the "menu moving to the middle" problem.. or does this fix something else? (I tried reverting it locally and it didn't seem to cause any bad changes)


Will do, probably.


Were you using `message.data` on your side?  If so, that can be catered for as I have done [here](https://github.com/rtc-io/rtc-signaller-socket.io/blob/master/index.js#L56).  If not, then yeah I'll simplify that code to exactly what you have put here.

I think the build should be passing now so it would be great if you could have a look at the current master branch and see if that code will still work in your use case. I haven't really changed anything so I think it should, I really just need to change a few things on the test server side to make everything work as expected...


In theory the output of `__html__` should be treated as safe data, regardless of the type.
But I guess it makes sense to also wrap the string methods with `safe_text`.


Extra credit for additional proposed scenarios?


this should be `metadata.catalogs`


OK, pr/sockets have the fix. Works with simple example.
Can you try when you get a chance?


How are we timing? Honor system?


sp variable


Certainly so, but floating point is contagious in Python and submodules using ``clk_freq`` may introduce rounding errors elsewhere if they were expecting an integer.

I also like to be able to use fractional frequencies (e.g. 83 1/3MHz is pretty common). For this purpose it helps to assume that ``clk_freq`` is either integer or ``Fraction``.


'in context' might be better


"Uncaught ReferenceError: foo is not defined". The first rule of Pull Request Club is - Test your code at least once before you send PR.


Postgresql explain output might be quite different than what you expect... At least it's very different compared to mysql output. Quite a lot like what you can get out of SQL Server in fact. Ok, here's one example (user 36 has just logged in and is shown tasklist with all projects, total 24, 17 active, rights to 6, global rights for all kinds of view permission 0, project specific view all for 3, view own for 2, view groups for 1, tasks total in all projects 1317, users total 63, groups total 38):
```
Group  (cost=2293.53..2293.63 rows=1 width=908)
  Group Key: t.task_id, t.task_severity, lc.category_name, p.project_title, p.project_is_active, lst.status_name, lt.tasktype_name, t.task_severity, lr.resolution_name, t.project_id, t.task_type, t.date_opened, t.opened_by, t.is_closed, t.date_closed, t.cl (...)
  ->  Sort  (cost=2293.53..2293.54 rows=1 width=908)
        Sort Key: t.task_id, t.task_severity, lc.category_name, p.project_title, p.project_is_active, lst.status_name, lt.tasktype_name, t.task_severity, lr.resolution_name, t.project_id, t.task_type, t.date_opened, t.opened_by, t.is_closed, t.date_closed, (...)
        ->  Hash Left Join  (cost=375.08..2293.52 rows=1 width=908)
              Hash Cond: (t.project_id = pg.project_id)
              Filter: (((t.mark_private = 0) OR ((t.mark_private = 1) AND ((t.opened_by = 36) OR (ass.user_id = 36) OR (pg.is_admin = 1) OR (gpg.is_admin = 1) OR (pg.manage_project = 1) OR (gpg.manage_project = 1)))) AND (((GREATEST(pg.view_groups_tasks, g (...)
              ->  Nested Loop  (cost=363.58..408.02 rows=182 width=936)
                    ->  Hash Join  (cost=2.65..11.17 rows=1 width=20)
                          Hash Cond: (gpg.group_id = gpuig.group_id)
                          ->  Seq Scan on flyspray_groups gpg  (cost=0.00..8.45 rows=6 width=24)
                                Filter: (project_id = 0)
                          ->  Hash  (cost=2.58..2.58 rows=6 width=4)
                                ->  Seq Scan on flyspray_users_in_groups gpuig  (cost=0.00..2.58 rows=6 width=4)
                                      Filter: (user_id = 36)
                    ->  Hash Left Join  (cost=360.93..395.03 rows=182 width=916)
                          Hash Cond: (t.product_category = lc.category_id)
                          ->  Hash Left Join  (cost=358.55..390.16 rows=182 width=907)
                                Hash Cond: (t.resolution_reason = lr.resolution_id)
                                ->  Hash Left Join  (cost=357.33..386.43 rows=182 width=829)
                                      Hash Cond: (t.item_status = lst.status_id)
                                      ->  Hash Left Join  (cost=356.04..382.64 rows=182 width=731)
                                            Hash Cond: (t.task_type = lt.tasktype_id)
                                            ->  Hash Left Join  (cost=354.81..378.91 rows=182 width=633)
                                                  Hash Cond: (t.project_id = p.project_id)
                                                  ->  Hash Right Join  (cost=346.38..367.98 rows=182 width=609)
                                                        Hash Cond: (ass.task_id = t.task_id)
                                                        ->  Seq Scan on flyspray_assigned ass  (cost=0.00..16.31 rows=1031 width=8)
                                                        ->  Hash  (cost=344.11..344.11 rows=182 width=605)
                                                              ->  Bitmap Heap Scan on flyspray_tasks t  (cost=13.69..344.11 rows=182 width=605)
                                                                    Recheck Cond: (is_closed = 0)
                                                                    ->  Bitmap Index Scan on is_closed  (cost=0.00..13.64 rows=182 width=0)
                                                                          Index Cond: (is_closed = 0)
                                                  ->  Hash  (cost=8.19..8.19 rows=19 width=28)
                                                        ->  Seq Scan on flyspray_projects p  (cost=0.00..8.19 rows=19 width=28)
                                            ->  Hash  (cost=1.10..1.10 rows=10 width=102)
                                                  ->  Seq Scan on flyspray_list_tasktype lt  (cost=0.00..1.10 rows=10 width=102)
                                      ->  Hash  (cost=1.13..1.13 rows=13 width=102)
                                            ->  Seq Scan on flyspray_list_status lst  (cost=0.00..1.13 rows=13 width=102)
                                ->  Hash  (cost=1.10..1.10 rows=10 width=82)
                                      ->  Seq Scan on flyspray_list_resolution lr  (cost=0.00..1.10 rows=10 width=82)
                          ->  Hash  (cost=1.61..1.61 rows=61 width=13)
                                ->  Seq Scan on flyspray_list_category lc  (cost=0.00..1.61 rows=61 width=13)
              ->  Hash  (cost=11.43..11.43 rows=6 width=24)
                    ->  Hash Join  (cost=2.65..11.43 rows=6 width=24)
                          Hash Cond: (pg.group_id = puig.group_id)
                          ->  Seq Scan on flyspray_groups pg  (cost=0.00..8.36 rows=36 width=28)
                          ->  Hash  (cost=2.58..2.58 rows=6 width=4)
                                ->  Seq Scan on flyspray_users_in_groups puig  (cost=0.00..2.58 rows=6 width=4)
                                      Filter: (user_id = 36)
              SubPlan 1
                ->  HashAggregate  (cost=12.60..12.65 rows=5 width=4)
                      Group Key: flyspray_users_in_groups.user_id
                      ->  Nested Loop  (cost=11.42..12.58 rows=5 width=4)
                            ->  HashAggregate  (cost=11.28..11.29 rows=1 width=8)
                                  Group Key: g.group_id
                                  ->  Hash Join  (cost=2.65..11.28 rows=1 width=8)
                                        Hash Cond: (g.group_id = u.group_id)
                                        ->  Seq Scan on flyspray_groups g  (cost=0.00..8.54 rows=8 width=4)
                                              Filter: ((project_id = 0) OR (project_id = t.project_id))
                                        ->  Hash  (cost=2.58..2.58 rows=6 width=4)
                                              ->  Seq Scan on flyspray_users_in_groups u  (cost=0.00..2.58 rows=6 width=4)
                                                    Filter: (user_id = 36)
                            ->  Index Only Scan using group_id_uig on flyspray_users_in_groups  (cost=0.14..1.24 rows=5 width=8)
                                  Index Cond: (group_id = g.group_id)
              SubPlan 2
                ->  HashAggregate  (cost=12.60..12.65 rows=5 width=4)
                      Group Key: flyspray_users_in_groups_1.user_id
                      ->  Nested Loop  (cost=11.42..12.58 rows=5 width=4)
                            ->  HashAggregate  (cost=11.28..11.29 rows=1 width=8)
                                  Group Key: g_1.group_id
                                  ->  Hash Join  (cost=2.65..11.28 rows=1 width=8)
                                        Hash Cond: (g_1.group_id = u_1.group_id)
                                        ->  Seq Scan on flyspray_groups g_1  (cost=0.00..8.54 rows=8 width=4)
                                              Filter: ((project_id = 0) OR (project_id = t.project_id))
                                        ->  Hash  (cost=2.58..2.58 rows=6 width=4)
                                              ->  Seq Scan on flyspray_users_in_groups u_1  (cost=0.00..2.58 rows=6 width=4)
                                                    Filter: (user_id = 36)
                            ->  Index Only Scan using group_id_uig on flyspray_users_in_groups flyspray_users_in_groups_1  (cost=0.14..1.24 rows=5 width=8)
                                  Index Cond: (group_id = g_1.group_id)
```


Yes, handlebars templates are incompatible with liquid.


How about some test w/r to library with other type? Seems it should raise a runtime error shouldn't it?


I did use `message.data` to carry the data in my first commit, but then removed it again when I started using `rtc-signal` as event. So this simplification is correct.

Will try this in my environment in a few minutes.


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

VSO State: Done


Seriously, @github for Windows? In what universe is it appropriate to ignore all of my line additions? The diff definitely showed both additions and deletions, so I don't know why it only committed the differences. At no point in time did MainForm.cs ever look like this. Maybe it's my fault for not scrutinizing the commit before pushing, but come on. This should never happen.


ok, cool - will just go with that then :)


Which class is causing this to be required?

Fwiw, since I initially did this Guice injector integration, we contribute it back to jackson and someone else came along and made it actually good.  We should consider moving over to that and just removing these classes:

https://github.com/FasterXML/jackson-module-guice


Just so you know this repo is far out dated, especially since the main one isn't public anymore


I'd even go as far as adding hidden-sm to the class on both these lines, and make it invisible for tablet too, wdyt?


I tested this patch and made sure it is destroying the VaapiDisplay on Terminate VA and destroying all the image pools. Looks good to me. +1


This is based on me just looking at the code, but I don't think this is used?


So it's a liquid issue? I'd expect everything I throw into a highlight tag to be treated like HTML special characters are.

But, hey, at least with v3.0.0, build fails and not silently breaks output :)


No, that was unintentional. Maybe we should just check for `nil` instead of blank. Makes more sense to me anyway.


Moving this is going to break other people's code if they have anything that depends on it...


Thanks for explanation, it makes perfect sense now. I have submitted a PR, that should fix the issue.

27 марта 2015 г. 1:26:50 GMT+11:00, Jack Palevich <notifications@github.com> пишет:
>The problem is that there are custom ROMs that install TEA as a system
>app,
>and copy the TEA libraries to the /system/lib directory. Then when
>users
>try to install the store version of Android Terminal Emulator, we can
>end
>up with a situation where the store version uses the /system/lib
>version of
>the shared library.
>
>(I'm not sure how exactly the two versions of the shared libraries
>conflict. From reading the bionic library loader source, It seems to me
>that the app version of the library should have library load path
>precedence. But maybe that's broken on some ROMs somehow.)
>
>Anyway, the way we work around this issue is by changing the version
>number
>every time we change the shared lib ABI.
>
>I would recommend changing both libs version numbers (the old lib
>should go
>to version 5 and your new lib should go to version 2.)
>
>
>On Thu, Mar 26, 2015 at 2:14 AM Alexander-- <notifications@github.com>
>wrote:
>
>> Oups! It was a public API all the way?! I mean, the native library
>was
>> versioned and corresponding Java native methods were very
>well-documented,
>> but there was no clear evidence of it being exported as API unlike
>> emulatorview... Btw, is there anything else I should have known
>before
>> making those changes?
>>
>> Regarding compatibility — it is totally gone. One method was moved
>into
>> libtermexec, one is removed, another one is about to be removed too.
>All
>> methods changed from using reflection on cached reference to
>FileDescriptor
>> to accepting integer file descriptors.
>>
>> I am going to submit a PR with last minor changes to native API and
>some
>> documentation ASAP. Let me know if there is anything specific you'd
>like to
>> see there.
>>
>> 26 марта 2015 г. 16:24:49 GMT+11:00, Jack Palevich <
>> notifications@github.com> пишет:
>> >Hmm. I guess we can live with two libraries. I regret that we broke
>> >downstream clients. But at least they have an upgrade path.
>> >
>> >Did the ABI of libjackpal-androidterm4.so change between v1.0.65 and
>> >v1.0.66 ? That is, are there any changes to the behavior or names of
>> >the
>> >exported functions of libjackpal-androidterm4.so in v1.0.66 ?
>> >
>> >If it did I think we need to change the version number to
>> >libjackpal-androidterm5.so
>> >
>> >On Wed, Mar 25, 2015 at 7:36 PM Alexander--
><notifications@github.com>
>> >wrote:
>> >
>> >> Hi Jack.
>> >>
>> >> Sorry for not making it clear enough before — first one is for ATE
>> >and
>> >> it's clients, using libtermexec, and later is for ATE only (stuff
>> >like
>> >> setting terminal mode etc.) I have separated them going by the
>same
>> >logic
>> >> as with libtermexec itself. If this is a considerable amount of
>bloat
>> >for
>> >> you, they can both be merged in libtermexec again. Would you like
>me
>> >to
>> >> submit a PR for this?
>> >>
>> >> 26 марта 2015 г. 13:05:06 GMT+11:00, Jack Palevich <
>> >> notifications@github.com> пишет:
>> >> >Hi Alexander,
>> >> >
>> >> >I was looking at Terminal Emulator for Android 1.0.66, and I
>notice
>> >> >there
>> >> >are now two native libraries that seem to be doing similar
>things.
>> >> >
>> >> >libjackpal-androidterm4.so
>> >> >libjackpal-termexec.so
>> >> >
>> >> >How come there are two? Couldn't we merge them into a single
>> >library?
>> >> >
>> >> >
>> >> >---
>> >> >Reply to this email directly or view it on GitHub:
>> >> >
>> >>
>> >
>>
>https://github.com/Alexander--/Android-Terminal-Emulator/commit/2c25a8ae37d28de2f848482156e701ce031e40c9#commitcomment-10403574
>> >>
>> >> --
>> >> Простите за краткость, создано в K-9 Mail.
>> >>
>> >> —
>> >> Reply to this email directly or view it on GitHub
>> >>
>> ><
>>
>https://github.com/Alexander--/Android-Terminal-Emulator/commit/2c25a8ae37d28de2f848482156e701ce031e40c9#commitcomment-10403886
>> >
>> >> .
>> >>
>> >
>> >
>> >---
>> >Reply to this email directly or view it on GitHub:
>> >
>>
>https://github.com/Alexander--/Android-Terminal-Emulator/commit/2c25a8ae37d28de2f848482156e701ce031e40c9#commitcomment-10405317
>>
>> --
>> Простите за краткость, создано в K-9 Mail.
>>
>> —
>> Reply to this email directly or view it on GitHub
>>
><https://github.com/Alexander--/Android-Terminal-Emulator/commit/2c25a8ae37d28de2f848482156e701ce031e40c9#commitcomment-10407437>
>> .
>>
>
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/Alexander--/Android-Terminal-Emulator/commit/2c25a8ae37d28de2f848482156e701ce031e40c9#commitcomment-10413141

--
Простите за краткость, создано в K-9 Mail.


instead of changing this, change the header file.


Hey Pull Request Club Leader, I just check once, can you check again? :P

En 26 de marzo de 2015 en 16:32:12, Aleksandar Erkalović (notifications@github.com) escrito:

"Uncaught ReferenceError: foo is not defined". The first rule of Pull Request Club is - Test your code at least once before you send PR.

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


On line 36 of PreferencesClerk.java, you have "BioNetGen-2.2.6" as the BNG directory name. Can that be made more robust, using regexs or something, so that the code doesn't have to be modified every time we release a new version of BNG?


Ah, I see, it's public and it's being used by other classes


Is there a reason that "c*" is bad?  I believe it is the idiomatic way that Cassandra people refer to the project...


Either you should add a default case or define $allowedEncosing before the switchh


It handles the case with python2 if you only define `__str__` that returns some utf-8 bytes data and don't use `@python_2_unicode_compatible`.
But I'm not sure if we should handle this or just let it fail with a `UnicodeDecodeError`.


Why the change to fully-qualified?


Vagrant will automatically sync any changes you make to these files from your host operating system, so you can use whatever editor/IDE you like (and have installed).


Good point


I like that, but the trouble is how are you going to customize the CSS
rules then, they are hardcoded , I guess you could just embed them in your
page, but then they still would not pick up if you view the result in
standalone.

On Fri, Mar 27, 2015 at 9:02 AM, Thomas McDonald <notifications@github.com>
wrote:

> {"sql": {"slow": x, "very-slow": y, "very-very-slow": z}}?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thomas-mcdonald/ui/commit/b50c682fb4ff678f6748506258c84b4a34e74131#commitcomment-10422283>
> .
>


I looked into this - to make a long story short, [it's a can of worms] (http://stackoverflow.com/questions/5917249/git-symlinks-in-windows/16754068#16754068).

While I, too, frown upon committing the same file more than once, I believe that on Windows either it should be done, or the pertinent `.tsclvl` level file should be modified to point to the actual OGG file.


I'm happy!


¿? ¿Dónde salta el tal Travis? :-m


Works like a charm :)


Could we maybe make this hidden if the person is not teaching any classes? Because from a student point of view it really doesn't make much sense to see this and may confuse the end users. We could just check the length of the teaching set in the view and then pass a teacher boolean that the template could use


Yes, because `c*` is not a valid URI scheme. If changing it to "URI_SCHEME" would help clarify, I can do that


Actually, this doesn't use URI anymore, that makes this a relic from an older implementation.


Guice Module vs jackson Module in the same file causes namespace collisions.


... Please implement as many of the remaining scenarios in `feathers/cart.feature' as you can within the next 60 minutes. Make whatever changes you want to the scenario implementations and source code, to accomplish that goal. ...


seems some of them changed. that or i derped


The only possible corner case would be if you were running multiple threads and deserialized in one thread and then *allocated a new model object* in another before the `_next_id` was set. The very best thing to do would be to throw up a lock on the model before deserializing and remove it after the `_next_id` is set. What do you think?


While I can do this, I worry that this will cause Composer to pull down PHPUnit, CodeSniffer, etc. that use features of 5.3+ but won't work in 5.2.


I'm still remembering and laughing at the comment this evening :)


Our contact details are in the email we sent you. (or maybe just use tech-interviews@hautelook.com?)


Same issue on cm-10.1 with 3.4 kernel


Sorry @ndeloof, completely overlooked this when I started on 71d933a840c263e0f3f3ce6998a42a7070a19648...

What's the status of this change?


![Deal with it](http://asset-f.soup.io/asset/7218/2267_f21a.gif)


Ouch, I committed this straight into master.

@RubenKid have a look. This will allow us to enable/disable toasts from the settings in Pixable


My first reaction to seeing this `if/else if` logic is that I wonder if it couldn't all be in a `CompressionUtils.decompress()` method that is just documented to do different things based on whether it is a .zip or .gz.

Also, actually, now that I think about it, the `.gz` support existed to handle one of the early segment storage formats that we had on S3 before Druid was even opened up.  We kept the code just so that things would continue to work for you guys.  If you don't have any of the old `.gz` segments on S3 that you reference anymore, it should be safe to just completely eliminate the code.


I assume it was RE: a conversation we had on IRC this morning about being easy to find information on?


And here's update to the demo, @RubenKid, FYI


90946736_WSHP_#4567 (Nigusse) - x86_64-MacOS-10.9-clang: OK (1664 of 1670 tests passed)

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


Sounds good, I'll make this change.


truth


Premake uses tabs.
Incidentally, might I suggest checking out .editorcong for use at work (http://editorconfig.org/). It installs from the VS extensions manager. I've been using it at all my offices for a while. Really helps sanitise coding standards, especially when 3rd party code with differing standards is used.


This seems to be the best method I can find of getting these environment variables in to to the call to machine.  Passing them inline wasn't working for me :disappointed:

Any suggestions @ehazlett or @slashk?  Or is this ok?


null check