[FLINK-34053][table-planner] Support state ttl hint for group aggregate

What is the purpose of the change
Currently, state ttl hint can work only on join node. This pr aims to support state ttl hint on group agg node.
Brief change log

support state ttl hint on group agg node
add IT cases and plan cases for this pr

Verifying this change
Some tests are added for it.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? Another pr will be introduced for it.

Hi, @LadyForest. After rebasing onto master, the test for GroupAggregateRestoreTest#AGG_WITH_STATE_TTL_HINT failed due to an unknown reason. It succeeded after the savepoint metadata was regenerated. Considering the feature freeze on January 30th, I will continue to monitor CI for any unstable tests after merging this feature and will promptly address any issues during the bugfix period.

Hi, @LadyForest. After rebasing onto master, the test for GroupAggregateRestoreTest#AGG_WITH_STATE_TTL_HINT failed due to an unknown reason. It succeeded after the savepoint metadata was regenerated. Considering the feature freeze on January 30th, I will continue to monitor CI for any unstable tests after merging this feature and will promptly address any issues during the bugfix period.

After debugging, I found that the _metadata has serialized the last accessed timestamp for keyed state when state retention is enabled. So I guess this test would fail in 4 days.


Added plugin Leaflet.AnimatedSearchBox to plugins.md

I added my plugin Leaflet.AnimatedSearchBox to the plugins.md.
It provides a collapsible search box.

Is there anything I can improve?


Use commit comment conventions to ensure they show up in the changelog

Prefix commits with fix:, enh:, maint:, etc. so that they are picked up by git-changelog.

we're generally doing this now.


Refactor redirects & add redirect for persistent storage

What's changed, or what was fixed?

Moves most web.dev redirects to /en/_redirects.yaml
Adds redirect for persistent storage doc

Whoops!
There were 1 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2016/06/persistent-storage.md - Unable to read file, was it deleted?


HIVE-26221: Add histogram-based column statistics

What changes were proposed in this pull request?

See the Jira ticket.
Why are the changes needed?

See the Jira ticket.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile="compute_kll_sketch.q,sketches_kll_test.q,stats_histogram.q,stats_histogram2.q" -Dtest.output.overwrite -pl itests/qtest -Pitests

mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile="sketches_rewrite_rank_partition_by.q,sketches_rewrite_rank.q,sketches_rewrite_percentile_disc.q,sketches_rewrite_ntile_partition_by.q,sketches_rewrite_ntile.q,sketches_rewrite_cume_dist_partition_by.q,sketches_rewrite_cume_dist.q,sketches_materialized_view_rank.q,sketches_materialized_view_percentile_disc.q,sketches_materialized_view_ntile.q,sketches_materialized_view_cume_dist.q" -Dtest.output.overwrite -pl itests/qtest -Pitests

mvn test -Dtest=LongColumnStatsAggregatorTest -pl standalone-metastore/metastore-server
mvn test -Dtest.groups=org.apache.hadoop.hive.metastore.annotation.MetastoreCheckinTest -Dtest=TestCachedStoreUpdateUsingEvents.java -pl itests/hive-unit -Pitests
mvn test -Dtest.groups=org.apache.hadoop.hive.metastore.annotation.MetastoreCheckinTest -Dtest=TestPartitionStat -pl standalone-metastore/metastore-server
mvn test -Dtest.groups=org.apache.hadoop.hive.metastore.annotation.MetastoreCheckinTest -Dtest=TestCachedStore -pl standalone-metastore/metastore-server
mvn test -Dtest=TestObjectStore -pl standalone-metastore/metastore-server

Please keep the PR open

Thanks @asolimando, leave some minor comments, and the changes look good to me

Another question I don't see here is how we generate the histogram statistics? by issuing an "analyze table" command?

Another question I don't see here is how we generate the histogram statistics? by issuing an "analyze table" command?

That was hard to figure out for me too at first. Statistics computation happens via an aggregate query, where different UDAFs are used to compute the different statistics.
ColumnStatsSemanticAnalyzer.java#L308-L325 generates the SELECT statement for the stats.
It's then calling ColumnStatsSemanticAnalyzer.java#L327 which has an enum with the different statistics, what we did was to add a new one for histograms and generated the code accordingly (see ColumnStatsSemanticAnalyzer.java#L355-L357).
Finally, the UDAF part is generated here: ColumnStatsSemanticAnalyzer.java#L494-L519.


Issue with transparent

With transaparent = true
-- my config in lazyvim
return {
  {
    "fcancelinha/nordern.nvim",
    priority = 1000,
    lazy = false,
    opts = {
      transparent = true,
    },
  },
  {
    "LazyVim/LazyVim",
    opts = {
      colorscheme = "nordern",
    },
  },
}

When focusing on a pane, other panes will be dimmed.

Hey! I'm currently investigating the issue, I'll patch the colorscheme once it's resolved.

@dangtran47 You can update to the latest version, the issued should be fixed, please let me know if there are other issues that you found that need fixing.


The butterfly emoji 🦋 is dead

Living butterflies don't hold their forewings ahead of their head, and instead keep them further back - forming a shape more like a trapezoid. It's only when they're pinned up for display that the wings are set in the positions shown in the emoji. So the butterfly emoji depicts a dead butterfly. RIP.

Some of these look more like the emoji representations to my eye... https://www.youtube.com/watch?v=aBfJtTm_XD4
I suppose it depends on the type of butterfly?

Hm, I don't really see it. The butterflies in that video don't appear to assume the posture shown in the emoji.

The pale yellow one, for instance. Anything with a tall wing that extends "ahead of their head" as you put it due to the sheer size of the wing. The profile resembles the emoji for those. And I think it's more pronounced in flight than at rest FWIW.

I see it, but it's a subtle resemblance at best. shrug

I suppose not having them as extremely forward would lean more toward "some butterfly with a tall wing maybe in mid-flight" and less "wing is pinned up to a board". I think it's a matter of degrees, not black/white.
BTW the butterfly emoji from all the vendors seem pretty similar in this regard: https://emojipedia.org/butterfly/
All have tall wings that are pretty far "forward" relative to the head. Google's is possibly the least-forward, considering the latest editions form each vendor.
(P.S. I am listed as "Contributor" to this repository, but that's just because I submitted some small patches to things. I'm not part of the Google team and have no official role at this repository. I'm just another person commenting on this.)

BTW the butterfly emoji from all the vendors seem pretty similar in this regard

Maybe, but this is the one that has a bug tracker I can submit issues to :wink: and also the one I actually use.

Sitting butterfly — well, OK. Butterflies in flight actually more resemble image on emoji.


InputDecorator errorText, helperText not padded correctly per the Material Design spec

Currently error and helper text are displayed immediately below the TextField's divider. They should be padded on top by 8:  https://material.io/guidelines/components/text-fields.html#text-fields-layout

From the Gallery text fields demo:

Although this is an easy fix to make it will likely break existing layouts. Some apps may apps may prefer to keep the current more compact layout.

This issue came up when we added the helper text support https://github.com/flutter/flutter/pull/10852

This was fixed in https://github.com/flutter/flutter/pull/13734


Make ui available at /ui instead of /ui/dist

Hey Jon,
Cool idea, I was looking into this myself.
So naturally I was happy to see someone had already started with it.
As a token of my appreciation here's a small fix.
The ui should be available at /ui so I've changed around how you extract it.
Let me know if there's anything you don't like about the change.
cheers,
David

Thank you very much David! I'm glad we were able to save each other some effort. =)


Can't build the project

What did you do?
I run
make build
What did you expect to see?
the roject is buikding
What did you see instead? Under which circumstances?
/home/cas12/prometheus-main/web/ui/node_modules/rollup/dist/bin/rollup:1342
valueString ??= String(value);
^
SyntaxError: Unexpected token '?'
at wrapSafe (internal/modules/cjs/loader.js:915:16)
at Module._compile (internal/modules/cjs/loader.js:963:27)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
at internal/main/run_main_module.js:17:47
npm ERR! Lifecycle script build failed with error:
npm ERR! Error: command failed
npm ERR!   in workspace<EMAIL_ADDRESS>npm ERR!   at location: /home/cas12/prometheus-main/web/ui/module/lezer-promql
make: *** [Makefile:64: ui-build] Error 1
Time: 0h:00m:23s
System information
Linux 6.8.0-45-generic x86_64
Prometheus version
prometheus, version 2.43.0 (branch: HEAD, revision: edfc3bcd025dd6fe296c167a14a216cab1e552ee)
  build user:       root@8a0ee342e522
  build date:       20230321-12:56:07
  go version:       go1.19.7
  platform:         linux/amd64
  tags:             netgo,builtinassets

Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

which version of Nodejs are you using. Make sure you use v16 or greater

Contributions to have such deps versions constraints checks done in the Makefile e.g. are welcome.

Contributions to have such deps versions constraints checks done in the Makefile e.g. are welcome.

#15106 I added a makefile target to check for correct versions of Nodejs, Go and NPM before building the app.


Stream Elements commands playing 'sorry'

Please provide us with the following information:

This issue is for a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] feature request
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

start up the chat client and run a known stream elements command such as !sfx or !so and the system will play the 'sorry' sound byte because it is not supported by the twitch client.

Any log messages given by the failure

Expected/desired behavior

If a known stream elements command is run, we want to ignore that and not play the sound byte.

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

@clarkio Could probably resolve this by adding the known Stream Elements commands to the effects.json file?


Transitive dependencies missing in SBT compared to Maven

[X] I've read the CONTRIBUTING guidelines

steps
Create SBT project with
libraryDependencies ++= Seq(
  "org.optaplanner" % "optaplanner-benchmark" % "7.20.0.Final"
)

and a Maven project with
    <dependencies>
        <dependency>
            <groupId>org.optaplanner</groupId>
            <artifactId>optaplanner-benchmark</artifactId>
            <version>7.20.0.Final</version>
        </dependency>
    </dependencies>

problem
The Maven project correctly identifies the transitive dependency on org.drools:drools-compiler:7.20.0-Final via org.optaplanner:optaplanner-core:7.20.0.Final, whereas the SBT project does not identify this transitive dependency.
expectation
Maven and SBT projects identify the same set of transitive dependencies.
notes
This Maven configuration includes a number of parent POMs which may be causing the issue with SBT. The files involved are:

http://central.maven.org/maven2/org/optaplanner/optaplanner-benchmark/7.20.0.Final/optaplanner-benchmark-7.20.0.Final.pom (direct dependency containing dependency on org.optaplanner:optaplanner-core)
http://central.maven.org/maven2/org/optaplanner/optaplanner-core/7.20.0.Final/optaplanner-core-7.20.0.Final.pom (transitive dependency that has dependency on org.drools:drools-compiler
http://central.maven.org/maven2/org/kie/kie-parent/7.20.0.Final/kie-parent-7.20.0.Final.pom (grandparent POM via http://central.maven.org/maven2/org/optaplanner/optaplanner/7.20.0.Final/optaplanner-7.20.0.Final.pom containing version for dependencies - specifically 7.20.0.Final)

sbt version: 1.2.8

@Woodz Thanks for the report.
I tried it with sbt 1.3.0-RC1, which uses Coursier instead of Apache Ivy, and it seems to identify drools-core-7.20.0.Final.jar.
I also needed to add "org.freemarker" % "freemarker" % "2.3.28" because it failed on org.freemarker:freemarker:2.3.26.jbossorg-1.

@Woodz Thanks for the report.
I tried it with sbt 1.3.0-RC1, which uses Coursier instead of Apache Ivy, and it seems to identify drools-core-7.20.0.Final.jar.
I also needed to add "org.freemarker" % "freemarker" % "2.3.28" because it failed on org.freemarker:freemarker:2.3.26.jbossorg-1.

@eed3si9n the issue is identifying drools-compiler, not drools-core - drools-core is picked up in both SBT and Maven.


ssl/bestrq recipe?

Hello I'm  Seung Woo
First of all, Thank you for your good work "ssl/bestrq". I came to train this model and compare my implementation
For these reason.. Is there any example using the model in librispeech or aishell ?
Thanks for in advance.
Seung Woo.

anticipating

training is unstable and always crash now 。 I am working on it

you can try new : https://github.com/wenet-e2e/wenet/pull/1824, recepie will in later pr

@yuseungwoo @rookie0607 你可以尝试新的：#1824， recepie 将在以后的 pr
cool！

I hope the recipe will come soon.

@yuseungwoo Hi, what's your final reproduction result? Do you have any good reproduction suggestions? My results on librispeech data are completely unable to do the results in the paper.


⚠ Bofrost.at has degraded performance

In f18f96a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2590 ms

Resolved: Bofrost.at performance has improved in 45d923a after 13 minutes.


Simplify traceID creation in DataSource and PooledConnection.

This will also make sure the traceID is set before used in toString().
The loggername handling seems to be a bit divergent- in some cases a literal package is used in others not. I am not sure if this is intentional, so I left it alone.
Sorry for the removal of trailing whitespaces, but I figured it it generally helpful. If you want me to isolate the changes, please let me know.
Signed-off-by: Bernd Eckenfels<EMAIL_ADDRESS>

Hard to see what the reason for those build failures is, but it looks not related. It is maybe a good idea to remove flaky timeout tests from the pull request review builds. Can this be retried?

hi @ecki thank you for your contribution, the team will take a look


KeyError : "normal"

Hello, when i use the PTv3 to train my dataset（ no normal vector ）, there is a  (key Error : "normal") occurs in the traning process. Do you know how to revise the code to train a dataset with no normal vector! Look forward to your reply,  thank you very much!    My dataset is s3dis format.

I got the similar Error when I ran a test on a PTv2 model trained on s3dis.
[2024-08-03 20:14:16,750 INFO test.py line 119 4866] >>>>>>>>>>>>>>>> Start Evaluation >>>>>>>>>>>>>>>>
Traceback (most recent call last):
  File "tools/test.py", line 38, in <module>
    main()
  File "tools/test.py", line 27, in main
    launch(
  File "/mnt/c/workspace/Pointcept/pointcept/engines/launch.py", line 89, in launch
    main_func(*cfg)
  File "tools/test.py", line 20, in main_worker
    tester.test()
  File "/mnt/c/workspace/Pointcept/pointcept/engines/test.py", line 161, in test
    for idx, data_dict in enumerate(self.test_loader):
  File "/home/owl/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 681, in __next__
    data = self._next_data()
  File "/home/owl/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1376, in _next_data
    return self._process_data(data)
  File "/home/owl/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1402, in _process_data
    data.reraise()
  File "/home/owl/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/_utils.py", line 461, in reraise
    raise exception
KeyError: Caught KeyError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/home/owl/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 302, in _worker_loop
    data = fetcher.fetch(index)
  File "/home/owl/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 49, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/owl/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 49, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/mnt/c/workspace/Pointcept/pointcept/datasets/defaults.py", line 175, in __getitem__
    return self.prepare_test_data(idx)
  File "/mnt/c/workspace/Pointcept/pointcept/datasets/defaults.py", line 142, in prepare_test_data
    data_dict = self.get_data(idx)
  File "/mnt/c/workspace/Pointcept/pointcept/datasets/defaults.py", line 120, in get_data
    np.ones(data_dict["coord"].shape[0], dtype=np.int32) * -1
KeyError: 'coord'

Maybe this issue helps: https://github.com/Pointcept/Pointcept/issues/196

@blueeaglex Did you use the official config we released?

@blueeaglex  Sorry， mine should be different from the question. I didn't use the right dataset. I've solved the problem after reading #147.


Update stable version in repo from 0.0.12 to 1.0.1

Think about update stable version to 1.0.1
Version: stable=0.0.12 (885 days old) => latest=1.0.1 (15 days old)
Installs: stable=218 (91.21%), latest=17 (7.11%), total=239
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 1.0.1 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.0.1 but the current stable version is already 1.0.1.
So this issue should be closed.
@mcm1957 for evidence


JSON-API: 'included' and 'linkage'

Further changes to bring JSONAPI formatting more in line with 1.0 format

Changed linked to included
Change format of links to include 'linkages'

And thank you for the pull request <3

My pleasure :) Yaks is extremely helpful.

@danelowe could you provide a full example for the issue with rel you describe above? This is what I get
$y = Yaks.new {}

class Customer ; include Attribs.new(:books) ; end
class Book ; include Attribs.new(:title) ; end

class CustomerMapper < Yaks::Mapper
  has_many :books
end

class BookMapper < Yaks::Mapper
  attributes :title
end

puts $y.map(Customer.new(books: [Book.new(title: "Freedom")])).pp

output:
Yaks::Resource.new(
  type: "customer",
  subresources: [
    Yaks::CollectionResource.new(
      type: "book",
      rels: ["rel:books"],
      members: [
        Yaks::Resource.new(type: "book", attributes: {:title=>"Freedom"})
      ]
    )
  ]
)

I'm happy to merge this, but it will need a rebase. Do you want to do that or shall I do it?

@plexus Apologies, it turns out the rels issue was simply that I had rel_template in my Yaks config. I originally used this to get started with Yaks: https://github.com/sinetris/yaks_grape_mongoid_example/blob/master/config/initializers/yaks.rb
I think I've done the rebase correctly.

it turns out the rels issue was simply that I had rel_template in my Yaks config

That still sounds like a bug, having a rel_template in your settings should simply mean that it uses that template when no explicit rel is given on an association. I'll see if I can reproduce. A self contained example would still be very much appreciated.
I merged the PR. Sorry it's taken awhile, I'm a bit backed up on my open source stuff.

@plexus Sorry for the slow reply, and even more sorry for pointing in the wrong direction.
The config I had that was causing the behaviour was this line:
https://github.com/sinetris/yaks_grape_mongoid_example/blob/master/config/initializers/yaks.rb#L4
rel_template 'rels:{dest}'
I just searched the yaks repository and found this:
https://github.com/plexus/yaks/blob/83da834d614ed837b0a41d5706b3d0f03fac9f4a/yaks/lib/yaks/breaking_changes.rb#L73
When specifying a rel_template, now a single {rel} placeholder is expected instead of {src} and {dest}.
I also just added the line back into my config and ran the debugger.
On a subresource, I actually got
rels=["rels:"]
Which is entirely consistent with what I would expect when using a placeholder in the template that no longer exists.
I just checked the date on that example repository. More than 6 months ago. Oops.

@plexus Also as per original comment, the template config I had was only causing issues where the rel wasn't explicitly given in the mapper.

Ok, that seems consistent with what I would expect :) thanks for looking it up!


can't copy text from chat with mouse

If I try to highlight text from the chat pane in order to copy and paste, it just drags the pane around.

We could add a dragging handler so only clicking on it (and not in any point of the frame) would trigger the dragging event. That's trivial from the implementation POV. We just need a good idea on how it should look like. Something like the titlebar of a window? Something that appears on mouse over? Some kind of collapsed tab?... Mockups are welcome.

After discussing alternatives with @mdeniz, @ChrisBr and @cwh42, it was decided that the best approach would be having two modes (with a button to toggle), one with all the dragging and resizing blocked and another one to play around with the layout.

@mdeniz self assigned to me because it's related to #74! JFYI


Can you guide me to use elliptical roi based on measurement data

Prerequisites

[x] Which version are you using? (Is it latest?)
[x] Are you reporting to the correct repository?
[x] Did you search existing issues? (Were any related?)

Description
I am new to cornerstone and have started it from 7 days back, so pardon me if my question is very basic or wrong.
I have loaded and displayed my Image in onInit and then I have a button on click of which I want to draw an elliptical tool of certain measurement. The code which I have written for it is :
const element = document.querySelector('.image-canvas');
const EllipticalRoiTool = cornerstoneTools.EllipticalRoiTool;

const measurementData = {
  visible: true,
  active: true,
  color: 'white',
  invalidated: true,
  handles: {
    start: {
      x: 300,
      y: 200,
      highlight: true,
      active: false
    },
    end: {
      x:<PHONE_NUMBER>285714,
      y:<PHONE_NUMBER>2857143,
      highlight: true,
      active: true
    },
    initialRotation: 0,
    textBox: {
      active: false,
      hasMoved: false,
      movesIndependently: false,
      drawnIndependently: true,
      allowedOutsideImage: true,
      hasBoundingBox: true
    }
  }
}

cornerstoneTools.addToolState(element, 'EllipticalRoi', measurementData);
cornerstoneTools.setToolPassive('EllipticalRoi');

Its adding elliptical tool when I move mouse over element
What else should I write now, to display this tool on my displayed Image without any mouse movement?

I found the solution by using updateImage method


When I execute mint nft, I report an error rpc error

HI,I use this method
await wallet.signAndExecuteTransactionBlock({
    transactionBlock: tx
  });

I report an error rpc error

when i use example page https://wallet-kit-demo.vercel.app/
it reports the same error，Is this a problem with my usage?

@songenen Hey, could you tell me which wallet you are using? Also this could be due to the network fluctuation, have you tried again after a period?

@songenen also, please check your @mysten/sui.js is up to date (v0.32.1)

@songenen Could you choose the "Network" panel in the chrome inspector, and find out the network request for your tx execution, see what's the response from RPC?

@bruceeewong

{
    "jsonrpc": "2.0",
    "result": {
        "data": [
            {
                "data": {
                    "objectId": "0x03b3648a6191075356349c3dad2e087217ea257b337f9b1057e994f498bec9cd",
                    "version": "153420",
                    "digest": "J6jZu3jGNhADjZ2u5LKhCzQthWyHBMRC16kKtRW2eC5d",
                    "type": "0x2::coin::Coin<0x2::sui::SUI>",
                    "owner": {
                        "AddressOwner": "0x10974b7c796893b50a74544e798bdbfa853b323796ff5ae731d7cd4acee9f206"
                    },
                    "display": {
                        "data": null,
                        "error": null
                    },
                    "content": {
                        "dataType": "moveObject",
                        "type": "0x2::coin::Coin<0x2::sui::SUI>",
                        "hasPublicTransfer": true,
                        "fields": {
                            "balance": "3970228204",
                            "id": {
                                "id": "0x03b3648a6191075356349c3dad2e087217ea257b337f9b1057e994f498bec9cd"
                            }
                        }
                    }
                }
            },
            {
                "data": {
                    "objectId": "0x63054f75259d467c4c5ebb62b29b1ee757f26f49451022e8e0b0991252502e17",
                    "version": "128551",
                    "digest": "9dgtP4sASThkniWr73E31t3dQKfZ8u6K3pNyk5rSLC9R",
                    "type": "0x3::staking_pool::StakedSui",
                    "owner": {
                        "AddressOwner": "0x10974b7c796893b50a74544e798bdbfa853b323796ff5ae731d7cd4acee9f206"
                    },
                    "display": {
                        "data": null,
                        "error": null
                    },
                    "content": {
                        "dataType": "moveObject",
                        "type": "0x3::staking_pool::StakedSui",
                        "hasPublicTransfer": false,
                        "fields": {
                            "id": {
                                "id": "0x63054f75259d467c4c5ebb62b29b1ee757f26f49451022e8e0b0991252502e17"
                            },
                            "pool_id": "0x84a29ef4b0104d3b7e6a72d1128088fed03a6d4d9ceb4abf8423aa3b0d67a004",
                            "principal": "1000000000",
                            "stake_activation_epoch": "2"
                        }
                    }
                }
            },
            {
                "data": {
                    "objectId": "0x74e555ea2054f10f007b182c02db75c318e957f3e206ebe83faf4795c38155d3",
                    "version": "153419",
                    "digest": "9WogeJeVcrM133ZiZ9wU1dPx67GF9jtzc2PdrPiDSrSd",
                    "type": "0x255ca79ed18e67ab321e7ec4689ff907fa8067a7892160ed51d03c4721ca757b::registrar::RegistrationNFT",
                    "owner": {
                        "AddressOwner": "0x10974b7c796893b50a74544e798bdbfa853b323796ff5ae731d7cd4acee9f206"
                    },
                    "display": {
                        "data": null,
                        "error": null
                    },
                    "content": {
                        "dataType": "moveObject",
                        "type": "0x255ca79ed18e67ab321e7ec4689ff907fa8067a7892160ed51d03c4721ca757b::registrar::RegistrationNFT",
                        "hasPublicTransfer": true,
                        "fields": {
                            "id": {
                                "id": "0x74e555ea2054f10f007b182c02db75c318e957f3e206ebe83faf4795c38155d3"
                            },
                            "name": "asdasd.sui",
                            "url": "ipfs://QmQcCq2igL2kbRC9Mzam1CRMMXU2PWQaw8LpARqJphXycP"
                        }
                    }
                }
            }
        ],
        "nextCursor": "0x74e555ea2054f10f007b182c02db75c318e957f3e206ebe83faf4795c38155d3",
        "hasNextPage": false
    },
    "id": "0bd9ccfd-5759-422f-9871-231d7ee72ea5"
}

I had the same issue with @suiet/wallet-kit 0.2.5. Upgrade to @suiet/wallet-kit 0.2.6 works for me.

I had the same issue with @suiet/wallet-kit 0.2.5. Upgrade to @suiet/wallet-kit 0.2.6 works for me.

I'm also using 0.2.6, and it's fine now. It seems to be a problem with network fluctuations

I assume it is because of the network fluctuation. I've tried with https://wallet-kit-demo.vercel.app/ using Sui Wallet as well as Suiet wallet today and it works. Please try again at your convenience.

I tried it, and it is like this, it seems to be a problem with network fluctuations, thank you


Ignore conflicting keep attributes, unless asked not to

This replicated the XST behavior of simply ignoring nets with conflicting KEEP attributes, as described in https://github.com/YosysHQ/yosys/issues/1733#issuecomment-595923804.
I've investigated options for writing BUF cells for those nets directly in the EDIF file, but because of the way EDIF works and the way the current back-end is written this turns out to be not feasible without a major rewrite of the EDIF back-end.
An alternative would be a different pass that adds the buffers before even calling the EDIF back-end.

This code fails in situations where it should work:
module top(input wire i, output wire o);

(* keep *)
wire a;

assign a = ~i;
assign o = ~a;

endmodule

yosys t.v -o t.edif:
Warning: Ignoring conflicting 'keep' property on net a. Use -keep to generate the extra net nevertheless.
IMO we should do a first pass through all wires and, when a keep wire is noticed, make the pass use this wire's bits as the "representative" (overriding whatever sigmap suggests), unless doing so would conflict with another keep wire (which should cause a warning)

IMO we should do a first pass through all wires and, when a keep wire is noticed, make the pass use this wire's bits as the "representative" (overriding whatever sigmap suggests), unless doing so would conflict with another keep wire (which should cause a warning)

Done.


list exercise error

According to https://github.com/scala-exercises/exercises-scalatutorial/blob/master/src/main/scala/scalatutorial/sections/StandardLibrary.scala#L72-L97
val nums = Nil.::(4).::(3).::(2).::(1)

nums match {
  // Lists of `Int` that starts with `1` and then `2`
  case 1 :: 2 :: xs => …
  // Lists of length 1
  case x :: Nil => …
  // Same as `x :: Nil`
  case List(x) => …
  // The empty list, same as `Nil`
  case List() =>
  // A list that contains as only element another list that starts with `2`
  case List(2 :: xs) => …
}

the last case of case List(2 :: xs) gives error because the inferred type is List[Int] but the case signature is expecting List[List[Int]]. Please correct me if I am wrong. If I am not then how should it be re-written?

Hi, it’s true that the example doesn’t compile. The idea was more to show various kinds of patterns.

For someone new to Scala like me this could be a bit confusing. I think this can be refactored to show two lists or remove the last case altogether.

I agree with @Ibrahim-Islam that any code should be copy-paste ready to try out and play with it. Maybe you want to add some useful return values, @Ibrahim-Islam?

Yes and while at it maybe do a PR to make this right for future learners :)
But would like your opinion on how it should be corrected.

@Ibrahim-Islam make a proposal :) Ping me if you like to discuss it :)


Expose metrics for better monitoring

What
Currently, pvc-autoresizer does not expose any metrics.
In some situations such as #24, the process exits with an error.
This is not monitoring-friendly, so we want to add another method to report such events.
Metrics is the best for monitoring.
How

Expose the below metrics as a prometheus metrics.

pvcautoresizer_kubernetes_client_fail_total (namespace=pvcautoresizer, subsystem=kubernetes_client, name=fail_total)

counter that indicates how many API requests to kube-api server are failed.

pvcautoresizer_metrics_client_fail_total (namespace=pvcautoresizer, subsystem=metrics_client, name=fail_total)

counter that indicates how many API requests to metrics server(e.g. prometheus) are failed.

pvcautoresizer_success_loop_total (namespace=pvcautoresizer, subsystem=, name=success_loop_total)

counter that indicates how many volume expansion processing loops succeed.

pvcautoresizer_failed_loop_total (namespace=pvcautoresizer, subsystem=, name=failed_loop_total)

counter that indicates how many volume expansion processing loops are failed.

pvcautoresizer_loop_seconds_total (namespace=pvcautoresizer, subsystem=, name=loop_seconds_total)

counter that indicates the sum of seconds spent on volume expansion processing loops

Do not exit when some error occurs

Checklist

[ ] Finish implementation of the issue
[ ] Test all functions
[ ] Have enough logs to trace activities
[ ] Notify developers of necessary actions

We found that reconcile method resizes multiple pvcs, so pvcautoresizer_success_loop_total and pvcautoresizer_failed_loop_total are not suitable. They are now pvcautoresizer_success_resize_total and pvcautoresizer_failed_resize_total.


Revert "Fixes listview shrinkwrap and hidden semantics node gets upda…

causes internal test failure
https://github.com/flutter/flutter/issues/84719
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

FWIW, I think the internal screen diffs that "failed" because of this actually look more reasonable now, but I'm not 100% sure - we should talk to the owners about it.

(reverting while we do that is of course a good idea :)

I wasn't sure about the ui to be sure, some of them seems to render more items than the origin. I have to look into the customers' code to be sure


Fix broken link to GitHub Importer in SVN Migration cheat sheet

This PR includes a fix for the broken link to the GitHub Importer as identified in issue #385.
Because the proposed URL fix (import.github.com) only redirects successfully for logged in users, this PR is WIP, pending further discussion.
Once the ideal link is determined, I can go ahead and update this PR.

@mattlebel I'm going to carry on conversation here instead of #385!
I love the fix, but I had no idea this didnt work for signed out users. What if we changed the text around this in general to say "read more about how to import your project into GitHub by looking at the full documentation"?

@brntbeer sounds good! Updated to:
"Read more about how to import your project into GitHub by looking at the full GitHub Importer documentation."

@mattlebel thank you so much for this~


gpu_voxels: 0.8.3-2 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository gpu_voxels to 0.8.3-2:

upstream repository: https://github.com/BV-OpenSource/gpu-voxels.git
release repository: https://github.com/BV-OpenSource/gpu-voxels-release.git
distro file: melodic/distribution.yaml
bloom version: 0.10.7
previous version for package: null

It looks like multiple licenses are being used but only one is listed in the package.xml. You can have more than one <license> tag. Should there be more than one in there?

@wjwwood You meant the package.xml in the release repository, right? In case positive, I think it's fixed.

The guidelines don't answer it explicitly, but if I'm interpreting section 2.ii correctly, the source repo might matter more than the release repo, e.g. "The LICENSE file only needs to be in the sources; a new release after adding a LICENSE file is not required." and "one or more ROS packages (meaning they have a package.xml in the source repository)."
Unless someone comments with an actual answer, it doesn't hurt to add it to the source repo.

I don't think you need a package.xml in the source repository, there are lots of projects where this isn't an option. It would be better if there was one upstream, but it's not required as far as I know.

@BV-OpenSource it looks like you still have just one license entry:
https://github.com/BV-OpenSource/gpu-voxels-release/blob/405e999cc230d03db054633f1f59481cb1590dc1/package.xml#L12-L15
If you all you did was update the file in master, then you need to run bloom again afterwards.

It will create 0.8.3-3 and unless you cancel it, it will open a new pull request too, which is fine.

New pull-request #31358 opened.


Fix LSP sample E2E test

Fixes  #149

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: PanAeon sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Looks good, thanks!


ENH: use Fraction for spin values

If the Spin class would have the types of its fields magnitude & projection to be of
the Fraction-type, it would simplyfy the validation of their values (multiples of 1/2,
projection <= magnitude, etc.)

@grayson-helmholz
https://docs.astral.sh/ruff/linter/#error-suppression


fix: fix, simplify and tidy exiting from piggyback events in OMG.State

Found when #855
Overview
When doing ALD compatibility, I managed to introduce a bug, where input piggybacks would crash the child chain (a bit silently, b/c #1141, hence our integration tests didn't pick this up)
Changes

fix bug - now child chain will completetly ignore input piggybacking events (like it did before)
added an explicit OMG.State.Core test for this
added a more explicit ChildChain integration test for this to be extra sure

Testing
mix test

Coverage remained the same at ?% when pulling 98bfbba3878b08360ff8194dc2597cfac7f56415 on pdobacz/fix_input_piggyback_processing into 70e9a6803b53009db296f25bd5ed590eb26ddd9c on master.

CC @jarindr


KCL docs: param list is hard to read

The current KCL docs list each function's parameters. The parameter list looks like:

In my opinion, this is hard to read. I think it's hard to immediately tell where the parameter name ends and the parameter type begins. Each one takes up an inconsistent visual height too.
I suggest we actually display a table, with one row for each parameter. Columns would be Name, Type and Required.

Name
Type
Required

num_repetitions
u32
Required

transform_function
Function
Required

extrude_group_set
ExtrudeGroupSet
Required

I love this idea. The tables for the API docs can serve as a reference if you like https://zoo.dev/docs/api/file/get-cad-file-mass?lang=typescript#parameters-material_density

i think @franknoirot they'd be markdown tables here and in the docs, then will convert when passed to website

for whoever wants this is generated in docs.rs, its just traverses the jsonschema and yes it could be A LOT better, we could generate files per types like SketchGroup as well and cross link, anyways, all i care about is we don't hand write we can get really nice things buy just making the generation better we have all the data.


Vague definition for AGEDBH

The SCRPPLE manual includes a vague definition for AGEDBH ("AgeDBH is a general relationship between Age and DBH for each species"). Could you please provide a more detailed definition for AGEDBH or update the manual? The manual also neglects to mention the units for maximum bark thickness. I assume this is in cm, but would like some clarification. Thank you for your time!
-Colin Mast
M.S. Student in Melissa Lucash's TEEL lab

Hi Colin,
Hope you are well. I will work to make this clearer in the next manual version. In the mean time the relationship between Age and DBH is defined as the following

here rho is the maximum bark thickness and alpha is the AGEDBH.
They were given the common name because in this structural equation rho *BETA will be the the maximum obtainable bark-thickness and alpha will determine the rate at which AGE increases DBH.
There are no inherent units for Bark thickness or maximum bark thickness, as you the user define the relationship of either variableto probability of mortality and between AGE and DBH.
If you would like an example of how to calculate these values please see the following :
https://github.com/LANDIS-II-Foundation/Project-Southern-Appalachians/tree/master/Research_Projects/Delayed Mortality/Building_the_Model#setting-the-bark-thickness-age-relationship


BLPOP Command timed out after 1 minute(s) causes output loss

Bug Report
We have BlockingQueue Implementation on top of RedisList, the consumer thread of queue invokes BLPOP command to keep listening queue until it receives any event. Whenever shared connections are enabled, Blocking Redis Commands times out, whereas when shared connections are disabled, lettuce honor the blocking nature of Redis command and having Apache Pool Configured has no impact on this, plus disable-shared connections add more performance overhead.
Now, in order to balance between performance overhead vs shared connection, we had a logic to invoke queue, on each timed out until it receives an event.  This behavior is leading to output loss.
Observing Lettuce Code especially "CommandHandler"."addToStack", stacks add redisCommand if "ChannelPromise" is void and I think "CommandHandler"."isWriteable" is checking whether redisCommand is Done or not, in case of BLPOP Command timeout, may be redisCommand it not Done;
When event-data are published to queue using "RPUSH", then lettuce assume those BLPOP which are timed out are still active and providing event-data with an error saying "Command timed out after 1 minute(s)" and also reduces stack size. In order to make the queue work back again, multiple event-data should be pushed clear all stack with timeout redis command.
We think the problem is still with Lettuce not cleaning up the stack of redis command, upon redis command time-out and in meantime it will discard all the event-data/bytes.

Logs

2019-11-07--03-30-36-237 [T: U: D: Tx:/ URI: M:] [lettuce-nioEventLoop-20-3] DEBUG io.lettuce.core.protocol.CommandHandler - [channel=0x109a692e, /##.##.###.###:32808 -> xxxxxx-xxxxxxxxxxxxxx-0001-001.xxxxxx-xxxxxxxxxxxxxx.xxxxxx.use1.cache.amazonaws.com/##.##.###.###:6379, chid=0x9] Received: 1328 bytes, 2 commands in the stack
2019-11-07--03-30-36-237 [T: U: D: Tx:/ URI: M:] [lettuce-nioEventLoop-20-3] TRACE io.lettuce.core.protocol.CommandHandler - [channel=0x109a692e, /##.##.###.###:32808 -> xxxxxx-xxxxxxxxxxxxxx-0001-001.xxxxxx-xxxxxxxxxxxxxx.xxxxxx.use1.cache.amazonaws.com/##.##.###.###:6379, chid=0x9] Buffer: *2^M
$33^M
��^@^Et^@^ZQUEUE_xxxxIFICATE_REQUESTS^M
$1275^M
��^@^Esr^@>xxx.xxxxxxxxxx.xxxxxxx.service.pki.xxxxxxxxxxxxxxxxxxQueueItem^@^@^@^@^@^@^@^A^B^@^CZ^@^QstopWorkerThreadsL^@^LmdmChannelIdt^@^PLjava/util/UUID;L^@"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxst^@^PLjava/util/List;xp^<EMAIL_ADDRESS>2019-11-07--03-30-36-237 [T: U: D: Tx:/ URI: M:] [lettuce-nioEventLoop-20-3] DEBUG io.lettuce.core.protocol.CommandHandler - [channel=0x109a692e, /##.##.###.###:32808 -> xxxxxx-xxxxxxxxxxxxxx-0001-001.xxxxxx-xxxxxxxxxxxxxx.xxxxxx.use1.cache.amazonaws.com/##.##.###.###:6379, chid=0x9] Stack contains: 2 commands
2019-11-07--03-30-36-237 [T: U: D: Tx:/ URI: M:] [lettuce-nioEventLoop-20-3] DEBUG io.lettuce.core.protocol.RedisStateMachine - Decode ClusterCommand [command=AsyncCommand [type=BLPOP, output=KeyValueOutput [output=null, error='Command timed out after 1 minute(s)'], commandType=io.lettuce.core.protocol.Command], redirections=0, maxRedirections=5]
2019-11-07--03-30-36-237 [T: U: D: Tx:/ URI: M:] [lettuce-nioEventLoop-20-3] DEBUG io.lettuce.core.protocol.RedisStateMachine - Decoded ClusterCommand [command=AsyncCommand [type=BLPOP, output=KeyValueOutput [output=KeyValue[[B@3348d498, [B@2534987f], error='Command timed out after 1 minute(s)'], commandType=io.lettuce.core.protocol.Command], redirections=0, maxRedirections=5], empty stack: true

Environment

Lettuce version(s): 5.1.8.RELEASE
Redis version: 5.0.5

Whenever shared connections are enabled, Blocking Redis Commands times out, whereas when shared connections are disabled, lettuce honor the blocking nature of Redis command and having Apache Pool Configured has no impact on this, plus disable-shared connections add more performance overhead.

Care to elaborate on shared connections? Lettuce itself isn't providing any means of shared connections.
I'm not sure I follow. There are two things that play into the general behavior: Timeout and protocol synchronization.
For timeouts, a command that does not receive within the given timeout a response is timed out in the synchronous, Future or Subscription frontend, depending on which API you're using. The command itself gets canceled to properly indicate the timeout.
The timeout counter starts at the time where the command is dispatched and written to the transport. In a typical arrangement, dispatch happens immediately when calling a Redis command method. In reactive arrangements, dispatch may happen at a later time.
Lettuce does not await completion of earlier commands before dispatching a new command as Lettuce uses pipelining. Commands that are written to the transport are added to the protocol stack awaiting a response. If a command in the stack gets canceled, then we have no means to communicate that fact to Redis. To retain protocol synchronization, we must wait until Redis sends a response. We cannot simply remove the command from the stack because such an action would desynchronize the protocol state.
If Redis never responds but the connection gets broken, then what happens is that non-canceled commands get retried as we already know that the caller of the command has received a signal that the command is not valid anymore.
Does this help?

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

Same Problem and solved by setting the default timeout of client to zero.
As the BLPOP command has a timeout argument, using that argument on connection, instead of default timeout will reduce confusion.

Same Problem and solved by setting the default timeout of client to zero.
As the BLPOP command has a timeout argument, using that argument on connection, instead of default timeout will be less confusing.

The problem with a timeout that is shared between the client and server is that the client side will always lead to a timeout exception. This is, because the timeout starts counting on invoking the command. That is before encoding and before the command has arrived at the server (imagine a multi-second latency).

So the problem is that the connection is opening with a timeout, before the command be known by
lettuce?

Tip: set timeoutOptions on your Redis(Cluster)Client.setOptions with a timeoutSource that uses a longer timeout for blocking commands (blpop, etc):
ClusterClientOptions.builder().timeoutOptions(
    TimeoutOptions.builder()
        .timeoutSource(new TimeoutOptions.TimeoutSource() {... })
        .build()
)


Fix #6263 Cannot scale VM with custom offering

Description
This PR fixes #6263.
The issue is that an Exception is thrown when scaling a customized offering, changing the CPU/Memory but keeping the same disk offering.
This PR proposes a fix for this issue by skipping the volume "upgrade" process in such situations, avoiding any operations and also the thrown exception.
Considering that the disk offering is the same, there is no need to run all actions of the respective method.
The skipped actions are, in summary:

run DB updates matching the "new offering";
publish EVENT_VOLUME_RESIZE;
if volume migration is required (and allowed), find a suitable pool to migrate volume in case the current pool does not match the new offering needs;
return a "Volume" object containing all the changes.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Bug Severity

[ ] BLOCKER
[X] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Scale (via UI or API) a VM with the custom offering, keeping the same offering but updating the Memory and/or CPU.
After skipping the validation, all worked out fine.

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3174

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian Build Failed (tid-3915)

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3916)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31089 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6267-t3916-kvm-centos7.zip
Smoke tests completed. 93 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File


List of references is duplicated for pyright

Description
Using :lua vim.lsp.buf.references() in a python file with pyright connected produces 2 entries for each reference in the quickfix list. For example:
pyfrontal.py|338 col 9| moments = cv.moments(contour[0])
pyfrontal.py|338 col 9| moments = cv.moments(contour[0])
pyfrontal.py|346 col 23| mu_prime_20 = moments["mu20"] / px_area
pyfrontal.py|346 col 23| mu_prime_20 = moments["mu20"] / px_area
pyfrontal.py|347 col 23| mu_prime_02 = moments["mu02"] / px_area
pyfrontal.py|347 col 23| mu_prime_02 = moments["mu02"] / px_area
pyfrontal.py|348 col 23| mu_prime_11 = moments["mu11"] / px_area
pyfrontal.py|348 col 23| mu_prime_11 = moments["mu11"] / px_area
pyfrontal.py|379 col 20| xbar = int(moments["m10"] / px_area)
pyfrontal.py|379 col 20| xbar = int(moments["m10"] / px_area)

Only having this issue with pyright. Other language servers are fine.
Neovim version
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "
/nix/store/5ksh5vbvdgy0vs2d4p32qlmxg3vbmfld-neovim-unwrapped-0.9.1/share/nvim
"

Nvim-lspconfig version
0d29cad8de3b2c654315203fc1fe12fde722a18a
Operating system and version
Arch Linux
Affected language servers
pyright
Steps to reproduce

nvim file.py
Place cursor onto an appropriate symbol
:lua vim.lsp.buf.references()
QFL is opened with the duplicate references

Actual behavior
QuickFix List is opened with two entries per reference
Expected behavior
QuickFix List is opened with a single entry per reference
Minimal config
return {
  {
    'neovim/nvim-lspconfig',

    config = function()

      local lspconfig = require('lspconfig')

      vim.api.nvim_create_autocmd('LspAttach', {
        group = vim.api.nvim_create_augroup('UserLspConfig', {}),
        callback = function(ev)
          local opts_desc = function(d)
            return { buffer = ev.buf, desc = d }
          end

          vim.keymap.set('n', 'gr', vim.lsp.buf.references, opts_desc('Go to references'))
        end,
      })

      lspconfig.pyright.setup {}

      vim.lsp.set_log_level('debug')
    end
  }
}

LSP log
See lines 57-64 of the log for the references request... you can search for each line number in the result and find the duplicates.
https://gist.github.com/carschandler/cadbbdfdf709eb20ee37e535d3e6c69f

So I have discovered that this is only occurring with one file in one of my projects... I’ll try to diagnose why it's only happening with that one.

I figured out that this is only happening for symbols that are defined inside of functions... happening in both root-directory mode and single-file mode. A minimal example would be:
def main():
    msg_dup = "hello"
    print(msg_dup)

msg_normal = "bye"
print(msg_normal)

Invoking vim.lsp.buf.references() on msg_normal yields:

while doing the same on msg_dup yields:

I also confirmed that this is not happening with Pyright when using Doom Emacs or VSCode

Thanks for your report. The configs in this repo are unsupported and provided only as a starting point. We depend on users (like you) to troubleshoot issues with their specific LSP setups and send improvements.
If you found a bug in the core Nvim  vim.lsp module (not part of this repo), the best way to get it fixed is to report to Nvim (not nvim-lspconfig) with:

steps to reproduce it without the particular LSP server and other factors particular to your environment
(optional, but very helpful): by adding a failing test case to lsp_spec.lua, which has code to setup a fake LSP server to simulate various scenarios

HI @carschandler, I also met this issue. Do you have any solution?

@stvhuang glad to see it's not just me! I don't have a solution yet. I've been meaning to do some more troubleshooting to determine what exactly is responsible for this, but haven't had the time yet. I’ll post an update here when I do.

@carschandler Got it! I will also try to troubleshoot it.

I'm also experiencing this issue. Will try to investigate when I get the chance.

@stvhuang @rickboks check the issue I opened on the pyright repo here https://github.com/microsoft/pyright/issues/6666. It seems that this is a pyright issue and that VS Code just happens to be de-duplicating them on its own!


Semi-fix input box sizing issue

As it stands now, typing more than a few characters in your input box causes the size to be set improperly. (see this example using your demo, where about 3 characters are cut off: https://gyazo.com/1d88753ea702fae2a00f33f0041a82fd). The previous way it was being set was incorrect, and caused the size attr to not reflect that actual size of the token. I have fixed this, but also added 5 to the normal value. This is because certain characters are wider than others, and can cause the same issue. The increased size accounts for that fact.
It's still not perfect, but it's much better.

This repository has chosen to disable pull requests.


Bug in Grid Tab Group.

Describe the bug
Classic Homepage is open on clicking New Tab in Tab Group enabled even you set Modern Homepage in settings.
To Reproduce

Set option Modern Homepage in Homepage settings.
Enable Grid(support tab group) in Tab switcher.
Open any website.
Long press on any link and click on open in new tab in group.
Tab group shows at bottom of browser.
Click on + icon to open new tab in group.
New tab open in Classic Homepage instead of Modern Homepage.

Expected behavior
If Homepage set to Modern Homepage then this setting also apply in Group tabs.
Screenshots
https://user-images.githubusercontent.com/122884037/226170854-a30e6871-be6e-45e9-a85e-e5f6d334b4b6.mp4
Smartphone (please complete the following information):

Device: [Micromax IN1b]
Android Version : [A11 ]
Kiwi Browser Version [ 110.0.5563.78 ]

@kiwibrowser

This also happens when you close all tabs, fully close browser (remove from recents) and then reopen it — it will start with classic homepage.
Version 107.0.5304.75 here.


Feature/v1.5.0 release

New features:

Option to disable the usage of CLI library, using a custom built-in argument parser.
Encapsulate agent's instantation on AgentInstance class.
Fix bug when launching PseudoTerminal Agent.
Option to disable the compilation of the default provided MicroXRCEAgent executable.
Snap release and deployment to Snapcraft store.

Build status:

Linux
Windows

Changelist:
#Add

New optional basic CLI interface (removes dependency with CLI library)
Added option to only build the library with no Micro XRCE-DDS Agent app
Added new snap package releases

#Fix

Build status:

Linux
Windows


CI tests failing on Master Branch

Description of the bug:
Hi Team,
One of the checks is failing on Master branch, which is a blocker for release this week. Kindly have a look.
buildkite/clion-plugin - Build #22429 failed (5 minutes, 54 seconds)
Which category does this issue belong to?
Intellij
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
Click on https://github.com/bazelbuild/intellij
Click on CI checks, failures can be seen.
Which Intellij IDE are you using? Please provide the specific version.
No response
What programming languages and tools are you using? Please provide specific versions.
No response
What Bazel plugin version are you using?
No response
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?

CI tests are passing now, hence closing the issue.


Fix h3_request to work with delays

Commit Message: fix h3_request to work with delays
Additional Description: It turns out the aiohttp functions named with the prefix send_ only sometimes send, and would be more accurately prefixed queue_, or more accurately still queue_or_send_. This fixes the tool to always send.
Risk Level: None, test-tool only.
Testing: Tested manually with the call that wasn't working right before, and it does now.
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/retest


How can I get selected asset on tapping collection view cell?

Right now I can get selected assets by tapping done button or setting autoCloseOnSingleSelect property. How can I get selected assets on tapping the collectionview cell?

Make a subclass of DKImagePickerControllerBaseUIDelegate, then override this function:
open func imagePickerController(_ imagePickerController: DKImagePickerController, didSelectAssets: [DKAsset])

I tried to override the method but found this issue.

I update pod to 4.0.0-beta and made subclass of DKImagePickerControllerBaseUIDelegate. I found the same issue.

Could you post the source code of DKImagePickerControllerBaseUIDelegate.swift here? You can find it in your Pod project :)

//
//  DKImagePickerControllerBaseUIDelegate.swift
//  DKImagePickerController
//
//  Created by ZhangAo on 16/3/7.
//  Copyright © 2016年 ZhangAo. All rights reserved.
//
import UIKit
@objc
public protocol DKImagePickerControllerUIDelegate {
/**
 The picker calls -prepareLayout once at its first layout as the first message to the UIDelegate instance.
 */
func prepareLayout(_ imagePickerController: DKImagePickerController, vc: UIViewController)

/**
 The layout is to provide information about the position and visual state of items in the collection view.
 */
func layoutForImagePickerController(_ imagePickerController: DKImagePickerController) -> UICollectionViewLayout.Type

/**
 Called when the user needs to show the cancel button.
 */
func imagePickerController(_ imagePickerController: DKImagePickerController, showsCancelButtonForVC vc: UIViewController)

/**
 Called when the user needs to hide the cancel button.
 */
func imagePickerController(_ imagePickerController: DKImagePickerController, hidesCancelButtonForVC vc: UIViewController)

/**
 Called after the user changes the selection.
 */
func imagePickerController(_ imagePickerController: DKImagePickerController, didSelectAssets: [DKAsset])

/**
 Called after the user changes the selection.
 */
func imagePickerController(_ imagePickerController: DKImagePickerController, didDeselectAssets: [DKAsset])

/**
 Called when the count of the selectedAssets did reach `maxSelectableCount`.
 */
func imagePickerControllerDidReachMaxLimit(_ imagePickerController: DKImagePickerController)

/**
 Accessory view below content. default is nil.
 */
func imagePickerControllerFooterView(_ imagePickerController: DKImagePickerController) -> UIView?

/**
 Set the color of the background of the collection view.
 */
func imagePickerControllerCollectionViewBackgroundColor() -> UIColor

func imagePickerControllerCollectionImageCell() -> DKAssetGroupDetailBaseCell.Type

func imagePickerControllerCollectionCameraCell() -> DKAssetGroupDetailBaseCell.Type

func imagePickerControllerCollectionVideoCell() -> DKAssetGroupDetailBaseCell.Type

}
////////////////////////////////////////////////////////////////////////////////////////////////////
@objc
open class DKImagePickerControllerBaseUIDelegate: NSObject, DKImagePickerControllerUIDelegate {
open weak var imagePickerController: DKImagePickerController!

open var doneButton: UIButton?

open func createDoneButtonIfNeeded() -> UIButton {
    if self.doneButton == nil {
        let button = UIButton(type: UIButtonType.custom)
        button.setTitleColor(UINavigationBar.appearance().tintColor ?? self.imagePickerController.navigationBar.tintColor, for: .normal)
        button.addTarget(self.imagePickerController, action: #selector(DKImagePickerController.done), for: UIControlEvents.touchUpInside)
        self.doneButton = button
        self.updateDoneButtonTitle(button)
    }

	return self.doneButton!
}

open func updateDoneButtonTitle(_ button: UIButton) {
    if self.imagePickerController.selectedAssetIdentifiers.count > 0 {

        let formatter = NumberFormatter()
        formatter.numberStyle = .decimal
        formatter.locale = Locale(identifier: Locale.current.identifier)

        let formattedSelectableCount = formatter.string(from: NSNumber(value: self.imagePickerController.selectedAssetIdentifiers.count))

        button.setTitle(String(format: DKImagePickerControllerResource.localizedStringWithKey("picker.select.title"),
                               formattedSelectableCount ?? self.imagePickerController.selectedAssetIdentifiers.count), for: .normal)
    } else {
        button.setTitle(DKImagePickerControllerResource.localizedStringWithKey("picker.select.done.title"), for: .normal)
    }

    button.sizeToFit()

    if #available(iOS 11.0, *) { // Handle iOS 11 BarButtonItems bug
        if button.constraints.count == 0 {
            button.widthAnchor.constraint(equalToConstant: button.bounds.width).isActive = true
            button.heightAnchor.constraint(equalToConstant: button.bounds.height).isActive = true
        } else {
            for constraint in button.constraints {
                if constraint.firstAttribute == .width {
                    constraint.constant = button.bounds.width
                } else if constraint.firstAttribute == .height {
                    constraint.constant = button.bounds.height
                }
            }
        }
    }
}

// Delegate methods...

open func prepareLayout(_ imagePickerController: DKImagePickerController, vc: UIViewController) {
	vc.navigationItem.rightBarButtonItem = UIBarButtonItem(customView: self.createDoneButtonIfNeeded())
}

open func layoutForImagePickerController(_ imagePickerController: DKImagePickerController) -> UICollectionViewLayout.Type {
	return DKAssetGroupGridLayout.self
}

open func imagePickerController(_ imagePickerController: DKImagePickerController,
                                  showsCancelButtonForVC vc: UIViewController) {
	vc.navigationItem.leftBarButtonItem = UIBarButtonItem(barButtonSystemItem: .cancel,
	                                                      target: imagePickerController,
	                                                      action: #selector(imagePickerController.dismiss as () -> Void))
}

open func imagePickerController(_ imagePickerController: DKImagePickerController,
                                  hidesCancelButtonForVC vc: UIViewController) {
	vc.navigationItem.leftBarButtonItem = nil
}

open func imagePickerController(_ imagePickerController: DKImagePickerController, didSelectAssets: [DKAsset]) {
    self.updateDoneButtonTitle(self.createDoneButtonIfNeeded())
}

open func imagePickerController(_ imagePickerController: DKImagePickerController, didDeselectAssets: [DKAsset]) {
    self.updateDoneButtonTitle(self.createDoneButtonIfNeeded())
}

open func imagePickerControllerDidReachMaxLimit(_ imagePickerController: DKImagePickerController) {
    let formatter = NumberFormatter()
    formatter.numberStyle = .decimal
    formatter.locale = Locale(identifier: Locale.current.identifier)

    let formattedMaxSelectableCount = formatter.string(from: NSNumber(value: imagePickerController.maxSelectableCount))

    let alert = UIAlertController(title: DKImagePickerControllerResource.localizedStringWithKey("picker.select.maxLimitReached.error.title"), message: nil, preferredStyle: .alert)

    alert.message = String(format: DKImagePickerControllerResource.localizedStringWithKey("picker.select.maxLimitReached.error.message"), formattedMaxSelectableCount ?? imagePickerController.maxSelectableCount)

    alert.addAction(UIAlertAction(title: DKImagePickerControllerResource.localizedStringWithKey("picker.alert.ok"), style: .cancel) { _ in })

    imagePickerController.present(alert, animated: true)
}

open func imagePickerControllerFooterView(_ imagePickerController: DKImagePickerController) -> UIView? {
	return nil
}

open func imagePickerControllerCollectionViewBackgroundColor() -> UIColor {
    return UIColor.white
}

open func imagePickerControllerCollectionImageCell() -> DKAssetGroupDetailBaseCell.Type {
    return DKAssetGroupDetailImageCell.self
}

open func imagePickerControllerCollectionCameraCell() -> DKAssetGroupDetailBaseCell.Type {
    return DKAssetGroupDetailCameraCell.self
}

open func imagePickerControllerCollectionVideoCell() -> DKAssetGroupDetailBaseCell.Type {
    return DKAssetGroupDetailVideoCell.self
}

}

The file looks correct. Could you set up a small project that would reproduce this issue?

I'm glad to hear that you have resolved the issue! 😄

@alice05  how can u get selected asset on tapping collectionview cell? please help me i am stuck on this.

@Uk44 There is a public property called "UIDelegate" of DKImagePickerController. You can subclass as above and assign to it. And then you can get the selected assets.

@alice05
class MyProfileViewController: UIViewController,DKImagePickerControllerUIDelegate
{
func imagePickerController(_ imagePickerController: DKImagePickerController, didSelectAssets: [DKAsset]) {
print("hello")
}
func imagePickerController(_ imagePickerController: DKImagePickerController, didDeselectAssets: [DKAsset]) {
print("hello")
}
}
i was implemented like this but after tapped colletion view cell this above method not called.

You can subclass like this:
class CustomClass:DKImagePickerControllerUIDelegate
{
    func imagePickerController(_ imagePickerController: DKImagePickerController, didSelectAssets: [DKAsset]) {
        print("hello")
    }

    func imagePickerController(_ imagePickerController: DKImagePickerController, didDeselectAssets: [DKAsset]) {
        print("hello")
    }
}

You can use in touch event like this:
let controller = DKImagePickerController()
controller.UIDelegate = CustomClass()
self.present(controller, animated: true, completion: nil)

@alice05  i will try with above mention code but i want to access only didselect and didDeselect method but above mention code give me error  like this.

@Uk44 I have used your code assuming those methods are present inside DKImagePickerControllerUIDelegate class. If not try to subclass DKImagePickerControllerDefaultUIDelegate class. You can debug now to find the solution.

hey @alice05 , thanks a lot for help  i replace DKImagePickerControllerDefaultUIDelegate this and it's working thanks a lot man for your support. 👍 :)


Lombok.config

After running a build I am often seeing the following in my Git status:
jonesn:egeria/ (master*) $ git status                                                                                [9:51:48]
On branch master
Your branch is ahead of 'origin/master' by 73 commits.
  (use "git push" to publish your local commits)

Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	deleted:    lombok.config

no changes added to commit (use "git add" and/or "git commit -a")

Needs investigation. Probably needs adding as a .gitignore

Probably relates to 41079079ec0e4e55a139be8fb89aa18fea3c852f , though have not yet identified what is causing this config file to be deleted

@planetf1, unfortunately I am not aware of the reason. I am back to building with maven (don't know if maven/gradle are related to this but nothing else comes to mind) and the file looks unaltered. At a first look on Google nothing shows.

Thanks. Going to close for now - will reopen if it reoccurs


Add EOF line

Pull Request
Thank you for contributing to @dvcs/gitignore and https://www.gitignore.io.
New or update
Select the appropriate check box for this pull request. This helps when merging to ensure there are no conflicts with other templates or misunderstandings of how thee template list works.
New

[ ] Template - New .gitignore template
[ ] Composition - Template made from smaller templates
[ ] Inheritance - Template similar to an existing template
[ ] Patch - Template extending functionality of existing template

Update

[x] Template - Update existing .gitignore template

Details
replace this section with links and/or info about the proposed request

My Heroku update mechanism is broken and I'll be fixing it in 2 weeks so these wont appear on the site until then.

No problem, thank you for letting me know.


News detail page - Include author name and photo, links to other posts by author

Definitely need this on the page, to credit the person who submitted it.  But it will also help to make the page look nicer.
Another element that will make the page not feel like a wasteland would be to have links to, say, up to the latest 5 posts by the same author.  If there are no other posts by the author, the latest 5 by any author.

@4down what context do you want in the list of 5 other posts? (title, date, etc)

@gregnewman I think just reusing the list display that's used on the listing page would be fine.  Is that what you mean?

Reuse the list display for the five most recent exactly as it is in the listing with icon, avatar and block?

On Nov 9, 2023, at 11:37 AM, Spencer Strickland @.***> wrote:
@gregnewman https://github.com/gregnewman I think just reusing the list display that's used on the listing page would be fine. Is that what you mean?
—
Reply to this email directly, view it on GitHub https://github.com/cppalliance/temp-site/issues/794#issuecomment-1804171697, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAAP2RINZUXZ4EU3A3BIVLYDUBENAVCNFSM6AAAAAA63GLCVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBUGE3TCNRZG4.
You are receiving this because you were mentioned.

@gregnewman Let's hold off on adding this to the page for now.  I know we want to get away from the Newer/Older buttons there, but the page looks a lot nicer with the changes you've made - so we'll table this unless any specific feedback comes up.
You've got the author/image showing on the page now so going to mark this as completed.


Use the Lua API for keymaps if available

This allows to use a function as the right hand side of the keymap

https://neovim.io/doc/user/api.html#nvim_set_keymap()

{opts}  Optional parameters map: keys are
|:map-arguments|, values are booleans (default
false). Accepts all |:map-arguments| as keys
excluding || but including |noremap| and
"desc". Unknown key is an error. "desc" can be
used to give a description to the mapping. When
called from Lua, also accepts a "callback" key
that takes a Lua function to call when the mapping
is executed.


Allow newer versions of terminal-table

terminal-table released a 2.0 and 3.0 which other gems starting to depend on. How do you feel about relaxing the requirement on it again here?

Thank you ❤


software version number

How can I get the version of my current installation? Something like
mono ThermoRawFileParser.exe --version

or a VERSION file would be helpful.

Hi Laurent, I've added a --version argument to the 1.1.7-SNAPSHOT release (-v is already taken for the verbose logging).


Failed to import the custom network

I think that I followed the instructions: create a new directory 'newNet' and place it inside the 'niftynet/extensions/' with 2 files: init.py and newNet.py. But still I got this error:

ValueError: Could not import network: Incorrect module name "newNet"; expected "module.object".

More info: the path for the model in the config file is: models/newNet
Any help would be appreciated!!

Hi @YilinLiu97 for a concrete example of loading /home/dev/mymodules/mynetwork.py (assuming the network class NewNet(...): defined in this .py file)

make the folder on the $PYTHONPATH:
export PYTHONPATH=/home/dev/:$PYTHONPATH

make mymodules a python path:
create an empty __init__.py file in /home/dev/mymodules/

run NiftyNet with a configure file, in which we specify the new network in the [NETWORK] section:
name = mymodules.mynetwork.NewNet

NiftyNet will decode the name string as a python command:
from mymodules.mynetwork import NewNet


Single sign on feature does not work as expected for Developer portal

Description
The single sign-on is not functioning as expected for the Developer Portal. Despite signing in to the Publisher Portal, when attempting to access the Developer Portal in the same browser session, it does not retain the signed-in state. As a result, users are required to click the sign-in button again. However, users are not prompted to re-enter their credentials.
In addition this works properly with admin portal.

Steps to Reproduce

Sign in to publisher portal
Using the same browser session open the developer portal

Affected Component
APIM
Version
4.3.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

This is the expected behaviour since the Dev Portal is an open portal that anyone can access without logging in.


Update editor doc links to 3.5 branch

So that DOCS_URL and other editor links point to the current version.

Thanks!


Update to 1.21 please!

... and thanks for making this mod!

Working on incapacitated 1.21 this week. Next week will (probably) be the horde manager.

1.21 port complete ahead of schedule. Turns out it's a lot easier when I figure out what I'm doing.

Thank you!
@CartoonishVillain I did run into an error trying to run a json horde using the example file, the other 2 commands work fine too....

Can I get a log on the issue? The example file ran fine in testing, so I'm curious to see what went wrong

Getting unknown or incomplete command with not much else:

json file is the same one from the sample, uploaded in the minecraft root alongside mods folder

Can I get a log on the issue? The example file ran fine in live environment testing, so I'm curious to see what went wrong

Apologies, it's just a user error on my end.  Maybe I just misread the docs but either way when you try the command with no args it fails saying incomplete or not found command which i supppose isn't wrong but itsn't helpful either... thanks for the quick responses!

It was late last night I felt pretty embarassed trying it again today and it was simple, and I'm a programmer too 🤦
Either way I think it might be a common mistake but I'd recommend just a small update to the readme example to actually have a swam as an arg because lazy ppl like myself tend to copy and paste... and maybe a better error message when no args are available, I unfortunately work in another stack entirely :)
Thanks for the amazing mod!


[Bug]: Invalid type relation for open records with only optional fields

Description
For open records that only have optional fields invalid assignments are allowed, probably due to invalid type relations
Steps to Reproduce
Following code shouldn't compile
import ballerina/io;
type R1 record {
    int foo?;
};

type R2 record {
    string bar?;
};

function r1() returns R1 {
    return { foo: 1, "bar": 10 };
}

public function main() {
    R2 r = r1(); // This should not work, (R1 is not subtype of R2)
    boolean test = r.bar is string?;
    io:println(test); // this result false
}

Affected Version(s)
2201.6.0
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Runtime checks should also be fixed.
import ballerina/io;

type R1 record {
    int x?;
};

type R2 record {
    string y?;
};

public function main() {
    R1 r1 = {x: 1, "y": 10};
    R2 r2 = r1; // should be an error
    // r2["x"] = "not an int"; // write panics

    io:println(r1 is R2); // true, should evaluate to false
}

This happens when the field is optional in the target type and the source type is an open record.

OpenAI tools need to be fixe d- https://github.com/ballerina-platform/openapi-tools/issues/1440


⚠ Bofrost.de has degraded performance

In 8d45c8c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1449 ms

Resolved: Bofrost.de performance has improved in 5f32bcc.


[Parse] Factor out parsing access path when parsing ImportDecl

If we want to support module-qualified member access sometime (I'm preparing a pitch actually),
we better reuse some of the code that parses import declarations for consistency.
This also replaces std::vector with llvm::SmallVector since most of the time access paths in import declarations consist of only one identifier.
cc @jrose-apple

The code looks fine to me, I personally don't think it's worth taking this until we have something else that's using it. More importantly to you, however, is probably the fact that Swift doesn't model Clang submodules as nested in any way; import Foo.Bar says "take the declarations in Foo.Bar and make them visible in Foo". (And there are no other kinds of submodules today.) So this refactoring may not be interesting after all.

cc also @brentdax

Can one of the admins verify this patch?

@broadwaylamb Do you still want this up? It seems like it'd be useful as part of the implementation of an eventual proposal, but for now I'd like to close it if it's not currently serving a purpose.

@CodaFi I think you can close this, yes.


Documentation

PR for adding documentation to the repo.
Remaining to do before we merge:

[ ] Add section on new badge graphics
[ ] Check inline jekyll documentation

cc @DylanMoriarty

Would recommend re-submitting after #125 is accepted, as many of the minor commits (non ai file additions) will conflict with the localization rebuild.

@abenrob :+1:

Agreed, closing this for now and will reopen.


Download failed when try examplss.

I try many different examples from ./zml/examples but all got failed.  Below is one log of them
==========Error Log==========
(rocm6) amd@AIG-PM:~/Repo/zml/examples$ bazel run -c opt //llama:OpenLLaMA-3B  --@zml//runtimes:rocm=true   -- --prompt="Once upon a time,"
Starting local Bazel server and connecting to it...
WARNING: For repository 'rules_zig', the root module requires module version<EMAIL_ADDRESS>but got<EMAIL_ADDRESS>in the resolved dependency graph.
WARNING: Download from https://mirror.zml.ai/zig/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz failed: class java.net.SocketException Connection reset
WARNING: Download from https://ziglang.org/builds/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz failed: class java.io.FileNotFoundException GET returned 404 Not Found
INFO: Repository rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux instantiated at:
  <builtin>: in <toplevel>
Repository rule zig_repository defined at:
  /home/amd/.cache/bazel/_bazel_amd/e9f8a4534bc94824063a91fd4855a228/external/rules_zig~/zig/private/repo/zig_repository.bzl:125:33: in <toplevel>
ERROR: An error occurred during the fetch of repository 'rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux':
   Traceback (most recent call last):
        File "/home/amd/.cache/bazel/_bazel_amd/e9f8a4534bc94824063a91fd4855a228/external/rules_zig~/zig/private/repo/zig_repository.bzl", line 119, column 51, in _zig_repository_impl
                download = repository_ctx.download_and_extract(**download_args)
Error in download_and_extract: java.io.IOException: Error downloading [https://mirror.zml.ai/zig/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz, https://ziglang.org/builds/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz] to /home/amd/.cache/bazel/_bazel_amd/e9f8a4534bc94824063a91fd4855a228/external/rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux/temp13491381570566576292/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz: GET returned 404 Not Found
ERROR: no such package '@@rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux//': java.io.IOException: Error downloading [https://mirror.zml.ai/zig/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz, https://ziglang.org/builds/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz] to /home/amd/.cache/bazel/_bazel_amd/e9f8a4534bc94824063a91fd4855a228/external/rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux/temp13491381570566576292/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz: GET returned 404 Not Found
ERROR: /home/amd/Repo/zml/examples/llama/BUILD.bazel:7:14: //llama:llama_lib depends on @@rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux//:zig_toolchain in repository @@rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux which failed to fetch. no such package '@@rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux//': java.io.IOException: Error downloading [https://mirror.zml.ai/zig/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz, https://ziglang.org/builds/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz] to /home/amd/.cache/bazel/_bazel_amd/e9f8a4534bc94824063a91fd4855a228/external/rules_zig~~zig~zig_0.14.0-dev.363_Pc3faae6bf_x86_64-linux/temp13491381570566576292/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz: GET returned 404 Not Found
ERROR: Analysis of target '//llama:OpenLLaMA-3B' failed; build aborted: Analysis failed
INFO: Elapsed time: 18.110s, Critical Path: 0.01s
INFO: 1 process: 1 internal.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target
FAILED:
    Fetching repository @@zml~~huggingface~OpenLM-Research-OpenLLaMA-3B; starting 16s
    Fetching repository @@zml~~rocm_packages~libpjrt_rocm; starting 16s
    Fetching https://github.com/zml/pjrt-artifacts/releases/download/v0.1.13/pjrt-rocm_linux-amd64.tar.gz; 62.2 MiB (46.7%) 14s
    Fetching https://huggingface.co/openlm-research/open_llama_3b/resolve/fcc2e809eb8f14dabba84d76a0ddc17b8ea05356/model.safetensors; 329.6 MiB (5.0%) 13s
(rocm6) amd@AIG-PM:~/Repo/zml/examples$

Would you mind retrying to directly download the failing package at this url: https://mirror.zml.ai/zig/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz ? Maybe there was an issue with Cloudflare or your connection with Cloudflare network is limited by something like vpn or so on.

Would you mind retrying to directly download the failing package at this url: https://mirror.zml.ai/zig/zig-linux-x86_64-0.14.0-dev.363+c3faae6bf.tar.xz ? Maybe there was an issue with Cloudflare or your connection with Cloudflare network is limited by something like vpn or so on.

Yes. It is causes VPN and firewall. It works without my VPN.

Glad to see the issue resolved. Don't hesitate to reach us through GitHub issues or our Discord (link available on readme) if need help about exploring ZML :)


Remove/hide reset filter from the filte modal

Related to this enhancement https://github.com/Esri/solutions-components/issues/647
Having two reset filters look redundant and let's hide/remove the first one as highlighted.

@jmhauck
To implement this we need updates in the instant app component to make reset buttons visibility configurable, just like we have a closeBtn.
FYI - @skrishn

@sumitzarkar I added another boolean prop resetBtn that when false will prevent the drawing of the "Reset Filter" button in IAC 1.0.0-beta.260. It's in our develop branch now

@jmhauck
We are able to use this prop, after setting it to false the reset button is not shown.
We will deliver the code in next code drop

@skrishn
Solution
Passed false value to resetBtn for the instant-apps-filter-list to hide the reset button

Dev Checklist

Items
Is required
Status
Comments

Required NLS changes?
No

Required Backward compatibility
No

Required 508
No

Need Sanitize?
No

Impact Analysis Report

Impacted Areas
Comments

Have Configuration changes?
No

Have Runtime changes?
Yes

Impact on RTL
No

Impact on instant app filter list
Single reset button should shown at the footer of the filter list component


Add state dict to stats trackers

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (14d4b99) 100.00% compared to head (23c724d) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #71   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           27        27
  Lines          746       777   +31
  Branches        91        93    +2
=========================================
+ Hits           746       777   +31

Impacted Files
Coverage Δ

src/arctix/stats/auto.py
100.00% <100.00%> (ø)

src/arctix/stats/base.py
100.00% <100.00%> (ø)

src/arctix/stats/continuous.py
100.00% <100.00%> (ø)

src/arctix/stats/discrete.py
100.00% <100.00%> (ø)

src/arctix/stats/noop.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Option to not truncate error output to viewportWidth

What is your Test Scenario?
Selector with a long CSS string is failing but the string is truncated using "..." in the error message
What are you suggesting?
Provide CLI option to not truncate error output to viewportWidth
What alternatives have you considered?
An option to write very detailed logging to a file (e.g. all actions/selectors) would also be very useful
Additional context
The error currently looks like this (id consists of several GUIDs - the last one is completely missing)

Cannot obtain information about the node because the specified selector does not match any node in the DOM
tree.
| Selector('#editor-5b8f3bc357b79b021655bd22_e5c03c18-2bf9-434e-9702-fbf449...)
 > |   .nextSibling()

@eug48
Hello,
   | Selector('#editor-5b8f3bc357b79b021655bd22_e5c03c18-2bf9-434e-9702-fbf449...)
 > | .nextSibling()

The only purpose of this error message is to show the .nextSibling() step in your selector's chain.
In the case of the narrow console window, you have all the necessary information to find an issue in your test scenario. For example:

We don't see any reason to add the "no truncate" CLI option.

Thanks @Farfurix for the response and helping to develop TestCafe. The problem is that my selector string is dynamically generated. Here is a screenshot:

Hi @eug48
You can see the full CSS string if you expand you terminal window to whole screen.


fix bug in import_file()

if fspec is a relative path it could be that a file is imported twice.
Fix:
First determine absolute path of fidl file and then checks if the file is already imported.

This is not completely correct.
Remember my little modification on your unitest test_import_file_chain()
self.processor.import_file(fspec)
self.processor.import_file("./Type1.fidl")
In this case the unit test runs successful but if you debug your code, you see that fidl file Type1.fidl is imported twice.
abs_fspec = os.path.abspath(fspec)
if abs_fspec in self.files:
# File already loaded.
fspec is maybe the filename e.g. fspec="./Type1.fidl"
abs_fspec = os.path.abspath(fspec) just join the current working directory with the file name
'E:\project\pyfranca\pyfranca\pyfranca\tests\Type1.fidl' in my case. But this is not the correct path and so "if abs_fspec in self.files:" failed. Later int the code the line
" temp_fspec = os.path.abspath(os.path.join(path, fspec))" creates the correct path 'E:\project\pyfranca\pyfranca\pyfranca\tests**\fidl\tmp**Type1.fidl'
Thats why it is important first to determine the the fspec path.

Coverage increased (+0.003%) to 93.889% when pulling a667626c22c9ab4f88f41b50d10cd706050e857e on jb090979:feature/bug_import_file into 6cc79e3f006955179bc59a6e6a2d484a7342e83b on zayfod:dev.

OK, thanks.


Add helm labels and annotations to all crds

Signed-off-by: David Cassany<EMAIL_ADDRESS>

Without this I still get
Error: UPGRADE FAILED: rendered manifests contain a resource that already exists. Unable to continue with update: CustomResourceDefinition "machineinventoryselectors.elemental.cattle.io" in namespace "" exists and cannot be imported into the c
urrent release: invalid ownership metadata; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "elemental-operator"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "c
attle-elemental-system"

on helm upgrade


Fix fetch response in prepareDataProto.

Looks like a logic regression bug.  Before it checked if there were no results after all targets had been looked up.
https://github.com/lomik/go-carbon/blob/a19deb850326715e3f846de6acd83094f7030b8c/carbonserver/carbonserver.go#L1276-L1282
I need a couple days to verify this actually fixes it though. :-)

I think it's still good idea to merge this PR, but I'd like to understand if it actually affects the result?

@Civil what I saw was that given i.e: /render/?target=host.one&target=host.two.
Currently, if the first target returns no data, then prepareDataProto exits early, and ultimately returns nothing.  Swap the targets around and data is returned.

In the middle of re-reading this, yes this patch is definitely wrong.  The continue in the loop is fine, the check for metricsFetched is ok.  But the += is not what I want, as that is an array that's appended to on each loop.
And I see that is also the case for metrics, memoryUsed and valuesFetched too.  All of that code should be hauled out of the range loop and done immediately after.

Ah, right, thanks!

@Civil - confirmed, this patch works.


Module transaction design

Styling of transactions vs proposals is off. I'm not 100% certain what information needs to be displayed here, but my take is that these transactions should use the same proposal title text style for consistency.

Link hover is wonky here, too:

Originally posted by @nicolaus-sherrill in https://github.com/decentdao/fractal-interface/issues/1578#issuecomment-2093679056

The link being wonky is covered by another Issue.
As far as the main issue here. The module transaction are included when we pull transaction from the safe API. These activities are included to my understand when safe module is interacted with. Honestly we can properly remove these as these can be confusing for general users and all interaction would be covered by a proposal anyway.

Let's remove them!

late to the game but agree with this

Closed via https://github.com/decentdao/fractal-interface/pull/1776


Update built-in samples to use lazy task configuration where it makes sense

in the gradle/gradle repo

I converted the Groovy samples and was going to look over the Kotlin samples now. There some tricky conversion that we will need to make a decision. I will comment in the PR https://github.com/gradle/gradle/pull/7976.

I would start with only the samples that show how to implement a plugin. We could probably also stop there for now.

Well, I already went through all the samples. What are we expecting here to be converted to lazy configuration?


Scan to struct

Hi all
Does this package support to scan result to struct?
Thanks

Yes

Dup of https://github.com/go-redis/redis/issues/672

@mikefaraponov Hi Mike
Thanks for your suggestion I'll try it

Update: This is now supported. https://github.com/go-redis/redis/blob/f8cbf483f4a193d441fac2cf14be3d84783848c6/example_test.go#L281

@knadh I don't think this works for Eval/EvalSha. I have been trying to use Scan for the output of my Lua scripts.
cmd := r.client.EvalSha(ctx, hash, []string{key}, args...)
EvalSha returns Cmd whose val is interface{} & it does not contain any Scan function
Could you let me know if I'm missing something?

@knadh I don't think this works for Eval/EvalSha. I have been trying to use Scan for the output of my Lua scripts.
cmd := rdb.EvalSha(ctx, hash, []string{key}, args...)
EvalSha returns Cmd whose val is interface{} & it does not contain any Scan function (cmd struct)
Could you let me know if I'm missing something?

The return value of the EvalSha command is unpredictable and depends on the return value of lua, so it does not provide the Scan function.
This may be achieved in RESP3 in the future.


Add "cause based" alerts for DNS operator

WHAT
Create cause-based alerts for the dns-operator
The tabletop scenarios can be used as inspiration for the alerts
possible alerts:

cannot access DNS Provider
conflicted default GEO

DONE

[ ] Create alerts for different scenarios for DNS operator
[ ] Create Promtool unit tests for each new alert

@R-Lawton can we get some information on how to setup and develop against alert manager locally?

going to move this beyond v1 as we will add a dashboard and the status will report if we cannot access a DNS provider, we will also add a doc to cover common causes of conflict #776


integration tests

Could some of the samples, smm seems particularly suitable, be massaged into integration tests?  It would boost my confidence in making changes and opening PRs.
I use travis for other projects.  If someone else sets up a script that returns 0 on pass and non-zero on fail, I'm willing to set up the rest.

I requested travis support from Hans earlier this we(I cannot enabled it right now...) exactly for this reason and as we have the JIT being pushed in a couple of days.
For now I suggest to use the test command after make and then grep'ing for diff. We simply need to put this into a bash script and we have our travis test.

make MNK="2 3, 23" test
grep "diff" samples/cp2k/cp2k-perf.txt | grep -v "diff=0.000"