 Ya, we do not attempt to handle Error: https://github.com/ramda/ramda/blob/afe98b03c322fc4d22742869799c9f2796c79744/source/internal/_clone.js#L40
Looks like lodash just returns a cloned Object. Similar with a couple other deep clone type packages I saw.
Seems to me like creating a new Error with some properties set on it would be better? Would you like to submit a PR?

Sure. I will handle this issue


Add tests for mounting without linking libfuse on OSX

loading the osxfuse kernel extension isn't supported: https://github.com/travis-ci/travis-ci/issues/10017


posix_apis: fails at test_posix_realtime for mimxrt1024_evk

Describe the bug
Assertion failed at WEST_TOPDIR/zephyr/tests/posix/common/src/clock.c:75: test_posix_realtime: (rts.tv_nsec not equal to mts.tv_nsec)

only this board has such issue, all other rt1xxx series boards are OK.
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
cmake -DBOARD=board\mimxrt1024_evk
make
See error

Expected behavior
pass
Impact
posix realtime
Logs and console output

*** Booting Zephyr OS build v2.6.0-rc1-323-gae4fb20f6733  ***

Running test suite posix_apis

===================================================================

START - test_posix_pthread_execution

Thread 0 starting with scheduling policy 0 & priority 15

Thread 1 starting with scheduling policy 0 & priority 15

Thread 2 starting with scheduling policy 0 & priority 15

Bounce test OK

Barrier test OK

 PASS - test_posix_pthread_execution in 0.643 seconds

===================================================================

START - test_posix_pthread_error_condition

 PASS - test_posix_pthread_error_condition in 0.1 seconds

===================================================================

START - test_posix_pthread_termination

Thread 0 starting with a priority of 4

Cancelling thread 0

Thread 1 starting with a priority of 3

Cancelling thread 1

Thread 1 could not be cancelled

Thread 2 starting with a priority of 2

Cancelling thread 2

Thread 3 starting with a priority of 1

Cancelling thread 3

Thread 3 could not be cancelled

 PASS - test_posix_pthread_termination in 1.14 seconds

===================================================================

START - test_posix_multiple_threads_single_key

Different threads set different values to same key:

thread 0: set value = -2147477408 and retrieved value = -2147477408

thread 1: set value = -2147477184 and retrieved value = -2147477184

 PASS - test_posix_multiple_threads_single_key in 0.17 seconds

===================================================================

START - test_posix_single_thread_multiple_keys

Single thread associates its value with different keys:

key 0: set value = -2147476984 and retrieved value = -2147476984

key 1: set value = -2147476984 and retrieved value = -2147476984

 PASS - test_posix_single_thread_multiple_keys in 0.17 seconds

===================================================================

START - test_posix_clock

POSIX clock APIs

POSIX clock APIs test done

 PASS - test_posix_clock in 2.5 seconds

===================================================================

START - test_posix_semaphore

 PASS - test_posix_semaphore in 5.1 seconds

===================================================================

START - test_posix_normal_mutex

mutex lock is taken

 PASS - test_posix_normal_mutex in 0.103 seconds

===================================================================

START - test_posix_recursive_mutex

recrusive mutex lock is taken

 PASS - test_posix_recursive_mutex in 0.3 seconds

===================================================================

START - test_posix_mqueue

 PASS - test_posix_mqueue in 0.11 seconds

===================================================================

START - test_posix_realtime

POSIX clock set APIs

    Assertion failed at WEST_TOPDIR/zephyr/tests/posix/common/src/clock.c:75: test_posix_realtime: (rts.tv_nsec not equal to mts.tv_nsec)

Nanoseconds not equal

 FAIL - test_posix_realtime in 0.17 seconds

===================================================================

START - test_posix_timer

POSIX timer test

Timer fires every 0 secs and  100000000 nsecs

Time remaining to fire 0 secs and  899000000 nsecs

Handler Signal value :20 for 1 times

Handler Signal value :20 for 2 times

Handler Signal value :20 for 3 times

Handler Signal value :20 for 4 times

Handler Signal value :20 for 5 times

Handler Signal value :20 for 6 times

Handler Signal value :20 for 7 times

Handler Signal value :20 for 8 times

Handler Signal value :20 for 9 times

Handler Signal value :20 for 10 times

Handler Signal value :20 for 11 times

Handler Signal value :20 for 12 times

 PASS - test_posix_timer in 2.111 seconds

===================================================================

START - test_posix_rw_lock

main acquire WR lock and 3 threads acquire RD lock

Thread 2 scheParent thread releasing WR lock

duling policParent thread acquiring WR lock again

Parent thread acquired WR lock again

y = 1 & prioParent thread releasing WR lock again

3 threads acquire WR lock

Main thread acquiring RD lock

Main thread acquired RD lock

rity 3 startMain thread releasing RD lock

ed

Thread 2 got RD lock

Thread 1 scThread 2 releasing RD lock

Thread 2 acquiring WR lock

Thread 2 acquired WR lock

heduling polThread 2 releasing WR lock

icy = 1 & priority 2 started

Thread 1 got RD lock

Thread 0 schThread 1 releasing RD lock

Thread 1 acquiring WR lock

Thread 1 acquired WR lock

eduling poliThread 1 releasing WR lock

cy = 1 & priority 1 started

Thread 0 got RD lock

Thread 0 releasing RD lock

Thread 0 acquiring WR lock

Thread 0 acquired WR lock

Thread 0 releasing WR lock

 PASS - test_posix_rw_lock in 0.79 seconds

===================================================================

START - test_nanosleep_NULL_NULL

 PASS - test_nanosleep_NULL_NULL in 0.1 seconds

===================================================================

START - test_nanosleep_NULL_notNULL

 PASS - test_nanosleep_NULL_notNULL in 0.1 seconds

===================================================================

START - test_nanosleep_notNULL_NULL

 PASS - test_nanosleep_notNULL_NULL in 0.1 seconds

===================================================================

START - test_nanosleep_notNULL_notNULL

 PASS - test_nanosleep_notNULL_notNULL in 0.1 seconds

===================================================================

START - test_nanosleep_req_is_rem

 PASS - test_nanosleep_req_is_rem in 0.1 seconds

===================================================================

START - test_nanosleep_n1_0

 PASS - test_nanosleep_n1_0 in 0.1 seconds

===================================================================

START - test_nanosleep_0_n1

 PASS - test_nanosleep_0_n1 in 0.1 seconds

===================================================================

START - test_nanosleep_n1_n1

 PASS - test_nanosleep_n1_n1 in 0.1 seconds

===================================================================

START - test_nanosleep_0_1

 PASS - test_nanosleep_0_1 in 0.1 seconds

===================================================================

START - test_nanosleep_0_1001

 PASS - test_nanosleep_0_1001 in 0.1 seconds

===================================================================

START - test_nanosleep_0_500000000

 PASS - test_nanosleep_0_500000000 in 0.501 seconds

===================================================================

START - test_nanosleep_1_0

 PASS - test_nanosleep_1_0 in 1.1 seconds

===================================================================

START - test_nanosleep_1_1

 PASS - test_nanosleep_1_1 in 1.1 seconds

===================================================================

START - test_nanosleep_1_1001

 PASS - test_nanosleep_1_1001 in 1.1 seconds

===================================================================

START - test_posix_pthread_create_negative

 PASS - test_posix_pthread_create_negative in 0.1 seconds

===================================================================

Test suite posix_apis failed.

===================================================================

PROJECT EXECUTION FAILED

Environment (please complete the following information):

OS: (e.g. Linux)
Toolchain (e.g Zephyr SDK, ...)
Commit SHA or Version used:  v2.6.0-rc1-323-gae4fb20f6733

@MaureenHelm , @dleach02

I find the test that is failing to be somewhat confusing.
        ret = clock_gettime(CLOCK_MONOTONIC, &mts);
        zassert_equal(ret, 0, "Fail to get monotonic clock");

        ret = clock_gettime(CLOCK_REALTIME, &rts);
        zassert_equal(ret, 0, "Fail to get realtime clock");

        zassert_equal(rts.tv_sec, mts.tv_sec, "Seconds not equal");
        zassert_equal(rts.tv_nsec, mts.tv_nsec, "Nanoseconds not equal");

it's entirely feasible that some number of nanoseconds could have expired between the two calls to clock_gettime(), so it's almost as if the test had been designed under the wrong assumption. Assuming that the timer resolution would always be coarse enough is probably not the safest assumption.
even assuming that CLOCK_REALTIME and CLOCK_MONOTONIC would report the same absolute times is very likely an unsafe assumption (even duration could be different, based on individual clock resolution).
the test should minimally include a call to clock_getres() (not yet supported)

Some related items do rank fairly high on the nice-to-have scale though (#6498, #25559, #28542) and it would be great to chat about it at the next review.

I reproduced this on the rt1024_evk I have.

@cfriedt is correct in his assessment. You can minimize the problem by putting the two clock_gettime() calls back to back without the zassert_equal() check. But I think this test case should account for edge conditions. I converted the tv_nsec to TICKS and did some test runs and when it fails it is typically off by 1. The "Nanoseconds not equal" should probably be ((rts-mts) <= 1) type of test or make the 1 be some sort of configurable value that would encompass the edge cases.
On the rt1024, there are 10000 ticks per sec so the two calls to have to happen within the same 100usec window.

@dleach02 - Thanks for tagging me - this has been on my to-do list for a very long time. Probably not going to make it for LTS but it would be great to being it up in dev-review at some point

I looked at this because it initially was flagged against an NXP board. I just received the board and tried it out. The "fix" I was suggesting would be the following:
`	k_usleep(1);
ret = clock_gettime(CLOCK_MONOTONIC, &mts);
ret2 = clock_gettime(CLOCK_REALTIME, &rts);

zassert_equal(ret, 0, "Fail to get monotonic clock");
zassert_equal(ret2, 0, "Fail to get realtime clock");

zassert_equal(rts.tv_sec, mts.tv_sec, "Seconds not equal");
zassert_true((k_ns_to_ticks_floor64(rts.tv_nsec) - k_ns_to_ticks_floor64(mts.tv_nsec)) <= 1, "Nanoseconds outside of threshold");

`

I would be open to removing the test

dev-review: (sept 2, 2021): suggestion to @cfriedt to open an issue to track rfc/enhancement around this area.


split aws/iam-instance-profile ops file into 2:

iam-instance-profile.yml to have the director use its profile when deploying vms
cloud-provider-iam-instance-profile.yml to deploy the director using the cli host's instance profile

this should fix the problem described @ https://github.com/cloudfoundry/bosh-deployment/commit/b37a696baa170ea30a2e4478d7292bbcb614306f
Signed-off-by: Connor Braa<EMAIL_ADDRESS>

i see those commite


Unable to import GKE multi-AZ zonal clusters

Internal ref: SURE-4599
Importing a local GKE-hosted cluster (v1.22.8-gke.200) with workers in multiple AZ fails in Rancher 2.5.12/2.6.x
Repro steps:

Provision of a GKE cluster - v1.22.8-gke.200 (manually  from the GCP console or using Terraform), which  multiple availability zones for worker nodes
Import this GKE cluster into Rancher

On investigation, when the UI gets the list of clusters (from GCP) for the given region its returning the location of the control plane and the locations that should be used by default for the nodes:
"location": "us-central1-c",
"locations": [
   "us-central1-b",
   "us-central1-c"
]

PR: https://github.com/rancher/ui/pull/4971

Issue https://github.com/rancher/dashboard/issues/8256 - Test Results - :white_check_mark:
Issue reproduced on unfixed version:

Issue fixed, verfied on build v2.7 - 0554da2, able to import GKE multi-AZ zonal & regional clusters.

GKE cluster:


ARROW-14554: [C++][CI] Fix OSS-Fuzz build failure

ARROW-14393 downgraded the bundled GTest version to a version that doesn't build on the OSS-Fuzz setup
(and there doesn't seem to be an obvious fix or workaround around those build errors).
Re-bump the bundled GTest version, and try to fix the issue in the Windows verification script differently.
Also, add a Crossbow task for the OSS-Fuzz build check, since the Github Actions cron job failure went unnoticed.

@github-actions crossbow submit -g cpp

@github-actions crossbow submit test-conda-*

@github-actions crossbow submit test-conda-*

@github-actions crossbow submit -g verify-rc-source

@github-actions crossbow submit vcpkg

Hmm, it appears we cannot exercise the release verification script using crossbow (it tries to fetch Arrow 1.0.0)? @kszucs

@github-actions crossbow submit test-conda-*

@github-actions crossbow submit -g fuzz

@github-actions crossbow submit -g fuzz

@github-actions crossbow submit -g fuzz

@github-actions crossbow submit -g fuzz

@github-actions crossbow submit -g fuzz

@github-actions crossbow submit -g fuzz

@github-actions crossbow submit -g fuzz

@github-actions crossbow submit test-conda-*

@github-actions crossbow submit test-conda-*

@github-actions crossbow submit test-conda-cpp-valgrind

@github-actions crossbow submit test-conda-cpp-valgrind

@kszucs Can we merge this? I'd like to unblock the OSS-Fuzz builds.

Hmm, it appears we cannot exercise the release verification script using crossbow (it tries to fetch Arrow 1.0.0)? @kszucs

We can only build releases with those tasks by providing additional parameters: https://github.com/apache/arrow/pull/11511#issuecomment-948918795

Re-bump the bundled GTest version, and try to fix the issue in the Windows verification script differently.

Do we have a JIRA for that?

I don't understand the question. What do you want a JIRA for?

I don't understand the question. What do you want a JIRA for?

Does this fix the windows verification issue as well?

(see diff in https://github.com/apache/arrow/pull/11583/files#diff-6c1e203cf55c63a1f177b4b0c97dc1aa62ea5eed5bd8e017e70703ddf119e343L90-L92)


why express does not implement Node.js style callbacks?

I have a question regarding the APIs. Why express does not implement Node.js style callbacks, i.e., Error-First Callbacks?
For example, router.METHOD:
router.get('/', function(req, res){
  res.send('hello world');
});

And furthermore, I also saw this Error handling:
app.use(function (err, req, res, next) {
  // logic
})

Then, my question is: after add the Error Handling middleware as above, can I regard my code as following?
router.get('/', function(err, req, res, next){
  res.send('hello world');
});

If so, I am able to use Q.denodeify converting the callback style to promise style right? Because Q.denodeify, as stated here, only deals with Node.js style (err, result) callback APIs

Several of the node event based API's are not node style call backs either.
For Example, all of the http namespace functions are not this way, it is
because the callback style is not suited for dealing with stream like
objects that can fail at multiple points, and have multiple locations which
you would want to handle them with.
Of course you can provide these abstractions your self, I personally have
no idea if the above example of denodeify would work, but it wouldn't hurt
to try and let the world know of your results.
On Tue, Mar 21, 2017 at 5:12 AM, Chang Liu<EMAIL_ADDRESS>wrote:

I have a question regarding the APIs. Why express does not implement
Node.js style callbacks, i.e., Error-First Callbacks?
For example, router.METHOD
https://expressjs.com/en/api.html#router.METHOD:
router.get('/', function(req, res){
res.send('hello world');
});
And furthermore, I also saw this Error handling
https://expressjs.com/en/guide/error-handling.html:
app.use(function (err, req, res, next) {
// logic
})
Then, my question is: after add the Error Handling middleware as above,
can I regard my code as following?
router.get('/', function(err, req, res, next){
res.send('hello world');
});
If so, I am able to use Q.denodeify
https://github.com/kriskowal/q/wiki/API-Reference#qnfbindnodefunc-args
converting the callback style to promise style right? Because Q.denodeify,
as stated here
https://github.com/kriskowal/q/wiki/API-Reference#interfacing-with-nodejs-callbacks,
only deals with Node.js style (err, result) callback APIs
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/expressjs/express/issues/3250, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKLqvbHoL93tw93SwR33Vk15yLaJeg0iks5rn6KOgaJpZM4Mjk-0
.

Hi @fluency03 none of the functions you're showing are callbacks in the sense that Node.js defines them (they document this in https://nodejs.org/dist/latest-v6.x/docs/api/errors.html#errors_node_js_style_callbacks).
The function you are giving to app.use, router.get, etc. is not performing some sort of operation and expecting to get called back (i.e. is a callback). Instead it is more of an event listener, waiting for a request event to come in to handle. Just like event listeners which do not have an error object as the first argument, neither do the middleware in Express.js.
In essence, all your middleware functions are just request listeners (Node.js defines them here: https://nodejs.org/dist/latest-v6.x/docs/api/http.html#http_event_request) with the additional argument to allow you to chain them together instead of writing your entire server in a single function body.
You can think of your event listener as being the function that does the operation (responding to the request) and the next argument is like the callback to your work function.
I hope that helps!


feat(side-menu) Migrating side menu

Describe pull-request
Migrating side menu
Solving issue
In case of GitHub issue add # plus the number of the issue (for example #123) OR if it is Azure then AB# and number of ticket
Fixes: AB#2415
How to test

check out the storybook and see if the side-menu is migrated in accordance with the migration guidelines.

Is this one still in "Draft" status?

Is this one ready to be merged?


Eureka: Remove encoded port from VIP address defaults

Motivation:
Currently, when armeria registers a server, it appends :${port} to Eureka VIP addresses to. While it makes sense to encode the port into the Eureka serviceId, the VIP address fields are meant to be hostnames or IPs. Encoding a port into the VIP address makes these unusable for certain clients like spring-cloud-sleuth who heuristically look for a port and only if it is absent trigger Eureka lookups.
Modifications:

I removed the logic that appended :${port} to both the vipAddress and secureVipAddress
I changed the test which actually proved the bug was present!

Result:

Manual integration with zipkin, sleuth is now able to look up its endpoint in Eureka.

fixed the description

test flake is unrelated

ps the really really long story about this small bug https://github.com/openzipkin/zipkin/releases/tag/3.0.6

there seems to be another glitch which I will look into (past this). More UX related? https://github.com/openzipkin/zipkin/issues/3697#issuecomment-1943240258


Setting up Lunaserver

Here, you will find progress reports on the status of Lunaserver, a milestone in this club's history that will allow members to access hardware remotely, whenever they need to.
Below is a list of critical milestones

[x] Receive Lunaserver (the computer itself)
[x] Establish an SSH connection from outside campus
[x] Start a DevContainer over SSH
[ ] Run telemetry and receive movement controls from outside campus (CANCELLED)
[x] Get any response from Pozyx
[x] View camera data from RViz over X11

Reaching all the milestones on this list gets us to the same level of progress we had before Lunaserver. This means that we will be able to continue pursuing the team's goals effectively while greatly reducing the amount of time we need to spend in the lab (at least for the Fall)

Was unable to use X11 successfully, but VNC worked correctly.


Use global Pdb instance via new

This is important with pytest using Pdb().set_trace, and not
pdb.set_trace() - where settings like sticky mode would be not
remembered with the new instance.

Codecov Report

Merging #234 into master will decrease coverage by 0.08%.
The diff coverage is 65.38%.

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
- Coverage   63.41%   63.33%   -0.09%
==========================================
  Files           3        3
  Lines        2247     2272      +25
  Branches      206      209       +3
==========================================
+ Hits         1425     1439      +14
- Misses        749      758       +9
- Partials       73       75       +2

Impacted Files
Coverage Δ

pdb.py
43.8% <50%> (-0.09%)
:arrow_down:

testing/test_pdb.py
78.29% <76.66%> (-0.33%)
:arrow_down:

Continue to review full report at Codecov.

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


Convert the blog to a Jupyter Book

Hi all,
Here comes the PR that reorganizes the blog into a jupyter book.
I have also fixed a few typos, fixed some broken links to images, and changed quite a few links to relative ones instead of absolute ones for easier maintenance.
Please feel free to add changes to the intro page or organization of ToC if you wish.
The preview is available at https://lazigu.github.io/blog/

Wow @lazigu ! This looks amazing!
One idea: I would put the recent 3-5 blog posts on the starting page...
Also you should write a blog post (about setting up a jupyter book? ;-) ) so that we can list you under authors :-)

I added 5 of the most recent blog posts and fixed displayed images. I think this is finished now :)

Looks super cool! I was wondering whether it would make sense to sort the posts according to topic or somehow add tags to it (similar how you can set topics in image.sc) but I think that's currently not possible. So - thumbs up from me :)

We had a discussion during lunch about the organization of posts according to the topic. However, since we do not have that many blog posts yet, many topics might have just a couple of blog posts in them and we would lose chronological order. What do you @haesleinhuepf think?

Yes, I agree. We definitely need more blog posts.
Ready? Shall we merge this? 🙃

From my side yes :)
We can come back to the tags and better posts organization idea when the blog expands :)


Does MutationMapping support reactive types?

Hi Spring Team,
We do NOT seem to support reactive types for input as we do for the @RequestBody
@MutationMapping
public Mono<Book> addBook(@Arugment Mono<Book> book){
  ....
}

I get this.
org.springframework.beans.BeanInstantiationException: Failed to instantiate [reactor.core.publisher.Mono]: Is it an abstract class?; nested exception is java.lang.InstantiationException
If i change as shown here, works fine.
@MutationMapping
public Mono<Book> addBook(@Arugment Book book){
  ....
}

Is it expected?

Technically everything is POST in graphql


MAC OS Not functioning properly

Which Cloudflare product(s) does this pertain to?
<IP_ADDRESS>
Subject Matter
Unable to connect to MAC OS after installation
Content Location
Unable to connect to MAC OS after installation
Additional information
No response

Hi @admin-xoxo, if you can provide more details on the specific section of Cloudflare's developer documentation that you believe need an update, we'd be happy to take a look. If we don't hear back, I'll need to close out this Issue. Thanks so much!


Change ESB Solution project name to Integration project

Description:
Currently, the main project creation wizard named as ESB Solution project. Need to change it as  Integration project.
Suggested Labels:
Type/Improvement

Fixed in - https://github.com/wso2/devstudio-tooling-esb/pull/1093


Update to Akka 2.5.0 and latest Akka HTTP

Thanks! Though I think the current release is binary-compatible with 2.5.0, or is that not the case and a release is needed?

Sorry I neglected to provide any context ...
Akka HTTP 10.0.2 isn't compatible with Akka 2.5.0.  I get the following exception when using embedded ElasticMQ 0.13.2 with Akka 2.5.0:
java.lang.NoClassDefFoundError: Fatal execution error, caused by akka/stream/Fusing$

which is fixed here: akka/akka-http@2e54e0fa4f4474547a910543d7e8daa1d551523b
So the bump to Akka 2.5.0 isn't strictly necessary, but perhaps good hygiene?
Thanks

Released in 0.13.3


Setup restcontroller for product

Backbone is finshed, but swagger does not document the Category property correctly. Need some googling later to see if I can find a workaround


Inconsistent playback and capture

zz.tar.gz

Description
Inconsistent playback and capture
Steps to reproduce
Environment

RenderDoc version: 1.33
Operating System: ubuntu 24.04
Graphics API: OpenGL ES

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


[2.x] Replace Travis and AppVeyor CI with GitHub Actions

This is a port of #1150 to the 2.x branch. It should also supersede #1152.

I had to fall back to pcov again since Xdebug is interfering: https://github.com/getsentry/sentry-php/pull/1154/checks?check_run_id=1522647653

Ok I should have shut down Appveyor correctly, and forced Xdebug 2 to avoid CI issues.
Closing and reopening to force a rebuild.

Well, even "Delete project" on Appveyor didn't work, it deleted it just for me...

AppVeyor has been disabled, by force pushing we should get rid of its status checks


Update for .NET 7 to work with MS Learn Sandbox. Other package refere…

Update to work with .NET 7 SDK included in the current Azure Sandbox tutorial for SAS signatures with storage accounts. The references for the below packages were also updated to the latest release builds from NuGet.
Azure.Identity
Azure.Storage.Blobs
Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation
https://learn.microsoft.com/en-us/training/modules/control-access-to-azure-storage-with-sas/4-exercise-use-shared-access-signatures

@microsoft-github-policy-service agree

Closing PR as duplicate.


Content Types Extended

This is more long-term and potentially not possible but it would be insane if:
// Would this let a content type define it's own constraints for the types rspc will accept?
impl<T: serde::Deserialize> RspcCompatible<Json> for T {
    // TODO
}

Taken from #200 that is incomplete.

[ ] Using futures joiners within resolver layer traits
[x] Refactor StreamOrFut and its sub-futures together - This ends up being hard cause Future vs Stream
[x] Replacing Stream with custom trait
[ ] Solution for batch decoding using framing -> working w/ Typescript
[ ] Solution for batch encoding using framing -> working w/ Typescript
[ ] Introduce ContentType & ContentTypes trait + generics based on Rspc
[ ] Using custom enum for Poll so custom values can be yielded through the chain (both up and down)
[ ] Remove the input: Value arg through layer system


nixos/mastodon: update service configuration

Motivation for this change
Changes:

Fix send e-mail notifications:

mail policyd-spf[20115]: prepend Received-SPF: None (mailfrom) identity=mailfrom; client-ip=...; helo=test.localdomain<EMAIL_ADDRESS>receiver=my@email

Added trusted proxy to correct view session ip.
Now mastodon uses unix connections by default.
optimize permissions - only a of mastodon group is allowed access.
fixed mastodon-init-db script

ев 14 20:55:22 tech mastodon-init-db-start[77967]: psql: FATAL:  database "mastodon" does not exist
фев 14 20:55:22 tech mastodon-init-db-start[77965]: /nix/store/rb6r505d9saxff81sjmnlh12sxsp8874-unit-script-mastodon-init-db-start/bin/mastodon-init-db-start: line 6: [: -eq: unary operator expected
фев 14 20:55:30 tech mastodon-init-db-start[77970]: Your database collation is susceptible to index corruption.
фев 14 20:55:30 tech mastodon-init-db-start[77970]:   (This warning does not indicate that index corruption has occured and can be ignored)
фев 14 20:55:30 tech mastodon-init-db-start[77970]:   (To learn more, visit: https://docs.joinmastodon.org/admin/troubleshooting/index-corruption/)

runnnig mastodon in sandboxing mode

cc @erictapen @happy-river
~@adisbladis~ (note from adisbladis: I'm not maintaining mastodon, please don't CC me on reviews for this package)
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@Izorkin Could you elaborate on that Fix send e-mail notifications? What is the bug here? Which service is throwing this log message?
Also did you test, that TRUSTED_PROXY_IP is necessary? I'm seeing the real IP addresses on the admin interface without using that variable and I never stumbled over the address of the proxy. Where would these show up?

By default with this configuration
  services.mastodon = {
    enable = true;
    configureNginx = true;
    localDomain = "mastodon.my-site";
    smtp.user = "admin";
    smtp.fromAddress =<EMAIL_ADDRESS>    extraConfig = {
      DEFAULT_LOCALE = "ru";
    };
  };

  services.nginx = {
    enable = true;
    virtualHosts."mastodon.my-site" = {
      forceSSL = lib.mkForce true;
      enableACME = lib.mkForce false;
      sslCertificate    = "/var/lib/cert/default.crt";
      sslCertificateKey = "/var/lib/cert/default.key";
    };
  };

Sidekiq service throws an error:
фев 14 07:46:16 tech sidekiq[34487]: 2021-02-14T04:46:16.262Z pid=34487 tid=46qn WARN: ArgumentError: SMTP-AUTH requested but missing secret phrase
фев 14 07:46:16 tech sidekiq[34487]: 2021-02-14T04:46:16.262Z pid=34487 tid=46qn WARN: /nix/store/mv22lz78wxx92kaywmxdfmy0vzpgmbjw-ruby-2.7.2/lib/ruby/2.7.0/net/smtp.rb:780:in `check_auth_args'

Need change services.mastodonauthenticate to false or configure send e-mail with external smtp server.
With services.mastodonauthenticate = true; service postfix sending e-mail on behalf of the local hostname - on behalf of the local hostname - helo=_local_hostname. Need change to helo=mastodon.my-site. Otherwise, the spam filter may delete the email message.
фев 13 17:21:05 mail postfix/smtpd[20068]: connect from unknown[2600:...]
фев 13 17:21:05 mail postfix/smtpd[20068]: Anonymous TLS connection established from unknown[2600:...]: TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature ECDSA (P-256) server-digest SHA256
фев 13 17:21:06 mail policyd-spf[20115]: prepend Received-SPF: None (mailfrom) identity=mailfrom; client-ip=2600:...; helo=tech.localdomain<EMAIL_ADDRESS>receiver=_my@e-mail
фев 13 17:21:06 mail postfix/smtpd[20068]: 4359E29: client=unknown[2600:70ff:b1c5:0:cfc9:30e9:ce1a:1555]
фев 13 17:21:06 mail postfix/cleanup[20117]: 4359E29<EMAIL_ADDRESS>фев 13 17:21:07 mail postfix/cleanup[20117]: 4359E29: milter-reject: END-OF-MESSAGE from unknown[2600:...]: 4.7.1 Try again later<EMAIL_ADDRESS>to=<_my@e-mail> proto=ESMTP helo=<tech.localdomain>

Yes, I checked TRUSTED_PROXY_IP. Before checking, need to re-login your account, then the session IP address will be updated. Without TRUSTED_PROXY_IP my ip address is shows <IP_ADDRESS>.

Added static uid and gid

Added configturations ti runnnig mastodon in sandboxing mode.

I'm having trouble reviewing all the changes at once. I guess merging the first 5 commits would be fine for me, but could you open a separate PR for nixos/mastodon: enable sandbox mode? I'd like to have a separate look on it, without letting the other commits waiting.
Also could you reword enable sandbox mode to something like harden module, as it is not a mode that can be disabled/enabled?
In general I'd prefer PRs opened on a feature-by-feature basis, not as a collection of useful changes.

Ok. Removing sandbox mode.
Added fix smtp authentication:
sidekiq[28674]: 2021-02-15T07:51:36.778Z pid=28674 tid=2vxe WARN: ArgumentError: SMTP-AUTH requested but missing secret phrase
sidekiq[28674]: 2021-02-15T07:51:36.778Z pid=28674 tid=2vxe WARN: /nix/store/mv22lz78wxx92kaywmxdfmy0vzpgmbjw-ruby-2.7.2/lib/ruby/2.7.0/net/smtp.rb:780:in `check_auth_args'

and
sidekiq[52823]: 2021-02-15T08:38:50.561Z pid=52823 tid=3rzr WARN: ArgumentError: SMTP-AUTH requested but missing user name
sidekiq[52823]: 2021-02-15T08:38:50.561Z pid=52823 tid=3rzr WARN: /nix/store/mv22lz78wxx92kaywmxdfmy0vzpgmbjw-ruby-2.7.2/lib/ruby/2.7.0/net/smtp.rb:777:in `check_auth_args'

Could you also split everything email related into a new PR? This currently breaks Confirmation emails for me and I'd like to merge the rest before tackling that problem.

Ok, updated PR.

@erictapen error working mastodon-init-db-start with remote database:
mastodon-init-db-start[68548]: psql: could not connect to server: No such file or directory
mastodon-init-db-start[68548]:         Is the server running locally and accepting
mastodon-init-db-start[68548]:         connections on Unix domain socket "/run/postgresql/.s.PGSQL.5432"?
mastodon-init-db-start[68546]: /nix/store/7rjxm6cmrkd9rifxpdaf8kbfrmb43vx8-unit-script-mastodon-init-db-start/bin/mastodon-init-db-start: line 6: [: -eq: unary operator expected
mastodon-init-db-start[68550]: Your database collation is susceptible to index corruption.
mastodon-init-db-start[68550]:   (This warning does not indicate that index corruption has occured and can be ignored)
mastodon-init-db-start[68550]:   (To learn more, visit: https://docs.joinmastodon.org/admin/troubleshooting/index-corruption/)

I can't fix the error.

@Izorkin I suppose this wasn't introduced by this PR but existed before?
Could you try setting PGDATABASE, PGHOST, PGPORT, PGUSER as env vars for mastodon-init-db.service? I have no setup with a remote database to test on.
We should test this in the NixOS integration tests I'm currently working on.

Temp fix:
diff --git a/nixos/modules/services/web-apps/mastodon.nix b/nixos/modules/services/web-apps/mastodon.nix
index 7df61376774..60b58c99ba5 100644
--- a/nixos/modules/services/web-apps/mastodon.nix
+++ b/nixos/modules/services/web-apps/mastodon.nix
@@ -436,7 +436,15 @@ in {

     systemd.services.mastodon-init-db = lib.mkIf cfg.automaticMigrations {
       script = ''
-        if [ `psql ${cfg.database.name} -c \
+        umask 077
+
+        PGPASSFILE=/tmp/pgpasswd$$
+        export PGPASSFILE
+        cat > $PGPASSFILE <<EOF
+        ${cfg.database.host}:${toString cfg.database.port}:${cfg.database.name}:${cfg.database.user}:$(cat ${cfg.database.passwordFile})
+        EOF
+
+        if [ `psql --host ${cfg.database.host} --user ${cfg.database.user} ${cfg.database.name} -c \
                 "select count(*) from pg_class c \
                 join pg_namespace s on s.oid = c.relnamespace \
                 where s.nspname not in ('pg_catalog', 'pg_toast', 'information_schema') \
@@ -446,6 +454,7 @@ in {
         else
           rake db:migrate
         fi
+        rm $PGPASSFILE
       '';
       path = [ cfg.package pkgs.postgresql ];
       environment = env;

However, it does not work with the local database.

Any idea to fix this error:
[error] 12367#12367: *436 upstream timed out (110: Unknown error) while reading response header from upstream, client: 2a0d:..., server: my_server, request: "GET /api/v1/streaming/? HTTP/1.1", upstream: "http://unix:/run/mastodon-streaming/streaming.socket/api/v1/streaming/?", host: "my_server"

@erictapen on your server is mastodon speed working? My new messages does not appear immediately - in a few minutes.

@erictapen found newf bug - not working video preview.

@Izorkin Nope, could you open an separate issue for that?

@Izorkin Nope, could you open an separate issue for that?

@erictapen fixed in this PR https://github.com/NixOS/nixpkgs/pull/113270


Humboldt Gymnasium Bad Pyrmont (hgbp.de)

@leonhoppe Thank you for your patience while waiting for a review and processing your request. Pull request merged.


[IOTDB-159] Fix NullPointerException in SeqTsFileRecoverTest and UnseqTsFileRecoverTest

JIRA link: https://issues.apache.org/jira/browse/IOTDB-159
To reproduce this bug:
The key to reproducing, is to test UnseqTsFileRecoverTest (or SeqTsFileRecoverTest) AFTER
first testing an integration test under the test/java/org/apache/iotdb/db/integration directory,
such as IoTDBAuthorizationIT.
git clone https://github.com/apache/incubator-iotdb.git
cd incubator-iotdb
mvn clean install -Dmaven.test.skip=true
cd server
mvn test -Dtest=org.apache.iotdb.db.integration.IoTDBAuthorizationIT,org.apache.iotdb.db.writelog.recover.UnseqTsFileRecoverTest

The cause of this bug:
FlushSubTaskPoolManager.getInstance() is shared between IoTDBAuthorizationIT and UnseqTsFileRecoverTest.
In the integration tests under the test/java/org/apache/iotdb/db/integration directory, tearDown() will call FlushSubTaskPoolManager.stop() and finally set FlushSubTaskPoolManager.pool to be null.
In UnseqTsFileRecoverTest (or SeqTsFileRecoverTest), test() will call:
performer.recover()
->redoLogs(restorableTsFileIOWriter)
->new MemTableFlushTask
->subTaskPoolManager.submit(encodingTask)

Therefore, if UnseqTsFileRecoverTest (or SeqTsFileRecoverTest) is tested behind integration tests, it will find that subTaskPoolManager.pool is null. Thus, NullPointerException is thrown.
To fix this bug:
Add FlushSubTaskPoolManager.getInstance().start() to setup() of SeqTsFileRecoverTest and UnseqTsFileRecoverTest. Accordingly, also add FlushSubTaskPoolManager.getInstance().stop() to tearDown().

very detailed analysis.


Avoid using deprecated RooDataSet constructor

The RooDataSet constructor that takes a weight variable name was deprecated in ROOT 6.32:
https://github.com/root-project/root/pull/14317
The general command argument constructor should be used, which doesn't change the functionality and is also backwards compatible with older ROOT versions.
Also, avoid a unused variable warning.

Thanks @guitargeek! I guess with ROOT < 6.32 it is not going to work, right? Should we add if statements and use the new constructors with ROOT version starting with 6.32?

It will work with any Root version. It was confusing to users which constructor they should actually use, hence ROOT 6.32 deprecated several redundant constructors.

ah, ok, from the documentation of the earlier version it is not that clear indeed, thanks!


Add a requirement system

Explained in #18

[09:40:13 INFO]: FireML issued server command: /daisylib requirementTest
[09:40:13 INFO]: [DaisyLib] [STDOUT] EXP:100
[09:40:13 INFO]: [DaisyLib] [STDOUT] Meets EXP Requirement: true
[09:40:13 INFO]: [DaisyLib] [STDOUT] HOLDING:STONE
[09:40:13 INFO]: [DaisyLib] [STDOUT] Meets Holding Requirement: true
[09:40:13 INFO]: [DaisyLib] [STDOUT] EXP:100
[09:40:13 INFO]: [DaisyLib] [STDOUT] HOLDING:STONE
[09:40:13 INFO]: [DaisyLib] [STDOUT] Meets both requirements: true

do i have at least 100 xp points? yes
am i holding stone? yes
do i meet both requirements? yes

[09:40:53 INFO]: FireML issued server command: /daisylib requirementTest
[09:40:53 INFO]: [DaisyLib] [STDOUT] EXP:100
[09:40:53 INFO]: [DaisyLib] [STDOUT] Meets EXP Requirement: true
[09:40:53 INFO]: [DaisyLib] [STDOUT] HOLDING:STONE
[09:40:53 INFO]: [DaisyLib] [STDOUT] Meets Holding Requirement: false
[09:40:53 INFO]: [DaisyLib] [STDOUT] EXP:100
[09:40:53 INFO]: [DaisyLib] [STDOUT] HOLDING:STONE
[09:40:53 INFO]: [DaisyLib] [STDOUT] Meets both requirements: false

do i have at least 100 xp points? yes
am i holding stone? no
do i meet both requirements? no

This new feature allows multiple different requirements to be checked at once, as seen above. An example configuration for something like this would be as follows:
requirements:
  - "exp:100"
  - "holding:stone"
  - "permission:example.permission"


leak users

Hi,
The Django REST framework leaks the list of users (with emails) through the "collaborators" selector of the HTML competition submission form (bottom of the page https://www.codabench.org/api/competitions/).

Edit: Seems it only works when logged in 🙌. The extraction is possible through the GUI "Add collaborator" auto completion feature, but giving out the list like that might not be desired.

Good catch, thank you for reporting this problem.
When using the editor through the UI to add collaborators, the behavior is as excepted:
Not able to see the email when not logged in as an administrator of the platform:

Able to see the email when logged in as an administrator of the platform:

It seems that, using the REST API, the emails are always given? That is an important leakage that we need to fix.

Emails are indeed given although I'm no admin. I reported the issue while looking at /competitions but the email issue is more obvious by looking at https://www.codabench.org/api/participants/. Seems the API is not following the email access rights you describe for the UI (visible only when admin). I'm not familiar with Django REST framework but I guess
the issue is with CompetitionParticipantViewSet or CompetitionParticipantSerializer.

I close this issue to keep track of this directly in the PR.

I re-open this issue because #929 got reverted. Indeed, this change is breaking the organizer interface used to approve / revoke participants (the buttons simply stops working).
We therefore need to:

[ ] Fix the change to merge it again
[ ] Check that is does not break anything
[ ] Add tests for the participants tab #996

Related:  #1008

@Didayolo this issue is done


techmap creates $MUX cell with 1'bx input (correct, but less efficient)

Steps to reproduce the issue
./yosys -p "proc; techmap; opt; dump" case1.v
Minimised from https://github.com/cliffordwolf/picorv32/blob/e0baf2e0bd49fdddef2e3440c1f6364478655154/picorv32.v#L1110-L1124
module top(input instr_lui, instr_auipc, instr_jal, output reg decoded_imm);
    always @*
            (* parallel_case *)
            case (1'b1)
                instr_jal:
                        decoded_imm <= 1'b0;
                |{instr_lui, instr_auipc}:
                        decoded_imm <= 1'b1;
                default:
                        decoded_imm <= 1'bx;
            endcase
endmodule

Expected behavior
My understanding is that opt_expr is expensive to run, particularly on a bit blasted netlist. We should avoid emitting 1'bx if possible?
Actual behavior

Never mind. Let me think about this some more.


clear and reset env vars for test to work properly

Different environments have different values for OS_* env vars. Also as we call setenv in this test, we should reset the variables when we are done

recheck

FAIL: TestRoutes is related to the CI job itself. The failure for TestGetConfigFromEnv we saw in [1] is green.
cc @animationzl @kiwik @liu-sheng
Thanks,
Dims
[1] http://logs.openlabtesting.org/logs/84/84/5a010ea01b7fa842ff1e845748b4b5eaa65781f1/check/openstack-cloud-controller-manager-unittest/662e175/job-output.txt.gz


chore: bdd-spring-1582928665 to 0.0.1

chore: Promote bdd-spring-1582928665 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
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:

OWNERS

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


prismlauncher/prismlauncher-qt5: update to v6.0 & add associations

Adds an optional associations registry file that supports the new feature to import mod-packs from the command line. https://github.com/PrismLauncher/PrismLauncher/pull/266
This also updates the launcher to v6.0 and switches to MSVC, as recommended by Scrumplex (a developer).

[x] I have read the Contributing Guide.

maybe you could add extras/vcredist2022 to suggests too

okay


playground is not working

https://lukasmasuch-streamlit-pydantic-playgroundplayground-app-711bhu.streamlit.app/

Fixed :) the playground is working again: https://st-pydantic.streamlit.app/


Rid of unreachable

rid of some unreachable!

Coverage remained the same at 94.937% when pulling 90f32050405b157bcf78f8ea7f8a1b38a8d785ad on revise-unwraps into b5eb87d5fd8cffdb0109a70216335096a548373e on master.

Codecov Report

Merging #134 into master will increase coverage by <.01%.
The diff coverage is 92.3%.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
+ Coverage   95.55%   95.56%   +<.01%
==========================================
  Files          18       18
  Lines        1867     1870       +3
==========================================
+ Hits         1784     1787       +3
  Misses         83       83

Impacted Files
Coverage Δ

src/frame/events.rs
89.16% <92.3%> (ø)
:arrow_up:

src/frame/frame_ready.rs
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Error when adjusting the Voxel size

Thank you for your great project! I get the following error when attempting at changing the voxel size from [0.05, 0.05, 0.1] to [0.16, 0.16, 4].
RuntimeError: /root/spconv/include/tensorview/helper_launch.h 17
N > 0 assert faild. CUDA kernel launch blocks must be positive, but got N= 0
It would be great if you can point out the potential solution for this :)
Thanks!

No worries, I figured this out.

hi @yaohungt ,
I ran into the same error, could you please kindly share your solution to this problem?
Thank you in  advance!

I got the same error, could you tell me how to solve this problem please?
Thanks you! Thank you very much.  @yaohungt    @turboxin

@Chi-Zaozao maybe try reduce voxel size in z dimension

@yaohungt I too tried to increase the voxel size, and end up with this error as well. how did you solve it?
@turboxin @Chi-Zaozao did you solve the problem? Thanks

The voxel size should be adjusted properly to ensure that the size of the generated feature maps can be correctly down sampled in the subsequent networks.

Hi, I also occurred this issue. How can I solve this plz..............

The voxel size should be adjusted properly to ensure that the size of the generated feature maps can be correctly down sampled in the subsequent networks. @philRudd69


fix: unable to show view as table modal

SUMMARY
fix the "view as table" in the dashboard is unable to show when the second load.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before
https://user-images.githubusercontent.com/2016594/183597857-150195d9-ec93-401c-97fa-9399d58c592c.mov
After
https://user-images.githubusercontent.com/2016594/183598309-f0748201-8011-4ff9-a07c-ac3cc8a3a786.mov
TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up


how to solve it?

when I run the script this massage appear
`Traceback (most recent call last):
File "faitagram", line 254, in
main()
File "faitagram", line 63, in main
Bruter(service, username, wordlist, delay).execute()
File "faitagram", line 89, in execute
if self.usercheck(self.username, self.service) == 1:
File "faitagram", line 106, in usercheck
driver = webdriver.Firefox()
File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/firefox/webdriver.py", line 152, in init
self.service.start()
File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/common/service.py", line 76, in start
stdin=PIPE)
File "/usr/lib/python2.7/subprocess.py", line 394, in init
errread, errwrite)
File "/usr/lib/python2.7/subprocess.py", line 1047, in _execute_child
raise child_exception
OSError: [Errno 8] Exec format error
`

me to have this error one one help us pleas


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 291

Questions ?
Please refer to the Backport tool documentation


SYNTAX ERROR coming from the package

The code:
from pilmoji import Pilmoji
The error:
File "/usr/local/lib/python3.7/dist-packages/pilmoji/sync/core.py", line 103     asset = asset.resize((width := int(emoji_size_factor * font.size), width), Image.ANTIALIAS)                                 ^ SyntaxError: invalid syntax
From my observation, the error came from the package.
@jay3332

Turns out my python version (on Google colab) is Lower than the required version, sorry for creating a wrong issue. @jay3332


Update Traefik docker-compose config to ensure DNS resolving and auto…

…discovery with multiple networks
Pull Request
Please write all text in English in order to facilitate communication and collaboration. Thank you!

This config ensures that Traefik properly discovers containers in the frontend network. I faced some issues when using multiple networks and services

Please consider ;)

Had some issues with intedation.
Should be fixed. This should help when using multiple networks and not all should be exposed.


How can I get the client's info on every api calls?

I want to log the client info on every api calls (not just authentication). How can I get that?
Here is my api config :
services.AddAuthentication(defaultScheme: IdentityServerAuthenticationDefaults.AuthenticationScheme)
        .AddIdentityServerAuthentication(options =>
        {
            options.Authority = "https://localhost:5000";
            options.ApiName = "api1";
            options.ApiSecret = "secret";

            options.EnableCaching = true;
            options.CacheDuration = TimeSpan.FromMinutes(10); // that's the default
        });

    // adds an authorization policy to make sure the token is for scope 'api1'
    services.AddAuthorization(options =>
    {
        // some policies here
    });

I ended up with catching this event :
        options.OAuth2IntrospectionEvents.OnTokenValidated = async (context) =>
                    {
                        var identity = context.Principal.Identity as ClaimsIdentity;
                        identity.AddClaim(new Claim("my_claim", "my_calim_value"));
                    };


Quality metric ease of use methods

Codecov Report
Patch coverage: 68.42% and project coverage change: -0.05 :warning:

Comparison is base (cf45384) 86.72% compared to head (36782ff) 86.68%.

:exclamation: Current head 36782ff differs from pull request most recent head 33b4760. Consider uploading reports for the commit 33b4760 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #388      +/-   ##
==========================================
- Coverage   86.72%   86.68%   -0.05%
==========================================
  Files         185      185
  Lines       23314    23363      +49
==========================================
+ Hits        20219    20252      +33
- Misses       3095     3111      +16

Impacted Files
Coverage Δ

...g/engines/compilers/conditional_effects_remover.py
90.96% <0.00%> (-0.60%)
:arrow_down:

...ngines/compilers/trajectory_constraints_remover.py
77.51% <0.00%> (-0.31%)
:arrow_down:

...g/engines/compilers/negative_conditions_remover.py
93.87% <50.00%> (-0.49%)
:arrow_down:

..._planning/engines/compilers/quantifiers_remover.py
91.47% <50.00%> (-0.72%)
:arrow_down:

unified_planning/model/metrics.py
75.26% <71.42%> (-1.09%)
:arrow_down:

...ngines/compilers/disjunctive_conditions_remover.py
87.22% <100.00%> (+0.05%)
:arrow_up:

unified_planning/engines/compilers/grounder.py
97.95% <100.00%> (+0.01%)
:arrow_up:

unified_planning/engines/sequential_simulator.py
78.96% <100.00%> (+0.08%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


EntityWithHashchain.Event.mutate is never called

I noticed that in the code for EntityWithHashChain.Created, the super call does not put in the class of self, but of its parent:
        def __mutate__(self, entity_class=None):
            # Call super method.
            obj = super(EntityWithHashchain.Event, self).__mutate__(entity_class)

            # Set entity head from event hash.
            obj.__head__ = self.__event_hash__

            return obj

This means the __mutate__ of EntityWithHashChain.Event is skipped.  In fact, fixing the super call will make the example code in the quickstart guide fail.  Probably the fix should be to remove the method.

Wow, well spotted! :)) you're good!

@chanhosuh I fixed it like this: https://github.com/johnbywater/eventsourcing/commit/94ffec3368d8a12b32eafaef6690809622a807b0

Tests passed so I'll close this. Please reopen if you need to. Many thanks for spotting this issue, and if you spot anything else, please don't hesitate to open another! :)

@johnbywater that's great!  I was feeling guilty about posting a sloppy issue and was planning to write a formal PR so I could run your tests etc... but you've pre-empted me :)


[FEATURE]

Description
Hi there! I was going through this review of Linktree. It mentioned that the only pro users are given the option to change their background. I think what can make Linkfree stand out is giving a similar option for free. What do you think?
Screenshots
No response
Additional information
No response

@CBID2 It's a great idea, we should definitely try it out.

Yeah sure! ping me whenever you are stuck or needed any kind of guidance.

Great thanks

Hey @money8203! :) Here's a rough draft of my solution. I followed [this tutorial] ( https://www.geeksforgeeks.org/design-background-color-changer-using-html-css-and-javascript/) to create it. For some reason, the CSS and JS is not working.

This requires a lot of thought as it could make someones page unusable if the colours of the background image clashes with the buttons
I think we will close this for now as we have other more requested features


Cherry-picks for the 4.2 branch (future 4.2.2) - 3rd batch

Long time no see 4.2...
WIP, I'll add more commits.
For now this batch includes:

Buildsystem changes
Thirdparty changes
Some high priority bug fixes and regression fixes

Added a few topic:rendering cherrypicks. The rest need to be discussed with Clay: https://github.com/godotengine/godot/pulls?q=is%3Apr+label%3Acherrypick%3A4.2+is%3Aclosed+label%3Atopic%3Arendering

Added a batch with all the documentation cherry-picks.

Added a batch with a handful of editor fixes (did not include all of them, some looked riskier or didn't cherry-pick cleanly).

And added a batch with platform specific bugfixes.

This is now ready for review.
I'd welcome some help to check the list of commits I cherry-picked, and flag any that seems potentially risky / best to drop for now and not cherry-pick to 4.2.x or consider for a later build.

Nitpick but if Transform3D class reference overhaul is in, so should https://github.com/godotengine/godot/pull/87175 and https://github.com/godotengine/godot/pull/87181 as they heavily borrow from one another.

Added a few topic:rendering cherrypicks. The rest need to be discussed with Clay: https://github.com/godotengine/godot/pulls?q=is%3Apr+label%3Acherrypick%3A4.2+is%3Aclosed+label%3Atopic%3Arendering

I removed the label from the PRs I don't think should be cherrypicked. The remaining ones are safe to include if you want

I don't see anything on the list that needs to be removed! Great work

I had a look at PRs, that I was involved in and are looking good.

I could not find https://github.com/godotengine/godot/pull/87346 in the list. It is critical to have this change.

I could not find #87346 in the list. It is critical to have this change.

I need to assess the pros and cons with @m4gr3d. I was earmarking it for 4.2.3 and trying not to increase the scope too much for 4.2.2, but this batch makes it somewhat big already so we might want to go all the way.

I appreciate, reviewing again. I have been trying to build plugins on top of 4.2.2 RC and if merged let me know I will do some testing for android export as well as plugin creations. If anything, else needs to be tested also let me know.

I could not find #87346 in the list. It is critical to have this change.

I need to assess the pros and cons with @m4gr3d. I was earmarking it for 4.2.3 and trying not to increase the scope too much for 4.2.2, but this batch makes it somewhat big already so we might want to go all the way.

Let's give it a go, and see how it fares under testing :)

Awesome!! thanks

I didn't know where to mention this, but initial tests on https://github.com/godotengine/godot/pull/87346 shows it is working fine.


wine fail to build on macOS

Describe the bug
A clear and concise description of what the bug is.
Steps To Reproduce
$ nix profile install nixpkgs#wine64Packages.stable
error: builder for '/nix/store/v154yglg5d83c8bpvk0h1nbcx0sg76gg-wine64-8.0.drv' failed with exit code 2;
       last 10 log lines:
       > dlls/ntdll/unix/loader.c:2247:10: error: use of undeclared identifier 'pthread_attr_set_qos_class_np'
       >     if (&pthread_attr_set_qos_class_np)
       >          ^
       > dlls/ntdll/unix/loader.c:2248:9: warning: implicit declaration of function 'pthread_attr_set_qos_class_np' is invalid in C99 [-Wimplicit-function-declaration]
       >         pthread_attr_set_qos_class_np( &attr, QOS_CLASS_USER_INTERACTIVE, 0 );
       >         ^
       > 1 warning and 1 error generated.
       > make: *** [Makefile:99281: dlls/ntdll/unix/loader.o] Error 1
       > make: *** Waiting for unfinished jobs....
       > 2 warnings generated.
       For full logs, run 'nix log /nix/store/v154yglg5d83c8bpvk0h1nbcx0sg76gg-wine64-8.0.drv'.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-darwin"`
 - host os: `Darwin 21.6.0, macOS 10.16`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.11.1`
 - channels(yihuang): `"nixos-22.11, nixpkgs"`
 - channels(root): `""`
 - nixpkgs: `/Users/yihuang/.nix-defexpr/channels/nixpkgs`

Tested and appears to no longer be failing as of 683f2f5ba2ea54abb633d0b17bc9f7f6de
/nix/store/avq6wph7nacnk0243y2a0c5zmksbnr0d-wine64-8.0.1


Brew formula upgrade

Issue Type

Feature request

Brew still contains molecule v1.20, I can make a PR for now but definitely won't be able to keep it up to date, can brew be added to a release process?

We don't manage this formula, nor is it a supported install method of Molecule.


navigateTo in middleware throws ERR_HTTP_HEADERS_SENT when redirecting to external URL on route without path

Environment

Operating System: Darwin
Node Version:     v18.12.1
Nuxt Version:     3.2.3
Nitro Version:    2.2.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction

Clone and run https://github.com/MikeBellika/navigate-to-500 locally
curl http://localhost:3000/notfound
Look at logs

Describe the bug
When using navigateTo in a middleware the following error shows up in the terminal:
[nitro] [dev] [unhandledRejection] Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at new NodeError (node:internal/errors:393:5)
    at ServerResponse.setHeader (node:_http_outgoing:644:11)
    at Object.errorhandler [as onError] (file:///Users/mike/code/playground/navigate-to-reproduction/.nuxt/dev/index.mjs:457:20)
    at Server.toNodeHandle (file:///Users/mike/code/playground/navigate-to-reproduction/node_modules/h3/dist/index.mjs:1278:27)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  code: 'ERR_HTTP_HEADERS_SENT'
}

This error only happens if:

The navigateTo points to an external site (with external: true)
The original path doesn't have a route

The behaviour is different in dev mode and in production:

When running in dev mode, the error shows up every time in the console, however the redirect still happens.
When running in production on Vercel, the error is flaky and occurs on every ~1/3 requests. The redirect doesn't happen and the server returns a 500.

Additional context
We're also seeing this error on paths that exists. This error is much more flaky and I haven't been able to reproduce it locally. From my testing it seems like it happens on approximately 1/1000 requests.
Feel free to move this issue upstream, if it's not a nuxt problem.
Logs
No response

Has the same issue with redirects on my Vercel production :(

I found the temporal solution. It is a script that patches build files. Should be called after the build step on Vercel:
import fs from 'node:fs'
import path from 'node:path'

const original = fs.readFileSync(
  path.join(process.cwd(), '.vercel/output/functions/__nitro.func/chunks/nitro/vercel.mjs'),
  { encoding: 'utf8' },
)

console.log('Pathing...')

const newVercelFunction = `
const vercel = (function(req, res) {
  const query = req.headers["x-now-route-matches"];
  if (query) {
    console.log('query detected');
    const { url } = parseQuery(query);
    if (url) {
      req.url = url;
    }
  }

  function debounce(func, delay) {
    let timeoutId;

    return function (...args) {
      const context = this;

      clearTimeout(timeoutId);

      timeoutId = setTimeout(() => {
        func.apply(context, args);
      }, delay);
    }
  }

  const debouncedEnd = debounce(res.end, 10)
  res.end = debouncedEnd

  return handler(req, res);
})

`

const pathed = original.replace(/const vercel = \(.*\)/gs, newVercelFunction)

fs.writeFileSync(
  path.join(process.cwd(), '.vercel/output/functions/__nitro.func/chunks/nitro/vercel.mjs'),
  pathed,
)

Some extra info: We switched to using routeRules for redirects on vercel, which fixed this bug. We can't redirect URLs with query params with this method but we don't have too many of those redirects.
Before we switched, we could see in the logs that we had a lot of ERR_HTTP_HEADERS_SENT errors in the logs. Multiple every minute and probably around 1-2% of all requests would give a 500 because of this. This happened even on pages without redirects. After removing the middleware this went to 0.


如何在Electron+Vue3+Vite中使用？

按照README中Vite的Demo进行配置后，未成功加载出编辑器，控制台没报错。

Yes. Tried to debug, and it seems that its not properly initialising component :(
Seems to be failing here:


 Thanks!

S, M, L, XL, XXL, XXXL would be convenient options

S, M, L, XL, XXL, XXXL would be convenient options

I'd prefer metric system. km, m, cm.
Also, we could use the fractionals. Like slices.

1+ for this feature


How to use cvxpy to cross and multiply vectors

Is your feature request related to a problem? Please describe.
I want to use the cross product of vectors as constraints in constraints, but I didn’t find the operation in the API documentation

Cross product is not an atomic operation. You can construct the cross product yourself using bmat.


Add support for @Min bean validation annotation

🆕🐥:dog: First Timers Only
This issue is reserved for people who never contributed to Open Source before. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
:space_invader: Description of the issue
Currently the dolphin-platform-bean-validation module do not provide support for all default bean validation annotations. One that is missing is the @Min annotation. For solving this issue the support for the @Min annotation must be added like it is done for several other bean validation annotations in the dolphin-platform-bean-validation module and at least one unit test must check that the annotation is supported.
📋 Step by Step

🙋 Claim this issue: Comment below.
🔄 replace the up for grabs label with in progress.
:fork_and_knife: fork the repository in github by simply clicking the 'fork' button.
:arrow_heading_down: check out the forked repository
:twisted_rightwards_arrows: create a feature branch for the issue. We do not have any naming definition for branches.
💾 Commit your changes.
🔀 Start a Pull Request.
🏁 Done 👍 Ask in comments for a review :)
🔬 If the reviewer find some missing peaces or a problem he will start a discussion with you and describe the next steps how the problem can be solved.
:boom: you did it! We will merge the fix in the master of the Dolphin Platform project.
:bouquet: Thanks, thanks, thanks for being part as an open source contributor for Dolphin Platform

🤔❓ Questions
You can join our community chat at Slack

I'll try this too.

@Ampretuzo go for it :)


"Sorting elements" string

Dear Alberto!
Can you tell me what is the meaning of "Sorting elements" string?
It takes a lot of time to proceed this step. Is it possible to fasten it somehow?

P.s. Very strange situation. After 1 hour of "sorting" processing the remote server (i'm using remote services) reports "connection error" and the remote session disconnects. Very pity ((( And how to start searching with "200M elements" i don't know. Probably i need my own home server ))

Please avoid using bsgs mode with more of 100K publickeys that process of sorting is fast even with 200 Millions items.
Did you use the -q option right?
Please alwayss add your fulln line of execution.

-q option quiet the thread output
speed is not showing until one thread finish his process.

The las one is made in that way to not show 0 keys/s
Wit more of 100 Millions publickeys please use xpoint mode.
Best regards

Sorry for inconvenience. Yes, I used the -q option. I used -k 960 for the first time, then a terminated process and run again. And it took only 50 minutes to start. The problem is solved! Excuse once again.

is ok, i will Add some ".. Done!" string when the sorting process will be complete.
Please don't close this issue until that par was done.
Regards!

Solved


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
14 seconds
sanity-inspect-image

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
10 seconds
sanity-label-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
4 seconds
show-summary


Support Orion model

Support Orion model:  https://huggingface.co/OrionStarAI/Orion-14B-Base

加油加油

Much appreciated for adding this!

thanks for adding this

Is this work still in progress?

Even this work hasn't been merged yet, I tested on dachengai:orion branch. It worked! Thanks for your contribution 😊

i wonder where dachengai:orion branch is? and how can i test this model with vllm

Even this work hasn't been merged yet, I tested on dachengai:orion branch. It worked! Thanks for your contribution 😊

i wonder where dachengai:orion branch is? and how can i test this model with vllm

dachengai:orion branch is here
, but i only added a few changes like model config and init shown in Files changed tab.
You can test this model by changing the model path or name to Orion!
You can download it from huggingface.

Much appreciated for adding this!

Even this work hasn't been merged yet, I tested on dachengai:orion branch. It worked! Thanks for your contribution 😊

i wonder where dachengai:orion branch is? and how can i test this model with vllm

dachengai:orion branch is here, but i only added a few changes like model config and init shown in Files changed tab.
You can test this model by changing the model path or name to Orion! and you can download from huggingface.

many thanks, it works well for me.

Any updates?

Why has it taken so long and still can't be merged in?

@esmeetu I addressed your review and updated the PR. Thanks again for the review!


σ behaves oddly

I suspect this is to do with the uncertainties system, with the parser interpreting σ as an uncertainty. @bshlgrs?

I decided to remove the ability to enter uncertainties since, as far as I can tell, you never actually use this.


Fake PR to test release branch

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: hklai
If they are not already assigned, you can assign the PR to them by writing /assign @hklai in a comment when ready.
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:

release/OWNERS

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-unit-tests.sh
a9ef42da6275336ba3d59996227f7a361c6e99cb
link
/test istio-unit-tests

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-unit-tests

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rkpagadala
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:

release/OWNERS [rkpagadala]

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

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rkpagadala
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:

release/pipeline/OWNERS [rkpagadala]

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

no longer needed, merged by other PRs


Update DataSegment.cs

True null verification

Thanks!
Can you make 2 minor style modifications?

Change
{ }
with
{}

Put spaces around ==

Done …;)


Search notifications

Add a search input to the bottom of the menu for easily filtering organisations and repositories.

Awesome!! :rocket:


Fix issue with missing DEFAULT_ICON variable

When installed in a docker installation the plugin bails out with a missing variable error

:+1 came here to post this exact PR.


Made the whole website responsive for even small screens and also improved the UI.

Please add hacktoberfest-accepted label to the merged pull request.

hacktoberfest-accepted label added

Thanks


[Icon Request] Support for API-Extractor

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: filename
Icon Name: api_extractor
Sample original Icon:
Filenames: api-extractor.json, api-extractor-base.json

More info: https://api-extractor.com/

Beautiful icon if you ask me. The only issue would be the background that should make it have a light version too.

We could use it without the circular background. Experiment on it.


Support for CLI

Describe the feature
Context
I am currently using unjs/citty to make CLI tools.
While creating CLI tools, I have been saved by many unjs stacks such as

🌆 unjs/citty
🐨 unjs/consola
💅 unjs/changelogen
📦 unjs/unbuild
🛣  unjs/pathe
🍟 unjs/jiti

There are others that are needed, but we believe the above are the minimum.
So, I would like to provide unkit for those who want to make CLI with unjs in the future.
Design

add src/cli.ts
add exports.cli options in package.json

Additional information

[X] Would you be willing to help implement this feature?

### Tasks

Hi @pi0 .
I would appreciate a review when you have time.
I'll create a Pull Request if you're okay with the policy.


How to hide the color-mixer with css?

On smaller screens in landscape mode it takes the most of the viewport space,
while a hue slider would be enough for the task.
Is it possible to hide or remove somehow the color mixer with css?
So far i tried with the display: none property, but it causes the mixer output only #NaNNaNNaN for choosen color.

Please stick to the Issue Template. Create a JSFiddle and describe what you've done as detailed as possible and break your issue down to a minimum. This will help me to resolve it as fast as possible.


Delete Beacon Process Not Working

I sent a request 4 days ago to delete my beacon, following a PDF documentation at the following address: http://www.gridresearchcorp.com/gridcoin/DeleteBeaconInstructions.pdf
The documentation says you should receive a response (with a return transaction of either .00011 or .00022) but after four full days I have received nothing. I tried sending the request a second time three days ago.
This was a result of my config file getting corrupted and me not having a backup (I had a backup of the wallet.dat file but not the config file) so currently I've got mismatched keys. Since I have no way to get to a non-corrupted copy of my old config file, I need to have my beacon deleted so I can re-send and get sync'd back up.
@GridResearchCorp

I was asked to close this issue because I was able to talk with someone on the IRC channel. I still believe there may be something wrong with this process, however. The linked procedure indicates that this is an instantaneous process (within one hour) so either the process is broken or perhaps the documentation/procedure should be altered? I just want to help so future GRC noobs like myself don't end up running into this and end up getting frustrated and move on to a different Cryptocurrency.
I deposited 35,000 GRC into my account last week to ensure timely payments. Right now I'm concerned about the security of those funds as well as the fact that I will not receive any payments until I'm able to get my beacon deleted and re-sent. As a security measure I may just transfer the 35,000 GRC out of my wallet and convert back to USD until this gets resolved. My concern is at some point I'm going to lose "control" of the funds in my account since the blockchain does not correlate my CPID with my address anymore.

you would not lose your funds regardless of cpid situation. as long as u have a backup of your wallet.dat the balance is always urs and under your control. your grc is tied to your wallet. the DPoR is tied to cpid.

Thanks for that explanation. I am new to CryptoCurrency and I'm still learning how exactly the blockchain works.
The person I spoke with in the IRC channel said the deletions are done in blocks and it looked like that only happens a couple times per month. I do still think a revision to the documentation for deleting a beacon is in order if that's the case. Is anyone willing/able to delete the beacon manually for me? I'd be happy to donate some GRC for your time.

the guy doing the beacon deletion i believe has a script that deletes them and it can take a bit for it to work. you need to patience. also if u advertised multiple beacon in the past little while it can cause some issues. and only your wallet.dat has the private keys to ur grc addresses. the transactions, etc are stored on the blockchain. You also haven't said why u need to delete your beacon here.

Sorry about that - I don't mean to sound impatient. I'm just very enthusiastic about GridCoin. :-)
I mentioned briefly in my first post, but my computer crashed overnight this past weekend while my GridCoin wallet was open, causing my config file to become corrupted. Once I got my system up and running again, I had a runtime error when trying to launch the GRC wallet. I did a search on the GridCoin Reddit and found an old topic that suggested I delete all files except my wallet.dat file and then re-run the GRC Wallet, allowing it to re-download and sync the blocks. Doing this allowed me to launch the wallet again and download blocks, but now my wallet console shows "CPID Invalid" and I haven't received any payments since. My GRC balance is correct and it even shows the correct magnitude, but I'm not able to claim my DPOR rewards I assume due to this CPID Invalid error.
Unfortunately I didn't know at the time that I needed a backup of my config file in addition to my wallet.dat file. I had plenty of backups of the latter but no backups of the config file so I believe that's what causes the key mismatch. I was told on the Reddit that one way to fix this would be to delete my beacon and send a new one.
Hope that helps.

ok that makes sense. keep checking beaconreport to see if ur cpid is in there. and yes the privatekey is not recoverable so deleting would be only way. make sure u sent the transaction to delete ur beacon from the main grc address that was used for registering ur beacon

I'm good there -- I only had the one GRC Address so far so I (hopefully) didn't screw that up - haha.
Thanks for talking me through this. Hopefully that script will run soon and my beacon will get deleted. I'll keep crunching on the interim.

Address in beacon for CPID 27d542179e931069dc31a42b0f512392 is S4HryKNwmKfoi66co9raMqQtEHbb48YWke and was sent from address S4HryKNwmKfoi66co9raMqQtEHbb48YWke (is indeed beacon tx for cpid, not visible in explrer). According to block explorer there are no small outgoing transactions from that address, only the one that you converted back to usd.

You beat me to it @tomasbrod . I saw your earlier post before you edited and I was going to say I think it just looks funky because the change address issue. My wallet still shows the ~35,000 GRC but (like you said) the change output isn't associating with my CPID.
Thanks for checking to make sure I did that correctly. I even tried sending the Delete Beacon to a different address associated with RTMoney's account on the off-chance that would work (it obviously didn't).
I think at this point I just have to keep crunching and wait for the request to finally go through. I've had various people assure me that this doesn't penalize my mining efforts on the interim so I guess no harm done. I just need to make sure I keep better backups from now on.

Thanks again everyone for the help. I'm going to close this issue but I do recommend updating this procedure/documentation (http://www.gridresearchcorp.com/gridcoin/DeleteBeaconInstructions.pdf) to be clear on the following:

You will not receive a response between "15 minutes to an hour."
The delete beacon requests are done in batches and it could take 2-3 weeks before the next batch is processed.

I also think when you press the "Backup Wallet" button on the Wallet console, it should back up both your wallet.dat file and your Config file. It was frustrating thinking I was backing up the correct file only to have people later tell me that the backup feature in the wallet is insufficient in the event that your config file becomes corrupt (which can happen from a simple computer crash that is not uncommon to BOINC users) to restore your wallet back to its previous state.
I would be happy to contribute toward any documentation updates. I am not a programmer but I would love to contribute to GridCoin research in other ways :-)

It is planned to store beacon keys inside the wallet.dat file.
we in the process of designing new beacon system that will hopefully not require deleting
by creating this issue @XaqFields created a pointer from higher search ranked github to the beacon deletion guide

Sever ran out of disk space around the 25th and I didn't know the wallet was stuck.  It is back up and running, noticed 15-20 transactions went through, should be a few beacons deleted next superblock. When server is in sync and script is running correctly you should get a reply from a beacon delete request within a few minutes if you send it to the right address.

@GridResearchCorp - sounds good, thanks for fixing that. I had a feeling there was something going on because I tried to trace back transactions on your account and it appeared as though ~May 25th there were regular delete beacon responses being sent out (small transactions for 0.00011 GRC) that's why I was stuck on this idea that something was going on - lol
Again - much appreciated! If there's anything you guys need help with from a non-programmer, please reach out to me. I'm willing to help out in any way I can.


Add AllowedMethods option to Style/OptionalBooleanParameter cop

Follow up https://github.com/rubocop-hq/rubocop/pull/8725
Closes https://github.com/rubocop-hq/rubocop/issues/8489

Looks good! Thanks!


Sectioned picker

Hi
How do I set sections for the CZPicker?

@godara you mean the internal tableview's sections? if that is the case, CZPicker now only supports one section in inner tableview and cannot set sections for now.

I think you should add this feature.  This is one of the most important thing, I also need it.


Implement cancellable overloads of all DbDataReader extension methods

Implement cancellable overloads of all DbDataReader extension methods.
Closes #26.

Sure.
I can't find the Test EXPRESS database used by StoredProcedureEFCore.Tests.SqlServer.
I see no migrations that create it either.
Was it supposed to be included with the codebase?

@verdie-g - I added tests to StoredProcedureEFCore.Tests.SqlServer.Program as that was the only place where one could observe actual cancellation behavior against SQL Server.
The test runs 16 cases where a long running (10 seconds) stored proc is cancelled approximately 1 second after it is started.
The tests focus on two different cancellation use cases:

stored proc cancellation before a resultset is present
stored proc cancellation after a resultset has already started streaming to the client

To observe the tests, apply the new version of dbo.ListAll to your Test database and run StoredProcedureEFCore.Tests.SqlServer. You should see output for each of the 16 test cases being started and then cancelled within a second.
Notes:

Please review the new signature of dbo.ListAll - added parameters to fake the delay of the sproc before and after the resultset has been streamed to the client.
Please review the new logic of StoredProcBuilder.ExecAsync and how it reacts to exceptions thrown from the passed in action lambda.
Please review the new logic of DbDataReaderExtensions.FirstAsync and how it pumps all the records from the reader in order to enable cancellation of long running stored procs.

Hmm indeed there are no tests for Async methods because DbDataReader.*Async methods are not virtual. I'll review your PR this week-end or next week.

I'm not using Windows anymore so I don't have access to an SQL Server. I'll install a VM or I'll update the stored procedures for pgSQL.

You can tun SQL Server in a Linux container 😋
https://hub.docker.com/_/microsoft-mssql-server


COST-1008 - add proxy config to http transport

Not suggesting this is wrong, but the insights-operator code is a bit more in depth (could just be based on something older 🤷 ): https://github.com/openshift/insights-operator/blob/0ab6950a140c77b771b10fa7716c1dd83a03804a/pkg/insights/insightsclient/insightsclient.go#L97-L120

That's a good point Chris. I think our issue is actually how we are creating the transport for the client with certificates.
I updated this PR to copy in the default transport from the http package, and add the certs to that transport instead. This way our cert based client has all of these other parameters that we were ignoring previously.


Fixed misspelling of RecordTimespanToFileAsync

@mscherotter: Thanks for your contribution. Going forward, CX asks that Microsoft contributors use VSTS to submit documentation changes. Future contributions via GitHub will be closed with a request to resubmit them via VSTS. More information is at http://aka.ms/windowsauthoring (see UWP instructions > UWP/WinRT API content). Thanks!

@drewbatgit : Drew, does this namespace belong to you? The PR passed checklist review. Please review for technical accuracy, and approve/merge if appropriate. Thanks!


Fixes PLAT-11951

Added opts parameter to several methods to allow one-time activation
configuration (e.g. disabling animation for hiding a floating panel).
Enyo-DCO-1.1-Signed-off-by: Ryan Duffy<EMAIL_ADDRESS>

It looks find to us. Thanks.


(PE-33893) Improve testing script and contributing documentation

Very helpful to understand the underlying complexity.  Thanks for doing this.
LGTM!


Fix for ErrorBoundary extra prop type definition

Description of the change
This PR fixes the type definition of the extra prop from the ErrorBoundary component.
When extra prop is a function, the correct signature should include (error, info) as an argument as described here.

These props take either a value or a function that will be invoked with the error and info from the Error Boundaries API's componentDidCatch method (i.e. signature is (error, info)).

Type of change

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Maintenance
[ ] New release

Checklists
Development

[x] Lint rules pass locally
[x] The code changed/added as part of this pull request has been covered with tests
[x] All tests related to the changed code pass in development

Code review

[ ] This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers assigned
[ ] Issue from task tracker has a link to this pull request
[ ] Changes have been reviewed by at least one other engineer

Released in v0.11.1.