Different preconfigured Maven pipeline than in doc

Maven package Java project Web App to Linux on Azure no longer exists. I'm not sure what stages/jobs/steps are performed in that preconfigured pipeline, but when I run the currently available Maven pipeline, it only goes through the package goal. So, after running the pipeline, the jar isn't actually deployed anywhere. When I browser to the url for my site, it is empty.

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

ID: 2464073a-cf55-d4c7-ad10-a7d7a889f03a
Version Independent ID: af0afd6e-bdaf-de9a-18e7-50e4a1689a58
Content: Build and deploy to a Java web app on Linux - Azure Pipelines
Content Source: docs/pipelines/languages/java-webapp.md
Product: devops
Technology: devops-cicd
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Same thing happened to me. Everything shows green in pipelines and build but the app doesn't appear. Is there a start up command missing?

@shashankbarsin When we follow the default instructions here, the website comes up as empty. Should we include any subfolder under the URL to be able to browse to the website?

Apologies for missing the notification on this issue. Assigning to @azooinmyluggage @N-Usha who have worked on the sample associated with this doc

worked but not as documented :
check KUDO : https://my-app-name.scm.azurewebsites.net/
and look for the deployed app in wwwroot/webapps/
here ROOT is what we see in https://my-app-name.azurewebsites.net
and not what we deployed
so the other site is the on we are looking for
https://my-app-name.azurewebsites.net/

Sorry for the delay @kayosoufiane. I'm updating the documentation now. The path for the build is http://my-app.azurewebsites.net/petclinic/.


#36

Added support for this paging taking a function or a boolean.

CC @michaelmcauley @hornetnz

👍

Looks good, appreciate the backwards-compatible way in which this was accomplished 👍


task1

Is your feature request related to a problem? Please describe.

Describe the solution you'd like:

Describe alternatives you've considered:

Additional context:

### Tasks
- [x] output error. Note opening in the ms edge

### Tasks
- [ ] https://github.com/microsoft/vscode-edge-devtools/issues/2164
- [ ] https://github.com/microsoft/vscode-edge-devtools/issues/2139
- [ ] https://github.com/microsoft/vscode-edge-devtools/issues/2165
- [ ] https://github.com/microsoft/vscode-edge-devtools/issues/2161

can you help with local host error and should open in micro soft edge.


Fix typo

Hey,
I would assume this is typo (unless this is something I'm unaware of).
Cheers!

Thanks, you are absolutely right! 😊🎉

No worries, happy to help out @Gummibeer


Error viewing, sending and exporting report in Company Communicator.

          Guys, good afternoon!

I'm trying to preview a message in the channel before sending and I get an error.

The same error occurs when trying to export the report.

Originally posted by @WaldekBarros in https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/issues/1066#issuecomment-1587773622

Hi @WaldekBarros ,
Thanks for raising the query. Could you please respond to the below queries.

What is the version of the app?
Is this a fresh deployment?
Just trying to understand, whether not able to send the message via any sending option?
If yes, please check the configuration end-end.

Hi @WaldekBarros ,

As you mentioned it was working normally at the beginning of the year, could you please share the deployment status of App service. Here is the reference.

To get more information on the errors. Here is the reference.
Go to resource group-->Application Insights-->Under investigate click Failures.

@WaldekBarros , I can see the last sync for app service done a day back. Could you please check whether the steps mentioned in v5 migration guide have been followed correctly or not
If followed correctly, please share the screenshots as mentioned in migration guide

Hi @gsv022, good morning!
I performed the migration procedure for and the first error reported is no longer appearing, but I am unable to send any message due to the following error.
Failed to prepare the message for sending:The orchestrator function 'SyncRecipientsOrchestrator' failed: "The activity function 'BatchRecipientsActivity' failed: "Object reference not set to an instance of an object.". See the function execution logs for additional details.". See the function execution logs for additional details.

Please share your outlook id(email id) so that we can discuss the issue internally.

Please let us know if the issue persists otherwise we will make it to closure.

@gsv022, tks for your help.
I was able to resolve the error by adjusting the key that had expired in App Service and the 3 functions.


Any one can reproduce pspnet result?

I can only achieve a mIoU of ~ 52% using the trained model of PSPNet on the val set even with the full resolution on cityscapes with this psp model, which is far away from the reported result. I didn't train, just validate on the trained pspnet101_cityscapes.caffemodel published by the team of PSPNet, and it should directly get the reported result. Any one reproduce the result successfully?

Hi, @sanweiliti .
You can do the followings:

Download the converted Caffe pretrained weights here
Set img_norm=False and version="pascal" arguments in data_loader (due to data preprocessing of original Caffe implementation)
Replace ptsemseg/models/pspnet.py#L134 with x = F.interpolate(x, size=inp_shape, mode='bilinear', align_corners=True)
Set the corresponding settings in your config file (e.g., img_rows: 1025, img_cols: 2049, resume: pspnet_101_cityscapes.pth)
You will get 78.65/96.31 (mIoU/pAcc) on the Cityscapes validation set.

Hi @adam9500370 ,
Thank you! Did you use the validate.py or the train.py? If you used validate.py, did you set eval_flip to True? I just found that CUDA is out of memory if I use validate.py since I can only use one GPU.

Hi, @adam9500370 I am trying to train the network from scratch. Are there any tips I should follow?

Hi, @sanweiliti . You can do the followings:

Download the converted Caffe pretrained weights here
Set img_norm=False and version="pascal" arguments in data_loader (due to data preprocessing of original Caffe implementation)
Replace ptsemseg/models/pspnet.py#L134 with x = F.interpolate(x, size=inp_shape, mode='bilinear', align_corners=True)
Set the corresponding settings in your config file (e.g., img_rows: 1025, img_cols: 2049, resume: pspnet_101_cityscapes.pth)

You will get 78.65/96.31 (mIoU/pAcc) on the Cityscapes validation set.

This is my results by running evaluation.py after changing things according to those mentioned above:
Overall Acc:     0.8243936553782907
Mean Acc :       0.46220855109731096
FreqW Acc :      0.7202854165179821
Mean IoU :       0.36694185899912246
any tips?

and
Overall Acc:     0.9424106973275609
Mean Acc :       0.7539844599183159
FreqW Acc :      0.895777388036502
Mean IoU :       0.676881206358246
after setting img_rows: 713, img_cols: 713.

I finally got normal results at https://github.com/hszhao/semseg after testing multiple codes. The key is to transform the gt labels rightly (for training and evaluation).
for cityscapes: https://github.com/fyu/drn/tree/master/datasets/cityscapes. (you can transform gt labels ahead)
for ade20k: https://github.com/CSAILVision/semantic-segmentation-pytorch/blob/master/mit_semseg/dataset.py#L70   (you can transform gt inside the code, not ahead training or evaluaiton)
def segm_transform(self, segm):
        # to tensor, -1 to 149
        segm = torch.from_numpy(np.array(segm)).long() - 1
        return segm

I have to calculate mIoU/mAcc/allAcc for my task, but semantic segmentation not my research, so it really takes me some time to calculate these metrics. Hope this is helpful to others.


vm template status is not updated correctly when adding a new template version

Description:
the vm template status of latestVersion is not updated correctly after adding a new template version, it should set the latestVersion to the version id of the newest template version.

step:
create a template version and check the set as default
check the template page, the default version number is correct, but the latest version isn't updated to the lastest one
reason:
in norman, UI can't update the object's status, even UI send status in the request body to the backend, the backend will use current status in etcd to replace it. But in steve, UI can update status. UI update the status to the old one.

@aiwantaozi In what cases the UI will update a vmtemplate with a stale version?

When UI set the default version may also send status with stale version, and on the template controller we don't recalculate the latestVersion before, so the latest version doesn't update.

link to PR https://github.com/rancher/harvester/pull/124

close in favor of https://github.com/rancher/harvester/pull/124


Add travis completion

https://github.com/travis-ci/travis.rb
and more @tswicegood 😉

Always wanted my own tab completion 😄


Collecting user data through Progressive Profiling

Hey Team,
In order to elicit users to share information about themselves, I've used Progressive Profiling, a feature for forms, where new questions appear for returning users who've completed the form prior.
As we think about how to garnish data, without overloading the user, let's consider 3 or 4 sets of questions that could be rotated as users engage with the app.
A description can be found here: https://auth0.com/docs/users/concepts/overview-progressive-profiling

Great idea Nathan! This will help engagement over time if we can get the data in pieces over time.
We should add this to our design flow.

@nathanboone ready to brainstorm this? DM in discord.


[Feature] Calender/Weather

it would be nice if the tracker could "grap" the information from calender/weather
I use in little very simple macro like this: document.getElementById("calendar-weather-precip").innerHTML;

I will handle this through a macro because I want to keep the UI as minimalist as possible.
In the next release (today or tomorrow), there will be a macro folder with the Weather to Aspect macro in it.


[core-rest-pipeline] await prepareFormData()

We should wait for the Promise to resolve
Packages impacted by this PR
@azure/core-rest-pipeline
Issues associated with this PR
#23849

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

I wonder if a unit test can be written for it

It may be possible to mock FormData.getLength() to delay and set some flag.  I don't feel it is worth for this change though.


add solution for 338. Counting Bits

Hi,
This shall be a problem belongs to Math Category. Could you please checkout to Math branch and send a pull request there? Also, do not forget to update the Math section on README file. 👍
Thank you,
Yi


ujust dx-group does not add user to dev groups

Describe the bug
When running the command ujust dx-group my user is not added the the dev groups even after a restart.

❯ ujust dx-group
Appending 1000 to /etc/group
[sudo] password for user:
Appending 10 to /etc/group
Reboot system and log back in to use docker, incus-admin, lxd, libvirt.

After restart

❯ groups
user wheel

What did you expect to happen?
I expected that my user was added to the dev groups.
Output of bootc status
❯ sudo bootc status
No staged image present
Current booted image: ghcr.io/ublue-os/bluefin-dx:stable
    Image version: 41.20241124.2 (2024-11-24 06:01:50 UTC)
    Image digest: sha256:b89a7d095116dbc7eefaf9caf248e3e620539fb12d96fd932254364241ddd065
Current rollback image: ghcr.io/ublue-os/bluefin-dx:stable
    Image version: 41.20241124.2 (2024-11-24 06:01:50 UTC)
    Image digest: sha256:b89a7d095116dbc7eefaf9caf248e3e620539fb12d96fd932254364241ddd065

Output of groups
user wheel

Extra information or context
I have just today (24-11-2024) installed bluefin-dx:stable via the installation image (iso).
Ran ujust update after installation.
Initially  I only did a logout/login after running a ujust dx-group. I know the terminal message says to reboot, however the the docs only mentions logout/login so I went with that.

This worked for me with a reboot with gts, does it work after rebooting?

This seems to be a bug with the recipe itself, but logging out/logging back in does not apply/remove the groups, we probably should update the docs to mention that. This is most likely because of some lingering setting in systemd-logind being set - Stuff like zellij sessions don't seem to get killed on log out in my system

I can confirm it now works. Thanks!


[USABILITY] Add better punctuation in help pop-up for color codes

Describe the usability issue
The list of conditions related to Orange color status is comma-separated. It's hard to parse for novice users who don't understand how the testing program works.
Each item in the list needs to be understood (by a user) as a separate and sufficient condition for Orange, not a combination of conditions.
Ideally this list would be a bullet list, but I get the impression that bullet formatting is hard to do in this pop-up.
Therefore, please use semicolon separator between each item in the list, as follows below.
Revised text:
Orange: First time user;  Past due for test;  Self-reported symptoms;  Received exposure notification; (or) Quarantined
To Reproduce
Go to Home > Your Health > Tap the circle i (i)
Help for status color definitions is displayed
Screenshots

Additional context
I am aware that Allie's more involved revisions for this pop-up are "on hold".
https://github.com/rokwire/safer-illinois-app/issues/242
The change I am asking for is not complicated, and will resolve frequently asked questions.

@joegrohens We will wait until we redo the entire panel. This is lower priority at the moment.


Add thin class wrapper around the gifski api

These are the changes I suggested in my review of #6
I made some changes to the progress handling as well. Let me know if you want me to split up this PR.
Need some more work on the error handling..
Feel like I could improve setProgressCallback a bit, but this is what I came up with now.

I removed the force unwrapping of CGImage.dataProvider, but it will fatalError if it is nil.
Should probably introduce better error handling in the convert function. Maybe it should take a callback that takes a Result type, but that may be better to discuss in a separate issue/PR?

Maybe it should take a callback that takes a Result type, but that may be better to discuss in a separate issue/PR?

Yeah, I agree. That would be much nicer. I think we should also make a convenience wrapper of generator.generateCGImagesAsynchronously in an extension that returns a Result when done.

This is just so much better than what we had before.
Thanks for helping out and for your guidance, @LarsJK. 🙌

Cool! Happy too help 😊


Put the commit hash in the uploaded artifacts so we can recognize them

These versions are known to the engine and accessible at //flutter/shell/version/version.h

I think we should revert this. My original comment never got a compelling answer and without a use case we shouldn't support something. Exposing versions in APIs is rarely a good idea.

We recently looked into using this to make sure the bots were uploading the right versions of engine artifacts to the cloud.

I've been talking to @dnfield about that. I think we can solve that more effectively by just grepping the binary for the version string, no need to expose that to the API.

If you reverted the engine patch, the version strings wont be in the binary :/

Wouldn't they still be there for embedder consumers?  Or they'd get stripped out because nothing actually needs them at runtime?

We can force them to be there, that's not an issue. We don't need to expose the API to embed them.

Reopening this, as it doesn't really seem resolved.

/cc @jonahwilliams FYI

Closing. The hashes are in there because these are now used by the persistent caches to version artifacts. The request for an API to access the same is a Wont-Fix per @Hixie s comment.

I ended up adding this to the flutter_test binary, and it can be viewed via invoking the binary with the --help argument.


Custom CSS to hide blocked messages is not working (Use of !important)

That's a warning, not an error. Also, that class is non-existent now.
This isn't an issue with BetterDiscord, so instead you should head to the support server for issues like this.

Doesn't custom CSS only work with BetterDiscord?

It looks like a recent update to Discord appears to have broken something about this aspect of the Custom CSS, as stated above. Script does not hide the element as intended. Did they perhaps change the description of the message group in some manner?

This works
div[class^="messageGroupBlocked-"], div[class*=" messageGroupBlocked-"] {
display: none;
}

Thank you.

This will hide the message and add a line back in so it's completely seamless.
div[class^="messageGroupBlocked-"], div[class*=" messageGroupBlocked-"] {     height: 0;     line-height: 0;     overflow: hidden;     margin: 0 20px 0 20px;     border-width: 1px 0 0 0;     border-color: #3E414A !important; }

latest discord update broke the it

Gotta love how they patch things just to cause grief to the people that improve their shitty application on their own time.

div[class^="blockedSystemMessage-"], div[class*="blockedSystemMessage-"] { display: none; }
Hey guys, they changed the div for this, so now you should use the above div to block the message group.

div[class^="blockedSystemMessage-"], div[class*="blockedSystemMessage-"] { display: none; }
Hey guys, they changed the div for this, so now you should use the above div to block the message group.

Thanks, but there's now a transparent gap where the original "1 blocked message" usually is. The messages aren't being readjusted properly.

div[class^="container-"]{ min-height: 0; } div[class^="blockedSystemMessage-"], div[class*="blockedSystemMessage-"] { display: none; }
Pretty simple fix for that, make it so that all of the containers no longer have a minimum height.

Sweet, thanks. I don't code so it's not simple, but I'm glad you do and are willing to address the situation so quickly :)

I can't comprehend why Discord refuses to add this and turns people to Bandaged and anti ToS shit like that. It's something that you can easily do with the basics of CSS and make it look flawless, yet they claim they aren't able to sort it out when they have direct access to their very own code. Clowns.

div[class^="wrapper-"]{ min-height: 0; } might now be needed for compact mode, just fyi

div[class^="container-"]{ min-height: 0; }
div[class^="blockedSystemMessage-"], div[class*="blockedSystemMessage-"] {

    display: none;
}

div[class^="groupStart-"], div[class^="da-groupStart"] {
    display: none;
}

I had to add the bottom part in order to hide everything after the latest update, but it is working well for me.

Discord doesn't load now after trying to edit the CSS on the newest version... Anyone else?

It happened to me, although I'm not sure whether or not that is the cause.
it's related to
On Mon, Mar 16, 2020, 09:21 PlsBogs<EMAIL_ADDRESS>wrote:

Discord doesn't load now after trying to edit the CSS on the newest
version... Anyone else?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Jiiks/BetterDiscordApp/issues/860#issuecomment-599563014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ6OIL7QHNUHAVW3BSMRRKDRHYYWXANCNFSM4FN4GSKQ
.

BetterDiscord doesn't keep any of those custom CSS when I leave the configuration page. I have the error "^= are slow".
Anyone else ?

Still works for me, btw if you want a super compact mode (zero padding between compact messages) then you can go with a .compact-T3H92H.wrapper-2a6GCs, .cozy-3raOZG.wrapper-2a6GCs{padding-top: 0; padding-bottom: 0;}

Doesn't custom CSS only work with BetterDiscord?

It literally says "BetterDiscordApp," bruh.

Does this still work as of December 2021?

FourWolves's code works for me


📝[documentation Update]: Make the navbar more visually appealing

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
Organize the navbar. Make one button that will have both login and signup section to make the navbar more compact. Please assign me this issue
Suggested Change
Organize the navbar. Make one button that will have both login and signup section to make the navbar more compact.
Rationale
No response
Urgency
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@Mahima-2404 please give preview screenshot or preview screen recording of your changes.

@Anjaliavv51 I would like to work on this issue . Can you please assign this to me?

@Anjaliavv51 Here is the preview. It can be changed if required
Before:

After:
https://github.com/user-attachments/assets/8bc344cd-d74d-4ec5-b4d4-5b03f5a85f66
Note: The display of the screen recording is a little faulty but the website is completely fine

@Mahima-2404 update your branch and add your fature.


qat plugin init container crashes with provisioning config

Summary
On OCP 4.12.26 with device plugins 0.26.2, when using the provisioning configmap with complete crypto or compress resources the qat plugin's init container crashes.
Details
For the first time when using this configuration, there are no issues with plugin.
If we delete and recreate with same or new resource (cy/dc), we notice this issue. The error is inconsistent and work around is unloading/reloading drivers. The resources not being cleaned up properly, could it be the cause? Thanks!
16
tee: '/sys/bus/pci/devices/0000:6b:00.0/sriov_numvfs': Invalid argument
16
tee: '/sys/bus/pci/devices/0000:e8:00.0/sriov_numvfs': Invalid argument

@hj-johannes-lee can you reproduce?

I cannot reproduce it..
Could you just show what you did?

@vbedida79 we cannot reproduce. is this still an issue on your side?

Checked it on OCP 4.14 OCP, did not notice any error. Thanks.
Closing this issue.


Resolved disallowed-html-tag build warnings in azautomation

Docs Build status updates of commit 40fdc92:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azps-7.5.0/Az.Automation/Start-AzAutomationDscCompilationJob.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-8.0.0/Az.Automation/Start-AzAutomationDscCompilationJob.md
:white_check_mark:Succeeded
View (azps-8.0.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


add finishTodo command and tests for finishTodo

Coverage decreased (-0.4%) to 81.685% when pulling a7ded7cab5b45efdebb365d3d3040544762c4023 on linnnruoo:linruo/todotask into a6d0001cda14b8b7661a699f99dd76ea32b4ad59 on CS2113-AY1819S1-W12-1:master.


DFE pollutes stdout

DART_CONFIGURATION=DebugX64 out/DebugX64/dart --snapshot=/b/s/w/ir/out/DebugX64/generated_compilations/app_jitk-linux-debug-x64/tests_standalone_2_io_unix_socket_test/out.jitsnapshot --snapshot-kind=app-jit -Dtest_runner.configuration=app_jitk-linux-debug-x64 --ignore-unrecognized-flags --packages=/b/s/w/ir/.dart_tool/package_config.json /b/s/w/ir/tests/standalone_2/io/unix_socket_test.dart

exit code:
255

stdout:
Info: Compiling without sound null safety
server started a socket Instance of '_RawSocket'
client sent a message
client sent a message without control data
server received message Instance of 'SocketMessage'
server received messageData Hello
server has written to the Instance of '_RandomAccessFile' file
server received message Instance of 'SocketMessage'
server received messageData EmptyMessage
server socket got readClosed
server started a socket Instance of '_RawSocket'
client sent a message
server received message Instance of 'SocketMessage'
messageData: H
server received message Instance of 'SocketMessage'
messageData: i
server socket got readClosed
server started a socket Instance of '_RawSocket'
stdout:>Info: Compiling without sound null safety
<
stdout:>client sent a message<
stdout:>
<
stderr:>client wrote to stderr<
stderr:>
<
stdout:>Hello, server!
<
stderr:>Hello, server too!
<

stderr:
Unhandled exception:
Expect.fail('Unexpected exception Expect.equals(at index 0: Expected <client sent a message\x0AHello, server!\x0A...>, Found: <Info: Compiling without sound null safety\x0Aclient sent a message\x0AHello, server!\x0A...>) fails. is thrown')
#0      Expect._fail (package:expect/expect.dart:698:5)
#1      Expect.fail (package:expect/expect.dart:280:5)
#2      main.<anonymous closure> (file:///b/s/w/ir/tests/standalone_2/io/unix_socket_test.dart:972:14)
#3      _RootZone.runBinary (dart:async/zone.dart:1665:54)
#4      runZonedGuarded.<anonymous closure> (dart:async/zone.dart:1775:18)
#5      _Zone._processUncaughtError (dart:async/zone.dart:1080:14)
#6      _ZoneDelegate.handleUncaughtError (dart:async/zone.dart:944:23)
#7      runZonedGuarded.<anonymous closure> (dart:async/zone.dart:1778:16)
#8      _Zone._processUncaughtError (dart:async/zone.dart:1080:14)
#9      _CustomZone.handleUncaughtError (dart:async/zone.dart:1284:5)
#10     Future._propagateToListeners (dart:async/future_impl.dart:717:16)
#11     Future._completeError (dart:async/future_impl.dart:575:5)
#12     _rootRunBinary (dart:async/zone.dart:1422:47)
#13     _CustomZone.runBinary (dart:async/zone.dart:1314:19)
#14     _FutureListener.handleError (dart:async/future_impl.dart:162:22)
#15     Future._propagateToListeners.handleError (dart:async/future_impl.dart:779:47)
#16     Future._propagateToListeners (dart:async/future_impl.dart:800:13)
#17     Future._completeError (dart:async/future_impl.dart:575:5)
#18     _rootRunUnary (dart:async/zone.dart:1406:47)
#19     _CustomZone.runUnary (dart:async/zone.dart:1307:19)
<asynchronous suspension>

--- Re-run this test:
python3 tools/test.py -n app_jitk-linux-debug-x64 standalone_2/io/unix_socket_test

Diagnostic output should go to stderr, not stdout, so that program used in pipelining can enable diagnostics without breaking. Cf. https://github.com/dart-lang/sdk/issues/32134

I think the diagnostics output is sent to stdout in the kernel service.

CL in progress https://dart-review.googlesource.com/c/sdk/+/268102

Landed https://dart-review.googlesource.com/c/sdk/+/268102


Add test for navigator_state.restorable_push_and_remove_until.0.dart

Contributes to https://github.com/flutter/flutter/issues/130459
It adds a test for

examples/api/lib/widgets/navigator/navigator_state.restorable_push_and_remove_until.0.dart

Pre-launch Checklist

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

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

One of the test is failing:
01:15 +185 ~2: /b/s/w/ir/x/w/flutter/examples/api/test/widgets/navigator/navigator_state.restorable_push_and_remove_until.0_test.dart: It pushes a restorable route and restores it
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following assertion was thrown running a test:
The current widget tree did not inject the root bucket of the RestorationManager and therefore no
restoration data has been collected to restore from. Did you forget to wrap your widget tree in a
RootRestorationScope?
'package:flutter_test/src/widget_tester.dart':
Failed assertion: line 768 pos 7: 'binding.restorationManager.debugRootBucketAccessed'

When the exception was thrown, this was the stack:
#2      WidgetTester.restartAndRestore (package:flutter_test/src/widget_tester.dart:768:7)
#3      main.<anonymous closure> (file:///b/s/w/ir/x/w/flutter/examples/api/test/widgets/navigator/navigator_state.restorable_push_and_remove_until.0_test.dart:38:18)
<asynchronous suspension>
#4      testWidgets.<anonymous closure>.<anonymous closure> (package:flutter_test/src/widget_tester.dart:189:15)
<asynchronous suspension>
#5      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:1027:5)
<asynchronous suspension>
<asynchronous suspension>
(elided 3 frames from class _AssertionError and package:stack_trace)

One of the test is failing:

Oops, I forgot to commit the modifications of the example


[Bug Report]  Conflict with the Media Extended plugin.

When the Media Extended plugin is enabled, checking tasks in the panel does not update the tasks in the notes to completed in real-time.

Hi @Leftzhf , could you confirm if the status in the origin md file where this task item is changed in real-time?

Hi @Leftzhf , could you confirm if the status in the origin md file where this task item is changed in real-time?
No changes have occurred.


Handle invite event

Brings in https://github.com/erming/shout/pull/631

Will take a look, thanks! Could you squash commits please?

I could invite someone with /invite someone #channel but not with /invite someone while being inside a channel. I know it's not the spec per se, but it's pretty standard (and useful!) to be able to invite someone in the current channel. Any chance you can add that?
Also, I was not being able to invite someone when not an op on a Freenode channel. I get that it's not a bug of this PR, but that's still pretty frustrating...

2 concerns detailed above, but apart from that it's looking good, that's pretty cool! Close to not be behind Shout anymore lol...

I could invite someone with /invite someone #channel but not with /invite someone while being inside a channel.

Create a separate issue for it, this is not what this PR is for.

Also, I was not being able to invite someone when not an op on a Freenode channel.

There's a channel mode to change that.

Create a separate issue for it, this is not what this PR is for.

Meh, I could argue but I won't. I'll open an issue and will actually code the feature myself if you don't mind: with all this organization/fork/PR-reviews-by-myself-on-shout stuff, I haven't been able to code a single line for weeks or more... Very frustrating! :-)
In the meantime, :+1:. Asking for a second review here because it's a bit more than minor.

Create a separate issue for it, this is not what this PR is for.

Alright, I wrote this, will open a PR with the code as soon as this one gets merged.

Alright, looks good to me. Merging.

Perfect, I'll open a PR with my addition (see above) tonight after work :-)


Replace structured search text pui

Changes in this PR:
Replace structured search text pui including url change
Jira card / Rollbar error (etc)
https://national-archives.atlassian.net/jira/software/projects/FCL/boards/136?selectedIssue=FCL-381
Screenshots of UI changes:
Before

After

[ ] Requires env variable(s) to be updated

We should probably add a redirect from the old URL to the new one, so existing links people have don't break.

@jacksonj04 okay added hopefully the correct way , please review


Bugfix/cmake config binaries

This local for github actions

Can you please stash the 26 commits :-)

Can you please stash the 26 commits :-)


Zoom in Firefox bug (Have find the problem)

Due to these reasons, in Firefox using the mouse wheel zooming will lead to stuck.

Yup. You're right.
line 1182 should read wheelDeltaY = -e.detail / Math.abs(e.detail);
https://github.com/cubiq/iscroll/pull/699#issuecomment-76771321


Fix Issue#55

This PR fixes issue #55 by changing any year that occurs before 1980 to 1980 so that the archive gets created. It also outputs a warning about the date change during compression.

Thank you for submitting this PR.
Before providing my review, I would like to hear from some others about the proposed changes.
@SteveL-MSFT: The APIs used to create an archive document the DateTime limits on the LastWriteTime attribute here. In the event when older documents are added to an archive, do we simply want to move forward their timestamp to within the allowed range, with a warning as is being done in this PR? I think it's a good compromise given the limits of the API being used for compression, although I would suggest the warning text should change to more clearly identify why it is warning the user, and I think the date should be set to the earliest possible date rather than just changing the year, but I'll handle those in a review after I hear your thoughts on this approach to this issue.

Given the limits of the api, I think a warning is sufficient.  Setting it to the earliest allowed date seems reasonable to me.

Thanks guys. I wasn't sure what to do exactly or what to log as I don't think this issue is all that straightforward to try and handle. The least complex thing I could think of was to bump up the date and warn.
I updated the PR to use [DateTime]::Parse and an ISO 8601 date with the default value in whatever the local time is for the machine performing the archiving. If you have any ideas about improving the log message let me know.

Thanks Ritch. This is looking good. I have two main thoughts after reviewing the latest changes:

In order to accept this PR, we need Pester tests added to the Pester test file to make sure this works and is properly tested with Pester. Tests should specifically include files after 1980, where you validate that the LastWriteTime of the file before/after archiving (after extraction) is the same as it was in the beginning, and files before 1980, where you validate that the LastWriteTime after archiving is set to 1980-01-01T00:00:00. Make sense?

The warning message should do more than just tell the end user when a file has a LastWriteTime earlier than 1980. It should tell them why that is a warning. I also think the path should be in single quotes since it may contain spaces. Maybe something more like this:
Write-Warning "'$currentFilePath' has LastWriteTime earlier than 1980. Any files with LastWriteTime values earlier than 1980 will be stored in the archive with a LastWriteTime of 1980-01-01T00:00:00."

Would you please make those additional changes and then update the PR?

I'm ok with the explicit parse. You're right, it's less ambiguous.

Given the rounding issue, I like your approach. The key point with this test was to verify that files with old dates had their dates properly updated post Expand-Archive, and that files with newer dates were unchanged, and using an inverted condition does that. Thanks for making those additional changes, Ritch! I'll do my final review now.

LGTM. Merging...


Getting compilation error.

Followed all the steps provided in the sample. I have created console application using .net framework 4.5.2
Getting below error. The version detail i kept  new Version (2,0,0).
Do i missing anything ?
Unhandled Exception: System.ArgumentException: Version must not contain revision. Only first three parts (major, minor and build) are supported by Wix.
1>  Parameter name: version
1>     at NineDigit.WixSharpExtensions.ProjectExtensions.SetProjectInfo[TProject](TProject project, Guid upgradeCode, ProjectNameInfo projectName, Version version, InstallScope installScope)
1>     at NineDigit.WixSharpExtensions.ProjectExtensions.SetProjectInfo[TProject](TProject project, Guid upgradeCode, String name, String description, Version version, InstallScope installScope)
1>     at Installer.Program.Main(String[] args) in D:\SampleCode\Installer\Program.cs:line 71

Hi! Can you copy snippet of your code, where the version is declared?

Sorry for the delay. The code i have shared only version declaration and  ManagedProject.
Not shared full code. do you need please let me know.
` static void Main(string[] args)
{
var version = new Version(1, 0, 0); // you may want to get version from assembly dynamically with
Tasks.GetVersionFromFile("path-to-your-app-assembly");
        var isDebug = IsDebug();
        var netCoreTargetVersion = new Version(3, 1);
        var solutionDirectory = Directory.GetParent(Directory.GetCurrentDirectory()).FullName;
        var sourceDirectoryPath = "my/app/publish/directory"; // directory where your app is published

        var setupProjectDirectoryPath = Directory.GetCurrentDirectory();
        var assetsFolderDirectoryPath = Path.Combine(setupProjectDirectoryPath, "Assets");
        var filesFolderDirectoryPath = Path.Combine(assetsFolderDirectoryPath, "Files");
        var filesProgramMenuDirectoryPath = Path.Combine(filesFolderDirectoryPath, "ProgramMenu");
        var localizationsFolderDirectoryPath = Path.Combine(assetsFolderDirectoryPath, "Localizations");
        var outputInstallerDirectoryPath = Path.Combine(solutionDirectory, "RELEASES");
        var outputInstallerFileName = InstallerOutputFileNamePrefix + version.ToString(3);

        var msiFilePath = new ManagedProject()
            .SetProjectInfo(
                upgradeCode: ProductUpgradeCode, // unique for this project; same value for all versions; must not be changed between versions.
                name: ProductFullName,
                description: ProductDescription,
                version: version) //Error.
            .SetControlPanelInfo(`

some more information.
https://github.com/ninedigit/wixsharpextensions/wiki/Setup-for-ASP.NET-Core-application
followed all the steps provided in the url. for that only i am getting the issue.

resolved the issue by setting correct version.


34 | Find first and last position of element in sorted array | Python

Leetcode Question no. - 34
Problem name - Find first and last position of element in sorted array
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed

Programming Language

[x] Python

Happy Coding

Hey @pawangeek . I would like to contribute. Can you please assign this issue to me?


fix bug with "center" attribute for panels in LISP and Perl

I found this difference in the code generation for panels, in LISP and Perl
I just copied the corresponding line from widgets/panel/codegen.py.

The first pull request :-)
Thanks. The support for non-Python languages is based on the old test cases and on a best effort for new features, so fixes are welcome.


Feat: add no-setter-return

This PR adds no-setter-return rule, requested in #4
Please merge this PR  after #6, because some test cases cannot be passed without #6.

@RDambrosio016 thanks for the detailed review! Addressed all the points. Please check again :)

Looks good to me!


🛠 Tooling: creation script e2e test fails on main with knip for vitest.config.ts

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
https://github.com/JoshuaKGoldberg/template-typescript-node-package/actions/runs/5986382888/job/16239455337:
> knip

Unused devDependencies (1)
console-fail-test  package.json
Unresolved imports (1)
/home/runner/work/template-typescript-node-package/template-typescript-node-package/test-repository/console-fail-test/setup  vitest.config.ts
 ELIFECYCLE  Command failed with exit code 2.

Additional Info
No response

https://github.com/webpro/knip/issues/219.


added polar-clock screensaver (3.0)

Thanks @fanquake , just fixed that.

Thank you for the contribution. It needed some fixes, so they were made in fd3fa998239dc5fc9c63c175373a18041d4a8e51. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Unable to resolve CodeQL SSRF warning for a HTTP request function that takes pip package names as input

I'm working on a function that returns a HTTP response from https://pypi.org/simple/ when Python's pip installer requests it for a package. When pushing my code onto GitHub, the CodeQL checks warn of the risk of server side request forgery (SSRF), and asks me to create validation checks for the "user-defined input" (which is pip, in this case).
I have already made many attempts at validating the URL to satisfy this SSRF warning, but GitHub CodeQL has not accepted any of them so far. How can I rewrite the following to satisfy GitHub CodeQL's requirements for guarding against SSRF?
The relevant block of code:
import requests
from fastapi import APIRouter, Response

pypi = APIRouter(prefix="/pypi", tags=["bootstrap"])

@pypi.get("/{package}/", response_class=Response)
def get_pypi_package_downloads_list(package: str) -> Response:
    """
    Obtain list of all package downloads from PyPI via the simple API (PEP 503).
    """
    url = f"https://pypi.org/simple/{package}"
    full_path_response = requests.get(url)

The following is a non-exhaustive overview of attempts I've tried in order to satisfy that SSRF warning. However, none of them have worked for me.

    # Attempt 1
    # Check that it's a PyPI URL
    url = f"https://pypi.org/simple/{package}"
    if "pypi" in url:
        full_path_response = requests.get(url)
    else:
        raise ValueError("This is not a valid package")

    # Attempt 2
    # Validate that package name is alphanumeric (allow _ and -)
    if package.replace("_", "").replace("-", "").isalnum():
        url = f"https://pypi.org/simple/{package}"
        full_path_response = requests.get(url)
    else:
        raise ValueError("This is not a valid package")

    # Attempt 3
    # Check that it's a valid connection
    with requests.get("https://pypi.org/simple/{package}") as http_response:
        if http_response.status_code == 200:
            full_path_response = http_response
        else:
            raise ValueError("This is not a valid package")

    # Attempt 4
    # Tried using RegEx matching to validate package name
    if re.match(r"^[a-z0-9\_\-]+$", package):
        full_path_response = requests.get(f"https://pypi.org/simple/{package}")
    else:
        raise ValueError("This is not a valid package")

    # Attempt 5
    # Use urllib.parse.urlparse to parse and validate the url
    def validate_url(url: str) -> bool:
        parsed_url = urlparse(url)
        if parsed_url.scheme == "https" and parsed_url.hostname == "pypi.org":
            return True
        else:
            return False

    def validate_package(package: str) -> bool:
        if package.replace("_", "").replace("-", "").isalnum():
            return True
        else:
            return False

    # Validate package and URL
    if validate_package(package) and validate_url(f"https://pypi.org/simple/{package}"):
        full_path_response = requests.get(
            f"https://pypi.org/simple/{package}"
        )  # Get response from PyPI
    else:
        raise ValueError("This is not a valid package")

    # Attempt 6
    # Using a Pydantic model
    from pydantic import BaseModel, HttpUrl, ValidationError

    class UrlValidator(BaseModel):
        url: HttpUrl

    def validate(url: str):
        try:
            UrlValidator(url=url)
        except ValidationError:
            log.error(f"{url} was not a valid URL")
            return False
        else:
            log.info(f"{url} was a valid URL")
            return True

    # Attempt at URL validation to satisfy GitHub CodeQL requirements
    url = f"https://pypi.org/simple/{package}"
    if validate(url):
        full_path_response = requests.get(url)

    # Attempt 7
    # Encoding string before injection
    from urllib.parse import quote_plus

    def _validate_package_name(package: str) -> bool:
        # Check that it only contains alphanumerics, "_", or "-", and isn't excessively long
        if re.match(r"^[a-z0-9\-\_]+$", package):
            return True
        else:
            return False

    def _get_full_path_response(package: str) -> requests.Response:
        # Sanitise string
        package_clean = quote_plus(package)
        print(f"Cleaned package: {package_clean}")

        # Validation checks
        if _validate_package_name(package_clean):
            url = f"https://pypi.org/simple/{package_clean}"
            print(f"URL: {url}")
            return requests.get(url)
        else:
            raise ValueError(f"{package_clean} is not a valid package name")

    # Attempt 8
    # The nuclear option of maintaining a list of approved packages
    approved_packages: list = [pkg.lower() for pkg in approved_packages]  # List of package names from running `conda env list`

    # Validate package and URL
    if package.lower() in approved_packages:
        url = f"https://pypi.org/simple/{package}"
        full_path_response = requests.get(url)
    else:
        raise ValueError(f"{package} is not a valid package name")

Thanks!

👋 Hey @tieneupin! I'm going to move this issue to github/codeql, since that is more suited for discussions around potential false positives or false negatives from CodeQL queries.


Allow users to upload their public keys

Add an option to allow regular users to upload their public keys in user profile.

Add an option to allow regular users to upload their public keys in user profile.

What do you meaan? It's already the case

I logged with a regular, non-admin user, and menu option "My public keys" is missing from menu.
Also, logged as admin, I didn't found an option to configure to allow users to setup their keys.

http://redmine-git-hosting.io/get_started/#step-10-add-your-ssh-keys

Ok, just added a test user, non-admin.
Logged, and "My public keys" is missing from user profile.
Users should be able to upload their public keys whatever is involved in a project or not.
Later he may select from his public keys some specific key for some specific project.

Users should be able to upload their public keys whatever is involved in a project or not.

I know. This may be improved but it's not guaranted due to how Redmine works.

Later he may select from his public keys some specific key for some specific project.

This would add too much complexity.

If it works for an admin user, I think it should work for a regular user too ;)
This approach is done also in GitLab.
I just want to add an user that will be involved in development and he is not able to upload his keys :(

If it works for an admin user, I think it should work for a regular user too ;)

This is due to how roles work in Redmine.

This approach is done also in GitLab.

I know. And Gitlab don't use Gitolite too ;)

I just want to add an user that will be involved in development and he is not able to upload his keys :(

Add the user as a project member.

OK, big thanks for clarification!
Anyway, hope to see this at least in v.2.0 ;)

OK, big thanks for clarification!

You're welcome.

Anyway, hope to see this at least in v.2.0 ;)

I'll see what I can do ;)


Example in README.md is dated

The example in the README.md is dated. I'm not even sure it will compile. Someone should test it to make sure ;-).
It would also be awesome if it used .net 5's single file functionality.

Proposed fix: #1162

Thank you.


check_python_dependencies.py detected by Comodo as a trojan  (IDFGH-4691)

After using https://dl.espressif.com/dl/esp-idf-tools-setup-2.4.exe to install the tools, when esp-idf cmd or powershell is opened comodo detects check_python_dependencies.py as a trojan(Generic.Trojan.C@6@1 )
(ESP-IDF v4.2-47-g2532ddd9f-dirty)
(ESP32-S2-kaluga)
(Windows 10 Pro)

Thank you @rouzbeh-afrasiabi for reporting this. We will look into this.

@rouzbeh-afrasiabi Thank you for reporting the problem. This is false positive identification of potential threat. You can check the .py file source code or run the full scan on folder.
Comodo has feature Auto-Containment which is enabled by default. This feature causes that anything which is not known to Commodo is running in virtualized environment. This will cause pop-up of the message on the first sight. The file can be then found in the list in Commodo - Advanced View - Unrecognized files.
Unfortunately, this feature will also cause problem using any idf.py command. The build process won't run, command line will just idly wait for Commodo to finish containerization. Solution: Turn off the Auto-Containment mode.
Go to Commodo Settings - Advanced View - Auto-Containment - Enabled and change the value to Disabled. Click Ok.
Now the build will run.

@rouzbeh-afrasiabi Hi! Also you can submit the detection as a false positive, as described in https://help.comodo.com/topic-394-1-767-12121-.html , it could help other comodo users.

Thank you @rouzbeh-afrasiabi for the link. It looks like a viable solution.

Thank you @dobairoland and @georgik. I am going to close this since the issue is due to Comodo.


Add a change log

[ci skip]

Content wise, LGTM
But my understanding of semver has always been that internals don't cause a bump. Are you specifically aware of someone using jade_encode_html_rules and jade_encode_char? If I recall, they were introduced in 1.11.0, so it would surprise me.
I guess it just feels more like a 1.2 rather than a 2.0

I'd say if we're being really strict (which we probably should be) then it could be considered a breaking change.  Version numbers are cheap after all.
To avoid repeating this problem though, rather than just stating that those things are considered internal, we should make them inaccessible from build (i.e. build should not let you do build(['jade_encode_html_rules']) or build(['jade_classes_object']).  That way they are less exposed.

I'd say if we're being really strict (which we probably should be) then it could be considered a breaking change. Version numbers are cheap after all.

+1.

we should make them inaccessible from build

Done in 9e4f31e.


Update simple.md

What this PR Does ?
fixes #3157

@MdSahil-oss https://plone.org/community/gsoc/2022/ideas
Note: we think trying to fix easy issues is a nice way to start learning about the Plone development process, but we are not necessarily expecting a pull request from you, if you have something valuable to contribute, that's very nice, but if that's not the case, that is totally fine and we will not misconsider your GSoC application for that. Also, if you do have an interesting PR, be aware review and merge might take time as the core contributors can be busy.

also, by fixing documentation I have no clue if you are able to tackle complex JS issues or problems or if you know ReactJS to complete a successful GSOC.
Btw, this fix is not correct, it should be:
    location ~ / {
        location ~* \.(js|jsx|css|less|swf|eot|ttf|otf|woff|woff2)$ {
            add_header Cache-Control "public";
            expires +1y;
            proxy_pass http://volto;
        }
        location ~* static.*\.(ico|jpg|jpeg|png|gif|svg)$ {
            add_header Cache-Control "public";
            expires +1y;
            proxy_pass http://volto;
        }

        proxy_set_header        Host $host;
        proxy_set_header        X-Real-IP $remote_addr;
        proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header        X-Forwarded-Proto $scheme;
        proxy_redirect http:// https://;
        proxy_pass http://volto;
    }

@sneridagh Done! as per your suggestion

@MdSahil-oss please add a changelog entry. Every pull request in this repo requires one.

@MdSahil-oss this PR cannot be merged in its current state. Please resolve https://github.com/plone/volto/pull/3159#pullrequestreview-942176719, and merge master into your branch and push, which should also resolve the merge conflict that is blocking the merge. Thank you!

@MdSahil-oss this PR cannot be merged in its current state. Please resolve #3159 (review), and merge master into your branch and push, which should also resolve the merge conflict that is blocking the merge. Thank you!

@stevepiercy  because of some issues, Let me close this PR first and creating a new PR for this issue.


Microsoft.Bot.Builder.Dialogs.Adaptive" Version="4.6.0-preview2 is missing Memory Extenstion

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
Microsoft.Bot.Builder.Dialogs.Adaptive" Version="4.6.0-preview2
Describe the bug
I have updated my updated my bot to 4.6 GA and using Microsoft.Bot.Builder.Dialogs.Adaptive" Version="4.6.0-preview2
This package is missing DialogContextExtensions
https://github.com/microsoft/botbuilder-dotnet/tree/4.6/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Memory
The only option i see is to use Microsoft.Bot.Builder.Dialogs.Adaptive" Version="4.7.0-
but in that case whole sdk will be preview version.
from where i can get a nuget package for 4.6 which include
https://github.com/microsoft/botbuilder-dotnet/tree/4.6/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Memory
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

The package includes it, you need to have a using statement for
using Microsoft.Bot.Builder.Dialogs.Memory;

Then you should see on dialogContext.GetState()

Thanks it worked.


Inspect deprecation warning

Hi, first thanks to all the contributors for the great job done so far!
I am opening this issue because I am using alpha_vantage in my trading script and when I run my unit test (pytest) I get this warning:
/home/alberto/.local/lib/python3.6/site-packages/alpha_vantage/alphavantage.py:93: DeprecationWarning: inspect.getargspec() is deprecated, use inspect.signature() or inspect.getfullargspec()
    argspec = inspect.getargspec(func)

I have resolved this locally by just replacing getargspec with getfullargspec.
Should I open a PR for this?

Oh thanks, yeah it is a good idea. Do the PR and I'll merge it. :)

Merged ;)


utc_offset value now incorrect for times in daylight saving

Hi there,
A few days ago (9 September 2015), the "utc_offset" value returned by meetup for events scheduled during daylight savings changed.  Previously it gave the offset including any local daylight savings. Now it gives the non-daylight savings offset.  So without knowing the local timezone we can not work out the local time.
For example, the event http://www.meetup.com/V-Stars-Sydney/events/bqmmklytpbkb/ occurs at 6pm, which is shown correctly on the Meetup website.  The api used to return:
1446879600000
<utc_offset>39600000</utc_offset>
but now it returns:
1446879600000
<utc_offset>36000000</utc_offset>
Notice that the utc_offset is now +10:00, which is non-DST.  It should be +11:00.
Is this a bug in meetup? Or am I missing something?
Greg

As of a few days ago, this problem seems to have been fixed.  The "utc_offset" field is now correct (again!).

Yep. Sorry for not posting back sooner. We recently launched a fix for this.

I've just found a case where the utc_offset field is incorrect.  For event http://www.meetup.com/melbourne-vegan/events/qjmxhjyvfbrb/ it returns 36000000, which is non-DST, but the time is during DST.  Not sure if this is a general error or a problem just with this event. The event time display OK on meetup.com.  The error is just with the API value.


What's the license?

Could you add a LICENSE file to the repo? E.g. MIT?

Added license file ae1ae5c7c5c9bbcdf4efc3ed44279d35526fcf25
😉


A little doubt about the EPOLL WAIT method.

Hi, community,
The netty_epoll_native_epollWait0 method uses timerfd to control the timeout. But epoll_wait can specify the wait time directly in the parameters. Why don't we use epoll_wait directly instead of timerfd?
code like this:
do {
    result = epoll_wait(efd, ev, len, tcSec);
    return result;
} while((err = errno) == EINTR);

@carryxyh because timerfd allows for nanoseconds ;)

Thx a lot :)
Very fast response speed!


Can also return -1

There is one edge case where this will not return 0 - 1. If you are using the iOS simulator and you have a desktop (or a VM) it will return -1. This happens because the iOS simulator sets the battery based on the computer but a desktop or VM has no battery. If you could update the documentation to say that, that would be great.

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

ID: 85864d78-2fe1-c7a6-7588-1b1a67c15612
Version Independent ID: 1155b7e1-7fbc-791b-16d0-f3e51fd5c0f6
Content: Battery.ChargeLevel Property (Xamarin.Essentials)
Content Source: docs/en/Xamarin.Essentials/Battery.xml
Product: xamarin
Technology: xamarin-essentials
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Fixed via https://github.com/xamarin/Essentials/pull/1049


Include metadata in output

Metadata is now added as properties on categories
Added deep links on categories, projects and tasks
Also had to add local server capabilities as I got throttled while testing

Closes #6


Port basic utilities (e.g. 2-qubit decomposition) out of google package

Several utilities got pulled into the google package along with the optimizers (because some were legitimately xmon specific). We should extract ones that make sense in other contexts back out if possible.

MergeRotations doesn't exist anymore.


"Stage selected ranges" command changes encoding to UTF-8

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.75.1
OS Version: Windows 11 Pro

Steps to Reproduce:

Modify several lines of a file encoded in a non-UTF-8 format (e.g .UTF8 with BOM).
Open source control view and stage only one line.
The staged file will be listed (and committed) as UTF-8, ignoring the original encoding.

This seems like a regression on https://github.com/Microsoft/vscode/issues/36219, but I am not able to comment on that issue so decided to open a new one. @joaomoreno mentioned this was solved by using git file system provider, but I have no idea what that is, or how it works, just using the default vanilla VS code install.
I use the this feature extensively for day-to-day work and have never noticed any issues until now, so I wonder if it is recent or if it is something that can be affected by specific git repo configurations.

/duplicate of #111915

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Incorporate new block plots from update_block_plots branch into the pipeline

This PR is based on Jai's work from the update_block_plots branch

Incorporate all plots into two tables BlockPatchPlots and BlockSubjectPositionPlots tables
Update BlockSubjectAnalysis to store running_preference_by_time and running_preference_by_wheel
Store ethogram data in BlockSubjectPositionPlots table
Update SciViz to render the new plots
Replacing the old plot table BlockPlots

Replacing #399

@jkbhagatio I've tested the newly committed changes. All good, this PR can be merged, feel free to do so (unless there's something else we should discuss/consider)


Logout Page

This is in the Django template in continuation with the  Sign Up and login page which overrides the default page. Confirmation will be taken from user if he/she wants to logout and then will be redirected to the login page .It functions properly.
Kindly preview and merge @codervivek5

can you please create this PR again..due to the unstructured of websitei have to revert this


converted code to Tensorflow 1 and changed skimage range

The code was converted using automatic conversion provided by Tensorflow.
Additionally, the most recent skimage was telling me its images needed to be between -1 and 1, the opposite of what your code and comments said. I changed that, adding a clip since some values were (very slightly) larger than 1.0. This is slightly suspicious...
Note: I also have a python 3 version, which is NOT included in this PR, but you may be interested in.

Thanks for the PR! Honestly I haven't touched this repo in a while so I'll need to re-download the data, but I'll try to test this and merge this weekend

Great, however if I push the py3, it may not be compatible with py2 anymore. Is that ok?

What did you change to make it work with python3? I’d love to have it be cross-compatible if possible

@martinosorb, @dyelax
Hey, I'm trying to run the code on my own data and I've made all the changes in this PR along with other issues of this repo. Still its throwing this error:
TypeError: Value passed to parameter 'shape' has DataType float32 not in list of allowed values: int32, int64
Would you help please.


[POC] Add deploy preview to PR body

Changes

grab the first 5 docs/data/*.md files that are created or updated and add to the comment.

What I learned
At first, I tried to move this deploy preview comment to github actions but failed due to the permission scopes.
In this article, the pull request from fork does not receive the repo's secrets and will always have read scope when the workflow runs. Danger cannot comment on the PR, so I revert the change and just updated the existing script to grab markdown files to the comment.

[ ] I have followed (at least) the PR section of the contributing guide.

In this article, the pull request from fork does not receive the repo's secrets and will always have read scope when the workflow runs.

I was a bit surprised by this because vale is a github action and it adds comments to the PR.
To be more precise, PR from a forked repository does not receive the write permission by default (github docs). But on the same page, you have an example where the action has writing permissions.

In this article, the pull request from fork does not receive the repo's secrets and will always have read scope when the workflow runs.

I was a bit surprised by this because vale is a github action and it adds comments to the PR.
To be more precise, PR from a forked repository does not receive the write permission by default (github docs). But on the same page, you have an example where the action has writing permissions.

The example you provide uses pull_request_target which mentioned in the security article that it should not be used if we have yarn install script.
We could write a custom script instead of using danger if we want to use pull_request_target in github actions but I don't want to go that far.

@siriwatknp Nice 👌

We could write a custom script instead of using danger if we want to use pull_request_target in github actions but I don't want to go that far.

Two thoughts on this:

One of the design objectives with mui-bot was to group PR comment notifications. I feel that we are achieving this now. Instead of having two comments added to the PR that triggers a notification.
I wonder how helpful https://github.com/errata-ai/vale-action is. I'm more used to running a script and reporting what's wrong and then making the change. On one end, it sounds great to have the comment right where the issue happens, but in practice, it's in the code editor that it needs to be fixed, so it doesn't help much, and it might create noise in the review. Not sure 50/50.

@alexfauquette Are we good to merge? do you see any blockers?

In practice, it's in the code editor that it needs to be fixed

For those who want, there exists a VSCode extension to run vale. So you can get the warnings in code editors.
But adding comments in the file diff has two advantages:

It's readable. If you just get a comment in your PR saying Avoid using 'we' in file xxx.md, line xx it would be a pain
It's a heads-up for the reviewer. Form what I see, reviewers tend to make suggestions about how to rephrase, which is easier when you have a fresh pair of eyes


Restrict middleware execution to function type

Is there a way to restrict middleware to just execute for certain types of functions? For example, only execute authentication middleware (add tenant and user ids to FunctionContext.Items, etc.) for inbound HTTP triggers. If not, what be the proper way to get the function type within the middleware to filter out the middleware's functionality? CosmosDB trigger would not need the authentication middleware, for example.
We are using the method below, but if there's a correct/better way, please advise. If below is sufficient, please close the issue. Thanks.

We now have the UseWhen extension method which allows you to register the middleware with a predicate.
https://github.com/Azure/azure-functions-dotnet-worker/blob/e358ca5b804dc282e1c98b48ab6097caca0fc0fb/samples/CustomMiddleware/Program.cs#L23-L28


Allow parsing "json" which is wrapped in "[ ]", not "{ }"

Would it be possible the library to parse json which is wrapped in [ ], not { }, like this file (from the settings file of Sublime Text, if you're interested)):
[
    {
        "caption": "Word Wrap: Toggle",
        "command": "toggle_setting",
        "args": {"setting": "word_wrap"}
    },
    {
        "caption": "Convert Case: Upper Case",
        "command": "upper_case"
    },
    { "caption": "Indexing Status", "command": "show_progress_window" },
]

That should already work, @AllanLRH, since your example is almost valid JSON: The only "error" is the trailing comma after the last list item.
barely_json parses it just fine:
In [1]: import barely_json

In [2]: source = '''
   ...: [
   ...:     {
   ...:         "caption": "Word Wrap: Toggle",
   ...:         "command": "toggle_setting",
   ...:         "args": {"setting": "word_wrap"}
   ...:     },
   ...:     {
   ...:         "caption": "Convert Case: Upper Case",
   ...:         "command": "upper_case"
   ...:     },
   ...:     { "caption": "Indexing Status", "command": "show_progress_window" },
   ...: ]
   ...: '''

In [3]: print(barely_json.parse(source))
[{'caption': 'Word Wrap: Toggle', 'args': {'setting': 'word_wrap'}, 'command': 'toggle_setting'}, {'caption': 'Convert Case: Upper Case', 'command': 'upper_case'}, {'caption': 'Indexing Status', 'command': 'show_progress_window'}, u'']

Note the empty string at the end of the list due to the trailing comma.
I'm closing this ticket, feel free to re-open if there is something I've missed.


Paid but didn't receive a name

Coinbase finished payment but did not receive a response.
Account name: eosocgamedev
Receipt: https://commerce.coinbase.com/receipts/6HR9HYB8
Tx: https://live.blockcypher.com/btc/tx/072aad05a29ad0912a4cafb01f3d1a2116cba7fb0a7e47b59a6068c21a0a131b/

Any update here?


Make more uids nullable

This just makes more entity UIDs nullable instead of having them use default. There are still some default uids being used in hands code, which is still leading to some bugs. However, #5634 should fix that, and I don't want to cause unnecessary conflicts.

Fix more null/default entityUid issues and some instances where non-nullable Uids were compared to null. Some of the changes overlap with stuff paul changed in the fixes a bunch of warnings commit. Hopefully it not to hard to deal with those conflicts and re-apply the other warning fixes.

Fix more null/default entityUid issues and some instances where non-nullable Uids were compared to null. Some of the changes overlap with stuff paul changed in the fixes a bunch of warnings commit. Hopefully it not to hard to deal with those conflicts and re-apply the other warning fixes.

i will never reapply that cursed commit, dw


Login/Logoff Issue when using MVC5 + IdentityManager

Hi,
I currently have a MVC5 web application that uses cookie authentication and have also added Identity Manager via NuGet with cookie authentication as well.  Whenever I use the Identity Manager to log in, the application grants me access to Identity Manager as well as my web application (this is desired).  However, when I log out of my main application (MVC5 default logout button on the upper right), I seem to get logged off of my main application but am still logged into Identity Manager(this is not desired).
Conversely, if I click the default MVC5 "login", I only log into my main application.  Whenever I try to access Identity Manager, I am presented with the prompt to log in.  Upon pressing the button though, I am not shown a login screen, but direct access to the Identity Manager menu.
How do I go about configuring my solution so that one login/logout logs me into/out of both Identity Manager and my main application at once?  It fees like these two are not synced to each other and are on different sessions.
I have a suspicion that the following has something to do with what I'm seeing, but am not sure exactly what's causing it:
Startup.Auth.cs:
        public void ConfigureAuth(IAppBuilder app)
        {
            // Configure the db context, user manager and signin manager to use a single instance per request
            app.CreatePerOwinContext(ApplicationDbContext.Create);
            app.CreatePerOwinContext<ApplicationUserManager>(ApplicationUserManager.Create);
            app.CreatePerOwinContext<ApplicationSignInManager>(ApplicationSignInManager.Create);

    public static class SimpleIdentityManagerServiceExtensions
    {
        public static void ConfigureSimpleIdentityManagerService(this IdentityManagerServiceFactory factory, string connectionString)
        {
            factory.Register(new Registration<ApplicationDbContext>(resolver => ApplicationDbContext.Create()));
            factory.Register(new Registration<UserStore>());
            factory.Register(new Registration<RoleStore>());
            factory.Register(new Registration<ApplicationUserManager>());
            factory.Register(new Registration<ApplicationRoleManager>());
            factory.IdentityManagerService = new Registration<IIdentityManagerService, SimpleIdentityManagerService>();
        }

If you notice, the ApplicationUserManager is used twice.

It could be that you don't use the same cookie name for your application and for IdentityManager. Usually, you set to use cookie authentication with something like this in the Startup OWIN class:
app.UseCookieAuthentication(new Microsoft.Owin.Security.Cookies.CookieAuthenticationOptions
    {
        AuthenticationType = "Cookies"
    });

The value Cookies is the one used in the samples for IdentityServer3 and IdentityManager. However, the default code for an ASP.NET MVC 5 project with the Individual User Accounts template is:
app.UseCookieAuthentication(new CookieAuthenticationOptions
    {
        AuthenticationType = DefaultAuthenticationTypes.ApplicationCookie,
        LoginPath = new PathString("/Account/Login"),
        Provider = new CookieAuthenticationProvider
        {
            // Enables the application to validate the security stamp when the user logs in.
            // This is a security feature which is used when you change a password or add an external login to your account.
            OnValidateIdentity = SecurityStampValidator.OnValidateIdentity<ApplicationUserManager, ApplicationUser>(
                validateInterval: TimeSpan.FromMinutes(30),
                regenerateIdentity: (manager, user) => user.GenerateUserIdentityAsync(manager))
        }
    });

If you check the value for DefaultAuthenticationTypes.ApplicationCookie, it's ApplicationCookie.
To the best of my knowledge (I may be wrong), the authentication type in IdentityManager is set by the HostAuthenticationType property of the SecurityConfiguration property of IdentityManagerOptions. See the Security Configuration page in the wiki. The HostAuthenticationType is "The type of Katana authentication middleware to consult to identify the identity of the user". That's the UseCookieAuthentication  middleware. Make sure you don't have 2 UseCookieAuthentication instructions.
As for the double usage of your dependencies, you could instruct IdentityManager to use the instance that is set by the ASP.NET Identity configuration in the OWIN context. What I do is to use a delegate when I register the dependencies in IdentityManager. Here is a simplified example:
factory.Register(
    new Registration<ApplicationDbContext>(GetFromOwin<ApplicationDbContext>));
factory.Register(
    new Registration<ApplicationUserStore>(GetFromOwin<ApplicationUserStore>));
factory.Register(
    new Registration<ApplicationUserManager>(GetFromOwin<ApplicationUserManager>));

and the GetFromOwin method is:
private T GetFromOwin<T>(IDependencyResolver resolver)
{
    return HttpContext.Current.GetOwinContext().Get<T>();
}

I'm using IIS to host, so I have a dependency on Microsoft.Owin.Host.SystemWeb. The IDependencyResolver is not used, it's necessary to conform to the interface for Registration<T>. That would be different for a self-hosted app.

@SylChamber
My AuthenticationType cookie name are the same value, so I'm positive that's not the issue.  However, I did try your tip regarding using the instances from OWIN.  I get a "Method, delegate or event is expected" when I declare your method.  To get around this issue and see if registering an instance would make a difference, I tried running the following:
            var dbContextInstance = HttpContext.Current.GetOwinContext().Get<ApplicationDbContext>(null);

When .GetOwinContext() executes, I get the following error:
No owin.Environment item was found in the context
If I do not pass in any argument to .Get(), I get a compile-time error:
No overload for method 'Get' takes 0 arguments.
I find this strange as all usages of this function that I have found online do not use any arguments.

Sorry for the delay -- I was traveling (as usual).
IdentityManager uses the cookie to bootstrap getting an access token into the client-side JS. This client side token is being managed by this library: https://github.com/IdentityModel/oidc-token-manager.
So, for you what we want is on logout to somehow tell the client-side token manager to remove the token. One easy way would be to create the OidcTokenManger on your logout page (in JS) and remove the token. There's a removeToken API to do so.
We'd need to add a setting server-side to configure how the client-side token manager library stores the token. I think for you what we'd want is to change the storage

Thank you Brock, I will look into this.

I just hit a similar issue, but not with an MVC app.  I have my IdentityManger app running, using OpenID Connect to my IdentityServer3 instance.  Works great.  But Logout from idm only clears the idm cookie.  Next time I click Login, same experience as OP here, where IDSvr still has my user logged in and immediately returns access token for the prior (unwanted) user.
Seems that Log Off from [client app] (idm in this case) needs to be sure to send a Log Off to the IDSvr.  Is that supported already and I missed something?

I had this same problem using the sample application which comes with IdentityManager.  For those encountering this same issue, the fix (which took a while to find) is very simple:
Inside Startup.cs there is a class StartupWithHostCookiesSecurity, find the line
// AdditionalSignOutType = "oidc"
and uncomment it.  Problem solved.

@bqduong have you found a solution for IdManager Login problem you described?


[ROM] Consider sensor_ctrl alerts filter

Description
https://github.com/lowRISC/opentitan/pull/21870 added support for filtering sensor_ctrl alerts to handle spurious alerts coming from AST. Since the ROM has the ability to configure the alert handler via OTP settings, we should consider adding a mechanism to setup the sensor_ctrl alerts as part of the process.

HI @andreaskurth, is this issue now superseded by #23150? If so, please feel free to close this issue. Thanks

ROM sensor-ctrl configuration was added by #23513.


Provide more info about vk.xml path and fix generator path in README

Weird, why would the build fail for changing the README xD

Seems an unrelated change from Zig master. We can merge this in the mean time (after the suggestions have been resolved)

I think all the suggestions have been resolved

Thanks :)


Crash with extension 'Json to dart model' installed

Hi !
I found a little bug that make this wonderful extension crash when the extension json-to-dart is installed when we want to create a profile. It comes from the following function in the utils.ts file:
export async function getAllExtensions() {
  const extPath = getExtensionsPath();
  let extensions: ExtensionValue[] = [];
  let obsolete: string[] = []; // default value

  if (fs.existsSync(extPath + ".obsolete")) obsolete = Object.keys(loadJSON(extPath + ".obsolete"));

  let all = await readdir(extPath);

  await Promise.all(
    all.map(async (name) => {
      if ((await stat(extPath + platform_slash + name)).isDirectory() && !obsolete.includes(name)) {
        let info: PackageJson = require(extPath + platform_slash + name + platform_slash + "package.json");
        if (!info.__metadata?.id)
          console.log(info);
        extensions.push({
          id: `${info.publisher.toLowerCase()}.${info.name.toLowerCase()}`,
          uuid: info.__metadata.id,
          label: info.displayName || info.name,
          description: info.description,
        });
      }
    }),
  );

  return extensions.sort((a: any, b: any) => {
    // eslint-disable-next-line curly
    if (a.label > b.label) return -1;
    // eslint-disable-next-line curly
    else if (a.label < b.label) return 1;
    // eslint-disable-next-line curly
    else return 0;
  });
}

It crashes at line 214 because the __metadata object is not present into the info object for the extension json-to-dart.
I propose that if the use of this __metadata object is mandatory for the extension then that the extensions which don't have it are skipped. Meanwhile I'm going to uninstall json-to-dart.

After uninstalling this extension it appears that the issue persist (with the same extension whereas I just removed it). So maybe there is another cause to this non existence of the __metadata object.

@Totodore Thank you, I will definitely consider improving the extension in the near future, I have no free time right now.

@Totodore I am very sorry that I could not address this issue earlier. The bug has fixed in version 1.0.9


Is this something that we can use to add a experimental module from node in the browser

Like FS

Hi @Kali95739, this is for Node.js, but the question is valid (browsers don't really offer this).


Ember::Handlebars::Template no longer provides template_target

emberjs/ember-rails@8adb52a19caba96bddb1556577286f7027781721
breaks emblem-rails by replacing the template_target method

Should be resolved with latest version of gem


On Linux root permissions are needed?

It would be nice to have a more verbose output of velad install that shows all the commands that are executed as root. Additionally a --dry-run flag would be helpful to inspect those in advance.

On Linux root permissions is needed yes. Because VelaD run K3s as cluster in Linux which need root permissions to install.
Yes, velad install is a "huge" command which do a lot of things. --dry-run thing is a good advice.

closing with #48. --dry-run option is added.


Create Upcoming Event Section

Deliverables

Create an Upcoming Event Component.
For Image, as of now, you can use any non-copyrighted image.

Sample

Additional context

Tech/Framework:- React Js
External Library :- You can use React Bootstrap

Can I work on this issue under GSSoC'21? Kindly assign it to me.

@AshuKV at the same time we assign only one issue. So kindly complete the assigned issue. after you can work on it :)

@codewithvk can I work on this issue, plz? I am GSSoC 2021 participant.

@codewithvk When this issue will be passed on to other participant?

@codewithvk  and @Castella1313 As I am new to React, it is taking time to complete it, please let me give my best. This chance will help me to level up. I will try to make PR till tomorrow's EOD, if I failed to do so, any decision will be accepted.

@codewithvk  I tried using React bootstrap
I know it is looking just okay, it will be better if u suggest some corrections or I make a PR.
And apologizes for long delay.

@rutuja2229 Please change the background color and make PR.

@codewithvk  when this issue will be assigned to others?

What's the update @rutuja2229 ?

@codewithvk My exams were going on, I will change the background color and making PR.


[WIP] Field options

This adds a fields parameter to the browse API endpoints. it's super basic - no tests, validation, etc but it works!
refs #5463, #5343
cc @kevinansfield this should be enough to work with for the auto complete :)

For clarity, this PR is in a bit of a weird state. I found specifying columns works fine, except when you choose one which is custom / not part of the model itself. E.g.

posts.notathing - it returns an empty "notathing" field - where the name is wrapped in quotes
posts.url - it returns 2 forms of url one wrapped in quotes, and one not.

Need to upgrade bookshelf & knex and see if the behaviour is still there, and then perhaps raise a bug.

I've rebased & fixed the tests on this PR. It's considered rudimentary support, as there's no proper validation or tests for the behaviour as yet, hence not marking it as closing #5601. However, it should be enough to help us ship a version of #5343


Passing date from frontend to server

Environment

Operating System: Darwin
Node Version:     v22.0.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, runtimeConfig, modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
Not required IMO.
Describe the bug
When I pass a date to my backend, the date that I received is now a string, which means I have to set the string to a date and after can call getDay, etc.
const response = await $fetch('/api/checkout', {
method: 'POST',
headers: {
'Content-Type': 'application/json',
},
body: JSON.stringify({
startDate: new Date(range.value.start),
endDate: new Date(range.value.end),
}),
});
const { startDate, endDate }: CheckoutBody =
await readBody(event);
console.log(startDate.getDay());
Additional context
No response
Logs
[request error] [unhandled] [500] startDate.getDay is not a function

because startDate will be the stringified date. Bodies will be converted into JSON before being sent (if you sent a JS object) - in this case, you already to it on your own (but wouldn't have to)


On Github curently there is no way to show the contributor that CCLA/ECLA/ICLA docs are signed/not-signed.

As discussed with @dealako and @pranab-bajpai during 21-Dec Stand-up call, currently Github doesn't show any message/info. to the contributor the reason of gating and the status of signed/not-signed CCLA/ICLA/ECLA docs.
@pranab-bajpai to share the design/text for the same.

This feature has been completed in the v2. Signatures are shown that are incomplete, active and disabled. Both the company portal and the project control center can view/show the signatures.


Add libmpv support

Hey there,
First thanks for the great work you made here,
It would be cool if we can use libmpv to render the video, it will be more simple to cross plateform app between windows, linux and macos, and MPV have great built-in features that we could use directly in Iced if this would be implemented!
I made some research on my side and someone made a rust abstraction on the libmpv.c library : https://github.com/kohsine/libmpv2-rs
I don't really know if this is really feasible, because libmpv render.rs use OpenGL to render the video and Iced change their backend from glow to WGPU some times ago (and if I saw right, WGPU don't really support OpenGL), so tell me what you think about that!
I would be happy to help! even if my knowledge are limited in that area (I want to learn more on graphics related things)

Hi, thanks for your interest in this project.
The problem with using libmpv is, as you said, the fact that libmpv targets OpenGL, whereas Iced uses WGPU (and for which we have no way to control the backend WGPU selects without changes in Iced). So, I don't think it would be feasible to use libmpv.
Having said that, are there features you feel are missing in this library or platforms that you found don't work? In general, gstreamer supports quite a lot of features and should work across MacOS, Windows, and Linux.

This is a little example, but you could customize your experience by using custom .glsl shaders files
For example I have this repo which upscale in realtime the video I watch.
I don't know if this would be possible too here ?


Pylance can not realize auto-complete function

hello :>
When the mouse hovers over the code, it always shows "LOADING", and there is no auto-complete function.
i make sure that me settings are correct.
Is there any possibility of causing such a problem?

Hi @Glutaredoxin! Apologies for the delay in our response.
Would you mind sharing your logs as described here: https://github.com/microsoft/pylance-release/blob/main/TROUBLESHOOTING.md#filing-an-issue, so we can investigate this issue?
Thanks!


categorise ruff as a linter and formatter

ruff is also a formatter and should appear under both the Linter and Formatter category.
ref: https://docs.astral.sh/ruff/formatter/

cc: @williamboman request for review please, should be a simple one :)


Fix video removal

This pull request adds support for the new API endpoint, getting the video IDs was changed slightly as well. The Reverse engineering Javascript behind Google+ button question along with the YouTube Playlist Duration Sort script were used as a guide.
Fixes https://github.com/avallete/yt-playlists-delete-enhancer/issues/4.

Hi there !
Many thank's for the PR, I really appreciate your work. However, I had already worked on a fix locally linked to #5 (my bad for not marking me as assignee on the issue when working on it).
I think I'll go with my changes since:

I also reversed the way API call work, but reduced the amount of needed data to the minimal core (all context, screen size, etc...) of what YT internal api require to work. Reducing the amount of boilerplate code. Since the only really required data are 2 headers, and the client name and version.
Fixing the "removing video" issue doesn't fully address the #4 issue. Because there is two cause into this issue, first one, all remove doesn't work anymore (because of the API change), but also, the way I retrieve the "full playlist" on playlist containing 100+ videos must be changed (since the API change also changed the way YT internally browse the paginated playlists). I'm working on a fix, for this specific issue as well.

Since the only really required data are 2 headers, and the client name and version.

It looks like you can also omit the client name and version, and the HTTP headers except for X-Goog-AuthUser and X-Origin.

For what I have tried:

missing client -> clientName: Error 400
invalid version in client -> clientVersion: Error 404
missing client -> clientVersion: Error 400

About the headers, I tried some tests, and the minimum required for a successful remove are:
Content-Type: application/json
X-Goog-AuthUser: 0
X-Goog-PageId: (wihtout this one, the say it cannot find the ressource)
Authorization: (containing the SAPISIDHASH)
Origin: https://www.youtube.com
Cookie: (with all yt cookies)

However, for the first playlist data fetch (/playlist?list={id}&pbj=1) since it still work on the "old" api system, you need at least thoses headers:
X-YouTube-Client-Name: 1
X-YouTube-Client-Version: 2.20210303.01.00
X-Youtube-Identity-Token: (base64 encoded token)
Cookie: (yt cookies)

But yeah, I may still want to trim some unuseful headers (like X-Goog-Visitor-Id) in some refactoring.

It's odd that you need the X-Goog-PageId header, this is the debug output of a successful request in Python:
send: b'POST /youtubei/v1/browse/edit_playlist?key=<key> HTTP/1.1\r\nHost: www.youtube.com\r\nAccept-Encoding: identity\r\nUser-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.96 Safari/537.36,gzip(gfe)\r\nAccept: */*\r\nConnection: keep-alive\r\nAuthorization: SAPISIDHASH<PHONE_NUMBER>_651a513219323de962b75d8d1c50e8dba1d7fe7e\r\nx-goog-authuser: 0\r\nx-origin: https://www.youtube.com\r\nCookie: <cookies>\r\nContent-Length: 1067\r\nContent-Type: application/json\r\n\r\n'
send: b'{"actions": [{"removedVideoId": "<videoId>", "action": "ACTION_REMOVE_VIDEO_BY_VIDEO_ID"}], "playlistId": "<playlistId>", "params": "CAFAAQ%3D%3D", "context": {"client": {"browserName": "Chrome", "browserVersion": "88.0.4324.96", "clientFormatFactor": "UNKNOWN_FORM_FACTOR", "clientName": "WEB", "clientVersion": "2.20210303.01.00", "connectionType": "CONN_WIFI", "countryLocationInfo": {"countryCode": "US", "countrySource": "COUNTRY_SOURCE_IPGEO_INDEX"}, "deviceMake": "", "deviceModel": "", "geo": "US", "gl": "DE", "hl": "en-GB", "osName": "X11", "platform": "DESKTOP", "screenDensityFloat": 1, "screenHeightPoints": 1643, "screenPixelDensity": 3, "screenWidthPoints": 1094, "timeZone": "America/New_York", "userAgent": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.96 Safari/537.36,gzip(gfe)", "userInterfaceTheme": "USER_INTERFACE_THEME_DARK", "utcOffsetMinutes": -300, "visitorData": "Cgt1a2NuVFdSeHRGYyiTpIWCBg%3D%3D"}, "request": {"consistencyTokenJars": [], "internalExperimentFlags": []}}}'
reply: 'HTTP/1.1 200 OK\r\n'
header: Content-Type: application/json; charset=UTF-8
header: Vary: X-Origin
header: Vary: Referer
header: Date: Thu, 04 Mar 2021 21:33:08 GMT
header: Server: ESF
header: Cache-Control: private
header: X-XSS-Protection: 0
header: X-Frame-Options: SAMEORIGIN
header: X-Content-Type-Options: nosniff
header: Set-Cookie: SIDCC=AJi4QfG_hQdaP6luC_F91bTXjXueJuigU3ZpJ1IV7CcAdZ0hxb8ld5SA_PAqQAVfREY8diinbP0; expires=Fri, 04-Mar-2022 21:33:08 GMT; path=/; domain=.youtube.com; priority=high
header: Set-Cookie: __Secure-3PSIDCC=AJi4QfHc-xS-ScYZaSVo2wec2geqeDiLTv-vzo8odrEZlGm0eXB5722yGiBSl8Bawmb-3DKF2gc; expires=Fri, 04-Mar-2022 21:33:08 GMT; path=/; domain=.youtube.com; Secure; HttpOnly; priority=high; SameSite=none
header: Alt-Svc: h3-29=":443"; ma=2592000,h3-T051=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43"
header: Accept-Ranges: none
header: Vary: Origin,Accept-Encoding
header: Expires: Thu, 04 Mar 2021 21:33:08 GMT
header: Transfer-Encoding: chunked
DEBUG:urllib3.connectionpool:https://www.youtube.com:443 "POST /youtubei/v1/browse/edit_playlist?key=<key> HTTP/1.1" 200 None

It's strange indeed on my side I got a 404 error if trying to do the request without this header. Here is some reproducable CURL commands.
 I may want to use it instead of ACTION_REMOVE_VIDEO instead. It will be more consistent to use only videoId across all the code.


ESP32-S3 backward/anti-clockwise movement ignores speed settings

Hi. I tried the following code. I found that when the motor is asked to move backward. It doesn't respond to setSpeedInHz(). It uses the old speed set before an instruction telling it to move forward.
steppers[0]->setSpeedInHz(speed);  // speed is negative for backward, positive for forward
steppers[0]->setAcceleration(acceleration);  // acceleration is negative for backward movement, and positive for forward
steppers[0]->moveByAcceleration(acceleration);

// set the speed
steppers[0]->setSpeedInHz(speed);
// set the acceleration
steppers[0]->setAcceleration(acceleration);
int32_t new_position = distance + steppers[0]->getCurrentPosition();
steppers[0]->moveTo(new_position);

I tried applySpeedAcceleration();. It didn't help.
For example, if the motor is set to move forward at 10hz, 20hz, 30hz using the above code, you can see the speed change.
If the motor is set to move backward, at 10hz, 20hz, 30hz, there is no speed change. If the board is reset and the motor hasn't been moved before, the motor doesn't move.
If the motor is set to move forward at 10hz, then backward at 20hz, it moves backward at 10hz.
If the motor is set to move forward at 10hz, then backward at 30hz, it moves backward at 10hz. If it is further set to move at other speeds for backward movement, it will keep moving at 10hz. Unless a new speed is set with forward movement instruction, followed by a new backward movement call.
In short, setSpeedInHz doesn't work if followed by a backward movement call, evenif apply speed acceleration is called. Only setSpeedInHz followed by a forward movement call changes the speed.

most of the calls yields an return code. could you please check those in order to narrow down the issue ?


Make it accessible please

This is a feature request! Could you make react-tooltip accessible according to this spec https://www.accede-web.com/en/guidelines/rich-interface-components/tooltips-aria/ ?
Or at least white-list aria- role attributes on both trigger and a tooltip container?
Thank you!

@stsvilik whilst I agree this should be baked in, the tooltip component allows extra props to be passed to it.
So what I've done is simply:
foo

<ReactTooltip ... role='tooltip' id='myTooltip'>hello world</ReactTooltip>

@mildrenben how do you trigger tooltip on say hover? When I tried to add anything to the triggering element other than data-tooltip it failed to activate the tooltip.

@stsvilik I've only used this module for the first time today, so I don't know all the ins and outs of it, but my code is as follows:
<Star
  data-tip
  data-for="vaultStar_tooltip"
  className="vaultStar"
  aria-describedby="vaultStar_tooltip"
/>
<Tooltip
  id="vaultStar_tooltip"
  effect="solid"
  place="bottom"
  className="vaultStar_tooltip"
  role="tooltip"
>
  Added to the repo
</Tooltip>

Hi, can you guys please review V5 PR to validate this issue?
Thanks!


Unable to delete countryCode

It is impossible to deletedial code at the moment after it's filled in. After pressing backspace at least one digit remains, additionally after pressing backspace again, the dial code fills in again. In release 4.3.0 that was not the case.

@MatthewAnstey Could you help to check this issue?
It seems related to #195.
Thanks!

@patw0929
Sure. Seems to happen when its uncontrolled and separateDialCode is true. Will look into it.

Fixed and released as v4.3.4.
Thanks for @MatthewAnstey's help!


[Carousel][Image] Images don't get the proper size/width when they are selected into the view

Getting the width from the first visible parent in case the image is hidden
Fixes #305

Codecov Report

Merging #306 into development will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             development    #306   +/-   ##
=============================================
  Coverage           82.3%   82.3%
  Complexity           871     871
=============================================
  Files                 82      82
  Lines               2786    2786
  Branches             517     517
=============================================
  Hits                2293    2293
  Misses               202     202
  Partials             291     291

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 126f355...49a3642. Read the comment docs.

Verified Edit/Preview mode on chrome, firefox, edge


Esc does not exit the search menu

Bug Report
IMPORTANT:
Please follow this template strictly when reporting bugs. Providing detailed and accurate information helps the development team to understand and address the issue effectively.
App Version: 2.0.0-dev17
Platform: Windows
Installation Method: Direct Download
Bug Description:
pressing Esc with the search bar focused does not close the search bar as expected
Steps to Reproduce:

Open the search bar Ctrl+F (this also auto-focuses it)
Press Esc

Expected Behavior:
The search bar would close and show me my file tabs again
Actual Behavior:
either a) nothing or, when I had a side panel open (like filters) b) the side panel was closed

@swantzter I would suggest the following behavior:

If search field is open and has a value, escape will empty it. That's how it is done in many of the GUIs I use.
If search field is open and empty, escape will close search field and show file tabs again
If drawer is open and search field is open, escape will not close drawer
If drawer is open and search field is closed, escape will close drawer

Yeah, that sounds sensible to me.
My only thought would be that for search field is open and empty I would only expect it to close if it was focused. (and I would expect Ctrl+F to re-focus it if it wasn't focused)

Fully agree. Will be part of the next release.
I'm thinking about releasing the current state as 2.0 soon. Besides the already reported minor issues and improvement requests, does this build feel stable in general?

I think so, I'll note that I started using sleek with 2.0 dev releases and haven't used 1.x, so I'm not sure if there's something glaring missing. But 2.0 has been working well and been stable for me in general.
Only minor thing I have noticed that I haven't opened an issue about (I'll try to remember to do that) has been with threshold dates when the filters are set to hide todos with thresholds in the future.
On the day that the task is supposed to become visible you need to take some action that causes the list to refresh (if you've left the app open) to make the task appear, such as adding a todo or completing a todo.

We could add some sort of an auto refresh every couple of minutes.

Added it to the backlog.

Tested the new shortcut, it does what I expect it to now. thanks!


Update Animate mode to MP4 experience

In a future phase, update the "Animate" mode such that...

[x] the animation plays over the full map extent; no adjustable animation window
[x] download format is MP4 rather than GIF
[x] similar workflow and styling as Landsat Explorer's proposed animation download experience

This feature has been pushed to PROD on 03/27/2024.


ISSUE#34 Support for optional schema evolution.

+1. @shahsank3t  we need to add a field in the UI that captures if the users want the schema to be evolved or not. Default should point to true and if user set this to false we should only allow them register one version of Schema.

@shahsank3t @harshach Raised JIRA for the earlier comment.


ISPN-6869 Synchronize starting and stopping DefaultCacheManager

https://issues.jboss.org/browse/ISPN-6869
An approach based on 2 locks.

@slaskawi maybe I'm missing something, but this IMHO doesn't prevent deadlock described in ISPN-6869 -  AFAIKT new cache can be still started while cache manager stop is in progress and thus result into the deadlock.

I was wrong here... It's a matter of stopping CacheManager and starting Caches (not CacheManager).. sorry for confusion!