update torch gpu to cpu project deps (closes #94)

With the release of poetry 1.5.0 and the according availability of the source attribute priority = "explicit" this PR changes the project dependency from the general GPU enabled torch that brings in all kinds of cuda/cudnn dependencies to the CPU version from the respective index. The former downside of poetry update searching the torch index for all dependencies and thus being slow, noted in the torch GPU vs CPU ADR, is resolved by marking the torch index explicit and using the source="pytorch-cpu" reference for the torch dependency. This should still allow downstream users to install the GPU version of torch instead of the CPU one without issues.

The issue with poetry not picking up all pytorch dependencies as seen in our CI/CD here (e.g. missing sympy) is related to a torch issue that you are aware of @aeivazi, right? Would you propose limiting torch to <2.0 for now? And could you link the corresponding torch issue here, please?

The issue with poetry not picking up all pytorch dependencies as seen in our CI/CD here (e.g. missing sympy) is related to a torch issue that you are aware of @aeivazi, right? Would you propose limiting torch to <2.0 for now? And could you link the corresponding torch issue here, please?

I restricted pytorch to <2.0 for now until the corresponding issue is fixed.

The issue with poetry not picking up all pytorch dependencies as seen in our CI/CD here (e.g. missing sympy) is related to a torch issue that you are aware of @aeivazi, right? Would you propose limiting torch to <2.0 for now? And could you link the corresponding torch issue here, please?

Reading your explanation, this is not related to torch==2.0.1. The issue I was facing was due to installing GPU torch version and missing nvidia-* dependencies: https://github.com/pytorch/pytorch/issues/100974.
This error might be specific to CPU torch?...
I've googled the error, there is a recommendation to install sympy have you tried? https://github.com/pytorch/vision/issues/7034

This error might be specific to CPU torch?...
I've googled the error, there is a recommendation to install sympy have you tried? pytorch/vision#7034

True, adding sympy works; I removed the upper bound again. All good from my side then, awaiting your review.


ListenerControl.cs: Fix typo in log message

:white_check_mark: Build ServiceBusExplorer 5.1.0-patch-1.16.build.792 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/e1a1f8058c by @gldraphael)

:white_check_mark: Build ServiceBusExplorer 5.1.0-patch-1.16.build.792 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/e1a1f8058c by @gldraphael)


Fix typo at network-policies.md

Change:

The endPort field must be equal than or greater to the port field.

To:

The endPort field must be equal to or greater than the port field.

/lgtm
/approve
/kind cleanup


New footprint type: "breakoutheaders"

These are two columns of headers

/bounty $25

/attempt #97
@seveibar
can you provide more details or a data sheet?

@seveibar

@AnasSarkiz we want something to help @Abse2001 /others with breakout boards for e.g. arduino nano
He currently has to use two headers to make it work, see this board: https://tscircuit.com/Abse2001/Robotic-Arm-Controller

@seveibar Do we need a 3D component for this?


DBeaver crashing with a certain SQL snippet

Description
While toying around with DuckDB, I found that executing this snippet causes DBeaver to crash:
CREATE OR REPLACE TABLE ProviderSupport (
	PhoneBrand varchar,
	PhoneModel varchar,
	PrimaryName varchar,
	Supported boolean not null,
	PRIMARY KEY (PhoneBrand, PhoneModel, PrimaryName),
	FOREIGN KEY (PhoneBrand, PhoneModel) REFERENCES Phone,
	FOREIGN KEY (PrimaryName) REFERENCES Provider(PrimaryName),
);

The crash began when I modified the last line (REFERENCES Provider(PrimaryName)), which is likely invalid, since Provider is a view table.
More context:

 Full SQL file
I believe this issue should be reproduceable without an existing database.
DROP TABLE IF EXISTS ProviderSupport CASCADE;
DROP TABLE IF EXISTS ProviderSubsidiary CASCADE;
DROP TABLE IF EXISTS Phone CASCADE;
DROP TABLE IF EXISTS Brand CASCADE;

CREATE OR REPLACE TABLE Brand (
	Name varchar PRIMARY KEY,
	Comment varchar,
);

CREATE OR REPLACE TABLE OperatingSystem (
	Name varchar PRIMARY KEY,
);

CREATE OR REPLACE TABLE Phone (
	PhoneBrand varchar,
	PhoneModel varchar,
	OS varchar,
	RetailCost decimal(5, 2) CHECK (RetailCost >= 0),
	SupportsAnyGSM boolean,
	SupportsAnyLTE boolean,
	Comment varchar,
	PRIMARY KEY (PhoneBrand, PhoneModel),
	FOREIGN KEY (PhoneBrand) REFERENCES Brand,
	FOREIGN KEY (OS) REFERENCES OperatingSystem,
);

-- https://en.wikipedia.org/wiki/List_of_mobile_network_operators_in_the_United_States#Facilities-based_service_providers
CREATE OR REPLACE TABLE ProviderPrimary (
	PrimaryName varchar PRIMARY KEY,
	SupportsGSM boolean,
	SupportsLTE boolean,
	SupportsNR boolean,
);

-- subsidiaries with identical support
CREATE OR REPLACE TABLE ProviderSubsidiary (
	PrimaryName varchar,
	SubsidiaryName varchar,
	PRIMARY KEY (PrimaryName, SubsidiaryName),
	FOREIGN KEY (PrimaryName) REFERENCES ProviderPrimary,
);

CREATE OR REPLACE VIEW Provider AS
	SELECT * FROM ProviderPrimary
UNION
	SELECT
		Subsidiary.SubsidiaryName,
		Main.SupportsGSM,
		Main.SupportsLTE,
		Main.SupportsNR,
	FROM
		ProviderPrimary AS Main
	JOIN
		ProviderSubsidiary AS Subsidiary
	ON
		Main.PrimaryName = Subsidiary.PrimaryName;

CREATE OR REPLACE TABLE ProviderSupport (
	PhoneBrand varchar,
	PhoneModel varchar,
	PrimaryName varchar,
	Supported boolean not null,
	PRIMARY KEY (PhoneBrand, PhoneModel, PrimaryName),
	FOREIGN KEY (PhoneBrand, PhoneModel) REFERENCES Phone,
	FOREIGN KEY (PrimaryName) REFERENCES Provider(PrimaryName),
);

-----

INSERT INTO Brand
	VALUES ('Nokia', null), ('Cricket', null), ('BLU', null);

INSERT INTO OperatingSystem
	VALUES ('KaiOS'), ('Android');

INSERT INTO Phone VALUES
	('Nokia', '6300 4G', 'KaiOS', 70.0, null, null, null);

INSERT INTO ProviderPrimary VALUES
	('AT&T Mobility', false, true, true,);

INSERT INTO ProviderSubsidiary VALUES
	('AT&T Mobility', 'FirstNet'),
	('AT&T Mobility', 'Cricket Wireless');

---

SELECT * FROM Provider;

hs_err_pid11210.log
dbeaver-debug-1732266996950.log

DBeaver Version
DBeaver <IP_ADDRESS>411010813
Operating System
Void 6.1.114_1 x86_64 AuthenticAMD notuptodate rrrmFFFFFFFFFFFFF
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Hello @ernieIzde8ski,
The fatal log shows that the crash happened inside the DuckDB driver. Can you please tell me what driver version you are using?
Additionally, you can try updating to the latest version of the driver if there's one. You can do it this way:

Open your Connection setting and press Driver settings

Go to Libraries tab. select the driver artifact and press Download/Update

Select newer driver version in the list and press Download

Press OK to close Driver settings window and try connecting again

It's been a while since there has been an update here. Please let me know if the issue is still present and provide the requested information.


Improved Ruby error classes

cc @adrianna-chang-shopify @casperisfine @composerinteralia @dhruvCW @matthewd
We made some improvements to the Ruby extension's error classes recently but based on recent discussions I think still have a way to go. I wanted to gather opinions in this issue and am happy to implement what we can come to a consensus on.
Problems I see:

QueryError is our default fallback error class. I don't think this makes any sense, as this is a ProtocolError subclass but all errors other than TRILOGY_ERR are IMO connection-level errors. I think we should make BaseConnectionError the default error class for things which haven't been given a specific meaning (alternatively we could exhaustively define error classes for all trilogy status codes).
"TRILOGY_CLOSED_CONNECTION". This error represents essentially an EOF over our socket, and it's one of the more common errors we come across. It's also the error we'll see if we attempt communication after a trilogy_sock_shutdown. Currently this appears as QueryError: TRILOGY_CLOSED_CONNECTION and we're forced to do string matching to detect it. I think this deserves its own error class, but also it should probably be distinct from ConnectionClosed, which is only raised from an explicitly closed connection. How about Trilogy::EOFError? For backwards compatibility I'd probably include "TRILOGY_CLOSED_CONNECTION" in the string.

Another possibility I'd like to entertain is deleting the translation of EPIPE errors to TRILOGY_CLOSED_CONNECTION, which would break compatibility with those doing string matching but I think otherwise is just better and more accurate information.

SyscallError vs ConnectionError. As raised in https://github.com/trilogy-libraries/trilogy/pull/98, there's some confusion between SyscallError and BaseConnectionError. One option is that we can make Trilogy::SyscallError inherit from ConnectionError and then have only one thing to rescue, or we could better document that users should rescue both classes (I don't feel very strongly either way).
I think ProtocolError's name is confusing. I read this as "protocol violation" rather than "we were told over the protocol that an error occurred", and I've seen confusion from others about the same. I preferred the previous DatabaseError naming but am open to other options. How about ServerError "an error reported by the server"? Whatever we choose we can make the classes aliases of each other, so there should be minimal compatibility problems.

I think the key thing to consider when designing error hierarchies, is the use case.
Which errors do users need to rescue?
For network libraries my answer is generally that errors fall in two main groups:

All network / IO related errors: you likely want to rescue them and eventually be able to retry.
Client errors (or query error), e.g. invalid query: you want them to be very distinct so you don't rescue them by accident.

You can then subclass this as much as you want for readability.
e.g. https://github.com/redis-rb/redis-client/blob/68dbe6b27d259f29af099128fe6ad83193160253/lib/redis_client.rb#L80-L131

I think we should make BaseConnectionError the default error class for things which haven't been given a specific meaning

This makes sense to me 👍 Also makes treating all the syscall errors as connection errors easy to reason about.

"TRILOGY_CLOSED_CONNECTION"

Definitely agree that this shouldn't be a QueryError as it currently is -- we're matching against TRILOGY_CLOSED_CONNECTION all over the place in our Rails monolith at Shopify, which is definitely a pain. Trilogy::EOFError would make sense to me.

Another possibility I'd like to entertain is deleting the translation of EPIPE errors to TRILOGY_CLOSED_CONNECTION

What are you thinking instead @jhawthorn ? Just let it bubble up as a syscall error?

SyscallError vs ConnectionError.

I think it's fair to treat all of the errno errors as connection errors. Seems like most of the ones consumers would actually encounter would be connection errors. That gets us around the headache of trying to figure out which ones we should classify as conn errors versus not, and I think, as the issue opened around EADDRNOTAVAIL demonstrates, it's confusing to users to have to rescue these as separate error classes.

I think ProtocolError's name is confusing.

Yeah, might as well just revert back to DatabaseError, since it's still there for compatibility reasons? Versus introducing a new error that needs to be aliased to both DatabaseError and ProtocolError.

All network / IO related errors: you likely want to rescue them and eventually be able to retry.

The thing that jumps into my head here is something like CommunicationError -- "[especially if you have a raw syscall error,] we potentially have no idea exactly what went wrong, it's (very likely, but) not certainly an issue between our networking stack and the remote end... but either way, we didn't manage to talk to the remote server".
(But that's just a passing thought, and even if we did think it's more descriptive, I'm doubtful it's worth the rename effort to change; ConnectionError is already a very reasonable spelling of the same concept: there are extremely few possible sources of syserror that aren't strictly a "connection" problem.)
I agree that SyscallError should include ConnectionError.
I don't like that BaseConnectionError sounds like it's the root of all connection errors, when it's actually the opposite... GeneralConnectionError? I was going to suggest "Unknown..", but while that might be reasonable for the fallback case (and honestly, many of the currently-enumerated code that boil down to "protocol violation"), it sounds too uncertain for the "expected" errors.
Agree not-TRILOGY_ERR should default to some sort of connection error, not a query error.
Also agree that I've misread ProtocolError as "error in protocol implementation" rather than "error encoded via protocol" more than once. I don't love DatabaseError just because of how generic it sounds: we're here to talk to a database, so that's arguably true of anything that could go wrong.
One idea I had was that we could consciously deviate from the "FooError" pattern as a gentle nudge that we're [using an exception as a means of] communicating an error from elsewhere, rather than directly reporting a "local" error up the stack: ErrorResponse, ErrorFromServer. Alternatively: QueryError? ISTM most of the other things that can go wrong all boil down to problems inside the supplied query; is there a lot of value in us uniquely distinguishing syntax/parse errors from other ("that column doesn't exist") errors?

In support of our SyscallError being a ConnectionError, turns out GitHub is running a small patch that didn't make it upstream for whatever reason that basically adds this around the call to ::Trilogy.new:
rescue Errno::ETIMEDOUT, Errno::ECONNREFUSED, Errno::ECONNRESET => error
  raise ActiveRecord::ConnectionNotEstablished.new(error.message)

There's a segfault happening in the OpenSSL library built into Ruby, and when TrilogyClient tries to communicate with the DB with a blank certificate, OpenSSL segfaults, and the whole Ruby process gets dumped.
https://bugs.ruby-lang.org/issues/19974
This crashes the entire Ruby process, because the crash happens in C-land.
PR to fix it is here: https://github.com/ruby/openssl/pull/690


Allow starting multiple environments at once

Is there any chance we start leaking networks (if eg things are killed forcibly)?
See #4834 (comment)

That's not a problem on GHA since nodes are recycled. Locally I've added env prune which does a cleanup of all environments (containers and networks).

Locally I've added env prune which does a cleanup of all environments (containers and networks).

I used to do manual prune before we switched to Launcher.
(prune was needed for containers, volumes, networks).
I enjoy the fact that i haven't had to use it since then because things Just Work™.
While I see benefit of being able to stand up 2 environments side by side, I feel it is a niche use case compared to other uses of Launcher.
Because of this, if there are downsides affecting all usecases, shouldn't we make "mutli env" an opt-in feature?
I will be glad to pay the cost of adding additional toggle to env up when when i need 2 envs side by side -- for the benefit of not using env prune or zsh/bash function to find my current environment in all the other cases.
cc @iirekm

@findepi I've dropped "multiple environments" part from this PR as it will require more changes (Kerberos tickets are not happy with other container names)


Create HotRodMetadata to avoid conflicts

Hot Rod was using Metada.version() and it conflicts with the write-skew versions
under discussion in https://issues.jboss.org/browse/ISPN-7672

I don't see any tests... (but I understand this is a preview).
Why can't you use SimpleClusteredVersion for the purpose HR uses NumericVersion? It seems that if you just move topologyId to first 2 bytes, you get the same, though I don't know if rank is really needed when the versions are generated on primary owner.

I think it is in the other way. You need the rank and not the topology id. The client can contact any node if the primary takes time to reply.
just to recall, there are 2 issues: the first is the use of VersionGenerator assuming a NumericVersionGenerator. It is not true for TX+OL+RR+WS. Second, it uses Metadata.version to store its version. It is bad because we store the versions internally to keep data consistent.

Rank itself has little significance: NVM uses viewId + rank to identify the node, so it's alternative way to encode the address.
So in case of HR, is it the originator who sets the version in metadata? Why can't we just rely on the version generated by WS, and not use any originator-provided version at all?

Rank is the view id + position in the view: https://github.com/infinispan/infinispan/blob/master/core/src/main/java/org/infinispan/container/versioning/NumericVersionGenerator.java#L101
In HR, the originator creates the version: https://github.com/infinispan/infinispan/blob/master/server/hotrod/src/main/java/org/infinispan/server/hotrod/CacheDecodeContext.java#L134
I guess we can use the WS version when it is available. I'm not aware of any issues at the moment. I'll give it a try.

Nope, rank == position in the view, and is 2 bytes long. (view id, rank) is called "version prefix".
Using WS only when it's available does not seem as clean design, either provide it always from the originator (as done before) or always use some version provided by primary (enforce some version generator in server).

@rvansa I think will continue improve this PR. Splitting the version from HR from the entry version. This way it will more future proof. Probably, we need a new version generator for eventual consistent as well.

@pruivo I am working on ISPN-7590 last few weeks and this needs separate mechanism (API in Metadata) - you can't just store one version, but entire invocations history in order to prevent retried application of given command.


Customise status with time remaining

I wanted to see a "time to next save" instead of just the time between saves, so for now I've just hacked away at https://github.com/tmux-plugins/tmux-continuum/blob/master/scripts/continuum_save.sh, so that I now see:

Do you have any plans to add functionality to make such customization easier / not require messing with the internals of this plugin (or is that already possible and I missed it)?

Could you please tell me how to incorporate that in my .tmux.conf? Thank you.

Could you please tell me how to incorporate that in my .tmux.conf? Thank you.

As I mentioned, it's not something you can currently achieve using just the .tmux.conf (which is why this issue exists!).
Here are the changes I made to  https://github.com/tmux-plugins/tmux-continuum/blob/master/scripts/continuum_status.sh:
diff --git a/scripts/continuum_status.sh b/scripts/continuum_status.sh
index 280cf00..83b488b 100755
--- a/scripts/continuum_status.sh
+++ b/scripts/continuum_status.sh
@@ -4,21 +4,39 @@ CURRENT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"

 source "$CURRENT_DIR/helpers.sh"
 source "$CURRENT_DIR/variables.sh"
+source "$CURRENT_DIR/shared.sh"
+
+get_interval() {
+	get_tmux_option "$auto_save_interval_option" "$auto_save_interval_default"
+}
+
+
+current_timestamp() {
+	echo "$(date +%s)"
+	"$(date +%s)"
+}

 print_status() {
-	local save_int="$(get_tmux_option "$auto_save_interval_option")"
+        local save_int="$(get_tmux_option "$auto_save_interval_option")"
 	local status=""
+	local style_wrap="$(get_tmux_option "$status_on_style_wrap_option" "")"
 	local style_wrap
 	if [ $save_int -gt 0 ]; then
-		style_wrap="$(get_tmux_option "$status_on_style_wrap_option" "")"
-		status="$save_int"
+            local last_saved_timestamp="$(get_tmux_option "$last_auto_save_option" "0")"
+            local interval_minutes="$(get_interval)"
+            local interval_seconds="$((interval_minutes * 60))"
+            local next_run="$((last_saved_timestamp + $interval_seconds))"
+            local now="$(date +%s)"
+            local secs="$((next_run - now))"
+            local til_next_save=$(printf '%02dh:%02dm:%02ds\n' $((secs/3600)) $((secs%3600/60)) $((secs%60)))
+            status="$til_next_save"
 	else
-		style_wrap="$(get_tmux_option "$status_off_style_wrap_option" "")"
-		status="off"
+	    style_wrap="$(get_tmux_option "$status_off_style_wrap_option" "")"
+	    status="off"

 	fi

 	if [ -n "$style_wrap" ]; then
-		status="${style_wrap/$status_wrap_string/$status}"
+	    status="${style_wrap/$status_wrap_string/$status}"
 	fi
 	echo "$status"
 }

Git diff attached as a txt file:
tmp.diff.txt
or as a
drop-in replacement:
continuum_status.sh.txt
NB: This needs to be renamed to remove the trailing .txt - that was just so that github would accept it!

The corresponding line in the .tmux.conf that shows the save time and the icon:
set -g status-right '💾#{continuum_status}#[fg=colour233,bg=colour241,bold] %d/%m #[fg=colour233,bg=colour245,bold] %H:%M:%S '

Thank you for the explanation.


Protocol Negotiation

I think some details are missing but a second review would be good for sanity checks,
1, commit: https://github.com/holepunchto/bare-http1/pull/11/commits/b25dc755862c306b1e09b4b9b0e8ce5f06908c43
Idk if all events needed are being detach, especially those from the client connection, if I detach the close event, it breaks the close connection if missing upgrade handler unit test (from our topic 3).
I also saw in some logs that the end event is not detached at node.js.
I am having trouble creating unit tests for this issue: Any client request tied to the socket should also be closed., that check should be done at both Client connection and Server connection?
2, commit:  https://github.com/holepunchto/bare-http1/pull/11/commits/9b901c7e6fcdb9a924f761fe6922219a4b5162b9
Added new logic only for the request upgrade because this node.js doc: https://nodejs.org/api/http.html#event-upgrade_1 ("Listening to this event is optional")
3, commit: https://github.com/holepunchto/bare-http1/pull/11/commits/153e906f54a12cf71fd05e904de400908cc33e3f

Idk if all events needed are being detach, especially those from the client connection, if I detach the close event, it breaks the close connection if missing upgrade handler unit test (from our topic 2), with the error Socket hungs up.

All events should be detached as, after emitting upgrade, the socket is entirely owned by the caller. You can find the equivalent code in Node.js here:

https://github.com/nodejs/node/blob/1b965270a9c273d4cf70e8808e9d28b9ada7844f/lib/_http_server.js#L930-L935
https://github.com/nodejs/node/blob/1b965270a9c273d4cf70e8808e9d28b9ada7844f/lib/_http_client.js#L558-L560
https://github.com/nodejs/node/blob/1b965270a9c273d4cf70e8808e9d28b9ada7844f/lib/_http_client.js#L576-L577

The issue with close not being emitted was due to the request not being destroyed on upgrade, which I've addressed in 07ce84e.

I am having trouble creating unit tests for this issue: Any client request tied to the socket should also be closed., that check should be done at both Client connection and Server connection?

You can test that on the client side by making sure that req.on('close') is emitted. On the server side, the req will have already been closed by the time the upgrade event is emitted so that's not trivial to test; let's skip that for now.

First, thanks for the detailed review and related links.
Maybe silly thing but I was wondering about the 'close' event of req parameter at server-side upgrade event.
Node.js docs only speaks about the data event: "After this event is emitted, the request's socket will not have a 'data' event listener (..)"
In the source code we see many events being detached. And by running the following sample code only the req-side is logged.
const server = http.createServer()
server.on('upgrade', (req, socket, head) => {
  req.on('close', () => console.log('[server] request closed'))

  socket.end(
    'HTTP/1.1 101 Web Socket Protocol Handshake\r\n' +
      'Upgrade: weird-protocol\r\n' +
      'Connection: Upgrade\r\n' +
      '\r\n'
  )
})

server.listen(1337, () => {
  const req = http.request({
    port: 1337,
    headers: { Connection: 'Upgrade', Upgrade: 'weird-protocol' }
  }).end()

  req.on('upgrade', (res, socket, head) => {
    req.on('close', () => console.log('[req] request closed'))

    socket.end()
  })
})

Continuing..
At our current implementation, we don't bind a callback to be detached and yet, the event 'close' is triggered motivated by the req.destroy() line at the _onupgrade fn.
Later, I started to hope that the current behavior was the intended and the socket parameter having a close event-listener wasn't a wrong behavior, which makes sense following the hint:

On the server side, the req will have already been closed by the time the upgrade event is emitted

So I updated the test:
req.on('close', () => t.fail('request close event listener should be detached'))

to:
req.on('close', () => up.pass('request closed after server upgrade event'))

After running the tests many times and now passing by CI, the req closure after the upgrade event seems to be deterministic.

Sorry about the blog post 😅, tell me please if I made a wrong assumption and/or if there's more work to do on this Pull request.

All good, I don't mind reading! 😂 Your assumptions are correct, the only thing left now is to land this! 👏


Update the following URL for Azure Government

Hello,
In this article the following URL *.hypervrecoverymanager.windowsazure.com is listed as a proper endpoint for Azure Government. It should be updated to reflect the ".us" DNS extension vs the ".com".
If you have any questions, please feel free to contact me.

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

ID: 77b55957-eb46-c35a-5f31-66ca20ad4390
Version Independent ID: 10500bf3-d343-0bc9-cc16-642c5116a131
Content: Azure to Azure disaster recovery architecture in Azure Site Recovery - Azure Site Recovery
Content Source: articles/site-recovery/azure-to-azure-architecture.md
Service: site-recovery
GitHub Login: @ankitaduttaMSFT
Microsoft Alias: ankitadutta

@AnthonyDelagarde
Thanks for your feedback! We will investigate and update as appropriate.

@AnthonyDelagarde We have verified and updated the document as required. The update would go live in less than 24 hours. Thanks again!


merge from arlac77/template-github,arlac77/template-kronos-module

package.json

chore(package): add >=14.0.0 remove >=13.13.0 (engines.node)

.travis.yml

chore(travis): (jobs.include)
chore(travis): remove 13.13.0 (jobs.include.node_js)

:tada: This PR is included in version 8.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


"Hacker" Tomu FPGA (Fomu) v0.0 support

Successfully flashed the Fomu v0.0 "Hacker" board (ice40up5k-uwg30) with this build (using fomu-flash) and was able to upload a user program with tinyprog -p.
arachne-pnr doesn't produce a working bitsream for me and nextpnr-ice40 seems very unstable as to whether or not things will work.  The /dev/ttyACM0 enumeration and programming works with the seed 12345678, even though icetime reports that it is only 19.69 MHz.

@osresearch - I would love to merge this support. Are you going to have any time to clean it up or should I try and find time to do it?

This needs to be re-tested with a more recent checkout, although I don't have my programming jig handy. I can test with the existing bootloader and write into the user application space, which will hopefully be sufficient to validate.

I've merged this with master and it seems to work fine on my one Fomu to test with. The 48 MHz clock "passes" timing at 38 MHz, which is unchanged from before.


[OSD-10467] Set Resources.Limits and Resources.Requests

As we are going to add new web server running in pod on the infra nodes. It has the risk that the service itself have some performance issue, which may cause the memory or cpu usage issue on that node.
What type of PR is this?
(feature/refactor)
What this PR does / why we need it?
Sets Resources.Limits and Resources.Requests for cpu and memory in ocm-agent deployment config
Which Jira/Github issue(s) this PR fixes?
Fixes # OSD-10467
Special notes for your reviewer:
Pre-checks (if applicable):

[ ] Ran make test command locally to validate code changes

Codecov Report

Merging #35 (5cbc0e2) into master (c97bfe2) will increase coverage by 0.35%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
+ Coverage   58.76%   59.11%   +0.35%
==========================================
  Files          22       22
  Lines        1147     1157      +10
==========================================
+ Hits          674      684      +10
  Misses        413      413
  Partials       60       60

Impacted Files
Coverage Δ

pkg/ocmagenthandler/ocmagenthandler_deployment.go
90.22% <100.00%> (+0.38%)
:arrow_up:

/lgtm


Support Minio SSE-C

We would like to employ goofys on top of Minio but due to incoming EU data protection laws would like to encrypt by default. Minio supports SSE-C, please will you add this to the options already available?

is there a reason why this was based on the append branch and not master?

hmm this doesn't seem to be an actual PR, closing


Create preset meshconfig

Description:
We still need to support setting mesh configurations from CLI during installation. This was removed when migrating from osm-config configmap to MeshConfig CRD.
In this PR, osm install will create a MeshConfig resouce preset-mesh-config using Helm. init-osm-controller container will use it to create a osm-mesh-config MeshConfig if does not exist. However if osm-mesh-config exists, init-osm-controller respects the existing MeshConfig, and ignores the preset.

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Demo                   [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

Codecov Report

Merging #3307 (12b35c1) into main (9d65643) will decrease coverage by 0.28%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3307      +/-   ##
==========================================
- Coverage   66.45%   66.16%   -0.29%
==========================================
  Files         165      165
  Lines        7807     7820      +13
==========================================
- Hits         5188     5174      -14
- Misses       2588     2615      +27
  Partials       31       31

Flag
Coverage Δ

unittests
66.16% <0.00%> (-0.29%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/init-osm-controller/init-osm-controller.go
46.66% <0.00%> (-9.79%)
:arrow_down:

pkg/configurator/validating_webhook.go
73.33% <0.00%> (-8.49%)
:arrow_down:

Continue to review full report at Codecov.

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

Closing this as I have created an alternative PR : https://github.com/openservicemesh/osm/pull/3331


Parse the correct data for aooverlaps

@ATenderholt I hope you weren't already coding this. Anyway, here is the fix and additional testing. I also removed the aooverlap test from unrestricted SP test, since they are the same.
Could you run the DFT SP unittests for 6.5 with output for aooverlaps?
P.S. I sent a message to nwchem-users about print "overlaps".

@langner I was going to try and code it tonight, but you beat me! I left two comments on your code (above).
Also, what do you mean by "Could you run the DFT SP unittests for 6.5 with output for aooverlaps?" There should be the alternate overlap file called dvb_sp_hf_overlaps.out. Is there a way to tell a unit test script to run against an alternate file?

Also, what do you mean by "Could you run the DFT SP unittests for 6.5 with output for aooverlaps?"

I mean the unittests logiles for SP, but with DFT instead of HF (called dvb_sp_ks instead of dvb_sp_hf). They are still missing aooverlaps for NWChem 6.5. If you have 6.5 deployed somewhere, then I wouldn't have to install it.

There should be the alternate overlap file called dvb_sp_hf_overlaps.out. Is there a way to tell a unit test script to run against an alternate file?

There is, but as far as we can see, the output is incomplete, right? So it would be better to maybe use this as a regression, and add a note to remember.

Ah yes, it makes sense to have the DFT SP test as well as the HF SP test. Just pushed it.
Also, I agree that the one of the overlap versions should be a regression. I've put examples of the compact overlaps in this branch for the time being.

Sounds good... anything else we need to take care of here?


Error in calculation of height?

I wonder if there is an issue in the calculation of the height of some pieces of latex: if you have the library render \debug x^2 you get

As you can see, the 2 sticks out of the top of the bounding box, and I think this translates to the _height of the overall box being calculated incorrectly. There seem to be lots of places _height gets set for various boxes, and I don't quite know where to start debugging this one - any tips?

As you can see, the bounding boxes (read from TFM) are slightly different from the rendered glyphs, I'm not sure how the original TFM was generated.

But this is not important now, I'm planning to transfer from plain ASCII TeX to unicode-math as mentioned with #21, thus this issue could be resolved.

Aha, so that's the reason. My workaround for now is to an extra pixel to the top and bottom to allow for this sticking-out, which seems to be fine in most contexts. Will be interesting to see how it looks after the font refactoring.

@NanoMichael
Is it progressing? want to use it in android platfrom

@Nanamare Yes， please take a look at branch openmath and this discussion #87, the Uni-math is still in progress, but I'm sure it will not take a long time.


Support Angular (Vite)

We need to investigate and see if Angular (Vite) is doing anything special in Vite to differentiate between client and server bundles and if not, we will add in support to auto insert into the bundle name to identify the different bundles.

I reviewed and it appears as of now (Aug 2024), Vite is only used in Angular's dev server (see https://angular.dev/tools/cli/build-system-migration#vite-as-a-development-server), so Angular (Vite) may not yet be a candidate to integrate for our production build monitoring plugin stack.
We can reopen this if/when this changes!

Vite as a development server
The usage of Vite in the Angular CLI is currently only within a development server capacity only. Even without using the underlying Vite build system, Vite provides a full-featured development server with client side support that has been bundled into a low dependency npm package. This makes it an ideal candidate to provide comprehensive development server functionality. The current development server process uses the new build system to generate a development build of the application in memory and passes the results to Vite to serve the application. The usage of Vite, much like the Webpack-based development server, is encapsulated within the Angular CLI dev-server builder and currently cannot be directly configured.
https://angular.dev/tools/cli/build-system-migration#vite-as-a-development-server


Transactions signed by polkadot js are invalid if paying fees with foreign assets greater than 0

https://github.com/polkadot-js/api/issues/5710


[->main] Update dependencies on server to remove prerelease versions

Also update package lock

nvm looks like there are already new changes going to the release branch, so getting all those at once

nvm looks like there are already new changes going to the release branch, so getting all those at once


Add Turnstile (captcha) to worker?

Added own/custom url to the worker. Within a short time I got Netcraft takedown notice to my accounts at Cloudflare and Registrar.
Only used it on own tenants.
How to add Turnstile so automatic scanners like Netcraft do not find my worker.

Hi, please read https://developers.cloudflare.com/turnstile/get-started/ to see how to add it :)


[Fix][Client] Fix pending message not complete when closeAsync

Motivation
In the current process of closeAsync() in ProducerImpl.java, the client will firstly set the producer to the Closing state, then stop timerTask of sendTimeout, and then send a http request to broker to close the producer. When the client receives a successful response from broker, it sets the producer to the Closed state, removes the producer from the producers in client, and complete the PendingMessage with AlreadyClosedException.
However, if the communication between client and broker is abnormal due to network or other reasons, the PendingMessage will not be cleared anymore. What's worse is that since the producer is already in the Closing state at this time, there is no chance to trigger the cleanup of PendingMessage by calling the closeAsync() again, which is obviously unreasonable.
Modifications
There is no need to wait for the broker's response to clean up PendingMessage.  At this time, failing quickly is more valuable than waiting.
Verifying this change

[x] Make sure that the change passes the CI checks.

Add a test case to ensure closeAndClearPendingMessages() has been invoke when request broker error after closeAsync().
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

There's a related issue #11780

There's also #12195

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.32%. Comparing base (bbc6224) to head (d2e261e).
Report is 803 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23761      +/-   ##
============================================
+ Coverage     73.57%   74.32%   +0.75%
+ Complexity    32624    32146     -478
============================================
  Files          1877     1838      -39
  Lines        139502   143048    +3546
  Branches      15299    16232     +933
============================================
+ Hits         102638   106323    +3685
+ Misses        28908    28355     -553
- Partials       7956     8370     +414

Flag
Coverage Δ

inttests
26.72% <100.00%> (+2.13%)
:arrow_up:

systests
23.67% <100.00%> (-0.65%)
:arrow_down:

unittests
73.71% <100.00%> (+0.86%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines
Coverage Δ

...va/org/apache/pulsar/client/impl/ProducerImpl.java
84.09% <100.00%> (+0.50%)
:arrow_up:

... and 1005 files with indirect coverage changes


API not working in local server

Hi there,
I'm trying to use this wonderful API for a class project and whenever I try to build the project in local it gives no error message.
However, when trying to get data from the API, for example: localhost:3000/api/games
it gives me INTERNAL SERVER ERROR

It's because you need to set the env variables to match the MongoDB database. Do you really need to run this API locally?

It's because you need to set the env variables to match the MongoDB database. Do you really need to run this API locally?

It'll be nice if I could run it locally since I got the 502 error very often and I think it is a problem of the remote server

It'll be nice if I could run it locally since I got the 502 error very often and I think it is a problem of the remote server

Yeah, I know that. It happens because NextJS wasn't exactly planned to work as a backend connected to a database directly. I will soon release a v2 that will remove the need to check the database and instead use local JSON files, so you could run locally too.

Yeah, I know that. It happens because NextJS wasn't exactly planned to work as a backend connected to a database directly. I will soon release a v2 that will remove the need to check the database and instead use local JSON files, so you could run locally too.

Ok I understand, nice work BTW. It would be damn cool if it had images but I guess you'll try to improve the API in the future. Keep coding bro, appreciate the work 😄

Ok I understand, nice work BTW. It would be damn cool if it had images but I guess you'll try to improve the API in the future. Keep coding bro, appreciate the work 😄

Thanks for the feedback! :smile:

Thanks for the feedback! :smile:


thtk: init at 12-unstable-2024-11-12

Fix #359985
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

@ofborg eval

I'm not sure if I have permissions to trigger another ofborg eval, but the derivation seems to build fine on my local machine, so i'm not sure what is causing the failures

Someone has missspelled a keyword https://github.com/NixOS/nixpkgs/commit/c4f05d1e2ba9a14b0c0531b2c5873b4905b9352b

@theobori Ah, ok. If you are able to change something and force push you should see the tests going green. I think even just an interactive rebase that changes the commit time would be enough


Unable to setup with neo4j inside VM

I have a neo4j connection within a VirtualBox VM, but I can't manage to make it work in PHPStorm. The neo4j instance is configured to accept remote connections, and I can access it from outside the box:
$ curl <IP_ADDRESS>:7474
{
  "management" : "http://<IP_ADDRESS>:7474/db/manage/",
  "data" : "http://<IP_ADDRESS>:7474/db/data/"
}

However trying this in PHPStorm I get the following error:

Is there anything else to configure besides what's mentioned in #5?

Hi @Anahkiasen,
Plugin is working only with Bolt connections.
You are specifying port "7474" which corresponds to HTTP connector.
You should use "7687" port, which is used for Bolt connections by default.
You can check you configuration for "dbms.connector.*" properties in neo4j.conf.
Also I believe that you will need to forward Bolt ports from VM to host machine.


Support cloning over ssh in epm

This would be useful to be able to work directly with epm installed packages without having to change the remote url first

If you are cloning from a hosted git service (GitHub, GitLab), they all supports cloning over HTTPS so this feature is not needed.
If you are cloning from a personal server, you can always clone manually - it's slightly less convenient, but since you won't be using paths from a personal server as a dependency, it's not a big issue.
Can you elaborate the use case here?

Lazyness, forking a repository on github and then just installing it with epm is quite handy to get the structure right
Private repositories, usually I try to get the base of package done before I open source it so cloning it over ssh would be quite useful for that

@SitiSchu I think it should not be too hard to add this, but to your points, I think both should be doable over HTTPS instead of SSH - just use the HTTPS URL of the repository. Am I missing something?

@SitiSchu, Ping? It's still unclear, at least to me, what the problem is. You seem to be suggesting an option for changing implicit "https://" imports to "ssh://" imports when executing epm:install. I say "an option" because I don't think we want anything other than https:// to be the default transport for this mechanism or make it dynamic by trying HTTPS and falling back to SSH. So, if I understand what you are requesting, instead of
epm:install github.com/xiaq/edit.elv

someone would be able for force the use of SSH for the transport of the install by doing this:
epm:install &ssh github.com/xiaq/edit.elv


bug: Kafka connector reports error when creating two sources to the same broker.

Describe the bug
When user creates two sources to the same kafka instance, it reports error:
ERROR:  QueryError: internal error: Meta data fetch error: BadMessage (Local: Bad message format)
Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
Local playground
The version of RisingWave
1.3.0
Additional context
It works in 1.2.0, only failed in 1.3.0

For more details, please ping @liurenjie1024 in private.

Neither @tabVersion  nor I could reproduce this problem later, so this shouldn't be a bug in RisingWave playground, but some issue from kafka side. What a coincidence😅


Docker images not available on arm64

Is your feature request related to a problem? Please describe.
When running on a arm64-based k8s instance the image fails to pull
 Type     Reason     Age                From               Message
  ----     ------     ----               ----               -------
  Normal   Scheduled  43s                default-scheduler  Successfully assigned database/emqx-operator-controller-manager-77879d7747-wlcmj to rpi5
  Normal   Pulling    28s (x2 over 42s)  kubelet            Pulling image "ghcr.io/emqx/emqx-operator:2.2.23"
  Warning  Failed     27s (x2 over 41s)  kubelet            Failed to pull image "ghcr.io/emqx/emqx-operator:2.2.23": rpc error: code = NotFound desc = failed to pull and unpack image "ghcr.io/emqx/emqx-operator:2.2.23": no match for platform in manifest: not found
  Warning  Failed     27s (x2 over 41s)  kubelet            Error: ErrImagePull
  Normal   BackOff    12s (x2 over 41s)  kubelet            Back-off pulling image "ghcr.io/emqx/emqx-operator:2.2.23"
  Warning  Failed     12s (x2 over 41s)  kubelet            Error: ImagePullBackOff

Describe the solution you'd like
Build the docker image for arm64
Describe alternatives you've considered
Migrating my cluster to X86, however this is going to be costly both in power and dollars
Additional context
Add any other context or screenshots about the feature request here.

Hi, @RonaldPhilipsen How did you deployment EMQX operator? looks you use the ghcr.io image, you should use docker hub image, it supported arm64: https://hub.docker.com/r/emqx/emqx-operator-controller/tags

Hi, @RonaldPhilipsen How did you deployment EMQX operator? looks you use the ghcr.io image, you should use docker hub image, it supported arm64: https://hub.docker.com/r/emqx/emqx-operator-controller/tags

Why are they different? Should it not be the same image in both registries?
Docker hub is frowned upon due to the pull limits imposed there, which the org members will not notice but most if not all others will. Please make sure the ghcr.io published images are also multi-arch or don't have them at all if you don't support using them.

Hi @Rory-Z, I´d have to concur with @samip5.
Pull limits are annoying, and it shouldn´t be too much effort to also deploy the same images to both registries

@samip5 @RonaldPhilipsen Thanks for your feedback, let me add multi arch in ghcr.io


Flutter tool general.shard/project_test.dart fails run locally when Android artifacts not downloaded

Either:

The test should trigger an artifacts download if required.
Fail by telling the user what command to run to get the artifacts locally.

Additionally, should this be in general.shard if it requires artifacts to be locally downloaded?  Maybe should be hermetic?
Failed to load "test/general.shard/project_test.dart": Directory listing failed, path = '/Users/abhishek/Desktop/open_source/flutter/bin/cache/artifacts/gradle_wrapper/' (OS Error: No such file or directory, errno = 2)
dart:io                                                      _Directory.listSync
2
ForwardingDirectory.listSync
package:file/…/forwarding/forwarding_directory.dart:43
transfer
test/general.shard/project_test.dart:853
_testInMemory
test/general.shard/project_test.dart:790
main.<fn>.<fn>
test/general.shard/project_test.dart:35
package:test_api                                             group
main.<fn>
test/general.shard/project_test.dart:34
package:test_api                                             group
main
test/general.shard/project_test.dart:33
2

✖ loading test/general.shard/project_test.dart
Exited (1)

Originally posted by @Abhishek01039 in https://github.com/flutter/flutter/issues/81106#issuecomment-846368729

We could either mock out that specific functionality, or do some sort of fake for the test. IIRC copying the gradle_wrapper is not really part of what is under test here


transpile to ES5

I also added the build step as a "prepublishOnly" npm script so it will auto-build on npm publish

No good reason.  I just googled "Babel and Rollup" and found https://github.com/rollup/rollup-plugin-babel/blob/master/README.md
I don't have any preference on how Babel is configured as long as the UMD ends up as ES5.

For the record, the readme I linked to says

Babel will respect .babelrc files – this is generally the best place to put your configuration.

And also

Since Babel 6.3 it's possible to deactivate module transformation with "modules": false. So there is no need to use the old workaround with babel-preset-es2015-rollup

So the changes you're suggesting go against the suggestions of this readme.

Oh, and lastly, babel-preset-es2015-rollup is based on http://babeljs.io/docs/plugins/preset-es2015/ and according to those docs,

This is deprecated. If you want to stay up to date, use the env preset

Cool, I'm convinced!
Now I wonder, since babel-core, babel-plugin-external-helpers and babel-preset-env are dependencies of rollup-plugin-babel, shouldn't we ignore those?

nah.  They're dev dependencies https://github.com/rollup/rollup-plugin-babel/blob/master/package.json#L43-L47, so they don't get automatically installed when we install rollup-plugin-babel.  They have to be added separately.

Thank you!


transliteration result for "ティ" and "てぃ"

"シティ" and "してぃ" is convereted to "SHITHI"
シ= SHI is fine
However
Japanese people cannot read "THI".  Some people may be able to read/imagine though.
Please find the proper conversions below (excerpted from http://xembho.s59.xrea.com/siryoo/roomazi_no_hyoo.html)
t	ティ	トゥ
ti	tu
d	ディ	ドゥ
di	du
w	ウィ	ウェ	ウォ
wi	we	wo
Thank you.

The new version 1.1.0 of kuroshiro imports the feature of romanization system and now supports three different systems: nippon-shiki, passport-shiki, hepburn-shiki, which might solve this issue. Have a try. 😉


v-link don't work in vue-google-maps

Vue 0.7, v-link in info-window in vue-gmail-map not working. This generated simple href and reload page. Please help..

Hi, thanks for filling this issue, however the issue tracker is reserved for bug reports and feature requests only, so please use our dedicated gitter or forum to ask questions. Thanks!


reactor errors upon backend terminating connection

Seen on d13f8ff5381f83a3521fc5067f5f310eba73fe73

2011-11-10 15:18:30,171 ERROR twisted.internet.epollreactor : Unhandled Error
Traceback (most recent call last):
  File "/usr/lib64/python2.6/site-packages/twisted/python/log.py", line 84, in callWithLogger
    return callWithContext({"system": lp}, func, *args, **kw)
  File "/usr/lib64/python2.6/site-packages/twisted/python/log.py", line 69, in callWithContext
    return context.call({ILogContext: newCtx}, func, *args, **kw)
  File "/usr/lib64/python2.6/site-packages/twisted/python/context.py", line 118, in callWithContext
    return self.currentContext().callWithContext(ctx, func, *args, **kw)
  File "/usr/lib64/python2.6/site-packages/twisted/python/context.py", line 81, in callWithContext
    return func(*args,**kw)
---  ---
  File "/usr/lib64/python2.6/site-packages/twisted/internet/epollreactor.py", line 217, in _doReadOrWrite
    why = selectable.doRead()
  File "/srv/code/products/rbuilder/x/include/txpostgres/txpostgres.py", line 556, in doRead
    _PollingMixin.doRead(self)
  File "/srv/code/products/rbuilder/x/include/txpostgres/txpostgres.py", line 136, in doRead
    self.continuePolling()
  File "/srv/code/products/rbuilder/x/include/txpostgres/txpostgres.py", line 108, in continuePolling
    state = self.pollable().poll()
psycopg2.OperationalError: server closed the connection unexpectedly
	This probably means the server terminated abnormally
	before or while processing the request.

2011-11-10 15:18:30,184 ERROR twisted.internet.epollreactor : Unhandled Error
Traceback (most recent call last):
  File "/srv/code/products/rbuilder/x/include/rmake3/lib/daemon.py", line 357, in doWork
    self.reactor.run()
  File "/usr/lib64/python2.6/site-packages/twisted/internet/base.py", line 1162, in run
    self.mainLoop()
  File "/usr/lib64/python2.6/site-packages/twisted/internet/base.py", line 1174, in mainLoop
    self.doIteration(t)
  File "/usr/lib64/python2.6/site-packages/twisted/internet/epollreactor.py", line 197, in doPoll
    log.callWithLogger(selectable, _drdw, selectable, fd, event)
---  ---
  File "/usr/lib64/python2.6/site-packages/twisted/python/log.py", line 84, in callWithLogger
    return callWithContext({"system": lp}, func, *args, **kw)
  File "/usr/lib64/python2.6/site-packages/twisted/python/log.py", line 69, in callWithContext
    return context.call({ILogContext: newCtx}, func, *args, **kw)
  File "/usr/lib64/python2.6/site-packages/twisted/python/context.py", line 118, in callWithContext
    return self.currentContext().callWithContext(ctx, func, *args, **kw)
  File "/usr/lib64/python2.6/site-packages/twisted/python/context.py", line 81, in callWithContext
    return func(*args,**kw)
  File "/usr/lib64/python2.6/site-packages/twisted/internet/epollreactor.py", line 230, in _doReadOrWrite
    self._disconnectSelectable(selectable, why, inRead)
  File "/usr/lib64/python2.6/site-packages/twisted/internet/posixbase.py", line 256, in _disconnectSelectable
    selectable.connectionLost(failure.Failure(why))
  File "/srv/code/products/rbuilder/x/include/txpostgres/txpostgres.py", line 170, in connectionLost
    self.continuePolling()
  File "/srv/code/products/rbuilder/x/include/txpostgres/txpostgres.py", line 108, in continuePolling
    state = self.pollable().poll()
psycopg2.OperationalError: connection not open

I managed to prevent continuePolling from causing a second traceback and I think one traceback has to be displayed when the backend gets terminated, otherwise you'd never know something happened.
I'm not 100% happy with the solution, but I think it's good enough for now, so I'll close this issue. Please reopen if you feel more things can be improved to make writing a reconnectable Connection class easier.


Mobile warning is shown to Firefox desktop users

Steps taken:

Visit https://v4nn4.github.io/glyphs-generator/ with Firefox on a desktop device.

Expected result:

See the glyph generator demo.

Actual result:

A mobile warning is shown:

Please switch to a desktop device for full website functionality.

Technical details:
The Regular expressions for mobile detection in /main/docs/script.js#L177 is incorrect. The part |gecko\/| does not only detect Firefox mobile but also Firefox on desktop devices. Here is an overview of current Firefox useragents:
https://www.whatismybrowser.com/guides/the-latest-user-agent/firefox
As an example: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:125.0) Gecko/20100101 Firefox/125.0

Thanks a lot for spotting this! I updated the script according to https://stackoverflow.com/questions/11381673/detecting-a-mobile-browser. It is fixed in https://github.com/v4nn4/glyphs-generator/commit/94305c70b8e8e64457c2de0b71aafa4f84c31b04.


PortraitController.Show tries to access a null portrait

When called, the method Show(Character character, string position) will create a new PortraitOptions variable with default values, and later will try to read the "portrait" property of this variable. Since the portrait is never properly set in this method, it will always error on the following two lines:
options.character.State.SetPortraitImageBySprite(options.portrait);
options.character.State.portraitImage.rectTransform.gameObject.SetActive(true);

Changing this Show method to require a string with the name of the portrait, and then setting the "portrait" property of the PortraitOptions variable fixes the issue. This is what the method looks like when fixed:
public virtual void Show(Character character, string portrait, string position)
        {
            PortraitOptions options = new PortraitOptions(true);
            options.character = character;
            options.portrait = character.GetPortrait(portrait);
            options.fromPosition = options.toPosition = stage.GetPosition(position);

            Show(options);
        }

I thought about submitting a pull request, but I don't know if this method is used by something else, and didn't want to risk breaking someone else's project by changing the API

What you're trying to achieve exactly?
Getting specific portrait can be done with just character.State.portraitImage(nameOfTheSprite);

I'm trying to display a character at a certain position on the stage. The character was created properly (with a portrait), and I tried to use the Show method to make the character appear, but the way it is, the method tries to access an invalid portrait. I edited the Show method locally to make it set the portrait according to the character's default character, and it's working properly


Comment: Check if comment list respects paging

Enable Query stuff over comments; also enable paging in lists.

Fixed.


add globalhub inspect

Related Issue:  stolostron/backlog#<ISSUE_NUMBER>
https://issues.redhat.com/browse/ACM-9153
Description of Changes:
the globalhub namespace in incorrect
What resource is being added:  | N/A
Is this a Hub or Managed cluster change?:
Hub Cluster | Managed Cluster | N/A
Notes:

/assign @clyang82

Do you want to keep consistent with main branch? or you want to address my comments?

yes, keep the same behavior with main branch

/lgtm


DesktopTextSelectionToolbar doesn't support keyboard navigation

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Run the code sample on a desktop platform.
Type some text, select it, and right-click to open the desktop text selection toolbar.
Try to navigate the toolbar with a keyboard. e.g. press the up/down arrow keys.

Expected results
Be able to navigate the desktop text selection toolbar with a keyboard.
Actual results
When trying to navigate with a keyboard, the toolbar changes, and is unable to navigate with a keyboard.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      theme: ThemeData(useMaterial3: true),
      home: const Example(),
    );
  }
}

class Example extends StatelessWidget {
  const Example({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('DesktopTextSelectionToolbar'),
      ),
      body: const Center(
        child: TextField(
          style: TextStyle(fontSize: 24.0),
        ),
      ),
    );
  }
}

Screenshots or Video
macOS (Keyboard navigation when hovering the toolbar)
https://user-images.githubusercontent.com/48603081/234251221-27e1069c-6048-4e3e-b59c-2d3e49a798e2.mov
Flutter (Keyboard navigation when not hovered)
https://user-images.githubusercontent.com/48603081/234251250-405298b1-c27a-4af0-a55c-9bd86eb6fa33.mov
macOS (Keyboard navigation when not hovering the toolbar)
https://user-images.githubusercontent.com/48603081/234251337-aa91ec3c-1cff-40b1-9808-86c313927da1.mov
Flutter (Keyboard navigation when not hovering the toolbar)
https://user-images.githubusercontent.com/48603081/234251394-003f22d2-c66e-4b87-9c1f-8a7773371da6.mov
Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[!] Flutter (Channel master, 3.10.0-12.0.pre.50, on macOS 13.3.1 22E261
    darwin-arm64, locale en-EE)
    • Flutter version 3.10.0-12.0.pre.50 on channel master at
      /Users/tahatesser/Code/flutter
    ! Upstream repository git@github.com:TahaTesser/flutter.git is not the same
      as FLUTTER_GIT_URL
    • FLUTTER_GIT_URL = git@github.com:NevercodeHQ/flutter.git
    • Framework revision 8395a2b2a1 (3 hours ago), 2023-04-25 04:00:07 -0400
    • Engine revision a26a479e86
    • Dart version 3.1.0 (build 3.1.0-38.0.dev)
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/tahatesser/Code/android-sdk
    • Platform android-33, build-tools 33.0.2
    • ANDROID_SDK_ROOT = /Users/tahatesser/Code/android-sdk
    • Java binary at: /Applications/Android Studio
      Preview.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b653.34-9746777)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b653.34-9746777)

[✓] VS Code (version 1.77.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.3.1 22E261
      darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 112.0.5615.137

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

! Doctor found issues in 1 category.

Thanks for the detailed report, Taha! This issue is reproducible on the latest stable and master channels.
It would be great if it also supports navigating to boundary items (top/bottom) by compose keys (Cmd + Up/Down) as well.

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.7.12, on macOS 13.0.1 22A400 darwin-x64, locale en-VN)
    • Flutter version 3.7.12 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (2 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.77.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (3 available)
    • SM T225 (mobile) • R9JT3004VRJ • android-arm64  • Android 13 (API 33)
    • macOS (desktop)  • macos       • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)     • chrome      • web-javascript • Google Chrome 112.0.5615.137

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[!] Flutter (Channel master, 3.10.0-12.0.pre.55, on macOS 13.0.1 22A400 darwin-x64, locale en-VN)
    • Flutter version 3.10.0-12.0.pre.55 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision aad9fd2d77 (19 minutes ago), 2023-04-25 07:53:24 -0400
    • Engine revision 54ddef68b1
    • Dart version 3.1.0 (build 3.1.0-39.0.dev)
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio Flamingo.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio Flamingo.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.77.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (4 available)
    • SM T225 (mobile) • R9JT3004VRJ                              • android-arm64  • Android 13 (API 33)
    • iPhone (mobile)  • d9a94afe2b649fef56ba0bfeb052f0f2a7dae95e • ios            • iOS 15.7.2 19H218
    • macOS (desktop)  • macos                                    • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)     • chrome                                   • web-javascript • Google Chrome 112.0.5615.137

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

! Doctor found issues in 1 category.

cc @justinmc from triage.

@gspencergoog Weren't we talking about this at some point?  I remember brainstorming with you how to keep a TextField focused while some buttons also have focus.  Do you know if that's currently possible?

@gspencergoog That seems to be exactly what I need, but I just tried it and the problem I'm having is that Shortcuts traversals always seem to start from the parent Focus.

 In this app, if I tap the button and press "Z", the top CallbackShortcuts is always triggered
CallbackShortcuts(
  bindings: <ShortcutActivator, VoidCallback>{
    const SingleActivator(LogicalKeyboardKey.keyZ): () {
      print('justin Z: top'); // always logged
    },
  },
  child: Focus(
    focusNode: focusNodeTop,
    child: CallbackShortcuts(
      bindings: <ShortcutActivator, VoidCallback>{
        const SingleActivator(LogicalKeyboardKey.keyZ): () {
          print('justin Z: bottom'); // never logged
        },
      },
      child: Focus(
        parentNode: focusNodeTop, // If I remove this line, the bottom one is logged instead.
        focusNode: focusNodeBottom,
        child: TextButton(
          onPressed: () {
            focusNodeBottom.requestFocus();
          },
          child: const Text('clickme'),
        ),
      ),
    ),
  ),
),

This is a simplified version of the problem at hand, where the top Focus would be the EditableText and the bottom would be the context menu.
Is this desired or a bug?  I want to be able to receive key events from the child Focus.

Yeah, because by specifying the parent node, you're bypassing the focus node that is embedded in the middle Shortcuts widget.  You've restructured the focus tree so that it is a second leaf node under the focusTopNode node.
Try printing the focus tree with debugDumpFocus() and you'll see what I mean.

What you probably want to do is wrap the subtree with a Focus widget that has the parentNode set, but is unfocusable and not traversable, to act as a bridge to the parent. Then all the children will find that Focus widget and parent from it.


Colors Not Being Set Correctly in iOS 11

Problem/Motivation

I have a Aeotec z-wave led bulb and in iOS 10, when I would ask Siri to set a particular color, it would be done. After updating to iOS 11, it doesn’t work anymore. Works normally through Home Assistant.

Expected behavior

I try to set the color (let’s say red), the bulb is set to the asked color (Home assistant says “rgb(255,0,0)”)

Actual behavior

I try to set the color (let’s say red), the bulb is set to on but no color (Home assistant says “rgb(0,0,0)”). This happens when asking Siri or attempting to set with HomeKit’s color wheel.

Proposed changes

I’m not sure if this is an issue with Home Bridge (I don’t have other bulbs to test) or an issue with HomeKit in iOS 11 (os is has several bugs as is, but since this is t an officially supported platform, I doubt I’ll get far with Apple)

Removed My Home, cleared Persist, and restarted HomeBridge. I then added back Home Assistant to  HomeKit and the issue resolved itself. Closing this issue.


[REQUEST] support Django 4.0 and/or DEFAULT_AUTO_FIELD set to django.db.models.BigAutoField

Is there an existing request for this?

[X] I have searched the existing Issues

Current problem or need
I'm trying to bring back to life a personal mezzanine site that was running a few years ago, and while I'm doing it I'm updating it's dependencies to the current versions.
While trying to get this site running locally on a development system I've found the "python manage.py migrate" was warning about missing migrations:
Your models in app(s): 'blog', 'conf', 'core', 'forms', 'galleries', 'generic', 'pages' have changes that are not yet reflected in a migration, and so won't be applied.
  Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.

If I ran python manage.py makemigrations blog then a 0004_auto_20220626_0447.py file was created in the virtualenv site-packages for mezzanine/blog/migrations. Now I'm not an experienced django developer, but that doesn't seem correct.
I found this to occur with django 4.0.6. I found this to occur if I had django 3.2.13 and if I also had DEFAULT_AUTO_FIELD = "django.db.models.BigAutoField".
I've been able to avoid this happening if I use django 3.2.13 and set DEFAULT_AUTO_FIELD = "django.db.models.AutoField"
Desired solution
I guess my desired solution is django 4.0 support, but I can see #2009 was done 6 months ago, so I'm not sure why I'm seeing this, but other people seem to have had success with django 4.0?
I wonder if a solution would be to have someone create migrations within mezzanine with DEFAULT_AUTO_FIELD set to django.db.models.BigAutoField, so that the next release of mezzanine includes these migrations. But I have no idea what that does to backwards compatibility.
Alternatives you have considered
No response
Anything else?
I considered this might actually be my fault, so I used mezzanine-project to create a new project. That project was showing the same behaviour. Here is the output of that:
cooke:~ geoffc$ mezzanine-project test_mezzanine_project
cooke:~ geoffc$ cd test_mezzanine_project/
cooke:test_mezzanine_project geoffc$ python manage.py migrate
Operations to perform:
  Apply all migrations: admin, auth, blog, conf, contenttypes, core, django_comments, forms, galleries, generic, pages, redirects, sessions, sites
Running migrations:
  Applying contenttypes.0001_initial... OK
  Applying auth.0001_initial... OK
  Applying admin.0001_initial... OK
  Applying admin.0002_logentry_remove_auto_add... OK
  Applying admin.0003_logentry_add_action_flag_choices... OK
  Applying contenttypes.0002_remove_content_type_name... OK
  Applying auth.0002_alter_permission_name_max_length... OK
  Applying auth.0003_alter_user_email_max_length... OK
  Applying auth.0004_alter_user_username_opts... OK
  Applying auth.0005_alter_user_last_login_null... OK
  Applying auth.0006_require_contenttypes_0002... OK
  Applying auth.0007_alter_validators_add_error_messages... OK
  Applying auth.0008_alter_user_username_max_length... OK
  Applying auth.0009_alter_user_last_name_max_length... OK
  Applying auth.0010_alter_group_name_max_length... OK
  Applying auth.0011_update_proxy_permissions... OK
  Applying auth.0012_alter_user_first_name_max_length... OK
  Applying sites.0001_initial... OK
  Applying blog.0001_initial... OK
  Applying blog.0002_auto_20150527_1555... OK
  Applying blog.0003_auto_20170411_0504... OK
  Applying conf.0001_initial... OK
  Applying core.0001_initial... OK
  Applying core.0002_auto_20150414_2140... OK
  Applying django_comments.0001_initial... OK
  Applying django_comments.0002_update_user_email_field_length... OK
  Applying django_comments.0003_add_submit_date_index... OK
  Applying django_comments.0004_add_object_pk_is_removed_index... OK
  Applying pages.0001_initial... OK
  Applying forms.0001_initial... OK
  Applying forms.0002_auto_20141227_0224... OK
  Applying forms.0003_emailfield... OK
  Applying forms.0004_auto_20150517_0510... OK
  Applying forms.0005_auto_20151026_1600... OK
  Applying forms.0006_auto_20170425_2225... OK
  Applying galleries.0001_initial... OK
  Applying galleries.0002_auto_20141227_0224... OK
  Applying generic.0001_initial... OK
  Applying generic.0002_auto_20141227_0224... OK
  Applying generic.0003_auto_20170411_0504... OK
  Applying pages.0002_auto_20141227_0224... OK
  Applying pages.0003_auto_20150527_1555... OK
  Applying pages.0004_auto_20170411_0504... OK
  Applying redirects.0001_initial... OK
  Applying redirects.0002_alter_redirect_new_path_help_text... OK
  Applying sessions.0001_initial... OK
  Applying sites.0002_alter_domain_unique... OK
cooke:test_mezzanine_project geoffc$ python manage.py migrate
Operations to perform:
  Apply all migrations: admin, auth, blog, conf, contenttypes, core, django_comments, forms, galleries, generic, pages, redirects, sessions, sites
Running migrations:
  No migrations to apply.
  Your models in app(s): 'blog', 'conf', 'core', 'forms', 'galleries', 'generic', 'pages' have changes that are not yet reflected in a migration, and so won't be applied.
  Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.
cooke:test_mezzanine_project geoffc$ python manage.py makemigrations blog
Migrations for 'blog':
  /Users/geoffc/.pyenv/versions/3.10.4/envs/geoff-blog/lib/python3.10/site-packages/mezzanine/blog/migrations/0004_auto_20220626_0509.py
    - Alter field id on blogcategory
    - Alter field id on blogpost

hi, any news?


xDS interop: set default socket timeout to 60

kubernetes library does not provide a way to configure the default socket timeout that will be used with urllib3 it uses under the hood. And urllib3 default socket timeout is infinity.
This PR sets the default socket timeout using python's socket.setdefaulttimeout() to 60 seconds.
This affects urllib3 directly,  and therefore kubernetes.
The changes is also picked up by the google-api-python-client, which does not use urllib3 (it uses httplib2), but respectes socket.setdefaulttimeout().
ref b/240068800

grpc/core/master/linux/psm-security
grpc/core/master/linux/grpc_xds_k8s_lb
grpc/core/master/linux/grpc_xds_url_map


binary search C++ code

Fixes #{ISSUE_NUMBER}
By submitting this pull request I confirm I've read and complied with the below declarations.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[x] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.
[x] This pull request shall only be reviewed and merged once the Travis build passes. No maintainer or supporter shall be obliged to review it before this condition is met.
[x] I have mentioned the issue number correctly (with hyperlink) in this pull request description.

add a line at the end.

Which line @theyashshahs. Please elaborate


Add support for more nuanced envoy-proxy CORS origins settings

Please describe your use case / problem.
envoy-proxy allows for more control over the CORS allowed origins than is exposed by ambassador. For instance, they allow for prefix, suffix and regex matching, not just exact matching.
See here and here for details.
Describe the solution you'd like
I'd like to be able to take advantage of the functionality provided by envoy-proxy that is currently masked by Ambassador.
For instance, one may have an API service fronted by Ambassador (api.example.com) and several peer or subdomains that wish to access it (www.example.com, feature-branch-1.dev.example.com, etc.). Allowing suffix matching would solve this handily, especially since the CORS spec disallows hardcoded wildcards other than "*" and even that has its own issues when working with credentialed requests.
Describe alternatives you've considered
I suppose one could forego the Ambassador CORS configuration and do a heavy-handed lua script to replicate this functionality, but that seems less than ideal. I don't know if there's a way to do an end-run around Ambassador to set these settings directly on envoy-proxy itself, but that also seems needlessly cumbersome when we could pass them through Ambassador.
Additional context
I have taken a stab at updating the v2/Mapping JSONSchema (backwards compatible, I believe) to accept the documented value types and have updated ircors.py to process the new values. See that work here. I don't know how to test this yet and I'm sure there's a fair amount of effort involved in updating docs and test cases. I'm just hoping to get this ball rolling. Feel free to riff on that commit if it proves useful. I wouldn't mind helping out with updating the docs.
I found this issue #3020 shortly after creating this one. Sorry for the spam, but it looks like I'm not the only one looking for this feature.

I think this is still relevant


Error zone.js in ie11

Hello excellent project, as the problem that solves that is to say, to be seen separately, tell me that this error is to run the application separately, I think by the zone.js

It should be mentioned that it is the application of angular separately, thanks for tu suggestions

Apologies for the slow response here -- this looks like it is related to https://github.com/angular/zone.js/issues/1031.

the best solution to deal with zone.js issue so far is
1, comment out import 'zone.js/dist/zone'; from the pollyfill.ts
2, add 	 into the index.html
since angular will compile zone.js into the bundle so this way the spa framework won't load zone.js from the child, and only the shell has one zone.js file acting.

This project is deprecated (see #36). Use single-spa-angular@3, instead.


Class VAE()

Hello, I would like to ask you if the Class VAE() in your code is used to extract 10 features and add them to the original features? I look forward to hearing from you!

Yes, I try to use the VAE to extract the latent space(10d) which can represent all the features.


FAQ-Q2

FAQ-Q2
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: FAQ_Q2

[x] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66615db686958d5971b9ae74--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66616aad87390868070d0c6f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://666186948052cb007ef71e21--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6661f6347003913794db3ce6--harness-developer.netlify.app


forwarder: log each batch write at debug level not info

Logging each batch write at info level is too verbose.

Lgtm

+1


Add Vector Search Samples and Tests

Description
Adds samples and tests for Cognitive Search's Vector search.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - search - tests


docs: need to document kargo verify command

This could probably be added to the Verifications section of docs/docs/15-concepts.md

@krancour sir i will solve this issue.

Thank you @Ayush9026!

Hi @fykaa ,
Thanks for following up! You can take up this issue if you'd like.


Increment versions for applicationinsights releases

Increment package versions for applicationinsights releases

API change check
API changes are not detected in this pull request.


00-setup-environment.md

Source:
Ideally, you should complete these labs in a hosted lab environment.
Current Translation:
ホストされているラボ環境でこれらのラボを完了するのが理想的です。
Suggested Translation:
これらのラボは、ホストされているラボ環境で実行することをお勧めします。

Fixed

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team


Using Yolov8 and Anomalib with MLX

I am currently working on computer vision and anomaly detection projects and was wondering how would I use yolov8 and anomalib with the MLX framework?
The plan is to use Mac Minis as edge computers to handle all of the image processing.
I'd like to add I am new to using github issues and hope this is the right place and right way to ask my question.
Any and all guidance is appreciated.
Thank you in advance.

I guess you cannot use both at the moment with MLX. You would need to re-implement / port the models you need over to MLX.

@AhmedHamadto Maybe you can wait until those pooling layers are released before implementing your version of yolov8 on MLX. :>

@NeptuneIsTheBest so basically once they release the layers you're suggesting I rebuild yolov8 from the ground up on MLX? Sounds interesting.

Hi, I'm trying to implement yolov5 with MLX.
So, I already made some blocks as : ConvBlock, C3f, Bottleneck and SPPF (their names in ultralytics repo).
But yes, MaxPooling is not implemented and I'm facing the same issue so I will code it using Python (I don't have C++ knwoledge to implement it).
Let me know If you want to collaborate on this ;)

Hi, I'm trying to implement yolov5 with MLX.
So, I already made some blocks as : ConvBlock, C3f, Bottleneck and SPPF (their names in ultralytics repo). But yes, MaxPooling is not implemented and I'm facing the same issue so I will code it using Python (I don't have C++ knwoledge to implement it).
Let me know If you want to collaborate on this ;)

I would be quite interested in that, I have a few colleagues whom have also expressed interest, I think a collaboration on this would be genuinely fruitful.

Professor @menzHSE , if you're interested in joining us it would be an honour and pleasure to have you onboard.

I have to pass on that, sorry. But I will follow your progress and am happy to try out and test your implementation.

@NeptuneIsTheBest so basically once they release the layers you're suggesting I rebuild yolov8 from the ground up on MLX? Sounds interesting.

@AhmedHamadto Hello, because I think this can reduce duplication of work. You don't need to make your code change frequently and just focus on high-level implementation. If I can help you, I hope to join you (because I also want to implement the MLX version of yolov7). It would be a great honor for me to be a part of your project. :>


Accessibility issue found using react-a11y

I have been using the NPM module react-a11y to test my project for accessibility. Following is the warning it is throwing:
TabLink You have a click handler on a non-interactive element but no `role` DOM property. It will be unclear what this element is supposed to do to a screen-reader user. http://www.w3.org/TR/wai-aria/roles#role_definitions

I get 10 of these warning even though I have only 1 tab in my project (eventually I will be having more).

How do I fix this?

You can either pass role="button" to each <TabLink> component and see if it helps, or (preferably :) submit a PR to add role="button" and aria-pressed (docs) so it's handled on this package's level.

I was able to fix this by setting role="button" and the onKeyDown event handler on the <TabLink>. Thank you.


[FR]: Increase API timeout

Is your feature request related to a problem? Please elaborate.
don't know if it is in your control or hard coded in the qbit API but is it possible to increase the timeout.
When my qbit instances reach around 10K torrents, I noticed that qbm often fails to connect due to API errors.
Now during the summer months with fewer releases it's less of an issue, but starting September it will be an issue again to keep the total number of torrent well below 10K.
Discord discussion
https://discord.com/channels/764440599066574859/921548253948964904/1121132898838380626
https://discord.com/channels/764440599066574859/921548253948964904/1121133384488456242
Describe the solution you'd like
If possible, make the timeout configurable or increase it
Does your solution involve any of the following?

[ ] New config option
[ ] New command option

Describe alternatives you've considered
Keep the number of torrents well below 10K
Who will this benefit?
everyone with a large number of torrents
Additional Information
No response

looks like just need to update https://github.com/StuffAnThings/qbit_manage/blob/ee871b4f928ec1bab2e7940a01a1d1e1e2a5883c/modules/qbittorrent.py#L198
based on qbit api
https://qbittorrent-api.readthedocs.io/en/latest/behavior%26configuration.html#requests-configuration


Various screenshots too small

Check Apache Wave and Telescope Nova. They are not readable.

"Let's Chat" image is also too small.

I meant to say that if this was an app market issue, that you couldn't make screenshots larger, you would report it here. Apps choosing to offer small screenshots is not a bug with the app market. You could potentially visit the apps' source links (found in the app market) and file issues requesting larger screenshots, though I know there is actually a file size limit on metadata, and metadata increases the size of the app package itself, so some may have chosen a small screenshot for those reasons.

You may feel that it is not an issue caused by sandstorm app market. But, it does affect overall user experience of app market. Maybe you should include it as a requirement in app submission guideline that all screenshots should be readable or big enough. I assume you would have some sort of quality metrics before accepting submission to app store.

It's actually a mild irritation to me that some apps don't even have screenshots. ;)
AFAIK, the Sandstorm team does need to accept app updates, but beyond the technical requirement that they work, aren't particularly demanding. As an open source platform, it's questionable what level of requirements the maintainers should have.
As a note to the use of the pronoun 'you', I do want to clarify that I am not a Sandstorm employee and do not speak for them. I just try to help where I can and provide answers, advice, suggestions, nitpicks, and very, very rarely, a little code.

If there are issues with file size limitation, to keep app package small, maybe the screen shot images should be kept as a reference URL instead of putting images in package.
App market is now large enough where a little bit of quality control might help especially since there are several apps offering similar functionality or in the same category.
As a long time open source engineer, I have always felt that one problem with open source is that there are too many options with varying levels of maturity and it is a lot of work to sift through lots of apps in the same category. Reviews help and similarly good screen-shots help.

Hi @paragjoshi ,
I totally agree that overall user experience of the app market is improved by great screenshots. I wholeheartedly agree with you that people judge Sandstorm on the quality of the apps, and they judge the quality of the apps via the screenshots before even installing the apps.
Furthermore, I think that it's a growth problem for the Sandstorm community that app screenshots are only modifiable by the app packager. There's a balance to be struck between "app authors have control over their apps" (i.e. via key signing the entire app package) and "the community can easily participate in improving apps", and I don't think we've struck the perfect balance there yet.
There's only a handful of us on the core team, and having improved screenshots is definitely an area where we could use help. If you want to help, I'd be honored to make that a reality.
There are two ways to do that -- the easy way for you, and the easy way for me. :)
Easy way for you
Take screenshots of the apps at better resolutions, with some sample text in them, and leave them as an attachment here. I can work on delivering them as pull requests to the apps.
Easy way for me
App screenshots are embedded into the app package, typically in the .sandstorm/ directory,  but always find-able via the sandstorm-pkgdef.capnp file. Most app authors are reasonably happy to accept screenshot modification pull requests.
For example, for Let's Chat, this is the link to the sandstorm-pkgdef.capnp:

https://github.com/jparyani/lets-chat/blob/sandstorm/sandstorm-pkgdef.capnp

On this line --https://github.com/jparyani/lets-chat/blob/sandstorm/sandstorm-pkgdef.capnp#L42 -- you can see the icon PNG files.
Therefore they live in the app-graphics directory.
If you submit a pull request to that app modifying the files in that directory, I'm pretty sure @jparyani would be very happy to merge it, and would fairly promptly update the app in the app market.
Let me know what you think. Thanks for filing this bug, @paragjoshi . I really do appreciate it; getting feedback from open source fans like yourself means that the Sandstorm project is reaching some of the right people.
I'll leave this ticket open until there's a clear plan for improving the screenshots of:

Apache Wave

Telescope Nova

Let's Chat


Add codesandbox links in docs

Ideally our storybook stories would have corresponding codesandbox links to let users open and experiment freely.

 Code Sandbox for Docs