Noindex /learn

As part of cleaning up our Google footprint, we want to de-index /learn. We don't want people to arrive from Google to the middle of a challenge (because they most likely won't be ready to sit down and code right then and there).
So we need to add this to all pages of /learn
<meta name="robots" content="noindex">

What about keeping indexes on the introduction pages for each curriculum section?

I do not think we would want to index anything from learn on the public search engines.

@RandellDawson We can create /news landing pages for each of these certifications and its subsections. These can be more keyword rich and betted designed for Google. Our current /learn pages aren't designed with traffic from Google in mind.

@raisedadead Based on what @QuincyLarson said above, is this a way we can just remove the introduction/landing pages from the curriculum?

Yes, I believe so.

@raisedadead are you working on adding the meta tags? If not, I can give it a go.

@hassaanp if you are interested in this here is a plugin that I intend to add to the client application:
https://github.com/itgalaxy/generate-robotstxt
Feel free to work on this.
Thanks.

@raisedadead - acknowledged.

Pushed a PR.
I have tested the code on my local setup by building the client and serving it via npx.
The learn page shows that the meta tag for noindex is added. The appropriate robots.txt is also generated.
I have used this plugin to generate the robots.txt file. If I am not wrong, the plugin is built upon the one referenced by @raisedadead. I just found it more convenient since it could be plugged directly into the Gatsby config.

Thanks for researching this and using a more customisable plugin. In this case, let's take this further and make sure that the only /learn/* paths are not indexed (which I assume the current PR does).
Also, I see there are options for picking up env values for host, etc. Lets implement those and configure this to be able to deploy for www.freecodecamp.org, www.freecodecamp.dev and so on.

The default values are picked from the siteMetadata in the config, which I assume will have the appropriate ${siteMetadata.siteUrl} defined when deploying for either mentioned URLs.
Yes, this PR is only targetting the /learn/* path. I have pushed another commit that removes the noindex header in the unmount lifecycle method to ensure it is not persisted across other paths.

Gosh! That reminds me that we need to make this dynamic:
https://github.com/freeCodeCamp/freeCodeCamp/blob/882cddd80076b232882c8960a784499d95e816e7/client/gatsby-config.js#L12-L14
Will you be interested in handling that in this same PR. If not let me know we can tackle this separately.

Apologies for the delay.
I have pushed a commit to make the siteUrl dynamic via the .env
Kindly review @raisedadead

Left some additional comments, this is mostly ready.


Cannot add Pipeline in the ReindexOnServer method

NEST/Elasticsearch.Net version: 7.16.0
Elasticsearch version: 7.16.3
Description of the problem including expected versus actual behavior:
When using the ReindexOnServer() method there is no way to add Pipeline, because it ends with error:
unknown field [Pipeline] did you mean [pipeline]?.
Probably it's because there is no [DataMember] attribute on the Pipeline property in the IReindexDestination
Steps to reproduce:

Create ReindexOnServer request
Add any pipeline to the destination's descriptor

Thanks for raising this @kplazinski. I suspect you are correct. I'll get this fixed and into the next patch release.

@kplazinski 7.17.2 has shipped with this fix.


Specify dtype of indptr and indices as int32 explicitly

indptr and indices are always int32 dtype.

If this is not fixing any bug, I'd suggest to avoid it. We need to support large sparse matrices (#3513), so I've advised to leave the type generic in new PRs so that it's less work for whoever picking up this conversion challenge 😅

Thanks 🙏


[FR] Wind direction abbreviation 'NE' instead of 'northeast'

I like the plugin as it is, but it would be really nice if the wind directions were given instead of 'northeast' or 'south' or 'east south-east' wording, in 'NE', 'S', or 'ESE' and so on.
That could save a lot of space giving the exact same information.
Thank you for your consideration. ❤

This is done in version v0.3.1
The only difference in the macros is this for the long version %winddirstr-today% and for the short version use %winddirstrshort-today%. Thanks, that was something I wanted to add for quite a while now. It does save a lot of space in the output.
Thanks, William.

Works great. I love it.
Thank you. 🙇♂

Plugin has now been added to the community plugins list. BRAT plugin is no longer required to install/update the plugin.
Thanks for all the input, William


[WIP]: fix clippy

@morenol what do we do with these doc-empty errors?

@morenol what do we do with these doc-empty errors?

I have trying to fix them but they are a lot and I am not familiar with all of them


Introduce Nightwatch for JS functionality testing

Introduces Nightwatch for testing JS functionality (which is always a nightmare in Capybara etc, and can test things that aren't unit-testable i.e. intended browser functionality when using JS).
Notes (to be added to README if and when this is merged to master)
Installation
Make sure you have nodejs >= 0.12 installed, then just npm install from the root directory.
Running the tests locally
Make sure the site is running on localhost:3000 then in another tab just do make test or to watch the tests run automated in Firefox, do make test-firefox
To run a single spec (there are only two at the moment) add spec=<specname> to the command, e.g.
make test spec=login

Looks good, automated would be a plus! :)

Yes, we'll need to do some setup on Jenkins/CI to spin up the Rails app then run npm install && make test or whatever. It's not part of the PR though. :+1:

I've created an issue (#582) to remind us, so merge at will! :)


spec to audio

first working attempt at reconstructing audio from its spectrogram. switched spectrogram process to save as grayscale 16-bit png using pillows. saving original sample rate and dB range for reconstruction. using Griffin-Lim algorithm to approximate lost phase data

oh for reference. this is where i pieced things together. along with chatGPT for the suggestion to use pillow instead of matplotlib as it applies figure formatting, color mapping, and anti-aliasing which is lossy or creates compression artifacts. PIL prevents this along with giving us better control over scaling, which makes it easier to invert later.


feat: add metric collection to catch sink errors

closes #365

@GrayFlash Can you please resolve merge conflicts o this one?


Cloudwatch input should not store pointers on entry

Description of Changes
Entries should not be storing fields with pointers. This causes complications further when we try to refer to these fields using other operators that cannot reflect the type because it is a generic pointer interface{}. For example cloudwatch entries going into the filter operator will get this error:
- type: filter
  expr: '$record.message != nil and $record.message matches ".*warm[\\d\\D]*"'

yields:
{"level":"error","ts":"2021-09-21T02:06:26.026Z","msg":"Running expressing returned an error%!(EXTRA zapcore.Field={error 26 0  interface conversion: interface {} is *string, not string (1:44)\n | $record.message != nil and $record.message matches \".*warm[\\\\d\\\\D]*\"\n |

And this is because $record.message was being stored as a *string rather than a raw string
Please check that the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Avg (%)
CPU Avg Δ (%)
Memory Avg (MB)
Memory Avg Δ (MB)

1
1000
1.5345148
-0.03453672
128.48747
+0.4816742

1
5000
5.2935476
+0.2244873
140.04001
+4.4063873

1
10000
9.534472
-0.20688152
145.69531
+0.18063354

1
50000
45.949375
-1.2581635
169.83392
-7.3786316

1
100000
91.44833
+3.2580261
233.5986
+8.87648

10
100
1.9482887
-0.13797975
134.31088
+0.10223389

10
500
5.81036
-1.017252
139.42336
+0.44126892

10
1000
10.931249
-0.15512943
143.40005
-4.5101013

10
5000
50.673145
-3.759369
176.13187
-1.6888428

10
10000
101.788635
+2.8210602
220.43008
+3.9994507

Log Files
Logs / Second
CPU Avg (%)
CPU Avg Δ (%)
Memory Avg (MB)
Memory Avg Δ (MB)

1
1000
1.3276005
-0.24145103
130.14655
+2.140747

1
5000
5.0346665
-0.034393787
136.2298
+0.59617615

1
10000
9.758646
+0.017292023
147.99799
+2.483307

1
50000
47.24204
+0.034500122
175.21107
-2.00148

1
100000
91.840836
+3.650528
232.72252
+8.000397

10
100
2.086254
-1.4543533e-05
134.37958
+0.17092896

10
500
6.3967566
-0.43085527
138.70999
-0.27209473

10
1000
11.207296
+0.120918274
146.15329
-1.7568665

10
5000
55.53401
+1.1014977
182.11261
+4.2919006

10
10000
101.10947
+2.1418915
225.9635
+9.532867


[Blazor 0.9.0] does not accept accents in the component parameters

Describe the bug
When I have a component with a string parameter, when passing a string with an accent, for example: "Início", the accented character is replaced by "?".
To Reproduce
I'm using the component this way
<AsideItem active="true" text="Início">
The browser appears as follows
In�cio

Thanks for contacting us, @eglauko.
@javiercn, can you please investigate this? Thanks!

@ajaybhargavb  can you please confirm this is fixed already? Thanks

Yup. I can no longer reproduce it. @eglauko, please re-open if you are seeing this issue in the latest version.

Hi @ajaybhargavb.
In preview 5 the error continues.
You can see it in this project: https://github.com/eglauko/Blazor.Chat

@eglauko - do you know what encoding your file is using? Is it saved as UTF8?
Do you have a repro project we could try out?

Eu não sei onde posso ver o encoding, mas deve ser o padrão do visual studio 2019.
Quando eu utilizo uma palavra com um acento como text do html funciona, o problema só ocorre quando é um parâmetro de componente.
Eu acho que o erro ocorre durante a geração do método BuildRenderTree
You can see it in this repro project: https://github.com/eglauko/Blazor.Chat

This way it works

Same problem here. I have a lot of projects that works with UTF8, only in Blazor project we get this errors.

This should be fixed already as part of https://github.com/aspnet/AspNetCore/issues/11435


Fixed documentation for the default values for the order rule (fixes #601)

Fixes #601, and replaces #608.
cc @ljharb

Hmm, looks like another dependency update breaking... Looking into it a bit.
Anyway, thanks @ljharb for your advice :)

Builds on master are broken, and the culprit is Babylon. I fixed the error and have made a PR over there https://github.com/babel/babylon/pull/170.
This is safe to merge, as it's just md files, but we'll need a new Babylon release before being able to have a working test suite. Tip: Do not remove your node_modules folder @benmosher :)

@jfmengels released<EMAIL_ADDRESS>

@hzoo Thanks! Tests are passing now :)

Changes Unknown when pulling aeb740296bbc217e66b185614ad96b9ece43b02c on fix-order-docs into ** on master**.

rebased and merged via 024a8a7

Cool, thanks!


logger rotating bug

Description
Should not treat logger filename in config as a directory. First, extract the logger directory. Then, try to create logger directory if it does not exist.
Steps to Reproduce
set logger file in nakama.yaml
logger:
  file: '/path/to/log/nakama.log'

Expected Result
create path/to/log/nakama.log
Actual Result
something else, mess
Context
javascript
Your Environment

Nakama: 2.4.0
Environment name and version:

"runtime":"go1.10.3"
"runtime":"go1.11.1"

Operating System and version:

macOS 10.13.6
Ubuntu 16.04.5 LTS

see PULL REQUEST #312

Fixed in b17aa8cee2f7703ed0d5ea02f26ecb6c20b49326 and will be part of the next release.


Error on creating the MyvmNva

First of all Awesome tutorials!!
Second I am unable to create the MyVmNva VM. The error states that its unavailable in EastUs. I will attach the error later tonight as I now must sleep.

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

ID: 5578dba8-a3fa-6d18-fa65-9f9d490b365a
Version Independent ID: eda00501-d2c7-3da3-877a-10b770b08b1c
Content: Route network traffic Azure PowerShell
Content Source: articles/virtual-network/tutorial-create-route-table-powershell.md
Service: virtual-network
GitHub Login: @jimdial
Microsoft Alias: jdial

@tapialuism thanks for the feedback. Please post your error when you are ready to continue.

Glad you like the tutorial. I copy/pasted the instructions and was able to create the NVA VM in the East US this morning. Since you're unable to, you may need to file a support request. #please-close

@tapialuism We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


svg and canvas elements text equivalent identification and use in accessible names

SVG and CANVAS can be used instead of IMG element in Heading and other content that can be used in creating accessible names for headings and labels.   Need to include their equivalents in name calculations

This will be part of version 1.1 of the OAA library


Update deps + flake.lock

The dependencies need to be updated, otherwise the build fails with the latest rust release:
error[E0282]: type annotations needed for `Box<_>`
  --> /build/cargo-vendor-dir/time-0.3.34/src/format_description/parse/mod.rs:83:9
   |
83 |     let items = format_items
   |         ^^^^^
...
86 |     Ok(items.into())
   |              ---- type must be known at this point
   |
help: consider giving `items` an explicit type, where the placeholders `_` are specified
   |
83 |     let items: Box<_> = format_items
   |              ++++++++

Context: https://github.com/NixOS/nixpkgs/issues/332957

I still get a build error, fyi
> nix run[romeo@Romeo-P1 presentation]$ nix run github:GaetanLepage/typst-lsp -- -h
warning: ignoring untrusted substituter 'https://ros.cachix.org', you are not a trusted user.
Run `man nix.conf` for more information on the `substituters` configuration option.
warning: ignoring the client-specified setting 'trusted-public-keys', because it is a restricted setting and you are not a trusted user
warning: ignoring untrusted substituter 'https://ros.cachix.org', you are not a trusted user.
Run `man nix.conf` for more information on the `substituters` configuration option.
warning: ignoring the client-specified setting 'trusted-public-keys', because it is a restricted setting and you are not a trusted user
error: builder for '/nix/store/hfbh0m88550i9mvh1si5ivkd2wjdgra8-typst-lsp-0.13.0.drv' failed with exit code 101;
       last 25 log lines:
       > error[E0446]: private type `UsageInfo` in public interface
       >     --> /build/cargo-vendor-dir/hayagriva-0.5.2/src/csl/rendering/mod.rs:1216:5
       >      |
       > 1216 |     fn will_have_info<T: EntryLike>(&self, ctx: &mut Context<T>) -> (bool, UsageInfo) {
       >      |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't leak private type
       >      |
       >     ::: /build/cargo-vendor-dir/hayagriva-0.5.2/src/csl/mod.rs:2699:1
       >      |
       > 2699 | struct UsageInfo {
       >      | ---------------- `UsageInfo` declared as private
       >
       > error[E0446]: private type `UsageInfo` in public interface
       >     --> /build/cargo-vendor-dir/hayagriva-0.5.2/src/csl/rendering/mod.rs:1245:5
       >      |
       > 1245 |     fn will_have_info<T: EntryLike>(&self, ctx: &mut Context<T>) -> (bool, UsageInfo) {
       >      |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't leak private type
       >      |
       >     ::: /build/cargo-vendor-dir/hayagriva-0.5.2/src/csl/mod.rs:2699:1
       >      |
       > 2699 | struct UsageInfo {
       >      | ---------------- `UsageInfo` declared as private
       >
       > For more information about this error, try `rustc --explain E0446`.
       > error: could not compile `hayagriva` (lib) due to 11 previous errors
       > warning: build failed, waiting for other jobs to finish...
       For full logs, run 'nix-store -l /nix/store/hfbh0m88550i9mvh1si5ivkd2wjdgra8-typst-lsp-0.13.0.drv'.


Docs rmsd to rssd

Closes #15907 by updating the documentation and changing a variable name.

cc @WarrenWeckesser

Thanks--the other idea that came up in the matching issue was a deprecation cycle and eventual returning of the genuine RMSD, but from the discussion it sounds like a clarification would still be useful even if we did deprecate/change later.


Debug console loses ability to differentiate lines of output

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.75.1 (Universal)
OS Version: macOS 13.0.1 (22A400) (M1 Pro)

Steps to Reproduce:
It's not 100% clear to me what the definitive recreate is. Initially, I thought it was something to do with code that outputs multiline log entries, but while testing with Extensions disabled, I realized that observation was a red herring.
I created a Loom to show the issue:
https://www.loom.com/share/66777e9fdb774a4186db493d02091667
In short, the Debug console starts by identifying individual output lines as I would expect; then it clumps a bunch of lines together as if they are one; then it goes back to single lines.
The impact is that output filtering doesn't work properly, which makes some debugging tasks very difficult.

I think that using Winston writing directly to stdout here will be the issue, because then there is just a stream of output and no correspondence between calling some log API and one "message". Grabbing some logs might help verify that's the issue.
/jsDebugLogs

Please collect trace logs using the following instructions:

If you're able to, add "trace": true to your launch.json and reproduce the issue. The location of the log file on your disk will be written to the Debug Console. Share that with us.
⚠ This log file will not contain source code, but will contain file paths. You can drop it into https://microsoft.github.io/vscode-pwa-analyzer/index.html to see what it contains. If you'd rather not share the log publicly, you can email it to<EMAIL_ADDRESS>

Thanks Rob.
I've attached the VSCode logs and the debug console output.
Here's the launch configuration:
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "node",
      "request": "launch",
      "name": "Launch Sutro Server",
      "skipFiles": ["<node_internals>/**"],
      "runtimeArgs": [
        "-r",
        "ts-node/register",
        "-r",
        "tsconfig-paths/register",
        "--expose-gc"
      ],
      "args": "${workspaceFolder}/server.ts",
      "console": "internalConsole",
      "internalConsoleOptions": "openOnSessionStart",
      "outputCapture": "std",
      "env": {
        "NODE_ENV": "DEV",
        "SUTRO_LOG_LEVEL": "debug"
      },
      "trace": true
    }
  ]
}

Note that outputCapture is set to std because, you're right that Winston writes directly to STDOUT (see https://github.com/winstonjs/winston/blob/master/lib/winston/transports/console.js#L77 for confirmation of this).
Each debug console log entry starts with a date, an option service name in parentheses, a level in square brackets and then the data. Some log entries are multiline.
The file lines starting at 702 all the way up to and including 1164 are identified in the Debug Console as being the same line.
vscode-debugadapter-835d37d5.json.gz
debug-console.out.zip

In the debugger we just do process.stdout.on('data', sendToVSCode), we don't make an effort to split lines, which means multiple lines could be sent in a single output event.
Code pointer: https://github.com/microsoft/vscode-js-debug/blob/cd4fe8e3d271db323323bdb13c881061d39dc657/src/targets/node/subprocessProgramLauncher.ts#L64-L65

You can have a multiline log output message too, the loom shows this. But maybe splitting single messages by line is better than combining lines randomly?

A framework-agnostic version could be to support ETX characters when capturing std instead of console.
This would need an opt-in approach.
In this mode, the JS DAP implementation would store incoming data and only call dap.output on receipt of an ETX character.
I'd be happy to have a crack at implementing this if you think it seems sensible.

Took a crack at an implementation. Be gentle :)

For the specific use case of Winston, I have the liberty of creating a specific Transport that could be configured specifically to support VSCode (if that's doable).

Or you could use a transport that calls console.log once per message 🤷

For the specific use case of Winston, I have the liberty of creating a specific Transport that could be configured specifically to support VSCode (if that's doable).

Or you could use a transport that calls console.log once per message 🤷

Ahem... Yeah... Or that :)

Or you could use a transport that calls console.log once per message 🤷

Even that's not guaranteed, we don't know how the OS will chunk output to us

Yeah, it looks like console.log is just a wrapper around writes to the stdout and stderr streams: https://github.com/nodejs/node/blob/2d9bf91fe1c545521e5578f6b31d59c79b402b82/lib/internal/console/constructor.js#L268
However, I was (perhaps mistakenly) under the impression from here that VSCode somehow intercepts calls to console.* differently... is this not the case?

Yea, we get them in that event over CDP. It does some magic, such as putting objects in the "console" group so that they can be examined and don't get garabage collected as soon as they fall out of scope, for example. Node.js emits that event, though I'm not sure where internally that happens.

Hey, @connor4312 I am new to programming and use vs code for my projects. I would love to contribute to this project.
But I don't know much programming currently learning OOP in java.
Please assign me some basic tasks so I can also start my open-source journey.

Thanks for volunteering, but Dan already put in a PR (which I just merged 🙂 )


Write tests

Hello friend!
This library looks really cool, but could use some tests to aid with development and collaboration.
I've written a few using pytest, and then pipenv to manage dependencies.
This issue is just so I can point my PR once it's ready.

Sounds great! Please let me know what testing framework you plan on using so that if I get to writing tests before you submit a PR, I'll make sure to write them in the same framework.

Closing since I merged your pull request


Module '"esri/core/Collection"' resolves to a non-module entity and cannot be imported using this construct.

Hi, I update my project with the latest API version 4.5.0 like this :
"arcgis-js-api": "4.5.0",
"@types/arcgis-js-api": "4.5.0"

After npm install, when I try to compile my project, I have this error with at-loader :
[at-loader] Checking started in a separate process...
[object Object]
[object Object]
[object Object]
[object Object]
[object Object]
Module '"esri/core/Collection"' resolves to a non-module entity and cannot be imported using this construct.

[at-loader] Checking finished with 6 errors

It seems there is a problem with the type definition, Collection is defined as an interface and as a class :
declare module "esri/core/Collection" {
  import Collection = __esri.Collection;
  export = Collection;
}

interface Collection<T = any> extends Evented {}
export class Collection<T = any> extends Accessor {...}

So, if I make import like this
import * as Collection from 'esri/core/Collection';  => not work
import Collection = __esri.Collection; => work

You will need to use import Collection = require("esri/core/Collection"); to import the AMD style classes.

It seems there is a problem with the type definition, Collection is defined as an interface and as a class :

This is a "trick" to declare multiple inheritance in TS, we use it internally all the time.

Thanks for the response.
But other import work with import * as XXX from 'YYYYY';
For example, these imports works.
import * as Map from "esri/Map";
import * as MapView from "esri/views/MapView";
import * as ScaleBar from "esri/widgets/ScaleBar";
import * as Home from "esri/widgets/Home";
import * as BasemapGallery from "esri/widgets/BasemapGallery"

That only works because the way we define most classes is using a special construct that allows you to define multiple inheritance, separate constructors and allows extending the definitions externally if you want to. This leads to the type definition to be interpreted as a module by TS, even though really it's a class. We can't use the same kind of definition for Collection because it is a generic type. The only way that you can import it is using require (which is also recommended for other class modules).

Thanks for your help.
Last question, when I set collection of sub layer of my sublayer, I have this error (not at compilation), but at runtine
let collection : Collection<Sublayer> = new Collection<Sublayer>();
subLayerParent.sublayers = collection;

init.js:327 [esri.core.Accessor] Accessor#set Assigning an instance of 'esri.core.Collection' which is not a subclass of 'esri.core.Collection<esri.layers.support.Sublayer>'
However I declare explicitaly the type of collection....?

Yes, this is something that we are looking at fixing (it should be valid to do this). The Collection<Sublayer> is a compile time type, but we also have a mechanism to enforce runtime types (and autocasting) for Collection. The way this works is by creating a unique subclass for a given element type using Collection.ofType, and we will verify at runtime when you set a collection that it has this correct subtype (this is where the error comes from). Hopefully in 4.6 (/cc @ycabon) we will allow automatic up-casting from general Collection to subtype Collection, but for 4.5 you can do (from the top of my head, not tested):
// Do this once somewhere, basically returns a new constructor for a dynamically created class
const SublayerCollection = Collection.ofType(Sublayer);

let collection = new SublayerCollection();
sublayerParent.sublayers = collection;

Cool, thanks for the trick ! :)


missing parent  ATP-dependent chromatin remodeler activity (GO:0140658)

Shouldn't
ATP-dependent chromatin remodeler activity (GO:0140658)
be a decendant of
GO:0015616    DNA translocase activity ?
@colinlog

Why do you say a remodeler is always a translocase?
ping @colinlog

I think both remodellers and helicases are always translocases + lots of other things. Another option  is to get rid of DNA translocase?

Ah right the way "DNA translocase is defined" the remodelers aren't translocases. I thought all this class were classed as translocases but maybe not. Closing...


Enhance split words

Keep "Response200" as "response_200", but don't make "s3config" become "s_3_config". :)
Related to the comments here: https://github.com/triaxtec/openapi-python-client/pull/375#issuecomment-831377313

Latest on main breaks e2e 😬. I made a branch enhance-split-words in this repo with the update (or you can just task re).

Rebase went fine on my side, not sure what you meant. :)

(and yeah, I did the task re)


Is there anyway to output a color-image of differ instead of black-white?

For example,
A1,1 is (255,64,0)
B1,1 is (80,160,0)
so output differ is
Out1,1 (|255-80|,|64-160|,|0-0|)=(175,96,0)

Hey YandexStudio, you should be getting color images by default. Have you tried on multiple sets of images? If so, can you send me a pair of that produces black and white output, where the original pictures are in color?

Here are the results I get when diffing your images (the first two would not work until I removed the alpha channel from the first image - I need to add to the docs that images must be the same size and have the same channels):

I'm not sure why yours are different, or why PIL did not complain for you about the alpha channel difference in the first pair of pictures. I think this may have to do with using WSL. I'll try to test it on a similar environment as well.

Could you please tell all the dependency of this tool?
I only installed graphviz ( in Ubuntu under WSL, I run diffimg and it said command not found, suggesting me install this, after install this, it can runs, but in CentOS, the system don't show any suggestions with "maybe missed package", just retruns "command not found" and I manually install this)
I've tried on CentOS 7 and also returns black-white result.
Also, this time I removed alpha channel with
convert input.png -alpha off input-no-alpha.png
So I think this issue may not related with alpha channel, I think maybe I lose some dependence package, and maybe that cause this.

My diffimg is not available as a standalone shell command, only through python. The way to run it is:
python -m diffimg <im1> <im2>

I believe you are using this diffimg: https://manpages.debian.org/jessie/graphviz/diffimg.1.en.html
The only dependency is the python library pillow which should be installed if you did pip install diffimg.

@YandexStudio Did this solve your problem?

Yes, it works now, correctly. But why if I don't specify a output file name，it then output a JPG？I think the differ shouldn't be compressed.

No, I think currently all my questions are solved. Thanks for your patience.


Spelling error

You spelled "Architecture" wrong in the README.. currently it says "Arcitecture".

I opened a pull request for you if you want to merge
https://github.com/Mozharovsky/CVCalendar/pull/44
Cheers
Mike

Hello @mengelhart,
Thank you for fixing this issue. I'll merge the pull request with releasing a new CVCalendar version.
Regards,
Eugene


Typo in COMMON_ARGS in saml2.config

Should referred_binding at https://github.com/rohe/pysaml2/blob/master/src/saml2/config.py#L42 be preferred_binding to match https://github.com/rohe/pysaml2/blob/master/src/saml2/config.py#L201 ?

Yes! I've fixed it now.


support for environment-modules

Once environment-modules has a json structure similar to spider -o spider-json $MODULEPATH. buildtest can implement a parser to extract module information to do all the module operations.
See https://github.com/cea-hpc/modules/issues/303 for more details.

@shahzebsiddiqui I think this belongs in the (other repository?) you are working on to just interact with modules. The more I learn and test, I think having lmod / modules in general as a hard requirement for the library severely limits it's usage.

I am kinda thinking of closing this issue, unless their is a desire from user to support environment-modules.  The Module API features are specific to Lmod. Some features like user collection and Spider are specific to Lmod and not environment-modules. I added this as  a reminder but likely wont get to it since this requires major changes to ModuleAPI or even have API specific to each module system.

I'm in support of that! I think the dependency on Lmod, if anything, needs to be somewhat lifted. If after all the work is done this is discussed as badly needed, then we can open a new issue.


Change detection doesn't work with objects and custom types

I've got a property $foo on my document that holds a mutable object. That object is converted to an array by a custom type when persisting the document. Unfortunately, when I change a property on that mutable object, $foo isn't recognized as having changed because of this condition:
if ($orgValue === $actualValue) {
    // ...
   continue;
    // ...
}

It's still the same instance, and so the contition evaluates to true.
Is there a better way to detect changes? Maybe something like comparing two spl_object_hash()es?

spl_object_hash() gave us enough headache with embedded documents since they can be reused, I'm not sure I would like to follow that way... As a temporary solution I'd recommend using immutable objects while we try to figure out a way to fix the bug.

Thanks, I already changed it to be immutable.

Just to update, so far I have no idea how this could be introduced in a bugfix manner...
Offhand I recalled there was #976 (but that'd be feature) and also I have an idea for 2.0 that could fix the issue at hand, namely I'm pondering over using database values for preparing change sets.

Hi!, happy 1 year anniversary for this issue.  This particular issue has long vexed me in one of my projects.  We use value objects extensively and in our world a value object with the same values should evaluate to be equal to a different value object with the same values.
I think ODM should make use of an interface offered in Common that mimics an interface never accepted into PHP core: https://github.com/doctrine/common/blob/master/lib/Doctrine/Common/Comparable.php
This code block could check if $orgValue implements Comparable, if so, use the compareTo($other) method.  If not, default to === check.

This is actually interesting, I had no idea such interface exists :) Given our current plans for 2.0 it indeed might be feasible to add as a feature earlier. @alcaeus @jmikola any downsides coming to your mind? From my perspective it can get into our way later if we'd like to remove the Comparable approach.

The Comparable interface looks like a good option here. I recall @guilhermeblanco trying to get this into core some time ago, but I wasn't aware it was added to Doctrine Common. Let's confirm if ORM is using it to address the same use case here.

@jmikola you need to work on your summoning abilities a bit ;)
Using the comparable interface would make sense - we could compare the objects using === if one of them doesn't implement Comparable and use that if both of them do. I'm moving this to the 2.0 milestone since I don't think I'll get to it before 1.2, which I'm trying to wrap up asap.

The issue has been reported almost 4 years ago. A good solution (using the Comparable interface) has been proposed three years ago. The problem has still not been solved? Is this really true or am I missing something?
Is there any other solution for this?

The problem has still not been solved?

Indeed it hasn't. If you're up for it, we're happy to assist while you create the pull request.


'reportOptionalMemberAccess' wrong report

Error
"name" is not a known member of "None"    Pylance[reportOptionalMemberAccess](https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportOptionalMemberAccess)
(variable) name: str | None

Source Code
@dataclass()
class An_Object(YRPC_OBJECT_BASE_CLASS):
    id: str | None = None
    name: str | None = None
    description: str | None = None

Can you provide a minimal, self-contained code sample that reproduces the behavior you're seeing? The code above references YRPC_OBJECT_BASE_CLASS, which is not defined. The screen shot also differs from the provided code, so it's not clear how item in the screen shot relates to the code above.

Can you provide a minimal, self-contained code sample that reproduces the behavior you're seeing? The code above references YRPC_OBJECT_BASE_CLASS, which is not defined. The screen shot also differs from the provided code, so it's not clear how item in the screen shot relates to the code above.

Now I know where is the problem.
    # wrong code
    if (item.an_object.name == None):
        print("no")

    # right code
    if (item.an_object != None and item.an_object.name == None):
        print("no")

I wonder why the python community still refuse to use this syntax:
if (item?.an_object?.name == None):
    print("no")

Can you do me a favor to raise this issue again in the python repo?
https://github.com/python/cpython/issues/103128


Review of testing code

@yangraymond leads us in a tour of his testing code.

[ ] Figure out how to view raw TAQ file
[ ] Match SAS code results with Python
[ ] Make DATA_FILES actually a list


add DolphinServiceMenu to apps.yaml

adding a Dolphin Service Menu to upload files to IPFS!

Isn't using globalupload.io a little far from decentralisation?
I bet we can integrate with Siderus Orion, or any local IPFS node to have a more decentralised experience.

Isn't using globalupload.io a little far from the purpose of decentralisation?
I bet we can integrate with Siderus Orion, or any local IPFS node to have a more decentralised experience.

Well, I wasn't aware of the Orion Project (thanks for that!). The main advantage of using an external service like globalupload is that we really don't have to rely on a local IPFS node running on our system. (I hope you can see how that might be convenient for the average user).
Thanks for the suggestion anyways!

Hello @amar-laksh, is this project still active?

Hello @amar-laksh, is this project still active?

Well yes :)


feat: add https middleware package

insert reference into main README.md

insert reference into main README.md

:tada: This PR is included in version @kittgen/nestjs-authorization-v1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @kittgen/nestjs-authorization-v1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Waiting for Zotero database at each startup

After installing Zotero 5 and better bibtex (zotfile is also installed), I have a message at each startup saying Better Bibtex is waiting the database of Zotero.
It delays my access for no reason (I have no automatic export in place.). Is it possible to disable it (or to put it in background)?

Why on earth would you assume I put it there for no reason? You think I like that delay?
It says "waiting for Zotero database" because it's waiting for Zotero's database, not mine. The auto-exports don't even need to access the Zotero DB, as they live in my own database, which is online well ahead of the Zotero database. I access the Zotero database (a.o.) to generate the citation keys, and the Zotero team has made it quite clear no-one is to touch the database until it has finished initializing. Having users interact with the references leads would kick off BBT activity leading to race conditions accessing the database that are really hard to recover from.
That the database now behaves this way in Zotero 5 is not my choice, nor do I have any influence over the point where Zotero gives the all-clear to extensions. Other extensions may not need the database during startup -- I do.
BBT is one of the more complex extensions out there, if not downright the most complex, modifying Zotero's behavior by patching the running program at numerous places. This is necessary because what BBT does cannot be done by the Zotero-supported extension API alone, it's a very delicate setup, and can easily break Zotero; if you post a bug report on the Zotero forums, one of the first things they'll rightfully ask is "does this also happen if you disable BBT". The wait-until-all-clear as instructed by the Zotero team is required to make the BBT/Zotero combination less fragile.

Accepted. I took offense to the "unnecessary". It is necessary for a combination of architectural and pragmatic reasons. I dislike the idea that people would assume I had put it there without thinking things through. And I really dislike the idea that I should do this-or-that to avoid blame. There's an easy way for me to avoid blame, which is to close this repo, take a new email address, and go on to do things for which I am appreciated.  Or, you know, use the stock BibTeX built into Zotero.
TL;DR the "waiting" screen is there for damn good reasons. It can go away when Zotero signals me sooner that I'm allowed to touch their database. Things might get better in Zotero 5.1. Until then, it stays.
It is possible to disable BBT from the addons screen, and only re-enable it for an export, but this will require a restart of Zotero. This way, BBT will get out of your way until you need it. But if you want BBT to be ready when you need it without the enable-and-restart, it needs to be in a stable state. It's like demanding from your legislator that no money goes to ambulances you never use, but then, but then being grumpy it takes 5 days for them to order a new ambulance when you do need one.
As to the "most common case": if you feed the bib(la)tex database, you need citation keys. For citation keys, I need to access the database. For access to the database, I need to wait. This has nothing to do with auto-export. This has everything to do with how BBT works through-and-through, and how long Zotero takes to give the go-ahead. In Zotero 4, extensions would have the guarantee that the DB was ready before they were even called. In Zotero 5, there are several things that are started at the same time, and the user is given the illusion that Zotero is ready because you can click things. As soon as you actually try to do something (like adding a reference, or exporting), you'll have to wait until Zotero is actually ready. BBT just makes that wait visible, because I was really tired of dealing with the fallout of the race conditions that this caused. The all-clear signal is what slows down BBT startup, from a recent startup on my system:
Waiting for Zotero database took 11.427 s
Loading citation key took 0.03 s
Starting auto-export took 0.001 s
Starting key manager took 0.846 s
Starting serialisation cache took 0.023 s
Loading journal abbreviator took 0 s
Installing bundled translators took 0.685 s

It's not the problem that for a second there are no citation keys. Actually it is a problem if you were to kick off an export in that time, because the export requires the cite keys to be present, but it's not the problem.
The problem is those 1.5 seconds in which Zotero has given everyone the go-ahead, and Zotero frees the interface to you, but BBT still needs to start. This has in the past lead to actual, not hypothetical, race conditions which broke the Zotero/BBT combo, for which I was also blamed. If I "buffer up" these changes to the point in time where BBT has gotten the go-ahead and finished its own initialization, that's more complexity and fragility to add to BBT, and would make it very easily the case that BBT exports are kicked off before the citekeys are all ready, which would make that export fail, and for that, of course, I would be blamed. So regardless of what I'll do, people will choose to blame me. If Zotero would give me a signal somewhere within those 12 seconds, I could easily startup within that time. But that is not the case.
So is it technically possible to get buffer-up activity within that 1.5 second time and act on it later? Yes. Is it pragmatically possible to do so, at the cost of quite a bit of complexity of behavior that I would then have to support for users whose systems I cannot inspect, for a single dev doing this in the wee hours alongside a family, a job, a PhD, and currently co-caring for a terminally ill family member? Occasionally trying to get some sleep?
No.
Getting BBT to work under Zotero 5 required a grounds-up rewrite which took 5 months of my life I really didn't have to spare.  I need BBT to behave in predictable ways if I am to do this. If the Zotero devs strenuously recommend waiting dor the DB, even if that means a 12 second delay before BBT can even start up its own init, that's out of my hands.
If someone is willing to volunteer to put up a patch that implements a stable delayed start that deals with the race conditions in a reliable way and promises to support it for the first six months in which it will break users' reference manager while they're under time pressure to get their thesis finished and come yelling here that I'm responsible for the disruption, then yeah, sure. But I don't see that happening, really. Of the Zotero 4 implementation, 0.102143691% came from others, the rest was me. For the Zotero 5 implementation, that number is 0.001399122212%. This is by all means a solo effort.
In the meantime, the "waiting" is necessary to keep BBTs behavior predictable to a level that can support it. If Zotero at some point manages to give the all-clear signal for DB access sooner, BBT will automatically benefit. Zotero 5.1 is going to add a separate field for cite keys and if that is actually filled always so I don't have to, a lot of problems are going to be resolved. But that is a might big "if".

Thanks for all these explanations. Waiting few seconds is really not a problem and you will see what's bring the 5.1 version.
I take this opportunity to thank you for the development of BBT. This very useful piece of software change the everyday life of plenty of users around the world. I was very glad to see it working when I upgraded to Zotero 5. I didn't know how I could work if it was not the case!

Sorry for sounding so strongly, but getting BBT to work under Zotero 5 has not been a pleasant experience.

:robot: this is your friendly neighborhood build bot announcing test build 4965 ("Merge branch 'master' into @lockless").

If you want to live dangerously: https://github.com/retorquere/zotero-better-bibtex/wiki/Configuration#lockedinit


Granting one SuperAdmin permission gives user all SuperAdmin permissions

Please complete the following fields as applicable:
Expected behaviour:
Giving an OrgAdmin the 'Manage Org Affiliation' permission should allow the user to see the 'Change Affiliation' box on the Templates page, not remove their existing permissions when they change Org affiliation, and NOT provide access to all other SuperAdmin functionality (e.g. Notifications and Themes)
Actual behaviour:
Giving the user the one permission results in access to all SuperAdmin functionality.
This is due to the fact that we just do a check for .can_super_admin? in most places which verifies that the user has at least one SuperAdmin permission. We should:

Update the can_super_admin? method so that it only returns true if ALL SuperAdmin permissions are granted.
Update the code to check the individual permission where appropriate instead of just can_super_admin?
Consider adding additional permissions

Steps to reproduce:

Give a test OrgAdmin user account the 'Manage Org Affiliation' permission.
Login as that User
Access the Notifications tab

This is an edge-case scenario. We are having our DMPTool editorial board help edit Funder templates so they need the ability to change Org Affiliation. I am going to patch DMPTool in place to allow it, but we should have a larger conversation about permissions.


ROS2 version in 'ros2cs' metadata doesn't match currently sourced version. This is caused by mixing versions/builds. Plugin might not work correctly.

https://github.com/RobotecAI/ros2-for-unity
I followed the official tutorial ，after  achieving topics
''Add ROS2TalkerExample.cs script to the very same game object.
Add ROS2ListenerExample.cs script to the very same game object.''
I started the project in Unity, Unity reported several errors as follows,i didn't see two nodes talking with each other in Unity Editor's console 。

the main error is
''ROS2 version in 'ros2cs' metadata doesn't match currently sourced version. This is caused by mixing versions/builds. Plugin might not work correctly.''
My ROS2 is humble ,Linux is Ubuntu 22.04.Unity is 2021.6.24f1 personal

Hello,
It looks like you have ROS 2 sourced in your environment and that version doesn't match the build version of Ros2ForUnity in the editor. Try to run the editor in a clean terminal (make sure the ROS 2 is not sourced) and try again.

hello ,I did have ROS2 sourced by  adding 'source /opt/ros/humble/setup.bash'  in ~/.bashrc. But now I removed this line of code from  ~/.bashrc. Then in a clean terminal,I input 'unityhub'.And the Unityhub is opened .And in the unityhub,I start my project. The error still exist. Looking forward to your reply.

Hello,
It looks like you have ROS 2 sourced in your environment and that version doesn't match the build version of Ros2ForUnity in the editor. Try to run the editor in a clean terminal (make sure the ROS 2 is not sourced) and try again.

Hello,
It looks like you have ROS 2 sourced in your environment and that version doesn't match the build version of Ros2ForUnity in the editor. Try to run the editor in a clean terminal (make sure the ROS 2 is not sourced) and try again.

你好，
看起来您的环境中有ROS 2源代码，但该版本与编辑器中Ros2ForUnity的编译版本不匹配。尝试在一个干净的终端中运行编辑器(确保ROS 2不是源代码),然后重试。

Have you built your own binaries or used one of the Ros2ForUnity releases? Also, which version are you using, overlay or standalone?
(I assumed standalone, therefore asked for "clean" terminal. If you are using overlay, then of course you need to source the appropriate version)
You can also try to run the editor from the clean terminal because running the editor from unity hub might also be the cause (change the editor and project path):
~/Unity/Hub/Editor/2021.6.24f1/Editor/Unity -projectPath <PATH_TO_PROJECT>

It works well after I follow the tutorial in a dual system.I guess there probably is something wrong with the vm‘s Graphics card.


Use one IntervalTree per unit

#70 was an incomplete fix for overlapping ranges; it only handled overlapping unit ranges, and not overlapping function ranges.
No changes to the external API yet, but FullContext could easily be removed after this.
Also a couple of refactoring commits that aren't really related.

Coverage increased (+0.4%) to 77.926% when pulling cd4017a1cf0ac08ea29e907990fe757122b9b8f3 on philipc:overlap into e4b9a3518c97c223bc6718b3ee680f15b9299537 on gimli-rs:master.


Ss18/pr1 enhancement

Fixed typos:
precident -> precedent
orginal -> original
Found with: https://github.com/ss18/grep-typos

This will be in next as well I guess.
cc @thymikee


virus_en.map - Antivirus Protection

Hi Martin,
in your Filter "virus_en.map" is a filter regarding "Expired Antivirus Software" where you FIlter for "/anti.?virus.{1,10}protection/i"
This is a little hard, because some of my customers have a Mail Footer, where they write, that they have a solution for "antivirus protection"..... every Mail gets filtered...
So i think you should change that...

Hi,
This rule is way to hard. I removed it a minute ago. Thanks for the advice.


Implement some more of the webhook validation skeleton for Pipelines.

This still doesn't add any validation, but implements more of the interface.

Ref https://github.com/knative/build-pipeline/issues/29


DH export tab popping error

Hi Damion, Sometimes when I try to export a file in DH the options doesnot show up it appears blank. I  mainly seen this for file which has samples 500+. Also I use Windows operating system and opens in Google Chrome. I am also wondering is that due to internet unstable. Can you please have a look. Thanks :)

I presume there are steps you take to try again and succeed?  What would they be?
Also, do you work on a mac? If so when this happens can you press and hold option-command and J keys, and then screen shot the "Console" tab of the popup - usually that's the default. That will tell us if a javascript error has occured.
Thankx,
d.

Woops, I glossed over that you use windows.  On windows you can get the popup javascript console message by clicking on upper-right tripple dot ... menu, then "More Tools > Developer Tools" (bottom menu item).

When I see the  error I just go back and click the tab again. But sometimes it takes a while to load.

So try to catch console screen to see if there was an error. In meantime if you could send me the dataset that is triggering this error situation I can see if I can replicate.  If however its a function of computer speed/memory etc, then I might have to depend on you to catch error report.

I have more details this error occurs when I open a DH file plus I try to capture the info but my console didn't show any error :(

Hi Nithu,
I think you sent me the output file that was successful when it works?  I need the input file that was causing the sporadic output failure.
d.
From: Nithu John @.>
Reply-To: cidgoh/DataHarmonizer @.>
Date: Friday, December 9, 2022 at 4:27 PM
To: cidgoh/DataHarmonizer @.>
Cc: Damion Dooley @.>, Comment @.***>
Subject: Re: [cidgoh/DataHarmonizer] DH export tab popping error (Issue #367)
I have more details this error occurs when I open a DH file plus I try to capture the info but my console didn't show any error :(
[Image removed by sender. image]https://user-images.githubusercontent.com/85367999/206814465-62dc4817-2c01-4da5-a17f-4acbd17af729.png
—
Reply to this email directly, view it on GitHubhttps://github.com/cidgoh/DataHarmonizer/issues/367#issuecomment-1344915636, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA6QWRWXD5QNHJLQJH4QZLDWMPCSNANCNFSM6AAAAAASXNUA7U.
You are receiving this because you commented.Message ID: @.***>

Ok, I'm not able to replicate the problem on my powerbook, so I'll have to see about testing on a windows machine, perhaps slower machine is the main variable.  Perhaps we can debug it on your computer before a friday lab meeting?
BTW It isn't about internet connection.  The app runs 100% locally, no internet needed.

@nithujohn can you verify if this comes up again with the new version of DH we've been testing. If not, we'll close this.  New handsontable code might resolve it.

Tested and looks good at my end! thank you


Add support for Alter Procedure in Babelfish

Description
This commit adds support for Alter Procedure implementation in Babelfish. It adds a new parser rule to handle alter procedure cases, and creates a new procedure in place of the old one. All of the existing privileges are carried to the new procedure. The user can choose to modify the procedure parameters and/or body.
Issues Resolved
BABEL-2661
Test Scenarios Covered

Use case based -

create procedure alter_proc1 as select 1
go

exec alter_proc1
go
~~START~~
int
1
~~END~~

alter procedure alter_proc1 as select 2
go

exec alter_proc1
go
~~START~~
int
2
~~END~~

create procedure alter_proc2 @l int as select @l
go

exec alter_proc2 @l = 200
go
~~START~~
int
200
~~END~~

alter procedure alter_proc2
    @l int
as
    if (@l = 1)
    begin
        select 100
    END
    ELSE
    begin
        select 0
    end
go

exec alter_proc2 @l = 1
go
~~START~~
int
100
~~END~~

exec alter_proc2 @l = 200
go
~~START~~
int
0
~~END~~

Arbitrary inputs -

create procedure alter_proc2 as select 1
go

create procedure alter_proc3
as
    exec alter_proc2
go

alter procedure alter_proc2
    @l int
as
    select @l
go

exec alter_proc2 @l = 1
go
~~START~~
int
1
~~END~~

exec alter_proc3
go
~~ERROR (Code: 201)~~

~~ERROR (Message: procedure alter_proc2 expects parameter "@l", which was not supplied.)~~

Negative test cases -

CREATE PROCEDURE alter_proc_p1
AS
    select * from alter_proc_users
GO

-- Expect error for procedure with same name
CREATE PROCEDURE alter_proc_p1 @param1 int
AS
    select * from alter_proc_orders
GO
~~ERROR (Code: 2714)~~

~~ERROR (Message: Function 'alter_proc_p1' already exists with the same name)~~

Minor version upgrade tests -
Test file added to schedule

Major version upgrade tests -
Test file added to schedule

Performance tests -

Tooling impact -
None

Client tests -
Manual Concurrency Testing:

-- Initialize Procedure
-- Open two threads
-- Thread 1: Runs a while loop executing the procedure
-- Thread 2: Alter procedure

-- Run the two thread in parallel

-- output: Thread 1 prints the output of the first procedure, and after alter is run, thread 1 outputs the results of the newly created procedure

-- Initialize Procedure
-- Open 3 threads
-- Thread 1: Runs a while loop executing the procedure
-- Thread 2: Alter procedure
-- Thread 3: Alter procedure

-- Run 3 threads in parallel

-- Output: One of the alter threads will error with "tuple concurrently updated", and Thread 1 will output the result of the altered procedure that did not error

-- Initialize Procedure
-- Open 4 threads
-- Thread 1: exec proc, Thread 2: Alter proc, Thread 3: exec proc, Thread 4: Alter proc
-- Run threads 1 and 2 in parallel, then run threads 3 and 4 in parallel
-- Thread 1 will output the initial proc result, followed by the altered proc from thread 2
-- Thread 3 will output the altered proc from thread 2 result, following by the altered proc result from thread 4

-- No error for "tuple concurrently updated" like in last example, because the alter proc threads are not executed in parallel

alter-procedure-schema.mix file contains tests demonstrating how permissions are carried over for procedures created on schema.
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

147 of 163 (90.18%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 72.745%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contrib/babelfishpg_tsql/src/pl_handler.c
79
86
91.86%

contrib/babelfishpg_tsql/src/hooks.c
57
66
86.36%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
41758

Relevant Lines:
57403

💛  - Coveralls


[SPARK-31934][BUILD] Remove set -x from docker image tool

What changes were proposed in this pull request?
Remove set -x from the docker image tool.
Why are the changes needed?
The image tool prints out information that may confusing.
Does this PR introduce any user-facing change?
Less information is displayed by the docker image tool.
How was this patch tested?
Ran docker image tool locally.

cc @dongjoon-hyun who brought this to my attention.

Thanks, I'll merge this to master & branch-3.0 :)

Test build #123651 has started for PR 28759 at commit e300be5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28274/
Test PASSed.

Test build #123651 has finished for PR 28759 at commit e300be5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123651/
Test PASSed.


Reactive UI changes

I've collapsed the lower two tables into dropdowns, as well as updated the UI so it works nicer on more screen sizes.

I'd like to move all those rules at the bottom to tooltips on the individual fields as a next step if this stuff looks good

I like the accordions.  I might move the avg dmg and kill chance above the accordions, OR they could become the text of the Accordion.Header elements!
I think there are two big facets where we want to go in different directions.

I want things horizontally squeezed together even when there is plenty of space, and I want this so 1a) your mouse/input-mechanism doesn't have to do a lot of travel to interact with elements, like modifying both shooting situations to use Px=1 and increment the BS. And 1b) it is hard to read table rows when the columns are way spaced out.  Table columns should be minimal width.  1c) Same thing as 1b but for glancing between Situation 1 and Situation 2.

I don't like to vertically stack two vertically-long things and require a lot of lengthy scrolling back and forth to change inputs or look at both outputs.  In mobile with your changes, the two situations are vertically stacked and vertically long, thus you have to scroll up and down a lot.  In mobile with what I have, you can do a quick horizontal scroll between the two situations or zoom out to see everything at once.

I think I agree with you on point one, I was trying to make maximal use of the extra space but we could shrink it a lot without it looking barren.  On point two, mobile sites and apps are designed around all the information being visible horizontally with some necessary vertical scrolling.  Maybe we could make a sliding pane so you can swipe between the two?

Note for self: admire the hover effect for the "KillChance vs Sv & W"
Some notes while we were talking...
Maybe use Carousel so mobile can swipe between situations and we have an option to go back to old way if they want to zoom out.  Ah, can just have button to switch to desktop site?  We don't want to have to explain how to force desktop site in the browser menu.
Some questions on if we should do the swipy based on screen size (so big tablets that don't need it won't have it), because I find that hard to correctly validate with browser screen/device emulation and I don't know all the ways to do this sort of thing.  I try to put in OnePlus 10 Pro's alleged 1440x3216 resolution with ~525 ppi density, but that doesn't work when naively inputted to firefox device list.  Even dividing by 2.75 (seen in other devices) didn't work.  Will have to empirically figure it out? Gross.  Ah, Daniel found a good site and term "viewport": https://yesviz.com/devices/oneplus-10-pro/ so I want 412x919 in firefox...confirmed.  Daniel cracked the case!
I still like having things compact and easy to glance between the 2 situations, rather than expanding things to use the space.  You bring up a great point to use horizontal space somehow, like moving the Notes and/or a combined situation table upwards and occupying unused side space.
Possible split up of work: Daniel on Carousel/swipy thing and Jacob to experiment with moving things like Notes to unused horizontal space and maybe making a both-situation result table.

PC version back to smaller size, all tables now highlight row on hover (and tap on mobile)

Carousel working on mobile

Great work, I hope soon to have some time to do my homework.
On my phone, the delay between completing my swipe gesture and the start of the carousel transition is annoyingly long.  With PC browser dev tools on your page, I changed the transition from 0.3s to 0.05s and was much happier with that.  Using 0.0s made it seem like nothing was happening.  What do you think about 0.05s?
Also, I'm worried about people keeping track of which situation they are on, and that we need something like "Situation X" to float/whatever-the-right-term-is at the top once you scroll down.  What do you think?

I was unsure about the transition time myself, it does still feel a little slow though for sure. I'll try .05s and see what that feels like.  And I actually had the same thought on the situation headers! I was planning on making them stick to top of screen when you scroll up when I have some time.

@daespinozah can you link me to the Accordion example where you got that accordion-related css?

@daespinozah can you educate me on why you put the app in a Stack element and how that changed layout?  Thanks.

Here's the link for the accordion stuff:
https://react-bootstrap.netlify.app/docs/components/accordion/#accordionbutton
As for the stack element, it's a more natural fit for elements arranged vertically that's the only reason I changed it:
https://react-bootstrap.netlify.app/docs/layout/stack

Thanks, but specifically where did you get the "accordion-related CSS?"

The accordions in the documentation are using it. I copied from the dev tools window.

Ah, thanks.

https://github.com/jmegner/KT21Calculator/commit/91f33ec1a34af7b3b9c2674bbe8edc149058056c incorporates Accordions based on your work.  I will look at the SwipeableViews stuff next.  Thanks again, and critique of what I did is very welcome.

I really like that you put the most relevant data in the header of the accordions, looks good!


python_info[version] will show python version number in the format "py-3.4.1"

python_info[version] will show python version number in the format "py-3.4.1" (without the quotes.)
Also changed prompt_name_preexec to prompt_name_precmd.
prompt_name_precmd is executed before the prompt is loaded.
This makes loading the prompt with python info sort of "proactive".

Why not use a Python version manager? The py- prefix can confuse users who have installed other Python implementations than CPython.

Why not use a Python version manager?
Python version will only show up when a virtualenv is activated. If a user is not using python module or if he hasn't activated virtualenv it wont show up.
The py- prefix can confuse users who have installed other Python implementations than CPython.
My pull request is only about python version and not about different interpreters. But I get your context. If you have a better format please recommend.

I've closed this in favor of the version included in master which gets the information from the python executable on the path in all instances.
However, I'd be open to improvements which only display the version when in a virtualenv or when using something like pyenv. Thanks for your contribution!


Ajax isnt working

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[X ] I am running the latest version
[X ] I checked the documentation and found no answer
[X ] I checked to make sure that this issue has not already been filed
[X ] I'm reporting the issue to the correct repository (for multi-repository projects)

Expected Behavior
Ajax used to display a form. (Works before used in conjection with Material Dashboard)
Current Behavior
What is the current behavior?
None- The <div> isn't showing ajax.

My Ajax:
<script>     function showUser(str) {     if (str == "") {     document.getElementById("txtHint").innerHTML = "";     return;     } else {     if (window.XMLHttpRequest) {         // code for IE7+, Firefox, Chrome, Opera, Safari         xmlhttp = new XMLHttpRequest();     } else {         // code for IE6, IE5         xmlhttp = new ActiveXObject("Microsoft.XMLHTTP");     }     xmlhttp.onreadystatechange = function() {         if (this.readyState == 4 && this.status == 200) {             document.getElementById("txtHint").innerHTML = this.responseText;         }     };     xmlhttp.open("GET","editdata.php?id="+str,true);     xmlhttp.send();     }     }     </script>


Cell animation view overlaps tabbar

I'm wondering if there any way to avoid tabbar overlapping during this case when cell is partly visible?
https://www.dropbox.com/s/wy62ktq29hw8rnl/IMG_6976.MOV

You can apply some heroModifiers to the tabbar and make it appears at the top. Try this first:
[.useGlobalCoordinateSpace, .forceAnimate]

thanks @lkzhao it helped!

@lkzhao it doesn't seem to work on iOS 11. Any ideas what modifiers should I try?

I just reviewed other issues and one of the suggestions that helps to fix overlapping was the following:  .beginWith([.zPosition(10)])
So the working solution for the views overlapping tab bar is adding the following modifiers to the tabbar:
[.useGlobalCoordinateSpace, .forceAnimate, .beginWith([.zPosition(10)])]

I have same issue, anyone solve this problem ?
when i use [.useGlobalCoordinateSpace, .forceAnimate, .beginWith([.zPosition(10)])] modifiers, whole screen has fade animation and background color of this screen is gone.


Demo Instance Limitations

Currently, the demo of goodwork throws a 403 Access Denied error when navigating to one of the projects visible on the logged-in site.

Additionally, attempting to create a new project fails:

Are these intentional limitations?

Just to clarify:
It's intentional to prevent the demo user from seeing anything, or doing anything, once they have logged in?

No, they have access to some projects, teams and offices where they can create tasks, discussions etc.

Specifically which projects, teams, or offices might I examine to examine the functionality mentioned?
When I attempted to access the visible projects on the home page of the demo, each of them returned the same error, would it not make more sense to hide project(s) if a user has not the permissions to (at least) view the project in question?

Project/team named "For Guests" should be accessible by guest user, are you facing any problem on accessing those?
Right now every project is visible but later there will be option for admin turn on/off visibility of private projects.

Was able to access the project.

Could not add a new task:

Is a project the same thing as a team?
Would it not make more sense to hide private projects by default?

project, team and office is basically same thing with same tools with same functionality.

Would it not make more sense to hide private projects by default?

it would, this will be implemented in future.


Nginx Subrequest return [Broken] CROSS_PROCESS

Please answer these questions before submitting your issue.

Why do you submit this issue?
[X] Question or discussion
[X] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
Hi, I'm trying to integrate Nginx with Skywalking APM, but I figure out an issue for the request that using subrequest, when I check the trace for the request with subrequest it show something like:

Endpoint: VNode - randomUUID
Span Type: [Broken] CROSS_PROCESS
Component: VirtualNode: #0

Question: What does that mean? and any Idea how to solve that?
For the request without subrequest it's work as expected.
To make it easier to check here is my nginx configuration: https://github.com/Ediliu13/nginx-sw8
Screenshots:
Without Subrequest

With Subrequest

Bug

Which version of SkyWalking, OS and JRE?
skywalking-8.1.0
skywalking-nginx-lua-0.2.0

Thanks in advance.

@moonming What is subrequest? Could you take a look?

@membphis is working on the Apache APISIX's plugin around skywalking 8.0 and nginx-lua 0.2, do you have time to check this issue? thx

I will take a look at this bug when I have time ^_^

@wu-sheng @moonming  you can assign this bug to me

@Ediliu13
I think the broken message is because subrequest doesn't run the log phase by default.
You can enable it via log_subrequest directive: http://nginx.org/en/docs/http/ngx_http_core_module.html#log_subrequest

@Ediliu13
I think the broken message is because subrequest doesn't run the log phase by default.
You can enable it via log_subrequest directive: http://nginx.org/en/docs/http/ngx_http_core_module.html#log_subrequest

Thank you @spacewander I just try to put that log_subrequest directive as your suggestion, in http, server, and location block but it still remains the same, so I don't think that is the proper solution for this issue.

auth_request doesn't run the body filter phase. Maybe you need to reimplement it with Lua code or add extra workaround for it.

@wu-sheng @moonming you can assign this bug to me

I'm havel same question with you too. Finally, how do you handler this question?


'Any' atom continues to move across the canvas while interacting with periodic table

Steps to Reproduce

Open Ketcher
Press 'Any' atom button
Press 'PT' button and move cursor on periodic table elements

Actual behavior
'Any(A)' atom continues to move across the canvas while interacting with periodic table
Expected behavior
'Any' atom dissapear while interacting with periodic table. If user has selected an element from the periodic system, then selected element appears under the cursor. If the user did not select anything, then the 'Any(A)' atom remains under the cursor.
Screenshots
https://github.com/epam/ketcher/assets/62464241/f3c97caf-c224-4326-8f0a-d4cb971b98f6
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 112.0.5615.138 (Official Build) (64-bit)

Ketcher version
Version 2.12.0-rc.1
[Version 2.10.0]

duplicate same behaviour as in https://github.com/epam/ketcher/issues/2804


Add pull request and issues templates

Add .github/ as a directory to the root project.
Add .github/pull_request_template.md that describes the expected format for PRs
Add .github/ISSUE_TEMPLATES and include bug_report.md and feature_request.md at minimum.

Completed in #37 . Closing


[BEAM-10120] Add dynamic timer support to portable Flink and Spark.

Please add a meaningful description for your change here

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.

 The changes I have made to java runner shared library can also used by Spark in the future. @ibzib

It seems like Spark doesn't support timers in portable execution. The changes I have made to java runner shared library can also used by Spark in the future. @ibzib

It seems like Spark doesn't support timers in portable execution. The changes I have made to java runner shared library can also used by Spark in the future. @ibzib

Portable Spark supports timers in batch, but not streaming mode.

It seems like Spark doesn't support timers in portable execution. The changes I have made to java runner shared library can also used by Spark in the future. @ibzib

Portable Spark supports timers in batch, but not streaming mode.

I'm going to merge this PR. Feel free to drop any comments here if you have any concerns @mxm . I'll follow up with any comments.

Thanks for the review!
I'm going to merge this PR. Feel free to drop any comments here if you have any concerns @mxm . I'll follow up with any comments.


Implemented Cascadia support

With this update .ssl files are now also supported. There were a few more extra optional columns that I could also add, but I didn't see the relevance. But if you want me to add them let me know. The columns that are used by cascadia are added.

Sure will do, what do you want me to call the generic type? ssl.rs? Or something with SpectrumSequenceList.rs, I would think ssl.rs to keep with the style of mztab?

The type can be the long version I think 'SpectsumSequenceList' butthe module name (and file name) can be ssl.rs indeed.


Handle SignalReconnecting event in ReconnectCheck

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

that makes sense, thank you for the contribution!


[Help] Really want someone add "Big or Small" event in this mod, fork from LC_Customdifficulty

@Venterok so i d like to see ant-man player model size bcz that's funny, i like to make the game more funny, this event change player model size ramdomly even they have custom model, that definitely require additional synchronization measures, but now LC_Customdifficulty mod have many glitch right now, so i hope u can do it
and also need to add vanilla support option (will be used in the future), "Big or Small" event need additional synchronization measures to change player model size, so if player still want to play this event in vanilla, need a option to make "player model size only change when lobby created"
note that this event won't decrease player speed but did can make turret can or cannot able to target the player in some angle due player size are changed; and must can able to use terminal after changed body size
i recently play LC_Custondifficulty very often, spawn many enemies outside and inside, i hope this mod can also make things more challenge

uh no one???

and also enemy spawn rate are too fast maybe, made a option to use Vanilla spawn rate pls? i still need to use MoreMonster mod for my modpack!

but base on vanilla player opinion: enemy spawn rate are too fast, need a option to use Vanilla spawn rate, HullBreaker should not change enemy spawn rate by default

wrong title


Bump version for http-server pkg to fix audit

@miniflare/http-server package has deps to selfsigned package and it was reported as low severity vulnerabilities.

Logs by `npm i miniflare`
~/Codes
❯ mkdir miniflare-deps

~/Codes
❯ cd miniflare-deps

~/Codes/miniflare-deps
❯ npm init -y
Wrote to /Users/leader22/Codes/miniflare-deps/package.json:

{
  "name": "miniflare-deps",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "Yuji Sugiura",
  "license": "MIT"
}

~/Codes/miniflare-deps
❯ npm i miniflare

added 40 packages, and audited 41 packages in 2s

4 low severity vulnerabilities

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

~/Codes/miniflare-deps
❯ npm audit
# npm audit report

node-forge  <1.0.0
Prototype Pollution in node-forge debug API. - https://github.com/advisories/GHSA-5rrq-pxf6-6jx5
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/node-forge
  selfsigned  1.1.1 - 1.10.14
  Depends on vulnerable versions of node-forge
  node_modules/selfsigned
    @miniflare/http-server  *
    Depends on vulnerable versions of selfsigned
    node_modules/@miniflare/http-server
      miniflare  >=2.0.0-next.1
      Depends on vulnerable versions of @miniflare/http-server
      node_modules/miniflare

4 low severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

This PR updates selfsigned to 2.0.0 (major version up), but actual implementation seems not changed.
https://github.com/jfromaniello/selfsigned/compare/v1.10.11...v2.0.0

Thank you! 😆


feature request: conditional property management

I think it would be really cool if we could set up conditional property management.
For example, the status property of a file with type: task  might be able to contain new, todo, done, etc. But for a note, it might contain new, in-progress, reviewed, etc.
Ideally (for me), the default behavior should be just one 'global' rule, but user can add more specific conditions with their own rules as needed for said property.

hello
this now possible with 0.2.5

You can find some detail in readMe at section "define a class for a file and authorized fields for this class"

Hi! Thanks for the amazing work. I think there is a slight misunderstanding here, but we're almost there. The new feature does bring us one step closer.
Let's say I have a file of fileClass: task
task.md
=========
status

And another file of fileClass: note
note.md
=========
status

I would like files of different classes to be able to contain different values inside their status field. My suggestion would be to specify possible values for classes as well. If such specification is done, override the global property manager settings.
Eg.
task.md
=========
status: [new, to-do, in-progress, cancelled, done, on-hold]

note.md
=========
status: [new, in-progress, reviewed]

Let me know if you think this would be a useful feature.

@weeyizhi I get your point.
Now that we can manage which field for which category of note, it shouldn't be too hard to define presets values for them. Just like in the settings where you can manage them globally, you could have those preset values in the "class file" for a category.
I'll have a look at that in the coming days

Hi this is now possible with version 0.2.9. Does it correspond with your request?

Hi. I tried for quite a while and could not get it to work. Then I noticed that somehow I'm unable to update to 0.2.9 via community plugins.
Clicking 'update' shows this (but the installed version only went from 0.2.7 to 0.2.8). Doing it again shows the same thing (installed version stays at 0.2.8). Reinstall also gives me the same result.

What is your obsidian version ? The minimal obsidian version for 0.2.9 is 0.12.4

Ah I see. I'm still at 0.12.3 because I'm not an insider! Will give this a go again when I get the update!

you should be able to get 0.2.10 that is available with 0.12.3 now

It works amazingly! The only comment I have is when we click on Manage <file class name> fields, instead of saying hello, it might be good to mention the fileClass name instead:

good catch! 😉
fixed in 0.2.11
I'm glad that you enjoy the feature, it's very helpful for my workflow too, thanks for the idea


Use AdminLTE as the base template for the project

The project is using AdminLTE as its base template, but it's better to insert the template one page per time instead of adding the entire template at once, since there are pages we're not going to use.


[ja] fix links to ephemeral volume types

en PR https://github.com/kubernetes/website/pull/33565/files
/assign @atoato88
/assign @nasa9084

I only updated the links listed here https://github.com/kubernetes/website/pull/33565/files
For some reason my IDE did the rest.

This PR updates two things:

updates links refered in PR discription.
replaces CR LF (\r\n) with LF (\n).

Almost all files in this repository use LF(\n) not CR LF(\r\n), so it's OK for me.
And links updated by this PR are OK too.

/lgtm

/approve


RFbridge doesnt update alarm sensor when disconnected from Internet (local)

RFbridge 2, firmware 3.5.2. integration (in Local or Auto). with Internet access for the bridge denied from the router, the buttons keep working but the window sensor doesn't update. I found that the bridge bridge connects to this IP (<IP_ADDRESS>) even with pihole block, and it stays online in the ewelink app.  Interestingly, if I allow internet access then pull the WAN cord out after the bridge initialized, the sensors keep working.
I am thinking that the bridge uses the connection to <IP_ADDRESS> to pull the clock time because the sensor triggers need "last triggered" which needs the clock on the bridge to be correct. What if we spoof this IP address and redirect it to a local NTP server? It wouldn't be this easy because this IP is not only n NTP but is also the connection to ewelink. But if anyone can figure how supply the time clock to sonoff devices without ewelink, that would be awesome... potentially solve all issues with sensors not working in local mode.

https://github.com/AlexxIT/SonoffLAN?tab=readme-ov-file#issues

There's nothing in the logs that would be helpful. This a problem between the bridge and the server. This is more a development suggestion sorry

Without device and any debug info I can't do anything

home-assistant_sonoff_2024-04-13T15-12-22.530Z.log
config_entry-sonoff-af35e4956ec234642bc36eb0ae423546.json
Sorry here you go

I can see local updates in your logs. You don't have them when there's no internet?
2024-04-13 10:12:17.263 DEBUG (MainThread) [custom_components.sonoff.core.ewelink] 100205192c <= Local3 | <IP_ADDRESS>:8081 | {'sledOnline': 'on', 'arming': True, 'rfTrig9': '2024-04-13T14:58:59.000Z', 'ssid': 'TP-LINK_E8A8', 'bssid': '8c:3b:ad:0f:8b:99'} | 682
2024-04-13 10:12:18.083 DEBUG (MainThread) [custom_components.sonoff.core.ewelink] 100205192c <= Local3 | <IP_ADDRESS>:8081 | {'sledOnline': 'on', 'arming': True, 'rfTrig9': '2024-04-13T14:58:59.000Z', 'ssid': 'TP-LINK_E8A8', 'bssid': '8c:3b:ad:0f:8b:99'} | 682

With no internet
2024-04-13 10:12:16.541 DEBUG (MainThread) [custom_components.sonoff.core.ewelink.cloud] CLOUD None => None 2024-04-13 10:12:16.902 DEBUG (MainThread) [custom_components.sonoff] 1 devices loaded from Cloud 2024-04-13 10:12:16.911 DEBUG (MainThread) [custom_components.sonoff.core.ewelink] 100205192c UIID 0028 | {'bindInfos': {}, 'version': 8, 'only_device': {'ota': 'success'}, 'sledOnline': 'on', 'ssid': 'TP-LINK_E8A8', 'bssid': '8c:3b:ad:0f:8b:99', 'init': 1, 'fwVersion': '3.5.2', 'rssi': -21, 'staMac': '34:98:7A:C3:CE:78', 'setState': 'arm', 'rfList': [{'rfChl': 0, 'rfVal': '57C602F6095E00C301'}, {'rfChl': 1, 'rfVal': '5E7E0308094C00C303'}, {'rfChl': 2, 'rfVal': '424402640738BB809E'}, {'rfChl': 3, 'rfVal': '498802680736BB8097'}, {'rfChl': 4, 'rfVal': '49460264073291A897'}, {'rfChl': 5, 'rfVal': '42080264073291A89E'}, {'rfChl': 6, 'rfVal': '42A2026A07408DE097'}, {'rfChl': 7, 'rfVal': '49E0026807428DE09E'}, {'rfChl': 8, 'rfVal': '49E6026A074052C897'}, {'rfChl': 9, 'rfVal': '49DA026A073E52C89E'}], 'cmd': 'trigger', 'timers': [], 'rfTrig0': '2024-04-12T12:05:04.000Z', 'rfTrig1': '2024-04-10T12:46:06.000Z', 'rfTrig3': '2024-04-12T22:08:29.000Z', 'rfTrig5': '2024-04-12T21:58:56.000Z', 'rfTrig2': '2024-04-13T14:44:49.000Z', 'rfChl': 1, 'rfTrig4': '2024-04-12T21:59:01.000Z', 'rfTrig6': '2024-04-13T13:16:17.000Z', 'rfTrig7': '2024-04-13T13:16:14.000Z', 'rfTrig8': '2024-04-12T20:01:32.000Z', 'rfTrig9': '2024-04-13T14:58:59.000Z'} 2024-04-13 10:12:16.913 DEBUG (MainThread) [custom_components.sonoff] LOCAL mode start 2024-04-13 10:12:17.263 DEBUG (MainThread) [custom_components.sonoff.core.ewelink] 100205192c <= Local3 | <IP_ADDRESS>:8081 | {'sledOnline': 'on', 'arming': True, 'rfTrig9': '2024-04-13T14:58:59.000Z', 'ssid': 'TP-LINK_E8A8', 'bssid': '8c:3b:ad:0f:8b:99'} | 682 2024-04-13 10:12:18.083 DEBUG (MainThread) [custom_components.sonoff.core.ewelink] 100205192c <= Local3 | <IP_ADDRESS>:8081 | {'sledOnline': 'on', 'arming': True, 'rfTrig9': '2024-04-13T14:58:59.000Z', 'ssid': 'TP-LINK_E8A8', 'bssid': '8c:3b:ad:0f:8b:99'} | 682 2024-04-13 10:12:19.915 DEBUG (MainThread) [custom_components.sonoff] Add 3 entities 2024-04-13 10:12:20.623 WARNING (SyncWorker_23) [py.warnings] /usr/local/lib/python3.12/site-packages/urllib3/connectionpool.py:1061: InsecureRequestWarning: Unverified HTTPS request is being made to host '<IP_ADDRESS>'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings   warnings.warn(

With no internet
home-assistant_sonoff_2024-04-13T15-54-10.296Z-no internet.log

Yes. Same TS will be skipped. I don't know if it safe to remove this check. Then another check would have to be made for local+cloud messages.
https://github.com/AlexxIT/SonoffLAN/blob/08070c312b52d877ffdfb31c73b91d696def4926/custom_components/sonoff/remote.py#L128-L130

I don't know how to fix this on the component side. Disabling the date check doesn't seem to be safe. You can also see that the message counter on the device side doesn't increase.

Pulling the WAN cable out the router while everything else is connected, the RF bridge continue to work normally for 2-3 minutes. Then it misbehaves


new replay engine UI is too small for PC // Laptops

yeah read title and also can you do resizable replay engine UI thanks

can you provide screenshot how it looks like

Nevermind I learned how to resize it 🗿, but my other issue is the orb
pulses and wave pulses doesn't sync
On Sat, Sep 21, 2024, 11:43 PM TobyAdd @.***> wrote:

can you provide screenshot how it looks like
—
Reply to this email directly, view it on GitHub
https://github.com/TobyAdd/GDH/issues/297#issuecomment-2365248681, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A2GLIJRMZMC7IKFN5246ZKLZXWO3JAVCNFSM6AAAAABOTTY7G2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGI2DQNRYGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

While there is no solution to this problem, since the pulse sync with orbs is still problematic to implement, disable pudding orbs and wave trail for now

Best thing to do the sync is just make your wave pulse to a timer that you can set

lol
lol
lol


Support for Testing Background Notifications in Terminated State (Cross-Platform: Android & iOS)

Use case
I'm currently developing a mobile app that requires handling notifications when the app is in a terminated state on both iOS and Android platforms. The core functionality involves receiving critical notifications that should trigger app behavior (such as opening a specific feature or action) even when the app has been manually closed or terminated by the OS.
For Android, I am using Firebase Cloud Messaging (FCM), and for iOS, I'm working with VoIP notifications via PushKit. I want to ensure that my tests cover these real-world scenarios where users may manually terminate the app. To address this, I'm integrating testing through Firebase TestLab, which supports Espresso, UI Automator (Android), XCTest, and Robo Test (iOS). However, I’m struggling with the following scenario:
Installing the app
Terminating the app
Validating notification reception in this state
Interacting with the notification and launching the app from terminated state
Continuing app execution based on the notification interaction

I haven't found clear guidelines or examples on how to fully automate tests for handling notifications in the terminated state. I need to replicate manual app termination, validate that the app responds correctly when reopened via a notification, and then continue interacting with the app as expected—all within the same test sequence.
This is especially critical because users often terminate apps, and important notifications (like those requiring immediate user action) still need to function properly.
Proposal
I would love to contribute by developing these tests, but I need some guidance or resources to better understand how to:
Use Espresso and UI Automator to simulate app termination, handle system-level notification validation, and resume app execution on Android.
Use XCTest for iOS to achieve similar results—terminating the app, testing notification handling, and then continuing app functionality based on the notification interaction.

If anyone has experience or insights on how to approach this type of testing, particularly using Firebase TestLab and these testing frameworks, your help would be greatly appreciated.
Thank you for considering this feature request and for your time!

Hi @valtbarbos!
 We’re have an issue for this feature which you can track. Unfortunately, it would require a lot of rework on Patrol.


Created a text counter

fixes #105
Text Counter Using Python
This Program counts the number of Words, Sentences, and Paragraphs in a given text file.
Input:
Takes the path of the text file as input file_path = input("Enter the file path: ")
Output:
Outputs the number of Words, Sentences, and Paragraphs in the given text
word_count = count_words(text)
sentence_count = count_sentences(text)
paragraph_count = count_paragraphs(text)
print("Word count:", word_count)
print("Sentence count:", sentence_count)
print("Paragraph count:", paragraph_count)

Don't forget to give a ⭐ & Happy coding!. 😊


[dependencygraph] Fix "command line too long" when sending to GitHub's dependency-graph API

We started to use the dependencygraph internally. It worked for some of our repositories, but for some other, it failed with no message, even when using --debug:
" 2>&1) # last line of the popen call
[DEBUG]
Dependency graph submission failed.

To understand the issue, I tried running the command curl directly (in a GitHub workflow) and I got curl: Argument list too long, which hinted that maybe we were hitting a limit on the command length with popen. I made the modifications I suggest in this pull request, and this indeed resolve the issue:
[DEBUG] 1288: popen( curl -w "\\nfcfad8a3-bb68-4a54-ad00-dab1ff671ed2%{http_code}" -X POST -H "Accept: application/vnd.github+json" -H "Authorization: ***" -H "X-GitHub-Api-Version: 2022-11-28" https://api.github.com/repos/<A_REPOSITORY>/dependency-graph/snapshots -d @/tmp/vcpkg/dependency_snapshot_9f868846-cd35-4af6-a062-2cd4a5d6b152.json 2>&1)
[DEBUG]   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
[DEBUG]                                  Dload  Upload   Total   Spent    Left  Speed
[DEBUG]
[DEBUG]   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
[DEBUG] 100  126k  100   227  100  126k    540   300k --:--:-- --:--:-- --:--:--  301k
[DEBUG] {
[DEBUG]   "id": <AN_ID>,
[DEBUG]   "created_at": "2023-07-31T20:54:53.465Z",
[DEBUG]   "result": "ACCEPTED",
[DEBUG]   "message": "The snapshot was accepted, but it is not for the default branch. It will not update dependency results for the repository."

 (As in, can curl use stdin for this purpose?)

Yeah I also thought about using stdin. 🙂
I tested it this morning. I started from the branch BillyONeal:process-stdin and I modified send_snapshot_to_api so that it uses stdin. Here's the very small diff between BillyONeal:process-stdin and klalumiere:process-stdin-plus-fix-command-line-too-long.
It works. On a Ubuntu 22.04 container image running on a Ubuntu 20.04 host, we get:
[DEBUG] 1288: execute_process(curl -w "\\nfcfad8a3-bb68-4a54-ad00-dab1ff671ed2%{http_code}" -X POST -H "Accept: application/vnd.github+json" -H "Authorization: ***" -H "X-GitHub-Api-Version: 2022-11-28" https://api.github.com/repos/<A_REPOSITORY>/dependency-graph/snapshots -d @-)
[DEBUG]   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
[DEBUG]                                  Dload  Upload   Total   Spent    Left  Speed
[DEBUG]
[DEBUG]   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
[DEBUG] 100  126k  100   227  100  126k    474   263k --:--:-- --:--:-- --:--:--  264k
[DEBUG] {
[DEBUG]   "id": <AN_ID>,
[DEBUG]   "created_at": "2023-08-01T14:26:32.030Z",
[DEBUG]   "result": "ACCEPTED",
[DEBUG]   "message": "The snapshot was accepted, but it is not for the default branch. It will not update dependency results for the repository."
[DEBUG] }
[DEBUG]
[DEBUG] 1288: cmd_execute_and_stream_data() returned 0 after   488006 us
Dependency graph submission successful.

I guess it should work on Windows too (I don't know the details about stdin enough on Windows).
What are the next steps? Do you want to include these 2 lines in your pull request?

I tested it this morning. I started from the branch BillyONeal:process-stdin and I modified send_snapshot_to_api so that it uses stdin. Here's the very small diff between BillyONeal:process-stdin and klalumiere:process-stdin-plus-fix-command-line-too-long.

Excellent. Let's see if I can get reviews on the other one so it can land, then this PR can use just that stuff.
Thanks again!

People aren't comfortable attaching my more complete rework to this higher priority bugfix so we're going to merge this as is and we can switch to using a stdin pipe later. Thanks for your contribution!


How to add wavesurfer plugin with ReactJs

Description

I want to integrate Region plugin into videojs-wavesurfer so the user can select the customized range by their self.

Steps to reproduce
 type = 'audio/wav';
          videoJsOptions = {
            controls: true,
            autoplay: false,
            loop: false,
            muted: false,
            fluid: false,
            width: 600,
            height: 200,
            bigPlayButton: false,
            playbackRates: [0.5, 0.75, 1, 1.5, 2],
            plugins: {
              wavesurfer: {
                backend: 'MediaElement',
                mediaType:'audio',
                pixelRatio:1,
                displayMilliseconds: false,
                debug: true,
                waveColor: 'grey',
                responsive:true,
                progressColor: 'lightblue',
                cursorColor: 'black',
                interact: true,
                hideScrollbar: true,

*** Start of the Wavesurfer Plugins which throw error ***
                plugins: [
                  // timeline
                  WaveSurfer.timeline.create({
                    container: '#wave-timeline'
                  }),
                  // regions
                  WaveSurfer.regions.create({
                    regions: [
                      {
                        start: 1.123,
                        end: 5,
                        color: 'rgba(255, 255, 205, 0.7)',
                        drag: false
                      }, {
                        start: 6.5,
                        end: 8,
                        color: 'rgba(205, 255, 255, 0.6)',
                        drag: false
                      }
                    ]
                  })
                ]
   *** End of the Wavesurfer Plugins which throw error ***
              },
            },
          };

Results
Expected
Wavesurfer plugin should work with ReactJs.
Actual
Getting error as below :

Error output
If there are any errors at all, please include them here.
Versions
videojs/wavesurfer
<EMAIL_ADDRESS>"3.3.1",
"videojs-wavesurfer": "3.3.0"

Make sure to import the timeline plugin beforehand and assign it to the wavesurfer namespace:
import TimelinePlugin from 'wavesurfer.js/dist/plugin/wavesurfer.timeline.js';
WaveSurfer.timeline = TimelinePlugin;

See https://collab-project.github.io/videojs-wavesurfer/#/react


Make objc-related expectations compile

Fixes #1004.
After looking at how msg_send! is supposed to be used, I realized that
we were erroneously passing type signatures to it.
The expectations compile now, but some of them are not formatted properly.
My guess is that rustfmt does not know how to format the following:
msg_send!(obj, arg1:1 arg2:2 arg3:3)

Notice the lack of separatros between the args.
Any advice on how to proceed here is appreciated.
r? @fitzgen

The expectations compile now, but some of them are not formatted properly.
My guess is that rustfmt does not know how to format the following:

This isn't the biggest issue in the worldh; there are places where rustfmt fails to nicely format assert_eq! macros in layout tests too.
That said, we could manually insert newlines after each name: arg pair and it might handle it a little bettter.
I'm going to go ahead and merge this PR as it is now, and leave exploring newlines for a follow up, since this gets the tests passing on macOS again, which is pretty important.
Thanks!
@bors-servo r+

:pushpin: Commit ca0aa04 has been approved by fitzgen

:hourglass: Testing commit ca0aa04995d557bc1e432e7422e8a75ba365afb7 with merge 053fb1debe276e55e78dda3b28fa7568a456ab94...

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing 053fb1debe276e55e78dda3b28fa7568a456ab94 to master...


Add onRemove callback

nodeOps/remove.ts should support callback when removing objects

(wontfix, new version)


Develop -> Master

This looks freekin amazing ! But, I think this requires a basic readme :). I've bootstraped the thing to give you a little motivation https://github.com/geeklearningio/gl-dotnet-authorizations/blob/feature/basic-readme/README.md
And a getting started on the wiki would be great. I think there is no code first support at the moment for the database. We should provide basic instruction to update it's schema and basic sample on how to provision and verify rights from domain code.

Yes you are perfectly right, documentation is a critical point !
It is definitively on my radar ;)