Trouble with "duplicate scope names"

Hello.
This is related with #2248 and #2262.
We have been using Identity Server 2.1.1 to implement our own identity provider that secures a large set of microservices and applications.
The API resources for these microservices were configured using this pattern (which I understand now was wrong):
Settings Microservice:

ApiResource = lithium-settings
Scopes = lithium-settings

Notifications Microservice:

ApiResource = lithium-notifications
Scopes = lithium-notifications

Accounting Microservice:

ApiResource = lithium-accauto
Scopes = lithium-accauto lithium-accauto-access lithium-accauto-creation

So you see that we do not have duplicates between the scopes of different microservices but we do have the "duplicate scopes error" because the ApiResource for each microservice and one of its scopes (the default one) share the same name.
We were planning our migration to Identity Server 3.0.1 when we this error started to popup.
How can we solve this problem without breaking all our microservices and/or their clients when we deploy this new version of Identity Server?
Why should the validation include duplicates on ApiResource/Scope?
Is it possible to at least have some mechanism to customize this validation?
If we change the ApiResource names... we would be required to deploy new versions of all the microservices at the same time we deploy IDS.
If we change the scope names... we would also need to deploy all the microservices and all their clients would be broken (as they would need to start requesting new scopes).
If we simply delete the scope that causes duplication, I think that we would not need to deploy the microservices but their clients would be broken nevertheless because IDS responds with invalid_scope when they request those scopes that now would only match the ApiResource name.
Is there any other way to work around this issue that I'm not seeing?

Sorry I've reviewed the code that was initializing the ApiResources in memory (used in unit tests). This was the code that started having the duplicate scopes error.
I was using the constructor with name and display name and then adding the same scope again...


The component search result tree immediately closes after finding the desired component.

System:
OS: macOS 14.5
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Memory: 83.17 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Yarn: 1.22.22 - /usr/local/bin/yarn
npm: 10.7.0 - ~/Library/Application Support/Herd/config/nvm/versions/node/v20.15.0/bin/npm
pnpm: 9.4.0 - /usr/local/bin/pnpm
Browsers:
Chrome: 126.0.6478.127
Safari: 17.5
npmPackages:
@vue/devtools: ^7.3.5 => 7.3.5

Reproduces with in standalone app as well as Vite plugin.

I will track it

Hi, can you provide a minimal reproduction? I cannot reproduce it on my local.

Hey @alexzhang1030
Certainly! My fault, initially, I believed the issue persisted in a fresh install, but it turns out the problem is isolated to my logo component.
Sandbox reproduction: https://stackblitz.com/edit/vitejs-vite-hfsdfn?file=src%2FLogo%2FQrutoLogo.vue

The issue appears on line 51 of the QrutoLogo component, where the QLogo component is embedded.


Add EZ Recipes Cards

Proposed Changes

Add project cards for the EZ Recipes iOS, Android, and web apps
Update browserslist

Will merge tomorrow since the GitHub API quota has been exceeded.


Unable to use CLN node as destination in defined activity

Describe the bug
When I attempt to use a CLN node as a destination in the defined activity, the simulation does not start and gives the error:
ValidationError("Destination node does not support keysend, cln-0000(0287eb...c97d33)"

However, CLN appears to have keysend enabled and it can send keysend payments when it is used as the source node in the defined activity.
To Reproduce
Configure a simple defined activity with an LND node as the source and a CLN node as the destination.
Possible Solution
I believe this is due to the feature flags not being parsed correctly when creating a CLN node in SimLN.
In the get_node_info function in cln.rs the feature bits are reversed before calling NodeFeatures::from_le_bytes
        if let Some(node) = nodes.pop() {
            Ok(NodeInfo {
                pubkey: *node_id,
                alias: node.alias.unwrap_or(String::new()),
                features: node
                    .features
                    .clone()
                    .map_or(NodeFeatures::empty(), |mut f| {
                        // We need to reverse this given it has the CLN wire encoding which is BE
                        f.reverse();
                        NodeFeatures::from_le_bytes(f)
                    }),
            })

But in the new function for ClnNode the feature bits are passed directly into NodeFeatures::from_le_bytes.
        let features = if let Some(features) = our_features {
            NodeFeatures::from_le_bytes(features.node)
        } else {
            NodeFeatures::empty()
        };

        Ok(Self {
            client,
            info: NodeInfo {
                pubkey,
                features,
                alias,
            },
        })

@carlaKC have you tried using a default CLN node as a destination? Still wondering if this has something to do with my config.


Crontab mapping does nothing

Manual mapping
mscs map from console with regular and minecraft user works, but when automated in crontab
sudo crontab -e -u minecraft
* * * * * mscs map and * * * * * mscs map >/mc/overviewerlog.txt
(* * * * *) for testing purposes
Nothing happens
redirecting the output of the command to a log file writes this:
The cached copy of the version manifest is up to date.
Use the force-update option to ensure a new copy is downloaded.
Running Minecraft Overviewer mapping: spigotDotMoeError downloading the Minecraft client software.

And nothing in overviwer.log

As the README mentions for backups, you probably need to define HOME and PATH prior to the mapping command:
# Define HOME and PATH
HOME=/opt/mscs
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

# Run the mapping program
* * * * * mscs map

Let me know if that works for you.

I already did, this si how my crontab looks, everything works but the mapping
HOME=/mc/home
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
# m h  dom mon dow   command
# Define HOME and PATH
HOME=/mc/home
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

# Run mscs backups
55 1 * * * mscs send spigotDotMoe say En 5 minutos el servidor realizara una copia de seguridad, se reiniciara y actualizara el mapa web.
56 1 * * * mscs send spigotDotMoe say Seguramente tarde unos 5 min en estar activo otra vez.
0 2 * * * mscs backup
1 2 * * * mscs restart
2 2 * * * mscs map

i changed all of the references to /opt/mscs to /mc/home, and backup works, so i highly doubt its that

The download error has me wondering if that is part of the problem. Have you tried updating the script again today?  I made some changes that may fix that issue.
Otherwise, I'm guessing that it is a path issue.
Is overviewer.py accessible from one of those paths?
What is the output of which overviewer.py from your minecraft user?
What is the output of echo $PATH from your minecraft user?

Nevermind, i had a typo in the home directory that caused this, now everything is working allright, thanks for your help


[Question]: Mistral Endpoint error 422 status code (no body)

What is your question?
I have deployed Mistral endpoint as per LibreChat instructions but, when I try to use any mistral model I get this response:
Something went wrong. Here's the specific error message we encountered: Error: 422 status code (no body).
More Details
This is my librechat.yaml:
version: "3.4"
cache: true
endpoints:
custom:
# Example using Mistral AI API
- name: "Mistral"
apiKey: "my_api_key"
baseURL: "https://api.mistral.ai/v1"
models:
default: ["mistral-tiny", "mistral-small", "mistral-medium"]
titleConvo: true
titleModel: "mistral-tiny"
summarize: false
summaryModel: "mistral-tiny"
forcePrompt: false
modelDisplayLabel: "Mistral"
addParams:
safe_mode: true
# NOTE: For Mistral, it is necessary to drop the following parameters or you will encounter a 422 Error:
dropParams: ["stop", "user", "frequency_penalty", "presence_penalty"]
What is the main subject of your question?
Endpoints
Screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Updated the example to make Mistral AI API work::

We made the API stricter. We did not expect that change to break anything other than incorrect requests, but unfortunately we had not properly documented the safe_prompt flag, which was incorrectly named safe_mode in the documentation (fixed now) and implemented as such by some people. That’s on us - sorry about that.


bug: reth uses latest alloy interface but imports older version

Describe the bug
revm_interpreter is patched to commit a28a543 (corresponds to v5.0.0).
This is the latest release of revm, but doesn't include the recent changes to remove initCodes argument to the validate_initial_tx_gas() fn, so the function still expects 5 arguments.
But primitives/src/revm/compat.rs it has been updated since this recent commit to the latest revm interface, so it calls with four args..
This leads to compilation error when importing reth-primitives with alloy-compat feature
Steps to reproduce
reth_primitives = { git = "https://github.com/paradigmxyz/reth",  package = "reth-primitives", rev = "v0.2.0-beta.9", features = ["alloy-compat"] }

Node logs
No response
Platform(s)
Mac (Apple Silicon)
What version/commit are you on?
v0.2.0-beta.8 / v0.2.0-beta.9
What database version are you on?
N/A
Which chain / network are you on?
N/A
What type of node are you running?
Archive (default)
What prune config do you use, if any?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

we currently rely on revm patches which are the root cause of this
https://github.com/paradigmxyz/reth/blob/ad0be4ca06e9bcd4650ee57be88af7c7f74d66b3/Cargo.toml#L464-L468

Duplicate of #8713.


Usernames with spaces are allowed

It is good practice not to allow spaces in usernames

hey @stefanoskapa i would like to work on this, also is this project accepted in hacktoberfest?

Hey! Can I work on this? I like your project a lot. Thank you!

Welcome @namangirdhar16 har16 and @fuscati ! Unfortunately, this project doesn't participate in Hacktoberfest. But let me know if I can be of any help regarding the project structure, the current issue or any other question you might have. You can start right away by making your own fork and then cloning the project into your computer.  Also, I will soon provide some instructions in case you want to test-run the project locally.

@fuscati be aware that in java regex you have to use double backslash, as single backslash is an escape character for String as well


Add Show Mode

Codecov Report

Merging #12 (ef92e41) into main (5de45f9) will decrease coverage by 8.69%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   96.87%   88.18%   -8.70%
==========================================
  Files           3        3
  Lines          96      110      +14
==========================================
+ Hits           93       97       +4
- Misses          3       13      +10

Impacted Files
Coverage Δ

src/truthtable.jl
52.38% <28.57%> (-47.62%)
:arrow_down:

Continue to review full report at Codecov.

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


Commit LICENSE file (not just README)

Would you be willing to commit a dedicated LICENSE file (not just the README) noting the MIT license? Apologies for the request, but this is a requirement for some legal teams.

Hi, thanks for using this package.
I have two thoughts on this:

I want as many people as possible to be able to benefit from this code. That's why it's open source rather than closed. You're welcome to do anything with it that's helpful to you. Also, this package is a part of blackfriday, which isn't my project, so I wouldn't want to do anything that impedes others using blackfriday.

I really dislike polluting nice, clean, simple packages with ugly legalese boilerplate that gets in the way when actually doing useful things like developing, testing, running the code.

However, point 1 is higher priority than 2, so I will resolve this. But I wish there was a way to do that without compromising on point 2... :(

Thanks for your flexibility!

Thank you for doing this.  I know the license stuff is annoying,


Compiling fails with error

Hi, I'm just trying to compile, but I get error related to gatb. Any way around this?
 make
[  0%] Built target H5detect
[  0%] Built target H5make_libsettings
[ 68%] Built target hdf5
[ 68%] Built target hdf5_postbuild
[ 68%] Building CXX object ext/gatb-core/src/CMakeFiles/gatbcore-static.dir/gatb/bank/impl/Bank.cpp.o
In file included from /home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/bank/api/IBank.hpp:31,
                 from /home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/bank/impl/Bank.hpp:31,
                 from /home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/bank/impl/Bank.cpp:20:
/home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/tools/collections/api/Iterable.hpp: In member function ‘virtual Item* gatb::core::tools::collections::Iterable<Item>::getItems(Item*&)’:
/home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/tools/collections/api/Iterable.hpp:85: error: there are no arguments to ‘exit’ that depend on a template parameter, so a declaration of ‘exit’ must be available
/home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/tools/collections/api/Iterable.hpp:85: note: (if you use ‘-fpermissive’, G++ will accept your code, but allowing the use of an undeclared name is deprecated)
In file included from /home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/bank/api/IBank.hpp:31,
                 from /home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/bank/impl/Bank.hpp:31,
                 from /home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/bank/impl/Bank.cpp:20:
/home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/tools/collections/api/Iterable.hpp: In member function ‘virtual size_t gatb::core::tools::collections::Iterable<Item>::getItems(Item*&, size_t, size_t)’:
/home/smomw240/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/tools/collections/api/Iterable.hpp:97: error: there are no arguments to ‘exit’ that depend on a template parameter, so a declaration of ‘exit’ must be available
make[2]: *** [ext/gatb-core/src/CMakeFiles/gatbcore-static.dir/gatb/bank/impl/Bank.cpp.o] Error 1
make[1]: *** [ext/gatb-core/src/CMakeFiles/gatbcore-static.dir/all] Error 2
make: *** [all] Error 2

Hi,
Thanks for trying our ORNA. Can you please tell us which compiler version are you using and what is your system configuration.

The cluster is running Scientific Linux release 6.7, wich is based on Red Hat Enterprise Linux. GCC is ver. 7.2.0.

Hi,
So one workaround for this (as suggested by the authors of GATB also) is to manually change the CMakelist.txt of GATB.
In the file thirdparty/gatb-core/gatb-core/CmakeList.txt, in the line

set (LIBRARY_COMPILE_DEFINITIONS) add the parameter --fpermissive inside double quotes.
You can find this in line 134, 138,143 and 156
You can proceed by deleting the build directory, recreating it and then running hte following command inside the build folder:
cmake ..
make -j 10

Hope this helps

Thanks, but it still does not compile, but the error(s) changed. Now I get many pages of warnings, and at the end:
/home/user/temp/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/tools/collections/impl/IteratorFile.hpp:177: warning: invalid conversion from ‘gatb::core::system::IFile*’ to ‘int’
/home/user/temp/temp/ORNA/thirdparty/gatb-core/gatb-core/src/gatb/tools/collections/impl/IteratorFile.hpp:178: error: type ‘int’ argument given to ‘delete’, expected pointer
make[2]: *** [ext/gatb-core/src/CMakeFiles/gatbcore-static.dir/gatb/debruijn/impl/Graph.cpp.o] Error 1
make[1]: *** [ext/gatb-core/src/CMakeFiles/gatbcore-static.dir/all] Error 2
make: *** [all] Error 2

It's not urgent, I could compile on a different machine, this was just for testing as ORNA had another problem I'll report in a separate issue.


setup.py requires requests library

https://github.com/rbw0/pysnow/blob/master/setup.py#L69 pulls in requests as a pre-req and thus causes a chicken and the egg problem.
Please break out version into its own module that doesn't rely on an import path for requests.  This should allow users to freshly install package.

Hello,
Thank you for reporting this.
Fixed in 2b5d732
I'll release it along with some tests in a couple of days.


Add a configuration option to disable the notification

Now even I don't provide any notification endpoints, distribution will start a goroutine for broadcast. I think we should add a option to disable the notification, this should be better.

Thanks for the suggestion @AdoHe .  Goroutines are very inexpensive to create and I think an option to control this would just add confusion without much benefit.

@RichardScothern I understand that Goroutines are very inexpensive to create, for notifications implementation, it use event bridge, we create wrapper of the origin repository, for repository middleware, it is the same. If we have an option to disable the notification, we have no need to create so many events, we do need event notifications, but we just care some kinds of notification, especially in private docker registry of companies.

@AdoHe Is this causing a problem for you?
If this is just a code recommendation, please just submit a PR so we can be explicit about what you mean.

@stevvooe Ok, I will.


DOCS-1896: Update Kubernetes requirements for OSS 3.25 - 3.27

Product Version(s):

Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

We may want to be careful about bumping supported k8s versions as each Calico OSS/Enterprise are pinned to a specific k8s dependency. In normal cases, k8s client/server APIs are pretty compatible so we don't see issues. However, Calico own apiserver or felix/node components might subject to vendor Version Skew Policy constraints. We might not be able to claim supported even though our tests seem to just work.

One argument about the Version Skew Policy, current Calico already claim supports against n-2 k8s version
For example, for calico 3.26, it supports 1.24

v1.24
v1.25
v1.26
v1.27


[to #268] feat: Add CR test for static TRT linkage.

Nit: I think this pull request could be split into several elements.

Configure cu102 and relevant TensorRT version in Dockerfile
Static link TensorRT and hide non-blade symbols
Fix TensorRT getErrorRecorder usage
Fix PyTorch API usage of alloc_cpu.h

@fortianyou  Thanks for your reminders. Well I just intent to do 2 and add test for static trt linkage as 1. 3 and 4 should be in another PR...


[Question] COnfiguration issues with mlcommon benchmarking

Hi,
I Am trying to bringup the setup for multinode GPU Hugectr training benchmark using the code https://github.com/mlcommons/training_results_v3.0/tree/main/NVIDIA/benchmarks/dlrm_dcnv2/implementations/hugectr
For single node am able to run the benchmark test , but while am executing the multinode (say 2 node) am facing issue shown below , could you please help me resolving this issue??
[HCTR][17:28:15.456][WARNING][RK0][main]: The model name is not specified when creating the solver.
[1695144496.484294] [hpci5201:103648:0] ib_device.c:1250 UCX ERROR ibv_create_ah(dlid=49152 sl=0 port=1 src_path_bits=0 dgid=::ffff:<IP_ADDRESS> sgid_index=3 traffic_class=106) for UD verbs connect on bnxt_re0 failed: Connection timed out
[hpci5201:103648] pml_ucx.c:419 Error: ucp_ep_create(proc=1) failed: Endpoint timeout
[hpci5201:103648] pml_ucx.c:472 Error: Failed to resolve UCX endpoint for rank 1
Traceback (most recent call last):
File "/dev/shm/data/hugectl/train.py", line 344, in
model = hugectr.Model(solver, reader, optimizer)
RuntimeError: Runtime error: MPI_ERR_OTHER: known error not in list
MPI_Bcast(&seed, 1, (static_cast<MPI_Datatype> (static_cast<void *> (&(ompi_mpi_unsigned_long_long)))), 0, (static_cast<MPI_Comm> (static_cast<void *> (&(ompi_mpi_comm_world))))) at create (/workspace/dlrm/hugectr/HugeCTR/src/resource_managers/resource_manager_ext.cpp:39)

Hi @raghavendrachari08
[1695144496.484294] [hpci5201:103648:0] ib_device.c:1250 UCX ERROR ibv_create_ah(dlid=49152 sl=0 port=1 src_path_bits=0 dgid=::ffff:<IP_ADDRESS> sgid_index=3 traffic_class=106) for UD verbs connect on bnxt_re0 failed: Connection timed out
[hpci5201:103648] pml_ucx.c:419 Error: ucp_ep_create(proc=1) failed: Endpoint timeout
[hpci5201:103648] pml_ucx.c:472 Error: Failed to resolve UCX endpoint for rank 1

Looks to me the error is related with multinode MPI setting. Could you check your multinode MPI setting by running some demo code?

hello @raghavendrachari08 Did your dlrm mlcommon training on single node, go through successfully. I am coming from perspective where i am facing issues of  related to the error https://github.com/NVIDIA-Merlin/HugeCTR/issues/445 mentioned in previous link. How did you solve it. I am doing it on a single Nvidia DGX H100 node. Any help will be highly appreciated.
#445


Feature/9663 Password-Protected-PDF

Description
Add password entry and error handling for protected PDFs.
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-9663
Screenshots

You should disable suggestions for the keyboard I guess.

You should disable suggestions for the keyboard I guess.

Good point! I added password capabilities to the text field.


Image size instructions not working

I am currently not sure if this is a standard, haven't done any research on that yet but our VSTS Wiki supports the following image resizing instruction:
![name](files/some-file.png =20%x)

This causes some "malicious" error during the check.
These instructions should be removed and then passed to the check I think.

Thanks for pointing this out. As I'd like to start with the glob feature, would you mind taking care of this? Or don't you have the capacity to do so?

Because of weird debugging experiences (working here, not working there, WTF moments...) I found out that these kinds of image links were extracted by markdown-link-extractor until marked version 0.3.19.
So the hook in markdown-link-extractor to remove the instruction does not work anymore since version 0.4.0.
Because their goal is to align with the GFM/CommonMark standards I think they removed it intentionally.
https://github.com/markedjs/marked/issues/1279

I've published v3.5.4 which depends on the improved markdown-link-extractor. If this isn't solved, please re-open the issue.


Need update: Power-hibernate for RHEL-8

Currently some commands in SetupHbKernel.sh doesn't work in RHEL-8. I'm working on adding commands for RHEL-8.

Merged. Close it.


feature request: support for IExplorer

Hi, would it be possible to add support for iexplorer (at least IE11 +) (run for windows 7)
I have seen 2 things fail: assing and startsWith. I try several polyfill but it has not worked.
Thanks ;D

Also when using postcss autoprefixer's grid autoplacement feature, it complains about grid-template-rows not explicitly being given in lightpick.css.

Yes! i was just about committed to adding this to my app but with no IE support makes it a no go. W3Counter says 3.25% of users are using IE11, so 3 out of every 100 visitors unable to use the datepicker would not be good. Sure hope this can get fixed!

I would argue that it is a waste of effort to support any version of IE before Edge or any browser below 20% (think the 80/20 rule). These people will never change unless they are forced to. The only way to do that is to stop supporting them.
Now, IE11 is technically still a supported browser. So you can argue that it should be supported as a target. You can also argue that MS should fix it.
To argue the above points, I don't think it should be supported as a target because it does not comply with standards. MS deliberately chose to ignore standards or common practice and decided to make things up as they saw fit. The also decided not to implement fixes for those decisions
However MS did actually fix it. They released a new browser. So there really isn't an issue here... except for that fact the people have chosen not to upgrade. Those people will give you plenty of excuses as to why they aren't upgrading or switching, none which are valid.
That 3.25% (other sites say less then 2%) number is also very misleading. In fact most sites list the total IE/Edge ecosystem at right around that 3.25%.
There can easily be that much error in their results. That result is also a global result. Where are your visitors coming from? What do your analytics show? I've built websites for many corporations only to have them tell me 100% of the people accessing them have problems. I show that 80% of the web is Chrome and it works fine in Chrome. Well their user base was all their employees who were forced to use IE7 at the time.
Point being, the % of users using X browser means nothing when not looked at for the user base you are targeting and that of which are actually visiting your site.
So with all that being said, if I needed to support a specific old browser from a modern library... I would fork it and add in the changes myself which I think is the best approach here.
3 reasons for this...

If you ask the author or community to make changes to cover an extremely small percentage of users, and they add those changes, you have no guarantee those changes will stay there.
Someone needs to support those changes. To try and keep those changes supported and functional in the future is a great amount of work.
Supporting the old 3% can greatly inhibit the ability to provide a better experience for the 95%+ using modern browsers.

Providing a slightly better experience for some people is no equivalent trade to make the component completely unusable to other people.
3% is not a lot, but not insignificant either. Would you spend effort to be able to potentially get 3% more revenue? Depending on how big the project is, easily.
Also, by now we can easily say that everyone who has a little affinity to technology will take the time to install Chrome or Firefox over IE. The only people remaining are people really oblivious to technology, and, what one must not forget, people working at corporations with company policies limiting installations on their work PCs, only allowing IE to be used; these latter people tend to have not insignificant salaries, and not supporting IE, you lose potential revenue of bored employees happening to browse around and finding what would be an interesting online product and would have otherwise purchased it in the spur of the moment.
Anyway, either way, I'm pretty sure most people using this library are ones using it in proprietary projects for their company. And no matter how you feel IE11 is terrible, should die, and everyone using it should suffer as punishment, the truth is that web development companies tend to require IE11 support in their web projects (because they care about the additional 3% revenue), and not supporting IE11 is a dealbreaker for what library to choose to use.
I'm actually quite ashamed of not that this project isn't willing to support IE11, but that it was hiding the fact that it doesn't support it by not mentioning it anywhere in the docs. I realized too late that this project didn't, and the work I spent trying to integrate it into the project I was working on at work went to waste. Though this is mainly in my perspective, but frontend devs tend to assume that 3rd party libraries they use support IE11, because all major frameworks do, and almost all libraries do. I really went on a tangent here though.

Believe me, I wish IE11 didn’t exist. My life would be much easier. I want to use this on my reservation platform, since this looks like one of the easiest and most intuitive for even the most basic users to figure out. My customers can list their properties, and in turn their customers can make a reservation. But if 3 out of 100 users can’t make a reservation, like @amcsi said, that’s 3% lost revenue for my customers, literally. Any old person still running windows 7 with the bowser that was installed the day they bought their computer will not be able to make a reservation. So for me and probably many others looking for a good date range picker, it has to be all or nothing since we have no control over what browsers our customers use, or for that matter our customer’s customers. It would be different if just some of the features didn’t work on IE, but for it to just do nothing on IE is not acceptable at all. Like I said. This looks like a really good date range picker, it’s definitely the best one for selecting a date range when using 2 separate text boxes. Props to everybody who has contributed to this! This is just my 2 cents on how that 3% really is a big deal for a lot of people.


Lite: setting extras & fix CI

What does this PR do?

Fixes #<issue_number>
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

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

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

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

Did you have fun?
Make sure you had fun coding 🙃
cc @carmocca @akihironitta @borda

Ooops, we accidentally imported from pytorch_lightning.accelerators import TPUAccelerator instead of from Lite:

___________ ERROR collecting tests/tests_lite/strategies/test_xla.py ___________
strategies/test_xla.py:28: in <module>
    from pytorch_lightning.accelerators import TPUAccelerator
../../src/pytorch_lightning/__init__.py:35: in <module>
    from pytorch_lightning.callbacks import Callback  # noqa: E402
../../src/pytorch_lightning/callbacks/__init__.py:[22](https://github.com/Lightning-AI/lightning/actions/runs/3281889739/jobs/5404532607#step:11:23): in <module>
    from pytorch_lightning.callbacks.lr_finder import LearningRateFinder
../../src/pytorch_lightning/callbacks/lr_finder.py:[24](https://github.com/Lightning-AI/lightning/actions/runs/3281889739/jobs/5404532607#step:11:25): in <module>
    from pytorch_lightning.tuner.lr_finder import _LRFinder, lr_find
../../src/pytorch_lightning/tuner/lr_finder.py:38: in <module>
    from tqdm import tqdm
E   ModuleNotFoundError: No module named 'tqdm'


Ability to expand columns in dimension leaderboards to see the full text

Is your feature request related to a problem? Please describe.
Some dimension names contain long/large strings that get cut off on display. When drilling into the leaderboard, users would like the ability to expand the column width to see more of the text of the dimension name.
Describe the solution you'd like
Ability to expand dimension tables - specifically in the leaderboard drilldown (vs. Explore)
Describe alternatives you've considered
Tooltips are helpful but only one dimension at a time

Closed by #3213


getting "Session closed. Most likely the page has been closed." in many bbb-meetings

i am getting following error for some bbb-meetings bbb has meeting url
id: '900',
0|Recorder-APP  |   status: false,
0|Recorder-APP  |   state: 'Server Error, Something went wrong.',
0|Recorder-APP  |   url: null,
0|Recorder-APP  |   bucket_url: null,
0|Recorder-APP  |   error: Error: Protocol error (Runtime.callFunctionOn): Session closed. Most likely the page has been closed.
0|Recorder-APP  |       at CDPSession.send (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/Connection.js:180:29)
0|Recorder-APP  |       at ExecutionContext._evaluateInternal (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/ExecutionContext.js:107:44)
0|Recorder-APP  |       at ExecutionContext.evaluate (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/ExecutionContext.js:48:23)
0|Recorder-APP  |       at ExecutionContext. (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/helper.js:112:23)
0|Recorder-APP  |       at DOMWorld.evaluate (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/DOMWorld.js:112:20)
0|Recorder-APP  |     -- ASYNC --
0|Recorder-APP  |       at Frame. (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/helper.js:111:15)
0|Recorder-APP  |       at Page.evaluate (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/Page.js:860:43)
0|Recorder-APP  |       at Page. (/home/ubuntu/bbb-recording/node_modules/puppeteer/lib/helper.js:112:23)
0|Recorder-APP  |       at recordVideo (/home/ubuntu/bbb-recording/routes/index.js:212:16)

I think this is happening because of xvfb wasn't install or running.


MultipleChooserPanel does not work without explicit through table? Bug or Feature?

Issue Summary
I have a fairly simple setup with 3 models:
@register_snippet
class BlogCategory(index.Indexed, models.Model):
    """M2M Through model for blog categories"""

    blog = ParentalKey(
        "blogs.BlogPage",
        null=False,
        blank=False,
        on_delete=models.CASCADE,
        verbose_name=_("blog"),
        related_name="blog_categories",
    )

    name = models.CharField(verbose_name=_("name"), max_length=80)
    slug = models.SlugField(verbose_name=_("slug"), editable=False, max_length=120)

    panels = [
        FieldPanel("name"),
    ]

    search_fields = [
        index.AutocompleteField("name"),
    ]

    class Meta:
        ordering = ["name"]
        verbose_name = _("blog category")
        verbose_name_plural = _("blog categories")
        unique_together = (
            ("blog", "name"),
            ("blog", "slug"),
        )

    def __str__(self):
        return self.name

    def save(self, *args, **kwargs):
        if not self.slug:
            self.slug = slugify(self.name)
        return super().save(*args, **kwargs)

class BlogPage(BasePage):
    """Represents Blog with settings and dedicated categories"""

    object_type = "article"
    LAYOUT_CHOICES = (
        ("columns", _("Columns")),
        ("rows", _("Rows")),
        ("cards", _("Cards")),
        ("bricks", _("Bricks")),
    )

    layout = models.CharField(
        verbose_name=_("layout"),
        max_length=32,
        choices=LAYOUT_CHOICES,
        default="columns",
        blank=False,
    )

    description = RichTextField(verbose_name=_("description"), features=["link", "bold", "italic"], blank=True)

    # Search
    search_fields = BasePage.search_fields + [
        index.SearchField("description"),
    ]

    # Panels
    content_panels = BasePage.content_panels + [
        FieldPanel("description"),
        FieldPanel("layout"),
        # InlinePanel("blog_categories", label=_("Categories")),
        MultipleChooserPanel("blog_categories", label=_("Categories"), chooser_field_name="id"),
    ]

    subpage_types = ["blogs.EntryPage"]

    class Meta:
        verbose_name = _("blog")
        verbose_name_plural = _("blogs")

class EntryPage(BasePage):
    object_type = "article"

    FORMAT_REGULAR = "regular"
    FORMAT_LINK = "link"
    FORMAT_CHOICES = (
        (FORMAT_REGULAR, _("Regular")),
        (FORMAT_LINK, _("Link")),
    )

    format = models.CharField(
        verbose_name=_("format"),
        max_length=32,
        choices=FORMAT_CHOICES,
        default="regular",
        blank=False,
        help_text=_(
            "An Entry Format is a piece of meta information that is used to customize its presentation of an entry."
        ),
    )

    body = StreamField(
        CONTENT_STREAMBLOCKS,
        verbose_name=_("body"),
        use_json_field=True,
    )

    excerpt = models.TextField(
        verbose_name=_("excerpt"),
        blank=True,
        max_length=500,
        help_text=_(
            "Entry excerpt to be displayed on entries list. "
            "If this field is not filled, a truncate version of body text will be used."
        ),
    )

    featured = models.BooleanField(
        verbose_name=_("featured post"),
        default=False,
        help_text=_("Featured entries are displayed above the list of entries."),
    )

    date = models.DateTimeField(verbose_name=_("Post date"), default=timezone.now)

    categories = ParentalManyToManyField("blogs.BlogCategory", blank=True)
    tags = ClusterTaggableManager(through="blogs.TaggedEntryPage", blank=True)

    # Search
    search_fields = BasePage.search_fields + [
        index.SearchField("body"),
        index.SearchField("excerpt"),
        index.FilterField("featured"),
        index.FilterField("format"),
    ]

    # Panels
    content_panels = BasePage.content_panels + [
        FieldPanel("body"),
        FieldPanel("excerpt"),
        FieldPanel("format"),
        FieldPanel("tags"),
        FieldPanel("categories", widget=forms.CheckboxSelectMultiple),
        # Cannot make it work here
        # MultipleChooserPanel(
        #     "blog_categories",
        #     heading="Categories",
        #     label="Category",
        #     min_num=1,
        #     chooser_field_name="blog",
        # ),
    ]

Issues:

BlogPage edit form contains Add categories button, but once clicked the following error appears in console:

Uncaught TypeError: this.chooserWidgetFactory.openModal is not a function
    at HTMLButtonElement.<anonymous> (page-editor.js?v=dfe05431:1:4815)

The only way I could make it work is by changing BlogCategory model to explicit through m2m model as follows:
class Category(index.Indexed, models.Model):
    name = models.CharField(verbose_name=_("name"), max_length=80)
    slug = models.SlugField(verbose_name=_("slug"), editable=False, max_length=120)

class BlogCategory(index.Indexed, models.Model):
    """M2M Through model for blog categories"""

    blog = ParentalKey(
        "blogs.BlogPage",
        null=False,
        blank=False,
        on_delete=models.CASCADE,
        verbose_name=_("blog"),
        related_name="blog_categories",
    )
    category = ForeignKey('blogs.Category', ....)
    ...

The other problem is with the EntryPage.categories field where I cannot make the panel work without explicit through table either. The only way is to define an intermediate table that links BlogCategory and EntryPage such as BlogEntryPageCategory.
Is that desired behavior or I'm missing something? InlinePanel did not have such a requirement.
Technical details

Django version: 4.19
Wagtail version: 5.0.1

InlinePanel did not have such a requirement

Are you sure? InlinePanel was only ever designed to work against one-to-many ParentalKey relations, not ParentalManyToMany - if that does work, it's only by accident.


flush the stalled list after the window is updated

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Issue with k8s.io/docs/tutorials/kubernetes-basics/update/update-intro/

This is a Bug Report

Problem:
I went to the page that supposedly says how to perform a rolling update, but it did not say how to do so.
Proposed Solution:
I'm pretty sure (to do it declaratively)  you just update the deployment's yaml somehow and run kubectl apply -f.  The page should explain that the following interactive tutorial shows how to do it manually (using kubectl set image) but it's better to do this declaratively.
Page to Update:
https://kubernetes.io/...

You can edit the yaml with kubectl edit deployment <name>
The command will automatically apply the changes if there is no errors.

You can edit the yaml with kubectl edit deployment <name>
The command will automatically apply the changes if there is no errors.

Yes thanks I am aware of that.  I guess I should have been more explicit that I am reporting a bug in the documentation.

/language en


If multi class in one image can be trained by FLORA?

I noticed the data structure should be divided in class1 folder or class2, but if an image  has different classes. How to train it?
Expect for you reply :)

Sorry for the late reply.
We do not have any restriction on the folder structure of the dataset. The structure mentioned in the README is the format related to ImageNet dataset. If your dataset have your own layout, you can create your own implementation dataset, then it might work.


[US] As a user, I want to retrieve an attestation from the registry

As a user,
I want to retrieve an attestation from the registry,
So that I can see an attestation directly from source

Description
This will require a few steps:

Retrieve the attestation using the getAttestation(bytes32 attestationId) method from the AttestationRegistry
Get the respective schema from the SchemaRegistry using the getSchema(bytes32 schemaId) with the schema id from the attestation metadata
Parse the schema string and use this to abi.decode the attestation data
Create an object from the values in the decoded data and the field names in the schema string
Optionally retrieve the portal information from the portal registry as qualifying information for the attestation

NOTE: initially we may be able to leverage a lot of this from the indexer and / or subgraph, which could do a lot of the parsing for us.  Whether this is feasible is TBD.

The PR have submitted. The link is: https://github.com/Consensys/linea-attestation-registry/pull/339


SWARM-1333: Openshift service address not provided

[x] Have you followed the guidelines in our Contributing document?
[x] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?

Motivation:
The IP address of services is not available through the topology-openshift.
Changes:
Update the openshift-rest-client library to 5.5.0 and exposes the service ClusterIP
Results:
Clients can retrieve full qualified service addresses through the topology openshift

On hold until we get a response to https://github.com/openshift/openshift-restclient-java/issues/271

@michalszynkiewicz This would update openshift-rest-client to 5.5.0. Would this be a problem?

There's a 5.6.0 out there, too.

@heiko-braun 5.5.0 (and 5.6.0) should be ok from prod perspective.

ok, i'd suggest we move straight to 5.6.0 in this case.


Use rawgit.com for getting started code

From https://rawgit.com/faq#why

Why is this necessary? Can't I just load files from GitHub directly?

When you request a file from raw.githubusercontent.com or gist.githubusercontent.com, GitHub usually serves it (in the case of JavaScript, HTML, CSS, and some other file types) with a Content-Type of text/plain. As a result, most modern browsers won't actually interpret it as JavaScript, HTML, or CSS.

They do this because serving raw files from a git repo is relatively inefficient, so they want to discourage people from using their GitHub repos for static file hosting.

RawGit acts as a caching proxy, forwarding requests to GitHub, caching the responses either for a short time (in the case of rawgit.com URLs) or permanently (in the case of cdn.rawgit.com URLs), and relaying them to your browser with the correct Content-Type headers.

The caching layer ensures that minimal load is placed on GitHub, and you get quick and easy static file hosting right from a GitHub repo. Everyone's happy!

:beers: for this lib, it is exactly what I needed to hack on some stuff today :smile:

@fgrehm  yeah great idea :+1:


MINOR: Revert incompatible behavior change to consumer reset tool

This patch reverts the removal of the --execute option in the offset reset tool and the change to the default behavior when no options were present. These changes were introduced in KIP-171, but they are not compatible with previous releases. We should at least give users a deprecation window for incompatible changes.
Test cases were not actually validating that offsets were committed when the --execute option was present, so I have fixed that and added basic assertions for the dry-run behavior. I also removed some duplicated test boilerplate.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

cc @guozhangwang @mjsax
This patch only fixes the consumer reset tool. I can do a separate patch for the streams reset tool if we are satisfied with the solution and we want to keep consistent options.

@hachikuji if we only change the consumer group command would that break the streams reset tool which is passing the parameters (dry-run or not) directly to the consumer group command?

cc @dguy We'll need this for 1.1 depending whether we all agree on reverting the change.

@hachikuji no probs - let me know

@guozhangwang

Just a meta question about its impact to the consumer reset and streams reset tool.

I think it has no impact. StreamsResetter does not use ConsumerGroupCommand but implements all logic natively. With reverting this change, it seems we only don't get "alignment" of used options, but both parts of the code seems to be independent.

I think it has no impact. StreamsResetter does not use ConsumerGroupCommand but implements all logic natively. With reverting this change, it seems we only don't get "alignment" of used options, but both parts of the code seems to be independent.

Yup, synced with @hachikuji this morning and realized it

@hachikuji if we only change the consumer group command would that break the streams reset tool which is passing the parameters (dry-run or not) directly to the consumer group command? I.e. the streams reset tool would not have execute option, and even if dry-run is not specified, it will still be a dry-run, so the execution will never happen.

Never mind, synced with @hachikuji offline and it was not an issue.

@mjsax @guozhangwang I've updated StreamsResetter for consistency. I noticed a couple apparent bugs when I was running test cases, but I may have misunderstood the logic, so please check.

What was the decision w.r.t getting this into 1.1?

@dguy Yes, planning to get this one in. I was just discussing offline with @mjsax and @guozhangwang to make sure we have the behavior we want right. I will submit an update shortly.


Error installing through PyPI

A dependency with cdmcfparser seems to be the root of the problem
Collecting inspect4py
  Downloading inspect4py-0.0.1-py3-none-any.whl (54 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 54.3/54.3 KB 367.0 kB/s eta 0:00:00
Collecting configparser
  Downloading configparser-5.2.0-py3-none-any.whl (19 kB)
Collecting setuptools==54.2.0
  Downloading setuptools-54.2.0-py3-none-any.whl (785 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 785.1/785.1 KB 2.3 MB/s eta 0:00:00
Collecting graphviz
  Downloading graphviz-0.19.1-py3-none-any.whl (46 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 46.3/46.3 KB 1.3 MB/s eta 0:00:00
Requirement already satisfied: click in ./env-scc/lib/python3.10/site-packages (from inspect4py) (7.0)
Collecting pigar
  Downloading pigar-0.10.0-py2.py3-none-any.whl (6.9 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 6.9/6.9 MB 5.1 MB/s eta 0:00:00
Collecting json2html
  Downloading json2html-1.3.0.tar.gz (7.0 kB)
  Preparing metadata (setup.py) ... done
Collecting astor
  Downloading astor-0.8.1-py2.py3-none-any.whl (27 kB)
Collecting docstring-parser==0.7
  Downloading docstring_parser-0.7.tar.gz (13 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting cdmcfparser
  Downloading cdmcfparser-2.4.0.tar.gz (117 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 117.3/117.3 KB 2.5 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
  Downloading cdmcfparser-2.3.2.tar.gz (116 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 116.6/116.6 KB 3.2 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
  Downloading cdmcfparser-2.3.1.tar.gz (115 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 115.7/115.7 KB 2.1 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
  Downloading cdmcfparser-2.3.0.tar.gz (115 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 115.7/115.7 KB 93.0 kB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
  Downloading cdmcfparser-2.2.3.tar.gz (115 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 115.5/115.5 KB 1.9 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
Collecting nbformat>=4.4.0
  Downloading nbformat-5.2.0-py3-none-any.whl (74 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 74.6/74.6 KB 1.7 MB/s eta 0:00:00
Requirement already satisfied: requests>=2.20.0 in ./env-scc/lib/python3.10/site-packages (from pigar->inspect4py) (2.27.1)
Collecting colorama>=0.3.9
  Using cached colorama-0.4.4-py2.py3-none-any.whl (16 kB)
Collecting traitlets>=4.1
  Using cached traitlets-5.1.1-py3-none-any.whl (102 kB)
Collecting jupyter-core
  Downloading jupyter_core-4.9.2-py3-none-any.whl (86 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 86.9/86.9 KB 1.8 MB/s eta 0:00:00
Collecting jsonschema!=2.5.0,>=2.4
  Downloading jsonschema-4.4.0-py3-none-any.whl (72 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 72.7/72.7 KB 1.7 MB/s eta 0:00:00
Requirement already satisfied: charset-normalizer~=2.0.0 in ./env-scc/lib/python3.10/site-packages (from requests>=2.20.0->pigar->inspect4py) (2.0.12)
Requirement already satisfied: idna<4,>=2.5 in ./env-scc/lib/python3.10/site-packages (from requests>=2.20.0->pigar->inspect4py) (3.3)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in ./env-scc/lib/python3.10/site-packages (from requests>=2.20.0->pigar->inspect4py) (1.26.8)
Requirement already satisfied: certifi>=2017.4.17 in ./env-scc/lib/python3.10/site-packages (from requests>=2.20.0->pigar->inspect4py) (2021.10.8)
Collecting attrs>=17.4.0
  Using cached attrs-21.4.0-py2.py3-none-any.whl (60 kB)
Collecting pyrsistent!=0.17.0,!=0.17.1,!=0.17.2,>=0.14.0
  Using cached pyrsistent-0.18.1-cp310-cp310-macosx_10_9_universal2.whl (81 kB)
Building wheels for collected packages: docstring-parser, cdmcfparser, json2html
  Building wheel for docstring-parser (pyproject.toml) ... done
  Created wheel for docstring-parser: filename=docstring_parser-0.7-py3-none-any.whl size=19109 sha256=8a0f5b516fb9ed5156a3ad1375a9f19a4307be3cf25491bea5c8e2aac081b36a
  Stored in directory: /Users/dakixr/Library/Caches/pip/wheels/06/08/0b/69b09c4b02dfe4b82ed139ea8208d32cb28a0eab1bb91004a8
  Building wheel for cdmcfparser (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py bdist_wheel did not run successfully.
  │ exit code: 1
  ╰─> [14 lines of output]
      pypandoc package is not installed: the markdown README.md convertion to rst failed: No module named 'pypandoc'
      running bdist_wheel
      running build
      running build_py
      file cdmcfparser.py (for module cdmcfparser) not found
      file cdmcfparser.py (for module cdmcfparser) not found
      running build_ext
      building 'cdmcfparser' extension
      In file included from src/cflowfragments.cpp:26:
      src/cflowfragments.hpp:26:10: fatal error: 'node.h' file not found
      #include <node.h>
               ^~~~~~~~
      1 error generated.
      error: command '/usr/bin/clang' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for cdmcfparser
  Running setup.py clean for cdmcfparser
  Building wheel for json2html (setup.py) ... done
  Created wheel for json2html: filename=json2html-1.3.0-py3-none-any.whl size=7609 sha256=68b089b07f03f7a14e8e9b607f52d06c4449ad6759c0a2743ad4ddab3c91bfe1
  Stored in directory: /Users/dakixr/Library/Caches/pip/wheels/e0/d8/b3/6f83a04ab0ec00e691de794d108286bb0f8bcdf4ade19afb57
Successfully built docstring-parser json2html
Failed to build cdmcfparser
Installing collected packages: json2html, cdmcfparser, traitlets, setuptools, pyrsistent, graphviz, docstring-parser, configparser, colorama, attrs, astor, jupyter-core, jsonschema, nbformat, pigar, inspect4py
  Running setup.py install for cdmcfparser ... error
  error: subprocess-exited-with-error

  × Running setup.py install for cdmcfparser did not run successfully.
  │ exit code: 1
  ╰─> [16 lines of output]
      pypandoc package is not installed: the markdown README.md convertion to rst failed: No module named 'pypandoc'
      running install
      /Users/dakixr/Desktop/github/scc/env-scc/lib/python3.10/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
        warnings.warn(
      running build
      running build_py
      file cdmcfparser.py (for module cdmcfparser) not found
      file cdmcfparser.py (for module cdmcfparser) not found
      running build_ext
      building 'cdmcfparser' extension
      In file included from src/cflowfragments.cpp:26:
      src/cflowfragments.hpp:26:10: fatal error: 'node.h' file not found
      #include <node.h>
               ^~~~~~~~
      1 error generated.
      error: command '/usr/bin/clang' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: legacy-install-failure

× Encountered error while trying to install package.
╰─> cdmcfparser

note: This is an issue with the package mentioned above, not pip.
hint: See above for output from the failure.

@dakixr,
which OS are you using? I had no issues in Unix, and we tested in macOS too.

Also, which Python version are you running this in? We need to update a little our dependencies for Python >3.9.

We have problems with control flow in Python > 3.9, this is a known issue. I need to test the libraries that have just been released.
Workarounds:

Use Docker.
Set up an environment with Python 3.7.
Install using pip install -e . and do not use the -cf option

Fixed now.


Random background image around card

There appears to be a random shadow image around the credit card.

This may be related to #12

Thanks for pointing that out - didn't notice that yet!
When you set the text field's background color, this color will currently also be set for the image view's background. The image view itself got a shadow in #35 which is what you see here as the shadow is drawn below this white layer.
This gets problematic, especially when the view constraints are changed to inset the image view ( #12 ), as the image view's size changes. I'll fix it asap.
Thanks!


chore: update searching STAC catalog tutorial

Updates "searching the STAC catalog" tutorial

Removes a lot of the hardcoded values, but adds back in a simple example at the bottom
Updates some of the language used around MAAP STAC

Added a couple comments under the notebook in ReviewNB 🙂

Oh yeah! Should we add the "Run This Notebook" section? I think we included it in the few other technical tutorial updates we've done so far

Oh yeah! Should we add the "Run This Notebook" section? I think we included it in the few other technical tutorial updates we've done so far

I thought about it, should we add it? It doesn't need to be ran within the ADE since it's using the STAC catalog

Cool! I think that should be fine then if it's not included


Strange role referenced in notification to user

[ ] Add to an appropriate Epic.

Tester (if different from issue creator):
Site: hcommons-dev.org
URL:
Test Account: hctester40
Area (CORE, sites, profiles, groups, general): groups
If issue discovered on dev, does it also exist on prod? no
Describe the issue
On joining a group the user receives a notification titled "You've been added to the group site "Shelley Appreciation Group" with the role of bbp_participant."
When joining groups on prod there doesn't seem to be this kind of notification.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Optional details, if they seem relevant
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Retest to see if this is fixed with re-sync

This is still funky. Now there's no role at all.


Update strings.js

@telis93 can you review please?

BTW @teldosas is my new account :wink:

Thanks a lot @teldosas! I have made the changes.

Thanks!


Add pi compute job example

Add an example that demonstrates:

Creating a job to compute pi with a program called y-cruncher
Writing the output from the job to a volume
Exposing the volume content using a Minio server

A Dockerfile is also included to make the example fully reproducible.
Document the example on an appropriate level to make it easier to
understand for those new to Kubernetes and OpenShift.

Looks good to me! Did a test deployment on Rahti following the instructions provided in the documentation, and everything went smoothly.


Viewport: fix 'size' binding from Rect2 to Vector2

Brought to attention by @letheed here.

Can't we detect such mistakes by checking the return type of the getter / param type of the setter?


Query parameter is double encoded

A query parameter with GET method is double encoded.  It converts "あ" into "%25E3%2581%2582", it should be "%E3%81%82".
query property of NSURLComponents will be encoded automatically when getting to complete NSURL. You can use NSURLComponents.percentEncodedQuery or just use NSURLComponents and NSURLQueryItem for query encoding.

Thank you for reporting! Could you check if this issue is fixed in @a3d9621?

Thanks! It is fine for iOS. But unable to build for Mac since NSURLQueryItem needs OS X 10.10.
(Sorry, I have mentioned NSURLQueryItem has no support for OS X 10.9.)
APIKit/APIKit/Request.swift:84:28: error: 'NSURLQueryItem' is only available on OS X 10.10 or newer

Ah, I've completely forgotten! This may be fixed in @d989ce8.
Please review #60 again.

Both the platform is Okay. Thank you for fixing.


Occassional Text Cutoff for "Show similar notes on top"

Occassionally, when the "Show similar notes on top" setting is enabled, the text will get cut off. This same occurs even when I am in source mode.
However, this does not occur all the time as seen here:

This problem seems to be caused by the "Banner" plugin. After disabling it, the spacing is fixed.

i tested with Banner plugin , but there was no cut off, do you have any css snippets?
I have added a minimum height in the new release so maybe it will be fixed for you

Looks like the minimum height fixed the problem. Thanks!


Grouping ComponentMetrics events by project UUID sometimes fails due to NullPointerExceptions

Occasionally, the metrics service crashes due to exceptions like the following:
2023-03-12 13:36:21,229 ERROR [org.apa.kaf.str.pro.int.TaskExecutor] (hyades-metrics-service-95e2d5ce-1f6d-455c-ae0d-6fa63b4f1fdb-StreamThread-2) stream-thread [hyades-metrics-service-95e2d5ce-1f6d-455c-ae0d-6fa63b4f1fdb-StreamThread-2] Failed to process stream task 0_1 due to the following error:: org.apache.kafka.streams.errors.StreamsException: Exception caught in process. taskId=0_1, processor=consume_from_dtrack.metrics.component_topic, topic=dtrack.metrics.component, partition=1, offset=3, stacktrace=java.lang.NullPointerException
        at org.hyades.MetricsTopology.lambda$metricsTopology$0(MetricsTopology.java:71)
        at org.apache.kafka.streams.kstream.internals.KStreamImpl.lambda$internalSelectKey$0(KStreamImpl.java:243)
        at org.apache.kafka.streams.kstream.internals.KStreamMap$KStreamMapProcessor.process(KStreamMap.java:46)
        at org.apache.kafka.streams.processor.internals.ProcessorNode.process(ProcessorNode.java:157)
        at org.apache.kafka.streams.processor.internals.ProcessorContextImpl.forwardInternal(ProcessorContextImpl.java:290)
        at org.apache.kafka.streams.processor.internals.ProcessorContextImpl.forward(ProcessorContextImpl.java:269)
        at org.apache.kafka.streams.processor.internals.ProcessorContextImpl.forward(ProcessorContextImpl.java:228)
        ...

The reported line number 71 corresponds to this part of the code:
https://github.com/DependencyTrack/hyades/blob/03bc9de84e92b68b3c6d0af6745fc6320dbe3963/metrics-service/src/main/java/org/hyades/MetricsTopology.java#L70-L73
While I can't say for sure which field is null here, I suspect it's getProject that returns null. That would indicate that the API server, when serializing the ComponentMetrics object, did not load the project field.
Probably related to #369, as we're currently sending too much data anyway, and we do not have sufficient control over the content of the events.

Searching in the Redpanda console reveals that it is indeed one occurrence where the project field(s) are populated with solely the active field.

So instead of project being null, it's the uuid. No idea why this is happening, all ~10k events in my case are for the same project, and only one of them has this problem.
I used bloated.bom.json, and this is not happening every time.


Role command is broken, someone fix.

https://github.com/AnIdiotsGuide/discordjs-bot-guide/blob/master/understanding/roles.md


path_1.default.join is not a function

typeError: path_1.default.join is not a function
at main.ts:9:40
at node_modules/utimes/dist/main.js (main.ts:19:1)
这是什么问题？我不知道怎么解决

Hi, thanks for reporting. The issue is because the package is being imported in a browser environment, and it's trying to resolve the native binding on the file system. Can you tell me what bundler or framework you're using, so I can reproduce and fix it? To clarify, this package will not work in a browser even after I fix the error. It must be run from the Node.js server or desktop side.
你好，感谢您的反馈。问题是因为您在浏览器环境中导入了这个包，它试图在文件系统中解析原生绑定。您能告诉我您使用的是哪个打包工具或框架，这样我可以复现并修复它吗？为了澄清，即使我修复了这个错误，这个包也不会在浏览器中工作。它必须在Node.js服务器或桌面端运行。

谢谢你的回答
我用  const { utimes } = require('utimes')   这样就可以使用了
我用的是 electron-egg  + vue3
vue3 项目里面使用的

I understand, thank you for the explanation.
Electron has two environments: the main process and the renderer process. The main process is for desktop application functionalities (node.js), and the renderer process is for the browser interface (vue).
It seems like you are invoking the utimes package in the renderer process. However, since this package is designed for the file system, you should use it in the main process. Please make sure to call this package in the main process of Electron to avoid issues in the browser environment.
I've released a new version (v5.2.0) which fixes the error you encountered. However, unless you call the functions from the main process (and not the Vue side of things), it will not be able to update any timestamps, and will silently fail.
明白了，感谢您的解释。Electron有两种环境：主进程和渲染进程。主进程用于桌面应用的功能，而渲染进程用于浏览器界面。您现在似乎在渲染进程中调用了utimes包。但实际上，由于这个包是为文件系统设计的，您应该在主进程中使用它。请确保在Electron的主进程中调用此包，这样您可以避免浏览器环境中的问题。
我已经发布了一个新版本（v5.2.0），修复了您遇到的错误。但是，除非您从主进程调用这些函数（而不是从Vue那边），否则它将无法更新任何时间戳，并且会静默失败。


Allow "auto" for gradient clipping in YAML

What does this PR do?

If we have gradient_clipping: auto in the DeepSpeed config inside an Accelerate config, the code currently fails with:
Traceback (most recent call last):
  File "/root/workspace/lora/tmp.py", line 3, in <module>
    args = TrainingArguments(
  File "<string>", line 124, in __init__
  File "/usr/local/lib/python3.10/dist-packages/transformers/training_args.py", line 1828, in __post_init__
    self.deepspeed_plugin = DeepSpeedPlugin()
  File "<string>", line 14, in __init__
  File "/usr/local/lib/python3.10/dist-packages/accelerate/utils/dataclasses.py", line 727, in __post_init__
    self.gradient_clipping = float(gradient_clipping)
ValueError: could not convert string to float: 'auto'

This can be reproduced running accelerate launch --config_file my_config.yaml script.py with script.py being
from transformers import TrainingArguments, Trainer, AutoModel

args = TrainingArguments(
    output_dir="/tmp/test",
    max_grad_norm=0.5,
)

trainer = Trainer(
    model=AutoModel.from_pretrained("bert-base-uncased"),
    args=args,
)

and my_config.yaml is
compute_environment: LOCAL_MACHINE
debug: false
deepspeed_config:
  gradient_accumulation_steps: 1
  gradient_clipping: auto
  offload_optimizer_device: none
  offload_param_device: none
  zero3_init_flag: true
  zero3_save_16bit_model: true
  zero_stage: 3
distributed_type: DEEPSPEED
downcast_bf16: 'no'
machine_rank: 0
main_training_function: main
mixed_precision: bf16
num_machines: 1
num_processes: 8
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

This PR fixes this issue by checking if the given value can actually be casted to float.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Add docs for Ubuntu 16.04 build dependencies

Additional information on how to build on Ubuntu 16.04 using Pyenv and
Python 3.4.4. FIXES: DCOS-254

🚢 . It'll be nice when we're on 3.5...


UI post-configuration should not require authentication for /Account/Logout

Logout.cshtml and Logout.cshtml.cs are used before and after the user is logged out. If user posts to the Logout page without a returnUrl value, the page displays a message: You have successfully logged out of the application. Unfortunately, the page is unreachable after logging out (i.e. requires authentication) due to the following line in the generated IdentityDefaultUIConfigureOptions.PostConfigure(...): https://github.com/aspnet/Identity/blob/394e4746b89485e6f6d0736b363d947361e7a378/src/UI/IdentityDefaultUIConfigureOptions.cs#L38
Because this setting is done in an IPostConfigureOptions<CookieAuthenticationOptions> implementation, it is extra hard for users to override.
See also aspnet/Scaffolding#734

Found while validating aspnet/Release#265


Add integration tests

Codecov Report

Merging #149 (64e6531) into main (22695d5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #149   +/-   ##
=======================================
  Coverage   85.16%   85.16%
=======================================
  Files          22       22
  Lines         209      209
  Branches       33       33
=======================================
  Hits          178      178
  Misses         31       31

Impacted Files
Coverage Δ

components/Form/index.tsx
60.86% <ø> (ø)

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 22695d5...64e6531. Read the comment docs.


fix: update for vue3 v-model breaking changes

https://v3.vuejs.org/guide/migration/v-model.html#overview

Hi @RyanNorooz!
Thanks for putting this PR together. The primary consumer of this package (Octo) is still on Vue 2 at the moment, so I can't update this to Vue 3 just yet. If you need Vue 3 compatibility in the meantime, you might be able to use a shim.
I'm in the process of migrating Octo to Vite this week, and then I'll be starting on the Vue 3 upgrade.

This component has been upgraded to Vue 3 as part of a bigger change, so this PR is no longer necessary.


"Add Prompt flow Code" won't format image input properly

Describe the bug
When right-clicking "Add Prompt flow Code" on a prompty file, if a markdown image is referenced in the prompt e.g.
...
sample:
  input_image: ./images/test_image.jpg
...
user:
![image]({{input_image}})

then the generated code will be:
...
json_input = '''{
  "input_image": "./images/test_image.jpg"
}'''
...

The image path string won't be converted into an image because the data representation and mime types are missing and mandatory. But adding them as an object following the documented format won't work either because the prompt flow conversion function only parses strings and uses a regex to extract the types.
As a result, the model answers that it cannot read the image, or worse, answers making a guess based on the filename :)
Workaround
Add the data representation and mime types as prefix in the image path string:
...
json_input = '''{
  "input_image": "data:image/jpg;path:./images/test_image.jpg"
}'''
...

Environment Information

Prompty extension version: v0.1.2024060511
Operating System: Windows 11
Python version : 3.11.9
VS Code version: 1.91.1
Prompt Flow extension version: v1.20.1
LLM: Azure gpt-4o v2024-05-13

Sorry this took a bit - I released a version of a standalone prompty runtime that addresses this issue. Let me know if this works!

Thanks, the image is now converted with the standalone runtime.
I couldn't get the CLI to work though (term not recognized in pwsh).

Yeah - it was a tad bit broken with the last release. It should work now.

Yeah - it was a tad bit broken with the last release. It should work now.

As of version 0.1.15 the prompty CLI term is still not recognized :/

OK! This has been fixed


fix(animation): animate chart when call chart.animate(true) (#3268)

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] commit message follows commit guidelines

Description of change

fix(#3268)

Pull Request Test Coverage Report for Build 549996909

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 93.578%

Files with Coverage Reduction
New Missed Lines
%

src/geometry/base.ts
4
94.74%

Totals

Change from base Build 547720557:
0.05%

Covered Lines:
10858

Relevant Lines:
11256

💛  - Coveralls

为啥我在本地单测成功，但是 CI test 失败？


Added signature method implementation.

Implementation according to the Nexmo signing documentation:
https://docs.nexmo.com/messaging/signing-messages
Supporting MD5 hash and HMAC-SHA1/256/512.

Hey @trancee - thanks for the PR! I've had a quick look, and it looks good. I just need to test it manually before merging - I'll try to get that done in the next week.
Just realised we don't have a contributors file for this project - would you like to be added to it when I create it?

Hi @judy2k. I am glad it looks good so far. Feel free to test it out manually. I have already used it in another project. I also noticed that we could add signatures to outgoing requests, e.g. in the send_message method. If you want me to create another PR for that purpose, I will.
Sure, you may add me in the contributors file if you create one.

Resolved with #60 (which was downstream of this PR)


[Rokfin] Search & login

Checklist

[X] I'm reporting a site feature request
[ ] I've verified that I'm running yt-dlp version 2022.03.08. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Example URLs
https://www.rokfin.com/discover
Description
See title.
Verbose log
Not applicable.

PR

This is not a properly filed issue. But idk why you even opened it, you should've just made the PR 🤷


AutomatedTestReport: hmtxThe left sidebearing 34 does not match the glyph's xMin 35 (glyph 418)

I copied the text below from an email by Jelle.B who looked at regular Ethiopic font (see the otf file in the attached zip) generated by the pipeline from the glyphs file (see the attached zip) on about 1/Feb/2016. The comments pertain to that version of the pipeline. See the bottom of this message for the reason why I cannot generate a new version of the fonts.
'Of the errors listed by our test tool I mention:
hmtxThe left sidebearing 34 does not match the glyph's xMin 35 (glyph 418)
13 more like this'
NotoSansEthiopicCondensed-Regular-km-error.zip
Note that NotoSansEthiopic-MM.glyphs generates as of 12/Feb/2016 "defcon.errors.DefconError: The kerning data is not valid" so I cannot check what the newest version of pipeline generates (see https://github.com/googlei18n/fontmake/issues/13 )

We should track down where the discrepancy is coming from.

I believe this has to do with this ufo2ft issue https://github.com/googlei18n/ufo2ft/issues/106 which should be fixed now. Please reopen if this is not the case.

Verified as fixed

@anthrotype : there's nothing to do; I just assigned it to you to give you credit for the investigation (in the future if you do any work please add yourself to assignees :-))


Subdomain shows default certificate

Hello,
I've successfully created a certificate for my subdomain webmail.example.com. (It's a real subdomain, not just the one used by webmail clients)
Plesk shows only the default certificate instead of this one. Also when I create a self signed one.
I had to reinstall my VPS and then these problems occurred. Before I reinstalled it, the letsencrypt extensions was working and I had a "green" certificate signed by the Let's Encrypt CA.
Is this somehow related to #19 or does this sound like a different problem?
Additional Info:

Ubuntu 14.04.4 64bit
Plesk 12.5.30 Update 23
Apache 2.4.7
Let's Encrypt-Plesk 1.4

Regards,
Stefan

+1


scenario: ember-canary failing

Currently the ember-canary scenario is failing due to ember-source removing import { backburner } from '@ember/runloop'; in-favor of import { _backburner } from '@ember/runloop';.
I'm not sure if there is sufficient dynamism here to have a backwards & forwards compatibility here. It may need @ember/runloop to reconsider the backburner -> _backburner transition.
The change in question

Actually it works really nicely now 😸
import { _backburner } from '@ember/runloop';

This syntax works on all supported Ember versions (going back a long way) as long as the addon or app in question has a new enough ember-cli-babel version (at least ember-cli-babel@7.25.0).

@rwjblue yup thats great, thank you!


No way to disable default music

There should be a way to disable the default Minecraft music, perhaps via a gamerule.

This requires that gamerules are syncchronized with the client, since right now they are entirely server-side.

Note that the above commit is only a 'fix' in the sense that it would work if the flag is set. Problem is that its impossible to change the NBT data of a player, since /entitydata simply doesn't allow that.

If Issue #54 is implemented, the above 'fix' can actually be used.

Issue #54 was implemented and that will be the fix for now. I will sync gamerules and add a gamerule in the next update.


Test with OCP, OCS and ACM stack

Scope: ensure Ramen works in OCP, OCS and ACM stack, and not just in Minikube and K8s only. Run e2e on downstream setups (at least once).
(from HackMD list)

Basic testing with an OKD cluster has been done, closing this.


Getting ERROR on JRuby?

Getting ERROR on JRuby environment
please help on it. How to install gem in JRuby.
ERROR :
Fetching: byebug-9.0.6.gem (100%)
Building native extensions. This could take a while...
ERROR:  Error installing byebug:
ERROR: Failed to build gem native extension.
vendor/jruby/bin/jruby -r ./siteconf20200625-23387-1wmouey.rb extconf.rb
creating Makefile
current directory: vendor/bundle/jruby/2.5.0/gems/byebug-9.0.6/ext/byebug
make "DESTDIR=" clean
current directory: vendor/bundle/jruby/2.5.0/gems/byebug-9.0.6/ext/byebug
make "DESTDIR="
make: *** No rule to make target 'vendor/jruby/lib/ruby/include/ruby/ruby.h', needed by 'threads.o'.  Stop.
make failed, exit code 2
Gem files will remain installed in vendor/bundle/jruby/2.5.0/gems/byebug-9.0.6 for inspection.
Results logged to vendor/bundle/jruby/2.5.0/extensions/universal-java-11/2.5.0/byebug-9.0.6/gem_make.out

I have the same problem.
Work-around: disable byebug for JRuby:
  unless RUBY_PLATFORM == 'java'
    spec.add_development_dependency 'pry-byebug', '~> 3.9'
  end

https://github.com/deivid-rodriguez/byebug/issues/179#issuecomment-152727003


improve floating point comparisons in vic_init.c

A few cleanup items.

[ ] closes #xxx
[ ] tests passed
[ ] ran uncrustify prior to final commit
[ ] ReleaseNotes entry

@UW-Hydro/vic-developers - this is ready for another review.

@bartnijssen - I think we're all good here. Want to take one more look?

@jhamman - review passed - just merging develop back into it since it said it was out-of-date. If the check passes I'll go ahead and merge.


export hash function

It would be useful to have access to the hash function in the case where one needs to ensure equality in custom objects. For example, if wanted to use Immutable.Set() to store a custom object, the set would contain duplicates:
class Principal {
  constructor(type, id) {
    this.type = type;
    this.id = id;
  }
}

const principals = [
  new Principal('type_0', 'id_0'),
  new Principal('type_0', 'id_0')
];

const principalsSet = Immutable.Set().withMutations((set) => {
  principals.forEach((principal) => {
    set.add(principal);
  });
});

principalsSet.size; // 2, but intend for it to be 1

In order for Immutable.Set to drop duplicate Principal objects, I would need to additionally implement valueOf, or both equals and hashCode, on the Principal class. Having access to Immutable.hash() would make this much easier.
I'm exposing the internal hash function along with the main Immutable API, however, it's worth considering exporting this as a standalone utility function.

I agree with you about the utility of this. I'll merge this for now, but we certainly should consider a separate module in the future

@leebyron I think that makes a lot of sense. Do you think this is should be extracted as another Facebook project, or would you consider relying on a 3rd party dependency, say if I were to create a new project with the current implementation of Immutable.hash() as the starting point?

I'd prefer not to rely on 3rd party dependencies - I'll likely consider setting up a multi-package distribution in the future that allows subsets of Immutable to be used in isolation

@leebyron any idea when there will be another immutable-js release? v3.8.1 was almost a year ago, and there have been a lot of changes since then.

I plan on getting to another release sometime before 2019.
Just kidding, I'm cooking up a new release in the next few days

A year later, the initial "sometime before 2019" estimate seems reasonable :)


HLSL Formatting does not work

When I press format document it's just do nothing. Currently I'm using last version of VSCode 1.74.3. I was trying to debug extension but I cannot fix it. It creates cpp file successfully but this line just return undefined (extension.ts, line 24)
return vscode.commands.executeCommand<vscode.TextEdit[]>('vscode.executeFormatDocumentProvider', doc.uri, options)             .then(r => (tmpFile.removeCallback(), r));
I tried to copy my hlsl code to cpp file and format and it works. But I do not why extension seems can't do it action 'vscode.executeFormatDocumentProvider' return undefined. I checked doc.uri and options it also seems correct I do not know where is the problem.

Was fixed with latest c++ update


mapString doesn't exist?

The readme uses mapString in the example code, but there only seems to be mapText or mapVarChar in AbstractMapping.

I changed some methods with the last major release and I must have forgot to update the README. 😳 I will fix it, or you can make a Pull Request changing it from mapString to mapText. 🙏

README has been updated. 👍


Fix travis CI

#196 #195

This change is

hmm, Travis ci might need an action.
https://travis-ci.com/github/sfc-aqua/quisp/requests
now Travis gives 10k credit to OSS projects at first and then we might need to claim it.
https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing
but "GitHub actions" is free for the public repository, so I want to use this and it releases us from the management of these credit issues.
I'll try GitHub Actions tomorrow.
https://docs.github.com/en/billing/managing-billing-for-github-actions/about-billing-for-github-actions

Thanks for reviewing!


Android icon for push notification

SDK version:  3.8.0 and ^1.0.0-beta.15 for expo plugin
Describe the bug
The Android notification icon for our app is not displaying correctly. Instead of showing our app's icon, it's displaying nothing in the status bar and notification drawer.

Hey @thozh. Thanks for reaching out and sharing this. This issue looks very similar to https://github.com/customerio/customerio-expo-plugin/issues/115 reported earlier in Expo plugin. Unfortunately, we haven't made any changes to the plugin yet. So the workaround suggested here remains the only option. I'll also +1 internal ticket on your behalf.
Closing this as we already have an issue open for the change. We'll keep you posted on any future improvements. Even though the issue is closed, please feel free to ask any questions you may still have. Have a great day!


Create 3 Sizes of Black rounded button

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Please Follow the button design system to see the padding for small, medium, large button
After building the component please raise a PR with a screenshot of the component and add the component in Button/src/Primary/text-button-rounded-black.html.
If you need to use any icon please use it from Hero Icons
Good luck.

I will like to work on this


Self-model references cause TS error

Hi! I feel like this used to work before:
import { model, prop, Model } from "mobx-keystone";

@model("app/MyModel")
export class MyModel extends Model({
  child: prop<MyModel>()
}) {}

now it causes this TypeScript error:
Type 'MyModel' recursively references itself as a base type.ts(2310)
'MyModel' is referenced directly or indirectly in its own base expression.ts(2506)

While this works:
export class MyPlainModel {
  child: MyPlainModel = new MyPlainModel();
}

Any idea? Thanks!

That's because you are saying that your model must have your model as child, without exceptions and recursively until infinite. If you put for example undefined in your prop you break this chain, e.g.:
@model("app/MyModel")
export class MyModel extends Model({
  child: prop<MyModel | undefined>() // note undefined here
}) {}```

In other words, if you do this you will get the same error:
interface A {
  a: A;
}

so you'd need to make an object like { a: { a: { a: { .... which is not possible

Oh that makes sense. Sorry, I got somehow confused.
Thanks for help.


Remove unused imports

Hi,
A few unused imports. Hopefully Travis-CI should be able to confirm they are not used.

Thanks for finding this. Also it seems that the Travis tests (as minimal as they are) were turned off after moving to payu-org. Will keep this one open until we get those reactivated.

Cripes that is a lot of cruft. Did you run a linter across the code to pick this up @kinow? If so, we should probably auto-lint with travis.

Just a warning that the current number of pylint warnings is very large :).

@aidanheerdegen found some typos, then ran pycharm's inspection. Which uses several linters (some pycharm custom ones too).
Then read the docs, read and executed unit tests, and finally created the class diagram to see what was connected to what.
I liked the design of Payu, and as a thank you for the docs and open sourcing, sent what appeared in the ide.

Definitely appreciate the input, thanks!
From the cylc activity, I'm guessing you are at NIWA or at least connected to similar activities, would be great to catch up if the opportunity arises.

From the cylc activity, I'm guessing you are at NIWA or at least connected to similar activities, would be great to catch up if the opportunity arises.

That's correct, I'm based in Auckland, working on Cylc for NIWA. Found Payu while searching cylc related things in GitHub, and found an issue related to Jinja2 in Payu I think. One thing leads to another, and then learned a bit about Payu 😁

One of the "must get around to doing this one day" was encapsulating the PBS stuff in a Scheduler class so we can support other schedulers, like slurm, but also maybe cylc.
Does that sound interesting to you @kinow?
Is there some prior art we could ~nick~ repurpose for this? I noticed the PBSpro sources have a large python test harness but I took one look and ran away (14K lines)
https://github.com/PBSPro/pbspro/blob/master/test/fw/ptl/lib/pbs_testlib.py

One of the "must get around to doing this one day" was encapsulating the PBS stuff in a Scheduler class so we can support other schedulers, like slurm, but also maybe cylc.
Does that sound interesting to you @kinow?

Oh, it does sound interesting!

Is there some prior art we could nick repurpose for this? I noticed the PBSpro sources have a large python test harness but I took one look and ran away (14K lines)

I think we have something similar implemented in Cylc, but in the next releases it may change to support Python setuptools' entry points, and maybe some object hierarchy. At the moment the logic in Cylc is in BatchSysManager and at the implementations in cylc.batch_sys_handlers.
We also have a ticket for DRMAA. Perhaps if a Scheduler class was generic enough, we could use that in Payu and also in Cylc, as Cylc also interfaces with these schedulers.
WDYIT @aidanheerdegen ?

Sorry .. the day caught up with me. Looks very interesting, and as always the devil is in the detail. I will try and remember to make an issue to continue the discussion.

@aidanheerdegen Just noticed this is still here, looks pretty innocent.  Easy enough to merge?


Infinite loop in String#index with binary data when MRB_UTF8_STRING is enabled

I've encountered an issue where enabling the MRB_UTF8_STRING flag leads to an infinite loop when using the String#index method on a string containing binary data.
Steps to Reproduce
The issue can be reproduced with the following steps:

Enable the MRB_UTF8_STRING flag and build.# build_config.rb
MRuby::Build.new do |conf|
  toolchain :gcc
  conf.gembox 'default'
  conf.cc.defines << 'MRB_UTF8_STRING'
end

Execute the following script:$ cat test.rb
p "\x80k\xb8".index("\n")
$ ./build/bin/mruby test.rb

The issue can be observed.

Expected Behavior
The expected behavior is that in CRuby, it would return nil.
Actual Behavior
The String#index method enters an infinite loop, and the process does not terminate.
Additional Information
During debugging with gdb, it appears that the issue occurs in the str_index_str_by_char_search function in src/string.c. Specifically, an infinite loop seems to occur in the following part of the code:
https://github.com/mruby/mruby/blob/master/src/string.c#L412-L415
I am not very familiar with the Quick Search algorithm (Boyer-Moore-Horspool algorithm), so I'm not sure how to resolve this issue 😭.
I would appreciate it if you could look into this issue. If any more information is needed, please let me know.

As a workaround, although it sacrifices speed, you can avoid this issue by using String#chars to convert the string to an array and then calling the index method:
p "\x80k\xb8".chars.index("\n")
#=> nil

p "\x80k\n\xb8".chars.index("\n")
#=> 2

I was able to reproduce the problem.
After a little research, I suspect that the problem is not in str_index_str_by_char_search(), but in mrb_utf8len().
I also get an infinite loop in other places that call mrb_utf8len().
e.g. bin/mruby -e 'p "\x81xyz".chop!'
Since mrb_utf8len() is also used to check if a character is illegal as UTF-8 (String#valid_encoding?), it may be desirable to separate the feature.
@matz, is there another preferred method of correction?

I've just pulled the latest commit and rebuilt the system, and I'm happy to confirm that the issue has been resolved. Thank you for the fix!

We are still working on refactoring index and chars2bytes. Notice us when you see any bugs.


tomato invalid memory access

when i open the program it doesnt show and dps or chat logs when i open with console it displays this:
Exception in thread "Thread-16" java.lang.Error: Invalid memory access
at packets.packetcapture.sniff.ardikars.NativeMappings.pcap_loop(Native Method)
at packets.packetcapture.sniff.ardikars.NativeBridge.loop(NativeBridge.java:78)
at packets.packetcapture.sniff.Sniffer$1.run(Sniffer.java:131)
at java.base/java.lang.Thread.run(Thread.java:1589)
java version:
java version "19.0.1" 2022-10-18
Java(TM) SE Runtime Environment (build 19.0.1+10-21)
Java HotSpot(TM) 64-Bit Server VM (build 19.0.1+10-21, mixed mode, sharing)

I would strongly suggest to follow the trouble shoot guide. I believe this problem is solvable following the steps in the guide.

i have tried some of those things and now nothing shows in console but still i dont see anything like chat or dps


Increased CPU limit; Add subdirectory to profiler path.

Increased CPU limit based on scale testing.  300m may not be enough but much better than 100m.
Only really needed by the inventory container but don't think the main controller container will ever actually use it.
Each container needs to write to a separate directory else the profiler in both containers write (overwrite) the same file.
Note: I HAVE NOT TESTED the changes but thought the PR would be the easiest way to describe the requested changes.

LGTM , tested forklift latest on OCP 4.7 using proper CR for changes :

Can you suggest how QE can validate  it? @jortel

There's nothing to validate here. It's a technical parameter that changes.
The best you can do is confirm that the controller container CPU limit is now "500m".


Test Issue Lang

PHP Version
Test Issue Lang
Shopware Version
Test Issue Lang
Expected behaviour
Test Issue Lang
Actual behaviour
Test Issue Lang
How to reproduce
Test Issue Lang

[public] This is just a test issue. We will close it
[created from LAN-174, comment 483952]


Add benchmark

TAP version 13

# benchmark keychain.get() max
    # 9) 100000000x = 55 ms
ok 1 - benchmark keychain.get() max # time = 55.653767ms

# benchmark keychain.get() medium
    # 6) 100000x = 6 ms
    # 7) 1000000x = 31 ms
    # 8) 10000000x = 287 ms
ok 2 - benchmark keychain.get() medium # time = 325.228981ms

# benchmark keychain.get() all
    # 1) 1x = 0 ms
    # 2) 10x = 0 ms
    # 3) 100x = 0 ms
    # 4) 1000x = 0 ms
    # 5) 10000x = 0 ms
    # 6) 100000x = 4 ms
    # 7) 1000000x = 30 ms
    # 8) 10000000x = 320 ms
    # 9) 100000000x = 2944 ms
ok 3 - benchmark keychain.get() all # time = 3299.117573ms

# benchmark keychain.get(name) max
    # 6) 100000x = 7902 ms
ok 4 - benchmark keychain.get(name) max # time = 7901.541281ms

# benchmark keychain.get(name) medium
    # 3) 100x = 8 ms
    # 4) 1000x = 81 ms
    # 5) 10000x = 802 ms
ok 5 - benchmark keychain.get(name) medium # time = 891.175884ms

# benchmark keychain.get(name) all
    # 1) 1x = 1 ms
    # 2) 10x = 0 ms
    # 3) 100x = 8 ms
    # 4) 1000x = 82 ms
    # 5) 10000x = 784 ms
    # 6) 100000x = 7938 ms
ok 6 - benchmark keychain.get(name) all # time = 8813.6628ms

# benchmark keychain.sub() max
    # 9) 100000000x = 3005 ms
ok 7 - benchmark keychain.sub() max # time = 3004.785736ms

# benchmark keychain.sub() medium
    # 6) 100000x = 5 ms
    # 7) 1000000x = 41 ms
    # 8) 10000000x = 422 ms
    # 9) 100000000x = 4269 ms
ok 8 - benchmark keychain.sub() medium # time = 4738.641061ms

# benchmark keychain.sub() all
    # 1) 1x = 0 ms
    # 2) 10x = 0 ms
    # 3) 100x = 0 ms
    # 4) 1000x = 0 ms
    # 5) 10000x = 1 ms
    # 6) 100000x = 5 ms
    # 7) 1000000x = 44 ms
    # 8) 10000000x = 427 ms
    # 9) 100000000x = 4217 ms
ok 9 - benchmark keychain.sub() all # time = 4694.193797ms

# benchmark keychain.sub(name) max
    # 6) 100000x = 7934 ms
ok 10 - benchmark keychain.sub(name) max # time = 7934.413522ms

# benchmark keychain.sub(name) medium
    # 3) 100x = 8 ms
    # 4) 1000x = 79 ms
    # 5) 10000x = 805 ms
ok 11 - benchmark keychain.sub(name) medium # time = 892.742609ms

# benchmark keychain.sub(name) all
    # 1) 1x = 0 ms
    # 2) 10x = 0 ms
    # 3) 100x = 8 ms
    # 4) 1000x = 83 ms
    # 5) 10000x = 801 ms
    # 6) 100000x = 7998 ms
ok 12 - benchmark keychain.sub(name) all # time = 8891.382471ms

1..12
# tests = 12/12 pass
# asserts = 0/0 pass
# time = 51448.494087ms

# ok

keychain.get() timing is a bit weird, you can see that the first test keychain.get() max doing 100000000x takes 55 ms.
Later in the test keychain.get() all the same 100000000x takes 2944 ms.

keychain.get(name) seems consistent


The step parameter is not being used

You should correct the step, since it is not referenced anywhere.
step={config.range.step}

Hi,
You are right. The purpose of the 'step' attribute would be to specify the range between allowed numbers in the scoring range.
You can see that this was resolved on here 8f8e383.
Thank you!


Headless Mode?

Hey Chris,
Can this run headless (e.g. cordova app is terminated by user but the plugin continues to write to firestore)?

Yes.


Option to display derived files in one row in the explorer (for example, .c and .h)

https://github.com/Microsoft/vscode/issues/4047


[i2c,dv] Add csr test exclusions for CSR.{target,controller}_event

Both of these registers are sw:"rw1c"/hw:"hrw".
Fixes #22761

CHANGE AUTHORIZED: hw/ip/i2c/data/i2c.hjson

CHANGE AUTHORIZED: hw/ip/i2c/data/i2c.hjson


Don't look for rules in subdirectories that start with a period

I'm using Kubernetes to mount Elastalert rules into a directory, and the way that Kuberentes mounts files is with links, like so:
/ # ls -alR /rules/
/rules/:
total 12
drwxrwxrwx    3 root     root          4096 Apr 12 13:33 .
drwxr-xr-x    1 root     root          4096 Apr 12 13:33 ..
drwxr-xr-x    2 root     root          4096 Apr 12 13:33 ..4984_12_04_13_33_37.139875337
lrwxrwxrwx    1 root     root            31 Apr 12 13:33 ..data -> ..4984_12_04_13_33_37.139875337
lrwxrwxrwx    1 root     root            17 Apr 12 13:33 rule1.yaml -> ..data/rule1.yaml
lrwxrwxrwx    1 root     root            17 Apr 12 13:33 rule2.yaml -> ..data/rule2.yaml

/rules/..4984_12_04_13_33_37.139875337:
total 16
drwxr-xr-x    2 root     root          4096 Apr 12 13:33 .
drwxrwxrwx    3 root     root          4096 Apr 12 13:33 ..
-rw-r--r--    1 root     root           598 Apr 12 13:33 rule1.yaml
-rw-r--r--    1 root     root           555 Apr 12 13:33 rule2.yaml

When I try to point my rule directory at /rules/, I get the following error:
Traceback (most recent call last):
  File "/usr/local/bin/elastalert", line 11, in <module>
    load_entry_point('elastalert==0.1.8', 'console_scripts', 'elastalert')()
  File "/usr/local/lib/python2.7/site-packages/elastalert/elastalert.py", line 1578, in main
    client = ElastAlerter(args)
  File "/usr/local/lib/python2.7/site-packages/elastalert/elastalert.py", line 95, in __init__
    self.conf = load_rules(self.args)
  File "/usr/local/lib/python2.7/site-packages/elastalert/config.py", line 425, in load_rules
    raise EAException('Error loading file %s: %s' % (rule_file, e))
elastalert.util.EAException: Error loading file /rules/..4984_12_04_13_30_12.887977537/rule2.yaml: Duplicate rule named Admin Privilege Gain

Can Elastalert ignore rules in subdirectores that begin with a period .?

I just found the scan_subdirectories flag is available and that solves my use case, but I'd still like to know if this is a feature worth providing. I'd be willing to implement it if so.

Just ran into the same issue myself. Thanks for pointing out that flag @micahhausler

Yeah this sounds reasonable.

@micahhausler You saved my day... very good catch. I've been struggling why ElastAlert fails due to "Duplicated Name" while I mounting only one rule.

I'm having this exact issue right now. But unfortunately when I set scan_subdirectories: false elastalert just doesn't start (no output to stdout).
Have there been any recent changes to this?

I don't believe so. You definitely have .yaml or .yml files in the rules_folder?

Sorry, should have run it in --verbose. It is in fact working, it's just very silent 😄
Thanks @micahhausler for the tip and @Qmando for responding!


Should we change defaults ?

Hey there! 👋🏻
I was wondering if we should change some defaults value.
For example, adding removeOnComplete to true by default may be beneficial to avoid too much mess in Redis.
:books: https://docs.bullmq.io/guide/queues/auto-removal-of-jobs

Hello,
I think it's a good idea because in my previous job we did not pay attention to this parameter and we ended up with thousands of jobs stored in redis for nothing.