fix creation of private key in tls secret.

With this PR, the way the private key is computed and saved to the TLS Secret has been fixed. In addition, dependencies have been upgraded and Go 1.22 is now used.

/lgtm


Fix date parsing

Fixes #268.

Did you checked benchmark? I understand problem, but don't like idea to increase number of regex calls.

The benchmark results look pretty similar on my machine, but this is by no means exhaustive. It doesn't look like any of the benchmark samples actually contain timestamps, so this is really only checking the slowdown of resolveYamlTimestamp. The differences appear to be within the variance I see from just running the benchmark multiple times.
Before
Sample: document_application2.yaml (360 characters)
 > current x 40,145 ops/sec ±1.55% (81 runs sampled)
Sample: document_huge.yaml (7182063 characters)
 > current x 4.99 ops/sec ±3.08% (15 runs sampled)
Sample: document_nodeca_application.yaml (2055 characters)
 > current x 33,652 ops/sec ±1.11% (89 runs sampled)
Sample: document_nodeca_database.yaml (394 characters)
 > current x 73,385 ops/sec ±1.56% (85 runs sampled)
Sample: document_nodeca_logger.yaml (727 characters)
 > current x 25,125 ops/sec ±1.51% (83 runs sampled)
Sample: scalar_double_quoted_binary_01.yaml (807 characters)
 > current x 129,090 ops/sec ±0.92% (86 runs sampled)
Sample: scalar_double_quoted_binary_02.yaml (12007 characters)
 > current x 8,955 ops/sec ±0.89% (83 runs sampled)
Sample: scalar_double_quoted_escaped_01.yaml (119 characters)
 > current x 522,485 ops/sec ±1.32% (85 runs sampled)
Sample: scalar_double_quoted_escaped_02.yaml (3444 characters)
 > current x 35,455 ops/sec ±1.58% (84 runs sampled)
Sample: scalar_double_quoted_simple_01.yaml (107 characters)
 > current x 774,084 ops/sec ±1.21% (86 runs sampled)
Sample: scalar_double_quoted_simple_02.yaml (3353 characters)
 > current x 40,661 ops/sec ±1.27% (87 runs sampled)
Sample: scalar_double_quoted_simple_multiline.yaml (2707 characters)
 > current x 37,276 ops/sec ±1.09% (85 runs sampled)

After
Sample: document_application2.yaml (360 characters)
> current x 41,155 ops/sec ±1.75% (81 runs sampled)
Sample: document_huge.yaml (7182063 characters)
> current x 5.09 ops/sec ±2.67% (15 runs sampled)
Sample: document_nodeca_application.yaml (2055 characters)
> current x 32,444 ops/sec ±0.77% (88 runs sampled)
Sample: document_nodeca_database.yaml (394 characters)
> current x 83,429 ops/sec ±1.52% (84 runs sampled)
Sample: document_nodeca_logger.yaml (727 characters)
> current x 24,824 ops/sec ±0.93% (88 runs sampled)
Sample: scalar_double_quoted_binary_01.yaml (807 characters)
> current x 121,229 ops/sec ±0.75% (86 runs sampled)
Sample: scalar_double_quoted_binary_02.yaml (12007 characters)
> current x 9,258 ops/sec ±0.82% (88 runs sampled)
Sample: scalar_double_quoted_escaped_01.yaml (119 characters)
> current x 560,702 ops/sec ±1.09% (86 runs sampled)
Sample: scalar_double_quoted_escaped_02.yaml (3444 characters)
> current x 25,252 ops/sec ±1.34% (88 runs sampled)
Sample: scalar_double_quoted_simple_01.yaml (107 characters)
> current x 772,908 ops/sec ±0.96% (88 runs sampled)
Sample: scalar_double_quoted_simple_02.yaml (3353 characters)
> current x 40,198 ops/sec ±0.96% (88 runs sampled)
Sample: scalar_double_quoted_simple_multiline.yaml (2707 characters)
> current x 42,134 ops/sec ±0.99% (90 runs sampled)

Ок, if benchmarks did not suffered - i see no problems.
/cc @dervus ?

LGFM.


add logo in about section

Issue Number
fixes #204

Describe the changes you've made
add the logo in about section of the readme file
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] I have added my name in the contributors list at the end of README.md file.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] The title of my pull request is a short description of the requested changes.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

@QAZIMAAZARSHAD  Check the about section pls


Phantomjs 1.9.7 - Border dotted and dashed - PDF

Hello, i'm using phantomjs 1.9.7 for generate PDF's, but on elements, by example tables, with border dotted or dashed, they are not printed correctly. In the PDF the borders show correctly, but on printing don't show.
PDF

Printing

+1


Bug located in ...

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
Any use of the export_model_obj function with an object list or file path used as input causes PyAEDT to crash (the pop-up with "do you want to save your data"). Passing only the export_as_single_objects arg does not cause this behavior.
Steps To Reproduce
Call export_model_obj on an HFSS (classic) design while passing either a file path or an object list (or both).
Which Operating System are you using?
Windows
Which Python version are you using?
3.9
Installed packages
anyio==3.6.2
argon2-cffi==21.3.0
argon2-cffi-bindings==21.2.0
arrow==1.2.3
asttokens==2.2.1
attrs==22.2.0
backcall==0.2.0
beautifulsoup4==4.11.2
bleach==6.0.0
cadquery==2.2.0
cadquery-ocp==7.7.0
casadi==3.5.5
cffi==1.15.1
clr-loader==0.2.5
colorama==0.4.6
comm==0.1.2
debugpy==1.6.6
decorator==5.1.1
defusedxml==0.7.1
executing==1.2.0
ezdxf==1.0.2
fastjsonschema==2.16.3
fqdn==1.5.1
idna==3.4
importlib-metadata==6.1.0
ipykernel==6.21.3
ipython==8.11.0
ipython-genutils==0.2.0
ipywidgets==8.0.4
isoduration==20.11.0
jedi==0.18.2
Jinja2==3.1.2
jsonpointer==2.3
jsonschema==4.17.3
jupyter==1.0.0
jupyter-console==6.6.3
jupyter-events==0.6.3
jupyter_client==8.0.3
jupyter_core==5.3.0
jupyter_server==2.5.0
jupyter_server_terminals==0.4.4
jupyterlab-pygments==0.2.2
jupyterlab-widgets==3.0.5
MarkupSafe==2.1.2
matplotlib-inline==0.1.6
mistune==2.0.5
multimethod==1.9.1
nbclassic==0.5.3
nbclient==0.7.2
nbconvert==7.2.10
nbformat==5.7.3
nest-asyncio==1.5.6
nlopt==2.7.1
notebook==6.5.3
notebook_shim==0.2.2
nptyping==2.0.1
numpy==1.24.2
packaging==23.0
pandas==1.5.3
pandocfilters==1.5.0
parso==0.8.3
path==16.6.0
pickleshare==0.7.5
platformdirs==3.1.1
plumbum==1.8.1
prometheus-client==0.16.0
prompt-toolkit==3.0.38
psutil==5.9.4
pure-eval==0.2.2
pyaedt==0.6.56
pycparser==2.21
Pygments==2.14.0
pyparsing==3.0.9
pyrsistent==0.19.3
python-dateutil==2.8.2
python-json-logger==2.0.7
pythonnet==3.0.1
pytz==2022.7.1
pywin32==305
pywinpty==2.0.10
PyYAML==6.0
pyzmq==25.0.1
qtconsole==5.4.1
QtPy==2.3.0
rfc3339-validator==0.1.4
rfc3986-validator==0.1.1
rpyc==5.3.0
Send2Trash==1.8.0
six==1.16.0
sniffio==1.3.0
soupsieve==2.4
stack-data==0.6.2
terminado==0.17.1
tinycss2==1.2.1
tornado==6.2
traitlets==5.9.0
typing_extensions==4.5.0
typish==1.9.3
uri-template==1.2.0
wcwidth==0.2.6
webcolors==1.12
webencodings==0.5.1
websocket-client==1.5.1
widgetsnbextension==4.0.5
zipp==3.15.0

Hi @stephennewberry,
Please could you attach the snippet code? I can not reproduce the issue:
I have created a simple HFSS design with a box called "Box1", I have launched this script:
import pyaedt

pyaedt.settings.use_grpc_api = False

app = pyaedt.Hfss(specified_version="2023.1", non_graphical=False, new_desktop_session=False)

x = app.post.export_model_obj(["Box1"])

app.release_desktop(False, False)

And it worked.

Hi @Samuelopez-ansys, my apologies for my delayed response. I went to reproduce the bug and am no longer able to see this issue. I assume this was an error in my configuration then. Sorry again - and thank you for the quick response!


LicenseFileUrl

@freddydk I'm currently relaunching our 'move to AL-GO' journey internally and have a question on the 'LicenseFileUrl' secret as referenced in various wiki pages / documentation. I was about to configure the 'LicenseFileUrl' secret as described in the wiki, but wondered if this is still valid / required while reading this post about updated Cronus demo licenses. Do we still need a developers license to build apps in CI/CD?
https://www.yammer.com/dynamicsnavdev/#/Threads/show?threadId=2172494757232640
Developers no longer need partner licenses or powerful customer licenses to do development.

Majority of people will probably be able to use the CRONUS (DEMO) license for their DevOps needs.
The only reason I can think of, where a dedicated developer license is needed for this is if your tests cannot run under the limitations of the CRONUS (DEMO) license.

@freddydk As licenseFileUrl won't be used by the majority of people, would it be possible to remove the warning?
I would like to get as little 'AL-GO' warnings as possible, so we can focus on the relevant codeAnalyzer warnings instead.

Yeah - probably a good point to remove the warning and leave it as an information in the log


fix: Remove physical device from integration tests

Fixes #1632
Test Plan

How do we know the code works?

IT should pass.
Checklist

[X] Integration tests updated

@flank-it

@adamfilipow92 please check IT

@flank-it


Fix aggregator test failures

See failures on this PR which is really just testing main:
https://github.com/web3well/bls-wallet/pull/270

I think this is just an analogue of https://github.com/web3well/bls-wallet/issues/268. Not confirmed yet though.
However, this requires code intervention since aggregator/test is doing the out-of-sync thing.


updates subscribe calls in service-bus with no-op handlers or handlers that logs

Using subscribe() without handlers triggers fallback error logging, which can be noisy. Update these calls with explicit handlers to either do nothing or log as needed.

API change check
API changes are not detected in this pull request.


Fix invalid example for http redirect

The example which redirects HTTP to HTTPS uses scheme HTTPS but emits the port. However, docs[1] mentions as "When empty, port (if specified) of the request is used.", so the access always is forwared to the HTTPS but 80 port.

The result of the test with Istio 1.17-alpha.ebe08a1368b65d93cf51c24675c29409567ca77c. Please find location: https://redirect.example.com:80/get.

* Mark bundle as not supporting multiuse
< HTTP/1.1 301 Moved Permanently
< location: https://redirect.example.com:80/get
< date: Thu, 19 Jan 2023 10:12:52 GMT
< server: istio-envoy
< content-length: 0

This patch fixes it.
[1] https://gateway-api.sigs.k8s.io/references/spec/#gateway.networking.k8s.io%2Fv1beta1.HTTPRequestRedirectFilter

Port is the port to be used in the value of the Location header in the response. When empty, port (if specified) of the request is used.

/kind bug
/kind documentation

Ah, I overlooked the doc "port (if specified) of the request" . So, if there are no :80 port on the request, it works and example assumed that :80 is not specified...
I close this.


Add log documentation for the couchdb integration

What does this PR do?
Add documentation for the couchdb log integration
Motivation
It was supported but not documented
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] 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
[ ] Feature or bugfix must have tests
[ ] Git history must be clean
[ ] If PR adds a configuration option, it must be added to the configuration file.

@DataDog/agent-integrations would it be possible to have a review on your end and some help merging this?
It's pure documentation.


Fix Windows Script Path

This PR is an attempt to fix how the distribution Flask server is targeted on Windows builds.

Trying now

Just for added context, this again

But launches fine immediately after that, here's the dev console
index-6be853e2.js:48423 [main-process]: Packaged Script Path D:\NWB GUIDE\resources\app\build\flask\flask\nwb-guide.exe
(anonymous) @ index-6be853e2.js:48423
index-6be853e2.js:48423 [main-process]: Resolved Script Path D:\NWB GUIDE\resources\app\pyflask\app.py

What's the actual path to the build .exe?

That's what I was just looking up: D:\NWB GUIDE\resources\app\build\flask\nwb-guide
So it's the double-flask that's causing it by the looks of it

Ah sure. Just need to remove PYFLASK_BUILD_SUBFOLDER_NAME from the path declaration

@CodyCBakerPhD Alright can you try again?

Negatory

Hmmm. Assuming you got the ENOENT error still?

Collecting package metadata (repodata.json): done
Solving environment: failed

PackagesNotFoundError: The following packages are missing from the target environment:
  - mkl
  - mkl-service

Hmmm. Assuming you got the ENOENT error still?

Always 😆

@CodyCBakerPhD This works now!

Confirmed!


Problem updating SystemJS to 0.19.18

System.config (index.html) is not working anymore using SystemJS v 0.19.18 for dev/prod.
If I downgrade to SystemJS v.019.17 it's still working.

Please describe more the error you see (stacktrace, etc). Can you run the tests?

@jmcota would you test against this branch https://github.com/mgechev/angular2-seed/tree/systemjs ?
If your issue is fixed and you don't have any other problems with the introduced changes we'll merge to master.

Closed by https://github.com/mgechev/angular2-seed/pull/436.


Simple form should override default form scaffolding

Generators are one of the best features of Rails. Currently Simple Form does not override the default form generator, which means that if we want to edit or style a form from a generator we would need to rebuild the entire form from scratch. Upon installation, Simple Form should over ride the default Rails form generator thus making the forms easier to edit and deal with in the future.
Does anyone have any suggestion for where to start?

When you run the install generator, it is supposed to copy a the form scaffold template to your app, and Rails should pick that up whenever you run new scaffolds going forward.
Are you looking for another generator?

To clarify, when I run the install generator it is not copying the simple_form template to the scaffold template.
System configurations:
ruby "3.1.1"

# Bundle edge Rails instead: gem "rails", github: "rails/rails", branch: "main"
gem "rails", "~> 7.0.3", ">= <IP_ADDRESS>"

gem 'simple_form'

Commands :
rails g simple_form:install
rails g scaffold Foo bar

output:
Default form scaffolding is still present (shown below)
<%= form_with(model: foo, class: "contents") do |form| %>
  <% if foo.errors.any? %>

<%= pluralize(foo.errors.count, "error") %> prohibited this foo from being saved:

        <% foo.errors.each do |error| %>

<%= error.full_message %>

        <% end %>

  <% end %>

    <%= form.label :bar %>
    <%= form.text_field :bar, class: "block shadow rounded-md border border-gray-200 outline-none px-3 py-2 mt-2 w-full" %>

    <%= form.submit class: "rounded-lg py-3 px-5 bg-blue-600 text-white inline-block font-medium cursor-pointer" %>

<% end %>

Expected output:
Running rails g simple_form:install should copy the simple_form template to my form scaffolding template

** Update **
I just created a fresh app and the template generator worked, I have no idea why the generator wasn't invoked in my previous application. I thought that the issue might have had something to do with the simple_form-tailwinded gem so I once again ran the installation commands and the generator was invoked, simple_form seems to be working properly.


get pid_ns of a process from the pids[PIDTYPE_PID] instead of nsproxy

the code use task->nsproxy to get the pid_ns of a process, but it's not process's real pid_namespace
static __always_inline u32 get_task_pid_ns_id(struct task_struct *task)
{
    return task->nsproxy->pid_ns_for_children->ns.inum;
}

just as man pid_namespace described:
task->nsproxy->pid_ns_for_children is a ns which the children of the process will be in

Calls to setns(2) that specify a PID namespace file descriptor
and calls to unshare(2) with the CLONE_NEWPID flag cause children
subsequently created by the caller to be placed in a different
PID namespace from the caller.  (Since Linux 4.12, that PID
namespace is shown via the /proc/[pid]/ns/pid_for_children file,
as described in namespaces(7).)  These calls do not, however,
change the PID namespace of the calling process

because pid_ns can be nested, so the implementation is as follows:
static __always_inline u32 get_task_pid_ns_id(struct task_struct *task)
{
    unsigned int level = task->pids[PIDTYPE_PID].pid->level;
    return task->pids[PIDTYPE_PID].pid->numbers[level].ns.inum;
}

Codecov Report

Merging #767 (710e4d2) into main (fd5028f) will decrease coverage by 35.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #767       +/-   ##
==========================================
- Coverage   43.51%   8.03%   -35.49%
==========================================
  Files          24      29        +5
  Lines        8562    9348      +786
==========================================
- Hits         3726     751     -2975
- Misses       4353    8494     +4141
+ Partials      483     103      -380

Impacted Files
Coverage Δ

KubeArmor/monitor/processTree.go
0.00% <0.00%> (-95.70%)
:arrow_down:

KubeArmor/monitor/logUpdate.go
0.00% <0.00%> (-71.72%)
:arrow_down:

KubeArmor/monitor/syscallParser.go
0.00% <0.00%> (-64.82%)
:arrow_down:

KubeArmor/core/crioHandler.go
0.00% <0.00%> (-63.87%)
:arrow_down:

KubeArmor/core/containerdHandler.go
3.33% <0.00%> (-53.76%)
:arrow_down:

KubeArmor/core/dockerHandler.go
0.00% <0.00%> (-52.81%)
:arrow_down:

KubeArmor/feeder/policyMatcher.go
0.00% <0.00%> (-48.75%)
:arrow_down:

KubeArmor/enforcer/appArmorProfile.go
0.00% <0.00%> (-45.93%)
:arrow_down:

KubeArmor/core/kubeUpdate.go
4.68% <0.00%> (-44.34%)
:arrow_down:

KubeArmor/enforcer/runtimeEnforcer.go
0.00% <0.00%> (-43.69%)
:arrow_down:

... and 13 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 37002bf...710e4d2. Read the comment docs.


Added sci chat workshop

Added sci-chat workshop's ARR commitment deadline

Hi,
Something seems to have gone wrong here. For adding a new commitment venue you should only edit the venues page:
https://aclrollingreview.org/dates
See other recently accepted PRs for examples.

Hey,
Yeah, my bad, I am doing a proper PR now.
Sorry for the trouble!


Pass kaldi error

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 57 (7.02%)  changed or added relevant lines in 6 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.6%) to 46.284%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sage/inout/voice.py
0
2
0.0%

sage/inout/socket.py
0
3
0.0%

sage/inout/terminal.py
0
3
0.0%

sage/run.py
0
4
0.0%

sage/asr/kaldi.py
0
14
0.0%

sage/bot.py
4
31
12.9%

Files with Coverage Reduction
New Missed Lines
%

sage/bot.py
1
34.74%

sage/asr/kaldi.py
2
0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.6%

Covered Lines:
492

Relevant Lines:
1063

💛  - Coveralls


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Vite 3 Regression] Vue 3 Component CSS not bundled (SSR)

Describe the bug
I've recently hit a bit of a stumbling block while upgrading to<EMAIL_ADDRESS>+<EMAIL_ADDRESS>+ @vitejs/plugin-vue@3
Everything seems to be working fine but when running a SSG build, some (random) components CSS are not getting included. Everything works fine in vite dev mode, this is only when running a vite build
It 'seems' to be an upstream issue with either Vite or the Vite Vue plugin, as my setup this works fine with<EMAIL_ADDRESS>+ vite@2 + @vitejs/plugin-vue@2
There are no errors with the build and there does't seem to be any reason for the specificity of the components that don't have their CSS bundled.
@brillout said to raise an issue here (and tag him)
You can see below, from simply upgrading Vite, the generated CSS files goes from 29 to 19.
With Vite 2

With Vite 3

The component UolSiblingNav is included on both pages but the component CSS does not get included on http://localhost:3000/page2.
Page 1

Page 2

Reproduction
https://github.com/nonbreakingspaceltd/vite-3-css-reproduction/
System Info
System:
    OS: macOS 12.4
    CPU: (10) arm64 Apple M1 Max
    Memory: 11.77 GB / 64.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: 3.1.1 - ~/.nvm/versions/node/v16.15.1/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 103.0.5060.134
    Firefox: 101.0.1
    Safari: 15.5
  npmPackages:
    @vitejs/plugin-vue: 3 => 3.0.1
    vite: 3 => 3.0.2

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[x] The provided reproduction is a minimal reproducible example of the bug.

Note: Using
legacy: {
    buildSsrCjsExternalHeuristics: true,
  },

Trying to get a repro together, but it's very hard to break my monorepo down

Reproduction added at: https://github.com/nonbreakingspaceltd/vite-3-css-reproduction/

Removing this line fixes this.
https://github.com/nonbreakingspaceltd/vite-3-css-reproduction/blob/6811a3795226b0f415b08e0187dd82fa6db238c4/packages/core/components/package.json#L29
I think the behavior here is correct because importing vue files has a side effect of injecting css.
To prevent the file to be tree-shaked, I think sideEffects field needs to be:
"sideEffects": ["**/*.vue"],

@sapphi-red thank you for that.
Is there any specific reason this wasn't a problem in v2 (just for my knowledge and anyone else stumbling upon this issue)?

I'm not sure about the reason of that. 🤔


Internal Error - Unknown encoding: 16

I'm trying to run a simple example program using  gakonst/ethers-rs but it seems to fail with the following error message:
Error: (code: -32603, message: Internal Error - Unknown encoding: 16. Please report a bug at <https://github.com/aurora-is-near/aurora-relayer/issues>, data: Some(Object({"cf-ray": String("6d968b7d12f73fd2-YYZ"), "host": String("relayer008.west.us.backend.internal.aurora.dev")})))

this is the Rust program:
use ethers::prelude::Provider;
use ethers::prelude::*;
use ethers::providers::Middleware;
use ethers::providers::Ws;
use std::time::Duration;
#[tokio::main]
async fn main() -> anyhow::Result<()> {
    let ws = Ws::connect("wss://mainnet.aurora.dev").await?;
    let provider = Provider::new(ws).interval(Duration::from_millis(2000));
    let mut stream = provider.watch_blocks().await?.take(5);
    while let Some(block) = stream.next().await {
        dbg!(block);
    }

    Ok(())
}

This same program works if I change the endpoint to be any other EVM rpc.
Any idea what's happening? Thanks!.

I am having the similar issue when called the eth_newBlockFilter method.
Calling the method using the following cUrl code
curl https://testnet.aurora.dev/ -X POST -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"eth_newBlockFilter","params":[],"id":1313161555}'
will return an error likes this
{"jsonrpc":"2.0","id":1313161555,"error":{"code":-32603,"message":"Internal Error - Unknown encoding: 16. Please report a bug at <https://github.com/aurora-is-near/aurora-relayer/issues>","data":{"host":"r001.westcoast.us.testnet.internal.aurora.dev","cf-ray":"6fe6738514018b4d-SEA","request_body":{"jsonrpc":"2.0","method":"eth_newBlockFilter","params":[],"id":1313161555}}}}

Hi @0x2447196. We released a fix. Please check if it helped.

Hi @0x2447196, this should be resolved by now, please reopen if this issue will reoccur again.


[1.11] Bump Mesos to nightly 1.5.x ba960ed

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/0216002744517a6053fd782b6b4dc3d6cf77dd5e...ba960ed45e80119eadf398abd72609538fbc983e

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

Closing in lieu of #3621


Workflow to file a bug for an issue failed with 401

A workflow for #17106 failed with an error:
Error: Failed request: (401)
    at RestClient.<anonymous> (/home/runner/work/azure-pipelines-tasks/azure-pipelines-tasks/node_modules/typed-rest-client/RestClient.js:202:31)
    at Generator.next (<anonymous>)
    at fulfilled (/home/runner/work/azure-pipelines-tasks/azure-pipelines-tasks/node_modules/typed-rest-client/RestClient.js:6:58)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  statusCode: 401
}

See https://github.com/microsoft/azure-pipelines-tasks/actions/runs/3353243842/jobs/5555903116#step:6:118
One of the PATs apparently has expired:
https://github.com/microsoft/azure-pipelines-tasks/blob/52c896b1a41ade68b76807aaefa624fd36797b11/ci/file-bugs.js#L7-L8

Same issue? https://github.com/microsoft/azure-pipelines-tasks/actions/runs/3324377063/jobs/5495861070#step:3:25

@jessehouwing apparently the workflow for every recently opened issue in this repo fails.

It may explain the lack of interactions with the issues.

Yeah, I noticed it too. Went through last few pages of issues, and none of them have any responses whatsoever.
I pinged few folks internally.

Looks like folks have prioritized the node 16 upgrade.

Hi @abatishchev
Thank you for reporting that, we will have a look

@abatishchev tokens updated, please re-open this issue if it is still actual.

@kirill-ivlev please have a process in place to avoid this from happening in the future too. thanks!

@kirill-ivlev can you please see if this is the same issue? "labeler" fails with 403: https://github.com/microsoft/azure-pipelines-tasks/actions/runs/3353238575/jobs/5603855160#step:6:123


Fix d2l-input-number so that invalid property reflects internal d2l-input-text validation state.

DE50568
This PR updates the d2l-input-number component so that it's invalid property properly reflects the validation rules defined for both d2l-input-number as well as the underlying d2l-input-text element. When the validity of the underlying element changes (ex. due to required), we need to update the validity of d2l-input-number.

Seems valid!

The validation stuff is kinda hairy.


Dockerfile test-path

Fixes the path of the testfile-directory

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Address initial jsonata.js test suite errors

@msmiths has updated the pom.xml to retrieve the jsonata.js tests and has created an AgnosticTestSuite to execute these tests. Many new errors / incompatibilities were created. This issue represents the initial set of changes to address these new problems. Where a particular set of tests are failing because that feature has not yet been implemented, a new issue will be created to track them and that test will be temporarily disabled to allow review of other problems. Separate comments will be created here for the problems that have been fixed under this issue.

Calling a function with just the $ reference like below failed to parse and then failed to execute:
blocks:case005: (  $func := function($arg) {$arg.Account.Order[0].OrderID};  $func($)) returns "order103"
The function call definition did not expect the context reference variable $ so it failed to parse. The MappingExpression.g4 was updated to allow correct parsing making what follows the $ optional:
| DOLLAR (('.' expr) | (ARR_OPEN expr ARR_CLOSE))?       # context_ref
Correcting this allowed the expression to parse. However, the visitPath was failing with an NPE because the rhsCtx was null so attempting to references this in resolvePath failed.
Adding tests for null rhsCtx and returning the context itself resolved this problem.

Calling an empty expression like "()" doesn't parse.
blocks:case000: "()" expects to return null (** no match**)
The parens declaration failed to allow an empty expression so was changed to make the internal expression optional:
| '(' (expr (';' expr)*)? ')'                            # parens
This allows the expression to be evaluated and return null (** no match**) as expected.

Calling an empty trailing expression in parens fails to parse
blocks:case002: "(1; 2; 3;)" expects to return 3 but fails to parse
The parens declaration failed to allow a semicolon to end the list of expressions in parens so was changed to allow this:
| '(' (expr (';' (expr)?)*)? ')'                         # parens
so the expected return value succeeds.

Starting a new expression block (parens) should start a new environment for variable assignment that lasts the lifetime of the block.
blocks:case003: ($a:=1; $b:=2; $c:=($a:=4; $a+$b); $a+$c) returns 10 because when $a is overridden to become 4, making c=6 inside the c block, it does not return to be 1 outside that block, so instead of 1+6=7 we are reporting 4+6=10.
Need to implement a stack of variable and function declarations when entering a block, and popping them when exiting a block. When searching for variables or function declarations, search current and if not found and there is another environment on the stack, pop, search, push to see if the parent block environment has the reference desired.

simple-array-selectors:case005 used a negative index and was returning null (** no match **)
In the visitArray, I had to ensure the group being reviewed in a SelectionArrayNode was an array so the resolvedIndexes can be tested for non-array elements.

** on a null should return null (** no match **)
descendent-operator:case016
Added test for null returnArray when descendants is null for visitDescendant

"foo".**.fud
descendant-operator:case003
foo.**."fud" expected to return [ "hello", "world" ] but was not testing the string "fud" in the path
descendant-operator:case004
Added the ability to transform a StringContext in the visitPath and in the visitDescendants into an IDContext so it would resolve correctly.

Account.Order.Product[$lowercase(Description.Colour) = "purple"][0].Price is returning just 34.45 but the test expects [ 34.45, 34.45 ]
predicates:case003
Not sure why the answer expects two entries, when [0] clearly identifies the first of the resulting 2 cell array. Will open as a new Issue and disable for now.

in in ["hello","world"]
inclusion-operator:case004
"world" in in
inclusion-operator:case005
IMHO, this correctly reports a parse error because in is an operator to find a value in an array per the documentation here: https://docs.jsonata.org/comparison-operators#in-inclusion
If the test were rewritten as "in" in ["hello","world"] we correctly report this as false. Likewise, for "world" in "in" we report false, however, the current parser doesn't even recognize in in as anything worth parsing so it is ignored. We could add specific "rules" for "in in" phrases but this seems like overkill. jsonata.js defines a Strings as follows at https://docs.jsonata.org/expressions
String literals can also be created by enclosing the sequence of characters in quotes. Either double quotes " or single quotes ' can be used, provided the same quote type is used for the start and end of the string literal. So, having just in without quotes is the operator.

dataset7: content.integration.$lowercase(name)
inclusion-operator:case008
We were not expecting a path in the parameter to $lowercase, so we hadn't allowed 2 parameters. This has been fixed.

IGNORED: case003.jsonata
comments:case003 we aren't parsing a function to reference $$ in the argument list
line 24:13 at [@85,698:698=')',<9>,24:13]: mismatched input ')' expecting '.'

FIXED: $trim() on null throws an exception when it should return null (** no match **)
function-trim:case002
Fixed TrimFunction to allow argCount == 0 to return null

FIXED: $fsds
missing-paths:case005
This is an reference to an undefined variable so rather than throw an exception, we return null now

IGNORED: $length(""𝄞💩")
function-length:case016
The emojis are not read correctly in Java so we get a 4 character string rather than a 2 character string and therefore our length is 4 not 2.

DEFERRED: ($twice:=function($f){function($x){$f($f($x))}}; $add3:=function($y){$y+3}; $add6:=$twice($add3); $add6(7))
high-order-functions:case000
This requires lambda functions

FIXED: $sqrt(4) expected 2 but got 2.0
function-sqrt:case000
Need to transform to Long if no fractional part.

FIXED: $lookup(Account.Order.Product.ProductID, "Product Name")
lookup-function:case002
expected null (** no match **) but got an empty array
Fixed to return null if nothing is found.

FIXED: $lookup($$, "gust")
lookup-function:case003
expected "null" but got parse error
Fixed ROOT to allow just "$$" without any path

FIXED: $spread(blah)
function-spread: case002
blah is undefined so the argument becomes null. Added tests to ensure a null response is returned when the argument to the function resolves to null.

FIXED (not not happy with approach): $string($spread(function($x){$x*$x}))
function-spread:case003
The lambda function declaration as a parameter to spread was parsed as null so $string(null) was returning "null". Added check on the type of ExprContext referenced so it would return a "" TextNode. Not sure this makes sense as $spread is supposed to work on objects, and having it applied to a lamba function declaration should probably return null (** no match **)...
        if (argObject == null) {
           ExprContext exprCtx = ctx.exprValues().exprList().expr(0);
           if (exprCtx instanceof Function_declContext) {
              argObject = new TextNode("");
           }
        }

FIXED: Account.Order.Product.(Price < 30 ? "Cheap")
conditionals:case006
We didn't expect a ternary expression to have only two arguments based on https://docs.jsonata.org/other-operators#--conditional. Regardless, we now support the missing false condition and return null.

FIXED: ["Red"[$$="Bus"], "White"[$$="Police Car"]][0]
conditionals:case001
Needed to add logic to processArrayContent to not add a null to the array as it signifies undefined and we don't store undefined in arrays.

FIXED: $number() with 5 set as context
context:case000
NumberFunction wasn't set up to expect nothing passed as a trigger to read the context. Fixed the test for using the context as input so it is true if there are no arguments passed.

FIXED: ($append($data.a, [2]); $data.a) using binding: {"data": {"a" : [1]}}
function-append:case005
We didn't support data binding in the test rig AgnosticTestSuite so several classes were refactored to allow access to the environment to set the data bindings during the TestCase execution.

FIXED: $trim()
function-trim:case002
We were throwing an exception (which is what try.jsonata.org does) but the test expects null (** no match **) so the TrimFunction was changed to return null.

FIXED: [1..5].("Item " & $string())
context:case002
Changed useContext to be true if there are empty parameters

IGNORED-DEFERRED: $encodeUrlComponent("?x=test")
function-encodeUrl-component:case000-002
We haven't created the function for encodeUrlComponent

FIXED: {"test":()}
object-constructor:case006
We didn't test for the value returned to be null. Now, if null we don't add it to the object.

IGNORED-DEFERRED: Account.Order.Product{$string(ProductID): Price}
object-constructor:case010
We don't expect an object construction as part of the path.

FIXED: $exists($exists)
function-exists:case017
ExistsFunction was only checking for the existence of variables so added a check for declared functions and program functions.

IGNORED: $exists(null)
function-exists:case006
Github repo is down so can't test with try.jsonata.org but local jsonata.js reports null yet the test claims it should report true so suspect test is wrong.

FIXED: $max([])
function-max: case003
$min([])
function-max: case017
We were throwing an exception but should have signaled problems by returning null

FIXED: [-2..2].($*$)
range-operator:case006
FIXED: [false..$x]
range-operator:case020
[1..10000000] ~> $count()
FIXED-KLUDGE? range-operator:case021 -- returned the right number but not in scientific notation which the test case specified so I expand all scientific notation before comparison of Strings in AgnosticTestSuite
I was treating $ and $$ as special variables, but have refactored the context stack to be part of the FrameEnvironment and changed getVariable to set $ and $$ from the context stack.

FIXED: Made changes to use SelectorArrayNode where createSequence() was used in jsonata.js

FIXED function-keys tests 001, 003, 004

FIXED comparison-operators tests 010, 027

FIXED function-sum:case002
Added number formatter for 2 decimal places as default for String concatenation.

FIXED function-reduce:case006
FIXED function-reduce:case007
Was throwing an exception rather than returning null for value that evaluates to null
Needed to test for declared functions before standard functions so you could override standard functions.

There was a bug in how I was processing the Content_refContext. I needed to replace the child0 in the context being visited, and not in the original context because the stack can change content on successive calls.

DEFERRED: added SKIP_TEST for all Lambda functions

FIXED: (  $seq := 1;  $reduce($seq, function($x){$x}))
hof-reduce:case008
Added a check for the function signature to ensure at least 2 variables are specified in the variable list. Now throws the exception: "The second argument of reduce function must be a function with at least two arguments" expected if less than 2 arguments

FIXED: (  $concat := function($s){function($a, $b){$string($a) & $s & $string($b)}};  $comma_join := $concat(' ... ');  $reduce([1,2,3,4,5], $comma_join))
hof-reduce:case001
I had to expand the FrameEnvironment to track named declared functions as well as variables referencing either named declared functions or Jsonata functions as exhibited by the hof-reduce:case001

FIXED: added URL encode and decode functions
Note: AgnosticTestSuite only tested for EvaluateException or ParseException, but I throw an EvaluateRuntimeException (to avoid having to explicitly catch exceptions for functions) so I updated the AgnosticTestSuite to allow this:
} catch (Throwable ex) {
   if (testCase.expectParseOrEvaluateException && (ex instanceof ParseException || ex instanceof EvaluateException || ex instanceof EvaluateRuntimeException)) {
      notifier.fireTestFinished(testCase.getDescription());
   } else {
      notifier.fireTestFailure(new Failure(testCase.getDescription(), ex));
   }
}

FIXED: added TypeFunction for function-typeOf tests

FIXED: function-power:case001 $power(4, 0.5)
FIXED: function-power:case005 $power(-2, 1/3)
Checked for whole number returns and changed to LongNode
Improved testing for NaN and Infinity to return proper error message

FIXED: $sum($split("12345", "").$number($))
function-split:case004
Needed to allow for argument count test to take into account that we are reading the arguments from the call stack using the context reference.

DEFERRED: tail-recursion tests
Need to implement time and count in AgnosticTestSuite

DEFERRED: transforms
Not implemented yet. See #47

FIXED: $substringAfter("Hello World", "ld")
function-substringafter
Needed to take length of search string into account and return empty string if we are at the end of the string.

FIXED: $boolean(-0.5) to return true
function-boolean
Needed to allow double value tests

DEFERRED: Context variable binding (@)
Not implemented see #48
See https://docs.jsonata.org/path-operators#-context-variable-binding

DEFERRED: Number Formatting
Not implemented see #49
See https://docs.jsonata.org/numeric-functions#formatnumber

FIXED: function-pad:case0010
Had to address unicode when considering lengths.

FIXED: $join([1,2,3], ", ")
function-join:case009
Needed to check for isTextual rather than isValueNode for elements of the array

( $foo := "defined"; ( $foo := nothing; $foo ) )
variables:case011
Because variables can now hold null values, we need to check whether the current _variableMap contains the variable key to allow null to be returned.

FIXED: [1, $two, ["three", $four]]
array-contructor:case006
FIXED: ["foo.bar", foo.bar, ["foo.baz", foo.blah.baz]]
array-constructor:case007
FIXED: foo.blah.baz.[fud, fud]
array-constructor:case011
Added tests for first and last step references to SelectorArrayNode to use groups, and added logic to track array construction to append vs add arrays

DEFERRED: # Positional  Variable Binding
See Issue #50

DEFERRED: expr": "$.{ type: $average(value), kind: $sum(value) }
object-constructor:case023
See Issue #51
Need to add fieldExpr to MappingExpressions.g4 to recognize the entire expression. At present tree only has the $ and the rest is ignored.

FIXED: function-string:case023
Needed to process additional parameter for prettify option

FIXED: $number()
context:case000 using data_0.json
Needed to remove the test for ctx.parent() == null when determining to use context.

$string(["string", 5], true)
FIXED: function-string:case027
Needed custom pretty printer for array indentation and newlines

FIXED double tests to take into account Infinity and NaN values

$string(1e21)
FIXED: function-string:case027
Needed to emulate JavaScript scientific notation for Java

FIXED: $string({  "string": "hello",  "number": 78.8 / 2,  "null":null,  "boolean": false,  "function": $sum,  "lambda": function(){true},  "object": {    "str": "another",    "lambda2": function($n){$n}  },  "array": []}, false)
function-string:case027
Updated Object construction to address function references as empty strings.

FIXED: determining when to use context variable as the first argument for functions.

DEFERRED: function-tomillis
Need to review the formatting options allowed. See #52

DEFERRED: partial-applications
Need to implement this  feature using ? as a function variable placeholder. See #53

FIXED: $average(1)
function-average:case007
Added logic to return long for whole number values

hof-zip-map: case000 and case001 fail because the array build with zip has an extra array wrapper, so it isn't presenting arrays of numbers to the $sum function.

DEFERRED: 55=>5 won't parse but doesn't throw a parse error either See #59

FIXED: $formatBase(99.5, 2.5)
function-formatBase:case006
Had to add rounding (even down, odd up) to both the number and radix presented.

( $square := function($x){$x*$x}; [1..5] ~> $map($square) ~> $sum() )
function-applications:case011
Needed to check for Fct_chainContext as the parent to use context.

FIXED: BasicExpressionTest simpleTest("[{"b":1}][a or true]", "{"b":1}"); // jsonata 1.8.2 no longer returns null);

DEFERRED: $0 should throw a parse exception since variable names can not start with a number
See issue #64
Commented out test in InvalidSyntaxTest

Commented out segment in pom.xml to enable jar files to be built

1.4.2 Release incorporates these fixes


Couldn't find CABLE Output (VB-Audio Virtual Cable) in application settings.

According to Audio Routing setup guide, after setting up VoiceMeeter Banana, I have to set input device to CABLE Output (VB-Audio Virtual Cable) in the application settings. The problem is, I couldn't find it anywhere. All I found were VoiceMeeter Output (VB-Audio VoiceMeeter VAIO) and VoiceMeeter AUX Output (VB-Audio VoiceMeeter AUX VAIO)

Here is my VoiceMeeter Banana setup.

Thank you.

Oh right, I didn't install VB-Cable. Sorry for that.


[#1894] improvement(client-java): Add default cases in switches

What changes were proposed in this pull request?
Add a default case to the switch statements to call super.accept(errorResponse)
Why are the changes needed?
code optimization.
Fix: #1894
Does this PR introduce any user-facing change?
No
How was this patch tested?
./gradlew :client:client-java:test

Hi @yuqi1129 this pr is ready for review and could you help review it? Thank you


Devouring Plague odd SpellDmg% modifier behaviour

Describe the bug
Devouring Plague deals significant less damage in the sim when choosing some other % damage debuff than the DK debuff.
To Reproduce
Steps to reproduce the behavior:

Go to Shadow Sim
Click on P4 Preset
Sim
Go To Settings
Switch from DK Spell % debuff to curse of elements or moonkin debuff
Sim
~600 DPS less overall

Expected behavior
As far as I am aware all those three debuffs should behave the same in terms of % modifier.
Sim Links and Screenshots
DK Debuff vs Moonkin Debuff
Desktop (please complete the following information):

OS: Win 11
Browser Firefox
Version 119

EP and Crypt Fever increase disease damage taken by the target by 30%
https://www.wowhead.com/wotlk/spell=49632/crypt-fever
https://www.wowhead.com/wotlk/spell=51161/ebon-plaguebringer


Integrate new test runner and test framework api's in stryker

Integrate the new TestRunner and TestFramework apis in Stryker. Make sure to support the options we already had, maybe rename them.

Work on this has started in the branch: https://github.com/infosupport/stryker/tree/next-runner

The following command:
node dist/src/Stryker.js -s test/sampleProject/src/Add.js,test/sampleProject/src/Circle.js -t test/sampleProject/test/AddSpec.js,test/sampleProject/test/CircleSpec.js

Should print the following output to the console:
INFO: Running initial test run
INFO: Initial test run succeeded
INFO: 11 Mutants generated
INFO: Waiting for all code coverage reports to finish
INFO: All code coverage reports have been collected
INFO: Testing mutants:
........S
Mutant survived!
C:\Users\simonl\Documents\workspace\stryker\test\sampleProject\src\Circle.js: line 3:22
Mutation: Math
-     return 2 * Math.PI * radius;
+     return 2 * Math.PI / radius;

Tests ran:
    C:\Users\simonl\Documents\workspace\stryker\test\sampleProject\test\CircleSpec.js

9 mutants tested.
2 mutants untested.
0 mutants timed out.
8 mutants killed.
Mutation score based on code coverage: 88.89%
Mutation score based on codebase: 72.73%

The messages about code coverage may be omitted due to the new coverage system. The 'tests ran' should now show the names of the specs which have been ran.


fix Add FE azure object storage UT and fix log

What problem does this PR solve?
Add an UT for azure blob storage in FE, this is a test framework, more UTs should be added to cover all functionalities.
Problem Summary:
Release note
None
Check List (For Author)

Test

[ ] Regression test
[x] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[x] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall

run buildall


Proposal: in Volto's code, rename OldIcon => Icon, Icon => VoltoIcon

This is a pet peeve of mine, but when working with Volto code I always have to check which type of icon import I'm dealing with (Semantic-UI or Volto's component). Making this small refactoring (which only refers to how the imports are handled, it wouldn't be a breaking change), would make the code nicer and easier to understand.
So, instead of:
import { Icon as OldIcon } from 'semantic-ui-react';
import { Icon } from '@plone/volto/components';

Now we would have:
import { Icon } from 'semantic-ui-react';
import { Icon as VoltoIcon } from '@plone/volto/components';

Even better, in @plone/volto/components/index.js, an alias can be created so that: import { VoltoIcon } from '@plone/volto/components works without aliasing in the import.


How to create a debug version of this program

Hi, andre
I have compiled a release version of this program, it works well with my input data . But when i use my data in debug version, it can't work , i.e some components of this program do not work. Do you have any suggestions on build a debug version of this program in visual studio ?

Hi @zjulk,
I'm not 100% sure but after reading your issue in the main mvs-texturing repo (nmoehrle/mvs-texturing#189) I assume it might be because the debug build of mvs-texturing uses a release build of libjpeg. Can you try to compile/link a debug build of mvs-texturing against a debug build of libjpeg?

Hi @zjulk, I'm not 100% sure but after reading your issue in the main mvs-texturing repo (nmoehrle#189) I assume it might be because the debug build of mvs-texturing uses a release build of libjpeg. Can you try to compile/link a debug build of mvs-texturing against a debug build of libjpeg?

I try to set CMAKE_CONFIGURATION_TYPES as Debug;Release in the CmakeLists under 3rdparty, mve, recompiled it . Recompile mvs-texturing with this libs, then i can run a debug version of it successfully. Thanks for your reply.


Discrepancy in setters for non-primitive data types between snappi and gosnappi

Version information
0.1.29
Describe the bug
Setters for Non-Primitive data types are available in gosnappi but not in snappi
snappi
No setter for required_object
    @property
    def required_object(self):
        # type: () -> EObject
        """required_object getter

        A required object that MUST be generated as such.

        Returns: EObject
        """
        return self._get_property('required_object', EObject)

gosnappi
setter for required object
// SetRequiredObject sets the EObject value in the PrefixConfig object
// A required object that MUST be generated as such.
func (obj *prefixConfig) SetRequiredObject(value EObject) PrefixConfig {

	obj.RequiredObject().SetMsg(value.Msg())
	return obj
}

Python should implement the .set() (https://github.com/open-traffic-generator/openapiart/issues/168) method which covers the gap.
.required_object().set(...)

Closed in favor of https://github.com/open-traffic-generator/openapiart/issues/168


fix(edge-stacks): add an endpoint to delete the status of an edge stack EE-2432

Counterpart of https://github.com/portainer/agent/pull/237

lint fails, should work if you rebase on current develop


chore(master): release 0.9.0

:robot: I have created a release beep boop
0.9.0 (2024-10-01)
Features

add entity matching temp solution (5cf1564)
add tests and minor modifications (2fb488f)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.9.0
:sunflower:


[Amsterdam - Holanda] Booking.com - iOS Developer

Booking.com
Booking.com BV (the company behind Booking.com™, the market leading online hotel reservation service in the world) and/or its various support companies throughout the world are looking for people to support the business in the fast-growing hotel markets. We are looking for engineers with strong experience developing native applications on the iPhone or iPad.
Descrição da vaga
Our guiding principle is that by helping our users we help our business, so we listen to our customers and give them features they want to use.  As a developer in our mobile team you’ll be a key part of the idea generation process, working in a small start up-like group which takes full ownership of its part of our platform. You’ll be working on the iterative improvements we make as part of our high-speed release cycles, as well as looking ahead and implementing broader changes to our mobile app.
As iOS developer you will:

Develop new features to our iOS products
Optimize UI for performance
Monitor and understand customer behavior
Build native iOS components
Solve issues prioritizing based on customer impact
Collaborate with back end developers, designers, product owners to constantly improve customer experience within iOS platform

Posição
iOS Developer
Tipo de contratação
Presencial
Requisitos

Obrigatórios

5+ years of experience as a developer
3+ years iOS application development using iOS SDK
Knowledge of Cocoa, Objective-C and Xcode
Knowledge of Object Oriented concepts and a strong passion for software development
Self-motivated to explore new technologies, applications and environments
Excellent attention to detail
Self-starter, highly motivated, team-player
Fully comfortable working in English, both in writing and spoken

Diferenciais

Experience with Apple Approval & Distribution Process, Ad Hoc & Enterprise Distribution
Have built an approved store App
Experience working with A/B testing
You are passionate about open source and contribute
You’re not afraid to solve deep technical challenges of a whole new level at a huge scale
You have an intrinsic curiosity in technological innovations and you are always on top of the latest trends

Benefícios
We are a performance-based company that offers career advancement and lucrative compensation, including bonuses and stock potential.  We also offer what we call the “Booking Deal” with other competitive perks and benefits.  The Technology department has monthly hackathons, training and attends/speaks at global conferences.
This position is open to worldwide candidates and in the case of relocation, we will assist you with a generous relocation package, ensuring a smooth transition to working and living in Amsterdam.  We have successfully relocated 300+ Technology professionals to Amsterdam in the last year!
The hiring company Booking.com BV is the company behind Booking.com™, the market leading online accommodation reservation service in the world. Booking.com BV is looking for a iOS Developer in Amsterdam, the Netherlands. All references to “Booking.com" refer to Booking.com BV.
Como aplicar
http://grnh.se/ujghd8
De preferência falar comigo se aplicar!
Contato
<EMAIL_ADDRESS>

@mauriciomeirelles Já fecharam a vaga? Pode fechar a issue?

Não fechou não!
Pelo que eu saiba o objetivo era contratar uns 20 esse ano ainda, só conseguiram contratar 5... Então tem muitas vagas ainda, se for bom e quiser morar em Amsterdam só avisar :)

@mauriciomeirelles mandei email.

@mauriciomeirelles a vaga Ainda está aberta?

Yep, sempre hahaha
Ouvi falar que o ritmo de contratação diminuiu um pouco mais ainda precisam de mais gente do que conseguimos achar :)

Olá Maurício, gostaria de saber sobre o processo seletivo. É muito difícil? Obrigada.

Então, não acho que seja difícil se você esiver preparada. As entrevistas fora do BR normalmente pedem um pouco mais de general programming e conceitos, nessa parte que normalmente as pessoas se complicam um pouco... também é legal estudar mais a fundo como o iOS / Objective-c realmente funcionam.
Se tiver alguma dúvida específica me chama no slack :)

opa, ainda estão contratando pra essa vaga?

@mauriciomeirelles opa, ainda estão contratando pra essa vaga? x2


Using GitHub with SSH

I have read and understood the Contributor's Certificate of Origin at the end of the template and I hereby certify that I meet the contribution criteria described in it.
Signed-off-by: (Johannes Bluhm<EMAIL_ADDRESS>

Hey, I'm sorry to inform you that we are closing your PR for the following reasons:

the scope of the tutorial is not really sufficient.
this tutorial of PR, which was already submitted before your tutorial, covers the scope of yours for the most part.

Hello,

what would a sufficient scope be? For example this PR seems to be even shorter and appears to be? https://github.com/netcup-community/community-tutorials/pull/20
I don't really see the issue with this, about half of the tutorials here explain how to generate SSH keys.


exception accured when dynamic remove charts

I use a list to control charts add/remove on the view dynamically,  add is perfect, but when do remove, error like below sometimes happened:
「Chart 7add47026b8146339fc7466da39115e5 must have at least one series」
Did I miss some setting when remove the chart?

Hello, I was able to reproduce this by switching pages multiple times on the demo site.

#336

I have made some improvments in v2.0.0.
Could you please update and close if it is fixed.
Thank you

I have made some improvments in v2.0.0. Could you please update and close if it is fixed.
Thank you

I will try it, thank for the improvement

I have made some improvments in v2.0.0. Could you please update and close if it is fixed.
Thank you

I update to v2.0.0, 「Chart 7add47026b8146339fc7466da39115e5 must have at least one series」has gone.
I try to add/remove charts like what I done before, the view update will not be stopped by the error, but sometimes, after I remove a chart, one of the left charts will  stop update itself (I still do not know the reason), but the exception isuue has gone, I will close the issue.


[SPARK-10088] [sql] Add support for "stored as avro" in HiveQL parser.

Merged build triggered.

Merged build started.

Test build #41153 has started for   PR 8282 at commit 2256b43.

Classic copypasta. LTGM.

Test build #41153 has finished for   PR 8282 at commit 2256b43.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41153/
Test PASSed.

Thanks!  Merging to master and 1.5.


Submit Flutter interview questions

I would like to work on this issue. I have been working for the last 1 year as a Flutter Freelancer and have interviewed some developers as well. I could ask some questions related to flutter.

Feel free to contribute to this project!😊

I would like to be assigned to this issue

I would like to be assigned to this issue

I would like to be assigned to this issue as well please 😃

Hi, Assigned both of you.

Can i also work on this issue by adding some valuable questions

Sure.

Hello there, I wish to contribute to this, Can you assign me too?

Hello! Can I please contribute to this work? Please?

Assigned. @mayankparida

I have sent a pull request. Please do check n merge. Thank you very much.

Can I please contribute to this work? Please?

sure @meetxpress

so can I create a PR?


Always use UTF-8 encoding for git subcommand stdout

On Windows, this will default to CP1252, while the text itself is still
UTF-8, leading to text encoding errors.

@linusg I am using scc without problems at the moment, probably because it only outputs ASCII. But it does have the same problem, so I pushed a fix.


General changes

Using 'export' instead of 'module.exports'
Using 'export const' instead of 'export default' for exporting the 'AccountSchema' .

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Thank you for your contribution 👍

please raise an issue first (please read the contributing guide https://github.com/EddieHubCommunity/LinkFree/blob/main/CONTRIBUTING.md)
this is not consistent with the other files, all need to be consistent - not sure why only 1 file is updated?

I will close this pull request, please do step 1 first if this change is required, inculde reasons why it is required and the benefits. If you are looking for green squares, please see existing issues with the label status: ready for dev


feat: Add certificate audit log for actor.

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

Comparison is base (d515489) 10.09% compared to head (56fe0fb) 16.81%.
Report is 10 commits behind head on main.

Files
Patch %
Lines

...pi/Controllers/MarketParticipantActorController.cs
0.00%
7 Missing :warning:

...ts/MarketParticipant/V1/MarketParticipantClient.cs
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2236      +/-   ##
==========================================
+ Coverage   10.09%   16.81%   +6.72%
==========================================
  Files          83      100      +17
  Lines        4150     4531     +381
  Branches      777        0     -777
==========================================
+ Hits          419      762     +343
- Misses       3721     3753      +32
- Partials       10       16       +6

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


"Fix" github PR templates

renames the PR-templates to lowercase, like issue templates
adds a bit text to the templates
adds feature-parity
adds fix (for bug fixes and pr fixes)

changes, because currently there is no option to apply a template for a PR, only one default one

Coverage remained the same at 85.176% when pulling e7c1df74bbb981b823bd62c094a4bd11d68485ee on hasezoey:feature/githubchanges into ea1c0ec6bbebdd6b27ba4580ec39c1d3535b2084 on szokodiakos:master.


Restructure EffectShaderNodes

this file is getting very hard to work with. there is bad editor support because of all the nested functions and closures.
@azeno do you have ideas how we can structure this better so we get full visual studio convenience back?

Well as a first step we could split it up in several files using the partial keyword. Like EffectShaderNodes.ShaderFX.cs, EffectShaderNodes.TextureFX.cs etc. So you can quickly navigate to the parts you work on.
The closures/nested methods allow to add state and new arguments easily. Without them we'd need to introduce a class for each of them - have the feeling of a lot of boiler plate...

yes, I've split it into files and lifted the main methods up, it needs a few more arguments, but at least the method drop-down works now.


fix Remove this 'public' modifier sonar violations (java:S5786) part 3

Check List:

Fixes NA
Unit tests : NA
Javadoc with a code example (on API only) : NA

Integrated, thanks @ShivakumarSwamy!


The first user tooltip is shown at the bottom on group chat

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

Action Performed:

Go to staging dot on web chrome
Create a group with multiple users
Click on user details
Hover over the first user and notice that the tooltip is shown at the bottom
But now hover over the remaining users and notice that the tooltip is shown at the top

Expected Result:
The first user tooltip should be shown at top like other users on group chat
Actual Result:
The first user tooltip is shown at the bottom on group chat
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.27-2
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/3ae79da6-0069-4531-bc18-abb14e248182
https://github.com/Expensify/App/assets/43996225/6760edcd-3b6d-4588-9653-7e2d00e851bf
Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686630260368039
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Agreed, seems like this is how a tooltip is supposed to behave


I could not find what I was looking for - Hierarchical Menu Sort

Hi Algolia,
I'm looking through the documentation on InstantSearch.js and I couldn't find an example of how to sort an hierarchical menu using a sort function.
What I want to achieve is sort the hierarchical menu based on the order I have decided on so it's on based on the name of them item or the number of items underneath it.
Can you please provide an example on how to achieve this?
Thanks Algolia!

@GT24--- Any luck here?

Yup! I managed to get it at the end.
Sorry for the late reply!
Thanks so much!

I have same question, can it sort by choosen velue be first ?

I switched over to refinement list from hierarchicalMenu but they both have the sortBy attribute.
You could create a numeric  attribute to sort by and use for sorting.
Here is some code:
 S.addWidget(instantsearch.widgets.refinementList({             container: "#search-facet-categories",             attributeName: "AttributeToSortBy",             sortBy: ["name:desc"],             operator: "or",             limit: 100,             autoHideContainer: true,             templates: {                 header: "Review Value",                },            },             cssClasses: {                 active: "selected"             },             transformData: function(e) {                 return e             }         }));

ohh, I see, i try another way using
sortBy: function(a, b) {
return -1;
},
so the first (choosen) will show the first.
but i have another  problem now, is there any whay to hide the first parent in hierarchicalMenu ?
I try
showParentLevel: false,
but it just hide the second and else parent, but the first will always show up.

Not sure if this is the best way to do it but you could inspect your DOM and hide the first parent using javascript.
Something like this in jQuery
$(".ais-refinement-list--header").css("display", "none");
Instead of ais-refinement-list--header, you would replace it with the class assigned to the first parent.

I think the choosen one will display none too.


#42: implemented BFS with the same format of DFS

fixes #42 . Please review. The format is the same as DFS in the directory.

@dubesar please review.


Rails 4 session values not persisting after redirect

I'm having an issue  where session values that I set in the SessionsController are not persisted when users are redirected to a different page.
In the following controller, I am setting the user_id and redirecting to a different controller's action. When execution gets to that controller, the value I previously set is no longer available. Similar logic works in a Rails 3.2 application so I'm not sure if it's a problem with OmniAuth or what.
class SessionsController < ApplicationController
  skip_before_filter :authenticate_user!

  def create
    user = User.find_or_create_by_auth_hash(auth_hash)
    session[:user_id] = user.id
    redirect_to root_path
  end

  protected

  def auth_hash
    request.env['omniauth.auth']
  end

end

Currently Experiencing the very same issue. Although i'm not working with omniauth session variables i set do not persist to the next controller on redirect in rails


promtool http.config.file format?

Proposal
When prometheus is using basic auth all promtool commands require "--http.config.file".
Documentation says: "HTTP client configuration file for promtool to connect to Prometheus."
I have been trying to find a sample  or reference to such a file, but no luck. I am unable to connect, neither use promtool in such scenario.
Is this reference missing or is it that obvious that does not require further introduction?

It should be similar to:
https://prometheus.io/docs/alerting/latest/configuration/#http_config
But we should indeed document this.

Hello :wave:
I'll come up with proposal to document it in a PR soon!

can i take this issue ?


I301 embodied and operational energy

Solves issues #28 . #301 . #306 and #312

@martin-mosteiro there are some new conflicts due to the other pull requests, would you please check, i will do the test then.

@JIMENOFONSECA should be okay now


overlayWith creates wrong colour for overlaid image.

I am trying to overlay one PNG over another.
This is the base image: http://www.pngmart.com/files/4/Car-PNG-Transparent.png
and this is the image which I want to overlay: https://vignette.wikia.nocookie.net/fantendo/images/6/6e/Small-mario.png/revision/latest
The Mario image which I am overlaying gets false colours as you can see form this resultant image: https://ibb.co/gt8Sw8

The problem here is that your Mario image has some ICC profile incorated in it. Check here:
http://metapicz.com
ProfileCMMType | APPL
-- | --
ProfileVersion | 2.4.0
ProfileClass | Display Device Profile
ColorSpaceData | RGB
ProfileConnectionSpace | XYZ
ProfileDateTime | 2011:04:04 15:47:16
ProfileFileSignature | acsp
PrimaryPlatform | Microsoft Corporation
CMMFlags | Not Embedded, Independent
DeviceManufacturer |
DeviceModel |
DeviceAttributes | Reflective, Glossy, Positive, Color
RenderingIntent | Perceptual
ConnectionSpaceIlluminant | 0.9642 1 0.82491
ProfileCreator | LOGO
ProfileID | 0
ProfileCopyright | Copyright by LOGO GmbH, Steinfurt
ChromaticAdaptation | 1.04309 0.03181 -0.05273 0.02284 1.00121 -0.02104 -0.00117 -0.0034 0.76315
RedMatrixColumn | 0.43509 0.21826 0.01617
GreenMatrixColumn | 0.39258 0.71776 0.07028
BlueMatrixColumn | 0.13654 0.06398 0.73845
MediaWhitePoint | 0.9642 1 0.82491
Luminance | 113.50124 119.62543 130.01166
RedTRC | (Binary data 14 bytes)
GreenTRC | (Binary data 14 bytes)
BlueTRC | (Binary data 14 bytes)
VideoCardGamma | (Binary data 1584 bytes)
ProfileDescription | CHG_09_110401

Which means that what you see when you're previewing it with your browser or even your computer is not the RAW data but the interprated one with the ICC profile. So as far as i know, you're overlaying just with the raw data. Could you provide your code?

original_image.overlayWith(await overlayImage.png().toBuffer());

One more observation: If I invert the process overlay car above the Mario, I get same issue with car colors: https://ibb.co/fKyVjT
For your reference, please find ICC profiles read by sharp:
For mario:
{ format: 'png',
  width: 1473,
  height: 1854,
  space: 'srgb',
  channels: 4,
  depth: 'uchar',
  density: 350,
  hasProfile: true,
  hasAlpha: true,
  icc:
   <Buffer 00 00 18 dc 41 50 50 4c 02 40 00 00 6d 6e 74 72 52 47 42 20 58 59 5a 20 07 db 00 04 00 04 00 0f 00 2f 00 10 61 63 73 70 4d 53 46 54 00 00 00 00 00 00 ... > }

For car:
{ format: 'png',
  width: 2000,
  height: 1312,
  space: 'srgb',
  channels: 4,
  depth: 'uchar',
  density: 72,
  hasProfile: false,
  hasAlpha: true }

The car photo does not have any ICC profile in it but its colors are not still not correct when overlaid.

Here is a working code for me:
const sharp = require("sharp");
const fs = require("fs");
async function main() {
  let car = sharp("car.png");
  let mario = sharp("mario.png");
  await car
    .overlayWith(await mario.resize(200,200).withMetadata().toBuffer())
    .toFile("result.png");
}
main();

result

In fact the gamma correction is applied with the ICC profile, so in your case if you import metadatas the correction is applied too.
Note: you're not forced to put .png() if it's already a png file.

I already import metadata but I don't write it in final output. I also checked that writing metadata in final output makes no difference. Please also note that not all images have input ICC profile and not doing gamma correction actually makes sharp producing wrong colors.
A detailed explanation is available here: http://www.ericbrasseur.org/gamma.html
I have listed down everything here so anyone who experiences this in future can take informed decision.

Sorry but your overlay is not importing metadata here:
original_image.overlayWith(await overlayImage.resize(800,800).max().png().toBuffer())

I was meaning importing the metadata of the overlay this way:
original_image.overlayWith(await overlayImage.resize(800,800).max().png().withMetadata().toBuffer())

I meant I tried the code with .withMetadata().
May be @lovell or @jcupitt can look into this? I feel this issue itself is a testimony to the problem. I think I will leave it open until some library developer looks into this.

vips_thumbnail has an option for linear-light resize:
http://jcupitt.github.io/libvips/API/current/libvips-resample.html#vips-thumbnail
I don't know if sharp exposes this. It'll give much better results than gamma.
The downside is that it's extremely expensive, up to perhaps 10x slower. The biggest slowdown is because it prevents the use of jpeg shrink-on-load, unfortunately.
Summary: don't use gamma for resize, use linear instead.

I am just trying to understand this. Do you mean to say doing linear resize is "correct" way to resize rather than gamma? I am no expert in this field but this section of this article says something different:
http://www.ericbrasseur.org/gamma.html#explanation.

I was about to add this information (which I just learnt).
Sharp documentation also says that when applying gamma correction, the library does not take advantage of shrink-on-load feature. They have also written that it first converts to linear scale, performs resize and then again converts to non-linear scale. The documentation is here: http://sharp.dimens.io/en/stable/api-operation/#gamma.
So in fact when we do .gamma(2.2) sharp actually applies linear logic which makes output correct. Now this also means that sharp would not be as fast as it claims. Doing approximate resize is super fast with sharp but doing 'correct' resize, sharp can be much slower.
Points by @jcupitt are also excellent. Rather than applying 2.2 factor for all color spaces, Sharp should use different logic for different color spaces. For example, Sharp default output is sRGB colorspace. Converting back from linear to non-linear colorspace with just factor of 2.2 makes output for sRGB inaccurate.
Correct formulas for this are given here: http://www.ericbrasseur.org/gamma.html#formulas. May be there should be separate issue to apply this correctness of formula.

@jcupitt Just curious here, what formula does vips_thumbnail use to convert to linear scale when there is no ICC profile in input image?

I am also curious that if input image is in sRGB and output is also sRGB, why would a resizing done in non-linear space will be wrong and resizing done in linear space is right? This also states that resizing will be done in sRGB space if linear parameter is not specified.

To summerise main concern, when overlay image is applied on original image along with gamma correction on original image, the overlay image gets wrong colors. I am keeping this open so someone can look into it.

libvips has its own sRGB <-> XYZ which implements the correct formula. It obviously won't work for things like CMYK with no profile.
Of course it's for the sharp maintainers, but in my opinion, gamma should only be used for artistic effect. If you want to do gamma-correct resize, use the linear flag of thumbnail.
Yes, you need to linerarize sRGB images even if the output is also sRGB. Try writing the formula out and you'll see why.

Understood. Thanks for your inputs. 👍

But @jcupitt could you give me a hint why with rendering the buffer of the overlay image withMetada works and without it doesn't?
Just trying to understand

Yes sounds like a gamma issue but as i mentionned in https://github.com/lovell/sharp/issues/1311#issuecomment-408595569 seems like the withMetada() func fix this issue. That's what i'm trying to understand.

Just to be clear, withMetadata() has no effect on this. I tried with and without and result was the same.

I mentioned in next comment. I was doing .gamma(2.2) on original image. I removed that and everything is working fine.

This does not answer the question but what ever... Seems like there is a difference of environnement if you've tried the sample i pasted and you haven't the same result.

Thanks all, I've created #1317 for adding a linear option to the resize operation.


Accessing the Autofac container to resolve a named instance

I guess I'm missing something with respect to your embrace of IoC.  Normally, with a framework like Stylet I would be able to gain access to the IoC container--it's usually ambient.  But it doesn't seem to be ambient in Stylet.
Most of the time, I simply inject my dependencies (property or constructor, depending), as we should.  Sometimes, however, obtaining an instance is a more dynamic affair.  Consider the following relevant bits of code (I'm using Autofac):
Bootstrapper.cs
protected override void ConfigureIoC(ContainerBuilder builder)
{
    base.ConfigureIoC(builder);
    builder.RegisterType<ContactsModuleViewModel>().Named<IScreen>("contactsModule");
}

ShellViewModel.cs
public class ShellViewModel : Conductor<IScreen>.Collection.OneActive {
public ShellViewModel()
{
    Tabs = new BindableCollection<NavTabViewModel>
    {
        new NavTabViewModel { Group = Directories, Name = "Contacts", GlyphResourceKey = "contactsDirectoryGlyph", ModuleKey = "contactsModule" },
        ...
    };
    ...
}

public void Navigate(object sender, SelectionChangedEventArgs e)
{
    var itemSelected = (NavTabViewModel)e.AddedItems[0];
    var moduleKey = itemSelected.ModuleKey;
    if (!string.IsNullOrEmpty(moduleKey))
    {
        this.ActiveItem = Container.ResolveNamed<IScreen>(moduleKey)   //***Doesn't work, of course***
    }
}
}

The last bit there, where I'm setting this.ActiveItem, is where I'm having trouble.  How do I gain access to the container?

Personally in those situations I inject a factory into the ViewModel - depending on requirements this can be something as simple as a Func<Something>, a Func<Something, string> if you need to use a key, a factory interface which the IoC container has constructed an implementation for, or a custom-built factory that I've registered with the IoC container. Referencing the container is definitely not required.
That said, you can sometimes register the IoC container with itself (StyletIoC does this, so if you inject IContainer it will give you the IoC container instance for you to use). Or you can create a static service locator, like this.

Ah, yes, Func<Something, string> is what I was looking for.  I'm still learning where StyletIoC and Autofac intersect.  Also, I studied one of your samples further I now have a better sense of your take on factories.
I'll give this a go tomorrow and post back if I have any issues.
Thanks very much.
Cheers.

OK.  I worked it all out: I went with an abstract keyed factory.  Also, I abandoned Autofac and with Stylet’s built-in IoC.  Very cool stuff you’ve done with your IoC!


Allow cross-domain-requests

This will allow exploring Pazuzu's API via a local running Swagger UI. The change does only affect the dev and dev-clean profiles.

This change is

:+1:


Release: AWS AMI 5.0.3

Release AWS AMI version 5.0.3

Pin k3s version used in image

5.0.3?
5.0.2 was the version just released 🤔


PNG format seems broken since 0.0.8

Hello,
I'm using lwip to generate image thumbnails.
In order to test my code, I'm comparing a generated thumbnail to a standard one in PNG format.
Everything was fine since 0.0.8 on which my test raises an error.
In order to diagnose the issue, I have tried to compare the generated image with the standard one by  using ImageMagick, I've got the following error:
11:17 $ LANG=C compare old.png new.png diff.png
compare: improper image header `new.png' @ error/png.c/ReadPNGImage/3921.
11:27 $ compare --version
Version: ImageMagick 6.9.2-3 Q16 x86_64 2015-10-02 http://www.imagemagick.org

new.png is the newly created thumbnail produced by lwip 0.0.8.
This might be related to https://github.com/EyalAr/lwip/pull/165 but I am not sure whether this is an ImageMagick issue or an lwip one.

Can you open new.png in another software?

new.png can still be opened in any image viewer.
I'm just concerned that the latest changes in lwip might not comply with the PNG standard.
As said before, the issue might be an ImageMagick issue...

You are correct that #165 changes the way PNGs are encoded, but the additional code only runs if metadata is actually passed in.
If you are not passing in metadata I don't see any reason why the encoding would be different.
Another thing that changed in 0.0.8 is the version of pnglib, which could somehow be related to this.
This will require more thorough checks to see if the problem is LWIP, ImageMagick or the new version of pnglib.
I'll install ImageMagick on my machine to check.
But before I do -
Does this happen with every image?
Can you upload here an image on which it does happen?
Can you verify you are not setting any metadata?
Can you verify this does not happen with LWIP 0.0.7?
Thanks.

In the meantime, I have reinstalled lwip 0.0.7 to work around the issue and reinstalled lwip 0.0.8 back again to reproduce the failing sample.
And now, I can just not reproduce the issue. What is bugging me is that before reinstalling, the issue was reproducible.
I still have this difference between the old PNG sample and the new one but now ImageMagick accepts it without complaining.
The original image I'm using to operate my tests is available here.
Here are both thumbnails produced by lwip 0.0.7 and 0.0.8:

And here is the difference that ImageMagick is now accepting to produce:

The source code I'm using to produce thumbnails:
var storeImageAsIcon = function (buffer, format, callback) {
	lwip.open(buffer, format, function (err, image) {
		if (err) { return callback(err); }
		var width = Math.ceil(50 * image.width() / image.height());

		image.resize(width, 50, function (err, image) {
			if (err) { return callback(err); }

			image.toBuffer('png', function (err, buffer) {
				if (err) { return callback(err); }
				var writestream = gfs().createWriteStream({
					metadata: { type: 'icon' },
					chunk_size: 1024,
					content_type: 'image/png'
				});
				writestream.on('close', function (file) {
					log.info('icon', file._id, 'stored');
					return callback(null, file._id);
				});
				streamifier.createReadStream(buffer).pipe(writestream);
			});
		});
	});
};

As you can see, the only metadata I'm manipulating is for gridfs and not for lwip.

If you cannot reproduce on your side, I will just close the issue...

I'll close it for now. But please let me know if the issue comes back.
A possible explanation is that when you first upgraded from 0.0.7 to 0.0.8, somehow the old binaries were not cleaned properly and were used with the new JS.


this channel does not have a video tab - but it exists

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
https://www.youtube.com/@Dior/videos
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp  -vU --cookies-from-browser firefox https://www.youtube.com/@Dior/videos
[debug] Command-line config: ['-vU', '--cookies-from-browser', 'firefox', 'https://www.youtube.com/@Dior/videos']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (debian*)
[debug] Python 3.11.2 (CPython x86_64 64bit) - Linux-5.11.0-18-generic-x86_64-with-glibc2.36 (OpenSSL 3.0.8 7 Feb 2023, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.2-3 (setts), ffprobe 5.1.2-3, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.46.0, pyxattr-0.8.0, secretstorage-3.3.3, sqlite3-2.6.0, websockets-10.4
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/home/elias/.mozilla/firefox/jafshm4s.default-esr-2/cookies.sqlite"
[Cookies] Extracted 5114 cookies from firefox
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[youtube:tab] Extracting URL: https://www.youtube.com/@Dior/videos
[youtube:tab] @Dior/videos: Downloading webpage
[debug] [youtube:tab] Selected tab: 'featured' (home), Requested tab: 'videos'
ERROR: [youtube:tab] @Dior: This channel does not have a videos tab
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/common.py", line 694, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/youtube.py", line 4438, in wrapper
    info_dict = func(self, url, smuggled_data)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/youtube.py", line 6264, in _real_extract
    raise ExtractorError(f'This channel does not have a {original_tab_id} tab', expected=True)

This is due to a bug in youtube's backend with the new @handle URLs.
See description of issue: https://github.com/yt-dlp/yt-dlp/issues/6305#issuecomment-1437438120
Same problem is happening here: the site redirects /@Dior/videos to /user/dior/home/videos, which is not valid
workaround:
get the channel_url from one of the channel's videos
$ yt-dlp -O channel_url "https://www.youtube.com/watch?v=ftWdAeIZFIQ"
https://www.youtube.com/channel/UC6xD-jKli-_qIYXDTsNv4dw

then you can append /videos to it:
yt-dlp "https://www.youtube.com/channel/UC6xD-jKli-_qIYXDTsNv4dw/videos"


[firebase dynamic links] deeplink didn't open app on facebook on ios

when share link on facebook the link always open store even if app installed on mobile
this happened on ios.
my last issue closed without solution

@AbanoubWagih94
I think you opened same issue earlier which was closed in favor of https://github.com/FirebaseExtended/flutterfire/issues/3861. Please don't open multiple issues for same reason.
I suggest you to follow the dynamic_links documentation and check if you have properly setup the deeplink and also check if it opens as expected on other platforms like WhatsApp and share your findings in the original issue so we can track and revisit it properly.
Closing this as self-duplicate.


fix(ui): when serializing resources as json, interpret zero value timestamps as undefined

Fixes #1783
When serializing resources as JSON, this change will interpret zero value timestamps (i.e. {seconds: 0, nanos: 0}) as undefined instead of throwing a parsing error.
@rpelczar this solves the problem that we were debugging together earlier.
@hoon I think you'd be the best person to review this. 🙏

@krancour Does the zero value timestamps throws error?
When I try to write a test code against this change (and the previous code), the zero value treated as 1970-01-01T00:00:00Z, not undefined 🤔
AssertionError: expected '1970-01-01T00:00:00Z' to be undefined
 ❯ tests/api/apimachinery.spec.ts:28:27
     26|   test('zeroValue', () => {
     27|     const time = new Timestamp({seconds: BigInt(0), nanos: 0});
     28|     expect(time.toJson()).toBeUndefined();
       |                           ^
     29|   });
     30| });

@devholic I should have been more careful in my use of the phrase "zero value" here.
I think, from Go's perspective, the zero value of time.Time (or any of the wrappers around it) is epoch (1970-01-01T00:00:00Z).
There are edge cases where a JS client sees a Time of {"seconds": 0, "nanos": 0} and this cannot be parsed correctly. (See #1782)
I assume these {"seconds": 0, "nanos": 0} values are the result of something lost in translation when the Go resource types are serialized as protobufs and then deserialized as generated JS types.

@krancour I still don't get it...

There are edge cases where a JS client sees a Time of {"seconds": 0, "nanos": 0} and this cannot be parsed correctly.

If so, shouldn't we change fromBinary or fromJson instead of toJson? Since fromJson and fromBinary are the only methods that involved with parsing logics.

toJson gets called by the UI in several places where they are taking an object and trying to prepare its JSON representation for editing.
But I see your point, that this could possibly be fixed in the Go --> proto serialization or the proto --> JS serialization. That is probably preferable, in fact. Perhaps you might want to PR that alternative?

Let me clarify: Should the zero value ({"seconds": 0, "nanos": 0}) should be treated as null?
Because wrapping date formatting logic (https://github.com/krancour/kargo/blob/bef13801db1ef6e7c3d957cbf5e652fcca9e6fe1/ui/src/gen/k8s.io/apimachinery/pkg/apis/meta/v1/generated_pb.ts#L2941-L2945
) doesn't affect anything since new Date(0) is a valid date (which is 1970-01-01T00:00:00Z).  Instead, we should return null (or undefined) early when the both seconds and nanos are zero.

It does something because it fixed #1783.
If you want to see, roll back to before this fix and #1783 should be easily reproduced.


Issue: User check in

User Story: Check In
URL: https://github.com/lukeware/solid-dragons/wiki/John-User-check-in
Size: Medium
I chose this story because it provides a basic user generated feature. Being able to make a note of which trails a user has visited builds a sense of accomplishment in having something to show in their profile. Additionally the number of check ins displayed on the trail page will show the popularity of the trail. It allows me to test out caching data on the site at an early stage.

Task branch name: check-in
Brief statement of work performed:
Added trail page with check in button toggle, and user profile page with check in list. Javascript for the functions was added into main.js and a css selector for the check in list display.
The button functions as expected. Have not gotten to complete local storage features for storing check in status. The user profile will not respond to the check-in status that is supposedly stored locally.


Check ActiveRecord version instead of Rails version in Reconciler

My apologies for missing this in my previous PR.
This adds to the changes in https://github.com/magnusvk/counter_culture/pull/190 to allow for reconciliation in non-Rails projects.

As far as I can tell, the CI test failures appear to be unrelated to this change.

@bencollab the failure is:
     NoMethodError:
       undefined method `gem_version' for ActiveRecord:Module

But only when run against Rails 4.0. Seems very related.

I opened https://github.com/airblade/paper_trail/pull/988 to address this in PaperTrail.

@bencollab thanks for this. I merged this via a cherry-pick and just skipped the PaperTrail tests for Rails 4.0. Released as 1.8.1.


Do you have plans for SQL plugin?

Plugin for .sql files.

Yeah. Consulo had some plugin for SQL (https://plugins.jetbrains.com/plugin/1800-database-navigator).
But i had disabled it due perfomance problem (very long delay on start by wrong idea of sql parsing by plugin author) - and without rewrite, i can't push it to repository
Thanks

From its name i imagine... Does it parse whole database structure on IDE start? :)
Speaking about which SQL plugin is more relevant, from my point of view it's more important for IDE to be able to highlight SQL in string constants of general languages. There're many tools for syntax highlighting of plain SQL and many tools for SQL execution. Of course, it would be tempting to have SQL IDE which is able to perform refactorings not only within single query, but analyzing whole structure of tables, views and procedures, though it's so complex to make and mantain (remember number of databases with different SQL dialects) so I don't think this deserves efforts.

No. It's analog SQL plugin from IDEA (it was created before JB plugin)
Author declarate SQL languages in XML files (each tokens, etc), each file contains around 2k lines.
On start he read this files (and process it with very deep registration). It's eat time and CPU at start.
Idk how fix it for now. Need rewrite lexer&parsing for that - for now i don't known when

For example. Oracle SQL xml file is 8k lines
https://github.com/consulo-trash/consulo-sql/blob/master/resources/com/dci/intellij/dbn/language/sql/dialect/oracle/oracle_sql_parser_elements.xml
MySQL (1.6)
https://github.com/consulo-trash/consulo-sql/blob/master/resources/com/dci/intellij/dbn/language/sql/dialect/mysql/mysql_sql_parser_elements.xml

I will restore my old plugin, but idk his fate

Maybe

Started but with low priority. At start will be only db viewer, later sql, etc.
https://github.com/consulo/incubating-consulo-database


Orientation Issues

Hello and thanks for this library.
To replicate the issue i am facing to the following:
-> open app in portrait (device)
-> sign & accept & tap done
-> tap 'add my signature' again but now do that in landscape.
-> Load default signature
Now you will see that the image is not showing but if you tap Done it gets saved and it exits the view.
Also if you sign in portrait and then rotate in landscape the signature gets stretched.
If while stretched you try to sign again the signature returns to correct aspect ratio and but gets moved downwards.

+1

+1

+1

Ive just resolved this myself and add a PR https://github.com/ipraba/EPSignature/pull/29

Merged the PR. Thanks.


Deploy to Heroku not working

I am attempting to deploy this game to Heroku but it fails saying "socket.io-client#1.3.5 client/bower_components/socket.io-client
remote:        module.js:338
remote:            throw err;
remote:                  ^
remote:        Error: Cannot find module '/tmp/build_3e4006a7a369453a3cb85accf7e6bf96/node_modules/gulp/bin/gulp.js'"
Could someone help me? I've never done this before.

You need to npm install.

I needed to move the devDependencies to Dependencies and create a Procfile for the project.

Actually, this is wrong, npm scripts should be updated and npm start should be an alias for node server/server.js

Yes that is what I did


Add reservations to reservationUnitByPk

this was missing from there.
(someday I hope we refactor away of these ByPks)

I fixed the test by removing ID from the response because that'll ruin snap tests. This is needed by frontend guys so I'll merge this with my own permission 😄


Consider GitHub OAuth2 in frontend

Applies to the experimental web front-end currently on #246.
Consider implementing social login/authentication with GitHub, and some actions which this would allow (such as opening a PR with a new catalogue entry on someone's behalf, for example).
It seems like it is not possible for a client-side-only webapp to do this (see the discussion here for example), but there may be a lightweight alternative (see links below).
Resources:

https://docs.github.com/en/developers/apps/building-oauth-apps/authorizing-oauth-apps#web-application-flow
https://github.com/deepakaggarwal7/react-social-login#github-specifics
https://github.com/prose/gatekeeper

Implemented and merged with #246.


[Theme] introduce Qt setThemeName methods to handle themed icons (dark/light)

From old PR https://github.com/medInria/medInria-public/pull/1090

This PR modernizes the way medInria handles icons. It is a step for the medInria5 objective "modularity" with themes.
What is actually done:

The line QIcon::setThemeName(QStringLiteral("dark")); in main.cpp declares that by default we are using the dark theme icons in the application (ie white ones). With "light" the app would automatically open with light theme icons (ie black ones).
The resources directories with icons are sort by theme (dark/light) with an index.theme file in each. Icons that are not impacted by themes (colored ones, application logo, etc) don't move. Themed icons share the same file names, for instance dark/24x24/open.svg and light/24x24/open.svg.
The medInria.qrc (and some qrc of some plugins) is updated with the index.theme files and paths.
The Registration Undo/Redo and crossed icons are updated with modern icons with free usage (even commercial).
The themed icons are called through the Qt line QIcon::fromTheme("open") for instance, with the name of the themed icon without the extension.
The system is working in plugins too with custom icons, as done in medSegmentation.

Documentation and this one.

:m:

@Florent2305 i have an addition to my answer about missing icons. If needed QIcon::fromTheme(const QString &name, const QIcon &fallback) allows to define default icons if the one asked is not present. I'm not sure it's needed for us for now but it's worth knowing it's possible.


issue when not a space in between emojis

> encode.process(":bamboo::cry:")
:bamboo::cry:
> encode.process(":bamboo: :cry:")
🎍 😢

😢 Thanks for reporting!

@wooorm any thoughtsfor this? Currently converting all "::" to ": :" but that seems a bit hacky...