What BlMouseMoveBenchmark class>>#exampleRun is supposed to do ?

I'm running examples from BlMouseMoveBenchmark class>>#exampleRun. A lot of windows are opened and a mouse cursor is painted at the center of each. But nothing is moving. What this benchmark is supposed to do and to measure ?

My config :
BlocBenchs dev-1.0
Bloc dev-1.0
Windows 11 Pro
Pharo-11.0.0+build.389

Thanks :)

Thanks you for asking... I didn't put any explanation in the code. One important information is that Transcript must be open before running the benchmark, to see the results.
I had to browse the class to remember! I will push some updates on the code...
Context: This was an ad-hoc benchmark that I developed in a Workspace not only to measure speed but also to understand more of how the "event bubbling" works in Bloc. Then at tome point, to do not loose the script, I made it a class. And then added this small arrow cursor mostly "for fun".
Subject: BlMouseMoveBenchmark>>#mouseMoverLoopOn:at: emulates a mouse being moved constantly.
The fake events are positioned where that small arrow points. The benchmark iterates over several geometries and hosts because in some combinations, the "hit testing" (determining if clicked point is inside an element) was ending in a FFI call to cairo_in_fill function and was significantly slower.
But after, I implemented the hit testing on Pharo for these geometries so there is no big difference:
(Copy-pasted from Transcript)
Cairo (400@400) (levels=10):
| #                          | Indirect |   Direct |
|----------------------------|----------|----------|
| BlEllipseGeometry          |   900.05 | 28728.76 |
| BlRoundedRectangleGeometry |   898.08 | 29360.18 |
| BlSquareGeometry           |   894.17 | 25400.44 |
| BlRectangleGeometry        |   900.02 | 26758.15 |
| BlCircleGeometry           |   907.22 | 30155.71 |
| BlTriangleGeometry         |   902.58 | 26069.14 |

Cairo (400@400) (levels=5):
| #                          | Indirect |   Direct |
|----------------------------|----------|----------|
| BlEllipseGeometry          |   451.62 | 71607.84 |
| BlRoundedRectangleGeometry |   451.67 | 80020.39 |
| BlSquareGeometry           |   450.82 | 75871.68 |
| BlRectangleGeometry        |   466.96 | 74082.75 |
| BlCircleGeometry           |   451.50 | 83274.95 |
| BlTriangleGeometry         |   450.36 | 62463.01 |

Cairo (400@400) (levels=1):
| #                          | Indirect |    Direct |
|----------------------------|----------|-----------|
| BlEllipseGeometry          |    90.60 | 252609.96 |
| BlRoundedRectangleGeometry |    90.29 | 259802.28 |
| BlSquareGeometry           |    89.96 | 250084.57 |
| BlRectangleGeometry        |    90.20 | 246696.46 |
| BlCircleGeometry           |    90.41 | 262999.40 |
| BlTriangleGeometry         |    90.35 | 222038.78 |

I pushed a little improvement on this benchmark in 36aae5d


fix bug where last character in search keyword couldn't be cleared

This fixes the bug in issue #825. User couldn't clear last character in search keyword because routes.js automatically sets it to be query string. We should check if user tries to clear last character before set searchQuery in Session.

Thanks! By the way, if you can try to make future PRs against the devel branch :)

Sorry I just created a new pull request against devel:  https://github.com/TelescopeJS/Telescope/pull/833


[InvalidArgumentException] Script "post-install-cmd" is not defined in this package

Laravel Version: 5.5.0
PHP Version: 7.1.7 (Homestead)
Database Driver & Version: MySQL (Homestead)

Description:
I'm using Homestead and since 5.5 when I run laravel new app it gives the following error:
 [InvalidArgumentException]
  Script "post-install-cmd" is not defined in this package

A related issue is that it's not automatically generating the application encryption key when I do a fresh install.
Steps To Reproduce:
Just run laravel new app

Hello @rodrigoroma
please update your composer global dependency packages,  using composer global update, it will solve your issue.

Please upgrade your installer. Your version is not the latest :)

Thanks @maldechavda and @themsaid !
In fact, it was on the documentation.


feat: upgrade native sdk 6.1.1-ps.411191

Update native sdk 6.1.1-ps.411191 dependencies

This pull request is trigger by bot, you can checkout this branch and update it.

Examples built:

https://nightly.link/littleGnAl/hoe/actions/runs/9218986916/agora_rtc_engine_android_6.1.1-ps.411191_20240524052142.zip
https://nightly.link/littleGnAl/hoe/actions/runs/9218986916/agora_rtc_engine_ios_6.1.1-ps.411191_20240524052417.zip
https://nightly.link/littleGnAl/hoe/actions/runs/9218986916/agora_rtc_engine_macos_6.1.1-ps.411191_20240524051813.zip
https://nightly.link/littleGnAl/hoe/actions/runs/9218986916/agora_rtc_engine_windows_6.1.1-ps.411191_20240524051654.zip

This comment is commented by bot, do not edit it directly


Error when using projectdiscovery/chaos-client: panic: runtime error: invalid memory address or nil pointer dereference

chaos-client version:
v0.4.0
Current Behavior:
I got some errors and scan whole terminal freezed while scanning for subdomains
Command i used
chaos -d test.com | httpx -o live.txt
Error i got
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x38 pc=0x6b4028]
goroutine 6 [running]:
github.com/projectdiscovery/chaos-client/pkg/chaos.(*Client).do(0xc00007a240, 0xc0000f05d0)
/home/sebin/go/pkg/mod/github.com/projectdiscovery/chaos-client@v0.4.0/pkg/chaos/chaos.go:37 +0x148
github.com/projectdiscovery/chaos-client/pkg/chaos.(*Client).GetSubdomains.func1(0xc000090120)
/home/sebin/go/pkg/mod/github.com/projectdiscovery/chaos-client@v0.4.0/pkg/chaos/chaos.go:120 +0x1bb
created by github.com/projectdiscovery/chaos-client/pkg/chaos.(*Client).GetSubdomains
/home/sebin/go/pkg/mod/github.com/projectdiscovery/chaos-client@v0.4.0/pkg/chaos/chaos.go:111 +0xe5
Screenshot:

@0xSebin fixed in latest release - https://github.com/projectdiscovery/chaos-client/releases/tag/v0.5.0


How to get HAR with Safari and IE via browsermob-proxy?

{
"log": {
"comment": "",
"entries": [],
"version": "1.2",
"pages": [
{
"pageTimings": {
"comment": ""
},
"comment": "",
"title": "http://www.example.co.uk",
"id": "http://www.example.co.uk",
"startedDateTime": "2015-09-30T19:00:58.134+03:00"
}
],
"creator": {
"comment": "",
"version": "2.1.0-beta-2-legacy",
"name": "BrowserMob Proxy"
}
}
}

Is there a question here? This appears to be a snippet from a HAR file.

It is HAR, but I need HAR with captureHeaders and captureContent.

You need to enable the capture types you want. See BrowserMobProxy.setCaptureTypes(...) or .enableCaptureTypes(...) for details.

This question appears to be answered, so I'll close it.


[ML] chore: Mark IndexOperations as experimental.

Description
This pull request addresses from API review feedback, and ensures that MLClient.indexes is marked as experimental.
Related to #35251
All SDK Contribution checklist:

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

General Guidelines and Best Practices

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

Testing Guidelines

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

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Chat Commands

Which file contains the commands. (i.e. -dark, -ping)
Also how do you add new commands such as add mass. (-massboost)

This is where you can register for new command and write the code logic for it
https://github.com/huytd/agar.io-clone/blob/2a03b5039de738c1212c1bfbc3cf6ade8fa1129f/src/client/js/app.js#L426-L459

I have added a command but it will not work. I have registered it in app.js and added logic in in sever.js. When I type it in the chat box and hit [ENTER] it just stays in the chat box and does not do anything else. Does not send it as a chat MSG or say the command does not exist, it just stays there.

@16beeyee Can you add the changes to your fork on GitHub? Can't really help if we don't know what you did.

There I put my changes in the src file in my fork.

Is there a guide on the command writing API / abstractions anywhere? Like the name of the user could be user.name or something.

@16beeyee how can you run your code? I mean, it's doesn't pass the test.
Just checked your fork, and here is what I get when running:
src/server/server.js: line 337, col 61, Expected ')' and instead saw ' used massboost successfully'. (W116)
src/server/server.js: line 337, col 91, Missing semicolon. (W033)
src/server/server.js: line 337, col 91, Expected an identifier and instead saw ')'. (E030)
src/server/server.js: line 337, col 91, Expected an assignment or function call and instead saw an expression. (W030)
src/server/server.js: line 331, col 17, 'datd' is not defined. (W117)

5 errors

After you fixed the errors above, it should work

This is what happened when i ran the code and typed the command.
I couldn't take a screenshot to show.
$ npm start

<EMAIL_ADDRESS>start /home/___________/agar.io-clone
gulp run

[09:09:13] Using gulpfile ~/agar.io-clone/gulpfile.js
[09:09:13] Starting 'lint'...
[09:09:13] Starting 'move-client'...
[09:09:21] Finished 'move-client' after 7.89 s
[09:09:21] Finished 'lint' after 8.29 s
[09:09:21] Starting 'build-client'...
[09:09:22] Starting 'build-server'...
[09:09:22] Starting 'test'...
[09:09:22] Finished 'test' after 7.48 ms
util.js
#massToRadius
✓ should return non-zero radius on zero input
✓ should convert masses to a circle radius
#validNick
[09:09:25] Finished 'build-server' after 3.56 s
✓ should allow empty player nicknames
✓ should allow ascii character nicknames
✓ should disallow unicode-dependent alphabets
✓ should disallow spaces in nicknames
#log
✓ should compute the log_{base} of a number
#getDistance
✓ should return a positive number
8 passing (2s)
[09:09:26] Version: webpack 1.12.6
Asset    Size  Chunks             Chunk Names
app.js  211 kB       0  [emitted]  main
[09:09:26] Finished 'build-client' after 4.89 s
[09:09:26] Starting 'build'...
[09:09:26] Finished 'build' after 7.37 μs
[09:09:26] Starting 'run'...
[09:09:26] Finished 'run' after 1.94 ms
[09:09:26] [nodemon] 1.8.1
[09:09:26] [nodemon] to restart at any time, enter rs
[09:09:26] [nodemon] watching: .
[09:09:26] [nodemon] starting node ./server/server.js config.json
{ x: 0, y: 0, h: 3500, w: 3500, maxChildren: 1, maxDepth: 5 }
listening on *:3000
A user connected! player
User #M3XK3xrDf_EN3aPuAAAA respawned
Player M3XK3xrDf_EN3aPuAAAA connecting
Player M3XK3xrDf_EN3aPuAAAA connected!
Total player: 1
User #M3XK3xrDf_EN3aPuAAAA disconnected

Are you sure you're running the version with your edits? It works fine for me after you fix the string to number issue. For better debugging, you could add a console.log along with sending the serverMSG

Alright

alright, I've tried to do that i just can't seem to remember how.


To make it clear stateful vs stateless execution

Might be better to separate folders

Separated now in config Stateful


Support TRIM_HORIZON shard iterator type

Currently, the trigger only supports LATEST shard iterator type, which reads from the latest record in the stream onwards. This can be fine depending on the task, but there are cases where it is desirable to read all existing records in the stream.
See https://docs.aws.amazon.com/cli/latest/reference/kinesis/get-shard-iterator.html#options for details on additional shard iterator types supported by Kinesis.

Resolved in version 0.3.0


compat: Walled Templates

Walled Templates 0.8+ on Foundry V12 has a tab on the Item Sheet.
Work with the module author to determine if we can establish compatibility.
https://foundryvtt.com/packages/walledtemplates
https://github.com/caewok/fvtt-walled-templates

Added in Walled Templates v0.8.1.

Thank you caewok and morepurplemorebetter!


[REQUEST] ArchesOrigins

I am adding a zip of the addons they are mods im not sure who the authors are but i dont have the time to write all the coding for the data pack rn so here are the links:
https://www.curseforge.com/minecraft/mc-mods/archesorigins/download/3373987
https://www.curseforge.com/minecraft/mc-mods/apocalypse-origins/download/3487938
https://www.curseforge.com/minecraft/mc-mods/extra-origins/download/3489452
https://www.curseforge.com/minecraft/mc-mods/origins/download/3488415

almost forgot for the elemental spirit i want a limit of 3 players and for everything else 1 player

Please only request one datapack/addon per issue

Ok sorry i didn't know

It's all good! Feel free to create separate issues for Extra Origins and Apocalypse Origins

i will do that tmrw i am going to sleep

ok i made all the seperate issues :D

how far is the progress so far?

how far is the progress so far?

Hey @Appearedzoo! Sorry for the late reply! I've been busy with school work lately, so I didn't have the time to work on it. However, I now have some time, and I might finish it by tomorrow :D

I completely understand and its fine

Here you go! arches-origins-limiter.zip
If something is not working properly, please do tell!


Modify swift version action and add swift 6

Summary by CodeRabbit

新功能

更新了Swift项目构建工作流，支持Swift 6.0和5.9版本。
新增步骤以显示当前使用的Swift版本。

变更

移除了对所有分支的push事件触发，仅保留针对main分支的pull_request事件。

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 13.20%. Comparing base (169830c) to head (41a8d3d).
Report is 7 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   12.91%   13.20%   +0.29%
==========================================
  Files         113      110       -3
  Lines       15513    19048    +3535
==========================================
+ Hits         2003     2515     +512
- Misses      13510    16533    +3023

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


Multiple javascript files are loading on static build in Nuxtjs 3

Environment
production
Reproduction

Setup new nuxtjs 3 project
Create an index.vue page
Add static content (only HTML and CSS)
run npm run generate

Describe the bug
I want to generate a static site which has 0 dependency on javascript. The project is only built using HTML and SCSS.
However, after opening the static generated file, it loads various javascript files in the browser’s network tab.
How do you get rid of javascript files after the build?
<link rel="modulepreload" as="script" crossorigin href="/_nuxt/entry.uEwHeBjl.js">
<link rel="modulepreload" as="script" crossorigin href="/_nuxt/default.D3HJ612v.js">
<link rel="modulepreload" as="script" crossorigin href="/_nuxt/fetch.X0yPUN-o.js">
<link rel="modulepreload" as="script" crossorigin href="/_nuxt/index.m3rl-4ZK.js">
<link rel="modulepreload" as="script" crossorigin href="/_nuxt/vue.f36acd1f.fZrwODD9.js">
<link rel="prefetch" as="script" crossorigin href="/_nuxt/error-404.3IAJHKGW.js">
<link rel="prefetch" as="script" crossorigin href="/_nuxt/error-500.M14LAvY2.js">
<script type="module" src="/_nuxt/entry.uEwHeBjl.js" crossorigin></script>

Additional context
No response
Logs
No response

JavaScript is necessary if you want interactivity and an SPA-like behavior after the page was loaded.
If you don't need it, you can disable it via noScripts (can also be done per route.
You might also want to look into Server Components for a more granular approach to that.


Support for OpenTracing naming conventions

Understand OpenTracing lacks a standardised interface specification however I think it would be a great step forward if the terminology were adopted by this project;
See also; https://github.com/aws/aws-xray-sdk-go/issues/31
And perhaps provide some support for the Flask tracer;
https://github.com/opentracing-contrib/python-flask

Hi @nfisher
We recently announced AWS X-Ray support for OpenTelemetry Python SDK. You can find the documentation here to get started.
Closing this issue. Please open an issue here for AWS related stuff with OpenTelemetry Python. Thanks!

Hi @nfisher
We recently announced AWS X-Ray support for OpenTelemetry Python SDK. You can find the documentation here to get started.
Closing this issue. Please open an issue here for AWS related stuff with OpenTelemetry Python. Thanks!


Add and keep a proper change log

We’ve gotten serious about doing this with Bourbon and I think it would be quite valuable to do the same for Bitters.

Here’s Bourbon’s change log
Keep a CHANGELOG

Also, I find SemVer a bit hazy when versioning UI, but this article has some particularly interesting thoughts on that: https://medium.com/design-optimizely/how-to-version-your-ui-library-1c7a1b7ee23a


[BUG] can't reset Backup Target Credential Secret

Describe the bug
I can see that there is an error when setting Backup Target Credential Secret to empty value
To Reproduce
Steps to reproduce the behavior:

Install Longhorn

Deploy testing backupstore and it's secret using https://raw.githubusercontent.com/longhorn/longhorn-tests/master/manager/integration/deploy/backupstores/minio-backupstore.yaml

Set Backup Target to s3://backupbucket@us-east-1/ and to minio-secret and click save, see that values have changed.

Open Browser console, Network tab and Try to set Backup Target Credential Secret to empty string and click save

Longhorn API 500 Server error

Reload the setting page, see that Backup Target Credential Secret still have minio-secret value.

Expected behavior
we should be able to reset Backup Target Credential Secret to empty value.
Log
If applicable, add the Longhorn managers' log when the issue happens.
You can also attach a Support Bundle here. You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment:

Longhorn version: master
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE

Number of management node in the cluster: 1
Number of worker node in the cluster: 4

Node config

OS type and version: Ubuntu 20.04
CPU per node: 4
Memory per node: 8 GB
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): AWS EC2
Number of Longhorn volumes in the cluster: 0

Additional context
N/A

Related PR https://github.com/longhorn/longhorn-manager/pull/970

As per the integration test run, Looks like it is working after the fix.

Validation: PASSED


Change deprecated std.string.munch to basic popFront

This mostly looks good, but I'd rather not loose the knowledge about the protocol embedded in those munch statements (namely, the knowledge of what the expected value is). Better yet, that info could also be harnessed to help detect if (for whatever reason) the communication somehow manages to go out-of-sync or otherwise horribly awry.
Could you add an extra statement just before each of the new popFront's and do a enforceEx!MYXProtocol(...) to make sure the value about to be popped is infact the value expected?
Or at the very least, add a comment to each one showing what the expected value is (so I can convert to appropriate enforceEx!MYXProtocol later) For example:
munch(s, ":"); --> popFront(s); // TODO: Expected: ":"

Thanks, looks good.


Adding Segment Anything to the model zoo!

Polishes up @Rusteam's work from https://github.com/voxel51/fiftyone/pull/3019.
Segment Anything is now officially available in the FiftyOne Model Zoo! 📈
Box prompt example
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset(
    "quickstart", max_samples=25, shuffle=True, seed=51
)

model = foz.load_zoo_model("segment-anything-vitb-torch")

# Prompt with boxes
dataset.apply_model(
    model,
    label_field="segmentations",
    prompt_field="ground_truth",
)

session = fo.launch_app(dataset)

Keypoint prompt example
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset(
    "coco-2017",
    split="validation",
    label_types="detections",
    classes=["person"],
    max_samples=25,
    only_matching=True,
)

# Generate some keypoints
model = foz.load_zoo_model("keypoint-rcnn-resnet50-fpn-coco-torch")
dataset.default_skeleton = model.skeleton
dataset.apply_model(model, label_field="gt")

model = foz.load_zoo_model("segment-anything-vitb-torch")

# Prompt with keypoints
dataset.apply_model(
    model,
    label_field="segmentations",
    prompt_field="gt_keypoints",
)

session = fo.launch_app(dataset)

Automatic segmentation example
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset(
    "quickstart", max_samples=5, shuffle=True, seed=51
)

model = foz.load_zoo_model("segment-anything-vitb-torch")

# Automatic segmentation
dataset.apply_model(model, label_field="auto")

session = fo.launch_app(dataset)

LGTM and it works on my machine 👍 . The one thing worth noting is that it doesn't install segment-anything for you. You need to install the package for it to work. Might consider adding the instructions:
pip install git+https://github.com/facebookresearch/segment-anything.git

FYI- without segment-anything installed, this command:
import fiftyone.zoo as foz

model = foz.load_zoo_model("segment-anything-vitb-torch")

should show a helpful error message:
PackageError: The requested operation requires that 'segment-anything' is installed on your machine.

If you think this error is inaccurate, you can set `fiftyone.config.requirement_error_level` to 1 (warning) or 2 (ignore).
See https://docs.voxel51.com/user_guide/config.html for details.

If you want the dependency to be automatically installed, one can use:
model = foz.load_zoo_model("segment-anything-vitb-torch", install_requirements=True)


ORC-651: Use GitHub Pull Request Template

What changes were proposed in this pull request?

Why are the changes needed?

How was this patch tested?

+1

Thank you, @pgaref .


[Compute] az capacity reservation group create/update: Add new parameter --sharing-profile to support sharing capacity reservation group across subscriptions

Related command

az capacity reservation group create/update
Description

Changes related to sharing a capacity reservation group. https://github.com/Azure/azure-cli/issues/28327
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


strategy is deprecated use type warning .

https://stackoverflow.com/questions/76723290/using-the-new-type-for-uuidgenerator-instead-of-strategy

https://stackoverflow.com/questions/77128492/uuidgenerator-bean-type-mismatch/77132434#77132434


Remove the hard coded two-input structure for Sector

At least I think we should probably do that, but not entirely sure, actually.

First step is to actually understand what equation MPSGE creates for a production function with more than two inputs :)

Also done, I think?

I believe so, yes. Thanks for closing.


Change Send ReturnType Description

Change the description for the Send return type to say send button rather than search

Docs Build status updates of commit eb3489e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Xamarin.Forms/ReturnType.xml
:white_check_mark:Succeeded

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


Support concatenation of arrays

We are in a situation where we want to prepend an element to an array.
HOCON does allow that, however the plugin doesn't recognize the syntax :(
@dragos If you could merge that and release a new version that would be really great! We already made a custom built and use the plugin with this PR included and it works great. Thank you!

Thank you!

If you're interested in maintaining the project I can look into giving you rights to release on Bintray. Hopefully it's not too complicated.

@dragos Yeah, why not? If you can grant me rights I will see what I can do :smile:
Also I just recognized that my PR ruined the outline view... Pretty sure because of (value += Array)+...
Do you have an idea how to fix this?

I need to create an organization in order to give you publish permissions, so it will take a few days to sort the Bintray UI out in the midst of all the other things I have to do now.
the outline view is very sensitive to the grammar, and that's the reason why #5 is still blocked. TBH I forgot all the details of it, but I think there is some code that tries to fix grammar issues here, maybe that's enough to get you started

Actually I alrady fixed the outline :smile:

@dragos Can you please release a new version? Thanks!

... or give me publish permissions? :wink:

I'm trying to release, but I'm running into some failed dependencies that come from nexus.codehaus.org, which is no longer available. I'll see what I need to do to upgrade.

@dragos A while ago I built it myself and published it in our local repository. What I did:
# make sure everything is clean:
find -name generated | xargs rm -rf ;
find -name generated-sources | xargs rm -rf ;
find -name src-gen | xargs rm -rf ;
find -name xtend-gen | xargs rm -rf ;
find -name target | xargs rm -rf ;

cd com.typesafe.hocon.releng/ ;
mvn -e install ;

# Now see the zip file in ~/.m2/repository/com/typesafe/hocon/com.typesafe.hocon.updatesite/${VERSION_FOLDER}

I'm getting a lot of
[WARNING] Checksum validation failed, expected <!DOCTYPE but is b6e79499f3d2a39bc5a2d77ca2ae3ef002b9d1ea for http://nexus.codehaus.org/snapshots/org/eclipse/emf/org.eclipse.emf.codegen/2.10.0-v20140127-SNAPSHOT/maven-metadata.xml

while downloading dependencies, no idea where these dependencies are coming from..

Just now I removed my ~/.m2 folder and built the project again. Everything worked.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] parent ............................................. SUCCESS [  2.213 s]
[INFO] Typesafe Config Core ............................... SUCCESS [ 57.480 s]
[INFO] Typesafe Config UI ................................. SUCCESS [  2.353 s]
[INFO] Typesafe Config Tests .............................. SUCCESS [  3.979 s]
[INFO] Typesafe Config Feature ............................ SUCCESS [  0.085 s]
[INFO] Typesafe Config Update Site ........................ SUCCESS [  1.450 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 05:27 min
[INFO] Finished at: 2017-09-08T14:03:27+02:00
[INFO] Final Memory: 129M/1379M
[INFO] ------------------------------------------------------------------------

I am using maven 3.5:
$ mvn -version
Apache Maven 3.5.0 (ff8f5e7444045639af65f6095c62210b5713f426; 2017-04-03T21:39:06+02:00)
Maven home: /opt/maven/maven-3.5
Java version: 1.8.0_144, vendor: Oracle Corporation
Java home: /opt/jdk1.8.0_144/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "4.10.0-33-generic", arch: "amd64", family: "unix"

I pushed a release. I also created a new Bintray organization, I'll send you an invite. You should then be able to release there. (current release is in the old place)

Do you have a Bintray username?

I just signed up now, it's mkurz

Can you confirm that you can publish to the new repository? Perhaps a re-take of 1.0.6, and if all works out we can change the official repo to be the new one.


Implement (missing?) period at end of doccomment

I'm not sure if this was missed or not in scope of the project but would you consider adding it?
https://dart.dev/guides/language/effective-dart/documentation#do-format-comments-like-sentences
Thank you

nvm I realized I should check the linter rules and see it is not an option, my apologies.

Hi, no problem. (: As you mentioned, there is no lint rule for that.


[framework] Closed Loop Token

Description
Adds the sui::token module to the framework.
Test Plan
To be ported in future updates.

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[x] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Adds a closed loop token to the framework

Updated the PR and removed the owner field (which was intended as TTO prevention).
Thanks @tzakian

Reverted snapshots for protocol version 32, which are too early to be created since this version hasn't shipped yet (31 is just shipping to testnet). Also reverted a change to the protocol version 31 for deepbook, that doesn't look like it should be there.


Voyager Broken menu builder

Laravel Installer 2.1.0
Voyager: v0.11
PHP 7.2.19
mysql:  8.0.16

Hello guys.
Suddenly my Menu Builder is empty and i can not change the navbar to the left. Can you help me please? Thank you! :)
 https://imgur.com/Xt0hsYF

Please fill out the rest of the issue template.
Those information are crucial for us to help you.
This can be re-opened once you filled out the rest.


fiberzerolog: Use string for latency field

Fixes https://github.com/gofiber/contrib/issues/704

Pls provide a documentation change and unittests for what this change involves so that it remains audited and documented

@ReneWerner87 The current tests dont check any of the fields used by this middleware

Actually it does check fields, but doesnt check latency at all. We can at least add a test to check the field type.
The regular logger and fiberzap all return "Strings" for latency but this middleware is not. The changes are more of a consistency thing.

govulncheck failure is related to go1.20.7 being released. Related: https://github.com/actions/setup-go/issues/405

I have added the unit test for latency

@raphielscape Thanks! Could you also added in the test at the bottom for "Logger All", the one with all the fields?

This test: https://github.com/gofiber/contrib/blob/6fa822cca2e7421e5a844746068ea4986f2f8d5c/fiberzerolog/zerolog_test.go#L143

Done

@raphielscape What I was thinking is adding the latency field on this list here: https://github.com/gofiber/contrib/blob/6fa822cca2e7421e5a844746068ea4986f2f8d5c/fiberzerolog/zerolog_test.go#L152 and then checking the expected value below.
You added a whole new Fiber App in that test which already has a FiberApp

@raphielscape What I was thinking is adding the latency field on this list here:
https://github.com/gofiber/contrib/blob/6fa822cca2e7421e5a844746068ea4986f2f8d5c/fiberzerolog/zerolog_test.go#L152
and then checking the expected value below.
You added a whole new Fiber App in that test which already has a FiberApp

I have explained it in the commit description, we can't use the same app because we can't do Equal Assertion as latency varies between runs

We can make the first fiber app have a endpoint that returns OK and use that in test instead, though

@gaby good?

Thanks! That looks good

For the reason why I don't use AssertEqual is that I have no idea how to make an equally valued latency and latency return always varies between test runs, please let me know if there was a way to make the latency in an equilibrium so that we got an equal value that can be compared so that I can use AssertEqual


0.2.0 release

See the CHANGELOG.md changes for what is included functionality wise.
In addition to the functional changes, this PR will enable the helm chart repo via GitHub Pages.

LGTM


Conn: Added appropriate responses to the recorded errors

When bufio.writer.Flush() records error now whole loop terminates because writer doesn't write after error is recorded.

When conn.Close() error is recorded then it's logged, worst case for now is that connections file descriptor is still open.

Fixes #140

Let's move assignments to if statements.
if err := ...;...  {}

What if here:
size := len(c.requestCh)
		if size > maxCoalescedRequests {
			size = maxCoalescedRequests
		}

size is equal to 0?
In that case, don't we have some form of active waiting?
Should we perhaps set size to be greater or equal to 1?

What if here:
size := len(c.requestCh)
		if size > maxCoalescedRequests {
			size = maxCoalescedRequests
		}

size is equal to 0? In that case, don't we have some form of active waiting? Should we perhaps set size to be greater or equal to 1?

I second that motion. Right now we do have active waiting.
@mmatczuk shall we deal with this in another issue or should I fix this here?

If size is 0 than you wait on channel it mean there is no request.

It looks that I was wrong and plain close on flush error is ok because it will be handled by draining handlers.


kubectl-gadget: 0.32.0 -> 0.33.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/inspektor-gadget/inspektor-gadget/releases.
meta.description for kubectl-gadget is: Collection of gadgets for troubleshooting Kubernetes applications using eBPF
meta.homepage for kubectl-gadget is: https://inspektor-gadget.io
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.33.0 with grep in /nix/store/9vmsdrpfnx4aq6ka0cmi4rf0aj1d59bp-kubectl-gadget-0.33.0
found 0.33.0 in filename of file in /nix/store/9vmsdrpfnx4aq6ka0cmi4rf0aj1d59bp-kubectl-gadget-0.33.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

kubectl-gadget

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9vmsdrpfnx4aq6ka0cmi4rf0aj1d59bp-kubectl-gadget-0.33.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/1451e665c4abc39c3cda2ce1829a5735c96d055f#kubectl-gadget

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9vmsdrpfnx4aq6ka0cmi4rf0aj1d59bp-kubectl-gadget-0.33.0
ls -la /nix/store/9vmsdrpfnx4aq6ka0cmi4rf0aj1d59bp-kubectl-gadget-0.33.0/bin

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

1 package built:

kubectl-gadget

Maintainer pings
cc @kranurag7 @devusb for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Result of nixpkgs-review pr 347096 run on aarch64-darwin 1

1 package built:

kubectl-gadget

@NixOS/nixpkgs-merge-bot merge

woah, didn't knew about this one. This looks like a good addition from the nix team. Thanks for your review.


Breaks Apollo query deduplication

Issue
Using either loader or gql from this package breaks apollo-client's query deduplication.
Steps to Recreate
In two different components, fire the exact same query by either importing the same .graphql file with loader, or defining the same query with gql from this package. Render both components together, and you will see two network requests fire.
Expectation
If you do the same as the above but with gql from the graphql-tag package, Apollo recognizes that the two queries are the same, and only fires one network request. Could be related to https://github.com/apollographql/apollo-client/issues/4930
Queries are deduplicated only according to the object reference of the DocumentNode.
I imagine this package is returning a different DocumentNode every time the query is imported / defined rather than building up a cache of already-compiled queries and check against that.
Is caching something that's in scope for this package?

For anyone else that comes by, my solution here was to configure Rescripts and use graphql-tag instead of this package.
You'll also require jest-transform-graphql if you're running tests against code that imports graphql files. This issue helped me correctly configure it for Jest.
Once you've done the above, you can simply import query from 'somewhere.graphql', and because graphql-tag caches correctly, Apollo's deduplication works as expected, and only one of the identical queries fires at a time.

When you have the same fragment multiple times in a query through nesting, it throws an error:
[GraphQL error]: Message: There can be only one fragment named "wallCommentFragment"., Location: [object Object],[object Object], Path: undefined


How to set Dark Mode?

Hello,
KeyboardKit supports dark mode but I couldn't find the "switch".
I want to enable/disable dark mode
Thank you

Hi @irakli941
If you mean system-wide dark mode, you will have to enable that from the system controls.

Hello! Thanks for the quick reply and great framework.
In the provided example there are two cells, light content and dark content which respectively call dark and light keyboards.
In demobutton.swift I can set the buttonview's tint, background, and text color. But I couldn't find how to change the keyboard's background color, as you can see in the attached picture, it is still light.

it should look like this

If you use asset catalog colors with dark mode variants, or the correct system colors, they should adapt automatically when you enable dark mode.
For instance, this is how the KeyboardKit demo app looks in dark mode:

Yes, I understand that, but I want to enable/disable dark mode programmatically

Oh, ok. I'm not sure - doesn't this UIViewController approach work?
overrideUserInterfaceStyle = .light/.dark

Sadly no, but your solution will work if you own the ViewController/App in which your keyboard lives. My app is a custom keyboard extension, but thanks anyway.
I'll close the issue

Sorry I couldn't help. Good luck!


Error while running demo

I got following error,

ValueError                                Traceback (most recent call last)
 in ()
2 if torch.cuda.is_available():
3     xx = xx.cuda()
----> 4 y = net(xx)
/usr/local/lib/python2.7/dist-packages/torch/nn/modules/module.pyc in call(self, *input, **kwargs)
489             result = self._slow_forward(*input, **kwargs)
490         else:
--> 491             result = self.forward(*input, **kwargs)
492         for hook in self._forward_hooks.values():
493             hook_result = hook(self, input, result)
/home/saddam/PY/ssd.pytorch2/ssd.pyc in forward(self, x)
101                 self.softmax(conf.view(conf.size(0), -1,
102                              self.num_classes)),                # conf preds
--> 103                 self.priors.type(type(x.data))                  # default boxes
104             )
105         else:
/home/saddam/PY/ssd.pytorch2/layers/functions/detection.pyc in forward(self, loc_data, conf_data, prior_data)
52                 boxes = decoded_boxes[l_mask].view(-1, 4)
53                 # idx of highest scoring and non-overlapping boxes per class
---> 54                 ids, count = nms(boxes, scores, self.nms_thresh, self.top_k)
55                 output[i, cl, :count] =
56                     torch.cat((scores[ids[:count]].unsqueeze(1),
ValueError: need more than 0 values to unpack

Have u solved it? I have the same problem.

See #154. That issue says to change line 62, but for me, it was line 49.


Brainstorm guiding questions about the data

When analyzing conversion trends, both land type and time will ideally be considered. For instance, possible questions that could be asked of the data include:
• From 2001 to 2019, was Deciduous Forest more likely to convert to developed land than Cultivated Crops?
• Where has land converted more recently (e.g. from 2016-2019 vs. from 2001-2019)?
Once these past conversions have been identified, algorithms to predict future conversion can be developed. Ideally, there would be multiple future scenarios projected, e.g. likely conversion by 2030, 2040, 2050, 2060, 2070 etc.

We could


I have issues...

Hi Rich,
made the plots for two assemblies with the previous version and they are great.
what do you think about replacing the colouring of the outer GC-band with a wiggle/dot-plot of GC and N's for each scaffold (using fixed windowsize or scaffold/contig mean)?
that way one could tell wether for example half of the biggest scaffold is made up of N's ... or if you have GC outlier in the biggest scaffolds.
cheers,
dom

Sounds like a good way to increase the data-density of the plot - I like it!
Should be straightforward to implement so expect to see it in an update soon...

Finally released a version with this implemented


Add fetch of node.js.

hmm what about the vcpkg-tool-nodejs port then?

hmm what about the vcpkg-tool-nodejs port then?

They don't do exactly the same thing, since this gives you that version or later and that gives you an exact version.
I think the right way to expose this kind of stuff is ultimately vcpkg_find_acquire_program and medium term I would like to see that changed to always be done in terms of vcpkg fetch.
The bits that we want to power with this though are strictly speaking unrelated; it's in service of another product that is trying to merge into vcpkg that happened to be done in typescript. (If we had started adding that to vcpkg of course we would not have done that, but .... )


Allow newer versions of IDEA to control test iterations

I'm a big user of the 'run until failure' option on the IDEA test runner for finding bugs. I just upgraded to the newest version of IDEA and it moved its JUnit runner, so we need a modified permission setting for that option to work.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

Codecov Report

Merging #6623 (c4b225f) into main (bdb4f7a) will decrease coverage by 0.05%.
The diff coverage is 48.64%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #6623      +/-   ##
============================================
- Coverage     70.77%   70.72%   -0.05%
+ Complexity    59156    59131      -25
============================================
  Files          4804     4804
  Lines        283102   283168      +66
  Branches      40813    40820       +7
============================================
- Hits         200361   200283      -78
- Misses        66305    66485     +180
+ Partials      16436    16400      -36

Impacted Files
Coverage Δ

...pensearch/common/logging/OpenSearchJsonLayout.java
0.00% <0.00%> (ø)

...in/java/org/opensearch/index/shard/IndexShard.java
69.50% <0.00%> (-0.75%)
:arrow_down:

...s/replication/SegmentReplicationTargetService.java
49.03% <0.00%> (-0.97%)
:arrow_down:

...x/store/remote/filecache/FileCachedIndexInput.java
50.00% <33.33%> (-5.56%)
:arrow_down:

...g/opensearch/common/settings/WriteableSetting.java
69.58% <50.00%> (-2.00%)
:arrow_down:

...search/index/store/remote/filecache/FileCache.java
84.61% <75.00%> (-6.69%)
:arrow_down:

...n/java/org/opensearch/common/settings/Setting.java
86.32% <100.00%> (+0.19%)
:arrow_up:

.../main/java/org/opensearch/env/NodeEnvironment.java
75.77% <100.00%> (-0.62%)
:arrow_down:

... and 468 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Bump Helm to v3.10.0

Signed-off-by: Luca Comellini<EMAIL_ADDRESS>
Bumps Helm from v3.9.0 to v3.10.0
Pull Request Motivation

Kind

/kind cleanup
Release Note

NONE

/assign @wallrj

fixed in #5346


Views module docs

I'm experimenting with a few things here.
I've added a prelude and I think it is a really nice change. Whenever I create a new file I always have to do use module::* for several modules anyways so I figured it was about time to add a prelude.
I'm also experimenting with, in the views module, only doing pub use module::* for the most important views that are always used such as stacks, svg, container, text. I was already doing this for a few of the widgets that I had created such as dropdown and now I'm moving a few more to be like that. I feel like this is a good change. Leaving them in the modules makes the documentation for those items  and the whole views module more consistent.
I'm considering not doing pub use module::* at all inside of the views module and instead selectively publicly exporting them from the prelude. This would really help to clean up the views module documentation


Linux server suppport

The server will be able to run on linux.

Linux server support
The server can flawlessly run on linux.


[question] How can we develop with qgis-server?

Hello,
Thank you for developing this great software. I believe this is one of the most exciting server implementation among recent FOSS4G.
I've tried to edit some codes in this repository and found bbox-map-server depends on qgis-server/umn-map-server of a host machine (and only on Linux?).
https://github.com/bbox-services/bbox/blob/94406fa57a20b1132f68bfbdeab84dd1875b3cb5/bbox-map-server/src/wms_fcgi_backend.rs#L49
It is not unnatural and in container image those dependencies are clearly resolved, it's great.
However, trying to develop, this is a problem for me.
I guess there might be some tips for developing with qgis-server conveniently.
If there is, could you tell me how to develop with qgis-server?

Sorry, missed that during holidays.
The location of the QGIS Server location should be configurable: https://www.bbox.earth/map-server/reference.html#qgisbackend
Usage example:
[mapserver.qgis_backend]
exe_location = "/usr/local/cgi-bin/qgis_mapserv.fcgi"

But this is currently broken! Will fix it make a release in the coming days.

0.6.0-beta1 with fix is released.


Serialization of "detached/client" entities

First of all sir, your trackable-entities rocks!
I worked in a couple of huge projects using the STE and your "paradigm transformation" is exactly what I was looking for except for one missing but important part, the ability to serialize the client graph.
I try to explain it better describing my typical scenario:

the human operator, during her daily work open the client (website or winform)
she pick up an house (WorkItem) from her work queue (Real Estate Evaluation to do)
at this point the client call the service asking all the data related to the house to be evaluated, here EF take part retrieving the whole house graph and passing it to the client
here happens the first serialization/deserialization, server model is serialized and the resulting text is deserialized back into the client model, it works!
the operator start working in her detached world adding/deleting/changing entities within the graph
the operator ends her working days but not her works and she decide to "freeze" the work done so far
the system "should" be able to serialize the client graph and save it into a store (File System for example, or a BLOB db column)
the day after the operator reopen the same WorkItem deserializing back the graph saved at the previuos point
the operator does additional work against the client graph
Operator bush the save button and the code ask the ChangeTracker to prepare the "Merge" package to be sent to the service and invoke the service, here happens your second serialization/deserialization where client model is serialized and the resulting text is deserialized back into the service model, it works too!
the Service take the "Merge" package, it attach it to a context and save it, after that it return the saved entities to the client
the client merge the saved entities to its graph and all the statuses are reconciliated.

If we remove the point between 6 and 9 (6 and 9 included) your trackable-entities work perfectly, the problem arise with the need to serialize/deserialize from and to client model to add a "shut-down" period in the middle of the client work.
I think the "problem" (but it is even true that it is the magic that lets your trackable-entities to work so good) is the double nature of the "ChangeTrackingCollection", it works as a "standard" List when the info are within the service and as a "stateful" object when it is within the client, this trick has made the "Collection" (used also to keep the client work done so far) quite difficult to serialize using the most common serializer (DataContractSerializer, XmlSerializer, Json.Net, DataContractXmlSerializer, DataContractJsonSerializer, etc...) without heavily hacking it.
I hope I has been able to describe the "problem" I'm facing replacing the STE with your trackable-entities.
Thanks in advance for your time Tony,
Monducci Marco

Hello Marco,
Nice to meet you, and thank you for your question! I have good news for you.  ChangeTrackingCollection is serializable just like any other collection, without any special treatment.  Just pick your favorite serializer and you're good to go.  In fact, this is precisely how WCF and Web API transmit these collections between client and service.
To demonstrate this, I have created a GitHub repo: https://github.com/tonysneed/demo-trackable-serialization.  Here I simply use the Json.Net serializer to write customer orders to a file as JSON, then deserialize them back to a change tracking collection.  Have a look and let me know what you think.
The demo ReadMe has instructions for creating the NorthwindSlim database which I am using.  Note that if your graph contains cycles, simply pass a JsonSerializerSettings object with PreserveReferences set to All.

Just chipping for a small point, I was using Microsoft workflows to serialize the entities. I had to add [CollectionDataContract] over ChangeTrackingCollection class to actually make it serializable :)

@harrisunderwork Actually, [CollectionDataContract] isn't necessary with POCO's (no DataContract or DataMember attributes), which is how v2.5 works.
I've updated the serialization demo to include XML serialization with DataContractSerializer.  I also added settings to handle cycles in the object graph. tonysneed/demo-trackable-serialization@0482db90c08020716d0c4449458adba1a31e9310.

@tonysneed it's not completely true; ChangeTrackingCollection is serializable but it has some private members that are lost in case of serialization/deserialization for example the private collection of deleted entities.
Your smart trackable-entities uses the same object ChangeTrackingCollection in two different contexts:

Within an Entity POCO to represent a NavigationProperty for a 1-to-Many relationship
As ChangeTracker at client side (here as a replace for the STE detached logic)

When used in the second scenario, when you connect an entity to keep monitored, the ChangeTrackingCollection register its own events and once triggered it store into some private members the imformation about data changed. Thanks to this mechanism it is able to create a package containing all the entities with the appropriate status (deleted, updated, added) that the client can send to the server and the server can attach and Save against the DB, once the DB finishes all the entities are passed back to the client as Unchanged and the ChangeTrackingCollection used as ChangeTracker can merge them with the client graph reflecting the update made against the DB.
So, to be able to add a cold shutdown point in time in the middle between

when I start the  ChangeTrackingCollection used as ChangeTracker
and when I ask it for the ChangedEntities

this object must offer a serialization that can serialize also all the private properties.

I have forked your demo project and I changed just the Program.cs to show you what I mean.
In my sample the system is NOT working how I need.
https://github.com/MonDeveloper/demo-trackable-serialization.git

Hello Monducci!
I've re-opened the issue here, so I'll delete the other issue you created.
Alright, I think I understand better now. :)  We just need to get the serializer to include the _deleted items private field (as all the other tracking information is contained in the entities themselves as public properties).  This can be accomplished in a couple of ways.  One would be to add Json and DataMember attributes to that field, but another might be to configure the serializer programmatically.  I'll have a look at at the second approach for now, and I'll push a solution to the demo-trackable-serialization demo I created for this issue.
Out of curiosity, do you prefer Json.Net or the DataContract serializer?  Most of the time, Json would give you better performance.
In terms of your fork, the best would be for you to submit a pull request in order to share your code with me.
Cheers!

Sorry for the new Issue I've created, I'm quite new to github and I discovered this one was closed after I replied.
About the Serializer I like both for different reasons, Json.NET is more flexible and DataContractSerializer is much coupled with WCF.
Anyway, for this particular need I think Json.NET could is the better choice.
About the approah, Even I prefer the second approach ("to configure the serializer programmatically") because in this way the "standard" Serialization occurring within WCF can continue working as of today.
If you are interested, I already played this game and I could have some info useful for you:
Using Json.NET Custom serialization I tried two ways:

Building directly the JsonArray
Creating a new object that contains the same information contained into the ChangeTrackingCollection but without implementing ICollection (all as public properties)
But both failed with Stack Overvlow exception (because the ChangeTrackingCollection used as Change Tracker contains entities that contain many ChangeTrackingCollection properties, etc...)

Using DataContractSerializer I tried to:

Define a new class extending your ChangeTrackingCollection called SerializableChangeTrackingCollection creating a public Getter and Setter for your private fields using reflection, and using this instead of your everywhere (I changed the t4 templates)
Also this tentative failed with an error from DataContractSerilaizer because it doesn't support serialization for "Collection with public Properties"

Hope this info can help you @tonysneed finding out a viable solution.
Thanks for supporting us!
mM

It took me a while to learn how to work with pull requests in GitHub, but now I love it.  I encourage you to get more familiar with Git and GitHub -- they are the way of the future, especially since Microsoft has fully embraced them.
From what I've read, configuring Json.Net involves creating a custom ContractResolver and overriding CreateProperties.  I'll see what I can come up with and push it to the demo repo.  Cheers!

It turns out the custom ContractResolver approach does not work with collections.  So instead I opted for serialization callbacks using a SerializableChangeTrackingCollection subclass.  This works for JSON but not XML, because the DataContract serializer does not support serialization callbacks with collection types.  Here is the commit for tonysneed/demo-trackable-serialization@fc6b0016c094f8a168d4409a4e82e6b0cde5f47b.  So if you use Json.Net for serialization, you should be OK.  The cached deletes are restored on serialization and re-cached on deserialization.

@MonDeveloper, ok to close this issue?


The animatedluasprite command isnt working,

Describe your problem here.
I've been trying to add an explosion animation. The explosion animation is in the mods/images folder, the animation name on the .XML file is called, "explosion idle", here's the script. My problem is that the animation doesn't play.
function onStepHit()
if curStep == 1025 then
makeAnimatedLuaSprite('explosion', 'explosion', 850, 1200)
playAnim('explosion', 'explosion idle', false)
end
end
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Its using the Ourple Guy mod. (Not source code just the mod.)

you didnt even add the animation :skull:

oh mb

They meant you didn't add the animation to the sprite, in your code
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Lua-Script-API:-Object-Functions#animations-functions

i did i thought? I put "explosion idle", in the playanim. Where do I put it then?

That's not how it works
To play an animation you first have to add it to the object, read the wiki link for functions

I got it, thank you guys!!!


Module: renovate

It might be nice to have a module to configure renovate for a jhipster-lite generated project, with e.g. docker-compose dependencies update for content in src/main/docker (as done in #9059) and good default settings.

Can I pick this up, if no one is working on it?
And if I understand correctly, this has to be done for all dependencies in generated projects and not just docker images?

:+1: Exact, it's for handling all dependencies, not just docker.


Add option to skip PHP string extraction

In addition to skipping JS string extraction and skipping string audits (#94) I was thinking about a --skip-php option.
This could be useful when one only wants to extract strings from JS files (for whatever reason).
Obviously plugin/theme meta data would still need to be scanned to find the destination folder etc.

I'm working on this for contributor day...


add support for minimumKubeletVersion

/retest

How are we planning to test this observer? Are we going to have an integration test? At the very least, we could check if the configuration contains the current node version, right? I think we could have a more end-to-end test in origin.

I was planning on doing an e2e test in origin. I'm not seeing integration tests in this repository, do you have a pointer @p0lyn0mial ?

I decided to take config observation from library-go as iterating is tricky and AFAIU this code won't be used elsewhere

/retest

/retest

/retest

/retest

BTW: do we want to merge this pr before https://github.com/openshift/kubernetes/pull/2104 ?

BTW: do we want to merge this pr before openshift/kubernetes#2104 ?

I think I'd actually prefer o/k first

BTW: do we want to merge this pr before https://github.com/openshift/kubernetes/pull/2104 ?

I think I'd actually prefer o/k first

should we merge the origin tests before the o/k PR?

I wouldn't expect it to pass without the tests, but if we're okay with a window where techpreview serial jobs fail then I'm open. If I were to choose the order I'd say o/k, o/ckaso, o/origin. Since the feature is gated in both implementation PRs there shouldn't be a risk to merging without a test

as expected https://github.com/openshift/origin/pull/29353 failed

/retest

I wouldn't expect it to pass without the tests, but if we're okay with a window where techpreview serial jobs fail then I'm open

We cannot afford it. Is there a way to test these three PRs to ensure everything works as expected?
Could you also get a review on the test and maybe even a tag, so that all PRs can land roughly at the same time?
We could put a hold on the test PR until the other two PRs land.

/retest

/retest


Blank page: fails to load xterm.js

I installed nbgitpuller following the instructions in the README, and when I access an nbgitpuller URL, I just end up at a blank page.  Looking at the browser console shows that xterm.js is failing to load (both the JS and the CSS).  My guess is that this is related to the updated version of xterm.js included with jupyterhub/notebook v5.5.0.
I can get it to load the JS by changing index.js:4 to 'static/components/xterm.js/index.js'.
Then there is an error in the vendored _fitTerm() function due to changes in xterm.js.  I changed it to:
    GitSyncView.prototype._fitTerm = function() {
        // Vendored in from the xterm.js fit addon
        // Because I can't for the life of me get the addon to be
        // actually included here as an require.js thing.
        // And life is too short.
        var term = this.term;
        if (!term.element.parentElement) {
            return null;
        }
        var parentElementStyle = window.getComputedStyle(term.element.parentElement),
            parentElementHeight = parseInt(parentElementStyle.getPropertyValue('height')),
            parentElementWidth = Math.max(0, parseInt(parentElementStyle.getPropertyValue('width')) - 17),
            elementStyle = window.getComputedStyle(term.element),
            elementPaddingVer = parseInt(elementStyle.getPropertyValue('padding-top')) + parseInt(elementStyle.getPropertyValue('padding-bottom')),
            elementPaddingHor = parseInt(elementStyle.getPropertyValue('padding-right')) + parseInt(elementStyle.getPropertyValue('padding-left')),
            availableHeight = parentElementHeight - elementPaddingVer,
            availableWidth = parentElementWidth - elementPaddingHor;

        rows = parseInt(availableHeight / term.renderer.dimensions.actualCellHeight);
        cols = parseInt(availableWidth / term.renderer.dimensions.actualCellWidth);

        term.resize(cols, rows);
    };

I don't know if these are the best fixes, but they seem to work.  I haven't corrected the CSS URL, though, as I'm not quite sure where that is now.

Thanks for catching this! Looks like this would be the fix for the css:
-<link rel="stylesheet" href="{{ static_url("components/xterm.js/dist/xterm.css") }}" type="text/css" />
+<link rel="stylesheet" href="{{ static_url("components/xterm.js-css/index.css") }}" type="text/css" />

I'll test your fixes soon...


Implementation question

https://github.com/quantastica/quantum-circuit/blob/3c526d60dd6fe9f8e0e27a3ceeff081b66b74abc/lib/qasm_import/QASMParser.js#L208
atn deserializer expects a number[] and we pass a string in here, is this intentional?
ATNDeserializer().deserialize(serializedATN)

@dmvjs the code is automatically generated by antlr4, so I guess it is intentional.


Stop building Pulsar Client for Python 2.7

Motivation
We currently have several dependencies on Python 2.7. There is no need to build anything new for Python 2.7, as it has been deprecated for a while.
Modifications

Remove the defunct homebrew script (the real one lives upstream).
Update comments to indicate supported python languages.
Update the python client build so that it only releases clients for python 3.x.

Verifying this change
I have manually run the scripts for these components to verify that the tests pass.
Does this pull request potentially affect one of the following parts:
Yes, this is a breaking change for the python 2.7 client. Given that python 2.7 is no longer supported, this is an acceptable breaking change.
Documentation

[x] doc

There are some minor doc updates as a part of this PR.

I've tried to enable Python3 for CI some months ago, see https://github.com/apache/pulsar/pull/9684 and https://github.com/apache/pulsar/pull/8624. It's harder than I've thought.
It's right that the apachepulsar/pulsar-build:ubuntu-16.04-pb3 image doesn't have the Python3 support. We need to update the Dockerfile and add Boost Python3 dependency. But the most hard thing is the Python Functions incompatibility. It looks like Python Functions relies heavy on Python2.

@BewareMyPower - thank you for the additional context!

Superseded by https://github.com/apache/pulsar/issues/15185.


can't find var defined in typings.d.ts in cli v1.5

for example, I define below code in typings.d.ts:
declare module 'admin-lte' { }
and types settings as below:
'
"types": [
"jquery",
"echarts",
"datatables.net",
"bootstrap",
"admin-lte"
]
'
I will get a exception:TS2688: Cannot find type definition file for 'admin-lte'.
but if I remove types settins form tsconfig.app.json,the exception disappears;
it seems to be the typings.d.ts has not been complied

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request

Versions.

Repro steps.

The log given by the failure.

Desired functionality.

Mention any other details that might be useful.

@dhcn the types array is for types installed via @types/something The included d.ts file will be included in the ts compiler by default

Thanks,Every thing run as normal.the exception no longer appears.


Replace any "Sprint" and "T-Mobile" references with "New T-Mobile."

With T-Mobile now owning Sprint in the United States, any references to either company should be replaced with the new merger's name of "New T-Mobile."
This will help Sprint or T-Mobile customers find the right product to utilize Microsoft's resources correctly.

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

ID: 8de75f7d-a749-ec5f-435b-62bc163a2b72
Version Independent ID: 8d221f64-7c6b-d604-3dfa-51d8b9163a3f
Content: Surface System SKU reference
Content Source: devices/surface/documentation/surface-system-sku-reference.md
Product: w10
Technology: windows
GitHub Login: @CoveMiner
Microsoft Alias: v-jokai

@officedocsbot assign @jvsam

Thanks for letting us know about this @jebeld17. Although the approved deal between the two is all over the news today, there is no official announcement yet, except for this and it does not indicate that it's a done deal (or officially closed).

...today announced they have taken an important step toward completing their proposed merger to create the New T-Mobile by entering into a consent decree with the U.S. Department of Justice (DOJ). The DOJ action moves the merger one step closer to closing, pending other regulatory approvals and the satisfaction of other closing conditions...

...The completion of the T-Mobile and Sprint combination remains subject to remaining regulatory approvals and certain other customary closing conditions. T-Mobile and Sprint expect to receive final federal regulatory approval in Q3 and currently anticipate that the merger will be permitted to close in the second half of 2019...

Once it's official that it's going to be the New T-Mobile, I am sure the Microsoft authors will be making the appropriate updates to the docs. Personally, it's not just simple branding change, I would assume this will need to go through legal first. FYI @CoveMiner
Thank you for your feedback and for being part of the Microsoft Docs community!

@officedocsbot close


How to set drone timeout?

Feature Description

Screenshots
No response

Hi, Thank you for your question. This is a question best asked of Drone. They have a chatroom you can find here: https://developer.harness.io/community/


Windows build: (1) string pooling only for release (2) default stack …

…probes
Without these changes, Visual Studio's debugger cannot be used to step through code for "openssl s_server -WWW".  These changes fix that.  With string pooling, code is so optimized that execution stepping is confusing and ineffective.  The use of /Gs0 is contrary to Microsoft documentation, which states that nondefault stack probing should only be used if you know exactly why you need to do it.  Worse, /Gs0 appears to break Visual Studio's debugger.  (The workaround is to use the disassembler window and step into and through _chkstk.)

I guess the most important modification is /Gs0...  it's something that came along a long time ago, I found it as far back as the first import of SSLeay to become OpenSSL, back in December 1998...  so the reason for it, as far as we're concerned, is lost in history.
Re /GF, I'm amazed that it trips the debugger...  after all, it only means that any address to the exact same string copy will point at the exactly same place in memory instead of others...  but ok, if that trips the debugger, Microsoft must do some funnies that we simply don't fathom.

Perhaps there is agreement.  If so, the only remaining issue is the cla-check.  I've submitted my CLA twice, with no effect, even though I received a reply that seemed to indicate that it was accepted.

I am going to close this PR and submit a new one that eliminates /GF.  Perhaps the CLA will be effective now.

@ideafarm, your CLA contains a different address than the one you use in your commits.  The one used in your commits seems to be typical of commits done directly in github.  That's why the CLA checker fails.

@levitte Thanks.  I'll take care of that before opening a PR revised per this conversation.


[$500]  IOU - LHN preview for IOU report does not update to the latest amount when amount is changed

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.19-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to FAB > Request money > Manual
Create a manual request
Click on IOU preview twice to go to details view
Click Amount and change the amount
Return to IOU report

Expected Result:
The amount requested in LHN for the IOU report will update to the latest amount
Actual Result:
The amount requested in LHN for the IOU report does not update to the latest amount
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/329e449b-d912-4c5e-9709-d40d5b82fbe4
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e4739cbceab8e8b5
Upwork Job ID:<PHONE_NUMBER>306668544
Last Price Increase: 2023-12-29

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU - LHN preview for IOU report does not update to the latest amount when amount is changed
What is the root cause of that problem?
=> ReportPreview.js
{getDisplayAmount()}
getDisplayAmount didnt update the value
What changes do you think we should make in order to solve the problem?
=> ReportScreen.js
add return()=>{} inside useEffect() so when ReportScreen.js goBack or closed, it will trigger to re-render.
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU - LHN preview for IOU report does not update to the latest amount when amount is changed
What is the root cause of that problem?
=> ReportPreview.js
getDisplayAmount didnt return update changes
 const getDisplayAmount = () => {
        if (hasPendingWaypoints) {
            return props.translate('common.tbd');
        }
        if (totalDisplaySpend) {
            return CurrencyUtils.convertToDisplayString(totalDisplaySpend, props.iouReport.currency);
        }
        if (isScanning) {
            return props.translate('iou.receiptScanning');
        }
        if (hasOnlyDistanceRequests) {
            return props.translate('common.tbd');
        }

        // If iouReport is not available, get amount from the action message (Ex: "Domain20821's Workspace owes $33.00" or "paid ₫60" or "paid -₫60 elsewhere")
        let displayAmount = '';
        const actionMessage = lodashGet(props.action, ['message', 0, 'text'], '');
        const splits = actionMessage.split(' ');
        for (let i = 0; i < splits.length; i++) {
            if (/\d/.test(splits[i])) {
                displayAmount = splits[i];
            }
        }

        return displayAmount;
    };

What changes do you think we should make in order to solve the problem?
Switch return getDisplayAmount return into  return ReportUtils.getTransactionDetails(transactionPropTypes);
 const getDisplayAmount = () => {
       return ReportUtils.getTransactionDetails(transactionPropTypes);
       }

What alternative solutions did you explore? (Optional)

Waiting for proposals

Still waiting for proposals

No proposals yet - waiting to see if we get any

@laurenreidexpensify I can look into it, as I was working on LHN previews 🙂 But I also see that the total is not updated here (right upper corner)

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU - LHN preview for IOU report does not update to the latest amount when amount is changed
What is the root cause of that problem?
The API call UpdateMoneyRequestAmountAndCurrency does not change the total value in the report_XXXXXXX onyx data instead adds the diff to nonReimbursableTotal as seen in the screenshot below.  Similarly the originalMessage?.amount has the original values when the record was created

But the LHN and Title reads from originalMessage?.amount and report.total respectively which still has old value as seen in the below codes.
https://github.com/Expensify/App/blob/76e5b1cbd80e7dc55c52f47c5572b2984a7f7ed6/src/libs/ReportUtils.ts#L2141
https://github.com/Expensify/App/blob/76e5b1cbd80e7dc55c52f47c5572b2984a7f7ed6/src/libs/ReportUtils.ts#L1697
What changes do you think we should make in order to solve the problem?
To solve this we need to subtract nonReimbursableTotal from the total. Make the following changes to achieve the right result
Replace
https://github.com/Expensify/App/blob/76e5b1cbd80e7dc55c52f47c5572b2984a7f7ed6/src/libs/ReportUtils.ts#L1697
With
    let nonReimbursableSpend = moneyRequestReport?.nonReimbursableTotal ?? 0;
    const total = (moneyRequestReport?.total ?? 0) - nonReimbursableSpend;

Replace
https://github.com/Expensify/App/blob/76e5b1cbd80e7dc55c52f47c5572b2984a7f7ed6/src/libs/ReportUtils.ts#L2141-L2147
With (amountToDisplay becomes in formattedAmount in the return)
    // We only want to show the actor name in the preview if it's not the current user who took the action
    const requestorName = lastActorID && lastActorID !== currentUserAccountID ? getDisplayNameForParticipant(lastActorID, !isPreviewMessageForParentChatReport) : '';
    return `${requestorName ? `${requestorName}: ` : ''}${Localize.translateLocal('iou.requestedAmount', {formattedAmount: formattedAmount})}`;

And finally replace
https://github.com/Expensify/App/blob/76e5b1cbd80e7dc55c52f47c5572b2984a7f7ed6/src/libs/ReportUtils.ts#L1720
with
    let totalSpend = (moneyRequestReport.total ?? 0) - nonReimbursableSpend;

This will fix the discrepancy in the split that we see in the report split seen below

The above fix also works when we have multiple requests in one report
Result :
Single request
https://github.com/Expensify/App/assets/2232432/b794bd97-1f0f-457e-960f-5533e18be13d
Multiple requests
https://github.com/Expensify/App/assets/2232432/ff1920c1-a7ae-4291-8ecd-2841373f0208
What alternative solutions did you explore? (Optional)
NA

thanks @koko57 I think the proposal from @kameshwarnayak looks good
@grgia or @luacmartins would you like to take this one as CME since you have experience with this?

Yea, I can take this one. Thanks for the work so far @koko57 and @kameshwarnayak. I looked at the OP and I actually think this is a backend issue. We're not updating the report total on the new method. Here's the PR to fix that https://github.com/Expensify/Auth/pull/9629

Are the optimistic updates correct then?

@mountiny Yes. The optimistic updates are correct. The API response doesn't change the total value

Please re-state the problem that we are trying to solve in this issue.
IOU - LHN preview for IOU report does not update to the latest amount when amount is changed
What is the root cause of that problem?
libs/SidebarUtils.getOptionData sets alternateText incorrectly when last visible report action is an iou report action.
What changes do you think we should make in order to solve the problem?
In libs/SidebarUtils.getOptionData check to see if last visible report action is an iou. If so, set alternateText correctly, using the latest amount of the iou transaction. Otherwise, use existing logic to set alternateText.

this PR should fix this issue

Nice! @koko57 can you please add this issue to the list of fixed issues in https://github.com/Expensify/App/pull/35818?

Adding @situchan since they reviewed the linked PR

Let's get this retested once https://github.com/Expensify/App/pull/35125#issuecomment-1928450354 is on prod

Offer sent to @situchan in Upwork https://www.upwork.com/jobs/~0194ec363293ea15d0

@laurenreidexpensify Do you need to include this issue in weekly retest? It has weekly re-test label.

@luacmartins can you confirm if payment is bundled under a payment in https://github.com/Expensify/App/pull/35818 or needs additional payment - I got a little confused here. Thanks!

The PR linked 2 issues. I am not assigned in https://github.com/Expensify/App/issues/34411 so payment can be handled here.
cc: @mountiny

Agree that we can handle it here since @situchan is not assigned in https://github.com/Expensify/App/issues/34411

Sounds good, lets handle the $500 payment here

Payment Summary:
C+: $500 payment issued in Upwork

Originally, issue was in backend as per https://github.com/Expensify/App/issues/33733#issuecomment-1884945388.
We can skip regression test as reported by QA team.

Issue not reproducible during KI retests. (First week)


ghidra 11.0

Ghidra specific tests are passing on my machine

From the test output:
[info] io.joern.scanners.c.CredentialDropTests *** ABORTED ***
[info]   java.lang.InstantiationError: org.h2.mvstore.FileStore

Turns out they now also ship an h2 database, which collides with the one that overflowdb ships...
Context: since ghidra doesn't publish a jar, we replicate their build and publish it ourselves...


Missing setup.py or pyproject.toml

I get the following error when I follow the installation instructions:
$ pip install -e .

Obtaining file:///projects/vision/azhang42/experiments/VoCo-LLaMA
ERROR: file:///projects/vision/azhang42/experiments/VoCo-LLaMA does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.

Does this project have the same requirements as llava?

Hi,
Thanks for the suggestion, I've added a pyproject.toml within the repository and the dependencies we're using are exactly the same as LLaVA.

Thanks!


Compatibility

Is this compatible with Emacs 24.3?

@JimSleezus I haven't tried, I use Emacs 25.3.1 but Emacs 24.3 will probably work


feature requst: measure Time To First X (TTFX) without counting in Time To Load

The result of using PackageName; @time @eval some_custom_function(). It can not be part of BenchmarkGroup because we want to measure only the first run. A new process might be needed repeatedly to get a minimum.

Here is a shell script I am using for TTFX and TTL https://github.com/Krastanov/QuantumClifford.jl/blob/master/benchmark/run_ttfx_bechmarks.sh


re-architecuture core

There are some problems with the current core.
One is that a large number of cache files can be created. This is a disadvantage, as it is not possible to implement it like putting a cache in a specific location, such as a --cache-location flag.
The other one is to automatically obtain the name of the package. I think this is an exaggerated function. This is like a normal name option. I think it's enough to pass it with a flag, so I'd like to delete it. It reduces the dependence.
https://github.com/azu/file-cache/blob/b85c6f4bc3c1486b6f14e0167f270519459a74f3/packages/core/src/index.ts#L79-L85

The advantage of the cache directory is that it increases the flexibility of the cache. Because, even if you revert the changes you made, the cache will still be effective.
The disadvantage is that it is a little troublesome to continue the cache. For example, in order to save the cache in CI, you have to save the directory.
Also, there are many unnecessary files in the cache directory, so it is difficult to clean the cache.
However, most CIs have a cache file that manages the version, so it might not be a problem.

sindresorhus/find-cache-dir: Finds the common standard cache directory
node_modules/.cache is standard directory in Node.js libraries.


Add output formats to unmanaged-cluster list

This ties in the ability to output in yaml, json, and the default table
format.
Fixes: #2836

table output :heavy_check_mark:
josh@u:~$ ./tanzu-unmanaged-cluster-linux_amd64 ls -o table
  NAME   PROVIDER
  doggo  kind

json output :heavy_check_mark:
josh@u:~$ ./tanzu-unmanaged-cluster-linux_amd64 ls -o json
[
  {
    "name": "doggo",
    "provider": "kind"
  }
]

yaml output :heavy_check_mark:
josh@u:~$ ./tanzu-unmanaged-cluster-linux_amd64 ls -o yaml
- name: doggo
  provider: kind

Question: do you know what the weird character the json table is outputting at the end of the array?

I don't see that on my system. Is it maybe some artifact of your $PS1 setting?
It does not add a newline to the output. Maybe it should. I will add that just in case.


How to use LoadMessagesBefore and LoadMessagesAfter

I don't have problem when using LoadMessages function for getting last x messages.
but when I want to get the "after or before" messages by using message id, I'm having trouble.
i got message "server responded with status 404".
any one can help how to use "LoadMessagesBefore and LoadMessagesAfter"  function?
my code:
<EMAIL_ADDRESS>//phonenumber
messageId:="46F8BE5627xxxxx"
output, err :=wac.LoadMessagesBefore(jidDest, messageId, 10)
log.Println(output, err)

sorry, it works.
it turns out that the Message ID sent should be of the one who has the session (From me = true), otherwise 404 not found.
is that how it's used?


Thaumcraft wooden wand recipie

The wooden wand from Thumcraft is uncraftable because it requires you to already have one. It should be regular crafting.

The recipe has been changed to the same as below i the 0.7 update


addRegexToken

Androd 2.3.3 stock browser reports typeof Regex object(/../) as function and so its added to regexes array by addRegexToken and tried to be called as a function which fails and whole moment parsing from a valid iso string fails. I heard there are some browsers that also report typeof Regex object as function.(http://stackoverflow.com/questions/5054352/why-use-typeof-for-identifying-a-function)
So, this needs to be fixed.
function addRegexToken (token, regex, strictRegex) {
regexes[token] = typeof regex === 'function' ? regex : function (isStrict) {
return (isStrict && strictRegex) ? strictRegex : regex;
};
}

Not sure I'm following.  What precisely is the proposed change?

it must be checked if it is really a regex object.

And if it is really callable.

So, it should be like this;
function addRegexToken (token, regex, strictRegex) {
	regexes[token] = typeof regex === 'function' && !isRegExp(regex) ? regex : function (isStrict) {
		return (isStrict && strictRegex) ? strictRegex : regex;
	};
}

function isRegExp(input) {
	return Object.prototype.toString.call(input) === '[object RegExp]';
}

So,following calls could be distinguished from RegExps reported as function.
'''javascript
    addRegexToken('Do', function (isStrict, locale) {        return isStrict ? locale._ordinalParse : locale._ordinalParseLenient;    });
'''

Got it.  So this is primarily for compat with older browsers that incorrectly report regex as function instead of object.  Sounds reasonable to me, though I'd suggest a single function:
function isFunction(input) {
    return typeof input === 'function' && Object.prototype.toString.call(input) === '[object Function]';
}

@lockedscope can you please double check if the change mj1856 proposed fixes your problem? If yes, we'll merge it, if no -- we'll have to work around in some other way (different addRegexToken functions, a boolean flag, etc).

yeah the change proposed by mj1856 fixes the problem.


Mixin not available in data() method

Link to minimal reproduction
https://stackblitz.com/edit/vue-5mmhqa?file=src/App.vue
Steps to reproduce

Goto https://vue-5mmhqa.stackblitz.io
Check console. You should see MIXIN VAR undefined.

What is expected?
I expect the mixin to be available in data method.
What is actually happening?
The mixin is undefined.
System Info
No response
Any additional comments?
No response

https://github.com/vuejs/core/issues/4285


rafthttp: cleanup tests

squash the commits. lgtm

actually you should squash these commits into your last pr. I do not expect you merge the previous pr with these commented out code.

I forgot that in that pr. :(
:ship:


feat!: change the default data storage version to "stable" (e.g. v2.0)

Closes #2394
This PR changes a few remaining tests.  Also, by changing the default to v2 we exposed a few minor inconsistencies with v1 that we fixed.

When creating a fragment we reported progress before adding the filename to the fragment.  We now add the filename to the fragment before reporting progress.
Nested projection was broken (existing nested projection tests passed by luck).  This required some slight change to how we calculate projection.

BREAKING CHANGE: new datasets will no longer be readable by versions older than 0.16

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

Project coverage is 78.00%. Comparing base (2a9f8b5) to head (359a686).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

rust/lance-datagen/src/generator.rs
89.18%
0 Missing and 4 partials :warning:

rust/lance/src/dataset/transaction.rs
72.72%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2829      +/-   ##
==========================================
+ Coverage   77.94%   78.00%   +0.06%
==========================================
  Files         229      229
  Lines       70106    70164      +58
  Branches    70106    70164      +58
==========================================
+ Hits        54643    54732      +89
+ Misses      12385    12335      -50
- Partials     3078     3097      +19

Flag
Coverage Δ

unittests
78.00% <95.88%> (+0.06%)
:arrow_up:

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

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

I think we are breaking a certain backwards compatibility by changing the default, right?

Backwards compatibility, no, I don't think so.  We won't lose any capability to read older files.
Forwards compatibility: kind of.
This only applies the v2 default to new datasets.  So old readers will continue to be able to read old datasets but yes, readers beyond a certain point (probably 0.16 to be safe but in reality maybe even older) will not be able to read new datasets created by new versions.  I'll update some docs somewhere.

We should probably state the earliest version of Lance that can read these new V2 files. I think we are breaking a certain backwards compatibility by changing the default, right?

I've marked this a breaking change.  I verified that we have a "minimum lance version" in the format section (and updated it from 0.15 to 0.16).  I've added some basic "future compatibility" tests to make sure we can, in fact, read lance files written with the latest version in 0.16.


Build fails

Latest builds are failing when trying to create packages for the services.
https://github.com/vlizBE/vocabserver-app/actions/runs/9081578703/job/24955819032

@cedricdcc Can this one be closed? If so, please go ahead. https://github.com/vlizBE/vocabserver-app/issues/52 can probably go with this then.


Implement regression test suite for Python scripts

[ ] Create test oracle files (host in DataSync, see RiboViz/bbsrc-nsf#5)
[ ] trim_5p_mismatch.py (recall data/testdata_trim_5p_mismatch.sam and data/testdata_trim_5pos5neg.sam are for use with this script)
[ ] check_fasta_gff.py

See #11 for background.

Requirements for regression, and unit, tests now added to issues to refactor individual Python scripts:
#24, #25


Add a new advanced command mode to s_client

The s_client command mode just looks at the first letter on a line
and if it is one that maps to a "command" then that command is executed
and everything else on the line is ignored.
This means it is impossible to send anything over s_client that starts with
one of the "magic" letters unless you switch off command processing
entirely.
With the forthcoming quic support that will be added to s_client we expect
a significant new number of commands being added (for example to send fin
on a stream, or to list or swap between different streams, etc). This will
use up more "letters" and reduces the usability of s_client.
This PR adds a new "advanced" command processing mode. Commands are enclosed
in braces {}, and can appear anywhere in the text typed into s_client. A
brace can be escaped by sending a double {{. This gives much more
flexibility without reducing the usability of s_client. Commands can also
have "arguments" which also extends the scope of what they can be used for.

I have no idea what should be done about tests for this. We just have no reasonable way at the moment to test features like this.

Updated to address the feedback and also to address the CI failures. Please take another look.

Ping for reviews. This PR is needed for the s_client QUIC support in #20580.

Setting tests: deferred in the hope that at some point in future we will have s_client tests covering this.

I rebased this to resolve a trivial conflict in CHANGES.md. I assume all approvals still hold and the 24 hour timer does not need a reset.

+1, no reset

I have no idea what should be done about tests for this. We just have no reasonable way at the moment to test features like this.

Why not break out the processing from user_data_process into its own function? That way it could be tested and fuzzed like anything else, without relying on other infra.

Note: this introduces a new parser and the OTC mandated that all new parsers be subject to fuzz testing.

Note: this introduces a new parser and the OTC mandated that all new parsers be subject to fuzz testing.

Where did OTC mandate this? I don't recall it and don't see it in the testing policy.

It was agreed on a call.  I don't see it in the minutes unfortunately :(

And even if such a decision was made - it would have been in the context of parsers in the library - not in the context of an application like that - I would expect.
But a decision made without minutes and without a vote with vague recollections doesn't help - that should be a topic for the next OTC meeting. The importance of formally voting on these sorts of decisions which have an impact on what goes into the code is something I've raised a number of times.

Yeah mandating fuzzing for s_client advanced command mode feels like an overkill.


Show predicted Hits Only

currently when using the predicted hit feature it shows how much xp you earned as well, having them ungrouped does the same displaying both predicted hit + experience earned.
Is it possible to separate these completely and have only the predicted hit displayed without showing the experience earned?
Also if seperated completely an option to not have the hitsplat icon too?

Does this work for you? https://github.com/l2-/template-plugin/issues/39

@l2- thankyou very much, yes this is exactly what i needed.