Flakey underline/hover tooltip behavior

Testing #23129
Not sure how else to describe it. Observations below:
Repro 1

echo http://github.com from terminal
try hovering over output text
tooltip appears, but no underline

Repro 2
1-3: Repro 1
4. hold ctrl
5. try hovering over output text
6. underline and tooltip appear, but clicking does nothing (because cmd is bound, not ctrl)
Repro 3
1-3: Repro 1
4. hold cmd
5. try hovering over output text
6. tooltip appears, but no underline, clicking works
Repro 4
1-3: Repro 1
4. switch to another application
5. switch back to VS Code
6. try hovering over output text
7. both tooltip and underline appears
Repro 5
1-3: Repro 1
4. echo https://github.com
5. try hovering over second output text
6. tooltip appears, but no underline
7. try hovering over first output text
8. neither underline nor tooltip appear
Repro 6
1-8: Repro 5
9. switch over to another application
10. switch back to VS Code
11. try hovering over first output text
12. underline appears, but no hover
13. try hovering over second output text
14. both hover and underline properly appear
--

VSCode Version: Code - Insiders 1.11.0-insider (a35c7da7b8f324c1e8cc7830173418e392542f6d, 2017-03-28T12:40:24.966Z)
OS Version: Darwin x64 16.4.0
Extensions: none

Repro 1: As designed
Repro 2: Fixed in https://github.com/Microsoft/vscode/commit/55732e69e5861bac06e34c0101b1881028f2edf6
Repro 3: Fixed in https://github.com/Microsoft/vscode/commit/55732e69e5861bac06e34c0101b1881028f2edf6
Repro 4: This happened because cmd was held when leaving VS Code, I don't have a good idea for a fix now. Moved to https://github.com/Microsoft/vscode/issues/23621
Repro 5: Duplicate https://github.com/Microsoft/vscode/issues/23484
Repro 6: Combination of repro 4 and 5

I verified on the Mac that Cmd over the link shows the link and a click opens it.
On Linux I still see strangeness. I first have to click on same line as the link before a link appears. Not sure if this has to do with my Linux in a VM setup.

Trying to verify on Linux (bare-metal):
It mostly appears to work, except for a quirk that I think is due to how this is presumably implemented. I have the impression that the link highlighting code listens on keydown/keyup for CtrlKey instead of using mouseEvent.ctrlKey.
i.e. Here I press Ctrl+Alt+Tab. Once the focus goes into the other window, I release all keys. Notice how the links in the editor do not get highlighted in blue, while the links in the terminal get the underline. I think that the link implementation in the terminal must be trying to keep track of keydown/keyup of the CtrlKey. Maybe I'm wrong, I don't know:

@alexandrudima you are right :smile: I was planning on reaching out to you to see how to handle it in the editor. So you just release on window blur? https://github.com/Microsoft/vscode/issues/23621

@Tyriar :+1: mouseEvent.ctrlKey is easier to implement and appears to always be correct. Presumably Chromium queries the modifier state on each mouse move.


Make content updates for project UI screens

Closes: https://issues.redhat.com/browse/RHOAIENG-4978
Closes: RHOAIENG-4846
Description

Updates to project UI screens were required per a content updates Google doc linked in the ticket above
How Has This Been Tested?

Ran unit and cypress tests locally, also visually when running the dashboard locally
Test Impact

N/A
Request review criteria:

Self checklist (all need to be checked):

[ ] The developer has manually tested the changes and verified that the changes work
[ ] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/ok-to-test

@simrandhaliw just FYI in case you would like to check the content updates

Tagging @jgiardino in this PR in case she'd like to review the content updates too.

Everything seems to be good. I think the updated logic for the pipelines card looks good but I can’t really confirm without seeing the UI. :wink:
Thank you for making these updates!!!


Enhanceenhance jdbc case to adapt to use case concurrency

Proposed changes
Issue Number: close #xxx

enhance jdbc case to adapt to use case concurrency
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


keyvault: : --include-pending requires true or false

Fix #11121: When using az keyvault certificate list, passing --include-pending now doesn't require a value of true or false, similar to --enable-soft-delete.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

Good idea. Tests added.


How to get the page after a submit

A part of the HTML:
<form class="login-form" method="post">

		<input class="mdl-textfield__input" type="text" id="username" name="username" value="">
		Username *

		<input class="mdl-textfield__input" type="password" id="password" autocomplete="new-password" name="password" value="">
		Password *

	<input type="submit" value="Login" class="mdl-button mdl-js-button mdl-button--raised mdl-js-ripple-effect mdl-button--primary" data-upgraded=",MaterialButton,MaterialRipple">
</form>

I have this code:
	AgoutiPage.FindByID("username").Fill(name)
	AgoutiPage.FindByID("password").Fill(password)
	AgoutiPage.FindByClass("login-form").Submit()

And now I want to check stuff on the page that the webserver sends me to.
But I can't find in the docs, how to do that.
When I dump the AgoutiPage.HTML() it gives me a kind of blank page

But that is not the correct page (pretty sure of it)
After further investigation:
	form := AgoutiPage.FindByClass(formClass)
	formCount, err := form.Count()
	if err != nil {
		ShowError("Login form '" + formClass + "' was not found in page\n" + err.Error())
	}
	if formCount == 0 {
		ShowError("Login form [0] '" + formClass + "' was not found in page")
	}

it seems that it throws the error and formcount = 0
The error is:
failed to select elements from selection 'Class: login-form [single]': element not found
But.... in the HTML there is the class as the example shows...
I also tried:
	test2 := AgoutiPage.FindByXPath("//*[contains(@class, 'login-form')]")

but it has the same result: it can't find the class :-(

Somehow, it dind't start on the correct start-page to begin with....
so all the above was invalid at once, after I found out.


The program cannot run under multi-threading

When I set export OMP_NUM_THREADS=1, the bfs can run successfully. But when I set export OMP_NUM_THREADS=2, it has a deadlock in the function MakeGraph().

Thank you for your feedback. Could you please specify which dataset you tried and how you preprocessed the dataset?


add tests and documentation for new pdf reporter

[x] Unit tests should be added here. Draft feature documentation is here
[x] Further, this feature should be added to the tutorial 2A integration test here. Draft tutorial documentation is here

documentation should be edited if necessary.

fixed w/ PR 62


Stability Test Improvement Ideas

The following Stability Test improvements were discussed between Matt and Wesley on 4/18.

Execution-id: Include the execution name in the execution id
Execution-id date: ISO date should be readable in the execution name
Have an option to run individual test suites and cases
Report metrics have each day reporting from firelens datajet telling whether the test is alive
Run the stability tests metric crunch periodically and compute the Stability Test metrics on failure rate per testcase. Add alarms when failure rate goes above a certain number.

the technique here might be an interesting way for FLB to report that its alive, have it cat and EMF metric file with an exec input and send that: https://github.com/aws-samples/amazon-ecs-firelens-examples/issues/111

Also we discussed:

test output should have two concepts: info needed to check results and audit log. Your audit log is all the config and the task ID and the cluster name and everything- that should go to S3. I think the test launcher should also output the execution ID and name and may be region and S3 bucket links to a file which we can commit to the repo as a record. Then this output is also used by the user to check the results later/create a doc on them.

Execute order 66 🚨


add sparse checkout to git

Changes
Hi,
I have added functionality to support sparse-checkout for git
This works by enabling the feature in the git config and then updating the sparse-checkout file before fetch.
The feature request is here
I have also added a test to check that .git/info/sparse-checkout is updated as expected
Thanks!
/kind feature

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

/ok-to-test

/test pull-tekton-pipeline-integration-tests

/test pull-tekton-pipeline-integration-tests

/test pull-tekton-pipeline-integration-tests

/cc @sbwsg

/cc @sbwsg

@a-rothwell thanks for making those changes.  You'll need to squash your commits into 1 prior to merge.

@a-rothwell thanks for making those changes.  You'll need to squash your commits into 1 prior to merge.

/lgtm
Thanks 🙇🏽

/lgtm
Thanks 🙇🏽

it woudl be great if we can the git-clone task updated to support this feature too 🙏🏽 @a-rothwell

it woudl be great if we can the git-clone task updated to support this feature too 🙏🏽 @a-rothwell

Thank you both for helping me with this!
I will update the task in the catalog

Thank you both for helping me with this!
I will update the task in the catalog


Few instances of gitops mentioned instead of Kam

What type of PR is this?

/kind cleanup

What does this PR do / why we need it:
Removes references of Gitops and replaced with Kam
Have you updated the necessary documentation?
Yes
Which issue(s) this PR fixes:
https://issues.redhat.com/browse/GITOPS-290

Codecov Report

Merging #20 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   61.11%   61.11%
=======================================
  Files          51       51
  Lines        2466     2466
=======================================
  Hits         1507     1507
  Misses        827      827
  Partials      132      132

Flag
Coverage Δ

#unittests
61.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 946070d...438b208. Read the comment docs.


Power Supply Limitations

Hello everyone. I was hoping to get some information on any possible LED brightness limits before submitting my files. I know that last year some programs were too demanding for the power supply, which caused issues with the animations. I also know the @thatmattparker has upgraded the power supply this time around, so I don't know if this is an issue anymore.
For now, I have set my most demanding animation to have every LED always on with only one rgb color value of 100.

You already closed this issue, so I guess you found the answer, but in case anyone else finds this and has the same question, look here,


added unit test for multi node simulator sanity

Motivation
Closes #1692
Changes
Added sanity test for multi node simulator
Test Plan
TODO

bors merge

bors merge

bors merge

bors try

bors try

bors try

bors merge

bors merge

bors try

bors merge


feat(l1): implement implement forkchoiceupdatedv2, newpayloadv2 and getpayloadv2 as enum variants

Motivation
We need to support post merge V1, V2 and V3 for GetPayload, NewPayload and ForkChoiceUpdated functions
Description
Implemented V2 for those functions
Closes https://github.com/lambdaclass/ethrex/issues/892, https://github.com/lambdaclass/ethrex/issues/1184, https://github.com/lambdaclass/ethrex/issues/1186
Makes to pass about 19 new hive tests
This PR replaces https://github.com/lambdaclass/ethrex/pull/1297 as it is the same solution using composition instead of traits

Included in https://github.com/lambdaclass/ethrex/pull/1411


Illegal offset type

https://github.com/laravel/framework/blob/71265318358ee150fe8b13f68a03cb9f869fb483/src/Illuminate/Database/Eloquent/Relations/BelongsTo.php#L177
When the owner field has a cast, and the result is an Object - Illegal offset type exception is raising, I suggest to use casting to a string

@driesvints Why did you close this without any comment?


Possible minor bug

Thank you for this repo! It looks really good.
I was able to replicated:
https://github.com/pylandtemp/pylandtemp/blob/master/tutorials/Tutorial_1- load_landsat8_image_from_amazon_and_google_cloud.ipynb
In code block 6
# Creating and applying the mask to the images. mask = tempImage10 == 0 tempImage10[mask] = np.nan tempImage11[mask] = np.nan
Should the mask be set twice?
# Creating and applying the mask to the images. mask = tempImage10 == 0 tempImage10[mask] = np.nan mask = tempImage11 == 0 tempImage11[mask] = np.nan

Hello @npr99
Thanks for reaching out. Indeed, the masks for bands 10 and 11 should be set differently.
mask10 = tempImage10 == 0
mask11 = tempImage11 == 0

tempImage10[mask10] = np.nan
tempImage11[mask11] = np.nan

Because, in this case, I was trying to mask out pixels with zero values and my assumption was that they could be the same for both bands 10 and 11. Indeed, they might not be the same.
Thanks for picking that up. I should update that part in the next release.


WIP: Add the uninstall script for slack package

Slack package is the only package with test cases in openwhisk-catalog,
so patch adds the uninstall script only for this package.
Also, put slack test cases in the desired position.

I am happy to see it fails. I intended to make it happen.
All Travis CI must guard the gate of this repo.


QRCodes up to Type 40

The QR Code types and RS blocks were hardcoded for types up to 10, most methods support up to type 40.
http://www.qrcode.com/en/about/version.html
todo: remove hardcoding.


Rename pkg/workers/downloader to resourcedownloader

What would you like to be added:
pkg/workers/downloader is technically not the only worker that downloads information(pkg/workers/document also downloads pages). Also it is only used to download png, jpeg, gif, ... under the __resource folder so it makes more sense to be called resourcedownloader
How to categorize this?:

/kind enhancement
/priority 3

/assign RadaBDimitrova

Closed by https://github.com/gardener/docforge/pull/347


How do I know if the user chooses to disauthorize?

How do you know if the user opts to reject an authorization message when it pops up?

@axiref
Actually, I don't remember there is a choice that a user can choose to accept or reject.
Usually, the authorization process is in silent, without user's notice.
But indeed, there is no way I can know if the user does choose to reject. Actually, I get authorization permission from https://open.weixin.qq.com/connect/oauth2/authorize. When accepted, it redirects to my page, otherwise, I don't think it does anything to notify me there is a rejection, it may just simply close or something.


Fix structs merging error for chart-operator

module github.com/imdario/mergo v0.3.9 has an issue with merging go structs and it results with errors as below from chart-operator side.
panic: template: kong-app/templates/deployment.yaml:65:10: executing "kong-app/templates/deployment.yaml" at <include "kong.controller-container" .>: error calling include: template: kong-app/templates/_helpers.tpl:383:4: executing "kong.controller-container" at <include "kong.ingressController.env" .>: error calling include: template: kong-app/templates/_helpers.tpl:238:20: executing "kong.ingressController.env" at <mustMergeOverwrite $autoEnv $userEnv>: error calling mustMergeOverwrite: cannot append two different types (bool, string)

Even though we are using the 0.3.8 version on our side, cluster-runtime is using 0.3.9 so it was override.
go mod graph
...
<EMAIL_ADDRESS><EMAIL_ADDRESS>
Therefore we need another override from our go.mod.
Checklist

[x] Update changelog in CHANGELOG.md.

Wow good find, thanks!
This looks good but don't we need a replace rule? Something like this.
// Use latest mergo to fix problem merging Go structs.
replace github.com/imdario/mergo => github.com/imdario/mergo 0.3.11

@rossf7 Thanks for the suggestion. Although I applied this replace directive into helmclient, chart-operator always exclude github.com/imdario/mergo 0.3.11 from go module. This means we need another replace directive in any helmclient-applied repository. So I'm inclined to leave it just like this, WDYT?

Yes we'd also need a replace in chart-operator but I think it's better to be explicit about the override.
It's unlikely that go modules will downgrade to 0.3.9 with the bug but if it does there can be a regression. I'm fine with a comment if you think that's better than the replace.

Sure, There are not many repositories whom using helmclient directly, but I would like to left this as indirect to make dev process easier.


Review Documentation PRs

Please review #112 and #115 !

@hasyimibhar please note that both PRs are a bit out of date, so review should be more about style and general description. If you're fine with that I'll update both of them before merging.


Document two-factor authentication

When this is added, build out documentation for setting this up and what it means for workflows.

@Mikejo5000  can we update https://docs.microsoft.com/en-us/nuget/nuget-org/individual-accounts to talk about 2FA?

Sure, I will plan to add that.

Thanks @Mikejo5000 !

@karann-msft Sure, let me know if you see anything that needs to be changed. Thx!


Wrong dependency version derivation

pip-compile fails to derive dependency version if different versions of package C are required by different versions of package B with an older version of B derived in the first round.
Here is the example:
First round:
Latest version of package A==2 requires old package B==1
Latest version of package B requires C==2
Second round:
B==1 requires C==1
Third round
C==1,2 fails to resolve.
I simplified things, but here is actual output if you are interested:
$ pip-compile -vvv --output-file requirements.txt requirements.txt.in -rU | grep -e django-ncbi-auth -e ncbi-myncbi-account -e django-ncbi-admin
  django-ncbi-admin
  django-ncbi-auth
  found candidate django-ncbi-admin==2.0.3 (constraint was <any>)
  found candidate django-ncbi-auth==5.1 (constraint was <any>)
  django-ncbi-admin==2.0.3 not in cache, need to check index
  django-ncbi-admin==2.0.3  requires django-ncbi-auth~=4.4.0, django<1.10.0,>=1.7.0
  django-ncbi-auth==5.1 not in cache, need to check index
  django-ncbi-auth==5.1     requires django<1.10,>=1.7, ipcalc~=1.99, ncbi-myncbi-account~=6.0
  adding django-ncbi-auth~=4.4.0
  adding ncbi-myncbi-account~=6.0
  django-ncbi-admin
  django-ncbi-auth~=4.4.0
  ncbi-myncbi-account~=6.0
  found candidate django-ncbi-admin==2.0.3 (constraint was <any>)
  found candidate django-ncbi-auth==4.4 (constraint was ~=4.4.0)
  found candidate ncbi-myncbi-account==6.0 (constraint was ~=6.0)
  django-ncbi-admin==2.0.3  requires django-ncbi-auth~=4.4.0, django<1.10.0,>=1.7.0
  django-ncbi-auth==4.4 not in cache, need to check index
  django-ncbi-auth==4.4     requires django<1.10,>=1.7, ipcalc~=1.99, ncbi-myncbi-account~=5.0
  ncbi-myncbi-account==6.0 not in cache, need to check index
  ncbi-myncbi-account==6.0  requires lxml~=3.4, ncbi-tx~=5.0, six
  adding ncbi-myncbi-account~=5.0
  django-ncbi-admin
  django-ncbi-auth~=4.4.0
  ncbi-myncbi-account~=5.0,~=6.0
  found candidate django-ncbi-admin==2.0.3 (constraint was <any>)
  found candidate django-ncbi-auth==4.4 (constraint was ~=4.4.0)
Could not find a version that matches ncbi-myncbi-account~=5.0,~=6.0
Tried: 1.0, 1.1, 1.1, 2.0, 2.0.1, 2.0.1, 2.0.1.post11+f5a85a6, 3.0, 3.0, 4.0, 4.0, 5.0, 5.0, 6.0, 6.0

Expected result
I expected, that the requirements of chosen B==1 on C will override requirements from B==2.
Actual result
But currently, they are combined resulting in impossible version requirements.

Can you verify if this is still the case in 1.8.1 ? I think I may have fixed your issue in #450. Thanks.

I confirm. Version 1.8.0 failed on my requirements.txt.in. But after upgrade to 1.8.1 pip-compile resolved dependencies correctly.
Nice work! Thanks!


API Design and Issues thread

It seems to me that "helpers" are usually some small utilities that complement the main functions, while in ClrMD the "helpers" build everything (especially the RuntimeBuilder which implements 10 I*Helpers).

I'm terrible at naming things.  =)
The original idea was that Builders actually construct Clr* types, where as Helpers are what various Clr* implementations use to implement its functionality.  That's not really how it's worked out in practice.
These kinds of naming issues are what I'm hoping to fix in "API Review" which I call out here:  https://github.com/microsoft/clrmd/issues/614.  Feel free to use this issue (or create new issues) for things that look weird/should be renamed in the surface area.  I want to do a full pass over the entire library on or before the date listed in 614.
(Though also I don't plan for IHelpers and IBuilders to be a "stable" API surface.  I haven't documented that fact yet, but those are implementation details that are there in case someone wants to mock the internals of ClrMD without having to construct an entire object model.  I don't plan to support a stable version of those APIs.)

Fix data readers: https://github.com/microsoft/clrmd/pull/630#issuecomment-619144883

Fix inconsistent "Minidump" vs "MiniDump" and "Coredump" vs "CoreDump".

Clarify that IDataReader.GetVersionInfo only works for native modules?

I've mostly finished API breaking changes/fixes.  I'm trying to scope my work down to things that need to be fixed before RTM, IE breaking changes.  The IBuilder/IHelper issue I'm leaving until a bit later since I'm not promising those interfaces will remain stable.  They are primarily meant for testing/mocking parts of the library.
Any feedback on API issues or changes within the Microsoft.Diagnostics.Runtime is greatly appreciated as I'm trying to ship 2.0 RTM on the 15th.

Clarify that IDataReader.GetVersionInfo only works for native modules?

I'm still debating what to do about this.  Having the version information for all modules (native or managed) is super useful for tools to have.  Especially when listing out modules to the user.  Removing ModuleInfo.Version or making it return bogus data for managed code makes me sad.
File layout issues with managed code make it very difficult to do anything sensible though.
I think IDataReader.GetVersionInfo is the last major API issue I have planned to work on unless you or other folks raise other API concerns.  I'm not sure what I'm going to do with it yet.

[ ] GetStaticFieldByName / GetInstanceFieldByName add nothing, they are simply wrappers around *Fields.FirstOrDefault.  Need to remove those unless we provide something significantly better than FirstOrDefault (like binary search, or hashset lookup, which I don't think we will do)

I propose to change ClrEnum.EnumerateValues to return IEnumerable<(string Name, ulong? Value)> instead. It's less common/useful to box integral types.

Consider adding the Public API Analyzer. It records the public API surface area in a per-project text file, making it easy to see when PRs modify public APIs.

I propose to change ClrEnum.EnumerateValues to return IEnumerable<(string Name, ulong? Value)> instead. It's less common/useful to box integral types.

That seems reasonable.  Feel free to submit a PR for that.

Consider adding the Public API Analyzer. It records the public API surface area in a per-project text file, making it easy to see when PRs modify public APIs.

Also looks reasonable, I'll take a look at that as soon as we ship.

At this point I think I'm happy with the surface area, at least enough to where I'm ok shipping 2.0.

Consider adding the Public API Analyzer. It records the public API surface area in a per-project text file, making it easy to see when PRs modify public APIs.

Implemented in #826.


Mounting components V1

I am trying to mount a few components as such
in my main.js file
 app = new Moon({
         root: "#app-moon",
         hooks: {
             init: function() {
                 _log('APP :: init');
             },
             mounted: function() {
                 _log('APP :: mounted');
             },
             updated: function() {
                 _log('APP :: updated');
             }
         }
     });

In an AJAX resonse
  var tc = Moon.extend("test-comp", {
        template: templates['test.html']
   });

tc.mount('#app-moon');

This only produces an error
TypeError: tc.mount is not a function
    at _initComp (app.js:14318)
    at app.js:14324
    at <anonymous>

Has this changed?  Thanks

The way you are trying to do it hasn't changed, but v1 actually provides a nicer API for manually mounting components.
There are two ways to do it:
const MyComponent = Moon.extend("MyComponent", {
  props: ["foo"],
  template: "<h1>{{foo}}</h1>"
});

// Preferred method
new MyComponent({
  root: "#mycomponent-app",
  props: {
    foo: "Property Value"
  }
});

// Manually call mount
new MyComponent().mount("#mycomponent-app");

That works, but with a difference. The way it was before, without declaring a root attribute. I had the "#app-moon" id attribute on the  tag, and any other Moon component declared would just be added to the body.
How can I add several components different components and variations of the same type to a Moon instance container like "#app-moon" on the  tag which was how it worked before.
The code below works, using the method with a root attribute and manually mounting
         var TestComp = Moon.extend("TestComp", {
             template: templates['test.html']
         });

         //  new TestComp({
         //      root: ".sparks",
         //      props: {
         //          foo: "Property Value"
         //      }
         //  });
         new TestComp().mount(".sparks");

Moon components will always replace the element. This is how it works even when you use them in a template like <TestComp/>.
If you want to add to a list, you'll have to appendChild a node to the wrapper, and mount to that child. An easier alternative would be to use a root Moon instance along with m-for.

...or use Moonscript. Got it. Thanks

HyperMoon*
No problem!


Session: There is an issue with the where clause

Describe the bug
update "session" set  where "session"."token" = $1 returning "id", "user_id", "token", "expires_at", "ip_address", "user_agent", "created_at", "updated_at" -- params: ["lLEU8HBrqpQA9dA_L034KMgL44bWiOQg"]

 ERROR   NeonDbError syntax error at or near "where"
at execute (webpack-internal:///(rsc)/../../node_modules/@neondatabase/serverless/index.mjs:1568:77)
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  at async NeonHttpPreparedQuery.execute (webpack-internal:///(rsc)/../../node_modules/drizzle-orm/neon-http/session.js:43:20)
  at async withReturning (webpack-internal:///(rsc)/../../node_modules/better-auth/dist/adapters/drizzle.js:15:9932)
  at async Object.update (webpack-internal:///(rsc)/../../node_modules/better-auth/dist/adapters/drizzle.js:15:11336)
  at async s (webpack-internal:///(rsc)/../../node_modules/better-auth/dist/index.js:128:14150)
  at async Object.updateSession (webpack-internal:///(rsc)/../../node_modules/better-auth/dist/index.js:128:17853)
  at async Object.setActiveOrganization (webpack-internal:///(rsc)/../../node_modules/better-auth/dist/plugins.js:138:28984)
  at async eval (webpack-internal:///(rsc)/../../node_modules/better-auth/dist/plugins.js:138:52718)
  at async handle (webpack-internal:///(rsc)/../../node_modules/better-call/dist/index.js:403:17)
  at async Ue.a.<computed> (webpack-internal:///(rsc)/../../node_modules/better-auth/dist/index.js:128:10227)
  at async handler (webpack-internal:///(rsc)/../../node_modules/better-call/dist/index.js:619:26)
  at async handler (webpack-internal:///(rsc)/../../node_modules/better-call/dist/index.js:670:19)
  at async AppRouteRouteModule.do (/Users/eneaxharau/Documents/Work/inventory-manager/node_modules/next/dist/compiled/next-server/app-route.runtime.dev.js:10:32976)
  at async AppRouteRouteModule.handle (/Users/eneaxharau/Documents/Work/inventory-manager/node_modules/next/dist/compiled/next-server/app-route.runtime.dev.js:10:38477)
  at async doRender (/Users/eneaxharau/Documents/Work/inventory-manager/node_modules/next/dist/server/base-server.js:1493:42)
  at async responseGenerator (/Users/eneaxharau/Documents/Work/inventory-manager/node_modules/next/dist/server/base-server.js:1834:28)
  at async DevServer.renderToResponseWithComponentsImpl (/Users/eneaxharau/Documents/Work/inventory-manager/node_modules/next/dist/server/base-server.js:1876:28)
  at async DevServer.renderPageComponent (/Users/eneaxharau/Documents/Work/inventory-manager/node_modules/next/dist/server/base-server.js:2290:24)
  at async DevServer.renderToResponseImpl (/Users/eneaxharau/Documents/Work/inventory-manager/node_modules/next/dist/server/base-server.js:2328:32)

To Reproduce
Steps to reproduce the behavior:

Using drizzle Adapter and NeonDB
Trying to set an active organization
Session doesn't get updated

Expected behavior
This is supposed to just update the session with the active organization, based on the id i passed on the function

OS: MacOS Sequoia
Browser: Arc Browser

Solved this, by adding activeOrganizationId in the Session Table, but nowhere in the docs it said so.

Solved this, by adding activeOrganizationId in the Session Table, but nowhere in the docs it said so.

sorry about that. Just got added!


Add model_to_solution and model_to_state.

変更点

変更前の時点では、model_to_solution 関数がommx.v1.Stateを返している状態だったため、関数名をmodel_to_stateに変更しました
ommx.v1.Solutionを返す関数model_to_solutionを新しく実装しました
この変更に伴い、solve関数の内部処理を新規実装したmodel_to_solutionに置き換えました

注意事項
model_to_solution関数の破壊的変更のため、最低でもマイナーバーションを上げる必要があります。

To my knowledge, there is no use case that requires retrieving only ommx.v1.State.

Since evaluating into ommx.v1.Solution is somewhat heavy, it would be better to use State instead of Soluiton when Python-MIP adapter is used as a component of higher level algorithm. Basically State is for computer and Solution is for person.

To my knowledge, there is no use case that requires retrieving only ommx.v1.State.

Since evaluating into ommx.v1.Solution is somewhat heavy, it would be better to use State instead of Soluiton when Python-MIP adapter is used as a component of higher level algorithm. Basically State is for computer and Solution is for person.

In fact, as you mentioned, from a computational point of view, if just getting the state is sufficient, then it's better to just return the state without any further computation. However, I personally can't think of a situation where you would use just the state without the solution when implementing advanced algorithms. Wouldn't you generally need not only the intermediate solution but also the associated optimal value when implementing algorithms? For example, in a simple example like branch-and-bound, we would use both the solution of the relaxed subproblem (according to State) and its optimal value.
Well, it's quite possible that I simply can't imagine such use cases due to my limited experience and knowledge of mathematical optimisation.
In any case, I have no intention of opposing the addition of a function that returns State.


NeDB not saving into file or calling back in Express

Hello!
I am writing a bulk import function for a password manager for myself and I have come across an issue.
There is an array of passwords to import and I'm using a forEach() method to iterate through each password to import.
 No error, no callback, no saving to file. Here is my code:
const express = require('express')
const app = express()
const { encrypt, decrypt } = require('./crypto')
const Datastore = require('nedb')

app.post('/bulkimport', checkAuthenticated, (req, res) => {
  var passwords = JSON.parse(req.body.passwords)
  var dbForUser = new Datastore('./passwords/' + req._passport.session.user + '.db')
  passwords.forEach(password => {
    function doc(code, link, name, password) {
      this.code = code
      this.link = link
      this.name = name
      this.password = password
    }
    var entry = new doc(password.name, password.url, password.username, password.password)
    console.log(entry)
      console.log('before insert') // gets logged
      dbForUser.insert(entry, function(err, doc) {
        console.log('after insert') // doesn't get logged
        if (err) return res.status(500).send()
        console.log(doc)
      })
  });
})

Thanks for the help!

Question moved to StackOverflow.

Question moved to StackOverflow.


Fix Selenium 4.0 deprecation message

Fixes #34

@rubendinho is there anything blocking this PR so that this issue can be resolved?

@rubendinho is there anything blocking this PR so that this issue can be resolved?

@blehoux17 I don't believe so - I don't have write permissions to this repo to merge it though.

@rubendinho What will merging this to do old versions of Selenium? Does this need to be a breaking version change?

Thanks for responding @dbalatero! I think this can easily be made backwards compatible:
browser = Capybara.current_session.driver.browser
logs = browser.respond_to?(:logs) ? browser.logs : browser.manager.logs
logs.get(type)

What do you think @rubendinho?

@synth I think this should work nicely.  Unfortunately, we're no longer using this library so I can't verify.

Hey @dbalatero! Just bubbling this back up and tagging you as one of the contributors of this repo. Is there anything blocking this PR from being merged? I see that you 👍'd  @synth's suggestion - I am wondering if you able to merge this in?

@dbalatero Any update?


WIP Fixes #4084 Port jetty to cats effect 3

@rossabaker The tests in JettyServerSuite all time out after 30 seconds and result in error. Where did I go wrong in that?

@rossabaker The tests in JettyServerSuite all time out after 30 seconds and result in error. Where did I go wrong in that?

Are we past the timeout problem now?

Are we past the timeout problem now?

Are we past the timeout problem now?

@rossabaker no, the timeout issue still persists.

Are we past the timeout problem now?

@rossabaker no, the timeout issue still persists.

Oh, the module just isn't enabled yet.  I didn't see the failing test and was hoping we were through it.  Do you have any ideas yet or do you need some help?

Oh, the module just isn't enabled yet.  I didn't see the failing test and was hoping we were through it.  Do you have any ideas yet or do you need some help?

Oh, the module just isn't enabled yet. I didn't see the failing test and was hoping we were through it. Do you have any ideas yet or do you need some help?

I am still trying to figure out the issue. Any help is welcome:)
I am also skeptical of the changes made in Issue454.scala. I think the dispatcher will be shutdown well AsyncHttp4sServlet gets a change to use it?

Oh, the module just isn't enabled yet. I didn't see the failing test and was hoping we were through it. Do you have any ideas yet or do you need some help?

I am still trying to figure out the issue. Any help is welcome:)
I am also skeptical of the changes made in Issue454.scala. I think the dispatcher will be shutdown well AsyncHttp4sServlet gets a change to use it?

Oh, I thought we were deleting Issue454 entirely since it isn't a test that gets run.

Oh, I thought we were deleting Issue454 entirely since it isn't a test that gets run.

Oh, I thought we were deleting Issue454 entirely since it isn't a test that gets run.

okay, I understood it as deleting the implicit line 😅
Will delete it:)

Oh, I thought we were deleting Issue454 entirely since it isn't a test that gets run.

okay, I understood it as deleting the implicit line 😅
Will delete it:)

Added some log statements. It confirms this line is causing the timeout.

Added some log statements. It confirms this line is causing the timeout.

Try unsafeRunAndForget instead of unsafeRunSync.

Try unsafeRunAndForget instead of unsafeRunSync.

Try unsafeRunAndForget instead of unsafeRunSync.

results in IOException: Server returned HTTP response code: 503..

Try unsafeRunAndForget instead of unsafeRunSync.

results in IOException: Server returned HTTP response code: 503..

@rossabaker The issue was in servlet module. async_ has signature k: (Either[Throwable, A] => Unit) => Unit): F[A] but Deferred's complete was returning an F[Boolean].

@rossabaker The issue was in servlet module. async_ has signature k: (Either[Throwable, A] => Unit) => Unit): F[A] but Deferred's complete was returning an F[Boolean].


hashability

make all the models hashable

closed by #27

note that GroupSpec and ArraySpec subclasses can't be hashable, because of the Attributes field inheriting from dict. Would be great to have a workaround for this.


Fix packagist

Hi!
Your library is on version 1.1.0, but from packagist we can download version 1.0.0 only.
Thank you!

@gidkom
Ref: https://packagist.org/packages/gidkom/php-openfire-restapi#1.0.0
Can we help?
Thanks!

1.1 is from Feb 2018
Perhaps release a new version and try to make it available via Packagist?
Thank you!

ok I'll do that

It is bot done yet: https://packagist.org/packages/gidkom/php-openfire-restapi


nginx with new lua headers on local

Describe the bug
on local environments, we don't have LAGOON_AUTOGENERATED_ROUTES or LAGOON_ENVIRONMENT_TYPE set, which causes
https://github.com/amazeeio/lagoon/blob/1fc2b1619e8a38ba25c250aa874ad441e24c144c/images/nginx/helpers/x-robots-header-development.conf#L6
to fail with:
nginx_1    | 2018/11/08 01:42:46 [error] 38#38: *9 failed to run header_filter_by_lua*: header_filter_by_lua:5: bad argument #1 to 'match' (string expected, got nil)
nginx_1    | stack traceback:
nginx_1    | 	[C]: in function 'match'
nginx_1    | 	header_filter_by_lua:5: in function <header_filter_by_lua:1>, client: <IP_ADDRESS>, server: , request: "GET /favicon.ico HTTP/1.1", host: "drupal-example.docker.amazee.io", referrer: "http://drupal-example.docker.amazee.io/"

Expected behavior
nginx should not fail

if you have that issue local, run :
docker image ls --format "{{.Repository}}:{{.Tag}}" | grep amazeeio/ | grep -v none | xargs -n1 docker pull | cat

and then
docker-compose build
docker-compose up -d


Option to clear the canvas

Hi! Any way to add and option to "clear" the canvas like the current signature-pad?

I worked it out. I forked the gem and added this to the bottom of the john_hancock.js
function signatureClear() {
var canvas = document.getElementById("JohnHancock-canvas");
var context = canvas.getContext("2d");
context.clearRect(0, 0, canvas.width, canvas.height);
}
Then I used this button code:  Clear

Want to open a PR to make this function available?


Fixed issues in .imshow_interactive()

Made imshow_interactive more robust, added xlims keyword, x/y buffer keywords and added a rescaling of plotted 2d rectangles with xlims so the 2d plot still looks nice zoomed in.

Added!


Inputs with floating labels

It would be cool to have an option for floating labels on text input boxes, similar to material design.

Please, please do that! Would be awesome!

You mean this label https://www.google.com/design/spec/components/text-fields.html#text-fields-labels?

Yep!

I think it's better to keep Spectre lightweight, if you want a Material Design input then why not use some material design stylesheet, there are tons of them.

@all I think I will not add that input. That is not Spectre style component. Sorry.


3.0.0-snapshot-459 - shortestpath

Hi,
It seems that the shortestpath function is not working in 3.0. Has something changed?
SELECT expand(path) FROM (
SELECT shortestPath($from, $to) AS path
LET $from = (SELECT FROM Profiles WHERE Name='Santo' and Surname='OrientDB'), $to = (SELECT FROM Countries WHERE Name='United States')
UNWIND path
)
--> No function with name 'shortestpath', available names are :
works as expected in 2.2.17
this is related to the demo queries I am creating for the new demodb
Thanks,

That's quite strange, let me check it...

Fixed in develop branch. Keeping this open because I also have to do some work on A* and Dijkstra functions (we need them in core since me implemented the new MultiModel API)

Ok, all the graph functions are now in Core module.
Closing, thanks!
Luigi


Localhost site crashes server process on Windows

Hi,
After just updating to your v3.0.2 and rebuilding /data/db, everything is working fine (expect still requiring cross-spawn on Windows), but when I go to load the localhost website it gives me this error:
08/14/2016 10:37:14 PM BST [      server] open http://localhost:3013/?secret=dc3d5ed33f476e8e to see a live graph.
D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:48
        if (typeof file === 'undefined') throw new Error('template not found: ' + p);
                                         ^

Error: template not found: home
    at render (D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:48:48)
    at ServerResponse.res.render (D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:70:23)
    at Runner.<anonymous> (D:\zenbot-3.x\plugins\server\controllers\home.js:11:11)
    at Immediate.nextRoute [as _onImmediate] (D:\zenbot-3.x\node_modules\motley\node_modules\middler\node_modules\bladerunner\index.js:51:23)
    at processImmediate [as _immediateCallback] (timers.js:383:17)
 08/14/2016 10:37:14 PM BST [    launcher] cmd `server` exited with code 1, respawning now.
08/14/2016 10:37:17 PM BST [    launcher] cmd `server` booting

I can't figure it out myself sorry, so I thought I'd report it.

hmm, looks like my templating library is failing on windows. not surprising, I don't test on windows at all. I recommend running this on a linux docker container or mac instead.

Heya, just updated and currently on my mobile phone (SSH FTW!). Bot seems
to be running smoothly. Although it would be verry pleasent to have
config.js moved to sample-config.js (one for Euro and one for USD). I wilk
report back as soon as possible.
There are a few things I am really hoping for like:

BTC-ETH Trading (GDAX)
Euro/USD (Seperated configs)
Config renamed to sample-eur-config.js. (example)
config.js added to .gitignore (saves trouble!)

Next to that this is some great stuff!
Yours Faithfully,
RDash
On Sun, Aug 14, 2016, 23:40 xangma<EMAIL_ADDRESS>wrote:

Hi,
After just updating to your v3.0.2 and rebuilding /data/db, everything is
working fine (expect still requiring cross-spawn on Windows), but when I go
to load the localhost website it gives me this error:
08/14/2016 10:37:14 PM BST [      server] open http://localhost:3013/?secret=dc3d5ed33f476e8e to see a live graph.
D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:48
if (typeof file === 'undefined') throw new Error('template not found: ' + p);
^
Error: template not found: home
at render (D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:48:48)
at ServerResponse.res.render (D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:70:23)
at Runner. (D:\zenbot-3.x\plugins\server\controllers\home.js:11:11)
at Immediate.nextRoute [as _onImmediate] (D:\zenbot-3.x\node_modules\motley\node_modules\middler\node_modules\bladerunner\index.js:51:23)
at processImmediate [as _immediateCallback] (timers.js:383:17)
08/14/2016 10:37:14 PM BST [    launcher] cmd server exited with code 1, respawning now.
08/14/2016 10:37:17 PM BST [    launcher] cmd server booting
I can't figure it out myself sorry, so I thought I'd report it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/carlos8f/zenbot/issues/13, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHMwQ_V6iiNBi-3ZXVIYNf4M7EaQ1wL0ks5qf4thgaJpZM4Jj_mB
.

Whoops, replied to the wrong mail :')
On Sun, Aug 14, 2016, 23:49 Daan Montijn<EMAIL_ADDRESS>wrote:

Heya, just updated and currently on my mobile phone (SSH FTW!). Bot seems
to be running smoothly. Although it would be verry pleasent to have
config.js moved to sample-config.js (one for Euro and one for USD). I wilk
report back as soon as possible.
There are a few things I am really hoping for like:

BTC-ETH Trading (GDAX)
Euro/USD (Seperated configs)
Config renamed to sample-eur-config.js. (example)
config.js added to .gitignore (saves trouble!)

Next to that this is some great stuff!
Yours Faithfully,
RDash
On Sun, Aug 14, 2016, 23:40 xangma<EMAIL_ADDRESS>wrote:

Hi,
After just updating to your v3.0.2 and rebuilding /data/db, everything is
working fine (expect still requiring cross-spawn on Windows), but when I go
to load the localhost website it gives me this error:
08/14/2016 10:37:14 PM BST [      server] open http://localhost:3013/?secret=dc3d5ed33f476e8e to see a live graph.
D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:48
if (typeof file === 'undefined') throw new Error('template not found: ' + p);
^
Error: template not found: home
at render (D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:48:48)
at ServerResponse.res.render (D:\zenbot-3.x\node_modules\motley-templ\node_modules\templ\index.js:70:23)
at Runner. (D:\zenbot-3.x\plugins\server\controllers\home.js:11:11)
at Immediate.nextRoute [as _onImmediate] (D:\zenbot-3.x\node_modules\motley\node_modules\middler\node_modules\bladerunner\index.js:51:23)
at processImmediate [as _immediateCallback] (timers.js:383:17)
08/14/2016 10:37:14 PM BST [    launcher] cmd server exited with code 1, respawning now.
08/14/2016 10:37:17 PM BST [    launcher] cmd server booting
I can't figure it out myself sorry, so I thought I'd report it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/carlos8f/zenbot/issues/13, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHMwQ_V6iiNBi-3ZXVIYNf4M7EaQ1wL0ks5qf4thgaJpZM4Jj_mB
.

_http_outgoing.js:399                                                                                throw new Error('Can't render headers after they are sent to the client');
^
Error: Can't render headers after they are sent to the client
at ServerResponse.OutgoingMessage._renderHeaders (_http_outgoing.js:399:11)                        at ServerResponse.writeHead (_http_server.js:184:20)
at ServerResponse.res.renderStatus (/home/gekko/zenbot/node_modules/templ/index.js:83:13)          at Object. (/home/gekko/zenbot/plugins/server/controllers/data.js:17:37)
at Immediate. (/home/gekko/zenbot/node_modules/sosa/api.js:15:12)
at runCallback (timers.js:570:20)                                                                  at tryOnImmediate (timers.js:550:5)
at processImmediate [as _immediateCallback] (timers.js:529:5)                                  08/15/2016 12:06:38 AM CEST [    launcher] cmd server exited with code 1, respawning now.
Linux too apparently

@RDash21 just published 3.0.3 which should resolve that.

Pulled & did not crash when attempting to break it. Great job!
On Mon, Aug 15, 2016, 00:20 Carlos Rodriguez<EMAIL_ADDRESS>wrote:

@RDash21 https://github.com/RDash21 just published 3.0.3 which should
resolve that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/carlos8f/zenbot/issues/13#issuecomment-239701366, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHMwQ1m_9YETvT3MRCHAgqvUP_xlO0v4ks5qf5SjgaJpZM4Jj_mB
.

@carlos8f Ah that's fair enough. A sell signal didn't execute either when running under Windows so I've not set up a virtual machine to run it on Ubuntu instead.


How does neural noise change with ageing?

How do different types of neural noise (measured with fMRI, EEG or cellular recordings) change with ageing? notes and ideas

Now at https://github.com/Fenying-Zang/noisy_aging/issues/13


grid.onUnselect

i'm trying to override the default .status message when the user unselects some records. i'm using event.onComplete within the onUnselect event.
the default status message "recid: x" (for 1 record,) or "n selected" (for multiple records) always gets shown when unselecting via the mouse drag and it appears it cannot be overridden.
it works just fine when ctrl-clicking to unselect, but it does not work when unselecting using the mouse drag.
it might be a bug, or maybe there is there another event i should be looking for instead of unSelect?
thanks!

for any users with the same issue, i settled on the following workaround:
`onUnselect: function(p_event)
{
p_event.onComplete = function()
{
update_status_message();
setTimeout(update_status_message, 100);

}
},`
the default status message still flickers in the status bar for a moment, a bit crappy, but hey it works.

The "problem" is, that unselect is triggered by the click event. The click event basically wraps the unselect event, and after the unselect event is completed, the click event will attempt to update the status message again.
Personally, I think calling this.status() in click() can/should be removed, because, AFAICT, there's no way to get out of the click event w/o triggering select or unselect - and those methods already set the status.
@vitmalina What do you think? Do you agree, that this.status() can be safely removed in click?


Performance struggles under linux when placing like 100 units (swordsman/archer units)

Just tested this. The performance issues are much greater with a large amount of archers than they are with a large amount of swordsmen, leading me to believe an optimization of projectiles will be a good step toward improving this.

Looks like in addition to optimizing projcetiles, the other thing that'll improve this is preventing units from colliding. I'm gonna try some things with GSAI to see if it fixes issues.
Current benchmark I'm testing is 7000 nodes, all swordsmen, which is around 7fps at the moment.


Example Out of Date

https://github.com/avthart/docker-consul-template/blob/master/examples/examples.md:
progrium/registrator has moved to gliderlabs/registrator

Thx! I have updated the consul and registrator docker images


Flakiness: iterate over a slice instead of iterating over a map

To reproduce the flakiness that is fixed in this commit, you can run:
go test ./pkg/... -run "TestSync/gateway-shim/if_a_Gateway_contains_two_.*" -count 10

While working on https://github.com/jetstack/cert-manager/pull/4293, we found that, since #4158, one of the unit tests was flaky:
I0803 17:14:34.151059 3741895 reflector.go:225] Stopping reflector *v1.Certificate (10ms) from pkg/mod/k8s.io/client-go@v0.21.2/tools/cache/reflector.go:167
--- FAIL: TestSync (0.14s)
  --- FAIL: TestSync/gateway-shim/if_a_Gateway_contains_two_listeners_with_different_Secret_names,_it_should_create_two_Certificates (0.13s)
  sync_test.go:2450: got unexpected events:
  exp='[Normal CreateCertificate Successfully created Certificate "foo-example-com-tls" Normal CreateCertificate Successfully created Certificate "bar-example-com-tls"]'
  got='[Normal CreateCertificate Successfully created Certificate "bar-example-com-tls" Normal CreateCertificate Successfully created Certificate "foo-example-com-tls"]'

At first, we thought that it was due to the fact that we were (wrongly) expecting the events to come in a deterministic order (i.e., the events arrive in the order they are sent), and we were looking at using EqualUnsorted instead of EqualSorted to check the events in AllEventsCalled.
But then, we found that this indeterminism was coming from the fact that we store the Gateway TLS hostnames in a map in sync.go:
tlsHosts := make(map[corev1.ObjectReference][]string)
for secretRef, hosts := range tlsHosts {
    // Undeterministic order!
}

This PR introduces fixes the indeterminism.

/kind bug
/release-note-none

I'll use EqualUnsorted instead of bothering with making the Sync func deterministic.

/assign @maelvls
Discussed on a call, this can be replaced with a simpler change

I'll fix this in https://github.com/jetstack/cert-manager/pull/4484/commits/7a3747084312bb6fd57800b11924624d9df0e9be


Accept React.Elements as text

Describe your changes here.

[x] Accepting JSX / React.Elements in the text prop.
[ ] Pass text to aria-label

Fixes https://github.com/github/primer/issues/1142.
Screenshots
Please provide before/after screenshots for any visual changes
Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[x] Tested in Chrome
[x] Tested in Firefox
[x] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Closing this after discussing it with @siddharthkp. Passing a react.element to an aria-label might be tricky and other solutions would make the API complicated.


Revert "fix(serverless-contracts): update dependency tsx to v4"

Reverts swarmion/swarmion#1124
I believe tsx v4 is incompatible with serverless-framework
cf https://github.com/swarmion/swarmion/pull/808#issuecomment-1833392426

Ah my bad, I was a bit too eager... We'll wait after removing the Serverless Framework then 😉


[BUG] changing demo's does not work

Describe the bug
When I change #define CONFIG_CORE_MQTT_MUTUAL_AUTH_DEMO_ENABLED to #define CONFIG_OTA_HTTP_UPDATE_DEMO_ENABLED in vendors/vendor/boards/board/aws_demos/config_files/aws_demo_config.h like mentioned in your docs... absolutely nothing happens!
System information

Hardware board: ESP32-DevKitC
IDE used: none
Operating System: MacOS
Code version: 202107.00-3-g19c657cf0
Project/Demo: anything other than mqtt_demo_mutual_auth

Expected behavior
The selected demo (in this case OTA HTTP demo runs)
My guess is that it looks like there is a second file where this defined vendors/espressif/boards/esp32/aws_demos/config_files/aws_demo_config.h

I can confirm changing the demo in both places
vendors/vendor/boards/board/aws_demos/config_files/aws_demo_config.h
AND
vendors/espressif/boards/esp32/aws_demos/config_files/aws_demo_config.h
works... I feel like I was eating crazy pills for a while here..

so I guess I see where I went wrong, and maybe this isn't a bug...

I guess the red parts are suppose to be variables, and I needed to replace them with my specific vendor and board... but the fact vendors/vendor/boards/board/aws_demos/config_files/aws_demo_config.h exists is maddening.. maybe this is used for a template when creating a new board?

Facing the same issue. Thanks for your input!


test host API calls

This PR tests that API calls to add, resize and remove host storage folders are handled properly, including various flavors of invalid calls.
Several errors from the storagemanager module are now exported, and I added a couple of exported functions so that the tests could access storagemanager's minimumStorageFolderSizeand maximumStorageFolderSize constants.
I also fixed some typos.
TODO:

a couple more tests to cover all error handling by the add/resize storage folder API calls
test the API call to delete a specific storage sector

Overall these tests are very good. These are exactly the types of tests that I'm hoping to have cover all the modules. The logic and test types are cleanly separated, the code is pretty straightforward, and the underlying modules are probed using exclusively the API, which means that it's checking that both the modules are conforming to specification and that the API does not have any bugs.

Appveyor is reporting that TestHostAutoRevisionSubmission in the host module was killed because it ran for too long (10 min).

seems like a NDF. I'll restart the build

This is happening in Jordan's PR as well.


Why does it pull images again at start of stale site?

This happened after the pull/start completed:

Ok, I think I know what might have happened here. I am running Docker Desktop and OrbStack together, they got into a fight 😆
Closing.


Single entrypoint

Right now we have multiple ways to run an experiment with sheeprl, namely:

lightning run model sheeprl.py COMMAND ...
python sheeprl.py COMMAND ...

It would be nice to have a single entrypoint so to launch every experiment with python sheeprl.py exp=..., where the exp= specifies the experiment config (predefined configs can be found under the sheeprl/configs/exp/ directory). This let us decorate only the entrypoint with the @hydra.main() decorator.
The only requirement is that algo.name=algorithm_name, where algorithm_name comes from algorithm_name.py, where algorithm_name.py is the file containing the algorithm implementation (it must contain at least one method decorated with the register_algorithm decorator)

Solved by #100


Could lead to data corruption

Could you add in the warning section about Disk caching that not following the recommendations could lead to data corruption ?
I am also trying to have a warning displayed on the Portal as users might not have the reflex of referring to this page before changing the Disk cache settings.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1a558845-e440-764f-d271-b02483a8ee95
Version Independent ID: 166d37f1-0082-11a7-2b2f-921ac1a2ba4f
Content: Azure Premium Storage: Design for performance on Linux VMs - Azure Linux Virtual Machines
Content Source: articles/virtual-machines/linux/premium-storage-performance.md
Service: virtual-machines-linux
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

I have created a PR: https://github.com/MicrosoftDocs/azure-docs/pull/55219

@johanburati Thanks for creating the PR for updating the document. I have updated doc author for reviewing the PR. The changes will go live once the PR is approved.


[Perf Framework] Restructuring the perf test folders

Description
This change doesn't affect how the perf tests are written.
This PR attempts to change the location of the perf tests to allow running the perf tests with the older track 2 versions and for smoother automation with the pipelines.
The main change here is that the perf tests are made into projects maintained by "rush" as opposed to in-place tests or separate npm projects.
Check https://github.com/Azure/azure-sdk-for-js/pull/13770 for proposal 2.
Testing Track 1

rush update
Navigate to sdk\storage\perf-tests\storage-blob-track-1
rush build -t perf-test-storage-blob-track-1
Add .env file as suggested in the readme
Run the tests as suggested by readme, example npm run perf-test:node -- StorageBlobDownloadTest --warmup 2 --duration 7 --iterations 2 --parallel 2

Testing Track 2 - current

rush update
Navigate to sdk\storage\perf-tests\storage-blob
rush build -t perf-test-storage-blob
Add .env file as suggested in the readme
Run the tests as suggested by readme, example npm run perf-test:node -- StorageBlobDownloadTest --warmup 2 --duration 7 --iterations 2 --parallel 2

Testing Track 2 - older versions

Example: To test 12.2.0

Update "@azure/storage-blob" version in package.json to 12.2.0
Add a new exception in common\config\rush\common-versions.json under allowedAlternativeVersions

"@azure/storage-blob": [..., "12.2.0"]

rush update (generates a new pnpm-lock file)
Navigate to sdk\storage\perf-tests\storage-blob
rush build -t perf-test-storage-blob
Add .env file as suggested in the readme
Run the tests as suggested by readme, example npm run perf-test:node -- StorageBlobDownloadTest --warmup 2 --duration 7 --iterations 2 --parallel 2

Closing since we are going with #13770


fft and ifft implementation

Please see https://github.com/google/jax/issues/1877 for reference

Happy to have some feedback to know if I'm on the right track.
I'm going to flesh out the tests next

Looks like the tests made it through the CI build.
I should add the documentation with this same PR, right?

pushed the requested changes

Thanks Archis!


WIP: Use distributed executor always

Shifts the remaining tasks namely dictionary learning and postprocessing onto the distributed cluster. With this change, no computational steps use ipyparallel directly any more. Thus it should be possible to remove use of ipyparallel as long as a distributed cluster is started by some means.

Post-processing works smoothly. However encountering some issues with dictionary learning where it appears we are loosing engines. Will need to take a closer look at what is going on.


Heatmap is not working inside of uib-tabset in angularjs

<uib-tabset class="tabbable border"> 	<uib-tab ng-repeat="floor in floorPlans" heading="{{floor.floor_name}}" select="heatmap()" >

 			 			<heat-map data="passed_data"></heat-map>

 	</uib-tab> </uib-tabset>

I'd appreciate a bit more context :)

is there an error in the console ?
does passed_data actually contain data? (maybe it's a scope issue)
are you using the official angularjs heatmap plugin that is published in this repository?

There is no error in console.
Yes im using official angularjs heatmap plugin get from this repo.

@pa7
And also i need a help to how to add background image as dynamic.
Thanks.

does passed_data actually pass down data?

i didn't get any response for passed_data.
The below response is actually i got it for your reference

<heat-map data="passed_data" class="ng-isolate-scope">
<canvas class="heatmap-canvas" width="0" height="0" style="position: absolute; left: 0px; top: 0px;"></canvas>
</heat-map>

Have you had a look at the example?
<heatmap id="heatmap-1" data="heatmapData" config="heatmapConfig" ng-click="updateData()"></heatmap>

it's <heatmap> not <heat-map>
if it still doesn't work, set the width and height properties:
<heatmap data="heatmapData" config="heatmapConfig" width="500" height="300"></heatmap>
if there's no data it will not show a heatmap

i just changed the name. otherwise the same code is here.
now i set the width and height properties that is working fine.
thanks a lot :)

you're welcome


Powershell under Linux fails to provide requested exit code

Steps to reproduce
powershell -command {exit 1} && echo $?
results in output

exit 1
0

Another suggested way:
powershell -command {throw "exception"} && echo $?
results in

throw exception
0

Expected behavior
In both of the cases I expected the exit code to be 1
Actual behavior
Environment data

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.0.0-alpha
PSEdition                      Core
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.9
CLRVersion
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Additionally: GNU bash, version 4.3.46(1)-release (x86_64-pc-linux-gnu)
OS: Ubuntu 16.04.1

The correct usage is:
powershell -command "& {exit 1}"
powershell -command "& {throw 'exception'}"
exit code is 1 as expected on Windows and Linux

This functions the same way as Windows PowerShell.  Perhaps @lzybkr knows why powershell -command {get-process} doesn't actually execute get-process

running powershell -command {get-process} from bash or cmd.exe actually just creates a script block and writes it out. It's like running {get-process} in an interactive powershell console.
What interest me is that powershell -command 'exit 1' && echo $? output nothing, while do it like following output 1 as expected.
powershell -command 'exit 1'
echo $?

so , how do we use the <native> && <name> syntax in bash with powershell -command?

@SteveL-MSFT - the syntax powershell -command "& {exit 42}" is a little more than necessary - powershell -command "exit 42" is sufficient.
Note that && short circuits when the return code is non-zero, so you'd use powershell -command "exit 42" || echo $?.
@daxian-dbw is correct about why powershell -command {exit 42} doesn't work - from bash/cmd, the curly braces are just random text, they know nothing about script blocks.
In PowerShell, we know what a script block is, and when we know we're calling another native executable that understands script blocks (these days, basically just powershell, but the deprecated mini-shells worked the same), we are smart enough to drop the curly braces.
It's true that this can be confusing - the same command line will do something different b/w powershell and bash/cmd, but it is useful syntax in PowerShell because you get syntax highlighting and parse time checking before running the script.

Note that && short circuits when the return code is non-zero, so you'd use powershell -command "exit 42" || echo $?.

Ah, thanks @lzybkr, that explains 😄
linux:~$ powershell -command "write-output hello" && echo world
hello
world

F:\temp>powershell -command "write-output hello" && echo world
hello
world

Submitted doc fix to make the example more clear which led to this misunderstanding
https://github.com/PowerShell/PowerShell-Docs/pull/649

I was very shocked by the importance of the quote type:
C:\Users\salinas>powershell -command "throw 1" && echo ok
1
En línea: 1 Carácter: 1
+ throw 1
+ ~~~~~~~
    + CategoryInfo          : OperationStopped: (1:Int32) [], RuntimeException
    + FullyQualifiedErrorId : 1

C:\Users\salinas>powershell -command 'throw 1' && echo ok
throw 1
ok

@rsalinas CMD strips off the double quotes and keeps the single quotes
C:\>E:\PowerShell\test\tools\TestExe\bin\Debug\netcoreapp2.1\win10-x64\testexe.exe -echoargs 'qrr'
Arg 0 is <'qrr'>

C:\>E:\PowerShell\test\tools\TestExe\bin\Debug\netcoreapp2.1\win10-x64\testexe.exe -echoargs "fafa"
Arg 0 is <fafa>


Parameter server needs sharding

Since the deep learning model is large, its gradients are large too. This results in the parameter server suffering a network bottleneck. The current design is unlikely to compete against a single-node-based approach.
A solution could involve

[x] hosting different parameter (chunks) on different nodes
[x] reduce disk usage by communicating gradients via Cassandra instead of MinIO

Resolved here.

parameters and gradients are sharded
gradients are communicated directly, instead of via any disks, including Cassandra

Resolved here.

parameters and gradients are sharded
gradients are communicated directly, instead of via any disks, including Cassandra


Fix Rubocop issues

@agrare Please review.
The first commit here fixes lib/swaggering.rb, but on inspection, do we even need this file?  It seems like it was a temporary tool to get us to a full Rails app, so can we just drop it?

@Fryguy yeah you're right that's a leftover from the sinatra code and can be removed


Failed to run the deviceDidReceiveIncoming method

Hello everyone,
I tried to complete a connect call from any user to client twilio (Andorid Device), but "deviceDidReceiveIncoming" not working.
Did you follow all the instructions as specified in the Twilio Quickstart repositories? Yes
What version of React Native are you running? 0.64
What version of react-native-twilio-programmable-voice are you running? "react-native-twilio-programmable-voice": "https://github.com/hoxfon/react-native-twilio-programmable-voice#feat/twilio-android-sdk-5"
What device are you using? (e.g iOS10 simulator, Android 7 device)? Mi A2 Lite - Android version 9
Is your app running in foreground, background or not running? **foreground & background **
Is there any relevant message in the log? No
Step to reproduce
When I make a call everything is ok, I can send call, I create a token with permissions to outgoing and incoming
Token Twilio abstract
"grants": {
    "identity": "ANDROIDDEVICENAME",
    "voice": {
      "incoming": {
        "allow": true
      },
      "outgoing": {
        "application_sid": "MYAPPLICATIONSID"
      }
    }

When calling meanwhile client is offline and calling from other device, the call immediately reject, but when client is online and calling from other device, the call is rining, how calling, but the "deviceDidReceiveIncoming" never run
TwilioVoice.addEventListener('deviceDidReceiveIncoming', function(data) {
            console.log('new Call',data);
            TwilioVoice.accept()
        })

Advanced:
Have you tried adding break point using AndroidStudio or XCode and analyse the logs? No, whe a try run with android studio, send message "unable to load script make sure you are either running a metro server or that your bundle "index.android.bundle" is packaged correctly for release"
can share a project with issue? Yes
Thanks so much !

Hi. I have the same problem. I used v4.4.0, but the app can't receive the background notification. How can we modify with this library?

I'm seeing this same issue on iOS, I've added a bunch of logging to the native code and all the other features are working as expected but the deviceDidReceiveIncoming is the only thing that's not working.

+1


Move Heroku buildpack to dedicated repo

Why do we want to move heroku to a dedicated repo? Last time we discussed the same for Ansible and agreed to stay here...

Because it does not work :) The way you push buildpacks is that the required directories need to be at the root level of the repo. I am not aware of a workaround -- appears to be how everyone else does it too.

Another option we tried in the past when galaxy had the same requirements was to keep a ~mirror of the deployment at a new project and manually refresh its contents (https://github.com/signalfx/ansible-signalfx-agent).  I think the two projects diverged almost immediately due to lack of clear coordination, so it probably would have been better to make the deployment directory its own repo and clone it from the relative path at the standalone project.
Unless there are clear reasons to keep the deployment in this project like shared, relative deps, updater scripts, or test infrastructure simply migrating is the easiest way.

No dependencies on this repo :)

Sounds good! Let's move it away


Improved base image parsing

Task name: Docker Commons
Description: This change fix the issue where the algorithm confuse the the argument --from as a FROM statement.
Documentation changes required: N
Added unit tests: N
Attached related issue: https://github.com/microsoft/azure-pipelines-tasks/issues/15111
Checklist:

[ x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

/azp run


No feedback when Stoa cannot connect to Agora

If Stoa starts and is unable to connect to Agora, this should be apparent to the user.
I would favor simply terminating the program after logging a proper error message.
In the future we can revisit this behavior (e.g. output messages and try reconnecting), but the current situation where nothing happens nor is displayed is very misleading.

Fixed by #161


Rewrite the balancing algorithm so that tx is always pre-balanced before ex units calculation

Closes https://github.com/Plutonomicon/cardano-transaction-lib/issues/892, Closes https://github.com/Plutonomicon/cardano-transaction-lib/issues/570 (we have a similar problem with CSL fee calculation)
TODO: description
Pre-review checklist

[x] All code has been formatted using our config (make format)
[x] The integration and unit tests have been run (npm run test) locally
[ ] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

@ngua Thanks for your review! I would like to make some more changes real quick, and then we can merge it


Custom battles page sometimes get stuck

When opening this page, sometimes the loader just spins indefinitely

Can repro sometimes when leaving a battle

Possible linked to https://github.com/beyond-all-reason/bar-lobby/issues/181


Remove extra semicolon

Hi!
There is an extra semicolon in the ER_UNUSED macro that is unnecessary and I would like to remove it.

Thank you for your approve, JD -san!!


helm options  doesn't all have the --output flag

the option to output to json is not available for helm dep
$ helm dep list
NAME VERSION REPOSITORY STATUS
tomcat 6.3.6 https://charts.bitnami.com/bitnami ok
$ helm dep list --output json
Error: unknown flag: --output
is it possible to add the --output option to all commands ?

Please feel free to work on a patch to provide this.

I'd love to take this on myself as a first issue.

go go go :)

@yonatankahana thanks for your help. there is another issue : https://github.com/helm/helm/issues/8066 that I open yesterday (more complicated.. it you like challenge)

@survivant I thought you'd like to work on it yourself, if you're not then of course I'd be happy. Let me know what you decide, for now I'll concentrate on this issue.

Go, take it please.
Le ven. 8 mai 2020 17 h 22, yonatankahana<EMAIL_ADDRESS>a
écrit :

@survivant https://github.com/survivant I thought you'd like to work on
it yourself, if you're not then of course I'd be happy. Let me know what
you decide, for now I'll concentrate on this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/helm/helm/issues/8064#issuecomment-626026323, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABO3BZVGR24XWFT4XESMCTRQRZYFANCNFSM4M2QDMOQ
.

@survivant I thought you'd like to work on it yourself, if you're not then of course I'd be happy. Let me know what you decide, for now I'll concentrate on this issue.

yes you can take it.  Please do :)

keep open

still valid

Still valid
Le jeu. 25 mars 2021 20 h 15, github-actions[bot] @.***>
a écrit :

Closed #8064 https://github.com/helm/helm/issues/8064.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/helm/helm/issues/8064#event-4510810604, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABO3B44DGLFSZOFCIAIGLLTFPGZDANCNFSM4M2QDMOQ
.


Redshift support binary type since december 2021

https://github.com/databricks/spark-redshift/edit/master/src/main/scala/com/databricks/spark/redshift/RedshiftJDBCWrapper.scala -
line 251
I don't know how to write a PR on this, but this would be nice :)

that would be nice! :)


Include Zeek Logs wise data source

Include Zeek Logs wise data source
Contributed from Malcolm (https://github.com/idaholab/Malcolm).
Changes:

as zeek creates a LOT of fields, increase max field count from 512 to 1024 (moloch.h, wise.c)
increase width of field name column in session detail (SessionDetail.vue)
added commented-out section for enabling zeek log data source in wise.ini.sample
added source.zeeklogs.js for wise data source creation

Data may be populated with Malcolm's Zeek Logstash filters:

https://github.com/idaholab/Malcolm/tree/master/logstash
https://github.com/idaholab/Malcolm/blob/master/logstash/pipeline-main/11_zeek_logs.conf

Clearly describe the problem and solution
Relevant issue number(s) if applicable
Did you run npm run lint from the viewer or parliament directory (whichever you are making changes to) and correct any errors
Did you Ensure that all tests still pass by navigating to the tests directory and running ./tests.pl --viewer
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

One option for running Logstash configured with the filters i'm using to populate this data would be to just run Malcolm's Logstash docker container standalone:
Dockerfile
context

Looks like these changes cause the wise tests to fail.  Are there other patches missing?

Hmmmm. No, nothing missing that I can see. I'll investigate.

internal/buffer.js:555
    throw new ERR_OUT_OF_RANGE('value', `>= ${min} and <= ${max}`, value);
    ^

RangeError [ERR_OUT_OF_RANGE]: The value of "value" is out of range. It must be >= 0 and <= 255. Received 479
    at writeU_Int8 (internal/buffer.js:555:11)
    at Buffer.writeUInt8 (internal/buffer.js:565:10)
    at WISESource.encode (/root/moloch/wiseService/wiseSource.js:309:11)
    at FileSource.WISESource.parseTagger (/root/moloch/wiseService/wiseSource.js:194:68)
    at fs.readFile (/root/moloch/wiseService/source.file.js:76:10)
    at FSReqWrap.readFileAfterClose [as oncomplete] (internal/fs/read_file_context.js:53:3)

Looks like the issue is the tests aren't expecting any data sources with more than 255 fields created by default, maybe, when the max should be higher (I set it to 1024 in the code elsewhere). I'll update the pull request with a fix.

Actually the issue isn't with my code, it's with the test framework.
Some time ago you increased the field size to 512 (see commit 328d1d2 and commit d1fd430). This pull request increases it further to 1024. However, that's besides the point: even though capture and wise are both fine with the increased number of fields, the tests.pl which is calling WISESource.encode is writing the field count with a writeUInt8 which won't work. So yes, this pull request does fail the tests, but any other WISE data source that tries to create > 255 fields would have hit the same problem, even though it works fine in production.

When is this feature expected to be supported? After all, the only shortcoming of zeek is that it cannot export pcap packets, while moloch can.


rename snapshot command

a new command to manage a set of cluster logs would be:
dcos node diagnostics ...
a zip file with collected logs is now called a diagnostics bundle

@darkonie we're going to have to add deprecation notices on the snapshot commands

@tamarrow thanks, will do

@tamarrow Is that neccessary? Have we actually released the snapshot command to the community yet? 1.8EA isn't until Monday.
CC @spahl

I think we fix it before EA then we don't need to add a deprecation warning.

okay


fix: target wheel missing from Makefile setup

Which also raises the question: do we want to handle the source and binary distribution separately? Now that we switched to flit as a build core (see PR https://github.com/jenstroeger/python-package-template/pull/208), we could just use flit build to build both…

Which also raises the question: do we want to handle the source and binary distribution separately? Now that we switched to flit as a build core (see PR #208), we could just use flit build to build both…

I would also prefer not to handle the source and binary distribution separately, and simply use flit build.


Running CLI requires poetry or correct venv

Need to have an environment where the package is installed or run:
$ poetry run {{ project_slug }}

This should now be obvious based on how testing with tox is implemented.


Infinite and grey widget on some devices

I have made an application with flutter, the release apk works fine on my phone. I also installed it on my sister's phone and it works fine but on the customer's phone the second page(showing the login form) display an infinite grey widget (keep scrolling and it doesn't finish). The code of this page(view) can be found here.
My phone is a Tecno and my sister has an Infinx, the customer has a Redmi.
This is how it looks on our phone: image1, and on the customer's phone: image2.

Open Flutter DevTools and save a timeline trace of the performance issue
so we know which functions might be causing it. See "How to Collect
and Read Timeline Traces" on this blog post:
https://medium.com/flutter/profiling-flutter-applications-using-the-timeline-a1a434964af3#a499
Make sure the performance overlay is turned OFF when recording the
trace as that may affect the performance of the profile run.
(Pressing ‘P’ on the command line toggles the overlay.)

The bug is on the release apk and i can not ask ts. the customer's phone to work with it or do tests.
Edit: I sent the APK to friends and the render is as same as the customer's view, they have a Xiaomi and a Huawei.
Target Platform: Android
Devices: Redmi, Xiaomi, Huawei

Hi @Asmitta-01, you are using a third-party package in your sample code. Please provide a complete minimal sample without any third-party packages so that we can investigate this.
Also, if you're not using a service like Sentry or Crashlytics, I would advise that you integrate it into your project so that can you track the errors in production. This would help you to discover the source of issues such as this.
Please also provide the output of flutter doctor -v.
Thank you

@danagbemava-nc  Thanks for the reply.

you are using a third-party package in your sample code. Please provide a complete minimal sample without any third-party packages so that we can investigate this.
I don't understand this please, are you talking about flutx ? All my project depends on it, if i remove it the expected behavior might not be the same.
Please also provide the output of flutter doctor -v.

PS C:\Users\Asmitta\StudioProjects\demo_tdgacademy> flutter doctor -v
[√] Flutter (Channel stable, 3.10.6, on Microsoft Windows [version 10.0.19045.3324], locale fr-FR)
    • Flutter version 3.10.6 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (5 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at C:\Users\Asmitta\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[X] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio (version 2021.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[√] VS Code (version 1.81.1)
    • VS Code at C:\Users\Asmitta\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.70.0

[√] Connected device (3 available)
    • TECNO KG5k (mobile) • 078163722C000563 • android-arm64  • Android 11 (API 30)
    • Windows (desktop)   • windows          • windows-x64    • Microsoft Windows [version 10.0.19045.3324]
    • Edge (web)          • edge             • web-javascript • Microsoft Edge 115.0.1901.203

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

I've installed sentry and i will start debugs, i will have to wait for my friend's responses(since on my phone the app works well).

I don't understand this please, are you talking about flutx ? All my project depends on it, if i remove it the expected behavior might not be the same.

We do not expect your minimal sample to be your actual app. So you can try with a smaller code sample on one of the devices that it reproduces on(if you can get your hands on it) then try removing the flutx package to see if you still get the issue. If the issue does not occur without the flutx library, then you may need to report it on their eind. If it does occur then it is a framework issue.

I've installed sentry and i will start debugs, i will have to wait for my friend's responses(since on my phone the app works well).

Please keep us updated

I modified my main function to this:
// main.dart
Future<void> main() async {
  //You will need to initialize AppThemeNotifier class for theme changes.
  WidgetsFlutterBinding.ensureInitialized();

  AppTheme.init();

  await SystemChrome.setPreferredOrientations([DeviceOrientation.portraitUp]);
  await SentryFlutter.init(
    (options) {
      options.dsn =
          'https://*****.ingest.sentry.io/45***';
      // Set tracesSampleRate to 1.0 to capture 100% of transactions for performance monitoring.
      // We recommend adjusting this value in production.
      options.tracesSampleRate = 1.0;
    },
    appRunner: () => runApp(ChangeNotifierProvider<AppNotifier>(
      create: (context) => AppNotifier(),
      child: const MyApp(),
    )),
  );
}

And i receive nothing in my sentry issues. So th grey widget is not an error ? Or i missed something during the configuration of sentry?

Update
I launch the app on the web, it is not the destination but i wanted to test it.. And i got a sentry issue:
No MaterialLocalizations found.
TextField widgets require MaterialLocalizations to be provided by a Localizations widget ancestor.
The material library uses Localizations to generate messages, labels, and abbreviations.
To introduce a MaterialLocalizations, either use a MaterialApp at the root of your application to include them automatically, or add a Localization widget with a MaterialLocalizations delegate.
The specific widget that could not find a MaterialLocalizations ancestor was:
  TextField
The ancestors of this widget were:
  UnmanagedRestorationScope
  TextFormField
  ...

I think i've found the problem, and because of this error the page is similar to the page of my friends(https://ibb.co/rFWR9HS). The stack trace is very long, it is why it seems infinite on mobile (smaller screen).
I'll try to resolve it.

Latest update:  I remove the part of the code including localizations. The app is in 'fr', i will read more the block about it.
Thanks for your availability, and the tip with Sentry,it is very useful)


number cast to int not double

{
     "totalManPrice": 0.0,
}

for example, i have a json snippet, and when i use command 'From Clipboard to Code Generation Classes', I get the result of
   totalManHour: json['totalManHour'] as int?,

is there some config to default num type as double?

@jinjilynn Hi 👋 Thank you for reporting.
This is known issue becouse 0.0 is equal to 0 and TypeScript read it as double and integer. Generator just prefer integer in this situation. To get double you need just correct to double as in the example below 👇
{
     "totalManPrice": 0.1,
}

I will try to fix it when I I'll have free time.

@jinjilynn Hi 👋 Thank you for reporting.
This is known issue becouse 0.0 is equal to 0 and TypeScript read it as double and integer. Generator just prefer integer in this situation. To get double you need just correct to double as in the example below 👇
{
     "totalManPrice": 0.1
}

I will try to fix it when I I'll have free time.

ok,thank you


Add possibility to "create" new channels

Add the option in "settings.json" to "create" new channels by summation (e.g. new_channel: LFP_R_234, sum_channels: [LFP_R_2, LFP_R_3, LFP_R_4].
Not an imminent priority, but would be nice to have.

Is still a relevant point for you @richardkoehler? If so I could also work on that point.

@timonmerk  Yes, I would say definitely still relevant. Maybe we can work on this next week.

@richardkoehler For my analysis it did not come into a priority so far, in case you think it will be useful, please feel free to add it


put-bucket-policy command creates new bucket instead of changing policy

Describe the bug
The put-bucket-policy command creates a new bucket, and returns an error if the bucket already exists.
Error:
An error occurred (BucketAlreadyExists) when calling the PutBucketPolicy operation: The requested bucket name is not available. The bucket name can not be an existing collection, and the bucket namespace is shared by all users of the system. Please select a different name and try again.

To Reproduce
Steps to reproduce the behavior.

Run versitygw.
Create a policy file containing something like the following:

{
    "Version": "2024-03-04",
    "Statement": [
        {
            "Principal": {
                "AWS": "arn:aws:iam::<id>:root"
            },
            "Effect": "Allow",
            "Action": "s3:PutBucketPolicy",
            "Resource": "arn:aws:s3:::test-bucket-luke-2"
        }
    ]
}

Attempt to put policy with a command like:
aws --no-verify-ssl s3api put-bucket-policy --bucket test-bucket-luke-3 --policy file:///Users/lukemccrone/devel/versitygw/versity-policy.json

Expected behavior
The policy is added and queryable with the get-bucket-policy query.
Server Version
output of
./versitygw -version
uname -a

Version  : v0.14
Build    : a75112f
BuildTime: 2024-03-01_06:54:22PM
Darwin Lukes-MacBook-Pro.local 23.2.0 Darwin Kernel Version 23.2.0: Wed Nov 15 21:54:51 PST 2023; root:xnu-10002.61.3~2/RELEASE_ARM64_T6030 arm64

Additional context
aws-cli/2.15.10 Python/3.11.6 Darwin/23.2.0 exe/x86_64 prompt/off

Fixed with #436


Move to Cm version 9.0.3 with updated executable name

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This merge request updates to cm version 9.0.3 which changes the name of the NameServer executable to CmNameServer.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Engineify

I've undertaken this work for @iiman, after previously discussing the health checks for the application with her. She wanted me to do some work to split the health checks in two: one for the critical things and another page for the delayed job message. This is so that we do not get alerted when DJ is rebooting on a deploy.
I have accomplished this by allowing for filtering of the checks using URLs such as /healthy.html?except=delayed_job for the regular healthcheck, and /healthy.html?only=delayed_job for the DJ check.

The code for this little gem was all wrapped in a piece of middleware. To clean it up, I have converted the code into an engine. So now the logic lives across controllers + views just like a normal Rails application. This makes the code much more legible and removes any need for doing query parsing using Rack methods.
I've also swapped to testing the functionality of the app using RSpec, with an embedded dummy application. It's just a little bit neater too.
I've altered register so that you register probes with a name now and you can filter by them using that name (see above). This makes more sense for our use cases than filtering by levels.
Overall, it behaves + looks exactly like the health check that we all know and love... It's just got a fresh coat of paint.

 Investigate about Pingdom alert after some deployment re:assets down


Add a Java function

The function is used to generate a screenshot of a website in Base64 format.

Hi @ShizhanLiu can you add some screenshots to show the working of the example? You can just edit your PR message and add it there.