Can't run on nanobox: can't modify frozen Array

Trying to run this on nanobox.io.  At first I was getting the "TypeError: no implicit conversion of nil into String" error found on https://github.com/crypto-rb/rbnacl-libsodium/issues/25.  I then updated the Gemfile to fetch the latest revision from github, and now I get this error:
Using rbnacl-libsodium 1.0.16 from https://github.com/crypto-rb/rbnacl-libsodium.git (at c1ee4d6@c1ee4d6)
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

current directory:
/app/vendor/bundle/ruby/2.3.0/bundler/gems/rbnacl-libsodium-c1ee4d621b2e/ext/rbnacl
/data/bin/ruby -r ./siteconf20180417-984-14el4ip.rb extconf.rb
extconf.rb:22:in `<main>': can't modify frozen Array (RuntimeError)

extconf failed, exit code 1

Isolated repo here: https://github.com/steveklebanoff/libsodium-nanobox-example/blob/master/README.md

@tarcieri it looks like it's failing on this line: https://github.com/crypto-rb/rbnacl-libsodium/blob/master/ext/rbnacl/extconf.rb#L22
it looks like we are trying to append to a frozen array.  this was introduced by https://github.com/crypto-rb/rbnacl-libsodium/commit/fb8bfb6f20151dfa4fd6d9ee005d3fd6bdb00f62

With the .freeze removed,  this new error appears:
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

current directory:
/app/vendor/bundle/ruby/2.3.0/bundler/gems/rbnacl-libsodium-6e720c884d5e/ext/rbnacl
/data/bin/ruby -r ./siteconf20180418-987-1rpbt3t.rb extconf.rb
-- sh -c "./configure
--prefix=/app/vendor/bundle/ruby/2.3.0/bundler/gems/rbnacl-libsodium-6e720c884d5e/tmp/x86_64-linux/stage/vendor/libsodium/dist
--host=x86_64-debian-linux"
sh: 1: ./configure: not found
extconf.rb:8:in `sys': ERROR: 'sh -c "./configure
--prefix=/app/vendor/bundle/ruby/2.3.0/bundler/gems/rbnacl-libsodium-6e720c884d5e/tmp/x86_64-linux/stage/vendor/libsodium/dist
--host=x86_64-debian-linux"' failed (RuntimeError)
from extconf.rb:36:in `block in <main>'
from extconf.rb:34:in `chdir'
from extconf.rb:34:in `<main>'

extconf failed, exit code 1

Gem files will remain installed in
/app/vendor/bundle/ruby/2.3.0/bundler/gems/rbnacl-libsodium-6e720c884d5e for
inspection.
Results logged to
/app/vendor/bundle/ruby/2.3.0/bundler/gems/extensions/x86_64-linux/2.3.0/rbnacl-libsodium-6e720c884d5e/gem_make.out

Building from github doesn't work. The git submodule for libsodium doesn't have the configure script, so it's going to fail.
I'll post a comment on your other issue regarding that.

FYI: I plan on deprecating this gem. Therefore I am closing all other open issues.
Please discuss any concerns on #29.


Feature/add integration tests and api fixes

Description
Fixing bunch of stuff related to new api docs, adding also docker-compose(s) and initial version for integration tests.
Related issues
Partially fixes #49 #52
Fixes #25
Type of change

[x] Bug fix (non-breaking change which fixes an issue), refactoring

Changes Made

dateCreated and dateModified is added for all objects
date results are parsed for response jsons
/submit endpoint response contains accession ids for all files sent through it
/schema endpoits returns answer that corresponds to api docs (title, details, url)
mongodb _id is removed from api responses
integration tests for  POST/GET with single objects is added. Some queries are also tested. tests are ran against production version dockerfile asyncronously, so if wanted, one can try to to submit big amount of objects to stress test endpoints.
new development docker-compose and dockerfile is added, allows quick development setup
refactored some functionalities:

parser module is now purely for xml-json parsing. Previosly it also added accessionIds and publishDates, but its more logical to handle these in operators (in future accessionIds are needed to be added to json submissions too).
handlers module had some copypaste stuff between /submit and /object/{schema} -endpoints, those are moved to operators module.

Also some other small fixes, see git history for more details.
Testing

[x] Unit Tests
[x] Integration Tests

@blankdots @csc-jm PR is almost ok. Frontend master doesn't have url routing, so integration tests were failing. Had to change them to run against development version of backend (which doesn't include frontend build).
However, now there's a new bug with aiohttp, not sure what. Would appriciate if someone would help debugging this:

https://github.com/CSCfi/metadata-submitter/blob/feature/add-integration-tests-and-db-fixes/tests/integration/run_tests.py
Here's the error: https://travis-ci.com/github/CSCfi/metadata-submitter/jobs/353108419

@otahontas sometimes sleep helps :D or waiting for the services to start.

getting annoyed with travis ... maybe we should switch to github actions

@otahontas sometimes sleep helps :D or waiting for the services to start.

Classic travis! Thanks for debugging

Fixed typos and added some tests, so coverage should be now ok again. @blankdots can you check if this is ok to merge


Check credentials without renewing them

Checklist

[X] I have looked into the README and have not found a suitable solution or answer.
[X] I have looked into the API documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Describe the problem you'd like to have solved
I would like to check current credentials without trying to renew them.
Describe the ideal solution
Add option to current credentials method to not refresh credentials even when they are expired.
Alternatives and current workarounds
No response
Additional context
No response

Hi @KlausJokisuo, thanks for raising this. Could you please tell us more about your use case for this?

@Widcket I would like to be able to check if the user has non expired refresh token. I will be working with offline first application, where I just need to check if refresh token is still valid and refresh the credentials when the app is back online.

The validity of the refresh token cannot be checked without attempting to renew the credentials. However, the CredentialsManager's hasValidCredentials() method does check for the presence of the refresh token, besides checking the validity of the access token: https://github.com/auth0/auth0-flutter/blob/main/auth0_flutter/EXAMPLES.md#check-for-stored-credentials

@KlausJokisuo does hasValidCredentials() work for your use case?

@Widcket Thanks 🙌, I thought that the hasValidCredentials only checks for credentials.


Page Nodes

Coverage remained the same at 79.828% when pulling 5b7408214d48789b33565ef19c97d4c7d6c7b258 on czpython:feature/3.5.x/tree-refactor into 07ad704c27ed9dc3c180eb4fb93501cfe323ac1d on divio:develop.

Coverage remained the same at 79.828% when pulling 850805e24e5f949185fe000c79d5aa9acaf444e9 on czpython:feature/3.5.x/tree-refactor into 07ad704c27ed9dc3c180eb4fb93501cfe323ac1d on divio:develop.

Coverage remained the same at 79.828% when pulling 32a09947c9f8ea5c976bd0ac5faaf7699195c883 on czpython:feature/3.5.x/tree-refactor into 07ad704c27ed9dc3c180eb4fb93501cfe323ac1d on divio:develop.

Coverage remained the same at 79.828% when pulling<PHONE_NUMBER>84b0ed12aaee9c660b244deb38a687 on czpython:feature/3.5.x/tree-refactor into 07ad704c27ed9dc3c180eb4fb93501cfe323ac1d on divio:develop.

Coverage remained the same at 79.828% when pulling 2bac13ea2389582ad95ad0aab76c3718a8109709 on czpython:feature/3.5.x/tree-refactor into 07ad704c27ed9dc3c180eb4fb93501cfe323ac1d on divio:develop.

Coverage remained the same at 79.828% when pulling 7694656a02c301776fb94f20c5537dba5026eaa4 on czpython:feature/3.5.x/tree-refactor into 07ad704c27ed9dc3c180eb4fb93501cfe323ac1d on divio:develop.

Coverage remained the same at 79.828% when pulling 3f5c8e122d83e89d4024d4860ff86591b37bf489 on czpython:feature/3.5.x/tree-refactor into 07ad704c27ed9dc3c180eb4fb93501cfe323ac1d on divio:develop.

Coverage remained the same at 79.828% when pulling 662c47520ae8547536375b309f8fe036d13f5c23 on czpython:feature/3.5.x/tree-refactor into 27071952c8523a93a73ce09022e371d081d42dd9 on divio:develop.

Coverage remained the same at 79.828% when pulling 05e4c94744d41fb314ba5f909ca0f6af942387e6 on czpython:feature/3.5.x/tree-refactor into 27071952c8523a93a73ce09022e371d081d42dd9 on divio:develop.

Changes Unknown when pulling 44eaf52ac18881bf144524d8eff50e03bc2d22c5 on czpython:feature/3.5.x/tree-refactor into ** on divio:develop**.

Changes Unknown when pulling f6c73dec2099a3bb37db7133aa36650a44aa3379 on czpython:feature/3.5.x/tree-refactor into ** on divio:develop**.

Coverage remained the same at 79.324% when pulling 5c649ffaa4b2eabfa84f382b1d570f8fa937ac2f on czpython:feature/3.5.x/tree-refactor into 6be269707edba10703c636f0e940cd2e4fa7f75c on divio:develop.

Coverage remained the same at 79.324% when pulling cf968af1e5b1d05c518b6fd5e7fbf7eb29052c36 on czpython:feature/3.5.x/tree-refactor into 6be269707edba10703c636f0e940cd2e4fa7f75c on divio:develop.

Coverage remained the same at 79.324% when pulling 2872ad2e4ac3731d91be538bb1867173375935c1 on czpython:feature/3.5.x/tree-refactor into 6be269707edba10703c636f0e940cd2e4fa7f75c on divio:develop.

Coverage remained the same at 79.274% when pulling 165730e7e30c4d50e9b87b0e96b02bf201bee635 on czpython:feature/3.5.x/tree-refactor into 58afb80bb4c021c709ef57e2a33d417eb39d4a8e on divio:develop.

@czpython I'm having a crazy week as usual. Hopefully the weekend will be better for me.

Coverage remained the same at 79.274% when pulling 1f6c109db7c2c256e197298eab0ecca4d7aa9f90 on czpython:feature/3.5.x/tree-refactor into 58afb80bb4c021c709ef57e2a33d417eb39d4a8e on divio:develop.

FE tests will be fixed once the FE branch for this feature is merged


Expose path Argument for the Reactive Web

Synopsis
The reactive web support simply does not report the path as an argument. This behavior is inconsistent with what we have in the Servlet stack.

Closed by #75


RDKTV-15390 : WPEFramework Crash  in Core::ResourceMonitorType

Reason for change: closing the Socket
Test Procedure: mentioned in the ticket.
Risks: None
Signed-off-by: Neeraj Sahu<EMAIL_ADDRESS>

Taken from the master! Oke to go.


Add newtag command

Closes ##113

Codecov Report

Merging #177 (175fd8f) into master (0067d2b) will increase coverage by 2.51%.
The diff coverage is 83.87%.

@@             Coverage Diff              @@
##             master     #177      +/-   ##
============================================
+ Coverage     66.16%   68.68%   +2.51%
- Complexity      416      446      +30
============================================
  Files            82       84       +2
  Lines          1398     1421      +23
  Branches        129      134       +5
============================================
+ Hits            925      976      +51
+ Misses          414      381      -33
- Partials         59       64       +5

Impacted Files
Coverage Δ

...oodrem/logic/commands/itemcommands/AddCommand.java
100.00% <ø> (ø)

...odrem/logic/commands/itemcommands/EditCommand.java
97.05% <ø> (ø)

...java/seedu/foodrem/logic/parser/FoodRemParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

.../parser/itemcommandparser/DeleteCommandParser.java
100.00% <ø> (ø)

...ic/parser/itemcommandparser/FindCommandParser.java
100.00% <ø> (ø)

...ain/java/seedu/foodrem/storage/JsonAdaptedTag.java
0.00% <0.00%> (ø)

...c/parser/tagcommandparser/AddTagCommandParser.java
66.66% <66.66%> (ø)

...drem/logic/commands/tagcommands/AddTagCommand.java
100.00% <100.00%> (ø)

...ain/java/seedu/foodrem/logic/parser/CliSyntax.java
83.33% <100.00%> (ø)

...in/java/seedu/foodrem/logic/parser/ParserUtil.java
95.23% <100.00%> (+0.23%)
:arrow_up:

... and 9 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM!


[FR] Allow tab completion in debug> mode

We should allow \theta<TAB> in debug> mode for checking the value of variables with Unicode variable names.

Fixed with https://github.com/JunoLab/Atom.jl/commit/f2c91ba46dc3a1e40c994753f54cef62a0898b3b.


Cannot install empty metapackage (mingw-w64-x86_64-autotools)

https://packages.msys2.org/package/mingw-w64-x86_64-autotools?repo=mingw64
Makefile:907: *** Unable to execute `bash -c "set -o pipefail && tar -tf 'cache/mingw-w64-x86_64-autotools-2022.01.16-1-any.pkg.tar.zst' --exclude='.*' | grep '[^/]$' | sed 's| |<|g'"`, exit code 1.  Stop.

grep exits with a status of 1 when it has no matches.

@stefantalpalaru Thanks for the report, should be fixed now. Please check.

It works now. Thanks!


HD Video Freeze after SD License Load with DRM - Fairplay

Have you read the FAQ and checked for duplicate open issues?
Yes
If the problem is related to FairPlay, have you read the tutorial?
Yes
What version of Shaka Player are you using?
4.7.7
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Demo App + Custom App
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Linux + ubuntu
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
None
What are the manifest and license server URIs?
What configuration are you using?  What is the output of player.getConfiguration()?
What did you do?
I configured Fairplay, in my manifest it starts with the video in SD and after a few seconds, depending on the quality of the internet, it switches to the HD video version. The SD video starts normally. I didn't use any extra options in player_.configure (just DRM for fairplay)
player.getNetworkingEngine().registerRequestFilter((type, request, context) => {
  if (type != shaka.net.NetworkingEngine.RequestType.LICENSE) {
    return;
  }

  const originalPayload = new Uint8Array(request.body);
  const base64Payload =
      shaka.util.Uint8ArrayUtils.toStandardBase64(originalPayload);
  const params = 'spc=' + encodeURIComponent(base64Payload);
  request.headers['Content-Type'] = 'application/x-www-form-urlencoded';
  request.body = shaka.util.StringUtils.toUTF8(params);
});

What did you expect to happen?
Shaka Player update itself the license between SD and HD automatically when is requested HD video and no stall.
What actually happened?
Fairplay - I request the SD video first to load quickly with DRM settings, and it works well. However, a few seconds later, when Shaka Player attempts to request the HD video, image stalls (audio keep running) because the license from the SD version is different from the HD version and won't update. I've tried using 'player.getNetworkingEngine().registerRequestFilter' to force an update of the license auto, but it didn't work.

other thing I tried as well was to send a new player.configure() without success
I taken this piece of code from this reply of @joeyparrish (https://github.com/shaka-project/shaka-player/issues/4827#issuecomment-1370249386)
player_.getNetworkingEngine().registerResponseFilter((type, response) => {

if (type != shaka.net.NetworkingEngine.RequestType.MANIFEST ||
response.originalUri != player_.getAssetUri()) {
// Not the master playlist.  Ignore it.
return;
}

const text = shaka.util.StringUtils.fromUTF8(response.data);
// Do something with the text, and call player.configure() with an appropriate config.
});

[Update 1] - On the Shaka Player demo, I observed the same behavior: if it starts with an SD video and switches to an HD video after a few seconds, the video stalls (this change happens automatically). If it doesn't switch and remains in SD quality, it keeps running.
[Update 2] - It seems to me that after a few minutes, Shaka Player automatically reverts to the SD version, making it work again because the main key provided was for SD

@mrshulk In order to help you, we need you to share the stream with the license server and the FairPlay certificate so we can test and see what is happening. Thank you!


Handle existing load balancer class

We frequently see an error like the following:
"error": "cannot upsert service \"stunner/gateway\": Service \"gateway\" is invalid: spec.loadBalancerClass: Invalid value: \"null\": may not change once set"

We're using the AWS load balancer controller, which sets that field when the service is created and handles the creation of a load balancer for us.
I traced this down to how the service gets recreated each time. We have a cache of the current state of the service in the global store, so we can use this to get the current values. We need to then be careful about adding ports, but other than that the change is pretty straightforward.

Duplicate of https://github.com/l7mp/stunner/issues/104

Sorry I missed the earlier bug report, thanks for referencing it.
I wanted to update that we built and tested this in dev and rolled it out to our production clusters. And, it's been working reliably without the error log. Happy to try to figure out a test for this if you think it'd be worthwhile.

Thanks a lot for the PR, we truly appreciate it that you made the effort! Don't worry about the duplicate: unfortunately some issues get reported to the STUNner repo and some issues here, and it's somewhat difficult to keep track of what's going on.


Possible incompatibility with golang.org/x/net/websocket

See https://github.com/aaugustin/websockets/issues/1094#issuecomment-1001313589

I figure out what happend. "golang.org/x/net/websocket" default server check origin header which python client does not set. So I use websocket.connect(..origin="XXX") instead example, It works.

I filed this ticket quickly to avoid losing track of a potential problem. In fact, I don't think there is a problem.
Some WebSocket servers may enforce Origin checking. Then, the WebSocket client must pass this header.
websockets provides a way to do this. All is good.


Fixed the current schema/current user confusion in OracleSchema

This pull request is to address some of the flaws mentioned in Issue #1550 related to the confusion between current_schema and user functions in Oracle database.
Main points:

Current schema name can be determined with SYS_CONTEXT('USERENV', 'CURRENT_SCHEMA') function.
Current execution context user name can be determined with USER function.
Modifying objects in shemas other than the user's own schema may require extra privileges or be impossible. Flyway checks if the current user and the affected schema coincide while executing Clean command.
User/schema names are case-sensitive and must be compared with equals() method. Flyway uses quoted schema names to work correctly.

This PR supersedes PR #1552 (closed due to a wrong branch selection).

Merged. Thanks for the excellent PR!


Documentation Links Broken

All documentation links, including the "Getting Started" link on the homepage appear broken.

Thanks. I can see two broken links, for now can you use the left navigation menu?

I can use the left nav, but I am still seeing broken links.  Examples:  Getting Started on OAI Documentation Home Page; The Open API Specification explained on Getting Started page.
Thanks

Yes, I see those too, will check if local dev server shows the same problem.


[BUG] Not updating after Chrome version 114

Describe the bug
Chromedriver is not getting updated after version 114
Additional context
This stackoverflow issue describes it well https://stackoverflow.com/questions/76932496/webdrivermanager-setup-failing-to-download-chromedriver-116
Sorry for not adding more information...

Thanks for pointing out this issue. I have fixed it in a new released version 6.0.0


BUG: Description of operations in the Status Center

Description
From the "_PRUEBAS" folder I have copied and deleted five items. (230530 Contactos 02.jpg, 230530 Contactos 03.jpg, 230530 Contactos 04.jpg, 230530 Contactos 05.jpg, 230530 Contactos 06.jpg,)
The action "Delete" in "Status Center" correctly describes the action performed: "Deleted 5 item(s) from _PRUEBAS".
However the "Copy" action in "Status Center" does not correctly describe the action performed: "Copied 5 item(s) to 230530 Contacts..." (Copied 5 item(s) to 230530 Contacts 06.jpg). I understand that by similarity with the action "Delete" it should describe something similar to : "Copied 5 item(s) in _PRUEBAS" or "Copied 5 item(s) from _PRUEBAS"...

Except that I do not understand very well the description of the action that Files wants to describe.
Steps To Reproduce

Copy 5 elements
Delete 5 items
Open Status Center

Requirements

e.g. Fix crash that occurs when...

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2506
Log File
I have reinstalled the system and no log file is available.

Thanks for the report

@dracp Is this still an issue in 3.2.4 https://files.community/download

We're having trouble reproducing this in the latest version of Files, please let us know if you can still repro and we'll try to investigate further.


Support click hyperlink in desktop application by pywinauto

Expected Behavior
I want to click hyperlink in desktop application by pywinauto
Actual Behavior
Steps to Reproduce the Problem
Please help me method can click hyperlink in desktop application by pywinauto
Thanks so much!
Short Example of Code to Demonstrate the Problem
Specifications

Pywinauto version: 0.6.8
Python version and bitness: 3.7
Platform and OS: windows 10

Did you find a way to do this??  I have a hyperlink in UIA that I'm trying to grab the link out of.

The link text can be obtained by .iface_value.CurrentValue property. For clicking on hyperlink use method .invoke(). If you have different problem, please provide more details.

I tried clicking on an element on the "Postman" app whose control_type was "Hyperlink" but nothing happened. Can you please help me with this.

.click_input() solved this problem for me , putting it here for anyone else who runs into this


2023 GV70 Unrecognized

Describe the bug
Upon full installation the display reads “Dashcam Mode Car Unrecognized”
Which car does this affect?
2023 Genesis GV70
Provide a route where the issue occurs
5d327dec6ec07b6e|2023-01-04--16-58-08--0
openpilot version
0.9.0
Additional info
No response

Addressed with Fingerprint addition
PR 26876


Upgrader cleanup tweaks

Streamline when upgrade cleans up old downloads
Previously, the upgrade notification logic checked for previous downloads of the upgrade packages in order to drive whether to notify the user about whether an upgrade is available. There were cases where the user updated outside of the upgrader logic, where the downloaded upgrade packages where not cleaned up, resulting in spurious upgrade notifications.
Since then, we have changed the logic to look at a sentinel file to indicate whether an upgrade is available, and stopped downloading packages in the background.
The NuGet upgrader did not clean up the downloaded packages, only the extracted packages.
This change:

UpgradeVerb and UpgradeOrchestrator no longer opportunistically deletes the download folder containing upgrade packages
UpgradVerb deletes the upgrade available sentinel file if run elevated
Upgrader deletes the download folder after running

Questions:
Q1) Do we want to always delete the downloaded packages after upgrade runs (if upgrade fails for some reason, it might be useful to have the package around for diagnostic purposes)
A) We can look at this as a future improvement
Q2) Do we need to be more proactive when running GVFS upgrade to remove the sentinel file if not update is available? One issue is that this file can only be modified by a process with elevated permissions.
A) We will delete the sentinel file if run as an admin. We can look at improving this more in the future based on feedback.
Fixes #833

Wouldn't this re-open #502. How do we mitigate this.

Wouldn't this re-open #502. How do we mitigate this.

I think the opportunity for this to happen is smaller now. When restarting the GVFS.Service, we will refresh the sentinel file (as opposed to before, where restarting the service would not refresh the upgrade notification). I think the cases of someone switching rings in between installs will be less, but that is a hole.
Currently, running as a non-admin will also have issues, as the non-elevated command will not be able to clear the sentinel file. Running from an elevated prompt will clear the sentinel file (with this change). One mitigation could be to have the user run as an admin (maybe we could include this in the instructions if we detect that the sentinel file is present, but no upgrade is available and the command is run non-elevated)

@jamill do you still need this PR?

@jamill do you still need this PR?

This is a bit outdated, I will close this for now.


renamed createIncognitoBrowserContext to createBrowserContext in order to support latest puppeteer breaking changes (v22.0.0)

renamed createIncognitoBrowserContext to createBrowserContext in order to support latest puppeteer breaking changes (v22.0.0): https://github.com/puppeteer/puppeteer/pull/11834

Thanks, a new version has been published!


Add decoration for removed files in Project Files

This PR adds an additional file decoration for files that were included in the last deployment, but are excluded for the next. They are now marked with a red R for removal.
The source control VS Code extension uses a red D for deletion, but that didn't feel quite right since the file is just being excluded (or removed from the next deployment).

Preview

Intent
Resolves #2222
Type of Change

[ ] Bug Fix

[x] New Feature

[ ] Breaking Change

[ ] Documentation

[ ] Refactor

[ ] Tooling

Approach
The approach here is extremely similar to the green A with a different v-if
Directions for Reviewers
To see this in action:

make a deployment
unselect files previously deployed in the Project Files view

I didn't include this one in the changelog for 1.1.6 so I'll hold it for the next release.


Containerize source-reminder

What
This PR:

Containerizes source-reminder AWS Lambda function for local development or manual runs

Moved function source code into source-reminder/src directory for organization
No longer relies on a credentials.json file, instead is configured through a configurable environment variable.

Adds a Github Action for continuously deploying source-reminder:latest docker image to AWS ECR (to be picked up by source-reminder AWS Lambda function.
Adds make commands for running locally:

 # build local image "source-reminder/lambda:local",
 # and runs as a container in the background mapped to local port  9000
 # with lambda function mounted as a container so local code changes are synced in the running container for testing
 make run-container

 # Simulates running lambda function by making curl request to running source-reminder container
 make test

Cleans up some AWS SAM references (another way of packaging AWS Lambda functions)
Updates README documentation to reflect these changes in developing and deploying

Why

Reduces friction for deploying changes to source-reminder function
Eases local development (for manually running source-reminder, or for developing in test mode
Provides example for how we can more seamlessly develop, build, and deploy lambda functions

How to test
Follow the new instructions in the README under Running Locally, which will help identify gaps in those instructions or places that could use more clarity.
Blockers

[x] Configure source-reminder AWS Lambda function to use container
[x] Need to update how the lambda function reads credentials.json for the Google Service Account, to read either from AWS Lambda configuration variable or pulling the .json from AWS Secret Manager.  To keep this PR from getting to large of scope, going to make a second small PR to reflect this.

Next Steps
Not blockers to this PR, but good to have for next steps

[ ] Integrate Sentry for error handling/reporting & slack alerts (see #16)
[ ] Update MANUAL RUN and DRY RUN to use environment variables
[ ] After this is merged, the new docker image will be built on AWS (currently the code has DRY_RUN=True, so we can run this on Lambda and confirm it works as expected.  After we do that, we can change DRY_RUN=False and enable the Friday AM scheduled event, then this should be pretty hands off for the foreseeable future.
[ ] Consider combining ^ somehow to ENVIRONMENT=[prod, test, dev] or something

Looks great Brandyn! Really grateful I got to learn from you while you worked on this. This is my first PR review so I hope I covered all the bases, but it works for me!

Looks great Brandyn! Really grateful I got to learn from you while you worked on this. This is my first PR review so I hope I covered all the bases, but it works for me!

Thank you @jbarrueta for the insightful and thoughtful review, and for pairing on this PR -- I wish more of the commits could have shown as co-authored to reflect our work together earlier in the process.  Appreciate your good questions, curiosity, and patience on this, it was very helpful for me!


Computed properties on objects appear to not be supported

If an object or struct is passed to a template, you can generally access the properties on the object/struct by using {{ objectName.propertyName }}. However, this appears not to work with computed properties.
In other words, with a class like:
class foo {
  var bar: String = {
    return "baz"
  }
}

If you instantiate a foo, put it in the context dictionary, and then try to print {{ foo.bar }} in the template, nothing prints.
In my actual code, I didn't want my bar value to be an actual normal property calculated when init-ing the struct, so I ended up working around it by implementing bar as a Stencil filter.

Swift Mirror does not support computed properties, it is used to lookup properties of swift types. If you use NSObject-based types KVO will be used. There is no workaround for that, as far as I know. You could though define an extension on your type that creates a context dictionary that you use in Stencil instead of accessing properties, but that's far from a good solution.


Storing plugin name

Summary of the pull request
A recent change was made to store the provider name (Github) and not the plugin name (Dev Home Github Extension)
Eric and I went over the RepoTool and changed all references to make sure Github was shown in the UI.
I missed one.  When selecting repos to show from the previous selected repos I search based on the provider name.  I compare the provider name in CloningInformation to the provider name in the provider drop down.
The provider name in CloningInformation is Github and the provider name DevHome is comparing against is "Dev Home github extension"
This prevented previously selected repos from showing up when the repo tool was re-opened and prevented seleted items from staying selected when filtering.
The fix is to add an PluginName to CloningInformation and store "Dev Home Github Extension" into InternalProviderName and search on PluginName instead of ProviderName.
References and relevant issues
Github issue
Detailed description of the pull request / Additional comments
Validation steps performed
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[ ] Documentation updated

/azp run

/azp run


Move Alerts

Moves alerts to the new adapter channel. The secret is already there so we don't need to wait to merge.

waiting to merge till we figure out bigquery


MGMT-15276: Add support for image mirroring in the worker

Mount /etc/containers from the host into the worker Pod as read-only.
Look for mirrors using github.com/containers/image/v5 library calls.
/cc @yevgeny-shnaidman

/lgtm


openembedded guidelines check: Signed-off-by

Accordingly to http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines each commit should have the following format (here is an example):
foobar: Adjusted the foo setting in bar

  When using foobar on systems with less than a gigabyte of RAM common
  usage patterns often result in an Out-of-memory condition causing
  slowdowns and unexpected application termination.

  Low-memory systems should continue to function without running into
  memory-starvation conditions with minimal cost to systems with more
  available memory.  High-memory systems will be less able to use the
  full extent of the system, a dynamically tunable option may be best,
  long-term.

  The foo setting in bar was decreased from X to X-50% in order to
  ensure we don't exhaust all system memory with foobar threads.

  Signed-off-by: Joe Developer<EMAIL_ADDRESS>
Be able to activate a (non default) check for Signed-off-by line which needs to exist in all pushed commits.

Duplicates #144, closing.


Any library can achieve something similar to Python textual

I have some doubts:
spectre.console is like python rich,  based on rich  python has the textual,  c# has something to achieve that

Not sure what you’re asking to be honest, but if you want to build interactive terminal apps in C#, you should check out https://github.com/gui-cs/Terminal.Gui


How do I initialize the datepicker on route navigation?

the date picker works if I refresh the page (or just manually type in the URL that has the datepicker)
but if I navigate to it, the datepicker isn't initialized, and the text field is just a text field.

Would be helpful if you could provide code, otherwise it's hard to understand what is going wrong.

Here are the two templates that I navigate between:
this one is at /events/:id//edit/
https://github.com/NullVoxPopuli/aeonvera-ui/blob/upgrade-to-ember-2/app/templates/events/show/edit/index.emblem
and
this one is at /events/:id/edit/options
https://github.com/NullVoxPopuli/aeonvera-ui/blob/upgrade-to-ember-2/app/templates/events/show/edit/options.emblem
If I type one of those paths in to the address bar, and then click a link-to that takes me to the other route, the second route's date-time-pickers don't work.
does that help?

Thank you! These links are helpful. Actually you are using different library, so this is a wrong repo for issue report :wink:
You should ask at https://github.com/plusacht/ember-bootstrap-datetimepicker. Because you are using bs-datetimepicker component in templates, it should be ember-bootstrap-datetimepicker library.

oh. haha. sorry about that :-)

No problem :wink: Wish you a great day and hope you will find solution for the problem very soon!


Enhancing Readability and Aesthetics: Elevating the H1 and Paragraph Elements in the Hero Section

Describe the bug
I want to add a little bit styling to the h1 and paragraph on hero section to make it more aesthethic and readable for better user experience.
To Reproduce
Steps to reproduce the behavior:

Navigate to 'https://akshitagupta15june.github.io/PetMe/index.html'.
Locate the hero section.
Observe the paragraph and h1 within the hero section.

Expected behavior
We can make it more aesthethic and more readable for better user experience and aesthethic of the page
Screenshots

Desktop (please complete the following information):

OS: [Windows]
Browser [Chrome]
Version [113.0.5672.127]

Smartphone (please complete the following information):

Device: [Realme c27]
OS: [Android]
Browser [chrome]
Version []

Additional context
I want to work in this under GSSoc contributor. Please assign this to me :)

ok assigning you


Quick pick: Support alternate representation of French/European characters

This should also match tache:

See https://github.com/microsoft/vscode/issues/196203 which this would be built upon.
cc @aiday-mar

For Russian, I would say the following are similar, and can be interchanged for quick typing or by mistake
ё, е
и, й
щ, ш
ъ, ь

For Chinese, it seems like the IDE already suggests English words for you
I'm unable to get a good example because the IME on my Windows device is busted and Google Pinyin doesn't seem to exist as a standalone app anymore, so here's an example picture of a mobile IME suggesting the English string "yk" along with certain characters:

@rzhao271 I think for Chinese and Japanese Kanji (Hiragana and Katakana could work) this isn't really possible due to the number of characters. It would make sense to translate input characters in a desktop Chinese IME into the qwerty alternatives, in case you accidentally started typing when in the wrong keyboard mode, but again we can only really map a handful of characters and/or a unicode code range.

What I'm saying is that Chinese and Japanese should be out of scope anyway with a good IME because all Chinese IMEs I used suggest English words for you if you type out an English word even while in Chinese input mode.

Calling out what @jrieken mentioned in slack:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/Collator/Collator#options
We may be able to use this for the simpler cases

Yep I think Collator will cover the latin cases, not sure about Cyrillic

For Russian, I've heard of ё being written as just е, but I'm unsure about the rest.
I'm unsure if it's because of the https://en.wikipedia.org/wiki/JCUKEN layout and how ё is way off in the corner.
'ё'.localeCompare('е', undefined, { sensitivity: 'accent' }); returns 1 and 'ё'.localeCompare('е', 'ru', { sensitivity: 'accent' }); also returns 1, though.


Non-ASCII file paths doesn't work on Windows

i came across this while debugging #25 ...
❯ ~/magiskboot_build/build/magiskboot_debug.exe unpack -h ./中国語テスト/boot.img
The system cannot find the path specified. (os error 3)

removing this check seems to help
❯ ~/magiskboot_build/build/magiskboot_debug.exe unpack -h ./中国語テスト/boot.img
Parsing boot image: [./中国語テスト/boot.img]
HEADER_VER      [4]
KERNEL_SZ       [46771916]
RAMDISK_SZ      [1380092]
OS_VERSION      [12.0.0]
OS_PATCH_LEVEL  [2023-08]
PAGESIZE        [4096]
CMDLINE         []
KERNEL_FMT      [raw]
RAMDISK_FMT     [lz4_legacy]
[boot\sign.rs:261] unexpected ASN.1 DER tag: expected SEQUENCE, got APPLICATION [1] (primitive)
VBMETA

landed in latest releases


map.css is application, not vendored code

move map.css to app css and probably make it scss

Resolved in #63


Release the latest version

@dhh can you release the latest version 🙏🏼 🙏🏼
Current release 0.2.2 doesn't work with webpacker since it is missing the definitions
(works from the the master, but... 😄 )

0.2.3 is out ✌


glTF (.glb) importer fails with errors importing file with no objects/meshes

Godot version
4.0.beta5
System information
macOS 13.0.1
Issue description
I recently discovered that it you try to import a glTF (.glb) file that contains no objects, the importer ungracefully fails with multiple errors that don't really help you understand what has gone wrong:
  modules/gltf/gltf_document.cpp:570 - Condition "!s.has("nodes")" is true. Returning: ERR_UNAVAILABLE
  modules/gltf/gltf_document.cpp:6927 - Condition "err != OK" is true. Returning: ERR_PARSE_ERROR
  modules/gltf/gltf_document.cpp:6597 - Condition "err != OK" is true. Returning: err
  modules/gltf/gltf_document.cpp:7032 - Condition "err != OK" is true. Returning: err
  editor/editor_file_system.cpp:1909 - Error importing 'res://Models/empty.glb'.

What I'd expect would be that the importer would not fail, just generate a scene with no mesh instances etc. Probably then also provide a helpful warning explaining there were no readable objects in the imported file as this was probably not intentional. Instead the importer now fails and breaks any scenes linking to the file.
This might seems like it's not a problem since you'd never try to import an empty file, but it can happen quite easily due to the way blender exports these files: A way I commonly use is to have a master blender file with several models in, and then export them to separate asset files by selecting them and exporting in blender using the "selected objects" option to only export the selected objects. What I discovered is that if you've accidentally deselected the object you're intending to export blender will still export a glb file with no warnings. Only the file will have no objects in it. And if this export has overwritten one of the imported models in your project Godot will now just provide the errors above and fail to load any scenes with the model linked to it.
While I'm not familiar enough with the code to know fully, a guess would be that the problem is here in GLTFDocument::_parse_scenes:
https://github.com/godotengine/godot/blob/77f4670d001b12a5c7e1f50c100301c8fb1b4d1e/modules/gltf/gltf_document.cpp#L567-L570
It incorrectly assumes that all files that result in no nodes loaded have failed to load. While the case above illustrates that it may be the case that it has loaded, it's just empty. And in this case it should probably issue a warning instead and produce an empty scene instead of failing with ERR_UNAVAILABLE.
Steps to reproduce
Simple:

Just try to import the empty.glb file linked below.

Full experience:

Import the example cube.glb model from below in a Godot project and instance this in your main scene.
Replace the imported cube.glb file with the empty.glb renamed to "cube.glb" to simulate overwriting an imported asset with an empty one.

Minimal reproduction project
empty.glb.zip
cube.glb.zip

Tried it again now in 4.0b15 and while the errors are still there (treating the file as invalid rather than empty) I'd say it now handles the situation a lot better. If you now try to load a scene with an instance of the now empty model file you get a warning about this, but an option to load it anyway. If you do the scene loads, just with the model removed. While I'd still say the bug is there, this is of course much better than preventing you to open the scene like in b5.
These are the errors that are output now when loading (The "Load anyway" option) a scene with an instance of an empty model:
  Failed loading resource: res://cube.glb. Make sure resources have been imported by opening the project in the editor at least once.
  scene/resources/resource_format_text.cpp:174 - res://Main.tscn:7 - Parse Error: [ext_resource] referenced non-loaded resource at: res://cube.glb
  editor/editor_data.cpp:574 - Index p_idx = 1 is out of bounds (edited_scene.size() = 1).
  modules/gltf/gltf_document.cpp:558 - Condition "!s.has("nodes")" is true. Returning: ERR_UNAVAILABLE
  modules/gltf/gltf_document.cpp:6983 - Condition "err != OK" is true. Returning: ERR_PARSE_ERROR
  modules/gltf/gltf_document.cpp:6660 - Condition "err != OK" is true. Returning: err
  modules/gltf/gltf_document.cpp:7088 - Condition "err != OK" is true. Returning: err
  editor/editor_file_system.cpp:2004 - Error importing 'res://cube.glb'.
  Failed loading resource: res://cube.glb. Make sure resources have been imported by opening the project in the editor at least once.
  scene/resources/resource_format_text.cpp:174 - res://Main.tscn:7 - Parse Error: [ext_resource] referenced non-loaded resource at: res://cube.glb
  scene/resources/packed_scene.cpp:178 - Node './cube' was modified from inside an instance, but it has vanished.

The editor window flickers for me every few seconds (super annoying, can't work like this), presumably because it tries to reimport the broken files.


cask-repair: update PR template

https://github.com/caskroom/homebrew-cask/pull/37862

https://github.com/caskroom/homebrew-cask/pull/37862#issuecomment-323991317


Identify nodes rebooted by kured

I am using kured and it works well but I am wondering if there is a way to know that nodes have been rebooted by kured and not by something else?
I can see the rebooted nodes in prometheus by looking at uptime but I don't know if those have happened because of kured or not.
I see there is a bunch of flag to add labels but I don't see if that could help. I am not using any notification to a chat system, it would be great to see this through prometheus

Hi @bobbyfisherman,
The best way at present is to use this facility:
https://github.com/weaveworks/kured/pull/296
Essentially, by including --set configuration.annotateNodes=true (using the helm CLI usage as an example) when you install the kured helm chart you will get the following annotations on nodes that are touched by kured:
When a node is currently being rebooted by kured:

weave.works/kured-reboot-in-progress

After a node has successfully rebooted:

weave.works/kured-most-recent-reboot-needed

Those annotations will have a timestamp as the value, so you know when things occurred (and the in-progress annotation is deleted after the node has rebooted).
Will that help?

@jackfrancis I saw that PR but I thought it was removed in the latest release since the flag isn't documented in the repo README as far as I can tell, but I see it present in the chart.
Thank you!

@bobbyfisherman that's totally understandable, updating the README here:
https://github.com/weaveworks/kured/pull/607


make code-only version of notebooks available

Description of the feature request:
The example notebooks are great, but if, like me, you are not big into notebooks, you may want to be able to just extract the code without all the explanatory Javascript.  Another folder with "code-only examples" would do the trick.
What problem are you trying to solve with this feature?
ease of access to code samples
Any other information you'd like to share?
Mentioned this to @logankilpatrick who liked idea.

We've added some standalone samples to the various SDK repositories, is this sufficient for what you're asking?
e.g. for Python & REST: https://github.com/google-gemini/generative-ai-python/tree/main/samples

Yes, those are great. Copypasta dreaming!

On Aug 27, 2024, at 7:53 PM, Mark McDonald @.***> wrote:
We've added some standalone samples to the various SDK repositories, is this sufficient for what you're asking?
e.g. for Python & REST: https://github.com/google-gemini/generative-ai-python/tree/main/samples
—
Reply to this email directly, view it on GitHub https://github.com/google-gemini/cookbook/issues/107#issuecomment-2313768173, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI4TSZA2VNOYB6VBALWCGTZTUGOXAVCNFSM6AAAAABHGGMRNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJTG43DQMJXGM.
You are receiving this because you authored the thread.


refactor: create subsections for helm charts

In this PR I've created a "Values Files" subsection for each helm chart as a starting point for this issue - https://github.com/kubeshop/monokle/issues/586
Changes

Fixes

How to test it

Screenshots

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

looks good - but the numbering after the headlines is a bit confusing as it shows the total number of files in the hierarchy - perhaps we could change this so:

the number after "Helm Charts" shows the number of Helm charts (which would be 6 in the screenshot)
we don't show a number after name of the helm chart itself


Enhance Branch Creation UX and Refactor Issue Branch Management

This PR introduces significant improvements to the branch creation and management process, focusing on enhancing the user experience and refactoring the existing code for better maintainability and performance. Key changes include:

Branch Selection UI Overhaul:

Introduced a QuickPick interface for selecting branches, allowing users to choose from existing local/remote branches or create a new one.
Visually highlighted branches related to the issue (e.g., branches containing the issue number).
Provided a custom branch name option directly in the selection UI for flexible branch creation.

Some Related Issues:

#2243: Detect and use already-linked PR branches when starting work on an issue.
#4120: Detect branch names referencing an issue and automatically start working on them.
#4509: Improve transparency when creating and checking out branches for issues.

These issues are addressed by improving the branch selection experience, handling remote branches more seamlessly, and enhancing error feedback.
Testing and Verification:

Manual Testing:

[x]   Verified that the QuickPick interface correctly displays existing branches and suggested branch names.
[x]   Tested creation and checkout of local and remote branches.
[x]   Confirmed proper upstream tracking for remote branches.
[x]   Ensured that validation errors (e.g., invalid branch names) display appropriate messages.

@microsoft-github-policy-service agree

@alexr00 Thanks for the feedback! I’ve addressed the specific comments and focused on ensuring the necessary changes related to the feature.


add dappos plugin

before we can merge yes, also we might require the plugin to be posted in ipfs. more on that later...
btw I tried it but couldn't access the  generated dapp.. I can give it a try again if you tell me that should work.

Hey Yann,
I just tried it and it's working as intended. Make sure you're using the
address of a deployed contract. Here are the docs for the plugin:
https://documentation.dappos.io/remix-ide-plugin/intro.
On Wed, Jun 21, 2023 at 4:06 AM yann300 @.***> wrote:

before we can merge yes, also we might require the plugin to be posted in
ipfs. more on that later...
btw I tried it but couldn't access the generated dapp.. I can give it a
try again if you tell me that should work.
—
Reply to this email directly, view it on GitHub
https://github.com/ethereum/remix-plugins-directory/pull/347#issuecomment-1600381545,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASITDZJKX4CMSBTFMA6BJJDXMKTP5ANCNFSM6AAAAAAYKSBUZA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey Yann,
here's the repo for the plugin https://github.com/Dappos-Inc/remix-plugin.
On Wed, Jun 21, 2023 at 2:33 PM Diego @.***> wrote:

Hey Yann,
I just tried it and it's working as intended. Make sure you're using the
address of a deployed contract. Here are the docs for the plugin:
https://documentation.dappos.io/remix-ide-plugin/intro.
On Wed, Jun 21, 2023 at 4:06 AM yann300 @.***> wrote:

before we can merge yes, also we might require the plugin to be posted in
ipfs. more on that later...
btw I tried it but couldn't access the generated dapp.. I can give it a
try again if you tell me that should work.
—
Reply to this email directly, view it on GitHub
https://github.com/ethereum/remix-plugins-directory/pull/347#issuecomment-1600381545,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASITDZJKX4CMSBTFMA6BJJDXMKTP5ANCNFSM6AAAAAAYKSBUZA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Make addLink accessor public

When customizing the links of a document it is rather challenging to have the addLink method not publicly available. This PR fixes that.

Hi @jkrzefski ,
Could you please precise me what kind of customizations you would like to achieve?
The AbstractLinks::addLink() method is intentionally hidden from the outside, because this way, only spec-compliant links can be added to documents/resources/relationships/errors (documents are an exception though because the DocumentLinks::setLinks() can be abused to pass non standard-compliant links).
If you really want to add unsupported link relations to let's say resources, then you can do the following:

extend ResourceLinks
define your custom methods in the new class which make it possible to pass custom links
instantiate and return the new class in your resource's getLinks() method

I don't know if this workflow suits your needs, but generally, this should be the way to go.

Yes, that was exactly my case. I wanted to extend the ResourceLinks and I found no public method to do it. So I did exactly what you suggested (extend the class) and then filed this PR to make extending links easier. To be honest, I was unaware of the fact that the links you can use are also part of the specification.
But still, having this accessor public does not mean, your library is incompatible with the specification. It only makes it a more versatile utility for people who (for whatever reason) have to use custom links. So I can still see this being useful somehow.
Or maybe you know a better (more compliant) way to add custom links to a resource. In my case the resource was a file and I added a link download for an instant download.

OK, I understand your point, it makes sense to make adding custom links easier.
However, the addLink() method is good for this purpose. It doesn't return anything (not suitable for chaining method calls) and it's name is also not the best.
So instead of making addLink() public, what about moving these methods from DocumentLinks() to the AbstractLinks class?
/**
     * @param Link[] $links
     */
    public function setLinks(array $links): DocumentLinks
    {
        foreach ($links as $rel => $link) {
            $this->addLink($rel, $link);
        }

        return $this;
    }

    public function setLink(string $name, ?Link $link): DocumentLinks
    {
        $this->addLink($name, $link);

        return $this;
    }

Of course, the return type declarating should be removed, and replaced with a @return static  type hint. Could you please do it?

I undid my accessor change and added your suggested change instead. I did one thing slightly different though: I used the return type self in the methods instead of a type hint for static. Do you think, it works this way? If not, I will change it to type hints for static, of course.

@jkrzefski Thanks for the change! I think the static PHPDoc type hint would be better (even if it's not a proper type declaration), because that way, PHPStorm (or maybe other IDEs too) will be able to type hint against the appropriate subclass of AbstractLinks.

Done.

Perfect, thanks for your work! I'll create a release with this change in the coming days (possibly today or tomorrow).


Remove reference to Name class in ReviewBuilder and JSONAdaptedReview

Resolves #121

Codecov Report

Merging #122 (e6cf760) into master (a1eab6f) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #122   +/-   ##
=========================================
  Coverage     67.21%   67.21%
  Complexity      417      417
=========================================
  Files            77       77
  Lines          1391     1391
  Branches        153      153
=========================================
  Hits            935      935
  Misses          397      397
  Partials         59       59

Impacted Files
Coverage Δ

...main/java/foodwhere/storage/JsonAdaptedReview.java
68.57% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add a visibility API endpoint for fetching pending workloads in a Local Queue

What type of PR is this?
/kind feature
/kind api-change

What this PR does / why we need it:
This PR continues the development of KEP#168-2
Which issue(s) this PR fixes:

Special notes for your reviewer:
This PR is based on https://github.com/kubernetes-sigs/kueue/pull/1362. I'll rebase it when the base PR is merged into main
Does this PR introduce a user-facing change?

Extend visibility API by adding an endpoint that allows a user to fetch information about pending workloads and their position in LocalQueue.

/assign @mimowo

/ok-to-test

I've manually tested RBACs, and they seem to work well

/assign @mwielgus

/lgtm
/assign @alculquicondor @tenzen-y

/retest

/release-note-edit
Extend visibility API by adding an endpoint that allows a user to fetch information about pending workloads and their position in LocalQueue.

Still LGTM, please squash.

Still LGTM, please squash.

I'm ok without squash. The prow automatically will squash commits.

/release-note-edit
Visibility API: Add an endpoint that allows a user to fetch information about pending workloads and their position in LocalQueue.


[ConfigureDataPermission]<Cannot read properties of undefined (reading 'role')>

Question

system/manage->user-> ConfigureDataPermission
is it an issue or config error？

sorry, it is my problem, customer plugin config error.

sorry, it is my problem, customer plugin config error.


small typo in the readme

Summary
when configuring the agent, the scope currently says:
"Your job is to assist a Customer Service Representative support guests with questions about reservations and experiences they can book at the hotel."
but is sounds like it should say:
"Your job as a Customer Service Representative is to assist guests with questions about reservations and experiences they can book at the hotel."
Salesforce Org Type
Trailhead Playground
Steps To Reproduce
N/A
Current Behavior
N/A
Expected Behavior
N/A
Relevant Log Output
N/A

Code of Conduct

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

See PR comment.


Header fixed on top when i scroll

Hello,
is it possible to set a minimum height for the header of my tableview to make it visible even when i scroll ?
Thanks

No, Afraid that is not possible right now.


[23.05] libvirt: add patch for CVE-2023-3750

Description of changes
https://nvd.nist.gov/vuln/detail/CVE-2023-3750
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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

[x] 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/)
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

[ ] Fits CONTRIBUTING.md.

For "add patch" commits it's always because it was bumped in master.


Is there any performance problem about every topic save as a dadger db?

This is a great project for distributed APP.  I ran the chat demo,  found the data directory has two directory: chat.com_room-2837 and chat.com_room-2837_indices, it means  every topic save as a dadger db? if there are 100,000 rooms, maybe the APP create 100,000 DB instantces in memory? is there any config to avoid it? or any other solution for hundreds of rooms? thanks.

Thanks for checking out the project. Yes, there are some limitations imposed by Badger. I believe I remember reading that there's a limit at around ~100 open Badger DBs before performance starts to degrade. We've been doing some work lately to consolidate some of the supporting DBs used by Redwood (peer store, tx store, key store, etc.), but so far, every state tree ("topic" in your message) has its own DB.
The reason for this is straightforward: when incoming updates to a state tree are being processed, Redwood needs to hold a Badger transaction open until the update is complete. If we put all of the state trees into a single DB, only one state tree can be updated at a time. For this reason, we've decided to separate them so that each tree can be updated in parallel, which hopefully improves performance/responsiveness.
This is a tricky balancing act, though, and I'm not sure it's going to work over the long term. It might make sense to make this more configurable, so that applications with many state trees (but fewer updates per second) can use a single DB. Applications with few state trees but many updates would be able to configure Redwood to use many DBs.
There also might be a more blended approach, where the system tries to intelligently consolidate a few state trees into each DB.
Anyway, curious to know if you have thoughts on this.

Thanks for checking out the project. Yes, there are some limitations imposed by Badger. I believe I remember reading that there's a limit at around ~100 open Badger DBs before performance starts to degrade. We've been doing some work lately to consolidate some of the supporting DBs used by Redwood (peer store, tx store, key store, etc.), but so far, every state tree ("topic" in your message) has its own DB.
The reason for this is straightforward: when incoming updates to a state tree are being processed, Redwood needs to hold a Badger transaction open until the update is complete. If we put all of the state trees into a single DB, only one state tree can be updated at a time. For this reason, we've decided to separate them so that each tree can be updated in parallel, which hopefully improves performance/responsiveness.
This is a tricky balancing act, though, and I'm not sure it's going to work over the long term. It might make sense to make this more configurable, so that applications with many state trees (but fewer updates per second) can use a single DB. Applications with few state trees but many updates would be able to configure Redwood to use many DBs.
There also might be a more blended approach, where the system tries to intelligently consolidate a few state trees into each DB.
Anyway, curious to know if you have thoughts on this.

Thanks for your reply, I ran the chat demo and chat in the room, so I think if develop a chatroom app based it, there will be many rooms, so worry about the performance.


[firebase_messaging] regression on Android: messages not delivered when app is terminated on a known good device (e.g. Pixel); this works on a normal Android app so am trying to figure out why Flutter has an issue in getting it to work

I have a "normal" Android app written in Kotlin that uses FCM. If the app is killed off it will still receive data messages. I have been working on porting the app to Flutter. However, on flutter the app does not receive messages unless it is in a running or background state.
So what is the reason for this? It would seem like it is doable. I've talked to a few people at IO 2019 from both the Flutter & Firebase teams and they suggested I open an issue. Tagging @amirh and @kroikie
Thanks,
Gerry

@gerryhigh At first investigation I am able to reproduce this issue. Looks like the message is getting to the device on the Android side but the broadcast does not work if the app is swiped away from recents. I'll investigate further and report back here.

@kroikie I'm glad to hear that you were able to reproduce it. After IO I have spent some time confirming that the issue exists not just on Android Q but also on Android O (Nexus 5X).  I have also done other testing to see if it was anything I was doing but finally yesterday tried the example program and found that it was not working either.
I'm CCing @eseidelGoogle as he asked me to include him on this issue.

After looking at this further, this will require some significant changes to the existing plugin to support data messages when the app process is ended. I will leave issue open and update here with any changes. Thanks for reporting this!

I do hope this can get resolved soon as it is a serious regression from native Android apps. In my case I'll have to evaluate and decide if this loss of functionality is worth moving forward or not with this particular app.

@gerryhigh There are some Flutter Engine changes that are required for this to be fixed. These changes are already in review so once those engine PRs land I will be able to address this issue. I'll leave it open and provide more updates here.

Thanks for the update. I was actually going to start comparing my Kotlin app vs the Android plugin and see if I could hack something up but given there are engine changes I will wait for your changes. I may end up releasing the iOS version first if it comes to that. Thanks again.

I'm facing the same issue with firebase_messaging, as per the table "Receiving Messages" states on https://pub.dev/packages/firebase_messaging : Receiving data messages while App is terminated on Android is not supported by the plugin.
I join @gerryhigh as it's a severe regression and I hope this can be fixed. If you ever find something @gerryhigh , I'll be more than pleased to know how you managed to trick it 😊
Thanks

+1 for this issue to be solved.

Hi All, just an update here.
The firebase_messaging plugin uses a Local Broadcast to invoke Dart side code when a message is received. This works fine when the App's process is running. However when the process is not running the app cannot listen for and receive broadcasts, this prevents the Dart side from being invoked. We are currently looking alternative ways of invoking the Dart side when a message is received.
I will continue to update here with our progress.

@kroikie, are you planning to use Background Execution with Dart to solve this?
There's someone that managed to do this, see this discussion.
Cheers

Got a PR to provide some level of functionality while the app is in the background.

I'm not ready yet to create a PR for this but I've got a branch with background execution working in both iOS and Android: https://github.com/athornz/plugins/tree/messaging-background-execution.

onMessage must be a top-level or static function
FirebaseMessagingPlugin.setPluginRegistrant(this) must be called on the Android side:

class MyApplication : FlutterApplication(), PluginRegistry.PluginRegistrantCallback {

    override fun onCreate() {
        super.onCreate()

        FirebaseMessagingPlugin.setPluginRegistrant(this)
    }

    override fun registerWith(registry: PluginRegistry) {
        GeneratedPluginRegistrant.registerWith(registry)
    }
}

And on the iOS side:
func callback(registry: FlutterPluginRegistry) {
    GeneratedPluginRegistrant.register(with: registry)
}
FLTFirebaseMessagingPlugin.setPluginRegistrantCallback(callback)

I've only tested remote messages so far.

@athornz Our approach on Android is similar to what you suggest. We will look into iOS next.

@kroikie May I please know if the iOS version would be updated soon?

@subhash279 we are currently working on this and updates to FCM support on iOS are coming soon.

@kroikie Just checking if there is any update on it pls.

i tried many things but was not not able to make push notification work on ios 13.5.1. I followed the official documentation and uploaded the build on test flight. Strangely i start receiving push notification on resume, on launch as well as on message. Also working fine with local notification plugin.

hello, @kroikie any update about this issue?

firebase_messaging is no longer part of this project; if this is still an issue it should be filed in the FlutterFire repository.


Proposal: alternative interface that's more aligned with current idioms

Background
Currently the generated functions are exposed as static methods on a class. For example:
export class AccountService {
  static CreateAccount(
    this: void,
    req: CreateAccountRequest,
    initReq?: fm.InitReq,
  ): Promise<CreateAccountResponse> {
    return fm.fetchRequest<CreateAccountResponse>(`/api/accounts`, {
      ...initReq,
      method: 'POST',
      body: JSON.stringify(req, fm.replacer),
    });
  }
}

I don't know the original intent of this pattern, but you lose any benefits of having a Service instance that tracks state and you get typescript that's not very idiomatic.
import { AccountService } from './accounts.pb';

const resp = AccountService.CreateAccount(newAccount, {
  pathPrefix: API_HOST,
  credentials: 'include',
  headers: {
    'X-CSRF-Protection': 1
  }
})

Proposal
Introduce a flag use_static_classes which defaults to true for now. When use_static_classes is false, you simply get an exported function that maps to the method name:
export const createAccount = (
  req: CreateAccountRequest,
  initReq?: InitReq,
): Promise<CreateAccountResponse> => {
  {
    return fm.fetchRequest<CreateAccountResponse>(`/api/accounts`, {
      ...initReq,
      method: 'POST',
      body: JSON.stringify(req, fm.replacer),
    });
  }
};

And then:
import { createAccount } from './accounts.pb';

const resp = createAccount(newAccount, {
  pathPrefix: API_HOST,
  credentials: 'include',
  headers: {
    'X-CSRF-Protection': 1
  }
})

Problem : conflicting function names
The main downside I see in this approach is that there could be ambiguous function names in the case where a single proto file contains multiple services with the same method. This seems unlikely in my experience, but a possible fix could be to generate multiple TS files.
For example: consider accounts.proto contains AccountService and AdminService, both with createAccount methods.
We could generate:

accounts.pb.ts → all the message definitions.
accountservice.gw.pb.ts → functions for calling AccountService gateway endpoints
adminservice.gw.pb.ts → functions for calling AdminService gateway endpoints

Additional Variant
As well as exposing simple functions, we could also introduce a more classical Client. The client instance could be used to store request configuration:
export class AccountServiceClient {
  constructor(initReq: InitReq) {
    this.initReq = initReq;
  }

  createAccount(
    this: void,
    req: CreateAccountRequest
  ): Promise<CreateAccountResponse> {
    return createAccount(req, this.initReq);
  }
}

@seanami would be curious to hear your opinions on this.

Nice! I like this as a way to make the code more idiomatic and also be able to have a more typical stateful client object that can preserve configuration across multiple requests.
I do use the service level class objects right now in some of my client configuration, but only to get the name of the service as a string for generating react-query cache names, and that can easily be replaced with something else.
I do highly suggest putting all of the request functions in per-service files. That will solve the potential dupe issue, and also allow those who want similar semantics as the existing implementation to do: import { * as AccountService } from ‘file’

Any thoughts on naming of the files? Given accounts.proto should it be accounts-accountservice.gw.ts or just accountservice.gw.ts.

Can service names be duplicated between separate proto files that are part of the same environment/project? E.g. Could I also have an additions.proto file in the same project which also defines an AccountService? (And if so, do the definitions get merged or what?)
If no duplicates are allowed, or the duplicates get merged into one definition, then I think just accountservice.gw.ts makes sense. Otherwise, I'd go with accounts-accountservice.gw.ts.


Rename provider / update README reqs

This commit renames the provider from faas to openfaas, but may
need a change upstream in the faas-cli to work.
The test script now references an OpenFaaS Node.js template for
the hello-world function.
README updated with steps for pre-reqs.
Signed-off-by: Alex Ellis (VMware<EMAIL_ADDRESS>
Description

Motivation and Context

[ ] I have raised an issue to propose this change (required)

Which issue(s) this PR fixes

Fixes #
How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[ ] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

As part of the rename we have to put "openfaas" in "provider" field in our YAML file. This breaks the faas-cli. I've created a patch and this is in the master branch of the faas-cli.


error code 500 when FS sessions high concurrency

[10/Oct/2020 09:57:19] "GET /esl?module=default HTTP/1.1" 500 -

Hi, thanks for the report. Could you please check whether the connection is denied on the freeswitch side or whether something goes wrong in the exporter.
I suggest to just try to connect using fs_cli in such situations (fs_cli connects to the esl socket as well). If that fails, then there is likely a problem with freeswitch.

Closing this due to inactivity. Please reopen if there is new data available.

I get this too. I have three machines: two with quite a bit of traffic. It's hit or miss with 500s for the two, but I consistently get a 200 for the one without load. After looking at the network traffic (and your code) it looks like you're looping through each call and uuid_dump'ing them... if a call (channel) goes away then we get "ERR No such channel!" and a 500.
I have freeswitch_exporter running on prometheus. Am I doing something wrong?

After looking at the network traffic (and your code) it looks like you're looping through each call and uuid_dump'ing them... if a call (channel) goes away then we get "ERR No such channel!" and a 500.

Thanks for this very valuable analysis. I'll gonna check whether there is a better way to handle this situation.

Opened PR #4 which potentially fixes that problem. Would you mind checking whether that reduces scrape failures?

Would you mind testing the attached wheel? It contains the fix and also #5 which should give us a better understanding where things are failing.
prometheus_freeswitch_exporter-1.0.0-py3-none-any.whl.gz

<IP_ADDRESS> - - [06/Apr/2021 08:19:43] "GET /esl?module=default&target=fs1.internal HTTP/1.1" 200 -
<IP_ADDRESS> - - [06/Apr/2021 08:19:48] "GET /esl?module=default&target=fs2.internal HTTP/1.1" 200 -
<IP_ADDRESS> - - [06/Apr/2021 08:19:55] "GET /esl?module=default&target=fs3.internal HTTP/1.1" 200 -
<IP_ADDRESS> - - [06/Apr/2021 08:19:58] "GET /esl?module=default&target=fs1.internal HTTP/1.1" 200 -
Exception thrown while rendering view
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/http.py", line 103, in view
    return self._views[endpoint](**params)
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/http.py", line 55, in on_esl
    output = collect_esl(self._config[module], target)
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/collector.py", line 296, in collect_esl
    return generate_latest(registry)
  File "/usr/local/lib/python3.8/dist-packages/prometheus_client/exposition.py", line 107, in generate_latest
    for metric in registry.collect():
  File "/usr/local/lib/python3.8/dist-packages/prometheus_client/registry.py", line 83, in collect
    for metric in collector.collect():
  File "/usr/local/lib/python3.8/dist-packages/asgiref/sync.py", line 147, in __call__
    return call_result.result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 432, in result
    return self.__get_result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 388, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.8/dist-packages/asgiref/sync.py", line 212, in main_wrap
    result = await self.awaitable(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/collector.py", line 285, in collect
    await ESLChannelInfo(esl).collect())
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/collector.py", line 234, in collect
    channelvars = json.loads(result)
  File "/usr/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
<IP_ADDRESS> - - [06/Apr/2021 08:20:03] "GET /esl?module=default&target=fs2.internal HTTP/1.1" 500 -

Great, thanks for that trace. Added an additional check before attempting to deserialize the json (cd1f36c387a9f30bf1a29fb935d50fcdc65393ec). The attached wheel includes that fix.
prometheus_freeswitch_exporter-1.0.0-py3-none-any.whl.gz

I'm getting the same output
Exception thrown while rendering view
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/http.py", line 103, in view
    return self._views[endpoint](**params)
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/http.py", line 55, in on_esl
    output = collect_esl(self._config[module], target)
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/collector.py", line 301, in collect_esl
    return generate_latest(registry)
  File "/usr/local/lib/python3.8/dist-packages/prometheus_client/exposition.py", line 107, in generate_latest
    for metric in registry.collect():
  File "/usr/local/lib/python3.8/dist-packages/prometheus_client/registry.py", line 83, in collect
    for metric in collector.collect():
  File "/usr/local/lib/python3.8/dist-packages/asgiref/sync.py", line 147, in __call__
    return call_result.result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 432, in result
    return self.__get_result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 388, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.8/dist-packages/asgiref/sync.py", line 212, in main_wrap
    result = await self.awaitable(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/collector.py", line 290, in collect
    await ESLChannelInfo(esl).collect())
  File "/usr/local/lib/python3.8/dist-packages/freeswitch_exporter/collector.py", line 239, in collect
    channelvars = json.loads(result)
  File "/usr/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
<IP_ADDRESS> - - [06/Apr/2021 09:05:33] "GET /esl?module=default&target=fs1.internal HTTP/1.1" 500 -

Ok, thanks for testing that. It looks like FS might be responding with -ERR No such channel! and that's obviously neither an empty string nor is it valid json. Thus it is necessary to cover that case as well. Pushed a fix into the other PR. A wheel including that fix is attached.
prometheus_freeswitch_exporter-1.0.0-py3-none-any.whl.gz

Great, works now... though I had to change startsWith to startswith on line 235 in collector.py

Great, works now... though I had to change startsWith to startswith on line 235 in collector.py

Oh, right. The other one is JavaScript syntax :sweat_smile:
I removed the initial/ineffective fixes in order to cleanup the PR. It would be much appreciated if you could give that a (hopefully) final test-round.
prometheus_freeswitch_exporter-1.0.0-py3-none-any.whl.gz

Yep, still looks good. 👍

Thanks for your help troubleshooting this issue. Released 1.0.1 on pypi which includes this fix.


FIO: Update NewFundsReq to testnet v0.9.0.

Description
Added a missing field to NewFundsRequest, to follow v0.9.0 testnet changes.
Still produces invalid tx, see issue #850 .
Also see https://developers.fioprotocol.io/api/api-spec/reference/new-funds-request/new-funds-request-model
Testing instructions
Unit tests, OK.
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] Prefix PR title with [WIP] if necessary.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] If there is a related Issue, mention it in the description (e.g. 'Fixes #444 ).

Don't merge yet; I made some progress with the help of FIO devs, fix is coming soon.

iOS has more stricter complier rules (like Implicit Integer conversion), could we add it to c++ side as well?


Refactor utilities to Game class

Refactor all game specific login into a game class that is used in all other classes. The utilities file should only contain helper methods that know nothing about the game object.

Finished c7c996955fac8eea2f6e3e8c8c30d2d150ced4e6


auro-avatar: use regional fallback image for valid airport codes

Is your feature request related to a problem? Please describe.
When a valid airport code does not yet have an avatar image to load we want to fallback to the avatar image for the region.
Describe the solution you'd like
In all cases of an invalid airport code we would continue to use the default image fallback.
In cases of a valid airport code we would check to see if the image loads for it. If not we would.

Fetch the list of airports with this
Get the matching region code from https://github.com/Alaska-ECommerce/ETInfoServiceCore/blob/main/ETInfoServiceCore/App_Data/ETAirlines.xml (@gusnaughton will update the XML to include necessary data)
Use the region code for the avatar image
Check that the URL using the region code actually returns an image
If that fails go ahead and use the default fallback

Describe alternatives you've considered
We could instead bake in a method for auro-avatar to fire an event when it fails to load an image for the designated code. This would enable the team to attempt to load the desired avatar, listen for the event flagging the failure and then determine an alternate code they want to use and insert that back in to the avatar.

Per Teams conversation with @gusnaughton
[8:14 AM] Gus Naughton
This is going to have to involve a larger conversation with the flight search backend team. The ETAirlines file is not the appropriate place to put location data, and the Airports files get generated and overwritten automatically.Get the matching region code from https://github.com/Alaska-ECommerce/ETInfoServiceCore/blob/main/ETInfoServiceCore/App_Data/ETAirlines.xml (@gusnaughton will update the XML to include necessary data)
[8:14 AM] Gus Naughton
I can provide the JSON representation of country codes to regions, but I'm not sure where the right place for storing those constants are. Maybe they can be design tokens?

We could instead bake in a method for auro-avatar to fire an event when it fails to load an image for the designated code. This would enable the team to attempt to load the desired avatar, listen for the event flagging the failure and then determine an alternate code they want to use and insert that back in to the avatar.

This is the approach we are planning on.
The idea of regional fallbacks is definitely a good idea for someone to do but that gets way to much into business logic to be an open source auro component.
Currently the component does support adding custom images and with that approach you can do anything you want and still keep the current url pattern.  I would recommend having a separate utility managed by flight search that handles regional or custom fallbacks before they are passed into auro-avatar.  And if that fails auro-avatar will still have a default backup image ready to go.

Closing after discussion as this issue is over a year old with no updates.  If this is still important, please message for re-opening.


Make stubs for HTTP requests extendable

User Stories
As a developer, I'd like to...

[x] Include stubs for Tinder requests

...so the specs for better_tinder follow open-closed principle and don't change the functionality of the parent

Closed on #5


No module named 'qiskit_terra'_How to solve this?

Environment

Qiskit version: 0.39.2
Qiskit Terra version: 0.22.2
Python version: 3.9.7
Operating system: macOS Monterey 12.6

What is happening?
I'm learning quantum adder through the materials from here and tried to run these code about class of quantum full adder. (I worry about invading the authors's copyright so I just write concerned codes I guessed. Sorry.)
from numpy import binary_repr
from qiskit.quantum_info import Statevector
# from qiskit.ignis.mitigation.measurement import (complete_meas_cal, CompleteMeasFitter)
# from qiskit.ignis.mitigation.measurement import (complete_meas_cal, CompleteMeasFitter)
from qiskit_terra.mitigation.measurement import (complete_meas_cal, CompleteMeasFitter)
from qiskit_terra.mitigation.measurement import (complete_meas_cal, CompleteMeasFitter)

NOTES: qiskit.ignis on the 4th and 5th line is now replaced to qiskit-terra.
Then retuned this error message.
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
/var/folders/6j/chq1zh5x0kg9tp5_v_m0hck00000gn/T/ipykernel_14708/533723145.py in <module>
      1 from numpy import binary_repr
      2 from qiskit.quantum_info import Statevector
----> 3 from qiskit_terra.mitigation.measurement import (complete_meas_cal, CompleteMeasFitter)
      4 from qiskit_terra.mitigation.measurement import (complete_meas_cal, CompleteMeasFitter)
      5

ModuleNotFoundError: No module named 'qiskit_terra'

I conformed that the PATH is already made in the following steps.

I opened a terminal tab and run pip3 show qiskit-terra and gained /Users/(My name)/opt/anaconda3/lib/python3.9/site-packages
I added

import sys
print(sys.path)

before importing qiskit-terra and jupyter notebook retured '/Users/(Myname)/.local/lib/python3.9/site-packages',  as one of available PATH.
How can we reproduce the issue?
as above
What should happen?
I'd like to run the concerned codes without errors.
Any suggestions?
No response

FYI we also have adders implemented in Qiskit directly, see e.g. the docs.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/networking e937f69...8925a50:
  > 8925a50 upgrade to latest dependencies (# 463)
  > d965698 upgrade to latest dependencies (# 462)
bumping knative.dev/pkg 7d1b0f1...dbcf4cf:
  > dbcf4cf removing default for KUBECONFIG to enable in cluster tests (# 2191)
  > 7764284 include OpenCensus in the list of metric backend (# 2189)
  > aff7376 drop stackdriver metrics and tracing exporters (# 2183)
bumping knative.dev/eventing 0af15fd...25bd8ef:
  > 25bd8ef move source.StatsReporter from knative.dev/pkg (# 5594)
  > 1677431 upgrade to latest dependencies (# 5592)
bumping knative.dev/reconciler-test ebe91f9...91e5b20:
  > 91e5b20 upgrade to latest dependencies (# 228)

/lgtm
/approve


Window flashes and quits. Won't run

System Details

OS: [e.g.  Ubuntu 22.04, Windows 7, 8.1, 10, 11]
Thorium Version [e.g.107.0.5347.0]

Problem
A clear and concise description of what the bug is.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional Notes
Add any other context about the problem here.

please try to use this version
https://github.com/Alex313031/Thorium-SSE3/releases/tag/M117.0.5938.157-1

works for me thx :)

@Alex313031 this issue is solved.


Tasks without return value would cause panic when Get is called

Tasks without return value would cause panic when Get is called.

I think Get method works correctly.
in gocelery_test.go, each test case can get err from Get method with 2s timeout.
res, err := asyncResult.Get(TIMEOUT)
if err != nil {
	t.Errorf("test '%s': failed to get result for task %s: %+v", tc.name, tc.taskName, err)
	cli.StopWorker()
	continue
}

However, panic is caused at releaseResultMessage function in message.go because v == nil, right?
func releaseResultMessage(v *ResultMessage) {
	v.reset()
	resultMessagePool.Put(v)
}

@kobayashi you are right. I am curious about how Celery checks if the method has been executed without checking the result.

added test case: https://github.com/gocelery/gocelery/blob/ab4dc82c38ba560a068ade9e0d4d9eb81fc42729/gocelery_test.go#L98-L170


Failed prop type: Invalid prop data[0].label

I am using the Dropdown with data such as:
const dropDownValues = [
    { value: "value1", label: "Label one" },
    { value: "value2", label: "Label two" },
];

The dropdown renders and functions fine, however I am getting a react-native warning:
Warning: Failed prop type: Invalid prop data[0].label of value Label one supplied to Dropdown, expected one of [null, null].
I suspect this is just a problem with the prop type definitions?

Thanks for reporting! Released fix in 0.5.1.


Fix coco/ozar merge mistake

Merge mistake fixed in ThangTypeConstants.coffee:
ozariaHeroes and ozariaCinematicHeroes were moved under heroes property by the merge, but they should be directly on ThangTypeConstants
This is how the original .ozar. file looked like.

We actually don't need any if/else for product-specific logic here. Ozaria can have all the CodeCombat heroes and hero classes listed under heroes and heroClasses–they just won't be used by Ozaria's hero-related views. Similarly, CodeCombat can have all Ozaria heroes and cinematic heroes listed under ozariaHeroes and ozariaCinematicHeroes. So we should get rid of the isOzaria check and just have a simple object of constants that encompasses those needed for both products.


Update driver for fully static Linux.

The Swift driver needs to understand the new triple and needs to know the platform name to which that corresponds.
rdar://123442522

@swift-ci Please test

@swift-ci Please test

@swift-ci test windows


Update Annotation Handling

The Request tab wasn't displaying when the only data that would show up there were annotations.
Re-rendered the examples with raml2html 6.6.0

Sorry for the long wait, and thanks for the PR!


Using "PlaylistItems: list" to get all the videos for a particular channel will cause an error.

Using PlaylistItems: list to get all the videos for a particular channel will cause an error.
Because it get broadcat too, you can not use CommentThreads: list in broadcat.

use Search: list instead of  PlaylistItems: list.
drop column channels.playlist_id


Editing Manual Test Cases: Are there more than the 3 States: Closed, Design, Ready?

Hello,
I am looking for Test Case States additional to the default three. For example, before a Test Case is Ready, it is being reviewed, but it is done being Designed. Can I add another State called Review?
Thank you

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

ID: 2cf72265-156c-de8c-d729-ba7b3b90193d
Version Independent ID: 1a0c7272-6d36-6036-06cf-5801b55e2f70
Content: Create manual test cases - Azure Test Plans
Content Source: docs/test/create-test-cases.md
Product: devops
Technology: devops-test
GitHub Login: @steved0x
Microsoft Alias: sdanie

@vividmonkey -- Vivid, thank you for your question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@steved0x -- Steve, please look into this issue.


Update FAQ for Jib CLI

When changing titles, don't forget to update the ToC and link refs.

And please test the new ToC links after merging. (I made a mistake with the Jib config FAQ before, if you remember 😄)

Thanks for the heads up! Verified the links to make sure they were okay:)


Register PolgyonAlgorithms.jl package

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/89492
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 049cf077a11d30180245beabee03930d0cde5a46
git push origin v0.1.0

Registration was successful


Antimony mining

The Antimony mining wont work.
After selecting the 14th stone,the macro go bact to Menü with this error message :
653: attempt to perform archmetric on a nil value

I have just tried to mine Antinomy this morning and cannot replicate this, It may be that automato is struggling to read your Main chat tab, I will PM you in game and see if I can help you solve it from there :)


[Docs]: translate the docs under the contribute to APISIX directory

Current State
Description of the current state/issue:
Translate the contents of the Contribute to APISIX directory, including:
contributor-guide
blog-contributing-guide
For documentation style, please refer to 中文文档写作指南.

hi, I want to do this job.

Recommend adding translation tag

blog-contributing-guide in progress, I will complete it recently


chore: add pypi base decorator

add pypi_base decorator.
TODO can be handled in this PR if support for disabled and python attributes gets merged in first.

fixed and merged into #1505


Set skipRange to match current version

/cherry-pick release-3.14

/cherry-pick release-3.14

Not sure why I thought this would work 🤦♂


Added FindByPhoneNumberAsync

This PR addresses the FindByPhoneNumberAsync functionality request in UserStore and UserManager, and the IPhoneNumberStore interface
Related issue https://github.com/aspnet/Identity/issues/704
I am still trying to get xunit to run normally to commit the unit tests as well...

Hi @SharePointRadi, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@SharePointRadi, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

I added a unit test to validate that we can find users by phone number.

@rustd, @HaoK, @blowdart I cannot recall what we decided in triage about this PR.
In any case, if we wanted to take it I am not sure about the name of the method. Across a few components we are pretty consistently using the Find prefix for methods that return zero or one instances based on a condition or key. This is more of a GetUsersByPhoneNumber() I think.

I am happy to follow through any decisions you guys make and amend it

We decided that we don't think this is appropriate as general functionality since apps can do this easily themselves.


[Vue.js] Error during build and in terminal (yarn encore --dev watch or yarn build)

I have one strange problem. When I include a component inside another another component (controller) I get not found error in terminal and when running yarn build.
But everything works as expected in the browser and no console errors.
So I'm Trying to include Loading.vue component inside CRUDIndex.vue file (which is a controller as described in Symfony UX documentation).
I opened issue here and not stackoverflow because it looks like an inconsistency as it works as expected, but throwing error in the terminal.
Here is the error in terminal:
ERROR  Failed to compile with 1 errors                                                                                6:17:51 PM

error  in /var/www/html/project/assets/vue/controllers/CRUDIndex.vue.ts                                                                  6:17:51 PM

[tsl] ERROR in /var/www/html/project/assets/vue/controllers/CRUDIndex.vue.ts(5,21)
     TS2307: Cannot find module '../components/Common/Loading.vue' or its corresponding type declarations.

It could be. I ended up removing typescript related stuff. But anyway there wasn't CrudIndex.vue.ts it was just CrudIndex.vue  and inside of it just included another component and everything was working in the browser, except that terminal error, so very likely encore problem.
I'll try to bring back typescript support and try the solution and see if the suggested solution in your provided link works an will debug a little more. Will let you know.
Thanks for your help @weaverryan 👍

I had the same issue - to fix it, I added shims-vue.d.ts to the ./assets/vue/ folder with the following contents:
// See https://stackoverflow.com/a/70803320/339213
// If we don't include this file, we get this compile error whenever we try to load a .vue file:
// TS2307: Cannot find module<EMAIL_ADDRESS>or its corresponding type declarations.
declare module '*.vue';

This should be included in the documentation.


cant find package json-server within node_modules

PNPM and NPM are reporting that it cannot find json-server from import statement within calling server.js file.
server.js file
import jsonServer from 'json-server';
console.log('Starting JSON Server');

package.json file
{
  "name": "crazy",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "type": "module",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "start": "node server.js"
  },
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "json-server": "^1.0.0-beta.1"
  }
}

The error:
ajackman@MarsVenus:~/projects/ep/crazy$ npm run start

<EMAIL_ADDRESS>start
node server.js

node:internal/modules/esm/resolve:210
const resolvedOption = FSLegacyMainResolve(packageJsonUrlString, packageConfig.main, baseStringified);
^
Error: Cannot find package '/home/ajackman/projects/ep/crazy/node_modules/json-server/package.json' imported from /home/ajackman/projects/ep/crazy/server.js
at legacyMainResolve (node:internal/modules/esm/resolve:210:26)
at packageResolve (node:internal/modules/esm/resolve:828:14)
at moduleResolve (node:internal/modules/esm/resolve:914:18)
at defaultResolve (node:internal/modules/esm/resolve:1119:11)
at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:542:12)
at ModuleLoader.resolve (node:internal/modules/esm/loader:511:25)
at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:241:38)
at ModuleJob._link (node:internal/modules/esm/module_job:126:49) {
code: 'ERR_MODULE_NOT_FOUND'
}
Node.js v22.2.0
npm version 9.5.1
pnpm version 9.1.0
node version 22.2.0
WSL2 / Debian 11 (Guest) / Windows 11 (host)

Same issue as above +1

This issue began with version 1.0 (anything).  I walked simple project
backwards thru all published packages. The one that worked was version
0.17.4.
On Tue, Jun 4, 2024, 17:30 Bankole @.***> wrote:

Same issue as above +1
—
Reply to this email directly, view it on GitHub
https://github.com/typicode/json-server/issues/1557#issuecomment-2148515672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACFHJTBYRQAKRIG5462HH3ZFY5Y5AVCNFSM6AAAAABIYO7K7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBYGUYTKNRXGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

i think this is because the line:
"main": "./lib/server/index.js",

which used to be in the package.json on 0.17.4 no longer exists on 1.0 versions.
which was pointing to the main script, but the 1.0 versions seem to only support the CLI variant.
and thus since this package has become no longer usable like that.
npm install --save-dev<EMAIL_ADDRESS>
should be the fix for now,
while asking the maintainers what their plans are surrounding support for running this as express middleware.

It would be nice if the maintainers had plans to allow it to run as an express middleware. In my case, I need to manipulate some header information that my app expects to see in server call results.


Add new Moving Average Processor

Purpose

A new data processor has been added. The Moving Average Filter is used to smooth the data of a data stream.
Approach

Samples

Remarks

Fixes:

Hey @SvenO3 thanks a lot for providing this processor.