Getting error when doing count(*) from elassandra table

Elasticsearch version:
"number" : "6.2.3",
"build_hash" : "e605ecd",
"build_date" : "2018-07-18T19:19:00.787857Z",
"build_snapshot" : false,
"lucene_version" : "7.2.1",
"minimum_wire_compatibility_version" : "5.6.0",
"minimum_index_compatibility_version" : "5.0.0"
Plugins installed: []
JVM version (java -version):
openjdk version "1.8.0_171"
OpenJDK Runtime Environment (build 1.8.0_171-8u171-b11-1~bpo8+1-b11)
OpenJDK 64-Bit Server VM (build 25.171-b11, mixed mode)
OS version (uname -a if on a Unix-like system):
Linux elassandra-0 4.15.0-1023-azure #24~16.04.1-Ubuntu SMP Wed Aug 29 12:54:36 UTC 2018 x86_64 GNU/Linux
Description of the problem including expected versus actual behavior:
Getting error when doing count() from elassandra table when it has more data" ReadTimeout: Error from server: code=1200 [Coordinator node timed out waiting for replica nodes' responses] message="Operation timed o ut - received only 0 responses." info={'received_responses': 0, 'required_responses': 1, 'consistency': 'ONE'}" to Getting error when doing count() from elassandra
Steps to reproduce:
Please include a minimal but complete recreation of the problem, including
(e.g.) index creation, mappings, settings, query etc.  The easier you make for
us to reproduce it, the more likely that somebody will take the time to look at it.

Provide logs (if relevant):

If your table is huge, count(*) can easily timeout.
You can increase the cqlsh timeout with :
 --connect-timeout=CONNECT_TIMEOUT
                       Specify the connection timeout in seconds (default: 5
                       seconds).
 --request-timeout=REQUEST_TIMEOUT
                       Specify the default request timeout in seconds
                       (default: 10 seconds).

Unlike a RDBMS, counting the number of records is an heavy operation in Cassandra (due to its distributed nature).  Checkout this article if you want more understanding.
You may want to use nodetool tablestats to get some rough approximations about your tables.

spark job is writing data into elassandra tables.  where to add --connect-timeout=CONNECT_TIMEOUT parameter as it should affect to all elassandra pods

Under spark, it should be spark.cassandra.read.timeout_ms, see
https://github.com/datastax/spark-cassandra-connector/blob/master/doc/reference.md (never tried myself)


🌐 Language request: finnish

🌐 Language name (native)
Suomi
🌐 Language code
fi-FI

Added to Crowdin :)


failed calling webhook when namespace is not kaito-workspace

Describe the bug
If you follow the installation docs here and here it is stated to install the workspace controller to the namespace kaito-workspace. But when you do this, you get an validation error when adding any preset model.
Steps To Reproduce
helm repo add kaito https://azure.github.io/kaito/charts/kaito
helm repo update
helm install workspace kaito/workspace --namespace kaito-workspace --create-namespace
kubectl apply -f https://raw.githubusercontent.com/Azure/kaito/main/examples/inference/kaito_workspace_phi-2.yaml

Expected behavior
Installation in any namespace should be to final solution, but until that the docs should be adopted.
Logs
Error from server (InternalError): error when creating "https://raw.githubusercontent.com/Azure/kaito/main/examples/inference/kaito_workspace_phi-2.yaml": Internal error occurred: failed calling webhook "validation.workspace.kaito.sh": failed to call webhook: Post "https://workspace.workspace.svc:9443/?timeout=10s": service "workspace" not found
Environment

Kubernetes version (use kubectl version): v1.28.5

Additional context
PR for this is created here #334

Thanks for pointing out this issue with the namespace reference in our Helm chart. We've identified the problem and are addressing it in Pull Request #337, which will be included in our next release. Until then, please install the workspace controller in the default namespace as a temporary workaround. Apologies for the inconvenience and appreciate your patience.


Corrected spelling mistake

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[X] None of my changes are plagiarized from another source without proper attribution.
[X] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @igodsmark
It seems that similar changes have already been accepted earlier for this article you're editing, sorry about that. 😓
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


while using the robin_pg_map, there was an error

Hi, there was an error(SIGABORT) using robin_pg_map in my code .  below was the cause.
prime_growth_policy
/usr/local/include/tsl/robin_growth_policy.h:283
robin_growth_policy.h:283 => TSL_RH_THROW_OR_TERMINATE(std::length_error, "The hash table exceeds its maxmimum size.");
The number of PRIMES in the prime_growth_policy is 40, if I add the number of primes, does the error resolve?

Hello,
Yes, raising the number of primes will allow the table to grow larger on a 64-bit platform. It's not possible on a platform with a 32-bit std::size.
I raised the limit in the last commit. The previous limit was the maximum prime that fitted a 32-bit unsigned integer thus limiting the hash table to 4 294 967 291 elements. I raised the maximum number to 6 597 069 766 657 elements.
Thibaut


Add value lists

This pull request implements the functionality to add CSV files as GH_ValueList  parameters to a BADGERFILE.
Check out the examples/planets/planets.json on how to set that up. The first column in the CSV file is used to display the dropdown list. The output is a json dict of the selected row, indexed by column name. Each value is converted to float if possible (this could maybe be improved in a future PR).
To test, run honey-badger.exe -i examples\planets\planets.json. Run Grasshopper and add the two planet things to the canvas. Add a panel for the output and link the three together. Spend some time exploring our solar system!

I am getting "Object reference not set to an instance of an object" errors when trying to build any json. I re-compiled honey-badger.exe, still not working. Am I missing something?

I get that too! Augh! Looking into it. In the meantime: ipy.exe honey-badger.py -i examples\planets\planets.json will work if you have ipy.exe in your PATH.

Doesn't work for me either. It says, it is missing honey-badger-runtime.dll. Tried to re-compile with make-honey-badger.bat, but it also says Object Reference not set to an instance of an object
This is the full error log when trying to build with ipy.exe:
C:\Program Files\IronPython 2.7>ipy.exe C:\Users\chwaibel\Documents\GitHub\honey-badger\honey-badger.py -i C:\Users\chwaibel\Documents\GitHub\honey-badger\examples\planets\planets.json Traceback (most recent call last):   File "C:\Users\chwaibel\Documents\GitHub\honey-badger\honey-badger.py", line 27, in <module>   File "C:\Users\chwaibel\Documents\GitHub\honey-badger\parameter_compiler.py", line 18, in <module> IOError: System.IO.IOException: file does not exist: C:\Users\chwaibel\Documents\GitHub\honey-badger\honey-badger-runtime\bin\honey-badger-runtime.dll    at IronPython.Runtime.ClrModule.AddReferenceToFileAndPath(CodeContext context, String file)    at IronPython.Runtime.ClrModule.AddReferenceToFileAndPath(CodeContext context, String[] files)    at Microsoft.Scripting.Interpreter.ActionCallInstruction2.Run(InterpretedFrame frame)
at Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame)
at Microsoft.Scripting.Interpreter.LightLambda.Run4[T0,T1,T2,T3,TRet](T0 arg0, T1 arg1, T2 arg2, T3 arg3)
at System.Dynamic.UpdateDelegates.UpdateAndExecute3[T0,T1,T2,TRet](CallSite site, T0 arg0, T1 arg1, T2 arg2)
at IronPython.Compiler.Ast.CallExpression.Invoke1Instruction.Run(InterpretedFrame frame)
at Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame)
at Microsoft.Scripting.Interpreter.LightLambda.Run1[T0,TRet](T0 arg0)
at IronPython.Compiler.RuntimeScriptCode.InvokeTarget(Scope scope)
at IronPython.Runtime.PythonContext.InitializeModule(String fileName, ModuleContext moduleContext, ScriptCode scriptCode, ModuleOptions options)
at IronPython.Runtime.Importer.LoadModuleFromSource(CodeContext context, String name, String path)
at IronPython.Runtime.Importer.LoadFromDisk(CodeContext context, String name, String fullName, String str)
at IronPython.Runtime.Importer.ImportFromPathHook(CodeContext context, String name, String fullName, List path, Func5 defaultLoader)    at IronPython.Runtime.Importer.ImportFromPath(CodeContext context, String name, String fullName, List path)    at IronPython.Runtime.Importer.ImportTopAbsolute(CodeContext context, String name)    at IronPython.Runtime.Importer.ImportModule(CodeContext context, Object globals, String modName, Boolean bottom, Int32 level)    at IronPython.Modules.Builtin.__import__(CodeContext context, String name, Object globals, Object locals, Object fromlist, Int32 level)    at IronPython.Runtime.Importer.ImportLightThrow(CodeContext context, String fullName, PythonTuple from, Int32 level)    at Microsoft.Scripting.Interpreter.FuncCallInstruction4.Run(InterpretedFrame frame)
at Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame)
at Microsoft.Scripting.Interpreter.LightLambda.Run1[T0,TRet](T0 arg0)
at IronPython.Compiler.RuntimeScriptCode.InvokeTarget(Scope scope)
at IronPython.Hosting.PythonCommandLine.RunFileWorker(String fileName)
at IronPython.Hosting.PythonCommandLine.RunFile(String fileName)`

ah. honey-badger-runtime.dll... that's an output of the c# solution. let's see if i can add that to the repo...

@christophwaibel ok, it should work now. note that the honey-badger.exe feature was removed - you'll have to run it with ipy honey-badger.py instead.

I can build successfully with ipy.exe, but now I'm getting these errors when opening Grasshopper.

@christophwaibel oops. Sorry about that. I've fixed it.


Simple balance request?

When I run your code I got this:
Array ( [success] => 1 [return] => Array ( [funds] => Array ( [usd] => 0
... etc.
But how I can get only that "[usd]" parameter, as value ($usdbalance) , like
<? somecode Array ( [usd] etc. -> echo $usdbalance; ?>
how?
Thank you so much!

Hi mignav,
It sounds like you're new to PHP, welcome! Simply, you can assign that array to a variable, then use the keys to lookup the item.
In a more detailed manor, you can follow these steps:
For example if that Array ( [success]... ) etc was assigned to $result:
$result = $Api->apiQuery(...your params...);
then you could access the usd by following the keys down, in this case:
[return], then [funds] then [usd].
You could therefore get a variable called $usdbalance like this:
$usdbalance = $result['return']['funds']['usd'];
You could then use this to variable to print out (echo $usdbalance) as in your question.
These are called associative arrays in PHP, some more information about these can be found here:
http://php.net/manual/en/language.types.array.php
and
https://www.w3schools.com/php/php_arrays.asp
Cheers,
Tony

Yea, I am for sure new in PHP but your code
$usdbalance = $result['return']['funds']['usd'];
does not work.
Anyway, you understand perfectly what I need, now just if you can please provide me a valid PHP code for that! Thank you so much.

Hi mignav,
In your original question what is the name of the variable that you are using var_dump/print_r to get the output: "Array ( [success] => 1 [return] => Array ( [funds] => Array ( [usd] => 0..."?
Tony

Dear Tony,
I am asking regarding your code here in PHP repository named "PHP-btce-api".
How to make echo $usdbalance; to return a proper valid value ?

Hi mignav,
Is btc-e still online? I get an FBI warning when I try to get onto it. It may not currently be possible?
Tony

No, not online but Wex.nz is online.


RFC: Colour Scheme

As requested by @nekrut on Gitter, we can try to rethink the colour scheme of our tutorials.
In this pull request I have configured one possibility, based in part on a colour palette suggested by @nekrut

Please feel free to suggest your own color schemes, and then we can all decide on our favorites. To do so, edit the assets/css/main.scss file; near the top you will find colour definitions:
$red: #f00000;
$orange: #f06d00;
$yellow: #be9900;
$green: #339e3a;
$blue: #009090;
$dark-blue: #2c3143;
$light-blue: #5daadd;
$purple: #882d60;
$light-gray: #9d9d9d;

and then assignments of colours to boxes:
// colors tutorial boxes
$overview-color: $dark-blue;
$agenda-color: $blue;
$keypoints-color: $dark-blue;
$tip-color: $purple;
$warning-color: $red;
$comment-color: $orange;
$handson-color: $green;
$question-color: $blue;
$solution-color: $light-blue;
$details-color: $light-gray;
$feedback-color: $green;

to preview it locally, serve the site and view a template tutorial with all box types (as from screenshots) at: http://localhost:4000/training-material/topics/templates/tutorials/tutorial1/tutorial.html
If you have a nice suggestion, please share a screenshot and/or your css color settings
I am also curious to hear from the colour blind among us which schemes do and do not work for you.
P.S. old color scheme for reference:

P.P.S I will have to undo some changes to the template before merging whatever colour scheme we settle on :)

+1 on making the colors lively. My 10c would be to decrease saturation quite a bit so the sections don't stand out too much from the overall document (screaming at you).

@martenson probably looks like screaming mostly due to the test page? I imagine real tutorials which have much larger blocks of text would be quite diffferent.

@erasche I actually had this feedback for a long time. I consider the boxes and their backgrounds too dominant given they are usually the least important thing on the canvas.

paletton.com may help with picking out a colour scheme

Yes please.

I'd argue that the boxes are important since they mean "hey listen, do something".
For any colour scheme that is chosen, it would be really nice if it could pass WCAG contrast guidelines. I've been using the "WAVE" plugin for this.

The proposed one gets quite close, I worry that going more light / pastel would present contrast issues for some of our users. (Or we go much more light and then black headings.)

similar idea, more pastel-y/muted (and black text for contrast).

$red: #de8875;
$orange: #f9b48a;
$yellow: #ffdf98;
$green: #559e83;
$blue: #8ca4d4;
$dark-blue: #2c3143;
$light-blue: #bae3f7;
$purple: #c2a1da;
$mid-gray: #adadad;
$teal: #66b2b2;

// colors general
$bg-color: $dark-blue;
$menu-color: $dark-blue;

// colors tutorial boxes
$overview-color: $blue;
$agenda-color: $purple;
$keypoints-color: $blue;
$tip-color: $green;
$warning-color: $red;
$comment-color: $orange;
$handson-color: $teal;
$question-color: $blue;
$solution-color: $light-blue;
$details-color: $mid-gray;
$feedback-color: $teal;

to change text colour to black, edit line 56 of assets/css/main.scss
// color: $color;  // white
color: $text-color; // black

not all boxes need to be different colours of course btw, e.g. I like the feedback boxes having same colour as hands-on sections, as users will associate this with "I need to do something" by the time they get there :)
..anyway, this is just to give another example, I am not particularly attached to any one of these schemes, and would be very happy to see any and all other suggestions :)

I consider the boxes and their backgrounds too dominant given they are usually the least important thing on the canvas.

I have to say I agree with this, the boxes with the big colour stripes right across the page are too dominant and in-your-face for me. Do the stripes have to go all the way across the page? Could they be smaller, thinner or instead have the emphasis on the icons? As I'm working on a tutorial at the moment and having 3 big stripes close to each other like below looks ugly to me:

But I can live with these big stripes if I have to and I really appreciate this cool templating system! Can't believe how easy it is to work with! Thanks for creating it!!! ❤

It is also possible to give some elevation to these boxes.

@willdurand good idea :) ..another thought: do you think it would it be possible to style the box headers more like tabs? i. e. that the colour background doesn't extend the full width but only as far as header text? (and maybe with some nice rounded corners).. might make them slightly less prominent/in-your-face to address comments by @mblue9 and @martenson?
(..and for tge question boxes it might be nice to have and actual second tab with the answers in that case? save a little box nesting? though we would have to see if this could work with pdf generation of course)

@shiltemann I am a big fan of your pastel-y version.
As of the markup changes you described I would suggest creating an issue for it and leaving it after GCC so we go in the conference with a stable setup.

the more muted version is VERY nice

@willdurand good idea :) ..another thought: do you think it would it be possible to style the box headers more like tabs? i. e. that the colour background doesn't extend the full width but only as far as header text? (and maybe with some nice rounded corners).. might make them slightly less prominent/in-your-face to address comments by @mblue9 and @martenson?

@shiltemann I love your suggestions! Would love to see what they look like at some point in the future if they're possible, although appreciate they would cost someone(s) time to implement. I agree, the pastels are the nicest colour but I don't think changing to pastel colour would alleviate the concern I have, which is that these bands of colour in the boxes dominate the view. If you look at another example from the tutorial I'm working on below, the colour in the boxes dominates the content (to me anyway) whereas imho the formatting should just enhance it. But very willing to accept I may be the only one with this opinion. Ideally I'd love to know what do (large) groups of users think and prefer, that's the most important opinion to me.

I really like this idea too:

(..and for the question boxes it might be nice to have an actual second tab with the answers in that case? save a little box nesting? though we would have to see if this could work with pdf generation of course)

I was thinking it's a bit too easy/low effort to see the solutions currently 😄 so if something like that is possible I think that would be great! 👍
Thanks for considering these changes!!!

We might then consider using the original colours from the first commit since they are much more easily distinguished and perhaps less overwhelming for those that were concerned about it:

I like your first one more.

For option A:
@mixin tutorial-box ($bg-color, $color: $white) {
  margin-top: 3 * $tutorial-box-spacing;
  padding: $tutorial-box-spacing $tutorial-box-spacing $tutorial-box-spacing;
  border: 1px solid $bg-color;
  & > h3 {
    vertical-align: 1em;
    display: inline;
    float: left;
    margin-top: -3.30rem;
    border-top-left-radius: 15px;
  }

For Option B:
@mixin tutorial-box ($bg-color, $color: $white) {
  & > h3 {
    background-color: white;
    border-bottom: 1px solid $bg-color;
    border-top: 0px solid $bg-color;
    border-left: 12px solid $bg-color;
  }

I still prefer the pastel/muted from @shiltemann ( https://github.com/galaxyproject/training-material/pull/815#issuecomment-395590558 ), maybe we can try with the box title text in white.

@nsoranzo with white titles, unfortunately, they are very low contrast.

Fair enough, I fear that.

I think my preference goes to option A in https://github.com/galaxyproject/training-material/pull/815#issuecomment-399070246
also, to follow up on @willdurand's idea, here is what it could look like with some shadow/elevation added (though he could probably do it nicer ;)):

Here some screenshots for @bebatut's tweaks. A bit more coherent set of colours I think, and the hands-on and question boxes are a more subdued colours since they account for most of the boxes and shouldn't distract from the tutorial contents

dummy tutorial
section of real tutorial

Checked over new colours and tested against wcag, all looked good except code blocks and hyperlinks (too light). So I've changed those as little as possible to meet wcag AA guideliness (not the stricter AAA guidelines.) The code blocks were always hard for me to read anyway.

Wuhaaaaaa! Awesome!!!


Migrate event history database

Description

Changed table size raw query to prepared statement. It's not recommended to use prepared statement for SELECT query which return cursor, retaining it as is.
Changed AndroidEventHistoryDatabase to create a new file in the path returned by getDatabasePath and migrate any existing contents from cache directory file.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Unresolved comment from #101

Let us not add db specific methods to FileUtil. I would just recommend adding a move() method to FileUtil to move the db from old location to new location.
I would recommend the init logic to be
File db_path = construct_new_db_path
if (!db_path.exists()) {
  // Try migration from cache dir
  File cache_dir_db_path = construct_old_db_path
  FileUtil.move( cache_dir_db_path, db_path);
}

// If it still does not exist, assume migration failed and create a new file.
if (!db_path.exists()) {
   db_path.createFile();
}


Rewrite: v13 --> v14

:boom: Djs v14! :boom:
What this PR changes:

Rewrite code in v14
Fix a few bugs

@ZeroDiscord Please check if everything is alright and lemme know if any errors might have crept in.

I think that should do it


JSON serialization should include key and url_safe_key ?

Apparently the entity default JSON serialization excludes the key and url_safe key.
According to a customer "Both of those are super useful, and IMO are parts of the datastructure, not internal. "

AFAIK, datastore doesn't use / render / speak JSON:  the API is either protobuf-over-HTTP or protobuf-over-gRPC.
$ git grep -i json gcloud/datastore/
gcloud/datastore/_generated/_entity.proto:    // In JSON requests, must be base64-encoded.
gcloud/datastore/connection.py:            raise make_exception(headers, error_status.message, use_json=False)

Even if we were speaking JSON, the Entity representation would have the key.

I wonder if the user is talking about the fact that Entity inherits from dict, and accesses the properties via __getitem__, but also has its key as an attribute.  That is a deliberate design choice (see that JSON representation in the docs, which also separates the key from the properties.

Hello,
One of the challenges of maintaining a large open source project is that sometimes, you can bite off more than you can chew. As the lead maintainer of google-cloud-python, I can definitely say that I have let the issues here pile up.
As part of trying to get things under control (as well as to empower us to provide better customer service in the future), I am declaring a "bankruptcy" of sorts on many of the old issues, especially those likely to have been addressed or made obsolete by more recent updates.
My goal is to close stale issues whose relevance or solution is no longer immediately evident, and which appear to be of lower importance. I believe in good faith that this is one of those issues, but I am scanning quickly and may occasionally be wrong. If this is an issue of high importance, please comment here and we will reconsider. If this is an issue whose solution is trivial, please consider providing a pull request.
Thank you!


false positive: "class implements undeclared interface"

running phan 3cedc55 on https://github.com/fruux/sabre-dav/commit/b2ed67cc48f7f5e8f3d94d5bffa6963ad75850fd using
find lib/ -type f -name '*.php' > phan.in
../phan/phan -f phan.in -3 vendor -o phan.out

emits false positive warnings like
lib/CalDAV/Backend/AbstractBackend.php:17 PhanUndeclaredInterface Class implements undeclared interface \Sabre\CalDAV\backend\backendinterface
lib/CalDAV/Backend/AbstractBackend.php:54 PhanUndeclaredClassMethod Call to method getCalendarObject from undeclared class \Sabre\CalDAV\backend\backendinterface
lib/CalDAV/Backend/AbstractBackend.php:111 PhanUndeclaredClassMethod Call to method getCalendarObjects from undeclared class \Sabre\CalDAV\backend\backendinterface
lib/CalDAV/Backend/AbstractBackend.php:115 PhanUndeclaredClassMethod Call to method validateFilterForObject from undeclared class \Sabre\CalDAV\backend\backendinterface
lib/CalDAV/Backend/AbstractBackend.php:139 PhanUndeclaredClassMethod Call to method getCalendarObject from undeclared class \Sabre\CalDAV\backend\backendinterface
lib/CalDAV/Backend/AbstractBackend.php:142 PhanUndeclaredClassMethod Call to method read from undeclared class \Sabre\CalDAV\Backend\vobject\reader
lib/CalDAV/Backend/AbstractBackend.php:144 PhanUndeclaredClassMethod Call to method __construct from undeclared class \Sabre\CalDAV\Backend\caldav\calendarqueryvalidator
lib/CalDAV/Backend/AbstractBackend.php:145 PhanUndeclaredClassMethod Call to method validate from undeclared class \Sabre\CalDAV\Backend\caldav\calendarqueryvalidator
lib/CalDAV/Backend/AbstractBackend.php:177 PhanUndeclaredClassMethod Call to method getCalendarsForUser from undeclared class \Sabre\CalDAV\backend\backendinterface
lib/CalDAV/Backend/AbstractBackend.php:189 PhanUndeclaredClassMethod Call to method calendarQuery from undeclared class \Sabre\CalDAV\backend\backendinterface

I guess phan does not recognize the backendinterface because of some case-sensitive/insensitive string compares.

Hi @staabm.
Can you give it a try again with the patch I just pushed to Phan and see if that resolves the issue?

@morria thx, some of the warnings are gone.. I will check what the remainings are, but those could indeed by caused by wrong typing ;-).
thank you!

@morria after latest commit (6633c00) I'm getting warnings like:
PhanSignatureMismatch Declaration of function
buildForm(\symfony\component\form\formbuilderinterface $builder, array $options) should be compatible with function
buildForm(\Symfony\Component\Form\formbuilderinterface $builder, array $options)
it seems that namespaces from files included from file_list entry in config file are not lowecased

@tybulewicz Thanks for the report. I was looking for test cases and then just found some of these getting emitted in Phan itself. Hopefully we'll have a fix soon.

@tybulewicz That last patch should resolve the issue you're seeing. Can you confirm?

@morria No issues reported using f706209 revision, thanks!


[question] Recover the task assignment after system restart

Hi,
I am using the UniqueHostAttrConstraint to ensure tasks assigned to different host.
However, after the system restart. The constraint is not working. I believe this is due to the assignment history loss after restart.
So what will be the correct way to presisent the assignment history and recover it after system restart?

Fenzo maintains in memory state which needs to be rebuilt after a system restart. For example, upon system restart we first initialize task states from persistence (outside Fenzo, in our framework) and input the state into Fenzo by calling TaskScheduler.getTaskAssigner().call(...) for each task that was in the assigned state. After all running/assigned tasks have been input into Fenzo, the scheduling loop that calls TaskScheduler.scheduleOnce() is started, as explained in here.
This way, Fenzo lets you use your favorite persistence and focuses purely on resource scheduling.

@MinosPong let me know if you need any further assistance on this topic.

Thank @spodila. Yes, your answer is helpful. I was using a old version of fenzo lib (0.8.2) which do not have ```
void setAssignedResources(AssignedResources assignedResources);
AssignedResources getAssignedResources();
Which make me confused how persistence the assigned resources. It looks good in the new version.

@MinosPong I am glad that worked for you.
I am curious as to what resources you are using for the AssignedResources in that call. We use that for a custom two level resource to represent assignable virtual network interfaces in EC2 VPC which can have multiple assignable IPs.

Feel free to reopen this or a new issue if you have more questions.


public vs. private

in speaking with ailey and others, the word "public" is confusing to people, because we actually mean "secret", so rather than calling things "public" vs. "private", perhaps we should call them "public" and "secret"?
so, ailey has to give us her secret url, not her public url.
thoughts?
@j6k4m8 @kunallillaney @alexbaden @randalburns

@jovo For the purpose it is the public URL since that data has to be publicly accessible. We can call it the secretly publicly accessible URL if you like.

dropbox doesn't call links that are shared "public", neither does gdrive.
gdrive says, under "sharing"
"Link to share (only accessible by collaborators)"
dropbox says:
"Who can view this?

[ ] Anyone with the link
[ ] Only people with the password
"
i don't see why we have to say public if they don't?

gdrive actually has 3 options:

public
shared with collaborators
private

we only have the first two.


API discussions

Some thoughts regarding a statsmodels API
Our state of departure is the following:
model.fit(cov_type='wildclusterbootstrap',
             cov_kwds = {'cluster' : cluster,
                         'B' : 9999,
                         'weights_type' : 'rademacher',
                         'impose_null' : True,
                         'bootstrap_type' : '11',
                         'seed' : None,
                         'param' : 'X1'}).summary()

The internal function which is called, wildboottest(), returns a pvalue for a bootstrapped t-test of the null hypothesis $X_1 = 0$ vs $X_1 \neq 0$. Further, users can impose the null hypothesis $X_1 = 0$ on the bootstrap data generating process via the impose_null argument.
In consequence, we can report a full "regression table" as below by looping over all parameters [x1, ..., x10] and also impose the null hypothesis for each hypothesis on the bootstrap dgp. For this use case, it would be nice if we could allow a 'param' value of e.g. 'ALL', which would loop 10 times over the internal wildboottest(), imposing the null if impose_null = True or not.
Second, if a user selects to test only one parameter, e.g. X1, it would be great if we could simply output the column for X1. I suppose that most times, researchers are not interested in inferences on the full set of regression coefficients, so computing all of them would only be wasteful. This is how fwildclusterboot, boottest and WildBootTests.jl all operate, but not what statsmodels does. Maybe we should ask the statsmodels maintainers what they think about this?
Last, once the bootstrapped vcov matrix is available, we can compute an F-statistic, but we can no longer impose the null hypothesis on the bootstrap dgp as we would use only one bootstrapped vcov matrix for all k coefficient tests. In consequence, specifying impose_null = True needs to lead to an error. With a bootstrapped vcov, inference (both p-values and confidence intervals) can then be based on bootstrapped t-statistics (percentile-t, strongly preferable and what currently happens) or asymptotic approximations (i.e. the t(G-1) distribution). In short, with a full vcov matrix, we should be able to support all features of a regular statsmodels regression table, including standard errors, at the cost of being able to impose the null hypothesis on the wild cluster bootstrap dgp.
All in all, providing a bootstrapped vcov and se's leads to deviations from fwildclusterboot, boottest and WildBootTests.jl, which does not necessarily make me happy. 😄
Still, I lean towards having both, but we need to make a few decision as sketched above.
I hope this is all understandable - let me know what you think @amichuda ! =)
                                 OLS Regression Results
=======================================================================================
Dep. Variable:                      y   R-squared (uncentered):                   0.737
Model:                            OLS   Adj. R-squared (uncentered):              0.735
Method:                 Least Squares   F-statistic:                              278.0
Date:                Mon, 17 Oct 2022   Prob (F-statistic):                   2.58e-279
Time:                        18:35:01   Log-Likelihood:                         -1769.6
No. Observations:                1000   AIC:                                      3559.
Df Residuals:                     990   BIC:                                      3608.
Df Model:                          10
Covariance Type:            nonrobust
==============================================================================
                 coef    std err          t      P>|t|      [0.025      0.975]
------------------------------------------------------------------------------
x1            -0.0457      NaN    -1.016      0.310      -0.134       0.043
x2            -1.1441      NaN    -24.947      0.000      -1.234      -1.054
x3             0.3851      NaN      8.243      0.000       0.293       0.477
x4            -0.6970      NaN    -15.284      0.000      -0.786      -0.607
x5            -0.5754      NaN    -12.380      0.000      -0.667      -0.484
x6             0.0367      NaN      0.790      0.430      -0.054       0.128
x7             0.2766      NaN      5.957      0.000       0.185       0.368
x8            -1.1516      NaN    -25.925      0.000      -1.239      -1.064
x9             0.5564      NaN     12.022      0.000       0.466       0.647
x10           -1.2981      NaN    -28.412      0.000      -1.388      -1.208
==============================================================================
Omnibus:                        0.445   Durbin-Watson:                   0.955
Prob(Omnibus):                  0.800   Jarque-Bera (JB):                0.327
Skew:                          -0.007   Prob(JB):                        0.849
Kurtosis:                       3.087   Cond. No.                         1.18
==============================================================================

Notes:
[1] R² is computed without centering (uncentered) since the model does not contain a constant.
[2] Standard Errors assume that the covariance matrix of the errors is correctly specified.
[3] Standard errors not available for wildclusterbootstrap

Yes! I agree. To be honest, if generating the vcov creates many problems (especially if there will be discrepancies between R and Julia), I don't think just keeping standard errors as NaNs are a bad idea.

Second, if a user selects to test only one parameter, e.g. X1, it would be great if we could simply output the column for X1. I suppose that most times, researchers are not interested in inferences on the full set of regression coefficients, so computing all of them would only be wasteful. This is how fwildclusterboot, boottest and WildBootTests.jl all operate, but not what statsmodels does. Maybe we should ask the statsmodels maintainers what they think about this?

My thoughts on this though is that perhaps, we should allow the wildboottest function to have a param argument if the user wants to, but if implemented in a statsmodels model instance, then the table would show all bootstrapped p-values and such. At least in my work, I'm usually interested in seeing significance of all variables, but more than happy to inquire with statsmodels maintainers if you feel strongly about it.
I finished up the pull request: https://github.com/amichuda/statsmodels/pull/1 and it seems to work okay. Here's a regular homoskedastic version and ours now:
                                 OLS Regression Results
=======================================================================================
Dep. Variable:                      y   R-squared (uncentered):                   0.737
Model:                            OLS   Adj. R-squared (uncentered):              0.735
Method:                 Least Squares   F-statistic:                              278.0
Date:                Tue, 18 Oct 2022   Prob (F-statistic):                   2.58e-279
Time:                        16:27:07   Log-Likelihood:                         -1769.6
No. Observations:                1000   AIC:                                      3559.
Df Residuals:                     990   BIC:                                      3608.
Df Model:                          10
Covariance Type:            nonrobust
==============================================================================
                 coef    std err          t      P>|t|      [0.025      0.975]
------------------------------------------------------------------------------
x1            -0.0457      0.045     -1.016      0.310      -0.134       0.043
x2            -1.1441      0.046    -24.947      0.000      -1.234      -1.054
x3             0.3851      0.047      8.243      0.000       0.293       0.477
x4            -0.6970      0.046    -15.284      0.000      -0.786      -0.607
x5            -0.5754      0.046    -12.380      0.000      -0.667      -0.484
x6             0.0367      0.046      0.790      0.430      -0.054       0.128
x7             0.2766      0.046      5.957      0.000       0.185       0.368
x8            -1.1516      0.044    -25.925      0.000      -1.239      -1.064
x9             0.5564      0.046     12.022      0.000       0.466       0.647
x10           -1.2981      0.046    -28.412      0.000      -1.388      -1.208
==============================================================================
Omnibus:                        0.445   Durbin-Watson:                   0.955
Prob(Omnibus):                  0.800   Jarque-Bera (JB):                0.327
Skew:                          -0.007   Prob(JB):                        0.849
Kurtosis:                       3.087   Cond. No.                         1.18
==============================================================================

Notes:
[1] R² is computed without centering (uncentered) since the model does not contain a constant.
[2] Standard Errors assume that the covariance matrix of the errors is correctly specified.

and this is ours:
                                  OLS Regression Results
=========================================================================================
Dep. Variable:                        y   R-squared (uncentered):                   0.737
Model:                              OLS   Adj. R-squared (uncentered):              0.735
Method:                   Least Squares   F-statistic:                              278.0
Date:                  Tue, 18 Oct 2022   Prob (F-statistic):                   2.58e-279
Time:                          16:27:09   Log-Likelihood:                         -1769.6
No. Observations:                  1000   AIC:                                      3559.
Df Residuals:                       990   BIC:                                      3608.
Wild Cluster Bootstrap assuming 11 bootstrap_type and rademacher weight type.

As more things pop up, we can add them, but for now things are running pretty okay and it wasn't really difficult.

works with pandas dataframes now too:
                                  OLS Regression Results
=========================================================================================
Dep. Variable:                  outcome   R-squared (uncentered):                   0.737
Model:                              OLS   Adj. R-squared (uncentered):              0.735
Method:                   Least Squares   F-statistic:                              278.0
Date:                  Tue, 18 Oct 2022   Prob (F-statistic):                   2.58e-279
Time:                          17:00:49   Log-Likelihood:                         -1769.6
No. Observations:                  1000   AIC:                                      3559.
Df Residuals:                       990   BIC:                                      3608.
Wild Cluster Bootstrap assuming 11 bootstrap_type and rademacher weight type.

I think with this basic functionality, I would almost be inclined to start a pull request to statsmodels and see what they think. We can add more functionality later with more pull requests, and I'm thinking there might still be more work to do if they have feedback on how to improve it.

Yes, this sounds like a good idea to me. Good to get feedback early on :)

Have you seen the statsmodels developer notes? Maybe we should ask in the issue first if they are fine with an initial experimental PR, to discuss functionality and the API with them, before we start working towards adhering to all requirements in the dev notes?

I'll close this for now =)


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #85.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

Fixes #85

@conda-forge-admin, please lint


exception from flutter start for iOS

If I run flutter start without having first run flutter ios --init on a project, with an iOS device plugged in, I see an exception:
[~/flutter/flutter_sunflower] flutter start
severe: To copy files to iOS devices, please install ios-deploy. You can do this using homebrew as follows:
$ brew tap flutter/flutter
$ brew install ios-deploy
ProcessException: No such file or directory
  Command: /usr/bin/env xcrun xcodebuild -target Runner -configuration Release
severe:
Exception:
dart:io                                                           Process.start
package:flutter_tools/src/base/process.dart 19                    runCommandAndStreamOutput.<async>
dart:async                                                        Future.Future.microtask
package:flutter_tools/src/base/process.dart                       runCommandAndStreamOutput
package:flutter_tools/src/device.dart 1103                        _buildIOSXcodeProject.<async>
dart:async                                                        Future.Future.microtask
package:flutter_tools/src/device.dart                             _buildIOSXcodeProject
package:flutter_tools/src/device.dart 201                         IOSDevice.startApp.<async>
dart:async                                                        Future.Future.microtask
package:flutter_tools/src/device.dart                             IOSDevice.startApp
package:flutter_tools/src/commands/start.dart 108                 StartCommandBase.startApp.<async>
dart:async                                                        Future.Future.microtask
package:flutter_tools/src/commands/start.dart                     StartCommandBase.startApp
package:flutter_tools/src/commands/start.dart 156                 StartCommand.runInProject.<async>

We should probably print out a message about first running flutter ios --init, if an ios device is connected.

cc @chinmaygarde

Just ran into this one, too :)

I just fixed this a few minutes ago.


Event-Hub input binding should propagate event meta-data

Currently the Event-Hub input binding only propagates the data payload of an event.
AFAIK I could see this is done in the following code-snippet:
                        handler(&bindings.ReadResponse{
				Data: event.Data,
			})

I would change this to propagate the full event including all SystemProperties like "iothub-connection-device-id".

Hi,
We have the same use case: we need to access to metadatas of event produced by IoT Devices. But currently it is not possible.
Is someone have some news about this issue?
Regards,

/assign

The IoTHub SystemProperties only got added to the Azure EventHubs golang SDK in version 3.2.0, but the Dapr EventHubs pubsub and binding components currently are on the deprecated v1.3.1, so minimally Dapr needs to:

update the version dependency to github.com/Azure/azure-event-hubs-go/v3
expose the SystemProperties in the bindings.ReadResponse to the caller.


Adding rating, fixes AB#50

Hi,
This pull request fixes issue(s) AB#50.
A summary of changes are:

Adding rating feature

Please proceed with a review as soon as all the status checks are valid.

@pmanlukas Deployment looks good, please merge


Finished pipeline

This fixes #92 #81 #80

I think this is the pipeline-run YAML. We need the pipeline yaml instead which is generic! Could you check and make the change?

@Anoushka21 Good catch! Give me a second


NESTED PAGES PATH DOESNT SEEM ON URL

@StackedApp(
  routes: [
    MaterialRoute(
      page: SplashView,
      initial: true,
      path: "/",
    ),
    MaterialRoute(
      page: OnboardView,
      path: onboardViewRoute,
    ),
    MaterialRoute(page: LoginView, path: loginViewRoute),
    MaterialRoute(page: MainView, path: mainViewRoute, children: [
      MaterialRoute(page: ProfileView, path: profileViewRoute),
      MaterialRoute(page: MenuView, path: menuViewRoute),
      MaterialRoute(page: TablesView, path: tablesViewRoute),
    ])
  ],
)

I see my pages path on url until mainview. When i came to main view stuck in the http://localhost:11227/#/main. Also i check my previousRoute is empty too. How can i fix this
    return Scaffold(
      resizeToAvoidBottomInset: false,
      body: ExtendedNavigator(
        initialRoute: MainViewRoutes.tablesView,
        router: MainViewRouter(),
        navigatorKey: StackedService.nestedNavigationKey(1),
      ),
    );

this extendednavigator use mainviewrouter which ancestor of profile, menu and tables

Hey, the web support is limited, we're mainly focused on mobile apps. It's usable, but you'll need to do work to get your Urls to be perfect.
In terms of the previousRoute being empty. You can add the StackedNavigatorObserver to your observers so that stacked can track the current and previous routes.


Update tests to use standard method.

This PR cleans up an awkward testing technique.

Thanks @calebjderosier


Retrying should support more advanced policies

I think it might be reasonable for clients to be able to distinguish between failure to communicate with a server on the one hand, and a response with a non-200 status code on the other hand. There are likely cases where the status code could be used to provide early feedback to the user, rather than spinning until the timeout.
What do folks think?

:+1: :+1: :+1: :+1:


Pure go apk.InitDB()

This PR implements the apk.InitDB() method in pure go to remove another call to
the apk binary. We still need to run as root as this function needs to create device
files in the apk filesystem inside the working directory.
The functionality is implemented in three utility functions:
apk.initCreateFileSystem()
apk.initCreateDeviceFiles()
apk.initWriteScriptsTarball()
Unit tests are included for these.
@kaniini I think I've captured everything happening during the initialization, can you confirm?

Closing this in favor of @deitch 's work in #426 :tada:


remove unnecessary arch names

The lines here can cause the following errors on MI300 node. I made these change (removed these lines) before, but after moving to the third-party folder, these lines are back, so make a change again.
RuntimeError: Triton Error [HIP]:  Code: 400, Messsage: invalid resource handle

I don't think we use the arch features anywhere currently so this is probably fine. Does the arch string not includes any features items on gfx94*?

I don't think we use the arch features anywhere currently so this is probably fine. Does the arch string not includes any features items on gfx94*?

No, if we does not remove the lines as in the PR, the string info is:
arch_features = +sramecc,-xnack


Can't run test suite

patreon-ruby (master)$ rspec

An error occurred while loading ./spec/api_spec.rb.
Failure/Error:
  describe Patreon::API do
    before do
      @api = Patreon::API.new("some token")
    end

    describe "Patreon::API#fetch_user" do
      before(:all) do
        @response = File.read(File.expand_path("fixtures/current_user.json", __dir__))
      end

NameError:
  uninitialized constant Patreon::API
# ./spec/api_spec.rb:1:in `<top (required)>'

An error occurred while loading ./spec/util_spec.rb.
Failure/Error:
  describe Patreon::Utils::JSONAPI::URLUtil do
    describe "build_url" do
      it "should build a url" do
        url = Patreon::Utils::JSONAPI::URLUtil.build_url("/")
        assert_equal url, "/"
      end

      it "should build a url with a string includes" do
        url = Patreon::Utils::JSONAPI::URLUtil.build_url("/", "include")
        assert_equal url, "/?include=include"

NameError:
  uninitialized constant Patreon::Utils
# ./spec/util_spec.rb:1:in `<top (required)>'
No examples found.

@wakproductions - Thanks for reporting this. I've created a bug report for my team to review this.

Hi there.  We use minitest, not rspec.  Until the recently-released 0.5.1, there was also a bug running the minitest suite, but
$ rake test

Should now execute our tests properly.  Please update to 0.5.1, and let me know if you have any more problems.


[Scheduled Maintenance] Site down for upgrade

Additional context
Who/what/when/where/why is this maintenance happening

Resolved: LabBricks is back up in 86cb1fd after 1 hour, 4 minutes.


Updated to use the v1beta1 APIs and project status to beta

@liyinan926 https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/pull/291 breaks backward compatibility for the CRD. Can we please wait to close out on those changes before merging this in ?  Thanks.

@akhurana001 sure, this is not urgent.

Hi @liyinan926 Is this PR good to be merged? If so can you do it?

This one is deprecated. I will create a new one.


core: support range for random selection

What problem does this PR solve?
This PR is mainly for supporting range for schedulers.
What is changed and how it works?
This PR changes the region map to region tree and supports random selection according to a given range.
Check List
Tests

Unit test

Side effects

Possible performance regression

Codecov Report

Merging #1783 into master will increase coverage by 0.1%.
The diff coverage is 86.73%.

@@            Coverage Diff            @@
##           master    #1783     +/-   ##
=========================================
+ Coverage   77.42%   77.53%   +0.1%
=========================================
  Files         163      163
  Lines       16432    16511     +79
=========================================
+ Hits        12723    12802     +79
+ Misses       2665     2664      -1
- Partials     1044     1045      +1

Impacted Files
Coverage Δ

server/core/basic_cluster.go
95% <100%> (ø)
:arrow_up:

server/core/storage.go
76.08% <100%> (ø)
:arrow_up:

server/cluster.go
84.21% <50%> (-0.26%)
:arrow_down:

server/core/region_storage.go
81.81% <50%> (ø)
:arrow_up:

server/core/region.go
90.7% <83.49%> (-2.45%)
:arrow_down:

server/core/test_util.go
95.31% <92.85%> (+0.15%)
:arrow_up:

server/core/region_tree.go
97.5% <94.44%> (-0.26%)
:arrow_down:

server/grpc_service.go
57.91% <0%> (-0.87%)
:arrow_down:

pkg/btree/btree.go
86.14% <0%> (ø)
:arrow_up:

... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6c0609...edb4e11. Read the comment docs.

master:
BenchmarkRandomRegion-40    	50000000	        37.0 ns/op	       0 B/op	       0 allocs/op
PASS

tree:
BenchmarkRandomRegion-40    	 1000000	      1688 ns/op	       0 B/op	       0 allocs/op
PASS

master:
BenchmarkAddRegion-40    	   50000	     45234 ns/op	    7413 B/op	     140 allocs/op

tree:
BenchmarkAddRegion-40    	   50000	     46224 ns/op	    8133 B/op	     164 allocs/op

master:
BenchmarkAddRegion-40    	  500000	      4909 ns/op	    1120 B/op	      21 allocs/op

tree:
BenchmarkAddRegion-40    	  300000	      5662 ns/op	    1797 B/op	      45 allocs/op


Request: publish typeahead and bloodhound on npm

In the era of browserify and commonJS it makes sense to publish also front-end-only packages on npm.

npm stores the tagged versions on it's own servers
when managing dependencies, i'd rather manage them all in npm (most of the dependencies I use are already on npm)
it's nice to have the option, isn't it?

This should be improved once v0.11 lands as both bloodhound and the jquery plugin will be wrapped in https://github.com/umdjs/umd.

+1

@jharding Sorry to insist, but with more than 150 open issues, and 37 pending pull requests, this repo is starting to look abandoned. Are there plans to revive it?

+1

For anyone visiting this after the fact: the package on npmjs.com is literally called typeahead.js, not typeahead.  This probably should have been obvious to me, but when I searched for simply typeahead on npmjs I got a different module.  Not sure why both results don't come up...  Likewise, adding a dep on typeahead v0.11.1 in your package.json will fail, it needs to be "typeahead.js":"0.11.1".

@santanubasu makes a very good point. I literally spent a whole day trying to figure out why the Typeahead I was using wasn't emitting the expected events, and then I discovered that Typeahead is a 1-year old version on NPM, while Typeahead**.js** is the correct, up-to-date version.

Ah, as it turns out, it still doesn't work. The factory() in the NPM version doesn't return anything, so require('typeahead.js') returns undefined. I haven't figured out what it's supposed to return to work.

FWIW, I did require("typeahead.jquery.js") and that worked.  However, I didn't assign the result (undefined I assume) to any variable.  Instead, I just used it via jquery, for example: this.$el.typeahead(...)

Thanks, @santanubasu. That's what I ended up doing, although personally I didn't like doing it that way. There should be a proper export so I can do Typeahead(inputElement, options) or something like that. Also I think I still had to make a change to the Typeahead JS file to get it working properly (maybe added a "return typeahead" or something at one point. I also had to require('jquery') in my own code or else it wouldn't work.

Including this library via npm is really annoying. I can include the library, but the module itself uses a require('jquery')... meaning it will pull its own instance of jquery, attach to $.fn.typeahead, and export nothing. So if I have somewhere else in my code var $ = require ('jquery'); $(...).typeahead() that particular $ isn't going to have fn.typeahead. Anyone have a solution to this?

If you are only using browserify, you might be able to get around your issue using browserify-resolutions (https://github.com/Updater/browserify-resolutions).  I had the identical issue you reported with the bootstrap datetimepicker library, and this, um... resolved it.
Note that this doesn't address a situation where you have jquery as a global via some other means, and you also require() it from inside a browserify bundle.  That still seems to break in the way you reported, if anyone has a solution to that, would love to hear it.

@aft-luke YES!  I'm dealing with this exact problem.  Did you figure out a way around it?  I've spent all day trying to come up with a solution.

Yes - you can use a combination of browserify-shim and the package.json browser: {} with a global key to load jquery from a file (or a cdn), point browesrify to that global jquery: 'global:jQuery', then all successive requires will actually be reaching into the singular jquery, which browserify pulled off of window.jQuery. More details here

I'm in the same boat with the browserify people.  For a while my app was using the latest jquery version (2.1.3), so typeahead resolved its jquery dependency using the same version.  Then when 2.1.4 came out, doing a recompile would make typeahead have its own nested version of jquery separate from the one I was setting globally.  The solution was to make my app's version of jquery to 2.1.4 to make typeahead use the same version -- not optimal, but a huge relief once I understood what was happening.

@nikvdp has packaged typeahead.js-browserify: https://www.npmjs.com/package/typeahead.js-browserify
It claims to solve these issues...trying it now!

@mikeymckay Let me know if you get it working.  Whenever I try to use this version, it brings in it's own version of jQuery.
I still haven't solved this problem in a way that I like.  I'm successfully telling Browserify to include the Bower Typeahead.js version with a Browserify shim, but I can't do it in a separated way.
By that, I mean, I can get Typeahead to use the global jQuery only if it is built into my app bundle.  But I like to have my app.js separate from my vendor.js.  For whatever reason, when I put typeahead.js into my vendor.js, Browserify simply won't use my global shim.  It is maddening.  So I have the Typeahead code being bundled with my app.js right now... not ideal, but at least it works.

Has anyone else had any luck with this?

My solution was to leave typeahead and use awesomeplete:
https://www.npmjs.com/package/awesomplete
On Thu, Feb 11, 2016 at 7:03 AM, Chase Gilliam<EMAIL_ADDRESS>wrote:

Has anyone else had any luck with this?
—
Reply to this email directly or view it on GitHub
https://github.com/twitter/typeahead.js/issues/743#issuecomment-182693221
.

@Ch4s3 I am using substack/browserify and found it easy enough to do the following in my package.json both before and after @jharding published typeahead.js to npm.
{
  "browser": {
    "bloodhound": "./node_modules/typeahead.js/dist/bloodhound.js",
    "typeahead.js": "./node_modules/typeahead.js/dist/typeahead.jquery.js"
  },
  "dependencies": {
    "typeahead.js": "0.11.1",
   // or just go directly to github and bypass the npm registry
   "typeahead.js": "github:twitter/typeahead.js#0.11.1"
  }
}

This repo appears to be abandoned. Maintained fork: https://github.com/corejavascript/typeahead.js


Change Font size: Missing _sass folder in mmistakes page starter repo.

I know that font size and types have been discussed in issues before. But all the solutions lead to changing the _sass/reset.scss file. However, i started building my site from the mm-github-pages-starter repo which didn't have _sass folder. And the minimal-mistakes repo has embedded folder layout which is confusing. Nonetheless, I copies the _sass folder into my repo and modified the _sass/reset.scss file, yet there is no change in font size. What am I doing wrong?

When using the theme as a remote_theme or gem, the Sass, layouts, and includes are bundled away and not directly visible... that's why you didn't find a _sass folder.
And because of how Jekyll deals with local overrides of theme files, you can't customize a single _sass file... you need the entire folder. Which isn't that great since you'll have to manage it if the theme ever updates.
What I would suggest is copying /assets/css/main.scss to your repo (which imports all of the themes Sass partials). In this file you can add any custom CSS like changing the fonts or overriding Sass variables after the @import lines.
Example:
---
# Only the main Sass file needs front matter (the dashes are enough)
---

@charset "utf-8";

@import "minimal-mistakes/skins/{{ site.minimal_mistakes_skin | default: 'default' }}"; // skin
@import "minimal-mistakes"; // main partials

// custom CSS here
$global-font-family: Comic Sans;
$header-font-family: Comic Sans;
$caption-font-family: Comic Sans;

body { background: red; }

It's more future proof as you won't have to constantly merge in CSS changes the theme makes. You're only managing a single file instead of the entire _sass folder.


[WinError 3] The system cannot find the path specified: 'C:\Users\v-vgujjula\.azure\cliextensions\connectedk8s'. Retrying ...

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
When installing k8s-configuration and connectedk8s extensions using latest version of az cli,  we are seeing the following issue.
Using an earlier version of the az cli works.

To Reproduce
Expected behavior
CLI-extension addition should succeed
Environment summary
az cli version: 2.35.0
Observed in certain Windows envs
Observed on 2 different setups as of now
Additional context
Excerpts from --debug logs:
``
Given no hashes to check 1 links for project 'pycryptodome': discarding no candidates
Collecting pycryptodome==3.9.8
Created temporary directory: C:\Users\v-vgujjula\AppData\Local\Temp\pip-unpack-nxnyjrrh
https://files.pythonhosted.org:443 "GET /packages/4c/2b/eddbfc56076fae8deccca274a5c70a9eb1e0b334da0a33f894a420d0fe93/pycryptodome-3.9.8.tar.gz HTTP/1.1"<PHONE_NUMBER>8
Downloading pycryptodome-3.9.8.tar.gz (15.6 MB)
-------------------------------------- 15.6/15.6 MB 759.2 kB/s eta 0:00:00
Added pycryptodome==3.9.8 from https://files.pythonhosted.org/packages/4c/2b/eddbfc56076fae8deccca274a5c70a9eb1e0b334da0a33f894a420d0fe93/pycryptodome-3.9.8.tar.gz#sha256=0e24171cf01021bc5dc17d6a9d4f33a048f09d62cc3f62541e95ef104588bda4 (from connectedk8s==1.2.7) to build tracker 'C:\Users\v-vgujjula\AppData\Local\Temp\pip-req-tracker-6brsqoeq'
Running setup.py (path:C:\Users\v-vgujjula\AppData\Local\Temp\pip-install-fljwg0g7\pycryptodome_00b584ba641944f0a9efb77b7443eb3a\setup.py) egg_info for package pycryptodome
Created temporary directory: C:\Users\v-vgujjula\AppData\Local\Temp\pip-pip-egg-info-p85c4npj
Preparing metadata (setup.py): started
Running command python setup.py egg_info
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "C:\Users\v-vgujjula\AppData\Local\Temp\pip-install-fljwg0g7\pycryptodome_00b584ba641944f0a9efb77b7443eb3a\setup.py", line 457, in
set_compiler_options(package_root, ext_modules)
File "C:\Users\v-vgujjula\AppData\Local\Temp\pip-install-fljwg0g7\pycryptodome_00b584ba641944f0a9efb77b7443eb3a\compiler_opt.py", line 341, in set_compiler_options
clang = compiler_is_clang()
File "C:\Users\v-vgujjula\AppData\Local\Temp\pip-install-fljwg0g7\pycryptodome_00b584ba641944f0a9efb77b7443eb3a\compiler_opt.py", line 251, in compiler_is_clang
return test_compilation(source, msg="clang")
File "C:\Users\v-vgujjula\AppData\Local\Temp\pip-install-fljwg0g7\pycryptodome_00b584ba641944f0a9efb77b7443eb3a\compiler_opt.py", line 82, in test_compilation
objects = compiler.compile([fname], extra_postargs=extra_cc_options)
File "D:\a\1\s\build_scripts\windows\artifacts\Python\Lib\site-packages\setuptools/_distutils/_msvccompiler.py", line 327, in compile
File "D:\a\1\s\build_scripts\windows\artifacts\Python\Lib\site-packages\setuptools/_distutils/_msvccompiler.py", line 224, in initialize
File "D:\a\1\s\build_scripts\windows\artifacts\Python\Lib\site-packages\setuptools/msvc.py", line 316, in msvc14_get_vc_env
File "D:\a\1\s\build_scripts\windows\artifacts\Python\Lib\site-packages\setuptools/msvc.py", line 270, in _msvc14_get_vc_env
distutils.errors.DistutilsPlatformError: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
Testing support for clang
error: subprocess-exited-with-error
python setup.py egg_info did not run successfully.
exit code: 1

We are also seeing this error, started today.

Same, also seeing this error

route to CXP team

Similar to https://github.com/Azure/azure-cli-extensions/issues/4608, https://github.com/Azure/azure-cli-extensions/pull/4610
MSI's bundled Python has been updated to 3.10 (https://github.com/Azure/azure-cli/pull/21746). k8s-configuration and connectedk8s depend on pycryptodome==3.9.8 which doesn't release wheels for Python 3.10:
https://pypi.org/project/pycryptodome/3.9.8/#files

The latest pycryptodome 3.14.1 works with Python 3.10 using ABIs:
https://pypi.org/project/pycryptodome/#files

While we will bump the pycryptodome dependency in k8s-configuration and connectedk8s, you may install Visual Studio so that pycryptodome can compile on Windows.

connectedk8s has been fixed by https://github.com/Azure/azure-cli-extensions/pull/4607 and released by https://github.com/Azure/azure-cli-extensions/pull/4647.

@ponatara Could you please check the above comment and let us know if you have any further questions on this ? Awaiting your reply.


sap.ui.layout.form.SimpleForm: contains duplicate "form" roles leading to same place

OpenUI5 version:latest
Browser/version (+device/version): All
Any other tested browsers/devices(OK/FAIL):
URL (minimal example if possible):https://openui5.hana.ondemand.com/explored.html#/sample/sap.ui.layout.sample.SimpleForm354/preview
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

Bring up the Form sample and examine the dom
Navigate down to the "__xmlview4--SimpleFormDisplay354--Form" div and note that it has role="form"
Navigate further down to the "__container1--Grid" div and note that it also has role="form"

What is the expected result? Expected only one "form" landmark/role
What happens instead? There are two nested "form" landmarks in the form. They are both directing to the same location as far as the user is concerned. Having multiple identical roles creates confusion for the impaired user.
Any other information? (attach screenshot if possible)

Hi and thanks for your message.
I have passed the issue to the colleague responsible for the control.
Tracked internally as<PHONE_NUMBER>
Kind regards,
Christiane


Use the same default Owner ID between EC2 models

When attempting to use Moto to do something similar to this:
@mock_ec2
def test_copy_image_example():
  conn = boto3.client("ec2")
  copy_resp = conn.copy_image(SourceImageId="one of the default ami ids from resources")
  describe_resp = conn.describe_images(Owners=["self"])
  describe_resp["Images"].should.have.length_of(1)

The test ended up failing, and describe_images responded with an empty list.
To me, the expected result would be that any image created with copy_image would be given the same OwnerId as images found when using Owners=["self"].
The first commit does just that, by creating a common OWNER_ID used by all the ec2 models.
The second commit adds a test case for the above example code to ensure future changes don't accidentally result in copy_image using a different owner id than the one used for self.
There may be additional test cases that could be useful to ensure the owner ids are the same between calls using the same resource types, and I would be more than glad to write those tests, but I'm not sure if they'd be helpful, or even what would need to be tested.

Coverage increased (+0.0002%) to 93.106% when pulling abf3db8d8a4287946a7dc0c5dfaa233139e3e71f on dkuntz2:unify-ec2-owner-ids into d82423788cf372134a9c7d055c8293b281e61803 on spulec:master.

Looks great; thanks!


Using document AI in PHP and getting json result

Hi,
Is there a way to use document AI in PHP, with the examples from google : https://cloud.google.com/document-ai/docs/process-documents-client-libraries#client-libraries-install-php
But getting json extract after precessing requesy, instead of a almost raw text?
Do you have piece of example code?
Thanks in advance to your

Hi @jhb2 I see the expected result is a JSON. We may need take the text from the JSON and process it.
Refer the page https://cloud.google.com/document-ai/docs/output to see the output expected.
Since it is expected result, I'm closing this issue. Thank you for asking.


Generated HTML offline page is not clickable

The title says it all, when you're inside of Structurizr Lite, you can click throughout the diagrams, but when you generate the offline HTML page, you don't have this option.
The mouse shows a (+) Zoom icon, however double clicking doesn't do anything.

I've just tried this with the latest version of Structurizr Lite, and the export is clickable for me.

@simonbrowndotje thanks for the reply. I'll try once more then and provide a reproducer if that's still the case.


bump to 4.3.1

I just had this idea was and checking release :D

haven't had a chance to e2e this which is why it hasn't merged yet

Just curious about something.
Are we gonna allow customers to choose specific versions during creation, and support old one ?
In term of E2e, I guess this mean we should plan to run tests against multiple versions (4.3.0 and 4.3.1 fo example)?

Not on day 1.  Later we may allow users to choose to deploy 4.3 or 4.4 or 4.5, but I'm not in a hurry to allow them to choose 4.3.x or 4.3.y.
Simple rationale: we'll be upgrading clusters to 4.3."latest" for security patching, as soon as any security patch comes out it invalidates the safety of all previous versions in that stream and would be immediately upgraded anyway.

/lgtm


LanguageMode - Shell Script: Syntax highlighting bugged

From the About popup:

Version: 1.42.1
Commit: c47d83b293181d9be64f27ff093689e8e7aed054
Date: 2020-02-11T14:50:36.977Z
Electron: 6.1.6
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Linux x64 5.3.0-40-generic

Steps to Reproduce:
# Set logfile
# shellcheck disable=SC2034
logfile="$logDir/${0##*/}.log"

Is being rendered like:

There are more highlighting problems but this is the one that caught my attention at moment...

Does this issue occur when all extensions are disabled?: No idea, did not try

Looks like it's one of the extension you have installed:

@alexr00  Looks like it's one of the extension you have installed:

Hmm weird, okay i will investigate further in that case...

Ok first try seems to have found the cause.
I disabled aaron-bond.better-comments and seems the highlight bug is gone, will try to file there...
:+1:


python3Packages.rasterio: don't propagate setuptools

Description of changes
setuptools should't be propagated via propagatedBuildInputs.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 302154 run on x86_64-linux 1

4 packages failed to build:

python312Packages.osmnx
python312Packages.osmnx.dist
python312Packages.rasterio
python312Packages.rasterio.dist

4 packages built:

python311Packages.osmnx
python311Packages.osmnx.dist
python311Packages.rasterio
python311Packages.rasterio.dist

I will address failure of python312Packages.rasterio in this PR as well.
python312Packages.rasterio is failing on LooseVersion which looks unused (and this import statement is not present in rasterio master branch). I'll try to remove it in postPatch phase.
rasterio> /nix/store/vsavc6ipf8jxw1agzgp4pj329ayq287z-python3.12-rasterio-1.3.9/lib/python3.12/site-packages/rasterio/rio/calc.py:7: in <module>
rasterio>     from distutils.version import LooseVersion
rasterio> E   ModuleNotFoundError: No module named 'distutils'

Result of nixpkgs-review pr 302154 run on x86_64-linux 1

8 packages built:

python311Packages.osmnx
python311Packages.osmnx.dist
python311Packages.rasterio
python311Packages.rasterio.dist
python312Packages.osmnx
python312Packages.osmnx.dist
python312Packages.rasterio
python312Packages.rasterio.dist

Result of nixpkgs-review pr 302154 run on x86_64-linux 1

8 packages built:

python311Packages.osmnx
python311Packages.osmnx.dist
python311Packages.rasterio
python311Packages.rasterio.dist
python312Packages.osmnx
python312Packages.osmnx.dist
python312Packages.rasterio
python312Packages.rasterio.dist

@l0b0 , all tests are passing now.


FixedBatchNormalization issue

Hi guys, I run "test_frcnn.py"  and I got this problem:"TypeError: int() argument must be a string, a bytes-like object or a number, not 'NoneType"
Could anyone help me? I think there is some problem in FixedBatchNormalization function
File "test_frcnn.py", line 122, in
shared_layers = nn.nn_base(img_input, trainable=True)
File "C:\Users\88693\Desktop\try\frcnn\keras-frcnn\keras_frcnn\resnet.py", line 188, in nn_base
x = FixedBatchNormalization(axis=bn_axis, name='bn_conv1')(x)
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\engine\base_layer.py", line 463, in call
self.build(unpack_singleton(input_shapes))
File "C:\Users\88693\Desktop\try\frcnn\keras-frcnn\keras_frcnn\FixedBatchNormalization.py", line 30, in build
trainable=False)
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\engine\base_layer.py", line 279, in add_weight
weight = K.variable(initializer(shape, dtype=dtype),
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\initializers.py", line 46, in call
return K.constant(1, shape=shape, dtype=dtype)
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\keras\backend\tensorflow_backend.py", line 649, in constant
value, dtype=dtype, shape=shape, name=name)
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\keras\backend.py", line 905, in constant
return constant_op.constant(value, dtype=dtype, shape=shape, name=name)
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\constant_op.py", line 246, in constant
allow_broadcast=True)
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\constant_op.py", line 284, in constant
impl
allow_broadcast=allow_broadcast))
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\tensor_util.py", line 463, in make_tenso
r_proto
if shape is not None and np.prod(shape, dtype=np.int64) == 0:
File "<array_function internals>", line 6, in prod
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\numpy\core\fromnumeric.py", line 2962, in prod
keepdims=keepdims, initial=initial, where=where)
File "C:\Users\88693\AppData\Local\Programs\Python\Python37\lib\site-packages\numpy\core\fromnumeric.py", line 90, in _wrapreduction
return ufunc.reduce(obj, axis, dtype, out, **passkwargs)
TypeError: int() argument must be a string, a bytes-like object or a number, not 'NoneType'

Hello,
Please I ve got the same problem ... did you find a solution??

Hi Guys,
I have the same problem, I have the latest versions of all the packages but still can't train it using either resnet or vgg.
Traceback (most recent call last):
File "train_frcnn.py", line 125, in
shared_layers = nn.nn_base(img_input, trainable=True)
File "....\Object detection\keras-frcnn\keras_frcnn\resnet.py", line 179, in nn_base
x = FixedBatchNormalization(axis=bn_axis, name='bn_conv1')(x)
File ".....\Python 3.7\lib\site-packages\tensorflow\python\keras\engine\base_layer.py", line 897, in call
self._maybe_build(inputs)
File "......\Python 3.7\lib\site-packages\tensorflow\python\keras\engine\base_layer.py", line 2416, in _maybe_build
self.build(input_shapes)  # pylint:disable=not-callable
File "......\Object detection\keras-frcnn\keras_frcnn\FixedBatchNormalization.py", line 32, in build
trainable=self.trainable)
TypeError: add_weight() got multiple values for argument 'name'
It needs to be fixed.

You have to work with these versions :
tensorflow==1.8.0
keras==2.2.0

Got it..
Thanks @BouchraHonnit .

For TensorFlow 2.3, how to solve the issue "TypeError: add_weight() got multiple values for argument 'name'"

@mikechen66 I think you should verify the versions of keras and Tensroflow, make sure that they are as follows : tensorflow==1.8.0
keras==2.2.0
maybe it will work ... good luck


Allow MPI command output to be verbose for OpenMPI

Description
Currently, the --quiet flag is added to the mpiexec command in the khiops-env script on Linux platforms, which utilize OpenMPI for native installations.
The goal of this issue is to add an environment variable KHIOPS_MPI_VERBOSE, which is set to "false" by default and, if set to "true", would remove the --quiet flag.
Thusly, MPI-related errors could be checked on demand.

I'd call it KHIOPS_MPI_QUIENT because there is an --verbose option in OpenMPI

I'd call it KHIOPS_MPI_QUIET because there is an --verbose option in OpenMPI in addition to --quiet.

One argument for the current KHIOPS_MPI_VERBOSE would be that, depending on the MPI back-end, this verbosity might mean different things:

for OpenMPI, which is the default back-end for native installations on Linux (but can be overridden via the KHIOPS_MPI_COMMAND and KHIOPS_MPI_LIB environment variables) we define this as being a switch on the --quiet flag (and this might change in the future - perhaps by combining it with the --verbose flag ? @bruno-at-orange ?)
for MPICH (possible alternative back-end on some platforms - e.g. khiops-core on Conda), this does nothing currently, but could do in the future,
etc.

The first proposal is fine: KHIOPS_MPI_VERBOSE which is set to "false" by default

What happens if it is not defined in the environment? It acts as a if it were false ?
What happens if it is defined values other than true and false ?

What happens if it is not defined in the environment? It acts as a if it were false ?
I would say "yes".

What happens if it is defined values other than true and false ?
I would say anything != true acts like false.

Ok

My understading is that in khiops-env the KHIOPS_MPI_VERBOSE variable is set to "false" by default (that is, if it is not already defined in the environment). If this variable's value is different from "true" (which includes the default "false" or whatever the user has set), then --quiet is added to mpiexec. If this variable's value equals "true", then the --quiet flag is not added to mpiexec.
Is this also your understanding @bruno-at-orange ?


Fixes in source code

Fix mistypes
Change mechanism of processing touches (from touchBegan, touchEnd etc. to gestureRecognizers)

Your PR is great.
Could you please explain why you change touchBegan to gestureRecognizers


Rendering of configuration is broken after loading a program

Steps to reproduce the behavior:

Go to Robotino
Create a program like:

Save it with e. g. "test"
Go to "my programs" and double click on it, to reload it
See error in program

go to the configuration tab and see error there:

expected configuration, like in the beginning:

not browser depend!

works


chore: update code of conduct to version 2.0

Description
Update Code of Conduct to version 2.0
Fixes #2
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up to date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

Additional information

To request a review, check out who's involved to see a list of contributors and committers
Use the mailing list if you are unsure, who to contact, or if want to engage in a general discussion
Check out our guide on how to contribute
Check out our Release Guidelines
Check out the Eclipse Foundation contribution guidelines

Looks good to me. Can I merge it?

Sure :)


anki not appearing in list of apps to allow through firewall

I've used this addon for a few different computers but on the newest computer, anki doesn't show up as an option of a program to manually allow through the firewall for private and public. I'm not sure if I'm missing some obvious way to make it show up. Thanks!

Hmm can't say I'm sure about that. Is Ankimote not working because of the firewall?

yes it's doing the thing where the page on the remote phone just never loads


cannot import name 'feature_selection' from 'selection'

Not able to resolve this issue

Hi @MonikaLamba , thanks for the details. It looks like this issue is related to your OS. Unfortunately, i haven't tested the pipeline on Windows but i'm currently preparing Google Colaboratory notebook for this pipeline,  so as a result you will be able to run it no matter your operating system.

Hi @MonikaLamba ,
I have added Google Colab you can use it to run the pipeline.
https://github.com/tlatkowski/deep-learning-gene-expression/blob/master/colab/deep_learning_feature_selection.ipynb

Thank you so much sir


.cancelable(true) doesn't seem to work

What are you expecting and what is happening?

I have read the description and according to that I'm expecting .cancelable(true) should allow user to click outside the outerCircle to dismiss the overlay. That is not happening.
Further, I request you may .cancelable(true) dismiss the overlay regardless of where the user has clicked. I believe that would make dismissing overlays friction-less.

That's odd. Can you share the code you're using to show the tap target and the version of the library you're using?
In regards to allowing the user to tap anywhere to dismiss: the material design guidelines specifically state that the user must tap outside the outer circle to cancel/dismiss.

Code:
new TapTargetView.Builder(HomeActivity.this)
                .title("title")
                .description("")
                .outerCircleColor(R.color.highlight_white)
                .targetCircleColor(R.color.primary)
                .textColor(R.color.text_primary)
                .dimColor(R.color.primary)
                .tintTarget(false)
                .drawShadow(false)
                .cancelable(true) // Whether tapping outside the outer circle dismisses the view
                .showFor(textView);

Lib version:
 compile 'com.getkeepsafe.taptargetview:taptargetview:1.0.0'
Oh, I didn't know these guidelines yet. Thanks for the info :)

Ah. Please update to version 1.0.1 that should fix your issue

Thanks! Closing this issue 👍


Incompatibility with Canary Mod

crashes no startup and notable line from the log is:
[pool-5-thread-1/WARN] [mixin/]: @Redirect conflict. Skipping canary.mixins.json:alloc.nbt.CompoundTagMixin from mod canary->@Redirect::removeOldMapAlloc()Ljava/util/HashMap; with priority 1000, already redirected by c2me-opts-allocs.mixins.json:MixinNbtCompound from mod c2me_opts_allocs->@Redirect::redirectNewHashMap()Ljava/util/HashMap; with priority 1100
log: https://pastebin.com/hBQdfA2H

you have a lot of mod try reproduce the problem in a clean instance

ok use radium instead (I try with radium and it work fine)


add light saber icon

Assign me this with Hacktoberfest tag
The icon I want is:

Name:
URL:
Link to icon:

Vector version of the logo:
Official style guide or brand guide:

Which web service uses a lightsabre icon?


Questions about pre-trained model

Hi,
Thanks for the great work!
I'm trying to directly load the pretrained model you provide "../datasets/R2R/trained_models/vitbase-6tasks-pretrain/model_step_130000.pt" and then fine-tune on r2r task. However, I encounter the problem:
 RuntimeError: Error(s) in loading state_dict for NavCMT:
size mismatch for bert.hist_embeddings.position_embeddings.weight: copying a param with shape torch.Size([50, 768]) from checkpoint, the shape in current model is torch.Size([100, 768]).

I also followed your provided scripts to pre-train the model (w/o e2e vit), and then fine-tune on r2r, which seems to work fine.
Just hope to check whether the provided pre-trained model (w/o e2e vit) is the correct one, and if I need to modify any code/hyperparameter to load the pre-trained model?
Thanks so much!

Hi, please change this line according to the param size of the pretrained model.


"the Help-Wanted label" hyperlink issue in CONTRIBUTING.md

The document CONTRIBUTING.md is needed to be corrected.
On clicking "the Help-Wanted label" hyperlink, a page with filters is:open label:Help-Wanted is opened which shows "No results matched your search" (as shown in the picture attached).

I am planning to create a pull request to fix this issue.

Looks like you fixed this in #5967!

@DHowett Thanks!!


Compiling plugins

Hi
Isn't the necessity of compiling the plugins first missing from the Build Instructions?
Any chance you could provide a walkthru for C++ novices using VisualStudio?

Yes, there should be at least a link to Visual Studio or something... I will add that!
After you have visual studio installed, Unreal will  automatically compile the plugin the first time you open the project. There is no interaction with Visual Studio needed to use the plugin!

Thanks! Wasn't aware of that. Thought compilation had to happen first in VS.


VSTestPlusPlus with UI

Hi @RanjanarMS, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Ranjana Rathinam). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

    "Patch": 11

I am assuming all the change in master before it is pushed we will have bumped up the patch version. else next sprint deployment wont contains these changes.

Refers to: Tasks/VsTest/task.json:20 in ea1559f. [](commit_id = ea1559f8db6d17c3e3e2e8d3fbca114777a30931, deletion_comment = False)

:shipit:


jetbrains: build from source code with jbr-no-jcef-17

Description of changes
Add JBR-17 and use it to build community versions of JetBrains IDEs.
Fixes issues introduced in #318036.
Update: I have checked this on #316185, both the two IDEs were successfully built and able to run.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@leona-ya I am considering reopening #320161 and adding this commit if everything works. Is it okay?

I can see how removing the duplication would make things more complex, so the trade-off seems fair.

I also think we can (and maybe should?) remove jdk 17 in unstable when 2024.2 is releases as – to my understanding – it's fully based on jdk 21.

@leona-ya I am considering reopening https://github.com/NixOS/nixpkgs/pull/320161 and adding this commit if everything works. Is it okay?

I think yes. I would wait for a few days tho to see if other regressions occur.

I think yes. I would wait for a few days tho to see if other regressions occur.

For sure. I will wait a few days or weeks to reopen it.


Create max-01.toml

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

This validator was not recognised as a previous validator and has update in the title. Please read the description.

@bengtlofgren, I dont see where I added "Update" in the title.


replaceBootstrap transformer fails when bootstrapModule() call is not chained with platformBrowserDynamic() call

Versions
Angular CLI: 1.5.4 (e)
Node: 6.11.3
OS: darwin x64
Angular: 5.1.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.5.4
@angular-devkit/build-optimizer: 0.0.35
@angular-devkit/core: 0.0.22
@angular-devkit/schematics: 0.0.41
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.4
@schematics/angular: 0.1.10
@schematics/schematics: 0.0.10
typescript: 2.4.2
webpack-concat-plugin: 1.4.0
webpack-dev-server: 2.9.7
webpack: 3.8.1

Repro steps

Create a new project with: ng new project-name
Run using: ng serve --prod
Open browser and it runs as expected
Change main.ts to split the call to platformBrowserDynamic() from the call to bootstrapModule()

const platform = platformBrowserDynamic();
platform.bootstrapModule(AppModule)
  .catch(err => console.log(err));

Run: ng serve --prod
Open browser and see error in console:
Error: No NgModule metadata found for 'function (){}'.

Observed behavior
Complete stack trace:
main.9d791305398abba6bf45.bundle.js:1 Uncaught Error: No NgModule metadata found for 'function (){}'.
    at e.resolve (main.9d791305398abba6bf45.bundle.js:1)
    at e.getNgModuleMetadata (main.9d791305398abba6bf45.bundle.js:1)
    at e._loadModules (main.9d791305398abba6bf45.bundle.js:1)
    at e._compileModuleAndComponents (main.9d791305398abba6bf45.bundle.js:1)
    at e.compileModuleAsync (main.9d791305398abba6bf45.bundle.js:1)
    at e.compileModuleAsync (main.9d791305398abba6bf45.bundle.js:1)
    at e.LMZF.e.bootstrapModule (main.9d791305398abba6bf45.bundle.js:1)
    at Object.cDNt (main.9d791305398abba6bf45.bundle.js:1)
    at n (inline.687454df27f852918dab.bundle.js:1)
    at Object.0 (main.9d791305398abba6bf45.bundle.js:1)

Desired behavior
It should work identically as when the calls are chained.
Mention any other details that might be useful (optional)
The reason for making this change to split into two lines is that in my case, in the handler for the promise returned from bootstrapModule() I need to do some initialization and I need access to the platfor. Calling platformBrowserDynamic() seems to return a different platform if you call it a second time so I need the one used to call bootstrapModule on:
// this is unique to my portal environment
declare var wrapper: any;

const platform = platformBrowserDynamic();
platform.bootstrapModule(AppModule)
  .then((ref: NgModuleRef<AppModule>) => {
    // setup required for my portal environment
    wrapper.setApplicationComponentNode(ref, platform);
  })
  .catch(err => console.log(err));

The only workaround at the moment for this is to create a separate main-prod.ts file, eject the webpack config and then modify it to use a different main file for prod vs. dev build.

I found a workaround for this by using:
platformBrowserDynamic().bootstrapModule(AppModule)
  .then((ref: NgModuleRef<AppModule>) => {
    // setup required for my portal environment
    const platform: PlatformRef = ref.injector.get(PlatformRef);
    wrapper.setApplicationComponentNode(ref, platform);
  })
  .catch(err => console.log(err));

However, a better error message or a comment in the generated main.ts would at least be an improvement.

I have the same issue when trying to fork my code in main.ts to an UnderConstructionModule when building or serving with --prod modifier:
**Error: No NgModule metadata found for 'function (){}**
// main.ts
import { enableProdMode } from '@angular/core';
import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';

import { AppModule } from './app/app.module';
import { UnderConstructionModule } from './under-construction/under-construction.module';
import { environment } from './environments/environment';

if (environment.production) {
 enableProdMode();
}

if (environment.production) {

 platformBrowserDynamic().bootstrapModule( UnderConstructionModule )
   .catch( err => console.log( err ) );

} else {

 platformBrowserDynamic().bootstrapModule( AppModule )
   .catch( err => console.log( err ) );

}

Without --prod modifier, ng serve and ng build work ok with both modules.
Versions
Angular CLI: 1.7.4
Node: 6.14.1
npm: 3.10.10
OS: win32 x64
Angular: 5.2.9
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
@ angular/cli: 1.7.4
@ angular-devkit/build-optimizer: 0.3.2
@ angular-devkit/core: 0.3.2
@ angular-devkit/schematics: 0.3.2
@ ngtools/json-schema: 1.2.0
@ ngtools/webpack: 1.10.2
@ schematics/angular: 0.3.2
@ schematics/package-update: 0.3.2
typescript: 2.5.3
webpack: 3.11.0

Hello,
I am facing the same issue of mcgraphix with:
Versions
`Angular CLI: 6.1.1
Node: 8.11.3
OS: win32 x64
Angular: 6.1.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.7.1
@angular-devkit/build-angular     0.7.1
@angular-devkit/build-optimizer   0.7.1
@angular-devkit/build-webpack     0.7.1
@angular-devkit/core              0.7.1
@angular-devkit/schematics        0.7.1
@angular/cli                      6.1.1
@ngtools/webpack                  6.1.1
@schematics/angular               0.7.1
@schematics/update                0.7.1
rxjs                              6.2.2
typescript                        2.7.2
webpack                           4.9.2`
Do you know if you will work on this limitation and when ?
Thank you very much.

As @JavierFuentes  and @guisaas mentioned there is an issue when use fork in the main.ts. It affects I believe all environments with production=true flag. Furthermore if you run ng serve or ng build with --configuration={evnironmentValue}, application will fail with "No NgModule metadata found for..."
I've played with configs a bit and discovered that 'aot' and 'buildOptimizer' must be disabled at angular.json configuration section to make application work.
I expect this behavior must work with aot=true as well.
Does anyone know workaround for now?
Thanks!

I've used workaround with angular.json -> configurations -> fileReplacements section. Replace your main.ts with another one that contains the second module. Works for me.

Any updates on this? I have a related issue, I'd like to create the custom platform:
export const myPlatform = createPlatformFactory(platformBrowserDynamic, 'myPlatform');
myPlatform().bootstrapModule(AppModule)

And get the same error, only with AOT:
Unhandled Promise rejection: No NgModule metadata found for 'function(e){}'

Chaining it directly doesn't help:
createPlatformFactory(platformBrowserDynamic, 'myPlatform')
.bootstrapModule(AppModule);

Produces the same error.
Any help appreciated

Heya, does using getPlatform() as described in https://github.com/angular/angular-cli/issues/8265#issuecomment-449610872 work for your usecase?
We try to identify the platformBrowserDynamic().bootstrapModule(AppModule); call to replace it properly but don't support variations.

@filipesilva  Hi! It's not related to my usecase.  In my case, I want to create a custom platform based on platformBrowserDynamic, and then bootstrap an app with it.
Both this
export const myPlatform = createPlatformFactory(platformBrowserDynamic, 'myPlatform');
myPlatform().bootstrapModule(AppModule)

and this
(createPlatformFactory(platformBrowserDynamic, 'myPlatform'))().bootstrapModule(AppModule)

leads to AOT error that I described


3rdpartylicenses.txt is empty

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/
@angular/cli: 1.2.0
node: 8.1.3
os: linux x64
@angular/animations: 4.2.5
@angular/common: 4.2.5
@angular/compiler: 4.2.5
@angular/core: 4.2.5
@angular/forms: 4.2.5
@angular/http: 4.2.5
@angular/platform-browser: 4.2.5
@angular/platform-browser-dynamic: 4.2.5
@angular/router: 4.2.5
@angular/cli: 1.2.0
@angular/compiler-cli: 4.2.5
@angular/language-service: 4.2.5

Repro steps.
ng new myproject
cd myproject
ng build --target=production
The file dist/3rdpartylicenses.txt will be empty.
The log given by the failure.
No log
Desired functionality.
3rdpartylicenses.txt should contain the licenses of all modules loaded from node_modules
Mention any other details that might be useful.

I am the author of the webpack plugin that outputs the licenses. I have already pushed a fix in version 0.5.1 of license-webpack-plugin. Upgrading the version of license-webpack-plugin should resolve the issue.
https://github.com/xz64/license-webpack-plugin/issues/12

Thank you xz64. I was just affected by this issue.
Any workaround for us "users" you can recommend except updating the package.json of @angular/cli manually to use 0.5.1 of your license-webpack-plugin?

Maybe the supplied pattern could be changed as well:
pattern: /^.*$/
So not only MIT,ISC and BSD licensed files are included, but, e.g., Apache as well

@mariuszBX I can't think of any workarounds for this. I submitted a pull request #7064 to upgrade license-webpack-plugin.

is there any way to avoid 3rdpartylicenses.txt generation while running below command:

ng build --prod window --aot --source-map=false --vendor-chunk=true --output-hashing none