This bit concerns me. Isn't there a possibility that BufferedReader will read more than the line length from the InputStream, and keep it in an internal buffer? And in that case won't that data be discarded when this function exits? The effect would be that if you typed a lot of lines quickly (or piped them to standard input) that this function would lose input data.

It'd be a lot safer to create the BufferedReader once, and re-use it for all the "readline" calls, but that would be more work and require some better understanding than I have about how the current command line works. (Or perhaps this has to be disabled when using the Rhino shell.)


Does this save to fedora values that are different to what grouper has?


Is this correct?  Elsewhere we subtract the offset to find the corrected accel.


you change name, but no occurence, does it mena it is not called anywhere?


Hm... I merged "nova: use signals with irqs in core" this morning.


I think the message is ok.

Considering the programmer as the user of the library, it's explicit enough and I do expect the programmer to dig into the logs.

Considering the situation in which the exception goes uncaught until reaching the user, I would not add more scaring information for something that is a programming error and the user will not see anyways.

Maybe something like adding "Programming error:" at beginning.


I just checked my frameworks/base & packages/apps/Settings and I don't have any use of either of the Interfaces listed in your logcat. ICMStatusBarManager or ICustomTileListener. I also just checked my website, where I have no issues posted. You really should be looking for the NPE in frameworks base, but I don't even have the interface in my sources that it is claiming caused the NPE.


2 not 800, right?


Not sure. It sounds like it's enough to support your original use case where the user code collects the namespaces upfront, but it otherwise seems to me like both serve different use cases. So I'm leaning more towards keeping them separate options.


LOLQ? AHAHAHAHAHAHAHAHAHA!


Ooops!


Bundler refuses to recognize this version format and blows up


Okay, moved in before the above-mentioned commit.


Title is  Kyojin no Doshin 1, not Kyojin no Doshin.


done


You're welcome!

Agreed. It would be best to upgrade jQuery and use only one version throughout the whole system.


We specify the string "ma/v1/assets" in a couple of places. Is it worth factoring this out into a constant somewhere? Possibly not in this case but if we find ourselves putting that same string elsewhere then it might be worth refactoring it.


Wrapping LoadDefinitions around a stopwatch without the parallel.foreach takes over 2000ms to complete on an i7 3770K
With this change, this function takes less than 900ms to complete.

Testing done on the dedicated server. Should also improve game loading speed


@yonatanmn: I think you are right.
@newtriks: I also think this should be corrected, maybe you could just add the Flux/Reflux stuff back in instead of all the shiny keywords? :)


Does it really have a "1" in the title?


but it's shown in a ADSP context. But as said, I'd need to check all this on a running build


True. :)


in RightsMetadata
`<access type="edit">
    <human/>
    <machine>
      <person>000000000</person>
      <group>northeastern:drs:repository:staff</group>
    </machine>
  </access>`
when i remove repository staff and add a different group -- loaders for example changes to
 `<access type="edit">
    <human/>
    <machine>
      <person>000000000</person>
      <group>northeastern:drs:repository:loaders:marcom</group></machine>
  </access>`
Is there anywhere else I should check?


的


@miguelgrinberg: I dont really see a requirement for the corekube repo to move over to rcbops. As an example, we did not need to move the gitlab project into rcbops, we just have a repository which contains a heat template for it.

I understand that the other solutions all have their own rcbops repo for the heat templates, but I am not sure that is "the way it should be". Thoughts?


@jlucerond DWMR (done with my review). Press on.


HI Gokul,
I think you are right.

1)
when I start sending messages from command line it works perfectly with this in server terminal its throwing below error.
Error:INFO Closing socket connection to /127.0.0.1. (kafka.network.Processor)
2)
I am using tail2kafka with your plug-in to check whether messages are displaying properly or not
when I start running same with tailing the log file its shows  below error

  Error :INFO Closing socket connection to /127.0.0.1. (kafka.network.Processor)  and
        out of memory:Java  heap space error.
I also increased system's Java Heap memory to 3GB
Thanks For a Valuable  Guidance.
Happy Week End.

Error Screen-shorts Are:

Kafka-server,producer & consumer
![Kafka-server,producer & consumer](https://cloud.githubusercontent.com/assets/12342549/7652495/cf850cd8-fb2a-11e4-8d5d-89ea54301e21.png)

[Kafka-server and tail2kafka

![Kafka-server and tail2kafka ](https://cloud.githubusercontent.com/assets/12342549/7652496/cf88d5fc-fb2a-11e4-8206-2083a56b04d3.png)


mamy konstruktor, ale nie przypisujemy argumentów nigdzie, dlatego tego nie zapisujemy, dlatego w toString wypisuja sie nulle


Thanks @asmecher, fixed. One line fix, was failing to get the grid control row. Grids that use the ordering stuff always reorder their items after inserting a new one. Failing to get the control grid row was leaving them all away from their main rows, when the spinner code tried to reach them it couldn't.


dodaj jeszcze jednego studenta, albo nie usuwaj az tylu, bo sie wywala


Did you try re-syncing and rebuilding? And when did you build? This commit:  https://github.com/CyanogenMod/android_frameworks_base/commit/e1dad90f5bc1cdf5b7dab7a6047135bcbc64257e ...without this here may give trouble. You can notice the java-symbol in first for config_externalCMServices and then the string-array attached to it in this change. So if you built without this it could give trouble. Besides that I don't know where to look, since cm's frameworks/base is still on 12.0 for default so you cannot search 12.1 Good luck. Usually it will work itself out the next day.


At your opionion, is it not too overcrowded with both fields displayed ?

If not, fine for me :+1:


The only issue with this, I think is that it will change how things currently work on Linux for csladspa. My reading of the code is that if there is no LADSPA_PATH set, then it will look for CSDs in the current directory. If this is thought to be a minor issue, I am happy to accept the patch.


Will remove it. Thanks.


This is wrong. The whole service would be hardcoded to the request active when the service was first requested, and not always the current one. However, we could argue if we might always want to use the master request?

/cc @Toflar @dtomasi @discordier


Also, I'm building now, so if I find the issue I will let you know, then I would have more info... but our (VanirAOSP) sources usually get tested & fixed right away.


You could just make `case_ids_on_phone` into a `SetProperty(unicode)`


maybe need to add a `None` key here to handle existing sync logs that don't have a `format` attribute.


(actually figured that out in a later commit)


Okay. Currently the rbf is generated into the quartus directory and then copied into the elf directory. I can change it to directly generate inside the elf directory.


Never mind


yup, tx ;)


Exact, I had an old version on my jenkins server.

Thx for the help.


I think it is still clean and not overcrowded ;)


scriptVersion should be 3.1.0.0 btw


Ha ha


have you joined the octgn-dev newsletter? https://groups.google.com/forum/#!forum/octgn-dev


If you use method call injection, your code must not depend on the service actually being there. If you must have the service, use constructor injection. Otherwise you should compare to `null` here before calling the method.


Then go for it !


Thank you for tip but my repo was already up to date..
Now I'm making a clean build and  I hope it will work..if it won't then I don't really know what to do
I also have facedetector bug that makes whatsapp crashing when sending an image..weird bugs


// TODO: Clean this up so we only get the list of servers which we should deploy to


Can you give me your email? We can talk on hangouts if you want


@jlucerond DWMR. Press on.


For what it's worth, I named the tests as I did because they were targeting specifically the `default` methods on the respective interfaces - I assumed tests for `DefaulltAttributeContainer` and `DefaultEntityContainer` would go in their own classes.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.18-DEV18](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237240&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


Indentation is all wrong.


Just checked quartus documentation, you can also directly generate the rbf with Quartus, simply add this line to the qsf

    set_global_assignment -name GENERATE_RBF_FILE ON

Note that the rbf file has the same file name like the sof !


Учи английский, братюня... :-)


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.18-DEV18](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237240&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Compilation error: src\redis4net\redis4net.csproj Build time: 0:0:54


Avoid '!' in conditionals. There is a 'blank?' method that is reverse to 'nil?' or 'unless' that is reverse to 'if'


Rather than appending everything to one big string and returning that, why not just yield return each line as IEnumerable<string>. It will reduce the responsibilities of this method/class and give more flexibility to the caller.


In Rails 4 you should always use strong_params instead of unfiltered params


Does this fix the bug described in https://github.com/playframework/playframework/issues/3120?


I know that this thread is old but isn't possibile to solve the situation by conditioning the execution of the line

 $this->name = $user->username;

by appending  if($user) just before that line of code?

May I open an issue and a pull request?


unless user

or

'if user' and swap two bloacks


typo ? `s/g_acsii_isspace/g_ascii_isspace` ?


май има два катинара в момента? кой от двата :)


see L683


Why 'to_s' ? What else can it be? (nil will be handled fine)


I would recommend to set bs=1M when calling dd (is much faster than default!!!)...

    $ dd if=${SOCEDS_DEST_ROOT}/embeddedsw/socfpga/prebuilt_image/sd_card_linux_boot_image.img of=/dev/<sd_card_drive> bs=1M


But this is not the case for the `ContainerAware` stuff, either. If something is `ContainerAware` in Symfony, they are using `$this->container` without "if not null", aren't they?


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.19-DEV19](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237257&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


Който могат да отворят, за синия комбинацията я знаем само ние и още не сме почнали да го ползваме :)


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.19-DEV19](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237257&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 0:0:52


I was just checking, want me to add it to the build script?


@jlucerond DWMR. 95% of my students are unable to get this checkpoint done without help. Well done. Press on


This whole condition can be substituted with

@credits = Credit.where(email: user.try(:email) || current_user.email )


Commented lines to upload original table.


Sure but I'm at work atm
On 15 May 2015 13:41, "Modmuss50" <notifications@github.com> wrote:

> I was just checking, want me to add it to the build script?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TechReborn/TechReborn/commit/55a3acff14c32fc359b11e9ded2c079b0b4164ea#commitcomment-11213416>
> .
>


Ok. I'll add the `Programming error: ` prefix as Ancor suggested below.


I still wonder what happens if the filter submitted has a wildcard in the string (eg thor::myfiles::*)... does it pass the above condition? If so, do you then treat that as a "fetch-all" tables condition?


Seems like there should be a configuration option for this.


I don't prefer the introducing dualprocshm-pcp.c, as the dualprocshm library is not having a host and pcp concept.

Though looking forward we may like to move the target specific processor files but I don't think this should be taken for Altera SoC.


Deployed, v9


Use route_helpers


I saw it and it's better now, but I doubt that was the reason of the problem. If you debug during the torrent removal you can get this behavior due to mutability. Note that having an invalid `ti` could (or could not) create more problems in this particular case.


@aaa707 I would like to properly attribute your contribution too. Would you please let me know what your name is? Or if you don't want to do that, then tell me so.


before the old sync logs would blindly add the case to the dependent list and rely on pruning at restore time to fix it. now it removes it immediately if it's not relevant.


No, we should probably change it. Although the service will only use the very first request, because the framework can only be initialized once.

What would you prefer?


Yes, I knew about it, never tried since we requiere more flexibility in the case of mac, but it's on my list when the time comes.


I think you intended this to be a Pull Request, but you only forked it and modified the file. You still need to open the Pull Request itself.


@rabio I would like to properly attribute your contribution too. Would you please let me know what your name is? Or if you don't want to do that, then tell me so.


You might want to remove that comment


@laelia44
cette fonction de récupération de IP ne retourne pas la bonne valeur
avec WAMP la valeur retournée est   ::1


cant use iframes


The node Id uses FPGA and HPS dip switches available on sw1 on dev board. If the return value is 0, then app takes the default node Id(240 in this case).

Do you prefer it returns 0 in case the FPGA switch is not there? The HPS switches can still be used to set smaller node Ids(CN).


Love


OK.


That looks good.


I was especting same behavior on both exception, so I believe a \ is missing


we can use iframes... they just need to be responsive


You should give the caller more options to call this with an IEnumerable instead of ICollection. It seems like the only thing you are doing with ICollection is Count. Or perhaps do an params[] instead.


@laruence
Why is not this change reflected in the 5.6.9 branch? The 5.6.9 branch differs from the 5.6 branch in this respect:
https://github.com/php/php-src/blob/PHP-5.6/ext/opcache/ZendAccelerator.h#L30
https://github.com/php/php-src/blob/PHP-5.6.9/ext/opcache/ZendAccelerator.h#L30
This raises questions if other changes in the 5.6 branch did not make it to 5.6.9.


I will remove it. The LCD module in reference uses delays of tens/hundreds of microseconds between commands, which is not good. And LCD works without that.


Is there any place to learn about why things get changed, and the results of experimenting with different rendering approaches? A mailing list maybe? I would find it very interesting.


wait, another release note below the one that links to this says "All blueprints have been updated to use shorthand ES6 syntax for importing and exporting"

So what's up with this?


Yes, I will change it. Thanks.


I see, thought this source file is board specific. But then "-pcp" is not the right suffix either...

Anyway the selection of the right dualprocshm-*.c should be done in tools/altera-nios2/stack.sh depending on the board.settings


Yes Okay.


btw, this commit is here just for reason to see the issue with scoreFunction as anonymous class


since you're traversing the entire string anyway... I would replace the (strchr(_tableFilter, '*') || strchr(_tableFilter, '%') condition above with this loop, thereby reducing the number of times you traverse the string (strchr), and  make the conversion on the fly


@jlucerond DWMR. Well done again. Press on.


Okay.


ok, fixed


Hmm, this commit made the loop type-unstable (`I` either has `typeof(X)` or its original type), which really kills performance.


review completed.


Is this a typo and you meant `p != null`?


No. This will create the tc_i_mem code sections in the linker.


@samanpwbb when you made the changes you set

sup { vertical-align: text-top; }
sub { vertical-align: text-bottom; }

this doesn't really give the text effects that would normally expected (actualy sub works more like a superscript with the above). I didn't find any actual use of either markup. So except if you have  stylistic/artistic reasons for the current values, would it be ok to change them?

Simon


I couldn't find the bug's description in the process 1.2.3 library change log (http://hackage.haskell.org/package/process-1.2.3.0/changelog). Is the bug described elsewhere?


Why assume the caller wants one string with all the output? Return the output directly while just hiding the Powershell type.


I was using `debounce` so that an error doesn't flash not the screen after typing the '-' in a value like `my-project`. Maybe set it to a smaller value?


I don't really know...


I can fix later, if no one beats me to it, but I have other obligations first.


For disjunction `\/` `rightMap` is just `map`, or am I missing something?


Excellent.

Thanks for your time, and fingers crossed setasign accepts your pull soon.


Let's hold off on this one for a while. I've got an idea for proper pretty group titles using yml, that would give Sarah full control of the cosmetic value for grouper groups


Sorry, I was confused :) You're right of course


Is there any risk of exceptions in the above code causing this to not get run? Does this need to be in a finally?


The binaries got huge and this introduced delays in the build. So I changed it to be configurable.
I will remove it.


Do we need the `|| []`? drop seems to return an empty array when suitable.


Yes it passes the above conditional test and drops into the bracketed code. And no it is not a fatch-all, you will see below before I call wssql I set "req->setTableFilter(sbTableFilter.str());"


It has broken some backward compatibility actually.


Is there any reason for the delay in the reference design?


:point_up_2: jajaja no pasa :one:


still failed to use with Eclipse as hardcode and with maven as well


Maybe fixing this is overkill, but this reads a bit weird – `attributes.plain` apparently means "names of plain attributes" and `plain_attributes_values` is for the values. A small change like `attribute_names.plain` is probably enough to make it less confusing.


I don't know if they do. But their manual (http://symfony.com/doc/current/components/dependency_injection/types.html#setter-injection) says:

> The disadvantages of setter injection are:
> - You cannot be sure the setter will be called and so you need to add checks that any required dependencies are injected.


Hm that doesn't look right I agree. Please change these rules :)


this passed eslint? But the previous one also got problem


Ok sounds good


Going to fix it.


> Although the service will only use the very first request, because the framework can only be initialized once.

Creating the service does not necessarily correlate with initializing the framework. There are listeners that only use the service to check if the framework has been initialized, but do not initialize it themselves.

> What would you prefer?

What do you mean?


Also this string can be reused +msgctxt  #16400

https://github.com/xbmc/xbmc/blob/master/addons/resource.language.en_gb/resources/strings.po#L7584-L7586


Does it matter if there is no function linked to the tc_i_mem?

Anyway, I would prefer to check if CFG_DRV_TCI_MEM_NAME is set or empty. For C5 SoC design set the variable in boards.settings empty to disable the TCI Mem usage.


Will do.


i think this will not work. You will need to use some like $dao->whereAdd('financial_type_id in (implode(',', array_keys($financialTypes))')


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


not really. But for now there is nothing else thats being put there. And I am not so sure about the sdk either as the preloader generates another software folder.
naming it ds5 would not be so clear about the purpose of the directory-files.
Is sdk-resources too descriptive?


yes this new function looks very spiffy


In order to create the binary the user has to execute make elf-to-bin or is it included in make all?


@deads2k Is this how the custom messages are returned?


Empty ?


this PR makes it so: https://github.com/liggitt/origin/commit/d0048716c1b14d211b6dc942f2f11ade3a55f747#diff-f758f86f072daa59bfdef62232df08ea


Okay. then I would have to remove the rbf generation step from driver and only rename and copy it.

Which way do you prefer?


It's very unconventional to make all your users get at your enums by forcing them to add the Enums prefix. Fine for internal facing code, but I wouldn't do this for anything public.


That is actually shorter string also :) so should fit nicely


Actually... renaming it to `attribute_names` makes line 31 a bit weird: `hash_values_with_defaults = attribute_names.default_values.merge(...)`.
What do you prefer?


I didn't know we have this file.


Added the library thus far to dexters git-hub.


generate rbf in Quartus, copy and rename it in driver.

And you? :)


OK.


Yes, I noticed that. Does it work for you? Sometimes the sdcards did not boot when using 1M block size. I have seen the FAE use 64K block size also.


Cast the pointer instead of the array, saves a runtime call that uses TypeInfo to translate the length.


Oops


My original idea was to push the code to set the attributes into an instance inside the `Attributes` class, but that was more complicated than I expected...


This string can be reused

https://github.com/xbmc/xbmc/blob/master/addons/resource.language.en_gb/resources/strings.po#L7584-L7586

msgctxt "#16400"
msgid "Post-processing"
msgstr ""


I will change it to 1M.
Are you okay with the make file there? It does not actually do anything as we thought creating disks from there might be harmful for the user's hard drive in case a wrong drive name is provided.


It seems to me that cookies is a concept related to the HTTP request, not the concept of having something crawlable. We should find another place to put this method.


Cri Cri Cri


I think this looks reasonable if the code here has to make its decision based on examining the log.

Ideally we would have something cleaner though.
How about changing install.sh to give different exit-codes for different errors? Then xapi can make decisions based on the exit-code.
But that would only work for future patches, not for existing ones that have been released.


Why the change and why the inconsistency between provided and unprovided messages?


The history of 5.6.9 shows why: there was only one cosmetical change, compared to 5.6.9RC1. Yet, it is strange to see there were a lot of substantial changes between the release of 5.5.25 and 5.5.25RC1 and almost none between 5.6.9 and 5.6.9RC1.


Same here `(cast(T*)p)[0 .. n];`.


Again, don't remove the blanks. Being picky is hard for me, but you should include blanks in:

```patch
diff --git a/avocado/multiplexer.py b/avocado/multiplexer.py
index 3c80971..6bcfbba 100644
--- a/avocado/multiplexer.py
+++ b/avocado/multiplexer.py
@@ -31,11 +31,13 @@ MULTIPLEX_CAPABLE = tree.MULTIPLEX_CAPABLE

 class MuxTree(object):
+
     """
     Object representing part of the tree from the root to leaves or another
     multiplex domain. Recursively it creates multiplexed variants of the full
     tree.
     """
+
     def __init__(self, root):
         """
         :param root: Root of this tree slice
diff --git a/examples/tests/raise.py b/examples/tests/raise.py
index d3404d0..a90d1dc 100755
--- a/examples/tests/raise.py
+++ b/examples/tests/raise.py
@@ -20,7 +20,7 @@ class Raise(test.Test):
         Build 'raise'.
         """
         c_file = self.get_data_path(self.params.get('source',
-                                    default='raise.c'))
+                                                    default='raise.c'))
         c_file_name = os.path.basename(c_file)
         dest_c_file = os.path.join(self.srcdir, c_file_name)
         shutil.copy(c_file, dest_c_file)
```


Ah. `attributes.plain_names` then? Or positional, whatever. Not perfect but probably good enough.


I think sdk is okay, since we have also an sdk dir in xilinx-z702/mn-dual-shmmem-gpio


There is no explanation. So common notion would be that the commands take some time to take effect. But these work fine without the delay.


Looks like the test module will log (and run) even when disabled


You should at least chain the exceptions to the one in flight.


If you start adding a variety of enum types this switch could get ugly. Should that happen consider using a dictionary where the keys are the enum type and the values are a Func<CredentialContainer, AWSCredentials>.


Looks good, reviewed


if it's no used, don't include it


Pelo menos saiu uma reticências


ok ho appena visualizzato


I love the mock.  It would be more valuable with real launch data.


trailing spaces at the end of line


This is great. Do you want to create a pull request and I'll get that into the source?


Honestly I didn't notice! Do you think I should rename them back? I guess we face the issue that you can only test them on an impl and that impl could override them! What do you think we should do?


The one issue you are likely to have is that Security::save() now takes a SecurityScheme as a parameter, instead of a PHP array.


Should the documentation does say `...as 'multiplex = True'` here? Since the logic has been inverted from `!join` to `!mux`.


- I tested it several times with 1M, looks good.
- I'm also okay if this makefile harms the user's hard driver :) instead of a makefile a shell script would make more sense. But that's optimization for the next release ....


How about realloc for ranges without length?


if there no key "title" in the POST you'll get error. Read carefully about dict's methods. Especially get


We handle the text clause as a _special_ simple clause.  After all of the simple clauses are handled and put on to the node tree, the text clause is processed and put onto the tree.  So yes, a text clause should always be after the simple clause(s).


If you define global (file level) variable - make it uppercase


Ok, right...I forgot we had this special codepath for the check updates.  It is ugly...and your code to call into the DBus side was cleaner.  But we can *also* fix this by having it  use the shared library API right?

I can help with draining the duplicated diff code after we land the daemon.


Since this program always runs in the console, it seems like it might be easiest to always run things through the color code-path to avoid the extra branches but simply clear the values in COLORS when color is not enabled.

The better way to do this might be to refactor to use something like Fabric's colors module – https://github.com/fabric/fabric/blob/master/fabric/colors.py – where the functions are defined in main() in an if block so they either colorize or do nothing and the subsequent callers don't need to check.


Ok! So basically if the lcdl_printText() is not called successively it works.

I would leave the commented code and add a descriptive command!


Module loader should be able to check the status of a module [enabled/disabled].


This is recursive... perhaps this approach is better?
```js
  function exec (cmds, userOptions) {
    cmds = (cmds instanceof Array) ? cmds : [cmds];

    cmds.forEach(function (cmd) {
      try {
        var options = Object.create(defaultOptions);
        for (var key in userOptions) options[key] = userOptions[key];
        return child_process.execSync(fill(cmd), options).trim();
      } catch (err) {
        return err;
      }
    });
  }
```


Either delete comments or space between comment and def


@vais didn't have the time yet,
the strategy would be to remove it from stdlib but keep it to run mri tests,
let me open an issue for that so we can keep track of it and have the conversation there.


I missed this line, so Cocoapods was being set up multiple times throughout the App lifetime (each Action). This marks the App was done being set up.


Yes, creation of the tc_i_mem code section in the linker generates the binary with initialized data for the address location.
Okay. I wll do this modification.


This does bail-out on the first error... or should we 'gather' all errors are report afterwards ?


I see. If the FPGA switches are not connected and I want to run an MN, I have to set the HPS switches to 0, right? If so I'm okay with that to enable a "smaller node ID" for CNs!


no it is not in make all, user has to call elf-to-bin.


spaces around '='  ---> syntax_id = m...


I agree with all of that, but I think we should have all of that logic in this function rather than in the callers.

E.g. tree_to_fs_path(base_path, tree_path, encoding) which then takes care of encoding the tree_path if necessary and joining the base and the tree path.

With regards to os.seperator, what prevents us from always doing the replacement on the filesystempath type?


API might be nicer as myAwsObject.TagWithName(credentials)


spaces around =


okay, then you can remove the if and add that option for all demos.


spaces after ',' and ':'


In fact, assignment in structure declarations needs to work for this I believe?


Same. I would do the changes.


@elia, cool got it. In the meantime, I will use the empty stub files to work around the problem. Thanks, @elia!


// TODO: Clean this up so we only get the list of servers which we should deploy to


I haven't looked at this in over a year so take this work a grain of two of salt, but yes that looks like a typo.

Thanks for picking up development on this!

Clayton

Pardon brevity and typos; sent from my mobile.

> On May 15, 2015, at 8:55 AM, Andres Kalle <notifications@github.com> wrote:
>
> Is this a typo and you meant p != null?
>
> —
> Reply to this email directly or view it on GitHub.
>


Okay.


{% load ... %} always at the start of doc


This vector was created inside the function. When the function exits, the object goes out of scope and will cease to exist.
Also, you shouldn't pass or return an object, only references or pointers. Whether the pass or return succeeds depends on the existence and actions of a copy constructor. Also, it's slow.


Consider creating two separate batches of tests, one unit, the other integration. The tests I see here are all unit, but it would be nice to also have an optionally run set of integration tests that directly hit Amazon to make sure everything works. If they change their services one day, you could easily identify where the pain points are and surgically fix.


Oops


isExistent. I meant naming boolean var. But functions with "is..." or exists ) Lets discuss offline ))


Okay then I would update the help and leave it as it is.


@vais I'd rather skip minitest entirely in the `dist` task, seems cleaner that adding an empty optparse to the stdlib


Yes.


There isn't, unfortunately. In a perfect world, we'd have time to document/communicate these things better. But in the real world, half an hour or an hour spent explaining all of this is time just taken away from Andrés or I actually fixing things. It's frustrating for us too.

The short version here is that:
* JOGL had some major bugs and development seemed to have stopped (last summer/fall?)
* @codeanticode had been trying out LWJGL2 to see how it fared (last fall)
* The LWJGL project has moved all their development effort to LWJGL3 (since then)
* Andrés spent the week rewriting OpenGL to use LWJGL3
* LWJGL3 is simply too unstable for us to use, would require major reworking of PApplet to remove *all* uses of AWT, and they seem to be still struggling with many fundamental issues (this week)
* Andrés went back to JOGL (last 48 hours) to find that many bugs had been fixed and development was continuing.
* We dropped LWJGL since JOGL is performing much better, and we're 99% sure that's the final decision for 3.0 (yesterday).


Have you considered leaving communication-related functions in IRAM rather than FLASH? The reason I suggest this is because access to the FLASH (and thus all the functions it holds) is disabled when reads/writes are taking place.


Probably the best way to do it would be to make mock implementations that don't overwrite any of the default methods, and then do the tests on those implementations.

The easiest thing to do would probably just be to copy the `Default` implementations into the test package as `TestEntityContainer` and `TestAttributeContainer`, and use those as mock implementations for testing purposes (since they don't overwrite the default methods currently).


HUEHEUE


Yes. Okay I would add a comment, thats what you meant right!!


The formatting changes for Scaladoc are undesired. Please configure your auto-format correctly.


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


sp. receive


I have the fix in #10525, too. I should have just pushed it to master right away.


use tag {% url 'name' %} instead of ugly hardcode like action="/pb/i18n/setlang/"


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.20-DEV20](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237392&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


Yes


Okay. Yeah.. didn't think from that side.. makes sense.


That's pleasingly little change... and a much better job than I made of this.


Please keep it as it was, the only debate is should it be whether to have **audio** or **Audio**. Personally I'd go with **audio** but lets see what da-anda has to say.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.20-DEV20](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237392&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Compilation error: src\redis4net\redis4net.csproj Build time: 0:0:53


Thank you! :bouquet:


Cleanup should go in TestCleanup but if this is the only test that creates a file then this is good too.


ty ;) Sorry I forgot to commit/push yesterday...

Disappointingly not as fast though, which still puzzles me :(


I'm fine with leaving it as it is for now. Soon we should have tests for the ```Default``` impls and if they come to override the ```default``` methods then I wouldn't accept the pull without a matching test. And we can make our decision there (we could always make a very simple and static mock one - rather than copying the more complicated concurrent safe ones). @wlmiller

I reverted my change - I think you were correct


@kurund, there you go. I'll be sure to make more mistakes like this so I can boost my commit count.


haven't notice that at first ;(
jQuery(elem).change(function(... ) would work here


Do you also need to copy the fpga.rbf file to the sdcard???


This is because I ran into an issue where the script successfully made a request, but it tried to tag it so fast that EC2 threw an error during the `add_tag()` saying that the request didn't exist.  The script then decided that the request didn't work and made another request, going over the requested utility level.

@klahnakoski:  I'm almost certain there's a better way to do this.  We should probably just tag everything at the end.  I just couldn't figure out how to properly get at the spot requests later on in the process after all requests were made.  `self.net_new_spot_requests` seems to have `dictwrap()`ed versions of the request objects, and I wasn't really sure what that would do to them.  Tips?

Other than that, I'm in great shape.  My little botnet army is growing.  Everything's working swimmingly :D


why you need "_"?


We will try and keep up with Netty development in order to get bug fixes and performance improvements. Nothing specific for 4.0.28 though


I did not, aren't they dropped by the second commit? They don't show in the diff of the full PR.


We should probably throw an exception.


Make header and footer as a separate file, and include them in base


How does your preferred solution look like? What do you want me to change?


muito bom


method contains should not be defined in Utils, but in service that is responsible for "backend". Define it in users-back.js file and Users factory.


@SzymonPobiega why add this to one sample? should this be better in documentation and then referenced via the various samples?


At a guess, try decreasing the timeout you get out of the min ever so slightly (to 90% maybe)?


problem with tabulation


Why not use constructor injection? We've discussed that already, the Command should be a service


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.21-DEV21](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237399&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


Btw - I asked about it some time ago, but got no answer - there are two options for ScalaDoc in options (text starting from first line or second line) and we have mixed style in codebase, sometimes even mixing those 2 styles in one file - which one should be used?


_ doesn't gives me any understanding of what type of functions it provides


Inspecting the documentation it's not a redundant parameter.
````
-t duration (input/output)
When used as an input option (before -i), limit the duration of data read from the input file.

When used as an output option (before an output filename), stop writing the output after its duration reaches duration.

duration may be a number in seconds, or in hh:mm:ss[.xxx] form.
````

In this case where you had no thumbnail created. -t would be an output parameter that caused the command to terminate before it completed.


{% block pastebin %} does not need one step tabulation at all


Tim, wilt ge de fix voor onzen build aub binnentrekken?  ;-)


Hmmm... It's not; I wanted to delete before the test (instead of after) to leave the generated file around for inspection, in case the test failed, but that's kind of silly seeing as someone could just set a breakpoint to give themselves a chance to see the file.

It would be great if this entire class emitted files to a temporary folder, but I wanted to keep my changes minimal for these fixes.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.21-DEV21](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237399&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 0:0:51


Thank you! That's already very useful info. I had plans to experiment with LWJGL and LWJGL3 myself, but it seems like it's better to wait at this point. I'll try to help with Processing this summer :)


never use inline JS and CSS


Again either **video** or **Video** and I'd go with **Video**


$url needs to be taken before $feed->load() to detect HTTP 301. This change broke the HTTP 301 permanent redirection. I will send a patch.


Sounds good, I was musing the same thing to myself about the whole structure of how the unit tests clean up. Since it doesn't have any formal structure present this small change looks correct.


space after ","


There's one thing I haven't accounted for.. When the rerun file has a location filter and _that location filter actually includes the external example group_. Unfortunately this simplistic addition can't really account for that case. Fortunately I'll have time tomorrow to investigate all the metadata that's available and check the internal workings of `MetadataFilter` to see what's appropriate.


The command **is** a service already. But I don't want to copy the original constructor, call `parent::__constructor()` and track all the changes Symfony might apply to their constructor. Therefore setter injection is the better solution.


Looks good!


Good call. :)


I know it's equivalent but do you want to use the `have_text` matcher here and below?


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.22-DEV22](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237417&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


fuck yeah
took only 1,5 hours


Well my preferred solution would be the same I created ;-)
Storing the request stack and retrieving the request on demand. Except if we agree that we should always use the master request…


OK to change the 16400 to 15060 to have "#empty strings from id 16336 to 17499"?


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.22-DEV22](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237417&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 0:0:53


TeamCity .NET :: NET - Redis4Net :: Release build [Build 0.5.5](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237419&buildTypeId=Net_NetRedis4Net_ReleaseBuild) is now running


I think we should (always use the master request).


TeamCity .NET :: NET - Redis4Net :: Release build [Build 0.5.5](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237419&buildTypeId=Net_NetRedis4Net_ReleaseBuild) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 0:0:52


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.23-DEV23](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237421&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


Great; we'd love the help!

I think LWJGL3 will be a nice game-centric platform (full screen, affordances for game input/behavior) in the coming months, but the direction they're having to go with 3 means they're moving further away from what we need in Processing with something *slightly* more general.


Delete only though POST methods


Yes, that I would update.


Sorry, but this change is not relevant anymore, I changed this whole part altogether so that there are no bullet points at all. But thanks you anyway!


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.23-DEV23](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237421&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Exit code 1 Build time: 0:0:52


Above Audio entry - you go with **audio** here you go with **Video** should be *video* also to match above then.

I just want to get rid of quotations here and only way is use the correct component name or lowercase it like you noted above.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.24-DEV24](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237439&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


This won't install Elasticsearch. Just the Python connector library for it.


gpg needs to be versioned if we want to a key per release.
ie: RPM-GPG-KEY-CentOS-SIG-Cloud7-Kilo


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.24-DEV24](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237439&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Exit code 1 Build time: 0:0:53


To break the APCAppDelegate dependency, could this become part of a protocol, e.g. `APCPermissionsManagerProvider`, to be implemented by the app delegate? I have started experimenting with this for the [onboarding manager](https://github.com/chb/AppCore/blob/91c010f2a708a34044b84006cb984ad18ee94f29/APCAppCore/APCAppCore/UI/Model/APCOnboardingManager.h): it would still require the app delegate to provide the permissions manager, but it would be asking for an app delegate conforming to the protocol, not the class `APCAppDelegate` specifically.

```objective-c
APCOnboardingManager *manager = [(id<APCOnboardingManagerProvider>)[UIApplication sharedApplication].delegate onboardingManager];
```

Another idea would be to make the permissions manager a singleton. My reasoning for doing this is that I'm not using AppCore as a whole, but only parts of it. The protocol approach allows me to use my own app delegate.


Thanks Keith!!


Just removed buffers script, its not needed anymore (just hung around by accident)


Made some changes like renaming the `Attributes` class to `AttributesDeclaration` and then its methods. Do you think it's better? Or should I still add a `_names`?


> In this case where you had no thumbnail created. -t would be an output parameter that caused the command to terminate before it completed.

Lines 80-81 still use `-t` when converting video (as part of `Convert()`) AND a `MaxVideoDuration` is specified.

When emitting a single frame, as part of `GetThumbnail()`, I have at least one MPEG-TS video file where ffmpeg doesn't create any JPEG files when both `-t` and `-vframes` are specified.


LGTM.


and I thought it is included in the BOOT.BIN :) spend the last hour to find the issue :)))


And whatafuck is it? Use django Forms! http://djbook.ru/rel1.7/topics/forms/


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.25-DEV25](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237442&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


if that string range is unused then by all means it should be marked as such.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.25-DEV25](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237442&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 0:0:54


We can make a voting. I prefer one char indent style with first line empty.


It's hell. Use from's which will provide you final value


Yep, this was fixed a while back. https://github.com/JMAConsulting/civicrm-core/blob/Civi-ACL-FT/CRM/Contribute/BAO/Premium.php#L130


Andy you broke inherited attributes, because of it


why not use [`SecureRandom.base64`](http://ruby-doc.org/stdlib-1.9.3/libdoc/securerandom/rdoc/SecureRandom.html#method-c-base64)?


Me too. Or, if doc comment is really short, to use one liner:

`/** Return associated value. */`


I like the compact one. We can compromise on using compact for one-liners, and java-style (first line empty) for multi-line.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.26-DEV26](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237455&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.26-DEV26](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237455&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1; inspections (.NET) execution failure (new)) Build time: 0:0:36


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.27-DEV27](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237456&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


I really don't have any use for it anywhere. However, I don't actually know what the expectation is with old versions of packages. Should the be left in or removed or what?


Das funktioniert ganz und gar nicht! Fehlerhafter commit?


LGTM.


Oh! Sorry for that. Partly naming it BOOT.bin is also responsible.


Irrational using of variable. Use "choices" for django models


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.27-DEV27](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237456&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **SUCCESS**
Summary: Running Build time: 0:0:54


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.28-DEV28](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237459&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


I think there are a few more reasons to use Docker / or any other type of container technologies:

1. portability.  Packaging up the runtime requirements of the app, with the app
2. Application centric
3. automated builds
4. versioning
5. component reuse.

I am basically paraphrasing what Solomon wrote here:

http://stackoverflow.com/questions/17989306/what-does-docker-add-to-just-plain-lxc

I think that's a great summary.


Less is more. https://doma.io/content

I'm not the average: https://doma.io/average

Thanks, hemingwayapp teached me a lot.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.28-DEV28](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237459&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **SUCCESS**
Summary: Running Build time: 0:0:55


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.29-DEV29](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237461&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


nice catch!


Okay.


*  The Changelogs has been updated with details for 0.9.3 and 0.9.4.
*  We are updating our build scripts to automatically resolve this for future releases.

> Please see the [ChangeLog](https://github.com/angular/bower-material/blob/master/CHANGELOG.md) for details.


Isnt this assert redundant?


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.29-DEV29](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237461&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **SUCCESS**
Summary: Running Build time: 0:0:54


TeamCity .NET :: NET - Redis4Net :: Release build [Build 0.5.6](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237463&buildTypeId=Net_NetRedis4Net_ReleaseBuild) is now running


What about shared?


Have you tried input?


TeamCity .NET :: NET - Redis4Net :: Release build [Build 0.5.6](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=237463&buildTypeId=Net_NetRedis4Net_ReleaseBuild) outcome was **SUCCESS**
Summary: Running Build time: 0:0:46


This selection is specific to the dualprocshm builds of nios.
Do you prefer this to be moved to stack.sh?


@romracer
Some testing of network mode tile and Smart Radio is needed on Moto dual SIM running 5.1.
Please. Only when you have time. Thanks.


> bit flags have been removed and replaced with method calls on the binding object
実際の内容見てもちょっと分からなかったんですが、「ビットフラグは削除され、バインドされたオブジェクトのメソッド呼び出しに置き換えられました。」的な感じですかねえ。


вв


Yes, because this file is a "settings" file, so it should just set variables if possible.


alternatively, `w = if dim.w > 400 then 400 else dim.w`, but I don't know if that's any more clear.


Looks great!

I don't really get the different .css() indentation ^^


The bug is not in "process", but in the Agda test code. The commit message doesn't make that very clear ....

The offending code is here:
https://github.com/agda/agda/blob/master/test/Exec/Tests.hs#L154-L164

For travis, this change doesn't matter time-wise right now, it only saves maybe 20secs.


OK, but why?  Isn't the purpose of this PR to enable them?

(if you need a working + development branch, we can use feature branches)


Keep


@liufengyun


Yaaaay! ^_^


Лучше писать на английском или хотя бы латиницей


I modified this regular expression and wrote a new version of linkify(`createLinks`), you can check it out here:

http://jsfiddle.net/8huL9qbz/

If it is OK, I will make a PR.


This commit was made to branch 'smallbugs_jelmer' and accidentally pushed here. Removing asap


I know without a mark it becomes really bad to find empty spaces.
Thought that there was only one string in a big range and to swap not over DSP to there, also to prevent me from bigger id changes on dsp (on begin I don't want single empty id's)


環境変数 `NODE_REPL_HISTORY_FILE` にユーザーのアクセス可能なファイルが指定されている場合


if you look into the rest of the code, all sameValueAs assertion are tested bidirectionally since this is a standard from the first version of the lib. Anyway, I agree this is not so useful and should be removed. Willing to open a PR?


10 * 1024 * 1024?


Please remove this -- triggers a nasty explosion when viewing databases with 50+ forms!!


```
while (1)
	;
```


Forgot to merge previous code before committing new chunk.  Can you review just the last commit, please?


nice catch :+1:


This looks a bit ugly, but making the `tests` folder a module is also ugly. Maybe @jirikuncar knows a better solution?


Yay :D


Nay D:
That's quite rude


This is the main algorithm. I change the update rule to be diminishing and the subspace distance to be Frobenius norm.


Hi @JamesXNelson
I could not find latest version of xapi on maven~.Has it been uploaded?


failed dhondt


Bored mind, idle hands ...

would you consider beautifying the torrentFreakTop10 template?


Getting all the SQL out of the controller is _awesome_, it looks so much better already!


I will see if I can come up with a better fix, but it may take some time.


Thanks! I'm just trying to make Halcyon work on my Trisquel system, which
doesn't get identified as one of the platform options. Sorry for the ugly
ad hoc solution. My knowledge of bash is almost non existent and I didn't
realize you'd get notified...

Thanks,
Diego Saa
On May 15, 2015 6:33 AM, "Miëtek Bak" <notifications@github.com> wrote:

> That’s another typo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/diegueins680/halcyon/commit/1c71916528db013a2664200c903e6f84d5f818e3#commitcomment-11212421>
> .
>


:clap:


Yep, right!


It's recommended that you terminate expressions with semi-colons. This will prevent Javascript from executing your code wrongly.


For good style, surround your operators with spaces.


it always makes sense to add some markdown to `special` words


This call is actually calling the original prototype method which should be doing the sanitizing. So I think you're good here.


The only problem is that both Scalariform and Scalastyle seem not to be able to check for Scaladoc correctly. We can only have a look during review.


Ha nice. I imagine you just stumbled on this, rather than that at the current scale of cases on a phone, the O(n) factor really has any effect yet.


Looks safe to me (see below comment). Also, it's the recommended solution according to their docs. I think we can remove these comments.


add a closing "For all more information on usage run `./make-distribution.sh --help`"


@itamarnet Uau, que bela faxinada! :+1:


Nice! `period` is also a nicer arg name, IMO


You should maybe apply some colour overlay to make it visually differ a tiny little bit from real end stone.


I was going by the Jira request (that was the syntax that was requested).  How about `setOverrideCheckColumnValue`?


Maybe we can just `return` the value returned by `function`, even if it's `None`? We're relying on its side effects anyway, but its return value could be useful.


is `postgresapp` some special option in senza?


...not really. XD


This is true. Not until it gets to the next step. Ugh.


Nein Stackenblochen!


yeah, thanks. i will remove them.


update


dfgdfgsdfsgdfgdf


Wow! :+1:


Why this? (kind of a forward declaration??)

But you are including the header anyhow in the cpp, so no harm in putting curl/curl.h here?


You can replace the `console.log` statement with a `return` statement.
Also, you should use more descriptive names for your variables.


You may already be aware of this, but at the moment you are still binding to the $scope in the app.js, because you assigned $scope to the products variable. It should be 'this' .

Again, think we've already gone over this but just in case!


Ohh yes, I though about it but forgot to change it :(.. Good you did it :D.


My suggestions:
1) In order to get a good coverage, you always needs to execute the same test…
2) … or in different environmentS.
3) term Multiplexation to assign multiple variants of the same test and uses…
4) unlike the tradicional multi-dimensional-matrices of parameters.


I think it looks good. I mean, I think just displaying server errors in a box would be enough. We don't have that many forms.

I'm confused about errors. What does that object look like? It looks like you copied the syntax `update` function but you're using strings for field names, and replacing the whole errors object every time. Wouldn't you want to merge the errors somehow?

Maybe I just need to see it in action.


nice! I like this approach.


reviewed.


Wouldn't it make more sense to change the target server to regular master, and reload it, *then* transition the existing master to shadow?


yes... that's where the admin one gets generated...


En årsag til fejring!


<copyeditor>s/proeprty/property/</copyeditor>


This will also catch when waterbutler responds with a 401 error code, since the request is empty. So I think ```if waterbutler_request.status_code == 401:
        raise PermissionDenied``` can be removed. What do you think?


Indentation: tabs => 2 spaces.


We posted it to Reddit under the old URL so there was really no choice...


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


I have a similar thought; use the `net_new_spot_requests` to tag them a little later.  The `time.sleep()` inside the request loop is an annoying slowdown.  The `dictwrap()` allows objects to look like dicts for use as either parameter bundles, or for general manipulation by the query operators.  You should be able to use a `dictwrap`ed object just like its original.

The whole `dictwrap` thing is confusing.  I will try to clean it up.

If you want me to solve this tagging problem, then send a pull request now.


Why an empty title here?


Thanks for your reply, I'll try that :)


Is it possible to move the `digital-cta` to  the `<div>` below? Might avoid mixing components.


Sorry, my mistake... is now removed.


The stack.sh might get messy in case we also have similar choices on hostif or use more such interface choices.


I modified `c.Lookup()` to log the state of `c.Container` just before returning and accept an integer to indicate the call site:
- `1` - call at the beginning of `c.start()`,
- `0` - call from within `c.Exists()`,
- `3` - call from within `c.Delete()`,
- `4` - call from within `c.start()` on the result of `c.Reset()` (right after `c.Delete()`).

`(con | err)`, `(inspected)`, `(not found)` is a mnemonic of the logged return point from `c.Lookup()`.

`c.Container has it: true` means `c.Container` field has a non-nil value.

Here's the most interesting part of the boot log I've got when the panic occured (watch for `foo`):
```
INFO[0004] !!! Lookup 'foo' 1 (not found) c.Container has it: false
INFO[0004] !!! Lookup 'ntp' 1 (inspected) c.Container has it: true
INFO[0004] !!! Lookup 'foo' 0 (not found) c.Container has it: false
INFO[0004] !!! Lookup 'console' 1 (inspected) c.Container has it: true
INFO[0004] !!! Lookup 'foo' 0 (inspected) c.Container has it: true
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x140 pc=0x5c06a0]

goroutine 53 [running]:
github.com/rancherio/os/docker.(*Container).start(0xc20802ea80, 0x1, 0x0)
        /source/gopath/src/github.com/rancherio/os/docker/container.go:584 +0x670
```

Looks like we're getting inconsistent results from docker here. Just a guess: can docker return different results looking up the containers while it's still starting up and hasn't yet reconciled all the containers from the previous run?

Here's the boot log beginning with launching system containers: https://gist.github.com/imikushin/061dd04ac4bef9e8bdda


On second thought, for now its fine. I will make the changes.


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


That works for me


Potential fix to #2166.  Leave temp files in place


On what basis? (Are you seeing lots of retransmits?)


Should this be private now?


This way round is much nicer!


Checked with example_run. It worked fine..
Thanks Andre..


Breaks with `cmake .. -G "MSYS Makefiles"`
https://gist.github.com/80bf81952cd9ad0ce4f3


Okay, if the stack.sh gets messy I'm okay with that, because the settings file is more likely be viewed by the customer than the stack.sh.


Can `bower` go as you are using `jspm`?


@asterite create awesome release posts. I think that helps to see where the language is going. Yet, I would not update them to the latest version. They are info related to the version is being advertised.

If a certain section is worth it to explain some language feature, yes, I will grab it and use it as docs, and there be updated.

But docs sometimes want's to be a guide, with some sort of global flow. Which does not come from a post.


Arthur here


I guess this code has nothing to do with you. At a glance, do we need the output variable being reassigned to?


@galstrom21 @metral I personally have no issue one way or another, but I don't have +2 power, just +1. I was just commenting because I've seen how the other templates came to be included. Have you guys talked to Bryan Thomson or Paul Sims? Are they on board with adding this template?

Also, more on the technical side, if you want this template to be included in the RPC kilo release then you will also need to patch the version of this file on the kilo branch, and I also recommend that you create a kilo branch on your repo to allow you to ensure future changes do not affect the deployed version.


Oh, thanks, I can solve it myself with that info.  Will do shortly.


On which line?


Does not work with e.g. Python 3.2.3, actually:

      File "/home/dynkle/mps-youtube/mps_youtube/main.py", line 1468
        seek = u"[{0}\u2190{1}] seek [{0}\u2192{1}]"
                                                   ^
    SyntaxError: invalid syntax

Why would we even drop support for python 2 at this point, where it's working perfectly fine?


I don't think I follow, but if you mean anything like changing a server to master while another master still works, it's usually a bad idea.


This needs to be removed from the pull request. Unrequired duplicate code.


I think we do need the reassignment. Line 22 sets up a default value if the next 2 lines don't take affect. My goal was to do no more than satisfy Rubocop and change a as little as possible. So that's copy-n-pasted.


Thanks David, I'll talk a look at these soon


It's only because this header gets included in platform.h, which gets included in lots of other source files so I didn't want to slow down compile times for all of those


cheating


Could you explain what you were trying to fix?


Fair enough


Maybe wind with join on opg table here will be better?


@gautric Would you mind adding your blog post about Camel RPi here?

@geertschuring Would you mind adding Markus interview with the Tinkerforge demo here?


Wanted to use React/JSX style tree building/transpiling for an (incomplete) webgl renderer: https://github.com/uken/DRL


On hand?


Hmm. I guess I am not sure the right way to do it. "rake build" was OK with it and created `conjur-cli-4.25.0.pre.rc1.gem`


Insufficient zeny?


why this restriction?  especially interfaces are types (in some languages at least I would think), aren't they?
also one could imagine  a language could allow to access nested functions as if it was merely namespacing (less likely, as those generally have closure features)
but maybe I'm confusing something that is only useful in the "namespace" completion


I'm interested in why this was deprecated -- is there discussion on a list or something where I can read more about the reason for this? (I'm not a user so it doesn't affect me, but it was on my radar and I might have used it it in the future)


Numbers only?


Bundler 1.3 says this:

Declare the gems that you need, including version numbers. Specify versions using the same syntax that RubyGems supports for dependencies.

```
gem 'nokogiri'
gem 'rails', '3.0.0.beta3'
gem 'rack',  '>=1.0'
gem 'thin',  '~>1.1'
```

Most of the version specifiers, like >= 1.0, are self-explanatory. The specifier ~> has a special meaning, best shown by example. ~> 2.0.3 is identical to >= 2.0.3 and < 2.1. ~> 2.1 is identical to >= 2.1 and < 3.0. ~> 2.2.beta will match prerelease versions like 2.2.beta.12.


Yeah but this option is literally exposed everywhere and can already be set?  `$ babel --jsx-pragma DRL.createElement` etc.


I don't think that number of dialogs is something that we should worry about. Users should never see those dialogs and we should aim at this goal. Also with WER enabled or disabled crash reporting is quite intrusive now, anyway.

> However he doesn't seem to think that MS will actually have a look unless it crashes like 100 000 000 times with the exact same crash (he said "and I'm not joking" about the number).

I completely understand what he mean. But still if we send reports 0 times they definitely won't look at it.

> I have to say that the DrDump guy somewhat destroyed the hopes I had that we could gain something from WER by having MS and the driver makers fixing bugs but it's apparently quite unlikely.

I think WER should be enabled regardless even if it won't do anything good for us. It is the only thing we can do about crashes that are not in our code. But I don't want to argue about it, so if you don't want WER I will have to live without it :)

All in all it is probably true that we will not be able to generate large volume of reports to be of interest for big companies. But it has not be proven.


I fear that this "overloaded" header file won't get pass Josef's strict examination :) Can you please try to reduce the number of macros here to a minimum?


Cool! Didn't know that. Thanks!


@psarna If you move the IP address (which happens when you demote a master back to a shadowmaster) it will sever any existing client connection and (in most cases) cleanly migrate clients to the "new" server.

Reading this it seems like there's a period of time where you are offline, which is less than ideal.


Misquici, valeu @itamarnet!


Was this an intentional change ? Won't this lead to an infinite loop ?


nice, but doesn't seem so commonly useful to me, especially as it's likely we miss the type for dynamic languages where chained calls are most likely.

Also, just FTR in C you'll sometimes see calls with the function name wrapped by parentheses to avoid preprocessor replacement `(foo)(1,2,3)`, and this won't get caught here.  But well, to properly do that we'd need perfect language knowledge, so it probably doesn't matter.


just learnin


Good job :+1:


This change wipes out the suffix code without replacing it as far as I can tell. Did you check the matmul report from `cabal bench` after this change?


Sweet! Thanks for suggesting this library and adding it in!


doesn't the return type also contain things like `const` or alike?


Good thing, this was a longstanding ugly solution to be improved.
Could we detect the default exe and desktop file without a conf var?


Don't agree.

Don't agree.

Agree.


Oh! now I understand.


Made new pull request, will check other pull request if they also need a file_exists statement.
https://github.com/Ron-e/sentora-core/commit/842df685756cc255930f518b5aa6a2f9203288a8


Couple of minor typos eg "enviroment" "Also here is are instantiated"


Despues de haber compilado el ejemplo de [APM RC Input](https://github.com/diydrones/ardupilot/blob/master/libraries/RC_Channel/examples/RC_Channel/RC_Channel.pde), he intentado rosificar esto.
El problema es que ROS necesita un int main() para hacer las inicializaciones (ros::Init, ...) y no parece compatible con el uso de AP_HAL_MAIN. (No me deja compilar, no lo hace si AP_HAL_MAIN() esta dentro de main()).

Yo creo que lo lógico es no utilizar el AP_HAL_MAIN() como lo he intentado [aqui](https://github.com/imuguruza/ardupilot/blob/ros_developer/ROS/apm_rc/src/apm_rc_main_old.cpp), pero esto no funciona, no lee valores.


Well, yeah, but this is just a test, which checks if lizardfs-admin works properly with `metadataserver-status` and `list-metadataservers` commands. All tests are run on a dedicated test installation, so downtime is not really an issue here.


Also, I would suggest opening an issue at ember-cli and referencing this commit. I think it makes sense to keep this diff clean of comments.


Ugh, adding `werkzeug` as a dependency for just a single function? I think we can extract it from there.


Ah. My mistake, carry on! :)
On 15/05/2015 10:35 pm, "Piotr Sarna" <notifications@github.com> wrote:

> Well, yeah, but this is just a test, which checks if lizardfs-admin works
> properly with metadataserver-status and list-metadataservers commands.
> All tests are run on a dedicated test installation, so downtime is not
> really an issue here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lizardfs/lizardfs/commit/060963e8d5d9c0151ca27173b4a3f25966d5817e#commitcomment-11215468>
> .
>


Si quito el main() y dejo sólo AP_HAL_MAIN() funciona, leo los valores de la radio.
Tengo dos opciones:
1º ROSificar utilizando el AP_HAL_MAIN()
2º Adivinar por qué razón no leo las entradas RC [aqui](https://github.com/imuguruza/ardupilot/blob/ros_developer/ROS/apm_rc/src/apm_rc_main_old.cpp), que es lo que me parece mas limpio y lógico


Space before the semi-colon, uncrustify leftover.


Need to update install script as well.


Uncrustify left-over


Cool!

Ajuna Kyaruzi wrote:
> Ran 'rubocop -a' which autocorrects common errors like spacing and
> using double quotes instead of single ones and deprecated method names
> etc.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CSC322-Grinnell/boardbank/commit/c35f252ce63b87114ed42bb70951f63426169579#commitcomment-11211131>.
>


Another uncrustify left-over


trivial, but only need to test *pos


Nah, that did not work. You should add it in CTOR initializer list.


Review complete


Resolve the error message at this line


It seems like this should be log.info or debug or perhaps not logging at all since people should be using DoNotIndex intentionally rather than to indicate errors. If they did need to log something, they could do that at the point just before raising the exception


Missing `private` (and brace placement is off).


via the project's


@erumhannan Is the issue with the Mapbox Android SDK or the Mapbox Android Demo app?  If it's the former, have you tried https://github.com/mapbox/mapbox-android-sdk#eclipse-hardcoding---not-recommended?


Yup, that does it.


<?php
echo "hello";


I don't think this was unnecessary.  This check allowed for the use of descriptor.get / descriptor.set inside class property decorators.


Have an example?


@millerdev This test failed whenever I rebased setvalue onto staging


你好啊 添加


This was short circuiting and not going through all validations on mug load


Good!  I see you noticed how slow this code is!

https://github.com/klahnakoski/SpotManager/issues/12


双方的发生地方


Yes, you are right about interface types (this is for instance the case in go). Will change.

About functions - this is rather the namespace search (when f is a function type you probably can only invoke () on it). For the namespace search you are theoretically right but since the ctags parsers skip function bodies, you won't get the nested function parsed anyway.

I rather went the stricter way everywhere because the main problem was we were getting too many irrelevant results (or on the opposite side, no results at all).


Ah yeah true, make sense :+1:


Thank you :-)
It is not of vital importance .....

What you think about to add a "Save as new" function to duplicate an item ?

![unbenannt](https://cloud.githubusercontent.com/assets/1033298/7655094/b54e3730-fb21-11e4-9861-49f54f7d8d0d.JPG)


So is this an issue of units? Are the units of precision different because 4326 and 3857 work in different coordinate units? My ignorance here is considerable.

I'm specifically asking because this PR is a reversion of https://github.com/Geolicious/qgis2leaf/commit/537872f44a6a9bd82a5b226bb69da56f04e52e36.


forgot to add the second param .... that's why. But the regexp itself do have some problem I believe.


@Myoldmopar Upon updating the test file `HeatPumpWaterHeaterStratified.idf` I am now getting this error:

```
   ** Severe  ** DualSetPointWithDeadBand: Unanticipated combination of heating and cooling loads - report to EnergyPlus Development Team
   **   ~~~   ** occurs in Zone=SPACE5-1 During Warmup, Environment=Miami Intl Ap FL USA TMY3 WMO#=722020, at Simulation time=01/01 08:30 - 08:45
   **   ~~~   ** LoadToHeatingSetPoint=NAN, LoadToCoolingSetPoint=NAN
   **   ~~~   ** Zone Heating Set-point=21.10
   **   ~~~   ** Zone Cooling Set-point=23.90
   **   ~~~   ** Zone TempDepZnLd=2483.75
   **   ~~~   ** Zone TempIndZnLd=NAN
   **   ~~~   ** Zone ThermostatSetPoint=0.00
   **  Fatal  ** Program terminates due to above conditions.
```
It seems like I've broken it pretty good. I'm not really sure what's causing the problem as I didn't touch any of the code where this error comes from. It could be some merging issues with the vshpwh, I don't know. Can you help me figure this out?


Yup :) It's manageable -- 40 seconds on my macbook air.  Definitely the bottleneck, though.


Should actually be `@returns`, but we can scoop up jsdoc issues in a separate PR. There's a bunch that need updating.


Tell me more about this.   May you include a sample settings file in /resources/config that uses this?

We may both implemented the same thing, but in different ways :/


tpo: "new only method" -> "the only method"


fi=le になってますね


:shipit:


What do you think about shortening this to something like `self.assertListEqual(sorted(res['_source']['id']), ['core.mockmodel.1', 'core.mockmodel.2'])` to avoid needing to freeze so much of the fixture data in this test?


par


What I mean, we already know $custom, could we simply set these two new vars acording to set $custom?


fixed in next commit.


@djberg96 - we should not mention VMM since it is a separate product that we are not integrating with in this project.


There should be no need to call `c.Lookup()` again from `c.Exists()` as it's just been called (right before calling `c.Exists()`)


I'd like to remove this special case and require the use of `Nullable!string` instead here.


Could you print the exception, or even better, reraise it?


E.g. I formatted them incorrectly in https://github.com/cfpb/cfgov-refresh/blob/flapjack/src/static/js/modules/util/web-storage-proxy.js (should be in code comment blocks for eslint to pick it up and use `@returns`).


I'm pushing this so I don't forget this branch exists, ignore this commit.


Yeah, without inheritance information it's not as useful. I just thought about all the cases where we have varType and where we could use scope completion - which is variable and function - so I added the function too. And the whole implementation means just skipping the parentheses.


addressed in version 0.0.2


collapse into a single appendf


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


redistribute NX and to continue the development of NX 3.x
as new upstream instead. The package will see a slight
name change to *nx-libs* starting with nx-libs v3.5.99.0.


It may not be obvious from the code, but ```Readers.reader(expectedFile)``` actually handles reading from stdin in the case that the expected file is null.

See
http://www.dishevelled.org/compress/apidocs/org/dishevelled/compress/Readers.html#reader(java.io.File)


There's a ```StringListArgument``` class for this use case, see
http://www.dishevelled.org/commandline/apidocs/org/dishevelled/commandline/argument/StringListArgument.html


One icon directly on the items in the list could be better as start.

After, as this functionality will be available, it will be easy to add it to the edition window of each item.

Did you start it ?

> Le 15 mai 2015 à 16:45, Egon <notifications@github.com> a écrit :
>
> Thank you :-)
> It is not of vital importance .....
>
> What you think about to add a "Save as new" function to duplicate an item ?
>
>  <https://cloud.githubusercontent.com/assets/1033298/7655094/b54e3730-fb21-11e4-9861-49f54f7d8d0d.JPG>
> —
> Reply to this email directly or view it on GitHub <https://github.com/ionize/ionize/commit/bf5a11105118488f29559dfdc1e5020a2a040dea#commitcomment-11215651>.
>


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Or contribute to either of those :)


See #2166

Leaves "scratch-for-unzip-12345" in place

```
----  shp_2015-05-15-10-14-16-169
      |   subwaylines_p_odp.cst
      |   subwaylines_p_odp.zip   (shapefile)
      |   wfsrequest.txt
      |--- scratch-for-unzip-12345  (examining files / repackage shp and separate pieces)
            |  subwaylines_p_odp.cst
            |  subwaylines_p_odp.dbf
            |  subwaylines_p_odp.prj
            |  subwaylines_p_odp.shp
            |  subwaylines_p_odp.shx
            |  wfsrequest.txt
```


It should not be hard-coded. You can pass host from rails by JsEnv. http://codetunes.com/2014/5-tips-on-how-to-use-angularjs-with-rails-that-changed-how-we-work/

You can specify your `host` using figaro gem e.g.: https://github.com/laserlemon/figaro


complete


On  Fr 15 Mai 2015 16:53:17 CEST, Mihai Moldovan wrote:

> redistribute NX and to continue the development of NX 3.x
> as new upstream instead. The package will see a slight
> name change to *nx-libs* starting with nx-libs v3.5.99.0.

@Ionic: can you commit those changes yourself? I have leave home now...

--

DAS-NETZWERKTEAM
mike gabriel, herweg 7, 24357 fleckeby
fon: +49 (1520) 1976 148

GnuPG Key ID 0x25771B31
mail: mike.gabriel@das-netzwerkteam.de, http://das-netzwerkteam.de

freeBusy:
https://mail.das-netzwerkteam.de/freebusy/m.gabriel%40das-netzwerkteam.de.xfb


Cool :)


wouldn't /usr/bin/python2 be more correct? I highly doubt anyone has a default python2 lower than 2.7 anyways, but maybe I'm nitpicking here.


very useful commit!


Extremely minor, but `.replace(/"/g, '\\"');` is the same in a (slightly) more readable form.


Yes, but it's getting a bit too language-specifid. Here I mainly thought about go where I added pointers to varType of variables too (in C this is not done - did I do this right or should I try to remove the pointers?). For functions we put full return values to varType to see the correct prototype in calltips.

So this is just a simple thing to try - if it fails, the completion just won't appear, no disaster.

Maybe I could add also other language's pointers - I'm aware of the pascal's ^ and then there is about every thinkable character for all the Rust pointers :-).


@bcardiff I still agree with @jhass: the memory internals are hidden behind a blog post about a 0.6.1 release, it's not very evident. I'll split it into two blog posts (for a start).


Agreed, that looks really really nice.  Now you just need to get that SHA-1 2017 certificate.  :)


That NaN load is suspicious of one of two things:

* Uninitialized variable, or
* Divide by zero somehow getting through

The setpoints look right, and the TempDepZnLd is good.  But he TempIndZnLd is NaN, so some term in that calculation is coming up badly.  You might take a debug into [here](https://github.com/NREL/EnergyPlus/blob/7d148e7b460747bf462f17c45102329e1c07a9b8/src/EnergyPlus/ZoneTempPredictorCorrector.cc#L2648) and see what term is going awry to help start your investigation.


Why the change from coffee?


Try it out Rocky


too much overhead for such a lightweight module


Oh, I didn't want to clutter the project with my settings file.  I'll commit it if you'd like, but here it is for reference.  An interesting note: I had to severely lower the utility of the 8xlarge instances, because in my workload, they just can't perform as well as 2 4xlarges.  As far as I can tell, the kernel just gets bogged down in context switches and system time goes through the roof without much more actual work getting done.  Interesting, no?

My AMI was built by our automated system (based on packer) with an 8GB root EBS volume.  My workload needs some scratch space, so I want to override the AMI's block device map for /dev/xvda to make it 20GB.  Cloud-init will grow the filesystem on boot to match the block dev size.  I'm purposefully not using ephemerals so that I can make use of instance types such as c4.

```
{
	"budget": 20.00,
	"max_utility_price": 0.02,
	"max_new_utility": 64,
	"price_file": "resources/aws/prices.json",
	"run_interval": "10minute",
	"bid_percentile": 0.8,
    "aws": { "region": "us-east-1" },
	"utility":[
		{"instance_type": "m3.medium", "utility": 1},
		{"instance_type": "m3.large", "utility": 2},
		{"instance_type": "m3.xlarge", "utility": 4},
		{"instance_type": "m3.2xlarge", "utility": 8},

		{"instance_type": "c4.large", "utility": 2},
		{"instance_type": "c4.xlarge", "utility": 4},
		{"instance_type": "c4.2xlarge", "utility": 8},
		{"instance_type": "c4.4xlarge", "utility": 16},
		{"instance_type": "c4.8xlarge", "utility": 20},

		{"instance_type": "c3.large", "utility": 2},
		{"instance_type": "c3.xlarge", "utility": 4},
		{"instance_type": "c3.2xlarge", "utility": 8},
		{"instance_type": "c3.4xlarge", "utility": 16},
		{"instance_type": "c3.8xlarge", "utility": 20},

		{"instance_type": "r3.large", "utility": 2},
		{"instance_type": "r3.xlarge", "utility": 4},
		{"instance_type": "r3.2xlarge", "utility": 8},
		{"instance_type": "r3.4xlarge", "utility": 16},
		{"instance_type": "r3.8xlarge", "utility": 20},

		{"instance_type": "d2.xlarge", "utility": 4},
		{"instance_type": "d2.2xlarge", "utility": 8},
		{"instance_type": "d2.4xlarge", "utility": 16},
		{"instance_type": "d2.8xlarge", "utility": 20}
	],
	"ec2": {
		"request": {
            "expiration": "20minute",
			"price": 0.001,
			"image_id": "ami-0a879562",
			"count": 1,
			"type": "one-time",
			"valid_from": null,
			"valid_until": null,
			"launch_group": null,
			"availability_zone_group": null,
			"key_name": null,
			"security_groups": null,
			"user_data": "#cloud-init\nfqdn_template: filemigration-@IP@.da\nmanage_etc_hosts: false\npuppetmaster: puppet01.da\nsilent: true\n",
			"addressing_type": null,
			"instance_type": null,
			"placement": null,
			"kernel_id": null,
			"ramdisk_id": null,
			"monitoring_enabled": false,
			"subnet_id": null,
			"placement_group": null,
			"block_device_map":
				{ "/dev/xvda":
					{
						"delete_on_termination": true,
						"volume_type": "gp2",
						"size": 20
					}
				},
			"instance_profile_arn": null,
			"instance_profile_name": "files-migration",
			"security_group_ids": null,
			"network_interfaces": [
				{ "subnet_id": "subnet-8cb4ccfb", "groups": ["sg-26442143"] },
				{ "subnet_id": "subnet-aeffc994", "groups": ["sg-26442143"] },
				{ "subnet_id": "subnet-e0811ab9", "groups": ["sg-26442143"] },
				{ "subnet_id": "subnet-666ae84d", "groups": ["sg-26442143"] }
            ],
			"dry_run": false
		},
		"instance": {
			"name": "filemigration"
		}
	},
	"instance":{
		"class":"file_migration.fm.FileMigrationManager",
		"minimum_utility": 2
	},
	"debug": {
		"trace": true,
		"cprofile": {
			"enabled": true,
			"filename": "results/file_migration.tab"
		},
		"log": [
			{
				"class": "logging.handlers.RotatingFileHandler",
				"filename": "results/logs/file_migration.log",
				"maxBytes": 10000000,
				"backupCount": 100,
				"encoding": "utf8"
			},
			{
				"log_type": "console"
			}
		]
	}
}
```


heyya..


Could you post some results ? I mean if it worth it we should do it


Now that we added `value_schema` back, we can do this the right way and distinguish between `JSONInteger` and `JSONNumber`. Maybe not now, but I think this deserves at least a `FIXME` comment.


fair enough :-).


hobaaa


when a script doesn't even build, one has to wonder how much it was tested


@rolodato could you check that this abuse of `null` was not being used in purpose inside an `if` check?


Did you mean to remove this?


whtf?


agreed, we've been changing to python2 everywhere else anyhow.


Olá professor, bom dia, fiz um pull request la no ""onte tem jogo?""

Fique um pouco confuso na hora de criar os atributos, mesmo assim fiz o
pull request para que o senhor possa me corrigir por la!
Peço desculpas por ainda estar um pouco noob, daqui a pouco eu pego o
jeito. kk

Obrigado desde já pela atenção professor.

Att Marcelo Beani


Try to avoid such long lines. Should not be longer then 80 chars. For example: http://dl.dropbox.com/u/33473986/e20c0c026974c02786e2d8f275e31b60.png

This line could looks like:
```slim
button.btn.btn-default(
  type="submit"
  ng-disabled="registrationForm.$invalid || !(user.password === user.password_confirmation)"
)
  Register
```


No, I'm being stupid. Ignore.


do we need these idea files in the git repo?


This is a good idea!


*grins*


Ok I see.


Better behaviour would be to have `_removeUnusedNamespaceDeclarations()` accept `keep_nsmap` which would negate the need add namespaces back in afterwards, but I'm not sure of impacts/where this happens upstream or if it's even possible without modifying libxml2/libxslt


> Yeah, without inheritance information it's not as useful

We do have inheritance for some languages ;) (at the very least PHP, but probably C++ too)

But what I was referring to was we can't know the type a .e.g Python function will return anyway, so it won't help for something like `var.lower().split()`


OK, I will first test locally to see if it has any chance, will try also different bonus values,
and should post something in a few days.


probably need to use a project relative path?


Ah, good catch - definitely not intentional.


I would like to move `user.password === user.password_confirmation` to controller function


I think it would be nice to have a link to MusicBrainz there.


containing the "format enumerated type"?  Cut'n'pasto from the htsExactFormat version?

Also, without wanting to get to htsjdk levels of "review" :smile:… could we put this declaration up beside `hts_format_description()`, which it rather parallels.


I think we're intentionally deprecating `_.matches`. https://github.com/jashkenas/underscore/pull/2057


> in C this is not done - did I do this right or should I try to remove the pointers?

I think C uses `tag->pointerOrder`


Well I will leave the final decision to @XhmikosR. I understand both points
of view and I don't know what is best.


Try to avoid such long lines. Should not be longer then 80 chars. For example: http://dl.dropbox.com/u/33473986/e20c0c026974c02786e2d8f275e31b60.png


@gkalpak - #woot.


The = ObjectGuid::Empty part is not required (and it will make it look like it's required)


Right now, this doesn't have much error handling when you pass an array; however, switching from `forEach` to `map` would easily return an array of errors.


hahaha PHP is like Python since PHP 5.4 as stated [here](http://php.net/manual/en/language.types.array.php):

> As of PHP 5.4 you can also use the short array syntax, which replaces array() with [].


@sheck Can you do similar to the CCV themes?


My computer cant handle building tc. deal with it


It's better to remove it :)
Also Gemfile.lock is missing here.


I find it depressing as heck that both Chrome and Firefox don't even raise a peep about the dh512 site.  We've axed 1024-bit RSA root certs already, and yet here we are accepting 512-bit DHE?  Booooo.


Format ZIP does not crash script when adding cyrillic-named files.


yep, integrate the updated stuff


I like your block_device_map: it is clear and extends the boto command elegantly.  My system handles the drive arrangements of the various instance types [1], lacks elegance, and depends on `setup()` being used to finish the job and make the file system(s).  We should keep both.

[1] https://github.com/klahnakoski/SpotManager/commit/88a5fec121013ff3df351b39f426e31a3a2354c7#diff-69fba58f1c44465829ad84476c0d606dR11


I tried this more comprehensively by replacing all instances of 4326 with 3857 in leafletWriter.py. However, the data is then misprojected. Can you experiment along similar lines before we commit this?


Maybe add a placeholder there?

    ...control", placeholder="(preferably in SVG format)") }}...


How are we going to handle that with Backbone relying on matches?


...and thank you for the config file.  At the very least I will use it for improving documentation.


The comments for this exercise (the instructions) have been reported to be too vague


Agreed.  Perhaps in the future the list of instances and their respective number of ephemeral drives could be either stored in a json file or maybe even retrieved from the EC2 API.  We could also give the InstanceManager a chance to populate the block device map before the instance is requested.

As to the config file, sorry for removing your comments -- my json syntax hilighter hated them :)


That's what I had actually thought that option should do: pass a list of
prefixes into the cleanup mechanism and when the prefix is found, do not
even try to remove it. I still think that's the best way to implement this.

This has nothing to do with libxml2, it's all implemented in lxml.


I should have been more clear; we're just deprecating the name `_.matches`, not the `_.matcher` function.


I'm sorry, it was not intentional...


Maybe just `return min(rest, max)`? Or even replace usages of `next_len` with `min` from `<util/math.h>`.


Я сделаю чтоб сам поправлял, я уже почти закончил.
On 15 May 2015 13:04, "Maxim" <notifications@github.com> wrote:

> Слушай, может ты прав. Я тут понапрасну тебе пишу наверное.
> Погоди пока. Я в Понедельник посмотрю.
>
> 2015-05-15 7:02 GMT+12:00 Markolm <notifications@github.com>:
>
> > Может там другой коментарий по длинней?
> >
> >
> > 2015-05-13 12:33 GMT+02:00 Maxim <notifications@github.com>:
> >
> > > Не к спеху!
> > >
> > > On 13 May 2015 at 18:12, Markolm <notifications@github.com> wrote:
> > >
> > > > Странно, после школы посмотрю.
> > > > On 13 May 2015 05:23, "Maxim" <notifications@github.com> wrote:
> > > >
> > > > > @Markolm <https://github.com/Markolm> работет!
> > > > > чуть чуть с whitespace неровно только:
> > > > > [image: image]
> > > > > <
> > > >
> > >
> >
> https://cloud.githubusercontent.com/assets/10990645/7603258/0da45c38-f984-11e4-83e4-901ccb80e0c1.png
> > > >
> > > >
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11168039
> > > >
> > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11169341
> > > >
> > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11172884
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11201373>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxim47/C-enums/commit/9a77151170039b68dccc19fdfa983a762e679c19#commitcomment-11212067>
> .
>


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I am definitely open to alternatives here.  What I use internal to google is not available outside.  I'm curious to know how other projects bundle test data in a proper python module.  aka, I don't think gdal's autotest setup is effective.  It's not a normally functioning test module ( http://trac.osgeo.org/gdal/browser/trunk/autotest ).


While this works nicely, there is a problem with it.
Adding user.props in our private branch also modifies the solution file which will then need to either be ignored or committed with the user.props file that will be missing later.

I'm not sure what a good solution here would be because it would be even worse to remove user.props from .gitignore...


Pretty sure thats the supported way to support python 2 and 3 in one codebase. Besides, A lot of what `futures` and `six` are doing I was manually doing, this just outsources that.


true


#10  now test mode seems working fine, but for version that don't need min, it stuck due to 88ba4ac9b9028fc8a5acf7a870242b9f90f708e6


Yes, I didn't realize there are other instances of 4326.


#10 the hash tag is added when write the file


Sorry.


Maybe I miss something but I specifically had a look at this and pointerOrder seems to be just read from existing tags files but never assigned from the parser result (which field in sTagEntryInfo is supposed to correspond to pointerOrder?)


Do you have more details on the rationale for this ? is it a quick fix for you, or do you think it's a bug in the proxy ?


Maybe change allow to allowed.


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2178 of 2200 tests passed)

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


`;` may be omitted here :)


Is it too simple of me to ask whether all of this might not be resolved by setting the default precision value in the UI to 15?


Right but Backbone [uses `_.matches`](https://github.com/jashkenas/backbone/blob/1.2.0/backbone.js#L950), which means we can't remove the alias without causing version hell between underscore and backbone.


Is it really difficult to build templates or am I just bad... I'm trying to insert a Vampire Lord template, but all I've managed to do is break my copy a couple of times haha.


Are you suggesting we leave the alias but not document it?


找了半天，没找到哪儿改了，-_-||


igor guarda se riesci un po' a far andare la tastiera con il form di input dinamici che ho creato, mi sta facendo smadonnare


This will be used...


Better fix is pushed. Console output can be a bit ugly with the fallback code for process<1.2.3 as it may interleave output from multiple processes, but running tests in parallel works again properly.

See https://github.com/agda/agda/commit/18accd185a5bd8be89d0f628b5653c37080e27db


class members with 'member_name_' style?


Yes

----- Reply message -----
From: "Tom Chadwin" <notifications@github.com>
To: "boesiii/qgis2web" <qgis2web@noreply.github.com>
Cc: "boesiii" <boesiii@yahoo.com>
Subject: [qgis2web] Changed leaflet crs to epsg 3857 (b308375)
Date: Fri, May 15, 2015 11:18 am
Is it too simple of me to ask whether all of this might not be resolved by setting the default precision value in the UI to 15?

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


related with #29


stabilized

The maintainers will release a new s/release/version/ ?


You don't need `@details.blank?` here with Yt :wink:


> Are you suggesting we leave the alias but not document it?

Yes.


Nitpicking, but I'd put `Billing address` on a separate line and remove parentheses from the text there:

Billing address

      * - full address is required if selected
      payment method is invoicing


Thanks for fixing this typo!


copyWithHash was aimed for no compressed files, but now we have a better idea, since file copy is not so time consuming (uglify is), we can just copy these file to dst folder as well.


`list_exclude_filter` is not a list function but it is filter function. so it have to be sperated from `import_order.listing` then create `import_order.filter` to place it. please push a new commit to fix it.


Instead of ContentWrapper, how do you feel about XmppStanzaGenerator?


You might want to use `@details.embed_html` here, to get the HTML code of the IFRAME from YouTube:

`# => "<iframe type='text/html' src='http://www.youtube.com/embed/...' width='640' height='360' frameborder='0' allowfullscreen='true'/>"`


use `str.format`


This relies on `function` raising in case of an error, a side effect. I think an explicit return value is better.


Publishing your secret keys on GitHub is a bad idea… :see_no_evil:


Removing this function makes it virtually impossible to scroll up to look at a previous line when the test is running, since every new result forces scrolling downwards.
Since this is a something that I occasionally do when tests are running, I'd like that to still be possible. Could we keep this function, but changing it to only scroll if you're at e.g. 80% scrolled down (or a similar reasonably high value)?


scala.meta's parser does create an AST, and it also has correct range positions + it remembers all whitespace, comments - everything that's written by the user.


You can't use root btw. In all cases.


That's weird, I swear I tried this.


would prefer to use various TLS flags (rather than port number) to determine whether we use HTTP or HTTPS URI


Fixed #10


Why is not (x == y) being used everywhere and not x ~= y ?

If you are aware of the habit, you have a chance to change it :]


Thanks, Bruno!


probably shouldn't hardcode this port.


These tests are a little hard to read... Could you make them more readable?


Thanks for providing the exact tool I needed!


would prefer to choose the content wrapper based on some config flag, not based on port number


prefer to select on flag, not port number


@yjroot why do you append `os.sep` after `path`?


Is it not possible to define the new `readCreateProcess` function for older versions of the process library? Something like the definition of the `traverseWithKey` function for older versions of the containers library implemented in https://github.com/agda/agda/blob/master/src/full/Agda/Utils/Map/Compat.hs?


This link is awesome, but author is not correct, the correct author is @ccoenraets
Thanks a lot! Going to add in next update. /o/


Use config flags, not port numbers