Oh My Zsh not enabled when doing su (root)

I install this in my username and i when i do su : root (there is no themes)..

Use sudo -E -s to log as root, that way you don't have to install OMZ for root (which is not recommended). I have that aliased:
alias suroot='sudo -E -s'

@mcornella your solution didn't work for me.
Can someone please help me use zsh as root?

 jase@yxia  ~  sudo -E -s
[sudo] password for jase:
[oh-my-zsh] Insecure completion-dependent directories detected:
drwxr-xr-x  11 jase users  4096 Jul 17 20:59 /home/jase/.oh-my-zsh
drwxr-xr-x 250 jase users 12288 Jul 17 20:59 /home/jase/.oh-my-zsh/plugins
drwxr-xr-x   2 jase users  4096 Jul 17 20:59 /home/jase/.oh-my-zsh/plugins/git

[oh-my-zsh] For safety, we will not load completions from these directories until
[oh-my-zsh] you fix their permissions and ownership and restart zsh.
[oh-my-zsh] See the above list for directories with group or other writability.

[oh-my-zsh] To fix your permissions you can do so by disabling
[oh-my-zsh] the write permission of "group" and "others" and making sure that the
[oh-my-zsh] owner of these directories is either root or your current user.
[oh-my-zsh] The following command may help:
[oh-my-zsh]     compaudit | xargs chmod g-w,o-w

[oh-my-zsh] If the above didn't help or you want to skip the verification of
[oh-my-zsh] insecure directories you can set the variable ZSH_DISABLE_COMPFIX to
[oh-my-zsh] "true" before oh-my-zsh is sourced in your zshrc file.

So... I am not clear on how to fix this without breaking anything... since you said it's not good to install Oh my ZSH as root.

Add this to .zshrc
su() {
	SUPARAM="su $*"
	bash -c "$SUPARAM"
}

If any issue run reset and retry...
Note: this is for bash... zsh need to be run after su...


CHAOS-7089 Command to enable workload identity in node pool

Command to enable workload identity in node pool
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673c65dff1beb4090541797f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673d80141d9cb1f959c9fc33--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673d82b300c64209e8219307--harness-developer.netlify.app


Added additional inbound sound files that reported errors on the server

我为模组添加了额外的音效文件 并确保格式正确 且在单人模式下可以正常被广播器播放   但放在服务端上会报错 无法选择保存音频至广播器
为此我又打算为其他用不上的音效 以替换文件名字的方式放进模组里
以上操作都经过了sounds. json重新添加 并且确认过名字与格式没有问题
但是在服务端上运行 使用音频依旧报错！
I added extra sound files to the module and made sure that they were formatted correctly and could be played by the broadcaster in single player mode, but when placed on the server there was an error and no option to save audio to the broadcaster
For this reason, I decided to replace the file name in the mod for other sound effects that I didn't need
The above operations have been re-added by sounds.json and the name and format have been confirmed
But running on the server using audio still reports errors!
latest.log

能提供一下 json 文件吗
似乎命名空间路径里有非法字符？

Of course. Sorry, it took me so long to see the message and reply to you.
Here is the json file
sounds.json
The following is a sound asset
sounds.zip
Due to size restrictions, only the assets I added are included here.

Minecraft by default does not allow Capital Letter as an identifier, you should name your sound 10line_xihaian instead of 10line_XiHaiAn.
This is a limitation imposed by Minecraft itself (And it doesn't work here on singleplayer either).
I suspect some mod you have installed on your client bypassed this limitation, for it to work universally your sound id must be lower case.


Decide on casing for interfaces.

xlang_unknown uses pascal casing today.  While the other PAL functions use snake casing.  We should come to an agreement on what to use for interfaces and do that throughout.  We should also decide on how interfaces are named and how casing is for struct members.

Yes, this has been bothering me for a while. The PAL for the Windows Runtime (along with all APIs) uses CapitalizedWords for all names, functions, and interfaces. It's only the C++ language projection that uses lower_case for non-projected names. It does so to distinguish between the Runtime/APIs and the library support provided by C++/WinRT.
I would find it far less confusing if the xlang PAL used CapitalizedWords throughout. Typically, language projections hide the PAL anyway, so it should not affect any personal choice made by any particular language. It's also very very confusing to see COM interfaces that are named using lower_case.

There's only one solution: ALL_UPPER_CASE.
After all, it's not real programming unless it looks like COBOL.
;)


feat: LSDV-5242: MVP for dynamic LLM chat

Hi @hlomzik!
This PR was created in response to a PRs in upstream repo:

https://github.com/heartexlabs/label-studio-frontend/pull/1438

/git merge develop


[Icon Request]: Kiwix Offline Reader

What is the name of the icon?
Kiwix
What is the website/page of the icon?
https://en.wikipedia.org/wiki/Kiwix
https://www.kiwix.org/en/

I've added the icon!


Enhance Values List to validate column type

🚧 WIP
This is subsequent PR of #648
Goal

Values List does not have schema so that there is no minimum validation for each column types yet
If there is no NULL, the first row will create column types
If there is NULL at first row, the next row having type will create colum type
How can we handle if specific column has NULL at all rows?

Todo

[x] get mutable column_types base
[x] get indexes of None
[x] if new row has type at those indexes, renew column_types
[x] if there is no None in column_types, break
[x] validate and cast type with try_info_value
[ ] refactor values_list into row with row::new

[ ] split common codes
[ ] blend column_defs from column_types, labels

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

73 of 88 (82.95%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 97.74%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/data/value/mod.rs
6
21
28.57%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
28975

Relevant Lines:
29645

💛  - Coveralls


kuma: 1.4.0 -> 1.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/kumahq/kuma/releases.
meta.description for kuma is: "Service mesh controller"
meta.homepage for kuma is: "https://kuma.io/"
meta.changelog for kuma is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1/bin/kuma-dp had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1/bin/kuma-cp had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1/bin/kumactl had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1/bin/coredns had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1/bin/kuma-prometheus-sd had a zero exit code or showed the expected version
0 of 5 passed binary check by having a zero exit code.
0 of 5 passed binary check by having the new version present in output.
found 1.4.1 with grep in /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1
found 1.4.1 in filename of file in /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1

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

12 total rebuild path(s)

6 package rebuild(s)

6 x86_64-linux rebuild(s)
6 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
kuma
kuma-cp
kuma-dp
kuma-experimental
kuma-prometheus-sd
kumactl

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1
ls -la /nix/store/ll3hkkginvawab01nx21mkygicljqx91-kuma-1.4.1/bin

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

6 packages built:

kuma
kuma-cp
kuma-dp
kuma-experimental
kuma-prometheus-sd
kumactl

Maintainer pings
cc @zbioe @kalbasit for testing.

Closing so the bot resubmit with an updated vendorSha256.


auto-download for marimo ipynb

Description
We already support auto-exporting HTML. We should add auto-exporting for ipynb in top-down order with the outputs, so that users can commit these files to github and view them natively.
Suggested solution
An additional option in this dropdown

Alternative
No response
Additional context
No response

How do you feel about supporting Quarto?
Instead of generating a complicated JSON, you could be generating a git-friendly .qmd markdown file.  Much simpler to compose and maintain.  I can help with that.
Quarto then can take care of IPYNB generation - .ipynb <-- quarto convert --> .qmd (it works via pandoc).  QMD files can also run directly like notebooks via VS Code plugin.
Either in addition to this, or as an alternative, how about:

set up an API for generating .qmd files; it needs a frontmatter template (YAML)
give an option for also saving .ipynb via quarto convert (also in file manager)

Ditto for #2960 - it's already solved in Quarto

A community member (@dmadisetti) has developed an experimental Quarto extension but it hasn't gotten much use yet I believe: https://github.com/dmadisetti/quarto-marimo
The main point of supporting ipynb is two-fold:

View notebook with outputs on GitHub (qmd doesn't render on GitHub I believe).
Compatibility with ecosystem of tooling built around ipynb files (such as static site generation and inclusion in docs), to make it easier for existing Jupyter users to adopt marimo.

In this case the point isn't to be git-friendly. If I understand correctly, qmd files don't include outputs like plots, etc, making them git-friendly but not suitable for output snapshots.

Check out #2994


Use twirapp for docker registry

Currently it's under satont scope, i'd like to change all docker images to twirapp account, which i created for such purposes.

Done https://github.com/twirapp/silero-tts-api-server/commit/695f6242e5334eb8fc01160c7451be1fa3575c16


An error in detailed_workflow.ipynb

🐛 Bug Description

To Reproduce
The keyword-"time_per_step" is missing in port_analysis_config
"executor": {
"time_per_step"
"class": "SimulatorExecutor",
"module_path": "qlib.backtest.executor",
"kwargs": {: "day",
"generate_portfolio_metrics": True,
},
},
Expected Behavior
It should be as follows
"executor": {
"class": "SimulatorExecutor",
"module_path": "qlib.backtest.executor",
"kwargs": {"time_per_step": "day",
"generate_portfolio_metrics": True,
},
}
Screenshot

Environment
Note: User could run cd scripts && python collect_info.py all under project directory to get system information
and paste them here directly.

Qlib version:
Python version:
OS (Windows, Linux, MacOS):
Commit number (optional, please provide it if you are using the dev version):

Additional Notes

"executor": {
"class": "SimulatorExecutor",
"module_path": "qlib.backtest.executor",
"kwargs": {"time_per_step": "day",
"generate_portfolio_metrics": True,
},
},

This is been fixed by https://github.com/microsoft/qlib/pull/1084

I confirmed it has been fixed.


[Codebase improvement] Reuse handlers devopsProject to devops

reuse  handler devopsProject to devops. and queryParam to query

@Feeeenng , thanks for your first contribution.
/lgtm
/cc @kubesphere/sig-devops

Thanks for your contribution. But please see the following warning (which come from Goland)

IMO,  some people use VSCode as their IDE which lint tool is golangci-lint, and no warning here due to .golangci.yaml agreed.
If we don't want to see the warning again, please configure lint tools in IDEA.
So I think there is no problem.


Regex catalog query fails

This query results in a 500 error:
{"$or":[{"name":{ "$regex":"forest", "$options": "i"}},{"entity.description":{ "$regex":"forest", "$options": "i"}}]}

This has been fixed. BTW this is a bug in the mongo-query-to-postgres-jsonb library. I raised an issue for this: https://github.com/thomas4019/mongo-query-to-postgres-jsonb/issues/28. The change here is a workaround


Add retries for getting yamls from Github

There are several places where we pull manifests from Github (i.e. here, here, and here, though there may be others as well).
Currently, the plugin only tries getting the manifest once, but to make the plugin more resilient we should add retries to mitigate against network issues that may cause a single try to fail.

/assign @psschwei


add db_file_storage to file_storage in Form wizard

Is there a way to get it work with Form wizards ?, I just added in my Form wizard class as:
file_storage = DatabaseFileStorage()
It raised this error:
list index out of range
The exception came from the method exists at line 104 in storage.py
I thought it was because I have to add the prefix like in upload_to: app_name/class/bytes/filename/mimetype, but I didn't know how to get it work.
If this issue doesn't go here, please drop it.
Thanks in advance.

Hi!
db_file_storage doesn't work with Form Wizards yet.
Thanks for the report! I am working on that for the next release.

@victor-o-silva
Thanks in advance.
Regards.

closed by #15
@rid3r77 , now there is a class suitable for use in Form Wizards.
It's on the version 0.3.1 of the library, already on PyPI.
There is a section in the docs for this new functionality.


[Proposal] Support Unix domain socket

In what area(s)?

/area runtime

/area operator

/area placement

/area docs

/area test-and-release

Describe the proposal
Now Dapr can work well with Service Mesh, while once use Service Mesh, one extra jump cannot be avoided.
If Dapr supports Unix domain socket to passthrough to Service Mesh, there would be a great acceleration, as result in https://stackoverflow.com/questions/14973942/tcp-loopback-connection-vs-unix-domain-socket-performance said.
Also, if Dapr listens to Unix domain socket, APP can passthrough to it then make acceleration too.

I agree, we recently looked into this. If you can create a POC and post your findings here that would be awesome.

This seems to be a duplicate of https://github.com/dapr/dapr/issues/2864


Clarify versionPattern documentation

It's not immediately clear what the versionPattern field is needed for. I understand that in the surface it allows the tool to identify each version section in the CHANGELOG.md file, but why does it need to do that? I thought this action just checked whether the CHANGELOG.md file is edited in a PR.
Also, and perhaps more importantly, the line "Your regex should match the version as the 2nd match group" in the versionPattern documentation doesn't seem correct. I'm not sure exactly what is meant there, but for example, the regex we use here doesn't even have a 2nd group (since we didn't use the (v|V)? part of the default regex).

I've left some comments there. Thanks for undeprecating it, I do agree that's the right call :)


error[E0433]: failed to resolve. Could not find Group in proc_macro

Hi,
i´m following your wasm tutorial on JavaScript to Rust and Back Again and i got this compiling error.
I'm quite new to rust and i ain't got a clue what to do here.
> cargo +nightly build --target wasm32-unknown-unknown
   Compiling proc-macro2 v0.3.5
error[E0433]: failed to resolve. Could not find `Group` in `proc_macro`
  --> C:\Users\workingj\.cargo\registry\src\github.com-1ecc6299db9ec823\proc-macro2-0.3.5\src\unstable.rs:63:45
   |
63 |                 let mut group = proc_macro::Group::new(delim, tt.stream.inner.0);
   |                                             ^^^^^ Could not find `Group` in `proc_macro`

error[E0433]: failed to resolve. Could not find `Op` in `proc_macro`
  --> C:\Users\workingj\.cargo\registry\src\github.com-1ecc6299db9ec823\proc-macro2-0.3.5\src\unstable.rs:72:42
.
.
.
error: aborting due to 36 previous errors

Some errors occurred: E0433, E0599.
For more information about an error, try `rustc --explain E0433`.
error: Could not compile `proc-macro2`.

To learn more, run the command again with --verbose.

now it works after rustup update nightly...


Is it possible to get the path of genome in GFA format file?

Hi,
I test ggcat with two similar genomes. From new version, I can get the GFA format file. But it only contains node information. I would like to know if it is possible to obtain edge connectivity information and the most important genomic pathways, just like standard GFA spec format files.
Best
Wenhai

Hi, yes you can obtain edge connectivity information with the -e or --generate-maximal-unitigs-links flag.
This will emit link information using gfa L fields
Andrea

@Guilucand: Should we also add this info next to the description of the argument
-h, --gfa                               Output the graph in GFA format ?


fix: Improved resolveFile performance

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Enable the default editorconfig support again

Hi, Summary
Removed the hardcoded --no-editorconfig option. This enables editorsupport again.
Test Plan
Tested on nvim 0.3.0 and prettier 1.13.0 with JavaScript.
Closes #141

thank you!


fixed security bug

fixes: CWE-601

Checklist:

[x] Add or update the relevant docs.
[x] Add an entry in CHANGELOG.md summarizing the change.
[x] Run tests, with no tests failed.

Veracode Static Analysis Pipeline Scan is broken, all other checks are passing


SSH password for private key file

Hi, when i add SSH connection and add my private (password-protected) Key-File, there is no option to enter a password for the key file.
So when i open this connection, i get
Using user's default private key: H:.ssh\id_rsa
Loading private key from H:.ssh\id_rsa
Cannot parse privateKey: Encrypted private OpenSSH key detected, but no passphrase given
I like the Terminus but it's basically unusable for me if this does not work.
Kind regards,
Ben

Just tested - confirm there is no password prompt for password-protected private keys.
Also - it would be great if the password for keys can be saved and auto-added.


Enable UDP connections

Hi,
I have been trying to find a way to setup voice over multiplayer proxy however due to udp ports not being open I have been unable to. Is it possible to be able to forward UDP traffic either on the official proxy or through possibly (relating to #31) hosting my own.
I think its currently possible to forward UDP through the UPNP connection with modifications to the source however since UPNP is unsupported on my router I have been unable to test. I haven't been able to find anything that might make the proxy work though.

The proxy/tunnel server would need to support it. Simple Voice Chat support is something I'm considering.

thanks for the quick response, personally I am more interested in plasmo voice chat support and I do believe it might be a bit easier to implement. However any voice chat support is good, best of luck in creating this.

Hi, I'm looking to use simple voice chat with world host as well, was this ever figured out?

i hope this is added


RUMM-1014 Tracing - do not add RUM context if RumFeature is disabled

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #479 (f4bcb6a) into master (0b44dd3) will increase coverage by 0.10%.
The diff coverage is 87.50%.

The diff coverage is 87.50%.


Update experiments.json

Codecov Report

Merging #7203 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7203      +/-   ##
==========================================
- Coverage   58.53%   58.45%   -0.09%
==========================================
  Files         486      486
  Lines       21460    21460
  Branches     3463     3463
==========================================
- Hits        12562    12544      -18
- Misses       8117     8133      +16
- Partials      781      783       +2

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
76.47% <0%> (-11.77%)
:arrow_down:

src/client/common/platform/pathUtils.ts
66.66% <0%> (-5.56%)
:arrow_down:

src/client/linters/pydocstyle.ts
84.61% <0%> (-5.13%)
:arrow_down:

src/client/common/logger.ts
68.55% <0%> (-4.41%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
78% <0%> (-4%)
:arrow_down:

src/client/interpreter/locators/helpers.ts
70.73% <0%> (-2.44%)
:arrow_down:

src/client/common/process/proc.ts
13.28% <0%> (-1.57%)
:arrow_down:

src/client/datascience/constants.ts
99.49% <0%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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


Password Caps

When typing in the password field with caps lock on, it should show that the caps lock is on.

This was added with the "show-caps" attribute. Closing.


Segregate the employees by roles

Is your feature request related to a problem? Please describe.

Right now, all of the employees are displayed on the employee page

Describe the solution you'd like

Separate the employees into the following Roles Employees, interns, contractors, and support staff

Things to do

[ ] Add the individual tabs for Employees, Interns, Contractors, and Support Staff. These tabs can be placed next to Employee blue tab. Also, add the icons in the tabs
[ ] Add two new roles in the system, Contractors and Support Staff
[ ] Each employee will have these roles assigned by User Management
[ ]  On Click of the tabs displays the employees according to their roles.

This issue PR is merged. Closing it.


Refactor and updates for Wagtail 5.2

Unable to move forwards with this until https://github.com/yakupadakli/python-unsplash/pull/14 has been merged and a new PyPI version released.

/describe

@CodiumAI-Agent /describe

@CodiumAI-Agent /describe

@CodiumAI-Agent /review

Persistent review updated to latest commit https://github.com/CamLamb/wagtail-unsplash/commit/08d9c9267ae2f798dae4ccb7abf7f19b96af485a

@CodiumAI-Agent /improve

This PR worked great for me. I installed it on Django 5.0.2 and Wagtail 6.0.1 (after updating pyproject.toml requirements) and it worked great. Would love to see this accepted and packaged up.

This PR worked great for me. I installed it on Django 5.0.2 and Wagtail 6.0.1 (after updating pyproject.toml requirements) and it worked great. Would love to see this accepted and packaged up.

Thanks for taking a look at this and trying it out!
I'm currently on holiday, but I will make sure to get this merged in and published when I get back!

Released as part of 1.0.0


add context-aware method to enable timeout + cancel for connections

@practice-golang, I couldn't push to your branch, so I've pulled your changes into a branch here. Some minor modifications have been made, but I'd like you to verify (if possible) that the sending still works as expected.
Each context should be passed in per call to SendWithContext so that a new context is used per dial, rather than using a global context at the package level. This isn't a super obvious pattern, and I'm sure your solution would have been ok in most cases! This is just a safer way to guarantee that no goroutines share the same context and end up colliding cancellations or timeouts.
If you can, please test in the sending environment you have been working in and let me know if there are any issues. Thanks for your help!

Hello @nilslice.
Ok, I will try it. 😄

Because I don't know how to edit codes in other branch, I wrote the codes here.
Example in README.md - If cancel is not needed,  _ under-score is ok.
ctx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
defer cancel()

email.go : 129 - Text in email was not sent.
_, err = msg.Write([]byte("DATA\r\n\r\n" + m.Body + "\r\n"))
// _, err = fmt.Fprint(msg, m.Body)
if err != nil {
	return err
}

Then, it works well.
Thank you @nilslice. 😄


Matching notification rule tag parameters should combine multiple tags with OR instead of AND

current swagger.yml: line 5533
- in: query
  name: tag
  description: Only show notification rules that match a tag pair. Uses `AND` to specify multiple tags.
  schema:
    type: string
    pattern: ^[a-zA-Z0-9_]+:[a-zA-Z0-9_]+$
    example: env:prod

if check status has tags [k1:v1, k2:v2] it is matched by notification rule NR1 which matches tags [k1:v1]
A GET to /api/v2/notificationRules with query parameters tag: k1:v1 tag: k2:v2 should return NR1.
Currently only query parameters tag: k1:v1 returns NR1

Turns out the title of this issue is misleading. The behavior we want is:

in NR tags
in query
returned?

[k1,v1], [k2,v2]
[k1,v1], [k2,v2]
yes

[k1,v1], [k2,v2]
[k1,v1], [k2,v2], [k3,v3]
yes

[k1,v1], [k2,v2]
[k1,v1], [k2,v3]
no

[k1,v1], [k2,v2], [k3,v3]
[k1,v1], [k2,v2]
??

in the 2nd example AND would have returned false
in the 3rd example OR would have returned true

notification rule with tags k1, equals, v1 and k2 equals v2 results in flux such that
monitor.from(start: -11m, fn: (r) =>(r.k1 == "v1" and r.k2 == "v2")...)

monitor.from in turn executes the filtering function such that..
from = (start, stop=now(), fn=(r) => true) =>
    influxdb.from(bucket: bucket)
        |> range(start: start, stop: stop)
        |> filter(fn: (r) => r._measurement == "statuses")
        |> filter(fn: fn)
        |> v1.fieldsAsCols()


Fix functional args and request parsing

Fixes inconsistency between MediaTypeGuard functional args, causing a bug.
Updates readme for consistency as well.

👍


[Bug]: iOS BaseWebAuth not performing

Description
First, Thank you for this useful package.
I am using this package and found that, while in Android version everything looks and works fine, in iOS it will somehow bypass "authenticate" function and ask for user credentials even if the user is logged in and has valid token in OAuth2 server.
In Android version, if the user is logged in it will return token and redirect automatically to the callback url while in iOS everything looks the same as android but "authenticate" function terminates on call. Without exception it just terminate the function and return null.
Also, in swiftFlutterWebAuth2plugin lines 77 , 79 has problem with "callback" on compile and after changing to this:
_session = ASWebAuthenticationSession(url: url, callbackURLScheme: callbackURLScheme, completionHandler: completionHandler!)
the problem will gone.
Is there anything specific regarding iOS version that I missed?
Thanks for any help in advance
Minimal Reproduction

Not possible to share
Compile example to see the compiler errors.

Exception or Error
No exception, it just terminate.
Example will send abort to the application in xCode
Expected Behaviour
If the user is logged in the authenticate should return callback url with valid token from OAuth server.
Screenshots
No response
Additional context
No response
Device
iPhone 15 pro
OS
ios 17.5.1
Browser
embed
Flutter version
3.24.3
flutter_web_auth_2 version
4.0.1
Checklist

[X] I have read and followed the entire troubleshooting guide and it has not provided the solution I need.
[X] I have provided all the information I can.

Is there anything specific regarding iOS version that I missed?

See: https://github.com/ThexXTURBOXx/flutter_web_auth_2?tab=readme-ov-file#upgrading-to-4x (in particular, now iOS 17.4 and macOS 14.4 are required)
Thanks to Apple, there is no way to use the preprocessor to decide whether the new function is indeed available.
This is a fatal design flaw, but Apple sadly does not care enough to handle this problem.

Version 3.1.2 works correctly as it has tested heavily.
From version 4.x on, 17.4 is required. Please note that this does not necessarily mean that OS version must be greater than 17.4. Your compile SDK version must be at least 17.4 (supported from Xcode 15.3 on).
That has also been tested heavily and hence I am very positive that it works correctly.

Version 3.1.2 works correctly as it has tested heavily. From version 4.x on, 17.4 is required. Please note that this does not necessarily mean that OS version must be greater than 17.4. Your compile SDK version must be at least 17.4 (supported from Xcode 15.3 on). That has also been tested heavily and hence I am very positive that it works correctly.

So you mean I have to set my sdk platform to ios 17.4? like before which it was ios 11?
The funny part is that in version 3.1.2 it is demanded to set ios platform too 11 which is not possible anymore in new version of XCode. it would be 12 at least.
If I use version 3.1.2 the platform will be ios 12 -> not working correctly on ios
If I use version 4.0.1 the platform should be 17.4 -> Not possible! the end-users will narrow down to very few.
Is it possible to use version 3.1.2 with ios platform 12 with some tweaks?

Yes, you need to set your SDK platform to 17.4.
It should have never been just 11 from flutter_web_auth_2's side of things - I have seen deployed apps which use newer SDK platform versions and the code compiles and works just fine.
You should continue to be able to use version 3.1.2 with any iOS version (since the old constructor for ASWebAuthenticationSession is only deprecated, but not yet removed: https://developer.apple.com/documentation/authenticationservices/aswebauthenticationsession/init(url:callbackurlscheme:completionhandler:)).
However, 3.x will not receive the same support treatment as 4.x from my side.

I did updated to xcode 16.
Set the ios Platform to 17.4 (see screenshot)
Set the target runner to 17.4 (see screenshot)
Still show the very same behaviour.

my callbackurlscheme is not https and the httpsHost and httpsPath is empty. FYI


JSDocs tags (e.g. @since ××× etc.)

The react-docgen-typescript library passes tags from JSDocs (lines with tokens starting with “at” character) which react-docgen-typescript-plugin ignores. This pull request amends this so that tags metadata is passed appropriately into the generated code.
I’ve prepared a pull request #53 to merge. Please release it so that it's working also (or predominantly) with webpack@4 [no problems here I guess] and react-docgen-typescript@1 [I’ve noticed there’re plans to bump to @2] as those are versions I am bound for now in code I maintain.
Thanks in advance for finding some time to deal with this issue!

Thank you for this, this is not working for me either

@hipstersmoothie bumping

@hipstersmoothie Can we reopen this? This is still the case.
At the moment, I'll probably workaround this by prepend with undersocre since that's working (e.g<EMAIL_ADDRESS>


Mention object-mode in README

This tiny PR:

Adds one line mentioning MultiStream.obj() to README.

Fixes a few typos I noticed in passing.

Thanks so much for this package.  This is one of those "elemental" streams I find myself using over and over again!
Best,
K

LGTM!


what is LEVEL1_Pin LEVEL2_Pin LEVEL3_Pin LEVEL4_Pin

I don't know the meaning of LEVEL1_Pin LEVEL2_Pin LEVEL3_Pin LEVEL4_Pin pins. Can you help me explain them clearly

Where are they?

Where are they?

Thank you for reply, In this function  "static void MX_GPIO_Init(void)" in main.c file

I connected 4 LEDs to these pins to represent the signal strength using LoRa_getRSSI function. But I forgot to delete them :)
You can ignore them.

I connected 4 LEDs to these pins to represent the signal strength using LoRa_getRSSI function. But I forgot to delete them :) You can ignore them.

thank you I've got it up and running.


update code to work with latest ziti bits

In my editor, ziti.NewContextFromFile was still not resolving (undefined). I re-ran the go get -u github.com/openziti/sdk-golang command and now it resolves and the project builds. Is there competition behind the scenes for sdk-golang version between this and the main ziti project, perhaps? If so, can it be resolved by pinning the version used by kubeztl?
This project's go.mod already pins sdk-golang v0.20.104, and I can build this project by first running go get -u<EMAIL_ADDRESS>or go get -u github.com/openziti/sdk-golang@latest.
It's not yet clear to me what caused the undefined error to manifest after it had built successfully, or how to guide a user to avoid the same problem when building kubeztl. Maybe it's necessary to add the go get command to BUILD.md.


Prevent people adding code by mistake

I am trying to keep it simple (and readable) for every language, I know there are a lot of file extensions that could have code for this languages and of course, I am just adding the common ones. Feel free to add more languages.
I have read the contributing guidelines before doing this pull request.

Done @davidflanagan
And thanks!

Wouldn't it be better to implement the suggestion in #8 ? Though the proposal has run into some discussion relevant to #48 -- I think it's important to resolve these discussions and establish a new best practice, though no code should really have no opinions

While I've 👍 the general idea, I disagree overall. One can have a file that implies code but does not contain code. The Dockerfile is a very good example.
For example, if one wanted to contribute a deployment script, such as deploy.sh with no code, it still executes.
Alternately, one can still contribute comments to sections of nocode without contributing code.

@asciifaceman I think these approaches are too opinionated for nocode. I've created #152 as a proposal that would have no opinions. Let me know your thoughts

When is this going to get to master please?

Suggest setting contents of .gitignore to *.* to prevent unforeseen additions.

hi @roryai, it can have conflict with the documentation files

I would suggest this method https://github.com/kelseyhightower/nocode/pull/209

Guys, I don't want to take the approach to ignore all the files because we may miss by mistake some documentation files, like Markdown, maybe images. That's why we never do this in any gitignore file.
@d0nutptr I just added your suggestions.


Graph query resources

EDN query language (EQL) is a GraphQL-like language specification: https://github.com/edn-query-language/eql
Pathom is a library which helps with implementing services that provide EQL APIs: https://github.com/wilkerlucio/pathom
Speaking of GraphQL, lacinia is a popular library for implementing GraphQL services: https://github.com/walmartlabs/lacinia

Thanks. Added these three in 4b534cb7fbda487d727706ff7ad4985785d05fdd


Local Backups are not listed in OneDrive

Hi there,
first of all I would like to thank you for this great tool to backup HA on a remote location!
I have set my system to save one local and 5 OD backups.
The internal HA backup function is disabled.
When I check the backups after a while, I recognize, that the UI shows 0 local backups and 5 remote backups although I see two backup.tar which are in /backup.
If I compare the file size, these two backups match to 2 remote backups on OD.
Because of space constraints I need to delete the rar files manually every now and then, but after a while the same mismatch happens again:
Backups are created, sometimes are shown as local (and they are physically in /backups), but one day, they are not listed in the UI as local, but obviously synched to OD, but still in the backup folder.
I have activated "monitor all local backups" since quite a while.

Not sure I got all the details but backup.tar is a weird artifact. If "monitor all local backups" is enabled and you believe there is a mismatch in what the addon is showing you, please make sure this is aligned with what you see in Home Assistant under Settings -> System -> Backups.  Are any backups listed there ?

Sorry for the delay, @lavinir
With backup.tar I meant all backup tars (e.g. 033485ff.tar created by OneDrive)
I currently have a mismatch again:
This yellow one is in backup:

But is not shown here::

In the OneDriveBackup View there is 0 local backups listed.
Every now and then I need to delete the local one due to space constraints, but the last time is about two weeks ago.
Any idea why OneDriveBackup is out of sync?
(under system > backup there are no backups listed)
Thank you.

If you have 1 configured local back up. It should keep the last one, however if you deleted that backup it will not be downloaded from OneDrive. Next local backup created will be kept. You can follow what's happening in the logs.
The addon is also querying Home Assistant to get the local backups and not assuming anything or trying to access the file system so if you aren't seeing any Home Assistant backups under System -> Backup it means Home Assistant does not see any local backups and neither will the Addon.
If you are not manually deleting any addons and still do not see one local backup retained after creation please send the logs.


Log In button not working

YASB (First Edition) Bug Report
This bug report filer is for First Edition X-Wing only!
Note: If you are experiencing problems with the permalink URL when you have Maul or Bomblet Generator in your list, this is a known issue.  For more information please see https://www.reddit.com/r/XWingTMG/comments/7c4q6h/yet_another_squad_builder_problems_with_maul/
Expected Behavior
Clicking the Log In button on the top should open a box to login and be able to view your saved squads.
Actual Behavior
Clicking the Log In button does nothing.
Language
English

Same issue here :(


[FLINK-16077][docs-zh] Translate Custom State Serialization page into Chinese

What is the purpose of the change
Translate Custom State Serialization page into Chinese
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Seems there is some conflict, I'll resolve this confict asap

@carp84 I'll create a new pr(#15761) for this issue and close this one, please take a look when you're free, sorry for the inconvenience


NPE in NTI

This occurred when playing with the debugger trying to report another separate bug. I realise what's wrong, but it's crashing:
goog.provide("ns1");

var ns = {};

/**
 * @type {function(new:?)}
 */
ns.something;

java.lang.NullPointerException
	at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:871)
	at com.google.javascript.jscomp.newtypes.ObjectType.makeObjectType(ObjectType.java:128)
	at com.google.javascript.jscomp.newtypes.NamespaceLit.computeJSType(NamespaceLit.java:49)
	at com.google.javascript.jscomp.newtypes.Namespace.toJSType(Namespace.java:246)
	at com.google.javascript.jscomp.NTIScope.freezeScope(NTIScope.java:749)
	at com.google.javascript.jscomp.GlobalTypeInfoCollector.process(GlobalTypeInfoCollector.java:408)
	at com.google.javascript.jscomp.PhaseOptimizer$NamedPass.process(PhaseOptimizer.java:304)
	at com.google.javascript.jscomp.PhaseOptimizer.process(PhaseOptimizer.java:230)
	at com.google.javascript.jscomp.Compiler.check(Compiler.java:1082)

https://closure-compiler-debugger.appspot.com/#input0%3Dgoog.provide(%2522ns1%2522)%253B%250A%250Avar%2520ns%2520%253D%2520%257B%257D%253B%250A%250A%252F**%250A%2520*%2520%2540type%2520%257Bfunction(new%253A%253F)%257D%250A%2520*%252F%250Ans.something%253B%250A%26input1%26conformanceConfig%26externs%26refasterjs-template%26CHECK_SYMBOLS%3Dtrue%26MISSING_PROPERTIES%3Dtrue%26TRANSPILE%3Dtrue%26CHECK_TYPES%3Dtrue%26CHECK_TYPES_NEW_INFERENCE%3Dtrue%26CLOSURE_PASS%3Dtrue%26PRESERVE_TYPE_ANNOTATIONS%3Dtrue%26PRETTY_PRINT%3Dtrue

Related:
https://closure-compiler-debugger.appspot.com/#input0%3Dgoog.provide(%2522ns1%2522)%253B%250A%250Avar%2520ns1%2520%253D%2520%257B%257D%253B%250A%250A%252F**%250A%2520*%2520%2540type%2520%257Bfunction(new%253A%253F)%257D%250A%2520*%252F%250Ans1.something%253B%250A%26input1%26conformanceConfig%26externs%26refasterjs-template%26CHECK_SYMBOLS%3Dtrue%26MISSING_PROPERTIES%3Dtrue%26TRANSPILE%3Dtrue%26CHECK_TYPES%3Dtrue%26CHECK_TYPES_NEW_INFERENCE%3Dtrue%26CLOSURE_PASS%3Dtrue%26PRESERVE_TYPE_ANNOTATIONS%3Dtrue%26PRETTY_PRINT%3Dtrue


WIP e2e test to confirm assumption of KU match on renewal

What this PR does / why we need it:
This was part of https://github.com/jetstack/cert-manager/pull/3545 to add tests to confirm the behaviour or certificates when their key usages get changed by the user
Release note:

NONE

/kind cleanup

Came across this while tidying PRs. This test looks like an improvement but is stale, and should be accompanied with a docs change. Converting to an issue: https://github.com/jetstack/cert-manager/issues/4459

I now remember why Maartje created this PR 😅 Somehow we thought that key usages weren't re-issuing the certificate, and she created a PR to prove that. After investigating, cert-manager definitely re-issues on key usages change (detailed in https://github.com/jetstack/cert-manager/issues/4245#issuecomment-920010215).
I suggest that we close #4459 to avoid more confusion on my part (anytime I see #4459 I think "whaat??")


Extending once again Notifications

And that should be it this time.
I split up the mentions, the notifications methods, etc. so that it should be quite more easy to use and expand.
The admin can pick the notification methods to allow the members to use and which are the defaults, then each member can decide which type of notification method to pick.
I also split up the email notification into direct, daily, weekly so that each can be selected separately, probably this has been overkill, though... whatever, I think it was the easiest way in terms of implementation.
Admin UI:

Profile UI:

the multi-select is not exactly perfect and on mobile may not be the best option, though in normal conditions seems the easiest approach to me.
This time the code should be configured in a way that is fully extendable, that was the main goal.

Codecov Report

Merging #3401 into development will increase coverage by 0.84%.
The diff coverage is 0.00%.

@@                Coverage Diff                @@
##             development    #3401      +/-   ##
=================================================
+ Coverage          21.24%   22.09%   +0.84%
+ Complexity         15877     2188   -13689
=================================================
  Files                385       65     -320
  Lines              79804    10342   -69462
=================================================
- Hits               16956     2285   -14671
+ Misses             62848     8057   -54791

Impacted Files
Coverage Δ
Complexity Δ

sources/ElkArte/Debug.php
6.12% <0.00%> (ø)
60.00 <0.00> (ø)

sources/ElkArte/UrlGenerator/Standard/Board.php
0.00% <0.00%> (-100.00%)
1.00% <0.00%> (ø%)

sources/ElkArte/UrlGenerator/Standard/Topic.php
0.00% <0.00%> (-100.00%)
1.00% <0.00%> (ø%)

sources/ElkArte/DataValidator.php
17.74% <0.00%> (-65.51%)
167.00% <0.00%> (ø%)

sources/ElkArte/Util.php
7.83% <0.00%> (-63.86%)
84.00% <0.00%> (ø%)

sources/ElkArte/BBC/ParserWrapper.php
6.00% <0.00%> (-63.00%)
44.00% <0.00%> (ø%)

sources/ElkArte/Errors/ErrorContext.php
15.92% <0.00%> (-53.99%)
63.00% <0.00%> (ø%)

sources/ElkArte/Priority.php
5.37% <0.00%> (-53.77%)
37.00% <0.00%> (ø%)

sources/ElkArte/Http/BrowserDetector.php
42.40% <0.00%> (-46.84%)
92.00% <0.00%> (ø%)

sources/ElkArte/MemberLoader.php
8.02% <0.00%> (-44.12%)
46.00% <0.00%> (ø%)

... and 332 more

Continue to review full report at Codecov.

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

Oh well, I'll merge this one. :innocent:
If there is bugs... I'll make more!


AzureDocumentDbDriver issue

https://dotnetfalconcontent.blob.core.windows.net/linqpad-cosmosdb-driver/v1-0-0-0/AzureDocumentDbDriver.lpx
I have downloaded the above driver, and it works perfectly on my office PC, with LINQPAD version 5. However, when I come back home to do the same identical installation of the same driver with the same LINQPAD version 5 on my home PC, I encounter this error seen here:
https://i.imgur.com/EvMZGs0.png
Thank you for your attentions.

error as below:
https://azurecosmosdbaccountname.documents.azure.com:443//Competition (Error: Exception - Cannot compile typed context: Source file 'C:\WINDOWS\TEMP\nodhodmh.0.cs' could not be found (line 0))


Vendor SDK for Azure Spring Apps API version 2023-12-01 and 2024-01-01-preview

This PR is about to add the SDKs for Azure Spring Apps API version 2023-12-01 and 2024-01-01-preview.
Related command

N/A
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.


Update kyverno to v1.8.0

How to categorize this PR?

/area dev-productivity
/kind enhancement
What this PR does / why we need it:
With https://github.com/gardener/gardener/pull/7902 kyverno-verify-mutating-webhook-cfg webhook timeoutSeconds get modified to 3 seconds, because of this cluster used for provider-extension setup gets following error -
MutatingWebhookConfiguration "kyverno-verify-mutating-webhook-cfg" is problematic and was remediated by Gardener (please check its annotations for details).

In follow-up PR https://github.com/gardener/gardener/pull/8034, adaption was made so webhook with config similar to this kyverno-verify-mutating-webhook-cfg doesn't get considered problematic.
But this will only work if kyverno version is >=v1.8.0
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/cc @oliver-goetz
Release note:

NONE

/test pull-gardener-e2e-kind-ha-single-zone

/retitle Update kyverno to v1.9.5
/lgtm
/approve


Retrieve the personal code from the stored resource and pass to template

IDVA3-1573
Note there is a similar, but different defect IDVA3-1528, but that relates to when going back as far as the Confirm Company screen.

CI: No security warnings found


Include LatestRoomState in MXRoomSummary

We need to know the canonicalAlias from MXRoomSummary object.
This information (canonicalAlias) is in the MXRoomState class. The problem is that to obtain the canonicalAlias we need to update/get the MXRoomState from room, and this function is asynchron if the MXRoomState is unknow. It is not easy to use.
When we look the Android Version, we see that the MXRoomSummary Class contain an variable with LatestRoomState. It is more usable.
This LatestRoomState come from the Matrix Android SDK, it is filled during the ReceivedEvent function.
It should be great to have the same in iOS SDK version.

MXRoomState access is asynchronous on iOS for performance reasons. A room state can contain thousands of state events, which makes it slow to load.
The MXRoomSummary purpose is to have the list of rooms as quick as possible. Including MXRoomState will make this list x100 times slower to get.
BUT you can add  MXRoomState or canonicalAlias to MXRoomSummary by yourself if your project requires it. You just need to feed MXRoomSummary.others with those data using the MXRoomSummaryUpdating delegation.
You can find an example of it in MatrixKit:
https://github.com/matrix-org/matrix-ios-kit/blob/cd38042c95c6e437d0d3b717b8106ec262c61c2c/MatrixKit/Utils/EventFormatter/MXKEventFormatter.m#L1690


[BUG] - quickstart link is broken

this link https://supereditor.dev/super-editor-guides/quickstart 404s
it's linked from here https://supereditor.dev/super-editor/guides/ the blue link, not the one in the sidebar. The one in the sidebar is https://supereditor.dev/super-editor/guides/quickstart/ which works, so I guess that's what that's the intended one.

@angelosilvestre can you update the link in the docs website? Apparently I changed the sidebar URLs but missed one of the links in the content.


Update Footer

Pull Request
Title
Add hover effect to footer link
Description
Change the CSS
Related Issues
#160
Changes Made
style
Checklist

[x] I have tested the changes locally
[x] Documentation has been updated (if necessary)
[x] Changes are backward-compatible

Screenshots (if applicable)

I have resolved the conflict and I will also consider your suggestion


I need to detect user touch coordinate

how can i get a coordinate whenever user touchs some point? (idependent of zoom level)
coordinate should be relative to pdf page and a float between 0 and 1.
Example: [0.0, 0.0] for upper left and [1.0, 1.0] for lower right corner.

Simply add your code to the source as all the existing methods are there to hook into..


name property for RedisServer

Add name property for RedisServer, so it's possible to add multi connection in the pool for a single server.

Coverage remained the same at 92.502% when pulling 4ab3d2336e655b74409788f30c0b5de3b1574ca0 on redlion99:master into f1c1101a87ebe858f7bcd6b4222e4698c06524df on etaty:master.

Thanks
Could you post a gist to help me understand the use case?

Sometimes we need to create more than one connection for a single server. without the name property if we setup a pool like
RedisClientPool( RedisServer("localhost"), RedisServer("localhost"))
We just get one active connection in the pool, because the case class in the key of the connection map has the same hashcode.
if we have a name property it works like this:
RedisClientPool( RedisServer("localhost", name = "a"), RedisServer("localhost"), name = "b")


Wk fetcher api

Based on recent discussions around audio biel, I've taken the small api Max rolled for processing files on the cdn and moving them through AZ. Service bus. Max can clear up, but based on looking at it, there's a few observations here.

It doesn't not duplicate fetcher pipeline.  It's not intended to recreate files on the cdn and reprocess everything in the event of a schema change or something. This merely reads the content path of the audio files, and will send a set of messages which the public data api schema could consume for audio projects.   That could be added later if needed, but I don't think it will.  The pipeling logic itself (the cron)  that generates those files on fetcher-pipeline is actually what would make those edits likelly.
There does seem to be a route for consuming those messages over http instead of connecting directly to the bus if someone wanted to look at them but not have to connect the Azure SB.
@danparisd  -> I stuck a super basic dockerFile in here and a makefile. The makefile is incomplete, but. I think that basic dockerfile shoudl be sufficient for running it next to the cdn.   The env vars at the top of main.py should be enough to run it.  it shouldn't really need any horsepower or memory much to run next to it.

Anything to add @mXaln ? Or questions @PurpleGuitar  & @danparisd ?
Edited to add: Ignore anything not in fetcher-pipeline-api. Those are some old commits from production I forgot to pull before making this PR

This change is

Closing in favor of https://github.com/WycliffeAssociates/Fetcher_utils, which is deployed for local dev assist.


Allow Support for Custom Color Palettes

It would be very useful to allow the possibility to use your own color palettes. And if this is possible, we would need to come up with a pragmatic and easy way to allow a user to import their colors (eg. maybe copy and paste HEX colors into a text field).
@Kully is already working on a solution for this. Here's a screenshot of the current WIP (branch and Pull Request will eventually come)

Hi @Kully , I like the idea of adding custom color functionality from users, but I think it would be more comfortable if we could let users choose their own colors through a small color picker panel like this image.

Making something like that isn't really difficult but it will probably change everything of the project's current color selection system. And there may be no more palette.js
Or an example of another version of a color picker where we can allow a user to import their colors and they can also create whatever color they want as well:

What do you think about my ideas? I just wanted to share with you these ideas because I actually have a bit of experience doing similar things but with my OpenGL, SDL2 projects and in theory JavaScript should be able to do the same thing. Although I may not have as much experience with JS as those. Anyway, I still really like this project.


[BUG:]  Login SignUp page Component

Describe the bug
Login SignUp page Component is not Proper onScroll it is showing a Scrollbar in right side

Steps to reproduce
Ho
Browsers
No response
Additional context (Is this in dev or production?)
No response

MacOS

Uploading Screen Recording 2024-07-05 at 2.23.57 PM.mov…


[BE] Implement the Newsletter API

Business Requirements
As a visitor, I want to subscribe to the LearnHub.mk newsletter  so that I can follow what's happening with the initiative.
Acceptance Criteria

There should be an API endpoint which will subscribe the visitor to the LearnHub.mk newsletter.
The data which will be needed is the following:

First Name
Email Address

There should be a Cloudflare Turnstile ReCaptcha validation to stop visitors from spamming the endpoint
There should be a API rate limiting of 5 requests per minute.
There should be request validation for each field.
An API documentation must be implemented using the Scribe package so that the front end team can integrate it easily.

Technical details
For the needs of sending newsletters, we will use Maicoach, which has an API SDK for the email list subscriptions.

The self-hosted version of Mailcoach does not seem to have an API so we can't really use the API approach I described in the original post above.
We will use a different approach instead, using the direct request to the subscribe endpoint which can be used directly from the front end but we would trigger that with the Http Client that Laravel offers out of the box.


Loved your wiki I decided to host it for you.

The software I am using costs $1000/mo so congrats I just hosted your wiki on this software.
This wiki can be found at https://tflare.atlassian.net/ and selecting Carbon Forum or visiting this url https://tflare.atlassian.net/wiki/display/CF/Carbon+Forum+Home!

@lincanbin Can you approve or no?

@alwaysontop617 Great job the wiki looks awesome

Invalid.


replaceAll is not a function

I'm having no text when I run this on a huwaei device in a capacitor app. I also get this error on sentry.
I suspect it's the webview in this case doesn't have replaceAll. I'm still investigating.
I'll fork the project to remove the replaceAll and see if it has any impact.

In the screen the chrome mobile view version is not showing, but its Chrome Mobile WebView 83.0.4103. According to this, replaceAll is only available since 85.

I can confirm that was the issue. I added this  polyfill in my app and it worked.

@jcardus  I have met this issue too, could you share your polyfill link? Thanks very much!

hello @zslzz, here it goes:
https://github.com/fleetmap-io/driver-app/blob/a729d7f05f0de57e155100c603fa81118a74dc6f/plugins/vant.js#L21


Question related to PVA

🚨 The issue tracker is not for implementation questions 🚨

[Question]
I wanted to find out if the Power virtual agent has an API we can call ?

Power VA questions should be asked on their forum.  https://powerusers.microsoft.com/t5/Power-Virtual-Agents-Forum/bd-p/10292019


Add a newline to the end of the LABEL line.

Is this really necessary? Updated dockerfile is written to filesystem with fp.writelines which should append newline after every line.

Try it. ;-) I got a docker error about looking for "=" in "uname",  because it was trying to parse this:
LABEL "foo"="bar"RUN uname -a && env

From the documentation: "writelines() does not add line separators". Note that readlines() gives you a list of \n-terminated strings.

Aha! Didn't know that. Thanks for this fix then.


Wrong request size calculation in pcap2ammo tool

Yandex Tank expects the request size in bytes (in utf-8 encoding)  whereas the tool calculates size as count of characters in the request. This solution worked for most of requests because they contained only 1-byte characters. Unfortunately, UTF-8 uses 1-4 bytes per character in general.

Fixed in https://github.com/gaainf/tanktools/pull/2


Automatically register new user on successful first SAML auth

If user is registered in the identity provider, they should automatically be registered in Study Tracker after a successful authentication. Currently, sign in will fail if the users is not already registered in Study Tracker.

Would likely have to extend SAMLAuthenticationProvider and implement a custom authenticate method. https://github.com/sannies/se-security-saml/blob/master/saml2-core/src/main/java/org/springframework/security/saml/SAMLAuthenticationProvider.java


Hot fix iota-client version to 1.1

Description of change
Hot fix for a compilation error on the main branch reported in #524 due to the iota-client git dependency being updated.
Bumps the iota-client version to 1.1 from crates.io.
This is the same fix performed for the dev branch in #503. There should be no functional changes from applying this patch.
Links to any relevant issues
Fixes issue #524
Type of change

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Fix

How the change has been tested
Tests and examples pass locally.
Change checklist
Add an x to the boxes that are relevant to your changes.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Audit check fails due to serde_cbor (#518) which is only a dev-dependency, and the time crate (#440) which we are also not really vulnerable to given the particular steps for it to occur.


Repository piniaStore() returns different piniaStoreInstance, when using a derivedModel

Discussed in https://github.com/CodeDredd/pinia-orm/discussions/1434

Originally posted by adm-bome July  3, 2023
Why does the Repository uses the $entity() as a name for the Pinia Store, while the Query uses the $baseEntity(). Is this by design or by mistake?
https://github.com/CodeDredd/pinia-orm/blob/43126f6b15a19dca099a05de7d9f78a13421c21b/packages/pinia-orm/src/repository/Repository.ts#L117-L119
https://github.com/CodeDredd/pinia-orm/blob/43126f6b15a19dca099a05de7d9f78a13421c21b/packages/pinia-orm/src/query/Query.ts#L163-L172
When creating a Model extended from an other Model we use baseEntity in the exteded Model.
// Base entity.
class Person extends Model {
  static entity = 'person'
  static types () {
    return {
      PERSON: Person,
      ADULT: Adult
    }
  }
  static fields () {
    return {
      id: this.attr(null),
      name: this.attr('')
    }
  }
}

// Derived entity.
class Adult extends Person {
  static entity = 'adult'
  static baseEntity = 'person'
  static fields () {
    return {
      ...super.fields(),
      job: this.attr('')
    }
  }

When we use the following code, a store is created with the name adult
const repo = useRepo(Adult)
const store = repo.piniaStore() // Wrong StoreInstance

But when we use the following code a store is created and data is saved into the piniaStoreInstance with the name person
const repo = useRepo(Adult)
repo.save([{ id: 1 }])

You are right about the piniaStore instance. That's a bug. 👋
The saving you mention behaves correct since you use typed STI everything is stored in the baseEntity.

@CodeDredd
Did you see my comment in this discussion, what do you think?
It solves:

the $entity() / $baseEntity() calls by always using $baseEntity()

It add abbility:

to use or rename the storeName different than the actial model name
to use a prefix to prevent duplicate stores.
to override the method piniaStoreName(), when a developer needs a different outcome, without interfering with the outcome of $entity() and $baseEntity() methods.


[WIP] Fix flaky tests kafka_consumer

What does this PR do?

kafka_consumer is still flaky:

https://dev.azure.com/datadoghq/integrations-core/_build/results?buildId=13176&view=results
https://dev.azure.com/datadoghq/integrations-core/_build/results?buildId=13177

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes


minute_to_session raises error if passed default_start date

Hi,
before version 4.x passing the default start date wouldn't raise an error and simply return the valid session i.e.
nys = get_calendar("XNYS")
nys.minute_to_session_label(pd.Timestamp("2002-12-11", tz="UTC"))

now nys.minute_to_session(pd.Timestamp("2002-12-11", tz="UTC"))
raises errors.MinuteOutOfBounds(calendar, ts, param_name)
Was that change intentional?

Hi @MBounouar, yes this change is intentional.
Here the minute is being passed as UTC midnight on the day of the calendar's first session, although the first minute covered by the calendar will be that session's first minute, (14:30 UTC). The exception's raised as the minute passed is earlier than the first minute covered by the calendar.
See the sessions and minutes tutorials for all things sessions and minutes in v4.
Cheers

Hi @maread99
If it was intentional, I will close the issue.
Though, I still would have expected that direction="next" (which is the default) wouldn't be so unforgiving in this particular case, and kept the old behavior.
Cheers,

Without being strict a user could, without realizing it, pass any minute before the calendar's first minute and wrongly, but reasonably, assume the calendar's first session is the corresponding session.
It makes no difference if the minute passed is 'just' before the first minute - without knowledge of the prior session's open/close it's not possible to know which session to assign a minute to (for example, for a 24h calendar the minute prior to the open should be assigned to the prior session).


Checking if a 7z file is encrypted will crash

Create a 7z file:
$ 7zz a test.7z

Check the entries are encrypted:
Scratch.fsx
#r "nuget: SharpCompress, 0.33.0"

open SharpCompress
open SharpCompress.Archives

let archive = ArchiveFactory.Open("test.7z")

for e in archive.Entries do
  printfn $"IsEncrypted %b{e.IsEncrypted}"

$ dotnet fsi ./Scratch.fsx

It will crash:
System.NullReferenceException: Object reference not set to an instance of an object.
   at SharpCompress.Common.SevenZip.SevenZipFilePart.get_IsEncrypted()
   at SharpCompress.Common.SevenZip.SevenZipEntry.get_IsEncrypted()

To see if I recreated the issue correctly, if you check if it is a directory before checking the IsEncrypted flag
if not e.IsDirectory then
Does that do anything for you?

This does not crash:
#r "nuget: SharpCompress, 0.33.0"

open SharpCompress
open SharpCompress.Archives

let archive = ArchiveFactory.Open("test.7z")

for e in archive.Entries do
  if e.IsDirectory then
    printfn $"%A{e} is a directory"
  else
    printfn $"IsEncrypted %b{e.IsEncrypted}"

Perhaps the error message should be improved?
Can a directory be encrypted?


fix: only do type checking while compiling file

closes #1470
revert #1472 and #1450
Background
In #1450 , we moved the type checking part to getCacheKey. However, this cost performance because ts compiler will do extra diagnostics check and cache them for files which are not the wanted-compiled files.
Changes
This PR does several things:

Move the type checking back to compiling logic. By putting in compiling logic, we only do diagnostics check and cache them for wanted-compiled files. So we don't need also the workaround in #1472
Introduce caching to file system for imported modules' path of test files. By reading this cache whejn creating compiler and fetch into memory cache, we can solve #1390
Add e2e tests for #1390

Caveat
There will be a slight decrease in performance since we need to do extra diagnostics check not only for test file which has changed imported modules but also the changed module to solve #1390

Pull Request Test Coverage Report for Build 4291

46 of 46 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 92.795%

Totals

Change from base Build 4290:
-0.2%

Covered Lines:
1146

Relevant Lines:
1182

💛  - Coveralls


Icons are replaced with text or whitespaces

I've recently installed the powerlevel9k theme, and I am using dritter's configuration. The issue I am facing is that some icons are replaced with text or a whitespace (eg. the tux icon is shown as Lx, Apple is OSX). I checked the icons.zsh file on functions, are the icons there are shown correctly, and I am using the same font on my terminal and text editor (Roboto Mono for Powerline, but the issue happens with any powerline/awesome/nerd font).
Here's screenshots from Terminator and Mousepad.
http://i.imgur.com/rPiSOfz.png
http://i.imgur.com/kkDzivV.png

Did you:

Choose the right POWERLEVEL9K_MODE according to your setup?
Set the awesome font as font in your terminal emulator?
Set POWERLEVEL9K_MODE='awesome-patched' before you source/configure the theme?

These are the main causes for this problem..

Choose the right POWERLEVEL9K_MODE according to your setup?
Yes. I've tried all modes and they all display text instead of icons. Double checking the icons.zsh, it displays the proper icons there.

Set the awesome font as font in your terminal emulator?
Also did that.  The icons are displayed correctly on any other program or text field.

Set POWERLEVEL9K_MODE='awesome-patched' before you source/configure the theme?
I've purged the theme and installed again, setting all the different modes, but the issue still persists.

I've purged the theme and installed again, setting all the different modes, but the issue still persists.
Just to be sure, you have somewhere in your ~/.zshrc something like:

POWERLEVEL9K_MODE="awesome-fontconfig"
source ~/my/path/to/p9k/powerlevel9k.zsh-theme
# or, if you use oh-my-zsh:
ZSH_THEME="powerlevel9k/powerlevel9k"

right?
And if you do print -P $'\uF201', you see a flash icon, right?
But if you do print_icon 'ROOT_ICON', you don't see the flash icon?

I've double checked my .zshrc, the ZSH_THEME is set to powerlevel9k/powerlevel9k.
For the prints, this is what gives

The ROOT_ICON though is set on the get_icon_names command. The LINUX_ICON, for exemple, isn't and only shows as Lx, but on the icons.zsh file, it is the penguin, displayed correctly.

Could you post your ~/.zshrc here? That would be a great help.

Here it is
http://hastebin.com/hokusuhedu.bash

Could you move the source $ZSH/oh-my-zsh.sh line at the bottom and try again?

Hah, that works! Icons are all correctly displayed now.
Thank you for the help!

Looks like we are all set! Happy you're up and running @Virkkunen!


Invalid JS built from custom 1.8 build

@LeonidVeriga sent me a custom build of 1.8 that had your JS-packaging in (see #355). In that build, there was a small error in the JS being built from our React bindings (this was the only change)...
Bridge.define('Bridge.React.AppDispatcher', {
config: {
    events: {
        _dispatcher: null
    }
},
register: function (callback) {
    this._dispatchercallback); // <-- This is invalid
},

The line of code used to be:
    this.add_dispatcher(callback);

I don't know what's caused this, but now we get syntax errors loading the JS in the browser. I'm going to try and build from latest code today to see if this still happens there and if so, dig further. The code we're building is mostly the same as @ProductiveRage's BitBucket repo I've sent previously.
I'll post more info as I find it...

Certainly a defect, and I think building from source will still reproduce the same issue.
We'll try to fix this defect right away. Thanks for reporting.

Yep; does look like it still happens in latest code. Let me know if you can't reproduce and I'll try and produce the smallest C# I can that shows the error.

Ok, can't reproduce this on live.bridge.net, but on latest code here I see this:
using System;

namespace Bridge.React
{
	public class AppDispatcher
	{
		private event Action<string> _dispatcher;

		public void Register(Action<string> callback)
		{
			_dispatcher += callback;
		}
	}
}

compile to this (bad):
/* global Bridge */

"use strict";
Bridge.define('Bridge.React.AppDispatcher', {
    config: {
        events: {
            _dispatcher: null
        }
    },
    register: function (callback) {
        this._dispatchercallback);
    }
});

but on live.bridge.net, it compiles to this (good):
/* global Bridge */

Bridge.define('LiveApp.AppDispatcher', {
    config: {
        events: {
            _dispatcher: null
        }
    },
    register: function (callback) {
        this.add_dispatcher(callback);
    }
});

All looks good now :+1:

Thanks for testing and giving the feedback!


⚠ Bofrost.be has degraded performance

In 6b3d822, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1044 ms

Resolved: Bofrost.be performance has improved in 181cd0d.


kcr edit doesn't play well with kakoune-state-save

Hi, @alexherbo2 !
If I have the kakoune-state-save plugin installed and open a file in Kakoune, it restores the selections from a previous edit.
It also works if I open the file using kak -p:
echo "edit somefile" | kak -p #session

However, if I do the same using kcr edit somefile, selections are not restored.

Thank you for fixing it, and particularly for being so fast!


Loading File Error Emergent

Dear development team,
My online diagrams file suddenly crashed when I was editing. When I tried to open this file from google drive, it showed: Loading File Error: (error on line145 at column 1: Extra content at the end of the document). My teammates and I could not open this file anymore. It cost me 3 hours to draw this diagram. I am wondering if you could please help me find my file back. Tomorrow would be the deadline and it is not possible to draw a new diagram now. Your reply would be appreciated. My account<EMAIL_ADDRESS>My file: Group10_BipVote_diagram

Please file an issue as a bug and fill in the template provided. https://github.com/jgraph/drawio/issues/new?assignees=&labels=&template=bug_report.md


[BUG] avx-512 sanitizers build

At the very least memory does not compile.

They are now running and no erros for now


Task: Enable Dependabot on Directory.Packages.props

I still see a mix of older and newer nugets in the Directory.Packages.props file:
    <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.0" />
    <PackageVersion Include="Microsoft.Extensions.Logging.Debug" Version="7.0.0" />

Dependabot, AFAIK, updates the packages in the shipping Nuget (which it did do in the last go around), and that package is referenced in the sample apps only.


Cannot run successfully in VSCode.

Hello,
When I run the project in VSCode.
Nothing has changed, and the compilation was successful.
When I click on Run Test in TestingWebApplicationTest.java.
I get an error:
Caused by: java.lang.NoClassDefFoundError: org/junit/platform/engine/EngineDiscoveryListener
I missed something or did something wrong?

We test by building from the command line, so we can't help with this. You should ask on Stack Overflow.


Can't create a new app

Description
I'm running commands like create-react-native-app foo, but they're failing. They've worked in the past with the same setup (same versions of node, yarn, OS, same network, etc.) - I think it may be an issue with a package that create-react-native-app depends on (duplexer3) being removed from the npm registry?
Expected Behavior
I expected a new react native app skeleton to be created.
Observed Behavior
create-react-native-app foo failed with output:
Creating a new React Native app in /Users/yashapodeswa/projects/foo.

Using package manager as yarnpkg with yarn interface.
Installing packages. This might take a couple minutes.
Installing react-native-scripts...

yarn add v1.3.2
info No lockfile found.
[1/4] 🔍  Resolving packages...
warning react-native-scripts > xdl > auth0-js > xtend ><EMAIL_ADDRESS>error Received malformed response from registry for "duplexer3". The registry may be down.
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.
Error: Received malformed response from registry for "timed-out". The registry may be down.
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:48907:15
    at Generator.next (<anonymous>)
    at step (/usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:92:30)
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:110:14
    at new Promise (<anonymous>)
    at new F (/usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:29389:28)
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:89:12
    at Function.findVersionInRegistryResponse (/usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:48946:7)
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:48963:28
    at Generator.next (<anonymous>)
Error: Received malformed response from registry for "timed-out". The registry may be down.
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:48907:15
    at Generator.next (<anonymous>)
    at step (/usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:92:30)
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:110:14
    at new Promise (<anonymous>)
    at new F (/usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:29389:28)
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:89:12
    at Function.findVersionInRegistryResponse (/usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:48946:7)
    at /usr/local/Cellar/yarn/1.3.2/libexec/lib/cli.js:48963:28
    at Generator.next (<anonymous>)
`yarnpkg add --dev --exact --ignore-optional react-native-scripts` failed

Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts: n/a
npm ls react-native: n/a
npm ls expo: n/a
node -v: 8.9.4
npm -v: 5.6.0
yarn --version: 1.3.2
watchman version: 4.9.0

Also specify:

Operating system: macOS Sierra (version 10.12.6)
Phone/emulator/simulator & version: n/a

Reproducible Demo
Run create-react-native-app <anything>

@theduke , I started with react-native yesterday and am experiencing same problem, anyway out?


Refactoring

Missing dependency (see ModuleNotFoundError: No module named 'deap' at https://github.com/dice-group/Ontolearn/blob/44571b74179473aa2fb5ca479916a6f06afd4848/ontolearn/concept_learner.py#L15)
No info pertaining to python modules (sortedcontainers-stubs,superprop)
Clean the repo
Update the readme
Interface of this project should at least resemble sklearn.
Move logging.conf into ontolearn folder.

Analyse the current state of old issues:
https://github.com/dice-group/Ontolearn/issues/188
https://github.com/dice-group/Ontolearn/issues/174
https://github.com/dice-group/Ontolearn/issues/169
https://github.com/dice-group/Ontolearn/issues/95
https://github.com/dice-group/Ontolearn/issues/28
https://github.com/dice-group/Ontolearn/issues/29
https://github.com/dice-group/Ontolearn/issues/56
https://github.com/dice-group/Ontolearn/issues/33

ConceptGenerator().get_object_property_ranges(prop: OWLObjectProperty) function should not return an empty FrozenSet, right ?
Property range and domain should be extracted regarless of the input data

from .abstracts import *
from .base_concept_learner import *
from .metrics import *
from .search import *

and many other modules should be added into core/.__init__.py so that ontolearn can be extended by using it as a library. Currently, we can use ontolearn BaseConceptLearner class as an abstract class outside of the ontolearn


prioritize connecting to full nodes that are already "caught up"

sometimes, voyager will connect to a full node that isn't caught up yet which is a drag for the user and can can possibly prohibit transactions from taking place. instead, before connecting to a full node, let's check to see if they are caught up or not. this will provide a superior UX for our users.

It's posible to get the node status with this endpoint and later get the result.sync_info from it. In this case you're suggesting to connect with the nodes that have result. sync_info. syncing = false value only ?

Please add the label discuss to all new issues.

if result. sync_info.syncing = true when a node is syncing / catching up - then yes.
do you think that's a good idea?

Closing this as we are connecting to a trusted full node already


[ready for review] Fixes for OAuth2 application registration

Refs [#OSF-4430] and [#OSF-4433].
Purpose
Addresses unclear or surprising behaviors when registering an OAuth2 Developer App on the OSF. This includes the case where an all-HTML name was blanked by the sanitizer, and the rare case where a user might want to deactivate a new OAuth2 application immediately.
Summary of changes

Move the string_required validator from framework.auth.core to a new centralized location, framework.mongo.validators
Use the above validator in the name field of ApiOAuth2Application.
Change the wording of the "deactivation failed" error message to specify that it may work if the user waits a few minutes

New error message sample:

Other
Cleaned up and reordered imports in framework.auth.core; auth exceptions were being imported in two different ways. I've separated the cleanup work into its own commit.

Good to merge once the build passes.


prices not refreshing?

my prices are sometimes not refreshing when I check the marketboard. this used to work just fine, but as of recently it stopped.

yes I have pressed alt, deactivated and reactivated the plugin, reloaded with config reset, uninstalled and reinstalled the plugin, just about everything.
I checked universalis and the price hasn't updated there, so I thought mayhaps my client was struggling to send info. I disabled and re-enabled the info collection from dalamud and still nothing. I restarted FFXIV and it still was not updating. Is it mayhaps an issue with universalis? is it an issue with my client? I'm unsure at this point. I don't think it's the plugin though. Sorry for the worthless issue report


zip file

the README file contains
Decompress the provided zip file:
Unzip the provided file. Inside, you will find the sample-solr-data folder needed for the installation.

I cannot find any zip file and cannot find a dir called sample-solr-data in the code

Hi Arno, the sample-solr-data folder is the Solr data that you gave us at the beginning to start the project.
We can post them on github but they are database data, are we sure we want to make them public?
Let us know and thanks!


Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/build.js

I am facing this issue very often, please let me know the solution for this
I tried the troubleshooting guidelines also, then also I faced the same issue.
 System Windows_NT 6.1.7601
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\raghuveer.gut
hikonda\\AppData\\Roaming\\npm\\node_modules\\node-sass\\node_modules\\node-gyp\
\bin\\node-gyp.js" "rebuild" "--verbose" "--libsass_ext=" "--libsass_cflags=" "-
-libsass_ldflags=" "--libsass_library="
gyp ERR! cwd C:\Users\raghuveer.guthikonda\AppData\Roaming\npm\node_modules\node
-sass
gyp ERR! node -v v4.3.1
gyp ERR! node-gyp -v v3.3.0
gyp ERR! not ok
Build failed
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\
node_modules\\npm\\bin\\npm-cli.js" "install" "-g" "node-sass"
npm ERR! node v4.3.1
npm ERR! npm  v2.14.12
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>postinstall: `node scripts/build.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/build.js
'.
npm ERR! This is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/build.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\npm-debug.log

npm-debug.txt
npm-err.txt
npm-log.txt

Please upload your logs as [https://gist.github.com](github gist's).

@xzyfer please check now I've reattached files to the same post

I see that but I cannot download your files on my system. Please post them into github gists so I can read them in my browser,

@xzyfer
https://gist.github.com/rguthikonda/dbb922870ee5ea2a8296

Looks like a simple networking issue.
npm info install<EMAIL_ADDRESS>Cannot download "https://github.com/sass/node-sass/releases/download/v3.4.2/win32-x64-46_binding.node":

connect ETIMEDOUT <IP_ADDRESS>:443

Hint: If github.com is not accessible in your location
      try setting a proxy via HTTP_PROXY, e.g.

      export HTTP_PROXY=http://example.com:1234

or configure npm proxy via

      npm config set proxy http://example.com:8080

This error should have appeared in your console telling you how to resolve the issue.
The error you're seeing is because you couldn't download the file above, and your system tried to compile the extension without Visual Studio installed.

@xzyfer thanks for the update
but I have VS 2013 installed in my system
and I have access to that url via browser
Cannot download "https://github.com/sass/node-sass/releases/download/v3.4.2/win32-x64-46_binding.node":
are there any other scenarios this error will come?

No, the error is a networking error.

I may have misread the error message but msbuild.exe cannot compile the extension. Maybe you have an incompatible version. Also, if you're on Windows XP there is an issue preventing v3.4.2 compiling.
gyp verb build type Release
gyp verb architecture x64
gyp verb node dev dir C:\Users\raghuveer.guthikonda\.node-gyp\4.3.1
gyp verb found first Solution file build/binding.sln
gyp verb could not find "msbuild.exe" in PATH - finding location in registry
gyp verb "Release" dir needed to be created? null
gyp verb copying "node.lib" for x64 C:\Users\raghuveer.guthikonda\.node-gyp\4.3.1\Release\node.lib
gyp info spawn C:\Program Files (x86)\MSBuild\12.0\bin\msbuild.exe
gyp info spawn args [ 'build/binding.sln',
gyp info spawn args   '/nologo',
gyp info spawn args   '/p:Configuration=Release;Platform=x64' ]
gyp ERR! build error
gyp ERR! stack Error: `C:\Program Files (x86)\MSBuild\12.0\bin\msbuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\temp1\node_modules\node-sass\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\temp1\\node_modules\\node-sass\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild" "--verbose" "--libsass_ext=" "--libsass_cflags=" "--libsass_ldflags=" "--libsass_library="
gyp ERR! cwd C:\temp1\node_modules\node-sass
gyp ERR! node -v v4.3.1
gyp ERR! node-gyp -v v3.3.0
gyp ERR! not ok
Build failed

@rguthikonda are you installing node-sass from the visual studio command prompt? Looks like Microsoft developer tools are not on your PATH (this is normal if you run normal cmd.exe). If you can't get the VS command prompt, there should be a vcvarsall.bat file somewhere which can set your environment variables correctly.


thumbnails and preview support

Based on the existence of sky.tab.thumb.png and sky.tab.png

merged and activated!


Add hooks.log_task_issue

Closes #566.
This PR adds a hook called log_task_issue. It creates a log trace entry with a particular format. Agents can use this hook as a standard way to log an issue with a task. Later, we can query the Vivaria database for runs that have a trace entry that looks like this.
Testing: Start a run with an agent that uses git+https://github.com/METR/vivaria.git@70bc77c3b39fdedd401bd0fd95ceffe67ebb5e41#subdirectory=pyhooks as its version of pyhooks and calls hooks.log_task_issue("Issue") at some point.

To be complete, should we not add a TaskIssueEC class to types.ts?

I don't think that'd work. LogEC is part of a discriminated union on the type field. These trace entries are just normal trace entries with type "log", so the discriminated union wouldn't be able to discriminate between these trace entries and other log trace entries.
We could add a new trace entry type and a new API endpoint to record trace entries of this type. However, it doesn't seem necessary for this situation.

I don't think that'd work. LogEC is part of a discriminated union on the type field. These trace entries are just normal trace entries with type "log", so the discriminated union wouldn't be able to discriminate between these trace entries and other log trace entries.
We could add a new trace entry type and a new API endpoint to record trace entries of this type. However, it doesn't seem necessary for this situation.

Oh, actually that's confusing, honestly. They're entries of type "log" with type: "task_issue"?

Yeah that's right. Good to know that it's confusing, I could probably find some other way to make them unique.
E.g. what if they were log entries that looked like ["The agent logged an issue with the task", task_issue]?
It seems a bit clumsier to query for these but not too bad.
And yeah maybe it makes sense to make this less confusing by adding a new trace entry type. It's a rather larger change. But IDK maybe the UI would like to be able to easily distinguish these from other logs someday.


Time until departure as hours and minutes

Currently, the time until departure is displayed as minutes only. This works really good for departures in the near future, but it’s a little cumbersome for departures later that day. Planning for a departure in 210 minutes is just not as intuitive as in 3:30 hours.
It would be great to add an option, which allows to select between minutes only, or hours and minutes.

That's a great idea, I could use that feature as well!
I'll try to implement it :)

Implemented in https://github.com/custom-cards/rmv-card/pull/10
Thanks @cifland


Support for .Net 7

.Net 7 is now in “preview” and will be released soon. An implementation is now needed for .Net 7 and 6.

Branch net7Support has been created.

This functionality has been enabled.

All done!


Implement header_added RPC WS subscription

Since https://github.com/nspcc-dev/neo-go/pull/3252 we have header_of_added_block WS subscription, and in this issue we need to implement header_added RPC WS subscription described as the second case in https://github.com/nspcc-dev/neo-go/issues/2751#issuecomment-1844726731.

Not a priority for now.


fix(integration/compat): Fix opendal 0.50 OpList has new field

Which issue does this PR close?

Follow up of https://github.com/apache/opendal/issues/5235
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Thank you @Zheaoli for the review.


change TPU build params

What does this PR do?
to have better samples from python and torch versions
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified
[x] Check that target branch and milestone match!

Did you have fun?
Make sure you had fun coding 🙃

ok, this would need to change required tests too


Draft for adding multi level select capabilities

This is a draft to gather feedback, and do some testing, so it's not considered ready for merging.
I would both like feedback on the UX of it (so please run the select example, and play around with it), and also on the approaches in the code. I think I've matched the existing style pretty well, but let me know if there's something you would like to be different.
If the feedback is good, I'll start looking at tests and documentation.
I haven't looked at navigating back and forth with the arrow keys yet, but I would really like to get that in before we merge this. But I'll let that wait until the other stuff is ready I think 🙂
Implements most of #382

Nice! I tried the example and it works already pretty well 👍 Tomorrow i should have time too look over your changes.

Nice! I tried the example and it works already pretty well 👍 Tomorrow i should have time too look over your changes.

Awesome, thanks! I'm pretty happy with how this is getting off the ground too 😄
Aside from the code changes, these are the UX changes that are worth having a look at I think:

When you navigate into a category, the title of the category is displayed at the top

What style should this use?
If you navigate inside another category, they show as "firstCategoryName/secondCategoryName"
If you navigate into a third (and beyond) category, all middle category names get truncated out of the path, ala "firstCategoryName/../thirdCategoryName"

In the list of selectable items, categories look like all other items, except that they have their own pointer

Should they have their own pointer in the first place, and if so, what should it be?
Should they look distinct in some other way?

If you're mixing categories and regular items, it might be nice to make them distinct, but if you e.g. have the first level only as categories, and the second level only as regular items, it might look funny?

The "Back" button also has its own pointer, and also a separator between it and the other options

Should it look different?

When searching, what should categories look like?

Currently categories also get highlighted if they match the input text, but I don't think they should
They also just look white at the moment when they don't match the input, which also looks weird I think.

Alright, I think I've addressed all concerns now 😊 The styling could probably use more feedback, and I'm waiting for more review before I start adding tests. So I'll let you have a look at it and test it out 🙂
You've talked about expanding this to other prompt types when this is done, but I'd like to end my involvement after this PR is merged, since I'm trying to cut down on things I have in flight at the moment 😅

As for removing value from categories, a thought occurred to me as I was making new example data: if categories are allowed to have a value, then it could allow the final prompt selection to not only be a string, but potentially a list of strings, denoting the path. This could also be useful some times 🙂

Hey, got any idea when you might have time to look this over? 😊 I don't need any specific time, but just "hopefully soon", "not soon", or "no idea" would be nice ☺

Sry for the late reply @KasMA1990, but I have been very busy lately.
My plan is to finish some refactorings for the prompt module first, then I will continue with this PR. But i'm almost done with it. I hope to get to it in the next few days.

@KasMA1990 I will look into it now and resolve the merge conflicts.

Hey @KasMA1990,
i did now some refactorings:

moved the logic the the generic prompt class to add support for it to the checkbox prompt as well.
show icons while searching
no longer disable groups while search to be able to navigate for and back while searching
splitted the option interfaces into options and group options
changed the group icons (opened/closed folder)
the group options no longer have a checked property, instead the group settings have a checked getter and there is now also a parially checked state for groups with ther own icon. i did not found a good icon for partial selection. currently i use the radio button icon ⦿ because it looks similar to some html checkboxes with partial selection but it's a circle and not a square.

I think it looks already pretty good and the search/sorting feels also good 👍
But there are still a few thinks that needs to be fixed and cleaned up.
Let me know what you think 😉

This is very nice I think! My only gripe at this point is probably that it doesn't feel natural that I can't search for groups, I can only search by items. But I also think that's fine to leave for now, and then come back to later if needed.
Which things are missing now you think?

Currently I am aware of the following things that are missing:

[ ] groups are not searchable, as you noted.
[ ] it should be possible to disable the group icons (I'm working on it right now)
[ ] it is currently not possible to enter a space in the search, because the space key checks/unchecks an option (checkbox query only)
[ ] it is currently not possible to submit when a group is selected because it enters the group (checkbox prompt only)

I'm not sure how to fix this. Maybe we should put the search behind a shortcut 🤔 or a "Submit/Done" option.

[ ] Back button/parent group can't be checked and the checked/unchecked icon is missing (only checkbox prompt).
[ ] tests are missing

@KasMA1990 Sry for the very long time, but i think this MR is finally ready to merge 🙂 Thanks again for your work 👍

And thank you for your work as well! I'm sorry I haven't been able to help get it over the finish line, but I'm stoked it's here! 😃