Report Null pointer error when install application

What happened:
report error when install application，the image without base
error：
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x242c53b]

goroutine 1 [running]:
github.com/alibaba/sealer/apply/processor.InstallProcessor.RunConfig(0x312d0d8, 0x42f6680, 0x30eb2c0, 0xc000831aa0, 0x0, 0x0, 0x0, 0x0, 0xc0000fe200, 0x0, ...)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/apply/processor/install.go:58 +0x3b
github.com/alibaba/sealer/apply/processor.(*Executor).Execute(0xc00067db88, 0xc0000fe200, 0x30db760, 0xc000698280)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/apply/processor/interface.go:45 +0x94
github.com/alibaba/sealer/apply/applydriver.(*Applier).installApp(0xc0003a0d80, 0xc000667ef0, 0x47)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/apply/applydriver/local.go:281 +0xf3
github.com/alibaba/sealer/apply/applydriver.(*Applier).reconcileCluster(0xc0003a0d80, 0x0, 0x0)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/apply/applydriver/local.go:155 +0x5e5
github.com/alibaba/sealer/apply/applydriver.(*Applier).Apply(0xc0003a0d80, 0x1c, 0x310e4b0)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/apply/applydriver/local.go:76 +0x54
github.com/alibaba/sealer/sealer/cmd.glob..func1(0x420e840, 0xc00062f2c0, 0x0, 0x3, 0x0, 0x0)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/sealer/cmd/apply.go:37 +0x77
github.com/spf13/cobra.(*Command).execute(0x420e840, 0xc00062f290, 0x3, 0x3, 0x420e840, 0xc00062f290)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/vendor/github.com/spf13/cobra/command.go:852 +0x472
github.com/spf13/cobra.(*Command).ExecuteC(0x42103c0, 0x20, 0x6, 0x7)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/vendor/github.com/spf13/cobra/command.go:960 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/vendor/github.com/spf13/cobra/command.go:897
github.com/alibaba/sealer/sealer/cmd.Execute()
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/sealer/cmd/root.go:46 +0x31
main.main()
	/home/runner/work/sealer/sealer/src/github.com/alibaba/sealer/sealer/main.go:26 +0x35

Environment:

sealer version (use sealer version):

{"gitVersion":"v0.8.2","gitCommit":"30ce682","buildDate":"2022-04-12 11:28:17","goVersion":"go1.16.15","compiler":"gc","platform":"linux/amd64"}

@sfeng1996 ，yes, sure, main branch already fixed this issue. but have process to release the new version. i will reply you later.

@sfeng1996 v0.8.3 fixed


Razor Pages do not have simple build-in ajax partial pages, it is enough to do not switch to Razor Pages

Razor Pages do not have simple build-in ajax partial pages reload, it is enough to do not switch to Razor Pages

cc @mkArtakMSFT

What is the status of Microsoft.jQuery.Unobtrusive.Ajax? Any guidance here, as we're not all switching to React/Angular/Vue etc. :)

I recently added a sample here - https://github.com/aspnet/Entropy/commit/fb40862a3b48545948bbb2f3ef787f44b57345e0 - showing partial page updates using Razor Pages and Unobtrusive.Ajax. One of the missing pieces to remove some of the boiler plate is to add a first class Partial action result helper to Razor Page \ Page Model - https://github.com/aspnet/Mvc/issues/7885. We'd be happy to consider a PR for this.


Support for Dune Analytics IDE

It would be great if the extension would support the Dune Analytics IDE www.dune.com
In Dune you can write SQL statements to aggregate data from the blockchain with DuneSQL, a variation of Trino.
https://dune.com/blog/introducing-dune-sql
https://dune.com/docs/query/

Even without specific code for DuneSQL we might be able to work reasonably if the dialect is similar to other SQL-family languages. It looks like they're using the Ace editor, which we don't currently have an integration for.

Consolidating this into #9.


System Status Not Reporting

Description
New Relic infrastructure agent running on linux server stops retrieving data for SystemStatusEvent after a few hours. As of now there is nothing within the NR infrastructure logs to indicate a problems (no errors). The call will consistently execute until it just stops.
We do not have visibility into the code that executes these but here it seems to stop attempting to connect to the client AS400.
This issue occurs trying to connect to various client servers. Some more frequently than others.
Client server logs does not show any odd activity around the time of last new relic user connection to AS400 for retrieval of status info. Based off of the NR logs, it looks as though the agent stops trying to run command for system status for that given client.
Resetting the connection, calling install_linux.sh will resolve the problem but as mentioned before will usually begin failing again after a few hours.
Steps to Reproduce
We cannot reproduce on demand but problem occurs multiple times daily.
Expected Behavior
Continuous data retrieval.
Relevant Logs / Console output
Your Environment

ex: Browser name and version:
ex: Operating System and version:

Additional context

Please share the feedback based on latest released version 1.2.0 and we will address the same.


GH-15102: [C++] Could not decompress arrow stream sent from Java arrow SDK

Closes: #15102

We should probably make sure this is covered in a unit test, and check in a file to the test data repo that covers this, too, so that any language using the integration testing infrastructure is also tested. (Also, we may want to implement this optimization in C++, though I fear that has a high likelihood of generating backwards-incompatible files...)

Also, we may want to implement this optimization in C++, though I fear that has a high likelihood of generating backwards-incompatible files...

Agreed that it unfortunately wouldn't be very friendly at this point. We can revisit in two or three years probably...

I'll do that. Oddly enough, I don't see any practical way in the Java implementation to actually write a compressed file or stream, without subclassing and poking the writer implementation...so I'll be making some other changes.

https://github.com/apache/arrow-testing/pull/85

So, I went out on a limb and implemented the optimization on the C++ side. Given the backwards-compatibility concerns, I added IpcWriteOptions::compress_always, which enables it (but not by default). I don't know if this is an ideal solution in the long-term, but I could imagine it being useful regardless of any current file incompatibilities. That being said, I partially did it to make testing the reader easier,  since it wouldn't be straightforward otherwise. In any case, let me know what you think.
Also, the original changes in reader.cc weren't correct, so that's been fixed.

This is ready for a second look.
I switched over to using a min_space_savings percentage instead of just a boolean (the name is a bit weird, but it supposedly has some precedence so that's what I went with - no strong opinions though).
I'm also not quite sure whether the parameter should be a literal std::optional or just double = 0.0 by default. The effect should be the same either way. Some kind of range-validation is probably in order though, at least in debug mode.

I think double = 0.0 is enough.

Do we want to incorporate https://github.com/apache/arrow-testing/pull/85 ?

I think it's a good idea. The unit test coverage here is fairly superficial (and relies on the reader's implementation details).
If we ever do a Dask-like sampling optimization then I suspect we'd need to reassess the generated files, but any uncompressible input should be sufficient for now.

We can always add more generated files when needed.
If you can confirm the generated files fail without the patch/pass with the patch, then I can merge the new files and then you can bump the submodule commit as part of this PR.

I'll try to complete this in the next few days - didn't mean to let it sit.

Update: Integration files still pass.

Ok, you should be able to bump to https://github.com/apache/arrow-testing/pull/85

Alright, should be good to go.

@zeroshade it looks like Go has the same bug as C++ here
@benibus we should have Go skip the new files for now

@lidavidm Ok, they should be skipped now (I assume archery was the right place to do this?)

Yup, that looks good. Thanks.

Filed #34385 for Go

@zeroshade Thanks! Should be all set now

Seems Go still panics in CI

Weird, I'll investigate and get back to you @benibus, in theory it should be working properly unless i mucked something up in my unit tests....

Fixed the Go issue, I made a silly mistake. Integration tests are all green now! :smile:

Does the current PR solve this issue: https://github.com/apache/arrow/issues/34432 ? @benibus @lidavidm

I don't believe so.


ci: add race+codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0ea1727). Click here to learn what that means.
The diff coverage is n/a.


Add documentation for using WSL with PhpStorm

Tell me if you see something i missed.
Feel free to edit if needed.

Thanks


Docker-compose files refer to outdated IAIS repository

As @sebbader mentioned here https://github.com/International-Data-Spaces-Association/ids-metadata-broker-open-frontend/issues/6 port 4567 seems to be outdated, but is still used in composefiles

@SebastianOpriel I have changed it directly as of now, hence closing this issue.


helm template --notes does not also specify setValues causing catalog app deployment to fail if catalog template uses "required"

With helm chart templates that have {{required "Value is Required" .Values.exampleval }}in them, Rancher will fail to deploy the helm chart when it tries to generate notes, as it does not call helm template --notes with the setValues properly.
On this line: https://github.com/rancher/rancher/blob/master/pkg/controllers/user/helm/common.go#L191 setValues appended to the command
whereas here: https://github.com/rancher/rancher/blob/master/pkg/controllers/user/helm/common.go#L206 it is not

Able to reproduce this issue when deploying "5-bigip-ctlr" app from catalog -https://github.com/F5Networks/charts.git. App fails to deploy and is stuck in "Installing" state with error - Installing Helm template --notes failed. Error: render error in "f5-bigip-ctlr/templates/f5-bigip-ctlr-deploy.yaml": template: f5-bigip-ctlr/templates/f5-bigip-ctlr-deploy.yaml:5:4: executing "f5-bigip-ctlr/templates/f5-bigip-ctlr-deploy.yaml" at <required "BIG-IP url...>: error calling required: BIG-IP url not specified - add to Values or pass with `--set` : exit status 1
With v2.0.8-rc4, Able to deploy "5-bigip-ctlr" app from catalog -https://github.com/F5Networks/charts.git successfully.


Release CI/CD pipeline

We need to create a release CI/CD pipeline for this project.
Use these documents as a reference:

How to release the first Eclipse version of projects
JSONP release job

Waiting for https://bugs.eclipse.org/bugs/show_bug.cgi?id=538890#c16

Job: https://jenkins.eclipse.org/metro/job/ws-test-harness-release/
Testing is blocked by previous request.

Problem with OSSRH resolved. Tested with master and EE4J_8 branches.


Write stream

Hello,
Thanks for providing this library.
I am trying to 'reverse' the extract operation of the msiinfo example: write a binary stream to a package, like so:
let mut fin = File::open(&args.name).expect("open file");
let mut stream = package.write_stream(&args.name).expect("write stream");
io::copy(&mut fin, &mut stream).expect("insert");

where args.name is a String with a value from the streams list.
This fails on the write_stream in the call to create_stream_with_path of the cfb crate with "Bad file descriptor".
Any advice would be appreciated.

Did you open the file in writable mode?  File::open opens in read-only mode.

Thanks for your response.
I am only reading from the file, so RO should be fine.
But I did not open the package RW, so when I changed msi::open to msi::open_rw, the error went away.
I should have seen that, thanks. But the error message from cfb could be clearer...

Would it make sense to have Package::write_stream fail if the package has been open RO?
I would be willing to prepare a PR and looked if there would be an easy way to check the file permissions of a package, but could not find any, other than adding a rw: bool field to the Package struct.


"Dobromir", "Cetatea" are listed in the drop-down menu

Description:
When user writes "Dobromir", "Cetatea" the location appears in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Dobromir"

Expected results
The location appears in the drop-down menu as "Sat Dobromir, CONSTANȚA, (Dobromir)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Cetatea"

Expected results
The location appears in the drop-down menu as "Sat Cetatea, CONSTANȚA, (Dobromir)"


feat: Migrate Ethereum : eth_get_code from "beerus_rest_api" to "beerus_rpc"

Feature Request
Describe the Feature Request
We are deprecating "beerus_rest_api" and now migrating all the api endpoints to "beerus_rpc".
Migrate eth_get_code and add tests on beerus_rpc.
Describe Preferred Solution

Describe Alternatives

Related Code

Additional Context

If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

[ ] Yes
[ ] No

Self-assigned


[SemanticVersioning][Experiment] Struct comparators

I want to see how switching to an enum-based typing would affect the performance of version matching.
I'll report my findings here.
public enum ComparatorType { Primitive, XRange, Caret, Tilde, Hyphen }

public readonly struct Comparator
{
    public ComparatorType Type { get; }
    public PartialVersion? Operand1 { get; }
    public PartialVersion? Operand2 { get; }

    internal bool IsSatisfiedByCore(SemanticVersion version)
    {
        switch (Type)
        {
            case ComparatorType.Primitive:
                /* ... */
                break;
            case ComparatorType.XRange:
                /* ... */
                break;
            /* ... */
        }
    }
}

Oh, nevermind, I'd need to either desugar all advanced comparators when parsing a version range, or keep them along with the primitives that they desugared to.
public readonly struct Comparator
{
    public ComparatorType Type { get; }
    public PartialVersion? Operand1 { get; }
    public PartialVersion? Operand2 { get; }

    private ComparatorType primitiveType1;
    private SemanticVersion? primitive1;
    private ComparatorType primitiveType2;
    private SemanticVersion? primitive2;
}

I don't want to desugar anything, and adding, like, 18-27 extra bytes per every comparator is a bit too much.


denormalize graphql schema

currently, querying simple information from the postgres db requires multiple table joins, expensive calculations and have really bad performance. Hence, some data denormalization is needed, and if possible, doing the required calculations on the fly as events are reported.

I noticed that our processor DB didn't use indexes at all.
It is crucial to add database indexes to the entity fields on which one expects filtering and ordering.
I would look first into adding indexes to our schema as the lack of indices is the most common cause of slow API queries.


Grid State including some new undefined keys

Cause issues with client app when comparing state on the client to state saved as a JSON blob (which cleans up undefined values with JSON.stringfy() before sending to the server

discussed with Don, and it appears to be just pinned.
I can't reproduce this in toolbox yet, although certainly see some paths where it could come about

As I said, could not reproduce, but this seems like it should fix the problem, and is safer/clearer


Copying cells not working in firefox and cannot copy row headers🐛

🏷 Version 1.26.0

Package
Version

@antv/s2
1.26.0

Sheet Type

[ ] PivotSheet

🖋 Description

I cannot copy cell values in the Firefox browser. (Selecting the cells with mouse and Cntrl+C)
I cannot copy row/column headers in the Chrome browser (I can copy cell values).
The columns are copied without newline characters.
When I right-click after selecting cells, there is no copy option in the pop-up menu.

Note:
Above 1: Uncaught ReferenceError: ClipboardItem is not defined
It looks like Firefox and several other browsers do not support ClipboardItem object by default (https://developer.mozilla.org/en-US/docs/Web/API/ClipboardItem/ClipboardItem).
It may be helpful to you to mention that I used navigator.clipboard.writeText(S2.copyData(datasheet,'\t',true))  instead of S2.copyToClipboard(data) to implement the copy button.
⌨ Code Snapshots

   const segment_s2Options = {
        width: screen.width*10/12,
        height: screen.height*0.3,
        style: {
            layoutWidthType: 'colAdaptive'
        },
        interaction: {
            enableCopy: true,
            selectedCellsSpotlight: false,
            hoverHighlight: false,
            selectedCellHighlight: true
        }
}

The latest version has been fixed, use sync copy when ClipboardItem is not supported


Add a retry register for sqlite3.BUSY

It may be favorable to retry endlessly when a function returns sqlite3.BUSY, as that's basically what sqlite expects you do to.
This can't be done in a transaction though, boo.
We may want sqlite3.BUSY to fail sometimes, so maybe add a register that controls this, e.g. em.retry
Thus it'll only retry if transaction == nil and em.retry.

Possibly implemented by fcfd034ae0defdc78be3f96e3a18893ef5c685ea but untested.


Add support for TOML frontmatter

A popular static site generate Hugo, has an option for TOML front matter:
+++
title = "spf13-vim 3.0 release and new website"
description = "spf13-vim is a cross platform distribution of vim plugins and resources for Vim."
tags = [ ".vimrc", "plugins", "spf13-vim", "vim" ]
date = "2012-04-06"
categories = [
  "Development",
  "VIM"
]
slug = "spf13-vim-3-0-release-and-new-website"
+++
Content of the file goes Here

Which isn't supported by remark-lint:
diff --git
index 8ecfdee..af60f1d 100644
@@ -1,9 +1,9 @@
-+++
+\+++
 title = "Step Definitions Orig"
 menu = "gherkin"
-+++
+\+++

As the parser ignores the front matter, I think this should be an easy fix?

Thank you @wooorm !

P.S. If you run into plugins that don’t support TOML nodes, raise issues on their repo’s and we’ll figure out how to support them!


Add FGVC-Aircraft Dataset

I've added a data wrapper for the FGVC-Aircraft dataset (http://www.robots.ox.ac.uk/~vgg/data/fgvc-aircraft).
I've included an option to download the dataset (like the MNIST dataset), but since the dataset is relatively much larger (2.75GB), this might want to be reconsidered.
Another thing to consider is what the class name (and file name) should be; currently the new file is titled aircraft.py and the class name is FGVCAircraft.

Thanks @ruthcfong for this contribution and sorry for taking that long to get back to you.
As you might now, there is a new dataset API being designed here: https://github.com/pytorch/vision/issues/5336
The plan for now is to wait a bit more until the API is stable and then port existing and new datasets to it.
For this purpose, I suggest that we help you port your work to the new API once it is ready or you can do it yourself if it is something you want to do of course. Credit for the implementation will be given to you of course once the PR is updated.
One additional question: since this PR is 4 years old, have you used other implementations while waiting for the PR to be merged?
Again, thanks for your contribution @ruthcfong. Have a good day. :hugs:

@yassineAlouini See https://github.com/pytorch/vision/issues/3562#issuecomment-1167017741.
@ruthcfong I'm sorry your PR was never reviewed. In the mean time, it was redone in #5178 and the dataset is now available through our API. Thus, I'm closing this.

@yassineAlouini See #3562 (comment).
@ruthcfong I'm sorry your PR was never reviewed. In the mean time, it was redone in #5178 and the dataset is now available through our API. Thus, I'm closing this.

Thanks @pmeier for pointing this out. :ok_hand:


Script wont open

When I open the script, it opens for a second and immediately shuts down. I did everything according to the tutorial.

I solved the problem, but after logging in and showing the number of my points, I was given this and a new line began.

Fixed with v3.0 #188


ios Safari: swipe not swiping when using virtual mode

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/p/sandbox/2q6pl5?file=/index.html
Bug description
I need to use the virtual mode on safari with Swiper Element (webview or browser directly). After hours playing with it, I tried the demo example directly and the swipe doesn't work. Try swiping and it moves one slide at a time.
I tried adding freeMode prop because I need it, but it's the same result.
On chrome desktop, it works as expected, slides smoothly.
https://github.com/user-attachments/assets/52080b8e-ec5c-4781-8b22-aae180a76a91
Expected Behavior
The slides moves smoothly.
Actual Behavior
It moves one slide at a time when swiping.
Swiper version
11
Platform/Target and Browser Versions
ios safari & chrome android
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

t0ggles-create swiper


Add Monocypher bindings

Link
https://github.com/konovod/monocypher.cr
Checklist

[X] - Shard is at least 30 days old.
[X] - Shard has CI implemented.
[X] - Shard has daily/weekly periodic builds (ideally with Crystal latest and nightly).

huh, looks like I messed with branches.


send audio buffer

how to send audio buffer?
i've tried ctx.reply({ audio: { buffer: buffer }})

@thoriqazzikraa  You can try passing the buffer into the "audio" property. For example:
let buffer = fs.readFileSync('./sample.mp3');
ctx.reply({ audio: buffer });


Investigation and Resolution of Unexplained .cancel() and .abort() Operations and Asynchronous WebSocket Behavior

Describe the bug
Code contains instances where .cancel() and .abort() operations are executed without defined reasons, risking potential problems. Also, WebSocket library might be processing messages asynchronously, causing out-of-order issues. A lock mechanism for synchronization is suggested.
Tasks:

Locate and document all instances of .cancel() and .abort() without defined reasons.

Review WebSocket library behavior for asynchronous message processing.

Evaluate and implement a lock for synchronous operations.

All instances of cancel or abort without a reason
RPCClient.ts
  @ready(new rpcErrors.ErrorMissingCaller())
  public async unaryCaller<I extends JSONValue, O extends JSONValue>(
    method: string,
    parameters: I,
    ctx: Partial<ContextTimedInput> = {},
  ): Promise<O> {
    const callerInterface = await this.duplexStreamCaller<I, O>(method, ctx);
    const reader = callerInterface.readable.getReader();
    const writer = callerInterface.writable.getWriter();
    try {
      await writer.write(parameters);
      const output = await reader.read();
      if (output.done) {
        throw new rpcErrors.ErrorMissingCaller('Missing response', {
          cause: ctx.signal?.reason,
        });
      }
      await reader.cancel();
      await writer.close();
      return output.value;
    } finally {
      // Attempt clean up, ignore errors if already cleaned up
      await reader.cancel().catch(() => {});
      await writer.close().catch(() => {});
    }
  }

    // Hooking up agnostic stream side
    let rpcStream: RPCStream<Uint8Array, Uint8Array>;
    const streamFactoryProm = this.streamFactory({ signal, timer });
    try {
      rpcStream = await Promise.race([streamFactoryProm, abortRaceProm.p]);
    } catch (e) {
      cleanUp();
      void streamFactoryProm.then((stream) =>
        stream.cancel(ErrorRPCStreamEnded),
      );
      throw e;
    }
    void timer.then(
      () => {
        rpcStream.cancel(
          new rpcErrors.ErrorRPCTimedOut('RPC has timed out', {
            cause: ctx.signal?.reason,
          }),
        );
      },
      () => {}, // Ignore cancellation error
    );
    // Deciding if we want to allow refreshing
    // We want to refresh timer if none was provided
    const refreshingTimer: Timer | undefined =
      ctx.timer == null ? timer : undefined;
    // Composing stream transforms and middleware
    const metadata = {
      ...(rpcStream.meta ?? {}),
      command: method,
    };
    const outputMessageTransformStream =
      rpcUtils.clientOutputTransformStream<O>(metadata, refreshingTimer);
    const inputMessageTransformStream = rpcUtils.clientInputTransformStream<I>(
      method,
      refreshingTimer,
    );
    const middleware = this.middlewareFactory(
      { signal, timer },
      rpcStream.cancel,
      metadata,
    );
    ```
    `RPCServer.ts`

```typescript
 @ready(new rpcErrors.ErrorRPCHandlerFailed())
  public handleStream(rpcStream: RPCStream<Uint8Array, Uint8Array>) {
    // This will take a buffer stream of json messages and set up service
    //  handling for it.
    // Constructing the PromiseCancellable for tracking the active stream
    const abortController = new AbortController();
    // Setting up timeout timer logic
    const timer = new Timer({
      delay: this.handlerTimeoutTime,
      handler: () => {
        abortController.abort(new rpcErrors.ErrorRPCTimedOut());
        if (this.onTimeoutCallback) {
          this.onTimeoutCallback();
        }
      },
    });

    const prom = (async () => {
      const id = await this.idGen();
      const headTransformStream = rpcUtilsMiddleware.binaryToJsonMessageStream(
        rpcUtils.parseJSONRPCRequest,
      );
      // Transparent transform used as a point to cancel the input stream from
      const passthroughTransform = new TransformStream<
        Uint8Array,
        Uint8Array
      >();
      const inputStream = passthroughTransform.readable;
      const inputStreamEndProm = rpcStream.readable
        .pipeTo(passthroughTransform.writable)
        // Ignore any errors here, we only care that it ended
        .catch(() => {});
      void inputStream
        // Allow us to re-use the readable after reading the first message
        .pipeTo(headTransformStream.writable, {
          preventClose: true,
          preventCancel: true,
        })
        // Ignore any errors here, we only care that it ended
        .catch(() => {});
      const cleanUp = async (reason: any) => {
        await inputStream.cancel(reason);
        await rpcStream.writable.abort(reason);
        await inputStreamEndProm;
        timer.cancel(cleanupReason);
        await timer.catch(() => {});
      };
      // Read a single empty value to consume the first message
      const reader = headTransformStream.readable.getReader();
      // Allows timing out when waiting for the first message
      let headerMessage:
        | ReadableStreamDefaultReadResult<JSONRPCRequest>
        | undefined
        | void;
      try {
        headerMessage = await Promise.race([
          reader.read(),
          timer.then(
            () => undefined,
            () => {},
          ),
        ]);
      } catch (e) {
        const newErr = new rpcErrors.ErrorRPCHandlerFailed(
          'Stream failed waiting for header',
          { cause: e },
        );
        await inputStreamEndProm;
        timer.cancel(cleanupReason);
        await timer.catch(() => {});
        this.dispatchEvent(
          new rpcEvents.RPCErrorEvent({
            detail: new rpcErrors.ErrorRPCOutputStreamError(
              'Stream failed waiting for header',
              {
                cause: newErr,
              },
            ),
          }),
        );
        return;
      }
      // Downgrade back to the raw stream
      await reader.cancel();
      // There are 2 conditions where we just end here
      //  1. The timeout timer resolves before the first message
      //  2. the stream ends before the first message

Discussion about RPCServer and RPCClient lifecycles:
https://github.com/MatrixAI/Polykey/pull/552#issuecomment-1742092717
Brian — Yesterday at 12:15 PM
Moving the RPCServer into NCM is what we discussed before. I  think it's fine so long as the server manifest is generic and passed in just to simplify testing.

Seems like the RPCServer could be StartStop pattern. It could just be a normal class but the handleStream is ready decorated. While the RPCServer doesn't really manage the streams it's handling. It is managing the handlers and fully allowed to cancel any active streams directly.

I think the main reason the RPCServer has a life cycle is to clearly track the existence of active streams. If we're ending the RPCServer, for simplicity we want any running handlers to end before the RPCServer has ended. This makes it clear that it's safe to stop any domains because we can be sure no handlers are actively using them.

That said, stopping the transport layer stuff serves the same function, but since the RPC is agnostic to transport we don't want to depend on that. Then again, the RPC can only advise abort unless we want to keep  a force destroy functionality around.
CMCDragonkai — Yesterday at 12:17 PM
Yes that's why I was thinking that stopping the  RPCServer is more appropriate and in stopping should only result in a signal abortion, not cancellation of streams
Because RPCServer is agnostic to the stream, and managing the lifecycle of the handlers is controlled by an advisory signal abortion instead of cancelling streams
Brian — Yesterday at 12:17 PM
Fair, In any case, the RPCServer should only by fully stopped once all handlers have completed.
CMCDragonkai — Yesterday at 12:18 PM
Yes after sending the signal abortion it can wait for all handlers to have completely resolved
Brian — Yesterday at 12:24 PM
Mmmk, So as far as changes go, based on that we're just removing the force: boolean option from rpcServer.destroy(), making sure it awaits all active streams. And possibly converting it to StartStop.
This will need to be changed on js-rpc.
CMCDragonkai — Yesterday at 12:33 PM
Including unary, cs, SS, and duplex
Brian — Yesterday at 12:33 PM
I'm not sure what you mean?
CMCDragonkai — Yesterday at 12:58 PM
I'm not sure we have force at all
It's just signal abortion
For all 4 different kinds of handlers
And signal abortion is always advisory
So you just wait until handlers have finished
Brian — Yesterday at 12:59 PM
Logic wise, setting force:true for RPCServer.destroy() just triggers abortion for the handlers before waiting for them to complete. force:false just waits for them to complete.

Conclusion is:

RPCServer can be Start/Stop. Where Stop has force and force true means sending a special abortion signal to all the handler types. (Not cancel streams)
RPCClient remains Create/Destroy - where Destroy has a force, and force true means sending a special abortion signal to all caller types. (Not cancel streams)

https://github.com/MatrixAI/js-rpc/blob/c885dc16eb1e47283e7e91a3ca9334dc20c4f62c/src/RPCServer.ts#L217-L236
Right now this says this.activeStreams. If this is the handler promises. This is a very bad name for this. It should be called this.handlerPs. This tells us that these are the active handler promises that still resolving.
And it needs to send a special reason for force true.

If the RPC handlers reject upon being aborted by stop, and it uses throwIfAborted, that will result in RPCServer.stop throwing.

Internal signal reasons are usually symbols. Not exception objects.

Ignore everything except than the OP. The OP has been rewritten to focus on what we need.

@amydevs is this also being done?

This is still pending as there is still cancellations occurring without reasons in RPCServer.ts.

i don't think there are any places where cancel or abort are used without reason, or closing the stream ungracefully where it is unneeded.
There is one place where it is done:
const reader = headTransformStream.readable.getReader();
      // Allows timing out when waiting for the first message
      let headerMessage:
        | ReadableStreamDefaultReadResult<JSONRPCRequest>
        | undefined
        | void;
      try {
        headerMessage = await Promise.race([
          reader.read(),
          timer.then(
            () => undefined,
            () => {},
          ),
        ]);
      } catch (e) {
        const newErr = new errors.ErrorRPCHandlerFailed(
          'Stream failed waiting for header',
          { cause: e },
        );
        await inputStreamEndProm;
        timer.cancel(cleanupReason);
        await timer.catch(() => {});
        this.dispatchEvent(
          new events.RPCErrorEvent({
            detail: new errors.ErrorRPCOutputStreamError(
              'Stream failed waiting for header',
              { cause: newErr },
            ),
          }),
        );
        return;
      }
      // Downgrade back to the raw stream
      await reader.cancel();

But this does not matter, as the headTransformStream is Tee of the actual underlying stream that is meant to only parse the header of each RPC call. Hence, the cancel does not propagate to the parent stream.
Therefore, this issue is considered no longer applicable.

When you say it doesn't matter, should we even keep it?


POST calls received as GET

For an unknown reason, when sending a POST call to a function (/api/v1/proxy/namespaces/default/services/foo), the function receive a GET request
Here are some facts:

Sending a curl POST request works fine.
I thought it was due to the cors-proxy used to fix cors in dev, but a curl request through the cors-proxy works fine.
Sending a POST request to create the function works fine.
Sending a POST request in the app, through the cors-proxy, to a sample api server (https://httpbin.org/post) works fine.

I'm thinking about a problem with headers on the kubernetes api server, if someone has any idea..?

which version of k8s and which version of kubeless are you using ?

I am going to wait for Tuna to tell me things are stable before testing things again.

Kubernetes: v1.6.0 (server)
Kubeless: v0.0.11

Fixed with #16, removing the cors-anywhere proxy


Couldn't find env file

F:\Scraperr-master>make up
docker compose -f docker-compose.yml --env-file .env up -d --force-recreate
Couldn't find env file: F:\Scraperr-master.env
make: *** [Makefile:40: up] Error 15
F:\Scraperr-master>

If you look at the command the Makefile is running, it is clearly stating that it is looking for an env file named .env. So either change your compose file and your name of your env file or change the command the Makefile is running.


hydra 2FA on cloud providers

I read a little the documentation but i couldn't figure it out so i'm asking:
Is it possible to configure 2FA with hydra on multiple servers(also on different cloud providers? aws,openstack..) for SSH access?

Hydra currently doesn't support 2FA but if it where it would work in a distributed environment, as long as they are backed by the same rethinkdb instance. If you have the ability to check an HTTP api for the 2fa steps, this should work.

Okk.. there is a web ui\dashboard to manage users? in the features i saw this : Real Time monitoring, with this feature is it possible to trace every API request? (Audit logging)

No, hydra does not provide Identity Provider capabilities. The real time monitoring is the rethinkdb admin ui, which supports real time monitoring. I suggest to read the guide and http api documentation.

2FA support is tracked as #69


Help Installing Ifopt-Ipopt w/o catkin

I'm hoping to build Ifopt (with ipopt in particular) on a Mac-OSX without the use of catkin.
I have Ipopt and Eigen installed properly, but I'm not sure about which lines need to be added/changed for a standard mkdir build; cd build; cmake .. to work.
In particular, I've put in my Ipopt directory in the set(IPOPT_DIR ... ) line of CMakeLists.txt but I'm not totally sure what it means to ensure that "IPOPT headers and library are installed; somewhere in the default search paths", and I got the following error:

Could not find a package configuration file provided by "catkin" with any
of the following names:

catkinConfig.cmake
catkin-config.cmake

Apologies for the novice question but I would very much appreciate some assistance!

@shoshievass I have a similar need, so I'll have a crack at this tomorrow.

Awesome! Please let me know how it goes!

I will submit a pr that you can look at in the next few days.

Great, thanks @jwdinius !
I'll have a look at it in the next few days.

I'm changing the pr to work-in-progress so you all can have a look at it and make comments.  I would like to clean up the implementation a little more before it's merged.

Hi @jwdinius, @shoshievass
Building with pure cmake works now (see #17), please let me know if you experience any problems.
Thanks for your help on this @jwdinius, sorry I couldn't merge your PR, I decided to do some beneficial structural code improvements as well while adding the cmake features.

No worries.  I actually thought that I had updated the PR with other work I had done, but I guess I was mistaken.  For your reference, https://github.com/jwdinius/ifopt/tree/pure_cmake.
I followed the outline provided by kindr in most places, including setting up package exporting.  Now, ifopt_ipopt(snopt) can use find_package() to find ifopt_core during the local superbuild or via an external cmake project.  Please have a look if you are interested in exporting these projects, which might be handy when trying to install ifopt from within another project (like how kindr is built within https://bitbucket.org/adrlab/ct).
I was able to verify that the packages were able to found from an external cmake project built with unit tests.  Hopefully something in my repo will be of use to you.  Cheers!

Hi,
I am trying to build using CMake. I have made changes according to instructions in the CmakeLists.txt. But I am getting the following error.
~/MyProject$ ls
bin       build      CMakeLists.txt  example.cpp  libipopt2.so
$cmake --build build -- -j3
[ 50%] Linking CXX executable ../bin/example
//usr/local/lib/libipopt.so.1: undefined reference to dpotrf_' //usr/local/lib/libipopt.so.1: undefined reference to dgemm_'
//usr/local/lib/libipopt.so.1: undefined reference to dnrm2_' //usr/local/lib/libipopt.so.1: undefined reference to dscal_'
//usr/local/lib/libipopt.so.1: undefined reference to dpotrs_' //usr/local/lib/libipopt.so.1: undefined reference to dtrsm_'
//usr/local/lib/libipopt.so.1: undefined reference to dlopen' //usr/local/lib/libipopt.so.1: undefined reference to daxpy_'
//usr/local/lib/libipopt.so.1: undefined reference to dgetrf_' //usr/local/lib/libipopt.so.1: undefined reference to dcopy_'
//usr/local/lib/libipopt.so.1: undefined reference to idamax_' //usr/local/lib/libipopt.so.1: undefined reference to dlclose'
//usr/local/lib/libipopt.so.1: undefined reference to dlerror' //usr/local/lib/libipopt.so.1: undefined reference to dlsym'
//usr/local/lib/libipopt.so.1: undefined reference to dsyrk_' //usr/local/lib/libipopt.so.1: undefined reference to dasum_'
//usr/local/lib/libipopt.so.1: undefined reference to dsyev_' //usr/local/lib/libipopt.so.1: undefined reference to dmumps_c'
//usr/local/lib/libipopt.so.1: undefined reference to dgetrs_' //usr/local/lib/libipopt.so.1: undefined reference to ddot_'
collect2: error: ld returned 1 exit status
CMakeFiles/example.dir/build.make:96: recipe for target '../bin/example' failed
make[2]: *** [../bin/example] Error 1
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/example.dir/all' failed
make[1]: *** [CMakeFiles/example.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

@sapan-ostic It looks like ipopt can't find blas.  Are you sure that your libipopt.so works as expected?  Did you follow the build stages at https://www.coin-or.org/Ipopt/documentation/node10.html?  Are you working on linux?  You could just apt install ipopt and it will handle the dependencies for you.  If you run into other problems, you could look at my fork https://github.com/jwdinius/ifopt.  I also wrote a blog post about the conversion process https://jwdinius.github.io/blog/2018/cmake/.  Hopefully one of these references will help.  Good luck!

I am having the same issue as @sapan-ostic.
I try to compile the ex_test_ipopt.cc using cmake.
find_package(IPOPT 3.11.10 REQUIRED)
find_package(ifopt 2.0.1 REQUIRED)
set(LIB_IPOPT ifopt_ipopt)
add_executable(${LIB_IPOPT}-example ex_test_ipopt.cc)
target_link_libraries(${LIB_IPOPT}-example
  PUBLIC
    ifopt::ifopt_core
    ifopt::ifopt_ipopt
)

And it gives me:
[ 20%] Linking CXX executable ifopt_ipopt-example
//usr/local/lib/libipopt.so.1: undefined reference to `dpotrf_'
//usr/local/lib/libipopt.so.1: undefined reference to `dgemm_'
//usr/local/lib/libipopt.so.1: undefined reference to `dnrm2_'
//usr/local/lib/libipopt.so.1: undefined reference to `dscal_'
//usr/local/lib/libipopt.so.1: undefined reference to `dpotrs_'
//usr/local/lib/libipopt.so.1: undefined reference to `dtrsm_'
//usr/local/lib/libipopt.so.1: undefined reference to `dlopen'
//usr/local/lib/libipopt.so.1: undefined reference to `daxpy_'
//usr/local/lib/libipopt.so.1: undefined reference to `dgetrf_'
//usr/local/lib/libipopt.so.1: undefined reference to `dcopy_'
//usr/local/lib/libipopt.so.1: undefined reference to `idamax_'
//usr/local/lib/libipopt.so.1: undefined reference to `dlclose'
//usr/local/lib/libipopt.so.1: undefined reference to `dlerror'
//usr/local/lib/libipopt.so.1: undefined reference to `dlsym'
//usr/local/lib/libipopt.so.1: undefined reference to `dsyrk_'
//usr/local/lib/libipopt.so.1: undefined reference to `dasum_'
//usr/local/lib/libipopt.so.1: undefined reference to `dsyev_'
//usr/local/lib/libipopt.so.1: undefined reference to `dmumps_c'
//usr/local/lib/libipopt.so.1: undefined reference to `dgetrs_'
//usr/local/lib/libipopt.so.1: undefined reference to `ddot_'
//usr/local/lib/libipopt.so.1: undefined reference to `dsymv_'
//usr/local/lib/libipopt.so.1: undefined reference to `dgemv_'
collect2: error: ld returned 1 exit status
CMakeFiles/ifopt_ipopt-example.dir/build.make:96: recipe for target 'ifopt_ipopt-example' failed
make[2]: *** [ifopt_ipopt-example] Error 1
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/ifopt_ipopt-example.dir/all' failed
make[1]: *** [CMakeFiles/ifopt_ipopt-example.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

I solved this by reinstall the ipopt with BLAS. Aka, remove the coinor-libipopt-dev, and compile ipopt from source after get the Thirdparty solver BLAS.


Fix godot stuck while thread import more ldtk file

Thank you! I had no idea how to tackle this and this is well-received.


Create missing Ad.Groups roles in Azure for the satellites

Global:

[x] Editor
[ ] Sub Editor
[ ] Local News Editor

Argentina:

[ ] Editor
[ ] Sub Editor

Brazil:

[x] Editor
[ ] Sub Editor

Fondene:

[x] Editor
[ ] Sub Editor

Japan:

[x] Editor
[ ] Sub Editor

South Korea:

[x] Editor
[ ] Sub Editor

Storage:

[x] Editor
[ ] Sub Editor

Germany:

[x] Editor
[ ] Sub Editor

Which groups are missing @fernandolucchesi ?

Which groups are missing @fernandolucchesi ?

I've updated the issue description with the groups overview.

Process started. Waiting now for ServiceNow to create necessary groups in AD.


NavigationCompleted event not fired on Windows Server

Why the Sample code works fine on Windows 10 but doesn't work on Windows Server 2012?
I traced the code, and I found that the NavigationCompleted event not fired on Windows Server, what's the reason?
FormTest.zip

Sorry, I don't know the answer to that. NavigationCompleted is a native WebView2 event, so you'd need to ask on the WebView2 forum.

Okay thanks


Unattended install from .NET

Hi all,
I'm interested in allowing our intranet users to create new SSL certs via a .NET application.
I have tried every which way I can think of to get WACS to run unattended with the necessary privileges and install a new certificate but it always fails.
Is there a way to create a new cert from a web app running on the same server and IIS as WACS is located without having to us elevated permissions..?
Thanks all

Hi, that should be possible as long as you avoid any plugins that require administrative access. From the top of my head:

IIS (target or install)
Self-hosting validation
Windows Certificate Store

Alternatives could be manual target, filesystem validation and the pfx file store.

Unfortunately that is exactly what I want to do, is install it to IIS with self hosting validation :(

Another option would be that instead of calling wacs.exe from your code, you create a renewal.json file in the configuration folder instead, and then trigger the scheduled task to run. That should be possible without being an administrator.

And if the certificate and binding don't exist in IIS then the renewal process will create it? I'll look into the renewal JSON format now

Yes, the scheduled task will do all of that, just like if you were running it from the menu.

Thanks,
I haven't been able to find any docs that reveal the structure of that JSON I'll keep looking.
Thanks

It's not documented, but on the other hand it's pretty straightforward. Just set up a renewal manually and use the resulting file as a template. Or peak at the source code if you want to know all the details.


Added support for TLS Certificate based Authentication

Added support for TLS Certificate based Authentication to be able to get the token with client TLS Certificate, as is required according to #206

@sudiptosarkar please fix the above issues so we can continue :)

@aviadhahami, Done. Please approve the workflows.

@sudiptosarkar please fix workflow errors 🙇

@sudiptosarkar please fix workflow errors bow

@aviadhahami, Done. Verified with npm run lint this time.

Codecov Report

Merging #207 (370ba51) into master (61273fb) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##            master      #207   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          141       141
=========================================
  Hits           141       141

Impacted Files
Coverage Δ

src/commands.js
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

thx for your contribution!


Support test modularisation

K6 and Locust by themselves support modular test structure (e.g. main test file which uses helper classes to run the test). However Kangal places limitation that only a single test file can be passed to Kangal and executed.
For more complex scenarios to have just a single file for each test doesn't make a lot of sense. I would be nice to be able to have reusable modules/test functions. I see that this might be achieved by having a whole test code repository available on Kangal master/worker nodes and being able to import modules in the main test script which is passed to Kangal for execution.
Do you see this as a possible improvement for the tool? Or could this be achieved by extra steps to clone the repository to images used by Kangal?

Or another way would be to have multiple .js files, but before starting test to combine multiple files into one and pass it to Kangal?


Logo is not properly aligned, navigation bar components are not properly visible

The logo for ML Fusion is not properly aligned. We can do some modifications using CSS to align it properly.
Also , when we hover on the components of navigation bar, the text is not visible properly, so for this, we can do some changes to make it properly visible.
The scroll up button and message button in the bottom right corner of the homepage are not well aligned with each other to which we can do some modifications with the help of CSS.
I would love to work on these mentioned parts of the project to make it more appealing.
Attaching a screenshot of the issues I would like to resolve!

I have seen many navbar issues and logo issues, and i have mentioned it before for contributors to not create issues regarding navbar and there should be no more navbar issues, so i have closed this issue, if you have any queries, you can join discord server for further updates and communication.
we are not looking for navbar updates for now, as it was done and changes many times before, please understand.
and already it is in updation, so please understand.
For now scroll up button and message is okay, don't required any updation for now, we will look it later.
So please Understand.


balena-yocto-scripts: Update to v1.21.7

Update balena-yocto-scripts from v1.21.1 to v1.21.7
Changelog-entry: Update balena-yocto-scripts to v1.21.7

lgtm

Can one of the admins verify this patch?

@resin-jenkins ok to test

@resin-jenkins add to whitelist


Page TimeOut

How to configure Helium so that the page load takes, say, no more than 15 seconds, so that after that you can catch the exception and do what I need.
The setup method does not work, for several days I have not been able to work with the problem.
In addition, as I understand it, by default, the page load time is equal to the timeout time of the script, which makes it impossible to catch the exception and continue working.
Please tell me how to set the page load timeout?

Sorry, all it,s Ok, i do:
start_chrome(Link).set_page_load_timeout(15)
instead:
driver = start_crhome()
driver.set_page_load_timeout(15)
go_to(Link)

Sorry, all it,s Ok, i do:
start_chrome(Link).set_page_load_timeout(15)
instead:
driver = start_crhome()
driver.set_page_load_timeout(15)
go_to(Link)


Rename read::error to read::error_no_replace

The error function exists upstream, while the error_or_replace does not. This means that when upstream adds a reference teo error where we would like to use error_or_replace instead, it's easy to miss the change, if it is not covered by tests.
With this PR, such a new reference to error will fail as no such function is defined, and we will need to decide which function to call.

I had hypothesized that #32 was required due to a missed change like this in a merge, but I don't think that's actually the case. Still, this is probably a good idea.


First option in radio group should be selected by default (figma)

Finding in Figma
Feedback from Marta Rutkowska: just for making sure that the radio button groups will be used properly: shouldn't we show in the examples that the first radio button is in "on" state by default?

This is not how it works in HTML or the ux in general, it's not mandatory to make choice so we don't provide this within figma or Storybook.


Fix Node not found error InkRuntime.init()

Checklist for this pull request

[x] I have read the guidelines for contributing.
[x] I have checked that my code additions did not fail tests.

Description
Using Node.get_node on a non-existant node and checking for null was causing a
Node not found error to always appear in the debugger.

Thanks for the PR, much appreciated @MageJohn!


Fix compilation on fedora 39 - aarch64

Issue
Compilation fails on mac m1 pro running asahi linux with the following error:
CXX(target) Release/obj.target/os_dns_native/binding.o
../binding.cc: In constructor ‘{anonymous}::ResourceRecord::ResourceRecord(ns_msg*, size_t)’:
../binding.cc:85:16: error: ‘runtime_error’ is not a member of ‘std’
   85 |     throw std::runtime_error(
      |                ^~~~~~~~~~~~~
../binding.cc:8:1: note: ‘std::runtime_error’ is defined in header ‘<stdexcept>’; did you forget to ‘#include <stdexcept>’?
    7 | #include <resolv.h>
  +++ |+#include <stdexcept>
    8 |

Description
This blindly applies the suggested fix which works.

Duplicate of https://github.com/mongodb-js/os-dns-native/pull/5


Unable to use with Internet Explorer 11 ... object properties not supported when trying to attach files

Type of issue

[x] Bug report
[ ] Feature request

Uploader type

[ ] Traditional
[ ] S3
[x] Azure

Note: Support requests cannot be accepted due to lack of time.

Bug Report
Version 5.14.5
Browser IE11
OS Windows 10; Windows 7 ...
Note: this does not happen with Chrome, Safari, or even with Edge. I tried using EDGE in emulation mode all the way down to Explorer 8 and it still worked. It seems to block on IE11 (as well as E11 with any emulation setting).
Exact steps required to reproduce the issue

Open download portal (using fine-uploader)
Attempt to attach file (form opens, select a file, click enter)
File does not upload

Detailed explanation of the problem
Developer Tools ... Console errors:
[Fine Uploader 5.15.4] Caught exception in 'onStatusChange' callback - Object doesn't support property or method 'find'
[Fine Uploader 5.15.4] Caught exception in 'onUpload' callback - 'Promise' is undefined
[Fine Uploader 5.15.4] Caught exception in 'onStatusChange' callback - Object doesn't support property or method 'find'
[Fine Uploader 5.15.4] Caught exception in 'onComplete' callback - Object doesn't support property or method 'find'
Updating the package version using yarn does not seems to change the version shown in the error messages. And the version does not actually match the packages installed (tried with 5.14.5 and also 5.16.2)

Open download portal (using fine-uploader)

what does this mean?


studyfile: “Comment[handedness]” -> “Characteristics[handedness]”

Need specification to identify Ambidextrous on the BIDS side. Otherwise values should be Left, Right, Ambidextrous.

Commented on BIDS draft, asking for spelled out labels.


Foreman configuration tag prep

Pulled these out of #2413

refactor / sanitized the yaml
add_ids has changed to not require a hash passed in. following through with previous refactoring
add impi_present

/cc @gmcculloug @brandondunne

LGTM

:+1:


#456 Elasticsearch doc manager should accept multiple hosts

Added support for multiple Elastic hosts to be supplied. Thus, ensuring HA of mongo-connector with regards to Elastic clusters

The changes look good. The only thing I'm concerned about is how to test TestElasticCluster. The travis file launches a single Elasticsearch instance so TestElasticCluster isn't actually running any tests against a cluster. @llvtt any thoughts?

It doesn't look like Travis supports running a full-blown Elastic cluster. It sets up Elastic in the "default" configuration, which is all we get.
We could spin up multiple Elastic nodes and run some tests connected only to a single node and others to all nodes, but that's probably more complicated than it's worth.
Alternatively, we could simply test that the Elastic Doc Manager can be created connected to multiple hosts (maybe assert something about doc_manager.elastic.transport.hosts), and test only against a single node. We assume that the Elastic client handles connections to multiple nodes transparently.

If it's not necessary to test against 2 elastic nodes we can replace your changes with a test that only checks that we can pass multiple hosts to elastic. Something akin to:
TestElasticMultipleHosts(unittest.TestCase):
    def test_multiple_hosts(self):
        elastic_doc = DocManager(elastic_nodes)
        self.assertEqual(len(elastic_doc.transport.hosts), 2)

I like that approach, @ShaneHarvey.

I'm going to merge this in. The tests are failing because of #21. We should document the new sniff_* connection defaults on the next release.


docs(component): fix broken link to progress bar GitHub docs in READM…

…E.md
Fix broken link to progress bar github documentation in README.md. The previous link was pointing to [https://flowbite-react.com/docs/components/progress-bar], but now it points to the correct URL: [https://www.flowbite-react.com/docs/components/progress].

[ ] I have followed the Your First Code Contribution section of the Contributing guide

Summarize the changes made and the motivation behind them.
I was going to use the Progress Bar & refered to that component's link in the github docs only to find it was a broken 404 link.
Reference related issues using # followed by the issue number.
N/A
If there are breaking API changes - like adding or removing props, or changing the structure of the theme - describe them, and provide steps to update existing code.
N/A

LGTM


Hardcoded use of <IP_ADDRESS> nameserver causes go mod tidy to fail

When importing a project hosted on an internal GitHub enterprise server go mod tidy fails due to use of <IP_ADDRESS> nameserver rather than system DNS settings.
What version of Go are you using (go version)?

$ go version
go version go1.14 darwin/amd64

Does this issue reproduce with the latest release?
Yes, I'm currently running the latest release, 1.14.
What operating system and processor architecture are you using (go env)?
macOS Catalina
Version 10.15.3
go env Output
$ go env
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"

What did you do?

go mod init
go mod tidy

What did you expect to see?
I expected a fully formed go.mod file to be created with current dependencies and versions defined.
What did you see instead?
Rather than using system-defined DNS servers for host name resolution it appears that go mod tidy has a fixed reference to the <IP_ADDRESS> nameserver.  <IP_ADDRESS> is not listed as a resolver, confirmed by scutil --dns.  git.[internal].com (true domain is masked in this post) resolves just fine with other applications (e.g. ping, go get, host)

    git.[internal].com/[project]<EMAIL_ADDRESS>verifying module<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>410 Gone
    server response: not found<EMAIL_ADDRESS>unrecognized import path "git.[internal].com/[project]" (https fetch: Get https://git.[internal].com/[project]?go-get=1: dial tcp: lookup git.[internal].com on <IP_ADDRESS>:53: no such host)

go get git.[internal].com/[project] works just fine as well.

I think that message is actually from the Go module proxy (as indicated by server response:), not from go mod tidy.  Check out this documentation: https://golang.org/cmd/go/#hdr-Module_configuration_for_non_public_modules

Yup, that resolved it.  Makes sense given that there wouldn't be a checksum in sum.golang.org.  I appreciate the help!


Contains Condition is Case-Sensitive

Suggestion to convert strpos to stripos if possible. Thanks!

+1 for me.


Awscli autocomple is not working

Hi, I am using prezto with zsh 5.2 on OSX. When i install awscli using homebrew, they copy an autocomplete file to /usr/local/share/zsh/site-functions/_aws. But look like prezto not source it by default, i need to manually run source /usr/local/share/zsh/site-functions/_aws to make it working.
I already enable module completion, other command like git and brew have autocomplete correctly.
Can you help me see where is the problem?

add the following line to your ~/.zshrc file,
fpath=(/usr/local/share/zsh/site-functions $fpath)
and then execute the following commands,
% rm -f ~/.zcompdump; compinit

Tried it but it isn't working. I already have /usr/local/share/zsh/site-functions in my fpath, other completion in that folder works fine except for aws.

Submit it to https://github.com/zsh-users/zsh-completions.


defineI18nConfig is not defined in i18n.config.ts

Environment

Operating System: Windows_NT
Node Version:     v16.17.0
Nuxt Version:     3.2.2
Nitro Version:    2.3.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      app, modules, i18n
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-ezxqtb
Describe the bug
defineI18nConfig is not defined in i18n.config.ts
Additional context
No response
Logs
[nuxt] [request error] [unhandled] [500] defineI18nConfig is not defined                                                               16:36:37
  at eval (./i18n.config.ts:1:217)
  at ViteNodeRunner.directRequest (file://./node_modules/vite-node/dist/client.mjs:348:11)
[nuxt] [request error] [unhandled] [500] defineI18nConfig is not defined                                                               16:36:37
  at eval (./i18n.config.ts:1:217)
  at ViteNodeRunner.directRequest (file://./node_modules/vite-node/dist/client.mjs:348:11)
[nuxt] [request error] [unhandled] [500] defineI18nConfig is not defined                                                               16:36:37
  at eval (./i18n.config.ts:1:217)
  at ViteNodeRunner.directRequest (file://./node_modules/vite-node/dist/client.mjs:348:11)

Thank you for your reporting!
Sorry, defineI18nConfig is still only available in the edge channel version.
Could you try to edge channel version, please?
https://github.com/nuxt-modules/i18n#install-edge-version

Thank you for your reporting!
Sorry, defineI18nConfig is still only available in the edge channel version. Could you try to edge channel version, please? https://github.com/nuxt-modules/i18n#install-edge-version

If I use objects without methods, as in the following example, there will also be errors
export default {
legacy: false,
locale: 'en',
messages: {
en: {
welcome: 'Welcome'
},
fr: {
welcome: 'Bienvenue'
}
}
}
errors: [nuxt] [request error] [unhandled] [500] (r.default || r) is not a function

Hi!
I've tried to reproduce your stackbliz codes + edge channel version.
Unfortunately, I could not reproduce this issue.
Could you try to use edge channel version, please?

Using the edge version can solve the above problem, and no errors will be reported whether using objects or methods.

I am happy to hear it from you.
Happy coding!

I have the same issue. I migrated the package to the edge version npm:@nuxtjs/i18n-edge. I got the following error when reinstalling the dependencies using yarn.
`
ERROR (0 , _debug.default) is not a function
at node_modules/@nuxtjs/i18n/dist/module.mjs:78:36
at evalModule (node_modules/jiti/dist/jiti.js:1:255106)
at jiti (node_modules/jiti/dist/jiti.js:1:253034)
at requireModule (node_modules/@nuxt/kit/dist/index.mjs:2244:26)
at loadNuxtModuleInstance (node_modules/@nuxt/kit/dist/index.mjs:2435:90)
at async installModule (node_modules/@nuxt/kit/dist/index.mjs:2408:47)
at async initNuxt (node_modules/nuxt/dist/index.mjs:3238:7)
at async loadNuxt (node_modules/nuxt/dist/index.mjs:3314:5)
at async loadNuxt (node_modules/@nuxt/kit/dist/index.mjs:2537:19)
at async Object.invoke (node_modules/nuxi/dist/chunks/prepare.mjs:31:18)
at async _main (node_modules/nuxi/dist/cli.mjs:49:20)
error Command failed with exit code 1.
Operating System:Ubuntu 23.04 Node Version: v18.17.0
Nuxt Version: 3.2.2
Nitro Version: 2.3.3
Package Manager<EMAIL_ADDRESS>Builder: vite

I have the same issue. I migrated the package to the edge version npm:@nuxtjs/i18n-edge. I got the following error when reinstalling the dependencies using yarn.
ERROR (0 , _debug.default) is not a function

  at node_modules/@nuxtjs/i18n/dist/module.mjs:78:36
  at evalModule (node_modules/jiti/dist/jiti.js:1:255106)
  at jiti (node_modules/jiti/dist/jiti.js:1:253034)
  at requireModule (node_modules/@nuxt/kit/dist/index.mjs:2244:26)
  at loadNuxtModuleInstance (node_modules/@nuxt/kit/dist/index.mjs:2435:90)
  at async installModule (node_modules/@nuxt/kit/dist/index.mjs:2408:47)
  at async initNuxt (node_modules/nuxt/dist/index.mjs:3238:7)
  at async loadNuxt (node_modules/nuxt/dist/index.mjs:3314:5)
  at async loadNuxt (node_modules/@nuxt/kit/dist/index.mjs:2537:19)
  at async Object.invoke (node_modules/nuxi/dist/chunks/prepare.mjs:31:18)
  at async _main (node_modules/nuxi/dist/cli.mjs:49:20)

error Command failed with exit code 1.

Operating System: Ubuntu 23.04
Node Version: v18.17.0
Nuxt Version: 3.2.2
Nitro Version: 2.3.3
Package Manager<EMAIL_ADDRESS>Builder: vite

It fixed by removing node_modules folder and reinstalling dependencies again.


Wma/constant propagation

fixes #233

Codecov Report
Patch coverage: 76.92% and project coverage change: -0.76 :warning:

Comparison is base (381dff9) 79.56% compared to head (ad39eea) 78.80%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #236      +/-   ##
==========================================
- Coverage   79.56%   78.80%   -0.76%
==========================================
  Files          68       68
  Lines        5568     5568
==========================================
- Hits         4430     4388      -42
- Misses       1138     1180      +42

Impacted Files
Coverage Δ

src/FinchNotation/instances.jl
63.38% <71.42%> (+3.95%)
:arrow_up:

src/symbolic/simplify_program.jl
78.72% <75.00%> (-0.45%)
:arrow_down:

src/lower.jl
96.65% <100.00%> (ø)

src/virtualize.jl
95.45% <100.00%> (+2.43%)
:arrow_up:

... and 8 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Characters get stuck on screen if no response

If the prompt errors or doesn't generate a response, the character's image gets stuck on the overlay until somebody redeems something else

An easy way to replicate this is adding a TTS only character as AI, they seem to have a higher chance of not generating a response at all.
Note: although the char's image gets stuck screen, the request will disappear from the control panel.


added tests for code that was not covered initially

Adding and refining tests, as well as refactoring code before adding new features.

Codecov Report

Merging #9 (9747cf7) into master (b350930) will increase coverage by 6.37%.
The diff coverage is 100.00%.

:exclamation: Current head 9747cf7 differs from pull request most recent head 94bc582. Consider uploading reports for the commit 94bc582 to get more accurate results

@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
+ Coverage   72.52%   78.89%   +6.37%
==========================================
  Files           9       11       +2
  Lines        1321     1355      +34
==========================================
+ Hits          958     1069     +111
+ Misses        363      286      -77

Impacted Files
Coverage Δ

pluto_python/mhd_jet.py
76.10% <100.00%> (+18.31%)
:arrow_up:

pluto_python/tools.py
83.33% <100.00%> (+39.33%)
:arrow_up:

tests/test_calculator.py
100.00% <100.00%> (ø)

tests/test_mhd_jet.py
100.00% <100.00%> (ø)

tests/test_pluto.py
100.00% <100.00%> (ø)

tests/test_tools.py
100.00% <100.00%> (ø)

pluto_python/spacetime.py
0.00% <0.00%> (ø)

setup.py
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b350930...94bc582. Read the comment docs.


Nuxt Firebase Config results in bunch of errors

I've followed instructions from https://firebase.nuxtjs.org/guide/getting-started on getting started and setting up a nuxt project with firebase. After doing everything as recommended, it results in a bunch of errors and warnings and crashes the main JS thread. This is likely a problem with Nuxt's official Firebase library.
Repository for reproduction: https://github.com/bacon-delight/nuxt-firebase-errors
Versions

nuxt: 2.14.6
node: 12.18.0

Reproduction

Additional Details

Steps to reproduce
npm run dev > Check console
What is Expected?
Project should run normally without any errors or warnings
What is actually happening?
Bunch of errors and warnings, most likely main JS thread crashing due to some issues in nuxt-firebase

@bacon-delight I appreciate the time you've taken to make this report. Is there any chance you could make it over at https://github.com/nuxt-community/firebase-module instead?


v0.4.0 docs update

What kind of change does this PR introduce?
Docs update

Coverage remained the same at 100.0% when pulling 79584a1b7306a2f80f16d684e03287bfec2ac073 on v0.4.0-docs into 375b82e33877cdb1bd3bf768b19434fda2db9816 on master.


Allow item class to be changed manually

This would be useful for other languages that could have inconsistent translations between the game and the API. One example is the korean crossbow.

Related to #326


Need support for favicon

While trying out<EMAIL_ADDRESS>to build piral instance
I noticed my output index.html is missing the favicon
Notes:
It seems we can add https://github.com/jantimon/favicons-webpack-plugin to support this use case

Here is my proposal from looking at the code.
Update in https://github.com/FlorianRappl/piral-webpack-tools/blob/master/packages/html5-entry-webpack-plugin/src/index.ts

From a 'zero config' standpoint, the piral instance dev just needed to add logo.png to their webpack context folder.

  const FaviconsWebpackPlugin = require('favicons-webpack-plugin')
  ...
  apply(compiler: Compiler) {
    const entry = compiler.options.entry;
    const [template] = getTemplates(entry);

    if (template) {
      const src = dirname(template);
      const templateContent = load(readFileSync(template, 'utf8'));
      const entries = extractParts(templateContent).map(entry => join(src, entry));
      const plugins = [
        new HtmlWebpackPlugin({
          templateContent: templateContent.html(),
        }),
        new FaviconsWebpackPlugin(),
      ];

      setEntries(compiler.options, template, entries);
      plugins.forEach(plugin => plugin.apply(compiler));
    }
  }

Full solution (not sure if this will work)

  const FaviconsWebpackPlugin = require('favicons-webpack-plugin')
  ...
  apply(compiler: Compiler) {
    const entry = compiler.options.entry;
    const [template] = getTemplates(entry);

    if (template) {
      const src = dirname(template);
      const templateContent = load(readFileSync(template, 'utf8'));
      const entries = extractParts(templateContent).map(entry => join(src, entry));
      const plugins = [
        new HtmlWebpackPlugin({
          templateContent: templateContent.html(),
        }),
      ];

      if (entry.endsWith('favicon.ico') {
         plugins.push(new FaviconsWebpackPlugin({
            logo: entry,
            prefix: publicUrl // need to get this from somewhere
         }))
      }

      setEntries(compiler.options, template, entries);
      plugins.forEach(plugin => plugin.apply(compiler));
    }
  }

I also encountered a similar scenario, which needed to be completed in collaboration with html-webpack-plugin. At present, I found one thing in common, that is, most plug-ins that need to work with html hooks. We can do this to share hooks:
import { dirname } from 'path';
const HtmlWebpackPlugin = require(require.resolve('html-webpack-plugin', { paths: [dirname(require.resolve('html5-entry-webpack-plugin'))] }));
// ...

And then push a pr to favicons-webpack-plugin to support the HtmlWebpackPlugin option, such as:
new FaviconsWebpackPlugin({
  HtmlWebpackPlugin,
  logo: entry,
  prefix: publicUrl // need to get this from somewhere
})

This is more flexible, hope that helps.

Sounds like a good idea, but it depends heavily that the favicons-webpack-plugin maintainer(s) accept this PR. I am no sure here (but I have no clue, maybe they are indeed very open about it...).
We could give it a try. Thanks for the great idea @carvinlo !

With the latest change the favicons-webpack-plugin will just work.


Simplify app view experience

Currently, Appsmith experience for an end-user is exactly same as what it is for a developer. However, organisations often want to a much simpler experience for end-users. Lot of features/ information that is relevant for developers typically is not relevant for end-users and often results in confusion when they are exposed to this confusion. The idea is to simplify end-user experience when they sign-up for Appsmith.

[ ] Figure our the current issues
[ ] Look for edge cases and other tools
[ ] How will GAC affect this?
[ ] Design changes

Here is the design research deck : https://www.figma.com/file/JlcP5XIvUuxZranbLibMRr/Viewer-Mode-Scenario%2FFlow?type=whiteboard&node-id=0%3A1&t=46M8tI5aHBSaWehn-1

Closing in favor of #23386


Add support for value localization + several bugfixes and refactorings

These changes add support for value localization, and also fixes several bugs around the multi-select dropdown relating to the asynchronous row pipeline.

Looks like the version of ngx-grid-core package needs to be updated.

This is pushed now, looks like I did update it but didn't push the change 😆


Add time-span highlight support. Demo with RasterPlotView.

Addresses SortingView issue #182 (https://github.com/magland/sortingview/issues/182)
This PR adds functionality to handle spans of intervals that are to be highlighted in the data. As detailed in comments to the above issue, these spans are expected to be defined as a list of start times and end times connected by index, so that each span begins at spans[0][n] and ends at spans[1][n]. Arbitrarily many span-sets can be supported for any data source, and span highlight color can be left as default or set to a different color per set of spans.
One feature to call out here: if the user specifies a particular alpha, that'll be used throughout; otherwise I set the alpha to 0.5 (half-transparent) for spans that are wider than 2 pixels, and 1 (full visible) for spans smaller than that. I found that otherwise it was either too hard to see small highlight regions, or things were too opaque when zooming in.
Spans do need to be added to each time series separately. It might be possible to add an overall set of spans to a composite view, but this would require some more complex contextual logic & would come at the expense of supporting spans in individual components.
Logic to handle spans, along with defining their data types, has been incorporated as part of the TimeSeriesView component. It would otherwise need to be repeated across each component type, and I'd rather define it centrally.
If we like this, it can be added to any other TimeScrollView-based component with just a couple lines to update the data type and pass the values in.
Demo: https://www.figurl.org/f?v=http://localhost:3000/&d=3efdf652e24633ba8345fb6a02af9d01fc67e5d1&channel=flatiron1&label=SPANS_TEST

More zoomed in:

Looks great! Merging.


Data sharding and Read-write splitting  throw Exception together.

version:3.0.0.M1
YML

Exception

Data sharding is failure when Data sharding and Read-write splitting were used together

Please provide more details for sql.show

By the way, for report a bug, we prefer you reference the issue template, and fill all required information.

Which version of Sharding-Sphere do you using?
3.0.0.M1
Expected behavior
Data sharding and Read-write splitting work well together.
Actual behavior
throw Exception that is org.springframework.jdbc.BadSqlGrammarException
Reason analyze
SQL：INSERT INTO t_order (order_id,user_id) VALUES (?,?)
but t_order is logic table.
the real table is t_order_0 and t_order_1.
so data sharding may not work.
File
https://github.com/naah69/sharding-jdbc-3-demo/tree/master/sharding-jdbc-rw-splitting-data-sharding
this is that code.

i had resolved this problem.
now i will explain my problem:
Read/Write Splitting doc:
http://shardingsphere.io/document/current/cn/manual/sharding-jdbc/usage/read-write-splitting

but this config should be configuration when  Read/Write Splitting was in use alone.
SpringBoot Config doc:
http://shardingsphere.io/document/current/cn/manual/sharding-jdbc/configuration/config-spring-boot/

this is real correct config when Data sharding and Read-write splitting work together.
i will upload some demo on my github.
if u are in trouble，we can discuss something about sharding-jdbc

Thank you for your reply.


Set up pre-commit for black formatting

Fulfills issue #82
Also adds pre-commit to required packages for conda install (from source only)

I think we should probably note somewhere that you must run pre-commit install before it will actually start running on commits, but otherwise it works for me and looks good.

Good point on the pre-commit install. I've added a line the the contributing.md file to instruct users on how to do this

Happy to merge this now!


Correct appeal type input

Change description
Correct appeal type input
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Changes merged in master already


Update CLI usage when using glob patterns.

Globs need to be quoted or they will be expanded by your shell (and not standard). This means that explicitly matching the default patterns (standard **/*.js **/*.jsx) will likely result in a different result than just relying on the defaults.

See https://github.com/Flet/standard-engine/issues/84 for confusion arising from this.

Thanks!


[WIP] framac: 18 -> 19 and update why3

Motivation for this change
Update framac to 19 (being released this Friday), clean up why3 and make sure it works well with framac, and add why3WithProvers to automatically configure provers for why3.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Looks pretty good at a first glance! I think investigating the plugin setupHook would also be very useful and easy to add, but feel free to follow up with a separate PR, etc for that, if you like.

@bobot I'm not so familiar with external Frama-C plugins, but I'll take your word for it and throw it in. Not sure how to test it though.

Right now it's not, but it feels like that's the right way to do it. Instead I just make sure to install the provers as well so they're in the path. How would we go about making why3 use absolute paths? I feel like we could either replace relative paths in the config or we could generate the config by hand. Thoughts?

Right, but how do we get those hardcoded paths into the config file?

I think whether you prefer to use substituteInPlace on the config file or write a bespoke config using pkgs.writeText and then wrap the executable with makeWrapper is up to you, honestly. If the configuration isn't especially complicated I'd probably use pkgs.writeText as it's a little less fragile, but you'd have to play with it.
One question is, can why3 load/merge multiple config files? I.e. if I point it to load provers.conf (containing a bunch of hardcoded absolute store paths), can I also load $HOME/.config/why3.conf (or whatever) too, and have the values merged? That would be the ideal case: we could just write out part of a config file that specifies the solver paths, and let other config options default/use what the user specifies.

I think whether you prefer to use substituteInPlace on the config file or write a bespoke config using pkgs.writeText and then wrap the executable with makeWrapper is up to you, honestly. If the configuration isn't especially complicated I'd probably use pkgs.writeText as it's a little less fragile, but you'd have to play with it.

I agree. I really don't know enough about the why3 configuration language though, and how smart why config --detect-provers is. So I'll see if I can rig up something with pkgs.writeText, but I'm curious if you guys know more about why3 and have opinions about if that's feasible/robust or not.

One question is, can why3 load/merge multiple config files? I.e. if I point it to load provers.conf (containing a bunch of hardcoded absolute store paths), can I also load $HOME/.config/why3.conf (or whatever) too, and have the values merged? That would be the ideal case: we could just write out part of a config file that specifies the solver paths, and let other config options default/use what the user specifies.

I think that using the --extra-config argument, as François suggested, takes care of this. I haven't tested it with another config, but it gives the following reassuring message when I run why3:
[Config] reading extra configuration file /nix/store/...-why3-1.2.0-with-provers-conf/share/why3/why3.conf

The user still need to run why3 config --detect as usual, but nothing else fancy.

Hmm, one of my goals was to avoid having the user run why3 config --detect. This is because if you are installing different versions of why3, you run into problems if you forgot to delete and regenerate your config, and I wanted nix to handle that for you.

I would have put them directly in the why3-config binary (src/driver/autodetection.ml), but I forgot that why3 is just a dependency of the second package.

Although I'd like to avoid it, we can still modify the why3 that is passed to us with some kind of patch that we generate, forcing it to be recompiled from source. In some ways that's actually cleaner.

Okay, I think this is good to go. I managed to substitute for absolute paths in the why3 using a generated awk script, and I think it works, so now I don't have to install the provers. I've squashed everything into one commit.

@vbgl Sorry for the delay, I think I've addressed all of your comments.

@GrahamcOfBorg build framac


Propagate 328 (JHU population fix and function refactor) to main

Oops, meant to find a way to see the conflicts instead of that.

Checked out the conflict. The incoming change is the correct one, the rest was refactored into functions.


ProGuard for Extensions

Resolves #2047
---
This PR implements ProGuard, a Java shrinker, optimizer, obfuscator, and preverifier. With ProGuard, extensions will be smaller and the source code will be obfuscated, making reverse engineering harder.
As previously discussed, this will not be enabled by default, it is a flag that goes with ant extensions. Currently, only the needed files are included in the changes (plus README and LICENSE), excluding source code ones. It uses ProGuard 6.2.2, the latest version.
As you can see in the implementation, only the built .jar for the extension is obfuscated, not the entire AndroidRuntime.jar. After generating the corresponding .jar in jarExtension for a extension, a call is made to the ProGuard task, which generates another .jar file, deletes the old one and renames the new one to the name of the old one (so "it is like the program generates the file with ProGuard already").
By checking the .aix file, it is clear that the extension has been obfuscated and shrunk with ProGuard.

Usage

ant extensions -Dproguard=1: will build extensions and add the extra-step for ProGuard
ant extensions -Dproguard=* or ant extensions: will build extensions without changes

Tests with ProGuard
Tests using an extension without libraries were made using the ColinTree ListView extension.

[x] Build extension without libraries
[x] Import extension into MIT App Inventor 2
[x] Test extension in Companion
[x] Test extension in real APK
[ ] Repeat with libraries

Results
Size Optimization

W/O ProGuard
W/ ProGuard
Improvement

Vishwas' Dialogs Extension
21,651 bytes
14,304 bytes
33.94 %

ColinTree's ListView Extension
49,166 bytes
32,250 bytes
34.41 %

Obfuscated Code

Note: at Kodular, we have added more customization to the proguard.cfg file, as we are also obfuscating the AndroidRuntime.jar. My first approach was using exactly the same file (including a dictionary for variable renaming), but then realized it was "probably" not going to be needed, so decided to keep it simple.
As you can see in the screenshot, all variables get obfuscated and code minified, keeping only the function and main class names, needed to call the component. In the test made with ColinTree extension, this did not cause issues.

Tested, and ready for review


Allow environment variable interpolation from the HCL configuration

As @razaj92 suggested in #14, it could be interesting to be able to interpolate environment variables within HCL configuration itself:

Ideally we could allow variable interpolation with in the field itself
eg:
workspace {
 name = "my_app-${var.env}"
}

this been done by @razaj92 in #17!


Minimum coins required

Hi !!
I am interested in this project as GSSOC '21 participant.
And I would add the problem - minimum coins needed  in C++
@tarun26091999 Please assign me with this issue.
Thanks.

@AasthaGoyalgit @chitrasingla solve the problem specified in the issue name.

@tarun26091999 I have made PR .Please review it and let me know if any changes are required.
Thanks!!

Hello! I am a GSSoC'21 Participant. I would like to work on this issue. Would be highly obliged if you could assign it to me. I will be using Java. Thanks!

Hey!! I am interested in this issue for adding DP problems and solutions in C++ as a GSSOC'21 participant. I'll be glad to work on this issue. Thanks!!

I would like to work on this issue in python. Kindly assign it to me @plazzy99  @raksha009  @tarun26091999

@Aarti002 you may jump from #806 to here

@tarun26091999 I have made PR .Please review it and let me know if any changes are required.
Thanks!!

link it here

@tarun26091999 I would like to work on this issue in C++. Please assign it to me.


Closes #17 Dynamically create inner schemas derived from spec schemas

Updated schema generation logic, improved tests.

I'm trying to generate a client for the OpenAI API (mainly focusing on Completions, Chat and Embeddings endpoints for now).

@davidmigloz - can you try this latest version of the generator on main (as of this PR) and see how it fairs? The only thing remaining is #13 and I think that would result in a completely type safe Dart client!


RUMM-1228: Reduce test coverage flakiness

What does this PR do?
One more round to reduce flakiness is some most noisy tests. Mostly switching to ParametrizedTest for tests, where set of input parameters is finite.
Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #800 (8705214) into master (7319670) will increase coverage by 0.19%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #800      +/-   ##
============================================
+ Coverage     88.93%   89.11%   +0.19%
- Complexity     1843     1855      +12
============================================
  Files           196      196
  Lines          6438     6438
  Branches        748      748
============================================
+ Hits           5725     5737      +12
+ Misses          486      478       -8
+ Partials        227      223       -4

Impacted Files
Coverage Δ

...ernal/instrumentation/gestures/GesturesListener.kt
89.15% <0.00%> (-1.55%)
:arrow_down:

...droid/rum/tracking/FragmentViewTrackingStrategy.kt
84.78% <0.00%> (ø)

...roid/rum/internal/domain/scope/RumResourceScope.kt
94.08% <0.00%> (+0.66%)
:arrow_up:

...id/rum/internal/domain/event/RumEventSerializer.kt
91.67% <0.00%> (+1.04%)
:arrow_up:

...g/internal/domain/event/DdSpanToSpanEventMapper.kt
100.00% <0.00%> (+3.85%)
:arrow_up:

...l/net/info/BroadcastReceiverNetworkInfoProvider.kt
93.81% <0.00%> (+10.31%)
:arrow_up:

SHouldn't this branch/commit reference RUMM-1858 instead ?

@xgouchet I didn't go for the full RUMM-1858 reference, because I wanted to see if using parametrized tests helps us and also full RUMM-1858 requires much more files to be changed.
So I decided just to try to reduce flakiness for some files which give the noise often and since this PR also includes file where reduce it achieved without parametrized tests, I decided to reference RUMM-1228 again.
Full implementation of RUMM-1858 will come later.


genome size

Dear 3D-DNA users,
I used 3D-DNA to assemble a plant genome. the initial genome size was 1.6G, after using 3D-DNA pipeline it became 1.4G. I want to know why this happened and where is the 0.3G data?
Thanks a lot!

Hello wj1106,
Are you running in diploid or haploid mode? Diploid does not preserve total sequenced bases count. If you are in haploid, what are you counting, the total length or the sequenced based? The total length may change due to gaps, while the total sequenced bases should remain the same. You can use the supp/print-n50.awk to count the total sequenced bases. Note that earlier versions of 3D-DNA substituted ambiguous bases for Ns, which could also lead to difs.
Best,
Olga


[SPA] 6.0 docs need a general update

The docs point to the "legacy" way of doing things with the old Spa.Extensions library.
We revamped the way this works in 6.0 to leverage the same architecture followed by frontend frameworks to integrate with backends. The docs for angular and react should reflect that and cover the common use cases.
There is content here with details on how this works.
We should cover the following:

How it works in development. (Frontend Dev Proxy proxies requests to ASP.NET Core)
How it works in production. (Apps are just static files sitting in your wwwroot folder).

The important files on the ClientApp:

proxy.conf.js (angular) to configure the webpack dev proxy (link to https://github.com/chimurai/http-proxy-middleware#options for all options)
setupProxy.js (react) to configure the webpack dev proxy (same as angular).

Development modes:

Launch frontend and backend simultaneously by launching the backend (as you are used to)
Launch frontend and backend separately (by launching the frontend manually with npm run start) and avoid having to start the frontend server every time.

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

ID: 32a18270-91cd-2a02-2eb8-2c8b1ed03737
Version Independent ID: c58d57e5-a0c2-201d-f6dd-076ad5971fcb
Content: Use the Angular project template with ASP.NET Core
Content Source: aspnetcore/client-side/spa/angular.md
Product: aspnet-core
Technology: aspnetcore-clientside
GitHub Login: @SteveSandersonMS
Microsoft Alias: stevesa

@javiercn Does the new approach support prerendering/SSR? We have a couple of sections in the 5.0 version for Angular that discuss providing data and SSR drawbacks, but I'm not sure if those will still be relevant for 6.0.


Warn about usage of non-breaking space

Reference issue: #85
What changed in this PR:
This PR adds a warning about usage of non-breaking space in a template. Regarding this issue, a unit test is added.

This issue could be addressed also by creating a new Token for non-breaking space and Melody would automatically throw a warning like, Invalid Token Expected symbol but found NonBreakingSpace instead.

Question: Do we also want to cover other unicode sequences which represent whitespace?
Resources:
https://codepen.io/rubikzube/pen/wolgB
http://xahlee.info/js/js_whitespace.html
https://mathiasbynens.be/notes/javascript-escapes
https://mathiasbynens.be/notes/javascript-unicode

Question: Do we also want to cover other unicode sequences which represent whitespace?
Resources:
https://codepen.io/rubikzube/pen/wolgB
http://xahlee.info/js/js_whitespace.html
https://mathiasbynens.be/notes/javascript-escapes
https://mathiasbynens.be/notes/javascript-unicode

I believe it is a good idea to address all unicode sequences which represent whitespace. It prevent the future bugs/bug reports regarding these type of issues.
We can also follow Babel example and use something similar to https://github.com/xtuc/charcodes/tree/master/packages/charcodes and use charCodeAt. Or just add the characters and address them in the appropriate function. One benefit of using charCodeAt + the separate package would be to target characters that cannot be easily read or shown.

Do what you think is best. :)


refactor: update style, simplify code

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


Peek's default binding disables input witching with Microsoft's Japanese IME

Microsoft PowerToys version
0.70.0
Installation method
Chocolatey
Running as admin
No
Area(s) with issue?
Peek
Steps to reproduce

Set your windows keyboard to "Japanese - Microsoft IME"
Try changing between hiragana input and romaji input in a text field using control + space key
Observe that the input type will not change.
Disable the peek extension.
Try changing between hiragana input and romaji input in a text field using control + space key
Observe that the input type will change.

✔ Expected Behavior
When not focused on a Windows Explorer file, input switching should still work.
❌ Actual Behavior
Input switching does not work.
Other Software
No response

I wasn't sure if I should report this in localization or bugs as it should only affect people using Microsoft's Japanese IME (or possibly other IMEs that share the same binding).

I do wonder... why Ctrl+Space, and with no option to remap? What was wrong with just using Space while on File Explorer?

/dup #26150


build and install kops from source with patch removing req on flannel…

… plugin
Issue #, if available:
Description of changes:
Apply patch to kOps to remove flannel plugin from hardcoded list of CNI plugins.
Do this by cloing kops from the desired tag, patching it and installing it, rather than just curling the release version.
ran ./install_requierments.sh to test
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


ca-ns

http://www.novascotia.ca/snsmr/access/land/land-services-information/ask-data.asp

@geobrando I have a long running conversation with several of the people in that department. I've also contacted the premier several times to petition for opening up this data
Dear Mr. Ingalls,

Thank you for correspondence of May 7th, 2015, to Premier Stephen McNeil, regarding the OpenAddresses Project.  The Premier has asked that I respond on his behalf.

The Nova Scotia Civic Address File, or NSCAF is the authoritative source for civic addressing in Nova Scotia.  The NSCAF includes several products that are available to stakeholders and the user community, including a street network with address ranges, civic address points, community boundaries, and municipal boundaries.  These datasets are available with an unrestricted license with the exception of the civic address points. Our unrestricted geographic data is available for download at www.GeoNOVA.ca in the GeoData section.

The restricted licensing currently assigned to the civic address point dataset is under review.  We regularly review the licensing assigned to all of the geographic data products that we distribute and we include jurisdictional scans as part of the review (for e.g. what licensing does British Columbia or New Brunswick use for distributing civic address points, etc.).  These reviews determine the distribution license associated with a dataset.  The review for the civic address point dataset, including a decision regarding licensing will be conducted during June.

If you are interested in additional information regarding civic addressing in Nova Scotia or any other geographic information-related product, please contact Colin MacDonald, Director of Geographic Information Services at<PHONE_NUMBER> or<EMAIL_ADDRESS>
Again, thank you for your interest and inquiry regarding the Nova Scotia Civic Address File.

Sincerely,

Labi Kousoulis
Minister of Internal Services

c.   Honourable Stephen McNeil, Premier

Currently waiting on  response to a followup email I sent about the results of the review.

closing as per: http://results.openaddresses.io/sources/ca/ns/province


Update base image and run as non-root user

We should do this: https://github.com/ranking-agent/aragorn-ranker/commit/2944d9cc3fec65485cdd17cd5745eae995af0aa1

Closed by #31


Document Vector2.dot() and Vector3.dot() more extensively

These methods are commonly used in games. It's time to make the documentation more explicit about them :)

Isn't this more documenting the math, rather than the engine's function?

It explains the purpose of the function and its possible outputs. I think this kind of documentation is great.

I think the explanation should mention that when the dot result is 1 then the two vectors are aligned and -1 for when they're in opposite directions. Explanations for dot product for game development usually start with 1, 0, -1 to make it more intuitive and and then talk about the values in between.
See this example: https://www.youtube.com/watch?v=2PrSUK1VrKA
Just saying that 0 -> 90 degrees and more than that is positive doesn't make it explicit enough what the other cases are and how the user should be thinking about making use of it.

@asheraryam The range of -1 to 1 only applies for unit vectors, and therefore it's not the most general behavior of the method. I think it makes sense to first mention values greater/lesser than zero, because non-unit vectors will have a dot product different from 1 when aligned. For example, the dot product of (1, 1) and (1, 1) is 2.

@aaronfranke Sure, but it could still have that extra information at the end, especially as using unit vectors with the dot product is the more common scenario.
So it would become something like

When using unit vectors, the result will always be between [code]-1[/code] (180 degree angle) when the vectors are facing opposite directions, and [code]1[/code] (0 degree angle) when the vectors are aligned.

Needs a rebase on current master to fix CI.

Thanks!

Seems like the cherry-pick is not necessary, this was already added to the 3.2 branch somehow.

@akien-mga That was me, I just copied the master files on top of the 3.2 files (then ran doctool and fixed issues).


v0.4.0

add watch functionality and bid smarter

Coverage decreased (-0.04%) to 93.141% when pulling c9a216dbafe0b9b8136b65b01831441d80ceaa00 on develop into 5380093b490e92c3a2a3ced0577ef281e9767bb9 on master.


add support for defensive referrer override

This is in response to #22.
This will take the defensive approach to overriding the referrer in the page object that we set using window.ga.
We should only override the referrer if the manually passed referrer property inside the .page() call is not the same as document.referrer so that we don't interfere with GA's default page tracking behavior.
@segment-integrations/core

LGTM