Metro 2033 redux - text is black

As the title says, the text turns up in black blocks?
http://i.imgur.com/TM5byU8.jpg

This also occurs for Metro Last Light Redux. I'm on Ubuntu 14.10 64bit with a GTX 680.
Note that with the 346.22 drivers, Metro Redux (both games) have severe lighting issues (white light seems to pervade the world). I'm unsure if that has anything to do with this however, just informing you in case.
I've also tested the 340.65 drivers (no white light issue), and the GLXOSD text is still black. If there's anything you want me to test, or need output, guide me on it and I'll try to provide it for you. Keep up the good work :)

@myromance123: you are seeing this bug: https://devtalk.nvidia.com/default/topic/795600/linux/metro-redux-driver-346-xx-very-high-quality-glitch/
offtopic: Steam (in Beta) got FPS display and with corner selection today, it's not GLXOSD, missing hardware sensors and such, but it helps.

The benchmark mode has an included FPS counter, or did I misunderstand what you said?

You are correct that it has an FPS counter in the benchmark. However, for my purposes, said counter is small, and often blends in with the background. GLXOSD allows me to resize the fonts to mega size and appropriately position the counter, which is what I need when I record the benchmarks :)

Also happens for me in Middle Earth: Shadow of Mordor.

To further add, it now also happens for Company of Heroes 2. It's black boxes where the text should be. This is on the latest Nvidia 355.06 drivers on Ubuntu 15.04 64bit.

Might be a problem with lighting. Possible fix: call glTexEnvi(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_DECAL) before rendering.

Just tried this, no change.

diff --git a/src/glinject/syncopengl.hpp b/src/glinject/syncopengl.hpp
index 55e89d1..3bbcee3 100644
--- a/src/glinject/syncopengl.hpp
+++ b/src/glinject/syncopengl.hpp
@@ -251,3 +251,5 @@ SYNCHRONIZE_FUNCTION_VOID_RET(glGenTextures, (GLsizei n, GLuint *arrays), (n, ar
 GLOBALLY_SYNCHRONIZE_FUNCTION_VOID_RET(glBindBuffer,(GLuint target, GLuint buffer), (target,buffer))
 GLOBALLY_SYNCHRONIZE_FUNCTION_VOID_RET(glBindVertexArray, (GLuint array), (array))
 GLOBALLY_SYNCHRONIZE_FUNCTION_VOID_RET(glUseProgram,(GLuint program), (program))
+GLOBALLY_SYNCHRONIZE_FUNCTION_VOID_RET(glColor4f,(GLfloat red, GLfloat green, GLfloat blue, GLfloat alpha), (red, green, blue, alpha))
+GLOBALLY_SYNCHRONIZE_FUNCTION_VOID_RET(glTexEnvi,(GLenum target, GLenum pname, GLint param), (target, pname, param))
diff --git a/src/glxosd/OSDInstance.cpp b/src/glxosd/OSDInstance.cpp
index 952aac9..8d03bf4 100644
--- a/src/glxosd/OSDInstance.cpp
+++ b/src/glxosd/OSDInstance.cpp
@@ -239,6 +239,9 @@ void OSDInstance::render(unsigned int width, unsigned int height) {
 	rgl(GetIntegerv)(GL_ELEMENT_ARRAY_BUFFER_BINDING,
 			&elementArrayBufferBinding);

+    rgl(Color4f)(1,1,1,1);
+    rgl(TexEnvi)(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_DECAL);
+
 	renderText(width, height);

 	//Revert buffer states

I'm wondering if this could be an issue with GL3 core contexts. I see that GLXOSD renders using the fixed pipeline, which wouldn't work in a GL3 core context.

I'm wondering if this could be an issue with GL3 core contexts. I see that GLXOSD renders using the fixed pipeline, which wouldn't work in a GL3 core context.

In https://github.com/skeeto/opengl-demo , a GL3 core application, GLXOSD seems to work.

I see that GLXOSD renders using the fixed pipeline

It actually doesn't, I found the code code that sets up a shader.

I'll try searching for the cause this evening using VOGL later today now that I have Metro.

Thanks @nickguletskii ! I'm almost certain that it's due to the texture being corrupted or loaded incorrectly. I can get a color to show up if I change the fragment shader to return a solid color.
Fixed function states like GL_TEXTURE_ENV_MODE shouldn't make any difference, since we're rendering with shaders.

I had a problem with Kerbal Space Program where it would replace my textures with its own. They seem to have hard-coded texture ids. This, however, may not be the case here.

Found the problem. I will try to deploy a fix tomorrow. I wasn't resetting the sampler (glBindSampler).

You were correct, resetting sampler state fixed the bug for me.

diff --git a/src/glxosd/OSDInstance.cpp b/src/glxosd/OSDInstance.cpp
index c157a9c..a996245 100644
--- a/src/glxosd/OSDInstance.cpp
+++ b/src/glxosd/OSDInstance.cpp
@@ -223,7 +223,7 @@ void OSDInstance::render(unsigned int width, unsigned int height) {
 	//Memorise buffer states
 	GLint pixelUnpackBufferBinding = 0, arrayBufferBinding = 0, activeTexture =
 			0, textureBinding2D = 0, vertexArrayBinding = 0,
-			elementArrayBufferBinding = 0;
+			elementArrayBufferBinding = 0, sampler = 0;
 	rgl(GetIntegerv)(GL_PIXEL_UNPACK_BUFFER_BINDING,
 			&pixelUnpackBufferBinding);
 	rgl(GetIntegerv)(GL_ARRAY_BUFFER_BINDING, &arrayBufferBinding);
@@ -233,10 +233,14 @@ void OSDInstance::render(unsigned int width, unsigned int height) {
 	rgl(GetIntegerv)(GL_ELEMENT_ARRAY_BUFFER_BINDING,
 			&elementArrayBufferBinding);

+	rgl(GetIntegerv)(GL_SAMPLER_BINDING, &sampler);
+	rgl(BindSampler)(0, 0);
+
 	renderText(width, height);

 	//Revert buffer states
 	rgl(ActiveTexture)(activeTexture);
+	rgl(BindSampler)(0, sampler);
 	rgl(BindTexture)(GL_TEXTURE_2D, textureBinding2D);
 	rgl(BindVertexArray)(vertexArrayBinding);
 	rgl(BindBuffer)(GL_PIXEL_UNPACK_BUFFER, pixelUnpackBufferBinding);

Should be fixed in version 2.4.0.

I can confirm this fixes the issue in Shadow Of Mordor, Company of Heroes 2 and Metro Last Light Redux.
Tested on Ubuntu 15.04 64bit with a GTX 680 2GB on Nvidia 355.06 drivers.


Corrected HTTP timeout env var

Signed-off-by: Jens De Temmerman<EMAIL_ADDRESS>
As per https://keda.sh/docs/2.2/operate/cluster/#http-timeouts, the correct env var should be KEDA_HTTP_DEFAULT_TIMEOUT
Checklist

[X] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] A PR is opened to update KEDA core (repo) (if applicable, ie. when deployment manifests are modified)
[ ] README is updated with new configuration values (if applicable)

@jensdt Can you fix DCO please?

Oh, didn't realize the e-mail address mismatch. Should be ok now!


Conda environment - no CUDA

Thanks for making this available, just a few comments on #installation
I've been trying to get the CONDA environment to work on my systems,
If I use the default environment I get the following errors:
Colossalai should be built with cuda extension to use the FP16 optimizer
If you want to activate cuda mode for MoE, please install with cuda_ext!
/proj/berzelius-2021-29/users/x_arnel/.conda/envs/FastFold/lib/python3.8/site-packages/torch/autocast_mode.py:162: UserWarning: User provided device_type of 'cuda', but CUDA is not available. Disabling
  warnings.warn('User provided device_type of \'cuda\', but CUDA is not available. Disabling')

I guess this is due to that the PyPI version of colossalai is not compiled with CUDA, see https://colossalai.org/download
Instead I had to do a postinstall:
pip install colossalai==0.1.6+torch1.11cu11.3 -f https://release.colossalai.org (perhaps this can be included in the .yml file ?)
but that created problems so I had to recompile which works after some minor tweaks (wrong nvcc paths and environmental variables,)

Thank you for your suggestion. FastFold currently does not depend on cuda_ext of colossalai. But FastFold does rely on nvcc which is compatible with the installed torch. You can check with bellow command:
$ python -c "import torch;print(torch.version.cuda)"
11.5
$ nvcc -V
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Thu_Nov_18_09:45:30_PST_2021
Cuda compilation tools, release 11.5, V11.5.119
Build cuda_11.5.r11.5/compiler.30672275_0


add Nord Anglia International School Hong Kong

official URL: https://www.nordangliaeducation.com/nais-hong-kong
a URL of a page on the official website where a long-term (>1 year) IT related course is offered by the university: https://www.nordangliaeducation.com/nais-hong-kong/academic-excellence/secondary/computing
a URL of a page showing that the university recognizes the domain which I am submitting as an official email domain for the enrolled students: https://www.nordangliaeducation.com/nais-hong-kong/contact-us , a contact email address at the footer section

@HelloWorld-er Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.

@HelloWorld-er Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start requesting the licenses in about an hour, to let the changes to propagate through our system.

Thanks! By the way, I have requested a license just one hour ago, but it's after 8 hours you merged it. Does it matter that I did not request it in one hour just after you merged it? Because I am in Hong Kong now, I did not check it at the time.


Currently does not support nested groups!

As I assume it will take a little longer until the feedback from my premier support ticket goes via the right channels and ends up here, I'll take this shortcut (so it might help others): At the moment, the What If tool does not support nested groups: if a user is in a group and that group is member of another group that is used in a conditional access policy, then What If does not correctly display the effect of that policy to the user (while Conditional Access by itself has no problems with nested groups).
Not something anyone would suspect, but I have been told MS is working on it.. :-)

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

ID: ff3f7178-bbd2-cff2-28a4-76a169823522
Version Independent ID: 6cd796b0-a31f-bc36-ee50-23af9e950fd0
Content: What is the what if tool in Azure Active Directory conditional access?
Content Source: articles/active-directory/active-directory-conditional-access-whatif.md
Service: active-directory
GitHub Login: @MarkusVi
Microsoft Alias: markvi

Hi @MohitGargMSFT, please assign this to me.

HI @SaschaSeipp, thanks for bringing this up!
I have added a note to the article.
#please-close

Thanks for adding the note, I'm sure that helps others - and hopefully it can be removed soon when the limitation is gone.. :-)


Memory issue during testing phase with custom dataset for semantic segmentation

I’m encountering a memory issue during the testing phase of a semantic segmentation task on a custom dataset. The training phase proceeds without any issues, but during testing, the memory usage continuously increases until it overflows, leading to an error: RuntimeError: DataLoader worker (pid xxxxx) is killed by signal: Killed.
I suspect that the aug_transform in the testing script might be the cause of this issue. I’ve tried to disable it by directly deleting it, but then the test errors out with AttributeError: 'ConfigDict' object has no attribute 'aug_transform', forcing me to add it back.
How can I properly disable the aug_transform without causing this error?
Alternatively, what might be the real cause of the memory overflow during testing?
Any guidance would be greatly appreciated.

New Progress:
I am using the script semseg-pt-v3m1-0-base . I have removed the data augmentation operations from the aug_transform in the testing part of this script, leaving only an empty list []. As a result, there is no memory overflow during testing. However, the test results are very poor, with several categories having an IOU/accuracy of 0, whereas the training process is normal.

Left one for inference, don't make it empty.

@Gofinge  I also stuck by this and I found that reducing aug_transform is the only way.

@Gofinge I also stuck by this and I found that reducing aug_transform is the only way.

Yes, determine the length of aug_transform by your computing resouce.


Will not start for the last 1-2 weeks

I like to come back sometimes to test and see if finally is stable, but now is worse than ever, I start it, add my credentials and crashes! Do not even have time to use it. Do you guys actually test this on GNU/Linux? I am using Arch and something broke in the last 2 weeks that I test it. I did remove ~/.cache/discordo and reinstalled no luck, also deleted the config.

Have you ever tried to see the log files, Arch user?

(Fellow Arch user here, might be able to help)

I did remove ~/.cache/discordo and reinstalled no luck, also deleted the config.

~/.cache/discordo contains the logs that discordo outputs. Can you post them here so we have something to work with?

@jumps-are-op smart ass you? don't you think I already had check that?

@r3k2 You never said you checked the logs, I thought you didn't check it.
No need for that kind of bad language here, Arch user.

There is no reason for @r3k2 to open an issue for a project that the maintainer is making completely out of his free time, being impolite and having higher standards that is provided purely out of patience.
You said discordo crashes, where is the crash information? If you get a completely empty output, run strace discordo and paste the contents here.
Please be polite when opening issues on open source projects.

I also experienced this issue on mac. The log files are also empty. Since then I stopped using Discordo and ended up deleting it some time after. I'm glad someone posted an an issue about it, but I do agree that this is no place for that kind of response.

I can also confirm that this is broken on Arch. But for me, it's simply not opening. I have a log file:

2024/02/18 01:44:42 ws.go:16: Gateway error: websocket: close sent
2024/02/18 01:44:42 main.go:22: websocket closed, reason: 4004 Authentication failed.

Fix, explanation??

Probably duplicate of #339

So can I fix it somehow then to where discordo works?

So to summarise this issue: an extremely bad attitude from the get-go, no logs and when asked for them, one is called a "smart ass"? Might I suggest you try respecting other people before asking us to "respect hacker ethics" as per your bio?
Token authentication is not hard and really does not warrant this kind of disrespect for a completely free project. For anyone else stumbling upon this and simply wanting to log in:

Browse to https://discord.com/app and sign in (if not already)
Open the dev tools with F12 and click "Console"
Paste (after taking the time to read and understand that it is exposing the getToken() method buried inside Discord's Webpack bundle) (webpackChunkdiscord_app.push([[''],{},e=>{m=[];for(let c in e.c)m.push(e.c[c])}]),m).find(m=>m?.exports?.default?.getToken!==void 0).exports.default.getToken(), then hit Enter
Copy the token output.
Run discordo --token <the token in question>.

Experiencing the same on Windows 11 Enterprise version 23H2 (build 22631.3593)
I have tried the above, and it asked me to log in with an email again.
I thought maybe it couldn't log in because the token was not stored correctly in the key-ring and deleted the discordo entry as a troubleshooting step and tried logging in again. Still fails.
Not sure how to collect a stack trace on windows, however; /appdata/local/discordo/logs.txt has collected these two lines:
2024/05/21 11:34:58 ws.go:16: Gateway error: websocket: close sent
2024/05/21 11:34:58 main.go:28: websocket closed, reason: 4004 Authentication failed.

Unsure if this is because discord added support for physical 2-step authentication or not. I am NOT using a security key though.


Is this gem actively supporting?

Hey guys. is this gem supportable with latest version of spree?

I really would love to know about. It seems it is not. I'm having a lot of troubles to make this work with spree.

@delblues you could use railsdog/spree_marketplace.

@nghiapdx
Will give a check, it seems that is the same code as here....tks anyway


Collect inhibitor and high not enough space on RHEL 6.

For inclusion into summary output and saving into leapp_inhibitors artifact.

Sample RHEL 6-7 analysis with EXTREME (inhibitor) and HIGH not enough space:
RUNNING HANDLER [infra.leapp.analysis : Display inhibitors] ********************
ok: [satrhel6-1.localdomain.local] => {
    "results_inhibitors.stdout_lines": [
        "preupg.risk.EXTREME: The /usr/ directory is located on a separate partition. The in-place upgrade is not possible."
    ]
}
Thursday 25 April 2024  11:04:01 +0000 (0:00:00.052)       0:04:01.701 ********
 [started HANDLER: infra.leapp.analysis : Display errors on satrhel6-1.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Display errors] ************************
ok: [satrhel6-1.localdomain.local] => {
    "results_errors.stdout_lines": [
        "preupg.risk.HIGH: There is not enough free space in /usr for migration!",
        "preupg.risk.HIGH: There is not enough free space in /var for migration!",
        "preupg.risk.HIGH: Not enough free space on the /boot. Release more space for upgrade."
    ]
}
Thursday 25 April 2024  11:04:01 +0000 (0:00:00.067)       0:04:01.769 ********
 [started HANDLER: infra.leapp.analysis : Preupgrade analysis report is done on satrhel6-1.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Preupgrade analysis report is done] ****
ok: [satrhel6-1.localdomain.local] => {
    "msg": "The preupgrade analysis report generation is now complete. WARNING: Inhibitors found. Review the tasks above or the result file at /root/preupgrade/result.txt."

Sample 6-7 where there are no inhibitors (EXTREME), but there are HIGH not enough space warnings, and so marked as inhibited:
RUNNING HANDLER [infra.leapp.analysis : Display inhibitors] ********************
skipping: [satrhel6-2.localdomain.local]
Thursday 25 April 2024  11:25:49 +0000 (0:00:00.037)       0:05:01.110 ********
 [started HANDLER: infra.leapp.analysis : Display errors on satrhel6-2.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Display errors] ************************
ok: [satrhel6-2.localdomain.local] => {
    "results_errors.stdout_lines": [
        "preupg.risk.HIGH: Not enough free space on the /boot. Release more space for upgrade."
    ]
}
Thursday 25 April 2024  11:25:49 +0000 (0:00:00.047)       0:05:01.157 ********
 [started HANDLER: infra.leapp.analysis : Preupgrade analysis report is done on satrhel6-2.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Preupgrade analysis report is done] ****
ok: [satrhel6-2.localdomain.local] => {
    "msg": "The preupgrade analysis report generation is now complete. WARNING: Inhibitors found. Review the tasks above or the result file at /root/preupgrade/result.txt."
}

A 7-8 Leapp one for comparison:
RUNNING HANDLER [infra.leapp.analysis : Display inhibitors] ********************
skipping: [satrhel7-1.localdomain.local]
Thursday 25 April 2024  11:03:20 +0000 (0:00:00.052)       0:03:12.163 ********
 [started HANDLER: infra.leapp.analysis : Display errors on satrhel7-1.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Display errors] ************************
ok: [satrhel7-1.localdomain.local] => {
    "results_errors.stdout_lines": [
        "Risk Factor: high (error)",
        "Title: Not enough space available on /var/lib/leapp/scratch: Needed at least 3591 MiB.",
        "Summary: {\"detail\": \"The file system hosting the /var/lib/leapp/scratch directory does not contain enough free space to proceed all parts of the in-place upgrade. Note the calculated required free space is the minimum derived from upgrades of minimal systems and the actual needed free space could be higher.\\nNeeded at least: 3591 MiB.\\nSuggested free space: 4591 MiB (or more).\"}",
        "Key: 6469b14a9772e9987c08c637e415d1724c425cfa",
        "----------------------------------------"
    ]
}
Thursday 25 April 2024  11:03:20 +0000 (0:00:00.051)       0:03:12.214 ********
 [started HANDLER: infra.leapp.analysis : Preupgrade analysis report is done on satrhel7-1.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Preupgrade analysis report is done] ****
ok: [satrhel7-1.localdomain.local] => {
    "msg": "The preupgrade analysis report generation is now complete. WARNING: Inhibitors found. Review the tasks above or the result file at /var/log/leapp/leapp-report.txt."
}

And an 8-9 Leapp one for comparison:
RUNNING HANDLER [infra.leapp.analysis : Display inhibitors] ***************************************************************************************************
ok: [satrhel8-1.localdomain.local] => {
    "results_inhibitors.stdout_lines": [
        "Risk Factor: high (inhibitor)",
        "Title: The installed OS version is not supported for the in-place upgrade to the target RHEL version",
        "Summary: The supported OS releases for the upgrade process:",
        " RHEL 8.6",
        "RHEL 8.8",
        "RHEL-SAPHANA 8.6",
        "RHEL-SAPHANA 8.8",
        "Key: 1c7a98849a747ec9890f04bf4321de7280970715",
        "----------------------------------------"
    ]
}
Thursday 25 April 2024  19:03:12 +0800 (0:00:00.051)       0:03:03.086 ********
 [started HANDLER: infra.leapp.analysis : Display errors on satrhel8-1.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Display errors] *******************************************************************************************************
skipping: [satrhel8-1.localdomain.local]
Thursday 25 April 2024  19:03:12 +0800 (0:00:00.040)       0:03:03.126 ********
 [started HANDLER: infra.leapp.analysis : Preupgrade analysis report is done on satrhel8-1.localdomain.local]

RUNNING HANDLER [infra.leapp.analysis : Preupgrade analysis report is done] ***********************************************************************************
ok: [satrhel8-1.localdomain.local] => {
    "msg": "The preupgrade analysis report generation is now complete. WARNING: Inhibitors found. Review the tasks above or the result file at /var/log/leapp/leapp-report.txt."
}

I like making it closer to be the same, the one question I have is, the High seems only related to free space, is that correct?


Fix a small inconsistency with Pigeon docs

What package does this bug report belong to?
pigeon
What target platforms are you seeing this bug on?
Web(site)
Steps to reproduce

Go to https://pub.dev/packages/pigeon#rules-for-defining-your-communication-interface
Scroll down to #5

Actual/Expected results
This line reads:

Generics are supported, but can currently only be used with nullable types (example: List<int?>).

This is no longer true because of #97848. It should just say "Generics are supported" or be removed entirely.

It felt a little weird filing this as a "bug report", but I didn't know how else to create this issue. I apologize if there's a better place to put it.


Update sqlite3pp.ipp

Get rid of 64bit warnings under Visual Studio 2015 update 1. There was a more complicated proposal to fix the same issue in the past, which was rejected. Please let me know if this idea is more in line with the library spirit.

Can you post the 64bit warnings in MSVS2015? I don't have MSVS.

C4267 'argument': conversion from 'size_t' to 'int', possible loss of data.
The int is 32bit on the 64bit Windows platform.
I have also tried with update 2 of the compiler (the latest version) - the same warning is there.
Microsoft docs have the following entry for the warning:
https://msdn.microsoft.com/en-us/library/6kck0s93.aspx

For type issues, I could not find a solution to work with every platforms. Sorry for your inconvenience.


Can't schedule a push - Parse Dashboard

Currently trying to send a push notification on an established Parse setup (been working for months up until now).
When configuring a push in the new dashboard, we get a 'Save Failed', please select devices, despite 'Everyone' selected as the audience:

In the old dashboard, under our production account, we do not see any devices (despite their being 340k in the new dashboard), and cannot schedule a push.

My apologies if this is not the right channel, it's been very challenging trying to find a support email address and all other avenues point to here. Also, the contact us form on the website gives a 500:

Closing as this has already been submitted through the official bug reporting channel for Dashboard issues (see https://developers.facebook.com/bugs/551553698343157/).


feat: cache Windows IIS image for test purpose

What type of PR is this?
/kind feature

What this PR does / why we need it:
Cache Windows IIS image for test purpose so we can use mcr.microsoft.com/windows/servercore/iis in e2e tests without specify arch.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall First build on abelch/cache-win-iis at 60.655%

Totals

Change from base Build<PHONE_NUMBER>:
60.7%

Covered Lines:
2576

Relevant Lines:
4247

💛  - Coveralls


Added types implementation

Suggested fix for #75
Signed-off-by: HRASTNIK Patrick Ing<EMAIL_ADDRESS>

Would be good to get this merged. @patrickhrastnik, not sure if you have time to resolve the conflicts otherwise @abmusse if you want to pull the changes in to a new PR and get it merged that would be great.

Would be good to get this merged. @patrickhrastnik, not sure if you have time to resolve the conflicts otherwise @abmusse if you want to pull the changes in to a new PR and get it merged that would be great.

One thing that I was thinking about that would impact this, is it would be really nice to have a varchar type, but that would require having the function/object return both the XMLSERVICE type as well as the varying="XX" attribute.
I think this could be done by having the function return an object with the attributes and using the javascript spread operator:
# old way
program.addParam({ type: '10A', varying: '4', value: 'Gill' });

function varchar(len) {
  return { type: len.toString() + 'a', varying: '2' }
}
function longvarchar(len) {
  return { type: len.toString() + 'a', varying: '4' }
}

program.addParam({ ...longvarchar(10), value: 'Gill' });

One thing that I was thinking about that would impact this, is it would be really nice to have a varchar type, but that would require having the function/object return both the XMLSERVICE type as well as the varying="XX" attribute.
I think this could be done by having the function return an object with the attributes and using the javascript spread operator:
# old way
program.addParam({ type: '10A', varying: '4', value: 'Gill' });

function varchar(len) {
  return { type: len.toString() + 'a', varying: '2' }
}
function longvarchar(len) {
  return { type: len.toString() + 'a', varying: '4' }
}

program.addParam({ ...longvarchar(10), value: 'Gill' });

One thing that I was thinking about that would impact this, is it would be really nice to have a varchar type, but that would require having the function/object return both the XMLSERVICE type as well as the varying="XX" attribute.
I think this could be done by having the function return an object with the attributes and using the javascript spread operator:
# old way
program.addParam({ type: '10A', varying: '4', value: 'Gill' });

function varchar(len) {
  return { type: len.toString() + 'a', varying: '2' }
}
function longvarchar(len) {
  return { type: len.toString() + 'a', varying: '4' }
}

program.addParam({ ...longvarchar(10), value: 'Gill' });

If we did that, it might also make sense to use camel case on the names:

Integer
Char
Varchar
LongVarchar
etc

Nice! The spread operator works out cleanly in this situation. I just tested it out in sample script:
function LongVarChar(len) {
  return { type: len.toString() + 'A', varying: '2' }
}

function addParam(param) {
  console.log(param)
}

addParam({ type: '10A', varying: '2', value: 'Gill' });
addParam({ ...LongVarChar(10), value: 'Gill' })

Output
{ type: '10A', varying: '2', value: 'Gill' }
{ type: '10A', varying: '4', value: 'Gill' }

The use of camel case be to minimize confusion for the reader?
I'm pretty sure the linter will complain but we can work around that.

One thing that I was thinking about that would impact this, is it would be really nice to have a varchar type, but that would require having the function/object return both the XMLSERVICE type as well as the varying="XX" attribute.
I think this could be done by having the function return an object with the attributes and using the javascript spread operator:
# old way
program.addParam({ type: '10A', varying: '4', value: 'Gill' });

function varchar(len) {
  return { type: len.toString() + 'a', varying: '2' }
}
function longvarchar(len) {
  return { type: len.toString() + 'a', varying: '4' }
}

program.addParam({ ...longvarchar(10), value: 'Gill' });

If we did that, it might also make sense to use camel case on the names:

Integer
Char
Varchar
LongVarchar
etc

Nice! The spread operator works out cleanly in this situation. I just tested it out in sample script:
function LongVarChar(len) {
  return { type: len.toString() + 'A', varying: '2' }
}

function addParam(param) {
  console.log(param)
}

addParam({ type: '10A', varying: '2', value: 'Gill' });
addParam({ ...LongVarChar(10), value: 'Gill' })

Output
{ type: '10A', varying: '2', value: 'Gill' }
{ type: '10A', varying: '4', value: 'Gill' }

The use of camel case be to minimize confusion for the reader?
I'm pretty sure the linter will complain but we can work around that.


How to put all the Widgets into a List

class _MyHomePageState extends State {
int _counter = 2;
List<Widget> t = [];
void _incrementCounter() {
setState(() {
_counter++;
});
}
@override
void initState() {
super.initState();
t.add(Text('$_counter'));
}
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
title: Text(widget.title),
),
body: Center(
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: t,
),
),
floatingActionButton: FloatingActionButton(
onPressed: _incrementCounter,
tooltip: 'Increment',
child: const Icon(Icons.add),
),
);
}
}
I want to isolate the widget List<Widget> t = [];, but it doesn't work.  Because I want to change the a part of the UI when the date changed. It's similar to VUE. Any suggestion?

@fayfive
Your question is better suited for support channels like StackOverflow, so please ask the same there.
Closing from here.

Thanks！


[possible xstate/graph bug?] extra quote marks generated by getShortestPaths serialization

Bug or feature request?
bug. although there is a chance that this is working as designed, just not a very intuitive design IMO
Description:
have a look at https://repl.it/repls/ElegantGlumFirewall
(Bug) Expected result:
{ loggedOut:
   { state: { value: 'loggedOut', context: undefined },
     weight: 0,
     path: [] },
  loggedIn:
   { state: { value: 'loggedIn', context: undefined },
     weight: 1,
     path: [ [Object] ] },
  accountStatus:
   { state: { value: 'accountStatus', context: undefined },
     weight: 2,
     path: [ [Object], [Object] ] },
  changePassword:
   { state: { value: 'changePassword', context: undefined },
     weight: 3,
     path: [ [Object], [Object], [Object] ] } }

(Bug) Actual result:
{ '"loggedOut"':
   { state: { value: 'loggedOut', context: undefined },
     weight: 0,
     path: [] },
  '"loggedIn"':
   { state: { value: 'loggedIn', context: undefined },
     weight: 1,
     path: [ [Object] ] },
  '"accountStatus"':
   { state: { value: 'accountStatus', context: undefined },
     weight: 2,
     path: [ [Object], [Object] ] },
  '"changePassword"':
   { state: { value: 'changePassword', context: undefined },
     weight: 3,
     path: [ [Object], [Object], [Object] ] } }

(Bug) Potential fix:
i'm not confident this is a potential fix, because it seems to somewhat be on purpose? https://github.com/davidkpiano/xstate/blob/9ee91f9aa6ee529ffe392af887ec189b330bdd5e/src/graph.ts#L169
however these extra quote marks dont seem to do anything and make the results harder to access.
Link to reproduction or proof-of-concept:
https://repl.it/repls/ElegantGlumFirewall

Good catch! This is actually by design, as weird as it looks. Try getting the shortest paths of a hierarchical state machine... how do you represent that state value as a key?
Example:
{
  '"green"': { ... },
  '"yellow"': { ... },
  '{"red":"walk"}': { ... },
  ...
}

Since it needs to represent every state value in a serializable way, JSON.stringify(...) is the way to go, and it makes sense (or at least to me... open to suggestions) that it's easier to JSON.stringify(...) everything instead of only some things. That way, you can represent all the generated state values as:
Object.keys(shortestPaths).map(key => JSON.parse(key));

oh i see. I will try that!


[Feature]: moderation system

Merlion Needed Moderation system
ex. ban, strike, role-adding, member-organizing.
currently its using code from someone's github and thats bad, we wanted to make our mod system ourself but don't know how

update: moderation system not moved to seperate moderation bot


TRUE-COLOR-S2-L1C layer configuration

I was trying to run the example land-cover-map over Switzerland on my local machine but ran into a problem:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-20-54b437ee3056> in <module>()
      8 results = workflow.execute({input_task:{'bbox':bbox, 'time_interval':time_interval},
      9                             export_val_sh:{'filename':f'CH_data/valid_count-L1C/{tiff_name}'},
---> 10                             save:{'eopatch_folder':patch_name}
     11                            })

~\Anaconda3\lib\site-packages\eolearn\core\eoworkflow.py in execute(self, input_args)
    280         input_args = {WorkflowResult.get_key(k): v for k, v in input_args.items()} if input_args else {}
    281
--> 282         _, intermediate_results = self._execute_tasks(input_args=input_args, outdegs=outdegs)
    283
    284         return WorkflowResult(intermediate_results)

~\Anaconda3\lib\site-packages\eolearn\core\eoworkflow.py in _execute_tasks(self, input_args, outdegs)
    303             result = self._execute_task(input_args=input_args,
    304                                         intermediate_results=intermediate_results,
--> 305                                         task_id=t_id)
    306
    307             intermediate_results[t_id] = result

~\Anaconda3\lib\site-packages\eolearn\core\eoworkflow.py in _execute_task(self, input_args, intermediate_results, task_id)
    331         inputs = tuple(intermediate_results[t_dep] for t_dep in self.deps[task_id])
    332         LOGGER.debug("Computing %s(*%s, **%s)", str(task), str(inputs), str(kw_inputs))
--> 333         return task(*inputs, **kw_inputs)
    334
    335     def _relax_dependencies(self, *, intermediate_results, out_degrees, current_task_id):

~\Anaconda3\lib\site-packages\eolearn\core\eotask.py in __call__(self, *eopatches, **kwargs)
     39
     40     def __call__(self, *eopatches, **kwargs):
---> 41         return self.execute(*eopatches, **kwargs)
     42
     43

~\Anaconda3\lib\site-packages\eolearn\mask\cloud_mask.py in execute(self, eopatch)
    246         # Raise error if last channel dimension is less than required
    247         if new_data.shape[-1] < len(self.classifier.band_idxs):
--> 248             raise ValueError("Data field has less than the required 10 bands")
    249
    250         # Compute cloud mask and add as feature to EOPatch

ValueError: Data field has less than the required 10 bands

This message was shown when running the cell [20] in eo-learn/examples/land-cover-map/2_eopatch-L1C.ipynb with some minor tweaks.
Now I know it might be an issue due to my own adaptation to the region of Switzerland. So I would like to try run the notebook exactly as shown in this repository. Therefore I need to set up the layer in the configuartor. What is the specification of the TRUE-COLOR-S2-L1C layer you mention in the 2_eopatch-L1C.ipynb?

Thanks for pointing it out. I've added a README (https://github.com/sentinel-hub/eo-learn/tree/master/examples) where instructions on how to add new configuration to your SH account using a predefined template that has all layers used in the examples are given -- including the TRUE-COLOR-S2-L1C and BANDS-S2CLOUDLESS.
Based on your error I assume you have changed the layer name in the following line
add_clm = AddCloudMaskTask(cloud_classifier,
                           'BANDS-S2CLOUDLESS',
                           cm_size_y='160m',
                           cm_size_x='160m',
                           cmask_field='CLM',
                           cprobs_field='CLP',
                           instance_id=INSTANCE_ID)

If you did, I suggest you to change it back to BANDS-S2CLOUDLESS, since the cloud detector needs the 10 bands in order to make the cloud classification.

Thank you very much, this helped to actually be able to use your notebooks for testing. Unfortunately, now I get the following error from the same cell:
DownloadFailedException: Failed to download with HTTPError:
400 Client Error: Bad Request for url: https://services.sentinel-hub.com/ogc/wcs/0f6c0cd7-002d-4460-8017-4d78502d7d35?SERVICE=wcs&BBOX=370230.5261411405%2C5085303.344972428%2C380225.31836121203%2C5095400.767924464&FORMAT=image%2Ftiff%3Bdepth%3D32f&CRS=EPSG%3A32633&RESX=160m&RESY=160m&COVERAGE=TRUE_COLOR&REQUEST=GetCoverage&TIME=2017-01-01T10%3A04%3A07%2F2017-01-01T10%3A04%3A07&MAXCC=80.0&ShowLogo=False&Transparent=False&EvalScript=cmV0dXJuIFtCMDEsQjAyLEIwNCxCMDUsQjA4LEI4QSxCMDksQjEwLEIxMSxCMTJd
Server response: "Layer TRUE_COLOR not found"

I did not change the notebooks in any way, so I am sure to use the proper version. Now I don't know why it asks for the TRUE_COLOR layer, since the layers specified are TRUE-COLOR-S2-L1C and BANDS-S2CLOUDLESS. The configuration is as explained in the README.
If it is of any help: The error occured after the cell was running for several minutes. Network traffic was on full load, so I guess it was actually downloading some data.

OK, I see now where the problem is. The AddCloudMaskTask actually doesn't create a SentinelHub request using the 'BANDS-S2CLOUDLESS' layer but does this by using eval script (simple script that returns the 10 necessary bands). Unfortunately, the OGC standard requires the LAYER parameter always to be specified although it has no impact on what kind of data is returned (10 bands in this case). The AddCloudMaskTask at the moment sets the layer to 'TRUE_COLOR' when it makes the request (the layer name is hardocded). If this layer doesn't exist then the above error is issued.
This clearly needs to be fixed in the AddCloudMaskTask, but until then please use the updated version of example notebook. The issue can be circumvented by downloading the 10 bands in another task. If the data is already available the cloud detector doesn't download the data again.

@fvivian the issue should be solved now. Please update your version of eo-learn and run the examples again.

Thank you for the explanation and quick fix. The example runs without any issues now.

@fvivian  Sorry for opening the issue again.
There is still/again a :
Server response: "Layer BANDS-S2-L1C not found"
Was running your notebook
Is therer a way to query the Layer names?

Hi @nilshempelmann
Make sure the layer configuration is set up properly as for these requirements.
Login to the Sentinel Hub configurator and add a new configuration using the Python scripts template. Then update your instance ID and the script should run.

Got it. Thanks !


checking out specific commits in setup instructions feels hacky

"the fact that you have to checkout specific commits in the instructions feels very hacky."

@SeanTAllen once we're on mainline ponyc as part of #1605 we should be able to get rid of this and instead tell them to check out a specific tag, correct?

@aturley correct.

Closing this as we no long do this as of 0.2.0


RiskProfiler map glitch on Chrome on iPad

On chrome on iPad when I tried to view fatalities and entrapments for the M7.1 Sidney scenario the map stopped working. I could use the '+' and '-' to zoom in and out but I couldn't move around.

Thanks @tieganh!  I am apologized to admit that I am/was a Google fanboy and only got Android and Chromebook tablets but no iPad (at least not yet... I should have acted more decisively and bought one when my wife found a good deal during Black Friday 2023)... 🤣  Anyhow, I'll hopefully be able to look into this in the not-too-distant future.  😅🙏  Cheers!


Store Topology canvas is not fit to full screen

Bug Report
Please answer these questions before submitting your issue. Thanks!
What did you do?
Open store topology page.
What did you expect to see?
Store location is rendered without any truncation.
What did you see instead?

What version of TiDB Dashboard are you using (./tidb-dashboard --version)?
latest master.

any repro?

@unbyte Just drag the canvas and you will see the graph is being truncated, restricted in a region smaller than the window.

Got, will fix it lately.

@baurine actually i've fixed it by setting a min height😂 however i think it's better to make the boundary of canvas more clear


pip install fails

I am trying to install in a clean virtualenv. I tried  pip install cairocffi.
I get the following error (copied from the tail of the pip.log)
Exception information:
Traceback (most recent call last):
  File "/home/creation/.virtualenvs/proto/local/lib/python2.7/site-packages/pip/basecommand.py", line 134, in main
    status = self.run(options, args)
  File "/home/creation/.virtualenvs/proto/local/lib/python2.7/site-packages/pip/commands/install.py", line 241, in run
    requirement_set.install(install_options, global_options, root=options.root_path)
  File "/home/creation/.virtualenvs/proto/local/lib/python2.7/site-packages/pip/req.py", line 1298, in install
    requirement.install(install_options, global_options, *args, **kwargs)
  File "/home/creation/.virtualenvs/proto/local/lib/python2.7/site-packages/pip/req.py", line 625, in install
    cwd=self.source_dir, filter_stdout=self._filter_install, show_stdout=False)
  File "/home/creation/.virtualenvs/proto/local/lib/python2.7/site-packages/pip/util.py", line 670, in call_subprocess
    % (command_desc, proc.returncode, cwd))
InstallationError: Command /home/creation/.virtualenvs/proto/bin/python -c "import setuptools;__file__='/home/creation/.virtualenvs/proto/build/cffi/setup.py';exec(compile(open(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-vYXHD_-record/install-record.txt --single-version-externally-managed --install-headers /home/creation/.virtualenvs/proto/include/site/python2.7 failed with error code 1 in /home/creation/.virtualenvs/proto/build/cffi

If its at all useful, the last few line of the install output were:
gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -DUSE__THREAD -I/usr/include/ffi -I/usr/include/libffi -I/usr/include/python2.7 -c c/_cffi_backend.c -o build/temp.linux-i686-2.7/c/_cffi_backend.o

c/_cffi_backend.c:13:17: fatal error: ffi.h: No such file or directory

compilation terminated.

error: command 'gcc' failed with exit status 1

U GET RIGHT @SimonSapin !
Thanks boss.

Thx SimonSapin


Add E2E tests for yarn start at app package

Sometimes dependabot upgrades break yarn start (e.g. #179), but currently there are no CI workflows to catch those errors in advance. We should add some basic E2E tests that ensures yarn start at app package doesn't break.
Is React Testing Library provide sufficient for providing such E2E tests?  Or should we consider introducing an E2E framework like Cypress?

Done with #322


[FeatureRequest] Provide setting option to shellcheck wiki link

Would be nice to able to setting custom shellcheck wiki link.
Eg,
from https://www.shellcheck.net/wiki/${ruleId}
to https://github.com/koalaman/shellcheck/wiki/${ruleId}.
But now seems hardcoded.
https://github.com/timonwong/vscode-shellcheck/blob/3a803d31912b863985405da042339db98fda0c56/src/linter.ts#L233
Besides, https://www.shellcheck.net/wiki/${ruleId} seems always redirect to https://github.com/koalaman/shellcheck/wiki/${ruleId}.

Hi @loynoir, do you know a use case for this?

As from my side,

https://www.shellcheck.net/wiki/${ruleId} seems always redirect to https://github.com/koalaman/shellcheck/wiki/${ruleId}
https://www.shellcheck.net/wiki/${ruleId} is way more slower than https://github.com/koalaman/shellcheck/wiki/${ruleId}
shellcheck repo is on https://github.com/koalaman/shellcheck

2. From my side, https://www.shellcheck.net/wiki/${ruleId} is way more slower than https://github.com/koalaman/shellcheck/wiki/${ruleId}. First connection stuck.

It may be worth to report as an issue to https://github.com/koalaman/shellcheck.net.
We could consider switching the default to the GitHub one directly, but I think @koalaman's goal at https://www.shellcheck.net/wiki was to prevent a move on the repository name or owner to break someone relying on https://github.com/koalaman/shellcheck/wiki/${ruleId}, or to allow moving to somewhere else than the wiki without breaking as  well.


React app, not able to rerender, refresh, reload content

I've been reading as much as I can and tried refresh(), render(), forceSync, etc, with no luck.
It seems the library doesn't include these methods.
My textarea is correctly updated using React local state, but it's imposible for me to update the editor from the textarea

react-simplemde-editor is a react component wrapper for simplemde which might solve your issues.

@DavidRayPatterson Thanks!


Add route-registrar for friendlier PCF urls

@mkuratczyk should I put in an issue for the new Route Registrar?  I believe it's broke per our last chat.

I've just pushed https://github.com/pivotal-cf/pcf-prometheus-pipeline/commit/ba4b798d79c2eaa999bbe689e98bb3cf2d155b5c to disable it by default.
@RMeharg there are a few problems with this:

It fails when deploying to an external BOSH director
It's explicit about release versions with but doesn't actually upload these versions
Even if the releases are there, sometimes they are only available as compiled for xenial stemcells so it still fails trying to deploy to trusty (we can change the default to xenial soon I guess)

@mkuratczyk good points

Yes, it has to be on the same director as CF until we have cross-director links (https://github.com/cloudfoundry/bosh-notes/blob/master/proposals/cross-director-links.png)
I'm not sure I understand, it follows the same format as prometheus-boshrelease, we could use github-release resource if it wasn't optional
All releases should be switching to xenial as soon as possible :)

I am fine with leaving this as an optional operations file.


Provide actually KMM app using gitlive

Would be really nice to provide an actual app showcasing some features like auth, firestore using this project

I've made some, feel free to take a look :)
https://github.com/murfel/kmp-wizard-firebase-auth/pull/1
https://github.com/murfel/KMP-App-Template-Firebase-Auth/pulls

@murfel have you tried notifications/messaging?

@carvalho2707 I haven't and using such is not in the plans, but do share an MVP if you do!


Isn't Adaptive Card's AdaptiveContainer's Spacing = padding should add padding to the container?

I am trying to add a padding to an AdaptiveCard AdaptiveContainer so I set its Spacing = "Padding". This seems to have no effect, so my question is:
isn't setting the Spacing = "padding" should add padding to the container?

Hi @arman-g
Perhaps you could help clear up some confusion. Your title says Space = padding but then your question says Spacing = "Padding" and Spacing = "padding". These inconsistencies make me worry that maybe you're not setting the property correctly. Can you show us what your code actually looks like?

Hi @v-kydela sorry for the confusion. I corrected the title and it is spacing. Below I provide C# code of adaptive card where I set the "Spacing" property to "Padding" for a container and column set. They have no effect on webchat.
Body.Add(new AdaptiveContainer
{
	**Spacing** = AdaptiveSpacing.Padding,
	Items = new List<AdaptiveElement>
	{
		new AdaptiveColumnSet
		{
			**Spacing** = AdaptiveSpacing.Padding,
			SelectAction = SelectAction = new AdaptiveToggleVisibilityAction
			{
				TargetElements = new List<AdaptiveTargetElement>
				{
					new AdaptiveTargetElement
					{
						ElementId = "malfeasance_explained",
						IsVisible = true
					},
					new AdaptiveTargetElement
					{
						ElementId = "show_info_icon",
						IsVisible = false
					}
				}
			},
			Columns = new List<AdaptiveColumn>
			{
				new AdaptiveColumn
				{
					Width = "stretch",
					Items = new List<AdaptiveElement>
					{
						new AdaptiveTextBlock
						{
							Text = messages[1126],
							Wrap = true
						}
					}
				},
				new AdaptiveColumn
				{
					Id = "show_info_icon",
					VerticalContentAlignment = AdaptiveVerticalContentAlignment.Center,
					Width = "16px",
					Items = new List<AdaptiveElement>
					{
						new AdaptiveImage
						{
							Spacing = AdaptiveSpacing.Padding,
							HorizontalAlignment = AdaptiveHorizontalAlignment.Center,
							UrlString = _httpContext.GetAbsoluteUrl("info.png"),
							PixelWidth = 16,
							PixelHeight = 16
						}
					}
				}
			}
		},
		malfeasanceExplained
	}
});

@arman-g - Thank you for the clarification
Web Chat uses the out-of-the-box HTML renderer from the Adaptive Cards Node.js SDK, so this issue might have been better suited for the Adaptive Cards repo. From my examination of the source code, there doesn't seem to be anything special about the "padding" value that makes the spacing behave differently. It looks like the default host config makes "padding" have the same spacing as "default."
You might consider trying different values for spacing. If you want to easily see how an Adaptive Card will look without having to send the card to Web Chat, you can try the Adaptive Cards Designer or you can play with the example HTML from the SDK.
Does this resolve your issue?

@v-kydela,
Using Adaptive Cards Designer and changing container spacing has no effect. I guess this is a question to AdaptiveCards team.
Thank you for your response and help.


[INFRA] Use Ubuntu Focal 20.04 in Travis tests

This will now use Octave 5.2.0 for tests.

Thank you!


fix pakiti.sql

issue #104

remove insert unknown OsGroup
move CveDef before Cve due to a foreign key

The problem has been fixed, thanks for the report.


Trading: Misc v2 features (settings, tx feedback + blocking modal)

Smaller features

Global settings

Change odds
Change default timezone
Change time format
Change language
Change to "max profit" or "max return"?
Settings: ability to turn off toast

Transaction feedback pattern

https://github.com/AugurProject/augur/issues/1725
For the order ticket we can't hide the button
Explore how this pattern could be applied across the app
Included in this is the unsigned orders modal for new market creation

"included" was in reference to purchasing flows - kept here to work on afterward

List of places this needs to work

Create market
Reporting
All trading flows
Onboarding
Forking

Blocking modal for singing transactions

Depends on the wallet, IF edge/portis there is no signing etc

➤ Joey Krug commented:
Get rid of change price of open order. Not relevant with 0x

Other settings options:
Receive Notifications (possibly through an email Alert)
Approve in/out of newsletter
Hide Chat
Hide Comments
Theme?


optimize qwen2 memory usage again

Description
attention_mask is not required when using sdp_causal and batch_size=1 or generating 2nd+ token, so don't generate it in these cases to save memory

PR validation: https://github.com/intel-analytics/ipex-llm-workflow/actions/runs/9805767952/job/27076276807


Why can't a filter be in a ref layer?

I can't seem to find much on using ref layers, but essentially I'd like to set up base layer and alter the paint based on a property.
I figured I could set up a layer and use ref to create styles that inherit a base style, kinda like Carto, where I can say
marker-fill: #000;
[day="MON"] { marker-fill: @mon }

So I tried this:
{
    "id": "signs",
    "type": "symbol",
    "source": "broom-signs",
    "source-layer": "data",
    "layout": {
      "icon-allow-overlap": true,
      "icon-ignore-placement": true,
      "icon-image": "circle.sdf"
    },
    "paint": {
        "icon-size":0.3,
        "icon-color": "#000"
    }
},
{
    "id": "mon-signs",
    "filter": ["==", "day", "MON"],
    "ref": "signs",
    "paint": {
        "icon-size":0.3,
        "icon-color": "@mon"
    }
}

However, I get "filter" is prohibited for ref layers.
I have a feeling I'm thinking about this wrong, because otherwise I have to duplicate a lot of JSON just to vary the color. I could stick the color in the data and use {color}, but that doesn't seem right either.
So: Is there a way we could filters in ref layers? Or is there another way of accomplishing this that I'm just not seeing?
Thanks

whoops, definitely the wrong repo.


Change example from Stackblitz to Codesandbox

Next.js did the same https://github.com/vercel/next.js/issues/49143#issuecomment-1539656028
WASM is a problem.

So nothing works right now.


Fix location of conf.py in readthedocs config

I had to do this to get the docs to build successfully in readthetodocs.io

Thanks for the contribution 🎉

Thanks for the contribution 🎉


The paws don't show up anymore.

A lot of people are now having an issue of the paws just becoming blank and not showing up anymore.

Hey, thank you for reporting! I'm sorry that the project was so inactive.
I think the bug had something to do with the installation process of the userstyle, because userstyles.org had a update.
I pushed an update to the userstyle so the issue should be now fixed. If it still occurs after a update or a reinstallation, please reopen this issue.


plugins: will be deprecated?

Just now seeing this in the readme.
# Format is the same as `dependencies` in a package manifest.
# DEPRECATED: This will be removed at some point.
plugins:
  additional-tools: { path: "/usr/local/additional-tools" }

Wondering if this is true and what the reasoning behind this is? I have found this feature quite useful. I have made packages which act as a means to run tooling via bender. For example, all of my components require a unit testing dependency that allows me to simply kick off a unit test, for any of my components, with something along the lines of bender rtl_sim <args>.  Anyway, I have found the feature quite useful and would be sad to see it go :(.

Looking back at this tonight, is this only for the configuration files that this feature will be removed? Or does this include the manifest file? I use this feature in my manifests. Thanks.

Hi @mac-edmondson,
Currently, there are no plans to remove this feature.
For some reason on the comment, plugins were originally introduced as a quick way to get scripts prior to scripts being introduced as their own command. Afterward, we had no specific use for plugins, but left them within bender in case they were useful to others, albeit without continually testing their functionality.
I hope plugins continue to be useful for you, feel free to contribute fixes if they do break.


fix: incorrect recip key with multi routing keys

Signed-off-by: Timo Glastra<EMAIL_ADDRESS>

Codecov Report

Merging #446 (43cf8e2) into main (55c0fa0) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #446      +/-   ##
==========================================
+ Coverage   85.93%   85.95%   +0.01%
==========================================
  Files         253      253
  Lines        5433     5433
  Branches      804      801       -3
==========================================
+ Hits         4669     4670       +1
+ Misses        764      763       -1

Impacted Files
Coverage Δ

packages/core/src/agent/AgentMessage.ts
90.00% <100.00%> (+2.50%)
:arrow_up:

packages/core/src/agent/EnvelopeService.ts
100.00% <100.00%> (+3.57%)
:arrow_up:

...onnections/messages/ConnectionInvitationMessage.ts
97.77% <100.00%> (-0.10%)
:arrow_down:

Continue to review full report at Codecov.

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


Prefer https:// for URLs when available

Rebased to resolve merge conflicts.

thanks for putting this together.


unfitting events

Solfall,
Some experiences in my Norway game I would like to point out. Similar to vanilla the population of Norway is almost always for 100% free trade. Other similarities is the population ideologies defaulting to 33% on the four marked ideologies except laissez faire. Then we have 66% secularized and 33% moralism. Similar thing with pacifism and citizenship.
Unfitting events for capital punishment. They still occur when you have abolished it.
I feel that many of the events are unfitting for the time period. Not talking about the description of them, but the events itself and the results of some of them. The citizenship events for example. "You see I have this dream of mine" with the 5% limited citizenship support effect becomes humorous since immigrants and minorities have equal rights and full citizenship.
In the text of some of the events, you can see the same demographic appear twice in different options.
Nations are not good at moving their armies around. They almost never participates in peacekeeping missions.

Most of the comments are unspecific -> no action.
Will change capital punishment preconditions and the "I have this dream of mine"

After further investigation, "I have this dream of mine" comes out of the stock event - called in onAction - may take this down the line but right now - no plans to fix.


Fieldsets: Legend element is outside the fieldset

Our recommended fieldset pattern involves a <legend> element outside the <fieldset> element. This is invalid, the legend needs to be a child of the fieldset.
See https://validator.w3.org/nu/?doc=https%3A%2F%2Fcfpb.github.io%2Fdesign-system%2Fcomponents%2Ffieldsets

We should fix this. There may be consequences to this depending on whether we depend on the current <legend> structure. I'll add it to our internal backlog.

This needs investigation to see if we are depending in some way on this broken pattern, and if we are, refactoring so that the pattern is both accessible and works as intended. @anselmbradford will look at this next sprint

PR --> https://github.com/cfpb/design-system/pull/2009


Fix IsWithin.java

Nasty bug made the matcher always return true.

A test would be great here

Good catch! Thanks for the PR. I'm with @marcomorain though -- could you add a unit test for the previously-erroneous cases? Once the test is in, I'll merge the PR.

Thanks a ton! Hope the lib's helpful -- I'm always game for suggestions.

Will get an updated version out to Maven Central soon -- the machine with my gpg signing keys and such is having a couple of issues at the moment.


add namespace to the lib?

the lib defined
//-----------------------------------------------------------------------------
// Structure for bit arrays used for HET and BET tables

typedef struct _TBitArray
{
    DWORD NumberOfBytes;                        // Total number of bytes in "Elements"
    DWORD NumberOfBits;                         // Total number of bits that are available
    BYTE Elements[1];                           // Array of elements (variable length)
} TBitArray;

I am build this lib with Unreal Engine 4, while Unreal Engine 4 already defined TBitArray

The structure was not really meant to be public, I made it private. The changes are in a separate branch. Could you please try if that will work for you?
Branch: https://github.com/ladislav-zezula/StormLib/tree/TBitArrayChangedToPrivate

The structure was not really meant to be public, I made it private. The changes are in a separate branch. Could you please try if that will work for you?
Branch: https://github.com/ladislav-zezula/StormLib/tree/TBitArrayChangedToPrivate

TBitArray and others macro have same name with UE4, such as PLATFORM_XXX, I think namespace and StormLib_ prefix will solve these problem
STORMLIB_PLATFORM_WINDOWS
STORMLIB_PLATFORM_LINUX
STORMLIB_PLATFORM_XXX

Next iteration:

PLATFORM_DEFINED changed to STORMLIB_PLATFORM_DEFINED
PLATFORM_XXX changed to STORMLIB_XXX

Please, try with your project.

Next iteration:

PLATFORM_DEFINED changed to STORMLIB_PLATFORM_DEFINED
PLATFORM_XXX changed to STORMLIB_XXX

Please, try with your project.

StormLib.h(483): error C4099: 'TBitArray': type name first seen using 'class' now seen using 'struct'

Changed to class. If that won't help, I'll remove it from StormLib.h altogether.

Changed to class. If that won't help, I'll remove it from StormLib.h altogether.

change TBitArray to StormLib_TBitArray fix compile error


listen_mqtt remove double case

Notice unlock and climate-off defined twice.

Will be fixed in a wider review.
Thanks for the heads up


Fix issues #183

Fix: https://github.com/no23reason/react-geolocated/issues/183

@giautm Thank you for the fix! Could you please update the commit message so that it adheres to the commit message format as specified in this project's contributing guide?

:tada: This issue has been resolved in version 2.3.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@no23reason : Tks, I forget to update commit message. Sorry about that. 👍


OpenId4Vc Support

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

Comparison is base (a641a96) 62.69% compared to head (a717378) 62.53%.

Files
Patch %
Lines

...r/src/presentations/PresentationExchangeService.ts
3.46%
152 Missing and 15 partials :warning:

.../presentations/selection/PexCredentialSelection.ts
6.45%
79 Missing and 8 partials :warning:

...4vc-holder/src/issuance/OpenId4VciHolderService.ts
80.07%
51 Missing and 2 partials :warning:

...holder/src/presentations/OpenId4VpHolderService.ts
43.42%
40 Missing and 3 partials :warning:

...es/openid4vc-holder/src/presentations/transform.ts
17.24%
24 Missing :warning:

...vc-holder/src/issuance/utils/claimFormatMapping.ts
46.15%
14 Missing :warning:

...c-holder/src/issuance/utils/IssuerMetadataUtils.ts
87.01%
10 Missing :warning:

...ackages/openid4vc-holder/src/OpenId4VcHolderApi.ts
88.00%
3 Missing :warning:

...ges/openid4vc-holder/src/issuance/utils/Formats.ts
83.33%
3 Missing :warning:

...odules/vc/data-integrity/SignatureSuiteRegistry.ts
0.00%
1 Missing :warning:

... and 1 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1639      +/-   ##
==========================================
- Coverage   62.69%   62.53%   -0.16%
==========================================
  Files         779      750      -29
  Lines       18128    17095    -1033
  Branches     3128     3047      -81
==========================================
- Hits        11365    10691     -674
+ Misses       6219     5862     -357
+ Partials      544      542       -2

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

We're eager to use these changes to test against an ACA-Py plugin we're building for OpenID4VCI.

We're eager to use these changes to test against an ACA-Py plugin we're building for OpenID4VCI.

Cool @dbluhm! Which format profiles are you supporting? We're focussing on JWT / SD-JWT for now

We're definitely taking an iterative approach but we're focusing on JWT first. SD-JWTs will likely come next. We've been testing against the sphereon wallet and I believe this is using the Sphereon SDK so hopefully that bodes well for compatibility!
Are you basing the SD-JWT format off of the HAIP? That's the only place I've seen it defined so far but there are some restrictions in that profile we're not thrilled about.

Are you basing the SD-JWT format off of the HAIP? That's the only place I've seen it defined so far but there are some restrictions in that profile we're not thrilled about.

Yes we're basing it off haip
Which restrictions are you not so thrilled about? I think our main concern was did support, but the spec recommends to use jwk, but you can use other ways to bind the credential to e.g. a did


fix(time): bar bandWidth with time axis #11145

fix(time): bar bandWidth with time axis #11145
before fixing (see bar2.html):

after fixing:

Bandwidth is calculated using the min gap between two adjacent time values of all series.
There's a bug with data filtering as reported in #11478, which is not raised in this pull request and can be reproduced with v4.4.0.

LGTM now


Geometry.js refers to Face3 and SolidColorMaterial which do not exist.

http://vorg.github.io/pex/docs/pex-geom/Geometry.html
face should be an array of vertices
material should be: var material = new materials.SolidColor();
Indeed this example isn't working at all it seems.

Fixed in https://github.com/vorg/pex-geom/commit/d69d057660e94188ddc6c9c2e2c37f70160b22f3
For a full working example please see here http://vorg.github.io/pex/examples/geom.Geometry/


Pushing signed images does not work as expected

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
If I push an image into quay.io via podman push --sign-by and the repository has content trust enabled, then the pushed tag does still not show-up as signed.
Steps to reproduce the issue:

Create a quay.io repo with enabled content trust:

Push a signed image:

> sudo podman pull docker.io/alpine
Trying to pull docker.io/alpine...
Getting image source signatures
Copying blob c9b1b535fdd9 [--------------------------------------] 0.0b / 0.0b
Copying config e7d92cdc71 done
Writing manifest to image destination
Storing signatures
e7d92cdc71feacf90708cb59182d0df1b911f8ae022d29e8e95d75ca6a99776a

> sudo podman tag docker.io/alpine quay.io/saschagrunert/alpine
> sudo podman push --sign-by<EMAIL_ADDRESS>quay.io/saschagrunert/alpine
Getting image source signatures
Copying blob 5216338b40a7 done
Copying config e7d92cdc71 done
Writing manifest to image destination
Signing manifest
Storing signatures

Verify that the image is not signed:

Describe the results you received:
The image is not signed, which can also be verified on a podman pull:
> cat policy.json
{
  "default": [{ "type": "insecureAcceptAnything" }],
  "transports": {
    "docker": {
      "quay.io/saschagrunert": [
        {
          "type": "signedBy",
          "keyType": "GPGKeys",
          "keyPath": "/home/sascha/.gnupg/trustdb.gpg"
        }
      ]
    }
  }
}
> sudo podman pull --signature-policy=policy.json quay.io/saschagrunert/alpine
Trying to pull quay.io/saschagrunert/alpine...
  A signature was required, but no signature exists
Error: error pulling image "quay.io/saschagrunert/alpine": unable to pull quay.io/saschagrunert/alpine: unable to pull image: Source image rejected: A signature was required, but no signature exists

Describe the results you expected:
A signed image should be pushed to the remote, right?
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
> podman version
Version:            1.8.0
RemoteAPI Version:  1
Go Version:         go1.12.17
OS/Arch:            linux/amd64

Output of podman info --debug:
> sudo podman info --debug
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.17
  podman version: 1.8.0
host:
  BuildahVersion: 1.13.1
  CgroupVersion: v1
  Conmon:
    package: conmon-2.0.10-25.1.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.10, commit: unknown'
  Distribution:
    distribution: '"opensuse-tumbleweed"'
    version: "20200229"
  MemFree:<PHONE_NUMBER>0
  MemTotal:<PHONE_NUMBER>8
  OCIRuntime:
    name: runc
    package: runc-1.0.0~rc10-2.1.x86_64
    path: /usr/sbin/runc
    version: |-
      runc version 1.0.0-rc10
      spec: 1.0.1-dev
  SwapFree: 0
  SwapTotal: 0
  arch: amd64
  cpus: 8
  eventlogger: file
  hostname: ws-sascha
  kernel: 5.5.6-1-default
  os: linux
  rootless: false
  uptime: 2h 32m 14.72s (Approximately 0.08 days)
registries:
  search:
  - docker.io
store:
  ConfigFile: /etc/containers/storage.conf
  ContainerStore:
    number: 0
  GraphDriverName: overlay
  GraphOptions: {}
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: btrfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 1
  RunRoot: /var/run/containers/storage
  VolumePath: /var/lib/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
> rpm -q podman
podman-1.8.0-2.2.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
I'm on the latest openSUSE Tumbleweed.

@vrothberg @mtrmac I think this is quay.io, does not understand simple signing.  And expects things to be signed by notary?

@vrothberg @mtrmac I think this is quay.io, does not understand simple signing. And expects things to be signed by notary?

My follow-up question on this would be: Which registry understands GPG-based signing?

Quay uses a Notary (-like ?) signing system.
“Simple signing” (the c/image thing) does not quite require registry support; the signatures are stand-alone blobs.
Anyway, the two are not interoperable at all.
AFAIK, there is only one implementation with direct support and no configuration required (just pull/push and the signatures are automatically included), the OpenShift integrated registry; for any other registry the signatures can be distributed separately using a static web server (and every client has to be configured with the location of that web server).
See https://github.com/containers/image/blob/master/docs/signature-protocols.md for the technical details, and maybe https://access.redhat.com/solutions/4039981 (subscription required) or https://medium.com/cri-o/secure-your-kubernetes-production-cluster-7ec607749583 for a tutorial (though I haven’t re-read any of the three documents right now to ensure accuracy, and at least the CRI-O blog has an indentation error in the sigstore-staging configuration).

Thanks for the clarification @mtrmac :pray: I wrote up a little tutorial in a GitHub gist. I'm wondering if it would be worth the effort to add it to this repo. WDYT?

The tutorial looks great. (Two nits: The default does not point at localhost:8000, and for general use mentioning the X-R-S-S extension = OpenShift integrated registry, at least in passing, might be useful.)
Adding it to https://github.com/containers/libpod/tree/master/docs/tutorials makes sense to me, but that’s up to the maintainers of the Podman repo. Or maybe a blog post? Or both 😎

I'd love to have the tutorial in either or both spots.   Dropping it in the tutorial section in Podman and I'd be happy to write a little blog post that links to it if you didn't want to on podman.io.

The tutorial looks great. (Two nits: The default does not point at localhost:8000, and for general use mentioning the X-R-S-S extension = OpenShift integrated registry, at least in passing, might be useful.)
Adding it to https://github.com/containers/libpod/tree/master/docs/tutorials makes sense to me, but that’s up to the maintainers of the Podman repo. Or maybe a blog post? Or both

Alright cool, I'll prepare the PR so we can close this for now. :pray:


Code climate

Add code climate support by updating karma.config and .travis.yml
Added badges to README

We might want to add the badge for test coverage, if possible later on.
Otherwise, great job!

Should I do this on this PR?

I would this on this PR if that’s ok with you

I would this on this PR if that’s ok with you

I have just added the badges to the README


angular.uppercase is not a function on angular >= 1.5.0

angular.uppercase has been deprecated since v1.5.0 and now in v1.6.10 and v1.7 has been deleted.
According to this changelog: https://github.com/angular/angular.js/commit/1daa4f2231a89ee88345689f001805ffffa9e7de
angular.uppercase should not be used and instead use the internal toUpperCase.
If this is an acceptable change I can try and do a PR.

Working on this...


Extend CfxBridge to be more evm compatible

Core space endpoint are not mandatory;
Multiple batch receipts implementation support.

This change is

Already merged in to #205


[docs] Add a note about advanced features in browsers launched with the all browser alias or with with a path

Browsers that are launched:

with the all browser alias
with a path

testcafe "path:`C:/Program Files (x86)/Google/Chrome/Application/chrome.exe` --lang=fr","path:`C:/Program Files (x86)/Google/Chrome/Application/chrome.exe` --lang=es" test.js

don't use browser-specific providers. Therefore, advanced features (like temporary clean profile) don't work in browsers launched this way.
We need to be more explicit about this (perhaps a note) in the docs.

We don't plan to address this issue in the near future. Feel free to reopen the ticket if the issue is important for you.


Run as command line tool

Hello! Would it be possible to use CSScheme as command line tool so it would be useful not only inside of Sublime?

Not at this moment, but it's certainly a good idea.


^  Error: Cannot find module 'pdfjs-dist' Require stack:

After installing the npm package I am getting this error. any solution?

Yes, install npm i pdfjs-dist.

Thanks for the response but I tried this too. but getting issues with pdfjs-dist. Any help on this?
const password = src.password ?? null;
^
SyntaxError: Unexpected token '?'

That doesn't sound like an easyinvoice or pdfjs-dist issue. So sorry, can't help you there.

this issue is coming after installing pdfjs-dist package only. anyway, thanks for your time.

This issue was coming because of the older node version. resolved now.


regex rule failing with whitespace

I have a regex rule in my grammar that looks like this (modified from the python sample):
comment: /\/\/[^\n]*/
Which matches fine when a comment does NOT have a trailing space, like this:
//I am a comment
However, when there IS a trailing space, like this:
//I am a comment
It fails, resulting in a ''Incomplete parse: Could not find a solution to input'' error.
The regex works fine in python, like so:
>>> teststring = "//I am a comment "
>>> c = re.compile('\/\/[^\n]*')
>>> c.match(teststring)
<_sre.SRE_Match object; span=(0, 17), match='//I am a comment '>

which does match the trailing whitespace

This does not appear to be the case when using parser='lalr'

If you can use 'lalr', then do so. It's also significantly faster.
Can you check if it happens with parser='earley', lexer=None ?

parser="earley", lexer=None:
Fails
parser="earley":
Fails
lexer=None:
'Scanless parsing (lexer=None) requires all tokens to have a width of 1 (terminal ANONRE_15: [0-9]+ is (1,<PHONE_NUMBER>))'

Ok. If at any point you want to provide an example that's failing, I'll be happy to fix it.

Here is a minimal example in 2 files:
file grammar.g:
start: comment+

comment: /\/\/[^\n]*/

%import common.ESCAPED_STRING
%import common.WS
%ignore WS

sample.py
from lark import Lark

comments = '''//no whitespace
//whitespace '''

with open('grammar.g','r') as grammarfile:
    parser = Lark(grammarfile.read(), parser='lalr')
    print(parser.parse(comments).pretty())

    grammarfile.seek(0)

    parser_def = Lark(grammarfile.read())
    print(parser_def.parse(comments).pretty())

Hi, sorry but your example code runs perfectly. Can you make sure you're using commit ec7da3b ?

(It's the latest commit to master)


[DRAFT] Prototype for Exporting DV Frames

Prototype for @sriyerg

Ok, I followed what we did with sw/otbn/meson.build, in terms of using dictionaries rather than parallel lists.


Panic in decision_tree

Increasing the max_features in my hyperparameters seems to make this more likely.
let mut tree_params = decision_tree::Hyperparameters::new(feat_matrix.cols());
tree_params.min_samples_split(10)
           .rng(StdRng::from_seed(&[100]));

let mut model = Hyperparameters::new(tree_params, 20)
                    .rng(StdRng::from_seed(&[100]))
                    .one_vs_rest();

The data is shape:
rows: 2247
cols: 442
thread 'main' panicked at 'index out of bounds: the len is 10 but the index is 11', ../src/libcore/slice.rs:442
The len/index number changes, but it's always an off by one.
9:     0x55ede9c87a43 - core::panicking::panic_bounds_check::ha883fe1527ce6884
10:     0x55ede9a1e0a2 - <[T] as core..slice..SliceExt>::swap::h9bf7fe18e2fa8251
at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/obj/../src/libcore/slice.rs:442
11:     0x55ede9a12fdc - collections::slice::<impl [T]>::swap::h6559c8ef586608a3
at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/obj/../src/libcollections/slice.rs:479
12:     0x55ede9a27b9e - rustlearn::trees::decision_tree::FeatureIndices::mark_as_used::hc0584f311f525fd4
at /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/rustlearn-0.4.0/src/trees/decision_tree.rs:103
13:     0x55ede9a298ae - rustlearn::trees::decision_tree::DecisionTree::build_tree::hdb922dd21da7568d

Thanks for the report, I'll look into it.

I added a tentative fix in #29.
You can test it by replacing you Cargo.toml entry for rustlearn with:
rustlearn = { git = "https://github.com/maciejkula/rustlearn.git", rev = "77b05b1717ce0bcf6070509956c251a9aa031baf" }

Please let me know if that fixes it!

This looks to have solved it - thank you.

The fix is now included in rustlearn 0.4.1 on crates.io. Thanks for reporting the bug!


[BUG] Navigate to the bottom of the chat

Chat should be scrolled at the bottom in following cases.

When a user login's in the group
After login, if the user scrolls above in the chat and if sends a message.

Hey , can you  assign me this?

I also want to part of this project. Please assign me.

Hi @Shweta-281,
Thank you for your interest.
This issue is being currently assigned to @anglerfishlyy. Please wait for a couple of days.
In the mean-time you can view our existing issues here.

Hi @anglerfishlyy,
Any updates?

Hi @jatiinyadav Please assign this issue to me


Add app quality to extension API

There are times we want to know the app quality in extensions - and while they can get it by reading the product.json like MSSQL was doing it'd be a lot easier/safer to instead add it to the extension API directly.
I chose to make it default to dev when there isn't a quality since that's the only (supported) scenario that'd happen right now. Every released version should have an actual quality.
I also fixed an issue with query history persistence - the storage folder is the same for all flavors of ADS currently but the secret storage is per-flavor so what was happening was that we would generate a new key/IV for each flavor which would then fail to decrypt the persisted file. This seems like fine behavior to me - I'm more of the opinion that the different flavors shouldn't actually share any state at all (like VS Code does) so having separate histories for each flavor is going to be expected behavior for now.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
28 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.006%) to 42.351%

Files with Coverage Reduction
New Missed Lines
%

extensions/query-history/src/queryHistoryProvider.ts
28
68.72%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
29381

Relevant Lines:
64550

💛  - Coveralls


STOR-2017: Bump all deps for 4.18

Bump all deps for 4.18
Notes:

Added spec.unhealthyEvictionPolicy: AlwaysAllow to all pdbs

/hold

/unhold

/hold

/test unit

/unhold

/hold there is an issue with CI monitor tests after upgrading library-go so we hold the bump for now

/close


Deploy release candidates to local maven repo for dependency check[skip ci]

Deploy release candidates to local maven repo for dependency check
To fix : https://github.com/NVIDIA/spark-rapids/issues/10164
Deploy release candidates to local maven repo leveraging the release deploy script.
Instead of using internal snapshots maven repo(having all the intermediate artifacts),
we check dependencies for the release candidates using the local maven repo,
which only contains release candidates.
Signed-off-by: Tim Liu<EMAIL_ADDRESS>

After this change get merged, we'll enable the release candidates dependency checking in our nightly Build CI job
@gerashegalov  Can you help review

Looks good already, just need to double-check if the test fails if we omit deploy-file of jdk-profiles

I ran the jenkins/dependency-check.sh script in CI the job, it would FAIL unless all the dependency jar/pom are available in the local-remote maven repo

build

LGTM
Is there a reason not to call dependency-check.sh from spark-nightly-build.sh so we do not have to guess how these two scripts are related? Are you going to add the Jenkins pipeline step for it in this repo?

Yes, I will add the dependency check into the Jenkins pipeline into another stage(dependency check).
The reason why not calling from spark-nightly-build.sh is that:
spark-nightly-build.sh has script to deploy anything into maven internal repo, if we add this check into nightly build script, there is another deploy call (deploy.sh[only for public artifacts]) plus dependency-check.sh, making nightly build script odd and complexed

build

I'll enable the dependency check in our internal nightly build CI, it will run in tonight's plugin nightly job  @gerashegalov  @sameerz


duration-[xs] is not working

arbitrary values for duration are no longer working after installing the package in a nextjs app

?

warn - The class `duration-[2000ms]` is ambiguous and matches multiple utilities.
warn - If this is content and not a class, replace it with `duration-[2000ms]` to silence this warning.

https://github.com/jamiebuilds/tailwindcss-animate/blob/main/docs/animation-duration.md?plain=1#L16 to be precise

A lot of open issues on this repo (#13, #35, #38, #50, #51) are due to these 3 conflicting classes (duration, delay, ease).
Till #46 gets merged by @jamiebuilds, you can use my fork where I have just updated all the conflicting classes, e.g.

duration-300 becomes anim-duration-300, etc.
delay-150 becomes anim-delay-150, etc.
ease-in becomes anim-ease-in, etc.

@jamiebuilds  Any updates on this? @anurag-roy thanks for the alternative 👍


Synchronize calls to eventHandler.handle

It turns out that having multiple async tests in a suite can cause the
tests to fail even when all of the individual tests pass. The issue is
that the eventHandle::handle method calls ArrayList::add, which is not
thread safe. If multiple threads call it at the same time, a null
object may be added to the result list that the child process sends back
to the parent. This manifests as an NPE in sbt.
This is really a bug in sbt, which should use a Vector instead of an
ArrayList, but I think the synchronization cost is likely minimal and
will fix the issue for all sbt versions.

I also made an upstream PR to sbt for this issue:
https://github.com/sbt/sbt/pull/4028

Nice catch, thanks @eatkins


test: use upstream TowerToHyperService

tower's MapRequest utility can be used to map request to use upstream TowerToHyperService.

Hi @tottoto. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.


tomcat test

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

No information provided - Closing


[Landing page] AAFE dev I have moved in depth installation instructions to an Install page

Description:
Landing page needs to look super clean.
The installation instructions and usage should move to another page.
Validation:
screenshots
Tech steps
make it look like this. https://jestjs.io/docs/configuration
easy installation.


kafka docker image is not support arm

Description
ghcr.io/banzaicloud/kafka:2.13-3.4.1  is not support  arm ? has plan in feature?
Expected Behavior
Please describe what you expected would happen.
Actual Behavior
Please describe what happened instead.
Affected Version
Please provide the version number where this issue was encountered.
Steps to Reproduce

First step
Second step
etc.

Checklist

[ ] I have read the contributing guidelines
[ ] I have verified this does not duplicate an existing issue

I believe this issue is a duplicate of https://github.com/banzaicloud/koperator/issues/1058

I believe this issue is a duplicate of https://github.com/banzaicloud/koperator/issues/1058

I saw it, but it doesn‘t seem to be. I want to say the running image of kafka, not the image of the operator.

Oh my bad, we currently don't have plan of supporting arm64 for the Kafka image just yet. It would be great if you can provide some help on this

[ ] kafka  https://github.com/banzaicloud/docker-kafka
[ ] kafka-connect https://github.com/banzaicloud/docker-kafka-connect
[ ] jmx-exporter https://github.com/banzaicloud/docker-jmx-exporter
[ ] cruise-control https://github.com/banzaicloud/go-cruise-control

But I don't seem to have found the cruise-control dockerfile. Could you please confirm that these are the only dependent docker images?


Figure out reward system

Finalize and describe what a user get awarded for different accomplishments

This is the idea I have come up with to determine a reward system.
1 point : trash taken out 3 times in a row
1 point : trash taken out 8 times in total
5 points are equal to 1 month of a pass. You also allocate 2 points bonus for the next month.
Some things that are acceptable:

Users are allowed to intervene and block others from getting consecutive points

What do you think about these rules @f-0-o


chore: Refactoring create query operation helpers to include modules in the list

Description
Refactoring create query operation helpers to include modules in the list
PR fixes following issue(s)
Fixes #26165 #26506
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing
How Has This Been Tested?

[x] Manual
[ ] JUnit
[ ] Jest
[x] Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test


writeToCsv() doesn't support the workbook column datatypes

The writeToCsv function doesn't support the workbook column datatypes. In my case, I have a date properly formatted as yyyy-mm-dd, but when it generates the CSV, queryToArrayForCsv() in csv.cfc detects the value as a date, and then formats it as datetime. It would be good to allow a format to be specified instead, and then default to datetime if nothing is passed in.

Hi Daniel. I'll have a think about this. Specifying data types is normally only applicable when passing data into a workbook.
I can't remember exactly why queryToArrayForCsv() forces the datetime format even if the value is a string. Perhaps the solution is that it should respect the cell type regardless. But you could work around the current behaviour by overriding the default datetime format to make it date only just for this operation:
Spreadsheet = New spreadsheet.SpreadSheet( dateFormats: { DATETIME: "yyyy-mm-dd" } )
data = QueryNew( "date", "VarChar", [ "2002-01-01" ] )
wb = spreadsheet.newXlsx()
spreadsheet
  .addRows( wb, data )
  .writeToCsv( wb, "c:/temp/test.csv", true )

In my case, I am first creating a workbook using spreadsheet.workbookFromQuery() where I pass in the data and datatypes. After I have that, if the file extension is .csv, I call spreadsheet.writeToCsv(), else I call spreadsheet.write().

How you create the workbook doesn't matter. The issue is with writeToCsv(). But as I say you should be able to control the format of your date value when written to the csv file by overriding dateFormats.DATETIME.

I've changed the csv handling so that it will no longer mess with date strings (i.e. a VarChar value formatted as a date). This should solve your issue without you having to override the DATETIME format.


wrap connection error

Then it's possible to assign error by errors.As to *net.OpError.
It allows us to determine if connection timeouted or was refused.

thx for opening a PR!
Please add a test that fails without your fix.

@microsoft-github-policy-service agree [company="Dataddo"]

@shueybubbles Hi, I have added tests, but when it comes to network, testing/mocking becomes a bit tricky.

@microsoft-github-policy-service agree

Please add appropriate verbiage to the CHANGELOG.md in the root and I'll merge

The tests are failing on Windows and older Go versions, pls check the appveyor results.
older go:
.\tds_test.go:935:7: undefined: errors.As
.\tds_test.go:957:7: undefined: errors.As
FAIL	github.com/microsoft/go-mssqldb [build failed]
Command exited with code 2

Windows with newer go and shared memory protocol:
--- FAIL: TestConnectError (0.00s)
    --- FAIL: TestConnectError/bad_port_-_refused_connection (0.00s)
        tds_test.go:936: Expected *net.OpError, got: *errors.errorString: Cannot open a Shared Memory connection to a remote SQL server
    --- FAIL: TestConnectError/bad_addr_-_host_will_keep_us_hanging (0.00s)
        tds_test.go:958: Expected *net.OpError, got: *errors.errorString: Cannot open a Shared Memory connection to a remote SQL server
2023/01/19 22:31:50 [{Hello} {World} {TVP}]
FAIL
coverage: 79.8% of statements
exit status 1
FAIL	github.com/microsoft/go-mssqldb	24.880s
Command exited with code 1

The tests are failing on Windows and older Go versions, pls check the appveyor results.
older go:
.\tds_test.go:935:7: undefined: errors.As
.\tds_test.go:957:7: undefined: errors.As
FAIL	github.com/microsoft/go-mssqldb [build failed]
Command exited with code 2

Do we really want to support version before Go 1.13?
https://pkg.go.dev/errors#As was added in Go 1.13, without it there is very limited way for error handling.
Also, there is no reason to stick with old version while Go has its backward compatibility promise.
Maybe some discontinued platforms, but that's very rare case and I can specify one from a head.
If we wanna support old versions, I can add build tags though. But it adds complexity and inconsistent behaviour across versions.

i will fix the test issues for lpc and np


[rest] add joiner to commissioner joiner table and get status of joiner …

…by indirect 'addThreadDeviceTask' processing
This PR is a carve-out from PR2514  to ease review. It implements the 'addThreadDeviceTask' to add a Joiner and auto-start on-mesh commissioner after POST:

POST addThreadDeviceTask on api/actions/
GET api/actions and GET api/actions/actionId
DELETE api/actions

The commit also provides limited integration tests, see tests/restjsonapi.
Please follow below steps to reproduce and install/build OTBR.

Checkout this PR

Build and Install OTBR as usual, e.g. on a Raspberry Pi

Restart the OTBR. sudo systemctl restart otbr-agent

To monitor the log [Errors|Warnings|Info] please open a different terminal instance and use following command:

tail -f /var/log/syslog | grep otbr

Send POST request using Bruno or cURL to join a new device into your network.

curl -X POST -H 'Content-Type: application/vnd.api+json' http://localhost:8081/api/actions -d '{"data": [{"type": "addThreadDeviceTask", "attributes": {"eui": "6234567890AACDEA", "pskd": "J01NME", "timeout": 3600}}]}' | jq

should return
{
  "data": [
    {
      "id": "2d5a8844-b1bc-4f02-93f0-d87b8c3b4e92",
      "type": "addThreadDeviceTask",
      "attributes": {
        "eui": "6234567890AACDEB",
        "pskd": "J01NME",
        "timeout": 3600,
        "status": "pending"
      },
    }
  ]
}

You may check the status and get the full collection of actions.

curl -X GET -H 'Accept: application/vnd.api+json' http://localhost:8081/api/actions | jq

should return
{
  "data": [
    {
      "id": "2d5a8844-b1bc-4f02-93f0-d87b8c3b4e92",
      "type": "addThreadDeviceTask",
      "attributes": {
        "eui": "6234567890AACDEB",
        "pskd": "J01NME",
        "timeout": 3600,
        "status": "pending"
      }
    }
  ],
  "meta": {
    "collection": {
      "offset": 0,
      "limit": 100,
      "total": 1
    }
  }
}

View the entry added to the commissioner's table sudo ot-ctl commissioner joiner table and expect

| ID                    | PSKd                             | Expiration |
+-----------------------+----------------------------------+------------+
|<PHONE_NUMBER>aacdea |                           J01NME |    3459027 |
Done

Start your joiner and after a few seconds repeat above steps 6. and 7.

For running the included test script install Bruno-Cli and run the bash script on your border router

cd tests/restjsonapi
source ./install_bruno_cli
./test-restjsonapi-server

Please be aware that there is also another PR adding the commissioner feature to REST API https://github.com/openthread/ot-br-posix/pull/2515


Enable mobile mode for HtmlWebpackPlugin

Most of the pages are at least readable ;-)
Catalog already supports responsiveness.

Awesome. Deployed. 👍

Cool, looks good on my iPhone!

How did you figure out that flag? On a quick look I can't see a direct reference to it in the source.

I found it in the template: https://github.com/reactabular/reactabular/blob/master/templates/index.ejs#L14-L16

Ah, yeah. Forgot about that entirely. Thanks. 👍


Performance - QuickOpen: Slow search performance in large repository

Oni Version: 0.3.6
Operating System: Windows
Issue:
The QuickOpen experience is slow in this project (on my machine): https://github.com/esy/esy. It does have quite a few files, especially if you bring in submodules (git submodule update --init --recursive). But this might be related to some of the slowdowns users have reported on large projects w/ QuickOpen.
It's kinda hard to see here, because the KeyDisplayer also stalls, but I'm continuously typing after esya:

Steps to reproduce:

git clone https://github.com/esy/esy
git submodulate init --update --recursive
oni esy
<C-p>
esyajaigoa

I also grabbed a profile:
performance-profile.zip
Looks like the bottleneck is definitely on the javascript side:

I think it might be here:
export const processSearchTerm = (
    searchString: string,
    options: Oni.Menu.MenuOption[],
    isCaseSensitive: boolean,
): Oni.Menu.MenuOption[] => {
    const filterRegExp = new RegExp(".*" + searchString.split("").join(".*") + ".*")

    return options.filter(f => {
        let textToFilterOn = f.detail + f.label

        if (!isCaseSensitive) {
            textToFilterOn = textToFilterOn.toLowerCase()
        }

        return textToFilterOn.match(filterRegExp)
    })
}

The regexp matching might be expensive, for a large amount of items. We could try doing a cheaper prune first potentially.

One thought. Would it be difficult to have the search be incremental. It could stop once it's found enough results to fill up the overlay (ten to fifteen), and then in another event loop, perform the rest of the search (adding results). The scrollbar jumping as it adds results can be jarring which is a downside.

with vim I use fzf.vim and ag(silver searcher) as the executable. Its amazingly fast on our huge monorepo.  I hope we can get this working in oni.
if executable('ag')
  " Use ag over grep
  set grepprg=ag\ --nogroup\ --nocolor

  " Use ag in CtrlP for listing files. Lightning fast and respects .gitignore
  let g:ctrlp_user_command = 'ag %s -l --nocolor --hidden  -g ""'
endif

@asa,
Noted, might give it a try later on.


PI-8488 changed dispatch order of getSearchResults and update history

Coverage remained the same at 32.634% when pulling 8a80a51cdee7754b3ed325c2b8baa265126ce0e2 on PI-8488-multiple-searches-not-working into d5e77954b24ee2a4f5241e39dd879197dc42adbb on master.


Finish sentence

Lots of things could go in here. I also considered "...from an item that is not yet overdue"

how about this:
"... or an overdue Loan might be distinguished from one that is fully paid."

Ok, so why not ...
"or a library book that is loaned out might be distinguished from one that is available."
Do you want me to wait for you to update this PR, or shall I do a separate commit?

Happy for you do do it.

changes committed to master
Thanks for the contributions Brad!


Understand the functionality of Catalog Uploads screen

This task is created for logging in efforts for understanding the functionality of Catalog Uploads screen.

Task is done.