vimPlugins.nvim-treesitter: handle duplicate grammars

Description of changes

In https://github.com/NixOS/nixpkgs/pull/319233, a new logic for colliding grammars didn't expect any duplicates, and if there were any, it raised a weird error: ln: failed to create symbolic link '/nix/store/...-vimplugin-treesitter-grammars/parser/lua.so': File exists. Previously, duplicates were silently ignored (though unsure, didn't investigate a lot).
In this PR I added a filter function, which should remove obvious duplicates (with the same commit). Though if two identical grammars with different revisions were added, it still raises ln: failed to create symbolic link ...: File exists (if you know how to improve the error, please let me know). The filtering logic itself was heavily inspired by lib.lists.unique
As this seems to be a common misunderstanding, error on duplicate grammar but different revs is a feature, not a bug (see https://github.com/NixOS/nixpkgs/pull/339076#issuecomment-2329585080).
See also https://github.com/NotAShelf/nvf/issues/368
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

CC @BirdeeHub, @stephen-huan, @FrothyMarrow (you provided nice feedback in https://github.com/NotAShelf/nvf/issues/368, so it would be good if you helped us reviewing this)

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

@PerchunPak thanks for opening the PR and apologies for my carelessness breaking people's setups!
The PR looks good, was pretty much exactly what I had in mind as well. Do you know what would happen before https://github.com/NixOS/nixpkgs/pull/319233 if multiple grammars with the same name were included? I'm assuming it would take the last grammar in the list (the last one to overwrite with cp) so we can get the same semantics with ln -sf, and people seemed to be fine with this.

I think ln -sf would be best because then it would be guaranteed to filter ALL cases where the collision would have occured, as hacky as it feels.

I don't think so, yes it filters all cases, but it actually just silently fails (who knows which grammar it will choose?). I think norg overlay should use grammar from nixpkgs or somehow remove nixpkgs' norg grammar. Will explore solutions tomorrow

I agree that we can do better than silently overwriting. I don't personally use alternative tree-sitter grammars, and don't have much of an opinion so I'll defer to @BirdeeHub who seems to know more about this. My personal opinion is that builds failing seem to be more annoying than using the wrong grammar, and if one is so picky about tree-sitter grammars then one can manage the plugins outside of the nvim-treesitter.withAllGrammars mechanism.

@PerchunPak thanks for opening the PR and apologies for my carelessness breaking people's setups!
The PR looks good, was pretty much exactly what I had in mind as well. Do you know what would happen before #319233 if multiple grammars with the same name were included? I'm assuming it would take the last grammar in the list (the last one to overwrite with cp) so we can get the same semantics with ln -sf, and people seemed to be fine with this.

I think ln -sf would be best because then it would be guaranteed to filter ALL cases where the collision would have occured, as hacky as it feels.

I don't think so, yes it filters all cases, but it actually just silently fails (who knows which grammar it will choose?). I think norg overlay should use grammar from nixpkgs or somehow remove nixpkgs' norg grammar. Will explore solutions tomorrow

I agree that we can do better than silently overwriting. I don't personally use alternative tree-sitter grammars, and don't have much of an opinion so I'll defer to @BirdeeHub who seems to know more about this. My personal opinion is that builds failing seem to be more annoying than using the wrong grammar, and if one is so picky about tree-sitter grammars then one can manage the plugins outside of the nvim-treesitter.withAllGrammars mechanism.

In theory, yes ln -sf should just overwrite it with the last one in the list.
This PR does the same thing, technically it could miss filtering stuff and still get an error, but with lib.getName it shouldnt miss any.
It kinda doesnt matter a ton.

I see one issue: Grammars are being filtered based on the meta information, but during the copy process, this meta information is dropped. This would create the same issue when different revisions of the same grammar exist.
As mentioned in https://github.com/NotAShelf/nvf/issues/368, the revision info can be appended to the grammar filename since treesitter only looks for filetype at the start of the filename, ignoring the rest.
I doubt anyone is including different versions of the same grammar, but for sake of correctness we should handle that edge case.

I see one issue: Grammars are being filtered based on the meta information, but during the copy process, this meta information is dropped. This would create the same issue when different revisions of the same grammar exist.
As mentioned in NotAShelf/nvf#368, the revision info can be appended to the grammar filename since treesitter only looks for filetype at the start of the filename, ignoring the rest.
I doubt anyone is including different versions of the same grammar, but for sake of correctness we should handle that edge case.

thats why I suggested lib.getName, outside of that the other option is ln -sf

Using ln -sf or lib.getName is not really acceptable to me. Sure, this will produce the same result, as before #319233, BUT this is undefined behavior. I have no idea what treesitter used to choose on duplicate grammars, but I would prefer explicit exclusion of grammars.
Ideally, the implementation just raises a human-friendly error on duplicate grammars (but different revs) and adds an option to exclude grammars.
But unfortunately, I don't really have time for this right now, feel free to propose your ideas, because I don't have any idea how to implement such exclusion mechanism. As this issue is blocking for many people, I opened a new PR with a workaround for now (ln -sf).
P.S. lib.getName shouldn't be used, as this filtering is pretty costly (O(n^2))

I added an explicit exclusion feature, a proper warning on duplicate grammars, and some other fixes. If you have any idea how to make the code less messy - let me know!

nix build --show-trace .
warning: Git tree '/home/birdee/birdeeSystems' is dirty
warning: updating lock file '/home/birdee/birdeeSystems/flake.lock':
• Updated input 'nixpkgsNV':
    'github:nixos/nixpkgs/b833ff01a0d694b910daca6e2ff4a3f26dee478c' (2024-09-01)
  → 'git+file:/home/birdee/temp/nixpkgs?branch=fix-treesitter-duplicate-grammar&ref=refs/heads/fix-treesitter-duplicate-grammar&rev=a3fe631e5218a66065fc9bef63f7569e8f746f19' (2024-09-08)
warning: Git tree '/home/birdee/birdeeSystems' is dirty
trace: error: found duplicate grammars (norg-grammar-0.20.4 and norg-grammar-0.0.0+rev=aa1a1a7). use '.filterOutGrammars' to filter out duplicates
example: 'vimPlugins.nvim-treesitter.withAllGrammars.filterOutGrammars (e: e.meta.name == "norg-grammar-0.20.4")'

(nvim-treesitter.withAllGrammars.filterOutGrammars (e: e.meta.name == "norg-grammar-0.20.4"))
       error: attribute 'filterOutGrammars' missing

       at /nix/store/02fnqlyy2l8rncav1s5hnfpk06gswpw3-source/common/birdeevim/default.nix:294:10:

          293|         nvim-treesitter-textobjects
          294|         (nvim-treesitter.withAllGrammars.filterOutGrammars (e: e.meta.name == "norg-grammar-0.20.4"))
             |          ^
          295|       ];

I also really dont think this is necessary. Users who care which version is added can add whatever grammars they want via withPlugins they dont need to use withAllGrammars to do so. They already have access to the list that would have been added, and the withPlugins method they would have been added to. They can filter that list however they want, or add them individually.
  # If users want complete control while still having the equivalent of withAllGrammars, they can filter this list
  pkgs.vimPlugins.nvim-treesitter.withPlugins (plugins: builtins.attrValues plugins)

All this is doing that couldnt be done before is adding build failures to random people who just wanted to not think about it and do withAllGrammars. withAllGrammars is for that kind of person, not the kind that wants control over exactly which ones are added. THAT is why I am so ok with ln -sf

I am ok with withPlugins throwing on duplicates, but I am not ok with withAllGrammars throwing on duplicates.

you're right


UF-8181: Removed server information

Yes it will. I think that as long as it is not very sensitive data (such as password, user and like) it doesn't need to be purged from history. If there is any repo containing highly sensitive data, the approach must be to also purge history (repo with such data has not been found yet).


Scarecrow infinite animation

If you talk to the scarecrow after creating your song in OoT, then the game goes into an infinite animation of the scarecrow jumping up and down.

Can you post the Version: ... line from your spoiler log? This is supposed to be fixed.

v1.1.2


Add request spec verifying no unauthenticated routes

Who is this PR for?
students, families, developers
What does this PR do?
Adds another guard to check for unauthenticated endpoints beyond ApplicationController.  Also removes unused ActionStorage endpoints.
Checklists
Does this PR use tests to help verify we can deploy these changes quickly and confidently?

[x] Included specs for changes
[x] Improved specs for existing code in need of better test coverage

selfie


prompts.py and "evaluating on a custom test set" section in the model evaluation tutorial

In this section in the tutorial, after applying the Alpaca prompt style on the data sample, all the contents of the data sample including "output" are in the formatted string under "Instruction." Since this formatted prompt is later fed to the LLM, isn't it concerning that the correct output itself is in the prompt? Is it a bug in the implementation in prompts.py? From the code of Alpaca class there, it seems only the "instruction" part in the data sample should be put under "Instruction" in the formatted prompt.
Furthermore, is prompts.py used in the finetuning? If so, would it be very concerning as LLMs would be finetuned to get used to having the correct response in the prompt?

To answer my own question "Is it a bug in the implementation in prompts.py?" it is more of an issue of how prompt_style.apply() is called in the example from the tutorial. Specifically, instead of
from litgpt.prompts import Alpaca

prompt_style = Alpaca()
prompt_style.apply(test_data[0])

this should return a more expected and appropriate formatted prompt
from litgpt.prompts import Alpaca

prompt_style = Alpaca()
prompt_style.apply(prompt=test_data[0]["instruction"], **test_data[0])

which is
"Below is an instruction that describes a task. Write a response that appropriately completes the request.\n\n### Instruction:\nName the author of 'Pride and Prejudice'.\n\n### Response:\n"

Still, I'd like to know if the apply() method of any prompt styles is incorrectly called in any other places, especially in finetuning code. Thank you!

Hi @zty12713 , thanks for opening this issue. You are right, this is indeed a typo in the evaluation tutorials. I am fixing it in #1790
In the finetuning code, it looks like it's correctly called:
https://github.com/Lightning-AI/litgpt/blob/a66dcc9876d185835796d3d2ff6c8f3e11a355b2/litgpt/finetune/full.py#L333C1-L337C50


[BEAM-8460][release-2.17.0] Ignore UsesStrictTimerOrdering for both batch and streaming flink

Backported from #9869.

Run Java Flink PortableValidatesRunner Batch


Add vuex support

I just add vuex support for template included

Folder structure follow document guideline
Add testing for store

@thangman22, every once in a while somebody wants to add vuex support, just like me few months ago, webpack template is kept as simple as possible so, unfortunately, vuex is not going to be added.

So when this one ll be solved finally?

 I personally install vuex every single time I am using this template

webpack template is kept as simple as possible

it kinda doesn't make sense having support for "simple" features like e2e testing and lint but no vuex support

webpack template is kept as simple as possible

There is webpack-simple template for this, I guess?


feat: update the implementation of deleting manifest for distribution-spec v1.1.0-rc3

Support deleting image indexes with a subject
Resolves: #535

Codecov Report

Merging #558 (7bf7a42) into main (79d7081) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #558      +/-   ##
==========================================
+ Coverage   74.12%   74.19%   +0.06%
==========================================
  Files          50       50
  Lines        4731     4732       +1
==========================================
+ Hits         3507     3511       +4
+ Misses        898      896       -2
+ Partials      326      325       -1

Files Changed
Coverage Δ

registry/remote/repository.go
70.90% <100.00%> (+0.31%)
:arrow_up:


dh1 - apt.grafana.com stable InRelease' doesn't support architecture 'i386'

the repository somehow does not look right and apt update fails
root@dh1:~# apt update
Hit:1 https://download.docker.com/linux/ubuntu jammy InRelease
Hit:2 https://apt.grafana.com stable InRelease
Hit:3 http://us.archive.ubuntu.com/ubuntu jammy InRelease
Hit:4 http://us.archive.ubuntu.com/ubuntu jammy-updates InRelease
Hit:5 http://us.archive.ubuntu.com/ubuntu jammy-backports InRelease
Hit:6 http://security.ubuntu.com/ubuntu jammy-security InRelease
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
All packages are up to date.
N: Skipping acquire of configured file 'main/binary-i386/Packages' as repository 'https://apt.grafana.com stable InRelease' doesn't support architecture 'i386'

please use dockers only
see working example: https://github.com/opiproject/otel/blob/main/docker-compose.yml

Removed install of prometheus and grafana using apt install, will use docker compose

removed install of prometheus and grafana with apt and will use docker compose instead

implemented https://github.com/opiproject/lab/blob/main/docker-compose.yml


An equilateral triangle is a special case of an isosceles triangle.

An isosceles triangle is a triangle with (at least) two equal sides. A triangle with all sides equal is called an equilateral triangle, and a triangle with no sides equal is called a scalene triangle. An equilateral triangle is therefore a special case of an isosceles triangle having not just two, but all three sides and angles equal.
http://mathworld.wolfram.com/IsoscelesTriangle.html

Thanks for the PR! Much appreciated.
The Rust implementation of the triangle exercise follows the current canonical test suite, which doesn't do these sorts of tests.
Instead of changing only our tests, I suggest a PR against the standard test suite. If that PR is merged, then all tracks that implement the Triangles can improve their tests.

@elahn I started a conversation on the topic here: https://github.com/exercism/x-common/issues/379

This should be reopened now, right?

I'd like to wait for the PR in x-common to get merged first.


Adding react 19 to peer deps

Description
Adding React 19 to the options list for the published packages' peer dependencies.

This PR #2811 needs to be merged before we can safely bump the versions, as there's some breaking change in React 19.

Thanks for the contribution @BenAHammond. This will be merged altogether with the other changes required to support React 19 in https://github.com/radix-ui/primitives/pull/2934


env(config): set sideEffects to false

Description
in webpack 4, libraries have to  provide hints to webpack's compiler for "pureness".
i think all of code does not contain side effects, just set sideEffectsto  false. If there is a file with sideEffect, it would be nice to add this file separately.
Reference
https://webpack.js.org/guides/tree-shaking/#mark-the-file-as-side-effect-free

@all-contributors please add @SoYoung210 for infra


[WINDOW]improve structured streaming window of calculated

What changes were proposed in this pull request?
Remove the CaseWhen，Modified the calculation method of the obtained window
Why are the changes needed?
More simpler and more efficient
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing test as this is just refactoring.

Can one of the admins verify this patch?


[KV] Metrics & Analytics docs for KV

[DO NOT MERGE]

Let's go ahead and close this out since we've got the replacement one up

Closed in favor of https://github.com/cloudflare/cloudflare-docs/pull/16328


Mapper.xml  element is not available under   ...

According to the public mapper dtd the <bind> tag is only allowed under a <select>, <insert>, <update>, <delete>, and <selectKey>. It should also be available under the various other nodes like <sql>, <if>, <when>, <otherwise>... as well. The analogous velocity lang extension, #set(...) can be called anywhere and the Mapper.xml should behave similarly.
The below is not currently possible in MyBatis 3.2.2
<mapper>
    <select ...>
        select
            f.column1,
            f.column2,
            <if test="isBarFilled">
               <bind name="columnPrefix" value="b_"/>
               <include refid="barColumns"/>
            </if>
        from foo f
        <if test="isBarFilled">
            <bind name="tableAlias" value="b"/>
            <include refid="barJoin"/>
        </if>
    </select>
</mapper>

Originally from https://groups.google.com/forum/#!topic/mybatis-user/rQBxuqLe5X8

Hi Is this fixed ?

@gocoolharsh Probably this is fixed already. please try!! If it cannot fix, please create a new issue. Thanks.


[Comfy] Left bottom sidebar broken (shifted to the right)

Hey, I found your fork of Comfy theme while searching for a fix for this. Since today, my bottom left sidebar looks like this: the settings etc. icons are shifted to the right. I narrowed the cause down to comfy theme. I tried updating classes (on my own fork), but it did not fix it. I am not very knowledgable of this stuff, so maybe you can have a look at it?

sorry just seeing this, pushed a fix, not exactly sure what broke it though


Are there plans to support Mybatis SQL Mapper XML functionality?

This is a feature that Mybatis comes with and is used very frequently in slightly larger projects.
I did some kind of available state by modifying the quarkus-mybatis source code, but unfortunately I don't have the development experience of quarkus. Merging it may not be appropriate.
Related documentation:
https://mybatis.org/mybatis-3/configuration.html#mappers
https://mybatis.org/mybatis-3/sqlmap-xml.html

I think we has supported for xml configuration but I'm not sure the sqlmap is working. So can you provide a simple test or project for using the Sql Mapper XML ? I'd  like to take a look at it. Thanks a lot !

I think we has supported for xml configuration but I'm not sure the sqlmap is working. So can you provide a simple test or project for using the Sql Mapper XML ? I'd like to take a look at it. Thanks a lot !

I may find the problem, I found that org.apache.ibatis.session.Configuration has been created two, the former lacks SQL Mapper Xml information. I am trying to organize a minimal test item to reproduce the problem.

@zhfeng Through the breakpoint of org.apache.ibatis.session.configuration.addMappedStatement (line: 768), I found two different Configuration, which happen to be Controller's acquisition of a Configuration without XML. Maybe that's the point.
It could also be that my error caused the xml configuration to not work properly.
Simple test project: https://github.com/Xinmudotmoe/quarkus_mybatis_sql_mapper_example

Thanks @Xinmudotmoe and I think this is similar with https://github.com/quarkiverse/quarkus-mybatis/issues/114
And current there is a workaround to add @MapperDataSource("xmlconfig") to specify using the xm configuration. I just make some changes with your example to make sure it works. Please check https://github.com/Xinmudotmoe/quarkus_mybatis_sql_mapper_example/pull/1
I will raise another issue to improve the xml configuration.

And feel free to close this issue if you get your example work properly. Thanks again !

@zhfeng  It's working. Thanks.


build(deps): Update queryst to 3.x

This resolves security advisories for the regex and thread_local crates, for which queryst 2.x relied on vulnerable versions.
regex: https://github.com/advisories/GHSA-m5pq-gvj9-9vr8
thread_local: https://github.com/advisories/GHSA-9hpw-r23r-xgm5

Hi @michiel , a friendly nudge to see if you can merge this PR. I'm building something with jsonapi and currently this package fails in cargo audit. Will be super glad if this can be fixed. Thanks!
PS: Thanks a lot for this library!

Thanks @ecthiender I'll bump the version later


Fabric parallel and serial decorators fix

Coverage increased (+0.02%) to 97.83% when pulling 5a4b001fc677f8e95199f7a4f0cc19d529fe1e34 on parallel_serial_fix into ca1d37ebca91780c02629693b0945d154523e2b0 on master.

Coverage increased (+0.02%) to 97.83% when pulling 468daa4e77a0e17cdd762da221a21ce83c547e62 on parallel_serial_fix into ca1d37ebca91780c02629693b0945d154523e2b0 on master.


create a jupyter web crate

Support connecting to Jupyter servers via websockets using the strongly typed JupyterMessage type from runtimelib
Handle both nbclassic and Jupyter server implementations

There's a main.rs here mostly just for a sanity check before doing integration work with Zed.

Once this is out there I might look to see how easy it is to support the newer binary protocol: https://jupyter-server.readthedocs.io/en/latest/developers/websocket-protocols.html


Fix suspension

Only generator executions can suspend, currently suspension is broken. Once we resolve this we should also export the generator version of resonate.

We have removed the generator version of the SDK, this issue is obsolete.


[WIP] Added new three detection events

Add WindowsThreat event for windows sysmon events.
Add NetworkThreat event for network events.
Add ExtraThreat event for misc events.

Codecov Report
Attention: 50 lines in your changes are missing coverage. Please review.

Comparison is base (e16a7df) 47.27% compared to head (135e652) 47.08%.
Report is 2 commits behind head on main.

Files
Patch %
Lines

src/graphql/event/network.rs
0.00%
25 Missing :warning:

src/graphql/event.rs
31.81%
15 Missing :warning:

src/graphql/event/log.rs
0.00%
5 Missing :warning:

src/graphql/event/sysmon.rs
0.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   47.27%   47.08%   -0.20%
==========================================
  Files          62       65       +3
  Lines       10377    10432      +55
==========================================
+ Hits         4906     4912       +6
- Misses       5471     5520      +49

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

pending petabi/review-database#176

@kimhanbeom
Please change Cargo.toml  to use review-database 0.23.0


Revert #52569

After giving the solution in https://github.com/Expensify/App/issues/52569 a try, we've realized that it creates a confusing situation in the Inbox, which is:

Workspace chat is GBRed
Report has a grey Approved button

So in short, our solution for approve and pay (specifically with a report that has some held expenses and some unheld expenses) ends up breaking GBR.
As a result, we're going to revert the PR from #52569. That restores GBR to the Inbox and workspace chat. In addition, for the Search page specifically, we are going to disable the Approve button for this case, just like we landed on doing for Pay.

@JmillsExpensify The expectation is that we revert the PR, and then disable Approve button as well as Pay as a new requirement? Would it be a single PR or two PRs - one revert and then second to disable Approve?
I can help with the review on this one if needed.

I think we want to:

Revert the behavior of changing the button colors.
On Search, disable the option in the bulk action dropdown if any selected items are on hold

Yes, I can pick this.

Great! All yours ❤

@mananjadhav how's this one going? Do you have an ETA for the PR?

I am a bit sick, I'll have the Pr by tomorrow.

Ah no worries. Hope you feel better soon.

Apologies I was unwell. I am just getting back and will have this ready in 1-2 days.

Will raise the PR today.


Change headline copy to "Welcome to Cody by Sourcegraph!"

Since we don't have username, the welcome message looks weird and bare:

Instead of "Hi", change to "Welcome to Cody by Sourcegraph!"

@iskyOS can you review this copy to make sure it looks good to you?


BUG: installing dependencies fails

Describe the bug:

Trying to install any dependenc with yarn or  nom fails.
yarn add ylq
warn preInstall No description field
warn preInstall No repository field
warn preInstall No license field
┌ [1/3] 🔍  Resolving dependencies
└ 𐄂 Failed
ERR!: Failed to resolve dependencies

error Command failed with exit code 1

~/projects/node-h6n33x

Link to the blitz that caused the error
https://stackblitz.com/edit/node-h6n33x
new created empty nodejs project
Steps to reproduce:

create new empty nodejs project
write in Terminal yarn add ylq
See error

Expected behavior:

install dependencies :-)
Version of webcontainer:
Hash: node-xbebwg
Screenshots:

Desktop (please complete the following information):
Browser name  = Chrome
Full version  = 91.0.4472.77
Major version = 91
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.77 Safari/537.36
performance.memory ={
jsHeapSizeLimit:<PHONE_NUMBER>
totalJSHeapSize: 63201213
usedJSHeapSize: 56440405
}
Additional context:

Hi @aheissenberger, thanks for trying out WebContainer. Have you tried other packages, and do they work? It doesn't look like the package you try to install exists https://www.npmjs.com/package/ylq. Could that be the problem?

you are right there is a typo in the package. But I tested again and it does not work:
npm install vlq
or
npm install lodash
warn preInstall No description field
warn preInstall No repository field
warn preInstall No license field
┌ [1/3] 🔍  Resolving dependencies
└ 𐄂 Failed
ERR!: Failed to resolve dependencies

error Command failed with exit code 1

Can you open your devtools and look at the network requests? Do you run into CORS perhaps? Try hard refreshing your browser and try it again.

It looks like because you misspelled vlq in your package.json, that it now fails on installing any package. Rename ylq in the dependency list in package.json to vlq and it will work.
@d3lm Is this intended behaviour? If you haven't run npm install yet, and then you run npm install lodash, that it also installs all other dependencies?

Yes that's intended behavior and it's the same with npm 7.x and yarn. Only older versions of npm (< 7) did not install all dependencies.

you are right - all my fault :-). great product

No worries, glad we figured this out 🙏 😄


Screenshots, Updates, Announcement

Inspired by deoplete.
This issue is a good place to upload screenshots for README.md.
Screenshots shoould not be put into git repo, otherwise it will make the plugin download time much much longer .
Additional tips screenshots will also be published in this issued,
Subscribe this issue if you are interested

All in one screenshot

Triggering Ultisnips or show popup hints with the same key <c-u>
let g:UltiSnipsExpandTrigger = "<Plug>(ultisnips_expand)"
inoremap <silent> <c-u> <c-r>=cm#sources#ultisnips#trigger_or_popup("\<Plug>(ultisnips_expand)")<cr>

Add css completion for inline style attribute:

Added C/C++ code completion with clang_complete

Just implemented a <c-n><c-x><c-n>... like completion source:
https://github.com/roxma/nvim-completion-manager/issues/24#issuecomment-283635150
Pick a key to map to <Plug>(cm_force_refresh)
imap <c-g> <Plug>(cm_force_refresh)

I also have tab mapping
      " smart tab for auto complete
      inoremap <expr> <silent> <Tab>  pumvisible()?"\<C-n>":"\<TAB>"
      inoremap <expr> <silent> <S-TAB>  pumvisible()?"\<C-p>":"\<S-TAB>"

<Tab> for selecting the popup item, <c-g> for refreshing the popup menu. So the key pressed in this screencast is <TAB><c-g><TAB><c-g>...

copied from https://github.com/neovim/neovim/issues/5522#issuecomment-284122091:
Since LanguageClient-neovim already has nice interface, I'm deprecating my previous nvim-cm-php-language-server
Here's a simple php language server package that could be easily setup to work with LanguageClient-neovim, and it is well integrated with nvim-completion-manager

Copied from #38 , This applies to users who want to use <Tab> for both ultisnips and completions:
Add the following code to your vimrc, this version should work better for both plugins.
And if there's an openning popup menu get in the way, use <c-y> to close the popup menu. Or simple map another key, <c-u> in this example, to trigger ultisnips.
    let g:UltiSnipsExpandTrigger        = "<Plug>(ultisnips_expand)"
    let g:UltiSnipsJumpForwardTrigger   = "<Plug>(ultisnips_expand)"
    let g:UltiSnipsJumpBackwardTrigger  = "<Plug>(ultisnips_backward)"
    let g:UltiSnipsListSnippets         = "<Plug>(ultisnips_list)"
    let g:UltiSnipsRemoveSelectModeMappings = 0

    vnoremap <expr> <Plug>(ultisnip_expand_or_jump_result) g:ulti_expand_or_jump_res?'':"\<Tab>"
    inoremap <expr> <Plug>(ultisnip_expand_or_jump_result) g:ulti_expand_or_jump_res?'':"\<Tab>"
    imap <silent> <expr> <Tab> (pumvisible() ? "\<C-n>" : "\<C-r>=UltiSnips#ExpandSnippetOrJump()\<cr>\<Plug>(ultisnip_expand_or_jump_result)")
    xmap <Tab> <Plug>(ultisnips_expand)
    smap <Tab> <Plug>(ultisnips_expand)

    vnoremap <expr> <Plug>(ultisnips_backwards_result) g:ulti_jump_backwards_res?'':"\<S-Tab>"
    inoremap <expr> <Plug>(ultisnips_backwards_result) g:ulti_jump_backwards_res?'':"\<S-Tab>"
    imap <silent> <expr> <S-Tab> (pumvisible() ? "\<C-p>" : "\<C-r>=UltiSnips#JumpBackwards()\<cr>\<Plug>(ultisnips_backwards_result)")
    xmap <S-Tab> <Plug>(ultisnips_backward)
    smap <S-Tab> <Plug>(ultisnips_backward)

    " This is optional if you're using NCM
    inoremap <silent> <c-u> <c-r>=cm#sources#ultisnips#trigger_or_popup("\<Plug>(ultisnips_expand)")<cr>

From https://github.com/roxma/nvim-completion-manager/issues/36#issuecomment-284948146:
Just take a deeper look into ultisnips, now it works with ultisnips, :smile:

Parameter expandsion for python, javascript, golang, via ultisnips/neosnippet

I have just finished adding NCM support to https://github.com/clojure-vim/async-clj-omni for Fireplace only. Acid.nvim will soon be supported also, once I have performed some refactoring.
This brings Clojure support to NCM.
There is currently no caching of completions, but there is contention between the underlying engine & NCM. I have some ideas of how to bring them together though, so expect caching as an upcoming feature.

Rust completion:
I've been learning rust recently and just implemented a completion plugin based on racer last night. I'm not sure I'm getting racer working the right way. I works but I still miss the macro completion.

Thanks to @lervag and @Shougo, vimscript completion is available via neco-vim

I have added simple Swift completion using sourcekitten: nvim-cm-swift-completer
Any suggestion to improve it is welcome

gtags complete source  gen_tags.vim

I added ncm source for neco-syntax
Thanks for @shougo, the source was already merged into neco-syntax

I also add neoinclude support too.
But I'm a little confused about the source priority that which one should be higher? neoinclude(file include) or neco-syntax(language syntax)
Currently I give 8 to both of them. neco-syntax mod was already merged to Shougo's repository, but neoinclude mod is not merged yet.
neoinclude screenshot here:

@jsfaint

But I'm a little confused about the source priority that which one should be higher? neoinclude(file include) or neco-syntax(language syntax)

It seems they  don't usually apear at the same time. So it doesn't matter.

@roxma Got it, thanks for your clarification 👍
I'll create PR for neoinclude

https://github.com/roxma/ncm-github
Github completion is added. The idea was borrowed from https://github.com/rhysd/github-complete.vim
I re-implement it in python, so that the sources should work asynchronously.

let g:cm_matcher = {'module': 'cm_matchers.substr_matcher', 'case': 'smartcase'}

Added sub-string based mathing.

Added a toy project: [ncm-phpactor](http://github.com/roxma/ncm-phpactor

Another plugin ncm-clang for c/c++ completion. This project deprecates roxma/clang_complete

Adding typescript source: https://github.com/mhartington/nvim-typescript/pull/81

I've created a source for the R language: https://github.com/gaalcaras/ncm-R

Please note that Nvim-R is required for ncm-R to work.

I'm sorry to announce that my interests moves to some other projects, due to lack of time I may not update this repo for a long while.
If anyone wants to keep a well-maintained fork, post your link here. I'll put it on top of the README page.

@roxma Do you think ncm stilll has any significant advantages over deoplete to make it worth maintaining it or would you recommend people to move over to deoplete in the long run?

@mfussenegger it has advantages over deoplete?

I think it has advantages of every other completion manager out there. I sincerely hope someone takes it on
--
John Shea
Cell:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>
On Apr 18, 2018, at 3:15 PM, Andrey Orst<EMAIL_ADDRESS>wrote:
@mfussenegger it has advantages over deoplete?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@coachshea not that I do not believe ncm has no advantages, could you be more specific in regards to what the advantages are?

@nhooyr, I should have been a little more careful with my language. Of course, I have more experience with some systems than others. I have just loved my experience with ncm and am sad to see it -- possibly -- come to an end. I did try deoplete several years ago and I don't remember specifically why I didn't stay with it. All I do know is that I tried several at the time and chose to stick with YCM. I was by and large happy with YCM and never thought I would try another, but for a variety of reasons, I gave ncm a shot, and I instantly loved it. I made the switch and never (until now) looked back. I'll list some of what I love best about it. It is certainly possible that deoplete, or some other completion manager has similar features and maybe I will have to give it a try again.
First, the start up time is instantaneous. One of my few complaints about YCM was startup time. There are a lot of things I liked about the completions, but maybe most of all was the "on-the-fly" snippet creation. I also loved that you could choose regular or fuzzy completion, and how easy it was for others to create add-ons for other filetypes (I realize that in that regard deoplete is similar).

I agree that ncm has been great. I had similar experiences as @coachshea a couple of years ago. However, after trying out deoplete now I have to admit that I can't remember why I didn't stick with it the last time. It seems to work great now.

One of my few complaints about YCM was startup time

@coachshea well I believe that the case long startup time of YCM is:

most of all was the "on-the-fly" snippet creation

Can agree here, it's like the only advantage of NCM over Deoplete for me. However, Language servers are providing snippets too. Only need to wait when snippet managers start supporting it. I'm waiting to try another Shougo's plugin Deoppet because it will support it as far as I know.

Thank you for all your vim work so far it's amazing. I hope someone can pick up this project, I'll have to check out deoplete again but this project and your other work seemed to integrate a lot more easily than competitors.

I just wanted to switch to deoplete, and first thing i noticed - there is no function param expansion (at least not with ultisnips) or I did not find it. That feature i will miss a lot.
I think i did not even notice the good job you did here until you quit maintainance - thanx for your work so far. Hopefully there will be anybody to keep this codebase alive.

I would like to take a stab at maintaining this project. If you could give me some direction, I'd love to contribute and see if I can handle this project.

@partounian If you need any help count me in! I'd love to help keep NCM alive

Maybe tag what would be a great first issue to tackle? @filipekiss @roxma

when isntall

@jirgn You need to install neosnippet for it.

@Shougo - I tried deoplete again, now with neosnippet, but can't get this feature to work. What I expected is, when having a completion candidate e.g a function selected in pum, just hit the mapping key for neosnippet expansion, and then the function signature params are expanded. These params act as snippet marks $1 - $n and i can navigate through them.
If you can tell me how to configure this in your setup, i'll give it another try.

@partounian Are you still interested in maintaining this project? I still think ncm is the best completion framework out there and would hate to see it die.

on windows, ncm is faster then deoplete

Recently I started to work on ncm2. It is still WIP, and it is far from complete.
The code is based on ncm but many behaviors has changed. All sources will be moved out and it is not going to be compatible with existing source. Snippt integration is going to be a standalone plugin.
I want it to be easier to integrate with other features/plugins, such as go-to-definition and so-on. And I'm going to have some funny idea to try, e.g. ncm2-highlight-double-stuck
It's easier to rewrite the code without legacy concerns, and with some evil premature optimizations removed.

@roxma yay! good luck and can't wait to see what yah get.

@roxma  So glad to see you back!

@roxma Great to hear see that! Are you planning on adding support of LSP snippets into that snippet plugin you're talking about?

@andreyorst

Are you planning on adding support of LSP snippets into that snippet plugin you're talking about?

Yes

Any way I can help with this new development? @roxma

@partounian
There're priorities of porting ncm sources to ncm2.
So far, I mostly use C/C++  at work, and with some js/css/html, I tend to use python and rust for my side projects.
I'll port these sources ASAP, and I'm going to extend the C/C++ support plugin. At least , I'm expecting it to be the same level of YCM.
As for other sources, it's kind of low priority for me. You could pick the one you need the most and try porting/implementing it with ncm2.
The documentation is not yet ready, but https://github.com/ncm2/ncm2-bufword is probably a good small reference.

I'll try to take a stab at python support 😄

@andreyorst
I create sources mostly because I need it, and it feels easier for me. I had the impression that LSP and cquery is not ready in the past. Maybe it is ready now. I'll spend some time and try
I  agree with your opinion on what a completion framework should behave. By extending c/c++ support, I mean creating extra plugins, I'm not gonna add any source directly into the core repo. ncm2 itself is going to be a clean and hackable completion framework.
IMO it's not making things complicated, instead we have more choices. If one completion plugin doesn't work so well as expected we can easily ditch it and look for alternatives.
By the way, maybe completing multiple languages in the same file, like when you're working with html and css and php in the same .html document simultaneously,  is kind of impossible for LSP based sources. It's easier for me to implement them based on ncm2. For LSP stuff, https://github.com/autozimu/LanguageClient-neovim is already a good choice, and I'll add  LanguageClient-neovim binding as a separate plugin for ncm2 in the near future.

@partounian
Can't wait for python support. Here's a working version: https://github.com/ncm2/ncm2-jedi
However, jedi has some issues, like #215 100% CPU  and #62 high memory consumption.
I think we should add some extra vimrc to kill the jedi process and clean the channel when the source is not active for an interval.


Disable php functions for defence in depth

This issue is imported from pivotal - Originaly created at Jul 4, 2018 by Bart Geesink
Our hosting partner recommends to disable the following php_functions:
disable_functions = exec,passthru,shell_exec,system,proc_open,popen,curl_multi_exec,show_source,pcntl_alarm,pcntl_fork,pcntl_waitpid,pcntl_wait,pcntl_wifexited,pcntl_wifstopped,pcntl_wifsignaled,pcntl_wifcontinued,pcntl_wexitstatus,pcntl_wtermsig,pcntl_wstopsig,pcntl_signal,pcntl_signal_dispatch,pcntl_get_last_error,pcntl_strerror,pcntl_sigprocmask,pcntl_sigwaitinfo,pcntl_sigtimedwait,pcntl_exec,pcntl_getpriority,pcntl_setpriority,
Can these be disabled for all PHP apps of OpenConext?
Furthermore, they recommend setting an open basedir restriction:
open_basedir = /var/www/html/:/tmp/:/dev/urandom/
Thoughts on that?

This would be a great defense-in-depth measure.
Regarding the disabled functions, I foresee no problems in EngineBlock (I've grepped the source) or any other project, except:

the user-lifecycle API which requires curl_multi_exec in web requests
the symfony console application might require shell_exec for cli requests

Regarding open basedir, there's only one way to find out... (Joris Steyn - Jul 4, 2018)

@bartgeesink have you had time to further look into this? Maybe we can safely test this on the test env? (Michiel Kodde - Sep 9, 2019)


.gmod gives error when enabling it

Latest version 2.28

Can confirm.
On my end. Bot responds in chat disabling gambling. But when enabling no response. Both enable and disable throw the same error.

@mstarbuck13 yep

Fixed in 2.28.1

Still gives error in 2.28.1

I'm officially retarded.

Can confirm both the above comments. #KwothTheIncompetent


Unicode error

Since "All text is Unicode" in Python v>3.0 then this line of code will throw an error says:
name 'unicode' is not defined
So, regarding to this answer, I guess unicode function should be changed to str or decode()

FYI, I opened #21 for this issue. If you look at the commit history the fix for python2 unicode issues broke python3.

I believe this is fixed. Please confirm!

Closing this. If this is still a problem, please reopen.


Python: ObjectAPI to ValueAPI: WrongNumberArgumentsInClassInstantiation

This PR is part of the ValueAPI modernization effort.

@BekaValentine this PR seems to introduce some FPs with subclasses. For a number of examples, see https://jenkins.internal.semmle.com/job/Changes/job/Python-Differences/53//artifact/changes.html
I can see that the test file doesn't have any subclassing, so if you could make a new PR that adds that and fixes the FPs that would be great :+1:


Tabs , Error while updating 'nativeBackgroundAndroid'

I have gone through these following points

[x] Check latest documentation: https://docs.nativebase.io/
[x] Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[x] Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[x] Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[x] For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[x] For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description

node, npm, react-native, react and native-base version, expo version if used, xcode version
System:
    OS: Windows 10 10.0.18363
    CPU: (4) x64 Intel(R) Core(TM) i3-8100 CPU @ 3.60GHz
    Memory: 2.05 GB / 11.85 GB
  Binaries:
    Node: 10.15.3 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.4 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.13.4 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK:
      API Levels: 23, 26, 27, 28, 29
      Build Tools: 23.0.1, 27.0.3, 28.0.3, 29.0.2
      System Images: android-19 | Google APIs ARM EABI v7a, android-19 | Google APIs Intel x86 Atom, android-21 | ARM EABI v7a, android-21 | Google APIs Intel x86 Atom, android-22 | Google APIs Intel x86 Atom, android-28 | Intel x86
 Atom_64, android-28 | Google Play Intel x86 Atom, android-29 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: Version  <IP_ADDRESS> AI-193.69<IP_ADDRESS>26763
  Languages:
    Java: 1.8.0_221 - C:\Program Files\Java\jdk1.8.0_221\bin\javac.EXE
    Python: 2.7.16 - C:\Python27\python.EXE
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.63.2 => 0.63.2
  npmGlobalPackages:
    *react-native*: Not Found

Expected behaviour
Tabs component must work an android kitkat
Actual behaviour
Tabs component has Error while updating 'nativeBackgroundAndroid'
Steps to reproduce

import React, { Component } from 'react';
import { Container, Content, Tabs } from 'native-base';

import TabOne from './tabOne';
import TabTwo from './tabTwo';

export default class TabsExample extends Component {
    render() {
        return (
            <Container>
                <Content>
                    <Tabs>
                        <TabOne tabLabel='One' />
                        <TabTwo tabLabel='Two' />
                    </Tabs>
                </Content>
            </Container>
        );
    }
}

Is the bug present in both iOS and Android or in any one of them?
I tested in android kitkat but I think it occured on api<21
Any other additional info which would help us debug the issue quicker.
###Important
If you want your issue to be looked at quicker, attach a snack reproducible with your issue. Makes it easier for us!

@sankhadeeproy007 @SupriyaKalghatgi @akhil-geekyants take a look please

Hi @sm2017 , Is it working on latest Android version? If not, please follow NativeBase Docs. You are not using Tab as shown in the code provided by you.
Let us know if the issue still persists.
import React, { Component } from 'react';
import { Container, Header, Content, Tab, Tabs } from 'native-base';

export default class TabsExample extends Component {
  render() {
    return (
      <Container>
        <Header hasTabs />
        <Tabs>
          <Tab heading="Tab1">
            ...
          </Tab>
          <Tab heading="Tab2">
          ...
          </Tab>
          <Tab heading="Tab3">
            ...
          </Tab>
        </Tabs>
      </Container>
    );
  }
}

Hi @sm2017 , closing for now due to inactivity. Let us know if the issue still persists.

Error while updating property 'nativeBackgroundAndroid' of a view managed by: RCTView
I've reproduced the issue. Android versions between 4.1 and 4.4 are crashing


ram usage without declaring variables

Applies To

[ ] Notebooks (.ipynb files)
[X] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
When ever I filter a dataframe with .iloc .loc .filter (not with slicing) It occupies RAM and never release it again. I do not assign any new variable, i'm only querying a subset of it and printing it in the interactive window.
Like:
df.iloc[:, 0:20] # occupies ram

I'm using Text Editor: Execute Selection feature.
Copied from https://github.com/microsoft/vscode-jupyter/discussions/14190
I'm slicing a dataset with loc iloc filter and not saving it to a varaible because I'm exploring a dataset, but it occupies RAM. How can I avoid this? Is it normal? I've never had this issue before, and I work a lot with jupyter in vscode. What's happening?

df.iloc[0:100000] # increases RAM every time I execute it!

I've tried this same code in ipython terminal, and does not increases my RAM usage. So this must be a thing of jupyter+vscode. Any clue?
Edit: I'm using .py files for this (not ipynb) with text selection execution configuration, but I allways do...
VS Code Version
1.81.1
Jupyter Extension Version
v2023.7.1002162226
Jupyter logs
No response
Coding Language and Runtime Version
python 3.9.2
Language Extension Version (if applicable)
v2023.14.0
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

Thank you for filing this issue, and sorry you are running into this.
A few questions

Do you have the variable viewer open in VS Code (this is displayed at the bottom and display the list of Variables in Jupyter Kernel)

If you have that open plese can you close it and try again

Can you replicate the same issue in Jupyter Notebooks (outside of VS Code)

Do you have the variable viewer open in VS Code (this is displayed at the bottom and display the list of Variables in Jupyter Kernel)
Nope, tried deleting existing variables also, but it only frees RAM that variable is using, not "the others" that were never assigned
Can you replicate the same issue in Jupyter Notebooks (outside of VS Code)
Yes, I've just replicated it.

I've found out that the issue only appears when I filter columns, not rows.

df.iloc[0:100000] # it's ok, does not increases RAM usage

 df.iloc[:, 10:300] # DO increase RAM

As this occurs even outside vscode, this isn’t specific to the product
Unfortunately I am unable to help you
Perhaps you can try stack overflow or the like
Or try the discussions in the repo
Thanks for visiting this isn’t specific to vscode


Address ambiguity in BRs

This should address https://github.com/zmap/zlint/issues/157.

@kumarde I can fix the comment, but I'm still curious about why the test certificate changed.


Android Studio Support

Currently not able to use the theme in Android studio.
Android Studio 3.1.3
Build #AI-173.4819257, built on June 4, 2018
JRE: 1.8.0_152-release-1024-b01 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Mac OS X 10.13.4

It would appear so, because it uses the 1.6.1 release of material ui and not the current :|

 VM438:2 # 17   Android Studio Support   https://api.github.com/repos/cyclic-reference/ddlc-jetbrains-theme/issues/17

next
https://pastebin.com/KsFmdN44

next
https://pastebin.com/4aP8pZWc

next
https://pastebin.com/zAVsu3rm

4 moar
https://pastebin.com/Xxb3F4S2

Next
https://pastebin.com/Xxb3F4S2

https://pastebin.com/GdXgyvXk
https://pastebin.com/8hhP2Dhi
https://pastebin.com/taN35Vwp

Todo: Gotta legacy support the buttons, that may fix a lot of the issues.

This cause the plugin not to show up
https://pastebin.com/Zs06FqW4

https://pastebin.com/XL3zDCsT

[ ] https://pastebin.com/KUK17ZA8

[ ] https://pastebin.com/VnZptjLf

@benammarkarim Android studio is supported now. If you are still interested, you can download the zip file off of the release page. https://github.com/cyclic-reference/ddlc-jetbrains-theme/releases/tag/v1.2.0
or wait until the plugin is publicly available.
Cheers!


Api project (and RPC) to allow ssl endpoints

The ssl key can be a stored as a file in disk with special permissions.

In order to avoid in flight data compromise
As an api consumer
I want all requests over https with a trusted certificate
feature complete when

http calls return 307 temporary redirect
all api calls require https
a valid certificate is used
api is on https on docker too (edited)

First Reviewer  @monsieurleberre
Second Reviewer (merger) @bokobza
(please thumbs up this if you approve the acceptance criteria)

It appears that support for SSL in RPC has been completely dropped in bitcoin.
SSL support for RPC dropped
Why is JSON-RPC over SSL “strongly discouraged”?
Enabling SSL on original client daemon
I suggest we don't do it in our node and this task has the "(and RPC)" removed from its title.
There is an argument there to say that if we remove SSL for RPC then we should also remove it for the API.
Thoughts?
@monsieurleberre @Aprogiena @dangershony

As I said before, I don't know why are we doing SSL. But if something makes sense, it is doing it for API because there we are actually sending password. So unless we switch to one-time tokens, and as long as we want to allow people to connect remotely to API, it makes sense to protect it. However, the argument on the bitcoin side is also strong because SSL is a big overkill for this task. There is a many years of history of super serious bugs inside of SSL and especially some implementations. This means that by implementing SSL for API, we will suddenly be open to large number of attacks that we can expect to be discovered in the future. I don't think we should do SSL now because people will not access API remotely anyway. Later we can come up with solutions without SSL.

Closed with #1897


Refine prefix behavior

Deactivate prefix mode after issuing a command: It would be nice to have a config option to end prefix mode the moment a command is issued. As it stands now, prefix mode remains active until the prefix-timeout has passed with no commands being issued. For instance, one can activate prefix mode and keep it active by continually switching back and forth between windows. I've kept my prefix-timeout short (ie: 0.35), but it still occasionally interferes when I try typing shortly after switching windows.
i.e. kwmc config prefix-quick-deactivate on
Let prefix mode be toggle-able: In addition, it would be nice if there were the config option to make prefix mode toggle-able so prefixed commands could be issued in bulk without worrying about the timeout. Ideally this would be a separate prefix-key so both functionalities would be accessible at runtime.
i.e. kwmc config prefix-toggle-key ctrl+shift+s

Deactivate prefix mode after issuing a command

Right now you can achieve this by making a shell script to use while in a binding mode. (kwmc mode activate <bindingmode>) It would execute the command you want and then run kwmc mode activate default. You could make it modular so that the script just executes the command you pass to it, that way you only need one script to handle running commands while in any prefix mode.

Let prefix mode be toggle-able

Use binding modes, and set the same key used to activate the binding mode, to activate the default mode, while you are in the binding mode. eg:
kwmc bindsym ctrl-m mode activate custommode
kwmc bindsym custommode+ctrl-m mode activate default

There is not really a notion of a prefix mode anymore. There are just modes.

Deactivate prefix mode after issuing a command

This can be achieved like this:
kwmc bindsym somebind command ; mode activate default
Replace default with whatever mode you want to activate.

Let prefix mode be toggle-able:

This is just what a normal mode is, if you do not perform kwmc my_mode prefix on
You'd have binds like the following:
// Activate my_mode
`kwmc bindsym ctrl-f mode activate my_mode`

// Random binds
`kwmc bindsym my_mode+some_binds some_command`

// Exit my_mode and enter default mode when ctrl-f is pressed
`kwmc bindsym my_mode+ctrl-f mode activate default`

LOL ill remote mine.

Wonderful. Will give master a try today. Thanks.


Peer dependencies handling

https://github.com/eslint/eslint/issues/3458#issuecomment-753534854

Hi @niksy ,
Do you know if installing peer dependencies is still an issue with eslint shared configs? i.e. do you still need to install them all manually?

@nkhil that should be handled with npm v7

There is no need to reference React or Vue plugins if they won’t be installed if project is not React or Vue based. We only get silly warnings when installing config.
It’s better to document when to install those plugins as opposed to tell users to install them every time.
There are certain plugins that must be installed for base config to work, but all other plugins can be installed when they’re needed.


foundation - grid overview

[ ] add image from Figma

As a design system user i want to understand whats the beste practice for specific grids columns.
e.g.

Where should i place a form in a layout
Which components are use on which grid / column

TODO

Document repeating patterns such as forms, accordions, layout for application (mybaloise).

Tickets needs to be sparred with the team.

[ ] Figma: How can we put more container together?
[ ] Extend Documentation with best-practise
[ ] Alignement Container / Grid
[ ] Visual example (Figma -> technical)

Currently we found out that we have only one gap size.
In Figma we have several more:
https://www.figma.com/file/TpLyniy71JAMZhps0qHhVU/Baloise-–-UI-Kit-(Team-Library)?node-id=37393%3A331070&viewport=-6098%2C-3969%2C0.26

move to https://github.com/baloise/design-system/issues/803


Troubles with LocalProject

Since we are not satisfied with the state of the LocalProject class, it would be nice to write down the problems. Feel free to add your specific problems and ideas about the ideal solution.
List of problems:

We are not sure, what will be calculated.
Name of the class.
Located in packit project. (Move to ogr.)
https://github.com/packit-service/packit/issues/348
Auto refresh/recalculate parameters on parameter change (@dhodovsk)
Is there a need for documenting the calculation order? (@dhodovsk)
No support for forks.
Anyone can improve LocalProject (even though it's Franta's baby)

Solution for 1:
Make the parse_* methods public:
my_project = LocalProject(url="url", working_dir......, refresh=False)
my_project._parse_git_repo_from_working_dir()
my_project._parse_ref_from_git_repo()

This will not solve transitive caluclations. (I need to calculate one attribute to get another.)

Solution for 1:
Use some builder-like syntax:
my_project = LocalProject.build(url="url", ....).working_dir().git_repo().calculate()

or
my_project = LocalProject(url="url", ....).working_dir().git_repo().calculate()

or
my_project = LocalProject(url="url", ....).calculate_working_dir().calculate_git_repo()

Could you specify what calculate means? - what are the steps done or information acquired? Or is it somewhere documented?
Something like:
step 1. do following to get config
a. look under the table for the config
b. if not a., look under Jirka's table
c. if not b., get it by screaming at the plant
step 2 - see if I got shoes in the function parameters, if not, try to steal some from Tomas
Or something like saying that having information A gets me B and I can do this and get information C.
It's because, I won't think it is magic, if I know what exactly is being done and I don't have to search in multiple-level function calls that something is done :)
You have powerfull refresh function that is called just once - wouldn't it be possible to run it on every "attribute-of-interest" change?
I look at the code and try to see if the code itself isn't self-explanatory enough... it actually is a really nice piece of code. So maybe I am just bitching for no reason.

Could you specify what calculate means? - what are the steps done or information acquired? Or is it somewhere documented?

I look at the code and try to see if the code itself isn't self-explanatory enough... it actually is a really nice piece of code. So maybe I am just bitching for no reason.

I hope the code itself is enough. If not, we can add the steps to docstrings...

Could you / You have

Am I the only one who should work on/with that code?..;-)

I don't have to search in multiple-level function calls that something is done :)

There are some parts in the project, that are duplicating the logic, that is already implemented in the LocalProject (or at least easily can be moved there). I do not see a spaghetti-code in the LocalProject. I see a little messy code on top of it => we need to:

make the LocalProject more usable
make it clear, how to use it

You have powerfull refresh function that is called just once - wouldn't it be possible to run it on every "attribute-of-interest" change?

Now, you can run it whenever you need manually, but it should be easy to run it on every change when you set refresh=True in the __init__. Nice idea!

@dhodovsk I've added your points to the top comment. Is that OK? Feel free to change that...

Auto refresh/recalculate parameters on parameter change (@dhodovsk)
Is there a need for documenting the calculation order? (@dhodovsk)

Solution for 1:
I've once heard about using object() as other default value. What about specifying what we need to calculate:
CALCULATE = object()

class LocalProject:
    pass

lp = LocalProject(url="some-url", working_dir=CALCULATE)

Since we have https://github.com/packit-service/ogr/pull/95 , we need a way to use it from packit.
When I am thinking about the implementation, it looks like we need to split the LocalProject into two parts:

LocalProjectBuilder

gets the PagureService/GithubService instances with the tokens
contains the parsing/calculating logic

LocalProject

contains the methods for easier manipulation with repo (checkout_pr, push_to_fork, ...)
the attributes are not refreshed anymore? (this is against the idea by @dhodovsk )

Solution for 1:
(Another vairant.)
lpb = LocalProjectBuilder(instances=[
            PagureService(token="asd", instance_url="pagure.io"),
            PagureService(token="asd", instance_url="src.fedoraproject.org"),
            GithubService(token="asd")])

lpb.get_local_project(url="http://url", git_project=CALCULATE)
lpb.get_local_project(url="http://another.url", working_dir=CALCULATE)

7\. No support for forks.

How we can achieve this? What API do we need?

I'd personally prefer a set classmethods which would accept a precise set of arguments and the calculate part would be 100 % deterministic

I'd personally prefer a set classmethods which would accept a precise set of arguments and the calculate part would be 100 % deterministic

Do you mean something like this?
lp = LocalProject.from_path_or_url(path_or_url="some-url")
lp = LocalProject.from_path_or_url_and_service(path_or_url="some-url", service=my_service)
lp = LocalProject.from_path(path="my/dir")
:
:

(+ variants with specific branch)
I am not sure if we can easily list all of the used combinations.

In your case, the handling of the service instances will be on top of this class?

lp = LocalProject.from_path_or_url(path_or_url="some-url")
lp = LocalProject.from_path_or_url_and_service(path_or_url="some-url", service=my_service)
lp = LocalProject.from_path(path="my/dir")

Exactly, I would even clearly distinct b/w from_path and from_url, and do the path_or_url magic in a dedicated function.

I am not sure if we can easily list all of the used combinations.

Yep, that would be pretty hard. That's why I debate if it's worth the effort.

In your case, the handling of the service instances will be on top of this class?

No idea, honestly.

ok

Feel free to reopen this issue if you wish the unleash the pandora box again.

We've discussed this a fair bit on Packit hackfest so I am reopening this issue, here are the main outcomes of the discussions:
The ways how we currently construct the object (inputs):

working_dir + ref
project URL + PR id
working_dir + git_url + instance of ogr project
working_dir + ogr project + ref + PR id

The methods of the class can be grouped into 4 categories:

Can be safely removed

git_checkout_block = unused context manager from the past

Can be removed with some effort (just wraps git.Repo operations)

stage
push
reset
rebase

Object construction:

refresh
_parse_*
...

convenience functions (mainly git helpers)

checkout_pr,
checkout_ref
commit
merge_pr
processing of diffs, ...

We should really remove 1) and 2) since they are pointless, this will slightly simplify the class. We agreed that separating the construction logic to a builder class sounds like the best way to go since it will also remove the whole 3) from the class itself making it contain only the git.Repo + remote API from ogr. As for convenience methods, some could stay, some could be taken out to a function, that's up to consideration.
This goes against the idea of auto-refresh as was mentioned in the opening comment of this issue, however this should not be a problem. If one wants to change something in the repo dynamically, constructing a new object using a builder (perhaps by passing to it the old object + something extra) should be easy enough.
As for the builder implementation, we should be explicit in specifying what we want the class to calculate to avoid this weird obfuscation that some magic calculations are done in the background. Take a look at the comments above for example builder calls.
@lachmanfrantisek anything else to add here?

Please, create a multiple pull-requests so the work is easier to review.


Sensors randomly stop reporting data & need integration reload

Describe the bug
I have a bunch of temp sensors throughout the house and they randomly stop reporting back to HassIO. They are working fine in the Tuya app.
I have to reload the integration to get the data to sync again

Expected behavior
Data to show in HassIO
Diagnostics for integrations and devices
config_entry-tuya-4f1fb717b5af9828c4c3d484b182bfe4.json.txt
Home Assistant Version
Supervisor 2023.10.0
Operating System 10.5
Frontend 20230911.0 - latest
Device info (please complete the following information, which can be found in log):
home-assistant.log
Device specifications (please complete the following information, which can be found in log):
home-assistant.log

I would say this is not just temp sensors. I see the same with human presence sensors. Work perfectly for a while and then stop in HA but reporting fine in the app.
Integration reload solves it for a while

same :(

same

Same problem. Need reload


Hand written generated code for sample template, registrar and registry

Added a sample template for report case
Added generated code (go and proto) for the sample template
Added a sample noop2 adapter that implements processort for the sample template
Added generated registrar code
Added generated registry code

TODO I have checked in two pb.go files. Ideally just the proto file should be sufficient. I will update this PR once I figure out how to use go_proto_library properly and delete the checked in pb.go files under pkg/templates
Next step will be to hand write generated code for

calling adapter config with adapter params,
calling adapter configure() with inferred types
calling handler process with instance objects

This change is

Jenkins job mixer/presubmit passed

Jenkins job mixer/pilot-regression passed

Jenkins job mixer/e2e-smoketest passed

Oh also everything outside of //adapter that we're using proto.Message should be replaced with adapter.Config as an alias.

Jenkins job mixer/presubmit passed

Jenkins job mixer/pilot-regression passed

Jenkins job mixer/e2e-smoketest passed

Jenkins job mixer/presubmit passed

Jenkins job mixer/e2e-smoketest passed

Jenkins job mixer/presubmit passed

Jenkins job mixer/pilot-regression passed

Jenkins job mixer/e2e-smoketest passed

@douglas-reid Friendly ping !!

Jenkins job mixer/presubmit passed

Jenkins job mixer/pilot-regression passed

Jenkins job mixer/e2e-smoketest passed

@douglas-reid Hey Doug, sorry for pinging on this again. Do you think I can submit this and you can continue to add comments on this PR (after it is merged. Not sure if it is possible). The reason for the hurry is, I have another PR that depends on this and has reached a state that is ideal to stop it and send for review.

Jenkins job mixer/presubmit passed


[stablehlo] fix ConvTranspose

This pull request fixes the original conversion of ConvTranspose to stablehlo. Speicifically, we should use outputPadding Attribute to calculate the padding for stablehlo.convolution, instead of doing conv then pad.

Jenkins Linux s390x Build #14377 [push] [stablehlo] fix ConvTran... started at 21:05

Jenkins Linux ppc64le Build #13372 [push] [stablehlo] fix ConvTran... started at 21:14

Jenkins Linux amd64 Build #14347 [push] [stablehlo] fix ConvTran... passed after 1 hr 15 min

Jenkins Linux s390x Build #14377 [push] [stablehlo] fix ConvTran... passed after 1 hr 31 min

Jenkins Linux ppc64le Build #13372 [push] [stablehlo] fix ConvTran... passed after 2 hr 2 min


[Hub Generated] Review request for Microsoft.Insights to add version preview/2017-11-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@msyyc Please review python SDK Breaking Change

@msyyc Please review python SDK Breaking Change

@ArcturusZhang Please review Go SDK Breaking Change

@ArcturusZhang Please review Go SDK Breaking Change

@kairu-ms the breaking changes are not introduced by this PR, therefore those are fine

@kairu-ms the breaking changes are not introduced by this PR, therefore those are fine

For breaking change is not caused by this PR

For breaking change is not caused by this PR

@kairu-ms so we can merge, right?

@kairu-ms so we can merge, right?


feat: wrap ShareProof in a ResultShareProof

Description
Contributes to https://github.com/celestiaorg/celestia-core/issues/1306

CI will be fixed with: https://github.com/celestiaorg/celestia-core/pull/1347

@staheri14 Will address the feedback in an upcoming PR. Thanks for the review


Hashtag link does not work

Jumping to https://docs.gny.io/http/api.html#_2-4-11-return-own-delegates-produced-blocks
does not work when reloading
This is not good when people link to a specific part of the document.

This issue has been reported.

https://github.com/vuejs/vuepress/issues/1499

It seems that we have to write a script to fix this issue.


Go.mod watch in addition to go files

I noticed you had a tool for general file watching but it would be nice if gow also watched the go.mod file for changes. Sometimes we forget to update a reference to a package and then push a change that just updates the go.mod.

Apologies for the late response.
This is similar to issue #2, but I could see supporting go.mod and possibly go.sum by default, since they're strictly Go-related. I'll kick it around locally to see if any issues come up.

Enabled watching *.mod files, used for a weeks, haven't noticed any issues. Just pushed an update to enable this for everyone. If this ends up problematic for some users, we might revert this. Anyway try it out, let me know how it works. 🙂

Sweet, I'll try to work it into our next sprint and testing cycle. I'll let you know if I run into any issues.


[BUG] - crc setup fails on setting up crc daemon in RHEL 8.7

General information

OS: Linux
Hypervisor: KVM
Did you run crc setup before starting it ? Yes
Running CRC on: Baremetal-Server

CRC version
CRC version: 2.12.0+74565a6
OpenShift version: 4.11.18
Podman version: 4.2.0

CRC status
DEBU CRC version: 2.12.0+74565a6
DEBU OpenShift version: 4.11.18
DEBU Podman version: 4.2.0
DEBU Running 'crc status'
crc does not seem to be setup correctly, have you run 'crc setup'?

CRC config
- consent-telemetry                     : yes
- preset                                : podman

Host Operating System
NAME="Red Hat Enterprise Linux"
VERSION="8.7 (Ootpa)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="8.7"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Red Hat Enterprise Linux 8.7 (Ootpa)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:8::baseos"
HOME_URL="https://www.redhat.com/"
DOCUMENTATION_URL="https://access.redhat.com/documentation/red_hat_enterprise_linux/8/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"

REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 8"
REDHAT_BUGZILLA_PRODUCT_VERSION=8.7
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="8.7"

Steps to reproduce

Run crc setup

Expected
Should exit without error
Actual
Throws error
Logs
DEBU CRC version: 2.12.0+74565a6
DEBU OpenShift version: 4.11.18
DEBU Podman version: 4.2.0
DEBU Running 'crc setup'
INFO Using bundle path /home/dhruvd/.crc/cache/crc_podman_libvirt_4.2.0_amd64.crcbundle
DEBU Checking if systemd-resolved.service is running
DEBU Running 'systemctl status systemd-resolved.service'
DEBU Command failed: exit status 3
DEBU stdout: * systemd-resolved.service - Network Name Resolution
Loaded: loaded (/usr/lib/systemd/system/systemd-resolved.service; disabled; vendor preset: disabled)
Active: inactive (dead)
Docs: man:systemd-resolved.service(8)
https://www.freedesktop.org/wiki/Software/systemd/resolved
https://www.freedesktop.org/wiki/Software/systemd/writing-network-configuration-managers
https://www.freedesktop.org/wiki/Software/systemd/writing-resolver-clients
DEBU stderr:
INFO Checking if running as non-root
INFO Checking if running inside WSL2
INFO Checking if crc-admin-helper executable is cached
DEBU Running '/home/dhruvd/.crc/bin/crc-admin-helper-linux --version'
DEBU Found crc-admin-helper-linux version 0.0.12
DEBU crc-admin-helper executable already cached
INFO Checking for obsolete admin-helper executable
DEBU Checking if an older admin-helper executable is installed
DEBU No older admin-helper executable found
INFO Checking if running on a supported CPU architecture
DEBU GOARCH is amd64 GOOS is linux
INFO Checking minimum RAM requirements
DEBU Total memory of system is<PHONE_NUMBER>76 bytes
INFO Checking if crc executable symlink exists
INFO Checking if Virtualization is enabled
DEBU Checking if the vmx/svm flags are present in /proc/cpuinfo
DEBU CPU virtualization flags are good
INFO Checking if KVM is enabled
DEBU Checking if /dev/kvm exists
DEBU /dev/kvm was found
INFO Checking if libvirt is installed
DEBU Checking if 'virsh' is available
DEBU 'virsh' was found in /usr/bin/virsh
DEBU Checking 'virsh capabilities' for libvirtd/qemu availability
DEBU Running 'virsh --readonly --connect qemu:///system capabilities'
DEBU Found x86_64 hypervisor with 'hvm' capabilities
INFO Checking if user is part of libvirt group
DEBU Checking if current user is part of the libvirt group
DEBU Current user is already in the libvirt group
INFO Checking if active user/process is currently part of the libvirt group
INFO Checking if libvirt daemon is running
DEBU Checking if libvirtd service is running
DEBU Running 'systemctl status virtqemud.socket'
DEBU Command failed: exit status 3
DEBU stdout: * virtqemud.socket - Libvirt qemu local socket
Loaded: loaded (/usr/lib/systemd/system/virtqemud.socket; disabled; vendor preset: disabled)
Active: inactive (dead)
Listen: /run/libvirt/virtqemud-sock (Stream)
DEBU stderr:
DEBU virtqemud.socket is neither running nor listening
DEBU Running 'systemctl status libvirtd.socket'
DEBU libvirtd.socket is running
INFO Checking if a supported libvirt version is installed
DEBU Checking if libvirt version is >=3.4.0
DEBU Running 'virsh -v'
INFO Checking if crc-driver-libvirt is installed
DEBU Checking if crc-driver-libvirt is installed
DEBU Running '/home/dhruvd/.crc/bin/crc-driver-libvirt version'
DEBU Found crc-driver-libvirt version 0.13.5
DEBU crc-driver-libvirt is already installed
INFO Checking crc daemon systemd service
DEBU Checking crc daemon systemd service
DEBU Checking if crc-daemon.service is running
DEBU Running 'systemctl --user status crc-daemon.service'
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: Failed to get properties: Transport endpoint is not connected
DEBU Could not get crc-daemon.service  status: Executing systemctl action failed:  exit status 1: Failed to get properties: Transport endpoint is not connected
DEBU Checking if crc-daemon.service has the expected content
INFO Checking crc daemon systemd socket units
DEBU Checking crc daemon systemd socket units
DEBU Checking if crc-http.socket is running
DEBU Running 'systemctl --user status crc-http.socket'
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: Failed to get properties: Transport endpoint is not connected
DEBU Could not get crc-http.socket  status: Executing systemctl action failed:  exit status 1: Failed to get properties: Transport endpoint is not connected
DEBU crc-http.socket is not running
INFO Setting up crc daemon systemd socket units
DEBU Setting up crc daemon systemd socket units
DEBU Setting up crc-http.socket
DEBU Running 'systemctl --user status crc-http.socket'
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: Failed to get properties: Transport endpoint is not connected
DEBU Could not get crc-http.socket  status: Executing systemctl action failed:  exit status 1: Failed to get properties: Transport endpoint is not connected
DEBU Starting crc-http.socket
DEBU Running 'systemctl --user enable crc-http.socket'
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: Failed to enable unit: Transport endpoint is not connected
Executing systemctl action failed:  exit status 1: Failed to enable unit: Transport endpoint is not connected

Are you running this through ssh? or are you directly connected to that machine? I expect systemctl --user status returns the same error?

I am running this through ssh. systemctl --user status return the same error. That makes me think, is it not a crc issue?

@desaidhr try workaround which is mentioned https://github.com/crc-org/crc/issues/2690#issuecomment-912505017 I think we do need to put that in the docs in case someone want to setup crc using ssh.

@praveenkumar I tried loginctl enable-linger $USER as suggested. Its not working. Gives the same error

@desaidhr Does the systemctl status also fail on that system or it is only failing for --user context?

@desaidhr Can you check systemctl status systemd-user-sessions.service if it is running? If not then can you try to start and enable it?

yes the said service is active
 systemd-user-sessions.service - Permit User Sessions
   Loaded: loaded (/usr/lib/systemd/system/systemd-user-sessions.service; static; vendor preset: disabl>
   Active: active (exited) since Thu 2023-01-05 22:08:37 IST; 13h ago
     Docs: man:systemd-user-sessions.service(8)
 Main PID: 3698 (code=exited, status=0/SUCCESS)
    Tasks: 0 (limit: 3355442)
   Memory: 0B
   CGroup: /system.slice/systemd-user-sessions.service

Jan 05 22:08:37 localhost.localdomain systemd[1]: Starting Permit User Sessions...
Jan 05 22:08:37 localhost.localdomain systemd[1]: Started Permit User Sessions.

Can you paste the output of loginctl user-status $USER ?

@desaidhr did you make any kind of configuration setting because on a clean 8.7 installation this is not reproducible? Can you provide the info around how did you install and configure the system?

Other than a proxy setting to enable internet connectivity, the machine doesnot have anything custom. RHEL was installed from the iso downloaded from redhat site. It is registered and is updated.

I faced a similar issue with RHEL-8.9. Fixed by changing /etc/ssh/sshd_config to:
X11Forwarding yes
X11DisplayOffset 10
X11UseLocalhost yes

and using ssh -X user@host to log in to the RHEL server.
Then systemctl --user status state was changed to running and crc setup completed successfully.

https://github.com/crc-org/crc/issues/2690#issuecomment-912505017 could also help with this.

@desaidhr can you try https://github.com/crc-org/crc/issues/3471#issuecomment-1687715057 one if that fix your issue?

@praveenkumar Thanks for reaching out. I have moved on from this thread and no longer have the setup to validate.

I have moved on from this thread and no longer have the setup to validate.

@desaidhr In that case, I am going to close this issue, Please reopen if you hit it again.


app update deletes all other apps

[public]

Status Change: In Review

Work on this ticket is complete and is now pending review.
[created from NEXT-39589, comment 512495]

[public]

FixVersion Change: <IP_ADDRESS>

This issue will be / was solved in Shopware version <IP_ADDRESS>.
[created from NEXT-39589, comment 512496]

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39589
[created from NEXT-39589, comment 512501]


How to set NewLine?

I tried data-balloon="Test test & # 13;& # 10; Test Test.\A Test Test  & # x a; Test Test" - but no one of them helps.
How can I break text in ballon to new line?
Thank you.

@ekatherina there are two ways to break lines in Balloon tooltips:
Option 1: Manually adding line-breaks:
You need to write the code
 without spaces, and using the attribute data-balloon-break. E.g.
This is a tooltip

Option 2: Using data-balloon-length to break lines automatically:
Additionally you can also let the tooltip break automatically using sizes:
This is a tooltip

The values for data-balloon-length are: small, medium, large and fit.

This is a tooltip

doesn't seem to work

If someone is doing this programatically, instead of
 use \n

I try repeat code, but I have extra commas

https://codesandbox.io/s/cool-wood-kso0e

I was using balloon.css in EJS
Simply putting attribute as data-balloon-break     didn't worked for me (strange) . Setting  data-balloon-break=""    worked perfectly✔  with '\n'


soroban-rpc: Add opt-in remote metrics push so we can collect usage data

What problem does your feature solve?
As SDF, we would like to know how the deployed soroban-rpc instances are running. What kind of resources they have available, and what sort of load they are dealing with.
What would you like to see?
An opt-in option for operators, which would periodically push their metrics to a remote target (e.g. a prometheus pushgateway server we're running to collect them).
We probably want to use: https://pkg.go.dev/github.com/prometheus/client_golang/prometheus/push
We should be able to tell:

Total number of RPC instances by

network (futurenet, testnet, mainnet)
locale

Top endpoints by request signature (ie: /getEvents, /getLedgerData)
Average request latency by endpoint
Total unique Users per RPC instance
Top error codes and failures
Pseudorandom identifier that would help us to identify the rpc instance
Instance uptime
Duration
Request rate
errors

Open questions?

How can we prevent/discern corrupt/false data being sent?

This is the metrics equivalent of: https://github.com/stellar/soroban-tools/issues/502


Fullstack v0.5.6: POST api and reverse proxy setup, complicated.

Hi,
Feel free to convert to a feature request, bug I consider this an issue right now.
1. Controlling the appended value to the /api/ calls
Using the latest up to date stable Dioxus fullstack, after controlling the network exchanges between the client and the server,  I noticed that every time the client does a request, the network packet looks like (excerpt from wireshark):
884	0.715949171	<IP_ADDRESS>	<IP_ADDRESS>	HTTP	679	POST /api/my_server_function4227210978949717125 HTTP/1.1  (application/x-www-form-urlencoded)
This is an issue when setting reverse proxies between the client and the axum server because it forces to discover the address and update the proxy configurations accordingly.
We should have an option for this that would acknowledge this possibility + a way not to append the unique numbers<PHONE_NUMBER>949717125 by two ways:

either by having the dev set the random number by himself :

#[server]
#[dioxus_tag = 123456]
pub(crate) async fn my_server_function()->Result<(), ServerError>{}

that would lead to
884	0.715949171	<IP_ADDRESS>	<IP_ADDRESS>	HTTP	679	POST /api/my_server_function#123456 HTTP/1.1  (application/x-www-form-urlencoded)
Cons: easy to forget. In case it's not present, it leads to issues if the apps on a same server have the same server name.
Not great.

give a random number while creating the app using dx new or init, and set it in the dioxus.toml:

[application]
unique_tag = "value generated by dx tool here" -> e.g "123456"

884	0.715949171	<IP_ADDRESS>	<IP_ADDRESS>	HTTP	679	POST /api/123456/my_server_function HTTP/1.1  (application/x-www-form-urlencoded)
This is better because nothing more has to be done on the dev side, no overlook.
And this value can be modified by the dev.
Cons: if modifiable, that would imply some machinery to update the all the related components.
2. And an option to prepare for a reverse proxy in between in the dioxus.toml:
[proxy]
address = ["w.x.y.z", 443]

And an option for the axum server (port 8965 given arbitrarily):
[server]
address = ["a.b.c.d", 8965]

I know v6.x has a port and address dynamic configuration via the command line, but this is more comfortable when multiple people have to check the configuration.
That would give the address of the proxy so that the client makes the request there and settle the Axum server to its own address.
This way, the dev can set the reverse proxy in between, which can be anything (leave it to the dev) and help checking the configuration in config files instead.
The reverse proxy can then be acknowledged to send the request at the listening servers addresses that are listed in dioxus.toml (ssl termination, url rewrites anything, but that is out of focus of Dioxus).
Maybe there is some machinery that I somehow overlooked to make this work but this is the 2 issues/feature requests we are having.
If you have any other questions, please tell me!

You can set the endpoint parameter in the server function macro to control the url the server function is served at:
#[server(
  endpoint = "endpoint123",
)]
pub async fn my_wacky_server_fn(input: Vec<String>) -> Result<usize, ServerFnError> {
  todo!()
}

That's great, completely overlooked it!
Thank you


Question about Dice function

Your code works very well, and I have been using and also modified for many projects.
However i just realized today that Dice and IOU are both non-differentiable function.
Then my question is, why it works in your code?
Thanks!

It's actually possible to make them differentiable. Dice and IoU both use intersection and union (which work basically with sets), but these can be approximated in real domain with * and + if you make sure your outputs are in range [0,1].

Yeah, you are right, I also searched for answers on google. Somebody did indicated the output shoud be within[0,1], and somebody else propose some 'soft' dice.
But I still don't quite understand why. I know this is definitely right, but I really need a example or some fomula.
Can you explain more details or are there some papers I can refer to?
Thanks!

I'm not aware if there is a paper or some resource which I can refer you to.
However, usually when I find myself in such confused state, I take a very simple example and do everything by hand.
I suggest you make up binary 3x3 mask and 3x3 prediction that is slightly off (i.e. 1 pixel wrong). Then derive gradients of Dice loss and you should be able to see more clearly what's going.
Of course, this takes time, but once you get it right you know it for life :).

@jocicmarko Ok, I'll try it. Thank you very much


Title should be always a string

Even when the author put in a number
Found this bug while creating the benchmark for https://github.com/middleman/middleman-blog/pull/350

Coverage remained the same at 92.8% when pulling dbf8eff507699f8ee4040750625f6630271dc06a on dtorres:once-a-title into e495bbebfac3d10b3093b9b4e6a9fb30f4e735d3 on middleman:master.

Coverage remained the same at 92.8% when pulling dbf8eff507699f8ee4040750625f6630271dc06a on dtorres:once-a-title into e495bbebfac3d10b3093b9b4e6a9fb30f4e735d3 on middleman:master.

Coverage remained the same at 92.8% when pulling dbf8eff507699f8ee4040750625f6630271dc06a on dtorres:once-a-title into e495bbebfac3d10b3093b9b4e6a9fb30f4e735d3 on middleman:master.

@dtorres thanks


Feature Request: Enable Audience mode regardless of 50Mbit/s mode

One of the most dreaded issues when flying in a group is being unable to join in with the audience mode when the goggle is still set to 50mbit/s mode. Connecting a quad usually is not a good idea as it will probably cost a pilot his signal.
I think it would probably be an easy patch to enable 25/50 mbit/s switching regardless of being connected to a vista/air unit. This way the patch could also just always allow audience mode and switch to 25mbit/s when selecting that.

The algorithm to fix this.
`setprop dji.glasses_service 0
write 0x00 to address 0xc0 in /blackbox/glass_user_setting.dat
setprop dji.glasses_service 1`
and some logic when.

If you want to test an early experimental version. Not sure it'll work on V2s. Hold down the "channel down" button for 8-10s to switch.
https://github.com/funneld/djifpv_offline_audience_switcher/releases/tag/v0.0.2


Add bun bundler

It would be interesting to see whether bun build is capable of bundling any of the libraries and how well it does.

PR welcome!


[3.x] Laravel 7 Support

This is a potentially breaking update.  With Laravel 7 there is new min requirements of PHP 7.2.5 (https://laravel.com/docs/7.x/upgrade#upgrade-7.0).
There is no way for dingo/api to work with PHP < 7 with implementing Throwable, not sure if that's an issue.
Note: dingo/blueprint has been removed as a core dependency. Ideally it wouldn't be a core package anymore and just exist as an optionally add-on. For now it can be re-added once there's a version with l7 support.

Hi laravel/lumen 7 not install here, that why no error occurs.
dingo/blueprint need first to supprt laravel 7

you can check installed laravel version on your local, before running test

Good work on this
I believe the blueprint package is needed for the docs generation, so we cannot drop that at this stage. Can you please put it back in, we will need to make it laravel 7 compatible also.
Also fix conflicts.
I created a new branch to work on laravel 7 compatibility, will get this merged into it.

I tagged blueprint as v0.4.0 for laravel 7.
https://github.com/dingo/blueprint/releases/tag/v0.4.0

Great, thanks !


Adds Support to Mini VIX (VXM) Futures

Expected Behavior
Support VIX Mini Futures.
Actual Behavior
Not supported.
Potential Solution
See contract specifications on:
https://cdn.cboe.com/resources/futures/VXM_Contract_Specifications.pdf
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Please implement this. VX is too big for most accounts. Should be easy to support as it's just 1/10th of VX


Pass block meta info into helper lookup

Enables local lookup for helpers
cc @krisselden @rwjblue

I think CI just flaked here.

@chadhietala no typescript check failed. 1) compiler is defined as some other type that does not define symbolTable 2) even if compiler was a Compiler, symbolTable is private. Need to figure out a better way to expose the meta.

Got things green


Package is not resolving

I tried to install library via SPM but not able to because of Package is not resolving

Fixed.
Try with version 0.0.2

Fixed. Try with version 0.0.2

thanks it worked
is it possible to stream whole view instead of camera only?

Hello,
Currently the library is a bit limited. You can stream camera or screen. You have a screen example, you can try with that and init the camera using CameraManager class.

I am Not able to find screen share example

https://github.com/pedroSG94/RootEncoder-iOS/blob/master/app/ScreenSwiftUIView.swift

https://github.com/pedroSG94/RootEncoder-iOS/blob/master/app/ScreenSwiftUIView.swift

connection failed: The operation couldn’t be completed. (RootEncoder.IOException error 0.)
TEARDOWN rtsp://<IP_ADDRESS>:8080/stream/id/live RTSP/1.0
getting this error while starting screen share session


Fix 404 link in README

Summary

Test plan

A picture of a cute animal (not mandatory but encouraged)

Deploy preview for netlify-cms-www ready!
Built with commit ed42d6ca1d5a5b26e9fce328ece16e6968ab23c3
https://deploy-preview-1853--netlify-cms-www.netlify.com

Deploy preview for cms-demo ready!
Built with commit ed42d6ca1d5a5b26e9fce328ece16e6968ab23c3
https://deploy-preview-1853--cms-demo.netlify.com

When I test the 'start with a template' link it doesn't appear to be broken. That link should point towards the internal docs page instead of the external templates.netlify.com website.


[Event Request] Codeunit 7326 "Whse. Item Tracking FEFO" OnAfterCreateEntrySummaryFEFO

I would like to request a new event to allow us to manipulate the list of viable lot or serial numbers before displaying them to the user.
This would be last row in the CreateEntrySummaryFEFO function.

Please provide event definition and list of required parameters

Please see images for event details and call. I would like the global variable TempGlobalEntrySummary passed to the event as a variable so we can manipulate the entries and return them.


Attempted import error: 'useIsFocusVisible' is not exported from '@material-ui/core/utils'.

My project was working well until today when I updated the library. The error I receive is
<EMAIL_ADDRESS>Attempted import error: 'useIsFocusVisible' is not exported from '@material-ui/core/utils'.

[ x] This is not a v0.x issue.
[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

Build should not fail
Current Behavior 😯

Build fails
Steps to Reproduce 🕹

Link: codesandbox

npx create-react-app my-app
cd my-app
npm install @material-ui/core @material-ui/icons @material-ui/lab
npm start

Context 🔦

Lift a backoffice in React
Your Environment 🌎

Tech
Version

Material-UI
v4.0.0

React
v16.8.3

Browser
Chrome or Firefox

This problem does not happen in version 4.0.0-alpha.15 working example

@elgambet
Hi!
I found this problem in using @material-ui/core": "4.0.0" and @material-ui/lab": "4.0.0-alpha.16". I check that @material-ui/core/util/index export useIsFocusVisible  in the latest code.
https://github.com/mui-org/material-ui/blob/master/packages/material-ui/src/utils/index.js
well, you can fix it by updating  @material-ui/core

@TatsuyaYamamoto is correct (thanks). You should see a peer dependency warning in your console when installing the lab package. A working version: https://codesandbox.io/s/material-demo-hd1l0.
The lab is in an alpha state, you might see breaking changes between two updates. It's what happens between alpha.15 and alpha.16 for the Slider component. Now, it should still work in the most simple use cases without any changes required.