API BIND plugin not Save after change

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
[-] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[-] I have searched the existing issues and I'm convinced that mine is new.
[-] The title contains the plugin to which this issue belongs
Describe the bug
When I add record via API on Bind plugin, the Action of SAVE is not running. There is no option inside API Bind plugin to SAVE configuration so I need access interface and click on Services / Configuration / Bind / Zones / -> Save
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:

Add a record BIND via API .
Example command with curl :

curl -v -X POST -d '{"record":{"enabled":1,"domain":"<uuid-zone.local>","name":"test","type":"A","value":"<IP_ADDRESS>"}}' -H "Content-Type: application/json" -k -u "xxx":"xxx" https://router-opnsense.adress/api/bind/record/addrecord

Test on a miscellaneous local server (with access to opnsense router OK of course) with the command :

dig @router-opnsense.adress test.zone.local

-> No Answer from router to this query
3. Access to OpnSense web interface to Save configuration
So go to Services / Configuration / Bind / Zones /
-> I can see the record added via API
-> I Click on SAVE button
4. Test on a miscellaneous local server (with access to opnsense router OK of course) with the command :
dig @router-opnsense.adress test.zone.local

-> Answer OK from router to this query
So Router Opnsense Resolve after save OK
Expected behavior
Do 'SAVE' on zone after a Record added via API
OR How to save Bind configuration via command line or API after change via API
Screenshots
If applicable, add screenshots to help explain your problem.
Relevant log files
If applicable, information from log files supporting your claim.
Additional context
Add any other context about the problem here.
Environment
OPNsense 21.1-amd64
FreeBSD 12.1-RELEASE-p12-HBSD
OpenSSL 1.1.1i 8 Dec 2020
ProxMox v6.2-11
CPU Host: 8 x AMD Ryzen 5 1500X Quad-Core Processor (1 Socket)
CPU VM: Common KVM processor (2 cores)
RAM VM: 2GB

Hi,
So i come back here solve this issue.
In fact documentation shoiw how to confirm change via API (it is not easy to find this information !)
Here you can find information : https://docs.opnsense.org/development/examples/api_enable_services.html
Exemple for my case with bind API :
-> to save changes with curl command I do :
curl -v -X POST -d '{}' -H "Content-Type: application/json" -k -u "xxxx":"xxxx" https://<IP_ADDRESS>/api/bind/service/reconfigure

Resolved !


fix: Use Number.doubleValue() instead of float(Number) for conversions

When testing VRP, it was discovered that occasionally, the location, represented as a pair of floats, was converted incorrectly.
It appears JPype's float conversion for Number is inconsistent, so we use Number.doubleValue() instead, which seems to consistency convert correctly.
Ideally, test_float.test_use_64_bits or the other float tests would have catch it, but somehow, those tests never failed; the exact condition for incorrect conversion is unknown.

Tried float(value.doubleValue()), but that also have the conversion error, which means we cannot really fix the issue without fixing it upstream in JPype.

Tried float(value.doubleValue()), but that also have the conversion error, which means we cannot really fix the issue without fixing it upstream in JPype.

So is this PR a fix or not?
And if not, please link the upstream issue.

Tried float(value.doubleValue()), but that also have the conversion error, which means we cannot really fix the issue without fixing it upstream in JPype.

So is this PR a fix or not? And if not, please link the upstream issue.

The issue still reproduces, but is much more rare with this PR (issue is, type(JDouble) != type(float), hence the test failures)
Exact location of the bug (where the value suddenly change to an unexpected value) is unknown.
Need further investigation.

Found an alternative solution: not using a distance map avoids the issue (w/o a performance cost)


Increase HH retry interval to 10 seconds

If a node goes down for any length of time -- failure or upgrade for example, the co-ordinator node continues HH. At an interval of 1 second for every target shard, ephemeral ports are quickly consumed which could result in eventual starvation and a network outage.

@jwilder @pauldix

:+1:

instead of doing this can you update the logic so that the wait interval goes up to a cap?

start at 1s, then go up by a few seconds until you hit like 1m wait

I took a look at this code tonight, it's not as easy after all to get the desired result. Check https://github.com/influxdb/influxdb/pull/4284 for more details.


Resubmitted with

macos/python3.8 for 1000x - https://github.com/dbt-labs/dbt-core/actions/runs/5742336046
windows/python3.8 1000x - https://github.com/dbt-labs/dbt-core/actions/runs/5742352781

Ran mac, linux and windows on python 3.8 for 1000 iterations with zero failures.  Also tried running the entire file 500 times on macos.  Still no failures.  Closing as not flaky.


[DOCFIX]modify the operation-command.yml in docs/_data/table/en/operation-command.yml

Update
"Copy the specified file specified by "source path" to the path specified by "remote path". This command will fail if "remote path" already exists."
to
"Copy the file specified by "source path" to the path specified by "remote path". This command will fail if "remote path" already exists.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17706/
Test PASSed.

Merged build finished. Test PASSed.


Bump LLVM/MHLO

The goals of the LLVM rotation were to

Keep the LLVM dependency up to date
Sync on a commit that matches other projects

The torch-mlir community has a weekly cadence.
Unfortunately LLVM didn't get updated for about a month. When it did get updated it was on one of the weeks that we don't uplift (this week).
I've done this second bump for this week so we can have a matching LLVM commit between both torch-mlir and onnx-mlir for the first time in over a month.
Specifically matching this parallel commit in torch-mlir https://github.com/llvm/torch-mlir/pull/2454

Jenkins Linux amd64 Build #12670 [push] Bump MLIR/MHLO (#2504)  ... started at 12:26

Jenkins Linux s390x Build #12690 [push] Bump MLIR/MHLO (#2504)  ... started at 13:26

Jenkins Linux ppc64le Build #11683 [push] Bump MLIR/MHLO (#2504)  ... started at 13:35

Jenkins Linux amd64 Build #12670 [push] Bump MLIR/MHLO (#2504)  ... failed after 1 hr 41 min

Jenkins Linux ppc64le Build #11683 [push] Bump MLIR/MHLO (#2504)  ... passed after 3 hr 58 min

Jenkins Linux s390x Build #12690 [push] Bump MLIR/MHLO (#2504)  ... aborted after 4 hr 13 min


Move checkAndUpdateWireless into VerifyPending and also handle override

Signed-off-by: Naiming Shen<EMAIL_ADDRESS>
moved the checkAndUpdateWireless() into VerifyPending()
leave the 'override' type of update also at the doUpdatePortConfigListAndPublish() to catch the case when the wifi override might not get into later verify stage
change the hack of ignore zedcloud wifi null configure by copying the override wifi config into other types of update, thus now the lastresort config will have the wificonfig set by override
tested on my device with baseimage upgrade, reboot, reboot with only wifi available

This looks reasonable @naiming-zededa -- but I guess we still need to address the issue of timing between this portion of nim and other device-steps sections (this is only relevant when time is completely out of whack on boot).
I suppose we can do it in a separate PR, but it would be great to address it (unless I somehow missed how this PR addresses it).

@eriknordmark addressed the comment on proper comments when controller supports this. Will do another testing/debug/verify later when controller is ready. There might need to be some change by then.


Fixing locale issues

Docs Build status updates of commit 654bfff:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

BingMaps/articles/bing-maps-on-azure-marketplace.md
:white_check_mark:Succeeded
View

BingMaps/getting-started/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


fix(es/decorator): Add support for private access expressions in legacy decorators

Related issue :

Closes: #9429

I have no clue about this performance regression.


fix error valid target in image-layout

In layout.md, The validated target should be image layout itself, not only image index.
I'd like to extract and land this firstly, while #452 is in WIP.
Signed-off-by: xiekeyang<EMAIL_ADDRESS>

@opencontainers/image-spec-maintainers This might be ready to merged, PTAL.

I'm not opposed, though oci-layout doesn't have a mediaType yet. Perhaps we need this for validating them and this example as well.

oh we do! I was searching for "oci-layout" and missed ./schema/image-layout-shema.json. I'll add more to that description.
LGTM

lgtm, will let @stevvooe ack/nack tho

LGTM

@xiekeyang Follow this up with a note somewhere that oci-layout is not a mediatype.

@stevvooe got it. i would present another patch for this.


Work shift show

This is where we will link to in reminder emails
Use show action instead of edit
Include job info
Include 'Cancel Signup' top button, redirects to index, confirm dialog

Originally created by Tom Smyth at 2018-01-17 15:44, migrated from redmine-#7844

Tags set to work

By Tom Smyth at 2018-01-17 20:45

Tags changed from work to work, work mvp

By Tom Smyth at 2018-01-17 20:52

Priority changed from ? to 2

By Tom Smyth at 2018-01-17 20:55

Description updated (diff)

By Tom Smyth at 2018-03-05 02:22

Subject changed from Work assignments form to Work shift form

Description updated (diff)

By Tom Smyth at 2018-03-07 14:00


Fix deleting connections and applications and customers

we no longer cascade on deletion of connections, because we want to force usage of syncService.deleteSync(id), so that we capture a task to clean up corresponding Temporal syncs.
We should implement the delete to clean the syncs up.

this is addressed for connections but not yet applications

not going to support this type of cascade delete. instead, return a better error to user to disallow deleting unless all associated connections are deleted


Update grammar, functions and commands for Arma 3 1.74 and ACE 3.10.2

CBA didn't update documentation for 3.4.0 yet, waiting for that.


replace mode cursor shape bug (Windows Terminal)

Describe the bug
When &t_SR or &t_SI is set, insert and Replace mode work as expected, but in replace, the cursor jumps visually five steps to the right (seems to correspond to the length of the string variable) while the operation is pending. After replacement the cursor returns correctly, but if the character was changed, the new character is visually followed by the string in &t_EI (without the ^[), and the cursor does not change back from the underline mode. This can be reset by triggering a redraw (^L).
I can't tell what changes in how r sends ansi escape codes compared to R, but something causes the two codes to interfere.
To Reproduce

In Windows Terminal, run vim --clean
:let &t_SR="\e[7 q", :let &t_EI="\e[2 q"
Write abcdefg
Place cursor on a
Press r, first bug: cursor moves to f
Press b, second bug: text becomes b[2 qfg, cursor is still the replace shape
Press <C-L>, text is bbcdefg

Expected behavior
The cursor should stay in the same place, the text should immediately become bbcdefg and the cursor should reset.
Screenshots

Environment (please complete the following information):

VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Dec 12 2019 13:19:27)
OS: Windows 10, 18363
Terminal: Windows Terminal

Update, using
let &t_SR="\<CSI>7\ q" and let &t_EI="\<CSI>2\ q" fixes the second bug, but the cursor still jumps (but to e now, I guess because &t_SR is one character shorter)

@gustaphe Were you able to find a fix for the cursor jumping forward? I'm currently dealing with this same issue and haven't been able to find a solution

No. In the WT convo linked above I've updated the description a little (new and interesting bugs ...), and there's a comment describing using C to solve it. One day I'll look into that, but I keep forgetting to.

(Implementation detail)
Win32 version of Vim uses its own terminal codes:
https://github.com/vim/vim/blob/ad6d9cc679259936b7c8453f17c0ec9d7ab7dcd0/src/term.c#L561-L570
and they are translated into actual escape sequences (or into Win32 console API calls):
https://github.com/vim/vim/blob/ad6d9cc679259936b7c8453f17c0ec9d7ab7dcd0/src/os_win32.c#L6454-L6462
Currently, this doesn't support t_SR, t_SI and so on. Maybe we need to pass through these kind of esc sequences.
BTW, I found that this function missing implementation for Windows Terminal:
https://github.com/vim/vim/blob/ad6d9cc679259936b7c8453f17c0ec9d7ab7dcd0/src/os_win32.c#L1523-L1528

Okay, now there's a plugin at gustaphe/winterm.vim

I'm now preparing a patch for this issue:
https://osdn.net/users/k_takata/pf/vim-ktakata-mq/scm/blobs/312c07eac6ba0528dde08cbd240fc7ea10fe3e44/win32-set-cursor-style.patch
The following .vimrc works after applying this patch.


Registry update contact

Hi,
I don't believe the contact Jeremy Adams [jeremy.adams<EMAIL_ADDRESS>in the readme.md is up-to-date. Who is the current maintainer for this service.
https://github.com/ga4gh/ga4gh-registry/blob/main/README.md?plain=1#L50
We wish to make a correction as detailed in https://github.com/dockstore/dockstore/issues/5803

FYI @jb-adams if you happen to know

Tagging @yash-puligundla or @ga4gh/ga4gh-tech-team

This issue has been picked up by the GA4GH tech team and the registry entry would be updated. More details to be shared.

Issue fixed. Entry updated.

Thanks!


Fix output logs by switching to kind-cluster context

Issue link

Jira: https://issues.redhat.com/browse/RHOAIENG-5131
What changes have been made

This was previously attempted to be fixed, however when the e2e test fails, the step to Switch to kind-cluster context is skipped due to previous step failure.

With this change, the step to switch context will always be made, enabling to print component logs.

As seen/tested here: https://github.com/project-codeflare/codeflare-sdk/actions/runs/8560772513/job/23460511229?pr=495
Verification steps

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

/lgtm -- I've made the same fix in my various branches.

@dgrove-oss awesome, thanks for reviewing!


java.lang.OutOfMemoryError when using parameters in query

I'm trying to import a table with about 50 million rows into elasticsearch with the jdbc river. At first I did a full import which worked like a charm:
{
    "sql": [{
        "statement": "select * from mytable"
    }]
}

As I need the ability to do incremental imports I started to experiment with the parameter feature and ran into an out of memory exception. The query below returns about 2.2 million rows and results in the exception. When I change the parameter to 2014-09-01 (360.000 rows) it works fine. I changed the fetchsize but this seemed to have no effect.
I tested on ES 1.3.2 with <IP_ADDRESS> and ES 1.2.4 with <IP_ADDRESS>. Database is a MySQL 5.5.35, mysql-java-connector is 5.1.28.
{
    "sql": [{
        "statement": "select * from mytable where datefield > ?",
        "parameter": ["2014-08-01"]
    }]
}

java.lang.OutOfMemoryError: Java heap space
java.util.concurrent.ExecutionException: java.lang.OutOfMemoryError: Java heap space
	at java.util.concurrent.FutureTask.report(FutureTask.java:122)
	at java.util.concurrent.FutureTask.get(FutureTask.java:188)
	at org.xbib.pipeline.simple.SimplePipelineExecutor.waitFor(SimplePipelineExecutor.java:118)
	at org.xbib.elasticsearch.plugin.feeder.AbstractFeeder.run(AbstractFeeder.java:241)
	at java.lang.Thread.run(Thread.java:744)
Caused by: java.lang.OutOfMemoryError: Java heap space
	at com.mysql.jdbc.Buffer.getBytes(Buffer.java:207)
	at com.mysql.jdbc.Buffer.readLenByteArray(Buffer.java:339)
	at com.mysql.jdbc.MysqlIO.nextRow(MysqlIO.java:2082)
	at com.mysql.jdbc.MysqlIO.readSingleRowSet(MysqlIO.java:3554)
	at com.mysql.jdbc.MysqlIO.getResultSet(MysqlIO.java:491)
	at com.mysql.jdbc.MysqlIO.readResultsForQueryOrUpdate(MysqlIO.java:3245)
	at com.mysql.jdbc.MysqlIO.readAllResults(MysqlIO.java:2413)
	at com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2836)
	at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2825)
	at com.mysql.jdbc.PreparedStatement.executeInternal(PreparedStatement.java:2156)
	at com.mysql.jdbc.PreparedStatement.executeQuery(PreparedStatement.java:2313)
	at org.xbib.elasticsearch.river.jdbc.strategy.simple.SimpleRiverSource.executeQuery(SimpleRiverSource.java:572)
	at org.xbib.elasticsearch.river.jdbc.strategy.simple.SimpleRiverSource.executeWithParameter(SimpleRiverSource.java:355)
	at org.xbib.elasticsearch.river.jdbc.strategy.simple.SimpleRiverSource.fetch(SimpleRiverSource.java:284)
	at org.xbib.elasticsearch.plugin.feeder.jdbc.JDBCFeeder.fetch(JDBCFeeder.java:335)
	at org.xbib.elasticsearch.plugin.feeder.jdbc.JDBCFeeder.executeTask(JDBCFeeder.java:179)
	at org.xbib.elasticsearch.plugin.feeder.AbstractFeeder.newRequest(AbstractFeeder.java:362)
	at org.xbib.elasticsearch.plugin.feeder.AbstractFeeder.newRequest(AbstractFeeder.java:53)
	at org.xbib.pipeline.AbstractPipeline.call(AbstractPipeline.java:87)
	at org.xbib.pipeline.AbstractPipeline.call(AbstractPipeline.java:14)
	at java.util.concurrent.FutureTask.run(FutureTask.java:262)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
	... 1 more

Sorry for reply a closed issue. Hope this help some people.
I ran into this issue. Finally found that the only solution to me is to optimize the sql statement, (which was too simple to be thought about.)
Add some proper db index to make sure the sql statement can be finish as soon as possible.
select * from mytable was fast, the major overhead is data transferring, and since we are doing a streaming import, data transferring is not a big deal. Through select * from mytable where datefield > ? should take a long time & lots of memory when you got millions of data without proper indexes.


[MNT] 0.12.1 has not been released on conda-forge

0.12.1 has not been released on conda-forge, even though this should have happened by automation.
@lmmentel was investigating, this is to track the issue.
I also think it is a must-have to solve before 0.13.0 mid-July, or the debt will accumulate - especially since we want to move to 3.10 support.
issue on sktime-feedstock: https://github.com/conda-forge/sktime-feedstock/issues/44

Currently the build on linux py39 fails with
The reported errors are:
- Encountered problems while solving:
-   - package sktime-0.12.1-py39h5a03fae_1 requires python >=3.9,<3.10.0a0, but none of the providers can be installed

more logs
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=530943&vi[…]413-330699242c05&t=c10fa5f2-fdf6-5338-3bdb-c4bea7c23412&l=3135

py38 says something different:
package sktime-all-extras-0.12.1-ha770c72_1 requires numba >=0.53, but none of the providers can be installed

actually, py39 says the same

I am trying to debug, so making changes to the PR

ah, makes sense.

more logs
failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform linux-64: {"hcrystalball[version='>=0.1.9']", "prophet[version='>=1.1']", "scikit-posthocs[version='>=0.6.5']", "tbats[version='>=1.1.0']"}

I found a config that works for sktime but there are still conflicts for sktime-all-extras

Adding prophet to a successful CI run, works for

linux py37
but fails for
linux py38
linux py39

with:

The reported errors are:
- Encountered problems while solving:
-   - package sktime-0.12.1-py39h5a03fae_1 requires numba >=0.53, but none of the providers can be installed

It seems that prophet is the problem... I think we can merge and release without prophet which then can be installed through pip, Appreciate some comments here @fkiraly @freddyaboulton @dhirschfeld
An alternative would be to debug what goes wrong in deps resolution with prophet but conda-forge build is not very straightforward and was wasn't able to reproduce the build process locally yet and I'm not sure if it's worth investing time into this.

I think we can merge and release without prophet which then can be installed through pip

I think that's fine & probably the best you can do for now. I'd say merge with this workaround and the prophet install can be debugged in a separate PR so as to not hold up this release.

Thanks @dhirschfeld. I managed to further restrict prophet install by platform and python version so prophet will be a dependency for sktime-all-extras for all platforms except linux with py38 or py39. I agree, we should push the release out and debug separately.

Here's a short summary and current status of the issues affecting conda release:

conda bot not creating a PR after the release on PyPI - AFAIK it is unresolved and undiagnosed, I looked into it but didn't get far
source url changing in the meta.yaml and now requiring and extra . after the v of the version - updated in the PR
prophet >=1.1  creating dependency resolution error on linux py38 and py39 - worked around by skipping this conda dep in the CI for the specific platforms
unnecessary deps specified in host section - cleaned up in the PR

0.12.1 is now released (manually), but we haven´t yet understood what is preventing automatic releases - renaming issue and downgrading importance from "must" to "should" for release board

@lmmentel made an observation, possibly the cause?

It seems the link to the github release assets changed. The assets are now under https://github.com/alan-turing-institute/sktime/archive/refs/tags/v0.13.1.tar.gz url so the template in the meta.yml would need to be updated.

Hello! Is there any progress on this issue or plans for a manual release for version 0.14.0? Especially since pandas 1.5 support is introduced in 0.13.3 it would be great to have a more up-to-date version installable through conda.

There's been an effort to fix conda release (see here) but it's quite hard to put together a recipe with all dependencies taking into account platform support and the fact that some of our soft deps don't bother with conda release or their conda release lag far behind their pypi releases.
Is there anything you cannot do though pip/pypi that would be enabled with conda @christopherbunn?

could we simply ignore the soft dependencies, @lmmentel?
Just release with the core dependencies?
Since the time the problem originally occurred, the soft dependencies should be fully isolated, i.e., the test suite will run with the core dependencies only.

could we simply ignore the soft dependencies

+:100:
IMHO it's very low priority to release a package with the soft-dependencies and it really shouldn't hold up releasing the core package. If it works, it's a nice convenience for users but if not, users can easily install the extra deps they need via conda or pip

well, let's do that then. The soft dependencies should be fully isolated as soon as this PR passes:
https://github.com/sktime/sktime/pull/3760

Hi! Now that we have the soft dependencies separated out, is there anything else blocking a conda release? If not, I'm happy to give updating the feedstock a shot if no one else is working on that already.

I don't see why not @christopherbunn. Go ahead and fee free to reach out if there's anything I can help with.


Revert "Merge pull request #3 from Psemp/staging"

This reverts commit 24a8c5c1d0409592fc4a0bb858a28f2a75fdf960, reversing
changes made to f49d2bc418ef2cbceaf3d6ca0c51503b7d1e7717.

missclicked somewhere, forkgive me


Add badges for Buddy

:clipboard: Description
Build status badges for Buddy
just like the others:
with optional logo after https://github.com/simple-icons/simple-icons/issues/1343

:link: Data
https://buddy.works/api/reference/getting-started/overview
Need to create an application at https://app.buddy.works/my-apps

:microphone: Motivation
There are already badges for other CI services, and while Buddy provides a badge itself, it cannot be customized beyond the default:

If anyone is interested in working on this, we've got a great Tutorial on how to get started!


error with cascade_rcnn_mdconv_c3-c5_x101_64x4d_fpn_1x_0.py

Thanks for your bug report and we appreciate that a lot.
Checklist

[ ] I have searched related issues but could not get the expected help.
[ ] The bug has not been fixed in the latest version.

Describe the bug
RuntimeError: While copying the parameter named layer2.0.conv2.weight, whose dimensions in the model are torch.Size([512, 512, 3, 3]) and whose dimensions in the checkpoint are torch.Size([512, 8, 3, 3]).
Error traceback

What command or script did you run?
/home/mmdetection/configs/cascade_rcnn_mdconv_c3-c5_x101_64x4d_fpn_1x_0.py
--validate
--gpus
8
--launcher
pytorch
Environment

OS: [e.g., Ubuntu 16.04.6]
GCC [e.g., 5.4.0]
PyTorch version [e.g., 1.1.0]
How you installed PyTorch [ conda]
GPU model [e.g., TitanXp]
CUDA 10.0

The command seems not complete. Could you provide the running command and config file (if you modified it)?

The full completed command is
python /home/mmdetection/tools/train.py /home/mmdetection/configs/cascade_rcnn_mdconv_c3-c5_x101_64x4d_fpn_1x_0.py
--validate
--gpus
8
--launcher
pytorch
And I got the following error
RuntimeError: While copying the parameter named layer2.0.conv2.weight, whose dimensions in the model are torch.Size([512, 512, 3, 3]) and whose dimensions in the checkpoint are torch.Size([512, 8, 3, 3]).
The base net is resnext101 which is downloaded from
'resnext101_64x4d': 'https://s3.ap-northeast-2.amazonaws.com/open-mmlab/pretrain/third_party/resnext101_64x4d-ee2c6f71.pth',  # noqa: E501

I cannot even run with the provided command above (other errors were raised before loading checkpoints), are you sure about that?
BTW, have you checked the second item in the checklist?


Bind variables do not passed from alias to script

I have faced an issue when tried to create an alias to save typing working with scripts. I looks like the bind variables are not passed to script if they are defined in alias.
It's very simple to reproduce with the args.js as example.
Check how it should work:
DEV>script args one two
0:args
1:one
2:two

create an alias:
DEV>alias test_args=script args :p1 :p2;
DEV>alias list test_args
test_args
---------
script args :p1 :p2

and run the script though the alias:
DEV>test_args one two

Command=test_args
0:args
1::p1
2::p2

No need to pass binds as the APIs have full access to everything.  Try this:
binds = ctx.getVarMap();
numVar = binds.get('NUM_VAR');
value = numVar.getValue();
ctx.write("num_var = " + value + "\n");

Great, thank you Kris!
Can I get API description of ctx and sqlcl globals somewhere, please?
Regards,
Valentine

It's coming just slowly.  There's a lot to cover so trying to come up with examples to match adding doc so it's clearer.

Thanks, examples are fine, will continue read your blog and the Oracle_DB_Tools repository.
Thank you for really beautiful tool!

You saw this I hope? https://github.com/oracle/Oracle_DB_Tools/blob/master/sqlcl/SCRIPTING.md
I'll be adding more and more to that.


Apply correct mime type for abstracts in DOI responses

The text of Work abstracts received from Crossref API may be marked up with JATS XML. We need to detect that and apply appropriate type.
At this moment, when we convert Crossref DOI Work to Relaton bibliographic item, we only fill in the content property of the abstract: https://github.com/ietf-ribose/bibxml-service/blob/c9e86a881df6680f9025fd621a4c3a96a2350cd3/doi/crossref.py#L116-L118
However, each abstract also supports format property (see BibliographicItem.abstract and its class inheritance, which aims to comply with Relaton model).

[ ] Confirm what’s the appropriate MIME type string. (Could be application/x-jats+xml, research whether this is correct.)
[ ] Write a function that takes any string and attempts to detect MIME type based on its contents. It must be capable of detecting at least text/plain, text/html, and whatever is the right MIME type for JATS XML.
[ ] Update handling of abstracts in Crossref DOI Work -> Relaton bibliographic item conversion code (doi.crossref module), use the above new function to provide correct format value on the abstract.

Once we do this, relaton-py can then handle paragraph extraction according to string format (https://github.com/relaton/relaton-py/issues/12).
Originally posted by @ronaldtse in https://github.com/ietf-ribose/bibxml-service/issues/208#issuecomment-1136917042

Upstream https://github.com/relaton/relaton-py/issues/12

Regarding MIME type:

https://documentation.cochrane.org/display/API/Review+Document+API#ReviewDocumentAPI-Encodingofresponse uses application/x-jats+xml, but it’s very difficult to find any other supporting examples for JATS (it doesn’t seem to be that popular)

@ronaldtse @opoudjis Any objections to assigning the above string as format (MIME type) for DOI abstracts that contain JATS XML?

Regarding detection:

According to Crossref uploader’s docs, they support JATS and MathML (https://www.crossref.org/documentation/schema-library/metadata-deposit-schema-5-3-1/).

So I suggest we rule out HTML in abstract contents, and just force apply JATS XML type to all abstracts obtained from DOI.

I am confused by the comment -- I don't think all DOI abstracts are only either JATS or MathML. At least plain text is supported.
The MIME type for JATS XML is accurate and fine (no one can read it anyway).

Have you checked https://data.crossref.org/reports/help/schema_doc/5.3.1/index.html ?
No one in the right mind uses JATS, especially for abstracts...

@ronaldtse

I am confused by the comment -- I don't think all DOI abstracts are only either JATS or MathML. At least plain text is supported.
The MIME type for JATS XML is accurate and fine (no one can read it anyway).

A JATS XML string that happens to not contain any tags is indistinguishable from plain text🤔
Frankly, it’s hard to test what they have because most entries don’t have any abstract at all. The only abstract I have seen yet is JATS🤷♂

Have you checked https://data.crossref.org/reports/help/schema_doc/5.3.1/index.html ?

Following your link I found abstract field only defined using JATS. All references to abstract point here: https://data.crossref.org/reports/help/schema_doc/5.3.1/JATS-journalpublishing1-3d2-mathml3-elements_xsd.html#abstract

For now it’s been changed to JATS for all abstracts from Crossref.

bibxml serializer in relaton-py now extracts text from jats:p tags as a list of paragraphs converted into <t> (https://github.com/relaton/relaton-py/issues/12#issuecomment-1140347166).

Example: https://dev.bibxml.org/public/rfc/bibxml7/reference.DOI.10.3390/fi11030055.xml


[FirebaseAuth]: Can't listen to Facebook login

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
Web
Description
I can listen to Google sign in + custom Email & password sign in. But, Facebook doesn't call the authStateChanges listener
FirebaseAuth.instance.authStateChanges().listen((User? user)
I can see that the proper User data is returned in the API responses though.
Future<UserCredential> signInWithFacebook() async {
    isFaceBookAuth = true;
    FacebookAuthProvider facebookProvider = FacebookAuthProvider();
    facebookProvider.addScope('email');
    facebookProvider.setCustomParameters({
      'display': 'popup',
    });
    return await FirebaseAuth.instance.signInWithPopup(facebookProvider);
  }

Reproducing the issue
void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  await Firebase.initializeApp(options: DefaultFirebaseOptions.currentPlatform);
runApp(MyApp());
}
------------------------------------------------------

class MyApp extends StatelessWidget {
  const MyApp({super.key});

void listenToAuthStates() {
    listener = FirebaseAuth.instance.authStateChanges().listen((User? user) {
      if (user == null) {
        print('User is currently signed out!');
      } else {
        print('User is signed in!' + "uid= " + user.uid);
        }
      }
    });
  }

  Future<UserCredential> signInWithFacebook() async {
    FacebookAuthProvider facebookProvider = FacebookAuthProvider();
    facebookProvider.addScope('public_profile');
    facebookProvider.setCustomParameters({
      'display': 'popup',
    });
    return await FirebaseAuth.instance.signInWithPopup(facebookProvider);
  }

  @override
  Widget build(BuildContext context) {
return Container(child:
Column(children;[
TextButton(
            onPressed: () {
             listenToAuthStates();
            },
            child: AppText(text: "Start auth listener"),
          ),
Sizedbox(height:25),
TextButton(
            onPressed: () {
              signInWithFacebook();
            },
            child: AppText(text: "Facebook auth"),
          ),
]),
);
}

Firebase Core version
3.1.1
Firebase Auth version
4.17.7
Flutter Version
3.22.2
Relevant Log Output
No response
Flutter dependencies

Expand Flutter dependencies snippet

  firebase_auth: ^4.17.7
  firebase_core: ^2.26.0

Additional context and comments
No response

@TarekkMA I modified the code example in my main question above. If u need extra info, please inform me.
Concerning the Facebook library, I only followed the instructions on Firebase on how to add Facebook auth:
https://firebase.google.com/docs/auth/flutter/federated-auth
I did not specify a Facebook library.

You will need to use a library for Facebook login. For example, in the docs, they use this library: flutter_facebook_auth. You can check this library's documentation for more details on how to use it with firebase.

@TarekkMA But, this is in the section concerning iOS+ & Android.
The web section says: "On the web, the Firebase SDK provides support for automatically handling the authentication flow using the Facebook application details provided on the Firebase console."

Oh, thank you for the clarification.
I've replicated the code you have written and I've got the authStateChanges() stream to emit the user change and I didn't encounter any issues. Please check this branch and try to run the example auth app to see if you still have the same issue:
https://github.com/firebase/flutterfire/tree/issue/13014

@TarekkMA I solved my problem by deleting the Facebook user from the Firebase auth portal. After that, everything worked fine.
I could be mistaken because I do not understand the logic of the solution, but now I have no problems.


cargo generate --git<EMAIL_ADDRESS>Git Error: failed to connect to github.com

Why this command is error? My git clone is ok on my ubuntu 2004.
ubuntu@u2004h:~/docker$ cargo generate --git git@github.com:rustwasm/wasm-pack-template.git
🤷   Project Name : wasm-game-of-life
🔧   Creating project called `wasm-game-of-life`...
Error: ⛔   Git Error: failed to connect to github.com: Connection timed out; class=Os (2)

ubuntu@u2004h:~/docker$ git clone git@github.com:rustwasm/wasm-pack-template.git metto
Cloning into 'metto'...
remote: Enumerating objects: 193, done.
remote: Counting objects: 100% (4/4), done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 193 (delta 0), reused 0 (delta 0), pack-reused 189
Receiving objects: 100% (193/193), 40.21 KiB | 3.09 MiB/s, done.
Resolving deltas: 100% (90/90), done.

And my ./.cargo/config have net.git-fetch-with-cli =true settings.
[net]
git-fetch-with-cli = true

My ubuntu 2004 is created by multipass on HOST windows 10。

You're using a ssh url, those are at the moment not supported.
Please use the https url of the repo then it should work.

Thanks @sassman . I use https://rustwasm.github.io/docs/book/game-of-life/hello-world.html command cargo generate --git https://github.com/rustwasm/wasm-pack-template, that is not work properly.

ubuntu@u2004h:~$ cargo generate --git https://github.com/rustwasm/wasm-pack-template
🤷   Project Name : wasm-game-of-life
🔧   Creating project called `wasm-game-of-life`...
Error: ⛔   Git Error: failed to connect to github.com: Connection timed out; class=Os (2)

Before this operation, I execute the command git config --global --unset<EMAIL_ADDRESS>

Could you please list all your git settings?
And can you ping github.com?
Can you do a git clone of this repo?

I can git clone,but I cannot ping github. com.

Ok, can you curl https://github.com?

When I use ssh insteadof parameter, I can git clone; I cannot https://github.com
ubuntu@u2004h:~$ curl https://github.com
curl: (28) Failed to connect to github.com port 443: Connection timed out
ubuntu@u2004h:~$ git clone https://github.com/cargo-generate/cargo-generate.git metooo
Cloning into 'metooo'...
fatal: unable to access 'https://github.com/cargo-generate/cargo-generate.git/': Failed to connect to github.com port 443: Connection timed out
ubuntu@u2004h:~$ git config --global<EMAIL_ADDRESS>https://github.com/
ubuntu@u2004h:~$ git clone https://github.com/cargo-generate/cargo-generate.git metooo
Cloning into 'metooo'...
remote: Enumerating objects: 1942, done.
remote: Counting objects: 100% (154/154), done.
remote: Compressing objects: 100% (93/93), done.
remote: Total 1942 (delta 99), reused 89 (delta 60), pack-reused 1788
Receiving objects: 100% (1942/1942), 721.06 KiB | 798.00 KiB/s, done.
Resolving deltas: 100% (1151/1151), done.
ubuntu@u2004h:~$ curl https://github.com
curl: (28) Failed to connect to github.com port 443: Connection timed out

My OS is ubuntu 2004 made using multipass on window10,  HOST PC have Lantern as VPN.

I'm very sorry but the issue must be related to your networking (maybe a http proxy server) setup.
It's not cargo-generate specific.
For the ssh urls there is a tracking issue #325
I recommend checking if your firewall or host os networking works expected.


Close #13577

I'm not sure if this is the only place

Hello @250846434
Thanks for the PR.
Can you please change the commit name from "Close" to the sentence that explains the main thought comment.

Hello @250846434
Thanks for the PR.
Can you please change the commit name from "Close" to the sentence that explains the main thought comment.

@LonwoLonwo sorry,Weekend off. I just saw that


CAS delegate Issue

@leleuj
I am using CAS 5.0.4 . I want to delegate my request to Oauth 2 Server for Autentication. i checked CasOauthWrapperClient but is not working for me. I changed my CAS 5.1.1 then i able to configure GenericOauth20Client with below properties
cas.authn.pac4j.oauth2[0].id=[ID]
cas.authn.pac4j.oauth2[0].secret=[SECRET]
cas.authn.pac4j.oauth2[0].authUrl=
cas.authn.pac4j.oauth2[0].tokenUrl=[AUTH URL]
cas.authn.pac4j.oauth2[0].profileUrl=[PROFILE URL]
cas.authn.pac4j.oauth2[0].profilePath=[NOT CLEAR WHY USING]
cas.authn.pac4j.oauth2[0].profileVerb=GET
Now after configuring above Properties in CAS 5.1.1 i am able to one link GenericOauth20 in 'OR login with' block.
But i am not able to understand its creating wrong redirect_url or i am doing something wrong.
Example:
My CAS Server(5.1.1) is running on 7000 which is acting as client and my another cas server(5.0.4) is running on 8443 from where i want to authenticate. Please check below image.

Please suggest way how i can move ahead using CAS 5.0.4 or CAS 5.1.1.
Thanks in Advance.

Plase use the CAS or pac4j-users mailing lists.


Invalid TOTP code. Try again?

Describe the bug
Invalid TOTP code. Try again?
Expected behavior
2 FA Should Work on Entering Correct Code
To Reproduce

Create New Account
Try To Enable 2FA
Every time i am getting  "Invalid TOTP code. Try again?"

System Details
* Browser : Chrome
* OS: Ventura 13.4
*  Platform: MAC M1 pro
Additional context

My Mobile Timing is also in sync.
Happening For    both test account(test.pypi.org) as well as main account(* pypi.org)
*Tried Below Apps for Generating Authentication Codes

Google Authenticator
Microsoft Authenticator
Duo

Attachments:

Closing this in favor of https://github.com/pypi/warehouse/issues/14710, which is a newer issue but where most of the discussion about this has been happening.


Relay: OnIncomingMessage not returning a msg body

consumer.OnIncomingMessage = func(consumer *signalwire.Consumer, msgObj *signalwire.MsgObj) {
   msgObj.?????????
}

I'm struggling to figure out how to correctly parse out the message body from the incoming message. How do I access the body?

thanks for reporting. it was an oversight. please see: https://github.com/signalwire/signalwire-golang/pull/85

merged #85 , closing this, please  reopen if needed or open other issues if you notice other wrong things.


Support toggling provisioning checks

cf #183
Testing performed:
Added / adapted tests

@ekarlso I edited your comment to remove the word 'fixes' as otherwise merging this PR would close the issue prematurely.

There is still failing tests

Dear IONOS Team,
I believe this is not the right way to address all the issues around cloud-init and Talos. And the way it will be addressed also involves flatcar support in some way.
Talos image can get qemu-agent via custom image, so it needn't to check it. But if there is any other issue regarding the agent, it can be addressed.
The problem involving cloud-init and Talos is related to cloud-init datasource. Up to date, any os running in a vm must expect network-config to be in cloud-init netplan format and must install cloud-init binary.
On the other hand, Talos expects network-config to be in cloud-init nocloud format and has no cloud-init binary installed.
Flatcar has another config file format, in json if I'm not wrong.
For all this reasons, I think all these issues should be addressed more schematically, maybe using a schema like this in proxmox_machine_type.go:
spec:
config:
engine: cloud-init|flatcar
flatcar: {}
cloudInit:
dataSource: netplan|nocloud
qemuAgent: true|false
With this new arrangement in place, it should be possible to address all the issues around that involve both reconciliation activities and engine config format in general.
I have started forking your project in order to be able to create a Pull Request. If you are interested in my idea, I will try to arrange the code. But be aware, I'm less than a newbie at GoLang. It will need a review.
I hope you will change your mind.

Hey your Proposal is nice
but we need to keep things in mind,
in debian-based os, we check cloud-init status via the qemuAgent.
so we need a way of skipping such a check.
in flatcar we can run qemuAgent but not cloud-init.
spec:
  config:
    engine: cloud-init|flatcar
    flatcar: {}
    cloudInit:
      dataSource: netplan|nocloud
  qemuAgent: true|false

here is some of my notes:

Having qemuAgent field is great
config.engine is also nice. however we can also check that based on generated bootstrap data.
config.cloudInit.dataSource I don't know why you need such a field, please explain.

However, can you also explain how we can actually implement the cloud-init checks and Qemu and under which conditions?
Finally, a word of advice, please create a new issue and start a discussion around the API additions or changes in the issue, before starting the PR,
you need the API approved, so you don't change it when you start the PR.
we would appreciate your contributions.

@ekarlso
Is that the case?
I thought the network-config v2 works well in your setup.
@ricottatosta Open new issue, and let's discuss there.


Update README.md

update deprecated product taxonomy slug
Purpose

...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

@yorek Can you merge this?


Update the pypi package info

pypi package update for release 0.9.4
build: python setup.py bdist_wheel

@rogerxfeng8  Hi Roger, I force-pushed to clean up main branch commit history, could you update yours for this PR? Thanks!


Use GEOS-FP Backgrounds

@rtodling wants to be able to tap into GEOS-FP directories for cycling 3DVar experiments. So one can request other experiments, depending on the background availability. The updated task allows that and it is used in the flow.cylc file as well. We can always revert it back to R2D2 at any point.
I tested with 3dvar_atmos and 3dfgat_atmos suites and there are some differences in the increment plots. Please take a look before we merge this. See q_500 increment below:

GEOS-FP!  Now you made me puzzled ... first let me ask you, where did the background for the picture above came from?
Now, assuming all that is ok - I would say we should not worry too much about Q: if you recall, I have showed results about Q and I have pointed to all that the issue is related to a buggy interpolation of the background fields when saber takes cubed fields into gsibec. This is associated w/ the OOPS interpolations and their inconsistency w/ ATLAS interpolations - that's how I interpret the issue.
But the increments of T and U/V and O3 should look decent ...
To me the key question is: where did the background come from?

Just FYI, this is a draft, I will make a few more changes.

This essentially provides an option for directly accessing GEOS X experiments background located at:
/gpfsm/dnb05/projects/p139/dao_it/archive/


Tune renovate config

Might be worth taking some time to tweak the renovate config. Things like scheduling (i.e. once a week) and grouping minor/patch bumps together can go a long way to reduce renovate's noisiness in PRs and commit history.
Happy to open a PR if you're open to what I might come up with, or offer suggestions. Feel free to take inspiration from Apollo Server's renovate config or any of the integrations in this org that use it!

Sounds great. I'd be super thankful if you could open a PR!


Update go-org & fix includes

see #6322
The org mode renderer supports including other files [1]. To resolve relative
paths we need the path of the current file - either absolute or relative to the
directory hugo is run from. DocumentName is the path relative to the content
directory, not the base directory that hugo is run from - we could get the full
path by looking up the content directory from the config and merging both; or
we just make the absolute path of rendered files available in the
RenderingContext.
[1]: e.g. #+INCLUDE: ./foo.py src python includes foo.py as a python source
block.

Hugo has this concept of a virtual file system:

With composable/union mounts etc.
With some built-in sandboxing (security motivation), most notably: Only the project (not themes) can mount files outside of the project dir, only the project has symlink support.

The first point may not be a deal-breaker, but I'm not thrilled about themes "poking around on my disk and including stuff".
So what I suggest instead is that you instead of sending the filename, send in an "OpenFilefunc. that returns aio.ReadCloser`.

The first point may not be a deal-breaker, but I'm not thrilled about themes "poking around on my disk and including stuff".

Could you elaborate why you say theme? This is part of go-org, i.e. the Org mode / .org renderer and should be unrelated to themes. I understand the security concerns though and like the OpenFilefunc idea - could you give me some pointers on where I could get that from? I didn't see anything obvious on a glance...

This is part of go-org, i.e. the Org mode / .org renderer and should be unrelated to themes. I

When we, recently, introduced Hugo Modules, we added support for file mounts, including content from themes/theme components (including org files).
As to "Open File"
See https://godoc.org/github.com/gohugoio/hugo/helpers#PathSpec => https://godoc.org/github.com/gohugoio/hugo/hugolib/filesystems#BaseFs
If you use the Content filesystem to open the files you can just Path and relative filenames.
As to how to get that filesystem to where you need it, I'm not sure. I have plans on moving these "content renderer" into its own package once I add Goldmark.

Thx! I got the BaseFS from page.pageCommon.
Would you be fine something along this changeset? If so I'll add ReadFile to go-org and update the PR with v0.1.6.
The current ReadFile allows relative includes by checking in the content FS first but falls back to trying to read the path relative to the project root. I guess that's a bad idea security wise so maybe we should just allow the content fs...

@bep ping. Ignore my last comment, I restricted it to just Content.Fs for now.

So, I don't see the security problem in reading files from the project root. We have plenty of examples of us doing that (there is even a Work filesystem in there that points to the project root.
But there are obvious benefits to having the files live in the standard Hugo folders:

You can compose/combine the fs via theme components.
The folders are watched in server mode
And when a file is changed, we can look at it and do a partial rebuild


Prep for 0.26.2

Ratatui 0.26.2 release will be a non-breaking release and then we'll release 0.27.0 with the breaking changes.
A quick TODO list:

[ ] Determine any PRs that we want to get finished and into 0.26.2 @ratatui-org/maintainers.
[ ] Determine if there are any easy-wins in the issues that we should fix
[x] Write release post - @orhun?
[x] Release @orhun

My short list of PRs that I think probably should be in 0.26.2 is:

https://github.com/ratatui-org/ratatui/pull/1001
https://github.com/ratatui-org/ratatui/pull/991 ... maybe

Sounds good, thanks for creating the checklist. Let's aim for next Monday.

I may not personally get to either of the two things I listed before Monday due to other things on my plate. We can release without them however.

Ready for tomorrow, feel free to review:

https://github.com/ratatui-org/ratatui/pull/1029
https://github.com/ratatui-org/ratatui-website/pull/566

Release is completed 🧀

https://github.com/ratatui-org/ratatui/releases/tag/v0.26.2
https://crates.io/crates/ratatui
https://ratatui.rs/highlights/v0262/


Latest version not published to npmjs

Please could you publish the lasted version (with the react warnings fix) to npm.
Thanks

Bump…

Just a tip for people here, you can use the version here directly instead of waiting for npm by replacing the version number in package.json with "coma/svg-reactify#master" (replace master with a commit-ish string for any specific version).
@coma If you don't care about this project any more, try to ask for additional maintainers to offload you. Add publish rights to the package on NPM and write rights here on Github: finished :boom:


add and use WarningsAsErrors cmake module (#3)

add and use WarningsAsErrors cmake module

export cxx_std_20 with target_compile_features()
but do not export pedantic Warning options!

Hey, thanks for the PR!
TBH I'm not very sure about using warnings as errors globally, as it could prevent users from using your library or introduce backwards compatibility problems as warnings may differ a lot between platforms and compiler versions.
Adding the warnings as errors was purposely used only in the tests path, as the feature is intended for the library developers to detect any unexpected warnings and should not to impact any users down the line.


#618 - Add ability to name specific wiremock instance

Added ability to name specific wiremock instance.
References
https://github.com/wiremock/wiremock/issues/618

Submitter checklist

[x] Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
[x] Recommended: If you participate in Hacktoberfest 2023, make sure you're signed up there and in the WireMock form
[x] The PR request is well described and justified, including the body and the references
[x] The PR title represents the desired changelog entry
[x] The repository's code style is followed (see the contributing guide)
[ ] Test coverage that demonstrates that the change works as expected
[ ] For new features, there's necessary documentation in this pull request or in a subsequent PR to [wiremock.org]

This is looking good so far but let's add add a test that checks the name is actually emitted in logs.
Also, what might be interesting is to also have it affect the Slf4jNotifier that's used when starting programmatically. Specifically, we could modify line 23 to make it an instance field and set the logger name to the server name, something like:
private final Logger log = LoggerFactory.getLogger(firstNonNull(serverName, "WireMock"));

Options#getServerName() is still a breaking change, because there is no default implementation https://docs.oracle.com/javase/tutorial/java/IandI/defaultmethods.html

incorporated


Erros when running finetune

Hello @cvondrick ,
It seems that audio_simple, the dataset variable defined in main_finetune.lua, is not valid for data.lua to load the input audios. It shows following error:
/home/yclin/distro/install/bin/luajit: /home/yclin/Workspace/soundnet/data/data.lua:24: Unknown dataset: audio_simple

I also tried to replace audio_simple with donkey_audio and donkey_audio_labeled, but none of them work.
Would you please have a look in the finetune section of README?

Sorry for the typo. The correct name of the data loader is 'audio_labeled'


Use implicit package references

As per https://github.com/dotnet/standard/issues/71, implicit NETStandard.Library references should be preferred over individual packages.

It sounds like package management is being improved in NetStandard.Library 1.6.2 and 2.0 and will no longer have the massive dependency list it currently does.


Header_actions on radial bar chart

Checklist

[X] I updated the card to the latest version available
[X] I cleared the cache of my browser
[X] I verified that I'm really running the lastest version in my browser console
[X] I checked if there is another issue opened with the same problem

Using header_actions does not work with radial bar chart. Clicking on the title has no effect. Tested with action: navigate and fire-dom-event.
Version: 1.10.1-dev1
This is the configuration I used:
    type: 'custom:apexcharts-card'
    config_templates: radial_bar
    header:
      show: true
      title: CDRXStation1
    all_series_config:
      header_actions:
        tap_action:
          action: fire-dom-event
          browser_mod:
            command: popup
            card:
              type: 'custom:swipe-card'
              cards:
                - !include ../frontend/station1.yaml
                - !include ../frontend/station2.yaml
                - !include ../frontend/station3.yaml
                - !include ../frontend/stationB.yaml
            style: |
              ha-dialog {
                --mdc-dialog-max-width: 100vw !important;
              }
              :host .content {
                width: 98vw !important;
              }
              ha-card {
                border-radius: 0.8vw;
                opacity: 0.9;
              }
            title: Synology NAS
    apex_config:
      title:
        margin: 0
      legend:
        horizontalAlign: left
      plotOptions:
        radialBar:
          dataLabels:
            show: false
    series:
      - entity: sensor.cdrxstation1_volume_1_volume_used
        name: Volume 1
        color: '#03a8f4'
      - entity: sensor.cdrxstation1_volume_2_volume_used
        name: Volume 2
        color: '#ffa31a'
    card_mod:
      style: |
        :host {
          height: 100%;
        }
        ha-card {
          box-shadow: none !important;
          height: 100%;
        }
        .wrapper.with-header {
          height: 100%;
        }
        .apexcharts-inner.apexcharts-graphical {
          transform: translate(-22.5%, -5%);
        }
        .apexcharts-legend {
          width: 50%;
        }
        div#graph-wrapper {
          transform: translate(0, -5%);
        }

Expected behavior
Action should be triggered by clicking on the header (e.g. the card title)
Desktop:

Browser: Chrome
Version: 95.0.4638.69 64bit

Smartphone:

Device: Realme X2 Pro
OS: Android 11
Browser: Home Assistant Companion App
Version: 2021.10.0-full

Nope, not stale, just dormant ;-)

Up, still not resolved.

I'll fix that soon, sorry, been quite busy lately 😊

No stress, just keeping the issue alive ;-)

Up ;-)

Up!

Not stale, just dormant!

I can't reproduce this issue, it works for me whatever the chart and whatever the action is (including browser-mod).
I'll soon publish a new version with updated dependencies which might have fixed the issue in my test env.
Please report once it's released :)

Interesting! Looking forward to testing the new release. In the meantime, a quick question: I only show the chart title in the header in this particular use case. Could it be that the title does not accept the header actions?

It's released as beta, please let me know :)

Thanks for the clarification. But if the title doesn't support actions, which part of the header does in a radial bar chart? As far as I've experienced, the states are not shown in the header in this type of chart, so basically that would mean there is no "actionnable" element in a radial bar chart.
Actions work fine in all bar/line/area/hybrid charts. Opening a FR for header actions to be supported on the title ;-)

You can activate the states in the header for all the charts.
header:
  show_states: true

Thanks! I can confirm it works in radial bar as well. Closing this issue, as it has now become a FR, see #323. Thanks @RomRider for your help.


Internal Server Error

New deployments of this web app from the Azure AI Studio Preview are resulting in an "Internal Server Error".  This occurs with or without deploying Chat History.

Hi, i'm trying to pull this repository from azure app service but it's giving "internal server error". Is this general problem?

I'm seeing the same behavior. App Service Logs and Diagnostic Logs do not show anything related to the error.

Hi, we're looking into an issue with AI studio web app deployment. In the meantime, you can try to deploy from Azure OpenAI Studio.

@cameronjy @jbrodsky-code You can also "rescue" your AI studio app by configuring the App Startup command like below:

Be sure to restart the app and give it a few minutes before trying again.

Thanks, I'll give it a try.

@sarah-widder That startup command worked.  Thanks again!

Worked for me to :)

it worked well Thx b.
I was struggling with persistent 'internal errors' during local deployment, but I managed to resolve the issue using the method described above.

I am getting this error as well


Output is very slow

Couldn't find an appropriate issue for this, but output in cmder seems to be very slow. This is particularly evident when there is a lot of output produced. For example, if I'm using Python and I run a large suite of tests, a lot of output is printed. If I run this in Powershell, it flies through the output and is finished in a couple of seconds. In cmder, however, it takes many seconds and although I can tell the tests are finished, it just seems to be taking a long time to clear the output.
Any clues as to what settings might effect this/if this seems to be a bug?

+1

I have the same problem - all commands are significantly slower than normal cmd (sometimes take a few seconds). :cry:

+1 the output is noticeably slow, even from a fresh download and just running the ls or ls -lh commands straight out of the box. Native commands seem to be reasonably fast (compare dir with ls).

I can verify this as well. I have downloaded cmder yesterday and ls is noticeably slower than when I execute it from cmd. Also, native commands like dir run much faster than say ls.

+1. I noticed that after re-installig cmder, everything went back to the normal, but after a few days of use it becomes slow again!

+1. Right after starting cmder with a fresh install, running "ls" took around 10-14 seconds. dir in the same console took less than few milliseconds.

The same here, just ls output  is significantly slow !

+1 Same here. On Windows 10 Pro 64bits.

I seem to be having the same problem, but it seems to be only when working within a git repo

+1

+1

I followed the advice on another thread and changed cmder.lua to return false in get_git_dir.
local function get_git_dir(path)
return false
--return get_dir_contains(path, '.git')
end
Native commands are still far faster (dir vs. ls), but at least it isn't slow just to type characters in cmder.

+1

+1 the trick with get_git_dir fixed it for me.


fix(linter): change consistent-function-scoping back to correctness

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5016  👈
#5015
#5011
#5010
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


optimize extension loader log

Motivation:
FIX https://github.com/alipay/sofa-rpc/issues/278
Modification:
Describe the idea and modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

Codecov Report

Merging #279 into 5.4 will increase coverage by 0.21%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##               5.4     #279      +/-   ##
===========================================
+ Coverage     71.8%   72.01%   +0.21%
- Complexity     963      968       +5
===========================================
  Files          342      342
  Lines        14274    14286      +12
  Branches      2296     2301       +5
===========================================
+ Hits         10249    10288      +39
+ Misses        2831     2803      -28
- Partials      1194     1195       +1

Impacted Files
Coverage Δ
Complexity Δ

...a/com/alipay/sofa/rpc/codec/SerializerFactory.java
93.75% <100%> (+1.44%)
0 <0> (ø)
:arrow_down:

...y/sofa/rpc/server/http/AbstractHttpServerTask.java
78.7% <100%> (+0.4%)
14 <0> (+1)
:arrow_up:

...a/com/alipay/sofa/rpc/codec/CompressorFactory.java
92.85% <100%> (+1.94%)
0 <0> (ø)
:arrow_down:

.../java/com/alipay/sofa/rpc/ext/ExtensionLoader.java
81.81% <50%> (-0.33%)
0 <0> (ø)

...m/alipay/sofa/rpc/server/http/Http2ServerTask.java
82.6% <60%> (-7.87%)
6 <0> (ø)

...com/alipay/sofa/rpc/context/RpcRuntimeContext.java
90.24% <0%> (+1.21%)
0% <0%> (ø)
:arrow_down:

...pay/sofa/rpc/transport/ClientTransportFactory.java
73.84% <0%> (+1.53%)
0% <0%> (ø)
:arrow_down:

.../sofa/rpc/bootstrap/DefaultClientProxyInvoker.java
88.7% <0%> (+1.61%)
0% <0%> (ø)
:arrow_down:

...c/transport/http/AbstractHttp2ClientTransport.java
80.48% <0%> (+2.43%)
29% <0%> (ø)
:arrow_down:

.../main/java/com/alipay/sofa/rpc/tracer/Tracers.java
42.3% <0%> (+2.56%)
0% <0%> (ø)
:arrow_down:

... and 4 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 73d1b87...aa1f371. Read the comment docs.


USER CAN VIEW HOVER STATE

[ ]  User should be able to view hover state of all components on the page.

hi boss, welldone
i want to bring to your attention that this is not the format we are using
we are not adding that user should in the title
You might want to use view hover state
Shey you get


[Snyk] Security upgrade idna from 2.7 to 3.7

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.txt

⚠ Warning
requests 2.20.0 has requirement idna<2.8,>=2.5, but you have idna 3.7.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

596/1000  Why? Recently disclosed, Has a fix available, CVSS 6.2
Resource Exhaustion SNYK-PYTHON-IDNA-6597975
idna:  2.7 -> 3.7
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Resource Exhaustion

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


Property Name "System" conflicts with Namespace in C#

We are using NSwagStudio 13.18.2 to generate a C# client for our backend.
One of our data types contains a property called "System":
 "PositionMatched": {
        "type": "object",
        "additionalProperties": false,
        "required": [
          "System",
          "X",
          "Y",
          "Z"
        ],
        "properties": {
          "System": {
            "type": "boolean",
            "nullable": true
          },
          "X": {
            "type": "boolean",
            "nullable": true
          },
          "Y": {
            "type": "boolean",
            "nullable": true
          },
          "Z": {
            "type": "boolean",
            "nullable": true
          }
        }
      },

This results into a conflict with the "System" namespace of the generated JsonIgnore statement:
    [System.CodeDom.Compiler.GeneratedCode("NJsonSchema", "<IP_ADDRESS> (NJsonSchema v<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>))")]
    public partial class PositionMatched
    {
        [System.Text.Json.Serialization.JsonPropertyName("System ")]
        [System.Text.Json.Serialization.JsonIgnore(Condition = System.Text.Json.Serialization.JsonIgnoreCondition.Never)]
        public bool? System { get; set; }

As the generated code already contains the following statement:
  using System = global::System;
it would be better to reference the System namespace within the generated code with
System::Text.Json.Serialization.JsonIgnoreCondition.Never
instead of
System.Text.Json.Serialization.JsonIgnoreCondition.Never

I'm running into this as well.

Hello, faced with it on 13.20.0


Workaround the crucial tx + full mempool + blocked forge scenario

This error was first seen in https://buildkite.com/input-output-hk/ouroboros-network-nightly/builds/220#6223fd68-d16b-4dc4-b4ad-3f652711e36a -- it's the insufficient growth error in RealTPraos. (The other is Issue 2506 again, whose fix is currently under review.)
PR #2543 changed the timing of the ledger access, which made block forging "faster", which meant that a node's crucial tx for a slot it leads would not necessarily be included in the block it forged. So PR 2543 also introduce explicit synchronization so that the node won't forge until the crucial transaction had entered the mempool.
This new failure is due to the mempool becoming full, so the addTxs for the crucial transaction blocks for multiple slots, and that in turn blocks the forge. This adds a "message delay" that the sensitive ThreadNet properties currently identify as an unexpected slow down in the chain growth (eg it sees that node 1 lead slot 446, but the corresponding new block didn't arrive until slot 460.)
This Issue is Done once we prevent this scenario from causing test failures.

Commit e1fda312c193c843d7b05a4deeb3d41618edd6f8 with RealTPraos:
The crucial transaction is the parameter protocol update arising from setupD /= setupD2.
repro :: TestSetup
repro = TestSetup
   { setupD = DecentralizationParam (2 / 5)
   , setupD2 = DecentralizationParam (1 / 5)
   , setupInitialNonce = Nonce (fromString "1f07d66d23b47096914bf4d5597f751611f8ed7bd9026743b38b345949f3787b")
   , setupK = SecurityParam 10
   , setupTestConfig = TestConfig
     { initSeed = Seed<PHONE_NUMBER>403606719
     , nodeTopology = NodeTopology (Map.fromList [(CoreNodeId 0,Set.fromList []),(CoreNodeId 1,Set.fromList [CoreNodeId 0]),(CoreNodeId 2,Set.fromList [CoreNodeId 1]),(CoreNodeId 3,Set.fromList [CoreNodeId 0,CoreNodeId 1,CoreNodeId 2]),(CoreNodeId 4,Set.fromList [CoreNodeId 0,CoreNodeId 1,CoreNodeId 3])])
     , numCoreNodes = NumCoreNodes 5
     , numSlots = NumSlots 481
     }
   , setupVersion = (NodeToNodeV_1,Shelley.ShelleyNodeToNodeVersion1)
   }

A few thoughts:

addTxs will block when the Mempool is full, but there is also tryAddTxs which does not block. Using this would prevent blocking when adding the crucial transactions, but the result would be that the crucial transaction didn't make it into the Mempool. Or do we know that the Mempool already contains the crucial transaction in question?
Not really a fix, but something that will also be beneficial for performance: when I last profiled the RealTPraos ThreadNet tests, genTx wall the hotspot. Maybe we can lower the number of transactions to generate? This is not a proper fix, of course, because the Mempool might still get full.

Yeah, tryAddTxs would unblock us. But the main semantics of the crucial transaction abstraction suggests we shouldn't move on until they've been added or rejected.
We use crucial transactions for a few things:

RealPBFT: update the deleg cert when rekeying a node. This happens in whatever slot the node is scheduled to rekey. The mempool might be full by then.

RealPBFT: add the protocol version update proposal in the first epoch. The mempool probably is not full yet.

RealTPraos: we use them to update the d parameter in the first epoch. The mempool is probably not full yet.

Cardano: we use them to initiate the hard fork in the first epoch. The mempool is probably not full yet.

The above sentences about whether the mempool is full or not are considering the first time the crucial transaction arises. However, the wrinkle is that once a crucial transaction is specified, the node attempts to add it to the mempool on every subsequent slot. This was necessary because it's possible for the mempool to get flushed into an ultimately invalid block, which means the crucial transaction (eg a Byron redeleg cert) might not end up on the chain. Therefore the node just continually adds it crucial txs for the rest of the test. If it's already in the mempool/on the chain, it should be rejected as a duplicate/stale (for Byron) or invalid (for Shelley, since its input has already been consumed). But apparently (see the failure that lead to this ticket eg) this rejection is blocked if the mempool is full.
This is all to say that I think we could avoid this issue by enriching the test infrastructure to explicitly detect when a crucial transaction makes it onto the immutable chain and then drop that transaction from the set of crucial-transactions-that-need-to-be-constantly-added-to-the-mempool. But that sounds difficult/expensive to implement and maintain.

This reminds me of https://jira.iohk.io/browse/ADP-412, i.e., the ability to ask the Mempool whether a transaction is valid or not without adding it to the Mempool (ignore the "explain clearly why it's invalid" part, that's for the ledger). Do you think implementing that (which we will have to do anyway and will be simple), would be enough?

This nightly https://buildkite.com/input-output-hk/ouroboros-network-nightly/builds/223#d2dae7ac-3c98-47a4-9a2c-b4b88a148ac6 also failed with insufficient growth. It has the transaction, and it fails after hundreds of slots, so I suspect it is the same issue. (Moreover, I'm not seeing how the reported leader schedule would otherwise explain the error.)

Do you think implementing that (which we will have to do anyway and will be simple), would be enough?

@mrBliss, it seems like a step in the right direction. I think the ideal behavior is that the node should stop trying to add the transaction at every slot onset once it knows it will never be valid again. For example, IIUC, that's true if the transaction is already on the immutable chain. And I think that should work for these RealTPraos errors we're seeing.
But I'm not certain that all crucial transactions (eg redelegation certificates in RealPBFT/Byron tests) can assume their validity wrt to the mutable current chain begins as "valid" and only transitions once, to "invalid" --- I believe it may waver back and forth or at least start as "invalid". I'll have to allocate some time to confirm that, but that's my recollection.

Also https://buildkite.com/input-output-hk/ouroboros-network-nightly/builds/235#f5b3c2be-0468-4594-b290-14476fc366e6 I think.


feat: :sparkles: add YouTube playlist prop

add additional props to allow for embedding of YouTube playlists.

Aight!
I hadn't merged this one because the Twitch one was still failing 🙃
I'll rebase an merge


Milestone 4: Improvements

[ ] #16
[ ] #17
[ ] #20

Sprint 1: https://github.com/orgs/cryptape/projects/46/views/1?sliceBy[value]=Sprint+1


Application template not looked for in theme directory

I have a route mapped to a template name
shop_domaines_show:
    path: /domaines/{slug}
    defaults:
        _controller: black_sylius_marketplace.controller.seller:showAction
        _sylius:
            template: 'Shop/views/domaines/show.html.twig'
            repository:
                method: findOneBySlug
                arguments:
                    - $slug

And a template in templates/Shop/views/domaines/show.html.twig
It works ok.
When I want to override this template in the theme it fails.
I put a new file in themes/MyTheme/Shop/views/domaines/show.html.twig
And it's not loaded.
If I remove the template in the templates dir, leaving only the one in the theme, here is the error I get :
Unable to find template "Shop/views/domaines/show.html.twig" (looked into: /project/vendor/knplabs/knp-menu/src/Knp/Menu/Resources/views, /project/templates, /project/templates, /project/vendor/symfony/twig-bridge/Resources/views/Form).

The loader is not looking in the theme directory. Note it seems to be looking twice in the /project/templates dir.
The problem only occurs for app templates. Bundle templates are loaded perfectly fine from the theme directory.
I'm using Sylius 1.8 and sylius/theme-bundle 1.5.1

A temporary fix is to add the theme directory to twig paths but it is not a solution
in config/packages/twig.yaml
twig:
    # DO NOT COMMIT THIS!
    paths: ['%kernel.project_dir%/templates', '%kernel.project_dir%/themes/MyTheme']


git clone via https protocol instead of ssh

Validators can't clone using their ssh keys.

@iamoskvin Thanks for your PR!
If you don't mind, can you please:

rebase your work against main (following changes on the main branch, your PR is now in conflict),
fix the commit message to respect the conventional commit formalism (see okp4/wiki/Naming-rules#commit-messages). You can prefix with fix:.

Many thanks. 😌

@iamoskvin Thanks for your PR!
If you don't mind, can you please:

rebase your work against main (following changes on the main branch, your PR is now in conflict),
fix the commit message to respect the conventional commit formalism (see okp4/wiki/Naming-rules#commit-messages). You can prefix with fix:.

Many thanks. relieved

Thank you.
I have fixed the commit message (git commit --amend).
Also, I tried to rebase but I suffer from conflicts between main branch and other people commits at this PR branch (file docs/index rename, I think). Unfortunately, I am not a git profi, so I don't know how to resolve this problem...
"git pull --rebase upstream main
remote: Enumerating objects: 45, done.
remote: Counting objects: 100% (45/45), done.
remote: Compressing objects: 100% (30/30), done.
remote: Total 45 (delta 25), reused 31 (delta 15), pack-reused 0
Unpacking objects: 100% (45/45), done.
From https://github.com/okp4/docs

branch            main       -> FETCH_HEAD
[new branch]      main       -> upstream/main
First, rewinding head to replay your work on top of it...
Applying: feat: correct grammar errors in abstract
Applying: feat: change title of index
Using index info to reconstruct a base tree...
A	docs/index
.git/rebase-apply/patch:10: trailing whitespace.

OKP4 Whitepaper and Documentation
warning: 1 line adds whitespace errors.
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): docs/index deleted in 28d6276c8c771bb67005c36d38f9062e3327fe17 and modified in feat: change title of index. Version feat: change title of index of docs/index left in tree.
error: Failed to merge in the changes.
Patch failed at 0002 feat: change title of index
The copy of the patch that failed is found in: .git/rebase-apply/patch"

@iamoskvin I see. We had to push on the principal and that's why your PR is now of a conflict. Sorry for that.
To solve the issue, I would like to suggest the following approach (by re-integrating https://github.com/okp4/docs/commit/774adb89fc8017de2549a0c770ce8ef5f13d0a98)
on your branch:
git reset --hard upstream/main
git cherry-pick 774adb8
git commit --amend --message "fix: git clone via https protocol instead of ssh"
git push --force

Tell me if it works. Thanks to you.

@iamoskvin I see. We had to push on the main branch and that's why your PR is now conflicting. Sorry for that.
To solve the issue, I would like to suggest the following approach (by re-integrating 774adb8)
on your branch:
git reset --hard upstream/main
git cherry-pick 774adb8
git commit --amend --message "fix: git clone via https protocol instead of ssh"
git push --force

Tell me if it works. Thanks to you.

Many thanks for your assistance!
I think it worked for me. I have picked another commit from my repo (I can't pick 774adb8).
Please check the result.

@amimart ok for you?

Great. Thanks :+1:


CCLS and unhandled "publishSkippedRanges" notification

OS and language server:
ArchLinux with ccls-git
How you installed LSP (Package Control or from git?):
PackageControl
Minimal reproduction steps

Install CCLS from git
Open end edit c++ file (.h or .cpp)
Save

Log
LSP:  --> textDocument/documentHighlight
LSP:      []
LSP:  --> textDocument/codeAction
LSP:      []
LSP:  --> textDocument/documentHighlight
LSP:      []
LSP:  --> textDocument/codeAction
LSP:      []
LSP:  --> textDocument/didChange
LSP: <--  textDocument/publishDiagnostics
LSP:      {'uri': 'include/core/application.h', 'diagnostics': []}
LSP: <--  $ccls/publishSkippedRanges
LSP:      {'uri': 'include/core/application.h', 'skippedRanges': []}
LSP: Unhandled notification: $ccls/publishSkippedRanges
LSP: <--  $ccls/publishSemanticHighlight
LSP:      {'symbols': [{'kind': 3, 'storage': 0, 'parentKind': 0, 'ranges': [{'L': 24, 'R': 31}], 'lsRanges': [], 'id': 1}, {'kind': 9, 'storage': 0, 'parentKind': 0, 'ranges': [{'L': 61, 'R': 72}], 'lsRanges': [], 'id': 2}, {'kind': 5, 'storage': 0, 'parentKind': 3, 'ranges': [{'L': 39, 'R': 50}, {'L': 77, 'R': 88}], 'lsRanges': [], 'id': 0}, {'kind': 6, 'storage': 0, 'parentKind': 0, 'ranges': [{'L': 76, 'R': 77}], 'lsRanges': [], 'id': 1}], 'uri': 'include/core/application.h'}
LSP: Unhandled notification: $ccls/publishSemanticHighlight
LSP:  --> textDocument/didChange
LSP: <--  textDocument/publishDiagnostics
LSP:      {'uri': 'include/core/application.h', 'diagnostics': []}
LSP: <--  $ccls/publishSkippedRanges

Hi, it looks like this language server requires some extra support on the client side, which you can add to LSP by writing a sublime package that registers a language handler.
Are you interested in implementing this?

I would try :D Never done that before.

Related issue: https://github.com/MaskRay/ccls/issues/95#issuecomment-430281049

Okay, I think the unhandled notification should be harmless. If you can tell us what the notification means and what Sublime Text should do with it, we can figure out what we have to gain from writing a language handler for it!


Sorts out the get error

https://github.com/MichaelHatherly/Docile.jl/issues/106#issuecomment-101039792

!!get errors / warning is much simpler to sort out and would
definitely be a good thing.

see inline question

ok


Not worked with belongs_to :relation, touch: true

Followings is some example:

  # models/message.rb
  class Message < ApplicationRecord
     belong_to :chatroom, touch: true
  end

      chatroom1 = create(
        :chatroom,
        chatroomable: create(:goods_item),
        user: user
      )
      create(:message, chatroom: chatroom1, created_at: Time.current - 13.hours)
      chatroom2 = create(
        :chatroom,
        chatroomable: create(:goods_item),
        user: user
      )
      create(:message, chatroom: chatroom2, created_at: Time.current - 2.hours)

What I expect is: chatroom1.updated_at is Time.current - 13.hours, and chatroom2.updated_at is Time.current - 2.hours.

Sorry,  This is worked.


Pills not converted properly in kicad file conversion

[x] Update https://github.com/tscircuit/kicad-mod-converter to support outputting "pills" for some kicad shapes
[x] Update https://github.com/tscircuit/pcb-viewer to support rendering pills
[x] Update https://github.com/tscircuit/kicad-viewer dependencies to fix conversion & pcb-viewer

Before

After

From the image above, that's still not quite correct, the kicad version is actually a pill not an oval

Yeah, I will add the pill shape now. Just wanted to do an oval first, will add the pill shape after this

@imrishabh18 that's amazing!!!!


Next: Merge from master: Improve rendering performance when pxPerSec is high (#909)

Adds an optional 'partialRender' parameter to enable
Calculates and renders peaks only for current visible waveform
Keeps track of currently calculated/rendered peaks to avoid
duplicate calculation and only incremental scroll changes are rendered

Tested all combinations of Canvas/MultiCanvas and Wave/Bars rendering
at various zoom levels.

@mspae is it possible to fix these travis failures here as well, e.g. https://travis-ci.org/katspaugh/wavesurfer.js/builds/193778276?


fix TLS and add JKS supporting

What problem does this PR solve?

Fix TLS: TLS test failure #2354 and TLS can't work when building TiSpark in jar #2356.
Add JKS support.
Fix TLS Github action #2350

What is changed and how it works?
Fix TLS

Dependency conflicting makes TLS test failure.
To solve it, I have added a netty-handler dependency in tikv-client

        <dependency>
            <groupId>io.netty</groupId>
            <artifactId>netty-handler</artifactId>
            <version>4.1.48.Final</version>
        </dependency>

ALPN lacking makes TLS can't work after shading io.netty.
It can be solved by affording some native dynamic libraries. Such as libnetty_tcnative_linux_x86_64.so.
But when shading io.netty, we should change lib names and add prefix, such as libshade_netty_tcnative_linux_x86.so.

Add JKS support.
Now you can use the JKS key store to build a TLS connection.
To enable JKS, you have to open it in configuration. such as,
spark.tispark.tikv.tls_enable=true
spark.tispark.tikv.jks_enable=true
spark.tispark.tikv.jks_key_path=/config/cert/client-keystore
spark.tispark.tikv.jks_key_password=123456
spark.tispark.tikv.jks_trust_path=/config/cert/server-cert-store
spark.tispark.tikv.jks_trust_password=12345678

When JKS and PEM cert are set simultaneously, JKS would have a higher priority.
That means TLS builder will use JKS cert first.
So, do not set spark.tispark.tikv.jks_enable=true when you just want to use a common PEM cert.
Correct TLS test command
Change Github action TLS test command:
mvn test -am -pl core -Dtest=moo -DwildcardSuites=com.pingcap.tispark.tls.JDBCTLSSuite,com.pingcap.tispark.tls.TiKVClientTLSSuite,com.pingcap.tispark.tls.TiSparkTLSSuite -DfailIfNoTests=false

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to be included in the release note

/run-all-tests

/run-all-tests

/run-all-tests

Can you open issues describing what problem is TLS and TLS Github action

/run-all-tests profile=spark3.2.1

/run-all-tests profile=spark3.2.1

Doc need be  updated too.

@shiyuhang0 You are right. I have tested Spark-SQL with TiSpark-jar in spark 3.0, 3.1, and 3.2.

/hold

/unhold

/run-all-tests

/run-all-tests

/merge

/merge


Clearing history causes next copy to fail

Describe the bug
When a clear history is performed the next copy will fail due to _debouncing++.
How To Reproduce

Click the trash icon to clear history
Select text and copy it

Versions
GNOME Shell 42.2
GCH   Version: 17
org.gnome.shell.extensions.clipboard-history cache-only-favorites false
org.gnome.shell.extensions.clipboard-history cache-size 100
org.gnome.shell.extensions.clipboard-history clear-history @as []
org.gnome.shell.extensions.clipboard-history confirm-clear true
org.gnome.shell.extensions.clipboard-history disable-down-arrow true
org.gnome.shell.extensions.clipboard-history display-mode 0
org.gnome.shell.extensions.clipboard-history enable-keybindings true
org.gnome.shell.extensions.clipboard-history history-size 1000
org.gnome.shell.extensions.clipboard-history move-item-first true
org.gnome.shell.extensions.clipboard-history next-entry @as []
org.gnome.shell.extensions.clipboard-history notify-on-copy false
org.gnome.shell.extensions.clipboard-history paste-on-selection true
org.gnome.shell.extensions.clipboard-history prev-entry @as []
org.gnome.shell.extensions.clipboard-history private-mode false
org.gnome.shell.extensions.clipboard-history process-primary-selection false
org.gnome.shell.extensions.clipboard-history strip-text false
org.gnome.shell.extensions.clipboard-history toggle-menu ['<Super><Shift>V']
org.gnome.shell.extensions.clipboard-history toggle-private-mode ['<Super><Shift>P']
org.gnome.shell.extensions.clipboard-history topbar-preview-size 10
org.gnome.shell.extensions.clipboard-history window-width-percentage 33

Additional context (if a crash, provide stack trace)
When _clearHistory() is called it calls _resetSelectedMenuItem() which calls _setClipboardText('') which in turn increments _debouncing and causes the next copy to fail.
A possible solution is to not increment _debouncing if the text is empty:
  _setClipboardText(text) {
    if (text !== '' && this._debouncing !== undefined) {
      this._debouncing++;
    }

    Clipboard.set_text(St.ClipboardType.CLIPBOARD, text);
    Clipboard.set_text(St.ClipboardType.PRIMARY, text);
  }

Hmmm, I wasn't able to repro. For me setting text = '' still results in a _processClipboardContent call. I also tried clearing my history and was able to copy something immediately afterwards and have it show up.

When you clear history is this._debouncing++ in _setClipboardText being executed for you?
For me it happens consistently and then causes this check to return for the next copy:
  _processClipboardContent(text) {
    if (this._debouncing > 0) {
      this._debouncing--;
      return;
    }

Those settings look like mine. For me, this._debouncing--; gets called when I clear history. Before the if statement, you could add log(Me.uuid, "_processClipboardContent"); and then look at the journal: journalctl -f /usr/bin/gnome-shell.

So I added some log output as requested:
  _setClipboardText(text) {
    if (this._debouncing !== undefined) {
      log(Me.uuid, "_setClipboardText _debouncing++:", this._debouncing);
      this._debouncing++;
    }

  _processClipboardContent(text, html) {
    if (this._debouncing > 0) {
      log(Me.uuid, "_processClipboardContent _debouncing--:", this._debouncing);
      this._debouncing--;
      return;
    }

Here is the output I get:
# Clear History ==>
<EMAIL_ADDRESS>_setClipboardText _debouncing++:, 0
# Copy text ==>
<EMAIL_ADDRESS>_processClipboardContent _debouncing--:, 1

I'm not sure how this._debouncing--; is getting called when you clear history?

I get this:
# Clear history
<EMAIL_ADDRESS>_clearHistory _debouncing:, 0
<EMAIL_ADDRESS>_setClipboardText _debouncing:, 0
<EMAIL_ADDRESS>_processClipboardContent _debouncing:, 1
# Copy text
<EMAIL_ADDRESS>_processClipboardContent _debouncing:, 0

No idea why they're different.
Maybe a different approach to debouncing is to unregister the clipboard change listener before writing to it and then re-register it.

No worries!


Licensing Requirements

There is nothing on this page about the type of license that must be assigned to users in each role. Please update the page to include this info, i.e., do people assigned to a Compliance Manager role, need O365 E1, E3, or  E5, can guests that don't have licenses be assigned a role?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9f85a2df-74e6-8c40-9d10-adac43191db2
Version Independent ID: 8f8586c5-bdea-eec8-b9f8-d5d4a15737c0
Content: Work with Microsoft Compliance Manager (Preview) - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/working-with-compliance-manager.md
Service: o365-seccomp
GitHub Login: @chvukosw
Microsoft Alias: chvukosw

Thank you for your input, we will be making updates to this page soon.


Xiaomi Air Purifier 3 - No AQI / RPS or other data

Before submitting a bug
[ x] I updated to the latest version available
[ x] I cleared the cache of my browser
Describe the bug
In Home Assistant 2021.9.7 it seems that the Attribute have been broken out into separate sensor entities when using the Xiaomi Miio integration, meaning that the AQI, motor speed, filter percentage and other data is no longer displaying in this card.
I am not sure if this only affects Air Purifier 3 /3H or if others are affected, but I noticed the change recently.
Instead of AQI, Filter % etc being attributes of the fan.xxxxx entity, they seem to be broken out to separate sensors as in sensor.xxxx_pm2.5, sensor.xxxxx_filter_life etc etc.
Steps to reproduce

Removed integration
Reinstalled using Integration in Home Assistant
Options still broken out as sensors and not as attributes of the fan.xxxx entity
No data in Air Purifier Card

Expected behavior
To see data in the air purifier card
Screenshots
If applicable, add screenshots to help explain your problem.
For example, JavaScript errors are shown in the web inspector (if applicable).
Versions:

Release: [e.g. 0.13.0]
Browser: [e.g. Google Chrome, (desktop and mobile) and Microsoft Edge (Chromium base, new version).
HomeAssistant version: [e.g 2021.9.7]

Because attributes now become entities.  I managed customize some of entities:
`type: vertical-stack
in_card: true
cards:

type: entities
no_card: true
title: Powietrze w domu
show_header_toggle: false
state_color: true
entities:

fan.air_purifier_2
sensor.air_purifier_pm2_5
sensor.air_purifier_humidity
sensor.air_purifier_temperature

type: custom:purifier-card
entity: fan.air_purifier_2
aqi:

entity_id: sensor.air_purifier_pm2_5
stats:
entity_id: sensor.air_purifier_filter_life_remaining
subtitle: Pozostało filtra
unit: '%'
entity_id: sensor.air_purifier_motor_speed
unit: RPM
subtitle: Obroty
entity_id: sensor.air_purifier_temperature
unit: C
subtitle: Temperatura
actions:
name: Cicho
icon: mdi:weather-night
speed: Silent
name: 25%
icon: mdi:circle-slice-2
speed: Favorite
xiaomi_miio_favorite_level: 3
name: 50%
icon: mdi:circle-slice-4
speed: Favorite
xiaomi_miio_favorite_level: 7
name: 75%
icon: mdi:circle-slice-6
speed: Favorite
xiaomi_miio_favorite_level: 10
name: 100%
icon: mdi:circle-slice-8
speed: Favorite
xiaomi_miio_favorite_level: 14
name: Auto
icon: mdi:brightness-auto
speed: Auto
show_name: true
show_status: true
show_state: true
show_toolbar: true
compact_view: false
`
But it does not work for aqi object.

AQI is probably most important one to me.
Thanks for the work around for some other sensors.   I noticed AQI is broken out as sensor.xxxx_pm2.5.   Might have to play with it until a proper fix is put in place

I'm facing the same issue, the problem with AQI not showing after setting the entity_id is because the way it's setting the value in code, when you set aqui in the config it's an array and the way it's trying to assigning the property is not accesión the array value, it's trying to map an object property.

Just noting this issue is still very much present, just to stop the stalebot grabbing it.

Just noting this issue is still very much present, just to stop the stalebot grabbing it.

Try this modification:
https://github.com/blacksheepinc/purifier-card/tree/master/dist

Hi blacksheepinc
How to install your repo?
I installed by custom url but no purifer card visible on the dashboard.

Hi blacksheepinc How to install your repo? I installed by custom url but no purifer card visible on the dashboard.

I think the easiest would be install from HACS, then change the purifier-card.js from this link (https://github.com/blacksheepinc/purifier-card/blob/master/dist/purifier-card.js), with some file editor or samba plugin.

I tried this way too and also does not work.

Same here AQI displays no data if entity is provided. This needs fixing ;-;

Hi blacksheepinc How to install your repo? I installed by custom url but no purifer card visible on the dashboard.

I think the easiest would be install from HACS, then change the purifier-card.js from this link (https://github.com/blacksheepinc/purifier-card/blob/master/dist/purifier-card.js), with some file editor or samba plugin.

This link doesn't work anymore :(

This project seems to have gone stale with no updates recently.  It is a shame as I have not found anything like it.
Unfortunately blacksheepinc seems to have pulled down their modification of the card also

@xury77 you have mistake in your config over here:
aqi:
  - entity_id: sensor.air_purifier_pm2_5

it must be an object, not an array:
aqi:
  entity_id: sensor.air_purifier_pm2_5
  unit: µg/m³

Folks, I don't see any problem or a bug in the card. The only thing I see in this issue is a common typo you all did.
aqi is an object, not a list. You should not use - when specifying entity_id for aqi object.
I use Xiaomi Air Purifier 3H, it also exposes pollution data as a sensor. Here's my config that works completely fine:
type: custom:purifier-card
entity: fan.air_purifier
show_name: false
show_state: false
aqi:
  entity_id: sensor.air_purifier_pm2_5
  unit: µg/m³
shortcuts:
  - name: Silent
    icon: mdi:weather-night
    speed: Silent
  - name: 25%
    icon: mdi:circle-slice-2
    speed: Favorite
    xiaomi_miio_favorite_level: 3
  - name: 50%
    icon: mdi:circle-slice-4
    speed: Favorite
    xiaomi_miio_favorite_level: 7
  - name: 75%
    icon: mdi:circle-slice-6
    speed: Favorite
    xiaomi_miio_favorite_level: 10
  - name: 100%
    icon: mdi:circle-slice-8
    speed: Favorite
    xiaomi_miio_favorite_level: 14
  - name: Auto
    icon: mdi:brightness-auto
    speed: Auto
stats:
  - entity_id: sensor.air_purifier_filter_life_remaining
    unit: '%'
    subtitle: Залишок фільтру
  - entity_id: sensor.air_purifier_motor_speed
    unit: Об/с
    subtitle: Швидкість мотору
  - entity_id: sensor.air_purifier_filter_use
    unit: год
    subtitle: Залишок фільтру
show_toolbar: true

Yes. You have to  only remove  a dash before entity_id:

Thanks, than worked perfect

@austwhite you don't have to spread panic around.
When the project doesn't have constant changes it is not stale, it is stable and mature. Having more than 4.5K users, it's important to carefully pick what changes are introduced. I can't please everyone, so I must pick what needs to be done.
Also, don't forget that I don't owe anything to anyone. I published my work for free, for everyone. Only a small fraction of those 4.5K users have donated at least something for my work.
Remember, I still have my personal life and my permanent job. If you need any specific changes, feel free to open a PR or fork this repo — the license allows that.
Let's maintain a healthy OSS culture, instead of pushing each other. Take care!

@denysdovhan   I have not donated to your project yet, but I do donate to many opensource projects when I continue using them.
I appreciate the information and pointing out our error, and this indeed does fix the AQI issue.
I am still having issues where the shortcut buttons do not work, but that may be another thing to deal with.
When using the below code for shortcuts, Home Assistant throws a failed to call service error and I get this in the logs.  The log
This may need another issue though
`Logger: homeassistant.components.fan
Source: components/fan/init.py:246
Integration: Fan (documentation, issues)
First occurred: November 30, 2021, 7:59:25 PM (37 occurrences)
Last logged: 6:04:41 AM
The fan.set_speed service is deprecated, use fan.set_percentage or fan.set_preset_mode instead`
`shortcuts:

name: Silent
icon: mdi:weather-night
speed: Silent
name: 25%
icon: mdi:circle-slice-2
speed: Favorite
xiaomi_miio_favorite_level: 3
name: 50%
icon: mdi:circle-slice-4
speed: Favorite
xiaomi_miio_favorite_level: 7
name: 75%
icon: mdi:circle-slice-6
speed: Favorite
xiaomi_miio_favorite_level: 10
name: 100%
icon: mdi:circle-slice-8
speed: Favorite
xiaomi_miio_favorite_level: 14
name: Auto
icon: mdi:brightness-auto
speed: Auto`

@austwhite this might be an actual bug that needs to be fixed. We need to migrate fan.set_speed to new counterparts.

@denysdovhan   Did you need a new issue for that one, or can you work off the information I just gave?