Implement modulo(%) binary operator

e.g.
12 % 3 = 4
10 % 12 = 10
..

Following tasks are expected to be done

add fn modulo to data/value/mod.rs
You can also add unit tests of this new modulo fn, there's a existing test case named arithmetic and you can use it.

add fn modulo to data/literal.rs
Similar to Value, but.. there's lack of unit tests though it is required.
You need to add a new one.

add fn modulo to execute/evaluate/evaluated.rs

add integration test cases of modulo to src/tests/arithmetic.rs
Here you can also cover code branches of evaluated.rs by adding % test cases to arithmetic.

ref https://github.com/gluesql/gluesql/pull/295

I'll do those tasks 👍


add app "Code RPGify"

Added our app "Code RPGify".
hope you merge this :)

Awesome, thanks for the contribution! ✨

hehe. It is great! thank you ;)


Text added for testing

Merging this just to trigger the failure in deploy-book and hopefully debug what's going on.


[WIP] Add interface for configuring ANR detection on Android builds

Goal
Adds a public interface on the Unity notifier for configuring ANR detection.
I've marked this as WIP as bugsnag-android still needs to be released and because this change still requires testing with changes to the NDK. The PR is ready for review of the C# interface, however.
Changeset

Adds DetectAnrs and AnrThresholdMs properties to the C# API
Calls setters on the Android configuration when initialising the native client
Updates bugsnag-android to a branch off the current master, with the ANR detection changes merged

Tests
I have verified the example app compiles and will test that ANRs are detected manually when changes to the NDK have been completed.

Why are the configuration classes now public?


How can I use the raw native api?

There have been cases where mongoskin incorrectly wraps the native api, and I'd like a way to circumvent mongoskin to call mongo native directly. However, doing something like this:
mongo._native.collection(collectionName).listIndexes()
gives me the error: no database. So how do I properly call on the native connection?

Same issue if I use mongoskin.open

Seems it was an error case from native: https://jira.mongodb.org/browse/NODE-595


Avoid unnecessary string allocation when calculating length

Description
Hi, I am trying to optimize YARD a bit to speed up documentation runs on a large codebase.
The codebase is private, so I am using https://github.com/watir/watir/ as a smaller stand-in to report benchmarking numbers from https://github.com/SamSaffron/memory_profiler.
Test set-up

Ruby 2.7.6 (installed via rvm on Linux)
Watir version: 293bed2b57c
MemoryProfiler version: 1.0.0
YARD baseline: 0a550939f9b
Test command:

ruby-memory-profiler --no-color --retained-strings=500 --allocated-strings=500 --max=300 -o prof.log ./run-yard.rb stats

run-yard.rb is a smaller version of /bin/yard (MemoryProfiler was not able to run /bin/yard)
I am using the stats command to avoid memory exhaustion

Performance data

I don't have concrete performance data because the stats command doesn't seem to trigger this code path
On the private codebase, the loop body was executed  ~375k times, so I think this optimization is worthwhile

Completed Tasks

[x] I have read the Contributing Guide.
[x] The pull request is complete (implemented / written).
[x] Git commits have been cleaned up (squash WIP / revert commits).
[x] I wrote tests and ran bundle exec rake locally (if code is attached to PR).

Thank you for this PR!


Safety issues 2024-02-11

No review needed.

coverage: 25.088%. remained the same
when pulling 5c8af746fde817c43fdec1094f4e6a1c48b50ef9 on andy/safety-2024-02-11
into 93c8b8a80e5aaf30b4fb5182f94b8a60b800e71a on master.


Modified releasenote about Beaver

The behavior described in the current releasenote
for beaver is no longer needed.
The releasenote has been updated to reflect the
new behavior.
Connects #1541

Your content preview is now ready. :bow:

There was some concern around changing existing releasenotes, and how that might cause some release notes to disappear from earlier tags, and to reappear in the subsequent one. However, my understanding is that, because of the way we tag our releases, reno doesn't exactly recognize our tags, thus lumping the releasenotes together.
@git-harry Did I explain that correctly?

Closing this in favor of not changing existing reno release notes


Validate desired network states

The following example is a completely valid desired state for nmstate:
interfaces:
- name: eth0
  type: ethernet
  state: up
  ipv4:
    address:
    - ip: <IP_ADDRESS>
      prefix-length: 24
    enabled: true
  ipv6:
    address:
    - ip: 2001:db8::1:1
      prefix-length: 64
    enabled: true

However, since it is missing a MAC address for the interface, host mapping will not be properly generated which in turn would make applying the configuration fail:
$ cat _out/host_config.yaml
- hostname: node1
  interfaces: []

Consider adding --dry-run option to nmc generate as a validation step before running NMC.

Investigate whether interface names can contain dots since we're currently replacing the file extension when applying the .nmconnection files.
Involves https://github.com/suse-edge/nm-configurator/issues/19 as part of the research.


Very unperformanced CaseSensitivePathsPlugin.

Bug report
Describe the bug
Very slow building time using npm run dev. CaseSensitivePathsPlugin takes a significant time to proceed.
Fast machine Core i7

Laptop Pentium

To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Install SMP to detect the time
Wrap the config with new SpeedMeasurePlugin().wrap(config)
run npm run dev
See the screenshot above

Expected behavior
The first plugin took a lot of time to execute. Should not be a issue in every repo.
System information

OS: Windows
Version of Next.js: [ 8.0.3]

Additional context
The issue could be caused by aliases, but no time to check.

I've tested this plugin with { debug: true } and found that it's checking all of the modules including node_modules, so it does 2k+ reads every time

[CaseSensitivePathsPlugin] Total filesystem reads: 2566

8.0.4 doesn't have CaseSensitivePathsPlugin for this reason.

Speed measure plugin looks nice btw, hadn't seen it before 💯

Speed measure plugin looks nice btw, hadn't seen it before 💯

Could you include this measure, btw? The thing is when I wrap it, it breaks further work: next.js does not render anything and a browser does not open the app.
So, to measure I need to enable it, then disable to test is app loading ok. I think, the reason is SMP is not a plugin, but a config wrapper

@timneutkens
After installing next 8.0.4
I've caught the SMP output (via ScreenToGif), cause you clear the console output xD.

Thanks!

🙏


Replacing Custom Tensorflow Operation

Hi trigeorgis,
Instead of installing your custom TF repo to gain the custom extract image patches operation, I tried to write the operation in python with TF 1.0. This is what I came up with:
`
        unset_patches = []
        for offset in tf.unstack(inits+dx, axis=1):
            batch_patches = tf.image.extract_glimpse(images, patch_shape, offset)
            unset_patches.append(batch_patches)
        # patches = tf.transpose(unset_patches, (1, 0, 2, 3, 4))
        patches = tf.stack(unset_patches, axis=1)
        patches = tf.reshape(patches, (batch_size * num_patches, patch_shape[0], patch_shape[1], num_channels))`

From here training is able to run, but no learning takes place and losses hover around 1.0 for thousands of steps.
Is there something I'm missing with this? Perhaps the optimizer can't back propagate losses through the operations taking place in python here, or there's an error in the code above when extracting and stacking these patches?
Let me know if you'd like any more information.
Help is greatly appreciated!

I began writing some code which does not rely on custom c++ ops that's in https://github.com/trigeorgis/mdm/tree/refactored so you can have a look at that.


Can't apply external force

Hi, I cannot apply an external force to the softbody.
The softbody itself behaves exactly how I want it - it's a soft circular blob.
Now I want to fire at that blob and make it react. How can I achieve it?
Functions such as apply_impulse() don't work

Hi.
The softbody itself contains a lot of rigidbodies. I could provide a function that applies impulse on all of them, but I'm not sure thats whats expected.
You van try getting all rigidbodies children and try some logic. You can also get the center rigidbody and move that.

As I'm a little newbie in this I would love to see an example or two

Ill try to add one when i have some time. What is ur use case though? What are you expecting to happen in your game? How do you want to move the softbody?

I want to shoot at the softbody blob, make it have a visual impact in place of collision, as well as change the blob's position in the direction of the shooting

Can you provide also an image or something of what you want? Or if you find in an example somewhere or if you can replicate with a rigidbody.

So roughly, here's the idea:

Imagine a balloon, filled with water. It stays in a resting state
An external force is applied to the left of balloon with the force direction Vector2(+1, 0). Example: balloon was kicked or shoot with a rubber bullet.
Balloon is moved to the right, as an impact of a force applied to it's left corner
Balloon returns to a resting state

Interesting. And is this not what happens when you hit the softbody right now? That would be what I would expect to happen without needing to apply any force to it.

I fire bullets at it and push myself against the softbody, however it has no effect. The physics work only with gravity. Maybe I'm missing something in my setup though.
The softbody is setup this way:
Node2D (root node)
-- SoftBody2D (texture is provided, bones are present)
If I simply drop the body from air against a surface, it works exactly as it should. But the problem is I don't understand how to apply external forces to it, like an impulse to the side

Can you provide an example project or a video?

A test project where you hit a softbody and it doesnt move. In my tests the softbody moves if it gets hit by another body, even if gravity is 0.

Where can I see you project? It may help me understand what I'm missing

Not unless you get all rigidbodies and check which one is closest.

I'm adding now some new samples for your use case though.

Thank you very much!

Meanwhile I did a few more tests myself - I created a softbody (its just a solid circle texture) with the following parameters:
Vertex interval - 173
Radius - 10
Total Mass - 10
Shape Type - CIrcle
Joint type - pin
No physics material override
Texture size is 512x512

Scene hierarchy is the following:
CharacterBody2D (Root)
-- SoftBody2D
-- RigidBody2D (for bullet collision test)
---- CollisionShape2D
-- StaticBody2D (for ground collision test)
---- CollisionShape2D
CharacterBody2D node transform is scaled down to 0.1 on both axes.
The idea was to test how the softbody reacts to collision against an external force - a bullet that hits the softbody from horizontal direction. I set bullet mass to a ridiculous number (1000000 kg). The softbody mass is 10 kg. The bullet speed is 2000 * delta in X direction.
Test result: softbody barely notices the bullet. Bullet does hit the softbody, I can see it in the debug view but softbody doesnt react to it

Can you as well send a video of this? From screenshot seems like the softbody isnt constructed but its a voronoi diagram instead.

Also if u can get on project discord, itll be easier to check whats wrong there.

Scale indeed was one of the reasons it didnt work very well. I think node scale forfeits bone radius, which results in wrong behaviour when external rigid body collides into bones

Added an example with apply_force in tutorial.gd file.


Roll Flutter Engine from b73367a30e9b to ca94c3ad9f01 (7 revisions)

https://github.com/flutter/engine/compare/b73367a30e9b...ca94c3ad9f01
2024-07-31<EMAIL_ADDRESS>Roll Skia from fa9ecbb258b1 to 999e3f113dbf (1 revision) (flutter/engine#54248)
2024-07-31<EMAIL_ADDRESS>Roll Dart SDK from f10a21fcc61c to edace067d950 (1 revision) (flutter/engine#54246)
2024-07-31<EMAIL_ADDRESS>Roll Skia from cb6ea5bd807d to fa9ecbb258b1 (3 revisions) (flutter/engine#54245)
2024-07-31<EMAIL_ADDRESS>Roll Dart SDK from 68e559110999 to f10a21fcc61c (2 revisions) (flutter/engine#54244)
2024-07-31<EMAIL_ADDRESS>Roll Skia from 26eb1876de72 to cb6ea5bd807d (2 revisions) (flutter/engine#54241)
2024-07-31<EMAIL_ADDRESS>Roll Skia from b6036baeebbd to 26eb1876de72 (1 revision) (flutter/engine#54239)
2024-07-31<EMAIL_ADDRESS>[iOS][macOS] Eliminate use of bitcode_strip (flutter/engine#54240)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Font change

See I want to change the font to JetBrains mono but everytime I search for something nothing shows up so please help
Nix-on-droid bootstrap aarch64 23.11

Define

everytime I search for something nothing shows up

Like I mean I don't see any search results of people changing the font on nix-on-droid

Well, there's an option for changing the font, implemented in a PR filed as a response to an issue.


[stable/falco] Performs a rollingUpdate when you change rules or sett…

…ings
Signed-off-by: Néstor Salceda<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
This PR allows chart to do a rollingUpgrade when you changes settings in falco configuration or if you add more rules.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @bencer


idf.py does not automatically find serial ports to flash (IDFGH-4600)

Environment
Module or chip used: ESP32-WROOM-32D
IDF version: v4.3-dev-2586-g526f682397a8
Build System: idf.py
Compiler version: xtensa-esp32-elf-gcc (crosstool-NG esp-2020r3) 8.4.0
Operating System: Linux
Power Supply: USB
Problem Description
Notice below issue after pull recent esp-idf changes:
$ idf.py  encrypted-app-flash monitor
Executing action: encrypted-app-flash
No serial ports found. Connect a device, or use '-p PORT' option to set a specific port.
// Below works
$ idf.py  encrypted-app-flash monitor -p /dev/ttyUSB0
Note, it was working with esp-idf: v4.3-dev-2398-g2bfdd036b2db

We are sorry for the inconvenience. We already fixed it and will be published soon.

We are sorry for the inconvenience. We already fixed it and will be published soon.

I ran into this also on a fresh install today (under Linux) and tracked it down to this line in serial_ext.py:
esp = esptool.get_default_connected_device(serial_list=ports, port=None, connect_attempts=3, initial_baud=args.baud)

It seems there is no function named get_default_connected_device in the esptool module.  It does exist in esptool.py, but not in commit 8270199, which is what's being used currently.

I ran into this also on a fresh install today (under Linux) and tracked it down to this line in serial_ext.py:
esp = esptool.get_default_connected_device(serial_list=ports, port=None, connect_attempts=3, initial_baud=args.baud)

It seems there is no function named get_default_connected_device in the esptool module.  It does exist in esptool.py, but not in commit 8270199, which is what's being used currently.

I update my tree to v4.3-dev-2940-g6e776946d01e
It's very easy to hit below issue.
(Also note, if I add -p /dev/ttyUSB0 it will always work)
$ idf.py monitor
Executing action: monitor
Serial port /dev/ttyUSB0
Connecting.................._____
/dev/ttyUSB0 failed to connect: Failed to connect to Espressif device: Timed out waiting for packet header
No serial ports found. Connect a device, or use '-p PORT' option to set a specific port.

I'm reopening this because there is another followup bug.

Just to remind the v4.3 still needs this fix. (The fix is only in master now)

Thank you @AxelLin for the reminder.

v4.3-beta3 does not include the fix. Maybe need to re-open the issue in case if it get forgetton.

I'm sorry @AxelLin for the inconvenience. It was merged internally a week ago but unfortunately it didn't make it into v4.3-beta3 which was created for testing earlier. Reopening the issue won't make this process faster. I'm sorry for that.


Tooltip example broken in Safari

In Safari (tested with version 8.0.5) on OS X, the example on the Tooltips page is broken. In Chrome, everything is fine.

That's because Autoprefixer isn't run for the CSS in the docs and Safari still needs the vendor prefix for transforms.

Yep, I saw that too. In the live version of Primer, Autoprefixer fixes this issue — all you need is a -webkit-transform.
On Tuesday, Mar 24, 2015 at 06:54, Heinrich Fenkart<EMAIL_ADDRESS>wrote:
That's because Autoprefixer isn't run for the CSS in the docs and Safari still needs the vendor prefix for transforms.
—
Reply to this email directly or view it on GitHub.


on Windows OS lower then 8.1 puppy is not working

This is my workaround. I modified platform.nim and windefs.nim
#windefs.nim
WINHTTP_ACCESS_TYPE_DEFAULT_PROXY* = 0

#platform.nim
import osinfo\win
let osvi = getVersionInfo()
let win81andup = osvi.majorVersion>6 or (osvi.majorVersion==6 and osvi.minorVersion>=3)

...
hSession = WinHttpOpen(
      cast[ptr WCHAR](wideUserAgent[0].unsafeAddr),
      if win81andup: WINHTTP_ACCESS_TYPE_AUTOMATIC_PROXY else: WINHTTP_ACCESS_TYPE_DEFAULT_PROXY,
      nil,
      nil,
      0
    )
...

of course osinfo must be installed with nimble install osinfo

Hello, thanks for reporting this. I am unable to test to confirm this works, nor really be sure Puppy works on pre-Windows 10 at all. Given everything pre-Windows 10 is not supported even by Microsoft, I'm not eager to take on responsibility for Puppy working on something I cannot test or then need to worry about over time. If you need this, maintaining a fork of Puppy for old versions of Windows is probably easiest since you can be sure it works for what you need but don't need to worry about others. We'll see what @treeform says.

be sure it works for what you need but don't need to worry about others. We'll see what @treeform says.
just letting you guys know. this is probably is not needed for 99.5% of people :-) but I had a few bug reports from people.saying that my software is not working on windows 7. so some dinosaurs still use it on old laptops for listening to music in their garages.

Instead of checking version codes which look like their own form of arcane knowledge (>= 6.3 == 8.1?), we could just try and then fall back on error as #71

Instead of checking version codes which look like their own form of arcane knowledge (>= 6.3 == 8.1?), we could just try and then fall back on error as #71

or just let user specify it as optional parameter to a procedure. it will be more simple and clean.

No, that doesn't make sense on Linux or Mac. I strongly oppose bringing anything platform specific into the cross-platform API especially something for vintage Windows like this.

We talked about this and given we do not know the differences between these settings and cannot test them we're not making any changes now. Basically https://github.com/treeform/puppy/issues/70#issuecomment-1094124598


"on player fishes" returns an error on <context.entity>

"on player fishes" will pretty much always return an error when I use <context.entity>.
Can it be set to null if there is no entity instead of breaking the entire tag/script?
Here is the debug, when I try to call <context.entity> whenever I use the fishing rod:
http://mcmonkey.org/paste/22202
Notice I first right-click to throw the hook and returns the error, it hits the pigs, then I right-click again to pull it back and the tag returns correct, being the pig.

this is why we have fallbacks.
<context.entity||null>
- if <context.entity||null> == null {
  - queue clear
}

Oh, right... whoops!


Can't run train.py because thinplate is missing

I can't run the script train.py because I don't have the thinplate package. The problem is that it was installed from a local package and therefore I can't install it on my pc:
https://github.com/hkchengrex/STCN/blob/4d8b66fad07ef811f23b012c0af6f765ea767245/docs/packages.txt#L233

Have you solved this problem?

Have you solved this problem?

It is explained in the README https://github.com/hkchengrex/STCN#requirements

thankyou!


infinite recursion with read_terragrunt_config

if child terragrunt file B includes parent terragrunt file A, and A reads B with read_terragrunt_config, terragrunt will be stuck in an infinite loop of evaluating file B's include block, then file A's include block, then back to file B's include block, etc. when this happens no error is printed, the only way to tell is to look at debug logs, which will print messages like DEBU[0059] Found locals block: evaluating the expressions. forever. file A:
locals {
  child_terragrunt_hcl = read_terragrunt_config("${get_original_terragrunt_dir()}/terragrunt.hcl")
}

file B:
include "global_configs" {
  path           = find_in_parent_folders()
  merge_strategy = "deep"
  expose         = true
}

I tried doing a string replace on the contents of file B before reading it with read_terragrunt_config and replacing it with a different path that just sources an empty terragrunt config, but the function doesn't take a string, only a filepath, and i can't think of a way to get file B, do the necessary string replacement, then write the file somewhere else, and anyway that's an awkward way to do it.
it'd be nice for there to be some smarts in the way read_terragrunt_config parses the include block - maybe some kind of check to make sure the include "tree" doesn't ever include the same file twice? being able to actually do this without encountering the forever loop would be ideal, but some kind of error instead of just hanging (assuming log level < debug) is probly a minimal fix

btw if anyone knows a workaround for this i would love to hear it. specifically I need to somehow get the value of the child terragrunt dependencies block from inside the parent terragrunt file

Hi,
depends on what is required to achieve, if it is required to load child-specific configuration - configuration can be extracted in separated HCL file and loaded from parent.
# parent.hcl
locals {
  child_terragrunt_hcl = read_terragrunt_config("${get_original_terragrunt_dir()}/config.hcl")
}

# child terragrunt.hcl

include "global_configs" {
  path           = find_in_parent_folders("parent.hcl")
  merge_strategy = "deep"
  expose         = true
}

inputs = {
  x = include.global_configs.locals.child_terragrunt_hcl.locals.data
}

# child config.hcl

locals {
  data = "child config"
}

Full example:
https://github.com/denis256/terragrunt-tests/tree/master/recursion/app


Add no_proxy to index searches

Adding no_proxy option for index searches.
Fixes #5497

 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.Dennis Whitney seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Resolves #5497

@armab Updated

@armab I've done another commit to do similar to what the bug fix is supposed to provide from the requests library.
Could you also do me a favor and just double check my upstream merge. I get confused on dealing with merge conflicts and such.

I thought about writing a test earlier, but I'm not sure how I would write a test for this. I'm still learning on writing good unit and/or integration testing. This one is particular difficult in my mind because of the looping mechanism. Any suggestions?


Where are the links?

Please provide links to each of the configuration/setup items listed on this page.

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

ID: 9a33e2f6-15ac-c9d1-b9f8-257c6f64ca6b
Version Independent ID: 50108411-90ad-07c4-0a6a-2282f9f49c33
Content: Set up unified routing
Content Source: ce/customer-service/set-up-routing-process.md
Service: dynamics-365-customerservice
GitHub Login: @neeranelli
Microsoft Alias: nenellim

Hello @anbhar
Thank you for your inputs. We are pushing out an update by next week and the links will be updated. I will update this thread soon as it is available.
Best regards,
Neeraja

@anbhar
The How unified routing and assignment works article has been updated with links. Please let me know if you have further questions.
Thank you.
Neeraja

Closing the issue as no other action is required and the user hasn't responded.


RouteApi routeId issues

Describe the bug
The actual id for a route can differ from the RouteApi's constructor ids shown from TS when using routegen and "layouts"
Your Example Website or App
https://stackblitz.com/~/github.com/V1RE/tanstack-router-route-api-issue
Steps to Reproduce the Bug or Issue
The layout-a page shows issues when trying to find the route based on the ts autocompleted route id.
Expected behavior
As a user I expect the autocompletions for the routeApi to resolve to the correct route id's
Screenshots or Videos

Platform
Irrelevant
Additional context
Could either be solved by making the routeApi constructor match the id transformation of the route constructor, or by changing the auto suggestions, but the latter option seems counter-intuitive

This seems to be a problem of file based routes, rather than RouteApi. Same issue with e.g. useNavigate(from: '...') hook.

@schiller-manuel It seems to stem from the logic in the Route constructor and the way the route get's matched based on the "from" property if I'm not mistaken

I think the issue lies in the generated code:
import { Route as rootRoute } from "./routes/__root"
import { Route as IndexRoute } from "./routes/index"
import { Route as LayoutRoute } from "./routes/_layout"
import { Route as PostsRoute } from "./routes/posts"
import { Route as PostsIndexRoute } from "./routes/posts.index"
import { Route as LayoutLayoutARoute } from "./routes/_layout/layout-a"
import { Route as LayoutLayoutBRoute } from "./routes/_layout/layout-b"
import { Route as PostsPostIdRoute } from "./routes/posts.$postId"
import { Route as PostsPostIdDeepRoute } from "./routes/posts_.$postId.deep"

declare module "@tanstack/react-router" {
  interface FileRoutesByPath {
    "/": {
      parentRoute: typeof rootRoute
    }
    "/_layout": {
      parentRoute: typeof rootRoute
    }
    "/posts": {
      parentRoute: typeof rootRoute
    }
    "/posts/": {
      parentRoute: typeof PostsRoute
    }
    "/_layout/layout-a": {
      parentRoute: typeof LayoutRoute
    }
    "/_layout/layout-b": {
      parentRoute: typeof LayoutRoute
    }
    "/posts/$postId": {
      parentRoute: typeof PostsRoute
    }
    "/posts_/$postId/deep": {
      parentRoute: typeof rootRoute
    }
  }
}

Object.assign(IndexRoute.options, {
  path: "/",
  getParentRoute: () => rootRoute,
})

Object.assign(LayoutRoute.options, {
  id: "/layout",
  getParentRoute: () => rootRoute,
})

Object.assign(PostsRoute.options, {
  path: "/posts",
  getParentRoute: () => rootRoute,
})

Object.assign(PostsIndexRoute.options, {
  path: "/",
  getParentRoute: () => PostsRoute,
})

Object.assign(LayoutLayoutARoute.options, {
  path: "/layout-a",
  getParentRoute: () => LayoutRoute,
})

Object.assign(LayoutLayoutBRoute.options, {
  path: "/layout-b",
  getParentRoute: () => LayoutRoute,
})

Object.assign(PostsPostIdRoute.options, {
  path: "/$postId",
  getParentRoute: () => PostsRoute,
})

Object.assign(PostsPostIdDeepRoute.options, {
  path: "/posts/$postId/deep",
  getParentRoute: () => rootRoute,
})

export const routeTree = rootRoute.addChildren([
  IndexRoute,
  LayoutRoute.addChildren([LayoutLayoutARoute, LayoutLayoutBRoute]),
  PostsRoute.addChildren([PostsIndexRoute, PostsPostIdRoute]),
  PostsPostIdDeepRoute,
])

The path does not match the id, see /layout vs /_layout.

Ah! Good one, I have some spare time, will try to create a PR

@schiller-manuel Tried changing the logic of the codegen, but ran into TS issues when removing the underscores there, they seem to be important for inferring the route context types correctly. Changing the logic in matches.tsx seems to fix the issue, but might have overlooked something.


Move back to ask-fsdl collection name

Switched to ask-fsdl-llm while we were maintaining two separate instances, should switch it back

closed in #23


Netcdf3 support

Do we expect/plan to support netcdf3 for CMIP6+/CMIP7?
If the answer is yes, we need to track and support https://github.com/TomNicholas/VirtualiZarr/issues/86

I don't think you need to follow https://github.com/TomNicholas/VirtualiZarr/issues/86 - we will support generating references from netCDF3 files regardless. That issue is more about whether we can do something clever with the chunking in the case of uncompressed data.

FYI virtualizarr has supported generating references from netCDF3 files for a while now.


Converting Donut model results in IndexError: index out of range in self

System Info
optimum version: 1.13.2
platform: linux
python version: 3.10.12

Who can help?
@fxmarty @michaelbenayoun
Information

[ ] The official example scripts
[x] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)
Running
optimum-cli export onnx -m naver-clova-ix/donut-base-finetuned-rvlcdip out

Results in the following error:
Traceback (most recent call last):
  File "/usr/local/bin/optimum-cli", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.10/dist-packages/optimum/commands/optimum_cli.py", line 163, in main
    service.run()
  File "/usr/local/lib/python3.10/dist-packages/optimum/commands/export/onnx.py", line 232, in run
    main_export(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/__main__.py", line 486, in main_export
    _, onnx_outputs = export_models(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/convert.py", line 752, in export_models
    export(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/convert.py", line 855, in export
    export_output = export_pytorch(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/convert.py", line 572, in export_pytorch
    onnx_export(
  File "/usr/local/lib/python3.10/dist-packages/torch/onnx/utils.py", line 506, in export
    _export(
  File "/usr/local/lib/python3.10/dist-packages/torch/onnx/utils.py", line 1548, in _export
    graph, params_dict, torch_out = _model_to_graph(
  File "/usr/local/lib/python3.10/dist-packages/torch/onnx/utils.py", line 1113, in _model_to_graph
    graph, params, torch_out, module = _create_jit_graph(model, args)
  File "/usr/local/lib/python3.10/dist-packages/torch/onnx/utils.py", line 989, in _create_jit_graph
    graph, torch_out = _trace_and_get_graph_from_model(model, args)
  File "/usr/local/lib/python3.10/dist-packages/torch/onnx/utils.py", line 893, in _trace_and_get_graph_from_model
    trace_graph, torch_out, inputs_states = torch.jit._get_trace_graph(
  File "/usr/local/lib/python3.10/dist-packages/torch/jit/_trace.py", line 1268, in _get_trace_graph
    outs = ONNXTracedModule(f, strict, _force_outplace, return_inputs, _return_inputs_states)(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/jit/_trace.py", line 127, in forward
    graph, out = torch._C._create_graph_by_tracing(
  File "/usr/local/lib/python3.10/dist-packages/torch/jit/_trace.py", line 118, in wrapper
    outs.append(self.inner(*trace_inputs))
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1488, in _slow_forward
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/model_patcher.py", line 197, in patched_forward
    outputs = self.orig_forward(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/vision_encoder_decoder/modeling_vision_encoder_decoder.py", line 609, in forward
    decoder_outputs = self.decoder(
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1488, in _slow_forward
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/mbart/modeling_mbart.py", line 1850, in forward
    outputs = self.model.decoder(
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1488, in _slow_forward
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/mbart/modeling_mbart.py", line 1049, in forward
    positions = self.embed_positions(input, past_key_values_length)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1488, in _slow_forward
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/transformers/models/mbart/modeling_mbart.py", line 134, in forward
    return super().forward(positions + self.offset)
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/sparse.py", line 162, in forward
    return F.embedding(
  File "/usr/local/lib/python3.10/dist-packages/torch/nn/functional.py", line 2210, in embedding
    return torch.embedding(weight, input, padding_idx, scale_grad_by_freq, sparse)
IndexError: index out of range in self

Expected behavior
The model should export correctly. I have tested the model itself, and it operates correctly when using the example code in the docs (see first example for "Step-by-step Document Image Classification").

This is probably a bug in the finetuned model config.
The base model has max_position_embeddings>max_length while the finetuned model has max_position_embeddings<max_length.
To accommodate you can set --sequence_length=8 in your export command.

Thank you @xenova @baskrahmer, indeed the model itself is broken, see https://huggingface.co/naver-clova-ix/donut-base-finetuned-rvlcdip/discussions/2. Feel free to open a PR on the Hub!


:bug: Ask user to 'save as' if save fails when closing tab, fixes #7708

This probably needs testing on mac where users are prompted for credentials when saving files they don't have permissions for.

This fails the list test, I'll fix it and push a new one


feat(Package): Travis CI

Update Travis CI configuration

Coverage remained the same at 62.5% when pulling 8ac53264b418ce3310457532b9933f71cd8852f0 on feat/package/travis into 4ad0d9e969d79e00002a938f48cd1b502c737037 on develop.


Moved CI from Github Actions to AZ Pipelines

CI now includes e2e tests from previous ADO pipeline (https://dev.azure.com/AzureContainerUpstream/Kubernetes/_git/ado-build-definitions?path=%2Fpipelines%2Fosm-helm-chart.yml) as well as tests that were previously in GH workflow, makefile, and tests folder. I also removed the makefile, and .github and tests folders, as these e2es are now in the pipeline.

Are you still planning on removing the Makefile? I think it'll be good to keep it so that people can run the tests locally. What do you think @SanyaKochhar

Are you still planning on removing the Makefile? I think it'll be good to keep it so that people can run the tests locally. What do you think @SanyaKochhar
I actually re-added it in the PR, let me update description

Please squash the commits before merge.

Are you still planning on removing the Makefile? I think it'll be good to keep it so that people can run the tests locally. What do you think @SanyaKochhar

@nshankar13 happy to see the Makefile is back, it's definitely really useful for local testing


ARM Template Extension fails to validate Azure Recovery Services Backup Policy

Hi,
I am trying to create a vm snapshot policy using Bicep (AzureIaasVM), which itself seems to be fine but when the ARM template is produced, it doesn't seem to validate that the template is valid as documented here - [https://learn.microsoft.com/en-us/azure/templates/microsoft.recoveryservices/2022-03-01/vaults/backuppolicies?pivots=deployment-language-bicep](Backup Policy Template )

Bicep Code:
resource vmSnapshotBackupPolicy 'Microsoft.RecoveryServices/vaults/backupPolicies@2022-03-01' = {
  name: 'vm-snapshot-policy'
  location: resourceLocation
  parent: recoveryServicesVault
  properties: {
    protectedItemsCount: 0
    backupManagementType: 'AzureIaasVM'
    instantRPDetails: {
      azureBackupRGNamePrefix: 'vmsnapshot'
    }
    instantRpRetentionRangeInDays: 2
    policyType: 'V1'
    retentionPolicy: {
      retentionPolicyType: 'LongTermRetentionPolicy'
      dailySchedule: {
        retentionDuration: {
          count: 7
          durationType: 'Days'
        }
        retentionTimes: [
          '2022-10-21T02:00:00Z'
        ]
      }
      monthlySchedule: {
        retentionDuration: {
          count: 6
          durationType: 'Months'
        }
        retentionScheduleDaily: {
          daysOfTheMonth: [
            {
              date: 1
              isLast: false
            }
          ]
        }
        retentionScheduleFormatType: 'Daily'
        retentionTimes: [
          '2022-10-21T02:00:00Z'
        ]
      }
      weeklySchedule: {
        daysOfTheWeek: [
          'Sunday'
        ]
        retentionDuration: {
          count: 4
          durationType: 'Weeks'
        }
        retentionTimes: [
          '2022-10-21T02:00:00Z'
        ]
      }
      yearlySchedule: {
        monthsOfYear: [
          'July'
        ]
        retentionDuration: {
          count: 2
          durationType: 'Years'
        }
        retentionScheduleDaily: {
          daysOfTheMonth: [
            {
              date: 1
              isLast: false
            }
          ]
        }
        retentionScheduleFormatType: 'Daily'
        retentionTimes: [
          '2022-10-21T02:00:00Z'
        ]
      }
    }
    schedulePolicy: {
      schedulePolicyType: 'SimpleSchedulePolicy'
      scheduleRunFrequency: 'Daily'
      scheduleRunTimes: [
        '2022-10-21T02:00:00Z'
      ]
      scheduleWeeklyFrequency: 0
    }
    timeZone: 'UTC'
  }
}

JSON:
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "metadata": {
    "_generator": {
      "name": "bicep",
      "version": "<IP_ADDRESS>0",
      "templateHash": "13131918354489128151"
    }
  },
  "variables": {
    "resourceLocation": "westus"
  },
  "resources": [
    {
      "type": "Microsoft.RecoveryServices/vaults/backupPolicies",
      "apiVersion": "2022-03-01",
      "name": "[format('{0}/{1}', 'recoveryServicesVault', 'vm-snapshot-policy')]",
      "location": "[variables('resourceLocation')]",
      "properties": {
        "protectedItemsCount": 0,
        "backupManagementType": "AzureIaasVM",
        "instantRPDetails": {
          "azureBackupRGNamePrefix": "vmsnapshot"
        },
        "instantRpRetentionRangeInDays": 2,
        "policyType": "V1",
        "retentionPolicy": {
          "retentionPolicyType": "LongTermRetentionPolicy",
          "dailySchedule": {
            "retentionDuration": {
              "count": 7,
              "durationType": "Days"
            },
            "retentionTimes": [
              "2022-10-21T02:00:00Z"
            ]
          },
          "monthlySchedule": {
            "retentionDuration": {
              "count": 6,
              "durationType": "Months"
            },
            "retentionScheduleDaily": {
              "daysOfTheMonth": [
                {
                  "date": 1,
                  "isLast": false
                }
              ]
            },
            "retentionScheduleFormatType": "Daily",
            "retentionTimes": [
              "2022-10-21T02:00:00Z"
            ]
          },
          "weeklySchedule": {
            "daysOfTheWeek": [
              "Sunday"
            ],
            "retentionDuration": {
              "count": 4,
              "durationType": "Weeks"
            },
            "retentionTimes": [
              "2022-10-21T02:00:00Z"
            ]
          },
          "yearlySchedule": {
            "monthsOfYear": [
              "July"
            ],
            "retentionDuration": {
              "count": 2,
              "durationType": "Years"
            },
            "retentionScheduleDaily": {
              "daysOfTheMonth": [
                {
                  "date": 1,
                  "isLast": false
                }
              ]
            },
            "retentionScheduleFormatType": "Daily",
            "retentionTimes": [
              "2022-10-21T02:00:00Z"
            ]
          }
        },
        "schedulePolicy": {
          "schedulePolicyType": "SimpleSchedulePolicy",
          "scheduleRunFrequency": "Daily",
          "scheduleRunTimes": [
            "2022-10-21T02:00:00Z"
          ],
          "scheduleWeeklyFrequency": 0
        },
        "timeZone": "UTC"
      }
    }
  ]
}

Hi @StephenWeatherford - were you able to replicate the issue?

@joehubbert Generally this means some property is not as expected for the AzureIaasVM backup management type, although it's not immediately clear what's wrong.
Does the template deploy?

Swagger: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/recoveryservicesbackup/resource-manager/Microsoft.RecoveryServices/stable/2022-03-01/bms.json
JSON: https://github.com/Azure/azure-resource-manager-schemas/blob/b0f00205f706eabfa77567aeaaf1214a2718e2af/schemas/2022-03-01/Microsoft.RecoveryServices.Backup.json

NOTE: There are several examples with AzureIaasVM in the quickstarts: https://github.com/Azure/azure-quickstart-templates/search?q=AzureIaasVM
Some of them pass validation but none of them use a long-term retention policy.

@StephenWeatherford The template does not deploy as Bicep as it fails the pre-flight validation checks.
I have edited the template so it is now a V2 policy and seems to mirror this example which does use a long-term retention policy
Updated Bicep
resource vmSnapshotBackupPolicy 'Microsoft.RecoveryServices/vaults/backupPolicies@2022-03-01' = {
  name: 'vm-snapshot-policy'
  location: resourceLocation
  parent: recoveryServicesVault
  properties: {
    protectedItemsCount: 0
    backupManagementType: 'AzureIaasVM'
    instantRPDetails: {
      azureBackupRGNamePrefix: 'vmsnapshot'
    }
    instantRpRetentionRangeInDays: 2
    policyType: 'V2'
    retentionPolicy: {
      retentionPolicyType: 'LongTermRetentionPolicy'
      dailySchedule: {
        retentionDuration: {
          count: 7
          durationType: 'Days'
        }
        retentionTimes: [
          '2022-10-21T02:00:00+00:00'
        ]
      }
      monthlySchedule: {
        retentionDuration: {
          count: 6
          durationType: 'Months'
        }
        retentionScheduleDaily: {
          daysOfTheMonth: [
            {
              date: 1
              isLast: false
            }
          ]
        }
        retentionScheduleFormatType: 'Daily'
        retentionTimes: [
          '2022-10-21T02:00:00+00:00'
        ]
      }
      weeklySchedule: {
        daysOfTheWeek: [
          'Sunday'
        ]
        retentionDuration: {
          count: 4
          durationType: 'Weeks'
        }
        retentionTimes: [
          '2022-10-21T02:00:00+00:00'
        ]
      }
      yearlySchedule: {
        monthsOfYear: [
          'July'
        ]
        retentionDuration: {
          count: 2
          durationType: 'Years'
        }
        retentionScheduleDaily: {
          daysOfTheMonth: [
            {
              date: 1
              isLast: false
            }
          ]
        }
        retentionScheduleFormatType: 'Daily'
        retentionTimes: [
          '2022-10-21T02:00:00+00:00'
        ]
      }
    }
    schedulePolicy: {
      schedulePolicyType: 'SimpleSchedulePolicyV2'
      scheduleRunFrequency: 'Daily'
      dailySchedule: {
        scheduleRunTimes: [
          '2022-10-21T02:00:00+00:00'
        ]
      }
    }
    timeZone: 'UTC'
  }
}

Updated ARM
{
      "type": "Microsoft.RecoveryServices/vaults/backupPolicies",
      "apiVersion": "2022-03-01",
      "name": "[format('{0}/{1}', parameters('recoveryServicesVaultName'), 'vm-snapshot-policy')]",
      "location": "[parameters('resourceLocation')]",
      "properties": {
        "protectedItemsCount": 0,
        "backupManagementType": "AzureIaasVM",
        "instantRPDetails": {
          "azureBackupRGNamePrefix": "vmsnapshot"
        },
        "instantRpRetentionRangeInDays": 2,
        "policyType": "V2",
        "retentionPolicy": {
          "retentionPolicyType": "LongTermRetentionPolicy",
          "dailySchedule": {
            "retentionDuration": {
              "count": 7,
              "durationType": "Days"
            },
            "retentionTimes": [
              "2022-10-21T02:00:00+00:00"
            ]
          },
          "monthlySchedule": {
            "retentionDuration": {
              "count": 6,
              "durationType": "Months"
            },
            "retentionScheduleDaily": {
              "daysOfTheMonth": [
                {
                  "date": 1,
                  "isLast": false
                }
              ]
            },
            "retentionScheduleFormatType": "Daily",
            "retentionTimes": [
              "2022-10-21T02:00:00+00:00"
            ]
          },
          "weeklySchedule": {
            "daysOfTheWeek": [
              "Sunday"
            ],
            "retentionDuration": {
              "count": 4,
              "durationType": "Weeks"
            },
            "retentionTimes": [
              "2022-10-21T02:00:00+00:00"
            ]
          },
          "yearlySchedule": {
            "monthsOfYear": [
              "July"
            ],
            "retentionDuration": {
              "count": 2,
              "durationType": "Years"
            },
            "retentionScheduleDaily": {
              "daysOfTheMonth": [
                {
                  "date": 1,
                  "isLast": false
                }
              ]
            },
            "retentionScheduleFormatType": "Daily",
            "retentionTimes": [
              "2022-10-21T02:00:00+00:00"
            ]
          }
        },
        "schedulePolicy": {
          "schedulePolicyType": "SimpleSchedulePolicyV2",
          "scheduleRunFrequency": "Daily",
          "dailySchedule": {
            "scheduleRunTimes": [
              "2022-10-21T02:00:00+00:00"
            ]
          }
        },
        "timeZone": "UTC"
      },
      "dependsOn": [
        "[resourceId('Microsoft.RecoveryServices/vaults', parameters('recoveryServicesVaultName'))]"
      ]
    }

Closing issue as I have raised on the Bicep repository issues

@joehubbert Perfect, that's what I was probably going to do, thanks.


Create React App 5 (CRA5) errors when using basis-theory-js

There is a known issue with CRA5 when using a module that has a submodule using cjs. axios a library basis-theory-js is dependent on uses cjs and in version 1.1.3 broke CRA5 when being used as a submodule. This results in the following errors:
Error: TypeError: Cannot read properties of undefined
(reading 'transformRequest')
at new BasisTheoryService (BasisTheoryService.js:34:1)
at new BasisTheoryTokens (BasisTheoryTokens.js:23:1)
at new Create (CrudBuilder.js:13:1)
at new Delete (CrudBuilder.js:46:1)
at new List (CrudBuilder.js:53:1)
at new BasisTheoryTokensElementsDelegate
(tokens.js:10:1)

at BasisTheoryReact.init (BasisTheory.js:80:1)
at async useBasisTheory.js:24:1

or
Error: The Basis Theory SDK is not supported with CRA 5, downgrade to CRA 4 or see https://github.com/facebook/create-react-app/issues/11889#issuecomment-1114928008 for a workaround.
    at new BasisTheoryService (BasisTheoryService.js:27:1)
    at new BasisTheoryTokens (BasisTheoryTokens.js:23:1)
    at new Create (CrudBuilder.js:13:1)
    at new Delete (CrudBuilder.js:46:1)
    at new List (CrudBuilder.js:53:1)
    at new BasisTheoryTokensElementsDelegate (tokens.js:10:1)
    at BasisTheoryReact.init (BasisTheory.js:80:1)
    at async useBasisTheory.js:24:1

This is a CRA issue which is no longer being maintained we do not see the framework fixing this issue - because of this we (Basis Theory) do not want to pin our axios version to 1.1.2 as this will only be a short term solution until we are forced to upgrade axios in the future.

Option 1:
CRA is no longer officially maintained meaning one option is to downgrade to react-scripts to 4.0.3 which resolves the above issue.
Option 2:
Use the workaround listed in the [CRA5 issue comments](https://github.com/facebook/create-react-app/issues/11889#issuecomment-1114928008) with craco to overwrite the webpack settings for CRA5.
Option 3:
Use an older version of the Basis Theory library  basis-theory-js or basis-theory-react :
"@basis-theory/basis-theory-js": "1.77.0"

"@basis-theory/basis-theory-react": "1.12.1"

Option 4:
Use overrides on npm or resolutions on yarn to pin the axios to 1.1.2:
NPM:
"overrides": {
	"@basis-theory/basis-theory-js": {
		"axios": "1.1.2"
	}
},

Yarn:
"resolutions": {
      "@basis-theory/basis-theory-js/axios": "1.1.2"
  },


4 areas that we may want to remove

These 4 areas return no data for most things and I'm not sure if we want to keep them as available areas from the API:
https://northernclimatereports.org/report/area/FIRE10
https://northernclimatereports.org/report/area/EL25
https://northernclimatereports.org/report/area/NC12
https://northernclimatereports.org/report/area/CD2

Ref https://github.com/ua-snap/iem-webapp/issues/560
Closing this ticket here, the API behavior is correct and probably we want to address this in the frontend.


OSBS.create_build(): increase API compatibility

Shouldn't need to provide git_branch=None or target=None.

LGTM


[BUG] Sarama SDK does not behave as expected using initialPosition=Oldest

Describe the bug
The initialPosition=Oldest option does not work as expected using the Sarama integration test. If you run the Sarama integration test repeatedly, messages are re-delivered each time.

Other SDKs, e.g. Confluent's Go SDK, and the Apache Java SDK work as expected when using "auto.offset.reset": "earliest".
I have run the Sarama integration test program against a standalone Kafka server and it behaves as expected.
I have upgraded the integration test go.mod to Sarama 1.28 to no effect.
I see the same behavior using other Sarama based programs, so this issue is not specific to the integration test.

To Reproduce
Steps to reproduce the behavior:

Configure standalone 2.7.1 Pulsar with 2.7.1 KOP.
Run the Sarama integration test producer once, sending 10 messages: KOP_PRODUCE=1 ./sarama-golang
Repeatedly run the consumer: KOP_CONSUME=1 ./sarama-golang
Every time you run the consumer it will exit after receiving 10 messages.

Expected behavior

The consumer should only exit once and subsequent runs should not receive any messages. The expectation is that for a given groupId messages will not be redelivered once they are marked as committed (sess.MarkMessage(msg, "")).

There may be something wrong with the old design of KoP offset and the process of LIST_OFFSET request. Maybe some corner cases are not covered.
Since the offset design was changed from https://github.com/streamnative/kop/pull/296 with the latest feature of Pulsar 2.8.0-SNAPSHOT,  we won't pay much attention to maintain the old branches.
You can try latest KoP master with a snapshot release of pulsar (https://github.com/streamnative/pulsar/releases/tag/v2.8.0-rc-202102252222). It works well with go-sarama client.

Thanks for the quick response. I'll give your suggestion a try.

Is there a reason you recommended that tag? It's quite a bit behind master. TIA

The KoP master branch is compatible with the next major release of Pulsar, which is 2.8.0 currently. The v2.8.0-rc-xxx is the daily release Pulsar of StreamNative. We usually upgrade KoP's Pulsar dependency to a newer daily release for the latest feature of Pulsar (like #296) or just Pulsar broker has some significant API changes so that KoP cannot run with Pulsar's latest master (like #337).
I've just tested with latest Pulsar (4d878d7), KoP also works. I recommend that tag just because it's KoP's latest dependency now and I'm not sure if Pulsar broker introduced some new API incompatibilities.


redirect_stdio is broken in Julia 1.7

As described in https://github.com/JuliaLang/IJulia.jl/issues/968#issuecomment-762459535 and https://github.com/JuliaLang/IJulia.jl/issues/968#issuecomment-776272479, the stdio redirection mechanism needs to change for Julia 1.7 to use the new mechanism from https://github.com/JuliaLang/julia/pull/39132

(Seems to work fine in Julia 1.6.)


Passing variables to routePerimeterAction in nuxt

Hi,
I'm trying to follow your examples re guarding routes in nuxt.  You add a perimeter with routePerimeterAction = 'update' in the example at https://github.com/JiriChara/vue-kindergarten#usage-with-nuxtjs
However, when this code then runs, using the example article perimiter further up the page, the "article" property of update(article) is always undefined. Is this expected? Is there a way to wire up values from the route into the perimeter action?
Thanks

Hi,
I have the same problem, have you figured out an solution?
Thanks


Two-way binding of content not working

In the example it says that content has two way binding, which doesn't seem to be correct. Changing content programmatically doesn't update the editor content. Also giving the ref an initial value doesn't seem to work. Is this a bug or rather misleading docs?
In case of the later, is there any way to populate the editor with initial data from outside?
Thanks 🙏🏽

Thanks 🙏🏽

The solution to this issue is custom plugins for Two-way binding like this
<LexicalComposer :initial-config="config" @error="onError">
    <EditorBindingPlugin v-model="content" />
    ...
</LexicalComposer>

//EditorBindingPlugin.vue
<script setup lang="ts">
import type { EditorState } from "lexical";
import {
  TRANSFORMERS,
  $convertToMarkdownString,
  $convertFromMarkdownString,
} from "@lexical/markdown";
import { useEditor, LexicalOnChangePlugin } from "lexical-vue";
import { computed, ref, watch } from "vue";

const emit = defineEmits<{
  (e: "update:modelValue", value: string): void;
}>();
const props = defineProps({
  modelValue: {
    type: String,
    default: "",
  },
});

const editorUpdate = ref(true);
const editornum = ref(0);
const content = computed<string>({
  set(v) {
    editornum.value++;
    // onChange is fired when create LexicalOnChangePlugin with an empty value
    if (editornum.value > 1) {
      editorUpdate.value = false;
      emit("update:modelValue", v);
    }
  },
  get() {
    return props.modelValue;
  },
});

const editor = useEditor();

watch(content, () => {
  if (editor && editorUpdate.value && content.value.length >= 0) {
    editor.update(() => {
      $convertFromMarkdownString?.(content.value);
    });
  }
  editorUpdate.value = true;
});

function onChange(editorState: EditorState) {
  editorState.read(() => {
    content.value = $convertToMarkdownString(TRANSFORMERS);
  });
}

defineExpose({
  editor,
});
</script>

<template>
  <LexicalOnChangePlugin @change="onChange" />
</template>

@abdulmlik this is a great snippet! I was able to use it as a component in my Nuxt 3 app with only one minor change: I removed editornum count variable with the related logic, since it's common to have an initial value in reactive connections.

Thank you @abdulmlik. Closing this for now.


Reduce initial number of positions for topNRowNumber

The operator allocates memory eagerly for that number of positions.
The current setting results in big allocations and unnecessary
memory usage for the common case.

looks good


Error about loss during training

Why is the loss value in max_width much greater than in min_width and random_width during the training process? For example, in max_width, the loss is 59, while the other two have a loss of only about 0.001

请问一下你有没有添加预训练权重 pretrain_weight.pth 在训练过程中我的精度Pre特别小，是不是因为没有加预训练权重，但是不知道是不是官方的预训练权重是可用，我还没有尝试 就想来问一下你的效果如何


Swiftmailer configuration reset after registration of SwiftmailerServiceProvider

Given:
$config = [
    'swiftmailer.options' = [ 'host' => 'smtp.example.com', ... ],
];
$app = new Application($config);

Inside my Application class:
public function __construct(array $values = array())
{
    parent::__construct($values);
    $this->register(new \Silex\Provider\SwiftmailerServiceProvider());
}

So far so good for configuring most providers. But my swiftmailer.options settings are overwritten by the SwiftmailerServiceProvider constructor, see the source here.
I'm trying to split the registration of providers and configuration of providers, so that I can create a configuration file on a per environment basis.
I would expect that an already defined configuration should not be overridden. What Silex concept am I missing here?

Change the sequence

public function __construct(array $values = array())
{
    // register before set values
    $this->register(new \Silex\Provider\SwiftmailerServiceProvider());
    parent::__construct($values);
}

Extend SwiftmailerServiceProvider

public function register(Container $app)
{
    parent::register($app);
    // configuring
}

I usually choose the second option.

I'm trying to split the registration of providers and configuration of providers, so that I can create a configuration file on a per environment basis.

configuration: https://github.com/kamilsk/TinkoffCurrencyAnalyst/blob/master/web/index.php#L9-L12
registration: https://github.com/kamilsk/TinkoffCurrencyAnalyst/blob/master/web/index.php#L18-L20

Silex app has second argument, and as you can see here https://github.com/silexphp/Silex/blob/master/src/Silex/Application.php#L86-L93, https://github.com/silexphp/Pimple/blob/master/src/Pimple/Container.php#L272-L281 it register ServiceProvider first and after that setup values/config for ServiceProvider

Change the sequence

That works for the SwiftmailerServiceProvider, but the TwigServiceProvider (for example) can only be registered after calling parent::__construct(). Isn't a really sustainable solution.

Extending SwiftmailerServiceProvider

Is an option, but requires a lot of extra code. Plus the configuration of the application is scattered over a lot of files. I'd like to have them bundled together on a per-environment basis.

Silex app has second argument

Seems like a good approach. Will try.
Thanks for thinking along :-)


Add missing animations prop to options interface.

Hi, I can see options.animations being passed to new Animations(chart, animOpts) object here:
https://github.com/chartjs/chartjs-plugin-annotation/blob/v1.0.2/src/annotation.js#L140
(and I am actually using that option)
Animation:
https://github.com/chartjs/Chart.js/blob/a8337a3/types/animation.d.ts#L29
AnyObject:
https://github.com/chartjs/Chart.js/blob/a8337a32c976393e4648de633dcc49cdec2760b3/types/basic.d.ts#L2

AnimationSpec might be better type for it.
Could also pass the TType, to make the scriptable context properly typed:
https://github.com/chartjs/chartjs-plugin-annotation/blob/2d485b799e04159b5b20b6d6519976a57394e7c5/types/index.d.ts#L6-L8


Create a page for Platform API

Description
Create a page for Platform API
What is the problem?
Platform API is mentioned in [Platform Specific Code](the https://reactnative.dev/docs/platform-specific-code) page, however, this page does not mention all of the features included in the Platform API, such as isPad to detect iPad.
There are many other functions included in the Platform API, so adding all the cases in Platform specific code page might not be the best solutions for new learners.

How can we address it?
Create a page for Platform API with list of available functions.

Why is it important?
To provide a documentation for Platform API

Who needs this?
New Learners

When should this happen (use version numbers if needed)?
Whenever

closing as duplicate of https://github.com/facebook/react-native-website/issues/1509


Add option to write time series qc movies as mp4

We should be able to call out to ffmpeg much like we do with convert to make gifs, but an mp4 will be smaller and easier to control to focus on specific artifacts.

Actually, this may be easier to do using matplotlib animation tools. Look at the code I wrote for generating stimulus movies as an example.


Latest 3.0 build breaks actions

@istnv
In the lastest 3.0 beta the module connects but none of theactions will work.  When I select initiate recording or one touch record a "recording control" greyed out text box appears where there used to be only one dropdown box.  The text is greyed out and says "unknown type text"  Screenshot attached.  This appears to happen with any action selected not just recording.

Meant to add that the instance shows connected but even feedback doesn't work either when started manually.

I'll take a look when I can. I do not have any hardware to test with so it's kind of a shot in the dark :)

Whatever I can do to help just let me know!  Thank you!Sent from my iPhoneOn May 18, 2023, at 16:37, John Knight Jr @.***> wrote:
I'll take a look when I can. I do not have any hardware to test with so it's kind of a shot in the dark :)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@istnv .  Not sure its helpful since it was obviously right before the rewrite but I went back to build 5797 and it works as expected.  There is the "Recording Control" header but not dropdown or text box there at all FWIW.

"Error; Socket not connected." is the error in the module log.

Fix should be in the next rebuild.

Awesome!  Thanks again so much for what y’all do!Sent from my iPhoneOn May 18, 2023, at 17:36, John Knight Jr @.***> wrote:
Fix should be in the next rebuild.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@istnv .  Tried out the build this morning and the actions work but none of the feedbacks are working.  Looked at the console but nothing shows up there as far as errors or anything.  Let me know if I can help diagnose.
Thanks!

@obcocav Try the next build

@obcocav Has this been tested, yet?

Yes it has!  Sorry forgot to update.  All seems well now.  Thank you!Sent from my iPhoneOn May 31, 2023, at 21:19, John Knight Jr @.***> wrote:
@obcocav Has this been tested, yet?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


[docs] rancher-nfs improvements/examples

Catalog item examples fix
More examples of options possible when using docker-compose.yml

Done in https://github.com/rancher/rancher.github.io/pull/843


[wip] Support internal connection pools for cluster URLs

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing due to concerns over using a pool for cluster URLs, particularly because they can resolve to different instances over time


Fails with jenkins behind ssl-proxy

Seems the script fails when jenkins is running behind a ssl-proxy (apache in our case). Investigate and fix?

should be fixed in the branch master-s13_14-46227

sadly we have to disable the ssl-stuff in the tests as gevent 1.0.1 doesn't like to play with python 2.7.9 regarding ssl...


Update README.md

Thanks for the action, yay :D !

Cheers for the heads-up with the typo @Veetaha!
Somehow I came across your fork, noticed the commit & made a change myself before seeing this PR 😅
I'll squash it in eitherway - Appreciate it.


Is the documentation URL object in use?

How are API providers using or not using the documentation URL object, providing references to other resources?

API providers can use the documentation URL object in the OpenAPI specification to provide references to other resources and documentation for their API. Here are a few common ways API providers utilize this object:
Linking to API reference documentation: The documentation URL object is often used to provide a direct link to the API reference documentation. This documentation typically includes details about endpoints, request/response formats, authentication methods, error handling, and other technical specifications.
Pointing to interactive API documentation: API providers may use the documentation URL to direct users to interactive API documentation tools, such as Swagger UI, Redoc, or Postman. These tools allow developers to explore and interact with the API endpoints directly in the browser, making it easier to understand the available resources and test API requests.
Referencing external developer portals or knowledge bases: Some API providers maintain dedicated developer portals or knowledge bases that provide comprehensive documentation, tutorials, guides, and other resources for developers. The documentation URL can be used to link to these external resources, providing users with access to a wealth of information beyond the OpenAPI specification.
Including support channels or community forums: API providers may use the documentation URL to point users to support channels or community forums where they can seek assistance, ask questions, and collaborate with other developers using the API. This can foster a sense of community and provide additional resources for troubleshooting and learning.
It's important to note that not all API providers use the documentation URL object, and some may opt for alternative methods to provide documentation and references to additional resources. For instance, they may have their own custom documentation URLs, rely on separate documentation websites, or use different tools specific to their API ecosystem.
In summary, the documentation URL object in the OpenAPI specification offers flexibility for API providers to provide references to various resources, including API reference documentation, interactive API documentation tools, external developer portals, support channels, and community forums. The specific usage may vary depending on the provider's preferences and the available documentation infrastructure


Declare translations directly in pages does not work

Describe the bug
Declare your translations directly in pages does not work
To Reproduce
Steps to reproduce the behavior:

Follow all basic setup
Translation on defineAstroI18nConfig works
Add i18n folder to the page's folders
Route translation works but text translation does not

@Alexandre-Fernandez Could I know the reason you close this?


Best way to get the polar radius

I have a geos projection CRS object and I want to get the polar radius, but it seems like it is not computed.
Out[5]:
<Projected CRS: +proj=geos +lon_0=-89.5 +a=6378137.0 +b=6356752.31 ...>
Name: unknown
Axis Info [cartesian]:
- E[east]: Easting (metre)
- N[north]: Northing (metre)
Area of Use:
- undefined
Coordinate Operation:
- name: unknown
- method: Geostationary Satellite (Sweep X)
Datum: unknown
- Ellipsoid: GRS 1980
- Prime Meridian: Greenwich

In [6]: crs = scn['C01'].attrs['area'].crs
In [9]: crs.ellipsoid.semi_major_metre
Out[9]: 6378137.0

In [10]: crs.ellipsoid.semi_minor_metre
Out[10]: nan

In [11]: crs.ellipsoid.is_semi_minor_computed
Out[11]: False

Does nan mean it wasn't computed? How do I compute it? If this is a property can't it be computed when I ask for it?

This is the method used to get the information:
proj_ellipsoid_get_parameters. The docstrings may help get closer to the answers.

Does nan mean it wasn't computed?

Nan is equivalent to NULL in the docs. So, not computed or in the definition.

How do I compute it? If this is a property can't it be computed when I ask for it?

Not sure why it wasn't computed. I would ask the PROJ devs about the internal logic for proj_ellipsoid_get_parameters.

Can this issue be closed?

I thought I had created an issue for this or brought it up on gitter but now I can't find it. Let's close this here and if it becomes something that is inconvenient enough I'll make an issue with PROJ.

Apologies - just had a huge :man_facepalming: moment when looking into this issue:
https://github.com/OSGeo/PROJ/issues/1842

Fix coming soon.

Workaround for older version is to use _semi_minor_metre, but I just removed it in the fix, so you will have to check for it.

Thanks! Now if only I could remember where I wanted to use this... :thinking:


I116 export collection

First look at the export function.
Comments are welcome :)
Closes #116

 Export collection to CSV

@smeetz The code has been updated according to feedback, except for the feedback entry which I gave an answer to.

Updated after feedback

Updated


Show differences between local and server folder

Given the nature of TFS, it's easy to miss adding a file (exists in your directory but not on the server) or deleting a file in your local but keep it on the server.
It would be useful to have a command that shows files that are only in your local and files that are only on the server, so you can run "tf add" or "tf delete"/"get -force".

In progress. Will be a feature in TFS directories special buffer.
Considering making the function default to the server path of the current file and to $/ when called from a file outside the buffer or with prefix arg.

Closed with commit 4e073cf.
It's not recursive, but the way the TF tool lists files and dirs doesn't make this easy to implement, and also during testing it became too slow for even 3 folders depth.


Utkarshjain/elastic private repo sync 09 2022

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify : The new API version are already merged in the private repo (azure-rest-api-specs-pr). This PR sync the repo and to generate required SDKs.

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[x] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

Could you fix the lint errors in https://github.com/Azure/azure-rest-api-specs/pull/20791/checks?check_run_id=8503659102 , these are required by new ARM guideline

@msyyc could you help with the Python SDK failure ?

@jianyexi  Regarding the linting errors, we have made the same changes in the private swagger repo which were approved and merged [PR1(https://github.com/Azure/azure-rest-api-specs-pr/pull/7741) and PR2 Can this be suppressed or can we get an exception ?

@msyyc could you help with the Python SDK failure ?

Will fix it soon

@jianyexi Can you take a look at the Breaking Change test which is failing? I have checked the error which says XmsEnumChanged . However, I am not finding any different in the swagger versions mentioned in the error.
Also, can you help us explain why the Staging Service Api Readiness and TrafficValidation test is failing but same is passing for non-staging ones

@jianyexi Can you take a look at the Breaking Change test which is failing? I have checked the error which says XmsEnumChanged . However, I am not finding any different in the swagger versions mentioned in the error. Also, can you help us explain why the Staging Service Api Readiness and TrafficValidation test is failing but same is passing for non-staging ones

@jianyexi  Do we have any update on this ?

/azp run

@jianyexi Can you take a look at the Breaking Change test which is failing? I have checked the error which says XmsEnumChanged . However, I am not finding any different in the swagger versions mentioned in the error. Also, can you help us explain why the Staging Service Api Readiness and TrafficValidation test is failing but same is passing for non-staging ones

@jianyexi Do we have any update on this ?

Sorry for the delay, it's false alarm, already fixed

@raosuhas Can you please review the responses to your comments ?

@raosuhas Responded to the review comments. As with most of the comments these concerns were not raised in the ARM review of the private swagger PR. As the changes are in production this will be a breaking change.

@raosuhas Can you please review the responses to your comments ?

@jianyexi Can we proceed with the merge? We have got the ARM sign off

@jianyexi do we have any update on this? Is anything else needed for PR approval?


Copy paste - cannot send message without modifying text

Hi team,
When copy and pasting text, it's not possible to send the message unless the text is modified.
Also not able to send a mesasge with incorrect characters.
Example with incorrect character:

Action required: Reminder- Touch ID

<<<
Is it possible to not require modification to pasted entries?

@KirstyWilson can you write a repro on this?
We recently fixed a bug that pressing CTRL + V key would fail to paste a message. Is it related?

I believe we look at onChange - which doesnt fire after a paste, until a blur. Perhaps we should also listen to onPaste ?

I checked on Edge 16 and Chrome 62.
After we paste, the send arrow turns blue immediately after paste with CTRL + V or right-click.
After paste, pressing ENTER would also send the message out.


Reconsider Diagnostic.actions (CodeAction property) for quickfix

Ref https://github.com/microsoft/language-server-protocol/issues/581
Ref https://github.com/microsoft/language-server-protocol/issues/887
Problem
Hi, I am a maintainer of Scala language related tooling, for example Zinc its incremental compiler.
We are currently in the process of implementing actionable diagnostics, which allows the Scala compiler and build tools to expose code edit suggestions via Diagnostic. Likely this will proceed using the Diagnostic.data.actions field.
As far as I can tell, most languages interested in using Diagnostic.data are interested in passing along CodeAction quick fixes. So far Scala, Erlang, Rust, XML.
Am I wrong to assume that not having this standardized means that the LSP clients need to implement the embedded CodeAction support one by one for each language?
Possible Solution
I think this is a clear signal that we should standardize list of CodeAction as Diagnostic.actions. Maybe it could be paired with WorkspaceEditClientCapabilities to opt-into or out of having actions attached.
Notes
The primary reason "quick fix" did not happen previously seems to be based on the assumption that that computing the code suggestion would be expensive. Some code suggestions might be, but:

There are many other cheap code fixes, often already suggested via deprecation warning messages. See for example https://github.com/scala/scala/pull/10406.
There are also some class of warnings that only compiler can accurately emit because it has the precise knowledge that is often expensive for external linters to re-emulate. In Scala community, there are syntactic Scalafix and semantic Scalafix, and the latter can be very slow. An example of this is unused import statement. Another example is organizing import statements, because import in one line can affect the imports in latter lines.

+1, clangd already supports this (as an extension).

Isn't this already computable by the client, since CodeActions have a diagnostics field that says which Diagnostics it resolves? So a client could ask for diagnostics and code actions, and then match them up by looking through the code actions. Less efficient, perhaps, but still possible.


add working hours to portal

As a citizen of Ukraine I want see woking hours of divisions on the map and on the list of division details.

add working hours to the map
add working hours to division details

change "Робочий час" to "Графік роботи"


Windows client

We have an existing issue for this already #43. We're hoping that we'll be able to do this with the 1.2.0 release that we have planned but have run into a few issues that we haven't solved with the Qt6 port. We're hoping to have this done sometime in Q2 along with an installer. It's not clear yet how much code will need to change to support Windows but we do need to identify a screenshot solution for it as the built in tools do not provide the necessary functionality.
Going to close this as a duplicate.


chore: Release version 2.28.9

Current dependencies on/for this PR:

main

PR #40

PR #41   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: Je0Vzki5i1gX182k5g3W


SSRProvider it`s not necessary in React18

Description
In React 18, SSRProvider is not necessary and is a noop. You can remove it from your app.
CodeSandbox/Snack link
https://snack.expo.dev/CB2NknOQA
Steps to reproduce
Just start any application
NativeBase Version
3.4.28
Platform

[ ] Android
[ ] CRA
[ ] Expo
[ ] iOS
[ ] Next

Other Platform
No response
Additional Information
Remove SSRProvider in node_modules/native-base/src/core/NativeBaseProvider.tsx file.

Native base is not longer maintained. And it has bad performance. DROP IT. https://github.com/GeekyAnts/NativeBase/issues/4302#issuecomment-1650492491


Fix release build

The dry-run of 3.5.24 is failing due to license pb.
This PR fixes this

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.


Add perform method param validation

Closes #70

So interesting the build actually failed with this validation: https://codeship.com/projects/82628/builds/15488682. I'll have to look deeper to see if this validation actually caught a bug that the tests weren't verifying.

oh no, it's not that.
1) Error:
when a worker is defined and there is a custom Sneakers app_name#test_0007_must send message values to perform_async:
NameError:  method `perform' for class `PumpItUp'
/home/rof/src/github.com/giftcardzen/la_gear/lib/la_gear/worker.rb:68:in `method'
/home/rof/src/github.com/giftcardzen/la_gear/lib/la_gear/worker.rb:68:in `work'
/home/rof/src/github.com/giftcardzen/la_gear/test/test_worker.rb:80:in `block (3 levels) in <class:TestWorker>'

The PumpItUp test worker doesn't have the perform method defined. this method inadvertently validates the existence of a perform method as well.


Enable template caching in Oracle DB template loader

The freemarker.cache.TemplateLoader interface has 3 separate methods for template loading:

findTemplateSource for getting a template "locator" - a small object that references to a template, but does not contain its body.
getLastModified for checking the found template's freshness - a timestamp value that can be compared with the previously saved in order to make a decision whether to reload the template or not.
getReader for getting the template's stream - this method is called only if the template is outdated.

The current implementation does it wrong. It fetches a template containing CLOB each time the template being loaded. Thus caching makes no sense.
Need to refactor this behavior.
TODO list:

Divide the PL/SQL loading procedure into 3 separate subprocedures:

resolver - gets a template name and returns the full description of its container object.

checker - gets the resolved name and returns the container object's timestamp from the data dictionary.

loader - gets the resolved name and returns the template's body as a CLOB

Create a class for storing a template locator (fully described container object) - this class must override equals and hash methods.

Refactor the DB template loader class, implement each of its methods in a right way. The class's constructor must take 3 separate DB calls - 1 for each of the implemented methods.

Set a proper cache storage for the configuration, e.g. MruCacheStorage, instead of current NullCacheStorage.

As a separate issue - make template caching easily configurable.

Done.


imgur.com

website is block

Fixed in https://github.com/mitchellkrogza/Ultimate.Hosts.Blacklist/commit/fcaa791f48194829bd090e8a15d87bb4de6b0e1d


Does not work with Node 10.0.0

Seems does not work with node v10.0.0.
This error is thrown:
src\node_contextify.cc:631: Assertion `args[1]->IsString()' failed.
1: node::DecodeWrite
2: node::DecodeWrite
3: uv_loop_fork
4: v8::internal::interpreter::BytecodeDecoder::Decode
5: v8::internal::RegExpImpl::Exec
6: v8::internal::RegExpImpl::Exec
7: v8::internal::RegExpImpl::Exec
8: 000000FD52684281
Steps to reproduce:

Open node REPL
Type require("fstream")
The above error is trhown

I'm closing this. More info can be found here:
https://github.com/EvanOxfeld/node-unzip/issues/116


MBL-1573: Testing + small UI details

📲 What

Added a bunch of testing
Small UI additions

🤔 Why

Wanna be sure we do not have bugs
Confidence changing/modifying/adding code in the future

👀 See

|  |  |
📋 QA

on Crowdfund for the deadline disclaimer go to any project and navigate to the checkout screen

Story 📖
MBL-1573

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 25.00000% with 27 lines in your changes missing coverage. Please review.

Project coverage is 67.75%. Comparing base (0329373) to head (cb429e4).

Files
Patch %
Lines

...ui/views/compose/checkout/PledgeItemizedDetails.kt
0.00%
13 Missing :warning:

...i/activities/compose/projectpage/CheckoutScreen.kt
0.00%
9 Missing :warning:

...ewmodels/projectpage/CrowdfundCheckoutViewModel.kt
66.66%
0 Missing and 4 partials :warning:

...iewmodels/usecases/SendThirdPartyEventUseCaseV2.kt
50.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                         Coverage Diff                         @@
##             feature/pledge-redemption-ml1    #2107      +/-   ##
===================================================================
+ Coverage                            66.71%   67.75%   +1.03%
- Complexity                            2136     2183      +47
===================================================================
  Files                                  366      366
  Lines                                22462    22491      +29
  Branches                              3216     3224       +8
===================================================================
+ Hits                                 14985    15238     +253
+ Misses                                5791     5515     -276
- Partials                              1686     1738      +52

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


Feature Request: Additional Metadata for VTT files

Hi Jerome,
We are starting to work with WebVTT files. Would it be possible to get information similar to SCC / SMPTE-TT for WebVTT formats from MediaInfo.

Duration
StartTime
endTime
EventCount
LineCount
Max count of lines per event

We want to use MediaConch to validate the webvtt files coming out of our transcoders. Having this information would let us compare the webvtt to the media but also compare it to the source to ensure we didn't lose any events during the conversions.  Will see how practical it is to use the event / line count between formats.

@JeromeMartinez Do you have an idea on when this feature could be worked on?  Just trying to see if we'll have this available for an upcoming project.

We plan to work on it for end of November.

@JeromeMartinez As part of this effort, one thing we'd like to solve for is the minimum duration of on screen text. Is it possible to log this as part of MediaInfo/MediaConch. We get bitten by this quite often where assets get processed and during processing the conversion causes the in/out to be shorter and cc / subtitles to flash on screen..

one thing we'd like to solve for is the minimum duration of on screen text

Not complicated, we'll do.

We get bitten by this quite often where assets get processed and during processing the conversion causes the in/out to be shorter and cc / subtitles to flash on screen.

Note that we have a CEA-608/708 (from MPEG Video or SCC, actually any content supported by MediaInfo) to SRT (so not far from VTT) proof of concept, which was not intensively tested (it was mainly used for checking forbidden words) but could be of some interest for you if you are not happy with the tools you are currently using for transcoding.

@JeromeMartinez Is this work still targeted to end of November?

Yes.

@JeromeMartinez Checking back in on this one.

I hope to provide a build by tomorrow.

Implemented (except overlapping times and timestamp tags inside content, will be in a future PR)
With e.g.
WEBVTT - This file has cues.

14
00:01:14.815 --> 00:01:18.114
- What?
- Where are we now?

15
00:01:18.171 --> 00:01:20.991
- This is big bat country.

16
00:01:21.058 --> 00:01:23.868
- [ Bats Screeching ]
- They won't get in your hair. They're after the bugs.

Result:
Text
Format                                   : WebVTT
Duration                                 : 9 s 53 ms
Start time                               : 1 min 14 s
End time                                 : 1 min 23 s
Compression mode                         : Lossless
Count of events                          : 3
Minimum duration per event               : 2 s 810 ms
Count of lines                           : 5
Maximum count of lines per event         : 2

In latest snapshots.

@JeromeMartinez Thank you very much for the VTT work.  Do you know what PR this work will be included?

Implemented (except overlapping times and timestamp tags inside content, will be in a future PR)

@joe-sciame-wm overlapping times were done at https://github.com/MediaArea/MediaInfoLib/pull/1628 few days after this PR. timestamp tags not yet implemented, do you have some real world examples? IIRC I have 1 example and could implement anyway with it, I just not prioritized, I didn't think it is a really used.


Replace HttpResponse(json.dumps(...)) with JsonResponse

Pull request: https://github.com/yourlabs/django-autocomplete-light/pull/976
When you forget to set the mime-type, Nginx+uwsgi serves json wrapped inside HTML tags (html, head, body), breaking production.

I guess it is fixed and merged now.


tagging limit

Error: 409-TenantCapacityExceeded, Maximum number of tagDefault [20] reached in compartment
Suggestion: The resource is in a conflicted state. Please retry again or contact support for help with service: Identity Tag Default
Requested Information
Documentation: https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/identity_tag_default
API Reference: https://docs.oracle.com/iaas/api/#/en/identity/20160918/TagDefault/CreateTagDefault
Request Target: POST https://identity.us-ashburn-1.oci.oraclecloud.com/20160918/tagDefaults
Provider version: 5.1.0, released on 2023-06-13. This provider is 3 Update(s) behind to current.
Service: Identity Tag Default
Operation Name: CreateTagDefault

Hey @mike-leuer, we've resolved this issue in the version that we'll release. I'll let you know the release date shortly. Many thanks for all your positive feedback and suggestions.

I am trying out the latest now.

Hi @mike-leuer ,
We had fixed this issue , please use the latest version v2.1.1.


Separate out fields by type

Roughly, group:

Short answer (likely fill in the blank or "slot in" but not always)
Long answer (likely to be "created" answers where user hadn't thought of the answer before seeing form, or has to draft it)
Choices (checkboxes) - hard to say if likely to be "created" or "slot in" answers

We should display statistics about the number of "difficult to answer" questions using this very rough heuristic from the style of input.

Note: once the field allows for a "long" answer, it probably doesn't matter how much room the field gives for how complex the answer is. A lot of court forms only allow a small amount of space for something that will take the litigant a lot of room to explain, just because they want to fit the form on one page. Except: a full page text field is probably expecting something like an affidavit.


Icon names are difficult to discover

The names used by Steeze UI are not consistent with the names of the underlying icon packs.
This is not an issue, but it seems hard to find what the actual names are.
For example, in SimpleIcons, there is an npm logo named npm and in Steeze UI this is called Npm (note capitalization).
Is there a good way to discover these names? Or perhaps a pattern that Steeze UI uses that can be documented?

When creating steeze-ui/icons I took some inspiration from  heroicons react to use Upper Camel Case (Pascal Case) for all icons with the rule: if the original icon name starts with a number a underscore is prefixed.
I know searching for a specific icon can be annoying, thats why I had a icon searcher planned
here but no time to realize it yet..

@JustinVoitel very much appreciate  your work! Has been very useful
Is there code somewhere that implements this pattern?

Aha I think I found it in here: https://github.com/steeze-ui/icons/blob/main/packages/core/src/index.ts


Advertise wl_output’s refresh rate on the Wayland backend

Currently when running either sway or rootston, weston-info reports a refresh rate of 0 Hz on the wl_output, which is obviously wrong. It would make sense to report the same one as the parent compositor’s wl_output, or 60 Hz as a fallback (e.g. if this one too reports 0 Hz).
It could make sense to re-emit the mode when the compositor’s surface is moved to an output with a different refresh rate, in case the outer compositor has more than one output exposed.

I'm against this for several reasons:

The Wayland protocol has been updated to allow zero refresh sizes for outputs that are e.g. virtual. I think we should do the same for refresh rate. I'll try to send a patch.
FreeSync outputs won't have a fixed refresh rate anyway, so it makes sense to do this change outside the scope of virtual outputs.
A Wayland window can be on multiple outputs. We don't know which one the compositor has decided to "pin" the surface on.
Clients shouldn't use this value anyway
The other parts of the mode must be faked anyway, since the size in pixel of the surface is different  from the parent compositor's output size

For reference: https://lists.freedesktop.org/archives/wayland-devel/2018-September/039496.html
I quite agree with that direction, but imo this should become a hint instead of being made optional, for legacy clients.  Both the FreeSync thing and the multiple output things would benefit from that, setting the maximum value possible sounds acceptable to me.

I think zero is fine. In fact, I'd consider setting it to zero for all outputs, because I don't want to encourage users to render frames at any rate other than when frame events come in.

Yeah, instead of adding workarounds in the compositor, buggy clients should be fixed.


Change health_check to accept an array

The array is a 3-tuple of the average load over the past 1, 5, and 15 minutes

Is this still an issue that needs to be addressed? If so, do you want me to take it on?

Maybe keep this on hold again for now, it's probably not the most important thing. it's a nice-to-have but we still have to finish core stuff first

Well just send a single value


Implement Stop function

Stop function not implemented for plugins such as screenshot since 2.0 update.

Plugins are given a CancellationToken that can be used in conjunction with the jobkill command to be requested to exit