add tests for kiwi container image

Part of https://github.com/SUSE/BCI-tests/issues/536
Assumes test setup as - https://github.com/SUSE/BCI-tests/pull/537

@dcermak @dirkmueller few questions:

is there a kiwi container image being published to registry.suse.com
(quick search on the registry doesn't show anything)

is there a way to run podman rootful tests in BCI-tests (containers created with sudo) using pytest-container framework?

I've added tests based on the steps in kiwi quickstart guide.
Outside these, is there any specific kiwi functionality that needs to be tested?

FYI -  test_kiwi_create_image() requires containers created with podman rootful mode to mount loop devices (/dev/loop*), which is not possible with podman rootless run. So, I've disabled the test in case of container_runtime is not docker.
ref: https://github.com/containers/podman/issues/17715#issuecomment-1460227771

re: failing CI / Lint source code (pull_request)
************* Module tests.test_kiwi
tests/test_kiwi.py:9:0: E0611: No name 'KIWI_CONTAINERS' in module 'bci_tester.data' (no-name-in-module)
tests/test_kiwi.py:40:0: C0116: Missing function or method docstring (missing-function-docstring)
tests/test_kiwi.py:67:0: C0116: Missing function or method docstring (missing-function-docstring)

This should be addressed with changes in https://github.com/SUSE/BCI-tests/pull/537

@Priyankasaggu11929 I merged the other PR now, please rebase (that should turn this into a greener state)

@dirkmueller, rebased PR with the kiwi test plumbing changes, and addressed ^ review comments.

Kiwi tests are green in the CI now.
Although, 3 out of the 7 failing CI tests are marked as Required but doesn't have kiwi container images related failure, so, not sure how to bypass them here.

I've added tests based on the steps in kiwi quickstart guide.
Outside these, is there any specific kiwi functionality that needs to be tested?

@dirkmueller, also need your input on ^. Thanks!

Although, 3 out of the 7 failing CI tests are marked as Required but doesn't have kiwi container images related failure, so, not sure how to bypass them here.

that's because you are submitting from a fork, you don't have the necessary credentials available there for those CI checks to turn green. don't worry about them.

I've added tests based on the steps in kiwi quickstart guide.
Outside these, is there any specific kiwi functionality that needs to be tested?

@dirkmueller, also need your input on ^. Thanks!

I'm okay with the state as is, although I would have loved to see something passing with podman also, but we can always improve on this in a followup.


Add delayed corrections

Delayed corrections would allow one to send a message, send another message, then send a third message, correcting the first.
Credit to /u/danopia for providing this idea.

Hey,
How far from the old text can the correction be?
I am thinking off taking this issue :smile:

That'd be great, @moar55! If you need any help with this issue, feel free to let me know, and I will help you the best I can.


Can't fetching secrets

I reproduced the structure from your example, but all trying ended with errors.

I have secrets on the following path, that can be read and listed.
any/path/daytona_test/{api-key|database}
I exported the variable VAULT_SECRETS_APPLICATION=any/path/daytona_test and try to fetching that secrets, but i got 405 error.

URL: GET https://awesome.vault/v1/secret/data/any/path/daytona_test?list=true
Code: 405. Errors:

* 1 error occurred:
        * unsupported operation

Trying to fetch secret by direct path. Use the variable: VAULT_SECRET_APPLICATION=any/path/daytona_test/api-key. Got non-string value error.

DAYTONA - 2019/08/15 12:18:05 Starting secret fetch
DAYTONA - 2019/08/15 12:18:06 secret 'api-key_data' has non-string value: map[string]interface {}{"value":"1234"}

Try again:
DAYTONA - 2019/08/15 12:21:17 Starting secret fetch
DAYTONA - 2019/08/15 12:21:17 secret 'api-key_metadata' has non-string value: map[string]interface {}{"destroyed":false, "version":"1", "created_time":"2019-08-15T05:02:57.750127434Z", "deletion_time":""}

If i send in the API by Postman i get next:
{
    "request_id": "a273f8e1-342f-7ee1-550f-de0734456154",
    "lease_id": "",
    "renewable": false,
    "lease_duration": 0,
    "data": {
        "data": {
            "value": "1234"
        },
        "metadata": {
            "created_time": "2019-08-15T05:02:57.750127434Z",
            "deletion_time": "",
            "destroyed": false,
            "version": 1
        }
    },
    "wrap_info": null,
    "warnings": null,
    "auth": null
}

Is this an unexpected response for Daytona? What am I doing wrong?
Vault version: 1.1.1

Hi @rbakhtaraev - Are you using the KV Secrets Engine V2?
To determine version number, run vault secrets list -detailed and inspect the Options column for version:2.

Hi @broamski, sorry for the delay. We are using KV Engine V2.

@broamski There is any special consideration we need to have with KV Engine V2? It doesn't work for me. Works fine when I use Engine V1. I suspect is related to the existence of multiple keys in V2 (Secret Metadata).


Using --outputs-file with sst start does not produce an output.json

We would like to use the outputs from sst start to do some e2e testing for our current project.
According to the docs this should be possible.
But passing the --outputs-file parameter in the cli does not generate an outputs.json file.
There was already an issue regarding this problem #532 but it was closed.
The issue #263 linked to a pull request #265 but it doesn't seem to call the appropriate function writeOutputsFile that is called in the deploy script.
Is this the intended behavior or are we doing something wrong?

Hey @Pr0gramWizard, this is fixed in v0.40.5!
Give it a try and let me know if it works for you!

Yes it works now!
Thank you very much.


Crashes with file formats...

Log
/yt2009/back/backend.js:4166
        let url = player.streamingData.formats[0].url
                                       ^
TypeError: Cannot read properties of undefined (reading 'formats')
    at handlePlayer (/yt2009/back/backend.js:4166:40)
    at /yt2009/back/backend.js:4270:9
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

And the problems with loading video
/yt2009/node_modules/node-fetch/lib/index.js:1501
			reject(new FetchError(`request to ${request.url} failed, reason: ${err.message}`, 'system', err));
			       ^
FetchError: request to https://rr4---sn-p5qlsn7s.googlevideo.com/videoplayback?expire=1715365109&ei=lRA-Zr-jMpSSlu8PtfaYiAY&ip=<IP_ADDRESS>&id=o-AMPAOtklZcDpqg3d-8ZU_6yJtXMNrn926cwMGK5hA8tK&itag=18&source=youtube&requiressl=yes&xpc=EgVo2aDSNQ%3D%3D&mh=05&mm=31%2C29&mn=sn-p5qlsn7s%2Csn-p5qs7nzy&ms=au%2Crdu&mv=u&mvi=4&pl=23&bui=AWRWj2QEYuB4oJsN50dxuNXKAcl2zFQMc6N9dSkm-_Q9HRCa5W2liZj3so2gu9JhI5UmKqEAmSlPj3BK&vprv=1&svpuc=1&mime=video%2Fmp4&ns=plWc1iQ_Q8phIYOaOoUsm68Q&rqh=1&gir=yes&clen=2061568&ratebypass=yes&dur=43.467&lmt=1248461926534338&mt=1715342987&fvip=3&c=TVHTML5_SIMPLY_EMBEDDED_PLAYER&sefc=1&n=etYUrE9n9-rD7f7p-&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cxpc%2Cbui%2Cvprv%2Csvpuc%2Cmime%2Cns%2Crqh%2Cgir%2Cclen%2Cratebypass%2Cdur%2Clmt&sig=AJfQdSswRAIgZMMM-3Wztrrv2BWsFOJ5Hp3BTALTSYIaIPyzAcXSaI8CIBDASYfclMhDjA7Y1M5C3AeFE-xXrqF-0Q9q1tgG0sNv&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl&lsig=AHWaYeowRQIhAN_Z0yWkDGE-yJoRNXzcZ6cZX3IXRn4JRIW52PqAERspAiA6xcOCn8UDAFEM-oD2VHIp6nwzC6p51N2mp-T5BSqp-g%3D%3D failed, reason:
    at ClientRequest.<anonymous> (/yt2009/node_modules/node-fetch/lib/index.js:1501:11)
    at ClientRequest.emit (node:events:519:28)
    at TLSSocket.socketErrorListener (node:_http_client:500:9)
    at TLSSocket.emit (node:events:519:28)
    at emitErrorNT (node:internal/streams/destroy:169:8)
    at emitErrorCloseNT (node:internal/streams/destroy:128:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  type: 'system',
  errno: 'ETIMEDOUT',
  code: 'ETIMEDOUT'
}

/yt2009/back/backend.js:4166
        let url = player.streamingData.formats[0].url
                                       ^
TypeError: Cannot read properties of undefined (reading 'formats')
    at handlePlayer (/yt2009/back/backend.js:4166:40)
    at /yt2009/back/backend.js:4270:9
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

looks like you're trying to view an age-restricted video, those require login and use a bypass which is not 100% reliable. support for adding accounts for better age restrict bypass is planned.

errno: 'ETIMEDOUT',
code: 'ETIMEDOUT'

sorry, can't do anything about internet issues, especially when the link opens fine for me. the link is a direct video link, and unlike googleusercontent/ggpht i don't really see a way to bypass that aside from straight up not playing the video which is far from ideal.


A bug in Model forward

I noticed an issue in the BCFN （CIM）forward function. Currently, the torch.chunk operation is applied twice to x1, instead of once each to x1 and x2. As a result, x2 is being ignored in the chunking process, which means the model is not properly using the data from x2.

Thank you for your interest in PETDet. We have confirmed that the bug you reported in BCFN does exist. We will fix the bug as soon as possible and upload new models.

Thank you for raising the issue. We have fixed this bug and released new models.


http://ehcache.org is down

It's been down since at least February 8th, 2021 (i.e., 3 days as of today). This makes it harder to find the documentation during development.

It's working for me, and for others: https://downforeveryoneorjustme.com/ehcache.org

Strange: this is the only site that's down for me. I'll ping my ISP...
If your site is doing anything cutting-edge, i'd appreciate if you let me know.

FYI: the issue on this end has been resolved.


Routes to take to indiviual pages

It is done when:

[ ] I click on a button and I am taken to a new page
[ ] that page displays the forfeit data on that specific golfer


Suppress decrypted file list from helm secrets template output

I want to pipe the output of helm secrets template to some yaml processor while an error occurred because an extra line in the output
...

files/secrets.yaml.dec

It seems helm secrets print the list of decrypted files to stdout. Is it possible to suppress this message?

This is the same issue I described here: https://github.com/futuresimple/helm-secrets/issues/126

Bump.  👀

Came here for this. helm secrets needs a --quite option, pressing all output, except desired (e.g. templates would output the template).

I want to point out this fundamentally breaks a helm template's use case of allowing you to pipe into kubectl apply - e.g. helm template | kubectl apply -f -
When there's a newer .dec file, you're given a prefix line saying the .dec is newer, then the template. When there is not, you're given a suffix line outputting the decrypted file. This makes is horrible in developing automated systems to pipe out to kubectl. I actually had to manually decrypt the file first, just so I can predict the output being on the first line so I can remove it first, before sending to kubectl.
In my opinion, this needs to be moved up in priority. How do we get more sights on this issue?

Just ran into this. Unfortunately will need to fork as this interferes with the helm template workflow. A good philosophy might be that helm secrets should produce stdout that is YAML compatible comments, a list of files preceded with # would have avoided this issue.


SqlServer: Char with converter does not work well in query

SqlServer does not support char so we use char to string converter. When comparing character to constant/parameter, we run the value through converter so we get a string constant/parameter. But char is always wrapped in conversion to int on client side. So we also generate cast in SQL query. Which works well for SQLite but on SqlServer it fails to convert "varchar" to "int" and throws.

Funcletizers leaves convert around char because of https://github.com/aspnet/EntityFrameworkCore/issues/14159
Convert nodes cannot be identified if implicit or explicit. We could just make a guess that char to int convert is always implicit and we can remove convert on both side while translating to SQL. @roji with regards to #14159 do you see any issue with this approach?
SqlServer can intercept such conversions and generate parameter/constant of required type directly. May have to constraint on char to int since some explicit conversions may be used for truncation by user.

@smitpatel always removing char->int conversion nodes would certainly fix #14159 for me. I'm a little concerned that this would silently disallow explicit casts which may be legitimately used by the user in some cases - but I don't have a good scenario in mind. On the other hand I'm not sure there's a database out there which supports ASCII casting char->int (when text of any kind is cast to int, PostgreSQL attempts to parse it numerically and throws if a non-digit is found).
So removing the cast node seems to make sense... Assuming I've understood everything correctly :)

Same for ushort (and probably all types which upcast to int for equality operator)

So removing the cast node seems to make sense... Assuming I've understood everything correctly :)

I remove all impicit cast (Convert nodes) in my data provider.
One problem - need rebuild the functions and binary operations nodes. I say about local calculations.
For example, the call of datetime.AddYears(int) may transformed into extension method AddYears(this DateTime, int?) or vice versa.
Plus, I'm losing the call "NullableValue.Value" - EFCore (or linq?) translates this call to Convert node.


Non-Play Store installation methods

The old React Native client is also regularly updated on F-Droid. I was told at https://github.com/zulip/zulip-mobile/issues/5795#issuecomment-1977743319 that eventually APKs for the Flutter client will be made available as GitHub Releases, too. Will any store other than Google Play also feature the new client?

Bump

Thanks. Yeah, it's probably time we started posting release APKs on GitHub.
I'm planning to make a v0.0.15 release this week. So I'll see about working that into the release process then.

@foxyseta Release made — please take a look!
https://github.com/zulip/zulip-flutter/releases/tag/v0.0.15

@gnprice I think we should keep this open to also investigate publishing to F-Droid, which would require a build that doesn't depend on Google Play services (com.google.gms:google-services):

F-Droid Inclusion Policy
FAQ

So, that will mean we would need to integrate something like UnifiedPush for notifications (which may also need changes to server).

Had to uninstall 0.0.14 before installing 0.0.15 via apk for some reason. Anyway, yes, the original post is talking about stores.

Had to uninstall 0.0.14 before installing 0.0.15 via apk for some reason.

Yeah, that's expected. Android requires that when the APKs are signed with different keys, and the signing keys we're using for this app in the Play Store and in the GitHub release artifacts are different:

The APKs you get from the Play Store are signed by a key that was generated in Google's infrastructure and lives only there.

The APK and AAB uploaded to GitHub are signed by our own key, at build time. (The same AAB, with the same signature, is the artifact I upload in turn to the Play Store. The Play Store breaks that apart and recombines its contents in order to make a smaller APK specialized to each user device's CPU architecture and display density.)

When you make upgrades within either of those distribution channels, the signing key will remain the same and you shouldn't need to uninstall in order to upgrade.

For F-Droid, the production/legacy zulip-mobile app is there today. So I expect that when this app is ready and we start rolling it out to replace the legacy app — that is, when we start publishing builds of this app with the package ID com.zulipmobile, rather than com.zulip.flutter, so that it offers a direct upgrade from the legacy app — we'll also go on to have this app appear on F-Droid too.
The way the existing F-Droid version of zulip-mobile works is that the F-Droid build just removes the notification support. So that's what I expect would initially happen with this app too.
There's an issue thread in the zulip-mobile (or maybe zulip/zulip?) tracker where we've talked about UnifiedPush and things like it. That might be something we do in the future. We won't have attention to spare for that before we complete the migration from zulip-mobile to zulip-flutter, though.

There's an issue thread in the zulip-mobile (or maybe zulip/zulip?) tracker where we've talked about UnifiedPush and things like it.

That's probably https://github.com/zulip/zulip-mobile/issues/3838 (and it's duplicate, https://github.com/zulip/zulip-mobile/issues/5712).


Help building an API using microk8s

Hello everyone, and thank you for the amazing work.
I'm trying to build the rudderstack-server using helm:
https://github.com/rudderlabs/rudderstack-helm
What I want to achieve:
Run the API and expose it to a port, so I can use Nginx as a reverse proxy on a domain.
What I tried:
git clone git@github.com:rudderlabs/rudderstack-helm.git
cd rudderstack-helm/
microk8s enable dns dashboard registry
microk8s kubectl config view --raw > $HOME/.kube/config
helm install myproject ./ --set rudderWorkspaceToken="MYCODE"

When running kubectl get --namespace default svc -w myproject-rudderstack I get:
NAME                    TYPE           CLUSTER-IP       EXTERNAL-IP   PORT(S)        AGE
myproject-rudderstack   LoadBalancer   <IP_ADDRESS>   <pending>     80:31455/TCP   23m

or kubectl get services
I get:
NAME                                        TYPE           CLUSTER-IP       EXTERNAL-IP   PORT(S)        AGE
kubernetes                                  ClusterIP      <IP_ADDRESS>     <none>        443/TCP        26m
myproject-rudderstack                       LoadBalancer   <IP_ADDRESS>   <pending>     80:31455/TCP   24m
myproject-rudderstack-postgresql            ClusterIP      <IP_ADDRESS>   <none>        5432/TCP       24m
myproject-rudderstack-postgresql-headless   ClusterIP      None             <none>        5432/TCP       24m
myproject-rudderstack-transformer           ClusterIP      <IP_ADDRESS>    <none>        9090/TCP       24m

I'm very new to Kubernetes, and I want to have an external IP that I can proxy through Nginx to serve the API URL on a specific domain but I'm stuck here.
Can you please help?
I'm trying to run this on a single machine.

There is a metallb addon which can help provide a kubernetes service external ip.

Do you have any example how to use it?
On Sun, May 31, 2020, 1:52 PM balchua<EMAIL_ADDRESS>wrote:

There is a metallb addon which can help provide a kubernetes service
external ip.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ubuntu/microk8s/issues/1254#issuecomment-636448008,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNKBZMTBAPGCC25XHOFLYLRUISGXANCNFSM4NOYOOQA
.

First you identify the external ips you want your pods to have.  For example in my internal network, my ip ranges from <IP_ADDRESS>/24 which means it has up to 255 devices.  And i want to allocate <IP_ADDRESS>-<IP_ADDRESS>
Then enable metallb like this.
microk8s enable metallb:<IP_ADDRESS>-<IP_ADDRESS>
After you enable metallb, for every new Service of type LoadBalancer metallb will allocate an ip from the range you specified above.

For more info on metallb check their site.
https://metallb.universe.tf/
I believe the metallb addon is using layer 2 mode.


Merging warning fixes from develop

Please revert with hard overwrite of history

You should only fast forward

Git checkout staging
Git reset --hard origin/develop
Git push origin staging

Sorry about that.  Done as instructed.

Thanks, it has been just me for a long time. We should document someday.


[Feature]: Developer can cf logout from a cf-k8s foundation

Blockers/Dependencies

cloudfoundry/cf-k8s-api#56

Background
cf login run against a cf-k8s foundation doesn't really perform a log in: we ask the user to select an auth-info which has been previously generated. For consistency and symmetry, we want to forget the selected auth-info on cf logout. cf logout does not remove any information about the the targeted API.
Acceptance Criteria
GIVEN I am logged into a cf-k8s foundation
WHEN I run cf logout
THEN I I should see something like this in $CF_HOME/config.json:
"CFOnK8s": {
    "Enabled": true,
    "AuthInfo": ""
}

PR


"SWS/wol: Set selected envelope height to maximum" doesn't seem to take into account automation item lane (like 16 pixels?)

Title says it all:
"SWS/wol: Set selected envelope height to maximum" doesn't seem to take into account automation item lane (like 16 pixels?) while scrolling/zooming the envelope to the max

Also, I'm not sure how this scales with dpi, so that too needs to be taken into account (I haven't tested much DPI stuff, so I'm not sure if you can just multiply with DPI modifier or you gotta check for pixel sizes for each DPI scale)

Confirmed! This happens in REAPER v6 only. The scrolling doesn't have that weird offset in v5 (assuming the condition for triggering bug #1240 isn't met).


CORS errors resolve to statusCode: 0

Try this in a browser:
request('GET', 'http://yahoo.com')
.then(function(res) {
  console.log("Success:", JSON.stringify(res));
})

This will fire off a CORS handshake:
=> OPTIONS "http://yahoo.com/" \
  access-control-request-headers:"x-requested-with" \
  Access-Control-Request-Method:"GET"

<= HTTP/1.1 401 Forbidden

This gets you this result, however:
Success: "{"statusCode":0,"headers":{},"body":""}"

Unfortunately, that is all that gets given to JavaScript.  This stack overflow post: http://stackoverflow.com/questions/19325314/how-to-detect-cross-origin-cors-error-vs-other-types-of-errors-for-xmlhttpreq does a pretty good job of explaining it.

interesting. superagent manages to find out though, i wonder what workaround they're using

Indeed, they're using if (status == 0) on xhr.onreadystatechange.
https://github.com/visionmedia/superagent/blob/master/lib/client.js#L890-L893
...from experience I can say this is pretty hacky and sometimes shows up in place of connection refused errors.

We should definitely look at doing something better.  I think the best bet is probably to check the status code and return an error with a message that says it could be caused by CORS.

agree, with if (xhr.status === 0) throw new Error("Request failed"); at the very least. A 0 should not count as a success.


Perplexity overshoots after resuming the training from the last checkpoint using multilingual transformer.

❓ Questions and Help

What is your question?
I am training a multilingual transformer model using the default parameters as listed in the official example.
I trained the model for 20 epochs and the validation perplexity was 24.56 after the end of that epoch. But, after resuming the training from 21th epoch, the perplexity just overshoot to 2540.74. So should I reset the parameters before resuming from the last checkpoint?
What's your environment?

fairseq Version : latest master
PyTorch Version : 1.5.0
OS : ubuntu 18.04
How you installed fairseq : source
Build command you used (if compiling from source):  pip install --editable ./
Python version: python 3.6.9
CUDA/cuDNN version: 10.1
GPU models and configuration: Tesla T4

when you say you resumed training, how exactly did you do this?
in general there is no need to reset parameters. you just run the same command you did originally and make sure that --restore-file points to your latest saved checkpoint

Hello @alexeib, thanks for your comment. I used the same training code without any change and didn't use the --restore-file  argument as the training should resume from the checkpoint_last.pt automatically. Or do I need to specify it with --restore-file?

--restore-file is defaulted to checkpoint_last.pt so if you are pointing to the same checkpoint dir, yes, it would continue automatically.

Thanks for your suggestion. I've retrained the model from scratch, then resumed from checkpoint_last with and without --restore-file argument. And the perplexity is quite normal. But, I still don't get it why there was sudden hike in perplexity at the first place.


Tools: reconsider whether all old "official" tools should be listed

          This one is very confusing. It speaks about the "Schema", points to the guidelines website and links the main repo as repository...?

Originally posted by @musicEnfanthen in https://github.com/music-encoding/music-encoding.github.io/pull/588#discussion_r1532132127

@musicEnfanthen can this be closed after the redesign of the Tools section?

I think you opened it AFTER the redesign of the Tools section ;)
It was mostly about the confusing info in the "tool" called "MEI Schema" which is still confusing.

Jikes, you’re absolutely right. Looking at it, I think we should delete it from the tools collection.


fix: remove cargo lock file from git as it is a lib not a binary project

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

GitHub release
crate.io packages (1 packages published)

Your semantic-release bot :package::rocket:


Why are there spaces between method name and the opening peren?

.

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

ID: 4bb713b4-53a5-e5c4-1155-0562ffd34960
Version Independent ID: daad2dfe-3e42-88f6-6f25-8f66c435a6f6
Content: Consuming a RESTful Web Service - Xamarin
Content Source: docs/xamarin-forms/data-cloud/consuming/rest.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @StingyJack
It's the old coding standard used by Mono, that used to be enforced by Xamarin Studio.


Map upload size limit shows 2097152 bytes, likely should be 4 MB like assembly upload experience

Describe the Bug with repro steps
Newer (and cleaner) upload experience for assemblies shows a 4 MB limit:

But the older map upload experience shows a lower limit (likely the exacting count of bytes in 2 MB):

Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
No response

Fixed by https://msazure.visualstudio.com/One/_git/AzureUX-LogicAppsPortal/pullrequest/7912873


Allow subscript numbers in identifiers

Subscripts will format from __ followed by consecutive digits.
Identifiers are canonicalized during parsing so that A__1 and A₁ resolve to the same identifier (A₁) even when code is not formatted.

Could cananicalization be done in the lexer? I think it makes more sense there.

That's what I initially thought, but at least from what I've gleaned from the code, the lexer returns identifier tokens as just the span they occupy in the code, so it's not possible for me to change the content of the actual identifier returned by the lexer.

Also, since there are multiple canonicalization steps now, should they be separated into a complete canonicalization function?

Also, since there are multiple canonicalization steps now, should they be separated into a complete canonicalization function like canonicalize_ident that just calls canonicalize_exclams and canonicalize_subscripts?

Yes probably

Okay I added canonicalize_ident.


SQL: Column ordering after groupby

Describe the bug
The column ordering of the output after a groupby and selection on keys seems to be not consistent with what the user specifies

To Reproduce
No response
Expected behavior
No response
Component(s)
SQL
Additional context
No response

resolved via https://github.com/Eventual-Inc/Daft/pull/3177


Internet Explorer 8 support

Is there any chance for IE8 support in Quail?

I just got a SauceLabs integration running on the master branch. So I can at least know now when things ar ebrea


DOC-570: Rewrite SDK doc section Support and FAQ

Documentation changes made to support-faq.md:

Other pages impacted:

catalog.md
analytics.md
README-dev.md: fixed broken link in text.

Renamed page to Support
Contact: Removed header
Related links: Removed section and replaced with a Jupyter notebooks section
FAQ: Removed all but 1 to respective pages where they are more relevant.

"I'm having trouble installing the SDK on Windows": Removed because installation.md For Windows (conda) tab answers this and provides the two links.
"Where is my config.json file with the project credentials?": Removed because authentication.md Step 3 In a configuration file tab shows how to create a config.json file.
"Where can I find the default parameters for a block?": Removed because finding the default parameters is shown in analytics.md Step 2.
"I want the catalog search results as JSON instead of a dataframe": In catalog.md  Step 4 added this information.

@andher1802 Can you please review and approve it?


Option to preview with remote scene tree should be disabled by default

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3 beta 1
Issue description:

Preview scene starts with default remote scene tree activated, this become in a very bad and choppy performance of the game loop. This option should be manualy activated clicking the remote option in panel only for testing/debugging, not always by default.

Link to minimal example project:

Agreed, until the performance is improved it should be opt-in, not opt-out.


change local source branch name from main

Description of changes:

since we create PR against controller repo's main branch, the local branch name should not be main

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Merging this change manually since it is very small to unblock controller auto generation.
If there is any feedback, please leave it in comment. I will address it ASAP.


VSTS Create Dashboard API

Creating the simpler dashboard widgets using the create functionality works really well, however the more complicated widgets (CFD, BurnUp, and Velocity) always seem to give me an error about needing a "Type Id" before posting. The documentation says Type Id is deprecated and replaced by the Contribution Id. Either way, whether I have the Type Id or I don't include it in the jSon, the same error comes back saying that it can't be empty.  Just something to be aware of regarding creating widgets

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

ID: 24135945-2c86-d7cb-b800-f95b30b0beaa
Version Independent ID: d4e01ce1-397c-47f3-59d9-8e0ca9bc86eb
Content: Widgets (VSTS Dashboard)
Content Source: docs-ref-autogen/dashboard/Widgets.yml
Service: unspecified
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

Hi Evan,
Got it, It sounds like the error handling is stale- can you describe which specific REST operation you were trying to exercise, so we can sanity check the details?
Thanks

The VSTS Rest API I was using is the Create Widget function. The documentation is here: https://docs.microsoft.com/en-us/rest/api/vsts/dashboard/widgets/create
I used the create Dashboard function to throw up a dashboard and I was using this to append the widgets to it. I also tried the widgets in the dashboard creation and I received the same error.
Sent from my iPhone

On Apr 24, 2018, at 6:14 PM, PeterAntal<EMAIL_ADDRESS>wrote:
Hi Evan,
Got it, It sounds like the error handling is stale- can you describe which specific REST operation you were trying to exercise, so we can sanity check the details?
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Also, I was doing a POST request. Sorry I forgot to mention that earlier.
Sent from my iPhone

On Apr 24, 2018, at 6:14 PM, PeterAntal<EMAIL_ADDRESS>wrote:
Hi Evan,
Got it, It sounds like the error handling is stale- can you describe which specific REST operation you were trying to exercise, so we can sanity check the details?
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

(For benefit of anyone reviewing this issue)
In touch with Evan over email also now... Requested the error message content, to isolate where in code the failure was produced.


add support for app_home_opened event

Add support for add support for app_home_opened event according to
https://api.slack.com/events/app_home_opened

@nlopes @james-lawrence hi guys, could you please review my pr?
Thanks


swap 'any' for '*'

Codecov Report

Merging #284 (0fa5228) into main (4268c1b) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #284   +/-   ##
=======================================
  Coverage   69.87%   69.87%
=======================================
  Files         120      120
  Lines        6396     6396
  Branches      514      514
=======================================
  Hits         4469     4469
  Misses       1798     1798
  Partials      129      129

Impacted Files
Coverage Δ

pyEX/metadata/__init__.py
76.47% <100.00%> (ø)

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 4268c1b...0fa5228. Read the comment docs.


Installation issues / conflict with xdebug

On a fresh install of Taskboard, when attempting to login for the first time I would get the error "API Error: Ensure api directory is writable."  I searched the issues here and tried everything suggested to no avail.  Long story short, it turns out RedBean conflicts with xdebug, which I do have installed.
Background
I'm using PHP 5.6 on Apache.  Installed taskboard according to instructions.  Ran the composer build script.  Did chmod -R 777 api.  Confirmed apache mod rewrite and expires enabled.  AllowOverride all.
When attempting to login with admin/admin, in my browser console I found the following error and stack trace:
{
	"message": "API Error.",
	"data": "SQLSTATE[HY000]: General error: 1 no such table: user",
	"alerts": null,
	"trace": [{
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/vendor\/gabordemooij\/redbean\/RedBeanPHP\/Driver\/RPDO.php",
		"line": 337,
		"function": "runQuery",
		"class": "RedBeanPHP\\Driver\\RPDO",
		"type": "->",
		"args": ["SELECT COUNT(*) FROM `user`  -- keep-cache", []]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/vendor\/gabordemooij\/redbean\/RedBeanPHP\/Driver\/RPDO.php",
		"line": 374,
		"function": "GetAll",
		"class": "RedBeanPHP\\Driver\\RPDO",
		"type": "->",
		"args": ["SELECT COUNT(*) FROM `user`  -- keep-cache", []]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/vendor\/gabordemooij\/redbean\/RedBeanPHP\/Adapter\/DBAdapter.php",
		"line": 164,
		"function": "GetOne",
		"class": "RedBeanPHP\\Driver\\RPDO",
		"type": "->",
		"args": ["SELECT COUNT(*) FROM `user`  -- keep-cache", []]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/vendor\/gabordemooij\/redbean\/RedBeanPHP\/QueryWriter\/AQueryWriter.php",
		"line": 988,
		"function": "getCell",
		"class": "RedBeanPHP\\Adapter\\DBAdapter",
		"type": "->",
		"args": ["SELECT COUNT(*) FROM `user`  -- keep-cache", []]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/vendor\/gabordemooij\/redbean\/RedBeanPHP\/Repository.php",
		"line": 526,
		"function": "queryRecordCount",
		"class": "RedBeanPHP\\QueryWriter\\AQueryWriter",
		"type": "->",
		"args": ["user", [], "", []]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/vendor\/gabordemooij\/redbean\/RedBeanPHP\/OODB.php",
		"line": 457,
		"function": "count",
		"class": "RedBeanPHP\\Repository",
		"type": "->",
		"args": ["user", "", []]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/vendor\/gabordemooij\/redbean\/RedBeanPHP\/Facade.php",
		"line": 1179,
		"function": "count",
		"class": "RedBeanPHP\\OODB",
		"type": "->",
		"args": ["user", "", []]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/api\/helpers.php",
		"line": 386,
		"function": "count",
		"class": "RedBeanPHP\\Facade",
		"type": "::",
		"args": ["user"]
	}, {
		"file": "\/var\/www\/clients\/client10\/web12\/web\/taskboard\/api\/api.php",
		"line": 30,
		"function": "createInitialUser",
		"args": []
	}]
}

Solution
I didn't want to disable xdebug completely from my server because I use it for other things.  So instead I just added the following at the beginning of api/api.php:
if(function_exists('xdebug_disable')) { xdebug_disable(); }

Suddenly, everything works!
Sharing this in hopes it may help someone else.

Ran into the same issue and your fix works for me. Thanks!


A question about how to draw triangles in Cesium?

hello, I want to know how to draw triangles in Cesium.
Each vertex is a geopoint(lon,lat,elevation),and render each point use diffrent color?
here is my code but it always has some mistake
Thank you very much!!!
An error occurred while rendering. Rendering has stopped.
DeveloperError: All attribute lists must have the same number of attributes.
Error
at new DeveloperError (http://localhost:9313/myCesium/Source/Core/DeveloperError.js:43:19)
at Function.Geometry.computeNumberOfVertices (http://localhost:9313/myCesium/Source/Core/Geometry.js:189:27)
at addPerInstanceAttributesToGeometry (http://localhost:9313/myCesium/Source/Scene/PrimitivePipeline.js:146:41)
at addPerInstanceAttributes (http://localhost:9313/myCesium/Source/Scene/PrimitivePipeline.js:177:17)
at geometryPipeline (http://localhost:9313/myCesium/Source/Scene/PrimitivePipeline.js:226:9)
at Object.PrimitivePipeline.combineGeometry (http://localhost:9313/myCesium/Source/Scene/PrimitivePipeline.js:438:26)
at combineGeometry (http://localhost:9313/myCesium/Source/Workers/combineGeometry.js:12:41)
at DedicatedWorkerGlobalScope. (http://localhost:9313/myCesium/Source/Workers/createTaskProcessorWorker.js:55:42)
var mypositions = Cesium.Cartesian3.fromDegreesArrayHeights([ 104.317776,31.59491,10,
105.317776,32.59491,20,
106.317776,33.59491,30]);
var pos=new Float64Array(mypositions);
            var geometry = new Cesium.Geometry({
                attributes : {
                    position : new Cesium.GeometryAttribute({
                        componentDatatype : Cesium.ComponentDatatype.FLOAT,
                        componentsPerAttribute : 3,
                        values : pos
                    }),
                    normal:new Cesium.GeometryAttribute({
                        componentDatatype : Cesium.ComponentDatatype.FLOAT,
                        componentsPerAttribute : 3,
                        values : new Float32Array([0.0,0.0,1.0,0.0,0.0,1.0,0.0,0.0,1.0])
                    })
                },
                indices : new Uint16Array([0,1,2]),
                primitiveType : Cesium.PrimitiveType.TRIANGLES,
                boundingSphere : Cesium.BoundingSphere.fromVertices(pos)
            });

            var myInstance = new Cesium.GeometryInstance({
                geometry : geometry,
                attributes: {
                    color: Cesium.ColorGeometryInstanceAttribute.fromColor(Cesium.Color.RED)
                }
            });

            viewer.scene.primitives.add(new Cesium.Primitive({
                geometryInstances : [myInstance],
                appearance : new Cesium.PerInstanceColorAppearance({
                    closed : true,
                    translucent : false
                })
            }));

Thank you!
However in China, it is not available to visit the Cesium forum. I have to ask some questions by email.
From some example ,I know in order to draw triangles,we should set the position and indices.
Here is my code.
but it says that
"DeveloperError: Appearance/Geometry mismatch. The appearance requires vertex shader attribute input 'position3DHigh', which was not computed as part of the Geometry. Use the appearance's vertexFormat property when constructing the geometry."
I don't know how to solve this problem?
var mypositions = Cesium.Cartesian3.fromDegreesArrayHeights([ 104.317776,31.59491,10,
105.317776,32.59491,20,
106.317776,33.59491,30]);
var pos=new Float64Array(3*3);
pos[0]=mypositions[0].x;
pos[1]=mypositions[0].y;
pos[2]=mypositions[0].z;
pos[3]=mypositions[1].x;
pos[4]=mypositions[1].y;
pos[5]=mypositions[1].z;
pos[6]=mypositions[2].x;
pos[7]=mypositions[2].y;
pos[8]=mypositions[2].z;
var geometry = new Cesium.Geometry({
attributes : {
position : new Cesium.GeometryAttribute({
componentDatatype : Cesium.ComponentDatatype.FLOAT,
componentsPerAttribute : 3,
values : pos
}),
normal:new Cesium.GeometryAttribute({
componentDatatype : Cesium.ComponentDatatype.FLOAT,
componentsPerAttribute : 3,
values : new Float32Array([0.0,0.0,1.0,0.0,0.0,1.0,0.0,0.0,1.0])
})
},
indices : new Uint16Array([0,1,2]),
primitiveType : Cesium.PrimitiveType.TRIANGLES,
boundingSphere : Cesium.BoundingSphere.fromVertices(pos)
});
var myTriangle = new Cesium.GeometryInstance({
geometry : geometry,
attributes: {
color: Cesium.ColorGeometryInstanceAttribute.fromColor(Cesium.Color.RED)
}
});
viewer.scene.primitives.add(new Cesium.Primitive({
geometryInstances : myTriangle,
appearance : new Cesium.PerInstanceColorAppearance({
closed : true,
translucent : false
})
}));
At 2014-12-26 03:08:38, "Patrick Cozzi"<EMAIL_ADDRESS>wrote:
Please ask questions like this on the Cesium forum.
If you are making a custom geometry, a simple example to look at is the box geometry:
BoxGeometry.js
createBoxGeometry.js
—
Reply to this email directly or view it on GitHub.


Inventory Collection Builder

Issue: https://github.com/ManageIQ/manageiq/issues/17396

[ ] (dependent) PR: https://github.com/ManageIQ/manageiq-providers-amazon/pull/438
[ ] (dependent) PR: https://github.com/ManageIQ/manageiq-providers-ansible_tower/pull/81
[ ] (dependent) PR: https://github.com/ManageIQ/manageiq-providers-azure/pull/254
[ ] (dependent) PR: https://github.com/ManageIQ/manageiq-providers-nuage/pull/86
[ ] (dependent) PR: https://github.com/ManageIQ/manageiq-providers-openstack/pull/288
[ ] (dependent) PR: https://github.com/ManageIQ/manageiq-providers-ovirt/pull/252

New Interface for definition/creating InventoryCollection from persister.
Accessible via ManagerRefresh::Inventory::Persister.add_collection() method
Intended to replace:

add_inventory_collection() method
has_inventory() method

Classes ManagerRefresh::InventoryCollection::Builder and descendants can replace:

ManagerRefresh::InventoryCollectionDefault and subclasses (using add_inventory_collection interface)
ManagerRefresh::Inventory::Core module (and classes including it - using has_inventory interface)

IC model class is derived from persister class and IC association value
IC inventory_object_attributes are derived from setters of model_class
(can be disabled or redefined)
Cc @Ladas, @agrare, @kbrock

I have few comments, but overall this is a great work @slemrmartin :+1:

Time to review. Old interfaces and files have to be removed in separate PR after merge of all dependent PR's

@slemrmartin will this change affect also the newly added Redfish provider? This is a physical provider, similar to Lenovo's XClarity (contrary to Lenovo we are using inventory instead of the legacy parser), e.g.

https://github.com/ManageIQ/manageiq-providers-redfish/blob/master/app/models/manageiq/providers/redfish/inventory/persister.rb
https://github.com/ManageIQ/manageiq-providers-redfish/blob/master/app/models/manageiq/providers/redfish/inventory/persister/physical_infra_manager.rb

/cc @tadeboro

Redfish provider will need some fixes when this gets merged, but those will be minimal, since we have no targeted refresh implemented yet.
Also, after the pull requests #17426 is merged, we will need to add a builder for physical infra manager if I understand changes in this pull request correctly.

@gberginc At this moment, this PR defines new interface but keeps the old ones. If you're implementing Redfish provider using add_inventory_collection(), merge of this code doesn't affect your code.
@tadeboro Yes, you're right. It should be added before moving Redfish to this interface

@agrare, @Ladas The most controversial feature of this interface is auto-guessing of inventory object attributes.
It's up to you to make a decision about it

@slemrmartin if we're auto-detecting the model class how much of the specific provider collections (e.g. https://github.com/ManageIQ/manageiq-providers-amazon/pull/438/files#diff-0f974d25c81f9b0090224fd98112a9d6) can just be in ManageIQ::Providers::CloudManager::Inventory::Persister?
I'd like to get to the point where we only need any collection definitions there to override the base cloud/infra/etc... definitions.

@agrare That will be subject of my next step. I think at least amazon and azure have some "duplicates" but I need to do more analysis. It'll be better to refactor all providers of one type (cloud/infra) without moving definitions among gems and then extract common at once.

@gberginc @tadeboro Is Redfish ready to refactoring? which means enough tests which covers refresh process

The refresh process for the Redfish provider is decently covered with tests. That being said, even if you break something when refactoring, the world will not stop turning, since I am the only one currently using this provider. So feel free to refactor.


Segfault when importing global file to DB

Context
I've separated out my read and write commands into two different steps. I am running in minimal docker images and experienced this panic after ~1.5hrs of processing.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x5a0231]

goroutine 290 [running]:
github.com/omniscale/imposm3/cache.(*DeltaCoordsCache).CheckCapacity(0xc00012c3c0)
      /root/imposm/gopath/src/github.com/omniscale/imposm3/cache/delta.go:352 +0x111
github.com/omniscale/imposm3/cache.(*DeltaCoordsCache).getBunch(0xc00012c3c0, 0x16a7e1d7)
      /root/imposm/gopath/src/github.com/omniscale/imposm3/cache/delta.go:330 +0x3c5
github.com/omniscale/imposm3/cache.(*DeltaCoordsCache).FillWay(0xc00012c3c0, 0xc004a22780)
      /root/imposm/gopath/src/github.com/omniscale/imposm3/cache/delta.go:180 +0x193
github.com/omniscale/imposm3/writer.(*RelationWriter).loop(0xc00018ae70)
      /root/imposm/gopath/src/github.com/omniscale/imposm3/writer/relations.go:91 +0x31b
created by github.com/omniscale/imposm3/writer.(*OsmElemWriter).Start in goroutine 1
      /root/imposm/gopath/src/github.com/omniscale/imposm3/writer/writer.go:52 +0x3e

Expected Behavior
imposm continues to run till completion, exit 0. This worked fine with the smaller regions of monaco and central USA.
Actual Behavior
Panic, segfault
Possible Fix
Unknown
Steps to Reproduce

Using FROM debian:12.8, install released version v0.14.0
Download global OSM pdb from S3, aws s3 cp --no-sign-request s3://osm-pds/2024/planet-241202.osm.pbf ./country.pbf
Run via import -mapping /config/mapping.yml -write -cachedir /cache -connection postgis://osm:osm@db/osm -srid 4326 -deployproduction -optimize

Context
I'm trying to get some local maps running to turn them into PMTiles
Your Environment
Debian Linux running Docker
Linux [hostname] 5.10.0-28-amd64 #1 SMP Debian 5.10.209-2 (2024-01-31) x86_64 GNU/Linux
sudo docker version
Client: Docker Engine - Community
 Version:           26.0.1
 API version:       1.45
 Go version:        go1.21.9
 Git commit:        d260a54
 Built:             Thu Apr 11 10:53:52 2024
 OS/Arch:           linux/amd64
 Context:           default

Server: Docker Engine - Community
 Engine:
  Version:          26.0.1
  API version:      1.45 (minimum version 1.24)
  Go version:       go1.21.9
  Git commit:       60b9add
  Built:            Thu Apr 11 10:53:52 2024
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.31
  GitCommit:        e377cd56a71523140ca6ae87e30244719194a521
 runc:
  Version:          1.1.12
  GitCommit:        v1.1.12-0-g51d5e94
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Thanks all, great project!

Yeah this is not handling invalid data correctly...
panic: proto: Way: illegal tag 0 (wire type 7)

goroutine 195 [running]:
github.com/omniscale/imposm3/cache.(*WaysCache).Iter.func1()
      /root/imposm/gopath/src/github.com/omniscale/imposm3/cache/ways.go:90 +0x17a
created by github.com/omniscale/imposm3/cache.(*WaysCache).Iter in goroutine 1
      /root/imposm/gopath/src/github.com/omniscale/imposm3/cache/ways.go:77 +0x78
import-write-1 exited with code 2


Add example for NST via model optimization

Cc @t-vi. We could port the weights from the PyTorch example, since we are fully compatible. I'm waiting for https://github.com/pytorch/examples/issues/837 if this possible within the license.

Cool, thank you!


Community backup and restore

Content in this branch is adapted from original files in /molecula/documentation

/_includes/com-config > include files relating to those in /docs/community/com-config
/docs/community/com-config > Community configuration help, including a homepage, backup and restore pages

It's tested and builds.
Broken links relate to other files coming to /com-config and sql-guide/statements files coming in existing PR

Hi @greg, thrown to you because you'll prolly know who needs to actually review this :) More PRs coming for the other config stuff.

@ljsinclair throwing this in a ticket for eng team: https://molecula.atlassian.net/browse/FB-1933.
Do the changes to the help on help and css files need to be here? Can those be removed and put in a separate PR?

@ljsinclair throwing this in a ticket for eng team: https://molecula.atlassian.net/browse/FB-1933.
Do the changes to the help on help and css files need to be here? Can those be removed and put in a separate PR?

I just merged with MAIN which already has the change to CSS so this should remove the issue :)

@ljsinclair - is this the final version or there will be more commits?

Okedokey, I've fixed everything @HHans09 suggested, and as far as I can tell this is ready to roll :)

@ljsinclair this was approved. Anything holding this up from merging (other than the conflicts you need to address :) )


Make Territory tree box resizable

Territory tree box stopped to be resizable. Not high priority, i.e. does not prevent us from working, but quite inconvenient on smaller screens.

@davidzbiral  it never was

I have edited the name of the issue - should not be a problem to implement this even this was never part of the application.

@adammertel OK, it somehow feels smaller. I thus change it into a feature request: make it resizable. Thanks!

@davidzbiral It could feel smaller because of adding the root T so everything has one lvl more indentation.
Ok I'm gonna implement that.

@Ptrhnk I think that's it, you found the reason for the feeling of the panel being smaller!

Moving to 1.4.1 as this might take more time to implement.


markdown: Refactor Markdown element spacing.

This PR adjusts the inter-element spacing for all rendered-Markdown elements. The purpose of these adjustments is to make it easier to create a reasonable set of variables for declaring space between Markdown elements as part of the user-adjustable information density project, #12611.
The PR shifts all spacing to margin-bottom, rather than a combination of top and bottom margin, except for specific sibling combinators (e.g., p + p), where margin-top is necessary to increase the desired spacing when one element follows a specific other.
Additionally, space around the rendered-Markdown area that was previously set by margins bleeding out from Markdown elements is now more reliably set by top and bottom padding on the Markdown area itself.
In addition to making the rendered-markdown area easier to reason about within the .messagebox-content grid, there are subtle improvements here, including especially that all messages begin on the same vertical line, with or without a sender. Previously, for example, single-line list item messages were taller than single-line paragraph messages. Those are now uniform. Relatedly, a message opening with a list sat a pixel higher in the message area than those messages that opened with a paragraph.
Spacing around widgets is now also uniform, and in the case of poll widgets, achieved purely through CSS (previously, the poll widget depended on a break tag to maintain its bottom spacing (!).
There are, however, two elements that are not changed by this PR, which introduces some additional spacing when these elements appear at the bottom of the message list:

[ ] Image thumbnails, which should wait to be addressed until #29300 is solidified and merged; in the meantime, there will appear to be 3px of additional padding beneath messages ending with image thumbnails.
[ ] Spoilers, which have the largest current margin-bottom of any custom Markdown by far, at 15px (18px at the bottom, now, with the changes proposed in this PR). There should probably be a design discussion about spoilers, as hinted at by @alya in a comment on #29715.

Additionally, a follow-up PR will introduce a set shared line-height for the sender row and timestamp. That will better prepare adjustments to the icons in the hover area as well as ensure that timestamps associated with inline media and KaTeX (as can especially be seen in the KaTeX screenshot below) no longer shift into unpredictable vertical orientation with regard to the message content.
Screenshots and screen captures:
Note that comparing before & after images is in certain cases less useful than comparing after images against other after images, e.g., for seeing how message text all begins on the same horizontal line with this refactoring.

One-liners, Before
One-liners, After

Previews, Before
Previews, After

Widgets, Before
Widgets, After

Cool! Most of the changes seem good or neutral to me (just visually comparing the screenshots). The two where it seems like there's two much space now are the ones you noted: images and spoilers.
Do we need to do a design pass on spoilers, or could we just reduce the amount of space around them?

Looking at these, I'm also feeling like the spacing might be too tight within a message around quotes, and maybe around bulleted/numbered lists as well. (This PR doesn't affect it one way or the other.)

@alya said:

Cool! Most of the changes seem good or neutral to me (just visually comparing the screenshots). The two where it seems like there's two much space now are the ones you noted: images and spoilers.
Do we need to do a design pass on spoilers, or could we just reduce the amount of space around them?

Let me open a #frontend discussion with your questions and we can talk about it there. My feeling is that spoilers should be treated space-wise the same as the poll and todo widgets (same with other widget-like content, including link previews and images). That would look good and predictable. But it's definitely something worth discussing on CZO.

Okay. So what I've elected to do here is just normalize the margin-bottom across the two outlier elements that didn't already have a 5px value: p (which had been 3px) and blockquote (which had been 6px). That was standing in the way of using a negative margin value to pull back against the bottom margin of the Markdown elements, because there was that lack of uniformity.
But in light of this CZO discussion about setting 5px beneath widgets, it seems reasonable to unify all margin-bottom values around that.
That being said, I was not satisfied with the initial results of using negative margins on the rendered-Markdown area. So what I've done in that regard is to remove any zeroing of the bottom margin on Markdown elements: now that they're all 5px, they'll at least be uniform. Further adjustments along those lines can be addressed more confidently in a future PR (just the element normalization here invites enough regression potential already). For now, in other words, this PR maintains the status quo with how Markdown elements participate in the sizing of the message box.
Here are some select updated screenshots showing how things stand as of the force push above this comment:

One-liners, before
One-liners, after

Widgets + text, before
Widgets + text, after

Works for me!

Cool, I'll get this updated on chat.zulip.org today.


Don't publish all users with details to logged-out users

When not logged in, the users collections is still published to the client with all details, which means, that all email addresses, real names and roles are visible on the client:
One such message on the DDP:
a["{\"msg\":\"added\",\"collection\":\"users\",\"id\":\"KZsNMd7AC5kcu5yLj\",\"fields\":{\"emails\":[{\"address\":\"real.name@email.de\",\"verified\":true,\"fromLDAP\":true}],\"profile\":{\"name\":\"Real Name\"},\"roles\":{\"5zJmLcj4mCeuH86mZ\":[\"10\"]},\"username\":\"userid\"}}"]

For details see the Chrome Developer network view of the DDP websocket: F12 -> Network -> "websocket" -> Frames

Interesting observation!
We have a setting in server side configuration settings.json called "trustedIntranetInstallation". This is set to "true" in our internal instalation, but "false" on e.g. 4minitz.com.
This setting lowers/raises the grade of confidentiality on exposing data to the clients and/or users - e.g. while sending protocols to participants. When set to false, indivisual, seperate emails are sent to each participant, thus keeping the mail adresses of others private.
Regartding your case this setting configures, if mail adresses are published to the client at all - or not.
So, could you please check in https://www.4minitz.com if you see something "spooky" there?

On 4minitz.com all users are still published, but only with their login, so no real names, no email adresses.
The DDP messages look like this:
a["{\"msg\":\"added\",\"collection\":\"users\",\"id\":\"XkjKit8cz8HXtjjTP\",\"fields\":{\"username\":\"rain\",\"roles\":{\"AESFC9TuqcsjxYsJH\":[\"01\"]}}}"]

So good to know that the trustedIntranetInstallation flag also removes real names from the user list.

So, on "trustedIntranetInstallation: false" nothing really bad happens...
Nevertheless with a fix we'll prevent to publish any user data to the client if no user is logged in.
Thx. for the hint.  ;-)


add gitignore file

May as well add coverage too. and
# Logs
logs
*.log
npm-debug.log*

I think thats all we need.
(see https://www.gitignore.io/api/node for example node gitignore and desirable order.)
I didn't set this up yet because I have global gitignores set up for all this stuff.
Good idea to put it in the repo.

@camjc @omgaz Merge please?

Merging! :zap:


[WFCORE-3622] Fix the SuspendOnSoftKillTestCase cleanup work

https://issues.jboss.org/browse/WFCORE-3622
Test was calling serverController.start() on a killed controller, assuming that would start the server again to allow a subsequent undeploy op. But the serverController didn't know it was killed, so the 'start()' call did nothing and the undeploy wasn't getting executed.
Use https://github.com/wildfly/wildfly-core/pull/3119/files?w=1 to look at this or the change looks much bigger due to removal of a try/catch/finally surrounding the main part of the test.

#3115 fixes the test failure.


ATTN: Status of this seed

Everyone who comes upon this seed please read this first:
This seed was originally developed for the following reasons:

To enhance Minko Gechev's angular-seed with more feature support out of the box
Provide a proof of concept to how flexible Minko Gechev's seed actually is
Push the boundaries for what is possible in a monorepo to handle a full shared codebase between Web and Native mobile iOS/Android and Desktop
Provide ideas and open up a conversation around the type of multi-platform development made possible by all 3 very exciting technologies: Angular and NativeScript and Electron

The parent angular-seed is built off SystemJS. At the time of it's first conception a lot was evolving in modern JavaScript development. Most importantly angular-cli has come to fruition over this time period as well as the amazing progressions of webpack as a build tool.
This seed will always attempt to mirror all the latest changes from Minko's seed which is rooted in SystemJS which will always make this seed rooted in SystemJS for at least the web build side of things at the moment.
Free free to use the seed if it suits your needs and/or improve upon it in ways you believe should be. I continue to monitor pull requests however have not been able to respond to issues lately. At the moment I plan to take some time in late Fall 2017 to bring a number of things up to date with this seed with latest versions of Angular/NativeScript/Electron and apologize to anyone who relies on this seed for more support as I have been busy on other things.
In meantime, I would encourage those who come upon this seed to evaluate several things:

Give it a try, if it works for your case, you are welcome!
If you find ways to improve it, please consider contributing back 👍
Explore this option: https://github.com/TeamMaestro/angular-native-seed
Continue exploring to find what works well for your project goals.

❤

Wonderful! Thank you for the great work.
Soon I'll be reaching you out to discuss some improvements I've made on modularization and theming, which I'm using (and reusing) with success on my projects.
-- luiz

Can we use the seed in production environment? Is there a list of known issues? Is it being updated Regularly?


cmake:init protected-mode for CMake build

Summary
This patch refines protected-mode based on the current kernel libs SPLIT feature.

adjust link options for userspace elf
disable link option in Toolchain file for userspace elf
specify system libs and apps lib to only link with nuttx target in flat build mode

Impact
enbale protected-mode build
Testing

cmake -B build -DBOARD_CONFIG=lm3s6965-ek:qemu-protected

# because the current CMake support is not perfect yet,
# need to close  EXAMPLES_ELF ,  EXAMPLES_MODULE  and NSH_SYMTAB

cmake --build build

# the binary product is in the build directory and can be tested using QEMU
qemu-system-arm -semihosting \
        -M lm3s6965evb \
        -device loader,file=build/nuttx.bin,addr=0x00000000 \
        -device loader,file=build/nuttx_user.bin,addr=0x00020000 \
        -netdev user,id=user0 \
        -nic user,id=user0 \
        -serial mon:stdio -nographic

@xuxin930 let's switch some kernel/protected defconfig from Makefile to CMake to improve the ci coverage.

@xuxin930 let's switch some kernel/protected defconfig from Makefile to CMake to improve the ci coverage.

Okay, then I also need to port some features, such as elf module and SYMTAB


Is this Version Compatible with Multiple Size of Devices?

Hello, I want to ask if this version is compatible and adjustable for all kinds of size in devices?
If it is not, is there anyway to customize it to fit our needs?
Thank you very much.

Hello @gOzaru , Current version doesn't supports size adjustments, We are working on it to make it fully customisable, it will be out soon in few days in next version. stay tuned!

Hello @gOzaru  , It's auto responsive for web, desktop & mobile but doesn't supports size adjustments, We are working on it to make it fully customisable, it will be out soon in few days in next version. stay tuned!

I got an error where the image file is no available....
Maybe you can fix this.
I combined this widget with Flutter Screenutil and it managed to get resized without doing anything.
Well done.

@gOzaru We have fixed the image error, please update the version to the latest from your pubspec.yaml file.

Okay, thank you for the hard work.
All authors of Dart packages which I used inside the project will be regarded as contributors and they will receive rewards from company later from the net revenue. I will let you know as you are also considered as one of them.
Keep making a good package.
Oh yeah,
I've been looking also for tutorials of how to run Flutter with Cloud Run. If you know one, let me know. All tutorials authors which I learned from whether it is on YouTube or website [medium, etc.], they will also receive this same reward.


remove middleman & ruby

@iphilgood this is the big PR for Release v3.0.0

switch from sass to scss syntax
switch from grunt to gulp
switch from haml to jade
add browser-sync for local server functionality
add gulp compilation of sass, coffee, jade, bourbon, neat, icons, ...
replace grunticon with gulpicon
remove ruby & middleman dependency

add .nvrmc file for nvm
add scss-lint.yml (for local scss-lint in e.g. sublime) with rules
lint all sass files


Add antijoin verb

This may be possible with other verb combinations, but an antijoin operation is useful in some data manipulation situations.
Join T1 to T2 on KEY, deleting entries in T1 that had join values.

Hey, I wanted to work on this issue. Has it been solved/taken up by someone else already?


(124302607) Path-based CocoaErrors should use both NSFilePathErrorKey and NSURLErrorKey

Our recent rewrites of Data/FileManager have switched around whether we use NSFilePathErrorKey or NSURLErrorKey in some implementations. While we don't make any guarantees about which will be used in a given API, we can update the implementation to provide values for both keys for compatibility.

@swift-ci please test


Support alias names for bundled templates

This new feature allows us to create alias names for existing templates.

To create an alias template, simply create a text file named <alias_name>.alias whose first line just contains exactly the name of the template to point to.
Note: Any text after the first line of this *.alias files is ignored, so the next lines in the file can be used as comments if necessary.

Introducing android-java and swift4 template names
This feature allowed us to:

Rename the original template android to android-java, for better consistency with the new android-kotlin template
But still provide an alias to it via the name android, so that using -t android will still work (and not break the setup for existing users of gyro) and use the same template as if we used -t android-java explicitly
Also provide an alias named swift4 pointing to the template swift3, since the swift3 template is also compatible with Swift 4 so using the alias allows for a more logical name in the invocation of gyro.

This PR also closes #29

I have a question about alias for custom path. Do you want to authorize creating custom alias for custom templates or not ?
Actually we have two resolution methods for templates. Template.find_by_path(path) and Template.find_by_name(name), this last method implement alias resolution but not the first.


Django Q is still a young project.

Django Q is still a young project.

From: https://django-q.readthedocs.io/en/latest/install.html#compatibility
the project is six years old. Do you still think it is young?

There hasn't been much development on it since the early days. Very few major upgrades. So yes, I would still call it young. I guess it's more about maturity through iterations than it is just the passing of time.

Thank you for your fast reply. If you think this is still true, then I close this issue.


fixed parsing failure when additional spaces occur before semicolon

when VAL_ or VAL_TABLE_ finish with semicolon that is prefixed by multiple spaces (3 in my case), then the ExtensionsAndHelpers.ToDictionary records variable will hold \r\n  (there is space after). This causes the tokens will be an array of two items with values of string.Empty for both. This causing int.Parse to fail.
I've fixed that so it won't occur anymore.

@Whitehouse112, please review this contribution.
Thanks
A

@Puchaczov  thanks for spot the issue and fix it. It will be merged for the next release
Regards

Thank you guys for the library, it is super usefull :-)


Extra space added after line using icon within paragraph

Here's an odd thing that happens when I use a fontawesome icon :

I don't see any line break in the source code though :

Paragraph

Lorem ipsum dolor sit amet consectetur  adipisicing elit. Quam nihil enim maxime corporis cumque totam aliquid nam sint inventore optio modi neque laborum officiis necessitatibus, facilis placeat pariatur! Voluptatem, sed harum pariatur adipisci voluptates voluptatum cumque, porro sint minima similique magni perferendis fuga! Optio vel ipsum excepturi tempore reiciendis id quidem? Vel in, doloribus debitis nesciunt fugit sequi magnam accusantium modi neque quis, vitae velit, pariatur harum autem a! Velit impedit atque maiores animi possimus asperiores natus repellendus excepturi sint architecto eligendi non, omnis nihil. Facilis, doloremque illum. Fugit optio laborum minus debitis natus illo perspiciatis corporis voluptatum rerum laboriosam.

Here's how to reproduce the issue :

Use the exampleSite from the repo.
Add the fontawesome CDN to layouts/partials/head.html.
Use a fontawesome icon within any paragraph.

Any advice on how to get rid of the extra spacing would be appreciated.
Thanks for the nice theme !

Hello @cacsfre ,
Thanks for using our template, it would be great if you add a star to the repository.

Hello @cacsfre ,
Thanks for using our template, it would be great if you add a star to the repository.

Thanks @somratpro ! I just added a star. I see that you closed the issue, does it mean this is expected behaviour ?

Thanks @somratpro ! I just added a star. I see that you closed the issue, does it mean this is expected behaviour ?

Hello @cacsfre
I am really sorry about that. I may be misunderstood your issue. I am reopening it now. Can you give me any live URL so I can check it out?

Hello @cacsfre
I am really sorry about that. I may be misunderstood your issue. I am reopening it now. Can you give me any live URL so I can check it out?

Live URL replicating the issue using the exampleSite. The code is here just in case.

Live URL replicating the issue using the exampleSite. The code is here just in case.

Hi @cacsfre
Try this

Hi @cacsfre
Try this

That works, thank you @somratpro !

That works, thank you @somratpro !


Update sws_extension.vcxproj

added Amagalma files

amagalma asked me to close his PRs because the added functionality is available meanwhile.


FAILED: CMakeFiles/native-lib.dir/native-lib.c.o

Solved

Hi, How did you solve this

Solved

Hi, How did you solve this.


Error handling

When including file doesn't exist gulp takes off. And I have to use gulp-plumber. It's bad.
So, I recommend you to use for exceptions throw new gutil.PluginError(pluginName, message); (new necessarily) and to check file existance with fs.existsSync(filename);

Hm.. it's strange.
Sometimes gulp catches an errors, sometimes crashes in:
events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: recursion detected in file: D:\_host\githost\gulp-file-include\example\index.txt

Maybe problem is in gulp itself.
Error exception in your code no need to replace. It's good for now. I just replaced event-stream with through2 in pull request.

maybe relate to other plugins.

No, I use just include for html

Gulp-plumber catches errors, but it doesn't point to the file where the error occurred, for ex.:
[14:52:14] Plumber found unhandled error:
 Error in plugin 'gulp-file-include'
Message:
    active is not defined

Any way to know the file with the error? Sometimes it is almost impossible to find it manually.

1


E_DEPRECATED Non-static method

When error_reporting = E_ALL calls handler_order lead to error E_DEPRECATED Non-static method

Did you run this locally? Build failed:
PHP Fatal error:  Declaration of Slim\Handlers\Strategies\RequestResponse::__invoke() must be compatible with Slim\Interfaces\InvocationStrategyInterface::__invoke($callable, Psr\Http\Message\ServerRequestInterface $request, Psr\Http\Message\ResponseInterface $response, array $routeArguments) in /home/travis/build/slimphp/Slim/Slim/Handlers/Strategies/RequestResponse.php on line 19

Okay, I wanted to show that the use of "callable" leads to an error in the future (although not critical).
I am ready to correct all of the code which uses this interface, if it's interesting.
In my project, I made this inetfeys the scope of the framework.

Well I do not fully understand what you are trying to solve here.
When would the callable lead to a deprecated warning?
And what is handler_order? I cannot find it as PHP function.

When the class is being Route on line 312 or 316 is a call to a function of class building InvocationStrategyInterface, it provides php depricated on callable reporting that function should be static if it is a function of class.
Example:
namespace App\DependencyInjection;

use Interop\Container\ContainerInterface;

use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;

class FoundHandler implements InvocationStrategyInterface
{
    /** @var \DI\Container */
    protected $container;
    public function __construct(ContainerInterface $container){
        $this->container = $container;
    }
    /**
     * Invoke a route callable.
     *
     * @param callable $callable The callable to invoke using the strategy.
     * @param ServerRequestInterface $request The request object.
     * @param ResponseInterface $response The response object.
     * @param array $routeArguments The route's placholder arguments
     *
     * @return ResponseInterface|string The response from the callable.
     */
    public function __invoke(callable $callable, ServerRequestInterface $request, ResponseInterface $response, array $routeArguments)
    {
        if(is_string($callable) && strpos($callable, '::') !== false){
            $callable = preg_split('/::/', $callable, 2);
        }

        //TODO формирование Request и Response, т.к. возможны subRequest
        $args = array(
            'request' => $request,
            'response' => $response
        );

        $args = array_replace($args, $routeArguments);

        return $this->container->call($callable, $args);
    }
}

Error:
Strict Standards:  Non-static method App\Controller\Test::hotelList() should not be called statically in C:\OpenServer\domains\hotelengine\app\DependencyInjection\FoundHandler.php on line 34
P.S. error_reporting = E_ALL

Ahh, I think I found the problem now.
Since you are using the syntax class::method PHP will return true for is_callable() whether the method can actually be called staticly or not. This is why CallableResolverAwareTrait::resolveCallable() returns the string instead of a \Closure.
Further on in the InvocationStrategyInterface::invoke() a callable is expected. Now PHP does not reconize the string as callable because the method cannot be called staticly.
So the root if the problem lies in that PHP thinks it can resolve a string formatted as class::method.

I guess I do not understand something, but my calls have the form:
$app->group('/api/b2b/v1', function () use ($app) {
	$app->group('/hotel', function () use ($app) {
		$app->map(['GET', 'POST'], '/list[/{id:\d+}]', [\App\Controller\Test::class, 'hotelList']);
		$app->map(['GET', 'POST'], '/rates', [\App\Controller\Test::class, 'hotelRates']);
	});
});

Ah well same thing. Although then I do not unserstand the following code in you example as you are not passing a string:
if(is_string($callable) && strpos($callable, '::') !== false){
    $callable = preg_split('/::/', $callable, 2);
}

But back to the case:
PHP returns true for is_callable([\App\Controller\Test::class, 'hotelList']), because it only checks if there is a method named hotelList in the class App\Controller\Test.
Bu when passed as argument to a method which expectecs a callable (typehint) PHP sees that [\App\Controller\Test::class, 'hotelList'] is not actually callable, because the method is not static and cannot be called.
Do you see the difference?

If you want to use PHP class name resolution with the default InvocationStrategies provided in Slim you will have to do it like:
$app->group('/api/b2b/v1', function () use ($app) {
    $app->group('/hotel', function () use ($app) {
        $app->map(['GET', 'POST'], '/list[/{id:\d+}]', App\Controller\Test::class . ':hotelList');
        $app->map(['GET', 'POST'], '/rates', App\Controller\Test::class . ':hotelRates');
    });
});

If you decide to implement your own Strategy I am afraid we can't help you much with that, but the signature of these Classes are unlikely to change at the moment.

I think this is just an oversight in the implementation of the default strategy. I have no problems with those type hints being removed. However, it should not be just string as the @param phpdoc.

Thank you very much, I like the idea to make $callable string|array|callable.

No, I think InvocationStrategyInterface MUST receive a callable. If you want a different notation you should solve that with a custom CallableResolverInterface implementation.

Thank you very much, so I did.
namespace App\DependencyInjection;
use RuntimeException;
use Interop\Container\ContainerInterface;
use Slim\Interfaces\CallableResolverInterface;

use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;

final class CallableResolver implements CallableResolverInterface
{

	/**
	 * @var \DI\Container
	 */
	private $container;

	/**
	 * @var string
	 */
	private $toResolve;

	/**
	 * @var callable
	 */
	private $resolved;

	/**
	 * @param ContainerInterface $container
	 * @param string             $toResolve
	 */
	public function __construct(ContainerInterface $container, $toResolve = null)
	{
		$this->toResolve = $toResolve;
		$this->container = $container;
	}

	/**
	 * Receive a string that is to be resolved to a callable
	 *
	 * @param  string $toResolve
	 */
	public function setToResolve($toResolve)
	{
		$this->toResolve = $toResolve;
	}

	/**
	 * Resolve toResolve into a closure that that the router can dispatch.
	 *
	 * If toResolve is of the format 'class:method', then try to extract 'class'
	 * from the container otherwise instantiate it and then dispatch 'method'.
	 *
	 * @return \Closure
	 *
	 * @throws RuntimeException if the callable does not exist
	 * @throws RuntimeException if the callable is not resolvable
	 */
	private function resolve()
	{
		// if it's callable, then it's already resolved
		if (is_callable($this->toResolve)) {
			$this->resolved = $this->toResolve;

			// check for slim callable as "class:method"
		} elseif (is_string($this->toResolve)) {
			$callable_pattern = '!^([^\:]+)\:([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)$!';
			if (preg_match($callable_pattern, $this->toResolve, $matches)) {
				$class = $matches[1];
				$method = $matches[2];

				if ($this->container->has($class)) {
					$this->resolved = [$this->container->get($class), $method];
				} else {
					if (!class_exists($class)) {
						throw new RuntimeException(sprintf('Callable %s does not exist', $class));
					}
					$this->resolved = [$this->container->get($class), $method];
				}
				if (!is_callable($this->resolved)) {
					throw new RuntimeException(sprintf('%s is not resolvable', $this->toResolve));
				}
			} else {
				throw new RuntimeException(sprintf('%s is not resolvable', $this->toResolve));
			}
		}
	}

	/**
	 * Invoke the resolved callable.
	 *
	 * @return \Psr\Http\Message\ResponseInterface
	 */
	public function __invoke()
	{
		if (!isset($this->resolved)) {
			$this->resolve();
		}

		$args =  func_get_args();

		return  $this->container->call($this->resolved, ['request' => $args[0], 'response' => $args[1]] + $args[2]);
	}
}

As per https://github.com/slimphp/Slim/pull/1447#issuecomment-132149192, this should be solved via a CallableResolver.


rate us feature

Description
i would like to add rate us feature to the website.
kindly add required labels.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@Codewithmeowmeow right now we don't need it.

please ⭐ our repo

Join our official Discord server & discuss about new features and bugs raise issue & PR accordingly.

And also follow us on Linkedin


Fix agent tunnel address with dedicated supervisor port

Proposed Changes
Fix issue where rke2 tunnel was trying to connect to apiserver port instead of supervisor:
Note the incorrect port for the tunnel connection in RKE2 where the apiserver and supervisor are on different ports:
time="2024-12-06T20:35:57Z" level=info msg="Started tunnel to <IP_ADDRESS>:6443"
time="2024-12-06T20:35:57Z" level=info msg="Stopped tunnel to <IP_ADDRESS>:9345"
time="2024-12-06T20:35:57Z" level=info msg="Connecting to proxy" url="wss://<IP_ADDRESS>:6443/v1-rke2/connect"
time="2024-12-06T20:35:57Z" level=info msg="Proxy done" err="context canceled" url="wss://<IP_ADDRESS>:9345/v1-rke2/connect"
time="2024-12-06T20:35:57Z" level=info msg="error in remotedialer server [400]: websocket: close 1006 (abnormal closure): unexpected EOF"
time="2024-12-06T20:35:57Z" level=error msg="Failed to connect to proxy. Response status: 403 - 403 Forbidden. Response body: {\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"forbidden: User \\\"system:node:e8266a6ea5f6\\\" cannot get path \\\"/v1-rke2/connect\\\"\",\"reason\":\"Forbidden\",\"details\":{},\"code\":403}\n" error="websocket: bad handshake"
time="2024-12-06T20:35:57Z" level=error msg="Remotedialer proxy error; reconnecting..." error="websocket: bad handshake" url="wss://<IP_ADDRESS>:6443/v1-rke2/connect"

Types of Changes
bugfix
Verification
requires pull-through to rke2
Testing
Linked Issues

https://github.com/k3s-io/k3s/issues/11334

User-Facing Change

Further Comments

OK, all done poking at this one.


Expand intersect is not working

Hello, i am using version 2.1.4 and getting empty result on following query:
SELECT expand(intersect($q1, $q2))
LET q1 = (SELECT FROM Product WHERE title LUCENE 'ham*'),
q2 = (SELECT expand(IN('Belong')[@class = Product]) FROM Category WHERE type = "ebooks")

Strange is, that without expand I can see the list of records in intersect property.

Walk-around for this issue is not to use expand, but use intersect with the fetchplan:
SELECT intersect($neco, $neco2) as res
LET neco2 = (SELECT FROM Product WHERE title LUCENE 'ham*'),
neco = (SELECT expand(IN('Belong')[@class = Product]) FROM Category WHERE type = "ebooks")
fetchplan res:1

Any suggestion?

Hi @janjilek
it seems the intersect is not returning a proper object, so that expand can manage it.
Could you please try this and see if it returns the expected results?
select expand(foo) from (
SELECT intersect($q1, $q2) as foo
LET q1 = (SELECT FROM Product WHERE title LUCENE 'ham*'),
q2 = (SELECT expand(IN('Belong')[@class = Product]) FROM Category WHERE type = "ebooks")
)

In the meantime, I'm flagging it as a bug
Thanks
Luigi

Looks, that returns expected result!
Thanks
Jan

Fixed in both 2.1.x and develop branches.
Closing
Thanks