471 clean up stellar secret keys

Closes #471
Makes use of environmental variables for the stellar secret keys instead of reading them from the resources files, when running tests. We don't use clap for passing the keys given that is not straightforward to use with cargo test.
We maintain the usage of the same cli commands where we specify the stellar_vault_secret_key_filepath, but use the environment keys for testing.
The use of DEFAULT_MAINNET_DEST_SECRET_KEY and DEFAULT_TESTNET_DEST_SECRET_KEY is now replaced by destination keys read from environment. In order to keep the use of only 4 consistent source and destination keys, the keys previously supplied from by these constants are changed.
NOTE: the tests should be failing until we add the github secrets corresponding to the stellar keys.

@ebma Thanks for the comments. After I change the code for the suggestions I think I can "migrate" the accounts. Just to verify, all 4 accounts need to have the same trustliness and the only difference is that source accounts will have the asset in question right? Also, all 4 need some amount of lumen.


Parser.to_activerecord_query gives a TypeError on parsing filters

When trying to query for, say, 'emails eq ""', the parser gives a TypeError
My code:

My test case and results:

Source code where shit hits the fan:

I ran into the same error message, and on my side it turned out to be an incorrectly configured queryable_attributes on the CompanyUser model.
Could you share your CompanyUser configuration? Especially the queryable_attributes part.
My changes:
### user.rb -- before

def User
  ..

  self.scim_queryable_attributes
    { displayName: :display_name }
  end

to
### user.rb -- after

def User
  ..

  self.scim_queryable_attributes
    { displayName: { column: :display_name } }
  end

EDIT: Nevermind, turns out .email is a function in my model

Yep :)


Multimedia Learning Approach

I'm new to data visualization, though I've done it in Excel for years.  The combination of written narrative, video, and a live test environment is excellent for exposing, learning, and trying the concepts.  Great job.

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

ID: 69c7a5b0-75bd-3ba7-1fac-7557b790d511
Version Independent ID: b89aefeb-6cc8-4874-e514-7a1428611c8c
Content: Getting data - Power BI
Content Source: powerbi-docs/guided-learning/gettingdata.yml
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi @jrsauter - that's great to hear! Thanks for your nice comments, and we hope to create more content like this in the future. Thanks for the feedback!


typo "狗仔"

Type of issue
Typo
Description
第一句里的“狗仔”修改为“构造”
Page URL
https://learn.microsoft.com/zh-cn/dotnet/csharp/tour-of-csharp/tips-for-python-developers
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/tour-of-csharp/tips-for-python-developers.md
Document Version Independent Id
607b4dc4-11d1-c653-09e1-7d39cdf306f9
Article author
@BillWagner
Metadata

ID: e81edea7-f42c-f344-f774-c380f94c4695
Service: dotnet-csharp
Sub-service: get-started

I’m considering helping, but first I’d like to understand the translation process and where the error is coming from.
Could you share more details?


Solar panel crash on server

Not sure what happens but this crash we have been getting a lot lately and are not sure what is causing it https://gist.github.com/Bennyboy1695/e35cd6a856ba83952449

I'd say some other mod is patching around wildly and is damaging Ender IO while doing so. As you might have noticed, one part of Ender IO is trying to call another part of Ender IO...which is not there. If that was a coding error on our side, Ender IO would have failed to compile in the first place.
Nothing we can do here. Sorry.

Dude update your cauldron to version 1492. 152 and don't go for the latest. just use version 152


Upgrading from 1.0.4 to 1.2.2

Hi
Maybe more a question than an issue. After I upgrade the binaries, I run
./siad --sia-directory /var/lib/sia/
Loading...
(0/6) Loading siad...
(1/6) Loading gateway...
(2/6) Loading consensus...
(3/6) Loading transaction pool...
(4/6) Loading wallet...
(5/6) Loading host...
And it hangs there. Almost 1 hr so far.
Then..
./siac
Could not get current consensus state: API call not recognized: /consensus
Do I need to wait or is it something wrong?

wow so old :D
Yes host rebuilding uploaded files and preparing free space for upload.
So need wait long time.
Time depends on your host storage size.
So keep calm :)

I share 7TB, 5% rented

I dont remember approx. building time/storage ratio. :)
But at least few hours, maybe whole day.

Done, 27k seconds


feat: add INE for ALTER TABLE ADD COLUMN

Add support for IF NOT EXISTS on PostgreSQL (1), BigQuery (2), and Generic.
Add COLUMN displaying properly, to match the original query.

[1] : https://www.postgresql.org/docs/current/sql-altertable.html
[2] : https://cloud.google.com/bigquery/docs/reference/standard-sql/data-definition-language#alter_table_rename_column_statement

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 39 (84.62%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.009%) to 86.268%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser.rs
4
5
80.0%

src/ast/ddl.rs
7
12
58.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
11836

Relevant Lines:
13720

💛  - Coveralls

@alamb don't want to hurry or anything, but are you waiting for anything to review this?
I have quite a few PR's in row hehe

@alamb don't want to hurry or anything, but are you waiting for anything to review this?

No I am not waiting on anything. I will try and find time to work through the sqlparser-rs review backlog. Thanks for bringing it to my attention


Publish the Swagger UI (to GitHub pages)

It would be convenient to have the Swagger UI available without building the project locally, so auto-publishing the latest Swagger UI e.g. to GitHub pages would be great. Maybe these projects could help:

https://github.com/peter-evans/swagger-github-pages
https://github.com/Legion2/swagger-ui-action

I second this, would be helpful to see and inspect the API even without building the project.

Now as our initial Server UI is in the codebase, we would really need this for planning the way forward; access to the API spec via GitHub would be very beneficial, as not everyone in our team wants to always run the local setup just to be able to inspect the API.

@mnonnenmacher is there a way to prioritize this? This would be really convenient for front-end development still.

Mostly, yes, but as you know there are still issues with the Compose setup, and for quickly looking up something it'd be more convenient to have a public reference. But I just realized I could also just use Swagger from our staging deployment.

I have done some testing with https://github.com/PaloAltoNetworks/docusaurus-openapi-docs and it looks promising:

It currently fails for some endpoints due to invalid specs, but once #946 is merged I will look into adding this.

The API docs are published at https://eclipse-apoapsis.github.io/ort-server/api/ort-server-api.


Look up if node enabled path finding (Version: 1.9.3)

Issue Description

For a node it is possible to set path_search_max to zero in its rippled.cfg to disable path finding.
Ripple Testnet (https://s.altnet.rippletest.net:51234/) seams to do so. I also had issues with S1 (https://s1.ripple.com:51234/) in the past.
When set path_search_max to zero and someone is calling the path_find method, one only gets an empty array for alternatives. This could be a problem if you are trying to find a path between a trading pair that is illiquid anyway. One could think that there simply are no paths available, even if there are.
Path finding is one of the key features of the XRPL, so we should have an easy way to look up if a node has it enabled.
Option 1
Return the field path_search_max in the server_info response, to have the opportunity to look it up.
Option 2
Have a warning in the path_find method's response, which explains that the node does not support path finding.
This affects all path finding methods (path_find, ripple_path_find).
Steps to Reproduce

Option 1

https://xrpl.org/websocket-api-tool.html#server_info
Choose Testnet (s.altnet.rippletest.net (Testnet Public Cluster))
"Send request"

Option 2

https://xrpl.org/websocket-api-tool.html#path_find
Choose Testnet (s.altnet.rippletest.net (Testnet Public Cluster))
"Send request"

Expected Result

Option 1 (field for server_info)
A response with a path_search_max field.
{
  "id": 1,
  "result": {
    "info": {
      "build_version": "1.9.3",
      "complete_ledgers": "30410348-30983432",
      "hostid": "HICK",
      "initial_sync_duration_us": "255201252",
      "io_latency_ms": 1,
      "jq_trans_overflow": "0",
      "last_close": {
        "converge_time_s": 2,
        "proposers": 6
      },
      "load_factor": 1,
      "network_id": 1,
      "path_search_max": 0,  <<-------------------
      "peer_disconnects": "3879",
      "peer_disconnects_resources": "137",
      "peers": 67,
      "pubkey_node": "n9LGH8mv1XumE6eRV8sY7tRLdJSqbJ874G15mztbQB4BTCiaJXjP",
      "server_state": "full",
      "server_state_duration_us": "514336593812",
      "state_accounting": {
        "connected": {
          "duration_us": "252153624",
          "transitions": "2"
        },
        "disconnected": {
          "duration_us": "1045046",
          "transitions": "2"
        },
        "full": {
          "duration_us": "514336593812",
          "transitions": "1"
        },
        "syncing": {
          "duration_us": "2002505",
          "transitions": "1"
        },
        "tracking": {
          "duration_us": "74",
          "transitions": "1"
        }
      },
      "time": "2022-Sep-07 19:50:48.492886 UTC",
      "uptime": 514591,
      "validated_ledger": {
        "age": 3,
        "base_fee_xrp": 0.00001,
        "hash": "3198869EF19293B9ACB22DDA66658E22EA6822AFC396829CE9CADED60E5CA82A",
        "reserve_base_xrp": 10,
        "reserve_inc_xrp": 2,
        "seq": 30983432
      },
      "validation_quorum": 5
    }
  },
  "status": "success",
  "type": "response"
}

Option 2 (warning for path finding methods)
{
  "id": 8,
  "result": {
    "alternatives": [],
    "destination_account": "rf1BiGeXwwQoi8Z2ueFYTEXSwuJYfV2Jpn",
    "destination_amount": {
      "currency": "USD",
      "issuer": "rvYAfWj5gh67oV6fW32ZzP3Aw4Eubs59B",
      "value": "0.001"
    },
    "full_reply": false,
    "id": 8,
    "source_account": "rf1BiGeXwwQoi8Z2ueFYTEXSwuJYfV2Jpn"
  },
  "status": "success",
  "type": "response",
  "warnings": [ <<-------------------
      {
        "id": ...,
        "message": "This node does not support path finding."
      }
    ]
}

Actual Result

Option 1
{
  "id": 1,
  "result": {
    "info": {
      "build_version": "1.9.3",
      "complete_ledgers": "30410348-30983432",
      "hostid": "HICK",
      "initial_sync_duration_us": "255201252",
      "io_latency_ms": 1,
      "jq_trans_overflow": "0",
      "last_close": {
        "converge_time_s": 2,
        "proposers": 6
      },
      "load_factor": 1,
      "network_id": 1,
      "peer_disconnects": "3879",
      "peer_disconnects_resources": "137",
      "peers": 67,
      "pubkey_node": "n9LGH8mv1XumE6eRV8sY7tRLdJSqbJ874G15mztbQB4BTCiaJXjP",
      "server_state": "full",
      "server_state_duration_us": "514336593812",
      "state_accounting": {
        "connected": {
          "duration_us": "252153624",
          "transitions": "2"
        },
        "disconnected": {
          "duration_us": "1045046",
          "transitions": "2"
        },
        "full": {
          "duration_us": "514336593812",
          "transitions": "1"
        },
        "syncing": {
          "duration_us": "2002505",
          "transitions": "1"
        },
        "tracking": {
          "duration_us": "74",
          "transitions": "1"
        }
      },
      "time": "2022-Sep-07 19:50:48.492886 UTC",
      "uptime": 514591,
      "validated_ledger": {
        "age": 3,
        "base_fee_xrp": 0.00001,
        "hash": "3198869EF19293B9ACB22DDA66658E22EA6822AFC396829CE9CADED60E5CA82A",
        "reserve_base_xrp": 10,
        "reserve_inc_xrp": 2,
        "seq": 30983432
      },
      "validation_quorum": 5
    }
  },
  "status": "success",
  "type": "response"
}

Option 2
{
  "id": 8,
  "result": {
    "alternatives": [],
    "destination_account": "rf1BiGeXwwQoi8Z2ueFYTEXSwuJYfV2Jpn",
    "destination_amount": {
      "currency": "USD",
      "issuer": "rvYAfWj5gh67oV6fW32ZzP3Aw4Eubs59B",
      "value": "0.001"
    },
    "full_reply": false,
    "id": 8,
    "source_account": "rf1BiGeXwwQoi8Z2ueFYTEXSwuJYfV2Jpn"
  },
  "status": "success",
  "type": "response"
}

So turns out the path_search_max field gets checked for both path_find and ripple_path_find and returns an error.
So how come that you always get an empty array on Testnet instead of an error?
Error when path finding is disabled:
{
  "error": "notSupported",
  "error_code": 75,
  "error_message": "Operation not supported.",
  "id": 8,
  "request": {
    "command": "ripple_path_find",
    "destination_account": "rf1BiGeXwwQoi8Z2ueFYTEXSwuJYfV2Jpn",
    "destination_amount": {
      "currency": "USD",
      "issuer": "rvYAfWj5gh67oV6fW32ZzP3Aw4Eubs59B",
      "value": "0.001"
    },
    "id": 8,
    "source_account": "rf1BiGeXwwQoi8Z2ueFYTEXSwuJYfV2Jpn",
    "source_currencies": [
      {
        "currency": "XRP"
      },
      {
        "currency": "USD"
      }
    ]
  },
  "status": "error",
  "type": "response",
  "warnings": [
    {
      "id": 1004,
      "message": "This is a reporting server.  The default behavior of a reporting server is to only return validated data. If you are looking for not yet validated data, include \"ledger_index : current\" in your request, which will cause this server to forward the request to a p2p node. If the forward is successful the response will include \"forwarded\" : \"true\""
    }
  ]
}


Problem with skill's response

Hi Etienne,
This is exactly what I was looking for.
Thank you so much for putting this together.
Although I'm not a Pyhton expert, I have two working custom Alexa skills. So I have done this before. I followed all your instructions. The Lambda function is set up and linked to the Alexa skill both default region and north america. The skill builder checklist is all green. But when I try the command, I get the following response: "there was a problem with the requested skill's response".
In the test, I get the following from the skill i/o log input:
{
	"version": "1.0",
	"session": {
		"new": true,
		"sessionId": "amzn1.echo-api.session.c4de319e-4ab6-4ef1-8c4a-5a5be897bda9",
		"application": {
			"applicationId": "amzn1.ask.skill.54d3c756-8dc2-40d9-9ab4-fa6cd68a5062"
		},
		"user": {
			"userId": "amzn1.ask.account.AH5K766CW7PDCEBDQX6OZT62G5PEG2MIMTCZQMZMNRY5A5UC3IHQJNRKGDVGC2ITOEAIAAXUTJVB4BPGO4LW5NXXSQZNTRAXQVGTHIWRTEJI3EH62DHYYU4CBE3EBVD5XFTQXQZA2KPMU5ZYIFQTA5QHTRUOQNG55G5ETLTIZQKJMBE4LOW5VITOX7FX6ZPZIJCL4VWIKRH5AXY"
		}
	},
	"context": {
		"AudioPlayer": {
			"playerActivity": "IDLE"
		},
		"Display": {},
		"System": {
			"application": {
				"applicationId": "amzn1.ask.skill.54d3c756-8dc2-40d9-9ab4-fa6cd68a5062"
			},
			"user": {
				"userId": "amzn1.ask.account.AH5K766CW7PDCEBDQX6OZT62G5PEG2MIMTCZQMZMNRY5A5UC3IHQJNRKGDVGC2ITOEAIAAXUTJVB4BPGO4LW5NXXSQZNTRAXQVGTHIWRTEJI3EH62DHYYU4CBE3EBVD5XFTQXQZA2KPMU5ZYIFQTA5QHTRUOQNG55G5ETLTIZQKJMBE4LOW5VITOX7FX6ZPZIJCL4VWIKRH5AXY"
			},
			"device": {
				"deviceId": "amzn1.ask.device.AG7CMCJYRRZ2WXOGAMTINVRQBBONTQZ5S5VI6UEAI3RPL3JZLD4GPPDFCKYT4LV3XIMQS4FHD32NGO5GJGXMV52656JUZPBDCD6E64MCUJ4HDTBY277NTFEFKAIJBSLAQSJPOLITW3DWMIM2SLKC3CF6HQDHZQMAQVWF5ODTNUQGJPUCELRAK",
				"supportedInterfaces": {
					"AudioPlayer": {},
					"Display": {
						"templateVersion": "1.0",
						"markupVersion": "1.0"
					}
				}
			},
			"apiEndpoint": "https://api.amazonalexa.com",
			"apiAccessToken": "eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImtpZCI6IjEifQ.eyJhdWQiOiJodHRwczovL2FwaS5hbWF6b25hbGV4YS5jb20iLCJpc3MiOiJBbGV4YVNraWxsS2l0Iiwic3ViIjoiYW16bjEuYXNrLnNraWxsLjU0ZDNjNzU2LThkYzItNDBkOS05YWI0LWZhNmNkNjhhNTA2MiIsImV4cCI6MTUyMzI5NjQzNywiaWF0IjoxNTIzMjkyODM3LCJuYmYiOjE1MjMyOTI4MzcsInByaXZhdGVDbGFpbXMiOnsiY29uc2VudFRva2VuIjpudWxsLCJkZXZpY2VJZCI6ImFtem4xLmFzay5kZXZpY2UuQUc3Q01DSllSUloyV1hPR0FNVElOVlJRQkJPTlRRWjVTNVZJNlVFQUkzUlBMM0paTEQ0R1BQREZDS1lUNExWM1hJTVFTNEZIRDMyTkdPNUdKR1hNVjUyNjU2SlVaUEJEQ0Q2RTY0TUNVSjRIRFRCWTI3N05URkVGS0FJSkJTTEFRU0pQT0xJVFczRFdNSU0yU0xLQzNDRjZIUURIWlFNQVFWV0Y1T0RUTlVRR0pQVUNFTFJBSyIsInVzZXJJZCI6ImFtem4xLmFzay5hY2NvdW50LkFINUs3NjZDVzdQRENFQkRRWDZPWlQ2Mkc1UEVHMk1JTVRDWlFNWk1OUlk1QTVVQzNJSFFKTlJLR0RWR0MySVRPRUFJQUFYVVRKVkI0QlBHTzRMVzVOWFhTUVpOVFJBWFFWR1RISVdSVEVKSTNFSDYyREhZWVU0Q0JFM0VCVkQ1WEZUUVhRWkEyS1BNVTVaWUlGUVRBNVFIVFJVT1FORzU1RzVFVExUSVpRS0pNQkU0TE9XNVZJVE9YN0ZYNlpQWklKQ0w0VldJS1JINUFYWSJ9fQ.G-OU8rReighh5QbWDIypaDgaLs9Pox4bH7U5ec5IRjhd4hUyZZ8VJsEknRHK1DzXj1YgcKCUFvO0zM4opF_I1M-MGWVxQWSygjG-2CcDkwPa-I6lkapICL1Fnoh7ZPCzYc0h5IVngZ-ZmQ8Vuqja7UAfUgLL3WCYdL5xRPprfvcPCKuAeBu3ruDapR_PfgZynDXr8Y50ksmPwGrsSMPa7oNs4MHv5AFU_AOnu0P2FH-zh-vB_rNEEIXeaoaR_Pxzo5wwaGDsKwx3TWaTCMuKNODd4UaOPmzdub2xebYLbA0LFDE88wai3ZdH0Ri2Amh5HoRHgfeoKtzAwq5yRlbSmw"
		}
	},
	"request": {
		"type": "IntentRequest",
		"requestId": "amzn1.echo-api.request.1bcfd478-8d22-494c-9f19-1897132cf5ab",
		"timestamp": "2018-04-09T16:53:57Z",
		"locale": "en-US",
		"intent": {
			"name": "CleanIntent",
			"confirmationStatus": "NONE"
		}
	}
}

The output is just NULL.
I checked the interaction model and it all seems ok.
Can you help?
Jordan.

Hi Jordan.
I would be happy to help.
Have you tried taking that request and feeding it to your lambda function directly through the AWS website? Just copy and paste your JSON request in a test event and see what error you are getting.
Let me know how that goes.
Etienne

Hi Etienne,
I copy and pasted this JSON as is in a new test on the AWS console.
I ran the test and got a this error message:
"errorMessage": "2018-04-09T18:31:22.172Z 3181f5da-3c24-11e8-b381-77d62d90a578 Task timed out after 3.00 seconds"
Here are the details:
START RequestId: 3181f5da-3c24-11e8-b381-77d62d90a578 Version: $LATEST event.session.application.applicationId=amzn1.ask.skill.54d3c756-8dc2-40d9-9ab4-fa6cd68a5062 on_session_started requestId=amzn1.echo-api.request.1bcfd478-8d22-494c-9f19-1897132cf5ab, sessionId=amzn1.echo-api.session.c4de319e-4ab6-4ef1-8c4a-5a5be897bda9 on_intent requestId=amzn1.echo-api.request.1bcfd478-8d22-494c-9f19-1897132cf5ab, sessionId=amzn1.echo-api.session.c4de319e-4ab6-4ef1-8c4a-5a5be897bda9 END RequestId: 3181f5da-3c24-11e8-b381-77d62d90a578 REPORT RequestId: 3181f5da-3c24-11e8-b381-77d62d90a578	Duration: 3003.27 ms	Billed Duration: 3000 ms 	Memory Size: 128 MB	Max Memory Used: 43 MB	 2018-04-09T18:31:22.172Z 3181f5da-3c24-11e8-b381-77d62d90a578 Task timed out after 3.00 seconds

Under Basic Settings, try setting timeout to 15 sec. The skill takes a little while to connect to the xmpp server before it can actually send a command to your vacuum, generally around 7 or 8 seconds.
Thank you for your feedback, I'll be updating the README.

Hi Etienne,
That did the trick.
It's working now! Thank you so much for your help.
I have two enhancement suggestions:

Do you know if there is a command for spot cleaning, e.g. "ask my vacuum to spot clean"
Can you add an utterance for timed cleaning (I believe it's already in the 'sucks' code, see line 164 of cli.py).  "ask my vacuum to clean for 20 minutes"

Thanks again.
J.

Glad you like it. I can definitely add a spot clean command, I simply did not see a use for it. I am also planning a few more intents, like battery percentage and current status. Keep an eye on the repo for updates.
While it is true that the CLI offers a timed clean functionality, keep in mind that the Sucks CLI tool is different than the Sucks library, which is what I’m using here. The way the CLI does this, is it sends a clean command, then waits 20 minutes before sending a charge command. That means that your Lambda instance would be running for 20 minutes before it could disconnect, and I’m not sure that’s something you would want.
Thank you for sharing your ideas.
Etienne


Add support for bearer authentication and SSO token providers

Describe the feature
Add support for bearer-token authentication & sso-session
Use Case
these changes are required to support AWS Code Catalyst and the smithy.api#httpBearerAuth trait
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

A note for the community

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Is there a workaround for this? SSO sessions are becoming increasingly more popular. If not, any hints on what changes are necessary to have this implemented?

We're working on this and we expect it to be complete at or before GA. I can't give a date for GA at this time.

We're using it in a CLI tool right now. So I'm doing aws configure sso followed by cargo run -p <crate> -- <args>. Just reverting the versions worked for us. Not using it in any app.
Can you share the env/config for logging?

https://github.com/awslabs/aws-sdk-rust/issues/703 just be careful to redact any sensitive data prior to sending

cc @ysaito1001, I wonder if this is related to credentials caching changes. If you're preloading this doesn't seem to be related to the profile parse issue.

do you use credential_source in your profile? If possible an anonymized version of your profile file would be very helpful

ah! I found it—are you using no-default-features? You need to add credentials_sso

do you use credential_source in your profile? If possible an anonymized version of your profile file would be very helpful

Nope.

ah! I found it—are you using no-default-features? You need to add credentials_sso

Yep, that fixed it. Would've been nice to have a better error message or a mention in the changelogs.

yep I'm fixing the error as we speak, new error:
the credentials provider was not properly configured: ProfileFile provider could not be built: This behavior requires following cargo feature(s) enabled: credentials-sso

@rcoh We also ran into this error:

"ProfileFile provider could not be built: profile ... was not defined: sso_region was missing"

[profile ...]
sso_session = ...
sso_account_id = ...
sso_role_name = ...
region = us-west-2

[sso-session ava]
sso_start_url = https://....awsapps.com/start
sso_region = us-east-1
sso_registration_scopes = sso:account:access

any update?

Sorry for the long delay in providing an update here. I'm working on adding this support now. It'll take a while, so I appreciate everyone's patience.

What's the timeline on this? This sso nonsense has been broken in aws sdks across the board for wayy too long

work is ongoing: https://github.com/awslabs/smithy-rs/pull/2917

First of all, thanks for your hard work, this feature is highly appreciated!
Since the PR has been merged, do you have an ETA for the next release?
Thanks

Just to clarify, for that PR, it's making the existing credentials provider work with the latest CLI. It's not adding SSO token support to the AWS config/profile files (e.g., ~/.aws/credentials). That will still require more work.
We should be releasing in the next 2-3 weeks.

This also appears to be broken with Cloud9 using AWS managed temporary credentials.
$ cargo lambda deploy
Error:   × failed to create function role
├─▶ service error
├─▶ unhandled error
├─▶ unhandled error
╰─▶ Error { code: "InvalidClientTokenId", message: "The security token included in the request is invalid", request_id: "01406350-d28b-4873-a31c-4d64d0936b4a" }

This was resolved in error. Re-opening.

For people just looking for a workaround to be able to use aws-sdk-rust when using the currently recommended way to grant privileges to developers, AIM Identity Center, you need the following:

Enable the sso feature in the aws-config create configuration in your Cargo.toml
Manually edit your $HOME/.aws/config to merge the contents of the profile you want to use with the referenced sso-session. This undoes the changes that are described in this PR
Obtain new tokens using aws sso login and your web browser

This is a snippet that I use from my Cargo.toml:
[dependencies]
aws-config = {  version="0.57.1" , features = ["sso"]}
aws-sdk-sts = "0.35.0"

And these are the changes I made to my (anonymised) $HOME/.aws/config
--- config.old	2023-11-14 21:39:14
+++ config	2023-11-14 21:39:32
@@ -1,10 +1,8 @@
 [default]
-sso_session = s0
 sso_account_id =<PHONE_NUMBER>8
 sso_role_name = AdministratorAccess
 region = eu-north-1
 output = json
-[sso-session s0]
 sso_start_url = https://d-deadbeef.awsapps.com/start#
 sso_region = eu-north-1
 sso_registration_scopes = sso:account:access

Hopefully, teaching aws-sdk-rust the new style $HOME/.aws/config sso config with the sso_session is not too far off, and steps 2 and 3 will no longer be needed.

Any timeline for this being implemented and released?

Just ran into this, it would be great if this was resolved soon.

Where does everyone live? Do we want to get a beach house for a weekend or something?

Any timeline for this being implemented and released?

Can't give a timeline, but I'm picking up work on this again. Hopefully soon.

The January 25th release of the SDK includes support for [sso-session] in the AWS config file for SSO assume role credentials.
I'll continue working on SSO bearer token auth to support CodeCatalyst.

Thanks a lot for this! I will check it out soon.

@jdisanti does that mean #699 is complete?

The bearer token support went out in yesterday's release.


Failed to resolve dependency from jcenter

Can't get dependency from jcenter:

Failed to resolve: com.infstory:kotlinx-android-sharedpreferences:1.0.0

Hi @timurgilfanov ,
Sorry, it no works now, could you get deps from jitpack instead?


Is it "使用"？

yes, please try PR and let's fix it

yes, please try PR and let's fix it

OK，let me try pr


Search preview

The search results preview of content/page description (below the page title) is messy and should be improved.

I would expect description to show up as the description in search.


Simpler Examples

[Enter feedback here]
https://docs.microsoft.com/en-us/powershell/module/exchange/remove-mailboxrepairrequest?view=exchange-ps#example-1
Get-MailboxDatabase -Identity "EXCH-MBX-01" | Get-MailboxRepairRequest | Remove-MailboxRepairRequest
A simpler form would be
Get-MailboxRepairRequest -Database "EXCH-MBX-01"  | Remove-MailboxRepairRequest"
https://docs.microsoft.com/en-us/powershell/module/exchange/remove-mailboxrepairrequest?view=exchange-ps#example-2
Get-MailboxDatabase -Identity "EXCH-MBX-02" | Get-MailboxRepairRequest | Format-List Identity
A simpler form would be
Get-MailboxRepairRequest -Database "EXCH-MBX-02"  |  Format-List Identity
I leave example-3 as an exercise for the Microsoft author

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

ID: bb90d652-6389-b81d-ee92-147fbe6e2607
Version Independent ID: 5ccfedb4-f815-bcae-5a47-196c8577f454
Content: Remove-MailboxRepairRequest (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Remove-MailboxRepairRequest.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@Arne-RFA, fixed as you described.
In the future, you too can make updates like this yourself without opening a GitHub issue first. For instructions, see https://github.com/MicrosoftDocs/office-docs-powershell/blob/main/README.md.
Thanks for the feedback.
Closing


build.rs cargo:rustc and such directives are unusable

Let's say we have a build.rs that looks a bit like this:
fn main() {
  …
  println!("cargo:rustc-link-lib=static=foo");
  …

When we run cargo build on this package, it's going to first build the build.rs file and run it. It will then build the rest of the project.
However, in this case, cargo sees the cargo:rustc-link-lib=static=foo and when building the rest of the project, it's going to pass -l static=foo to rustc.
The builder from crate2nix however just ignores it.
We can't work-around it with something like extraRustcOpts as those will apply during build of build.rs too: if we're creating the foo static library in build.rs however, it is not available.

Some more directives that cargo supports but presumably we just ignore: https://doc.rust-lang.org/cargo/reference/build-scripts.html#outputs-of-the-build-script

Maybe this has nothing to do with crate2nix and should be opened against nixpkgs?

I dived into the nixpkgs source and I found extraLinkFlags; something like this extraLinkFlags = [ "-l" "static=foo" ]; works around the issue at least (but obviously very poorly)

I made the ticket on https://github.com/NixOS/nixpkgs/issues/119382 , closing this one


false-positive (?) on null deferencing

error message:

code that it's referencing:
    public float foo() {
        float key = 1f;

        if (map.get(key) != null) {
            return map.get(key);
        } else {
            //...
        }
    }

changing the key to a Float "fixes" the bug.
strangely, hardcoding the key also doesn't generate a bug report, e.g.:
if(map.get(1f) != null){
  return map.get(1f);
}

is fine according to Infer.

Hi @cxzhang2. Is 1f a global constant in your example ?

It is local

Do you still get the NPE reported if you use:
  if (map.containsKey(key) != null) {
    return map.get(key);
  } ...

?

yes

Yes, sorry.
Thanks for checking. So Infer does not understand that map.get(key) called twice in a row will return the same value. So Infer assume that map.get(key) can return null in return map.get(key); does dereference the returned value to unbox the float value. This is indeed a false positive, thanks for reporting it. I will check how we can make Infer not reporting anything here.
However, I would suggest to rewrite the code as:
  Float value = map.get(key);
  if (value != null) {
    return value;
  } else {
     ...
  }

which avoids an unnecessary lookup from the map.

Hey @jeremydubreil thanks for the suggestion I'll do that for now.  My intuition is that Infer does a ==check on the keys.  This would explain why float generated the bug report but Float and the hardcoded 1f doesn't.   i.e. looking just at the keys of the two map.get() calls, these are the three scenarios:
float key = 1f;
Float.valueOf(key) == Float.valueOf(key)  //false, null check is ignored, report bug

Float key = 1f;
key == key //true, null check is honored

1f == 1f //true, null check is honored

I think infer's heuristic has changed regarding this type of bugs: now it assumes that two calls to the same function will return the same result. Infer version 0.15.0 does not report anymore on this example:
import java.util.HashMap;

class Foo {

  public static Float foo(HashMap<Float,Float> map) {
    float key = 1f;

    if (map.get(key) != null) {
      return map.get(key);
    }
    return key;
  }

  public static void goo(HashMap<Float,Float> map) {
    Float f = foo(map);
    if (f.equals(1f)) { }
  }

}


custom seperators veriable added

it cannot be  multiple seperators  in current situation. i solved in this way.

Please provide examples.

the variable were allowed to only one  seperator but it may be more than one. for instance, i wanted to be both dot and underline as seperator but only one accepted. therefore i   found a such solution as well

I don't really understand why changing the separator setting is not sufficient for you use-case?

yes it is not sufficent for me because  when I  import I want to seperate both dot and underline however, it one accepts.

Can you provide an example file please so I can test this. I'm not 100% sure about your solution to be honest.

you can download sample excel data in following, if you are notice, I put two different seperator veriable that  it is both seperator and underline however,  I said before, it is accepting only one. I hope, I could tell :)
https://github.com/onerciller/sampledata

I also wanted to use a dot, instead of '_' for everything. Eg. user.last_name, because user is a different model, while last_name just uses the seperator for a space..

As I didn't get any reply on the code review anymore, I implemented it myself as "slug_whitelist".


Add external labels support

This PR adds support for external labels so we can configure the workload cluster and so on
Checklist

[x] Update changelog in CHANGELOG.md.
[x] Make sure values.yaml and values.schema.json are valid.

Just a small fix, I added support for externalLabels set as global value so we can configure them in PMO


Remove Ed25519SimpleAddressKeyringEntry

Closes #420

Very nice. I think you just missed an export in @iov/cli

Thanks! There is one issue with @iov/cli imports: since the CI tests a global installation of @iov/cli, I can only import stuff that is available in the last version released on npm. Not cool but the other option is to remove the global installation test again


[AutoPR securityinsights/resource-manager] Cases swagger updates

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6761

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5722


Set views path

Not sure why yet, but I had to adjust the views path when hosting it on Heroku. It wants to look in the lib/gdash directory otherwise. This worked for me but is probably not the right fix.
diff --git a/lib/gdash/sinatra_app.rb b/lib/gdash/sinatra_app.rb
index 05775f8..3df4a7e 100644
--- a/lib/gdash/sinatra_app.rb
+++ b/lib/gdash/sinatra_app.rb
@@ -35,6 +35,7 @@ class GDash

         set :static, true
         set :public, "public"
+        set :views, "/app/views"

         get '/' do
             erb :index

Seems just saying
set :views, "views"
was enough, I also added some other bits like a Gemfile and a placeholder for the graph templates dirs.  It now Just Works on Heroku I think :)

Confirmed. Works for me.


cs231n.github.io visited error!!

Please solve it!!!

https://cs231n.github.io/ loads fine for me -- maybe double-check your internet connection?

Thank u! I can visit it now.


rework policy picker to not be a dropdown

Description
The idea explored was a side list:
https://www.figma.com/file/Q5z8lPJeU26763q30S9GTy/MinK?type=design&node-id=1473-35106&mode=design&t=bPAKe928kR4PCSt0-0

For reference, the design was updated at some point: https://www.figma.com/file/Q5z8lPJeU26763q30S9GTy/MinK?type=design&node-id=1770-33657&mode=design&t=yiCwgKqGAPnEPcXk-0


add morgan.zoemp.be

Howdy, is it possible to contribute to your blogroll with my personal space :-) ?
Thanks a lot !

Welcome aboard @SansGuidon ! Thanks for joining us!


In-page scrolling doesn't make driver update

Example: https://jsfiddle.net/oLj3ksr3/4/
I'm not sure how hard it would be to detect these scroll events, but it would be great if driver.js could handle this case.

Duplicate of https://github.com/kamranahmedse/driver.js/issues/112


In the year 2019 could we consider removing "vbscript" as a possible match for

Before:
 subLanguage: ['actionscript', 'javascript', 'handlebars', 'xml', 'vbscript']

After:
 subLanguage: ['actionscript', 'javascript', 'handlebars', 'xml']

Thoughts?

Seems it still lives on in other places and the CLR, but as Internet Explorer was the only browser that supported it on the web, isn't it pretty much dead now as far as HTML is concerned?

I’m ok with dropping it.

https://github.com/highlightjs/highlight.js/pull/2168


PinLayout now handle correctly parents (superviews) with transforms.

Codecov Report

Merging #108 into master will increase coverage by 0.18%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
+ Coverage   90.37%   90.56%   +0.18%
==========================================
  Files          34       34
  Lines        6232     6356     +124
==========================================
+ Hits         5632     5756     +124
  Misses        600      600

Impacted Files
Coverage Δ

Sources/Impl/Coordinates.swift
100% <100%> (ø)
:arrow_up:

Tests/TransformSpec.swift
100% <100%> (ø)
:arrow_up:

Sources/Impl/PinLayoutImpl+Coordinates.swift
87.65% <100%> (ø)
:arrow_up:

Sources/Impl/PinLayoutImpl.swift
83.08% <100%> (+0.21%)
:arrow_up:

Sources/Impl/PinLayoutImpl+Layouting.swift
88.88% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


make files optional when deserializing form data

Fixes form events on desktop broken in #965. (I only noticed this issue when updating other PRs with the changes)
Form events on desktop never contain the file field which causes crashes when deserializing input events. This PR makes the field optional when deserializing which fixes the issue

I would really like to get some auto testing setup for all of our platforms at some point. We keep having these little regressions that can't really be caught with our current setup.

I would really like to get some auto testing setup for all of our platforms at some point. We keep having these little regressions that can't really be caught with our current setup.

Working on some automated browser tests in #995


Add Users

I'd like to have a blog with multiple users posting. How do I do that?

https://github.com/HubPress/hubpress.io/issues/76 explains some background about this. Please continue the discussion on this issue.


Integrate syntax highlighting

As an end user, I wish to see nicely rendered code snippets. Add support for the syntax highlighting of code!

Resolved


[0.36] Commits to cherry-pick into stable branch

Please pick https://github.com/facebook/react-native/commit/26be005b0a8349afc7f00d8eb0bfe92be85021af to solve some image crashers.

I cherry picked 26be005b0a8349af into 0.36-stable and am sending 0.36.0-rc.1 to CI.

Cherry pick 9c3bfe0cbb3ac3dbbac2ddc3838f31774ef6fe00 to solve Permissions crashes on Android

I cherry picked d91a5c4ffb20224e19e13424068098ea05438640 and db474d95b47e66b5eb6ddbfd5368bdb3d93dd70d into 0.36-stable in order to make sure the Headless JS docs are published alongside the same RN version that introduces the API. These commits only touch the docs on the website, so I did not bump the RC version.

Cherry picked c2ed2ece953e45dc24aebbb5f10a298455e230d7 into 0.36-stable.


Fix #20287: Add flexible types to Quotes library

Fix #20287
To fix the missing case, we have to add flexible types to Quotes library.
The code does not work right now.

Hi @nicolasstucki, do you have any idea why the extractor for FlexibleType does not work in Extractors.scala and SourceCode.scala: TypeRepr is not a valid result type of an unapply method of an extractor.

This should probably be included in 3.5 as it is a part of https://github.com/scala/scala3/pull/18112. Specifically, the changes in the TASTy format require this change in the reflection API.

The core meeting decided to take this PR in on 3.5.0.


Python Tests fails to load PYTHONPATH

Type: Bug

Behaviour
Expected vs. Actual
I have this in my settings file: "terminal.integrated.inheritEnv": true
This seems to have correctly imported my PYTHONPATH that I set in my .bashrc for the purposes of development. (i.e. code completion, jump to definition etc...) all work as expected. However when I write a test file, the test functionality in VSCode fails. I'm able workaround the failure by manually including the path of my custom module via sys.path.append, however, this should not be necessary.

My workspace:
bug_test
-- classA.py
-- tests
---- test_A.py

classA:
class A:
    pass

test_A:
import sys
import unittest

from bug_test import classA
class Test1(unittest.TestCase):
    def test1(self):
        self.assertEqual(4,4)

The expected behavior is for the test to pass.
The actual behavior is:
ModuleNotFoundError: No module named: bug_test

Workaround:
Adding this line in test_A.py : sys.path.append('/home/grechkay/yyg_projects')
fixes the issue and the test passes.

However, this has already been added to the PYTHONPATH in the bashrc file, which is recognized by all of the other VSCode functionality except for the tests.
Steps to reproduce:

Add: export PYTHONPATH='{path_to_module}':$PYTHONPATH in .bashrc
structure your workspace as above
run the tests

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.8.10
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Global
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

User Settings

languageServer: "Pylance"

testing
• unittestArgs: "<placeholder>"
• unittestEnabled: true

Extension version: 2023.12.0
VS Code version: Code 1.80.1 (74f6148eb9ea00507ec113ec51c489d6ffb4b771, 2023-07-12T17:22:25.257Z)
OS version: Linux x64 5.15.0-76-generic
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10700K CPU @ 3.80GHz (16 x 3800)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
0, 1, 1

Memory (System)
31.01GB (11.41GB free)

Process Argv
. --crash-reporter-id cbbfbe56-bd2c-4171-b7c8-8bf66b35fe73

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263cf:30789497
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
282f8724:30602487
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
2i9eh265:30646982
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30771522
hideindicator:30785051
pythongtdpath:30769146
i26e3531:30780429
pythonnosm12tcf:30779713
pythonidxpt:30784022
pythonnoceb:30776495
h7j2d465:30786200
dsvsc013:30789517

Thank you for your issue report. We are looking into this now! In the meantime, you are likely on the new testing rewrite and this is why you saw a change in behavior since we just moved to this experiment to a larger group of people receiving it. You can opt out of the rewrite as I get this fix in by setting this in your user settings: "python.experiments.optOutFrom": ["pythonTestAdapter"],. If this doesn't work let me know as this will mean it is a different issue. Thank you and I will send updates in this thread as I get a fix in.

Hello quick question, is this behavior a regression, ie have you had this work in the past and it no longer does? Secondly, can you check to see if you can see this in your logs Experiment 'pythonTestAdapter' is active? Finally, can you provide all the settings related to testing that you have set? It should be either pytest or unittest args in your workspace settings.json. Thanks

Had the same issue and adding "python.experiments.optOutFrom": ["pythonTestAdapter"] fixed the issue. It did not log Experiment 'pythonTestAdapter' is active in logs.

As far as I know this is not a regression; I have not had this work before. I'm not sure how to check my logs for Experiment 'pythongTestAdapter'
The only settings I have related to testing is: "python.testing.pytestEnabled": true,
However I just checked today and reran the test and everything somehow works now. I can no longer reproduce the error. I'm pretty confused as to what happened, everything even works when I remove the ""terminal.integrated.inheritEnv": true," setting. I'll attempt to reproduce a similar error tonight.

Yea... now VSCode seems to inherit the PYTHONPATH variable that I exported in .bashrc even with an empty settings file. So, I guess I don't have any issues relating to this anymore, although I'm not really sure what happened between when I opened the issue and now.

I had the same problem originally reported here, except that it was solved by adding "python.experiments.optOutFrom": ["pythonTestAdapter"], to my user config.

The same to me. After the latest update, pytest cannot be discovered. I put an .env file in my workspace folder, and specialized PYTHONPATH in it. It can work normally until I updated the extension. Any workaround here? @eleanorjboyd

Thank you, for your responses. Can someone else check to see if they also only see this issue on the rewrite?
You can opt out of the rewrite as I get this fix in by setting this in your user settings: "python.experiments.optOutFrom": ["pythonTestAdapter"], to join the experiment add this setting to your users settings.json "python.experiments.optInto": ["pythonTestAdapter"]. (so depending on if it is working or not you can try the other boolean)
You can confirm you have the rewrite enabled by setting  "python.analysis.logLevel": "Trace",  in your settings and from the command palette "developer: set log level..." and select trace. Reload your vscode window then in your python output panel check for Experiment 'pythonTestAdapter' (should be at the very top of the python output)
Let me know if the rewrite fixes your issue. Thanks!

I'm pretty sure this happens to me as well, but toggling the "python.experiments.optInto": ["pythonTestAdapter"] setting either way doesn't have an effect.  I have a multi-root workspace where different roots are different python libraries, each with their own virtual env.  So if I have library A and library B and library B depends on A, I get a ModuleNotFound error [trying to] test B (unittest discovery error).  PYTHONPATH set with launch.json or .env doesn't help running tests but seems to help the debugger.  My workaround has been to install library A's wheel file into library B's virtual environment.  But that's not really ideal.  I'd like to be making live changes in both places.

Looks like I have this issue too, when using "python.envFile": "${workspaceFolder}/.vscode/.env" to prepend local paths to the PYTHONPATH. When pytest discovery is running the value of PYTHONPATH is /Users/me/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles and discovery fails with module load errors. If I opt out of the rewrite it works as it used to.
Double checked 'pythonTestAdapter' is active is showing in the log too.

Another case here. I described my situation extensively in https://github.com/microsoft/vscode-python/discussions/21872.

@eleanorjboyd  I'm fine to close out the issue if you are.

Great thank you!

@eleanorjboyd I still see this issue if I don't opt out of the new test adapter experiment, and we keep having to opt out team members if they've been included. Is there any more info I can provide?
https://github.com/microsoft/vscode-python/issues/21642#issuecomment-1659654444

Hi @woodyza, what version of vscode and the python extension are you on? Can you also describe your project setup more so I can look to replicating it?

found the issue- seems we are not referencing the user-defined env file as part of the environment when configuring the sub-process for the test run. I will add it now but this will require a PR. In the meantime hopefully, your workaround can suffice and I will close this issue when the fix is merged. Thanks

Thanks @eleanorjboyd!

Is anyone able to verify if this is fixed? If not I can have a team member do so before our next stable release. Thanks!

Using the v2023.19.12961014 pre-release of the extension it works for me, using that simple test case and without setting tool.pytest.ini_options. Thanks!


Restrict app creation to admins only

@fbjork asked about the ability to restrict new app creation to admins only. I think this may be part of a larger discussion around permissions. Related is #3976.

Something along these lines would be great: https://devcenter.heroku.com/articles/org-users-access

@fbjork Can you elaborate? Do you think the implementation of organizations would be useful, or the permissions split between admins/users that Heroku uses?
We try to be Herokuish whenever possible. We're also working to rewrite the controller into smaller services, and auth is going to be one of those, so we have the opportunity to revisit our design for user authentication.
/cc @mboersma @bacongobbler @technosophos

Permissions between admins and users is what we need. For example we want some users to deploy apps, view configuration, but not create new apps or edit configuration.

viewing configuration is a sensible point, as it's where secrets and passwords are stored. I don't think it's a good idea, or it would need some special config treatment

No, this likely won't become a default, however users who have special use cases like @fbjork should be able to tune the permissions to their liking. Today you can tune the settings by changing the values set as seen here in the AppViewSet (updating the unit tests to ensure it's correct) and then deploying your own controller.
I think it's a matter of documenting how users like @fbjork can do this to cover their own use cases, but a larger proposal on how we can make this easier would probably need to have a larger discussion through a design document.

I think it's a matter of documenting how users like @fbjork can do this to cover their own use cases

I don't think asking everyone who wants to do this to build a custom controller component is a solution - we're going to need to address customizable permissions at some point, and breaking down the controller into microservices seems to be a good time to do this.
But agreed on the design document for a proposal. :+1: Assigning this issue to @technosophos to think about when taking a stab at the controller rewrite.

@fbjork Does https://github.com/deis/deis/issues/4150 fit your use case? I'd love any feedback you have!

PR #4265 will fix this.

closed via https://github.com/deis/controller/pull/1116.


Performance: Update perfect-scrollbar dependency to use passive listeners

Running Lighthouse test on site that uses this library throws a following error, that decreases the overall performance score:
Does not use passive listeners to improve scrolling performance
This is a problem of a perfect-scrollbar dependency itself. This topic was mentioned here https://github.com/mdbootstrap/perfect-scrollbar/issues/975 and solved.
So basically updating the perfect-scrollbar dependency to to its latest version should fix the issue.

The PR with update is ready.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 2583 ms

Resolved: Bofrost.be performance has improved in 197a061.


Fix SQ model restore loading

Type of Change
[NLPTOOLKIU-1299] smoothquant qconfig restore error - IT JIRA (Supports IC/ITS Data) (intel.com)
Description
detail description
JIRA ticket: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

https://inteltf-jenk.sh.intel.com/view/lpot-validation/job/LLM_accuracy_INC_ITREX/266/
https://inteltf-jenk.sh.intel.com/view/lpot-validation/job/LLM_accuracy_INC_ITREX/267/


Please add support for FlowSmart Control Meter & Valve Controller

Hello,
I've recently added a new water valve/flow meter to my hub and noticed I've been getting errors in my log (I've redacted some identifiers, but let me know if this is required):
YoLink device type: 'WaterMeterController' is not supported (YS5006-UC (xxxxx) YoLink Water Meter) (handleGet)
Please report all bugs at https://github.com/dkerr64/homebridge-yolink/issues
{
"state": {
"valve": "open",
"meter": 7828
},
"alarm": {
"openReminder": false,
"leak": false,
"amountOverrun": false,
"durationOverrun": false,
"valveError": false,
"reminder": false,
"freezeError": false
},
"battery": 4,
"powerSupply": "PowerLine",
"valveDelay": {
"ch": 1,
"off": 0
},
"attributes": {
"openReminder": 0,
"screenMeterUnit": 0,
"meterUnit": 0,
"alertInterval": 0,
"meterStepFactor": 10,
"leakLimit": 10,
"autoCloseValve": true,
"overrunAmountACV": false,
"overrunDurationACV": false,
"leakPlan": "schedule",
"overrunAmount": 500,
"overrunDuration": 150
},
"version": "0603",
"tz": -7,
"recentUsage": {
"amount": 33,
"duration": 23
},
"loraInfo": {
"netId": "010201",
"devNetType": "D",
"signal": -36,
"gatewayId": "XXXXX",
"gateways": 2
}
}
I'm not quite sure what data I would expect to see, but I know some of my other devices (like washer/dryer) will show a faucet icon in home with a timer of how long it runs.
Thanks!

Duplicate of #98 please read discussion in there and try installing plugin beta.


Errors due to account in ignored OU

Describe the bug
I am trying to deploy LZA as per the implementation guide.
There are a couple of accounts placed in a "Suspended" OU. Most of the accounts in that OU are closed and are in "suspended" state except one which cannot be closed yet due to the 30 days quota limitation.
I have edited the organization-config.yaml to ignore the whole OU as follows:
enable: true
organizationalUnits:
  - name: Security
  - name: Infrastructure
  - name: Suspended
    ignore: true
serviceControlPolicies: []
taggingPolicies: []
backupPolicies: []

Now when I try the deploy, the AWSAccelerator-Pipeline code pipeline fails on the Bootstrap stage complaining in my case about AWSControlTowerExecution IAM role failing to do ssm:GetParametersByPath on the /accelerator/iam/role SSM parameter due to SCP explicit deny.
The error is "correct" since I put a deny everything SCP on the "Suspended" OU to deny everything as I am in the process of decommissioning the accounts in it.
Some questions stem from the above behavior:

Can you ignore a specific account explicitly or you have to put it in a OU and then ignore the whole OU in organization-config.yaml ?
If only the latter is the available option from above then why despite the whole OU being ignored, the LZA deployment tries to do stuff on an account in it?
All of the accounts are in "Suspended" state except this "problematic" one. Does that matter? If yes, why (if I am ignoring the whole OU)?

To Reproduce

ignore an OU with a non suspended account in it in organization-config.yaml
add deny all scp policy to the ignored OU
try to deploy LZA
observe errors for that account

Expected behavior
I expect the LZA to either not attempt to do anything on ignored OU or have some configuration where you can ignore accounts directly.
Please complete the following information about the solution:

Version: v1.5.2
Region: eu-west-1
Was the solution modified from the version published on this repository: No
Have you checked your service quotas for the services this solution uses: Yes
Were there any errors in the CloudWatch Logs? Yes, /aws/codebuild/AWSAccelerator-ToolkitProject from CodeBuild

Screenshots
n/a
Additional context
The management account was cleaned up from Control Tower LZ and then re-onboarded (essentially being re-done).

Hi Bozhidar! Thank you for using the Landing Zone Accelerator on AWS solution. There are added steps that you'd need to take in the AWS Control Tower service when it comes to closing an account. Have you attempted to follow the steps in our Implementation Guide?

Hi Bozhidar! Thank you for using the Landing Zone Accelerator on AWS solution. There are added steps that you'd need to take in the AWS Control Tower service when it comes to closing an account. Have you attempted to follow the steps in our Implementation Guide?

Hi, yes I did.
The whole idea of raising this bug is that actions are attempted on an account that is put in ignored OU.

Thank you for confirming, Bozhidar. To answer your questions:
Q: Can you ignore a specific account explicitly or you have to put it in a OU and then ignore the whole OU in organization-config.yaml ?
A: Right now we only support ignoring accounts through an Organization Unit as you can verify from our config.
Q: If only the latter is the available option from above then why despite the whole OU being ignored, the LZA deployment tries to do stuff on an account in it?
A: I haven't come across this issue before and this is probably a legitimate bug. I'm going to go ahead and create a bug to try and replicate and resolve this issue.
Q: All of the accounts are in "Suspended" state except this "problematic" one. Does that matter? If yes, why (if I am ignoring the whole OU)?
A: From the above, it shouldn't given that it's failing at the Bootstrap phase for this account. The OU, and more specifically, the accounts in that OU shouldn't be bootstrapped.
I will keep you up to date on this issue as we'll leave it open for further correspondence while we have this bug open.


About the structure of PFF

Hello, excellent author, the idea of your paper is very novel, but for the PFF module I have a question for you: is it possible to have two convolutions without any normalization and activation operation between them?

In theory, it is possible to do like this, but we haven't done any experiments to see if it will work.
At 2023-05-06 15:38:57, "Maxwell Wang" @.***> wrote:
Hello, excellent author, the idea of your paper is very novel, but for the PFF module I have a question for you: is it possible to have two convolutions without any normalization and activation operation between them?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you for your prompt reply, as I found in your code model.py in lines 67-68 that
down = DepthwiseConv2D(2, strides = 2, padding = 'same') (pre)
down = Conv2D(shape[3], (1, 1), strides = 1, padding = 'same') (down)
There are no normalization and activation operations between them, so why can we not add non-linear operations between these two lines of code?

This is depthwise separable convolution, which is an independent structure, and thus we don't add non-linear operations between these two lines of code
At 2023-05-06 16:45:53, "Maxwell Wang" @.***> wrote:
Thank you for your prompt reply, as I found in your code model.py in lines 67-68 that
down = DepthwiseConv2D(2, strides = 2, padding = 'same') (pre)
down = Conv2D(shape[3], (1, 1), strides = 1, padding = 'same') (down)
There are no normalization and activation operations between them, so why can we not add non-linear operations between these two lines of code?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


fixed quickly Commenting on PRs hit api limit

Please review @roryabraham
Details
Added following changes to markPullRequestsAsDeployed GH action:

Send comment requests sequentially.
Added waiting period of one second after each comment request.

Fixed Issues

Fixes #2759

@roryabraham Were you expecting something like these changes? Please review.

It seems Token is not being passed. Could you please help?

@parasharrajat try replacing this.octokit in all the static methods with GithubUtils.octokit ?

Also, if you find it more complex, then also just feel free to move the octokit initialization to the regular-old constructor and leave the non-static functions as non-static.

That this is fine it is referencing to the constructor. But it seems that tests are failing as we are not passing the Github_token from env.
https://github.com/Expensify/Expensify.cash/blob/5e50f17c4599071de29f90f62ab7f9bbf55e1602/.github/workflows/test.yml#L27
If we update the test.yml file to include  GITHUB_TOKEN: ${{ secrets.OS_BOTIFY_TOKEN }} in env, will it work.
previously test file was using @octokit/rest and it was not using GITHUB_TOKEN. But now we updated to GithubUtils and now its all static so we have to pass a token.
I have one thing to bypass this.

Keep const {Octokit} = require('@octokit/rest');
Pass the new Octokit() to

GithubUtils.octokitInternal = new Octokit();

This bypasses the Octokit default instance creation.

@parasharrajat Actually I think what we want to do is just mock core.getInput for the GithubUtilsTest unit test.

Sorry, @roryabraham. But I do not seem to understand this. As per me we are using the real Octokit instance.
const {Octokit} = require('@octokit/rest');
const GithubUtils = require('../../.github/libs/GithubUtils');

on the top of the file.

It's a unit test so we don't want or need a real token

Agree to this, I think we can pass the Octokit instance from const {Octokit} = require('@octokit/rest'); like we were doing previously. But this time we have another way to do this.
GithubUtils.octokitInternal = new Octokit();

After this, I am sure the test will pass.
We are only mocking the Octokit at one place.
https://github.com/Expensify/Expensify.cash/blob/45b99bbaf7bbc33037c0d8ad8b8bb7aff7055611/tests/unit/GithubUtilsTest.js#L411

@parasharrajat Okay, I'm not sure I'm following your solution, so feel free to update the PR so I can see what you mean. There are many examples of mocked octokit methods throughout the GithubUtilsTest.js file. Here's one.
So the solution I'm suggesting is that you don't change the implementation of static get octokit at all. What about something like this in the unit tests:
// This is a mock Octokit object. When we need to mock something in Octokit, we would just do something like:
// moctokit.issues.listForRepo = jest.fn().mockResolvedValue({data: []});
const moctokit = {};

beforeAll(() => {
    GithubUtils.octokitInternal = moctokit;
});

And then change all the instances like this one to:
// Before
GithubUtils.octokit.issues.listForRepo = jest.fn().mockResolvedValue({data: []});

// After
moctokit.issues.listForRepo = jest.fn().mockResolvedValue({data: []});

@roryabraham Updated.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Great news! It seems this didn't catastrophically break all our GH Actions, which means that it probably works as expected!


[Performance] [Audit]: messages with Views nested in Text for inline code are 700% slower to render on Android

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

This report is part of #3957, scenario "Rendering Individual chat messages".
Flamegraph
This flamegraph shows commit c153. The full commit log is available HERE to inspect in Flipper / React Devtools (you will have to unzip it first).

Commit c153 refers to one RenderHtml component with 20 words wrapped in inline code. Something like this:
`inline code spanning multiple lines inline code spanning multiple lines inline code spanning multiple lines inline code spanning multiple lines`

Its rendering time is 112ms (yellow bar in chart below). All other messages (green bars), including some messages with as many words are rendered between 8 to 14ms. So that's a minimal overhead of 112 / 14 = 8 or +700%.

Proposal: Review InlineCode
Review if the design requirements that have led to implementation of #2527 are worth the rendering cost. You might want to profile this on iOS also.

Triggered auto assignment to @laurenreidexpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Interesting. I'm not sure if this is a mainline case that we need to prioritize for inline code blocks are kind of a power user feature. I think we are more generically concerned with the typical usage scenarios which is probably ... not using any markdown at all. But not too sure. Will let others share their thoughts on this.
700% sounds pretty bad but we can also test the performance here...
Benchmarking idea:

Create two chats. 1 with no inline comments and one with several multi line inline code comments.
Compare the time to switch between both on Android.

Does it take an excessive amount of time to render the chat with inline code blocks?

This is still on hold

Will review this when N6 hold is off


[Tempest] Fix private key permissions

This patch changes the mount destination for the private key. The reason for this change is to make it possible to change the permissions for the private key to 600.
Depends-On: https://github.com/openstack-k8s-operators/tcib/pull/159

I'm waiting for a tempest image that contains this change. If I did not wait I could break some jobs.

https://quay.io/repository/podified-antelope-centos9/openstack-tempest?tab=tags&tag=latest

the depends on is merged


Does not build with VS 2017 Version 15.7

The solution does not build using Visual Studio 2017 Version 15.7  targeting Windows SDK 10.0.17134

You must have the Fall Creators Updated SDK installed.


AudacityTeam.Audacity version <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

[Policy] Needs-CLA

Stop contacting me
Sent from Yahoo Mail for iPhone
On Saturday, June 24, 2023, 3:55 PM, microsoft-github-policy-service[bot] @.***> wrote:
Hello @Nickster888,
This PR cannot be merged until you sign the Contributor License Agreement (CLA). More information on this process can be found on the Microsoft Open Source website.
Template: msftbot/needsCLA
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


404 link in docs

Expected content
Information on making an SMTP sendgrid email
Actual content
404
Link to page:
https://sendgrid.com/docs/for-developers/sending-email/building-an-smtp-email/
Came from:
https://sendgrid.com/docs/for-developers/sending-email/integrating-with-the-smtp-api/

Duplicate of 4825


Make app visible in Share menus

In many apps, Share, Send To, or similar buttons (see picture) let you quickly share to other apps without needing to browse the file from the destination app. I think it'll be useful to have this as a feature; currently, the app doesn't appear in these menus.

I don't know how this can be done or if it's even possible. Anyway, thanks for the app; it works like a charm.

Hi @swaree
thanks for your Input. I have this on my todo, but it's not as easy as including it in an app you control yourself. I guess the only possibility is to fake drag-drop the file to the whatsapp chat - but then the correct chat needs to be open, which is not very user friendly/can be confusing. If I come up with an easier way, I'll add it. Maybe someone else also has another idea

@92lleo I love that idea, what if some bar would show up below the appbar when the app "receives" some shared stuff, saying: "Select chat, and press [here] to insert shared content. [CLOSE]" ?
Then it would be a quite clean and user friendly solution too. The user would also be able to close that bar with the close button if the operation was successful. OR send the same stuff to other users, which would make this even more powerful.

+1 for this, and a bit of context for my use-case:
For security reasons, I always strip exif metadata from media before sharing it publicly. To do this with WhatsappWebToGo, the workflow is:

Open the image in Gallery
Share the image in Gallery
Select Scrambled Exif in Share menu
Select WhatsappWebToGo in subsequent Share menu

Unfortunately this doesn't work because WhatsappWebToGo doesn't appear in the Share menu.
Is there any workaround where I (as the admin of my own damn phone), can specify that WhatsappWebToGo should appear in this share menu myself?


Adds support to send Arrays as more extras.

Add support to send Object arrays as more extras to zxing Readers.

Thanks, this will be useful for a couple of cases.


Add benchmarks and use cursor/uncons instead of pos/drop

See related discussion in #1 (which this replaces) and #3.
This adds a few benchmarks that attempt to simulate a few different use-cases:

streamingBigReadSmallChunksCount feeds a bunch of small chunks of data into a parser that is looking for a large chunk of data at a time by using count with a large n.
streamingBigReadSmallChunksTake is similar to streamingBigReadSmallChunksCount but uses take instead of count.
streamingSmallReadSmallChunks feeds a bunch of small chunks of data into a parser that is looking for small chunks of data at a time.

disclaimers

I don't know how well these scenarios line up with real-world use-cases. I tried to pick some cases that seemed like they could hit performance bottlenecks but that weren't too far from scenarios that I might expect to pop up in the wild.
This is a very small set of benchmarks. Ideally many more cases would be tested. For example, I haven't done any benches where we are feeding large chunks of data at a time. I was starting to lose steam writing benchmarks and wanted to get something out there for review.
Benchmarks don't tell the real story yada yada yada

results
The are throughput measurements, so bigger numbers are better. In these particular examples, the cursor/uncons approach with Chain seems to have orders of magnitude better performance than the other two approaches.
My speculation about why the other two approaches have such low throughput would be that it relates to the repeated size checks, indexing into, and dropping elements from the front of these structures. With the cursor-based approach, we only have to go back and start at the beginning of input if we need to backtrack for an orElse. With the pos-based approach, we are constantly taking the original input and dropping elements (probably an O(pos) operation) to get to the relevant position in the structure. When we move on to the next parser in the chain, we have to do this again (with a slightly higher pos). The length check on this structures is likely even worse, as it is generally O(input.length). I tried using lengthCompare instead of length in the pos-based implementations, but I didn't see a significant improvement.
raw data
current master (chain with pos/drop)
code
[info] Benchmark                                      Mode  Cnt  Score   Error  Units
[info] ParserBench.streamingBigReadSmallChunksCount  thrpt   10  0.061 ± 0.001  ops/s
[info] ParserBench.streamingSmallReadSmallChunks     thrpt   10  0.046 ± 0.003  ops/s
[info] ParserBench.streamingBigReadSmallChunksTake   thrpt   10  0.509 ± 0.008  ops/s

Queue with pos/drop
code
[info] Benchmark                                      Mode  Cnt  Score   Error  Units
[info] ParserBench.streamingBigReadSmallChunksCount  thrpt   10  0.258 ± 0.004  ops/s
[info] ParserBench.streamingBigReadSmallChunksTake   thrpt   10  1.362 ± 0.028  ops/s
[info] ParserBench.streamingSmallReadSmallChunks     thrpt   10  0.195 ± 0.007  ops/s

chain with cursor/uncons
code
[info] Benchmark                                      Mode  Cnt    Score    Error  Units
[info] ParserBench.streamingBigReadSmallChunksCount  thrpt   10  192.938 ±  8.650  ops/s
[info] ParserBench.streamingBigReadSmallChunksTake   thrpt   10  204.053 ± 11.377  ops/s
[info] ParserBench.streamingSmallReadSmallChunks     thrpt   10  145.526 ±  7.355  ops/s

Oh I guess that Iterator.unfold only exists in Scala 2.13+ :\
I'll see if I can work around it.

Collections and 2.13, but its good to know that it looks like such an improvement.
I mean 200 or 400x is pretty worth it as long as we communicate about the space requirements.

I don't know why I was using unfold anyway... there was a much simpler way 😬

Oh right I meant to leave a disclaimer about not assessing memory usage.

We should just admit that is can be as much as 2x, but it is likely much less.

@ChristopherDavenport I can add a note about performance. Do you want it in the readme?

Yeah, that sounds good. Then we'll get a release out with this so we can see if it better fits our needs. 👍


What's the connection between Zip and StoreKit?

I meet a problem really wired, the Zip module effects the In-App purchase, I don't know why.
The problem is like this:

At first, everything goes fine, both Zip function and Purchase function;
Next morning, Zip suddenly not available. Xcode said: "Can't find module minizip" (something like this, maybe not exactly the same);
I drag unzip.h into the framework folder of the project, it works fine;
Next morning, the In-App purchase not working. As went to "SKPaymentQueue.default().add(SKPayment)", it crashed and said "EXC_BAD_ACCESS, code =1 etc.";
I tried a lot to find the bug, at last I found something really wired. As I print object IAP product list, the output said:

"warning: Swift error in module Zip. Debug info from this module will be unavailable in the debugger."

I just delete the unzip.h from project, everything is fine, again;
Next morning, this problem occurred again. Whatever I do, it always crash when IAP triggered, and show this message:"warning: Swift error in module Zip." even I remove Zip framework!

My question is, why Zip module related StorKit module? Why these problems seems occurred randomly? Please help me, thank you!

Thanks @lvcloudgit, that's one strange issue. Zip doesn't have anything to do with StoreKit.
Can you share some code so I can investigate your issue further?

@marmelroy Thanks for your reply!
My code is a bit chaos and I don't know which part should be posted... But I found another possibility, still it's not work every time, but maybe it shows some tips?

The Build Settings of TARGETS has an Option "Always Embed Swift Standard Libraries", the value is "No" by default. I change it to "Yes", the error just disappeared!
But still... It's really bothered me, next morning, the error happened again! And I just change this option back to "No", the error disappeared!
So right now, seems the solution is just switch this option... would you take a guess where the problem could be?
This is the IAPHelper.swift code:
import Foundation
import StoreKit

public typealias bookId = String
public typealias BooksRequestCompleteHandler = (_ success: Bool, _ books:[SKProduct]?) -> ()

open class IAPHelper: NSObject {

    static let IAPHelperPurchaseNotification = "IAPHelperPurchaseNotification"

    var books: Set<bookId>
    fileprivate var purchasedBookIds:Set<bookId> = Set()
    var aRequest:SKProductsRequest?
    var booksRequestFinishedHandler: BooksRequestCompleteHandler?

    init(bookIds: Set<bookId>) {
        books = bookIds
        for id in bookIds {
            let purchased = UserDefaults.standard.bool(forKey: id)
            if purchased {
                purchasedBookIds.insert(id)
            } else {
            }
        }
        super.init()
        SKPaymentQueue.default().add(self)
    }
}

// MARK: StoreKit API
extension IAPHelper {

    public func requestProducts(_ completionHandler: @escaping BooksRequestCompleteHandler) {
        aRequest?.cancel()
        booksRequestFinishedHandler = completionHandler
        aRequest = SKProductsRequest(productIdentifiers: books)
        aRequest!.delegate = self
        aRequest!.start()
    }

    public func buyProduct(_ product: SKProduct) {
        let payment = SKPayment(product: product)
        SKPaymentQueue.default().add(payment)
    }

    public func isProductPurchased(_ productIdentifier: bookId) -> Bool {
        return purchasedBookIds.contains(productIdentifier)
    }

    public class func canMakePayments() -> Bool {
        return SKPaymentQueue.canMakePayments()
    }

    public func restorePurchases() {
        SKPaymentQueue.default().restoreCompletedTransactions()
    }
}

// MARK: SKProductsRequestDelegate
extension IAPHelper: SKProductsRequestDelegate {
    public func productsRequest(_ request: SKProductsRequest, didReceive response: SKProductsResponse) {
        let booksGet = response.products
        booksRequestFinishedHandler?(true, booksGet)
        clearRequestANdHandler()

    public func request(_ request: SKRequest, didFailWithError error: Error) {
        booksRequestFinishedHandler?(false, nil)
        clearRequestANdHandler()
    }

    public func clearRequestANdHandler() {
        aRequest = nil
        booksRequestFinishedHandler = nil
    }
}

// MARK: SKPaymentTransactionObserver
extension IAPHelper: SKPaymentTransactionObserver {

    public func paymentQueue(_ queue: SKPaymentQueue, updatedTransactions transactions: [SKPaymentTransaction]) {
        for transaction in transactions {
            switch (transaction.transactionState) {
            case .purchased:
                completeTransaction(transaction)
                break
            case .failed:
                failedTransaction(transaction)
                break
            case .restored:
                restoreTransaction(transaction)
                break
            case .deferred:
                break
            case .purchasing:
                break
            }
        }
    }

    fileprivate func completeTransaction(_ transaction: SKPaymentTransaction) {
        UserDefaults.standard.set(true, forKey: transaction.payment.productIdentifier)
        deliverPurchaseNotificatioForIdentifier(transaction.payment.productIdentifier)
        SKPaymentQueue.default().finishTransaction(transaction)
    }

    fileprivate func restoreTransaction(_ transaction: SKPaymentTransaction) {
        guard let productIdentifier = transaction.original?.payment.productIdentifier else { return }
        deliverPurchaseNotificatioForIdentifier(productIdentifier)
        SKPaymentQueue.default().finishTransaction(transaction)
    }

    fileprivate func failedTransaction(_ transaction: SKPaymentTransaction) {
        if transaction.error!._code != SKError.paymentCancelled.rawValue {
            print("Transaction Error: \(transaction.error?.localizedDescription)")
        }

        SKPaymentQueue.default().finishTransaction(transaction)
    }

    fileprivate func deliverPurchaseNotificatioForIdentifier(_ identifier: String?) {
        guard let identifier = identifier else { return }
        purchasedBookIds.insert(identifier)
        UserDefaults.standard.set(true, forKey: identifier)
        UserDefaults.standard.synchronize()
        NotificationCenter.default.post(name: Notification.Name(rawValue: IAPHelper.IAPHelperPurchaseNotification), object: identifier)
    }
}

/// get productIdentifiers
class PurchaseBooks {
    var bookListArray:[String] {
        didSet {
            self.bookIds = Set(bookListArray)
            self.store = IAPHelper(bookIds: self.bookIds)
        }
    }

    var bookIds:Set<bookId> = []
    var store:IAPHelper?
    init() {
        bookListArray = bookListData?.allKeys as! [String]
        bookListArray = []
        bookIds = Set(bookListArray)
        store = IAPHelper(bookIds: self.bookIds)
    }
}
var bookPurchase = PurchaseBooks()

** This is the

The part of Unzip:
func resetDefaultBookOffline() {
        let defaultZip = Bundle.main.path(forResource: "defaultResource", ofType: "zip")
        let booklistFile = "\(documentPath)/BookList.plist"
        let fileMgr = FileManager.default
        if fileMgr.fileExists(atPath: booklistFile) == false {
            do {
                try _ = Zip.unzipFile(URL(string: defaultZip!)!, destination: URL(string:documentPath)!, overwrite: false, password: nil, progress: {
                    (progress: Double) -> () in
                    // print("Unzip progress: \(progress)\n")
                })
                try? fileMgr.removeItem(atPath: documentPath + "/__MACOSX")
                try? fileMgr.removeItem(atPath: documentPath + "/default.zip")

                print("Unzip complete, path：\(documentPath)")

            } catch let error {
                print("Unzip error：\(error.localizedDescription)")
            }
        }
    }

I'm sorry, it still can't work well. The error is:

It's not always happened, but sometimes. What's this mean? Why is that?
Sometimes it just happened in running Profile mode.

I just use unzip once all over the project, as the code I just posted, pretty simple.
I think it's not caused by any specific code, maybe it caused by some build settings options?


Undefined index: remember_token

When logging out I am getting the following error:
Undefined index: remember_token
I am assuming this is because Laravel is trying to use the default Auth::logout() function and the remember_token is not available when logging in via WHMCS.
I have been able to fix it but I don't know if this is the correct or best solution.
In the WhmcsUser.php file I added:
public function getRememberToken()
{
    return '';
}

Hello, @tecreasey and thank you for the report.
Unfortunately, I'm unable to test it myself currently, but I think your solution will work. I will just change an emtry string to null, since that's the default if the token is not present. The new release is on it's way.. :)

Brilliant. I have updated my version and the issue is solved. Logging a user out now completes without an issue.


Fix dashboard

What this PR does / why we need it:
1.Add dashboard profile replication/snapshopt/qos policy
2.Fix dashboard package.json
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
@xing-yang @leonwanghui
Release note:

Coverage remained the same at 43.667% when pulling 5873f475d8da6b910dcf4c5493f21c09f093df4e on jimccfun:development into 517c8e27ca7e116ad32d0dbf59a2bdb67f3d0325 on opensds:development.


Add/Link to instructions for calculating the $base64Thumbprint

The documentation for this page does not include instructions for calculating $base64Thumbprint.

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

ID: 7e44b783-3ca5-6b6a-1f33-d7a107b8f8e0
Version Independent ID: aa2bb5fd-bd0d-830c-84c7-0525657b6d4f
Content: Certificate credentials in Azure AD
Content Source: articles/active-directory/develop/active-directory-certificate-credentials.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

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

@centromere You can refer to this Configuring a X.509 public cert for your application for creating base64 thumbprint certificate.

@SaurabhSharma-MSFT Thanks! Could this article be enhanced by having a link to that page?

@CelesteDG Can you please check if it is possible to add a reference link in the current document for creating a base64 thumbprint certificate?

Hello @centromere and @SaurabhSharma-MSFT - Thank you both for the suggestion and the link. I'm happy to include this as a link. I'll link the PR to this issue once I've got that going. Thank you.

Hi,
Is there any additional information about the actual constraints associated to the customKeyIdentifier ?
I am using the New-AzureADApplicationKeyCredential, and I can't make sense on how this works 😅:

I have successfully used other values (email addresses for example) than the base64Thumprint computed in the documentation.
Using the same key associates indeed an identical value in the application's manifest addition in the KeyCredentials.
I have no idea on where the value comes though 😁.
Uploading the public key from the portal creates an entry with a different customKeyIdentifier and more disturbingly with a different thumbprint showing up in the Azure portal.
This thumbprint is the same as it shows up in the Certificate Management application.

Any light would be appreciated 😅

@chaami, I suggest you have a look at the README.md of the Authenticating to Azure AD in daemon apps with certificates sample, it shows New-AzureADApplicationKeyCredential works. My understanding is that the customKeyIdentifier is just an identifier (unique) for the public key / certificate, and as such the thumbprint is a good candidate for a unique id (even if it's not very human readable). But you can actually use whatever string you want.

Hi,
Thanks @jmprieur for your suggestion, snooping around the link you gave me and reading this script I ended up understanding.
The script New-AzureADApplicationKeyCredential command pushes the Base64 encoding of the value you put there.
I actually put the name of the key holder to identify the keys easily (just have to base64 encode it 😉 ).
I am still at loss regarding the Thumbprint that ends up associated to the key in the Azure portal...
Which makes it almost impossible to remove a particular key through the website.
Again uploading the key associates it to the expected thumbprint (meaning the one showing up in the Microsoft Management Console).
Removal remains possible through editing the manifest and computing the customKeyIdentifier from the value used 😅
Thanks for reading so far 😁

Thanks for the update, @chaami
I'll ask the Azure portal people for an answer about the portal behavior.

Hello @CelesteDG thanks a lot for the changes !
My question is probably now more related to the Azure Portal display in the context of an application creation.
Maybe more related to active-directory-integrating-applications document (but it doesn't mention the powershell commands) ?
But if you are ok to leave it open, it's fine by me 😉

@chaami - Thank you! Actually, if you don't mind opening a new Github issue within that topic, and then repeating your question and perhaps referencing this thread, that would be great. I've found that other customers look at the open and closed github issues that exist/were opened for a specific topic so maybe there are other customers who have a similar question within the  active-directory-integrating-applications topic and our discussion might be of interest to them. Then we can close this issue.

Hi @CelesteDG ,
I have created https://github.com/MicrosoftDocs/azure-docs/issues/12476.
Let me know if you think I should phrase it differently.
I want to post a comment in the New-AzureADApplicationKeyCredential documentation page, but I just can log into the system 😆
Feel free to close this issue and thanks again.
Kind regards,

Thank you, @chaami ! :)
@MohitGargMSFT @SaurabhSharma-MSFT @MicahMcKittrick-MSFT @Mike-Ubezzi-MSFT - The original issue has been fixed and update is live. Separate issue opened to track other question. #please-close


Add currency support

Currently, the symbol written in the project is RMB => ￥, now we need to add USD =>$ support

Added


All cars which are on the track are shown yellow flag

Hey Lenny,
in the version 1.5.6, all the cars which are on the track are shown with yellow flag. This is Cayman GT4 at Zolder. I will add the log after the race.

Yes i am aware but thank you for noticing as well.
The game had a bug and when they fixed it, they changed something else. Now the yellow flag detection does not work correctly.
The issue has been brought up to kunos on the forums but there is no response yet.
I want to give them a little more time to have any reaction before i do something about it.
If they dont adress this issue then the only real "fix" for it is to turn off the yellow flag detection since it simply will not work with the current state of the game.
Lets wait until tomorrow this time to see if the decide to response on the forum.

Ah okay, understood. Well lets hope they will fix that :) Thanks for taking care of the app, really a life saver!

This might actually get fixed which is nice.
So im happy to wait a few more days for that.

Hey Lenny, any update on this? Was it fixed in the game? Or have you found some workaround? The yellow flag detection was extremely handy for streaming..


Invalid weeks returned for the pattern YYYY[W]WW

Describe the bug
Every time I try to get a week of the patter "YYYY[W]WW" for the year > 2020 (2021, 2022 ...) I am getting the week increased by one
moment("2022W03", "YYYY[W]WW").week()
result : 04
To Reproduce
Steps to reproduce the behavior:

Go to 'moment("2022W03", "YYYY[W]WW").week()'
returned "4"
Expected result : "3"

Expected behavior
Action: console.log("end of 03 week: " +moment("2022W03", "YYYY[W]WW").week());
Result: end of 03 week: 4
Expected: end of 03 week: 3
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MacOS Mojave 10.14.6
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser Chrome/76.0.3809.100
Version /76.0.3809.100

Moment-specific environment

The time zone setting of the machine the code is running on
The time and date at which the code was run
Other libraries in use (TypeScript, Immutable.js, etc)

Please run the following code in your environment and include the output:
console.log((new Date()).toString())
console.log((new Date()).toLocaleString())
console.log((new Date()).getTimezoneOffset())
console.log(navigator.userAgent)
console.log(moment.version)

Fri Aug 16 2019 12:17:07 GMT+0300 (Moscow Standard Time)
quickorder.js:47 8/16/2019, 12:17:07 PM
quickorder.js:48 -180
quickorder.js:49 Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.100 Safari/537.36
quickorder.js:50 2.24.0
Additional context
Reproducible for all years after 2020

Use GGGG with WW


Support Sequel

I use Sequel, and rspec-set can't run without ActiveRecord.

Hello @elado,
I haven't used sequel much. rspec-set requires the following API from ActiveRecord new_record?, reload, destroyed?. Do you use objects that implements some sort of Active Record or Data Mapper pattern? If they respond to such method, then it's not hard to get rspec-set to play nicely with them.
Have a look at the code, it's only 30 lines or so. It should be easy to make your own. :)

Closing this issue as it hasn't been updated for 4 months.


RuntimeError: Address already in use on 'ddp' mode pl 0.8.0

Trainer configuration:
    trainer = pl.Trainer(
        logger= CometLogger( api_key="ID"),
        auto_select_gpus=True,
        gpus=3,
        distributed_backend="ddp",
   )

The error:
GPU available: True, used: True
No environment variable for node rank defined. Set as 0.
CUDA_VISIBLE_DEVICES: [0,1,2]
CometLogger will be initialized in online mode
CometLogger will be initialized in online mode
initializing ddp: LOCAL_RANK: 0/2 WORLD_SIZE:3
Traceback (most recent call last):
  File "train.py", line 156, in <module>
    main()
  File "train.py", line 64, in main
    main_train(model_class_pointer, hyperparams, logger)
  File "train.py", line 148, in main_train
    trainer.fit(model)
  File "/user/anaconda3/envs/docBert/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 884, in fit
    self.spawn_ddp_children(model)
  File "/user/anaconda3/envs/docBert/lib/python3.7/site-packages/pytorch_lightning/trainer/distrib_data_parallel.py", line 395, in spawn_ddp_children
    self.ddp_train(local_rank, model, is_master=True)
  File "/user/anaconda3/envs/docBert/lib/python3.7/site-packages/pytorch_lightning/trainer/distrib_data_parallel.py", line 425, in ddp_train
    model.init_ddp_connection(self.proc_rank, self.world_size, self.is_slurm_managing_tasks)
  File "/user/anaconda3/envs/docBert/lib/python3.7/site-packages/pytorch_lightning/core/lightning.py", line 962, in init_ddp_connection
    torch_distrib.init_process_group(torch_backend, rank=proc_rank, world_size=world_size)
  File "/user/anaconda3/envs/docBert/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py", line 393, in init_process_group
    store, rank, world_size = next(rendezvous_iterator)
  File "/user/anaconda3/envs/docBert/lib/python3.7/site-packages/torch/distributed/rendezvous.py", line 172, in _env_rendezvous_handler
    store = TCPStore(master_addr, master_port, world_size, start_daemon, timeout)
RuntimeError: Address already in use

Env
* CUDA:
        - available:         True
        - version:           10.1
* Packages:
        - numpy:             1.18.4
        - pyTorch_debug:     False
        - pyTorch_version:   1.5.0
        - pytorch-lightning: 0.8.0-dev
        - tensorboard:       2.1.0
        - tqdm:              4.46.0
* System:
        - OS:                Linux
        - architecture:
                - 64bit
                -
        - processor:         x86_64
        - python:            3.7.7
        - version:           #97-Ubuntu SMP Wed Apr 1 03:25:46 UTC 2020

check ps -elf | grep python. maybe a previous run is occupying that process.

I'm not sure, because pkill -f train.py fixed it, but it makes sense.
I will update if it will happen again, closing.
thanks for the framework, and the responsiveness!

 kill -9 $(ps aux | grep main.py | grep -v grep | awk '{print $2}')
This solved my problem. However, if I did not use kill -9 it does not work for me.

Use torchrun instead of python solve this problem?


tracing: Optionally disable trace logging

Rename NullRecorder to LoggingRecorder and make it a noop recorder.
LoggingRecorder is used by default when no tracing endpoint is
specified and will write trace details to the logger.
On dev installs of r2 there's no tracing endpoint specified so the application logs are dominated by the traces. This change will let services (or r2) decide whether to log traces when calling configure_tracing().
👓 @ckwang8128 @spladug

:nail_care: thanks for adding this option! small documentation request

💇

missed the test failure in first pass, I think the error message there is pretty clear, just need to fix the assertion with the new recorder:
FAIL: test_null_recorder_setup (tests.unit.diagnostics.tracing_tests.TraceObserverTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/reddit/baseplate/tests/unit/diagnostics/tracing_tests.py", line 28, in test_null_recorder_setup
    self.assertEqual(type(baseplate_observer.recorder), NullRecorder)
AssertionError: <class 'baseplate.diagnostics.tracing.LoggingRecorder'> != <class 'baseplate.diagnostics.tracing.NullRecorder'>

:fish: I've filed https://reddit.atlassian.net/browse/IO-1629 to standardize a config object for tracing and then do existing client migration. Adding this for now would be a temporary stopgap that alleviates current issues with the debug logging causing pains in staging.

c1f56aeb6492f5af024062406398849bc81f380f


setup packages used by fastai course

Wondering whether any of these seem worthwhile:

Cloning into global /storage rather than /notebook

Linking instead of copying helps use git to manage configuration

Could set up a few packages used by fastai course - or maybe better to leave for learners to experience during the course

p.s. I find that I can't use !pipi -U somepkg from within a notebook.  I wonder if a script wrapper for pip and mamba in /root/.local/bin would better than alias.
or maybe something like this... (note, I haven't tried it)
https://www.quora.com/How-can-I-make-IPython-import-the-aliases-defined-in-my-bash-profile-upon-startup

Cloning into global /storage rather than /notebook

I would second this. As a user may be working in different projects, they may not want to take their /conf folder every where.

!pipi -U somepkg
I also couldn't use it in my custom installer .sh files.  e.g sh installer.sh would give errors if it uses pipi