[Enhancement] support element expr prune subfield

Fixes #issue

For support rewrite unpack semi-type feature later, refactor PruneSubfieldRule.java, split inner class to SubfieldAccessPathComputer.java, SubfieldAccessPathNormalizer.java, SubfieldExpressionCollector.java, and move to dir prunesubfield

support prune subfield on the children of ARRAY/MAP's value-column, like: the type of a2 is ARRAY<STRUCT<a int, b int>>

MySQL test> explain verbose  select a2[1].a from sc3;
+---------------------------------------------------------------+
| Explain String                                                |
+---------------------------------------------------------------+
| RESOURCE GROUP: default_wg                                    |
|    ......                                                     |
|   |                                                           |
|   0:OlapScanNode                                              |
|      table: sc3, rollup: sc3                                  |
|      preAggregation: on                                       |
|      partitionsRatio=1/1, tabletsRatio=3/3                    |
|      tabletList=7265851,7265853,7265855                       |
|      actualRows=3, avgRowSize=17.0                            |
|      Pruned type: 3 <-> [ARRAY<struct<a int(11), b int(11)>>] |
|      ColumnAccessPath: [/a2/INDEX/a]                          |
|      cardinality: 3                                           |
+---------------------------------------------------------------+

support more prune mode(INDEX) on ARRAY/MAP, for support choose special index/key on array/map
support prune ARRAY's element-column on method ARRAY_LENGTH, CARDINALITY
ColumnAccessPath support is_from_predicate field, to mark the path is normalized from scan's predicate, for support  late materialize the children of Struct type
refactor ColumnAccessPath handle process and the interface of column_iterater on BE, for support  late materialize the children of Struct type

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 270 / 275 (98.18%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/ColumnAccessPath.java
12
13
92.31%
[135]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathNormalizer.java
96
98
97.96%
[123, 163]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathComputer.java
98
100
98.00%
[129, 130]

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
20
20
100.00%
[]

:large_blue_circle:
com/starrocks/planner/ScanNode.java
14
14
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldExpressionCollector.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/PruneSubfieldRule.java
8
8
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 270 / 275 (98.18%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/ColumnAccessPath.java
12
13
92.31%
[135]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathNormalizer.java
96
98
97.96%
[123, 163]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathComputer.java
98
100
98.00%
[129, 130]

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
20
20
100.00%
[]

:large_blue_circle:
com/starrocks/planner/ScanNode.java
14
14
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldExpressionCollector.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/PruneSubfieldRule.java
8
8
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 265 / 270 (98.15%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/ColumnAccessPath.java
12
13
92.31%
[135]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathNormalizer.java
91
93
97.85%
[105, 149]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathComputer.java
98
100
98.00%
[129, 130]

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
20
20
100.00%
[]

:large_blue_circle:
com/starrocks/planner/ScanNode.java
14
14
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldExpressionCollector.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/PruneSubfieldRule.java
8
8
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 265 / 270 (98.15%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/ColumnAccessPath.java
12
13
92.31%
[135]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathNormalizer.java
91
93
97.85%
[105, 149]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathComputer.java
98
100
98.00%
[129, 130]

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
20
20
100.00%
[]

:large_blue_circle:
com/starrocks/planner/ScanNode.java
14
14
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldExpressionCollector.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/PruneSubfieldRule.java
8
8
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 265 / 270 (98.15%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/ColumnAccessPath.java
12
13
92.31%
[135]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathNormalizer.java
91
93
97.85%
[105, 149]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathComputer.java
98
100
98.00%
[129, 130]

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
20
20
100.00%
[]

:large_blue_circle:
com/starrocks/planner/ScanNode.java
14
14
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldExpressionCollector.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/PruneSubfieldRule.java
8
8
100.00%
[]

[FE PR Coverage Check]
coverage check fail. please retry. :fearful:
Please let me know when error again.
com.naver.nid.cover.parser.coverage.exception.ParseException: java.nio.file.NoSuchFileException: /home/runner/_work/starrocks/starrocks/result

[FE PR Coverage Check]
:heart_eyes: pass : 268 / 270 (99.26%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathNormalizer.java
91
93
97.85%
[105, 149]

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
20
20
100.00%
[]

:large_blue_circle:
com/starrocks/planner/ScanNode.java
14
14
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldExpressionCollector.java
22
22
100.00%
[]

:large_blue_circle:
com/starrocks/catalog/ColumnAccessPath.java
13
13
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/SubfieldAccessPathComputer.java
100
100
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/prunesubfield/PruneSubfieldRule.java
8
8
100.00%
[]

@Mergifyio backport branch-3.1


Add callback to authorize access token before each request

Addresses issue: #REG-31
Actions:

Add before_action callback to validate access token each request.
Add gamora initializer generator.
Update README documention.

@javierg I made some comments in the conversations. Could you please take another look in the PR?


Remove unused Newtonsoft.Json package references

Also, replace with System.Text.Json in Settings test that was still using Newtonsoft.Json

Summary of the Pull Request

PR Checklist

[ ] Closes: #xxx
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

/azp run


Multiple DIMSTYLE issues

When I add any number of DimStyles, the generated DXF document won't open on AutoCAD.
Inspecting the file I've found that there is an AcDbDimStyleTable subclass before each DIMSTYLE and this is causing the error.
As a workaround I changed the DxfDimStyle and DxfTable classes.
The DxfDimStyle class I commented this line:
dx.subclassMarker('AcDbDimStyleTable');

And on the DxfTable:
class DxfTable {
    constructor(name) {
        this.name = name;
        this.maxNumberEntries = 0;
        this.ownerObjectHandle = '0';
        this.handle = Handle.next();
        this.records = [];
    }
    dxify(dx) {
        dx.type('TABLE');
        dx.name(this.name);
        dx.handle(this.handle);
        dx.push(330, this.ownerObjectHandle);
        dx.subclassMarker('AcDbSymbolTable');
        dx.push(70, this.records.length);
        // Added this IF clause to ensure only one AcDbDimStyleTable
        if (this.name == "DIMSTYLE") {
            dx.subclassMarker('AcDbDimStyleTable');
        }
        for (const record of this.records)
            record.dxify(dx);
        dx.type('ENDTAB');
    }
}

This change allowed me to open the files on AutoCAD

Hi,
Thank you for the report, you are right and its fixed in v2.3.5.
Regards.


Creates a TokenMetadataStore to return startPosition of tokens in results

I took a stab at prototyping a solution for https://github.com/olivernn/lunr.js/issues/25, and took your comments in https://github.com/olivernn/lunr.js/pull/58 into consideration.
I think you're right that the long term goal should be to refactor to replace existing tokens with a lunr.Token. Unfortunately this seems a bit overwhelming, as it presents a lot of complexities with previously serialized dataStores and complicates the TokenStore. As a step in the right direction, I propose a lunr.TokenMetadataStore, which maps doc refs and indexed tokens to a lunr.Token. In this case, by "indexed token" I mean present day token (string) and a lunr.Token, which is an object I introduce to encapsulate metadata about tokens (like StartPosition). This MetadataStore lives on the sidelines, and is only added to when a document is indexed, and data is only retrieved when a document is surfaced in search results.
So now you'll get back a result set that looks like
// idx.add({ id: 2,
//   body: "Some are born great, some achieve greatness, and some have greatness thrust upon them."
// })
// idx.search("greatness")

[{
    "ref": 2,
    "score": 0.12345
    "tokens": [
      {indexedAs: "great", raw: "great", startPos: 15, field: "body"},
      {indexedAs: "great", raw: "greatness", startPos: 35, field: "body"},
      {indexedAs: "great", raw: "greatness", startPos: 60, field: "body"}
    ]
}]

Overall, I created three new object types:

lunr.Token which is a container for metadata like startPosition
lunr.TokenList is essentially an array of lunr.Tokens, but has some helper method to extract the indexed tokens and the raw tokens.
lunr.TokenMetadataStore is a dataStore as described above.

The only changes I had to make to the existing codebase are outlined as follows:
pipeline.js#run
This is probably the most substantial change I made. Running the pipeline now returns a lunr.TokenList instead of an array of strings. This works with old tokenizers that return strings or new tokenizers that return lunr.Tokens. The pipeline passes string tokens to the stack, so backwards compatibility is preserved for 3rd party pipeline functions and 3rd party tokenizers.
index.js#add
Updated this method to store the lunr.Tokens returned by the pipeline in the lunr.tokenMetadataStore. I added a configuration variable on the lunr.Index.prototype called useTokenMetadata which controls this behavior.
index.js#search
Once a list of documents is found, return a list of lunr.Tokens that are associated with the docment and were in the query string.
tokenizer.js
This was a pretty big change, and I did a quick and dirty job. I tried to keep the runtime relatively sane, but didn't worry too much about code organization. This should definitely be cleaned up if/when a merge happens.
Notes:

Drawbacks: This will use up a lot of memory, since we create an individual token for every single string that gets indexed.
Nomenclature: I started using the term "index token" to describe a present day "token", i.e a string that has been run through the pipeline (and will eventually end up in the index). A "raw token" is a String with the value of how the token appeared in the original document (with the exception of being lowercased, since we currently lowercase int he tokenizer).
Tests: I updated a few things in the tests, but otherwise they all passed. I wrote a few tests just to smokescreen my additions. If you think this is headed in the right direction then I can sure it up with more tests.

@olivernn Did anything become of the next branch, or of @cambridgemike's patch?

@aschuck sadly no, its all still available on github, but I haven't had a chance to take these any further.

I guess there are no updates with this, sadly. :disappointed:


Install FiraCode

https://github.com/tonsky/FiraCode

homebrew/cask-fonts is already taped, so there is just to do:
homebrew_cask_packages:
  - font-fira-code


python310Packages.coconut: 2.1.1 -> 2.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.coconut is: "Simple, elegant, Pythonic functional programming"
meta.homepage for python310Packages.coconut is: "http://coconut-lang.org/"
meta.changelog for python310Packages.coconut is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.1.1 with grep in /nix/store/z4g76glha7l61sal2j9k83w0cgr41ml7-python3.10-coconut-2.1.1
found 2.1.1 in filename of file in /nix/store/z4g76glha7l61sal2j9k83w0cgr41ml7-python3.10-coconut-2.1.1

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
coconut
python310Packages.coconut
python39Packages.coconut

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/z4g76glha7l61sal2j9k83w0cgr41ml7-python3.10-coconut-2.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z4g76glha7l61sal2j9k83w0cgr41ml7-python3.10-coconut-2.1.1
ls -la /nix/store/z4g76glha7l61sal2j9k83w0cgr41ml7-python3.10-coconut-2.1.1/bin

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

2 packages built:

coconut (python310Packages.coconut)
python39Packages.coconut

Maintainer pings
cc @fabianhjr for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.coconut is: Simple, elegant, Pythonic functional programming
meta.homepage for python310Packages.coconut is: http://coconut-lang.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2.0 with grep in /nix/store/mrka7aafpi02gvmcbyp3717dkfq9n4zk-python3.10-coconut-2.2.0
found 2.2.0 in filename of file in /nix/store/mrka7aafpi02gvmcbyp3717dkfq9n4zk-python3.10-coconut-2.2.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
coconut
python310Packages.coconut
python311Packages.coconut

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mrka7aafpi02gvmcbyp3717dkfq9n4zk-python3.10-coconut-2.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mrka7aafpi02gvmcbyp3717dkfq9n4zk-python3.10-coconut-2.2.0
ls -la /nix/store/mrka7aafpi02gvmcbyp3717dkfq9n4zk-python3.10-coconut-2.2.0/bin

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

1 package failed to build:

python311Packages.coconut

1 package built:

coconut (python310Packages.coconut)

Maintainer pings
cc @fabianhjr for testing.


Add release certifying workloads

The PR introduces several improvements to OMB and add release certifying workloads.
Improvements:

Track errors over time
Add sample rate params for using more precise time buckets for smoke workloads
Besides regular p-values (50, 75, 99 etc) start tracking min latency
Track blocking part of the async methods
Add new chart generation scripts (see README)
Add swarm topology

Swarm topology
For distributed workloads (DistributedWorkersEnsemble) OMB dedicates part of the client nodes for producing and another part for consuming. Since the start and the end of the end-to-end operation happens on the different nodes OMB relies on the wall clock to calculate the latency. Because the global time is an abstraction the clocks can't be perfectly synchronized and the e2e latency data are subject to clock drift and aren't accurate on the small scale.
This PR introduces a new topology for distributed test execution (SwarmWorker):

each node acts as a producer and as a consumer
records include a producer's node id and its time
each node uses its own consumer group meaning each node reads the whole log
a consume ignores records originated on different nodes and only measure e2e latency of the records written by the same node

The downside of the approach is the increased read traffic. So if before with a consumer and a producer we were able to saturate network with 1.2MM ops/s now we're capped at 400K ops/s: with two nodes each node writes X records and reads 2X records resulting in X = 1.2MM / 3 = 400K
Certifying workloads
We have smoke and full workloads. Smoke workloads are used for the fast estimation of performance and full workloads tests long lasting load tests. Duration of the tests isn't the only difference. The smoke tests use different rates to check when the client starts to choke (400k, 625k, 900k and 1250k).
We test the following client configurations:

acks=1
acks=all, idempotency off
acks=all, idempotency on

Each configuration includes (linger.ms=1 and max inflight set to 5).
With the following scale:

simple: two produce-only clients, single partition, 50K rate
e2e: two clients, each hosts a producer and a consumer, single partition, 50K rate
load: two clients, each hosts 16 producers and 8 consumers, 100 partitions, rates - 400k, 625k, 900k and 1250k

We use simple to estimate performance of a single raft group. Also it's useful for estimating the impact of the external operations on the performance. For example if we were using a hundred partitions even if one gets stuck this signal could have been dominated by the good partitions.
We use e2e to see if reads affects writes and vice versa.
load is work horse which gives the main signal. When we talk about performance in general, it's about the load workload.

we should seriously consider submitting this to the opensource repo

that's the plan


Clean up main folder in repo

There seems to be a bunch of html and config files that may or may not need to be in the main folder in the repo. Confirm for each file that it is okay being put into a different folder (and update paths in the code to these files as needed). This will be good for making the repo cleaner

Not worth the work to do it


Add isort config and run it on luigi_pipeline

Accidentally closed this by deleting the underlying branch, which should have already been deleted.


Crash when replacing number with an int with a none cell

Traceback (most recent call last):
  File "/mnt/c/data/elow/main.py", line 73, in <module>
    curses.wrapper(main)
  File "/usr/lib/python3.10/curses/__init__.py", line 94, in wrapper
    return func(stdscr, *args, **kwds)
  File "/mnt/c/data/elow/main.py", line 70, in main
    render(console, display.render())
  File "/mnt/c/data/elow/display.py", line 121, in render
    return self.table_display.render()
  File "/mnt/c/data/elow/display.py", line 95, in render
    col_sizes = self._get_col_sizes()
  File "/mnt/c/data/elow/display.py", line 34, in _get_col_sizes
    col_sizes[col] = max(len(str(cell)), col_sizes[col])
  File "/mnt/c/data/elow/cell.py", line 108, in __str__
    return self._cell_displayers[self.type](self)
  File "/mnt/c/data/elow/cell.py", line 85, in display_formula_cell
    return self._display_number(val)
  File "/mnt/c/data/elow/cell.py", line 64, in _display_number
    if num.is_integer():
AttributeError: 'int' object has no attribute 'is_integer'

I was replacing a number cell, that has 12 in it with a none cell.

Failed to reproduce, if anyone finds a way to reproduce, please comment here.


Debug mechanism for showing OpenAI responses

I figured out a pattern for this here and it's really useful: https://github.com/simonw/llm/issues/284#issuecomment-1724793763

This is particularly useful for writing tests, since seeing the raw HTTP response from the OpenAI endpoint shows me what I need to mock.

I'm going to do this with an environment variable:
LLM_OPENAI_SHOW_RESPONSES=1 llm -m chatgpt 'three word slogan for an an otter-run bakery'

Output:
data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"role":"assistant","content":""},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":"\""},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":"Fresh"},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":"ly"},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":" B"},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":"aked"},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":" Happiness"},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":"\""},"finish_reason":null}]}

data: {"id":"chatcmpl-80MK5s4E5KmkkskxTKVzEyD1WUcHc","object":"chat.completion.chunk","created":1695096501,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{},"finish_reason":"stop"}]}

data: [DONE]

"Freshly Baked Happiness"


keybinding 'alt' collides with keybinding of atom-core

linter adds this keybinding: alt	linter:set-bubble-transparent	Linter	atom-text-editor:not(.mini)
It collides with
Atom > Settings > Core Settings > Auto Hide Menu Bar
Automatically hide the menu bar and toggle it by pressing Alt. This is only supported on Windows & Linux.
This combination results in not being able to toggle the menu bar within editor windows. This might affect many users of atom, because it is a core keybinding. I suggest to change the default binding for linter to something else.

What alternative key would you suggest for a peek below the bubble command?

I don't have a preference, because I am quite new to atom (~2 hrs) and just setting it up and getting to know it :-)

/cc @k2b6s9j @AsaAyers is ctrl key used by any core commands?

On Windows? Or everywhere?

Everywhere.

Ctrl is free on my OS X install.

Alt was taken up by Nuclide's click to symbol.

Yeah, same story on linux and windows but not sure, Can someone else confirm that?

I tried to map CTRL on Linux, but that did not work for me for some reason (Ubuntu 15.04)

What about switching to toggle-behaviour for transparency? It could then be a more complex keybinding, like CTRL+ALT+[a-Z0-9]
In this case please consider, that keyboard layouts can be quite different around the globe (mostly for special characters) - standard letters should generally be better than special characters. In fact I had to override the other keybinding alt-shift-, because shift-, is interpreted as ; with my current layout and the keybinding did not work, too. But that did not seem as relevant as this one, so I did not file it as a bug. My point is: If you are looking for new keybindings, please try to avoid shift + special-char, because that could lead to unexpected behaviour on different keyboard-layouts.

Another idea:

default to 'transparent bubbles' when the cursor is moved
on 'next-error' switch to 'opaque bubbles'
** if the bubble was transparent before (i.e. the cursor had moved) than stay at the current position
** else go to the next error

I think that might work, too.

@colin-kiegel I have just tested mapping ctrl and it works flawlessly.
Here's my keymap.cson file
'atom-text-editor:not(.mini)':
  'ctrl': 'linter:set-bubble-transparent'

that does not work for me (neither left-ctrl or right-ctrl) - I don't know why - but this does
'atom-text-editor:not(.mini)':
  'alt-ctrl': 'linter:set-bubble-transparent'

but only if alt is pressed before ctrl - which is not good either
Anyway - I think mapping it to ctrl is better than mapping it to alt to avoid the collision

What other packages are you using by the way?

build, cargo-test-runner, keyboard-localization, language-rust, linter, linter-rust, merge-conflicts, minimap, racer, run-command2, rust-api-docs-helper, script, vim-mode
it doesn't help if I deactivate everything except linter, linter-rust and language-rust

Okay so enable Key Binding Resolver from the command pallete. and then press the control key, which commands do you find associated with that key?

There are no key bindings associated with ctrl. Can it be OS-related? (Ubuntu Gnome 15.04?) It doesn't make sense to me.
I think, you could change it and see if other people complain.

Weird, I am running the same OS Ubuntu GNOME 15.04 with GNOME 3.16 with Atom 0.208.0 and it works flawlessly, for me and other people I know.

Could be a driver thing.

@k2b6s9j linux doesn't really have drivers, windows does.

FWIW the alt+shift+. does not work for on mac or linux (haven't tested windows).
On mac it inputs ˘ and on linux it's interpreted as alt+> and thus not working. There are no conflicting key bindings and I was rather surprised by changing it away from ctrl+k ctrl+n in the first place. Since that worked flawlessly and now isn't used by anything I have installed.

@despairblue I think this ticket is actually about a different keybinding - please have a look at https://github.com/AtomLinter/Linter/issues/667 :-)

@colin-kiegel true, I simply put it here because you mentioned a similar issue and the key binding could be addressed by this ticket as a whole instead of opening a ticket for each non working key binding.

+1 for resolving the collision with Core "Auto Hide Menu Bar"... honestly don't have a positive suggestion for a replacement though.  Even with "Auto Hide Menu Bar" off though, this binding prevents using, for example, Alt+F to access the File Menu.
I haven't tried switching it to Ctrl, but I suspect it would have similar problems, blocking Ctrl-C for example.  Binding a single command to a Chord Key as a default is probably never a good idea.

@mcuznz I don't think that the alt binding is preventing you from using alt-f. It's rather this one from core:
'atom-text-editor':
  'alt-f': 'editor:move-to-end-of-word'

Because e.g. alt-v is working for me.

@despairblue I'll disagree, because alt-f to open the File Menu (or alt-e for Edit, or any other menu item) work just fine as soon as I disable Linter - or for that matter, if I override it's "alt" keybinding.

@mcuznz did you unset alt-f as well? I'm just saying that if you disable linter, reload atom and open a file alt-f will not work unless you unset  editor:move-to-end-of-word, which is a core binding. It's true that the menu bar is not shown by pressing alt because of the linter key binding, but the navigation shortcuts like alt-e and or alt-v should work nevertheless, with the exception of alt-f and alt-h, because these are taken up by atom itself.
Could you check the keybinding resolver to make sure that it's really linter that is prevent alt-f from working?

My apologies, I was a bit unclear earlier.
The Chorded input Alt-F (pressed simultaneously) triggers the core "move-to-end-of-word" function as expected, regardless of Linter's enable/disable state.
With Atom's "Auto Hide Menu Bar" option turned on, the Alt key (single press, not chorded) is required to reveal the Menu Bar, wherein after a brief delay I would typically press "F" to access the file menu, "V" for the view menu, etc.
However, with Linter enabled, pressing the Alt key is captured by "linter:set-bubble-transparent" which prevents the hidden Menu Bar from being revealed, thus making all menu items inaccessible by an additional keystroke.
Further, if I add the following to my personal Keymap File, the default "Show Menu" functionality is restored:
'atom-text-editor':
  'alt': 'unset!'

This solves the problem, but of course effectively disables this feature of Linter :)

This is weird, since it works for me under linux, but I don't use Ubuntu/Unity. Maybe Unity does something weird there, though I don't see the connection to linter in that case :confused:.
In my case the menu bar does not show when pressing alt, but it does as soon as I press v as well, opening the view menu. That's also why the alt shortcut does not annoy me since I can still open the menu bar that way (also the nuclide is also using it, which is annoying, but not linter's fault)

FWIW, I'm running Ubuntu 14.04 / Gnome Shell 3.10.4, not Unity ;)

good choice imo :smile:

Got the same problem here, I was already reporting this issue to atom (before running it in safe mode) until I saw this here.
I think this should have a great priority, since more people can experience it and lose time trying to figure out what happened.
I understand that may be hard to choose another shortcut, but the package shouldn't conflict with the atom's core. Right?
As a fast workaround, I commented the shortcut.

Any progress on this? i would also like to see this fixed. I tryed to override it, but apparently it's not possible.

@Fedcomp We just haven't found any alternative key to alt, but we are totally open to ideas :)

I highly suggest to just comment it untill you find appropriate key, because alt menus is much, much more important than some linter:set-bubble-transparent
It's just frustrating to me that i have either to remove the extension (which would totally help me see rust errors) or work with tabs open (thoose dont support themes and use system colors, so i may see dark theme and eye-hurting white menus).
Imo, general keys, such as cmd, ctrl, alt should never been binded as too common. Especially in extensions.
Or is there a setting in the linter to override the keys? i would totally like to have an option, or way to get rid of alt keybinding.

Since a lot of people already 'complained' (see https://github.com/atom/atom/issues/7750) - I suggest to just use anything else. But to do this rather soon. I think 'being quick' on this is very important, because a lot of people are experiencing atom for the first time right now - and this might just hurt the overall experience. We want people to have a good first impression, right?
So let's just pick the first good alternative that's working for everyone:
For example alt-?. On US/GB keyboards this should be close to alt-> and on DE keyboards (german) it's ok too.
I think it's really really important to move away from alt quickly.

Requesting suggestions for alternatives People.

I recommend turning off the feature until it can be done right.
On Jul 18, 2015 11:14 AM, "Steel Brain"<EMAIL_ADDRESS>wrote:

Requesting suggestions for alternatives People.
—
Reply to this email directly or view it on GitHub
https://github.com/atom-community/linter/issues/648#issuecomment-122552007
.

Fixed by #755


support for ubuntu 20.04

hi , i am done & success build with zimbra 8.8.15 & 9.0 using your clue script
i am already doumentation my gits
https://gist.github.com/alsyundawy/be1a00e909e73a3c452a2ab3d1a1beed
can you make yor script full support for ubuntu 20.04?

I did have support for Ubuntu 20.04 but there were problems with the Zimbra repositories during installation so it was dropped.  I will have to check it entirely to ensure that it can be built and installed.  Since previously it was only possible to build but not to install it, then it was a bit pointless.
I'll check and see if it's possible or not.

this for 9 or 8.8.15?

Version 9.0.0.

already done for ubuntu 20.04?

It seems Zimbra have fixed their repos.  Scripts now support Ubuntu 20.04 again.


messageFrom does not appear on notification screen anymore

Hi,
I used to see the "messageFrom" value from notification payload in the notification ballon. Now I can't. My phone is having iOS 11.1.
I think they might changed something in APN, as other apps continue to show notifications with title and body as before (I use the "messageFrom" as notification title).

To my knowledge, messageFrom is not part of the specified APNS payload arguments.

This is really, really strange. I was using the example from the README of this project where "messageFrom" is specified and indeed it was used as notification title, until some moment. Very strange.

I was searching for it, because we are not using an argument by that name and I was curious. I really only found a reference to the term in this project. If you find out anything, I'd love to know about it :)


Update supported os

Added support for Linux on Z(s390x)

This is a deprecated cookbook that we're not maintaining or releasing at this point.  I'm going to close this PR out.


Minor Compressions in statements in folder.py

This PR just compresses two statements into one as mentioned here
https://github.com/pytorch/vision/pull/867#discussion_r278529368

Codecov Report

Merging #874 into master will increase coverage by 0.21%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #874      +/-   ##
==========================================
+ Coverage   55.03%   55.24%   +0.21%
==========================================
  Files          36       36
  Lines        3367     3365       -2
  Branches      552      551       -1
==========================================
+ Hits         1853     1859       +6
+ Misses       1377     1372       -5
+ Partials      137      134       -3

Impacted Files
Coverage Δ

torchvision/datasets/folder.py
76.25% <0%> (+1.85%)
:arrow_up:

torchvision/transforms/transforms.py
82.54% <0%> (+1.29%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c0c696f...2bfd7b5. Read the comment docs.


[HOLD for payment 2024-11-01] Unread chats are missing from LHN (left hand navigation)

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

Version Number: v9.0.42-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: @tgolen
Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1727795413457949
Action Performed:

Sign into NewDot
Make sure you are in focus mode
Have some unread chats in your LHN
Open the chat finder window (magnifying glass on the top-right of the LHN)

Expected Result:
The Chat Finder page should not show the hidden chats as bold/unread. The hidden chats should remain invisible in LHN.
Actual Result:
There are unread chats in the chat finder which don't appear in the LHN
Workaround:
The workaround is to open the chat finder to locate the unread chats.
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @bfitzexpensify

Proposal
Please re-state the problem that we are trying to solve in this issue.
There are unread chats in the chat finder which don't appear in the LHN
What is the root cause of that problem?
The RCA here is the unread chat has isHidden is true and shouldOverrideHidden doesn't cover the case the chat is unread
https://github.com/Expensify/App/blob/f12d016ff0f4d0ab3235f270d26f7a7e61748d7a/src/libs/SidebarUtils.ts#L137-L145
What changes do you think we should make in order to solve the problem?
We should add the check unread to shouldOverrideHidden to avoid hiding the unread chat if we're in focus mode.
const shouldOverrideHidden =
  // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
  hasValidDraftComment(report.reportID) ||
  hasErrorsOtherThanFailedReceipt ||
  isFocused ||
  isSystemChat ||
  report.isPinned ||
  isExpenseReportWithoutParentAccess ||
  (ReportUtils.isUnread(report) && isInFocusMode);

https://github.com/Expensify/App/blob/f12d016ff0f4d0ab3235f270d26f7a7e61748d7a/src/libs/SidebarUtils.ts#L137-L145
Or if we always want to show the unread chat in LHN we can remove && isInFocusMode in the condition above.
What alternative solutions did you explore? (Optional)

Yeah, we want to be sure unread chats show in either view mode. @nkdengineer Is this a recent regression?

I am not sure if the expected scenario here is correct, discussing here

After discussion, I have updated the expected behaviour:

The Chat Finder page should not show the hidden chats as bold/unread; they should remain invisible in LHN.

@ZhenjaHorbach @nkdengineer can you update the PR to reflect that please?

@mountiny Thanks for the update, I will update the PR soon.

Offers sent to both @ZhenjaHorbach and @nkdengineer
@ZhenjaHorbach, please complete the BZ checklist

@nkdengineer, please accept the offer in Upwork.

@bfitzexpensify Done 👍

Great - payment complete, closing this one out. Thanks everyone!


[refactor]: Replace thiserror with derive_more

          We could consider replacing `thiserror` with `derive_more`, given that this pattern would be covered with `derive_more::From`, and we can derive `Display` independently from `error`.

Originally posted by @appetrosyan in https://github.com/hyperledger/iroha/pull/3138#discussion_r1113756103
thiserror::Error could be replaced with combination of derive_mode::Error/Display/From.
This will give us more convenient way to derive From impls.

@DCNick3 addressed the main concern


Build failure: temporal-cli lacks Hydra job for Darwin

Not strictly a build failure, but rather the lack of a build on Hydra at all.
Steps To Reproduce
Steps to reproduce the behavior:

nix build -j0 --builders '' nixpkgs/nixpkgs-23.05-darwin#temporal-cli

Build log

error: unable to start any build; either increase '--max-jobs' or enable remote builds.

but expected this to be available in cache.nixos.org
Additional context

This does not occur on nixpkgs-22.11-darwin, as there is a Hydra job for it there
There is no temporal-cli.x86_64-darwin or temporal-cli.aarch64-darwin job in Hydra nixpkgs-23.05-darwin or trunk
My guess is it was introduced in f326911cb926d264d784186219f4608bf557170f, but I’m not sure why
There are no apparent related evaluation errors (searched for “temporal-cli” and “tctl”)

Notify maintainers
@aaronjheng
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

Thank you for the report. I tried it myself and indeed encountered an issue. However, what's strange is that the build on the master branch doesn't have any problems, even though there aren't any significant differences in the definitions between these two branches. I will investigate this issue further later on.

I see the issue on master as well—perhaps you had it already built locally?

Yes.
I tried again, and this time both branches were successful. wat.


[SR-4793] Add check that target path is not referenced by absolute path

Referencing target path with AbsolutePath is error prone and should be avoided.

When subpath is checked I intentionally not throwing Error from RelativePath(validating:) to make it more clear to the user where to fix this

LGTM!

@swift-ci please smoke test


feat(wva_ct_app): Add scraper for WVA CT APP

Add West Viriginia Court of Appeals Scraper
Resolves #599

Nice. Easy-peasy on this one.


Proposal: Drop Node.js 0.12 support

Node.js 0.12 is slow.
We want to drop support for Node.js 0.12 @ textlint 7.0 #184
It means that removing - "0.12" from .travis.yml

Other projects:

https://github.com/eslint/eslint/issues/4483
https://github.com/springernature/shunter/issues/110
https://github.com/expressjs/express/issues/2755

Node.js version share
https://twitter.com/seldo/status/736004442433036288

min-version: 4.0


linux build problem

How often does this problem occur?
I pushed a fix for the include mentioned in your message, so it should be fixed but I can't guarantee there aren't others.


Barcode list for DropSeq

Very excited at the kallisto/bustools release! Just in time for my re-analysis project.
Would you suggest creating a whitelist by running UMITools on the fastq from DropSeq or just skipping the bustools correct step?
https://github.com/CGATOxford/UMI-tools/blob/master/doc/Single_cell_tutorial.md#Variations

There is a bustools whitelist command you can use to extract a whitelist from the barcodes (that you can afterwards correct with bustools correct). The whitelist command is implemented in a fork (https://github.com/laureneliu/bustools) so you would have to compile from source. It's currently under review on a pull request and will be official with the next release of bustools. I can tell you it has been extensively tested, although not with Drop-seq data, so if you try it feedback would be appreciated.

Wow this is slick (and for others reading,  https://github.com/laureneliu/bustools/commit/a199123faa25138b7ed7f5f75d3ce9145e54d7c0 can compile on OS X). I assume that you've hard-coded the barcode/UMI patterning into kallisto bus?
This is a really easy workflow to get matrices of counts by cell/gene that extends across multiple technologies with just one flag in one tool (-x TECHNOLOGY)!
kallisto bus r1.fq r2.fq -x TECHNOLOGY -i INDEX -o sample
bustools sort sample.bus -o sample.sorted.bus
bustools whitelist sample.sorted.bus -o WHITELIST.txt
bustools correct sample.sorted.bus -w WHITELIST.txt -o sample.sorted.correct.bus
bustools count # well just go see https://www.kallistobus.tools/getting_started and https://www.kallistobus.tools/documentation

(this is ignoring the fact that these all should pipe together for clarity)

Many of the frequently used technologies are hardcoded but you can also specify custom patterning (see the kallisto manual for details: https://pachterlab.github.io/kallisto/manual)

What?! (checks HPC kallisto version). Oh, I've been using 0.45.
Not that I have to do this (thank goodness), but you would give the custom bc:umi:seq info with the -x flag?
https://pachterlab.github.io/kallisto/manual#bus

Yes.


ci: add config for gci linter

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

v1.0.0
GitHub release

Your semantic-release bot :package::rocket:


Create 1456 maximum-number-of-vowels-in-a-substring-of-given-length.go

File(s) Modified: go/1456-maximum-number-of-vowels-in-a-substring-of-given-length.go
Language(s) Used: Go
Submission URL: _https://leetcode.com/problems/maximum-number-of-vowels-in-a-substring-of-given-length/submissions/1444038246/_

Leetcode formatted and solution matches Neetcode video.

@Jasoncheung94 Thanks for your contribution 👍


Update admins input UI

Resolves #241
Preview link

Hey @ori-near, please take a look at the UI and let me know if it works as expected

Hi @Megha-Dev-19 – this looks awesome! A few follow up requests:

We must require at least one admin. So a user cannot remove all admins. Just everyone except for the first one.
It would be great if we can leverage the suggested mention module you fixed in the previous ticket. For example, if I type @el, I would see a list of possible suggestions and clicking on them will auto complete the person. Also, ideally it wouldn't show any users who've already been added.
I think the "Add Admin" part could use some UX improvement, as it's not entirely clear that you need to type and then click enter. But we can revisit this later.

Hey @ori-near
I am not sure about the first point.
Thinking as a user, I wouldn’t always enter the correct first admin, or maybe I don’t want them anymore and if I am restricted from removing them, it’s an issue.
I can disable the submit button if no admins are added. Or show an error too, to add one.
What’s your views about it.

Hi @Megha-Dev-19 – we want to avoid a situation where a group is left without any admins. Essentially whoever creates the group to begin with is the default admin. They can't remove themselves unless they add a secondary admin. Let's say you are the admin of a page you created. You won't be able to remove yourself, but if you add me, I can remove you or you can remove yourself. But the last person cannot be removed unless they add someone else.

We have a prevention on contract side, so if you're removing the last admin, contract would detect it and prevent you from doing so. Of course, It would be nice to have that validation on frontend

@ori-near I have added your 2nd point.
for 1st point when only one admin is there I have disabled delete (not restricted to first item), let me know if you want to change it.
I have made some updates for 3rd point as well, let me know if it is clear now
Preview link

Hi @Megha-Dev-19 – this looks great, nicely done. I just have one last additional request:

When you click on an admin (after they've already been added, can we open their profile in a new tab instead of in the existing window? This will ensure that we don't lose progress).

With 5 done, I consider this ticket accepted 🚀 and ready to merge.

@ori-near updated. Link


kotlin2cpg: set names for arguments of ctor calls

useful feature when in need to find patterns of secrets which point to the hardcoded secrets of a service when taken together
ExampleRemotePaymentService(
            appId = "example.fingrid.corp",
            secretKey = "mohdeigh6nae_not_real_datasuco5ahBei0AeDi9xo",
            url = "https://api.kx.com/highfreqdata?latest="
        )


Slots cannot be toggled using v-if directive

Version
2.3.0
Reproduction link
https://jsfiddle.net/teddyrised/nkomyf9g/
Steps to reproduce

Set up app so that we have a built-in toggle, say showSlot, stored in the data object of the app
The app contains components, in which contain slots
In the component template, define slots so that one can be toggled using v-if based on the showSlot value
Go to console. Type in app.showSlot = true

Markup:
<div id="root">
  <custom desc="Slot that is always shown"><template slot="test-slot">{{ this.content }}</template></custom>
  <custom desc="Slot that is conditionally shown"><template v-if="this.showSlot" slot="test-slot">{{ this.content }}</template></custom>
</div>

JS:
Vue.component('Custom', {
  template: '#template',
  props: {
  	desc: String
  },
  computed: {
    hasSlotData() {
        return this.$slots['test-slot'];
      }
  }
});

var app = new Vue({
  el: '#root',
  data: {
  	content: 'foobar',
    showSlot: true
  }
});

What is expected?
The <template v-if="this.showSlot" slot="..."> should be rendered when app.showSlot is set to true.
What is actually happening?
The <template v-if="this.showSlot" slot="..."> remains unrendered.

I was creating an app where the slot should be conditional, so that it is not rendered when not necessary.

Never use this in the template:
<template v-if="showSlot"
not
<template v-if="this.showSlot">
second mistake, use
`v-if="$slots['test-slot']"
instead of computed property because $slots are not reactive
FIxed jsfiddle: https://jsfiddle.net/q4kmyqdh/

@sqal Thanks for the quick response :+1. I didn't know that this.$slots are not reactive. Is it not possible to encapsulate that logic in a method, instead of writing it in the markup?

@terrymun Sure, you can move the logic to the method, it will work because method will be called on every re-render

@sqal Thanks again! You really opened my eyes on how computed properties and methods can be so different. And I didn't know you can use methods like that--I've always thought they are meant to be triggered by user interaction :)

Similar to #3517


support  any delay for producer

FEATURE REQUEST

Please describe the feature you are requesting.
Now, rocketmq just support fixed delay message instead of any delay, which is not convenience for users. somtimes, we want to send message with any delay time: 1s, 2s and so on

Provide any additional detail on your proposed use case for this feature.

Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

If there are some sub-tasks using -[] for each subtask and create a corresponding issue to map to the sub task:

sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...

这个特性通过我通过在客户端加hook实现过，已经在生产环境运行了两年，想请教下你的实现方案
<EMAIL_ADDRESS>From: xujianhai666
Date: 2020-03-10 12:02
To: apache/rocketmq
CC: Subscribed
Subject: [apache/rocketmq] support any delay instead of fixed delay (#1832)
FEATURE REQUEST
Please describe the feature you are requesting.
Now, rocketmq just support fixed delay message instead of any delay, which is not convenience for users. somtimes, we want to send message with any delay time: 1s, 2s and so on
Provide any additional detail on your proposed use case for this feature.
Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?
If there are some sub-tasks using -[] for each subtask and create a corresponding issue to map to the sub task:
sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@xujianhai666 It is recommended to write a RIP first and then discuss it in the dev mailing list(dev@rocketmq.apache.org),  because it is a new feature that is widely concerned by the community

#1882 Please review


Event Request: Codeunit 225 Gen. Jnl.-Apply - OnBeforeRun

The event OnBeforeRun raised in Codeunit 225 Gen. Jnl.-Apply should implement the handled pattern to avoid standard code from being executed.

I'm working in the CU15 release and having problems avoiding the standard code in Codeunit 225
When will this "handled pattern" be implemented?


[lab] Fix import paths in generated declaration files

Rewrites import() types from material-ui/src and material-ui-styles/src to their corresponding bare imports specifiers (@material-ui/core and @material-ui/styles).
These are hardcoded at the moment to be careful with what we rewrite. I hope that this isn't something we have to maintain long-term.
Closes #24112

This is why we write tests: https://github.com/mui-org/material-ui/pull/24380/checks?check_run_id=1687093030

This is why we write tests: https://github.com/mui-org/material-ui/pull/24380/checks?check_run_id=1687093030

Going to merge even if we can't confirm the new paths are correct. The paths are definitely incorrect before the change so even if the change does not fix it, we don't make it worse (but code being code I wouldn't be surprised if it did make it worse).

Going to merge even if we can't confirm the new paths are correct. The paths are definitely incorrect before the change so even if the change does not fix it, we don't make it worse (but code being code I wouldn't be surprised if it did make it worse).


Hide search icon.

Hello,
Is there any way you can merge the hide_search: from this fork into yours?
https://github.com/natehome/kiosk-mode
It would be nice to be able to hide the entity search but still have the prospect of future support.

It would be nice. I have the same need ;)

This NEEDS to be done. You can nicely lock things down with this, but just pressing on the '?' brings up a list of all entities and totally negates whatever other security this adds.

Agreed regarding the search icon. Adding an option to hide the search icon would be greatly appreciated!

I will merge that.


Unexpected behavior by the JNC statement

MVI C, 00H
LXI H, 28E0H
MOV B, M
INX H
MOV A, M
ADD B
JNC 080DH
INR C
INX H
MOV M, A
INX H
MOV M, C
HLT
It's a simple adder which stores carry too. So, when I run it, there are two ways this program goes depending on the values to be added (at memory location 28E0H and 28E1H).
If the sum generates a carry, it works perfectly, but if it doesn't generate a carry, program doesn't run and says infinite loop detected. I tried to debug it by using the "Run one instruction" and it turns out JNC carry statement doesn't work like it should. That is, instead of setting PC register to 080DH, it sets it to 100DH.

Fixed.


Rebalanced uplink explosives

About the PR
So the last minibomb rebalance was a bit of a fiasco, and just made it worse than a grenade in every way. This makes it a utility explosive with a 10 second timer and very small, but very powerful radius. The spacing restriction was transferred to grenades. Useful for surgically destroying structures or bodies, but pretty useless against people with its 10 second timer.
Screenshots

New explosion radius. However, this will gib bodies (not living people) and instantly destroy substations. TC cost increased by 1 so you can only buy 2 with your starting TC to prevent too much egregious gibbing.
Changelog

:cl: Rane

tweak: Minibomb radius massively reduced.
tweak: Minibomb timer increased to 10 seconds.
add: Minibombs have a new, distinctive timer sound.
tweak: Minibomb will reliably destroy the tile under it, including gibbing dead bodies (but not living people.)
tweak: Grenades can no longer space tiles.
tweak: TC cost of uplink explosives has been adjusted based on their relative strength.

with C4 also being added minibombs are in an even weirder place, not sure how i feel

with C4 also being added minibombs are in an even weirder place, not sure how i feel

IMO from my time playing ss13, c4 isn't really supposed to be comparable to an grenade/minibomb unless you stacked like 3 of them together and even then you'd have to get the triggers together, even then it's not supposed to gib or space floors, which I think should be the minibomb's job.

yeah these are in a really weird state with c4 now tbh, maybe these are fine still as they can space areas and C4 cant? it feels like one of them should be removed though

I think c4 should take another hit to it's ss13 counterpart and do 50 damage to people instead of 100 to seperate it further from a standard grenade or require it to be placed to be activated and not usable in hand.

I think this would have decent use just for gibbing, if c4's gibbing ability is nerfed a bit. It's a decent amount of TC to gib 1 target imo

yeah we can see how this goes then


The EPSG code is reported as 0 altough for known coordinate system

Example: The EPSG code for coordinate system SWEREF99 TM is 3006, but
KnownCoordinateSystems.Projected.NationalGridsSweden.SWEREF99TM.EpsgCode returns 0 instead of 3006.
There are many other projections where the EPSG code returned is 0.

I see know that EpsgCode is obsolete and replaced with AuthorityCode but the problem is the same.

dup #729

At least for Swedish projections it is fixed, so I'm closing it. For others leave open #729

@pedwik I did some refactoring in Swedish projections initialization. Take a look into it. If you know mappings for other national grids, and have time to fix them - feel free to create PRs.


Add an option to inject user's public key

It'd be nice to be able to inject ~/.ssh/id_rsa.pub as a bind mount into ~/.ssh/authorized_keys inside the container based on some CLI flag.

On Wed, 7 Jul 2021 at 10:03, Michael Kashin @.***>
wrote:

It'd be nice to be able to inject ~/.ssh/id_rsa.pub as a bind mount into
~/.ssh/authorized_keys inside the container based on some CLI flag.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/srl-labs/containerlab/issues/497, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLKV5IKLLSPZUDRE23PAEDTWQC4PANCNFSM476DW3AA
.

yes, exactly.

sound idea
What about the following proposal:

the key --authz-keys can take a list of paths to public files. By default the single value of ~/.ssh/id_rsa.pub is assumed
the code will take those paths and create will add them to the defaults section of the Topology - https://github.com/srl-labs/containerlab/blob/ef66122378cd58905e7ff62d648b2fb0d0432371/types/topology.go#L14
with that, the rest of the code won't need to change, as those binds will be processed as usual

yeah, makes sense.

looking at the current code it looks like default values are ignired if any other binds are specified. Is this by design?
func (t *Topology) GetNodeBinds(name string) []string {
	if ndef, ok := t.Nodes[name]; ok {
		if len(ndef.GetBinds()) > 0 {
			return ndef.GetBinds()
		}
		if len(t.GetKind(t.GetNodeKind(name)).GetBinds()) > 0 {
			return t.GetKind(t.GetNodeKind(name)).GetBinds()
		}
		return t.GetDefaults().GetBinds()
	}
	return nil
}

that looks like an error, it needs to be an inheritance chain from defaults->kind->node, with node being most specific
I think as a first iteration we can just append the values from defaults and kinds to the nodes.

I think it was intended to be like this, even in earlier code.

initially we only added support for binds under either kinds or nodes - https://github.com/srl-labs/containerlab/pull/124
even defaults were not considered to have binds
but now I do not see why we wont just append all the binds all the way from defaults to nodes

there's also an issue with sudo. You'd need to do sudo -E in order to preserve users' home directory (unless you're root in the first place). Does anyone know a better way? I tend to run my linux as a non-privileged user.

Does "sudo -E containerlab --authz-keys ~/.ssh/id_rsa.pub" work? Will the
homedir function we use inside clab to parse the homedir correctly?
If yes, what do you want to shorten there?
Maybe sudoers file can somehow implement the preservation of users home dir
On Fri, 9 Jul 2021 at 18:04, Michael Kashin @.***>
wrote:

there's also an issue with sudo. You'd need to do sudo -E in order to
preserve users' home directory (unless you're root in the first place).
Does anyone know a better way? I tend to run my linux as a non-privileged
user.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/srl-labs/containerlab/issues/497#issuecomment-877292058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLKV5KYN6O7IXCSJPGRMSLTW4M2PANCNFSM476DW3AA
.

yes, that command will work, but the flag is easy to forget, hence why I asked if there are any alternatives
modifying sudoers will also work although I dunno how good it is to suggest that -- it effectively means that homedir for sudo will always point to users' homedir which may break other things.

In my view that is ok to assume that people will resort to reading docs in
case of failures. In the flag docs we will mention that the sudo users
should either use -E flag with sudo command or they have to provide the abs
path of the private key
On Fri, 9 Jul 2021 at 18:38, Michael Kashin @.***>
wrote:

yes, that command will work, but the flag is easy to forget, hence why I
asked if there are any alternatives
modifying sudoers will also work although I dunno how good it is to
suggest that -- it effectively means that homedir for sudo will always
point to users' homedir which may break other things.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/srl-labs/containerlab/issues/497#issuecomment-877313658,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLKV5PFL3HW57A5WCBQSQDTW4QZ7ANCNFSM476DW3AA
.

With sudo containerlab --authz-keys ~/.ssh/id_rsa.pub, the ~ is parsed by the shell. Containerlab will see the resolved path, so this is not an issue.

I think the problem is that it'll get resolved to /root and not /home/user where my key is likely to be.

No, your shell is running with your identity and HOME=/home/user. Your shell will expand ~ to /home/user, then provide the arguments to sudo.

I am interested in doing that. I was about to implement it in vrnetlab, but maybe this is just simpler to let containerlab do that. Just to be sure, my plan was to add the option and implement it for XRv9k and Juniper MX kinds by implementing a AddSSHKey() to the node and invoke it if it exists. I am a bit unsure if it is the right approach in case the OS is containarized. Can we also rely on SSH in this case? It's just that bindmount should be defined earlier, but maybe this is not incompatible: for containers, AddSSHKey() can stay undefined and the --authz-keys parameter can be used earlier by appending to bind mounts.

yeah, you're right. I must have dreamt it

Hi @vincentbernat
how did you plan to inject the key into the xrv and vmx?
it seems that you would need to inject the public key into the configuration of those network systems, and the mechanism to do that is custom per each node

for vrnetlab based routers though, it might be more feasible to implemented the code in the launch.py that is part of the vrnetlab image, because then we know when the node is booted and can provision the key

I was planning to use SSH and issue the appropriate commands. And same for injecting an initial configuration file. Doing it inside launch.py was my initial option. If you think this is better this way, I'll go this way.

In #716 I created a c.Dir.Lab/authorized_keys file that containers pub keys from all the files found at ~/.ssh dir
this file can be then mounted to the relevant path inside the particular node if needed. Example - https://github.com/srl-labs/containerlab/pull/716/files#diff-84e2ae483c1b7bb1fe7b4e080f06f3b468a9ff01968d90da1747b0ce2d8acc84R159

SR Linux node kind has been updated to mount the generated authorized_keys file
other nodes will be done upon a request


[BUG]

Before you begin...

[X] I have searched the existing issues
[X] I am not using version 13.x of node  (if so, please upgrade)

Description of the problem
in last version
i give this error on building.
node_modules/uuid/dist/cjs/index.d.ts:1:1 - error TS1383: Only named exports may use 'export type'.
1 export type * from './types.js';

in version:11.0.2  work without problem. but in last version is not work
Recipe for reproducing
No response
Additional information
No response
Environment
No response

Duplicate #837


[release-4.13] OCPBUGS-27342: Suggest workload partitioning annotation in the namespace

/jira refresh


react and react-dom throw error TS1316 "Global module exports may only appear at top level"

Prerequisites

[ x ] Did you check the FAQ?
[ x ] Did you search open and closed issues to see if your issue has already been reported?

Description
When making the command typings install dt~react --global --save it says it is an external module, and I should omit the --global. When I do that ( typings install dt~react --save )it is happy and I can continue. Same for the react-dom typings. However on compilation I get the error TS1316 'Global module exports may only appear at top level'.
Currently it's not a showstopper and I can code (with syntax highlighting etc) and compile my react & react-dom code correctly. It is however strange behaviour I don't understand.
E:/Dev/electron-starter/typings/modules/react-dom/index.d.ts(10,1): error TS1316: Global module exports may only appear at top level.
E:/Dev/electron-starter/typings/modules/react/index.d.ts(11,1): error TS1316: Global module exports may only appear at top level.

Steps to Reproduce
Installation

Try typings install dt~react --global --save and see the error popup
Try 'typings install dt~react --save' and see that 'typings' is now happy

Git repo

Go to the git repo I have setup to reproduce (https://github.com/aredfox/electron-starter).
Do an npm install first
Type gulp js:typescript and you should get this output (minus the path locations ofcourse).

> gulp js:typescript
[08:47:27] Using gulpfile E:\Dev\electron-starter\gulpfile.js
[08:47:27] Starting 'js:typescript'...
E:/Dev/electron-starter/typings/modules/react-dom/index.d.ts(10,1): error TS1316: Global module exports may only appear at top level.
E:/Dev/electron-starter/typings/modules/react/index.d.ts(11,1): error TS1316: Global module exports may only appear at top level.
[08:47:30] TypeScript: 2 semantic errors
[08:47:30] TypeScript: emit succeeded (with errors)
[08:47:30] Finished 'js:typescript' after 3.17 s

Expected behavior:
react and react-dom to be treated as external modules, or vice versa. not it us just confusing and I first thought there was an error in my code.
Actual behavior:
react and react-dom cannot be installed as global modules via typings, and have to be installed as external modules - yet the compiler then complains with error TS1316.
Versions

Typings 2.1.0
tsc: <IP_ADDRESS>
typescript npm module: 2.1.5
gulp-typescript npm module: 3.1.4
react and react-dom npm modules: 15.4.2
tsconfig.json https://github.com/aredfox/electron-starter/blob/master/tsconfig.json
typings.json https://github.com/aredfox/electron-starter/blob/master/typings.json

{
  "globalDependencies": {
    "electron": "registry:dt/electron#1.4.8+20170104211658",
    "node": "registry:dt/node#7.0.0+20170124134802"
  },
  "dependencies": {
    "react": "registry:dt/react#15.0.0+20170125182642",
    "react-dom": "registry:dt/react-dom#0.14.0+20170123203653"
  }
}

References:
https://github.com/DefinitelyTyped/DefinitelyTyped/issues/14313

For DT typings, it is best to install through npm i @types/<package> as the format had changed.
See https://github.com/typings/typings/pull/756 for detail

thanks @unional - I now have an empty typings.json, I presume that is correct?
I couldn't find out from #756 or the typings README section whether when I installed my types now as (eg) npm i @types/react it resolved automatically now?
Both typings.json and my folder where I point to my types, typings/index.d.ts are empty..

Never mind, the typescript handbook on consumption told me this now: http://www.typescriptlang.org/docs/handbook/declaration-files/consumption.html


Fixed proximity zone incorrectly using name instead of zone setting.

Description:
Fixed proximity zone incorrectly using name instead of zone setting.
Related issue (if applicable):
Discussed in forum: https://community.home-assistant.io/t/proximity-with-multiple-users/6411

Hi @jjmontesl,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Since the tests are failing, it seems to me that this was intentional.

Thanks for your review. It's indeed weird.
I have pushed a change that fixes the tests. I cannot however understand why these changes are needed... it seems to me that the component should be taking the same zone name ('home') as a default  (as seems to be in https://github.com/jjmontesl/home-assistant/blob/0357cb02b0d2320b55d5154fc5965281b66f6ff4/homeassistant/components/proximity.py#L42 ).
Any idea about why the default is not working?
Regarding the fix, still seems to me that 'zone' shall work as per the proposed patch, since that is what the documentation seems to suggest (https://home-assistant.io/components/proximity/):

zone (Optional): The zone to which this component is measuring the distance to. Default is the home zone

I can imagine this may be a breaking changing for those relying on the friendly name for zone matching, but that behavior seems like a glitch? Any suggestion is welcome. Understanding why the default 'home' zone setting is not working would be a good start.

Looked at the code, your code makes sense 🐬

btw, you probably don't want to use proximity but use our template helpers distance and closest instead: https://home-assistant.io/topics/templating/#home-assistant-template-extensions


Add PSP Removal banners

Fixes #6873

add deprecation banner for PSPs on cluster dashboard page to be displayed only if the user has any PSPs and if kubernetes version is 1.21 or higher

Screenshot

Test

Create a PodSecurityPolicy
Make sure cluster is running kubernetes version 1.21 or higher
Check if warning Banner appears on the cluster dashboard page

https://github.com/rancher/dashboard/issues/6873

@catherineluse @nwmac thanks for all the feedback.
I have updated the PR according to you feedback + #6943  updated requirements.

I have your PR checked out locally, but the banner isn't showing for me. I have this downstream cluster with several PSPs:

But when I go to the overview page for the cluster, the banner is not shown:

I have your PR checked out locally, but the banner isn't showing for me. I have this downstream cluster with several PSPs:

But when I go to the overview page for the cluster, the banner is not shown. I tried refreshing on cluster explorer, going to the PSPs list view and navigating back, going to the local cluster and navigating back:

So, couple of problems here (sorry about it @catherineluse ):

major and minor come as strings not int's so I wasn't parsing them correctly (don't know why it was working on my system 🤷 when I tested it )
wasn't using kube API to fetch the policies, so the request v1/policy.podsecuritypolicies was giving me the PSPs for the local cluster and not for the downstream cluster we are at. I have fixed that now.

I think it's all good to go. Will ask for another review from your side. Thank you for your time and sorry about my mess 🙏


parse error 'import' and 'export' may appear only with 'sourceType: module' (1:0)

Error in plugin 'gulp-ng-annotate'
Message:
visitor\carousel\carousel.js: error: couldn't process source due to parse error
'import' and 'export' may appear only with 'sourceType: module' (1:0)

ng-annotate can't parse your source. I guess you need to transpile it.


Clean and improve the CyHy asset sync script and ASM data collections

🐛 Summary
Expanded to include these tasks:

[x] Clean CIDRs/IPs by adding last_seen flags
[x] Clean domains by adding last_seen flags
[x] Add the CyHy sync script to the pe-reports module
[x] Check and flag if IPs are multi-tenant
[x] Make sure database queries are updated to account for all new flags

Motivation
As we are adding more ASM functionality we need to refine our process and ensure we are in sync with any stakeholder updates.

Needs PR @aloftus23 pe_asm


Invisible spirits + scared cats

This is two suggestions since the second relies on the first to make much sense.
Will there ever be a system to make the spirits invisible on the first plane, requiring goggles or even more powerful items to see more powerful spirits? Or possibly disguise as other mobs?
If this does get added in the future it would also be interesting if cats/ocelots ran away from spirits even when they're invisible/disguised, giving a way for players to detect them without proper equipment.
I don't expect this to get added soon since its not super useful, but would be neat since it would be more like the books. It could be useful for future content, if bodyguards were added for example.

While this is a fun idea, it's currently not planned - I did toy with the idea a bit when implementing the otherworld blocks and goggles, but it would require quite a lot of time to implement for a feature that would be mainly visual in nature, so I decided against it.


Routes: Http Server via Tapir

Hi! Being pretty impressed with kyo's features i tried to migrate one of my application from ZIO to KYO. Since it is a tapir-based backend the first thing i needed was to implement some kind of tapir-kyo integration module. Result of my attempt can be found here. sttp's design relies on MonadError, so, I decided to bound my effect with Aborts. Here came the first problem - Flat's: it's impossible to add an extra bound to an arbitrary typeclass method. For the experiment I ended up with Flat.unsafe.unchecked. On the next step I finally stuck with another problem - Tag's, which again couldn't be provided.
Am I mistaken in the way I designed my instances? While it's not entirely necessary - I agree we can get rid of tapir and build an application without cats/scalaz/whatever, I think it's good to have a way to live side-by-side.
Anyway, thanks for the fantastic effort on KYO, Quill and all other brilliant libraries!

Thanks for the feedback! It's odd that I haven't seen bug reports yet, did you encounter any?
I'm also in need of an HTTP server for Kyo. Did you see the initial integration in the repo? It uses this monad definition from the sttp module but it also relies on Flat.unsafe for error handling.
I'll take a look if it'd be possible to avoid the flat check for error handling (Tries.run). It's a common operation and ends up making the check appear in several unexpected places.

Thanks @fwbrasil, I definitely missed that code 😅 I will try to adopt it. As for bugs - I didn't encounter something yet, I'll report back after minimal valuable rewrite

@danslapman I used Kyo for a performance benchmark challenge and finalized the initial version of the Tapir integration for it. I've submitted a few optimizations to Tapir as well. The documentation for the module is still missing but you can take a look at the usage here: https://github.com/fwbrasil/rinha-2024-q1/tree/main/src/main/scala/rinha

Closing in favor of https://github.com/getkyo/kyo/issues/219


Hackium cli needs a quiet mode and a way to disable the REPL

Hackium scripts that output data via console.log() can not output cleanly because hackium outputs at least one Info log, the return value of the hackium script, and the REPL prompt. The REPL also can't be easily closed programmatically and there should be an easier way to close the browser automatically.
sample script
test.js:
console.log(JSON.stringify({ test: "data" }));

Command
$ hackium -e test.js

Actual
Browser opens and stays open. REPL instantiates after script finishes and stays open.
Output:
{"test":"data"}
undefined
Info: Hackium launched
>

Expected
Browser closes automatically, REPL never opens (or closes, silently, on its own)
Output:
{"test":"data"}

The REPL is good for troubleshooting and first-time users and may not be best as a default. It may make more sense to have an '--interactive' flag/option that turns on the REPL when running via CLI. For backwards compatibility maybe -e should turn interactive on by default and another flag should exist for dev-centric usage.


Remove usage of Ember.evented.

Ember.evented is no longer considered best practice in Ember.

You'll need to restart the FastBoot job.
I'm not sure why canary is broken, but it's unrelated to this change.


fix(galaxy): whitespace

Some script (ESLint probably) touches this newline character at the end of the package.json, let’s just commit this change.

Oops, empty commit? 😄


Import Crash Loop

Godot version
4.0
System information
Windows 10
Issue description
Attempted to drag a bunch of files to my FileSystem, unaware I was importing an FBX file; this prompted a window to download FPX2glTF. Clicking cancel will begin the import process in which it has crashed every time. I imported multiple types of 3D file formats, and it seems to crash at random. Closing the program and opening it immediately starts the prompt over again, thus unable to get out of the crash loop.
Steps to reproduce
Drag and drop multiple folders. Seems to be fine when one folder is imported but crashes on multiple.
Minimal reproduction project
N/A

Duplicate of #73319


Address point monthly update (2 of 3)

Sprint 3

Start updating some counties

Sprint 4

complete the November update

completing today

Done


Adding option to skip Load_Platform

Adding an option to compose init to pass "--without-load" that prevents the Load_Platform job from being called. This is useful if you want to override the URL it will use for adop-platform-management, such as during development where a fork is preferred.

@nickdgriffin : I have tested the functionality through cli and it works. could you please resolve the conflicts?

@SachinKSingh28 Rebased it, can you give it another test please?

Looks good!


Cow: Watch justsnap.sg

Cow requests the watch of the watch_keyword justsnap\.sg. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

justsnap\.sg has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Cody Gray in Charcoal HQ


Error when running reliably scan terraform with the ./tests/terraform directory

Describe the bug
Using a local build on the main branch, if you run a terraform scan on the /tests/terraform directory you get a WARN output that shows an error has occurred
To Reproduce
./bin/reliably scan -v terraform ./tests/terraform
The output generated is:

if you run with the verbose command output is:

The WARN[0000] An error occurred while scanning resource: [aws_launch_template] is misleading as its labelled with WARN, then indicates an error has occurred, also there is not enough information on the error.
Expected behavior
The output should be consistent between WARN and error and the error message should provide information about the error.
Additional context
Note: This is running on a local build of the main branch of the CLI

Agreed @gtfisher
This error in particular is for a resource that simply has no policy. This is likely to happen often going forward. So something like:  WARN: no existing policy for resource: [some_resource_name] may be better

Yes, that would work, I get similar with scan live from my minkube cluster:


[python] Add multiple version of document

Using sphinx-multiversion to support multiple version
of python api document
close: #8227

Codecov Report

Merging #11391 (ccead60) into dev (0464123) will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #11391      +/-   ##
============================================
+ Coverage     39.44%   39.48%   +0.03%
- Complexity     4608     4615       +7
============================================
  Files           980      980
  Lines         37149    37179      +30
  Branches       4142     4148       +6
============================================
+ Hits          14655    14679      +24
+ Misses        20982    20980       -2
- Partials       1512     1520       +8

Impacted Files
Coverage Δ

...erver/master/processor/queue/TaskEventService.java
69.64% <0.00%> (-10.72%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
55.12% <0.00%> (-1.29%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.29% <0.00%> (-0.57%)
:arrow_down:

...ler/plugin/task/api/enums/TaskExecutionStatus.java
59.52% <0.00%> (-0.48%)
:arrow_down:

.../server/master/runner/WorkflowExecuteRunnable.java
7.88% <0.00%> (-0.02%)
:arrow_down:

...inscheduler/server/master/metrics/TaskMetrics.java
77.14% <0.00%> (ø)

...eduler/api/service/impl/DataSourceServiceImpl.java
28.47% <0.00%> (ø)

...cheduler/server/master/utils/DependentExecute.java
0.00% <0.00%> (ø)

... and 14 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

ref: https://github.com/apache/dolphinscheduler-website/pull/822


Lambda functions in Symbolic_KANLayer attributes are not serializable using pickle  (torch.save)

Hello,
The new Symbolic_KANLayer has lambda functions as attributes. This makes it hard to save model checkpoints. Default model saving in torch uses pickle, and lambda functions are not serializable using pickle unless one uses a serialization package like dill (please see this  for a concrete example).
I don't know if the Symbolic_KANLayer will be used in the future for large-scale experiments. If so I think it is advisable to get rid of all the attributes that are lambda functions in Symbolic_KANLayer, to avoid by construction that kind of problems.
For referencing, the error was:
AttributeError: Can't pickle local object 'Symbolic_KANLayer.__init__.<locals>.<lambda>'. Did you mean: '_return_value'?
See also: AttributeError: Can't pickle local object <locals>.<lambda>

Minimal example to reproduce this behavior:
import torch
import kan

torch.save(kan.KAN([5,5]), 'test.pt')

This last gives the error: AttributeError: Can't pickle local object 'Symbolic_KANLayer.__init__.<locals>.zero_fun'
A work around using dill package:
import torch
import kan
import dill

torch.save(kan.KAN([5,5]), 'test.npz', pickle_module=dill)

This last works.


Switch from invididual invariant arguments to an "Invariants" class

Re: advice from Fabrice

Just wondering, are you going to include the other 3 invariants here?
(From Design Doc)

No courses will overlap in a schedule

The total number of credits contained in a schedule must be within the min/max bound of credits (specified by the user)

If a course is marked as required, it must be in the schedule.

There can’t be multiple sections of the same course in the same schedule

Seems like you check invariant 4 in Section.java, will you check for the other invariants here?

No. This serves as a capsule for invariants that might change (which for right now is the min/max number of credits). So it's more of a wrapper for actual data as opposed to different logical implementations. The ideas of no course overlap, same section overlap, and required courses are more changes in the logic of the generateSchedules() function in the scheduler class.

Ok sounds good. I understand what you mean now.


"Header too Large"

Only happens with the bgcombine workflow, for some reason it gives Header Too Large errors. I tried both models, different image sizes, resize settings, checkpoint models, redownloaded the model. Always get this error. The rest of the workflows work fine!
Traceback (most recent call last):   File "/opt/rd/apps/ComfyUI/execution.py", line 151, in recursive_execute     output_data, output_ui = get_output_data(obj, input_data_all)   File "/opt/rd/apps/ComfyUI/execution.py", line 81, in get_output_data     return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)   File "/opt/rd/apps/ComfyUI/execution.py", line 74, in map_node_over_list     results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))   File "/opt/rd/apps/ComfyUI/nodes.py", line 814, in load_unet     model = comfy.sd.load_unet(unet_path)   File "/opt/rd/apps/ComfyUI/comfy/sd.py", line 569, in load_unet     sd = comfy.utils.load_torch_file(unet_path)   File "/opt/rd/apps/ComfyUI/comfy/utils.py", line 14, in load_torch_file     sd = safetensors.torch.load_file(ckpt, device=device.type)   File "/opt/rd/miniconda3/envs/CMFY/lib/python3.10/site-packages/safetensors/torch.py", line 308, in load_file     with safe_open(filename, framework="pt", device=device) as f: safetensors_rust.SafetensorError: Error while deserializing header: HeaderTooLarge

Solved by redownloading model again. :)


DEVX-6617: updating JWT dependency

Updates gemspec to change JWT dependency from nexmo-jwt to vonage-jwt

Codecov Report

Merging #250 (05e6864) into dev (099e925) will decrease coverage by 0.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #250      +/-   ##
==========================================
- Coverage   97.73%   97.28%   -0.46%
==========================================
  Files          71       71
  Lines        1457     1435      -22
==========================================
- Hits         1424     1396      -28
- Misses         33       39       +6

Impacted Files
Coverage Δ

lib/vonage/jwt.rb
33.33% <0.00%> (-66.67%)
:arrow_down:

lib/vonage/files.rb
76.92% <0.00%> (-23.08%)
:arrow_down:

lib/vonage/voice/ncco.rb
90.90% <0.00%> (-0.76%)
:arrow_down:

lib/vonage/voice/actions/notify.rb
96.15% <0.00%> (-0.15%)
:arrow_down:

lib/vonage/voice/actions/record.rb
97.14% <0.00%> (-0.08%)
:arrow_down:

lib/vonage/voice/actions/stream.rb
97.14% <0.00%> (-0.08%)
:arrow_down:

lib/vonage/voice/actions/talk.rb
97.36% <0.00%> (-0.07%)
:arrow_down:

lib/vonage/voice/actions/conversation.rb
96.49% <0.00%> (-0.07%)
:arrow_down:

lib/vonage/voice/actions/input.rb
96.66% <0.00%> (-0.06%)
:arrow_down:

lib/vonage/voice/actions/pay.rb
98.21% <0.00%> (-0.04%)
:arrow_down:

... and 6 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Should be branching off dev not main


Inclusive ranges

\n should probably be considered as part of the current line. Only the next character should advance the line number.
.node should be inclusive of the end, not exclusive. Will need some new kind of combinator like lastIndex, if possible

https://github.com/jneen/parsimmon/issues/331

\n should probably be considered as part of the current line. Only the next character should advance the line number.

I'm expecting this to happen!

.node should be inclusive of the end, not exclusive. Will need some new kind of combinator like lastIndex, if possible

Just to reiterate what was said in the issue referenced above, I still don't think inclusive range is a good idea (almost all Range implementations have exclusive end index).

If you want to work on a PR feel free. I haven't been actively developing bnb for a while, but it is 0.x so I can release breaking changes if need be lol

I tried to create a PR but then I realized bnb does not have the same behavior as Parsimmon. bnb handles \n exactly as I wanted, line number does not increment until after it.
The same old example:
// parsimmon:
const parser = P.seqMap(P.index, P.digits, P.index, function (start, value, end) {
  console.log(start, value, end);
  return value;
});

parser.parse('1234\n');
// => {offset: 0, line: 1, column: 1} '1234' {offset: 4, line: 2, column: 0}

// bnb:
const parser = bnb.all(bnb.match(/\d+/).node('num'), bnb.text('\n').node('newline'));
console.log(parser.parse('1234\n').value[0]);
// =>{"type":"ParseNode","name":"num","value":"1234","start":{"index":0,"line":1,"column":1},"end":{"index":4,"line":1,"column":5}}

So apparently this issue does not exist in bnb at all.

Turns out I don't even remember how my own library works lol. Well that's great! I suppose I'll close the issue then.
Internal bnb code to handle line/col info:
https://github.com/wavebeem/bread-n-butter/blob/main/src/bread-n-butter.ts#L455-L472


Cannot find module node_modules\vidstack\plugins.js

Current Behavior:
I have followed the guide here to bundle Vidstack:
I https://www.vidstack.io/docs/player/getting-started/installation/vue?bundler=nuxt&provider=youtube&styling=default-layout
Furthermore we are using Nuxt v3.9.2 and Vue v3.3.8
However when running npm run dev I get the following error:

In node_modules\vidstack\plugins.d.ts it has:
import * as _esbuild from 'esbuild';
import * as _rollup from 'rollup';
import * as _vite from 'vite';
import * as _unplugin from 'unplugin';

type FilterPattern = Array<string | RegExp> | string | RegExp | null;
interface UserOptions {
    include?: FilterPattern;
    exclude?: FilterPattern;
}
declare const unplugin: _unplugin.UnpluginInstance<UserOptions | undefined, boolean>;
declare const vite: (options?: UserOptions) => _vite.Plugin<any> | _vite.Plugin<any>[];
declare const rollup: (options?: UserOptions) => _rollup.Plugin<any> | _rollup.Plugin<any>[];
declare const webpack: (options?: UserOptions) => WebpackPluginInstance;
declare const rspack: (options?: UserOptions) => RspackPluginInstance;
declare const esbuild: (options?: UserOptions) => _esbuild.Plugin;

export { type FilterPattern, type UserOptions, esbuild, rollup, rspack, unplugin, vite, webpack };

Expected Behavior:
To be able to bundle Vidstack with Nuxt and Vite.
Environment:
Example:

Framework: Vue
Meta Framework: Nuxt.js
Node: 20.11.0

Anything Else?

What version are you on? It was included in 1.10.3@next because I forgot to include it package files.

Sorry for not mentioning, it's on v1.10.1
I see there the newly released v1.10.2
Upgraded to latest release and it builds now 🎉🥳
I don't see the custom element rendered though.
<media-player title="Sprite Fight" src="https://stream.mux.com/VZtzUzGRv02OhRnZCxcNg49OilvolTqdnFLEqBsTwaxU/low.mp4">
    <media-provider></media-provider>
</media-player>

I tried configuring the isCustomElement here in nuxt.config.ts:

and in component:
<script lang="ts" setup>

import 'vidstack/bundle';

</script>

<template>
  <div class="video">
    <media-player title="Sprite Fight" src="https://stream.mux.com/VZtzUzGRv02OhRnZCxcNg49OilvolTqdnFLEqBsTwaxU/low.mp4">
      <media-provider></media-provider>
    </media-player>
    Video
  </div>
</template>

anything missing?

You have no layout? You can use either the Default Layout, Plyr Layout, or build your own :)

Default Layout needs to be selected, not Default Theme. The Default Theme is for building your own layout using our theme. I need to add more information to the styling step to help with deciding between the options. Sorry about the confusion.

Thanks, I see the difference now at "11. Player Markup".
I was also wondering why I couldn't select both as I think "layout" is how/where the elements are rendered in DOM and "theme" how the elements are styled. Some plugins use theme/skin how the component is styling, but often only a single layout (or possible to configure via some options).


Added Azure.MariaDB.MinTLS

PR Summary
Fixes #1853
Added Azure.MariaDB.MinTLS rule.
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Rule changes

[x] Unit tests created/ updated
[x] Rule documentation created/ updated
[x] Link to a filed issue
[x] Change log has been updated with change under unreleased section

Other code changes

[ ] Unit tests created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

@BernieWhite
Should be ready for review. Looks like we have an issue in the CI?

/azp run

@BernieWhite
Removed the customer reason now and updated test. Much better.

@BernieWhite
CLA signed.


Installing on android (MainActivity.kt)

Hi @srfaytkn,
I ran through your setup and have it working in iOS but the android setup doesn't seem to work. Current versions of react-native (I'm using 0.60.5) create a MainActivity.kt file instead of a MainApplication.java, how would I link this in Kotlin versus your instructions in Java? Thanks in advance!
Ben

@srfaytkn

Hi @srfaytkn, I've searched for a solution but I'm still not finding a way to make this work. Can you please comment and update the readme for the kotlin install steps?

I think @srfaytkn don’t have enough time for this.

@srfaytkn ping! Can you at least respond?
@farad4y thanks for trying to direct this to the right person :)

@srfaytkn Olum su adama bir cevap yaz, aci cekiyor belli ki.

@jbburf MainApplication.kt should be like this
class MainApplication : NavigationApplication() {     protected val packages: List<ReactPackage>         get() {             val packages = PackageList(this).packages             packages.add(RNSpotifyPackage())             packages.add(YouTubeSdkPackage())             return packages         }      ... }


always run in transact() mode

I find that I use transact all over the place, and I "thought" I was doing it correctly.
store.something.transact();
store.something.set('a',1);
store.something.set('b',2);

That has been working great ... but not I updated freezerjs to the latest and am getting detach warnings all over.
Looks like I really should be doing ...
date = store.something.transact();
data.set('a',1);
data.set('b',2);

So I actually have 2 questions ...

the way I have been doing seems to work fine, what are the issues with doing it the first way above.

is there a way to put the store into transact() mode globally, to make things a bit cleaner for my scenario.

Hey aktxyz
If you want to update 2 elements in something you can do
store.something.set({a:1,b:2});

Also the update methods are chainable so this will do the same, but it's less efficient since it updates the tree completely twice:
store.something
  .set(a, 1)
  .set(b, 2)
;

The first set(a,1) return the updated node to let you edit it again.
I think that transaction is a last resort mechanism, and I have found myself using it only twice in the years I've been using Freezer. You need to update the whole tree hundreds of times in the same tick to feel that there is a performance object, so if I am not doing hundreds of updates I never use it.

I use the 2 methods you describe quite a bit also, but there are cases where the code for the store updates are not all together, so end up having to .transact() to work.
If I do a .transact() at the root, will that apply to all children?

I usually organize the functions that update the state by the part of the state they update. Eg, for I would create a file, somethingReactions.js to edit what's inside of something.
In that file I use to create a helper function to make simpler run the updates something like:
// 'd' after 'domain'
var d = function (){ return freezer.get().something };

That way I rest assured that I am updating the current content of the store:
d().set('a',1);
d().set('b',2);

I think transact is not a good thing to use so lightly, because it works stopping the tree updates on that node and if I don't call run immediately after using it, I feel like I am loosing the control of what's happening. I might be doing some other updates to the store before the transaction run, without knowing that there is a transaction going on, maybe in the same store branch I am working.
That's the reason that I never use it, unless I have to make lots of updates in one node's children, and I always call run afterwards.
By the way, I am writing an article to celebrate the 1000 stars, where I try to explain how I use freezer :)
If you want to preview and share your thoughts I'll be so grateful:
https://medium.com/@arqex/7-tips-using-react-with-freezer-f4d75e363b94

great idea on the article, I will read it and let you know


Cleaner guide for local installation

Describe the bug
Right now there are several points in the readme that are not up to date and simply confusing.

This should also contain an explanation of the MongoDB setup.


Add install targets

This PR adds install targets to the CMakeLists.txt so the package can be installed properly. It also adds an exec_depend on xacro and bumps the package version to 0.0.1.

Thanks for the quick response :)

Oops, I misspelled "bump" :(

Bummer ;)


controller: Fix event ordering assumption in TestStreamReleaseEvents

We don't guarantee that events will be delivered in a specific order.

LGTM