PureRenderMixin

Hi,
I just thought of that you should include PureRenderMixin with your other mixins, for tree and cursor. That way components does what you state in the documentation, only renders themselves :-)
This would be a great addition!

Yes I might do that but I do not want this library to include react as a dependency to do so. I don't see any other option right now than to let the user pass the mixin as an instantiation option.

The mixin itself is really easy to recode however (two lines of mixins and a shallow comparison function).

Good point! But what if you allowed it as an option?
var baobab = new Baobab(

  // Initial data
  {
    palette: {
      name: 'fancy',
      colors: ['blue', 'green']
    }
  },

  // Options
  {
    maxHistory: 5,
    clone: true,
    mixins: [React.PureRenderMixin]
  }
)

This would allow for users to attach other default mixins too.

Nice. I'll consider this asap.


Preview thumbnails and videos not displayed

Description
Within the Power App, the thumbnails and videos are not displayed:

Steps to reproduce

Start the app
Fill out first page requirements
Check collaboration spaces

Expected results
Preview thumbnails and videos are displayed
Actual Results
Preview thumbnails and videos are not displayed
Solution component
Power App
Operating system (environment)
Windows
Additional Info
No response

@wapster182 did you edit the app and reconnected the data sources to the environment variables again? If you miss this step then this happens.

Thank you, it works now! My mistake...

Weird that it was only affecting the Images. Well, nevermind.


problem restarting a group subscriber

I have a group subscriber that wants to consume 10k messages, then stop, then some time later listen for another 10k messages.
It seems that calling brod_group_subscriber:stop( Pid ) is leaving the consumer in some unusable and not restartable state. When the resubscribe happens, I get something like the following:
(I am using brod from Elixir, but I can't image that would have an effect)
iex(6)> {:ok,pp} = Support.Tstore.Transaction.Source.start( Support.Tstore.Transaction.FractionEmitter, "ttest" )
{:ok, #PID<0.8887.0>}
iex(7)>
14:10:32.680 [info]  Group member (ttest,coor=#PID<0.8888.0>,cb=#PID<0.8887.0>,generation=0):
connected to group coordinator kafka-server-2:9092

14:10:33.331 [info]  Group member (ttest,coor=#PID<0.3045.0>,cb=#PID<0.3043.0>,generation=45):
re-joining group, reason::RebalanceInProgress

14:10:42.681 [info]  Group member (ttest,coor=#PID<0.8888.0>,cb=#PID<0.8887.0>,generation=0):
failed to join group
reason::timeout

14:10:42.681 [info]  Group member (ttest,coor=#PID<0.8888.0>,cb=#PID<0.8887.0>,generation=0):
re-joining group, reason::timeout

14:10:52.681 [info]  Group member (ttest,coor=#PID<0.8888.0>,cb=#PID<0.8887.0>,generation=47):
elected=true

14:10:52.683 [info]  Group member (ttest,coor=#PID<0.8888.0>,cb=#PID<0.8887.0>,generation=47):
assignments received:
  transaction:
    partition=0 begin_offset=54806
    partition=1 begin_offset=52013
    partition=2 begin_offset=55609
    partition=3 begin_offset=53955
    partition=4 begin_offset=48756
    partition=5 begin_offset=52280
    partition=6 begin_offset=50178
    partition=7 begin_offset=42360
    partition=8 begin_offset=44805
    partition=9 begin_offset=44236

But no messages ever come in. I can restart the brod application/client, then process proceeds as it should. This tells me something isn't getting cleaned up properly.  I've been looking through brod and the tests for groups which seem to just use stop.
In case this matters, the handle_message function is the place where stop is being called, since it's using the state to keep track of the number of messages consumed.
Hints on what it could be or where I should look?
Many thanks.

Hi @danj3
I will try to reproduce it.
But are you sure the stop call was made in handle_messages callback?
Because that should cause a deadlock.

Ah! That's exactly what it was @zmstone. Since stop() does the monitor/receive, it was in fact deadlocked. In my handle_message I now put gen_server:cast( Pid, stop ) and now it's good. Is what I've done the right solution? I'm curious, why stop is written to block?

It is a blocked call because it is a blocked call by its look.
It's to avoid race when a restart is triggered right after stop.
I guess what we need here is to introduce a new return code {ok, stop, NewState}.


Build and IDE guides

This breaks the old Contributor's Guide into separate guides for building Cesium and setting up an IDE.
The "new" Build Guide is the same as before just with these tweaks: f56ac390bce1ae7075cb3b857426e1fc1f42fae0
Does anyone have a one-liner for:

Delete all branches expect master

TODO

[x] WebStorm and Eclipse IDE Guides
[x] Update wiki to point to new guides

Delete all branches expect master

There's no good cross-platform one-liner for this, I recommend suggesting people use the GitHub GUI by going to the all branches page for the fork (i.e. https://github.com/AnalyticalGraphicsInc/cesium/branches/all for cesium itself).

Seems like the guide would be harder to maintain if it's broken up, but I'm okay with it if everyone else is.

This is ready.
Added the WebStorm Guide and Eclipse Guide.
These are largely untouched except for minor wording tweaks.

This is ready.

Made a few tweaks to CONTRIBUTING.md in this branch.  I don't plan to change anything else in this PR.

Looks good!  Thanks @pjcozzi


App.Metrics.Reporting.Socket.Client.SocketSettings validates address incorrectly

In the validation of SocketSettings.cs, an endpoint is validated by creating it with string endpoint = $"{address}:{port}";
In the Endpoint getter however the endpoint is created by using {protocol}://{address}:{port}.
This means that entering an ip address as the address wil result in an invalid operation exception, entering tcp://[ip] will pass this validation, but wil result in the endpoint tcp://tcp://[ip], which does not work.

I think it is related to here if I enter the following in overUdp the validation is failing
builder.Report.OverUdp(new MetricsInfluxDbLineProtocolOutputFormatter(), "<IP_ADDRESS>", 4444);

It fails in the SocketSettings.cs in line 69
<IP_ADDRESS>:4444 must be a valid absolute URI

duplicate https://github.com/AppMetrics/AppMetrics/issues/549


Updating docs based on change in OSS PR 13581

Description

Updating the docs for a change in Kong OSS, updating the correlation ID priority order

https://github.com/Kong/kong/pull/13581
Testing instructions
Preview link:
Checklist

[ ] Review label added
[ ] Conditional version tags added, if applicable.

review:copyedit

review:copyedit

@andrewgkew thanks for opening this PR! This list is generated via a script though, so you don't need to update it manually. Whenever your change gets merged and goes into a release, this page will get updated then (and I'll remove the note).

Closing this as it should be (mostly) automated on our side.


Secondary updates to static site for results uploader messaging

Related Issue or Background Info
Related to Issue #4579, this issue adds other updates to the static site around the bulk results uploader.
Changes Proposed

Highlighting "What's new" page on Using SimpleReport landing
Removing "rapid" from messaging in "Getting started" section
Adding subheader for organizations and facilities in "How to sign up"
Making it clear you can report to places on SR list (not just be located in them)
Add note about results uploader in "Test devices we support"

Additional Information
See Mural audit
Checklist for Author and Reviewer

[ ] Copy is grammatically correct and in plain language
[ ] Changes don't break and links or designs

Screenshots
Updated "Getting started landing

Updated sign-up page

Updated note on test devices page

**Language change on where SR is live **

Link to "what's new" on Using SimpleReport landing

Design

[x] Any UI/UX changes have a designer as a reviewer, and changes have been approved
[ ] Any large-scale changes have been deployed to test, dev, or pentest and smoke-tested by both the engineering and design teams

Content

[x] Any content changes (including new error messages) have been approved by content team

Support

[x] Any changes that might generate new support requests have been flagged to the support team

@lauraykerr could you resolve the merge conflicts? after that I can merge main into this branch so we can have a clean diff

@lauraykerr could you resolve the merge conflicts? after that I can merge main into this branch so we can have a clean diff

@mehansen Pairing with Bob in a bit to resolve this — thanks for taking a look at this!


How to access https://github.com/StevePny/DataAssimBench?

Hello,
I am interested in the benchmark library used in this GitHub project.  I noticed that your code seems to be using DataAssimBench (https://github.com/StevePny/DataAssimBench), but I am currently unable to access it.  Would it be possible for you to provide some information about how you are using it and how it has helped you in your project, or perhaps suggest an alternative way for me to access the library?  I am curious to learn more about this library and would appreciate any insights you can provide.
Thank you!

Hi @wuxinwang1997 the DataAssimBench is now public:
https://github.com/StevePny/DataAssimBench
We are preparing a publication to describe its use and potential applications.


building on v0.11

i made a testbuild on travis on node v0.11.3 which failed.
i did not investigate further, maybe that will be an issue when 0.12 is released.

@kelly when do you think this will be complete? We still can't install cylon-i2c on any version of Node greater than 0.10.x .

Looks like https://github.com/kelly/node-i2c/issues/69 is a duplicate of this issue.

@bchr02 the current i2c version works on 0.12.9 and under.

@bchr02 the current i2c version works on 0.12.9 and under.

Thanks but when will it work with 0.4.x and 0.5.x?

@bchr02 since it requires a full re-write of the c++ code, it's not an immediately a priority for me. If anyone wants to take a shot at it, submit a pull request using NAN and i'll merge it in.

@EMCP this is closed, so your issues aren't related to this bug.

I'm just trying to get the status of this project running on versions beyond .12

strangely, the project that used this library was on an older version 0.1.1 .. after upgrading the errors have dropped away on my platform.


Updated for build 52124

FYI I added 'replay.ReplayVersionMajor', which will be '2' for HotS 2.0 replays, and '1' for everything else
This may be helpful for your code too since there may be a couple more patches with a higher build number, but still not have the 2.0 update


I'd like to help support and improve Slate-Angular

Hello! I have a large project where I need to invest in a rich text editor and I really appreciate how you've approached SlateJS.
Perhaps your stakeholders would allow me and my engineers to create PRs to bring Slate-Angular up to date with Slate and Angular's latest features?
Regarding Standalone Angular Components
I have already converted the package to standalone Angular components and would like to create a PR for your team to review.
My entire app is standalone and must import standalone components.
As you may know, standalone components are supported within modules. However, without standalone support developers like myself will not be able to use SlateJS.
I have refactored the package and demo app to fully standalone but would need permission on the repo to push.
There is an error regarding packages/src/view/container-item.ts:78 that I would need help with.
ERROR TypeError: componentFactory.create is not a function

https://github.com/worktile/slate-angular/pull/236

@BenRacicot The master branch already supports standalone components.
How to reproduce the following errors？
ERROR TypeError: componentFactory.create is not a function

@BenRacicot I am pleased to hear that you are using slate-angular. However, our team is upgrading to Angular 16 and will also support standalone. Any other PRs are welcome. I have invited you to this repo for write permission and look forward to your contributions. Good luck!

That's excellent! Thanks so much for including me.


There is a vulnerability in Quartz Enterprise Job Scheduler 2.2.3 ,upgrade recommended

https://github.com/apache/incubator-dolphinscheduler/blob/aa0974fd1f759e96430d3f1b8dac291d6ea7388c/pom.xml#L67
CVE-2019-13990
Recommended upgrade version：
2.3.0-3

https://github.com/apache/incubator-dolphinscheduler/blob/aa0974fd1f759e96430d3f1b8dac291d6ea7388c/pom.xml#L67
CVE-2019-13990
Recommended upgrade version：
2.3.0-3

why? what's the difference?

Thank you so much for your feedback

How to modify:
Modify the corresponding jar version in the incubator-dolphinscheduler/pom.xml.
Modify the version of the corresponding jar in tools/dependencies/known-dependencies.txt.
Modify the relevant jar version in the dolphinscheduler-dist/release-docs/LICENSE file.

i want to fix it

Looking forward to your submission of pr, if you encounter any problems, you can leave a message on this issue.


Minified link on Parallel.js homepage returns 404

This link on the project homepage doesn't seem to be working for me right now. (D'oh!) No biggie, I'll just get it from NPM, but I thought I'd bring to your attention. Thanks for the cool utility, I'm excited to try it.

This project is pretty much deprecated.
Check the graphs:
https://github.com/adambom/parallel.js/graphs/contributors

@amilajack Good call. Are there any libraries you could recommend with a similar feature set? In particular, I'd love to be able to execute map/reduce/filter operations in parallel (when possible).

I would recommend:
https://github.com/steelbrain/Worker-Exchange
I haven't personally used it but its tests are passing.

Terrific suggestion. Thanks!


Python 3.7 No longer supported

Solution currently using a no longer supported version of Python runtime. Needs to be updated to a supported version. Probaly needs to be combine with  #25 .
./cfct/templates/vpc_flowlog_automation_in_hub.template#L227

Hi, I am also getting the same error on my end. ... "Resource handler returned message: "The runtime parameter of python3.7 is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (python3.12) while creating or updating functions.  This is on deploying/setting up the lifecycle event part.

thanks for raising this issue and I apologize for long delay. I am taking a look at this issue now.


Question: About changing the minute interval of the TimePicker

Hi, thanks for the great library.
I have a question about TimePicker.
When selecting minutes, I assume the value changes by 1. Is it possible to change this by 15 minute increments or even 30 minute increments?
I am currently developing a schedule adjustment app and would like the values to change in 15 or 30 minute increments, like Google Calendar, rather than minute-by-minute changes.
If others have a need for this, please consider developing it.

The ability to advance time in 15-minute and 30-minute increments is no longer necessary.
Therefore, this Issue and Pull Request are closed.

The documentation is here (for later use)
https://shahabyazdi.github.io/react-multi-date-picker/plugins/time-picker/#utilizing-step-for-arrow-icons

@shahabyazdi
Oh!It was already there.
Excuse me, I could't find it.
Thanks for the reply!


Isolates support

Hey @davidmartos96, do you mind reviewing this? Are there any issues I overlooked?
I basically refined your PR and added a unit test.

@felixblaschke LGTM, thank you!

Alright... I published 1.2.0 with your isolates contributions: https://pub.dev/packages/shelf_plus/versions/1.2.0#multithreading
Thank you!


WIP Adding istioctl and upgrade release qualification

These were moved from https://github.com/istio-releases/daily-release so they can be run and tested more easily.
Also refactored the e2e test logic in lib.sh for reusability.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

prow/OWNERS [hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-mixer-no_auth.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-mixer-no_auth.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-mixer-no_auth.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-mixer-no_auth.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-mixer-no_auth.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-mixer-no_auth.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-multicluster-e2e.sh
9ac45dfd5bee2e2525feffae4455496e22124939
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
aaac72b1151bc1292dbe22e1671c32b94e525e40
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add seperate clock class

Basically a new game clock class representing CClock in the exe, whose implementations (our code, not the game) share with clock stuff in World class, but with proper read/write access to the date time. No month truncation happening.
I took the the idea of MonthZero from month0 of the chrono crate for Rust.

Clock.Year's documentation was incorrect, not Clock.GetDayOfWeek. Try modifying the year to 0, and the day will be 1985, which will happen when the date put was out of the range.


polars plugins tutorial

What does this PR do?
Fix #242 by adding the awesome guide on how to write polars plugins by MarcoGorelli.
Requirements for your Awesome item list

[X] Only has awesome items. Awesome lists are curations of the best, not everything.
[X] Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
[X] Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
[X] Has consistent formatting and proper spelling/grammar.
- The link and description are separated by a dash.
Example: - [AVA](…) - JavaScript test runner.
- The description starts with an uppercase character and ends with a period.
- Consistent and correct naming. For example, Node.js, not NodeJS or node.js.
[X] Doesn't use hard-wrapping.
[X] Your item(s) fit into an existing sections. (if you think a new section is needed, please explain why your item does not fit into an existing one and what would be the interest of your new section.
[X] Add your new item to the bottom of the list in a section.
[ ] If a duplicate item exists, discuss why the new item should replace it.
[X] Check your spelling & grammar.
[X] The item must follow this format:- [item name](https link) - Description beginning with capital, ending in period.

[ ] Be sure to have read the contributing guidelines.

Great idea, I thought I'd add it today. Thanks a lot!


Stop when pdf is closed

Description
When running
typst watch test.typ --open $READER

If I close the pdf, typst continues running.
I would like an option to get the compiler to stop if the reader is closed.
Use Case
It would allow to run something like
nohup typst watch test.typ --open $READER > /dev/null 2>/dev/null &

without having to worry about killing the right typst process after we are done.

I'm not sure whether this would really be the expected behaviour.

Maybe it is not the expected default behaviour, but it would be nice to have an option that works with --open that stops typst when the pdf is closed.

In testing this issue I have yet to find a reader that, using watch and --open, will automatically update when a new version has been generated. There might exist some reader that does update automatically, but since there is already a sort of disconnect between the typst process updating the file and the reader (since it might not be displaying the latest version), I don't think that this is a problem that necessarily needs a solution.

This works in Zathura works for me

In testing this issue I have yet to find a reader that, using watch and --open, will automatically update when a new version has been generated. There might exist some reader that does update automatically, but since there is already a sort of disconnect between the typst process updating the file and the reader (since it might not be displaying the latest version), I don't think that this is a problem that necessarily needs a solution.

Sumatrapdf works on windows

I'm not sure this is actually possible. But even if, it still feels a bit too niche to carry its weight as a feature. I think the real solution is using an integrated environment like VS Code or the web app.


[1.19.2] Incompatibility with 'Snow! Real Magic!' GameRules

So I seem to be getting a launch failure and config wipe when setting any default gamerule for Snow! Real Magic! The following gist is the relevant information printed by the log during launch when setting blizzardFrequency to 0.

Consider this a duplicate of #3, you've used a number value and that unfortunately turned out to make things go haywire; This will be fixed on an upcoming 1.19.4 and 1.19.2 update


value() arguments

The value() function is supposed to return an unbiased estimate of the value of executing a policy from a belief state. One way to do this is to perform a rollout simulation.
POMCP uses value estimates based on the actual state in addition to the belief state. i.e. value(policy, b, s) should return an estimate of the reward accumulated by executing policy on belief b given that the actual state is s.
Does it make sense to add value(::Policy, ::Belief, ::State) to the interface? or should this just be a POMCP-specific thing?
Also, to perform a rollout, one needs a belief updater. Should the belief updater be an argument to value(), or should the updater be created from the policy via the updater() function?

I can see the utility of having value(::Policy, ::Belief, ::State). As for the belief updater, my preference would be to supply the updater as an argument to value(). This way a non-default updater can be used or the default updater can be configured with optional parameters before being used for rollouts.

Would having State as an optional argument make sense for this type of function? If state is not passed, you can sample your initial state from the belief. If the user wants to specify a starting state then they can.
I vote against adding the belief updater into the function call. The belief updater can be passed with the policy type if needed. This will keep the interface cleaner in my opinion.

I agree with @etotheipluspi.

I think we settled on the form value{S}(::Policy, ::S)


System chart should pull images from the cluster-level registry

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

run Rancher v2.3.4-rc8 single installed in an air-gapped env or upgrade from v2.3.3
add a global registry
add a custom cluster with a cluster-level registry
enable the cluster monitoring / project monitoring / external DNS / logging / alerting / Istio in the cluster

Result:

the images of system charts are pulled from the global registry

Other details that may be helpful:

we expect to see the images  are pulled from the cluster-level registry

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.3.4-rc8
Installation option (single install/HA): single install

@jiaqiluo  Does it work in 2.3.3? there is no changes in 2.3.3 for monitoring.

@loganhz the global registry is the system-default-registry. in v2.3.3 it also pulls images from the system-default-registry
what is the behavior if only the cluster-level registry is set, will it pull images from the cluster-level one?

Pushing out to 2.4. Need discussion with @cloudnautique on global registry vs. cluster registry

@jiaqiluo can you review the docs changes and close this issue?

The PR https://github.com/rancher/docs/pull/2203 LGTM.
this issue is closed.


Need Examples on AdminDirectory.Orgunits.update

Hi there,
I've spent a better part of the day trying to figure out the correct JSON body and arguments to update an OU; I'm actually trying to rename an OU. I'm close but in any case, the solution has so far escaped me.
I've referenced these docs so far:

https://developers.google.com/admin-sdk/directory/v1/reference/orgunits/update#http-request
https://developers.google.com/resources/api-libraries/documentation/admin/directory_v1/python/latest/admin_directory_v1.orgunits.html#update
Here's the logger output thus far:

[19-06-17 17:39:39:165 PDT] /Users/200 COGS/Education
[19-06-17 17:39:39:166 PDT] my_customer, id:03ph8a2z39wdr3v, {"kind":"admin#directory#orgUnit","parentOrgUnitPath":"/Users/200 COGS","name":"255 Education","orgUnitPath":"/Users/200 COGS/255 Education","parentOrgUnitId":"id:03ph8a2z1lakohp","blockInheritance":false,"orgUnitId":"id:03ph8a2z39wdr3v"}
[19-06-17 17:39:39:166 PDT] my_customer, id:03ph8a2z39wdr3v, {"kind":"admin#directory#orgUnit","parentOrgUnitPath":"/Users/200 COGS","name":"255 Education","orgUnitPath":"/Users/200 COGS/255 Education","parentOrgUnitId":"id:03ph8a2z1lakohp","blockInheritance":false,"orgUnitId":"id:03ph8a2z39wdr3v"}
[19-06-17 17:39:39:198 PDT] {"message":"API call to directory.orgunits.update failed with error: Parse Error","name":"GoogleJsonResponseException","fileName":"GSuiteOrgUnits","lineNumber":573,"stack":"\tat GSuiteOrgUnits:573 (renameOU)\n\tat GSuiteOrgUnits:600 (test_renameOU)\n","details":{"message":"Parse Error","code":400,"errors":[{"domain":"global","reason":"parseError","message":"Parse Error"}]}}
Here's my test function thus far as well:
/* Testing Renaming of an OU */
function test_renameOU(){
/* Args:
customerId: string, Immutable ID of the G Suite account (required)
orgUnitPath: string, Full path of the organizational unit or its ID (required) (repeated)
body: object, The request body. (required)
The object takes the form of:
{ # JSON template for Org Unit resource in Directory API.
"kind": "admin#directory#orgUnit", # Kind of resource this is.
"parentOrgUnitPath": "A String", # Path of parent OrgUnit
"name": "A String", # Name of OrgUnit
"etag": "A String", # ETag of the resource.
"orgUnitPath": "A String", # Path of OrgUnit
"parentOrgUnitId": "A String", # Id of parent OrgUnit
"blockInheritance": True or False, # Should block inheritance
"orgUnitId": "A String", # Id of OrgUnit
"description": "A String", # Description of OrgUnit
}
*/
/* Function to perform rename */
function renameOU(customerId, orgUnitPath, body){
Logger.log(customerId + ", " + orgUnitPath + ", " + JSON.stringify(body))
try{
var org = AdminDirectory.Orgunits.update(customerId, orgUnitPath, body)
}catch(e){
Logger.log(JSON.stringify(e));
}
}
/* Arguments */
var customerId = 'my_customer';
var oldOUname = "Education";
var parentOrgUnitPath = "/Users/200 COGS";
var orgUnitId = "id:03ph8a2z39wdr3v";
var orgUnitPath = parentOrgUnitPath + "/" + oldOUname;
var parentOrgUnitId = "id:03ph8a2z1lakohp";
var newOUname = "255 Education";
Logger.log(orgUnitPath);
var body = { //# JSON template for Org Unit resource in Directory API.
"kind": "admin#directory#orgUnit", //# Kind of resource this is.
"parentOrgUnitPath": parentOrgUnitPath, //# Path of parent OrgUnit
"name": newOUname, //# Name of OrgUnit
"orgUnitPath": parentOrgUnitPath + "/" + newOUname, //# Path of OrgUnit
"parentOrgUnitId": parentOrgUnitId, //# Id of parent OrgUnit
"blockInheritance": false, //# Should block inheritance
"orgUnitId": orgUnitId, //# Id of OrgUnit
}
/* Call Rename Function */
Logger.log(customerId + ", " + orgUnitId + ", " + JSON.stringify(body))
renameOU(customerId, orgUnitId, body)
}
Oh, I have also posted about this on MacAdmins Slack, in case you are on there?
https://macadmins.slack.com/archives/C9H76PLNN/p1560815072006400
Thanks in advance for any help and additional info you can add to the docs!

Posted on Stack Overflow as well" https://stackoverflow.com/questions/56657176/question-about-admindirectory-orgunits-update-method

I've posted a solution to Stack Overflow. We likely won't be adding per-method code snippets for Apps Script, so closing this issue.


not calling callback

Future beforePrintHello() async {
print("Line253");
final int helloAlarmID = 0;
print("Line255");
await AndroidAlarmManager.initialize();
print("Line257");
await AndroidAlarmManager.periodic(
const Duration(seconds: 3),
helloAlarmID,
printHello,
);
}
void printHello() {
print("Line246");
final DateTime now = DateTime.now();
final int isolateId = Isolate.current.hashCode;
print(
"Line240 [$now] Hello, world! isolate=${isolateId} function='$printHello'");
}

Hello @prabhunath111 👋,
Thanks for opening an issue.
Have you followed the steps from the documentation here? You have to add some configurations to the native platforms, as mentioned in the documentation.
Otherwise, it would be helpful if you share a link to your repository as this is not enough code to reproduce the issue you are having.
Hope that helps!

Is the printHello() function a top level function? It won't work if it isn't.

My callback also doesn't work.
Here is my code repo: https://github.com/rulim34/restologue_flutter

Flutter Doctor
[√] Flutter (Channel stable, 2.2.3, on Microsoft Windows [Version 10.0.22000.132], locale id-ID)
    • Flutter version 2.2.3 at C:\Users\Rulim\AppData\Local\Programs\Flutter\flutter
    • Framework revision f4abaa0735 (7 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Rulim\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.1.0)
    • 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.10+0-b96-7249189)

[√] IntelliJ IDEA Community Edition (version 2021.2)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition
    • Flutter plugin version 59.0.4
    • Dart plugin version 212.4746.57

[√] VS Code (version 1.59.0)
    • VS Code at C:\Users\Rulim\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.25.0

[√] Connected device (3 available)
    • Redmi 5A (mobile) • f1aed3be7d05 • android-arm64  • Android 10 (API 29)
    • Chrome (web)      • chrome       • web-javascript • Google Chrome 92.0.4515.159
    • Edge (web)        • edge         • web-javascript • Microsoft Edge 92.0.902.67

• No issues found!

My callback also doesn't work.
Here is my code repo: https://github.com/rulim34/restologue_flutter
Flutter Doctor

Problem solved. I think you have to mention that the callback should be static function clearly on the doc.


[core-client] set custom headers after serializing headers

so that they can properly override existing headers.

This is a great fix. Must have been introduced when serialization moved out into its own policy


Purpose new option: [View] - Scroll to next END process

Hello Dev
I switch from Process Hacker Nightly to System Informer with pleasure.
Will the ProcessHacker_2.39_R124 project also be renamed to System Informer? In any case, these are utilities that I cannot do without :)
I would like to propose an option that would facilitate real-time monitoring of a program that stops a subroutine and launches a new one, for example. So as said in the topic "[View] - Scroll to next END process". It seems to me that it is as important to know what starts as what stops.
Awaiting your return,
regards,
Nico

Scroll to next END process

Isn't this the same as the current option in the View menu > Scroll to new processes?

thank you for your return.
Unfortunately not, the "Scroll to new processes" option only goes to a starting process, not a terminating process.
Nico

Scroll to next END process

Isn't this the same as the current option in the View menu > Scroll to new processes?
--
Reply to this email directly or view it on GitHub:
https://github.com/winsiderss/systeminformer/issues/1487#issuecomment-1323144517
You are receiving this because you authored the thread.
Message ID: @.***>

Keep showing terminated processes for more seconds? or this is no sense.

I probably wasn't explicit enough... Is it clearer for you?

Yes, I got it now. I did not notice this setting before. The feature you requested maybe a good idea.

Ok, j'espère que vous l'implémenterez prochainement :)

oops! Ok, hope you implement it soon :)

ScrollToRemovedProcesses to 1

Excuse me, I'm not comfortable in the github environment.
"Yes, I got it now. I did not notice this setting before. The feature you requested maybe a good idea."
((>Can I hope that my "good idea" will be integrated in the next version Nightly Build ?


GPU/NPU Support?

Hi,
You wrote in this projects readme that gpu as well as npu support may not be possible without going into detail on that assumption. I was wondering if there's any reasoning behind this/any technical blockers for supporting the platforms npu/gpu?
Cheers

Good question, let me explain it:

The GPU driver of TH1520 is closed source, and it's high coupling with the operation system, make it quite difficult to make it work on NixOS.
As for NPU, no one has seem to be done any work related to it yet, and even licheepi4a's official docs does not contain any relevant contents(only a TODO).

Therefore, it may be not possible for me to fix these two drivers.


Don't make issues for JS and TXT files on Sider.

🛠 Fixes Issue

Related Issue/Addition to code

As, I'm getting not any genuine issues from JS and TXT files and they are mainly clogging out the issues tab, I want to set it to not display any issues for JS and TXT files on Sider.

👨💻 Changes proposed

Ignore JS and TXT files from showing as issues on Sider

Type of change

[ ] This change requires a documentation update
[ ]

📄 Note to reviewers

@TechStudent11 @aliraza944 Do you think it's ok to be to set it to not create issues for JS, cause rn, I'm getting weird issues as displayed in #285 , but if there's a genuine misspselling or something, it won't display the error.
I want to do txt files for sure, as it's giving errors like "gonna" is informal, etc.. But txt files aren't code, their just put in the repo so users can down a txt file of the lyrics.

@TechStudent11 @aliraza944 Do you think it's ok to be to set it to not create issues for JS, cause rn, I'm getting weird issues as displayed in #285 , but if there's a genuine misspselling or something, it won't display the error.

@TechStudent11 @aliraza944 Please get back to me asap on this.

*misspelling
Sure, I guess.

*misspelling

LOL, was typing fast.


Not able to disable columns_panel

Issue:
Not able to disable columns_panel with configure_side_bar.
Why?
This is because https://github.com/PablocFonseca/streamlit-aggrid/blob/main/st_aggrid/grid_options_builder.py#L168 overrides the value passed in the parameter in https://github.com/PablocFonseca/streamlit-aggrid/blob/main/st_aggrid/grid_options_builder.py#L145

Any progress on this?


Cone templates automatically snap to corner and can't be placed orthogonally anymore

When I use the "place cone template" option from the chat after using a spell the template will always snap automatically to one of the nearest corners, and ignores orthogonally placement.

This also seems to affect line and emanation templates - the snap to corner effect should only affect bursts according to https://2e.aonprd.com/Rules.aspx?ID=354 and to the areas diagram https://2e.aonprd.com/Images/Rules/Rules354.png (although for the orthogonal templates it is not clear since the diagram shows differences between the 15 feet and larger templates)

I played around a bit and found out you can get the orthognal template while hovering over a corner with the template and use Ctrl+scroll


Is it possible to avoid a warning for exports used only in the same file?

With --include exports, I get warnings for exports which are used in the same file they are defined. But then they (and their dependencies) aren't dead code and removing export isn't a benefit. Can Knip be configured to only give warnings on completely unused exports?

Yes, sorry.


Attempting to create a project from a submission with a decimal value results in an error

When attempting to create a project from an accepted submission that has Requested Funding with a decimal value (ie. 5478.50),  the error Field 'value' expected a number but got <decimal funding value> is produced.
I believe this issue lies within the project model only accepting a positive integer.

I can take a stab at this one

We have earlier taken measures to only allow int values here. But since you have seen flots it is not working 100%.

So this should be an integer across the board? I've seen quite a few existing submissions on production that had cents in the requested funds

My comment only refer to the "Requested amount" custom field. I know OTF do not use that mush anymore, opting for a generic field instead. Do not now/remember the reasoning around that.
But values in generic fields are not copied over to projects so should not matter.


fix(base-path): fix base path replacement in index.html file

Description

Fixes the regex to replace the basePath even if the <base> is not defined at the beginning of the string (which it is not)

:tada: This PR is included in version 4.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Ignore any CSPDEBUG queryparam (fixes #8)

This PR fixes #8 so Serenji can debug REST endpoint code.

@gjsjohnmurray, I think it would be better to centralize handling of this in OnPreDispatch (or something like that) to make sure we can actually debug all endpoints.

Also - @gjsjohnmurray just to set expectations, I'll look at this next week. Where this is a copy of an internal project changes will go through an internal-to-InterSystems review process and we won't usually merge PRs directly. (see: https://github.com/intersystems/isc-rest/blob/main/CONTRIBUTING.md)

Sounds good @isc-tleavitt


Ana Dobrescu minesweeper game

Your tests fail.

You said in your report that the first move should never result in blowing up, but it did the first time I tried:
Board Characters and Their Meanings:
  * : Unrevealed cell.
  . : Revealed safe cell with no adjacent mines.
  X : Revealed mine (you lose if you see this!).
  F : Flagged cell (you think there's a mine here).
  1-8 : Revealed safe cell with the number of adjacent mines.

 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
Enter your move (e.g., 'R row column', 'F row column', or 'Q' to quit):
> R 3 5
YOU EXPLODED! GAME OVER!!!
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  X  *  *  *  *
 *  *  *  *  *  X  *  *
 X  *  *  *  *  *  *  *
 *  *  *  *  *  *  X  *
 *  *  X  *  *  *  *  X
 *  *  *  *  *  X  X  *

Your revealing function doesn't reveal everything it should reveal. If a user hits an empty cell, all adjacent empty cells should be revealed, as well as all numbers on the boarder.
Enter your move (e.g., 'R row column', 'F row column', or 'Q' to quit):
> R 0 0
 .  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  2  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
Enter your move (e.g., 'R row column', 'F row column', or 'Q' to quit):
> R 0 1
 .  .  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  2  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *
 *  *  *  *  *  *  *  *


Verify MSI support on Azure Spring Cloud

There is a com.azure.identity.AppServiceMsiCredential and ServiceFabricMsiCredential but no AzureSpringCloudMsiCredential, confirm whether we support MSI on Azure spring cloud.

Closing this issue because current aad-b2c already set http header.

Hi, @saragluna , maybe this issue is not only related to aad-b2c?

it should be an azure-identity issue, @saragluna would you open another one and assign to the azure-identity team

Opened https://github.com/Azure/azure-sdk-for-java/issues/23548.


Move dropdown menu in burger below button

Move the dropdown menu below the button with an arrow pointing upward to the button from which it originated.  (See archestration project for reference)

Done.


WebApplicationException should be formatted

In my eyes, the result of thrown WebApplicationException should be formatted like other unhandled bubbled exceptions, in LoggingExceptionMapper. Currently they are just passed through:
        if (exception instanceof WebApplicationException) {
            return ((WebApplicationException) exception).getResponse();
        }

Any WebApplicationException which was thrown with a response entity set (eg. through throw new WebApplicationException(Response.status(..).entity(...)...)) will not be passed to any ExceptionMapper: it's just processed in the way the thrower intended.
However, this leaves WebApplicationExceptions which were thrown with a message, but without any entity (eg. throw new WebApplicationException("not found", 404)). These end up in LoggingExceptionMapper, in the code branch shown above.
The problem is that even though a message was specified, the returned Response does not have any entity data. That means the error will end up in Jetty, which will just produce a generic HTML page for the associated HTTP status code, without the message.
It would be nicer to format these exceptions in the same other unhandled exceptions are formatted (like {"code": 404, "message": "not found"}).
Even if someone throws WebApplicationException with some specific format (eg. a text/plain entity), it will not be re-formatted, because these are not passed to the ExceptionMapper. And finally, if someone wants a different for all WebApplicationExceptions, they can just implement their own ExceptionMapper<WebApplicationException> because the mappers are chosen by most specific first.

I made a pull request for this: https://github.com/dropwizard/dropwizard/pull/1193

Closed by #1193


Why Mistral Nemo notebook uses alpaca template?

Open colab

It doesn't really make any sense. Why is the Alpaca template used by default? Mistral Nemo has the following chat template and should be used instead.
"{%- if messages[0]['role'] == 'system' %}\n    {%- set system_message = messages[0]['content'] %}\n    {%- set loop_messages = messages[1:] %}\n{%- else %}\n    {%- set loop_messages = messages %}\n{%- endif %}\n\n{{- bos_token }}\n{%- for message in loop_messages %}\n    {%- if (message['role'] == 'user') != (loop.index0 % 2 == 0) %}\n        {{- raise_exception('After the optional system message, conversation roles must alternate user/assistant/user/assistant/...') }}\n    {%- endif %}\n    {%- if message['role'] == 'user' %}\n        {%- if loop.last and system_message is defined %}\n            {{- '[INST] ' + system_message + '\\n\\n' + message['content'] + '[/INST]' }}\n        {%- else %}\n            {{- '[INST] ' + message['content'] + '[/INST]' }}\n        {%- endif %}\n    {%- elif message['role'] == 'assistant' %}\n        {{- ' ' + message['content'] + eos_token}}\n    {%- else %}\n        {{- raise_exception('Only user and assistant roles are supported, with the exception of an initial optional system message!') }}\n    {%- endif %}\n{%- endfor %}\n"

Apologies on the delay! Mistral Nemo Base does not have a chat template by default, but the Instruct version does

My bad, I didn't see it was a base model in Colab. It looks like the huggingface page has the wrong link then: https://huggingface.co/unsloth/Mistral-Nemo-Instruct-2407

OOOHH sorry all our notebooks do that sadly - we can't make many notebooks, so we default to 1 for each model


Add null check to configure embedding

Description
Fixing error where louvain does not exist in Configure Embedding.
Steps to reproduce the issue:

Run an experiment, configure embedding, select configure embedding by cluster.
Go back to Data Management, change something so that the experiment has to be re-run. Process the experiment.
Go to configure embedding.

The page throws an error because it is looking for the louvain cluster, while clustering has not yet finished.
Details
URL to issue
https://this-is-biomage.slack.com/archives/C036HK7S2R0/p1676392565784489
Link to staging deployment URL (or set N/A)
N/A
Links to any PRs or resources related to this PR
N/A
Integration test branch
master
Merge checklist
Your changes will be ready for merging after all of the steps below have been completed.
Code updates
Have best practices and ongoing refactors being observed in this PR

[ ] Migrated any selector / reducer used to the new format.

Manual/unit testing

[ ] Tested changes using InfraMock locally or no tests required for change, e.g. Kubernetes chart updates.
[ ] Validated that current unit tests for code work as expected and are sufficient for code coverage or no unit tests required for change, e.g. documentation update.
[ ] Unit tests written or no unit tests required for change, e.g. documentation update.

Integration testing
You must check the box below to run integration tests on the latest commit on your PR branch.
Integration tests have to pass before the PR can be merged. Without checking the box, your PR
will not pass the required status checks for merging.

[ ] Started end-to-end tests on the latest commit.

Documentation updates

[ ] Relevant Github READMEs updated or no GitHub README updates required.
[ ] Relevant Wiki pages created/updated or no Wiki updates required.

Optional

[ ] Staging environment is unstaged before merging.
[ ] Photo of a cute animal attached to this PR.

Rolled in with other fixes Stefan is making.


console.log('screen was destroyed!') in 0.7.1 release

Hi @Yomguithereal,
Thanks for the 0.7.1 release.
It looks like some debug noise got into the released build.  The source maps in the build show the following changes when compared to my checkout of 0.7.1.
--- ../src/fiber/fiber.js
+++ node_modules/react-blessed/dist/index.js.map#../src/fiber/fiber.js
@@ -222,18 +222,16 @@
   return function render(element, screen, callback) {
     screenRef = screen;

-    // TODO: doesn't this leak? Shouldn't we use some weak map?
     let root = roots.get(screen);
     if (!root) {
       root = BlessedReconciler.createContainer(screen);
       roots.set(screen, root);
-
-      screen.once('destroy', () => {
-        BlessedReconciler.updateContainer(null, root, null);
-        roots.delete(screen);
-      });
     }

+    screen.once('destroy', () => {
+      console.log('screen was destroyed!');
+    });
+
     // render at most every 16ms. Should sync this with the screen refresh rate
     // probably if possible
     screen.debouncedRender = debounce(() => screen.render(), 16);

There is something weird indeed. The actual source on the repo is not the one that was built and shipped. Let me fix this quickly in a 0.7.2 release.

v0.7.2 is live on npm and should hopefully fix your issue. I was bitten by the changes npm made to the prepublish script.

Looks good.  Thanks again.


WARNING : recurrentModule should be added to sharedClones in constructor. Adding it for you.

Hello,
I have written a model using nn.GRU and a nn.Sequencer decorating the GRU for forwarding the video frames, that are organized as a table. During training, I am getting the following warning:
WARNING : recurrentModule should be added to sharedClones in constructor.
Adding it for you.
Also, the model doesn't seem to learn. Is there any relation between this warning and the model not-learning from data?
Could you please give any hint on possible causes?

Can share the reproducible code?

Sorry. Changed the code so much that I could not reproduce it.
I will get back when I face this warning again.


refactor omit

How do you like this @1-liners/maintainers ?
Object.keys(object).reduce((res, k) => Object.assign(res, props.includes(k) ? null : {[k]: object[k]}), {}); // before
props.reduce((res, v) => (({ [v]: x, ...r }) => r)(res), obj); // after

much less verbose :+1:

:+1: – But it does take a while to understand what’s going on.

But it does take a while to understand what’s going on.

yeah maybe the argument naming should be more descriptive, instead of singular v, x, etc thoughts?

how about:
props.reduce((newObj, val) => (({ [val]: dropped, ...rest }) => rest)(newObj), obj);

But it does take a while to understand what’s going on.

Yop, but the one before was not so easy to understand either.

Yop, but the one before was not so easy to understand either.

:+1: Definitely not.

props.reduce((newObj, val) => (({ [val]: dropped, ...rest }) => rest)(newObj), obj);

Wow, way better!

:100:

renamed :checkered_flag:

:8ball:


cloud config panic

This PR appends AAD SP details to cloud-config.
This is needed to fix PVC panic, until cherry-pick lands.
 Just double checking

@mjudeikis what did you verify that gave you confidence to remove the hold?

There was test failure with failing to scale up. I created a cluster and tried this, and inspected what is cloud-config and API server revision. I think failure is unrelated to this PR.

@jim-minter simplified tests


Bind mount fix for volume mapping issue in LCOW, affecting RabbitMQ and others

** Description**
As you know a standard Rabbit MQ image cant be run on Win 10 Docker set to Windows Containers  due to volume mapping issues.
Do you know when a fix for this will be released? The last Bind Mount work I could see was merged in last year.
Many thanks
Link to doc:
https://docs.microsoft.com/en-us/virtualization/windowscontainers/deploy-containers/linux-containers#known-app-issues

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Feature report jinja templates

Upgrades pyGSTi html reports to use jinja templates and produce a single html file without Ajax calls to facilitate better browser compatibility.

@enielse I pushed a few changes, mostly offloading the new HTML generation to the templates. I would say this branch is good to merge, but I'd like you to check that generated reports still load correctly in Safari with the changes I made. If it works, go ahead and merge.
One note: twice while testing your changes on Chromium, switching tabs at the wrong time seemed to prevent a table from loading. I've tried and I just can't figure out how to reproduce this, but there's most likely a (rare) race condition in the figure loading pipeline. Something to be aware of, I guess.

I like the additional HTML templating, but the reports weren't loading without an internet connection for me.  I tracked this down to the onerror handler of


When the device goes offline, stop the KafkaDataSubmitter

Right now the KafkaDataSubmitter only checks for HTTP connectivity, and retries every so often if it fails. If we listen for internet connectivity on the device, we can stop the data submitter proactively.
Bonus: check for WiFi/Ethernet vs 3G/4G, and decrease the amount of messages in the latter case.

Right now the KafkaDataSubmitter only checks for HTTP connectivity, and retries every so often if it fails.

Is it worth having an incremental backoff, rather than a periodic check?

Yes that might be better. Right now it's a randomised period between 5 and 15 minutes so at least different clients don't fall in sync.
Listening for local connectivity events is far more effective though if the connectivity problem is local.

Listening for local connectivity events is far more effective though if the connectivity problem is local.
agreed.

This fix also adds a exponential back-off, as suggested.


Improvement: break-down runtimes

Also have a break-down for runtime:

model run
nrnivmodl

Temporarily drop timeout in CI


Update virtual-machines-common-sizes-gpu.md

apparently current data disk capacity is smaller than originally documented, we haven't gotten to the newer numbers yet.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@MikeBazMSFT : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@cynthn Please review these proposed changes. Thanks.

Pinged doc owner.

#sign-off @MikeBazMSFT thanks for the update! Sounds like Karan confirms that these are the correct values.


Sending a transaction shows different IDs

Instructions
When sending a transaction, the output shows different IDs, not sure why:
Sending transaction with ID eddb3dd6f10fabf6873d1578306722dd510d4b21fd00c5757a8aee3a67e3e48c

ID	 1cff804f008c5618dd8b7fe77382d90d6f713058c91cc1275fdcc1c6d9a626a1
Status	 SEALED
Payer	 16fdd39a7bf5afd6
Events
	 Index	 0
	 Type	 A.7e60df042a9c0868.FlowToken.TokensWithdrawn
	 Tx ID	 1cff804f008c5618dd8b7fe77382d90d6f713058c91cc1275fdcc1c6d9a626a1
	 Values
		 amount (UFix64)	1.00000000
		 from ({}?)			16fdd39a7bf5afd6

Problem
The first ID (eddb3dd6f10fabf6873d1578306722dd510d4b21fd00c5757a8aee3a67e3e48c) seems to be wrong, the second ID is valid (1cff804f008c5618dd8b7fe77382d90d6f713058c91cc1275fdcc1c6d9a626a1).
Steps to Reproduce

Acceptance Criteria
IDs should be consistent and valid

This is also fixed in the new pending PR, however there it was removed to just say "sending transaction...", but I added it back to say "sending transaction with id" and it is now correct ID. Before it was wrong because it showed ID before transaction was signed.

Awesome! 👍


[FEATURE] Replace the issue link with wjx.cn research

问卷星 No.3

Only replace it if the locale is ZH_CN


idle connection kicker using 'stats conns' and event notifier frameworks

Still needs some cleanup and a stats counter, but submitting now for review. Could you please let me know if I'm missing anything obvious and that I'm following the suggested path for making this happen?
Possible problems/issues/observations:

it wasn't real obvious how many connections to do per run through the thread, or how long to sleep. I ended up picking an arbitrary number (100) and sleeping for enough time that it takes roughly 1 second to cycle through all the possible connections. This may be a bit heavyweight (we could probably get away with once per 10 seconds or once per minute)
That number is just a #define right now. I don't really see much value in making it tunable at runtime, though.
There might be a more efficient way to doing this by figuring out which connection would time out next, and then scanning the entire connection list to find a new one when that connection expires/updates, but that requires doing a full connection scan and probably a mutex'd variable that's checked on every command, plus is way more complex, so probably isn't worth it.
Only works on idle (rather than stuck-in-transmission) connections. Not sure how hard it would be to make it kill 'active' connections too.

This is more or less the right approach. Feels weird putting it into memcached.c though - but I guess the rest of the conn structures are in there already.
There're some whitespace changes that make the code changes harder to see; please submit those as a separate PR. There's also a TODO or two in here? And the docs files need updating.
If you want to get fancy, you can adjust the scan time based on the given idle timeout and maybe a histogram. On a scan, populate some buckets for how many connections have been idle for so long. If the idle_timeout is 60, and 0 connections were idle when you scanned, there's no point to waking up until 60s later.
If 10,000 connections are open, idle_timeout is 60, and the last scan had 100 conns idle for at most 50s, you can wake up in 10s and scan again. Probably no point in optimizing it further, but you could in a few ways.

This is more or less the right approach. Feels weird putting it into memcached.c though - but I guess the rest of the conn structures are in there already.
There're some whitespace changes that make the code changes harder to see; please submit those as a separate PR. There's also a TODO or two in here? And the docs files need updating.
If you want to get fancy, you can adjust the scan time based on the given idle timeout and maybe a histogram. On a scan, populate some buckets for how many connections have been idle for so long. If the idle_timeout is 60, and 0 connections were idle when you scanned, there's no point to waking up until 60s later.
If 10,000 connections are open, idle_timeout is 60, and the last scan had 100 conns idle for at most 50s, you can wake up in 10s and scan again. Probably no point in optimizing it further, but you could in a few ways.

Oh, and use the per-thread stats for the stats counter. Very few things should be under the global stats lock.

Oh, and use the per-thread stats for the stats counter. Very few things should be under the global stats lock.

Any interest in finishing this thing? :P I think this might be closer than the other one (#62). I'd like to get this into a release after 1.4.25. I'll end up cleaning it up myself but it'd be nice if someone else did too.

I will look into finishing this up in the next week or two!

perfect! Remember to rebase to master/next instead of merging it in. Let me know if you need a more thorough code review to go on as well. My old notes are still accurate as well.

 :( Life has gotten in the way much more than I'd really like. I do still intend to finish this up, unless you get to it first.
(I could also use more time in the day. I really don't understand why nobody has invented a time machine yet.)

:)

Okay, I updated this (finally).
Changes from last time:

Stats updates are now thread-local
Find the longest idle we're not in the process of booting, and sleep for long enough that we'll kick that one on-time when we wake next
Protocol docs updated
Added "stats settings" line for config setting

Still to do:

Split that whitespace change (I am apparently saving that for last)

:))) Noting this again just in case: https://github.com/memcached/memcached/pull/62 - anything from their approach useful/etc. Otherwise I'll close it out in favor of this one.

Okay, there, split that whitespace change. ;)
I just flipped through #62, it's a valid approach, but I'm not sure it's a better approach, partially because it seems harder to follow what's going on. I think this one (mine) will also make it easier to add a timeout for "data pending for this client but no progress made in a long time", which I want to do in a future PR.
(Which I'm open to opinions on how best to add. I think adding most of the logic would be straightforward, but I'm not sure how to handle shutting down the socket correctly.)
And to be fair, #62 is probably more CPU-efficient, though I'd not expect the difference to be particularly noticeable.

Huh. I did a pass that caught most of that, must have gotten lost when I was rebasing stuff. Will fix.

left some pretty minor style comments. read over the other PR again, and I think this might end up being less resource intensive. Managing events in libevent adds more overhead than you'd expect: I've benchmarked just adding one firing once per second in the main loop and it dropped more than 5% of overall performance.
So I like this one better? it has docs and this and that.

Ok, should have all those fixed up.

Thanks! can you fix the one minor issue on the other PR please?

(looks like you did fix it here, you just need to push it to the other branch)

merged into next, thanks!
could I interest you in any other small tasks? :)

You could! Whatcha got?

shoot me an e-mail directly and we'll discuss :)


TS unbound breakpoin

vscode-debugadapter-c2e7456c.json.gz

@yilikun

Please note that your extension must be activated and loaded before a breakpoint becomes "bound".
and make sure that the "outfiles" glob pattern matches all JS files, e.g. use something like "${workspaceFolder}/webpack_dist/**/*.js"

You set a breakpoint in
d:\Projects\final\cyclone-designer-robot\product\cyclone_starter\main\main.ts

However, there were no files with the name "main" loaded by Electron.
As Andre said, your outfiles should generally be a glob as well.

Thanks for replying so fast.

My extension all activated and loaded.
I have updated "ortFiles"

But it doesn't work.

Please share a sample repo. I don't have enough info to find a solution from what's been provided so far.

More infomation:
When I run ./node_modules/.bin/electron ./webpack_dist/product/cyclone_starter/main.bundle.js --inspect,
then

It works.

You can try just removing the outFiles. That will cause the entire workspace to be scanned which may be sufficient to pick up on your sourcemaps.

sorry,you mean delete outFiles item in launch.json?
And my main.bundle.js is large, this file has 485143 lines.

Yes, delete the outFiles

I have deleted outFiles, debugger launch slowly, breakpoint is also unbound...
More infomation:

It doesn't look like it's running any of your code. I notice that you're passing arguments when you run it from the CLI https://github.com/microsoft/vscode/issues/117227#issuecomment-783840130 but you don't inside your launch.json. You may want to provide these are runtimeArgs.

--inspect is a electron command. I think it will launch a debugger.
But if I use vscode launch, maybe I can not use this command?
I still add this property

Then

I'm going to cry...

Okay, thanks. I'm not able to see what's going on from the info here. Are you able to push your project up to a Github repository (either public or private) or create a small sample of what you're facing so I can look into it? I can also screenshare with you.

Okay, thanks. I'm not able to see what's going on from the info here. Are you able to push your project up to a Github repository (either public or private) or create a small sample of what you're facing so I can look into it? I can also screenshare with you.

Thanks for your reply. I was a bit busy some time ago.I have created a small sample, but I can set the breakpoint smoothly...
So i think it might be because the real project file is too large?
As follow:

sorry,you mean delete outFiles item in launch.json?
And my main.bundle.js is large, this file has 485143 lines.

New discovery!!
When I upgrade electron version from 3 to 9, I can set the breakpoint!!

SO, maybe it is a electron bug.........

but when my mouse hover the variable, its value is not show............


license?

Hi, which license applies to this project?
I would like to learn from your project how to implement such a filesystem, but can not without a suitable license specified...
Kind regards
Neo

Hey Neo, sorry about the very late reply. I just added the MIT license. Its mainly based on Prof Pfeiffer's tutorial from here https://www.cs.nmsu.edu/~pfeiffer/fuse-tutorial/ and the rest is code to use sqlite to store and retrieve files using tags.

Hey Neo, sorry about the very late reply. I just added the MIT license. Its mainly based on Prof Pfeiffer's tutorial from here https://www.cs.nmsu.edu/~pfeiffer/fuse-tutorial/ and the rest is code to use sqlite to store and retrieve files using tags.

Thank you!


Code does not continue after purchase

Bug Information
Version Number of Plugin: <IP_ADDRESS>
Device Tested On: Android
Version of VS: 2019
Version of Xamarin: <IP_ADDRESS>
Versions of other things you are using:
Steps to reproduce the Behavior
After purchasing a consumable item, the code never continues.
I have the following code, The purchase is successful, But after the purchase is made, it is supposed to save the results to a file (which never happens). For testing purposes, I am using an AlertDialog to show that a purchase was made. This never happens either.
   public async void MakePurchase(object sender, PurchaseEventArgs e)
    {
           var connected = await _billing.ConnectAsync(ItemType.InAppPurchase);
            if (connected == false)
            {
                await DisplayAlert("Error Connecting", "Could not connect to the Google Play purchases", "OK");
                return;
            }

            //try to purchase item
            InAppBillingPurchase purchase = await _billing.PurchaseAsync(e.ProductId, ItemType.InAppPurchase, e.AccountId.ToString());
            if (purchase != null)
                await DisplayAlert("Is Purchased?", purchase.State.ToString(), "OK");

            if (purchase == null)
                await DisplayAlert("Error Purchasing Item", "There was an error processing the in-app product.", "OK");
            else
            {
            //... Consume the item. Tell the server it was consumed. Save the token to a file...
            }
            await _billing.DisconnectAsync();
     }

I'm having exactly the same problem. The following code is not hit, and no exceptions are thrown. I have added try/catch individually around each call to the plugin in order to isolate the problem.
The effect of this is that none of the code after billing.PurchaseAsync() is called, meaning that the purchase is not consumed, which then means that I am unable to purchase the item again
private async Task<bool> Purchase()
{
    // CONNECT
    try
    {
        await Logger.Debug(PURCHASE, "Connecting to In-App purchases");   // LOGGED ok
        connected = await billing.ConnectAsync(ItemType.InAppPurchase);
        if (!connected)
            return await Logger.Info(PURCHASE, "Connection failed");

        await Logger.Debug(PURCHASE, "Connected");    // LOGGED ok
    }
    catch (InAppBillingPurchaseException purchaseEx)
    {
        return Logger.Error(PURCHASE, "InAppBillingPurchaseException", purchaseEx.Message);
    }
    catch (Exception ex)
    {
        return await Logger.Error(PURCHASE, "Exception", ex.Message);
    }

    // PURCHASE
    await Logger.Info(PURCHASE, "Beginning purchase", $"SKU: {purchase.ProductId}");     // LOGGED ok
    try
    {
        purchase = await billing.PurchaseAsync(vm.SelectedProduct.SKU, ItemType.InAppPurchase, Constants.InAppBillingPayload);
        await Logger.Debug(PURCHASE, "PurchaseAsync - exception NOT thrown"); // NOT LOGGED
    }
    catch (InAppBillingPurchaseException purchaseEx)
    {
        await billing.DisconnectAsync();
        return await Logger.Error(PURCHASE, "InAppBillingPurchaseException", purchaseEx.Message);  // NOT LOGGED
    }
    catch (Exception ex)
    {
        await billing.DisconnectAsync();
        return await Logger.Error(PURCHASE, "Exception", ex.Message);
    }
    await Logger.Debug(PURCHASE, "Purchase operation returned");  // NOT LOGGED

    // CONSUME
    if (purchase != null && Device.RuntimePlatform != Device.iOS)
    {
        await Logger.Info(PURCHASE, "Android purchase successful - need to consume it");  // NOT LOGGED
        try
        {
            consumed = await billing.ConsumePurchaseAsync(purchase.ProductId, purchase.PurchaseToken);
            if (consumed != null)
                await Logger.Info(PURCHASE, $"{Device.RuntimePlatform} purchased and consumed");  // NOT LOGGED
            else
                await Logger.Warning(PURCHASE, $"{Device.RuntimePlatform} purchased but NOT consumed");   // NOT LOGGED
        }
        catch (InAppBillingPurchaseException purchaseEx)
        {
            await billing.DisconnectAsync();
            return await Logger.Error(PURCHASE, "InAppBillingPurchaseException (consumption)", purchaseEx.Message);    // NOT LOGGED
        }
        catch (Exception ex)
        {
            await billing.DisconnectAsync();
            return await Logger.Error(PURCHASE, "Exception (consumption)", ex.Message); // NOT LOGGED
        }
    }
}

So even though the UI reports that the first purchase is successful, the code to consume the purchase is never reached. Consequently, subsequent purchases fail because the item has already been purchased and not consumed.
The logging output I receive on the first run is as follows
Connecting to In-App purchases
Connected
Beginning purchase

... and on the seccond run
Connecting to In-App purchases
Connected
Beginning purchase
InAppBillingPurchaseException - Already Owned

None of the catch blocks are triggered
I am using the following tools
Plugin.InAppBilling	v2.0.0
Xamarin.Forms	v<IP_ADDRESS>7036
Tested On		Android 9 and 10
Visual Studio		2019 (Community)

I have also tried the beta version 3.0.0-beta, with exactly the same outcome

I have managed to respove the problem. it was being caused becuase of the following code in the MainActivity.cs of the Android project...
using Plugin.InAppBilling;

protected override void OnActivityResult(int requestCode, Result resultCode, Intent data)
{
    base.OnActivityResult(requestCode, resultCode, data);
    // InAppBillingImplementation.HandleActivityResult(requestCode, resultCode, data);
}

The call to InAppBillingImplementation.HandleActivityResult() was commented out (because the associated using Plugin.InAppBilling; was missing). As soon as I restored this call, the full workflow was in place, and the call to await billing.PurchaseAsync() was able to complete, and all of the subsequent logic then worked.
Thanks

I have this code in my activity already:
   protected override void OnActivityResult(int requestCode, Result resultCode, Intent data)
    {
        base.OnActivityResult(requestCode, resultCode, data);
        InAppBillingImplementation.HandleActivityResult(requestCode, resultCode, data);
    }

But it still never executes past the purchase. The only way I have gotten it to work is by saving a file to the phone once the player enters the store. Then, when the app is started again after the purchase, it checks if there the special file is there, if so, then it queries the store for any purchases and consumes them then. This is not ideal though as it creates a lapse in the process that error could happen.

Figured out how to resolve this. I was finishing the activity on pause do to other related code. This, in turn cause the OnActivityResult to not continue the code.


Better container split criteria

This line caused a split, which shouldn't:
16/04/20 09:13:10 INFO DefaultWriterContainer: Using user defined output committer class org.apache.parquet.hadoop.ParquetOutputCommitter

I missed these the other day, sorry, and thanks!


Problem resolving es6-object-assign

I'm getting the following error in one of my pipelines. I've switch to using<EMAIL_ADDRESS>since that appears to be the last version that doesn't cause this error.
Module not found: Error: Can't resolve 'es6-object-assign' in '/opt/atlassian/pipelines/agent/build/node_modules/vuejs-logger/dist'

@cory-buckley thanks for reporting, working on it now.


Copy multiple cells including output

Selecting multiple cells and copying all their input+output at once as plaintext would be very helpful, especially for pasting into StackOverflow or ChatGPT.
This was requested before but is locked and can't be upvoted https://github.com/microsoft/vscode-jupyter/issues/13207

still not generating any supporting votes, closing.


Build fails with "HashConflictOnDifferentRoleAssignmentIds"

Main build sometimes fails with an intermittent issue:
ERROR    resources.resource:resource.py:78 Failed to deploy status message: 62ce9147-d327-44ed-b9ee-6b0deb79d135: Error context message = ╷ │ Error: authorization.RoleAssignmentsClient#Create: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="HashConflictOnDifferentRoleAssignmentIds" Message="A hash conflict was encountered for the role Assignment ID. Please use a new Guid." │  │   with module.airlock[0].azurerm_role_assignment.sa_import_approved_reader, │   on airlock/storage_accounts.tf line 294, in resource "azurerm_role_assignment" "sa_import_approved_reader": │  294: resource "azurerm_role_assignment" "sa_import_approved_reader" { │  ╵ Error: error running command /cnab/app/terraform terraform apply -auto-approve -input=false -var client_secret= -var address_space=<IP_ADDRESS>/24 -var app_role_id_workspace_owner= -var app_role_id_workspace_researcher= -var app_service_plan_sku=P1v3 -var auth_client_id=******* -var auth_client_secret=******* -var auth_tenant_id=******* -var client_id=auto_create -var aad_redirect_uris_b64=W10= -var enable_airlock=true -var enable_local_debugging=false -var location=*** -var register_aad_application=true -var scope_id= -var shared_storage_quota=50 -var sp_id= -var tre_id=*** -var tre_resource_id=62ce9147-d327-44ed-b9ee-6b0deb79d135 -var workspace_owner_object_id=ade81c51-edee-4f17-ac8d-1becb3dd7352: exit status 1 Error: mixin execution failed: exit status 1 Error: 2 errors occurred: 	* container exit code: 1, message: <nil>. fetching outputs failed: error copying outputs from container: Error: No such container:path: a827214dce7ccb5302fcc3e9599315ad99d36c07368ab8b6776eb1caf87500a2:/cnab/app/outputs 	* required output app_role_id_workspace_owner is missing and has no default   az login --identity -u 261b930f-39a8-4266-bf69-abdb98fddebe && az acr login --name ***mgmt && porter install "62ce9147-d327-44ed-b9ee-6b0deb79d135"  --reference ***mgmt.azurecr.io/tre-workspace-base:v0.3.19  --param address_space="<IP_ADDRESS>/24" --param arm_use_msi="true" --param azure_location="***" --param client_id="auto_create" --param id="62ce9147-d327-44ed-b9ee-6b0deb79d135" --param register_aad_application="True" --param tfstate_container_name="tfstate" --param tfstate_resource_group_name="rg-***-mgmt" --param tfstate_storage_account_name="***mgmt" --param tre_id="***" --param workspace_owner_object_id="ade81c51-edee-4f17-ac8d-1becb3dd7352" --allow-docker-host-access --force --cred ./vmss_porter/arm_auth_local_debugging.json --cred ./vmss_porter/aad_auth.json && porter show 62ce9147-d327-44ed-b9ee-6b0deb79d135

Example runs:
https://github.com/microsoft/AzureTRE/runs/7457454389?check_suite_focus=true
https://github.com/microsoft/AzureTRE/runs/7505818171?check_suite_focus=true

Known mitigation is to retry

Also see the corresponding issue in azurerm provider: https://github.com/hashicorp/terraform-provider-azurerm/issues/17155

I've attempted to get a reliable way to reproduce this error and created the terraform below (note, when running, add -parallelism=50 :smile:).
This creates container_count containers in a storage account and assigns the running user 4 different roles against each container.
I've run this multiple times with 150 containers and haven't seen the issue 😞
terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "=3.16.0"
    }
  }
}

provider "azurerm" {
  features {}
}

variable "rg_name" {}
variable "storage_name" {}

variable "container_count" {
	default = 150
	type = number
}

data "azurerm_client_config" "current" {
}

resource "azurerm_resource_group" "test" {
  location = "northeurope"
  name     = var.rg_name
}

resource "azurerm_role_assignment" "rg_contributor" {
  count                = 1
  scope                = azurerm_resource_group.test.id
  role_definition_name = "Contributor"
  principal_id         = data.azurerm_client_config.current.object_id

  depends_on = [
    azurerm_resource_group.test
  ]
}

resource "azurerm_storage_account" "test" {
  resource_group_name      = azurerm_resource_group.test.name
  location                 = azurerm_resource_group.test.location
  name                     = var.storage_name
  account_kind             = "StorageV2"
  account_tier             = "Standard"
  account_replication_type = "LRS"
}
resource "azurerm_storage_container" "example" {
  count                 = var.container_count
  name                  = "container-${format("%04d", count.index + 1)}"
  storage_account_name  = azurerm_storage_account.test.name
  container_access_type = "private"
}

resource "azurerm_role_assignment" "container_data_contributor" {
  count                = var.container_count
  scope                = "${azurerm_storage_account.test.id}/blobServices/default/containers/${azurerm_storage_container.example[count.index].name}"
  role_definition_name = "Storage Blob Data Contributor"
  principal_id         = data.azurerm_client_config.current.object_id
}
resource "azurerm_role_assignment" "container_data_reader" {
  count                = var.container_count
  scope                = "${azurerm_storage_account.test.id}/blobServices/default/containers/${azurerm_storage_container.example[count.index].name}"
  role_definition_name = "Storage Blob Data Reader"
  principal_id         = data.azurerm_client_config.current.object_id
}

resource "azurerm_role_assignment" "container_contributor" {
  count                = var.container_count
  scope                = "${azurerm_storage_account.test.id}/blobServices/default/containers/${azurerm_storage_container.example[count.index].name}"
  role_definition_name = "Contributor"
  principal_id         = data.azurerm_client_config.current.object_id
}
resource "azurerm_role_assignment" "container_reader" {
  count                = var.container_count
  scope                = "${azurerm_storage_account.test.id}/blobServices/default/containers/${azurerm_storage_container.example[count.index].name}"
  role_definition_name = "Reader"
  principal_id         = data.azurerm_client_config.current.object_id
}

I've run this multiple times with 150 containers and haven't seen the issue 😞
... and immediately after this, I attempt to deploy a workspace and hit the issue!


Typo in Why AES is Secure Doc

In the Confusion and Diffusion section you say "Confusion refers to the property that each character of the plaintext depends on several parts of the key..."
Plaintext here should be ciphertext.

@wrv Thanks! How did you come across this repo?

Fixed in https://github.com/wei2912/blog_src/commit/0e345e5350cfb60c6a064897e3ee14d3378b7d86, closing.

Professor linked to the blog post as a deeper dive into AES, and then I followed the link at the bottom of page to find this repo. Quality stuff! Thanks for addressing the issue.

@wrv That's quite a surprise to me -- I wrote this post for the sole purpose of consolidating what I knew about the subject, so I didn't expect them to be noticed by anyone else. Thanks for letting me know!


Create codeql-analysis.yml

Thanks @hkaiser for pointing out that this exists. I've no clue what it will actually do, but we'll hopefully find out soon enough (this config is unchanged from what GitHub suggested)!

I'll close this for now as it seems like it requires some additional setup and I'm not yet sure what we gain from this. This would still be interesting to look at in the future.


Undefined symbols RTCIceCandidate and RTCIceServer in ios device and sim builds..

I am trying out the AppRTCDemo for iOS. The revision is 12128.

I have the same inssue.


Stop throwing generic Exceptions from Computation domain.

These things (or their predecessors) used to live at the Application layer so we weren't bothered about domain exceptions, but after moving them down to Computation I forgot to do this.


Is there currently a solution for using Webhooks with asv?

We'd like to have our server automatically run asv when master is pushed to on github. I didn't see anything in the asv codebase, but I thought asv or astropy might already have a daemon script or something similar in place for this?

Can't speak for astropy, but I haven't seen such scripts. I'd guess most setups run it from cron.

That's right.  For astropy, we run it nightly with cron.  If you don't need results immediately, it works pretty well (it still tests every new commit) and doesn't require setting up the test machine with open ports in the Internet.

Cool, thanks. I have a few more questions. I opened #272 to track them.


Parsed results not showing IO Latency values

after collecting data of a mixed FIO run on RAID'd NVMe drives, the parsed output is showing all latency values to be zero. unzip and run iotrace -parse opn these trace files.
sample.tar.gz

Probably the same issue as in #98, closing this issue as to not duplicate the issues.


build: attempt to fix the build - please don't merge

wrapping the mustache call with a promise

Coverage remained the same at 100.0% when pulling 4bbd7503adbb0121a75ffd91e2602e4e4a5a8da3 on helio-frota:build-fix into a0bb9030087a00497071008413901e37274fcd65 on bucharest-gold:master.


GNARL disconnecting from Loop every 5 minutes

Hi. I have installed GNARL on my TTGO v1 board and it just disconnects from Loop every 5 minutes. When I try to reconnect GNARL with Loop, it just gives an error message like "Device Error. Unknown characteristic: C842E849-5028-42E2-867C-016ADA-DA9155: The RileyLink was temporarily disconnected. Make sure the device is nearby, and the issue should resolve automatically". And when I try to reconnect it again, it just reconnects. And I need to reconnect GNARL with Loop every 5 minutes, otherwise it doesn't work. How can I fix it?
Maybe it will help, idk:

iPhone 12 mini, iOS 14.7.1
Latest Loop build, built with Xcode 12.5.1
ESP-IDF v4.3
Latest GNARL build
Medtronic Paradigm Real-Time (722, World-Wide, 868Mhz)

I have the same problem, with  TTGO v1 board without display.
1.     iPhone SE 2020, iOS from 15.1 to 15.3.1
2.     Latest Loop build with Xcode 13.2.1
3.     ESP-IDF  v4.3 and v4.4
4.     Latest GNARL build
5.     Medtronic Paradigm Real-Time (722, World-Wide, 868Mhz)

Are you using multiple devices by any chance? https://github.com/ecc1/gnarl/commit/cce3d24db2213d42e490335cdb40e1992581775f comes to mind if no commands are sent through GNARL it will reset.

You could try reverting the commit (ie. disable the watchdog) to narrow down the problem.

Unfortunately it didn't help .


Microsoft.Azure.Management.Batch - Deprecated

Library name and version
Microsoft.Azure.Management.Batch
Query/Question
Where is the specific replacement for this deprecated SDK? Documentation, examples etc? The project URL points to generic nonsense.

Thx!
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Fixed the note for this version, thank you @BC89 for reporting this issue.


feat(network): broadcast stream messages (#649)

feat(network): broadcast stream messages (#649)
Add StreamMessage, a struct that contains any T that can turn into a protobuf or Vec.
In addition it contains a sequence and stream IDs and a fin boolean.
Allows broadcasting a stream of messages (e.g., ConsensusMessages).
feat(mempool): implement account nonces (#534)
refactor(mempool): reorder account nonces tests (#755)
wip

This change is

#787  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @TzahiTaub and the rest of your teammates on  Graphite


Downloads announcement instead of chapters

Wuxiaworld has an announcement on top of the novels right now and the script takes the announcement as the chapter and downloads that instead.

Thanks for reporting this issue!
Have quite a bit of stuff to do so I didn't manage to fix it up to this point. Hopefully I'll come around fixing in the next couple of days.

Fixed thanks to the courtesy of @ashussen.


Changed behaviour of 'isFilePath'

When Imagick is installed and used as the driver it can work with a
pageselect in the imagepath, note that this only works if ghostscript
is installed. But it then allows the library to quickly load a single
page instead of having to load the entire document; therefor
increasing the parsing speed of the document quite dramatically. When
ghostscript is not installed it will not break when imagick is used.
We also added another testcase to make sure this works. To get this
to work we had to mock the 'Config' facade, which has been done using
a mock proxy.

Sorry, can't merge this. Config is part of Laravel? Intervention Image should not be tied to any framework.


[Feature] Add helper method for simulation

Adds a new helper method that clones a chessboard and calculates possbíble moves of a piece after reowning pieces of the same piecetype. This can be used for the cannon piece.

I think we don't need this anymore, I solved it differently. I used the function setCommonPiecePlayer(). I didn't know that we had such a function.

Okay nvm


Strict mass-assignment protection preventing display of object.

(pry) output error: #<ActionController::UnfilteredParameters: unable to convert unpermitted parameters to hash>

Unfortunately, for historical reasons, our Rails app has some data serialized to a text column named data as YAML that includes a class of ActionController::Parameters, and a def data method that does a YAML.load of the raw attribute.
When we try to display the offending model at the console, we get the above and no other output.  No useful stack trace, no meaningful info about the object, etc.  The inspect method for the model works fine, because it's displaying the column attribute (I.E. a String), not the parsed YAML.
While this is a pathological design problem on our end, the failure mode of not displaying anything is an impediment to debugging.
Perhaps you could catch exceptions, and try falling back to inspect?  Or maybe provide more context about what went wrong (I.E. the path into the object at which the exception occurred)?
Or, perhaps propagate the error in a way where Pry's _ex_ is properly set?

@MrJoy @CentroDL @ndbroadbent may be the last 2 commits in my from are doing what you expected (somehow)

Thanks @obourdon, this would be great to merge in! I’m always getting this error while debugging a controller action and trying to see the params.

I honestly don't even remember this issue -- and am no longer at the organization that was experiencing it.  That said, your changes certainly look like they would help.  Thank you!

@obourdon I think that here:
https://github.com/obourdon/awesome_print/commit/5567d544b9c83a29b9e889d38a9c37cbe994480e
you would want to be able to call object.to_enum.to_h which is what will turn the Parameters to a hash. but then you lose out on the permitted:true attribute. 🤔


Add new section- guildline to take care of pet

we can add a section where the user may get some tips about the dos and don'ts.
Tips and precaution while having a pet. Kindly assign me this issue, I am willing to work on this under the GSSOC tag.

Hey, Can i work on this issue under GSSOC tag, also we can improve the designing as it is now looking a bit weird

hey @akshitagupta15june i would like to work on the issue
there can be two ways either we create a complete section for this or we can add the guidelines after someone clicks  on a pet.
would you please assign me the issue.
I am willing to work on this under the GSSOC tag


Fix deadmin on join cvar for nukies

About the PR

Fixes #19271 . Deadmins the player if the cvar is active and they're an admin.
Technical details

This was broken as RulePlayerSpawningEvent is broken immediately as the spawning event is called, within SpawnPlayers whereas the cvar was used in SpawnPlayer after the event is called. I don't just iterate through it in SpawnPlayers (doing so in  in the nukeopsrulesystem instead) as it still needs to exist in SpawnPlayer as I believe that method is used elsewhere, but SpawnPlayers eventually calls SpawnPlayer. I'm sure it wouldn't break anything but it made more sense to do it here. I believe nukeops are the only antag that currently don't use the standard station spawning system.
I'm not certain how the webhook works so maybe someone smarter than myself can inform me.

Tests work on this now

Will be working on this soon.