InteractiveBrowserCredential has no way to sign out

Package Name: @azure/identity
Package Version: 2.0.0-beta.3
Operating system: Windows
[ ] nodejs

version: 14.15.0

[ ] browser

name/version: Edge Insiders 92.0.891.1

[ ] typescript

version: 4.1.2

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Using the InteractiveBrowserCredential in a React SPA gives no way to sign out once the user is authenticated. I attempted to force a sign out by redirecting the browser to `` which appeared to work, but the next usage of the credential succeeded without any login prompt.
It appears that there are some values left in session storage after the sign out that is causing this.

To Reproduce
Steps to reproduce the behavior:

Unzip repro project signout-repro.zip
Register a SPA app in the Azure portal with redirect http://localhost:3000
Paste your app ID in App.tsx where it says YOUR_APP_ID
Run yarn install or npm install
Run the app with yarn start or npm start
Click Sign in, sign in with an account
Click Sign out
When returned to the app, click Sign in again.

Expected behavior
You should be prompted to login again inside a pop-up window.
Ideally the credential object should have some function to explicitly invalidate or log out.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@jasonjoh We have provided the following information in our documentation: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/identity/identity/samples/AzureIdentityExamples.md#authenticating-with-the-azuremsal-browser-public-client Let me know if that’s helpful!


Convert log timestamps to CEST time zone

closes #1694
Checklist before merging

[ ] I added an appropriate entry to CHANGELOG.md
[ ] I added logging to new code - if appropriate.
[ ] I merged main into the current branch.

Current logging at commit https://github.com/fractal-analytics-platform/fractal-server/pull/1719/commits/ff9d01665140e24764df4b7e594480167672e4c3 :
2024-08-30 16:08:53(+2:00)::409 - fractal_server_settings - DEBUG -   FRACTAL_SLURM_SSH_WORKING_BASE_DIR: None
2024-08-30 16:08:53(+2:00)::410 - fractal_server_settings - DEBUG -   FRACTAL_API_SUBMIT_RATE_LIMIT: 0
2024-08-30 16:08:53(+2:00)::411 - fractal_server_settings - DEBUG -   FRACTAL_RUNNER_TASKS_INCLUDE_IMAGE: Copy OME-Zarr structure;Convert Metadata Components from 2D to 3D
2024-08-30 16:08:53(+2:00)::412 - fractal_server_settings - DEBUG -   FRACTAL_API_V1_MODE: include
2024-08-30 16:08:53(+2:00)::413 - fractal_server_settings - DEBUG -   FRACTAL_MAX_PIP_VERSION: 24.0
2024-08-30 16:08:53(+2:00)::414 - fractal_server_settings - DEBUG -   FRACTAL_LOG_TIMEZONE: Europe/Rome
2024-08-30 16:08:53(+2:00)::494 - fractal_server.lifespan - INFO - End application startup

2024-08-30 16:08:57(+2:00)::842 - uvicorn.error - INFO - Finished server process [26359]
2024-08-30 16:08:57(+2:00)::849 - gunicorn.error - ERROR - [pid 26354] - Worker (pid:26359) was sent SIGINT!
2024-08-30 16:08:57(+2:00)::930 - uvicorn.error - INFO - Waiting for application shutdown.
2024-08-30 16:08:57(+2:00)::931 - uvicorn.error - INFO - Waiting for application shutdown.
2024-08-30 16:08:57(+2:00)::933 - uvicorn.error - INFO - Application shutdown complete.
2024-08-30 16:08:57(+2:00)::933 - uvicorn.error - INFO - Waiting for application shutdown.
2024-08-30 16:08:57(+2:00)::933 - uvicorn.error - INFO - Application shutdown complete.
2024-08-30 16:08:57(+2:00)::934 - uvicorn.error - INFO - Finished server process [26358]
2024-08-30 16:08:57(+2:00)::935 - uvicorn.error - INFO - Finished server process [26357]
2024-08-30 16:08:57(+2:00)::935 - uvicorn.error - INFO - Application shutdown complete.
2024-08-30 16:08:57(+2:00)::936 - uvicorn.error - INFO - Finished server process [26360]
2024-08-30 16:08:57(+2:00)::943 - gunicorn.error - ERROR - [pid 26354] - Worker (pid:26358) was sent SIGINT!
2024-08-30 16:08:57(+2:00)::944 - gunicorn.error - ERROR - [pid 26354] - Worker (pid:26357) was sent SIGINT!
2024-08-30 16:08:57(+2:00)::945 - gunicorn.error - ERROR - [pid 26354] - Worker (pid:26360) was sent SIGINT!
2024-08-30 16:08:58(+2:00)::038 - gunicorn.error - INFO - [pid 26354] - Shutting down: Master

2024-08-30 12:25:46(GMT-2:30)::182 - fractal_server_settings - DEBUG -   FRACTAL_API_V1_MODE: include
2024-08-30 12:25:46(GMT-2:30)::183 - fractal_server_settings - DEBUG -   FRACTAL_MAX_PIP_VERSION: 24.0
2024-08-30 12:25:46(GMT-2:30)::184 - fractal_server_settings - DEBUG -   FRACTAL_LOG_TIMEZONE: Canada/Newfoundland
2024-08-30 12:25:46(GMT-2:30)::220 - fractal_server.lifespan - INFO - End application startup
2024-08-30 12:25:46(GMT-2:30)::253 - fractal_server.lifespan - INFO - End application startup
2024-08-30 12:25:46(GMT-2:30)::267 - fractal_server.lifespan - INFO - End application startup

2024-08-30 16:56:26(GMT+2)::442 - fractal_server_settings - DEBUG -   FRACTAL_API_SUBMIT_RATE_LIMIT: 0
2024-08-30 16:56:26(GMT+2)::443 - fractal_server_settings - DEBUG -   FRACTAL_RUNNER_TASKS_INCLUDE_IMAGE: Copy OME-Zarr structure;Convert Metadata Components from 2D to 3D
2024-08-30 16:56:26(GMT+2)::444 - fractal_server_settings - DEBUG -   FRACTAL_API_V1_MODE: include
2024-08-30 16:56:26(GMT+2)::446 - fractal_server_settings - DEBUG -   FRACTAL_MAX_PIP_VERSION: 24.0
2024-08-30 16:56:26(GMT+2)::446 - fractal_server_settings - DEBUG -   FRACTAL_LOG_TIMEZONE: Europe/Rome
2024-08-30 16:56:26(GMT+2)::523 - fractal_server.lifespan - INFO - End application startup

Closing, see https://github.com/fractal-analytics-platform/fractal-server/issues/1694#issuecomment-2337368554


Fix typo

Fixes a typo.

There are some more, in the code comments. :wink:

The others are for components we do not actively use, nor should we change them, because they will just come back from upstream. ;) Merged, thanks!


Creating a new project - blog website using html css javascript

Hey there, I would like to contribute in your repo 50 days 50 web project by adding a web project of blog website. This will be a functional websites with great features. Plz assign the issue so that I can start working on it.

done @Ayushswirlon
do changes and ss while making PR

hey @dhairyagothi i was going to start the work on blog page but someone already do it #184 and mine was #169
so should i make a new one or just update the already one as it was too basic

He has made something else
you continue with your issue no worries i will merge and give points
make a PR after completing @Ayushswirlon

ok then thanks 👍

completed


Add support for las files with both, 1byte and 2byte color information

Right now, a color range of 0-65000 is expected but a lot of point clouds are in range 0-255.
If possible, range should automatically be detected by analyzing the first chunk of points.
Additionaly, if the range parameter is set, this should be used.

Done: https://github.com/potree/PotreeConverter/commit/614aec477b98a26197f9ac94c43a3ae9d7970606


python-bitcoinlib doesn't need to be in requirements.txt

It is already included transitively from squeaklib.
https://github.com/yzernik/squeaknode/blob/f73226b621a82721b1e0ffb595e7011aeb0b384d/requirements.txt#L15

Keep it because the code is imported explicitly.


Get devise key from ENV

We can use the same secret for secret_key_base and devise_secret. That will need to be set in the Heroku environment.
Should address #144

@Ross-Hunter I think we need to tell Heroku to generate this value for review apps via app.json.

I just set DEVISE_SECRET for staging and production.


Kafka-backup.jar is missing

Hi
I cloned the source file from github in the (latest) stable version and use it like described in the READ.me. When I run the following command + arguments
$ bash backup-standalone.sh --bootstrap-server localhost:9092 --target-dir /home/azhmago/ --topics 'First-Topic' --debug
on my kafka-host, I get the following error:
Cannot find the kafka-backup.jar in .. Please set --backup-jar accordingly
I cannot find the kafka-backup.jar file in the whole repository or find the error itself somewhere described.
Can you help me, please?
Best,
M.

Done by myself


Can't load png dataset with 4 channel (RGBA)

I try to create dataset which contains about 9000 png images 64x64 in size, and they are all 4-channel (RGBA). When trying to use load_dataset() then a dataset is created from only 2 images. What exactly interferes I can not understand.

Hi! Can you please share the directory structure of your image folder and the load_dataset call? We decode images with Pillow, and Pillow supports RGBA PNGs, so this shouldn't be a problem.

Hi! Can you please share the directory structure of your image folder and the load_dataset call? We decode images with Pillow, and Pillow supports RGBA PNGs, so this shouldn't be a problem.

I have only 1 folder that I use in the load_dataset function with the name "IMGDATA" and all my 9000 images are located in this folder.
`
from datasets import load_dataset
dataset = load_dataset("IMGDATA")
`
At the same time, using another data set with images consisting of 3 RGB channels, everything works

Okay, I figured out what was wrong. When uploading my dataset via Google Drive, the images broke and Pillow couldn't open them. As a result, I solved the problem by downloading the ZIP archive


[Snyk] Upgrade @aurodesignsystem/auro-menu from 3.9.5 to 3.10.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @aurodesignsystem/auro-menu from 3.9.5 to 3.10.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 24 days ago, on 2023-05-23.

Release notes

Package name: @aurodesignsystem/auro-menu

3.10.0 - 2023-05-233.10.0 (2023-05-23)
Bug Fixes

layout: correct margin on selected icon #152 (d5c2360)

Features

disable: enable disabling entire menu rather than just options individually #175 (db507fd)

Performance Improvements

deps: update several dependencies to latest versions (685ee09)
examples: refactor js example execution so it works on docsite #176 (ffe429f)

3.9.5 - 2023-02-073.9.5 (2023-02-07)
Bug Fixes

slots: pass noCheckmark attribute down on each slot change #170 (782e5e5)

    from @aurodesignsystem/auro-menu GitHub release notes

Commit messages

Package name: @aurodesignsystem/auro-menu

170f3af chore(release): 3.10.0 [skip ci]
685ee09 perf(deps): update several dependencies to latest versions
d5c2360 fix(layout): correct margin on selected icon #152
db507fd feat(disable): enable disabling entire menu rather than just options individually #175
ffe429f perf(examples): refactor js example execution so it works on docsite #176

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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


[BUG] The information of keys should be hided if not click for drop down icon

Describe the Bug
When click account keys in wallet page and navigate to account keys page, it shown all information of keys.
The video which show this error atta
https://github.com/Outblock/FRW-Android/assets/87978227/935a1fd7-865f-41bd-882b-bee89028a2b0
ched below.
Expected experience
According to UI design, the page should only show name of keys, not all information.

Steps To Reproduce

Go to profile page.

Environment
- Android: 17

Already solved.
The actual Ui is same as Ui design now.


Add basic pre-commit checks

Adds some basic pre-commit checks and modified required files to pass all added checks:
black
codespell
flake8
isort
mypy

Well thats unexpected

Okay it appears this is due to the fact the current CI pipeline is still using python3.9 instead of python3.10

The only thing missing rn is mypy but we can add that once support for python3.9 is removed


Auth.refreshSession does not work inside API/getServerSideProps

Maybe this issue could be solved with #62
I wanted to have a NextJS API or page with getServerSideProps that will refresh the session to get the latest user_meta_data of the authenticated user.
I was doing it client side now but sometimes it was failing because I needed to wait a little bit for a trigger to finish before fetching the new user_meta_data, but I thought anyway it was better to do this server side so I created an API for that. (Any suggestion for a best solution is accepted).
I then tried using the supabase.auth.refreshSession() both on an API and on a getServerSideProps but it is failing with the following error:
Refreshed user:  {
  data: null,
  user: null,
  error: Error: No current session.
      at SupabaseAuthClient.eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:557:27)
      at Generator.next (<anonymous>)
      at eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:8:71)
      at new Promise (<anonymous>)
      at __awaiter (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:4:12)
      at SupabaseAuthClient._callRefreshToken (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:554:16)
      at SupabaseAuthClient.eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:242:46)
      at Generator.next (<anonymous>)
      at eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:8:71)
      at new Promise (<anonymous>)
}
Error while refreshing the user data:  Error: No current session.
    at SupabaseAuthClient.eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:557:27)
    at Generator.next (<anonymous>)
    at eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:8:71)
    at new Promise (<anonymous>)
    at __awaiter (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:4:12)
    at SupabaseAuthClient._callRefreshToken (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:554:16)
    at SupabaseAuthClient.eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:242:46)
    at Generator.next (<anonymous>)
    at eval (webpack-internal:///../../node_modules/.pnpm/@supabase+gotrue-js@1.22.12/node_modules/@supabase/gotrue-js/dist/main/GoTrueClient.js:8:71)
    at new Promise (<anonymous>)

API code:
import {
  supabaseServerClient,
  withAuthRequired
} from '@supabase/auth-helpers-nextjs';

export default withAuthRequired(async (req, res) => {
  let { returnTo = '/' } = req.query;
  returnTo = Array.isArray(returnTo) ? returnTo[0] : returnTo;
  returnTo = returnTo.charAt(0) === '/' ? returnTo : `/${returnTo}`;

  // Refreshes the user data
  const refreshedUser = await supabaseServerClient({
    req
  }).auth.refreshSession();

  console.error('Refreshed user: ', refreshedUser);

  if (refreshedUser.error) {
    console.error(
      'Error while refreshing the user data: ',
      refreshedUser.error
    );
  }

  res.status(200).json({ refreshedUser });
  // res.redirect(returnTo as string);
});

Basically same code for the getServerSideProps:
export const getServerSideProps = withAuthRequired({
  redirectTo: '/',
  async getServerSideProps(ctx) {
    // Refreshes the user data
    const refreshedUser = await supabaseServerClient(ctx).auth.refreshSession();

    console.error('Refreshed user: ', refreshedUser);

    if (refreshedUser.error) {
      console.error(
        'Error while refreshing the user data: ',
        refreshedUser.error
      );
    }
ab
    return {
      redirect: {
        destination: '/',
        permanent: false
      }
    };
  }
});

Watching in the lib code it seems that here (https://github.com/supabase/gotrue-js/blob/master/src/GoTrueClient.ts#L671) the refresh_token is empty so the currentSession loaded server side does not have the refreshToken but has the accessToken set because it does not fail here (https://github.com/supabase/gotrue-js/blob/master/src/GoTrueClient.ts#L336).
I tried using the auth.api directly to refresh the accessToken:
  const refreshedUser = await supabaseServerClient({
    req,
  }).auth.api.refreshAccessToken(req.cookies["sb-refresh-token"]);

but it's not working either.
The refresh token is "refreshed" but apparently not saved because the next time you access the same page the refresh token is invalid and you get an error.

You can use the getUser method server-side: https://github.com/supabase-community/supabase-auth-helpers/blob/main/src/nextjs/README.md#server-side-rendering-ssr---withauthrequired

Alternatively, from version 1.4.0 onward you will be able to call supabase.auth.refreshSession() client-side to refresh the JWT & user object throughout.

@thorwebdev I'll use the new function has soon as available then! Thanks!


gtid test is not stable

Which jobs are flaking?
gtid
Which test(s) are flaking?
DM-IT
Jenkins logs or GitHub Actions link

[2021-12-31T03:51:47.253Z] dmctl test cmd: "start-task /home/jenkins/agent/workspace/dm_ghpr_integration_test/go/src/github.com/pingcap/tiflow/dm/tests/gtid/conf/dm-task.yaml --remove-meta"

[2021-12-31T03:51:50.571Z] check diff successfully

[2021-12-31T03:51:50.571Z] grep: /tmp/dm_test/gtid/worker2/dumped_data.test/metadata: No such file or directory

[2021-12-31T03:51:50.571Z] new_gtid1 7b92d616-69ec-11ec-947a-62c5a03f7bac:6 new_gtid2 :3

[2021-12-31T03:51:51.136Z] ERROR 1774 (HY000) at line 1: Malformed GTID specification ':3'.

[2021-12-31T03:51:51.136Z] ...

https://ci2.pingcap.net/blue/organizations/jenkins/dm_ghpr_integration_test/detail/dm_ghpr_integration_test/2383/pipeline
Anything else we need to know

Does this test exist for other branches as well?

Has there been a high frequency of failure lately?

related to https://github.com/pingcap/tidb-tools/issues/583


'Zip' dependency not handled for Carthage users

Currently this library doesn't have a Cartfile to address the 'Zip' dependency for Carthage users. I've made a small patch at #31, which is still a work in progress.

This is handled I believe by #31 if the Examples directory is deleted. Otherwise, there will be a build error due to a conflict between all of the files from the 'Zip' library that have been manually copied to the Examples directory and the library itself when checked out with Carthage. Why is the Examples directory something included in the library?

Yes this seems to be the case, will review your PR and update the fix hopefully during this week!

Awesome, thanks so much @mostafaberg 👍 😄

Fixed now with help from @jakerockland 's PR #33 :)


I didn't open.

Sorry guyz, I am new in Android Studio.  I have an error like this
"error running 'ARCore-Location-master-app': Default Activity not found"
Thank you for help.

In the Activity to MAIN action and category LAUNCHER manifest file, you must set
        <activity
            android:name="<Activity class name>">
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>


Help with NEC input

Hello,
I am trying to use your code to send an IR signal though a 3.5 mm cable. The NEC code that I am trying to send is 0x40F740BF (32 Bits). How would I send that using the necTransmitter.SendNEC() command? When I try inputing the hex code as is I get an error message.

Hi Curtis. As explained here, a NEC command is actually 16 bit, not 32 bit. That's why the API expects 2 (address + command) 8 bit values.
In your case, your input 0x40F740BF translates in binary to 0b01000000111101110100000010111111, which doesn't seem fully correct.
The command is 0b01000000 (the last part 0b10111111 is the inverse), which translates to 0x40 or 64.
The 16 address bits don't match op though, the first 8 bits are not the logical inverse of the next 8 bits, are you sure the input is correct?
Given the address is correct (and the logical inverse incorrect), the command for you would be:
SendNEC(0x40, 0x40)

I hope this helps.


a different theme option

added the theme I use myself as an option and changed the docs and directory structure to make it simple to add more. Hope you think it's worth including :)

Hey, I've been on hiatus this year and just got back. We could try to create other variations of the theme for those that want it. I'd be more happy to accept your theme and others as well.


Investigate whether OR conditions can be supported.

Migration of https://github.com/michaellavelle/spring-data-dynamodb/issues/3

+1

After doing a little research, it looks like implementing the OR condition is dependent upon first implementing filter expressions (see #42) as the previous implementation path using setConditionalOperator is now considered a legacy parameter.
AWS Documentation on the Conditional Operator (such as in a DynamoDBQueryExpression):

This is a legacy parameter, for backward compatibility. New applications should use FilterExpression instead. Do not combine legacy parameters and expression parameters in a single call; otherwise, DynamoDB will return a ValidationException exception.
http://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_Query.html#ConditionalOperator

AWS Documentation on Filter Expressions:
http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/QueryAndScan.html#FilteringResults
I did not find anywhere in the code base in which the conditional operator for a query is explicitly set to AND. I'm assuming that AND is already supported because the conditional operator defaults to AND.

That's correct. #28 needs to go away first and would make things easier for sure in general


Set up clang-tidy in CI

clang-tidy seems like a useful linter. C++ could really use all the linters it can get.
https://github.com/marketplace/actions/clang-tidy-review looks useful but it might
be nice to be able to run clang-tidy locally as well thru nix. Perhaps this action
is better for that: https://github.com/marketplace/actions/clang-tidy-action . ie.
run clang-tidy thru nix, get the output file, then use the action to post inline comments
on the PR.
Would also be nice to only clang-tidy on changed lines, much like git-clang-format.

+100 to more linting


Adding a button to create a .sql file

It would be nice to have this option.

@roypaulin do we close this one?

Yes


Fix windows tests

Windows tests are failing due to cookiecutter issue:
https://github.com/cookiecutter/cookiecutter/issues/1688
This is solved with a PR to pytest-cookie but it's not merged (despite months of time...):
https://github.com/hackebrot/pytest-cookies/pull/61
So this PR uses that PR branch for pytest-cookies to eliminate the fails.
If the PR is every merged, can go back to just pytest-cookies

Hmm, this PR may be even better of a fix?
https://github.com/hackebrot/pytest-cookies/pull/63

The other fix PR has been merged:
https://github.com/hackebrot/pytest-cookies/pull/63
So it may be possible to revert this PR, assuming a release is made.

I was released, so feel free to make such PR. I will review it.


UI to "dump curves"

Save all currently applied curves to a user specified folder, selected through the file picker

And shotsprocessing?!


support obsolete postgres json format

because, in fact, people still use it :D

I think in general that there are two things we'd ideally like to support:

Extract data from legacy json columns as regular portable DataType::Json.
Insert DataType::Json data into existing json columns in already-created tables.

I'm not convinced that it's worthwhile to support creating new tables with legacy json columns. The universal simplifying assumption of dbcrossbar is that we focus on best practices and "reasonable" data types, and we don't try to support the full set of data types of a specific database.

Besides (1) and (2) above, can you think of any other use cases for legacy json types?

@emk agreed in general, the only use case i need to support here is copying INTO existing json columns.

This branch contains a variety of different fixes, including at least:

Fixes for strange column names. (Fixed in #66.)
A workaround for legacy json columns. (Fixed differently in #67.)
Disabling of BigQuery UPSERT checks. The recommended workaround for this is to edit the schema to mark all merge key fields as NOT NULL, which will inform dbcrossbar that you're taking responsibility for the issue.

Since the major features in this branch are fixed elsewhere (or have workarounds) and there's no plan to make this branch mergeable, I'm going to close this PR. Please feel free to re-open it if you want.


Unknown locale u'ia'

Details

Project URL: https://readthedocs.org/projects/linuxmint-installation-guide-ia
Build URL (if applicable): https://readthedocs.org/projects/linuxmint-installation-guide-ia/builds/6647565/
Read the Docs username (if applicable): linuxmint

Expected Result
The documentation should be in Interlingua.
Actual Result
The documentation is in English.
More Info
Hi,
There seem to be a regression recently. This documentation has been compiling successfully for a month now and always properly showed up in Interlingua.
A few days ago, we noticed it now showed up in English instead.
The RTD configuration didn't change and neither did the PO translations.
Looking at the build logs, it seems Sphinx was updated and now seems more strict than before with non-supported languages.
The list of supported languages by Sphinx is available at http://www.sphinx-doc.org/en/stable/config.html#confval-language. It does not contain Interlingua.
In the latest build (which fails to be translated in Interlingua), we see the following warning:
Running Sphinx v1.6.5
making output directory...
loading translations [ia]... not available for built-in messages
loading pickled environment... done
building [mo]: targets for 17 po files that are out of date
writing output... [  5%] ../po/ia/LC_MESSAGES/burn.mo
WARNING: reading error: /home/docs/checkouts/readthedocs.org/user_builds/linuxmint-installation-guide-ia/checkouts/latest/docs/../po/ia/LC_MESSAGES/burn.po, unknown locale u'ia'

In previous builds (where the documentation was properly translated), we saw warnings as well, but they didn't prevent the translation from taking place:
Running Sphinx v1.5.6
making output directory...
loading translations [ia]... not available for built-in messages
loading pickled environment... not yet created
building [mo]: targets for 17 po files that are out of date
writing output... [  5%] ../po/ia/LC_MESSAGES/burn.mo
WARNING: reading error: /home/docs/checkouts/readthedocs.org/user_builds/linuxmint-installation-guide-ia/checkouts/latest/docs/../po/ia/LC_MESSAGES/burn.po

I'm not sure that's the cause of the issue. I'm not even sure it's related. I'm mentioning it in case it helps.

Closing this as isn't a RTD issue but Sphinx.


🐛Config entry 'Xplora®' for xplora_watch integration not ready yet: 'text'; Retrying in background

What version of Integration has the issue?
@Ludy87
I did install version 2.7.0 and set-up the integration but it not starts and I get following messages in my logs
Logger: homeassistant.config_entries
Source: config_entries.py:424
First occurred: 21:21:24 (1 occurrences)
Last logged: 21:21:24
Config entry 'Xplora®' for xplora_watch integration not ready yet: 'text'; Retrying in background
AND
This error originated from a custom integration.
Logger: custom_components.xplora_watch.coordinator
Source: custom_components/xplora_watch/coordinator.py:130
Integration: Xplora® Watch (documentation, issues)
First occurred: 21:21:24 (6 occurrences)
Last logged: 21:24:18
Unexpected error fetching xplora_watch-5115 data: 'text'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 239, in _async_refresh
self.data = await self._async_update_data()
File "/config/custom_components/xplora_watch/coordinator.py", line 123, in _async_update_data
data = await self.data_loop(wuids, message_limit, remove_message)
File "/config/custom_components/xplora_watch/coordinator.py", line 130, in data_loop
res_chats = await self.controller.getWatchChatsRaw(wuid, limit=message_limit, show_del_msg=remove_message)
File "/usr/local/lib/python3.10/site-packages/pyxplora_api/pyxplora_api_async.py", line 368, in getWatchChatsRaw
result = ChatsNew.from_dict(result.get("chatsNew", None))
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/api.py", line 83, in from_dict
return _decode_dataclass(cls, kvs, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 201, in _decode_dataclass
init_kwargs[field.name] = _decode_generic(field_type,
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 275, in _decode_generic
res = _decode_generic(type_arg, value, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 263, in _decode_generic
res = get_type_cons(type)(xs)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 306, in
items = (_decode_dataclass(type_arg, x, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 201, in _decode_dataclass
init_kwargs[field.name] = _decode_generic(field_type,
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 273, in _decode_generic
res = _decode_dataclass(type_arg, value, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 159, in _decode_dataclass
field_value = kvs[field.name]
KeyError: 'text'
System Health details
System Information

version
core-2023.2.3

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.10.7

os_name
Linux

os_version
5.15.90

arch
aarch64

timezone
Europe/Bucharest

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
4708

Installed Version
1.30.1

Stage
running

Available Repositories
1203

Downloaded Repositories
20

AccuWeather

can_reach_server
ok

remaining_requests
46

Home Assistant Cloud

logged_in
true

subscription_expiration
February 15, 2023 at 02:00

relayer_connected
true

remote_enabled
true

remote_connected
true

alexa_enabled
false

google_enabled
true

remote_server
eu-central-1-4.ui.nabu.casa

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 9.5

update_channel
stable

supervisor_version
supervisor-2023.01.1

agent_version
1.4.1

docker_version
20.10.22

disk_total
113.9 GB

disk_used
41.3 GB

healthy
true

supported
true

board
odroid-n2

supervisor_api
ok

version_api
ok

installed_addons
Spotify Connect (0.12.2), Home Assistant Google Drive Backup (0.110.1), File editor (5.5.0), SSH & Web Terminal (13.0.2), Samba share (10.0.0), MariaDB (2.5.2), ESPHome (2022.12.8), Assistant Relay (0.7.4), Mosquitto broker (6.1.3), Zigbee2MQTT (1.30.1-1)

Dashboards

dashboards
1

resources
12

views
11

mode
storage

Recorder

oldest_recorder_run
February 3, 2023 at 14:24

current_recorder_run
February 13, 2023 at 21:18

estimated_db_size
1601.95 MiB

database_engine
mysql

database_version
10.6.10

Sonoff

version
3.4.0 (5406fa7)

cloud_online
10 / 10

local_online
10 / 10

Spotify

api_endpoint_reachable
ok

Checklist

[X] I have enabled debug logging for my installation.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).

Describe the issue
I did install version 2.7.0 and set-up the integration but it not starts and I get following messages in my logs
Logger: homeassistant.config_entries
Source: config_entries.py:424
First occurred: 21:21:24 (1 occurrences)
Last logged: 21:21:24
Config entry 'Xplora®' for xplora_watch integration not ready yet: 'text'; Retrying in background
AND
This error originated from a custom integration.
Logger: custom_components.xplora_watch.coordinator
Source: custom_components/xplora_watch/coordinator.py:130
Integration: Xplora® Watch (documentation, issues)
First occurred: 21:21:24 (6 occurrences)
Last logged: 21:24:18
Unexpected error fetching xplora_watch-5115 data: 'text'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 239, in _async_refresh
self.data = await self._async_update_data()
File "/config/custom_components/xplora_watch/coordinator.py", line 123, in _async_update_data
data = await self.data_loop(wuids, message_limit, remove_message)
File "/config/custom_components/xplora_watch/coordinator.py", line 130, in data_loop
res_chats = await self.controller.getWatchChatsRaw(wuid, limit=message_limit, show_del_msg=remove_message)
File "/usr/local/lib/python3.10/site-packages/pyxplora_api/pyxplora_api_async.py", line 368, in getWatchChatsRaw
result = ChatsNew.from_dict(result.get("chatsNew", None))
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/api.py", line 83, in from_dict
return _decode_dataclass(cls, kvs, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 201, in _decode_dataclass
init_kwargs[field.name] = _decode_generic(field_type,
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 275, in _decode_generic
res = _decode_generic(type_arg, value, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 263, in _decode_generic
res = get_type_cons(type)(xs)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 306, in
items = (_decode_dataclass(type_arg, x, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 201, in _decode_dataclass
init_kwargs[field.name] = _decode_generic(field_type,
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 273, in _decode_generic
res = _decode_dataclass(type_arg, value, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 159, in _decode_dataclass
field_value = kvs[field.name]
KeyError: 'text'
Reproduction steps
Not Available
Debug logs
Not Available

@Ludy87 this did solve my problem, thanks!

I'm having the same issue, with v2.7.0 as well as v2.7.1:
`Logger: custom_components.xplora_watch.coordinator
Source: custom_components/xplora_watch/coordinator.py:136
Integration: Xplora® Watch (documentation, issues)
First occurred: 21:19:57 (12 occurrences)
Last logged: 21:26:03
Unexpected error fetching xplora_watch-66674 data: 'text'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 239, in _async_refresh
self.data = await self._async_update_data()
File "/config/custom_components/xplora_watch/coordinator.py", line 127, in _async_update_data
self.watch_entry.update(await self.data_loop(wuids, message_limit, remove_message))
File "/config/custom_components/xplora_watch/coordinator.py", line 136, in data_loop
res_chats = await self.controller.getWatchChatsRaw(wuid, limit=message_limit, show_del_msg=remove_message)
File "/usr/local/lib/python3.10/site-packages/pyxplora_api/pyxplora_api_async.py", line 365, in getWatchChatsRaw
result = ChatsNew.from_dict(result.get("chatsNew", None))
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/api.py", line 83, in from_dict
return _decode_dataclass(cls, kvs, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 201, in _decode_dataclass
init_kwargs[field.name] = _decode_generic(field_type,
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 275, in _decode_generic
res = _decode_generic(type_arg, value, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 263, in _decode_generic
res = get_type_cons(type)(xs)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 306, in
items = (_decode_dataclass(type_arg, x, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 201, in _decode_dataclass
init_kwargs[field.name] = _decode_generic(field_type,
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 273, in _decode_generic
res = _decode_dataclass(type_arg, value, infer_missing)
File "/usr/local/lib/python3.10/site-packages/dataclasses_json/core.py", line 159, in _decode_dataclass
field_value = kvs[field.name]
KeyError: 'text'`

@Milenco I had to delete and add one of the 2 watches I had to my phone app, after that did work for me

Thanks for your quick response @smarthomefamilyverrips. I've deleted the configuration and set-up again with my childs watch but received the same issue. I've removed the configuration, rebooted Home Asssistant and tried again, with the same result.
Will now try to completely remove the installation, re-install and re-configure it, with Home Assistant reboots in between.

@Milenco I did delete everything also actually, delete Integration, remove from HACS (custom component folder) reboot, delete watch from phone app so it does factory reset on watch, add watch again to app (logged in with my own phone number) after that installed xplora again with HACS, reboot and set up Integration using my phone number as account login

if it doesn't work after the new installertion, then do the following:
go to custom_components/xplora_watch folder
edit the manifest.json file
{
  "domain": "xplora_watch",
  "name": "Xplora® Watch",
  "version": "v2.7.1",
  "config_flow": true,
  "documentation": "https://github.com/Ludy87/xplora_watch/tree/main",
  "issue_tracker": "https://github.com/Ludy87/xplora_watch/issues",
  "dependencies": [],
  "requirements": [
    "pyxplora_api==2.5.20",
    "geopy==2.2.0",
    "dataclasses-json",
    "pydub"
  ],
  "codeowners": ["@Ludy87"],
  "iot_class": "local_polling",
  "integration_type": "device"
}

Thanks for clarifying @smarthomefamilyverrips . I just deleted the integration and removed from HASC, made sure the custom component folder was empty, rebooted Home Assistant and added it again. Want to prevent having to factory reset the watch since it's not near me at the moment.
Thanks @Ludy87 for your response as well. Will try that and report back here.

Yes, that did the trick @Ludy87: Updating the manifest to use API version 2.5.21 instead of 2.5.20 removed the issue. The integration seems to be fully working again :)
Thanks again!

@Ludy87 it did fix it for me also, actually when I did see this message and checked my devices I did see all is unavailable and doing a reload gave me again the "text" message, after the change in manifest.json and a HA restart all is back to normal again, thanks for the quick fix! 👍🏻

@Ludy87 and @Milenco can I ask if either of you uses a mesh network? I have the problem that my X6Play watches not see my ASUS aimesh network as a single network showing once the SSID but instead shows me every single node making it show the same SSID 4 times (once for each node/accesspoint) ... so was wondering if one of you experienced the same

@smarthomefamilyverrips Please use the discussion area!


Help with sending coverage data

Hi, I have generated my test coverage using jest --coverage command. But I am lost at sending them to codacy. Can anyone help me out with the exact steps I have to do to send the coverage to codacy ?

Have you looked at the README file? https://github.com/codacy/node-codacy-coverage#usage
This is how we send coverage for this project: https://github.com/codacy/node-codacy-coverage/blob/master/circle.yml#L11

I have looked at the readme before but didn't understand it. But after you left a comment I managed to figure out to send the coverage.
Steps were from -

Generate coverage from your favourite testing tool. Mine is jest so did a "jest --coverage" that generated all coverage.
Install node-codacy-coverage with either npm/yarn.
To send out the report you need to add codacy project token to your project either via .env file or in the next command.
CODACY_PROJECT_TOKEN='xxxxxxxxxxxxx'|cat ./coverage/lcov.info | ./node_modules/.bin/codacy-coverage .
This will send the coverage report to codacy.

Thanks for your help @pedrorijo91 . 👍

Trying to use an environment variable but with not success.
Here is what i've tried:
1)
//1)
"test-with-coverage": "CODACY_PROJECT_TOKEN=1234567890 jest --coverage || true && cat ./coverage/lcov.info | codacy-coverage"

//2)
"test-with-coverage": "jest --coverage || true && cat ./coverage/lcov.info | CODACY_PROJECT_TOKEN=1234567890 codacy-coverage"

//3)
"test-with-coverage": "jest --coverage || true && CODACY_PROJECT_TOKEN=1234567890 | cat ./coverage/lcov.info | codacy-coverage"

(Also tried every option with single quotes on the token, e.g '1234567890')
All gave the same result:
[error] "2019-02-13T12:55:05.485Z"  'Error sending coverage'
[error] "2019-02-13T12:55:05.487Z"  Error: Token is required
    at Object.module.exports [as handleInput] (myProject/node_modules/codacy-coverage/lib/handleInput.js:22:35)
    at Socket.<anonymous> (myProject/node_modules/codacy-coverage/bin/codacy-coverage.js:48:20)
    at Socket.emit (events.js:164:20)
    at endReadableNT (_stream_readable.js:1054:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)

What Am i missing?
P.S. I'm using "jest-cli": "21.2.1" and "babel-jest": "21.2.0"

@pedrorijo91 Any ideas?

maybe @rtfpessoa can help here


Roslyn folder removed when publishing using AWS Toolkit or the upload and deploy button on beanstalk

Expected Behaviour
The Roslyn folder should not be removed otherwise the website will not work. If I RDP into the environment and manually copy the Roslyn folder through the mapped remote drives then the web site works.
Current Behaviour
When publishing my .NET Framework 4.7 web site using C# via the Toolkit on VS2017, the published folder when uploaded excludes the Roslyn folder which contains the Roslyn compiler and its resources. Manually publishing it to a folder locally as 'release' includes the folder which I then zip up and try to upload through the AWS beanstalk console using the 'upload and deploy' button on my beanstalk environment which also seems to remove the Roslyn folder after having deployed. When I open the zip file, the Roslyn folder is there.
Obtained the following from the beanstalk logs
Exception type: HttpException
Exception message: Could not find a part of the path 'C:\inetpub\wwwroot\bin\roslyn\csc.exe'.

at System.Web.Compilation.BuildManager.ReportTopLevelCompilationException()
at System.Web.Compilation.BuildManager.EnsureTopLevelFilesCompiled()
at System.Web.Compilation.BuildManager.CallAppInitializeMethod()
at System.Web.Hosting.HostingEnvironment.Initialize(ApplicationManager appManager, IApplicationHost appHost, IConfigMapPathFactory configMapPathFactory, HostingEnvironmentParameters hostingParameters, PolicyLevel policyLevel, Exception appDomainCreationException)
Possible Solution
Include the Roslyn folder when publishing the website using the toolkit on VS2017 or through the upload and deploy button when uploading zip file which clearly has the folder when previewing it.
Steps to Reproduce (for bugs)
Create a .NET 4.7 web project on VS2017 and try to upload it using the toolkit or through the upload and deploy button within the beanstalk console on AWS site.
Context
Can't publish the site and it means I have to have a decent connection to the internet as it prevents me from mobile working. It's not as easy as it used to be where I could just publish and see it work. Should be the same as Azure where you click and it publishes. Azure just works but AWS seems to be a bit broken at the moment.
Your Environment

Operating System and version: Windows Server 2016 / IIS10
Visual Studio version: 2017
Targeted .NET platform: .NET Framework 4.7

.NET Core Info
Not using .NET Core.

I've run into the same issue last week, the problem for me seems to be with roslyn v.1.0.6 and v1.0.7.
At first I reported the issue to the roslyn team, but on further investigation it looks like the microsoft deploy includes the missing folder.
Your Environment

Operating system and version: Windows 10
Visual Studio version: 2017
Targeted .NET platform: .NET 4.6

I have produced the error as well with version 1.0.7 and like @ghost1face found it does work correctly with version 1.0.5. I'll keep investigating but for now the work around is to use version 1.0.5.

downgrading to version 1.0.5 worked for me as well. I did run into an issue after downgrading where the bin folder was locked as read-only .
Similar to what was reported here https://developercommunity.visualstudio.com/content/problem/71302/binroslyn-files-locked-during-build.html
Removing the read-only lock through the properties window on the bin folder resolved this , and I was able to get my app successfully published to the aws eb env.


Update rust-channel to nightly-2020-04-19

Also needed another patch for wee_alloc (https://github.com/Ma27/wee_alloc/commit/0f9954eddcbd210a296333f6106c8c492d78ea12).
Todos before merging:

[ ] consider creating a PR for wee_alloc to support latest nightly
[ ] merging the wee_alloc patch into our fork (and give me push-access :p)
[ ] tests/reviews to make sure my changes to the allocator API are at least somehow reasonable

• You should have push access now
• can you trigger the build of a new docker container? (just uptate the Dockerfile and push to master) and then update the drone.yaml to pass ci tests

Investigate possible usage of init_offset to avoid evil hacks in our code

deprecate the simple alloc and tidy up code @JanPlusPlus

You should have push access now

Thanks!

• can you trigger the build of a new docker container? (just uptate the Dockerfile and push to master) and then update the drone.yaml to pass ci tests

Done. I assume we have to restart the build once again when the container is built?

Done. I assume we have to restart the build once again when the container is built?

Sorry, I should have mentioned, that you have to change the docker tag in the drone.yaml as well.
Now the new version is published under the old tag

Do you know why the container still downloads dependencies during the test/format steps?
This does not occur during the build step and leads to longer ci times (the ci should not take more than 5 minutes for one build, currently 6:30 min)

@TrueDoctor I updated the nixpkgs rev in this change, but I decided to keep building the docker container using the shell.nix from master. Due to the newer revision, several things changed and we therefore need to download more stuff. By either merging this PR or building the container with the shell.nix from this branch, that should be fixed I guess.


Instrumented libraries not reported when called from run_in_executor

Description
Instrumentation of module works, but if call is wrapped in loop_run_in_executor then it no longer works.
For example, using mysqlclient database, this works fine:
database.connect().execute("select sleep(2)") # Shows up as MySQL on NR

But if I use run_in_executor then it's not detected properly:
await asyncio.get_running_loop().run_in_executor(
    None, lambda: database.connect().execute("select sleep(2)")
) # Shows up as Python on  NR

Expected Behavior
NewRelic would detect MySQL calls regardless of whether they were executed in run_in_executor
Troubleshooting or NR Diag results
Steps to Reproduce
Use run_in_executor to execute code for a instrumented library
Your Environment
Python 3.10.7
sanic 22.9.0
newrelic 8.2.1
Additional context
Both transaction as they were reported in NR:

This is currently expected behavior; however this does fit into a feature request category.  Our PM is aware of this and we have a FR to review with the request here.


Allow scaling of collision shapes and rigid bodies for easier subscene management

Sometimes you implement full part of scene in a subscene, including collision objects and rigid bodies,
for easier testing, then you need to integrate this part into the whole main scene, which requires scaling. With current approach you have to split the resulting scene into physics/collision objects and
visual objects and integrate these separately, actually recreating physics/collision part. There should be
some mechanism for making this less tedious task.

this is planned for 1.1. However, for sphere and capsule, only uniform
scale will work (This limitation is standard in all physics engines).
On Thu, Mar 12, 2015 at 12:12 PM, Sergey Lapin<EMAIL_ADDRESS>wrote:

Sometimes you implement full part of scene in a subscene, including
collision objects and rigid bodies,
for easier testing, then you need to integrate this part into the whole
main scene, which requires scaling. With current approach you have to split
the resulting scene into physics/collision objects and
visual objects and integrate these separately, actually recreating
physics/collision part. There should be
some mechanism for making this less tedious task.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1505.

err sorry, for 1.2
On Thu, Mar 12, 2015 at 12:14 PM, Juan Linietsky<EMAIL_ADDRESS>wrote:

this is planned for 1.1. However, for sphere and capsule, only uniform
scale will work (This limitation is standard in all physics engines).
On Thu, Mar 12, 2015 at 12:12 PM, Sergey Lapin<EMAIL_ADDRESS>wrote:

Sometimes you implement full part of scene in a subscene, including
collision objects and rigid bodies,
for easier testing, then you need to integrate this part into the whole
main scene, which requires scaling. With current approach you have to split
the resulting scene into physics/collision objects and
visual objects and integrate these separately, actually recreating
physics/collision part. There should be
some mechanism for making this less tedious task.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1505.

As far as I know, scaling is implemented for all collision shapes by now.
However, scaling rigid bodies is not yet possible.

neither works very well, but adding status checks / warnings to nodes
should make this more obvious
On Sun, Nov 1, 2015 at 12:17 AM, eska014<EMAIL_ADDRESS>wrote:

As far as I know, scaling is implemented for all collision shapes by now.
However, scaling rigid bodies is not yet possible.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/1505#issuecomment-152790004.

I think if straightforward approach doesn't work for subscenning of bodies
and
collision shapes, some cheating should be implemented...
On Sun, Nov 1, 2015 at 7:09 AM, Juan Linietsky<EMAIL_ADDRESS>wrote:

neither works very well, but adding status checks / warnings to nodes
should make this more obvious
On Sun, Nov 1, 2015 at 12:17 AM, eska014<EMAIL_ADDRESS>wrote:

As far as I know, scaling is implemented for all collision shapes by now.
However, scaling rigid bodies is not yet possible.
—
Reply to this email directly or view it on GitHub
<https://github.com/godotengine/godot/issues/1505#issuecomment-152790004
.

—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/1505#issuecomment-152792416.

neither works very well, but adding status checks / warnings to nodes should make this more obvious
@reduz

Warnings were recently added, IINW.

I think the idea is to rework the collision shapes system completely for 3.0, so it will likely be fixed at that time.

I think this is implemented in 6ba1e467 for 2D, but not yet for 3D.

Missing for 3D, will do when porting 3D demos
On Jun 28, 2017 11:20 AM, "Rémi Verschelde"<EMAIL_ADDRESS>wrote:

I think this is implemented in 6ba1e46
https://github.com/godotengine/godot/commit/6ba1e4677ba15992c750bddffcb9f1eacd1558a1
for 2D, but not yet for 3D.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/1505#issuecomment-311673669,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z22FnpfSE0iRaYfGGG88Et4IIET0qks5sImCigaJpZM4DtsV4
.

this is implemented now, closing


Adding cascade DML limitation

#sign-off

@arunkumarthiags: I'm sorry - only the author of this article, @HJToland3, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


support util.promisify in Node

Purpose (TL;DR) - mandatory

Fixes #223, similar to what we did for jest-fake-timers.
Noticed after starting there's already https://github.com/sinonjs/lolex/pull/227, but that seems stale and I'm not sure what some of the changes are for.
I'll do what I can to incorporate any review comments and get this through! :sweat_smile:
If there's consensus that storing a global.Promise backup is a good idea, can do that, but would do another PR first, changing this in existing code.

Anything more to do here? @fatso83 :)
@SimenB not sure how important the storing global.Promise thing is. Since it's not directly related to this, I'd do a separate PR for it, or we can go without it and see if someone actually runs into an issue? Your call

Anything more to do here?

You haven't addressed "Should do setImmediate at the same time".

we can go without it and see if someone actually runs into an issue?

It has hit people in Jest, which is why we have tests for checking that we still work even if people swap out globals (e.g. the whole reason https://github.com/facebook/jest/blob/master/scripts/babel-plugin-jest-native-globals.js exists. That transform only guards Jest's own source code though, not other libraries). I'm fine with doing it separately, it's nothing to block on

You haven't addressed "Should do setImmediate at the same time".

Right, always miss the comments in the reviews themselves :sweat_smile: pushed :+1:

It has hit people in Jest

I know, it was more of a "are the people changing globals going to be the same people using lolex fake timers" ;D

(Build error also happens on master, test passes for me locally)

@jeysal I can't see why this branch would fail to build on Circle. Would you mind rebasing it, to see if that might solve the build error in Circle?

done

worked 🎉

@fatso83 are you happy with the changes made? Is there anything left to do?

(I've put @simenb's suggestion about capturing global.Promise on my to-do list for a follow up, but this one is complete I think)

<EMAIL_ADDRESS>

By the way this doesn't work for setTimeout (global) because the setTimeout[util.promisify.custom] custom promisified function is not set correctly there.
I assume that's because this for loop is intent on copying the properties on clock.setTimeout to setTimeout (global).
However, the clock.setTimeout[util.promisify.custom] is not copied.


ulimit won't work if the current user doesnot have the limit higher or equal to 'rabbitmq' user

Hi,
I had been fighting to make rabbitmq get higher limit by specifing this attribute for 'rabbitmq' cookbook.
default['rabbitmq']['open_file_limit'] = 65536
However, because the nature of the inheritance of the ulimit. If the current user has lower limit than what rabbitmq user expect, even though he has 'sudo' privilege, it wouldn't work.
As it is said below,  http://unix.stackexchange.com/questions/108603/do-changes-in-etc-security-limits-conf-require-a-reboot
The only way to make it work is to add ulimit at /etc/default/rabbitmq, not /etc/rabbitmq/rabbitmq-env.conf
so that the 'root' user would read '/etc/default/rabbitmq' and applied the expected limit so that 'rabbitmq' user later inherited it.
This might be 'rabbitmq' init script's fault or not !?

another addition
currently the env file gets generated after the start of rabbit-mq - which is kind of not the right order.

Hi, I would like to update the fix.
It is involving of sudo module for pam. There is patch in this ulimit cookbook to fix it

@chrisduong @cmluciano How about #127?

I closed as ulimit value in /etc/default/rabbitmq would completely fix it.

Hi @jjasghar,
I have to reopen this, because I'm not sure how would you deal with ulimit at latest tag v4.0.0.
I can see that you keep both ulimit config in /etc/default/rabbitmq and /etc/rabbitmq/rabbitmq-env.conf.  This is duplicated and can cause trouble. The code managed rabbitmq-env.conf is put ahead and make rabbitmq-server restart immediately, then would fail right away if there is no patched for /etc/pam.d/sudo in ulimit cookbook.
template "#{node['rabbitmq']['config']}.config" do
  sensitive true
  source 'rabbitmq.config.erb'
  cookbook node['rabbitmq']['config_template_cookbook']
  owner 'root'
  group 'root'
  mode 00644
  variables(
    :kernel => format_kernel_parameters,
    :ssl_versions => (format_ssl_versions if node['rabbitmq']['ssl_versions']),
    :ssl_ciphers => (format_ssl_ciphers if node['rabbitmq']['ssl_ciphers'])
  )
  notifies :restart, "service[#{node['rabbitmq']['service_name']}]", :immediately
end

We should manage ulimit in one place. Either in /etc/default/rabbitmq or /etc/rabbitmq/rabbitmq-env.conf.

This is indeed a problem.  I am getting the following in my /var/log/rabbitmq/startup_err:
/usr/lib/rabbitmq/bin/rabbitmq-server: 12: ulimit: error setting limit (Operation not permitted)

I have the same issue and agree with the previous comments: This should only be set once, preferable in the /etc/default/rabbitmq which controls how the service is started. I'd not expect ulimit to be useful in the /etc/rabbitmq/rabbitmq-env.conf which should only hold environmental vars.
Could this be updated since it's a blocked for us to upgrade to the otherwise great new version. We specifically want to use the additional configuration options now supplied via template to get rid of the template changes we added.


Add support for ENS addresses across the application.

Currently there are places in the app that do not support using an ENS based ethereum address.  The most obvious is in the send token modal from the Assets page.

We should support ENS addresses as they are less error prone for users. Also worth considering reverse lookup functionality when people connect a wallet that has a ENS address attached to it.

@0xean I'd like to take a look.

The basic support functionality could be accomplished with a regex in @shapeshiftoss/chain-adapters/src/ethereum/EthereumChainAdapter.ts:

new RegExp(/([A-Za-z]*\.)?([A-Za-z]{3,})\.eth/).test('foo.vitalik.eth')

chain-adapter should then be able to handle the address since it uses ethers.js which has ENS support.

Reverse lookup can be achieved on the client with web3.eth.ens.getAddress.
That would require a spec (at least a simple one i.e which component to use to display the address) if there are visual changes needed.

I am not online much this weekend.  Please go ahead and we can make sure to get some FOX your way for this.  Thank you so much!

On Jan 8, 2022, at 8:57 AM, Alexandre GOMES @.***> wrote:

@0xean I'd like to take a look.
The basic support functionality could be accomplished with a regex in @shapeshiftoss/chain-adapters/src/ethereum/EthereumChainAdapter.ts:
new RegExp(/([A-Za-z]*.)?([A-Za-z]{3,}).eth/).test('foo.vitalik.eth')
chain-adapter should then be able to handle the address since it uses ethers.js which has ENS support.
Reverse lookup can be achieved on the client with web3.eth.ens.getAddress.
That would require a spec (at least a simple one i.e which component to use to display the address) if there are visual changes needed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.

@gomesalexandre - our product team is going to take a look at this and find all the place that will support ENS names.
We can add that information here once they have it compiled.

here you go @0xean and @gomesalexandre, please lmk if you have any questions!
If a user's address is set to resolve to an ENS name, they see the ENS name in place of the address in the following places:

TX History

Send Confirmation

If a user's address is set to resolve to an ENS name, they see the ENS name in addition to their address in the following places:

Receive modal

@willyogo @0xean looks good! I've just implemented the receive one in #704 and the send confirmation and tx history should be straightforward, more or less a similar implementation.
I'll run the flow many more times, if I see more places where resolution might be needed I'll add them here for reference

@gomesalexandre - awesome. Attaching a bounty to this reserved for you now.
Let's call the acceptance criteria:

ENS support in all places mentioned in above comment from @willyogo
Error messaging for an invalid ens address (one that cannot be resolved) or a malformed ens address
No regressions, passes CI tooling, pushes logic down into the correct packages in lib if needed

argh... gitcoin bugs are a real pain sometime. Opening a separate issue to track the bounty.

@gomesalexandre  - https://gitcoin.co/issue/shapeshift/web/721/100027439

@gomesalexandre - awesome. Attaching a bounty to this reserved for you now.
Let's call the acceptance criteria:
1. ENS support in all places mentioned in above [comment](https://github.com/shapeshift/web/issues/674#issuecomment-1009377401) from @willyogo

2. Error messaging for an invalid ens address (one that cannot be resolved) or a malformed ens address

3. No regressions, passes CI tooling, pushes logic down into the correct packages in lib if needed

Sweet, applied to it so no other contributor sees it and loses time for something that's already partly done - will continue work on it end of day so PRs should move out of draft soon!


Can't access static files in '/app/public' when enable egg-webpack

Before I enable egg-webpack I can access my css through http://<IP_ADDRESS>:7001/public/css/main.css.
But after I enable egg-webpack in plugin.local.js, add a webpack.config.js to webpackConfigList,I can't access http://<IP_ADDRESS>:7001/public/css/main.css(it returns 200 but empty content).
I can't work it out so I look for some help here.
Thx!

@sanmaopep latest version had fixed, you can try.   local mode , the file read from memory,  when file not exist,  the request will next eggjs process.


getFollowers bug

Setting startFollower does not appear to make a difference:
    steem.getFollowers('ned', 0, 'blog', 5, function(err, result) {
        console.log(result);
        // callback(result);
    });

Result:
[ { id: '8.0.63115',
    follower: 'aarnold',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.106374',
    follower: 'aaronkoenig',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.4991',
    follower: 'aaseb',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.70969',
    follower: 'aboundlessworld',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.3663',
    follower: 'acidsun',
    following: 'ned',
    what: [ 'blog' ] } ]

Incrementing startFollower by 5:
    steem.getFollowers('ned', 5, 'blog', 5, function(err, result) {
        console.log(result);
        // callback(result);
    });

Result:
[ { id: '8.0.63115',
    follower: 'aarnold',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.106374',
    follower: 'aaronkoenig',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.4991',
    follower: 'aaseb',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.70969',
    follower: 'aboundlessworld',
    following: 'ned',
    what: [ 'blog' ] },
  { id: '8.0.3663',
    follower: 'acidsun',
    following: 'ned',
    what: [ 'blog' ] } ]

Reproduced in #26

I have verified this bug as well. any projection on when it can be fixed?

What actually happens is that get_followers takes a start_follower parameter, which is an username, not a number.
The correct usage would be:
const result1 = await steem.getFollowersAsync('ned', 0, 'blog', 5)
// First 5
const result2 = await steem.getFollowersAsync('ned', result1[result1.length - 1].follower, 'blog', 5)
// Last entry + next 4

I'm creating an issue for parameter validation, but this doesn't seem like a bug in the wrapper itself.

#53


SDL2 Wayland support

Describe the bug
Wayland is not supported via SDL2.  Fedora and Ubuntu ship default with Wayland.  Only X11 is supported based on third_party/libsdl2/include/SDL_config_linux.h.
More current version of SDL2 than in Filament auto generates config header based on CMake detection.
To support Wayland today requires patching third_party/libsdl2/include/SDL_config_linux.h, and when doing that libsdl2 CMake doesn't generate protocol code.
Also when cross compiling wayland-scanner in libsdl2 is incorrectly detected.  https://github.com/libsdl-org/SDL/issues/4860
To Reproduce
Steps to reproduce the behavior:
CC=clang CXX=clang++ cmake .. -D BUILD_SHARED_LIBS=OFF -D CMAKE_BUILD_TYPE=Release -D FILAMENT_SUPPORTS_VULKAN=TRUE -D FILAMENT_SUPPORTS_OPENGL=FALSE -D FILAMENT_USE_SWIFTSHADER=FALSE -D FILAMENT_SUPPORTS_XCB=FALSE -D FILAMENT_SUPPORTS_XLIB=FALSE -D FILAMENT_ENABLE_LTO=TRUE

Expected behavior
Vulkan on Wayland is supported
Desktop (please complete the following information):

OS: Ubuntu 20 / Fedora 34
Backend: Vulkan

We'd be happy to take a look at a PR if you're up for it, yes. Thanks for offering.

What is the status of this one?

Done!


Add a copy of the MIT license to the repository

Could you add the MIT license text in a file called LICENSE.md at the top-level? Having just "License: MIT in README.md" may not be considered enough to be legally binding.

Oh yes, that was supposed to be in here but I think it got removed on accident. I'll fix that up when I have the chance later today!
Thanks @Calinou

Added in b50d5da


[flutter_tool] Reland: support --fast-start for Android applications (as an opt-in)

Description
TLDR: Reduces the time it takes to get from flutter run to an interactive debugging session. Locally this reduces the startup time for a dirty application from ~40 seconds in the flutter gallery to about ~12 seconds.
The default behavior of flutter run is to build the platform binary (APK, IPA) as necessary and update it on the device before starting a debug session. This artifact is almost always invalidated whenever the user restarts a debugging session, because it depends on all Dart sources, assets, and native dependencies. Depending on the speed of the user's workstation, this can take anywhere from several seconds to several minutes to rebuild.
Both Dart code and Flutter assets can be loaded at runtime from the devfs instead of from the APK - this is how hot restart works. If the Dart source code and assets are removed as inputs from the APK, then invalidations will only occur on Flutter version updates or plugin changes, which will be significantly less frequent than source code changes.
Instead of building the user's application in the initial APK, we build the empty "splash" application which looks like so:

After syncing files, the tool performs a silent hot restart to begin the debugging session.
Known issues: IDEs are not supported because we cannot hot restart into a paused state https://github.com/flutter/flutter/issues/45424

Currently testing if build-filters are the cause of the skipped tests

Testing that test fails...

Removed CI/build changes

@feinstein if you'd like to try this out, its available on the latest master. Remember, to see the improvements you have to build the APK with the flag at least once

Ok, but can't promise I will as I tend to avoid changing my flutter framework to avoid headaches.

How did this address the issues I raised with it last time?


[Task (tooling/wallet)]: Revisit "logout/wipe" setting

The wallet had a "logout" feature in advance settings which removed everything stored: all accounts, local storage etc... we should revisit this topic and think if this is something we want to have in the wallet.
This feature got disabled in task https://github.com/iotaledger/iota/issues/1800

This feature was kept so we can close this issue


Border fills node and ignores radius for wasm32

Bevy version
4648f7b
Relevant system information
AdapterInfo { name: "ANGLE (NVIDIA Corporation, NVIDIA GeForce RTX 2070 SUPER/PCIe/SSE2, OpenGL 4.5.0)", vendor: 4318, device: 0, device_type: Other, driver: "", driver_info: "WebGL 2.0 (OpenGL ES 3.0 Chromium)", backend: Gl }

What you did
Ran the borders example targeting wasm32
What went wrong
When I run the borders example targeting x86 it all renders as it should but when the target is wasm32 its rendered incorrectly.

Additional information
Initially posted this as a comment in #14955, link to comment
Everything renders correctly on my laptop
AdapterInfo { name: "ANGLE (AMD, AMD Radeon Vega 8 Graphics (raven LLVM 15.0.7), OpenGL 4.6)", vendor: 4098, device: 0, device_type: Other, driver: "", driver_info: "WebGL 2.0 (OpenGL ES 3.0 Chromium)", backend: Gl }

Also tested it on my computer at work
AdapterInfo { name: "ANGLE (Intel, Mesa Intel(R) UHD Graphics 770 (ADL-S GT1), OpenGL 4.6)", vendor: 32902, device: 0, device_type: IntegratedGpu, driver: "WebGL 2.0 (OpenGL ES", driver_info: "3.0 Chromium)", backend: Gl }

One thing that stands out when comparing the adapter info is that on the computers where its rendered correctly its using OpenGL 4.6 while the other computer uses OpenGL 4.5.0

Does https://github.com/bevyengine/bevy/pull/15163 help at all with this?

Does #15163 help at all with this?

Not at all unfortunately, but it seems to be even more broken now on bevy main.
Your branch

Bevy main (8bfe635)

Also tried it with the commit I used originally in the issue and it looks like it did before. Im going to try to bisect it to see what commit made it worse

a0f5ea0 is the commit that broke it further

Yeah that makes sense, that commit changed outlines to use the shader to draw the outlines with border radius too.
I could add a mode to the shader which would instruct it to ignore border radius and always draw square borders. It's not ideal but at least apps would be usable again.

Yeah, that useable apps are preferable :sweat_smile:  Let me know if you need me to do any more testing

This is still an issue, below is a screenshot from the Borders example on the bevy website.

Ive now also seen it on a colleagues computer, he also has an Nvidia GPU and is on Linux so the problem seems to be specifically with WASM on Linux machines with an Nvidia GPU.

I'm having the same issue. I'm seeing the same screen as the image above, running on a Linux machine with an Nvidia GPU, only in Wasm.


Check primenode keys on app start

Previously entering an invalid primenode key would start the wallet without error, if it was an invalid format it had the potential to throw an error otherwise the wallet would attempt to stake and not say the key was invalid until finding and creating a stake transaction and attempting to Accept the block (failing during ConnectInputs).
This will create a test signature and process it during init prompting if the key is invalid.

Requires #237 (will require rebase after #237's merge).


Add first cut of git-credential-fdoss

Add the initial version of git-credential-fdoss. It basically works
but has some unsupported cases such as:

Handling "prompts"
Handling locked secrets (probably requires prompts to work)
Using a non-default keyring
Creating a non-existent default keyring (not even sure we we should,
but this is something libsecret does, and hence
git-credential-libsecret too)
Encrypted sessions via dbus (not "plain")
Non-pageable memory for holding secrets

Also missing:

Go unit tests
Expanded readme with setup instructions

Note: This is an updated version of PR #2, which was automatically
closed when the base branch was deleted.

still lgtm


Allowing whitelisted multi-parameters

Adding an option (defaults to True) to include multi parameters in the whitelist. e.g.
# whitelists "some_param=", "some_param[]=", "some_param[1]="
@torender.prerenderable(params=["some_param"])

# whitelists "some_param="
@torender.prerenderable(params=["some_param"], allow_multi_param=False)

IIRC, tornado encodes query parameters with more than one value with [] at the end of the key, correct? If so, I'd say let's just always check for it, rather than having it as an option.


Authentication

I have setup osm-auth but when a user grants access, when id reloads, the user is not authenticated in iD. The info bar (info-list) at the bottom is not showing the user name (object "userLink").
I am using this to authenticate:
 var auth = osmAuth({
    singlepage: true,
        oauth_consumer_key: '5A043yRSEugj4DJ5TljuapfnrflWDte8jTOcWLlT',
        oauth_secret: 'aB3jKq1TRsCOUrfOIZ6oQMEDmv2ptV76PA54NGLL',
        landing: "land_single.html"
    });
document.getElementById('authenticate').onclick = function() {
        auth.authenticate(function() {
        });
    };

I must be missing something.

Are you calling preauth, like openstreetmap-website does ?  Your land_single.html page will need to collect all those tokens and pass them to iD.  You don't strictly need to have a land_single.html page - instead you can just check for the cookies to be set with JavaScript and only start iD if they are present.   Let's move this discussion to https://github.com/osmlab/osm-auth/issues/21 and I will try to create a better example in that repository of how to authenticate with a single page instead of a popup.
Also, another issue I see - this is not preventing your authentication from working, but a nice thing to do would be to register your application on openstreetmap-website, rather than reusing the iD oauth tokens.  To do this, go to openstreetmap.org, click on "My Settings" and then click "oauth settings", and scroll to the very bottom where it says "My Client Applications", and register your application.

I was just looking at that, funnily enough. No, I am not calling preauth. Thank you for the example!
I completely forgot to register the application, thanks for this!

When clicking on an authentication link and using landing: 'index.html' the app loads with "?oauth_token=" but iD doesnt load with the user's authentication even if oauth_token is in the url.

The reason why I posted here is because the osm-auth repo doesnt appear to be active and questions asked in the issues are not likely to be discussed.

The reason why I posted here is because the osm-auth repo doesnt appear to be active and questions asked in the issues are not likely to be discussed.

Its ok, I only just started watching osm-auth 2 weeks ago, but I'm going to assume a more active role in maintaining it..


⚠ File access has degraded performance

In 0b156a1, File access (https://fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 3732 ms

Resolved: File access performance has improved in ced5c3a after 10 minutes.


Minor issue on .dragToDismiss for middle popups on v2.5

Hello again! Another minor issue I've found is when you try to drag the middle popup up. Not down to dismiss it, but up.
It will "jump" up and then return to its middle position.
You should have .dragToDismiss = true. This behavior is only seen on >v2.5. Previously, when you dragged the popup up nothing happened (as I believe is the correct behavior).
Thanks again for that wonderful library!

Hey @BredBurr, thanks for letting me know! Please give version 2.5.7 a go, have a marvellous day!


operators community-windows-machine-config-operator (5.1.0)

Signed-off-by: Alina Ryan<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[ ] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/lgtm

/merge possible

/approve

/lgtm

/merge possible

I guess @sebsoto 's approval didn't actually set the status to Approved.  Maybe it is time to join the org to get this label added automatically?

if we can have write access, then we can merge it as well. not sure who can add us!

@alinaryan @selansen @mtnbikenc You can add yourself to reviewer list in ci.yaml and next PR can be automerged. More info : https://redhat-openshift-ecosystem.github.io/community-operators-prod/operator-ci-yaml/#reviewers

/merge possible


Improve site contents

Tutorials are hard to find. That page should be more visible. Something as on Crossbar.io Web site would be far better. A "Documentation" link would help.
Moreover, the TOC should contain a little text, as done here.

fixed. pls see http://autobahn.ws/python/index.html#documentation-overview


IRegistrator is missing generic version of RegisterPlaceholder

Hi, it seems that the registration role does not have the generic variant of RegisterPlaceholder, whereas IContainer has both. I expected parity between IContainer and IRegistrator with regard to registration overloads.

IRegistrator.RegisterPlaceholder<TServiceType>() <-- missing
IRegistrator.RegisterPlaceholder(Type serviceType)
IContainer.RegisterPlaceholder<TServiceType>()  <-- exists
IContainer.RegisterPlaceholder(Type serviceType)

@penenkel Nice finding, will add.


chore(AlteredSliceTag): Migrate to functional

SUMMARY

Refactor component to functional and update tests
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up


Sandbox Instructions do not work

Type of issue
Code doesn't work
Feedback
Running the last line of Sandbox install code produces this error:
Add-AppxPackage Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle
Add-AppxPackage : Deployment failed with HRESULT: 0x80073CF3, Package failed updates, dependency or conflict
validation.
Windows cannot install package Microsoft.DesktopAppInstaller_1.22.10582.0_x64__8wekyb3d8bbwe because this package
depends on a framework that could not be found. Provide the framework "Microsoft.UI.Xaml.2.8" published by
"CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US", with neutral or x64 processor
architecture and minimum version 8.2310.30001.0, along with this package to install.
NOTE: For additional information, look for [ActivityId] 925968c0-6a72-0000-19fe-5992726ada01 in the Event Log or use
the command line Get-AppPackageLog -ActivityID 925968c0-6a72-0000-19fe-5992726ada01
At line:1 char:1

Add-AppxPackage Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundl ...

  + CategoryInfo          : WriteError: (C:\Users\WDAGUt...bbwe.msixbundle:String) [Add-AppxPackage], IOException
  + FullyQualifiedErrorId : DeploymentError,Microsoft.Windows.Appx.PackageManager.Commands.AddAppxPackageCommand

I cannot figure how to install the Xaml.2.8 dependency.
Page URL
https://learn.microsoft.com/en-us/windows/package-manager/winget/
Content source URL
https://github.com/MicrosoftDocs/windows-dev-docs/blob/docs/hub/package-manager/winget/index.md
Author
@mattwojo
Document Id
b49c32ea-2efa-b090-7123-87c31d10279b

Hey @dleeprx -  Thanks for filing this issue. I just heard from the WinGet PM that there is a new dependency due to a new version of the UI.Xaml framework release for WinGet 1.7. The PR to fix the PowerShell module is in progress:
https://github.com/microsoft/winget-cli/pull/4218
Once the fix is merged and we are able to test and ensure that the WinGet installation works properly in Windows Sandbox, we will update the doc to reflect the new install script.
Since this is a duplicate issue of https://github.com/MicrosoftDocs/windows-dev-docs/issues/4882, I will close this one, but will post updates in #4882.


add more projects that use pyright

The couple ones we already have have been disproportionately useful for implementing new typing features in mypy
https://github.com/awslabs/gluon-ts
~https://github.com/canonical/operator~
~https://github.com/astropenguin/xarray-dataclasses~
https://github.com/Rapptz/RoboDanny
https://github.com/DisnakeDev/disnake
https://github.com/GraiaProject/Avilla

Given mypy_primer now is used as a pyright primer (ish) too, I suspect this issue could probably be closed? Or maybe pushed off to, like, pyright's maintainer at least! They know best!


Bug view display value

Python version
3.10
Django version
5.0.3
Package version
0.12.0
Current behavior (bug description)
The list with all extra variables shows all possible VALUES and not the values for the particular type.

Expected behavior

Yes (collectstatic), I have also tried with anonymous window and nothing.

@fabiocaccamo
Effectively it is not in static neither the css nor the js, but in the collecstaric it does not pick them up.

@fermenen could you check if there are errors in the browser console please?

Hello @fabiocaccamo, I had a bug with collecstatics, is resolved!


Increment version for servicebus releases

Increment package version after release of Azure.Messaging.ServiceBus

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

Failures are unrelated and caused by issues with sparse checkout of the repository.   The majority of steps passed and there's enough coverage of platforms.   Merging.


Add AppVeyor, Travis build status

Fixed in https://github.com/aspnet/HtmlAbstractions/commit/736f2bcefbd1c1d9bb437bf19d4b0d72593cfb68


Fix for specific manga

fix #92
Changes:

add check for self.user

Thank you, the PR has been merged 👌


chore(deps): update terraform github.com/terraform-ibm-modules/terraform-ibm-resource-group to v1.0.3

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-resource-group
module
patch
v1.0.2 -> v1.0.3

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox. ⚠ Warning: custom changes will be lost.

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Data Dictionary

I have a question regarding the data dictionary... What actually means “doses allocated”?

@dprata00 "doses allocated" means the number of vaccine doses that the region/State has been assigned. Is that more clear?

@dprata00 "doses allocated" means the number of vaccine doses that the region/State has been assigned. Is that more clear?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_eb0b8zk3ty/clone/recipe/meta.yaml:13: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_eb0b8zk3ty/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_eb0b8zk3ty/clone/recipe/meta.yaml:32: python_build_tool_in_run: The python build tool setuptools is in run depends
ERROR: /tmp/abs_eb0b8zk3ty/clone/recipe/meta.yaml:46: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_eb0b8zk3ty/clone/recipe/meta.yaml:46: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_eb0b8zk3ty/clone/recipe/meta.yaml:46: missing_description: The recipe is missing a description
ERROR: /tmp/abs_eb0b8zk3ty/clone/recipe/meta.yaml:46: missing_dev_url: The recipe is missing a dev_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b2yytlmaey/clone/recipe/meta.yaml:13: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_b2yytlmaey/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_b2yytlmaey/clone/recipe/meta.yaml:32: python_build_tool_in_run: The python build tool setuptools is in run depends
ERROR: /tmp/abs_b2yytlmaey/clone/recipe/meta.yaml:46: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_b2yytlmaey/clone/recipe/meta.yaml:46: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_b2yytlmaey/clone/recipe/meta.yaml:46: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_b2yytlmaey/clone/recipe/meta.yaml:46: missing_description: The recipe is missing a description
Errors were found


gollvm: add linkname for runtime variables

What version of Go are you using (go version)?

$ go version
go version unknown linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

$ cat m.go
package main

import _ "unsafe"

//go:linkname memstats runtime.memstats
var memstats struct {
  Alloc int64
  TotalAlloc uint64
  Sys uint64
  Lookups uint64
  Mallocs uint64
  Frees uint64
  HeapAlloc uint64
  HeapIdle uint64
  HeapInuse uint64
  HeapReleased uint64
  HeapObjects uint64
  StackInuse uint64
  StackSys uint64
  MSpanInuse uint64
  MSpanSys uint64
  MCacheInuse uint64
  MCacheSys uint64
  BuckHashSys uint64
  GCSys uint64
  OtherSys uint64
  NextGC uint64
  LastGC uint64
  PauseTotalNs uint64
  PauseNs [256]uint64
  PauseEnd [256]uint64
  NumGC uint32
  NumForcedGC uint32
  GCCPUFraction float64
  EnableGC bool
  DebugGC bool
  BySize [61]struct {
          Size uint32
          Mallocs uint64
          Frees uint64
  }
}

func main() {
  println(memstats.Alloc)
}

$ go run m.go
# command-line-arguments
./m.go:5:3: error: memstats is not a function; '//go:linkname' is only supported for functions

What did you expect to see?
No error
What did you see instead?
Error
Discussion
I suggested a patch here  and want to have an issue I work with.
The situation is the following: I'm porting a Go project from GC to GoLLVM. This project uses some third-party packages with linkname to runtime internal object. I have no possibility to avoid it so I want to make GoLLVM use linkname for variables from compiler internal packages. The suggested patch is a prototype that enables linkname for variables from runtime package.

I think that what we need here is a way to say whether a particular variable is intended to be a reference to a variable defined in another package, or whether it is intended to be a definition of the variable.  I don't see a way to do that with a single go:linkname directive.  I think we need to define two new directives and support them in the gc compiler and the gofrontend code.  The way do that is to write a proposal (https://go.dev/s/proposal) describing the new proposed semantics.

I understand that you have a problem with existing code.  However, I want to set things up so that future code can be correct.  I don't want to insert a hack that supports existing code in a way that doesn't prepare us for the future.  I'm not going to add your suggestion to the current implementation.  Sorry.

After some discussions with product teams we came to conclusion that they will not change their code. So now the proposal for new directive is not needed because nobody will use it.
Thanks for comments and discussion. I'll close this issue till better days.


Pass killGracePeriodSeconds value to Mesos for pods

JIRA Issues: MARATHON-7952

nice work in short time frame

@timcharper while not the norm... I would support a merge.    @jeschkies rejection was addressed and I wouldn't want to wait all weekend to get another review.     There isn't a "rush" except for the customer that needs this... needs a back port.


Add ability to fill fields from left-to-right instead

According to the first comment of #7, fields are filled right-to-left.
This causes issues when instantiating a struct that must read data starting from the left.
For example, reading a Command Word from MIL-STD-1553 follows:

Address
Transmit
Sub-address
Words

Bit range
0 - 4
5
6 - 10
11 - 15

Creating a representative struct with the help of bilge:
#[bitsize(16)]
#[derive(FromBits, DebugBits, PartialEq)]
struct Command {
    address: u5,
    transmit: bool,
    subaddress: u5,
    words: u5,
}

We can initialize the fields in proper order manually:
let cmd1: Command = Command::new(u5::new(5), true, u5::new(7), u5::new(3));
assert_eq!(u5::new(5), cmd1.address());
assert_eq!(true, cmd1.transmit());
assert_eq!(u5::new(7), cmd1.subaddress());
assert_eq!(u5::new(3), cmd1.words());

However, when reading the data from a stream in native ordering from the bus, the equivalent to the above would be 0b00101_1_00111_00011. Attempting to use this directly does not yield the desired result:
let cmd2: Command = Command::from(u16::new(0b00101_1_00111_00011));
assert_eq!(cmd1, cmd2);

thread 'test_command' panicked at 'assertion failed: `(left == right)`
  left: `Command { address: 5, transmit: true, subaddress: 7, words: 3 }`,
 right: `Command { address: 3, transmit: true, subaddress: 19, words: 5 }`'

Obviously the right-to-left parsing causes the mix-up. However fixing it client-side isn't quite straightforward. It's not a matter of endianness:

Address
Transmit
Sub-address
Words

Hex

Expected
00101
1
00111
00011

2C E3

Received
00011
1
10011
00101

1E 65

Swapping bytes (2CE3 -> E32C) would yield a drastically different result:

let cmd3: Command = Command::from(u16::from_be(0b00101_1_00111_00011));
assert_eq!(cmd1, cmd3);

thread 'test_command' panicked at 'assertion failed: `(left == right)`
  left: `Command { address: 5, transmit: true, subaddress: 7, words: 3 }`,
 right: `Command { address: 12, transmit: true, subaddress: 12, words: 28 }`'

We would instead need to pre-format data so that bilge can handle it:
let original: u16 = u16::new(0b00101_1_00111_00011);
let fixed: u16 =
      (original & 0b11111) << 11
    | ((original & 0b11111_00000) >> 5) << 6
    | ((original & 0b1_00000_00000) >> 10) << 5
    | ((original & 0b11111_0_00000_00000) >> 11);
let cmd4: Command = Command::from(fixed);
assert_eq!(cmd1, cmd4);

This does not scale well as more structs are defined. Additionally, casting the struct to an integer is done right-to-left as well:
assert_eq!(original, cmd1);

thread 'test_command' panicked at 'assertion failed: `(left == right)`
  left: `11491`,
 right: `6629`',

Address
Transmit
Sub-address
Words

Hex

Expected
00101
1
00111
00011

2C E3

Received
00011
0
01111
00101

19 E5

Much of these issues could be resolved if there was an option to instead fill fields left-to-right. Perhaps something like #[derive(FromBitsLeftmost)]

Thank you for the detailed writeup. Someone else wanted to switch the order too, since specs usually start with the top bits.
I just checked, and reversing the field definition works.
Can you try if that works and gives you valid values?

A while back, I did something similar yet far more invasive. The idea was that any field could be regarded as either big or little endian. I have a protocol where both the order of the fields can change as well as the endianness of integers...
I started implementing this as a fork of modular-bitfield https://github.com/vhdirk/modular-bitfield, though I can't remember what state I left it in. Perhaps this could serve as inspiration?


[YoutubeDL] extract_info: Default 'download' to False

That's what I would expect from the method name

This may break behavior of some third party code that uses extract_info.

I know, but I think that default makes more sense and it should be easy to detect the break.


Wanted to add footer section in the website

Please assign me to do this for more user friendly website

Please Elaborate more about the idea about footer.

This type of feature I wanted to say !

Pls assign this task to me adding footer section in the website with lables

@akshattiwarii Assigning you. Please Add only relevant links in the footer.  Not links mentioned in the above image.

Will give level based on the work.

ohk thanks


fallback to client account

I am providing a client to createSmartAccountClient that is already "bound" to an account, so it would be nice to fallback to this value.

Hey Holic, thanks for your help!
The client passed to createSmartAccountClient should be a publicClient, which means it should have public actions available. The publicClient isn’t expected to have an account, and if it does, it’s not supposed to be a smart account. That's why we can't pass it forward.


Add initial governance structure

Description
This proposes a formal start to defining the governance structure for the Tinkerbell community.
Rendered docs:

https://github.com/tinkerbell/proposals/blob/5917ee2c1bb772c6fa8a6697d33383b606e98e4a/proposals/0024/GOVERNANCE.md
https://github.com/jacobweinstock/proposals/blob/95812ba687e90924d978655a42bab25783d86f2a/proposals/0024/README.md

Why is this needed
https://github.com/tinkerbell/tink/discussions/504
Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

View rendered proposals/0024/GOVERNANCE.md
View rendered proposals/0024/README.md

Thank you all for the approvals. I don't have permission to merge this PR. Would someone with those rights be able to merge this??