Start new tasks and wait for their health checks to pass before killing unhealthy tasks

Similar to #2104, but this ticket goes further.
Marathon's task killing can happen faster than new instances can start up. The following conditions can aggravate this:

Your application only has 1 or a few instances configured
Something happens (e.g. #3317) that causes many healthchecks to die at once.
Your application takes a long time to start up
The Mesos cluster does not have enough resources free for replacement tasks (i.e. some other framework is taking those resources before Marathon can use them to start replacements)

When Marathon detects an unhealthy instance, it should start a new instance and wait for its health checks to pass before killing the old ones.
I'm not sure what the correct behavior should be when the unhealthy instance becomes healthy again. I think it's valid to stop starting a new instance, or go ahead and start the new instance anyway, then kill the old one.
Ideally, this would all be controlled by the minimumHealthCapacity and maximumOverCapacity parameters.

cc @solarkennedy @nhandler @rob-johnson

@EvanKrall Thanks alot for reporting this! We will look into it along with mesosphere/marathon#3317
In the meantime I suggest to use command health checks as proposed in mesosphere/marathon#3317 as well as sufficient upgradeStrategy and acceptedResourceRoles settings to mitigate the issues.
For details please refert to https://mesosphere.github.io/marathon/docs/rest-api.html and https://mesosphere.github.io/marathon/docs/health-checks.html

I feel like this is the current behavior (if appropriately configured) but feel free to reopen if it isn't.


EntityList is leaky

https://github.com/2004Scape/Server/blob/02b24d3712fa0f6c7b6533947672334fc8e52c4b/src/lostcity/entity/EntityList.ts#L74C1-L80C6
The set function stores an Entity into the backing array collection.
The remove function does not remove possible Entity out. This is protected though due to the way the get function is implemented.
Because of this, Players & Npcs that are dynamically added into the array are stored in heap forever, keeping all of their referencing as well, so this is never picked up by the GC.
Possible fix includes:
this[index] = undefined;. Using array splice would change the length of the backed array.

This is my memory usage over time on my local computer with the fix in place over the last 8 hours.
The top is the oldest and the bottom is the most recent:

fixed: https://github.com/2004Scape/Server/commit/8786120909ec697cdee795ac2fdc22153fd6a1bc


Can't register

I always failed to register. I didn't change any config

Hey! I think registrations were broken briefly after a deploy around then. It should be sorted now if you retry?

It's still not working

Not working here, either on macOS with brew, or on Debian 11.
Same error, same line on both OS
Debian's output
Error: failed to register user: failed to add user

Location:
    /home/runner/work/atuin/atuin/atuin-client/src/api_client.rs:63:9

macOS
Error: failed to register user: failed to add user

Location:
    /private/tmp/atuin-20230228-5854-ice2iz/atuin-13.0.1/atuin-client/src/api_client.rs:63:9

@tfkhdyt btw, this is largerly out of scope of this thread but what do you use to show your git branch ? Thanks.

@tfkhdyt btw, this is largerly out of scope of this thread but what do you use to show your git branch ? Thanks.

It's starship prompt.

EDIT : After digging a little it just seems that it's blocking custom domain ? Using an icloud address make it work, using a custom domain throw this error. Hm..

We actually don't even verify that it's an email 😅
@tfkhdyt I just checked the email in your screenshot, and you already have an account - can you try logging in?

@tfkhdyt I just checked the email in your screenshot, and you already have an account - can you try logging in?

Unfortunately, still not working

I had this exact same issue when I setup atuin on my self-hosted server. It was driving me nuts when I couldn't register. After a while I found out that the sync_address URL in the client config file had a forward slash at the end.
Try removing the slash, save, and try to register again. If your config says sync_address = "https://atuin.domain.com/" try https://atuin.domain.com instead.

I always failed to register. I didn't change any config：
Error: failed to register user: failed to add user

I'm facing the same issue:
Error: failed to register user: failed to add user

Location:
    /private/tmp/atuin-20240226-5245-z9ks0z/atuin-18.0.2/atuin-client/src/api_client.rs:71:9

@bhavika could you open a new issue detailing your setup please?

Are you self hosted or using the main server?
Have you changed any config?
What OS?
What shell?

I'm going to close this one because it's difficult to follow without the setup of each person involved.

@bhavika I just checked, and you already have an account - signed up last month. The error message should probably be improved for an already-existing email.

@bhavika I just checked, and you already have an account - signed up last month. The error message should probably be improved for an already-existing email.

Thanks for the quick follow-up @ellie! It does not seem like I can reset my password if I've forgotten it. I'll create a new account instead.

@bhavika could you open a new issue detailing your setup please?

Are you self hosted or using the main server?
I was using the main server.
Have you changed any config?
Nope.
What OS?
MacOS Sonoma 14.3.1
What shell?
Zsh

I'm going to close this one because it's difficult to follow without the setup of each person involved.

Fair!

Thanks for the quick follow-up @ellie! It does not seem like I can reset my password if I've forgotten it. I'll create a new account instead.

Not presently - we don't verify emails yet, which would be required for reset requests


Can't install spleeter using Git Bash on Windows 7

The install didn't work for me.
git clone https://github.com/Deezer/spleeter
conda env create -f spleeter/conda/spleeter-cpu.yaml
This results in:
CommandNotFoundError: No command 'conda env'.

You need first to install a Conda distribution on your system in order to be able to setup a Conda environment.


Icons: add craft, exhibit and newspapers

Craft: Hammer (Mjolnir)
Exhibit: Portrait (Mona Lisa)
Newspapers: Newspaper (The Daily Bugle) :sweat_smile:
I've updated the sprite in the repository.

:rocket: :rocket: :rocket:


Updating release information for Notifications API

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/extensions/api-support/extension-api-roadmap.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks @kypflug!


my app doesn't run on profile mode

i tried to run an app on profile mode.
but it didn't run.
i tried 2 ways.
At first i tried to run on VSCode with this launch option
{
      "name": "Flutter-profile",
      "request": "launch",
      "type": "dart",
      "flutterMode": "profile",
    }

vscode said
Could not run build/ios/iphoneos/Runner.app on XXXXXXX-XXXXXXX.
Try launching Xcode and selecting "Product > Run" to fix the problem:
  open ios/Runner.xcworkspace

Installing and launching...                                        22.4s
Error running application on XXXXXXXX.

then
i tried to run on xcode with profile mode
but xcode said
error: failed to launch '/private/var/containers/Bundle/Application/8AD4849D-565F-XXXXXXXXXXX-A4F7-34BF6398D714/Runner.app' -- XXXX-XXXXX has denied the launch request.
Internal launch error: process launch failed: failed to get the task for process 5046

by the way it run properly on debug mode even if device is real device(iPhone XR)

Hi @shiratsu
I can run the app with no issues
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Flutter",
            "program": "lib/main.dart",
            "request": "launch",
            "type": "dart",
            "flutterMode": "profile"
        }
    ]
}

Try a new project with this configuration
If the problem persist,  Can you please provide your flutter doctor -v your flutter run --verbose and a complete reproducible minimal code sample
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


kill unwanted parent directories when a segment is deleted from LocalDataSegmentKiller

fixes #2272

:+1: even if my comment isn't addressed

:+1:, looks good to me.

:+1: after travis

@fjy @nishantmonu51 @drcrallen  can one of you merge this one?


fix: switch to TypeScript

IMPORTANT

Use pnpm as your package manager.
Use pnpm install for initial repository configuration.
Start the development server with reload by pnpm run dev.

Changes / notes

Fixed a problem that attributes are changed and broken during svg parsing by changing the parser to linkedom.
Fixed a problem where binaries could not be sent by express.
Package manager uses pnpm
TypeScript Runner uses Bun internally

TypeScript runner was deno or Bun and chose Bun. Even if either runner was selected, the built-in installer could not properly install sharp, making it necessary to select NPM or pnpm.
In light of the benefits of using Docker, we are installing with pnpm.
Internally, we use Bun, but we don't need to be aware of it since it is already configured in the Dockerfile and further added to devDependencies.

I just ran docker compose up but it failed to install sharp. The error indicates a mismatch in the version of libgcc.
Please check in your environment with Docker Compose.
Thank you.
Attaching to texgaatonet-app-1
texgaatonet-app-1  | 32 |     if (loadedModule) {
texgaatonet-app-1  | 33 |       const [, loadedPackage] = loadedModule.match(/node_modules[\\/]([^\\/]+)[\\/]/);
texgaatonet-app-1  | 34 |       help.push(`- Ensure the version of sharp aligns with the ${loadedPackage} package: "npm ls sharp"`);
texgaatonet-app-1  | 35 |     }
texgaatonet-app-1  | 36 |   }
texgaatonet-app-1  | 37 |   throw new Error(help.join('\n'));
texgaatonet-app-1  |             ^
texgaatonet-app-1  | error:
texgaatonet-app-1  | Something went wrong installing the "sharp" module
texgaatonet-app-1  |
texgaatonet-app-1  | /usr/lib/libgcc_s.so.1: version `GCC_4.8.0' not found (required by<EMAIL_ADDRESS>texgaatonet-app-1  |
texgaatonet-app-1  | Possible solutions:
texgaatonet-app-1  | - Install with verbose logging and look for errors: "npm install --ignore-scripts=false --foreground-scripts --verbose sharp"
texgaatonet-app-1  | - Install for the current linux-x64 runtime: "npm install --platform=linux --arch=x64 sharp"
texgaatonet-app-1  | - Consult the installation documentation: https://sharp.pixelplumbing.com/install
texgaatonet-app-1  |       at /app/node_modules/.pnpm/sharp@0.32.6/node_modules/sharp/lib/sharp.js:37:8
texgaatonet-app-1  |       at globalThis (/app/node_modules/.pnpm/sharp@0.32.6/node_modules/sharp/lib/sharp.js:37:33)
texgaatonet-app-1  |       at overridableRequire (:1:20)
texgaatonet-app-1  |       at /app/node_modules/.pnpm/sharp@0.32.6/node_modules/sharp/lib/constructor.js:11:0
texgaatonet-app-1  |       at globalThis (/app/node_modules/.pnpm/sharp@0.32.6/node_modules/sharp/lib/constructor.js:439:17)
texgaatonet-app-1  |       at overridableRequire (:1:20)
texgaatonet-app-1  |       at /app/node_modules/.pnpm/sharp@0.32.6/node_modules/sharp/lib/index.js:6:6
texgaatonet-app-1  |       at globalThis (/app/node_modules/.pnpm/sharp@0.32.6/node_modules/sharp/lib/index.js:16:17)
texgaatonet-app-1 exited with code 1

Here is version of docker:
$ docker version
Client:
 Version:           24.0.6
 API version:       1.43
 Go version:        go1.20.7
 Git commit:        ed223bc
 Built:             Mon Sep  4 12:28:49 2023
 OS/Arch:           darwin/arm64
 Context:           orbstack

Server: Docker Engine - Community
 Engine:
  Version:          24.0.6
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.7
  Git commit:       1a79695
  Built:            Mon Sep  4 12:31:30 2023
  OS/Arch:          linux/arm64
  Experimental:     false
 containerd:
  Version:          v1.7.6
  GitCommit:        091922f03c2762540fd057fba91260237ff86acb
 runc:
  Version:          1.1.9
  GitCommit:        82f18fe0e44a59034f3e1f45e475fa5636e539aa
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Here is vension of docker compose:
$ docker compose version
Docker Compose version v2.22.0


Do not update to latest version if it is prerelase

Fixes #10.

I'm not sure how to check that my changes are actually working correctly. Also, the CI is failing for some reason.

I also formatted the code using purty because I was too lazy to format it manually.

@mhmdanas I'll have some time to come back to this tomorrow for a proper review. In the meantime, would you mind not formatting with Purty? It's much easier to review if I can see just what's changed (vs. the whole file being changed). Any formatting should ideally happen in a separate PR.
That said, due to some formatting issues with Purty I'm not quite ready to switch to it across these projects. I hope to someday, but if you could try and keep the formatting roughly the same as what exists already I would really appreciate that.

I have no problem with that. Will try to revert the formatting as much as possible.

I reverted all the formatting of the already existing code. I should probably ask if I can format the code before I actually do it. I'll investigate why the CI is failing.

I dug a bit into the generated JS, using good ol' console.logs along the way (should I learn how to use a debugger?), and found something interesting. The response of this request is a Not found error:
Right {
  value0: {
    body: {
      message: 'Not Found',
      documentation_url: 'https://docs.github.com/rest'
    },
    headers: [
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader],
      [ResponseHeader], [ResponseHeader]
    ],
    status: 404,
    statusText: 'Not Found'
  }
}

It seems I fixed the problem. I was already suspicious that the slash before the question mark was the cause. Is this the new "I forgot to add a semicolon"?

seems like there's a version that doesn't comply with SemVer. What should I do?

We should only update to the latest version if it is a) a SemVer version and b) not a pre-release version.

How do we know if it's actually a version, even if it's not SemVer? It could be a non-SemVer version, or it might not even be a version. Should I not care about this and just ignore any invalid versions?

Yea, anything invalid should be ignored. We only want the latest valid SemVer, non-prerelease version. In fact, I saw in https://github.com/purescript-contrib/governance/pull/25 that draft releases are also included in the GitHub response, so we'll want to filter those out too. Jordan wrote some similar code in that PR to what you're writing here, which may be helpful!

I see. Should I log a warning about invalid versions?
For the PR you linked to, Jordan's code does look similar to mine. I'll look in it if I need anything.
Also, for the default number of entries per page, I used 10, but I think 5 might be enough. Should I keep it as it is or change it?

Should I log a warning about invalid versions?

I think that would be a nice addition! In the github-actions-toolkit bindings there's a function for producing warnings (I believe it's actually called warning).

Also, for the default number of entries per page, I used 10, but I think 5 might be enough. Should I keep it as it is or change it?

I think 10 entries is fine. We could even pull the maximum, which I believe is 30. It's better to bring in the larger response so that we have to make fewer requests in the worst case. I think the difference between pulling 5 and 30 results will be negligible.

what are draft releases exactly? First time I heard of them.

When you publish a new release on GitHub -- not a tag, but the release itself -- you can choose to make it a "draft" to indicate it is not actually published yet. A draft release won't be listed as the "latest" release. It may not even be visible in the UI except to project collaborators, I'm not sure. In any case we definitely don't want to pull in draft releases and set that in CI!

By the way, as of #13 the CI should be fixed. You can merge that into this branch to get builds working again.

I think that would be a nice addition! In the github-actions-toolkit bindings there's a function for producing warnings (I believe it's actually called warning).

Thanks! I'll try using that.

I think 10 entries is fine. We could even pull the maximum, which I believe is 30. It's better to bring in the larger response so that we have to make fewer requests in the worst case. I think the difference between pulling 5 and 30 results will be negligible.

The maximum is actually 100. I asked if I should make it 5 because currently the tools mostly don't use pre-releases. But you're right in that the difference is negligible. I personally think we should keep it at 10, but if you think otherwise I'm fine with it.

When you publish a new release on GitHub -- not a tag, but the release itself -- you can choose to make it a "draft" to indicate it is not actually published yet. A draft release won't be listed as the "latest" release. It may not even be visible in the UI except to project collaborators, I'm not sure. In any case we definitely don't want to pull in draft releases and set that in CI!

I'll see how I can get that from the JSON response.

No need! It's failing because of an issue with the FFI in the actions toolkit bindings, and that's fixed and released. I'll update this repository to use that release.

Actually, as you can see in one of my comments, I had an extraneous slash which caused GitHub to return a "Not Found" error. So it's also on me!

I personally think we should keep it at 10, but if you think otherwise I'm fine with it.

Fine by me!

I'll see how I can get that from the JSON response.

You can see a JSON codec that Jordan implemented in his PR that I linked. It includes that field.

Thanks!


RocketPants 2.x Planning

The "What"
This issue hopefully will help solicit and facilitate a discussion around any proposed changes being incorporated into the 2.x release.
The "Why"
While working with rocket_pants I noticed the lack of support for some semi-standardized (well nothing in API land is truly standard) like content-type based routing and hypermedia-based pagination to name a few. Out of this I released the rocket_shorts gem (see jsmestad/rocket_shorts) that provided this varied functionality through a series of shims and overrides.
After releasing it, I solicited @sutto for some feedback on the direction, which quickly turned into proposition to merge that work into "the next major version of rocket_pants"
The "Where"
Drafted Proposal: 2.0 Planning

Eagerly waiting for the 2.0 release... has that been dropped?

To be brutally honest - It's not been dropped, but I've definitely dropped the ball. All of my time has been very, very focused on my day job (http://gyde.tv) so I've had zero time to work on it. I still want to see 2.0 done (and have a good idea of what needs to be done) - but I've been distracted by work that I haven't been able to get it done - or organising stuff for people to do.
I still want to see 2.0 - I use rocket pants dailly, but we're on an older rails 3.2 app / nothing new has been needed that has given me the excuse to work on it... yet.


Request to join working group

May I please join the working group? When are its next meetings?

Sure, I will send you a meeting invite.
(I think I saw your email in another issue)
Every 2 weeks, Mondays 8am PST

Invite send


Broken syntax highlighting in JavaScript template strings with 'Light' or 'Dark' themes

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T14:02:52.799Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

vs-keybindings
ms-vscode
0.1.3

In JavaScript, when using template strings like this:
`My name is ${name}`

the variable 'name' is the same color as the rest of the string instead of white. This is only present when using 'Light' and 'Dark' themes and not in 'Light+' and 'Dark+'.

Thanks for the report. The bug here seems to be that the ${ and } were not highlighted in JS. The new colorization for the plain light/dark themes should now look like this:

@mjbvz please don't the verification-needed label for bugs, but the bug-label which also "triggers" verification but after testing new things


fix: enforce consistency of contract negotiation request and transfer request

What this PR changes/adds
This PR fixes two issues of the current implementation:

in contract negotiation request, the asset id part of the offer id can be different from the asset id provided in the target field of the Policy.
in transfer request, the asset id provided in the request can be different to the one referred by the contract id also provided in this same request. This leads to inconsistent EndpointDataReferenceEntry being generated.

Why it does that
Fix.
Linked Issue(s)
Closes #4239
Closes #4240
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

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

Project coverage is 19.85%. Comparing base (7f20ba5) to head (b1909e3).
Report is 299 commits behind head on main.

Files
Patch %
Lines

...es/transferprocess/TransferProcessServiceImpl.java
0.00%
7 Missing :warning:

...ract/validation/ContractValidationServiceImpl.java
0.00%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4264       +/-   ##
===========================================
- Coverage   71.74%   19.85%   -51.90%
===========================================
  Files         919     1046      +127
  Lines       18457    20987     +2530
  Branches     1037     1171      +134
===========================================
- Hits        13242     4166     -9076
- Misses       4756    16716    +11960
+ Partials      459      105      -354

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


Typechecked and more thorough README

@raulraja take a look, addresses some of your points in #38. I'm not a big fan of putting al the docs in the README but I think showing what the library can do is enough, users that want more information will go to the more thorough documentation site.
Do you think a table of contents is needed?

Forgot to mention, see the end result

Looks great and no table of contents is needed. I think this covers those user that may run into the lib without a specific need but are curious to see what it does. If there is new stuff we may want to have Maureen do a quick Lang pass. LGTM!

No new stuff yet except for the change in fetch's signature, going to release 0.2.0 with this and the new syntax you contributed! :shipit:


Add docker build + package workflow, host on ghcr.io

Blueprint: https://github.com/tillkuhn/letitgo/blob/main/.github/workflows/release.yml

done


[Feature Request] Freehand Segmentation Tool with Scalar Data

What feature or change would you like to see made?
Hi,Dear @sedghi ,I have a confusion could you answer me?
I write the segmentation with the Freehand Segmentation Tool like below picture,it was powerful.

But I found the scalar didn't changed when i cache.getVolume() like this:

But I useing the BrushTools,and change the type of segmentation to Labelmap then cache.getVolume again,It can get the changed.
like this:

So the question is: you will plan to connect the Freehand segmentation Tool and Scalar?or I need to change my code to let it work.
Thank you very much!
Why should we prioritize this feature?
d

oh，I see the update!
Thanks！


doc/chore: document the fields in 1000.yaml

And remove the 'identifiers' field: the field was only kept for the upstream repository, and the field was removed there.

bors merge


some questions

1.difference between infer_sample and infer_batch from the source code?i don't find difference between them.infer_sample will also try to batch the input if max_batch_size>1 which should be done by infer_batch.
2.in server.py,how could we return the results? the docs don't elaborate this.
for example,we have a result as below
imgs=np.ones((8，1920，1080，3),dtype=np.uint8)

which way should i use to return this value?
return [imgs] or return dict(result=imgs)
i define the triton_config as follow:
with Triton(config=triton_config) as triton:
        logger.info("Loading ResNet model.")
        triton.bind(
            model_name="ResNet",
            infer_func=resnet_factory(DEVICES),
            inputs=[
                Tensor(name="image", dtype=np.uint8, shape=(-1, -1, 3)),
            ],
            outputs=[
                Tensor(name="result", dtype=np.uint8, shape=(-1,)),
            ],
            config=ModelConfig(
                max_batch_size=32,
                batcher=DynamicBatcher(max_queue_delay_microseconds=5000),  # 5ms
            ),
        )
        logger.info("Serving inference")
        triton.serve()

Another question:
When result key's shape is defined as
shape=(-1,)
shape=(-1,-1,-1,3)
shape=(-1,-1,-1,-1,3)

the server can always successfully return the results,this is very confusing.In the above situation,result key's shape should only be (-1,-1,-1,3)?
3.i notice you add a CLI inference_timeout_s,this will raise timeout exception at client side.But the server side is still processing and can't receive new request.I recommend a more proper way which can raise timeout exception in thread at server side,When this happens,the server side will stop the processing immediately when receive new request. please refer to https://github.com/dr-luke/PyTimeoutAfter
4.What's the speed of sending request to server side from client?I decode the img at client and then send them to server side.I find  this is quite slow.The transport between client and server is really time consuming.
Sorry for so many quesitons.Thanks very much!

Hi @lfxx,
Thank you for your questions. They are valuable in helping us improve both the PyTriton documentation and the PyTriton itself. Please find our responses below:

PyTriton offers the infer_sample and infer_batch methods to provide user-friendly API of the ModelClient. These methods adapt the provided input data to match the ModelConfig. If the model expects data with a batch axis (max_batch_size > 1), a new batch axis will be added when using the infer_sample method.

The lack of result shape verification is responsible for this behavior. We are currently working on improvements regarding the verification of returned results.

Regarding the timeout issue, your observation is correct. Currently, we have only provided a timeout option on the client-side. However, implementing a server-side timeout is part of our roadmap.
As a temporary solution, you can use request headers to pass the expected timeout to the server side and try to handle it within the inference function. Here is a sample code snippet:
# client script
with ModelClient(
    url,
    model_name,
    inference_timeout=infer_timeout_s,
) as client:
    result = client.infer_batch(
        ...,
        headers={“inference-timeout-s”: infer_timeout_s}
    )

# server script
@batch
def infer_batch(**inputs):
    ...

def infer_fn(requests):
    for request in requests:
        infer_timeout_s = request.parameters.get(“inference-timeout-s”)
        …
        infer_batch([request])   # handle timeouts per request

For breaking model inference function, you may try to use one these helper packages i.e. stopit, mentioned above PyTimeoutAfter, or some custom code.
Please note that this solution has some drawbacks, interrupting model calls from infer_fn is sometimes not possible.

Can you share your benchmarks scripts (client.py and server.py)?
One common issue we’re observing is the creation of a new client for each inference call. During initialization or the first inference request, there is an exchange of metadata requests between the client and server, which can significantly impact inference performance.
client = ModelClient()  # correct place to create client
for data in dataset:
    # creating a model client before each request
    # will cause the model metadata to be retrieved again.
    # slow_client = ModelClient(...)
    client.infer_sample(data)

Hi @lfxx, Thank you for your questions. They are valuable in helping us improve both the PyTriton documentation and the PyTriton itself. Please find our responses below:

PyTriton offers the infer_sample and infer_batch methods to provide user-friendly API of the ModelClient. These methods adapt the provided input data to match the ModelConfig. If the model expects data with a batch axis (max_batch_size > 1), a new batch axis will be added when using the infer_sample method.

The lack of result shape verification is responsible for this behavior. We are currently working on improvements regarding the verification of returned results.

Regarding the timeout issue, your observation is correct. Currently, we have only provided a timeout option on the client-side. However, implementing a server-side timeout is part of our roadmap.
As a temporary solution, you can use request headers to pass the expected timeout to the server side and try to handle it within the inference function. Here is a sample code snippet:
# client script
with ModelClient(
    url,
    model_name,
    inference_timeout=infer_timeout_s,
) as client:
    result = client.infer_batch(
        ...,
        headers={“inference-timeout-s”: infer_timeout_s}
    )

# server script
@batch
def infer_batch(**inputs):
    ...

def infer_fn(requests):
    for request in requests:
        infer_timeout_s = request.parameters.get(“inference-timeout-s”)
        …
        infer_batch([request])   # handle timeouts per request

For breaking model inference function, you may try to use one these helper packages i.e. stopit, mentioned above PyTimeoutAfter, or some custom code.
Please note that this solution has some drawbacks, interrupting model calls from infer_fn is sometimes not possible.

Can you share your benchmarks scripts (client.py and server.py)?
One common issue we’re observing is the creation of a new client for each inference call. During initialization or the first inference request, there is an exchange of metadata requests between the client and server, which can significantly impact inference performance.
client = ModelClient()  # correct place to create client
for data in dataset:
    # creating a model client before each request
    # will cause the model metadata to be retrieved again.
    # slow_client = ModelClient(...)
    client.infer_sample(data)

1.very clear.
2.This is quite urgent,do we have a deadline?
3.i have used PyTimeoutAfter to temporary achieve this.Hope for offical implements.
4.finally,i find i decode the img to wrong format which consume lots of time.the speed between server and client is fast.
Thank you for your detailed explainment.

@lfxx we are currently working on improving the outputs validation. No exact ETA at the moment, but we will let you know once is available. Thanks for the feedback!

In the 0.2.4 version there is strict=True (default is False) flag in triton.bind method that enable strict validation of result. This include a shape and dtype verification.


What requirements does the Spitfire .dll have to run.

Hi!
I've gotten spitfire to run on my computer, and am able to connect with multiple clients and the communication works. I would however like to deploy this to a separate machine to run continously. This is however proving quite difficult.
I keep getting the below error.
So my question is, what dependencies does Spitfire have, why would I be seeing that error on several other machines (tried 2 windows 10 machines, and one windows server 2016 machine) when it runs perfectly fine on my Windows 10 development machine.
I can create a build release on my windows 10 machine, copy that folder to any location on my dev machine and run it just fine. The same folder/build fails on any other machine I've tried with the below error.
Something must be missing.
I created a stackoverflow question about it here: https://stackoverflow.com/questions/53161400/cant-read-dll-on-production-server-but-works-on-dev-machine but no one has yet been able to help me find a solution.

System.IO.FileNotFoundException: Could not load file or assembly 'Spitfire.dll' or one of its dependencies. The specified module could not be found.
File name: 'Spitfire.dll'
at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
at System.Reflection.Assembly.Load(AssemblyName assemblyRef)
at Owin.Loader.DefaultLoader.AssemblyDirScanner.d__1e.MoveNext()
at Owin.Loader.DefaultLoader.SearchForStartupAttribute(String friendlyName, IList1 errors, Boolean& conflict)    at Owin.Loader.DefaultLoader.GetDefaultConfiguration(String friendlyName, IList1 errors)
at Owin.Loader.DefaultLoader.LoadImplementation(String startupName, IList1 errorDetails)    at Owin.Loader.DefaultLoader.Load(String startupName, IList1 errorDetails)
at Microsoft.Owin.Hosting.Engine.HostingEngine.ResolveApp(StartContext context)
at Microsoft.Owin.Hosting.Engine.HostingEngine.Start(StartContext context)
at Microsoft.Owin.Hosting.Starter.HostingStarter.Start(StartOptions options)
at Communication.WebRTC.WebRTCHandshaker.WebRTCLoop(String url) in F:\SW\DedicatedServer\Communication\WebRTC\WebRTCHandshaker.cs:line 56

Thank you for the help!

The problem was missing VC++ redistributable. After installing vc_redist.x64.exe everything worked.


Create .github folder

... and move ISSUE_TEMPLATE.md and PULL_REQUEST_TEMPLATE.md inside it. 👍

Yes, we can. My bad. It's done now. Feel free to merge when you can 😄


Logs not making it to Loki

just working up a proof of concept.
.net 7 console application repo
installed grafana/loki/promtail using the docker setup. endpoints like /metrics and /ready work from localhost and from another box. ip address in nlog.config is good. no auth, so i omitted username and password.
curl from localhost works to post events to loki
curl -v -H "Content-Type: application/json" -XPOST -s "http://localhost:3100/loki/api/v1/push" --data-raw   '{"streams": [{ "stream": { "foo": "bar2" }, "values": [ [ "1675029208000000000", "fizzbuzz" ] ] }]}'
curl from another box (where app is running) works to post events to loki
curl -v -H "Content-Type: application/json" -XPOST -s "http://my.good.ip.address:3100/loki/api/v1/push" --data-raw   '{"streams": [{ "stream": { "foo": "bar2" }, "values": [ [ "1675029208000000000", "fizzbuzz" ] ] }]}'
app itself doesn't post events to loki
i confirmed the curl statements above worked by using the logcli tool on the box. also grafana seems to see them ok. but no luck with the events from my app.
im tossing some random events at it. my label/value's may not make sense.
my goal is to use nlog.loki in a handful of different console applications for different tenants to log a handful of different types of events and then visualize with grafana.

Hi @davidlfox.
Sorry to hear you don't get the logs working in your app. I don't have a definitive idea about what could cause the issue.
Just out of the blue, since the logger batches and pushes messages asynchronously, it could be that the app is closed before the logs could be pushed asynchronously. Could you force an async delay before the end of the app ?
await Task.Delay(10000);

Also, try to force flush the logger before closing the app with https://nlog-project.org/documentation/v4.0.0/html/M_NLog_LogManager_Flush.htm.
Alternatively, force an infinite loop like this sample.
Let me know if that works for you!

I'm glad your issue was resolved. I'm not sure what the exact implementation is for a shutdown that was not sufficient with a flush. It is implemented by NLog though, not by the loki target for NLog.
See: https://github.com/NLog/NLog/wiki/Tutorial#5-remember-to-flush


[Missing Feature] Ranks with Flair/Asterisk

Optio: &a*
Centurion: &6*
Dux: &5*
Internet Famous: &1*
Administrator: &6❖
Moderator: &c❖
Senior Moderator: &4❖
Event Coordinator: &3❖
Winner: &f*

Some of these will be implemented, however not all. I plan on having Mod, Admin, and Dev be ranks.

You forget Ref, Livestreamer and other ....
@Edit Ref: §3❖

You could add all with no use, but would be great to have them all

You don't need any "premium package" in a scrimmage server tho

I will consider Ref, but that would be about it.

@twizmwazin Livestreamer too? Official?

@AngelCasillas01 Why do we need a livestreamer rank? What would an official be?

@twizmwazin Livestreamer for livestream matches

@ElFlambMC Who livestreams scrims? And if they do, why do they need a badge to do it?

@twizmwazin Instead maybe add a config file to add these prefixes/flairs? I don't see the need for these extra flairs but if some people want it for the looks, maybe add a config file to add flair types and have a /flair command to assign it?

@twizmwazin Pl0x can you make that for all guys

@SungMatt @ElFlambMC Make a PR if you want it that badly.

@twizmwazin idk how make it.

@ElFlambMC It isn't hard to learn

I think that OP and ref would be enough. Not dux or optio, I don't see it useful. Nor developer cause except twiz, elliot, ely and a few more there are not developers so I would consider that if a person says that he is a developer, he is, in some way, taking your job...

@mrthesharpie :)

@MrTheSharpie The developer rank would be hard-coded in. This means that developers would have the rank on all servers, and you wouldn't be able to just give someone the rank.

@twizmwazin Okey, that's fine :)

@twizmwazin Will it be contributors or people who have done 20+ commits?

A person will be considered a developer if their contributions were in some way significantly transformative.

@twizmwazin how would you implement this? Using Player.setPlayerListName?

Sounds like it could work. We'd have to add it in chat too.

public void playerJoin(PlayerJoinEvent e) {
            Player p = e.getPlayer();
            String pname = p.getName();
if (p.hasPermission("flair.SeniorModerator")) {
                    String member = "" + ChatColor.DARK_RED +  "❖" + ChatColor.RESET + pname + ChatColor.RESET + "";
                    p.setDisplayName(member);

Something like this?

@SungMatt Probably. I'd have to test how it interacts with teams and the like.

@twizmwazin If that's the case, we can always convert to either getName or getDisplayName depending on the situation.

@twizmwazin Depending on how you are setting the team color of a player, setting their flair when they join may be override by the new display name, containing the colour being set. Either way it should be easy to do, I will do permissions later today if I have time

@CaptainElliott player.setDisplayName(flair + player.getDisplayName) should avoid the overriding issue.

@twizmwazin Yes, that would be done when the player joins, however when they join a team their display name will then be set to just their name, overriding the flair. However as I said that's an easy fix

@CaptainElliott oh yeah, oops

@CaptainElliott ChatColor.DARK_RED +  "❖" + TeamUtils.getTeamByPlayer(player) + player.getName() + ChatColor.RESET;

@dentmaged You wouldn't use "❖," you would use the Unicode char.

@twizmwazin okay, could I make a PR and start work on this?

if a player has dux, centurion or option, can you edit the snowflakes multiplier?

@Jonmuxu Donor ranks won't be added, as stated earlier.

D:

private final String opFlair = ChatColor.GOLD + "❖";
private final String modFlair = ChatColor.RED + "❖";
private final String devFlair = ChatColor.DARK_PURPLE + "❖";

Thanks :3

@Jair07mx Not sure what you are trying to say. Additionally, it isn't proper to directly copy the symbol, rather Unicode codes should be used.


config.pl configuration should come from settings file

Currently configuration comes from constants in facts in config.pl
Recommend these facts become rules that call settings/2 so this stuff can migrate to a settings file
(or ditch this and use settings directly?).

We now set from environment variables.


Getting Path From Uri?

Hello,
I am using Aviary to edit a photo that was selected from picker. This returns a Uri in onActivityResult. How can I get the correct path from this Uri result to update my image with Image.setpath?
Thanks

@GitUser4732 Did you want to update the Image object?
I'm not familiar with Aviary result Uri could you please give me some String representation from that?

Yes I want to update my Image objects path to be that of the edited image result.
The Uri result is content://media/external/images/media/243255.
I need help to get a result from this Uri like below.
/storage/emulated/0/Pictures/aviary-image-1505709867315.jpeg

@GitUser4732 my best shot is using this function from aFileChooser 😄
https://github.com/iPaulPro/aFileChooser/blob/master/aFileChooser/src/com/ipaulpro/afilechooser/utils/FileUtils.java#L257
Since this is not exactly library issues, i will close this one. Cheers!


GH26: Add support for installing Cake using tool manifest

fixes #26

@devlead isn't it kind of counter-intuitive to have mutually exclusive options cake-tool-manifest and cake-version?
What do you think of modifying the current cake-version property:

add for consistency:
# equivalent to the current default when cake-version is omitted
# always uses the latest version
cake-version: latest

add instead of cake-tool-manifest
# use version from manifest
cake-version: tool-manifest # or manifest, from-manifest, something like that

This could then, given some time, become the new default if cake-version is omitted.

Also, documentation of the new feature is still missing.

@nils-a makes total sense, adjusted to cake-version parameter supporting tool-manifest
steps:
  - name: Run the Cake script
    uses: cake-build/cake-action@v1
    with:
      cake-version: tool-manifest

@ecampidoglio I've turned CakeVersion into a union type and incorporated your suggestions.

I tested it - I love it 👍

Thank you for your contribution! 🎉

Thank you for your contribution! 🎉

Excellent 👍, can't wait for a release to take this for a spin 😎

@devlead You can try it right now by referencing master in your workflow:
uses: cake-build/cake-action@master

@devlead You can try it right now by referencing master in your workflow:
uses: cake-build/cake-action@master

That actually I knew :) I used my fork before merged and tried master after. But have a few repos to PR and those have policies of pinning versions.


Audio Imbalances and Other issues

In my experience, in the Mixtape mod settings, under the Game tab, adjusting the values of the volume does not change anything. Also, I'm using More C418 Music by Spleenczar as well. This music seems to be much louder than the vanilla for some reason. I'm not sure if this is a Mixtape issue but I thought I would post it here.
Finally, I rarely ever hear the creative music in survival since I turned that setting on. Is it really that rare, or is there an issue with Mixtape?

Thanks for letting me know! I'll look into it after I fix another bug. And yeah the non-vanilla music was originally a different volume and I had to edit them to make them fit in with the vanilla sounds, i guess i'll look over them again, but i thought i had done well enough. I'm not sure what you mean C418 music by Spleenczar. And for the creative music playing in survival, the survival music already plays in creative so i had just set it to play the creative music set in survival and its worked for me previously but i'll also check on that. Thanks for letting me know!
If you wanna talk a bit easier you could join my discord server (https://discord.gg/WyrTsYFweK)

I can confirm that the creative music option is working as intended. Also if you use mixtape you won't need to use More C418 Music by Spleenczar because mixtape adds built in resourcepacks that also adds the same c418 music, but at the same volume as regular mc music.

I would but I only use More C418 Music pack because I can delete certain tracks. Is there a way I can access the built in music packs from Mixtape and remove certain songs? Maybe that pack is also interfering with the creative mode music, so if I can swap, I would love to!

Which tracks are you deleting? mixtape adds multiple built in packs (one of them removes the newer music and adds the unused music)

I understand Mixtape has those packs but since I have access to the other pack and the ability to delete various songs within it, I use that. I delete some of the unused songs within the pack. I can't find a way to remove the songs from your packs and also, your packs don't include the track Droopy Likes Richochet, which I like. Does that make sense? Otherwise, I would use your pack that removes the newer music but adds some unused music.

Which tracks are you deleting? I can make a version of the resourcepack that doesn't add all the unused songs, and where is droopy likes richochet used? I might also be able to add that.

Oh but I wouldn't want you to publish an entire new pack just for my personal taste! If it isn't possible to customise your packs then I can continue using my current set-up. Although I will link the tracks I'm currently using that are included within More C418 Pack.

Droopy likes Richochet is from the Minecraft Volume Alpha. If it isn't possible to modify your music packs, then please consider this issue closed, I can continue to use my current set-up:)

I meant to ask if you had droopy playing as a disc or just ambience music. Also I managed to fix the first issue about the music not changing volume with the slider

Oh it's ambient music. And great to hear about the fix:)

I was also wondering if it would be possible to make a pack that allows the main menu music to be played in the overworld within one of your packs like the No Caves and Cliffs pack? Just the idea of getting as much C418 music in the overworld, specifically non-biome specific, as possible. Thank you for your time:)

Definitely! I have already made a non-biome specific resource pack that will be added in the next release (coming within a week hopefully)

Fixed in v1.5.0


[impeller] gpu benchmark probes have been accidentally disabled

https://github.com/flutter/engine/pull/52087 has caused some performance probes to drop to zero.
examples:

99th_percentile_gpu_frame_time
worst_gpu_frame_time
90th_percentile_gpu_frame_time

cc @chinmaygarde
skia perf
https://flutter-flutter-perf.skia.org/e/?begin=1713375187&end=1714657908&keys=Xdab252144b6c666305544d303f8161ea&requestType=0&selected=commit%3D40587%26name%3D%252Carch%253Dintel%252Cbranch%253Dmaster%252Cconfig%253Ddefault%252Cdevice_type%253DSM-A025V%252Cdevice_version%253Dnone%252Chost_type%253Dlinux%252Csub_result%253D99th_percentile_gpu_frame_time%252Ctest%253Danimated_blur_backdrop_filter_perf__timeline_summary%252C&xbaroffset=40591

Also cc @jonahwilliams since I think he was the one that originally implemented those gpu probes.

Ah, I see what I missed. There are calls to the tracer deep in the KHR impl that I missed https://github.com/flutter/engine/blob/df2efe7f3896eda012000a8d08de461f4c4641ed/impeller/renderer/backend/vulkan/swapchain/khr/khr_swapchain_impl_vk.cc#L366. Will need to make that abstract and shared across impls.


Write multicast group error.

Hi,
I'm having an issue when I try write a multicast group with p4runtime. The connection is OK and i can read/write counters and tables, but when I try write a multicast group I receive: grpc._channel._Rendezvous: <_Rendezvous of RPC that terminated with (StatusCode.UNKNOWN, )>
the log of my controller is this:
connected with swith s1 on <IP_ADDRESS>:50051
connected with swith s2 on <IP_ADDRESS>:50052
connected with swith s3 on <IP_ADDRESS>:50053
sending write request:
device_id: 2
election_id {
low: 1
}
updates {
type: INSERT
entity {
packet_replication_engine_entry {
multicast_group_entry {
multicast_group_id: 1
replicas {
egress_port: 1
instance: 1
}
replicas {
egress_port: 2
instance: 1
}
replicas {
egress_port: 3
instance: 1
}
}
}
}
}

This repo is for spec issues only. Please use the p4lang/PI or p4lang/behavioral-model repos next time, as they implement P4Runtime.
The UNKNOWN error code usually indicates that there was an error in the target as it was processing the required operations. Assuming you are using bmv2, you will need to look at the bmv2 logs to figure out what the issue is. If bmv2 crashed, a stack trace would be useful.

I tried this message and I didn't get any issue, so you'll definitely need to take a look at the bmv2 logs.

Hello. I'm having trouble writing a multicast group with p4runtime. Can you show me how to do it exactly? I can't find anything about it.

https://s3-us-west-2.amazonaws.com/p4runtime/docs/master/P4Runtime-Spec.html#sec-multicastgroupentry
https://github.com/p4lang/behavioral-model/blob/master/targets/simple_switch_grpc/tests/test_pre.cpp

Closing this. If you need further help, please send an email to the p4-dev mailing list or reach out to us on the p4.org slack channel.


error

what's problem

Delete import secretvars
ss = secretvars.secretvars() from bot.py
TOKEN = ss.tokenid
GUILD = ss.guild delete ss.token id and put "TOKEN" same with guild.


imgtool: Create simple signing tool

The imgtool is a simple image signing tool based on the libraries in the
newt tool, but appropriate for images that live outside of a Mynewt
project.
This initial version is able to generate, extract public keys from, and
sign ECDSA P-256, P-224, and RSA keys.  The signing tool is also able
to add an image trailer to mark this image for upgrade.

Awesome.  Looks good to me.


the CalculateUserPointsJob job doesnt seem to work

Describe the bug
a wonderful project, i have to say
but the CalculateUserPointsJob seems unfinished, always gives me error that the object injected is null
To Reproduce
Steps to reproduce the behavior:

Create new discussion
login to another user
comment on the discussion
See failed at job report
See failed jobs table updated with records

Expected behavior
the job should work and update the points given to the user when user comments on another user's discussion
Screenshots

Describe the bug a wonderful project, i have to say but the CalculateUserPointsJob seems unfinished, always gives me error that the object injected is null
To Reproduce Steps to reproduce the behavior:

Create new discussion
login to another user
comment on the discussion
See failed at job report
See failed jobs table updated with records

Expected behavior the job should work and update the points given to the user when user comments on another user's discussion
Screenshots

Have you fixed the point related error? Please share with me. :(


Color Picker Extension

Is your feature request related to a problem? Please describe.
The Color Picker Extension is a versatile and user-friendly tool designed to help you select, analyze, and manage colors directly from your web browser. Perfect for web designers, developers, graphic artists, and anyone who works with digital media, this extension provides a seamless experience for color management.

I know this issue is already assigned but there hasn't been any work done on it so I would like you to assign it to me.

Color picker extension already added in my profile. Kindly raise new Issues


Allow licenses to be specified during cargo new

This pull request would allow a user to specify a license to include
with their project. If the user specifies a license, a line is added
to Cargo.toml specifying the license, and in the case of some
licenses, the actual LICENSE file is added to the initial project.
A user can put their choice(s) of license in a .cargo/config file,
like so:
[cargo-new]
license = "MIT/Apache-2.0"

or it can be specified on the command line when a project is created:
$ cargo new project-name --license "MIT/Apache-2.0"

There are 5 licenses that I have included the actual LICENSE files for:

MIT
Apache-2.0
BSD-3-Clause
MPL-2.0
GPL-3.0

If the user chooses any of these licenses, the corresponding LICENSE
file is generated and added to their project. In the case of multiple
licenses, each LICENSE file has a suffix added to the file name. So,
if a project was dual-licensed "MIT/Apache-2.0", their project would
include the files LICENSE-MIT and LICENSE-APACHE-2.0.
If the license is not one of these, the entry is still added to
Cargo.toml, and a LICENSE file is still added to the repository,
though it is content-less.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

I did a short survey of the licenses used in projects on crates.io while I was preparing this PR, in case you wanted some reasoning about why I included the licenses I did: http://paul.woolcock.us/posts/crates-io-license-survey.html.
I added the GPL to the list after some feedback on the article.

Whoa! This is awesome, thanks for doing this @pwoolcoc! Some thoughts of mine:

An unknown --license may want to generate an error rather than an empty license file, but perhaps --license-file and [cargo-new.license-file] could be supported options as well? That corresponds to what the manifest accepts as well at least.
Things like the MIT license need a year/copyright holder name, and it looks like those aren't filled in just yet?
Perhaps all the licenses could be pre-formatted to an 80-character line limit?
I wonder if we may want to start warning on a missing --license parameter  perhaps?
Could you also update the documentation about configuration, and perhaps even about cargo new to indicate that a license is possible to pass?

cc @brson, @wycats, sure you guys will have opinions as well!

Like the idea. Agree that bogus licenses should generate an error.
Since this includes a bunch 3rd-party text in the product, I am wondering what the copyright status of the licenses themselves is, and how that impacts Cargo's licensing. This isn't a question I've ever considered before.

The license for the GPL is "Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed.".

Perhaps the license files that contain copyright year and holder information could be modified to use the formatting syntax like
Copyright (c) {year} {holder}

along with a method like
impl License {
    fn write_text(&self, f: &mut std::fmt::Formatter, year: u32, holder: Option<&str>)
        -> std::fmt::Result
    {
        match *self {
            License::MIT => write!(f, include_str!("../../../src/etc/licenses/MIT"),
                               year = year, holder = holder.unwrap_or("(Unknown)")),
            License::APACHE2 => write!(f, "{}", include_str!("../../../src/etc/licenses/APACHE2")),
            ...
        }
    }
}

in order to fill in the values where applicable.

:umbrella: The latest upstream changes (presumably #2022) made this pull request unmergeable. Please resolve the merge conflicts.

@gerv I'm interested in your opinion about the legalities of bundling a bunch of license texts into Cargo, re my earlier comment.

License texts vary - some are "no modifications" (like the GPL), some are "you can make derivative works but you have to change the name" (like the MPL). However, license texts themselves are generally considered exceptions to a project's licensing policy, as long as they are distributable (as all open source license texts are). Some people argue that for software to be open source, it has to use a license whose text is itself open source, but that view is a minority one.
So I'd say bundle away. I think the license list you have is good - I certainly wouldn't add any others. The BSD license and the MIT license are equivalent in effect, and both are deprecated in favour of licenses which actually address patents specifically. The Rust license is partly MIT so you probably have to include that, but if you left out BSD-3-Clause, I would not weep a single tear. The licenses you have left - Apache -> MPL 2 -> GPL 3.0 - cover the broad spectrum of copyleft options pretty well.
Gerv

Thanks @gerv. I need to rebase this branch and finish off a couple minor things from @alexcrichton 's list, and then I will ask about getting it merged again.

Have you had a chance to get back to this @pwoolcoc?

Yea, I mentioned in #rust yesterday that I am just getting back to this. I am working on this tonight a bit, hopefully I can get it merge-able either tonight or tomorrow.

An unknown --license may want to generate an error rather than an empty license file, but perhaps --license-file and [cargo-new.license-file] could be supported options as well? That corresponds to what the manifest accepts as well at least.

I have implemented both these.

Things like the MIT license need a year/copyright holder name, and it looks like those aren't filled in just yet?

Done.

Perhaps all the licenses could be pre-formatted to an 80-character line limit?

Done.

I wonder if we may want to start warning on a missing --license parameter perhaps?

Haven't touched this, as I didn't see any additional feedback about this.

Could you also update the documentation about configuration, and perhaps even about cargo new to indicate that a license is possible to pass?

I think I got all the docs updated, let me know if I missed something.

r? @alexcrichton

The logic here for writing out the LICENSE file itself makes me a little uncomfortable, but I'm not sure if there's much we can do about it. It definitely seems to be standard practice to have a file like that, but it involves Cargo parsing licenses and emitting files, both of which are a little unfortunate.
I'd love to take a more conservative approach to start off with by not worrying about foo / bar licenses or things like that and instead just emit one LICENSE file if the string is recognized, and perhaps emitting a warning otherwise. I'm not sure how feasible that'd be though?

I will remove the foo/bar stuff, if necessary, but I had originally implemented it because of the prevalence of the MIT/Apache-2.0 licensed projects on crates.io. I am afraid that, if that is removed, this feature becomes a lot less useful.

MIT/Apache 2.0 is the license of Rust. It would be a very bad idea not to have that be an option (heck, even the default) for Rust project license generation.

One possibility would be to special-case MIT/Apache 2.0 if you don't want to support dual-licensing in general.

Hm yes in retrospect let's just special case the MIT/Apache-2.0 license for now. The error message can help guide towards that if it's what's intended, and we don't have to worry about parsing license strings in Cargo itself.

:umbrella: The latest upstream changes (presumably #2081) made this pull request unmergeable. Please resolve the merge conflicts.

Shall I try adapting this for cargo init?

@vi yeah I suspect that this would want to modify cargo init now as well, @pwoolcoc would you be ok rebasing this and addressing the last few comments?

@alexcrichton sure, I'll try to get back to this tonight.

This has been inactive for some time now, so I'm going to close this, but feel free to resubmit with a rebase!

Is it a good idea to "take over" and re-submit this thing myself instead of waiting for original author?

@vi I don't have a problem with it if you pick it up -- I just haven't had a chance to get back to it, if you have the time, awesome!

Is scanning existing LICENSE or COPYING file and filling in license: or license-file: in cargo init a good idea?

No. Detecting licenses is the task of a dedicated license scanner; you don't want to try implementing that code. :-) You'd be amazed at how many variants there are.

@gerv You can easily make a program covering 99% of all cases.

@gerv, Maybe detect not the license itself, but just license file and then fill in license-file: COPYING?

Oh, I see. That seems less problematic if you just look for files called COPYING, LICENSE, LICENCE or the equivalent ".txt" versions.


Builtin-in function to fail/panic?

Maybe there is a way to do this already, but I was hoping to have an easy way to make the justfile failed on unsupported configurations.
For example if an OS is not supported, can I do something like this:
target := if os() == "linux" {
      "x86_64-unknown-linux-gnu"
} else if os() == "macos" {
     "x86_64-apple-darwin"
} else {
    panic("Unsupported operating system")
}

Such a panic-function should ideally print to stderr and return an error code != 0.

There isn't a panic function, but there is a nice hack:
target := if os() == "linux" {
  "foo"
} else {
  `exit 1 # unsupported OS`
}

The backtick will return status code 1, so the just will stop evaluation.

Being able to error about invalid configurations, such as a missing environmental variable, is a feature I really miss.
I'll try casey's hack to tide me over.

This would be a good first issue. I think a function that takes a single argument, and always returns an error containing that argument, would be pretty easy. The name could be error.

I'll take a stab at this.

Done - https://github.com/casey/just/pull/1118


Add community files

We should probably start out with an MIT license after all. This allows us to switch the license more easily later on.

Ok.

Changed it now to MIT License.


dogfood the most recent setup-dart action

The setup-dart action was recently re-written - ported from Bash to Dart. We should dogfood the latest commit in main to make sure that no functionality has regressed. To dogfood:
      - uses: dart-lang/setup-dart@8f7850f0f6056c5b91a1b9c206a53c9b1be28e4e

[x] dart-lang/ecosystem (https://github.com/dart-lang/ecosystem/pull/82)
[x] dart-lang/linter (https://github.com/dart-lang/linter/pull/4113)
[x] dart-lang/test (https://github.com/dart-lang/test/pull/1964)
[x] dart-lang/webdev (https://github.com/dart-lang/webdev/pull/2017)
[x] dart-lang/build (https://github.com/dart-lang/build/pull/3468)
[x] dart-lang/tools

After the dogfooding is complete, we'll close this issue; at that time you can convert your use back to a regular dep (uses: dart-lang/setup-dart@v1).

v1.5.0 has been published and the v1 tag moved to point to v1.5.0.


INDArray serialization issues

Tested on nd4j-native only
Two separate bugs here:

Serialization using Nd4j.write(INDArray,DataOutputStream) seems to fail on views.
Serialization of an INDArray via standard java object serialization results in correct data, but

(See also, which might be possible to do at the same time: https://github.com/deeplearning4j/nd4j/issues/784)

Issue 1
    @Test
    public void testSerializationOnViews() throws Exception {
        int length = 100;
        INDArray arr = Nd4j.linspace(1,length,length).reshape('c',10,10);

        INDArray sub = arr.get(NDArrayIndex.interval(5,10),NDArrayIndex.interval(5,10));

        try(DataOutputStream dos = new DataOutputStream(new FileOutputStream(new File("TempTestSerializaton.bin")))){
            Nd4j.write(sub,dos);
        }

        INDArray arr2;
        try( DataInputStream dis = new DataInputStream(new FileInputStream(new File("TempTestSerializaton.bin")))){
            arr2 = Nd4j.read(dis);
        }

        assertEquals(sub,arr2);   //java.lang.IndexOutOfBoundsException: 55
    }

Issue 2
The following unit test fails (both on the first assert equals, and the .add() op).
Note that the fromBytes array below seems to have the correct data in the buffer, and the correct shape... maybe pointer related??

    @Test
    public void testSerialization() throws Exception {
        int length = 10;
        INDArray arr = Nd4j.linspace(1,length,length);

        ByteArrayOutputStream baos = new ByteArrayOutputStream();
        try (ObjectOutputStream oos = new ObjectOutputStream(baos)) {
            oos.writeObject(arr);
        }

        byte[] asBytes = baos.toByteArray();

        INDArray fromBytes;
        try (ObjectInputStream ois = new ObjectInputStream(new ByteArrayInputStream(asBytes))) {
            fromBytes = (INDArray) ois.readObject();
        }

//        System.out.println(arr);
//        System.out.println(fromBytes);
        assertEquals(arr,fromBytes);

        INDArray temp = arr.add(Nd4j.ones(1,length));
        INDArray temp2 = fromBytes.add(Nd4j.ones(1,length));   //java.lang.IllegalArgumentException: Length must be >= 1
        assertEquals(temp,temp2);
    }

Fixed here: https://github.com/deeplearning4j/nd4j/pull/867


chore: split analytics test

Description of changes
This splits analytics tests into separate jobs.

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

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

Can we give descriptive names to the test files (and may be describe string as well)

@Amplifiyer I renamed the files, though I left describe-it part as is - these seem to be good enough.
Fwiw, we had similar discussion in the past here https://github.com/aws-amplify/amplify-cli/pull/11080#issuecomment-1266051296 .


Add To Slack Example

Can we include an example for using oauth to connect with the "Add to Slack" button?

think we can follow this django plugin's lead? https://github.com/os/slacker/blob/master/slacker/init.py#L658

Binance Free Tokens Giveaway - Claim Yours Now! 🎉💰
Welcome to the exclusive Binance Free Tokens giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this fantastic opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Steps to Claim:

Spread the Excitement on Twitter:

Share the news about claiming your free tokens!

Tweet: Claimed my free tokens on Binance! 🌟 #BinanceFreeTokens #Crypto

Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

Interact with the Contract:

Once your wallet is connected, go to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

Act swiftly and enjoy your tokens! This limited-time offer won't be available forever. 🌐💰


Some chart values.yaml include comment out parameters

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one):

Version of Helm and Kubernetes:
Which chart:
Charts which have some comment out parameters, such as stable/grafana, stable/prometheus etc.
What happened:
Take grafana as an example, it is defining the storageClass as comment out here https://github.com/kubernetes/charts/blob/master/stable/grafana/values.yaml#L84
    ## Data Persistent Volume Storage Class
    ## If defined, volume.beta.kubernetes.io/storage-class: <storageClass>
    ## Default: volume.alpha.kubernetes.io/storage-class: default
    ##
    # storageClass:

And the pvc just checking the logic as this https://github.com/kubernetes/charts/blob/master/stable/grafana/templates/pvc.yaml#L7-L10
  {{- if .Values.server.persistentVolume.storageClass }}
    volume.beta.kubernetes.io/storage-class: {{ .Values.server.persistentVolume.storageClass | quote }}
  {{- else }}
    volume.alpha.kubernetes.io/storage-class: default

If someone want to develop a dashboard for those helm charts and get values from values.yaml, it may got some problem for those comment out parameters, as those parameters cannot be got from the values.yaml and thus cannot be listed on the customized dashboard, this caused the end user cannot customize those parameters.
For such case, if encounter some deprecated parameters, the chart deploy will be failed. Take stable/grafana as example, as the customized helm dashboard cannot get the comment out storageClass from values.yaml, and this caused all of the charts cannot set the storageClass and always use default, the default volume.alpha.kubernetes.io/storage-class has been deprecated since Kubernetes 1.3, this caused all of the charts which have storageClass will always failed to deploy.
What you expected to happen:
Do not comment out parameters in values.yaml but always give some default parameters instead.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:

/cc @kubernetes/helm-maintainers any comments for this?

@gyliu513 : this is by design... they are commented out as they aren't necessary for a default deployment. If you want to specify your own storage class (or any other value), you can use the --set cli argument to override individual values, or specify your own .yaml file containing the custom config using --values cli argument.
Keep in mind that these can be repeated, and the right most parameter will supersede others (if they specify the same settings).

@dmccaffery thanks, but the problem is that if I want to read those parameters from dashboard, then I cannot get the parameters which are comment out, thus the dashboard will have no chance to set those parameters, but with CLI, I can always use --set to set the parameters. Any comments for how to handle the case if I read the values.yaml and how to get the values which are comment out?

Any comments for how to enable the dashboard to get those commented out keys from values.yaml? @kubernetes/helm-maintainers

We've talked about duplication of documentation in NOTES.txt, readme.rst and values.yaml... I'd kind of like to see the possible values with comments stored in a machine readable format somewhere and generate parts of NOTES/readme from it. Would also allow a web ui and cli to display better options too. Someone should probably raise this issue again at the dev meeting?

@kfox1111 is there a link where can I put this to the agenda of dev meeting? Want to raise this issue so that we can fix this soon.

@gyliu513 some discussion in this issue: https://github.com/kubernetes/charts/pull/804
Not sure where else.

@kfox1111 seems there are some misunderstanding here, my issue is as following: I have a customized kubernetes UI which can enable end users configure the parameters from values.yaml, but if the parameters comment out in values.yaml, then I will not able to display them on UI and thus user is not able to configure those parameters, this will cause some issues, such as storageClass, as I cannot enable end user to configure storageClass in the dashboard, then the chart will always use volume.alpha.kubernetes.io/storage-class: default, this will cause some errors when using kubernetes 1.6/1.7 as the parameter volume.alpha.kubernetes.io/storage-class: default has been deprecated.

+1 to @kfox1111, I want to see a javadoc style format adopted for values.yaml so that all values and associated documentation is machine readable. Once we have this it can pave the way for supporting values in Monocular (https://github.com/kubernetes-helm/monocular/issues/245) as well as other tools.

@gyliu513 I guess I'm arguing what you want to do is a subset of a greater problem of needing a way to get possible values, not just the defaults to users and software. Readme/notes are just another relay mechanism today that isn't very maintainable/easy to use.

@kfox1111 yes, that make sense, thanks.

@gyliu513 @kfox1111 closing this particular issue, but if it would be good to chat more about a machine-readable values.yaml format in a future charts meeting (see #charts in slack) or sig-apps meeting.


fix: Updated the move lines of keymaps for mac

Issue
The <A-j> and <A-h> couldn't working in mac.

Description
When I  tried to use both of keymap in mac,
<A-j>, I got this symbol ∆
<A-k>, I got this symbol ˚

Snapshot

Before
After

The move lines keymaps wasn't working: https://github.com/LazyVim/LazyVim/assets/82575487/07c62a83-404f-405a-ab76-3f8a54a76cb1
The move lines keymaps is working: https://github.com/LazyVim/LazyVim/assets/82575487/6da48575-a94b-452b-b98a-c3ba52dccb2a

What's changed
Added the compatibility keymaps into default keymaps.lua.

This might be terminal emulator related, as being on MacOS while using WezTerm I have no issues with the said keybinding

@skoch13, @alesbrelih
Thank you both for your replies!!!
Indeed, I'm using iTerm2, and adjusting setting of the left option key solved the issue!!
Thanks again for your responses.


🐞[bug]: Signup page looks wierd

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Signup page CSS and alignment looks improper. Signup page generally looks similar to login page.
Expected behavior
Their should be proper gap between signup form and card
Add ScreenShots
signup -

Login page:

What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

Hello! Could you assign this to me? @KrishPatel1205

Its have been 2 days , please assign this task to me

@satyamra1 it's already been assigned.

@satyamra1 it's already been assigned.

When you assigned me?


Use correct tag for launcher image

Use the launcher image and tag, as specified on the commandline, instead
of a hardcoded launcher image.
Fixes #245

Functional tests are passing.

Regression tests are failing because not all references to NewTemplateService have not been updated to take multiple arguments.
NewTemplateService is referenced quite a few places.
grep -r -n "NewTemplateService" .
./cmd/virt-controller/virt-controller.go:52:	templateService, err := services.NewTemplateService(*launcherImage)
./pkg/virt-controller/watch/migration_test.go:81:		templateService, _ := services.NewTemplateService("kubevirt/virt-launcher")
./pkg/virt-controller/watch/migration_test.go:114:		templateService, err := services.NewTemplateService("whatever")
./pkg/virt-controller/watch/migration_test.go:781:	temlateService, err := services.NewTemplateService("whatever")
./pkg/virt-controller/watch/vm_test.go:63:		templateService, _ := services.NewTemplateService("kubevirt/virt-launcher")
./pkg/virt-controller/watch/vm_test.go:96:			temlateService, err := services.NewTemplateService("whatever")
./pkg/virt-controller/watch/vm_test.go:150:			temlateService, err := services.NewTemplateService("whatever")
./pkg/virt-controller/services/vm_test.go:56:		templateService, _ := NewTemplateService("kubevirt/virt-launcher")
./pkg/virt-controller/services/template.go:144:func NewTemplateService(launcherImage string) (TemplateService, error) {
./pkg/virt-controller/services/template_test.go:37:	svc, err := NewTemplateService("kubevirt/virt-launcher")

Is this patch a stop gap to moving the migration script into a separate container? It seems a bit weird to be overloading the virt-handler container with this logic, but it's possible there is some context to this I'm missing.

@davidvossel There is no real reason. I think splitting it makes sense.

this looks good to me. we can address splitting out migrations into a separate container later.


Removed typo/unnecessary text from local-accounts.md

@AndreaBarr The PR that introduces minor typo fixes is available for quick review & merge. Thanks.


Gzip compression not actually turned on

I noticed that gzip compression was not turned on for ngnix. There is a missing statement, in particular:
gzip_proxied any;

This is because any Heroku instance is proxied, so ngnix will ignore gzip compression directives unless this statement is present.
Thanks,
Ava

Oops, I guess I never really saw this because I was running everything through CloudFlare. I assume nginx will still respect the Accept-Encoding header even with that parameter set to any? I'll test that out this weekend and make the change unless you've already tested it.


Enable Hypershift

Description
Airflow to build a new Management and Hosted cluster on ROSA. Cleanup functions are not included, so have to do manually before rebuild.
TODO:
probably part of another PR
cleanup

Destroy Hosted cluster
Delete created S3 storage bucket
Finally delete management cluster
Reusable management cluster

platform connector

Hosted cluster

AWS security group changes

Hosted cluster

Benchmarks

Management cluster

Fixes
Tested a regular ROSA 4.10 build as well as Hypershift build.

Codecov Report

Merging #126 (794d68f) into master (a12665a) will increase coverage by 0.51%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #126      +/-   ##
==========================================
+ Coverage   93.42%   93.94%   +0.51%
==========================================
  Files          23       24       +1
  Lines         898      974      +76
==========================================
+ Hits          839      915      +76
  Misses         59       59

Flag
Coverage Δ

gha
93.94% <100.00%> (+0.51%)
:arrow_up:

python-3.9
93.94% <100.00%> (+0.51%)
:arrow_up:

unit
93.94% <100.00%> (+0.51%)
:arrow_up:

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

Impacted Files
Coverage Δ

dags/openshift_nightlies/dag.py
97.16% <100.00%> (+0.36%)
:arrow_up:

dags/openshift_nightlies/models/release.py
94.87% <100.00%> (ø)

dags/openshift_nightlies/tasks/benchmarks/e2e.py
91.93% <100.00%> (ø)

dags/openshift_nightlies/tasks/index/status.py
96.55% <100.00%> (+0.12%)
:arrow_up:

...t_nightlies/tasks/install/hypershift/hypershift.py
100.00% <100.00%> (ø)

dags/openshift_nightlies/util/executor.py
94.73% <100.00%> (+0.61%)
:arrow_up:

dags/openshift_nightlies/util/manifest.py
97.14% <100.00%> (+0.31%)
:arrow_up:

dags/openshift_nightlies/util/var_loader.py
96.15% <100.00%> (+0.23%)
:arrow_up:

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 a12665a...794d68f. Read the comment docs.


Gathering all changes in one place

Fixes and new tickers supported https://github.com/sciris/mpld3/pull/7
Testing added using a headless browser (puppeteer) https://trello.com/c/ZM1mcxnb/6-automated-testing-of-mpld3-outputs
Bug fixes "Legend Scatter test - legend dots no longer show up"

@parhammmm do you want to resolve the build errors? seems mostly RGB vs HTML color notation...
https://travis-ci.com/sciris/mpld3/jobs/182113639

@cliffckerr sure


Something strange with default in import useRelay from 'react-router-relay';

I'm started develop relay-northwind-app example app about 10 days ago.
All works perfectly before I make it public and got responses that it does not work. Crashed with  the following error:

It's because of https://github.com/nodkz/relay-northwind-app/blob/dd59c2e55d899dd8cdfb2d36571b5c4f4f5418d5/.babelrc#L28
react-router-relay exposes an ES2015 module build. If you're using webpack 2, but have module transpilation turned on in Babel, you're going to get the above error.
I added that invariant in React Router exactly to catch this case.

After reinstall node_modules from scratch I got same error. react-router, react-router-relay did not change. So I think that was changed some internal babel dependency (now will use shrinkwrap).
So problem soved by
applyRouterMiddleware(useRelay.default ? useRelay.default : useRelay

... got answer! @taion thanks!

@taion definitely needs an ability to setup env via babel-loader options: https://github.com/babel/babel-loader/issues/283

Personally, I just don't transpile modules for my Node scripts in projects where I use webpack 2. Actually I don't really use Babel there at all in many cases given the broad ES2015 support in Node 6.

Did you somehow alive import/export for Node 6?
How I know V8 does not support it yet. And Chakra is not my way.
PS. I'm using HMR on client/server, so for proper work of babelRelayPlugin on my code under webpack, I should use babel for it. Otherwise, needs to rewrite all app from import to require (but this is crazy).

Oh, in that case consider using a server webpack bundle perhaps.

Already use it.
Anyway, all works now. Will wait for V8 with imports, for removing applyRouterMiddleware(useRelay.default) hack with my build routine.
Thanks!

You could also drop module from mainFields in your webpack config. As-is, everything that actually has an ES module build under module will break in the same way if you try to use default imports.

I'm seeing this as well with Webpack 2. I have module compilation disabled in my babel-loader config: https://github.com/sfu/student-dashboard/blob/router/webpack.config.js#L53
Any ideas?

You need to disable the use of .babelrc in your Babel loader.

Well, that was easy. Thanks!


Get-AzureADUser -All seems not to be worked

I think that -All option of "Get-AzureADUser" is not worked.

You have to Type in a Boolean, so please use $true :)

Please use this command as Get-AzureADUser -All $true.
Proceeding to close this issue #please-close.
Feel free to reopen the issue should you have any followup on this issue.


Import resources

Hi, I am considering using this provider to streamline some processes we're handling manually, however, I can't find any docs regarding importing existing infrastructure. Has anyone imported existing resources before? I'm quite new with terraform so any help would be greatly appreciated.

@Taoquitok do you have any tips or suggestions on how to connect new apps to existing publishers or is that not possible as I can't get the publishers into my tf state?

New private apps with existing publishers should be easy enough as there's a publishers data source
You would need to use the filter to limit down to just the one you need, or collect all and use locals or inline logic in the resource to filter down to the appropriate publisher.
I haven't used the data source for publishers so below is an educated guess based on the docs schema for data source + new private all resource, but I'd suspect this either works, or is close to working
data "netskope_publishers" "example" {
  filter = "publisher_name your_publisher and registered eq true"
}

resource "netskope_privateapps" "PrivateApp" {
            app_name = "Eaxmple-Private-App"
            host     = "site1.example.internal, site2.example.internal"

            protocols {
                type = "tcp"
                port = "22, 443, 8080-8081"
            }

            protocols {
                type = "udp"
                port = "194"
            }

            publisher {
               # For the data reference, as it's a list of objects at the ".publishers" level
               # I'm assuming you need to [0] index... but you'll need to test to confirm this
                publisher_id   = data.netskope_publishers.example.publishers[0].id
                publisher_name = data.netskope_publishers.example.publishers[0].name
            }
}


Pytesseract is not sufficient for reliable results

Pytesseract struggles with a lot of invoices, some very big clear text are unable to be read.
This is somewhat addressable by doing some preprocessing in cv like adding blurs, threshold, but requires such an enormous amount of preprocessing to the point where some other invoices might start failing if they go through the same preprocessing.
EasyOCR picks it up fine, but is incredibly slow. I'd be happy to provide some working and non working examples privately.

The text is super clear, but i suspect tesseract struggles with the surrounding border and background. I'm able to remove most of it with preprocessing, but again, then pytesseract performs worse on some other invoices.

@viktor2097 try this one https://scandocflow.com/pdf-extraction/invoice

@viktor2097 try this one https://scandocflow.com/pdf-extraction/invoice

I'm not sure how this is relevant. EasyOCR works fine, and I can extract the data fine with Google Cloud Vision, pytesseract however struggles to extract proper data, since this project uses pytesseract, maybe it's something the owner is interested in solving since these are pretty standard invoice documents.

@viktor2097 That's a great idea. I'm also trying to replace the Pytesseract sections with EasyOCR.

@viktor2097 That's a great idea. I'm also trying to replace the Pytesseract sections with EasyOCR.

@LordDraagonLive Let me know once you have done 🥇  :)

I have been working on trying to integrate EasyOCR into InvoiceNet and basically provide an option to choose between tesseract and easyocr. However, it looks like EasyOCR doesn't return bounding box coordinates at the word level and instead returns bounding boxes for phrases. The attend-copy-parse framework requires bounding box information at the word level for accurate n-gram generation which tesseract does by default.
I'm not too familiar with the EasyOCR API so if there's a way to get bounding box information at the word-level, please let me know.

@naiveHobo yeah I tried that and faced the same issue and it was extremely slow with easyocr. I just edited the code to use Azure computer vision for now to improve the ocr part a bit and it seems to work.

@naiveHobo yeah I tried that and faced the same issue and it was extremely slow with easyocr. I just edited the code to use Azure computer vision for now to improve the ocr part a bit and it seems to work.

I came to the same conclusion, preparing training data takes way too long with EasyOCR ( We're talking hours with 800 invoices ) running on a 3090.
I managed to edit the code to use Azure instead too. It would be kinda cool if you had the option out-of-box to use a third party OCR engine like Azure or Google Cloud Vision. It's a necessity right now if you want any sort of reliable result good enough for production.

@naiveHobo yeah I tried that and faced the same issue and it was extremely slow with easyocr. I just edited the code to use Azure computer vision for now to improve the ocr part a bit and it seems to work.

I came to the same conclusion, preparing training data takes way too long with EasyOCR ( We're talking hours with 800 invoices ) running on a 3090.
I managed to edit the code to use Azure instead too. It would be kinda cool if you had the option out-of-box to use a third party OCR engine like Azure or Google Cloud Vision. It's a necessity right now if you want any sort of reliable result good enough for production.

I managed to edit the code to use Azure instead too.

I'd definitely merge it if you open a PR :eyes:

I managed to edit the code to use Azure instead too.

I'd definitely merge it if you open a PR :eyes:

@wbtan7 added support for AWS Textract as an alternate OCR option in #56. Install boto3 and setup your AWS credentials according to the documentation to use Textract!

@wbtan7 added support for AWS Textract as an alternate OCR option in #56. Install boto3 and setup your AWS credentials according to the documentation to use Textract!


Changes after the last AV WG meeting.

Signed-off-by: Robert Trypuz<EMAIL_ADDRESS>

@kulicki, could you approve this PR, please.


Month order in plots not alphabatically

Make an annual cycle function with the possibility to add errorbars

merged in #328


VnV Plan Review

Hi Dr. Smith,
Please review my VnV Plan
Thanks,
Hunter

Hunter_VnVPlan.pdf

Addressed with 5cefbf5 (subsequent commits are for formatting). Thanks again!


MaxListenersExceededWarning: Possible EventEmitter memory leak detected

Output from a unit test:
  ExtHostSearch
    File:
      ✔ no results
      ✔ simple results
      ✔ Search canceled
      ✔ provider returns null
      ✔ all provider calls get global include/excludes
      ✔ global/local include/excludes combined
      ✔ include/excludes resolved correctly
      ✔ basic sibling exclude clause
      ✔ include, sibling exclude, and subfolder
(node:66710) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 exit listeners added to [process]. Use emitter.setMaxListeners() to increase limit
(Use `Code - OSS Helper (Renderer) --trace-warnings ...` to show where the warning was created)
      ✔ multiroot sibling exclude clause
      ✔ max results = 1
      ✔ max results = 2
      ✔ provider returns maxResults exactly
      ✔ multiroot max results
      ✔ works with non-file schemes

/duplicate https://github.com/microsoft/vscode/issues/192432

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


TSS/1898: Updating dependabots

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

Project coverage is 83.56%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #165   +/-   ##
=======================================
  Coverage   83.56%   83.56%
=======================================
  Files          78       78
  Lines        2257     2257
=======================================
  Hits         1886     1886
  Misses        371      371

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


Make the various Guids used in the debug engine unique from PTVS

Adds the debug guids to the central guid file, and ensures they're unique from the PTVS guids.
Fixes #147

Hi @paulvanbrenk, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Paul van Brenk). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

LGTM. Based on the change to CRLF, I assume you intentionally change the line endings on this one?
https://github.com/Microsoft/nodejstools/pull/176/files?w=1 without line endings

Actually, one of the GUIDs is still the same:
A241707C-7DB3-464F-8D3E-F3D33E86AE99

Yes, the RemoteDebugPortSupplier is the same, that wasn't changed in PTVS either. and they don't collide. So I kep that the same.

And VS changed the line endings for me..


Feat/create sqlalchemy orm models for clide cdms

Creates sqlalchemy models for CLIDE CDMS using sqlacodegen.

Resolves #9


Users should have more control over chunk distribution

Users should be able to choose between assigning chunks towards the front or back of the available date range or evenly.

I don't actually want to do this. The user can control chunk distribution by manually chunking tasks (upon the resolution of #19) and shouldn't need this capability. Besides, it would be a lot of work.


Adapt Tutorials to the new organization of Luos engine repo.

In this branch, I update the link to work with the actual repo name on the main branch. We will rename the Luos repo into Luos_engine. No big deal because GH will redirect, but we will have to update it again.
Also in this branch, I slightly updated the way we talk about the tech by making a difference between Luos and Luos engine.
And I rework the Bootloader and FreeRTOS tutorials.
PLEASE DO NOT MERGE THIS BRANCH SINCE THE LUOS REPO IS NOT READY

This branch has to be merged for 2.4.0

I (think I) resolved the conflict, however I'd like you @K0rdan to review the sidebar and particularly the file sidebarsTutorials.js please, because it was a mess.


Travis CI is not running style transfer unit tests

Running the style transfer unit test in Travis CI causes a core dump. The likely cause is that it's running out of memory. Travis containers only have 4 GB of RAM which is the minimum allowable for style transfer.

4 GB should be sufficient for style transfer; we should not crash in this case. Relabeling as a bug.

Just to be totally clear the travis documentation claims that Ubuntu Trusty containers have 4GB of memory. I have not verified that's what we are getting.
I think the deeper issue here is that we just don't know if it has too little memory or not. Currently the Style Transfer Toolkit does not verify the system has the minimum amount of memory. I created #1040 to track that issue.

As of #2063 test_one_shot_object_detector.py has also been removed from Travis CI.


Fixed file types

Since version 1.1.0 the file type of the server is .tar.gz and not .zip. This update now works with every version since 1.1.0.

I forgot to change the file extension from .zip to tar.gz. This pull request fixes this error.

zip extension at the end

Thank you


🐛[Bug Report]: Sending in v.2.12.3 Not Working [Send Crash]

Device model
Google Pixel 4a
OS version
Android 14 | Level 34 (Default)
Which version of Litewallet are you running?
Other
Describe what happened.

Fetch LTC Address
Enter the LTC into the Send tab
Try to send
Wait for send
Transaction Rejected Toast

Add any notes you would like to add.
Team Dir of Clips: https://drive.google.com/file/d/12y2RfFUsBkLODBe_ympMZjTKFPAoMYLK/view?usp=drive_link
Screen capture

Clip

Resolved


Swap old networkx usage to retworkx

Summary
Since NetworkX 2.8, calling networkx.adjacency_matrix issues a
FutureWarning that causes the documentation build to fail.  There's no
reason for us to be using networkx since we have retworkx now.

Details and comments
NetworkX 2.8 released on the 9th of April and started emitted this FutureWarning, but we've fortunately not had any CI failures with this yet because our pip cache is still valid and pulling version 2.6.3.  Kevin's tox environment for the docs pulled the new version in and caused a failure, though.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 83.969%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
54238

Relevant Lines:
64593

💛  - Coveralls