MVC assumes IHostingEnvironment.ApplicationName contains the entry point assembly's name

Some parts of MVC seem to assume that IHostingEnvironment.ApplicationName actually contains an assembly name.
The situation:

We implemented a custom application container (think generic entry point assembly to bootstrap a pre-configured environment + pipeline etc. for lightweight services/applications).
We set the ApplicationName property so it matches whatever the actual implementation is handed off to (an application-level project/assembly).
Our arbitrary application names often do not match assembly names.

Typically these applications have no views (only RESTful APIs), so we didn't come across any issues until now. But this one has a minimal UI too, and the result was a FileNotFoundException buried deep in a stack trace down to the RazorViewCompiler.
It took some digging in the sources to find that the CSharp compiler tries to load an assembly based on that property, and looking further revealed that there are other places in the code base, for example around the assembly parts discovery, that rely on IHostingEnvironment.ApplicationName returning an assembly name.
The good news is that today I learned about the concept of dependency context and its compilation options. The bad side of this was that it was super-non-obvious what happens and that it took too long to understand what the actual problem is.
"ApplicationName" is really bad semantics for this, but I guess that's baked now and cannot be changed. At least make the documentation very clear on what the purpose/intention in the context of MVC is, so people don't change it randomly to "nicer" things. The documentation of IHostingEnvironment simply says it's set to entry point assembly name, but the documentation on hosting fundaments says you can happily change this and even shows (bad) sample code of how to set it to something like "MyApp". Btw. there seems to exist an older issue for that (milestone 1.0.1) that apparently never had been completed.

The doc does needs an update. We should remove that configuring the application name to something arbitrary.
/cc @Rick-Anderson @tdykstra

@MisterGoodcat  would you be able to create a PR or draft the changes needed?
/cc @guardrex

I'll note on https://github.com/aspnet/Docs/issues/1870 that it might surface setting <AssemblyName> in the project file, but the topic that we would hope to link doesn't actually exist yet: Documenting .NET Core MSBuild targets and properties/CSPROJ (docs/#2642). It would prob link to Common MSBuild Project Properties.
@MisterGoodcat If u just want to PR a one-liner to cover this, you don't need to sweat the full https://github.com/aspnet/Docs/issues/1870 ... I added it to a list of backlog issues that I hope to address in the coming few weeks. Just note in the issue comment: Addresses #1870; or conversely if u prefer I shoot a quick one/two line fix in, let me know, and I'll do that.

@davidfowl Are u saying "remove" as in "remove it completely" or as in "just remove the arbitrary value in setting it"?
How about something like ...
new WebHostBuilder()
    .UseSetting("applicationName", "MyDependencyContextAssemblyName")

... along with a combined note based on current language, IHostingEnvironment interface text,  [@]danroth27's language, and a note on the <AssemblyName> ...

This configuration setting specifies the value for `IHostingEnvironment.ApplicationName`. This property is automatically set by the host to the assembly containing the application entry point. You can set the property in cases where you want to specify the dependency context assembly yourself. If you wish to set the app's assembly name, use the [<AssemblyName>](https://docs.microsoft.com/en-us/visualstudio/msbuild/common-msbuild-project-properties) property in your project file.

@davidfowl Are u saying "remove" as in "remove it completely" or as in "just remove the arbitrary value in setting it"?

Remove it completely.

I'll patch it now.

Hi and sorry for the late reply, I was out of town for a few days. Do you still need anything from me?

@MisterGoodcat The AssemblyName host config language was removed from the fundamentals/hosting topic. They may introduce a FriendlyName in v3 :point_right: https://github.com/aspnet/Hosting/pull/1180#issuecomment-323616208. AFAICT u can close this issue.


Google Authentication support

Following https://github.com/bazelbuild/bazel/issues/12135, I figure I'd open an issue to track adding support for Google Auth access tokens in bazel-remote. This would allow exposing it on the internet.
There are two endpoints to verify tokens. I'm thinking of basically adding which domains one wants to authorize users from via the hd and email_verified keys.

fwiw, fronting bazel-remote with a dedicated proxy service in the style of https://github.com/oauth2-proxy/oauth2-proxy will be more flexible and doesn't require this project to implement auth for all those auth providers

fwiw, fronting bazel-remote with a dedicated proxy service in the style of https://github.com/oauth2-proxy/oauth2-proxy will be more flexible and doesn't require this project to implement auth for all those auth providers

It would be good to add some documentation for this setup- have you tried it already?

I've not: the cluster I run can get by with no auth + security groups to avoid the latency overhead of checking auth

I've looked a oauth2-proxy, and unfortunately, it manages its own sessions: https://oauth2-proxy.github.io/oauth2-proxy/configuration/sessions
If you folks know if there is a proxy that would only valide tokens against Google's token-info, I'm all ears.


Gaussian integration measure now allows for diagonal covariance

Issue #, if available:
Description of changes:
A function f is integrated w.r.t. an integration measure p, for example the Gaussian measure. EmuKit so far allowed the Gaussian integration measure only to have a scalar covariance (isotropic). This was a quite big limitation. This PR extends the measure to allow for diagonal covariance matrices as well. The kernel embedding for the RBF kernel is adjusted as well (this is the only kernel that had an existing kernel embedding w.r.t. Gaussian measure). Tests and docs are adjusted, too.
Visible (user facing) changes are:

IsotropicGaussianMeasure rename to GaussianMeasure, its property variance is now a 1d array instead of a scalar.
QuadratureRBFIsoGaussMeasure renamed to QuadratureRBFGaussianMeasure

It is still possible to obtain the previous functionality by simply instantiating the Gaussian measure with a scalar variance.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@apaleyes , I am changing names of 2 classes here that might be user facing if EmuKit quadrature is used in a plug-and-play style. The old names are outdated and too restrictive. These are GaussianMeasure and QuadratureRBFGaussianMeasure. Just wanted you to be aware (or voice any objections to this). I think it's long overdue to rename those though. The mentioned classes are not interfaces.


Getting voice activity from voice channel

Tried to implement it but haven't figured it out yet, kept returning false even if I was talking. Anyone is welcome to give it a go. The code is in the Dev branch.

I've changed the code in Dev to check how many members are in the audio channel. This is a temporary fix.

didn't mean to close it, my bad :)


Zoom level resets to default 80% when changing image

Hello lovely maintainers,
I have been battling with this problem for a long time and it holds me back from doing object detection projects sadly.
Problem: Zoom level resets to default 80% every time I go to a new image.
How I would like it to work: After setting the Zoom level to 100% it should stay 100% until I close the app.
I have tried: Finding the place to edit in the code. I got close but had to give up. I also tried googling the problem in many different formulations. With no solution in response.
Thanks for your answer and all your work on this great software 🥇

My version is 1.8,5


[SuperEditor] - refactor mobile widget tree to match the new desktop structure

We refactored SuperEditor's desktop widget tree structure in #645 to allow for carets in a document overlay. This involved separating scrolling from gestures.
We already had to separate scrolling from gestures on mobile, because mobile drag handles sit in an overlay.
De-duplicate any Scrollable, auto-scrolling, overlay, and gesture behaviors that we can, between mobile and desktop.

I think I filed a duplicate ticket of this and we recently did this work. There might still be opportunity to centralize mobile auto-scrolling and other more nuanced improvements. But the desktop and mobile widgets trees are now generally de-duplicated.


"Bosh Lite Director" name is not compatible with credhub

Hi,
The name of the director suggested in this example:
https://github.com/cloudfoundry/docs-bosh/blob/master/bosh-lite.html.md.erb#L52
is not compatible with Credhub - if you add Credhub to your bosh deployment and then try to deploy something using Credhub to generates credentials, it fails because by default credentials will be located under /director_name/deployment_name/credential_name and "Bosh Lite Director" is not a valid path (spaces are not allowed).

You're right it's currently an issue. I'm going to close this in favor of #392 which proposes to change it.


Random hard freezes on Linux on v1.14.3

Bug
(Opening this in place of @Footsiefat, poke them for more info)

After the latest northstar update im getting random hard freezes where I have to pkill the game
anyone else having this/knows a fix?
1.14.2 works, 1.14.3 breaks

https://discord.com/channels/920776187884732556/972909610140839956/1118412950655475772
Steps to reproduce

Use Linux
Install Northstar v1.14.3
Play some games
At some point random freeze

Specifications

Northstar version: v1.14.3
Platform: ???

nslog2023-06-14_17-33-31.txt
nslog2023-06-13_17-50-46.txt

Closing as not reproducible


http tweaks

must use URI to form http request

soda-fountain #354 SUCCESS
This pull request looks good


Support for small screen sizes

Hello,
This is just an observation when checking the kitchen sink demo on iPhone SE running iOS 11 (I assume this will be reproducible in similar android phones with small screen sizes as well).

In the first image the words for the tabs are wrapped.
In the second image the icons are not correctly aligned:

With latest NativeBase, the issue is only with Footer
Rest of the components work fine on devices of smaller dimensions

Footer issue doesn't appear now


New class instance sometimes doesn't have data

I posted this issue on Flutter and I wonder if it is truly a dart-lang issue. Please see here: https://github.com/flutter/flutter/issues/21222

Can you reproduce outside Flutter?

I'm not sure how I would model this outside Flutter since it is responding to a scroll listener.


bug(mat-tree): How to control a tree expansion state if treeControl is deprecated?

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
mat-tree has [treeControl] (and many other related classes and parts) deprecated. How are we supposed to control the expansion state of the tree from our component if there is to no treeControl? Was this an oversight in documentation? even your own examples in the documentation use the deprecated features!
Reproduction
N/A
Expected Behavior
I expect deprecated features / componenets / classes to have a new replacement or documentation on HOW to use the new way of doing things that accounts for the deprecated feature.
Actual Behavior
In this case (CdkTree):
    /**
     * The tree controller
     *
     * @deprecated Use one of `levelAccessor` or `childrenAccessor` instead. To be removed in a
     * future version.
     * @breaking-change 21.0.0
     */
    treeControl?: TreeControl<T, K>;

levelAccessor and childrenAccessor have nothing to do with expansion state. Please tell us what we are supposed to use in place of treeControl!
Environment

Angular: 18.2.2
CDK/Material: 18.2.2
Browser(s): N/A
Operating System (e.g. Windows, macOS, Ubuntu): N/A

I had the same feelings, after trying a few stuff, I realised that when you add #node=matNestedTreeNode on your mat-nested-tree-node tag, then you have access node.isExpanded in place of the treeControl.isExpanded(dataNode).

As I was trying to migrate to, I found it through API documentation. You can use MatTree to expand/collapse/isExpanded with, for example:
@ViewChild(MatTree)
tree: MatTree

I agree though, documentation is not yet ready to guide us and migrate away from deprecated features. It looks like non-TreeControl implementation is not mature enough anyway for now.


[VS Code] Joining workspace: Unable to connect to the host. Verify both you and the host are online.

<!--
For Visual Studio problems/feedback, please use the Report

@caidaol  Thanks for reporting the issue. Did restarting the collaboration session help to resolve the issue?
Was this a one off issue, or does it reproduce consistently? Consolidating this with https://github.com/MicrosoftDocs/live-share/issues/327 Please add your comments there.


add excerpts

I added a returned excerpt with each CFI. It'll break the API for backwards compatibility because search results will be returned in the format:
[
  {
    ...
    cfis: [{ cfi: 'CFI_HERE', excerpt: 'EXCERPT_HERE' }],
    ...
  }
]

Let me know if you have any ideas for refactoring it to still support excerpts without breaking the current API.

Hi @andrewlinfoot, sorry for the late late reaction.
Thank you for contributing this new feature! For now I only merged this commit https://github.com/larsvoigt/epub-full-text-search/commit/c59a6d7efd48ae2765554a9bb917d6df2886ff10. By the rest of your P I'm not sure what is going on :) ? And Travis will also fail.
I`m interested in your project. Perhaps we can get in touch<EMAIL_ADDRESS>By the way in the near future I will come up with a newer version (new concept).
Thanks
Lars

@larsvoigt awesome, thanks!
The other commits were just a quick fix because I was having some babel issues in my projects and it was easier to just pre-build the source. Those definitely should not be added to the main repo. I forgot I had this PR open
I run a dev shop in San Francisco and I'm building a react-native app for a client that includes and ebook/e-reader. Needed to display the search results in the app and they are much prettier with an excerpt vs just showing the user a CFI :)
Sure, will shoot you an email.


Create mohanish.md

Adding mohanish.md to contributors folder.

Good job @mohanishdc7, could you pls run git pull origin master to sync the latest changes, that should fix the current error in CI. Thanks


Pkg updates

Better data handling that prevents variables from being overwritten.

Coverage increased (+3.3%) to 71.926% when pulling 2df9d6a3d6e3209c07aea4e872a09ae070c4fa6d on hover2pi:pkg_updates into 784e542d089dd6ba4553ed180cdc46326947fda5 on spacetelescope:master.

Closing this PR in favor of a direct spacetelescope/awesimsoss <= hover2pi/awesimsoss PR #64 .


docs(typo): fix typo in README

I spotted some typos in the README and the settings so I corrected them :)

Tks for fix typo and make this repo better. 🎉


Upstream: New podman desktop release with badge support

We will need to use a newer release of podman desktop with the included badge support from https://github.com/containers/podman-desktop/pull/5522

New release has gone out 👍


[Improve] improve vue-vben-admin's license issue

[Improve] improve vue-vben-admin's license issue

add the LICENSE file from https://github.com/vbenjs/vue-vben-admin/blob/main/LICENSE to ui-licenses
Remove "streampark-console/streampark-console-webapp/LICENSE", There is already a LICENSE in the root path, We don’t need to this LICENSE.

cc @tisonkun PTAL thx.

Push a commit to resolve my comments.

Push a commit to resolve my comments.

Thanks for the driving, it's exciting to have a professional push, it's looking a lot better now.


first attempt at running the app in gha

Let's see what happens!

At least it seems to start running!


Issue embedding fonts

Hi!
Your library is fantastic, thanks so much for your work here.  I'm having some kind of problem embedding fonts.  I'm using pdf-lib to write to files, and pdf.js to display them.  Here's my scenario.  I have a pdf that I've embedded a non standard font in, and written some text, I can view it in pdf.js without issue.  If I then write another string of text to it in a different font, either standard or non-standard, I get warnings from pdfjs about not being able to find the first font I embedded.
Here's an example error from pdf.js:

Warning: fontRef not available
Warning: Error during font loading: Font La Belle Aurore is not available

Here's some of my code I'm using to draw text in the pdf:
drawText: function(pageIndex, x, y, text, fontFamily, fontSize){
    var me = this;

    var pages = me.pdfDoc.getPages();
    var page  = pages[pageIndex];

    //https://github.com/Hopding/pdf-lib/issues/10#issuecomment-402326518
    var toWidth = page.get('MediaBox').array[2].number;
    var toHeight = page.get('MediaBox').array[3].number;

    var embeddedFont = null;
    if(me.isStandardFont(fontFamily)){
      embeddedFont = me.pdfDoc.embedStandardFont(fontFamily)[0];
    }
    else{
      embeddedFont = me.pdfDoc.embedFont(me.getFontData(fontFamily))[0];
    }

    fontSize = +fontSize.split('px')[0];

    page.addFontDictionary(fontFamily, embeddedFont);
    var contentStream = me.pdfDoc.createContentStream(
      window.PDFLib.drawText(text, {
        x: x,
        y: toHeight - y - fontSize,
        size: fontSize,
        font: fontFamily,
        colorRgb: [0,0,0],
      })
    );

    page.addContentStreams(me.pdfDoc.register(contentStream));
 }

It appears that if I re-embed the first non-standard font I used, things seem to work.  In order to view the changes I made, I call PDFLib.PDFDocumentWriter.saveToBytes(), process that into a base 64 string, and pass that into pdf.js.  It works except for this font issue.
I know that's complicated, and involves a library that isn't yours.  Does that make sense?  Do you have any ideas I could try?  Thanks for your help!

Hello @kevinswartz. The scenario you are describing seems like it should work. If you can provide some additional information, I'd be happy to dig further into this.

Can you please share the code/library/function you are using to perform the base64 encoding?
Can you create and share a minimal example that I can use to reproduce the issue? Just an HTML file or Node JS script will do fine.

@kevinswartz are you still having trouble with this?

Yes, sorry for the slow response.  I'll get you some code to reproduce a little later this evening.  Thanks!

Hi @Hopding,
Here's my method to get a base64 string for a document from pdf-lib:
getb64Doc: function(){
    var me = this;

    var uintArray = window.PDFLib.PDFDocumentWriter.saveToBytes(me.pdfDoc);
    var binary = '';
    var len = uintArray.byteLength;
    for (var i = 0; i < len; i++) {
      binary += String.fromCharCode(uintArray[i]);
    }
    var str = window.btoa(binary);
    return "data:application/pdf;base64," + str;
  },

I take output from that function, and directly input into this one to render the updated pdf:
buildPdfDocument: function(uri){
    var me = this;

    var pdfjsLib = window['pdfjs-dist/build/pdf'];
    pdfjsLib.GlobalWorkerOptions.workerSrc = '/node_modules/pdfjs-dist/build/pdf.worker.js';
    pdfjsLib.getDocument({
        url: uri,
        cMapUrl: "/node_modules/pdfjs-dist/cmaps/",
        cMapPacked: true,
      })
      .then(function(pdf) {
        me.pdfObj = pdf;
        me.currentPage = 0;
        me.scale = 1;

        var pages = [];
        var pageFinishedCount = 0;
        for(var num = 1; num <= me.pdfObj.numPages; num++) {
          pages.push({})
          me.pdfObj.getPage(num)
            .then(function(page){
              pages[page.pageIndex] = page;
              pageFinishedCount++;

              if(pageFinishedCount === me.pdfObj.numPages){
                me.pages = pages;
                me.doneLoading();
              }
            });
        }
      }, function(err){
        me.handleNoPreview();
      })
  },

Does that get you close enough?  My project is built with an older version of ExtJS, and it would take a while to break out my font and pdf loading into an easy example.

@kevinswartz Thanks for the code snippets!
I suspect your problem is coming from your UintArray -> Base64String conversion. Unfortunately, JavaScript makes it easy to do base64 encoding/decoding incorrectly (see https://github.com/Hopding/pdf-lib/issues/34#issuecomment-428765074). There's a great NPM package called base64-arraybuffer that I recommend you take a look at. It has a function that encodes Uint8Arrays to base64 strings. If you aren't using a package manager, you can just copy the encode function from here.
However, if the only reason you are doing the base64 encoding is to feed pdf-lib into pdf.js, I think that you're incurring unnecessary time/space overhead. The docs for pdf.js indicate that it accepts TypedArrays as input to getDocument. So you should be able to do something like this:
 pdfjsLib.getDocument({
        data: window.PDFLib.PDFDocumentWriter.saveToBytes(me.pdfDoc),
        cMapUrl: "/node_modules/pdfjs-dist/cmaps/",
        cMapPacked: true,
      })
      .then(function(pdf) {
        ...
      })

Let me know if this helps, or if you're still stuck after trying these approaches.

Hi Anthony,
So you fixed my issue by skipping the step of encoding to base64, and passing the raw bytes directly into the data param of pdfjsLib.getDocument().  Thanks so much for the help!  You're doing some great work here.


possible issue in documentation showing how to register custom scheme for Android

CustomURLSchemeInterceptor documentation Issue
Using code from following section of the documentation did not produce expected behavior
https://github.com/xamarin/Xamarin.Auth#code
Changing LaunchMode to LaunchMode.SingleTask instead of  LaunchMode.SingleTop produced the expected behavior for me.
Tried on Android 5.1.1
Expected behaviour
Custom Tabs should close (UI should return back to app) if user cancels, or Signs in with the third party.
Actual behaviour
UI does not return back to app after hitting cancel or after successful sign in.

@sanjogs  Can you please try this with the latest version.
Regards

no response for two weeks, closing


config(ticdc): ignore the can-merge and ptal labels

Because when using /merge command the bot will record the current commit to keep track of whether there are new commits, and then cancel merge.
Since cherry-pick's PR does not record the commit, the label disappears automatically when the bot merges base.
So it is currently not possible to copy the can-merge label directly.
cc: @amyangfei

@amyangfei  Could you please take a look? Thanks!

Does this mean we should type in /merge in a cherry-pick PR

Does this mean we should type in /merge in a cherry-pick PR

Yes.

/merge


Facilitate unit testing of client code of nats.net.v2

Feature Request
Make testing easier by using interfaces instead of sealed classes.
Use Case:
Writing tests for code that uses nats.net.v2 is challenging, because the types nats.net.v2 exposes are sealed classes which cannot be replaced with test doubles.
Proposed Change:
Define interfaces for at least NatsConnectionPool and NatsConnection to facilitate the use of test doubles so that users of nats.net.v2 can easily replace those and test their code without requiring nats-server.
+ public interface INatsConnectionPool : IAsyncDisposable {
+  INatsConnection GetConnection();
+  IEnumerable<INatsConnection> GetConnections();
+  INatsConnection GetCommand(); // I'm wondering why this is named GetCommand but that's a separate issue
+}

- public sealed class NatsConnectionPool : IAsyncDisposable
+ public sealed class NatsConnectionPool : INatsConnectionPool

// Similar changes for NatsConnection

Who Benefits From The Change(s)?
Every user of nats.net.v2 who wants to test their code.
Alternative Approaches
Have nats-server configured and running when executing tests.

I don't see why we shouldn't create an INatsConnectionPool interface.
We already have an INatsConnection interface, but extension methods (I think just request-reply methods) works with NatsConnection due  to internal calls. We'd need to revisit that.
As for running nats-server for tests, we already have a test utility for that. Do you think it would be handy to publish the test utility on NuGet?

I don't see why we shouldn't create an INatsConnectionPool interface.

Great, I can spend some time on it if you don't mind and hopefully come up with a proper proposal.

That'd be brilliant! Could be a quick PR if you're happy with that.

We already have an INatsConnection interface, but extension methods (I think just request-reply methods) works with NatsConnection due to internal calls. We'd need to revisit that.

I found NatsRequestExtensions and NatsRequestManyExtensions. Technically those methods could be moved to INatsConnection and become ordinary instance methods I suppose. But there is probably a reason for them being extension methods.
Alternatively, the few internal calls used by the extension methods could possibly be made public and become part of the interface. Mocking would then work, although it can be a bit cumbersome to setup.

The idea is to keep public interfaces as succinct as possible. @caleblloyd's input would be invaluable here.

I'm not disagreeing about having interfaces, but our clients run unit tests with actual servers. The java one starts a new server for every single test, the .NET V1 does something similar. It's minimally slower, if at all. If you don't like running a server for every test, you can run an instance per suite of tests. I would say why not have a real server running since you can.

We already have an INatsConnection interface, but extension methods (I think just request-reply methods) works with NatsConnection due to internal calls. We'd need to revisit that.

I think that they are only truly valuable as extension methods if they are extending INatsConnection.  That would mean another implementer of INatsConnection would get Request/Reply extensions via the extension methods without having to do their own implementation.
As extension methods of NatsConnection they might as well be methods on NatsConnection, and members of INatsConnection

I'm not disagreeing about having interfaces, but our clients run unit tests with actual servers. The java one starts a new server for every single test, the .NET V1 does something similar. It's imperceptibly slower. If you don't like running a server for every test, you can run an instance per suite of tests. I would say why not have a real server running since you can.

I personally prefer running my tests against the real thing in general. nats-server makes this so easy with the tiny multi-platform executable with no dependencies. That is one of NATS' great benefits in terms of testing we take it for granted.
I can also see cases when developing applications depending on NATS Client where one might want to write tests all in memory to only test the application logic for example. It might even be a personal preference depending on other factors. This doesn't necessarily help with this NATS client library development (since we are already using the server almost exclusively in our tests) but  it would be useful to provide interfaces (or other utilities) to make application testing easier when using this library.

Setup a draft PR with some comments #109. There are still some things to do, but I would appreciate a quick review.
Just to clarify, this isn't meant to help NATS Client development in this repo but code that calls into it and needs to test fault handling, does not want/cannot take external test dependencies etc.
Not being able to do that was the first issue I encountered when I started to play with nats.net.v2 which I highly appreciate :)


Support of path elements

https://github.com/fel88/DeepNestPort/blob/50dad9f91fcd0f8474cf5c0632a5d35c0bc4cbeb/DeepNestLib/SvgParser.cs#L25
The file SvgParser suggests that path and rect elements are supported (lines 25-74) , but when loading  svg's that contain path elements, no shape gets recognized. I am only able to nest the example svg's that consist of polygon points, using the examples included in this repo. I'm attaching an example SVG with paths to this comment (renamed  to .CSV so that github let's me upload it here), that does not work for me.
I have not found an easy way to transform complex paths to polygons, so this route is not an easy work-around.
If you have an idea what the problem is, I would like to contribute/help!
example_path.csv

Unfortunately, Svg 'paths' are not supported at the moment. Actually, it is better to use the DXF format. SvgParser was used just for debugging purposes.


BuildImage_Peer fails if using local custom Docker base images

In openchain/container/vm.go [lines 134 and 155] the options passed to the
Docker remote API include the "pull" option. According to the Docker docs.:
   https://docs.docker.com/engine/reference/api/docker_remote_api_v1.20/

this means
 "Attempt to pull the image even if an older image exists locally."

What happens however is that the image build fails unless the named image is
available remotely - it seems to ignore the local image in this case. This
means that it is not possible to use private local Docker images as the basis
for OBC Docker images - at least if they are built as documented via invoking
go test -run BuildImage_Peer

I think the option here should be changed to "Pull: false". It's not clear why
one would want to re-pull an image from Docker Hub if a local copy exists, and
there are several reasons why one might want to start from custom Docker
images. (In my case, because I'm porting to POWER and experimenting with
GCCGO).

This is resolved now. We no longer use Pull: true


[WIP] Adding docs for hosting with Apache+wsgi

This PR adds documentation on how to self-host Read the Docs using Apache and WSGI.
Currently a WIP while I work out the kinks and verify all steps.

This is probably not the spot for these docs. We don't support Apache httpd and our production stack is not documented in full here either.

Hmm... OK so what is the preferred method for self-hosting RTD in a production environment?
If you can tell me that, I might be able to investigate the bare minimum requirements and update the docs accordingly.
Simply using python manage.py runserver is not sufficient, as per the Django docs:

Now’s a good time to note: don’t use this server in anything resembling a production environment. It’s intended only for use while developing.

We cover some of this here: http://docs.readthedocs.io/en/latest/open-source-philosophy.html
In short, we don't support any other production instances outside our own. This means you won't find our nginx configuration details in our docs either. Having httpd prod setup in our docs would lead to additional confusion, and almost certainly, questions for us on usage that we won't support.
More generally, we assume if you are trying to run a production instance that you know how to maintain Django (including the proper way to run a server), nginx/httpd, Redis, Elastic Search, Postgres, etc in a production environment. We don't have the bandwidth to maintain this information, nor can even offer support on it.
However, I think these docs should exist -- I just personally don't think our official docs are the place.

Understandable, thanks for the feedback.
I don't have the details for your production environment, so I can't help document that. Sorry!


Empty vendor/cache folder causes Capistrano deployments to fail

My vendor/cache directory is checked in with its contents ignored. On deploying, the bundle:install cap task fails with the following message:
Some gems seem to be missing from your vendor/cache directory.
Removing the vendor/cache directory from version control solves the problem.
This only seems to occur through Capistrano. Running the same bundle install command on the server seems to work fine.
It would be nice to put a note about this in the docs, I suspect it may be tripping up other people:
https://github.com/carlhuda/bundler/issues/1345
http://stackoverflow.com/questions/5958614/how-can-i-add-gems-to-my-vendor-cache-directory-in-a-rails-3-app
rvm 1.8.4
ruby 1.9.2p290
Bundler version 1.0.18
Capistrano v2.9.0

+1


find_element_by_image throws FileNotFoundError,  No such file or directory.  Any help in resolving it?

with open("saveButtonImage.png", "rb") as ref:
buttonStr = base64.b64encode(ref.read())
time.sleep(3)
saveBt = driver.find_element_by_image(buttonStr)
saveBt.click()

The error is self explanatary


Move attributes icons in details page

I think that the attributes icon list would better stay inside the details page instead of description page.
In that page there is already a good summary with compact information about the cache. It could be more complete with the attributes.
What do you think?

Fully agree. I have also voted for this in some other discussions.
But the argumentation at the moment is, that it will make scrolling needed for the detail-page, so some think it is no good idea to do it, before the detail-page has been redesigned.
For my opinion it could also be done before, as I have no problem with scrolling.

More or less duplicate of #841. I'm aware of this and this will be the first thing that I'll do when we have enough space there.

There seems to be a line wrap problem for the icons after moving them to the detail-page.
I am not sure how it was before, but today I ha a cache with 10 attributes and the 10th icon was cut off at the screen edge.

You're right. Please open a new issue so that I don't forget it. ;)
The rows are set manually, we shouldn't use an own calculation of the size. There are enough possibilities to make a flow-layout.


Not a bug, just a small tweak request

An Option to have a Dark background with White-Grey chunck grid?
It's pretty hard for the eyes da white background.

done - better late than never


Add class to placeholder container for overriding style

Currently if you provide a placeholder component, it is wrapped in a container DIV with minHeight set as an inline style:
render() {
    return this.visible ?
           this.props.children :
             this.props.placeholder ?

{this.props.placeholder}
 :

;
  }

This is problematic in my use case since I'm styling my placeholder to fit an aspect ratio, and the minHeight leaves some extra whitespace hanging around.
It seems like the styling of the container should either be removed and left to the consumer to implement or at least a class could be added to override any default styling.
If you agree with this and want me to do the PR, I can do that.

I get your point, but the problem of letting module consumer to add height or minHeight is unreliable, and if there isn't a height or minHeight set, LazyLoad would fail to calculate whether it's visible or not.
Guess we have to emphasize this in our docs, our do we have some enforcement checking to make sure height or minHeight exist?

It seems like that would be something that could be emphasized in the docs - that the placeholder has to have some height in order for the visible calculations to work.
If you don't want to remove that inline styling, would you be up for adding a className to the placeholder container to allow for (messy, but more targeted) overrides?

It seems reasonable, working on it.

fixed in 66d9e534517358c4e18503ecbdd441226364c255

docs
test

released 2.0.3

Currently if I don't use a number height, there'll be a warning message. It's a bit annoying because I can use height={'100%'} or I can specify the height in css. In my view height should be optional

Yeah, I myself also hate warning, but for now I can't figure out a friendly and non-intrusive way to inform people using lazyload component without specifying height (I tried using window.getComputedStyle to find out if the height of the component being lazyloaded  exceeds a certain threshold, say 100px, but that's inaccurate), and without a specified height lazyload might not work properly.
Thoughts?

If no placeholder is provided, use thinks that lazyload will put a placeholder which has the same size as the real element. Then they just ignore the height option. In this case, the height usually will be 0. So warn the user if the height is not specified and 0.
If a placeholder is provided, then the user should be responsible for his own element as it's not provided by lazyload. So no warning message is needed.

Cool, maybe I should reconsider the warning mechanism.


Typo "Azure event hub"→"Azure Event Hub"

https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-diagnostic-logs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#please-close

#please-close


Block bad words for package names

Can we somehow block bad words for package names? We need this feature in the server as well and this can be implemented here:
https://github.com/poac-dev/poac/blob/6e5cdda4c82f3b8a7607891ab0293abbe63d46a2/src/Manifest.cc#L202-L242

https://github.com/finnbear/rustrict


swagger validation error

while the swashbuckle / swagger is working i see an error message that i can not figure out.
here is the message i get
{"schemaValidationMessages":[{"level":"error","domain":"validation","keyword":"oneOf","message":"instance failed to match exactly one schema (matched 0 out of 6)","schema":{"loadingURI":"http://swagger.io/v2/schema.json#","pointer":"/definitions/securityDefinitions/additionalProperties"},"instance":{"pointer":"/securityDefinitions/oauth2"}}]}
and from an online validator it says:
Swagger schema validation failed.
Data does not match any schemas from 'oneOf' at #/securityDefinitions/oauth2
Additional properties not allowed: scopes,tokenUrl,authorizationUrl,flow at #/securityDefinitions/oauth2
Missing required property: in at #/securityDefinitions/oauth2
Additional properties not allowed: tokenUrl at #/securityDefinitions/oauth2
Additional properties not allowed: authorizationUrl at #/securityDefinitions/oauth2
Additional properties not allowed: authorizationUrl at #/securityDefinitions/oauth2
No enum match for: implicit at #/securityDefinitions/oauth2/flow
JSON_OBJECT_VALIDATION_FAILED
anyone know what this means and how to fix it ??

It will validate if you don't set the TokenUrl. You should still be able to use implicit grant through swagger ui without this.

interesting that i do  not always get the error and that the spec seems to list the item token url as valid
https://swagger.io/docs/specification/authentication/
very confusing error message and documentation from the swagger / open api folks.


🐛 REST - Get all members don't restitue all members

Close in this PR https://github.com/zenika-open-source/opensource-statistics/pull/14


does marquee support 'no-scroll' when the marquee content only have one item?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

You want to pause the marquee if there is only one item?

You want to pause the marquee if there is only one item?

thanks for reply, the prop "pause" on version 4.1.0 can solve my problem.


add reason & code to callback #1

it works at my side.

Hi @jtlapp , when will you merge this PR?


Mostieri/export geometry utils interface

Implementation of the geometry interface in the export utils

Codecov Report

Merging #328 (46cdc3c) into main (c7f042d) will decrease coverage by 0.85%.
The diff coverage is 21.87%.

@@            Coverage Diff             @@
##             main     #328      +/-   ##
==========================================
- Coverage   81.12%   80.27%   -0.85%
==========================================
  Files          18       18
  Lines        2506     2535      +29
  Branches      439      383      -56
==========================================
+ Hits         2033     2035       +2
- Misses        319      346      +27
  Partials      154      154

Files
Coverage Δ

src/ansys/pyensight/core/utils/export.py
77.83% <21.87%> (-11.06%)
:arrow_down:

... and 1 file with indirect coverage changes


Where is le bean

Not sure what I’m doing wrong (on the second step of the tutorial), just can’t figure it out:
I do this:

And get this:

Oh the humanity! 🤕

Oh no!
Are you using a local file server? The browser won't allow JavaScript to access local file system unless they're served by a server.

Just wrote a setup guide that explains this.


API endpoint to retrieve pharmacogenomics data by variant identifier

Pharmacogenomics data so far only accessed by target identifier. However, on the variant page, the pgx widget needs to be populated by variant identifier. The data is not yet available. We've been in conversation with the EVA team, however the data would not be ready for 24.09. The data team can provide a patched dataset.
Course of action

[x] Solidify the schema (where is the variantId field, what does it contain exactly) Let Backend team knows. @opentargets/data-team
[x] Generate patched dataset. @opentargets/data-team
[ ] Update API to expose the new field. @opentargets/be-team
[ ] Provide new endpoint to retrieve data from API by variantId. @opentargets/be-team

The pharmacogenomics field is exposed in target and drug does this but the item only refers to target. Does this change has to be implemented in target only?

I just remember we discussed adding the pharmacogenomics field to the variant query. Am I correct?

As we discussed in the August 7th meeting, having this inside variant is good. We will reuse the same abstraction for drugs and targets.

@remo87 , the updated data is here: gs://otar001-core/Pharmacogenetics/json/pharmacogenetics-2024-08-02.json.gz
The data looks like this:
{
  "datasourceId": "pharmgkb",
  "datasourceVersion": "2024-05-05",
  "datatypeId": "clinical_annotation",
  "drugs": [
    {
      "drugFromSource": "ivacaftor"
    },
    {
      "drugFromSource": "tezacaftor"
    }
  ],
  "evidenceLevel": "1A",
  "genotype": "CTT/CTT",
  "genotypeAnnotationText": "Patients with the CTT/CTT genotype (do not have a copy of the CFTR F508del variant) and cystic fibrosis may have decreased response when treated with ivacaftor/tezacaftor combination as compared to patients with the del/del genotype. This genotype is not an indication for use of the combination drug of ivacaftor/lumacaftor according to the FDA-approved drug label for this drug combination and response may depend on the presence of other CFTR variants. Other genetic and clinical factors may also influence response to ivacaftor/tezacaftor.",
  "genotypeId": "7_117559591_TCTT_TCTT,TCTT",
  "literature": [
    "29099344",
    "29099333",
    "28930490"
  ],
  "pgxCategory": "efficacy",
  "phenotypeText": "decreased response",
  "studyId": "1449154729",
  "targetFromSourceId": "ENSG00000001626",
  "variantFunctionalConsequenceId": "SO_0002073",
  "variantRsId": "rs113993960",
  "variantId": "7_117559591_TCTT_TCTT"
}

The new field is the variantId

Preliminary testing on 14/08 meeting looks good.
Unblocked @gjmcn for FE work.


config file not found

The documentation leads me to believe that .gnmic.yaml in $PWD would be found automatically, but I am not seeing that. I have to use the config flag to have it read.
What am I doing wrong?
cl@cl:$ echo $HOME
/home/cl
cl@cl:$ pwd
/home/cl
cl@cl:$ cat .gnmic.yaml
username: admin
password: admin
skip-verify: true
cl@cl:$ gnmic capabilities -a <IP_ADDRESS>:50051
username:
^C
received signal 'interrupt'. terminating...
cl@cl:$ gnmic capabilities -a <IP_ADDRESS>:50051 --config .gnmic.yaml
Capabilities Response:
gNMI version: 0.5.0
supported models:
....snipped.....
cl@cl:$
cl@cl:~$ gnmic version
version : 0.7.0
commit : dcbe8d3
date : 2021-01-28T01:58:29Z
gitURL : https://github.com/karimra/gnmic
docs : https://gnmic.kmrd.dev
I am using Ubuntu 20.04 running natively in WSL2.

Please consider upgrading (gnmic version upgrade)
The change to .gnmic.yml has been added in 0.8.0+

Thank you, that worked!


Show unclaimed vita amount

/kind ui-change
Fixes #739

Should be merge after the https://github.com/iotexproject/iotex-explorer/pull/828

@nmhung1210  rebased branch from master


Flagged info alert not showing in proof request

Description of problem
Proof request asking for credentials with flagged attributes (ex. age) does not bring up flagged information alert. The alert appears sometimes when the Person credential is requested but not when it's just the age (predicate).
Expected behavior
Proof request should bring up alert if flagged information is being requested
Steps to reproduce

Get a person credential and LSBC member card on one device
Go to any proof request asking for age or any proof request just asking for the lawyer credential in mobile verifier
Scan the QR code to get the proof request

Screenshots and/or log output
Private Zenhub Image
Private Zenhub Image
Private Zenhub Image
Environment

[X] Occurs on Android
[X] Occurs on iOS

Build #: 1203
Android Device Model: OnePlus 6
iOS Device Model: iPhone SE
Workaround
Severity

[ ] High
[X] Medium
[ ] Low

@cvarjao I thought we decided against flagging predicates as PII since they are zero knowledge proofs

@wadeking98 Oh you're right. Then the bug fix should remove the flagged UI on the card itself. In the screenshot, where it's asking for JUST the age, there's the warning icon and the font is amber. That should not be the case.

seems to be resolved in testflight now

Just checked. Looks like it works as intended now.


Add support for multiple README files (retrocompatible)

Overview
This code was merged before on #118 and reverted on #235, due to the breaking of Package.readme is renamed to Package.readmes and it is a Tuple[Path] instead of simply Path.
In this PR you will find mostly the same thing of #118 with a few little improvements.
Description
This code reuses the "readme" entry in a way to allow the user to declare in the old way or the new way.
With the changes, the two following declarations are valid:
Single file
readme = "README.rst"

Multiple files
readme = [
    "README.rst",
    "HISTORY.rst"
]

If the user declares files in different formats, the strict validation will issue.
Notice
The class Package suffered an important change: readme was renamed to the plural readmes. Properties for the single form were introduced to ensure retrocompatibility.
Pull Request Check List
Resolves: python-poetry/poetry#873

[x] Added tests for changed code.
[x] Updated documentation for changed code (python-poetry/poetry#4743).

I'm still not the biggest fan of adding another helper, but we can always move it later during a cleanup of that module.
LGTM after feedback is addressed.

@neersighted I have to say that I deeply consider about this decision. But the lack of a new helper would mean to store the content type into Package, which doesn't fit IMHO. So I decided to have the helper.


PDF Font Size

It's a feature request.
When I create a long sheet, I tend to go to word first, because the PDF export is too big in fonts for my taste. Word to PDF works for me of course, no problem, but it would have been nice to be able to set up a font size for PDF exports. (Or maybe get a hint from the zoom level of the right pane)

Zooming the view do not actually zoom the Html document content. For that purpose, please add the following line on the top of your code:
'<style>body {zoom: 0.8;}</style>
This will affect both the Html and the Pdf export.
You can also try to use the print command with some of the available pdf printers like CutePDF:
https://www.cutepdf.com/index.htm
There is a zoom option and some others as well.
Currently, we are not able to customize much the pdf export. We are using an external project for that purpose:
https://github.com/wkhtmltopdf/wkhtmltopdf
Unfortunately, this project is not supported anymore and probably we will need to search for another solution in the future.

'<style>body {zoom: 0.8;}</style> is good enough for me, thanks.


DatePicker calendar partially covers input when near the bottom of the page

Detailed description

Describe in detail the issue you're having.

If the DatePicker is near the bottom of the page, the calendar will open "above" the input, but partially overlapping (so the input is mostly covered).
(If the DatePicker is near the top of the page, the calendar opens below the input, and looks fine.)

Is this a feature request (new component, new icon), a bug, or a general issue?

Bug

Is this issue related to a specific component?

DatePicker (in "single" mode)

What did you expect to happen? What happened instead? What would you like to see changed?

The "above" calendar shouldn't cover the input (like the "below" calendar)
Steps to reproduce the issue

Put DatePicker near bottom of page
Click in the input (so the calendar opens)

Additional information
https://codesandbox.io/s/nnyp7ypr4m

Hi 👋 at a glance Flatpickr’s “auto-flip-up” mechanism does not work well with how Carbon sets up with the UI. Will see if we can work it around in future. Meanwhile I’d recommend putting enough space below the date picker input.


update mondo api to monzo url

First of all, thanks for the great mondo oauth implementation.
I thought I would update quickly the api.getmondo.co.uk urls to the new api.monzo.com url.

👍


LinkResolveOperation

Closes https://github.com/ucbepic/docetl/issues/108

Thanks for this! Do you have a test or any short docs/example usage? I can add tests/docs with an example/intended usage snippet

The only data I have now is a bit too big for an example, but will try to build something minimal for the docs

Added a docs markdown file with an example (it's a bit stupid, but works)...


hamming: add hint to @Ignore annotations

Add a hint with the value "Remove to run test" to the @Ignore annotations in HammingTest. See issue #426 for discussion.

Yay I did this one :smile:

Fixed in PR #495 :)


Localstorage storage provider

LocalStorageProvider is a class for client-side web applications implements StorageProvider interface and connects GlobalStorage to localstorage (Web Storage API Interface), see specs: https://developer.mozilla.org/ru/docs/Web/API/Window/localStorage

@tshemsedinov, I've stacked with warning building library for browser:
WARNING in ./node_modules/metarhia-common/common.js
20:35-42 Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
 @ ./node_modules/metarhia-common/common.js
 @ ./gs.js

I think it can be resolved with refactoring:
const submodules = [
    'utilities', // Common utilities
    'math', // Math common function
].map(path => './lib/' + path).map(require);

to
require('./lib/utilities');
require('./lib/math');


Split FAQ into sections

The FAQ topics under About Mattermost have been split apart into smaller sections to make the content easier to read and navigate for users.

Thanks, @wiersgallak! I've added a ticket to our v6.1 docs board to develop a process for quarterly PM reviews of this content going forward.


[ios] NativeScript compatibility

I am trying to create a CartoMobile plugin for NativeScript.
I am facing an issue related to the CartoMobile framework.  It wont work because it is missing a required module.modulemap. You can see the explanation here
Also i created a Info.plist inside the framework. But that i am not sure is a requirement.
Creating those 2 files make the framework work correctly
Is there any chance you could update the build process to make the framework compatible (cocoapod too)?
Thanks

@Nikituh did you have time to look at automatically adding the module.modulemap in the release?
I still need to add it with rc2 which makes it impossible to use the Pod.
Thanks

@Nikituh Don't worry i got time :D Do you want me to share it with or without the modified framework?

I'll take the whole lot, so it'd work out of the box!

here you go https://ufile.io/ghjlu

Great, got it, thanks 👍

@farfromrefug is your project supposed to work out-of-the-box? After changing the bundle identifier and development team, I'm getting the following error:
ld: framework not found CartoMobileSDK
clang: error: linker command failed with exit code 1 (use -v to see invocation)

How odd, I still can't get your project to build and am, in fact, getting the same errors you pointed out:
const mapView:CartoMap = page.getViewById("mapView");
app/main-page.ts(10,11): error TS2322: Type 'ViewBase' is not assignable to type 'CartoMap'.
  Property 'mapView' is missing in type 'ViewBase'.

and
ld: framework not found CartoMobileSDK

Looking at the pathing in XCode, the .framework file is, indeed, missing from where it is pointed:

This may take more time than initially anticipated. I'll need to take a look at plugin creation in depth and what's up with module.modulemap and Info.plist
In the mean time, development builds are always available at the following link:
https://github.com/CartoDB/mobile-sdk/wiki/Using-dev-build#ios

@Nikituh weird that it does not work with the one i sent you. It should.
Thanks for the dev links. Will keep an eye on it

@farfromrefug
In the end, I wasn't able to get your project to run. However, the good news is that we decided that the iOS SDK should, indeed, be built as a dynamic library, and that will include the module.modulemap.
Hopefully we'll have it by the release of 4.1.2, but I can't make that promise right now.

@Nikituh published my first version of my plugin.
https://github.com/farfromrefug/nativescript-carto
I have trick to use your pod because of the missing module files.
After the mod is installed i copy thos files:
https://github.com/farfromrefug/nativescript-carto/tree/master/src-native/ios/FrameworkFix
into the product framework.
I hope you are going to fix this soon!


Confusing message associated with DEPRECATED_MEMBER_USE when doing a prefixed import

The following code:
import 'package:analyzer/analyzer.dart' as analyzer;

Produces the hint info • 'analyzer' is deprecated and shouldn't be used.  This is really misleading because it makes it sound like the whole analyzer package has been deprecated.  In point of fact all that's been deprecated is the library package:analyzer/analyzer.dart.
We should adjust the message text to something like 'package:analyzer/analyzer.dart' is deprecated and shouldn't be used.

https://dart-review.googlesource.com/c/sdk/+/93066


Lamp56

Docker with
PHP 5.6
Apache 2.4
MySQL 5.7
phpMyAdmin

Hi @Usuario001
Thanks for the contribution, I can not merge it directly to the master branch. I will create a separate branch for PHP 5.6 and then we can merge your changes to it.

Yes, its correct I forgot to tell about it. And maybe could be usefully use php5.6 actually I made this 'cuase I need it.

Hi @Usuario001
I have created a branch for PHP 5.6.x - https://github.com/sprintcube/docker-compose-lamp/tree/5.6.x
You can check out that branch and submit your changes if required.


add padding to table explorer

the buttons in the table explorer (https://github.com/ploomber/jupysql/pull/420) are hidden by the horizontal scrollbar, making it difficult to click, we need to add some padding:

@yafimvo: it the CSS logic implemented here or in the jupysql-plugin repo?

@edublancas here.
Everything related to the TableExplorer widget is in this project (jupysql).


Api Key creation entry point

closes #13170

I've moved the PR to CR so we can make faster progress. I'll be implementing tests meanwhile in this same branch

Added "do not merge yet" label because this contains a migration. This PR must be splitted after acceptance, before deployment, in order to deploy just the migration first.

Failing tests.

retest this, please


Python 3 compatibility

Fails in Python 3 environment with:
eggs/nose-1.3.7-py3.5.egg/nose/plugins/manager.py:395: RuntimeWarning: Unable to load plugin selecttests = noseselecttests:NoseSelectPlugin: cannot import name 'ClassType'

I've pushed Python 3 branch, feel free to test it: https://github.com/domenkozar/nose-selecttests/commit/9098fa4d3bb0eb73d91b4e6631e630cd7977a6a6

LGTM

Closed via e75ab8d43c627d7daa55ca19b258e80059374bb7


Tactical CSAT Fixes for Package Manager Console

NuGet Package Manager Console Guide
(Sept) Last 30 days: 23400 PVs (17.5K search, 2.1K direct, 2.7K MS ref)
50.0% CSAT
Analysis

Install and uninstall are the most common scenarios; opening the console, finding packages, and updating packages are second.

Actions

Review the flow of these scenarios with PM
Rewrite subsections to show primarily quick-example PS commands with comments rather than textual descriptions.
Add information about using the console across different versions of Visual Studio

All tasks completed.
For the last item, the only instance of this is Find-Package, which isn’t available in VS 2013 and earlier; made a note of that.


Flamegraph python -m cudagrad.mlp

There's also this, which is no dependencies and built-in:
https://docs.python.org/3/library/profile.html


Create uk_ua.json

Really sorry for the late review. I will try to make a separate branch for internationalizations soon. Since I know nothing about the language I'll fully trust your translations XD
Thank you!


iOS 17 Animation Issue

Animation glitches when reaching the minimum size of the animation before expanding again.
https://github.com/maustinstar/liquid/assets/796205/237384b6-0269-44f8-995f-c027f7424802

Upon further investigation the "glitch" seems to happen at the end of the "period" time. The view seems to be instantiating itself again, triggering a layout.

The same for me

same

any solution?

created a pr for the issue: https://github.com/maustinstar/liquid/pull/12
FYI: @kingmeers  @iosnickse  @timbueno @maustinstar


Improve / enhance data download funciton

For these indicators:
https://a816-dohbesp.nyc.gov/IndicatorPublic/beta/data-explorer/housing-maintenance/?id=45#display=summary
https://a816-dohbesp.nyc.gov/IndicatorPublic/beta/data-explorer/asthma/?id=2380#display=map
Thanks!

Elaborating a bit...

From the map display, 'download: current table view' downloads the finest available geography regardless of what's being displayed.
Curiously, housing maintenance has UHF42 data available in the map (good) but not in the table, so, hard to use table view as a workaround to download those data.

Semi-related - possibly a new issue. For instances where we're showing NTA, can we also import the standard NTA code? @cgettings, I know you had created another (can you document that in a readme in EHDP/geographies?) because of issues with alphanumeric and the Data Explorer js - wondering if we can still include the alphanumeric so that when somebody downloads, they also get the standard NTA codes.

Proposed function:

Download the data that you're seeing right now (this view)
Download full indicator

add href="" to get mouseover button, and this.preventDefault() ?

I might restructure those footer buttons, too.

nb, added pointer to data download buttons and did a little thinking/investigating:

This Table View is for the table, but it appears on the other views, and I think it communicates "download the data you're seeing now" - that's a problem. We should have a "Download the data you're seeing right now" option as the primary option.
The Table view ("full table") is nicely formatted with the right info - IMO that's what people probably want when they download and that's what we should aim to build.
But for most of the view functions, we pass in a data object. But we can't just send that to the download, because it's missing a ton of information: it doesn't have measure names (just measure IDs), for example. We could build a better download object as a separate function, but it would require some table manipulation (Arquero?) to combine the right information.

@cgettings and @etorem, could you please do a little bit of stress testing on hotfix-de-download? I've rebuilt the download function. There are now two options:

This view: downloads the data that's currently visible: table, map, trend, links, or disparities.
Full table: fairly straightforward, all geographies, measures, and times for the indicator

Mainly, we're shooting to provide data files that have the expected information - ilke the indicator and measure names, and the data that somebody is seeing in the chart on the screen. I've run into some issues this morning with this where  parts of the data download have interfered with other parts of the DE (obvious issues when toggling options, console errors, etc), but I think I've solved that (although the trend/comparisons data file isn't quite as tidy as I'd like).
If you two could please take a close look at the downloads and make sure they make sense, as well as click around a bundle and let me know if you see anything broken, that would be great! This was a little tricker than I anticipated and I broke some things along the way - and fixed them - but want to make sure that we find any other potential bugs before release.

Merged and queued for deployment - going to close this issue.


No sticky scrolling for markdown headers

Testing #156175

Enable sticky scrolling
Open a md file with headers and scroll through it

Bug
Nothing shows up for sticky scrolling
I'd expect the headers to stick (notice how they are present in the outline view)

I just tried this and had the same expectation!

Fixed by https://github.com/microsoft/vscode/pull/157483

Looks good now. Verified.


Show Orders with an ERROR state

This is dependent on an enhancement to the server https://github.com/OpenBazaar/openbazaar-go/issues/838
User Story
The server will sometimes put an order into an ERROR state and not provide it to the client. As a user, I want to know when my orders are in in an error state, so I know they happened and don't think the application is just broken or missing data.
As a user, when I see an order in an error state, I want some guidance on why it happened and what to do to resolve it.
As a seller, if an order has been rejected due to the listing being updated and a stale listing being purchased, I would prefer to have the order go through so I can choose whether to honor it.
Functional Requirements

[ ] When an order is stale due to an out of date listing being purchased, the order should work as normal. No indication needs to be given that the order was out of date.
[ ] When an order has a state of ERROR_PENDING, an "Error" status will be shown.

[ ] The buyer can cancel the order the same as if it was in a PENDING state, if it is not moderated.
[ ] If the order is moderated, the buyer will have to dispute it to retrieve their funds.
[ ] The seller cannot take any action on an order in the ERROR_PENDING state.
[ ] The server will return a error_reason parameter, it should be shown in the Order Details.

@morebrownies what state should errored orders map to?

I suppose we could create another filter. If we do that, we may want to consider making another one for the PAYMENT_FINALIZED state, since right now that's "weirdly" mapped to Fulfilled. Maybe adding two would better balance things than adding 1?

Maybe we could consolidate the 3 dispute filters into 1? Most people shouldn't have a huge number of disputes. A single "some stage of disputed" filter would probably be fine.

I tend to agree with what @jjeffryes is suggesting. I was working on designs for this view yesterday on a smaller device and having 3 dispute filters eats up a fair amount of space. I don't think that level of granularity is necessary for 99.99% of users.
I'd suggest condensing disputes into one filter, which will then open up space for the Error state. @rmisio

@morebrownies ok, it will open up 2 spaces actually. Should I turn PAYMENT_FINALIZED into a separate filter too or just leave it as Fulfilled?

It's probably fine in Fulfilled for now @rmisio. Not sure if more granularity is actually needed at this point. Notifications will help alert the user that payment has been finalized too.

@morebrownies and the collapsed Dispute filter should just be labeled Dispute?

Probably needs to be plural @rmisio, Disputes


homepage styling and signin signup styling

⚠ Is your feature request related to a problem? Please describe
yes. this webpage has little styling to make the page really pop

💡 Describe the solution you'd like
id like to put a background image and change the rendering strcture a bit to give the page more visual appeal

add a picture of a house and stack the signin and signup pages vertically
🤚 Do you want to develop this feature yourself?

[x ] Yes
[ ] No

Just as a note, please make sure that you do any changes to the rebranding branch.

Hello @Tobi406 I would like to work on this issue if its still available

hey @Tobi406  i like to contribute in redesigning  the Homepage and signin and signup . If you can help me this it would be grateful as i am new to open source

i would like to work on this


Pass some env vars that affect msvs tool setup.

MSCommon already makes sure a few variables are propagated before calling the the dev env setup script (vcvarsall.bat or relative). This change adds two more: VSCMD_DEBUG enables VsDevCmd.bat (which is eventually called) to log the environment before and after setup, and VSCMD_SKIP_SENDTELEMETRY, if set, inhibits the new (VS 2019) functionality to send telemetry information.
The change is motivated by the telemetry code generating undecodable output for certain locales.  This should allow suppressing it, but doesn't really fix the underlying problem that we're not
handling text right on Windows (on Py2 this would not have failed, but that was "by accident")
This is intended to work towards fixing #3572, though not sure it's a complete solution there.
Signed-off-by: Mats Wichmann<EMAIL_ADDRESS>Contributor Checklist:

[ ] I have created a new test or updated the unit tests to cover the new/changed functionality.
[X] I have updated master/src/CHANGES.txt directory (and read the README.txt in that directory)
[X] I have updated the appropriate documentation

yes, agree the mbcs is wrong (as is mentioned in the Python PEP 528). Small steps...

Sigh... this is irritating. Does this actually need to be a fail (Appveyor build)?
Test execution time: 2.0 seconds
583/1184 (49.24%) C:\\Python35\\python.exe test\MSVC\msvc.py
Using precompiled headers was not fast enough:
slow.obj:  0.891s
fast.obj:  0.828s (expected less than 0.802s)
FAILED test of C:\projects\scons\src\script\scons.py
	at line 189 of test\MSVC\msvc.py
Test execution time: 12.3 seconds

Sigh... this is irritating. Does this actually need to be a fail (Appveyor build)?
583/1184 (49.24%) C:\\Python35\\python.exe test\MSVC\msvc.py
Using precompiled headers was not fast enough:
slow.obj:  0.891s
fast.obj:  0.828s (expected less than 0.802s)
FAILED test of C:\projects\scons\src\script\scons.py
	at line 189 of test\MSVC\msvc.py
Test execution time: 12.3 seconds

I'm o.k. with leaving the test messaging, but removing test.fail_test()
I'm guessing precompiled headers don't have the performance improvement they used to have..

There's some other test that usually doesn't "pass", but it doesn't register as a fail. I can't find it at the moment, but there's kind of a precedent.  Yes, the information is interesting and I wouldn't just drop it.

Do we need a captured output bytestream to be able to test this?

It would help me figure out if we are doing the right stuff - a mocked tool could return a known-problematic string and see if we can handle it.

It would help me figure out if we are doing the right stuff - a mocked tool could return a known-problematic string and see if we can handle it.

Hmm.. I think we'd also need the "oem" locale set correctly for the decode() to work properly?
Or just keep a polish install of windows 7 and/or 10 around?
I'm ok with changing to oem, and if that works for @lukaszgo1  and doesn't break the rest of our regressions to merge that.  Just make copious comments around that code why it's what it is so someone doens't change in the future unknowingly as repro-ing is non-trivial

This link provided some education for those of us who don't understand what Windows is doing here:
https://discuss.python.org/t/choosing-correct-encoding-for-subprocess-popen/3388/3
The latest drop of the PR changes "mbcs" to "oem" in the function in question.

@lukaszgo1  - are you able to give this latest version a try? If it works for you and doesn't break our regresssion tests I'll merge and it will go into the next release.

@bdbaddog The current code from this PR works great on my machine. @mwichmann Thanks for your work!

@mwichmann - ugh.. py3.5 doesn't seem to have oem encoding?

Indeed.  I didn't realize it was "new":  https://bugs.python.org/issue27959.  Fixed in 3.6.

Yeah, that pretty spectacularly tipped over the CI test on Py3.5: Failed the following 844 tests:
There's one other weird fail, but that seems to be AppVeyor's fault:
The license for Visual Studio has expired.
The evaluation period for this product has ended.

Maybe use mbcs for < 3.6 and oem for the rest?
Yup...Appveyor's license expired for some version of MSVS (again)..

Before we found out about OEM Encoding in Python I have proposed the following:
from ctypes import windll

OEM = "cp{}".format(windll.kernel32.GetConsoleOutputCP())

Not very Pythonic but working regardless of the Python version in use.
I am slightly afraid that for some obscure locale we would hit decoding error when using  MBCS in 3.5

@lukaszgo1 - ah good point. @mwichmann - can you use that for 3.5?

@mwichmann Have you encountered any difficulllties when applying above suggestion from @bdbaddog ? I certainly don't want to rash you just wanted to make sure this wouldn't miss next release of Scons.

I'll try to get back to this soon.

@lukaszgo1 - The plan is that this would be resolved in 4.0.0. (as in 4.0.0 won't go until this is resolved, assuming we can, and that we can get some test to work for it)


V1.4.0 Uncaught TypeError: Failed to construct 'URL': Invalid URL

Dear Single Spa Layout Maintainers,
When bootstrapping a new Single SPA Root Config Application, utilizing Single SPA Layout as of v1.4.0, a type error has been thrown for trying to construct a URL object with the appropriate route paths. This is currently happening with the default EJS file configuration with the Single SPA Demo Application being added to the import map and as a default fallback.
However, this error also persists when running your root config app that is loading in your micro app bundles either over localhost in the local import map and/or from a hosted import map.
This error does not currently seem to be affecting functionality of your root-config or micro app. Just was hoping to have this error removed.
Images attached from vanilla CLI bootstrapped project (no changes or additions).

When downgrading one minor version to 1.3.x, this error does NOT appear

Thank you,
Eric

May be from a chrome extension. Will reopen if I can reproduce.

Hi @ericzorn93, my guess about what would cause this issue is if you're not using the latest version of single-spa in your import map. Note that it's the version of single-spa in the import map in the index.ejs file of your root config that matters - not the package.json version. The reason that this would cause the error is that single-spa-layout assumes that evt.detail.oldUrl and evt.detail.newUrl are defined on routing events, but they were only introduced in https://github.com/single-spa/single-spa/releases/tag/v5.7.0<EMAIL_ADDRESS>started calling new URL() because that was needed to fix a couple bugs. See https://github.com/single-spa/single-spa-layout/pull/106/files#diff-45b83d99dcd0b64a6370843b6833286a554ad98c3bf27bb91b8325fcfed78c78R416.

Thank you for clarifying @joeldenning. I will update as soon as I confirm what version of SingleSPA I am using shortly.

@joeldenning, this fixed the issue. I was running 5.5.0 accidentally. Thank you for your help

👍  glad to hear you got it working.


fix: small typo on readme

I also took the liberty to update the example to reflect the comment description.

Thanks a lot :)


chore: return error msg on RPC call

Purpose
fixes: #2103
https://github.com/Taraxa-project/taraxa-evm/pull/105
How does the solution address the problem

Changes made

prnet cleaned-up successfully


Apply configuration avoidance recommendations for tasks

Expected behaviour
All tasks should be created lazily to avoid extra overhead during configuration phase.
Actual behaviour
Tasks are created eagerly using getTasks.create method.
Steps to reproduce
run gradle help --scan on any project that uses Shot. Check "performance -> configuration". You will see that tasks like "executeScreenshotTests" are created, while they should be created only if they are executed.
Version of the library
4.3.0
More details
Documentation about configuration avoidance: https://docs.gradle.org/current/userguide/task_configuration_avoidance.html
Basically it should be enough to register tasks (tasks.register) instead of creating them (tasks.create) and make sure that all tasks configuration happens inside configure block instead of getting instance of the task.

 Thanks for the report and the pull request. I'm going to review it today!


Create levels for Chapter 4 (Cannoli Sandbar)

Chapter 4 should introduce a few new level gimmicks.

[ ] #2102
[x] #2103

Chapter 4 should also have about 12-or-so levels. It already has 12 levels, but we'll want to double-check that its level goals are reasonable and leverage the two new gimmicks.

[ ] Level with short time goal
[ ] Level with long time goal
[ ] Level with low score goal
[ ] Level with high score goal
[ ] Level with low lines goal
[ ] Level with high lines goal
[ ] Level
[ ] Level
[ ] Level
[ ] Level
[ ] Level
[ ] Level
[ ] Boss level

#2102
We should eventually introduce a critter gimmick for chapter 4, but we didn't get to it in time. We can add this later once the game is functionally complete.


Email Notification for both Ticket owner and Assigned Techsupport

Still having hardtime with Events Trigger/Automation, is it possible to implement this one?
After creating and assigning a ticket to a Techsupport, both the ticket owner and assigned techsupport should received the email summary.
Also with the response/comments and ticket updates afterwards?

This should be possible, just make sure you configure smtp host and all that. Then create an insert trigger on the table with the send_email action. You can create multiple such triggers, one to send to the user in the row and went to a fixed email address

Thanks for the idea, maybe I'll just create another table that will be filled with both ticket owner and assigned techsupport agent after a ticket has been created or updated.


Add a sample code

Issue description

step is not easy to perform with a sample code
Feedback Type (Optional)
examples or samples
Page Title on SAP Help Portal (prefilled)
Overview
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/INTELLIGENT_SITUATION_AUT/64b367baf8b346b09702672666b0c0ae/4832a67a390d42ca9f876a1ded9ea84d.html?locale=en-US&state=PRODUCTION&version=1.0

we are reviewing and will update you soon


A new user perspective

Hi,
I started following your project in 2020 after seeing Tauri Fireside #1: History and Philosophy.

I'm writing this to offer you a perspective of what it feels like to get started with Tauri.
This type of feedback is kinda unusual, but I hope it's helpful to the Tauri's team.
Please note that I'm a system programmer, so maybe some of my problems/interrogations are due to my lack of web skills and not Tauri.

A journal of my questions and thoughts about Tauri, as a first-time user.
Installation

I need to install webview2 on Windows, does this imply my user will need to also install it to use my app?
The website is easy to navigate

I want to test Tauri without any other dependency, so I want to create a vanilla js project.

Why do I need NPM to create a project? Why not just cargo tauri init?
The CLI is nice and friendly.
What does the Add "@tauri-apps/api" npm package? (Y/n) options means?
I'm not sure what using "Tauri as a local package with npm" implies.

Exploring what files npm x create-tauri-app has produced:

Why is there a node_modules/ directory in my project? To my (limited) understanding of Webviews, I don't understand why this is needed.
Understanding the tauri.conf.json file seems critical, but the docs are not clear about what needs to be done to get started.
Why does Tauri use Rust 2018 version even if I have the 2021 edition installed?
Why is Tauri imposing formatting config with rustfmt.toml?
The fn main() is not bloated. It's great.

Trying the dev tools

In the section about cargo tauri dev, there's a mention of auto-reloading:

You can make changes to your web app, and if your tooling enables it, the Webview should update automatically just like a browser.
I'm using vanilla javascript and it doesn't seem to refresh even when I press F5.

I get this message every time I press F5: "Asset favicon.ico not found; fallback to index.html". I don't understand what is the "icons/" purpose.

Building the default project

cargo tauri build feels like magic. I'm impressed!
When invoking my compiled binary located at  .\src-tauri\target\release\test3.exe with a terminal, the app is working great.
I was surprised about the default config resources usage: test.exe is spawning 8 processes and taking 56 MiB.

This is a lot higher than I expected (bigger than Spotify!). Did I miss something?  My guess is that the answer lies in the  tauri.conf.json.

Modifying the project

I'm trying to add a custom rust command as per the documentation. It doesn't seem to work: I see nothing on the console.
In the webview console, I get the following message : "Uncaught SyntaxError: Cannot use import statement outside a module"
The example talks about setting withGlobalTauri to true; but for me, this creates a linker error.
Maybe I should of known this, but I needed to add the module type in the HTML: <script type="module" src="script.js"></script>.
Now I get "Uncaught SyntaxError: Identifier 'invoke' has already been declared".
Name collision maybe? Let's try commenting line 3 of the example// With the Tauri API npm package:
import { invoke } from '@tauri-apps/api/tauri'
// With the Tauri global script, enabled when `tauri.conf.json > build > withGlobalTauri` is set to true:
// const invoke = window.__TAURI__.invoke

// Invoke the command
invoke('my_custom_command')

Now I get "Uncaught TypeError: Failed to resolve module specifier "@tauri-apps/api/tauri""
I tried to modify the path but without success.
Yet, the file seems to exist<EMAIL_ADDRESS>

Even though my first experience had a bit of friction, I'm still excited about Tauri's future.
I think improving the documentation could help facilitate Tauri's adoption, but I recognize that good documentation is a lot to ask for an open-source project in beta.
Let me know if you have any questions and keep up your good work!
Best,
Gabriel

Thanks for participating in the discussion @Darunada.

Maybe this is better suited for the Discussion/Forum?

I'm writing this to offer you a perspective of what it feels like to get started with Tauri.
This type of feedback is kinda unusual, but I hope it's helpful to Tauri's team.

This type of feedback is so helpful. We aim to increase the DX as much as we can.

I need to install webview2 on Windows, does this imply my user will need to also install it to use my app?

Correct, but our bundler will generate an msi installer that will download and install it for users if they haven't done so already and it is pre-installed on Win11.

I want to test Tauri without any other dependency, so I want to create a vanilla js project.

Why do I need NPM to create a project? Why not just cargo tauri init?

you don't need NPM, you can do cargo install tauri-cli --version 1.0.0-beta.7 and do cargo init. I think we added that but maybe we missed a place or two, can you tell us where so you think this info should be added ?

The CLI is nice and friendly.

Was the questions clear enough or needs a better wording ?

What does the Add "@tauri-apps/api" npm package? (Y/n) options means?

This a npm package that provides the tauri JS api found here, it is mainly aimed to be used with a bundler like webpack and rollup but if you don't want to use it or don't have a bundler (vanilla js) then you can just use window.__TAURI__ to access all the apis but you'd need to enable tauri.conf.json >  build > withGlobalTauri: true.

I'm not sure what using "Tauri as a local package with npm" implies.

npm or any JS package managers can install packages globally so instead of doing npm run tauri dev, you'd just do tauri dev directly but we discourage that and it should be installed locally to the current project so different projects can have different versions of the cli.

Exploring what files npm x create-tauri-app has produced:

Why is there a node_modules/ directory in my project? To my (limited) understanding of Webviews, I don't understand why this is needed.

it has the JS dependencies of your app.
if you are going vanillajs you don't need it since you want to use  cargo tauri init instead of create-tauri-app as it is only used from JS package managers and we expect its users to be familiar with JS workflow and they don't want to use the tauri-cli directly through cargo and they want to use their favorite JS framework.

Understanding the tauri.conf.json file seems critical, but the docs are not clear about what needs to be done to get started.

the generated tauri.conf.json is good to go by default and don't need tweaking but you might wanna look at this

Why does Tauri use Rust 2018 version even if I have the 2021 edition installed?

do you mean our lib or the generated project that you will use? it doesn't matter as the next release will bump both to use 2021 edition but nothing is stopping you from changing it in the generated project now.

Why is Tauri imposing formatting config with rustfmt.toml?

Yeah some people brought this up before and I definitely think we should remove it.

The fn main() is not bloated. It's great.

That's possible because of the brilliant macros by @chippers @nklayman and @lucasfernog and initial spark by @parker-codes

In the section about cargo tauri dev, there's a mention of auto-reloading:

You can make changes to your web app, and if your tooling enables it, the Webview should update automatically just like a browser.

I'm using vanilla javascript and it doesn't seem to refresh even when I press F5.

If you are using a devServer which most JS frameworks have then you wouldn't even need to press F5 but if you are using vanillajs you need to restart the whole app, there is a feature for implementing hot reload for it directly in our cli so you won't need to install a separate devServer.

I get this message every time I press F5: "Asset favicon.ico not found; fallback to index.html".

your html, has a tag that tries to load a favicon. you can either remove it and you won't see the error. adding a favicon to your html in a Tauri app ia uselss anyway or just add a favicon in the distDir.

If the favicon is not found, what's the purpose of the "icons/" directory?

These are used for the app icon of the executable itself and window icon in the top left corner of your window.

I was surprised about the default config resources usage: test.exe is spawning 8 processes and taking 56 MiB. 😮

This is a lot higher than I expected (bigger than Spotify!). Did I miss something?  My guess is that the answer lies in the  tauri.conf.json.

This can depend on a lot of things but actually those memory usages for spotify and tauri app is incorrect, they should be both around 300mb, spotify has more processes that you didn't include here and tauri uses Webview2 which is chromuim based so the high memory usage is expected, unfortunately we can't control that.

Modifying the project

I'm trying to add a custom rust command as per the documentation. It doesn't seem to work: I see nothing on the console.

related to your next point.

In the webview console, I get the following message: "Uncaught SyntaxError: Cannot use import statement outside a module"

If you are using vanillajs then you need to enable withGlobalTauri and use window.__TAURI__ instead of importing from @tauri-apps/api npm package or use <script type="module" > but it will require you to download the npm package somehow and put inside your distDir.

The example talks about setting withGlobalTauri to true; but for me, this creates a linker error.

we would love to see the error and minimal repro if possible.

Maybe I should of known this, but I needed to add the module type in the HTML: <script type="module" src="script.js"></script>.
Now I get "Uncaught SyntaxError: Identifier 'invoke' has already been declared".
Name collision maybe? Let's try commenting line 3 of the example// With the Tauri API npm package:
import { invoke } from '@tauri-apps/api/tauri'
// With the Tauri global script, enabled when `tauri.conf.json > build > withGlobalTauri` is set to true:
// const invoke = window.__TAURI__.invoke

// Invoke the command
invoke('my_custom_command')

Now I get "Uncaught TypeError: Failed to resolve module specifier "@tauri-apps/api/tauri""
I tried to modify the path but without success.
Yet, the file seems to exist<EMAIL_ADDRESS>yeah you should only use one of them and not both. if vanilljs you need to go with window.__TAURI__ as it is less overhead. to use the import method you need to download the npm package manually and copy it to your distDir.

In the end,

🙂I was able to install Tauri and its dependencies (with some unanswered questions).

if you have any questions, don't hesitate to ask here or preferably in our discord.

🤔I was able to create an app (with a very large footprint though)

the final executable, memory usages or the project folder size ?

🙁I was not able to create a communication link between the webview frontend and the rust backend.

I guess that was because of the errors mentioned above due to your unfamiliarity with web development in general.

I think improving the documentation could help facilitate Tauri's adoption, but I recognize that good documentation is a lot to ask for an open-source project in beta.

we are aware of that and we will definitely improve it thanks to your feedback and other beta users.

Thanks for your thorough answer @amrbashir.

if you don't want to use it or don't have a bundler (vanilla js)  [...]

it has the JS dependencies of your app. If you are going vanillajs you don't need it [...]

If you are using a devServer which most JS frameworks have then you wouldn't even need to press F5 but if you are using vanillajs you need to restart the whole app,  [...]

If you are using vanillajs then you need to enable withGlobalTauri and use window.__TAURI__ instead of importing from @tauri-apps/api  [...]

It seems like a lot of my problem comes from using vanilla javascript. 😅
I recognize that, as a system programmer, I'm probably not the norm.
But here's why supporting vanilla js as a first class citizen is important to me:
Pedagogy:
When I want to learn a new tech, I like to experiment with it with as little dependencies as possible.
This is why, to me, vanilla js is my preferred option to learn Tauri.
Philosophy
Using the word "philosophy" is a bit much, but here's why/how I want to use Tauri:
I don't want to write a Webapp that runs on the desktop.
I want to build a Rust native app, that leverage Webviews as a GUI library.
This is why the minimal Tauri configuration (vanilla js) is important to me: I want to touch web frameworks as little as possible.

Thanks again for your answer.
I don't have a lot of time right now, but I will try Tauri again soon.
Also, if Tauri's team is interested, I'm willing to give you feedback when you update the documentation.
Cordialement,
-Gabriel

It seems like building the application selects which features in cargo.toml or maybe I'm just crazy. I think cli was autoremoved until I added a cli config into tauri.conf.json and then it was auto-added. Currently I'm using: tauri = { version = "1.0.0-beta.5", features = ["api-all", "cli", "window-all"] }

That is correct, the cargo features are added or removed depending on your tauri.conf.json


check for nonempty instead of null for link name on export

https://github.com/rancher/rancher/issues/6445

@alena1108 I feel like this PR should go green :)


Code Helper uses too much memory

Type: Bug
Code Helper (Renderer) Activity monitor shows CPU-heavy usage entire computer is frozen, and can't save a small .py file. Not sure how it's related, but can't use Crome either: Google Helper (Renderer) is frozen too, and consumed all memory available, can't even clear a trash bin. The system complains about memory when I have some Mb still available.  The screenshot shows on;y a few lines there are a bunch of the same lines of Helper, hope it makes sense, because it doesn't to me.

VS Code version: Code 1.74.2 (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:26:09.430Z)
OS version: Darwin x64 19.6.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz (8 x 1400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
3, 4, 6

Memory (System)
16.00GB (0.12GB free)

Process Argv
--crash-reporter-id ea2def40-eca0-4ac7-a2d8-16a99bf5e21a

Screen Reader
no

VM
0%

Extensions (21)

Extension
Author (truncated)
Version

vscode-color
ans
0.4.5

cssrem
cip
3.1.1

prettier-vscode
esb
10.1.0

terminal
for
0.0.10

live-sass
gle
6.1.1

language-cython
guy
0.0.7

isort
ms-
2023.10.1

python
ms-
2022.20.2

vscode-pylance
ms-
2023.5.30

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.14

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

sqltools
mtx
0.27.1

typescript-hero
rbb
3.0.0

LiveServer
rit
5.7.9

code-settings-sync
Sha
3.4.3

cython
tcw
0.1.0

bootstrap4-vscode
the
6.1.0

JavaScriptSnippets
xab
1.8.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30880072
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
89544117:30613380
2i9eh265:30646982
showlangstatbar:30737416
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxpt:30866567
pythonnoceb:30805159
dsvsc013:30795093
dsvsc014:30804076
dsvsc015:30845448
pythontestfixtcf:30871695
pythonregdiag2:30871582
pyreplss1:30886140
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30886110
dsvsc017:30886112
dsvsc018:30886114
aa_t_chat:30882232

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.84.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Use tempfile for temporary JSON files.

After merging #4125 , we generate a lot of temporary JSON files while running tests. These files are short-lived and are created and deleted with every test passing, although if the tests fail these files have to be deleted manually.
You can see the number of json files being generated when test suit fails many times :

We can use tempfile to generate temporary files to fix this. See : https://docs.python.org/3/library/tempfile.html

You can assign this to me, I will open a PR soon.

This is true for notebooks as well. They make images and other files, but do not clean them up

For the notebooks, I think it's better to not use tempfiles so as to not mislead newcomers to PyBaMM and Python, plus users might want to inspect the images after the notebook/cell has been executed


Feature/qkm deploy #628

PR Description

Add Quorum Key Manager v21.9.3
Remove old KeyManager
Update Ingress
Fix orchestrate Migrations

Fixed Issue(s)

Documentation

[ ] I thought about documentation and added the documentation label to this PR if updates are required.

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


creating a field named type shadows the json schema type

So I accidentally created a field in a new schema called type and when it's converted to JSONSchema it masks out the "type" field with it's own value. @smyrman What's the correct way to deal with this other than to throw an error if we detect it?

Never mind, I mis-read my output :)


Vision debug

This is a very small script which enables the debug output of the vision with dynamic reconfiguration. I think it is very useful because usually, the vision is running on a robot without debug output enabled.
I did not add any ".sh" stuff because I'm thinking about adding the scripts directory to the PATH variable which would allow us to call it and other scripts easily in the terminal.

Yes, i did.

Something I just remembered as well: Why is this not in the scripts folder inside bitbots_vision?

@NFiedler ping

This script should not be necessary, bit-bots/bitbots_vision#116 should be used instead.


feat: add discount details block

Background
Adding the UI extension block for discount details.
The extension target is named: admin.discount-details.function-metafields.render
Target is behind an app beta flag, extension_only_discounts_app
Fixes: https://github.com/Shopify/core-issues/issues/71151, https://github.com/Shopify/core-issues/issues/74963
Tech design for this project: https://docs.google.com/document/d/1r9mRFJi-cNz2GlbETPX53EMvlBgAMLLqbozws9DSBhw/edit#bookmark=id.kbfcs1ypqgus
Solution
Added based on the instructions found here: https://github.com/Shopify/app-ui/wiki/Creating-new-admin-extensions#introducing-new-extension-targets-in-admin
🎩
Checklist

[x] I have :tophat:'d these changes
[x] I have updated relevant documentation

Thanks for the context, @devisscher. I left a comment in a Slack thread where we were discussing target naming, but my TL;DR is that I think we should not introduce an additional segment here that we did not introduce for the validation UI extension target.

This extension gets it's data from the DiscountNode query which fetches a price rule. Said price rule can contain a metafield, which is passed to the extension and the developer can then allow the merchant to configure this metafield.

Can you describe specifically how the developer receives the data? For example, in the Validation UI extension, the developer receives a data object, with shopifyFunction and validation objects on it, where validation has an array of the first 20 metafields on that validation. Because this data is provided statically to the extension, we need to fully reload the extension when the resource is saved, so it is informed of its new metafields.
Is all of the above also true for this target, but with validation replaced by discount (or maybe discountNode, to match how you query those metafields in the Function itself).


Access to Gear

Any plans to allow to get the list of the gears for the activity?

I have implemented new GarminGear and GarminGearType records and GetGearTypes, GetUserGears and GetActivityGears methods. Also I added support for the "yyyy-MM-dd\THH:mm:ss.f" format and ""null" value used by Gears at DateTimeConverter.
I will contribute it as soon as I can.

Pull with gears created

@HSyr Thanks for the contribution!


Feat: add the refresh token to user info method

add refresh token to user info method to be utilized in function
not sure how to test this to be honest though but could looks good in my opinion
not sure, if i should tag you but just in case @thgh

No need to tag 😉

Thanks for the PR! Released in v2.2.0 🎉


feat: add buildUrl and jobUrl for Travis

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove deprecated 'report' function

This PR removes the deprecated report function in preparation of the 1.0.0 release.

Thanks @adrian-livefront