KAFKA-4293 - improve ByteBufferMessageSet.deepIterator() performance by relying on underlying stream's available() implementation

also:
provided better available() for ByteBufferInputStream
provided better available() for KafkaLZ4BlockInputStream
added KafkaGZIPInputStream with a better available()
fixed KafkaLZ4BlockOutputStream.close() to properly flush
Signed-off-by: radai-rosenblatt<EMAIL_ADDRESS>

The long-term correct solution to the bug is still a wire format change.
However, shorter term this PR makes the code rely on InputStream.available() to detect EOF without catching an exception. this works because for this particular case the reads are all memory-backed (as opposed to directly from socket) and so all input is at hand - all that's required is careful implementation of available().

Have we verified that this change makes an actual difference outside the profiler? I ask because many profilers are known to suffer from safepoint bias. Reference: http://psy-lob-saw.blogspot.co.uk/2016/02/why-most-sampling-java-profilers-are.html

@ijuma - thank you for the article, it was fascinating, and it may indeed be that our profiling runs were affected.
to gauge the effect of this change i wrote the following "micro-benchmark" (quoted because i didnt bother with a full JMH one):
@Test
def perfTest(): Unit = {
   val batchSize = 1
   var nanosTotal: Long = 0
   var runsTotal: Int = 0
   (1 to 1000000).foreach(i => {
     val messages = (1 to batchSize).map(m => {
       new Message((s"message num $m" + System.currentTimeMillis()).getBytes)
     })
     val messageSet = new ByteBufferMessageSet(GZIPCompressionCodec, messages: _*)      var count = 0
     val start = System.nanoTime()
     messageSet.foreach((offset: MessageAndOffset) => {
       count += 1
     })
     val elapsed = System.nanoTime() - start
     assertEquals(batchSize, count)
     if (i > 100000) { //warm up iterations
       nanosTotal += elapsed
       runsTotal += 1
     }
   })
   val avg = nanosTotal / (runsTotal * 1000000.0)
   println(s"$avg ms on avg")
}

comparing the avg per-betch iteration time between vanilla Kafka and my patch shows it highly depends on the batch parameters.
The most extreme case is small batches and fast compression - batch size 1 + snappy show a ~x4 time improvement. this would be typical of environments where kafka would be used for real time messaging (tiny batches, fast compression).
On the other extreme (batch size 10000 + gzip) there's no noticeable difference.
Also no difference in case no compression was used.

@ijuma - have you guys had time to evaluate this?

@radai-rosenblatt I haven't reviewed the patch in detail, but it seems like it might be worth including. @junrao, what do you think?

@ijuma @junrao I think the patch makes sense - at the very least it avoids the use of exceptions for flow-control. That said, our original toy benchmark was flawed so the savings are not as dramatic as we hoped. Still, it does show some improvement as @radai-rosenblatt noted above.

Had a closer look at the javadoc for available:
* Returns an estimate of the number of bytes that can be read (or
     * skipped over) from this input stream without blocking by the next
     * invocation of a method for this input stream. The next invocation
     * might be the same thread or another thread.  A single read or skip of this
     * many bytes will not block, but may read or skip fewer bytes.
     *
     * <p> Note that while some implementations of {@code InputStream} will return
     * the total number of bytes in the stream, many will not.  It is
     * never correct to use the return value of this method to allocate
     * a buffer intended to hold all data in this stream.
     *
     * <p> A subclass' implementation of this method may choose to throw an
     * {@link IOException} if this input stream has been closed by
     * invoking the {@link #close()} method.
     *
     * <p> The {@code available} method for class {@code InputStream} always
     * returns {@code 0}.
     *
     * <p> This method should be overridden by subclasses.
     *
     * @return     an estimate of the number of bytes that can be read (or skipped
     *             over) from this input stream without blocking or {@code 0} when
     *             it reaches the end of the input stream.
     * @exception  IOException if an I/O error occurs.

So even though I think it makes sense to avoid the exceptions, it seems a bit brittle to rely on that method as an alternative to reading until the end. Yes, we are overriding all the implementations to tighten the contract, but it seems possible that someone would add a new implementation that satisfies the contract and breaks the current expectation.
It would seem that it would be better to have a separate method (and an interface perhaps) that does what we need?

Yes we did notice that detail and it's a valid concern. Your suggestion on having a more explicit interface makes sense. The codecs extend a sealed trait and we could add some protection (in the tests) that all codecs are tested.

....and this is exactly why I think the long term correct solution is a wire format change - declare sizes and counts up front.
how about StrictInputStream as an interface that extends InputStream with method boolean hasMore() ?

An abstract class that extends InputStream may be a better option in that case.

KafkaGZIPInputStream extends GZIPInputStream  <--- i have to do this to get access to a protected field :-(

The other option is to drop this altogether :)
Perhaps the savings aren't sufficient to justify these concerns.

the issue with GZIPInputStream has been fixed in jdk9 (which means my class from this patch could go away at some distant point in the future) - http://bugs.java.com/bugdatabase/view_bug.do?bug_id=7031075

@radai-rosenblatt I've submitted the following patch as part of KAFKA-5150 that should accomplish the same thing with a smaller set of changes, in case you are interested
https://github.com/apache/kafka/pull/2967/commits/97e3ce81cea2736232a9527dc6a8cbd29e366ccd

i'll defer to  @xvrl 'spatch


[IOTDB-1391]  Add a new Aggregation function ext

Description
Add a new aggregation function: ext
Content1 ...
Content2 ...
Content3 ...

This PR has:

[ ] been self-reviewed.

[ ] concurrent read
[ ] concurrent write
[ ] concurrent read and write

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods.
[ ] added or updated version, license, or notice information
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious
for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold
for code coverage.
[ ] added integration tests.
[ ] been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

Could you give an explanation of what ext does?
I used to think that it functions as the max absolute function,
but the test example in the AggregateResultTest.java says that if we have two values 1 and -2, ext will return -2. So I'm confused.

Coverage decreased (-0.06%) to 67.905% when pulling d24c51e76aad45c615ce1fef2ae56a4814257192 on lyssom:master into 8b1ee93c51932d72cb7fd7c5268f40f7b3a74f25 on apache:master.


gate failure cause all CI failed

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened:
something wrong for all CI test like this
https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/cloud-provider-openstack/2527/openstack-cloud-controller-manager-e2e-test/1750554290813931520
ovisioner-84db5d44d9-b7s5x    0/1     Pending             0          5m23s   <none>      <none>       <none>           <none>",
        "coredns-6799fbcd5-p7m2z                    0/1     Pending             0          5m23s   <none>      <none>       <none>           <none>",
        "openstack-cloud-controller-manager-rhpgn   0/1     ImageInspectError   0          2m24s   <IP_ADDRESS>   k3s-master   <none>           <none>",

What you expected to happen:
How to reproduce it:
Anything else we need to know?:
Environment:

openstack-cloud-controller-manager(or other related binary) version:
OpenStack version:
Others:

https://github.com/k3s-io/k3s/issues/1857  seems has invalid capacity 0 on image filesystem as well
and found the image is actually built , so need inspect what's the reason here
docker buildx build --output type=registry \
	--build-arg VERSION=v0.0.99 \
	--tag localhost/openstack-cloud-controller-manager:v0.0.99 \
	--platform linux/amd64 \
	--target openstack-cloud-controller-manager \
	.

I built image and run locally successfully ,so not sure what happened in CI gate ..
        image: <IP_ADDRESS>/openstack-cloud-controller-manager:v0.0.99

I think the cause is Mirantis/cri-dockerd#299. Nevertheless, as mentioned on the ticket, nobody's testing K8s with Docker anymore and there's no point in us doing that. My fix in #2529 solves this by switching to containerd as a container runtime of the K8s we deploy in the CI.


[Bug] Prettier-eslint takes presidence

I have a elsintrc config in my project but although I have some rules I don't want them to apply automatically when save, but only prettier to kicks in.

It should be caused by your other vim plugin.

I just disabled everything, clean my init.vim file, left only coc-prettier an try again. Same results. The problem is with react-exhaustive eslint rule. If I disable it, problem disappears.
I also run :CocCommand workspace.showoutput to check if there is anything running except prettier.
None. Is this something that has to do with prettier itself?

Intrestingly enough if I run npm install --save-dev eslint-config-prettier then the problem goes away


Add HUNePi summary statistics to lineage CSV

Since we already provide a way for users to download a CSV with lineage-specific statistics, we should add the HUNePi statistics to the relevant JSON data file and transmit these stats to the front-end (but don't display for now).

This is in dev, waiting PR to main


[Support] Upgrade CF to 11.2, increase test parallelism, respond to CVEs

What
Upgrade CF deployment to 11.2
Whilst on support we received a user query about a docker application not health-checking properly, this is a bug in CAPI which was fixed in https://github.com/cloudfoundry/capi-release/releases/tag/1.85.0 which is present in CF deployment >= 11.1 (we are on 11.0)
I didn't go straight to 12 because this is a support task and CF deployment 12 allegedly needs a change to BOSH DNS service aliases, which we can look into in the actual story. Also it is firebreak so I want to fix the tenant's issue with a small review.
Increase test parallelism
Read the commit message, but this drops my CF acceptance test time in my pipeline to 25 minutes from 40 (previously 1 hour).
Respond to CVEs
See the last two commits for details.

Bump stemcells to 456.27 (alphagov/paas-bootstrap PR todo)
Bump cflinuxfs3 to 0.129.0

How to review
Code review
Travis
Look at my dev environment
Who can review
Not @tlwr

Going to let the pipeline run with the last two commits finish my dev-env before pushing the button


Update domains.json - status-token.com

Fake airdrop, asking for private key
https://etherscamdb.info/scam/2144/


Module compiled with Swift 4.2 cannot be imported by the Swift 5.0 compiler

I am using Xcode 10.2 beta3, while running an app it showing Module compiled with Swift 4.2 cannot be imported by the Swift 5.0 compiler on import Swinject. I tried to update from cathlog,  following with command 'carthage update --plateform iOS --no-use-binaries', but it gets failed with exit code 65. tried clean and build but not woked.
In my cartfile-github "Swinject/Swinject"                          github "Swinject/SwinjectStoryboard"
What's going wrong here, not able to figur out.

Swinjet is not yet compatible with Xcode 10.2 / Swift 5. There is work in progress at #391

yeah got the point. I downgraded the Xcode version, which is compatible with Swift 4.2.

Hello, is this issue still happening?
If not, should we close it?

Our latest release supports Swift 5. Let's close this issue. If we still see the issue, feel free to re-open it 👍


[Transaction/history enpoint] Wrong time on mobile client

request : https://safe-client-nest.staging.5afe.dev/v1/chains/5/safes/0xfcFab1Eb61d3b1E8ff8F1eF1CD8e46F8be30b77f/transactions/history?timezone_offset=7200000
"If the time_zone_offset-parameter is  0, then the backend  sends a wrong timestamp of<PHONE_NUMBER>000 which is Mon Jun 12 2023 12:40:12 GMT+0000 and Mon Jun 12 2023 14:40:12 GMT+0200 (Central European Summer Time)  When it should be 8:40 UTC to be 10:40CEST.

I've looked into this and we are returning timestamps in UTC according to the specified timezone_offset correctly.
I've summarised my findings and am waiting to see what the mobile team says.

Mobile have confirmed that the issue is on their end so I will close this.


Opening a website not working

Opening a website with the launch command isn't working for me.
For example "setState('lgtv.0.launch', 'http://www.iobroker.net')" won't start the browser, nothing is happening.
Everything else is working fine.
TV Model: LG 49UJ635V
WebOS Version: 05.80.10

Hello,
can you please set the adapter log level to "debug" and try it again?
Then please post the log here.
I tried the same command just now on my tv and it works.

Strangely there is no log output.
Also I tried to be sure, but even starting "com.webos.app.browser" works.

It seems that LG removed the direct launch of website adresses from the WebOS stack.
So I think I could'nt help at the moment :(
I can't find any documentation about the "new" WebOS "features" :(

@blia's fork of the lgtv library has a working open_browser_at() function.
Maybe I'll try to port that function as @blia's and @hobbyquaker's (which is currently used) forks differ quite a lot.

It would be great if you can try it by yourself.
If you have any questions feel free to ask here or other developers on iobroker forum.

Added it in #24.
It can also be used to open images or videos over a URL.

I've merged your pull request! Thank you very much for the excellent work :)


Dependency on libssl1.1 (OpenSSL 1.1.1)

I found the crunch bot is depending libssl1.1 (OpenSSL 1.1.1) to run. OpenSSL 1.1.1 reached its end of life on September 11, 2023. Are you planning to upgrade to use the libssl3 (OpenSSL 3.x)  instead?
I get the following error when try to start the service:
Nov 18 12:38:57 xxxxxxxxxx crunch[1777]: /crunch-bot/crunch: error while loading shared libraries: libssl.so.1.1: cannot open shared object file: No such file or directory
Nov 18 12:38:57 xxxxxxxxxx systemd[1]: crunch.service: Main process exited, code=exited, status=127/n/a
Nov 18 12:38:57 xxxxxxxxxx systemd[1]: crunch.service: Failed with result 'exit-code'.

Hey could you let me know which OS are you using?

I am currently using the Ubuntu 22.04.5 LTS.

Ok, the binary available on the release is being compiled under Ubuntu-20.04 and when used on Ubuntu 22.04.5 LTS it raises the issue. There were some incompatibilities with Debian users also, I'll have a look again.
As a workaround you can always compile from source or install previous libssl1.1.1 in the meantime.

I am using the workaround for now (it won't be a safe long-term solution), but I would love to upgrade as soon as a release that uses the libssl3 (OpenSSL 3.x). Thank you.


Attempting to alter a table with a column with default value (JSON_ARRAY()) by adding a foreign constraint fails with an error on SQLite

Laravel Version
11.22.0
PHP Version
8.3.6
Database Driver & Version
SQLite 3.45.1 for Linux on amd64
Description
In Laravel 11.15.0, https://github.com/laravel/framework/pull/51373 introduced the ability to add / drop foreign keys in SQLite, by re-creating the table and copying all data.
However, I run into an error when the table being altered includes a JSON column with JSON_ARRAY() as default value. The error is something like:

SQLSTATE[HY000]: General error: 1 near "(": syntax error (Connection: sqlite, SQL: create table "__temp__items" ("id" integer primary key autoincrement not null, "flags" text not null default JSON_ARRAY(), "item_id" integer, foreign key("item_id") references "items"("id")))

The issue is that the SQL query the column definition should be something like  "column_name" text not null default (JSON_ARRAY())), while the query executed has  "flags" text not null default JSON_ARRAY()) (without the brackets). This results in a syntax error.
Steps To Reproduce
Create a table with a JSON column with (JSON_ARRAY()) as default value:
<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Query\Expression;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

return new class extends Migration
{
    public function up(): void
    {
        Schema::create('items', function (Blueprint $table) {
            $table->id();
            $table->json('flags')->default(new Expression('(JSON_ARRAY())'));
        });
    }
};

Alter the table to add a foreign key constraint:
<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

return new class extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up(): void
    {
        Schema::table('items', function (Blueprint $table) {
            $table->foreignId('item_id')->nullable();

            $table->foreign('item_id')
                ->references('id')
                ->on('items');
    }
}

Running the second migration results in an error like:

SQLSTATE[HY000]: General error: 1 near "(": syntax error (Connection: sqlite, SQL: create table "__temp__items" ("id" integer primary key autoincrement not null, "flags" text not null default JSON_ARRAY(), "item_id" integer, foreign key("item_id") references "items"("id")))

Under the hood, the first migration execute the following SQL statement:
create table "items" ("id" integer primary key autoincrement not null, "flags" text not null default (JSON_ARRAY()))

while the second migration executes:
create table "__temp__items" ("id" integer primary key autoincrement not null, "flags" text not null default JSON_ARRAY(), "item_id" integer, foreign key("item_id") references "items"("id"))"

The issue is that the second statement includes JSON_ARRAY() instead of (JSON_ARRAY()).
As a comparison, manually running the .schema items in SQLite we get:
sqlite> .schema items
CREATE TABLE IF NOT EXISTS "items" ("id" integer primary key autoincrement not null, "flags" text not null default (JSON_ARRAY()), "item_id" integer);

@hafezdivandari anything we can do here maybe?

@driesvints I'll look into this asap.

You may check #52678


Remove the vite dependency

Clear and concise description of the problem
vitest is currently 22.9 MB, out of which 14.3 MB comes from the vite dependency, which among other things depends on esbuild which (correct me if wrong) is not even used for vitest execution at all.
Suggested solution
Remove the vite dependency and move the parts that are shared between vitest and vite into a shared module.
This has the benefit that server-side projects which use vitest but not vite don't need to download the unnecessary dependencies.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Vitest cannot work without Vite.

Why not? Which parts of vite can not be moved out to a shared library?

Also, can you not close this so fast please? I'd like to hear viewpoints from other contributors as well. The "Why Vitest" docs say:

Vitest aims to position itself as the Test Runner of choice for Vite projects, and as a solid alternative even for projects not using Vite.

So if vitest positions itself for projects that don't use vite, it should not depend on it imho. The package is currently way too heavy in size imho.

Also, can you not close this so fast please? I'd like to hear viewpoints from other contributors as well. The "Why Vitest" docs say:

Vitest aims to position itself as the Test Runner of choice for Vite projects, and as a solid alternative even for projects not using Vite.

So if vitest positions itself for projects that don't use vite, it should not depend on it. The package is currently way too heavy in size imho.

Vite processes paths, files and transforms code. 90% of everything Vitest does is done by Vite.

So if vitest positions itself for projects that don't use vite, it should not depend on it.

Vite does all the work.

For frontend project, I agree, vite certain does the heavylifting.
But for simple deno/node project where no transpilation is happening, vite is just used as a config resolver, the whole vite transpilation pipeline is not needed.
But I see you're favoring the frontend projects, so deno/node projects should go look somewhere else if they want a lightweight test runner.


No idea about what to put in new connection form

I can't guess the required format of my connection values, maybe some placeholder or a (? - help) key and of course, documentation about it can help.
Here's how I wanted to create a connection to the MySQL database of my company

I'll add that to the documentation, but for now, check this: https://github.com/xo/dburl?tab=readme-ov-file#example-urls


Protocol based Func implementation

FuncProtocol added
FuncProtocol.callAsFunction method added

I think that calling Func instances directly makes sense so I added this feature.
I implemented it as an extension for a Function protocol to enable users to implement their own specific function containers with this feature (may be useful when writing clients for TCA for example)
public struct CacheClient<Key: Hashable, Value> {
  public init(
    saveValue: CacheOperations.Save<Key, Value>,
    loadValue: CacheOperations.Load<Key, Value>,
    removeValue: CacheOperations.Remove<Key, Value>,
    removeAllValues: CacheOperations.RemoveAllValues<Key, Value>
  ) {
    self.saveValue = saveValue
    self.loadValue = loadValue
    self.removeValue = removeValue
    self.removeAllValues = removeAllValues
  }

  public var saveValue: CacheOperations.Save<Key, Value>
  public var loadValue: CacheOperations.Load<Key, Value>
  public var removeValue: CacheOperations.Remove<Key, Value>
  public var removeAllValues: CacheOperations.RemoveAllValues<Key, Value>
}

public enum CacheOperations {
  public struct Save<Key: Hashable, Value>: Function {
    public init(_ call: @escaping Signature) {
      self.call = call
    }

    public typealias Signature = ((Key, Value, Int?)) -> Void
    public let call: Signature
    public func callAsFunction(_ value: Value, withCost cost: Int? = nil, forKey key: Key) {
      call((key, value, cost))
    }
  }

  public struct Load<Key: Hashable, Value>: Function {
    public init(_ call: @escaping Signature) {
      self.call = call
    }

    public typealias Signature = (Key) -> Value?
    public let call: Signature
    public func callAsFunction(forKey key: Key) -> Value? { call(key) }
  }

  public struct Remove<Key: Hashable, Value>: Function {
    public init(_ call: @escaping Signature) {
      self. call = call
    }

    public typealias Signature = (Key) -> Void
    public let call: Signature
    public func callAsFunction(forKey key: Key) { call(key) }
  }

  public struct RemoveAllValues<Key: Hashable, Value>: Function {
    public init(_ call: @escaping Signature) {
      self.call = call
    }

    public typealias Signature = () -> Void
    public let call: Signature
  }
}

let cache = CacheClient<String, Data>.default()
cache.saveValue(data, withCost: data.count, forKey: "test") // You can override callAsFunction to get custom labels
cache.removeAllValues() // this `callAsFunction` method is provided by the `Function` protocol

PS. (for someone, who may use it as an example)
For more specific clients I prefer creating:
struct MyClient {}
extension MyClient {
 enum Operations {}
}
extension MyClient.Operations {
 // ...
}

But cache operations may be reused for UserDefaultsClient or KeychinClient for example (actually it can't be used because UserDefaultsClient and KeychinClient are pretty specific and they need their own operations declared, but at the moment I wrote CacheClient I didn't notice it yet 😅)

Also, I use my own protocol
public protocol ActionContainer {
 associatedtype Input
 associatedtype Output
 typealias Action = (Input) -> Output

 init(action: @escaping Action)
 var action: Action { get }
}

extension ActionContainer {
 public func callAsFunction(_ input: Input) -> Output { action(input) }
}

extension ActionContainer where Input == Void {
 public func callAsFunction() -> Output { action(()) }
}

But I noticed, that it's pretty the same, so that's why I implemented it here, so I'll be able to generalize code a bit 🌚
And I wonder won't it be more expressive to use Input & Output associated types for Function protocol and Func type instead of A and B? It's a breaking change and may be released as a major update, but what's your point about it? 🙂
(I get that mathimatically it's maybe more expressive to use A&B to declare operators like flatMap etc, but at the call site it's not very convenient to see these type constraints)

Also, I'd like you to check if the logic of new methods is the same, it wasn't tested (I mean before my changes) and I think I didn't break anything, but maybe it is worth looking through diffs to ensure if it is ok. Also maybe it is worth extracting Function protocol to another file when you check the diffs.

@maximkrouk This is interesting, thanks!
Just wanted to get some communication going, since we're not super active on the Prelude repo. We used this repo to experiment with FP concepts in Swift and in our earlier projects powering pointfreeco. We also leveraged that work in the isowords, but mainly for time :)
Over the years we've been trying to capture these raw ideas in more refined ways that feel more at home in Swift, so our focus on maintaining this repo has diminished. It was always considered an "alpha" (from the README), and more of an experimental area for us to play in rather than an external library to support. Brandon and I will chat about this soon, but we may not merge much in that isn't directly used in pointfreeco/isoword.
Common in other languages with "prelude" modules is that teams grab what they like and maintain their own, so we'd still say please maintain a fork from this repo of whatever is useful to you!

Thanks again for sharing, @maximkrouk! @mbrandonw and I looked at this today and it's a lot of fun :smile:
Given what I mentioned in the previous comment, I think we're going to close this out for now, but if you'd like to further surface and discuss the idea, the discussions area might be a good spot!


When the channel is empty, the request will be blocked

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description

Ⅱ. Describe what happened
If there is an exception, please attach the exception trace:
Just paste your stack trace here!

Ⅲ. Describe what you expected to happen
Ⅳ. How to reproduce it (as minimally and precisely as possible)

xxx
xxx
xxx

Minimal yet complete reproducer code (or URL to code):
Ⅴ. Anything else we need to know?
Ⅵ. Environment:

JDK version(e.g. java -version):
Seata client/server version:
Database version:
OS(e.g. uname -a):
Others:

I would like to claim this task


Manual approval required for workflow run<PHONE_NUMBER>: Release t0rn refs/tags/v1.19.1-rc.0

Please approve runtime upgrade with github release t0rn v1.19.1-rc.0
Runtime upgrade dry run output:
💈 Script started at 1935381 block in t0rn chain

🐡 Running with dryrun flag!

🐙 checking out v1.19.1-rc.0...
✅ tag checked out

🔎 making sure runtime version got updated...
✅ runtime versions updated

🫧 check WASM artifact...
🔢 WASM blake2_256 hash: 0xbe0bbd16f575bb704b193c2e11fd9cdeb040e253a8c5cee6fe8ba4efbff42d74
🔢 WASM blake2_256 hash fetched from release: 0xbe0bbd16f575bb704b193c2e11fd9cdeb040e253a8c5cee6fe8ba4efbff42d74
✅ WASM blake2_256 hash is matching
⚙ set_code runtime upgrade... dry
✅ runtime upgrade executed... dry

Workflow is pending manual review.
URL: https://github.com/t3rn/t3rn/actions/runs/4939998293
Required approvers: [AwesomeIbex MaciejBaj chiefbiiko petscheit palozano]
Respond "approved", "approve", "lgtm", "yes" to continue workflow or "denied", "deny", "no" to cancel.

Workflow cancelled, closing issue.


[ bug Report ] Strange sliding

TombEngine version
v<IP_ADDRESS> (Latest Release)
Tomb Editor version
v1.7.2
Describe the bug

Lara slides along the geometry in a completely wrong direction at some point.
THE PROJECT FILE
https://mega.nz/file/tMQhCKiT#WKCTl2QGZwi7lNJ0pk9NonLNGOr8d-R7h723px28OYo
To Reproduce

Jump on the platfom in the video
https://youtu.be/9geyzmDOBQo
Expected Behaviour
She should slide down. I included the level.
Additional Content
No response
Minimal reproduction project
https://mega.nz/file/BdA3FIrA#wntRWNNls6oWkPJipYu9beI3u5nBrrughCTLLvrKNJY

seems to be fixed, reopen if not fixed


Fix reading nested lists from parquet files

Which issue does this PR close?

Closes #1515.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Interesting, I cannot reproduce the Parquet file by writing a simple Parquet file in test.
The main difference is:
For the original Parquet file, the path to the name column is:
 ["table", "table_info", "name"]

Because table_info is a list of struct, so we need the change in this PR.
I use the same message type to write out a Parquet file in test. But when I read it back. For the same name column, the column path is different:
parts: ["arrow_schema", "table_info", "table_info", "name"]

For the first table_info, get_arrow_field will simply pick up same name field from arrow schema, which is the list field. Then moving to next table_info, it picks the struct field from there. Next part is name, and current field is struct, it can get the correct field.
I checked the arrow schema for the Parquet file and the one I wrote in test, they are the same. So I am currently not sure why the column paths for same column name could be different for two cases.

Let me dig why there is the difference.

Codecov Report

Merging #1517 (56d718f) into master (c5442cf) will increase coverage by 0.10%.
The diff coverage is 53.70%.

@@            Coverage Diff             @@
##           master    #1517      +/-   ##
==========================================
+ Coverage   82.68%   82.79%   +0.10%
==========================================
  Files         188      190       +2
  Lines       54361    54717     +356
==========================================
+ Hits        44951    45301     +350
- Misses       9410     9416       +6

Impacted Files
Coverage Δ

parquet/src/arrow/array_reader.rs
87.07% <ø> (+2.71%)
:arrow_up:

parquet/src/arrow/array_reader/builder.rs
67.91% <53.70%> (+1.61%)
:arrow_up:

arrow/src/array/equal/utils.rs
74.45% <0.00%> (-3.24%)
:arrow_down:

arrow/src/array/transform/mod.rs
86.35% <0.00%> (-0.23%)
:arrow_down:

arrow/src/array/builder.rs
86.68% <0.00%> (-0.05%)
:arrow_down:

arrow/src/compute/kernels/filter.rs
88.00% <0.00%> (ø)

arrow/src/compute/kernels/length.rs
100.00% <0.00%> (ø)

arrow/src/array/equal/union.rs
100.00% <0.00%> (ø)

arrow/src/ffi_stream.rs
79.03% <0.00%> (ø)

arrow/src/buffer/mutable.rs
90.53% <0.00%> (+0.02%)
:arrow_up:

... and 11 more

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 c5442cf...56d718f. Read the comment docs.

Thanks @viirya

Thank you @alamb !


Expose datagram content as Bytes?

Thank you for your library, very useful for my experiments with WebTransport :+1:
A Datagram internally uses a Bytes instance, but only exposes a Deref<Target=[u8]>.
On one hand, it may be useful for the implementation because it can change from Bytes to something else internally without changing the API. But on the other hand, the caller is forced to copy the content, even if it uses Bytes:
let datagram = conn.receive_datagram().await?;
let bytes = Bytes::copy_from_slice(&datagram);

Since Bytes is very common, and is also exposed in the quinn API, it might make sense to expose the payload as a Bytes instance. Something like this:
diff --git a/wtransport/src/datagram.rs b/wtransport/src/datagram.rs
index f2cd4dc..419bcab 100644
--- a/wtransport/src/datagram.rs
+++ b/wtransport/src/datagram.rs
@@ -57,6 +57,11 @@ impl Datagram {
     pub(crate) fn into_quic_bytes(self) -> Bytes {
         self.quic_dgram
     }
+
+    /// Return the payload
+    pub fn payload(&self) -> Bytes {
+        self.quic_dgram.slice(self.payload_offset..)
+    }
 }

 impl Deref for Datagram {

This would allow to avoid a whole copy for each datagram received.
What do you think?

On one hand, it may be useful for the implementation because it can change from Bytes to something else internally without changing the API.

You got the right point.
Being at the early stage of releasing, I am trying to be slightly more conservative on the API of wtransport, also with re-export external dependencies.
By doing so, I think it allows the library to grow reflecting real needs on feedback like yours (thank you, btw).
Having said that,

Since Bytes is very common,

I agree on this point and, thus, I think it might be worth it to expose Bytes (also considering it practically comes for free, as you mentioned). I am going to open a PR as you suggested.
Off topic: I still don't completely understand why bytes crates does not implement something that can leverage the borrowing system of rust. For instance with lifetimes for having a memory view to an existing slice, Bytes<'a>::from_ref(&buffer)


Set certificate expiration to August 2, 2018

Updates the gen-* scripts to take a -days argument, and set that days
argument to 3 years from now. I believe without this argument the certificates
default to a 30 day expiry. Code lifted from
https://github.com/shazow/urllib3/blob/master/dummyserver/certs/README.rst.
Generates new SSL certs with a long expiry.

whoops... need to check if this passes tests. 1 minute.

Something isn't right with our tests lately, it's on my TODO list to investigate why they timeout so badly.

When I opened the PR the branches hung on npm install (not sure why). Triggered a rebuild and it looks like the tests are all passing.

Thanks, now you can pull master :+1:


Postfixadmin: Can't connect to database.Access denied for user postfix

Hi there -
I pulled the latest can and I get this message

In my docker-compose file I setup the password. When I tried to update postfix password with the instruction here https://github.com/hardware/mailserver/wiki/Admin-password-Lost. Root password does not work.

Let me know

Remove /mnt/docker/mysql (If you do not have important data), restart mysql and try again.

Wow. This has been really frustrating!
Here is my log file:

See postfix not able to login access denied. I installed and reinstalled the container with same outcome. Has anyone experienced this behavior?

Give me your docker-compose.yml file please.

here it "
version: '3'
services:
  mailserver:
    image: hardware/mailserver
    container_name: mailserver
    domainname: mydomain.com               # Mail server FQDN & reverse = mail.watuforms.com.
    hostname: mail
    # restart: always
    # extra_hosts:                        - Required for external database (on other server or for local database on host)
    #  - "mariadb:xx.xx.xx.xx"            - Replace with IP address of MariaDB server
    ports:
      - "25:25"     # SMTP                - Required
    # - "110:110"   # POP3       STARTTLS - Optional - For webmails/desktop clients
      - "143:143"   # IMAP       STARTTLS - Optional - For webmails/desktop clients
    # - "465:465"   # SMTPS      SSL/TLS  - Optional - Enabled for compatibility reason, otherwise disabled
      - "587:587"   # Submission STARTTLS - Optional - For webmails/desktop clients
      - "993:993"   # IMAPS      SSL/TLS  - Optional - For webmails/desktop clients
    # - "995:995"   # POP3S      SSL/TLS  - Optional - For webmails/desktop clients
      - "4190:4190" # SIEVE      STARTTLS - Optional - Recommended for mail filtering
    environment:
      - DBPASS=itdidnotwork
    # - ENABLE_POP3=true          # Enable POP3 protocol
    # - GREYLISTING=gross         # Enable gross greylisting policy server
    # - DISABLE_CLAMAV=true       # Disable virus scanning
    # - DISABLE_SPAMASSASSIN=true # Disable SPAM checking
    # - DISABLE_SIEVE=true        # Disable ManageSieve protocol
    volumes:
      - /mnt/docker/mail:/var/mail
     # - /mnt/docker/mail/opendkim:/etc/opendkim/keys
     #- /mnt/docker/nginx/certs:/etc/letsencrypt
      - /mnt/docker/nginx/certs/live/mail.mydomain.com
    depends_on:
      - mariadb

  # Administration interface
  # https://github.com/hardware/postfixadmin
  # http://postfixadmin.sourceforge.net/
  # Configuration : https://github.com/hardware/mailserver/wiki/Postfixadmin-initial-configuration
postfixadmin:
    image: hardware/postfixadmin
    container_name: postfixadmin
    domainname: mydomain.com
    hostname: mail
    # restart: always
    environment:
    - DBPASS=lostpassword1
    depends_on:
      - mailserver
      - mariadb

  # Webmail (Optional)
  # https://github.com/hardware/rainloop
  # https://www.rainloop.net/
  # Configuration : https://github.com/hardware/mailserver/wiki/Rainloop-initial-configuration
  rainloop:
    image: hardware/rainloop
    container_name: rainloop
    # restart: always
    volumes:
      - /mnt/docker/rainloop:/rainloop/data
    depends_on:
      - mailserver
      - mariadb

  # Web server
  # https://github.com/Wonderfall/dockerfiles/tree/master/nginx
  # https://nginx.org/
  # Configuration : https://github.com/hardware/mailserver/wiki/Reverse-proxy-configuration
  nginx:
    image: wonderfall/boring-nginx
    container_name: nginx
    # restart: always
    ports:
      - "80:8000"
      - "443:4430"
    volumes:
      - /mnt/docker/nginx/sites-enabled:/sites-enabled
- /mnt/docker/nginx/conf:/conf.d
      - /mnt/docker/nginx/log:/var/log/nginx
      - /mnt/docker/nginx/certs:/certs
    depends_on:
      - postfixadmin
      - rainloop

  # Database
  # https://github.com/docker-library/mariadb
  # https://mariadb.org/
  mariadb:
    image: mariadb:10.1
    container_name: mariadb
    # restart: always
    # Info : These variables are ignored when the volume already exists (databases created before).
    environment:
      - MYSQL_ROOT_PASSWORD=lostpassword
      - MYSQL_DATABASE=postfix
      - MYSQL_USER=postfix
      - MYSQL_PASSWORD=lostpassword1
    volumes:
      - /mnt/docker/mysql/db:/var/lib/mysql

ok. I was able to get past the access denied issue.
Now I have this issue where the email can be sent externally.
2017-03-18T21:47:57.037877+00:00 mail postfix/smtp[348]: connect to alt1.gmail-smtp-in.l.google.com[2a00:1450:400b:c00::1b]:25: Network is unreachable
2017-03-18T21:47:57.087639+00:00 mail postfix/smtp[349]: connect to alt2.aspmx.l.google.com[<IP_ADDRESS>]:25: Connection timed out
2017-03-18T21:47:57.087689+00:00 mail postfix/smtp[349]: connect to alt1.aspmx.l.google.com[2a00:1450:400b:c00::1a]:25: Network is unreachable ```

Close this issue. I was able to resolve.
Note that there is an issue with including SSL certificate that was purchased from vendors like Positive SSL. We need to be able to include SSL in any folder we are able to and point to it.
Thanks


Is there a way to use --download-archive while also use --skip-download

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

So my goal is to be able to download twitch vod information (in the form of a -o templated file)
To clarify I want to write a file using -o which writes a file while using --skip-download
To get any file written using the -o template, while using --skip-download, I chose to use --write-thumbnail
The problem is that everytime I run the command on a users twitch vod list it has to go through all of the vods and not just the most recent one. If downloading a video the solution would be to use --download-archive. But that doesn't work with --skip-download.
Is there any way to get vod information (uploader, url, title, etc) written to a file while also using --download-archive?

No.

No.


Sinchan Potrait Using Python

Description
This is a Sinchan Portrait build using python turtle and geometry. In this program, the turtle will build the model while the iconic sinchan song is played in the background.
Fixes #975
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@kaustubhgupta please have a look

@Arnab11917676 your amazon scraper files are still here. Create a separate branch for this issue and then make your PR. It has caused us a lot of issues in the past. I am closing this PR


Allow Date Picker to behave more like a input type date

Summary
The desktop version of the datepicker withtype="date" changes the type to text and therefore writing the "formatted" value into the form control.
{
  "date": "03/14/2020"
}

Even though this behaviour is documented in the examples, this is highly confusing since the expected format of type="date" input value is yyyy-mm-dd
This issue makes it even more complicated as the behaviour is inconsistent when using the datepicker with a mobile browser, where the native type="date" is used and therefore the value in the form control is:
{
  "date": "2020-03-14"
}

This forces the developer todo special device handling.
Use case
Further the formatted date behaviour makes it hard to implement a simple dynamic min/max handling:
<clr-date-container>
          <input name="from" clrDate #from [max]="to.value" type="date" >
</clr-date-container>
<clr-date-container>
          <input name="to"  clrDate #to [min]="from.value" type="date" >
 </clr-date-container>

Since min/max expect the ISO8601 but the value is in the correct format this needs extra care

This sounds like a duplicate of https://github.com/vmware/clarity/issues/2056 where depending on if you bind to ngModel or clrDate you get a different format (which was initially intentional but now seems to be buggy).

This is also changed if you want to use the Clarity Core date control which adheres more directly to HTML spec. Closing as something we have already.


Documents upload error

Due to Server Change Default Credential mis-matched.
Now from next time we are creating containers with default credentials, we can more specifically target container with storage service account


ImportError: cannot import name '_C'

Dear Monodet team,
Thank you for your work and I hope this message finds you well. I am writing to report an issue that I encountered while working with Monodet.
I have created a new environment with Python 3.6, PyTorch 1.5.0 and Detectron2 0.1.3 as per the instructions in the README file. However, I am still facing this issue.
from detectron2 import _C
ImportError: cannot import name '_C'
also i have used another method used your folder detectron2 but the same problem
Could you please help me with this problem? Any help or guidance would be greatly appreciated.
Thank you for your time and consideration.
Best regards,

@Sondosmohamed1 Excuse me to ask, did you fix this？


Mm 25435

Summary
It revamps emoji picker UI
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/15636

Hello @bhargav50,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Thanks @bhargav50, adding the work in progress label as you and @asaadmahmood get it the PR ready for review 👍

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

@bhargav50 @asaadmahmood anything we can help with here to move this PR along?

@bhargav50 @asaadmahmood anything we can help with here to move this PR along?

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

@bhargav50 just checking if you're still working on this?

@bhargav50 just checking if you're still working on this?

@bhargav50 just checking if you're still working on this?

@esethna  I need help with that expanding animation in this PR. I think I commented about this last time as well but can't find that conversation anymore.

@bhargav50 just checking if you're still working on this?

@esethna  I need help with that expanding animation in this PR. I think I commented about this last time as well but can't find that conversation anymore.

@asaadmahmood are you able to help with this on the PR? I think @bhargav50 is referring to the skin tone selector and how it animates out to provide the skin tone options. @bhargav50 please add any more details as needed

@asaadmahmood are you able to help with this on the PR? I think @bhargav50 is referring to the skin tone selector and how it animates out to provide the skin tone options. @bhargav50 please add any more details as needed

@asaadmahmood are you able to help with this on the PR? I think @bhargav50 is referring to the skin tone selector and how it animates out to provide the skin tone options. @bhargav50 please add any more details as needed

Yeah @esethna  that animation only, I am not able to visualize how mathematically I can modify the divisions to render that animation. If someone can just point me toward a high-level approach I will figure out a way to implement it then.

@asaadmahmood are you able to help with this on the PR? I think @bhargav50 is referring to the skin tone selector and how it animates out to provide the skin tone options. @bhargav50 please add any more details as needed

Yeah @esethna  that animation only, I am not able to visualize how mathematically I can modify the divisions to render that animation. If someone can just point me toward a high-level approach I will figure out a way to implement it then.

@esethna @bhargav50 I have added that particular effect and make some other UI changes.

But for the sliding animation etc, we should be good to go.

@esethna @bhargav50 I have added that particular effect and make some other UI changes.

But for the sliding animation etc, we should be good to go.

@esethna I am closing this PR for now. Will raise it again once it is ready for review.

@bhargav50 just checking how this is coming along? Need any help or guidance?


feat: Adds netstandard2.0 and .NET Framework support

Description

Adds netstandard2.0 and .NET Framework support

Closing in favor of #226 which removes some legacy test components not compatible with windows runners


Remove unused MVC components

These are leftovers from pre-symfony times. No longer used, so lets get rid of them.

Done; I've removed assetic and mrclay/minify the list of dependencies.

Rebaseed the branch on master to resolve dependency conflicts.


Can this project achieve continuous deformation compensation in different parts of the ultrasound scanning process, rather than a single part?

Can this project achieve continuous deformation compensation in different parts of the ultrasound scanning process, rather than a single part? For example, venous ultrasound scanning of the arm continuously compensates for venous deformation along the direction of the arm. Thanks！

Yes, we also aim to achieve the continuous deformation during sweeping instead of only palpation in the same location. Since the input for the designed method is (force, global stiffness, us image), the global stiffness value can only obtained during the palpation. Therefore, during sweeping we can use an average stiffness for continuous compensation. You can refer to our previous paper about the the 3d visualization for sweeping sequence on the phantom data. [https://doi.org/10.48550/arXiv.2107.08411]


Odd sorting of tags / releases if number of tags exceeds a particular value and pagination is added

Gogs version (or commit ref): <IP_ADDRESS>7 (Docker-Image)
Git version: 2.11.1
Operating system: Docker Image: Linux <hostname> 4.6.0-0.bpo.1-amd64 #1 SMP Debian 4.6.1-1~bpo8+1 (2016-06-14) x86_64 Linux
Database:

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gogs.io:

[x] Yes try.gogs.io/rudeks
[ ] No
[ ] Not relevant

Log loglog.txt:

Description
I encountered a problem recently, where tag sorting appears to be broken, as soon as pagination is added in the /releases view for a project. Unless a particular number of tags is reached, everything is fine and the tags appear in the right order. After reaching a particular number of tags is reached, newer Tags appear on later pages.
This occurs mainly when you git push --tags your local repository with tags that and if at least one tag resembles a name that does not follow the format of other tags. In the sample repository provided, you will find 25 tags that follow the form v-[0..24].0.0 and two tags that do not follow the form, but, however, were created subsequently after v-25.0.0_Reproduzierbarkeit. One of these tags (that one, that is named Noch-ein-Test-der-Reproduzierbarkeit and does not follow the form at all) appears somewhere at the end of the Tags and Releases-List; in the /releases-resource the get param ?after=v-6.0.0 is appended to the URI. On this page the Noch-ein-Test-der-Reproduzierbarkeit-Tag shows up.
Expected behavior
As the tag v-25.0.0_Reproduzierbarkeit was created right after v-24.0.0, and Noch-ein-Test-der-Reproduzierbarkeit was created after v-25.0.0_Reproduzierbarkeit, one expects that Noch-ein-Test-der-Reproduzierbarkeit appears after (or before, depending of the sorting) the tag created chronologically before.

You're right, shouldn't use refname(code) as sort key but taggerdate.

I created a PR to fix this.  If more work is needed just let me know, thanks!

This is claimed to be fixed by merging https://github.com/gogs/git-module/pull/36, please help test on develop branch.

Not sure this is fixed, I have a very strange release sort order on <IP_ADDRESS>6

@AlgyTaylor Please define "strange" and try to reproduce on https://try.gogs.io

@Unknwon - thanks for the reply, and also for writing Gogs - it's a really useful bit of software and I recommend it to a lot of people.   (Screenshots below)
Anyway, on the "releases" page, the first page shows releases from over a year ago.  I go on to page 2, and it shows some releases that have happened since the first entry on page 1, and also some duplicate entries from that page.  I've attached 2 screenshots of those pages below so you can see.  I add a release roughly once a week, so it should show v1.10.0, v1.9.5, v1.9.4, v1.9.3, ...
I'll try to recreate it now
This is the first page of my 'releases' page ...

... and here is page 2 ...

@AlgyTaylor thanks for the screenshot! If it could be reproduced on demo site, would be very helpful to solve the problem!

Just a quick note to say this seems to have been fixed in the last 2/3 releases - thanks :) :+1:


Set angular as default in blueprint

@runspired after some heavy use of this addon, it seems like you were right about angular being the more superior style. I've had a much better experience once I started using it instead of ember.

LGTM 👍


added hotlink to NTLM article

added hotlink to NTLM article for IT Professionals new to NTLM/Kerberos

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Update Http  links with Htpps

Updating the HTTP links with HTTPs;

Thanks for your input @oke-aditya , sounds like we're on the same page. I'll close the PR, thanks for opening the discussion @abhi-glitchhg


"Catkin" referenced in quick start?

In the TL;DR section a number of commands reference catkin. Is this accurate? It looks like they should be referencing colcon
https://colcon.readthedocs.io/en/latest/user/quick-start.html

Good catch, looks like those should be colcon test and colcon test-result --all. Would you mind opening a PR on https://github.com/colcon/colcon.readthedocs.org ?

Addressed in colcon/colcon.readthedocs.org#32.


fix: fixes walletconnect signed tx type

Updates the type of request sent to Wallet Connect to allow for Metamask and many other wallets to properly sign transactions

New package versions published!


Update main-ci.yml

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2c09e94) 66.75% compared to head (6527a98) 66.75%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   66.75%   66.75%
=======================================
  Files         108      108
  Lines        3929     3929
  Branches      569      569
=======================================
  Hits         2623     2623
  Misses       1097     1097
  Partials      209      209

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[CSL-1154] Split Pos.Launcher.Launcher into submodules

We are running out of memory, sir!

@Martoon-00  WE NEED MORE MEMORY

Lil, it doesn't help anymore :shrug:


Remove DCO requirements since EasyCLA is already required

Can we remove DCO requirements since EasyCLA is already required? See #57 for an example.

This was done. Thanks @Judith!


Epinio UI test - Add Cypress s3 and ext reg scenario to CI

This scenario has to be added in the CI:
https://github.com/epinio/epinio-end-to-end-tests/blob/main/cypress/integration/scenarios/install_with_s3_and_external_registry.spec.ts
Maybe on another browser to allow us to test multi web browser?
Check if we can use a kind of workflow template for avoiding code duplication.

Both scenarios passed last night:
https://github.com/epinio/epinio-end-to-end-tests/actions


Multi-board script

Basically a copy of "read_to_csv" but with hardcoded ports for all board reads. Haven't tested the datetime file naming as I don't have actual boards to test it on right now.

Finally got around to trying to implement your suggestions. Haven't been able to test it though as we haven't gotten the remote access set up again. Figured I'd get any opinions on it in the meantime anyway.

latest commit has been tested with two boards. I assume that it should scale to N arbitrary, short of threading limits. Consequence is auto-file naming, though.

@xdylanm - did a first attempt on creating a sqlite3 sink. Tried to mimic design patterns with your influxDB version but I think the indexing is different with sql? Not 100% sure but influx DB seems to work more like a single dictionary rather than a table set up but I've done very little with SQL calls so I might not be conceptualizing it properly.
Else, not sure if the data collection loop should be separate from the write to db loop - I assume the latter, despite being somewhat async, is fast enough not to worry about it but might make sense to separate. Not sure how much we should also worry about try/except blocks - I assume the influx DB libraries you're using handle some of that or it's caught later with the 'with' python syntax, of which I'm not as familiar.
Anyway, hope this doesn't complicate your own attempt to look at it this weekend...
As for using influxDB, might be sufficient in some ways as is. Using it with AWS we could setup a web portal as the dashboard, likely. Costs wouldn't likely be too bad except the # of requests for the accel, but we might want to average some of those out - in discussion with Daniel though I'm less convinced of how useful it is (maybe absolute value them at least) - so that we don't actually write that often - maybe 1Hz is still sufficient. TBD.


Push2

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

Codecov Report

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

@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage        ?   81.94%
  Complexity      ?      182
=======================================
  Files           ?       17
  Lines           ?      997
  Branches        ?      108
=======================================
  Hits            ?      817
  Misses          ?      132
  Partials        ?       48

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


Lot's of ISO C99 requires at least one argument for the "..." warnings.

warning: ISO C99 requires at least one argument for the "..." in a variadic macro

Strange. Looks like a compiler bug.
What is the compiler and its version? Do some other test generate this error?

This is under MSYS2. The compilation command is: cc -std=c99 -Wall -W -pedantic -Wshadow -Wpointer-arith -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes -Wswitch-default -Wswitch-enum -Wcast-align -Wpointer-arith -Wbad-function-cast -Wstr                                                ict-overflow=5 -Wstrict-prototypes -Winline -Wundef -Wnested-externs -Wcast-qual -Wshadow -Wunreachable-code -Wlogical-op -Wstrict-aliasing=2 -Wredundant-decls -Wold-style-definition -Wno-unused                                                -function -ftrapv -Wsign-conversion  -I.. -g -pthread test-malgo.c -o test-malgo.exe
The cc --version output is as follows:
cc --version
cc (GCC) 7.3.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I cannot reproduce the problem (See https://github.com/P-p-H-d/mlib/runs/837501754?check_suite_focus=true for CI on MSYS2).
The command line is a little bit different however.
Do some other test generate this error?

I can reproduce the issue on linux with your version https://github.com/vladipus/mlib.git however
It seems the issue is in your path :)

Lol, sorry then. Let me check this out.

You were right. There were some M_C using mistakes, I've made. Sorry for taking your time. The issue is now fixed in my master.


Template - Orders - Use Cases

Create a new order POST api/orders
Edit an existing order PUT api/orders/{id}
View an existing order GET api/orders/{id}
Submit an existing order (this means status goes to Submitted) POST api/orders/{id}/submit
Cancel an existing order (this means status goes to Cancelled) POST api/orders/{id}/cancel

create with POST without @RequestBody, inside, this method, first firstly do findbyId to find the order then do order.status = "submitted", then repository save order
(instead of string type, use enum type for statuses)

POST api/orders (values are 0 and null, doesn't take actual values that were posted)
GET -> why is there orderItem instead of orderItems
POST -> why sending orderItem instead of orderItems
POST orders - one to many with orderItems not working
POST - orderDate should be generated by the REST API
POST- how to work with enums


Request for "all" option for vpn_region_ids attribute for openvpncloud_user_group resource

It would be handy to be able to specify something like
  vpn_region_ids  = ["all"]

which is an option in the UI.  To work-around this, I've defined my own local data (see below). This is going to get out of date though as regions are added or removed.
Perhaps, a better option would be to provide a data source that could return all a list of the regions ie:
data "openvpncloud_vpn_regions" {}

resource "openvpncloud_user_group" "foo" {
  connect_auth    = "AUTH"  # On prior auth timeout - 24 hours
  internet_access = "LOCAL" # split tunnel
  max_device      = 2
  name            = "foo"
  vpn_region_ids  = data.openvpncloud_vpn_regions.region_ids
}

I don't think that's possible here:
https://github.com/OpenVPN/terraform-provider-openvpn-cloud/blob/main/openvpncloud/data_source_vpn_region.go
locals {
  # The order in this list is important so it matches "All regions" in the UI
  all_vpn_region_ids = [
    "eu-west-1",      # London
    "eu-central-1",   # Frankfurt
    "us-east-1",      # Ashburn (VA)
    "us-central-1",   # Chicago (IL)
    "us-west-1",      # San Jose (CA)
    "us-south-1",     # Dallas (TX)
    "us-west-2",      # Los Angeles (CA)
    "ca-east-1",      # Toronto
    "ap-southeast-1", # Singapore
    "ap-south-1",     # Mumbai
    "af-south-1",     # Johannesburg
    "eu-central-2",   # Amsterdam
    "ap-southeast-2", # Sydney
    "br-gru",         # Sao Paulo
    "us-cmh",         # Columbus (OH)
    "ca-yvr",         # Vancouver
    "ca-yul",         # Montreal
    "fr-cdg",         # Paris
    "es-mad",         # Madrid
    "us-mia",         # Miami (FL)
    "se-arn",         # Stockholm
    "it-mxp",         # Milan
    "pl-waw",         # Warsaw
    "il-tlv",         # Tel Aviv
    "jp-hnd",         # Tokyo
    "us-bos",         # Boston (MA)
    "ch-zrh",         # Zurich
    "fi-hel",         # Helsinki
    "ie-dub",         # Dublin
    "us-atl",         # Atlanta (GA)
    "us-ewr",         # Newark (NJ)
    "us-mci",         # Kansas City (MO)
    "be-bru",         # Brussels
    "us-pdx",         # Portland (OR)
    "us-sea",         # Seattle (WA)
  ]
}

resource "openvpncloud_user_group" "foo" {
  connect_auth    = "AUTH"  # On prior auth timeout - 24 hours
  internet_access = "LOCAL" # split tunnel
  max_device      = 2
  name            = "foo"
  vpn_region_ids  = local.all_vpn_region_ids
}

We've published new version of Terraform Provider to Registry: https://registry.terraform.io/providers/OpenVPN/cloudconnexa/latest
Going forward only https://registry.terraform.io/providers/OpenVPN/cloudconnexa/latest will be developed.
Please give new provider a try, hopefully it will fix your issue.
In case if problem persisst - please open new "Issue" (https://github.com/OpenVPN/terraform-provider-cloudconnexa/issues)
Sorry for inconvenience.


Cant change language using locale

Vee-Validate: 2.0
I am trying to change language like this in app.js file:
window.Vue = require('vue');
var VueValidation = require('vee-validate');
var VueValidationSwe = require('vee-validate/dist/locale/sv');

Vue.use(VueValidation, {
  locale: 'sv',
  dictionary: VueValidationSwe
});

I get no errors when I compile but the error messages still display in english.
Thanks

You are merging the locale as a root dictionary which doesn't work, try this instead:
window.Vue = require('vue');
var VueValidation = require('vee-validate');
var VueValidationSwe = require('vee-validate/dist/locale/sv');

Vue.use(VueValidation, {
  locale: 'sv',
  dictionary: {
    sv: VueValidationSwe
  }
});


25 investigate margin of error for decompose depolarizer with jax

updated default dtype for Jax from float32 to float64 to fix precision errors. Also corrected Kenji's last name in paper

am a little confused on error as while I am using a venv with python 11, still confused on why there's the given error. will look into

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.10%. Comparing base (5247930) to head (4bcdb65).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   73.10%   73.10%
=======================================
  Files           3        3
  Lines         316      316
=======================================
  Hits          231      231
  Misses         85       85

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

shall I merge and close?


DNS over TLS

It’s a thing we could do.
https://tools.ietf.org/html/rfc7858

Curious, what was the objective here? A full DNS implementation? or an implementation of okhttp3.Dns (strictly a subset) for securing OkHttp clients?
If its the latter, would it be worth submitting a dns-over-https implementation (https://tools.ietf.org/html/draft-hoffman-dns-over-http-00)?  This is a working prototype for dns.google.com https://github.com/yschimke/oksocial/blob/master/src/main/java/com/baulsupp/oksocial/network/GoogleDns.java

This appears overtaken by DNS over HTTPS, is there much industry support for this?

DNS over HTTPS is best.


Add AlmaLinux 8 & 9 support

Added AlmaLinux 8 and 9 to the supported operating systems in metadata.json.
These changes ensure that we start testing with AlmaLinux 8 and 9 instead of CentOS 8 Stream. This update is part of our transition to better support AlmaLinux in our Puppet modules.
See, for details: https://github.com/theforeman/foreman-infra/issues/2087

This fails because of the PIDFile issue. You may notice we don't test on CentOS 8 because of that reason.

This fails because of the PIDFile issue. You may notice we don't test on CentOS 8 because of that reason.

@ekohl what do you mean by PIDFile issue? it is unclear for me.

If you look for pidfile in the repository you'll find:
https://github.com/theforeman/puppet-dns/blob/3f53f46ee50a5476504e227164680f0bd548c8b6/.github/workflows/ci.yml#L17
This input is sent to https://github.com/voxpupuli/puppet_metadata

If you look for pidfile in the repository you'll find:
https://github.com/theforeman/puppet-dns/blob/3f53f46ee50a5476504e227164680f0bd548c8b6/.github/workflows/ci.yml#L17
This input is sent to https://github.com/voxpupuli/puppet_metadata

@ekohl this supports array right, can we not do it this way, thought i don't understand the boolean part of it.
pidfile_workaround: 'CentOS,AlmaLinux'

@ekohl this supports array right, can we not do it this way, thought i don't understand the boolean part of it.
pidfile_workaround: 'CentOS,AlmaLinux'

Take a look at https://github.com/voxpupuli/puppet_metadata/blob/7c7dcad9f71cc873cd2cbffd608f635a2602d081/HISTORY.md?plain=1#L10
This is saying that you can set pidfile_workaround to true|false and apply it for all OSes. Or you can supply an array of specific OSes that the workaround should be applied to. Try setting https://github.com/theforeman/puppet-dns/blob/3f53f46ee50a5476504e227164680f0bd548c8b6/.github/workflows/ci.yml#L17 to an array with CentOS and AlmaLinux in it.

Done @ehelms

Thanks @ehelms @ekohl for helping me ✨


Doc fix: move style tag outside of script tag

It's not valid syntax to have a <style> tag within the <script>
tag in a .vue file. Instead, move this example to its own top-level
attribute.
Notably, this matches the example in docs/documentation/installation.md

@DavidCain Thank you!


Choices not clickable after a dialogue without choices

Choices not clickable after a dialogue without choices
Describe the bug
I've created a vendor NPC, which who I can talk with choices:

Show me your goods
Not right now

This works everytime I talk to this or other NPCs with choices. But having one dialogue in between, totally unrelated, without choices, breaks the NPC choice dialogues for me. I can't click on the choice anymore.
To Reproduce
Steps to reproduce the behavior:

Having a dialogue with choices -> works

Having a dialogue without choices -> works

Having the same dialogue with choices again -> can't click

Expected behavior
Choices in dialogues should be clickable everytime the dialogue spawns.

 So after killing this properly, it is working as expected.


Picuki Bridge failed with error 403

They have added cloudflare protection it seems.
HttpException: https://www.picuki.com/profile/ff resulted in `403 Forbidden` in lib/contents.php line 181

index.php(7): RssBridge->main()
lib/RssBridge.php(15): RssBridge->run()
lib/RssBridge.php(88): DisplayAction->execute()
actions/DisplayAction.php(136): PicukiBridge->collectData()
bridges/PicukiBridge.php(43): getSimpleHTMLDOM()
lib/contents.php(366): getContents()
lib/contents.php(181)

Query string: action=display&bridge=PicukiBridge&context=Username&u=ff&format=Html
Version: dev.2022-06-14 (git.master.f53e0e4)
Os: Linux
PHP version: 7.4.30

I get the 403 issue with Picuki, too. How do you come to the conclusion regarding Cloudflare? At least with NoScript/uBlock I can't see anything being loaded outside.

Application Error

The application could not run because of the following error:
Details
Type: HttpException
Code: 403
Message: https://www.picuki.com/profile/sdsdf resulted in `403 Forbidden: <!DOCTYPE html>   <title>Just a moment...</title> <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"> <meta http-equiv="X-UA-Compatible" content="IE=Edge"> <meta name="robots" content="noindex,nofollow"> <meta name="viewport" content="width=device-width,initial-scale=1"> <link href="/cdn-cgi/styles/challenges.css" rel="stylesheet">

main()
#1 lib/RssBridge.php(15): RssBridge->run()
#2 lib/RssBridge.php(88): DisplayAction->execute()
#3 actions/DisplayAction.php(136): PicukiBridge->collectData()
#4 bridges/PicukiBridge.php(43): getSimpleHTMLDOM()
#5 lib/contents.php(366): getContents()
#6 lib/contents.php(181)

I recognize the cloudflare from the title: <title>Just a moment...</title>


Bug(?) for Force Field

There are 2 books for the same spell but categorized in two different categories. It exists in both Wit and Grace lore books but is learned by Wit methods:

Not sure which it was really intended for but both of them stack separately with their respective bookmarkers (Blue or Green) so we have collectively 9 total but 6 in the wit stack and 3 in the grace stack.
DarkRPG ver. 4.6.0.

I changed Forcefield to a wit spell in the latest update, which didn't update the nbt flag in existing lore books that determines their bookmark color. As such there may be some books floating around with the old tag that are just an artifact of the spell tweaking.


Creating pull request for 10.21105.joss.03739

If this looks good then :shipit:

Yikes, just noticed a sentence fragment in the summary:
Current version:

File types include csv, excel, json, xml (but not pdf). Internal structure choices (e.g., xml tags, or column, row, blank line and header choices for flat files).

Should read:

File types include csv, excel, json, xml (but not pdf), with arbitrary internal structure choices (e.g., xml tags, or column, row, blank line and header choices for flat files).

Correction just made in joss-submission branch


2023.1 support

Hi, it would be great to get it working with 2023.1. Thanks.

Ah, I see, PR is also ready: https://github.com/arhont375/intellij-thrift/pull/3


Adding unit tests for http.

More to add, but this was able to uncover a few corner cases where we could get stuck.
Signed-off-by: Scott Nichols<EMAIL_ADDRESS>

Not done, added a bunch but still working on more coverage..


Run tests on CI

Since this flake is intended for aarch64-darwin platform, I guess we are blocked until github provides such an environment where to run actions - unless someone donates an arm gh runner -. See https://github.com/actions/virtual-environments/issues/2187.

Do you think it can work with a self hosted runner?
On this link it only mentions x86 for macOS: https://docs.github.com/en/actions/hosting-your-own-runners/about-self-hosted-runners#architectures

yep I guess that would be the way to go.


CheckTimeLeft

Issue #, if available:
Description of changes:
Adding new snippet to help decide whether to add a new contact to the queue or take another action (like terminating the call) based on the queue length and the business hours for that day.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for the contribution, you Pull Request has been integrated in release 1.1.0


Assign ADF Pipeline to same trigger multiple times

Cannot assign an ADF Pipeline to same trigger multiple times (with different parameters) via the ADF GUI.  You can modify the trigger JSON directly and add the references, however you cannot do this via the ADF GUI.
When you go to create a new trigger reference from the pipeline, if that pipeline is already associated to that trigger, then it will not show up in the drop down.

There is no document that I know of.


Vih 7946 skip dupe nuget packages

JIRA link (if applicable)
N/A
Change description
Duplicate packaged due to re-run builds causing issues with team delivery as seen here
https://hmctsreform.visualstudio.com/VirtualHearings/_build/results?buildId=36824&view=results
this will skip the push when there are duplicate packages
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

This will require refactoring of all pipelines to pass an Environment variable. Not required just yet so closing the PR


Use GitHub MathJax support in README

The docs index looks great thanks to Documenter.jl, but the math in the README on GitHub wasn't rendering.
I had to put these on their own lines in text for them to render properly separately, for some reason. It's still all one paragraph in the rendered version.

Yes, that is much nicer.   Thank you for the fix.


Possible to display play button without seekbar?

Hi there,
is it possible to dispaly the play button in the center of the screen without displaying the control bar at the bottom?

You would have to not show the controls and use the overlay widget or create custom controls.
chewieController = ChewieController(
      ...
      overlay: YourOverlay(),
      customControls: YouCustomControlls(),
      showControls: false,
);

Thanks. I missed the overlay option somehow :)


Fixing small typo in hpa

@yasn77

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

Ah... my bad. Good catch @ChethanUK. That's difficult to pick in tests since all the k8 clusters I tested with support v2beta2, I so i never hit that code branch :/

pick in tests since all the k8 clusters

Ya.


Hardware sync pyrealsense 2

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ D435}

Firmware Version
(5.14.0)

Operating System & Version
{Win 10) / Ubuntu 22.04

Kernel Version (Linux Only)
( 4.14.13)

Platform
PC

SDK Version
{ 2.53.1 }

Language
{python }

Segment
{others }

Issue Description
I want to try an external trigger, I already set up the function generator and send the trigger signal. Whenever I run the code I got this error:
Cannot change Inter-camera HW synchronization mode while streaming
Here is my code:
import pyrealsense2 as rs
pipe = rs.pipeline()
prof = pipe.start()
dev = prof.get_device()
ds = dev.query_sensors()[0]
ds.set_option(rs.option.inter_cam_sync_mode, 1)
ds.get_option(rs.option.inter_cam_sync_mode)

Hi @dashizm  I recommend moving the inter_cam_sync_mode code to the line before the pipe start instruction.
Also, the script that you are using was originally designed for setting a RealSense camera as the master that generates the trigger signal.  As you are using an external trigger signal though, inter_cam_sync_mode should be set to 2 (slave) instead of 1 (master).  This is because the function generator is acting as the master and not a RealSense camera.  Setting the mode to 1 would be instructing the camera to also generate its own trigger signal instead of listening for the trigger from the function generator.
This is confirmed in the section of the 400 Series multiple camera hardware sync paper linked to below, which states "If an external sync is being provided, then all (cameras) should be set to slave".
https://dev.intelrealsense.com/docs/multiple-depth-cameras-configuration#a-hw-sync-command

Thank you for your reply. The point is that I need to define a device before using the "ds.set_option(rs.option.inter_cam_sync_mode, 2) "command and it needs to define a profile with "pipe. start()".  How I can use the "set_options" command before using the "get_device" command? in other words, I should introduce the sensor to the "set-option" command and it needs to start the pipeline.

https://github.com/IntelRealSense/librealsense/issues/9483 demonstrates using a ctx command to define inter_cam_sync_mode before the pipe start and find the sensor by looking for the first depth sensor available instead of defining a sensor name.
ctx.query_devices()[0].first_depth_sensor().set_option(rs.option.inter_cam_sync_mode, 'mode number')
pipe = rs.pipeline(ctx)

Thanks, it works for me.
ctx = rs.context()

You are very welcome, @dashizm - thanks very much for the update about your success!

Case closed due to solution achieved and no further comments received.


Simplify menu: remove duplicates

On the model
PROCDEF processOne [In1, Out11, Out12 :: Int ] () ::=
    In1 ? x >-> Out11 ! x >-> Out12 ! x >-> processOne [ In1, Out11, Out12 ] ()
ENDDEF

PROCDEF processTwo [In21, In22, Out2 :: Int ] () ::=
    In21 ? x1 >-> In22 ? x2 >-> Out2 ! x1 + x2 >-> processTwo [ In21, In22, Out2 ] ()
ENDDEF

PROCDEF buffer [ In, Out :: Int ] () ::=
    In ? x >-> Out ! x >-> buffer [ In, Out ] ()
ENDDEF

PROCDEF processLine [In1, In2, Out1, Out2 :: Int] () ::=
    HIDE [BIn, BOut :: Int] IN
            processOne [In1, Out1, BIn] ()
        |[ BIn ]|
            buffer [BIn, BOut]()
        |[ BOut ]|
            processTwo [ BOut, In2, Out2] ()
    NI
ENDDEF

CHANDEF Channels ::= In1, In2, Out1, Out2 :: Int ENDDEF

MODELDEF Model ::=
    CHAN IN In1, In2
    CHAN OUT Out1, Out2
    BEHAVIOUR processLine[In1, In2, Out1, Out2]()
ENDDEF

I got after
TXS >> trace
TXS >>  .....1: Act { { ( In1, [ -63 ] ) } }
TXS >>  .....2: Act { { ( Out1, [ -63 ] ) } }
TXS >>  .....3: Act { { ( In2, [ -30 ] ) } }
TXS >>  .....4: Act { { ( In1, [ -20 ] ) } }
TXS >>  .....5: Act { { ( Out1, [ -20 ] ) } }
TXS >>  .....6: Act { { ( Out2, [ -93 ] ) } }
TXS >>  .....7: Act { { ( In1, [ -3 ] ) } }
TXS >>  .....8: Act { { ( Out1, [ -3 ] ) } }
TXS >>  .....9: Act { { ( In2, [ -80 ] ) } }
TXS >>  ....10: Act { { ( In1, [ 74 ] ) } }
TXS >>  ....11: Act { { ( Out2, [ -100 ] ) } }
TXS >>  ....12: Act { { ( Out1, [ 74 ] ) } }
TXS >>  ....13: Act { { ( In2, [ 23 ] ) } }
TXS >>  ....14: Act { { ( Out2, [ 20 ] ) } }

when typing menu
TXS >> menu
TXS >>  [ ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$44$13$1$ ], ( 74 == $"BOut$$$1061"$44$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$46$13$1$ ], ( 74 == $"BOut$$$1061"$46$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$48$13$1$ ], ( 74 == $"BOut$$$1061"$48$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$50$13$1$ ], ( 74 == $"BOut$$$1061"$50$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$53$13$1$ ], ( 74 == $"BOut$$$1061"$53$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$55$13$1$ ], ( 74 == $"BOut$$$1061"$55$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$58$13$1$ ], ( 74 == $"BOut$$$1061"$58$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [ $"BOut$$$1061"$60$13$1$ ], ( 74 == $"BOut$$$1061"$60$13$1$ ) )
          ( { In1[ $"In1"$1066$13$1$ ] }, [], True )
          ( { In1[ $"In1"$1066$13$1$ ] }, [], True )
          ( { In1[ $"In1"$1066$13$1$ ] }, [], True )
          ( { In2[ $"In2"$1067$13$1$ ] }, [], True )
          ( { In2[ $"In2"$1067$13$1$ ] }, [], True )
        ]

Could this be simplified by removing duplicates and even mapping hidden variables onto one?

related to #334


Additional run modes: dry-run & add-only

I just used this tool to sync imdb ratings to trakt, very useful!
Since I already added something to trakt manually, in the logs it showed some deletions: 1 show from watchlist, and 1 movie and 2 shows from history+ratings.
So I'm thinking two additional run modes would be useful:

dry-run: to see what would be applied, and prevent unwillingly erasing data (I have yet to find which movie and 1 of the show it deleted from history)
add-only: to permit syncing only additions

Also, a log file with all stuff that was executed would be perfect in case some huge erasing happens, in order to be able to revert them (even manually). In the github action you could just cat it after run to have it logged.
From my side I plan to only use trakt from now on, so these would be useful additions for others in the future, rather than for me specifically.
Thanks for the effort!

Thanks a lot for the suggestion, @McCio!
I'll be sure to add this functionality soon 🫡


There appear to be duplicate embeddings in the production embedding store

Running reindexCardEmbeddings appears to add embeddings even ones that should already be in the store?
If you look at any given point ID and search by similar you'll find a huge number of duplicates with the same card id and version.

Yeah, the number of vectors in production is 100k, instead of the expected ~15k, 10x larger than expected, likely due to these extra reembeddings.

[ ] Figure out why every time reindexCardEmbeddings is called it's storing a whole new item (did it ever work?)
[ ] Switch back to free tier of Qdrant once it's under control and embedding index rebuilt

My guess is that it's in reindexCardEmbeddings, it's bulk-fetching all of the items, but  the cardsInfo is coming back incorreclty. Looks like it gets the content field but not the card_id field?

BTW this "lots of duplicates of the same card content and embedding" is likely why the semanticSort in #688 was finding so many non-existent embeddings? Maybe? Because it was fetching a random embedding for that cardID?

The bug that has been fixed was leading to a lot of duplicate embeddings being stored, every time reindexCardEmbeddings was run, which was on every deploy.

This is now fixed and deployed into production


Ref build pipeline

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


Default Server is not connected at First Startup. Prompted for Server Address Instead (open.rocket.chat)

My Setup

Operating System: W10
App Version: 3.2.2
Installation type: exe

[ x] I have tested with the latest version
[ x] I can simulate the issue easily

Description
Install fresh rocket.chat install on fresh windows 10 profile. Modify servers.json to add your own default server. Then startup rocket.chat client.
Current Behavior
I can see my default server is loaded in the sidebar, however the client does not prompt for username and password for my server. Instead the focus is on the the open.rocket.chat server. For me, not a big  deal. I can click on my server in the sidebar and select my default server, but for end users they have no clue. They just click connect and end up registering on open.rocket.chat.

Expected Behavior
When I start Rocket.Chat client it should connect to my default server (set in servers.json) automatically and prompt user for credentials to my server.

Please test if the problem persists with our latest release. I will be closing the issue but if there is any problem yet, we will open it again.


Question about the resize in preprocessing pipeline of KITTI

Hello, saic-vul!
Thanks for your great work imvoxelnet. I am curious about the resize operation as it resize images of KITTI into ((1173, 352), (1387, 416)), which is a little different from other kitti mono3d detection methods. The common size of kitti is like (1280, 384) or (1242, 375). I wonder why you choose the strange settings for resize. Thanks a lot !

Hi @gujiaqivadin ,
As you can see for testing we resize the image to the common size of (1280, 384) .
However the training resize that you mentioned is just the same  (1280, 384)  with a small scale augmentation. You can check the Resize class in mmdetection for better unserstanding of this range.

Thanks a lot!


Use skim/fzf for fuzzy finding jobs

The new TUI is pretty awesome, but the native python implementation to complete and find jobs could do with some improvement. Rather than reinventing the wheel here, it may be better simply to call out to a program that already does this quickly and efficiently. Fzf and Skim are two fantastic options, and their APIs are almost completely interchangable, so we could actually use either or and leave it up to the user if desired.

This seems pretty nice. Currently, we're trying to improve the TUI, and these two libs look great.

Hopefully we will soon start working on #989.
We're considering getting rid of main.nix files and rather have everything described in makes.nix files, which are imported via imports = [] like in nixos. That way we won't have to spend CLI init time searching for files within the repo, which gets especially painful in monorepos

Since this is marked as completed, should I understand that the new Makes CLI enables fuzzy matching with fzf (or similar) when looking for jobs instead of the previous "one-letter-longest match" mode ?
(That would be a great improvement indeed because I type much faster than the time it would take me to check what it auto-completed to figure out what the next letter should be.)
(I don't see it in the README screenshot, is that just meant to be updated later?)
Thanks! 😊

The CLI currently does not support fuzzy matching. Once we start working on #989 such feature will most likely be implemented. This issue was closed in favor of #989


TypeError: The "from" argument must be of type string. Received undefined with Docusaurus >=3.6.0

This is a new error with Docusaurus 3.6.x
You can reproduce fairly easily.
npx create-docusaurus-openapi my-website
cd my-website
npm start

will throw it.
I think that fundamentally this is happening because siteDir is not set in
https://github.com/cloud-annotations/docusaurus-openapi/blob/aafdc8a550fa882c379978c19c6d5128c49bad62/packages/docusaurus-plugin-openapi/src/index.ts#L273-L290
Maybe the fix is as simple as setting siteDir to a value like . in there? I haven't dug any further than that.

Could be related to https://docusaurus.io/blog/releases/3.6#plugins


Introduce App monad

AppM monad is Aff + Config. Allows every component to get access to the application config via MonadAsk

Well, on the whole the idea is clear, yes, but it seems quite complicated for me in the way of how to think it out by myself. I'm talking about thoughts and decisions you were guided by deciding which monads to use and in which way to implement. Processes of reviewing someone's code and writing it by your own really differ, and for now I'm sure I won't succeed in creating something like this due to lack of experience and practise even in spite of the fact I got almost everything you've made. So it will be great if you explain not only how it works (btw i have some question with this too), but also the logic and the way of thinking which you used :)
@Unisay


Display metadata in HTML test reports

Fixes #31681.
Metadata keys are printed at the group and test event levels.  Metadata events with multiple keys are rendered together.  Known value types are rendered.  URIs are rendered as clickable links.
See tests in CustomTestTaskHTMLTestReportMetadataTest.

WARN: Based on labels, this pull request addresses notable issue but no changes to release note found.


Claude not work

How do I use it now if Claude doesn't work?

You don't. Having Claude on Slack is kind of a hard requirement for this.

So, Claude in Slack is dead? It doesn't let me add it anymore


Release: 0.8

Things to cover in release notes

Blueprint storage disabled on web.
transform visualization
highlight some bugfixes?
pinhole XYZ viewcoordinates stuff
extrinsics and intrinsics can be logged on same path
upcoming python & c++ apis
2D / 3D Space Views are now separate types
Rust serve api

Regressions to fix before release:

[x] https://github.com/rerun-io/rerun/issues/2758
[x] https://github.com/rerun-io/rerun/issues/2537
[x] https://github.com/rerun-io/rerun/issues/2232
[x] #2507
[x] #2556
[x] #2808
[x] #2690
[x] #2811
[x] #2819
[ ] anything tagged with milestone 0.8

Other easy bugs to fix before release

[x] #2575

it was done!


Use a Monaco Editor for the output as well

In addition to using a Monaco editor for the input, we now use one for the output, and colorize C# outputs with the built-in colorization.

Any ideas on this Jan?

Any ideas on this Jan?

Don't worry about it, automatic deployments of fork branches don't work for some reason, I wasn't yet able to figure out why.


Allow setting membership type in influx org members add

influx org members add is hard-coded to add new users to an org as a "member". We should add a way for users to be added as an "owner" (the API already supports this, so we need to add a toggle to the CLI).
influx org members list and influx org members remove may also need to be updated to account for owners as well as members.

Until this is implemented, you can get effectively the same thing by granting a user an all-access token on the org, since users inherit the permissions of all the tokens that the own:
influx auth create --all-access -u <userNeedingPermission> -o <theorg>

Is it possible to link on how to do this with the API?

What release is this in?

@glacieruk it hasn't been released yet. Next release!


Add Nepali news classification dataset

Checklist for adding MMTEB dataset
Reason for dataset addition:

[x] I have tested that the dataset runs with the mteb package.
[ ] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[ ] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[ ] intfloat/multilingual-e5-small

[ ] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[ ] I have considered the size of the dataset and reduced it if it is too big (2048 examples is typically large enough for most tasks)
[x] I have filled out the metadata object in the dataset file (find documentation on it here).
[ ] Run tests locally to make sure nothing is broken using make test.
[ ] Run the formatter to format the code using make lint.
[ ] I have added points for my submission to the points folder using the PR number as the filename (e.g. 438.jsonl).

Marking this as draft. Feel free to convert it back when you have gone through the PR checklist.


filter update

as branch name said

This pull request will never be merged. Code is destroying working features and doesn't improve any part of the framework.


Generate Genesis Page Style updates

Type of change

Bug fix

Description

UI alignment fix

oh you also have a sytle/lint failure on OSNJoin.js, check the github job logs: https://github.com/hyperledger-labs/fabric-operations-console/runs/7527362224?check_suite_focus=true


[BugFix] exchange source forgets running closure in two cases

Signed-off-by: fzhedu<EMAIL_ADDRESS>What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #https://github.com/StarRocks/starrocks/issues/12981
Problem Summary(Required) ：

the callback function closure-Run() is ignored being called in exchange source. In detail:
place 1
https://github.com/StarRocks/starrocks/blob/c354c8f0933342454edf5bde4689437fa35db516/be/src/runtime/sender_queue.cpp#L677-L689
https://github.com/StarRocks/starrocks/blob/c354c8f0933342454edf5bde4689437fa35db516/be/src/runtime/sender_queue.cpp#L496-L536
before enqueue blocks into _chunk_queues, the cancelled status may be set, so the new added blocks's callback functions are not called anymore.
place 2
https://github.com/StarRocks/starrocks/blob/c354c8f0933342454edf5bde4689437fa35db516/be/src/runtime/sender_queue.cpp#L417-L435
if early return at L420, the existing closure is not called anymore.
by the way, append fragment_status to the http interface.
Checklist:

[ ] I have added test cases for my bug fix or my new feature (mannel test 20 times ok)
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

run starrocks_be_unittest

run starrocks_be_unittest

@mergify backport branch-2.5


Quick Fix: Suggest correct capitalization when importing a file with incorrect capitalization

TypeScript Version:  2.3
Mac does not enforce file name capitalization. I had an issue where tests was passing on my Mac and I was getting an error like this in Linux:
Cannot find module './isKind' from 'index.ts'

File was named iskind by mistake.
It should be easy to detect errors like this by reading all files in target directory and look for similar file names.  This can go beyond capitalization and can suggest correct file names with edit distance.
Code
import {foo} from './myModule'

Would love to take a PR for this.


Use the className param in ol.layer.Heatmap

fixes #11542

Looking good, thanks! Some tests are still failing though, not sure why?

it was the prettier test, it's now fixed


Fix for duplicates in Element tree

Looks like this actually broke back in #237. We missed 1 tiny detail in the middle of a very large refactoring job.
When we transmit DOM changes to Chrome, we need to keep track of all the elements for each transmission so we don't send over duplicates. When we send over DOM.childNodeInserted(E), which includes E and its whole sub-tree "by value", we must not send a subsequent DOM.childNodeInserted(*) for any elements within E's sub-tree, otherwise they'll show up as duplicates in the DevTools UI.
This was managed with a HashSet indirectly populated by DOM.createNodeForElement() and checked by ShadowDocument.applyDocumentUpdate(). The code is still there in the latter, but it got lost in the former. It was only storing the root node of any transmitted sub-tree into the HashSet (indirectly via the Accumulator interface).
Closes #339

lgtm, though I would like to see the JSON unit test improved a bit.


⚠ Bofrost.at has degraded performance

In 7d46900, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 815 ms

Resolved: Bofrost.at performance has improved in 6a19fe1 after 7 minutes.