MultiSample VK_FORMAT_R32_SINT does not resolve correctly via vkCmdResolveImage on Intel UHD 6xx

Hi,
I believe I have found a bug when trying to use vkCmdResolveImage on a multi-sample R32 INT texture. User error not ruled out either :)
When doing a gpu-> cpu read back of the resolved texture, I don't get any values above 0 back.
(I do appear to be able to get values below 0, but only did limited testing on this).
The same setup works ok on Windows and Linux.
(I've tested Ubuntu 18 with Intel. Redhat 7 with Nvidia card, Window 7/10 with Nvidia cards)
When I switch the texture to multi-sample R32 FLOAT things also work via MoltenVk.
Or if I disable multi-sampling, but keep the R32 INT texture, I also get the correct values when doing the gpu->cpu read back.
I'm using:
Vulkan SDK <IP_ADDRESS>.
MacOs Mojave.
Intel gfx card.
I can privately provide a sample app if you have trouble reproducing this problem.
Thanks!
-Kees

@lumonix

I can privately provide a sample app if you have trouble reproducing this problem.

That would help. You can send it (or a link to it) to<EMAIL_ADDRESS>and I'll have a look at it.

Hi Bill,
I tried another machine with an Geforce 750 in it, which works fine for both the INT32 and F32 msaa buffers.
I believe this problem might be limited to Intel UHD 6xx (Maybe even only Mojave) and thus probably not a MoltenVK bug.
If you are still interested to see it, I've managed to strip down the app enough to publish it here:
https://github.com/lumonix/BugReports/blob/master/repro_bug828_msaa_int_vs_float.zip
The instructions are to run the executable and in the top-left where it says "No renderer" switch to the renderer to see two boxes on screen.
In the float-version you can pick either box.
In the int-version, on Intel, you can only pick the top box.
This is because in the int-version the int32 attachment is either not filled with the correct int-value that uniquely represents each object for picking. Or the read-back is going wrong and we just always get 0 back (or -1 when clicking in empty space).
This seems msaa related since without msaa it also works on Intel.
Feel free to close this bug if you cannot repro or believe with the new info its not a MoltenVk bug (which I believe it isn't, now that I tried other machines)

Thanks for submitting your sample app. It helped to figure this out.
The issue appears to be in the attempt to resolve the integer texture.
Metal supports MSAA for most integer formats, but does not support resolving integer formats into single-sample textures. Enabling Metal validation when running your app results in the following assertion appearing:
`_MTLValidateResolveTexture:340: failed assertion `PixelFormat MTLPixelFormatR32Sint cannot be a MSAA resolve target'`

A Vulkan implementation can inform the app that MSAA is supported for an image format via VkImageFormatProperties::sampleCounts, but unfortunately, there doesn't seem to be any way to inform the Vulkan app that it cannot be used as a resolve destination.
However, we should at least add error reporting to MoltenVK to return an error during the creation of a a render pass if it asks for a resolve target with an unresolvable format.
BTW...regarding your picking algorithm, the Vulkan spec says:

During the resolve the samples corresponding to each pixel location in the source are converted to a single sample before being written to the destination. If the source formats are floating-point or normalized types, the sample values for each pixel are resolved in an implementation-dependent manner. If the source formats are integer types, a single sample’s value is selected for each pixel.

So, even if Metal supported resolving the image, whether you use float or integer formats, the resolved pixel will contain an arbitrary value that might not be an accurate measure of the color from the MSAA pick-painted pixels. For float formats, the resolved value will be a blended value, not necessarily any actual painted value, and for integers, the resolved value will be fairly arbitrary. In both cases, the resolved picked color might not match what was painted.
Is there a reason why you are rendering the scene for picking using MSAA? Might it be better to define the integer-format picking render target as a single-sample image, and avoid resolving it altogether? That way, you'll avoid this issue, and I should think you'll get more accurate results from the pixel-picking. Plus it will likely be a bit more performant.

@billhollings thank you for the detailed answer.
Regarding why the pick buffer is multi-sample:
Doesn't Vulkan (like OpenGL) require all color attachments to have the same sample count?
So if I were to make the pick-buffer single-sample, I would have to render twice.
(Which could be an option, just confirming with you I am not missing out on some extension that allows mis-matching sample-count color attachments that MoltenVk also supports)
Thanks!

Doesn't Vulkan (like OpenGL) require all color attachments to have the same sample count?

Yup...that is the case...

All attachments in pColorAttachments that are not VK_ATTACHMENT_UNUSED must have the same sample count

I was thinking old-school, where the picking pass would only be rendered interactively as a secondary pass after a user interaction (click, touch, etc).

Makes sense. Thanks Bill.

I've left this as a bug until we add error reporting to MoltenVK to return an error during the creation of a a render pass if it asks for a resolve target with an unresolvable format.

PR #843 fixes this to report an error when a VkFormat cannot be used as a resolve destination:
[mvk-error] VK_ERROR_FEATURE_NOT_PRESENT: vkCmdResolveImage(): VK_FORMAT_R32_SINT cannot be used as a resolve destination on this device.

@billhollings so the solution to this problem was adding an error?
Looks like this is non-conformant behavior from Vulkan point of view. A portability implementation must be able to resolve non-filterable formats like the integer ones, or otherwise we need to work on an additional flag in the portability spec.

otherwise we need to work on an additional flag in the portability spec

Yes. Or for required feature capabilities, we can modify the spec wording itself to say a required format capability is not required in the presence of the VK_KHR_portability_subset extension.
I'm tracking those kind of proposed spec rewordings here, and platform-specific issues here, although I haven't documented this particular outlier in either of those issues, probably because it hasn't shown up, or been addressed from CTS testing.

The problem is that Vulkan doesn't have a specific capability for "resolve". Vulkan requires any MSAA texture to be able to resolve. So there is no easy way to just modify the spec wording.


Better file handling through Google Drive

Multiple users have been trying to import M3u playlists using the Google Drive option. Of course, this is not intended, causing the file to be interpreted as JSON and causing the app to constantly crash. This should be mitigated by either giving up-front warnings about the import or allowing users to import other types of files.

Adds warning for invalid Drive import


Add syntax highlighting

This adds syntax highlighting to the code snippets in the components section of the docs site. To see it in action, take a look at:
https://ascott1.github.io/capital-framework/components/
Changes

Moved to markdown-it for syntax highlighting as it is actively maintained and resulted in far fewer rendering issues
Added the syntax style from Prism.js (outputted by illuminate.js)

TODO

Unify syntax theme styles (though this needn't hold up the PR)

Ask me about
Every single node markdown parser and syntax highlighter. All of 'em.

Notable changes are in https://github.com/ascott1/capital-framework/blob/gh-pages/gulp/tasks/usage.js

Beautiful! :heart_eyes_cat:


Extend GitHub actions

Add job to test image
Add mender push
Some documentation
Enable docker login in tests

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


[devops:bot] update deps in nix/sources.json (from: launch-deversifi@4251f51

update nix refernces for launch-deversifi and|or nixpkgs to match https://github.com/rhinofi/launch-deversifi/blob/4251f51f593bfeb238eb0ba1aa1415eb3d7393f6/nix/sources.json (branch: dev)

superseded by #29


devspace build tries to create a namespace? Shouldn't it also work without a kubernetes cluster?

What happened?
I'm running devspace buildas one of the steps of a CI/CD pipeline, where I only want to build and push the images.
I don't have a valid kube config file because I don't intend to deploy the images at this stage.
Without a valid kube config file, I get:
Unable to create new kubectl client: kube config is invalid: please make sure you have an existing valid kube config. You might want to check one of the following things:
* Make sure you can use 'kubectl get namespaces' locally

If I create a formally valid but fake kube config file, I get:
[fatal]  unable to create namespace: get namespace: Get "https://kubernetes.docker.internal:6443/api/v1/namespaces/feat1": dial tcp: lookup kubernetes.docker.internal on <IP_ADDRESS>:53: no such host

What did you expect to happen instead?
It should just build and push the images.. Why does it try to create a namespace?
How can we reproduce the bug? (as minimally and precisely as possible)
Just put a formally valid but not real kube config file in ~/.kube/config
Local Environment:
devspace version 5.17.0.
mac and linux.
Kubernetes Cluster:
None
Anything else we need to know?

/kind bug

@wollowizard thanks for creating this issue! DevSpace tries to verify if the current namespace exists and if not tries to create it, in this case it cannot retrieve the current namespace because of the no such host error. It seems to me that the kubernetes cluster is not reachable with the kubeconfig you have provided

Hi @wollowizard good point. Thanks.
Kaniko and buildkit need valid kubeconfig to do incluster image build but we can try to refactor the build to not use the kubeconfig if it's not doing incluster build.

Hi @wollowizard good point. Thanks. Kaniko and buildkit need valid kubeconfig to do incluster image build but we can try to refactor the build to work without kubeconfig if it's not doing incluster build.

Hello Pratik, that would be realy useful. we are indeed using docker desktop (local) and Docker-in-Docker service (gitlab pipeline), not kaniko or buildkit. It would be great if you could do a "lazy initialization" of the k8s cluster connection, in order to initialize it just if it is really needed.

@wollowizard thanks for creating this issue! DevSpace tries to verify if the current namespace exists and if not tries to create it, in this case it cannot retrieve the current namespace because of the no such host error. It seems to me that the kubernetes cluster is not reachable with the kubeconfig you have provided

Hello @FabianKramm , I think you're absolutely correct. The point is, I do not really want or need to create a namespace on any cluster. I'm using this in a a build step of a pipeline with multiple steps. I'm using docker in docker for the build (no kakino). So I don't really want to configure devspace to access any cluster and try to create any namespace, just to build and push the image to the docker registry.
Indeed even at a later stage (deplyoment step of the pipeline) I don't want devspace to connect to k8s, but I'll just "render" the manifests ( see separate issue )and push them to my gitops repo, where that config will be picked up and deployed in k8s automatically (think argocd).
I believe devspace can be a great tool on local running minikube / k8s in docker desktop, but also on remote pipelines with a gitops approach. In order to do the latter, I'd ask you to please lazily initialize the k8s connection only when and if really necessary. Or maybe provide a flag to skip the k8s cluster steps (like a dry run ..)

@FabianKramm @pratikjagrut Created a PR. Thanks for taking a look


[Issue-2207] Browser waste time when load extension

Related issue(s)

#2207

Is your feature request related to a problem(s)? Please describe.

Decrease required data to display UI

Describe the solution you'd like

Decrease required data to display UI

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-2224--sw-web-runner.netlify.app


update with-react-intl example

Changes:

removed PageWithIntl HOC and moved that code to _app.js
updated README.md

Awesome PRs @Tomekmularczyk thank you very much for keeping them up to date 😌 🙌


Add gene_labels and labels to plot_genes()

Resolves #407.

Looking good...

@alimanfoo How does this padding look to you?
Ideally I'd like a more robust mechanism for accommodating these labels, because this is currently tailored to the default height of the plot, i.e. 120, so specifying a different height, e.g. height=240, results in a skewed layout, e.g.

Thanks @leehart, I think it's probably ok for now, generally unlikely to need to vary the height of this track by much.

Hi @leehart, could we rename the labels param to gene_labelset? That way when the parameter appears in functions like plot_h12_gwss() it will be more obvious that the parameter will apply to the genes track.

Looks like codecov/patch wants some test coverage for the new gene_labels param. Will do.

Looks like codecov/patch wants some test coverage for the new gene_labels param. Will do.

Cool, thanks @leehart. Happy to be merged once codecov/patch passes.


Update cell type checkpoints to include the updated reference file names

Related to https://github.com/AlexsLemonade/ScPCA-admin/issues/691
We recently updated the reference file names for both SingleR and CellAssign. Before doing this, we had previously run a few samples through both SingleR and CellAssign. To skip cell typing, we check that the reference file name stored in library_id_cellassign/scpca-meta.json and library_id_singler/scpca-meta.json match the reference file names that have been passed through the workflow via the project metadata. If we want to run the projects through again and skip running CellAssign for samples that already have CellAssign results, then these reference files need to be updated in the scpca-meta.json files.
Here I'm adding a script that specifically updates the cell type scpca-meta.json files to make sure that the reference file names match what's in scpca-project-celltype-metadata.tsv. It's mostly modeled after the script we use for updating the mapping related scpca-meta.json files, but here we need to update two checkpoint files per library, one for each method.
Also we want to account for values that may already be there, but with a different file name. So we directly compare what's in the checkpoint file vs. what's in the metadata file and update accordingly. Additionally, if NA is in the project metadata, then we don't set the path and fill with NA. Although this shouldn't really affect any of the files that will get updated here. I removed all old SingleR files, so in reality only the CellAssign results are getting updated, and they only exist if there was a reference file in the first place.
I've tested this with runs in scpca/processed. Once this gets approved I'll run for scpca-prod.

The structure here seems good, but I think we want to be a bit more strict about what files names we change, so that we don't accidentally run this in the future and update things we don't want to update. Specifically, we should check that the file name is of the "old" format before updating, so we don't accidentally update version numbers in the future when we don't really want to.

This is a great point! I went ahead and added a check for a version string. If that's present in either of the filenames that are in scpca-meta.json, then no updates are made. I also updated the description of the script at the top of the file to mention that.


V.0810C

👷 Deploy Preview for analyticalnutrition processing.
🔨 Explore the source changes: 3b638003ce1dfd835e97ab7239f1ea8e5f7d43d1
🔍 Inspect the deploy log: https://app.netlify.com/sites/analyticalnutrition/deploys/611337632d2b8d0007ada953


Morning Group Review #1

We as a group reviewed the project and stated:
*Project is following the original template's design guidelines
*Suggested reviewing the "featured speakers" section for the desktop version in order to display properly.

Thanks 👍


dashboard wifi pane is crowded when using travelmate

travelmate creates a lot of uci stanzas in wireless (one per network it knows how to roam between) but it keeps most of them disabled most of the time.  It'd be nice if the dashboard view did not show (or could be configured not to show) these disabled networks.
I see, skimming the source, that there are checks for .visible of both radios and devices, but I don't know what would make that attribute falsey.

Can you please attach a screenshot illustrating the problem?

Here's an example with two AP radios and one client radio with three possible settings.  The two client configurations that are not active are also disabled in uci and it would be nice if they were not shown.


headers-whitelist does not pass headers

Hello,
I wanted to use parameter headers-whitelist but (as I  said in #364), Cerebro doesn't pass my headers.
Cerebro version: 0.8.5

conf/application.conf:
...
hosts = [
  {
    host = "http://localhost:9200"
    name = "http://localhost:9200"
    headers-whitelist = ['authorization', 'x-forwarded-for', 'x-forwarded-host', 'x-forwarded-server', 'x-forwarded-user', 'x-proxy-user']
  }
]

Checking by ngrep:
ngrep -q -W byline '' 'tcp port 9200 or tcp port 9000' -d lo
Output:
T ::1:34654 -> ::1:9000 [AP] #368
POST /connect HTTP/1.1.
Host: localhost:9000.
Authorization: Basic dGVzdGVyOnRlc3Rlcg==.
Accept: application/json, text/plain, */*.
Sec-Fetch-Dest: empty.
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.116 Safari/537.36.
DNT: 1.
Content-Type: application/json;charset=UTF-8.
Origin: https://test-cerebro.local:4433.
Sec-Fetch-Site: same-origin.
Sec-Fetch-Mode: cors.
Referer: https://test-cerebro.local:4433/.
Accept-Encoding: gzip, deflate, br.
Accept-Language: en-US,en;q=0.9,sk-SK;q=0.8,sk;q=0.7,cs;q=0.6.
X-Proxy-User: tester.
X-Forwarded-For: ::1.

T ::1:34654 -> ::1:9000 [AP] #370
X-Forwarded-Host: test-cerebro.local:4433.
X-Forwarded-Server: test-cerebro.local.
Connection: Keep-Alive.
Content-Length: 32.
.
{"host":"http://localhost:9200"}

T <IP_ADDRESS>:57252 -> <IP_ADDRESS>:9200 [AP] #372
GET /_cluster/health HTTP/1.1.
host: localhost:9200.
accept: */*.
user-agent: AHC/2.1.
.

Thanks for help

@kiblik Can you tell me a little about how does your setup looks like?

@lmenezes
HTTP-auth-proxy:443 (kerberos authentication - pass 'authorization' and add 'x-forwarded-...') -> cerebro:9000 -> ES:9200
Is it understandable?

ping @lmenezes

I have the same problem. Have you solved it

I think it's the host name configured in application.conf is not matching.
Try https://github.com/lmenezes/cerebro/pull/516


Feature/improve rag

This pull request resolves "Refactor RAG to include not only the search engine snippets in fact-checking, but the articles themselves too." in #408

/payout -x 3


UI improvements to new SpeedSlider control

this is a start.
I am getting this error:
• No instance for (RealFloat a)
        arising from a use of ‘showFFloatAlt’

That error happens because state w refers the state of the underlying simulation.  The TimeLabel control has type Control Double (see line 2242), so state w is guaranteed to have type Double.  But SpeedSlider has the type Control a, so you don't know what type state w will have.
This is okay, because you don't want to show the underlying state.  You want to show the playback speed instead!  Try changing state w to playbackSpeed w.

Oh, I see. That's what it was referring to when it gave the error of RealFoat (ie Double). Thanks! Makes sense.

Yes! There are numbers! :D :1st_place_medal:

what's the next best step? Should I start the snap function?

I think the next step is to make dragging work as expected.

thanks. I'll work on this tomorrow.


types: export more types in @pandacss/dev

Potential duplicate of https://github.com/chakra-ui/panda/pull/1318
Will close whenever #1318 also exports PatternConfig
📝 Description
Add missing types (PatternConfig, RecipeConfig, RecipeVariantRecord) to solve a TypeScript issue (The inferred type of
xxx cannot be named without a reference...)
💣 Is this a breaking change (Yes/No):
no

closed in favor of https://github.com/chakra-ui/panda/pull/1318


Popup box of detailed info under next widget box

Doing a mouse-over of the most recent data point (right end of the graph), the popup will be below the next box of the next widget.
See the attached Screenshot.

Should be fixed with https://github.com/novus/nvd3/issues/1187


Beefy full client

Quick test of a beefy client that does full signature verification instead of 2-step interactive random protocol, just for interest sake - likely won't merge
Results:

Running flow with 200 validators: Tx successful - gas used: 2343853
Running flow with 400 validators: Tx successful - gas used: 4235531
Running flow with 667 validators: fail, out of gas
Running flow with 1000 validators: fail, out of gas

Very obsolete, closing this off.


Sourcemap support with Raygun

I'm currently using Raygun with our apps Flutter web build and it works very well, thank you! 👍🏻
BUT Flutter web release builds produce minified JS which currently gives not very useful stack traces in Raygun for the Flutter web app.
Raygun's documentation outlines support for sourcemaps so I was hoping to use it with Flutter web as currently Flutter is perfectly able to produce a sourcemap. Because we don't want to make our sourcemap publicly available I attempted to upload the sourcemap to Raygun per the documentation but it seems to have no affect on the stack traces shown in Raygun.
According to the Raygun sourcemap docs, they want you to upload the un-minified JS file as well as the sourcemap file, but this doesn't exist for Flutter as the sourcemap maps to Dart sources not JS.
Would you have any advice on how this can work with Raygun and raygun4flutter?

Hi @maks
At the moment, there's no support for sourcemaps on the Raygun site for traffic coming from the raygun4flutter.
We are going to research for a possible solution to this, which will probably need some changes both in the Flutter package and the server side.
Meanwhile, it would help us a lot if you could provide the exact command you use to compile your Flutter Web application, compilation logs and your flutter doctor -v output, so that we don't miss any important detail.
Thank you!

Thanks for the quick reply @miquelbeltran !
I'm just using fairly standard build command for our web app:
flutter build web --dart-define=SOME_MODE=true --source-maps -t lib/main_webapp.dart

(some things redacted out).
and doctor output:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.0.5, on Ubuntu 22.04 LTS 5.15.0-41-generic, locale en_AU.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.69.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

Please let me know if I can help with any other information or testing.

Also looking for this. May need to switch to sentry

Hi @JaredEzz I'll chat with the team to see what we can do here in terms of priority. Thank you for your patience!

The latest release 3.1.0 adds support for reporting column numbers, as well the website has been updated to support source map processing.
You can find information regarding generating and uploading source maps in the README.md in the section "Flutter web source maps": https://github.com/MindscapeHQ/raygun4flutter/tree/develop?tab=readme-ov-file#flutter-web-source-maps
Thanks for your patience, let us know if you have any more questions!

The documentation on this is not entirely complete. For example, the "Finding a source maps URI section" that is reference on the docs here https://raygun.com/documentation/language-guides/javascript/crash-reporting/source-maps/ is not filled out or linkable.
I'd really like to get this integrated into our CI/CD pipeline. Sentry has a cli tool that really makes this easy.

Hi @JaredEzz, my apologies that the link is missing. I'm double-checking with my colleague to see where the section is. I'll keep you posted

Hi all @JaredEzz @TheRealAgentK
We've just put up a fix to our documentation to include the missing section. You can find it here - https://raygun.com/documentation/language-guides/javascript/crash-reporting/source-maps/#understanding-and-finding-the-url-for-sourcemap-uploads
@JaredEzz Please do consider @TheRealAgentK response too :)
Thank you for your patience,
Sumitra

Unfortunately, I will not be moving forward with Raygun for this. I'm running into too many issues, and I don't believe Raygun's js map solution is mature enough for Flutter Web dart to js minified files.

@TheRealAgentK yes, I understand I can use curl to upload the files; however, the uri parameter in the API request is unclear. I attempted to lookup cr.source-maps:write on the api docs here, but there's no additional information.
I attempted to do the upload manually, but still, the instructions for the URL input are completely unclear.

After uploading a source map it is important to add the correct URL in the URL field for the source map, if this is not set correctly then Raygun will fail to use your source map when attempting to desymbolicate a stack trace

Why should I need to provide a url to the file when I've directly uploaded it? And even if I were to get this to work, does the URL parameter in the UI correspond to the uri parameter in the curl request? Unclear.
With my various attempts, I continue to get the following error on my issues. Unable to determine a suitable SourceMap file for the following minified files. Please ensure your minified files include a "//# sourceMappingURL=..." comment on a separate line at the end of the file. Alternatively, the SourceMap file can be specified in a "SourceMap" HTTP header alongside the minified file. Even though my file clearly includes this at the end

What are all the files that need to be included? Sentry's implementation with their CLI tool uploads all of the following, and takes care of flattening out indexed source maps, locating local file references, and validating.
(see their docs)

It would be possible to upload all of these files via the API in our CI/CD scripts, but without the relevant information or knowing if Raygun can adequately ingest them, it's not currently worth the work. I also fed the source map files into Raygun's source map validator, only to be told they are invalid (though Sentry ingests them just fine, and they work great there).

I understand that my lack of knowledge of source maps contributes to the difficulty I'm experiencing but seeing as I had no trouble setting it up with Sentry's tooling and documentation, I was disappointed at the difficulty and lack of clarity around Raygun's solution. As my team is an existing Raygun customer, and we have alerting integrations all set up, I was hoping to be able to move forward here.

What would your preferred solution for dealing with source maps look like?

I failed to answer this in my previous response. A preferred solution would be a flutter package similar to https://pub.dev/packages/sentry_dart_plugin
I include the following steps in my Github Actions build pipeline. Note that directly after the flutter build web command, I run the sentry_dart_plugin and it handles uploading the source maps via the cli (see also https://github.com/getsentry/sentry-dart-plugin)

jobs:
  ...
  flutter-deploy:
    ...
    name: Deploy Flutter Web App
    needs:
      - flutter-test
      - java-test
    runs-on: ubuntu-latest
      - name: Build Flutter Web application
        run: flutter build web --release --source-maps --csp --dart-define=ENV=dev --dart-define=RAYGUN_API_KEY=${{ secrets.RAYGUN_API_KEY }} --dart-define=FLUTTER_SENTRY_DSN=${{ vars.FLUTTER_SENTRY_DSN }}
      - name: Run Sentry Dart Plugin
        run: flutter packages pub run sentry_dart_plugin --sentry-define=auth_token=${{ secrets.SENTRY_AUTH_TOKEN }} --log-level=debug
        ...

This is great to hear. I've moved onto another project at the moment but I will definitely look into this. One question I do have about the uri parameter that the CLI requires also - does that have to be a publicly hosted version of the source map? I'm not familiar with where that would be located

does that have to be a publicly hosted version of the source map?

The URI corresponds to where the Flutter web app is installed, e.g. https://localhost:44689/main.dart.js as it appears in the registered stack trace.
e.g.
at Object.Cf line 6971, column 18 (http://localhost:44689/main.dart.js:6971)
at a_s.$0 line 57311, column 47 (http://localhost:44689/main.dart.js:57311)

I want to make it this as flexible as possible, e.g. also giving the possibility to pass in the base URI instead (so just https://localhost:44689/) and figure out the rest, but at the moment Raygun requires you to provide the full file URI.

FYI, the tool has been published on https://pub.dev/packages/raygun_cli, and we have moved the project to https://github.com/MindscapeHQ/raygun-toolbox


Allow Jetty to be pluggable

It would be great to break apart spark-core from Jetty (spark-jetty?).
For example, in the latest TechEmpower web benchmarks. raw HTTP performance from Undertow was nearly 3 times higher than Jetty:
http://www.techempower.com/benchmarks/#section=data-r9&hw=peak&test=json
it would be great to try to plug in a different web server into Spark and see the performance difference.
Also separating core from Jetty would make it easier to embed in a more robust container like Dropwizard (which gives you YAML-based configuration, etc).

+1 on this. We really need this functionality as well.

+1

at one time when i thought i was going to use spark for a project i did this work (and also refactored a ton of the spark code base at the same time). i ended up using jersey and haven't had the time to do anything this else w/ it.
if someone wants to pick up and run w/ it, you can find id here: https://github.com/jgangemi/spark/tree/server
the one thing i never got working was letting spark run in tomcat - that code still needs to refactored to break dependencies there.

+1 plenty of opportunities if spark becomes server independent

I forked this project some time ago to do that (among other things) and I have a working Undertow backend. You can check it here: https://github.com/jamming/sabina

+3
YES!!
https://github.com/perwendel/spark/issues/288

+1, It will be awsome

This is in the backlog.

+1

+1 ... another very useful thing out of this will be to be able to configure a little bit the jetty webserver.
One exceptional annoying problem (that happens when you run jetty on Windows) is that by default Jetty locks your static files and thus you can't work with them if server is running.
The solution is to change some jetty defaults in the jetty ResourceHandler.
Currently, because Spark is so deeply coupled with Jetty you just can't do that.
I did picked Spark for simplicity. But if it doesn't offer also some flexibility for those who want to pay the price :) ... then you just can't get around problems like the above.

Almost 3 years later - any news on this one? I would like to run the application on the Undertow server.

I submitted PR #674 for separating Jetty from the "core" Spark quite a while ago. I'll take a look at how sour the changes have gone, hopefully can resolve conflicts without a huge effort.

Should old issues be closed if actions are not taken?

That's a good question.. If we close them, they're going to show up again at some point, but without all the existing comments. I've been keeping most issues/PRs that may happen open for that reason.

I forked the project some time ago with this feature as one of its goals... You can check it here, now it is unmaintained as I'm working in a pure Kotlin framework which includes this feature... You can check it here... Sorry for the advertisement 😁


InputField, Input, Label, Icon, Tooltip as compound components

I have been talking to @odmln and it seems that we had a confusing about building the Input component. Input should include, icon right, icon left, label, error states part of its behavior. Ideally it should be build as a compound component in my opinion. Meaning you can import Input and pass Icon (left or right) Tooltip, Label as children.
Here is a demo of the Input Behavior (with outdated tooltip):

Input states:

Zeplin link:
https://app.zeplin.io/project/596d11bc29e9db6303116850/screen/59ce7dfd4fb3d1a16d85589d
The tooltip should be able to show up up and down for the error states. #140
before:
<InputField label='' icon='' />

after:
<InputField>
  <Icon />
  <Label />
  <Input />
  <Icon />
</InputField>

This is a great idea! React composition is super powerful, and this particular example would mean less custom API surface with the design system – if you know how React works, then you know how to move the icon from the left to the right side. I'd suggest we consider this with all of our composite components – Banner is another similar example: https://github.com/pricelinelabs/design-system/pull/111#discussion_r146889428

Going to try to knock this out now, as using it for the type-ahead will be much cleaner that what I'm doing now.

Closing this since it should be addressed in #162.


pybullet.error: createCollisionShape failed.

It is on Ubuntu 16.04LTS, and it shows this error:

It doesn't look like an error on TensorFlow's end. Can you please share your bullet-test.py script?

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

ok， I have solved it. thank you!

@JobSmith how did you solve it ?

For anyone still looking GEOM_BOX requires the halfExtents argument, even though in the quickstart guide it claims that there is a default. So p.createCollisionShape(p.GEOM_BOX) won't work, but p.createCollisionShape(p.GEOM_BOX, halfExtents=[1, 1, 1]) will

@JobSmith 你怎么解决的，我也遇到这个问题了


Update module github.com/stackitcloud/stackit-sdk-go/core to v0.1.1

This PR contains the following updates:

Package
Type
Update
Change

github.com/stackitcloud/stackit-sdk-go/core
require
patch
v0.1.0 -> v0.1.1

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (v0.1.1). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


No module named 2captcha

No plan why that happens. Cant really say anything than that it works when I start it in my IDE, so maybe its a problem with PyInstaller again, maybe you have a good alternative, saw that you're normally not using it. I'm getting that error before or while making a post request.
https://prnt.sc/qxhmwl

yes, the problem is pyinstaller, as i posted in your other ticket you didnt respond too..
https://github.com/VeNoMouS/cloudscraper/issues/72#issuecomment-524593291

yes, the problem is pyinstaller, as i posted in your other ticket you didnt respond too..
#72 (comment)

Ok thanks that helped, didnt see that you replied in time sry and thanks you very much for help and for you great module! :)


Fix/users image handling

Description:
Merge this into main to fix the image naming convention
Issue Reference URL
https://github.com/awslabs/aws-eks-data-maker/issues/

Do not change content below. Mark applicable check boxes only.
Thank you for submitting a contribution to AWS Orbit Workbench.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a ISSUE associated with this PR?
[ ] Does your PR title start with ISSUE-XXXX where XXXX is the issue number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[ ] Has your PR been rebased against the latest commit within the target branch (typically 'main')?
[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under Apache License 2.0?

This mangled PR was replaced by PR 451


Composer instructions obsolete

The README suggests that you can install this package via:
composer require league/tiny
There is no longer such a package on Packagist, however. I managed to install it using:
composer require zackkitzmiller/tiny
but believe this is also somewhat dated. It does work though. I think updating it to the current one would really help others get hold of and use this great and petite package.

I came into this issue when trying to install Tiny.
Also the zakkitzmiller version has a different namespace

I've actually reverted the commit that was preparing this library for the PHP League. It had some bugs in it. The Readme should be good to know now.


Support DateFormatConverter using other formats

DateFormatConverter always generates with
    public DateFormatConverter()
    {
        DateTimeFormat = "yyyy-MM-dd";
    }

The yyyy-MM-dd should be customizable for interacting with APIs that use other formats.

I've just run into the issue of a platform providing null dates as "0000-00-00" which will not deserialise cleanly with this converter.
I see two fairly clean solutions that would solve this one

provide configuration to disable the generation of this class & it's attributes entirely (allowing use of SwaggerToCSharpClientGeneratorSettings.CSharpGeneratorSettings.JsonConverters to be used in it's place)
provide configuration for providing a custom type name to use for this converter type in object attributes (and disable generation of the class)

You can override/implement your own converter by override the DateFormatConverter.liquid template

Yeah I realise that, and i've done that for now although it feels a  bit messy since I need some global type converters anyway and it would have been nice to include them all the same way.

You could leave the template file empty and define a converter somewhere else with the name and add the using with AdditionalNamespaceUsages

But yes, changing the tpl feels messy but so does adding a new setting for every scenario... each setting adds new complexity for everyone and increases maintenance cost. Also the generated clients should be self contained, ie should just work correctly for the spec and should not require any external changes/implementations.
Maybe we should just change the date converter to convert 0000-00-00 to null if the target type is DateTime? ? Dont see why everyone would want the actual year 0 in a nullable datetime?


Indicator of current desktop

Description of the new feature / enhancement
As a virtual desktop user I'd like to have a visual indication of the current desktop so I do not have to think about it and can navigate without the overhead of orienting myself.
Scenario when this would be used?
I've started using FancyZones and discovered Virtual Desktops. The good part is: FancyZone configuration is stored per desktop, which is great. So I am using 2 or 3 desktops with different FancyZones Layouts but I have no clue if I am using the right one and I already made the mistake multiple times to change the FancyZone layout instead of just navigating to my other desktop - the one that already has the desired zone layout.
Supporting information
No response

This could be either addition to FancyZones, or separate mini-feature

/dup #12246

Dear bot,
I’d like to inform you I feel very much offended by you bluntly lying my request has been completed. It is not because it is a duplicate so I am still waiting for the original issue to be solved / completed.
Please consider marking this issue in accordance ☺
Kind regards,
Gerco
Gerco de Jager
| Specialist | Teamleider | Research & Development|Tuesday - Friday
T +31 317 49 33 51 | M +31 629 72 65 49 | @.*** | www.marin.nl
From: msftbot[bot] @.>
Sent: woensdag 28 december 2022 23:04
To: microsoft/PowerToys @.>
Cc: Jager, Gerco de @.>; Author @.>
Subject: Re: [microsoft/PowerToys] Indicator of current desktop (Issue #22776)
You don't often get email from<EMAIL_ADDRESS>Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Closed #22776https://github.com/microsoft/PowerToys/issues/22776 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/22776#event-8123356594, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGMAMNLJB4TYQHCYFCTDEL3WPS2OJANCNFSM6AAAAAAS6E76A4.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Add security tokens for API consumers

This is just to make sure that our hosting services can expand as per the need of the users, and to make sure we have a way to communicate with API consumers


Basic redux integration for thread component

Here is a very basic integration with redux. I've moved all the complex logic from the Thread component to the reducer, which is also responsible for restoring the initial state from the local storage. Reducer is already covered with a couple of specs, would be also useful to have a spec for getters. Related to #95 and #96

@Guria Thanks for the review, I think I'm done. Please let me know if we need to change something else

@Guria I'm stuck with the devtools :( I've configured everything according to the docs, aliased all the react-related things, but redux dev tool does not see the store, while react dev tool is "waiting for react" or "waiting for the roots to load", what am I missing here?

Ok, nevermind. We can do it in a separate PR. But we need to resolve new conflicts here.

Done

@Guria Are we good to go? :) I have one more PR to submit

@DmitryTsepelev please don't use yarn. only npm 6.1.0+

ah ok, got it


Adds support for Lumen

The included changes will allow this package to be used with Lumen. Additionally, README.md has been updated to include instructions for setup within Lumen.

I think this repo is dead and should be transferred to organization or person who will maintain it. @maknz

Package is ded


Feature/413 inject adapters

implemented a simple injection mechanism via ODataClientSettings.
an instance of the IAdapterFactory can be assigned and then will be used to create IODataAdapters for the supported protocol versions.
Sample code using this mechanism to inject a custom Adapter that uses a custom serializer to parse the payload for special responses. (Use a custom JSON parser for OData v1-v3 messages)
Assign the Factory
var settings = new ODataClientSettings("http://services.odata.org/V3/Northwind/Northwind.svc/");
settings.AdapterFactory = new JsonODataAdapterFactory();

How that factory looks like
public class JsonODataAdapterFactory : IAdapterFactory
    {
        private static readonly string[] _supportedVersions = { "1.0", "2.0", "3.0" };

        public bool CanLoadForVersion(string protocolVersion)
        {
            return _supportedVersions.Contains(protocolVersion);
        }

        public IODataAdapter LoadAdapter(string protocolVersion, ISession session, string metadataDocument)
        {
            return new JsonODataAdapter(session, protocolVersion, metadataDocument);
        }
    }

The simple Adapter inherits from the library's ODataAdapter (V3)
public class JsonODataAdapter : ODataAdapter
    {
        private ISession _session;

        public JsonODataAdapter(ISession session, string version, string metadataDoc)
            : base(session, version, metadataDoc)
        {
            _session = session;
        }

        public override IResponseReader GetResponseReader()
        {
            return new JsonResponseReader(_session, Model);
        }
    }

And the reader that want's to parse JSON payload by itself. (note, atom payload will be forwarded to the base class)
public class JsonResponseReader : ResponseReader
    {
        public JsonResponseReader(ISession session, IEdmModel model) : base(session, model)
        {
        }

        public override Task<ODataResponse> GetResponseAsync(HttpResponseMessage responseMessage)
        {
            var contentHeader = System.Net.Http.Headers.HttpContentHeadersresponseMessage.Content.Headers;

            // only act for JSON payload
            if (contentHeader.ContentType.MediaType == "application/json")
            {
                return Task.Run(async () =>
                {
                    // create an ODataResponse with your custom JSON parser here
                    // call your serializer (e.g. newtonsoft) and create an ODataResponse from the result
                });
            }
            else
            {
                // no JSON payload => call base method
                return base.GetResponseAsync(responseMessage);
            }
        }
    }

Thank you so much for your work and PR. I am currently struggling with other projects, have almost no time for anything else. Will try to check this out shortly.

Hi Niko,
Sorry for the inconvenience, but after applying the PR #422  from @sixlettervariables (that goes to current version 4) there's a merge conflict in AdapterFactory. The other PR optimized use of cached metadata, and it also touched some of the code you changed. Can you please review the conflict and reissue the PR?
Thanks!

Hey Vagif,
Sure, it will resolve the issues as soon as I can.
Best!
Niko

Am 28.07.2017 um 08:48 schrieb Vagif Abilov<EMAIL_ADDRESS>Hi Niko,
Sorry for the inconvenience, but after applying the PR #422 from @sixlettervariables (that goes to current version 4) there's a merge conflict in AdapterFactory. The other PR optimized use of cached metadata, and it also touched some of the code you changed. Can you please review the conflict and reissue the PR?
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@nrusitska as I'm the one who introduced the conflicts for you, if you need any help let me know!

Hey folks,
I briefly checked the conflicts and I need to refactor some things to match the changes from #422.
I'm on vacation for a week and will start to fix this when I'm back.
Best regards!
Niko

Hey folks,
sorry for the long delay - I was captured in some other projects longer than I planned to be.
I've updated my PR to match Christopher's changes.


feat(utils/evaluation.py): support evaluate

Motivation
Support evaluate model on validation datasets.
Modification
Some important updates:

data/dataset.py: add func get_dataset_dict for generating a dictionary of datasets from a folder containing data files for validation.
utils/evaluation.py: add func evaluate_on_val_dls for evaluating model on validation datasets.

Use cases (Optional)

User can set validation dataset folder by parameter valid_folder in configs/7B_sft.py. Dummy validation dataset will be generated if valid_folder is None.

VALID_FOLDER = "/path/to/dataset"
data = dict(
    seq_len=SEQ_LEN,
    # micro_num means the number of micro_batch contained in one gradient update
    micro_num=4,
    # packed_length = micro_bsz * SEQ_LEN
    micro_bsz=2,
    valid_bsz=4,
    valid_every=50,
    ...
    # train_folder=TRAIN_FOLDER,
    valid_folder=VALID_FOLDER,
)

Checklist
Before PR:

[x] Pre-commit or other linting tools are used to fix the potential lint issues.

该分支依赖feat/support-uniscale-logger，请先合并pr 152，再合并本pr


support send response data to dingtalk, feishu, business wechat webho…

…ok and also support send custom text
Describe what problem this pull request solves and how to solve it. If there is an issue associated with it, please attach a link.
添加了发送request body、request header、response到飞书、钉钉、企业微信机器人的功能，也可以发送自己输入的信息，提高前后端联调效率。
Pull Request Checklist

[ ] I have read the UME contribution document and understand how to contribute, commit the code according to the rules. 我已阅读过 UME 贡献文档，并了解如何进行贡献，按照规则提交了代码
[ ] I have added the necessary comments in code to ensure that other contributors can understand the reason for the change. 我在修改中已经添加了必要的注释，以确保便于其他贡献者理解修改原因
[ ] The code has been formatted by dartfmt before push. 代码在提交前已经经过 dartfmt 进行了格式化
[ ] Change does not involve the adjustment of test cases. Or all existing and new tests are passing. 改动不涉及测试用例调整，或 example 工程与单元测试已经完全跑通

If you need help, consider Join ing the ByteDance Flutter Exchange Group.
如有任何问题，可以加入字节跳动 Flutter 交流群。
Or contact author.
或随时联系开发者。

I'm not sure why an interceptor should send such content, but a log interceptor might do so. If you want to collect requests and responses, it might be better to expose the method to obtain, rather than nested the upload function.
FYI @talisk.

Please write example code in Example project. And then I will ask @AlexV525 to review the code.

When you register the DioInspector, there are three parameter in it's constructor, only the dio is required. Not sure how to write the example code.


Taking a Screenshot of UWP

Hi,
I'm wondering, how to take a Screenshot, when using Appium with WinAppDriver. I want to take screenshots from specific pages of my UWP application on Desktop and later on my mobile device or emulator and don't know, how to achieve this.

Well I should have read further. It is actually possible by
var img = mySession.GetScreenshot();
img.SaveAsFile("D:\\myfile.jpg", ImageFormat.Png);

(don't use jpg, as the compression is very high and thus the quality quite crappy)


Often cannot attach

We often cannot attach to our process running in production. Specifically, it appears to attach, but I see no IEX prompt. This is what I see:
$ bin/booking attach
Using /var/www/booking/releases/2.6.1-5566-g1ade97f/booking.sh
Attaching to /var/www/booking/tmp/erl_pipes/booking/erlang.pipe.4 (^D to exit)

^L

Pressing ^D does exit, but that's it. Sometimes I can stop the service and restart it and get it to attach, but obviously that's no good :smile:

Is it predictable? Or does it happen randomly? I'm not sure what would cause that to happen..
It might also be useful to know your deployment workflow, maybe I can find a way to reproduce if I know what steps lead to that happening.

We are using edeliver to deploy. We're not doing hot upgrades. After a deploy, we do a stop followed by a start.
I don't know how to reproduce it consistently, unfortunately. It just seems to happen. I go to attach to a process that has probably been running for a bit and it does that. I wonder if the ^L is a clue. There are multiple people accessing it, so it's very possible that one person has a bad habit that is getting it into this state. Of course, it shouldn't be possible to get it into this state... but...
For a bit today I thought it was that we had hit Ctrl-C and then Ctrl-D once attached, but that has a different (also bad) behavior. It leaves the service in a state where it cannot be stopped normally, you have to kill it. It looks like there is an erl option (+B) to disable the break menu, I wonder if we should (and how) enable that.
Ok, so I think I may have found at least one way to repro it. It appears someone left the console running instead of the service itself. So, if you run console and then attempt to attach to it, you get the behavior I described. I'll have to check again to see if that is what is happening next time I see this. I have let the team know that they should not do this :smile:

Ok, so I actually misunderstood how things work. As soon as you start a service with bin/booking start it starts a console, that's why I saw it in my ps. I'm totally stumped.

 This morning, I got the ^L thing. I'm pretty sure no one did anything between those times, so it just appears to be time that causes it.

Hi,
I've found this bug report while looking into a similar issue on Erlang release with relx.
Not sure if it helps your problem, but in my case it was an issue with the TERM environment not being there, when the release is started from an RH init script at boot time. Adding e.g. TERM=xterm to the environment in the init script before starting made attach work for me.
Regards,
Ferenc

Got the same issue with rebar 2.6.2 under Ubuntu 16.04

With https://github.com/bitwalker/exrm/commit/420f1a66ff521ef7bc5ca51ecab2542e758931f9 attach always works as described (hangs with ^L and no prompt)

https://github.com/bitwalker/exrm/commit/420f1a66ff521ef7bc5ca51ecab2542e758931f9 seems to be breaking behavior for whats expected for start (foreground seems to be the case you want to fix that "bug"). Rebar and in turn relx suggest start should start a console that you can pipe into. http://www.rebar3.org/docs/releases#booting-upgrade-and-inspecting

+1
I'm having the same attach issue (^L character, hangs) in exrm 1.0.6. Downgrading to 1.0.2 seems to fix the issue, so something must have changed in the interm. Any ideas?

Much like @fholzhauser, I found this issue while having a similar ^L hanging issue (with an Erlang release via rebar3 as prod tar, when deployed via Amazon's CodeDeploy). TERM didn't seem to be set, so adding the TERM=xterm allowed successful attach.

@fholzhauser @derek121
It works.
THANKS VERY MUCH !!


Bump ember-light-table dependency

closes TryGhost/Ghost#9434
Just bumping the dependency didn't seem to cause any issues 🤔- I tested it manually as well as ran tests locally. @kevinansfield were there specific things you noticed that were broken?

Coverage remained the same at 71.965% when pulling 7cf3fe76000397ae3407b7257071ddb1d095796a on acburdine:update/light-table into 2c4deefc5b470f2f6cafc3b9b5b048696ac914bf on TryGhost:master.

Hmm, I can't remember now, I'm sure when I tried the upgrade previously everything blew up in spectacular fashion so I just left it due to time constraints. Maybe it was just a dodgy module install or a conflict with something else 🤔
Anyway, this looks fine! Thanks for looking into it 😄


(Uncommited) Improved clarity and readability of the repository README.md file

As team
I want well maintained and clarity  project README.MD
so that external contributors gets a quick and comprehensible  overview of the product
Hints / Details

[ ] https://gitlab.com/apparmor/apparmor  is a good practice

Outcome / Acceptance Criteria

Outcome

...

Acceptance Criteria

[ ] Align layout to those of product apparmor
[ ] Describe  Source Layout of respository
[ ] Describe  how to Getting in Touch
[ ] Matrix Chat
[ ] Issues
[ ] Conversations
[ ] etc.
[ ] Describe IRS Testsuites (transparent )

[ ] Add Cucumber and Tavern Test description
[ ] Regression testing
[ ] Functional and module testing
[ ] Unit testing and reporting
[ ] Load Testing Gatling
[ ] Stress Tests and Smoke Tests
[ ] Link to version matrix

Out of Scope

...

Planning 2:
Make sure that the README aligns with TRGs https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-1

Cucumber and tavern README added in PR #781.

[ ]  Align layout to those of product apparmor
[ ] Describe Source Layout of respository
[ ] Describe how to Getting in Touch
[ ] Matrix Chat
[ ] Issues
[ ] Conversations
[ ] etc.
[ ] Describe IRS Testsuites (transparent )

[ ]     Add Cucumber and Tavern Test description
[ ]     Regression testing
[ ]     Functional and module testing
[ ]     Unit testing and reporting
[ ]     Load Testing Gatling
[ ]     Stress Tests and Smoke Tests

Conversations/Discussions  >> Should link to https://github.com/eclipse-tractusx/item-relationship-service/discussions

As consulted with @ds-jhartmann the pull request was opened to eclipse-tractusx:main directly:
https://github.com/eclipse-tractusx/item-relationship-service/pull/456

@jzbmw ready for PO review


Client not working on Wayland (Swaywm)

Hey, thanks for sharing this project, it's exactly what I've wanted for a while.
Unfortunately it (both keyboard and mouse) doesn't currently work for me when using  Wayland (Sway).
I've tested that it works for me using X (i3).
Have you tested rkvm on Wayland at all?
My setup:
Server: X (i3)
Client: Wayland (Sway)
Additionally, on Wayland I've tested that libinput is indeed receiving the events using libinput debug-events.
I'm not really sure where the problem lies, but when I was trying to reimplement barrier in rust (it's just a prototype) a while ago, I remember having lots of trouble with this. It's a little different because it uses absolute events for the mouse due to how barrier works, but seen as the keyboard also doesn't work with rkvm that difference probably doesn't matter too much. I just tested my barrier-rust prototype on a Wayland client and the mouse does work.
I think it might be with how you setup the virtual input devices because I remember it being really finicky, e.g. even though libinput receives the events, some other system (maybe Wayland or the specific compositor or something?) doesn't read the events if the device doesn't conform to some heuristic based on its capabilities, e.g. If you try create a mouse without a left button it will ignore mouse events from that device. Unfortunately I didn't document exactly what I found so I can't be sure.
In this case it might be that you created a single virtual device for both the keyboard and mouse, instead of separate virtual devices for each.
Do you have any thoughts on this?

This worked with Sway when I tested it like 2 months back. However it's no longer working and is just freezing the server side Sway even when no clients are connected.

This worked with Sway when I tested it like 2 months back. However it's no longer working and is just freezing the server side Sway even when no clients are connected.

I've got a branch where I've separated the mouse and keyboard into separate devices and modified the supported event types for each and this makes it work on sway. I'm not sure whether it's sway specifically or wayland or wlroots etc and I'm not sure whether this is a bug or expected behaviour, but I'm probably going to use this branch and clean this up some time so that I at least have a usable kvm system with wayland. The keyboard supported events still needs to be expanded on my branch though to work 100%.

I've got a branch where I've separated the mouse and keyboard into separate devices and modified the supported event types for each and this makes it work on sway. I'm not sure whether it's sway specifically or wayland or wlroots etc and I'm not sure whether this is a bug or expected behaviour, but I'm probably going to use this branch and clean this up some time so that I at least have a usable kvm system with wayland. The keyboard supported events still needs to be expanded on my branch though to work 100%.

After doing some binary searching on enabling ranges of key types for the keyboard, I have found that the issue seems to be trying to enable key types that don't exist with the range [0..KEY_MAX]. E.g.
Given the snippet:
#define KEY_ROTATE_LOCK_TOGGLE	0x231	/* Display rotation lock */

#define KEY_BUTTONCONFIG		0x240	/* AL Button Configuration */

when enabling the range [0..=KEY_ROTATE_LOCK_TOGGLE] then the keyboard works, but the range  [0..=KEY_BUTTONCONFIG] no longer works.
The range [0x232..=0x239] of key events don't seem to exist.
So I would say this is a bug with rkvm. It's more work, but we should probably enable events in a type safe way rather than just using ranges, unless you can do both.

After doing some binary searching on enabling ranges of key types for the keyboard, I have found that the issue seems to be trying to enable key types that don't exist with the range [0..KEY_MAX]. E.g.
Given the snippet:
#define KEY_ROTATE_LOCK_TOGGLE	0x231	/* Display rotation lock */

#define KEY_BUTTONCONFIG		0x240	/* AL Button Configuration */

when enabling the range [0..=KEY_ROTATE_LOCK_TOGGLE] then the keyboard works, but the range  [0..=KEY_BUTTONCONFIG] no longer works.
The range [0x232..=0x239] of key events don't seem to exist.
So I would say this is a bug with rkvm. It's more work, but we should probably enable events in a type safe way rather than just using ranges, unless you can do both.

I can report the same for Gnome 40 on Wayland (Archlinux) as Client (Archlinux with XFCE as Server) - Using the version from this repository does not work, the switch seemingly happens, but mouse and keyboard don't work on the Client Side. Using the Branch from @BrendanBall with separated Mouse and Keyboard Devices does work.

@BrendanBall did you ever managed to take a look at the key ranges ? This is the only kvm solution working on my wayland client ( thanks to your branch ) and now im just missing some of my keys to be completely happy for now :D

My setup has changed so I no longer need rkvm/barrier. I haven't looked at this in a long time, but from rereading this, I believe my branch was just to test whether using proper key ranges fixes the issue with wayland. I don't think I ever finished the work.
So you'll have to build on my branch to include all the valid key  ranges for a keyboard.

@BrendanBall alright, I might fork the project and take a look at it sometime soon, this is still the only kvm working on wayland so far which is kinda absurd.
Appreciate your efforts !

I am using this with no problems between two Debian "bookworm" 12 systems running the distro packaging for sway (version 1.7) and having no problems.

Could there be a fix put upstream on rkvm @jimdigriz ?
Like, it feels like there is only more fragmentation in this open source effort, everyone is forking/branching but no real effort is made to put the fixes upstream ?

@AkechiShiro I do not understand what you mean by "fix put upstream", no fixes are needed to the project to make it work.
As for "everyone", it looks like two (2) people have forked the project and actively done anything with it, one of those forks is a pending PR so again confused what you are referring to here?

@AkechiShiro I see there are some earlier forks with useful work, but you have to remember, htrefil/rkvm is not our project its @htrefil so they get to decide what helps them. We are all just secondary concerns...people who nag and whine and are best ignored for ones own sanity :)
Forks are good thing. Everyone wants someone from a given project and that venn diagram rarely overlaps 100%, so they should be encouraged!
i would suggest building your own fork and merge in all the other work you are interested in.

Sorry @jimdigriz if I came of as rude here but I thought there was no pending PR about the fixes mentioned here.
You also mentioned an issue with compiling, if I can get to fix this and there is no PR, I'll put one.
I'm not against forking but if this project is still actively maintained then, there people forking and fixing bugs should PR, they're not forced to PR but I believe they should do that effort.
Now if the maintainer does or doesn't pick up the PR that is another matter.


Update deprecated github actions to latest version

What does this PR do?
This PR updates all deprecated github actions using NodeJS 12 in order to use NodeJS 16. The list of actions updated are:

actions/checkout@v2 > actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9 # v3.5.3
actions/setup-go@v2 > actions/setup-go@93397bea11091df50f3d7e59dc26a7711a8bcfbe # v4.1.0
<EMAIL_ADDRESS>> codecov/codecov-action@eaaf4bedf32dbdc6b720b63067d99c4d77d6047d # v3.1.4

Which issue(s) does this PR fix
Fixes https://github.com/devfile/api/issues/1224
PR acceptance criteria
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened.

Open new test/doc issues under the devfile/api repo
Check each criteria if:
There is a separate tracking issue. Add the issue link under the criteria
or
test/doc updates are made as part of this PR
If unchecked, explain why it's not needed

[ ] Unit/Functional tests

[x] QE Integration test

[x] Documentation

[x] Client Impact

How to test changes / Special notes to the reviewer
I've tried to test the new versions of actions in my fork. Some of them might fail as we are inside a forked repo but the github actions run without an issue:

CI
Code Coverage Report **this workflow fails also in devfile/api repo.
Publish devfile schema
Release typescript models

I've included a fix to the jsonschema-cli dependency. I've created the validate_schemas.py file which uses jsonschema instead of jsonschema-cli. After the update of cython to 3.0.0 we couldn't install the pyyaml < 6.0.1 as requested from this dependency and as a result we had to find another way to validate our samples. As the jsonschema-cli is just a cli for the jsonschema, I think the safest way to remove this dependency was to use directly jsonschema-cli.

I've included a fix to the jsonschema-cli dependency. I've created the validate_schemas.py file which uses jsonschema instead of jsonschema-cli. After the update of cython to 3.0.0 we couldn't install the pyyaml < 6.0.1 as requested from this dependency and as a result we had to find another way to validate our samples. As the jsonschema-cli is just a cli for the jsonschema, I think the safest way to remove this dependency was to use directly jsonschema.
More detailed the error trace from the previous state of this check can be found here https://github.com/devfile/api/actions/runs/5962813005/job/16271495000

Looks like the "latest" release of jsonschema-cli was from a few years ago, so it's a good approach to not use it anymore since it's no longer maintained.


reenactment speed

Hi,
the provided demo reenactment on subject 1 and 2 is estimated to take very long on my 4090 machine ~10hours. Is this normal ?
Best regards

Hi,
I think that is not normal. Which script did you run, and with what arguments?

It seems this was caused by a slight mistake in the reenactment command, which has been brought up to me by someone recently.
The readme has been fixed.


Dispatch error code

Dispatch the catched error at the same time that the WebcamJS error message.
Allow a better errors handling.

Thanks man!


Make mouse= the default

Mouse support is very confusing. Users expect right-clicking in a terminal to bring up a context menu, with absolutely zero exceptions. I always turn mouse off. I use vim because it does not support the mouse. I'm sure many people use it for that reason.
A command-line program hijacking the mouse from the terminal is almost as bad as a GUI program not supporting the mouse. The features in the terminal emulator context menu are crucial, while mouse support is a command-line editor are not.
People who want mouse support in vim use gvim, right?

Why open a new issue? You've just said the same thing in another issue where this has been thoroughly discussed.
You also make a lot of claims about what "everyone" does and wants. Vim is used in an enormous variety of situations, not just your use-case.
There are also many terminal applications which can and do use the mouse - Vim is not magically adding mouse support to the terminal, it is using standard terminal functionality.

On 2019-12-11, scoopgracie wrote:

Mouse support is very confusing. Users expect right-clicking in a terminal to
bring up a context menu, with absolutely zero exceptions.

Right-clicking in an xterm does not bring up a context menu, nor
does it in mintty.

                                                      I always turn mouse

off. I use vim because it does not support the mouse. I'm sure many people use
it for that reason.
A command-line program hijacking the mouse from the terminal is almost as bad
as a GUI program not supporting the mouse. The features in the terminal
emulator context menu are crucial, while mouse support is a command-line editor
are not.
People who want mouse support in vim use gvim, right?

Wrong.  I hardly ever use gvim because:  1) I'm always working at
the command line; and 2) :sh in gvim does not give me a
full-featured terminal.  I use the mouse in vim in an xterm for
dragging window borders and for scrolling, and occasionally for
copying and pasting to and from external windows, but I usually use
y and p for that.
Regards,
Gary

this has been discussed to death, as you know, since you left your opinion at the other issue. If you do not want mouse support, you can configure vim to not use the mouse. Changing the default is going to upset the other half of the users that are happy with the current settings so it is not going to happen.

On 2019-12-11, scoopgracie wrote: Mouse support is very confusing. Users expect right-clicking in a terminal to bring up a context menu, with absolutely zero exceptions.

Right-clicking in an xterm does not bring up a context menu, nor does it in mintty.

Indeed. Right-clicking in xterm selects a range of lines. AFAICT the only context menu in xterm is the window manager's, obtained by right-clicking the titlebar.
But right-clicking in Vim or gvim gives me a context menu in all cases. In gvim it looks like a GTK2 popup, in vim it is a text popup, but the contents are the same in both cases, and relevant to Vim; and they aren't the same in Normal, Visual, and Insert modes. How do I get that? Simple (it's documented): set mousemodel=popup

I always turn mouse off. I use vim because it does not support the mouse. I'm sure many people use it for that reason. A command-line program hijacking the mouse from the terminal is almost as bad as a GUI program not supporting the mouse. The features in the terminal emulator context menu are crucial, while mouse support is a command-line editor are not. People who want mouse support in vim use gvim, right?

Wrong. I hardly ever use gvim because: 1) I'm always working at the command line; and 2) :sh in gvim does not give me a full-featured terminal. I use the mouse in vim in an xterm for dragging window borders and for scrolling, and occasionally for copying and pasting to and from external windows, but I usually use y and p for that.
Regards, Gary

I use both gvim and vim. When I'm in a shell command-line context, I may temporarily start Vim in that same terminal, more or less like Gary does. In addition I have a gvim runing more or less permanently, which I use when an edit would take me a long time, or to browse the Vim help, or for little tasks in a GUI context, for instance to get hard-to-type (by non-Vim methods) UTF-8 strings into the clipboard for pasting into a browser textarea (thanks to either a keymap or to the |i_CTRL-V_digit| function of |c_CTRL-V|).
So I use both vim and gvim quite a lot, and I find it best when they look and behave the same. I have found out how to get indistinguishably (to my eye) identical colors in both a 1677216-color gvim and a 216-color vim-in-terminal so they look the same to me. Alas, they don't act 100% the same because the terminal hijacks some keypresses (e.g. Shift-F11) which I can reliably map in gvim. About the mouse, if it didn't work in Console Vim it might drive me crazy (assuming I'm not already). At least left-cllicking moves the Vim cursor and right-clicking gives a Vim context menu in almost identical fashion in both gvim and vim.


Adding internationalization options

I've added some basic configs for weekdays, dateFormat, start of the week etc. for i18n.

@Norreman thanks alot for your PR. can you also please add this to README?

@jkuri What should i add to README? I've added the new configs allready :-)

sorry I missed it :) thanks!


Please update the ffmpeg and gnutls versions used by libopencv

Feature Request
For the latest pip-installed version of opencv-python (<IP_ADDRESS>), libopencv is using very dated versions of ffmpeg and gnutls, which are known to have vulnerabilities. Would it be possible to release a version with those dependencies updated?
Please let me know if I'm posting this request to the correct repo.
Issue submission checklist

[✅  ] This is not a generic OpenCV usage question (looking for help for coding, other usage questions, homework etc.)
<!--
<p>-->

[✅  ] I have read the README of this repository and understand that this repository provides only an automated build toolchain for OpenCV Python packages (there is no actual OpenCV code here)
<!--
<p>-->

[ ✅ ] The issue is related to the build scripts in this repository, to the pre-built binaries or is a feature request (such as "please enable this additional dependency")

[✅  ] I'm using the latest version of opencv-python

Related issues: https://github.com/opencv/opencv-python/issues/622, https://github.com/opencv/opencv-python/issues/614

OpenCV-Python team works on security-fix release of the package.

Done with releases <IP_ADDRESS> and <IP_ADDRESS>.


[BUG]

Describe the bug
[06/03/2021, 16:56:26] [Harmony] (Harmony)ERROR (1)- refreshCurrentActivity Error: WebSocket closed with reason: undefined (undefined). - Stack : Error: WebSocket closed with reason: undefined (undefined).
at WebSocketAsPromised._handleClose (/usr/local/lib/node_modules/homebridge-harmony/node_modules/websocket-as-promised/src/index.js:353:19)
at WebSocket.listener (/usr/local/lib/node_modules/homebridge-harmony/node_modules/websocket-as-promised/src/index.js:311:64)
at WebSocket.emit (events.js:315:20)
at WebSocket.emitClose (/usr/local/lib/node_modules/homebridge-harmony/node_modules/ws/lib/websocket.js:198:10)
at Receiver.receiverOnFinish (/usr/local/lib/node_modules/homebridge-harmony/node_modules/ws/lib/websocket.js:815:20)
at Receiver.emit (events.js:315:20)
at finish (internal/streams/writable.js:657:10)
at processTicksAndRejections (internal/process/task_queues.js:80:21)
[06/03/2021, 16:56:26] [Harmony] (Harmony)WARNING - socket closed
Config
{
"platform": "HarmonyHubWebSocket",
"name": "Harmony",
"hubIP": "<IP_ADDRESS>",
"cleanCache": false,
"publishSwitchActivitiesAsIndividualAccessories": false,
"activitiesToPublishAsAccessoriesSwitch": [
"Regarder CanalSat",
"Regarder Apple TV"
],
"switchAccessories": true,
"TVAccessory": false,
"showTurnOffActivity": "inverted",
"activitiesToPublishAsInputForTVMode": [],
"remoteOverrideCommandsList": [],
"skippedIfSameStateActivities": [],
"devicesToPublishAsAccessoriesSwitch": [],
"sequencesToPublishAsAccessoriesSwitch": [],
"homeControlsToPublishAsAccessoriesSwitch": [],
"otherPlatforms": []
},
Additional context
Homebridge v1.3.2
Any idea of the problem which occur randomly.

Hi, it is a network issue, nothing I can do about it. The socket was closed, if it reopens and works after it means that  it is correctly handled.
If not, check that your hub ip is fixed and that there is no dhcp renewal (every 24H or around that)


Distributed, Asynchronous Traning, SubprocessVecEnv, Trainer class

This is a big issue, so if anyone else is willing to contribute, just comment below

I'll do this in parallel with DQN

Sure

Also, it would be better to resolve #72 first.

Yes, we should start with VectoredEnv and then maybe try to use this with the shared memory concept of the rlpyt library. I'll start with the vectoredEnv implementation.


Grammar

What are the rules needs to follow for hooks? -> What rules need to be followed for hooks?

Thanks @LewWadoo for PR


wt.exe from command line with named console binary picks the wrong tab icon and wrong tab title

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.450]
Windows Terminal version (if applicable): 1.2.2381.0

Steps to reproduce
start Windows terminal from console or Run... with a chosen named .exe that produces console output.
eg. win+R wt thiscommandisjustanexample enter
(picking a binary that actually does exist is not making a difference for this bug)

Expected behavior

Tab gets it's title from the binary's name

Tab gets it's icon from the called binary / tab gets a generic console-binary icon, since the binary does not contain one

Actual behavior
The tab inherits several pieces of behaviour from the default profile.
This includes,

the icon that is defined for defaultProfile in settings.json
the tabTitle that is defined for defaultProfile in settings.json
the default profile's backgroundImage with it's settings for backgroundImageStretchMode and backgroundImageAlignment applied as defined in settings.json

Reflection
You are just applying the default profile to anything unknown that runs, don't you?
Awesome. In that case, I guess, Windows Terminal requires a hidden default profile that is invoked if nothing else applies. Instead of just picking the one that is opened as default on start.

It looks like Windows Terminal does not attempt to use the icon and title of the command at all. I have Git Bash as the default profile, and if I run wt cmd, then Windows Terminal starts cmd.exe in the Git Bash profile and gets the icon and title from that profile. I can override the title with a command like wt --profile "Command Prompt" --title "nslookup" nslookup but cannot get the command-specific icon.
In the console host (cmdhost.exe or openconsole.exe), SystemConfigurationProvider::GetSettingsFromLink tries to find an icon and a title for the console window. If it later becomes possible to set Windows Terminal as the default terminal in Windows (https://github.com/microsoft/terminal/issues/492, https://github.com/microsoft/terminal/pull/7414), then it would make sense for Windows Terminal to choose the icon and title in a similar way. (If a user just runs wt thiscommandisjustanexample, then there is no .lnk file involved, and many features of that function cannot be used.)
What is wrong with using the background image of the default profile?

@KalleOlaviNiemitalo I use profile specific backgrounds. One for Debian, one for PoweShell, and so on.
Calling wt.exe with a console binary in Windows and getting Debian's logo (because Debian is my default profile) on it...something does not sit right with me about that.
It's not the bug that will hinder Windows Terminal from shipping, but it's also Not Right^tm

Perhaps wt command and wt new-tab command should choose the profile this way:

If the -p or --profile option is given, use that.
Otherwise, if commandProfile is defined in settings.json, use that. This would be a new setting. It could refer to a hidden profile. The name of the commandProfile setting would not depend on command.
Otherwise, if defaultProfile is defined in settings.json, use that.
Otherwise, use the first profile.

commandProfile would not be used for plain wt or wt new-tab without command.
That would solve the background image issue but would still not use a command-specific icon and title.

So, this is going to become more important once we land #492. The solution we're going to go with in the interim is...

If there is a defaults block (profiles.defaults), apply that to any applications started up by the default terminal interface.

I don't think we can generally grab icons from whatever you launch (without grovelling in the process tree to figure out whose executable is on the hook for running a given commandline), but we can definitely fall back to a default icon. Perhaps we should do the process tree thing.
When we apply this change to commandline invocation (which I do think we should), there will be a split between -p x command and command invocation, just like there should have been.
After we land that, we can talk about process name matching. ConEmu supports taking over as the default terminal on Windows, and it lets you specify "when x.exe is running, my profile is y". We should absolutely have something similar.
After that, we can hook up default mappings (any cmd.exe => Command Prompt unless hidden, any pwsh.exe, powershell.exe => a powershell profile).

As a workitem that's going to require nontrivial work and design discussion even though its outward appearance is that of a bug, this is going to be one of our rare "task-bugs".

/cc @miniksa this is topical for your recent defterm investiation

I may xlink this to #1504, #9458, #4768. This might just be a dup of that trio.

Hmm. This is... trickier than it might seem on the surface. Cause here's the thing - we almost always get the tab icon from the profile directly. That's right. So in the profiles.defaults + commandline case, we don't really keep the original commandline around.
Goddard, options:

We persist the icon as a part of the Pane, and then get the icon out of the active pane, rather than the profile of the pane.
We persist the icon as a part of the TerminalSettings / TermControl. This is the same as the first, but with more plumbing (for a future where clients could like, set their own icon (#1868))
We persist the NewTerminalArgs as a part of the Pane. There's been discussion of this before, in #5047.

Though, in that thread, we actually decided it would be best to build up a new NewTerminalArgs from the current settings of the Pane, rather than stashing the original.
So that sounds kinda like option 1

Okay so option 1 - put the icon in the pane. Follow-up problem: now, when the settings reload, we have to re-use that commandline to rebuild what the icon for the pane should be. Barf.

~I bet after #15843, if you set the icon of the profiles.defaults to null, then wt -- foo.exe might just auto pick the icon of foo.exe~
Nope, no, it won't. It'll still try to pick the icon of the profile, because we still don't do the NewTerminalArgs propagation thing. We're literally asking for the Profile of the pane, and getting the icon from the profile. Not from the TerminalSettings or anything like that.

I hope we solve the issue of icon picking differences between conhost and wt!
conhost nu and wt nu


Modify leg components fusion model to fit mx64 motors in knees

CAD models of the leg components are complete. They will be printed sunday or monday. Mx64 motors will be installed in the knees tuesday or wednesday.


cannot reproduce the accuracy on CUB

I am trying to reproduce the result of the CUB dataset, which is 90.6 acc ( table 1 in the paper). However, I use the same config and startup script as the code repo, but only get 90.03 acc for the last epoch. I notice that the total training epoch for fgvc task is not reported in the paper. So what is the proper epoch to get the 90.6 acc? Are there any other reasons that could affect reproducing the acc?
Please see attachment for my training log. Thanks
train.log
!

Hi, thanks for your interest in our paper. As mentioned in our paper, the final results reported in Table 1 are based on multiple crops and horizontal flipping. You can use the infer.py to obtain the final performance. The log only records the single-view accuracies measured during training.

Thank you for your reply! I use the infer.py and the model_bestacc.pth to do the test process. The test acc is 90.37, which is higher than the acc reported during training, but still slightly lower than the result in table 1. How can I further improve the accuracy? Does it cause by the variance between different types of GPU or different random seeds?

I think the difference may be caused by the environment (GPU types, Pytorch/CUDA versions) or random seeds. In my log, the max accuracy during training is 90.0 and the multi-crop accuracy is 90.6.

really appreciate your help!


Wrap all APIs in LogDB to process request gracefully

We'll wrap LogDB API with try/except and if error happen with LogDBBackend we'll get exception and write error to the logger. But LogDB APIs will proceed gracefully with failures on LogDBBackend.

Please take PR: https://github.com/dmwm/WMCore/pull/5781


[Azure Monitor OpenTelemetry] Adding properties in Live Metrics Documents

Packages impacted by this PR
@azure/monitor-opentelemetry

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


Uplift third_party/tt-metal to 6d7cc2c38ae3efc36326f5407698b05fb314b5b0 2024-12-16

This PR uplifts the third_party/tt-metal to the 6d7cc2c38ae3efc36326f5407698b05fb314b5b0

will merge, is passing tt-forge-fe (link) when running manually on passing commit (latest tt-forge-fe main is broken). This uplift brings these tt-metal commits and moves us back to tt-metal main instead of branch where I reverted 2 commits on the weekend.
git l ed413ee399ad289740105b709c20e9144ec4f035..388e187b27f8924ea09512719f666407c40f6bd9
2024-12-16 (388e187b27) by GitHub (Author<EMAIL_ADDRESS>: #15780: div ops debug (#15992)
2024-12-16 (1c542ec3ce) by GitHub (Author<EMAIL_ADDRESS>: Update CODEOWNERS - add experimental CCL section (#16039)
2024-12-16 (ac77db7c74) by Pavle Josipovi<C4><87> (Author<EMAIL_ADDRESS>: #16014: Fix yolo4 e2e perf measurement
2024-12-16 (7271651bd0) by GitHub (Author<EMAIL_ADDRESS>: #0: support unequal ranked inputs for broadcast in binary_ng (#15957)
2024-12-15 (6d7cc2c38a) by GitHub (Author<EMAIL_ADDRESS>: #0: Prevent slice from padding up a 0 volume tensor (#15988)
2024-12-14 (201eff7fcb) by GitHub (Author<EMAIL_ADDRESS>: Bump umd (#15967)
2024-12-13 (a3801c498d) by GitHub (Author<EMAIL_ADDRESS>: Add support for padding along width dimension to ttnn.pad (#15985)


add missing value

Thanks @mikhailborodin42 for the report. We should have a new minor version of the SDK for you tomorrow. Thanks!

This release resolves this issue: https://github.com/clerk/clerk-sdk-python/releases/tag/v1.4.0


bug: DOC|DOCX and PPT filetypes are UNK in web-app/docker-container

In our efforts to refine the Dockerfile in this PR we found that both the container and the web app for the API successfully parse some document types but for .doc, .docx and .ppt raise ValueError: Invalid file. File type not support in partition. The filetype at the point of this error is FileType.UNK inside detect_filetype(filename, file) in unstructured.partition.auto.partition This happens despite counting with unstructured v0.5.0 installed.
How to reproduce:

For the image/container: make docker-build && make docker-start-api
For the web app: make run-web-app
curl -X 'POST' 'http://localhost:8000/general/v0.0.4/general' -F<EMAIL_ADDRESS>(OR using @sample-docs/fake.docx or @sample-docs/fake-power-point.ppt.

Definition on done:

Successfully curl POST the .doc, docx, .ppt, .html and .eml documents in the sample-docs folder (web app and docker container).

What's the difference between this and https://github.com/Unstructured-IO/pipeline-general/issues/40 ?

What's the difference between this and https://github.com/Unstructured-IO/pipeline-general/issues/40 ?

#40 is a different error, Could not initialise NNPACK! Reason: Unsupported hardware happens only inside the container and is probably an issue with building detectron from source that we might solve by switching to a different base container. The web-app can parse .pdf and .jpg with no problem. Here in #41 the error seem unrelated and maybe is due to something in the tools code.

Testing inside the container it works using filename param but not file. The gist with the traceback is here.

With fix #44, docx is now FileType.UNK and part of the scope of this Issue.

Updated parsing outputs:
Testing your branch fix/docx-pptx-in-api:

.ppt: web-app: ☑ | curl container: traceback gist here | terminal container file and filename params: traceback gist here.
.doc: web-app: ☑ | curl container: ☑ | terminal container file and filename params here.
.docx: web-app: ValueError: Invalid file /var/folders/01/tgx6z.../T/tmp9.... File type not support in partition (FileType.UNK). | curl container: ValueError: Invalid file /tmp/tmpcug12hvc. File type not support in partition. | terminal container file param: ☑, terminal container filename param: ☑

Looks like all these filetypes are working locally and in the container now. Closing this via https://github.com/Unstructured-IO/unstructured-api/pull/82

Does this also work if the file submitted to the API does not have a file content type specified or is "application/octet-stream"? [.ppt and .doc in particular, pretty sure .pptx and .docx are fine]

Yep, verified that it works locally and in the container by printing out the inferred content type (application/vnd.ms-powerpoint and application/msword.


"Reset to defaults" Menu and Semantic Enrichment

Issue Summary
Activating Reset to defaults does not consistently set the MathJax configuration back to the default. When activated, collapsible math or Semantic Enrichment is obtrusively enabled and remains enabled indefinitely.

When "reset to default" is pressed, the collapsible math setting becomes enabled indefinitely (see screenshot below).
If a change is made in the a11y menu or subsequent activation of "reset to default" is pressed, the Math content disappears

Steps to Reproduce:

When the page loads confirm that the default configuration is accurate (via MathJax Menu or via the DOM)
Open the MathJax Math Settings menu and activate "Reset to defaults"
Observe that that collapsible math becomes enabled (and checked in the MathJax Menu)
You can use the console to view the (default config) options as indicated below:

It is my understanding that resetting to "default" settings via the MathJax menu should always reset to options in the configuration noted below.
Technical details:

MathJax Version: 4.0.0-beta.7
Math Rendering SVG
Windows 10 19045.5131
Chrome

I am using the following MathJax configuration via:
<script src="/mathjax/custom-mathjax-config.js" id="MathJax-Config-script"></script>
The Custom Config script contains:
options: {
	a11y:{
		braille: true,
		speech: false, //we generate our own aria-label to match the alttext attribute of math element
		subtitles:false,
		viewBraille: false,
	},
	menuOptions:{
		settings:{
			assistiveMml: true,
			collapsible: false
		}
	}

  },

If menuOptions.settings.enrich is set to false within the configuration, the Semantic Enrichment menu item remains enabled along with Collapsible Math when the steps above are followed.

OK, I was able to track down the problem, and it is a little bit complicated.  When the reset menu item is selected, the menu goes through the settings and tries to return them to their original settings for the page.  Unfortunately, the list of original settings turns out to be incomplete (the a11y settings aren't included, because they are actually stored in a different component, and it isn't always available when the menu starts up).  When those settings are reset, they end up getting undefined values.  One unexpected consequence of that is triggered when the highlight item is reset.  Since that value should be one of None, Hover, or Flame, setting it to undefined means that tests for which of those values it is don't work properly.  In particular, the Hover and Flame values require the collapsibility extension, and when the highlight value is not None, the menu code loads that.  That ends up turning on the collapsible setting, which is why you are seeing it active after the reset.
The issue with the pre-rendering has to do with a promise that is not being properly initialized during the initial typesetting of the page.
Both use issues can be worked around with the following configuration:
MathJax = {
  options: {
    a11y:{
      subtitles: false,
      viewBraille: false,
    },
    menuOptions: {
      settings: {
        braille: true,
        speech: false, //we generate our own aria-label to match the alttext attribute of math element
        enrich: false,
      }
    }
  },
  startup: {
    ready() {
      MathJax.startup.defaultReady();
      MathJax.startup.rerenderPromise = MathJax.startup.promise;
      const menu = MathJax.startup.document.menu;
      menu.defaultSettings =
        Object.assign({},
          menu.document.options.a11y,
          MathJax.config.options.a11y,
          menu.defaultSettings
        );
    }
  }
}
</script>
<script<EMAIL_ADDRESS></head>
<body>

$$x+y\over z-1$$

</body>
</html>

Note that I have moved the speech and brailleoptions to themenuOptions.settings` block.  Although they exist in both options blocks, when the menu code is included in the MathJax component, the menu option takes precedence.  That probably needs to be straightened out, but that's the current situation.
I will make a PR to handle both the issues you have raised.


beforeAll should ignore non-function values

Describe the bug
beforeAll accepts a optional teardown function returned from the callback, but there is a typecheck being performed which throws a TypeError: hook teardown value must be function, received "boolean" when truthy non-function values like true are returned. It is useful for one-liners like:
beforeAll(async () => await connectDatabase()); // if connectDatabase returns `true`, vitest throws

This typecheck breaks compatibilty with jest which does not typecheck the return value, so I suggest vitest to just silently ignore any non-function value returned in beforeAll.
Reproduction
Works in jest but throws in vitest:
beforeAll(() => true); // TypeError: hook teardown value must be function, received "boolean"

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is expected behaviour, please refer to the migration guide.

I see, so it's documented, but it still feels wrong that vitest does this typecheck, it could just typecheck for function and run it if present, removing the need for this migration step.

I think it's better to be strict in this case, because hooks implementation differs from jest, and if we don't throw for non-functions, users might wrongly think it is the same, and accidentally return a teardown function. TypeScript types also don't allow anything but a void/undefined/null/function.

Okay with me.

@sheremet-va I ran into this also, and have a follow up question. FYI I am investigating moving from Jest to Vitest. I have several beforeAll / afterAll hooks that need to perform multiple asynchronous tasks (e.g. promises). For example, we might need to create a bunch of test data in our DB. The documentation says that if you return a promise from the before/after hooks, it will wait. But, it does not specify that the promise must resolve in a null or undefined value.
This does not work...
beforeAll(() => {
  return Promise.resolve('hello');
})

But this does...
beforeAll(() => {
  return Promise.resolve(null);
})

Unfortunately for us, we have many, many before and after hooks that look something like this...
beforeAll(() => {
  return Promise.all([
    somePromise(...),
    somePromise(...),
    somePromise(...),
    etc...
  ]);
})

Obviously, this causes the error in question.
Certainly, we can refactor our before/after hooks, and that is what I will likely do. I just wanted to add another comment to consider.
Maybe in the docs for the before/after, some information is added that emphasizes that IF you return a promise, the promise must also resolve to null or undefined.
Thanks!

We would need to do something like this...

Why can't you just await?
beforeAll(async () => {
  await Promise.all([
    somePromise(...),
    somePromise(...),
    somePromise(...),
    etc...
  ])
})

@sheremet-va I also feel the same way. It should be a warning (at most) instead of throwing an error.
In my case, it's a bit different. I'm using imba which compiles to JavaScript.
And in Imba we have implicit returns. i.e: the last statement of a function is returned.
Now, instead of writing
beforeAll do browser = await chromium.launch!
afterAll do browser.close!

beforeEach do page = await browser.newPage!
afterEach do page.close!

I need to add two lines of code per hook in all my test files.
beforeAll do
	browser = await chromium.launch!
	return
afterAll do
	browser.close!
	return

beforeEach do
	page = await browser.newPage!
	return
afterEach do
	page.close!
	return

I think at this stage, vitest is not an alternative to Jest anymore. It's a fully fledged test runner ...

To me it feels wrong that vitest throws on
beforeAll(() => true)

but not on
beforeAll(() => false)

Might as well go all the way and throw on anything besides undefined and function. Thought I still think it should just silently ignore everything but function.


Discovery implements EventEmitter interface

Coverage remained the same at 73.394% when pulling<PHONE_NUMBER>8c0e0cd6bc21abcec8339b1322bfa3 on feat/eventemitter-pattern into 67f2d6db0abb2535672242e79e71e941fc6fd785 on master.


Add gofmt and govet to run via golangci-lint

Add gofmt and govet to run via golangci-lint. With this, we can get rid of fmt and vet and use a universal linter in both ci (present already) and local
Signed-off-by: Nithya Natarajan<EMAIL_ADDRESS>

Now lint task will run everything. So, do we need the fmt, vet make tasks now?

@arunvelsriram we can remove them from make tasks. I wanted to get an opinion before removing them. Shall I also make those changes?

@nithyanatarajan  if lint target can run the other tasks, we can remove the other make targets

Modified the .golangci.yml to fail if there are format errors.
make fmt formats the code if any formatting needed while make lint fails the build if any formatting needed (use to -s option)
Removed make vet as it is redundant

@arunvelsriram @jskswamy Kindly review the changes


[kotlin] Update 1.6.20

https://github.com/JetBrains/kotlin/releases/tag/v1.6.20

Regex was using old variables, fixed that.