Fix issues with the NFS broker testcase

Hi,
This PR updates the NFS broker testcase to use the presence of the NFS broker password to determine whether or not to register a service broker, instead of the username, to match existing logic in run_acceptance_tests_with_bbl_env.sh. It also updates the credhub-compatible/update-integration-config CI task to add support for an optional parameter to include the NFS broker testcase, which is needed now that the test case has been disabled by default.
#159975576
Please let me know if you have any questions.
Dave

I'm on the fence about switching from using the presence/absence of the nfs broker password to determine whether or not to run the test case to using a CI task param, so please let me know what you think.

Please hold off on merging this. I may need to make a change.

I think this is good to go now. It's passing in our CI.

I suspect that this failed because the broker is not currently being registered by the nfsbrokerpush errand (we have an open PR for this with cf-deployment that is blocked on us having DRATs test coverage), so you will need to set the NFS_CREATE_SERVICE_BROKER param to get the tests to pass. Once the PR is merged and the change has been merged to master, you will be able to remove this param again.
I'll check in with y'all tomorrow and could x-team pair if that would help.

Hi @davewalter
Thank you for this PR. CI failed because we use a separate integration_config.json in our CI than the one in the DRATS repo. We have added "nfs_create_service_broker": true and have retriggered the job.
We don't think the update-integration-config task needs to handle INCLUDE_NFS_BROKER_TESTCASE and NFS_CREATE_SERVICE_BROKER. Instead the corresponding flags should be set in the integration config passed to the task via INTEGRATION_CONFIG_FILE_PATH. The main responsibility of the update-integration-config task is to populate credentials. Please could you remove those env vars from the task?
Also, we have issued a deprecation notice for the use of env vars as a means to configure DRATS in favour of using integration config only. So please could you remove support for NFS_CREATE_SERVICE_BROKER from the drats CI task and env.go?
Best wishes
Mirah and @jamesjoshuahill

Hi @MirahImage

We don't think the update-integration-config task needs to handle INCLUDE_NFS_BROKER_TESTCASE and NFS_CREATE_SERVICE_BROKER. Instead the corresponding flags should be set in the integration config passed to the task via INTEGRATION_CONFIG_FILE_PATH. The main responsibility of the update-integration-config task is to populate credentials. Please could you remove those env vars from the task?

Fair enough. The name of the update-integration-config task made it seem like it was acceptable for it to do more than just injecting credentials. I was also attempting to clean up existing logic in that task that set the include-cf_nfsbroker flag based on the presence/absence of an NFS broker password. I have instead removed that logic completely.
I have been trying to use as much of the setup/tooling in the DRATs repo as possible to reduce the tooling that we had to maintain, but perhaps that isn't the correct approach. Do you have any plans to offer a generic task that will allow users to set flags in the integration-config.json, or are the CI tasks in DRATs not intended for external use? Also, given that you don't use the integration-config.json in DRATs in your own CI, perhaps it should be deleted in favour of the example in the README? That would eliminate the temptation for people to try to use it.
At this point, the only two places where the INCLUDE_NFS_BROKER_TESTCASE and NFS_CREATE_SERVICE_BROKER environment variables are mentioned is in the run_acceptance_tests_with_bbl_env.sh and the README. Given that it sounds like you are also deprecating the scripts that use/manipulate environment variables in favour of run_acceptance_tests_local_with_config.sh, should I remove them completely?

Also, we have issued a deprecation notice for the use of env vars as a means to configure DRATS in favour of using integration config only. So please could you remove support for NFS_CREATE_SERVICE_BROKER from the drats CI task and env.go?

Sounds good.
Let me know if you have more questions/concerns.
Regards,
Dave

Thank you @davewalter!
We agree that ci/integration-config.json should be deleted to avoid confusion in favour of the copy in README.
When we deprecate environment variables we'll review the scripts as well so that they all use integration config.
Best wishes
Mirah and @jamesjoshuahill


Update Azure CLI

Tool information

Tool name: Azure CLI
Add or update? Update
Desired version: 2.24.1
Azure CLI 2.24 that is used by the win2016 images from 20210525.0 causes a bug where webapp config appsettings set command causing all values to default to "false". According to their release note, v 2.24.1 has a hotfix to fix this issue https://docs.microsoft.com/en-us/cli/azure/release-notes-azure-cli?tabs=azure-cli#june-01-2021

Area for Triage:
Scripting and command line
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[x] Windows Server 2016 R2
[ ] Windows Server 2019

Hi @adeliab!
I'm going to close this as a duplicate of https://github.com/actions/virtual-environments/issues/3508
Please feel free to contact us if you have any concerns.
Thank you!


Issues running local model on Linux

Running it on kali linux and im having an issue utilizing the local model. the check box is tick on but when i click install nothing happens... it would be nice to have an acknowledgment or a progress bar...

It appears there are currently issues with the webLLM implementation when using the neurite.network host.
I will look into that, for now, you should be able to get it working from a cloned version of the repo.
The next update should allow for any ai backend to be used more easily than before, so being limited to webLLM should not be an issue very soon.
If you are using the neurite.network host, you should at least see a red x when trying to install the local ai. (I will look into getting it to work on the github pages host again as well, potentially relates to the new domain name?)
Otherwise,, when running Neurite locally with the right build setup as given in the readme, you should see the below circular loading icon that slowly fills up as the parameters install.

I am not using Linux, but it could be useful to see if you have any error logs, or what your environment is/ what browser you used.

I have decided to remove webLLM in favor of Ollama. The next update will reflect that change.


[Bug]:   Mini breaks not working with long breaks

Version

[X] I'm using version 1.7.0

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

What operating system are you using?
Linux
Operating System Version
Linux box 5.10.53-1-MANJARO #1 SMP PREEMPT Mon Jul 26 07:18:28 UTC 2021 x86_64 GNU/Linux
Reproduction steps

Wait for 3 mini breaks to pass
It resets and goes back to mini break 1
...
Reproduces how often: 100% (unless i manually skip breaks)

Expected Behavior
Long breaks to start after x mini breaks
Actual Behavior
Long breaks never start unless manually skipping through them
Relevant log output
[2021-07-31 06:59:03.401] [info]  Stretchly: initializing...
[2021-07-31 06:59:03.409] [info]  Stretchly: loading preferences
[2021-07-31 06:59:03.419] [info]  Stretchly: starting Idle time monitoring
[2021-07-31 06:59:03.420] [info]  Stretchly: starting Do Not Disturb monitoring
[2021-07-31 07:28:43.322] [info]  Stretchly: showing Mini Break notification
[2021-07-31 07:28:53.597] [info]  Stretchly: showing window 1 of 2
[2021-07-31 07:28:53.598] [info]  Stretchly: starting Mini Break
[2021-07-31 07:28:53.625] [info]  Stretchly: showing window 2 of 2
[2021-07-31 07:28:55.937] [info]  Stretchly: postponing Mini Break
[2021-07-31 07:30:45.938] [info]  Stretchly: showing Mini Break notification
[2021-07-31 07:30:56.212] [info]  Stretchly: showing window 2 of 2
[2021-07-31 07:30:56.220] [info]  Stretchly: showing window 1 of 2
[2021-07-31 07:30:56.220] [info]  Stretchly: starting Mini Break
[2021-07-31 07:35:56.242] [info]  Stretchly: finishing Mini Break
[2021-07-31 07:35:56.304] [info]  Stretchly: pausing breaks because of idle time
[2021-07-31 07:36:02.313] [info]  Stretchly: resuming breaks after idle time
[2021-07-31 08:05:52.314] [info]  Stretchly: showing Mini Break notification
[2021-07-31 08:06:02.589] [info]  Stretchly: showing window 1 of 2
[2021-07-31 08:06:02.590] [info]  Stretchly: starting Mini Break
[2021-07-31 08:06:02.603] [info]  Stretchly: showing window 2 of 2
[2021-07-31 08:11:02.609] [info]  Stretchly: finishing Mini Break
[2021-07-31 08:11:04.000] [info]  Stretchly: pausing breaks because of idle time
[2021-07-31 08:11:08.004] [info]  Stretchly: resuming breaks after idle time
[2021-07-31 08:16:12.260] [info]  Stretchly: pausing breaks because of idle time
[2021-07-31 08:16:29.275] [info]  Stretchly: resuming breaks after idle time
[2021-07-31 08:46:19.278] [info]  Stretchly: showing Mini Break notification
[2021-07-31 08:46:29.552] [info]  Stretchly: showing window 1 of 2
[2021-07-31 08:46:29.553] [info]  Stretchly: starting Mini Break
[2021-07-31 08:46:29.579] [info]  Stretchly: showing window 2 of 2
[2021-07-31 08:46:31.016] [info]  Stretchly: postponing Mini Break
[2021-07-31 08:48:21.018] [info]  Stretchly: showing Mini Break notification
[2021-07-31 08:48:31.323] [info]  Stretchly: showing window 2 of 2
[2021-07-31 08:48:31.336] [info]  Stretchly: showing window 1 of 2
[2021-07-31 08:48:31.337] [info]  Stretchly: starting Mini Break
[2021-07-31 08:53:31.355] [info]  Stretchly: finishing Mini Break
[2021-07-31 08:53:32.096] [info]  Stretchly: pausing breaks because of idle time
[2021-07-31 08:53:37.100] [info]  Stretchly: resuming breaks after idle time

Preferences
{
	"microbreakDuration": 600000,
	"microbreakInterval": 2400000,
	"breakDuration": 1200000,
	"breakInterval": 3,
	"breakNotification": true,
	"microbreakNotification": true,
	"breakNotificationInterval": 30000,
	"microbreakNotificationInterval": 10000,
	"microbreak": true,
	"break": true,
	"microbreakStrictMode": false,
	"breakStrictMode": false,
	"morningHour": 6,
	"microbreakPostpone": true,
	"breakPostpone": true,
	"microbreakPostponeTime": 120000,
	"breakPostponeTime": 300000,
	"microbreakPostponesLimit": 1,
	"microbreakPostponableDurationPercent": 30,
	"breakPostponesLimit": 1,
	"breakPostponableDurationPercent": 30,
	"mainColor": "#ffffff",
	"transparentMode": true,
	"opacity": 0.9,
	"audio": "crystal-glass",
	"volume": 1,
	"fullscreen": true,
	"ideas": false,
	"naturalBreaks": true,
	"naturalBreaksInactivityResetTime": 300000,
	"allScreens": true,
	"useIdeasFromSettings": false,
	"language": "en",
	"notifyNewVersion": true,
	"isFirstRun": false,
	"posLatitude": 0,
	"posLongitude": 0,
	"useMonochromeTrayIcon": false,
	"useMonochromeInvertedTrayIcon": false,
	"silentNotifications": false,
	"monitorDnd": true,
	"microbreakStartSoundPlaying": false,
	"breakStartSoundPlaying": false,
	"themeSource": "system",
	"endBreakShortcut": "CmdOrCtrl+X",
	"breakWindowWidth": 0.85,
	"breakWindowHeight": 0.85,
	"checkNewVersion": true,
	"breakIdeas": [
		{
			"data": [
				"Not alone",
				"Do you find it hard to take a break alone? Try to do it with a co-worker. Aside from making it easier to stick to breaks, you will have a chance to get to know them better. Taking breaks together increases productivity."
			],
			"enabled": true
		},
		{
			"data": [
				"Step away",
				"Do you ever notice how your brain can figure things out by itself? All it takes is to step away from the computer and take a break to think about something totally unrelated."
			],
			"enabled": true
		},
		{
			"data": [
				"Microbreaks",
				"Rest is a key component in ensuring the performance of the musculoskeletal system. Frequent breaks can decrease the duration of a task and help lower the exposure to ergonomic injury risk."
			],
			"enabled": true
		},
		{
			"data": [
				"Meditation",
				"Research studies suggest that mindfulness-based exercises help decrease anxiety, depression, stress, and pain, and help improve general health, mental health, and quality of life. Not sure how to start? There are numerous apps to help you out."
			],
			"enabled": true
		},
		{
			"data": [
				"Blink",
				"Looking at screens for a long time causes you to blink less, thus exposing your eyes to the air. Blink rapidly for a few seconds to refresh the tear film and clear dust from the eye surface."
			],
			"enabled": true
		},
		{
			"data": [
				"Ergonomics",
				"Improper height and angle of the keyboard, mouse, monitor or working surface can cause health problems. Take some time to read about desk ergonomics."
			],
			"enabled": true
		},
		{
			"data": [
				"Move",
				"There are a lot of ways you can exercise within your office. Try marching in place or doing desk push-ups."
			],
			"enabled": true
		},
		{
			"data": [
				"Change",
				"Do you have a stability ball or standing work desk?
			],
			"enabled": true
		},
		{
			"data": [
				"Notice",
				"Are you daydreaming or having trouble focusing? It is a sign that you need to take a break."
			],
			"enabled": true
		},
		{
			"data": [
				"Tech",
				"How about taking a no-tech walk?"
			],
			"enabled": true
		},
		{
			"data": [
				"Metabolism",
				"Emerging research shows that sitting for long periods of time contributes to risk of metabolic syndrome, heart attack and stroke risk and overall death risk, among others. Taking regular walking breaks can help your circulation, working to counteract some of those problems."
			],
			"enabled": true
		},
		{
			"data": [
				"Active Meetings",
				"How about moving meetings from the conference room to the concourse? Walking not only burns calories but it may even foster a sense of collaboration."
			],
			"enabled": true
		},
		{
			"data": [
				"Fruit",
				"Take your time and eat some fruit. Slowly. Notice the flavor, the texture, the freshness."
			],
			"enabled": true
		},
		{
			"data": [
				"Bathrooms",
				"Walk to the farthest bathroom in the worksite facility when going to the restroom."
			],
			"enabled": true
		},
		{
			"data": [
				"Coffee break",
				"Going on coffee break? Consider doing a 5-minute walk every time you go for one."
			],
			"enabled": true
		},
		{
			"data": [
				"Colleagues",
				"Do not email or message office colleagues, walk to their desks to communicate with them."
			],
			"enabled": true
		},
		{
			"data": [
				"Learning",
				"In a study of healthy volunteers, NIH researchers found that taking short breaks, early and often, may help our brains learn new skills."
			],
			"enabled": true
		},
		{
			"data": [
				"Exercise",
				"Evidence suggests small amounts of regular exercise can bring dramatic health benefits, including measurably reducing stress."
			],
			"enabled": true
		},
		{
			"data": [
				"Repeat",
				"Have you found your stretch-ly-routine? Do not forget to repeat it for more than once to better fight effects of prolonged sitting."
			],
			"enabled": true
		},
		{
			"data": [
				"Wrist and forearm",
				"Extend your arms with the palms facing towards you, then slowly rotate the hands four times clockwise, then four times counter-clockwise."

			],
			"enabled": true
		},
		{
			"data": [
				"Mobilize",
				"For every thirty minutes of stagnation, you should have at least one minute of stimulation."
			],
			"enabled": true
		},
		{
			"data": [
				"7 Minute Workout",
				"This workout packs in a full-body exercise routine in a fraction of the time. But as with any exercise, be careful. There are numerous apps to get you started."
			],
			"enabled": true
		},
		{
			"data": [
				"Pulse",
				" Raise your pulse rate to 120 beats per minute for 20 straight minutes four or five times a week doing anything you enjoy. Regularly raising your heart rate results in improved cardiovascular health."
			],
			"enabled": true
		},
		{
			"data": [
				"Take the stairs ",
				"Studies have shown that stair climbing, which is considered vigorous-intensity physical activity, burns more calories per minute than jogging."
			],
			"enabled": true
		},
		{
			"data": [
				"Make art",
				"Art therapy is known to have great mental health benefits, especially when it comes to stress management. How about writing a quick poem, taking a picture or painting something small?"
			],
			"enabled": true
		},
		{
			"data": [
				"Declutter",
				"A clean space helps your focus at work and is often linked to positive emotions like happiness."
			],
			"enabled": true
		},
		{
			"data": [
				"Lunch outside",
				"Nature is linked to positive emotions and decreased stress and anxiety. Whenever possible, try to take your daily lunch break outside, surrounded by some greenery."
			],
			"enabled": true
		},
		{
			"data": [
				"Public transport",
				"If you use public transport regularly, you can stand instead of sitting. If it is possible, try to replace as many of your daily trips as possible with walking or cycling."
			],
			"enabled": true
		},
		{
			"data": [
				"Yawning",
				"Yawning can be really helpful, as it produces tears to help moisten and lubricate the eyes."
			],
			"enabled": true
		},
		{
			"data": [
				"Focus change",
				"Hold one finger close to the eye and focus on it. Slowly move the finger away, focus far into the distance and then back to the finger. Bring the finger back and focus on something far away."
			],
			"enabled": true
		},
		{
			"data": [
				"Palming",
				"While seated, brace elbows on the desk and close to the desk edge. Let your weight fall forward and cup hands over eyes. Close your eyes and inhale slowly through nose and hold for few seconds. Continue deep breathing."
			],
			"enabled": true
		},
		{
			"data": [
				"Hand squeezes",
				"Squeeze a pair of balled-up socks or a soft rubber ball, hold for 5 seconds. Repeat whole process few times."
			],
			"enabled": true
		},
		{
			"data": [
				"Slow Breathing",
				"Emerging research suggests potential for use of controlled slow breathing techniques as a means of optimising physiological parameters that appear to be associated with health and longevity."
			 Roll your shoulders forward for 10 seconds, then backward.",
			"enabled": true
		},
		{
			"data": "Raise your right arm, stretch it over your head to the left, and hold for 10 seconds. Repeat on the other side.",
			"enabled": true
		},
		{
			"data": "With your right hand, grab each finger of your left hand in turn and squeeze. Repeat on the other side.",
			"enabled": true
		},
		{
			"data": "Stand up and do a lunge. Hold for 10 seconds, then do the other leg.",
			"enabled": true
		},
		{
			"data": "Close your eyes and simply notice whatever arises in current moment, without judgement.",
			"enabled": true
		},
		{
			"data": "Focus every 20 minutes for 20 seconds on an object at 20 feet distance.",
			"enabled": true
		},
		{
			"data": "If you need help, ask for it.",
			"enabled": true
		},
		{
			"data": "Do one thing at a time.",
			"enabled": true
		},
		{
			"data": "Is your attention spent wisely?",
			"enabled": true
		},
		{
			"data": "Change your sitting posture.",
			"enabled": true
		},
		{
			"data": "Expose your eyes to natural light.",
			"enabled": true
		},
		{
			"data": "With your eyes closed, slowly and gently raise your eyes to the ceiling and back down to the floor.",
			"enabled": true
		},
		{
			"data": "With your eyes closed, slowly and gently move your eyes to the left, then slowly to the right.",
			"enabled": true
		},
		{
			"data": "Shake your hands out to get some relief.",
			"enabled": true
		},
		{
			"data": "One at a time, touch the tip of each finger to the tip of your thumb so they make an O-shape.",
			"enabled": true
		},
		{
			"data": "Make a fist and then slide your fingers up until they point toward the ceiling, like you're telling someone to stop.",
			"enabled": true
		},
		{
			"data": "Make a fist and then fan your fingers out and stretch them as far as you can.",
			"enabled": true
		},
		{
			"data": "Sit tall, arms hanging at your sides, and slowly rotate neck in one direction in a large circle.",
			"enabled": true
		},
		{
			"data": "Stand tall and slowly tilt your head toward the shoulder using the same hand until you feel a strech.",
			"enabled": true
		},
		{
			"data": "Stand tall with your arms by your side. Squeeze your solder blades together and hold.",
			"enabled": true
		},
		{
			"data": "Stand tall with your arms raised along your shoulders. Move hands slightly behind your back, hold for a second and return.",
			"enabled": true
		}
	],
	"showBreaksAsRegularWindows": false,
	"appExclusions": [],
	"pauseForSuspendOrLock": true,
	"pauseBreaksShortcut": "",
	"resumeBreaksShortcut": "",
	"screen": "primary"
}

Additional information
No response
Code of Conduct

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

Hi @stilldg ,
your breaks get reset because of idle time:
[2021-07-31 07:30:56.220] [info]  Stretchly: starting Mini Break
[2021-07-31 07:35:56.242] [info]  Stretchly: finishing Mini Break
[2021-07-31 07:35:56.304] [info]  Stretchly: pausing breaks because of idle time
[2021-07-31 07:36:02.313] [info]  Stretchly: resuming breaks after idle time

You can disable this behavior in Preferences: Monitor system idle time or change the time after which breaks are reset in idle time: https://github.com/hovancik/stretchly#natural-breaks-inactivity-time-

Hi,
I assumed that it would pause on the current break, and as soon as it detects some action then it resumes in place. I think that this would make a lot more sense, since usually I leave my computer for longer than the break time, so would you ever consider adding an option for this in the future?
Thanks.

Hi, what do you mean by "it"? Idle time detection? Or something else?

To help with your issue, I think you could set "naturalBreaksInactivityResetTime": 300000, to let's say 600000 which is 10 minutes and idle time would reset breaks only after 10 minutes of idle time.

By "it" I was indeed referring to the idle time detection. So for example, if I am away from my computer for say 5 minutes, I would like to pause the current timer in place, and then when I return to my computer it would start the timer again right where it left off, since otherwise I would rarely reach the long breaks as sometimes I am away from my computer for some time, so it would be a nice feature to have.
I understand this isn't may not be priority/desire for you to implement, and if that is the case then I would be willing to create a fork and attempt to add this, and then if it works well you can consider merging it. There's also some issues I have noticed in KDE Plasma where I had to add the following to my window settings: https://i.imgur.com/1INJb68.png or else it would show white background with barely visible text. None of the themes work but that is how I fixed the issue to get the text to display black on a white background. Here's some examples:
Before: https://i.imgur.com/SY7vl8E.png
After: https://i.imgur.com/1vK4Epp.png
I have to use the Snow white theme to get the text to display with this method, but I would like to see if there's anything I can do to fix this as I prefer dark themes.
By the way, thank you for creating this. I will be donating soon as this project has done a lot for my productivity :) -- let me know what you think about my proposed idea + fixes.

Hey, sorry for late reply, life gets in a way sometimes.
Some time ago, I tried to implement what you describe: when there is idle time, remember where you left of and continue there, but I had some issues. Don't remember what exactly. Stretchly is complicated. I think there are some issue where people asked for this functionality. You can try do it yourself :)
With white texts, there was issue in 1.8, that was fixed in 1.8.1, but not sure what version you are running. There are still some issues where people get empty break windows, but I did not manage to reproduce it.


[Feature Request] Cascader does not support multiple cascade selector

Existing Component
是
Component Name
Cascader
Description
如题

We have no plan to support this.


Add options to prevent download if sponsorblock entry not found

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Hi,
A great feature to add would be the ability to prevent a download if sponsorblock info is not found. In my setup, I check a channels a few times a day ( and don’t download videos younger than a day old) and the sponsor block info isn’t updated in time.
Given at the moment it seems sponsorblock is checked after download, if the sponsorblock check finds nothing it could delete the file. This obviously isn’t the most efficient so the check could come before the download. Not familiar with the codebase so don’t know how hard this would be.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
N/A
No response

I believe it can already be done with --match-filter. Pls provide examples I can test commands on

Here's the long-awaited example URL: https://www.youtube.com/watch?v=nGIpdiQrFDU
yt-dlp does indeed fetch the sponsorblock data after --match-filter is checked against, so it is not currently possible to do this in a single run of yt-dlp.
There is a workaround, however; you could do something like this:
yt-dlp --write-info-json --skip-download --sponsorblock-mark all -o "sponsorblock_check.%(ext)s" "https://www.youtube.com/watch?v=nGIpdiQrFDU" && yt-dlp --load-info-json "sponsorblock_check.info.json" --match-filter sponsorblock_chapters

Any additional options for downloading could be passed to the second invocation of yt-dlp (so anywhere after --match-filter sponsorblock_chapters), e.g. if you want to use --sponsorblock-mark all or --sponsorblock-remove all, you would add them there.
Note that you do not need to pass the URL to the second invocation; all data is being extracted from the info json written by the first yt-dlp call.

This is a valid feature request and a PR would be welcome, but this is not a priority for the core devs


[MLS Spec change] Clarify rules around Add proposals. (#705)

Link to the exact changes
https://github.com/mlswg/mls-protocol/pull/705
Description of the changes

Closes #700

Can you assign this to me?

Sorry for the long time to implement this. I was sick in the previous weeks and couldn't focus on it.


Added getAssignee

Hi! I added the getAssignee( ) function in the ChangeApiRestClient class.
While I was working on it, I did some small refactoring as well. I did not open a separate PR for those since they are quite small changes. They are the following:

Added blank space between comma and parameter in DraftApiRestClientTest.
Removed unused import in HashtagParserTest.
Moved the method getHashtags() in ChangeApiRestClient to follow the order of the implemented interface ChangeApi.

Thank you @EFregnan!


UI Adjustments: Spacing of Manage ownership

This will create more consistent spacing and better visual grouping.

Updates merged.


Update docs

I only found out that there's a callback associated to send because of an unaccepted PR to make send promisified. Any docs around the promises or callbacks would really help new users.

Yeah, we're in 2017 not in 90's: world is async/await'd now.

There are PRs from 2015 to update the docs — broken links. I think this repo is dead.


Not bypassing Smart Screen

Even though it does a great job but Windows Smart Screen and UAC still tell that app is coming from unknown publisher so AV already that the sign is fake?

This was never advertised to bypass Smart Screen or UAC.


Touch support for 0.2.5

Kudos to @michaelguild13 for the touch-supported fork.

@nyurik fixed a handful, appears the tab changes are misleading.

Includes https://github.com/Leaflet/Leaflet.draw/issues/473


Zoom/grid select broken in color correction

In color correction mode, when a grid is selected and a user zooms in and clicks on the grid, the selection should be cleared and it is not.

Closed by #906


Plugin Manager various fixes, leak, swing thread issues...

Hello Andrei,
Find in this PR some fixes to Plugin Manager and by the way congratulations for your idea ! :

Resources leaks
Issues in access to AWT managed objects
some code cleanup

Regards
Philippe M.
@philmdot

Current coverage is 59.94% (diff: 30.00%)

Merging #122 into master will decrease coverage by 0.05%

@@             master       #122   diff @@
==========================================
  Files           334        334
  Lines         20512      20527    +15
  Methods           0          0
  Messages          0          0
  Branches       2320       2329     +9
==========================================
- Hits          12306      12304     -2
- Misses         7362       7374    +12
- Partials        844        849     +5

Powered by Codecov. Last update 930d10b...6fb61de

Thanks for contribution! I'm busy now on business trip and will review this a bit later.

Thanks for contributing!


Potential energy to be used

Is this the the potential energy function for the Na4Cl4 cluster that we are going to use for it optimization?

Yes, you are right

Correct.


feat(go-ipfs): Integrating go-ipfs

:tada: This PR is included in version 1.4.0 :tada:
Your semantic-release bot :package::rocket:


Add Agent KeyManager template

This is the first of many templates that I plan on introducing to the repository to give plugin authors a jumping off point.
There is going to be a lot of repetition between the templates but I thought trying to do some sort of code sharing would confuse things.
There is no KeyManager specific guidance in here. The key manager proto definition and documentation is the canonical source of the interface behavior.
TODO's have been added at each point of consideration.

This is a great first pass. How do folks find these templates? Do they always reflect the "latest" version of the interface?
In the future, I think we can go a few steps further with boilerplate. For example, defining the logger and setting it up via the hook. I think in general, the goal should be that someone can copy/paste this file, and have the absolute minimum number of lines to edit (i.e. business logic only) with as few distraction as possible. I think we are mostly there.

For example, defining the logger and setting it up via the hook.
That is super easy to add now

How do folks find these templates? Do they always reflect the "latest" version of the interface?
I have a table defined with links to both the protos and the template for each plugin type. I think the template should always reflect the most recent version, so we'll want to refresh the template whenever we bump the version, IMO.

SGTM!


requests LE certificate and will refresh every 10 days ..

Hi

script/entrypoint.sh requests LE certificate and will refresh every 10 days in case if certificate is close to expiration (30day)

For the test, I launched the server on September 9, the certificate has not yet been updated ... what could be wrong?

It checks a certificate expiration date once in 10 days, and renews it if less than 30 days left till it's expiration.
What is behavior you expect to see and what you are observing in reality?

Ouch ... did not understand the documentation correctly, thanks


executor docs refresh

No code changes.
Removed duplication between module docs and readme.md. Apparently this is the 'right' way to do it these days.
Please dive in with any suggestions on how we can re-word the docs to make anything clearer.
(And also if I've misunderstood anything and am twisting the original docs meaning)

Great build error:
error[E0658]: arbitrary expressions in key-value attributes are unstable
Except they're not:
https://github.com/rust-lang/rust/pull/83366
Hmm and that was a while ago....


Segfault if command does not write to STDOUT, OK if command does write to STDOUT

The rule of silence, also referred to as the silence is golden rule, is an important part of the Unix philosophy that states that when a program has nothing surprising, interesting or useful to say, it should say nothing. source

Currently the command utility for this compiler has a quite unfortunate bug where not writing to STDOUT results in a segfault. Writing anything from anywhere in the program stops the crash. This is probably something related to invalid memory references or stale data, but there are a few things that anyone could do to help diagnose this bug:

Try running make deploy on your Linux distribution

This bug is currently reliably occurring on my Windows Subsystem for Linux installation of Ubuntu 20.04.2 LTS.
Running make deploy will compile the compiler twice and check to see that the result is stable.
On my system the crash results in a SEGFAULT during code generation.

Does this bug remind you of anything familiar?

The compiler currently depends on x86-64 Linux system calls sys_open, sys_close, sys_read, sys_write, sys_exit, sys_brk. and nothing else. There are zero dynamically loaded libraries such as libc etc.

Are you a debugging pro?

This is an interesting opportunity to improve the debugging tools for LM. A sealed release has been created to reproduce this bug at any point in the future.

Are there any debugging tools that you think might help diagnose this bug?
Are there any debugging symbols etc. that might be helpful?
What can we do for this project to improve the debugging experience?

Coincidentally fixed by the large return value patch.


Feedback for “Account Balance History”

Transactions after 2024-10-31 are not loading.
Expected behavior:

All transactions should be shown, most recent first.

Actual behavior:

Only txs before 2024-11-01 are retrieved
No tx are shown for November

NOTE: The endpoint does retrieve values, so it's not an auth problem.

Anybody?

@ericm301
If fromDateTime and toDateTime are not provided, the default is set to retrieve data for a one-month period from the current date. Adding specific values for these parameters will allow you to customize the date range as needed.

 Sometimes, it gives me 11-01, sometimes 10-31, sometimes earlier. It does seem that successive requests tend to older results.
I am not adding any query parameters to the avove requests. This, as mentioned above, should return the most recent 20 balance entries. I have about 15 transactions on this address since 2024-11-01, each of which should have an entry in the responses.
If I add fromDateTime=2024-11-01&toDateTime=2024-11-10 to my request, I only get one result, for 2024-11-01.

@ericm301
We are currently operating with physically distributed databases and have encountered a data inconsistency issue. We are actively working to resolve this synchronization bug. Thank you for notifying us of this matter.

The data seems to be slowly catching up; it's at 2024-11-14 right now.

@ong-ar

We encountered some issues while updating the asset data structure.
We're currently working to catch up on the data, and it will be updated soon.
However, historical data from 11/14 to today may be lost.
Thanks.


Improved pattern

Setting the daemon to wings improves matching performance and not forcing the start of the log message allows for more reliable matches (the previous regex never matched for me when using systemd as a log source.

Hello, thanks for pulling.
But I don't understand the goal of the instruction journal?
Can you explain to me plz?

And I think there is an error because you use the _daemon and you add an extra s.
So the result will be wings.service

And I think there is an error because you use the _daemon and you add an extra s. So the result will be wingss.service

Yeah that is causing a error for me it doesnt work but when i use the syslog or wings log directly it works.


Invalid format for X-Auth Key header

I have installed the exporter via helmfile.
helmfile apply
With content:
repositories:
  - name: "cloudflare-exporter"
    url: "https://lablabs.github.io/cloudflare-exporter"

releases:
  - name: "cloudflare-exporter"
    namespace: "monitoring"
    version: "0.0.3"
    chart: "cloudflare-exporter/cloudflare-exporter"
    wait: true
    values:
      - env:
        - name: CF_API_KEY
          value: R**
        - name: CF_API_EMAIL
          value: ma*
        # Optionally select zones
        # - name: CF_ZONES
        #   value: "<zone_id1>,<zone_id2>,..."
        # DEPRICATED Optionally, you can filter zones by their adding IDs following the example below.
        # - name: ZONE_XYZ
        #   value: <zone_id>

The pods are crashing due to:
time="2021-10-16 15:39:37" level=info msg="Beginning to serve on port :8080"
time="2021-10-16 15:39:38" level=fatal msg="error from makeRequest: HTTP status 400: content \"{\\\"success\\\":false,\\\"errors\\\":[{\\\"code\\\":6003,\\\"message\\\":\\\"Invalid request headers\\\",\\\"error_chain\\\":[{\\\"code\\\":6103,\\\"message\\\":\\\"Invalid format for X-Auth-Key header\\\"}]}],\\\"messages\\\":[],\\\"result\\\":null}\""

The API key has read access to any resource in my CF account.  THis is where I got it from:

Do you know anything about this particular issue?

Remove the key and email variables and set it as CF_API_TOKEN  instead.

This is what I have tried:
time="2021-10-16 15:34:53" level=info msg="Beginning to serve on port :8080"
time="2021-10-16 15:34:53" level=fatal msg="invalid credentials: key & email must not be empty"

I bumped it to the last version and now it seems working despite the fact that I'm only receiving metrics for Cloudflare workers.
I can see the following error:
time="2021-10-17 10:22:59" level=error msg="graphql: zone '80bd8825592fdf24cf1cf7dbe27f6358' does not have access to the path"

Same issue here

I bumped it to the last version and now it seems working despite the fact that I'm only receiving metrics for Cloudflare workers.
I can see the following error: time="2021-10-17 10:22:59" level=error msg="graphql: zone '80bd8825592fdf24cf1cf7dbe27f6358' does not have access to the path"

@darox you get this error for each zone that is not on enterprise. If you're not paying for enterprise at all, then simply set FREE_TIER=true.


Mark current release as old upon new release

We should deprecate the current Conda package whenever we release a new version. Can we automate this?

Should be fixed by newest commit. We will see next release


fix(p2wdb): Trying this without _this

:tada: This PR is included in version 1.10.4 :tada:
Your semantic-release bot :package::rocket:


Update xy-grid.md

Minor style edit

Hi @rbrtmrtn. From https://github.com/zurb/foundation-sites/pull/11175 all pull requests must be opened on develop, even for changes documentation.


Build Error with AFNetworking 3.0.x

With AFNetworking ver 3.0.4,
this library raises build error that
'AFHTTPRequestOperationManager.h' is not found
in SRGVersionUpdater.m:
#import "AFHTTPRequestOperationManager.h"

This is because of AFNetworking changes:
https://github.com/AFNetworking/AFNetworking/wiki/AFNetworking-3.0-Migration-Guide
Classes Removed

The following classes have been removed from AFNetworking 3.0:

AFURLConnectionOperation
AFHTTPRequestOperation
AFHTTPRequestOperationManager

Hello,
Have fixed this in PR #5
Thanks.

thanks a lot @vijaytholpadi :smile:  i merged #5.


Fix custom back button props & visibility

I have to have option to add custom styled back button.
So I want to provide a React element styled button.
I need navigation props to be passed to component, that I can call this.props.navigatios.goBack()
I also don't want to handle visibility of button on my own.
If I'm in root it should return null, but that functionality is implemented below custom renderer.
For testing you should build custom back button.
e.g.
  <TouchableOpacity onPress={() => { his.props.navigation.goBack(null) }} style={styles.row}>         <View style={styles.backCircle}> <Icon             name='ios-arrow-dropleft-circle-outline'             size={20}             color="#112233"           />         </View>         <Text>{'back'}</Text>       </TouchableOpacity>
Then you can use it e.g.
headerLeft: props => <BackButtonContainer {...props} />
Test when it is showing, test on press.

no, this was changed just now.
the way you changed it, it will not allow for a left button on the 0-index screen.
this pr should be not accepted.
actually, i will push a new pr today, that adds a headerBack. if you use headerLeft, its an element, does whatever you want. if you use headerBack, its an element that is still a back button. if you use headerBackTitle and headerBackTitleStyle is a normal os back button, and you only control the text and the style of it.
look forward for it later today.

It's not enaught to change style and text
My client wants

So i need to do it like that

yes. thats what im writing now. you will do
headerBack: (
<View>
  <View style={styles.backCircle}>
    <Icon name='ios-arrow-dropleft-circle-outline' size={20} color="#112233" />
  </View>
  <Text>{'back'}</Text>
</View>
)

Can you please handle somehow 0-index hide?
You definately know best solution. Thanks a lot.

yes. the back button will only appear on non-zero indexes. like a back button should :D

#1492 <- and here is it

HeaderLeft is not necessarily the back button, so this change will remove the ability for custom headerLeft on pages where the screen index is zero.

https://github.com/react-community/react-navigation/pull/1538 here's another PR that might help

#2712 is a similar PR

A PR which allows to customize the back button behaviour was recently merged: #2712
Since the mentioned PR covers the use cases mentioned here, I will close this PR as being redundant. With that being said, thank you for your efforts, @Misioka !


navigation2: 0.4.1-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository navigation2 to 0.4.1-1:

upstream repository: https://github.com/ros-planning/navigation2.git
release repository: https://github.com/SteveMacenski/navigation2-release.git
distro file: foxy/distribution.yaml
bloom version: 0.9.7
previous version for package: 0.4.0-1

Updates a dependency for nav2_util to build to unblock upstream packages binary runs


[apptools] Add nodeunit tests for apptools

Failure analysis:
https://crosswalk-project.org/jira/browse/XWALK-4695
Impacted tests(approved): new 29, update 0, delete 0
Unit test platform: [Android]
Unit test result summary: pass 28, fail 1, block 0

LGTM


Update Zim Speed link and Zim response

FYI, I've also posted the same summary to an announcement here.


AWS Serverless-Integration corrupts assets

Environment
Deploying an application with nuxt version 3.0.0-rc.6 as a lambda function, integration in an AWS REST-API (not HTTP API).
Reproduction

Deploy an application with AWS lambda function serving a PNG file as an asset.
Set it up in an AWS Rest API (Api Gateway v1)
Set up binaryTypes with */*.
Access the asset via a browser.

Describe the bug
Accessing any asset (e.g. an PNG file) causes broken data to be returned. I could track down the reason for this, and ended up in nitropack (https://github.com/unjs/nitro/blob/main/src/runtime/entries/aws-lambda.ts).

The nitroApp returns an utf8 encoded string of the assets (which is not reasonable in this case), and not a base64 encoded one. Further, the return value of the method should have included isBase64Encoded: true, when an asset is returned.
If I alter the code by hand (in a deployed lambda function) the assets are returned correctly.

I have a hard time finding flaws in my setup, when I cannot figure out, how the nitropack code would work properly for assets in the first place.
Additional context
No response
Logs
No response

This should be resolved upstream. If not, please do open a new issue there.


Add behavior for enemies to periodically spawn blasts

Certain enemies should periodically spawn blasts. This should be done through the behavior system.

#15


AP-2406 application detail report

Add new columns to the applications detail report for MP
Link to story

Added new columns
Converted tests requiring multiple proceedings to use the new factory traits
Added scope to ApplicationProceedingType to order records in created_at sequence
Added rake task mp:pop to turn on multiple proceedings and populate section 8 proceeding types

Checklist
Before you ask people to review this PR:

[ ] Tests and rubocop should be passing: bundle exec rake
[ ] Github should not be reporting conflicts; you should have recently run git rebase master.
[ ] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[ ] The PR description should say what you changed and why, with a link to the JIRA story.
[ ] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2723 (f246225) into master (0cdb001) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #2723   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          532       532
  Lines        11100     11130   +30
=========================================
+ Hits         11100     11130   +30

Impacted Files
Coverage Δ

app/models/dependant.rb
100.00% <ø> (ø)

app/models/application_proceeding_type.rb
100.00% <100.00%> (ø)

app/models/cfe/base_result.rb
100.00% <100.00%> (ø)

app/models/legal_aid_application.rb
100.00% <100.00%> (ø)

...ervices/reports/mis/application_detail_csv_line.rb
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Job aborts with "Error: Resource not accessible by integration"

We've just added this to our repo but unfortunately it doesn't seem to work.
The "Checking #PR" step aborts with Error: Resource not accessible by integration
See: https://github.com/raiden-network/raiden/runs/1486971356?check_suite_focus=true

Looks like this is a known issue with github actions and forked PRs.
Apparently using pull_request_target instead of pull_request solves the problem: https://github.blog/2020-08-03-github-actions-improvements-for-fork-and-pull-request-workflows/#improvements-for-public-repository-forks
Will try and report back.

Ok this fixed the issue.
So maybe changing the documentation to use the pull_request_target trigger would make sense.

Thanks for investigating this @ulope. I will read more about this in the morning and update the docs accordingly :+1:

You are right. I updated the docs accordingly.


clean/fix role validation for upgrade

@vkareh @ciaranRoche @zgalor  PTAL

@vkareh @ciaranRoche I still have roleCompatible in the operatorRole alone  just to support the older versions. Not sure if we need to do it in the upgrade cluster without asking the user to use upgrade command.

[Verified](https://gist.github.com/ciaranRoche/81b122f13f1e00b864ed509dbbd5e4c8 :+1: )

@vkareh @ciaranRoche  Updated . PTAL

/lgtm


SSL Certificate for https://jabbr.net Expired

The SSL certificate for https://jabbr.net expired 64 days ago and shows a warning in chrome.

Abandoned. :(


warn compatibility function causes write guard warning in OpenResty

Loading Penlight in OpenResty gives me a warning on stdout:
[warn] 19#19: *2 [lua] _G write guard:12: __newindex(): writing a global Lua variable ('warn') which may lead to race conditions between concurrent requests, so prefer the use of 'local' va
riables
stack traceback:
        /usr/lib/openresty/luajit/share/lua/5.1/pl/compat.lua:230: in main chunk
        [C]: in function 'require'
        /usr/lib/openresty/luajit/share/lua/5.1/pl/utils.lua:9: in main chunk
        [C]: in function 'require'
        /usr/lib/openresty/luajit/share/lua/5.1/pl/pretty.lua:12: in main chunk
        [C]: in function 'require'

Would it be possible to set it using rawset, i.e. change its definition from
https://github.com/lunarmodules/Penlight/blob/16743e96a219cf5c07856b8d7dc36900eecac6c5/lua/pl/compat.lua#L230
to
local function warn(arg1, ...)
...
end
rawset(_G, "warn", warn)

? It would make it compatible with current versions of OpenResty.

can you sent a PR please?
2 caveats;

make sure the docs still render properly
please add a comment that it's a workaround for OpenResty

Done, it doesn't change anything in docs as the change is contained within enclosing code block (and I verified it by generating docs in my branch)


Add  as root element

Configuration and Module are the only current root elements. Adapter should also be one, but only of none of the others are present.

This might not even have to be done. There is an issue in the frank-doc ibissource/frank-doc/issues/93


Research Spike: Evaluate the feasibility of "light" project set (less than 4 accounts) provisioning for Private Cloud

Describe the issue
This feature would enable teams to select which of the dev, test, tools and prod namespaces their want in their project set, rather than getting all four by default, a user can select any number of namespaces from 1 to 4 with no mandatory namespace as long as there is one selected.
TBD - we should clarify if there will be any mandatory namespace(s). We need to understand the pricing plan and requirements for having any particular namespace type.
Rational/Why
Teams may not want to pay for all four name spaces in their project set.
Additional Context

this functionality exists in public cloud.

Dependencies

Internal, external, who to talk to if more information is needed

Tests
Definition of done/Acceptance Criteria

[ ] - get a decision about this registry feature if we are proceeding with this change
[x] - confirm all dependencies with Provisioner and Ian

If the decision will be "yes" to proceed with this change:

[ ] - define the UI changes in the Registry
[ ] - define NATS messages settings

The provisioner correctly processes a NATS message that contains less than four namespaces.  The '.namespaces' list of the message should include only those namespaces that have been requested.
Vault access setup works

Zero quota support has been added to the provisioner.  To demonstrate how to set up the NATS message, first here is a normal record from the namespaces list:
  {
    "name": "e95e91-tools",
    "quota": {
      "cpu": "cpu-request-1-limit-2",
      "memory": "memory-request-2-limit-4",
      "storage": "storage-1",
      "snapshot": "snapshot-5"
    },
    "quotas": {
      "cpu": {
        "requests": 1,
        "limits": 2
      },
      "memory": {
        "requests": "2Gi",
        "limits": "4Gi"
      },
      "storage": {
        "block": "1Gi",
        "file": "1Gi",
        "backup": "1Gi",
        "capacity": "1Gi",
        "pvc_count": 60
      },
      "snapshot": {
        "count": 5
      }
    }
  }

Now here is the same namespace configured for zero quota:
  {
    "name": "e95e91-tools",
    "quota": {
      "cpu": "cpu-request-0-limit-0",
      "memory": "memory-request-0-limit-0",
      "storage": "storage-0",
      "snapshot": "snapshot-0"
    },
    "quotas": {
      "cpu": {
        "requests": 0,
        "limits": 0
      },
      "memory": {
        "requests": 0,
        "limits": 0
      },
      "storage": {
        "block": 0,
        "file": 0,
        "backup": 0,
        "capacity": 0,
        "pvc_count": 0
      },
      "snapshot": {
        "count": 0
      },
      "pods": 0
    }
  }

Note that I have added quotas.pods, set to zero.  It's not normally included, because the default value is set in the provisioner and we do not allow users to update this setting via the registry.
Custom resources, such as GitOpsTeams and Sysdig-Teams, can still be created and processed in a tools namespace with a zero quota.

@IanKWatts
Thank you for the prompt research. Since a zero quota (empty) namespace is possible, we can offer the project team the option to scale down the size of unused namespaces rather than providing a checkbox to select which namespaces they want. This way, we can maintain the current workflow, which creates four separate namespaces (dev, test, prod, and tools), while giving them the option of an empty quota to save budget on unused namespaces. This simplifies the process and avoids handling cases where namespaces need to be removed from existing projects.

Olena's response from Oct 11, 2024:
Yes, zero quota namespaces will not have any cost. I like this option too since we can enable it easily but just adding an additional option in the quota dropdown

After further research and discussion, we've decided to proceed with the second option: adding an empty quota option to the dropdowns. This approach maintains the current workflow and simplifies development efforts by avoiding the use of namespace checkboxes.


Unit tests failing due to missing CultureInfo.InvariantCulture in ToString()

Description
The following unit tests fail on machines that do not use the US locale (invariant culture):

DecimalValueTest
DoubleValueTest
SingleValueTest

Information
The issue can be observed on machines not configured to use the en-US locale (or similar locales with identical number formats).
Repro
Set your locale to de-DE, for example, making sure your local number format is different from the en-US format. For example, "one thousand decimal 5" would be formatted like:

1,000.5 in the US (using a comma to separate thousands a point as the decimal separator) and
1.000,5 in Germany (using a point to separate thousands and a comma as the decimal separator).

Simply using number.ToString() will produce different results for different cultures. To get the same result when comparing numbers in string format on machines with different locale / culture settings, you must use number.ToString(CultureInfo.InvariantCulture).
Observed
The above-mentioned unit tests fail on machines using number formats different from those used in the US (invariant culture).
Expected
The above-mentioned unit tests pass regardless of the local number formats.

Is this a problem with just the unit tests or is there an issue with the parsing of the SDK?

It was just a problem with the unit tests. The expected values produced by using number.ToString() rather than number.ToString(CultureInfo.InvariantCulture) depended on the culture settings of the machine on which those unit tests were run.


ENH: Cythonize scipy.spatial.transform.Rotation

What does this implement/fix?
I cythonized part of the code in scipy.spatial.transform.Rotation. I used this package a lot in my own projects and I found that sometimes the quaternion operations could be a bottleneck, so I wrote a Cython version of the Rotation class.
I only implemented part of its member functions and left others as it was. I also ran the tests locally and they passed. Please help me with more testing and maybe more optimization.
Also note that since cython doesn't support np.deprecate decorator, I removed them in the cython version. Since those functions will be depricated in the next version, I think it will be ok.
Additional information
Here is a small benchmark
import timeit

def benchmark():
    m, n = 10000, 10
    setup_str = '''
import numpy as np
from scipy.spatial.transform import Rotation
np.random.seed(12345)
'''

    matrix_test = f'''Rotation.from_matrix(Rotation.random({n}).as_matrix())'''
    print(timeit.timeit(matrix_test, setup=setup_str, number=m))

    euler_test = f'''Rotation.from_euler("XYZ", Rotation.random({n}).as_euler("XYZ"))'''
    print(timeit.timeit(euler_test, setup=setup_str, number=m))

    rotvec_test = f'''Rotation.from_rotvec(Rotation.random({n}).as_rotvec())'''
    print(timeit.timeit(rotvec_test, setup=setup_str, number=m))

    mul_inv_test = f'''Rotation.random({n}) * Rotation.random({n}).inv()'''
    print(timeit.timeit(mul_inv_test, setup=setup_str, number=m))

if __name__ == "__main__":
    benchmark()

In my PC, the original python version yields:
1.1189252500189468
3.943033237999771
0.8586579390102997
1.1278571970178746

Cython version yields
0.24510287103476003
1.0782683069701307
0.18285539897624403
0.28258025902323425

Although the difference of them will become smaller as n becomes larger, I still find it useful since usually I'm not storing the quaternions in a single Rotation object.

Also note the comparison when n=1, python results:
1.029303212009836
3.651804458000697
0.7752523259841837
1.0055766080040485

cython results:
0.1583877899684012
0.4474479290074669
0.09767879103310406
0.20034146698890254

Most tests have been fixed, only one left where I'm not sure where leads to the error of zero values...

Done @rgommers

Any problem for merging the pr?

Any problem for merging the pr?

There are no conceptual problems, I think it's a very solid improvement!
You need to resolve a conflict which appeared. Do rebase on master.

I've already rebased onto the master. I can't see where the files have conflict...

Failures here are related to Rotation -- https://travis-ci.org/github/scipy/scipy/builds/697283538
There are no real errors so it seems,  you only need to adjust tolerances slightly. Perhaps floating point arithmetic in Cython worked somewhat different.

There is a small tolerance violation in test_get_item https://travis-ci.org/github/scipy/scipy/jobs/697674504

Let's wait couple of days before merging for other people to comment if they want to.

@rgommers @pmla
Do you have any more suggestions? Do you approve this to be merged?

OK, it was enough waiting, I think it will be fair that I finally merge it.
@cmpute good job!


upstream issue with slate-react 0.66

see https://github.com/ianstormtaylor/slate/issues/4603

still open


Redirect errors with valid cookies

Using the latest release (0.4.2) like this:
$ edge-gpt-image --prompt "A funny cat" --cookie-file /tmp/cookies.json

Results in:
Traceback (most recent call last):
  File "/nix/store/6xycw16a59gf3wlpbs7cpabayz10q40w-python3.10-edge-gpt-0.10.11/bin/.edge-gpt-image-wrapped", line 9, in <module>
    sys.exit(main())
  File "/nix/store/5h6j028kglkpysjpsp517dvcb1a6w703-python3.10-bing-image-creator-0.4.2/lib/python3.10/site-packages/BingImageCreator.py", line 467, in main
    asyncio.run(
  File "/nix/store/6qk2ybm2yx2dxmx9h4dikr1shjhhbpfr-python3-3.10.11/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/nix/store/6qk2ybm2yx2dxmx9h4dikr1shjhhbpfr-python3-3.10.11/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()
  File "/nix/store/5h6j028kglkpysjpsp517dvcb1a6w703-python3.10-bing-image-creator-0.4.2/lib/python3.10/site-packages/BingImageCreator.py", line 394, in async_image_gen
    images = await image_generator.get_images(prompt)
  File "/nix/store/5h6j028kglkpysjpsp517dvcb1a6w703-python3.10-bing-image-creator-0.4.2/lib/python3.10/site-packages/BingImageCreator.py", line 299, in get_images
    raise Exception("Redirect failed")
Exception: Redirect failed

Full error:
errors.txt
The same cookie works in edge-gpt:
edge-gpt --prompt "A funny cat" --cookie-file /tmp/cookies.json

        EdgeGPT - A demo of reverse engineering the Bing GPT chatbot
        Repo: github.com/acheong08/EdgeGPT
        By: Antonio Cheong

        !help for help

        Type !exit to exit

Initializing...
Enter `alt+enter` or `escape+enter` to send a message

You:
A funny cat

Bot:
Searching the web for: `funny cat`
...

When opening saving the response as html, it will result in the following:
There was a problem creating your images
We're working to resolve this issue and your boost won't be used. Please try creating again later.
In the meantime, [Explore other creations](file:///images/create?FORM=GERRLP) on the Image Creator homepage.

Upgrade BingImageCreator and try python3 -m BingImageCreator --cookie-file ./cookies.json --prompt "creepy Microsoft"

Upgrade to what? I am already using the latest code: https://github.com/acheong08/BingImageCreator/commit/c43e9a291ab6026ddfaf5eec522ebe6b10b39f03

$ python3 -m BingImageCreator --cookie-file ./cookies.json --prompt "creepy Microsoft"
Sending request...
ERROR: <!DOCTYPE html><title>creepy Microsoft - Image Creator from Microsoft Bing</title><meta content="text/html; charset=utf-8" http-equiv="content-type" /><meta name="referrer" content="origin-when-cross-origin" /><meta property="og:description" content="Intelligent search from Bing makes it easier to quickly find what you’re looking for and rewards you." /><meta property="og:site_name" content="Bing" /><meta property="og:title" content="creepy Microsoft - Bing" /><meta property="og:url" content="https://www.bing.com/images/create?q=creepy Microsoft&rt=3&FORM=GENCRE" /><meta property="fb:app_id" content="570810223073062" /><meta property="og:image" content="http://www.bing.com/sa/simg/facebook_sharing_5.png" /><meta property="og:type" content="website" /><meta property="og:image:width" content="600" /><meta property="og:image:height" content="315" /><meta content="text/html; charset=utf-8" http-equiv="content-type" /><meta name="referrer" content="origin-when-cross-origin" /><link href="/sa/simg/favicon-trans-bg-blue-mg.ico" data-orighref="" rel="icon" /><link href="/sa/simg/Roboto_Regular.woff2" rel="preload" type="font/woff2" as="font" crossorigin="anonymous" /><link href="/sa/simg/Roboto_Semibold.woff2" rel="preload" type="font/woff2" as="font" crossorigin="anonymous" />
Traceback (most recent call last):
  File "/nix/store/6qk2ybm2yx2dxmx9h4dikr1shjhhbpfr-python3-3.10.11/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/nix/store/6qk2ybm2yx2dxmx9h4dikr1shjhhbpfr-python3-3.10.11/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/tmp/agitated-kapitsa-VVjWIV/BingImageCreator/src/BingImageCreator.py", line 481, in <module>
    main()
  File "/tmp/agitated-kapitsa-VVjWIV/BingImageCreator/src/BingImageCreator.py", line 464, in main
    image_generator.get_images(args.prompt),
  File "/tmp/agitated-kapitsa-VVjWIV/BingImageCreator/src/BingImageCreator.py", line 139, in get_images
    raise Exception(error_redirect)
Exception: Redirect failed

Yeah. Hard to tell if the actual error is only server side. Will try to see if I can replicate this with a different account later.

Also having this error

@Mic92 @mak448a
Please, try so set the env var :
BING_URL=https://cn.bing.com
despite, i'm in France ... I've don't need this error using chineese bing
perhaps it could work better for you ...
I'm pretty sure it could help you too

I tried cn.bing.com without success. I can see it's beeing used from the error message containing cn.bing.com

I was able to use it now with a different account...
The new account I created also didn't work, but another old one did.


Adding calculator for octahedron

I would like to add a calculator for finding different parameters of octahedron.
Please assign this issue to me.

Go ahead
deadline : 23 April 2021


project-434358444477

434358444477

Why are you in my Providers app

434358444477

I don't know what your providers app is I don't know how anything works either

README.md


Add hierarchical multiple select in sitemap.js

I got such a result: https://dl.dropboxusercontent.com/u/23236615/Easy Export Import.png

This looks like a fine option for package and site developers to be able to use.


How to make scribble option to pertain even after cropping the image and re-setting

I did some scribble on the image and done cropping and changed the original image with cropped one. Now the scribble part getting removed. Stickers/Emoji text everything is getting showed except the scribble.
Any idea to retain the scribble after cropping?

BrushDrawingView class -> We can comment this line "drawPath.reset();" in order to show the scribble even after cropping.
We can reset the path later on this method in same class.
void clearAll() {
if(drawCanvas!=null) {
drawCanvas.drawColor(0, PorterDuff.Mode.CLEAR);
if(drawPath!=null)
drawPath.reset();
invalidate();
}
}
I achieved above by this way. It can help someone

Above solution doesn't worked. It changes the color of already drawn scribble if we change the color and draw some other portion. So I just made image to be cropped along with drawings like parentview.getDrawingCanvas


[Feature][docker] Add UI to standalone server docker images

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Currently, standalone servers does not have UI components and users could not log in via UI after starting it. As an easy way to run dolphinscheduler, we should add full functionality to it and we should add UI to standalone docker images
ref #9862,
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

I both test in branch dev and 3.0.0-beta-prepare and it work, maybe we have some issue when we build the 3.0.0-alpha docker image. ref:


Update the game over message

Update the game over message so people know how to play again!

Finish


CI: strip public CI routes

[ ] Copy entire routes quickly to private, preserved CI bucket: commadataci/commadataci
[ ] Then download qlog/rlog, strip, and upload to public CI bucket: commadataci/openpilotci (any cameras?)

[ ] use miniray, downloading, stripping, uploading can be slow


Bug Report: Difficulty Choice not appearing after playing a song

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
windows 64bit
What version of the game are you using? Look in the bottom left corner of the main menu.
1.6.1
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
Please describe your issue. Provide extensive detail and images if possible.
After playing a stepmania type chart , the difficulty choice does not appear on the top right corner anymore. heres a video of it :
https://user-images.githubusercontent.com/81717689/128138999-73fe30e9-4330-4bc2-935b-8af5c4c180f9.mp4

https://github.com/KadeDev/Kade-Engine/commit/78b7faf83a2a0988ae6d76074404edf186e65c1a


Fix coverity issues with certificate compression

Checklist

[X] tests are added or updated

I don't know how to confirm if this fixes the coverity issues?

I don't know how to confirm if this fixes the coverity issues?

We'll see if they disappear after merging.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

Merged with end of line white space removal.  Thanks.

Where does one see the Coverity results? I looked under actions, but no details are provided.

https://scan.coverity.com/projects/openssl

The 1516093 is still unfixed by this PR. IMO there should be some maximum possible length check on uclen.

The 1516093 is still unfixed by this PR. IMO there should be some maximum possible length check on uclen.

That might be the case. Coverity isn't always clear as to how to fix things.


Need help i am newbie..

I have black screen while using some apps any idea how to fix this I am newbie.. Thanks in advance

The app prevents to take screenshot, and this is totally enforced in Android 12. See #2129.


Macro custom_graph! add generics support

Make the custom_graph! macro support generics:
Examples:
custom_graph!{
    struct SomeGraph
    where AdjListGraph<u32,()>
}
custom_graph!{
    struct NoEdgeWeightGraph<V>
    where AdjListGraph<V,()>
}

Also need to handle lifetimes:
custom_graph!{
    pub struct G<'a>
    where AdjListGraph<'a,u32,()>
}
custom_graph!{
    pub struct G<'a,V,W>
    where AdjListGraph<'a,V,W>
}

Landed complete generic/lifetime support in v0.1.5.


Add block-image feature which forces rendering images using half blocks

spotify-player with enabled image cannot run in zellij in kitty - it hangs on viuer::get_kitty_support(). To enable running spotify-player in zellij I added feature block-image which forces rendering images using half-blocks.

spotify-player with enabled image cannot run in zellij in kitty - it hangs on viuer::get_kitty_support(). To enable running spotify-player in zellij I added feature block-image which forces rendering images using half-blocks.

Does it work in kitty outside zellij or in kitty inside other terminal multiplexers like tmux? It might be zellij bug.
That said, I found it weird to add an additional feature to force rendering images as ascii blocks. Is it better to not use image feature instead of using one with block rendering?

@aome510 it works well in kitty outside zellij (render full-res image). In tmux however it renders images using half-blocks, because the viuer::get_kitty_support() does not detect kitty.
I think having option to render images with ascii blocks when full-res is not available would be nice. I found a workaround for that without any changes in code - simply set TERM env var to anything and run spotify-player, so if you think this feature is not a good idea, feel free to close this PR.

Glad that you find a workaround. IMO, the issue is zellij-specific, so it should be fixed from that side. Adding a separate feature on top of image to render ascii blocks, which is the "fallback" option if no full-res option is available, feels kinda hacky to me.
I'll close this PR for now. Thanks for raising the issue and taking time to implement this though.


Fix image upload when using G+-Photos app

When using the G+-Photos app, it seems that photo URIs returned by the pick image intent are no longer associated with a filename. Use the activity's content resolver to get a stream instead and use that for uploading.
I have not tested this with the AOSP Gallery app because my phone (Nexus 4 with factory L image) only has the G+-Photos app. It would be great if somebody with this app could try the patch.
ContentManager now no longer sets Content-Length. During my testing on matrix.org, this didn't seem to be a problem, but reading the synapse source it actually should (see https://github.com/matrix-org/synapse/blob/771892b3144e5a45739a097f818e71a7ce5d8ce2/synapse/media/v1/upload_resource.py#L48). Not sure what's going on here.

This looks good (inspected diff only; not tested), but I would prefer that the logic to go from the Intent to the InputStream be factored out into ResourceUtils or another appropriate utility class, to avoid the duplication that has sprouted here (RoomActivity:312 and SettingsActivity:227)
In addition, as bad form as it is, please wrap your calls to openInputStream and getType in a generic Exception block, not just FileNotFoundException. This is because some versions of Android will throw undocumented exceptions like IllegalStateException when accessing these APIs. By not catching them, this will crash the app, but they are entirely recoverable errors.

Thanks for the hint about Android API nastyness regarding exceptions. I've added a blanket catch-all.
I've moved the code for opening the resources to ResourceUtil as you suggested. I used null as a catch-all error return value to make the code a bit simpler for callers. (Is there something like Optional in Guava for Android?)

Fixes look good, thanks! I'm not sure on Optional for Guava. Will give it until tomorrow for anyone else to chime in, then merge this in.


making a drawer open on the right

Hello, I am trying to construct a drawer that shows pulseaudio, and then shows mpris on hover like so:
  "modules-left": ["group/audio"],
   "modules-center": ["hyprland/workspaces"],
   "modules-right": ["network", "battery", "clock", "tray"],
   "group/audio": {
     "orientation": "inherit",
     "drawer": {
         "transition-duration": 500,
         "children-class": "not-power",
         "transition-left-to-right": true,
     },
     "modules": ["pulseaudio", "mpris"]
   },
   //configs for mpris and pulseaudio

I have 1 minor and a major problem.

since the module is on the left, the drawer needs to slide out on the left, but it comes out on the left and I can't find a fix.

my modules are round, so pulseaudio needs to have different css when the drawer is extended vs non-extended.

clarification: when the drawer is closed it should look like this:

accident sry

I can recreate your problem. transition-left-to-right doesn't seem to change anything.
When looking at the Object Hierarchy with this, Revealer comes before EventBox (the leaderObject):

Since changing the transition_type of revealer doesn't change anything, I tried  moving Revealer after the leaderObject:
diff --git a/src/group.cpp b/src/group.cpp
index cad36e51..997ab628 100644
--- a/src/group.cpp
+++ b/src/group.cpp
@@ -73,7 +73,7 @@ Group::Group(const std::string& name, const std::string& id, const Json::Value&
     revealer.get_style_context()->add_class("drawer");

     revealer.add(revealer_box);
-    box.pack_start(revealer);
+    box.pack_end(revealer);

     addHoverHandlerTo(revealer);
   }

It worked:

@Alexays did I miss something on how revealer works?
I'll add an If statement to check which function to use based on transition-left-to-right and make a PR.
I might even be able to remove to whole preferedTransitionType thing.


Youtube Downloader not updating in debian 11

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE
[youtube] 8Ml4kWMwUro: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

[x] I've searched the bugtracker for similar bug reports including closed ones

Learn how by reading #30839 (again?).

[x] I've read bugs section in FAQ

Apply this.


SR use case should be reordered to before offline?

In general all the single stream use cases are run before the offline use case.
May be we should have SR use case as well, before offline?

Let's do it. @anhappdev


request options (jar) clarification

From README.md, on request(options, callback):

jar - If true, remember cookies for future use (or define your custom cookie jar; see examples section)

var j = request.jar() is also used to create a custom jar.
and we can set cookies in the jar by using j.setCookie(cookie, url).
I'd just like to ask for clarification. Is the jar used for sending cookies to a server, storing cookies obtained from the server or both?
As far as the README.md explains, it's only for storing cookies obtained from the server.

Seems to work for both for me...


AOT Optimizer with Ivy enabled for a hybrid Angular/AngularJS app doesn't include all the necessary components

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Yes, it used to work with the 8.x versions because Ivy was not enabled by default.
However, it still works correctly with Ivy disabled, so it's not a critical regression.
Description
For a hybrid Angular/AngularJS application, having Ivy, AOT and the Optimizer enabled will prevent the production bundle to contain all the necessary components.
Disabling any of those 3 (Ivy, AOT, or the Optimizer) will allow the builder to include all the used components.
This issue seems to be related to the fact that AngularJS is in charge of the routing and the missing components are only used in AngularJS templates.
🔬 Minimal Reproduction

You can checkout this project: https://github.com/maximecarlier/angular-hybrid-ivy-bug
Run npm i
Run ng serve and see that the string "Angular/AngularJS Hybrid" is displayed
Run ng serve --prod  and see that that's not the case anymore
Possible actions with the same result:

Disable Ivy (in tsconfig.json, set angularCompilerOptions.enableIvy to false)
Disable AOT (in angular.json, set projects.angular-test.architect.build.options.aot to false)
Disable the scripts optimization (in angular.json, set projects.angular-test.architect.build.configurations.production.optimization.scripts to false)

Run ng serve --prod  and see that the string appears again

🔥 Exception or Error
No exception or error, the components are just not part of the bundle file for production.
🌍 Your Environment

Angular CLI: 9.0.2
Node: 12.13.0
OS: linux x64

Angular: 9.0.1
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, upgrade
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.2
@angular-devkit/build-angular     0.900.2
@angular-devkit/build-optimizer   0.900.2
@angular-devkit/build-webpack     0.900.2
@angular-devkit/core              9.0.2
@angular-devkit/schematics        9.0.2
@angular/cli                      9.0.2
@ngtools/webpack                  9.0.2
@schematics/angular               9.0.2
@schematics/update                0.900.2
rxjs                              6.5.4
typescript                        3.7.5
webpack                           4.41.2

Anything else relevant?

Hi, I think this has the same root cause as https://github.com/angular/angular-cli/issues/15809. See the following comment https://github.com/angular/angular-cli/issues/15809#issuecomment-543281509 for way on how to make this work.


Doc updates batch environment

Creating documentation for batch environments module.

The text looks ok, if a little brief, but I think this can be squashed into a single commit instead of 7.

@rstobaugh did someone already show you how to rebase?

@rstobaugh I've rebased this to make it just one commit. Here's a mini-tutorial for how to do it yourself next time on the command line (it might be easier if you're using a GUI):
git fetch origin
git checkout your_branch
git rebase origin/master

If there were no conflicts this should return successfully. Now, because this rewrites git history you need to force push the changes to the remote:
git push --force-with-lease

If, after git rebase there are messages about conflicts you can do git status to see which files have conflicts. When you open the files they will have the conflicts marked like this:
<<<<<<<
conflicting changes from master branch
=======
conflicting changes from your branch
>>>>>>>

You'll have to choose which changes to keep. Once that's done you have to git add files/that/had/conflicts, and do git rebase --continue. The rebase process continues like this until there are no more conflicts. Once done git push --force-with-lease.


How does filtering on custom attributes work?

[Enter feedback here]
While trying to filter on string custom attributes, it does not work whereas same works for Boolean custom attributes.
For eg -
Scenario 1 : Filtering on string custom attribute does not work
https://graph.microsoft.com/v1.0/users?$select=id,displayName,givenName,surname,extension_[..]_customerId,extension_[..]_title,extension_[..]_locked,identities,preferredLanguage&$filter=startsWith(extension_[...]_Title,'He')
Scenario 2 : Filtering on Boolean custom attribute works.
https://graph.microsoft.com/v1.0/users?$select=id,displayName,givenName,surname,extension_[..]_customerId,extension_[..]_title,extension_[..]_locked,identities,preferredLanguage&$filter=extension_[..]_locked eq false
Can you please provide few clarifications here -

How does filtering behaves with custom attributes in B2C tenant?
Is there a general limitation  that filtering with $filter on string custom attributes doesn't work? Is it supported for any other datatype (Boolean or so)?

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

ID: c29c26fd-dad6-8435-0694-ba82c784818a
Version Independent ID: fab7c0ac-b35b-83f5-5a1e-c8a6ce26dc8b
Content: Define custom attributes in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/user-flow-custom-attributes.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@Shweta-1405  Thanks for reaching out. We will investigate and update the thread accordingly.

@Shweta-1405  I tested $filter parameter with extension attribute (string type) in my B2C tenant and it worked for me without any issue.
Below is the call for your reference:
GET https://graph.microsoft.com/v1.0/users?$select=id,givenName,surname,extension_2ad62ac450xxxxxxxxfd85dea302d1b5_ShoeSize&$filter=startsWith(extension_2ad62ac450xxxxxxxxfd85dea302d1b5_codename,'Al')
Make sure you use the GUID of B2C extension application (without hyphens) after extension_ in the attribute name.
If you still face any issues, feel free to tag me in your reply or send an email to AzCommunity[at]Microsoft[dot]Com with subject Attn:Amanpreet and I will respond you back.

Hi Aman
I initially created the question here:
https://docs.microsoft.com/en-us/answers/questions/403031/graph-api-azure-b2c-filter-for-string-extension-at.html?childToView=417903#comment-417903
The command you mentioned doesn't work on my end, check my scenario in the link above. Anything i'm overseeing here?
Cheers
André

@acrettaz The graph call you are using looks correct to me. Could you please send an email at AzCommunity[at]Microsoft[dot]Com so that we can schedule a call to look into it.

@amanmcse
does it possible to filter by custom attributes with IN and NOT IN clauses or HAS clause?
https://graph.microsoft.com/beta/users?$filter=extension_24182594d2c140e4b5e1d30b3eb3b72b_AppRole in ( '')
here is an error while using IN operator with custom attributes.
{
"error": {
"code": "BadRequest",
"message": "Filter IN operator on dynamic property not supported.",
"innerError": {
"date": "2022-02-08T13:51:02",
"request-id": "e5513c68-bf41-4b0f-a7ae-53d50ae63b31",
"client-request-id": "e5513c68-bf41-4b0f-a7ae-53d50ae63b31"
}
}
}


Ug changes

Codecov Report

Merging #171 (2b8e475) into master (6f9c9f5) will increase coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #171      +/-   ##
============================================
+ Coverage     39.24%   39.27%   +0.03%
  Complexity      366      366
============================================
  Files           113      113
  Lines          2138     2136       -2
  Branches        198      197       -1
============================================
  Hits            839      839
+ Misses         1224     1222       -2
  Partials         75       75

Impacted Files
Coverage Δ

...n/java/seedu/travelr/model/component/Location.java
64.28% <ø> (ø)

src/main/java/seedu/travelr/model/trip/Trip.java
60.24% <ø> (ø)

src/main/java/seedu/travelr/ui/TripCard.java
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix(playground): add GA to the site for Netlify too

Related to #235 and #323

@mhagmajer , could you review please?


Support 1D signals

See initial discussion in https://github.com/google-research/receptive_field/issues/1
@Permafacture offered to help with this.
The main thing to be done would be to generalize/abstract the layer parsing functions to handle 1D (besides 2D): get_layer_params, _stride_size, _conv_kernel_size, etc in https://github.com/google-research/receptive_field/blob/master/receptive_field/python/util/parse_layer_parameters.py#L41
From the discussion in https://github.com/google-research/receptive_field/issues/1, we see that the 1D conv may be done as a Conv2D with ExpandDims (from @Permafacture:
ValueError: Weight layer's name input to conv layer does not end with '/read' or '/Conv2D/ReadVariableOp': projection_layer/conv1d/ExpandDims_1)
The most efficient way to make progress here is to create a new test network for a 1D case in https://github.com/google-research/receptive_field/blob/master/receptive_field/python/util/parse_layer_parameters_test.py, this will help understand if a proposed implementation is parsing the parameters correctly.

Should I make a pull request that adds that test network?

Hmm, I guess a test using the 1D network wouldn't pass. So I'd suggest to only send a pull request once the code is adapted to also parse correctly parameters for the 1D case.

Hi. @andrefaraujo, do you have any updates about support 1D signal?

Not really, @okovalchuk98 . Would you be interested in helping with it?


Cleaned up loss implementation and usage

I think I have fixed the issue mentioned in #2
The TempCombLoss takes target1 (the base model output) and  target2 (the ground truth) and these are applied appropriately.

Thanks.


Symfony\Component\Debug\Exception\FatalThrowableError in GET /nova-api/users

Error in Apiary
Symfony\Component\Debug\Exception\FatalThrowableError in GET /nova-api/users
Return value of App\Nova\Filters\UserActive::apply() must be an instance of Illuminate\Database\Eloquent\Builder, instance of Illuminate\Database\Eloquent\Relations\BelongsToMany returned
View on Bugsnag
Stacktrace
Nova/Filters/UserActive.php:28 - App\Nova\Filters\UserActive::apply
Http/Middleware/CASAuthenticate.php:143 - App\Http\Middleware\CASAuthenticate::handle

View full stacktrace
Created by RoboJackets Admin via Bugsnag

Occurred when filtering users on major view


Access instructions need updating

The page layout probably changed a little bit on the Azure portal.
Under the API access section, step 2 and 3 probably should be replaced by the following:
Click "API access (new)" on the navigation bar.

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

ID: d3fefb59-9060-78bc-6f1c-deac80093225
Version Independent ID: cebd7747-5d1a-e541-eb8a-3c268c28356e
Content: Get started with Azure AD authentication - Azure Media Services v3
Content Source: articles/media-services/latest/access-api-howto.md
Service: media-services
Sub-service: rest-v3
GitHub Login: @Juliako
Microsoft Alias: juliako

@felixtsem5 Thanks for the feedback! We are currently investigating and will update you shortly.

@felixtsem5 Thank you for bringing this to our attention. We have gone ahead and updated the doc. The changes should be visible within 24 hours.


Automatically work on siblings?

Like:
<d ion>

....

So, ion on parent, and then it just functions on children.
Maybe have options for this?

I am happy with how it is for now. Maybe think about doing this later.


Add the ability to configure UUID representation during serialization

Solves #155
This adds the ability to configure the binary representation of UUID's from within objects during serialization in jackson.

Merged. Thank you!


Issues from wiring friends to Warp

There were some bugs on Stu's PR for wiring Friends to Warp that we wanted to create into a ticket rather than hold up PR

[ ] When copying the DID sometimes User will face a crash

[x] When User sends a friend request that they have already sent app will crash

[ ] Sometimes will face a crash when sending the first friend request

[x] UI does not update when someone accepts your friend request

[ ] adding, removing, and blocking friends updates the UI but doesn't trigger the required warp commands

in ui/src/components/friends/add.rs i should probably save the user's ID the first time and not make future calls to warp

i believe this is fixed now


File selection at runtime

Hi I'm trying to use a file dialog to select a file to give to the video stream. I create a new FFmpegVideoStream and set its file property to the file selected by the dialog but the video streamer fails to play the video. Idk if its a real bug or the file property is not designed to be used as I'm trying to.
I'm attaching a project with what I'm trying to do.
Great addon btw!
VideoPlayback.zip

System info: Godot v4.3.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1050 Ti (NVIDIA; <IP_ADDRESS>90) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)

Hi!
add video launch to the code:
video_stream_player.play()

Hi!
thanks for the response, I guess I should've tried that.
Great Extension btw!


Event in popover element

[x] event when the popover is called to open
[x] event when the popover is called to close
[x] wich element have the called popover

The PR https://github.com/locaweb/locawebstyle/pull/1514 resolves this.


View/apikeys

The most recent commit also fixes #139

@lkuchenb, could you have a look at this, I think I need this PR to be merged to carry on.

@MoritzHahn1337, if you agree with the above, I could also make the changes. You would just need to ensure that the frontend send isoformated expiration dates. Would that be OK?

@KerstenBreuer Okay, I will make sure the frontend sends in isoformat. Would you pack changing the backend into #146 or would you stay in this PR?

That's a good idea, I will include that, just let me know once the required changes in the front end are in place
Holen Sie sich Outlook für Androidhttps://aka.ms/ghei36

From: MoritzHahn1337<EMAIL_ADDRESS>Sent: Monday, March 8, 2021 6:08:45 PM
To: ghga-de/datameta<EMAIL_ADDRESS>Cc: Kersten Breuer<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ghga-de/datameta] View/apikeys (#140)
@KerstenBreuerhttps://github.com/KerstenBreuer Okay, I will make sure the frontend sends in isoformat. Would you pack changing the backend into #146https://github.com/ghga-de/datameta/issues/146 or would you stay in this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ghga-de/datameta/pull/140#issuecomment-792908202, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGVV65KKSQMQU23IMWANQ7DTCUAB3ANCNFSM4YTXY33A.

The most recent commit should take care of that.


fix window function first_value/last_value should be always nullable

Proposed changes
Because the window may be empty, the function first_value/last_value should always be nullable no matter whether the argument is nullable or not.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Helm chart: Respect custom proxy_init image value for enable-core-dump container

Fixes https://github.com/istio/istio/issues/7857.
Used the same logic as for the istio-init container.

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

Needs approval from an approver in each of these files:

install/OWNERS

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

Hi @ewbankkit. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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

/assign @gyliu513

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [gyliu513]

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

/ok-to-test

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9f654d192e3ed5f62a87c9bbc7805c275dbf4708
link
/test istio-unit-tests

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

/test istio-unit-tests

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9f654d192e3ed5f62a87c9bbc7805c275dbf4708
link
/test istio-unit-tests

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9f654d192e3ed5f62a87c9bbc7805c275dbf4708
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
9f654d192e3ed5f62a87c9bbc7805c275dbf4708
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

New changes are detected. LGTM label has been removed.

Rebased.

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
9f654d192e3ed5f62a87c9bbc7805c275dbf4708
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
e0b8caaf2df87cef37e01829990dd638a603f8d6
link
/test istio-unit-tests

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

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
9f654d192e3ed5f62a87c9bbc7805c275dbf4708
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-unit-tests.sh
e0b8caaf2df87cef37e01829990dd638a603f8d6
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
e0b8caaf2df87cef37e01829990dd638a603f8d6
link
/test e2e-simple

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [gyliu513]

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

/test e2e-simple

I kicked off circleci tests.  You may need to rebase.  circleci needs a rebase in some conditions to pull in fixed tests and code.

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e0b8caaf2df87cef37e01829990dd638a603f8d6
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
e0b8caaf2df87cef37e01829990dd638a603f8d6
link
/test e2e-simple

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

New changes are detected. LGTM label has been removed.

Rebased. Fingers crossed...

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
e0b8caaf2df87cef37e01829990dd638a603f8d6
link
/test e2e-simple

prow/istio-unit-tests.sh
773bc3ec9b03497889297be301e0de56b64f6b48
link
/test istio-unit-tests

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [gyliu513]

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


feat: allowedOutOfOrderRecordings can now include "*"

:tada: This PR is included in version 7.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: