python 3.8

fixes #197

@endrift seems like python's dev.msi for python 3.8 does not include libpython.a only python.lib, any ideas on how to workaround that?

It seems like there might be different rules for linking against libpython for 3.8: https://docs.python.org/3/whatsnew/3.8.html#debug-build-uses-the-same-abi-as-release-build

@christopherhesse not sure if it'll still link properly, but you might be able to just rename it to libpython.a. They're in the same format, it's just the lib file is targeted for MSVC.

here's a converted one from https://code.google.com/archive/p/lib2a/ in case that does not work convert.zip

another option:
from https://stackoverflow.com/questions/11793370/how-can-i-convert-a-vsts-lib-to-a-mingw-a
curl -O https://www.python.org/ftp/python/3.8.2/amd64/core.msi
7z x core.msi
apt-get install --yes apt-get install mingw-w64-tools
gendef python.dll
/usr/bin/x86_64-w64-mingw32-dlltool --as-flags=--64 -m i386:x86-64 -k --output-lib libpython.a --input-def python.def

Thanks for the suggestion @endrift!  I couldn't get that to work but the dlltool thing at least finished compiling.


Add tests for SubgroupDispatch

SubgroupDispatch is a new feature in v1.1 adding a new capability, new execution models, and new instructions.
Extended the testing framework to allow testing with different SPIR-V versions.
@antiagainst @AWoloszyn @dneto0 @Qining PTAL.

-1

Missing OpGetKernelLocalSizeForSubgroupCount in OpcodeRequiresCapabilities.cpp
Two places where const should be used
Please avoid increases in complexity in test code that have little to no benefit.

Addressed feedback so far.

-1.  Please follow the predominant pattern in the rest of the tests.

+2

Squashed and merged to master via 2ea54f5.


Reshape ZeroStage=0 FP16 Checkpoint

What is the best way for reshaping a checkpoint trained with zero stage = 0 & fp16?
I see two options:
a) Continue training with zero stage 1 for 1 step & adapt this PR to work with fp16
b) Adapt the script here to work without the need of zero ckpts; The difficult part will just be reshaping the optimizer states in the mp_rank files
Maybe @tjruwase could give me a quick hint if a) or b) makes more sense before I waste my time? Thanks!

@Muennighoff, thanks for your question. Can you please clarify a bit more because zero_stage=0 actually disables ZeRO and is pure DDP. The only reshaping needs that I can imagine in such cases will be due to tensor parallelism or pipeline parallelism.

@Muennighoff, thanks for your question. Can you please clarify a bit more because zero_stage=0 actually disables ZeRO and is pure DDP. The only reshaping needs that I can imagine in such cases will be due to tensor parallelism or pipeline parallelism.

Yes there's no ZeRO used only TP (based on Megatron) & PP. The TP is based on the Megatron-DS implementation. Specifically, I am looking at a TP=4, PP=4 model. Based on my understanding I need to change the layer files due to TP & the  mp files due to TP & MP.
How would you go about it?

Great. Thanks for the clarification. Also, do you need reshaping of just the model weights or also of the optimizer state? The reshaping logic you reference is split across bigscience/megatron-deepspeed and deepspeed, very new, and only tested with bf16 + pipeline parallelism + zero stage1.
In terms of your proposed options, I feel (b) is more straightforward and thus easier. Option (a) will require (1) creating zero ckpts only for the sake of reshaping and (2) porting the reshaping changes in the bf16_optimizer into fp16 zero_stage_1 optimizer. Although, option (b) requires changes to the reshaping script, I think those changes will be useful anyways for the non-zero training scenarios such as yours. Does that make sense?
Perhaps @stas00, who is the co-author of the reshaping feature, might have some thoughts as well.

Yes, I need continue training in the new shape, so I think I will also need to reshape the optimizer states. I will continue training with zero stage 1, however.
Thanks for your thoughts! I will work on (b) then. I think I only need to figure out how to merge the optimizer states in the mp_rank files correctly.

Yes, once bf16/z0 PR is merged we can look at fp16/z0 next.
The other approach is to:

start with a random optim states
run for some steps with LR=0 to let the optimizer catch up
resume training with normal LR

The details and math of how many steps to run are in the 104B chronicles, I can dig up the link if you want to explore this option.


add new cpu plugin options introduced in collectd 5.5

This makes new cpu options configurable, see https://collectd.org/documentation/manpages/collectd.conf.5.shtml#plugin_cpu

Can you add a quick spec test as well?

Thanks for the contribution!


Crash with message: "CanvasKitError: Failed to initialize CanvasKit. CanvasKit.MakeGrContext returned null"

Copied from internal bug b/267073848:
Stack trace:
Translated error message:
   CanvasKitError: Failed to initialize CanvasKit. CanvasKit.MakeGrContext returned null.
Deobfuscated stack trace:
  [Mapped Dart].wrapException(/v2/sdk/lib/_internal/js_runtime/lib/js_helper.dart:1128)
  [Mapped Dart].throwExpression(/v2/sdk/lib/_internal/js_runtime/lib/js_helper.dart:1159)
  [Mapped Dart]._createNewCanvas(/flutter_web_sdk/lib/_engine/engine/canvaskit/surface.dart:325)
  [Mapped Dart].Surface.createOrUpdateSurface(/flutter_web_sdk/lib/_engine/engine/canvaskit/surface.dart:174)
  [Mapped Dart].Surface.acquireFrame(/flutter_web_sdk/lib/_engine/engine/canvaskit/surface.dart:109)
  [Mapped Dart].Rasterizer.draw(/flutter_web_sdk/lib/_engine/engine/canvaskit/rasterizer.dart:27)
  [Mapped Dart].CanvasKitRenderer.renderScene(/flutter_web_sdk/lib/_engine/engine/canvaskit/renderer.dart:349)
  [Mapped Dart].render(/flutter_web_sdk/lib/_engine/engine/platform_dispatcher.dart:645)
  [Mapped Dart].render(/flutter_web_sdk/lib/ui/window.dart:107)
  [Mapped Dart].RenderView.compositeFrame(/dart/flutter/lib/src/rendering/view.dart:240)
  [Mapped Dart].rootNode(/dart/flutter/lib/src/rendering/object.dart:939)
  [Mapped Dart].renderView(/dart/flutter/lib/src/rendering/binding.dart:233)
  [Mapped Dart].RendererBinding.drawFrame(/dart/flutter/lib/src/rendering/binding.dart:520)
  [Mapped Dart]._WidgetsFlutterBinding&BindingBase&GestureBinding&SchedulerBinding&ServicesBinding&PaintingBinding&SemanticsBinding&RendererBinding&WidgetsBinding.drawFrame(/dart/flutter/lib/src/widgets/binding.dart:865)
  [Mapped Dart].RendererBinding._handlePersistentFrameCallback(/dart/flutter/lib/src/rendering/binding.dart:381)

We don't have a repro for this, but given the very specific error message, I wonder if it is already actionable?

Do we have any information about the device/OS/browser?

There seems to be a bit of information in the internal crash reporting system. See my update on b/267073848.

🤞  this PR is fixing this: https://github.com/flutter/engine/pull/42672

I have a PR which fixes this, but it needs a better test. Will update when the PR is ready for review.


Updated eslint and fixed some dev dependencies

Current coverage is 78.81% (diff: 100%)

Merging #50 into master will not change coverage

@@             master        #50   diff @@
==========================================
  Files            85         85
  Lines          2223       2223
  Methods          69         69
  Messages          0          0
  Branches        194        194
==========================================
  Hits           1752       1752
  Misses          364        364
  Partials        107        107

Powered by Codecov. Last update 1a750cf...d27e889


Perform search by url query parameter

Is it possible to perform a search by a url query parameter with Lunr?
And to automatically update the url to include the search term as a query parameter?
This would allow to "link to a search".

@inwardmovement This is more a matter of the UI than something lunr itself handles.  I'm sure this is possible with whatever UI framework you're using, or even just vanilla JavaScript - typically I think you'd update the URL fragment when doing a search, and check the URL fragment upon application load to implement this kind of functionality.

Yes we should be able to do so on the application level you are right — I was just wondering if Lunr could "facilitate" this haha. Thanks for the answer!


UI: Properties with complex types rendered as [Object]

Now some properties are real complex types rather than just strings, the properties panel needs to reflect this and iterate the object to show the real value rather than [object]

^ this is in the details tab. We can hide all complex values in the details tab if we want, but if i'm looking for the details I think i'd want to see this?

I can't think any reason a  researcher would want to know the redirect URLs, or many of the other fields. As an admin/dev, to debug, maybe... but they could go to cosmos.
We need a UI Schema for the forms, to sort ordering, is it possible to have a UI schema for forms, cards, and details tab?

Yea the UI schema for the form is still to-do, we could look to extend that to include a list of props to show on the properties panel, and then use the template to drive what we see on those panels.
The card is maybe a bit trickier due to the space available.

For the card, the things that spring to mind are VM related, like CPU cores, RAM etc. (actually a single property?).


feat: Add option to pass arguments to func start command

👋
Fixes #289
Added the --func-args option that allows passing additional arguments to the func start command.
I verified that this will enable scenarios like attaching a debugger to the func host.
I verified that this enabled attaching the VS Code debugger to the func host when I used the following command:
swa start http://localhost:3000/ --app-location=frontend --run "npm run start" --api api --func-args="--javascript --language-worker -- \"--inspect=9229\""

LGTM


Voting a Feature Request Before Create Work

My suggested workflow is
Feature Requests -> up-voting 20 -> Backlog -> Milestone -> Work on it -> PR -> Merge
Some practice:

https://github.com/microsoft/vscode/wiki/Issues-Triaging#managing-feature-requests
https://github.com/microsoft/vscode/issues/162138

I think there should be no problems to implement this as external tool using gitea webhooks listening on issue changes and adding to backlog using board API

Except I don't think the 20 upvotes is feasible for Gitea.
That might work for a client-side application like VS Code with a much larger audience where users are likely to look through issues and upvote them.
But in general, people mostly expect server-side apps to just work, where they only file issues if they have an issue themselves and or if they have an actual idea what to propose.
From what I've seen so far if we do that, only the enormously big issues such as CI (#13539) and federation (#18240) will be merged.
Not even the plugin system (#20126) or "GitHubs useful functionality" (#8659) would then be implemented since they don't get enough likes.
I don't think we can afford to do that.

If that is meant for using in Gitea development process I don't see much point in that as there won't be much point in voted backlog if anyway noone volunteers to implement that... I think most contributors implement what they find interesting or need themselves not based on request popularity.


Add opportunity to set message template parameters for parametrized Replay

Description
Add opportunity to set message template parameters for parametrized Replay
https://www.figma.com/design/lwL5jneveZczAj5RLcLFH2/DIAL-Chat?node-id=12113-75057&t=vF0pV3vNbBngaC2Q-1
Confidential information

[X] I confirm that do not share any confidential information

Verified the first iteration on staging env. Found issues were discussed with the developer. Have not been reported yet.


Wholesale updates for tritonserver version

General refresh of some out-of-date tritonserver image references

@gpucibot merge


Credential problem for usecase of Github Actions OIDC federated IAM Role

This article
AWS federation comes to GitHub Actions
explains we can use OIDC federated IAM Role in github actions workflow.
But now I got this error with aws-actions/aws-codebuild-run-build@v1.
Error: No credentials. Try adding @aws-actions/configure-aws-credentials earlier in your job to set up AWS credentials.

Simply comment out this assert, it works well.
https://github.com/aws-actions/aws-codebuild-run-build/blob/8945a85e94fd346070a0d8a28da303dbdd80b4bf/code-build.js#L228_L230
 assert(
    codeBuild.config.credentials && cloudWatchLogs.config.credentials,
    "No credentials. Try adding @aws-actions/configure-aws-credentials earlier in your job to set up AWS credentials."
  );

so for the moment I suggest just log this not assert but warning.

Here is a workaround until this issues is addressed. Not super clean but it does get sts creds with assume-role-with-web-identity and works with aws-codebuild-run-build. I followed the same post AWS federation comes to GitHub Actions for setting up my OIDC provider and IAM role with federated trust policy.
- name: Get AWS Credentials Using OIDC
  id: aws_sts_creds
  run: |
    export AWS_ROLE_ARN=arn:aws:iam::0123456789012:role/ExampleGithubRole
    export AWS_WEB_IDENTITY_TOKEN_FILE=/tmp/awscreds
    export AWS_DEFAULT_REGION=us-east-1

    curl -H "Authorization: bearer $ACTIONS_ID_TOKEN_REQUEST_TOKEN" "$ACTIONS_ID_TOKEN_REQUEST_URL&audience=sigstore" | jq -r '.value' > $AWS_WEB_IDENTITY_TOKEN_FILE

    aws sts assume-role-with-web-identity \
      --role-arn $AWS_ROLE_ARN \
      --role-session-name github-actions \
      --web-identity-token file://$AWS_WEB_IDENTITY_TOKEN_FILE \
      --duration-seconds 1000 > /tmp/aws-creds

    export AWS_ACCESS_KEY_ID="$(cat /tmp/aws-creds | jq -r ".Credentials.AccessKeyId")"
    export AWS_SECRET_ACCESS_KEY="$(cat /tmp/aws-creds | jq -r ".Credentials.SecretAccessKey")"
    export AWS_SESSION_TOKEN="$(cat /tmp/aws-creds | jq -r ".Credentials.SessionToken")"

    echo ::add-mask::$AWS_ACCESS_KEY_ID
    echo ::add-mask::$AWS_SECRET_ACCESS_KEY
    echo ::add-mask::$AWS_SESSION_TOKEN

    echo ::set-output name=aws_access_key_id::$AWS_ACCESS_KEY_ID
    echo ::set-output name=aws_secret_access_key::$AWS_SECRET_ACCESS_KEY
    echo ::set-output name=aws_session_token::$AWS_SESSION_TOKEN
    echo ::set-output name=aws_default_region::$AWS_DEFAULT_REGION

- name: Configure AWS Credentials
  uses: aws-actions/configure-aws-credentials@v1
  with:
    aws-access-key-id: ${{ steps.aws_sts_creds.outputs.aws_access_key_id }}
    aws-secret-access-key: ${{ steps.aws_sts_creds.outputs.aws_secret_access_key }}
    aws-session-token: ${{ steps.aws_sts_creds.outputs.aws_session_token }}
    aws-region: ${{ steps.aws_sts_creds.outputs.aws_default_region }}

@celliott
Thank you for this workaround helping us a lot!
terraform nice. I used too.

@takaaki-inada I'm glad to help. I spent a little more time today and found a much cleaner way to use oidc to get sts aws creds. This has been tested with<EMAIL_ADDRESS>name: GitHub Action AWS OIDC STS Creds
on:
  workflow_dispatch: {}
  pull_request: {}
concurrency: ${{ github.repository }}-github-action
env:
  AWS_ROLE_ARN: arn:aws:iam::0123456789012:role/ExampleGithubRole
  AWS_WEB_IDENTITY_TOKEN_FILE: /tmp/awstoken
  AWS_REGION: us-east-1
jobs:
  plan:
    runs-on: ubuntu-latest
    permissions:
      id-token: write
      contents: read
    steps:
      - name: Checkout code
        uses: actions/checkout@v2
      - name: Set Environment Variables
        run: echo "REPO_NAME=${GITHUB_REPOSITORY#*\/}" >> $GITHUB_ENV
      - name: Get AWS Credentials Using OIDC
        run: |
          curl -H "Authorization: bearer $ACTIONS_ID_TOKEN_REQUEST_TOKEN" \
            "$ACTIONS_ID_TOKEN_REQUEST_URL&audience=sigstore" | jq -r '.value' > $AWS_WEB_IDENTITY_TOKEN_FILE
      - name: Configure AWS Credentials
        uses: aws-actions/configure-aws-credentials@master
        with:
          aws-region: ${{ env.AWS_REGION }}
          web-identity-token-file: ${{ env.AWS_WEB_IDENTITY_TOKEN_FILE }}
          role-to-assume: ${{ env.AWS_ROLE_ARN }}
          role-duration-seconds: 1800
          role-session-name: ${{ env.REPO_NAME }}-github-action
      - name: AWS Get Caller Identity
        run: aws sts get-caller-identity

@celliott
Simple! LGTM
I learned "aws-actions/configure-aws-credentials" supports web-identity-token-file.

@takaaki-inada web-identity-token-file hasn't been released in a version yet. Please note that the action is this example is pinned to master in this example.


theres a bug that it automatically closes and it wanted me to change into windows 7

Describe the bug
A clear and concise description of what the bug is.
Desktop (please complete the following information):
OS & version [e.g. Windows 10, 1822]:
MSFT Build of OpenJDK Version:
The output from java --version
To Reproduce:
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots, Logs etc
If applicable, add screenshots, logs to help explain your problem.
Additional context
Add any other context about the problem here.

Can you please provide a description of the problem?

Closing due to no additional info.


Tiles not loading outside of metadata bounds

Fixes issue #1318 where tiles outside of metadata bounds are not loading. This fix allows over zooming so that map features that are not on higher zoom levels will still show from their lower zoom level tiles.
For example, the offline mbtiles file might include world tiles in z1 only, but when you zoom in farther those tiles are blank unless you move to the metadata bounds. With this fix, if you zoom into an area outside of the bounds you will get over zoom from z1 which makes for a much nicer looking map without blank tiles.

Thanks for working on this. The behavior you describe sounds like the correct one, i.e., only show data from inside the bounds.
@louwers you might want to coordinate with @HarelM on how this behavior is done in MapLibre GL JS.

@wipfli Does MapLibre GL JS support reading local MBTiles?

MapLibre GL JS does not have offline mbtiles support. My think is that independent of the source - be it an offline mbtiles file or a zxy http endpoint - the behavior for unavailable tiles should be the same.

openmaptiles, which many people use to create offline mbtiles files will create a a world tile at z1 where you can see all continents, countries, etc. It makes sense to continue to show these as you zoom into different parts of the world using over zoom. Without that, an mbtiles file would need to be created with world data on every other zoom level making the file significantly larger.

I don't understand why your changes implements the behavior you say it does. Could you explain it?

This small code change will, when receiving an error for no tile, instead allow the previous zooms tile to show instead of showing blank. E.g. overzoom.

This would be a pretty significant change. Not all users will want to have this behavior in all use cases, so I think it would need to be configurable.
Please create a discussion or a design proposal so we can discuss this problem first.

This would be a pretty significant change to the tile loading behavior. Not all users will want to have this behavior in all use cases, so I think it would need to be configurable.
Please create a discussion or a design proposal so we can discuss this problem first.

That is fine. I have fixed it in my fork for my purposes.


Example in Documentation

added the first three samples for the examples (example_1.rst)

Is this the structure (chapter/images) helpful for the users
Is something too much or too less?
Look into the helper scripts docs/assets/examples named create_thumbnail_images.py and create_rst.py

there is an benchy_v2024.py that implements a yaml-doc-string to fill the example_1.rst blocks with the helper script create_rst.py
there is a README.md in ./examples documenting the steps (most of them are now automated in create_rst.py)

If I understand correctly, the create_rst.py script would primarily be used during the batch process of converting all of the existing examples to this new format.  Therefore, I wonder if it should be part of the build123d repo? Similarly with create_thumbnail_image.py, I would typically use imagemag for this type of thing (it work on batches of files).
With the benchy there are several images but it looks like only the first appears in docs.  Since creation is done manually, I wonder if the extra work of creation is justified.
What if we just start with the rst files and associated images contained in the PR?  This should help get a feel for what the docs will look like and allow others to provide feedback.

ok. I can remove the helper python scripts and the yaml files.
About the Benchy you are right,  I missed the example with more images - will be rectified.

@gumyr followed your lead.

removed the helper scripts
inserted the missing benchy images in a "poor mans gallery" hidden detail section
removed too detailed information (author, history ...)

Content-wise Concepts

Have a quick overview of an example with a main image to show the model
Include information about builder and algebra mode (symbols)
Short overview of the example in a paragraph
No too detailed information
OPTIONAL: More images in a gallery if it makes sense (maybe not for the benchy - just to show how it would work)
OPTIONAL: (to be discussed) add the examples code -> PRO: this really helps in searching for example Mesher it would also find the example benchy.py which  gives the user a great tool to quickly find a code snippet in context of the examples

Process-wise Concepts
7. Description how to add examples (in the example_1.rst AND a readme.md)
8. ˜˜OPTIONAL: support the developer/contributer to quickly generate RST from command line (REMOVED FROM PULL REQUEST)˜˜

Feel free to ask me if I should present the concept in more depth


It seems FlaskTracer does not set span.kind tag on the server span

Closing as #21 fixed this.

Is there a new release coming?

Yes! ;) This week (had expected to do it last week but was too packed)

For reference, I will be merging the v1.0.0 branch


Purge build deps to shrink the 15.3 image from 2.32 GB to 1.61 GB

For our official CircleCI Docker Convenience Image support policy, please see CircleCI docs.
This policy outlines the release, update, and deprecation policy for CircleCI Docker Convenience Images.

Description
This shrinks the size of the 15.3 image from 2.32 GB to 1.61 GB, tested locally on my M1 mac. The 15.3 image is the latest published one, 15.3-latest is one that I built from the tip of main 3 hours ago, and "take11" is with the change from this PR.
~/src/cimg-postgres $ docker image ls | awk 'NR==1 || /15.3/'
REPOSITORY                  TAG                         IMAGE ID       CREATED             SIZE
cimg/postgres               15.3-no-build-deps-take11   a7b386491c87   6 seconds ago       1.61GB
cimg/postgres               15.3-latest                 34946eada78d   3 hours ago         2.32GB
cimg/postgres               15.3                        a035a3479341   4 weeks ago         2.45GB

A drawback of this approach is that we install & remove build deps twice, once in each layer that uses them. I don't think it's a big problem given these are built relatively rarely compared to how often they're used.
Reasons
I want to shrink the postgresql image sizes. This should improve spin-up time a little, especially if the image is not in the local Docker Layer Cache. It also helps for local development.
I achieved the space savings by installing & purging build-time dependencies in the postgres and pg_cron installation steps.
Checklist
Please check through the following before opening your PR. Thank you!

[x] I have made changes to the Dockerfile.template file only
[x] I have not made any manual changes to automatically generated files
[x] My PR follows best practices as described in the contributing guidelines
[x] (Optional, but recommended) My commits are signed

@stig - ideally, we would probably want to have build deps installed and removed in one layer. were the space savings for that ever tested? if these are not being changed all that much, maybe it makes more sense to keep the extensions we include in this postgres parent/base image in one layer, especially if it ultimately does not change much


[Feature Request] Donation link option

There should be a donation option on settings > about > donate so people can donate to MD

#422


Exception in thread "main" java.lang.NullPointerException 	at ro.cs.s2.S2ProductDownloader.main(S2ProductDownloader.java:336)

I followed the installation instruction from the Readme on Windows Server 2012 R2. Java and mvn should be installed properly (I have jre and jdk 1.8)
When running the example command from the Readme I am getting the following NullPointerException:
Command:
java -jar S2ProductDownloader-1.0.jar --out D:\Products --tiles 34TFQ 35TNL --products S2A_OPER_PRD_MSIL1C_PDMC_20160103T183955_R064_V20160103T085753_20160103T085753 --store AWS
2016-08-03 09:03:43	[INFO]		Executing with the following arguments: 2016-08-03 09:03:43	[INFO]		o=D:\Products 2016-08-03 09:03:43	[INFO]		t=34TFQ 2016-08-03 09:03:43	[INFO]		p=S2A_OPER_PRD_MSIL1C_PDMC_20160103T183955_R064_V20160103T085753_20160103T085753 2016-08-03 09:03:43	[INFO]		s=AWS Exception in thread "main" java.lang.NullPointerException 	at ro.cs.s2.S2ProductDownloader.main(S2ProductDownloader.java:336)
Any ideas?
May be it has to do with the installation because I had to copy the .jar file and lib folder manuelly. During the assembly I got the warning:
[WARNING] Assembly file: D:\awsdownload\target\S2ProductDownloader-1.0-bin is not a regular file (it may be a directory). It cannot be attached to the project build for installation or deployment. [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 2.608 s [INFO] Finished at: 2016-08-03T09:08:50+00:00 [INFO] Final Memory: 20M/1182M [INFO] ------------------------------------------------------------------------

This is because you have not supplied your user name and password for SciHub.
I have made them mandatory, you can take the changes from GitHub.

Thanks a lot, that indeed solved the issue.


Batching with vLLM

Hi @noamgat,
First off, thanks for this cool repo!
I was going through the sample notebook for the vLLM integration and noticed that it doesn't implement batching, unlike the sample for the HuggingFace's transformers integration.
Is there a specific reason for this difference? Just curious to know!

Hi! The vLLM integration supports batching, it just wasn't covered in the demo to keep it simpler. PRs welcome :)
The only thing I would pay attention to, is that you want to apply the monkeypatch once, outside of the batching loop.

Also, if this PR is accepted by vLLM, it will make the integration much more elegant:
https://github.com/vllm-project/vllm/pull/1469
Please thumbsup on the PR so the vLLM team know :)

Hi @noamgat,
Thanks for replying!

The vLLM integration supports batching, it just wasn't covered in the demo to keep it simpler.

A preliminary experiment suggests that the latency scales linearly with respect to the requested batch size (I didn't check the actual batch size used by vLLM's dynamic batching). What am I missing?

Latency in seconds (y-axis) over the number of requested prompts (x-axis)

Added a batching example to the vllm sample notebook here:
https://github.com/noamgat/lm-format-enforcer/blob/main/samples/colab_vllm_integration.ipynb
I am receiving better time-per-prompt when making the jump from 1 to 20. Above the batch size, you expect to see linear results.

@noamgat, thank you for adding a batching example to the vLLM sample notebook. Can you specify which version of vLLM you used for this? I've noticed that the current commit on the main branch of vLLM already implements a similar procedure.
Regarding your observation on the improvement in time-per-prompt from batch sizes of 1 to 20: Could you share specific numbers to illustrate this? Additionally, when you mentioned expecting to see linear results for batch sizes above 20, could you share data or insights on how the performance scales for those larger batch sizes?

My timings, as seen in the example notebook after the update:
Time taken for 1 player: 1.8789570331573486
Time taken for 19 players: 6.5615339279174805. Time per player: 0.3453438909430253
I ran it several times and got consistent results.
I am not a vLLM expert so I cannot dive much deeper into vLLM, but this does show that something is working.

Thanks @noamgat! Before I close this issue, I wanted to share the results below:
I repeated the previous experiment, but this time I measured only the latency of the inference (i.e., not including the latency of Pydantic—as in the previous experiment). On Google Colab with a single T4 GPU, it seems like the latency of generating batches of sizes 1, 10, 20, 32, 64, and 100 remains linear with respect to the batch size, and we can fit the following linear line:


Identify and review services and artifacts that needs to be migrated off of Operate First

See also: https://github.com/os-climate/os_c_data_commons/issues/270

no other services identified - went thru apps


Update test3.yml

Hi @steviestainz I am looking into why this didnt process automatically now, I will let you know as soon as I can, thanks!

Hi @steviestainz I see it looks like you already completed the course successfully so I assume you must have already figured this out, I am going to close the ticket. Please let me know if I missed anything and thanks so much for participating!

Hello Arthur,
The workflow was failing during the download of yq which I realise now was because of the internet web cache issue which was in the news yesterday. It was between GitHub and wherever yq is hosted it seems.
Thanks for caring and following up!
Steve Walker
SBC PureConsult Ltd.
www.sbcpureconsult.co.uk
+44 (0) 7773 782910

From: Arthur Fewell @.>
Sent: Tuesday, June 8, 2021 11:24:20 PM
To: ModernAppsNinja/vspheretanzu101_vt7301 @.>
Cc: Steve Walker @.>; Mention @.>
Subject: Re: [ModernAppsNinja/vspheretanzu101_vt7301] Update test3.yml (#979)
Hi @steviestainzhttps://github.com/steviestainz I see it looks like you already completed the course successfully so I assume you must have already figured this out, I am going to close the ticket. Please let me know if I missed anything and thanks so much for participating!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ModernAppsNinja/vspheretanzu101_vt7301/pull/979#issuecomment-857219407, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AI3V73JQB27ID46TZSLQ57LTR2KBJANCNFSM46JTZNIQ.


NullPointerException in com.datarank.tempest.allocator.model.ModelBalancer.evaluateBalance

I am deploying a new Elasticsearch image which contains the Tempest plugin in my cluster:

Shutdown the cluster
Deployed 3 ES master nodes

I am getting the following NullPointerException on the master node and the other master nodes cannot connect to it:
[2016-03-07 18:38:08,997][ERROR][discovery.zen            ] [ibm-aug5-elasticsearch-master-wikxwmhl] unexpected failure during [zen-disco-join (elected_as_master)]
java.lang.NullPointerException
at com.datarank.tempest.allocator.model.ModelBalancer.evaluateBalance(ModelBalancer.java:78)
at com.datarank.tempest.allocator.LocalMinimumShardsAllocator.rebalance(LocalMinimumShardsAllocator.java:136)
at org.elasticsearch.cluster.routing.allocation.allocator.ShardsAllocators.rebalance(ShardsAllocators.java:81)
at org.elasticsearch.cluster.routing.allocation.AllocationService.reroute(AllocationService.java:190)
at org.elasticsearch.cluster.routing.allocation.AllocationService.reroute(AllocationService.java:161)
at org.elasticsearch.cluster.routing.allocation.AllocationService.reroute(AllocationService.java:147)
at org.elasticsearch.discovery.zen.ZenDiscovery$3.execute(ZenDiscovery.java:388)
at org.elasticsearch.cluster.service.InternalClusterService$UpdateTask.run(InternalClusterService.java:374)
at org.elasticsearch.common.util.concurrent.PrioritizedEsThreadPoolExecutor$TieBreakingPrioritizedRunnable.runAndClean(PrioritizedEsThreadPoolExecutor.java:196)
at org.elasticsearch.common.util.concurrent.PrioritizedEsThreadPoolExecutor$TieBreakingPrioritizedRunnable.run(PrioritizedEsThreadPoolExecutor.java:162)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
[2016-03-07 18:38:12,005][INFO ][com.datarank.tempest.allocator.LocalMinimumShardsAllocator] [ibm-aug5-elasticsearch-master-wikxwmhl] Rebalancing.

Just want to add this this is using the 1.0.0 version installed from here (which looks like the latest release)
https://github.com/datarank/tempest/releases

We'll need to know the version of ES too. They have changed a lot since 1.x days.

This is ES 1.7.1, but we may also be interested in using Tempest in an 2.x environment.  Is that also supported?
I see that there may have been a hot fix for this:  https://github.com/datarank/tempest/commit/b3231a98a5665bfec6a61df2af8bf6a9807a161e
How can I install that fix?
Thanks!

Fixed in 2.x which has been back ported to ES 1.x


a11y - sharing page - Minimum contrast - cms ui

Describe the bug

Some texts, boxes and placeholders don't have enough contrast - gray background with gray text, for example, isn't recommended

Expected behavior

It's not recommended to use gray background with gray text.
Respect the minimum contrast and size for any kind of text

Screenshots

Software (please complete the following information):
Volto Version 17.0.0-alpha.19
Plone Version 6.0.4

I'm excited to take on this issue as my first open-source contribution. Please consider assigning it to me. I'm eager to contribute and eager to learn!

@16achint please read and follow Contributing to Plone and First-time contributors.


Add to app engine does not report correct life cycle state during warm up

Steps to Reproduce

follow this document to setup add to app https://github.com/flutter/samples/tree/master/experimental/add_to_app/SimpleIOSExample
update the main function in this file
experimental/add_to_app/example_module/lib/main.dart

// Copyright 2019 The Flutter team. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Module Title',
      routes: {
        '/': (context) => FullScreenView(),
        '/mini': (context) => MiniView(),
      },
    );
  }
}

class FullScreenView extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('A Full-screen Flutter View'),
      ),
      body: const MiniView(showExit: true),
    );
  }
}

class MiniView extends StatefulWidget {
  final bool showExit;

  const MiniView({this.showExit = false});

  @override
  _MiniViewState createState() => _MiniViewState();
}

class _MiniViewState extends State<MiniView> with WidgetsBindingObserver {
  int count = 0;

  AppLifecycleState _lastLifecycleState;

  @override
  void initState() {
    super.initState();
    WidgetsBinding.instance.addObserver(this);
  }

  @override
  void dispose() {
    WidgetsBinding.instance.removeObserver(this);
    super.dispose();
  }

  @override
  void didChangeAppLifecycleState(AppLifecycleState state) {
    setState(() {
      _lastLifecycleState = state;
    });
  }

  @override
  Widget build(BuildContext context) {
    final mediaInfo = MediaQuery.of(context);
    print('current state $_lastLifecycleState');
    return SizedBox.expand(
      child: Stack(
        children: [
          Positioned.fill(
            child: DecoratedBox(
              decoration: BoxDecoration(
                color: Theme.of(context).scaffoldBackgroundColor,
              ),
            ),
          ),
          Positioned.fill(
            child: Opacity(
              opacity: .25,
              child: FittedBox(
                fit: BoxFit.cover,
                child: FlutterLogo(),
              ),
            ),
          ),
          Center(
            child: Column(
              mainAxisAlignment: MainAxisAlignment.center,
              children: [
                Text(
                  'Window is ${mediaInfo.size.width} x '
                  '${mediaInfo.size.height}.',
                  style: Theme.of(context).textTheme.headline,
                ),
                SizedBox(height: 16),
                Text(
                  'Taps: $count',
                  style: Theme.of(context).textTheme.headline,
                ),
                SizedBox(height: 16),
                RaisedButton(
                  onPressed: () => setState(() => count++),
                  child: Text('Tap me!'),
                ),
                if (widget.showExit) ...[
                  SizedBox(height: 16),
                  RaisedButton(
                    onPressed: () => SystemNavigator.pop(),
                    child: Text('Exit this screen'),
                  ),
                ],
              ],
            ),
          ),
        ],
      ),
    );
  }
}

run the app, and the widget is first build during initialization with lifecyclestate = null which it should be pause.

@xster

I assume you specifically meant that _lastLifecycleState is null because WidgetsBindingObserver.didChangeAppLifecycleState is never called right?

It is never called, but there is more to it.
When the binding first initialized, it will poll the state from window.initialLifecycleState which engine should attach its state to. The engine, however, doesn't seems to attach anything to it during the warm up. Otherwise, WidgetsBindingObserver.didChangeAppLifecycleState should be called.

Change reverted and is pending on https://github.com/flutter/flutter/issues/45270


"Paste" from "copy button" going wrong

Information:

Prism version: PrismJS 1.26.0
Plugins: Toolbar and Copy to Clipboard Button
Environment: Firefox Browser 96.0.1.

!
Description
The copy-paste button does not work as expected. The paste will show the numbering in the opposite order. This was always working (been using prismjs for years).
Something clearly changed to the copy button which unfortunately is causing this bug in Firefox. Edge/Chrome is working fine.
Example

Ensure that you have the latest PrismJS version which is PrismJS 1.26.0.

Add some text in the section, for example:

Test1
Test2
Test3

Click the copy button

Paste in PowerShell

You can see that it's in a different order!

Paste in Notepad

You can see that it's in the correct order.

This seems to be a general bug with Firefox's clipboard.writeText implementation. I was able to reproduce this bug on a very simple test page.
<!DOCTYPE html>
<html>
<head>
	<title>Document</title>
</head>
<body>
	<button>Click me</button>
	<script>
		document.querySelector("button").addEventListener("click", () => {
			navigator.clipboard.writeText("abc\ndef\nghi\n")
				.then(() => console.log("yay"), () => console.log("oh no"));
		});
	</script>
</body>
</html>

The problem seems with FF literally writing the given text to clipboard without accounting for Windows line ends. If the line ends are changed to \r\n everything works as expected (as in Chrome).

Since this seems like a bug in Firefox to me, I reported it. https://bugzilla.mozilla.org/show_bug.cgi?id=1753202

The discussion in the bug report I linked above points to this problem being a spec issue with Clipboard API. Since this has nothing to do with Prism and I don't think that there is a workaround we can do, I'll now close this issue. While problem persists, the fix has to be done by WC3 and Mozilla.


Bug: NotAFolder

Description
When you set File as the default file explorer, when you click on this computer, it will get the error NotAFolder!

Steps To Reproduce
When set to the default File Explorer
Delete the Files background
File Explorer opens from this PC
After a while
The NotAFolder error occurs
Files Version
3.7.11
Windows Version
10.22631.3155
User ID
No response
Log File
2024-11-15 11:18:16.7453|Information|App launched. Launch args type: ProtocolActivatedEventArgs
2024-11-15 11:18:18.2984|Information|Drive added: C:, Fixed
2024-11-15 11:18:18.3683|Information|Drive added: D:, Fixed
2024-11-15 11:18:18.4099|Information|Drive added: E:, Fixed
2024-11-15 11:18:18.4861|Information|Drive added: F:, Removable
2024-11-15 11:18:18.5409|Information|Drive added: F:
2024-11-15 11:18:19.8782|Information|SIDELOAD: Checking for updates...
2024-11-15 11:18:20.3792|Information|SIDELOAD: Current Package Name: Files
2024-11-15 11:18:20.3794|Information|SIDELOAD: Remote Package Name: Files
2024-11-15 11:18:20.3795|Information|SIDELOAD: Current Version: <IP_ADDRESS>
2024-11-15 11:18:20.3796|Information|SIDELOAD: Remote Version: <IP_ADDRESS>
2024-11-15 11:18:20.3800|Warning|SIDELOAD: Update not found.
2024-11-15 11:18:28.6882|Information|The app is being activated. Activation type: ProtocolActivatedEventArgs
2024-11-15 11:18:29.0102|Information|Running on ARM: False

This is not available on the old version of Files, this is the new version of Files will appear, and this BUG really affects the actual experience

A fresh install doesn't work, I've reported this problem before, but you guys don't think it's a bug, but I'm pretty sure that in the same system, the old version of V3.0 doesn't have this problem, I hope it can be fixed as soon as possible, I've also tried a fresh install of Windows, but to no avail.

Thank you for your report.
We're tracking this on in #15083. Click Subscribe so you can get notified when we have news on this matter.

We're tracking this on in #15083. Click Subscribe so you can get notified when we have news on this matter. The current workaround is to disable running on the background.

Tried it. But it doesn't work

Ah sorry, how about this? "Current workaround - Backup your settings in Files settings > Advanced > Export, then reinstall Files."

Ah sorry I referenced another issue, how about this? "Current workaround - Backup your settings in Files settings > Advanced > Export, then reinstall Files."

No,I'm. Sorry.This still doesn't work

Thanks for the feedback, We're tracking this in #16326. You can check there or click subscribe in the right sidebar for updates.


FileNotFoundError: [Errno 2] No such file or directory: 'model_ckpts/GPT-Neo-XT-Chat-Base-20B/checkpoint_5/prank_0_checkpoint.pt'

Ubuntu Ubuntu 22.04.2 LTS
After downloading the model and now trying to convert:
(OpenChatKit) georgi@georgi-hackintosh:~/Documents/GitHub/OpenChatKit$ python3.10 tools/convert_to_hf_gptneox.py --ckpt-path model_ckpts/GPT-Neo-XT-Chat-Base-20B/checkpoint_5 --save-path huggingface_models/GPT-NeoXT-Chat-Base-20B --n-stages 8 --n-layer-per-stage 6
loading stage 0
Traceback (most recent call last):
  File "/home/georgi/Documents/GitHub/OpenChatKit/tools/convert_to_hf_gptneox.py", line 110, in <module>
    load_decentralized_checkpoint(
  File "/home/georgi/Documents/GitHub/OpenChatKit/tools/convert_to_hf_gptneox.py", line 43, in load_decentralized_checkpoint
    checkpoint = torch.load(os.path.join(input_path, f'prank_{i}_checkpoint.pt'), map_location=torch.device("cpu"))
  File "/home/georgi/miniconda3/envs/OpenChatKit/lib/python3.10/site-packages/torch/serialization.py", line 771, in load
    with _open_file_like(f, 'rb') as opened_file:
  File "/home/georgi/miniconda3/envs/OpenChatKit/lib/python3.10/site-packages/torch/serialization.py", line 270, in _open_file_like
    return _open_file(name_or_buffer, mode)
  File "/home/georgi/miniconda3/envs/OpenChatKit/lib/python3.10/site-packages/torch/serialization.py", line 251, in __init__
    super(_open_file, self).__init__(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: 'model_ckpts/GPT-Neo-XT-Chat-Base-20B/checkpoint_5/prank_0_checkpoint.pt'

Any ideas?

Looks like you need to specify the correct checkpoint path. Can you paste the output of ls model_ckpts/GPT-Neo-XT-Chat-Base-20B from the root of the repo, please?


Fix relative URLs in README files

Fixes #2502
This turned out to be a real bug rather than a phantom. The original rendered README files we got from scraping GitHub included all parts of the URL from a github.com base URL. The README file API gives back URLs without owner/repo/branch parts.
We also get one additional bit of functionality here, which would have been difficult to do previously. We now get to fix up truly relative URLs as well as root relative ones.

Deploying as 2.89.3


test

/full-test-run

/run-full-test

/run-full-test

/run-full-test

/run-full-test

/run-full-test

project_name

/run-full-test

/run-full-test

/run-full-test

/run-full-test

/run-full-test

/run-full-test

/run-full-test

/run-full-test


Fixing the bug in conversion and updating documentation

The bug with converting GAFs has been fixed (the problem was the variable storing the cigar string also stored 'cg:Z:' which caused a duplication of the tag on output).
Test cases added for reverse alignments
Updated documentation for find_path and view

Seems like it passed all the checks, I trust you :D


Run VSIX tests on all platforms

Updated build.yml to run VSIX tests on Windows and macOS as well. This closes #912 .

Codecov Report

Merging #930 (b4408fa) into main (b4cdcbc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #930   +/-   ##
=======================================
  Coverage   94.55%   94.55%
=======================================
  Files         302      302
  Lines       13682    13682
  Branches       12       12
=======================================
  Hits        12937    12937
  Misses        745      745

Flag
Coverage Δ

dotnet
95.20% <ø> (ø)

typescript
25.78% <ø> (ø)

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

@majastrz I'm not sure why this is still pending since the build is successful. Do we need to alter the branch protection rules?

@majastrz I'm not sure why this is still pending since the build is successful. Do we need to alter the branch protection rules?

Yeah, the name of the status check has changed. I'll fix it and merge it.

Ok fixed it and merged it to avoid affecting other PRs.

Ok fixed it and merged it to avoid affecting other PRs.

Thanks!


Fix request headers

Fixes an error when request has multiple headers with the same key but different value
json: cannot unmarshal array into Go struct field Request.headers of type string: UnmarshalTypeError

Hmm, this didnt work with either 1 or 2 headers. I'm getting this error for all requests:
json: cannot unmarshal string into Go struct field Request.headers of type []string: UnmarshalTypeError

Here's the json payload with one header:
{
  "method": "GET",
  "headers": {
    "host": "vercel-bug-repro-8yegv12of-styfle.vercel.app",
    "x-real-ip": "<IP_ADDRESS>",
    "foo": "bar",
    "x-forwarded-host": "vercel-bug-repro-8yegv12of-styfle.vercel.app",
    "accept": "*/*",
    "x-forwarded-proto": "https",
    "x-vercel-deployment-url": "vercel-bug-repro-8yegv12of-styfle.vercel.app",
    "x-forwarded-for": "<IP_ADDRESS>",
    "user-agent": "curl/7.64.1",
    "x-vercel-forwarded-for": "<IP_ADDRESS>",
    "x-vercel-id": "cle1::l8mnx-1636756062755-3789c4f68932"
  },
  "path": "/",
  "host": "vercel-bug-repro-8yegv12of-styfle.vercel.app"
}

Here's the json payload with two headers:
{
  "method": "GET",
  "headers": {
    "host": "vercel-bug-repro-8yegv12of-styfle.vercel.app",
    "x-real-ip": "<IP_ADDRESS>",
    "foo": [
      "bar",
      "baz"
    ],
    "x-forwarded-host": "vercel-bug-repro-8yegv12of-styfle.vercel.app",
    "accept": "*/*",
    "x-forwarded-proto": "https",
    "x-vercel-deployment-url": "vercel-bug-repro-8yegv12of-styfle.vercel.app",
    "x-forwarded-for": "<IP_ADDRESS>",
    "user-agent": "curl/7.64.1",
    "x-vercel-forwarded-for": "<IP_ADDRESS>",
    "x-vercel-id": "cle1::pkmmp-1636755907234-8ee8499e420c"
  },
  "path": "/",
  "host": "vercel-bug-repro-8yegv12of-styfle.vercel.app"
}

@jarneson Noticed the foo header changes from string value to array of strings. Maybe we need to marshal this twice to see which one works?

Ok fixed with custom unmarshal json


pull request for e2e test at 01/20/2017 23:26:13

:white_check_mark: Validation status: passed
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.


Run simple test to check whether Invoke-AtomicTest is working

The checks failed because T1221 has dependency_executor_name defined even though it doesn't have any dependencies.
Fixed the atomic here #2648


_this2.dash.getDebug(...).setLogToBrowserConsole is not a function

Current Behavior
Playback works but the console displays the following error during DASH playback.
_this2.dash.getDebug(...).setLogToBrowserConsole is not a function
Expected Behavior
No errors in console. And full support for DASH JS
Steps to Reproduce
                    <ReactPlayer
                        url="https://dash.akamaized.net/envivio/EnvivioDash3/manifest.mpd"
                        playing={true}
                        controls={true}
                        config={{
                            file: {
                                forceDASH: true,
                            },
                        }}
                    />

Environment

URL attempting to play: https://dash.akamaized.net/envivio/EnvivioDash3/manifest.mpd
Browser: Chrome
Operating system: MAC OS

Other Information
Thanks

It might be due to your version because I am unable to reproduce this.
here is the sandbox of your problem. no error in console.
try updating your version to latest.

Thanks for the fast response. It looks like it is due to the Dash JS version.
setLogToBrowserConsole was deprecated and removed on version 3.
The issue is on Line 309 on FilePlayer.js
https://github.com/Dash-Industry-Forum/dash.js/issues/3018

Currently I am using the npm version of react player:
"react-player": {
            "version": "2.3.0"

However the contents of the file FilePlayer.js in the npm packag look different from master branch.

However the contents of FilePlayer.js in the npm package look different from master branch...

Line 147: https://github.com/CookPete/react-player/blob/master/src/players/FilePlayer.js

Fixed by https://github.com/CookPete/react-player/pull/933


Create and validate a build pipeline for the java microprofile collection

Checked in a dev-pipeline along with other artifacts in the kabanero-piepline repo that will build an appsody java microfile repo and publish the image to docker hub.  We will be adding security tasks to this pipeline in the near future.


Left sidebar layout example is missing the last element

The last entry "99" is not visible:

The window height seems to affect the last visible elements.
With a small window, all items are visible:

With a big window, the last two items are not visible

Oh yes I know why. Will fix today or tomorrow. Thanks for the report


modrinth pull - java won't start

Describe the problem
fresh docker  install - using portioner to push it as a stack ( it just launches the yml with a web editor)
sometimes fails because modrinth kills downloads on repeated fast try (causing missing modpacks)
but the issue is java is broken
changed PID AND GID. to 0.  or the docker container user..  made no diffrence
added JAVA VERSION env and it dint fix it.
and im stumped.. google the error didn't give me anything useful. ( mostly reddit users trolling noobs with rm-rf stuff)
trying to migrate the container and world to a new machine...
fresh install and  just copy the world folder.
can add container def to examples if you like
Container definition
version: "3.8"

networks:
  default:

services:
  Minecraft4a:
    image: itzg/minecraft-server
    hostname: mcround4
    tty: true
    stdin_open: true
    ports:
      - "25565:25565"
      - "8123:8100"
      - "8100:8100"
    environment:
      MOD_PLATFORM: MODRINTH
      MODRINTH_MODPACK: https://modrinth.com/modpack/hermitcreateflavors
      OVERRIDE_SERVER_PROPERTIES: TRUE
      MOTD: Respectful - do not steal ask first - friendly play
      SPAWN_PROTECTION: 0
      RCON_PORT: 25575
      TZ: America/Vancouver
      DIFFICULTY: HARD
      ALLOW_FLIGHT: TRUE
      RCON_PASSWORD: rconrodney
      PVP: TRUE
      MAX_PLAYERS: 6
      OPS: 37a79502-60ce-4115-b8d3-ddec19516c46
      MAX_TICK_TIME: 0
      EULA: TRUE
      SERVER_NAME: Hermit Create Flavors - Round4A
      JVM_OPTS: "-XX:+UseG1GC -Dsun.rmi.dgc.server.gcInterval: 600000 -XX:+UnlockExperimentalVMOptions -XX:+DisableExplicitGC -XX:G1NewSizePercent: 20 -XX:G1ReservePercent: 20 -XX:MaxGCPauseMillis: 50 -XX:G1HeapRegionSize: 32"
      MEMORY: 4G
      REMOVE_FILES_SPEC: ._*
      LOG_LEVEL: info
    volumes:
      - /Volume1/public1/DockerStuff/Minecraft/backups:/backups:rw
      - /Volume1/public1/DockerStuff/Minecraft/mcround6:/data:rw
    restart: unless-stopped

Container logs
[mc-image-helper] 20:39:02.981 INFO  : Modpack hermitcreateflavors version 4.1 is already installed
[mc-image-helper] 20:39:07.760 INFO  : The file ./ops.json already exists, so no changes will be made
[init] Setting initial memory to 4G and max to 4G
[init] Using Forge supplied run.sh script...
Error: Could not find or load main class 600000
Caused by: java.lang.ClassNotFoundException: 600000
2023-09-17T20:39:14.156-0700    WARN    mc-server-runner    Minecraft server failed. Inspect logs above for errors that indicate cause. DO NOT report this line as an error.    {"exitCode": 1}

seems related to java options line
 -XX:+UseG1GC -Xmx4G -Xms4G -Dsun.rmi.dgc.server.gcInterval=600000 -XX: +UnlockExperimentalVMOptions -XX:+DisableExplicitGC -XX:G1NewSizePercent=20 -XX:G1ReservePercent=20 -XX:MaxGCPauseMillis=50 -XX:G1HeapRegionSize=32. seems to work better..     there was a space in the wrong place.. !!!
am testing..
need a better way to tune th settings for the memory allocated to reduce overhead on GC and determining size for install dmodpucks needed for estimated users..  and system hardware...  am just guessing here... any suggestions ?
now get
error: a fatal exception error occurred. program will exit```

removing the jam options  got it going .. this can be closed its not your problem
ill figure out what options are good for the memory optimization ..    stage this tho because I have a mother machine on linux that has no problem with these exact options..
this machine is windows.. so somethings diffrent
ALSO :  just a thought..  is there abetter way to reduce log generations. down to show only warns and errors..  so it doesn't spam logs with  all the info ..

ALSO : just a thought.. is there abetter way to reduce log generations. down to show only warns and errors.. so it doesn't spam logs with all the info ..

 That's an issue to report to Mojang and every mod author. Their logs produce 5000 times more lines than my container startup logs.

I thought that’s just how Java is.  I can almost never figure out Java
errors.  So much big dump
On Mon, Sep 18, 2023 at 4:13 PM Geoff Bourne @.***>
wrote:

ALSO : just a thought.. is there abetter way to reduce log generations.
down to show only warns and errors.. so it doesn't spam logs with all the
info ..
 That's an issue to report to Mojang and every mod author.
Their logs produce 5000 times more lines than my container startup logs.
—
Reply to this email directly, view it on GitHub
https://github.com/itzg/docker-minecraft-server/issues/2386#issuecomment-1724588055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQBDDOEMWZTBM6WISH7HLDX3DISDANCNFSM6AAAAAA44DDTMU
.
You are receiving this because you authored the thread.Message ID:
@.***>


This is because, to this security group a network interface is bound. after manual deleting eni- aws-K8S- it keep going further.

@bryantbiggs Why you closed this issue? The eni interface is created by module and should be deleted by module.
I create clean eks and after is up and running I'm deleting it, and problem with stuck interface esists.
Interface has description `
Elastic network interface | aws-K8S-i-0013bb5a69faf8ff9
`
and was created during "apply" process.

no, the module does not create the ENI that you are describing. You are either dealing with the VPC CNI where it has not had time to clean up the resource it creates (it creates ENIs for container networking), or a load balancer. Both of those are resources that are deployed *onto the cluster but are not relevant to the module. Again, you need to ensure the resources deployed onto the cluster are cleaned up first, before trying to destroy the cluster

Thank you for clarification. I'll check when this interface is created and why is not deleted. It was confusing, because it description was aws-K8S-i and these interfaces are created during setting up eks.

@bryantbiggs please   ##help.
I set new and clean EKS cluster from module. After it was up and running I was adding and deleting my resources. All went well and all was deleted.
This interface was created during EKS creation and wan't deleted after instance was terminated.
interfaces
instances
As you can see instance was terminated but interface still exists.
Is this AWS problem? Where to request it?


Error: Could not parse JSON (lexical error: invalid char in json text.)

Hi,
I have few problems with your script. At first the bar is missing few fields. It looks like this:

After calendar(and the other icons on the right) click event it will show this error:

Hello, I suppose some scripts are not working correctly. You should try them one by one, for example :
cd ~/.config/i3status/
./cpu.py
./disk.py
# ... and sh scripts

You should get some interesting errors.

The problem was that I specified relative and not full paths in both the I3 config file and also into the my_bar.sh
Also you should note in the .md there are hardcoded paths that must be changed in the the my_bar.sh
Lastly i wanted to ask.... I changed the internval (sleep) from 10 to 1 and it did refresh at 5 seconds. Is there something i'm missing?
Thanks

Thanks for the suggestion. The README is updated.
There are some APIs : meteo, crypto values, IP address... I guess there is latency and all answers are not done in 1 second. And it is too much requests for thoses API. You may get "403 too many requests" from them. Your IP address, the weather are not changing each second. So you should disable some. Or find a way to set a refresh time for each script. Or stay at 10 seconds.


Add logo url

The logo filename can be found from the wikipedia data, if you know the wikipedia URL you can derive the total URL to the logo if it exists.
Example:
https://en.wikipedia.org/wiki/Alphabet_Inc.#/media/File:Alphabet_Inc_Logo_2015.svg
Ideally, the logo name would be Alphabet_Inc_Logo_2015.svg and the URL would be https://en.wikipedia.org/wiki/Alphabet_Inc.

Determined that this will not be a feature as the problem is more complex.  Notably the logo file names aren't deterministic and there is an alternative path to solve this problem that is more general.  Essentially the Go-base dockerized service called Icon Server will be used externally for deriving the logo image and URL for companies.  Therefore, we will close this issue.


Double click on Editor tab to expand it in full window mode.

Sometimes, when open many editor windows (or even editor group) on a notebook computer where screen real estate is very limited, to compare codes or other things, it would be really great if we can double-click on the editor tab at the top to temporary expand the editor to cover the entire window of VS Code for easier editing. And when press ESC key, it is going back to the previous layout.
Please note that this is not the same as the available Zen mode as Zen mode will still keep showing the editor group.

When you say editor group, you actually mean the title area of the editor - where you can see the editor tabs, right?
So you would basically like Zen Mode to hide the title area if I understood this correctly?

No, that's not what I mean. Let's see these screen shots and imagine that we are working on a 13" or 14" inch laptop.

Since the screen is quite small, everything will looks tiny unless we use Ctrl+plus to increase font size and everything will blow up like this.

In the above, picture you'll see 2 editor groups. In Zen mode, it just removes other things except all the editors, but it is still showing those 2 editor groups.
In my case, I would like to temporary switch a single editor tab to a full window by double-click on its editor tab (where it shows the file name) to work on it, so I can see everything more and clearer. Then when I'm finish I can just double-click on its tab again (or press ESC) to go back to the same arrangement and continue working on what I was doing previously.

It seems that what I really like to have is a command to "temporarily hide other editor group".
Sorry, if this has caused any confusion.

Understood.
This is a fair feature request which we currently do not have on the plan, thus keeping it open and assigning to the backlog.

Well, a bit disappointed to see this one goes as I really think it would be really help people who is working on a small screen laptop. However, that's understandable. Thanks for the shot, though.


Add religious symbols

Add three religious symbols: \latinCross ✝, \starAndCrescent ☪, and \starOfDavid ✡. I think it's a very bad look for the only religious symbols in this entire dictionary to be the swastikas from #87, regardless of how innocently it is intended.
Potentially some other religious symbols could be added (this Wikipedia list is a good place to look). But ultimately I'm not familiar enough with enough religions to say which symbols are important, so I just left it at these three for this PR.

On my computer it seems that even with the Variation Selector 15 (U+FE0E) it doesn't display as text but it does seem like that's a good idea to have that for systems that do honor that.


Issue 3179 - Set-DbaLogin add PasswordPolicyEnforce parameter

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[x] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Add PasswordPolicyEnforce parameter to Set-DbaLogin to allow enabling/disabling the password policy for a login.
Approach

Added parameter and if it is passed it is updated.
Commands to test

Enable Policy
Set-DbaLogin -SqlInstance localhost\sql2016 -Login testlogin -PasswordPolicyEnforced
Disable Policy
Set-DbaLogin -SqlInstance localhost\sql2016 -Login testlogin -PasswordPolicyEnforced:$false
Screenshots

N/A
Learning

N/A

man, you're awesome! 🔥 🚒


Does withRole restore localStorage?

Wondering if the new withRole feature restores items in localStorage? I'm noticing that with jwt based authentication where access tokens are stored in localStorage, session are not properly restored.

@astanciu We thought about this scenario, but decided to postpone local storage sandboxing implementation for Roles. Now when there is a demand for the feature, I'll add it to the planned.
@churkin What we need from hammerhead side to implement it?

+1 for this. As it stands, unless I'm missing something, I cannot find a way to run multiple tests within 1 fixture w/o each test needing to do its own login. Alternatively, it would be great if the tests within a fixture would share the same session

@inikulin All is ready, we just need to add backupStarage and restoreStorage methods to the hammeread's API

any updates here? Is this something that will happen, and if so, is there a rough timeline?

Hi @astanciu,
We plan to implement this in one of the next two releases.

https://github.com/DevExpress/testcafe-hammerhead/issues/1125


Add "pivotal" acknowledgement to the README

The license that originally came with the "pivotal" 64-bit time routine, incorporated into libmseed as ms_gmtime_r(), contains this clause:
Redistribution and use in binary form must reproduce an
acknowledgment: 'With software provided by http://2038bug.com/' in
the documentation and/or other materials provided with the
distribution, and wherever such acknowledgments are usually
accessible in Your program.

To comply as much as possible, add the acknowledgement to the README and possibly to the man pages for the time routines.

Fixed in c169ce1a47b5304d0569d1512148227087825ace.


feat: add Romanian translation

I've added the translated strings in Romanian.
Thank you.

Thank you Openoms, köszönöm szépen!

@i2dor one more thing before merging.
Can you please run in your branch:
yarn update-translations

and commit the change?

Thanks again Openoms, this was my first pull request. 🫡 🙏🎇


Js api calls

Continued work on distance location. Distance is now being returned with a setTimeout delay of 5 seconds to wait on user's location to be returned

awsome


主题商城与配置工具无法访问

https://arco.design/themes/ 无法正常访问

https://github.com/arco-design/arco-design-vue/issues/3396#issuecomment-2561600256


Store agent manifest(YAML) in the database during cluster register mutation.

Currently, we are dynamically generating the agent manifest. Request for storing the manifest(yaml) during the cluster register mutation.

cc: @gdsoumya

Hi, I'd like to work on this. Can you give me a brief idea on how to go about implementing this?

@rajdas98 I don't think this is needed anymore

Okay


Can not get debian netinst image after clicking install grub menu

Describe the issue
The iso boots, and shows the grub menu, and one you hit the enter button there is just a black screen with like a blinking cursor. It never makes it further than that.
ISO USED: https://cdimage.debian.org/debian-cd/current/arm64/iso-cd/debian-10.7.0-arm64-netinst.iso
Configuration

UTM Version: 2.0.18
OS Version: 11.1
Apple Silicon: M1

Crash log
No Crash Log
Debug log

Can you try using console mode? It may not have the right drivers.

Can you try using console mode? It may not have the right drivers.

Closing since it is an issue with the guest OS.


[BUG] Remaining Missing Test IDs on PDF

Context & versions

Some Several components still lack proper test IDs, impeding effective testing and automation

Missing test IDs

Proposal Card

Governance action type -> data-testid=governance-action-type
Proposed on date -> data-testid = proposed-date
It will be better to change the proposal card testId like proposal-${governanceActionType}-card as the previous test ID conflict with filtering

Proposal Details

Governance action type-content -> data-testid=governance-action-type-content
Title content -> data-testid= title-content
Abstract Content -> data-testid=abstract-content
Motivation Content -> data-testid=motivation-content
Rationale Content -> data-testid=rationale-content
Link text content -> data-testid=link-${index}-text-content

Click to copy link -> data-testid=copy-link

Verify identity button -> data-testid=verify-identity-button

Close button of user modal -> data-testid=close-user-modal

Hello @teske00,
I have noticed that the test IDs for filters have different naming conventions. For example, "Info" and "Treasury" start with a capital letter, whereas "submitted for vote" and "active proposal" starts with a lowercase letter. Please standardize the naming conventions of the test IDs for all filters.

@teske00, furthermore the status should be inside the proposal card. I cannot test which status belongs to which proposal card

@kneerose The component is structured like this, the only thing we can do now so as not to break the entire logic is to set the name of the status in the data-testId of the status, for example 'data-testid': proposal-${proposal?.id}-{status_name }-status. Also, for the proposal card, in addition to proposal-{status_name}-card, add proposal_id in the name, so that it becomes proposal_{proposal_id}-{status_name}-card. In this way, it is possible to connect which status belongs to which card. Is it ok to do it like this?

@teske00, I don't think this approach is ideal. We are retrieving a list of proposal locators proposal-${govactionType}-card and checking the status inside each proposal card. If the component is structured this way, could you change the data-test-id of the proposal card to be the parent element of the status and the current <div> tag which is using the data-test-id of the card. ie. proposal-${govactionType}-card. If it is possible.

@kneerose As far as I understand you, you mean to put the card and status in a div to which we will add the test id "proposal-${govactionType}-card", and remove it from the current div (card). And what do you think if we change the testId for draft cards to follow that structure, and not to be draft-${proposal?.id}-card, how is it now?

@kneerose And is it perhaps necessary to add the proposal id to the testID in addition to the name of the status?

@kneerose Is this the desired structure, the card component with test id wraps the card content and status. I didn't change the other testIDs, I just moved the current testIDs that were in the wrapper components. Is this enough?

@teske00, yes this is the desired structure

@kneerose The old testId names can remain as they are now, we just need to do this?

@teske00 yes, only thing you have to do is change the position of the card test ID.

@teske00
I''ve located the missing test IDs mentioned above. The test IDs are now available in the test environment.
This issue can be closed now. CC: @bosko-m

Closing as per the comment above.


Feature: Add hover tooltip to reactions on highlights

Type of feature
🍕 Feature
Current behavior
When a user hover's on a highlight reaction, it shows no information about users that reacted

Suggested solution
Let's add a tooltip similar to GitHub's for reactions with the name of the first user and a +${restUsers.length}

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

@brandonroberts @0-vortex This might need API updates... The users that reacted should be included in the highlights reaction response object.

Yep, this will need to be added

Opened a ticket for it in the backend 👍 https://github.com/open-sauced/ops-backend/issues/48

under no circumstances. stop trying to do what github can’t, it should be a first red flag something is not possible at scale.
this is not the only thing happening on hover that can’t/won’t scale …

There might a way to do this, we can have something like what facebook and twitter is doing:
Twitter example:
they have a button for reactions and when you click it, it opens the information about reactions:

Instagram has something similar:
when you click on likes it show a new tab for that:

Facebook is showing up to 20 on hover:

GitHub is showing 3-4 people on hover,
I think we can allow the user to click the text that we show on hover to open the full reaction list.
something like: ogdev-01 and others reacted with thumb, see full list
Or even add a place like twitter saying: {count} reactions
where it would be clickable and have a tab showing reactions.

@brandonroberts was this done in the backend or should I work on that first?


'arrayBuffer' is declared but its value is never read.

Refer to
https://github.com/discord/discord-interactions-js/blob/main/src/index.ts#L96-L97
      const arrayBuffer = value.buffer.slice(value.byteOffset, value.byteOffset + value.length);
      return new Uint8Array(value);

ts warning...
'arrayBuffer' is declared but its value is never read.

So the question is arrayBuffer didn't use when return, is this a bug or intend?

Looks like it is unused, and therefore should be removed

Fairly sure this was addressed :)


{"error":"ENOENT: no such file or directory, open '/home/wyrx/textcla/logs/5m78AlYQ/trials/sbm7r/stderr'"}

Hello, I have set logDir in config.yaml, that's it
authorName: wyrx
experimentName: textcla
logDir: logs
trialConcurrency: 2
maxExecDuration: 100h
maxTrialNum: 500
trainingServicePlatform: local
searchSpacePath: ./space.json
There is only db and log under "/home/wyrx/textcla/logs/5m78AlYQ/", no trials

Hi @LilAnthony123, please set logDir as an absolute path value and try again.

hello @LilAnthony123, could you follow this and update the status of this issue? Thank you!

Hi @LilAnthony123, please set logDir as an absolute path value and try again.

hello @LilAnthony123, could you follow this and update the status of this issue? Thank you!

Hi @LilAnthony123, please set logDir as an absolute path value and try again.

I have the same problem and doing so won't help

hello @LilAnthony123, could you follow this and update the status of this issue? Thank you!

Hi @LilAnthony123, please set logDir as an absolute path value and try again.

I have the same problem and doing so won't help

please run experiment with --debug mode and provide nniManager.log here.

hello @LilAnthony123, could you follow this and update the status of this issue? Thank you!

Hi @LilAnthony123, please set logDir as an absolute path value and try again.

I have the same problem and doing so won't help

please run experiment with --debug mode and provide nniManager.log here.

I have a similar issue. It would be great if you could help with it. Here is the nniManager.log I got with debug mode.
nnimanager.log
The dispatcher.log is also changed with debug mode. Here is it if you need it.
dispatcher.log

The issue is solved after I move the files from cloud drive to the local machine.


Should be able to execute the same script with different User Interface (UI) librarys

Expected behavior
I should be able to execute a todo app script recorded for reactjs with angularjs
Actual behavior
The script seems hung when executing with angularjs
Steps to replicate

Run script

const { openBrowser, goto, write, press, click, near, checkBox, closeBrowser } = require('taiko');
(async () => {
    try {
        await openBrowser({headless:false},{timeout:60000});
        await goto("http://todomvc.com/examples/react/#/");
        await write("Working with examples");
        await press("Enter");
        await click("Working with examples");
        await click(checkBox(near("Working with examples")));
        await click("Clear completed");
    } catch (e) {
        console.error(e);
    } finally {
        await closeBrowser();
    }
})();

This works as expected

Now run the script

const { openBrowser, goto, write, press, click, near, checkBox, closeBrowser } = require('taiko');
(async () => {
    try {
        await openBrowser({headless:false},{timeout:60000});
        await goto("http://todomvc.com/examples/angularjs/#/");
        await write("Working with examples");
        await press("Enter");
        await click("Working with examples");
        await click(checkBox(near("Working with examples")));
        await click("Clear completed");
    } catch (e) {
        console.error(e);
    } finally {
        await closeBrowser();
    }
})();

This script seems hung. It is not able to write on to the field with default focus

Version
commit - 7482e63f741fde16ff9fb9db5267b37dc51931e3

The same holds good when the URL http://todomvc.com/examples/typescript-backbone/ is used

I believe taiko does not handle virtual DOMs well. taiko-react is an example where a plugin can bridge this gap.

@sriv According to the issue, the script works on React applications but does not on non virtual dom frameworks like angular and backbone.

ah right. I stand corrected.

The issue was happening in headful mode as the goto action was performed before openBrowser was complete. Fix available in 50bbc07, should work fine for all frameworks.

Verified and tested this issue with the commit version 50bbc07.
The below script is working with different user interface libraries:
const { openBrowser, goto, write, press, click, near, checkBox, closeBrowser } = require('taiko');
(async () => {
    try {
        await openBrowser({headless:false},{timeout:60000});
        //  await goto("http://todomvc.com/examples/angularjs/#/");
        await goto("http://todomvc.com/examples/react/#/");
        await write("Working with examples",into(inputField({placeholder:"What needs to be done?"})));
        await press("Enter");
        await click("Working with examples");
        await click(checkBox(near("Working with examples")));
        await click("Clear completed");
    } catch (e) {
        console.error(e);
    } finally {
        await closeBrowser();
    }
})();


Can this project use cmake to build project instead of code blocks

code blocks is buggy and is hard to use. Can you use cmake to build this project ????

I use the latest code blocks to open this project and find I can not find the header file and almost all header file can not open..

This project can meet your needs: https://github.com/lvgl/lv_port_linux

code blocks is buggy and is hard to use. Can you use cmake to build this project ????

Yes, works perfectly with cmake.
I used the CMakeLists.txt from lv_port_pc_vscode for this project and adapted it slightly (basically just commented out "find_package(SDL2 REQUIRED SDL2)" and set the “USE_...” variables to 0).
Compiled and runs flawlessly, completely without codeblocks, only with cmake and make.


Allow parameters to be hidden from help text

cc @NTaylorMullen so we can parse dispatcher version using the command line app but not put it the help text.

Changes look ok. Maybe rename some of the test before checking it in.
:shipit:


NamedContextFactory.java does not return null in case beans count is zero

2.2.5.RELEASE version
https://github.com/spring-cloud/spring-cloud-commons/blob/dc3a4674c44011e7b421b8ab25b04c2a22eae19d/spring-cloud-context/src/main/java/org/springframework/cloud/context/named/NamedContextFactory.java#L182-L188
2.2.6.RELEASE version
https://github.com/spring-cloud/spring-cloud-commons/blob/82fb22649b17cad6b3b996800908154787d9080a/spring-cloud-context/src/main/java/org/springframework/cloud/context/named/NamedContextFactory.java#L225-L229
Could someone help me understand why null is removed as a return value for the above method as part of the https://github.com/spring-cloud/spring-cloud-commons/pull/826/files ?
I encounter issue due to this change because another package is expecting null instead of empty map. Not sure if there are other places but I found it here https://github.com/spring-cloud/spring-cloud-openfeign/blob/d30540d1445afd7acfdf9f30bd8cb3ae654c9fed/spring-cloud-openfeign-core/src/main/java/org/springframework/cloud/openfeign/FeignClientFactoryBean.java#L206-L212

Looks like the change was made due to this bug in Spring Cloud Gateway
https://github.com/spring-cloud/spring-cloud-gateway/issues/1941
I think all we need to do here is check for the empty Map instead of null
CC: @spencergibb @OlgaMaciaszek what do you think?

	        Map<String, T> res = BeanFactoryUtils.beansOfTypeIncludingAncestors(context, type);
		return res.length > 0 ? res : null;

@ryanjbaxter Is there any reason why the change can't be like above (store in local variable, check length and return the result from variable) ?

@spencergibb would be able to say for sure.  To me that seems like it would still result in the same problem.

@spencergibb @ryanjbaxter - anything on this, when can we expect fix ?

I'm still unsure what problem this is causing. Like @ryanjbaxter said, a simple check for an empty map where you pointed to is an optimization, but it isn't broken.

@spencergibb - from what I see getInheritedAwareInstances method is using beansOfTypeIncludingAncestors which returns either the [1] Map of matching bean instances, or [2] an empty Map if none. It will never return null. That means the code in if block will always run and it will cause unexpected results.
That's why I would say that this is a bug, flow is being changed accidentally.

@spencergibb - If the Feign.bulder is configured with interceptors before the configureUsingConfiguration call is made, due to requestInterceptors != null condition, all interceptors are removed/overriden with empty List.
I'm not that familiar with spring concepts so it's very hard for me to explain it in better words, sorry for that.

@jBesic  What you've described is happening, but it does not seem to be a bug - it's just how the method was implemented. If it's causing actual errors or issues, please provide a minimal, complete, verifiable example that reproduces the issue.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


UI Updates

[x] Move filter to a tab on the left tool box instead of the drawer
[x] Update all icon, use trashcan icon for clear all charts, replay icon for reset filter.
[x] Update About modal, adding team names and more links.
[x] Show little x for extra pair plot only on hover
[x] Consistent UI for adding chart element: add an insert chart icon for add button on top, and change the icon in chart to blue that will add elements.
[x] Add a "mode" for showing patient detail and ids.

I'm seeing this issue with the blood components when I load in. The filter range is set to 0-0 instead of the whole range

It's possible to get into a broken state where you have case info showing with no cases in the selector above by brushing and then clicking the "x" in currently selected

is this after the data being loaded?

It's possible to get into a broken state where you have case info showing with no cases in the selector above by brushing and then clicking the "x" in currently selected

I'm a bit confused

is this after the data being loaded?

This is just when loading into the application for the first time. So yes, after all the data has loaded

I'm a bit confused

Maybe this will help show what I mean
https://user-images.githubusercontent.com/36867477/135891279-f3cd0e14-8af0-4993-bd72-61c599ea2756.mp4

I'm a bit confused

Maybe this will help show what I mean https://user-images.githubusercontent.com/36867477/135891279-f3cd0e14-8af0-4993-bd72-61c599ea2756.mp4

broken link?

the selection issue should be fixed. I also fixed the issue of scatterplot repeatedly having actions registered in provenance but not actually updating the selection (which caused the undo/redo not functioning properly)


Ensure mobile predicate return boolean

A fix for https://github.com/fnando/browser/issues/342 if you're interested

That's the expected behaviour. Please read https://github.com/fnando/browser/issues/321 for more info.


Horizontally scaling UAA without configuring sticky sessions in Load Balancers

What version of UAA are you running?
4.13.4
How are you deploying the UAA?
We are deploying UAA inside a Tomcat 9 container, as a WAR.
What did you expect to see? What goal are you trying to achieve with the UAA?
We would like to load balance requests between instances. Surprisedly, sessions seem to be stored in memory by default, forcing us to configure sticky sessions in our load balancers.
We just want to be able to load balance traffic without configuring sticky sessions or any form of consistent hashing. Ideally, using cookies as storage since we don't plan to store anything in the UAA session.

I don't understand how we could implement this without in-memory sessions. Is there another project doing this that you think we should look at to learn about other approaches? Or do you have specific thoughts you could share as a PR? @c4milo

We're very focused on the use case for UAA within the context of CF deployments where relying on gorouter sticky sessions has not been a problem. Storing sessions somewhere else like Redis could be interesting but it's not something we've had CF customers/users asking us for.
Any thoughts on this @tnwang ?

Have anyone tried spring-session  for distributed access ? Will it help in scaling horizontally ?

Closing due to inactivity.

BTW spring-session has been integrated into the UAA and sessions can be externalized to a DB now.


TransferUtility issues with main UI thread

I'm using TransferUtility (latest build, but the issue seems to be there for previous versions too) to upload videos. Everything works fine but after few uploads (mostly one after another, but also in parallel), UI starts to freeze. All my calls are in separate threads therefore it seems TransferUtility does something on UI thread which it shouldn't. I tried to release all objects as soon as the upload is finished but the UI issues are still visible. Can you give some advice?

After deeper investigation seems it was not TransferUtility what caused main thread issues. Obsolete.


WIP: Expand correlation measures

Reworks how Mutual Information is calculated
Adds support for Pearson Correlation
Adds general method for calculating one or all of MI, Pearson, and the absolute max of both
Closes #1157

@rwedge Given that all the tests pass, is this ready for review?

@gsheni yes

Open question: add get_valid_pearson_columns to match get_valid_mi_columns?

@bchen1116 thanks for the review, updated


mindate option not locking previous year

As you can see in the following image, if we're using the mindate option, we are still able to go to the previous year.

It's happening because when checking if it's after the mindate, we show or hide the button for the month and the year at the same moment.
if(!this.isAfterMinDate(startOfMonth, false, false))
{
    this.$dtpElement.find('a.dtp-select-month-before').addClass('hidden');
    this.$dtpElement.find('a.dtp-select-year-before').addClass('hidden');
}
else
{
    this.$dtpElement.find('a.dtp-select-month-before').removeClass('hidden');
    this.$dtpElement.find('a.dtp-select-year-before').removeClass('hidden');
}

It's fixed. Can you confirm it works fine for you ?
Thank you

Nice. it's working.
But noticed that if the mindate is set for the first day of one month, we can go to the previous month (despite none of the day is available for selection)


Firmware Scan？

Can OSV-SCANNER support firmware scan?  such as .bin, .image,. o ,. elf,.pkg
I want to use it to test a firmware package for the Internet of Vehicles, such as TBOX

No, these are not currently supported. Further to what you're trying to accomplish, do you have any examples of existing vulnerability records in OSV.dev that you'd hope to be able to find?


Stop ingesting packets after the client has errored

Fixed #21.
@cefn please test and see if it solves your issue.

Indeed this branch helps our production test suite to proceed beyond the error I reported.
We're now having much subtler issues when running with Aedes (versus Mosquitto) which only appear when interactively testing. I'm investigating if it's to do with failing to honour the Last Will and Testament - what's the status of LWT for Aedes?
It seems that the Last Will and Testament is being written to the topic tree before the normal trigger event (when the browser is shut down) or potentially something is forcing the client to be disconnected and hence correctly triggering the LWT (which is somehow swallowed as an error in our production code).
On the plus side I can see that the topic tree reported by mosquitto_sub -t '#' -v from a Aedes production test run and a Mosquitto production test run is so similar that I'm having to investigate in some detail what is the difference which makes one work and the other not, which is a pretty good sign. Given the complexity of the production code, there's still the chance it is some kind of race condition given the timings of the brokers responding could be different. Will let you know if I can recreate in a simpler test suite.

Forgot to mention for reference that Mosca does satisfy our integration testing suite, but just super-slow owing to performance issues as detailed at https://github.com/mcollina/mosca/issues/385 which which we couldn't easily hide from users.
Aedes is much, much more performant in our integration testing. It's not as fast as the newly-patched version of Mosquitto (Ral helped me eliminate the 200ms round-trip delay, at the cost of burning some cycles in poll loops), but is fast enough in end-to-end behaviour that we could certainly use it in production and has much better potential for provisioning and administering access-control lists from within our node server logic, so we could take the hit on raw performance in favour of these other aspects I think.
For those reasons I'm very interested to keep sweating Aedes until it satisfies our integration testing suite, assuming I can figure out the issue we're hitting right now.

@cefn are you running with some form of keepalive? Might be some form of https://github.com/maxogden/websocket-stream/issues/69.
BTW, I would be really happy if you can shelve off some ms from here as well.

I assume that given our production code is Node and V8 in Chrome, running MQTT.js then we should be golden (https://www.npmjs.com/package/mqtt#client and https://github.com/maxogden/websocket-stream/issues/69#issuecomment-122546881 suggest keepalive would be 10 seconds by default in MQTT.js without any special configuration I think).
Generally I have had trouble getting meaningful performance profiling out of V8 so far (so much indirection in the libraries we were using before) but may be better now we've removed almost all our promise and stream code in the latest iteration. With a bit of luck I'll be able to pull out the Aedes functions which run the longest and explore any marginal improvements. Our tests are frequently disadvantaged by a cold-start without optimised hot paths so I'm not sure there's any real performance problem yet.


Modular design for generic functionalities

There are several generic functionalities that could be applied to different entities (pages, events, imprint, locations, ...) like:

Translation management
Revisions
Automatic saving
Entity based search
Role management

Do we want to build these functionalities modular so that we can add them to different entities now and in the future? If so, how?

In one of our last calls we decided to move functions that are used into modules that can be reused. See directory backend/cms/views/utils/


About saving weights

Thank you for your excellent work. I have some questions about saving weights. For example, the weight with the smallest loss is saved in your source code for testing. However, this may not be the highest value of Dice. Why don't you save Dice's best weights? If the best Dice weights are saved, higher accuracy can be achieved. Is there a difference between these two?

@PURSUETHESUN
Hello, testing the Dice metric on the test set at every step will lead to a significant increase in training time.


Add support coex for ESP32C6

Hi @bugadani, @bjoernQ,
I created a PR to support Coex for ESP32C6. Those changes were discussed from https://github.com/esp-rs/esp-wifi/pull/300

Enabled COEX feature and created new COEX example for ESP32C6.
Added big-heap when run COEX mode.
Calling G_COEX_FUNCS in COEX mode
Increase tx_queue_size to be greater than rx_queue_size

Works great! Thanks a lot for this
I'm not sure about changing the defaults and this needs a cargo fmt - otherwise I'm fine with merging this.
I tested on all supported targets and nothing broke 👍

Be sure to remove esp32c6 from this list when you rebase :https://github.com/esp-rs/esp-wifi/blob/7632e74285a70e155a10e8dee4b6dde50d78394f/esp-wifi/build.rs#L22

I'd really love to get this merged.
Unfortunately reviewing this is a bit difficult since there is a merge commit in bc3017a (i.e. +3,083 −9,239) - would you mind removing that commit and rebase on main instead?
It also seems that in esp-wifi/lib.rs the default of tx_queue_size: usize, is still changed to 10

it also seems it stopped working for me 😢

it also seems it stopped working for me 😢

Ah - it works when using the big-heap feature. Maybe something to address in src\lib.rs when defining the DEFAULT_HEAP_SIZE
Got it to work by reverting bc3017a, doing what Scott said in https://github.com/esp-rs/esp-wifi/pull/327#issuecomment-1789143242 , reverting the change to the default of tx_queue_size and using big-heap
@TuEmb do you want to do these changes or should I take over if you don't have the time to do this (certainly keeping your original commit)

Thanks for reviewing @bjoernQ ,
Let me do these changes soon 🤝

You just need to fix the formatting issue then we can get this in

I updated formatting issue


Change to splice method so it is observable

The existing splice method uses the indexer:
array[i + at] = insert[i]
This is not a natively observable change using modern Javascript Reflection APIs.
This PR changes the method to achieve the same effect but using native Array.splice.
This may be useful when using Backbone with other frameworks e.g. it may be desirable to use Backbone modelling for the excellent collection methods and REST APIs, but Aurelia or Angular for UI data-binding.
The latter frameworks can't observe changes on collection.models array if they're done with the indexer. Splice is observable.

Spread was added in https://github.com/jashkenas/backbone/pull/3663. I'll defer to @jridgewell on this one

I removed splice because of its horrible performance in loops (adding an array of models at a particular index).
How is #splice observable bot indexed sets aren't?

I'm in favor of this change, but the performance claim needs quantification before the change can be made. By "quantification", I mean something a little more serious than a nanobenchmark on jsperf. Not to mention that jsperf seems to be broken.


How to parallelize

This is my code
`
import json
import os
from concurrent.futures import ThreadPoolExecutor as Exe
from io import BytesIO
from time import time
from imageai.Detection import ObjectDetection
execution_path = os.getcwd()
detector = ObjectDetection()
detector.setModelTypeAsRetinaNet()
detector.setModelPath(os.path.join(execution_path, "resnet50_coco_best_v2.0.1.h5"))
detector.loadModel(detection_speed='fast')
with open('SeriImageArray.json', 'rb') as f:
res = json.load(f)
pars = [[detector, BytesIO(bytes(image['Image']))] for image in res['Imagelist']]
def detect_pictures(_pars):
t = time()
model, image = _pars
detections = model.detectObjectsFromImage(input_image=image,
minimum_percentage_probability=30,
input_type="stream",
output_image_path='111.png'
)
for eachObject in detections:
    print(eachObject["name"], " : ", eachObject["percentage_probability"], " : ", eachObject["box_points"])
    print("--------------------------------")

print(time() - t)

if name == 'main':
# for par in pars:
#     detect_pictures(par)
with Exe() as Pool:
Pool.map(detect_pictures, pars)`
but nothing happened.
thanks

`
import json
import os
from concurrent.futures import ThreadPoolExecutor as Exe
from io import BytesIO
from time import time
from imageai.Detection import ObjectDetection
execution_path = os.getcwd()
detector = ObjectDetection()
detector.setModelTypeAsRetinaNet()
detector.setModelPath(os.path.join(execution_path, "resnet50_coco_best_v2.0.1.h5"))
detector.loadModel(detection_speed='fast')
with open('SeriImageArray.json', 'rb') as f:
res = json.load(f)
pars = [[detector, BytesIO(bytes(image['Image']))] for image in res['Imagelist']]
def detect_pictures(_pars):
t = time()
model, image = _pars
detections = model.detectObjectsFromImage(input_image=image,
minimum_percentage_probability=30,
input_type="stream",
output_image_path='111.png'
)
for eachObject in detections:
    print(eachObject["name"], " : ", eachObject["percentage_probability"], " : ", eachObject["box_points"])
    print("--------------------------------")

print(time() - t)

if name == 'main':
# for par in pars:
#     detect_pictures(par)
with Exe() as Pool:
Pool.map(detect_pictures, pars)
`


stlink-tool: init at unstable-2023-10-20

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build stlink-tool

@GrahamcOfBorg build stlink-tool

@wegank Thank you for the review, and sorry for the long delay. I adjusted (mostly) everything according to your suggestions and rebased.

@GrahamcOfBorg build stlink-tool

@GrahamcOfBorg build stlink-tool
@Lassulus Would you please consider to merge this?


More guides docs changes to use Dependency Injection & createPlugin API

Fixes #142

docs/guides/dependencies.md | rendered
docs/guides/working-with-secrets.md | rendered

!merge