Mirego-base-setup 4 of X: Styling

📖 Description
Add basic stylesheet with font and colors
🦀 Dispatch

#dispatch/kmp

🦀 Requesting reviewers for this pull request:

@marcantoinefortier (contributor with 4 commits in the last 90 days and 29 commits overall)
@martingagnon (reviewer for the kmp stack)

🦀 Mentionning users for this pull request:

@ElCamelo (learner for the kmp stack)
@flambert (learner for the kmp stack)
@laurentboileau (learner for the kmp stack)


i18n plug-in

This is a good tool, so it should be internationalized. Are there any plan?

Thank you so much for your comment!
At present, due to limited resources, I didn't refactor the code for text translation.
I do believe i18n will be an important feature for the tool, so it is considered to be added in the next major update.


Preventing duplicate route entries

Cherrypick of https://github.com/istio/istio/pull/13431, with:

more tests for gateway
increment metric
more consistent logging

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

prow/e2e_pilotv2_auth_sds.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio_auth_sds_e2e

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio_auth_sds_e2e

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth
/test istio_auth_sds_e2e

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth.sh
d7c3bb411ea79ea74d2b0bed199eba8556a5dc1e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm @rshriram could one of you approve this?

[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by: howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Mp emails

Please review before merge; I just exported from Sheet12 with updated entries;

Has some newline issues, and easier for me to re-export then fix conflicts here.


Couple changes

This time the commit messages should be quite descriptive. Can you take a look? What do you think?

This is starting looking very good! good job :+1:


Training on Colab - CUDA out of memory

Hi, I would like to ask if someone have tried to train the model on Colab. Yesterday I tried to launch training with GPU, but it runs out of memory. It instantly fills almost all 15Gbs. I tried with smaller batches (6 and 8) but same problem.
Also I replaced the model inside the training of VQGAN , with the same used as inference to the transformer (vq_f16).
Additionally, If @dome272 could upload pretrained weights for both model I would be grateful (I need for my exam project at uni help ahaha)
Many Thanks

Hi, unfortunately not. A last thing I would like to try is gradient accumulation but I don't think it will solve the problem.
By the way I was looking to do the same as you. If I succeed I can post my solution here.
You can find also here a pretrained VQGAN and also MaskGIT: https://huggingface.co/llvictorll/Maskgit-pytorch/tree/main
Their GitHub: https://github.com/valeoai/Maskgit-pytorch.
They modified some parameters and other few things of transformer, so I suggest to use only their VQGAN if you want to follow the original implementation of MaskGIT.


Cli callbacks

We wanted a way to run some checks before allowing rez-release to run, so I added a configurable callback system for cli commands
basically, you can set arbitrary python code to run before or after specific commands (and the pre-callbacks have a chance to abort)

Hey Paul, what about the release_hook plugin? This is specifically for pre-
and post- release operations. There's a 'command' type that allows you to
run any arbitrary command, with options to abort the release and so on. We
use this pretty extensively. Another hook - emailer - sends emails on
package release.
These hooks will work regardless of whether a release happens as a result
of someone using the rez-release tool, or whether a release occurs via code
(not something we're doing yet but who knows). Is there need for a general
callback system, especially if it only works with cli tools?
Thx
A
On Tue, Aug 18, 2015 at 4:35 PM, Paul Molodowitch<EMAIL_ADDRESS>wrote:

We wanted a way to run some checks before allowing rez-release to run, so
I added a configurable callback system for cli commands
basically, you can set arbitrary python code to run before or after
specific commands (and the pre-callbacks have a chance to abort)
You can view, comment on, or merge this pull request online at:
https://github.com/nerdvegas/rez/pull/238
Commit Summary

add configurable pre/post callbacks for cli commands
post-callbacks: fix for commands which call sys.exit

File Changes

M src/rez/cli/_main.py
https://github.com/nerdvegas/rez/pull/238/files#diff-0 (86)
M src/rez/config.py
https://github.com/nerdvegas/rez/pull/238/files#diff-1 (2)
M src/rez/exceptions.py
https://github.com/nerdvegas/rez/pull/238/files#diff-2 (10)
M src/rez/rezconfig.py
https://github.com/nerdvegas/rez/pull/238/files#diff-3 (34)

Patch Links:

https://github.com/nerdvegas/rez/pull/238.patch
https://github.com/nerdvegas/rez/pull/238.diff

—
Reply to this email directly or view it on GitHub
https://github.com/nerdvegas/rez/pull/238.

Hmm, didn't know about that plugin - I'll check it out when I get into work, but i'm guessing it would serve our needs.
We don't have a specific need for any other callback type - iIf you want to axe this PR, I'm fine with that.

I'll close it for now. Let me know if there are issues with the release hook at all. There's been talk in the past of extending it to support more hook points, such as pre/post-build, that is certainly an option also.

Cool... just got our callback working with the existing release_hooks.command... I like it, does exactly what we need!

Awesome, apologies it isn't documented anywhere yet. Starting on that soon.
A
On Wed, Aug 19, 2015 at 3:20 PM, Paul Molodowitch<EMAIL_ADDRESS>wrote:

Cool... just got our callback working with the existing
release_hooks.command... I like it, does exactly what we need!
—
Reply to this email directly or view it on GitHub
https://github.com/nerdvegas/rez/pull/238#issuecomment-132805350.

These hooks will work regardless of whether a release happens as a result
of someone using the rez-release tool, or whether a release occurs via code
(not something we're doing yet but who knows)

Just to pick up this comment... we do a release via code and not the normal rez release cli.


Try to get danger-js path before trying to get danger path

Warnings

:warning:

Any changes to library code should be reflected in the Changelog.
Please consider adding a note there and adhere to the Changelog Guidelines.

  Generated by :no_entry_sign: Danger Swift against 9ee07d8dd245bbfe6eadc5cdb1fe8f0257d0f968


Can't link directly to Network page

Description
If you click a link to a Network page (e.g., /discover/tv/network/213), it takes you to /login and then to /.
Version
1.5.0
Steps to Reproduce

Copy link to a network (e.g., [your base url]/discover/tv/network/213).
Paste somewhere.
Click it.
End up on the main page.

Screenshots
No response
Logs
No response
Platform
desktop
Device
Laptop
Operating System
Manjaro
Browser
Brave
Additional Context
If you load Jellyseerr and then paste the URL into the URL bar on the same tab, you can get there directly, but not if you're clicking a link from somewhere else.
Code of Conduct

[X] I agree to follow Overseerr's Code of Conduct

This should be fixed with: c84ca4307465af4278f3dad5cf9c2b8cbae3fada

Still happening . I'm on v1.8.1. Whenever i use any link to jellyseerr content it goes to the login page. This happens mostly when the link originate from another app.. Like email(jellyseerr request notification), telegram etc.

@Sarin-jacob can you hop onto discord to troubleshoot this. Because we fixed this recently and I can't reproduce this.

Since this issue was closed, I kinda opened another issue  #723 ..😅
I'm mainly using jellyseerr through chrome, android, and i checked it in desktop too, same issue.
So should i create a new support post in discord?

@Sarin-jacob can you hop onto discord to troubleshoot this. Because we fixed this recently and I can't reproduce this.

Since this issue was closed, I kinda opened another issue  #723 ..😅
I'm mainly using jellyseerr through chrome, android, and i checked it in desktop too, same issue.
So should i create a new support post in discord?

Yes. I just tried on my android chrome, opened a link and it went directly to that page


Error when using electronize start behind proxy?

Version: 11.5.1

Target: .NET 5.0.102 & node.js 12.18.3

Running electronize start throws exception 1408F10B:SSL routines:ssl3_get_record:wrong version number
I am operating from behind a proxy. How do I resolve this issue?
Should I change some settings?
Steps to Reproduce:

Create a sample Blazor app
After making necessary changes to launchSettings.json file, run electronize start from the project directory

🎉🚀 New Electron.NET version 23.6.1 released 🚀🎉
With native Electron 23 and .NET 6 support. Your problem should be fixed here. If you continue to have the problem, please let us know. Please note the correct updating of your API & CLI. Info in the README. Have fun!


Refine UG

Codecov Report

Merging #182 (5eec623) into master (93e68ad) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #182   +/-   ##
=========================================
  Coverage     67.54%   67.54%
  Complexity      615      615
=========================================
  Files           101      101
  Lines          1975     1975
  Branches        232      232
=========================================
  Hits           1334     1334
  Misses          563      563
  Partials         78       78

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 93e68ad...5eec623. Read the comment docs.


Add features and feature values indexation

This wasn't working on faceted search 2. Now working.

This change is

Tests are coming


Add compatibility for Windows

We would like to add compatibility for Windows in ivadomed. Currently we are only Linux and Mac compatible.
Motivation for the feature
In this review, some of the reviewers mentioned that they had issues trying to install ivadomed on Windows: https://github.com/openjournals/joss-reviews/issues/2868#issuecomment-752687437
Description of the feature
To address the issue in the review, we will add to our documentation that we do not currently support Windows. However, it would be nice to support Windows in the future.

So, a quick recap on the potential process to start tackling this as it may become a multi-headed beasts soon if we start unearth more issues.
Per IvadoMed Meeting agenda discussion item here at the time, there does not appear to be any underlying libraries that fundamentally prevents the adoption of IvadoMed as library or usable binary on Windows platform. Further extensive and comprehensive validation and testing are required to ensure Windows compatibility and  accompanying concerns.
So, in light of that, what I propose as simple and rather naïve starting steps to attempt to resolve this:

Examine existing unit tests performance on Windows (as preliminary results already indicating a few unit tests are failing).
Evaluate the cases where they are failing (with GPU or with CPU, as failed test cases appear to VARY). More issues might needs to be unearthed, identified/opened soon based on these.
Issue PR fixes and supplement existing unit tests to address above concern.
Documentation update to address limitations or potential issues once complete.
Issue PR to conclude Documentation update.

More details to be updated.

So, a quick recap on the potential process to start tackling this as it may become a multi-headed beasts soon if we start unearth more issues.
Per IvadoMed Meeting agenda discussion item here at the time, there does not appear to be any underlying libraries that fundamentally prevents the adoption of IvadoMed as library or usable binary on Windows platform. Further extensive and comprehensive validation and testing are required to ensure Windows compatibility and accompanying concerns.
So, in light of that, what I propose as simple and rather naïve starting steps to attempt to resolve this:
1. Examine existing unit tests performance on Windows (as preliminary results already indicating a few unit tests are failing).

2. Evaluate the cases where they are failing (with GPU or with CPU, as failed test cases appear to VARY). More issues might needs to be unearthed, identified/opened soon based on these.

3. Issue PR fixes and supplement existing unit tests to address above concern.

4. Documentation update to address limitations or potential issues once complete.

5. Issue PR to conclude Documentation update.

More details to be updated.

I have rewritten some of the tests to work on Windows here. I haven't made a pull request yet as the priority was low.
The major issue is that PyTorch Windows wheel is not available on PyPI, so it has to be downloaded from PyTorch's server itself.

Further investigation into this issue has confirmed complications that @ssinad has mentioned, 1.5.0 not available on Windows. A supplementary optional conda.yaml has been generated to help automated conda environment setup in single step. Pip sole setup is typically not enough for GPU heavy ML/DL  project because CUDA, CUDNN setup may be required which PIP cannot handle to install. So far, have not found away get those pathed properly except manually. Therefore a conda heavy approach is extremely appreciated here... if possible.
As for test, CPU based pytest result in the following error: 2021-03-26T002118EST_AutomatedTrainingLog.txt
Ongoing investigation needed as I dive into automated  "test_automate_training_run_test".
To be updated.

Further investigation into this issue suggestions it is happening around evaluate() function, specifically when subj_acq is encoded into the results_pred["image_id"] as that is the main place image_id is updated. Given the platform specific nature when core logic is in tact (i.e. Ubuntu/MacOS functions past without issue). All signs suggest it is highly likely related to pathing and how that is handled in Windows, likely string pathing vs object oriented passing will probably help resolve this problem. Still digging.

Further investigation into the issue INDEED suggest this is a PATH related problem.

This suggests that all predictions are not being saved to the right place on Windows platform and this bug exists in the "testing.py" module affecting all Windows users of IvadoMed functionalities!
This further emphasize the importance of having Pathlib adoption to further avoid the way to parse path as shown in the screenshot.
The reason the unit test failed on Windows side is because the way malformed path is being interpreted across both platform.
On Windows platform, the prediction files were formed but were not placed in the proper location.
Amazing... how this worked on the Linux platform. How... did that even work?
Primary Recommendations:

Cease using relative pathing for file references. Internally, always convert path to absolute path ASAP.
Avoid string path manual parsing. It is not cross-os compatible and cause weird malformation as shown above.

You can use either pathlib.PurePath.name
https://docs.python.org/3/library/pathlib.html#pathlib.PurePath.name or
os.path.basename()
https://docs.python.org/3/library/os.path.html#os.path.basename to
address the second recommendation.
On Sun, Apr 11, 2021 at 8:26 PM Yang Ding @.***> wrote:

Further investigation into the issue INDEED suggest this is a PATH related
problem.
[image: image]
https://user-images.githubusercontent.com/6720068/114331441-fad5d880-9b11-11eb-82e7-ddc23d699125.png
This suggests that all predictions are not being saved to the right place
on Windows platform and this bug exists in the "testing.py" module
affecting all Windows users of IvadoMed functionalities!
This further emphasize the importance of having Pathlib adoption to
further avoid the way to parse path as shown in the screenshot.
The reason the unit test failed on Windows side is because the way
malformed path is being interpreted across both platform.
On Windows platform, the prediction files were formed but were not placed
in the proper location.
Amazing... how this worked on the Linux platform. How... did that even
work?
Primary Recommendations:

Cease using relative pathing for file references. Internally, always
convert path to absolute path ASAP.
Avoid string path manual parsing. It is not cross-os compatible and
cause weird malformation as shown above.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ivadomed/ivadomed/issues/604#issuecomment-817435920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVIPJ2ZCMFPO4C5CH7KBWTTIJK43ANCNFSM4VZQMUBQ
.

Yup. A fix has been implemented and now all the unit tests are passing on windows CPU tests at least.  Gonna test it out more on GPU base and see before getting a PR opened up.


skipLabel is not respected on reruns of the action

Describe the bug
Adding the skipLabel to a PR after the changelog-enforcer action has failed once doesn't cause subsequent reruns of the action to succeed.
To Reproduce
Steps to reproduce the behavior:

Feel free to fork our repo and create a dummy PR that doesn't update the changelog.
Wait for the "Check CHANGELOG" action to fail.3.
Add the Skip-Changelog label to the PR.
Rerun the Check CHANGELOG action.

Expected behavior
The Check CHANGELOG action succeeds.
A link to or sample of your workflow
Here is our workflow and here is an example of a PR exhibiting this bug.

Hi @mplorentz! Sorry for the delay. I just got back from a vaction.
Looking at your action workflow, you are missing the specific labeled and unlabeled action types. See the specifics (especially the notes) on the pull_request event details
Do you think we need to update the useage section to make it more clear?

Ah, thanks so much for taking a look! Adding the labeled and unlabeled types fixed it for me. I think the usage section is clear enough, I just skimmed it the first time around :) and anyone else who runs into this problem will hopefully find this issue. Thanks for your work on this, it's been a really nice addition to our CI!


Is there have anyway to load config file or read environment variables

I want to use hertz, but i find there is no example of how to load config file, i have to use viper load config by manually.
Will there be plans to support modules that load configuration files or environment variables in the future?

refer to https://github.com/cloudwego/cwgo

refer to https://github.com/cloudwego/cwgo

thanks


224304- [High] [Newsroom] [E2E Testing] https://tempnewsroom.accenture.co.uk/search - Different number of Results for "Business" search

Description of the issue:  Upon accessing the search page of Newsroom, try to search a keyword "business" then it will be directed to the search result page.  On Legacy, there are 76 results for this keyword and has pagination after the news cards lists.  Pagination has tooltip and positions at the center as per migrated.
Steps to replicate:

Access the Legacy page and the Migrated page.
Compare the look and feel of the search page.
Try to search a keyword on the search bar, using "business"
The search result page will show.
Check the number of results and look for pagination.

Legacy:  https://newsroom.accenture.co.uk/search_results.cfm
Migrated:  https://tempnewsroom.accenture.co.uk/search?q=business
Expected:

Actual:

Azdo link: https://dev.azure.com/accenturecio26/WebContentDigAssetMgmt_PR1093/_workitems/edit/224304
cc: @ravkiran @amol-anand @raymond-h-dev

@veejayricana Please understand that legacy and Franklin search are different. Legacy search will search in the whole content where as franklin will search in the title/description/abstract which is typically sufficient. So the results count will be different.


Add aggregatestore persistence subsystem.

@danil I will trade you reviews on your persistence PR, for reviews on mine ;)

hehehe, deal!


BSS ordering

BSS is ordered in an inconvenient way in the original DOL, both for Pikmin 2 and Pikmin 1. This prevents Dolphin.a OS.c from linking into the main build, as the BSS in that file needs to be static.

update: os.c links now. knowledge gained: static bss is ordered in reverse declaration order, and goes before global bss, which is in normal declaration order.

tentatively resolved from the bss rework commit

reopening as the issue was not fully resolved- hoping to repair it today

additional note: Pikmin 1 and 2 need the -common on flag, as that's what causes the BSS to be ordered like this in the first place


Are you interested in adding (other downloaders / parsers for) other countries?

Over at https://github.com/vk6flab/contest-logger/issues/20 we have been asking, approximately, "wouldn't it be good if there was a global version of hamcall.dev?". You already cover US...

Canadian calls can be downloaded from https://ised-isde.canada.ca/site/amateur-radio-operator-certificate-services/en/downloads
@vk6flab says "I can see how I could convert the VK data stored by the ACMA into such a thing"
3 down, only 100-and-something to go?   :stuck_out_tongue_winking_eye:

I am more than willing to host a more global collection of callsigns. My priorities in hamcall.dev are

Free (free to use, cheap to run)
Fast
Open

Additionally for my own use the program I wrote will even produce a sql-lite database. That database could also be distributed for easy offline use.


Should be able to limit max and min time in CupertinoDateTimePicker.

The CupertinoDateTimePicker currently only supports limiting the maximum and minimum date. Sometimes you may also want to limit time. For instance, picking a DateTime before or after now.
This should work in time mode and datetime mode.

CC @darrenaustin

Any updates on this?


openresolv: add gnused and dbus to PATH

Description of changes

Openresolv uses sed (here) and dbus (here), but they were not added to PATH. I added them in this PR.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Oops, it's been too long since this PR was first opened, and I don't use openresolv currently so I can't test the changes. Will close this for now. :(


WOO-77: Rename Merchant Token to Access Token

The tests are failing

The tests are failing

Fixed


Failed to to find pattern #91

just another pattern thing. Wondering how people find the broken one and find the new value. if someone could tell me the second an update hits I would make a pull request.
Help for anyone who can actually update it.
Hint:
findHudElement needs to be updated

I opened this pr on osiris and not a fork. anyways, anyone reading this know how to fix it so I can fix a forked version of Osiris

well, the signature in osiris works after the update (or in other words how osiris gets the pointer to function)
anyway here is the noob version of how to get a new signature:
open client.dll in ida pro, wait untill it's done dissassmbling the file, and then open strings subview search for "[%d] Could not find Hud Element: %s\n" and then you can see the function where this string is referenced in :
https://imgur.com/BZCRPBf
now you can take a signature of it with sigmaker.
it's:
direct reference: [actual address in first opcode] E8 ? ? ? ? 8B 4D 04 89 45 FC
of course you need to convert that shit to absolute address in your cheat
in case of other signatures when you don't know how to get it, then you can download old version of the file (i.e. client.dll or engine.dll) and reverse that in ida pro and search for unique thing in the function like strings, vtable reference, etc...


Installation problem

The installation hint is not complete ;-(
I installed and used exactly the example.php as shown, with my BOT_TOKEN ("12345678:xxxxxxx") in quote marks.
I get the following response to php example.php
PHP Warning:  include(/home/ronald/example.com/MyTelegram/basics.php): failed to open stream: No such file or directory in /home/ronald/example.com/MyTelegram/example.php on line 5
PHP Warning:  include(): Failed opening '/home/ronald/example.com/MyTelegram/basics.php' for inclusion (include_path='.:/usr/share/php') in /home/ronald/example.com/MyTelegram/example.php on line 5
PHP Fatal error:  Uncaught Error: Class 'React\EventLoop\Factory' not found in /home/ronald/example.com/MyTelegram/example.php:12
Stack trace:
#0 {main}
  thrown in /home/ronald/example.com/MyTelegram/example.php on line 12

Hi!
May I ask you which directions you followed in order to correct it?
Also: all the example files will try to include some common values defined in basics.php. What you define here is up to you 👍
Greetings,
Camilo Sperberg

I followed: https://github.com/unreal4u/telegram-api/wiki/Composer-basics
I put that example.php into the basis directory.

MyTelegram$ tree -L 2
.
├── composer.json
├── composer.lock
├── composer.phar
├── example.php
├── example.php~
└── vendor
├── amphp
├── autoload.php
├── clue
├── composer
├── daverandom
├── evenement
├── guzzlehttp
├── kelunik
├── psr
├── react
├── unreal4u
└── yoshi2889

I have updated the wiki, can you try again please? 👍
Greetings,
Camilo Sperberg

Thanks! That works now.

Great! Closing this issue now :) Should you have any more questions, feel free to create new ones!
Greetings,
Camilo Sperberg


Performing user creation during shop authentication

How can I intervene the shop authentication process and create a user based of the response that has been received. I'm aware that the response after authenticating with Shopify will have the user credential. Is there any controller that I can override? Thanks.

 So you can effectively override the auth functionality if you want... by including the trait and making a route to your controller to override the package route.
If you just need to make a user though, AfterAuthenticate job would be a good option, you can set it to run inline (no background dispatching). This way, you can check if a user exists for the shop, if not, simply inject one in, and it will do it before they ever see the landing page of your app.

I’m dispatching AfterAuthenticate job to a queue because the job will do some heavy lifting and it will be blocking if I run it online. So I suppose it’s not possible to get user from the shop right? And using rest API to get user is not a good way as it only works for Shopify Plus shops.

Oh I see I thought you meant Laravel User.
You want to grab associated_user from the Oauth request?
IE:
  "associated_user": {
    "id": 902541635,
    "first_name": "John",
    "last_name": "Smith",
    "email"<EMAIL_ADDRESS>    "email_verified": true,
    "account_owner": true,
    "locale": "en",
    "collaborator": false
  }

Yea that's what i want to achieve. I have to override the auth trait in order to achieve that right?

Yes, but I should also build this in as well since it would beneficial to the package.

Ah cool! Thank you. Would be nice if you could update this issue after the changes has been done so that I can update my package. 😁

This commit on the dependant package will enable this, I have to build a config option into this package to support it... stay tuned.

@ohmybrew is this feature tested for the new release?

Yes, tests now in and passing, just need some physical tests.


Fix other nerdpack vulnerability

We had a report of a vulnerability in this nerdpack that we'll want to resolve. please refer to the internal JIRA issue for specifics
Acceptance Criteria

see Ticket ID: jira/INFOSEC-3218
resolve issue
deploy the update via the catalog manager to the EU / Stage and Prod env

pr was merged, but gonna leave this open until all environments are updated (production, eu, staging).

eu & staging have been updated. closing.


"ObjectHandler not defined"

Within the main.py file I have this function defined as follows:
def new_game(self):
    self.map = Map(self)
    self.player = Player(self)
    self.object_renderer = ObjectRenderer(self) # <--- this has no problem
    self.raycasting = RayCasting(self)
    self.object_handler = ObjectHandler(self) # <----- this throws "ObjectHandler is not defined"
    self.weapon = Weapon(self)
    self.sound = Sound(self)
    self.pathfinding = PathFinding(self)
    pg.mixer.music.play(-1)

I've tried a lot of different workarounds but to no avail.


startwifi suggestion

Hi,
I really like this initiative. I use a similar structure for custom smart devices. Yours is more extensive with the dash, so I might consider switching.
In my situation, there are lots of AP's and SSID's. Sometimes I noticed that they keep on trying to connect with unlogical AP's that don't necessarily provide the best connection. To save some pain you may want to add the following to the "startwifi -> STA" section before connecting.
WiFi.setScanMethod(WIFI_ALL_CHANNEL_SCAN);
WiFi.setSortMethod(WIFI_CONNECT_AP_BY_SIGNAL);

Thanks a lot, great work.

Hey! Thanks for the suggestion. I've made the changes. Glad you're finding it useful :)


We need to support spin-off episodes

On June 1, regular Aktenzeichen episode #579 will be airing.
Four weeks later, on June 29, we’ll be seeing the pilot of a brand new spin-off, Aktenzeichen XY … vermisst.
The series has had several spin-offs before, and we’ve had good reasons to ignore them:
Acknowledging spin-offs in our queue would mean the app picks it up, stealing the limelight from the upcoming regular episode. That’s why we have logic in place that filters out spin-offs, and in fact we don’t even track them in our database.
However, with Vermisst, things are going to be different.
Not only is the Vermisst pilot (episode #1) taking the Wednesday slot, but no regular episode is scheduled yet after that.
I just added the Vermisst pilot to our database. However, the app isn’t picking it up because of our logic. So starting June 2, the site would degrade to show a “don’t know” message:

@GwennyGreen That’s what we’d see from June 2 on.
It’s not the end of the world. But if we want, we can fix it before it happens.
What we need to do is:

[x] Change the logic so the app ignores spin-offs as long as there’s at least one upcoming regular episode
[x] Re-deploy the site

Deployed and live.


Connect to Private End point of Storage Account

Hi,

Can Self Hosted IR help us to connect to Storage Account which has a private end point enabled ?
If we use Azure Managed Vnet Runtime, then connection between the Vnet ( azure Managed ) and Azure Data Factory is over a public IP of Data Factory which is not secured. So Self IR is the only way to get that security correct?

Thanks!
[Enter feedback here]

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

ID: 32785d15-8202-6e44-f1d2-420ef0600f3f
Version Independent ID: 3af5fa49-7bd4-18ce-f8dc-aa4a2b8c6478
Content: Create a self-hosted integration runtime - Azure Data Factory
Content Source: articles/data-factory/create-self-hosted-integration-runtime.md
Service: data-factory
GitHub Login: @lrtoyou1223
Microsoft Alias: lle

@chaisen2006 Thank you for sharing feedback. We are actively validating it. We will get back to you soon.

@chaisen2006 Thanks for the question. Github forum is dedicated for docs related issues. For any technical queries or clarifications, we encourage to utilise Microsoft Q & A platform. Kindly raise your query on Microsoft Q&A Platform


Adding compatibility delayed feature sampling and empty block table case

Add compatibility between feature "delayed sampling" and empty block table case

Closing as 1.18.0 is closed


[SPARK-14664][SQL] Fix DecimalAggregates optimizer not to break Window queries

What changes were proposed in this pull request?
Historically, DecimalAggregates optimizer is designed to transform general sum/avg(decimal).
However, it breaks recently added windows queries like the followings. The following queries work well without this optimizer.
Sum
scala> sql("select sum(a) over () from (select explode(array(1.0,2.0)) a) t").head
java.lang.RuntimeException: Unsupported window function: MakeDecimal((sum(UnscaledValue(a#31)),mode=Complete,isDistinct=false),12,1)

Average
scala> sql("select avg(a) over () from (select explode(array(1.0,2.0)) a) t").head
java.lang.RuntimeException: Unsupported window function: cast(((avg(UnscaledValue(a#40)),mode=Complete,isDistinct=false) / 10.0) as decimal(6,5))

This PR aims to fix DecimalAggregates optimizer not to handle aggregation queries over windows.
Note that this PR does not aim to implement optimization over Window aggregation.
How was this patch tested?
Pass the Jenkins tests (with newly added testcases)

Test build #55940 has started for PR 12421 at commit 48003ee.

@dongjoon-hyun good catch!
Can't we just make the optimizer respect the window expression? I.e. wrap the entire window expression in the Cast/MakeDecimal expression?

Oh, thank you for quick review. Actually, I tried to do first like that. There occurs exceptions about type mismatch due to the difference from input schema. So, it seems not so straightforward to me.
However, I'll try again according to your comments. Thank you, @hvanhovell !

Test build #55940 has finished for PR 12421 at commit 48003ee.

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

Merged build finished. Test PASSed.

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

Hi, @hvanhovell . According to your comments, I've worked this for a week, but I'm afraid of being a bottleneck.
I feel there is a gap between preventing and optimizing.
Do you think you could take over this issue? If you do that for me, I will close this PR after merging yours.
I think you are the best person to extend DecimalAggregate optimizer for Window query.

Test build #56602 has started for PR 12421 at commit 7f64e62.

Test build #56602 has finished for PR 12421 at commit 7f64e62.

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

Merged build finished. Test PASSed.

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

Hi, @hvanhovell .
Could you give me some advice about how to handle this issue and this PR?

Rebased.

Test build #56840 has started for PR 12421 at commit 39b5d14.

Test build #56840 has finished for PR 12421 at commit 39b5d14.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #56881 has started for PR 12421 at commit d83d8f8.

Oh, my bad. Today, I tried again to implement DecimalAggregate optimizer for Windows again and handled to implement it. Last week, I think I made some mistake during that.
Thank you so much for your review and guiding, @hvanhovell ! After this PR passes the final Jenkins test, I will update the content of PR and JIRA, too.

Test build #56881 has finished for PR 12421 at commit d83d8f8.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

The failures are MemorySinkSuite. It seems to be irrelevant to this PR. I'll rebase to retrigger Jenkins.
[info] *** 2 TESTS FAILED ***
[error] Failed: Total 1962, Failed 2, Errors 0, Passed 1960, Ignored 28
[error] Failed tests:
[error] 	org.apache.spark.sql.streaming.MemorySinkSuite
[error] (sql/test:test) sbt.TestsFailedException: Tests unsuccessful

Test build #56885 has started for PR 12421 at commit c1d09d7.

Test build #56885 has finished for PR 12421 at commit c1d09d7.

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

Merged build finished. Test PASSed.

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

Hi, @hvanhovell .
Now, it's ready for review again.
Thank you so much. I couldn't fix this without your help.

Test build #56935 has started for PR 12421 at commit 45beddd.

Test build #56935 has finished for PR 12421 at commit 45beddd.

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

Merged build finished. Test PASSed.

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

More test cases are added.

Rebased.

Test build #56968 has started for PR 12421 at commit 14fe6b9.

Test build #56968 has finished for PR 12421 at commit 14fe6b9.

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

Merged build finished. Test PASSed.

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

Hi, @hvanhovell .
Could you review this PR when you have some time?

Hi, @rxin .
Could you review and merge this PR?
This fixes RuntimeException error for Window aggregation queries.

@dongjoon-hyun I am gonna get to this today.

@dongjoon-hyun The PR is in pretty good shape. I left a few small comments/questions.

Test build #57147 has started for PR 12421 at commit 152c6c2.

Test build #57149 has started for PR 12421 at commit b2ac335.

@hvanhovell .
Thank you for squeezing your precious time for reviewing my PR several times.

Test build #57147 has finished for PR 12421 at commit 152c6c2.

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

Merged build finished. Test PASSed.

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

Test build #57149 has finished for PR 12421 at commit b2ac335.

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

Merged build finished. Test PASSed.

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

LGTM

Merging to master. Thanks!

Thank you so much, @hvanhovell !


Could MEF performance suffer from ProfileOptimization.StartProfile()?

Since vs-mef is quite reflection heavy, I'm wondering if it is generally a good (or bad) idea to call the ProfileOptimization.StartProfile() method before building the graph (or loading it from disk).
ProfileOptimization.SetProfileRoot(path);
ProfileOptimization.StartProfile("profile");

I will try to benchmark this myself, but I thought it would be good to hear your thoughts since you have greater understanding of how your library works.

Thanks for your inputs. I confirmed that this API has a positive effect on the composition. At least in our application.


Error in RequestIntegration when HTTP header name is numeric

How do you use Sentry?
Self-hosted/on-premise
Version
3.2.0
Steps to Reproduce

Use latest sentry-laravel @ 2.13.0.
Send a query that use a numeric HTTP header such as curl -v -H "123: test" https://www.example.com
Notice an error in PHP error log (see below).

Expected Result
No error is produced in PHP error log.
Actual Result
From PHP error log:
PHP Fatal error:  Uncaught TypeError: strtolower(): Argument #1 ($string) must be of type string, int given in /data/site/vendor/sentry/sentry/src/Integration/RequestIntegration.php:179
Stack trace:
#0 /data/site/vendor/sentry/sentry/src/Integration/RequestIntegration.php(179): strtolower(1)
#1 /data/site/vendor/sentry/sentry/src/Integration/RequestIntegration.php(157): Sentry\Integration\RequestIntegration->sanitizeHeaders(Array)
#2 /data/site/vendor/sentry/sentry/src/Integration/RequestIntegration.php(115): Sentry\Integration\RequestIntegration->processEvent(Object(Sentry\Event), Object(Sentry\Options))
#3 /data/site/vendor/sentry/sentry/src/State/Scope.php(370): Sentry\Integration\RequestIntegration->Sentry\Integration\{closure}(Object(Sentry\Event), Object(Sentry\EventHint))
#4 /data/site/vendor/sentry/sentry/src/Client.php(264): Sentry\State\Scope->applyToEvent(Object(Sentry\Event), Object(Sentry\EventHint))
#5 /data/site/vendor/sentry/sentry/src/Client.php(147): Sentry\Client->prepareEvent(Object(Sentry\Event), Object(Sentry\EventHint), Object(Sentry\State\Scope))
#6 /data/site/vendor/sentry/sentry/src/Client.php(139): Sentry\Client->captureEvent(Object(Sentry\Event), Object(Sentry\EventHint), Object(Sentry\State\Scope))
#7 /data/site/vendor/sentry/sentry/src/State/Hub.php(138): Sentry\Client->captureException(Object(TypeError), Object(Sentry\State\Scope), Object(Sentry\EventHint))
#8 /data/site/app/Exceptions/Handler.php(151): Sentry\State\Hub->captureException(Object(TypeError))
#9 /data/site/app/Exceptions/Handler.php(87): App\Exceptions\Handler->reportWithSentry(Object(TypeError))
#10 /data/site/vendor/laravel/framework/src/Illuminate/Foundation/Bootstrap/HandleExceptions.php(191): App\Exceptions\Handler->report(Object(TypeError))
#11 /data/site/vendor/laravel/framework/src/Illuminate/Foundation/Bootstrap/HandleExceptions.php(259): Illuminate\Foundation\Bootstrap\HandleExceptions->handleException(Object(TypeError))
#12 [internal function]: Illuminate\Foundation\Bootstrap\HandleExceptions->Illuminate\Foundation\Bootstrap\{closure}(Object(TypeError))
#13 {main}
  thrown in /data/site/vendor/sentry/sentry/src/Integration/RequestIntegration.php on line 179

Thanks for bringing this up 👍
I opened #1370

Thank you @cleptric!
Once the next release of sentry-php if produced, can you also release a new version of sentry-laravel?
Thanks!

@boutetnico You can just upgrade sentry-laravel, and the just released sentry-php version 3.8.1 will be installed.


Feat: Added showing admin which user marked favorite to which post successfully issue 500

Pull Request Resolves [ #500 ]
Title: Added showing admin which user marked favorite to which post successfully.
Description

In backend, I created the controller and routes for fetching the favorite posts by users.
In admin, I created a interface for favorite posts and created Favorites.tsx in which i am fecrhing the above api in the table.
The design is responsive.

Related Issues
Fixes #500
Changes Made
New feature: Added showing admin which user marked favorite to which post.
Screenshots

I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.
Thank You for this contribution.

@Ultimateutkarsh11 please add level 3 after complete evaluation 👍 because of frontend(admin) + backend integration !

Resolve merge conflicts

@VaibhavArora314 @Ultimateutkarsh11 Done 👍


Missing entries in history of documentation

The current version of the testsuites management is 0.7.0 (packageversion.py). But the latest entry in history of the PDF documentation is 0.4.0
This causes a gap in the history that is not nice. In general I would prefer to have for every increased major number and also for every increased minor number (patch number not relevant enough) a corresponding entry in history. The history should give information about what happened in the software. If this is not aligned a history makes no sense. Please try to recover the corresponding milestones for missing entries 0.5.0, 0.6.0 and 0.7.0.

@namsonx : What is the status of this issue?

Hello Holger,
I traced back the old commits and updated the history.
Thank you,
Son

Ready for sprint 0.13.0 [09.08.24](jsonpreprocessor / tsm)


How to obtain current search string inside keypress listener?

Say I want to achieve behavior similar to Tagging without multiple in this plunker. I assume I would have to listen to keypress event and add new object to the list each time key is pressed while removing old shorter version of it, right?
But how can I get hold of current search string inside my keypress handler? Since my model isn't plain string but an object and untill any option is selected it's value remains empty.

I am doing something similar using the select search to filter a different list of items from the text. I use data-refresh="getChoices($select.search)" on the ui-select-choices and in my getChoices method I filter my list of options and use the text to filter my secondary list.
I imagine you could use it in a similar way to get your desired result.

thanks for your help, clydetacoma!


Update basic service model with demo links

The links stats-demo.virtual-labs.ac.in/test-stats and stats-demo.virtual-labs.ac.in/stats need to be described and mentioned in the model.

The model was updated with the demo links and their description in the basic analytics service.


Connection lost

I try to connect my TK4 session (which I can connect OK by e.g. dm3270.jar) by tnz.
I get there following error:
Session: NONE> goto <IP_ADDRESS>:3270
Traceback (most recent call last):
File "C:\Users\pc\anaconda3\lib\site-packages\tnz\tnz.py", line 3689, in __connect
await loop.create_connection(protocol, host, port,
File "C:\Users\pc\anaconda3\lib\asyncio\base_events.py", line 1065, in create_connection
raise exceptions[0]
File "C:\Users\pc\anaconda3\lib\asyncio\base_events.py", line 1050, in create_connection
sock = await self._connect_sock(
File "C:\Users\pc\anaconda3\lib\asyncio\base_events.py", line 961, in _connect_sock
await self.sock_connect(sock, address)
File "C:\Users\pc\anaconda3\lib\asyncio\selector_events.py", line 500, in sock_connect
return await fut
File "C:\Users\pc\anaconda3\lib\asyncio\selector_events.py", line 535, in _sock_connect_cb
raise OSError(err, f'Connect call failed {address}')
ConnectionRefusedError: [Errno 10061] Connect call failed ('<IP_ADDRESS>', 3270)
SESLOST = '192
FYI, I see for a short time the Host-window, but the connection is immediatly lost with the error above.

it might not accept secure connections. try this:
Session: NONE> set SESSION_SSL 0
Session: NONE> goto <IP_ADDRESS>:3270

Yes, that was it!
Thank you very much.
Regards
Gabor
On Wed, Nov 30, 2022 at 8:35 PM Neil Johnson @.***>
wrote:

it might not accept secure connections. try this:
Session: NONE> set SESSION_SSL 0
Session: NONE> goto <IP_ADDRESS>:3270
—
Reply to this email directly, view it on GitHub
https://github.com/IBM/tnz/issues/57#issuecomment-1332649355, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQEP3XOTCOQC5RC5UFN5DTWK6UATANCNFSM6AAAAAASPVPPTQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Example request code not working

Hi there,
I just came across your software and it's really something I would like to use in our testing environment.
I'm just trying something simple and it's not working, I'm new to node.js and I have not done JS in a while, so please I apologize if this is not the right forum to ask this.
I'm just running your example to send an option request out, and I get an error when I run the script. can you just tell me if it's a problem with the example code ..
const Srf = require('drachtio-srf');
const srf = new Srf();
const assert = require('assert');
srf.connect({
host: '<IP_ADDRESS>',
port: 9022,
secret: 'cymru'
});
srf
.on('connect', (err, hostport) => {
console.log(connected to a drachtio server listening on: ${hostport});
})
.on('error', (err) => {
console.log(Error connecting to drachtio server: ${err});
process.exit(1);
});
<EMAIL_ADDRESS>{
method: 'OPTIONS'
}, (err, req) => {
// req is the SIP request that went out over the wire
req.on('response', (res) => {
console.log(received ${res.status} response to our OPTIONS request);
});
});
output
[root@SIPp1 geo-test]# node app.js
assert.js:339
throw err;
^
AssertionError [ERR_ASSERTION]: The expression evaluated to a falsy value:
assert(typeSocket(socket))
at sockPort (/root/dev/geo-test/node_modules/drachtio-srf/lib/drachtio-agent.js:25:3)
at DrachtioAgent._getDefaultSocket (/root/dev/geo-test/node_modules/drachtio-srf/lib/drachtio-agent.js:505:56)
at DrachtioAgent._normalizeParams (/root/dev/geo-test/node_modules/drachtio-srf/lib/drachtio-agent.js:254:21)
at DrachtioAgent.request (/root/dev/geo-test/node_modules/drachtio-srf/lib/drachtio-agent.js:328:25)
at Function.proto.(anonymous function) [as request] (/root/dev/geo-test/node_modules/delegates/index.js:40:31)
at __x (/root/dev/geo-test/node_modules/drachtio-srf/lib/srf.js:923:19)
at Srf.request (/root/dev/geo-test/node_modules/drachtio-srf/lib/srf.js:927:7)
at Object.<anonymous> (/root/dev/geo-test/app.js:22:5)
at Module._compile (internal/modules/cjs/loader.js:776:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)

[root@SIPp1 geo-test]# s

wait until you get the 'connect' event before you try to send your request

That worked ... guess I have a lot to learn about programing with nodejs
Thank you for the quick response.

Excellent!  Don't worry, you will love node.js


Filter data and use relative abundance

Hi,
Thank you for developing SNB algorithm，  is it possible to add the function of inputting relative abundance to calculate SNB? Because our data may be derived from metagenomic or other, and not count abundance, such as relative abundance, TPM,....
Looking forward to your help.
Thanks,
Qiusheng WU

Hi @wqssf102,
Thanks for your question! Currently we do not allow for this because in the paper we used an absolute abundance cut-off (of 5 reads) for the pairwise dissimilarity calculations. However, I have included the option for a relative abundance cut-off for the pairwise dissimilarity calculations in this code (with the --c2 / --pairwise-comparisson-cutoff set < 1) so I can implement allowance for relative abundance tables as well. I put it on the to-do list.
In the meantime, our data is also metagenomic, and the 'absolute' count refers to number of sequenced reads. So even though this is compositional data, what the script needs is read counts and total number of taxonomically annotated reads per sample in the header. This is what for example TPM is also calculated from. Could you structure your tables in that way?
As a side note, I have briefly tested a relative --c2 cut-off and results seem to be quantitatively very similar to an absolute abundance cut-off at least for our dataset.
Best wishes,
Bastiaan

Thanks for your answer, when I use --c2 to control the relative abundance, what does the table header of the input dataset look like?

I have uploaded a new version of the script (v0.2) that allows for relative abundance tables as an input. The script assumes whether the input file is a count table or relative abundance table based on the header. Read count tables should have total number of taxonomically annotated reads in the header, and relative abundance tables just the sample names. It throws an error if the content does not match the header.
If a relative abundance table is supplied the --c2 / --pairwise_comparisson_cutoff has to be set < 1. With a read count table, --c2 / --pairwise_comparisson_cutoff can be set to any value. Note that in the paper, we used a cut-off of 5 reads.

Great, thank you very much!


Add security mailing list to readme and releases

Signed-off-by: Michael Crosby<EMAIL_ADDRESS>Closes #1337

Codecov Report

Merging #1436 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1436   +/-   ##
======================================
  Coverage    40.8%   40.8%
======================================
  Files          23      23
  Lines        2924    2924
======================================
  Hits         1193    1193
  Misses       1453    1453
  Partials      278     278

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 cf09e32...b5b3b10. Read the comment docs.

LGTM

As a first step, all maintainers should have received an invite to this group.

@mlaventure fixed

Yeah, I think we can add detail re: guidelines as a follow-on.


Remove some outdated scripts

This goes along with https://github.com/AppScale/appscale-tools/pull/758.

http://ocd.appscale.net:8080/job/Daily Build/6635/


[TwigComponent] Handling boolean values with component HTML syntax

Hey,
I've just had some issues working with booleans especially with nested components. I came to the conclusion that it only works this way:
<twig:ComponentA aBoolProp />

<twig:ComponentB aBoolProp={{ compoA.aBoolProp ? '1' : '0' }} />

I've somehow started using regular HTML style, but using aBoolProp="true" and aBoolProp="false" will always evaluate to true at the end and this was a easy to oversee. Using :aBoolProp="false" works, but this cannot be composed depending on another value.
So... i'd be great if booleans could somehow be "parsed" from the template.
Cheers
Matthias

Hum interesting your components are anonymous components or only class component?

So can you show an example of what you want to do precisely ?
Not sure if you want to pass "false" as a string, or if something does not work with booleans for you

Is something not working there ?
<twig:ComponentB aBoolProp="{{ compoA.aBoolProp }}" />

Hmm... I have to check this. I could be that I missed to test the simplest solution. :/

The one thing you may not know, but forces us to keep "false" as a string in some cases, is that in HTML there is no difference between

checked="true"
checked
checked="false"

They are all considered "checked"

Now i present you all the cases and i'm pretty sure you'll find one that you're fine with :)


Loading home page is too late

Loading home page takes too much times. Requires to fix it.

Not only homepage, but also profile page is loaded lately. In local env it is very fast but in deploy it takes too much time. It seems related to deployment process.

@SWE574-Fall2023-Group1/backend team has done necessary actions to speed up the home page 🚀
@aykutkantas Could you check again please?

It looks so good, thanks team :)


Close eye detection

Before proposing this issue, plz search the existed issues by your keywords first.
Describe the bug
It looks the current model use 10 expressions, I found if close eye, there is no expression for that.
To Reproduce
Expected behavior
I hope close eye could be detected
Screenshots
Platform:

macOS, Linux, or Windows

Additional context
Add any other context about the problem here.

It relies on eye-closing samples. Labeled or unlabeled may benefit. You can refer to MGCNet and other works.


[AppService] functionapp: Added support for in preview dotnet-isolated runtime.

Description

This is mainly a stacks update to add support for the new and in preview dotnet-isolated runtime. This is different from dotnet because it runs in a separate process from the functions host. I also had to add a bit of temporary custom code because of a known bug that's being fixed for creating dotnet-isolated linux consumption apps (linuxFxVersion needs to be blank in this scenario). It will be removed as soon as the bug is fixed.
Testing Guide

I added 3 news tests to make sure the CLI is able to create dotnet-isolated apps for a variety of scenarios (windows consumption, linux dedicated and consumption).
test_functionapp_on_linux_app_service_dotnet_isolated
test_functionapp_consumption_linux_dotnet_isolated
test_functionapp_windows_runtime_dotnet_isolated

History Notes

[AppService] az functionapp create: Added preview support for creating dotnet-isolated apps.

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

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

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

@qwordy -- sorry for the late notice but is there anyway we can get this in for S183? Need this for ignite. Let me know if you need anything else from me, thanks!

@yungezz for awareness

/azp run

Oh I can’t run the pipeline :/ does it run automatically or does someone have to kick it off?

/azp run

So I'm not really sure what's failing here... I looked into the exceptions being thrown and I'm seeing
2021-02-24T07:31:32.0407264Z vcr.errors.CannotOverwriteExistingCassetteException: Can't overwrite existing cassette ('/opt/hostedtoolcache/Python/3.6.12/x64/lib/python3.6/site-packages/azure/cli/command_modules/appservice/tests/latest/recordings/test_functionapp_windows_runtime_dotnet_isolated.yaml') in your current record mode ('once').
2021-02-24T07:31:32.0408927Z No match for the request (<Request (GET) https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.Storage/storageAccounts/clitest000002?api-version=2021-01-01>) was found.
2021-02-24T07:31:32.0409762Z Found 1 similar requests with 1 different matcher(s) :
2021-02-24T07:31:32.0410017Z
2021-02-24T07:31:32.0411017Z 1 - (<Request (GET) https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.Storage/storageAccounts/clitest000002?api-version=2019-06-01>).
2021-02-24T07:31:32.0411992Z Matchers succeeded : ['method', 'scheme', 'host', 'port', 'path']
2021-02-24T07:31:32.0412403Z Matchers failed :
2021-02-24T07:31:32.0413008Z _custom_request_query_matcher - assertion failure :
2021-02-24T07:31:32.0413389Z None

But all these tests are passing locally. The weird thing is none of the recording yaml files even have a mention of https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.Storage/storageAccounts/clitest000002?api-version=2021-01-01 with that api-version. They all use the api-version=2019-06-01 mentioned as a similar request found. These are also tests that were copy/pasted from older tests that pass fine with just a few values adjusted.
Is this a recent change or is there some kind of config I need to set? I tried rebasing and re-recording all the tests locally to see if that helps at all.

@qwordy can you azp run this again? I may have fixed it.
I saw this commit: https://github.com/Azure/azure-cli/commit/fffde7ec2b08501805b03e6f15a17d5cb98f159f#diff-05ee69a5b7cf47cbda6c6280aefb2002174a518dc091d2bc6a899ada9145d39e
and I'm thinking I might not have been up to date when I ran the tests. I noticed all the functionapp tests now have calls with api-version=2021-01-01 after that commit so I re-ran them after rebasing and now they all have calls to that api-version

So I'm not really sure what's failing here... I looked into the exceptions being thrown and I'm seeing
2021-02-24T07:31:32.0407264Z vcr.errors.CannotOverwriteExistingCassetteException: Can't overwrite existing cassette ('/opt/hostedtoolcache/Python/3.6.12/x64/lib/python3.6/site-packages/azure/cli/command_modules/appservice/tests/latest/recordings/test_functionapp_windows_runtime_dotnet_isolated.yaml') in your current record mode ('once').
2021-02-24T07:31:32.0408927Z No match for the request (<Request (GET) https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.Storage/storageAccounts/clitest000002?api-version=2021-01-01>) was found.
2021-02-24T07:31:32.0409762Z Found 1 similar requests with 1 different matcher(s) :
2021-02-24T07:31:32.0410017Z
2021-02-24T07:31:32.0411017Z 1 - (<Request (GET) https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.Storage/storageAccounts/clitest000002?api-version=2019-06-01>).
2021-02-24T07:31:32.0411992Z Matchers succeeded : ['method', 'scheme', 'host', 'port', 'path']
2021-02-24T07:31:32.0412403Z Matchers failed :
2021-02-24T07:31:32.0413008Z _custom_request_query_matcher - assertion failure :
2021-02-24T07:31:32.0413389Z None

But all these tests are passing locally. The weird thing is none of the recording yaml files even have a mention of https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.Storage/storageAccounts/clitest000002?api-version=2021-01-01 with that api-version. They all use the api-version=2019-06-01 mentioned as a similar request found. These are also tests that were copy/pasted from older tests (that pass fine) with just a few values adjusted.
Is this a recent change or is there some kind of config I need to set? I tried rebasing and re-recording all the tests locally to see if that helps at all.

@Zunli Hu upgraded API version of storage in Azure CLI recently. So, it can't find matching request in yaml files. Just rerun the failed test cases in --live mode. Remember to pull latest code from dev.
run "azdev test testcase --live" or set environment variable "AZURE_TEST_RUN_LIVE=True" to run live test.

@qwordy can you azp run this again? I may have fixed it.
I saw this commit: fffde7e#diff-05ee69a5b7cf47cbda6c6280aefb2002174a518dc091d2bc6a899ada9145d39e
and I'm thinking I might not have been up to date when I ran the tests. I noticed all the functionapp tests now have calls with api-version=2021-01-01 after that commit so I re-ran them after rebasing and now they all have calls to that api-version

@gzuber Sure. The checks should start automatically. Github is not stable.

/azp run

It looks like it failed test_postgres_flexible_server_vnet_mgmt_supplied_subnetid (command_modules.rdbms.tests.latest.test_rdbms_flexible_commands.FlexibleServerVnetMgmtScenarioTest)
I'm seeing rdbms tests failing on dev intermittently like here:
https://github.com/Azure/azure-cli/runs/1968101738
https://github.com/Azure/azure-cli/runs/1966927311
https://github.com/Azure/azure-cli/runs/1966786703
I didn't make any changes that should affect these tests and I'm pretty close to being up to date with dev (other than commits less than a day old). Are these just flaky tests that we can get an override to merge for? Or do we have to re-run until they pass?
I went ahead and rebased on dev again to get the latest commits if we can try running again

/azp run

@gzuber rdbms has nothing to do with your code. I don't think it is your fault. We will have a look so that it will not block your PR.

Oh good it passed! This should be good to go -- Thanks so much for all your help!


Reset middlewares

Hello! Your package is very cool! But, please add the ability to reset middlewares in nested route.
Example:
path: /api/auth
routes.js // middlewares check jwt token
path: /api/auth/login
reset.js // reset or reset and set new middlewares
Or: add routes.js options for reset or add middlewares.
Thanks!

Cheers!
Indeed, there's a need to avoid applying middleware to all routes inside a folder.
I'm not quite sure if it should be the responsibility of the library, though, at least for now. I'd prefer leaving it to the middleware to decide they do not need to run for a specific route. For example, by wrapping them as such:
// /api/auth/routes.js or maybe /api/routes.js
module.exports = except('/api/auth/login', checkJWTToken)

/**
 * Creates a middleware that runs the given list of `middlewares`
 * (or single middleware) unless the request path matches the given `path`
 * @param path String
 * @param middlewares Function|Array<Function>
 * @return Function
 */
function except(path, middlewares) {
	// Automatically wrap arrays of middlewares
	if (Array.isArray(middlewares)) {
		return middlewares.map(middleware => except(path, middleware))
	}

	return function(req,res,next) {
		// Implementation might be a bit naive here
		// update to suit your own needs
		if (req.path !== path) {
			return middlewares(req,res,next);
		}
		next();
	}
}

function checkJWTToken(req, res, next) {
	// ...
}

Guess there's a slight overhead as the check would happen for each middleware. Not sure if Express or a library provides an API that allows to combine multiple middlewares into one:
module.exports = except(path, group(mw1,mw2,mw3))


WISE icon as SVG

Create an SVG version of WISE logo

Both versions added. Let me know if you need more versions.
https://github.com/noisy-witness/steem-wise-voter-page/pull/30


Question: Is it possible to fetch an array of devices based on a number of UUIDs?

Hi,
I was scanning through the meshblu documentation amd Readme.md file for examples on how I can query devices and subscribing to their events through a set of UUIDs and it seems like there is no way to do this? Please correct me if I am wrong on this.
Not particularly important as I have got a work around for it by simply fetching all devices owned by the owner and then manually filtering through them.
Cheers,
Sam

We have an API endpoint called MyDevices (https://meshblu-http.readme.io/docs/mydevices).  This endpoint returns all UUIDs that belong to an "owner" UUID.  You could then iterate through this array and subscribe to each of the UUIDs.


fix: use local volume instead of emtpy dir for vector data

Description
This PR uses a local volume for the Vector DaemonSet instead of a emptyDir volume

Thanks for the pull request, @Ian2012!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate.

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


procps-ng: Use official release tarballs, fix version.

Also no need to re-generate config bits,
which is good since we encountered an error when trying to do so.
Fixes #39538.

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Failure on x86_64-darwin (full log)
Attempted: procps-ng
Partial log (click to expand)
cannot build derivation '/nix/store/3cjxnhpgk2j1jrm8kjrrasdr7q57flpn-xcbuild-wrapper-0.1.2-pre.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/kqd1bgiq9l741j2aws9ipg50wsh3yw5y-developer_cmds-osx-10.11.6.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/qa1ilg0ip1ix3rjp2rv5nz7580g1w2a0-libutil-osx-10.11.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/g4583f74f84pvyda1dpjd3bsjdh2lys2-Librpcsvc-osx-10.11.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/dhb79qbz6dd0dr7vyw43gsfhq1axzh4b-top-osx-10.11.6.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/w1mcdflbs6k04gsq02n21106jwysdrf6-system_cmds-osx-10.11.6.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/s0lmzgg4qzqncswskvwbm2h1g5bdy1xj-top.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/b8wvgczd67i3sqkvqiy0p8441yac0423-sysctl.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/qgwprmicg3mchw94ynrdizrahxfwqmb0-procps-compat.drv': 6 dependencies couldn't be built
error: build of '/nix/store/qgwprmicg3mchw94ynrdizrahxfwqmb0-procps-compat.drv' failed

Success on aarch64-linux (full log)
Attempted: procps-ng
Partial log (click to expand)
shrinking /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/bin/free
shrinking /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/bin/ps
shrinking /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/lib/libprocps.so.6.1.0
gzipping man pages under /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/share/man/
strip is /nix/store/i3k26ygq416sr5ng9f87041f3zcid9di-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/lib  /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/bin  /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/sbin
patching script interpreter paths in /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14
checking for references to /build in /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14...
moving /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/sbin/* to /nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14/bin
/nix/store/ray8qyjnz8kxnbvlykrphinwa8n7nl1x-procps-3.3.14

Success on x86_64-linux (full log)
Attempted: procps-ng
Partial log (click to expand)
shrinking /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/bin/watch
shrinking /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/bin/top
shrinking /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/sbin/sysctl
gzipping man pages under /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/share/man/
strip is /nix/store/jfnxjmhyjin5h5mz8w26g9mq0x8gy5kg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/lib  /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/bin  /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/sbin
patching script interpreter paths in /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14
checking for references to /build in /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14...
moving /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/sbin/* to /nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14/bin
/nix/store/xyg58x7mb11y4aidq82ix0i30qxywal3-procps-3.3.14


Add initialVisibleMonth fn prop to DayPicker

Related to https://github.com/airbnb/react-dates/issues/17
Adds a new prop to DateRangePicker, SingleDatePicker, and DayPicker
that passes a fn to DayPicker. On the initial focus, that fn will be called
and set the initial month to the returned moment object.

this PR is on hold until https://github.com/airbnb/react-dates/pull/32

There's some larger issues about how focused is passed to the different components I need to understand better, closing for now.


qemu-utils: remove qemu dependency

Description of changes
qemu-utils was pulling qemu which is a 900MB dependency. By removing
reference to it (unneeded), we're saving space on our deployments.
qemu-utils is a dependency of cloud-utils
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I'm always a bit scared to use remove-references-to, because who knows what those references are used for.  I'd like to spend a bit of time looking into where this dependency came from before we use the nuclear option.  We should also use disallowedRequisites to ensure it doesn't come back in future.

Okay, looks like it's the same as in https://github.com/NixOS/nixpkgs/pull/162473#issuecomment-1058517696, so it should be fine to use remove-references-to.  Please add disallowedRequisites = [ qemu ]; and then this LGTM.

@alyssais wow, thank you so much for digging why there was a reference! I couldn't figure it out.


I got Unexpected key(s) in state_dict error

Unexpected key(s) in state_dict: "model.logit_scale", "model.text_hidden_fcs.0.0.bias", "model.text_hidden_fcs.0.0.weight", "model.visual_embeddings.bias", "model.visual_embeddings.weight", "model.visual_fc.bias", "model.visual_fc.weight", "ret_input_embeddings.weight".
I tried to fine tune your multi-modal model with my own dataset using your pre-trained data but I got the error above with activate --resume. After I installed requirements.txt and executed main.py then I installed many modules to figure out several no module errors.  With this errors above, to check the environment, I tried to install requirements.txt again and got several  version related errors. is that related with each other?

I think this is the same issue as https://github.com/kohjingyu/fromage/issues/29#issuecomment-1740491866. Can you see if that helps?

Yes, I figured out it. Thanks for your advise!


bugfix(FDS-279)  Make sure empty class only present for empty or loading widgets

This also cleans up the logic for when to display a component or not into the widget itself where we decide if we should even render children

Messages

:book:

packages/cascara/src/ui/Dashboard/DashboardDX.fixture.js: This fixture has been changed.

  Generated by :no_entry_sign: dangerJS against 9f67c7360b218c8bd7e75db2965dda7bf83667d9


Remove one malloc/free from compression

By making the buffer part of the zstd context structure, one malloc/free
can be removed from the compression.

Good catch Xavier.
This was supposed to be introduced into next version of zstd, but since your patch is ready, let's use it now ;)


Add boundary feature to the drag gesture.

Inspired by the review on #146182.
This PR adds boundary feature to the drag gestures, including MultiDragGestureRecognizer and DragGestureRecognizer. The GestureDetector widget will also benefit from this.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Reimplemented and ready for review. @Renzo-Olivares

Thank you @yiiim, I will take a look this week.

@Renzo-Olivares Please do not forget this here. 👻

@Renzo-Olivares Hello, thank you for your comments. It seems I got a bit off track. I have re-implemented this and greatly simplified the code. I apologize for the additional work this has caused for your review.

Hi @yiiim, sorry for the delay on this, it is still in my queue. Will get to is as soon as I can.

We don't just use this within the framework; when users use this as a drag boundary, they might use OffsetBoundaryProvider.

But I think it can be generalized into "dragging a box inside another box". Even when the object being dragged is a non-box, they are probably still calculated using their bounding box. With this simplification OffsetBoundaryProvider is just a box with zero size, and we'll no longer need the delegate.

I noticed that in iOS, UIButton can be dragged outside its bounds when held, and an animation is triggered when it exceeds a certain range. If we want to implement similar functionality for CupertinoButton, we might need an OffsetBoundaryProvider.
https://github.com/user-attachments/assets/b5a43e19-53e2-4d29-b937-b2f194b5e94f

Yeah, this behavior has been filed in https://github.com/flutter/flutter/issues/91581. I don't think it's related to drag boundary though, since it doesn't post any "limit" to the region that an object can move to, instead it only needs the button to react to the distance between the pointer and the button.

In the earliest design, I provided DragBoundary directly to DragGestureRecognizer, and it only served drag gestures. After some discussions, DragBoundary was decoupled from DragGestureRecognizer. When it was completely decoupled, I found that it was no longer limited to dragging, so I made it an independent feature. Its function is to provide a range that can be used to adjust the position of a certain rectangle within that range. The two important points here are the range and the rectangle position. It can be determined that the range is the widget's range, while the rectangle position can be more abstract and can be other shapes. Therefore, I designed ShapeBoundary. As of now, I might be focusing too much on the shape, but what is actually important is the position. Based on your suggestion, my understanding is reflected in the following code. Do you think my understanding is correct? (Most of my English comes from translation software, so please forgive any inappropriate expressions).
class BoundaryRegulatorFactory {
  BoundaryRegulatorFactory(this.boundary);
  final Rect boundary;

  BoundaryRegulator forRect(Size size) {
    return RectBoundaryRegulator(size, boundary);
  }
}

class RectBoundaryRegulator extends BoundaryRegulator {
  RectBoundaryRegulator(this.rectSize, this.boundary);
  final Size rectSize;
  final Rect boundary;
  bool isWithinBoundary(Offset position) {
    /// ...
  }

  Rect nearestPositionWithinBoundary(Offset position) {
    /// ...
  }
}

class BoundaryProvider extends InheritedWidget {
  const BoundaryProvider({required Widget child, Key? key}) : super(child: child, key: key);

  static BoundaryRegulatorFactory of(BuildContext context) {
    final InheritedElement? element = context.getElementForInheritedWidgetOfExactType<BoundaryProvider>();
    final RenderBox? rb = element!.findRenderObject() as RenderBox?;
    final Rect boundary = rb!.localToGlobal(Offset.zero) & rb.size;
    return BoundaryRegulatorFactory(boundary);
  }
}

My other idea is to keep BoundaryRegulator<T>. However, in the API design, use the following approach.
abstract class BoundaryRegulator<T> {
  bool isWithinBoundary(T shapePosition, {ShapePositionGlobalToLocal? globalToLocal});
  T nearestPositionWithinBoundary(T shapePosition, {ShapePositionGlobalToLocal? globalToLocal});
}

class _RectBoundaryRegulator extends BoundaryRegulator<Rect> {
  _RectBoundaryRegulator(this.boundary);
  final Rect boundary;
  @override
  bool isWithinBoundary(Rect shapePosition, {ShapePositionGlobalToLocal? globalToLocal}) {
    /// ...
  }
  @override
  Rect nearestPositionWithinBoundary(Rect shapePosition, {ShapePositionGlobalToLocal? globalToLocal}) {
    /// ...
  }
}

class BoundaryProvider extends InheritedWidget {
  const BoundaryProvider({required Widget child, Key? key}) : super(child: child, key: key);

  static BoundaryRegulator<Rect> forRectOf(BuildContext context) {
    final InheritedElement? element = context.getElementForInheritedWidgetOfExactType<BoundaryProvider>();
    final RenderBox? rb = element!.findRenderObject() as RenderBox?;
    final Rect boundary = rb!.localToGlobal(Offset.zero) & rb.size;
    return _RectBoundaryRegulator(boundary);
  }
}

I'm not very optimistic about passing in Offset. I think it's still a bit confusing. It's easier to understand if the input and return types are the same. Additionally, using generic input parameters can carry user data for future expansion.

@dkwingsmt  There might not be other cases that can use this, so I reverted back to DragBoundary. Regarding globalToLocal, I provided a more elegant solution, which you can check in the code. I think I have solved most of the issues (maybe?).

I'm not very optimistic about passing in Offset. I think it's still a bit confusing. It's easier to understand if the input and return types are the same. Additionally, using generic input parameters can carry user data for future expansion.

Good point!
Regarding your two ideas (I really appreciate the detailed explanation with examples!), yeah I was hoping for the first one, but the additional class does look clumsy. After much consideration I think your current API with forRectOf might be the best. Thank you.

Update: I've asked @Renzo-Olivares for final review.

Updated the suggested documentation. Additionally, I set useGlobalPosition to true by default. We should use global coordinates by default and only use local coordinates when it is confirmed to be within the boundary locally.


Missed logo

H3c Logo
Description

Urgency

[ ] Blocker
[v] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

Missed Logo