Search box - Arrows up & down then Select on Enter

Rancher 2.6.8
Context Search box:

IMHO it would be great to select options with up & down arrows and select them by press Enter.
Currently Enter button has no effect.

On 2.7-head commit id: 65089ee

Navigate to cluster explorer
Select the namespace picker dropdown. User is able to select using up & down arrow keys. And is able to select by hitting Enter
Navigate to Workloads section
User is unable to select by hitting Enter any namespace from the namespace picker. The up and Down arrow keys however works. Reopening because of this issue.

🚧 REOPENED
Reproduction Environment

Component
Version / Type

Rancher version
2.7.1

Installation option
Helm (high availability)

If Helm Chart k8s cluster
v1.24.7+k3s1

Cert Details
Let's Encrypt / traefik

k3s ha external db
Aurora MySQL 5.7.mysql_aurora.2.11.1

Helm version
v2.16.8-rancher2

Downstream cluster type
RKE1 Linode

Downstream K8s version
v1.24.9-rancher1-1

Authentication providers enabled
local

Logged in user role
admin, standard user

Browser type
Microsoft Edge

Browser version
110.0.1587.63

🚨 Additional Reproduction Setup Details: Click to Expand

K3s install command for node one. Replace values in <>

curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=<K3S_VERSION> sh -s - server --token=SECRET --datastore-endpoint=<DB_ENDPOINT> --tls-san <TLD> --node-external-ip <IP>

K3s install command for node two. Replace values in <>. Main difference here is the --token

curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=<K3S_VERSION> sh -s - server --token=<TOKEN_FROM_NODE_ONE> --datastore-endpoint=<DB_ENDPOINT> --tls-san <TLD> --node-external-ip <IP>

Reproduction steps

Navigate to cluster explorer
Select the namespace picker dropdown. User is able to select using up & down arrow keys. And is able to select by hitting Enter
Navigate to Workloads section
User is unable to select by hitting Enter any namespace from the namespace picker. The up and down arrow keys however work

Additional Info
RESULTS
✅ Expected
To be able to use Enter to make selections
❌ Actual
Unable to use Enter to make selections

Validation Environment

Component
Version / Type

Rancher version
v2.7-6cca23fd75c3ccc447a6013c7a0a3482656515d6-head

Rancher commit link
https://github.com/rancher/rancher/commit/6cca23fd75c3ccc447a6013c7a0a3482656515d6

Installation option
Helm (high availability)

If Helm Chart k8s cluster
v1.24.7+k3s1

Cert Details
Let's Encrypt / traefik

k3s ha external db
Aurora MySQL 5.7.mysql_aurora.2.11.1

Helm version
v2.16.8-rancher2

Downstream cluster type
RKE1 Linode

Downstream K8s version
v1.25.6

Authentication providers enabled
local

Logged in user role
admin, standard user

Browser type
Microsoft Edge

Browser version
110.0.1587.63

🚨 Additional Reproduction Setup Details: Click to Expand

K3s install command for node one. Replace values in <>

curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=<K3S_VERSION> sh -s - server --token=SECRET --datastore-endpoint=<DB_ENDPOINT> --tls-san <TLD> --node-external-ip <IP>

K3s install command for node two. Replace values in <>. Main difference here is the --token

curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=<K3S_VERSION> sh -s - server --token=<TOKEN_FROM_NODE_ONE> --datastore-endpoint=<DB_ENDPOINT> --tls-san <TLD> --node-external-ip <IP>

Validation steps

Navigate to cluster explorer
Select the namespace picker dropdown. User is able to select using up & down arrow keys. And is able to select by hitting Enter
Navigate to Workloads section
Select namespaces with Enter, this now works, however you cannot read the namespace name once it's selected now

Additional Info
RESULTS
✅ Expected
To be able to use Enter to make selections (and for the UI not to change drastically)
❌ Actual
To be able to use Enter to make selections (unable to see the namespace name after selecting wth Enter)
When selecting the namespace with a mouse, or laptop touch pad, you can still see/read the namespace name.
However, if you select the namespace with Enter the namespace name isn't readable anymore. See the screenshot below. This also happens with all of the listed browsers below

#
Browser Tested With
Version
Issue Present?
Present Light Mode
Present Dark Mode

1
Microsoft Edge
110.0.1587.63 (Official build) (x86_64)
YES, ISSUE PRESENT
YES
YES

2
Opera GX
LVL4 (core: 94.0.4606.107) (x86_64)
YES, ISSUE PRESENT
YES
YES

3
Google Chrome
110.0.5481.177 (Official Build) (x86_64)
YES, ISSUE PRESENT
YES
YES

4
Firefox Developer Edition
111.0b8 (64-bit)
YES, ISSUE PRESENT
YES
YES

5
Safari
16.3 (186<IP_ADDRESS>.6)
YES, ISSUE PRESENT
YES
YES

@torchiaf Can you take a look?

@torchiaf Can you take a look?

first bug, Enter doesn't work anymore after a selection: I am looking into it.
second bug, invisible text: there is a PR that will resolve this issue: https://github.com/rancher/dashboard/pull/8306 @brudnak

reopened
Setup

Component
Version / Type

Rancher version
v2.7-7913e283ff90eebd1f8cc48860a6b410d72cd4dd-head

Rancher commit link
https://github.com/rancher/rancher/commit/7913e283ff90eebd1f8cc48860a6b410d72cd4dd

Installation option
Helm (high availability)

RKE binary version used
v1.4.0

If Helm Chart k8s cluster
v1.24.6

Cert Details
Let's Encrypt / nginx

Docker version
20.10.7, build f0df350

Helm version
v2.16.8-rancher2

Downstream cluster type
not applicable

Downstream K8s version
not applicable

Authentication providers enabled
local

Logged in user role
admin, standard user

Browser type
Google Chrome

Browser version
111.0.5563.110

Describe the bug
Can't use Enter in conjunction with the mouse when selecting namespaces to filter by.
To Reproduce

Start Rancher on 2.7-head
Navigate to Workloads
Click the namespace filter dropdown
Using the mouse click the cattle-global-nt under Project: System
Now that is selected use the down arrow key to navigate to cattle-system also under Project: System
Try pressing Enter on cattle-system
The Enter key does not work after 1st using the mouse to make an initial selection

Result
Can't use Enter to select namespaces after using the mouse for an initial selection
Expected Result
You can use Enter to make an initial selection, then use the mouse to make subsequent selections. So I would expect the opposite of that to also work as well.
Enter works for selections if you only use Enter.
See the video below, using a tool to show which keys are being pressed.
Screenshots
https://user-images.githubusercontent.com/40877149/228078148-816df1b6-e728-4c64-8937-392abfe4a979.mp4
Bug Frequency

Frequency
Selected Option

Consistent (10/10) - A consistent bug can be replicated every time the test is executed.
✅

Frequent (7/10)- A frequent bug can be replicated like 7 out of 10 times when a test is replicated

Occasional (5/10) - Occasional bug is more hard to replicate than a frequent bug.

Once thus far, twice thus far - Few bugs cannot be reproduced every time you run the test.

Thanks @brudnak . This is not actually a regression, it's just the way it works the filter.
When the user hits a namespace with the mouse, the focus moves on the scrollbar and then any shortcut won't work.
I did a test on version v2.6.7 using the spacebar, since the Enter key was not implemented yet in that version, and the behavior is the same there.
IMO we can close this issue.

Sounds good, thanks @torchiaf. Checks here look good then. Will open a new issue for this. Thanks!

Checks for Enter passing ✅
New bug found while validating: https://github.com/rancher/dashboard/issues/8555


✨ Markdown button when writing paste

Summary
Clicking will add --- to top of textarea, unclicking will remove.

Will be closed by #54


System.TypeLoadException: 'Could not load list of method overrides due to Method not found: int SQLitePCL.ISQLite3Provider.sqlite3_win32_set_directory(int,string)'

Hello friend, I hope you're doing well.
I've been having this issue when running my project on devices with Android 11 and later.
System.TypeLoadException: 'Could not load list of method overrides due to Method not found: int SQLitePCL.ISQLite3Provider.sqlite3_win32_set_directory(int,string)'
Could you inform me where I might be going wrong? Thanks in advance!
Here is the list of packages added to the project:
"Newtonsoft.Json" version="13.0.3" targetFramework="monoandroid11.0"
"sqlite-net-pcl" version="1.9.172" targetFramework="monoandroid11.0"
"SQLitePCLRaw.bundle_e_sqlite3" version="1.1.4" targetFramework="monoandroid11.0"
"SQLitePCLRaw.bundle_green" version="2.1.2" targetFramework="monoandroid11.0"
"SQLitePCLRaw.core" version="2.1.2" targetFramework="monoandroid11.0"
"SQLitePCLRaw.lib.e_sqlite3" version="2.0.0" targetFramework="monoandroid11.0"
"SQLitePCLRaw.lib.e_sqlite3.android" version="2.1.2" targetFramework="monoandroid11.0"
"SQLitePCLRaw.provider.e_sqlite3" version="2.1.2" targetFramework="monoandroid11.0"
"SQLitePCLRaw.provider.e_sqlite3.android" version="1.1.14" targetFramework="monoandroid11.0"
"System.Buffers" version="4.5.1" targetFramework="monoandroid11.0"
"System.Memory" version="4.5.5" targetFramework="monoandroid11.0"
"System.Runtime.CompilerServices.Unsafe" version="6.0.0" targetFramework="monoandroid11.0"
"Xamarin.AndroidX.Annotation" version="<IP_ADDRESS>" targetFramework="monoandroid11.0"
"Xamarin.AndroidX.Migration" version="1.0.8" targetFramework="monoandroid11.0"
"Xamarin.AndroidX.MultiDex" version="<IP_ADDRESS>" targetFramework="monoandroid11.0"

Hmmm.  Obviously, the win32-specific call should not be involved or required on Android.
Your list of packages isn't showing up for some reason.
After that, I can think of several diagnostic questions I might ask to try to figure out what's going wrong, but they're the same questions that were in the issue template, so I'm not sure where to go from here.

The problem previously reported, for some reason, stopped occurring after I reviewed all the packages. However, I am now facing another problem in the line below:
var connection = new SQLiteConnection(ConnectionString, SQLiteOpenFlags.ReadWrite | SQLiteOpenFlags.Create, false) ;
Error:
System.DllNotFoundException: 'e_sqlite3 assembly: type: member:(null)'

Forgive me for insisting! I really need to solve this problem because I have been struggling with it for some time. I would be very grateful if you could help me! Please!

Like I tried to explain above, I can't help you without more information, to figure out what's going wrong.
The first pieces of information I need are the answers to the questions in the issue template.  But you are apparently unwilling to answer those questions.  So I don't see any way to help you.

Actually, it looks like I can give at least SOME help from the packages list.
You still have references to packages from SQLitePCLRaw 1.1.14.  These should be removed.
It looks like you're still on packages.config, or converted from that.  You don't need to explicitly list transitive packages.  Try keeping bundle_green but remove all of SQLitePCLRaw.provider.* and SQLitePCLRaw.lib.*

Okay, sorry, but it wasn't very clear what information you need. I have resent the packages used as requested...
Anyway, I rolled back the packages to version 2.10 and set them to the same version across all projects. And it worked!
Thank you for your support and for the time you dedicated to me! It's always good to know that there is someone willing to help. It gives us more confidence. May God always bless you to continue with this work.
🙏🙏🙏

Very good. So glad it's working for you now.
Unsolicited advice for the future:  Whenever you ask an open source maintainer for help, always give LOTS of information and context.  😁

Hello my friend, I hope you're doing well...
Here I am again with a new problem...
However, I don't know what more detail I could provide... if you need more information, please ask me and I will send it to you.
I'll try to provide as much information as I can...
I have this method:
public static IList QuerySQL(string sql, object[] args = null) where T : new()
{
lock (_lock)
{
var db = GetConnection();
    if (args == null)
        args = new object[] { };

    try
    {
        return db.Query<T>(sql, args); // <-- the error occurs here: System.ArgumentException: 'method arguments are incompatible' when executing
    }
    finally
    {
        DataAcessHelper.ReleaseConnection(db);
    }
}

}
The "args" parameter is empty, i.e., it was constructed with this line args = new object[] { };.
The select query I am running is this:
ConfiguracaoEmailDTO ModuloVendas.Query.QueryParametros.IVendedorQuery.GetConfiguracaoEmail()
{
const string sql =
@"SELECT Email AS Email,
Usuario AS Usuario,
Senha AS Senha,
SMTP AS SMTP,
Porta AS Porta,
'' as POP,
0 as PortaPOP,
PrecisaAutenticar AS PrecisaAutenticar,
0 AS UsarSsl,
'' as EmailsPermitidos,
'' as CopiasOculta
FROM ConfigEmail";
var result = DataAcessHelper.QuerySQL<ConfiguracaoEmailDTO>(sql).FirstOrDefault();

return result;

}
I have checked the table in the database, and all fields exist. Here is the DDL of the table:
CREATE TABLE "ConfigEmail" (
"Email" varchar,
"Usuario" varchar,
"Senha" varchar,
"SMTP" varchar,
"Porta" integer,
"PrecisaAutenticar" varchar(1) );
The table has only one record in the database.
I would be extremely grateful if you could shed some light on the problem.
Below I list the packages and versions I am using. Here they are:
"sqlite-net-pcl" version="1.9.172" targetFramework="monoandroid11.0"
"SQLitePCLRaw.bundle_green" version="2.1.2" targetFramework="monoandroid11.0"
"SQLitePCLRaw.core" version="2.1.2" targetFramework="monoandroid11.0"
"SQLitePCLRaw.lib.e_sqlite3.android" version="2.1.2" targetFramework="monoandroid11.0"
"SQLitePCLRaw.provider.e_sqlite3" version="2.1.2" targetFramework="monoandroid11.0"
"System.Buffers" version="4.4.0" targetFramework="monoandroid11.0"
"System.Memory" version="4.5.3" targetFramework="monoandroid11.0"
"System.Runtime.CompilerServices.Unsafe" version="4.5.2" targetFramework="monoandroid11.0"
"Xamarin.AndroidX.Annotation" version="<IP_ADDRESS>" targetFramework="monoandroid11.0"
"Xamarin.AndroidX.Migration" version="1.0.8" targetFramework="monoandroid11.0"
"Xamarin.AndroidX.MultiDex" version="<IP_ADDRESS>" targetFramework="monoandroid11.0"

(1) In most repos and forums, it is considered good practice is to always start a new issue in a new issue.  In other words, since this latest problem is probably unrelated to the first one, it would have been better if you opened a separate issue to discuss it.  The main reason for this is so that it is possible to close issues when they are completed.
(2) Another problem with your latest question is that this repo is probably not the right place to ask.  In other words, this new problem appears to be specific to sqlite-net, because Query<T>() is a method in that package, not in SQLitePCLRaw.  The repo for sqlite-net is at https://github.com/praeclarum/sqlite-net
(3) That said, I wish I could give you a helpful answer anyway, but I'm not familiar with the API you're calling.  It seems clear that Query<T>() doesn't like the args you are passing in, but I don't know what those args are for.  I speculate they are things to be bound into substitution parameters for the SQL query, and the args you pass in need to be type-compatible with the corresponding SQL expressions.  But I don't see any parameters in the SQL SELECT statement you included, so I'm not sure my guess is useful.

Okay my friend, forgive my ignorance and thank you very much for the guidance. I will follow your advice, as I am new to this type of communication via forums and therefore end up making these basic mistakes. I will check these issues more carefully, as I am certainly getting confused and that's why I've been having these problems. Once again, thank you very much.


Linux version leaves zombie bash shells.

Type: Bug
On a private (single-user) Linux machine,

Observe the bash shells running on your system.
Start VSCode.  If it does not start bash shells itself, start some.
Quit VSCode normally.
Observe the bash shells running again.  Your system has added whatever bash shells were running under VSCode when you exited.

VS Code version: Code 1.77.3 (704ed70d4fd1c6bd6342c436f1ede30d1cff4710, 2023-04-12T09:16:52.732Z)
OS version: Linux x64 5.19.0-41-generic snap
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
AMD Ryzen 7 3700U with Radeon Vega Mobile Gfx (8 x 1804)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
2, 1, 0

Memory (System)
9.60GB (7.23GB free)

Process Argv
--no-sandbox --force-user-env --unity-launch --crash-reporter-id 69fc3609-9779-4891-8da9-c5cd75468cdd

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
wayland

Extensions (11)

Extension
Author (truncated)
Version

python
ms-
2023.6.1

vscode-pylance
ms-
2023.5.10

jupyter
ms-
2023.3.1201040234

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

cmake-tools
ms-
1.14.31

cpptools
ms-
1.15.4

cpptools-extension-pack
ms-
1.3.0

cmake
twx
0.0.17

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vscaac:30438847
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30610679
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
2i9eh265:30646982
6233i204:30730053
vsctsb:30705553
azdwalk:30721579
pythonms35:30701012
pythonfmttext:30731395
pythoncmv:30731968
fixhidewlkth:30730051
hidesbindicator:30730055

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.78.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I believe this is already fixed, can you see if you can reproduce in the Insiders build?
/label info-needed

Megan,
It is already fixed.  I realized the bug itself was preventing snapd from updating my VSCode.  Snapd was seeing processes started by VSCode active, so it would not apply an update. To get to 1.78.1, I had to reboot, start VSCode, and force an update without ever starting a terminal. Thanks for checking on this.
Jonathan
On Tuesday, May 9, 2023 at 07:16:04 PM EDT, Megan Rogge @.***> wrote:
I believe this is already fixed, can you see if you can reproduce in the Insiders build?
/label info-needed
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Use node:-prefixed imports for Node.js core modules

FYI: https://github.com/nodejs/node/issues/36098
The node:-prefix was introduced to avoid namespace conflicts when importing Node.js core modules.
At this time, it does not affect the operation without prefixes, but since some modules, such as node:test, require prefixes, I suggest that you modify your entire codebase with prefixes while you are at it.
Also, as a side effect of prefixing, you may be able to run bolt on Cloudflare Workers.
Cloudflare Workers has announced the availability of Node.js compatible APIs, but the node:-prefix is now required to use them.
Details of how to run on Cloudflare Workers should be a separate issue, but this possibility is a great motivation to introduce prefixes.
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Hey @mirror-kt, thanks for sharing your knowledge!
Since bolt-js internally relies on a few modules that do not work on Cloudflare Workers or other similar platforms, which do not natively support Node.js runtime APIs. In particular, using axios for HTTP communications is a key blocker for it. Until axios and some other modules start supporting cf-workers, simply adding the prefix on this project's side does not help.
With that being said, if this option proves valuable for a broader audience in the future, we may consider working on it. Thanks again for the suggestion!

FWIW, if you are looking for a currently available solution to build Slack apps on Cloudflare Workers, my personal hobby project may be useful for you: https://github.com/seratch/slack-cloudflare-workers The project is still very young, but I've been actively working on it lately!

The current major version of bolt supports all the way back to node 14. I am slowly working towards a new major version, which will drop node support prior to v18. It was hard to find information about when exactly node: prefix was introduced, but it seems like some minor versions of v12 and v14 support it. So, I think we can make this switch when we move to work on bolt v4.

Resolved in #2259 and released in bolt v4!


[STORM-729] Include Executors (Window Hint) if the component is of Bolt type

This was missing only from the Bolt case.

+1

+1


Parameters for global descriptor extraction

Hi there,
I am writing regarding your 3DV paper "Benchmarking Image Retrieval for Visual Localization". Really cool paper with many insights! Thanks a lot for open-sourcing the code and for the extensive documentation.
I am looking for the pre- and post-processing parameters used to extract the global descriptors such as NetVLAD, AP-GeM, and DELG. I am particularly interested in the size of the input images, the exact scales (if multiscale extraction), whether whitening was applied, etc. I could not find such details in the paper and this repository does not seem to mention the arguments of the extraction scripts (e.g. extract_kapture.py for AP-GeM). These would be very useful to reproduce the benchmark results and compare them with other datasets.
Thanks a lot!

Hi Paul-Edouard!
Thanks for your interest in our benchmark. Sorry if we missed some implementation details in the paper or the suppl. mat. Let me summarize this here.
If not indicated differently, we used the default parameters of the scripts. We also provided the unchanged dataset images to the feature extractors.

DenseVLAD: Code available at http://www.ok.ctrl.titech.ac.jp/~torii/project/247/. For the multi-scale variant, we used 4 different scales corresponding to region widths of 16, 24, 32 and 40 pixels.
DELG: We used this repo snapshot. In the meantime, the DELG code changed a bit. Furthermore, the authors of DELG released more models.
AP-GeM: We used this script with default params: https://github.com/naver/deep-image-retrieval/blob/master/dirtorch/extract_kapture.py
NetVLAD: https://github.com/Relja/netvlad

Note that we provide all extracted features in the kapture dataset downloader as well. Please find a detailed tutorial of how to reproduce the results using the provided features here.
I hope this helps and please let us know if something is unclear.
Best,
Martin

Hi Martin,
Thank you for the swift reply! I thought that multiscale inference was used for AP-GeM (since it usually improves the results) and thus thought that some parameters were missing. I was wrong and the paper indeed includes all the necessary details. Thank you for clarifying.

Hi Martin,
Thank you for the swift reply! I thought that multiscale inference was used for AP-GeM (since it usually improves the results) and thus thought that some parameters were missing. I was wrong and the paper indeed includes all the necessary details. Thank you for clarifying.


why is #r "nuget: Flips,2.4.4" saying .net framework and not netstandard?

See below. I checked the google.ortools website, and it seems to say that it is netstandard. Why isn't flips pulling it in as netstandard?
I am on windows, .net 5.0.200.
- #r "nuget: Flips, 2.4.4"
- ;;
C:\Users\nicho\AppData\Local\Temp\nuget\5000--4c0436a8-8001-4579-9353-44451b77434d\Project.fsproj : warning NU1701: Package 'Google.OrTools.runtime.win-x64 8.1.8487' was restored using '.NETFramework,Version=v4.6.1, .NETFramework,Version=v4.6.2, .NETFramework,Version=v4.7, .NETFramework,Version=v4.7.1, .NETFramework,Version=v4.7.2, .NETFramework,Version=v4.8' instead of the project target framework 'net5.0'. This package may not be fully compatible with your project.
C:\Users\nicho\AppData\Local\Temp\nuget\5000--4c0436a8-8001-4579-9353-44451b77434d\Project.fsproj : warning NU1701: Package 'Google.OrTools.runtime.win-x64 8.1.8487' was restored using '.NETFramework,Version=v4.6.1, .NETFramework,Version=v4.6.2, .NETFramework,Version=v4.7, .NETFramework,Version=v4.7.1, .NETFramework,Version=v4.7.2, .NETFramework,Version=v4.8' instead of the project target framework 'net5.0'. This package may not be fully compatible with your project.
[Loading C:\Users\nicho\AppData\Local\Temp\nuget\5000--4c0436a8-8001-4579-9353-44451b77434d\Project.fsproj.fsx]
namespace FSI_0002.Project

That is a great question that I have not been able to figure out. I have tried repeatedly to get the package manager to pull in the netstandard version with no luck. I have found that the library still works though. I believe it has something to do with how Google OR Tools publishes since this didn't used to be an issue two versions ago for the OR Tools library.

Thanks for that info. I just wasn’t sure if it was my install or a general thing.
Would paket framework restrictions help? I don’t know how this interacts with the *.fproj target framework, but it might force getting the netstandard version of Google OR. Just a thought, not sure if it’s a fix.
https://fsprojects.github.io/Paket/dependencies-file.html#Framework-restrictions

@nhirschey / @matthewcrews worth bringing the issue to OR tools repository if it used to work in previous release with both netstandard and framework, they'll be able to track what changed in the delivery pipeline.
Their engineering team gave me very quick turnaround in a previous interaction.
@nhirschey are you able to proceed by discarding the warnings?

@nhirschey I did try restrictions with Paket and it did not help. I'll try reaching out the the Google OR Tools team about this.

Just for confirmation, the intro problem works on my machine despite the warning.

It would still be great to be able to get rid of that warning in F# interactive, while using it with .NET 8 (I expect it to appear with the other .NET versions too).

warning NU1701: Package 'Google.OrTools.runtime.win-x64 9.0.9048' was restored using '.NETFramework,Version=v4.6.1, .NETFramework,Version=v4.6.2, .NETFramework,Version=v4.7, .NETFramework,Version=v4.7.1, .NETFramework,Version=v4.7.2, .NETFramework,Version=v4.8, .NETFramework,Version=v4.8.1' instead of the project target framework 'net8.0'. This package may not be fully compatible with your project.

Not sure how your view is on this issue but maybe it would be these days sufficient to just update the Google.Or lib to the newest version @matthewcrews

Yeah, I'll get that taken care of today


Difference lat-lon maps only show half for several obs datasets

In v2.9.0rc2, several lat-lon maps (bottom figure) only show half, including SST_HadISST, AOD_550 and OMI-MLS (@golaz's new run also shows this issue)

I saw the same issue when developing the GPCP v3.2 processing script. Rotating longitude from [-180,180) to [0, 360) resolved the issue. I suspect that a dependency library for visualization no longer auto consolidating  longitude with [-180,180) or [0, 360).

After some troubleshooting, it ruled out change in cartopy as the cause. Looking at the intermediate files being saved, already data already missing half after regridding. It suggests regriding with changes in cdms/esmf is the cause. I tried to roll back to older version/builds of cdms and esmf, but ran into numpy, alias deprecation problems. Pinning for old version is not ideal for coordinate with e3sm-unified. For a immediate solution, it might make sense to convert longitude from [-180, 180) to [0, 360) for input dateset being impacted to work around this problem.

Found an old cdms issue pointing out the same behavior.
https://github.com/CDAT/cdms/issues/371
I'm following the approach suggested in this post to wrap the longitude with [0,360), since cdms somehow failed to do it upon open datasets.

The regriding step also produce warning as follows:
/global/cfs/cdirs/e3sm/zhang40/conda_envs/edv290_numpy/lib/python3.10/site-packages/cdms2/avariable.py:1289: Warning:
avariable.regrid: regridTool = 'esmf' but your version does not
seems to be built with esmf, will switch to regridTool = 'libcf'

  warnings.warn(message, Warning)

It seems cdms is not properly built with esmf. Not sure if this is relevant to the regriding issue here.

@chengzhuzhang, I'm going to see if https://github.com/conda-forge/cdms2-feedstock/pull/87 fixes this.  I patched the imports in cdms2 to hopefully bring in esmpy, since the package has been renamed since v8.4.0 from ESMF to esmpy.

@xylar I'm glad to report: your fix https://github.com/conda-forge/cdms2-feedstock/pull/89 works out!
With esmfy correctly imported by cdms, the regriding behavior gets back to normal.

@chengzhuzhang, could you add a test somewhere in E3SM diags where you test for ESMFRegrid?  It could just be as simple as:
from regrid2 import ESMFRegrid

but you might want to have a try/except ImportError where you give an explanation about this meaning that an incompatible version of cdsm2 is installed that is not able to import ESMPy.
Such an error would tell us right away that there's a problem, rather than noticing it only when plots are half missing.

Great suggestion. I think it makes sense to add it as one of the CI/CD tests.

Fixed.


unable to bring th events on side of calendar

Bug description / Feature request:
Hi i am trying to get the events of the calendar on different div not on the calendar , i am trying from many days but i am unable to .I am working on year view , and i want the events to to be displayed in different div and they should change when i click on any month in year view.This is my project requirement .Please help me with that.
Link to minimally-working plunker that reproduces the issue (starter template: http://plnkr.co/edit/LE4F4U7AnnD3tjM9ZH4G?p=preview)
Versions
Angular:
Calendar library:
Browser name and version:

Please use stackoverflow for support requests, this issue tracker is only for bug reports and feature requests. Thanks!

Hi mattlewis,
Thanks .
and you make an awesome calendar
Laxmi


侧滑菜单响应不敏感

When the sliding menu and viewpager appear in the same scene, sliding menu response is not sensitive

I'd rather open this issue in react-native-viewpager repo. Seems it prevents a default event propagation, so I can't fix it on the side-menu component level.


Steps_per_epoch cannot be easily used with DataLoader/ndarray, because there is no .repeat

Hi,
if a torch.utils.data.DataLoader or a np.ndarray is used in fit together with steps_per_epoch, then when it runs out of data, a warning is generated and the epoch interrupted:
https://github.com/keras-team/keras/blob/85c9a58892d4626d0a37a6fdc13113a6dbb301d8/keras/trainers/epoch_iterator.py#L103-L113
However, while with tf.data.Dataset it was trivial to just add .repeat() to the dataset, this is not the case with a DataLoader or an ndarray. It would be much more useful if the EpochIterator just created a new iterator for the DataLoader/ndarray. Of course, there could be an option to do it only when requested; but I believe the reasonable default is to create the new iterator unconditionally, because the user opted in by specifying non-default steps_per_epoch.
The fix would not be complicated, in the lines linked above, instead of the warning and the two lines, one would just use a single line self._current_iterator = self._get_iterator(return_type) and the self._insufficient_data could be removed.

Hi @foxik, we agree that it's not as straightforward to repeat in torch nor in numpy (you would have to manually deal with these). That being said we don't want to assume users should want the same dataset repeated. For the time being we will keep defaults but feel free to reopen if this gets more interest. Thank you!

Hi @foxik , looking at this again. Once https://github.com/keras-team/keras/pull/19176 is submitted, you can use a torch.utils.data.DataLoader with a custom torch.utils.data.IterableDataset that internally loops forever. This will have the same effect as tf.data.Dataset.repeat(). Then, you can (and must) set the steps_per_epoch.

Hi @hertschuh , thanks for the response :+1: And thanks for adding support for the IterableDataset :rocket:
Although from my point of view, even if it is possible to use the IterableDataset, it is still much less comfortable than just adding a .repeat() to a tf.data.Dataset, and I still think that when steps_per_epoch is set, the dataset should be iterated repeatedly.
BTW, in our case, we ended up creating a custom sampler that just iterates a given dataset in random permutations as many times as needed (and remembers unused data between iterations). That way we have a MapDataset with a-priori known size (and the batching is done by the DataLoader), but instead of giving the steps_per_epoch to the .fit method, we give it to the sampler when creating the DataLoader.

@foxik , oh yes, a custom sampler is a really good solution for this.


Move mathjax include to head from document

The MathJax <script> tag belongs to either <head> or <body>´, but not into the top-level `.
Since I touched that part of the template, I also updated MathJax to its newest version (2.7.5).

@danielschemmel thank you! I currently have mathjax disabled due to the way it's currently implemented.


Window.scrollTo has a different behavior on PC and Smartphone

My program shows a box when clicking on an item. The Onclick calls the server which computes the content of the box and send back the same page with the box overlapping it. To position correctly the box I save the window.scrollY in a cookie and use it to reposition the window scroll when the page comes back,  using the JS window.scrollTo(0,y) function.
On a PC this method works fine.
On a smartphone Samsung J5, the window.scrollto(0,y) command does not always scroll to the good (y) position. I spent a week to discover that the calculus of the scroll seems to be done before the whole page is loaded. Even when putting the calculus in the  ii did not work.
The solution I discovered is to prepare an hidden box and delay the show of the box with :

I little bit weird .....
Is it a know problem ?

This sounds like it might be a problem because on the phone the JavaScript is executing before the DOM has finished loading. This can quite commonly cause problems, e.g. with errors because an object that JS is expecting is not there yet, etc.
We've got a section on script loading strategies that could offer some help here:
https://wiki.developer.mozilla.org/en-US/docs/Learn/JavaScript/First_steps/What_is_JavaScript#Script_loading_strategies
Basically, do you still experience this problem with loading the script with a defer attribute, or executing the script inside a DOMContentLoaded event handler?
If the problem still persists, then this is more interesting than I first thought ;-)


fix UOp const_factor for ADD [run_process_replay]

it's not really used now because we break add chain first, but simple enough to keep it for completeness
negatives are wrong too will fix separately


add share urls feature

Seems it is in there somewhere as I arrived her from the medium article on generating the aphex twin song and it was the only song listed.
Now I wanted to share the didgeridoo beats on with my friend and cannot get at it as a direct url?

I will add a button or it, but for now you can construct your own by doing generative.fm/music/alex-bainter-lower-kebab-case-of-piece-name.
For example:

generative.fm/music/alex-bainter-didgeridoobeats
generative.fm/music/alex-bainter-a-viable-system
generative.fm/music/alex-bainter-timbral-oscillations
generative.fm/music/alex-bainter-aisatsana

Could you use history.replaceState() to update the URL?


Request TGI NeuronX DLC to support Flan T5 models in SageMaker

Feature request
Currently TGI NeuronX loads the artifacts with NeuronModelForCausalLM class, which gives error when loading Flan T5.
Unrecognized configuration class <class 'transformers.models.t5.configuration_t5.T5Config'> for this kind of AutoModel: AutoModelForCausalLM.
Model type should be one of BartConfig, BertConfig, BertGenerationConfig, BigBirdConfig, BigBirdPegasusConfig, BioGptConfig, BlenderbotConfig, BlenderbotSmallConfig, BloomConfig, CamembertConfig, LlamaConfig, CodeGenConfig, CpmAntConfig, CTRLConfig, Data2VecTextConfig, ElectraConfig, ErnieConfig, FalconConfig, FuyuConfig, GitConfig, GPT2Config, GPT2Config, GPTBigCodeConfig, GPTNeoConfig, GPTNeoXConfig, GPTNeoXJapaneseConfig, GPTJConfig, LlamaConfig, MarianConfig, MBartConfig, MegaConfig, MegatronBertConfig, MistralConfig, MixtralConfig, MptConfig, MusicgenConfig, MvpConfig, OpenLlamaConfig, OpenAIGPTConfig, OPTConfig, PegasusConfig, PersimmonConfig, PhiConfig, PLBartConfig, ProphetNetConfig, QDQBertConfig, ReformerConfig, RemBertConfig, RobertaConfig, RobertaPreLayerNormConfig, RoCBertConfig, RoFormerConfig, RwkvConfig, Speech2Text2Config, TransfoXLConfig, TrOCRConfig, WhisperConfig, XGLMConfig, XLMConfig, XLMProphetNetConfig, XLMRobertaConfig, XLMRobertaXLConfig, XLNetConfig, XmodConfig

Motivation
Flan T5 is a popular model in SageMaker.

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!


Support Witness role

From Spanner https://cloud.google.com/spanner/docs/replication
Witness
Witness replicas don’t support reads but do participate in voting to commit writes. These replicas make it easier to achieve quorums for writes without the storage and compute resources that are required by read-write replicas to store a full copy of data and serve reads. Witness replicas:

Are only used in multi-region instances.
Do not maintain a full copy of data.
Do not serve reads.

Participate in leader election but are not eligible to become leader.

It is helpful for us to support crossing IDC replication too.

Most of the time, the leader can only send Log meta (not including Log data) to the witness, but if the log entry contains ConfChange or Admin(like Split, Merge), we need to send these log data too.
We can check ConfChange with https://github.com/pingcap/raft-rs/blob/master/proto/eraftpb.proto#L6 directly, but for Admin, maybe we need to add a flag to https://github.com/pingcap/raft-rs/blob/master/proto/eraftpb.proto#L24.
IMO, we can only introduce the Witness role to the Raft lib, the Witness can't be promoted to Follower or Learner because it has no data.
The Raft module still sends whole messages to the Witness, and we will decide whether to filter the messages outside. This can make the Raft library simple. Another way is to introduce a send-hook and do the filter in the Raft send function. But I prefer the first way.
We can also support Witness in etcd if possible. /cc @xiang90

Witness mode can be implemented outside raft by:

remove data in log entry
drop voting message

But I am fine adding it into raft though.

Please also take a look at https://github.com/lni/dragonboat/issues/106.


[DOCFIX] Improve troubleshooting doc

A few small improvements on troubleshooting

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.5 please

@LuQQiu need your permission?

alluxio-bot, cherry-pick this to branch-2.5 please


I/O from and to midi devices

I have a feature request, to add support for I/O from/to MIDI devices connected to the computer.
Is this possible?

I never thought about using a MIDI device in games. Is it a common use case? If not, then this is better suited in a custom module.

https://www.youtube.com/watch?v=3M7CCn0egcI

Certainly that's one case of game use. My point is that it's not worth adding the burden of MIDI I/O to the core if it's gonna be used only by a handful of specific games.

As a measure Steam has roughly 22000 games when you browse all games and when you look at "music" tag showing 185 games. If MIDI device usage is only limited to games about music this may give an idea about the necessity of MIDI device feature in core.

MIDI is of course music related but can be used as an overall controller as well. I have seen it being used to non musical tasks, for example keeping completely different and not music related but midi equipped devices in sync. I think it's still useful as a simple, kind of slow but popular I/O.
Is it worthy to put it in the core? Maybe. There could always be enable_MIDI=no setting during compilation if someone needs streamlined core.
@vnen >  "Is it a common use case?"
I feel like, if we ask this kind of questions, we might be in danger of seeing Godot used to create mostly loads of platformers and nothing really original... because unfortunately, that is the common use case. Sometimes it might be good to add non-mainstream feature to stick out from the competition.
@hubbyist Midi controllers were used for many years for non musical games.
Some to the extreme I may say https://www.youtube.com/watch?v=fc79UYr_oeI  :D
Programs like http://www.pcworld.com/article/2010931/xpadder-play-pc-games-with-a-game-controller-instead-of-a-keyboard.html  or  http://www.bome.com/products/miditranslator/overview/classic
were used with sturdy midi hardware to play no music related games. It always worked great with all kind of simulations, fighting games etc. Maybe the problem comes from thinking about MIDI controllers as cheap yamaha keyboards for kids, when today's MIDI controllers come in many form and shapes - http://www.sweetwater.com/store/detail/LaunchpadPro - and are basically that thing that lets you build your own Enterprise's cockpit at home ;)
They are also usually much better made then typical computer game-pad-controllers.
So, I would go with MIDI, in core or as a custom module, as long as it's going to be up to date and maintained. AFAIK reduz has a great pro experience with programming audio related stuff. For him it might be a piece of cake to make midi capable Godot. Audio engine rewrite is planned anyway, so he might find time for midi to.
We never know what kind of of new and original games midi would bring to Godot.
Just my $0.02.

I feel like, if we ask this kind of questions, we might be in danger of seeing Godot used to create mostly loads of platformers and nothing really original... because unfortunately, that is the common use case.

Well, this will be the case anyway, whether we ask it or not, since we can't control what the community will create. OTOH, adding edge-case features might make the engine bloated with stuff that few people will actually use.
Maybe adding MIDI makes no significant difference in the end (for the ones who won't use it). Or maybe it's worth it. The power of open source is that people can contribute with what they want if it's not already there (and IMO this is what makes Godot able to do awesome new creative stuff, not really what's already in core).
In the end I might be closer to Godot dev team, but in no way I can really say "no, it won't happen". I just see no reason to rush this feature when there's many other stuff being worked on right now. If someone in the community want to tackle this: go ahead. As always, PRs are welcome.

TL;DR -- feel free to make a module or a dynamically-linked library (when they come around).

Actually, the game I was going to make used MIDI to communicate with a launchpad mini, that acted as a display for it

The easier it is to add native modules, the easier it will be to add such features. I would see such functionnality added as a module too (and even some functionnality already in the core, actually :p)

I was just searching for the possibility of connecting MIDI devices in godot yesterday. Using a MIDI device for I/O would be a great feature for me as well.

Another use case would be to play music, to reproduce old-school genre of games or reduce memory usage. Many musical games can benefit from it.

MIDI I/O is not the same thing as MIDI playback. The latter is something I would like to see too. Currently you can convert them to Mod files, which Godot is able to play.

Conversion of MIDI to MOD Is (mainly) lossy, can only convert 4-note polyphonic MIDI, and has limited samples, if you wanted to play MIDI files you'd be better putting them through fluidsynth to get actual audio files.

Just made a mediocre MIDI I/O module if someone needs it.

I wrote a thin wrapper around RtMidi as a module if anyone is interested.
[https://github.com/srejv/godot/tree/midi-module](midi-module branch)

Closing for now, as there is the above module now and consesus was that this should be a module, not in core.


spago test hangs in CodePoints  test?

Test hangs
Simply running spago test with spago (0.15.3) / purs (0.13.8) seems to hang on the CodePoints test - when commented out the CodeUnits test completes normally.
(MacBook Pro, 10.15.7)

...or maybe just takes a very long time? (if that's the case maybe a comment "this'll take a while" on the test would be good). I'm leaving it running now to see if it eventually completes

(it is indeed just that it takes a very long time to run)


[Feature Request] Ability to add lastmod tositemap.defaults or to the routeRules

Clear and concise description of the problem
I am currrently moving a small project from nuxt 2 -> nuxt 3 and saw that the @nuxtjs/sitemap module is not maintained. So I moved over to this and everything works fine. Except, I had a lastmod date default in my config, to add this date to all entries.
I could not find a way to archive this in this module. Maybe this is by design, and the lastmod date is not required for sitemaps. If so, I'll add a PR to the docs of this module to point this out.
If you think this option is also useful, I can submit a PR for it.
Suggested solution

I tried to apply it to the sitemap option, but thesitemap.defaults option is not available.

sitemap: {
    hostname: "https://example.com",
    defaults: { lastmod: new Date() },
}

I tried to apply it to the routeRules, but this option is also not available.

routeRules: {
  '/': { sitemap: { changefreq: 'weekly', priority: 1, lastmod: new Date() } },
},

Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Hey @madebyfabian
Thanks for the issue! A great suggestion. I believe this was already supported, just the types hadn't been set up.
Can you upgrade to v1 and let me know how it goes.
You might need to set up the route rule as
  '/about': {
      sitemap: {
        lastmod: new Date().toString(),
        changefreq: 'daily',
        priority: 0.3
      }
    },

@harlan-zw Works like a charm! Upgraded and added the option. Thank you!


Use Literate.jl for examples

This uses Literate to turn the examples into
sections in the documentation.
See https://clima.github.io/OceananigansDocumentation/dev/generated/eady_turbulence/ for inspiration.
julia --project=docs -e "import Pkg; Pkg.instantiate()"
julia --project=docs docs/make.jl

Ends up looking like https://cesmix-mit.github.io/LAMMPS.jl/previews/PR12/generated/snap/ and we can also use mathematical expression and render them correctly


IE 11 Deprecated

Regadring Write-DfsrHealthReport HTML reports, since IE11 is being deprecated, how do I view the HTML Report files?  I've tried Chrome, EDGE with IE11 compatibility mode.
If there isn't a way, is there a list that tell me exactly what all the checks the powershell does so I can try scripting them with other commands?

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

ID: c0fcd72c-6b6b-0d4b-ebde-24f2a549a579
Version Independent ID: 350eeda9-d9b0-3dd6-fa1d-7f43a0352228
Content: Write-DfsrHealthReport (DFSR)
Content Source: docset/winserver2022-ps/dfsr/Write-DfsrHealthReport.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

If IE is uninstalled (which it will be)... these reports can't be viewed (IE11 compatibility mode requires IE 11 to be installed).

@jtsai8585 , .html files can be viewed in msedge browser. right of .HTML file , click open it,

Edge can't view these files when IE 11 is uninstalled since it's no longer supported.

@jtsai8585 . i have disabled IE 11 from the windows feature.just now  I  seen .HTML file using msedge

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Discussion: should ignoreUrls and whitelistUrls affect XHR breadcrumbs?

A user has pointed out that it is unfortunate that XHR breadcrumbs can include requests by 3rd-party scripts.
So, I'm thinking that the ignoreUrls and whitelistUrls config options can impact breadcrumb collection. Our docs encourage you to specify base URLs (e.g. domains), so I feel like this is appropriate ... but at the same time, I'm worried that overloading the meaning of these could also become painful. For example, you want to collect errors from your domain, but don't want to collect AJAX breadcrumbs to a reporting endpoint on your domain – if we go this way, there will be no distinguishing.
Other ideas:

dedicated config option for breadcrumb URLs (could affect both XHRs and nav events)
a callback function for breadcrumbs (like shouldSendCallback) that allows you to mutate outbound data or cancel recording of an individual event

Thoughts?
cc @mattrobenolt @mitsuhiko

As the user, I'd prefer the callback for maximum flexibility.  Regexes can't handle every scenario, and the URL isn't the only attribute we may want to examine.

For the particular use case that led me to this issue, using ignoreUrls to not collect breadcrumbs from XHR requests to that URL would work well for me.  But I agree that it's probably best to keep a distinction in the API between the filter applied to errors and filter applied to XHR breadcrumbs.
For example, in my use case, I WANT to debug everything, but there's certain very frequent API calls that I want to prevent from taking up any of my 20 valuable breadcrumb slots.

I am +1 on a callback for mutating/suppressing outbound xhr breadcrumbs. For our usecase, I'd like to (when available) attach a request id header that's sent back from our api. Currently, we emit a separate breadcrumb with that information, but it'd be nice to have a supported way to do that.

A breadcrumb mutate/suppress callback could be really useful in another way too.  You could use it to create really descriptive breadcrumbs by letting the implementer turn a Sentry-generated "ui.click button#buy-more-stuff-18462816" into "Clicked Buy Now button"

We added breadcrumbCallback, so I think this ticket can be closed.


[Rh-che] Add periodic job

Add periodic job for free-stg cluster.

looking at the vault secret devtools-osio-ci/rhche-periodic-free-stg-creds, it seems that the credentials there are your own, rather then some service account. this poses a problem in cases where people leave the company and their accounts are removed. i think we would rather that the two things are not tied together. what do you think?
other then that, lgtm.

@maorfr I absolutely agree, I've created an account for that purpose but I was said that there is few resources so that account should not be provisioned. This discussion happened on Mattermost service-del channel and as there were no other reactions I though this account can not be provisioned.
I would rather have some testing account, so if you can, please help me with that. I created an account with user rhche-freestg and email<EMAIL_ADDRESS>for that. If you need any action from my side, please let me know.

@ibuziuk can you take a look at this one?

I created an account with user rhche-freestg and email<EMAIL_ADDRESS>for that.

@alexeykazakov @pbergene do we have a possibility to provision yet another user on free-stg ?

@ibuziuk technically I think everyone has access to the cluster, but if I recall correctly there would need to be a specific set of steps in the signup process for the tenant services to provision a tenant on there.

but if I recall correctly there would need to be a specific set of steps in the signup process for the tenant services to provision a tenant on there.

@alexeykazakov any comments on that? Will it be possible to provision<EMAIL_ADDRESS>against free-stg ?

Will it be possible to provision<EMAIL_ADDRESS>against free-stg ?

@xcoulon could you please create such user in prod preview? But please use free-stg cluster instead of east-2a.

From discussions, it was agreed not to test free-stg environment. Closing.


 To import native modules you need to have your own native code set up for both android and ios.
Remove native modules from this import and it will work.

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information, or the list of reproduction steps is not complete. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue that meets the requirements set forth in the template. Thanks!


can't run the project

mvn jetty:run -am --projects cqf-ruler-r4
[INFO] Scanning for projects...
[ERROR] [ERROR] Could not find the selected project in the reactor: cqf-ruler-r4 @
[ERROR] Could not find the selected project in the reactor: cqf-ruler-r4 -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MavenExecutionException
cqf-ruler-r4 % mvn jetty:run -am --projects cqf-ruler-r4
[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[WARNING] 'parent.relativePath' of POM org.opencds.cqf:cqf-ruler-r4:${revision} (/cqf-ruler-r4/pom.xml) points at org.example:cqf-ruler instead of org.opencds.cqf:parent, please verify your project structure @ line 4, column 13
[FATAL] Non-resolvable parent POM for org.opencds.cqf:cqf-ruler-r4:${revision}: Failure to find org.opencds.cqf:parent:pom:${revision} in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced and 'parent.relativePath' points at wrong local POM @ line 4, column 13
@
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project org.opencds.cqf:cqf-ruler-r4:${revision} (/Users/jigneshpatel/development/cqf-ruler/cqf-ruler-r4/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for org.opencds.cqf:cqf-ruler-r4:${revision}: Failure to find org.opencds.cqf:parent:pom:${revision} in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced and 'parent.relativePath' points at wrong local POM @ line 4, column 13 -> [Help 2]
I am getting same error for JDK 8 and JDK 11? AmI suppose to run the program in JDK 11?

Yes, the project requires Java11, we typically use OpenJDK11 and OpenJP: https://adoptopenjdk.net/?variant=openjdk11&jvmVariant=openj9
I just did a fresh clone and ran the mvn jetty:run -am --projects cqf-ruler-r4 and it worked.
Are you also running at least Maven 3.5.3?

The is error is caused by a version of maven that does not support the "revision" parameter.


Update axios dependency

Due to SSRF vuln
See: https://www.npmjs.com/advisories/1594

@ndom91 thanks for your helping! I will release the next version soon


layers: update device tree blob list

Add new device tree blobs to the list to bring in support for new hardware such as merus-amp and more. From https://github.com/raspberrypi/linux/tree/rpi-5.4.y/arch/arm/boot/dts/overlays
Change-type: minor
Signed-off-by: Aaron Shaw<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Based on this overlay_map list there are some deprecated device tree blobs and a number which are platform specific, which are not reflected in our layer.conf
Shall I update this PR to reflect the information there?

@resin-jenkins test this please

@resin-jenkins test this please

@acostach think I missed a backslash on line 202. Will amend.
What did you think about the overlay_map thing?

@acostach think I missed a backslash on line 202. Will amend.
What did you think about the overlay_map thing?

@resin-jenkins test this please

@resin-jenkins test this please

@resin-jenkins test this please

@resin-jenkins test this please

@acostach fixed the missing slash now. Let me know what you think about dealing with the overlay_map list in respect to the different machines?

@acostach fixed the missing slash now. Let me know what you think about dealing with the overlay_map list in respect to the different machines?

@resin-jenkins test this please

@resin-jenkins test this please

thanks @shawaj , I don't know much about what the overlay_map does, I'll have to look into it to give an opinion.

thanks @shawaj , I don't know much about what the overlay_map does, I'll have to look into it to give an opinion.

@acostach I believe the overlay map list basically shows some overlays that are deprecated or only valid on certain devices.
For example, the below shows i2c6 only for bcm2711, a deprecated one and a renamed one:
i2c6 {
		bcm2711;
	};

	lirc-rpi {
		deprecated = "use gpio-ir";
	};

	pi3-act-led {
		renamed = "act-led";
	};

My thought was to remove any deprecated and renamed ones and for the bcm2711 specific ones only have those on the pi 4 builds.
What do you think?

@acostach I believe the overlay map list basically shows some overlays that are deprecated or only valid on certain devices.
For example, the below shows i2c6 only for bcm2711, a deprecated one and a renamed one:
i2c6 {
		bcm2711;
	};

	lirc-rpi {
		deprecated = "use gpio-ir";
	};

	pi3-act-led {
		renamed = "act-led";
	};

My thought was to remove any deprecated and renamed ones and for the bcm2711 specific ones only have those on the pi 4 builds.
What do you think?

@resin-jenkins test this please

@resin-jenkins test this please

@shawaj at this moment only the rpi4 is on kernel 5.4
Is your intention to add these overlays for rpi4 or for which boards?

@shawaj at this moment only the rpi4 is on kernel 5.4
Is your intention to add these overlays for rpi4 or for which boards?

@floion the intention is to add these overlays for all Pi versions

@floion the intention is to add these overlays for all Pi versions

@resin-jenkins test this please

@resin-jenkins test this please

@acostach does the Jenkins bot give any output as to why a build is failing? I clicked on details but it requires a login :-)

@acostach does the Jenkins bot give any output as to why a build is failing? I clicked on details but it requires a login :-)

@shawaj the Pi4 and 400 are on the 5.4 kernel and the others are on the previous version 4.19 where some of the dtbos you add don't exist. But the plan is that we update all boards to 5.4.y

@shawaj the Pi4 and 400 are on the 5.4 kernel and the others are on the previous version 4.19 where some of the dtbos you add don't exist. But the plan is that we update all boards to 5.4.y

Superseded by https://github.com/balena-os/balena-raspberrypi/pull/587

Superseded by https://github.com/balena-os/balena-raspberrypi/pull/587


Parse asterisk's "core-en_US.xml" file to get all the manager function + parameter definitions

How about using the /var/lib/asterisk/documentation/core-en_US.xml file which the asterisk make process generates automatically since asterisk-1.8. It will provide you with all the manager functions for that asterisk version including the parameter it accept. After parsing it once, you only have to verify if the timestamp on that files has changed. It should make 90-95% available and only require manual intervention for some quirky Message and the few external channel drivers like chan-sccp-b chan-dongle chan-lcr etc. If you like you could persuade them to produce a similar file (i would be willing to do this for the chan-sccp-b project would if so requested).
The content of that files looks like:
        <manager name="MixMonitorMute" language="en_US">
                <synopsis>
                        Mute / unMute a Mixmonitor recording.
                </synopsis>
                <syntax>
                        <xi:include xpointer="xpointer(/docs/manager[@name='Login']/syntax/parameter[@name='ActionID'])" />
                        <parameter name="Channel" required="true">
                                <para>Used to specify the channel to mute.</para>
                        </parameter>
                        <parameter name="Direction">
                                <para>Which part of the recording to mute:  read, write or both (from channel, to channel or both channels).</para>
                        </parameter>
                        <parameter name="State">
                                <para>Turn mute on or off : 1 to turn on, 0 to turn off.</para>
                        </parameter>
                </syntax>
                <description>
                        <para>This action may be used to mute a MixMonitor recording.</para>
                </description>
        </manager>

Which should be pretty easy to parse using php. Is't just a suggestion, possibly saving you a lot of maintenance work

Currently pami is used to control any asterisk version, without the need of having it installed in the same box as asterisk. So one box can control multiple asterisk installations, with potentially different versions.
How would that work in this case?

I do see your point, however there are still a number of ways this could be benifital.

Include these xml files into your project and use them if there are no local versions available After checkng the version number of the asterisk box you connected to.
Access an online version if specified server is not "<IP_ADDRESS>"
Or parse it locally on your system, to generate the Message classes for you (might be helpfull).

There are loads of ways of doing this, just mentioning a few.
Just trying to help you out, focussing your attention on the important stuff :-)

Generating classes on demand means parsing an xml file of 500k (for asterisk 10, for example), that's a big DOM and will make pami use more memory, more cpu and have a bigger startup time. Accessing it online will only make things worst (latency, network errors, dns poisoning, and so on). And we can't be sure that user will actually generate and install that file. Even more, we can't be sure it will be syntactically correct.
I think that the only viable way of doing this is to have every xml for every asterisk version at the time of the build, and distribute the auto generated classes, without depending on the files that the user provides (and sparing them the need to generate and make them available). It also means that we know for sure what pami will do once it is started, no matter where it's installed.
Also, as you pointed out, we should autodetect the asterisk version and pray for it to be already supported, or choose a compatible version somehow.
What happens with asterisk versions < 1.8? A lot of them are still around and a lot of users still use pami to monitor them.
I'm still not convinced that this would add substantial value (we already have in place a lot of actions and messages written and tested with a lot of different asterisk versions, besides the unit tests), but will probably add a lot of entropy into how pami will work.. and some other things to be concerned about, making debugging more difficult.
Makes sense?

One more thing: What happens if the xml structure changes? The code will have to support different xml structures depending on the asterisk versions

First of: It was only a suggestion :-)
The XML file is generated directly from the source files, so should be as correct as the sources themselves, can't get closer than that. It is delivered with a DTD file, to verify that the XML at least it is syntactically correct. The file structure has not changed since asterisk-1.6.2 as far as i could tell (from a quick snoop).
The size of the file is large: agreed. There is a lot of information that you don't need, so this can be discarded on the first parse, and cache the result locally (possibly in a denser (even binary) format). You could even split by message an load as required to reduce the memory load. There are ways to deal with that.
I do see your point(s) and a lot of work has already been done, kudos to that (This might have made more sense if i had made this comment back in the day when you started). I thought it might be a nice way to provide thirdparty channel driver information in the same format / manner.
Maybe it can be or will be useful in some other way (for example to verify the currently implemented message against).
I will resort to implementing the chan-sccp-b AMI Messages + Tests manually and post a pull request for them when i have some time.
Thanks for a great project by the way.

Hey I do appreciate the suggestion, and that's why I played a bit with the idea first, and then these questions came to my mind :) Just playing the devil's advocate a bit to see where this leads us.
I think that the more we talk about it, the more details come up and more complexity is added to make it work, and more code is added to support a "simple xml parser" without enough evidence of a clear advantage over a class with a few lines of code (I do agree that we do have a lot of classes lying around, point taken, but nothing more is wrong with that, it just works).
I will close the issue now, but I do look forward to your pull request :) And thanks again for the suggestion, perhaps a working proof of concept might convince me that it's definitely the way to go :)


[move-prover] Fixed hyper edge ordering problem

Motivation
This is an attempt to fix the Move prover problem reported and analyzed in https://github.com/move-language/move/issues/270
Please note that the bytecode translator changes are failing - due to changes that have been made expected result no longer matches the expected output - if the solution is correct we should update expected output.
Admittedly this solution has been crated "by feel" rather than by a super-deep knowledge of the borrow analyzer, but perhaps even if it's not 100% correct, it will inspire a proper solution.
Some observations:

It seems to me that hyper edges in borrowed_by and borrowed_from sets should be reverted, which is currently not the case (hence edge revert operation in consolidate).
The failure in the bytecode translator for the example in the related issue (https://github.com/move-language/move/issues/270) is due to a hyper edge in the borrowed_from set being in the wrong order, which means that considering the bullet above, it's the borrowed_from set should keep the "correct" (from the bytecode translator's perspective) order
The final bit is that in constructing function annotation (in summarize and then construct_hyper_edges) we use borrowed_from edge ("correct" order) to construct a borrowed_by edge (incorrect order), so we have to reverse the hyper edge (but not any edge as before).

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
I don't think we have Boogie backend tests, but with this PR in the test exposing the bug (added in https://github.com/move-language/move/pull/284) no longer crashes the prover.

Thanks! Awesome investigation @awelc ! Your solution looks sensible to me but will let Wolfgang check and approve it.

Superseded by https://github.com/move-language/move/pull/298


Update anime-web-tier-01 for Radarr and Sonarr

Pull request
Purpose
Added sam to anime web tier 01 for Radarr and Sonarr
Approach
If this Pull Request is created to solve a issue explain how does this change address the problem?
Open Questions and Pre-Merge TODOs
Check all boxes as they are completed

[x] Use github checklists. When solved, check the box and explain the answer.

Learning
If you're adding a new Custom Format make sure you follow the Radarr/Sonarr Custom Format (JSON) Guidelines.
Requirements
Check all boxes as they are completed

[x] These changes meet the standards for contributing.
[x] I have read the code of conduct.

@bakerboy448 from my understanding with what RG has said the group spec doesn't play nice with Anime groups


Correct baseVars to match Yii3

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
-

Thanks!


Easily send gifs from Giphy

Be nice if there was a button you can click that searches giphy.com based on text you enter.

seems cool :)

I agree. The only ways I have found to do it so far are pretty clumsy. (Maybe I just haven't found the non-clumsy way.) Cut-and-paste from giphy.com only gets the still image. To get the animation, I have to download to a file, and then drag that file into yakyak.
Anyone know any easier way?


feat: add adv_2022

Codecov Report

Merging #51 (8360dfd) into master (ad0439f) will decrease coverage by 0.06%.
The diff coverage is 99.52%.

:exclamation: Current head 8360dfd differs from pull request most recent head 443eb48. Consider uploading reports for the commit 443eb48 to get more accurate results

@@             Coverage Diff             @@
##            master      #51      +/-   ##
===========================================
- Coverage   100.00%   99.93%   -0.07%
===========================================
  Files           20       24       +4
  Lines         1190     1625     +435
  Branches       221      298      +77
===========================================
+ Hits          1190     1624     +434
- Misses           0        1       +1

Impacted Files
Coverage Δ

solutions/adv_2022_22.py
99.52% <99.52%> (ø)

solutions/adv_2022_24.py
100.00% <0.00%> (ø)

solutions/adv_2022_25.py
100.00% <0.00%> (ø)

solutions/adv_2022_23.py
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


multiple addresses listener: remove legacy interface

Commit Message: multiple addresses listener: remove the legacy interface
Additional Description:
Since the multiple addresses listener is implemented,  all the code moves to use the new multiple socket factories interface, then those old single socket factory interfaces can be removed now.
Risk Level: low
Testing: unittest
Docs Changes: n/a
Release Notes: n/a
Fixes #11184

/assign @mattklein123 also

/assign @mattklein123

@adisuissa you're first pass for this one.  PTAL

actually looks like adi is out today so @KBaichoo can you do a pass today?

@alyssawilk this should probably be stamped by a senior maintainer.


Branch edit student command

Codecov Report

Merging #241 (1483184) into master (ed38f04) will increase coverage by 0.52%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master     #241      +/-   ##
============================================
+ Coverage     58.80%   59.33%   +0.52%
- Complexity     1042     1050       +8
============================================
  Files           161      161
  Lines          3945     3910      -35
  Branches        539      529      -10
============================================
  Hits           2320     2320
+ Misses         1447     1412      -35
  Partials        178      178

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/EditCommand.java
50.35% <28.57%> (+5.23%)
:arrow_up:

.../seedu/address/logic/parser/EditCommandParser.java
92.30% <100.00%> (+24.12%)
:arrow_up:

...va/seedu/address/logic/commands/DeleteCommand.java
60.00% <0.00%> (-25.72%)
:arrow_down:

...du/address/model/assessment/AssessmentResults.java
78.37% <0.00%> (-5.41%)
:arrow_down:

.../main/java/seedu/address/model/person/Student.java
39.02% <0.00%> (+12.19%)
:arrow_up:

.../model/person/StudentHasTutorialNamePredicate.java
100.00% <0.00%> (+66.66%)
:arrow_up:

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 ed38f04...1483184. Read the comment docs.


Make sure all classifiers are seeded by random_state

for reproducibility, all classifiers should be seeded by random_state. In our experiments, the random_state is set to the resampleID. This is not correctly set in all the considered classifiers. The current list to check is
all_classifiers = [
"BOSSEnsemble", "TemporalDictionaryEnsemble", "MUSE", "WEASEL",
"ProximityForest", "ElasticEnsemble", "KNeighborsTimeSeriesClassifier",
#"ShapeDTW",
"RandomIntervalSpectralForest",
"TimeSeriesForest",
"ShapeletTransformClassifier", "MrSEQLClassifier", "RocketClassifier"
]

We found that all algorithms are seeded with random state and that there is a test for this: check_fit_idempotent
We found that in ensembles, all ensemble members may be seeded with the same seed and hence may not exhibit any variation.

@dicicch any chance that you can push the code you worked on during the sprint to make sure that the different ensemble members aren't seeded by exactly the same seed?


GH#1102: Fix spelling errors

Product Version(s):
Calico (3.24, 3.24.2, 3.25, 3.26), Calico Enterprise (3.15, 3.16, 3.17, 3.18), and Calico Cloud (3.14.0, 3.16.0, 3.16.1, 3.17)
Issue:
#1102
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:

A couple of recently merged commits fixed spelling items that I ran across but didn't maintain table alignment. I personally don't care about alignment, but I had attempted to retain proper alignment and have thus retained the alignment corrections in this PR -- I've tagged those changes with distinct commit messages of whitespace: -- I'm more than happy to drop them.
Merge checklist:

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

@tigera/docs

Thanks @jsoref. We'll need some time to comb through this, but will be back to you after a while.

@jsoref I'm sorry we let this one go stale. We've since brought in another tool, Vale, and have started picking away at these and other inconsistencies.
If you care to address any of those that remain in revised PR, please feel free. For now I'll close this issue.


Fix/snapshot staging is not atomic

resolves #1884
Description
Previously, snapshots built a staging table in two discrete queries (one was a create table as and the other was an insert). This meant that if source data changed between the two statements (ie. if new data was loaded) then dbt would generate an asymmetric set of inserts and updates.
This manifested as records in the snapshot table becoming invalidated (the dbt_valid_to column was set to a not-null value) but no new record was inserted for the corresponding change.
This PR combines the two queries (which is actually how this previously worked) to ensure that both inserts and updates are generated from a consistent view of the source table.
Some notes:

This was only an issue on Snowflake and BigQuery. Pg/Redshift support transactions, and we do run this code inside of a transaction, so both queries would have an identical view of source tables throughout the transaction. Snowflake's auto-committing of DDL terminates the transaction, and BigQuery does not support transactions at all, so each query could have a different view of source data on those databases.
This is a race condition / temporal issue, so the automated test here hacks the real-world scenario in a repeatable and non-brittle way.

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

good call @jtcohen6 - just opened up a new issue to dig into this here: https://github.com/fishtown-analytics/dbt/issues/2392
It's definitely worth reducing the number of queries like this that dbt runs. This will be a performance benefit, and it will make the logs way easier to consume too :)
Merging this one - thanks for the help everyone!

@drewbanin We are running a dbt snapshot on top of Snowflake. We're often encountering a Duplicate row detected during DML action error when running the snapshot, despite the fact that the source table does not have any duplicate rows. I'm wondering if it's caused by the issue described in this PR.
The source for the snapshot is a Snowflake table that is synced from a production SQL table using Fivetran. I'm not super-familiar with the inner workings of snapshots, but one hypothesis I have is that Fivetran is syncing the row in question while the snapshot is running. The image below shows the snapshot data for the row in question - can you tell from the data whether this hypothesis might be correct?

The next question is: How could my hypothesis be correct, given that this PR makes snapshots atomic? Well, I learned through painful experience that CTEs are not guaranteed to only evaluate once during a given query execution. So I wonder if snapshot_query could be evaluated twice in certain scenarios: once by insertions_source_date and once by updates_source_data.
If my hypothesis is correct, my guess is that your suggestion in #1884 to combine insertions_source_date and updates_source_data into a single query would resolve it.
Thanks for your time :)


Documentation: Update network (Key Concepts) page

Signed-off-by: Waleed Mortaja<EMAIL_ADDRESS>Type of change

Documentation update

@denyeart Thank you for your fast response!

In this case some of the text PRed has been refactored out of main already, so this one is ok to go directly to release-2.2.

Is there a way to check if a text has been refactored out of main already or not yet (i.e., how to decide to PR against release-2.2 or main) ?

You could do a separate PR against main branch to ensure the updates applicable to main get into every future release.

I will do that in a couple of hours. By the way, should I mention this PR there?

Happy to help :)


The mod don't take effect

I used forge-1.16.5-36.2.2

Could you elaborate on this? I'm testing it out right now on the same version and the illagers do seem to spawn with armor, unless the problem is with something else.
Also sorry for the late response.

Could you elaborate on this? I'm testing it out right now on the same version and the illagers do seem to spawn with armor, unless the problem is with something else.
Also sorry for the late response.
One factor that could lead to illagers not appearing to wear armor would be if you have a resource pack that changes the illagers model (ie fresh animations), as this mod has to override their models.

I only installed this mod, not even optifine installed

Can you verify if the mod is installed correctly and if the game recognizes its existence? If it is, could you send your config files in a pastebin.  Also the version of the mod you have would help as well.
Also do you mean the illagers just don't spawn with armor at all or the mod just isn't recognized?

Can you verify if the mod is installed correctly and if the game recognizes its existence? If it is, could you send your config files in a pastebin. Also the version of the mod you have would help as well.
Also do you mean the illagers just don't spawn with armor at all or the mod just isn't recognized?

is displayed in the mod list, but it does not take effect in the game.
forge version is 1.16.5-36.2.2
mod version is 1.0.4
As the above friend(Diycnel) said, it's not just me

Does the pillager spawned with this command
/summon pillager ~ ~ ~ {ArmorItems:[{},{},{},{id:"minecraft:iron_helmet",Count:1b}]} render the iron helmet on hand?

Does the pillager spawned with this command
/summon pillager ~ ~ ~ {ArmorItems:[{},{},{},{id:"minecraft:iron_helmet",Count:1b}]} render the iron helmet? You could also try setting the chances for illagers to spawn with armor in raids to 1.00 (reset the config after testing) and seeing if they render any armor.
If either one works and the illagers render armor in either of those instances it most likely means you just have bad rng

Well, it's settled. So is this a configuration problem? The default configuration isn't enough for them to wear armor


[#1414 #1421] parse metadata from esri shapefile xml

This is to address #1414 and #1421

@hyi Can you review this?

+1

@hyi  Thanks for giving +1


Create the Login functionaity for the project.

Like every other application, this also requires login functionality. The basic process of this functionality for a REST API is to first check the user credentials and return an error response if they are wrong. On successful verification of the user's credentials, the server should return a secret (JWT token in this case) to the User. The same secret will same be used in the future to verify which user is raising a request to the server.

The Task
The Login functionality will be written in a function in the file controllers > auth.controller.js. The function attributes are the default attributes of a request handling function, i.e., (req, res).
The request structure can be found here. The Response structure is also present here.
For checking the password, you will have to first convert the password into hash, the hash_password function present in the utils > password.js should help in converting password into hash. Now you will have to match this hash to the password stored in the database.
We will also be applying ReCaptcha validation in login, for that, you can use the verifycaptcha function present in the utils > recaptcha.js.
Once the user is successfully verified, you will have to create a User Secret, which is actually a JWT token. For creating this JWT token, you have to use the getJwt function present in the utils > password.js and the structure of payload object for this function is this:
{
    "id": "the _id of the MongoDB object of user",
    "email"<EMAIL_ADDRESS>}

To make returning a response easier, some pre-defined response functions according to the status code of the response are written in utils > responseCodes.js. It is recommended to use them for returning successful or error response. The message passed to this function should be something related to the response, as they are for developer reference, the data passed to this function is the same as present in the Response structure present in this file.

For any doubts or difficulty, you can ping me on discord, I'll try to help you out as soon as possible.
All the Best!!

claim

Claim

Assigned to @GammaMicrowave. All the Best!!

claim


sdktypes rewrite

This is the anticipated sdktypes rewrite.

Refactor each type not to be an actual type and not just an alias. This allows to add methods on these types.
IDs and Symbols can now be compared directly without .String()-ing them. Objects need to be compared using the .Equal(other) method though.
Validations are part of the type now, not a member function, which was wrong.
Get rid of sdktypes.Name. Everything "name-ish" is not a Symbol.
Get rid of all icky oneofs in proto.

How to review:

Start by looking at the proto first, not much changes there (mostly the oneofs good ridance).
Then move to sdktypes: first read doc.go, then go over object.go, id.go, and str.go  by that order.
Look how its being used everywhere in sdktypes.
Go over the rest of the code which is just using the new sdktypes instead of the old.

"Enjoy".

Still haven't reviewed everything,
I think @tebeka's comment sums the main point here
we have a tight coupling between the API (protos) and the business models we use inside (the sdk types)
maybe we should think of business models types for internal use and only provide an sdk wrappers for clients


[Feature request] Implement Empty Canvas Option for Text-Only Designs

Platforms
Android, iOS
Description
Please include the following feature:
When a user creates a text design, the editor should initially display an empty canvas with the user's specified width and height. This will allow the user to easily edit their text, stickers, and other elements. Currently, I am experiencing issues with this functionality.
Why
if the user only create text design then there is no option for only text design with empty canvas

I'm not sure if I understand you correctly, but you want to open the editor without a background image where you can add text, emojis and stickers, right?
If so, you can follow this example. Just remove the onAfterViewInit part that initializes the editor with a sticker. Then you can add text, stickers and emojis without background. You can set the width and height in _createTransparentImage.

Upon opening the editor, I would like to be presented with an empty canvas.
The dimensions (height and width) and background color of this canvas
should be customizable, aligning with the standard behavior of other
editors.

Message ID: @.***>

Okay, so I understand you correctly already. As I said here this is already possible. You can follow the example which I posted in my previous answer.

thank you
Thank you for the clarification. I appreciate your time and patience in
explaining the process to me. I will refer to the example you provided in
your previous answer and give it a try.
Best,
Ikram Ullah
On Mon, Jun 24, 2024 at 1:53 PM Alex Frei @.***> wrote:

Okay, so I understand you correctly already. As I said here
https://github.com/hm21/pro_image_editor/issues/127#issuecomment-2182556878
this is already possible. You can follow the example which I posted in my
previous answer.
—
Reply to this email directly, view it on GitHub
https://github.com/hm21/pro_image_editor/issues/127#issuecomment-2185961204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVGV4TWCJNQZNOKZGVUXHJLZI7NAZAVCNFSM6AAAAABJVVNHRGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBVHE3DCMRQGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Scheduler: Enable cron style interval settings

Environment

Python version: n/a
Nautobot version: 1.3?

Proposed Functionality
As Tom, the network operator, I would like to have more complex intervals in the scheduler.

Use Case
I have some jobs which need to be run at different intervals (e.g. 5min) then currently possible.

Database Changes
Possibly enhancing the model with the cron style fields.

Implementation
Celery supports already crontab style schedules: https://docs.celeryproject.org/en/stable/userguide/periodic-tasks.html#crontab-schedules,
so I guess just having an "advanced" interval, which has free text fields for minute, hour, day_of_week, day_of_month and month_of_year could be enough, and of course support in the backend for this.
Thanks,
Tom

Thanks for submitting this feature request @sirtux!
Feature sounds pretty straightforward and can potentially be implemented without being a breaking change on the interval handling execution.
It is now working its way into prioritization.


Order numbers are different due to soda/snack sales

At the moment the numbers that the people that order their tostis see is not the same as the number shown in the admin screen if there have been soda or snack sales before they have ordered. Figuring out a way to ensure these numbers stay the same (perhaps by just giving soda/snack sales roman numerals or no numbers at all) might be convenient, i.e. when people come to pay and communicate their order number to make sure the correct order is set to "paid".

With the new tabs, order numbers are not at all related any more. But since we have last names just now as admin, I think it's best to just do it by name only anyway.

At least imo the index in 'to be processed' is a useful estimation of waiting time, whereas replacing that with non-sequential numbers that correspond to what the buyers see would be chaotic for us


SDK error when creating pipeline from C++

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 16.04.3

Kernel Version (Linux Only)
4.13.0-36-generic

Platform
Compulab FitPC

SDK Version
2.10.1

I'm writing a ROS node for embedding Intel RealSense D435 perception into a robotics project.
I installed the SDK 2 following the official guide (I encountered some problems with sudo apt-key adv --keyserver keys.gnupg.net --recv-key 6F3EFCDE because GPG key could not be retrieved, but I managed to manually download it and add to keyring).
Aside from some glitches (some warning and error popups), RealSense Viewer is working properly now.
After that, I wrote a very simple C++ test node to try fetching frames following the "rs-capture.cpp" example provided (I already updated my CMakeLists.txt to correctly compile this node):
#include <ros/ros.h>
#include <librealsense2/rs.hpp>

int main(int argc, char** argv)`
{
    // ROS initialization
    ros::init(argc, argv, "realsense_driver");
    ros::NodeHandle nh;
    ros::NodeHandle np("~");

    // Device initialization
    rs2::pipeline pipe;
    pipe.start();
    ROS_INFO("[REALSENSE] Streaming pipeline started");
    while (ros::ok())
    {
        rs2::frameset frames = pipe.wait_for_frames();
        rs2::depth_frame depth = frames.get_depth_frame();
    }

    return 0;
}

After issuing the usual catkin_make command I received many error messages (I don't report them here for brevity), so I reduced the body to just rs2::pipeline pipe; to better identify the problem and that's what I receive:
CMakeFiles/realsense_driver.dir/src/RealSenseDriverNode.cpp.o: In function `rs2::error::error(rs2_error*)':
/usr/include/librealsense2/hpp/rs_types.hpp:76: undefined reference to `rs2_get_error_message'
/usr/include/librealsense2/hpp/rs_types.hpp:78: undefined reference to `rs2_get_failed_function'
/usr/include/librealsense2/hpp/rs_types.hpp:78: undefined reference to `rs2_get_failed_function'
/usr/include/librealsense2/hpp/rs_types.hpp:79: undefined reference to `rs2_get_failed_args'
/usr/include/librealsense2/hpp/rs_types.hpp:79: undefined reference to `rs2_get_failed_args'
/usr/include/librealsense2/hpp/rs_types.hpp:80: undefined reference to `rs2_get_librealsense_exception_type'
/usr/include/librealsense2/hpp/rs_types.hpp:81: undefined reference to `rs2_free_error'
CMakeFiles/realsense_driver.dir/src/RealSenseDriverNode.cpp.o: In function `rs2::error::handle(rs2_error*)':
/usr/include/librealsense2/hpp/rs_types.hpp:127: undefined reference to `rs2_get_librealsense_exception_type'
CMakeFiles/realsense_driver.dir/src/RealSenseDriverNode.cpp.o: In function `rs2::context::context()':
/usr/include/librealsense2/hpp/rs_context.hpp:84: undefined reference to `rs2_create_context'
/usr/include/librealsense2/hpp/rs_context.hpp:84: undefined reference to `rs2_delete_context'
CMakeFiles/realsense_driver.dir/src/RealSenseDriverNode.cpp.o: In function `rs2::pipeline::pipeline(rs2::context)':
/usr/include/librealsense2/hpp/rs_pipeline.hpp:349: undefined reference to `rs2_create_pipeline'
/usr/include/librealsense2/hpp/rs_pipeline.hpp:349: undefined reference to `rs2_delete_pipeline'
collect2: error: ld returned 1 exit status

It seems a problem inside the SDK, but I really cannot understand why it cannot find its own structures and types... does anyone knows what I'm missing here?
Thanks,
Best regards

Hi @GuidoBartoli ,
These look like regular linkage issues, telling you that the linker can't find the implementation of these functions (rs2_*). It looks like you forgot to tell CMake to link the realsense2 library.
Please make sure that you properly installed librealsense, make sure that /usr/lib or /usr/local/lib has realsense2.so. You might want to try to run ldconfig to tell the linker to refresh itself.
On another note- we provide a ROS wrapper, you might want to have a look at it.

Wow, what a blunder from me... I forgot to link the node to library!
add_executable(
    realsense_driver
    src/RealSenseDriverNode.cpp
)
add_dependencies(realsense_driver ${catkin_EXPORTED_TARGETS})
target_link_libraries(
    realsense_driver
    realsense2
    ${catkin_LIBRARIES}
)

Thanks for pointing it out! :)
I will take a look to the ROS wrapper, I did not know it existed.

@GuidoBartoli Hello, i have the same error in Eclipse. I've binded all libraries. But still has it problem. Did you update your Makefile?

@dmitryshendryk Yes, I edited the CMakeLists.txt file as I reported in my last post. The node executable was missing the linking to the realsense2 library.


feat(compute): modify region tag to make them unique and as similar as folder name

…s folder name
Description
Fixes #333070003
Note: If you are not associated with Google, open an issue for discussion before submitting a pull request.
Checklist
Readiness

[x] Yes, merge this PR after it is approved
[ ] No, don't merge this PR after it is approved

Style

[x] My sample follows the rules described for Terraform in the Effective Samples style guide
[x] My sample follows the other requirements and best practices in the Contributing
guide

Testing

[x] I have performed tests described in the Contributing guide:

[x] Tests pass: terraform apply
[x] Lint pass: terraform fmt check

Intended location

[x] Yes, this sample will be (or already is) included on cloud.google.com
Location(s):

[ ] No, this sample won't be included on cloud.google.com
Reason:

API enablement

[ ] If the sample needs an API enabled to pass testing, I have added the service to the Test setup file

Review

[ ] If this sample adds a new directory, I have added codeowners to the CODEOWNERS file

/gcbrun


Fixes mocks, tests and adds circleCI configuration

Description:
This PR fixes figures mocks and tests and integrates circleCI.
JIRA:
https://edlyio.atlassian.net/browse/EDLY-2311

@asadmanzoor93 I have looked through the mocks and it would take tedious effort to further clean them up as with juniper there are a lot of changes that are dependent on each other in some way

This PR is passed according to the Acceptance criteria. Merging it


Role permission restrictions can't be updated

What I'm trying to achieve
I am trying to restrict a user to certain permissions.
Steps to reproduce the problem

Pull latest dashboard and core versions.
From home Navigate to Settings > Staff Members > "User" > Permissions
Uncheck "user has full access to the store". Uncheck specific permissions (i.e. I unchecked the first 2 permissions)
Save

What I expected to happen
When I refresh the page the user appears to have full permissions while I removed 2 permissions.
Screenshots

System information
Operating system: Heroku
Browser: Chrome

You probably tried to change superadmin's permissions, which is not going to work. But it seems that UI should indicate that this user is superadmin.

@dominik-zeglen this is possible. The main issue is I can't invite new staff users to the application from the dashboard interface. It trows back an error.

Maybe there is something wrong with my email configuration. However it looks ok to me.
{'EMAIL_FILE_PATH': '', 'EMAIL_HOST_USER'<EMAIL_ADDRESS>'EMAIL_HOST_PASSWORD': 'xxxxxxxx', 'EMAIL_HOST': 'smtp.gmail.com', 'EMAIL_PORT': 587, 'EMAIL_USE_SSL': True, 'EMAIL_USE_TLS': False, 'EMAIL_BACKEND': 'django.core.mail.backends.smtp.EmailBackend'}

Can you post error that comes from the API?

it seems a permission issue. But this is quite surprising given that I am using a "SUPERUSER" account. Check it out below.

Is  the API suppose to trigger a notification email if successful?

Yes.
It seems that dashboard is doing okay and the problem lies within the core. Which version do you use (release/hash of the last commit)?

@dominik-zeglen I pulled the latest commits of core, dashboard and storefront. Email is still not working I get the ERR above.

Go to the 'network' tab in browser's inspector, find the request with this mutation and post response here, we'll see what's happening here 🤷♂

My fault.. looks I did not set ALLOWED_HOST variable in the environment with the correct subdomain. Thanks!