Sweep: Adding other elements that in Scopes to delete functions in upsonic_on_prem/utils/scope.py (✓ Sandbox Passed)

Description
This pull request adds additional elements to the delete function in upsonic_on_prem/utils/scope.py.
Summary

Modified: upsonic_on_prem/utils/scope.py
Changes:

Added elements to the delete function:

mistakes
required_test_types
tags
security_analysis
code
requirements
python_version

Fixes #72.

🎉 Latest improvements to Sweep:

New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To get Sweep to edit this pull request, you can:

Comment below, and Sweep can edit the entire PR
Comment on a file, Sweep will only modify the commented file
Edit the original issue to get Sweep to recreate the PR from scratch

Rollback Files For Sweep

[ ] Rollback changes to  upsonic_on_prem/utils/scope.py

This is an automated message generated by Sweep AI.

Apply Sweep Rules to your PR?

[ ] Apply:  All new business logic should have corresponding unit tests.
[ ] Apply:  Refactor large functions to be more modular.
[ ] Apply:  Add docstrings to all functions and file headers.

This is an automated message generated by Sweep AI.

Also delete the version history.

Working on it...
This is an automated message generated by Sweep AI.


Nested sliders unwanted behaviour with touch/mouse drag

Issue description:
I am looking for a slider that provides the following functionality:
I have an outer slider, of which a few slides can contain inner sliders.
Outer and inner sliders only work with touch/mousedrag.
So I would like to be able to stop the outer slider as soon as I am in the inner slider and slide only the inner. When I reach the end of the inner slider the outer should keep on sliding.
Here is a short video that shows what I currently got – which is unacceptable (one can't even see the nested slides). The outer slider will always scroll further and the inner is hidden.
Maybe this slider does not provide this functionality but I've seen the nested examples (which don't cover touch) and I thought asking might help.
If anyone knows another slider that provides this functionality, let me know.
Cheers
Demo link/slider setting:
https://streamable.com/peakx
Tiny-slider version:
"tiny-slider": "^2.9.1"
Browser name && version:
Firefox, but is it not related to Browsers

I have the same problem. Solved like this.
For every init slider define own function to events on dragStart, touchStart and call to event stopPropagation.
slider.events.on('dragStart', this.myFunction);

myFunction = (info) => {
	info.event.stopPropagation();
}

I hope it helps
Tiny-slider version:
"tiny-slider": "^2.9.2"


feat: add type for HistoryLimit

Changes
add type for HistoryLimit
Submitter Checklist

[x] feat/add-build-quota included
[x] Follows the commit message standard
[x] Meets the contributing guidelines (including
functionality, content, code)
[x] Test cases with documentation and functionality works as expected using current and related github repos (MUST deploy and check)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes

Any changes in behavior

For pull requests with a release note:
add type for HistoryLimit

先不统一结构体，后续如果需要再统一


Error check on z/OS init commands

Small tweak. No executes without an error check.

This looks ok, but are you sure every zOS server will support these commands? Because if it fails we will crash with an exception right during connection which might make the library look bugged.

These two commands are currently defined for EVERY z/OS CS FTP Server.
I understand the worry. Ok, I would agree to be really really sure, maybe you should revert.
I am 99% sure that these commands have been included in even the earliest versions of the IBM FTP server since many many years ago.
I actually thought you might prefer good error checking….
Von: Robin Rodricks @.***
Gesendet: Sonntag, 19. Dezember 2021 18:59
An: robinrodricks/FluentFTP
Cc: FanDjango; Author
Betreff: Re: [robinrodricks/FluentFTP] Error check on z/OS init commands (PR #804)
This looks ok, but are you sure every zOS server will support these commands? Because if it fails we will crash with an exception right during connection which might make the library look bugged.
—
Reply to this email directly, view it on GitHub https://github.com/robinrodricks/FluentFTP/pull/804#issuecomment-997434439 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMFOTW6WKLNTVHNXYL7742LURYMNTANCNFSM5KJZUVDA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AMFOTW2O3BBPLI44N7OX3LTURYMNTA5CNFSM5KJZUVDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHNZ2IRY.gif Message ID: @.***>

No, let it be. I always prefer good error checking, sure, but I also like to make a library that doesn't break. Since you are the undisputable champion of zOS support for this library I will differ to your judgement and leave this as the default behavior. Thanks for the efforts.


Training Val is not working (Precision recall and f1 at 50 always)

I have a dataset of two classes and in the training the values are not coherent. They are the same. The dataset is unbalanced and I don't know if it could be a problem in this case. The values of validation do not change during the training but the values of the training does. This is the output:
{'accuracy_top-1': 100.0, 'accuracy_top-5': 100.0, 'support': 15.0, 'precision': 50.0, 'recall': 50.0, 'f1_score': 50.0}

I’m closing this issue because it has been inactive for a few months.
Please reopen if you still encounter this issue with the latest stable version. You can also contribute directly by creating a PR
Thank you!


[WFLY-13255] Upgrade Apache wss4j to 2.2.5

Issue: https://issues.redhat.com/browse/WFLY-13255
Update wss4j to version in line with Cryptacular 1.2.4

@jimma could you review please?

@spyrkob  Can we please upgrade these components too ?
jasypt 1.9.2 -> 1.9.3
opensaml 3.3.0 -> 3.3.1

@jimma updated the PR with the other component upgrades. Are those related to Cryptacular update as well?

@spyrkob  wss4j 2.2.5 upgraded these versions, so we should upgrade them in WFLY.


Can push direclty on main

We should redo the setting because i just push directly on main without a pull request

You're now admin.


Mint login failure: timeout in Multifactor Method Loop

Started yesterday afternoon...
[ivh@ci] ./wx.ad.mnt.py ro
Login to Mint failed due to timeout in the Multifactor Method Loop
Traceback (most recent call last):
 File "/usr/local/lib/python3.8/dist-packages/mintapi/signIn.py", line 401, in sign_in
  WebDriverWait(driver, 5).until(
File "/usr/local/lib/python3.8/dist-packages/selenium/webdriver/support/wait.py", line 90, in until
  raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
 File "/usr/local/lib/python3.8/dist-packages/mintapi/api.py", line 169, in login_and_get_token
  self.status_message = sign_in(
File "/usr/local/lib/python3.8/dist-packages/mintapi/signIn.py", line 407, in sign_in
 raise RuntimeError(
RuntimeError: Login to Mint failed due to timeout in the Multifactor Method Loop

Anyone else experiencing this?
Here's my setup:
% pip show mintapi
Name: mintapi
Version: 2.7
% pip show selenium
Name: selenium
Version: 4.4.0
% apt list -a google-chrome-stable
Listing... Done
google-chrome-stable/stable,now 104.0.5112.79-1 amd64 [installed]
% ll
lrwxrwxrwx 1 root root  38 Aug 15 20:34 chromedriver -> rls/chromedriver_linux64_104.0.5112.79*
dr-x------ 2 ivh  wx   138 Aug 15 20:34 rls/
Still getting the timeout in the Multifactor Method Loop errors, since late last week.
Any help appreciated!

I was able to workaround this by enabling MFA; closing this issue.


Background not moving ?

For some reason my background doesnt move ingame, even tho I set the speed in Game Maker. When I press on: Show Animations, it moves in Game Maker, why doesnt it move ingame ?

Nvm, found out why: It needs to be called Backgrounds, not Background. Now it works flawless ;)


Table not found when creating GlobalTable resource

Describe the bug
I am trying to create a global table and I am getting the following error:
ack-dynamodb-controller-5c4f76b845-xpdlk controller 2021-10-18T04:34:25.065Z
ERROR	controller-runtime.controller	Reconciler error	{"controller": "globaltable", "request": "kube-system/ack", "error": "TableNotFoundException: Table not found: Table: 'acktable' not found in region: 'us-east-1'."}

Does the DynamoDb controller uses  DynamoDB Global Tables V2 (version 2019.11.21) or  DynamoDB Global Tables V1 (version 2017.11.29)?
If it's using V1, could you confirm if I need to create the table in both regions before creating the global table resource? Is V2 going to be supported in the near future?  Thank you very much.
Steps to reproduce

Create the resources

cat  <<EOF | kubectl apply -f -
apiVersion: dynamodb.services.k8s.aws/v1alpha1
kind: GlobalTable
metadata:
  namespace: kube-system
  name: ack
  labels:
    table: acktable
    version: v1.0.0
    helm.sh/chart: dynamodb-v1.0.0
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: dynamodb
    app.kubernetes.io/instance: ack
spec:
  globalTableName: acktable
  replicationGroup:
    - regionName: ap-southeast-2
    - regionName: us-east-1
---
# Source: dynamodb/templates/table.yaml
apiVersion: dynamodb.services.k8s.aws/v1alpha1
kind: Table
metadata:
  namespace: kube-system
  name: ack
  labels:
    table: acktable
    version: v1.0.0
    helm.sh/chart: dynamodb-v1.0.0
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: dynamodb
    app.kubernetes.io/instance: ack
spec:
  tableName: acktable
  billingMode: PAY_PER_REQUEST
  attributeDefinitions:
    - attributeName: ForumName
      attributeType: S
    - attributeName: Subject2
      attributeType: S
  keySchema:
    - attributeName: ForumName
      keyType: HASH
    - attributeName: Subject2
      keyType: RANGE
  streamSpecification:
    streamEnabled: true
    streamViewType: NEW_AND_OLD_IMAGES
EOF

Expected outcome
DynamoDb table created in both regions.
Environment

Kubernetes version
1.19
Using EKS (yes/no), if so version?
yes : eks.6
AWS service targeted (S3, RDS, etc.)
DynamoDB

Hey @acunabit, here is the model we use to generate the dynamodb controller: https://github.com/aws/aws-sdk-go/blob/main/models/apis/dynamodb/2012-08-10/api-2.json#L4.

could you confirm if I need to create the table in both regions before creating the global table resource?

That's correct.

Thank you very much for your response @A-Hilaly.

@a-hilaly - Did you figure out whether you're using the 2017 Version or the 2019 Version? Based on the docs and your comment that the table needs to already exist in all regions it sounds like you're using the 2017 version.
The GlobalTable documentation need a lot of improvements here:

Mention that the table needs to already exist in all regions first.
Mention that it's using version 2017 (unless it's not.)
Provide any working example of using it.


Suggestion: create marker wizard for polygons dataset

Adding a new wizard for polygons that applies CartoCSS for points so that polygons are represented as their centroids, for example.
cc @javisantana @saleiva

This is the bubble map, right? It's there!

More specifically, a wizard equivalent to the one that shows for Point geometries with support for image markers. Like this:


naoqi_libqicore: 2.9.0-4 in 'melodic/distribution.yaml' [bloom]

@Pandhariix FYI I opened https://github.com/ros-naoqi/libqicore-release/pull/2 to pass over maintainership.
After merge we'll need to run bloom/release again

By the way, naoqi_libqicore depends on naoqi_libqi: should I wait for naoqi_libqi to be released at least once for melodic before running bloom/release again for naoqi_libqicore?

Some updates were added to naoqi_libqicore, we'll need to make a new release anyway. I'm closing this PR, once naoqi_libqi is properly released for melodic, I'll open a new one for naoqi_libqicore


Node.raise() is misleading

Godot version:
3.1.1
Issue description:
The description of this method says:

Moves this node to the top of the array of nodes of the parent node.

What does the "top of the array" mean? I interpreted it as it moves it as first child, but actually it's the opposite. The documentation should clearly state where the node will move instead of using vague terms.

It is called raise() because it was made for bringing the node visually on top of its siblings in 2D. For that, the node has to be moved to the bottom of the list, so the doc is wrong indeed.

Definitely a candidate for renaming, neither the docs nor the name say what it does.

The name is also confusing to anyone expecting raise() to be a counterpart to Python's raise statement.

If this method is desired, I think it would make sense to move it to CanvasItem.

@aaronfranke the problem with doing this is that while raise() is mostly useful only for CanvasItem, its functionality does not depend on CanvasItem in any way, and thus it makes no sense to be there.

@bojidar-bg does it have a use case in 3D though? It seems to be way less common.

Tree order affects e.g. processing order, so technically it has use in any node.

@KoBeWi This has an effect for sure, but then the use case is different.
Do we want a general move_to_end function that moves a node to the end in an attempt to fulfill different use cases ;
or, do we want "bring to front", in which case it makes sense to move it to CanvasItem, matching the use case it was originally intented for.
The function is a shortcut, so use case matters more than what it actually does.

The doc was updated, but what should we do with the method? Rename? (could be called move_to_top()) Remove?


Update cuda/cutlass_gemm to CUTLASS 3.5.1

Pull Request: Resolve RowBroadcastDescriptor Namespace Issues in cutlass_kernel.cu
This pull request fixes compilation errors in kernels/cuda/cutlass_gemm/cutlass_kernel.cu related to the incorrect namespace usage of RowBroadcastDescriptor. The modifications ensure compatibility with the latest CUTLASS version (I was not able to run the code with previous version of CUTLASS as it was not mention which version or tag was used for the kernel).

HI @SamirMoustafa  - this looks great, thanks for the PR.  Let me review it tomorrow so we can land.

ah, the new error is specific to 3.6 TMA.. let me add a note that this is built for 3.5.1 for now in the readme which as you noted, we forgot to call out what version:
Upcoming CUTLASS 3.6 release will include a breaking refactor to the Hopper TMA pointer array batched epilogue in order to support grouped GEMMs.


chore(proto): add u2f and totp challanges to session

Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

Closing because the new definitions would require refactoring.


Edit EGG domain

Add space for ...
Etherscan: https://etherscan.io/address/0x2b7018Fca0eccfBc738828e1E8f1C7038EBBCaa8
Website: https://eggy.finance
https://twitter.com/EggyFinance

Closing this PR for now, feel free to open it again once you've done the changes.


Avoid redundant enumeration of all package versions during solve failure

error: Because the current Python version (3.9.6) does not satisfy Python>=3.11,<4.0 and all versions of rooster-blue depend on Python>=3.11,<4.0, we can conclude that all versions of rooster-blue cannot be used.
And because only the following versions of rooster-blue are available:
    rooster-blue==0.0.1
    rooster-blue==0.0.2
    rooster-blue==0.0.3
    rooster-blue==0.0.4
and you require rooster-blue, we can conclude that the requirements are unsatisfiable.

There's no reason for this message to be so verbose. We already said all versions of rooster-blue cannot be used. The conclusion could be:
And because you require rooster-blue, we can conclude that the requirements are unsatisfiable.

How would you go about solving this?

I'm not sure. I think the latter half comes from
https://github.com/astral-sh/uv/blob/57ea55d2185213236e68fef82135f1cfc1686043/crates/uv-resolver/src/pubgrub/report.rs#L280-L293
which doesn't have a obvious way to inspect that we previously concluded that "all versions cannot be used".
Maybe we don't need to know that we said that previously and instead we should just have better handling at
https://github.com/astral-sh/uv/blob/57ea55d2185213236e68fef82135f1cfc1686043/crates/uv-resolver/src/pubgrub/report.rs#L92-L99
and collapse that to "all versions" when we see we're enumerating all of the available versions


docs: provide helpful example on how to enable password auth when using docker compose

provide helpful example on how to enable password auth when using docker compose so that whoever uses docker compose can easily add password auth if they wanted to

Instead of editing docker-compose.yml, we should add this documentation here:
https://microsoft.github.io/garnet/docs/welcome/releases
Can do this by a PR to this file: https://github.com/microsoft/garnet/blob/main/website/docs/welcome/releases.md

thanks badrishc,
i've updated the PR, something like this? https://github.com/microsoft/garnet/pull/209/files#diff-0348affd91795660ce75ed4eead47960881f2511e6b5f63b224680c4f044d161R41


Generated icons have margin problem do you have any suggestions ?

Hi, I was tried many svg icon converts with fantasticon but everytime has margin problem like this example:

You can see the bus is not at the center. How can I fix this problem?

Try adding the following code into the .fantasticonrc. These options are then passed directly to svgicons2svgfont which has options for centering. For me it solved the problem.
formatOptions: {
    svg: {
        centerHorizontally: true,
        centerVertically: true
    }
}

@anh4n  thanks.
this should be documented somewhere.


How add additional column Header (Column Header Span)?

How add additional column Header like Product/Customer on this screen (like Column Header Span in prev. version):
http://allenfang.github.io/react-bootstrap-table/images/react-bootstrap-table-sample.png
??

@Menogus so far we dont have column span feature, I will implement it when I get time.

Close it due to duplicated issue


SSH connect to windows page has a copypaste error where it refers to RDP files

Under https://learn.microsoft.com/en-us/azure/virtual-machines/windows/connect-ssh?tabs=azurecli#connect-from-azure-portal section, step 4 it refers to downloading an RDP file.
This appears to be an accidental copy-paste from the RDP page
Quote with bold for emphasis:
"Select the SSH tab. If the VM has a just-in-time policy set, you first need to select the Request access button to request access before you can download the RDP file. For more information about the just-in-time policy, see Manage virtual machine access using the just in time policy."

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

ID: 19d379b5-89e1-a347-adb7-aaab19068661
Version Independent ID: 3d232c7e-e92b-0414-4caf-504198a07869
Content: Connect using SSH to an Azure VM running Windows - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/connect-ssh.md
Service: virtual-machines
GitHub Login: @cynthn
Microsoft Alias: migreene

@k290
Thanks for your feedback! We will investigate and update as appropriate.

@PriyankaRudramaina
Please review it .

@k290,
Thank you for reporting. I have made the correction in the state. This will take some time to publish the changes. Please report back if you see any other issues.


Update/graal 21.2.0

This PR contains the following updates:

Update to Graal 21.2.0
Fix building only PTX backend, when --ptx is exclusively used
Added options for OpenJDK 11, OpenJDK 16

I have tested the script in Ubuntu 18.04 and MacOS Big Sur (11.5.2).

Thank you @stratika . This LGTM.


Bump internal tooling to v0.9.8

Seems to be working finer on Linux so far, when everything is local. Checking if this works fine on macOS if Dagger Engine is local, rather than running on a remote Docker.

@gerhard just tried this too (gh pr checkout 6562 and ./hack/make engine:testimportant) and it is working direct on macos for me (with docker on the same machine), just FYI.

Yes, I am seeing the same thing. All working fine when everything is locally, both on macOS & Linux.

First run failed on macOS too:

Re-running to check if this is a flake or genuine issue.

I think this was the case in the past, but just realised that even though nothing changes, each test run takes just as long as the first, meaning that there is no caching between runs. Leaving a note for self here - this was surprising!

After the 30m timeout bump, hit this on Linux:

Clearing all local volumes and starting again with a clean v0.9.8 Engine.

./hack/make engine:test succeeded after 15m 31s on macOS:

I ran a simpler pipeline with 0.9.8 Go SDK & a remote Engine & everything worked as expected. All good to move on.

FWIW I ran locally on Linux (vm on my m1 mac) and while I did also need the 30m timeout it passed.

Thank you for confirming @sipsma . There must be something different about my Linux setup then. Don't think that running NixOS Unstable is helping. Will switch back to 23.11 and see if that improves things.


Reuse code for setting the backup label in submarinerAgentController

Define a generalized addBackupLabel method used for both the Broker and globalnet ConfigMap. To facilitate this, the controllerClient is now used for accessing the Broker resource instead of the dynamicClient.

/cc @skitt @vthapar @sridhargaddam

/lgtm

/approve


Node.js -  Set memory > 2GB

<--- Last few GCs --->

[66320:0x102808e00]   113371 ms: Mark-sweep 1375.0 (1463.1) -> 1375.0 (1463.1) MB, 733.8 / 0.0 ms  allocation failure GC in old space requested
[66320:0x102808e00]   114313 ms: Mark-sweep 1375.0 (1463.1) -> 1374.9 (1427.1) MB, 941.7 / 0.0 ms  last resort GC in old space requested
[66320:0x102808e00]   115160 ms: Mark-sweep 1374.9 (1427.1) -> 1374.9 (1427.1) MB, 846.8 / 0.0 ms  last resort GC in old space requested

See more on this issue here: https://stackoverflow.com/questions/26094420/fatal-error-call-and-retry-last-allocation-failed-process-out-of-memory


video on /web/tools/chrome-devtools is not loading

https://developers.google.com/web/tools/chrome-devtools/

I think this is the URL to the video: https://www.youtube.com/embed/dJR-n8szgBc?controls=2&modestbranding=1&showinfo=0&utm-source=crdev-wf

nm


refactoring of external feed class

yes go for it! Let me know when you're done

yes go for it! Let me know when you're done

Done for now! I still need to update the corresponding params but I'll need to save that for later.

Great!
Btw, do you know if the GUI has a provision for 'normal' vs 'uniform' distribution. I am a little puzzled to see it in the code but not in the GUI?

Great!
Btw, do you know if the GUI has a provision for 'normal' vs 'uniform' distribution. I am a little puzzled to see it in the code but not in the GUI?

You're right, I don't see the 'uniform' burst distribution referenced anywhere in the GUI or mentioned in the tutorials. I'm guessing this is due to lack of a use-case. Physiologically speaking, I can't think of a situation where a burst event would ever occur with sharp edges. Even a near-synchronous Poisson process wouldn't contain the edge effects present at the bounds of a uniform (i.e., rectangular) distribution.

okay sounds good. I ask this because it means we don't have to expose an argument called distribution (at least for now).

@rythorpe I'm going to do a pass now. Do you have any commits to push?

Go for it. I won’t be able to work on it until Monday probably.

@rythorpe I have to stop for today and I may not be able to get to this tomorrow.
I left some comments. Hopefully it begins to make sense. What we're trying to do is:

discourage users from editing the params object directly but rather through an object oriented interface. This allows you to have proper checks and do things more transparently
cleaning some code along the way (getting rid of dead code for instance) + adding tests
documenting the parameters in the docstrings

@jasmainak Can I push something I've been working on?

Go for it!
On Mon, Jun 22, 2020 at 6:27 PM Ryan Thorpe<EMAIL_ADDRESS>wrote:

@jasmainak https://github.com/jasmainak Can I push something I've been
working on?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jonescompneurolab/hnn-core/pull/108#issuecomment-647800425,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADY6FIX4PJCZXX5VB7GTILTRX7LE5ANCNFSM4OA7C2XA
.

@jasmainak We should be clear to move forward with the PR now. At some point we should work on adding more tests for the cell module.

@jasmainak I'm considering making the following changes:

interactive rebase to delete the changes to vecevent.mod in https://github.com/jonescompneurolab/hnn-core/pull/108/commits/fc60b7fcfcc47def543f1233152ba7d7ed5f6f29

deleting the check_random_state code in a new commit to retain history

additional commits to address your other concerns mentioned above

Do these sound good to you? If so, do you have any commits to push before I work on this?

@rythorpe that sounds like the right sequence of steps!

Otherwise LGTM. Let's try to merge this PR soonish so we can move on with life :)

@jasmainak check out the test I added for the _ArtificialCell class. If we're good there, I'm happy with merging and moving on

@rythorpe please set the PR to [MRG] once it's good to go for you

Codecov Report

Merging #108 into master will increase coverage by 0.15%.
The diff coverage is 90.38%.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
+ Coverage   89.92%   90.08%   +0.15%
==========================================
  Files          17       17
  Lines        1708     1704       -4
==========================================
- Hits         1536     1535       -1
+ Misses        172      169       -3

Impacted Files
Coverage Δ

hnn_core/feed.py
65.18% <50.00%> (-0.81%)
:arrow_down:

hnn_core/cell.py
86.60% <100.00%> (+0.53%)
:arrow_up:

hnn_core/network.py
87.33% <100.00%> (ø)

hnn_core/tests/test_cell.py
100.00% <100.00%> (ø)

hnn_core/tests/test_feed.py
100.00% <100.00%> (ø)

hnn_core/tests/test_network.py
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 015124c...61857ec. Read the comment docs.

@rythorpe please set the PR to [MRG] once it's good to go for you

Just for reference, what does MRG stand for?

[MRG] = merge
[WIP] = work in progress
these are two common titles used for PRs :) In the days when github didn't have the approve button folks would also do [MRG +1] and [MRG +2] and then merge the PR once it's [MRG +2]

whatever happened to Travis CI?

@blakecaldwell should we wait for your PR #79 to be merged first before merging this to avoid rebase mess? What do you think? One of the PRs has to do the rebase ... not sure which one is easier

@blakecaldwell should we wait for your PR #79 to be merged first before merging this to avoid rebase mess? What do you think? One of the PRs has to do the rebase ... not sure which one is easier

Thanks for checking. Yeah, that would be nice to push #79 first. After these two, we will have a good handle on the Network and NeuronNetwork refactoring that remains.

@rythorpe we will need to rebase this PR now. Up for the challenge? ;-)

@rythorpe we will need to rebase this PR now. Up for the challenge? ;-)

I need to spend some time wrapping my head around the new structure, so yeah, I'll give it a shot!

@rythorpe I did a rebase after squash. Can you check if this was done correctly? If not, can you push fixes?

Thanks for the rebase @jasmainak! Aside from the TravisCI issue on the MacOS build job, all tests now pass. I'm ready to move forward whenever you and Travis are.

okay let's move on. The MacOS build needs to be fixed separately. We should probably pin the NEURON version for now.
thanks a lot for the effort @rythorpe !


Feature signaling again

@blink1073 @dwillmer let me know what you think of the updated signaling API.
If you guys are +1 on the library changes, I'll merge after give the changes to the build system a once-over.

+1.

@sccolbert I just started using the latest phosphor signaling. Can you give me the highlights of those changes? It pretty much look like a rewrite - and a bit closer to the very first version.

I'll see if I can explain.

This is closer to the Qt model, and was borne of a desire for the user to be able to connect to a signal without explicitly knowing about what a Phosphor signal is.

@SylvainCorlay it basically just changes the order of constituents:
connect(someObject, someSignal, someReceiver, someMethod);

becomes
someObject.someSignal.connect(someMethod, someReceiver);

The advantage, as Steven mentioned, is that signals can be used by the consumer without any extra imports. It also allows signals to be declared as part of an interface, which wasn't previously possible.
All of the semantics remain the same and most of the internal logic was reused.

@blink1073 it's closer to the PySide/PyQt model. The previous version was actually closer to C++ Qt.

And the previous previous version was also closer to that.

Oh, I don't ever talk about C++ Qt.

@SylvainCorlay if your code is public, I can update it for you and submit a PR with the changes.
Barring world-changing events, I'm not changing the API again. Steve did make a good case for this which is to allow people to use the new notebook without necessarily needing to know anything about phosphor.

My code is here: https://github.com/ipython/ipywidgets/pull/46 if you are interested.
I just rebased on master. I have not pushed the version were I used the more recent API. Don't bother, it should be straightforward for me to make the change.


REST API to delete multiple/all rules

What would you like to be added/modified:
I propose the addition of a new endpoint that accepts a list of rule IDs to be deleted in a single API call. This would allow users to perform bulk deletions without having to make multiple individual requests, thereby reducing network overhead and improving system performance.
Why is this needed:
As a user of ekuiper, I often find the need to efficiently delete multiple rules through a REST API. Currently, the system only supports the deletion of individual rules through the API. To enhance the usability and efficiency of ekuiper, it would be beneficial to introduce a new REST API endpoint that allows for the simultaneous deletion of multiple rules.

You can loop through rules to send delete request. Actually, if we implement that, it is almost the same, just to loop and delete. So I doubt if this is needed.

Thank you for your answer.
I think it can be very helpful to reduce http trafic and avoid exceeding rate limit of eKuiper itself (if configured) or API Gateway


New version: Mads v1.3.0

Registering package: Mads
Repository: https://github.com/madsjulia/Mads.jl
Created by: @montyvesselinov
Version: v1.3.0
Commit: 6ac597c5fa964ec954bf6c2bc8c1779b9e143682
Reviewed by: @montyvesselinov
Reference: https://github.com/madsjulia/Mads.jl/commit/6ac597c5fa964ec954bf6c2bc8c1779b9e143682#commitcomment-73047706
Description: MADS: Model Analysis & Decision Support

remove

why this fails?!
Please let me know or manually add.
[no_block]

why this fails?!
Please let me know or manually add.
Can you please merge?!
Thank you!
[no_block]


tweaking some things

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Fix the compilation errors due to ballerina version upgrade 1.1.0

Purpose

Upgrade ballerina version to 1.1.0

Approach

Resolve the compilation errors occurred due to version change

Release note

Upgrade ballerina version to 1.1.0

Can you please update the module-jms/jms/Ballerina.lock file with the new Ballerina version as well?

Can you please update the module-jms/jms/Ballerina.lock file with the new Ballerina version as well?

added via https://github.com/wso2-ballerina/module-jms/pull/49/commits/1ab684e5ec9cf1a3a604d7ecf39f2e2bb5f8abb6


Warning: componentWillReceiveProps has been renamed

Please, update package to eliminate debug warning message:
Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

* Move data fetching code or side effects to componentDidUpdate.
* If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
* Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run `npx react-codemod rename-unsafe-lifecycles` in your project source folder.

Please update the following components: TimePicker

I updated the package and i'm still with this error on IOS.


Nested keywords should match intend of the first keyword in *** Settings ***

AlignSettingsSection does align the Test/Suite Setup/Teardown settings quite well:
Suite Setup         Start Session
...                     host=${IPADDRESS}
...                     user=${USERNAME}
...                     password=${PASSWORD}

Hovewer if the setting contains run keyword variant, the new IndentNestedKeywords will break the alignment:
Suite Setup    Run Keywords
...   Load Test Data "data.yaml"
...   AND
...   Connect to hardware "${VAR}"

It should match (+extra indent) the alignment of the first line:
Suite Setup    Run Keywords
...              Load Test Data "data.yaml"
...              AND
...              Connect to hardware "${VAR}"

It was doable in the IndentNestedKeywords itself but I've noticed that it's enough to just change the order of the transformers - IndentRunKeywords should run before alignment transformers (such as AlignSettingsSection) and the indentation will be fixed.


akoo ako package improvement for TCE

What this PR does / why we need it
This PR is for improving package quality for ako-operator and load-balancer-and-ingress-service package in TCE, adding more detailed docs and contact mailing info
Which issue(s) this PR fixes

Fixes: #
Describe testing done for PR

Special notes for your reviewer

@XudongLiuHarold Since you're a member of both the AKO Operator and load-balancer-and-ingress package owner teams, would you mind reviewing this PR?


403 ERROR  The request could not be satisfied.

I am getting the error on my ios safari and chrome both browsers when i run the website. It was working when i was back in japan in the same device but it didn’t work when i came to Pakistan. The error is

(Request blocked. We can't connect to the server for this app or website at this time. There might be too much traffic or a configuration error. Try again later, or contact the app or website owner.
If you provide content to customers through CloudFront, you can find steps to troubleshoot and help prevent this error by reviewing the CloudFront documentation.
Generated by cloudfront (CloudFront)
Request ID: yAVJq61sXZF698L6U0_OLf56TAV3ABYMca5wFC73ZQh1FcReiuFzwA== )

I will look into this.


Question about finetuning

Hello,
could you please share the steps to finetune the model on a different dataset.
Thanks

Nvm, I found some in issues section. I'm closing this.


[Relay][Frontend] Preserve Pytorch Span Names

In accordance with discussion and RFC at https://discuss.tvm.apache.org/t/frontend-preserving-pytorch-spans/16044, we provide the ability to preserve the raw Pytorch scope name like "bert.encoder.layer.11.output.dense" as opposed to getting renamed to an arbitrary "internal" TVM name like "aten::conv2d_1, aten::dense" etc.

@masahi @tqchen, would appreciate any review / feedback!

@masahi @tqchen any feedback would be awesome! Lmk if y'all prefer some changes


Updating shorturl script to give the user more information about image urls with no titles

Dear developers,
This request is updating shorturl.py plugin to give the user more information about image urls with no titles by using clarifai api for images processing. For more information please go to https://github.com/Clarifai/clarifai-python and http://www.clarifai.com/ . 5000 requests per month are free.
This Plugin will protect users from catcontent and other pictures and is really useful for blind people.
best regards
Christoph J.

Do you have any updates?

Just added a nice plug in the show the mensa menu of the day. The Mensa of FH Erfurt is just some 100m away and cheap for students (at the moment the api of openmensa don't support other prices :-1: )

Sorry for the delay, will look at both features as soon as possible. In the meantime a few remarks on your commits:
Please PR only one feature at a time. The easiest way to do so is to use one branche per feature and then create a pull request for that feature branch.
Also, don't commit commented out code - if you remove code, really remove the code. If you might need the code at a later point in time, you can use git to look at older code versions(as long as you commited often enough ;) )


Added notebook to initialize and print string variables with exercise.

This message was generated by git-bob (version: 0.2.7, model: gpt-4o-2024-08-06), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.
Added a new Jupyter notebook test2.ipynb to the playground folder, which includes step-by-step instructions and code for initializing and printing two string variables in reverse order. The notebook also contains markdown explanations for each step and an exercise for users to modify and experiment with the string variables.
closes #243

execute test2.ipynb


Experiments cannot produce the same effect

Hey bro, did you get this data directly from scratch?

I used the same method to train on RAF-DB and could only achieve an accuracy of over 80.


Closes softlayer/sl-ember-components#1452

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Deserialization for Kernel

Kernel is now an object-safe trait so we can use its type as &dyn Kernel, it has also a name() function to have a string representation of the kernel as LinearKernel, RBFKernel, ...
We would like to provide a functionality that instantiate the kernel's default from a string:
"LinearKernel" -> LinearKernel::Default()

via a enum
pub enum KernelName {
    Linear(LinearKernel),
   ...
}

This possibility is known to be not possible in vanilla Rust but there is a library that provides it, see blogpost.
This feature should be only available and loaded in pairing with the "serde" feature so to avoid adding another library to the deps.
This will allow parameters that contain pub kernel: Option<&'a dyn Kernel<'a>>, to be deserializable.

I can take this, I think that we can get ride of name in Kernel trait, and instead handle that in the enum

ok, the problem with Rust comes when is needed to go string->instance. there is no easy way like in Python getattr(module, string) to read an object from a string.


new transaction is waiting for open operation

Hi,
I am trying to import preloaded DB in React-native app but on a console, neither success nor failure message comes in. Please check attached image. Where I'm getting wrong?
tblCategory is table name ... DB name coming through Pragma query is main and file name is IDB_DB_New.sqlite
this IDB_DB_New.sqlite file is in www folder
    let db = SQLite.openDatabase( {name: 'tblCategory', createFromLocation : "IDB_DB_New.sqlite", location: 'Library'}, this.openCB, this.errorCB);

new transaction is waiting for open operation

and main name I checked from here

What is the runtime (ios/Android) ? Did you inspect the IDE debug log ?


Investigate docs misunderstanding about filtering records

Latest status: I've investigated - the user is correct about the error message, but I don't think the doc is relevant to the issue.
See Slack discussion: https://prisma-company.slack.com/archives/C02FZENHD4N/p1661242347725959
Original issue: https://github.com/prisma/prisma/issues/14426
Update: 25 Aug 2022. IMO the user's interpretation of the docs is reasonable, so the next step is to determine if the docs are wrong and if so, fix them.
Update 2: I've replicated the issue. Basically, with AND, the filter returns all records. However, with OR, it depends on whether the filter value is undefined or empty. If it's empty, the filter returns all records. If it's undefined, it returns []. IMO we should add a note to the doc below the example that when you pass nothing at all with OR, you get all the records
Details:
In the following doc...
https://www.prisma.io/docs/concepts/components/prisma-client/null-and-undefined#the-effect-of-null-and-undefined-on-conditionals
It says:

This is because passing an undefined value to an AND or NOT operator is the same as passing nothing at all, meaning the findMany query in the example will run without any filters and return all the users.

The user says:

I'm trying to filter an array of videos, using keywords array, that if they exist, it will fetch all the records that contain some of the values (using hasSome filter) however, if I get undefined from graphql, it'll return all the records and not filter at all.

For some reason, if prisma gets undefined in the array filter it returns an error.
I've looked at the docs and it says that whenever a filter option gets undefined, it's as if it wasn't written and it'll ignore it.

I may be wrong, but factually whenever I pass an undefined value using my graphql resolver to the prisma, it ignores it, and returns the values as if there wasn't a filter, so this is the behaviour I expected throughout every filter, including array filtering.

We need to investigate the following:

Is the doc misleading?
If so, what do we need to change to make it clearer

Looking at this issue again, I've got too focussed on the notion of empty strings, which is irrelevant here - and, from a chat with @aqrln - I realize that empty strings produce expected behavior.
The issue here is with the user's error message when they tried to filter on a string[] array by passing undefined - they got an error, instead of the behavior as described in the doc. However, the doc section in question is about filters used with conditionals, and this wasn't what the user was doing. The docs section is clear about that. Therefore, this isn't a docs error, but we could maybe signpost the user's issue better in our VSCode plugin.


altitude in mix_v5 doesn't  have MIXS ID

The  altitude structured comment name (row 8 on the MIxS tab in mixs_v5.xlsx) doesn't have a MIXS ID assigned to it.
BTW who is managing IDs? Are they always unique? Do the IDs persist across MIxS versions?

The IDs came from an old database that is no longer maintained. Chris is correct that for the latest version, terms were added, but no ID was created. We also have no way to track splits or merges in the old MIxS.  This could be assessed programmatically and manually, and some mappings made to older versions, once we finish moving to RDF.

No action items for this ticket, at this time.


Create Newcommer Guide

this file could help new commer to proceed in open source contribution and github

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

i Have changed


Feature request for "Guest Permission Level"

Description
This a feature request for defining a new permission level named as Guest. Rights for this level will be:

Add, view Issues (Close only his/her issue)
View Wiki (no updating, deleting)
Watch/Unwatch Project
Star/Unstart Project
Will be able to view and download from releases
Will not be able to view or update private repository code and files
Will not be able to view or update private repository pull requests
Will not be able to fork, create repository, migration or organization

Actually Guest permission level will be a restricted version of Read level without any privileges on private repository codes/files.
This permission is important to make internal or external users to add/watch issues to private repositories themselves.
Please check here. As far as I see, Gitea needs Guest permission level. By this permission level Gitea;

Can be used for documentation(wiki),
Binary distribution(releases)
Bug tracking(issues)

for PRIVATE repositories.
A small comparison:

Gitea
GitLab

Admin
Owner/Maintainer

Read
Reporter

Write
Developer

!!!
Guest

I think this is already possible with org teams and unit type permissions.

Not quite, as you can not specify write to issues but read only for other stuff

I've checked Gitlab. And released that what I want is Guest permission level. Please check here. I think Gitlab is also using static permission levels(no way to define new levels).  As far as I see, Gitea needs Guest permission level.(not reporter, I'll rename title).
A small comparison:

Gitea
GitLab

Admin
Owner/Maintainer

Read
Reporter

Write
Developer

!!!
Guest

Regards
PS: Edited namings similar to Gitlab's, and updated issue content and this message's content for this change.

@lafriks AFAIK a read only team is able to create issues at the moment

@derkan is https://github.com/go-gitea/gitea/pull/6274 what you wanted?

@6543 thanks for ping back. This PR brings a use case: A company runs a Gitea instance that requires login. Most repos are public (accessible/browsable by all co-workers). At some point, a customer or third party needs access to a specific repo and only that repo. Making such a customer account restricted and granting any needed access using team membership(s) and/or collaboration(s) is a simple way to achieve that without the need to make everything private.
But in my feature request this case is needed:  A company runs a Gitea instance and wants to allow public users to read wiki,add/read his issuses, download releases for a repo if rights given to him. By this we can use gitea for:

Binary distribution(automatic updates with using release files)
Online help (wiki)
Issue handling
Currently, we can not limit user for not accessing source codes(and PR's)


Add pre commit

:warning: This PR involves a big diff and a potential change in workflow. I don't know if it is something that is of interest for the project maintainers, but I think it is a great improvement as it can unify code formatting easily.
Feel free to turn it down if it goes completely out of scope for what you want for the project :sweat_smile:
What this PR does / why we need it:
This adds pre-commit support to the repo.
In essence, it:

Adds the pre-commit (and derivates) config
Applies a 1st run that automatically reformats current code
Fixes some linting errors currently present that needed manual action:
3.1. Rename roles, replacing - with _.
3.2. Set file and directory permissions when creating them on tasks (for files, mode: 0644 and, for folders, mode: 0755)
3.3. Rename the traefik.toml file to traefik.toml.jinja, to better specify that it is a template and avoid linting errors.
It also adds a Github action for running pre-commit on each commit, which would also allow to remove that step from the Travis config. I don't know what would be the opinion on completely migrating to GH Actions, but it also brings some advantages (such as better integration in PRs and code).

Any other useful info:
The pre-commit configuration is based on https://github.com/copier-org/autopretty, which is a copier template that helps generate and maintain these configurations. If you don't know copier, it is a project scaffolding tool, which is great to keep this configs up to date and maintainable. It basically bootstraps the pre-commit config using the template (which could be left as is), but it also allows for easy updates when the template upstream changes, in which case, you only need to run copier -a .copier-answers.autopretty.yml update (check https://github.com/copier-org/autopretty#get-updates for more info).

I've not seen pre-commit before - it looks cool! 👍
Whilst I really like the look of this, it raises a few points:

Lots of deps are added for anyone that wants to do dev on ansible-nas. We'd need setup scripts for linux (and windows?) to get the right bits installed. Looks like node, python3, pip?
I'd like to see the settings in this PR, and once we're agreed on what tehy should be, the actual application of them in a separate PR.
I do like the thought of GH actions, yeah. I've not spent any time looking at it yet though.

Thanks!

I've not seen pre-commit before - it looks cool! 👍
Whilst I really like the look of this, it raises a few points:

Lots of deps are added for anyone that wants to do dev on ansible-nas. We'd need setup scripts for linux (and windows?) to get the right bits installed. Looks like node, python3, pip?
I'd like to see the settings in this PR, and once we're agreed on what tehy should be, the actual application of them in a separate PR.
I do like the thought of GH actions, yeah. I've not spent any time looking at it yet though.

Thanks!

Thank you! I am happy to bring something new to the project, then. 🙂
Pre-commit basically manages several git hooks that will be executed each time you commit to ensure the code follows some expected format and rules and, in some cases, automatically reformat.
For example, it can run ansible lint before each commit, instead of only running that in the CI.
About your questions:

Lots of deps are added for anyone that wants to do dev on ansible-nas. We'd need setup scripts for linux (and windows?) to get the right bits installed. Looks like node, python3, pip?

Actually, that's one of the advantages of pre-commit. It manages all the hook's installation and dependencies through a virtual environment and the developer doesn't need to worry about it. The only thing needed is:
pip install pre-commit # install pre-commit itself through pip (or pipx, better 😊)
# In the protect folder
pre-commit install # install pre-commit in the git hooks so that in can run at each commit

Then, pre-commit automatically installs everything in the next commit and runs each hook through the changed files (or you can run pre-commit run -a to run all hooks against all files).

I'd like to see the settings in this PR, and once we're agreed on what tehy should be, the actual application of them in a separate PR.

I see. However, once you install pre-commit, it will try to run in each commit next and reformat current code. So, IMHB, it makes more sense to make those changes in the same PR. I have separated them, however, in different commits: the 1st adds the configuration, the 2nd applies the auto formatting, and the 3rd applies changes that required manual action.

I do like the thought of GH actions, yeah. I've not spent any time looking at it yet though.

Ok. It is a good alternative to Travis as it is built into GitHub (so, better integration) and doesn't have as many usage limitations, I think.
It is also easy to set up with a lot of "actions" you can use to easily run some steps.
Thanks again for the feedback. I hope to have helped 😊

Thank you! I am happy to bring something new to the project, then. 🙂
Pre-commit basically manages several git hooks that will be executed each time you commit to ensure the code follows some expected format and rules and, in some cases, automatically reformat.
For example, it can run ansible lint before each commit, instead of only running that in the CI.
About your questions:

Lots of deps are added for anyone that wants to do dev on ansible-nas. We'd need setup scripts for linux (and windows?) to get the right bits installed. Looks like node, python3, pip?

Actually, that's one of the advantages of pre-commit. It manages all the hook's installation and dependencies through a virtual environment and the developer doesn't need to worry about it. The only thing needed is:
pip install pre-commit # install pre-commit itself through pip (or pipx, better 😊)
# In the protect folder
pre-commit install # install pre-commit in the git hooks so that in can run at each commit

Then, pre-commit automatically installs everything in the next commit and runs each hook through the changed files (or you can run pre-commit run -a to run all hooks against all files).

I'd like to see the settings in this PR, and once we're agreed on what tehy should be, the actual application of them in a separate PR.

I see. However, once you install pre-commit, it will try to run in each commit next and reformat current code. So, IMHB, it makes more sense to make those changes in the same PR. I have separated them, however, in different commits: the 1st adds the configuration, the 2nd applies the auto formatting, and the 3rd applies changes that required manual action.

I do like the thought of GH actions, yeah. I've not spent any time looking at it yet though.

Ok. It is a good alternative to Travis as it is built into GitHub (so, better integration) and doesn't have as many usage limitations, I think.
It is also easy to set up with a lot of "actions" you can use to easily run some steps.
Thanks again for the feedback. I hope to have helped 😊

Just wanted to say thanks for the idea with this one - whilst the pre-commit config in this PR is a bit more aggressive than I want to go, pre-commit works great for the things I do want to do. I've started the flip over in the repo now, and will turn it on properly soon. Thanks again! 🙏


[WIP] Various groundwork for adding the new private

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

@swift-ci Please test

@CodaFi Mind looking over this when you get the chance? (Post-commit is okay.)

Michael wanted to bump up the priority of one of his test jobs (and did ask me first).
@swift-ci Please test Linux platform

@swift-ci Please test OS X platform

@swift-ci Please test OS X platform

@swift-ci Please test OS X platform


❓ Dependency update

Question
I was having a look at the dependencies, maybe some would be better to update them to the latest version.
Such as:

react-native-reanimated
react-native-video
and others

devDependencies:

example/dependencies:

example/devDependencies:

docs/devDependencies:

What I tried
No response
VisionCamera Version
2.11.2
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Sure, you can try upgrading them on your end and if everything works then you can send me a PR :)


Add rate limiter for bulk request

Description

Add rate limiter for bulk request
Without rate limit, it would send request until the destination OpenSearch throttle requests (especially when NONE refresh policy is used).
Added config param spark.datasource.flint.write.bulkRequestRateLimitPerNode to specify the rate limit per node.
The default is set to 0, which means no rate limit.

Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Added [Experimental] to the doc.


mootensai/jsblock * -> no matching package found.

Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for mootensai/yii2-enhanced-gii * -> satisfiable by mootensai/yii2-enhanced-gii[0.0.1].
- mootensai/yii2-enhanced-gii 0.0.1 requires mootensai/jsblock * -> no matching package found.
Not uploaded yet?

I have fix this. Please try again..

Thanks. It works!


Editor crash when confirming node deletion with Enter

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3.0 a5db25bd27351564f391c5f0516dff64ea61544b, macOS 10.12.6, Windows 10 x64, Windows 7 x64
Issue description:
Editor crashes when you confirm node deletion with Enter
Steps to reproduce:
Select a node on a scene, press Delete, press Enter.

Can't reproduce with https://github.com/godotengine/godot/commit/3f7f65cd1c1d5e247f7f55c49a7e7e2a84fbbec7 (on Windows 10).

Can't reproduce with 7a10d3ab78975a95f55b1bf97705ff8cc300da08

Linux/64
Wine/64

Oops, indeed, there was a bug in my fork. Thanks for checking guys and sorry for the false alarm.


Bug fixes for directed edge

Based upon #553. This updates the tests with some cases found via fuzzing to the test suite.

Coverage decreased (-0.04%) to 98.143% when pulling 9900b826abd21dd8696b99bbe3c17724eeda8ab3 on isaacbrodsky:llvm-fuzzer-directed-edge into a1157d8ee6c1b671c33066c8c5b9519f023414f3 on uber:master.


Germany ranking is broken

Currently, my badge doesn't work, it sometimes happens if I do not have enough follower to be in the list.
But this time when I wanted to check if it is the case at https://committers.top/germany
I realized that something is wrong in the webpage too. I took a screenshot for clarification. It says
This list contains all public commits for each user. There are 0 total users in the region and you need at least -1 followers to be on this list.

 I'm not sure how or why that happened, need to handle it properly. Seems to have happened for b65e260897f215c0d30ad4792cf0cf67c9881f35 as well, I suspect they're part of the same run.

@mustafaozhan please check now, it should be fixed via 087c8df2cb8963bdf0f164959c8c0011d278676b for gabon and b62487d06c1e442f9fcfa7f2d4df7f40b65b80f3 for germany.

Confirming it is fixed, thank you @ashkulz


Can't open valid Word document

I have a large document that opens fine and edits without issue under Microsoft Word 2016. It contains 67 pages, and some images and tables. When trying to open this document using the OpenXml libraries, I receive the following exception. Any suggestions?
An exception of type 'DocumentFormat.OpenXml.Packaging.OpenXmlPackageException' occurred in DocumentFormat.OpenXml.dll but was not handled in user code
Additional information: The specified package is invalid. The main part is missing.

@JustinMinnaar :
Please share the WORD document so that we can debug the issue AND openxml test code that you are using to reproduce the issue.
Thx

Regret that the document is a confidential document and I cannot send it to you. I'll try create a dummy document tomorrow to reproduce the issue. Sent from my Huawei Mobile-------- Original Message --------Subject: Re: [OfficeDev/Open-XML-SDK] Can't open valid Word document (#128)From: "TarunChopra[MSFT]" To: OfficeDev/Open-XML-SDK CC: Justin Minnaar ,Mention @JustinMinnaar :
Please share the WORD document so that we can debug the issue AND openxml test code that you are using to reproduce the issue.
Thx
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OfficeDev/Open-XML-SDK","title":"OfficeDev/Open-XML-SDK","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OfficeDev/Open-XML-SDK"}},"updates":{"snippets":[{"icon":"PERSON","message":"@tarunchopra in #128: @JustinMinnaar : \r\n\r\nPlease share the WORD document so that we can debug the issue AND openxml test code that you are using to reproduce the issue.\r\n\r\nThx"}],"action":{"name":"View Issue","url":"https://github.com/OfficeDev/Open-XML-SDK/issues/128#issuecomment-263623935"}}}

Sure, we will wait for the artifacts. Thanks.

Hi. My apologies, I've determined that this error actually means that the file is not found. I deleted everything out of the file and it still raised the error, until I realized that the file was not being copied into the debug folder.
Perhaps it would be worth raising the FileNotFoundException instead?
Kindest regards,
Justin Minnaar

From: "TarunChopra[MSFT]"<EMAIL_ADDRESS>Sent: 29 November 2016 20:53
To: "OfficeDev/Open-XML-SDK"<EMAIL_ADDRESS>Cc: "Justin Minnaar"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Subject: Re: [OfficeDev/Open-XML-SDK] Can't open valid Word document (#128)
Sure, we will wait for the artifacts. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@JustinMinnaar : Yes, you are correct and I'm able to verify\reproduce the issue. I'll add a validation check for File.Exists at all the entry points.

Great. As I was unable to Edit and Continue during debugging, I ended up wrapping the functions I needed to extract the data into my own classes. This also made it easier to write unit tests for the functions I needed. Thanks for providing me with a clean method of extending Word. It's really useful. Justin Sent from my Huawei Mobile-------- Original Message --------Subject: Re: [OfficeDev/Open-XML-SDK] Can't open valid Word document (#128)From: "TarunChopra[MSFT]" To: OfficeDev/Open-XML-SDK CC: Justin Minnaar ,Mention @JustinMinnaar : Yes, you are correct and I'm able to verify\reproduce the issue. I'll add a validation check for File.Exists at all the entry points.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OfficeDev/Open-XML-SDK","title":"OfficeDev/Open-XML-SDK","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OfficeDev/Open-XML-SDK"}},"updates":{"snippets":[{"icon":"PERSON","message":"@tarunchopra in #128: @JustinMinnaar : Yes, you are correct and I'm able to verify\reproduce the issue. I'll add a validation check for File.Exists at all the entry points."}],"action":{"name":"View Issue","url":"https://github.com/OfficeDev/Open-XML-SDK/issues/128#issuecomment-263931120"}}}

@tarunchopra are you still planning on doing this?

@JustinMinnaar Can you share the code that you are using to open the package? I'm interested which overload you are using that causes the package to fail to load. A small repro will ensure we are addressing the right issue for you (and then find if it's a pattern for other overloads)

Hello @twsouthwick . I've not worked on this project for almost a year now. It was a confidential project at a client. I regret I can't recall anymore what exactly the issue was at the time.

Ok. I think I found the location.
https://github.com/OfficeDev/Open-XML-SDK/blob/138977da97202c440ea647a86622bdcbccefa567/DocumentFormat.OpenXml/src/Framework/OpenXmlPackage.cs#L314-L35
@tarunchopra Are you going to submit a PR for this? It is a breaking change, but probably a good one. We'll want to document it, though.


Vertical aligns horizontal forms using flex instead of math

This PR changes vertical alignment of labels on horizontal forms from paddings and clac to a more robust solution with flex. Let's let the computer do the math 😄
The visual result is the same.
This PR is related to #24328
@XhmikosR @mdo what do you guys think?

I haven't thought about Form row :( I'll close it till I find a solution.


LGTM.com - false positive C# string.format arguments missing on result of string lookup

Description of the false positive

This code is marked as "The format string ignores this supplied value".

violations.Add(string.Format(CultureInfo.InvariantCulture,
  Strings.Get("Err_ClauseDuplicateName"), rule.Name, duplicateClause.Key));

However, the result of Strings.Get("Err_ClauseDuplicateName") in this case does return a string with the appropriate number of fields. LGTM of course has no way to knowing that, so this rule should just not trigger if you have string.format(f(x),args) because Semmle doesn't know if there will ultimately be enough spots for arguments.
URL to the alert on the project page on LGTM.com
https://lgtm.com/projects/g/microsoft/AttackSurfaceAnalyzer/snapshot/39c6dd21e4c4528d449c65c9ddf890cac749883f/files/Lib/Utils/Analyzer.cs?sort=name&dir=ASC&mode=heatmap#x53edbeb4d58111:1

Hi
Thanks for reporting this.
This one is tricky; the value "Err_ClauseUnbalancedParentheses" can in fact flow through Strings.Get (when "Err_ClauseUnbalancedParentheses" is not a key in the stringList dictionary), so that is why the alert is reported.
One possibility is to restrict the data flow analysis from string literals to string.format() by excluding steps through join-like program points (for example, methods with multiple returns or variables that can be assigned multiple values), but I will have to think about that a bit more.

Thanks for the response.
I forgot about that short circuit when the item isn't in the strings table - so it is ultimately possible for this string format to lose its arguments here. Maybe the right solution is to have the arguments in the placeholder string as well...
-------- Original Message --------
On Apr 27, 2020, 6:06 AM, Tom Hvitved wrote:

Hi
Thanks for reporting this.
This one is tricky; the value "Err_ClauseUnbalancedParentheses" can in fact flow through Strings.Get (when "Err_ClauseUnbalancedParentheses" is not a key in the stringList dictionary), so that is why the alert is reported.
One possibility is to restrict the data flow analysis from string literals to string.format() by excluding steps through join-like program points (for example, methods with multiple returns or variables that can be assigned multiple values), but I will have to think about that a bit more.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I'm going to address this in our code by adding the arguments into the label strings so the shortcut still preserves the arguments. https://github.com/microsoft/AttackSurfaceAnalyzer/issues/405
Now that I've thought about it more I think ultimately this is a good detection, but it would be nice to have more detail in the error about where the value that is flowing through and doesn't have the arguments is coming from.

it would be nice to have more detail in the error about where the value that is flowing through and doesn't have the arguments is coming from.

Indeed, I also thought about this, and I will convert queries like this one to path queries, in order to visualize the flow.


Update luerl dependecy to 1.2.0

Proposed Changes
Update luerl dependecy to 1.2.0
Types of Changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes issue #XXXX)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (correction or otherwise)
[ ] Cosmetics (whitespace, styles...)
[x] DevOps (Build scripts, pipelines...)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask on the
mailing list. We're here to help! This is simply a reminder of what we are
going to look for before merging your code.

[ ] I have read the CODE_OF_CONDUCT.md document
[ ] All tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if needed)
[ ] Any dependent changes have been merged and published in related repositories
[ ] I have updated changelog (At the bottom of the release version)
[ ] I have squashed all my commits into one before merging

Further Comments
If this is a relatively large or complex change, kick off the discussion by
explaining why you chose the solution you did and what alternatives you
considered, etc.

@mths1 Incidentally, I've seen that the retain_SUITE errors out since a while back because it has empty groups added to the all() definition:
'Invalid reference to group mqttv4waitsync in vmq_retain_SUITE:all/0' (mqttv4waitsync and mqttv5waitsync)
Did you plan to add more tests or just run the existing ones? (although the tests are only single node tests)


test: introduce a new %swift-plugin-dir macro

Use this to define the macro location rather than the "host" dir (which is actually for the build and not the host).  Furthermore, on Windows, the build dir is /usr/lib/swift as the host content is in the SDK.
This prepares the tests for Windows.

@swift-ci please test

@swift-ci please smoke test

@swift-ci please smoke test

Please test with following PRs:
https://github.com/apple/swift/pull/68321
@swift-ci please test Windows platform

@swift-ci please test Windows platform


Cache variables in form.js

Caching variables in the file form.js.

[Current coverage][cc-pull] is 30.39%

Merging [#469][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #469   diff @@
==========================================
  Files             3          3
  Lines           352        352
  Methods          34         34
  Messages          0          0
  Branches         42         42
==========================================
  Hits            107        107
  Misses          245        245
  Partials          0          0

Powered by Codecov. Last updated by [01d1a49...565acc6][cc-compare]
[cc-base-branch]: https://codecov.io/gh/fossasia/open-event-webapp/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/fossasia/open-event-webapp/compare/01d1a49af2e17b8bcd69d191a7cd464c4376add1...565acc6e941af41533533c57e1bed0ffd7d673ac?src=pr
[cc-pull]: https://codecov.io/gh/fossasia/open-event-webapp/pull/469?src=pr

@aakash-cr7 we are not really using form.js right now (it's doing pure HTML5 form POST).
before this, if you want to, you can help fixing the AJAX request. The current code does not work (the backend does not receive the file) .

@championswimmer Great! I'll look into the AJAX request part and let you know if I have any issues.

@championswimmer in main.js file, we are adding a click event to a button with id #btn, but there exists no such button?

@aakash-cr7 form.js has significantly changed in development branch. this is not needed anymore.
Please continue to contribute to development branch, your contributions have been really helpful


Add TypeScript example to "Build an App"

Now that we have TypeORM support, this would be really nice to have.
See also:

#4571

@ericharmeling Moving this to P-1 based on recent interest from hackathons and our general strategy to prioritize Javascript/Typescript tooling.

@ericharmeling Moving this to P-1 based on recent interest from hackathons and our general strategy to prioritize Javascript/Typescript tooling.

@ericharmeling Let me know if this is something that could be prioritized in the next month. Marketing is interested in featuring a Node.js ORM and TypeORM is something we have full support for.

I think we could get something out over the next month, based on this:
https://github.com/typeorm/cockroachdb-example


Update XCM function for Kintsugi chain

Description the bug
Update XCM function for Kintsugi chain

Information from Interlay team:

"Hi guys, we got a runtime upgrade coming up momentarily on kintsugi that includes a change in the xTokens interface. This is the change that is going to be included: https://github.com/open-web3-stack/open-runtime-module-library/pull/841/files#diff-9413d5e6d4c0aee4abccc79435f7885925fd1e4d26f8cfa1c5a7a5f377557215R232 . This PR changed the type of the weight argument of xtokens.transfer function. Are there any changes required on your side?
For reference, this is how we made our code compatible with both versions of the call: https://github.com/polkawallet-io/bridge/pull/28"

Actual: Send xcm failed
https://user-images.githubusercontent.com/102292476/224689628-0d847d10-8b8b-4036-8e06-f69b1375ad2b.mp4
=> Temporarily Solution: https://github.com/Koniverse/SubWallet-Extension/issues/1086

Resolve later in the https://github.com/Koniverse/SubWallet-Extension/issues/1264


WINDOWS ON SD CARD (windows to go) USEFUL INFORMATION

I couldn't get Windows on Go to boot on an SD card using rEFInd GUI installation. However, I was able to solve the problem.
Problem description: I could boot into Windows via the Steamdeck BIOS, and after rebooting the device, Windows would automatically boot. This wasn't a problem until I booted into the rEFInd dual boot menu. Using the Windows launch button in this menu, I got the error:
"Invalid loader file! Error: Not found while loading bootmgfw.efi".
I tried everything: I checked that the SD card was inserted into the device when I created the config in rEFInd GUI, and I also checked that I selected WINDOWS (SD) as the second boot method. I tried to manually change the config refind.conf (according to the instructions "Optional Windows from Micro SD card instructions"), after which I tried to install this config via the "install config" button in rEFInd GUI, still no result.
As it turned out: For some reason (possibly a bug), rEFInd GUI does not change the Partition UUID value in the config located in the root folder esp, which means that rEFInd dual boot menu does not have precise instructions on where to load the system from.
Here's what I did:
Going into the konsole I wrote: sudo nano /esp/efi/refind/rEFInd.conf
Then I changed the value in the volume line to the UUID of my SD card, as shown below: instead of ***** i've inserted UUID (this UUID can be found in KDE Partition Manager, again, based on the instructions in the "Optional Windows from Micro SD card instructions" section)
menuentry "Windows SD card" {
icon /EFI/refind/icons/os_win11.png
volume *****
loader /EFI/Microsoft/Boot/bootmgfw.efi
graphics on
disabled

After I manually entered the UUID of the SD card, I pressed CTRL + O and followed the instructions at the bottom of the console to save the file. Only after that did dual boot work without errors!
It should also be noted that the instructions in the "Disable Windows EFI entry or run the Windows' side script" section did not work for me. No matter how hard I tried, booting via SteamOS Recovery USB, entering commands to delete the partition, I saw the error: "Boot entry 5 not found
set active state for Boot0005: No such file or directory".
So, in order not to suffer with Linux further, I downloaded the script from the video on YouTube, the script is called Dual boot fix, and ran it in Windows, after which Windows stopped starting automatically after rebooting. For some reason, the author did not provide a direct link to the dual boot fix, the link to download this script is hidden like a needle: the link to the dual boot fix is in the description of the video with instructions for installing the script, the link to the video is in the rEFInd GUI section, and the section itself is somewhere on the main page of the Github repository. A hassle.
Here is a direct link to Dual boot fix: https://www.mediafire.com/file/w7jswsuctvnnd7k/Dual+Boot+Fix.zip/file
Run the required shortcut as an administrator while in Windows and that's it, Windows will not try to put itself above others.
Suggestions:

Add dual boot fix link in Disable Windows EFI entry or run the Windows' side script section of github page
Fixing issue with rEFInd GUI not changing SD value in config hidden into esp partition (/esp/efi/refind/rEFInd.conf)

I'm not sure how to suggest changes here. Or how to notify other users of my findings, so if this post should be left somewhere else, please correct me.

To address a few points here...
Yes, the GUI not correctly grabbing the volume partition UUID is a bug, likely from SteamOS under the hood changes for SD cards. I will investigate this when I can find the time.
The script used to also automatically disable the Windows EFI entry with a proper efibootmgr command. This has been broken for several months as it first manifested itself as an issue on SteamOS 3.5 . The easiest way to solve this issue now is (as you discovered) to run the batch script in the Windows/Install rEFInd folder. I would welcome any quality pull requests to help update the main README and/ or GUI README page, as I have primarily moved on to using ASUS ROG ALLY and ROG ALLY X, and I have not spent much time updating this repo for quite a while.

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

oh, nevermind, i found a button for this.

No PowerShell window: Previously, when you logged in, a window would open that displayed PowerShell. In this new version, that no longer happens.
The goal of these changes is to make the login experience smoother and more user-friendly without opening additional windows.
https://www.mediafire.com/file/o2hqlk1eiebihkp/Dual_Boot_Fix_Ver._2.zip/file


docs: correct the mint inflation schedule

Overview

Part of https://github.com/celestiaorg/celestia-app/issues/2029
Unless I'm missing something, one value needs to be updated in the specs:

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Where is the screenshot from? I sourced the 1.647129056757192 value from ADR-019 https://github.com/celestiaorg/celestia-app/blob/965eafb4357376aec31f84f3628f7703c5587f9a/docs/architecture/adr-019-strict-inflation-schedule.md?plain=1#L45

@rootulp I did it manually in sheets: https://docs.google.com/spreadsheets/d/1XHc0o-2kK54vyn8lS_jd0-oeSQ96S2VX7JtMG4XhL8Q/edit?usp=sharing
I'll change in the ADR if this is correct

The thing I am investigating now is that we're using that exact value in the tests and it's not complaining... not sure what's the difference

Codecov Report

Merging #2030 (41be81f) into main (83f91ca) will increase coverage by 0.00%.
The diff coverage is 52.17%.

@@           Coverage Diff           @@
##             main    #2030   +/-   ##
=======================================
  Coverage   21.54%   21.54%
=======================================
  Files         126      127    +1
  Lines       14285    14293    +8
=======================================
+ Hits         3077     3079    +2
- Misses      10915    10922    +7
+ Partials      293      292    -1

Impacted Files
Coverage Δ

app/extend_block.go
0.00% <0.00%> (ø)

app/prepare_proposal.go
0.00% <0.00%> (ø)

app/process_proposal.go
0.00% <0.00%> (ø)

pkg/proof/proof.go
74.28% <100.00%> (ø)

pkg/square/builder.go
78.21% <100.00%> (+0.99%)
:arrow_up:

pkg/square/square.go
56.17% <100.00%> (ø)

I suspect Google sheets is dropping at that level of precision so I don't think the specs / ADR need to be updated but this raises a good point that all the values in the specs should have an associated unit test


Travel website[GSSOC'23]

Related Issue
Closes #502

Description
Travel Website
Issue No. #502

This repo contains my project on Travel Website ✈ using HTML,CSS . It contains the landing page of a travel tour company website👨🏻💻.We oftenly search for many travelling places here & there , So Here is it🚤.
Screenshots

Video
https://github.com/ssitvit/Code-Canvas/assets/114172928/d0c6bdb2-a470-4f7d-9f7b-9d535d3d50ac

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@Shreyaar12 @cleveranu @Dharmesh177 @artimanputra please take a review of my PR & please merge it under GSSOC'23.

@cleveranu I have already resolved the conflicts.  please kindly look at those. & after reviewing please merge it under Gssoc'23.

Plz look again

@cleveranu I have again resolved the conflicts. please kindly look at those. & after reviewing please merge it under Gssoc'23.


[FTU] -  Make code more readable

Is your feature request related to a problem? Please describe.
While code is currently small it is still little difficult to understand.
What is the feature request?
make code more readable.
Describe the solution you'd like
Add better comments, more descriptive variable names and make code simpler.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

i would like to sanitize code.

Sure, you can open a pull request and link it to this issue. :D Thanks for dropping by!


Docs jobs failing due to tweedledum

Build shows version 1.0.0 installed but raises exception in docs:
Cell raised uncaught exception:
ImportError                               Traceback (most recent call last)
 in
2
3 @classical_function
----> 4 def grover_oracle(a: Int1, b: Int1, c: Int1, d: Int1) -> Int1:
5     return (not a and b and not c and d)
6
~/work/qiskit/qiskit/.tox/docs/lib/python3.7/site-packages/qiskit/circuit/classicalfunction/init.py in classical_function(func)
115
116     source = inspect.getsource(func).strip()
--> 117     return ClassicalFunction(source, name=func.name)
~/work/qiskit/qiskit/.tox/docs/lib/python3.7/site-packages/qiskit/circuit/classicalfunction/classicalfunction.py in init(self, source, name)
44             raise QiskitError('ClassicalFunction needs a source code as a string.')
45         if not HAS_TWEEDLEDUM:
---> 46             raise ImportError("To use the classicalfunction compiler, tweedledum "
47                               "must be installed. To install tweedledum run "
48                               '"pip install tweedledum".')
ImportError: To use the classicalfunction compiler, tweedledum must be installed. To install tweedledum run "pip install tweedledum".

I believe this is caused by a recent release of tweedledum (1.0.0) which is incompatible with the classical function compiler in terra 0.16.x. The next terra release should fix compatibility (and bump the minimum requirement version to 1.0.0). In the meantime we can should cap the tweedledum requirement to < 1.0.0 until terra is released.


Ability to launch application in private subnets only

Is your feature request related to a problem? Please describe.
We have a customer who would like to migrate from SOCA to RES, but are not comfortable exposing the application to the public internet.
We previously installed SOCA in private subnets, with an internal ALB, so we could make it impossible for users to connect unless they were on an office/VPN network.
Authentication/encryption safeguards are not enough to make them comfortable with exposing a portal to servers containing their intellectual property on the public internet.

Describe the solution you'd like
The entire RES stack can be deployed without public subnets, behind an internal ALB

Describe alternatives you've considered
Restricting access to public IPs.  This is a highly dynamic list of IPs, and is not a suitable approach for us

This feature request is tracked in our backlog. Thank you!

This feature request is tracked in our backlog. Thank you!

Thank you.  Can you provide an estimate on when this might be ready?  We have a large customer who is very eager to migrate to RES

Hi Cody, I don't have an exact date yet, but I can tell you we are actively looking into it now.

We already added support for VPC.


"Invalid Instruction" tags are cleared on reanalysis

Version and Platform (required):

Binary Ninja Version: 2.5.3145-dev
OS: macOS
OS Version: 12.0.1

Bug Description:
Certain "invalid instruction" tags are created when the arch fails to disassemble an instruction, but reanalyzing the function deletes them and does not recreate them.
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Open /System/Library/Kernels/kernel.release.t6000
Observe Invalid Instruction tags

Expected Behavior:
I expected the tags to persist.
Screenshots:

Additional Information:
Possible cause: invalid instruction tags are only created when analyzing basic blocks. Function reanalysis clears all auto tags. Basic blocks are not analyzed on subsequent reanalyses unless the file contents change, so the invalid instruction tags are cleared and never recreated.
Possible solutions:

Reanalyze basic blocks when functions are reanalyzed, so the tags are created again
Have a set of preserved tags for reanalysis
Have a set of preserved tag types reanalysis
Have a tagtypetype for them specifically to facilitate 2 or 3
Only remove tags of certain types when clearing auto analysis data

Related to https://github.com/Vector35/binaryninja-api/issues/3435. The only difference is that I never see the "invalid instruction" tag show up at all.


Fix broken link

Fixes #20983
Thanks @mateid! :rocket:

@guardrex awesome, thank you!

I merged to live, so it should show up on the live site in a few minutes.


nbsp; in svgs.

When I use the tool to generate svgs - they contain   but   isn't actually legal in svg files. When I try to open them as svg file instead of embedded into an html, they produce errors.
As far as I know, there isn't a good reason to be escaping spaces. Certainly, its not correct to uniformly convert spaces into non-breaking spaces.

How does you use the tool? CLI or the JS library.
In CLI I do distinguish the context on whether a SVG would like to be inlined in HTML or as a individual SVG file.
https://github.com/Myriad-Dreamin/typst.ts/blob/5121d2e9c093c9ff24bd5ebb593bdea8e28c7d77/exporter/svg/src/backend/mod.rs#L180-L184
https://github.com/Myriad-Dreamin/typst.ts/blob/5121d2e9c093c9ff24bd5ebb593bdea8e28c7d77/exporter/svg/src/lib.rs#L111
But in JS library I was lazy and assume users always embed them into some HTML element. But anyway, I'll let JS library aware of that that in next release.

I am using the JS library.
Rather than putting an option in, I'd suggest using   instead of   This should work no matter how the svg is used.
Still better, I think, would be to not escape whitespace at all, but set white-space: pre  in the css file.
I'd like to contribute back by sending you a PR, if you'd be amenable to one of my proposals.

I like the second one. That is very smart. We can set white-space for only the pseudo text elements by a css class selector .tsel, which should work.

close in flavor of #482 and #489


[patric-lunarg][wsi-selection]

This PR enables fallback WSI selection for gfxr replay.
The logic in this PR results in the following behavior...
If there is a WSI selection on the CLI, gfxr-replay will attempt to use that extension.
If the CLI WSI selection fails, gfxr-replay will attempt to fallback to the WSI extension used at capture time.
If the capture time WSI selection fails, gfxr-replay will attempt to fallback to an available WSI extension.
Note that there is no guarantee that WSI configuration will be successful if the playback WSI selection is different from the capture time selection.
This should address issue #583

CI gfxreconstruct build queued with queue ID 9376.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: patrick-lunarg:x: PatrickPurcellYou have signed the CLA already but the status is still pending? Let us recheck it.


add mixin template StandardExceptionConstructor

Convenience mixin for trivially subclassing exceptions. Documentation and unittest included.

Also, this code has no business in object.d. It's a helper template, not something that needs to be pulled into every piece of D code ever. I'd have commented on that before, but I wasn't paying enough attention to where you put it. It should go in std.exception.

So, feel free to create a PR with this template mixin in std.exception, but I'm closing this PR, because it doesn't below here.


Feature/codecov

Codecov Report

Merging #11 (aeb410e) into main (1e4e7cf) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head aeb410e differs from pull request most recent head 0fff73f. Consider uploading reports for the commit 0fff73f to get more accurate results

@@           Coverage Diff           @@
##             main      #11   +/-   ##
=======================================
  Coverage   76.02%   76.02%
=======================================
  Files          13       13
  Lines         342      342
  Branches       40       40
=======================================
  Hits          260      260
  Misses         79       79
  Partials        3        3

Impacted Files
Coverage Δ

notifications/models/email.py
88.23% <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 1e4e7cf...0fff73f. Read the comment docs.


Fix formatting typo in %time docstring

This paragraph shouldn't be formatted as code.
I have barely any experience with Sphinx, so please check my work before pulling.

Sorry, wrong repo!


Last step action error

Hello,
I'm getting this error on the last step of the action.

https://github.com/quicoto/mastodon-get-replies/actions/runs/4487261467/jobs/7890512058
https://github.com/quicoto/mastodon-get-replies/actions/runs/4487339677/jobs/7890665584

Thank you

ouch! Will disable this for now, and then try again later.

Fixed in v4.1.2


performance: Prevent duplicate reactions query in do_send_messages.

It's a quirk of our message-sending code that we produce two nearly identical dictionaries for messages being sent:
 982         message_dict_markdown = message_to_dict(message['message'], apply_markdown=True)
 983         message_dict_no_markdown = message_to_dict(message['message'], apply_markdown=False)

The message_to_dict code is mostly fast enough that the duplication is negligible, but it does lead to an extra query on the Reaction model.
Fixing this cleanly requires a bit of refactoring.  It shouldn't be a super messy refactoring, but we do want to be thoughtful about we do this.

This was fixed by the MessageDict.finalize_payload refactoring last year.


Create new MCMC trace plot for age-1 index

I'm working on an equivalent of Figure 18 (MCMC trace plots for biomass survey index) for the age-1 survey. It then makes sense to:

[ ] replace Figure 11 with a plot of just the age-1 survey
[ ] have a Figure 11 equivalent in the age-1 excluded appendix, as the figure is showing the reasonable fit to the age-1 when the age-1 index is excluded.
[ ] Update a bit of the new text regarding age-1 index.
[x] Add an mcmc fit figure for the age1 index just after the similar figure (18) that we used for the 2+ survey.

Agree, that makes sense. I edited it to add one more thing.

Ha ha. So here's the new trace plot (haven't tweaked axis and legend yet):

Some years above, some below.
Figure 11 data were scaled to match the data (because the model didn't use the data). Which is why the numbers in that don't match the age.1.index values or those in the new plot (that threw me for a bit).
The trace plot figure looks good - I don't need to scale by a q or use base.model$mcmc$"Q_extraSD_Age1_Survey(3)" do I? RT says they don't think the q on the age-1 is near 1 (so it's a relative index not an absolute). Think I'm just forgetting something here...

Looks good and no you don't need to scale by a q here.  If you were comparing this index to some measure of absolute age-1 then you would, but we don't need that here. Yes, just a relative index with an analytical derived q (no q estimates)

Thanks. It is an absolute measure of age-1 though (numbers of fish, output from the model). (Maybe you meant relative).

ha ha, semantics.  The survey is an absolute measure of age-1 from what they see, but effectively it is a relative index because q does not = 1.

[ ] Something still to fix related to above commit, since numbers don't match Table 17 or the recruitment plots. Will come back to fresh.

[ ] Also include the revised Figure 11 (age-1 survey without fits) in the Exec Summary? Would probably make sense. I think it best to have a more intuitive linear y-axis than the log axis.

Given these comments, I'm going with 'Relative age-1 index (billions of fish)' as the y-axis. Maybe caption can say that it's relative because catchability is not assumed to equal 1. Okay @aaronmberger-nwfsc ?

Yes, that's fine.  I guess technically our acoustic survey is also a relative index because catchability is not = 1 (it's very close though).  I think we are fine calling it relative age-1 index b/c Q is so different than one, and it matches what we think - that the survey doesn't attempt to get all age-1s and the survey work up doesn't include kriging like the age-2+ survey does.

That's a better reason (when just showing data) than talking about catchability. I'll add that into caption, thanks.

Andy: wrt to your remaining bullet at the top - please check over the new paragraph in the Model Results Section - Model Fit; paragraph starting "The base model fit to the index of age-1...";  I also added a couple sentence related to the age-1 index in the Model Uncertainty section.


Crash when computing redirectee type for erroneous program with multiple equal redirecting factory constructors

The program
class Supertype {
  factory Supertype() = X;
  factory Supertype() = X; //# 01: compile-time error
}

class X implements Supertype {
  X();
}

crashes with
Crash when compiling null,
at character offset null:
NoSuchMethodError: The getter 'typeArguments' was called on null.
Receiver: null
Tried calling: typeArguments
#0      Object.noSuchMethod (dart:core/runtime/libobject_patch.dart:50:5)
#1      KernelClassBuilder.computeRedirecteeType (package:front_end/src/fasta/kernel/kernel_class_builder.dart:1496:51)
#2      KernelClassBuilder.checkRedirectingFactory (package:front_end/src/fasta/kernel/kernel_class_builder.dart:1572:9)
#3      KernelClassBuilder.checkRedirectingFactories (package:front_end/src/fasta/kernel/kernel_class_builder.dart:1724:9)
#4      SourceLoader.checkRedirectingFactories (package:front_end/src/fasta/source/source_loader.dart:875:17)
#5      KernelTarget.buildOutlines.<anonymous closure> (package:front_end/src/fasta/kernel/kernel_target.dart:287:14)
<asynchronous suspension>
#6      withCrashReporting (package:front_end/src/fasta/crash.dart:122:24
[...]

Fixed by ad62f21e5b2966a121c45d05f93a304d282c1cfd.


Plugin: StarkNet ecosystem and compiler support

Elevator pitch:
Add support for StarkWare's StarkNet L2 networks and Cairo compiler/transpiler
Value:

Dependencies:

Design approach:
https://github.com/Shard-Labs/starknet-devnet
https://github.com/OpenZeppelin/nile
https://github.com/starkware-libs/cairo-lang
https://github.com/software-mansion/starknet.py
Task list:

[ ] Tasks go here

Estimated completion date:
Design review:

Do not signoff unless:

agreed the tasks and design approach will achieve acceptance, and

the work can be completed by one person within the SLA.
Design reviewers should consider simpler approaches to achieve goals.

(Please leave a comment to sign off)

I'm currently using ape in conjunction with nile so this would be great.

I'm currently using ape in conjunction with nile so this would be great.

:eyes: connect w/ me on telegram?