One login integration migrations

Context
We want to allow our candidates to login into apply with one login. We also want to allow them to recover their account if their one login email is different from the candidate's email.
These are all the migrations we need for our one login integration, to allow a candidate to login or recover their account.
This is based on the one login spike
Changes proposed in this pull request
We need to allow the user to reclaim their account with any code valid in the last 1 hour for example. That's why AccountRecoveryRequest has_many AccountRecoveryRequestCodes
erDiagram
    Candidate ||--o{ OmniAuthLogin : has_one
    Candidate ||--o{ AccountRecoveryRequest : has_one
    Candidate {
      string email_address
      boolean recovered
      boolean dismiss_recovery
    }
    AccountRecoveryRequest ||--o{ AccountRecoveryRequestCode : has_many
    OmniAuthLogin {
      string token
      string email
      id candidate_id
    }
    AccountRecoveryRequest {
      string previous_account_email
      id candidate
    }
    AccountRecoveryRequestCode {
      string hashed_code
      id account_recovery_request_id
      datetime created_at
    }

Guidance to review
Does it make sense?
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] If this code adds a column that may include PII, the sanitise.sql script and 0025-protecting-personal-data-in-production-dump.md ADR have been updated.
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[x] Inform data insights team due to database changes
[x] Make sure all information from the Trello card is in here
[x] Rebased main
[x] Cleaned commit history
[x] Tested by running locally
[x] Add PR link to Trello card

 Add one login migrations and feature flag


One click duplicate for Runbooks and Reactions

Provide a duplicate button that copies all the values of a current Runbook / Reaction and opens the editor to adjust those values. Perfect for when you want to change a variable if a service is down and restore it when it's back up.

Agreed! In fact we also have a few open issues that this is really the parent of.

[ ] #333
[ ] #332
[ ] #377

I think it would make sense to do a copy for monitors and reactions separately. Mainly because Reactions can be attached to multiple Runbooks. I think the export/import would be the best for tackling a situation where the whole Runbook needs to be tweaked and copied.


Update concepts-messaging.md

The name in the text description does not match the name in the diagram.
https://pulsar.apache.org/docs/en/next/concepts-messaging/#failover
The figure pulsar-failover-subscriptions.png contains only Consumer-B-0 and Consumer-B-1;
https://pulsar.apache.org/docs/en/next/concepts-messaging/#shared
The figure pulsar-shared-subscriptions.png contains only Consumer-C-1, Consumer-C-2 and Consumer-C-3.

run java8 tests

run java8 tests


Path to place the .uasset files. And, is anything else needed?

Hi,
Not sure where to place the .uasset files in the UE5 project folder structure. Should it be in Plugins? In a subfolder in plugins? If so, does the folder need a specific name?  Finally, after I have copied the folders in the indicated place, i will run the python script.. After that do I need to do anything (restart the editor or enable the plugin)??
Thanks Very Much

Hey @nickknyc, first thanks for trying out UnrealGPT.  Since it's an editor utility you just need to place the downloaded UnrealGPT.uasset somewhere inside your content folder.
Open the project where you've placed it in the content folder and find it in your content browser, right click on it, and then click on "Run Editor Utility"
Right now you have to open the blueprint once the first time to enter your api key as a variable but we're pushing an update soon to allow configuring that directly through utility UI though.

@TREE-Ind - thanks that worked for me. But now, I have no idea what a good prompt would look like, so I can't really tell if it's working. Can you help me out? Thanks


The connection to the terminal's pty host process is unresponsive, the terminals may stop working.

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

VS Code Version: 1.67.2
OS Version: Windows 10 & 7

Steps to Reproduce:
After VSCode has been running for a while, this error pops up, whether a terminal is actually launched or not. I've tried to disable ConPTY, without any improvement
"The connection to the terminal's pty host process is unresponsive, the terminals may stop working."

/duplicate https://github.com/microsoft/vscode/issues/130320


How do I do inpainting?

The paper has inpainting examples and the training code also seems to have provision for user-provided background images, but how would I get it working for just test time playing around?

same question

This will be released later

Even I am eagerly waiting for the inpainting. @jiangyzy can you release the inference code for this.


adding IfContains and IfNoContains for collections properties

Codecov Report

Merging #44 (f27d9e3) into main (8722d93) will increase coverage by 0.15%.
The diff coverage is 96.87%.

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   93.87%   94.02%   +0.15%
==========================================
  Files          34       34
  Lines         881      937      +56
==========================================
+ Hits          827      881      +54
- Misses         54       56       +2

Impacted Files
Coverage Δ

src/Validators/Validator.Collections.cs
91.13% <93.93%> (+1.13%)
:arrow_up:

...ions/ValidatableExtensions.CollectionProperties.cs
100.00% <100.00%> (ø)

...bleExtensions/ValidatableExtensions.Collections.cs
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Migrate Azure Storage Blob Changefeed to Test Proxy

Description
Migrates azure-storage-blob-changefeed to Test Proxy and JUnit 5.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

/check-enforcer override


Lint the XML ruleset files and check them for consistent code style

Add checking of the XML files to the build testing as discussed here: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/pull/911#discussion_r109780017
Notes:

Runs only once per build (SNIFF=1)
Checks for valid XML
Checks for consistent codestyle of the XML by doing an auto-format and comparing that against the original file.

I've tested this extensively with deliberately malformed XML and incorrect formatting and am confident this will work as intended.

For clarity, the code style is as follows:

Tabs must be used for indenting.
Self-closing tags must not have a space before the />.

@JDGrimes Yes, thanks, I should have mentioned that.
For prosperity:

Tabs as the rest of the project uses tabs everywhere as well, so that is most inline with the WP codestyle.
No space in self-closing tag: that's just how the auto-formatter is set up and not configurable ;-)
Indentation has to be 1 level for each nesting.

From the second commit msg regarding rule 3 and changes to comply with this codestyle:

The Core ruleset used additional indentation to show which rules belonged with each handbook paragraph, making the ruleset more easily scannable.
As this extra indentation violates the formatting rules, the handbook paragraph headers have been given "fencing" to maintain the scannability of the document.


Automated Package Upgrade

:tada: This PR is included in version 28.2.31 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Recording to 2 files from 2 D415 in parallel

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win10

Platform
PC

SDK Version
2.12.0

Language
C++

Issue Description
I need to record frames from 2 D415 cameras connected to my PC and write the frames to disk. This needs to happen at the same time in parallel to ensure data is captured at the same time. I have run the following code in different threads, using OpenMP/C++11 but I always get unpredictable recorded data.
pipe.stop();

cfg.enable_stream(RS2_STREAM_INFRARED, 1280, 720, RS2_FORMAT_Y8, 30);
cfg.enable_stream(RS2_STREAM_DEPTH, 1280, 720, RS2_FORMAT_Z16, 30);
cfg.enable_stream(RS2_STREAM_COLOR, 1280, 720, RS2_FORMAT_RGB8, 30);

// create ros .bag file on disk for recording
cfg.enable_record_to_file(filename);

// start pipeline of device with desired config
pipe.start(cfg);

Since threads will often block each other it will lead to unequal recorded data.
However, the Realsense Viewer is able to record data from multiple cams very well. Can someone help me how this is done in the Viewer?

Im not sure I understand, are you trying to record to the same file?

No. Recording to two different files.

The Viewer is recording using the rs::recorder instead of using a pipeline. That's probably the main difference, not sure how can your threads block each other, if they use different instances. In fact, why are you using more than a single thread?

I tried to use rs2::recorder, however it only creates a 5kb file but doesn't write any data to the file. This is the code I have. What am I missing here? Thanks for taking the time to reply.
std::shared_ptr<rs2::recorder> _recorder;
_recorder = std::make_shared<rs2::recorder>(filename1, view_port1.device);
Sleep(10000);
_recorder.reset();

Did you call enable_stream and start on view_port1.device?
Regarding the multiple threads part, well you would also get some delay between the two. I don't think you should worry about recording at the same time, but more about how to start the two playbacks at the same delay as you have between starting record of one device and the other.
If you need "perfect" timing, you might want to think of using a recorder on a software_device you create which you will pass all frames from both devices, this way you will get a single file containing frames from both devices, but will lose part of the data per device (like options and info)

pipe1.stop();
cfg1.enable_stream(RS2_STREAM_INFRARED, 1280, 720, RS2_FORMAT_Y8, 30);
cfg1.enable_stream(RS2_STREAM_DEPTH, 1280, 720, RS2_FORMAT_Z16, 30);
cfg1.enable_stream(RS2_STREAM_COLOR, 1280, 720, RS2_FORMAT_RGB8, 30);
pipe1.start(cfg1);
_recorder = std::make_shared<rs2::recorder>(filename1, view_port1.device);
Sleep(10000);
_recorder.reset();

This is the code I'm running. It still outputs a 5KB file.
I hadn't thought of using a software_device. I could try that but any idea why the above code doesn't work?

Not sure why it doesn't work, can you try the following first: (since i'm not sure where view_port1.device is from):
pipe1.stop();
cfg1.enable_stream(RS2_STREAM_INFRARED, 1280, 720, RS2_FORMAT_Y8, 30);
cfg1.enable_stream(RS2_STREAM_DEPTH, 1280, 720, RS2_FORMAT_Z16, 30);
cfg1.enable_stream(RS2_STREAM_COLOR, 1280, 720, RS2_FORMAT_RGB8, 30);
auto profile1 = pipe1.start(cfg1);
_recorder = std::make_sharedrs2::recorder(filename1, profile1.get_device());
Sleep(10000);
_recorder.reset();

Your code worked. Thanks :)
But, it doesn't work if I store the "rs2::device device_" as a private global variable in the constructor and use this variable in _recorder = std::make_shared<rs2::recorder>(filename1, device_);
It seems very strange.

Sorry for repeating myself but since you only share bits of you code and logic at a time I can't know for sure what it is you are doing wrong, or doing right and where the bug is.
I don't see an issue with storing a rs2::device as a private member. Many of the samples use a this as a "global" (in the sample it's just on the main stack so not really global) variable.
The main thing you should understand is that rs2::device acts more as a shared pointer than a value type, and that the pipeline is free to choose a device according the the demands in the config object passed to it (or any device is not config is passed). The only time you know which device the pipeline is using is between pipeline.start() until pipeline.stop(), and you can get this device the either by using the returned profile from start() or by querying for pipline.get_active_profile().get_device() (not sure about the API, but something like it).

That's very helpful. Thanks for taking the time @zivsha!

I'm still struggling with this.
This is the code I'm using -
File1.cpp
int main()
{
---
  rs_device_custom connected_devices;
  rs2::device_list devices = ctx.query_devices();
  connected_devices.record_device(devices[0]);
---
}

File2.cpp
void rs_device_custom::record_device(rs2::device device)
{
	std::string serial_number(device.get_info(RS2_CAMERA_INFO_SERIAL_NUMBER));

	rs2::pipeline pipeline;
	rs2::config cfg;
	std::shared_ptr<rs2::recorder> _recorder;

	std::string dir = "D:/Workspace/Intel Realsense/proj/data/streams/";
	std::string filename;
	std::string date_time = tv_utils::currentDateTime();

	filename = dir + serial_number + "_" + date_time + std::string(".bag");

	cfg.enable_stream(RS2_STREAM_INFRARED, 1280, 720, RS2_FORMAT_Y8, 30);
	cfg.enable_stream(RS2_STREAM_DEPTH, 1280, 720, RS2_FORMAT_Z16, 30);
	cfg.enable_stream(RS2_STREAM_COLOR, 1280, 720, RS2_FORMAT_RGB8, 30);
	cfg.enable_device(serial_number);

	auto profile = pipeline.start(cfg);

	_recorder = std::make_shared<rs2::recorder>(filename, device);
	Sleep(10000);
	_recorder.reset();
}

This code does not work.
However, if I change this line from
_recorder = std::make_shared<rs2::recorder>(filename, device);
to
_recorder = std::make_shared<rs2::recorder>(filename, profile.get_device()); the file is recorded to disk.
If I just use the function parameter device, the file recorded is always a 5kb file.
What am I doing wrong here? Sorry if I'm missing something obvious.

I also have another question.
You said I can the current device using the API profile.get_device(). This requires me to use the pipeline workflow.
If I'm not using a pipeline to stream frames from the device, and instead I use
sensor.start([&](rs2::frame f) {  });, how do I get the current device?

Regarding the first question, I'm not sure if this is a bug or expected (yet not obvious) behavior. The pipeline is probably choosing the same underlying device, but I'm guessing that a different instance of a device is used by the pipeline which probably causes some of the device's state to be missing in the device instance you pass to the recorder. I would leave this one for the LibRS team to look at and decide if it should be fixed or if it is even a bug(@dorodnic ).
Anyways, if you are using the pipeline you do not need the rs2::recorder and you can use cfg.enable_record_to_file(filename);. I only mentioned it since the Viewer is using it but the viewer is not using the pipeline either. If you are still having issue with this one, please share the code that fails and we'll see about that.
If you are not using the pipeline, you are using rs2::context which allows you to query rs2::device which allows you to query its rs2::sensors. See the sensor-control sample on how to use it.

Okay, sure! Look forward to some clarity on the first question. Thanks for everything else!

[Realsense Customer Engineering Team Comment]
@mutp
The main difference is the actual binding between user configuration and the underlying hardware that satisfies that.
When the configuration is passed onto pipeline, the pipeline resolves it internally and assign a HW camera to handle it.
Retrieving device from pipeline’s profile will guarantee device validity.
From your code, you should also check if the device sent from external is the same with the one you configured with serial number.
1- Requested the pipeline to mach a specific device with serial number:
…
cfg.enable_device(serial_number);
auto profile = pipeline.start(cfg);
2- Then tried  recording rosbag from a device received externally
_recorder = std::make_sharedrs2::recorder(filename, device);
There is no check that those two device are actually the same.


Support for meta_struct payloads in traces

Motivation
Support new data format. DataDog/dd-trace-py#8474
Changes
Support for msgpack in msgpack.
Decoded payload  (meta is collapsed in this screenshot)

Add support for appsec json structure in meta struct in all tests. Tracers moving the data from meta to meta struct should not see any changes in system tests.
Tested with https://github.com/DataDog/dd-trace-py/pull/8474, both on CI and locally with all appsec tests
Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present
[ ] To R&P team: locally build and push the image to hub.docker.com

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team
[ ] Once merged, add (or remove) it in system-test-dasboard nightly

Can be merged after peer testing scheduled for Friday 23 Feb pm, if everything's ok.


Update workflows

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

Project coverage is 100.00%. Comparing base (c71441b) to head (57db948).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #102   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           39        39
  Branches         4         4
=========================================
  Hits            39        39

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


Mac: Bash Functional Test Hang

Add '''s to directory being accessed
Previously we were trying to delete a directory with a space indefinitely.  By adding 's around the directory it is deleted successfully.  While we're in here ensure all paths are in 's.
Resolves #554

Please set the appropriate labels and milestone before merging


Some changes in client (cookie jar)

Please, remake Session struct, now in Session "CookieJar" field is struct, remake to interface (in fhttp we have http.CookieJar struct, but type of CookieJar is cookiejar.Jar (is struct))
Thanks!

And sorry for issue, i cannot contribute repo at this moment because forget password from account :(


Export / Import features

Hello,
It's possible to add an export / import feature to bind9 or csv format ?
Some powedns admin tools provides this functionnality

Hi,
This would be easily achievable by adding the native export button to Datatable.
Eg : https://datatables.net/extensions/buttons/examples/html5/simple.html
With it, it's possible to export in CSV.
Regards,

I'd love import/export to/from BIND9 format. IMO this is the only major feature that's missing from PowerDNS-Admin, and it exists in other web frontends like nsedit and Opera DNS UI.

I love PowerDNS-Admin but this is a major feature that cannot be missing.
Is it in roadmap?

In the powerdns API there is the export feature :
/api/v1/servers/server/zones/zone_id/export
Can you add it ?
It's great to have an Import / Export feature

I love PowerDNS-Admin but this is a major feature that cannot be missing.
Is it in roadmap?

I strong agree with Simone.    Import/Export is a primary feature !

Hello,
This request is stale for too long. Regarding our policy to cleanup this project, we are closing it. As this is a Feature Request, it will be listed in #1178
Regards
PowerDNS Admin Team
See #1153 for the policy


[BEAM-6527] Use Gradle to parallel Python tox tests

This PR use Gradle to parallel Python 2 tox, Python 3 tox and DataflowRunner integration tests by splitting them in different Gradle sub-projects:
beam-sdks-python:preCommitPy2
beam-sdks-python-precommit-py3:preCommitPy3
beam-sdks-python-precommit-dataflow:preCommitIT
Note: cython tests should't run together with other tests. We can start it only after other tox tests are finished.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] 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.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

One of tox test keep failing due to error: [Errno 2] No such file or directory: 'apache-beam-2.11.0.dev0/apache_beam/runners/common_test.py'. This happens during tox packaging the distribution though the time it happens may slightly different in each run.
The build execute multiple tox commands in parallel so it's possibly a race condition. However, I set toxworkdir in tox.ini based on name of the task to avoid shared directory among builds, but it's likely I missed something else.
Triggered this test few tests and consistently failed regardless of beam nodes. Out of idea how to move forward.

+R: @tvalentyn @charlesccychen
+cc: @RobbeSneyders @Juta

PTAL @tvalentyn

Run Python PreCommit

Is this ready for review, @markflyhigh ?

Yes, please take a look. @tvalentyn

R: @aaltay

Thank you!

BTW, the reason why I choose copy SDK sources to isolate build is explained in https://github.com/apache/beam/pull/7793#issuecomment-462495172.


Question: Is there a correct way to add a total or summary row?

Hello,
I'm trying to add a total row by adding up values from the results, something like this in my class
  def results
    original_results = super
    total_value = original_results.rows.map {|row| row.last.to_f}.sum.round(2)
    original_results.rows << ['Total', total_value.to_s]
    original_results
  end

This somewhat works however now my report has 2 additional rows:
Total	4546.1
Total	9092.2
The first one is correct, the second is exactly double. Is there a recommended way of adding a total or summary row? Thanks for the help.

In case anyone faces this issue, the problem seems to be that report.results gets called 4 times in the report show, which adds a row each time and recalculates the sum for all previous rows. I found two ways to get around the issue:

Call report.results only once in the report show, save the result in a temp variable - res = report.results, and use that to render the report.
Use an if statement to stop adding rows once the first one was added

  def results
    original_results = super
    unless original_results.rows.last[0] == 'Total'
      total_value = original_results.rows.map {|row| row.last.to_f}.sum.round(2)
      original_results.rows << ['Total', total_value.to_s]
    end
    original_results
  end

Neither of those two solutions seems particularly elegant. I tried using report.results.body and report.results.footers to do the calculation instead but that didn't work since the gem automatically assigns the last N rows as footers then puts whatever is left in the body.
It could be worth considering a way to add N empty footers which the user can then fill as needed instead of splitting one dataset between body and footers.


ra3 release note

adjusted formatting and small tweaks
Julia's release note request here: https://www.notion.so/dbtlabs/d8ad3a7636204c5dba5dea7813c1136e?v=f482c97c0f4e4ef2b469325cfa0fc4bd&p=490cfdbc10704b43b7a4804b92d55270&pm=s
Description & motivation

To-do before merge

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects

should it be "now support" vs. "now supports"? Maybe there's a better way to phrase this altogether.

should it be "now support" vs. "now supports"? Maybe there's a better way to phrase this altogether.

good callout @schottj ! I've rephrased it here so it's more targeted:

Cross-database queries for RA3 instances are now supported by dbt Cloud projects that use a Redshift connection.


Webpack discouragement in readme

Why webpack would be discouraged as a bundling tool?
zxcvbn can be simply put into separate bundle and loaded on demand.
That's the magic of webpack :)

Hi @karolmie1, definitely, but the question is -- bundled with what?
The README only aims to discourage bundling it with site-wide JS that gets loaded everywhere. If you do already have a separate bundle for pages that contain password flows, by all means, webpack is a great choice.


Refactor automated a11y scans

✨ Refactor: A11y scans
The A11y scan runs automatically after each push or manually.
That should be changed, because we need to await the deployment process, before running the a11y tests.
Additionally the paths that shall be tested need to be listed in tests/a11y.spec.ts. With the new blog feature, that should be automated, to include the latest blog posts as well.
Acceptance criteria

A11y scans should include all deployed paths automatically
The A11y scans should be run after each successful deployment

As for now, I have disabled the A11y tests for svg elements since https://github.com/Niels-IO/next-image-export-optimizer is not creating a unique id for every blur filter. This needs to be addressed upstream.
See https://github.com/porscheofficial/porscheofficial.github.io/commit/9a124e4103c60c23572d12ed3ff25786f06c010f#diff-f5fca1f01ef788e9a4af430797533798b8352aef23031a49446e0beae8361112R17


Extension is not immediately usable when using the binder link (rebuild is required)

Actual behavior
When opening up the binder link, a build is recommended, and our extension is not immediately usable.
Expected behavior
When opening up the binder link, the extension is immediately usable.
To Reproduce
Steps to reproduce the behavior:

Open up our binder link
A JupterLab build is suggested

Screenshot

Desktop (please complete the following information):

OS: iOS
Browser: Chrome
Version: [85.0.4183.102

Additional context
N/A

Not requiring a rebuild on my laptop anymore. Closing for now.


Tool tips for main functions are broken

The tooltips for StyleManager, BlockManager, LayerManager, Component Manager, seem not to be working with this preset. They are broken in the demo as well.
Also, with my implementation, I'm not getting the fancy tooltips, just plain black-on-grey ones.
Am I doing something wrong?

I found that tooltip.css was not included in the preset, so I just copied it out of the demo into my implementation, as well as setting the data-tooltip-pos="bottom" attribute on all titles.


Update dependencies: eslint, eslint-plugin-jsdoc, snyk, stylelint, stylelint-config-prettier, esbuild, jest, jest-cli, jest-environment-jsdom

Changes

Update dependencies: eslint, eslint-plugin-jsdoc, snyk, stylelint, stylelint-config-prettier, esbuild, jest, jest-cli, jest-environment-jsdom
Remove declaration-colon-newline-after deprecated rule from stylelint config.
Refresh npm cache.

How to test this PR

yarn build should pass.
yarn jest should pass.
yarn lint should pass with deprecation warnings.

I believe I'm seeing more than just deprecation warnings. Is this expected?

I believe I'm seeing more than just deprecation warnings. Is this expected?

Yes these are existing warnings. You'll see them on yarn lint on main branch.


StartWithSrpAuthAsync fails with a parse exception

I have a cognito user pool built using the email as a sign-in alias. When I use<EMAIL_ADDRESS>as a username using StartWithSrpAuthAsync, the library throws an exception "The value could not be parsed."
I narrowed this down to the fact the salt string is a negative hex value, which the lib is assuming is always unsigned.  This error is occuring in AuthenticationHelper.cs:
var salt = BigIntegerExtensions.FromLittleEndianHex(saltString);
Is it possible to support signed salt values here?  I tried this in nodejs Amplify and they seem to handle it w/o an issue.

OK I guess this happens when trying to initiateAuth using an email which hasn't been verified yet.
The fix for me is to specify email_verified = true on user creation, then the SRP authentication works as intended.
Not sure if any action needs to be taken to handle that particular case, otherwise I can close this

Thanks for the feedback.  I'll look into whether we can use the signed salt values.  I don't want to recommend setting email_verified = true unless it has been verified or isn't critical for you.

Getting the same issue. However, do not want to set email_verified as true as I need to log in to obtain a session from which I can call ResponseToNewPasswordRequiredAsync in order to change the temporary password (and confirm the email)

I am getting the same issue when a non-existent username is used. I was expecting to get a UserNotFoundException in this case. Here is what I'm doing:
`private async Task LoginUser(string username, string password)
{
var provider = new AmazonCognitoIdentityProviderClient(new AnonymousAWSCredentials(), REGION_ENDPOINT);
        var userPool = new CognitoUserPool(POOL_ID, CLIENT_APP_ID, provider);

        var user = new CognitoUser(username, CLIENT_APP_ID, userPool, provider);
        var authRequest = new InitiateSrpAuthRequest()
        {
            Password = password
        };

        var authResponse = await user.StartWithSrpAuthAsync(authRequest).ConfigureAwait(false);

`
Is this incorrect?

Having the same issue where I use Email as UserName for not activated accounts. This way I can't response to challenge and require to validate Email by providing validation code in next step.

I have email_verified as true and I'm getting the same issue

Getting this issue as well on a user that has been verified and has been able to log in prior.
Stack trace below
at System.Numerics.BigNumber.ParseBigInteger (System.ReadOnlySpan'1[T] value, System.Globalization.NumberStyles style, System.Globalization.NumberFormatInfo info) [0x0001e] in /Library/Frameworks/Xamarin.iOS.framework/Versions/Current/src/Xamarin.iOS/external/corefx/src/System.Runtime.Numerics/src/System/Numerics/BigNumber.cs:386 \n  at System.Numerics.BigNumber.ParseBigInteger (System.String value, System.Globalization.NumberStyles style, System.Globalization.NumberFormatInfo info) [0x0000e] in /Library/Frameworks/Xamarin.iOS.framework/Versions/Current/src/Xamarin.iOS/external/corefx/src/System.Runtime.Numerics/src/System/Numerics/BigNumber.cs:374 \n  at System.Numerics.BigInteger.Parse (System.String value, System.Globalization.NumberStyles style, System.IFormatProvider provider) [0x00000] in /Library/Frameworks/Xamarin.iOS.framework/Versions/Current/src/Xamarin.iOS/external/corefx/src/System.Runtime.Numerics/src/System/Numerics/BigInteger.cs:675 \n  at System.Numerics.BigInteger.Parse (System.String value, System.Globalization.NumberStyles style) [0x00000] in /Library/Frameworks/Xamarin.iOS.framework/Versions/Current/src/Xamarin.iOS/external/corefx/src/System.Runtime.Numerics/src/System/Numerics/BigInteger.cs:665 \n  at Amazon.Extensions.CognitoAuthentication.Util.BigIntegerExtensions.FromUnsignedLittleEndianHex (System.String hex) [0x0000b] in <bfb077a8c7a341ef900f115d002a1c51>:0 \n  at Amazon.Extensions.CognitoAuthentication.Util.AuthenticationHelper.AuthenticateUser (System.String username, System.String password, System.String poolName, System.Tuple2[T1,T2] tupleAa, System.String saltString, System.String srpbString, System.String secretBlockBase64, System.String formattedTimestamp) [0x0003b] in :0 \n  at Amazon.Extensions.CognitoAuthentication.CognitoUser.CreateSrpPasswordVerifierAuthRequest (Amazon.CognitoIdentityProvider.Model.InitiateAuthResponse challenge, System.String password, System.Tuple2[T1,T2] tupleAa) [0x000a8] in <bfb077a8c7a341ef900f115d002a1c51>:0 \n  at Amazon.Extensions.CognitoAuthentication.CognitoUser.StartWithSrpAuthAsync (Amazon.Extensions.CognitoAuthentication.InitiateSrpAuthRequest srpRequest) [0x000f2] in <bfb077a8c7a341ef900f115d002a1c51>:0 \n  <ommited project specific stack>

Hi @timcromarty / @dtarczynski / @vazquezbonilla / @DevonHansen,
Good afternoon.
Please review the above comments and let me know if this is still an issue. Else, since the issue is not reproducible, we could close this issue.
Thanks,
Ashish

This seems to have fixed itself for me overnight. I don't manage our Cognito instance so can't say for certain if something has changed there.
I was previously testing with<EMAIL_ADDRESS>and password and getting the FormatException, but now getting NotAuthorizedException as you'd expect.

I'll look to see if I can replicate sometime today. We have a workaround where we assume that FormatException is a reason to not log the person in for invalid details.


Navbar disapper during tutorial guide

Fixes #2251
Describe the changes you have made in this PR -

Moved the quick button outside the Navbar div. (Since whenever quick button got selected, navbar also got highlighted)
Changed the navbar position property from relative to inherit to avoid it moving.
Entire navbar to be highlighted while tutorial of Help drop-down is shown.

Screenshots of the changes (If any) -

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.731% when pulling c57d43cc680c6ee08735027ca4eacfddb46cea2e on devartstar:NavbarDisapperDuringTutorialGuide into 4c1e0050bf03e047ab97fdad003e1fe12800a00e on CircuitVerse:master.

Hi @satu0king @abstrekt ...
Can you please review this Pull Request.


CMake/preprocess: fix typo PREPROCES -> PREPROCESS

Summary
CMake/preprocess: fix typo PREPROCES -> PREPROCESS
correct the marco define from PREPROCES to PREPROCESS
Signed-off-by: chao an<EMAIL_ADDRESS>Impact
N/A
Testing
ci-check

[Experimental Bot, please feedback here]
Yes, the PR appears to meet the NuttX requirements.
Justification:

Summary: Clear and concise, explaining the change (typo fix), its location (CMake/preprocess), and how it works (correcting the macro definition).
Impact:  Accurately identifies the change as having no user, build, hardware, documentation, security, or compatibility impact (N/A).
Testing: Indicates testing was performed via CI ("ci-check"). While providing actual logs would be ideal, the presence of CI checks suggests a level of automated testing.

Recommendation:

Include snippets of the testing logs (before and after) to provide concrete evidence of the fix. This significantly strengthens the PR.


Mark release<PHONE_NUMBER>48 as latest

This is a minor change but it would be nice to mark release<PHONE_NUMBER>48 as latest. The only reason for this is then various scripts can download it from https://github.com/tlaplus/tlapm/releases/latest instead of having to store and type out the version number. Good preparation for rolling releases.

@muenchnerkindl, can we consider this release as the "best"? If yes, I can mark it as the latest.

@kape1395 Sounds reasonable to me.


Add codeclimate/eslint integration

Add codeclimate/eslint integration

:white_check_mark: coverage increased from 0% to 74.83%

:white_check_mark: coverage increased from 0% to 75.33%


question: What is "config.lazy"

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.5
Operating system/version
Ubuntu 22.04.3
Describe the bug
My stock install comes with a single line in init.lua:
require("config.lazy")
I have scoured the documentation and the subreddit and cannot find "config.lazy" mentioned anywhere.  Where is this file?
Steps To Reproduce

Install LazyVim
Open .config/nvim/init.lua

Expected Behavior
Finding the config file
Repro
-- bootstrap lazy.nvim, LazyVim and your plugins
require("config.lazy")

Decided to go back to vanilla, I'm too lazy for lazyvim

@CoeJoder 1. this is lazy.nvim and you should be looking into the lazyvim/lazyvim repo
2. this is just a lua module
this means it's either in the lua/config/lazy.lua or lua/config/lazy/init.lua file

@max397574 Thank you


Null hypothesis of Kolmogorov Smirnov test is not correctly described

The statement

Under the null hypothesis, the two distributions are identical, F(x)=G(x). The alternative hypothesis can be either ‘two-sided’ (default), ‘less’ or ‘greater’.

in the documentation of stats.kstest (https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.kstest.html) is not correct unless alternative == 'two-sided'. The other alternatives test F >= G vs F < G and vice versa.
Same problem for stats.ks_2samp (https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.ks_2samp.html#scipy.stats.ks_2samp):

This is a two-sided test for the null hypothesis that 2 independent samples are drawn from the same continuous distribution.

and stats.ks_1samp (https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.ks_1samp.html):

Under the null hypothesis, the two distributions are identical, F(x)=G(x).

The alternatives are correctly described in the R documentation:
https://stat.ethz.ch/R-manual/R-patched/library/stats/html/ks.test.html

I'm not sure about this,
The null hypothesis is always F(x)=G(x), only the alternative changes in one sided test.
By monotonicity, this usually implies F(x)<=G(x). However, the test distribution is computed under the assumption that F(x)=G(x).
(The distinction is more important in likelihood ratio and score tests.)

In case it helps...
From stats.ks_2samp:

We generally follow Hodges’ treatment of Drion/Gnedenko/Korolyuk [1].

Hodges, J.L. Jr., “The Significance Probability of the Smirnov Two-Sample Test,” Arkiv fiur Matematik, 3, No. 43 (1958), 469-86.

...which is available at https://doi.org/10.1007/bf02589501

This acknowledges the fact that sometimes the null hypothesis is written differently depending on the alternative, but that writing the null hypothesis the same way in all cases is also acceptable.

However, be aware that many researchers (including one of the co-authors in research work) use = in the null hypothesis, even with > or < as the symbol in the alternative hypothesis. This practice is acceptable because we only make the decision to reject or not reject the null hypothesis.

I favor the way it is currently written, as I think the distribution we used to determine the p-value should be derived from the null hypothesis (as it is stated; not from some variation.). For the KS test, does the distribution used to calculate p-values depend on the alternative hypothesis being tested? If not, I think that the way the null hypothesis is written is correct regardless of the alternative.

@chrisb83 BTW would you still like to change these, or do you think they are ok?

@mdhaber I would be in favour of being more precise in the documentation, following the R doc. I just had a look at the document you shared:

However, be aware that many researchers (including one of the co-authors in research work) use = in the null hypothesis, even with > or < as the symbol in the alternative hypothesis. This practice is acceptable because we only make the decision to reject or not reject the null hypothesis.

that makes sense. however, i think there are often situations where practitioners assume H0 holds unless it is rejected. in that case, it is important to interpret H0 correctly.

favour of being more precise

H0: F = G is precise
the p-value is computed under this assumption
If F < G, then the true pvalue would be larger. i.e. if we have a joint null hypothesis, then we have to argue with max or min pvalue in the subset of the parameter space under the null.
In likelihood ratio and score tests, the test results might not be the same for H0: a <= a0 and H0: a = a0. So you couldn't make an argument there that would be consistent with KS tests or simple t-tests which are monotonic.
I'm not sure about KS test with estimated parameters.

practitioners assume H0 holds unless it is rejected

that's an argument for babysitting and not for being precise.
maybe and explanatory sentence would be useful
If the tests does not reject for alternative F > G against null F = G, then it would also not reject against F < G.

I just realized/remembered that the KS test is a functional hypothesis, and the single parameter analogue doesn't work
F >= G vs F < G  are not complements of each other.
F and G can intersect, in that case we also don't reject, so the opposite of F > G is not F > G which is an even weirder null.

Actually I don't remember how inequalities are defined in onesided ks test, but any ordering has to allow for intersections of the cdfs for cases not F=G, whether they are included in some null or alternative.

the intersection case is in the alternative, there needs to be an "at least for some range"
https://stats.stackexchange.com/questions/107668/does-it-make-sense-to-perform-a-one-tailed-kolmogorov-smirnov-test
H0:F_Y(t)≥F_X(t)
H1:F_Y(t)<F_X(t), for at least one t
quote "You could in some cases write the null as an equality" but then uses weak inequality
(I got suspicious with my initial intuition for the functional case when looking at the confidence band implied by KS-test. In the one-sided case the ecdf has to be completely on one side (the inside side) of one confidence band boundary to not reject the null. If at least a small piece of the ecdf crosses or is on the outside of the confidence band, then we reject.)
So that still leaves it open whether null is specified as equality or weak inequality.

going back to this

practitioners assume H0 holds unless it is rejected

The main problem is that some practitioners do this also when their sample size is too small and the test doesn't have much power.
If the sample size is not large, then we would also NOT reject if the ecdf or true cdf wiggles around the null cdf, or in two sample case if the two cdfs wiggle around each other.
In that case we have neither F <=G nor F >= G. If we don't reject, then we can only say that there are no big deviations in parts of the cdf in the alternative direction.

yes, I also checked in the stats book by Conover. The one-sided case is
H0: F(x) <= G(x) for all x
H1: F(x) > G(x) for at least one x

sounds fine to me then to use the last version with weak inequality in null.
I'm sticking to equality null in statsmodels, because I don't want to get into composite nulls when we don't need or use it.


sasjs login to sas fails when userLongName = unknown

I'm using sasjs to connect my webapp and SAS and I recently started facing an issue.
When I open my Web App, if I'm not connected the sasjs login returns userLongName = unknown as displayed in the picture below, while userName is empty

The fact that userLongName is not empty makes sasjs think that I'm connected and it does not appear the login mask.
As a side note, if I login in another SAS app and then open my webapp, sasjs is able to correctly recognize my user

Thanks for raising @saramartinelli1992 - we'll investigate this asap

@saramartinelli1992 which version of the adapter are you using? The latest version is 4.2.1. If you are not using the latest, please update the version and check if the problem persists.

@sabhas I was using a previous version, I now tested the lates one and I see the window for login, but it shows the following

Following a call with @saramartinelli1992 we discovered that SAS9 M8 no longer redirects from SASLogon/home (but it does redirect in SAS9 M7, also Viya 4)
We did some tests and it seems the logon is consistent across versions when just using /SASLogon, we also did a build and tested this change on M8 and that worked fine.  PR to follow.
Relevant commits:

https://github.com/sasjs/adapter/commit/d9c27efa8d5aa81dd58126141ba63b32b218a1d4
https://github.com/sasjs/adapter/commit/830a907bd1c6546ed74fa969546b37681cc7967f

@all-contributors please add @saramartinelli1992 for UserTesting and Platform


IntelliSense: Some format problems of the new version of vscode (1.6.0)

When I upgraded the vscode to the new version (version 1.6.0), there are some problems of IntelliSense shown in the following screenshots.

VSCode Version: 1.6.0
OS Version: Ubuntu 14.04

Problems:

When writing code, if the IntelliSense is too long:

It only happened when writing the code. After written completed, I hover the mouse on the function, the problems didn't occurred.

When hovering the mouse on a function:

When hovering the mouse on a function:

What language are you developing on? What extension do you use for it?

@joaomoreno I develop using Python language with the extension Python by Don.(https://marketplace.visualstudio.com/items?itemName=donjayamanne.python)

I've had a problem similar to 1 using typescript.
Also the hover seems to be limited to the current editor when using split editors. This leads to unfortunate formatting like (also in typescript):

(Let me know it it's better to open a new issue for this)

@CaselIT that's 13004

@joaomoreno Had not found it. Thanks

@Kongsea , @joaomoreno this is an issue (https://github.com/DonJayamanne/pythonVSCode/issues/368) with the Python extension.
The issue (in the Python extension) has been resolved and fix will be in the next release.


Failures in Github Actions after v0.3.8 was released

What was wrong?
Yesterday our CI pipelines started to fail and it seems to have happened just after v0.3.8 was released.
Logs
This is from a private repo but let me know if you need any additional information. The coverage files comes from Python.
 Regex match error: UTF-8 error: byte 2 top bits not 0x80
Coveralls Coverage Reporter v0.3.8
Error: Process completed with exit code 1.

Hey! Thank you for the issue. I am going to fix it as soon as possible. The reason is probably the upgrade of Crystal version (1.7.2 -> 1.8.1, and it uses pcre2, which raises such an error). I'll notify you when I release a fix.

I've just release v0.3.9 with a version downgrade. Please, rerun the failing workflow in your CI. Everything should work correctly now.

Thank you, it's working again!


WFS layers cannot be retrieved from local storage

The features are not loaded back in the map

add a new service with local WFS
add a layer
save
load

"{"layers":[{"type":"Group","properties":{"type":"base-group","title":"Base Maps","opacity":1,"visible":true,"zIndex":0,"id":"sdk-layer-1"},"children":[{"type":"Tile","properties":{"type":"base","title":"OSM Streets","opacity":1,"visible":true,"zIndex":0,"preload":0,"useInterimTilesOnError":true,"id":"sdk-layer-5"},"source":{"type":"OSM","properties":{"attributions":["© OpenStreetMap contributors."]}}},{"type":"Tile","properties":{"type":"base","title":"CartoDB light","visible":false,"opacity":1,"zIndex":0,"preload":0,"useInterimTilesOnError":true,"id":"sdk-layer-4"},"source":{"type":"XYZ","properties":{"attributions":["© OpenStreetMap contributors, © CartoDB"],"urls":["http://s.basemaps.cartocdn.com/light_all/{z}/{x}/{y}.png"]}}},{"type":"Tile","properties":{"type":"base","title":"CartoDB dark","visible":false,"opacity":1,"zIndex":0,"preload":0,"useInterimTilesOnError":true,"id":"sdk-layer-3"},"source":{"type":"XYZ","properties":{"attributions":["© OpenStreetMap contributors, © CartoDB"],"urls":["http://s.basemaps.cartocdn.com/dark_all/{z}/{x}/{y}.png"]}}},{"type":"Tile","properties":{"type":"base","title":"ESRI world imagery","visible":false,"opacity":1,"zIndex":0,"preload":0,"useInterimTilesOnError":true,"id":"sdk-layer-2"},"source":{"type":"XYZ","properties":{"attributions":["Tiles © Esri — Source: Esri, i-cubed, USDA, USGS, AEX, GeoEye, Getmapping, Aerogrid, IGN, IGP, UPR-EGP, and the GIS User Community"],"urls":["http://server.arcgisonline.com/ArcGIS/rest/services/World_Imagery/MapServer/tile/{z}/{y}/{x}"]}}}]},{"type":"Vector","properties":{"title":"USA Population","emptyTitle":false,"id":"topp:states","name":"topp:states","isWFST":true,"isRemovable":true,"isSelectable":true,"popupInfo":"#AllAttributes","url":"/geoserver/wfs","opacity":1,"visible":true,"zIndex":0,"wfsInfo":{"featureNS":"http://www.openplans.org/topp","featurePrefix":"topp","featureType":"states","geometryType":"MultiPolygon","geometryName":"the_geom","attributes":["CARPOOL","DRVALONE","EMPLOYED","FAMILIES","FEMALE","HOUSHOLD","LAND_KM","MALE","MANUAL","P_FEMALE","P_MALE","PERSONS","PUBTRANS","SAMP_POP","SERVICE","STATE_ABBR","STATE_FIPS","STATE_NAME","SUB_REGION","UNEMPLOY","WATER_KM","WORKERS"],"url":"/geoserver/wfs"},"styleName":"population"},"source":{"type":"Vector","properties":{"format":{"type":"GeoJSON"}}}}],"view":{"projection":"EPSG:3857","center":[-10670043.94,4604255.59],"resolution":9783.93962050256,"zoom":4,"rotation":0}}"

This is because the source's url is a function and cannot be serialized directly


replication factor proteolysis or replisome cleavage

Please provide as much information as you can:

name: replisome cleavage

Definition (process replisome substrates during DNA synthesis in an ATP-dependent manner.)

**Reference, in format PMID:38554706,
(REQUIRED)

Gene product name and ID to be annotated to this term

Parent term(s)

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

Any other information

HI @CP-SB
are you requesting an activity, or a process?
I think in this situation the correct MF would be
GO:0004176  ATP-dependent peptidase activity  (or similar)
you would add the replisome substrates as inputs (we don't create terms  activity terms to describe specific substrates)
so your annotation would be something like:
MF GO:0004176  ATP-dependent peptidase activity has_input(gene)|has_input(gene)
part_of GO:0031297 replication fork processing

Thanks Val, will use that recommendation.

Were you annotating  the  activity of the  complex "protein complex composed of AAA+ ATPases SPATA5-SPATA5L1 together with heterodimeric partners C1orf109-CINP (55LCC)." or the protease?
The complex might be better described as a "dissagregase activity"
https://www.ebi.ac.uk/QuickGO/term/GO:0140545
ATP-dependent protein disaggregase activity
An ATP-dependent molecular chaperone activity that mediates the solubilization of ordered protein aggregates. PMID:26312418
but we might need to extend the definition here to include complexes.
Note that this complex appears to have a similar activity during ribosome assembly, where it extracts RSL24D1 from pre-60S subunits
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC9004343/
SPATA5 and SPATA5L1 (Figure S6D), further suggesting that these proteins carry out non-overlapping functions in removing RSL24D1 from pre-60S subunits. Phylogenetic analysis shows that CINP and C1ORF109 are likely related to one another. Orthologs of these proteins are not present in yeast, but CINP-like proteins are found in Drosophila and C. elegans, among other species. Matchmaker alignments of AlphaFold predictions (Meng et al., 2006; Senior et al., 2020) indicate that CINP and C1ORF109 share remarkably similar structure......
So, it should have the same MF term for both processes, coupled to different substrates for the respective biological  processes.

Hi Val, I was trying to annotate the activity of the complex and have tagged it with GO:0140545 but requested the additional term to capture the activity of protease...thinking to separate the processes.

@CP-SB I discussed this one with @pgaudet
We are trying to avoid terms which describe proteases as processes, and historical terms which described proteolysis in this way have been/are being obsoleted
see for example https://github.com/geneontology/go-ontology/issues/28575 GO:0097304 lipoprotein biosynthetic process via signal peptide cleavage
and
GO:0000469   obsolete cleavage involved in rRNA processing |
-- | --
GO:1901308   obsolete regulation of sterol regulatory element binding protein cleavage
GO:0000469   obsolete cleavage involved in rRNA processing
GO:1901308   obsolete regulation of sterol regulatory element binding protein cleavage
so the correct way to annotate this  protease would be
MF GO:0004176  ATP-dependent peptidase activity has_input(gene)|has_input(gene)
part_of GO:0031297 replication fork processing (or replication termination)


Ignore is not respecting in a group dependencies

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
bundler
Package manager version
2.4
Language version
ruby 3.2.2
Manifest location and content before the Dependabot update
No response
dependabot.yml content
version: 2
updates:

package-ecosystem: "bundler"
directory: "/"
schedule:
interval: "daily"
groups:
production-dependencies:
dependency-type: "production"
development-dependencies:
dependency-type: "development"
dependabot.yml-file#ignore
ignore:

dependency-name: "parser"
update-types: ["version-update:semver-major", "version-update:semver-minor"]

Updated dependency
parser
What you expected to see, versus what you actually saw
I want to ignore all major and minor, only accept patches, but it changing my Gemlock to major.
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
https://github.com/AlexcastroDev/dependabot_parse/pull/1

Updates: if I ignore all version, it works, but if I ignore major and minor, dependabot create a PR updating major version

I suspect I have the same issue here with Dependabot-config here


Minimum validation for the date entered on scholastic standing creation

User Story
As an Institution User, I want to be subject to validations on the dates when I can report a scholastic standing event: I should only be able to report scholastic events which fall within a student's study dates.
Acceptance Criteria

[ ] Only allow the date entered during Early Completion and Withdrawal and Switch from FT to PT to be within a student's study dates (actual start and end dates are inclusive within the eligible dates)
[ ] Only allow the date entered during Early Completion and Withdrawal and Switch from FT to PT to be before or equal to today. I.e. an institution cannot report a withdrawal date that is in the future.
[ ] The action of reporting the event itself can occur until application is archived.

Current-State Mockups

Business Context
None

For lather. Hoping to discuss the validation of UC weeks as well and have them have a top end limit of offering weeks for the same reason as the other two changes.


[BUG] Function pointer type variable is recognized as a non-function type by cython.

Describe the bug
I have been testing out numpy's Iterator API and realized that I cannot correctly compile cython files that make use of the NpyIter_IterNextFunc* function pointer type. For some reason cython recognizes a variable assigned this type as a "non-function" type and fails to compile, as a result. I am not sure exactly why this happens and hope to find some pointer here on how to fix this issue?
Cc @seberg
Here is the error message:

Error compiling Cython file:
------------------------------------------------------------
...
    iternext = NpyIter_GetIterNext(it, NULL)
    dataptr = NpyIter_GetDataPtrArray(it)

    while True:
        dataptr[2][0] = do_something(dataptr[0][0], dataptr[1][0])
        if not iternext(it):
                      ^
------------------------------------------------------------

example.pyx:63:23: Calling non-function type 'NpyIter_IterNextFunc'
Traceback (most recent call last):
  File "/home/zoj/.local/share/micromamba/envs/bin/cythonize", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/zoj/.local/share/micromamba/envs/lib/python3.12/site-packages/Cython/Build/Cythonize.py", line 225, in main
    cython_compile(path, options)
  File "/home/zoj/.local/share/micromamba/envs/lib/python3.12/site-packages/Cython/Build/Cythonize.py", line 97, in cython_compile
    ext_modules = cythonize(
                  ^^^^^^^^^^
  File "/home/zoj/.local/share/micromamba/envs/lib/python3.12/site-packages/Cython/Build/Dependencies.py", line 1115, in cythonize
    cythonize_one(*args)
  File "/home/zoj/.local/share/micromamba/envs/lib/python3.12/site-packages/Cython/Build/Dependencies.py", line 1238, in cythonize_one
    raise CompileError(None, pyx_file)
Cython.Compiler.Errors.CompileError: /home/zoj/dev/example.pyx

Code to reproduce the behaviour:
# cython: language_level=3
cimport numpy as np
from cpython.ref cimport Py_INCREF

np.import_array()

cdef extern from "numpy/ndarrayobject.h":
    np.PyArray_Descr* PyArray_DescrFromType(int)

    ctypedef struct NpyIter:
        pass

    cdef enum:
        NPY_FAIL
        NPY_SUCCEED

    cdef enum:
        NPY_ITER_ALLOCATE
        NPY_ITER_READONLY
        NPY_ITER_WRITEONLY
        NPY_ITER_ZEROSIZE_OK

    NpyIter* NpyIter_MultiNew(np.npy_intp nop, np.PyArrayObject** op,
                              np.npy_uint32 flags, np.NPY_ORDER order,
                              np.NPY_CASTING casting, np.npy_uint32* op_flags,
                              np.PyArray_Descr** op_dtypes) except? NULL

    ctypedef int (*NpyIter_IterNextFunc)(NpyIter* it) nogil
    NpyIter_IterNextFunc* NpyIter_GetIterNext(NpyIter* it, char** errmsg)
    char** NpyIter_GetDataPtrArray(NpyIter* it) nogil
    np.PyArrayObject** NpyIter_GetOperandArray(NpyIter* it)
    int NpyIter_Deallocate(NpyIter* it)

cdef double add(double x, double y) nogil:
    cdef double z = x + y
    return z

cdef object broadcasted_add(np.ndarray a, np.ndarray b):
    cdef:
        char** dataptr
        NpyIter* nditer
        NpyIter_IterNextFunc* iternext
        np.npy_uint32* op_flags = [NPY_ITER_READONLY, NPY_ITER_READONLY,
                                   NPY_ITER_WRITEONLY | NPY_ITER_ALLOCATE]
        np.PyArrayObject** ops = [<np.PyArrayObject*>h, <np.PyArrayObject*>z, NULL]

    it = NpyIter_MultiNew(3, &ops[0], NPY_ITER_ZEROSIZE_OK, np.NPY_KEEPORDER,
                          np.NPY_NO_CASTING, &op_flags[0], NULL)

    iternext = NpyIter_GetIterNext(it, NULL)
    dataptr = NpyIter_GetDataPtrArray(it)

    while True:
        dataptr[2][0] = add(dataptr[0][0], dataptr[1][0])
        if not iternext(it):
             break

    out = <np.ndarray>NpyIter_GetOperandArray(it)[2]
    Py_INCREF(out)
    NpyIter_Deallocate(it)
    return out

Expected behaviour
I expected this to compile with no issues but I get the error message shown above.
OS
Linux
Python version
3.12 and 3.9
Cython version
0.29.36 and 3.0.3
Additional context
No response

NpyIter_IterNextFunc* iternext

This defines a pointer to a function pointer. Change it to
NpyIter_IterNextFunc iternext

NpyIter_IterNextFunc* iternext

This defines a pointer to a function pointer. Change it to
NpyIter_IterNextFunc iternext

I'm not sure this solves the issue because now I get:
rror compiling Cython file:
------------------------------------------------------------
...
        np.PyArrayObject** ops = [<np.PyArrayObject*>h, <np.PyArrayObject*>z, NULL]

    it = NpyIter_MultiNew(3, &ops[0], NPY_ITER_ZEROSIZE_OK, np.NPY_KEEPORDER,
                          np.NPY_NO_CASTING, &op_flags[0], NULL)

    iternext = NpyIter_GetIterNext(it, NULL)
                                  ^
------------------------------------------------------------

example.pyx:53:34: Cannot assign type 'NpyIter_IterNextFunc *' to 'NpyIter_IterNextFunc'

NpyIter_GetIterNext returns a NpyIter_IterNextFunc * not NpyIter_IterNextFunc. If I add something like
	cdef:
        NpyIter_IterNextFunc iternext
        NpyIter_IterNextFunc* iternext_ptr
	...
    iternext_ptr = NpyIter_GetIterNext(it, NULL)
    iternext = iternext_ptr[0]

It compiles, but using the code raises this error:
error: subscripted value is pointer to function
       3452 |   __pyx_v_iternext = (__pyx_v_iternext_ptr[0]);

Clearly im doing something wrong here but not sure what. Any more suggestions @da-woods ?

Final comment from me on the issue because this isn't a Cython bug and support questions should be asked on the Cython user's mailing list instead
This looks to be to do with how NpyIter_IterNextFunc is defined in Numpy which doesn't match your Cython definition:
https://github.com/numpy/numpy/blob/972c06c78cba81875ceb0aa7f5b14dcb19cce58b/numpy/core/include/numpy/ndarraytypes.h#L1004
typedef int (NpyIter_IterNextFunc)(NpyIter *iter);

The definition here isn't a pointer - and thus in the C code NpyIter_IterNextFunc* is a function pointer.
However in your Cython definition you define it as a pointer
    ctypedef int (*NpyIter_IterNextFunc)(NpyIter* it) nogil

And thus as far as Cython knows NpyIter_IterNextFunc* is a pointer to a function pointer. You need to make sure the two match.
That may involve defining a non-extern type
ctypedef int (*NpyIter_IterNextFunc_ptr)(NpyIter* it) nogil

and then using that type in your Cython code instead.

Quick clarification (since I think my previous comment was a little vague on the actual solution):
You definitely can't do a "non-pointer function type" in Cython, so can't declare NpyIter_IterNextFunc exactly as it appears in the Numpy headers.
Therefore you do want to define your own function pointer typedef outside the extern block to use in Cython.
So to spell it out more fully:
ctypedef int (*NpyIter_IterNextFunc_ptr)(NpyIter* it) nogil

cdef extern from "numpy/ndarrayobject.h":
     # Note that this definition isn't exactly the same as the Numpy header, but that's fine - it returns the same type under
     # a typedef name
    NpyIter_IterNextFunc_ptr NpyIter_GetIterNext(NpyIter* it, char** errmsg)

    # ...

cdef object broadcasted_add(np.ndarray a, np.ndarray b):
    cdef:
        NpyIter_IterNextFunc_ptr iternext
    # ...
    iternext = NpyIter_GetIterNext(it, NULL)
    # ...

The inability to declare NpyIter_IterNextFunc as it appears in the Numpy headers is probably a bit of an omission from Cython, but  it's a slightly odd way of writing C code.

Your suggestion worked very well, thanks @da-woods


Web worker for search

No more tab freezes!
Will fix #40
The searching process has been moved to a web worker script, it will work in a separate thread.

@Eboubaker any progress here? 😄

@connorhsm Test and merge

Good that you've tested. it was indeed not working, it should be fixed now.

Thank you! ❤


Add useStateWithCallback

Is your feature request related to a problem? Please describe.
It is often desired to have a callback after useState state mutates.
Describe the solution you'd like
Add a callback to useState.
Describe alternatives you've considered
const useStateWithCallback = (defaultState, callback) => {
  const [currentState, setState] = useState(defaultState);

  useUpdateEffect(() => {
    callback(currentState);
  }, [
    currentState,
  ]);

  return [currentState, setState];
};

Example:
const [dateRange, setDateRange] = useStateWithCallback({
  from: undefined,
  to: undefined,
  ...props.dateRange,
}, (newDateRange) => {
  props.onChange(newDateRange);
});

Maybe it could be called useStateEffect or useStateWithEffect, as that callback is basically React effect.

Maybe it could be called useStateEffect or useStateWithEffect, as that callback is basically React effect.

I wouldn't rush with that name or that implementation. It depends where you put the callback, e.g. This might work just as well:
const useStateWithCallback = (defaultState, callback) => {
  const [currentState, setState] = useState(defaultState);

  return [
    currentState,
    (newState) => {
      callback(newState);

      setState(newState);
    },
  ];
};

why not to simply use
const [state, setStaet] = React.useState({});
React.useEffect(()=>{
// anything you want
}, [state]);

it is way more controllable;

why not to simply use

That entirely depends on what you want the hook to do, as illustrated in my previous comment. I do not recommend any specific implementation. As a consumer, I simply (often) have a use case for a callback straight after useState setter is used – as such, having it as a callback makes sense.
In fact, useLayoutEffect might make more sense if the intention is to batch changes, i.e.
const useStateWithCallback = (defaultState, callback) => {
  const [currentState, setState] = useState(defaultState);

  useLayoutEffect(() => {
    callback(currentState);
  }, [
    currentState,
  ]);

  return [currentState, setState];
};

Correct me if I am wrong, but the latter was the API of setState in React class implementation.

Yeah, it has it:

(Ignore my suggestion regarding useLayoutEffect. It is not fit for this use case.)
Based on the docs, it seems that it setState original API was equivalent to using useEffect.
I stick with my original argument that it makes sense to have a callback as a member of setState simply because it makes it easier to understand the effect of state change without scanning all instances of useEffect.
On the other hand, I do agree with @xobotyi it takes only few lines to implement useEffect and it has the benefit of being able to observe multiple states.
No strong feelings either way. Your call @streamich

It feels like this would just be creating hooks because you can. It would probably make a codebase harder to read and understand without knowing all the custom hooks of this library. Shorter code is not better code.
The example from @xobotyi does the same thing and is a lot more clear about what it does (if you know the React Hooks API). Maybe we should add a section to the docs for common patterns like this to the docs?

Yes, I agree. Not a good suggestion. Withdrawing.


feat: add getIdentitiesByPublicKeyHashes method

Issue being fixed or feature implemented

Adding  getIdentitiesByPublicKeyHashes method
What was done?

Added  getIdentitiesByPublicKeyHashes method
How Has This Been Tested?

Breaking Changes

None.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[x] I have assigned this pull request to a milestone

Don't forget: https://dashevo.github.io/js-dapi-client/#/


AI suggested data reappears even after being accepted and uploaded already

Description
These buildings data was already accepted and uploaded and days after is being shown again for no reason
Screenshots

Version
2.4.3
What browser are you seeing the problem on? What version are you running?
edge
The OS you're using
Win10
Steps to reproduce

Go to areas that data was accepted and uploaded (For Buildings)
You will find all data as not uploaded and suggested again
If you accept any of such data it will show you two buildings overlapping warnings

The browser URL at the time you encountered the bug
No URL from the app, but the location is -17.1253509, -64.3456647
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
No response

Thanks for reporting, I'm seeing this too a lot lately, this is being tracked on #1295


using self DatabaseName value

Description
This small fix is related to ()[https://github.com/mongodb/terraform-provider-mongodbatlas/issues/304]
just extracting the db name in case of update minimal change.
Tested locally (custom build) before updating the reference in the terraform repo
leofigy@DESKTOP-IQOJR0Q:~/mongo/staging> terraform apply

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # mongodbatlas_database_user.atlasexternal will be created
  + resource "mongodbatlas_database_user" "atlasexternal" {
      + auth_database_name = "$external"
      + aws_iam_type       = "NONE"
      + id                 = (known after apply)
      + project_id         = "REDACTED"
      + username           = "C=US,ST=California,L=CITY,O=COMPANY,OU=UNIT,CN=sampleuserX"
      + x509_type          = "CUSTOMER"

      + labels {
          + key   = "TestUser"
          + value = "External test"
        }

      + roles {
          + collection_name = (known after apply)
          + database_name   = "admin"
          + role_name       = "readWriteAnyDatabase"
        }
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

mongodbatlas_database_user.atlasexternal: Creating...
mongodbatlas_database_user.atlasexternal: Creation complete after 1s [id=YXV0aF9kYXRhYmFzZV9uYW1l:JGV4dGVybmFs-cHJvamVjdF9pZA==:NWNmNWE0NWE5Y2NmNjQwMGU2MDk4MWI2-dXNlcm5hbWU=:Qz1VUyxTVD1DYWxpZm9ybmlhLEw9Q0lUWSxPPUNPTVBBTlksT1U9VU5JVCxDTj1zYW1wbGV1c2VyWA==]

 ~ update in-place

Terraform will perform the following actions:

  # mongodbatlas_database_user.atlasexternal will be updated in-place
  ~ resource "mongodbatlas_database_user" "atlasexternal" {
        auth_database_name = "$external"
        aws_iam_type       = "NONE"
        id                 = "cHJvamVjdF9pZA==:NWNmNWE0NWE5Y2NmNjQwMGU2MDk4MWI2-dXNlcm5hbWU=:Qz1VUyxTVD1DYWxpZm9ybmlhLEw9Q0lUWSxPPUNPTVBBTlksT1U9VU5JVCxDTj1zYW1wbGV1c2VyWA==-YXV0aF9kYXRhYmFzZV9uYW1l:JGV4dGVybmFs"
        project_id         = "5cf5a45a9ccf6400e60981b6"
        username           = "C=US,ST=California,L=CITY,O=COMPANY,OU=UNIT,CN=sampleuserX"
        x509_type          = "CUSTOMER"

        labels {
            key   = "TestUser"
            value = "External test"
        }

      - roles {
          - database_name = "admin" -> null
          - role_name     = "readWriteAnyDatabase" -> null
        }
      + roles {
          + collection_name = (known after apply)
          + database_name   = "admin"
          + role_name       = "atlasAdmin"
        }
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Terraform will perform the following actions:

  # mongodbatlas_database_user.atlasexternal will be updated in-place
  ~ resource "mongodbatlas_database_user" "atlasexternal" {
        auth_database_name = "$external"
        aws_iam_type       = "NONE"
        id                 = "dXNlcm5hbWU=:Qz1VUyxTVD1DYWxpZm9ybmlhLEw9Q0lUWSxPPUNPTVBBTlksT1U9VU5JVCxDTj1zYW1wbGV1c2VyWA==-YXV0aF9kYXRhYmFzZV9uYW1l:JGV4dGVybmFs-cHJvamVjdF9pZA==:NWNmNWE0NWE5Y2NmNjQwMGU2MDk4MWI2"
        project_id         = "REDACTED"
        username           = "C=US,ST=California,L=CITY,O=COMPANY,OU=UNIT,CN=sampleuserX"
        x509_type          = "CUSTOMER"

        labels {
            key   = "TestUser"
            value = "External test"
        }

      - roles {
          - database_name = "admin" -> null
          - role_name     = "readWriteAnyDatabase" -> null
        }
      + roles {
          + collection_name = (known after apply)
          + database_name   = "admin"
          + role_name       = "atlasAdmin"
        }
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

Apply complete! Resources: 0 added, 1 changed, 0 destroyed.

Link to any related issue(s):
Type of change:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Required Checklist:

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

Further comments

@leofigy did you mean to cite the #292 bug?

Also would suggest specifying this impacts auth database name and not database name (I know it's a bit confusing, which is how I think we had an issue w/ this as we added more than SCRAM-SHA auth!)

Hi @themantissa , right. Sorry I updated to point the right issue #292 and fixed the tittle :).

@gssbzn could you or one of your team give this a quick review/check?


⚠ Bofrost.be has degraded performance

In c6fb23b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1943 ms

Resolved: Bofrost.be performance has improved in 793ef22 after 13 minutes.


Reboot and permission requirements

Requiring a reboot before start and requiring raspi-io to be installed as a user but ran as a root makes it very difficult to use raspi-io with dockerized builds such as through BalenaCloud.
https://github.com/fivdi/pi-io doesn't require a reboot so why is this needed with raspi-io?

My apologies for the delay, I've been on vacation.
The reboot is required to enable I2C on the device. Note: if you enable I2C on your own and reboot it yourself before installing raspi-io, then you do not need to reboot again. If you do not use I2C, you can ignore this warning. pi-io assumes that you have already configured your device for I2C use manually, which is why it doesn't print a warning. If you try and use pi-io to access I2C without enabling I2C first, you'll get a cryptic error.
This is not my own requirement, it's set by Raspbian itself. Unfortunately this means that I don't have any control over it.

Ah thank you. So I guess pi-io's documentation ideally would make a remark about having to manually enable I2C, and perhaps raspi-io's documentation should state that I2C only needs to be enabled, if so by installing and rebooting or manually?
Thanks for responding anyways.

raspi-io's documentation should state that I2C only needs to be enabled, if so by installing and rebooting or manually?

Like I mention here?  😉

Hmm, I see that it is mentioned, but it seems like it is a hard requirement to do the reebot to be able to use any of the features:

Note: You must reboot your Raspberry Pi the first time you install Raspi IO on it!

On 10 May 2019, at 01:17, Bryan Hughes<EMAIL_ADDRESS>wrote:
raspi-io's documentation should state that I2C only needs to be enabled, if so by installing and rebooting or manually?
Like I mention here? 😉
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

--
https://www.ardoq.com/contact
https://www.ardoq.com/contact
REALIZE YOUR DIGITAL ROADMAP
Enterprise Intelligence Graph for Compliance,
Governance, and Transformation
ardoq.com
https://ardoq.com/?utm_source=email_sig&utm_medium=email


Error on the Publish Python distribution to PyPI and TestPyPI Action

The "Sign the Python Distribution" job is returning the following error:

Additionnally, I can see that the current release on Github is 1.5.1
I thought the Action would automatically create a release on github along with publication on pypi, but that does not seem to be the case.

It worked :muscle:

Yes! Good job!

The workflow is creating a new release. The name of the release is copied from the name of the branch or tag that triggered the workflow.
By my understanding, in order to avoid an error in future pull requests, we need to add a tag (on the branch you want to merge into OMG_master) if we want an automatic release on GitHub. I will try to add a line of code to skipt the GitHub-release job if the pull request does not have a tag.

The last pull request #43 did not resolve this issue. But I am not sure if made a mistake in the order of the Git commands that I wrote
1.- I committed the new line of code,
2.- Pushed the commit to GitHub,
3.- Added a tag,
4.- Pushed the tag to GitHub
5.- Made the pull-request.
i.e. the commit and the tag where made in different push.
Maybe we have to commit, add a tag, and after that push to GitHub (and make a pull request).
Would you like to try? For example:
1.- create a new branch,
2.- change "pyproject.toml" version = "1.6.3" to version = "1.6.4",
3.- commit the change,
4.- add a tag (for example v1.6.4),
5.- push
6.- Make a pull request to OMG_master.


PGPSecretKey.copyWithNewPassword defaults to S2K USAGE_CHECKSUM

Hey!
When setting a passphrase on a key which previously was unencrypted, BC chooses S2K usage USAGE_CHECKSUM which is not an ideal choice.
OpenPGP.js for instance rejects this S2K usage as insecure.
Could you instead change the implementation of PGPSecretKey.copyWithNewPassword() to default to USAGE_SHA1 as default if the previous usage was NONE?

This is now done. I needed to add an extra method to allow a PGPDigestCalculator to do it first. In line with the current RFC only SHA-1 is accepted for it. Should show up on github soon.


Update items.json

Added Snow Boots and Sand Boots (however the Sand Boots without any upgrades are still missing, but all upgraded variations are included)
EDIT: Added Snow and Sand boots without upgrades in the newest commit.

Pulled locally and merged. Thanks!


'New Script' Option Added to FileSystem Docker Right Click Menu

This is a new feature added to Godot's filesystem docker that allows you to create scripts through the right click context menu. Mentioned here: #18435

Does anybody have any advice for correcting the failed Travis-ci build test?

Looks like I resolved the issue. :D


Turn off variable shadowing warning

I've always felt this warning was actively harmful. Looking at the code, I can see places where you have actively harmed your code to adhere to it :(
digest list = case list of
    []  -> "none"
    [x] -> x
    xs  -> head xs ++ " (and " ++ show (length xs - 1) ++ " more)"

In any normal code I would expect digest xs, but then you get name shadowing - a harmless situation, but which leads to a warning.
I appreciate this is personal preference, but I always turn off this warning.

To be honest I never found it difficult to fix this warning if need be. I do agree that in most cases it doesn't actually highlight an error, although on rare occasions I found it useful.
In the above example, I find that digest list actually reads better, but only marginally, so I don't mind to accept the change you proposed in #144.

I've always felt this warning was actively harmful.

Can you elaborate? I may underestimate the harm. My current understanding is that it only forces us to hide unused imports, which is indeed annoying.

I often use variable shadowing to reduce the possibility of error. For example, in digest, if you use xs as the scrutinee and the test then you can guarantee in the alternative that you don't refer to the wrong variable, since it's shadowed. It's even more useful in large monadic blocks to avoid things you have "updated".
I generally find it to be annoying and make my code less robust, so always turn it off. But it's ultimately up to your preferences and I'll follow.

The two sides of this coin we are discussing are the following, if I understand correctly:

One the one side (and I think this is the intended warning for the warning), you want to prevent using variables you think refer to something in a more global scope but you accidentally shadowed it by a local binding you might not have noticed to have done so actually. In this case it prevents a case of hard to debug errors, but also requires that both bindings are of the same type!
On the other side, we have the issue of explicitly finding new names for local variables that would be more concise using a common naming scheme. Or the inverse case of the first side, which @ndmitchell alludes to, in that you want the local binding, but accidentally obtain the global binding because you forgot to rename your variable in your subscope.

If you stick to the idea of having a consistent naming scheme, you should not run afoul of the second point, as a binding will always refer to the most recent one. But then the shadowing warning makes no sense, as shadowing is part of your style.
I don't see a clear stronger side here. Both do not prevent you from using the wrong name. The first one though keeps all bindings in scope, while the second one can purposefully reduce the number of bindings in scope by actively using shadowing.
I feel I'm a bit more on @ndmitchell's side, as less bindings should mean less potential to use the wrong binding.
From a language point of view I guess shadowing can be considered messy. From a users point of view, too many bindings might be considered messy.

OK, let's switch the warning off. I never used shadowing purposefully, but this may indeed be useful. My Haskell coding style is not yet settled and my preferences are vague (I'm a newbie), so I can switch from one to another and see how it goes.
@ndmitchell Could you do this as part of you PR?


[ENH] Refactor num_kernels

taking the opportunity of V1.0 to refactor all the rocket related estimators to use n_kernels instead of num_kernels.
I have also removed the transformer MiniRocketMultivariateVariable. It handled unequal length in an unapproved, unpublished way and is I feel legacy code.
closes #1699 There are other things in 1699, but I think a new issue is better for the actual improvements/changes. This is the tidying up finished for me

yup it should also be n_kernels for the gpu one

@MatthewMiddlehurst @hadifawaz1999 I did a search for num_kernels before, GPU one is different, it uses n_filters.


Nimbus VC to Lodestar BN shows attestation errors

Describe the bug
We're running a Nimbus VC v24.3.0 in a cluster of several BNs, including a Lodestar BN 1.17.0, and are regularly seeing the following message in the VC logs:
WRN 2024-04-04 20:14:55.043+00:00 Beacon node reports internal error         reason="Unable to decode error response: [Serialization error];500;getAggregatedAttestation(best);internal-issue" node=http://xx.xx.xx.xx:5052[Lodestar/v1.17.0/def26ac] node_index=2 node_roles=AGBSDT
NTC 2024-04-04 20:14:55.064+00:00 Aggregated attestation published           delay=64ms947us130ns service=attestation_service validator=831bb96c@785062 attestation="(aggregation_bits: 0b1111111111111111111111111101111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111101111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111, data: (slot: 8786472, index: 6, beacon_block_root: \"676aa960\", source: \"274576:b5e91a14\", target: \"274577:5b9344bd\"), signature: \"930cc3dc\")"
NTC 2024-04-04 20:14:55.514+00:00 Beacon node is online                      agent_version=Lodestar/v1.17.0/def26ac node=http://xx.xx.xx.xx:5052[Lodestar/v1.17.0/def26ac] node_index=2 node_roles=AGBSDT
NTC 2024-04-04 20:14:55.560+00:00 Beacon node is compatible                  node=http://xx.xx.xx.xx:5052[Lodestar/v1.17.0/def26ac] node_index=2 node_roles=AGBSDT
NTC 2024-04-04 20:14:55.582+00:00 Beacon node is in sync                     head_slot=8786472 sync_distance=0 is_optimistic=false node=http://xx.xx.xx.xx:5052[Lodestar/v1.17.0/def26ac] node_index=2 node_roles=AGBSDT

On the Lodestar BN it shows the following error:
Apr 04 22:14:55 ethereum-v5-fr-2 lodestar[115637]: Error: No attestation for slot=8786472 dataRoot=0xbcf1127f79d3e25defb09d0108262aa08991fce8aaf788aa3481cb51e49ff925
Apr 04 22:14:55 ethereum-v5-fr-2 lodestar[115637]:     at AttestationPool.getAggregate (file:///usr/local/bin/lodestar/packages/beacon-node/src/chain/opPools/attestationPool.ts:136:13)
Apr 04 22:14:55 ethereum-v5-fr-2 lodestar[115637]:     at Object.getAggregatedAttestation (file:///usr/local/bin/lodestar/packages/beacon-node/src/api/impl/validator/index.ts:1041:47)
Apr 04 22:14:55 ethereum-v5-fr-2 lodestar[115637]:     at processTicksAndRejections (node:internal/process/task_queues:95:5)
Apr 04 22:14:55 ethereum-v5-fr-2 lodestar[115637]:     at Object.handler (file:///usr/local/bin/lodestar/packages/api/src/utils/server/genericJsonServer.ts:45:23)

This suggests that connecting a Nimbus VC to a Lodestar BN creates attestation issues.
Expected behavior
Attestations should work when running Nimbus VC with Lodestar BN. No errors should be logged on both services.
Steps to reproduce

Run Lodestar BN
Attach Nimbus VC with active validators
Observe logs on both services

Additional context
This issue seems to be related to #5553 and #6419
Operating system
Linux
Lodestar version or commit hash
v1.17.0/def26ac

Hey, thanks for reporting. Good news is that it is not an attestation issue but rather an issue with producing an aggregated attestation, so it shouldn't really impact your effectiveness or cause missed attestations.

regularly seeing the following message in the VC logs:

I would require debug logs over a longer period to give you a proper answer on why this is failing, I have been running Nimbus VC before with Lodestar BN and have not had those errors, must be something that only happens recently.

This issue seems to be related to https://github.com/ChainSafe/lodestar/issues/5553 and https://github.com/ChainSafe/lodestar/issues/6419

The reported issue with Lighthouse VC was actually just noise and fixed on Lighthouse in the end. Might be similar here.
Based on the Beacon Node <> Validator Client compatibility matrix from the EF devops team there are issues with Nimbus VC <> Lodestar BN, will check with them if it's related to what you are seeing.
So for now, if you could provide debug logs from the Lodestar BN would be great to further investigate this

Just noticed that even the aggregate attestation was published correctly right after the error. Is your validator by any chance connected to multiple beacon nodes?

Thank you for your reply. I'll run Nimbus and Lodestar with debug logs and come back.

Thank you for your reply. I'll run Nimbus and Lodestar with debug logs and come back.

Lodestar already has debug logs enabled by default but those are only written to log files not stdout unless you have changed --logFileLevel to something else.
You can find the logs in --dataDir, see data retention for details.

Please find the logs attached. Hope this helps. Please let me know if you need anything else for debugging.
lodestar-bn.log.tar.gz

Please find the logs attached. Hope this helps. Please let me know if you need anything else for debugging. lodestar-bn.log.tar.gz

Thanks for providing the logs. I summarized this issue here https://github.com/ChainSafe/lodestar/issues/6634#issuecomment-2042672443 with an explanation on why it happens. The tldr is that it's not a real issue and just noisy logs which will be improved  in our next release https://github.com/ChainSafe/lodestar/pull/6648. I have also created an issue on the Nimbus side https://github.com/status-im/nimbus-eth2/issues/6184 to potentially improve error handling on their end.

lodestar-bn.log.tar.gz

Regarding those logs, I don't see a single "No attestation for slot" in there, did you change something in our setup, or are those logs from the primary (first) node while the error was observed on a fallback node?

lodestar-bn.log.tar.gz

Regarding those logs, I don't see a single "No attestation for slot" in there, did you change something in our setup, or are those logs from the primary (first) node while the error was observed on a fallback node?

No, I didn't change anything in the setup and these are the logs from the affected node. I was the referring to the type of error that you can find with the following command:
cat beacon-2024-04-05.log.bak|grep "Error: No attestation for slot"

cat beacon-2024-04-05.log.bak|grep "Error: No attestation for slot"

Thanks for double checking that, I was looking at the wrong log file... sorry for that. Turns out it is submitting attestations for the same slot, and the unaggregated attestation is also gossiped really timely. I can't really explain why for example for slot 8791364 the aggregate failed, your attestation made it onchain at least, likely aggregated by another validator.
While what I explained in https://github.com/ChainSafe/lodestar/issues/6631#issuecomment-2042749346 still applies and would be the expected case where this can happen, what we are seeing on your node is not. I am not sure what the Nimbus VC does differently than ours..
Another problem right now is also that we don't have good enough debug logs in our attestation pool (cache) to analyze this based on the logs. What's definitely clear is that most of the time producing the aggregate failed (28 times) while succeeded only once.
Will try to reproduce this in a simpler setup, for now, I am assuming it's an issue with our attestation pool and not something Nimbus does wrong.
While this is not that critical of an issue, please also be aware that there is also a reported block production issue using Nimbus VC with Lodestar (https://github.com/ChainSafe/lodestar/issues/6634) but since you have multiple bns connected that shouldn't be an issue.

A fix https://github.com/ChainSafe/lodestar/pull/6668 has been included in our latest release v1.18.0, feel free to reopen if you can still observe this issue


Removing and/or from ontologies

Remove all the examples of and/or from term names.
There are 18 in total. I think they can all be replaced
with just 'and'. Add comments.
Reported by: jl242
Original Ticket: "geneontology/ontology-requests/1555":https://sourceforge.net/p/geneontology/ontology-requests/1555

thanks www.movied.org


I have two problem(1. don't appear resize pointer and (week and day) Time.

mouseover Right end of event, I don't show resize pointer(<-> this pointer)

don't show time of week and day.

how do I?

@minyoungk On the first question, every event slot can be configured for specific options, can you check if the
editable, startEditable, durationEditable properties are set to true ?
Each of these keys configures specific aspects of the editting of an event. For more information, check the following link : http://fullcalendar.io/docs/event_data/Event_Object/
As for the second question, would you mind sharing your configuration of ui-cal ( and if possible the language / timezone that you are in ) ? That way i can take a look at what's going on ( and perhaps even provide a solution ).


feat: support macos log

Changes
Support logging on macOS in the same way as iOS.
Logs can be viewed through Instruments and Console.
Checklist

[ ] An issue to be fixed by this PR is listed above.
[ ] New tests are added to ensure new features are working. Please refer to this page to see how to add a test.
[ ] ./frb_internal precommit --mode slow (or fast) is run (it internal runs code generator, does auto formatting, etc).
[ ] If this PR adds/changes features, documentations (in the ./website folder) are updated.
[ ] CI is passing. Please refer to this page to see how to solve a failed CI.

Remark for PR creator

./frb_internal --help shows utilities for development.
If fzyzcjy does not reply for a few days, maybe he just did not see it, so please ping him.

Good job! After fixing CI (maybe applying the suggestion above first and see what is going on) ready to merge it

ci failed just wait for new version rust nightly. https://github.com/rust-lang/cargo/pull/14370
error: "/home/runner/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/Cargo.lock" does not exist, unable to build with the standard library, try:
        rustup component add rust-src --toolchain nightly-x86_64-unknown-linux-gnu

Sure, then just wait a bit!


Calculate the pitch for texture update

Hi guys,
I'm trying to setup V4L2 with SDL and I already have a frame / byte[] . Now I want to set a texture and updating the texture I need the pitch. What should be the pitch?
From general SDL_UpdateTexture() usage, it seems that I should use frameWidth * sizeof(uint32) but is this still true for go-sdl2? How should I calculate the row size in bytes?

So that byte[] variable, I can simply save it like: ioutil.WriteFile("./test.jpg", frame, 0644) if that helps somehow...

Hi @Sendoushi, you can get the pitch from using Lock() on the texture. However the bytes you get won't be encoded in JPEG so you will need to encode it first before saving into a JPEG file.

Here is the solution I found, and it is working but maybe not the best solution, what do you think?
        // decode into image
	img, err := jpeg.Decode(bytes.NewReader(f))
	if err != nil {
		return nil, errors.New("jpeg-decode:" + err.Error())
	}
	if img == nil {
		return nil, nil
	}

	// TODO: check if needs to be converted

	// convert pixel type
	ycbcr := img.(*image.YCbCr)
	b := ycbcr.Bounds()
	rgba := image.NewRGBA(image.Rect(0, 0, b.Dx(), b.Dy()))
	draw.Draw(rgba, rgba.Bounds(), ycbcr, b.Min, draw.Src)

Hi @Sendoushi, sorry I've been away for the lunar new year (Happy new year by the way!).
Did you manage to solve all your problems? From the code snippet itself, I don't really see if there's any potential issue with it. If you can give us a complete minimal code that runs, I'd be happy to check it!

Yep. Using the golang image standard library makes more sense.
I ended up improving the code to convert to RGBA just in case it isn't.
This also removes the dependency from sdl-image and SDL is used essentially to show data. Relying more in the go standard library. Makes sense to me.


Wait until after CSS transitions to focus and enable text selection

Fixes #721. Fixes RSP-1864.
This fixes two issue:

Animations when opening a tray in the docs are choppy due to usePress changing user-select in the middle of the transition.
Opening Picker/MenuTrigger with VoiceOver on iOS doesn't work because VoiceOver scrolls the page to the focused element which is off the screen, causing the tray to close.

These are fixed by waiting for transitions to complete before performing these actions. A new util, runAfterTransition, is added, which tracks all of the transitions currently occurring on the page. The callback is called immediately if no transitions are taking place, and queued to occur after transitions if some are currently occurring.
For focus issues, a new focusSafely function is added to @react-aria/focus. This function is kinda like focusWithoutScrolling, except that it additionally waits for transitions if any are occurring. However, this is only done if we detect the user is interacting with a screen reader, otherwise it seemed weird for focus to only show up after the tray had fully completed animating. Given that it's not an issue unless VoiceOver is running, this seemed ok.
I'm a little concerned about the potential for performance issues tracking all of these transition events, but so far in testing it hasn't been an issue. Will need to test across more devices.

@devongovett I've tested this branch with VoiceOver on Safari for iOS and waiting for the CSS transition to finish before focusing seems to fix the problem of the Overlay opening, receiving focus and immediately closing on blur.
The one separate issue we should probably raise is that the popover will occasionally close, as if blurred, when a VoiceOver for iOS user navigates to the "dismiss" button that follows the Overlay. This may be because this second dismiss button is actually outside of the viewport when the Overlay displays as a Tray.

Ah interesting. We could probably investigate positioning those buttons with css somehow such that they aren't out of view maybe.


setNewRoutePathis not called in Routing 2.0

After adding setPathUrlStrategy();, setNewRoutePath() was not called when overriding AppRouterDelegate to implement Routing 2.0.
This takes away from the ability to have URLs navigate the screens.

Hi @praharshbhatt, thanks for using the package, but routing is not part of the package.
The whole PathUrlStrategy is actually handled in the Flutter framework - this package is only for toggling it on.
So unfortunately, we will not be able to help you here. Please file an issue at https://github.com/flutter/flutter.