mvn package -Dmaven.test.skip=true works, but -DskipTests does not work

sometimes we just want to compile the UT codes, rather than to run the UT codes.
mvn package -DskipTests should achieve that but now it does not work.
Now, I can only use mvn package -Dmaven.test.skip=true. However, in this way, all the UT codes are not compiled.

Hi, I know about it now.
If you want to skip running the UT of some module  use -Dmodulename.test.skip=true
For example, I do not modify TsFile, JDBC, Cli and RPC, and  want to skip their UT. Only IoTDB module's UT is desired, then run: mvn package -Dtsfile.test.skip=true -Djdbc.test.skip=true -Dcli.test.skip=true


.env file support ?

Hi folks,
Any workaround to use .env files for this library?

Any workaround to use .env files for this library?

You'll need to provide more context.  Not sure how or when you would expect a .env file to be used.  For local testing?

Any workaround to use .env files for this library?

You'll need to provide more context. Not sure how or when you would expect a .env file to be used. For local testing?

Problem: I have many environments [local, stage, production]
It is very challenging to connect to these environments to debug locally, we end up hardcoding values, so thought of maintaining multiple .env files for different environments and load according to NODE_ENV.
I recently came through vcap file, it would be a great help if some snippet demonstrates how VCAP file should be used, please !


Add colors to labels/due date.

In todoist I have colors set for my various labels and todoist adds colors to the due date automatically. I assume that this feature wasn't included because of concerns about theme compatibility? In either case I'm confused because the checkboxes are styled with colors based on their priority.
This probably falls under the same issue as issue 169.
It seems like this would be relatively easy to fix as Todoist has this object for labels in their official API documentation.
{
id: "2156154810",
name: "Food",
color: "charcoal",
order: 1,
isFavorite: false
}

This already seems implemented for overdue dates:

so could be added fairly easily?

It would also be great if the project can also be color coded like the suggestion. Having all the same looks makes it really hard to intuitively check which task belongs to what project.

There's actually a redesign that's landed in master and will be available with the next public release. As a sneak peak, this is what that looks like with the default theme:

Right now this is implemented for projects (it styles the hash icon on the right) and adding it for labels as well won't be too hard.
Due dates can be styled with CSS, I'll make sure to add some example snippets to the documentation site with the release to make this easier. In the medium term, I was thinking about using https://github.com/mgmeyers/obsidian-style-settings to expose more built-in style settings. How does that sound?

This looks great, excited!

Hey all, this has been released as part of the latest release - v2.0.0!


Task edit page crashes

InfluxDB master
Clicked on a task name to edit and got a crash screen with this error in console:

@AlirieGray Just tried to reproduce this myself, but was unable to. Do you have the task that caused this issue?


opt: client supports requests to server-streaming endpoints without parameters

What is the purpose of the change?
https://github.com/apache/dubbo/pull/15026 支持了无参的外部调用的情况下，而直接使用dubboconsumer时，由于定死了参数数量为2，导致请求时会报错，故对consumer侧的实现进行了兼容处理
using DubboConsumer directly causes an error during the request because the number of parameters is fixed to 2. Therefore, compatibility handling has been implemented on the consumer side.
Checklist

[x] Make sure there is a GitHub_issue field for the change.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[x] Make sure gitHub actions can pass. Why the workflow is failing and how to fix it?

Codecov Report
Attention: Patch coverage is 46.15385% with 14 lines in your changes missing coverage. Please review.

Project coverage is 29.23%. Comparing base (1da23ea) to head (ff27461).

Files with missing lines
Patch %
Lines

...g/apache/dubbo/rpc/protocol/tri/TripleInvoker.java
36.36%
6 Missing and 1 partial :warning:

...he/dubbo/rpc/model/ReflectionMethodDescriptor.java
55.55%
3 Missing and 1 partial :warning:

...g/apache/dubbo/rpc/model/StubMethodDescriptor.java
0.00%
2 Missing :warning:

...bbo/rpc/protocol/tri/ReflectionPackableMethod.java
75.00%
0 Missing and 1 partial :warning:

:exclamation:  There is a different number of reports uploaded between BASE (1da23ea) and HEAD (ff27461). Click for more details.
HEAD has 1 upload less than BASE

Flag
BASE (1da23ea)
HEAD (ff27461)

integration-tests
1
0

Additional details and impacted files
@@             Coverage Diff              @@
##                3.3   #15029      +/-   ##
============================================
- Coverage     35.57%   29.23%   -6.35%
+ Complexity    10847     8876    -1971
============================================
  Files          1882     1882
  Lines         85997    86004       +7
  Branches      12881    12884       +3
============================================
- Hits          30597    25144    -5453
- Misses        50960    56886    +5926
+ Partials       4440     3974     -466

Flag
Coverage Δ

integration-tests
?

samples-tests
29.23% <46.15%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


feat(CLI): Add category and refine help text

The new help text looks like:
 envd - Development environment for data science and AI/ML teams

 Usage:
    envd up --path <path>

    envd run --name <env-name> --command "pip list"

 Build and launch envd environments. Get more information at: https://envd.tensorchord.ai/.
 To get started with using envd, check out the getting started guide: https://envd.tensorchord.ai/guide/getting-started.html.

 Advanced Commands:
  pause, p   Pause the envd environment
  resume, r  Resume the envd environment
 Basic Commands:
  build, b          Build the envd environment
  destroy, d        Destroy the envd environment
  envs, env, e      Manage envd environments
  images, image, i  Manage envd images
  run               Spawns a command installed into the environment.
  up, u             Build and run the envd environment
 Management Commands:
  bootstrap  Bootstrap the envd installation
  context    Manage envd contexts
  init, i    Initializes the current directory with the build.envd file
  prune      Clean up the build cache
 Other Commands:
  version, v  Print envd version information

 Global Options:
  --debug     enable debug output in logs (default: false)
  --help, -h  show help (default: false)

Signed-off-by: Ce Gao<EMAIL_ADDRESS>

Cool! It would be great if basic commands show up at the top instead of advanced commands. Should bootstrap be in basic commands instead if it will be used by first users?

SGTM.
It is hard to reorg the order of the commands with our CLI frameworks. I will investigate how to achieve this.


sepolia deployments

Deploying Multi giveaway and LandWeightedSANDRewardPool

Deploy files look good, not sure why CI is failing let's review when it's completed


Chat program handshake exception

I get the following with the examples/chat example client and server.js :
io.socket.SocketIOException: Error while handshaking
at io.socket.IOConnection.handshake(IOConnection.java:322)
at io.socket.IOConnection.access$600(IOConnection.java:39)
at io.socket.IOConnection$ConnectThread.run(IOConnection.java:199)
Caused by: java.io.IOException: Server returned HTTP response code: 400 for URL: http://localhost:3000/socket.io/1/
at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1838)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1439)
at io.socket.IOConnection.handshake(IOConnection.java:313)
... 2 more
Jun 28, 2015 6:56:01 PM io.socket.IOConnection cleanup
INFO: Cleanup
Any help is greatly appreciated.

Hello, I have the same problem, any news about this?

Hello Mklueh,
I never got java io.socket to work, so I redesigned my app in javascript (NODE).
Good luck!

Thank you for your fast response! I´ll try it with JavaScript too

This project only supports socket-io 0.9. If you use a newer version of socket-io on your server application, you can try https://github.com/socketio/socket.io-client-java, or downgrade the server version to 0.9.17 or lower.


Fix thunks in Byron and CertFakeVRF

With the fixes of this PR, https://github.com/input-output-hk/cardano-base/pull/126, and some local fixes in consensus, I don't get any more unexpected thunks when running the consensus tests.

Hmm, CI failed because cardano-ledger-test timed out on darwin. But it didn't time out on linux...

Hmm, CI failed because cardano-ledger-test timed out on darwin. But it didn't time out on linux...

I'll restart it


Per-PL perplexity vs. pass@k rate

For @canders1, @ytzi, and me

closing this issue as we finished the scaling laws


Misplaced 'Recent Activity' popover

Describe the bug
The 'Recent Activity' popover is incorrectly placed on mobile, which makes it impossible to read.
To Reproduce
Steps to reproduce the behavior:

On mobile, connect wallet
Tap on the 'activity' icon in the top nav.
View the popover.

Expected behavior
The popover should be positioned within the viewport.
Screenshots

Note: The account dropdown also has the same error. Please fix this too.


Release purview microsoft.purview preview/2024 04 01 preview

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.

Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

Regarding the LintDiff warning about boolean vs enum, I believe for the proposed field, a boolean should work as it is an irreversible state and only intended to be true or false. It is also scoped inside of the containing object as that is the only scenario it is intended to be set in and is for a specific use case and not a generalized use case.

      "x-ms-client-flatten": true

FWIW I think adding x-ms-flatten is sdk breaking, and not really considered 'better practice' these days. I don't think it was really worth it just to fix linter warnings, in case that is the only reason its being added

Refers to: specification/purview/resource-manager/Microsoft.Purview/preview/2024-04-01-preview/purview.json:2701 in ecf736f. [](commit_id = ecf736f5936839b8ca4c48d5541a92c12a35954a, deletion_comment = False)

Adding a new optional property to an API contract is not breaking; however, it still requires a new service api-version so customers can know what api-version supports the new property.

Please add a workitem to refactor the private links related definitions.

@rkmanda here is the task to update to common-types for Private Links: Task 3198663: Update Private Links/Connections/Endpoints to use Common Types for GA.
Would you be able to remove the Change Request from Tim to unblock merging or was that intentional?

/pr RequestMerge


drivers: watchdog: wdt_sam: use the generic disable option

Use the generic option WDT_DISABLE_AT_BOOT instead of the SAM specific
one WDT_SAM_DISABLE_AT_BOOT (note the generic one has been introduced
after the SAM one).
This also have the consequence of changing the default value for yes to
no, fixing the watchdog tests.
Fixes #13290
Signed-off-by: Aurelien Jarno<EMAIL_ADDRESS>

@cinlyooi-intel That should fix the issue you reported in #13290

Codecov Report

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

@@           Coverage Diff           @@
##           master   #14210   +/-   ##
=======================================
  Coverage   52.68%   52.68%
=======================================
  Files         307      307
  Lines       45473    45473
  Branches    10530    10530
=======================================
  Hits        23956    23956
  Misses      16640    16640
  Partials     4877     4877

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 d7ae2a8...97d3369. Read the comment docs.


Investigate Symmantic versioning

Symmantic versioning for the greenfield and the actions. Probably more important for the actions.

Long since solved.


Style :confval:

Resolved by https://github.com/TYPO3-Documentation/t3SphinxThemeRtd/commit/497e09907ff4c17b3ddb1f6a06ae23057a360312
Example 1:

Example 2

Example 3

Example 4


What does $select.search refer to?

I see it all over the demos, and nowhere in the documentation.

It's the text that's been typed into the into the input area. I'm using it on ui-select-choices like this
data-refresh="getChoices($select.search)"
getChoices is my custom method to update the choices list.

Ah -- makes sense. Thanks.


Add AMD-SD dataset

This PR adds a new OCT dataset for multi-class lesion segmentation. GTG from my side!

I'll go ahead and merge this for my experiments. Let me know if you spot something unusual @constantinpape


GPII-3624: Add FAQ and extra debugging for couchdb split brain '1 of 2 pods have…

… joined the cluster' problem.

looks good

LGTM


Tab shortcut interference

Version
4.1.3
Browser and OS info
Chrome 64 / OSX 10.12.6
Steps to reproduce

Open multiple browser tabs and in one of those tabs engage Vue Dev Tools.
Try using the shortcut for switching browser tabs (⌘   1 through ⌘   8)

What is expected?
I should be able to use the standard browser shortcut to switch browser tabs
What is actually happening?
Vue dev tools uses this shortcut (⌘   1 through ⌘   8) and prevents me from switching browser tabs

If you are not focusing the devtools, you should be able to switch to another tab (tested in Firefox and Chrome on MacOS).


Remove axisN keyword arguments from linalg.diag and linalg.trace

This PR

removes the axis1 and axis2 keyword arguments from linalg.diag and linalg.trace.
updates the API to perform batching over the last two dimensions.
supersedes gh-222.

Background
linalg.diag and linalg.trace, as currently specified, are unique in their parameterization. Namely, for APIs which allow specification of axes, common convention is to use axis or axes. Furthermore, prior art (NumPy) used 0 and 1 as the default axes, which runs counter to the current prevalent practice of batching along the last (innermost) two dimensions.
gh-215 proposed changing the defaults to -2 and -1, respectively. Subsequent discussion (both in gh-215 and in consortium meeting 2021-07-29) contended that they should be removed altogether.
This PR follows consensus found in the consortium meeting to simply remove the axisN keyword arguments. If future needs arise to specify axes in the future, such additions can be made to the specification in a future revision.

Thanks, @kgryte!


Request payload breaks when passing an integer with a leading zero

Laravel Version: 5.5.40
PHP Version: 7.1.0
Database Driver & Version: MySql

Description:
I have an endpoint who needs a payload data for a PUT method. One of the fields must be integer type.
My client sometimes pass a integer value starting with 0. When this happens, all the payload breaks. I can't even handle the value because it gets in controller already NULL.
One example of breaking payload:
{"status: "waiting", "code": 09324}
To make a diagnosis of the problem I followed the request trace and stopped in Symfony Request file (Symfony\Component\HttpFoundation\Request.php). On function "get" the $key passes by all verifications and return the default value (which is null for me)
public function get($key, $default = null)
    {
        if ($this !== $result = $this->attributes->get($key, $this)) {
            return $result;
        }

        if ($this !== $result = $this->query->get($key, $this)) {
            return $result;
        }

        if ($this !== $result = $this->request->get($key, $this)) {
            return $result;
        }

        return $default;
    }

Maybe the library is trying to understand an octal value instead of integer? I can't change my business rule (for example change from integer to a string on payload). What's the best way to fix this situation?
Steps To Reproduce:
Make a request for a PUT endpoint with a integer field in body.
Pass a value starting with 0 (example: 0255)
Try to receive this value on controller with the Request library (vendor/symfony/http-foundation/Request.php)

Octal values in JSON are invalid: https://stackoverflow.com/questions/27361565/why-is-json-invalid-if-an-integer-begins-with-0
In any case you can use the json method or input method to retrieve JSON values. See "Retrieving JSON Input Values" here: https://laravel.com/docs/5.7/requests#retrieving-input


DOC: Updating documentation relating to Models->llms->integrations->*

Issue with current documentation:
The page I am referring with this issue is Llama.cpp. I am hoping to update the documentation regarding some windows specific instructions which others might find useful.
I've forked the repo and my working changes I would push there and make a pull request here.
I understand that to do this; langchain/docs/_dist/docs_skeleton/docs/modules/model_io/models/llms/integrations/llamacpp.ipynb needs to be updated.
It also appears that docs/_dist is now ignored by .gitignore.
Could you please let me know if I am looking in the wrong place? If so please correct me.
Idea or request for content:
No response

could you check to see if the page is updated now? there was a sep bug a week ago

It is not updated yet.
Did you check the pull request? It seems like the vercel deployment is broken.

@dev2049 If you can grant me write access, I would be able to clean up the documentation around many model integrations. This is why pull request is not passing.


Exception in signOut

Hitting this in the Auth.signOut() call:
TypeError: Cannot read property 'clearCachedId' of null
I've actually wrapped the call in a try-catch, as it swallows the exception and does nothing otherwise.
Must be an aws-amplify issue, however.

@aloukissas Oh. Do you have some steps to reproduce this?

Following the code in https://serverless-stack.com but using phone instead of email for username.


Remove training type from training script

Expected Behavior
npm run train

# With language option
npm run train en

Actual Behavior
To train expressions, we need to run the following script:
npm run train expressions

# With language option
npm run train expressions:en

Proposal
Currently to train expressions, we need to pass a "type" argument. At the first place, I did it to allow other types of training in the future. However, it is not necessary to have one. So to make the code a bit simpler the type argument should be removed.
Also, the CLI is coming, so we can improve this part in the future via the CLI if necessary.

 🔧 [1.0.0-beta.5] Remove training type from train.js and npm scripts. Only need one here actually

Done in https://github.com/leon-ai/leon/commit/ddfebb0d7402ecbb3e00c04bb42082bd54541c15.


Test cases

This PR has test cases for lang_occurs, lang_transitive, lang_export and a fix for singleton variable warning

These tests are not failing locally

These tests are not failing locally. I have added fixme to the failing test in holds. But it is still in the build failure report. fixme in .pl and .plt files act different?

they fail locally here

These tests are not failing locally. I have added fixme to the failing test in holds. But it is still in the build failure report. fixme in .pl and .plt files act different?

I'll have a look at the unrecognized fixme case


Housekeeping/Update workflows

Closes #
Changes proposed in this pull request:

...

Also bump .release-version on merge since release version is now tracking on develop branch merge


App goes behind iPhone X notch in Landscape mode

I have gone through these following points

[x]   Check latest documentation: https://docs.nativebase.io/
[x]   Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[x]   Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[x]   Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[x]   For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[x]   For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description

node, npm, react-native, react and native-base version, expo version if used, xcodeversion
Node: v8.11.1
NPM: 5.6.0
"dependencies": {
"native-base": "^2.12.1",
"react": "16.8.3",
"react-native": "0.59.1"
},
Expected behaviour
Safe area should work in Portrait and Landscape modes
Actual behaviour
Safe area only works in Portrait mode.  In Landscape mode, the app partially disappears behind the notch (see screenshots below).
Steps to reproduce

Simple test repository here: https://github.com/brownieboy/nbiphonexheadertest
Is the bug present in both iOS and Android or in any one of them?
iOS and then only on devices with the notch.  Tested on iPhone X Simulator.  (Don't have real hardware to test.)
Any other additional info which would help us debug the issue quicker.

If I wrap the <Container> in a react-native <SafeAreaView>, it fixes the Landscape mode issue, but it messes up the <Header> height in Portrait mode, see #2440.

Commit https://github.com/GeekyAnts/NativeBase/commit/167e2d13ebed09905c1c1eed69722bd4d255f80c

I encountered this issue in the context of #2947, where you can't use this fix without getting a warning about nested VirtualizedLists. My workaround is described here: https://github.com/GeekyAnts/NativeBase/issues/2947#issuecomment-603953760


Why not call write directly?

https://github.com/havenwood/roda-sse/blob/a7f8b531a6a065d7e5f12d3f52f4a667a24df247/lib/roda/plugins/sse.rb#L31

I was originally calling write directly and handling closing the stream at the Roda app level. As a feature, I wanted to have r.sse do |stream| blocks ensure stream.close but that requires either having the plugin author ensure their block returns stream, which seemed onerous, or some way to capture the stream right before it's called.
Roda doesn't really give Rack 3 stream access, and I couldn't find a way to reliably get to the connection from Roda to close it manually in the plugin. I borrowed the Stream abstraction from the Roda streaming plugin, but felt it could handle both intercepting the stream and wrapping the write with Async.
First thought I'd do Async stuff in app land, but found I needed a Barrier to loop (whether it's actually needed or a skill issue on my side, I'm not sure). I encountered a deluge of Tasks that weren't closing on disconnection errors. My thinking in adding the barrier was to ensure that the current Task and all other pending ones for the stream are closed. With the barrier, I just see the single error percolate up.
I very much appreciate you taking a look. Any feedback or correction is super welcome. Thank you!

P.S.  I added an example/ app that streams like crazy to reproduce the orphaned Tasks issue I was trying to address with Async::Barrier.


Fix a typo that combined 2 list items.

Copy-edit, not a content change.

@bendnorman another languishing minor change.


/multiworld command doesn't work anymore

Using the /multiworld command fails with an error message: AttributeError("'coroutine' object has no attribute 'id'")

The command works again.  There is a bug that's causing the "Join" and "Leave" buttons to show an "Interaction failed", but they buttons do work.  This is fixed in a coming update.


Python transaction binding not allowing "allow_any_incoming_length" argument

When setting the allow_any_incoming_length argument to True on the libcsp.transaction method it returns the error "function takes exactly 6 arguments (7 given)"

I tested the python binding example few days ago with the current develop branch(https://github.com/libcsp/libcsp/commit/f21c1aa923303c02b9f788c7b6286bc8219c37cb) and the error is not present.
I'm not sure yet for the libcsp-2-0 because with this branch build with python binding fail. (I will investigate more on libcp-2-0 branch)

I'm closing this.  @skoold2003 if you still have problems, please reopen it.


RFC: Online gameplay

Actors
Game client
An instance of MKW-SP running in client mode, which is the default. Can connect to several types of servers.
Game server
An instance of MKW-SP running in server mode, which can be enabled via a configuration file or in game. Can host a single game and connect to up to 12 game clients.
Matchmaking server
An instance of an external program that can receive requests from game clients and send them the address of a game server.
Master server
A unique instance of an external program that handles authentication of game clients.
Operation
Registering a new license

A game client sends a register command to the master server with its device certificate.
The master server checks that the certificate is valid and rate limits, then sends a challenge to the game client.
The game client resolves the challenge and sends the result to the master server.
The master server assigns a unique identifier to the license and sends it.

Creating an open room or a friend room

A game client sends a create command to a matchmaking server with its unique identifier and track pack.
The matchmaking server sends the address of an available game server if there is one.

Joining an open room

A game client sends a search command to a matchmaking server with its unique identifier and track pack.
The matchmaking server sends the address and settings of the game servers of ongoing rooms if available.

Joining a friend room

A game client sends a register command to the master server with its device certificate.
The master server checks that the certificate is valid and rate limits, then sends a challenge to the game client.
The game client resolves the challenge and sends the result to the master server.
Since the master server is aware of all connections, it can send room information for all friends.

Connecting to a game server

A game client sends a connect command to the master server with its device certificate, unique identifier and the game server's certificate.
The master server checks that everything is valid, then sends a challenge to the game client.
The game client resolves the challenge and sends the result to the master server.
The master server assigns 2 unique tokens and sends them to the game client.
The game client sends the first token to the game server.
The game server sends its certificate and the token to the master server.
The master server checks that everything is valid, then sends a challenge to the game server.
The game server resolves the challenge and sends the result to the master server.
The master server sends the second token to the game server.
The game server forwards it to the game client.
The game server sends the state of the room to the game client (lobby or race and metadata).

Lobby commands
Most are sent by a game client to the game server, then the game server sends it to all game clients.

Join the room (only sent by the game server).
Quit the room.
Change the host (on open rooms this is done automatically if the host quits).
Change a setting (open rooms on launch only).
Kick a player (friend rooms only).
Send a message (friend rooms only).
Start the race (done automatically on open rooms).

Race start

Game clients have a fixed amount of time to send character, vehicle, drift mode and course vote.
Then the game server sends the chosen course to the game clients.
The game server knows that the game clients are ready when receiving the first race packet.
When all game clients are ready the game server sends a last command with the frame number of the race start (different for each game client).

Race packets

The packets sent from game clients to the game server contain only the frame number and the inputs for that frame.
The packets sent from the game server to game clients contain the corresponding frame number and the state of players, items and objects for that frame.
Packets are sent over UDP and are not retransmitted. When a packet is missing the game server can assume that it was identical to the previous one.
The TCP connection is kept for data that must be reliable such as lap splits.

Synchronous model
With this model, when sending an input frame the game clients wait until they receive the corresponding game state from the game server. This means that the clients don't do any game state computation and only render what they receive from the server. This is simpler to implement but leads to very long input latency.
Asynchronous model
With this model game clients keep their own state and are always ahead of the game server. For local players they can simply read inputs while for remote players they have to guess (items and objects should be deterministic and only depend on what the players do). The game clients always receive state from the server that corresponds to a past frame. If it doesn't correspond to their prediction they have to retroactively adjust the current state to it. That can be very complex to implement but should give a low latency and mostly lag free experience because most of the time the predictions are correct.
Race end
This can happen implicitly for the game client when all finish times have been received.
Security
Game clients against game clients
The information sent by the game server is minimal, but not all of it is easily available with an unmodified games (such as player and item positions). That means that a modified client may gain an unfair advantage by actually making use of that information. A controversial but effective workaround is to give that information to everyone.
Game clients against game servers
A malicious server may send invalid game state to clients, which means than strong validation is needed. The connection process is also designed to prevent servers to collect the client identifiers and impersonate them, although that might be doable in fewer steps.
Matchmaking servers against game clients
The matchmaking servers do not authenticate the clients, but that's fine because they do not do any kind of reservation: they will still advertise a game server until one (or multiple for existing rooms) client actually connects to it.
Communications that need both confidentiality and authenticity
Anything involving the master server, game state updates from game server to game client.
Communications that only need authenticity
Matchmaking server requests and responses, lobby commands, game client inputs.

This should be closed in favour of the document and current information available on the discord.

I disagree, I think this should be kept up to date instead.


Rewrite Entity API

Bonus: Fix a ClassCastException in Quadtree.

If anything is blaringly wrong, just tell me and I'll fix it up.

I think the issues are fixed.. I'd love a proofread to make sure.

danke

bitte


Add "Enable Tracking" option

Hello,
To be GDPR compliant, you would need to load the Braze SDK only after it has been consented. Take the following user flow:

user goes to the page
Braze is not loaded as per GDPR
user accepts cookies
Braze is loaded and Braze.initialize() is called through this GTM tag
user then refuses cookies
Braze.disableSDK() is called through the GTM action tag
user reloads the page
Braze is not loaded as per GDPR
user then re-accepts cookies
Braze.initialize() is called though this GTM tag but the user still has the "disable cookie" present, so Braze skips all activity

In this case, we would need to call enableSDK after Braze has been loaded, but before Braze.initialize()
You can probably workaround this by duplicating the initialization tag and user tag sequencing + the GTM action tag with enable SDK but that would open two sessions in a row, or maybe not, I don't know Braze internals and I don't really want to know.
I basically modified the template with
  if (data.disableTracking) {
    const disableSdkName = majorVersion >= 4 ? 'disableSDK' : 'stopWebTracking';
    callInWindow(sdkObject + '.' + disableSdkName);
  } else {
    if (data.enableTracking) {
      const enableSdkName = majorVersion >= 4 ? 'enableSDK' : 'resumeWebTracking';
      callInWindow(sdkObject + '.' + enableSdkName);
    }

    // Initialize the SDK with api key and base url (from user input)
    callInWindow(sdkObject + '.initialize', data.apiKey, options);

This is not really an issue for 99% of the users, but for non technical people of a product team this can get confusing that it doesn't work

Hi @Minishlink, calling the disableSDK method on each page would unfortunately have some performance impact.
As an alternative, you can sequence the Action Tag set to Enable SDK prior to initialization if the user has opted in.
If the user has opted out, then you can trigger the Action Tag set to Disable SDK and don't initialize the SDK.
Let me know if you see any issues with this approach.

Hello, we don't call disableSDK method on each page
By default we don't call the GTM initialization tag. So Braze is not loaded and not initialized.
If the user consents, the tag is called.
If the user refuses, we call disableSDK when he refuses - if he had not previously consented, nothing happens because Braze is not loaded; if he had previously consented, disableSDK disables the SDK and set the opt out cookie.
Now the problem is in the flow that I explained earlier:

user accepts
user refuses
now when the user loads a fresh page: the Braze SDK is not loaded and he has an opt out cookie
if the user accepts again the cookies, then we have to load the Braze SDK, call enableSDK to clear the opt out cookie, followed by Braze.initialize

However, this is not possible with the current tag. The only possible way is to call the initialization tag (which loads Braze and call Braze.initialize()), and then call enableSDK, and then call again the initialization tag (because you need to call Braze.initialize() after enableSDK). Unfortunately, GTM is already not a very maintainable piece of software, so there's no way I'm adding this complicated logic (which might work in theory but in practice I'm quite sure it would take more time than needed for this problem)
So the best approach is to modify this template and add an enableSDK option (that could be called only if the SDK is disabled)

I believe you can accomplish the same thing using Tag Sequencing. For example, set the "Resume Tracking" action as a prerequisite for running the Initialization tag:

Please read my messages carefully, all the information is there.
You can't call resume tracking before initialization, because you need Braze loaded and that's through the initialization tag.
The simplest and most efficient way to circumvent this is to add the enable tracking option that calls enable SDK between loading and initialization
If you still don't see the problem, I think the best course of action would be for you to do the process of creating a blank app with Braze while respecting GDPR (this should have been done from the beginning), so with the following constraints:

Braze is not loaded by default (GDPR is optin not optout)
It is initialized and enabled on user consent
It can be disabled after being enabled and initialized
It can be enabled again and initialized after an optin followed by optout and a page refresh

Thanks @Minishlink I understand now. I'll check with our team and let you know what we can do.

Hi @Minishlink - thanks for creating this issue. The new "Re-enable Tracking" initialization option has been added. It should be available to upgrade once Google picks it up from github.
https://www.braze.com/docs/developer_guide/platform_integration_guides/web/google_tag_manager/#upgrading


Rework the basic content in the website

There are a whole bunch of content that is currently not needed for the ChuckNorrisJoke website. Remove some of them without causing bugs to the website.

I thought it's for a website but not really, my apologies.


Remove old unused fields from entity [OFBIZ-12858]

Removed unused old fields (deprecated) exist.

oldPickStartDate
oldMaritalStatus
oldSquareFootage
oldInvoiceSequenceEnumId
oldOrderSequenceEnumId
oldQuoteSequenceEnumId

Hi @pushprajsinghjadoun,
If we remove oldMaritalStatus, we can also remove UpgradeServices.java in party and its reference in services_upgrade.xml (in party)
Did you intentionnaly keep oldInvoiceSequenceEnumId in InvoiceServicesScript.groovy?
Same question for oldOrderSequenceEnumId and oldQuoteSequenceEnumId in OrderServicesScript.groovy.
Are you aware of https://cwiki.apache.org/confluence/display/OFBIZ/Revisions+Requiring+Data+Migration+-+upgrade+ofbiz ?
TIA

Hi @JacquesLeRoux ,
Thanks for sharing the suggestion, I will review it again and remove the unused field references.

Hi @JacquesLeRoux,
I have made the changes as per your suggestion. Can you please review them again?

Hi @pushprajsinghjadoun ,
Sounds good to me, just that you missed to remove oldQuoteSequenceEnumId in OrderServicesScript.groovy.
Also please think about what to do, if necessary, in https://cwiki.apache.org/confluence/display/OFBIZ/Revisions+Requiring+Data+Migration+-+upgrade+ofbiz. I did not, too lazy (too hot here) :)

Hi @JacquesLeRoux ,
I apologize for not removing the oldQuoteSequenceEnumId field from the QuoteServicesScript.groovy file. I have now removed it.
Also, I am having trouble understanding what you are referring to in this document: https://cwiki.apache.org/confluence/display/OFBIZ/Revisions+Requiring+Data+Migration+-+upgrade+ofbiz.

Hi @pushprajsinghjadoun,
Don't worry about the wiki page. I checked there is nothing to do there, all was already documented there or very old (>10 years). Thanks for your work, I'll push it now.


Delete Timeouts on 2.7.4

After some inconsistent duration of time, delete operations will start to time out and the CPU usage will gradually increase.
Steps to reproduce:

Execute various delete operations on Influx
Eventually things start to time out

Expected behaviour:
Delete performance stays consistent.
Actual behaviour:
Delete requests time out in the web client.
CPU usage gradually increases.
Query operations appear to still function properly.

Since request duration metrics are written after the web request response is written, it leads me to believe the timeouts are delete requests that somehow get stuck internally in Influx and never get back to the web layer, and then consume more CPU as something internal to the system is babysitting more and more stuck requests / goroutines.
Our only workaround right now is to periodically restart influx, but this doesn't mitigate the problem entirely.
Environment info:
Ubuntu VM
Influx 2.7.4
Logs:

http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="0.025"} 105993
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="0.05"} 117921
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="0.1"} 120840
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="0.25"} 122566
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="0.5"} 122990
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="1"} 123320
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="2.5"} 123394
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="5"} 123396
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="10"} 123396
http_api_request_duration_seconds_bucket{handler="platform",method="POST",path="/api/v2/delete",response_code="204",status="2XX",user_agent="unknown",le="+Inf"} 123396

Deletion is an expensive operation in InfluxDB. If possible, it is better to have data age out through the retention policy than to deliberately delete it.  If you are deleting data explicitly, only one deletion will run at a time, so do not submit a new deletion until the previous one completes.


Remove undefined symbol

PTR is not supported by gcc since 2022-05-10 given the changelog:
Theis seems to be an old symbol used for compatibility between non ANSI-C and ANSI-C codes. I'm not sure it is still relevant to make the distinction; if the library was to be compiled with an old enough compiler that would require the "untyped pointer" to be declared as char *, there would be many more issues elsewhere in the code.

LGTM, thanks.


Use CRD based RegEx validation instead of webhook where possible

I believe this is limited to AddressType validation right now, but one of the final bits of feedback from @thockin on #1086 was that if it was possible to validate something with CRD validation, we should do that. Here's the rationale:

If something can be done with CRD validation, it should, webhook should be last resort
It seems unlikely that we'd frequently be adding additional address types
We should be able to reuse one of the regexes we're already using for domain prefixed path (GatewayController) and add a IPAddress|Hostname| prefix

/assign


[Backport 2.0] Add Abstraction for BWC tests with a basic test (#367)

Signed-off-by: Naveen Tatikonda<EMAIL_ADDRESS>Description
Add Abstraction for BWC tests with a basic test
Backport https://github.com/opensearch-project/k-NN/pull/367
Check List

[x] All tests pass
[x] Commits are signed as per the DCO using --signoff

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

Codecov Report

Merging #388 (afe3feb) into 2.0 (771d5d8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.0     #388   +/-   ##
=========================================
  Coverage     84.01%   84.01%
  Complexity      911      911
=========================================
  Files           130      130
  Lines          3879     3879
  Branches        359      359
=========================================
  Hits           3259     3259
  Misses          458      458
  Partials        162      162

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 771d5d8...afe3feb. Read the comment docs.


factorio: link to values in README

LGTM, not that it counts :)

@gtaylor All reviews count. Thanks for taking reviewing!

LGTM


FancyZone does not work for tabbed applications anymore (but it used to)

Microsoft PowerToys version
0.53.1
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Open Visual Studio
Shift + Drag tabs (not the whole window) and zones didn't pull out anymore.
Same thing happens to Unreal Editor as well.
When shift+drag on the main window:

When shift+drag on a window tab:

Older versions of FancyZone supports this feature.
I wonder if this issue relates to my settings, but I barely changed them (almost as default)

✔ Expected Behavior
Shift + Drag tabs (not the whole window) and zones should pull out.
❌ Actual Behavior
Shift + Drag tabs (not the whole window) and zones did not show up.
Other Software
Visual Studio 2019
Unreal Engine 4.27

I also have this issue on my laptop,
Intel i7-10510U
Windows 10
Version	10.0.19042 Build 19042
Visual studio 2022
Powertoys v0.53.3
I also tried replicating issue on my desktop PC (also windows 10, but Ryzen 5 3200, and same software),  but it seems to be working there. So only my laptop misbehaves.

I also have this problem.  When working in Visual Studio 2019, the child windows used to be able to use fancy zones.  Not since I upgraded last week.  I have this problem with the Find results window

Yep, same problem here
I am running PowerToys in two different desktop PCs and in both stopped working.
I think it was a couple updates ago (I can't remember exactly which one).
My current version of PowerToys is 0.53.3 and the failing VS versions are 2017 Professional and 2019 Community

/bugreport

Could you attach a full bug report file?
Right-click on the tray icon and click Report Bug. Then drag-and-drop the generated file into the GitHub comment to upload it. Thanks!

/needinfo

This is my bug report - I experience this with Visual Studio 2019 Enterprise

Yep, same problem here I am running PowerToys in two different desktop PCs and in both stopped working. I think it was a couple updates ago (I can't remember exactly which one). My current version of PowerToys is 0.53.3 and the failing VS versions are 2017 Professional and 2019 Community

I use FancyZones heavily, so I'm pretty sure there should be some changes before ver 0.53.1 that broke this feature.

/dup #15426


Image rendered too large

With contentMode UIViewContentModeCenter…
Aimv = [[OLImageView alloc] initWithImage:[OLImage<EMAIL_ADDRESS>Aimv.contentMode = UIViewContentModeCenter;

…the image is rendered much larger than if using a UIImageView.
It seems that the scale of the image is not being accounted for. Any idea how I can work around this?

I think this might be because the content is provided directly to the layer and it doesn't take into account any modes.
See: https://github.com/ondalabs/OLImageView/blob/master/OLImageView.m#L175
Sadly, I don't have any workarounds for this nor the time to look into this further in the near future.
Any workarounds that you find or a PR to fix this are welcome.


CSharpScript::callp !valid error when creating a new C# script in the editor

Godot version
4.2.beta6, master (40cdd22d570024d906c9a5795fa079342dce2197)
System information
Mageia 9 - Vulkan (Forward+) - dedicated AMD Radeon RX Vega M GL Graphics (RADV VEGAM) () - Intel(R) Core(TM) i7-8705G CPU @ 3.10GHz (8 Threads)
Issue description
In a clean new project, when I add a new C# script using the "Attach Node Script" dialog, the following error is printed each time:
ERROR: Condition "!valid" is true. Returning: Variant()
   at: callp (modules/mono/csharp_script.cpp:2612)

It also happens when loading an existing script.
Building the script then still works fine and the project can run without further error.

That error doesn't happen in 4.1-stable, so it appears to be a regression.
I tested 4.2 beta 4 to beta 6 and latest master, they all show the error. I'll test earlier builds now.
Steps to reproduce

Create new project with .NET build
Add node
Attach C# script to that node

Minimal reproduction project
n/a

Sounds like a duplicate of https://github.com/godotengine/godot/issues/83351


Fuzzy Search Issues

imo the current fuzzy search is useless because it seems (in my experience) to be so 'fuzzy' as to search for individual LETTERS.
imo, a useful proper 'fuzzy' search for match for each of the individual words: not the whole string. the issue here would be to be an OR or an AND match. i could see that BOTH would be useful.
my advice is to have comma delimiters if you want an OR search of each word, and + delimiters if you want an AND search for each word (but individually), and no separators if you want to match the whole string. that works logically with those delimiters.

yeah, I'm only using the precise search for myself.
For the fuzzy search, I've just used an existing library / algorithm. I'm not sure how much influence I have on that particular behavior, without building my own OR / AND matching based on words / separated strings myself.
But you can actually configure how fuzzy it should be. If you decrease this, does it work better for you? Maybe the defaul isn't very good. Maybe what's a good value depends on how many typos users usually make? 😁
searchFuzzyness: 0.6

See https://github.com/Fannon/search-bookmarks-history-and-tabs/blob/main/popup/js/model/options.js#L46

i wrote uFuzzy out of frustration with how fuzzy/unpredictable most fuzzy search libs are ;)

i wrote uFuzzy out of frustration with how fuzzy/unpredictable most fuzzy search libs are ;)

@leeoniya : Nice, I'll have a look into this when there is some time. Already exchanged the fuzzy library once, so why not another time if it's an improvement.

@leeoniya : I've switched out fuzzysort with uFuzzy and so far it looks like a change for the better. The library and your approach to fuzzy search is really impressive. Well done!
Here is the PR in case you want to have a look at it: https://github.com/Fannon/search-bookmarks-history-and-tabs/pull/61
@decard123 : I'll try out the fuzzy search a bit locally and when I feel confident in it and how its configured / fine tuned, I'll create a 1.7.0 release. After that it usually take a few days for the extension to become available to everyone.

thanks! let me know if you need any pointers with tweaking opts to improve match quality :)

thanks! let me know if you need any pointers with tweaking opts to improve match quality :)

Thanks for the offer! Do you have any preferences regarding the options?
By default the extension will now go with intraIns of 2 and intraMode of 0.
If fuzzyness is set to 0, then both are 0. If its set to 1 (maximum), it will be intraIns of 4 and intraMode of 1.
Is this a reasonable range and default? Thanks!

Thanks for the offer! Do you have any preferences regarding the options?

that's a tough one. i really depends on the corpus and the use case. generally i just like to avoid silly typing errors, so intraMode 1 or intraIns 1 plus out-of-order (needle permutation) cause who knows what order terms may be in!
but there are also autocomplete/typeahead cases that work well with intraIns: Infinity and intraChars: . where you just want to mash keys in sequence that can e.g. autocomplete prometheus_remote_storage_bytes by typing promremstor and not prom rem stor.
for multiple terms includes() will probably be slower than uFuzzy with default settings. but uFuzzy is case insensitive and ignores punctuation, so might not exactly fit the bill.

Thanks for the feedback!
So what I get from it:

intraMode and intraIns are probably fine the way I tuned them for now
Out of Order definately needs to be enabled, but for this to work well, it needs to be combined with caching previous results

for multiple terms includes() will probably be slower than uFuzzy with default settings. but uFuzzy is case insensitive and ignores punctuation, so might not exactly fit the bill.

I've run a few searches, but simple includes search seems still to be ~2x as fast and it already does "out of order" search. But now I also added search result caching to it, so that may have helped.


Add builder support for GDL

@adamnsch maybe that's also useful for testing your k-cut impl. See the examples on how to use it :)


Searvey Improvement

Project Description
Searvey  is a package designed for on-demand data retrieval from various sources, currently including:  - U.S. Center for Operational Oceanographic Products and Services (CO-OPS). - Flanders Marine Institute (VLIZ). - Intergovernmental Oceanographic Commission (IOC). - U.S. Geological Survey (USGS).  This project aims to enhance the capabilities of Searvey by incorporating additional observational resources and expanding the range of oceanic variables covered. The objective is to improve our capacity for evaluating numerical models by expanding the available observational data.
Expected Outcomes
1- Extended on-demand data retrieval for water levels from additional sources such as the Canadian Hydrographic Service (CHS) and the Australian Bureau of Meteorology (BOM).  2- Extended on-demand data retrieval to include more variables such as currents, water temperature, and salinity.
Skills required
Python; Libraries: Xarray
Mentor(s)
Soroosh Mani (@SorooshMani-NOAA), Atieh Alipour (@AtiehAlipour-NOAA )
Expected Project Size
175
What is the difficulty of the project?
Intermediate

Hello, my name is Flora, and I'm  Master’s degree student in physical oceanography in Brazil. I was interested in your proposal because I already worked with numerical models during my graduation, and have some questions about it.
How will the project handle data retrieval for additional variables such as currents, water temperature, and salinity? Are there specific APIs or protocols that need to be followed?
What are Searvey's weaknesses that you would like to focus on improving?

Hi Flora, thank you for expressing your interest in this project.
Searvey is developed with the aim for it to be a light package which serves as a one-stop shop for ocean data retrieval from various sources for the purpose of ocean model validation. The idea is to return the data as close to original data source as possible. We leverage existing APIs provided by data sources (e.g. for COOPS and IOC) or light packages built on top of those APIs (e.g. dataretrieval).  So we're not developing web APIs, we're developing a Python API (simple to use functions from a ocean modeler perspective) to use existing data source APIs.
The initial focus was on water level data, but we'd like to expand that to include any available variables (such as  like currents, water temperature, and salinity) for searvey to become a more comprehensive tool for model evaluations and analyses.
There is an ongoing effort to also unify the retrieval API between different sources supported by searvey which is relevant to any future development/additions: https://github.com/oceanmodeling/searvey/pull/125. This specific PR, for example, also adds the ability to query different variables from COOPS API.
I'd say there are two main goals with this project:

Unifying, simplifying and cleanup of the existing searvey APIs
Adding more sources of data (like CHS, BOM mentioned in the description)

Please feel free to reach out if you have any further questions.

Hello, my name is Abdu, I am a computer science student in the UK. I am interested in this because of its impact and importance for the environment.
I have a question about the proposal, would it be okay to use more than just CHS and BOM but also others from around the globe?

@abdu558 sure, the more sources we can cover the more useful the tool becomes

Thank you Soroosh for the quick response. I have submitted a proposal, hope you like it.
Thanks,
Abdu


Only allow SOC in Git Commit messages

We should merge this after some days.

I'll keep an eye on Gerrit and will merge once we have to a large extend only SOC issues in the backlog.

Afaik this check does not get reexecuted on merge, so having it sooner is better than later..

Seams to be a good time now to merge this.


add automated tests for the chart

Towards https://github.com/giantswarm/roadmap/issues/2008

It's finally working !

It would be good to have the values file somehow in sync with what we have in production.

In addition to what was suggested, I also disabled the loki-multi-tenant-proxy as well as the ingress to match this : https://github.com/giantswarm/shared-configs/pull/147/files

It would be good to have the values file somehow in sync with what we have in production.

The test-values I use is as close as it can be from a production loki as it's basically grizzly's chart-values tuned just enough to make it work in the ATS environment.


SimpleElastix 3D Rotation Application?

Hi,
I have a question regarding applying a user-determined 3D rotation matrix to an entire 3D image. Is there a function in SimpleElastix that allows for this?
Thanks!

Angela

Yes, you can manually change the TransformParameters variable of a transform parameter map and then apply it via Transformix.
Run an affine registration with ElastixImageFilter and use elastixImageFilter.SetParameterMap(sitk.GetDefaultParameterMap('rigid')) to get a working transform parameter map. Then you can adjust the transform parameters. The order of the parameters are explained here http://lists.bigr.nl/pipermail/elastix/2010-January/000007.html

Right now, I'm using sitk.VersorTransform() and setting the rotation to get a rotation matrix. I am not trying to register two images. Is there any way to use the rotation matrix to rotate my image without creating a parameter map? Thanks!

I am trying to rotate an image without registering it to another image. Instead, I am trying to specify a rotation matrix. Is this possible? If so, how? Thanks!

Yes that is possible. The registration outlined above is only to get a "template" of a transform parameter map that you can tweak yourself. No need to run it every time. After you have your "template", you can adjust the TransformParameters setting to your heart's content and warp as many images as your like.
The affine TransformParameters contain parameters [p1, ..., p12] of the affine transform: the first 9 are the rotation matrix in row order, the last 3 the translation. So,
T(x) = Ax + t

where t is the translation vector
[ p10 p11 p12 ]^T

and A is the rotation matrix,
[ p1 p2 p3
  p4 p5 p6
  p7 p8 p9 ]

Be default the image is rotated around the image origin. To specify another point, you can use the "CenterOfRotationPoint" found in the transform parameters file. The equation then becomes
T(x) = A ( x - c ) + t + c

where c is the center of rotation which is specified in "CenterOfRotationPoint".
Note that all parameters have to be specified as strings. For example, the identity transform is specified like this:
p[0]['TransformParamerters'] = ['1', '0', '0', '0', '1', '0', '0', '0', '1', '0', '0', '0']

Thanks, that seems to work. However, the rotated image using this method always has the same size as the original image. Is there a parameter I can change to get rid of this constraint?

You multiply the rotation matrix with a constant to change the scale of the transformed image. For example, if you want to make the image twice as large, you multiply all rotation parameters with 2.

I understand the scaling, but it seems that the choice of a fixed and moving image limits the scope of the points that can be represented in the rotated matrix. For example, if I want to translate my image by 100 voxels in the x direction, they won't show up. Is this right?

Sorry, but there is too little information to tell you what is right or wrong. You have to post working code to get such detailed answers :)
In general, the field of view is limited to that of the fixed image, so if you new points are transformed to an area outside the fixed image world field of view, they will be lost. If you want to just reorient dimensions, you can achieve this without an explicit transformation by using GetDirection() and SetDirection() on the image.

Okay, I see. I will see what I can do. Thanks Kasper :)


Match mode option: true/false

Simply print true of false depending on whether a match was possible or not. Ideally this would come with a way to customise the exact strings printed.

Now that stages can be mixed, this can be done manually:
M`regex
^0$
false
\d+
true

It might still be nice to have an option to do this more concisely though.


求添加对于时序数据库influxdb和druid和opentsdb的支持哟

求添加对于时序数据库influxdb和druid和opentsdb的支持哟
#394

We will not add influxdb for now. If you want, you can contribute this feature to MLSQL.
We will close this issue. You can reopen it or create a new one if you have any other questions.


From literals to template strings

Taken out all strings I find in html to _data/trans directory.
Replaced " to ' in language templates to improve readability

Keep in mind we can use apostrophes for words like "it's" without using quotes. Not saying anything is wrong, just keep that in mind.


Global replacement of file, directory, and path functions with boost

This is a large endeavor. Perhaps a class could be made from boost files like the ICU files.

No longer valid given that C++ has std::filesystem as part of the language.


Move connection types to the submariner.io API

We want to expose the connection status in the operator status. The
initial plan was to make the status available over HTTP, and query
that from the operator; however, the operator only updates its status
in the reconciliation loop, and apart from that the status is static.
This means that operator status updates require watching Kubernetes
objects which reflect the state encapsulated in the operator status.
To enable this, we can expose the status types in Submariner alongside
the existing Cluster and Endpoint types; to avoid package import
loops, we move everything there, and adjust the existing cable engine
and drivers accordingly.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

https://travis-ci.com/github/submariner-io/submariner/jobs/318192563#L510
why did this fail on CI?, it passes locally for me.

Ah, the return order is not deterministic, you fixed that for one of the cases, and not the other... investigating.

Added another commit on top to handle the unit test failures which happen sometimes, and cleaning up tests.

@skitt I started a follow up here to report the status: https://github.com/submariner-io/submariner/pull/502
still in WIP status, will finish it tomorrow along with the CRD definitions to test it


Radarr Setup

Is this correct for Radarr setup V3?

Yeah they changed the language around a little, I'll update the readme
Attached my settings for reference

Awesomeness! Thanks!

0738d706361ae87b3ec2986f846c518117b5fe25


python312Packages.libtmux: 0.36.0 -> 0.37.0, tmuxp: 1.46.0 -> 1.47.0

Description of changes

tmuxp: 1.46.0 -> 1.47.0
python312Packages.libtmux: 0.36.0 -> 0.37.0

Replaces: #314214

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 316971 run on x86_64-linux 1

8 packages built:

arsenal
arsenal.dist
python311Packages.libtmux
python311Packages.libtmux.dist
python312Packages.libtmux
python312Packages.libtmux.dist
tmuxp
tmuxp.dist

Darwin error from python3Packages.libtmux:
Disabled tests path "tests/legacy_api/test_test.py" does not exist. Aborting

Sorry, but I can't assist with that as I don't have access to a Darwin device.

Result of nixpkgs-review pr 316971 run on x86_64-linux 1

8 packages built:

arsenal
arsenal.dist
python311Packages.libtmux
python311Packages.libtmux.dist
python312Packages.libtmux
python312Packages.libtmux.dist
tmuxp
tmuxp.dist


docker-buildx: pass version to ldflags

With this change docker buildx version behaves as
expected. Fixes #177540.
Signed-off-by: Robin Gögge<EMAIL_ADDRESS>Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 177560 run on x86_64-linux 1

2 packages failed to build:

python310Packages.jupyter-repo2docker
python39Packages.jupyter-repo2docker

9 packages built:

charliecloud
docker (docker_20_10)
docker-buildx
docker-client
docker-gc
fn-cli
nvidia-docker
out-of-tree
pipework

Result of nixpkgs-review pr 177560 run on x86_64-linux 1

11 packages built:

charliecloud
docker (docker_20_10)
docker-buildx
docker-client
docker-gc
fn-cli
nvidia-docker
out-of-tree
pipework
python310Packages.jupyter-repo2docker
python39Packages.jupyter-repo2docker

Shell completion and version check are missing for this package.

Shell completion and version check are missing for this package.
@rgo3 I'll approve as it is. But consider adding that.


specs: VEP-2272 Complete Data Job Configuration Persistence

This change introduces VEP-2272, which aims at proposing an improvement
to Versatile Data Kit by switching the source of truth
from Kubernetes to a database.
Note: This PR will cover only the first four sections.
Signed-off-by: Miroslav Ivanov<EMAIL_ADDRESS>

I think you need to rebase your branch. A lot of stuff there :)


Add a Notification System

The Notification System will be optional. The users will have the choice when creating their profiles to either add a token or just proceed with the rest questions.
I can build one using PushBullet API.
here is a sample of one of my PR for the RaspberryPi:

I just got the chance to review PushBullet and test and I would admit that it is a great idea to implement this. Once the PR passes all tests, we would definitely merge it @GPhantom. Kudos!

Thank you so much @tanay1337, glad to hear that :)

Fixed here: https://github.com/Melissa-AI/Melissa-Core/pull/160. Thank you @GPhantom for your wonderful contribution to Melissa and welcome to the team.


User entered data needs to be sanitized to prevent malicious content.

For example, tags like 'script' should not be allowed.

Is there a spec for this? Which fields need sanitisation? What is the agreed set of "acceptable" contents for each field?


Use try_from instead of the cast crate.

try_from is stable now, so cast is unnecessary.
This closes #169

@kubkon I've converted those two crates too!
I notice there were places in the code that had the pattern
#[cfg(target_pointer_width = "32")]
let sig_id = VMSharedSignatureIndex::new(cast::u32(len));
#[cfg(target_pointer_width = "64")]
let sig_id = VMSharedSignatureIndex::new(cast::u32(len).unwrap());

I assume that this is written as such because the cast crate does conditional compilation too, and provides a different signature based on the target pointer size - Is there any problem with doing an unconditional unwrap, since the try_from trait always provides a Result?

@kubkon I've converted those two crates too!
I notice there were places in the code that had the pattern
#[cfg(target_pointer_width = "32")]
let sig_id = VMSharedSignatureIndex::new(cast::u32(len));
#[cfg(target_pointer_width = "64")]
let sig_id = VMSharedSignatureIndex::new(cast::u32(len).unwrap());

I assume that this is written as such because the cast crate does conditional compilation too, and provides a different signature based on the target pointer size

Correct!

Is there any problem with doing an unconditional unwrap, since the try_from trait always provides a Result?

None at all. To the best of my knowledge, we can safely chuck away the cfgs here and just have a single version using try_from.

Alright, then this should be ready to merge :grin: I made sure to cargo fmt too

Looks great, thanks! For the future, you can run format-all.sh to do the formatting for you, and test-all.sh to check the formatting, builds, and run some basic spec tests ;-)


Doorbell switch is exposed even when configured as false

The doorbell switch is exposed even when configured as false:
motion : true
doorbell : false
switches : true
hsv : true
Both motion and doorbell switches are exposed.
5.0.12 and before…

Hey @iuliusmaximilian
thx for reporting, i will take a look

fixed in v5.0.13-beta.1

Tested successfully - thanks!

@seydx
this same happens for motion sensor, is exposed even in config is disabled.
motion: false
in home app is exposed

@seydx this same happens for motion sensor, is exposed even in config is disabled.
motion: false
in home app is exposed

also fixed in the current beta

In b.2 not fixed

In b.2 not fixed

Hm sure? It works for me. Do you have "motion": false in your config.json? If "motion" is not set in config most likely homebridge-config-ui-x has set it to true because it is set in config.schema.json as "default": true

@grzegorz914
Ah ok sry i missunderstood. I thought you mean the motion "switch"
Yes, that's right because you have HSV enabled and HSV will create its own motion sensor if not given by the camera

OK, understood now.

Please look all on prebuffer length, alway after open config is set to sec x 1000

Please look also on prebuffer length, alway after open config is set to sec x 1000

oh ok, thx for reporting

fixed in beta 3


This example does not work out of the box for v4 of azure Tools

[Enter feedback here]

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

ID: 62965f3a-d852-cbe8-3d01-a5126a16c660
Version Independent ID: 2af2bfc6-bf21-08c6-a874-467781a4af90
Content: Connect Azure Functions to Azure Cosmos DB using Visual Studio Code
Content Source: articles/azure-functions/functions-add-output-binding-cosmos-db-vs-code.md
Service: azure-functions
GitHub Login: @ThomasWeiss
Microsoft Alias: thweiss

@SiyabongaVulindlelaBlessingMngomezulu Thanks for your feedback. Can you please elaborate more about the issue/error you faced and language (C# or JavaScript)? That would help me in investigating the issue and update as appropriate.

@SiyabongaVulindlelaBlessingMngomezulu Just a follow up on my previous comment, to see if you can share additional details on the issue you faced. I am looking to your response.

@SiyabongaVulindlelaBlessingMngomezulu Sorry I didn't get the info needed to investigate the issue. I am now proceeding to close the request. Feel free to reopen anytime with the info and will gladly assist you.
Thanks again for reaching out to us.

#please-close


Unexpected token 'export' when used in NextJs

When I import Themes inside NextJs application from themes directory('react-syntax-highlighter/dist/cjs/styles/hljs') as:
import { atomOneDark, atomOneLight } from "react-syntax-highlighter/dist/cjs/styles/hljs";

It throws a syntax ERROR as
Unexpected token 'export'

I looked for some solution from Internet, at tried dynamic import as:
import dynamic from 'next/dynamic'

const { atomOneDark, atomOneLight } = dynamic(
  () => import('react-syntax-highlighter/dist/cjs/styles/hljs'),
  { ssr: false }
)

And,
import dynamic from 'next/dynamic'

const atomOneDark = dynamic(
  () => import('react-syntax-highlighter/dist/cjs/styles/hljs/atom-one-dark'),
  { ssr: false }
)

Still not solved.
Then I tried to simpley import individual theme directly from theme file, fortunately it works for me;
import atomOneDark from "react-syntax-highlighter/dist/cjs/styles/hljs/atom-one-dark";
import atomOneLight from "react-syntax-highlighter/dist/cjs/styles/hljs/atom-one-light";

Please have a look at it.

Please raise this issue against react-syntax-highlighter, that's not our package.

Okay, my bad. Thanks Sent from Mail for Windows From: Josh GoebelSent: 26 December 2021 16:08To: highlightjs/highlight.jsCc: Azhar Zaman; AuthorSubject: Re: [highlightjs/highlight.js] Unexpected token 'export' when used in NextJs (Issue #3435) Please raise this issue against react-syntax-highlighter, that's not our package.—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


youtube-dl: 2020.07.28 -> 2020.09.06

https://github.com/ytdl-org/youtube-dl/releases/tag/2020.09.06

Motivation for this change
Things done

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

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

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

Was able to build it. Works, able to download youtube videos.

Thanks! Ported to stable as b0976a302c7a495f0ea8e0bc1bf511fc4154e9b9.


Unsure how to fix "cannot find type Long in module super" in my code

I have a piece of code where I try to familiarize myself with graphql-client (and rust) and I end up hitting following compile error: https://github.com/braincow/hslstop-rt/issues/1
My suspicion is that I have something funky in the schema, but if someone with more experience could help me out with this?

I think you probably have a Long custom scalar defined in your schema. If Long is something like a float (for example) you can fix it by defining it (something like type Long = f64; next to your query struct - the thing you #[derive(GraphQLQuery)] on).
I just realized that we don't have a fix for this if you are using the CLI, I'll open a bug issue.

Similar example in the tests: https://github.com/graphql-rust/graphql-client/blob/master/graphql_client/tests/custom_scalars.rs

@tomhoule I used the cli to dump the schema. I was thinking along those same lines that it must be custom scalar or something else with the schema, but was unsure on how to fix it.
Code now compiles. Thank you. Closing this issue.

Is there some way we can make this a little less black magic / more discoverable?

I'll open an issue for it. I don't have a fundamentally better solution in mind because in the codegen code we don't know where scalars could be defined, but what we can do is:

Make where the scalars live configurable (needed for the CLI).
More importantly, document it better. I've been wanting to have a proper book for a long time, and it should definitely have a section for common type errors and how to debug them.

We could also think of a compeletely different system where the required scalars would have to be passed in a different way (through derive attributes for example) but it seems even less ergonomic to me.


Fixed typo in passing-data-deeply-with-context.md

Typo fix: "far removed" to " far above"

It might be below. So removed covers all directions.
Since these are all subjective nits that don't seem to alter or clarify the original meaning, I'm going to close. But thanks for your suggestions.

@eps1lon Oh I see, where can I help?
Is it okay to add new challenges where there are few challenges or non? Or only typos, error fixes are approved by external contributors.


Use GitHub actions

Coverage increased (+0.0007%) to 99.858% when pulling c8a24778858fb29fef16e3f34c02967e71ea947e on github-actions into 9f64846c4950e8482e0e92a7fec4c60f8cb99ed2 on master.


release: 0.13.1

This is the release PR for 0.13.1.
If I don't hear otherwise, I will release this tomorrow.
Proposed release text:

This release is a set of quality-of-life improvements and bugfixes for PyO3 0.13.0. All new functionality adds flexibility to pyo3 in rarer cases, such as fixing #[pyclass(dict)] support for the abi3 feature (on compatible Python versions). In addition, a long-overdue initiative to reconcile PyO3's ffi definitions with the current state of the upstream Python API has started. This resulted in a number of new deprecations where mismatches were found.
For full details of all changes, see the CHANGELOG.
Thank you to everyone who contributed code, documentation fixes, design ideas, bug reports, and feedback. The following users' commits are included in this release:
@cecini
@dalcde
@daniil-konovalenko
@davidhewitt
@kngwyu
@nw0

How about including #1371?
It may slow the release for one or two days, though.

How about including #1371?
It may slow the release for one or two days, though.

Ok 👍

Ok 👍

I'd also like to include #1372, as that affects the release notes.

I'd also like to include #1372, as that affects the release notes.

Rebasing and releasing shortly 🟢

Rebasing and releasing shortly 🟢


"invalid expresson" in Internet Explorer when attribute contains tab characters

Version
2.4.2
Reproduction link
https://codepen.io/anon/pen/awegqr?editors=1111
Steps to reproduce
when using a component which contains an object property, and splitting the object across multiple lines in a Pug template, Internet Explorer fails to compile the template with the error message "invalid expression".
while experimenting trying to fix this issue i found that removing the tab characters will cause the component to successfully render.  it's probably due to html-encoded characters (tabs, among others) which can't be or aren't decoded before eval.
before (does not render in IE): https://codepen.io/anon/pen/awegqr?editors=1111
after (renders in IE): https://codepen.io/anon/pen/VWoJXv?editors=1111
both examples correctly render in Chrome,Firefox
What is expected?
bound prop containing object is evaluated
What is actually happening?
in IE, string cannot be evaluated as it contains tab characters (other characters may have a similar effect)

I don't see how we could do anything about this, since we can't access the attribute's value before IE itself.

if i do not instantiate the vue app and inspect the element in IE:

el.attributes['v-bind:settings']

this is the result:

{						component: 'alpha',						worksInIE: false				}

(i don't know how to display this in markdown, but all the tab characters are still there, perfectly preserved)
if i then later manually instantiate the app, i get the error "invalid expression".  meanwhile, in Firefox, all the above is exactly the same except that the component successfully renders.

Unfortunately, there is nothing we can do as @LinusBorg said. The error is unrelated to Vue and happens before Vue even gets to parse the template

that's strange since
el.attributes['v-bind:settings']
returns exactly the same string in Firefox and IE.  are you saying that Vue uses a different method to access attribute values?

I'm saying that the error is unrelated to Vue as it happens before Vue runs. BTW, it runs fine on IE11

to try and explain my confusion, i have produced a new example in HTML (eliminating the variable of Pug template)
before (renders in FF, Ch, not in IE 11.4) https://codepen.io/anon/pen/MoNNdK?editors=1111
after (renders in FF, Ch, IE 11.4) https://codepen.io/anon/pen/GEVVaY?editors=1111
to reiterate, in (my particular) IE (11.4), and the IE of dozens of our users (this is how i found the issue, due to bug reports about a missing field), the first example does not render, and the second example does, and the only difference between the two is tab characters.  if you edit the first example and remove the tabs one by one, the moment you remove the final tab the component will render correctly.
if it's still something that is considered not relevant to vue, would you be able to point me toward the part of the source code which reads attribute values?

I'l' re-open this to take a closer look, maybe it's related to template parsing?

i am guessing that instead of using the DOM-nodes, Vue uses innerHTML, which i tested and does indeed return a different value in FF and IE.  whereas using el.attributes['v-bind:settings'] returns the same value.

Ok, thanks for the further explanation, it makes more sense. I thought you were having an error invalid expression in IE, but it's Vue generating that error.
I don't know why, I just get no error in my IE...

Should be fixed in cfd73c2386623341fdbb3ac636c4baf84ea89c2c


Action Server UI not handling fields properly

Target components
action_server
What is wrong
With a data such as:
from typing import Dict, List
from robocorp.actions import action
from robocorp import actions
from pydantic import BaseModel

class SomeData(BaseModel):
    rows: List[int]
    others: Dict[str, List[bool]]

@action
def do_something(arg: int, data: SomeData) -> str:
    """
    Test receiving a secret.

    Args:
        arg: Some arg
        data: Some data
    """
    print(secret.value)
    return ''

I can't get the input to work. Initially the int is not accepted (and then after changing it seems to be correct, so, just the initial value is not ok), but I can't seem to input the custom data at all (the field can't be edited at all).
System info
Windows
Example script
No response

Fixed in https://github.com/robocorp/robocorp/commit/44fac6baf3dc1436dd4922cdd6b23efb20067ee7


How to activate Magic Wand tool in 1.7.2

I have upgraded to V 1.7.2 but I can't find how to activate Magic Wand Tool.

I agree with you, I couldn't find on templates, too.

Please check docs here:
https://labelstud.io/tags/magicwand.html

Thank you for your answer,
I find that fflag_feat_front_dev_4081_magic_wand_tool is default set as true. But I still can activate it. Am I finding the right file?
Best regards

Hi I also, am having issues activating the magic wand function? I have upgraded to 1.7.2. But can't find any option in the feature flags to activate the magic wand?

@a-goose95 did you use <MagicWand> tag? doesn't it work?

Related: https://github.com/heartexlabs/label-studio/issues/3903

@YugangDuan @a-goose95 @createemir if you use config from docs and you see Magic Wand in toolbar, but it doesn't work, try to disable another conflicting feature flag via env var like this:
fflag_feat_front_lsdv_4583_6_images_preloading_short=0 label-studio
or in any other convenient way.
We'll release fix for this clash soon.

Unfortunately, I did not see the Magic Wand toolbar. And try to edit the config like the Magic Wand example, it shows that can not find the module.

Sorry for the Typo. I mean I still can not activate it, and I am not sure whether I find the right setting file.

Please follow this instruction about the feature flag:
https://labelstud.io/tags/magicwand.html#Feature-Flag

Thanks, I can activate with the code on docs:

Sorry if the question is not elevant, but how do you limit the expanding of MagicWand? I see how to expand the region by dragging the mouse, but what can I do to limit the region to a selected zone?


Design and implement a better training Loop (was issue #6)

From May 16 on language-api, @wrathagom entered:

At the moment we are just going to kick off training at each POST/PUT that modifies a model. But this is less than ideal. We should have control over when we train such that all modifications can be completed.
We also need to implement a training test and feedback loop. When a model is first created we make sure that we can parse all intents and entities supplied in the training data. Whenever that model is modified we need to make sure we still can pass it.

I think this is tackled with training detachment:

At the moment we are just going to kick off training at each POST/PUT that modifies a model. But this is less than ideal. We should have control over when we train such that all modifications can be completed.

Regarding this:

We also need to implement a training test and feedback loop. When a model is first created we make sure that we can parse all intents and entities supplied in the training data. Whenever that model is modified we need to make sure we still can pass it.

Is something that we could include alongside with the https://github.com/samtecspg/articulate/issues/171 in some section for Agent test

In my mind it's not #171 this is a UI specifically geared towards reviewing user requests and labeling them if mislabeled. Agree that we have implemented the training control piece of this.

Oh I see. And what would happen with those? are they going to be part of the training data after relabel?

yep, similar to https://dialogflow.com/docs/history


[Feature Request] Support for SDXS-512, allowing for real-time image generation on the CPU (~0.6 seconds per image)

It would be great if support for the SDXS-512 model could be added: https://github.com/IDKiro/sdxs
Especially for quickly generating images on the CPU, this is a major new development, so it would fit this project really well. https://github.com/rupeshs/fastsdcpu already supports it (requires Python though and really isn't lightweight and dependency-free like stable-diffusion.cpp).
I benchmarked SDXS-512 in FastSDCPU on my CPU (3950X):
FastSDCPU Regular:
RAM Usage: 700 MB
Image Generation Time (Latency): 2 seconds
FastSDCPU OpenVINO:
RAM Usage: 3.6 GB
Image Generation Time (Latency): 0.6 seconds
So it's really very fast, and even super memory efficient (when not using OpenVINO). And it even looks much better than the best reasonably fast results I can get with stablediffusion.cpp, which is using Dreamshaper LCM with ~5 steps, 2 GB RAM usage and 19 seconds generation time. SDXS-512 looks better, is 20 times faster and uses only 1/3 the RAM - and that's with a Python implementation.

indeed, if its gonna faster than sdxl turbo 512x512 with 1step then it is gonna be great to have it

@idkiro could u please take a look at this, we can use this for koboldcpp for chat and image generation, thank u

@IDKiro could u please take a look at this, we can use this for koboldcpp for chat and image generation, thank u

Yes, SDXS is much faster than SD Turbo, and I recommend using the SDXS-512-DreamShaper we put out, which is compatible with original SD1.5 VAE and Tiny VAE.
The SDXS-512-DreamShaper is based on SD 1.5, but with changes in the structure of the UNet, and we have marked the changes in the diffusers configuration file:
{
"block_out_channels": [
320,
640,
1280
],
"down_block_types": [
"DownBlock2D",
"CrossAttnDownBlock2D",
"CrossAttnDownBlock2D"
],
"layers_per_block": 1,
"up_block_types": [
"CrossAttnUpBlock2D",
"CrossAttnUpBlock2D",
"UpBlock2D"
],
}

thank u for ur answer idkiro, hope someone add this to sd cpp
@leejet can u please make changes to support this one as well, thank u


Create Blog “2022-01-22t17-39-25-482z-menulis-kode”

Automatically generated by Netlify CMS

👷 Deploy Preview for ibnuraffi processing.
🔨 Explore the source changes: e32a5d0011f517ad117a57a5de015451488573fb
🔍 Inspect the deploy log: https://app.netlify.com/sites/ibnuraffi/deploys/61ec47ec72a54400071df4e2


Make the MapRenderer CLI usable

All the bot apparatus was commented out anyway, soooo
Requires running in release because of test bugs
dotnet run -c Release --project Content.MapRenderer knightship
Nothing relevant to changelog or screenshots here

mapping docs should be updated when this is merged

Should we bundle the renderer with the local server for mappers?

that would be quite nice

So to be clear, am I supposed to be making it bundle the renderer with the local server? Or...???


Shotgun is awesome

My new fav. combo with guard-livereload, it really made my day. Thanks Ryan !

Thank you.


fbthrift: init at 2<IP_ADDRESS>

Motivation for this change
Added Fbthrift library and its dependencies
New packages:

Fbthrift (v2<IP_ADDRESS>)
Fizz (v2<IP_ADDRESS>)
Wangle (v2<IP_ADDRESS>)
rsocket-cpp (git master)

Updated packages

Folly (v2<IP_ADDRESS> >> v2<IP_ADDRESS>)

Fixed packages

Fixed fmt cmake manifest generation

Things done

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

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

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

Adding maintainers:
@abbradar (Folly)
@jeroendehaas (Fmt)

Please split commits into separate ones for different derivations.

Comments have all been fixed.
Commits have been split by derivation.

@pierreis do you still want this to get in? now fmt and folly are in a merge conflict.

Closing due to OP not responding.


Add prettier

add ESLint config
run eslint --fix automatically pre-commit using husky and lint-staged

I take this one. Have an automated script to set it up. 🤣

Only talking about using Prettier for JavaScript aren't we? Haven't used it yet for Handlebars / Glimmer templates but heard that it's not that stable yet. Would love to test it out but not sure if this is the best repository to start with. Or do you have good experience with it?

Only talking about using Prettier for JavaScript aren't we?

Yes.

automatically pre-commit using husky and lint-staged

I'm not convinced by this one:

Running ESLint and prettier is an expensive task. yarn lint:js --fix needs 10 seconds on my machine for this repo.
Running prettier in a pre-commit hook is not needed for my setup. I have configured my editor (VS Code) to fix linting errors when a file is saved. This is very cheap (read: performant) as it's only run against one file and not all.

I think my setup is a very common one. I would suggest that we don't add the precommit hook but document in CONTRIBUTING.md how one could configure the local development environment to run prettier automatically? We can list both options: Do it in editor on save or as a local pre-commit hook.

Oh haven't read carefully enough. lint-staged would also only run the linting against the files which will be committed. So there wouldn't be a noticeable performance impact except for commits touching a lot of files. Such commits doesn't happen that often.
Still not seeing a need for it as I expect most developers to have their editors configured to fix linting issues on safe. But it would help with onboarding new developers for sure.

Added husky and lint-staged to #1038. Directly noticed that it's slower even if touching only a few files. Committing a change to only one file takes 1.5 seconds with husky + lint-staged. It's only a few milliseconds without it. It's basically adding a waiting step that wasn't there before. To be honest I don't like it. But I can live with it if you prefer to have it in.

Still not seeing a need for it as I expect most developers to have their editors configured to fix linting issues on safe. But it would help with onboarding new developers for sure.

I do remember the times where I got failing builds for some silly prettier linting issues when contributing to other codebases that had prettier set up, but I was not yet a regular prettier user (and did not have my IDE set up to auto-fix). So that was annoying for me. I am not sure we can assume everybody is already using prettier, so that might help. Given that you don't commit that often, and 1-2 secs is not that terrible, I would slightly prefer it. But have not really strong feelings...

I do remember the times where I got failing builds for some silly prettier linting issues when contributing to other codebases that had prettier set up, but I was not yet a regular prettier user (and did not have my IDE set up to auto-fix). So that was annoying for me. I am not sure we can assume everybody is already using prettier, so that might help.

I fear the setup in #1038 would already annoy developers before they commit if not used to prettier. Prettier is integrated as ESLint plugin. Code not formatted as prettier expects is treated as a linting error. Many editors show such linting errors. Developers used to linting but not used to Prettier may try to fix by hand before committing...
Maybe documentation is key to make it easier for new developers? Will add a chapter to CONTRIBUTING.md.

Closed by #1038