Mulit Sector Expansion

This PR includes:

change from techs.csv -> techs.yml in order to be more flexible with a tree instead of table structure
multiple carriers (like electricity, hydrogen...) can be used and an energy balance is calculated for each carrier
demand is a technology to allow different sources of demand of the different carriers
storage is simplified and split into storage that really stores something and conversion that converts from one carrier to another
:GEN and :FLOW are sparse Variables to reflect that not every technology has a generation or flow in all carriers
run_opt configuration is more general to allow lost-load, lost-emission and emission-limits for different carriers and impacts
sets are defined as set[dimension][tech_group] to simplify the different set-groups needed for modular setup
change of dependencies ClustForOpt.jl -> TimeSeriesClustering.jl
issue #20 handled
CodeCov included

These look like really useful additions, I look forward to having a look. Will do so in the coming days.
In the meantime, do you think it would make sense to update master with the current dev branch, potentially enhanced by a dependency on TimeSeriesClustering instead of ClustForOpt?
#24 was closed, so it could be useful to have a compatible version in master already.

These look like really useful additions, I look forward to having a look. Will do so in the coming days.
In the meantime, do you think it would make sense to update master with the current dev branch, potentially enhanced by a dependency on TimeSeriesClustering instead of ClustForOpt?
#24 was closed, so it could be useful to have a compatible version in master already.

This will cause merge problems, but I addressed the request in a new PR

Thank you, that sounds great. I'll revisit this PR over the coming days, it has lots of cool changes and I want to make sure I take the time that I follow how they work.

One question that we did not address yet:

Do the old tests still work on the new version (Was it possible to use the old JLD2 file with the new interface)? It would be great to compare results without recomputation of the tests, because there are a lot of changes.
On that note: It seems that the jld2 was recalculated in the last step. Is there any way to compare at least a subset of the tests to the old version?


You can change the coordination server on the android client

https://github.com/tailscale/tailscale-android/commit/d8aedf721ae17326561fa12b2699bdb19cc677f9
https://github.com/tailscale/tailscale-android/commit/b1501eb5d8e75bfb94f85ea7ea14bc02964d84d6
Update the documentation on the readme and maybe add some direction on how to enable.

We have merged this in https://github.com/juanfont/headscale/pull/703


MB-5490 Backend changes for CANCELLATION_REQUESTED shipment status

Description
This PR implements the backend changes needed to accommodate the new CANCELLATION_REQUESTED shipment status. It includes:

A migration to add CANCELLATION_REQUESTED to the mto_shipment_status enum
Updating the UpdateMTOShipmentStatus service object to handle the new logic for this status
Adding tests to both the UpdateMTOShipmentStatus service object and the GHC API PatchMTOShipmentStatus handler function
Updating the ghc.yaml file to include the new status in the MTOShipmentStatus definition, and sharing this definition across other shipment payloads to eliminate redundancy

Setup
To test this endpoint, you'll need to set up Postman or another REST API client to talk to the GHC API. You can do so by following the gorgeous instructions outlined here: Test Admin and Office APIs with Postman.
Once you are all set up to hit the GHC API, follow these steps to test the endpoint:

Start your dev environment and your client:

make server_run
make office_client_run

Go to the Office Interface and click "Local Sign-In". Select the TOO role to sign in.

In the queue, find a move with a shipment.

If you cannot find a move with a shipment, you may need to create one using the Customer Interface, the Support API, or by hacking your way through the database.

Check the state of the shipment. If it is SUBMITTED, please click approve it using this interface. This will mark both the move and the shipment as "APPROVED."

If it is already APPROVED, you have nothing to do here.

Once you have APPROVED the shipment, open Postman or whatever REST API Client you set up at the beginning.

You'll have the locator code for this move from the interface/URL, but you'll need the IDs of the move and the shipment to test this endpoint.
To get the ID for the move, go the getMove endpoint and put the locator in the URL. You should get a response that includes the ID for the move. Grab this value.

Navigate to the listMTOShipment endpoint. Enter the move ID in the <moveTaskOrderID> param in the URL. Submit the request, find your approved shipment in the output, and grab this shipment's id and eTag values.

Now go to the patchMTOShipmentStatus endpoint. Put the move and the shipment IDs in the path parameters, and put the eTag value in the If-Match header.

You'll also need to set your X-CSRF-Token header (please refer to Test Admin and Office APIs with Postman again for more details).
Use the following as the request body:

{
  "status": "CANCELLATION_REQUESTED"
}

Submit the request and verify that you get a 200 response code.

If you have a DB viewer set up, go ahead and take a peek in the webhook_notifications table and verify that you see an entry with the CANCELLATION_REQUESTED status.

Code Review Verification Steps

Any new migrations/schema changes:

[x] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database

[ ] Request review from a member of a different team.
[x] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change.

Fails

:no_entry_sign:
Yarn Audit Issues Found:
high - Regular Expression Denial of Service
Package three
Patched in >=0.125.0
Dependency of @trussworks/react-file-viewer
Path @trussworks/react-file-viewer > three
More info https://npmjs.com/advisories/1639
1 high vulnerabilities and 0 critical vulnerabilities found

Messages

:book:
:link: MB-5490

  Generated by :no_entry_sign: dangerJS against 68fd6cba599a9e00e3e5e5b4d1920d7b988e1de3


NO-JIRA: Fixes to the CSV, Godoc

This PR proposes minor fixes to adjust how the operator's pages are shown in the OperatorHub sections of the console.

Improved Golang doc for the CRD
Improved description of the operator and operand
Removed some labels from the CR (@Prashanth684 we were unsure of this, but checked other operators and they don't ship those labels either)
Added an annotation to declare the minimum openshift version we can install this operator on: unfortunately, this needed an hack in the Makefile bundle target as operator-sdk doesn't consider custom annotations in the bundle/metadata/annotations.yaml file (xref threads in the internal slack)
Adds suggested namespace to deploy the operator in the default openshift-multiarch-tuning-operator namespace.

Finally, when deploying the operator through OLM or OperatorHub in a namespace different than the default one (openshift-multiarch-tuning-operator), the operand's serviceaccount are not bound to the expected Roles and ClusterRoles properly. This is because OLM doesn't patch the subject's namespace in the ClusterRoleBinding and RoleBinding manifests shipped by the bundle.
I like neither this solution, nor alternatives:

The operator should be able to create ad-hoc cluster role, roles and clusterolebindings. (And maybe the operator's service account should be allowed to perform the same actions that the role or clusterrole is meant to perform, plus the ones for managing clusterroles, clusterrolebindings, roles and rolebindings)
Create a single service account and use it for both the operator and the operands
Deploy all the operands, with 0 replicas, and scale them based on the CR config

IMO, the best would be that OLM patches the subjects' namespaces (perhaps with some condition/selector) like Kustomize does.
Here some screenshots showing the changing in the console.

/test unit

/override "Red Hat Konflux / multiarch-tuning-operator-enterprise-contract / multiarch-tuning-Operator" "Red Hat Konflux / multiarch-tuning-operator-enterprise-contract / multiarch-tuning-Operator-bundle"

/approve

/retest-required

/lgtm

/retest-required
/hold
(For a last ack from Prashanth)

/lgtm

/hold

/retest-required

/test e2e-gcp-multi-operator

/test e2e-gcp-multi-operator

/override "Red Hat Konflux / multiarch-tuning-operator-bundle-on-pull-request" "Red Hat Konflux / multiarch-tuning-operator-on-pull-request"

/lgtm

/unhold


Release a new version with ruby 3.2 support

There are changes in the repository to support ruby 3.2 but there is no release.
Please, deploy a new release with these fixes.

Thank you for using writeexcel.
It's been some time, but I've just released new version v1.0.6 that works on Ruby 3.2.

I am using it for quite some time already, definitely working for us on ruby 3.2, in production since April 2028


Functions with allocations

Here I added functional solution, but with heap allocations like in objects.cpp

@MCJOHN974 why do you change objects.cpp in this PR? Is it necessary?

@yegor256 i remove changing of objects.cpp

@MCJOHN974 the build is not green here. Did you run make and make sa locally?

@yegor256 lint is green here

@MCJOHN974 thanks!


Cannot resize for the basic usage

Hi guys!
Thank you for such a wonderful library! It's just when I was playing around with the basic build, I ran into this issue. I wasn't be able to resize the c block like the documents say.
Could you guys please have a look and tell me what I did wrong? Many thanks!
Here is the sandbox for it: https://codesandbox.io/s/loving-swirles-cvitn

You need to add the right styles to your sandbox. Once you do that, you'll see the small resize handlers on the bottom right corner.
<link
rel="stylesheet"
href="https://cdnjs.cloudflare.com/ajax/libs/react-grid-layout/0.16.6/css/styles.min.css"
/>

Hi @carolina-cascade, thank you for your reply. Is this the same with this instruction
. I don't quite understand about the part of installing the style packages. Are they not applied when I type npm install react-grid-layout?
Your solution works very fine but is it any other alternatives for this approach. I can't access to the index.html file when it is built to add this tag.

I found the answer here in #944 so it is closed. Many thanks!


Upgrading from 1.20 to 1.21 failed, kubelite unstable

Hi,
We have microk8s 1.20 running and tried upgrading to 1.21 multiple times. The last time we tried the upgrade failed due to an unstable kubelite, the process keeps exiting and restarting. Based on the logs we where unable to point a root cause on how to fix this issue. We also tried upgrading directly to 1.22 but also without success. I added the inspect tarball. I hope we can get some pointers on how to successfully upgrade to 1.21.
Thanks in advance!
inspection-report-20211111_124059.tar.gz

Hi @vincentheet I see int he logs this error causing kubelet to fail to start:
Nov 11 11:51:12 00-avl-lb1 microk8s.daemon-kubelite[19461]: E1111 11:51:12.721175   19461 server.go:158] "Failed to parse kubelet flag" err="unknown flag: --enable-cadvisor-json-endpoints"

You can check the logs of the k8s services with journalctl -fu snap.microk8s.daemon-kubelite .

We indeed enable this flag, we will disable it and try to upgrade again soon. Thanks for your support!

Disabling the cadvisor json endpoints dit resolve the issue. Thank you.


Tmdb provider is down

Description
Server does not play content error message from java

Provider
Tmdb
Reproduction steps
Select TMDB provider then select movie java script error
Operating system
Nvidia Shield Android TV OS version 11.0
Logs
Java error message content does not play

Please confirm the following

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] If related to a provider, I have checked the site and it works, but not the app.

I get this error in the tmdb provider in the series and movies the tmdb provider is down is there a solution for this problem and error
HTTP  404

I get this error in the tmdb provider in the series and movies the tmdb provider is down is there a solution for this problem and error
HTTP 404

I also get that HTTP 404 message in the tmdb provider in streamflix

The tmdb provider has been fixed

https://github.com/stantanasi/streamflix/assets/165403113/293e8a84-5e84-4e57-a9c5-b3ac46759691
What appears in the video is the problem of the tmdb provider with animes and movies and series

I wouldnt use this app for anime there are dedicated apps for that....you can also switch to h!anime provider in the app and SFLIX provider

I wouldnt use this app for anime there are dedicated apps for that....you can also switch to h!anime provider in the app and SFLIX provider

So the tmdb provider has been fixed, there are no more problems and errors, but there will be an update for the tmdb provider when there will be improvements also for streamflix and other providers for streamflix

Working for me.

I get this HTTP 404 error message in some movies and series This is a bug I think there is a solution With this HTTP 404 message that appears when playing movies and series the video

The 404 error will come to movies and tv shows not already released or with  no server access available

The 404 error will come to movies and tv shows not already released or with no server access available

But there is a solution with that, with the problem with the series and movies I have tried with a series called Powerpuff Girls Z and Hi Hi Puffy AmiYumi
with the episodes and
the HTTP 404 message appears

Because the app is just a scraper it wil not have every single tv series, movie or anime content hence the 404 error message....consider other apps or even KODi for ur anime

Because the app is just a scraper it wil not have every single tv series, movie or anime content hence the 404 error message....consider other apps or even KODi for ur anime

So the problem is the animes in the tmdb provider, not the movies and series. I wanted to see some Hollywood movies and series, some are not available, others are available, movies and series and they should add more providers on streamflix

Have u really looked at the app there are other providers in StreamFlix smh SFLIX is a provider for movies and tv shows as well and its working

Because the app is just a scraper it wil not have every single tv series, movie or anime content hence the 404 error message

Exactly. Good luck finding a source that has everything you search for.

So the problem is the animes in the tmdb provider, not the movies and series. I wanted to see some Hollywood movies and series, some are not available, others are available, movies and series and they should add more providers on streamflix

This TMDb provider is a multisource provider and there's only 1 provider available at this time. Stantanasi hasn't had time (he's building this by himself) to implement more extractors for the rest of the providers. When he is able to do so there will be more content available.


feat/Add schema docs for config.yml

Codecov Report
Base: 32.36% // Head: 43.64% // Increases project coverage by +11.27% :tada:

Coverage data is based on head (84de24d) compared to base (cc3b063).
Patch coverage: 61.53% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #288       +/-   ##
===========================================
+ Coverage   32.36%   43.64%   +11.27%
===========================================
  Files          28       34        +6
  Lines        2249     2669      +420
===========================================
+ Hits          728     1165      +437
+ Misses       1521     1504       -17

Flag
Coverage Δ

unittests
43.64% <61.53%> (+11.27%)
:arrow_up:

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

Impacted Files
Coverage Δ

dbt_coves/tasks/generate/metadata.py
22.22% <0.00%> (ø)

dbt_coves/tasks/generate/properties.py
33.88% <0.00%> (-0.55%)
:arrow_down:

dbt_coves/tasks/generate/sources.py
81.36% <ø> (+58.24%)
:arrow_up:

dbt_coves/tasks/generate/templates.py
42.22% <33.33%> (ø)

dbt_coves/__init__.py
100.00% <100.00%> (ø)

dbt_coves/config/config.py
88.96% <100.00%> (+24.57%)
:arrow_up:

dbt_coves/tasks/generate/base.py
50.21% <100.00%> (+33.91%)
:arrow_up:

dbt_coves/tasks/dbt/main.py
42.18% <0.00%> (ø)

dbt_coves/tasks/load/main.py
100.00% <0.00%> (ø)

... and 20 more

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

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


title

Relevant Issues

Description

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9b51ff5). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage        ?   86.05%
=======================================
  Files           ?        5
  Lines           ?      294
  Branches        ?        8
=======================================
  Hits            ?      253
  Misses          ?       40
  Partials        ?        1

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 9b51ff5...fb73817. Read the comment docs.


fix: update column

reorder column (and fix wrong column title)

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


Update COMMUNITY.md

Changes

Add Notifico to "Built with SeaORM" list

Oh nice! Really love this project. Do you plan to implement push notification without third-party services, e.g. integrating https://github.com/pimeys/rust-web-push ?

Oh nice! Really love this project. Do you plan to implement push notification without third-party services, e.g. integrating something like https://github.com/pimeys/rust-web-push ?

Thanks :) Web push is definitely in the roadmap


feat!: rename struct-specific TypeDefinition -> StructDefinition

Description
Problem*
TypeDefinition is currently struct-specific
Summary*
Additional Context
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

@TomAFrench why was this closed & re-opened? Can we merge it?

Github was having some issues with not running CI last week which meant that when you set auto-merge before it would never merge. I closed+reopened to force CI to reset but I didn't notice it turned off automerge.


uodate Patch

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: GhostsOfHiroshima sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Save Grant data provided by FFIS to the Grants DB

DOD: Whenever a new S3 object with a key ending in /ffis.org/v1.json is saved to the "Grants Prepared Data" S3 bucket, relevant data is extracted and saved (via upsert) to a database record that represents the canonical state (according to USDR) for that grant.
Blockers:

usdigitalresponse/grants-ingest#3

Implementation details:

Create a "Persist FFIS data to Grants DB" Lambda function that:

Allows invocation from the "Grants Prepared Data" S3 bucket
Expects to receive an S3 Bucket Notification from the "Grants Prepared Data" bucket for the s3:ObjectCreated event that indicates a file containing JSON-serialized grant data from FFIS is fully (not partially) saved to S3, with the following object key filters:

Suffix: /ffis.org/v1.json

When invoked, performs the following actions:

Retrieves the newly-created S3 object and parses its data
Saves the parsed grant data to the "Grants" DynamoDB table as an "upsert" operation. Note that this is achieved by using  the UpdateItem (not PutItem) method of the DynamoDB API, which will ensure that a new record will be created if none previously existed, but will otherwise atomically update the specified fields on an existing item.

Hi,
Getting started, I have a few data-related questions from looking at one of the FFIS spreadsheets (looking at "Competitive Grant Update 23-16"). If there's a data model documented somewhere, feel free to point me to that and I'll try to answer on my own; apologies if these are naïve:

What is the cardinality of program to CFDA to opportunity? Is it something like this?

erDiagram
    PROGRAM ||--|{ CFDA: has_many
    CFDA||--|{ OPPORTUNITY : has_many

What does "Match?" refer to/mean?
Is "opportunity number" a reliable key for a given grant/opportunity?
Is a Program something that has an identifier and canonical source? Or do we just have the names to go off of?
How much do these fields change from week to week? Is the 23-16 spreadsheet (I can provide if you don't have it) pretty representative? Are there ever extra sheets or anything like that?

Implementation/technical questions:

Is the Grants table defined somewhere in IaC? I didn't find it in this repo.
Multiple processes could be writing to this record, right (thinking the GG data)? I'm guessing we'll want the transactional API's?

Answers to above from call today:

Updated that in place, but acts (FKA programs) have multiple CFDA's which can each have multiple grants/opportunities.
Dunno, but we only care about the act/bill for now.
Opportunity points to grant number, so yeah.
No, we may need to solve that later but for now, assume `program/act names don't change.
They don't change much in structure. Single sheet.
Other questions
Yes, it's coming in #46
Going to use single column update here so it shouldn't matter.


RequestContextAwareCompletableFuture callback not invoked when context mismatches.

Let's say we have the following code:
ServiceRequestContext ctx1 = ServiceRequestContext.of(...);
ServiceRequestContext ctx2 = ServiceRequestContext.of(...);
try (SafeCloseable sc1 = ctx1.push()) {
    CompletableFuture f = ctx2.makeContextAware(new CompletableFuture());
    AtomicBoolean callbackCalled = new AtomicBoolean();
    f.whenComplete((res, cause) -> callbackCalled.set(true));

    f.complete(null);

    // The callback will never be invoked and
    // the following assertion will fail.
    assert callbackCalled.get(); // <-- Failure!
}

It seems like everything is working as expected. The callback (BiConsumer) was not invoked because it was decorated by ctx2.makeContextAware(BiConsumer) and the callback is invoked only when ctx2.pushIfAbsent() succeeds (see), to protect our users from the accidental context conflict like the above example.
The problem is that a user does not have a way to know this happened. It is really confusing because the callback will never be invoked. It is neither possible to handle the IllegalStateException raised by pushIfAbsent():
f.whenComplete((res, cause) -> callbackCalled.set(true))
 .exceptionally(cause -> {
     // We never reach here, either, because this callback
     // will be decorated again and fail at `pushIfAbsent()` again.
     ...
 });

What would be the best way to let a user know about this problem?

How about logging at WARN level than raising an exception for the future callbacks? If a user wraps something explicitly via ctx.makeContextAware(), we could raise an IllegalStateException as before, though, i.e.
class AbstractRequestContext ... {
    // Stay same.
    BiConsumer makeContextAware(BiConsumer consumer) {
        return (t, u) -> {
            try (SafeCloseable ignored = pushIfAbsent()) {
                action.accept(t, u);
            }
        };
    }
}

class RequestContextAwareCompletableFuture ... {
    @Override
    public CompletableFuture<T> whenComplete(BiConsumer<? super T, ? super Throwable> action) {
        return ctx.makeContextAware(super.whenComplete(laxMakeContextAware(action)));
    }

    BiConsumer laxMakeContextAware(BiConsumer consumer) {
        return (t, u) -> {
            try (SafeCloseable ignored = push()) {
                action.accept(t, u);
            }
        };
    }

    SafeCloseable push() {
        final RequestContext oldCtx = RequestContext.currentOrNull();
        if (oldCtx != null && oldCtx != this.ctx) {
            // Warn. Do not raise an exception.
            if (!warned) {
                logger.warn(...);
                warned = true;
            }
        }
        return ctx.push();
    }
}

The problem described in #2282 is also related with this problem. I understand pushIfAbsent() provides some protection against unnecessarily nested contexts, but there are valid cases of nested contexts, such as:

A service triggering a client request (described in this issue)
A client triggering another client, i.e. RetryingClient (described in #2282)

Therefore, how about relaxing pushIfAbsent() to allow:

pushing a client context when the current thread local context is a service context
pushing a context when one of its parent contexts is the current thread local context.

We will have to add RequestContext.parent() to handle this case, where newDerivedContext() sets the parent context automatically. I'm not sure if this is the best idea though. Thoughts? @minwoox @ikhoon @anuraaga

I think these basically make sense but does it mean we need to make AttributeMap read from parents too? I don't know if it's ever good to completely overwrite the current context (it's what the ifAbsent check is protecting against), for example if a span is saved in the current context I think it still needs to be accessible after such a push.

That's a good question which leads us to #1083. Perhaps we should let a user specify whether an attribute value needs to be copied when a context is derived or nested? e.g.
static final AttributeKey<TraceContext> TRACE_CONTEXT_KEY =
        AttributeKey.valueOf(TraceContextUtil.class, "TRACE_CONTEXT",
                             /* inherited */ true);

We will have to fork/replace Netty AttributeKey to do this, though.

pushing a context when one of its parent contexts is the current thread local context.

I think this should be:
pushing a context when it is one of the parent contexts of the current thread local context.
Yes, I think RequestContext.parent() is a good choice for that and also having the AttirbuteKey which a user can specify whether to copy or not looks good to me.

Do we have any examples where we wouldn't want to inherit attributes? I feel as if when a context is mounted, semantically it should always be mounted until popped - which is a little different than starting a thread which may or may not have a related lifecycle, so always inheriting seems reasonable. If it seems ok we wouldn't need to fork code keeping things simpler.

Yeah, we could default to 'always inherit' and then wait for a counter use case.

By the way, once we introduce RequestContext.parent() we might not need to copy the attributes. When not found, could just check the parent.

Ran into a somewhat related issue
https://github.com/openzipkin/zipkin/issues/2932#issuecomment-561021463
Just need to make sure that with what we do here, in the context of tracing, we don't end up with two client spans in a parent/child relationship. I don't think it's what we're planning here, but that discussion reminded me of this one :)

we don't end up with two client spans in a parent/child relationship.

Had a chat with @trustin about this.
In the example described in #2282, the order of the contexts in thread-local will be:
ServiceRequestContext -> ClientRequestContext1 -> ClientRequestContext2
This will have the parent/child relationships between ClientReqeustContexts.
This will be not a problem if the ClientReqeustContext1 and ClientReqeustContext2 do not have any attribute and look up to find an attribute only in ServiceRequestContext. But it's a problem when ClientReqeustContext1 has the attribute that ClientReqeustContext2 is looking for.
So we thought the word parent is kinna vague to represent what it really is and
we needed to use the more specific term like ClientRequestContext.serviceContext() (or serviceRequestcontext?).
What do you think @anuraaga?

So if I understand correctly, a client context will either read from itself, or an ancestor service context in the context chain. That seems fine to me.

Exactly, let me work on it. 😉

How about logging at WARN level instead of raising an exception for the future callbacks?

That sounds good. Let me work on this after #2322 is merged.

How about logging at WARN level instead of raising an exception for the future callbacks? If a user wraps something explicitly via ctx.makeContextAware(), we could raise an IllegalStateException as before, though, i.e.

Had a chat with @trustin and decided to leave warning logs multiple times to let the users know something is going wrong.
We will still throw an exception because otherwise some critical resources can be shared such ae access token, etc.


UI responsiveness delay in multiple native windows

Tested versions

Reproduced in 4.3.beta1
Works fine in 4.3.dev6

System information
Windows11
Issue description
4.3.beta1 creates multiple windows in the scene and uses the Windows native window style (not embedded). When switching between windows, the UI response inside has serious delays. The game's frame rate is normal, but the UI response is unacceptably slow.
This problem does not exist in 4.3.dev6.
Steps to reproduce
https://github.com/godotengine/godot/assets/160569414/2df322ca-4819-445c-8765-085cc85c40fe

As shown in the video, when only one window A is opened, the hover event of the button in the new window responds very sensitively.
Then open window B, and you can see that the hover of the button in window B is a bit slow.
Finally, switch back to window A, and you can see that the hover switching becomes very slow.

In the actual project, if more UI controls are included, the delay will reach more than ten seconds.
Minimal reproduction project (MRP)
window_bug.zip

cc @bruvzg

Can you take a look this comentary (https://github.com/godotengine/godot/issues/92954#issuecomment-2158618652) and do the mentioned test to check if your issue is the same?

Can you take a look this comentary (#92954 (comment)) and do the mentioned test to check if your issue is the same?

The PR mentioned there can fix this issue!

yes, it solved!


Double dismiss

Hey, How to trigger double dismiss() transition when I have 2 navigation controllers (each as independent xcoordinator) presented on a vc? I need to trigger it from the one coordinator which is on the top of this stack ofc.
Btw awesome job guys. Very useful framework.

Ok I found solution. All you need is to pass a router from the very first vc and trigger it's dismiss action on the very last.


test_sys_executable fails on Windows

I'm seeing this test failure in test_easy_install when running on Windows:
_____________________ TestCommandSpec.test_sys_executable _____________________

self = <setuptools.tests.test_easy_install.TestCommandSpec object at 0x000001B5944E9208>

    def test_sys_executable(self):
        """
            CommandSpec.from_string(sys.executable) should contain just that param.
            """
        writer = ei.ScriptWriter.best()
        cmd = writer.command_spec_class.from_string(sys.executable)
>       assert len(cmd) == 1
E       assert 3 == 1
E        +  where 3 = len(['C:\\Program', 'Files\\Python', '3.5\\python.exe'])

setuptools\tests\test_easy_install.py:587: AssertionError

My guess is this test will only fail if Python is installed in a directory with spaces in the name, something that should be supported.

This test was removed in #616.


Add setOneShotPreviewCallback to get single preview frame

This adds setOneShotPreviewCallback to get single frame from preview. It is different fro$

It only returns one frame and removes callback after
It produces frame that has no tearing, unlike the ones from frame processor where move$

Im not sure whether it belongs in another routine or if the runBlocking is needed at all.
I quickly did it like focusRoutine and it works for me, feedback is welcome.
Issue: https://github.com/Fotoapparat/Fotoapparat/issues/200

Thanks @Raenar4k ! I have added 2 comments, so this will need to some extra work.
runBlocking is needed to obtain the available camera.


how to call a function when received a notification when the app is in foreground

When a notification is received how can we call another function to update the state inside the app without clicking the notification on the panel like firebase notification which gives listener when the app is in foreground we execute some logic to update the state directly.

Hi @santhosh-keyworks,
We'd be glad to assist with your inquiry.
To access and update a Component state when a notification is received and the app is in foreground, please modify your App.js as follows:

Add the following line of code after the import statements:

let appInstance;

Add the following line of code after super(props); inside your App component's constructor method:

// Make component accessible to callbacks
appInstance = this;

Add the following line inside the Pushy.setNotificationListener and/or Pushy.setNotificationClickListener callback methods, updating the state accordingly:

// Update state from notification callbacks
appInstance && appInstance.setState({ message: data.message });

Please let us know if you face any issues.


"Hide Property Selection" setting missing for pie charts

The setting to hide selectors is missing in the pie chart, we should (re)add it back in

Fixed as part of https://github.com/neo4j-labs/neodash/pull/744.


gigasecond: Confusing Initial File

As seen here, the library template starts the user off with a class. This is confusing because in the tests (1) there is no instantiation of the class, (2) the tests call the function add(DateTime).
This should be fixed by either (1) refactoring the tests to call the method from a class, (2) refactor the library template to start out with the function called by the tests.

@james-r-smith Or the user can make the changes necessary to pass the test. Exercism is not meant to teach people programming concepts, just serve as a way to introduce experienced programmers to another language.
Still, it would't hurt to make this small change.. If anyone is interested in tackling this, I see no issue with it.

I can look into this one. Am I correct to assume that the only required change would be to update the starter code from an empty class to an empty version of the function in the example? If that's the case I can have a PR submitted rather quickly to fix this.

Fixed in #258


Creating mirrors via API

I was looking into creating mirrors via the Go client, but that doesn't seem to be supported yet.
Is this simply a matter of adding the missing fields to the request (I'll make a pull request), or will this be difficult to add?

Hi, please see https://github.com/gogits/go-gogs-client/wiki/Repositories#migrate

Oh, that's fantastic, I'll see what I can do!

Fixed in gogits/go-gogs-client#9


Consider getting rid of aligned dependency subtree

This eventually pulls in two versions of generic-array which is a bit weird. Maybe take over aligned and/or as-slice and add some feature flags there? Or make itm::write_aligned itself an optional feature?
re. https://github.com/rust-embedded/cortex-m/issues/183

Oh this is where all the generic-array dependencies come from? I was wondering about that.
Pulling in 4 dependencies for a single function seems very excessive, so I'm on board with removing them. We can add our own alignment-enforcing struct to the itm module instead.

Yeah for some (compatibility?) reason as-slice pulls in both.

Yeah, otherwise you run into weird compatibility problems.

None of my dependencies that depend on generic-array use 0.12, so I' be curious what compatibility issues there might be?


Improve testing

Addresses some of the issues in #137. Enables simulation of cmds and disabling invariants for testing matchers.
@gniquil and @jon-thompson I haven't done the tests or doc changes yet. I want to see what you think of this implementation first.
Basically, now you can say
//Cmd.run
expect(getCmd(result).simulate({result: 123, success: true})).toEqual(actionCreator(123));
expect(getCmd(result).simulate({result: 123, success: false})).toBe(null);

//Cmd.actiom
expect(getCmd(result).simulate()).toEqual(actionIPassedToCmd);

//Cmd.list
expect(getCmd(result).simulate([
  {result: 123, success: true},
  {result: 456, success: false}
])).toEqual([actionCreator1(123), actionCreator2(456]);

alternatively, we can expose something like Cmd.simulate(cmd, simulation), but I kind of like it on the cmd itself. Another option is to break this logic out into a separate package, but I think it's simple enough to not bother.

Works for me too.


Logging in with the correct credentials

As a user
I need to be able to enter my username and password
So that I can enter the inventory tracking system
Details and Assumptions

I am a user with login credentials
I am authorized to enter the inventory tracking system

Acceptance Criteria
Given I am a user
When I enter my login credentials on the login page
Then I will authenticate my identity (2FA) and be allowed to access the inventory tracking system.

This has been completed.


Python script to ease data collection for config files.

I have created a simple script to help with the all the data collection from RPA,
You can find it in a gist, here.
Assumptions
This script assumes the following initial file structure:
./
./data/
./data_parser.py

Operation

Choose an O/F ratio to work on.
Use RPA to generate data for each chamber pressure (Pe) you need in increasing order, saving the results in ./data/ with an individual and also increasing name (e.g. 1.txt, 2.txt, etc.).
Run the python script when you have all data you desire for the current O/F ratio.
A file results.txt will appear in ./, containing all the key values formatted for the config.
You must delete all files in ./data/ and ./results.txt before starting the next O/F ratio! I have not added any file handling other than reading the data files and writing the results.

I may update this with further improvements, but this saves so much time already. Just make sure to watch for any questionable data while doing this.

I've updated the script, here: https://gist.github.com/dgeastman/29a26dd8c08fb156660fefea1a8ffda8
It now has the Cpc and Cpe fields, and I also moved the results.txt file into the ./data directory, so that you can easily archive the input and results in case we need to go back again.

@ferram4 Do you foresee needing any data on propellant mixtures/mass ratios, nozzle inlet/throat, or anything else that RPA calculates?

@Tyaedalis I don't see any reason to keep the nozzle inlet data, since in this situation it's identical to the injector data.  I'm not using the throat data currently, but I can see that I might at some point so keep that.  Only other data I'd archive is the stoichiometric O/F somewhere (shouldn't change between O/Fs).  If you want to plan ahead for solid fuel, keep track of the mass fraction of condensed phases (liquid or solid) at the nozzle exit, since that'll be useful for handling solid rocket losses.

Hey, I just gotta say I've been busy with other work, and not only disappointed that I have to redo everything. BUT! I do have plans to redo my script and regenerate everything. I've only been quiet here because of other business. I really want to see this project reach finality.

Oh, don't worry too much.  I made some minor updates to the kerolox config since everything with an O/F lower than ~O/F 2.2 had the wrong area ratio set (to 1.5 rather than 1.3, understandable) so that's fixed except for 0.8 since that requires the averaging process and I've been putting off doing that one.
  I don't think it requires changes in the data capturing procedure, I'll adjust it in code instead.


Allow specify build artifact dir (allows automation to set the dir of the .deb file)

Codecov Report

Merging #3954 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #3954      +/-   ##
=========================================
+ Coverage   70.57%   70.6%   +0.02%
=========================================
  Files         429     429
  Lines       27314   27314
  Branches     4168    4168
=========================================
+ Hits        19277   19285       +8
+ Misses       6793    6783      -10
- Partials     1244    1246       +2

Impacted Files
Coverage Δ

...li-cloud/azure/cli/command_modules/cloud/custom.py
14.45% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
70.66% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/cloud.py
73.18% <0%> (+2.27%)
:arrow_up:

...c/azure-cli-testsdk/azure/cli/testsdk/utilities.py
86.36% <0%> (+13.63%)
:arrow_up:

Continue to review full report at Codecov.

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


Load all pages at once with PDFViewer

How can I load all the pdf pages at once with PDFViewer, in viewer.js I found method for forceRendering method however it is not working in my case? Do we have any work around to this?

You can do it like the SVG viewer at https://github.com/mozilla/pdf.js/blob/master/examples/svgviewer/viewer.js#L62 which renders all pages.


plot.pcolormesh fails with shading='gouraud'

xarray.plot.pcolormesh() fails when you pass the matplotlib.pyplot.pcolormesh() keyword argument shading='gouraud' to it.
Code Sample, a copy-pastable example if possible
import matplotlib.pyplot as plt
import numpy as np
import xarray as xr

lon, lat = np.meshgrid(np.linspace(-20, 20, 5), np.linspace(0, 30, 4))
lon += lat/10
lat += lon/10

da = xr.DataArray(np.arange(20).reshape(4, 5), dims=['y', 'x'],
                  coords = {'lat': (('y', 'x'), lat),
                            'lon': (('y', 'x'), lon)})

da.plot.pcolormesh('lon', 'lat', shading='gouraud')
plt.show()

Problem description
This gives an error:
Traceback (most recent call last):
  File "mwe.py", line 17, in <module>
    da.plot.pcolormesh('lon', 'lat', shading='gouraud')
  File "/home/tegn500/Documents/Work/Code/xarray/xarray/plot/plot.py", line 721, in plotmethod
    return newplotfunc(**allargs)
  File "/home/tegn500/Documents/Work/Code/xarray/xarray/plot/plot.py", line 662, in newplotfunc
    **kwargs)
  File "/home/tegn500/Documents/Work/Code/xarray/xarray/plot/plot.py", line 864, in pcolormesh
    primitive = ax.pcolormesh(x, y, z, **kwargs)
  File "/home/tegn500/anaconda3/envs/py36/lib/python3.6/site-packages/matplotlib/__init__.py", line 1805, in inner
    return func(ax, *args, **kwargs)
  File "/home/tegn500/anaconda3/envs/py36/lib/python3.6/site-packages/matplotlib/axes/_axes.py", line 5971, in pcolormesh
    X, Y, C = self._pcolorargs('pcolormesh', *args, allmatch=allmatch)
  File "/home/tegn500/anaconda3/envs/py36/lib/python3.6/site-packages/matplotlib/axes/_axes.py", line 5559, in _pcolorargs
    C.shape, Nx, Ny, funcname))
TypeError: Dimensions of C (4, 5) are incompatible with X (6) and/or Y (5); see help(pcolormesh)

Expected Output
This should give almost the same image as in the documentation, just with smoother shading:

I found this on googling: https://github.com/matplotlib/matplotlib/issues/8422

Oh thanks @dcherian !
That's confusing because this works:
plt.pcolormesh(da['lon'].values, da['lat'].values, da.values, shading='gouraud')

but it looks like matplotlib is the problem, I'll close this.

This might be an infer_interval_breaks issue?

I tried
da.plot.pcolormesh('lon', 'lat', shading='gouraud', infer_intervals=True)

but got the same error unfortunately.

I had the same problem. I found that
da.plot.pcolormesh('lon', 'lat', shading='gouraud', infer_intervals=False)

does work. Would be nice if shading='gouraud' worked by default though.
I guess the problem is that 'gouraud' needs the coordinate positions at the grid points, and I assume infer_intervals=True is creating coordinate values at the grid-cell corners (so an n x m DataArray is plotted using coordinates with lengh (n+1) and (m+1). I haven't looked into what infer_intervals actually does though. Would it be safe to just turn it off if shading='gouraud' is used?

Oh... I saw this comment, but I tried with infer_interval = True not False. Yes, it works with False.
I think we can pass the shading argument trough - plot_field -> plot_field_monitor_data -> plot_scalar_array -> field_data.plot. Or create a **args in plot_scalar_array and input it into field_data.plot.


shaded.databricks.org.apache.hadoop.fs.azure.AzureException: hadoop_azure_shaded.com.microsoft.azure.storage.StorageException: Server failed to authenticate the request

Could you change this in section 3
wasbs_path =<EMAIL_ADDRESS>% (blob_container_name, blob_account_name, blob_relative_path)
spark.conf.set('fs.azure.sas.%s.%s.blob.core.windows.net' % (blob_container_name, blob_account_name), blob_sas_token)
print('Remote blob path: ' + wasbs_path)
with this
blob_sas_token = r""
wasbs_path =<EMAIL_ADDRESS>% (blob_container_name, blob_account_name, blob_relative_path)
spark.conf.set('fs.azure.sas.%s.%s.blob.core.windows.net' % (blob_container_name, blob_account_name), blob_sas_token)
print('Remote blob path: ' + wasbs_path)
as mentioned in https://github.com/MicrosoftDocs/azure-docs/issues/88403
Sincerely
Best Regards
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 355fba0e-958e-f572-18c2-892c91d20b13
Version Independent ID: 7fe76247-c969-2068-2c3c-bf327fbbe39e
Content: Quickstart - Run a Spark job on Azure Databricks Workspace using Azure portal
Content Source: databricks/scenarios/quickstart-create-databricks-workspace-portal.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

Thanks for providing feedback that helps improve our documentation. This issue has been resolved and this outdated content has been removed. We are closing the issue.
#please-close

#please-close


Fix Rocky Linux 8 arm64 GCC Compiler Warnings

This patch addresses the compiler warnings raised by the GCC 11.2.1
compiler on Rocky Linux 8 arm64:
-Werror=shift-negative-value in I_Continuation.h:
In file included from src/ts_asf_master_test_build/iocore/eventsystem/I_Action.h:29,
                 from src/ts_asf_master_test_build/iocore/eventsystem/I_EventSystem.h:32,
                 from src/ts_asf_master_test_build/iocore/eventsystem/I_VConnection.h:29,
                 from QUICStreamVCAdapter.cc:24:
src/ts_asf_master_test_build/iocore/eventsystem/I_Continuation.h: In instantiation of _constexpr int (Continuation::* continuation_handler_void_ptr(int (C::*)(int, T*)))(int, void*) [with C = QUICStreamVCAdapter; T = void; ContinuationHandler = int (Continuation:
:*)(int, void*)]_:
QUICStreamVCAdapter.cc:29:3:   required from here
src/ts_asf_master_test_build/iocore/eventsystem/I_Continuation.h:74:10: error: left shift of negative value [-Werror=shift-negative-value]
   74 |   return static_cast<ContinuationHandler>(fp2);
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

-Werror=sign-compare in fastlz.cc:
fastlz.cc: In function _void flz_copy64(uint8_t*, const uint8_t*, uint32_t)_:
fastlz.cc:209:17: error: comparison of integer expressions of different signedness: _int_ and _uint32_t_ {aka _unsigned int_} [-Werror=sign-compare]
  209 |   for (c = 0; c < count * 8; ++c) {
      |               ~~^~~~~~~~~~~

-Werror=maybe-uninitialized in header_rewrite:
header_rewrite/conditions.cc: In static member function _static void ConditionInbound::append_value(std::string&, const Resources&, NetworkSessionQualifiers)_:
header_rewrite/conditions.cc:1131:56: error: _tags_ may be used uninitialized [-Werror=maybe-uninitialized]
 1131 |     TSHttpTxnClientProtocolStackGet(res.txnp, tags.size(), tags.data(), &count);
      |                                               ~~~~~~~~~^~
In file included from header_rewrite/conditions.cc:29:
/opt/rh/gcc-toolset-11/root/usr/include/c++/11/array:176:7: note: by argument 1 of type _const std::array<const char*, 8>*_ to _constexpr std::array<_Tp, _Nm>::size_type std::array<_Tp, _Nm>::size() const [with _Tp = const char*; long unsigned int _Nm = 8]_ declared here
  176 |       size() const noexcept { return _Nm; }
      |       ^~~~
header_rewrite/conditions.cc:1128:33: note: _tags_ declared here
 1128 |     std::array<char const *, 8> tags;
      |                                 ^~~~

-Werror=type-limits in traffic_dump:
experimental/traffic_dump/json_utils.cc: In function _int {anonymous}::esc_json_out(const char*, int64_t, std::ostream&)_:
experimental/traffic_dump/json_utils.cc:117:18: error: comparison is always true due to limited range of data type [-Werror=type-limits]
  117 |       if ('\x00' <= c && c <= '\x1f') {
      |           ~~~~~~~^~~~

Cherry-picked to v9.2.x


Added rememberCurrentState and cacheCurrentPage to the Public API

We needed some custom behaviour in our application, we want to cache dynamic pages (backbone/marionette).
We always call these two methods rememberCurrentState and cacheCurrentPage together, so creating one method that does both would also work for us.
If we completely missed a better way of doing this, please let us know!

I don't understand why you're calling pushState / replaceState manually and would prefer not exposing Turbolinks's internals like that.
Feel free to reopen if this is still an issue for you.


Perpetual ECONNREFUSED when running on bare-bones nextjs app

bug report

version 1.14.0
platform OSX

Repro steps
I ran into an issue on my fairly large nextjs app when moving from a custom server which ran using https to the next start server running on http. I was able to make a minimal repro here. (kindly ignore the repo name. That was initially created for a different issue)

yarn install
yarn test:integration

running curl http://localhost:3333 returns the html string
I'm using http-get on account of the webpack dev server.

expected behavior

start-server-and-test recognizes the running server and kicks off the test suite

actual behavior

Permanent loop of ECONNREFUSED errors until timeout limit is reached

I've tried tcp:3333 syntax, with the same result. I'm really not sure why it can't reach the server address.

Well, interestingly enough, it works fine using <IP_ADDRESS>:3333, rather than localhost:3333
I also see the same behavior using just wait-on, so this seems to be related to that dependency, not this module.

Ran into the same problem
Related issue in wait-on: https://github.com/jeffbski/wait-on/issues/109


Master

...

Could you, please, close this request? I am pretty sure it is made by mistake.

"master" branch is dead post 0.2.20, all changes to develop in the past two years have been made into releases via the 0.3.0 branch.


remove table row borders

It should be either thead or the line should be removed.
Our new docs say

Tables are borderless by default.

http://next.materializecss.com/table.html

Tables are meant to have borders by default now to be more in line with material spec. The docs just needs to be updated to reflect this.

But this does not work with the borderless tables which have borders because of this.

bordered and borderless table have the same styles as you can see at http://next.materializecss.com/table.html

Fixed documentation here: a7f1154a4cc3717b3620bd7c199dc9e6b82170c9


[INLONG-409] Adjust the configuration items of protobuf

JIRA link - [INLONG-409] created by gosonzhang

fixed by https://github.com/apache/incubator-inlong/pull/311


[Tutorial proposal] Monitoring change in forest vegetation condition

What is the objective of the proposed tutorial?
This tutorial will demonstrate an approach to assess forest vegetation condition using long term trends in vegetation status using MODIS data. It will walk through the choice of vegetation index and a particular season for computing annual composites, followed by trend analysis to compute trends in vegetation status change, a logic to infer vegetation condition, and visualize results. It will demonstrate calculating annual seasonal composites, reducers for trend analysis and area calculations, result charting and image visualizations. This is from a workshop I co-taught with a colleague at an earth engine summit last year
What is the scope of the proposed tutorial?
Forestry
Please provide an outline of the structure of the proposed tutorial?

Import images, features
Compute annual summertime composites
Apply regression and infer vegetation condition
Visualize results: extent and direction of change

In what format will you be submitting the tutorial?
Markdown
Issue labels:
proposal, tutorial

Hi @pskoulgi,
Thank you for submitting this clear, well-defined proposal. We think it would make an excellent addition to our Community Tutorials at https://developers.google.com/earth-engine/tutorials/community.
Please proceed with instructions at https://developers.google.com/earth-engine/tutorials/community/write to create and submit your tutorial for review. I will then do a quick first editorial pass for form and structure. Once edits are received and comments are resolved, I will then assign a colleague to do a more thorough review. Please bear with us as we go through the review process, which can take a few iterations to complete.
If your tutorial seems much longer than typical tutorials found in our documentation, we may ask that you split it into two or more parts. For now, please feel free to submit the tutorial in its entirety and we can then discuss.
Looking forward to receiving your first pull request!
-Gino

Hi @gino-m,
I put together a first draft and just sent a pull request. Let me know how it looks, when you get a chance. This is my first pull request, so excuse me if I didn't get something right -- happy to learn more and fix any mistakes.
One question on how to correctly use and reference vector data I have used here: where I needed some forest boundaries in the code, I have used one of my EE table assets which is imported from a shapefile that is published in a public repository. I have added pointers to that in the description of the table asset. Would that suffice or is it recommended that I do something more?
Looking forward to your review comments.
-Pradeep

One question on how to correctly use and reference vector data I have used here: where I needed some forest boundaries in the code, I have used one of my EE table assets which is imported from a shapefile that is published in a public repository. I have added pointers to that in the description of the table asset. Would that suffice or is it recommended that I do something more?

Good questions, sorry I just saw this now. Would it be possible to replace this asset with a few hard-coded polygons? This would allow the tutorial to be usable independently of your assets dir. Wdyt?

I found that I can use the WDPA dataset instead. See here.


Is that possible to set screen size?

I have written this code:
lv_obj_t * InboxScreen=lv_obj_create(NULL, NULL);
lv_obj_set_size(InboxScreen,LV_HOR_RES,LV_VER_RES-30);
lv_obj_set_pos(InboxScreen,0,LV_VER_RES-30);
lv_scr_load(InboxScreen);
but I've got an error in run time in setting object position.( seems that lv_obj_set_pos doesn't work pro parley, this issue title should be "set screen position" not "set screen size" )

Hi,
You can't modify the screen size and position. But you can create an lv_obj on a screen and use it as a container.

Ok thank you Kisvegbor

Your welcome!


Small update to the Payment Handler article

What's changed, or what was fixed?

An update to https://developers.google.com/web/fundamentals/payments/payment-apps-developer-guide/web-payment-apps#optional_let_chrome_skip-the-sheet

Updated conditions to skip the Payment Handler sheet

[ ] This has been reviewed and approved by (NAME)

[x] I have run npm test locally and all tests pass.

[x] I have added the appropriate type-something label.

[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele @sahel-sh

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L271 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L294 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L271 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L294 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L271 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L294 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L271 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/fundamentals/payments/payment-apps-developer-guide/web-payment-apps.md#L294 - Common typo found: 'ServiceWorker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.

@petele can you please help us approve this and merge?
Thanks!


Data assess doc

Added dataset assessment documentations
Updated DatasetModule parameters design for simplicity and flexibility
Updated sensitive feature predictability ROC-AUC score to use for categorical features too
Created dataset assessment demo to walk a user through the assessment steps, algorithms, and results interpretations

Went through demo notebook, which was awesome. Loved the section breaking down the different functions.
It broke on the predicting the sensitive feature with xgboost. I fixed that error. I also wanted to get feature importance from that model, which was difficult because of the preprocessing (column transformer). I found some code that lets us get the feature names out of the preprocessor and used that to add "sensitive_feature_prediction_feature_importances" to the results of the dataset assessment. This helps a lot!

Updated.
As I was going through it again I still don't think the module should take in data. It lead to unexpected performance.
For instance, when I include the sensitive feature in CredoData's X, I should see that it is one of the features in the dataset module's mutual information plot. And the ROC of the overall proxy detection should be 1. Instead, those features are dropped from the dataset, without my knowledge!
The module is making choices, presuming that I removed the sensitive feature and label from my data. This prevents the module from catching some real issues! If my X data truly does contain a sensitive feature, I should see that... I have a proxy for the sensitive feature in my X.
I believe the module could be changed to accept X, y and sensitive features with minimal changes to the code, while being more predictable. Right now it's not making use of the preprocessing we do in CredoData, which is breaking some modularity. We want all modules to interact with the same data.


Update Doc: Include New Relic as one of the third-party solution.

The Doc page "Baseline architecture for an Azure Kubernetes Service (AKS) cluster" is updated (minor) to include "New Relic" as one of the third-party solution.
AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Cloud Architecture Content Team (CACT). Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors

Topic
Guidance

Draft PR
If your PR will be a work-in-progress for more than a day or two, select the Convert to draft link in the upper right of the page (under Reviewers) to change it to a draft. For future reference, you can also do this using the Create pull request button drop-down during PR creation.

ms.date metadata
Don't update an article's "ms.date" metadata property unless you've done a full freshness review of the content. A full freshness review includes changes required to correct or improve the full technical accuracy of the article.Don't update "ms.date" if you're doing targeted changes to improve non-technical aspects of the article, such as editorial quality, art improvements, article template alignment, etc.If you've changed any "ms.date" properties for work that wasn't part of full review for freshness, please reset them to their previous value.

Placement and linking
If you're creating a new article or articles, include updates to the related TOC.yml file to propose where the article(s) should be placed. Also consider other places within the document set where it would be beneficial to cross-reference and link to your new article(s).

PR build
After you open your PR, and for each successive commit that you push to your branch, the publishing platform will run validation on the files in your pull request. A summary of the build results for each file will be inserted inline into your pull request, which includes any build suggestions/warnings/errors. PRs cannot be merged until all build errors and most warnings are resolved.

Publishing
Following a successful merge, most repos publish to the live site at least once per (business) day, usually around 10am Pacific.

Additional resources
Learn.Microsoft.Com contributor guide

Additional guidance for private repos and internal contributors

Topic
Guidance

PR size
If your PR is more than ~5 lines of changes, or you'd like for the changes to go through editorial or larger review, open a contribution request at https://aka.ms/Contribution and include a link to the PR in response #8. Once it's processed, you'll be notified of the next steps.

PR title prefix
Select the Edit button to the right of the PR title if you need to revise it. The following prefixes are reserved for specific contribution types:[Quality Check] - maintenance work related to content quality (edit passes, art improvements, template alignment)[LinkFix] - recurring/adhoc PRs to correct link URLs[Pipeline] - new/updated contributor success pipeline content[WIP] - a work-in-progress draft requiring several days/weeks

PR preview
Following successful build of your PR, publishable files will also include Preview URL links to staged previews of your new/updated articles. Be sure to review these for verification of your intended contributions, or to send to other internal contributors for review.

PR sign-off (public repo)
If an article you own is updated in a public repo PR, you are responsible for sign-off. You will be automatically notified via email. The PR will not be merged until you've had a chance to review and sign-off.

PR sign-off (private repo)
After you've completed your proposed changes, addressed build warnings, and completed all review work, you can begin the sign-off process for review and merge:If your PR is in draft mode, remove "[WIP]" from the title and select Ready for review button at the bottom of the PR.Enter "#sign-off" in a new comment. This comment indicates that you're confident the work meets or exceeds Microsoft's standards for publication, and will trigger the review process.Your PR may be selected for initial review by the CACT. Following CACT review, you may receive questions or requests for additional changes. You should have initial feedback from CACT review within a few business days. If you have an urgent request or need to contact the team, please mention @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers in your PR and someone will get back to you. After CACT review is complete, a CACT #sign-off will be added.Final review/merge is done by the PR review team. The PR team may also respond with feedback, categorized as "Blocking" (requires action from you), or "Non-blocking" (to be addressed in a future PR).

Additional resources
Learn.Microsoft.Com internal contributor guideAuthoring templates: architecture-center-pr, well-architected-prTo contact the CACT use e-mail, or @mention our GitHub team in your PR comments using: @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers

I'm fine adding NR into here, but can you please list them in alphabetical order (to make sure we're not implying any particular preference ordering)

@ckittel I would like to sort it in alphabetical order. As there was Grafana, Datadog text from previous contributors, I refrained from make any change to the order.

Yeah, was probably an oversight before.  So please do here in this PR.

Yeah, was probably an oversight before. So please do here in this PR.

Change is done.

CACT #sign-off
#remove-label:"do-not-merge"
#label:"ready-to-merge"


Move model_config to common

Addresses https://github.com/triton-inference-server/server/issues/2759: many of the functions in model_config.h are generically useful even on the client side and only depend on the generated protobuf headers. Therefore, it is useful to have them more broadly accessible via the common repo.
The header and cc files are copied here from the core repo (except for a few functions with non-trivial dependencies that stay in core), with the namespace changed to triton::common. A new library triton-common-model-config is built. Because many of the functions depend on the generated protobuf headers, this library is only built when the protobuf part of this repo is enabled.
This PR goes with https://github.com/triton-inference-server/core/pull/60.

@GuanLuo done

thanks for handling the last comment @jbkyang-nvi


Dimmed background blurring when switching between special workspaces

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit c5e28ebcfe00a510922779b2c568cfa52a317445  (props: bump ver 0.37.1).
Date: Sat Mar 16 14:51:49 2024
Tag: v0.37.1

flags: (if any)

System Information:
System name: Linux
Node name: D30
Release: 6.8.1-zen1-1-zen
Version: #1 ZEN SMP PREEMPT_DYNAMIC Sat, 16 Mar 2024 17:15:23 +0000

GPU information:
09:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 32 [Radeon RX 7700 XT / 7800 XT] [1002:747e] (rev c8) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Regression
Description
When switching between special workspaces, the blurred background seems dimmed a bit at first (which happens even when I set dim_special=0 in hyprland.conf) while toggle an existing one won't have this issue.
This issue occurs since version 0.33.0-1.
How to reproduce
Open multiple special workspaces, and switching between them.
Crash reports, logs, images, videos
https://github.com/hyprwm/Hyprland/assets/162549262/f505dd20-3661-47a4-9ffd-86312f0191f4

does it happen without blur too?

does it happen without blur too?

Yes, the same issue occurs and maybe more evident if no blur but transparency enabled

hm, sounds like double rendering. Is this a regression? does this happen in e.g. 0.34?

hm, sounds like double rendering. Is this a regression? does this happen in e.g. 0.34?

Okay, it seems to be a regression, happens on 0.33.0-1, but not on 0.32.3-1

could you git bisect?

Sorry that I'm not familiar with git, please tell me anything so that I can help...

Seems to be from the same time when the other bunch of double rendering issues happened. It was actually mentioned before here: https://github.com/hyprwm/Hyprland/issues/3888#issuecomment-1826900936

forgot about that, heh. Wanna fix or should I do it

you fix pls

Bruh use git bisect

no need

Another way to reproduce -
Hyprland, built from branch HEAD at commit<PHONE_NUMBER>014997776e091ff8903acfb311dd2f  (props: bump ver to 0.38.0).
Date: Mon Apr 1 18:30:37 2024
Tag: v0.38.0

Kooha-2024-04-03-16-00-41.webm

@BRS5672023  after 0.38.1, does this bug still happen for you? you can close the issue if it dosen't.

@BRS5672023 after 0.38.1, does this bug still happen for you? you can close the issue if it dosen't.

Yes, it still happens to me...

Yes, it still happens to me...

I downgraded to 38.0 and tried reproducing your bug ->
bind = $mainMod, S, togglespecialworkspace,
bind = $mainMod, R, togglespecialworkspace , what

Opened floating terminal on one and anki on other, switched between them. could reproduce what you have shown in the video.
I was able to reproduce the bug i posted above.
Not sure if the bug i was facing the same thing or not.

You could try to see if you can reproduce same thing with clean hyprland config with enough changes to reproduce the bug.

Yes, it still happens to me...

I downgraded to 38.0 and tried reproducing your bug ->
bind = $mainMod, S, togglespecialworkspace,
bind = $mainMod, R, togglespecialworkspace , what

Opened floating terminal on one and anki on other, switched between them. but could not reproduce what you have shown in the video.
I was able to reproduce the bug i posted above.
Not sure if the bug i was facing the same thing or not.
You could try to see if you can reproduce same thing with clean hyprland config with enough changes to reproduce the bug.

I'm not sure what do you mean for "floating", you mean floating in a special workspace?
workspace=special:dropterm,on-created-empty:kitty
workspace=special:dropterm1,on-created-empty:kitty

bind=$mainMod,T,togglespecialworkspace,dropterm
bind=$mainMod,M,togglespecialworkspace,dropterm1

With lines above, I could reproduce this bug on version 0.38.1-1 by alternatively toggling between two special workspaces.

workspace=special:dropterm,on-created-empty:kitty
workspace=special:dropterm1,on-created-empty:kitty
bind=$mainMod,T,togglespecialworkspace,dropterm
bind=$mainMod,M,togglespecialworkspace,dropterm1

With lines above, I could reproduce this bug on version 0.38.1-1 by alternatively toggling between two special workspaces.

ok, i could reproduce it too now, it wasn't working before because i had xray = true # blur section.

Is it still a thing?

Is it still a thing?

I can still repro it on AQ


vue-template-compiler 2.6.12 vue packages version mismatch when build

Version
2.6.11
Reproduction link
https://github.com/vuejs/vue
Steps to reproduce
npm run build
What is expected?
build success
What is actually happening?
vue packages version mismatch when build

Vue packages version mismatch:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>

We stumbled over the same issue. We are using renovate for dependency grooming though. After adding the package rule for the vue group the dependencies got updated together.
{
  "packageRules": [
    {
      "extends": "monorepo:vue",
      "groupName": "vue monorepo"
    }
  ]
}

Maybe that is a valid workaround for some of you.

Add the following lines to your Package.json
"vue-template-compiler": "^2.6.12"
and then run npm install in terminal.

Add the following lines to your Package.json "vue-template-compiler": "^2.6.12" and then run npm install in terminal.

thanks . it's worked for me


error: "infinite or missing values in 'x'

Hi,
I use data on a protein level and after classic criteria are calculated I get an error: "infinite or missing values in 'x'" and no results for PSS and no figures. I loaded the data as instructed in the manual, where for the first column I selected type "other" because instead of protein IDs, I'm using gene names. Do you know how I can fix this error?

Hi,
that is complicated and I guess you did not prepare your data well, could you send your data (the Expression data and the Samples information data) to me? I will check it for you and try it. Your data is safe here. My email<EMAIL_ADDRESS>Bests


ControlPlane-TenantManagement-TenantDelete: Accepts tenant ids that don't exist

Describe the bug
When running DELETE /tenants/{tenantId}, if tenantId doesn't exist, the service accepts the request, and writes a new record to the TenantDetails DynamoDB table.

Expected Behavior
When TenantID does not exist, the service should return 404.
Current Behavior
Returns 200 and writes new entry to TenantDetails table
Reproduction Steps
invoke DELETE /tenant/tenantId with tenantId that does not exist in the database
Possible Solution
No response
Additional Information/Context
No response
CDK CLI Version
2.131.0 (build 92b912d)
Framework Version
No response
Node.js Version
v18.19.0
OS
Debian GNU/Linux 12 (bookworm)
Language
TypeScript
Language Version
No response
Other information
No response

Thanks for reporting this!
Fix proposed in #11


Fails to format multiline doc comments containing comment end characters

Hi,
I tried using this for generating code based on some schema and the comments in that schema contain file glob patterns that contain the comment end characters */. It seems like this library fails to handle them although it works when just using the code inside a proc macro. See https://github.com/mohe2015/prettyplease-bug/blob/main/src/main.rs for a reproducible example.
Kind regards
Moritz

Thanks — Fixed in 0.1.21.


Minor recipe fixes

And make it noarch

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


Should toString for 4-vector components use rounding to 2-3 decimals?

Since the goal is mainly didactic for this tool, this might make sense.
Speeds (β) would be not be rounded, because the values of β are often so close to 1.

Now using 4 decimals.


Ensure types for TypedStruct in Tests

I try to use now the library for the first time and I really like the idea.
I am using it together with TypedStruct and see some weird behaviour in the tests
Environment

Elixir version (elixir -v): 1.12.3
Domo version (mix deps | grep domo | head -1): 1.3.4
TypedStruct version (mix deps | grep typed_struct | head -1): 0.2.1

Actual behavior
I have a module like (simplified)
defmodule Entity do
  use TypedStruct
  use Domo

  typedstruct do
    field :meta, map(), default: %{}, enforce: true
  end

  def config(data) do
    config = %{meta: data.meta}
    Entity.new!(config)
  end
end

In the iex console all works as expected, but when running the tests, I noticed some type inconsistencies. So I did a test.
I modified the typedstruct to take a wrong type
  typedstruct do
    field :meta, String.t(), default: %{}, enforce: true
  end

Again. In the console, running something like Entity.config(%{meta: %{k: :v}}) would fail, trying to respect the new String.t()
Invalid value %{hello: 1} for field :meta of %Faktor.Entity{}. Expected the value matching the <<_::_*8>> type.

But in tests, no change, all tests pass, and they are passing maps to the meta.
Expected behavior
Expected the tests to fail when the type is not valid
Is there anything I'm missing regarding running tests?

Hey, thanks for your feedback.
Can you, please, attach an example project to the issue? I'd like to see the test file

I created a minimal example that can reproduce the issue. That was a good exercise as I discovered that the issue mentioned above happens only in specific conditions:

having a __using__ macro that use Domo

https://github.com/iacobson/domo_example
To see it in action you can run the tests

this will fail: https://github.com/iacobson/domo_example/blob/main/test/domo_example_test.exs#L19
but if you remove this line https://github.com/iacobson/domo_example/blob/main/lib/domo_example.ex#L12, it will pass

In the iex console, everything works as expected

Thanks for the example project. Turns out that it was a mistake in the runtime mode detection when any module using Domo was added to the test .exs file. That kind of scenario is unsupported and Domo of version 1.4.0 raises an error about it.
To have structs using Domo specific to your test environment, add them into a separate directory and put the path to it in your mix.exs like the following:
def project do
  ...
  elixirc_paths: elixirc_paths(Mix.env())
  ...
end

defp elixirc_paths(:test), do: ["lib", "test/support"]
defp elixirc_paths(_), do: ["lib"]

Very nice, thanks!
So upgrading to 1.4.0 immediately raised the error.
Then I refactored as you suggested, and all works as expected.
This is the commit, in case anybody needs it:
https://github.com/iacobson/domo_example/commit/0625d13ddc48d340e76ba25163a4f68aafe3283a
From my point of view, I will close this issue.


remove a marker from a map

If I add a marker(s) via the .url so for example..
map.add(po.geoJson()
.url("...")
.on(.....)
and each feature has an id... so that gets assigned to the marker inside active scale element.
However how do I use map.remove, not having any joy with markers..
From testing about I can remove the necessary element with something as follows
element.parentNode.parentNode.parentNode.parentNode.removeChild(element.parentNode.parentNode.parentNode)
but despite the few+ attempts at map.remove.. I've had no success.
Thanks
Dave

I've been saving the result of po.geoJson()… in a variable and then passing that variable to map.remove().


Using OpenCV for localization.

After much struggling, it seems that we won't be able to use the modified QRCodeStateEstimation library on the final robot, due to the ZBAR library being incompatible with the BeagleBone, as well as our struggling to get BOOST library working to work.
Looking at the source code for that library though, it's not that complicated and we should be easily recreated in python.
All you really need is a set of (known) points, and to be able to identify them in the images captured by the webcam.
This is the part that made Zbar really simple. When it finds a QR code, it also tells you where the vertices are, and this is the part that I'm running into problems with now.
I was hoping to solve this problem using SURF points
The change in how we're approaching this is explained in my senior design presentation:

Basically the problem I'm running into is that even though I can now identify a bunch of known points on the QR code, there's still a few problems:

Not all of the SURF points are unique to each QR code.
When we find SURF points on the cameras captured frames, there are points on everything.
How do we isolate the points just on the QR code?
Is there something better than surf points to use?

Found this:
http://docs.opencv.org/3.0-beta/doc/py_tutorials/py_feature2d/py_matcher/py_matcher.html

This looks promising now
https://gist.github.com/AhmedSamara/0ce6a836dd45e16b68ba

It looks pretty promising.

Is this running on the BBB?
On Sun, Jan 24, 2016 at 5:43 PM, Ahmed Samara<EMAIL_ADDRESS>wrote:

[image: matches]
https://cloud.githubusercontent.com/assets/2405319/12539494/ebbd985c-c2c1-11e5-92b3-af8da39d49a5.png
It looks pretty promising.
—
Reply to this email directly or view it on GitHub
https://github.com/IEEERobotics/bot/issues/442#issuecomment-174349006.

http://stackoverflow.com/questions/7263621/how-to-find-corners-on-a-image-using-opencv
Here's another strategy for generic objects.

Two cameras might help, here are instructions for setting up the second:
http://ubuntuforums.org/showthread.php?t=168221


[één] Add support for multiple video's on 1 webpage

[x] I've verified and I assure that I'm running youtube-dl 2<IP_ADDRESS>
[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Feature request (request for a new functionality)

Thanks for adding support for één.be (#10605) 😃
But when you go to the review-link for Thuis, only the first video gets downloaded, but there are 5  video's available (at this moment).
Maybe also support for choosing which one you would like to download?

@dstftw Maybe you can look into this one since you also did #10605?


admission.certmanager.k8s.io/v1beta1: the server is currently unable to handle the request

Describe the bug:

Expected behaviour:

Certmanager DiscoveryCheck should be passed.
Steps to reproduce the bug:

Install Cert-manager.Below is the deployment manifest file. Run any kubectl command to see the error.
Anything else we need to know?:
Environment details::

Kubernetes version: 1.21
Cloud-provider/provisioner: AWS
cert-manager version: quay.io/jetstack/cert-manager-controller:v1.0.2
Install method: e.g. helm/static manifests : static manifests

/kind bug

cert-manager v1.0.2 is no longer supported, so you probably want to upgrade to a supported (ideally latest) version. Do let us know if you still see the issue then.

I'm seeing this issue installing v1.11.1
E0414 16:56:53.230112    2767 memcache.go:255] couldn't get resource list for metrics.k8s.io/v1beta1: the server is currently unable to handle the request

Installing with
helm install cert-manager jetstack/cert-manager --namespace cert-manager --create-namespace --version v1.11.1


Typo "Revise" instead of "Resolve patient duplicate"

In https://build.fhir.org/ig/IHE/ITI.PIXm/branches/master/ITI-104.html#23104421-trigger-events "The Revise Patient trigger" should be The Resolve patient duplicate trigger".

thanks for catching this


BACK button goes to previous page, not to list

Je suis sur une fiche RANDO, j'affiche sa carte puis de nouveau sa fiche.
En cliquant sur le bouton de RETOUR tout en haut de la page, cela me ramène à la page précédente (carte de la rando) et non à la liste des RANDOS.

This is intended. We return to the calling page of the modal. It's not always the list. It can be the map or another step of a tour ...

It is not the good way.
We go on a trek detail page, we check the map and then go back on detail.
When we click on BACK, we want to go on TREKS LIST, not on TREK MAP, as we just went on it and can clic on MAP button if we want to check MAP again...
This BACK button, should have only one action = GO BACK TO TREKS LIST.

I disagrea. Example :

you come from the main map with all treks and you show detail page. Back is expected to send back to map, not list

Not false. :-)
Didnt thought to this case.
Anyway the main case when you use it is when you check a trek detail page, go on map then go back on trek details and then want to check another trek.
So going back to PREVIOUS page is maybe not the perfect way.
But I understand with you case that just having one ACTION that would go back on list is not the good way.
We keep it for later. :-)


[dracula/duckduckgo] Side wiki font is unreadable

There is a bug on dracula/duckduckgo theme. As you can see on the picture, right-side wiki's font color is black and it is unreadable.
Opened issue since last year: https://github.com/dracula/duckduckgo/issues/1#issue-545308816
Example view:

Looking into this in the theme's repo issue.

Resolved.


Update GoDocs

GoDocs should be more user-friendly.
https://godoc.org/github.com/wmnsk/gopcua

[ ] ...

This is too ambiguous to be left as an issue.
More specific uer-friendly points should be reported as an individual issue.


checkErrorResponse documented in service tests

I think we should recommend using checkErrorResponse in the documentation. This will help new users in creating a good code from the beginning and should reduce amount of work in code review (https://github.com/badges/shields/pull/1645#discussion_r181862274, https://github.com/badges/shields/pull/1657#discussion_r183265223)

Warnings

:warning:

This PR modified the server but none of the service tests. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @platan!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS

Locally all server unit tests (main) are passing.

Re ran test again, passed this time.
Always seems to be the same 1 that occasionally fails on circleci.


Normalise Python typing syntax to 3.8+

What
This TBD PR attempts to normalise all python code to idiomatic, Python 3.8 compatible, syntax, primarily around typing ((through from __future__ import annotations), but also other aspect covered by PyUpgrade such as f-strings, etc.
This has wide ranging impact on the Python code, so could potentially mess-up existing PR.
TBD:

remove set -e from py-lint (annoying when errors are triggered by out-of-PR files, but impacts CI)

My take: leave as is. It's annoying when lints dont pass (as was the case as of ~ 370e16fc56ba0daecef13b7380d5cc6b6387091c), but this shouldn't be the norm.

from __future__ import annotations was "manually" added, so it's not done by py-format nor checked by py-lint.

My take: ok as is, but let's make it a concession to always add from __future__ import annotations in python files.

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[ ] I've included a screenshot or gif (if applicable)

PR Build Summary: https://build.rerun.io/pr/2361

Docs preview: https://rerun.io/preview/de126a6/docs
Examples preview: https://rerun.io/preview/de126a6/examples

Actually, Ruff appears to have something like that already: https://github.com/astral-sh/ruff/pull/1762
I'll try and add that.

Actually, Ruff appears to have something like that already: astral-sh/ruff#1762
I'll try and add that.

This can wait for a follow-up imo; I'd rather we merge this before monday kicks in, this is a ticking merge conflict bomb :grimacing:

Good point


Added support for Flutter 3.22

Hi, I've made a very small change to support Flutter 3.22

@dengyin2000 could you please review PR and merge it. It is solving error during compilation since bodyText2 is removed
for more info - https://docs.flutter.dev/release/breaking-changes/3-19-deprecations#texttheme
@aletorrado thank you for the fix.


Kitematic cannot download image

Hi,
I have an issue with Kitematic, i start it behind a proxy with the following command line.
http_proxy=http://Myproxy https_proxy=http://Myproxy open /Applications/Docker/Kitematic\ (Beta).app/
Kitematic launch perfectly and can connect on the Docker Hub without issue.

When i choose an image i cannot download it without issue message ...
I have this screen endlessly

It's strange because yesterday it works well....
Any idea why i have this issue...
I have this issue on MACOS and Windows 2008R2
Regards

Same here, I can't download either (no error message). And I'm not behind a proxy.

I am also getting this error. However I do get a message:

@wfowlks your error has nothing to do with the other 2 - You need to select the proper tag prior to creating the container.
You can do so by clicking on the ellipsis next to  'create' and selecting a valid tag.

Hi all,
No one have any solution for this issue... i have try to deeply unistall Kitematic & Virtualbox and reinstall... and i keep the same issue :(
Regards

@sreux If you look at the wiki information, you'll notice that the VM also needs to be setup with the proper proxy settings:
https://github.com/docker/kitematic/wiki/Common-Proxy-Issues-&-Fixes

Closing this issue due to lack of update/activity - Feel free to re-open if appropriate.


HandleWheel error

Hi I try this wersion with nextjs and react 16.2 and I have lot of errors in handleWheel function of react-virtual-scroller2
it on last line of of this code
key: 'handleWheel',
    value: function handleWheel(event) {

      var props = this.props;

stacktrace
commons.js:616 Uncaught TypeError: Cannot read property 'props' of undefined
    at handleWheel (index.js:279)
    at HTMLUnknownElement.callCallback (commons.js:548)
    at Object.invokeGuardedCallbackDev (commons.js:587)
    at Object.invokeGuardedCallback (commons.js:444)
    at Object.invokeGuardedCallbackAndCatchFirstError (commons.js:458)
    at executeDispatch (commons.js:842)
    at executeDispatchesInOrder (commons.js:864)
    at executeDispatchesAndRelease (commons.js:962)
    at executeDispatchesAndReleaseTopLevel (commons.js:973)
    at Array.forEach (<anonymous>)

Anyone else tries this with react 16.2?

We are using this with react 16.2.0 without that issue.
Can you console.log the this in the render function where handleWheel is called?
https://github.com/stevelacy/react-virtual-scroller2/blob/master/dist/react-virtual-scroller.js#L238

sure, this is undefined

here is the source of sample page served by nextjs
import React, { PureComponent } from 'react'
import DataGrid from 'react-datagrid2'

const defaultColumns = [
  { name: 'id' },
  { name: 'title' },
  { name: 'count' }
]

const createRows = () => {
  let rows = []
  for (let i = 1; i < 1000; i++) {
    rows.push({
      id: i,
      title: 'Title ' + i,
      count: i * 1000
    })
  }

  return rows
}

export default class Grid extends PureComponent {
  render () {
    const gridData = createRows()
    const gridColumns = defaultColumns
    return <DataGrid idProperty='id'
      dataSource={gridData}
      columns={gridColumns}
      style={{height: 400}}
    />
  }
}

As soon as I try to scroll, errors occurred. Chrome 64.0.3282.167 (Official Build) (64-bit)

Interesting on codesandbox it works without problems
https://codesandbox.io/s/n069yloop

It must be something with nextjs :-(

I am wondering... how does nextjs create the scope for the classes?
Is there a way to run nextjs in codesandbox?

It looks like it can be caused by HMR in dev mode

I'm just guessing, but maybe react-virtual-scroller2 missing removing listeners.
Elements should not be added in render...
https://github.com/stevelacy/react-virtual-scroller2/blob/master/src/index.jsx#L149

I know that it is strange, but when I explicitly bind events to this like this - line 146:
if (!hasTouch){
      events.onWheel = this.handleWheel.bind(this)
    } else {
      events.onTouchStart = this.handleTouchStart.bind(this)
    }

and then scroller at line 342:
var onScroll = this.onVerticalScroll.bind(this)
it starts working even in dev mode

want to send a PR?

https://github.com/stevelacy/react-virtual-scroller2/pull/2


Fix a long "loading terrain" screen with Player.setSkin()

When using Player.setSkin(), the player would get a "loading terrain" screen for a while, this pr fixes that by sending ChangeGameStatePacket(ChangeGameStatePacket.Reason.LEVEL_CHUNKS_LOAD_START, 0) after the respawn packet.

This is fixed in https://github.com/hollow-cube/minestom-ce/commit/e7b0e702f69f6c47a37c190ec260dc21bc8d06ee, but I did it inside refreshClientStateAfterRespawn to handle the other case that method is used.


Missing Latest CocoaPods Documentation

Please see https://github.com/CocoaPods/cocoadocs.org/issues/483.

Fixed.


Failed send logs to server

First time to use this, it fails to send data to server.
[09:19:19] Failed sending logs to server
Traceback (most recent call last):
  File "/app/gprofiler/log.py", line 167, in try_send_log_to_server
    self._api_client.post(self._path, data=self._logs[:logs_count], api_version="v1")
  File "/app/gprofiler/client.py", line 137, in post
    return self._send_request("POST", path, data, **kwargs)
  File "/app/gprofiler/client.py", line 130, in _send_request
    resp.raise_for_status()
  File "/usr/local/lib/python3.8/dist-packages/requests/models.py", line 960, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: https://profiler.granulate.io/api/v1/logs?key=****&service=****&hostname=****&timestamp=2022-07-14T09%3A19%3A19

Hi @gavin-jeong , this error refers to the debug logs (not the profiling data itself - your data should be visible in the Performance Studio despite this message).
We had several issues with the logging endpoint the past week, and profilers failed to send their logs, that's probably what you're seeing here. I'm sorry for the confusion. In any case, it is resolved now.

I'm closing as the logging endpoint is functioning okay and no further details are given here. If the problem returns and/or more details on the problem can be given, you may re-open the issue and continue the discussion.


Clicking the active tab removes focus from active shell session

Environment
Windows build number: 10.0.19042.0

Windows Terminal version (if applicable): 1.5.3242.0 (Preview)

Any other software? none

Steps to reproduce
With a tab active, click the tab in the title bar.
Expected behavior
The tab stays active, and focus remains in the shell session
Actual behavior
Focus is removed from the shell session.
Further details
I have searched issues for multiple different ways to describe this issue and cannot find any currently open issues. This issue is particularly annoying when switching applications into Windows Terminal by clicking the title bar and hitting the active tab. Expecting to be able to start typing and finding that you can't because focus is no longer in the shell.
Edit to include a gif of reproduction:

I think this is probably being tracked by issue #3609.

I think this is probably being tracked by issue #3609.

Sure is! Thanks!
/dup #3609

Sure is! Thanks!
/dup #3609


Add support for leaving stdin open on attach

What type of PR is this?
/kind feature
What this PR does / why we need it:
Making leaving stdin open work.
Which issue(s) this PR fixes:

Fixes #736
Special notes for your reviewer:
We have to disable critest for now until we support the new client API in CRI-O main.
Still verifying that it works in https://github.com/cri-o/cri-o/pull/6251.
Does this PR introduce a user-facing change?

Added support for leaving stdin open on attach.

/lgtm


fix: Fix module augmentation and install latest version of react-select again

This PR is a part two to #284 in which I fixed the broken module augmentation for projects using TypeScript with moduleResolution="bundler". I finally got some replies on my PR to the react select project: https://github.com/JedWatson/react-select/pull/5762#issuecomment-1762700437
And the maintainers basically explained that the method for implementing the module augmentation in the docs was outdated. The new approach is to augment them from the higher level (and already public) package react-select/base:

Thanks for the PR but these internal files aren't really intended to be public API and be augmented directly like this so I don't think this makes sense to merge.
The various types here can already be augmented by using the place where they're exported publicly. e.g. the Props type can already be augmented via react-select/base
import Select, { GroupBase } from "react-select";
import type {} from "react-select/base";

declare module "react-select/base" {
  export interface Props<
    Option,
    IsMulti extends boolean,
    Group extends GroupBase<Option>
  > {
    something?: string;
  }
}

<Select something="" />;

Here is an example project with broken module augmentation in v4.7.2, before I pushed a temporary fix that downgraded React Select's version to v5.7.0: https://codesandbox.io/s/323lhc?file=/app.tsx
You can see the working version generated below.

This is great news for ESM projects, thank you!