Support for Gradle Kotlin DSL

Expected behaviour
Shot is the last plugin on our project that is not providing a way to use a new plugins block and alias. It would be nice if it was fixed
Actual behaviour
Right now the only way I found is via classpath. Surely there is a lengthy workaround to overcome this by copying everything manually but I feel like Kotlin DSL is the staple for Gradle and Shot snould provide this out of the box
Steps to reproduce
Try to add Shot
Version of the library
Any

FYI, this is the kind of API I expect to add this plugin to my codebase : https://docs.gradle.org/current/userguide/plugins.html#sec:plugins_block

Here is the way I was able to add it in my project ( Haven't tested it if this works, but I was able to do gradle sync successfully )
CREDIT GOES TO : https://www.caravellecode.fr/android-gradle-plugin-resolution/
settings.gradle.kts
pluginManagement {
    repositories {
        google {
            content {
                includeGroupByRegex("com\\.android.*")
                includeGroupByRegex("com\\.google.*")
                includeGroupByRegex("androidx.*")
            }
        }
        mavenCentral()
        gradlePluginPortal()
    }
    ##### ADDED THIS SECTION #####
    resolutionStrategy{
        eachPlugin{
            when(requested.id.id){
                "shot" -> {
                    useModule("com.karumi:shot:${requested.version}")
                }
            }
        }
    }
}

libs.versions.toml
[versions]
shot="6.0.0"

## make sure id is "shot", I set it "com.karumi.shot" first time, but id from resolved repo ( i.e "shot" ) did not match with the "com.karumi.shot"
[plugins]
shot = { id="shot", version.ref="shot" }

Project level build.gradle.kts
plugins {
    ## not sure why I have done this, but assumed from other integrations ( my 1st week in native android development )
    alias(libs.plugins.shot) apply false
}

app/module level build.gradle.kts
plugins {
    alias(libs.plugins.shot)
}


ClientConfig not saving

Any ideas on this? Cannot save anything. A small "Saving.." box shows up and stays there indefinitely. This is what shows up in chrome dev tools:

Thanks in advance.

That 403 sounds like a file permission or mod security issue... The js error is probably a result of the failed Ajax request

Thanks, it was indeed mod_security...odd that I was able to make changes a few days ago though.


Telemetry Middleware

Initial port of Telemetry Middleware.

Could you format the code to be black format compliant?
pip install black
On the root of the project:
black libraries

Nice tool!


lambda dlq support

Per call, the idea here is to expose this config in the LambdaFunction class(es) in mu.py.

I'm reading this as extend LambdaManager._create_or_update.

that sounds right, also while your there... we also want lambda tag support.

effectively execution-mode is going to grow some more lambda config pass through


Feature/pixelate op

A "pixel perfect" img fx operator. Its TileAmount parameter allows to set the exact amount of tiles per axis

Nice! Although I personally would have called the parameter "Divisions" :-)


⚠ Bofrost.lu has degraded performance

In d6a4807, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1137 ms

Resolved: Bofrost.lu performance has improved in c4e34ef after 6 minutes.


[RFC] Support multiple isolation mechanisms using Kata

Description of problem
Kata recently added support for the Firecracker VMM. What that means is that we can now support different hypervisors in Kata.
However it is more than just that. Firecracker is a VMM, but in reality it is a different enough that it can be considered a different isolation mechanism even though the underlying hardware framework is VT-x.
For example the resource profile/model of a Firecracker POD will be significantly different. This is in addition to any security differences (i.e. sandbox capabilities) and limitations.
We have multiple ways of exposing these isolation mechanisms to the end user of kubernetes

Single runtime with different annotations within the POD (which becomes more kata specific).
This may not be ideal just like non standard annotations that came before.

Expose it directly as different runtime classes.
This means that there may have to be two different kata binaries with different configurations, or the same kata binary that chooses a different configuration based on argv[0] (like busybox).
So kata-runtime-firecracker would chooser configuration-firecracker.toml as kata already has support for

GLOBAL OPTIONS:
   --kata-config value               Kata Containers config file path

Note: Both 1 and 2 are very kata specific.

Expose at the runtime class level by allowing the runtime class spec to have more properties. This would allow us to use the same runtime binary but expose it at the top level as multiple runtime classes each with different properties and limitations.

type RuntimeClassSpec struct {
    // RuntimeHandler specifies the underlying runtime the CRI calls to handle pod and/or container
    // creation. The possible values are specific to a given configuration & CRI implementation.
    // The empty string is equivalent to the default behavior.
    // +optional
    RuntimeHandler    string

    // Properties to be passed to the runtime on each invocation
    RuntimeProperties map[string]string
}

Note: 3 is requires changes to both kubernetes and crio/containerd.

Enhance crio and containerd to support specifying configuration along with the runtime binary. This is the equivalent of 2 but a cleaner and natural abstraction for other runtimes to reuse. This also allows us to use runtime class the way it is defined and implemented in kubernetes today.

Hence the crio.conf would look like
[crio.runtime.runtimes.kata]
runtime_path = "/usr/bin/kata-runtime"

[crio.runtime.runtimes.katafirecracker]
runtime_path = "/usr/bin/kata-runtime"
runtime_params = "--kata-config /etc/kata-runtime/firecracker.toml"

And the RuntimeHandler would need to be enhanced along the lines of
https://github.com/kubernetes-sigs/cri-o/blob/master/oci/oci.go#L125
   // config table.
   type RuntimeHandler struct {
   RuntimePath string `toml:"runtime_path"`
   RuntimeParams string `toml:"runtime_params"`
   }

Note: 4, would not need any changes to kubernetes. It is also generic and not kata specific.

/cc @egernst @sameo @sboeuf @gnawux

/cc @tallclair

@sboeuf @egernst when we move to containerd-shim-v2 based implementations where CRIO talks to kata via gRPC we need this toml information to come through.

In the case of containerd-shim-v2 today the options has a binary name but no configuration.
It may be worthwhile to explore supporting passed of additional arguments for each plugin.
https://github.com/containerd/containerd/blob/9e372ff01d7a81c325ff1c15ed84efa95e7fb5a6/runtime/v2/runc/options/oci.pb.go#L39
type Options struct {
	// disable pivot root when creating a container
	NoPivotRoot bool `protobuf:"varint,1,opt,name=no_pivot_root,json=noPivotRoot,proto3" json:"no_pivot_root,omitempty"`
	// create a new keyring for the container
	NoNewKeyring bool `protobuf:"varint,2,opt,name=no_new_keyring,json=noNewKeyring,proto3" json:"no_new_keyring,omitempty"`
	// place the shim in a cgroup
	ShimCgroup string `protobuf:"bytes,3,opt,name=shim_cgroup,json=shimCgroup,proto3" json:"shim_cgroup,omitempty"`
	// set the I/O's pipes uid
	IoUid uint32 `protobuf:"varint,4,opt,name=io_uid,json=ioUid,proto3" json:"io_uid,omitempty"`
	// set the I/O's pipes gid
	IoGid uint32 `protobuf:"varint,5,opt,name=io_gid,json=ioGid,proto3" json:"io_gid,omitempty"`
	// binary name of the runc binary
	BinaryName string `protobuf:"bytes,6,opt,name=binary_name,json=binaryName,proto3" json:"binary_name,omitempty"`
	// runc root directory
	Root string `protobuf:"bytes,7,opt,name=root,proto3" json:"root,omitempty"`
	// criu binary path
	CriuPath string `protobuf:"bytes,8,opt,name=criu_path,json=criuPath,proto3" json:"criu_path,omitempty"`
	// enable systemd cgroups
	SystemdCgroup bool `protobuf:"varint,9,opt,name=systemd_cgroup,json=systemdCgroup,proto3" json:"systemd_cgroup,omitempty"`
}

@mcastelino The Options type is completely opaque to containerd and mostly opaque to the cri plugin.
It is shim implementation specific.
Kata runtime can define their own options, e.g. KataOption, and add whatever useful there. And the cri plugin can configure those options based on the RuntimeClass.  To add a new options type support, just add several lines here https://github.com/containerd/cri/blob/master/pkg/server/helpers.go#L479

@mcastelino
I don't see the point in adding RuntimeProperties map[string]string to the RuntimeClass since k8s only cares about asking for a specific type of runtime. The list of parameters that needs to be passed down to the runtime is directly defined through CRI-containerd or CRI-O config.
As mentioned by @Random-Liu, there is no need for containerd to know about the options format, but instead we can simply assume the shim implementation will know what to do with it. That being said, we need to define a common behavior about what to do from CRI-containerd and CRI-O with those extra parameters, and how to handle them.

I don't see the point in adding RuntimeProperties map[string]string to the RuntimeClass since k8s only cares about asking for a specific type of runtime. The list of parameters that needs to be passed down to the runtime is directly defined through CRI-containerd or CRI-O config.

@sboeuf as indicated in original issue the preferred choice is option 4. Not option 3. So I think you are agreeing?
And yes, we need to unify the behavior across CRI-containerd and CRI-O for option 4.

To add a new options type support, just add several lines here https://github.com/containerd/cri/blob/master/pkg/server/helpers.go#L479 As for containerd, it will just blindly pass the options from the cri plugin to the shim process.

@Random-Liu that makes sense. The only small issue I see here is that this helper needs to be modified each time a new runtime comes along that may want its own custom options.

I would argue that for flexibility some k8s users will want to run pods with different isolation mechanisms (qemu, firecracker, etc). On the other hand, I think just having the config option either in containerd or CRIO is simpler.
What's the argument behind not letting this be handled in a common /etc/kata-runtime/configuration.toml file as an always default isolation mechanism? (Support a single isolation mechanism at a time)

What's the argument behind not letting this be handled in a common /etc/kata-runtime/configuration.toml file as an always default isolation mechanism? (Support a single isolation mechanism at a time)

@raravena80  not sure I understand what you mean by "Support a single isolation mechanism at a time". Are you suggesting a given node should only use one type of isolation mechanism? That is possible even with this proposal. But then you need to start tagging nodes at the kubernetes layer.
All this proposal is trying to do it allow the flexibility for kata to expose multiple isolation mechanisms on the same node without duplicating binaries. This paired with an admission control policy would allow us to choose the most restrictive isolation mechanism that kata can provide while satisfying the needs of a given POD.
Addressing your other concern, the reason for multiple configuration files, one per isolation mechanism is to address the fact that each isolation mechanism has different limitations and profiles. And the toml file today contains tuneable sections for other kata components besides the hypervisor. For example you would typically run kata with macvtap networking and firecracker with tcfilter for optimal performance.

And yes, we need to unify the behavior across CRI-containerd and CRI-O for option 4.

Tracking the current CRI-O proposal. We need to unify what we do across both.
https://github.com/kubernetes-sigs/cri-o/issues/1991

What's the argument behind not letting this be handled in a common /etc/kata-runtime/configuration.toml file as an always default isolation mechanism? (Support a single isolation mechanism at a time)

Chatted with @mcastelino. The answer here is that given the example of firecracker and qemu, we can allow flexibility for the user or cluster operator to run on nodes with firecracker and qemu support, and also run on nodes with only firecracker or only qemu support.  Possibly, having RuntimeClass: kata-firecracker and RuntimeClass: kata-qemu.

To add a new options type support, just add several lines here https://github.com/containerd/cri/blob/master/pkg/server/helpers.go#L479 As for containerd, it will just blindly pass the options from the cri plugin to the shim process.

@Random-Liu It seems runhcs is broken when trying to decode its own options?
https://github.com/containerd/containerd/blob/master/runtime/v2/runhcs/service.go#L421

@mcastelino

@sboeuf as indicated in original issue the preferred choice is option 4. Not option 3. So I think you are agreeing?

Yes sorry I missed the fact that 3 and 4 were two different approaches... And yes I agree 4 is the best approach here.


Exyte Chat Crashing When Delete Row or Rows

Hi developer, Thank you for creating an excellent chat library In SwiftUI.
when I tried to delete the item at binding dataSource source, will crash immediately at performBatchUpdates in UIList, below is the log:
Thread 1: "Invalid batch updates detected: the number of sections and/or rows returned by the data source before and/or after performing the batch updates are inconsistent with the updates.\nData source before updates = { 1 section with row counts: [3] }\nData source after updates = { 1 section with row counts: [2] }\nUpdates = [\n\n]\nTable view: <UITableView: 0x125850a00;
I check the code, and the applyInserts  function already exists to deal with delete cases.

Finally,I solved the issue with modify theapplyEdits function in UIList.swift
`for iPrevDate in 0..<prevDates.count {
let prevDate = prevDates[iPrevDate]
guard let section = sections.first(where: { $0.date == prevDate } ),
let prevSection = prevSections.first(where: { $0.date == prevDate } ) else {
tableView.deleteSections([iPrevDate], with: .automatic)
continue
}
        var resultRows = [MessageRow]()
        for iPrevRow in 0..<prevSection.rows.count {
            let prevRow = prevSection.rows[iPrevRow]
            guard let row = section.rows.first(where: { $0.message.id == prevRow.message.id } ) else {
                tableView.deleteRows(at: [IndexPath(row: iPrevRow, section: iPrevDate)], with: .automatic)
                continue
            }
            resultRows.append(row)

            if row != prevRow {
                DispatchQueue.main.async {
                    tableView.reloadRows(at: [IndexPath(row: iPrevRow, section: iPrevDate)], with: .none)
                }
            }
        }
        result.append(MessagesSection(date: prevDate, rows: resultRows))
    }`


Interface for creating and editing dashboards

This is currently only possible in the Django Admin. This will also implement edit permissions, taking over from #27.

I'm going to put this interface at at /dashboard/<slug>/edit/
Creating dashboards will start with a button on /dashboard/ - which will redirect the user to the new /dashboard/<slug>/edit/ page for further changes.


Initial commit for Horae

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] This pull request is on a feature branch.

I have read the CLA Document and I hereby sign the CLA

recheckcla


Error Message(like: Vac banned)/Colored Message in Lobby

Would be nice if you could add this
https://www.unknowncheats.me/forum/counterstrike-global-offensive/224252-color-message-lobby.html

Not an issue.


Add IDENTIFIED in mysql keywords

Fix #691

I see IDENTIFIED is in mysql keywords page. But not in mariadb keywords page.
Should I remove the change for mariadb?

MySQL docs list all keywords, but MariaDB docs only the reserved ones (marked by "(R)" in MySQL docs).

As IDENTIFIED is not a reserved word, it can be freely used as an identifier. Simply adding it to the keywords array will result in a query like SELECT identified FROM foo to be formatted as SELECT IDENTIFIED FROM foo.
A better approach would be to add "IDENTIFIED BY" to the reservedPhrases array in *.formatter.ts files.
This also applies to SingleStoreDB dialect (which is also a close relative of MySQL).

Thanks. Merged.


Implementation of Themes

Closes #9
Changes Made:

Created themeprovider.dart

Created Function for System, Dark and Light Modes.

Created a dropdown box in about_view.dart and implemented themes through that box.

Reference Images:

System Mode

Light Mode

Dark Mode

Also can you update the theme switch drop down button style? You could add some padding and maybe make it look more modern.

Where specifically you want me to place the button then? I'll make the button bit more modern thats fine

Where specifically you want me to place the button then? I'll make the button bit more modern thats fine

I think placement is fine. Or at best if we could somehow place the a single icon button in the app bar, that would be the best.
The icon would show what theme the app is currently running in. For example a moon when it is in dark mode.
Once tapped, it can perhaps change to sun icon and the theme changes accordingly.
When tapped once more, the theme changes to system. And this cycle repeats.

Sure, I'll try. Also any specific icon you would like for 'System theme'?

Where specifically you want me to place the button then? I'll make the button bit more modern thats fine

I think placement is fine. Or at best if we could somehow place the a single icon button in the app bar, that would be the best.
The icon would show what theme the app is currently running in. For example a moon when it is in dark mode.
Once tapped, it can perhaps change to sun icon and the theme changes accordingly.
When tapped once more, the theme changes to system. And this cycle repeats.

Sure, I'll try. Also any specific icon you would like for 'System theme'?

Nothing as such. Choose what suits best.

Check I have made changes

Nice work🙌


feat(make): introduces help target

This PR cleans up Makefile and provides simple help target which prints all the targets which are annotated with ## comment as well as all variables which can be set for the build.
That's how it looks in the console:

Not sure if you need a jira ticket for this...

Dunno, I just do little boyscouting as I work with it. :)

closing as the ior component has been moved into maistra/istio


Update data.py

The distance at which each corner is located previously returned NaN and now distance is calculated correctly.
Corner Distance helps to display corner number in speed vs distance and other telemetry plots

Hi, that's actually intentional. That function isn't intended as public API. You should use Session.get_circuit_info() instead. The corner distance is then calculated from a reference lap.
Why is it done that way? I was advised by the Multiviewer dev to not use their distance info for FastF1 because it might not be accurate because they're calculating it a bit differently. Therefore, FastF1 recalculates the distance data. But that requires a reference lap with reference telemetry.
The function in data.py should probably be prefixed with an underscore to make it clearly private. But there was something about the linter complaining about private imports so I just excluded it from the documentation. Not the cleanest way.


error when starting

already installed electron with npm install electron and with npm install electron --save-dev
here is error:
C:\Users\Liam\Downloads\BotClient-0.2.0-alpha\index.js:14
app.on('ready', () => {
^
TypeError: Cannot read property 'on' of undefined
at Object. (C:\Users\Liam\Downloads\BotClient-0.2.0-alpha\index.js:14:5)
←[90m    at Module._compile (node:internal/modules/cjs/loader:1101:14)←[39m
←[90m    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)←[39m
←[90m    at Module.load (node:internal/modules/cjs/loader:981:32)←[39m
←[90m    at Function.Module._load (node:internal/modules/cjs/loader:822:12)←[39m
←[90m    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12)←[39m
←[90m    at node:internal/main/run_main_module:17:47←[39m

@DarkGuy10 how to fix this?

holy shit, why didnt i get notifs for these issues
anyways, like i said for the other issue, you need electron-nightly instead of electron (since electron isnt compatible with node 16.6+)


Mcgov/dpdk send receive refactor

Cleaning up some logic which has been tripping send/receive testing up.

refactor send/receive

ensure receiver starts first
simplify multiple queue test logic to allow use of shared logic
simplify queue and core selection to fix issues across variable threads/core/numa configurations.
rename variables to make logical core vs physical core model in DPDK EAL explicit.

Open for review, merge issues resolved and testing is in progress.

Something weird is happening w github, going to open a new PR to refresh the contents.


Incorrect reportMissingImports errors for namespace packages

Type: Bug

Behaviour
Expected vs. Actual
Statement such as:
from rad.util.frame import Frame
are flagged with "could not be resolved" errors, from PyLance, "reportMissingImports".
Steps to reproduce:

Implement an environment (I'm using venv) with a PEP420-compliant namespace package installed.
Set up that environment as the selected interpreter in VSCode
Edit a python script in VSCode with an import from that package.

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.10.12
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

2023-09-12 10:30:10.286 [info] Experiment 'pythonPromptNewFormatterExt' is active
2023-09-12 10:30:10.286 [info] Experiment 'pythonPromptNewToolsExt' is active
2023-09-12 10:30:10.286 [info] Experiment 'pythonTestAdapter' is active
2023-09-12 10:30:10.286 [info] Test server listening.
2023-09-12 10:30:10.286 [info] Python interpreter path: ~/src/venv/rti/bin/python
2023-09-12 10:30:13.080 [info] Starting Pylance language server.
2023-09-12 10:32:16.975 [info] Discover tests for workspace name: connext-rpi - uri<EMAIL_ADDRESS>2023-09-12 10:32:16.975 [info] Running discovery for pytest using the new test adapter.
2023-09-12 10:32:17.003 [info] > . ~/src/venv/rti/bin/activate && echo 'e8b39361-0157-4923-80e1-22d70d46dee6' && python ~/.vscode/extensions/ms-python.python-2023.14.0/pythonFiles/printEnvVariables.py
2023-09-12 10:32:17.004 [info] shell: bash
2023-09-12 10:32:17.052 [info] > ~/src/venv/rti/bin/python -m pytest -p vscode_pytest --collect-only --ignore=./rad_rpi/validation rad_rpi rad_util
2023-09-12 10:32:17.052 [info] cwd: .
2023-09-12 10:32:18.535 [info] Test server connected to a client.
2023-09-12 10:32:18.579 [info] Using result resolver for discovery
2023-09-12 10:32:18.952 [error] Error while trying to run pytest discovery,
Error: 2023-09-12 10:32:17.332 | DEBUG    | rad.rpi:<module>:27 - importing rad.rpi

2023-09-12 10:32:18.958 [error] pytest test discovery error
 [n [Error]: 2023-09-12 10:32:17.332 | DEBUG    | rad.rpi:<module>:27 - importing rad.rpi

	at ChildProcess.<anonymous> (/home/mcowpert@na.jnj.com/.vscode/extensions/ms-python.python-2023.14.0/out/client/extension.js:2:241907)
	at Object.onceWrapper (node:events:628:26)
	at ChildProcess.emit (node:events:513:28)
	at maybeClose (node:internal/child_process:1091:16)
	at Socket.<anonymous> (node:internal/child_process:449:11)
	at Socket.emit (node:events:513:28)
	at Pipe.<anonymous> (node:net:322:12)]
2023-09-12 10:32:32.930 [info] Discover tests for workspace name: connext-rpi - uri<EMAIL_ADDRESS>2023-09-12 10:32:32.930 [info] Running discovery for pytest using the new test adapter.
2023-09-12 10:32:32.952 [info] > ~/src/venv/rti/bin/python -m pytest -p vscode_pytest --collect-only --ignore=./rad_rpi/validation rad_rpi rad_util
2023-09-12 10:32:32.952 [info] cwd: .
2023-09-12 10:32:34.181 [info] Test server connected to a client.
2023-09-12 10:32:34.218 [info] Using result resolver for discovery
2023-09-12 10:36:58.385 [info] Discover tests for workspace name: connext-rpi - uri<EMAIL_ADDRESS>2023-09-12 10:36:58.385 [info] Running discovery for pytest using the new test adapter.
2023-09-12 10:36:58.410 [info] > ~/src/venv/rti/bin/python -m pytest -p vscode_pytest --collect-only --ignore=./rad_rpi/validation rad_rpi rad_util
2023-09-12 10:36:58.410 [info] cwd: .
2023-09-12 10:36:59.369 [info] Test server connected to a client.
2023-09-12 10:36:59.391 [info] Using result resolver for discovery
2023-09-12 10:57:39.478 [info] Experiment 'pythonPromptNewFormatterExt' is active
2023-09-12 10:57:39.478 [info] Experiment 'pythonPromptNewToolsExt' is active
2023-09-12 10:57:39.478 [info] Experiment 'pythonTestAdapter' is active
2023-09-12 10:57:39.653 [info] Test server listening.
2023-09-12 10:57:39.678 [info] Python interpreter path: ~/src/venv/rti/bin/python
2023-09-12 10:57:43.242 [info] Starting Pylance language server.
2023-09-12 11:04:23.784 [info] Discover tests for workspace name: connext-rpi - uri<EMAIL_ADDRESS>2023-09-12 11:04:23.784 [info] Running discovery for pytest using the new test adapter.
2023-09-12 11:04:23.907 [info] > . ~/src/venv/rti/bin/activate && echo 'e8b39361-0157-4923-80e1-22d70d46dee6' && python ~/.vscode/extensions/ms-python.python-2023.16.0/pythonFiles/printEnvVariables.py
2023-09-12 11:04:23.907 [info] shell: bash
2023-09-12 11:04:23.967 [info] > ~/src/venv/rti/bin/python -m pytest -p vscode_pytest --collect-only --ignore=./rad_rpi/validation rad_rpi rad_util
2023-09-12 11:04:23.967 [info] cwd: .
2023-09-12 11:04:23.988 [info] Discover tests for workspace name: undefined - uri<EMAIL_ADDRESS>2023-09-12 11:04:23.988 [info] Running discovery for pytest using the new test adapter.
2023-09-12 11:04:26.504 [info] Test server connected to a client.
2023-09-12 11:04:26.558 [info] Using result resolver for discovery
2023-09-12 11:04:28.713 [info] Discover tests for workspace name: undefined - uri<EMAIL_ADDRESS>2023-09-12 11:04:28.714 [info] Running discovery for pytest using the new test adapter.
2023-09-12 11:04:28.753 [info] > ~/src/venv/rti/bin/python -m pytest -p vscode_pytest --collect-only --ignore=./rad_rpi/validation rad_rpi rad_util
2023-09-12 11:04:28.753 [info] cwd: .
2023-09-12 11:04:30.981 [info] Test server connected to a client.
2023-09-12 11:04:31.031 [info] Using result resolver for discovery

User Settings

venvFolders: "<placeholder>"

languageServer: "Pylance"

formatting
• provider: "none"

testing
• pytestArgs: "<placeholder>"
• pytestEnabled: true

Extension version: 2023.9.10
VS Code version: Code 1.82.0 (8b617bd08fd9e3fc94d14adb8d358b56e3f72314, 2023-09-06T22:07:18.759Z)
OS version: Linux x64 6.2.0-32-generic
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 4293)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
15.23GB (4.03GB free)

Process Argv
--unity-launch --crash-reporter-id 3c46e375-6a37-43a7-9f7f-e83c17a92013

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30792226
vscaac:30438847
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30803844
2e4cg342:30602488
89544117:30613380
showlangstatbar:30737416
962ge761:30835153
03d35959:30757346
7ij38806:30736111
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxpt:30805730
pythonnoceb:30805159
asynctok:30821568
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572
dsvsc015cf:30829746

It would be easier to discuss this if you could use a public package in your repro steps. Is rad.util.frame coming from a public package? I tried pip install rad, but there's no util module in there.

rad.util is an in-house and proprietary package, one of several in a rad package ecosystem.
I tried creating a micro-example but it turns out that the namespace-package aspect is not the problem; my bad assumption.
But I can't give you the code to debug with and I don't know what else to look at to fix this problem. All I know is, Python has no problems with the imports that Pylance is flagging as errors.

Can you take a look at our Typing Guidance for Python Libraries doc page, in particular the Library Interface section, and make sure you are using supported patterns for exposing your public interface?
Foe example, a common problem is for a module to import something using from X import A and expect that type checkers will treat that as part of the importing module's public interface, whereas you need to use from X import A as A instead.

Thanks for that, I'll read it more in-depth later. I was unaware of the "redundant alias" technique.
However, that doesn't apply for the cases where I'm seeing the problem; rad.util is a pretty simple library and each module defines a single class at the top level; class Frame is defined in frame.py. There are no attempts at exporting an imported symbol here.
Most of the imports from this package are from rad.util.MODULE import XXX but a few are from rad.util import MODULE; after going through the client sources, it seems all of these are flagged with reportMissingImports. The error is specifically reported for rad.util.MODULE in the first case or rad.util in the second, and the error-underlining applied to the same tokens, as if Pylance can't find the package itself. But the package is definitely installed in the selected interpreter, and if I open a Python with Code's bash shell, I can successfully enter the import statements manually.

Can you please provide a Pylance log with tracing enabled as described under "Filing an issue" in the troubleshooting guide?

No need for that. The document you provided led me to the solution: rad.util was an editable install.
https://microsoft.github.io/pyright/#/import-resolution?id=editable-installs
I don't quite follow the bit about .pth files; there already was __editable__.rad.util-3.0.0.pth file in site-packages, created by pip, but evidently it wasn't pyright enough. However, it's not necessary for that package to be editable, so I just reinstalled the wheel, and the unresolved errors disappeared.
Thanks again for the document.

I don't quite follow the bit about .pth files; there already was __editable__.rad.util-3.0.0.pth file in site-packages

Maybe we need to clarify that. The concern is whether the .pth file simply contains paths to add to sys.path or if it also contains executable lines that configure import hooks. https://docs.python.org/3/library/site.html


Specify outbound driver

With SlackBot it was possible to use the reply outside of the standard hear etc to send a message. Is there a way to specify the driver to keep using this functionality?
ie. bot initiated message

Well you could use the reply method outside of the listener callbacks, but you need to specify the channel, recipient, etc. nonetheless. How did you do this using the old version of this package?
There is no such thing as being initialized because BotMan only responds to incoming webhooks.

$slackbot = new \SlackBot();
        $slackbot::reply('hello', [
          'channel' => 'dev',
          'username' => 'Bot',
          'icon_emoji' => ':robot_face:',
      ]);

Doing this with BotMan no longer seems to work anymore, do we need to pass the driver in?

Ah right. That will no longer work because BotMan doesn't know which driver to use.
Sol it'd be best to pass the driver classname to the rely method in this case.
But isn't that basically the "fallback"  method you're looking for?

Do you have an example of how to to pass the driver classname into the reply method?

It's currently not possible to do this.
Please take a look at the fallback method, which gets triggered if no keyword matches your bot:
https://github.com/mpociot/botman#fallback-replies

Thanks, I guess we'll stick to SlackBot for a while.
It's more our app uses both the bot and sending bot initiated messages for other notifications. I'll look into submitting a PR if I get some time.
Thanks!


chore: release main

:robot: I have created a release beep boop
base: 0.0.38
0.0.38 (2024-10-24)
Features

✨ update base to net8 (#200) (b97318b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

base-v0.0.38
:sunflower:


twister: Fix inconsistency among DT compat filters

This small change concerns the following filter functions:

dt_compat_enabled(C):
There's a node with compatible C and status "okay".

dt_enabled_alias_with_parent_compat(A, C):
There's a node with alias A and status "okay", and its parent
has compatible C.

dt_label_with_parent_compat_enabled(L, C):
There's a node with label L, and its parent has compatible C and status "okay".

All three functions involve checking whether some node or its parent has a given compatible, but the way this has been checked is inconsistent. Function (1) has done it with this Python conditional:
   compat in node.compats

while (2) and (3) have used:
   parent.matching_compat == compat

The first check works well with nodes that have multiple compatibles, and it is more aligned with the notion of "has_compat" as seen in the devicetree macros for C, CMake, and Kconfig.
Arguably, matching_compat shouldn't have been used here, because it is actually a property of a node's binding, moreso than of the node itself. In practice, it's usually equal to the first compatible for which edtlib has found a binding, which at first glance is just more constrained than the node.compats check. However, there also exist obscure cases where the node.compats are empty, while the node.matching_compat is not.
For now, the three functions can use a combined check, to improve consistency and utility while avoiding breakage:
   node.matching_compat == compat or compat in node.compats

@nashif please take a look

@gchwier @golowanow @PerMac  @hakehuang  please review


Replying includes the person’s username at the start of the reply

Sorry if the title is confusing.
 This is what I mean, I’m not sure if this is a per-app thing though

This is usually client specific behaviour, it's possible SkyBridge could trim it off but I worry about possible side effects when it's intended in a post


starsector: 0.96a-RC10 -> 0.97a-RC8

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for starsector is: Open-world single-player space-combat, roleplaying, exploration, and economic game
meta.homepage for starsector is: https://fractalsoftworks.com
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.97a-RC8 in filename of file in /nix/store/0l8hdhs3g4c6dcp5mwkmmz0gavfag019-starsector-0.97a-RC8

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
starsector

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0l8hdhs3g4c6dcp5mwkmmz0gavfag019-starsector-0.97a-RC8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A starsector https://github.com/r-ryantm/nixpkgs/archive/ad7edcb4f1fe7ed46fbc957a04c21f48963acca4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ad7edcb4f1fe7ed46fbc957a04c21f48963acca4#starsector

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0l8hdhs3g4c6dcp5mwkmmz0gavfag019-starsector-0.97a-RC8
ls -la /nix/store/0l8hdhs3g4c6dcp5mwkmmz0gavfag019-starsector-0.97a-RC8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

starsector

Maintainer pings
cc @bbigras @rafaelrc7 for testing.

~ $ result/bin/starsector
OpenJDK 64-Bit Server VM warning: ignoring option PermSize=192m; support was removed in 8.0
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=192m; support was removed in 8.0
Conflicting collector combinations in option list; please refer to the release notes for the combinations allowed
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.
~ [1]$

This release relies on jre7 and is incompatible with 8, at least in the startup script.

close in favour of #286586


perf: screencast mode mouse indicator position

add calcPositionY | calcPositionX To calculate the position, do not write in the string template

This feature doesn't really have perf issues, but thanks anyway!
If you're looking for ideas how to contribute, start here: https://github.com/microsoft/vscode/wiki/How-to-Contribute#contributing-to-visual-studio-code


Update module to version 1.0.3

Just a few things:
I'm not sure I see any benefit in top scoping every variable here.
If you're removing a parameter, that according to Semantic versioning is a breaking change, and therefore there should be a major version bump (2.0)
And if you're feeling slightly bored, consider performing a modulesync with the puppet-community modulesync_configs - puppet-lint is likely not happy with the alignment of the variable blocks.

Considering this module is no longer supporting Puppet 2.7 I have to agree with @juniorsysadmin. If this has been the style from the beginning sure but now it's just a lot of extra churn.
This should indeed also be a major version upgrade. You could leave the parameter in place and gracefully degrade but removing it outright needs a different version bump.

Alright. I removed the changes regarding :: and just changed the value of install_dir instead of removing the parameter.
I will also take a look at modulesync_configs, because my current Lint setup does not complain about the alignment.

Done. I also removed the validations of $version and $scala_version.

I would love to fix this, but somebody removed me from the team modules/admin.
dhoppe@macbook ~/Projects/puppet/puppet-kafka (git)-[v1.0.3] % git push origin v1.0.3
ERROR: Permission to voxpupuli/puppet-kafka.git denied to dhoppe.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

@dhoppe Yes, I did. That team is meant for people to be in to facilitate transferring of the module, not a blanket team to just be in and have admin access to everything. I've just given collaborators access, as should've been done from the start, so you can push again.

ugh, i'm very very sorry about this, @dhoppe!

I would like to see the this patch being merged, because there are still other issues left.

Hello:
Can you rebase and/or squash?

@jyaworski I am confused. Why do I need to perform a rebase/squash?

@dhoppe it turns your 9 commits into one, and makes the PR cleaner.

@jyaworski Done, but it still makes no sense. Before the rebase / squash every commit was well documented.

Thanks for the contribution, @dhoppe!


404 on consecutive calls

I noticed that the first time I hit my url it will work fine, but if right after I try again it will give me a 404.
If I wait 9 minutes or so it will work again.
This seems to be associated with the mod_auth_openidc_session being cached.
Is there a way to prevent this behavior?

Can you provide a debug log of that and send your http.conf?

Unfortunately I cant.
You suspect that something in apache other than the auth oidc module is rejecting the request?
This is the bad request I observe in the browser
https://some.com/oidc/abc
GET /oidc/abc HTTP/1.1
Host: some.com
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
Cookie: __utma=264805659.704981365.1480452213.1480455841.1480457718.3; __utmz=264805659.1480452213.1.1.utmcsr=(direct)|utmccn=(direct)|utmcmd=(none); _ga=GA1.3.704981365.1480452213; __utmb=2648056<IP_ADDRESS>0457718; __utmt=1; _gat_UA-74627609-1=1; mod_auth_openidc_session=08d59cbe-9506-294c-a9bb-0a5ef3ce5bc3; JSESSIONIDSSO=52DB5F2300F70AD8981B913BD309C84B; __utmc=264805659
Connection: keep-alive
Upgrade-Insecure-Requests: 1
HTTP/1.1 404 Not Found
Date: Tue, 29 Nov 2016 22:19:40 GMT
Server: Apache/2.4.12 (Win32) OpenSSL/1.0.2 mod_flvx/0.4
Set-Cookie: mod_auth_openidc_session=08d59cbe-9506-294c-a9bb-0a5ef3ce5bc3;;Secure;Path=/;HttpOnly
Content-Length: 206
Connection: close
Content-Type: text/html; charset=iso-8859-1

really hard to say anything about it based on just the client-side info; the server-side debug log would tell you what mod_auth_openidc is doing before the 404 is returned and would probably also show another module/function kicking in

I found out that the issue is the cookie
mod_auth_openidc_session
is not cleared at the end of the authentication.
Is there an option to clear it after authentication is complete?

mod_auth_openidc_session is keeping track of the fact that the user has already authenticated; if that cooke would not be set after authentication, the user would be sent off to the Provider again to authenticate and you'd send the browser in an endless loop...


Refactor clustering into generic "grouping" interface.

This PR re-thinks, and in my opinion improves, the interface we have in AttractorsViaFeaturizing. Now, a generic "grouping" step is performed. This detaches featurizing from clustering. Trajectories are mapped into features as normal, but now, how these features are "grouped" into attractors is fleible. We have three implementations:

clustering
nearest features
histogram

As you will see in the source code, AttractorsViaFeaturizing now knows nothing about clustering. All it needs is some kind of configuration struct that implements the function group_features. any kind of struct that can group features therefore can be used.
We can use the same interface in the continuation PR#10 . All we need is one more function group_features_along_parameter that also takes into account the fact that we have a paramater axis that may, or may not, be used.

The PR also re-writes the "templates" code (now nearest features) and It also improves drastically performance of the "templates" version.

Now the templates version and the histogram version allow the syntax mapper(u0) that gives you the attractor label given a single initial condiiton, that clustering cannot do.

One more thing this PR does: changes the "clustering" code to never use matrices. Vectors of static vectors are used instead. pairwise is super efficient in this form, and all related computations remain super efficient as well. I never understood why we make the features into a matrix. pairwise works perfectly fine with vectors of vectors.

Tests seem to pass. They won't pass here because Entropies.jl is broken at the moment. Only after I update Entropies.jl, I'll make another PR here that finishes the Histogram method and fixes the tests (because at the moment tests will never pass no matter what).

I left only minor comments. I will need to change some things in PR #10 to adapt to the new interface. As soon as this is merged I will finish the PR.

Alright, I am merging this in. Some comments:

@awage I made the source code for group_features(..., ::GroupViaClustering) as modular as possible. This will allow you to write your own function that calls inner functions normally, without having to pass in distances or do if distances === nothing hacks.
@awage now that this is merged, I would recommend that you actually close #10 , and open a new pull request that is branching directly from main branch and adds your featurizing continuation. It will make the pull request much easier to review, because at the moment we have so many different parallel commits here and there that kind of inter-depend on each other. It will be much cleaner with a new PR from current main.
Tests do not pass. Documentation does not build. The histogram method is not finished. However, all of these issues stem from the fact that Entropies.jl is broken at the moment. Soon I'll fix Entropies.jl and then open a new pull request that implements the histogram method and fixes the tests.

The package doesn't compile due to the missing FixedRectangularBinning. It makes it harder to adapt the PR #10 to the new interface. Is there a quick fix to ignore this error?

Remove the Histogram file in grouping, or remove the type FixedRectangularBinning from the type. I 'll probably finish this today.


Crashing on execution

the evillimiter.exe program crashes on execution without starting a gui on windows build 1909

Completely uninstall and reinstall using setup.exe.  A new version of npcap will be installed and it will work fine after.

I have the same issues. Reinstalled twice and finally tried to launch from the command line. The error message displayed was that it could not find _curses_cffi. Any help?

@dave7895 When you got the ncap installation prompt did you select the box next to select the box where it says "Install npcap in WinPcap API-compatible mode?"

yes I did but the error persists.

PATH isn't find tools; try and reinstall.

You installed the linux version of EvilLimiter on Windows. That will not work.


Fixes Color.fromCssColorString when color string contains spaces

Fixes #9015

Thanks for the pull request @sambakk!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Thanks for the PR @sambakk!  That was my only comment

Oh, also please add a comment about this to CHANGES.md under the Fixes section

Thanks for the PR @sambakk! That was my only comment

You're welcome @hpinkos, I updated the PR as requested.

Thanks for the contribution @sambakk!


Update README.md

Summary of Changes
Include a summary of changes then remove this line
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test
[ ] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

cc @youngyjd @jinhyukchang

Codecov Report

Merging #51 into master will not change coverage.
The diff coverage is n/a.

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


chore: Enable PMD check during build

This PR enables PMD during build.
For now, the initial setup has disabled any rule that has issues in the existing code base.
Expectations is that in the future, and if a certain rule is required, a PR to enable it by fixing the code, should be created.

Modified files are excluded from the automated pipeline, so I ran it manually and here's the result -- All green :)


Layer sync failed

When synced, "Amazon_def_fronts" layer won't show edit layer or create view buttons

The last geometry of this layer is null, causing a sync fail.


Migrating user data

Thanks for the great tutorial.
I would really like to see that last piece come into play, which is the migration of the anonymous user data in case a user reinstalls the apps, makes more changes even though he has signed up before, and then tries to sign in again. Do you think you could add this in a separate post or point to a solution?

Thanks for the kind words, @hoschkara !
Writing (and vlogging) about this solution is high up on my list. I will update this issue as I make progress. Would you be interested in providing feedback on a draft of the article?

It would be a pleasure!

Ok, great - can you DM me your email address (my Twitter DMs are open), and I'll invite you to the Google doc.

I don't use twitter, but email is<EMAIL_ADDRESS>also schreib mir einfach :)

You got this message @peterfriese?

I did - hope to start working on the draft this week :-)

How is it going? :)

Not trying to be annoying at all, but do you have an approximate release date for the tutorial? Just for me to plan my own agenda :)

If you're curious about the implementation, check out this branch, @hoschkara
This works, but it's still WIP and might change. Drafting of article has begun.

I will, thanks!

Implementation merged into the main branch.
The article is now published at https://peterfriese.dev/replicating-reminder-swiftui-firebase-part4/
Hope this helps - feel free to leave feedback.


Genre_Books

A list of books with their genre and synopsis.

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Good one @sanjana1391
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow these steps.

Star this repository. ⭐
Follow Clueless on GitHub
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard!

ClueLess Profile - https://www.clueless.tech/profile/admin/55924455
@nikhil25803


Add --python flag for 3.10 / 3.11 support (3.12 too unstable for some use-cases)

tpux --is_tpu_pod n --install_zsh y --add_path_to_shell_config y --python 3.10

Thanks for your feedback! The TPU environment is intended for development, so we want to encourage users to always develop based on the latest version. If there's a need for different versions, you can create and install different Python versions or set up venv.


GANExchange

@notadamking Here, ready to merge also fixed a few issues which were breaking the StochasticExchange After your changes to the SimulatedExchange.

ok, I will Do the refactor

@notadamking I am closing this and will track the new pull request #194


AOT compiler seems to remove switch branches that are actually needed at runtime

dart info:
#### General info

- Dart 3.3.2 (stable) (Tue Mar 19 20:44:48 2024 +0000) on "macos_arm64"
- on macos / Version 14.1.1 (Build 23B81)
- locale is en-DE

#### Project info

- sdk constraint: '^3.3.2'
- dependencies: freezed_annotation
- dev_dependencies: build_runner, freezed, lints, test

#### Process info

|  Memory |  CPU | Elapsed time | Command line                                                                               |
| ------: | ---: | -----------: | ------------------------------------------------------------------------------------------ |
|   41 MB | 0.0% |     18:54:33 | dart devtools --machine --allow-embedding                                                  |
| 3542 MB | 0.0% |     18:54:33 | dart language-server --protocol=lsp --client-id=VS-Code --client-version=3.84.0            |
|  926 MB | 0.0% |     01:58:29 | dart language-server --protocol=lsp --client-id=VS-Code --client-version=3.84.0            |
|  904 MB | 0.0% |     01:38:51 | dart lib/custom_lint_client.dart ::1 62540                                                 |
|   63 MB | 0.0% |     18:54:33 | flutter_tools.snapshot daemon                                                              |
|   69 MB | 0.0% |     01:46:44 | flutter_tools.snapshot debug_adapter                                                       |

Operating System
I have reproduced the issue on MacOS as well as iOS and Android (with Flutter)
How to reproduce
I have set up a minimal example that reproduces the error (at least on my machine): https://github.com/Yegair/dart-aot-error-reproduction
About the Error
The error first showed up in a Flutter release build on Android, but it was also reproducible on iOS as well as in a non-Flutter project running on MacOS (see reproducible example above).
Apparently the compiler removes some branches from a switch expression that are actually needed at runtime. The expression looks something like in the following example. WrapperA and WrapperB are subclasses of some sealed class Wrapper. The implementations of those classes have been generated using the freezed library. Without the usage of freezed, I have not been able to reproduce the issue, so this might be where the problem originates from, but I have been unable to figure out the exact cause so far.
switch (wrapper) {
    WrapperA(inner: Inner(other: final Other other)) => other.text,
    WrapperA(inner: Inner(other: null)) => "no other",
    WrapperB(inner: Inner(other: final Other other)) => other.text,
    WrapperB(inner: Inner(other: null)) => "no other",
}

From what I can tell, the last two branches are removed during compilation, which at runtime leads to the error:
Attempt to execute code removed by Dart AOT compiler (TFA)

I assume the compiler for some reason concludes, that the first two branches already cover all possible cases, which is not that far from the truth, because rewriting the expression like the following fixes the issue.
switch (wrapper) {
    Wrapper(inner: Inner(other: final Other other)) => other.text,
    Wrapper(inner: Inner(other: null)) => "no other",
}

//cc @alexmarkov

@Yegair please add some open source license to your project, we can't look at the unlicensed code.

I was able to reproduce incorrect compilation on the following code:
class Other {
  String text = 42.toString();
}

class Inner {
  Other? other = int.parse('3') == 3 ? Other() : null;
}

sealed class Wrapper {}

class WrapperA extends Wrapper {
  Inner inner = Inner();
}

class WrapperB extends Wrapper {
  Inner inner = Inner();
}

var obj = int.parse('1') == 1 ? WrapperB() : WrapperA();

void main() {
  foo(obj);
}

void foo(Wrapper wrapper) {
  print(switch (wrapper) {
    WrapperA(inner: Inner(other: final Other other)) => other.text,
    WrapperA(inner: Inner(other: null)) => "no other",
    WrapperB(inner: Inner(other: final Other other)) => other.text,
    WrapperB(inner: Inner(other: null)) => "no other",
  });
}

Kernel AST before modular transformations:
   core::print( block {
      core::String #t1;
      final synthesized foo::Wrapper #0#0 = wrapper;
      synthesized foo::Other? #0#3;
      synthesized core::bool #0#3#isSet = false;
      synthesized core::bool #0#5;
      synthesized core::bool #0#5#isSet = false;
      #L1:
      {
        {
          final hoisted foo::Other other;
          if(#0#0 is foo::WrapperA && (#0#3#isSet ?{foo::Other?} #0#3{foo::Other?} : let final dynamic #t2 = #0#3#isSet = true in #0#3 = #0#0{foo::WrapperA}.{foo::WrapperA::inner}{foo::Inner}.{foo::Inner::other}{foo::Other?}) is foo::Other) {
            other = (#0#3#isSet ?{foo::Other?} #0#3{foo::Other?} : let final dynamic #t3 = #0#3#isSet = true in #0#3 = #0#0{foo::WrapperA}.{foo::WrapperA::inner}{foo::Inner}.{foo::Inner::other}{foo::Other?}) as{Unchecked} foo::Other;
            #t1 = other.{foo::Other::text}{core::String};
            break #L1;
          }
        }
        {
          if(#0#0 is foo::WrapperA && (#0#5#isSet ?{core::bool} #0#5{core::bool} : let final dynamic #t4 = #0#5#isSet = true in #0#5 = (#0#3#isSet ?{foo::Other?} #0#3{foo::Other?} : let final dynamic #t5 = #0#3#isSet = true in #0#3 = #0#0{foo::WrapperA}.{foo::WrapperA::inner}{foo::Inner}.{foo::Inner::other}{foo::Other?}) == null)) {
            #t1 = "no other";
            break #L1;
          }
        }
        {
          final hoisted foo::Other other;
          if(#0#0 is foo::WrapperB && (#0#3#isSet ?{foo::Other?} #0#3{foo::Other?} : let final dynamic #t6 = #0#3#isSet = true in #0#3 = #0#0{foo::WrapperA}.{foo::WrapperA::inner}{foo::Inner}.{foo::Inner::other}{foo::Other?}) is foo::Other) {
            other = (#0#3#isSet ?{foo::Other?} #0#3{foo::Other?} : let final dynamic #t7 = #0#3#isSet = true in #0#3 = #0#0{foo::WrapperA}.{foo::WrapperA::inner}{foo::Inner}.{foo::Inner::other}{foo::Other?}) as{Unchecked} foo::Other;
            #t1 = other.{foo::Other::text}{core::String};
            break #L1;
          }
        }
        {
          if(#0#0 is foo::WrapperB && (#0#5#isSet ?{core::bool} #0#5{core::bool} : let final dynamic #t8 = #0#5#isSet = true in #0#5 = (#0#3#isSet ?{foo::Other?} #0#3{foo::Other?} : let final dynamic #t9 = #0#3#isSet = true in #0#3 = #0#0{foo::WrapperA}.{foo::WrapperA::inner}{foo::Inner}.{foo::Inner::other}{foo::Other?}) == null)) {
            #t1 = "no other";
            break #L1;
          }
        }
      }
    } =>#t1);

WrapperB cases are incorrectly optimized out by the AOT compiler because of the incorrect AST.
Instead of calling WrapperB.inner getter in the WrapperB cases, there are calls to WrapperA.inner:
#0#0 is foo::WrapperB && ... #0#3 = #0#0{foo::WrapperA}.{foo::WrapperA::inner}{foo::Inner} ...

This looks like incorrect lowering of patterns in CFE.
@johnniwinther @chloestefantsova Could you take a look?


Scrap URLs when articles are submitted

This PR makes sure CreateArticle calls would invoke scrapUrls over the submitted text so that all the URLs in the submitted messages can be scrapped, also fills in its hyperlinks field for future searches.
Related to #41
Major changes

Add scrapUrls to CreateArticle to fill hyperlinks field

Minor changes

Remove unnecessary cacheKeyFn from urlLoaderFactory because URL strings are already serializable
Alter scrapUrls so that the returned url in ScrapResult always matches the one in input text

Coverage increased (+0.1%) to 92.491% when pulling b2b71f0f576d8ced50aab61a8f1a04770e378f33 on 41-create-article into 47255aad82c673b721b86716a934f286f7dfdca8 on 41-url-preview.


NameError: name 'Wav2Vec2FeatureExtractor' is not defined

Cant load any model of speechbrain over colab, gives this error:
---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)
<ipython-input-23-5a18e5459f72> in <module>()
      1 from speechbrain.pretrained import EncoderDecoderASR
----> 2 asr_model = EncoderDecoderASR.from_hparams(source="speechbrain/asr-wav2vec2-transformer-aishell", savedir="pretrained_models/asr-wav2vec2-transformer-aishell")
      3 asr_model.transcribe_file("speechbrain/asr-wav2vec2-transformer-aishell/example_mandarin.wav")

11 frames
/usr/local/lib/python3.7/dist-packages/speechbrain/lobes/models/huggingface_wav2vec.py in __init__(self, source, save_path, output_norm, freeze, freeze_feature_extractor, pretrain, apply_spec_augment)
     80         # Download the extractor from HuggingFace.
     81         # The extractor is only used to retrieve the normalisation
---> 82         self.feature_extractor = Wav2Vec2FeatureExtractor.from_pretrained(
     83             source, cache_dir=save_path
     84         )

NameError: name 'Wav2Vec2FeatureExtractor' is not defined

@TParcollet

@Aksh97 ,
Strange behaviour,
we already check if you have the Wav2Vec2FeatureExtractor in https://github.com/speechbrain/speechbrain/blob/29b3b7418c113615c40a78f67aef313d28f525ff/speechbrain/lobes/models/huggingface_wav2vec.py#L19
Could you please share if transformers lib is already installed in your collaboration ?
otherwise
pip install transformers

is it working for you ?

Since this has been left unaddressed. At  least for me transformers was not installed and installing it resolved the problem.


CSUB-1099: Make cli integration tests more robust

Description of proposed changes

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

FTR example of the newly added global.beforeEach() timing out because the chain had stopped finalizing blocks:
https://github.com/gluwa/creditcoin3/actions/runs/8449418081/job/23144017513?pr=289
I was expecting an assertion failure but probably when finalization stops the queries we're waiting for don't return in time hence the timeout.


Estimate API Cost

...since our friends at ADB are not very responsive, we might need to purchase API access like a commercial client. Please estimate how much our use case would cost us.

Whipped up a quick PDF for ease of sharing the information. Is this up to standard for you to show McKenna?
Cost Estimation API.pdf

@michaelweinold
We have a bit of change here. I contacted the ADB-Team about the API request we will use for the flight data and they confirmed what I thought might be true: One API request returns the data for 7 days and not just one specified date. See screenshot:

This would leave us with the following new numbers for the Cost Estimation API.docx


Change the log level of adding credits from Info to Verbose

Description
Stress tests found that there were too many add credit logs.

Eventhubs processor can create ~500MB of logs per day logs , most of which are:

Servicebus processor can create ~200MB of logs per day, most of which are:

It is also mentioned in the log tuning issue: https://github.com/Azure/azure-sdk-for-java/issues/20836#issuecomment-999052877
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Mypy performance fix

Using a Union TypeAlias with lots of members can significantly slow down mypy, especially the uncompiled version. Noticed it while testing a mypy dev build with Home Assistant. The CI run took significantly longer, 30+ min compared to 7-8min normally.
The slowdown in HA only happened after the 0.37.0 update, since that's when py.typed was added. Previously mypy just ignored the dependency.
--
Specifically, the MultilevelSensorScaleType alias with 30+ members is a problem. The fix here is relatively simple. We can create a common enum base class and use that instead of the union TypeAlias. The only thing to remember is that no members can be added to the base class directly. Otherwise subclassing enums wouldn't work.

Small followup now that Home Assistant has been updated to use 0.37.1, even the normal mypy runs are significantly faster. Sub 1:45min for a full mypy CI run, compared to 2:30-3min previously. Wasn't expecting it, but obviously great to see 🚀

excellent, thanks for your help with this!


[prod] Use embed to distribute the software as a single binary

Use the https://pkg.go.dev/embed package to package the default theme inside the binary

Closed by #48


DRAFT: Make all french-law related tests use modules

Currently all inline tests pass, but there remains some work for the other expected artefacts like Python files, json schemas, etc.

Replaces https://github.com/CatalaLang/catala/pull/547


Training data is available or not?

Dear All,
I'd like to know whether the training data is available or not.

Hi Ensheng Shi, you can download the file issuedata.zip (the whole dataset for the paper) from Zenodo and follow README.md to reproduce the experiments in the paper. Meanwhile, data_guide.md is a more detailed description of the dataset.

Many thanks 😊 ！！！

Many thanks 😊 ！！！

My pleasure : )


Recieving ENORESTARGET for versions with a + in them

I'm getting the ENORESTARGET error for bootswatch right now, but my guess is maybe an issue with versions that include +<build>. It worked on the initial bower install --save bootswatch. I have several other dependencies in my bower.json that are plain x.y.z version format that seem unaffected (mostly the default ember 2.3 stuff).
If bootswatch is not in my bower.json then I receive this output:
app>bower install --save bootswatch
bower cached        git://github.com/thomaspark/bootswatch.git#3.3.6+1
bower validate      3.3.6+1 against git://github.com/thomaspark/bootswatch.git#*
bower cached        git://github.com/twbs/bootstrap.git#3.3.6
bower validate      3.3.6 against git://github.com/twbs/bootstrap.git#~3.3.6
bower install       bootswatch#3.3.6+1
bower install       bootstrap#3.3.6

bootswatch#3.3.6+1 bower_components\bootswatch
└── bootstrap#3.3.6

bootstrap#3.3.6 bower_components\bootstrap
└── jquery#2.1.4

After that if anyone without bootswatch runs a bower install they will receive this output:
app>bower install
bower not-cached    git://github.com/thomaspark/bootswatch.git#^3.3.6+1
bower resolve       git://github.com/thomaspark/bootswatch.git#^3.3.6+1
bower ENORESTARGET  Tag/branch ^3.3.6+1 does not exist

Additional error details:
Available tags: v2.0.0, v2.0.1, v2.0.2, v2.0.3, v2.0.4, v2.1.0, v2.1.1, v2.1.1+1, v2.2.2, v2.2.2+1, v2.2.2+2, v2.3.0, v2.3.0+1, v2.3.0+2, v2.3.1, v2.3.1+1, v2.3.2, v3.0.0, v3.0.0-rc1, v3.0.0-rc2, v3.0.1, v3.0.2, v3.0.2+1, v3.0.2+2, v3.0.3, v3.0.3+1, v3.1.0, v3.1.0+1, v3.1.1, v3.1.1+1, v3.2.0, v3.2.0+1, v3.2.0+2, v3.2.0+3, v3.2.0+4, v3.3.0, v3.3.1, v3.3.1+1, v3.3.1+2, v3.3.1+3, v3.3.2, v3.3.4, v3.3.4+1, v3.3.5, v3.3.5+1, v3.3.5+2, v3.3.5+3, v3.3.5+4, v3.3.6, v3.3.6+1

Available branches: gh-pages, master

I have confirmed this behavior in ConEmu, plain ol' cmd.exe, and Git Bash. I have also tried disabling Inject ConEmuHk (per https://github.com/bower/bower/issues/852#issuecomment-145169743) in ConEmu with no effect on behavior:

What I have found is if I bump the version down to v3.3.6 instead of v3.3.6+1 then the bower install command succeeds:
app>bower install
bower bootswatch#^3.3.6         cached git://github.com/thomaspark/bootswatch.git#3.3.6+1
bower bootswatch#^3.3.6       validate 3.3.6+1 against git://github.com/thomaspark/bootswatch.git#^3.3.6
bower bootstrap#~3.3.6          cached git://github.com/twbs/bootstrap.git#3.3.6
bower bootstrap#~3.3.6        validate 3.3.6 against git://github.com/twbs/bootstrap.git#~3.3.6
bower bootswatch#^3.3.6        install bootswatch#3.3.6+1
bower bootstrap#~3.3.6         install bootstrap#3.3.6

bootswatch#3.3.6+1 bower_components\bootswatch
└── bootstrap#3.3.6

bootstrap#3.3.6 bower_components\bootstrap
└── jquery#2.1.4

I'm running:
Windows 7 Professional SP 1
git version 1.9.5.msysgit.0
node v4.2.1
bower 1.7.7

@bower/windows please can you have a look at this one?

This isn't a windows specific issue, I'm getting the same error when try to do a bower install on Mac OSX 10.11.3
bower bootswatch#~3.3.6+1            ENORESTARGET Tag/branch ~3.3.6+1 does not exist
The bower.json I'm installing from contains the same reference to the latest version of bootswatch which happens to contain a "+" in the version number. This does seem a little odd as all other versions I've come across are decimals, just digits and dots.

I actually tested, I upgrade semver to 4.0 and this solve the issue, but I can see that there are some test that fail when I run npm test, things related with GitResolver, to be honest I'm not an expert in test so I will have to figure it out how to solve the test before I send a PR.

We cannot upgrade to sember to 4.0 because it breaks backward compatiblity (it treats prereleases differently). You can install tag directly though, even with +: bower install 'https://github.com/thomaspark/bootswatch.git#3.3.6+1'


feat(fs): improved API

It doesn't compile with the watch feature enabled. Just a fairly small change though (the main thing is that resolve_path expects a SafePathBuf) but i won't push it to not risk conflicts if you have local commits left :)
(p.s. opened https://github.com/tauri-apps/plugins-workspace/pull/761 to test more features in ci)
The main api change looks solid to me though (though i do think node's fs api is a bit weird idk), and what i tested so far worked nicely! 🥳 (i'll do some more testing tomorrow)
I do however think that this PR should have 2 approvals (maybe i just don't trust myself enough lol).

Alright, watcher flag should be fine now, and feel free to push your changes if you need, I don't have any more changes planned.

though i do think node's fs api is a bit weird idk)

The API is inspired from Deno and Node and the main goal was to make it more familiar to those coming from node.

Why was the close export removed? Isn't that required for open to make sense? (honestly already forgot everything about this pr so genuine question)

I guess for security reasons, since close accepts any rid, I think they should use .close() on the returned FsFile class

btw this PR is blocked until https://github.com/tauri-apps/tauri/pull/8370


rescanblockchain isn't implemented in rpc

rescanblockchain is working on LTC and BTC but no on dash.
do we need it, to capture past transactions for a view-only address that we added right now?

try to use rescan https://docs.dash.org/en/stable/wallets/dashcore/cmd-rpc.html?highlight=rescan#wallet-options

Ok, glad you found a solution that's working for you! 😄


The following module(s) are not found yet:

Describe the bug
When I use Pigar in a pipenv environment, it show The following module(s) are not found yet:
However, all packages have been correctly installed with pipenv.
To Reproduce
Steps to reproduce the behavior:
pipenv shell
pigar generate

Expected behavior
Pigar should recognize these packages installed in the pipenv virtual environment, rather than going to pypi to retrieve them again .
Context (Environment)

OS: Windows
Version of Python: 3.11.0
Version of pigar: 2.1.1

Additional context
Add any other context about the problem here.
pigar -l DEBUG [command] will give you debug level logs.

I believe you should ensure that pigar recognizes your virtual environment. This implies that you may need to install pigar in your virtual environment.

Is it possible for pigar to be automatically recognized, if I install it globally, and execute this program under pipenv shell?
Alternatively, pipenv can add a specified local index path. For example, specify .venv/Lib/site-packages

I believe you should ensure that pigar recognizes your virtual environment. This implies that you may need to install pigar in your virtual environment.

I've been away from the world of Python for quite some time, but I believe you can execute pigar like this: python -m pigar.

Or maybe not..


fix(pci): import urls and redirect_urls constants

fix(pci): import urls and redirect_urls constants
Description of the Change
88f8a24c — fix(pci): import urls and redirect_urls constants
/cc @jleveugle @frenautvh @antleblanc

CDS Report installation#1142.0 ✘

Stage 1

Install ✘


Update OracleTableMetaCache.java

schema info in sql statement may be lower,

Ⅰ. Describe what this PR did
Oracle sql的语法支持schema为小写，但是通过getColumns的schemaPattern传参必须为大写，否则无法获取表的字段信息 。
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2118 into develop will increase coverage by 0.03%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop    #2118      +/-   ##
=============================================
+ Coverage      55.16%   55.19%   +0.03%
- Complexity      2513     2515       +2
=============================================
  Files            446      446
  Lines          14915    14915
  Branches        1765     1765
=============================================
+ Hits            8228     8233       +5
+ Misses          5921     5919       -2
+ Partials         766      763       -3

Impacted Files
Coverage Δ
Complexity Δ

...asource/sql/struct/cache/OracleTableMetaCache.java
6.12% <0%> (ø)
3 <0> (ø)
:arrow_down:

...o/seata/server/coordinator/DefaultCoordinator.java
48.03% <0%> (+0.39%)
28% <0%> (+1%)
:arrow_up:

...in/java/io/seata/server/session/GlobalSession.java
84.61% <0%> (+0.48%)
68% <0%> (+1%)
:arrow_up:

...server/store/file/FileTransactionStoreManager.java
57.46% <0%> (+0.95%)
29% <0%> (ø)
:arrow_down:

Waiting for your feedback for a long time. The bug already fixed, please check #2283.

The bug already fix. So I will close this pr, waiting for your contribution next time.


Accessibility Bug Fix: Tab focus doesn't reach Dark mode toggle button

Description
This is a fix for accessibility bug

if you press tab then the focus doesn't reach dark mode toggle button
After focus should reach the dark mode toggle button, pressing enter button should change the theme of website

e.g. Fixes #(issue)
-->
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How has this been tested

[ ] I have run bash ./tools/deploy.sh --dry-run (at the root of the project) locally and passed
[x] I have tested this feature in the browser

Test Configuration

Browser type & version:
Operating system: Windows
Ruby version: 2.7.4p191
Bundler version:  2.2.30
Jekyll version: 4.2.1

Checklist

[x] I have performed a self-review of my code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

@cotes2020


Formulas page not changing when clicking page 2 etc

Describe the bug
A clear and concise description of what the bug is.
When at Formulas page, and clicking next to page 2 or 3 etc the formulas doesnt change, but still show default 20 formulas, that you see on page 1. I can change that to 100 views and still cannot change to page 2 seeing the next row of 100
To Reproduce
Steps to reproduce the behavior:

Go to '...' Formula Management -> Formulas
Click on '....' If you have more formulas than can be shown on page 1, 20-100 formulas and click page 2 to see next batch of fomulas 21 or 101 and click next no formulas is changing
Scroll down to '....' Where you can click next to see next batch of formulas
See error, no formulas shows

Expected behavior
A clear and concise description of what you expected to happen.
I would expect that the next batch of formulas shows, and not staying on the same formulas view as in page 1 but show the next batch of formulars
Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following information:

Platform: [e.g. docker] docker
Browser [e.g. chrome, safari] all browser
PV Version [e.g. 9.1] v9.5

Additional context
Add any other context about the problem here, also provide details regarding your installation.

Fixed in version 9.6


Update kube-apiserver.md

The current link the glossary entry points to non-existence page and section. Corrected the link to points to correct location.
In the exiting page https://kubernetes.io/docs/concepts/cluster-administration/flow-control/ the 'kube-apiserver' tooltip is taking to page with wrong section.
After this change it will be fixed.
Preview Page link https://deploy-preview-47743--kubernetes-io-main-staging.netlify.app/docs/concepts/cluster-administration/flow-control/
(Click the tooltip in second line of the intro section to see the new link in action)

/lgtm
/approve


fix cmake breaking

Cmake errors in utxo pool Software wouldn't build -- not sure how we merged this

Looks good, will setup CI so we don't do this again. Merging


client access via global handle rather than arg passed everywhere

On new operator SDK branch.

https://github.com/bluek8s/kubedirector/pull/191


[INLONG-625] Add unit tests for metamanage.metastore.impl.*

Fixes #1213

rebase project branch code;
add unit tests for metamanage.metastore.impl.*

Codecov Report

Merging #1438 (d0a957e) into INLONG-570 (13a6156) will increase coverage by 4.53%.
The diff coverage is 23.06%.

@@               Coverage Diff                @@
##             INLONG-570    #1438      +/-   ##
================================================
+ Coverage          7.72%   12.25%   +4.53%
- Complexity          565     1048     +483
================================================
  Files               355      392      +37
  Lines             31081    32537    +1456
  Branches           4972     5117     +145
================================================
+ Hits               2401     3988    +1587
+ Misses            28190    27799     -391
- Partials            490      750     +260

Impacted Files
Coverage Δ

...ng/tubemq/client/consumer/ConsumerSamplePrint.java
14.28% <0.00%> (-1.10%)
:arrow_down:

.../apache/inlong/tubemq/corebase/TBaseConstants.java
0.00% <ø> (ø)

.../apache/inlong/tubemq/corebase/TokenConstants.java
0.00% <ø> (ø)

...ache/inlong/tubemq/corebase/cluster/TopicInfo.java
28.00% <0.00%> (-3.35%)
:arrow_down:

...ong/tubemq/corebase/utils/AbstractSamplePrint.java
0.00% <ø> (ø)

.../inlong/tubemq/corebase/utils/KeyBuilderUtils.java
0.00% <0.00%> (ø)

...nlong/tubemq/corebase/utils/SettingValidUtils.java
0.00% <0.00%> (ø)

...rg/apache/inlong/tubemq/corebase/utils/Tuple3.java
0.00% <0.00%> (ø)

...rg/apache/inlong/tubemq/corebase/utils/Tuple4.java
0.00% <0.00%> (ø)

...ng/tubemq/server/broker/msgstore/MessageStore.java
0.00% <0.00%> (ø)

... and 148 more

Continue to review full report at Codecov.

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


Fix hibernate config, make MXE processor public

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1162   +/-   ##
=========================================
  Coverage          ?   33.59%
=========================================
  Files             ?      201
  Lines             ?     5069
  Branches          ?     1395
=========================================
  Hits              ?     1703
  Misses            ?     3366
  Partials          ?        0

Continue to review full report at Codecov.

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


Introduces Cloud Foundry Task Launcher

Resolves #196
tested using file:// and http://
Will have to open another story about using maven URI

I'm not sure what you mean by prefix in general?
On Fri, Oct 7, 2016 at 11:55 AM, Eric Bottard<EMAIL_ADDRESS>wrote:

@ericbottard commented on this pull request.
In tasklauncher/spring-cloud-starter-stream-sink-
tasklauncher-cloudfoundry/README.adoc
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240
:

+== Using the TaskLauncher
+The task launcher will use the --uri from the TaskLaunchRequest to obtain the
+task artifact to deploy, as well as the properties and arguments to execute the
+task.
+An example of this using Spring Cloud Data Flow would look like this:

+Create The Stream
+
+stream create w7 --definition "http --port=9000 | taskLaunchRequestTransform +--uri=<uriToJar> +--deploymentProperties=spring.cloud.deployer.cloudfoundry.services=mysql +| cftask"
+
+Deploy The Stream
+
+`stream deploy w7 --properties "app.cftask.spring_cloud_deployer_cloudfoundry_url=https://api.local.pcfdev.io,
Are you asking for a suggestion of a prefix?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqdeVhdnL9aLP47nGOfbyRAP_yw9pFGks5qxmuNgaJpZM4KOBrj
.

Yep.  there are no properties that go with it.
On Thu, Oct 13, 2016 at 9:48 AM, Eric Bottard<EMAIL_ADDRESS>wrote:

@ericbottard commented on this pull request.
In tasklauncher/spring-cloud-starter-stream-sink-
tasklauncher-cloudfoundry/src/main/resources/META-INF/
spring-configuration-metadata-whitelist.properties
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240#pullrequestreview-4075964
:

@@ -0,0 +1 @@
+configuration-properties.classes=

Well, that is a bit ... empty :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240#pullrequestreview-4075964,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqdeelWfw-74EBbP88nHavC0KMrcl4rks5qzjbAgaJpZM4KOBrj
.

This will need to be rebased on top of current master btw

BTW, the parent of the tasklauncher intermediate maven project should be changed to
	<parent>
		<groupId>org.springframework.cloud.stream.app</groupId>
		<artifactId>starter-parent</artifactId>
		<version>1.1.0.BUILD-SNAPSHOT</version>
		<relativePath>../starter-parent</relativePath>
	</parent>

This should be done as part of this PR

First, the PR needs to be rebased.
THEN, when you run a full build again (with -Pfull), documentation for the props should automatically appear between the markers, and you should commit that.

The README needs to be included in the reference doc

We should add an integration test that validates the passing of properties, especially given the custom prefix we use

This does not make sense. We should investigate rather
On Wed, Oct 19, 2016 at 3:42 PM, Glenn Renfro<EMAIL_ADDRESS>wrote:

@cppwfs commented on this pull request.
In tasklauncher/spring-cloud-starter-stream-sink-
tasklauncher-cloudfoundry/pom.xml
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240
:

@@ -8,6 +8,14 @@
1.1.0.BUILD-SNAPSHOT

<!--Can remove these specific version when the main project updates netty
</li>
and reactor dependencies-->

<reactor.version>3.0.2.RELEASE</reactor.version>

Nothing has changed in the main project. Basically we get a Class Not
Found when running the integration test. So that is why the dependencies
were added.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATIlp9vdaUJJrAzRQZ-_R-NFZAmqW9Sks5q1h5fgaJpZM4KOBrj
.

I think you're commenting on an old commit, but in a nutshell:
SCDF will not pass any property that starts with spring.cloud.deployer. to
the intended app. Instead, this would go to the deployer (used by SCDF)
that is deploying said app.
Our case here is a bit peculiar, as the app we're trying to deploy (via
SCDF) itself embeds a deployer (the CF one) and so would like to customize
it. But it can't do that using the (default for that deployer) default
prefix of spring.cloud.deployer.cloudfoundry.
Hence the retargeting of those @ConfigurationProperties beans to use a
different prefix (which also happens to be shorter, which is nice to have).
Hope that makes sense
On Wed, Oct 19, 2016 at 5:42 PM, Michael Minella<EMAIL_ADDRESS>wrote:

@mminella commented on this pull request.
In tasklauncher/spring-cloud-starter-stream-sink-
tasklauncher-cloudfoundry/src/main/java/org/springframework/
cloud/stream/app/task/launcher/cloudfoundry/sink/
TaskLauncherCloudfoundrySinkConfiguration.java
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240
:

+import org.springframework.cloud.deployer.spi.cloudfoundry.CloudFoundryDeploymentProperties;
+import org.springframework.cloud.stream.annotation.EnableBinding;
+import org.springframework.cloud.stream.messaging.Sink;
+import org.springframework.cloud.task.launcher.annotation.EnableTaskLauncher;
+import org.springframework.context.annotation.Bean;

+/**

Configuration class for the TaskLauncherSink.

@author Glenn Renfro

*/
+@EnableBinding(Sink.class)
+@EnableTaskLauncher
+public class TaskLauncherCloudfoundrySinkConfiguration {

@Bean(name = "taskDeploymentProperties")

This goes along with my previous question. Why do we need to do the
"tricks" in the first place?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/spring-cloud/spring-cloud-stream-app-starters/pull/240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATIllVczthrjCQNbrm_6D7rqYeBDJ4Sks5q1jpmgaJpZM4KOBrj
.

Merged as 18f47b2


refactor demangling part in profiling.rs

…r_index

issue #7665
replace the match statement with a function already exist

Would you mind rebasing this PR? Once that's done, I'm happy to click the merge button :+1:


Use Sorl Thumbnail with Django Rest Framework

How can I use sorl thumbnail with DRF? I follow instruction Low API: http://sorl-thumbnail.readthedocs.io/en/latest/examples.html#low-level-api-examples and get error: my_file is not defined in model. Please help me.

if obj.photo:     return get_thumbnail(obj.photo, '30x30', crop='center', quality=99).url
and photo is ImageField

Thank you. Hope Sorl Thumbnail support for DRF in the future :)

Can you put it in the documentation?

FYI, I've found useful package for DRF: https://github.com/dessibelle/sorl-thumbnail-serializer-field

@kidig Can you put it at documentation?

@maxim25
https://github.com/dessibelle/sorl-thumbnail-serializer-field/blob/master/sorl_thumbnail_serializer/fields.py
There is the whole codebase. Nothing about any extra database queries. Please, check your querysets.


Adding CODEOWNERS to the Repo

Codeowners are auto added to PR and one of these needs to approve before a merge.
https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners

This was already approved in discord by all the ones mentioned


(PDB-1791) Upgrade to version 5 of the PuppetDB module

This commit upgrades PuppetDB to use v5 of the PuppetDB module (latest
at the time of this commit). This consists of using the new globals
class to specify the PuppetDB version.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1664/

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1665/


[BUG]: Archive ZIP and TAR.GZ Xoops implementation loop bug

Which module?
altsys
Description of the bug
A Tar file is an archive that consists of multiple files put into one, while GZ is a compressed file format.
Thus, combining TAR and GZ into a TAR. GZ provides you with a compressed archive.
It's a really useful feature of Template management. However, all versions of Xoops
have a third-party library dependency with a terrible ( loop bug and fatal crash).
Modules concerned by the bug
Altsys, LegacyRender and any other module relying on archive/tar class
Implementation

Select a set of templates to replace.
Select the tar.gz file containing the templates (.html)
Automatically extract all templates regardless of directory tree structure.

Steps To Reproduce

Go to admin components
Select Advanced Templates
Upload tar.gz or zip to overwrite template set
Loop Bug
Error 500 (delete corrupted template set)

Additional Information
No response

🚩 Input type file with accept=extension is not reliable ( browser bug)
https://html.spec.whatwg.org/multipage/input.html#file-upload-state-type-file
✔ XOOPSCube ActionForm is a reliable solution for File Extension/Type Validation
public $_allowExtensions = ['tar'];


Migrate examples to junit5

migrate examples from junit4 to junit5
#6921

hi @eddumelendez, examples are migrated to junit5. Could you please review the PR, thanks!

Using JUnit 5, public can be removed from test classes and methods. Please apply this change.
Thanks!

It has been removed, thanks.
The ci job for tubo-mode is failing with timeout, is there any solution for that?

Thanks for your contribution @samed-bicer !


Uncaught PDOException: SQLSTATE[HY093] / Quoted elcosure issue

Hi,
It seems that the makeColumns function doesn't work in my case, my first column keep his double quote even if I specify the enclosure option :
this line (from Csv class / makColumns)
$columns = array_flip(str_getcsv(fgets($handle), $this->delimiter, $this->enclosure));
return
array(3) {   [""FOO""]=>   int(0)   ["BAR"]=>   int(1)   ["FUZ"]=>   int(2) }
So it trigger a PDOException: SQLSTATE[HY093]

Okay my bad, my first line contain some weird invisible chars at the begining, so the str_getcsv fail to parse it.
Sorry, I will close this issue :)


Have to tap 3 time on Android to zoom

Hi, great plugin !
I think there is an issue on tap to zoom on Android.
I have to tap 3 times to actually zoom the image.
Any idea ?
Thanks

Indeed I reproduced the problem and it should now be fixed with the latest 1.2.1 release 👍
If after updating the problem persists feel free to comment on this issue.
Thanks again


BIGTOP-3506. Update hadoop-httpfs resources based on hadoop-3.

https://issues.apache.org/jira/browse/BIGTOP-3506
Resrouces for hadoop-httpfs such as init scripts, service configuration and puppet manifests should be updated based on hadoop-3 same as done for hadoop-kms in BIGTOP-3501 and BIGTOP-3505.

I tested this by following command on CentOS 8. --smoke-tests httpfs seems not to work since there is no resouces under bigtop-tests/smoke-tests/https while there is code under bigtop-tests/test-artifacts/httpfs.
$ ./docker-hadoop.sh \
   --create 1 \
   --image bigtop/puppet:trunk-centos-8 \
   --memory 16g \
   --repo file:///bigtop-home/output \
   --disable-gpg-check \
   --stack hdfs,httpfs

$ ./docker-hadoop.sh --exec 1 curl -i GET "http://localhost:14000/webhdfs/v1/?user.name=root&op=LISTSTATUS"
curl: (6) Could not resolve host: GET
HTTP/1.1 200 OK
Date: Mon, 22 Feb 2021 10:06:46 GMT
Cache-Control: no-cache
Expires: Mon, 22 Feb 2021 10:06:46 GMT
Date: Mon, 22 Feb 2021 10:06:46 GMT
Pragma: no-cache
Content-Type: application/json
X-Content-Type-Options: nosniff
X-XSS-Protection: 1; mode=block
Set-Cookie: hadoop.auth="u=root&p=root&t=simple-dt&e=1614024406086&s=zDyncElMPPUnIvcqbuahEUuNfKl0etD2EIQlrGfJ2h8="; Path=/; HttpOnly
Transfer-Encoding: chunked

{"FileStatuses":{"FileStatus":[{"pathSuffix":"apps","type":"DIRECTORY","length":0,"owner":"hdfs","group":"hadoop","permission":"755","accessTime":0,"modificationTime":1613987721791,"blockSize":0,"replication":0,"childrenNum":1,"fileId":16415,"storagePolicy":0},{"pathSuffix":"benchmarks","type":"DIRECTORY","length":0,"owner":"hdfs","group":"hadoop","permission":"777","accessTime":0,"modificationTime":1613987721651,"blockSize":0,"replication":0,"childrenNum":0,"fileId":16394,"storagePolicy":0},{"pathSuffix":"hbase","type":"DIRECTORY","length":0,"owner":"hbase","group":"hbase","permission":"755","accessTime":0,"modificationTime":1613987721642,"blockSize":0,"replication":0,"childrenNum":0,"fileId":16392,"storagePolicy":0},{"pathSuffix":"solr","type":"DIRECTORY","length":0,"owner":"solr","group":"solr","permission":"755","accessTime":0,"modificationTime":1613987721646,"blockSize":0,"replication":0,"childrenNum":0,"fileId":16393,"storagePolicy":0},{"pathSuffix":"tmp","type":"DIRECTORY","length":0,"owner":"hdfs","group":"hadoop","permission":"1777","accessTime":0,"modificationTime":1613987721622,"blockSize":0,"replication":0,"childrenNum":1,"fileId":16386,"storagePolicy":0},{"pathSuffix":"user","type":"DIRECTORY","length":0,"owner":"hdfs","group":"hadoop","permission":"755","accessTime":0,"modificationTime":1613987920595,"blockSize":0,"replication":0,"childrenNum":15,"fileId":16395,"storagePolicy":0},{"pathSuffix":"var","type":"DIRECTORY","length":0,"owner":"hdfs","group":"hadoop","permission":"755","accessTime":0,"modificationTime":1613987721609,"blockSize":0,"replication":0,"childrenNum":1,"fileId":16387,"storagePolicy":0}]}}

I got following error on Ubuntu 18.04. I'm looking into the cause.
Error: Systemd start for hadoop-httpfs failed!
journalctl log for hadoop-httpfs:
-- Logs begin at Mon 2021-02-22 10:09:24 UTC, end at Mon 2021-02-22 10:11:13 UTC. --
Feb 22 10:11:08 65add4f9bec9 systemd[1]: Starting LSB: Hadoop httpfs...
Feb 22 10:11:08 65add4f9bec9 runuser[5252]: pam_unix(runuser:session): session opened for user httpfs by (uid=0)
Feb 22 10:11:08 65add4f9bec9 hadoop-httpfs[5231]: WARNING: httpfs.sh is deprecated, please use 'hdfs [--daemon start|status|stop] httpfs'.
Feb 22 10:11:08 65add4f9bec9 hadoop-httpfs[5231]: /usr/lib/hadoop/sbin/httpfs.sh: line 64: /bin/hdfs: No such file or directory
Feb 22 10:11:08 65add4f9bec9 runuser[5252]: pam_unix(runuser:session): session closed for user httpfs
Feb 22 10:11:13 65add4f9bec9 hadoop-httpfs[5231]:  * Failed to start Hadoop httpfs. Return value: 3
Feb 22 10:11:13 65add4f9bec9 systemd[1]: hadoop-httpfs.service: Control process exited, code=exited status=3
Feb 22 10:11:13 65add4f9bec9 systemd[1]: hadoop-httpfs.service: Failed with result 'exit-code'.
Feb 22 10:11:13 65add4f9bec9 systemd[1]: Failed to start LSB: Hadoop httpfs.

I got expected result on both CentOS 8 and Ubuntu 18.04 by the last commit.

+1, the same check also succeeded on CentOS 7 and Debian 10. Thanks @iwasakims!


Disable so that app does not break

Coverage increased (+0.03%) to 35.283% when pulling 97de5979a5adf3b55e2c7d1f7de38e78e6ea3f46 on qingweibinary:qingwei/temp_disable_selected_asset_cache into 72c9edf076c9be71fd34534359a6c124882d3361 on binary-com:master.


Create master

https://www.pinterest.co.uk/

https://www.pinterest.com/pin/537828380480333120/


fix(stdlib): correct tecurve::contains formula

Description
The tecurve::contains formula for extended coordinates should be
$$
z^2\left(ax^2 + y^2\right) = z^{\color{red}{4}} + dx^2y^2
$$
Some other changes:

Simplify equality/zero checks
Clean up boolean checks

cc @ax0
Problem*

Resolves
Summary*

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[x] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Not sure why the test is failing. The non ssa_refactor one seems to be passing

Sorry about that, I'll look into it

@TomAFrench Done. Passing now :)

@ax0 Were you happ with the changes made here?


Relative Path Breaks Script

Describe the bug
If prosToMinted.py is not run from the project's root directory, then it will cause issues with finding header and source files.
To Reproduce
Steps to reproduce the behavior:

Go to scripts/
run python3
See error

Expected behavior
The script should correctly execute from anywhere in the project directory.
Desktop (please complete the following information):

OS: Windows 10
Python 3.7.9

Additional information
Short Term Solution 🚧: always run the script from the root directory
Long Term Solution 🛠: implement python package pathlib for os specific paths & implement absolute paths. See More

Learning how to use pathlib here

Long Term Solution Implemented! Now pyToMinted uses pathlib for cross platform compatibility


Redesign Home Page

Remove everything with "בפיתוח"
add new component of "what can we done here"
add video
add text section

Specific to Desktop:
Specific to Mobile:
shrink top image

[ ] bdika
[ ]


ConvertToModel for IContent

I'm attaching to the ContentService.Published event because I need to take some action after content is published. I'd like to work with the published items using my strongly typed models. However, in the ContentService.Published event, you are given a collection of IContent and the ConvertToModel extension method expects an IPublishedContent.
I'm thinking about writing another extension method that can take an IContent and do the same thing. Before I do so however, I have two questions:

Does this code already exist somewhere and I'm just not seeing it?
If not, and I write the code, is this something you'd like to see a Pull Request for?

Thanks!

I ended up not needing this.