[keyrock] If themes are not enabled certificates' PVC is not mounted even if it is enabled in Values.yaml.

in Keyrock helm chart v0.1.3/v0.1.4 enabling certs PVC with certs.pvc.enabled = true creates the PVC but then it is not mounted as volume in the pod if themes are not enabled at the same time.
It seems that the following code in the statefulset template prevents the certs PVC mount, i.e. the {{- if .Values.theme.enabled }} condition:
          {{- if .Values.theme.enabled }}
          volumeMounts:
            - name: themes-theme
              mountPath: {{ .Values.theme.mountTheme }}
            - name: themes-img
              mountPath: {{ .Values.theme.mountImg }}
            {{- if .Values.theme.sourceFonts }}
            - name: themes-fonts
              mountPath: {{ .Values.theme.mountFonts }}
            {{- end }}
            {{- if .Values.theme.sourceFavicon }}
            - name: themes-favicon
              mountPath: {{ .Values.theme.mountFavicon }}
              subPath: "favicon.ico"
            {{- end }}
            {{- if .Values.certs.pvc.enabled }}
            - name: certs
              mountPath: /opt/fiware-idm/certs
            {{- end }}
          {{- end }}

Thanks for posting this issue.
This should be fixed now with this PR: https://github.com/FIWARE/helm-charts/pull/67
Could you check the chart release 0.2.3?


Health Check?

I'm getting this loaded into AWS behind an application load balancer, and one thing that would help is a way of doing a health check.
What I've done for now is just have the health check hit /, which returns a 400 error and the message:

Request URI must be absolute

In AWS, you can set which HTTP codes are OK, so I told it that 400's are OK, and it's working, I suppose, but it'd be nice to have a real healthcheck, even if it just returned a 200 code and "OK".
This seems like go 101, so I'll poke around a bit more and see if it becomes obvious to me.

Hm, I saw that prometheus seems to be supported (or maybe it's a WIP?), and gave that a try, but couldn't get the server to return anything useful. Didn't seem like the requests were going through, but it seems like a great solution to the issue, if it works.
Here's what I tried...

* Connected to localhost (<IP_ADDRESS>) port 2112 (#0)
> GET /api/v1/ HTTP/1.1
> Host: localhost:2112
> User-Agent: curl/7.81.0
> Accept: */*
>
* Recv failure: Connection reset by peer
* Closing connection 0
curl: (56) Recv failure: Connection reset by peer

I think that URL is correct (/api/v1/), but I tried the root path too and couldn't get anything different. Hm.


Feature Request: OSX support

As a user, I love 1Password integration in iOS apps (e.g Hipchat). I would love to see something similar made possible on OSX! Note: I have not dug in to how this works technically to understand if it is possible, so please feel free to tell me I'm crazy :-)

Thanks for taking the time to write in, @corydolphin :+1:
The 1Password App Extension API for iOS is heavily based on the NSItemProvider, NSExtensionItem, and UIActivityViewController  APIs provided by Apple. They ensure a very convenient and secure way for your iOS app to comminicate to the 1Password extension thanks to the UIActivityViewController 's completionWithItemsHandler, which is not present in the current Mac APIs.
This means that we can't rely on these Awesome Apple APIs for a 1Password API for the Mac. We'd have to write it from scratch ourselves. This is definitely something that we are considering and we want to make sure that our Mac API would be as simple and convenient to integrate as our iOS API. We also need to make sure that  this said Mac API is Mac App Store/sandbox friendly.
Hope that this helps :smiley:
Have an amazing weekend!


Per client or service configuration of sensitive-element-names and sensitive-protocol-header-names not honored

This week I discovered that our client specific configuration for sensitive-element-names stopped working after upgrading from 2.5.0 to 3.8.2 all the way back in May, i.e. config properties of the form quarkus.cxf.client."client-name".logging.sensitive-protocol-header-names
I did some incremental updates and when updating from 2.6.1 to 2.7.0.CR1 the configuration stopped working.
Comparing: https://github.com/quarkiverse/quarkus-cxf/compare/2.6.1...2.7.0.CR1 it seems that the structure of lines 121-122 of the old file was copied in LoggingFactoryCustomizer.java lines 78-79 without taking into account any configuration present in the PerClientOrServiceLoggingConfig  object and so that only the global config is added to the configured LoggingFeature.
As per client logging configuration is still documented I assume this must be a bug.

Substituting the lines 78-79 with the following lines will fix the issue:
        clientOrServiceConfig.sensitiveElementNames()
                .or(() -> global.sensitiveElementNames())
                .ifPresent(feature::addSensitiveElementNames);
        clientOrServiceConfig.sensitiveProtocolHeaderNames()
                .or(() -> global.sensitiveElementNames())
                .ifPresent(feature::addSensitiveElementNames);

And (I know this is goldplating 😃) I would suggest a similar approach in the addList method:
    private void addList(Optional<List<String>> global, Optional<List<String>> perClientOrService, Consumer<String> consumer) {
        perClientOrService
                .or(() -> global)
                .map(list -> list.stream().collect(Collectors.joining(";")))
                .ifPresent(consumer);
    }

Looking into this issue has been fun so far. I'll go ahead and create a pull request

@rudiger3d thanks for the report. I can reproduce and in between I have implemented a test.

I am releasing 3.8.7 and 3.15.2 right now.
@rudiger3d thanks again for reporting this security relevant issue. If you would like to get acknowledged in the associated CVE, please send me your name to ppalaga @ redhat.com
BTW, this made us aware that we did not have reporting of security issues enabled for this repository, which we have fixed in between: https://github.com/quarkiverse/quarkus-cxf/security/advisories
Please report any potentially security relevant issues there in the future.

Thanks for the quick response. I have sent an email (and added my name to my github profile)


Select Avalanche Center (debug) menu does not scroll vertically

Problem is in title.
cc @floatplane

Ugh, VS Code wants to suggest imports from tamagui even though it's not fully working (see #337)
If this happens again then I think we should remove tamagui from main until we're ready to actively work on it. Maybe I can configure ESLint to warn on imports from that package.

Updated #337 to add an ESLint warning for this


Bugfix/home page typo

typo

 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: saxena009 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

sorry


Monolog

Add Monolog for logging SOAP Requests/Reponses.
https://github.com/dafiti/correios/blob/master/src/Dafiti/Correios/Adapter/SoapAdapter.php#L72

The main log issue relates to the content sent and received from the api, with new validation this could be included as a configuration, but is not a priority at this moment.


parse doesn't match

Hi,
Thanks for this library!
So I started to write a little cli app with https://github.com/fatih/color, and tried to add some unit tests.
Eg. writing a color in my code looks like
...
import (
...
	"github.com/fatih/color"
...
)
...
colFunc = color.New(color.FgWhite, color.BgGreen, color.Bold).SprintFunc()
...
colored := colFunc("something")
...

Then later on I write that text t the console with another library github.com/gosuri/uilive, and it looks like white on green, but that doesn't matter, really.
Here I wanted to add some unit tests, because I have different styles and texts, and if I used the same color.New(... code, it doesn't look as good as parsing the output and checking out that it's really the same text with white on green background, yaaay!
So I found your library, wired it in the tests, aaand it's not always working. I can't tell if it's the same issue as the previous one.
So eg. here's what I got here in plain text: something but then parsing it with parsed, err := ansi.Parse(col) gives a parsed[0].FgCol something like:
&ansi.Col{Id:7, Hex:"#c0c0c0", Rgb:ansi.Rgb{R:0xc0, G:0xc0, B:0xc0}, Hsl:ansi.Hsl{H:0, S:0, L:75}, Name:"Silver"}

and here, result foreground name "Silver" doesn't match "White" what I'm trying to find here.
Also, Red is parsed as Maroon, Yellow as Olive and Blue as Navy.
Can you please help what can I do with this?

Best thing to do is add a test showing the problem then I can take a look 🙏

Ok, I just made the following replacements, and it works fine. Thanks a lot! :)
Also I checked out how it looks like, and BgHiRed is really more red than BgRed, so it totally makes sense to call the previous Maroon.
color.FgWhite => color.FgHiWhite
color.BgRed => color.BgHiRed
color.BgYellow => color.BgHiYellow
color.BgWhite => color.BgHiWhite
color.BgBlue => color.BgHiBlue


Structure Your Work Experience

Organize your work history in a clear and easy-to-read format, prioritizing the most relevant experiences for your target job.

creating bullet points highlighting achievements;
using strong action verbs;
quantifying results whenever possible.

I don't no what to discuss, so I added it just to pass the test.


Mac OS X Version Compilation Attempt

Hi,
I attempted to compile a version of Springseed on Mac OS X and when I run it with electron, I get this window:

The plus button does not work, and clicking and dragging in the white area makes my cursor into the text-editing one.
P.S. I'm not sure if this is actually an issue, or just something useful to know when porting Springseed.

This usually happens to me when I make a mistake in application.js file. Do you observe anything strange in the output? (start Springseed using a terminal and it prints out some output)

Only this, but I see it in lots of different applications:
Electron[19116] <Error>: The function ‘CGContextErase’ is obsolete and will be removed in an upcoming update. Unfortunately, this application, or a library it uses, is using this obsolete function, and is thereby contributing to an overall degradation of system performance.

Also, I managed to use the built version that you provided for download just fine.

@kman14367  Finally, what was the problem behind that weird UI? Also, I think you can close this issue now.


feat(helm): Added env variable SERVER_WORKER_AMOUNT

SUMMARY

I added the environment variable SERVER_WORKER_AMOUNT to the helm values file. It is commented out so it won't affect any running instances. We had severe performance problems caused by superset workers timing out. This was solved and the general Superset performance was improved by increasing the Server Worker Amount. Although this variable can have a huge impact on performance it is never mentioned, neither in the docs nor in the helm chart.
The standard value of 1 is set in run-server.sh.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Change the variable, at the start of the containers you can see the workers starting up in the logs.
ADDITIONAL INFORMATION

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

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

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

From my point of view I would also appreciate the consolidation of gunicorn related environment variables into a common prefix as some of them already start with GUNICORN_*. This makes the configuration interface more consistent and easier to read.
However this is a more breaking change as a switch from SERVER_WORKER_AMOUNT to e.g. GUNICORN_WORKER_AMOUNT might break existing settings and deployments.

From my point of view I would also appreciate the consolidation of gunicorn related environment variables into a common prefix as some of them already start with GUNICORN_*. This makes the configuration interface more consistent and easier to read.
However this is a more breaking change as a switch from SERVER_WORKER_AMOUNT to e.g. GUNICORN_WORKER_AMOUNT might break existing settings and deployments.

Totally agree, this can probably be done without breaking anything

Totally agree, this can probably be done without breaking anything

@dpgaspar what would be your idea for implementing this without breaking the existing configuration interface? A change of all variables to a common GUNICORN_* prefix might break configurations that users currently applied via the old environment variables.
Offering backward compatibility for these options from my point of view would require a lot of logic checking for the different variants and defining a set of precedence rules for the different options.
What was your idea?

Totally agree, this can probably be done without breaking anything

@dpgaspar what would be your idea for implementing this without breaking the existing configuration interface? A change of all variables to a common GUNICORN_* prefix might break configurations that users currently applied via the old environment variables.
Offering backward compatibility for these options from my point of view would require a lot of logic checking for the different variants and defining a set of precedence rules for the different options.
What was your idea?

Could be DRYer but something similar to:
if [[ -z "${SERVER_WORKER_AMOUNT}" ]]; then
  GUNICORN_WORKERS=${GUNICORN_WORKERS:-1}
elif [[ -z "${GUNICORN_WORKERS}" ]]; then
  GUNICORN_WORKERS=${SERVER_WORKER_AMOUNT}
fi

echo ${GUNICORN_WORKERS}

?


Testing on Windows Server 2008R2

TODO

Launch Windows Server 2008R2 machine using Azure
Confirm that this plugin works on that platform

@daipom  I'm going to check this this evening.

I tested this plugin on Windows Server 2008R2 (based on c71bba9178107c877dfaa46953e063361ec87409).
It worked. Here is the screenshot taken during the testing:
Screenshot

Thanks!

Addendum: The TD Agent version used in this testing was td-agent-4.3.0-x64.msi.


libpng warning: IDAT: Extra compressed data

I seem to be getting this warning on finishing writing a png with a StreamWriter:
libpng warning: IDAT: Extra compressed data

I can't seem to find any information about whether it is harmless or not. Doing the exact same operation, but not using a StreamWriter, does not produce a warning. What might this imply?
If it is truly harmless and unavoidable, it would be nice to have an option to suppress the warning.

Are you seeing any corruption in the image contents? If not, then it is probably harmless but still something that should be tracked down and fixed.

As far as I can tell, the image is identical, but I can't actually confirm that because my use-case has a bit of randomness to it.

This is caused by an off-by-one error in ChunkWriter::flush.
https://github.com/image-rs/image-png/blob/26f8f8bbc8dab63ba4aa34146016166203f7f9c0/src/encoder.rs#L401
Should be ..self.index. Since the buffer is extremely unlikely to be exactly full at the end, this will write one extra byte at the end of the zlib stream, causing "extra compressed data".

There's another much rarer error in ChunkWriter::write:
https://github.com/image-rs/image-png/blob/26f8f8bbc8dab63ba4aa34146016166203f7f9c0/src/encoder.rs#L387
If the index happens to be self.buffer.len() - 1, the buffer get written out with one extra byte. This causes a similar error, "invalid stored block lengths".

This is caused by an off-by-one error in ChunkWriter::flush.
https://github.com/image-rs/image-png/blob/26f8f8bbc8dab63ba4aa34146016166203f7f9c0/src/encoder.rs#L401
Should be ..self.index. Since the buffer is extremely unlikely to be exactly full at the end, this will write one extra byte at the end of the zlib stream, causing "extra compressed data".

There's another much rarer error in ChunkWriter::write:
https://github.com/image-rs/image-png/blob/26f8f8bbc8dab63ba4aa34146016166203f7f9c0/src/encoder.rs#L387
If the index happens to be self.buffer.len() - 1, the buffer get written out with one extra byte. This causes a similar error, "invalid stored block lengths".


Remove unsafe style cops

This is a partial revert of dbc3bc0d. These cops can result in false positives (as we've seen in
search-api) as described here [1].
https://rubocop.readthedocs.io/en/latest/cops_style/#stylehashtransformkeys

Sorry @bilbof, I was waiting as it looked like there were changes to push: I think we'll now get warnings about unconfigured cops again, unless I'm missing something?

@benthorner I thought this would get handled by setting enabled: false? https://github.com/alphagov/rubocop-govuk/pull/27/files#diff-eca608f03ea2a1932a1e7078d109a2d5R20

Ahhhh, I completely missed the move to a different file. Only minor comment would be to link to something (RuboCop docs in this case), so we know what we mean in the future. Hopefully the file won't change to much and we can find the commit/PR easily 👍 .

 Do we have a link to the false positives at all?  If we ever consider re-enabling this we'd want to have an example of what it was failing against so we can re-test that.

@kevindew I saw this on finder-frontend and search-api when bumping the gem versions. Here's an example: https://github.com/alphagov/search-api/pull/2018/files#diff-8612af24ee0bbe14a438c1bd6f9f01c1R11

@kevindew I saw this on finder-frontend and search-api when bumping the gem versions. Here's an example: https://github.com/alphagov/search-api/pull/2018/files#diff-8612af24ee0bbe14a438c1bd6f9f01c1R11

I'm a bit confused about the line in question, but I did a little bit of playing and it could be that I'm missing something but I'd suggest embracing the rule actually produces subjectivity better code (it's pretty grim the hash creation stuff going on in search-api). It's in this commit: https://github.com/alphagov/search-api/commit/6554794bd02ae2ed4d9d7de1b60d8c9a65d9eac0
Similar story in Finder Frontend: https://github.com/alphagov/finder-frontend/commit/3b6411a86ce472a05691a1dec54f8db6a12daf22 where the code looks improved by the linting rules.
Only case I saw of a false positive was https://github.com/alphagov/search-api/commit/6554794bd02ae2ed4d9d7de1b60d8c9a65d9eac0#diff-8612af24ee0bbe14a438c1bd6f9f01c1R107 but that was easily resolved with a to_h on the array


feat: Executors

According to #130 sylvia should provide reply_on method on the executor builder structure. From the sylvia point of view it's better not to duplicate the interface of SubMsg. Instead of adding reply_on() method on ExecutorBuilder, user should use build() method and then use on of the reply_* methods of SubMsg to create that strucutre.


Add X-Api-Info-Location header to the profile document

What is the problem this PR solves?
Closes #676
Checklist:

[x] The swagger.yaml doc has been updated with any required changes
[x] The openapi.yaml doc has been updated with any required changes

Thank you @mattmcneeney
While it looks good to me, and because this spec change just follows the de-facto implementation in Cloud Foundry, there is a possibility for a confusion. As @waterlink mentioned in the discussion, CF sends that header in a different format:
X-Api-Info-Location: api.example.org/v2/info

I personally like the proposed format with an explicit platform type mentioned in the value, besides, it is consistent with the Originating Identity header.

Ah, good point. I missed the bit about CF not sending the platform value today. @waterlink would this be easy to add?

The other thing I don't like about this is the fact that all other headers start with X-Broker, and this one doesn't, but since this header already exists today and may be being used by brokers registered with CF, I didn't feel like we should change this.

If this header is only supported by CF, the specification should only go into the profile document.

It is currently supported by CF, I think the goal is to make it optional for other platforms (k8s) as well.

I responded in the issue that there really isn't a k8s equivalent to this, for several reasons. I don't think adding this to the spec is a good idea.

That makes sense. Are you happy if I detail this header in the profile doc only?

Updated the PR, and also removed the header from the openapi and swagger docs since this isn't part of the core spec. cc @jberkhahn

Hey @vorbidan
We discussed this on today's call and spoke about whether or not we could, instead of using this existing header that CF sends, add a field to the context object along the lines of platform_url which is set to the URL of the platform making the request. That fits the existing pattern of sending platform-specific information on API calls.
What do you think?

@mattmcneeney
I agree - it makes the API cleaner and more explicit, besides it already has a place for platform-specific stuff.

I've moved this header to the profile doc. Reviews please!

Review please @fmui @tinygrasshopper

@zrob pointed out on the call today that with the CF CAPI v2->v3 transition going on, the location and contents of this header may change. We might want to wait until we have this functionality in v3.
Heroku also may have a URL and other info that they may want to make available to service providers. This would help their service brokers support multiple platforms where they need to call back into them.
@waterlink mentioned that service brokers will also need to know how to interact with the platform API. The context object does inform brokers of the platform name, but that is only retrievable on certain API calls.
We could introduce a second header informing the service broker of the platform that the URL relates too, or we could include it in the encoded header value, e.g.:
X-Api-Info-Location: cloudfoundry api.example.org/v2/info

Let's continue the discussion on here.

No objections on today's call on postponing this until the CAPI v3 transition is over.

@mattmcneeney
The X-Api-Info-Location header is not only necessary to identify distinct cloudfoundry platform clients, but also to enable brokers to discover the OpenIdConnect endpoints and CloudFoundry service instance permission authorization endpoint that are necessary to perform developer AuthN and AuthZ upon service dashboard web accesses.
Here are the current pains associated with using the CC API v2/info endpoint in the Osb cloudfoundry profile:

the endpoint is subject to change in CF API v3
service brokers have to make an outbound call to fetch the Json content, generating extra unnecessary network traffic
the endpoint response is lacking an href to the service instance permission endpoint. Service brokers have to make the assumption that the service instance permission endpoint lives on the same FQDN as the v2/info endpoint
the endpoint response has some content which is unrelated to OSB and should be ignored by service brokers, or redundant with other OSB spec fields

{
  "name": "",
  "build": "",
  "support": "https://support.run.pivotal.io",
  "version": 0,
  "description": "Cloud Foundry sponsored by Pivotal",
  "authorization_endpoint": "https://login.run.pivotal.io",
  "token_endpoint": "https://uaa.run.pivotal.io",
  "min_cli_version": "6.22.0",
  "min_recommended_cli_version": "latest",
  "app_ssh_endpoint": "ssh.run.pivotal.io:2222",
  "app_ssh_host_key_fingerprint": "e7:13:4e:32:ee:39:62:df:54:41:d7:f7:8b:b2:a7:6b",
  "app_ssh_oauth_client": "ssh-proxy",
  "doppler_logging_endpoint": "wss://doppler.run.pivotal.io:443",
  "api_version": "2.141.0",
  "osbapi_version": "2.15",
  "routing_endpoint": "https://api.run.pivotal.io/routing",
  "user": "526cc2f7-..."
}

I first considered having the X-Api-Info-Location header instead contain relevant content for brokers such as:
{
  "authorization_endpoint": "https://login.platform.example.com",
  "token_endpoint": "https://uaa.platform.example.com",
  "service_instance_permission_endpoint": "https://api.platform.example.com/v2/service_instances/:guid/permissions"
}

However, after a second thought, these fields might be better suited in the response of the  PUT /v2/service_instances/:instance_id endpoint as part of the CloudFoundry context
Benefits of including OIDC and Service-instance-permission endpoints in the context object on the provisionning endpoint:

May loosen the OSB API dependency to a specific CC API version (V2 vs V3), and delaying this PR: the OSB CF profile can be kept backward compatible between V2 and V3 if the V3 service-instance-permission endpoint returns the same payload
Easier for service brokers authors to consumme OSB (avoid extra network calls and figuring out useful data in /v2/infoendpoint )
Since the contract is standard (OpenIdConnect + simple REST endpoint), the broker dependencies on cloudfoundry is reduced, and this same contract can be used for some other clients.
The model can be made consistent with K8S clients, which could choose to return the K8S SubjectAccessReview webhook as service instance permission endpoint. In which case, the fields could be made directly available in the  PUT /v2/service_instances/:instance_id endpoint response body.

Removing the v2.16 milestone from this, and leaving this blocked given the comment above:

No objections on today's call on postponing this until the CAPI v3 transition is over.


bug: FlatscreenWindow position code is overridden by Registry on startup

https://github.com/StereoKit/StereoKit/blob/f8905b1f8745424a012f4c9d24e6ef28434b0dd3/StereoKitC/platforms/win32.cpp#L226-L236
If the window for the simulator has ever been moved the registry code will overwrite the target window position.
Note: What I would really like is for the stored window position to be on a per window basis, maybe based on appName or a custom window ID or similar?
User Story: As a multiplayer developer I want to be able to open multiple separate stereokit windows in diffent spots on my screen to be able to quickly iterate multiplayer sessions. Currently I need to manually move the windows each time I restart the app as all windows start at the position of the last closed StereoKit window.
PS:
If this takes some time to fix and anyone else for any strange reason needs this fixed. This works as a workaround to move the window after StereoKit starts:
    [DllImport("user32.dll", SetLastError = true)]
    internal static extern bool MoveWindow(IntPtr hWnd, int X, int Y, int nWidth, int nHeight, bool bRepaint);

    public void Init()
    {
        MoveWindow(Process.GetCurrentProcess().MainWindowHandle, Settings.flatscreenPosX, Settings.flatscreenPosY, Settings.flatscreenWidth, Settings.flatscreenWidth, true);
        // snip

Fix is inbound, this will work as expected for when SKSettings has specified any window positioning/sizing info! Will be in tomorrow's build.
Per-window may be nice, but I did actually choose not to do this intentionally! So I guess I'd probably enjoy some feedback on this thought:

Since the Simulator is a development tool, keeping the simulator in the same location for all apps in development would be helpful for a dev arranging their windows. Especially if that dev works on more than one SK app.

Since the Simulator is a development tool, keeping the simulator in the same location for all apps in development would be helpful for a dev arranging their windows. Especially if that dev works on more than one SK app.

Not sure if I agree that different apps debug windows should appear at the same space. But as you say, its a debug help thing so I'm ok with leaving this complexity for us us edge-case users

Yeah okay, I do see where you're coming from! I'm going leave this with the current fix, and let this marinate in my mind for a while and see what comes out of it. I still have some more code to do with the windows recovery feature in the near future, so may be more opportunities to improve it yet :)


after exiting, iframe still remains in the DOM.

const onExit = useCallback((error, metadata) => {
// log onExit callbacks from Link, handle errors
// https://plaid.com/docs/link/web/#onexit
console.log(error, metadata);

}, []);


HG: handle reference count too high after failed forward()

Describe the bug
This was discovered with a new Margo unit test https://github.com/mochi-hpc/mochi-margo/blob/main/tests/unit-tests/margo-comm-error.c.  The scenario is that the unit test deliberately creates a properly formatted but invalid na+sm address (https://github.com/mochi-hpc/mochi-margo/blob/main/tests/unit-tests/margo-comm-error.c#L149), looks up the address, and attempts to forward an RPC to the resulting address.
The error codes are all as expected (the lookup succeeds and the forward fails).  However, after completion of the failed forward attempt, the handle reference count is left at 2 when it is expected to be 1.  This causes a problem in Margo because it relies on the reference count to determine whether the handle can be returned to an internal pool for reuse (https://github.com/mochi-hpc/mochi-margo/blob/main/src/margo-core.c#L756).  Ultimately the handle is leaked because Margo loses track of it at that point.  The unit test therefore works correctly but fails a memory leak analysis.
To Reproduce
I don't have a pure Mercury reproducer, but I think I can create one if needed.
Expected behavior
The resulting handle count after forward() should be the same whether the operation succeeded or failed, so that users can gracefully handle the failure.
Platform (please complete the following information):\

Ubuntu 21.10
gcc 11
na+sm plugin

Thanks @soumagne !  I confirmed that my problem is fixed in rc3.


Add Error Prone code style verification

See https://errorprone.info/
It allows to capture errors like mising switch case branches,
missing override, non-static inner class, etc
The verification can be run locally via ./gradlew -PenableErrorprone classes
Note: Java 11+ is required
The changes do not seem to be very big for me, however, it seems to flag important issues: it forbids the use of deprecated methods, it flags the use of the default encoding, it suggests to avoid LinkedList, and so on.
I fixed the warnings by running classes -PenableErrorprone -Pwerror=false --rerun-tasks in my IDE, so it printed all the warnings, then I could click on each error to navigate and resolve it.
The biggest commits are
91851b17974b1fd2744414ada6b2ac7937e82735 Replace LinkedList with ArrayList
89 files changed, 249 insertions(+), 241 deletions(-)
d668dd388105b6b8d297c937026980e54fe8764c Remove uses of deprecated APIs
52 files changed, 154 insertions(+), 90 deletions(-)
be8438d2cfd517dda856d2674bfff096b8a0445a Suppress DefaultCharset usage: either suppress the warning or use a charset
27 files changed, 85 insertions(+), 94 deletions(-)

Codecov Report

Merging #632 into master will increase coverage by 0.04%.
The diff coverage is 68.27%.

@@             Coverage Diff              @@
##             master     #632      +/-   ##
============================================
+ Coverage     55.30%   55.35%   +0.04%
+ Complexity    10091    10088       -3
============================================
  Files          1038     1036       -2
  Lines         63858    63779      -79
  Branches       7229     7212      -17
============================================
- Hits          35314    35302      -12
+ Misses        26049    25983      -66
+ Partials       2495     2494       -1

Impacted Files
Coverage Δ
Complexity Δ

...n/java/org/apache/jmeter/util/BeanShellClient.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...va/org/apache/jmeter/assertions/HTMLAssertion.java
20.56% <0.00%> (ø)
15.00 <0.00> (ø)

...org/apache/jmeter/config/RandomVariableConfig.java
71.42% <ø> (ø)
23.00 <0.00> (ø)

...va/org/apache/jmeter/control/ModuleController.java
70.12% <ø> (ø)
21.00 <0.00> (ø)

...ctor/json/render/AbstractRenderAsJsonRenderer.java
84.21% <0.00%> (ø)
18.00 <0.00> (ø)

...er/extractor/json/render/RenderAsJsonRenderer.java
91.30% <ø> (ø)
9.00 <0.00> (ø)

...ava/org/apache/jmeter/modifiers/CounterConfig.java
91.02% <ø> (ø)
35.00 <0.00> (ø)

...va/org/apache/jmeter/modifiers/UserParameters.java
64.00% <0.00%> (ø)
15.00 <0.00> (ø)

...a/org/apache/jmeter/timers/PoissonRandomTimer.java
72.97% <ø> (-5.41%)
9.00 <0.00> (-1.00)

...issonarrivals/ConstantPoissonProcessGenerator.java
58.57% <ø> (ø)
14.00 <0.00> (ø)

... and 284 more

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 5d34a52...967ac65. Read the comment docs.

@pmouawad , the PR passes tests, and it looks like it is ready for review and merge.

My biggest concern with this PR at the moment is its size. Could we add the feature (errorprone) without changing every nag it shows? I tend to get bored when looking at big PRs and either skip parts, or abort looking at it altogether.
Apart from that, I am +1 on adding the feature (if it doesn't break the build when running with Java 8). (Haven't tried it)

if it doesn't break the build when running with Java 8

All the previous CI jobs pass, so it works.
In practice, it is activated only in case -PenableErrorprone is added as a command-line option or gradle.properties setting.

Could we add the feature (errorprone) without changing every nag it shows?

Well, most of the warnings are true issues. There is an unfortunate case when the tool discourages the use of Hashtable, Enumeration, and LinkedList in the same check.
Nobody uses Hashtable nowadays unless that is required for an ancient API, however, people can easily use LinkedList without realizing it is way slower and less efficient than ArrayList and ArrayDeque


Bso trivia

🐛 Bug
Bso trivia didn’t give box 2x in a row for correct answers.

That’s the test bot, not BSO :)

That’s the test bot, not BSO :)


Make 'tcp://' optional when reading DOCKER_HOST

This better matches the behaviour of docker itself, so it avoids the
problem of "hey, docker works, why doesn't dockerode?" and some hair
pulling.
@apocas hopefully this PR works out better than #43 did ;-)
/cc @altsang @sam-github

@apocas Anything more I can do to help get this merged/released?

Ehehe I think this one will definitely work better :)


After updating, UI displays a blank screen but devices are online and working/reporting

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
<IP_ADDRESS>b831e2
ZwaveJS version
11.14.2
Describe the bug
Docker running on a seperate Pi4 not HA add-on, after updating to latest docker image UI displays a blank screen but zwave devices report and function normally
To Reproduce
Attempt to access the UI via HTTPS
Expected behavior
UI presents a the usual login page
Additional context
I had replaced the batteries in a number of dead nodes (radiator TRVs not in use over the summer) and noted that HA had lost connection to all Zwave devices. Stopped & deleted the container then pulled a fresh image. Zwave devices came online for HA but UI is inaccessible.
apt update/upgrade performed on Pi and rebooted. Docker image stopped, deleted and re-pulled with same outcome. UI displays a blanks screen.
To be clear the back end functionality is working, devices are reporting to HA and are controllable as normal but UI is not functioning.
Tried Firefox and Edge with the same result
Browser log:
browser console.log
Settings.json
settings.txt
Docker console
docker console.txt
Messages logged when UI is accessed:
zwave log.txt

Ah great, thanks, I did search but obviously not well enough, nice one


Change Possible from Prism to Optional and add link in website

fix #450

Merging as @kenbot is the most familiar with this code


bar plot bug fix for fetching colors from layers

Updating the get_colors() definition to fix the bar plot plotting bug.

changed the label_layers[0] to label_layers[1] in "get_stats" function in _dock_widget file to fixes bag plot bug


Failure to simplify units in very simple case

> s = 34 gallons to L
128.704000656 L
> k = 10.7 kwh / L
10.7 kWh / L
> s * k
1377.1328070192 L kWh / L

Thanks for the report. The reason this fails to simplify is that because you're first converting s explicitly to a specific unit, fend tracks that s should not be auto-simplified further. You can see this by running @debug s, which shows not simplifiable as one of the flags. This flag is set whenever you use to to explicitly convert to a specific unit.
Multiplying s with another value carries over this not simplifiable flag, causing fend to not simplify s * k.
If you instead calculate e.g.:
> s = 34 gallons
34 gallons
> k = 10.7 kwh / L
10.7 kWh / L
> s * k
1377.1328070192 kWh

fend manages to correctly simplify the units.
I'm not sure how to best fix this. One option would be to automatically drop the flag when the units change due to multiplication or such, but I'll need to carefully test to make sure this doesn't cause unintentional regressions.

I think it would be appropriate to simplify it anyway if the unit is exactly the same, i.e., not not just the same quantity but also the same actual unit. Also, what is the intended value of this "not simplifiable" flag and is it possible to disable with config?

The flag is important because it prevents fend from auto-simplifying any calculations that are explicitly converted to a specific unit with to.
For example:
> 4 watts * 3 seconds
12 joules
> 4 watts * 3 seconds to N m
12 N m

It makes it possible to use units like Newton-metres without having them be auto-converted to joules.
But anyway, reenabling auto-conversions after multiplying values (or doing other maths operations) does make sense I think, to make sure that calculations like (34 gallons to L) * (10.7 kwh / L) work properly.


Add python3-parameterized key for rclpy test

Debian https://packages.debian.org/buster/python3-parameterized
Fedora https://apps.fedoraproject.org/packages/python3-parameterized
Gentoo https://packages.gentoo.org/packages/dev-python/parameterized
Ubuntu https://packages.ubuntu.com/bionic/python3-parameterized
Test code for rclpy  depend on python3-parameterized (refer to https://github.com/ros2/rclpy/pull/469).
So apply to add this dependency package.

@nuclearsandwich
Thanks for your review.
But based on advice from dirk-thomas (https://github.com/ros2/rclpy/pull/469#discussion_r353249166), I decide to drop this PR.


When I run python caffe_client.py cifar10 unique_client_identifier server_ip_addr -gpu 4？ I find an error

When I run python caffe_client.py cifar10 unique_client_identifier server_ip_addr -gpu 4
There is an error:  Connection failed - goodbye!
Why this error happened？

Make sure you have the q_server running. Try running the q_server and the client on the same machine to ensure that it is not a connection issue on your end.
Let me know if you have further issues!

Thank you very much!  I have solved this problem.


Red test failure output in editor gets wrapped over lines

VS Code version
1.61.2
Extension version
2021.11.1422169775
OS type
Linux
OS version
Linux x64 3.10.0-1160.45.1.el7.x86_64
Python distribution
Anaconda
Python version
3.9.0
Language server
Pylance
Expected behaviour
Test failure output should only show on single line. It should be truncated or something, not wrap over multiple lines.

Logs
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Sorry for not leaving a response on this. This should be something that can be handled better when we re-write the pytest adapter. #17242

I can no longer reproduce this

Closing this for now.


[cxx-interop] allow shared ref retain function to return self

Many existing C APIs for retaining references, including Apple's own, return the reference. Support this pattern, along with the existing void return signature, with when importing reference types from C++.
More: https://forums.swift.org/t/swift-shared-reference-signature-error/76194/7

Note to self: need separate error message for invalid retain return value.

Tests are now failing, investigating.

Tests passing again.

LGTM

Just double checking we don't need to do anything extra at the retain call site to ignore the return value.

@swift-ci please smoke test macOS

@swift-ci please smoke test

What's the next release one might expect to see this in?

Once it's merged to main, we can cherry-pick to the release/6.1 branch for the 6.1 release (Q1 2025)

@swift-ci please smoke test

@swift-ci please smoke test Linux

@lhoward I'm going to merge now. Would you cherry-pick this to the release/6.1 branch? There are instructions at https://forums.swift.org/t/swift-6-1-release-process/75442#p-344524-pull-requests-for-release-branch-8


Always set original and resulting organization when creating change events

The data model expects these properties to be set most of the time (with some exceptions), although it wasn't needed for the frontend to work properly.

I converted it back to draft because Yassin needs to confirm if the current logic is ok.

I converted it back to draft because Yassin needs to confirm if the current logic is ok.
Is it ready to be reviewed?

@nbittich yes, sorry I was going to ping you after the standup but I forgot :smile:.


attempting to call undeclared routine: 'hasCustomPragma'

Seems to raise error at compile time.
C:\Users\\\nimdash_hold\experiments\scorpertest.nim(19, 4) template/generic instantiation of `addRoute` from here
C:\Users\\.nimble\pkgs\scorper-1.0.13\scorper\http\routermacros.nim(12, 15) Error: attempting to call undeclared routine: 'hasCustomPragma'

import scorper

const port{.intdefine.} = 8888
when isMainModule:
  type AsyncCallback = proc (request: Request): Future[void] {.closure, gcsafe.}

  proc handler(req: Request) {.route("get","/one"),async.} = discard
  proc handler2(req: Request) {.route(["get","post"],"/multi"),async.} = discard
  let r = newRouter[AsyncCallback]()
  r.addRoute(handler)
  r.addRoute(handler2)

  const address = "<IP_ADDRESS>:" & $port
  const flags = {ReuseAddr}
  var server = newScorper(address, r, flags)
  server.start()
  waitFor server.join()

close in https://github.com/bung87/scorper/commit/d1f9400386921b90e1a272ad7f0f94a6f319f866


Revert "[suggestion] chore: add annotation typescript usage in README"

Reverts msgpack/msgpack-javascript#136
Since #139 resolved this problem.
cc: @bananaumai

Codecov Report

Merging #142 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #142   +/-   ##
=======================================
  Coverage   98.30%   98.30%
=======================================
  Files          15       15
  Lines         943      943
  Branches      205      205
=======================================
  Hits          927      927
  Misses         16       16

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 d60d4e1...ba38734. Read the comment docs.


Horizontal scroll bar not automatically hidden when no text is out of bounds

Lapce Version
0.2.8
System information
Windows 11
Describe the bug
In the code editor pan, if the lines do not exceed horizontal space available, the horizontal scroll bar does not appear, of course, once the line gets too long and get out of bounds, the bar appears, and you can scroll.
The issue is that, if for example you split the long line and stay in bound, the horizontal scroll bar still does not disappear, even though it is not needed. VSC has this behavior correctly, you will know for sure there is nothing left at the end of the line if this behavior is corrected to have the bar disappear once text is back in bound. If you restart the application the bar goes away, but it should have been gone automatically. This is not really a breaking bug, nor does it feel like a feature, something to fix, hope someone attends to this, I hope I have been clear, I have attached a video, I could tackle this myself too actually, if given the chance, thank you.
https://github.com/lapce/lapce/assets/81725875/6dbe6afe-85b8-40c4-bd42-8e83b048eb35

This issue is fixed in master already

I have built from masters and the horizontal scrollbar related issue does indeed seem to be fixed.
However some other features from 0.2.8 seems to not be in master or rather bugs present in master. Will open new issue for that.
Anyway, as mentioned, issue seems to be fixed, I will now send issue for closing.

Will open new issue for that.

Please don't. #2417

gotcha, I missed the pin, have not actively checked repo in a while (is there actually another way to stay up to date on what's happening to Lapce? I know less platforms the better, just asking. I am new to git etc)
Some stuff seems to be missing in the task list in the migration task list. I will mention it as a comment in the pinned issue. How to actually contribute to this repo, I found some very easy to fix (actual) bugs.

we are most active on Discord, 2nd place would be here (GitHub Issues and Pull Requests), anywhere else it's mostly major news


Support for Dialog Deligation

Issue #125

Thank you!

Would love to see this feature on PyPI. Any idea when a new version will be pushed?
Thanks!

Updated to 0.9.7 which includes your PR. Thanks again

@johnwheeler Hi, I'm trying to use this feature, but I couldn't find related to this pull request. how can I use deligation for slot filling?


Move optional Jakarta specific MP JWT spec content to JWT bridge spec doc

@dblevins @ayoho Can you please look into it ? I can help with the PR, is it what we want to move,
https://github.com/eclipse/microprofile-jwt-auth/blob/main/spec/src/main/asciidoc/interoperability.asciidoc#recommendations-for-optional-container-integration
?
Also CC @starksm64

I'll give this one a try

Hi Adam, please go ahead with this one when you get a chance, I'll look at #13

Closed by #14


[Feature Request]: prBot should allow the author to request a new reviewer

What would you like to happen?
Currently prBot will only allow an assigned reviewer to assign a new reviewer. It would be nice if the author could also ask to assign a new reviewer, e.g. if they know that the current reviewer is on vacation and unlikely to respond for some time.
Issue Priority
Priority: 2
Issue Component
Component: infrastructure

It took me a while to figure out why prBot didn't respond to an assign to next reviewer comment from the author. It's because here: https://github.com/apache/beam/blob/8fb494544d7a3e1276372cb6710798b96c330d86/scripts/ci/pr-bot/shared/userCommand.ts#L79
we lookup the label that the comment author is assigned to review for. The author is not assigned to review any labels.

A couple of possible ways to address this:

let the author specify a reviewer to unassign, and/or
if there's only one reviewer unassign them

respect the "busy" parameter in user's GitHub status

I don't think that's enough on it's own though, since people often don't set that status.

Just got bit by this...
Once a user manually assigns a reviewer, PRbot sets: stopReviewerNotifications = true, which causes PRbot to stop any further processing.
Suggestion: Allow PRBot to look for the commands when stopped, but only handle "assign set of reviewers" which will reset stopReviewerNotifications to false


[S3] IPC stack too low (IDFGH-6452)

Hi,
here is situation i am trying to test:

custom event loop
task sending events
task is running esp_event_loop_run
very important: tick rate = 1000 and CPU freq = 240MHz (in all??? other cases code works fine)
when i stop posting events, code works fine
second custom event loop created with a task, so no need to call esp_event_loop_run
when i disable second event loop, code works
when i disable logs in event loop handler, code works
few more combinations of parameters are giving different outputs, either crash or works
esp-idf: master branch
on old esp32 the same code seems to work fine

Rebooting...
�ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0xc (RTC_SW_CPU_RST),boot:0x8 (SPI_FAST_FLASH_BOOT)
Saved PC:0x42004692
0x42004692: find_desc_for_int at /home/chegewara/esp/master/components/esp_hw_support/intr_alloc.c:131

SPIWP:0xee
mode:DIO, clock div:1
load:0x3fcd0108,len:0x1724
load:0x403b6000,len:0x8c8
load:0x403ba000,len:0x2ed8
entry 0x403b61c0
I (29) boot: ESP-IDF v5.0-dev-676-g5c33570524-dirty 2nd stage bootloader
I (29) boot: compile time 20:02:32
I (29) boot: chip revision: 0
I (33) boot.esp32s3: Boot SPI Speed : 80MHz
I (38) boot.esp32s3: SPI Mode       : DIO
I (42) boot.esp32s3: SPI Flash Size : 2MB
I (47) boot: Enabling RNG early entropy source...
W (52) bootloader_random: RNG for ESP32-S3 not currently supported
I (59) boot: Partition Table:
I (63) boot: ## Label            Usage          Type ST Offset   Length
I (70) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (78) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (85) boot:  2 factory          factory app      00 00 00010000 00100000
I (93) boot: End of partition table
I (97) esp_image: segment 0: paddr=00010020 vaddr=3c020020 size=08130h ( 33072) map
I (111) esp_image: segment 1: paddr=00018158 vaddr=3fc90970 size=02718h ( 10008) load
I (116) esp_image: segment 2: paddr=0001a878 vaddr=40374000 size=057a0h ( 22432) load
I (127) esp_image: segment 3: paddr=00020020 vaddr=42000020 size=17bc0h ( 97216) map
I (148) esp_image: segment 4: paddr=00037be8 vaddr=403797a0 size=071c8h ( 29128) load
I (155) esp_image: segment 5: paddr=0003edb8 vaddr=50000000 size=00010h (    16) load
I (160) boot: Loaded app from partition at offset 0x10000
I (160) boot: Disabling RNG early entropy source...
W (166) bootloader_random: RNG for ESP32-S3 not currently supported
I (184) cpu_start: Pro cpu up.
I (184) cpu_start: Starting app cpu, entry point is 0x403750ec
0x403750ec: call_start_cpu1 at /home/chegewara/esp/master/components/esp_system/port/cpu_start.c:164

I (162) cpu_start: App cpu up.
I (198) cpu_start: Pro cpu start user code
I (198) cpu_start: cpu freq: 240000000 Hz
I (198) cpu_start: Application information:
I (201) cpu_start: Project name:     events
I (206) cpu_start: App version:      5a0dba2-dirty
I (211) cpu_start: Compile time:     Dec 17 2021 20:02:28
I (217) cpu_start: ELF file SHA256:  cf57a8a58399ff66...
I (223) cpu_start: ESP-IDF:          v5.0-dev-676-g5c33570524-dirty
I (230) heap_init: Initializing. RAM available for dynamic allocation:
I (238) heap_init: At 3FC93AE0 len 0004C520 (305 KiB): D/IRAM
I (244) heap_init: At 3FCE0000 len 0000EE34 (59 KiB): STACK/DRAM
I (251) heap_init: At 3FCF0000 len 00008000 (32 KiB): DRAM
I (257) heap_init: At 600FE000 len 00002000 (8 KiB): RTCRAM
I (264) spi_flash: detected chip: generic
I (268) spi_flash: flash io: dio
W (272) spi_flash: Detected size(8192k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (285) sleep: Configure to isolate all GPIO pins in sleep state
I (292) sleep: Enable automatic switching of GPIO sleep configuration
I (299) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.

assert failed: esp_ipc_isr_init_cpu esp_ipc_isr.c:73 (uxTaskGetStackHighWaterMark(NULL) > 128)

Backtrace:0x40375a4a:0x3fcf12500x40378f09:0x3fcf1270 0x4037ee91:0x3fcf1290 0x42001e3a:0x3fcf13b0 0x4037bb91:0x3fcf13d0
0x40375a4a: panic_abort at /home/chegewara/esp/master/components/esp_system/panic.c:403

0x40378f09: esp_system_abort at /home/chegewara/esp/master/components/esp_system/esp_system.c:123

0x4037ee91: __assert_func at /home/chegewara/esp/master/components/newlib/assert.c:85

0x42001e3a: esp_ipc_isr_init_cpu at /home/chegewara/esp/master/components/esp_system/port/arch/xtensa/esp_ipc_isr.c:73 (discriminator 1)

0x4037bb91: vPortTaskWrapper at /home/chegewara/esp/master/components/freertos/FreeRTOS-Kernel/portable/xtensa/port.c:130

I understand it may be edge case, and i am not sure if it require to be fixed, but i thought i will report it.
Thanks
EDIT: when i add this line of code in app_main then no crash anymore:
printf("min stack size: %d\n", configMINIMAL_STACK_SIZE);

This fixed issue (increase stack by 10 bytes):
void esp_ipc_isr_init(void)
{
    for (unsigned i = 0; i < portNUM_PROCESSORS; ++i) {
        portBASE_TYPE res = xTaskCreatePinnedToCore(esp_ipc_isr_init_cpu, "ipc_isr_init", configMINIMAL_STACK_SIZE + 10, NULL, 5, NULL, i);
        assert(res == pdTRUE);
        (void)res;
    }
}

@chegewara Thanks for the investigation, really appreciated! We are aware of this issue already. Most likely there are some configurations still not considered when configuring the minimal stack size. Or the minimal stack size is wrong altogether. We are aware of this issue and working on it, possibly making it a Kconfig option at first. If you have any new insights, please let us know!

Im glad that espressif is working on it, and yes, Kconfig option seems to be good start.
Thanks

will this be integrated in Release 4.4 ?
so a 4.4.1 ?
Thanks !

@franz-ms-muc Thanks for the report, the fix doesn't make into IDF v4.4 release, currently we are back porting this to release/4.4, will update once the fix on release/4.4 is available, thanks.

aha, got into this issue today (building from v4.4 of 16Feb22), will manually increase the stack of ipc_isr_init for the time being.
Please update when fix is into v4.4

Thanks for reporting and sorry for slow turnaround. The fix is included in v4.4.1, https://github.com/espressif/esp-idf/releases/tag/v4.4.1. Thanks.


[5455] Revert qts awards and show award reversal in timeline

Context
In this pull request, we address an issue where 127 trainees were mistakenly marked as awarded by their providers. The proposed changes aim to revert this incorrect state and ensure that the reversal is properly reflected in the trainee timeline.
Changes proposed in this pull request
The following changes are included in this pull request:

Data Migration:

A data migration is added to remove the award state for a specific list of trainee slugs. This migration will correct the mistakenly awarded trainees by removing the award status from their records.

Logic for Handling and Displaying Reversed Awards:

The necessary logic is implemented to handle and display reversed awards correctly. This ensures that the trainee timeline accurately reflects the reversal of the mistakenly awarded status.

Guidance to review
To review the changes in this pull request, please follow these steps:

Run the migration against production-like data:

Execute the provided data migration script against a production-like dataset to verify its correctness and effectiveness in removing the award state for the identified trainees.
bin/rails data:migrate

Check the timeline of an updated trainee:

Select a trainee from the affected list and review their timeline. Ensure that the reversal of the awarded status is accurately displayed and that the timeline reflects the corrected state.
Test on <IP_ADDRESS> - nPsk5npdB2wiJNrYUkdAHuSG

By carefully reviewing and validating these changes, we can ensure that the issue of mistakenly awarded trainees is resolved, and the trainee timeline accurately represents the corrected state.
Timeline

With subsequent deferral

Important business

[x] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 [S] Update trainee timeline to handle undo of awards

 Revert QTS Awards

 Move trainees accidentally awarded QTS back to TRN received.


Update dependency: Result 3.2.4 -> 4.0.0

Moya 11.0.2
CocoaPods

Yeah, @cooler333, are you willing to make a PR with the change? The target would be our development branch as this would be a breaking change (major upgrade of a dep)

This is done by #1668


Expose a parameter in takeScreenshot() to customize screenshot file name.

From<EMAIL_ADDRESS>on June 12, 2012 00:59:28
Could you please expose a parameter in takeScreenshot() method  so that we can customize screenshot file name?
We have so many screenshots to capture and need to name it following some rules.
Original issue: http://code.google.com/p/robotium/issues/detail?id=277

From<EMAIL_ADDRESS>on June 12, 2012 02:44:20
Sure. Will be available in the next release.
Status: Accepted
Owner<EMAIL_ADDRESS>Labels: -Type-Defect Type-Enhancement

From<EMAIL_ADDRESS>on August 07, 2012 01:52:17
This has been included in 3.4.
Status: Verified


Allow phoenix_html 4.0 as a dependency

I am not sure where Phoenix.HTML gets used, but seems like its only a dependency to ensure the test still works.
Updating this allows us to upgrade phoenix_html to 4.0 in phoenix apps using this package.

Thank you @wkirschbaum! Merging this and releasing a version with the fix.


nixos/azure: add Gen 2 VM, aarch64 and accelerated networking support

Description of changes
[azure-agent]

Removed x86 assertion as the latest waagent already works on aarch64.

[azure-common]

Added option to enable support for accelerated networking, following official doc for custom vm image: virtualisation.azure.acceleratedNetworking = true.
Remove reference of headless profile to make the console input work (and being able to select boot entries).

[azure-image]

Added option for generating Gen 2 VM image (efi partition): virtualisation.azureImage = "v1" or "v2".
Added option for specifying esp partition size: virtualisation.azureImage.bootSize

I have created an example flake for generating a test vhd for this change: github:codgician/azure-aarch64-nixos.
The result is promising:

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4451

thanks for your work!
Given the modules haven't been touched since 2016, I propose to merge your work once you confirm that everything is working as intended.

Assuming that the :+1: means it's all good

Thanks for having tested this with a gen v1, I wonder if there is a way to access systemd-boot's generation for gen2 during boot, I haven't seen any documentation about that, I believe most Azure images are using GRUB for gen v1 or v2.

@AkechiShiro Yes, it's possible.
For example, I use this config to achive with systemd-boot on an Azure V2 VM (on https://github.com/rudesome/nixos-on-azure/ instead of on this PR, since I'm on nixos-24.05, but the result should be equivalent):
    # Note that using `systemd-boot` breaks `switch-to-configuration boot`
    # (but not `nixos-rebuild`), see https://github.com/NixOS/nixpkgs/issues/82851:
    # > * For systemd boot, the new generation is not even accessible through the default boot entry.
    # The problem there is that `switch-to-configuration boot` only creates
    # bootloader entries that are system profiles, and does itself NOT create
    # system profiles.
    # This will only be fixed with the new `apply` script in NixOS >= 25.05.
    # Until then, we manually create a new system profile:
    # We must run manually during the deployment (before `switch-to-configuration boot`):
    #
    #     N=$(ls -1 /nix/var/nix/profiles | sed -nE 's/system-([0-9]+)-link/\1/p' | sort --numeric-sort | tail -n1) ; NEW_SYSTEM_LINK="/nix/var/nix/profiles/system-$((N+1))-link" ; ln -s "$STORE_PATH" "$NEW_SYSTEM_LINK" && nix-env -p /nix/var/nix/profiles/system --set "$NEW_SYSTEM_LINK"
    #     ls -la /nix/var/nix/profiles  # verify output
    #     $STORE_PATH/bin/switch-to-configuration boot
    #
    # Be aware of also this issue regarding `gcroots`:
    #     https://github.com/NixOS/nixpkgs/issues/73812

    # nixpkgs's `azure-common.nix` sets this to 0 for speediest boot times,
    # but we want to see the menu to be able to rollback from the serial console.
    boot.loader.timeout = (lib.mkOverride 99) 5;
    boot.loader.systemd-boot.enable = true;
    boot.loader.efi.canTouchEfiVariables = true;


Add video of HTML representation of Pipeline

I recently learned that  sklearn.set_config option display='diagram' is actually interactive and apparently I was not the only one surprised by this. A short video could show how to manipulate these HTML diagrams while motivating the use of a pipeline for assembling complex models.

The concept is a video running the following lines of code "life" and dynamically show the info displayed in the resulting diagram
import pandas as pd

ames_housing = pd.read_csv("../datasets/house_prices.csv", na_values='?')

target_name = "SalePrice"
data, target = ames_housing.drop(columns=target_name), ames_housing[target_name]
target = (target > 200_000).astype(int)

data.head()

numeric_features = ['LotArea', 'FullBath', 'HalfBath']
categorical_features = ['Neighborhood','HouseStyle']
data = data[numeric_features + categorical_features]

from sklearn.pipeline import Pipeline
from sklearn.impute import SimpleImputer
from sklearn.preprocessing import StandardScaler, OneHotEncoder

numeric_transformer = Pipeline(steps=[
    ('imputer', SimpleImputer(strategy='median')),
    ('scaler', StandardScaler())])

categorical_transformer = OneHotEncoder(handle_unknown='ignore')

from sklearn.compose import ColumnTransformer

preprocessor = ColumnTransformer(transformers=[
    ('num', numeric_transformer, numeric_features),
    ('cat', categorical_transformer, categorical_features)
])

from sklearn.linear_model import LogisticRegression

model = Pipeline(steps=[('preprocessor', preprocessor),
                      ('classifier', LogisticRegression())])

from sklearn import set_config

set_config(display='diagram')
model

from sklearn.model_selection import cross_validate

cv_results = cross_validate(model, data, target, cv=5)
cv_results

The use of the SimpleImputer is optional and is introduced here only to illustrate a slightly complex pipeline.
If we decide to keep it, then we should explain in the video that using imputers is beyond our scope.

This example could serve as a baseline for addressing  #435 and eventually #414.
What I have in mind for the latter is adding the following lines of code to the ones above:
new_categorical_features = categorical_features.remove('Neighborhood')
new_numerical_features = ['Latitude','Longitud']

preprocessor = ColumnTransformer(transformers=[
    ('num', numeric_transformer, numeric_features),
    ('cat', categorical_transformer, new_categorical_features)
])

model = Pipeline(steps=[('preprocessor', preprocessor),
                      ('classifier', LogisticRegression())])

from sklearn.neighbors import KNeighborsClassifier

new_preprocessor = ColumnTransformer(transformers=[
    ('lat_lon', numeric_transformer, new_numerical_features)
    ])

neighbors_model = Pipeline(steps=[('new_preprocessor', new_preprocessor),
                                 ])

from sklearn.ensemble import StackingClassifier

estimators = [('logistic', model),
              ('KNN',  neighbors_model)
             ]
clf = StackingClassifier(
    estimators=estimators, final_estimator= KNeighborsClassifier()
    )

clf

It uses StackingClassifier to make the model a bit more complex, feeding "longitude" and "latitude" of each neighborhood to a KNN classifier

Thanks @ArturoAmorQ.
Some notes on the recording you sent us:

I like the general tone of the video and I think it's good to have a video that shows some code to wrap up the points presented throughout the first module in a different way for better assimilation and the interactive features of the pipeline visualization widget is not necessarily easy to discover on its own.
For the intro you start with something like "Welcome to another video about machine learning with scikit-learn. Today we are going to talk about pipeline...". Assuming that readers might watch several videos in the same session, these intro at the beginning of each video might feel redundant. Maybe you could start directly with an intro sentence that more precisely state the purpose of this video: "Hello! In this video I will present how to define a simple scikit-learn machine learning pipeline and use visualize its structure in a jupyter notebook!"
When you introduce the columns you chose, I think you should say that you picked them up semi-arbitrarily in the hope that they will be predictive of the price. The way you currently phrased it could imply that you know of a systematic way to select the good predictive variable and that you use this method to choose them and I am sure that people will ask about it in the forum but this is not the goal of this video.
To explain OneHotEncoder(handle_unknow="ignore"), I think it would be better to explain that "ignore" means encoding unknown categories with 0s everywhere.
The cursor of the mouse is not visible and this can be confusing when you click on the diagram. I believe I fixed that problem in OBS by recording a full screen (on a second screen) instead of a specific window.
I think it's important to try to record in a room with no echo. To record my videos I used a blanket put on a panel that I installed a meter away on my side to avoid echos between the plain walls of my room.
using an external microphone (e.g. on a head set) is usually much better than the computer microphone
when recording with OBS, it's possible to add a noise filter to the audio capture device. It's not perfect but better than nothing. But the first 2 recommendations are more important and the OBS filters cannot properly fix echo effects of the room.

The tone is great!
There is some background noise. You'll need to get a better mike
The notebook probably needs a bit more text (and we will make it available to students. I would suggest to invest on this before on the video
I think the video should stress a bit more its didactic goal: "The goal of this video is to make you comfortable with pipelines, which can get a bit complex". The video should probably be coming to this goal on a regular basis in the video: "We now have a more complex pipeline: how to we visualize it, and how do we understand it?"
Pipeline are not only for convenience, but also because that the help avoiding leakage between train and test during model validation
In the part where you talk about cross-validation, at the end, mention that the pipeline makes it easy to manipulate this complex chain of transformations (coming back to the didactic goal of the video)

Some rephrasing suggestions:

Be more specific on the presentation of the pipeline at the beginning

The pipeline is a really nice tool of scikit-learn to combine one or more data transformation steps with a final classifier or regressor model. The resulting pipeline object can itself be treated as a machine learning model and it avoids repeative coding and data leaking ...

I think you can drop the sentence on "The more complex your model is, the more complex the pipeline will be" as it does not really bring actionable information.

about ignore in OHE, using the expression a "label of 0" is a bit misleading because 0 is not really a label but a value after OHE encoding. Let me suggest either a concise but approximate:

it will encode the variable with 0s everywhere

or more precisely:

it will encode it with 0 values in all the columns that encode that specific categorical feature.

Miscellaneous remarks:

I edited the indenting of the code for the definition of the pipelines and column transformers in the code snippets of https://github.com/INRIA/scikit-learn-mooc/issues/465#issuecomment-927724624 . I think using a more consistent indentation style in this notebook would help making it slightly easier to follow. I agree that the indentation style in our notebooks is not necessarily that consistent and we would probably benefit from re-exploring the opportunity to reformat them all to be more "vertically indented", for instance using black with a 80 char limit but this is to be discussed in another issue with everybody. Small manual formatting changes should be enough for this video.

Not that important but I noticed you tend to pronounce "colium" instead of "column"

https://www.google.com/search?q=how+to+pronounce+column
I think this is not a problem as most people will understand anyway.

I think the final click on the toplevel "Pipeline" header in the HTML widget could be skipped entirely because of the full definition of the pipeline is too large to be readable inside that HTML widget and it's already well formatted in the cell above.

Also: "Hello! Today we are gonna talk about pipelines." => "Hello!  In this video I will introduce pipelines."

Also: "Hello! Today we are gonna talk about pipelines." => "Hello! In this video I will introduce pipelines."

I am just wondering if  saying "In this video I will introduce pipelines" is not something that may depend on where we place the video (i.e. before or after the first notebook mentioning it)?

But appart from that, thanks @GaelVaroquaux and @ogrisel for your time and feedback! I will take all of the above into account.

I am just wondering if saying "In this video I will introduce pipelines" is not something that may depend on where we place the video (i.e. before or after the first notebook mentioning it)?

I thought the same but I don't think it matters. I think it's fine even if we put the video after we notebook that introduce the pipeline using written text.

My main remark was more about avoiding "Today" and also trying to avoid "gonna" which I do not find it pretty even I tend to use it a lot myself as well.

In the video I say "going to", as I agree that "gonna" does not sound very professional.

Indeed, I don't know I wrongly memorized you said "gonna".


serialize-javascript CVE-2020-7660

Versions

nuxt: 2.14.1
node: v14.0.0

Reproduction
https://www.npmjs.com/advisories/1548
Steps to reproduce
npm audit
What is Expected?
0 vulnerabilities
What is actually happening?
                       === npm audit security report ===

                                 Manual Review
             Some vulnerabilities require your attention to resolve

          Visit https://go.npm.me/audit-guide for additional guidance

  High            Remote Code Execution

  Package         serialize-javascript

  Patched in      >=3.1.0

  Dependency of   nuxt

  Path            nuxt > @nuxt/core > @nuxt/server > @nuxt/vue-renderer >
                  vue-server-renderer > serialize-javascript

  More info       https://npmjs.com/advisories/1548

  High            Remote Code Execution

  Package         serialize-javascript
  Patched in      >=3.1.0

  Dependency of   nuxt

  Path            nuxt > @nuxt/core > @nuxt/vue-renderer > vue-server-renderer
                  > serialize-javascript

  More info       https://npmjs.com/advisories/1548

found 2 high severity vulnerabilities in 1788 scanned packages

I'm sure something is in the works for a fix already, any date on that?

https://github.com/vuejs/vue/pull/11589

Hey @ImSeaWorld. Just FYI there is dependency but we don't use it and instead using https://github.com/nuxt/devalue

I think this can now be closed as the lowest version of serialize-javascript depended upon in the Nuxt dependency tree is 3.1.0.
https://github.com/nuxt/nuxt.js/blob/v2.14.5/yarn.lock#L11708


Nodejs webserver with sqlite Prisma client not able to write to db

Discussed in https://github.com/prisma/prisma/discussions/19595

Originally posted by leo-mangold June  4, 2023
I simply want to be able to give someone a folder and make them install node and then they can run my webserver, but I have no idea how to include prisma in that.
More information:
I created a SvelteKit webserver application that needs to run locally on the clients network. I'm using prisma with an sqlite database.
I thought I should be able to just build the project with adapter-node and then run node build/index.js but sadly the database part does not really work.
When I run the build version (node build/index.js) it reads everything from the database, but I cannot create anything. Creating just does nothing, but deleting does work. I think this is very weird, any ideas how to fix this?
Running npm run dev or npm run preview (after building) works perfectly fine.
Instantiating prisma client in the server code
const prisma = new PrismaClient({
  datasources: {
    db: {
      url: 'file:/Users/leo/dev/mi/echo/database.db'
    }
  },
  log: ['info', 'query', 'warn', 'error']
});

Prisma schema
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "sqlite"
  url      = "file:../database.db" // this will be /Users/leo/dev/mi/echo/database.db
}

Output when running npm run dev and creating a survey
 VITE v4.2.1  ready in 481 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
  ➜  press h to show help
[typesafe-i18n] ... all files are up to date
prisma:info Starting a sqlite pool with 21 connections.
prisma:query SELECT `main`.`Questionnaire`.`id`, `main`.`Questionnaire`.`title`, `main`.`Questionnaire`.`description` FROM `main`.`Questionnaire` WHERE 1=1 LIMIT ? OFFSET ?
5:04:40 PM [vite-plugin-svelte] ssr compile in progress ...
5:04:40 PM [vite-plugin-svelte] ssr compile done.
package      	files	  time	  avg
questionnaire	   12	95.6ms	8.0ms
prisma:query BEGIN
prisma:query INSERT INTO `main`.`Questionnaire` (`title`, `description`) VALUES (?,?) RETURNING `id` AS `id`
prisma:query SELECT `main`.`Questionnaire`.`id`, `main`.`Questionnaire`.`title`, `main`.`Questionnaire`.`description` FROM `main`.`Questionnaire` WHERE `main`.`Questionnaire`.`id` = ? LIMIT ? OFFSET ?
prisma:query COMMIT
prisma:query SELECT `main`.`Questionnaire`.`id`, `main`.`Questionnaire`.`title`, `main`.`Questionnaire`.`description` FROM `main`.`Questionnaire` WHERE (`main`.`Questionnaire`.`id` = ? AND 1=1) LIMIT ? OFFSET ?
prisma:query SELECT `main`.`Question`.`id`, `main`.`Question`.`title`, `main`.`Question`.`type`, `main`.`Question`.`data`, `main`.`Question`.`optional`, `main`.`Question`.`position`, `main`.`Question`.`questionnaireId` FROM `main`.`Question` WHERE `main`.`Question`.`questionnaireId` IN (?) LIMIT ? OFFSET ?
prisma:query SELECT `main`.`Questionnaire`.`id`, `main`.`Questionnaire`.`title`, `main`.`Questionnaire`.`description` FROM `main`.`Questionnaire` WHERE 1=1 LIMIT ? OFFSET ?

Output when running node build/index.js and creating a survey
Listening on <IP_ADDRESS>:3000
prisma:info Starting a sqlite pool with 21 connections.
prisma:query SELECT `main`.`Questionnaire`.`id`, `main`.`Questionnaire`.`title`, `main`.`Questionnaire`.`description` FROM `main`.`Questionnaire` WHERE 1=1 LIMIT ? OFFSET ?

Can anybody tell me how to make this work?

I see you've received a reply on the discussion, so I'll close this issue, as that is the right place to have that conversation.


nz-form-item-required don't recieve data

What problem does this feature solve?
When I want to hide/show the red * in FormItem label, I can't  do it by this directive, for, the source code function NzFormItemRequiredDirective : this.nzRequired = true;
What does the proposed API look like?
[nzFormItemRequired]="required"

just set [nzRequired]="false"


[DO NOT MERGE] Upgrade Pub/Sub samples.

Tweak region tags/comments to match GoogleCloudPlatform/nodejs-docs-samples#216
Split Pub/Sub samples into topics.rb and subscriptions.rb
Add CLI to samples
Switch to using google-cloud-pubsub package

Compare to:

GoogleCloudPlatform/nodejs-docs-samples#216
GoogleCloudPlatform/python-docs-samples#543

See also Proposals implemented by the 3 PRs:

Pub/Sub sample has been updated to use the latest google-cloud-pubsub library.  Existing Pub/Sub code is continuing to be improved, as is.
This code does not match idioms of Ruby, Google Cloud Ruby library, or that of other samples in this repository.  Also missing test coverage.  Can be reopened pending fixes and offline consistency decision making.


Missing Celo info page for some accounts

Expected Behavior
Every registered account has a Celo info page.
Current Behavior
0xBCD25468E4D433Ac442b87f32B37B39678BF7077 doesn't show an info page, probably because the name is empty.

Going to be fixed in https://github.com/celo-org/blockscout/pull/49


name main branch main in all repos

Thanks to @MozammilQ for compiling this list:

[x] https://github.com/Qiskit-Extensions/QuantumOpsDemos
[x] https://github.com/Qiskit-Extensions/QuantumOps.jl
[x] https://github.com/Qiskit-Extensions/ElectronicStructure.jl
[x] https://github.com/Qiskit-Extensions/electronic_structure_data
[ ] https://github.com/Qiskit-Extensions/QuantumRegistry

Thanks @MozammilQ !

The QuantumRegistry.jl is managed programatically by LocalRegistry.jl. I don't want to try to reconfigure it (if this is even possible) at the moment.


Falha ao criar personagem feminina

Inclui personagem vindo sem roupas

womans doesn't have chest by default in red dead, you just have to put clothes


[Bug Report] "For You" tab returns on homepage after viewing likes tab on own profile

If you have the algorithmic "For You" tab hidden, it becomes visible again after you view the likes tab from your own profile, then go back to your Twitter homepage.

Twitter currently has a bug where viewing a profile breaks the page title (if you go back to Home it will still have the profile's title), which breaks Control Panel for Twittter.
I thought they'd have fixed it by now (it's been a few days), but will see if we can put in a temporary workaround..


[Feature] Allow user to override NEXT_PUBLIC_CLOUDINARY_CLOUD_NAME

Feature Request
It would be nice if we could allow the user to specify the cloud name when using the component.
Is your feature request related to a problem? Please describe.
I want to be able to dynamically select the cloud name when creating the component.
Describe the solution you'd like
Have the cloud name specified in the config parameter in CldImage be used if specified.
Describe alternatives you've considered
There's not too many, as NEXT_PUBLIC_CLOUDINARY_CLOUD_NAME is replaced at build time with the cloud name, so it is hard coded
See my PR

Closed by #291


Font Uniformness

I have observed that font family used in the website is not uniform.
Objective:
Ensure a consistent font family is used across the website for a cohesive look.
Optimize font sizes to enhance readability and improve the overall visual appeal.
Please assign this issue with the label hacktoberfest and gssoc ext 24.

Hi! I am eager to contribute towards this issue. Kindly assign this issue to me.


Small bug in NodeParser

There is probably a small bug in the NodeParser function
"parent.visibile =... " should probably be "parent.visible=..."
(line 32 of nodeparser.js)

Thanks, as the NodeParser is getting rewritten this shouldn't be an issue after that.


Fix google analytics template

Fixes #29

Duplicate of #28

Looks good, thanks for your contribution! :)


chore: bump monaco-editor 0.34.0 -> 0.34.1

Ugh, thanks to npm oddities, I had to regenerate the package-lock. Otherwise, monaco-editor-webpack-plugin would for some reason still pull in 0.34.0, despite it specifying a version float ^ on monaco-editor. C'mon npm.

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[x] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

covered by #9110


Originally posted by @WesleyKhoiVo in https://github.com/tensorflow/tensorflow/issues/14607#issuecomment-785031790

please delete it


Start Playback from the cloud

Hi,
I want to use the Spotify Web Player API to start playback on my Raspberry, but I want to execute this commande from the cloud (a web server is calling the Spotify API).
It doesn't work because my Pi is not visible on Spotify connect if I'm not on the same wifi.
I thought that it should work if I add my Spotify username & password on /etc/default/raspotify config file.
But apparently not, do you have any idea ?
Thanks a lot!

Forget what I said, it works fine from a distant network one you have successfully add your Spotify username & password in the config file.
It's just that you have to use your spotify username and not you account email (this is the mistake I made).

up someone?

It seems like you already filled username and password in raspotify conf file.
I have a similar issue with the one at work, I 'm not sure it's a raspotify issue, it seems to be related to Spotify itself since my device is available and working for days and start disapearing without any modification. Restarting the RPI does not help by the way.

Thanks for your answer, yes exactly. If someone have found a solution, do not hesitate :)

Just to follow up this issue, while it was impossible to read from Spotify two days ago, everything is fine today. No RPI restart. I would say my local network is less congested today.

For me nothing has changed :(

Sometimes, restarting raspotify service helps :
sudo systemctl restart raspotify

Might be impossible as 'official' Spotify connect device typically only work if they are in the same network. So I could be something that Spotify is blocking. You could setup and VPN server and connect to it then it would accessable every where in the world.

@110101011 Actually, you don't need to be on the same local netwok since if you add Spotify credentials in Raspotify conf, your device will show up on Spotify App :

@5AMsan, I thought Spotify would have blocked that, never checked to see.
If it should be accessable outside of the network, forwarding the right port to the Raspberry Pi should do the trick.

@110101011 I never had to modify any port on the gateway or RPi to get it working, going fine out of the box.


main.go: add svg format

This commit adds the svg format to the query options.
The container image will increase in size because cgo has to be enabled
for the graphviz package.
Signed-off-by: leonnicolas<EMAIL_ADDRESS>

sorry for the big delay in reviewing :ship:


Add gcc-12 (macOS)

Tool name
GCC 12
Tool license
GPL-2.0 and 2 other licenses found
Add or update?

[X] Add
[ ] Update

Desired version
12
Approximate size
No response
Brief description of tool
The GNU Compiler Collection includes front ends for C, C++, Objective-C, Fortran, Ada, Go, and D, as well as libraries for these languages (libstdc++,...). GCC was originally written as the compiler for the GNU operating system. The GNU system was developed to be 100% free software, free in the sense that it respects the user's freedom.
URL for tool's homepage
https://gcc.gnu.org/
Provide a basic test case to validate the tool's functionality.
gcc --version
g++ --version

Platforms where you need the tool

[ ] Azure DevOps
[X] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[X] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Can this tool be installed during the build?
brew install gcc-12

Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Note: I'm mainly asking for gcc-12 along side gcc-11 on the macos-latest image.

Hi @mathomp4, we will take a look at it.

Hi @mathomp4, gcc 12 will be deployed in next week.

Hi @mathomp4, gcc 12 will be deployed in next week.

Thanks!

Should be done


Some features needed but I have not found in Calcpad

1.How could I define of standalone function and call it from another worksheet? The problem is easy to understand.
2.How could I convert a parameter's unit in the worksheet? Is there any special function or syntax for that?

It's just appeared in prerelease, see #include
Calcpad version 5.8.1.
There exists 2 types of functions: nomal pure one-line (defines just like f(x;y)=x^2+y) and macroses (can contains text, see #def)
Like this?

x = 100mm'shows x=100mm
x'shows x=100mm
x|m'shows x=100mm=0.1m
x'shows x=100mm
y = x|m'shows y=x=100mm=0.1m
y'shows 0.1m

It's just appeared in prerelease, see #include
Calcpad version 5.8.1.
There exists 2 types of functions: nomal pure one-line (defines just like f(x;y)=x^2+y) and macroses (can contains text, see #def)
Like this?

x = 100mm'shows x=100mm
x'shows x=100mm
x|m'shows x=100mm=0.1m
x'shows x=100mm
y = x|m'shows y=x=100mm=0.1m
y'shows 0.1m

Thanks a lot.


[cross_file] Platform-aware factories.

The current implementation of cross_file relies on cross-platform constructors that "need to work" across all platforms. With time, and as features are added, these constructors end up growing, and now some parameters are being ignored by some platforms and used by others. This causes confusion/problems.
It seems that cross-platform constructors are not the way to go for this particular package; instead XFile users should be able to choose what factory they need for the platform that they're implementing. So, in places where dart:io is available, they could do:
XFileIO.fromFile(io.File file) or XFileIO.fromPath(String path)
But if dart:html is available
XFileWeb.fromHtmlFile(html.File file) or  XFileWeb.fromBlob(html.Blob blob, String name) or XFileWeb.fromBlobUrl(String blobUrl, String name, int length...)...
or any other "specialty" constructors that would make sense per platform.
The XFile class that most people use would end up being a read-only interface to retrieve data from the file, and the different backing implementations would extend it.
(This would also allow us to return XFile as the base-class name for all the objects, instead of being the top-most classname, which is weird from an OOP standpoint!)
Some questions:

What to do when the user wants to modify the contents of a file programmatically?
How to perform those operations so the modified file still uses an efficient backend?
Should we keep an "inefficient" Uint8List-backed constructor so people can build raw files from memory?

I ran into a pretty serious limitation caused by this issue.
Because XFile does not have a Blob constructor for web, it can't refer to a file on the user's machine. And without a Blob,  there's no way to get the data in the file back from the XFile. So, currently, the only way to pass the file data in an XFile is by reading the full file into memory and passing the Uint8List in the constructor.
I ran into this with a drag and drop implementation for web.

@Jjagg yes, having a Blob constructor on the web is my main motivation for this change. The current web version loads a file, internally stores its blob URL, and then it attempts to re-hydrate the Blob when reading data from it. It should just... store the File/Blob and be done with it.

Why doesn't it XFile.path have that blob url as a value ?

I've seen a huge improvement of the performance of file_selector by implementing a constructor fromFile for HTML Files (as suspected).
Here's a demo app: https://dit-tests.web.app; when selecting a large file you'll see that it'll get processed immediately, instead of being copied over as a network "resource".
The prototype also includes a fix for this issue:

https://github.com/flutter/flutter/issues/91867

I think this can be extended to every platform and that we should federate the plugin. Android and iOS have there own systems for reading and writing to files external to the application:
Android: https://developer.android.com/training/data-storage/shared/documents-files#java
iOS: https://developer.apple.com/documentation/uikit/uidocumentpickerviewcontroller?language=objc
For Android, I wasn't able to use dart:io to write or read from external files. I needed to use a ContentResolver. I'm not sure if this is something that we should expect dart:io to handle or if we need to write a plugin (like cross_file_android) to interact with this API.

I think this can be extended to every platform and that we should federate the plugin.

Indeed! @stuartmorgan and I discussed this briefly a couple of weeks ago and decided to push it to after new year, but it's on the radar!

This is still on my list to work on eventually, but not in the short term, so unassigned is correct for now.


Add slide about ACS Period Estimates

ACS summary to add as a slide:
The ACS is an annual survey of about 3.5 million people, where the responses are collected throughout the year. The ACS-5yr product pools responses for five years - it doesn't average the responses across years. So both the ACS 1yr & 5yr products are period estimates. When comparing data over time from the ACS 5 year product it is best practice to use data from non-overlapping 5 year periods. See this  article  for more information.

Added for April workshop


intermittent cache? failures running ko apply

Error: error processing import paths in "config/trillian/updatetree/200-updatetree-job.yaml": error resolving image references: error getting matching image from index: Image("sha256:22cda3953e236576e5d113fc6a9290b27e179fbe5937efb1fdc16d81ddfb6120"): open ~/ko/img/blobs/sha256/2a0679608f5a6374fbe886e22ef1dc4b5578165a97db9132a128c0aa92a72155: no such file or directory

Here's an example of where we've gotten a failure when using ko apply to a kind cluster within a GHA run. It is flaky, as I can re-run it and it may or may not succeed.
https://github.com/sigstore/scaffolding/actions/runs/6764519896/job/18404319467?pr=819
perhaps worth noting: we are setting KOCACHE in  for this action, as well as attempting a restore via actions/cache

This should be falling back to the registry, so I've messed something up here. If you add a -v flag to get verbose logs, it might help point out exactly what's wrong, but I'll see if I can figure it out either way.

Wild guess: https://github.com/ko-build/ko/pull/1175

https://github.com/sigstore/scaffolding/actions/runs/6774230596/job/18410845671?pr=812 here's a failure with -v added

Apologies for the laziness here, but can you add:
  with:
    version: tip

To your setup-ko to see if that was the issue? If not I'll try to get a local repro and actually debug it.

https://github.com/sigstore/scaffolding/actions/runs/6775027890/job/18414014648 still failing, though a slightly different error message

it did run clean on the first attempt though

friendly ping - any updates on this? It's failing pretty regularly on runs: https://github.com/sigstore/scaffolding/actions/runs/6817870310

You seem to still be installing the latest release of ko, and not building from head to get Jon's fix: https://github.com/sigstore/scaffolding/actions/runs/6817870310/workflow#L76
    - uses: ko-build/setup-ko@ace48d793556083a76f1e3e6068850c1f4a369aa # v0.6
      with:
        version: tip

You seem to still be installing the latest release of ko, and not building from head to get Jon's fix: https://github.com/sigstore/scaffolding/actions/runs/6817870310/workflow#L76
    - uses: ko-build/setup-ko@ace48d793556083a76f1e3e6068850c1f4a369aa # v0.6
      with:
        version: tip

you're right, the link i gave this morning was not running from tip - however https://github.com/sigstore/scaffolding/actions/runs/6775027890/job/18414014648 is and it still fails :/

after upgrading to tip, i'm not seeing this anymore, so i'm cool if you want to close this as fixed. do you know when you'll cut the next release?
thanks @jonjohnsonjr for the patch


Mirror / clone js properties in the dragging process

Hey there, thanks for this amazing library.
When a draggable item is dragged, the following happens afaik:

A copy of the corresponding html element is created at the original source position, indicating the origninal position (*)
The original element is hidden
A mirror element is created to create a nice preview of the the item being dragged

If the original element contains JS properties, these properties are not accessible during the dragging-process.
In my use case, I want to apply changes on the source element when dragging it onto another element. However, I cannot read the properties of the original element using the DragOverEvent, so it is not possible to drag it onto its own, since the over method does only return the copied element.
To circumvent this problem, I am using attributes for now, but using properties would be beneficial (e.g. for performance reason).
Kind regards

@GRBurst Again I have to apologize for the late reply.
Can you elaborate on properties? Also, maybe there is a scenario you can describe for me? (what things about the "over" element need to be transferred to the "source").
Codepen's (or any code demo) are always welcome 😄 just helps us understand the problem more quickly.
Sorry about for the delays.
cc/ @tsov

Thanks for the responses.
I am talking about properties of js Objects.
When I have an dom element, there is a representation in javascript that I get, e.g. when using getElementById. So when I set additional properties on that Object, these are applied on the javascript Object:
var elem = documents.getElementById("specialDragItem")
elem.specialProperty = "awesome"

In the case I want to read out this element with javascript during a dragging process, this works fine except for the element I am dragging. This is because the original element is hidden and I get events on the cloned source element, instead of the original element. I think that the specialProperty is not copied to the source element.
I do not want serialize properties into Strings by using html attributes but want to work on the objects directly. It would be nice if this can be added by an Option to copy all properties of the js Object.
I will try to make a short example/fiddle in the next days to further illustrate my issue.

@tsov @beefchimi
I'm new to this library and facing issue with clone element. I'm trying to use mirror element but no success. Can you explain how can i keep my original element to it's own place and drag and drop it's mirror element on drop area? I was trying that in existing codepan example https://codepen.io/pachecoder/pen/xPKNmg

@diptigajjar A workaround is to listen to drag:start and copy your required properties from originalSource to mirror:
// untested code:
draggable.on("drag:start", e =>
  e.mirror.myProperty = e.originalSource.myProperty
)

It would be nice to have an option in Draggable, to specify an array of property-names as strings which will be copied automatically (on drag:start) to the mirror element.

@diptigajjar @tsov @beefchimi strangely enough - this is exactly what I'm trying to do as well. Once the draggable item gets dropped it disappears from its original location--which I understand is the intended purpose. But for my case, I'd like to keep a list of permanent items that can be dragged and dropped several times.

@narrative1st sounds like you are using Droppable and should be using Draggable instead.

That could be true - I figured out how to get what I wanted with Droppable but then cancelling the drop event when droppable:dropped is fired off. But I can't find example where I can catch what container the dragged event is drag:stop in...(and if it's stopped outside of a container)

@fdietze @narrative1st  yes that's what I'm trying to achieve but don't have any idea how to do it.

@narrative1st @diptigajjar does https://github.com/Shopify/draggable/issues/263#issuecomment-413148571 help?

@fdietze looks like that is something which perhaps works. I'll give it a try

@diptigajjar Any luck ?

@GRBurst I am also not 💯 sure I get the issue.
There are three elements that matter during a drag operation

originalSource which is the original element that gets copied and is hidden while dragging
source which is a copy of the original source and get moved around as you sort things (it's the placeholder)
mirror which is a copy of the original source element and follows your cursor while dragging

You'll want to change styles for mirror and source during a drag operation and change styles for originalSource at the end of a drag operation.
Let us know if you need to more pointers

I want to apply transform:rotate(10deg) for mirror, but it is not working as mirror already has another transform style which is overriding my style. How can I do that? Thanks

@nsarvar If you don't mind an extra wrapper DOM, you can try https://codepen.io/1010543618/pen/qBNRwZK


Plugin report command is failing for openshift user namespaced operators

Currently the report command will fail if more than one deployment of the operator is detected. This behavior will not work in the case there are multiple namespaced deployments of CNPG.

I have started this change but am a little confused about how we intend to behave in Openshift environments vs vanilla Kubernetes environments. I am pausing for now to wait and speak with @jsilvela tomorrow.

@jlong49 probably we can close this one, since it's not related to the opensource project at all, what do you think ?

Closing this since it was covered on the issue #247


How to use composer under python3.8

** Environment **

** To reproduce
composer/utils/auto_log_hparams.py", line 13, in
def extract_hparams(locals_dict: dict[str, Any]) -> dict[str, Any]:
TypeError: 'type' object is not subscriptable
Steps to reproduce the behavior:

Expected behavior

Additional context

See chip on README

Composer supports a rolling list of 3 versions of python, which is currently Python 3.9-3.11. Accordingly, we recommend you upgrade.
You can also use an older release if you need Python 3.8.