Diagrams don't load in IntelliJ 2022.3.x

Hi,
As the title says, while using IntelliJ 2022.3 the diagrams no longer load.
The "tabs" are still visible at the bottom:

the "ImageFileEditor" tab shows the generated picture when in "PNG" mode
the "Text" tab shows the proper text when in "XML" mode
the "diagrams.net editor" tab fails to load, regardless of the used mode

Thanks for reporting this. I've just published version 0.2.0 of the plugin which now builds on JDK17 which fixed the problem on IntelliJ 2022.3 for me.
Please give it a try.
The pre-release of the plugin is available from GitHub releases.
To try out a pre-release:

Download the new release file diagrams.net-intellij-plugin-xx.xx.xx.zip from the Assets section of the release.
Go to menu File | Settings... | Plugins | Gear Icon (⛭).
Choose Install plugin from disk....
Choose the downloaded file in the file selection dialogue.
Restart the IDE to activate the new plugin version.

Hi @ahus1 ,
Thank you so much for the prompt reply!
Sadly, however, the provided solution still has the same behaviour for me.
0.2.0 applied:

Hm, did you restart you IDE after the update? Could you please add a screenshot of your ide, so I see what you see?

Hi @ahus1 ,
Yes, I did restart.
You can find the screenshot below, left side shows the XML view (OK) and the right side shows the editor view (NOK). On the editor view, all I get is either the loading icon or a black screen altogether.

Hi again @ahus1 ,
So, meanwhile I noticed the exact same behaviour when using the ADOC plugin, in that the preview pane looked exactly as the issue described above, so I went to dig on that side of the github-verse and caught this, which exactly describes what is happening here:
https://github.com/asciidoctor/asciidoctor-intellij-plugin/issues/499
In this issue, which I guess you are aware of, there are some references to "OpenAPI" and "Swagger" related errors, and indeed my version of IntelliJ actually comes with an "OpenAPI Specifications" plugin bundled insided!
Incredibly, all I had to do was disabled that plugin and both the ADOC preview and the Diagrams.net editor reported here started working again, like magic!
Oddly enough, I was then able to re-enable the "OpenAPI Specification" plugin, and they still work.
Sadly, after these actions, it is the "OpenAPI" one that behaves a bit wonky, but in a much preferable manner than the other 2's wonkyness.
With this, can I safely conclude that there is some compatibility issue between these plugins?
Please note that I am using the "Ultimate Edition", and this "OpenAPI Specification" plugin is bundled by default there -- I've no clue if it is also bundled in the "Community Edition".
Hope this helps track down the issue!

There is now a pre-release 0.2.1 which uses "off-screen" rendering which seems to be more robust.
If the diagram doesn't show, please choose the action "Open Devtools Window for the current Diagrams.net Preview"

This dialog opens the Chromium developer tools, which should show for example a console. If something problematic happens in the preview, this would then hopefully show some additional log messages.

There are additional information on how to capture JCEF information, see https://intellij-support.jetbrains.com/hc/en-us/articles/360016644459-Reporting-JCEF-problems. Unfortunately I didn't find information to run IntelliJ with the console open on Windows: for me it always spawned in the background, so it didn't help me much. It might reveal more information on other operating systems.
The pre-release of the plugin is available from GitHub releases.
To try out a pre-release:

Download the new release file diagrams.net-intellij-plugin-xx.xx.xx.zip from the Assets section of the release.
Go to menu File | Settings... | Plugins | Gear Icon (⛭).
Choose Install plugin from disk....
Choose the downloaded file in the file selection dialogue.
Restart the IDE to activate the new plugin version.

Hi @ahus1 ,
Super sorry for the delay in replying, specially since you have been so proactive on this issue!
I ended up taking so long that 0.2.1 is already normally available by now, so I didn't have to use the zip.
Anyway, with this, the problems remains exactly the same.
Even a bit more unfortunate, since the upgrade caused a re-lapse to the original behaviour reported here, and the workaround of disabling the "OpenAPI Specification" plugin no longer works on this version.
I also tried opening the "JCEF DevTools" as you mentioned, with the "Open Devtools Window for the current Diagrams.net Preview" action, but the window that pops up from this is just blank as well (same as the diagrams and ADOC's and so on):

Rolled it back to 0.2.0 whcih works (with the workaround).

Hi @ahus1 ,
This still has the "awaiting feedback" tagged onto to it, not really sure how to remove it for you.
Anyways, found another interesting workaround for this issue, in case someone else is facing it, and this one seems to work regardless of the version used.
Just add "ide.browser.jcef.gpu.disable=true" to your IntelliJ custom properties!
Cheers

@blaghed - I've now removed "awaiting feedback".
Thanks for finding out about the GPU disable. Can you explain a bit more about your environment? Are you using a remote desktop / virtual desktop, and what operating system are you running? Thanks!
Looking for hints on the IntelliJ tracker, I found for example this issue: https://youtrack.jetbrains.com/issue/IDEA-243056

Hi @ahus1,

Are you using a remote desktop / virtual desktop

Yes, I am accessing it remotely via Citrix.

what operating system are you running?

Local system is running Windows 11, while remote system is using Windows 10.
Both Windows 11 and Citrix are already themselves the cause of much grief on my side, but I honestly did not expect this to be related to this particular one 🤣/😭

Installing JavaFX Runtime for Plugins worked for me

I've been experimenting on how to show a hint to users when the JCEF preview doesn't work for the AsciiDoc plugin. If that works, I could try something similar for this plugin.
Could someone using a remote desktop try this for me, that is: when the registry setting is not done, it would show the hint?
See https://github.com/asciidoctor/asciidoctor-intellij-plugin/issues/1371#issuecomment-1869610054 for instructions.

I didn't get feedback on my previous comment, so I'm closing this issue.
In the meantime, there might be another cuase: This might be similar to https://github.com/docToolchain/diagrams.net-intellij-plugin/issues/298. I added a possible workaround there. Please have a look at that issue.

Sorry for forgetting, @ahus1 .
This is now working without the workaround mentioned before, so can't check that the notification pops up, sorry.


Latest IntelliJ 2023.2 is unusable with this plugin

And soon as I install this plug-in, my intellij  2023.2 became completely unresponsive. In order to use my ID E I had to launch it using CLI with plug-ins deactivated, and had to delete the plug-in. As soon as i launch my iDE  after installing the plug-in, it get hangs indefinitely.

Thanks for reporting!

IntelliJ generateds threadumps when a freeze is detected. Could you open the log folder (by Help > Show logs in ...) and attach a zip of subfolders threadDumps-* located inside the log folder? This would help to find the cause of this problem.
If possible, idea.log and idea.1.log would also be helpful to find the cause. Please note that these files may contain local paths, please review before attaching them.
Could you also attach the data copied by Help > About > Copy and close, please?

Thank you for your help!

@santus444 thanks for reporting this issue. We take this kind of problem very seriously because we know people need their plugins to be non intrusive. I would really appreciate it if you would provide the diagnostic information that Joachim requests!

@jansorg / @kgilpin   sure. I can provide that. Can you please share any support email to which I can forward the files to? Want to avoid uploading log files here.

@santus444 Please send the data to<EMAIL_ADDRESS>Thanks!

@santus444 We've fixed a major issue, which could freeze the IDE, in 0.47. Could you try the latest version (0.48) and report if your problem is still showing up?

Thanks. I am able to launch ide now. It does not freeze the ide anymore.

@santus444 if you have those logs available, please send them along!

I can check to see if I still have them as I removed earlier version.

Thank you for your reply! We're glad that the issue is fixed now.

@santus444 Unless the IDE starts freezing again on you, we don't need the logs.


AuthFactory.binder() doesn't support multiple calls with different parameterized types

I have some @Auth providers I'm migrating to 0.8.0. Some of the injected Auth types are User and AdminUser. There's common code for authenticating the two. I'm running into problems when trying to bind multiple auth factories for different parameterized types.
Here's an artificial, yet runnable example:
package io.dropwizard.auth;

import com.codahale.metrics.MetricRegistry;
import com.google.common.base.Optional;
import io.dropwizard.auth.Auth;
import io.dropwizard.auth.AuthFactory;
import io.dropwizard.auth.AuthenticationException;
import io.dropwizard.auth.Authenticator;
import io.dropwizard.jersey.DropwizardResourceConfig;
import io.dropwizard.logging.LoggingFactory;
import javax.servlet.http.HttpServletRequest;
import javax.ws.rs.GET;
import javax.ws.rs.Path;
import javax.ws.rs.Produces;
import javax.ws.rs.core.Context;
import javax.ws.rs.core.MediaType;
import org.glassfish.jersey.servlet.ServletProperties;
import org.glassfish.jersey.test.DeploymentContext;
import org.glassfish.jersey.test.JerseyTest;
import org.glassfish.jersey.test.ServletDeploymentContext;
import org.glassfish.jersey.test.grizzly.GrizzlyWebTestContainerFactory;
import org.glassfish.jersey.test.spi.TestContainerException;
import org.glassfish.jersey.test.spi.TestContainerFactory;
import org.junit.Test;

import static org.assertj.core.api.Assertions.assertThat;

public class MultipleAuthFactoryTest extends JerseyTest {
  static {
    LoggingFactory.bootstrap();
  }

  @Override
  protected TestContainerFactory getTestContainerFactory()
      throws TestContainerException {
    return new GrizzlyWebTestContainerFactory();
  }

  @Override
  protected DeploymentContext configureDeployment() {
    return ServletDeploymentContext.builder(new BasicAuthTestResourceConfig())
        .initParam(ServletProperties.JAXRS_APPLICATION_CLASS, BasicAuthTestResourceConfig.class.getName())
        .build();
  }

  @Test
  public void transformsCredentialsToPrincipalsA() throws Exception {
    assertThat(target("/test/a").request()
        .header("Foo", "good-guy")
        .get(String.class))
        .isEqualTo("good-guy-a");
  }

  @Test
  public void transformsCredentialsToPrincipalsB() throws Exception {
    assertThat(target("/test/b").request()
        .header("Foo", "good-guy")
        .get(String.class))
        .isEqualTo("good-guy-b");
  }

  @Path("/test")
  @Produces(MediaType.TEXT_PLAIN)
  public static class ExampleResource {
    @Path("/a")
    @GET
    public String showA(@Auth UserA principal) {
      return principal.getName();
    }

    @Path("/b")
    @GET
    public String showB(@Auth UserB principal) {
      return principal.getName();
    }
  }

  public static class UserA {
    private final String name;

    public UserA(String name) {
      this.name = name;
    }

    public String getName() {
      return name;
    }
  }

  public static class UserB {
    private final String name;

    public UserB(String name) {
      this.name = name;
    }

    public String getName() {
      return name;
    }
  }

  public static class UserAuthFactory<U> extends AuthFactory<String, U> {
    private final Class<U> generatedClass;
    @Context HttpServletRequest request;

    public UserAuthFactory(Authenticator<String, U> authenticator, Class<U> generatedClass) {
      super(authenticator);
      this.generatedClass = generatedClass;
    }

    @Override public void setRequest(HttpServletRequest request) {
      this.request = request;
    }

    @Override public U provide() {
      String fooHeader = request.getHeader("Foo");
      if (fooHeader != null) {
        try {
          Optional<U> user = authenticator().authenticate(fooHeader);
          if (user.isPresent()) {
            return user.get();
          }
        } catch (AuthenticationException e) {
          throw new RuntimeException(e);
        }
      }

      return null;
    }

    @Override public AuthFactory<String, U> clone(boolean required) {
      return new UserAuthFactory<>(authenticator(), generatedClass);
    }

    @Override public Class<U> getGeneratedClass() {
      return generatedClass;
    }
  }

  public static class BasicAuthTestResourceConfig extends DropwizardResourceConfig {
    public BasicAuthTestResourceConfig() {
      super(true, new MetricRegistry());

      final Authenticator<String, UserA> authenticatorA = new Authenticator<String, UserA>() {
        @Override
        public Optional<UserA> authenticate(String credential) throws AuthenticationException {
          if ("good-guy".equals(credential)) {
            return Optional.of(new UserA("good-guy-a"));
          }
          return Optional.absent();
        }
      };

      final Authenticator<String, UserB> authenticatorB = new Authenticator<String, UserB>() {
        @Override
        public Optional<UserB> authenticate(String credential) throws AuthenticationException {
          if ("good-guy".equals(credential)) {
            return Optional.of(new UserB("good-guy-b"));
          }
          return Optional.absent();
        }
      };

      // Which ever factory is registered second, a 500 is generated
      register(AuthFactory.binder(new UserAuthFactory<>(authenticatorA, UserA.class)));
      register(AuthFactory.binder(new UserAuthFactory<>(authenticatorB, UserB.class)));
      register(ExampleResource.class);
    }
  }
}

If I register the UserA auth factory first, then the UserB test returns 500. If I switch the order, the UserA test returns 500.

@sul3n3t Did you ever find a workaround for this?  I'm also trying to migrate to 0.8.1, and this is the next thing I've run into.
I don't know how you did this in 0.7, but I had a MultiBasicAuthProvider facade that would delegate to a BasicAuthProvider with the correct injectable type.
It looks like for Dropwizard 0.8 we'd have to create our own alternate AuthFactoryProvider that does the same thing?

I think facade with delegation makes sense. Jersey won't let you register the same component twice. If you follow the second call to register, you will hit this.
https://jersey.java.net/project-info/2.16/jersey/jersey-common/xref/org/glassfish/jersey/model/internal/ComponentBag.html#393

@carlo-rtr The facade looks more difficult to write in the 0.8 world, so it might be nice if that delegation was built into the AuthFactoryProvider.  I looked at it a little, but I'm not familiar enough with the H2K injection stuff to know what is injected at what point, and where it would be safe to delegate.

After looking a little more closely, I think you can just create a new binder class to wrap the second factory. This will get it registered, though I haven't tested that it will work end to end. I have tested that it will register. Let me know if it works.
        private AuthFactory<T, U> factory;

        public MyBinder(AuthFactory<T, U> factory) {
            this.factory = factory;
        }

        @Override
        protected void configure() {
            bind(this.factory).to(AuthFactory.class);
            bind(AuthFactoryProvider.class).to(ValueFactoryProvider.class).in(Singleton.class);
            bind(AuthInjectionResolver.class).to(
                    new TypeLiteral<InjectionResolver<Auth>>() {
                    }
            ).in(Singleton.class);
        }
    }

@moxie0 I ended up replicating much of AuthBinder and AuthFactoryProvider. My AuthFactoryProvider.createValueFactory looks at the parameter's type and picks the correct AuthFactory. I register a few concrete classes with HK2 instead of binding objects to AuthFactory.class. It's all public and the best starting point is AuthResolver.
I intended my solution to be a hack and not totally general. Maybe there's a way to bind AuthFactorys with a qualifier and inject a collection of them to AuthFactoryProvider. Or, AuthBinder create a collection of AuthFactorys, bind the collection, and inject it into AuthFactoryProvider.

Any idea why AuthFactoryProvider is marked package-local? It will be so much easier to just extend this class and put your own providers.

@vpandey-rms If you're moving into Dropwizard 0.9.0 land, I packaged up a little library that does straight 0.8-style Auth annotations (no Roles), and allows you to register multiple principal types:
https://github.com/WhisperSystems/dropwizard-simpleauth


deleteDNSRecord failed with nil pointer reference

when a container starts and aborts immediately, a new DNS record is created every time but failed to delete. I believe the DNS record has not been saved to Route53 yet when the deleteDNSRecord is called.

There are several situations where the DNS record is not deleted (for example if the server fails), the Lambda function was created to solve these issues, it checks the DNS names every 5 minutes and delete the records if the service is not answering. You should configure the Lambda function in the same VPC.


Support extended regular expressions in pyzet grep

By default, git grep supports only basic POSIX regex which is very limited, so I don't think that this is a good default in such a use case.
git grep supports either extended POSIX or Perl-compatible regex.
Currently, this behavior can be forced by passing additional flags after --:
pyzet grep foo bar -- -E
pyzet grep foo bar -- -P

One of them should be the default, but two others are probably also nice to have, so we need some more sophisticated logic on pyzet side.
Probably I need to read more about differences between these two, and make this decision. Perl-compatible regex needs an optional Git compile-time dependency, so it may be a less portable solution.

I did more research, and it seems, that there's really no problem to solve here (this is for sed, but applies also to grep):

In GNU sed, the only difference between basic and extended regular expressions is in the behavior of a few special characters: ‘?’, ‘+’, parentheses, braces (‘{}’), and ‘|’.
With basic (BRE) syntax, these characters do not have special meaning unless prefixed with a backslash (‘\’); While with extended (ERE) syntax it is reversed: these characters are special unless they are prefixed with backslash (‘\’).

TBH, I prefer the BRE behavior, as I more often want to match some verbatim string than use these special regex operators
Reference:
https://www.gnu.org/software/sed/manual/html_node/BRE-vs-ERE.html


https://developer.mozilla.org/en-US/docs/Learn/JavaScript/Objects/Object_prototypes

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

Incorrect and confusing text on the English version of "Object prototypes" page:
"The former is the property on each instance, and the latter is the property on the constructor. That is, Object.getPrototypeOf(new Foobar()) refers to a different object than Foobar.prototype."
They actually refers to the same object.

https://developer.mozilla.org/en-US/docs/Learn/JavaScript/Objects/Object_prototypes

@chrisdavidmills This may seem familiar ...

@hamishwillee yup, I know we discussed this before...I just couldn't find the previous issue ;-)

@chrisdavidmills I was teasing :-) Duplicate-ish here https://github.com/mdn/content/issues/1491 and my whinge here https://github.com/mdn/content/pull/1890#issuecomment-770572970

This part of the article lacks a graphic illustration like this one:
http://dmitrysoshnikov.com/wp-content/uploads/2017/11/js-constructor.png (taken here http://dmitrysoshnikov.com/ecmascript/javascript-the-core-2nd-edition/).
A drawing like this can perfectly demonstrate the difference between proto and the .prototype constructor property, etc.

I believe this wording got fixed.


:memo: Add deployment note about Cloudflare pages

Per the discussion in jpanther/congo#266.
Signed-off-by: Major Hayden<EMAIL_ADDRESS>

Thanks, this is a great addition. I've just made some minor formatting corrections and will merge this in.

Thanks, @jpanther.


✨ Add Opentelemetry Auto-Instrumentation to Backend Services

This PR adds OpenTelemetry Auto-Instrumentation to:
Featbit Api
Featbit Evaluation Server
Featbit Data Analytics Server
It also fixes CVE-2021-26701 & CVE-2021-24112 and adds a docker compose file with observabilty services including Prometheus for Metrics, Jaeger for Traces, and Seq for Logs.
To test this from the root project folder execute the following.
docker compose --project-directory .  -f .\docker\composes\docker-compose-otel-collector-contrib.yml up
This will start Prometheus, Jaeger, Seq, and the OpenTelemetry Collector
Next, from the root project folder execute the following.
docker compose --project-directory .  -f .\docker\composes\docker-compose-dev.yml up --build
you can exclude --build on subsequent runs

Observe Telemetry messages displayed in the terminals used to run the docker compose commands
Use Featbit as normal
Observe Metrics with Prometheus at http://localhost:9090
Observe Traces with Jaeger at http://localhost:16686
Observe Log with Seq at http://localhost:8082

This pr leaves has console exporting enabled for the api and evaluation when running via docker-compose-dev.yml. This shows the data that is sent to the opentelemetry collector in the terminal that is running the api adn eval server. This can be disabled in the /infra/otel/env/otel-api.env and /infra/otel/env/otel-els.env by setting the following values to false in the respective files.
OTEL_DOTNET_AUTO_LOGS_CONSOLE_EXPORTER_ENABLED="false"
OTEL_DOTNET_AUTO_METRICS_CONSOLE_EXPORTER_ENABLED="false"
OTEL_DOTNET_AUTO_TRACES_CONSOLE_EXPORTER_ENABLED="false"

Hi @wss-rbrennan, sorry for the late review. I've left some comment for you.

And the otel integration for da-server seems doesn't work as expected
Jaeger traces for da-server

Seq logs for da-server

And the otel integration for da-server seems doesn't work as expected
Jaeger traces for da-server

Seq logs for da-server

I fixed this with commit # fd18ac

Hi @wss-rbrennan, thanks for your time on this PR. I've tested the changes, and everything looks good. However, I'm planning to make the OTEL integration configurable for both API and evaluation services as part of addressing issue #520. I will merge your PR into the feat/otel branch, and continue working from there.


Improve matching service crawler efficiency

currently the crawler starts for every won node at whole the need list and crawls all needs, connection containers and connections from there in every crawling cycle (current default every 5 hours)
an optimization would be to start from the connection list instead the need list. The connections list uri must be added to the WonNodeInfo then and the crawler must be adapted.
also timestamp/paging mechanisms could be used to request only newest connections and not crawl everything in every cycle.

for now do not crawl need and connection multiple times (even if they could have been updated) but only once by using "?resumeafter=" http query parameters and using last crawled need and connection ids as values


[Feat][Spark] Spark 3.3.x support as a Maven Profile

Proposed changes
As discussed in #320

support of spark-3.3.x was added as a separate Maven Profile
test were modified a little
matrix-strategy was added into CI/CD

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Further comments
Close #319

It fails on read, not write. So, updating FileSystem did not help.

@acezen I found the right way to fix it. Sorry, it was my fault. Please, check the latest commit.


CEDS documentation for SLCF

Add CEDS documentation for slcf and co2.

It would be great to augment this with DOI links, so that an interested user can help themselves, rather than having to ask more questions, or spend an hour doing a fruitless google search that could be avoided

doi links added


EmulatorHelper bad uninitialized register warnings

Describe the bug
Running the EmulatorHelper will produce wrong warnings when the function is preserving "unaffected" registers.
To Reproduce
Steps to reproduce the behavior:

Use the Emulator and step through instrutions which preserve "unaffected" registers.
See a warning WARN  Uninitialized register read at {Address}: {"unaffected" register}  (EmulatorHelper.java:704)

Expected behavior
The EmulatorHelper should not warn about uninitialized "unaffected" registers being stored on the stack.

Are you instantiating your own EmulatorHelper instance of using the processor test framework? It would be simple enough to add an option to the instantiation.

Is this intended for testing? I was actually trying to use this to emulate a function. It is producing a warning everytime a save register is preserved on the stack which doesn't need any initialization. Am I misusing this?

The warnings can be ignored.  If unexpected results occur the warnings can be useful in identifying the cause.  Just because the register is specified in the cspec as unaffected does not mean that it is not important to an operation.  There are cases where such a register must be properly initialized before emulation is started (e..g, MIPS gp register).  We live with these warnings all the time during our processor module testing and only pay attention to them when troubleshooting.
If you like you can set your own MemoryFaultHandler via the EmulatorHelper.setMemoryFaultHandler method.


A small amount of 1.20.2 mappings

EnnuiL:john_madden

@EnnuiL do you intend to fix this up, or should I take it over?

oh god i completely forgot about this pr; i have been quite busy recently, but i'll see if i can tackle it later

actually, please take it over


CoreML Model

Have y'all experimented with exporting the uform model (which is fantastic, by the way) as a CoreML model, so it can be run on-device more efficiently?

Hello. We didn't try to export the uform model as CoreML model, but if you did it, we would love to see

Ahh, okay. Is there currently a way to batch embed these images/texts at once as opposed to in individual function calls?

Yes. First of all, you need to make the batch of preprocessed texts and images:
batch_texts = model.preprocess_text(['cat', 'dog'])
images = [Image.open('cat.jpg'), Image.open('dog.jpg')]
batch_images = []

for image in images:
    batch_images.append(model.preprocess_image(image))

batch_images = torch.stack(batch_images, dim=0)

Then, you can use encode_text and encode_image:
text_embeddings = model.encode_text(batch_texts) # 2 x embedding_dim
image_embeddings = model.encode_text(batch_images) # 2 x embedding_dim

So preprocess_text can accept a list, and the preprocess_image can't? If so, @kimihailv should we change that?


Users don’t test their datasources before saving it, leading to saving invalid connections

https://www.notion.so/Users-don-t-test-their-datasources-before-saving-it-leading-to-saving-invalid-connections-190c46337bce45faa0e33a97a4d0982c

Duplicate of - https://app.zenhub.com/workspaces/activation-6604f423cc09d10016792707/issues/gh/appsmithorg/appsmith/32151


Fix build system and issues with newly generated client distributables

I've started fixing the build system, but the new build fails to properly set up the client environment in the browser.

This can probably wait until after the "hello world" deadline since the existing distributable seems to work fine.


Add hyphenation_decompounder token filter docs #8062

Description
Add hyphenation_decompounder token filter docs
Issues Resolved
Closes https://github.com/opensearch-project/documentation-website/issues/8062
Version
All
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@udabhas Will you review this PR for technical accuracy? Thank you.

@udabhas Will you review this PR for technical accuracy? Thank you.

@udabhas @varun-lodaya This is over a month old. We need tech review approval to move it forward in the documentation process. Please review this week or provide a peer who can review it. Thank you.


Block files types from Web

Hello,
Sorry but we don't use Onedrive or sharepoint, we've choosen another collaboration tool. Unforfunately one drive is the only way to use Onenote with Mac users, because of this, we cannot completely disable One Drive (Thanks Microsoft!!)
How do we block all file types from being saved into One Drive web or through the Word/Excel/Powerpoint direct connection? These instructions only apply to syncing which is not helpful.
I'd prefer One drive and sharepoint weren't enabled like this, not every company is going to use your product espicially with great alternatives like Box, Dropbox, Google Drive etc.

The only option you have is to disable synchronization of specific file types. There are no file type blocks in place which is controlled at the Web Application level, which only Microsoft has access to.

@afield1235 Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


ReactSwipeableView is resulting in errors with react 18

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
SwipeableDrawer is resulting in the following console error in react 18:
Warning: Using UNSAFE_componentWillReceiveProps in strict mode is not recommended and may indicate bugs in your code.

There is a related issue here, but it is getting no traction so far.
Expected behavior 🤔
No error with react 18.
Steps to reproduce 🕹
Steps:

Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Why is this the wrong repository for this issue?  react-swipeable-views is a dependency of MUI.  If that component is not working, and will not be updated, then MUI has to switch to use a different dependency, no?

Maybe we should remove the demos in the documentation that use react-swipeable-views. I wonder what are the alternatives to this project noawdays.

Apologies, I actually was confused between SwipeableViews and MUI's SwipeableDrawer (both of which I'm using in the same layout), and I mistakenly thought that this dependency was coming from SwipeableDrawer.
I agree with @oliviertassinari the issue here is simply that the docs should really point to a solution that is currently maintained.

Now, in oliviertassinari/react-swipeable-views#558, some were interested in taking over the maintenance. Maybe we can still find someone with a track record of great execution to be added as a maintainer.

The thing is, react-wipeable-views is still a very popular package.  react-responsive-carousel comes up in a google search, but it also abandoned.  Is there a good alternative that is being actively maintained?

I migrated to Swiper.  The footprint is larger than react-swipeable-views, but it has quite a large feature set.  It is also fully maintained, and seems to be extremely widely used.
I suggest the MUI docs get updated with an example using Swiper.  The canonical example would be roughly the same size and complexity as the current react-swipeable-views example.
I'm willing to do a PR.

@markedwards but Swiper is for carousels no? react-swipeable-views wasn't built it solve this problem.
I'm reopening so we can acknowledge that this demo is not OK: https://mui.com/material-ui/react-tabs/#full-width. We can either:

Replace it with a native CSS implementation. It don't see why it wouldn't work. It wasn't possible 7 years ago when I created react-swipeable-views.
Find another project.
Remove the demo. This UX pattern has almost disappeared on mobile apps, nowadays.

Considering that after all this time nobody from the community has shown interest in continuing the development of react-swipeable-views, this will help me archive the project.

Yeah it’s a bit of a different feature set, but you can do many of the same things.  I’m not in love with Swiper to be honest.  I’m using it to do a TikTok-style vertical feed, and it just barely gets the job done.  It uses weird translate3D tricks and wreaks havoc with z index.  And I’m pretty sure it can be done in a more performant way.

@markedwards but Swiper is for carousels no? react-swipeable-views wasn't built it solve this problem.
I'm reopening so we can acknowledge that this demo is not OK: https://mui.com/material-ui/react-tabs/#full-width. We can either:
1. Replace it with a native CSS implementation. I don't see why it wouldn't work. It wasn't possible 7 years ago when I created react-swipeable-views.

2. Find another project.

3. Remove the demo. This UX pattern has almost disappeared on mobile apps, nowadays.

Considering that after all this time nobody from the community, with some track record, has shown interest in taking ownership of react-swipeable-views to replace me, this will help me archive the project.

Any updates on this? The example is still there and will probably influence people to use a deprecated package

react-swipeable-views is using for animate the Tab transition, the transition is also made a great user experience to show the tab changes, and it is important in complex we apps with many tabs, and also allowing tabs to swiped, the carousels is generally used for slide shows.

@markedwards but Swiper is for carousels no? react-swipeable-views wasn't built it solve this problem.
I'm reopening so we can acknowledge that this demo is not OK: https://mui.com/material-ui/react-tabs/#full-width. We can either:
1. Replace it with a native CSS implementation. I don't see why it wouldn't work. It wasn't possible 7 years ago when I created react-swipeable-views.

2. Find another project.

3. Remove the demo. This UX pattern has almost disappeared on mobile apps, nowadays.

Considering that after all this time nobody from the community, with some track record, has shown interest in taking ownership of react-swipeable-views to replace me, this will help me archive the project.

Any updates on this? The example is still there and will probably influence people to use a deprecated package

@markedwards but Swiper is for carousels no? react-swipeable-views wasn't built it solve this problem.
I'm reopening so we can acknowledge that this demo is not OK: https://mui.com/material-ui/react-tabs/#full-width. We can either:
1. Replace it with a native CSS implementation. I don't see why it wouldn't work. It wasn't possible 7 years ago when I created react-swipeable-views.

2. Find another project.

3. Remove the demo. This UX pattern has almost disappeared on mobile apps, nowadays.

Considering that after all this time nobody from the community, with some track record, has shown interest in taking ownership of react-swipeable-views to replace me, this will help me archive the project.

Any updates on this? The example is still there and will probably influence people to use a deprecated package

Any recent progress / discussions on this? I’ve landed here as seen the swipeable tabs demo on the docs but never took it further once I noticed react-swipeable-views wasn’t maintained.


Make OpenCV software library available for Appium test runs

Describe the solution you'd like
OpenCV is a library based on machine learning which allows to run visual tests using Appium server. I installed the OpenCV locally using these instructions https://opencv-java-tutorials.readthedocs.io/en/latest/01-installing-opencv-for-java.html and now we want to implement visual tests to our projects.
Describe alternatives you've considered
Installing OpenCV on your system
Additional context
Our company would like to start executing such visual tests as soon as possible. We use AppCenter for running functional tests on real devices, so it would be perfect for us to continue using your device cloud also for such new type of testing :)

Thanks for the feature request @annabutenko. Tagging in @Oddj0b who can shed some more light on this feature request.

Thank you for your feature request. We normally prioritize feature requests based on popularity and hopefully, this one will get enough 👍


Introduce pex3 cache {dir,info,purge}.

Re-structure the Pex cache to both support versioning as well as adding
access tracking for shared (normal) use and for exclusive use when
portions of the cache need to be deleted. With this new ground work, add
a new pex3 cache {dir,info,purge} family of commands for inspecting
and safely trimming the Pex cache.
Closes #1176
Closes #1655
Closes #2201

Pants folks - I added you since you have: https://github.com/pantsbuild/pants/issues/11167
This initial introduction of Pex cache management commands does not have any JSON output option - its just for humans currently. The linked Pants issue seems to require structured information though. Much like Pants needing to come up with a representation it prefers for dependency graphs (I assume that's still not done), this integration point also needs similar spec'ing. If you have opinions, or better, a spec, I'd be happy to follow up with support for various alternate output formats for the cache usage information.

Some examples.
Get cache info sorted by size:
:; python -mpex.cli cache info -HS
Path: /home/jsirois/.cache/pex

Pex Docs: docs/0
Artifacts used in serving Pex docs via `pex --docs` and `pex3 docs`.
0 bytes in 0 subdirectories and 0 files.

Abbreviated Platforms: platforms/0
Information calculated about abbreviated platforms specified via `--platform`.
291 kB in 13 subdirectories and 26 files.

User Code: user_code/0
User code added to PEX files using `-D` / `--sources-directory`, `-P` / `--package` and `-M` / `--module`.
531 kB in 117 subdirectories and 168 files.

Packed Bootstraps: bootstrap_zips/0
PEX runtime bootstrap code, zipped up for `--layout packed` PEXes.
1.42 MB in 2 subdirectories and 4 files.

Unzipped PEXes: unzipped_pexes/0
The unzipped PEX files executed on this machine.
3.12 MB in 483 subdirectories and 1824 files.

Packed Wheels: packed_wheels/0
The same content as 'installed_wheels/0', but zipped up for `--layout packed` PEXes.
6.32 MB in 6 subdirectories and 12 files.

Interpreters: interpreters/0
Information about interpreters found on the system.
14.4 MB in 374 subdirectories and 704 files.

Bootstraps: bootstraps/0
PEX runtime bootstrap code.
19.5 MB in 125 subdirectories and 1657 files.

Pex Tools: tools/0
Caches for the various `PEX_TOOLS=1` / `pex-tools` subcommands.
21.4 MB in 254 subdirectories and 1886 files.

Built Wheels: built_wheels/0
Wheels built by Pex from resolved sdists when creating PEX files.
29.6 MB in 156 subdirectories and 59 files.

Isolated Pex Code: isolated/0
The Pex codebase isolated for internal use in subprocesses.
45.0 MB in 367 subdirectories and 3803 files.

Scie Tools: scies/0
Tools and caches used when building PEX scies via `--scie {eager,lazy}`.
151 MB in 5 subdirectories and 40 files.

Lock Artifact Downloads: downloads/0
Distributions downloaded when resolving from a Pex lock file.
236 MB in 379 subdirectories and 1980 files.

Pip Versions: pip/0
Isolated Pip caches and Pip PEXes Pex uses to resolve distributions.
324 MB in 3349 subdirectories and 17300 files.

Virtual Environments: venvs/0
Virtual environments generated at runtime for `--venv` mode PEXes.
583 MB in 9165 subdirectories and 40569 files.

Pre-installed Wheels: installed_wheels/0
Pre-installed wheel chroots used to both build PEXes and serve as runtime `sys.path` entries.
961 MB in 8975 subdirectories and 51739 files.

Total: 2.40 GB in 23770 subdirectories and 121771 files.

Dry run purge of just installed_wheels cache:
:; python -mpex.cli cache purge --entries installed_wheels -nRH
Would purge requested entries from /home/jsirois/.cache/pex: installed_wheels/0
Would also purge those entries transitive dependents in: unzipped_pexes/0, venvs/0

Would have purged cache Unzipped PEXes from unzipped_pexes/0
3.12 MB in 483 subdirectories and 1824 files.

Would have purged cache Pre-installed Wheels from installed_wheels/0
961 MB in 8975 subdirectories and 51739 files.

Would have purged cache Virtual Environments from venvs/0
583 MB in 9165 subdirectories and 40569 files.

Total: 1.55 GB in 18623 subdirectories and 94132 files.

And go for it (no psutil):
:; python -mpex.cli cache purge --entries installed_wheels -RH
Purging requested entries from /home/jsirois/.cache/pex: installed_wheels/0
Also purging those entries transitive dependents in: unzipped_pexes/0, venvs/0

Failed to import psutil: No module named 'psutil'
Will proceed with basic output.
---
Note: this process will block until all other running Pex processes have exited.
To get information on which processes these are, re-install Pex with the
management extra; e.g.: with requirement pex[management]

Attempting to acquire cache write lock (press CTRL-C to abort) ...
^C
No cache entries purged.

With psutil:
:; python -mpex.cli cache purge --entries installed_wheels -RH
Purging requested entries from /home/jsirois/.cache/pex: installed_wheels/0
Also purging those entries transitive dependents in: unzipped_pexes/0, venvs/0

Waiting on 2 in flight processes (with shared lock on /home/jsirois/.cache/pex/access.lck) to complete before deleting:
---
1. pid 281904 started by jsirois at 2024-09-02 15:45:24
   Pex env: {'PEX': '/home/jsirois/dev/pex-tool/pex/empty.pex'}
   cmdline: ['/home/jsirois/.pyenv/versions/3.11.9/bin/python3.11', '/home/jsirois/.cache/pex/unzipped_pexes/0/292f879052303680091fdcb445c2a746967b4e0f']
2. pid 282594 started by jsirois at 2024-09-02 15:45:50
   Pex env: {'PEX_TOOLS': '1', 'PEX': '/home/jsirois/dev/pex-tool/pex/empty-tools.pex'}
   cmdline: ['/home/jsirois/.pyenv/versions/3.11.9/bin/python3.11', '/home/jsirois/.cache/pex/unzipped_pexes/0/2c278c488639385dd1ca190c245fc4e8da7a0f30', 'repository', 'extract', '-f', '/tmp/find-links', '--serve']

Attempting to acquire cache write lock (press CTRL-C to abort) ...
^C
No cache entries purged.

And ending the processes with the shared lock:
:; python -mpex.cli cache purge --entries installed_wheels -RH
Purging requested entries from /home/jsirois/.cache/pex: installed_wheels/0
Also purging those entries transitive dependents in: unzipped_pexes/0, venvs/0

Waiting on 2 in flight processes (with shared lock on /home/jsirois/.cache/pex/access.lck) to complete before deleting:
---
1. pid 281904 started by jsirois at 2024-09-02 15:45:17
   Pex env: {'PEX': '/home/jsirois/dev/pex-tool/pex/empty.pex'}
   cmdline: ['/home/jsirois/.pyenv/versions/3.11.9/bin/python3.11', '/home/jsirois/.cache/pex/unzipped_pexes/0/292f879052303680091fdcb445c2a746967b4e0f']
2. pid 282594 started by jsirois at 2024-09-02 15:45:43
   Pex env: {'PEX_TOOLS': '1', 'PEX': '/home/jsirois/dev/pex-tool/pex/empty-tools.pex'}
   cmdline: ['/home/jsirois/.pyenv/versions/3.11.9/bin/python3.11', '/home/jsirois/.cache/pex/unzipped_pexes/0/2c278c488639385dd1ca190c245fc4e8da7a0f30', 'repository', 'extract', '-f', '/tmp/find-links', '--serve']

Attempting to acquire cache write lock (press CTRL-C to abort) ...

Purged cache Unzipped PEXes from unzipped_pexes/0
3.12 MB in 483 subdirectories and 1824 files.

Purged cache Pre-installed Wheels from installed_wheels/0
961 MB in 8975 subdirectories and 51739 files.

Purged cache Virtual Environments from venvs/0
583 MB in 9165 subdirectories and 40569 files.

Total: 1.55 GB in 18623 subdirectories and 94132 files.


Pea Popper Fidget Toy

Watch this video of a pea popper fidget toy! https://youtu.be/Dg4ogREEzVs

This is not an issue.
Issues here are specifically for problems with Kade Engine.

this guy straight up begging for views x3


Update dlp-conditions-and-exceptions.md

@chrfox - Can this be merged?

@chrfox Please merge this PR. Thanks


mdl: Don't enforce one space after list markers

Common markdown styles usually show 4-column indents to separate the
list marker and the list item text.  That's a common template for
writing new markdown files.
On the other hand, we do have some files (such as CHANGES.md) where we
use a different style.
From a markdown perspective, both are perfectly OK, and there's no
reason to enforce either.
Therefore, the best thing is to exclude this particular rule.

OK for master and 3.0. The reasoning behind including this in 3.0 is that if we cherry-pick things from master we should not need to change things which are not enforced on master.

Agreed

Ping, @openssl/committers

This pull request is ready to merge

Merged
master:
2046f80bc84deff5f3b8e3f8ffe54528bc684658 mdl: Don't enforce one space after list markers
3.0:
fa229d4e07267052adaa7a52281a1a1c1ab1f2f0 mdl: Don't enforce one space after list markers


what is the difference between the pixel values and the new pixel values

https://github.com/Zhen-Dong/Magic-Me/blob/16bf220e7f6319a8a5456e879c077f51c987413e/train.py#L357

we apply the masks on the original pixels. The masks are used to only keep the subject of the images.


fix: ensure image can be built on m1 macos

Description
When running any of the image build make targets on an M1 MacOS, it fails with the following error:
 => [builder 7/9] COPY . ./                                                                                0.2s
 => ERROR [builder 8/9] RUN go mod vendor                                                                                0.2s
------
 > [builder 8/9] RUN go mod vendor:
#12 0.148 qemu-x86_64: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory
------
executor failed running [/bin/sh -c go mod vendor]: exit code: 255

Specify platform to ensure build uses the correct architecture.
Solution provided is based on https://stackoverflow.com/questions/71040681/qemu-x86-64-could-not-open-lib64-ld-linux-x86-64-so-2-no-such-file-or-direc
Verification Steps

Run make image/build or make image/build/internal and ensure it builds successfully

Verify this on MacOS
Verity this on Linux

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

Maybe a cleaner way would be setting the envvar if the user running the Makefile is on a mac?

This could be achieved by doing something like (pseudocode, needs to be reviewed):
OS=$(uname)
if [ "$OS" = 'Darwin' ]; then
  export DOCKER_DEFAULT_PLATFORM ?= linux/amd64
fi

This could be achieved by doing something like (pseudocode, needs to be reviewed):
OS=$(uname)
if [ "$OS" = 'Darwin' ]; then
  export DOCKER_DEFAULT_PLATFORM ?= linux/amd64
fi

It should also be verified that when running in a non-mac OS that variable is not set on the docker build

As a side note, I wonder if the same issue happens if podman is used, and also whether podman makes use of that envvar too

This could be achieved by doing something like (pseudocode, needs to be reviewed):
OS=$(uname)
if [ "$OS" = 'Darwin' ]; then
  export DOCKER_DEFAULT_PLATFORM ?= linux/amd64
fi

It should also be verified that when running in a non-mac OS that variable is not set on the docker build

As a side note, I wonder if the same issue happens if podman is used, and also whether podman makes use of that envvar too

I tested the suggestion with podman and docker, it works with docker but unfortunately it doesn't work with podman, seems like they don't use that env var 😞.
However, the build cmd does have a --platform parameter for both which we could specify if the os is darwin as an alternative.
ifeq ($(shell uname -s | tr A-Z a-z), darwin)
	DOCKER_BUILD_PLATFORM ?= --platform linux/amd64
endif

...
# example build cmd
image/build:
        $(DOCKER) build $(DOCKER_BUILD_PLATFORM) ...

This could be achieved by doing something like (pseudocode, needs to be reviewed):
OS=$(uname)
if [ "$OS" = 'Darwin' ]; then
  export DOCKER_DEFAULT_PLATFORM ?= linux/amd64
fi

It should also be verified that when running in a non-mac OS that variable is not set on the docker build

As a side note, I wonder if the same issue happens if podman is used, and also whether podman makes use of that envvar too

I tested the suggestion with podman and docker, it works with docker but unfortunately it doesn't work with podman, seems like they don't use that env var disappointed.
However, the build cmd does have a --platform parameter for both which we could specify if the os is darwin as an alternative.
ifeq ($(shell uname -s | tr A-Z a-z), darwin)
	DOCKER_BUILD_PLATFORM ?= --platform linux/amd64
endif

...
# example build cmd
image/build:
        $(DOCKER) build $(DOCKER_BUILD_PLATFORM) ...

Let's do that. But let's called it CONTAINER_IMAGE_BUILD_PLATFORM instead, to make the name agnostic of the container engine being used


reduce bundle size by deduping dependencies

Previously, the bundle size was quite big due to duplicate dependencies between this repo and anol.
This MR sets aliases for the biggest dependencies so that webpack only bundles them once.
This reduces the parsed bundle size by ~6mb.

very nice!

@simonseyock added some changes - could you take another look?


Fix command arguments for Get Group Membership command

Problem
There should just be a list, no out-of-band length.
Proposed Solution
Fix XML, fix implementation.

@rcasallas-silabs Is this complete?


Automated tests

Playbook should have test

blocked by: https://github.com/ansibl8s/setup-kubernetes/issues/36

first build:
https://travis-ci.org/ansibl8s/setup-kubernetes/builds/100156256


feat(deps): bump rspack to 0.5.7

Summary
Related Links

Checklist

[ ] Tests updated.
[ ] Documentation updated.

Thanks! ❤


Support relative root_map.

This is needed for databroker-pack. Needs tests.

ok, so we now know there is a bug in here, I'm inclined to merge this as is to unblock databroker-pack and then sort out why updating this is changing the return value of get_config().


Handler Step

In order to get this to work I needed to name the "Handler" to "main.lambda_handler" to match main.py.  Hope that helps someone!

Thanks for this.  I had to make this one change too.


Merge pull request #1 from spf13/master

update

 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.


Add Theming

Before you start

[x] Have you checked if a similar PR has already been requested?
[ ] Have you built and ran the app?

Related/Fixed issues
Address related/fixed issues

Related: #158

Description
Details of changes are listed here
Adds the ability for theming and adds some of @DeveloperWOW64's themes already made for Files.

@onein528 I've added a test theme - Nightshade.xaml - and added it to the csproj so all we need to do is add a drop down menu like in Files for the themes in the Appearance section - do you mind doing this?
This might help: https://github.com/files-community/Files/blob/main/src/Files.Uwp/Views/SettingsPages/Appearance.xaml

Nice! I will take a look!


Spine plugin: ScaleX and ScaleY of attachments

One more fix for spine region attachments, continuation of 5e96a5fad5f8a90eddf5a7a5715d91d9a5b577be

Thanks @ivanpopelyshev ! :+1:


state_store_write_batch_tuple_counts metrics missed for 40s

Describe the bug
This is the first time we hit this issue in the daily performance test. Maybe it's not easy to reproduce this issue. I filed this issue to record the bug in case we hit this again in the future.
The data from Prometheus shows that there are 40 seconds without this metrics data. The scrape interval is 5 seconds. This metrics is empty between [1680452011.803,\"8161\"],[1680452051.804,\"8161\"]
[1680452001.803,\"8161\"],[1680452006.803,\"8161\"],[1680452011.803,\"8161\"],[1680452051.804,\"8161\"],[1680452056.804,\"8161\"]

The promql is state_store_write_batch_tuple_counts{namespace="nexmark-bs-0-1-2-3-4-daily-20230402"}[24h].
To Reproduce
Run nexmark q0 and q1.
The buildkite pipeline job
Expected behavior
The metrics data interval should be 5 seconds.
Additional context

Did it ever happen again?

Closed since never happen again


Token Authentication Provider

This commit introduces a new authentication provider for doing Token
Authentication via the CLI.  It also introduces a split in how auth
providers are listed, so that admins can configure certain auth
providers to only work via the Web UI or the CLI.

In thinking more about this "two configs for the price of one" setup, the weirdness just comes out the other end, because now I have to register two OAuth applications (one for web / one for cli) to make things work.  Either that, or we do the "mirroring" when we read the config, but that leads to confusion as to why the doubling up.
Thoughts?

Sure, you have to register two OAuth apps, but you already need to do that. I guess the problem here is just the messaging to the user as to why they need to register twice? The alternative is changing the  way that we handle OAuth such that it can work with both UIs from one logical client.

After much discussion, I think we have a way forward that doesn't involve this PR:

Modify the Provider Initiator URLs to be able to differentiate between CLI / Web users (via ?web)
Have the Initiators set a cookie indicating which mode the user entered the flow in (web or cli)
Read (and unset) the cookie on the Provider Redirect URLs, and react appropriately:

Web Users will still get redirected with a Set-Cookie header
CLI Users will see a prett-ified, SHIELD-branded page that shows them the session ID for copying / pasting into the waiting terminal (or we use the local port hack)

This Token Provider will continue to be CLI only, but that disposition will be in code, not in configuration.

This approach requires neither split configuration, nor separate OAuth2 application registrations with the 3rd party systems, so it's a win-win!


[Bug]: Theme: Cannot run shopify theme dev for one particular store

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Theme
Expected behavior
shopify theme dev --store=asd works
Actual behavior
shopify theme dev --store=asd does not work
Stack trace
Running `shopify theme dev --store=asd` output
┏━━ Viewing theme… ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
┃ * Syncing theme #xxxxx on asd.myshopify.com
┃                                                                                100%
┃
┃ Serving .
┃
┃ Please open this URL in your browser:
┃ http://<IP_ADDRESS>:9292
┃
┃ Customize this theme in the Theme Editor, and use 'theme pull' to get the changes:
┃ https://asd.myshopify.com/admin/themes/xxxxxx/editor
┃
┃ Share this theme preview:
┃ https://asd.myshopify.com/?preview_theme_id=xxxxxx
┃
┃ (Use Ctrl-C to stop)
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ (14.54s) ━━
15:24:50 Synced » update assets/Comfortaa-Regular.woff
15:24:51 Synced » update assets/Comfortaa-Regular.woff2
15:24:51 Synced » update assets/Logo-white.svg
15:24:52 Synced » update assets/Logo.svg
15:24:54 Synced » update assets/Nunito-Bold.ttf
15:24:55 Synced » update assets/Nunito-ExtraBold.ttf
15:24:56 Synced » update assets/Nunito-Regular.ttf
15:24:57 Synced » update assets/Nunito-SemiBold.ttf
15:24:57 Synced » update assets/ajax-loader.gif
15:24:58 Synced » update assets/android-chrome-192x192.png
15:24:59 Synced » update assets/android-chrome-512x512.png
15:25:00 Synced » update assets/apple-touch-icon.png
15:25:02 Synced » update assets/da.png
15:25:03 Synced » update assets/favicon-16x16.png
15:25:04 Synced » update assets/favicon-32x32.png
15:25:04 Synced » update assets/fi.png
15:25:05 Synced » update assets/hero_slide_1.jpg
15:25:06 Synced » update assets/icons.eot
15:25:07 Synced » update assets/icons.svg
15:25:07 Synced » update assets/icons.ttf
15:25:09 Synced » update assets/icons.woff
15:25:11 Synced » update assets/img-bottom.png
15:25:12 Synced » update assets/img-top.png
15:25:12 Synced » update assets/info-circle-white.svg
15:25:14 Synced » update assets/lazysizes.min.js
15:25:14 Synced » update assets/jquery-2.2.3.min.js
15:25:15 Synced » update assets/modernizr.min.js
15:25:16 Synced » update assets/made-in-germany.png
15:25:17 Synced » update assets/nl.png
15:25:18 Synced » update assets/password-page-background.jpg
15:25:19 Synced » update assets/pinpoint.png
15:25:19 Synced » update assets/pl.png
15:25:20 Synced » update assets/sv.png
15:25:21 Synced » update assets/tlab-flags-da.png
15:25:21 Synced » update assets/tlab-flags-de.png
15:25:23 Synced » update assets/tlab-flags-fr.png
15:25:23 Synced » update assets/tlab-flags-fi.png
15:25:24 Synced » update assets/tlab-flags-nl.png
15:25:25 Synced » update assets/tlab-flags-pl.png
15:25:26 Synced » update assets/tlab-flags-pt-PT.png
15:25:27 Synced » update assets/tlab-flags-sv.png
15:25:28 Synced » update assets/tlab-flags-tr.png
15:25:28 Synced » update assets/translation-lab-language-switcher-floating.css
15:25:29 Synced » update assets/zendesk-embeddable.js

Everything looks good except that it doesn't work. :)
Couple of extra info:

It was working yesterday
I ran shopify theme dev (or shopify theme dev --store=some-other-store) by mistake in another project root and it started uploading theme files to "asd" store.
I deleted the previous development themes and that created a new development theme, but the page is still not working.
I tried in different browsers

Here's a screenshot

How can I investigate this? What can I do? I'm blocked and cannot do theme development now.
Reproduction steps

shopify theme dev --store=asd

Operating System
macOS Monterey 12.6
Shopify CLI version (check your project's package.json if you're not sure)
3.22.1
Shell
No response
Node version (run node -v if you're not sure)
v19.0.1
What language and version are you using in your application?
No response

@vfonic there is a --verbose flag you can use

Thank you @shauns!
I ran shopify theme dev -s xxx --verbose and saw that CLI is getting 500 response so I deleted all session/cookies for <IP_ADDRESS>:9292 and it works now! No idea why I didn't try removing application/site data before.
Here's a bit of info that might be useful:
DEBUG Refreshing preview _secure_session_id cookie
DEBUG Proxying HEAD https://xxx.myshopify.com/?preview_theme_id=133577244940&_fd=0&pb=0
DEBUG `-> 302 request_id: 7e122528-b474-4776-acb0-f453eebf68f0
DEBUG Proxying GET https://xxx.myshopify.com/?_fd=0&pb=0
DEBUG `-> 500 request_id: 2d5dd8f9-1fbe-4a39-abfb-578bca584e3f

The myshopify url is changed, but theme ID and request ID are correct.

Hi @vfonic, glad to hear you were able to resolve your issue! To confirm how you got into this state I wanted to clarify this step you listed:

I ran shopify theme dev (or shopify theme dev --store=some-other-store) by mistake in another project root and it started uploading theme files to "asd" store.

You ran shopify theme dev --store=some-other-store, the CLI didn't listen to your --store param and it started uploading to asd.myshopify.com?
Thanks!

No, the --store param was respected, but I ran the command in the wrong project root.
So it was one of these two scenarios:

/project_a > shopify theme dev --store=project_a
Ctrl^C
/project_a > cd ../project_b
/project_b > shopify theme dev

/project_b > shopify theme dev --store=project_a

Closing this issue because it's resolved, thanks again for reporting!


filebrowser icon does not work

The filebrowser icon does not work.
Used code:
module "filebrowser" {
  source        = "registry.coder.com/modules/filebrowser/coder"
  version       = "~>1.0.23"
  agent_id      = coder_agent.main.id
  agent_name    = "main"
  subdomain     = false
  folder        = "/home/coder"
  database_path = ".config/filebrowser.db"
  order         = 3
}

Is this related to that PR that updated or changed the filebrowser icon in the coder/coder repository?

Hi @gjrtimmer this is not an issue with module. The module icon was updated to use a builtin path in https://github.com/coder/modules/commit/f5ab7995d145971d25766ba826901b7eac4c5227
that did not exist and broke the icon display. It was fixed in https://github.com/coder/coder/commit/5a6d23a4a3a2225e3440f4a2dd83b3fc1fdffaa4 and is availble as per v2.17.0 and later versions of Coder.


update value handling in select component, apply trim to text and tex…

What this does

refactors the onChange event in the Select component to use the value property on the prop options instead of using the value on the input (allows for number support in options)
applies the trim method to Textarea and BaseInput to drop the need of v-model.trim modifiers

Notes
After looking closely at the HTMLSelectElement spec, it appears there is a way to determine which option was selected via an index value that can be mapped back to the options set on the component props.  This allows us to read the exact value set on the option allowing for both string and number support, technically even mixed uses.  The only situation not supported is mixed types where the string value is the same, ex: '3' and 3.  This potentially could be handled as well if we handle setting the selected attribute ourselves, but it seems unlikely to be necessary and may even be better classified as something we shouldn't do at all.  If a valid cases for it arises, I've got some ideas though.
Looking through Portal and Archive the use of the trim modifier is pretty heavy.  We've had that baked into. FormDisplay since the start, so it seems like a very useful default handling for those input types.

@profsmallpine @seedifferently I'll leave this open in case anyone wants a glance in the next day or two.  Feel free to merge it if you want to move forward with it on Archive.  See #1388 for the tweaks to FormDisplay, though that update shouldn't be required to get the benefits here.


Verify that "Gura Cornei" is listed in the dropdown menu when searched

Description:
"Gura Cornei" should appear in the dropdown menu with correct spelling and locality information.
Preconditions:
The website is up and running.

Step 1
Type "Gura Cornei" in the search bar.
Expected results:
The dropdown is listing "Gura Cornei" as the first location in the dropdown menu.

Step 2
Observe the dropdown menu.
Expected results:
"Gura Cornei" is displayed and selectable from the dropdown menu.


nrf52840_pca10090 reset pin & HCI

Opened one PR for convenience.

add a GPIO reset pin to nrf52840_pca10090
update H4 driver to use the GPIO reset pin with nrf52840_pca10090 controllers
update samples/bluetooth/hci_uart config for nrf52840_pca10090 to enable the GPIO reset and NOP

@carlescufi
Addresses: #15334.

Is this addressing #15334?

@galak yes, sorry I forgot to add it to the description, added now.

@jhedberg comments addressed, thanks for the review and innput!


Starting Firefox portable when another Firefox (regular install) is running just opens firefox.exe

Steps to reproduce:

Install Firefox from Mozilla (the default way)
Open a Firefox window
Install Portapps Firefox
Start firefox-portable

Observe how it just forks off another instance of the running firefox.exe.
I can only start firefox portable by closing all "regular" firefox instances.
It works the other way around too, by the way:

Open firefox-portable ->firefox portable window opens
Open mozilla-installed firefox.exe -> new firefox portable window opens

I've tried playing with the AllowMultipleInstances setting, but that does not change anything, unfortunately.

Hi @Forceflow, thanks for your report, I will take a look on this.

I think the process just checks for a running firefox.exe (regardless of origin), and requests a new window from that process. Renaming the executable probably fixes this.

@Forceflow That seems to be it, indeed. I'll try to find a way to get around this behavior.

@Forceflow Unfortunately I cannot reproduce your issue. Here is the scenario I realized:

Install Firefox from Mozilla (FirefoxSetup-69.0.3-win64.exe)
Install Firefox portable to C:\portapps\firefox-portable (firefox-portable-win64-69.0.2-26-setup.exe)
Launch Mozilla Firefox
Launch Firefox Portable

Everything works as expected. I've got 2 separated instances:

Tried the other way around too and it's ok for me. I've also tried to open a new window and everything is linked as expected.
Can you give me your Firefox (mozilla/portable) version and architecture (32/64). And also create a proper bug report please? Thanks.


Additional redis options

Hi all!
While setting up redisdown with Nx I discovered that some options are not passed on to node-redis.
The options look to be restricted to these options: https://github.com/hmalphettes/redisdown/blob/master/index.js#L264
For the connection to a password-protected cache, we also need the tls and password option.
Is it safe/correct to add these options or maybe even to remove the filtering altogether such that all options are passed on to node-redis?

Great questions.  I will have to take a look and try to understand what the original author had in mind with those restrictions and then see what it would take to pass at least the ones that you are interested in, on.  If you have ideas, please add a patch file or a PR with some tests to expedite the investigation. Thanks!

I see what you are seeing.  It looks as if the original author overloaded the meaning of _makeRedisId with option cleansing as well. _makeRedisId is used to cache the connections, which is valid to use a subset.  I will see if there is some reason to NOT pass on other options as necessary to make the client connection to Redis.

I don't see a reason why not to..so please go ahead and use it that way.  I am working on getting the correct changes in and updating some testing.


Request for Virus Detection Verification on MHC2Gen.zip Download

I attempted to download MHC2Gen.zip, which was released on June 10th, but the system detected a virus and refused to download it. Could you please help verify if this is a false positive by the antivirus software? Thank you!

You can paste the link to virustotal.com and check what other security vendors think about it.

You can paste the link to virustotal.com and check what other security vendors think about it.

Thank you for the information. After using an online virus scanning service, it appears that there's no issue; the virus alert was a false positive. I have already submitted the ZIP file to the Microsoft Defender Antivirus false positive reporting system online.


housekeeping: update xunit to latest version

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Housekeeping
What is the current behavior? (You can also link to an open issue here)
Older version of xunit used.
What is the new behavior (if this is a feature change)?
Newer version of xunit, includes some analysers etc.
What might this PR break?
Slight possibility of unit tests being broken, have run locally, and want to run in CI to verify.
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

Changes Unknown when pulling 130db58f58329519e43a2434555c7f5418b74613 on update-xunit into ** on develop**.

ty Olly


Fatal error: uncaught error – can some help me to solve this issue?

Fatal error: Uncaught Error: Call to undefined function get_field() in /home/.sites/869/site4366094/web/wavelife/wp-content/themes/proformat/functions.php:340 Stack trace: #0 /home/.sites/869/site4366094/web/wavelife/wp-settings.php(479): include() #1 /home/.sites/869/site4366094/web/wavelife/wp-config.php(90): require_once('/home/.sites/86...') #2 /home/.sites/869/site4366094/web/wavelife/wp-load.php(37): require_once('/home/.sites/86...') #3 /home/.sites/869/site4366094/web/wavelife/wp-blog-header.php(13): require_once('/home/.sites/86...') #4 /home/.sites/869/site4366094/web/wavelife/index.php(17): require('/home/.sites/86...') #5 {main} thrown in /home/.sites/869/site4366094/web/wavelife/wp-content/themes/proformat/functions.php on line 340

Do you have Advanced Custom Fields plugin installed and activated?


Add new PHP blog post

Summary

#34
Use Cases

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Do we have a localization preference or mechanism for these docs? I notice use of "favour" and "behaviour" instead of "favor" and "behavior".

@joshuatcasey using "-our" is just a habit from my childhood. I don't know if it hugely matters in our blog posts but I can change it if you feel strongly. :D

Release 5/12


Handle MultiPolygon geometries from tileInfo.json data geometry

The STAC geometry is transformed from the tileDataGeometry object in tileInfo.json, which is almost always of Polygon type.
Looks like there are maybe up to 4000 scenes (out of 11.6 million) that have MultiPolygons, which the spatial transform is not expecting.

Moved into a JIRA ticket.


Unable to create project with selected template ( v. 2.6.0)

`success Installed<EMAIL_ADDRESS>with binaries:
- create-tauri-app
✔ Choose your package manager · cargo
error: the svelte template is not suppported for the cargo package manager
possible templates for cargo are: [vanilla, yew]
error Command failed.
Exit code: 1
`

Svelte doesn't work with Cargo as the package manager as described in the error, you have to choose another package manager if you want to use Svelte or use another template that works with cargo, such as vanilla and yew.
I think you invoked the cli like this cargo create-tauri-app --template svelte and this is the second time someone has went this same path so I think I need to only prompt the supported package manager for the template.


Allow public access to the generated __{Operation}Input variables

Is your feature request related to a problem? Please describe.
I'd like to hook into the Marshal/UnMarshal methods on the __{Operation}Input struct definition to facilitate unit testing.
We're using httpmock and making assertions on what is being sent in the body of the request and mocking the data to be returned.
Since the request is generated with custom json marshalers from genqlient, there doesn't seem to be a good way to test that the body of the request is as expected without either writing out a []byte{"json":"here"} definition or creating a duplicate struct type that marshals similarly.
Describe the solution you'd like
I'd like to add a configuration option to remove the prepended __ in the code here
For example, given something like this in generated.go

// __AddCustomInput is used internally by genqlient
type __AddCustomInput struct {
	ID           string     `json:"ID"`
	Title        string     `json:"title"`
	Quantity     int        `json:"quantity"`
	Price        MoneyInput `json:"price"`
}

I'd like to declare the following in a unit test

expectedReq := graphql.Request{
	OpName: "AddCustom",
	Query:  AddCustom_Operation,
	Variables: &__AddCustomInput{ // I would want this to be publicly accessible
		ID: ID,
		Title:        title,
		Quantity:     quantity,
		Price:        price,
	},
}

reqBytes, _ := json.Marshal(expectedReq) // I want this to marshal using the Marshal method on __AddCustomInput
httpMock.Handler.On(
	"other",
	"fields",
	"to",
	"match on",
	[]bytes{reqBytes}, // so that this matches what genqlient would do internally
).Return(httpmock.Response{
	Status: 200,
	Header: http.Header{},
	Body: []byte(expectedRes),
})

Describe alternatives you've considered
I tried using mockery and testing against the MakeRequest call, but it runs into the same issue with the private __{Operation}Input type
Additional context
Looking in the example integration tests, it doesn't seem like the request variables are being inspected as noted here and in the test here with resolver here

These types are intentionally not public, because their structure may change over time; the only guarantee is the JSON over the wire. I'd suggest testing that! (The input types are probably a bit more stable than the premarshal types, but when you get into cases where the input type has a custom marshaler it's still nontrivial.)
(Of course you put your code in the same package as the generated code, nothing stops you from using these. But they should, as the name and documentation imply, nonetheless be treated as unstable across genqlient versions.)
There are also some notes on testing in the docs if helpful, feel free to suggest any updates.


Conditional routing in meteorkitchen

Hi - I hope I can get a tip on the best/prettiest way to implement this.
I have a dataview showing a collection.
It is linked up a details page using "Details route".
However, the dataview can display data of different types. And depending on the type of the element, I would like to go to different details pages.
I am able to set a route parameter "type" so I get a dynamically created page link:
collection.details/id=xyz&type=standard
I guess there are two solutions:

Go to the same details page for all types. Dynamically change the layout of this page based on the type parameter.
Go to different details pages based on the type parameter.

I cannot see that I am easily able to do either of the above.
Any advice on the best way forward would be much appreciated.

I guess if I could somehow override the OnBeforeAction hook for routes, then I could create my own routing to subpages of details. But that doesn't seem to be possible.

Hi,
What about using a custom template using a template helper ?
In the template
{{#if displayStandard}}
  <div id="standard">
    Standard display
  </div>
{{/if}}

julienrott, thanks - but if I do that it doesn't really play together with Meteorkitchen anymore, it seems.
I was thinking of an alternative solution, actually - where I could override the click event on the dataview and then handle the routing on my own to the correct dataview.details subpage.
That seems like a solution that plays relatively well together with Meteorkitchen.

Custom templates are a Meteorkitchen feature, what do you mean with "it doesn't really play together with Meteorkitchen anymore" ?
If you mean the save/cancel/back buttons or the automatic form generation, I think it is possible to reuse or inject the Meteorkitchen templates in the custom template.

Sorry, it is probably due to the fact that I don't have much experience with these. Thanks in any case, I'll look into it and see what's the best option

@xauxatz you can use page's controller_before_action. Add code to execute before route renders:
"controller_before_action": "switch(this.params.type) { case: \"standard\": Router.go(\"somewhere\", { id: this.params.id, type: this.params.type); break; default: this.next(); }"

This will result in onBeforeAction in your controller:
...
onBeforeAction: function() {
  switch(this.params.type) {
    case: "standard": Router.go("somewhere", { id: this.params.id, type: this.params.type }); break;
    case: "cool_type": Router.go("whatever" .......... ; break;
    default: this.next();
  }
}
...

This would be an exceptionally nice way to do it, except that it doesn't
seem to work.
Any code I put inside "controller_before_action" does not end up in the
finalized code after meteorkitchen processing :-(
Will it be fixed in the next release?
On 1/6/2016 12:56 PM, Petar Korponaić wrote:

@xauxatz https://github.com/xauxatz you can use page's
|controller_before_action|. Add code to execute before route renders:
|"controller_before_action": "switch(this.params.type) { case:
"standard": Router.go("somewhere", { id: this.params.id, type:
this.params.type); break; default: this.next(); }" |
This will result in onBeforeAction in your controller:
|... onBeforeAction: function() { switch(this.params.type) { case:
"standard": Router.go("somewhere", { id: this.params.id, type:
this.params.type }); break; case: "cool_type": Router.go("whatever"
.......... ; break; default: this.next(); } } ... |
—
Reply to this email directly or view it on GitHub
https://github.com/perak/kitchen-site/issues/205#issuecomment-169307784.

Hmmmm, it seems to work now ... Very strange.
I'll look into it and report back.
On 1/6/2016 12:56 PM, Petar Korponaić wrote:

@xauxatz https://github.com/xauxatz you can use page's
|controller_before_action|. Add code to execute before route renders:
|"controller_before_action": "switch(this.params.type) { case:
"standard": Router.go("somewhere", { id: this.params.id, type:
this.params.type); break; default: this.next(); }" |
This will result in onBeforeAction in your controller:
|... onBeforeAction: function() { switch(this.params.type) { case:
"standard": Router.go("somewhere", { id: this.params.id, type:
this.params.type }); break; case: "cool_type": Router.go("whatever"
.......... ; break; default: this.next(); } } ... |
—
Reply to this email directly or view it on GitHub
https://github.com/perak/kitchen-site/issues/205#issuecomment-169307784.

I now checked whether this would work, but unfortunately it does not.
The below code is called prior to displaying the dataview.
I need to call some code when a row in the dataview is clicked to
control the routing to the "details" page.
Your suggestion below doesn't seem to be able to do that.
On 1/6/2016 12:56 PM, Petar Korponaić wrote:

@xauxatz https://github.com/xauxatz you can use page's
|controller_before_action|. Add code to execute before route renders:
|"controller_before_action": "switch(this.params.type) { case:
"standard": Router.go("somewhere", { id: this.params.id, type:
this.params.type); break; default: this.next(); }" |
This will result in onBeforeAction in your controller:
|... onBeforeAction: function() { switch(this.params.type) { case:
"standard": Router.go("somewhere", { id: this.params.id, type:
this.params.type }); break; case: "cool_type": Router.go("whatever"
.......... ; break; default: this.next(); } } ... |
—
Reply to this email directly or view it on GitHub
https://github.com/perak/kitchen-site/issues/205#issuecomment-169307784.

@xauxatz that code should be in your details route, not in the route where is dataview.

So, when item is clicked, details route is called and this code executes, redirecting user to another routes depending on value of 'type'

Very very nice. It works!
Many thanks. Meteorkitchen makes me feel like I have superpowers!
On 1/7/2016 1:06 PM, Petar Korponaić wrote:

So, when item is clicked, details route is called and this code
executes, redirecting user to another routes depending on value of 'type'
—
Reply to this email directly or view it on GitHub
https://github.com/perak/kitchen-site/issues/205#issuecomment-169643624.

:D :+1:

Glad it works! Let's close this issue then.

@xauxatz one more way (I think simpler one) added into v0.9.56, see this issue: https://github.com/perak/kitchen-site/issues/163

Now you can add custom code on table row click.
Here is how to add custom code:
In dataview component, there is new on_item_clicked_code:
"on_item_clicked_code": "alert(\"You clicked to item with ID: \", this._id);\n console.log(this._id);"


feat(nix): create flake

Creates the nix flake together with a dev shell. Also creates
a new, thinner dockerfile.
Closes #1 and closes #2.

When we add a new node or redeploy an existent one, a process to generate the mnemonic and the node data is triggered.
The mnemonic is generated by cexpl cli so it doesn't concern to substrate, but with that mnemonic we need to generate the node data using the following command
/subspace/node-subspace key insert --base-path /node-config --chain main --scheme Ed25519 --key-type gran --suri "$(cat mnemonic_content)"

The first issue I had was a missing snapshot file, but I recommend testing until we get something valid inside /node-config


K9mail becomes very slow

I have K9mail 5.718 on my Android 10 smartphone. The current uptime is 66 hours.
The K9mail app becomes very slow with increasing uptime. When I activate it, it first shows an empty screen, then after 6 seconds the common folder of new messages. When I then swipe down to fetch new mail, it takes another 6 seconds until it changes the status line to indicate that it is actually fetching mail. Then, when I actually have one, which is indicated by a sound signal, it takes 10 seconds until it appears on this list.
Going to this mail and then going back with the 'back' button is very slow, too.
The other apps behave normal, so K9mail is the only app that shows this behavior.

I just restarted my phone, but unfortunately this app still is slow. So it has nothing to do with the uptime.

Just curious… What are the memory and processor specs on your phone/device?
On August 30, 2020 7:26:20 AM AKDT, muessjoeh<EMAIL_ADDRESS>wrote:

I just restarted my phone, but unfortunately this app still is slow. So
it has nothing to do with the uptime.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/k9mail/k-9/issues/4927#issuecomment-683433774

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

It has a Qualcomm 450 1.8GHz and has 2786MB RAM.

@muessjoeh: In cases like this please post to the support forum instead. Without being able to reproduce this issue there's little chance developers can fix any of the bugs that might cause this.
Posting to the support forum will reach more people. Maybe someone has experienced the same problem and is able to offer some advice. It's also a chance for more people to ask questions that will hopefully narrow down the cause.
In cases like this the question that should be asked is: what changed between the app working as expected and the app stopping to work as expected? You speculated that the device uptime might be involved, but ruled that out later. What else has changed? Did you download a lot of messages to your device? Did you change K-9 Mail's settings? The important thing is that only you can test this (e.g. start with a fresh install and download more and more messages, check whether the app becomes slower).
Feel free to create a forum post to update us about your investigation. But please don't create bug reports in this issue tracker that are not actionable.
I'm closing this issue. Once you can reliably reproduce the issue (app is fast, do XYZ, app is slow) on your device feel free to create a new bug report.

One more investigation should be possible: When you have the log file, you could determine the steps that take extraordinarily long.


How to Hide Drawer when click item

Hi, I have a question that I can't solve, how do I do it when I click on an item in the drawer that hides?

@LChris94 sorry for the delay. In version 2.0,  you can import useLayoutCtx and use it like this.
import { useLayoutCtx } from '@mui-treasury/layout';

// dont forget to put <Root>...</Root> to the root of your app.
// import { Root } from '@mui-treasury/layout';
const YourSidebar = () => {
  const { setOpened } = useLayoutCtx();
  return (
    <div>
       <ListItem onClick={() => setOpened(false)} />
    </div>
  )
}

https://mui-treasury.com/layout/basic-examples/control-sidebar


Fix oidc

Pull Request Template
Fixes bug which appears in Service Manager during runtime, because CLI is used as dependency to construct OIDC client. It crashes because the token in nil, but the code is trying to reference it.
Pull Request status
Use checklists as a means to represent the status of your Pull Request.

[x] Initial implementation
[x] Refactoring
[ ] Unit tests

Pull Request Test Coverage Report for Build 262

0 of 32 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 74.035%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/auth/oidc/common.go
0
32
0.0%

Totals

Change from base Build 258:
-0.2%

Covered Lines:
1132

Relevant Lines:
1529

💛  - Coveralls


Rwa 864: Ensure created and due date are present and populated

JIRA link (if applicable)
Change description
Rwa 864: Ensure created and due date are present and populated
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x] No

Closed in favour of https://github.com/hmcts/wa-task-management-api/pull/433
Due to database data being polluted need to re-create


Should parsing be case sensitive?

As of now unary functions, binary functions, and constants are parsed case insenstive, i.e. Cos(pI) parses well.  This is due to the no_case declarations in the primary rule.
https://github.com/hmenke/boost_matheval/blob/6824ed51d5b477f2a8a2c182dc4a28a6e2498c54/include/matheval.hpp#L422-L427
Is this behaviour desirable or do we want to be case sensitive?

Fixed in 3a5ed267a1c8c2ab588d04d6221b762786bb24d5.


Advance Menu only apears for first object

I was trying to use the Advanced option to be able to toggle two kinds of shirts, so when i toggle one, the other one is off, but if i turn it off, it doesn't turn on the other one.
But the advanced menu only appears for the first object

Also mentioning the translation turns into "Anvanced" instead of "Advanced"

advanced settings are showed only when the object is controled.

So dumb i am, how didn't i notice that. Sorry


Add config option for image root in markdown preview

I'm using VSCode to write some simple blog posts using the HUGO static site generator, and while the markdown preview is really nice, images can't load correctly in the preview.
This is because the project structure is:
blog
|-->content
| |-->posts
|   |-->post1.md
|-->static
  |-->image.png

however the path that you specify in the markdown of the post for the image is /image.png as the static folder serves as the root for image files.
It'd be nice to have a config option for the markdown preview, that'd allow you to set the image root, so that you could specify the root to be project_root/static
Thanks for a great editor :)

@Duckle29 Great idea! May I ask a favor? How do other editors do this? Is it as simple as I imagine, or is there any unexpected messiness in implementing this? For example, are there any problems arising from the upteen flavors of Markdown?

@Duckle29 Great idea! May I ask a favor? How do other editors do this? Is it as simple as I imagine, or is there any unexpected messiness in implementing this? For example, are there any problems arising from the upteen flavors of Markdown?

@xpe Honestly I've always just used VSCode and accepted it, making tweaks to images if they didn't look right once built. I just finally decided to make this post after having just ignored it for a while

@xpe Honestly I've always just used VSCode and accepted it, making tweaks to images if they didn't look right once built. I just finally decided to make this post after having just ignored it for a while

@Duckle29 I think you'll be happier with how this feature request goes if you find an editor that does it well and look at their design. You might even be able to write a PR yourself.

@Duckle29 I think you'll be happier with how this feature request goes if you find an editor that does it well and look at their design. You might even be able to write a PR yourself.

@xpe In this case I don't see how that could change much. It's not much of a design (or I'm probably missing something) it's just that when the markdown render built into VScode looks for images. it should start at markdown.image_root
so if there's ![nicepics/fancyImage.png] and the config has "markdown.image_root": "/home/dude/stuff/static" then it should load /home/dude/stuff/static/nicepics/fancyImage.png

I'd like to start looking into making a PR for this! I am a first-time VSCode contributor but it seems pretty doable. I'll likely ask on the Gitter later and I have plenty of reading to do, but does anyone have a general direction to look in for the markdown preview renderer code?
Thanks!

I manage my articles on zenn.dev as code, and it's necessary for image files to be stored in {project_root}/images. Additionally, for these images to be recognized on zenn.dev, they must start with /images. This functionality is essential for my workflow.
Reference: https://zenn.dev/zenn/articles/deploy-github-images (The article is in Japanese)

ref: https://stackoverflow.com/questions/75831497/how-can-i-paste-images-into-my-markdown-files-in-vs-code
I think the ${documentWorkspaceFolder} parameter solves this problem.
Referring to my Settings, the file is placed in the images directory by default.
"markdown.copyFiles.destination": {
"**/*.md": "${documentWorkspaceFolder}/images/${documentBaseName}.png"
}
@Duckle29 @xpe


CallStackSizeExceeded when accessing $i18n.localeProperties.value within a component with a local scope applied

Environment

Operating System: Linux
Node Version:     v16.17.0
Nuxt Version:     3.6.5
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      telemetry, modules, i18n
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://codesandbox.io/p/github/nuxt/starter/csb-ctzmpk/draft/cranky-brown?file=/pages/index.vue:12,1
Describe the bug
This combination in a page component results in a "Maximum call stack size exceeded"
<script setup>
import { useI18n } from "#i18n";

// Remove this block to remove the error...
const { t } = useI18n({
  useScope: "local",
});

// Or don't do this...
console.log(useI18n().localeProperties.value);
</script>

Additional context
No response
Logs
No response

Thank you for you reporting!
The root cause of this issue is the same as #2335


[IOTDB-2444] Refine /ping API and fix unauthorized NPE error

See JIRA https://issues.apache.org/jira/browse/IOTDB-2444

Thanks for your contribution!!!
Please modify the english doc as well :D

@CloudWise-Lukemiao PTAL. Why we need the auth check when executing the ping process? Is anything else (like the Grafana plugin) needed to be modified?

PTAL @CloudWise-Lukemiao


Any hints on how this works?

I've installed the Helm chart (after adding a ports section to it), provided the credentials to it and the app is running, but when I call the service /metrics only gives me internal app metrics and /apps produce a 404.
Did I miss anything or misunderstood something?
Thanks!

Hi @dploeger ,
Thanks for reporting. Really looks like a glitch with the helm chart. I will see if i can find some time to look at it.
Right now I need to confess that it is more of a pet project, i.e. not using it in production anywhere.

Thanks!

The endpoint should be /api/apps, btw.

Ohh, okay. Will try to deploy it again later and test that.

Hm. Calling that endpoint produces an exception:
INFO:     <IP_ADDRESS>:55890 - "GET /api/apps/ HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/uvicorn/protocols/http/h11_impl.py", line 366, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/usr/local/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in __call__
    return await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/fastapi/applications.py", line 261, in __call__
    await super().__call__(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/applications.py", line 112, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/usr/local/lib/python3.10/site-packages/starlette_exporter/middleware.py", line 135, in __call__
    await self.app(scope, receive, wrapped_send)
  File "/usr/local/lib/python3.10/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc
  File "/usr/local/lib/python3.10/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 656, in __call__
    await route.handle(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 259, in handle
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 61, in app
    response = await func(request)
  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 227, in app
    raw_response = await run_endpoint_function(
  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 162, in run_endpoint_function
    return await run_in_threadpool(dependant.call, **values)
  File "/usr/local/lib/python3.10/site-packages/starlette/concurrency.py", line 39, in run_in_threadpool
    return await anyio.to_thread.run_sync(func, *args)
  File "/usr/local/lib/python3.10/site-packages/anyio/to_thread.py", line 28, in run_sync
    return await get_asynclib().run_sync_in_worker_thread(func, *args, cancellable=cancellable,
  File "/usr/local/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 818, in run_sync_in_worker_thread
    return await future
  File "/usr/local/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 754, in run
    result = context.run(func, *args)
  File "/app/./api/apps.py", line 20, in get_all
    return self.service.get_all()
  File "/app/./services/azure_app_service.py", line 30, in get_all
    self.observe(apps)
  File "/app/./services/azure_app_service.py", line 58, in observe
    expiry: Optional[datetime] = min(map(lambda c: c.expires, app.credentials))
ValueError: min() arg is an empty sequence

The envs AZURE_CLIENT_ID, AZURE_CLIENT_SECRET and AZURE_TENANT_ID are set correctly.

Probably a permission issue or an edge-case where app registrations do not have any credentials/expirations.
This will need some debugging skills. You might to check on

permissions
credentials check using azure-cli

Basically, the exporter does a Microsoft Graph query like this

Ah, right. It fails on apps without credentials. See #7 for a fix.

Thanks for merging. It works now.
One final question: Are the metrics updated automatically or do I need to call /api/apps/ from time to time?