Dispose VBOs created during mipmap generation

I don't think this would've caused any problems because these are periodically free'd via
https://github.com/ppy/osu-framework/blob/7c5b248198ee8bfa6b65e56e710afc3a3af45f39/osu.Framework/Graphics/Rendering/Renderer.cs#L811-L823

This branch somehow is causing segfaults on my linux setup (ubuntu 22.04, NVIDIA GeForce RTX 3060 laptop version, nvidia proprietary drivers) that are not there on master:
[11570.418684] Draw (GameThrea[102898]: segfault at 0 ip 00007fa7fa9ea600 sp 00007fa800df8ae0 error 4 in libnvidia-glcore.so.525.105.17[7fa7f96fc000+2505000]
[11570.418695] Code: f8 41 8d 57 ff 44 29 fb c1 e0 10 25 00 00 ff 1f 0d 7c 05 00 60 89 01 48 8d 41 04 48 8d 4c 91 08 4c 89 e2 0f 1f 80 00 00 00 00 <0f> b7 32 48 83 c0 04 48 83 c2 04 44 01 f6 66 89 70 fc 0f b7 72 fe

Not sure where to even begin with this, but I'll have a poke...

Following patch appears to fix the segfault:
diff --git a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
index 524fa8d57..4d66540b4 100644
--- a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
+++ b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
@@ -81,7 +81,7 @@ protected virtual void Initialise()

         public void Dispose()
         {
-            Dispose(true);
+            Renderer.ScheduleDisposal(r => r.Dispose(true), this);
             GC.SuppressFinalize(this);
         }

I would be lying if I said I'm entirely following why, but that was the primary difference between GLVertexBuffer's and VeldridVertexBuffer's disposal. And the GLVertexBuffer destructor does that too, so it seems correct?

Hmm, that's pretty weird. It should definitely be a valid operation to delete a VBO after a glDrawX... Does it also fail if VeldridVertexBuffer is changed to dispose immediately?

Does it also fail if VeldridVertexBuffer is changed to dispose immediately?

Removing the schedule for veldrid vertex buffer does not seem to reproduce the same segfault, no. I have no clue why this is the case, but that's what I'm seeing.

Or… are you able to repro in debug? Can you check if it fails on the glDelete calls or potentially a call later on?

Debug doesn't do much here since it's a segfault, and so the kernel just kills the game process. What I can say, though, is that if I do something along the lines of
diff --git a/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs b/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs
index 904f20812..175fee0b8 100644
--- a/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs
+++ b/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs
@@ -314,8 +314,8 @@ public unsafe bool Upload()
                         // Perform the actual mip level draw
                         Renderer.PushViewport(new RectangleI(0, 0, width, height));

-                        quadBuffer.Update();
-                        quadBuffer.Draw();
+                        //quadBuffer.Update();
+                        //quadBuffer.Draw();

                         Renderer.PopViewport();
                     }

then the game looks broken (because mipmaps are now empty), but at least it stops segfaulting. So to my uneducated eye this appears to be an indication that the nvidia driver might be reordering gl calls such that both the update and draw calls end up being called after the VBO is disposed causing it to panic. It is also both calls that need to be commented out to observe this, commenting just any one of those two without commenting the other still causes a crash.

You could try inserting a glFinish after both calls to see if that's the case.
Another thing comes to mind is if VAOs should be unbound before deleting it. Veldrid for example generates one VAO and always keeps it bound for the lifetime of the graphics device. It would make sense why it doesn't fail when scheduling the disposal, because another VAO would've already been bound before deleting the current one.
Try this and see if it helps:
diff --git a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
index 524fa8d57..f94d2b0f3 100644
--- a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
+++ b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
@@ -177,6 +177,8 @@ void IVertexBuffer.Free()
         {
             if (isInitialised)
             {
+                GL.BindVertexArray(0);
+
                 GL.DeleteBuffer(vboId);
                 GL.DeleteVertexArray(vaoId);
             }

You could try inserting a glFinish after both calls to see if that's the case.

I tried applying this:
diff --git a/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs b/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs
index 904f20812..3b081c2aa 100644
--- a/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs
+++ b/osu.Framework/Graphics/OpenGL/Textures/GLTexture.cs
@@ -316,6 +316,7 @@ public unsafe bool Upload()

                         quadBuffer.Update();
                         quadBuffer.Draw();
+                        GL.Finish();

                         Renderer.PopViewport();
                     }

and it still segfaults.

Try this and see if it helps:
diff --git a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
index 524fa8d57..f94d2b0f3 100644
--- a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
+++ b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
@@ -177,6 +177,8 @@ void IVertexBuffer.Free()
         {
             if (isInitialised)
             {
+                GL.BindVertexArray(0);
+
                 GL.DeleteBuffer(vboId);
                 GL.DeleteVertexArray(vaoId);
             }

This one doesn't work either, but:

diff --git a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
index 524fa8d57..0fa00d9b4 100644
--- a/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
+++ b/osu.Framework/Graphics/OpenGL/Buffers/GLVertexBuffer.cs
@@ -114,6 +114,7 @@ public void Bind(bool forRendering)

         public virtual void Unbind()
         {
+            Renderer.BindVertexArray(0);
         }

         protected virtual int ToElements(int vertices) => vertices;
@@ -133,6 +134,8 @@ public void DrawRange(int startIndex, int endIndex)

             int countVertices = endIndex - startIndex;
             GL.DrawElements(Type, ToElements(countVertices), DrawElementsType.UnsignedShort, (IntPtr)(ToElementIndex(startIndex) * sizeof(ushort)));
+
+            Unbind();
         }

         public void Update()
@@ -150,6 +153,8 @@ public void UpdateRange(int startIndex, int endIndex)
             GL.BufferSubData(BufferTarget.ArrayBuffer, (IntPtr)(startIndex * STRIDE), (IntPtr)(countVertices * STRIDE), ref getMemory().Span[startIndex]);

             FrameStatistics.Add(StatisticsCounterType.VerticesUpl, countVertices);
+
+            Unbind();
         }

         private ref Memory<DepthWrappingVertex<T>> getMemory()

this one appears to stop the crashes.

That's interesting, I'm also wondering if commenting the glDeleteVertexArray operation stops the crash as well. Normally we wouldn't want to delete VAOs at all, but cache them for activating a specific set of vertex attributes.


Integrates recent changes from 'fsharp/fsharp master' into 'fsharp/fsharp branch fsharp4'

This brings the 'fsharp4' branch up to date with recent work in 'master'

Closing this in favour of #395


Add Algorithms in Java | As many as you can!

please assign me this issue.

please update the assignees field

Please assign me in this issue, I have lots of Java data structures and algorithms problem solutions

@nishant-ai  please review my pull request. #140


Manage changelog

I vote for making it automatic by following commit message conventions.
https://www.conventionalcommits.org/en/v1.0.0/
https://github.com/conventional-changelog/standard-version

Standard Version seems like it could work well here

cool
https://github.com/conventional-changelog/commitlint


👤 user mapping error vs other system errors 💻

In my application, I needed to differentiate between users not mapped and domain trust issues. I thought the change may be useful for others too. Thanks for writing a terrific script.

bump

In my application, I needed to differentiate between users not mapped and domain trust issues. I thought the change may be useful for others too. Thanks for writing a terrific script.


Provide ability for wranglers to see the report of what's in the Terra staging area

Wranglers currently don't have access to the Terra staging area unless they use the ingest exporter GCP credentials.
Would be nice to be able to generate a nice report to help with testing:

list of files in a project's directory
list of all files in the whole Terra staging area

This could be just a script that a dev can run and give the output report to the wranglers. Something similar may have already been done before to test the DCP MVP2 exporting. @mshadbolt @ESapenaVentura do you know what was done before to test the DCP2 MVP?

@aaclan-ebi this is the little report doc I generated when we were trying to understand what was in the staging area https://rpubs.com/mshadbolt_ebi/653495
the way I generated it was that rolando would give me a text file dump of all the files in the staging bucket and I wrote code to summarise into various tables. Though I think the file structure of the staging bucket as now changed so I guess this wouldn't work anymore
Anyways happy to help with trying to decide what summary statistics might be useful when checking things are exported to the staging area

Cool! Thanks @mshadbolt , I believe this will be helpful! I think this just needs a small tweak to make it work with the new project subdirectory structure.

From meeting with @prabh-t and @rays22 yesterday, we agreed on giving wranglers access to the ingest exporter service account for testing.
https://ebi-ait.github.io/hca-ebi-dev-team/admin_setup/Setting-up-access-to-Terra-staging-area.html
It's up to them if they'll create a script to get a summary of files to help with testing. Could reuse and tweak Marion's tool or use python script to do this. @rays22 please let us know if you need any help on this! Very willing to help if needed!

This ticket is no longer needed as wranglers should now have access to the Terra staging area.

Just to clarify, we gave access to HCAWrangler user to retrieve the exporter's dev GCP credential for testing exporting in dev or staging. Afaik, it should have access to Terra dev bucket (dev and staging data are in same bucket atm).


[Build] Fix CI and Notebook builds

Get the CI and Notebook builds working again. The main fix involves the authentication to the AzureAI endpoints; we now have an Entra ID to use, rather than an api_key. The instructions are on the Microsoft site.
Also update the chat templates again, for Phi3 and Mistral. The fix for Mistral is not ideal. It appears that it now 'supports' a system message by inserting it into the first user message. I don't think our approach can really support this.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 28.57143% with 5 lines in your changes missing coverage. Please review.

Project coverage is 49.27%. Comparing base (6e4ee06) to head (ed035eb).
Report is 2 commits behind head on main.

Files
Patch %
Lines

guidance/chat.py
28.57%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

:exclamation:  There is a different number of reports uploaded between BASE (6e4ee06) and HEAD (ed035eb). Click for more details.
HEAD has 24 uploads less than BASE

Flag
BASE (6e4ee06)
HEAD (ed035eb)

28
4

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #940      +/-   ##
==========================================
- Coverage   55.54%   49.27%   -6.27%
==========================================
  Files          63       63
  Lines        4733     4779      +46
==========================================
- Hits         2629     2355     -274
- Misses       2104     2424     +320

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

I think it can happen in a subsequent PR, but we should really start moving towards the long-term system outlined in the original chat PR, where we have:

Tests that explicitly check if "tier 1" HF tokenizers are in the chat template cache (for heavy hitter OSS, like phi, mistral, llama2, llama3). Tokenizers are very cheap to load and run -- you don't have to download the whole model to use its tokenizer -- and we'll get early alerts about if there are ever chat template updates

Explicit round-trip tokenizer compatibility tests, where we pass a series of messages style conversations through both guidance and the huggingface tokenizers apply_chat_template method, and verify that we get the same resulting text string back out. These test conversations should have a bit of variety, e.g. including a system role, ending on user, ending on assistant, etc.

Once we have 2), we can start designing an auto jinja2 -> guidance Chat class syntax, where if a model is not in our cache, we can try to auto-generate the chat template class for it, and test it against our unit tests in 2). If it passes all the tests, our auto-converter probably did a decent job, if it doesn't, we can fall back to ChatML like we do today.


[Bug] Window not opening full/at the middle of the screen 窗口不能完全打开/位于屏幕中间

Describe the bug
the launcher's window is going to the bottom-right of the screen, even if you close it by placing at the middle/fullscreen
Steps to reproduce the bug
stackoverflow question
Expected behavior
No response
Environment
WinBuild: 22621.2715
Fluent Launcher Version: (latest) <IP_ADDRESS>
Attachment

Fix in <IP_ADDRESS>

uh ok but is it available? ms store doesn't show an update for it

oh yeah it needs certificate, otherwise not updated: ms msixbundle sign certificate

https://github.com/Xcube-Studio/Natsurainko.FluentLauncher/wiki/如何安装-Msixbundle-包

Thank you for your support as well

Thank you for your support as well

i did it! and thanks for repair! now it's much more good


ci: increase memory size of the nfs server

So that it is less likely to be killed by the OOM killer.

LGTM


parseToHsl Doesn't seem to parse valid CSS Colors 4 space-separated values

polished version: 4.1.3
Any relevant JS library and version: @storybook/react v6.4.19

What You Are Seeing
Parsing values such as 'hsl(235 100% 50%)' or 'hsl(246deg 37% 15% / 100%)' throws an error saying: Couldn't parse the color string. Please provide the color as a string in hex, rgb, rgba, hsl or hsla notation.
I'm just trying to run storybook and integrate new brand colors by our design team, which are written in as hsl values in space-separated format. Attempting to use certain values in the storybook theme throw an error. I'm unsure where exactly this is used in storybook's lib, as the error doesn't seem to get thrown on some places (e.g. MDX stories), but I was able to reproduce the error by calling the parseToHsl function manually. I believe I can hardcode a different syntax in the storybook theme as a temporary workaround, but I would expect this syntax to be supported here.
What You Expected To See
No error should be thrown, because that is valid syntax supported by most browsers according to MDN here and here.

Reproduction
Just try calling parseToHsl('hsl(235 100% 50%)') or parseToHsl('hsl(246deg 37% 15% / 100%)')
EDIT: Sandbox link: https://codesandbox.io/s/focused-agnesi-mw1t4r?file=/src/App.js

Related Storybook issue

I was able to write a temporary workaround (hack) to ensure my input to the function is transformed with the parse-css-color package
import parse from 'parse-css-color';

const backportHsl = str => {
  const { values } = parse(str);
  return `hsl(${values[0]}, ${values[1]}%, ${values[2]}%)`;
};

And although this is untested code you could probably handle both rgba and hsla with something loosely like the below:
// unverified idea
const backport = str => {
  const { type, values, alpha } = parse(str);
  const suffix = type === 'hsl' ? '%' : '';
  return `${type}a(${values[0]}, ${values[1] + suffix}, ${values[2] + suffix}, ${alpha})`;
};

@josh-degraw Thank you for raising this. I've verified the bug and am working on a fix.

Hey @bhough, can I please get this re-opened?
Looking at your tests it appears that you solved for hsl sans alpha, and hsla with alpha, but looking at the colour spec you can provide an optional alpha value to hsl.
The following value is what was breaking both before and after the release:
hsl(156deg 99% 29% / 100%)

@taitems The issue is the use of a percentage in the opacity, this should work if you use a float. Will cut a new release shortly.

Thank you kindly @bhough!

Version 4.2.1, should now hopefully fully address this.

Thank you for getting this fixed!

Can confirm this is working, thanks again.


Update salmon-soup.md

Quickly Reformatted salmon soup ;fixes #51

What a good addition to our cook book. However, you miss the following ingredients in your list: stock, carrot, salmon and potatoes.


[UPD]

updated presentation

👷 Deploy Preview for cerulean-blini-883dc7 processing.

Name
Link

🔨 Latest commit
821042fce4ec1665502d0e0cd44291295a2906d4

🔍 Latest deploy log
https://app.netlify.com/sites/cerulean-blini-883dc7/deploys/625b3266c565c700085949b3


Feature/adding canary for cross cluster -> readme patch

As per request, adds a little documentation as to the canary change

Pull Request Test Coverage Report for Build 01818d00-697b-4587-b9a3-d669ea7102c8

0 of 0   changed or added relevant lines in 0 files are covered.
129 unchanged lines in 20 files lost coverage.
Overall coverage decreased (-0.04%) to 57.95%

Files with Coverage Reduction
New Missed Lines
%

client/history/client.go
2
38.05%

client/history/metricClient.go
2
45.3%

common/membership/hashring.go
2
83.54%

common/persistence/executionManager.go
2
78.2%

common/persistence/statsComputer.go
2
96.43%

common/task/weightedRoundRobinTaskScheduler.go
2
89.12%

service/history/handler.go
2
47.6%

service/matching/matcher.go
2
91.46%

service/matching/taskListManager.go
2
73.77%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
57.59%

Totals

Change from base Build 01818658-b840-46a7-887a-45ad5556547e:
-0.04%

Covered Lines:
83896

Relevant Lines:
144772

💛  - Coveralls


Allow for extracting uri based on mouse position instead of cusor position

Using getmousepos(), the mouse click position can be obtained. If the relevant methods were tweaked to be able to use this position optionally, mouse-click-based-mappings could be created to do things like following links when you <M-LeftClick>.

Thank you for the suggestion. I'd be happy if you could open a pr for this feature


Terraform 0.9.10

Motivation for this change
Also removes some old versions. Let's only keep minor versions.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

BTW, regarding https://github.com/NixOS/nixpkgs/pull/26907#discussion_r125087488
As terraform made by the same team and also contains Makefile among the sources, is there any difference between versions of the package compiled using buildGoPackage and stdenv.mkDerivation ?

We should probably switch to a stdenv.mkDerivation as the build is supposed to be driven by the Makefile. At the moment we can get away with emulating what the Makefile does using the postInstall phase in the terraform project but it's probably going to diverge in the next minor where the plugins are loaded dynamically.

Switching from buildGoPackage to mkDerivation has the disadvantage, that will contain a runtime reference on the go compiler itself. This is usually cleared by buildGoPackage.

@zimbatm
@shlevy Pointed out (about a week ago) that Terraform point releases are (supposedly) major, breaking changes: https://github.com/NixOS/nixpkgs/commit/4d2597981dc227dfa2f9ccb167f710bc0371a2b2
@shlevy If we should be retaining a handful of old point releases, perhaps we should indicate that as comments in the relevant places?

@cstrahan Yeah, we should add them back with comments on the ones we want to keep

How many versions are we supposed to keep? Personally I try to keep track of the latest release, maybe the one before is there are some major incompatibilities.

IMO anything community members stay they're still using that doesn't come with a huge maintenance overhead. So maybe we can put something in a comment for which user(s) are still using the version in question and people interested in cleanup can check in with them if they're still using it?

It would be easier for users to pin old versions if they are not able to keep up for some reason. Users come and go and might not let us know when they stop using NixOS or terraform.

@cstrahan @shlevy what versions are you currently using?

I also noticed that terragrunt hasn't been updated for a while. The new version is compatible with the latest terraform.

We're using terraform 0.9.4 and terragrunt 0.12.15. If we're not OK keeping around old versions, can we at least expose a function that lets us just pass in a version + src hash in the normal case?


Update Crds

Coverage increased (+14.9%) to 92.588% when pulling fdb81e8e185451aed12004612cd86fd8ac46e0c0 on crdAndWebhookhandler into a6d2b83cd70cc577a581838c30f120800086d79d on development.


Trouble getting started.

Can someone please make a step by step on how to achieve the vibrancy effect on Windows 10? I am new to this and can't figure out where to get started.

Hey, the instructions in README works. Though, the Electron version is hard-coded so if you have a dfiferent Electron version you need to recompile the module.
After cloning the repo and npm install, run electron --version to see your version. Then to build the module type node-gyp rebuild --target=1.7.1 --runtime=electron --arch=x64 --debug --dist-url=https://atom.io/download/atom-shell where 1.7.1 should be the version you have (output of electron --version. After it is successfuly compiled just run electron spec/app and the test app will run.

This is where I am stuck. I don't know what's the problem, I have the latest node, npm, windows-build-tools, and Visual Studio 2017, I don't have any others programs opened..

As you can all see, I managed to get it working, apparently I had a problem with permissions in %appdata%\Roaming\npm folder.. those were messed up and I couldn't install global npm modules there.
Aside from that, I wanted to know what happens if I build the app. I tried with electron-builder, on both x32 and x64, unpacked and with the nsis (Setup).. and lastly portable, but they all seem to give me the same error about some kind of binding.. does anyone know why, or am I doing something wrong and I am required to have some files there?

You need to use some kind of mechanism to build the native modules and put into the right place when you're packing the app. For example electron-react-boilerplate has a similar system. The error you have says that it cant find the Vibranch.node file inside the packaged app.

That's what I was thinking too, but I don't know where should I put the Vibrancy.node file.

After using the npm module of electron-vibrancy and setting it as a dependency, electron-builder seems to take care of the problem and package the app correctly. I'll close now the issue, if I encounter any problems, I'll come back. Thank you.


Update rubocop.yml

Many of the cops have changed since this was last updated.
This updates them and also adds new cops aligning it with Devex BE guild agreements.

I can remove everything but I left it all as was aligning it with the idea that we would have a shared rubocop file that we spoke about a long while back.

I can remove everything but I left it all as was aligning it with the idea that we would have a shared rubocop file that we spoke about a long while back.

While this makes sense for our internal applications, I think open source gems are a different use case and the rubocop configuration should be tailored for it.


Transformation map 0 must have exactly one source file.

🐞 Bug report
Command (mark with an x)

[x] build

Is this a regression?
Yes. I worked before I updated Angular from 13.0.3 -> 13.2.0.
However, the issue didn't appear until AFTER i cleared my node_modules and reinstalled. So it worked fine a few day after updating.
Description
When building our applications we get this error: Transformation map 0 must have exactly one source file.
It only happens during dev-build, not production.
Worth noting here is that the path on the first line, packages\web-app-edit\node_modules\@angular, does not exists.
We have all dependencies in the monorepo root.
🔬 Minimal Reproduction
I have a skeleton of our monorepo in a GitHub repository where I have removed all applications but one. All libs are still there since the app is dependent on them.
https://github.com/JonWallsten/monorepo-new/tree/angular-bug-transformation
🔥 Exception or Error

 | ERROR in<EMAIL_ADDRESS> | Module build failed (from node_modules\babel-loader\lib\index.js):
 | Error<EMAIL_ADDRESS>Transformation map 0 must have exactly
one source file.
 | Did you specify these with the most recent transformation maps first?
 |     at buildSourceMapTree (C:\Users\***\repo\oas-web\node_modules\@ampproject\src\build-source-map-tree.ts:33:13)
 |     at remapping (C:\Users\***\repo\oas-web\node_modules\@ampproject\src\remapping.ts:36:17)
 |     at mergeSourceMap (C:\Users\***\repo\oas-web\node_modules\@babel\core\lib\transformation\file\merge-map.js:19:30)
 |     at generateCode (C:\Users\***\repo\oas-web\node_modules\@babel\core\lib\transformation\file\generate.js:72:39)
 |     at run (C:\Users\***\repo\oas-web\node_modules\@babel\core\lib\transformation\index.js:55:33)
 |     at run.next ()
 |     at Function.transform (C:\Users\***\repo\oas-web\node_modules\@babel\core\lib\transform.js:25:41)
 |     at transform.next ()
 |     at step (C:\Users\***\repo\oas-web\node_modules\gensync\index.js:261:32)
 |     at C:\Users\***\repo\oas-web\node_modules\gensync\index.js:273:13
 |<EMAIL_ADDRESS>1:0-45 2:0-57 3:0-88 3:0-88 3:0-88 3:0-88 407:54-73 407:88-104 424:12-31 426:12-28 3043:55-71 3060:12-28 3834:39-54 3834:57-72 3841:42-57 3842:55-70
 |  @ packages\web-app-edit\src\app\common\services\condition-editor.service.ts 4:18-49 30:11-42
 |  @ packages\web-app-edit\src\app\components\common\condition-editor\condition-editor.component.ts 6:35-95 24:11-71
 |  @ packages\web-app-edit\src\app\app.module.ts 17:37-111
 |  @ packages\web-app-edit\src\main.ts 6:21-48

🌍 Your Environment

Angular CLI: 13.2.2
Node: 16.13.2
Package Manager: npm 8.1.2
OS: win32 x64

Angular: 13.2.1
... animations, cdk, common, compiler, compiler-cli, core, forms
... material, platform-browser, platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1302.2
@angular-devkit/build-optimizer   0.1302.1
@angular-devkit/core              13.2.2
@angular-devkit/schematics        13.2.2
@angular/cli                      13.2.2
@ngtools/webpack                  13.2.2
@schematics/angular               13.2.2
ng-packagr                        13.2.1
rxjs                              7.5.2
typescript                        4.5.5
webpack                           5.67.0

Anything else relevant?
I'm using @ngtools/webpack to build and not the CLI

I noticed the same issue while upgrading Babel to 7.17 in angular/angular. This is likely due to https://github.com/babel/babel/pull/14209 as that error originates from @ampproject/remapping.

@JoostK Great. I just saw that @alan-agius4 was working on a fix. Do you guys have a workaround for now? No one in our team can build at the moment.
https://github.com/angular/angular-cli/pull/22646

Revert to Babel 7.16

I'll probably have to downgrade @angular in that case since we're not using Babel as a direct dependency. I'll figure it out.

Isn't the CLI still on Babel 7.16 given that #22644 hasn't been merged yet?

@JoostK

Doesn't caret mean minor and patch versions?
In that case it makes sense that 7.17.x since it's the latest minor.

Yep, we just realized that that would be the one pulling in a newer version of Babel. You can use Yarn resolutions or npm >= 8.3 overrides.

I have ~20 other packages in the same situation `^7.x.x'.
I just tried out overrides, but I didn't work because our npm version is currently 8.1.x.
Will have to update.

This seems to be related to the change in Babel https://github.com/babel/babel/pull/14209 mentioned by @JoostK earlier.
What is happing here is that when using the linker on @angular/material the sourcemap will contain multiple sources example.
  [
<EMAIL_ADDRESS>    '/packages/web-app-edit/Users/src/material/card/card.html',
    '/packages/web-app-edit/Users/src/material/card/card-header.html',
    '/packages/web-app-edit/Users/src/material/card/card-title-group.html'
  ]

remapping which is the package used by Babel to merge sourcemaps doesn't support multiple sources. Previously, Babel failed silently and didn't generate a sourcemap https://github.com/angular/angular/issues/42769 at all.
As a workaround you can disable sourcemaps from the linker via the sourceMapping option.
test: /\.mjs$/,
loader: "babel-loader",
options: {
    compact: false,
    plugins: [
        [linkerPlugin.default, { sourceMapping: false }],
    ],
},

My suggestion would be that for the time being the compiler-cli team flip the default of sourceMapping to false since this doesn't seem to work as intended due to https://github.com/angular/angular/issues/42769.


Can I save my presentation in some format other than pptx, i.e. ppsx

I want to save my presentation in ppsx format, not in pptx format. Is it possible using pptxgenjs?

Not at this time.


Remove snapshot publication script as well as stage from build pipeline

Description
We moved snapshots publication for all components into their respective repo. Hence removing the stage and associated script from pipeline as well.
Issues Resolved
closes https://github.com/opensearch-project/opensearch-build/issues/3185
closes https://github.com/opensearch-project/opensearch-build/issues/1182
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #3329 (2f9345c) into main (c4de95b) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #3329   +/-   ##
=======================================
  Coverage   91.74%   91.74%
=======================================
  Files         172      172
  Lines        4991     4991
=======================================
  Hits         4579     4579
  Misses        412      412

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


fix inconsistent size and adjust debug handler

fixed https://github.com/ngaut/unistore/issues/466

LGTM


[Bug]: peripherial.NotifyCharacteristic seems to only work once in lifecycle of Android app

Component/Nuget
BluetoothLE Client (Shiny.BluetoothLE)
What operating system(s) are effected?

[ ] iOS (13+ supported)
[ ] Mac Catalyst
[X] Android (8+ supported)
[ ] Windows (.NET 7 Target - only Core is currently supported, BLE is coming)

Version(s) of Operation Systems
android 13
Hosting Model

[X] MAUI
[ ] Native/Classic Xamarin
[ ] Uno
[ ] Manual

Steps To Reproduce

scan and connect to device
subscribe to peripheral.NotifyCharacteristic
navigate back - device disconnects when leaving the page and all disposables are disposed of
repeat process without closing app or powering down the device - on the second time around the peripheral.NotifyCharacteristic does not seem to start

I tried following the bleclient sample as best I could.  I am probably missing something to cause this issue on my end.
Expected Behavior
on iOS - this process seems to work fine.  I expect every time I connect to a device is can subscribe to the NotifyCharacteristic
Actual Behavior
does not subscribe after first time.
Exception or Log output
------------FIRST TIME THROUGH PROCESS--------------------
[BluetoothGatt] connect() - device: F9:40:54:9F:A8:66, auto: true
[BluetoothGatt] registerApp()
[BluetoothGatt] registerApp() - UUID=56beaf07-3b9c-466d-8ed9-5cc4a9795a99
[BluetoothGatt] onClientRegistered() - status=0 clientIf=7
[BluetoothGatt] requestConnectionPriority() - params: 0
Thread started:  #13
[BluetoothGatt] onClientConnectionState() - status=0 clientIf=7 device=F9:40:54:9F:A8:66
[0:] MyBleDelegate: BLE Device Status Connected
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
Thread started:  #14
[BluetoothGatt] setCharacteristicNotification() - uuid: 6e400003-b5a3-f393-e0a9-e50e24dcca9e enable: true
[0:] got rx
[0:] Mtu: 20
[BluetoothGatt] configureMTU() - device: F9:40:54:9F:A8:66 mtu: 60
[0:] got tx
[Choreographer] Skipped 69 frames!  The application may be doing too much work on its main thread.
[BluetoothGatt] onConfigureMTU() - Device=F9:40:54:9F:A8:66 mtu=60 status=0
[0:] result: 60
[0:] Writing: SM=1
[apps.spartanble] Explicit concurrent copying GC freed 35451(1627KB) AllocSpace objects, 4(208KB) LOS objects, 83% free, 4701KB/28MB, paused 22us,15us total 9.879ms
[BluetoothGatt] onPhyUpdate() - status=0 address=F9:40:54:9F:A8:66 txPhy=2 rxPhy=2
[0:] Writing: RS=1
[0:] ContentRegionViewModel RegionNavigationCallback
[0:] key: page--value: T5800 DataPage
[0:] WriteWithoutResponse
[0:] WriteWithoutResponse
Thread started:  #15
Thread started:  #16
[0:] Writing: RS=1
[0:] WriteWithoutResponse
[0:] Writing: RS=1
[0:] WriteWithoutResponse
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=36 latency=0 timeout=42 status=0
[OnBackInvokedCallback] OnBackInvokedCallback is not enabled for the application.
[OnBackInvokedCallback] Set 'android:enableOnBackInvokedCallback="true"' in the application manifest.
[monodroid-assembly] open_from_bundles: failed to load assembly System.Diagnostics.TraceSource.dll
Loaded assembly: /data/data/com.canconapps.spartanble/files/.override/System.Diagnostics.TraceSource.dll [External]
[0:] ContentRegionViewModel OnNavigatedFrom
[BluetoothGatt] setCharacteristicNotification() - uuid: 6e400003-b5a3-f393-e0a9-e50e24dcca9e enable: false
[BluetoothGatt] close()
[BluetoothGatt] unregisterApp() - mClientIf=7
[0:] ScanViewModel OnNavigatedTo
[OnBackInvokedCallback] OnBackInvokedCallback is not enabled for the application.
[OnBackInvokedCallback] Set 'android:enableOnBackInvokedCallback="true"' in the application manifest.
[0:] ScanViewModel OnNavigatedFrom
[OnBackInvokedCallback] OnBackInvokedCallback is not enabled for the application.
[OnBackInvokedCallback] Set 'android:enableOnBackInvokedCallback="true"' in the application manifest.
[apps.spartanble] Explicit concurrent copying GC freed 21440(959KB) AllocSpace objects, 3(212KB) LOS objects, 83% free, 4725KB/28MB, paused 23us,19us total 11.688ms
------------------SECOND TIME THROUGH PROCESS---------------------
[0:] ScanViewModel OnNavigatedTo
[0:] key: device
[0:] timer started
[BluetoothAdapter] isLeEnabled(): ON
[BluetoothLeScanner] onScannerRegistered() - status=0 scannerId=5 mScannerId=0
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: SW-091C
[0:] BLE Device Found: T5800_1234
[0:] BLE Device Found: SW-091C
[BluetoothAdapter] isLeEnabled(): ON
[0:] timer stopped
[0:] Callback Cleared
[OnBackInvokedCallback] OnBackInvokedCallback is not enabled for the application.
[OnBackInvokedCallback] Set 'android:enableOnBackInvokedCallback="true"' in the application manifest.
[0:] ScanViewModel OnNavigatedFrom
[0:] key: Peripheral--value: crc642f3fa550470a3813.Peripheral@2ce7f45
[0:] key: deviceType--value: T5800
[0:] ContentRegionViewModel OnNavigatedTo
[0:] key: Peripheral--value: crc642f3fa550470a3813.Peripheral@2ce7f45
[0:] key: deviceType--value: T5800
[BluetoothGatt] connect() - device: F9:40:54:9F:A8:66, auto: true
[BluetoothGatt] registerApp()
[BluetoothGatt] registerApp() - UUID=fd3e0a12-08d7-4791-b350-ece5bee7c7f7
[BluetoothGatt] onClientRegistered() - status=0 clientIf=7
[BluetoothGatt] requestConnectionPriority() - params: 0
[BluetoothGatt] onClientConnectionState() - status=0 clientIf=7 device=F9:40:54:9F:A8:66
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[0:] MyBleDelegate: BLE Device Status Connected
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
[0:] got rx
[0:] Mtu: 60
[BluetoothGatt] configureMTU() - device: F9:40:54:9F:A8:66 mtu: 60
[0:] got tx
[Choreographer] Skipped 62 frames!  The application may be doing too much work on its main thread.
[BluetoothGatt] onConfigureMTU() - Device=F9:40:54:9F:A8:66 mtu=60 status=0
[0:] result: 60
[0:] Writing: SM=1
[0:] Writing: RS=1
[0:] ContentRegionViewModel RegionNavigationCallback
[0:] key: page--value: T5800 DataPage
Thread started:  #17
[BluetoothGatt] onPhyUpdate() - status=0 address=F9:40:54:9F:A8:66 txPhy=2 rxPhy=2
[0:] The operation has timed out.
[0:] The operation has timed out.
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=39 latency=0 timeout=42 status=0
Thread finished:  #6
Code Sample
https://github.com/mjphillips1981/spartanble.git
Code of Conduct

[X] I have supplied a reproducible sample is NOT FROM THE SHINY SAMPLES!
[X] I am a Sponsor OR I am using the LATEST stable/beta version from nuget (v2.7.1 stable - ALPHAS are not taking issues)
[X] I understand that if I am checking these boxes and I am not actually following what they are saying, I will be removed from this repository!

The tests are all passing and I've verified this in one of my projects.  You need to provide MUCH smaller reproducible samples if you want any sort of help especially with a beta

I will see what I can do. Thanks Allan!

here is a more basic sample...
https://github.com/mjphillips1981/myshinysample
let me know if this will work for you.  I am going to paste the output log in the next comment.

///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
//first connection
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
[BluetoothGatt] connect() - device: F9:40:54:9F:A8:66, auto: true
[BluetoothGatt] registerApp()
[BluetoothGatt] registerApp() - UUID=84f1774e-be96-4a6f-b83e-0c7c577cbc51
[BluetoothGatt] onClientRegistered() - status=0 clientIf=7
[BluetoothGatt] requestConnectionPriority() - params: 0
[BluetoothGatt] onClientConnectionState() - status=0 clientIf=7 device=F9:40:54:9F:A8:66
Thread started:  #16
[0:] Shiny.BluetoothLE.IPeripheral: Debug: GATT Connection State Change: Success - Connected
[0:] MyBleDelegate: BLE Device Status Connected
[companyname.app] Accessing hidden method Landroid/bluetooth/BluetoothGatt;->refresh()Z (unsupported, reflection, allowed)
[BluetoothGatt] refresh() - device: F9:40:54:9F:A8:66
[0:] Shiny.BluetoothLE.IPeripheral: Warning: Clear Internal Cache Refresh Result: True
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=6 latency=0 timeout=500 status=0
[BluetoothGatt] onPhyUpdate() - status=0 address=F9:40:54:9F:A8:66 txPhy=2 rxPhy=2
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
Thread started:  #17
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=39 latency=0 timeout=500 status=0
[BluetoothGatt] refresh() - device: F9:40:54:9F:A8:66
[0:] Shiny.BluetoothLE.IPeripheral: Warning: Clear Internal Cache Refresh Result: True
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=6 latency=0 timeout=500 status=0
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=39 latency=0 timeout=500 status=0
[0:] Shiny.BluetoothLE.IPeripheral: Debug: Char InstanceID: 18
[0:] Shiny.BluetoothLE.IPeripheral: Information: Subscribing to Notification Characteristic 6e400001-b5a3-f393-e0a9-e50e24dcca9e / 6e400003-b5a3-f393-e0a9-e50e24dcca9e
[BluetoothGatt] setCharacteristicNotification() - uuid: 6e400003-b5a3-f393-e0a9-e50e24dcca9e enable: true
[0:] got rx
[0:] Shiny.BluetoothLE.IPeripheral: Information: Subscribed to Notification Characteristic 6e400001-b5a3-f393-e0a9-e50e24dcca9e / 6e400003-b5a3-f393-e0a9-e50e24dcca9e
[BluetoothGatt] configureMTU() - device: F9:40:54:9F:A8:66 mtu: 60
[0:] got tx
[BluetoothGatt] onConfigureMTU() - Device=F9:40:54:9F:A8:66 mtu=60 status=0
[0:] result: 60
[0:] Writing: SM=1
[0:] WriteWithoutResponse
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
//on disconnect
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
[0:] DataViewModel OnNavigatedFrom
[BluetoothGatt] setCharacteristicNotification() - uuid: 6e400003-b5a3-f393-e0a9-e50e24dcca9e enable: false
[0:] Shiny.BluetoothLE.IPeripheral: Debug: Cleaned up characteristic subscription: 6e400001-b5a3-f393-e0a9-e50e24dcca9e / 6e400003-b5a3-f393-e0a9-e50e24dcca9e
[BluetoothGatt] close()
[BluetoothGatt] unregisterApp() - mClientIf=7
Thread started:  #19
Thread started:  #20
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
//after trying to reconnect
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////
[BluetoothGatt] connect() - device: F9:40:54:9F:A8:66, auto: true
[BluetoothGatt] registerApp()
[BluetoothGatt] registerApp() - UUID=fb6c4efb-2b2d-483f-8158-c96811fcb528
[BluetoothGatt] onClientRegistered() - status=0 clientIf=7
[BluetoothGatt] requestConnectionPriority() - params: 0
[BluetoothGatt] onClientConnectionState() - status=0 clientIf=7 device=F9:40:54:9F:A8:66
[0:] Shiny.BluetoothLE.IPeripheral: Debug: GATT Connection State Change: Success - Connected
[0:] MyBleDelegate: BLE Device Status Connected
[0:] Shiny.BluetoothLE.IPeripheral: Warning: Clear Internal Cache Refresh Result: True
[BluetoothGatt] refresh() - device: F9:40:54:9F:A8:66
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=6 latency=0 timeout=500 status=0
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
Thread started:  #28
[BluetoothGatt] refresh() - device: F9:40:54:9F:A8:66
[0:] Shiny.BluetoothLE.IPeripheral: Warning: Clear Internal Cache Refresh Result: True
[BluetoothGatt] discoverServices() - device: F9:40:54:9F:A8:66
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=39 latency=0 timeout=500 status=0
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=6 latency=0 timeout=500 status=0
[BluetoothGatt] onSearchComplete() = Device=F9:40:54:9F:A8:66 Status=0
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=39 latency=0 timeout=500 status=0
[0:] got rx
[0:] result: 60
[0:] Writing: SM=1
[0:] got tx
[BluetoothGatt] onConnectionUpdated() - Device=F9:40:54:9F:A8:66 interval=39 latency=0 timeout=42 status=0
[0:] The operation has timed out.
[0:] Writing: RS=1
[0:] The operation has timed out.

this only happens if I never leave the app.
If I close the app and remove it from my recent, and then try the process it works fine.

Did the smaller sample work for ya?  I'm not sure what else to do here.  Let me know if there is anything else I can provide.

There is still a lot of unnecessary noise in your sample, but the basic principle is disconnect/reconnect/call notifitycharacteristic.   I have a device unit test passing on this for android.  I don't run your sample beyond looking the code.
Check your peripheral to ensure it is unhooking & rehooking.  You're getting timeouts from something which isn't related to the notification peripheral.


Build issue with clang++ and g++

I'm using Lua 5.3.2
clang++ --version outputs:
clang version 3.7.1 (tags/RELEASE_371/final)
Target: x86_64-unknown-linux-gnu
Thread model: posix

g++ --version outputs:
g++ (GCC) 5.3.0

I'm getting the following errors when building with g++: http://termbin.com/c4e7
With clang++: http://termbin.com/dqoz
I don't know where to begin deciphering those error messages so I was hoping I could have some help in fixing this issue. If I should provide more information let me know and I'll get that to you as soon as I can.

You need to build with -std=c++14 in the command line.


Why there are a lot of blank areas even if I set the tissue_percent to 80.00?

I'm trying to extract some WSI tiles, but there are a lot of blank areas even if I set the tissue_percent to 80.00. I'm using histolab 0.3.0 on Windows11.

The code is
import os
import histolab.tiler as tiler
from histolab.slide import Slide
def extract_patient_id(filename):
parts = filename.split('-')
patient_id = parts[0]
return patient_id
def get_tile(filename):
tiles_extractor = tiler.GridTiler(
tile_size=(512, 512),
level=0,
check_tissue=True,
tissue_percent=80.0,
prefix=filename + '/',
suffix=".png"
)
processed_path = "E:/Tiles"
image = Slide(files_directory + '/' + filename, processed_path)
tiles_extractor.extract(image)
return
files_directory = "E:/Slides"
data_list = []
for filename in os.listdir(files_directory):
if filename.endswith(".ndpi"):
get_tile(filename)

I think histolab 0.3.0 is quite old and can be buggy. First things first I'd update the version and re-try it. Let us know please

I think histolab 0.3.0 is quite old and can be buggy. First things first I'd update the version and re-try it. Let us know please

Sorry, I'm using on Windows11 and higher versions don't support it. I will try it on Ubuntu. Thank you.

Yes better try on a different system than windows :D

I tried it on my Ubuntu on VMware, but there are still many blanks. The python version is 3.8 and the histolab is 0.7.0
And the code is
import os
import histolab.tiler as tiler
from histolab.slide import Slide
def extract_patient_id(filename):
parts = filename.split('-')
patient_id = parts[0]
return patient_id
def get_tile(filename):
tiles_extractor = tiler.GridTiler(
tile_size=(512, 512),
level=0,
check_tissue=True,
tissue_percent=80.0,
suffix=".png"
)
processed_path = "/media/liuhongxing/固态移动硬盘/test1"
image = Slide(filename, processed_path)
tiles_extractor.extract(image)
return
file_name = "/media/liuhongxing/固态移动硬盘/F155519-1.ndpi"
get_tile(file_name)

Hi @a1ecbennington :)
The GridTiler.extract method has an extraction_mask parameter, that defines the binary mask from where to extract the tiles --> https://histolab.readthedocs.io/en/latest/api/tiler.html#histolab.tiler.GridTiler.extract
The default is the BiggestTissueBoxMask object, which defines the box containing the largest contiguous tissue area --> https://histolab.readthedocs.io/en/latest/api/masks.html
So what you describe is actually the expected behaviour. Try using an instance of the TissueMask for the extraction_mask parameter instead, that is probably what you want :)

Hi @a1ecbennington :)
The GridTiler.extract method has an extraction_mask parameter, that defines the binary mask from where to extract the tiles --> https://histolab.readthedocs.io/en/latest/api/tiler.html#histolab.tiler.GridTiler.extract The default is the BiggestTissueBoxMask object, which defines the box containing the largest contiguous tissue area --> https://histolab.readthedocs.io/en/latest/api/masks.html
So what you describe is actually the expected behaviour. Try using an instance of the TissueMask for the extraction_mask parameter instead, that is probably what you want :)
Yes, it is working. Thank you very much!


fetch upstream to continue working

First attempt at merging from the master into a branch of mine.  I intend to keep working on this branch and then commit back.

figured out i can do this locally.


SecurityError: Attempt to use history.replaceState() more than 100 times per 30 seconds

We've been seeing this in production, via Sentry — almost exclusively on iPhones. I assume it's related to this: https://github.com/sveltejs/kit/blob/ad9551c47727b74fcc7c31614962e7bc37792633/packages/kit/src/runtime/internal/router/index.js#L49-L63

What if you changed it to only run every 500ms? Then it would only run a max of 60 times in 30 seconds. I feel like that is still pretty frequent for restoring the scroll position later.

It should not be running when continuously moving the mouse or when not moving the mouse, but only when the mouse was moving and then stops. Changing it to 500s sounds like a copout instead of really understanding the issue as to why debouncing isn't working here.

I see what you mean!

Not entirely sure if it should go here or in a separate issue. Happy to move it to a separate issue if needed.
I'm observing the same issue mentioned here but I also have some (iOS only) errors about pushState in my Sentry log.
SecurityError: Attempt to use history.pushState() more than 100 times per 30 seconds

Might be related somehow.

seeing these issues in production on iOS 14.7.1 devices (iPhone 12 mini - "iPhone13,1")
SecurityError: Attempt to use history.replaceState() more than 100 times per 30 seconds

SecurityError: Attempt to use history.pushState() more than 100 times per 30 seconds

Can confirm I'm picking this up via Sentry too:
SecurityError: Attempt to use history.replaceState() more than 100 times per 30 seconds

Mobile Safari
Version:14.1.2
iOS
Version:14.7.1
iPhone

May be we can refer to this solution.
https://github.com/ampproject/amphtml/pull/23938/files

Check out my PR on this issue: https://github.com/sveltejs/kit/pull/2738

I tested this on my iphone today and I discovered that ios emits the scroll event later than other browsers. Intervals between scroll events rapidly fluctuate between 30 - 150ms. And an easy way to aggressively trigger this bug is to quickly scroll up and down, that would consistently give 200-300ms.
With these said, our debounce function runs at a 50ms interval, which explains how the security error was caused, because we essentially had no debouncing at all. I'll submit a PR to increase it to 200ms as that gives the least false positives.

What is the fix for this issue. I am also getting the same error. can anyone please help?
Attempt to use history.pushState() more than 100 times per 30 seconds

This is safari specific

This is still happening on Safari and potentially iOS but it is specific to Safari.


update fast-classpath-scanner version to 1.9.20

Coverage remained the same at 92.629% when pulling 4ef01d5efa9ebf10315f52805e524dfc9b0e0524 on PascalSchumacher:update_classpath_scanner into 94a2c5b5ff22cf02d1bed1b62e749fca9554f548 on benas:master.

I don't know why version eye does not see this update: https://www.versioneye.com/user/projects/56c6d7fa19f173000c237adc
May be in next parsing job (I think it runs once per day)

Yes it was release only two hours ago: https://github.com/lukehutch/fast-classpath-scanner/commit/451e86c614db7f742858fe216026c825bff232a3

Tell me, you are quite fast you heinh 👍


header background colour is not changing  in android

"react": "16.2.0",
"react-native": "0.53.0",
"react-navigation": "^1.0.3"
Expected behaviour
  <Header backgroundColor="#2196F3"  androidStatusBarColor="#000000">
header colour change to this  #2196F3 colour to its by default colour in android
Actual behaviour
when i press ctrl + s by hot loading header colour changes to this  #2196F3 colour. but when i rebuilt my app by giving command react-native run-android header colour comes to its by default  colour
Steps to reproduce (code snippet or screenshot)

Screenshot of emulator/device

Is the bug present in both ios and android or in any one of them?
i don't know about ios but in android
Any other additional info which would help us debug the issue quicker.
i don't know

@rizwan92 as a workaround, you can specify the color in style prop of header.
 <Header
          style={{ backgroundColor: 'red' }}
          androidStatusBarColor="red">
</Header>

@rizwan92 backgroundColor prop of <Header/> will be removed. Use style instead.
This will be documented.

https://github.com/GeekyAnts/native-base-docs/issues/81

@raajnadar checked this. toolbarDefaultBg is working fine for me. Attaching a gif.
Gif

crazy how this doesn't work...could it be because I have headerMode off ?


Dockerizing a Django application

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
###Dockerizing a Django application.
Introduction paragraph (2-3 paragraphs):
Docker is an open platform for building, shipping, and running distributed applications. It gives programmers, development teams, and operations engineers the common toolbox they need to take advantage of the distributed and networked nature of modern applications.
Docker allows you to run your apps in a controlled environment called a container, which is built according to your instructions.
While building web applications, you may  want to run your application in a manner that is closer to your production environment.
This application focuses on building a django application and dockerizing this application
Key takeaways:

Understanding Docker and how it is used,
Building a simple Python Django application
Creating a simple Dockerfile to build a container running a Django web application server.
How to set up CI/CD pipelines

References:
https://docs.docker.com/samples/django/
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

You have suggested this topic two times.


Use pinned cspell in Export-API.ps1 and aggregate-reports

Supersedes https://github.com/Azure/azure-sdk-for-net/pull/26218 but without all the codeowners noise that comes from a messy rebase.

aggregate-reports is failing on link check, unrelated to this PR

/check-enforcer override


A few small cleanup items

Fixing a few small issues that I have run across:

Update deprecated ImageRegistry() with ImageRegistryV1alpha1()
Use snake case instead of CAMEL_CASE
fix using %q when we shouldn't be
variable name masks imported package name (and package was imported twice"

Each in it's own commit for easy review.

@bparees ptal, should be a quick review

/lgtm


Enable zooming

See e.g. https://stackoverflow.com/questions/52576376/how-to-zoom-in-on-a-complex-svg-structure for a potential implementation

Closed with #32


L2TP interface causes Traffic Shaping/ipfw rules to fail to load

Version OPNsense 17.7.7_1-amd64
It would appear that when there is an L2TP interface (I have not tested other Point to Point interfaces) the ipfw/traffic shaping ruleset fails to load in its entirity
root@fw00:~ # ipfw /usr/local/etc/ipfw.rules
Are you sure? [yn] y

Flushed all rules.
00100 allow pfsync from any to any
00110 allow carp from any to any
00120 allow ip from any to any layer2 mac-type 0x0806,0x8035
00130 allow ip from any to any layer2 mac-type 0x888e,0x88c7
00140 allow ip from any to any layer2 mac-type 0x8863,0x8864
00150 deny ip from any to any layer2 not mac-type 0x0800,0x86dd
00200 skipto 60000 ip6 from ::1 to any
00201 skipto 60000 ip4 from <IP_ADDRESS>/8 to any
00202 skipto 60000 ip6 from any to ::1
00203 skipto 60000 ip4 from any to <IP_ADDRESS>/8
01002 skipto 60000 udp from any to <IP_ADDRESS> dst-port 53 keep-state
01002 skipto 60000 ip from any to { <IP_ADDRESS> or <IP_ADDRESS> } in
01002 skipto 60000 ip from { <IP_ADDRESS> or <IP_ADDRESS> } to any out
01002 skipto 60000 icmp from { <IP_ADDRESS> or <IP_ADDRESS> } to any out icmptypes 0
01002 skipto 60000 icmp from any to { <IP_ADDRESS> or <IP_ADDRESS> } in icmptypes 8
01003 skipto 60000 udp from any to <IP_ADDRESS> dst-port 53 keep-state
01003 skipto 60000 ip from any to { <IP_ADDRESS> or <IP_ADDRESS> } in
01003 skipto 60000 ip from { <IP_ADDRESS> or <IP_ADDRESS> } to any out
01003 skipto 60000 icmp from { <IP_ADDRESS> or <IP_ADDRESS> } to any out icmptypes 0
01003 skipto 60000 icmp from any to { <IP_ADDRESS> or <IP_ADDRESS> } in icmptypes 8
Line 53: hostname ``l2tp'' unknown

There are no rules trying to shape L2TP traffic, just part of the standard ruleset - removing the L2TP specified rules allows the ipfw ruleset to load and consequently traffic shaping can operate:
add 1005 skipto 60000 udp from any to l2tp dst-port 53 keep-state
add 1005 skipto 60000 ip from any to { <IP_ADDRESS> or l2tp } in
add 1005 skipto 60000 ip from { <IP_ADDRESS> or l2tp } to any out
add 1005 skipto 60000 icmp from { <IP_ADDRESS> or l2tp } to any out icmptypes 0
add 1005 skipto 60000 icmp from any to { <IP_ADDRESS> or l2tp } in icmptypes 8

Presumably 'l2tp' should be a macro for the l2tp1 interface IP address (however this is done in ipfw, am more pf familiar), or perhaps a via statement for the interface?

@EdSt it should have ignored the l2tp part for this section. can you try https://github.com/opnsense/core/commit/6278fc000f27a5ad97c15bb917df1829833e78b4 ?
opnsense-patch 6278fc0

This probably also needs “pptp”

On 31. Oct 2017, at 10:03, Ad Schellevis<EMAIL_ADDRESS>wrote:
@EdSt it should have ignored the l2tp part for this section. can you try 6278fc0 ?
opnsense-patch 6278fc0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

That does the job, much obliged.  Yes, looks like it needs PPTP as well for the same issue.

@iMiMx thanks for confirming


101819816 custom tests

Total rework of Ra to allow writing custom tests. Based around testing with pytest and WebTest.
Changes:

A pytest plugin collects test definitions, automatically binding requests to the resource/http method as well as the app.
Automatic testing can be triggered with api.autotest()
Tests use example properties from the RAML by default, but custom object factories can be used to generate request bodies
Hooks can be declared before/after the suite, tests in a resource, or individual tests, and filtered by resource path or http method.
Responses are by default automatically validated against the RAML, as well as tested for a 2xx/3xx response code by WebTest (can be overridden)
Documented in doc/

Testing it out (from the README):
    $ pip install -r requirements.txt
    $ cd tests/apps/ramses_test
    $ py.test

See the docs in docs/, and the api.raml and tests/test_api.py in
tests/apps/ramses_test

Sorry, meant develop.


SDK status

Hey, we've been using Go with MercadoPago since early 2023 when this SDK did not exist yet. If it is safe to use yet then we would ofc. prefer to use the SDK provided by you, is it already feature complete or are there any breaking changes to be expected soon?

Awesome thanks!
Is there any chance to get the exchange rate used to convert the requested currency into UYU? I.e. I request a payment for 40 USD and your system always charges UYU - how will I know which exchange rate was used to do the conversion? Stripe for instance offers an "ExchangeRate" field but I couldn't find any. Any ideas? Otherwise I'm contacting support - sorry for asking it here.


Please make a 32-bit Windows build

Currently I have a system at work that is 32-bit only, but it seems the binaries are 64-bit ;( Please consider creating a 32-bit build of this tool.

Good suggestion, let me see what I can do!

All set. Let me know if you have any issues with the 32-bit version.


"Export to" button becomes sensitive after test or upload

Pluto ver 1.0.11
If there are errors in the validation rule (ie: a required field is not populated) the "Export to" button is correctly marked insensitive.  However if you subsequently click the "Test" or "Upload" button the "Export to" button will become sensitive.  I would expect this to stay insensitive to avoid any invalid configurations to be exported.

Are there specific steps you take to produce this? When I load up a validation and invalidate a required property the "Test from Local" and "Upload" buttons are disabled so I can't click them.

We have a property set to required which renders the validation unable to "export" on initial creation.  However you can still test or upload in that particular case.

Just as a reminder how to set this up: When I create a new validation without rules the "Test", "Test from local file", "Upload", and "Delete" buttons are enabled. "Save" and "Export to Production" are not. (I have one property that is required and not set.) If I press either of the first three buttons the validation is run but the "Export..." button is no enabled.
I'm looking into what controls the enabling of the button to see if I can see what could be causing it to be improperly enabled.

This happens for Andrew when the required property is in the Notification "rule".

I've set up a config that has a required rule in the notification area and it now reproduces the problem.

Okay. I've figured out what's causing this. The code is properly setting the "invalid" state for the property. This can be seen by the red error text. When "Test" is pressed, upon completion of the test the page is rebuilt. For some reason a second copy of the Notification rule is displayed. This second copy is immediately remove but in doing so it's error state is cleaned up, which cleans up state for both instances, and as a result the error state is lost and the buttons become enabled.
So, the question to solve now is, why is a second instance of the notification introduced and this doesn't happen for the other types of rules?

It would be worthwhile investigating why there's a second instance of this control being created. I've tried and haven't had any luck though.


[Compute] az sig image-version create: Add new parameters --virtual-machine and --image-version to support creating image version from different source

Description

Add new parameters --virtual-machine and --image-version to support creating image version from different source
Close #22424
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

@yanzhudd good beginning:)

@yanzhudd good beginning:)

Thanks:)

Nice work! 🎉


Support building when package is not in the root directory

From @zeriyoshi - original proposal in #82

Add extension-src directive to the php-ext section in composer.json.
This enables support for patterns where extension source code is not located in the project root, which is commonly seen in third-party extensions.
https://github.com/protocolbuffers/protobuf/tree/main/php/ext/google/protobuf
{
    "name": "google/protobuf",
    "type": "php-ext",
    "description": "protocolbuffers php extension",
    "php-ext": {
        "extension-name": "protobuf",
        "extension-src": "php/ext/google/protobuf"
    }
}

Investigate options, check this makes sense and is clear. Documentation would be needed, and upstream Composer schema support.

I will revisit this after #83 is merged

@asgrim This option can specify the name of the extension DLL.
Other automated scripts already use the zip attached on the GitHub release to install an extension automatically. It's hard to change the dll name into the zip archive without breaking something.

@asgrim This option can specify the name of the extension DLL?
Other automated scripts already use the zip attached on the GitHub release to install an extension automatically. It's hard to change the dll name into the zip archive without breaking something.

This is an entirely separate issue

@zeriyoshi I gave this some thought, I think build-path may be a better name for this.
https://github.com/ThePHPF/pie-design/pull/28
If there is no further feedback, please leave it with me, I have a PR updated and also will send a patch upstream to Composer.

Sorry, I missed this one. I don't think it's a problem.


Commands go to the wrong tab

I showed the issue in this 22 sec video:
http://www.youtube.com/watch?v=4LkhUYik2BA
(I used screenkey for the overlay, it doesn't show shift keypresses)
it just happens after I change one of the keybindings... if I go back to the .keymap file and do some other kind of change, then when I later switch to another tab, the keyboard commands will actually go where intended

Looking through old issues and it's not clear what the issue is. [:editor "pmeta-shift-left" :paredit.grow.left] works for me as a keybinding on osx. If you're still interested, please try on latest LT and provide reproducible steps. Otherwise I'll close in a few days

I cannot test the latest version, since LT fails to upgrade. I see this in the console:
Error: incorrect header check
    at Zlib._binding.onerror (zlib.js:295:17)

Is this a known (transient?) error?
I'll try again later on.
Can you please try to reproduce it on Linux?

Is this a known (transient?) error?

Unless you are behind a proxy, there shouldn't be an issue with downloading a new version of LT

Can you please try to reproduce it on Linux?

I'll need reproducible steps e.g:

Go to file X
Type abc
See Y

It's not clear in the video what to look for

Unless you are behind a proxy, there shouldn't be an issue with downloading a new version of LT

Nope, I was on the train so I thought it might be my tethered internet, but I tried again now when at home, and LT is not sending any packets whatsoever.
The LT upgrade is apparently borked. I presume that LT fetched a corrupted zip in the past (how could it be? surely LT should use a checksum before trusting what it downloaded) and now it won't redownload it again, even if it sees it's corrupted?
I could report it as a new bug

It's not clear in the video what to look for

The video is pretty clear, maybe to properly see the problem you'd need to play it back at its native (1080p) resolution?
Anyhow, you can compare it between seconds 0.04 and 0.18: there's a { before "backspace" that gets shifted all the way before :editor, that happens due to the :paredit-grow-left command, which is incorrectly executed when another tab is in the foreground... that's due to the editor pool not properly being updated

@berdario What version of LT are you running now? You might need to download a release package because older versions look for new versions at an endpoint that the open source team doesn't control or have access to. Going forward, LT looks for new releases on GitHub.

The about page says
Light Table version
0.8.0
Binary version
0.8.4
But when opening up LT, it says

Would you like us to download and install version 0.8.0?

If this is known issue due to the change in the releases, I'll just redownload LT separately and ignore this

@berdario That's weird. Your binary version really says "0.8.4"? It should be "0.34.5". I'm actually pretty curious how that happened.
I think you're going to need to just re-download LT.

Yup, weird indeed...
Anyhow, now that I got a new release package I tried to reproduce again this bug, but it's been quite a while and it doesn't happen right away, I'd need more time using LT to find it out... and since I'm not using LT day-to-day anymore, I think I'll just close this issue for now and reopen it if needed


Create Kadanes.md

Greetings !!
This closes #571
Sir I have succesfully completed my Documentation of Kadane's Algorithm !
Thank You :)

I guess this is done now sir...
Thank You again :)


<EMAIL_ADDRESS>build errors on macOS after<EMAIL_ADDRESS>build errors are fixed in 8.4.2b02

Describe the bug
  >> 1769    Undefined symbols for architecture x86_64:
     1770      "__ZNKSt13runtime_error4whatEv", referenced from:
     1771          __ZNK8nlohmann16json_abi_v3_11_26detail9exception4whatEv in libesmf.a(ESMC_InfoCDef.o)
     1772          __ZNK8nlohmann16json_abi_v3_11_26detail9exception4whatEv in libesmf.a(ESMC_InfoCDefGeneric.o)
     1773          __ZNK8nlohmann16json_abi_v3_11_26detail9exception4whatEv in libesmf.a(ESMCI_Array.o)
     1774          __ZNK8nlohmann16json_abi_v3_11_26detail9exception4whatEv in libesmf.a(ESMCI_Base.o)
     1775          __ZNK8nlohmann16json_abi_v3_11_26detail9exception4whatEv in libesmf.a(ESMCI_Field_F.o)

     ...

     3079          _c_esmc_arrayread_ in libesmf.a(ESMCI_Array_F.o)
     3080          _c_esmc_arraysmmstoreind4_ in libesmf.a(ESMCI_Array_F.o)
     3081          _c_esmc_arraysmmstoreind8_ in libesmf.a(ESMCI_Array_F.o)
     3082          _c_esmc_arraysmmstorenf_ in libesmf.a(ESMCI_Array_F.o)
     3083          ...
     3084    ld: symbol(s) not found for architecture x86_64
  >> 3085    collect2: error: ld returned 1 exit status
  >> 3086    make[2]: *** [bin/cub2latlon.x] Error 1
  >> 3087    make[1]: *** [base/CMakeFiles/cub2latlon.x.dir/all] Error 2
     3088    make[1]: *** Waiting for unfinished jobs....
     3089    /usr/local/Cellar/cmake/3.23.1/bin/cmake -E cmake_copy_f90_mod include/MAPL.generic/mapl_statespecification.mod generic/CMakeFiles/MAPL.generic.dir/mapl_statespecification.mod.stamp GNU
     3090    /usr/local/Cellar/cmake/3.23.1/bin/cmake -E touch generic/CMakeFiles/MAPL.generic.dir/StateSpecification.F90.o.provides.build
     3091    [ 91%] Building Fortran object generic/CMakeFiles/MAPL.generic.dir/ComponentSpecification.F90.o

The missing symbols error in esmf 8.4.1, fixed in 8.4.2b02, seems to be handed down to mapl.
To Reproduce
Build<EMAIL_ADDRESS>with<EMAIL_ADDRESS>Expected behavior
It builds
System:
macOS with clang + gfortran
Additional context
n/a

@climbfuji - just to clarify something for me, the previous version of ESMF that used to work in this context under Spack on macOS was v8.3.0b09? Then a few things in Spack changed, and we also pushed out the v8.4.1. With the new Spack, you tried v8.4.1, and first it failed the ESMF installation on macOS... which is what I addressed in v8.4.2b02. Now we are where MAPL shows the link issues on macOS. Is that a correct description, or am I missing something relevant wrt the Spack/ESMF/macOS progression?

That is correct.

Do you by any chance have the same kind of log file, that you posted for the failing case, for the ESMF build on macOS under the old configuration with v8.3.0b09, that still worked fine?

log.install.unified-env.001.esmf830b09-mapl2220.gz
@theurich See here please

@climbfuji - please give v8.4.2b03 a shot. Thanks!

You are producing beta snapshots faster than I can test them ... will do as soon as I have time, probably over the weekend.

I started this before I left for Vienna and of course forgot to check if it worked ... redoing it on my other laptop now.

@climbfuji - I just noticed that I had not actually pushed the revision that v8.4.2b03 was referencing to the GitHub remote. When you get a chance, let me know what ESMF hash your test was ending up... It might still be a valid test, because what was missing was just some additional output in the ESMF info, but I want to make sure. Sorry about that! And thanks for doing this while on travel!

This is the checksum I got today:
+    version(
+        "8.4.2b03",
+        sha256="e9b3e25b5a90fb77649e2f57da233318ccccf6ed47592e859c3fc909202f6298",
+    )

and with that I could build on my other mac cleanly, both esmf and also<EMAIL_ADDRESS>- thanks very much!

This is great news! When you get a chance, please post the ESMF build portion of the log.  Thanks!


(build) Upgrade Dependencies

Upgrades the package.json file to bring in updated dependencies and remove
security vulnerabilities.

easymde 2.13.0 > ^2.15.0
gulp-sass ^4.1.0 > ^5.0.0
node-sass ^5.0.0 > ^6.0.1

@gep13 I will email you about this one shortly.


system details tweaks #428

Description
see #428
Testing instructions
Add a set up instructions describing how the reviewer should test the code

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage
[ ] {more steps here}

Agile board tracking
closes #428

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.64%. Comparing base (200caf8) to head (216f2d1).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #429      +/-   ##
===========================================
- Coverage    93.65%   93.64%   -0.01%
===========================================
  Files           60       60
  Lines        15938    15936       -2
  Branches      2065     2065
===========================================
- Hits         14926    14924       -2
  Misses        1007     1007
  Partials         5        5

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


AWS cost attribution (1/4): write documentation on how to deploy the system to another community

#4453 didn't include how to deploy what was deployed to openscapes to another AWS cluster. This issue is meant to capture documenting what one would do currently in order to accomplish that.

More useful docs link: https://infrastructure.2i2c.org/howto/cost-attribution/


Run peaq dev parachain with docker commands

This pull request is just to test the docker image work flow. This pull  request will be closed without merging

We don't need to merge this MR because it's only for generating the docker image. However, after the peaq-dev forked successfully, we needed to update the docker image again because the parachain's chain spec changed.

Close again


Why check for existence of key when the database already has foreign key constraints

In the experiments endpoint, it is checked whether the simulation id already exists before creating an experiment with that simulation id. What is the need for this when the database already has foreign key constraints?
location: https://github.com/tudelft-atlarge/opendc-web-server/blob/master/opendc/api/v1/simulations/simulationId/experiments/endpoint.py#L81
@leonoverweel

It's easier to throw the right error code this way, because the foreign key exceptions thrown by SQLite are a bit weird to catch.

ok. Maybe we could change that sometime in the future, when we move to a db with better errors. :-)

Yes. The web server can probably be abstracted a bit more too - I have a couple of ideas. Closing this issue for now; we'll reopen when we switch away from SQLite.


Very annoying glitch

So, with the new update if you make a mistake in update function code, you will be stuck with alerts showing endlessly, so I need to reload the page. Please fix it as soon as possible, thanks

Hi
Sorry about. Fixing it right now i should start testing the editor more carefully after these changes :/
Sorry about it. Tahnks a lot for reporting the error.
Cumps

Should be fixed now (web version)


Installation issue in windows 7

After installing the program it is showing this pop-ups-

i've downloaded the 'python39.ddl' from internet paste it in /system32 as well as replace it in /TwitchLink. but same popup is coming.

Sorry, TwitchLink only supports Windows 10 or higher.


intialize memorymanager based on cache strategy

What changes were proposed in this pull request?
intialize memorymanager based on cache strategy
How was this patch tested?
UT

LGTM @zhixingheyi-tian  any further comments?

Hi @yma11 , for now we can specify the memory manager with cache type. But I am wondering whether we need to have one kind of memory manager which support both index and data cache in the same memory space. We will bring this back when requirements come out.

@yma11 Please address those remaining comments in a separate PR.


"Make" Error

cybers@ubuntu:~$ cd sprout/
cybers@ubuntu:~/sprout$ make
/home/cybers/sprout/mk/cassandra.mk:48: /home/cybers/sprout/build/cassandra/interface/thrift/gen-cpp/Cassandra.d: No such file or directory
/home/cybers/sprout/mk/cassandra.mk:48: /home/cybers/sprout/build/cassandra/interface/thrift/gen-cpp/cassandra_types.d: No such file or directory
cd /home/cybers/sprout/modules/thrift && ./bootstrap.sh
./bootstrap.sh: 1: ./bootstrap.sh: automake: not found
make: *** [/home/cybers/sprout/modules/thrift/configure] Error 1
help please

what is this to do with matrix-doc?


too@TahaTesser

@TahaTesser
FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':firebase_remote_config:uploadArchives'.
> Could not publish configuration 'archives'
   > Could not write to file '/Users/ayushpgupta/.pub-cache/hosted/pub.dartlang.org/firebase_remote_config-0.4.0/android/build/poms/pom-default.xml'.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.
-----------
* Where:
Script '/Users/ayushpgupta/development/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 897

* What went wrong:
Execution failed for task ':flutter:compileFlutterBuildRelease'.
> Process 'command '/Users/ayushpgupta/development/flutter/bin/flutter'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

* Get more help at https://help.gradle.org

BUILD FAILED in 56s

Gradle task assembleAarRelease failed with exit code 0.

I am getting this error on running flutter build aar. Is this issue related with this current issue?
Originally posted by @apgapg in https://github.com/FirebaseExtended/flutterfire/issues/2398#issuecomment-680887961

Hi @wlong800
It could be related, I added more labels to better describe the issue
Closing as duplicate of https://github.com/FirebaseExtended/flutterfire/issues/2398


[Task] add VoloPlugin

Description
Volo is a Rust RPC framework that developed by bytedance.
This plugin should base on WasmPlugin.
For client, wait for shenyu-client-rust.
Task List

[ ] add shenyu-plugin-volo module;
[ ] add shenyu-spring-boot-starter-plugin-volo module;
[ ] add shenyu-integrated-test-volo module;
[ ] add doc for shenyu-plugin-volo;

Relevant Skills
Know the use of Apache ShenYu, especially the wasm plugin.
Familiar with Java and Rust.

see https://github.com/cloudwego/volo/issues/202 , maybe we can try kitex first.

Hello, any progress ? Have you tried Kitex ?


Atomic Placeholder Creation for MacOS

This WIP PR implements the design described at #234 to increase the reliability of VFSForGit by making the process in which we create placeholder files/directories 'atomic' (we create them outside of the virtualization root in a known temporary directory and move them into place once they're 'complete').

[x] Functionality in the provider to create placeholders in the temp directory
[x] Provider can now register a temp directory with the kext
[x] Kext will no longer consider changes in a registered temp directory as one that should be considered
[x] Rename .temp and put it under the provider folder
[x] Use random names when creating placeholders in the temp directory
[ ] Clear temp folder on provider mount
[ ] Investigate if flushing is needed after setting file flags (and perf impact)
[ ] Validate functionality on a repo larger than Adium
(There are currently issues with the Adium build from the first commit of this series onward. Needs further investigation if these are related to the existing race conditions that we're investigating)
[ ] Call UpdatePlaceholderIfNeeded when in a retry scenario where we could potentially be overwriting user data
[ ] Performance analysis
[ ] Polish
[ ] Use our kext to prevent the user from doing anything to our .staging folder

Retiring this PR in favor of a new branch since this one since the rebase was getting messy.


Fixed a bug in argparse for Python 2.7.9 + added some new tests

Just wasted five hours of my live on this... this is the best workaround that I could think of.
Note that the behavior was completely messed up on Python 2.7.9 (which uses the same argparse as Python 3) but we didn't have any tests that caught it.

Sweet! :)

This could probably have been solved a little cleaner, but me and argparse aren't best friends.
Another benefit is the non-HDFS tests are running much faster now


catchup

@dbmuse Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Not sure we need this (and it refers to the OPs fork).


Implement Log in Throw Exception Method

This PR implementa Log in Throw Exception Method
Check out this Pull Request Locally
git fetch upstream pull/86/head:ImplementLog

I'm too late, but I don't think it's appropriate for a library to log errors. You're job is to communicate errors to the client of your library (via error codes and/or exceptions) and let them decide what to do. Perhaps they want to log, perhaps they don't. @buldrinie  @jdajes3

I agree that logging should not be responsibility of the library and if you see it from an SDK point of view could be a facility provided for the developer. The consumer could decide to disable it through a config file parameter and implement its own logging system/not logging at all or use it as is, considered a troubleshooting point for the custom application.

I think of the .Net framework. All errors are communicated to me thru exceptions. There is no place where Microsoft expects you to hook up a logging system to figure out your issues. It seems  that using logging would be an excuse to not make your exceptions/errors as valuable as they need to be.
If there is something happening in the background asynchronously, you could consider adding an Error event that the client could attach and event handler to so that they gain visibility into to the issues.
Having said that, I'm still fine with you making whatever decisions you think are best. I'm just providing a different perspective.

Sure, all exceptions remains in place. The SDK log is just an additional "safe" layer that could be useful to troubleshoot when the exceptions are not handled or logged in any place by the consumer application.

Is logging turned off by default? I don't think a client would want things written to disk without their explicit knowledge/consent.

I was looking more closely at the log messages. The issue with logging is

If the client decides to log, now they'll get 2 log messages for every exception you throw. Original Metasys code is littered with these sorts of issues. We quickly learned: 1) if you have enough context to figure what is going wrong and correct it then go ahead and log it, correct the issue but don't throw an exception or 2) don't log it but go ahead and throw an exception. You only do one or the other; else you just complicate debugging, not enhance it.
The messages aren't helpful. They aren't even correct.

Here's the first example when a 404 happens: ""An error occured while getting Http Status Code"
There was no error getting the status code. The error was that the resource you asked for was not found. For this log message to be useful you'd maybe want to write something like "The resource asked for could not be found by the server". But even with that message it's hard to understand how this is a useful debugging/diagnostic tool when you are already throwing an exception that conveys the exact same information.
The next message: "An error occured while parsing Flurl exception"
Again, the error wasn't with parsing a Flurl exception. The error was parsing the response from the server. So a message like this might be useful: "A response was returned by the server that could not be understood."
Finally, the time out exception: ""An error occured while getting timeout Exception in FlurlHttp"
There was no error trying to get a timeout. The error was that the request timed out.

The readme doesn't mention anything with how to disable logging.

Michael, in the current release version 3.2.0 the logger is just provided as a facility to be used in the consumer application. No logging is performed by the library, there was an issue with the initialization of the class and we didn't include this PR.  At this point, we could just move the classes of the Logging namespace in the example apps to have a demo on how exceptions should be trapped. My only concern is about the Exceptions that could be lost if not logged by consumer applications. I'll set a time slot to discuss more in depth this.


Add solution part to different calculators in Progression section

Describe the solution you'd like
Will add the solution steps for different calculators in Progression such as -

Arithmetic Progression
Geometric Progression

This issue will be under JWOC

@KushalGhosh9899 this issue is already taken up by someone else


[BEAM-13216] [Playground] Add client to send found tagged file to the backend

Add client to send found tagged file to the backend;
Add python proto files;

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @pabloem @damondouglas

lgtm


feat: nonce filling layer

Motivation
This is a replacement for NonceManagerMiddleware in ethers
Solution
A layer that fills nonces for transaction requests. It does not require you to specify a sender address like in ethers, instead we use a dashmap internally.
I would like some ideas on how to test that this works with sending transaction requests for the same sender from multiple threads at the same time (i.e. a sanity check on the atomics)
Closes #275
PR Checklist

[x] Added Tests
[x] Added Documentation
[ ] Breaking changes

@onbjerg could it be that the ManagedNonceLayer doesn't track contract deployments? I'm getting some strange errors where at once it complains that either the nonce is too high (1) or too low (0) after deploying a contract with the same provider.
For context see: https://github.com/alloy-rs/examples/pull/20/files

It should provided the function you use to deploy uses send_transaction

It should provided the function you use to deploy uses send_transaction. Where exactly are you getting the error?

I've opened an issue on Alloy to explore this with a minimal reproduction example: https://github.com/alloy-rs/alloy/issues/406 + https://github.com/zerosnacks/alloy-repro-406


Collada dae loader

Is there a plan to support other model formats? Like collada dae

Hi, no sorry. There are currently no plans to develop this any further. Fell free to fork it though


fixing typo in console

A slight typo in console comes right through on the redis IO home page :)

relates to redis/redis-website#17


TextField does not accept characters like @

Hi,
I'm using some TextFields in my application and I came across a problem. Every time I try to write an '@' character into the field it spawns a 'q' instead. Does somebody have a solution for this? Thanks in advance.
Greetings

In my it spawns well.

Hmmm,
at first thanks for your answer. I'm also running it on Windows 10 and Terminal.Gui version 0.70.0. I can't help it, but my TextFields do not accept '@' or '€'. Other characters that are created with an "Alt" combination like '|' or '²' are correctly written into the TextField.
Am I missing some property that I need to set?
I can't post my original code because of it is too long. Here is a little sample that also has this problem.
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Terminal.Gui;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            Application.Init();

            var topWindow = new Window("test runner");

            TextField randomTextField = new TextField("") { Width =29 };
            topWindow.Add(randomTextField);

            Application.Top.Add(topWindow);

            Application.Run();
        }
    }
}

No, you don't missing anything. You are right. Try with my pr at https://github.com/migueldeicaza/gui.cs/pull/345

Still not working on my machine
here I tried to write '@' and '€', it's also not working when I try to copy it.

In my pr the '€' really spawns to 'e'. I' will see that. But '@' spawns correctly. Try in the console to verify if it spawns alright too.

In the normal console the mentioned characters spawn correctly.
When I set '@' as text in the constructor of Textfield it works. But while running the app it doesn't.
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Terminal.Gui;

namespace ConsoleApp1
{
class Program
{
static void Main(string[] args)
{
        Application.Init();

        var topWindow = new Window("test runner");

        var rndTextField = new TextField("@@@@@@@@@@@@");

        topWindow.Add(rndTextField);

        Application.Top.Add(topWindow);

        Application.Run();

    }
}

}

But when I try to set '€' as text in the constructor of TextField I run into a "System.IndexOutOfRangeException"

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Terminal.Gui;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {

            Application.Init();

            var topWindow = new Window("test runner");

            var rndTextField = new TextField("€");

            topWindow.Add(rndTextField);

            Application.Top.Add(topWindow);

            Application.Run();

        }
    }
}

Ok, I already got it. It's a bug in Stack and there is an pr to fix at https://github.com/migueldeicaza/NStack/pull/42. I created a package and is working at constructor. Writing inside the text there still have some incorrect mapping.

Nice to hear you found a solution for one of the problems.

I am confused, the issue for the NStack#42 should have been fixed a long time ago, I do not think I need #42 do I?   It shows a conflict for a change that is already there.

There is a pr in https://github.com/migueldeicaza/NStack/pull/42/commits/fe8866e169dcf2ab1657d2e0f2041ae6f3146d2c that needs to be merged in NStack to solves this issue.

I am confused - that PR was already merged.

This is my version of the fix, and it is quite old:
https://github.com/migueldeicaza/NStack/commit/3fc024fb2c2e99927d3e12991570fb54db8ce01e#diff-8ddd7307a3d5c77d1e274e484581031c

But why they are still open in the pr? Another day I merged into a new branch and used in the Gui and I didn't get the error in the ColumnWidth method of the Rune class on the textfield at Redraw. And I've some changes that will be possible to use the AltMask + ControlA + delta and special characters, like €. I'm still working to solve some small bugs that yet persist. If you can update the Stack I'll include that improvments too. Thanks.

@migueldeicaza, that is the pr that solves the ColumnWidth of Run class error
https://github.com/migueldeicaza/NStack/pull/44

It's working well now. Some cache problem with the NStack nuget package, I think. Sorry

Good point, I should keep at least the test.


Doc Update: Refer to sqlalchemy_redshift

redshift_sqlalchemy is deprecated. I have changed references to the latest sqlalchemy_reshift. Also see discussion, https://groups.google.com/a/continuum.io/d/msg/blaze-dev/UYVVWBAlabk/BUxsff-0AgAJ

hi, could someone give me feedback on how I can improve this PR to better help Odo documentation?

I think a few more things needed to be updated to completely resolve the consequences of the renaming of redshift-sqlalchemy --> sqlalchemy-redshift. I made a pull request against your branch. Please review, merge if it looks good, and the new changes should show up here.

Thanks @branning! I have created a new PR https://github.com/blaze/odo/pull/489 as I couldn't figure out how to do it here.


K.cumsum for CNTK backend

Summary
Related Issues
PR Overview

[x] This PR requires new unit tests [y/n] (make sure tests are included)
Tests for cumsum and cumprod have been split for the time being
[N] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[Y] This PR is backwards compatible [y/n]
[N] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thank you for the PR, @abhaikollara. The computation looks inefficient to me. Could you follow the official example in CNTK?

I'm sorry for my poor comments. The result in the official example is a function which takes a Sequence not a variable. Thus, it is not appropriate to define it as a Keras' backend operation. Your implementaion is for the best for now.

Thus, I think it is better to merge this PR if you resolve the merge conflicts.

I'm sorry for my poor comments. The result in the official example is a function which takes a Sequence not a variable. Thus, it is not appropriate to define it as a Keras' backend operation. Your implementaion is for the best for now.

Aah, thanks. I did try to make that work without any luck.
I'll resolve the conflicts now

@taehoonlee I've resolved the conflicts, but there seems to be an issue with Travis builds. If it's not too much trouble could you check out #12478

@taehoonlee The issues have been solved.