labofsahil/github-event-dataset-2012 huggingface dataset:
That was more because I was having issues implementing an __init__ and save function in meeplib more than anything.  Obviously the right place for it is in there, but I couldn't really get it all to work properly from within meeplib.  I'm going to work on fixing it this weekend.  Once I get it switched over it'll only be a few lines to edit in the meep app to delete.


On Feb 8, 2012, at 9:25 PM, C. Titus Brown wrote:

> OK -- does load/save belong in the Web app, or in meeplib?  Where is all the data stored (in what module)?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/hkb261/meep/commit/a16ec637d704e9614321968f494c8162b1b2dab6#commitcomment-947949
This clean Sprockets logger but we won't modify Rails::Rack::Logger, see https://github.com/rails/rails/pull/3795#issuecomment-3549669
1. То, что ты написал -- далеко не самый общий случай merge.
2. На данный момент конструировать потоки, задавая в индексах произвольные выражения типа [2*i], [2*i+1] -- нельзя. Объясню почему. Если я объявляю элемент потока на каждом шаге как a[i] = f(a[i-1], a[i-2], ...) и никаким другим образом, то статически мне известны необходимые начальные элементы и размер окна, т.е. все необходимое, и я могу гарантировать, что это корректный поток, для которого поведение в каждом состоянии определено. Если я разрешаю писать выражения вида a[\<expr\>] = f(a[\<expr\>], ...), то статически ничего нельзя проверить. Что делать, если я задаю такое выражение: a[i*i-5*i+4] = ...;  или a[f(i)] = ...? Их можно разрешать, но будь готовым к runtime error в любое время
Need inline docs for `@opts[:views]`.
@ecki - I only tried on debian and red hat based systems.
Looking at this now (a year later), I don't know why I didn't just specify the column header.

This should do the trick:
`ps -p $$ -ocmd | awk 'NR==2 {print $1}' | xargs basename`
I added this test and "gathered" the common code for c_powers and
num_exp together and added additional tests.
You forgot a cowfile
regarding point 2) I see your point there, seeing as your test can be automated (most of the test pages I created in the past year all needed some kind of manual operation on the user's part). However, seeing as it means duplicating the logic server-side anyway, I'd still be in favor of having this in a separate file. What's the harm really? Hopefully, we'll move to nodejs at one point or another and not have duplication at all.

So, to summarize, I'd rather have good debugging and, since logic has to be duplicated, then separate the duplicates into two different files (include_js.php and concat_js.php?). Don't you think it would makes things a bit easier to follow overall?

Oh, also, I ensured all iframes using incluse_js.php would use the exact same file config as the main page in the rewrite (dunno if you noticed).
parfait. as always.
Has to?
The block is passed as an argument, so it was necessarily already constructed.
Wouldn't it have been constructed with those exact values already?
What's the purpose of wrapping `git branch | grep ...` in `$(...)`? You don't want to execute the result (which is always nothing, due to `-q`) as a shell command, do you?
Err. :-)
Fixed: 9d98450b90d5b7bc2edee82fa3e30736d4d8e659
Vim's auto-indentation should be use with more caution :(
For the time being, we want to separate all these rules.  We will condense them in the future.
Isn't better to check if player who is doing gosip to Guild Master is in guild then show him TabardDesign, if he doesn't has guild, show him Guild Creation message?
Whoa whoa, take it easy Andy. We should only replace the idle animation. Also it looks like this sprite is 128x128...
Who has made off with the .sql file?
would suggest using %s instead of +str(...) stuff.
I went into the yesod subfolder, cabal cleaned, then cabal install and it worked.
I don't understand why it was given that I was in a clean virthualenv.
Anyway. Thanks.
Guilty on all charges ;-)
这个是用来说帐号密码错误提示的
@Thermaltake Спасибо, собственно так и сделал =) В смысле LogTimestamp установил =)
if you want this to be the `profile.favorites_collection()` from line 106 you'll need to rename this to something else since you have a different `fav` you've defined on line 107
是为了和root-group区分

其实没什么意义- -
https://gist.github.com/d4a248683772108d753f
ev. return "." ;) ...nur so als idee^^
The Runner had a bug that cause an infinite loop if a not implemented action was added to a Driver, and that action was called later on.
It was just a test. I try it for 1M iterations for Re = 7.5k and higher and the results wasn't good. I will try another number of iterations.
That's embarrassing...I'm sorry.

Thanks Adam. Clearly I didn't bother to recompile MiniB3D after I made the changes! I guess I just copied the code from Prime.Math3D and left it as is.

I will push new commits (for anyone else who stumbles upon this) when I get a chance.
Ye. It is called all-threads-crash-dump.
Folder : Map,Vmap,DBC in folder /server/data/ and Bin file in /server/R2/bin
Not lag yet, but why only spaws statistics for map 571?
http://pastebin.com/238LyXXz
I supose because is the map taking most time to update, right?
Also if i want info from more maps i should increase VMSS.Statistic.Threads to same Map.UpdateThreads?
If still right, statistics printed are getted from maps being updated at print time?
ah si, lo que pasa es que se los mande por email a rsa, ahí van:

http://pastebin.com/QJVAzk88
http://pastebin.com/EU478nRZ
http://pastebin.com/SXCKP96A
http://pastebin.com/tnCfBeJ6
http://pastebin.com/BMX6cnqn
no, for GM's check passed in any case. custom areatriggers need only for players.
Nope, it works: http://cl.ly/313v1t1s1C2k0a1V3n40
On Sun, Jan 22, 2012 at 3:11 AM, Charlie Sharpsteen
<reply@reply.github.com>
wrote:
> This could also be expressed more compactly as:
>
> ```ruby
> (etc+'dict.conf').write <<- EOS
> ```
>
> Which spares the need for saving the heredoc in a variable and then writing out the contents of the variable.
>

Thx for the tips. I fixed it, rebased it and repushed it.

-- 
Nicolas Desprès
D'oh, you already did :)
What are you objecting to exactly? The use of List rather than Array, or the removal of the messy unused entries stuff? Both choices seem pretty sensible given how this is used..
maybe `help_text` or something to indicate what this flag is for?
Where is +IV-CONSTANT (+IV-CONSTANT+ ?)?  I don't see it anywhere in this patch.
Hmm, perhaps this is better named `slide_section` thoughts anyone?
I think it's worth making this function callable like RANDOM with an &OPTIONAL (PRNG *PRNG*) argument.
Thanks for catching this. Fixing it now.
Hi Twobee,
Thanks! I wasn't aware of that limitation, that makes a lot more sense why display errors has not been implemented.

cheers, - Paul
Cool - been thinking we needed to do this (hover is a pretty mobile hostile concept).  I'm thinking of tweaking the layer selector to do something similar (or, at least, a click-to-open with some sort of timeout to auto-close, versus the stay-expanded default behavior)
Perfecto, gracias por la respuesta.
+/**
 	 16	
+* нужно закачать кнопки, они уже нарисованы... 
 	 17	
+* http://ttl.loc/templates/administrator/images/tree/add_page.png
 	 18	
+* http://ttl.loc/templates/administrator/images/tree/add_subdir.png
 	 19	
+* http://ttl.loc/templates/administrator/images/tree/edit_dir.png
 	 20	
+*/
wtf?
Be consistent with indents. Also CSS rules write consistently (space after `:`). Even more: never use inline CSS.
BacklogSettingType? since its not really the status.
>  If you have a browser with such a bug, let me know. But Chrome 8 doesn't really count.

Heh so you basically just wrote *"Show me where it break... Oh, you did? Well that doesn't count so show me again."*

The old detection was better than the current "fix". I already pointed
to a mobile device that has a Chrome 8 like browser (or at least V8
and isn't auto updated like the desktop flavor). So the newer weaker
inference is a bust. The solution is to revert and add 

~~~ js
fake.style.background = '';
~~~

> Additionally, the HTML5 styles are mainly attached for setting display: block for mostly only sectioning elements. The audio/video/[hidden]/mark part is just a bonus and not important/don't have to be bulletproof.

The `[hidden]` attribute seems to be pretty important as its controlling display also it's style is unlikely to be reversed or mucked with by other stylesheets on the page so it makes for a better test.
Since we want this to be False, do we need this code?

Actually, we should probably just delete this code.
thxxxx
Will this throw the non-ssl pods offline?
Issue in thread 20:

Thread 20 (Thread 32686):
#0  0x00007ffc2ded0e87 in kill () from /lib/libc.so.6
No symbol table info available.
#1  0x000000000069d419 in Master::_OnSignal (s=11) at /root/mangos/src/mangosd/Master.cpp:648
No locals.
#2  <signal handler called>
No symbol table info available.
#3  0x0000000000c64031 in dtNavMeshQuery::findPath (this=0x7ffadc671ed0, startRef=<value optimized out>, endRef=220676394626057718, startPos=<value optimized out>, endPos=0x7ffc20c2a460, filter=0x7ffc20c2a858, path=0x7ffc20c2a5a0, pathCount=0x7ffc20c2a7f0, maxPath=74) at /root/mangos/dep/recastnavigation/Detour/Source/DetourNavMeshQuery.cpp:650
@lloyd - Change from "MISSING" to "NOT_SUPPORTED" for clarity.
From our bootstrap.js:       

     var modules = [
         this.jqueryPath,
         'lib/jsuri.js',
         'lively/Migration.js',
         'lively/JSON.js',
         'lively/lang/Object.js',
         'lively/lang/Function.js',
         'lively/lang/String.js',
         'lively/lang/Array.js',
         'lively/lang/Number.js',
         'lively/defaultconfig.js',
         'lively/localconfig.js',
         'lively/Base.js',
         'lively/lang/Closure.js',   // FIXME: require module instead
    ];

Right now Lively.Closure is a class and Base delivers the class system, but requires, for example, the _Functions_ helper that gets defined in lively/lang/Function.js. So, if you really want them to be in a single file, we either need to remove dependencies from Base to lively/lang/Function.js, I guess.
To maintain future compatibility, it might be better to specify a comparator functor explicitly when instantiating the set template rather than adding stuff into the Qpid namespace. For an example, see the AddrComparator in  mh_object.cpp.
you would
Is ownerKey used anywhere?
It will be a button with a picture on it
Don't use the built in eclipse for matter!
And this one again http://paste2.org/p/1885647
glad to help! I hope to contribute in the future as I use openphoto on my server!
Change the default title to "Microtime"
Font-size 1em isn't necessary unless resetting another style. For spans this should be default already.
Bumping this just as a reference to @immortius as he's been tinkering with the idea of an event system tied to the ES - unsure if these would relate, but just in case :-)
I wish I'd known you were working on this - just been working on the same myself :)
What's the reason for reverting this?
this only works with python 2.6+, go test with python 2.5. look at how we import json all over sb for example on how it should be done
Using git update-index --assume-unchanged sumimprovement.db to prevent DB changes from being committed. See https://46bit.com/blog/2011/08/dev-sparks/.
Okay, let's see how that goes. Thanks!
thank you very much!  could you please

* provide a file with sample log lines (ideally at least one per each of the entries in the failregex) -- should go under testcases/files/logs/asterisk
* anchor the ends of the lines in the failregex with $

Thanks in advance
I think we should give "403 Forbidden" status instead of "401 Authorization Required" when credentials are wrong. Can you check the specs?
It is an internal non-public distribution.  It has a lot of dependencies, and I couldn't track down the specific origin dependency that was causing it.  It was failing ultimately on common::sense.  I would like to be able to add a test for this, but I'm not 100% sure as to the exact scenario that reproduces it as I'm brand new to the code.  Thoughts?
http://www.facebook.com/psiplus
What happened to Unk UInt32 4/5?
If it was removed on Cata it should have an if with "if removedinversion(4.2.2) readuint.."
Will this compile? \& is needed?
@Lokaltog my MacVim become much slower after this commit. And it especially noticable with LustyExplorer. It opens files almost immediately with this commit reverted, but after it, there's 0.5-0.8s delay - it's noticeable and not comfortable for me anymore :-(
Oops. I see the mistake I was making now. I forgot that `[1, 2, 3] === [1, 2, 3]` is 'false' because they are two different objects. Still, you have a problem here that I was trying to correct. The JSON object is only available in newer browsers; that means IE8+. See https://en.wikipedia.org/wiki/Json#Native_encoding_and_decoding_in_browsers
really not sure if this is required, but:

are you sure you treated m_bIsHeroic correctly?
Because i think in ToCRaid most special cased named "heroic" would only happen in 10H or 25H, but not in 25 mode

Also I wouldn't add here an empty Reset or UpdateAI
I should have said last except for max!
True, it's easy !
I am glad it's working now but there are still a few things I would like to figure before blindly merging things. As I said in previous posts, I am more interested in the reasons why the patches are needed d than in patches themselves:

1) libsnes fix to state.c seems more like a hack to me and I'm sure I can find a solution that fits everyone's need if you explain me exactly why those changes are needed. As far as I can tell, the only changes are the name of the functions parameter and the returned size when saving. I can't figure how the first one could be an issue to the frontend and as for the second one, it should be up to libsnes to simply ignore the returned value of state_save and use STATE_SIZE instead. 

2) I still don't get why include to stdlib.h need to be moved out m68kops.h, this does not make any sense to me so can you at least give me the error or warning you are getting ?


3) why are custom blitting functions needed for libsnes in NTSC filters ? is that because it uses 15-bit RGB ? in that case, a better solution would be to use the existing define for that. Also, I think there is a #ifdef in vdp_render.c that prevents using NTSC filter if anything else than 16-bit RGB is used. Has this been modified as well ? Again, a better understanding of what libsnes is specifically needing can help me figuring a more portable solution.

More generally, what I am trying to do is to get ride of port specific defines (like those existing #ifdef NGC as well) and figure better portable solutions, because I doesn't feel like doing all these stuff again for every future platform ports... In my current implementation, this is already true for specific defines in vdp_render.c, sms_cart.c and system.c (now passes the sound buffer pointer to audio_update function). 


Some other stuff you might want to fix in your ports:

- the framerate values you are using when calling audio_init are SPECIFIC to the Wii/Gamecube ports, as those values are only required to keep Wii/Gamecube audio & video hardware in sync, using exact frequency when VSYNC is used. In my current (uncommited) version, this is hopefully better explained and I added an option to force VSYNC OFF. audio_init & sound_init functions have also been modified so that passing 0 as framerate parameter forces the emulator to run at the original clock speed instead. As you probably don't care about that on PS3, better use default framerates.

- similarly, you shouldn't care about interlaced mode changes (bitmap.viewport.changed & 4), this is only for Wii/Gamecube ports, again when VSYNC is used in original video mode. In this mode, video mode is switching between non-interlaced and interlaced like a real Genesis would do and VSYNC timings are changing, hence why I need to reinitialize framerate value and audio timings to keep everything in sync.
Unstable for Public servers? They are not supported by Trinity xD
ok, the greeter seems to be working. the box is still failing the test, though using the test variables with the box results in output that works, using prints and all. 

i keep getting the error "'list' object is not callable", which doesn't seem like it would be the result of my code given that im not using lists.
a textarea does not have a `value` field. instead, it's "value" is everything that is between the `<textarea>...</textarea>` tags
see [here](http://www.w3schools.com/tags/tag_textarea.asp) . Indeed, if `value` is used, the engine.io server does not receive anything in it's `d` textarea
because of possible null pointer in handlescript :P
I have expirenced problem after this. For example when DK using Acherus Eye i have in log - HandleSetActiveMoverOpcode: incorrect mover guid: mover is 17379391440358862097
(creature - Entry: 28511) and should be 12
This is a needless change, IMO.  metavars are for display purposes only for the end-user.
When exactly does LaunchServices NOT launch Quicksilver? Most of the time?

If that's the case, then this is going to rescan unnecessarily every 10 minutes. If it's rare, then I guess it's fine.
Maybe we could make this static and move it into Concept.
The package with the source files.
Maybe better to name it such a getter.
I understand the point as I also play currently with the C on my data.
I would go for C='auto' by default that corresponds to C=n_samples.
The default behavior would then match libsvm/liblinear C=1 and it would
be valid for cross-validation.

WDYT?
``` erb
<% @location.delivery_list.each do |location| %>
  <%= location[:name] %>
  <%= location[:address %>
<% end %>
```
This change breaks the labels for widgets that extend LabelWidget, CalendarWidget, SelectionWidget, or MutliSelectionWidget ... because they used to have their own names (so would get a label rendered in field.pt) but now they inherit render_own_label = True.

For backwards-compatibility we should do checks like, e.g. in CalendarWidget:
    def render_own_label(self):
        return bool(self.getName() == 'CalendarWidget')
True. Checking the diff is good enough.  But when more people begin contributing (*fingers crossed*), might get confusing.
Steven, the changes looks good to me. Please add the proper license header to the new files and add your name to the license header of older files. Thanks again.
Hi Dav,
I guess this project is not maintained anymore...

Is it still necessary to use YUI to calculate module dependencies ? 

I tried to just concatenate the files , and it seems to work great...
So I posted this project : https://github.com/neyric/express-yuicombo
Are we need operate on fall or jump????
You should 'slugify' the title - so if I enter "My New Post", the filename turns into "my-new-post"
removed in a new commit
you motherfucker
beter geen strtotime gebruiken op datumformaten. strtotime is een fantastisch ding, maar op datumformaten is 't heel picky; zie bijvoorbeeld http://www.php.net/manual/en/function.strtotime.php#100144
Ik zou dus eerder zelf dag/maand/jaar uit de datum extracten, of op z'n minst nog enkele operaties uitvoeren op de doorgegeven datum, om te verzekeren dat die volgens een bepaald correct formaat is.
De datum die je doorgeeft vanuit de widget is y-m-d, dus 12-02-09; als je dit naast de bovenvermelde post vanop php.net legt, die je dat dit kan foutlopen :)
Use !empty instead of isset for username/display_name check since display_name is returned no matter what in the response.
That is an option. The advantage to letting Socket.IO serve it itself is that updates will be automatically propagated without needing to recopy the files. In production mode it is minified, etagged, gzipped, and served statically to minimize the total impact.
Depends on how you blog, I guess. You could use it to post news or status updates, but maybe that's what you're using Hongfire for.
the fix is required if u use the 2.6.35 cpu freq dirver (the one in the new semc kernel sources)... the recent one i use is my own modified cfpfreq driver from 2.6.32 caf msm kernel, which doesnt need that fix...
Why are we keeping the old code around?  It's in git if we ever want it.
this should be removed as ``void`` does not make sense in PHP (you would get ``null`` if you get the result of the method). And this is even part of the coding standards: http://symfony.com/doc/current/contributing/code/standards.html#documentation
I think the current behavior is correct. A constraint at a getter method should be validated for _Validator#validate()_ and for method level validation. If we introduce a flag the current behavior should be the default. Not sure whether we need to specify this now. Maybe the OVal people will comment on this!?
Thanks for the advice, you can even say for the care munjeni
This makes no sense. the variable cannot be null after a constructor invocation.
Makes me wonder if not invoking shell.getShellServer() really is semantically correct.
thx for the heads-up

in what way exactly ?

so in this case you either have more smoothness or better colors (voodoo),

yeah, there was a discussion in the past that afaik e.g. 72 was even worse (due to the jumping FPS) and if I recall correctly also more voodoo issues ...
no problem :)

just add GalaxyTeam, OK :)
yep.. only froyo and eclair.
Sorry, but I changed the tag to `multi-node-testing` to be consistent with `multi-jvm-testing`
I don't think Viktor ment a package file, but just the order of the imports, i.e. place the language import at the top
There is a one letter difference between my version and the original. If I change the segment line to be -2 instead of 3, it actually fits more letters than the original (without clipping).
Having lowTri as a fit method was a hack for creating a Tensor from previous fit results. This should probably just be replaced with a constructor that returns a TensorFit from previously calculated fit values.
why is this dict wrapped in ()? this isn't needed
This needs to be generalized for plus cards
Yes, c+p error, sorry about that. Should be corrected now.
Тоже не нужно, кэширование в yii достаточно
int(form.cleaned_data['action_type']) - лишняя проверка. Потому что она делается уже на уровне формы в методе is_valid.
Remove this enum class
It's been discussed to death already.. It's a sacrifice that has to be made for ORM's ability to run code on delete etc. Any other way of handling it would likely require significant changes to ORM..
The in-string procedure may be dead (I don't see any references to it in this file or the demos).  Maybe verify that that's the case and just delete it?

I think the use of expr->string in frp-snip.rkt should be exercised very rarely -- afaict, you'd need to create a switching signal that starts as an event and then becomes something else.  If you want to try it, set the language level to FrTime and do:

(define c (new-cell (changes seconds)))
c
(set-cell! c "not a number")

This desperately needs to be cleaned up, but for now is it safe to just replace it with (format "~a" ...)?
This is the non-chunk data at the end of the string (if any)
... or define this exception in a more generic location if you don't think you need such a specific exception?
Very nice! Great job Endre!
This is what we do everywhere else.
Can you please revert this change? I don't think there's been a good discussion yet about how to handle TwUI version numbers, but we shouldn't bump it until that happens.
That's odd.... I'll fix that.
Maybe I don't understand the use cases here.  Why do you want to use raw byte buffers with Process, but float arrays with ReadArray/WriteArray?
function Deferred() {...} compresses better as well (usually), and I tend to do it that way as well.  Will change.
Could you not use <code>rose.resource</code> here?  (here=popen.py, line 206)
Spaces around the `*`
J'ai roulé le scaffolding de nouveau pour avoir la base d'un test de contrôleur avec rspec
I don't see why this is a bad thing.
Yep. I do this with all my HTML, it's much simpler, and <head> doesn't really do that much.
это надо как-то обобщить с generateProperties()
I rebased and updated :-)
Good catch, seems like that should have been picked up by a test...
This may change the file's encoding
Perhaps you can just do:

```ruby
Class.new(Post) do
  belongs_to "author"
end
```

Then the model wouldn't be necessary.
I prefer

```
options.is_a? Hash
```

over

```
Hash === options
```

would also maybe change `options[:method] => options[:method].blank?` to protect against anyone using an empty string
Ha ha, github interpreted my

    tpe_*

uses as emphasis between the underscores.
For CaseDef I've expanded the information about the phases that process pattern matching. Since 2.10 it's patmat. Before 2.10 or with -Xoldpatmat it's explicitouter.
`.normalize` is redundant here, right?
maybe a comment would be nice: 'returns all distinct symbols with name "CC" that occur in the signature of method `inMethod` of type `T`'

PS: could also be good to replace "CC" by name in the code (and my proposed comment)
What does the string version of this look like?
you mean "verisons", right? (or are there others)?
Btw, this is only program docu, not public api docu. But never mind, will fix it.
Dominik
Will `subdiagram_from_objects([A, C])` work? What if `g * f` is not explicitly provided?

Perhaps clearer: what properties of the generators ensure that any composite of generators which has domain and codomain in a specific subset of the objects is a composite of generators which all have domain and codomain in the specific set?
Oh, that's right.
This can be left as Matrix. Though now that I'm thinking about it, MatrixBase may be better, so it catches any of the *Matrix classes.
>
> ```py
> args = list(self.args)
> ```
>
> That creates a new list doesn't it.
>
>
Elsewhere I think I use args = self.args; args = list(args). Does that
create and instantly delete an intermediate list or do you still end up
with two lists in memory?
from sympy.core
Don't use a bare except. We know exactly the case where the logic here fails, so I think a try block is unnecessary.
I guess create another rule for pulling out the coefficients.
Well, if we can't have a sufficiently expressive way of representing permutations, I think this will just cost us more in maintenance and work-arounds than it's worth. Meanwhile, couldn't you use something like

```python
def permutation_matrix(s, n):
       return FunctionMatrix(n, n, KroneckerDelta(s(i), j)
```
with `s` some callable like `s = Function('s')` or `s = lambda i: A[0, i]` if you like row vectors? I didn't test it but it ought to have nearly the same behaviour and performance characteristics as this PR.
Yes, this is bad.  We really need to create a `DummyFunction` already.
the first `condition` should e renamed `expr`. I see that this should be changed in `sampling_E` as well (my mistake).
Exact, but you will (almost ?) never write it as more atomic expressions: you can replace it by, for example, a dimension and a number, but we never do this. Si for me `Unit` was an atomic thing, even if inside it gathers several objects. But if you think `Expr` fits better I can change it.
Should $q be removed from the comment doc?
Could you remove this, and add documentation to load this class thanks to composer ?
This first section should be [Lifecycle](https://github.com/github/objective-c-conventions#documentation-and-organization).
Really good idea - amended in [400138e](https://github.com/josh-/slate/commit/400138eb2a3af8506c8bf447b69073036ca0cdbb).
Handle EINTR here.
Add comment that this method should be executed only from @thread
I recommend you to install IISExpress. See its benefits here: http://weblogs.asp.net/scottgu/archive/2010/06/28/introducing-iis-express.aspx
Yes, but a goal of tests is to identify problems, if this test fails you first need to start looking whether the form didn't exist or if logging in didn't work. Tests should be locating the problem as narrowly as they possibly can. You can also use @depends annotation to make sure that a test is not run if another doesn't work. So you can have one form to check if the login form is displayed, and then a test which submits the login form, each only asserting what they test. With the login form submission test not even running if the login form display test fails.
Indeed, but as @bantu points out the name of the function should really be phpbb_style_is_active($style_id) then and not phpbb_validate_style.
Document in docblock.
I have tested this on lighttpd with fastcgi
In olympus, no new line should be present.
$readable_forum_id doesn't seem to be a single id. Is it an array? So it should be $readable_forum_ids
Probably does not need a line break between the other objects and this one.
Plenty of other functions in the phpBB codebase use the 'gen' abbreviation for 'generate'. Why is this any different?
```php
$username = $username ?: $user->data['username'];
```
Should be sqlite3 configuration, not mysql.
Trailing whitespace can be removed?
Why don't use DOXYGEN_DOT_FOUND? Why is HAVE_DOT heeded?
No, in this particular case dimension represents width and height sizes. Edge usually means distance from centre like clipping edge or something similar.
I'm still understanding the cases to force constant search through ::.  Why ::Shoes here?
Let's just pretend I was sleeping then.

Delegation is great.

Peter Fitzgibbons
(847) 859-9550
Email: peter.fitzgibbons@gmail.com
IM GTalk: peter.fitzgibbons
IM AOL: peter.fitzgibbons@gmail.com


On Wed, May 30, 2012 at 8:32 AM, Andy Lindeman <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,17 @@
> > +require 'delegate'
> > +
> > +module Shoes
> > +  module Logger
> > +    class Log4j < SimpleDelegator
>
> I'm trying to understand your reservations about the delegator here, but
> not seeing it.
>
> My goals were:
>
> * Provide some logging mechanisms totally decoupled from anything else in
> the app
> * Allow other code (including code outside of the `shoes` gem) to register
> and use new loggers
>
> The delegator is simply there to provide setup (e.g., requiring java)
> before delegating things like `#debug`, `#info`, etc.. to the "real"
> instance.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/shoes/shoes4/pull/26/files#r900040
>
Such beautifully unholy generics. :)
http://yuilibrary.com/projects/yui3/ticket/2532854
My understanding is that the URL addon is in charge of generating URLs, not file system paths. As such, we should not be using libpath here at all, or else we get URLs with backslashes.
If you don't want to implement a method yet, you better throw an exception, so people wont be bugged for hours.
Is this function required here and in Util.js?
text is wrong. 
* not a drug purchase order - this is a billing item
Will axes=2 cause an error if x or y has fewer than 2 axes?
Zen comment -- what is the sound of hash mismatch with no mismatch?
Should be equivalent to blpop, or?
it is pre-incr on purpose?
So if a user decides they want to enable entropy now they need to restart Riak for that to take effect.  Should we add a function to entropy mgr to allow dynamic adding of child?
Can yoffset have a value that is neither 0 nor continuous with the previous function?
these params can be removed since they arent used
Oh s*** i did mean to do that, been a long week!

On 31/03/12 21:11, Joe Groff wrote:
>>
>> -private define noPrecBinaryOp;
>> -[OP]
>> -inline overload noPrecBinaryOp(forward a, #OP,forward ..args)
>> -    = forward a,OP,..noPrecBinaryOp(..args);
>> -[OP,T,U when BinaryOpDefined?(OP,T,U)]
>> -inline overload noPrecBinaryOp(forward a:T, #OP,forward b:U, forward ..args)
>> -    = forward ..noPrecBinaryOp(OP(a,b), ..args);
>> -[OP,T,U when integerEquals?(countValues(..CallOutputTypes(OP,T,U)),0)]
>> -inline overload noPrecBinaryOp(forward a:T,#OP,forward b:U, forward ..args)
>> -    {OP(a,b);}
>> -[OP,T,U when BinaryOpDefined?(OP,T,U)]
>> -inline overload noPrecBinaryOp(forward a:T,#OP,forward b:U)
>> -    = forward OP(a,b);
>> +overload binaryOp(..args) = ..args;
> The `binaryOp` overloads should probably all be inline, and forward their arguments and return values
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jckarter/clay/pull/251/files#r630109
This fixes the warning and does not seem to be buggy as I used the t1 and t2 previously. :)
```
    Matrix cv_x = x;
    Matrix XXTrans = trans(cv_x);
    XXTrans = cv_x * XXTrans;
```
This seems a little magical to me. Why 24? Is there a link you can provide to document this?
fixed in 2c7d21e9929f966fdcd8d00827d99fb74ae6cb27
No need to change this to md5 - keep it as sha1.
Autoload is preferred to require
Je vois pas pk vous faites tous le temps des runtime exception c'est chiant, il y a pas la ligne d'erreur ...
I checked and I'm fairly sure request.args contains undecoded byte values (.args is constructed using urlparse.parse_qs or the form-encoded data equivalent).
I'm fine with pulling the examples out of these commits, if that's what you want.
It's a tiny point, but there's an extra blank line here.
```
from teleportd import Teleportd
```
would be awesome otherwise
```
from teleportd.api import Teleportd
```
Use `sys_get_temp_dir()` here.
Went the second way (`.zip` rather than requiring `--file`), but left commit history with `--file=file.zip` just in case we change our minds.
Hi KASPAR - no such configuration is necessary - it is definitely the default behaviour from fluid.set as you can see from the test cases. Can you describe the error message you are getting and what the situation/arguments are when you get it?
I'm committing packages into git, so this is not strictly necessary. But doesn't hurt either.
I did not know the answer to that so I took the conservative tack of retaining this functionality in case someone relies on it.
Can you put the comment on its own line?  (Style preference.)  Thanks.
Can remove this MKNetorkKit include
These styles should be in the component `_playlist-button.scss` file.
position:fixed should work fine. We rely on scroll events only as a trigger to redraw the highlights, so the position of the elements doesn't matter.
Unlike on event, we're not checking anything here. So do we still need this one?
Hmm.  It seems like the ideal thing to do would be either (a) make it break on path separators (but still not break on any/all chars), or (b) some form of smart truncation (some native apps preserve the start & end of the path, but replace the middle with ellipses).  I think (a) might be relatively easy if we can just inject "zero-width space" chars or some such in the display string... e.g. http://stackoverflow.com/questions/2046530/whats-the-opposite-of-a-nbsp

In the interest of getting this landed soon though, I'd rather tackle that separately (this patch isn't changing the existing way paths/URLs are displayed... it's just moving the old code up a few lines).  Would you mind spinning this off as a bug & assigning to me?
It looks like this removes a line that wasn't added by the ALF checkin.
None of the Windows editors I tried select the line when you go to it; most just put the cursor on column 0.  Personally I think that's better, since usually when you go to a line you want to read it (rather than copy/cut/delete the whole line), and it's easier to read without the selection highlight.
I do not see a wo.set_arguments() in WavelocOptions class...  How does wo know which arguments to set ??
Doesn't sound very idiomatic. {} is for making blocks. 

On Nov 15, 2012, at 3:55 PM, Jesse Williamson <notifications@github.com> wrote:

> In c_src/eleveldb.cc:
> 
> > + ErlNifCond*    work_queue_pending; // flags job present in the work queue
> > + ErlNifMutex*   work_queue_lock;    // protects access to work_queue
> > +
> > + bool shutdown;                     // should we stop threads and shut down?
> > +
> > + public:
> > + eleveldb_thread_pool(const size_t thread_pool_size);
> > + ~eleveldb_thread_pool();
> > +
> > + public:
> > + void lock()                    { enif_mutex_lock(work_queue_lock); }
> > + void unlock()                  { enif_mutex_unlock(work_queue_lock); }
> > +
> > + void submit(work_item_t* item) 
> > + { 
> > +    lock(), work_queue.push(item), unlock(); 
> I sometimes use them (more often, lately) to indicate "grouping" rather than "separating" with ';'-- especially when written on one line as here. The intent is to show visually that the three operations always go together. It's easily argued that the function itself already does that and should be three lines of semicolon-separated code, but that's the reason.
> 
> —
> Reply to this email directly or view it on GitHub.
>
pushed changes. think i addressed all this. changes visible on PR #237.

i copied that markup from elsewhere in the codebase so some of the issues
you list (particularly <small class="label">) are out in the wild.


On Wed, Nov 14, 2012 at 1:17 AM, Biyeun <notifications@github.com> wrote:

> In corehq/apps/users/templates/users/account.html:
>
> >                          {% if forloop.first %}<legend>Registered Phone Numbers</legend>{% endif %}
> >                          <div class="control-group">
> > -                            <label class="control-label">+{{ phone_number }}</label>
> > +                            <label class="control-label">+{{ phone.number }}
> > +                                {% ifequal phone.status "verified" %}
> > +                                <br><span class="help-inline"><small class="label" style="background-color: #468847;">VERIFIED</small></span>
>
> Okay, lots of no nos here.
>
>    - Help text should not be in a label.
>    - if you want help text to be on a new line, don't use help-inline,
>    use help-block. Please familiarize yourself with bootstrap forms:
>    http://twitter.github.com/bootstrap/base-css.html#forms
>    - Be consistent with label background colors. Absolutely no
>    background-color modifications in the style tag.
>    http://twitter.github.com/bootstrap/components.html#labels-badges
>    - labels are either span or div elements, not small. put the small tag
>    inside the label if you wish to make the text inside the label small.
>
>  —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/core-hq/pull/236/files#r2120616>.
>
>
I have converted all old, non-skinned dialogs to use new ones. I have noticed that even though MessageDialog does support two distinct types, message and status, they both look exactly same. I have redefined status one to look more like a progress dialog, similiar look. I have also fixed imdb plugin to use the new style dialog.
I think comparing `start` to `end+1` would be better before creating the line string since it saves having to call `substring` altogether.
It seems possible to use unicode characters for branch names:

```
[aaron@higgins rugged (こんにちは)]$ git status
# On branch こんにちは
nothing to commit (working directory clean)
[aaron@higgins rugged (こんにちは)]$
```

Returning ASCII-8BIT is safe for now, but we really need to figure out what the encoding policy is for git internals like this.  Is it always unicode?  Does it depend on the filesystem?  Or something else?
Just hard-code. I would not raise ValidationError here but some other (like AttributeError).
Please give better feedback to the user than "Submit". "Register" is a good candidate. ;-)
Yap, it looks better. I've pushed another commit for this. Thanks!
Again, this docstring does not describe what the method is doing, and the wait should be after the action that causes the change in UI.
This mark has no effect. If the test is destructive it should just not be marked as nondestructive.
Oh, right that makes sense.. haha
Yeah, just move this to the top of the stylesheet you include
Hahahahh I don't even know... what happened.
I agree. However, I think that's another enhancement on top of this ticket. File a new one?
Yes. Very much. They all have `butter-dialog-button` as a class instead.
Yes, I'd done the ridging dividers, but never committed them, because I pulled in the transparency change before pushing it out. So no, you didn't clobber it.
Is `!important` necessary?
Yes - I am clever like that - FYI the deploy script is at: https://github.com/mozilla/source/blob/development/bin/update_site.py
It's used by CreateView to redirect after successfully saving. Since this is covered in the example above it could be removed, but then the example would result in an exception if copy-pasted directly.
`FilesystemIterator` will skip dots by default, allowing you to get rid of the special check.
Looks like you accidentally blew away Selleck's custom Handlebars language handler.
at least some logging?
发现nginx -l选项无效，不能显示动态加载模块的指令
Need to figure out best way to fence this since including this gem breaks rubymine and excluding it apparently blocks other debugger styles.
You actually don't need so specify the filter manually anymore.  see package.rb or errata.rb:

  def self.index_settings
    {
        "index" => {
            "analysis" => {
                "filter" => Katello::Search::custom_filters,
                "analyzer" =>Katello::Search::custom_analyzers
            }
        }
    }
  end
What is the reason for

    add_index(:custom_info, [:informable_type, :informable_id, :value], :unique => true)

exactly? Didn't you mean this:

    add_index(:custom_info, [:informable_type, :informable_id, :keyname], :unique => true)

Can you elaborate why would we need to include the value in there?

Regarding the index for keyname - I don't really think it is necessary, more indicies means more resources for CUD.
This is slooow. No need to return here all keys.
Test it please using
len(rec) > 0
Maybe we don't need to throw here anymore, but istead just return CancellationToken.None.
I think if we let it propagate it will be  caught by window.onerror and reported.  But then testApp() won't be able to continue and shut the app down correctly.

I thought about registerCleanupFunction(), but if I used that, then each test could only call testApp() once.  browser_all_apps.js calls it many times.
No problem - I was just curious, but I'm not a cosmologist, so I have no authority here :-)
Maybe the use of `numpy.broadcast_arrays` could be of some help here: it would do out of the box shape matching and avoid creating an array filled with zeros that in some cases may be huge *and* useless... or maybe there are corner cases which I didn't thought of ?

side note : how do you do a comment based on multiple lines ?
My thinking was that thru SEO, google, and us sending people the link  / link sharing, some people might actually end up on this as the first page.... don't care a whole lot either way  but thought it might make sense.
I don't think so. The reason the QWidget gets changed is because in Qt 5 compared to Qt 4 widgets have been split from the QtGui module into a new QtWidgets module.

See here http://www.kdab.com/porting-from-qt-4-to-qt-5/
OK fixed, thanks for the feedback.

On Fri, Sep 28, 2012 at 7:55 AM, ADmad <notifications@github.com> wrote:

> In lib/Cake/Cache/Engine/WincacheEngine.php:
>
> > @@ -183,6 +183,7 @@ public function groups() {
> >   * @return boolean success
> >   **/
> >  	public function clearGroup($group) {
> > +	        $success = null;
>
> Indentation is still incorrect, better use the code sniffer.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cakephp/cakephp/pull/872/files#r1714815>.
>
>



-- 
James Michael DuPont
Member of Free Libre Open Source Software Kosova http://flossk.org
Saving wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com
Contributor FOSM, the CC-BY-SA map of the world http://fosm.org
Mozilla Rep https://reps.mozilla.org/u/h4ck3rm1k3
Free Software Foundation Europe Fellow http://fsfe.org/support/?h4ck3rm1k3
I see. And I now realize this is related to your changes to the WFSGetFeature and WMSGetFeatureInfo plugins.

We should make sure these changes do have negative impacts on the FeaturesWindow plugin, which also listens for `querystarts` and `queryresults`.

I think the `querystarts` `queryresults` sequence logic should be unit-tested. I'm volunteering to take a look at it.
im sorry I dont understand what me to do here

> In core/src/script/CGXP/plugins/GetFeature.js:
>
> > +                }, this);
> > +            }
> > +        }, this);
> > +
> > +        return {
> > +            internalLayers: internalLayers,
> > +            externalLayers: externalLayers,
> > +            unqueriedLayers: unqueriedLayers
> > +        };
> > +    },
> > +
> > +    getMessage: function() {
> > +        var tpl = new Ext.Template(this.queryResultMessage);
> > +        var key = 'CTRL';
> > +        if (Ext.isMac) {
> > +            key = 'META';
>
>     also I dont understand what you mean by "This also must be taken
>     into account in the control. This doesn't seem to be the case
>     currently." can you explain ?
>
> You should use this:
>
> |box: {
>      keyMask: Ext.isMac ? OpenLayers.Handler.MOD_META :
>          OpenLayers.Handler.MOD_CTRL
> }
> |
>
> in the |handlerOptions| passed to the |GetFeature| control.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/camptocamp/cgxp/pull/308/files#r2447265>.
>


-- 
Camptocamp SA
Oliver Christen
PSE A
CH-1015 Lausanne

www.camptocamp.com

+41 21 619 10 23 (direct)
+41 21 619 10 10 (centrale)
+41 21 619 10 00 (fax)
please add the missing type
Indeed. =)
Ahhhh! That's great. I've always wondered how to make migrations work when the model gets renamed. 

Cheers for that.
Again, not sure if TitleName is unique - if it is, maybe a dict would make more sense?
Is this vivibug still in here? or is it gone now?
I wanted to annotate it, but forgot about this github line note thingy...
The others use `if-else-if`, rather than `else if-else-if`. I've used the far more concise `if` that doesn't need the subsequent `else-if`. 
So I think it is consistent with the other lines, just a lot nicer :-)
I considered changing the entire equals method, but didn't want too much noise of other changes in the commit(s) associated with this fix.
I'm guessing you are aware that the comment and the actual prototype do not agree? I was just starting to read this all over to see how things we're going... Are things in a state where they are worth going through?
Use three small gettext calls instead of a trans block here. It's nigh-impossible to make this work without including some HTML, but we might as well minimize it.

Add an l10n comment above each of them as well:

```jinja
{# L10n: Small piece of text that appears to the side of the download button on the homepage. #}
```
Wow--that's so cool! I never thought of doing that.
Also, the ``> 0`` is unnecessary here. It won't hurt anything, but you could just do ``if ($(options.container).length) {``
I realize you're just cleaning this part of the code, but since we are not enforcing single validator for create AFAICT, I wonder if we should remove this check. Delete the validator if you want to.
Maybe message should say something related to the error I was having... nodejs not installed.
why break so late on bail? why not put this right after the case block?
Missing: return $this;
Ah, my bad. You should keep it as before, i.e. put

    strlen((string) $constraint->min)

instead of

    $constraint->min
why not `getContents()` ?
``@param callable $callback``
there is no ``doDespawn`` method
If second argument in fread will be not greater than zero then PHP throw error.
Why this file path is hardcoded here? Can't just return `array('en', 'root')`?
``string`` is lowercases in the phpdoc.

And your phpdoc does not follow the standard formatting: you should have a short description (``Sets the name of the default command.`` in your case), then an empty line before the long description, and another empty line before the param tags
Well, the variable name should follow the coding standards used in the project, and should be understandable by people who went to a different school than you :smile: 
So yeah, please rename it.
Maybe just:

```
do {
    $dependentClasses = self::extractDependentClasses($classes);
    $orderedClasses = array_merge($classes, $orderedClasses);

    $classes = $dependentClasses;
} while (!empty($dependentClasses));
Should be "cause" instead of "course"