Incorrect parsing of version into url hostname

Saw a weird error about cannot reach the URL for a particular repo. Digging further, I found the following in the log:
updater | INFO <job_254312325> Updating github.com/company/redactedrepo from 0.49.0 to 0.50.0
  proxy | 2022/01/11 09:37:01 [094] GET https://github.com:443/company/redactedrepo/info/refs?service=git-upload-pack
  proxy | 2022/01/11 09:37:01 [094] * authenticating git server request (host: github.com)
  proxy | 2022/01/11 09:37:01 [095] GET https://v0.50.0:443/?go-get=1
  proxy | 2022/01/11 09:37:01 [095] WARN: Cannot read TLS response from mitm'd server dial tcp: lookup v0.50.0 on <IP_ADDRESS>:53: no such host
  proxy | 2022/01/11 09:37:01 [097] GET https://v0.50.0:443/?go-get=1
  proxy | 2022/01/11 09:37:01 [097] WARN: Cannot read TLS response from mitm'd server dial tcp: lookup v0.50.0 on <IP_ADDRESS>:53: no such host
  proxy | 2022/01/11 09:37:01 [094] 200 https://github.com:443/company/redactedrepo/info/refs?service=git-upload-pack
updater | INFO <job_254312325> Handled error whilst updating github.com/company/redactedrepo: git_dependencies_not_reachable {:"dependency-urls"=>["github.com/company/redactedrepo v0.50.0\ngo get"]}

Looks to me like the version number was extracted and then for some reason it was used as the hostname?
I took a quick look at git history of the go modules code, and nothing changed recently. Maybe something changed in a lower layer of general URL parsing within dependabot?

That is pretty strange. Those requests should all be coming from go get so I think either the version was treated as a separate arg (go get -d github.com/company/redactedrepo v0.50.0 instead of go get -d<EMAIL_ADDRESS>or perhaps something was parsed correctly from the go.mod.
When did you first see this issue for this repo? We have been rolling out some changes to the GOPRIVATE setting (https://github.com/dependabot/dependabot-core/pull/4568) and could try disabling them for this repo in the off-chance it's causing the issue.

When did you first see this issue for this repo?

I first noticed them this morning, but I haven't been watching the dependabot runs closely, as most of our repos are suffering from https://github.com/dependabot/dependabot-core/issues/4536.
Here's another Job ID with similar failure but a different failing dep: 254513777
Agreed that it is weird. If I get time, I'll try to see if I can repro this locally, but that may not happen for a bit.

I see this same error has occurred back in Dec 16, 2021 so that would rule out the GOPRIVATE setting.

I dug into this a little more...
https://github.com/dependabot/dependabot-core/blob/c4f3e6eae592c5c42332719b9e597c8bd47a2db9/go_modules/lib/dependabot/go_modules/file_updater/go_mod_updater.rb#L174 is returning the error string:
"go: downloading github.com/<redacted>/<private-repo-that-we-are-directly-running-dependabot-against> v0.39.0\ngo get: reading google.golang.org/grpc/go.mod at revision v1.33.0: unknown revision v1.33.0\n"

That is correctly caught as a repo resolvability error and ends up here: https://github.com/dependabot/dependabot-core/blob/c4f3e6eae592c5c42332719b9e597c8bd47a2db9/go_modules/lib/dependabot/go_modules/file_updater/go_mod_updater.rb#L243-L246
But then things go sideways, because this line grabs the wrong repo URL:
https://github.com/dependabot/dependabot-core/blob/c4f3e6eae592c5c42332719b9e597c8bd47a2db9/go_modules/lib/dependabot/go_modules/resolvability_errors.rb#L9
Look again at the error message above... it's giving the name of the repo that we are running dependabot against, which is located on GitHub. But then the subdependency which is actually causing the error is not on github. The regex is looking for the last url that contains "github", so it grabs the wrong url.
At that point, the code starts working as expected... it throws a GitDependenciesNotReachable which then manifests as this red herring error message about needing access to private registries.
So there's four variants here:

neither url is github -> no match as expected - there's already a test case for this.
both urls are github -> matches the final one as expected, couldn't find a test case
first url is github, second url is not -> unhandled
first url is not github, second url is github -> matches the final one as expected, couldn't find a test case

Unfortunately, while looking at the code, it looks like case 2/4 are not handled properly within the url splitting of https://github.com/dependabot/dependabot-core/blob/c4f3e6eae592c5c42332719b9e597c8bd47a2db9/go_modules/lib/dependabot/go_modules/resolvability_errors.rb#L18-L23
For example, if I modify the original error message so that both URLs point at github:
"go: downloading github.com// v0.39.0\ngo get: reading github.com/grpc/go.mod at revision v1.33.0: unknown revision"
results in a repo_path value of "github.com/grpc/go.mod at revision v1.33.0"... when it should probably be "github.com/grpc"
So there's a few things going on here...

Ah, nice find! The purpose of that code is to raise Dependabot::GitDependenciesNotReachable if the error is a potential access issue for a github.com repo. In that case the error presentation to the user may suggest adding it to the private repo access setting. If the access issue is for a non github hosted module then it's fine to return Dependabot::DependencyFileNotResolvable instead.

I was trying to create a test project for this, and noticed the following little nuance... this error only shows up when the go mod cache is empty:
# first run throws full error
[dependabot-core-dev] ~/dependabot-core/tmp/<redacted-org>/repo1 $ GOPRIVATE=* go get -d<EMAIL_ADDRESS>go: downloading github.com/<redacted-org>/<redacted-repo1> v0.39.0
go get: reading google.golang.org/grpc/go.mod at revision v1.33.0: unknown revision

# second run doesn't include the `go: downloading...` line
[dependabot-core-dev] ~/dependabot-core/tmp/<redacted-org>/repo1 $ GOPRIVATE=* go get -d<EMAIL_ADDRESS>go get: reading google.golang.org/grpc/go.mod at revision v1.33.0: unknown revision v1.33.0

# clean the modcache
[dependabot-core-dev] ~/dependabot-core/tmp/<redacted-org>/repo1 $ go clean -modcache

# the full error shows up again
[dependabot-core-dev] ~/dependabot-core/tmp/<redacted-org>/repo1 $ GOPRIVATE=* go get -d<EMAIL_ADDRESS>go: downloading github.com/<redacted-org>/repo2 v0.39.0
go get: reading google.golang.org/grpc/go.mod at revision v1.33.0: unknown revision

So the fix will probably be to catch / strip that go: downloading ...  line if it matches the repo that is currently being go get'ed... I wonder if other errors might be hiding behind this as well?

Actually, thinking about this more... as a user, it's helpful to know which direct dep is triggering the failing indirect dep.
So while we need to strip out the go: downloading... line in order to know how to handle the error, ideally when surfacing the final error message back to the user it'd be helpful to add it back.
That may not be possible with the current code structure, and it's not a deal breaker if so because they can use other go cmd tooling to dig into it, but it would help.

Fixed by #4910
Defensive to prevent similar bugs: #4911
Defensive to emit the error message sooner in the pipeline if something is amiss: #4913
That should take care of everything I commented on here... My original take that the mod_path parsing needed improvement was incorrect, because I didn't realize the underlying problem was that filter_error_message() was returning a multi-line string when it should have returned a single-line string with only a single URL.


Parameters of each command are not listed completly

I expect to see a complete list of parameters for each command or a link to some page which covers more details.
As an example, Update-Database can get a -Context parameter which is not listed in this document.

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

ID: 22803277-a2a7-0adf-acfc-35ed8e4238c8
Version Independent ID: 19869482-5737-adff-3f7f-60b4217b977d
Content: EF Core tools reference (Package Manager Console) - EF Core
Content Source: entity-framework/core/miscellaneous/cli/powershell.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @bricelam
Microsoft Alias: bricelam

I can see now that I was wrong. You've separated common parameters to some other section.
I'm sorry for the my rush to create an issue.
I'll close this issue.

I think your feedback is valid. Maybe we should link back to the common parameters under each command.


broken network once lwM2M is resumed after pause

As already discussed with @jheiskan81 via mail I want to report a bug that I was observing when trying to use LwM2M pause and resume using non-dtls. When I use dtls pause/resume works fine in ma application.
[00:00:03.676,116] <dbg> net_lwm2m_message_handling: lwm2m_udp_receive: reply 0x2000c980 handled and removed
[00:00:03.676,269] <dbg> net_lwm2m_message_handling: lwm2m_udp_receive: checking for reply from [<IP_ADDRESS>]
[00:00:03.676,330] <dbg> net_lwm2m_message_handling: lwm2m_engine_default_content_format: No accept option given. Assume OMA TLV.
[00:00:03.676,422] <dbg> net_lwm2m_observation: engine_observe_node_init: OBSERVER ADDED 3/0/0/0(2)
[00:00:03.676,483] <dbg> ct: lwm2m_observe_cb: Added observer for 3/0
[00:00:03.676,605] <dbg> net_lwm2m_observation: engine_observe_node_init: token:'0dfbdec49d53d0fb' addr:<IP_ADDRESS>
[00:00:03.677,001] <dbg> net_lwm2m_message_handling: lwm2m_udp_receive: checking for reply from [<IP_ADDRESS>]
[00:00:03.677,062] <dbg> net_lwm2m_message_handling: lwm2m_engine_default_content_format: No accept option given. Assume OMA TLV.
[00:00:03.677,154] <dbg> net_lwm2m_observation: engine_observe_node_init: OBSERVER ADDED 4/0/0/0(2)
[00:00:03.677,215] <dbg> ct: lwm2m_observe_cb: Added observer for 4/0
[00:00:03.677,276] <dbg> net_lwm2m_observation: engine_observe_node_init: token:'1f48a7cf28172cc7' addr:<IP_ADDRESS>
[00:00:08.676,116] <dbg> ct: ciot_suspend_client: Suspending client
[00:00:09.003,265] <inf> net_lwm2m_rd_client: Suspend client
[00:00:09.003,326] <dbg> ct: lwm2m_event_cb: LwM2M engine suspended
[00:00:09.003,326] <inf> app: ************** CIOT_HUB_SUSPENDED
[00:00:09.009,521] <inf> net_lwm2m_engine: LWM2M engine thread paused
[00:00:10.511,383] <inf> app: Modem temperature: 23.0 degrees celsius
[00:00:10.511,444] <dbg> net_lwm2m_registry: lwm2m_engine_set: path:32999/0/27001/0, value:0x2001a3a8, len:8
[00:00:14.003,417] <inf> app: ********** resuming
[00:00:14.003,448] <dbg> ct: ciot_resume_client: Resuming client
[00:00:14.003,540] <inf> net_lwm2m_rd_client: Resume Client state
[00:00:14.013,580] <inf> net_lwm2m_engine: LWM2M engine thread resume
[00:00:14.018,585] <dbg> net_lwm2m_observation: lwm2m_notify_observer_path: NOTIFY EVENT 3/0/13
[00:00:14.018,676] <dbg> net_lwm2m_message_handling: generate_notify_message: [MANUAL] NOTIFY MSG START: 3/0/0(2) token:'0dfbdec49d53d0fb' [<IP_ADDRESS>] 14018
[00:00:14.018,951] <dbg> net_lwm2m_message_handling: generate_notify_message: NOTIFY MSG: SENT
[00:00:14.019,561] <err> net_lwm2m_message_handling: Failed to send packet, err 121
[00:00:14.534,454] <dbg> net_lwm2m_registry: lwm2m_engine_get: path:1/0/1, buf:0x200188f8, buflen:4
[00:00:14.534,545] <dbg> net_lwm2m_rd_client: sm_send_registration: registration sent [<IP_ADDRESS>]
[00:00:14.535,156] <err> net_lwm2m_message_handling: Failed to send packet, err 121
[00:00:19.531,036] <dbg> ct: at_notification_cb: +CSCON: 0

The bug was originally introduced by commit 21e551b6bcd653470c709c7f849ff586cfd2af35. @jheiskan81 asked me to open a ticket. He is going to work on it.

Thanks @chmielewskiandreas for raising this. Just minor update that issue only happen without DTLS.

@chmielewskiandreas I tested with nerf-sdk 2.1.1 release without security and i can't re-produce your issue.

Hi @jheiskan81. I actually can reproduce this issue with lwm2m_client ...
diff --git a/samples/nrf9160/lwm2m_client/prj.conf b/samples/nrf9160/lwm2m_client/prj.conf
index 188f4b896..27b3db7ed 100644
--- a/samples/nrf9160/lwm2m_client/prj.conf
+++ b/samples/nrf9160/lwm2m_client/prj.conf
@@ -115,7 +115,7 @@ CONFIG_DOWNLOAD_CLIENT_HTTP_FRAG_SIZE_1024=y
 CONFIG_FOTA_DOWNLOAD=y

 # Set LwM2M Server IP address here
-CONFIG_LWM2M_CLIENT_UTILS_SERVER="coaps://leshan.eclipseprojects.io:5684"
+CONFIG_LWM2M_CLIENT_UTILS_SERVER="coap://leshan.eclipseprojects.io:5683"

 # Application Event Manager
 CONFIG_APP_EVENT_MANAGER=y
diff --git a/samples/nrf9160/lwm2m_client/src/main.c b/samples/nrf9160/lwm2m_client/src/main.c
index 522d046f2..eb2e5671b 100644
--- a/samples/nrf9160/lwm2m_client/src/main.c
+++ b/samples/nrf9160/lwm2m_client/src/main.c
@@ -266,6 +266,34 @@ static void rd_client_update_lifetime(int srv_obj_inst)
 	update_session_lifetime = false;
 }

+static int counter = 0;
+
+static void suspend(struct k_work *work);
+static void resume(struct k_work *work);
+
+K_WORK_DELAYABLE_DEFINE(suspend_work, suspend);
+K_WORK_DELAYABLE_DEFINE(resume_work, resume);
+
+static void suspend(struct k_work *work)
+{
+    if (counter == 0) {
+        LOG_INF("********** suspending");
+        lwm2m_engine_pause();
+        counter++;
+        k_work_schedule(&resume_work, K_SECONDS(5));
+    }
+}
+
+static void resume(struct k_work *work)
+{
+       if (counter == 1) {
+           LOG_INF("********** resuming");
+           lwm2m_engine_resume();
+           counter++;
+       }
+}
+
+
 static void rd_client_event(struct lwm2m_ctx *client, enum lwm2m_rd_client_event client_event)
 {
 	switch (client_event) {
@@ -300,7 +328,7 @@ static void rd_client_event(struct lwm2m_ctx *client, enum lwm2m_rd_client_event
 			/* Read a current server  lifetime value */
 			rd_client_update_lifetime(client->srv_obj_inst);
 		}
-
+        k_work_schedule(&suspend_work, K_SECONDS(20));
 #if defined(CONFIG_APP_LWM2M_CONFORMANCE_TESTING)
 		lwm2m_register_server_send_mute_cb();
 #endif

causes this
[00:00:04.383,392] <inf> lwm2m_security: Need to write credentials, requesting LTE and GNSS offline...
[00:00:04.453,369] <inf> lwm2m_neighbour_cell: RRC connected, measure when idle
[00:00:07.698,364] <inf> lwm2m_security: Requesting LTE and GNSS online
[00:00:10.007,232] <inf> lwm2m_security: Modem connection restored
[00:00:10.007,873] <inf> net_lwm2m_engine: Connected, sock id 0
[00:00:10.120,880] <dbg> app_lwm2m_client: rd_client_event: Registration complete
[00:00:10.121,032] <inf> app_lwm2m_client: Obtained date-time from modem
[00:00:10.123,596] <dbg> gnss_module: start_gnss: Started GNSS scan
[00:00:10.123,626] <inf> net_lwm2m_rd_client: Registration Done (EP='9QWjIXCaof')
[00:00:10.123,779] <dbg> gnss_module: gnss_event_handler: A-GPS data request not supported.
[00:00:30.121,002] <inf> app_lwm2m_client: ********** suspending
[00:00:30.521,362] <inf> net_lwm2m_rd_client: Suspend client
[00:00:30.521,392] <dbg> app_lwm2m_client: rd_client_event: LwM2M engine suspended
[00:00:30.525,085] <inf> net_lwm2m_engine: LWM2M engine thread paused
[00:00:35.525,177] <inf> app_lwm2m_client: ********** resuming
[00:00:35.525,238] <inf> net_lwm2m_rd_client: Resume Client state
[00:00:35.535,308] <inf> net_lwm2m_engine: LWM2M engine thread resume
[00:00:36.027,374] <err> net_lwm2m_message_handling: Failed to send packet, err 121
[00:00:40.475,189] <err> net_lwm2m_message_handling: Failed to send packet, err 121
[00:00:49.371,093] <err> net_lwm2m_message_handling: Failed to send packet, err 121
[00:01:07.163,085] <err> net_lwm2m_message_handling: Failed to send packet, err 121
[00:01:42.747,070] <err> net_lwm2m_message_handling: Failed to send packet, err 121
[00:02:53.915,039] <wrn> net_lwm2m_rd_client: Registration Update Timeout
[00:02:53.915,069] <dbg> app_lwm2m_client: rd_client_event: Registration update failure!
[00:02:53.915,344] <dbg> app_lwm2m_client: rd_client_event: Disconnected
[00:02:53.915,405] <inf> net_lwm2m_rd_client: Stop LWM2M Client: urn:imei:352656106103065
[00:02:54.165,527] <inf> net_lwm2m_rd_client: Start LWM2M Client: urn:imei:352656106103065
[00:02:55.077,270] <inf> net_lwm2m_rd_client: RD Client started with endpoint 'urn:imei:352656106103065' wit
h client lifetime 30

Thanks. I can now re-produce issue. I was using automatic lte registration status for triggering pause and resume.

@chmielewskiandreas
I spot a problem. Socket start call is not happen if DTLS is disabled. Comment those lines like me below It fix problem.
Try modify Function lwm2m_engine_connection_resume:
int lwm2m_engine_connection_resume(struct lwm2m_ctx *client_ctx)
{
	int ret;

	if (client_ctx->connection_suspended) {
		lwm2m_close_socket(client_ctx);
		client_ctx->connection_suspended = false;
		ret = lwm2m_open_socket(client_ctx);
		if (ret) {
			return ret;
		}
/*  Changes  */
		// if (!client_ctx->use_dtls) {
		// 	return 0;
		// }
/* end of changes*/
		LOG_DBG("Resume suspended connection");
		return lwm2m_socket_start(client_ctx);
	}

	return 0;
}

Bug fix PR: https://github.com/zephyrproject-rtos/zephyr/pull/51646

It works :) Thx - I actually had this fix already before writing to you but did not know whether it is correct. It is possible for you to implement that on lwm2m_rd_client_resume a proper event LWM2M_RD_CLIENT_EVENT_ENGINE_RESUMED) is raised as well? I also can implement this if you like ...

I prioritize bug fix. If you see value for LWM2M_RD_CLIENT_EVENT_ENGINE_RESUMED you can add it.


Thievery Pack Changes 2

Bug Fixes

Null Hammer can no longer steal Back Attack power, preventing a softlock
Fix Witchcraft not giving you a potion when you finish combat with it

Balance Changes

Null Hammer can no longer steal the enemy's Strength and make them go below 0

This also fixes an inconsistency of whether you can steal the enemy's Strength or not. Previously, you could always steal the enemy's Strength, except when that enemy gains strength to 0 (causing the power icon to disappear). Now you can only steal when the enemy's strength goes over 0, and only steal positive portion of it.

Fair Trade damage: 5(7) -> 6(8)
Magnet cost: 1 -> 0
Strength Sap stealing amount: 1(2) -> 1(1)
Strength Sap now has Exhaustive 2(3)

Wording

Changed the wording of Smokescreen(EVERYONE) and Thievery Mastery(Master Thief)

Code

Used Reformat Code to remove unnecessary imports and fix formatting

Fix issue where using Mind Control on Awakened One softlock the game


Switch for silent upload

hello,
is there a switch to prevent output?
i want do get rid the verbose upload output without using 2/dev/null.
which will also drop any error messages!
thanks in advanced,
mark
ps. if not it will be a feature request ;-)

There isn't a switch to do that, but it'd be fairly easy to add. This is probably a good excuse to generally revisit all of the output code, clean up the overall approach. (Will try to do this soon.)

Ok, you can now do things like skicka -quiet upload /foo /bar and see no output (unless there's an error along the way).  Let me know if I missed anything.

i will start to test and let you know.
thanks for the fast implementation.
mark

thanks looks good.
mark


B2B Sync Mac

This works great for me on Windows but on either of my Mac's defaults write com.microsoft.OneDrive Tier "Insiders" doesn't seem to allow it to work. I keep getting the same error as before. Any ideas?

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

ID: 69433cdb-79fb-e59d-e208-0c1201b74bfa
Version Independent ID: e69001d7-2292-9f1a-e3d8-cf0afb9b18df
Content: B2B Sync
Content Source: OneDrive/b2b-sync.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@bovie2k  - thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

@officedocsbot close


Increase the size of draggable image

Hi There,
I'm loving your library so far! It does almost everything I want it to, one last thing I cannot figure out how to implement is increasing the size of the draggable item.
Here is my current implementation:

And here is what I am trying to achieve:

Is there a way to achieve what I want with your library? I have spent the last 3 days reading through the code, but I can't seems to be able to figure it out.

Hi. That feature is not present for now, so I will implement it in next release. Thanks for a good suggestion!

@sprejjs Hi. I added a new feature to configure the appearance of the dragging item. Could you try the latest develop branch code via JitPack. Thanks!
Usage: (Scale 1.5x dragging item with animation)
mRecyclerViewDragDropManager.setDragStartItemAnimationDuration(250);
mRecyclerViewDragDropManager.setDraggingItemScale(1.5f);

// this new ItemAnimator is required to make item drop animation work properly
mRecyclerView.setItemAnimator(new DraggableItemAnimator());

Gradle:
repositories {
    maven { url "https://jitpack.io" }
}

dependencies {
    compile 'com.github.h6ah4i:android-advancedrecyclerview:c253ef247079973b1e42549c94e77f8bd831fd3f'
}

All newly added methods:

void setDraggingItemScale(float scale)
float getDraggingItemScale()
void setDraggingItemRotation(float rotation)
float getDraggingItemRotation()
void setDraggingItemAlpha(float alpha)
float getDraggingItemAlpha()
void setDragStartItemAnimationDuration(int duration)
int getDragStartItemAnimationDuration()
void setDragStartItemScaleAnimationInterpolator(Interpolator interpolator)
void setDragStartItemRotationAnimationInterpolator(Interpolator interpolator)
void setDragStartItemAlphaAnimationInterpolator(Interpolator interpolator)

This is fantastic. Thank you very much!


Added null checks for issue #4618

This pull request adds null checks for missing configuration elements such as network.protocols, network.listeners and network.sockets to allow a simplistic configuration mentioned in issue #4618.

Makes sense. Thanks.

Merging it manually.


net/http: clarify that (*Request).WithContext affects the Response too

What version of Go are you using (go version)?
https://golang.org/pkg/net/http/#Request.Context
Does this issue reproduce with the latest release?
Same docs on https://tip.golang.org/pkg/net/http/#Request.Context
What did you do?
I read:

For outgoing client requests, the context controls cancelation.

I thought the context only affected its associated Request, and that the Response returned from Client.Do(Request) would be unaffected by the context.
I searched the net/http Godoc for any mentions of Context relating to a Response but didn't find anything.
Finally, after a long discussion with coworkers, we came up with a manual test case demonstrating that the request context does in fact affect the response:
package main

import (
	"context"
	"io"
	"io/ioutil"
	"log"
	"net/http"
	"net/http/httptest"
	"time"
)

func main() {
	s := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		log.Println("serving", r.URL)
		io.WriteString(w, "Hold on...")
		if f, ok := w.(http.Flusher); ok {
			f.Flush()
		} else {
			log.Println("cannot flush!")
		}
		time.Sleep(10 * time.Second)
		io.WriteString(w, "... okay")
	}))
	defer s.Close()

	req, err := http.NewRequest("GET", s.URL, nil)
	if err != nil {
		log.Fatal("Error making request:", err)
	}

	ctx, cancel := context.WithTimeout(context.Background(), time.Second)
	defer cancel()

	req = req.WithContext(ctx)

	resp, err := http.DefaultClient.Do(req)
	if err != nil {
		panic(err)
	}
	defer resp.Body.Close()

	log.Println("client made request, starting to read response (this should time out in 1s)")
	body, err := ioutil.ReadAll(resp.Body)
	if err != nil {
		log.Fatal("Error reading response body:", err)
	}
	log.Println("client read response:", string(body))
}

// Output like:
// 2018/06/27 17:07:06 serving /
// 2018/06/27 17:07:06 client made request, starting to read response (this should time out in 1s)
// 2018/06/27 17:07:07 Error reading response body:context deadline exceeded

We would have saved a lot of time today if the doc said something more like "For outgoing client requests and their corresponding responses, the context controls cancelation."

It might be worthwhile to be more explicit on the details before deciding on the best documentation to balance conciseness and accuracy.
Expect: The request's context has dominion for the lifetime of http.DefaultClient.Do(req). As soon as Do returns/the initial headers are received, the context no longer applies.
Actual: If the response to a request has a body, the request's context has dominion until the response body is closed. (Notable exclusion being HEADs.)
Is this correct? Would you like to upload your suggestion?
Somewhat related: https://github.com/golang/go/issues/23262

Is your Actual note how it actually behaves?
I'm not sure of the precise actual behavior, and I'm also unsure of the correct language to use to be brief and accurate regarding the relationship between a context, request, and response; so I'm going to leave it to the Go authors or a more-ambitious-than-me contributor to take on a CL for this doc update.

Actual: If the response to a request has a body, the request's context has dominion until the response body is closed. (Notable exclusion being HEADs.)

That sounds fine to me to be the new documentation. Re the exclusions, the exclusion is any time there is no body correct?

Sent https://go-review.googlesource.com/c/go/+/129155


Have you completed your first issue?

[X] I have completed my first issue

Guidelines

[X] I have read the guidelines
[X] I have the link to my latest merged PR

Latest Merged PR Link
https://github.com/UTSAVS26/PyVerse/pull/368
Project Description
I want to contribute to the tutorial section of the repository by adding code examples and explanations for fundamental data structures such as lists, sets, tuples, and dictionaries, along with tutorials on methods, file handling, and more. This will help enhance the repository with valuable learning resources for new users.
Full Name
Ankit Ojha
Participant Role
Contributor GSSOC - EXT

@ojhankit work on 1 issue at a time, you can reopen it after completing the previous one and tag me to assign you.


Expose CredentialProvider

Which issue does this PR close?

Closes #4163
Rationale for this change

This exposes the CredentialProvider abstraction added in #4225. This allows users to provide custom mechanisms for sourcing credentials (#4163) and opening the door to exposing the authorisation logic (#4223)
What changes are included in this PR?

Are there any user-facing changes?

cc @roeap


[Suggestion] Add blacklist config

I know you intend to add configuration to the mod, but it'd be a good idea to add a blacklist to the config. You could add specific structures or perhaps even blacklist entire mods from being affected.

Thank you for your suggestion! This is a good idea, I'll definitely try to implement that in the future. I'm still very new to Minecraft modding, and I've never done config before, so the update won't come super quickly.
In the meantime, if you really need to blacklist a structure, you can achieve that by finding the config file for your structure and divide by two it's separation and spacing... It's kinda hacky, but it works. I hope you won't have to wait too long for this feature to come out!

Coming to this suggestion again, I think I found a good way to implement this, UX side: add structure-specific spread factor, if you want to blacklist a structure from this mod, simply use a a factor of 1, or if you want to make it more common, a factor of 1/2 for example (or even 3 if you want to make it even rarer). I might start working on it today or tomorrow.
What do you think @FusionSwarly?

Hello! Are you still working at your mode? Will it be ported to newer versions?

Coming to this suggestion again, I think I found a good way to implement this, UX side: add structure-specific spread factor, if you want to blacklist a structure from this mod, simply use a a factor of 1, or if you want to make it more common, a factor of 1/2 for example (or even 3 if you want to make it even rarer). I might start working on it today or tomorrow. What do you think @FusionSwarly?

That's a much better idea! That'd be super useful.

Hello! Are you still working at your mode? Will it be ported to newer versions?

@Ignat34 Unfortunately, I have loads of work to do for school, but I'll start working on it next week and during the vacations! I'll try to port it to 1.20.2 (it should already be compatible right now, you could try). I'll also work on adding the blacklist.

That's a much better idea! That'd be super useful.

@FusionSwarly Thx! I'll try adding it next week / in two weeks.

Updated to 1.20.2 and configurable.


Where is the DateIOType exported from ?

Hello guys,
First I wanna thank for the amazing job that you are doing.
I am working on a library that uses material-ui-pickers and I am using typescript to build it.
I noticed that the type of the onChange function in the docs is this:
(date: DateIOType) => void

I just want to know where can I import the DateIOType from ??
Thank you so much

There is a hint on documentation about it

But in real world it is declaring right inside used @date-io/{lib] module

@date-io/moment Doesn't Export DateIOType
Where To Find It Exactly ?

You should not use it. If you are using moment you need useState<Moment>


Add inspect and to_s to EksekResult

What would it provide? stdout + stdin? The command? ALL of it? 🤔

to_s should probably be something very simple e.g. either the command or the output (maybe output is easiest because--unlike the command--it cannot be an array-or-string.
Inspect should be a much more verbose version that is helpful when debugging (it is used with the p method, for example).
puts "Hello" # Hello
p "Hello" # "Hello"
puts 3 # 3
p 3 # 3
puts [1, 2, 3, 4] #1<newline>2<newline>3<newline>4
p [1, 2, 3, 4] # [1, 2, 3, 4]

The default implementation of inspect returns a string like:
#<EksekResult:0x00007fed131aafd8 @env={}, @cmd="ls", @exit_code=0, @success=true, @stdout="Gemfile\nGemfile.lock\nREADME.md\nlib\np.rb\nspec", @stderr="">

Maybe we should leave it?

@tacconthommy I updated the description of this issue to suggest a format for EksekResult#to_s.


Getting 2 error alerts while trying to change description ("uknown error" & "GitHub returned error 401")

Creating Gists works great, opening and updating them works great.
When I try to change the description of a gist, I get these 2 alerts; first of all the "unknown error (please, report a bug!)"  and underneath it, a second alert which says I have bad credentials.
This strikes me as weird, because my credentials are obviously fine if I'm able to do all other operations.

Thank you for your contribution!


Django Taggit cant save through admin

I am having a small issue with django admin and django taggit, when i try to save tags through django admin save_model function in my ModelAdmin class it saves the tagged element and then removes it or overwrites the element later on. There is no error with the code it just overwrites the entry later on with the value for tags in forms.
I want to be able to add tags through the ModelAdmin class as i am fetching the keywords through the meta content of the link present in the model Content_Master. The get_metadata function is returning me meta keywords from the url given. Any help on this would be a big help. Thanks !
Models.py
class Content_Master(models.Model):
...
content_link = models.URLField(default=None, null=True, blank=True,unique= True)
tags = TaggableManager(blank=True)
Admin.py
def save_model(self, request, obj, form, change):
keyword = get_metadata(identifier = obj.content_link)
obj.save()
artic = Content_Master.objects.filter(content_title = title,published_date = time).values('content_id')
art1 = artic[0]
art = art1.values()[0]
co = ContentType.objects.get(model="content_master")
if keyword is not None:
tagg = keyword.split(',')
#print tagg
for taggs in tagg:
tag, created = Tag.objects.get_or_create(name = taggs)
tagged = TaggedItem(object_id=art, tag=tag,content_type=co)
#print tagged
tagged.save()

this is built-in functionality in the save method I believe, not sure this is really incorrect behaviour.


Suggestion: register.js

Apologies if this has been discussed before...
If I'm writing a library that uses isomorphic-fetch internally, I want a way to import it for local use inside my library only, without global side effects that might confuse my users.
I like the convention of using import 'foo/register' for global side-effects (installing/polyfilling/whatever), while import foo from 'foo' would just gets you a local import without any global side effects. This gives people the choice not to create a global. It also makes it more explicit (I was surprised to learn that import fetch from 'isomorphic-fetch would create a global and export me a local variable, as it means the fetch from bit is redundant).
Would you be up for updating isomorphic-fetch to work like this? (Would need a major version bump, obviously.)
loud-rejection and coffeescript use this convention, and I think there are a few others.

Don't use this in a library.
If you are writing a library for browsers then leave the polyfilling to the user, just like this library leaves the polyfilling of Promise to you.
If you are writing a Node.js library then use node-fetch since the side effect is the whole point of this library.

Yep I didn't know this lib just exports node-fetch, thanks


Add state output and input control to realization

Warm states restart are essential
In many applications of hydrologic models, there's a need to restart the model at a specific time. In order to be able to do that the model should provide the state variables at every time step as output and at the same time it must be able to accept the state variables as inputs. Otherwise, when you restart the model, all the state variables will be assumed to have their default values (usually zeros) which will significantly affect the calculated runoff.
To provide an example, assume you run the model from 1979 to 12/31/2016. Now if you want to get the streamflow for the year 2017, one option is to run the model all over again from 1979 till 12/31/2017. The other option is to simply run it from where you stopped (12/31/2016) but using the state variables calculated at the end of 12/31/2016. Otherwise, the result for the year 2017 will be significantly different for these two scenarios.
Noah-MP and WRF-hydro models do this by providing a 2nd output file called the RESTART file (one file for each timestep). You can also provide this RESTART file as input by providing the path to it in the "name list.hrldas" file in WRF-hydro.
The same thing is needed for NextGen to be able to perform sequential procedures such as data assimilation.
Building from advice issued in (https://github.com/NOAA-OWP/cfe/issues/73), we started to try to address this in (https://github.com/NOAA-OWP/cfe/pull/86) and we will be happy to give help if we can.

This is definitely something on our radar, as far as supporting. Technically, much of what is needed in ngen is already there--the model_params feature (this needs better documentation) allows for the passing of variables via BMI methods at the beginning of the simulation (after Initialize but not at every timestep), and this could be used to pass state variables into any BMI module that supports it. Of course, for this to be useful, the module has to support the setup of state variables using this mechanism--and that's a per-module need.
What is more directly on our roadmap is some way of detecting the proper variable names to retrieve and set--and then a mechanism that can retrieve the state (e.g. at the end of a run, in your scenario). This may happen in various ways--@peckhams has proposed a feature for a future version of BMI that will have more categories of variables than just "input" and "output" (to include state variables) directly in the spec, and we've also considered the use of MCM files extended with an additional namespace to capture state (and calibration parameters) variable names for this purpose.
In summary, yes, essential, and yes we plan to add it. No ETA yet though, that's for sure.


headless_defaults example panics

Bevy version
main
bisected to #3884
Operating system & version
MacOS 12.2.1 (M1)
What you did
cargo run --example headless_defaults
What you expected to happen
I think that this example is meant to display a blank window? At least that's what it did prior to #3884
What actually happened
thread 'main' panicked at 'Requested resource bevy_render::renderer::render_device::RenderDevice does not exist in the `World`.
                Did you forget to add it using `app.add_resource` / `app.init_resource`?
                Resources are also implicitly added via `app.add_event`,
                and can be added by plugins.', crates/bevy_render/src/texture/image_texture_loader.rs:74:23
stack backtrace:
   0: rust_begin_unwind
             at /rustc/9d1b2106e23b1abd32fce1f17267604a5102f57a/library/std/src/panicking.rs:498:5
   1: core::panicking::panic_fmt
             at /rustc/9d1b2106e23b1abd32fce1f17267604a5102f57a/library/core/src/panicking.rs:116:14
   2: bevy_ecs::world::World::resource
             at ./crates/bevy_ecs/src/world/mod.rs:748:21
   3: <bevy_render::texture::image_texture_loader::ImageTextureLoader as bevy_ecs::world::FromWorld>::from_world
             at ./crates/bevy_render/src/texture/image_texture_loader.rs:74:17
   4: <bevy_app::app::App as bevy_asset::assets::AddAsset>::init_asset_loader
             at ./crates/bevy_asset/src/assets.rs:325:22
   5: <bevy_render::texture::ImagePlugin as bevy_app::plugin::Plugin>::build
             at ./crates/bevy_render/src/texture/mod.rs:50:13
   6: bevy_app::app::App::add_plugin
             at ./crates/bevy_app/src/app.rs:764:9
   7: <bevy_render::RenderPlugin as bevy_app::plugin::Plugin>::build
             at ./crates/bevy_render/src/lib.rs:282:9
   8: bevy_app::plugin_group::PluginGroupBuilder::finish
             at ./crates/bevy_app/src/plugin_group.rs:118:21
   9: bevy_app::app::App::add_plugins
             at ./crates/bevy_app/src/app.rs:791:9
  10: headless_defaults::main
             at ./examples/app/headless_defaults.rs:4:5
  11: core::ops::function::FnOnce::call_once
             at /rustc/9d1b2106e23b1abd32fce1f17267604a5102f57a/library/core/src/ops/function.rs:227:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Pinging @Shatur and @kirusfg who previously worked on headless-related stuff and @superdump .
It looks like there's now some code in the image loaders that use the render device to determine which compressed textures are supported by the GPU.
Does it make sense to use a theoretical CompressedImageFormats::all() if no RenderDevice is present and things aren't going to be rendered anyway?
Is there a simple example of compressed texture usage somewhere that I could test with?

An example could be added but it would have to have feature dependencies and would probably have to either load a format that would only work on some GPUs or depend on basis-universal that currently will not work on wasm.
I don’t think we can just claim support for all. I think if there is no render device we would have to claim support for none as there is no GPU to do any compression. But does headless have to be GPU-less? Isn’t it just windowless?

Probably a simple scene with quads with textures with text labels as to the formats of the textures and all texture formats (well, not all variants of ASTC or anything, just JPEG/PNG/ASTC/ETC2/BCN/BasisU.) That would be a good test to have. I could put that together.

Does it make sense to use a theoretical CompressedImageFormats::all() if no RenderDevice is present and things aren't going to be rendered anyway?

I agree with this. This is how it done in Godot.

I don’t think we can just claim support for all. I think if there is no render device we would have to claim support for none as there is no GPU to do any compression.

Then users have to handle all logic that contains images in a special way which is unusual.

But does headless have to be GPU-less?

Yes. The main purpose of it is to run it on server.

So headless is GPU-less but for some reason, say software rendering, you may still want to load textures on a GPU-less server? They will not currently be able to decompress any of the compressed formats without further extension. Do they not need to for some reason?

So headless is GPU-less but for some reason, say software rendering, you may still want to load textures on a GPU-less server?

The use case is to execute systems that have images, but don't do anything with them. I assumed that CompressedImageFormats::all() is required. If not, then let's just remove the panic :)

Also, GPUs provide the support for decompression of the textures, built into the hardware. There are software decompressors. But what I would advise to do really depends on the use case. When shipping games, you would either ship a universal format (UASTC or the Basis format) and transcoder (to convert to ASTC/BCN/ETC2) or you would ship those compressed formats directly. ASTC is supported on most mobile, ETC2 on a bit older mobile, and BCN on desktop GPUs. Apple M1 is an exception and supports all of them. Intel iGPUs have supported ASTC I think. So compressed texture format wise, it’s just a complicated and fragmented world in terms of support.
What are the use cases?

So headless is GPU-less but for some reason, say software rendering, you may still want to load textures on a GPU-less server?

The use case is to execute systems that have images, but don't do anything with them. I assumed that CompressedImageFormats::all() is required. If not, then let's just remove the panic :)

Ok. I guess we could just load anything.


Unify namespaces

Most of configuration has namespace defined as monitoring but some of samples use default.
Should this project define a recommendation for namespace? My preference is to unify all the docs over monitoring namespace.

The one and only reason why we used the default namespace anywhere is because it doesn't need to be created, so it's one step less to try out things.


custom aggregates wrong number of arguments

Hi there! Great repo. Thanks for supporting custom aggregates (not many sqlite wrapper implementations do).
However, when I tested the example given by the documentation here, running exactly
db.register(function* addAll() {
  var total = 0;
  yield function (rowValue) {total += rowValue;};
  return total;
});

with var totalTreasure = db.prepare('SELECT addAll(<col>) FROM <table>').pluck().get();
I got wrong number of arguments. Just wondering if this still works for you. Thanks.
I tried setting varargs to true as well, but it just errors later with Custom function returned an invalid value.
Thanks!

I found that it was an issue with Typescript --- apparently the method generated by Typescript for generators somehow doesn't fit the structure expected.

Ahh okay interesting. A crude way of getting around this would be to use the varargs: true option, as described in the docs. But the better solution would be to open a ticket or PR with typescript.

So I actually tried the varargs but then it complains down the line about invalid values (I think? don't remember the exact error message).
I have no idea what might be causing the issue; might be hard to open a reasonable PR. For the moment I just reverted to JS (just to use better-slqlite3!), since the codebase is small.


Copy CoreDataEditor to Applications on every run

Thanks! May I ask why you want Core Data Editor to be copied to /Applications?

Hi, It is very handy to run Core-Data-Editor directly from the Applications rather than opening the project and run it. I thought it will help people too.

I think it would be even easier for people to just download Core Data Editor as an App. Well let me checkout your PR real quick... Btw: I have just merge a big PR to develop... it requires macOS 10.12 though...

I was searching for the link to download Core Data Editor App but couldn't find it.

https://github.com/ChristianKienle/Core-Data-Editor/releases/tag/5.2

Thanks 👍🏻. It would be great to see a brew cask of Core Data Editor App sometime.


AttributeError: 'NoneType' object has no attribute 'split'

❓ Questions and Help
I have installed the repo following the instruction on INSTALL.md. When I was following this instruction for installing apex, I used "python setup.py install" instead of "python setup.py install --cuda_ext --cpp_ext", as suggested in this issue. When I try to run the demo, I get the following error
(maskrcnn_benchmark) ParisdeMacBook-Air:demo paris$ python webcam.py --min-image-size 300 MODEL.DEVICE cpu
Traceback (most recent call last):
File "webcam.py", line 6, in
from predictor import COCODemo
File "/Users/paris/maskrcnn-benchmark/demo/predictor.py", line 6, in
from maskrcnn_benchmark.modeling.detector import build_detection_model
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/modeling/detector/init.py", line 2, in
from .detectors import build_detection_model
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/modeling/detector/detectors.py", line 2, in
from .generalized_rcnn import GeneralizedRCNN
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/modeling/detector/generalized_rcnn.py", line 11, in
from ..backbone import build_backbone
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/modeling/backbone/init.py", line 2, in
from .backbone import build_backbone
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/modeling/backbone/backbone.py", line 7, in
from maskrcnn_benchmark.modeling.make_layers import conv_with_kaiming_uniform
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/modeling/make_layers.py", line 10, in
from maskrcnn_benchmark.layers import Conv2d
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/layers/init.py", line 10, in
from .nms import nms
File "/Users/paris/maskrcnn-benchmark/maskrcnn_benchmark/layers/nms.py", line 5, in
from apex import amp
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 668, in _load_unlocked
File "", line 638, in _load_backward_compatible
File "/Users/paris/anaconda3/envs/maskrcnn_benchmark/lib/python3.7/site-packages/apex-0.1-py3.7.egg/apex/init.py", line 2, in
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 668, in _load_unlocked
File "", line 638, in _load_backward_compatible
File "/Users/paris/anaconda3/envs/maskrcnn_benchmark/lib/python3.7/site-packages/apex-0.1-py3.7.egg/apex/amp/init.py", line 1, in
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 668, in _load_unlocked
File "", line 638, in _load_backward_compatible
File "/Users/paris/anaconda3/envs/maskrcnn_benchmark/lib/python3.7/site-packages/apex-0.1-py3.7.egg/apex/amp/amp.py", line 3, in
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 668, in _load_unlocked
File "", line 638, in _load_backward_compatible
File "/Users/paris/anaconda3/envs/maskrcnn_benchmark/lib/python3.7/site-packages/apex-0.1-py3.7.egg/apex/amp/lists/torch_overrides.py", line 69, in
File "/Users/paris/anaconda3/envs/maskrcnn_benchmark/lib/python3.7/site-packages/apex-0.1-py3.7.egg/apex/amp/utils.py", line 9, in get_cuda_version
AttributeError: 'NoneType' object has no attribute 'split'
Can anyone help me figure this out?

I have the same problem as you. But then I realize that I download CPU version of pytorch-nightly, so it can't find my GPU.
Then I download GPU version of pytorch-nightly again at this website
https://anaconda.org/pytorch
Because the pip way is too slow for me, so I download .tar.bz2. Notice NOT to choose CPU!
And now it works. I hope this will be helpful!

I also faced this problem too,and I find the apex/amp/lists/torch._overrides.py and its 69 row is to get a GPU version ,I  # it and this problem has been solved. Hope helpful for you.

Hi.
in case of inference on CPU on machine, getting below error :
from predictor import COCODemo
error:
AttributeError: 'NoneType' object has no attribute 'split'

comment:
if version_num < 1.1:
# FP32_FUNCS.append('mean')

if utils.get_cuda_version() >= (9, 1, 0):
# FP16_FUNCS.extend(_bmms)

else:
# FP32_FUNCS.extend(_bmms)

I also faced this problem too,and I # (.apex/amp/lists/torch._overrides.py) 77-80 rows; and # (.apex/amp/utils.py) 8-9 rows. Then the problem was sloved, hope it's helpful for you.


Lock/Unlock groups as a single toggle instead of two separate binds

Would be better to have just one bind for the lockgroups dispatcher, similar to how togglefloating works, with a boolean property "locked" or "grouplocked" to be associated with the grouped windows for showing up on, hyprctl/IPC similar to the floating, pinned etc. properties

done in 489ef7c51c7d33eeba6d6a66b6697f52441ddf45, use toggle in lockgroups


Fix LLM cold start deeplinks to platform apps & deeplinks to platform apps for private & experimental apps

📝 Description
Issue 1: deeplinking to private & experimental apps
Deeplinking to ledgerlive://discover/lmarket-dev was not working as it's a "private" app.
For deeplinking to platform apps, private: true apps were filtered out by default & branch: "experimental" were only accessible if experimental apps were enabled in the settings.
Now that we have a spec, spec defining that these apps should always be accessible through deeplinking (if they are fo), I added a way to override these default filters for deeplinking.
FYI the spec now is:

platform overrides all others (if mobile, only works on mobile, if desktop, only works on desktop)
Links work in any case if on the correct platform
private = true does not display in the catalog whatever the settings are - but links fork
branch=experimental : only displays if the experimental apps are activated, but links work in any case

I tested on both platforms (iOS & Android), both for private:true apps and branch:"experimental" apps. It works.
Issue 2: deeplinking to platform apps with cold app start
Deep links to platform apps triggering a cold start of the app were not working as the list of manifests hasn't loaded yet.
The fix for that is to detect whether the list of manifests has had time to load once, and then if that's not the case, do an optimistic redirection to the PlatformApp screen (which will then dynamically load the manifest and if there is none, display an error message).
❓ Context

Impacted projects: llm
Linked resource(s): LIVE-2916

✅ Checklist

[ ] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo
Here's a demo with Market which is a private: true app.
https://user-images.githubusercontent.com/91890529/177744058-17862eea-ceb8-4d7d-9772-21647ce6b52b.mp4
https://user-images.githubusercontent.com/91890529/177744074-534ab747-01c5-42ac-ba45-329305f7a3bf.mp4
🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Closed because in the end it has been merged in the context of a hotfix.


Include tax with shipping

Create a new field ShippingVat, which is the shipping cost including tax.

Fixed with f0f55d9bcd730f45bd408fa65dd4d6f963547718


all the scrollbar disappear when close the modal dialog or bootstrap dialog

I call the bootstrap modal dialog and the scrollbar could work well, when I call another modal dialog based on the current opened modal dialog, the scrollbar could work well. But when I close the second modal dialog which removes all the scrollbar containing it's parent modal dialog's.
Does anyone know what's wrong with the modal dialog?
For my understanding, it's caused by close or hide event. But I have no idea how to update it. I think it's a bug of the Bootstrap modal. By the way, I use bootstrap 3.3.4.
I meet the same issue when I call bootstrap confirm  dialog based on a modal dialog, when the confirm dialog close, all the scrollbars have been removed.
The following code show the style I define for the modal dialog.
//The first modal dialog
.two-column-modal .modal-dialog {
width: 750px;
overflow-y: initial !important
}
//The second modal dialog
.modal-dialog-selector {
width: auto;
overflow-y: auto;
}

As stated in our documentation, Bootstrap explicitly does not support having multiple modal dialogs open at the same time. Thanks!


nixos/borgmatic: change the default value

Description of changes
close #329602
source_directories and repositories can't be null.
An error occurred while parsing a configuration file at /etc/borgmatic/config.yaml:
At 'repositories': None is not of type 'array'

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

LGTM. We should probably add a nixos Test for borgmatic to catch those kinds of mistake..

There is an open PR #290913 adding nixos test for borgmatic.

Tested locally in a VM once with populated and once with empty source_directories and repositories. Although I think it is irrational to have both empty, it is ultimately the users responsibility  to populate both. With this change borgmatic at least does not fail to start.
One could argue about removing the default values and forcing the user at evaluation time to populate repositories and source_directories, but I'm fine either way.


ssl: temporariliy fix utility test

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>Description:
Quick fix of time dependent SSL utility test introduced in #4686.
Risk Level: Low
Testing: CI
Docs Changes: N/A
Release Notes: N/A

Also thank you for the quick fix, @lizan!!


feat(recursion): public values constraints

Can you write a small test to test that the public values are properly working?


Upgrade/validate PHP 8.3

Quick validation indicates that using the PHP 8.3 stack works fine for the deployment. However, as of today, REDCap does not yet support 8.3 officially, so keeping this issue on the backlog.

Test code exists at https://github.com/SvenAelterman/azure-redcap-paas/tree/84-php83


Added Landscape detection Folder

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change

[x] Added a new machine learning frameworks, libraries or software.
[x] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

I have added running code for landscape detection


Wasted for Murkish & Hukin

Song Name
WASTED (murkish & huken nightcore remix)
Artist Name
Juice WRLD, Lil Uzi Vert
Source
Youtube
Youtube Link
https://youtu.be/-UHnOA3Caww?si=yBvb2Nh_8eooCRd1
Direct File Link
No response
Song ID
1199528
Start Offset [ms]
No response
Extra
Used in the level "W A S T E D"
92979046

accept

accept


Gif in not played when using RoundedImageView class

try using other image lib, RoundedImageView does not support gif

using this https://github.com/rishabh876/RoundedImageView/issues/4#event-3723309236


Show example of intended server -> client RPC call

In documentation, there is no example shown of how to send a request from server -> client over an RPC call, only client -> server.
I am not sure if this is just a missing feature, or just missing documentation, but I find this a critical feature for my application, and would like to see the "right way to do it".
I have implemented an interval-polled subscribe system which I use to accomplish this, but if there's a "right way" to do this, that would be preferred.
Something maybe like this:
// in API module
@ClientRpc
interface ClientEventService : ClientRemoteService {
    suspend fun receiveEvent(event: Event)
}

// in client module

class ClientEventService : ClientEventService {
    override suspend fun receiveEvent(event: Event) {
        when (event) {
            is MessageEvent -> {
                println(event.message ?: return)
            }
            else -> println("Unhandled event: $event")
        }
    }
}

registerService<ClientEventService> { ctx -> ClientServiceImpl(ctx) }

// in server module code under an RPC request something like:
val clientEventService: ClientEventService = request.withService()
clientEventService.receiveEvent(MessageEvent("Hello world!"))

Here is a stripped-down version of my implementation to achieve something similar, although with the drawback that it only polls at a set interval and thus has polling overhead + delay:
API module (client and server both depend on this code):
@Serializable
sealed interface Event

@Serializable
data class MessageEvent(
    val message: String?
) : Event

@Rpc
interface EventService : RemoteService {

    suspend fun subscribe(uuid: Long)

    suspend fun pollEvents(uuid: Long): List<Event>?

}

Server module:
class EventServiceImpl(
    override val coroutineContext: CoroutineContext
) : EventService {

    private val uuidToEvents: MutableMap<Long, BlockingQueue<Event>> =
        ConcurrentHashMap()

    private fun emitEvent(
        uuid: Long,
        event: Event
    ): Boolean {
        val events = uuidToEvents[uuid] ?: return false
        return events.add(event)
    }

    override suspend fun subscribe(uuid: Long) {
        check(!uuidToEvents.containsKey(uuid)) {
            "UUID $uuid is already subscribed"
        }

        uuidToEvents[uuid] = ArrayBlockingQueue(256, true)

        emitEvent(uuid, MessageEvent("Successfully subscribed UUID $uuid"))
    }

    override suspend fun pollEvents(uuid: Long): List<Event>? {
        val events = uuidToEvents[uuid] ?: return null

        val polledEvents: MutableList<Event> = mutableListOf()
        events.drainTo(polledEvents)

        return polledEvents
    }

}

Then a client can utilize it like this:
const val OUR_UUID = 1234L
const val POLL_INTERVAL_NANOS = 100L * 1000 * 1000 // 100 milliseconds

val eventService: EventService = rpcClient.withService()

eventService.subscribe(OUR_UUID)

fun pollEvents(uuid: Long) {
    val events: List<Event> = eventService.pollEvents(uuid) ?: return
    for (event in events) {
        when (event) {
            is MessageEvent -> {
                println(event.message ?: return)
            }
            else -> println("Unhandled event: $event")
        }
    }
}

while (true) {
    val elapsedNanos = measureNanoTime {
        pollEvents(OUR_UUID)
    }
    val delayNanos = POLL_INTERVAL_NANOS - elapsedNanos
    if (delayNanos > 0) {
        val delayMillis = delayNanos.milliseconds
        if (delayMillis > 0) {
            delay(delayMillis)
        }
    }
}

The protocol is not yet standardized for good. but at this time, I think the intended solution for server->client communication is flows.
Every instance of RemoteService is tide to a websocket session. So, you could give each client connection a flow instance as a field in the remote service to subscribe to.
@Rpc
interface MyService : RemoteService {
    val serverToClientRequests: Flow<MyServerRequest>
}

Also, you could return a flow as a result of a remote service function invocation.
@Rpc
interface MyService : RemoteService {
    fun subscribeToServerRequests(): Flow<MyServerRequest>
}

Sorry for weak answer. I am only a user not a contributor


FIND-75 / FIND-73 Search Results Layout

Pull Request
Ticket URLs:
https://national-archives.atlassian.net/browse/FIND-75
https://national-archives.atlassian.net/browse/FIND-73
About these changes
Layout of elements based on the following design:
https://www.figma.com/proto/YVFPU9zQzYwHmuygVE970Z/NuSearch-wireframes?page-id=0%3A1&node-id=1-84&node-type=frame&viewport=-1254%2C1621%2C0.5&t=8Jq4OudZwK4UWjbP-1&scaling=scale-down-width&content-scaling=fixed&starting-point-node-id=1%3A84
How to check these changes
Compare local build (branch: feature/search-results-layout) with design.
Before assigning to reviewer, please make sure you have
[x] Checked things thoroughly before handing over to reviewer
[x]  Checked PR title starts with ticket number as per project conventions to help us keep track of changes
[x] Ensured that PR includes only commits relevant to the ticket
[x] Waited for all CI jobs to pass before requesting a review
[x]  Added/updated tests and documentation where relevant

Closing PR as it willl be superseded.


google plus doesn work

google plus doesn't work

It's working for me - http://jasonwatmore.com/post/2014/08/01/AngularJS-directives-for-social-sharing-buttons-Facebook-Like-GooglePlus-Twitter-and-Pinterest.aspx
Can you create a plunk showing the issue?

it doesnt work other there too..  here's the screenshot ..
https://www.dropbox.com/s/ubmzmz3fdg6780s/Screenshot 2015-04-09 16.51.59.png?dl=0

see the google icon became weird.  Try it out yourself.

This is how I'm seeing it, my guess would be it's something related to your machine or local network, maybe a proxy or firewall blocking a request from the google+ script?

Not an issue with the directive


TikZ support

I have used kroki with tikz support before, and I was wondering if there was interest in adding tikz support here. Thanks!

Sure, feel free to add it 👍🏻


if user cancels connection to wallet, the user is prevented from further interaction with the wallet selector

this was fixed in 1.0.0-beta.12


Reading sensor values from notifycallback

Hi, please I am new here. I am working on BLE project using two ESP32, one as a client and the other as a server. My issue is how to read sensor values sent to the client inside notifycallback. I have been trying to work it out but with no success.
All I want is to read sensor reading and display it on Oled LCD.
Please a quick research will be appreciated.
Thanks.

https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/tests/BLETests/SampleClient_Notify.cpp#L28-L35

Depending on your datatype, you can access the data inside the notifyCallback() function that @chegewara is referring to in the following way:
	uint16_t wd = ((uint16_t)pData[2] << 8) | pData[1]; 	float temp = wd / 100.0;
Keep in mind this is a static function. I have tried to move all BLE code into a library for a specific device, but i struggle to update the class variables from this static function. Let me know if anyone have a clue for how to do this.

Thanks for your prompt feedback @chegewara, I managed to retrieve it using this method below. It might not be the best way to do it but it works.
char rxValue[5];
For(int i = 0; i < length; i++)
{
rxValue[i] = pData[i];
}
rxReading = (((String)rxValue).toFloat());
Serial.println(rxReading);
//Global variable
float rxReading;
Please, I am not expert in this, but I could receive the same values I was sending from BLE server.
If there's any improvement on this, I will really appreciate it.

I really appreciate your time and effort putting into your explanation, it really helps @hetlelid.
But any improvement on my method would be appreciated.

Try to use this macro in notificationCallback() to output log value and paste here results:
ESP_LOG_BUFFER_HEX("my value", pData, length);

Thanks for your prompt feedback @chegewara, I will definitely do that. My appreciation.

Thanks for your help @chegewara, though I couldn't understand how to use what you suggested, but just putting it in my code does the job. Now I can receive true values from notificationCallback().
…..here is my code:
static void notifyCallback(
BLERemoteCharacteristic* pBLERemoteCharacteristic,
uint8_t* pData, size_t length, bool isNotify)
{
Serial.print("Notify callback for characteristic ");
Serial.print(pBLERemoteCharacteristic->getUUID().toString().c_str());
ESP_LOG_BUFFER_HEX("my value", pData, length);
Serial.print(" of data length ");
Serial.println(length);
Serial.print("Received Value: ");
for (int i = 0; i <length; i++) {
rxValue[i]= pData[i];
Serial.print(pData[i]);
Serial.print(" ");
}
fsrValue = (((String)rxValue).toFloat());
}
…..here is serial monitor output.
`BLE Advertised Device found: Name: FSR0, Address: b4:e6:2d:8d:78:b7, serviceUUID: 4fafc201-1fb5-459e-8fcc-c5c9c331914b, txPower: -21
00:41:51.490 -> haveServiceUUID: 1
00:41:51.490 -> device's UUID: 4fafc201-1fb5-459e-8fcc-c5c9c331914b
00:41:51.490 -> Found our device!  address: We are in loop.
00:41:51.490 -> We are not connected to the BLE Server yet.
00:41:51.490 -> Forming a connection to b4:e6:2d:8d:78:b7
00:41:51.490 ->  - Created client

Connected to server
Found our service
00:41:54.536 ->  - Found our characteristic
00:41:54.536 -> The characteristic value was: 0.00
00:41:54.536 -> We are now connected to the BLE Server.
Force: 0.00Kg
00:41:54.583 -> Notifications turned off
We are in loop.
00:41:56.589 -> Force: 0.00Kg
00:41:56.589 -> Notifications turned on
Notify callback for characteristic beb5483e-36e1-4688-b7f5-ea07361b26a8 of data length 4
00:41:57.692 -> Received Value: 48 46 48 48
We are in loop.
00:41:58.593 -> Force: 0.00Kg
00:41:58.593 -> Notifications turned off
We are in loop.
00:42:00.599 -> Force: 0.00Kg
00:42:00.599 -> Notifications turned on`

Looks like you have it working.

Yes, thanks for your help @chegewara. I will have to close this issue now,  since my problem is solved.
Thanks once again for your time.
Regards.


Fix ORAS HM items

Resolved #161

Thanks! Looks good to me.

Thanks again. Sorry this got ignored for so long.


Unable to download PDF

When clicking on the download PDF - I get an error page.

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

ID: a0cd4aef-33f1-d647-85eb-1a055ef216bd
Version Independent ID: 73febada-4141-88bf-7a55-e8cc375dde92
Content: Business events overview - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-ops-core/dev-itpro/business-events/home-page.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

Bug 382686 created to track this issue.

@sy-huss  - please try downloading again. It seems that this issue has been resolved.


feat(custom-scrollbars): DLT-1473 add scrollbars directive

Add custom scrollbars directive

Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[ ] Fix
[x] Feature
[ ] Performance Improvement
[ ] Refactor

:book: Jira Ticket
DLT-1473

:book: Description
Adds the scrollbar directive based on the POC.
Adds doc page.
Adds the scrollbar to the MessageInput component.
Note: I couldn't add it to the emoji picker because it caused more problems when filtering emojis.
Doesn't add a storybook since it doesn't have any props to change.
Adds test to check the the directive correctly sets up the OverlayScrollbars. For the internal behavior I am relying on the test coverage of the library.

:bulb: Context

:pencil: Checklist

For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

For all Vue changes:

[x] I have added / updated unit tests.
[x] I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script. Read docs here: Dialtone Vue Sync Script
[ ] I have validated components with a screen reader.
[x] I have validated components keyboard navigation.

For all CSS changes:

[x] I have used design tokens whenever possible.
[x] I have considered how this change will behave on different screen sizes.
[x] I have visually validated my change in light and dark mode.
[ ] I have used gap or flexbox properties for layout instead of margin whenever possible.

If new component:

I am exporting any new components or constants:

[x] from the index.js in the component directory.
[x] from the index.js in the root (either packages/dialtone-vue2 or packages/dialtone-vue3).

[x] I have added the styles for the new component to the packages/dialtone-css package.
[x] I have created a page for the new component on the documentation site in apps/dialtone-documentation.
[ ] I have added the new component to common/components_list.cjs
[ ] I have created a component story in storybook
[ ] I have created story / stories for any relevant component variants in storybook
[ ] I have created a docs page for the component in storybook.
[ ] I have checked that changing all props/slots via the UI in storybook works as expected.

:crystal_ball: Next Steps

Add image for the components overview
Create a ticket on the Dialpad project to add this to the left bar

:camera: Screenshots / GIFs

:link: Sources
https://kingsora.github.io/OverlayScrollbars/

@francisrupert @fede-dp whenever you are available, can you take a look at the docs? https://dialtone.dialpad.com/deploy-previews/pr-391/components/scrollbar.html

I don't suppose we can add this to any scrollable region in the doc site? e.g. .dialtone-sidebar. That'd be an awesome demo of this new feature.
For that matter, I'd love it if the code snippet tab panels have this applied with class="d-hmx332". And since hovering that region would show the scrollbar, it would provide a nice affordance that it's a scrollable region.

Does this plugin support that?

Should I think.. It works like that on their docsite

I don't suppose we can add this to any scrollable region in the doc site? e.g. .dialtone-sidebar or the "Classes" docs (.dialtone-doc-table). That'd be an awesome demo of this new feature.
For that matter, I'd love it if the code snippet tab panels have this applied with class="d-hmx332". And since hovering that region would show the scrollbar, it would provide a nice affordance that it's a scrollable region.

Of course, I can add it!

Looking darn good. Love the default showing when mouse enters scrollable region.

Can we show an example of each of the Variants, using the same User list?

Should Leave (default option) be the first Variant listed?

https://github.com/dialpad/dialtone/assets/1165933/92e89131-d143-4c96-a571-39aa0b45b716
Perfect timing for showing scrollbar when mouse enters scrollable area. Can we have it disappear when mousing out of the region.
https://github.com/dialpad/dialtone/assets/1165933/fc67d074-45ea-4da1-ae30-eecb67d96013

Documentation for both Scroller and Scrollbar needs clarification on the difference between them.
Scroller in particular doesn't really say what its for. Those two doc pages should reference one another and distinguish them.
Meanwhile, shouldn't we build Scrollbar into Scroller.

Documentation for both Scroller and Scrollbar needs clarification on the difference between them.
Scroller in particular doesn't really say what its for. Those two doc pages should reference one another and distinguish them.
Meanwhile, shouldn't we build Scrollbar into Scroller.

We should clarify what the scroller is for, since it's something completly different from the scrollbar (out of the scope of this PR). And since the scrollbar is a separate component, it should have it's own page.

Scroller component was built for (the extinted) dp2 to handle the conversation view.
As it says at the beginning of the scroller documentation:
A scroller component that allows blazing fast scrolling of any amount of data.
The idea was the capability to handle a massive amount of data without performance issues.
It is a technical component for technical purposes.
Some doc here https://dialtone.dialpad.com/vue3/index.html?path=/docs/components-scroller--docs
If we want to have a better naming here we can update it to Virtual scoller.


Failing m2m send: "failed to find route"

brand new channel and node states, it's def confirmed

Do you remember how much you were trying to send? Says outbound_htlc_maximum_msat is 5000000 msats, so that's only 5k sats.

Closing. Need more detailed logs and context. If it happens again when it was expected, open up a ticket.


Update LGs/OLGs

Need to update the OLGs, LGs, and SC in the main document.

Updated for 2019-2020 school year.


ensure names reflect the kinds of properties

I noticed that there are names in some definitions that are not quite reasonable.
For example, https://github.com/agda/agda-categories/blob/master/Categories/Object/Product/Core.agda
    commute₁  : π₁ ∘ ⟨ h , i ⟩ ≈ h
    commute₂  : π₂ ∘ ⟨ h , i ⟩ ≈ i
    universal : π₁ ∘ h ≈ i → π₂ ∘ h ≈ j → ⟨ i , j ⟩ ≈ h

here universal shouldn't be universal, but unique, because it reflects the uniqueness of product. rather, commutes are universality.
probably the names should be reviewed to achieve consistency.

I completely agree that universal seems to be mis-named. And certainly commute₁ is not very indicative of the actual property of the first/second projection.

universal is renamed to unique in https://github.com/agda/agda-categories/commit/68ea72ea3bec356b50bc8bc569a8ecc7be15bd11

I renamed commute to project (and to inject for Coproduct). I think we agree about this, and should enshrine this in our naming conventions. Thus this issue can be closed.


Where the trained result saved in ?

Sorry, I am a beginner.
I could not find the results after training.
It makes me feel that actually testing and training are two different things.
Thank you very much

add those code by your self:
for epoch in range(0, Config.train_number_epochs):
for i, data in enumerate(train_dataloader, 0):
img0, img1, label = data
img0, img1, label = img0.cuda(), img1.cuda(), label.cuda()
optimizer.zero_grad()
output1, output2 = net(img0, img1)
loss_contrastive = criterion(output1, output2, label)
loss_contrastive.backward()
optimizer.step()
if i % 10 == 0:
print("Epoch number {}\n Current loss {}\n".format(epoch, loss_contrastive.item()))
iteration_number += 10
counter.append(iteration_number)
loss_history.append(loss_contrastive.item())
show_plot(counter, loss_history)
save the training res
PATH = "./net.pt"
torch.save(net.state_dict(),PATH)


Need to force the use of RAII. Make Printer::Object default constructed with no action so it can be moved.

Printer::Array should be able to be default constructed and moved.


Improve UX when creating open and closed shapes

This PR aims to improve user experience when creating shapes as discussed in #117 and #101.
The SS2D_Shape node combines the functionality of open and closed shapes. A shape can be closed at any time by adding a point on top of the first point. Additionally, the new Cut Edge tool allows for opening shapes and can also split a shape into two smaller shapes (#101). Furthermore, the update includes the ability to create points with Bezier curves directly while adding them. The added functionality is fully compatible with the editor's undo-redo system.

SS2D_Shape_Open and SS2D_Shape_Closed are marked as deprecated and inherit from SS2D_Shape. These
classes are empty now and do not provide any additional functionality. They are needed for backwards compatibility. However, they can be removed in a future revision if deemed unnecessary. Certain unused methods are left commented out in case they are still needed for something.
All tests pass, and new tests have been added. The unit tests that verify closed shape functionality needed to be updated to accommodate changes in how closed shapes are handled.
Testing
While under review, my PRs (#118, #123 and #126) are available merged in this branch: https://github.com/limbonaut/SmartShape2D/tree/integration

I'm trying to bring the code up to date with all the PRs.  This one is in conflict.

I'll see what I can do. Need to get my head back into it.

@remorse107
Phew, I believe I have sorted it all out. You can merge it now.


Ignore failed signature check if no signature is present.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

If an app is unsigned, the signature check should not fail.

Review period will end on 2022-08-30 at 00:00:00 UTC.

Isn't "no signature" an issue for ARM macs?

No idea about ARM, we also don't have ARM CI for casks. I just keep seeing PRs failing with this error (e.g. https://github.com/Homebrew/homebrew-cask/pull/130308 and https://github.com/Homebrew/homebrew-cask/pull/129899).

Well, if this means they can't normally run on ARM I think the audit should stay since that's what most people will use.

https://eclecticlight.co/2020/08/22/apple-silicon-macs-will-require-signed-code/
Suggests that if an app has native arm code but no signature it won't run until people disable quarantaine, which is exactly the pattern of issues in trying to catch here.

Okay, so in these cases we refuse new casks or updates until upstream has fixed the wrong/missing signature?

if an app has native arm code but no signature it won't run until people disable quarantaine

So you cannot allow individual unsigned apps you can only disable Gatekeeper completely on ARM?

It doesn't trigger on updates

Any reason to not also check updates?

So you cannot allow individual unsigned apps you can only disable Gatekeeper completely on ARM?

You can disable quarantaine on the cask download. Which would allow an install because Apple wouldn't know it was downloaded.

Any reason to not also check updates?

Because I wanted to try it on new casks first. And upstreams likely won't remove signing on updates.

You can disable quarantaine on the cask download. Which would allow an install because Apple wouldn't know it was downloaded.

So the difference is that on ARM you would have to install with --no-quarantine since you cannot do right-click open to override it?

Because I wanted to try it on new casks first. And upstreams likely won't remove signing on updates.

I see. Yeah, I'm not saying upstreams will remove signatures, not intentionally at least.

So the difference is that on ARM you would have to install with --no-quarantine since you cannot do right-click open to override it?

Yes, unless the software is signed.

Review period ended.


Add methods "Pi.read" and "_callback_handler.remove"

Before this changes, trying to cancel a callback would raise an exception (because the "remove" method of the "_callback_handler" class wasn't implemented. This also makes "Callback.cancel" a coroutine.
Also, I implemented the "Pi.read" method. Was there any reason to not implement it?

Sorry for the delay, I somehow managed to miss your PR.
No particular reason for omitting the read method, apigpio is still very incomplete and I tend to implement methods from the original pigpio only if and when I need them.

merge PR with read and remove methods


Move to pino logging

In progress

Faster logging using pino
Centralized dependency sharing on node via AppFramework.js
Allows runtime adjustment of logging level through monitored file

These changes are needed:
diff --git a/src/lib/Installer.js b/src/lib/Installer.js
index 37f8332..7c6ecfc 100644
--- a/src/lib/Installer.js
+++ b/src/lib/Installer.js
@@ -2,7 +2,7 @@ var path = require('path');
 var Promise = require('bluebird');
 var fs = Promise.promisifyAll(require('fs-extra'));
 var execAsync = require('child-process-promise').exec;
-var logger = require('AppFramework').logger;
+var logger = require('./AppFramework.js').logger;
 var Installer = function () {
 };
 Installer.prototype.Install = function (installDir) {
diff --git a/src/system-plugins/platform-manager/platforms/beaglebone/board.js b/src/system-plugins/platform-manager/platforms/beaglebone/board.js
index 27bd7fc..d3dd39c 100644
--- a/src/system-plugins/platform-manager/platforms/beaglebone/board.js
+++ b/src/system-plugins/platform-manager/platforms/beaglebone/board.js
@@ -1,7 +1,7 @@
 const Promise = require('bluebird');
 const fs      = Promise.promisifyAll(require('fs'));
 const path    = require('path');
-const logger = require('logging.js')
+const logger = require('AppFramework.js').logger;

 class BoardInterface
 {
diff --git a/src/system-plugins/platform-manager/platforms/beaglebone/boards/2x/setup.js b/src/system-plugins/platform-manager/platforms/beaglebone/boards/2x/setup.js
index f401a7c..3dadb97 100644
--- a/src/system-plugins/platform-manager/platforms/beaglebone/boards/2x/setup.js
+++ b/src/system-plugins/platform-manager/platforms/beaglebone/boards/2x/setup.js
@@ -6,7 +6,7 @@ var spawn           = require('child_process').spawn;
 var ArduinoBuilder  = require('ArduinoBuilder');
 var ArduinoHelper   = require('ArduinoHelper');
 var Hardware        = require('./bridge.js');
-var logger          = require('AppFramework.js');
+var logger          = require('AppFramework.js').logger;
 var debug           = {};

 var SetupBoardInterface = function (board)
diff --git a/src/system-plugins/ui-manager/public/bower.json b/src/system-plugins/ui-manager/public/bower.json
index bdc6539..f152c1d 100644
--- a/src/system-plugins/ui-manager/public/bower.json
+++ b/src/system-plugins/ui-manager/public/bower.json
@@ -31,6 +31,6 @@
   },
   "resolutions": {
     "polymer": "1.6.1",
-    "webcomponentsjs": "0.7.22"
+    "webcomponentsjs": "0.7.23"
   }
 }

Also need to add to system/src/index.js
// To eliminate hard coding paths for require, we are modifying the NODE_PATH to include our lib folder
var oldpath = '';
if (process.env.NODE_PATH !== undefined) {
  oldpath = process.env.NODE_PATH;
}
// Just in case already been set, leave it alone
process.env.NODE_PATH = '/opt/openrov/cockpit/src/lib:' + oldpath;
require('module').Module._initPaths();

Opening for the review given I pushed to stick it in the merge of the other issues.  It is working on the latest dev image.  There are still follow on options that need to be completed:

[ ] Finish removing the rest of the node side console.log statements
[ ] Make the pretty output option configurable in development
[ ] Swap all occurrences where we log errors to let the framework serialize the error message for us


popover: Scroll to top of long message when viewing message actions.

Previously, when a user scrolls down a long text of message and presses the hotkey for viewing the message actions, the popover menu would continue to open where the message actions button. Thus, the popover menu would be cut short and sometimes off the screen. These changes will scroll the client to the top of the message and ensure that the popover menu is always visible.
This is a rebase of #23830 to continue and push forward the progress on the issue. No additional changes were made from it.
Fixes: #23774.

Screenshots and screen captures:

Before:

After:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Works for me in manual testing! @timabbott tagging for integration review, since you were looking at this earlier.

This is great, merged, thanks @Joelute!


Agregada dependencia JavaFX-media

Ahora el proyecto puede usar audio

Codecov Report

Merging #17 (e1245f9) into master (55148e3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #17   +/-   ##
=========================================
  Coverage     64.06%   64.06%
  Complexity      153      153
=========================================
  Files            38       38
  Lines           576      576
  Branches         35       35
=========================================
  Hits            369      369
  Misses          196      196
  Partials         11       11

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 55148e3...e1245f9. Read the comment docs.


Frame limiting

On my machine space marines uses about 50% of a processor. This seems high to me, although maybe it isn't an issue. Amethyst supports frame limiting as a way to reduce unnecessary cpu usage.

Yes, this is a know issue and seems to be originating from the Rayon crate. For more details check here.


Add TODOs after sending the first patch series via GitGitGadget

I just submitted my first patch series via GitGitGadget, and it worked,
mostly:
https://github.com/gitgitgadget/git/pull/3
<EMAIL_ADDRESS>
There were only minor glitches that I noticed immediately (and maybe
major ones I missed). Let's remember to fix them.
Signed-off-by: Johannes Schindelin<EMAIL_ADDRESS>

@dscho This PR seems like a text-only change, but the tests failed due to the ...\.test-dir not existing. Did something else change, or are these node.js tests flaky?

This PR seems like a text-only change, but the tests failed due to the ....test-dir not existing. Did something else change, or are these node.js tests flaky?

The tests do not pass on master currently... which will be fixed once #2 is merged.
BTW on GitHub, I would like to have a different strategy than in our team when it comes to merging PRs: the PR model is a fork-driven one, so most of the contributors will never be allowed to complete their PRs. So I would like to make this consistent and have always the reviewer merge the PR when they feel it is ready. I trust you with that, too ;-)

I don't have write access

Gaah! Sorry about that! It should be fixed now.


[m2e] Add m2e hint

Hint for m2e (maven 2 eclipse) to tell it to do nothing with this plugin so users of plugin don't need to setup anything in their pom such as is setup in this plugin already for maven-plugin-plugin regarding m2e.

For reference the older m2e config is no longer needed in pom to override but only documented location to this point has been a note here https://eclipse.dev/m2e/documentation/release-notes-17.html#new-syntax-for-specifying-lifecycle-mapping-metadata.  And its not exactly clear to end users that way.  Having it in the plugin metadata helps avoid the additional setup.

Thank you for your contribution @hazendaz!


Support getting an auth token when logging in through OpenID Connect

There are several ways of generating a Django REST Framework auth token but they all either require a username and password or by using the admin or manage.py commands. There isn't, as far as I can tell, a self-service way for a user authenticating with OpenID Connect to get an auth token.
Implementing such a view should be seems pretty simple though. Essentially we need a view that requires SessionAuthentication and generates and returns a token.
The motivation for having such a view is to allow OpenID Connect users of CloudLaunch to be able to authenticate from the cloudlaunch-cli.

If I'm following along, I think this would also pave a path for authenticating with cloud providers without us needing to store or acquire credentials directly from the user but could instead request temporary creds from a cloud provider (e.g., AWS) based on the identity token from an authentication provider (e.g., Google).

@afgane this is not nearly so ambitious as that 😄
Let me try to explain it this way. If a user goes to https://beta.launch.usegalaxy.org/today and logs in through GitHub, how would that user obtain a DRF auth token? The user doesn't have a password so the user can't go to https://beta.launch.usegalaxy.org/cloudlaunch/api/v1/auth/login/ to login and get an auth token. That's why I think, in this situation, we need a custom view that would generate an auth token for a user if the user is already logged in (SessionAuthentication).

This clarified the workflow/idea. Yes - sounds like we'll need that.


ci: remove [skip ci]  from gitbot actions commit for upgrading dependencies

We should ensure that our dependabot CI jobs pass before skipping the CI in the changeset commit. We can achieve this by committing the changesets and allowing the CI to run afterwards.

This is the user info set for the changesets PR: https://github.com/luizstacio/changesets-action/blob/6a46d2c28377497e64763ba5f1faeabbd3d88c33/src/gitUtils.ts#L4


Image embedding not working jupyter notebook

VSCode Version:
Version: 1.40.2
Commit: f359dd69833dd8800b54d458f6d37ab7c78df520
Date: 2019-11-25T14:52:45.129Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 18.6.0

OS Version: Mojave

Steps to Reproduce:
Not able to render the images in python Jupyter notebook

![name](mrp1.png "Title")

This works perfectly fine in browser, but renders as below in VS code.

Please open this issue on the Python extension: https://github.com/microsoft/vscode-python/issues


need to restore original spreadsheet and create new spreadsheet with Y2K changes

It looks like the original spreadsheet https://github.com/STAT325-S24/HolyokeUrbanRenewal/blob/main/data-raw/riverview_renewal_data_2024_02_27.xlsx was modified by this commit:
https://github.com/STAT325-S24/HolyokeUrbanRenewal/commit/6ecf673f85f19349eea257f0045af05d50c7bc6c
@arogers24 can you please:

restore the previous version of the spreadsheet (prior to the Y2K changes, #17) and
save the revised spreadsheet with a new date

This will help clarify the data provenance (and allow me to potentially add the Y2K fixes to the https://github.com/STAT325-S24/HolyokeUrbanRenewal/blob/main/data-raw/DATASET.R
Thanks in advance, Nick

I've restored the data and changed the name https://github.com/STAT325-S24/HolyokeUrbanRenewal/commit/042fa551f6e153121bdb3d4a3bbd47996c051f45

Many thanks!All the best,NickOn Mar 1, 2024, at 4:08 PM, arogers24 @.***> wrote:
I've restored the data and changed the name 042fa55
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


php7.1.7 composer 安装报错

  composer require "riverslei/payment:~3.1"
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - riverslei/payment v3.1.4 requires php >=5.6 -> your PHP version (7.1.7) overridden by "config.platform.php" version (5.5.9) does not satisfy that requirement.
    - riverslei/payment v3.1.3 requires php >=5.6 -> your PHP version (7.1.7) overridden by "config.platform.php" version (5.5.9) does not satisfy that requirement.
    - riverslei/payment v3.1.2 requires php >=5.6 -> your PHP version (7.1.7) overridden by "config.platform.php" version (5.5.9) does not satisfy that requirement.
    - riverslei/payment v3.1.1 requires php >=5.6 -> your PHP version (7.1.7) overridden by "config.platform.php" version (5.5.9) does not satisfy that requirement.
    - riverslei/payment v3.1.0 requires php >=5.6 -> your PHP version (7.1.7) overridden by "config.platform.php" version (5.5.9) does not satisfy that requirement.
    - Installation request for riverslei/payment ~3.1 -> satisfiable by riverslei/payment[v3.1.0, v3.1.1, v3.1.2, v3.1.3, v3.1.4].

Installation failed, reverting ./composer.json to its original content.

安装问题自己根据报错信息差。版本问题

额，不是php >= 5.6吗，目前还不支持php 7.1.7吗

你的错误提示很明显了，虽然你的php是7，但是版本被重写成了：5.5.9。所以没办法安装。

ok，找到了composer.json上有配置，覆盖了
{
"config": {
"platform":{
"php":"5.5.9"
}
}
}