Update scalafmt-core to 3.3.3

Updates org.scalameta:scalafmt-core from 3.3.2 to 3.3.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (3.3.2).
You might want to review and update them manually.
docs/website/yarn.lock

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Codecov Report

Merging #881 (7f1a2cd) into main (786f51a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #881   +/-   ##
=======================================
  Coverage   53.31%   53.31%
=======================================
  Files          72       72
  Lines        2144     2144
  Branches       62       62
=======================================
  Hits         1143     1143
  Misses       1001     1001

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 786f51a...7f1a2cd. Read the comment docs.


support for modern Win32 terminals (windows 10 and newer)

This replaces https://github.com/joachimschmidt557/linenoize/pull/8
Updated to newest zig version and rebased on top of master branch

Thanks a lot!

appreciate this a lot! I was worried I was going to have to switch dependencies for my repl eventually...


SIL: Don't serialize imported conformances nested in non-public types

Apparently you can use a swift_name attribute in Clang to import types
as members of Swift types. If the Swift type is not public, we would
still try to serialize the witness thunks, which tripped SIL verifier
checks since the witnesses themselves were not serialized.
Note that the fix here is to just delete the special case of an
imported conformance; the regular "type is public and protocol is
public" condition suffices here.
Fixes rdar://problem/48218483.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please smoke test Linux

Uh…hm. This fixes the immediate problem, but it doesn't fix it well since the Clang type might be accessed from outside the original module. On the other hand, we don't want to ban this because a plain old app target should work without any custom access control at all.
I think the "most correct" answer is to say that import-as-member things are still public even when imported as members of internal types, but I don't know what effect that will have on everything else.

Discussed this with @jrose-apple "offline". The long-term solution here is to treat swift_name in this special case as adding a typealias to the outer non-public type, instead of importing it as a member. But this PR should not actually impede that, since it just streamlines the logic here.


[Documentation] Add anchor links

Add anchor links to all instruction steps in README.md

this looks like its working??

@lesliekimm some of the instruction steps i.e. the steps with numbering should also be linkable.
See Airbnb JavaScript Style Guide


Add script to modify include guard variables in parquet-amalgamation.hpp

Add script to modify include guard variables in parquet-amalgamation.hpp. The script finds all the include guard variables in the file, then substitutes them with _DUCKDB prefixed to them. This is to avoid interfering with the include guards from Thrift files used by the new parquet reader.

@jwyles-ahana Thank you for submitting the PR so quickly! Would you make the commit title and PR title shorter? Ideally we want them to be within 50 characters.

Moreover, bitwise_cast() in the hpp was not behind any include guard, and will have duplicate definition if we include the standalone Thrift library. Will you be able to see which header file it belongs to, and put it back behind its original include guard?

@jwyles-ahana The commit message needs to follow this guideline: https://gist.github.com/robertpainsi/b632364184e70900af4ab688decf6f53


Test/section ordering

Coverage decreased (-0.01%) to 47.235% when pulling e23432517b8929f238f36e8316dda6f25975a3b2 on test/section-ordering into faa9d7b57e5b174bba70741fec86bc997019befb on master.


chore: stop building sphinx docs

Pull Request Checklist

[ ] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[ ] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated
Design: For significant features, design document added to design folder

[ ] Title and Description

Change type: title prefixed with fix, feat and module name in parens, which will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

Don't you want to take this opportunity and get rid of the docs build stage? and the docs/ directory?

Totally, just forgot about it.


update dependencies in Arch SRCINFO templates, and add gtk2 dependency

I had forgotten to update the SRCINFO templates in the past. It's
unfortunate that we have duplication around this, but it saves us from
making the build dependent on Arch tools. I've added a comment to make
it harder to forget next time.
@modalduality, could you review this?

I have verified these packages exist, can't do much more :)

Good enough for government work!


Update link for GH wiki on home

This is not needed anymore. We automatically find and replace every link in the docs folder


add connection error (Ubuntu 24.04.1)

Hi,
Thanks for maintaining eduVPN. I have installed eduVPN using apt and ran into this connection issue.
$ eduvpn-cli -v
eduVPN CLI version: 4.4.0 with eduvpn-common version: 2.1.0

$ eduvpn-cli connect -n1
2024-09-28 08:22:51,892 - MainThread - ERROR - eduvpn.nm - nm.py:917 - add connection error: g-dbus-error-quark: Message recipient disconnected from message bus without replying (4)

Edit: It works in commandline when I manually import the configuration as https://github.com/eduvpn/eduvpn-common/issues/61 and bring up the connection afterwards. However, the eduVPN GUI didn't work before and after a reboot. It showed
The following error was reported: Failed to connect to server: 'https://eduvpn.otago.ac.nz/'. The cause of the error is: Timeout reached contacting URL: 'https://eduvpn.otago.ac.nz/.well-known/vpn-user-portal'.

Hmmm that error just indicates you have no internet?

Thanks @jwijenbergh. I have no issues with internet.

Can you make a video of using the GUI?

moved to https://codeberg.org/eduVPN/eduvpn-common/issues/71


Upscale or stretch any video on the web

Hello, this necessary and important extension because we watch a lot of streaming sites and often there are side black bars and this extension allows to remove them, I tried with Chromium and Ungoogled-Chromium and it didn't work, sure because this extension officially only supports firefox and google chrome, so i had to fiddle around and search and i tried one of these solutions and it worked! please can you add official support for this incredible essential extension, ie add an option to be able to change the aspect ratio of the videos in an easy way without having to manipulate the experimental flags of the browser?
Approach 1
Run the browser with a specific flag.
Quit the browser
Right click on the browser's shortcut and go to Properties
Within the Target box, add --disable-direct-composition-video-overlays` to the end (outside of quotation mark) and apply the changes
Launch the browser
Approach 2
Change graphics back-end for ANGLE.
Launch the browser
Open browser://flags/ (replace word browser, with the one you're using, e.g. chrome://flags/)
Search for ANGLE (full option name: Choose ANGLE graphics backend)
Choose D3D9 or OpenGL
Restart the browser
Approach 3
Disable hardware acceleration.
Launch the browser
Go to browser settings
Search for hardware (full option name: Use hardware acceleration when available)

lol i forget to post the extension : https://github.com/dvlden/ultrawideo
Upscale or stretch any video on the web, to make it look great on UltraWide screens.
The only extension on the market with two significant functionalities that no other extension in this category possess.
Works on all video streaming platforms. ✅
Works on all embedded videos. ✅
It doesn't matter if there are multiple direct or embedded video players on a single page. The extension is smart enough to manipulate the video you put into a fullscreen, without affecting other video players.
If you are using multiple browsers on the same or different devices, this extension is cross-browser and will work in all major browsers (except Safari). I tested it in Chrome, Firefox, Edge, Brave, Whale, and Opera; without a single issue. Many other browsers use Webkit engine, hence this extension will work in those browsers too.

I konw it is diffilcult hard to integrate new option :( i think even it's hard that we are talking about an important feature, being able to watch videos without black band and it will be great that all browsers  integrates a native "change aspect ratio" option in order to dazzle the world even if it's very difficult, we know it....

There is bug, i asked extension developer but there is no solution,  tried many extensions to stretch video on my favourite streaming site that's why i suggest this, for exempe integrate an native extension to manual selection so that the user can increase itself the height and width of the video therefor the developer of browser could konow and fix the bugs....
https://github.com/dvlden/ultrawideo/issues/79#issuecomment-1199999162
https://github.com/dvlden/ultrawideo/issues/80


vite_ssr_import_0.xxx is not a function

Reproduction
https://stackblitz.com/edit/nuxt-starter-i3vypr?file=composables%2FuseContract.ts
Steps to reproduce the bug
pnpm run dev
Expected behavior
can start dev server
Actual behavior
can't start dev server
Additional information
according to my tests, this problem is related to the composables/useContract function. It should be caused by the fact that useContract has the same name as the parameter in sdk/util/common.ts. When I try to delete default in useContract After that, this error will not occur. This may also be related to the logic of auto-import.

Please, next time consider using the Discord server, Discussions (if enabled in this repository), or StackOverflow for questions first. But feel free to come back and open an issue if it turns out to be a bug 🙂

yes, i will. I thought this might be a bug in nuxt or vite-node, so I closed the issue and created an issue in nuxt repo: https://github.com/nuxt/nuxt/issues/26543


Added 3 Leetcode problems

Added the Filenames as

#7 Reverse Integer.cpp
#13 Roman to integer.cpp
#35 Search Insert position.cpp

Changes Proposed  in the PR. Describe
I added questions for leetcode as per #13
I am assuring that the file is not redundant and making the Pull Request

[x] Yes

Congratulations 🎉  your PR has been Reviewed and merged!


Language selector not showing.

Since version 0.30, my second language's pages don't have a button for selecting language. I've downgraded to 0.22 and it works flawlessly. Is there something that was changed and i need to add to my config?
Thanks.

I've tried every single version (0.30, 0.31, 0.40, 0.41, 0.44 and 0.45) with the same results. Sadly it's a private work repo so i can not share it here. I'll try to reproduce the problem on a new one, it's just the translated HTMLs missing the button on the bar, i don't know why.

Sure, whenever I can reproduce, I'll make sure there's a test + a fix mate!

mkdocs-static-i18n-bug.zip
I managed to reproduce it with a small example:
Created a venv, installed requirements_doc.txt, ran mkdocs serve.

Ok @champsblanc it's actually written in the build logs:
WARNING  -  mkdocs-material language switcher contextual link is not compatible with theme.features = navigation.instant

If you remove navigation.instant, it works as expected.
Instant navigation uses JS which is not respectful of the localized page links and breaks localization language switcher I'm afraid.

 Thank you so much!


Some clarification to instructions and a couple new images.

11168
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

@ShayShmeltzer Please resolve the conflicts in this redwood-extensions/saasservice/saasservice.md  file.


Update Set-CsTeamsVoiceApplicationsPolicy.md

Update Set-CsTeamsVoiceApplicationsPolicy.md with documentation for new settings.

Learn Build status updates of commit aea9965:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 82b57f2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

@JohanFreelancer9

@dariomws @get-itips @tseward

@JianpingChen Gentle reminder

@JianpingChen Please implement the suggested changes, so that we can proceed to merge this PR. Thanks

@JianpingChen Please respond and implement the suggested changes, in the even of no reply we may close this PR. Thanks

@tiburd kindly help. Thanks

Learn Build status updates of commit c3f9d1b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md
:x:Error

Details

teams/teams-ps/teams/Get-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Grant-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/index.yml
:warning:Warning

Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Remove-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/x2it-s/teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md:8:(53) 'Set-CsTeamsVoiceApplicationsPolicy [-AllowAutoAtte...'
 Expect Heading, please try to use PlatyPS validate the content

teams/teams-ps/teams/Get-CsTeamsVoiceApplicationsPolicy.yml

Line 11, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/Grant-CsTeamsVoiceApplicationsPolicy.yml

Line 16, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/index.yml

Line 121, Column 5: [Warning: xref-not-found] Cross reference not found: 'MicrosoftTeamsPowerShell.Set-CsTeamsVoiceApplicationsPolicy'.

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.yml

Line 17, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/Remove-CsTeamsVoiceApplicationsPolicy.yml

Line 18, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
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.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 8dae63d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md
:x:Error

Details

teams/teams-ps/teams/Get-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Grant-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/index.yml
:warning:Warning

Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Remove-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/cqs0-s/teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md:8:(53) 'Set-CsTeamsVoiceApplicationsPolicy [-AllowAutoAtte...'
 Expect Heading, please try to use PlatyPS validate the content

teams/teams-ps/teams/Get-CsTeamsVoiceApplicationsPolicy.yml

Line 11, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/Grant-CsTeamsVoiceApplicationsPolicy.yml

Line 16, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/index.yml

Line 121, Column 5: [Warning: xref-not-found] Cross reference not found: 'MicrosoftTeamsPowerShell.Set-CsTeamsVoiceApplicationsPolicy'.
Line 139, Column 5: [Warning: xref-not-found] Cross reference not found: 'MicrosoftTeamsPowerShell.Update-TeamsAppInstallation'.

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.yml

Line 17, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/Remove-CsTeamsVoiceApplicationsPolicy.yml

Line 18, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
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.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 74a9e6a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md
:x:Error

Details

teams/teams-ps/teams/Get-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Grant-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/index.yml
:warning:Warning

Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Remove-CsTeamsVoiceApplicationsPolicy.yml
:warning:Warning

Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/oyoy-s/teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md:8:(53) 'Set-CsTeamsVoiceApplicationsPolicy [-AllowAutoAtte...'
 Expect Heading, please try to use PlatyPS validate the content

teams/teams-ps/teams/Get-CsTeamsVoiceApplicationsPolicy.yml

Line 11, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/Grant-CsTeamsVoiceApplicationsPolicy.yml

Line 16, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/index.yml

Line 121, Column 5: [Warning: xref-not-found] Cross reference not found: 'MicrosoftTeamsPowerShell.Set-CsTeamsVoiceApplicationsPolicy'.

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.yml

Line 17, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

teams/teams-ps/teams/Remove-CsTeamsVoiceApplicationsPolicy.yml

Line 18, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-CsTeamsVoiceApplicationsPolicy.yml'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
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.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit d7aa1e3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Set-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

@dariomws @get-itips


showcase, var - CLI tool for variables

here is the source - https://github.com/denyzhirkov/var

Something strange with tests and cheks


Added transitions on buttons send otp and login

Added transition duration of 0.4s when hovered upon the mentioned buttons.
closes #289

@Tejas1510 , Please have a look so that we can merge the PR

Looks Good to me @AmeyaJain-25
Merging the PR


use pre-commit to invoke linters

adds a pre-commit configuration so that its easy to invoke all of our linter checks w/associated configuration, as well as install the pre-commit and pre-push hooks consistently:

quickstart
do this now @mr-tz @mike-hunhoff @yelhamer @colton-gabertan (FYI)
$ pre-commit install    # now pre-commit hook installed!

or if you want to run all the linters on-demand:
$ pre-commit run --all-files

the PR also introduces some additional plugins (and fixes) for flake8:

bugbear - find likely bugs, which it did, such as use of str.lstrip("many characters here")
encodings - enforce text read from files has an explicit encoding
comprehensions - use list/tuple/set comprehensions where appropriate (and don't, when not appropriate)
logging-format - don't use format strings in logging calls
no-implicit-concat - catch potential bugs by disallowing implicit string concatenation, like "foo" "bar" == "foobar"
print - catch print statements in library code
todos - enforce TODOs are formatted like TODO(author): message and have an associated link to a tracking github issue

im really excited about some of these lints (like logging format), since we've often had to explain and manually check for these during PR reviews. the other ones enforce better hygiene (imho).
closes #1579
closes #1391
Checklist

[ ] No CHANGELOG update needed

[x] No new tests needed

[x] documentation update

just getting tests to pass and then i'll open this up for review

i have a few more linters in the works (flake8 plugins) so i'll get those hammered out before we merge.

@ajnelson-nist check out how we integrated pre-commit here (thanks to your pointer).
specifically, we use a local configuration rather than using the third-party pre-commit hooks. this let's use declare the linter dependencies in setup.py/pyproject.toml and therefore get updated by Dependabot while still getting invoked by pre-commit.
https://github.com/mandiant/capa/blob/f983307c978b82911d823ac291f4e397393f4ba2/.pre-commit-config.yaml

@williballenthin And now you've explained something I hadn't quite understood from the examples I'd drawn from.  Local configurations!  That's how they'd gotten around pinning-vs-range version freshness issues.
This was a fruitful back and forth.  Thanks for letting me know what you found!


Fix tests on Windows

$ pipenv run inv tests
usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --cov=.
  inifile: None
  rootdir: C:\Users\user\Projects\nathanurwin\cookiecutter-git

https://github.com/pytest-dev/pytest/issues/3434

The complete fix after rereading that pytest issue is to $ pipenv run pip install -r requirements.txt. Then you can $ pipenv run invoke tests. I'll make sure to add it to the Contributors section of the README! 👍


Nav2 namespaces and prefixes

Followup of #51
For supporting multiple robots/platforms it is required to add support for these settings. For this some work has to be done first to the controller configuration and launchfiles to support generic namespaces/prefixes. After that the same has to be done for the Nav2 config file.
For this RewriteYAML could come in handy, but would requure nav2_commons for everything platform related. I think that this would be acceptable as most people will only use the platform in combination with it anyways.

For future reference, these are the settings required to launch the platform with a namespace
ros2 launch irc_ros_bringup cpr_platform.launch.py use_laserscanners:=false namespace:=/platform_1 prefix:=platform_1_ platform_controller_config:=src/iRC_ROS/irc_ros_bringup/config/controller_cpr_platform_medium_namespace_1.yaml use_laserscanners:=true


Error running Gulp Unexpected character '`' on yarn run blendid -- build

Hi all!
I'm not sure if this is the correct place to put this and I can cross post this to Foundation's issues as well if this isn't the proper location, but I figured I'd start here.
When using Foundation's new rc version, I'm getting the following error when running yarn run blendid -- build:
[16:36:41] gulp-notify: [Error running Gulp] Error:
app-9d242fac1e7f31a031bb.js from UglifyJs
Unexpected character '`' [app-9d242fac1e7f31a031bb.js:10358,124]

When trying to debug this with a colleague, we think it could potentially be a bad babel configuration for node modules, but that's the best guess we could come up with. This is what ultimately lead me to start my search here.
I've created a test case if it is any help!
https://github.com/ryanjavens/broken-foundation-blendid
I look forward to any input you guys might have!
Thanks!

In case anyone else encounters this issue, there's a fix for it that I got from the issue I made in Foundation's repo: https://github.com/zurb/foundation-sites/issues/10161#issuecomment-309084581.


Handle MFHD with illegal lengths in 008:s

A library exports their Sierra c-records in MFHD format, and for some reason these records have the length of 40 characters in the 008, where the allowed length is 32 characters (40 is allowed in bibs)
FOLIO happily accepts these records, but QuickMARC throws an error.
Proposed solution: cut down the characters on the 008:s and report on the contents of the 8 last characters.
Example record:
=LDR  00277ny   22001093n 4500
=008  \\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
=090  \\$aKF879.527$b.Q8 1991
=866  \\$aSupplementation through 2004.
=951  \\$b.b24821111
=951  \\$c.c2000008x
=998  \\$s-$c-$r-$v1$ldxhis
=923  \\$aSupplements


Fix LifecycleListener's dispose method not being called on LWJGL3 backend

See https://github.com/libgdx/libgdx/issues/3673#issuecomment-172115381 and https://github.com/libgdx/libgdx/issues/3673#issuecomment-207808454 .

Looks good. Please synchronize on the listeners.

@badlogic The listeners are synchronized now, anything besides that?

I resolved the merge conflict.


Use new endpoints for adding/removing users to/from groups

(not yet implemented in the backend, I will update this issue when it is)
In the current status:

We cannot remove users from a user group
Adding users to a user groups is meant to also work for the "bulk" case, even though we only add one user at a time in the current fractal-web UI.

As soon as https://github.com/fractal-analytics-platform/fractal-server/issues/1841 (starting from https://github.com/fractal-analytics-platform/fractal-server/issues/1841#issuecomment-2490531931) is implemented and released, two things will change:

The current way of adding users will be deprecated
There will be two more atomic endpoints that add/remove a single user and then return the updated group object:

POST /group/{group_id}/add-user/{user_id}/
POST /group/{group_id}/remove-user/{user_id}/

Relevant PR:

https://github.com/fractal-analytics-platform/fractal-server/pull/2101/

Side-note: we are leaving the possibility of bulk user addition/removal for the future, and it may eventually be part of POST /auth/group/{user_group_id}/add-users-bulk/ and POST /auth/group/{user_group_id}/remove-users-bulk/.

It is possible to remove members from the group both by clicking on the X button on the user badges and by dragging them outside the "drag users here" area.


Added Code Review to Contributing.md

Fixes #547
Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added in line documentation to the code I modified

Short description of what this PR does:

Added Code Review to Contributing.md

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

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

Thanks for taking the time to review @m1guelpf!

Hello @mptap,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


SSL Library Error - BUF_MEM_grow:openssl_malloc failure

Hi,
Our 32-bit process acting as EAP-TLS server running on Windows Server 2012 R2 platform uses OpenSSL third-party libraries for EAP protocols. During the debugging of a customer issue in EAP-TLS protocol, we got to understand that “SSL_accept” has returned failure due to internal failure of methods BUF_MEM_grow which failed due to openssl_malloc failure.
The “SSL_accept” errors are noticed over a period of running 2-3 days from the time process started in case of nominal traffic (EAP-TLS requests). During peak hours, it is observed sometime in 3-4 hours itself. The moment “SSL_accept” starts failing due to openssl_malloc failure, then 90% of the subsequent requests arriving during this time period also fails with the same error and the process has to be restarted to bring back services to normal.
Since ours is a 32-bit process, it has the capability for its process size to grow till 4GB at the maximum. However, when we measure the process size during the occurrence of the error it is just around 800-900MB. The openSSL version that we are using is OpenSSL 1.0.2h.
We earlier raised a ticket https://github.com/openssl/openssl/issues/2763 for the same issue and it was closed since we could not provide the required data for OpenSSL Team to investigate because the issue was not simulated in Lab. Now another customer has reported this issue and we are also able to simulate the issue in our Lab and can give any information that you ask for.
As a first level of investigation, we introduced few logs in OpenSSL library where from our application we invoke SSL_accept. The error occurs in the following sequence
ssl3_accept -> BUF_MEM_grow -> openssl_malloc/openssl_realloc (failure occurs in memory allocation)
The parameters that are passed to BUF_MEM_grow are normal numbers and not any abnormal parameters.
While we are doing a deep dive of OpenSSL code, could you please provide us pointers on the following,

During the occurrence of the error, we can see that our process (acting as an EAP-TLS Server) is handling approximately around 600 active TLS handshake challenges at a single point of time. We expect that OpenSSL will also be maintaining TLS session information. Is there any fixed size for the buffer that is preventing to allocate further memory? Is it not able to maintain 600 challenge handshake sessions in a single point of time?
If OpenSSL maintains any kind of buffers for maintaining TLS sessions and if it is pre-determined to some size. Which OpenSSL structures to look for?
It is also mentioned that behaviour of SSL_accept() depends on the underlying BIO. Anything to investigate from this side?

Any further pointers that you can provide us to investigate this issue will really help us a lot.
As the issue is simulated in our lab we can also provide any debug information that you are looking out for.
Awaiting your response on the same.
Regards,
Nomalatha A

We are sorry for neglecting your issue but 1.0.2 versions are out of support for a long time. Do you still have the problem with newer releases?

marking as inactive, to be closed at the end of 3.4 dev barring further input


server - waiting for db connected...

gateway-controller_1      | [2022-09-25T04:46:33.390] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:34.342] [ERROR] server - db:sys_db connect failed MongoServerSelectionError: connect ECONNREFUSED <IP_ADDRESS>:27017
gateway-controller_1      |     at Timeout._onTimeout (/app/node_modules/mongodb/lib/sdam/topology.js:325:38)
gateway-controller_1      |     at listOnTimeout (node:internal/timers:559:17)
gateway-controller_1      |     at processTimers (node:internal/timers:502:7) {
gateway-controller_1      |   reason: TopologyDescription {
gateway-controller_1      |     type: 'ReplicaSetNoPrimary',
gateway-controller_1      |     servers: Map(1) { 'mongo:27017' => [ServerDescription] },
gateway-controller_1      |     stale: false,
gateway-controller_1      |     compatible: true,
gateway-controller_1      |     heartbeatFrequencyMS: 10000,
gateway-controller_1      |     localThresholdMS: 15,
gateway-controller_1      |     setName: 'laf',
gateway-controller_1      |     logicalSessionTimeoutMinutes: undefined
gateway-controller_1      |   }
gateway-controller_1      | }
gateway-controller_1      |
gateway-controller_1      | ><EMAIL_ADDRESS>start
gateway-controller_1      | > node ./dist/index.js
gateway-controller_1      |
gateway-controller_1      | [2022-09-25T04:46:39.833] [INFO] server - server logging level is DEBUG
gateway-controller_1      | [2022-09-25T04:46:40.043] [INFO] server - start init base route
gateway-controller_1      | [2022-09-25T04:46:40.058] [INFO] server - start scheduler loop
gateway-controller_1      | [2022-09-25T04:46:40.063] [INFO] server - listened on 9001
gateway-controller_1      | [2022-09-25T04:46:40.083] [INFO] server - create route successful
gateway-controller_1      | [2022-09-25T04:46:40.256] [INFO] server - create route successful
gateway-controller_1      | [2022-09-25T04:46:40.319] [INFO] server - create route successful
gateway-controller_1      | [2022-09-25T04:46:40.320] [INFO] server - create route successful
gateway-controller_1      | [2022-09-25T04:46:40.321] [INFO] server - create route successful
gateway-controller_1      | [2022-09-25T04:46:41.060] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:42.061] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:43.061] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:44.062] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:45.063] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:46.063] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:47.064] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:48.064] [INFO] server - waiting for db connected...
gateway-controller_1      | [2022-09-25T04:46:49.064] [INFO] server - waiting for db connected...
docker-compose_gateway-controller_1 exited with code 137
system-server_1           | 2022/09/25 04:47:03 Timeout after 1m0s waiting on dependencies to become available: [tcp://mongo:27017 http://oss:9001]
system-server_1           | 2022/09/25 04:47:07 Problem with dial: dial tcp: lookup mongo on <IP_ADDRESS>:53: no such host. Sleeping 1s
system-server_1           | 2022/09/25 04:47:08 Problem with dial: dial tcp: lookup mongo on <IP_ADDRESS>:53: no such host. Sleeping 1s
system-server_1           | 2022/09/25 04:47:09 Problem with dial: dial tcp: lookup mongo on <IP_ADDRESS>:53: no such host. Sleeping 1s

$ docker-compose logs -f mongo
Traceback (most recent call last):
  File "urllib3/connectionpool.py", line 677, in urlopen
  File "urllib3/connectionpool.py", line 392, in _make_request
  File "http/client.py", line 1277, in request
  File "http/client.py", line 1323, in _send_request
  File "http/client.py", line 1272, in endheaders
  File "http/client.py", line 1032, in _send_output
  File "http/client.py", line 972, in send
  File "docker/transport/unixconn.py", line 43, in connect
PermissionError: [Errno 13] Permission denied

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "requests/adapters.py", line 449, in send
  File "urllib3/connectionpool.py", line 727, in urlopen
  File "urllib3/util/retry.py", line 410, in increment
  File "urllib3/packages/six.py", line 734, in reraise
  File "urllib3/connectionpool.py", line 677, in urlopen
  File "urllib3/connectionpool.py", line 392, in _make_request
  File "http/client.py", line 1277, in request
  File "http/client.py", line 1323, in _send_request
  File "http/client.py", line 1272, in endheaders
  File "http/client.py", line 1032, in _send_output
  File "http/client.py", line 972, in send
  File "docker/transport/unixconn.py", line 43, in connect
urllib3.exceptions.ProtocolError: ('Connection aborted.', PermissionError(13, 'Permission denied'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker/api/client.py", line 214, in _retrieve_server_version
  File "docker/api/daemon.py", line 181, in version
  File "docker/utils/decorators.py", line 46, in inner
  File "docker/api/client.py", line 237, in _get
  File "requests/sessions.py", line 543, in get
  File "requests/sessions.py", line 530, in request
  File "requests/sessions.py", line 643, in send
  File "requests/adapters.py", line 498, in send
requests.exceptions.ConnectionError: ('Connection aborted.', PermissionError(13, 'Permission denied'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker-compose", line 3, in <module>
  File "compose/cli/main.py", line 80, in main
  File "compose/cli/main.py", line 189, in perform_command
  File "compose/cli/command.py", line 70, in project_from_options
  File "compose/cli/command.py", line 153, in get_project
  File "compose/cli/docker_client.py", line 43, in get_client
  File "compose/cli/docker_client.py", line 170, in docker_client
  File "docker/api/client.py", line 197, in __init__
  File "docker/api/client.py", line 222, in _retrieve_server_version
docker.errors.DockerException: Error while fetching server API version: ('Connection aborted.', PermissionError(13, 'Permission denied'))
[17309] Failed to execute script docker-compose

PermissionError: [Errno 13] Permission denied
这个应该是没权限执行

PermissionError: [Errno 13] Permission denied 这个应该是没权限执行

上面的记录改为了 我加了 sudo 执行之后 的结果

👌 看到日志是 mongo 没启来， 可能你要排查一下 mongo 的这个错误来源，根据我的经验可能和你的环境相关：

cpu 架构和型号发一下，确认是 mongo 支持的处理器（bitnami/mongo 镜像不支持 arm 和 较老的 x86）

👌 看到日志是 mongo 没启来， 可能你要排查一下 mongo 的这个错误来源，根据我的经验可能和你的环境相关：

cpu 架构和型号发一下，确认是 mongo 支持的处理器（bitnami/mongo 镜像不支持 arm 和 较老的 x86）

cpu 是 j4025，系统是群晖 nas 的 dsm7，应该是基于 FreeBSD

MongoDB requires the following minimum x86_64 microarchitectures:
For Intel x86_64, MongoDB requires Sandy Bridge or later.
For AMD x86_64, MongoDB requires Bulldozer or later.
https://jira.mongodb.org/browse/DOCS-14684
https://www.mongodb.com/community/forums/t/mongodb-5-0-cpu-intel-g4650-compatibility/116610
https://www.intel.com/content/www/us/en/products/sku/197307/intel-celeron-processor-j4025-4m-cache-up-to-2-90-ghz/specifications.html
你这款处理器比较老， mongodb 5.0 要求处理器支持 avx （sandy bridge AVX）能力。可以看上面我帮你找的链接参考。
可以改成 mongodb 4.4.13 尝试一下


Added my linkedin url and Resume in About.html

arthi change linked in url and resume link in about.html bit its have merge conflict


Is dgrad+relu with fp32 supported?

I have try to fuse dgrad with some pointwise operation like relu,
but when cudnnBackendFinalize(plan), there is a error.
CUDNN_STATUS_NOT_SUPPORTED.
I am not sure if my code error.
I want know, is dgrad + pointwise support now in cudnn 8.6 for Turing Arch?
Could you give me some sample about dgrad fusion?
In this project, only one case for dgrad, but it it only support for Ampere Arch.

Here is my test code.
https://github.com/tjich/design_patterns/blob/main/lite_engine/cudnn_backend_api_test.cu
I try to fuse dgrad with some pointwise, but even fusion with only one relu operation is not ok.

@tangjicheng46 has you resolved your issues?

@tangjicheng46 has you resolved your issues?

I'm sorry, but I don't think I have solve this issue at that time.
However, it's been a long time, and I'm not sure if this problem still exists.

@tangjicheng46 has you resolved your issues?

I think we can just close this issue,
I'm not following up on this issue right now.


Refactor to allow custom errors for custom formats

As the title says, that would be nice.

@anilanar are you looking for custom error message based on the specific format (so that not all format errors return the same string? If so, there is an err.Details() function that returns a map[string]interface{} and you can do something like this in your application:
msg := err.Description()
if err.Type() == "format" {
   details := err.Details()
   switch details["format"] {
      case "formatA":
         msg = "Format A error"
      case "formatB":
         msg = "Format B error"
  }
}

return msg

IMO the library shouldn't provide that level of customization, it should return the errors and all of the detail needed and library users can convert the output to a custom format. Some users may want error strings, others might want custom structs with json or xml tags.
I'm using this library in an api "framework" and I have a validator function that validates and maps json schema errors from this library to a specific struct with defined Field/Type/Message fields that can be outputted to json or xml.

@cristiangraz
I've just checked the code written back in Nov 2015. I have some custom format validators. One of them:
import (
	"regexp"

	"github.com/xeipuuv/gojsonschema"
)

var regex = regexp.MustCompile(`^(?=[A-Z])(?=[a-z])(?=[1-9]).{8,}$`)

type Password struct{}

func (p Password) IsFormat(input string) bool {
	return regex.FindStringIndex(input) != nil
}

func RegisterPasswordValidator() {
	gojsonschema.FormatCheckers.Add("password", Password{})
}

I guess I wished to be able to define error message on the struct, maybe by defining a GetMessage method? I don't really remember though.

Yeah that's a good use for formats, and I can understand the need to want to pass along custom messages to your users.
There is an open issue #62 about converting format checkers to functions (which was just my oversight when I initially build out the format checker functionality). If custom formats did support custom messages, it would be a good reason for using a struct and having an additional method to meet the interface for defining a formatter with it's own validator (IsFormat) and it's own message (GetMessage maybe).
I'm not a library maintainer so not my call, but personally think it's beyond the scope of the library which IMO should just be to validate JSON schema and provide detailed errors back.


CC0 license and CLA for translations

What version of Spook are you using?
1
What version of Home Assistant are you using?
1
The problem

By contributing translations to the Spook project, you agree that your translations are published under the CC0 (Creative Commons Zero v1.0 Universal) license, meaning you wave copyright (and any other right); making your contributed translations public domain.

That is not what CC0 does in all jurisdictions, and it isn't compatible with the previously committed work under MIT because it has a required part that isn't part of the CC0 license.
Adding a CoC to the project without asking contributors after taking contributions is also egregious, but legal.
Anything in the logs? Paste it here!
No response

Would like like me to revert your contributions? I am happy to do that if that makes you feel more comfortable?
Also, please note, this project was never MIT licensed. As a matter of fact, by using this code you've been in violation of the license already.

It is not about comfort, but the copyright of any of the contributors.
https://hosted.weblate.org/changes/?component=integration&project=spook&lang=&action=55&user=&start_date=&end_date=
It was MIT
and the bottom of https://github.com/frenck/spook is MIT
and 6.2 of https://hosted.weblate.org/legal/terms/ would make it MIT too.

It is not and has never been MIT. As a matter of fact, this repo never carried any OSI-approved license.

The message in Weblate has been adjusted.
../Frenck

Right, so you took the MIT license, filled in
Copyright <YEAR> <COPYRIGHT HOLDER>
and replaced
and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
with:
but NOT including the right to run, execute or use the Software or any executable binaries built from the source code.
Actually selecting the MIT license for the translations is what made me miss that.
The same reason why nobody ever tries to pull a fast one with the project license is why people are more likely to contribute to it in error.
Please do remove my contributions.

Right, so you took the MIT license

No, I did not. I choose the passive aggressive license for this project source code, which is an existing license. A small Google search would have learned you that.
The reason is that, I wanted to keep this open as much as possible, yet, using this is a bad idea. This is also documented extensively in the documentation along with this repository.

Actually selecting the MIT license for the translations is what made me miss that.

I don’t think that statement is correct. The translations are published now under the CC0 license. This to comply with the requirements of Weblate Libre that explicitly states the translations need to be published under an OSI approved license.

Please do remove my contributions.

Sure, no problem. I am currently traveling, but will pick this up when I return in a little over a week 👍


.NET Core Bindings: Bindings not discovered on slow machines

When the solution is opened in a way that a feature file gets active immediately (feature file was open and active before solution close), on some slow machines the bindings are not discovered.
On the output pane it displays:
Info: .ctor: Starting Deveroom...
Info: CreateProjectScope: Initializing project: BddWithSpecFlow.GeekPizza.Specs
Info: StartRetryInitializeTimer: Invalid project settings collected, retry in 5 seconds...
Info: StartRetryInitializeTimer: Invalid project settings collected, retry in 5 seconds...
Info: StartRetryInitializeTimer: Invalid project settings collected, retry in 5 seconds...
Info: RetryInitializeTimerTick: Project settings could not be initialized. Build the project to reload settings.

The bug is that the bindings are not discovered even if you rebuild afterwards.
Workaround: close all feature files, close and reopen solution, build the project and only after open the first feature file.

Better workaround: Rebuild helps

fixed in v1.0.3


Remove longhorn.io docs from longhorn/longhorn

This PR should be merged after https://github.com/longhorn/website/pull/48 is merged because it links to the longhorn.io docs.

https://github.com/longhorn/longhorn/issues/1131


Development 27062022

changes of:

readme and input.jsonnet texts (copywriter edits)
fixing potential problem with phases in ad platform to googlesheet template

@JakubMatejka please, review this PR

mám :-)


⚠ T:Riza Docs has degraded performance

In bb90636, T:Riza Docs (https://docs.triza.dev) experienced degraded performance:

HTTP code: 500
Response time: 26819 ms

Resolved: T:Riza Docs performance has improved in ddb565d.


Reports list views don't work well on an iPhone

create a sub
have some other users report things in your sub
on your iPhone, choose Moderation from the top bar and then either Open Reports or Closed Reports

result: the table is too wide for the screen whether you hold the phone horizontally or vertically, and you can't make it scroll to see everything
expected: view should be usable on an iPhone

Resolved by #401.


Construct Playlist page

Criteria:

add/remove/update playlists
add music to one/many playlists
display playlists name in the sidebar, along with delete button
display music and relevant fields in grid, and playlist name/created on date in header

This has been completed


Fail to open the second service in Storage Explorer

Repro steps:

Make sure Storage Explorer is installed.
Expand one storage account -> Right click one blob container and select 'Open in Storage Explorer'.
Check the blob container can be opened in Storage Explorer successfully.
Right click another blob container -> Select 'Open in Storage Explorer'.
Check whether it can be opened in Storage Explorer successfully or not.

Expect:
The second opened blob container can be opened successfully in Storage Explorer.
Actual:
Fail to open the second blob container in Storage Explorer.

This issue doesn't reproduce, So close it.


Hi, I meet a problem when evaluate the nerf;

I just run above script,

@Dengzhi-USTC Are you using the latest version? I cannot reproduce the error...
$ python main_nerf.py data/fox/ --workspace trial_nerf --fp16 --ff --cuda_ray
Namespace(path='data/fox/', test=False, workspace='trial_nerf', seed=0, num_rays=4096, cuda_ray=True, num_steps=128, upsample_steps=128, max_ray_batch=4096, fp16=True, ff=True, tcnn=False, mode='colmap',
 preload=False, bound=2, scale=0.33, gui=False, W=800, H=800, radius=5, fovy=90, max_spp=64)
NeRFNetwork(
  (encoder): HashEncoder: input_dim=3 num_levels=16 level_dim=2 base_resolution=16 per_level_scale=1.4472692374403782 params=(6328829, 2)
  (sigma_net): FFMLP: input_dim=32 output_dim=16 hidden_dim=64 num_layers=2 activation=0
  (encoder_dir): SHEncoder: input_dim=3 degree=4
  (color_net): FFMLP: input_dim=32 output_dim=3 hidden_dim=64 num_layers=3 activation=0
)
[INFO] Trainer: ngp | 2022-03-16_10-23-47 | cuda:0 | fp16 | trial_nerf
[INFO] #parameters: 12676090
[INFO] Loading latest checkpoint ...
[WARN] No checkpoint found, model randomly initialized.
==> Start Training Epoch 1, lr=0.010000 ...
loss=0.0157 (0.0188): : 100% 49/49 [00:01<00:00, 40.71it/s]
==> Finished Epoch 1.
==> Start Training Epoch 2, lr=0.010000 ...
loss=0.0207 (0.0228): : 100% 49/49 [00:01<00:00, 41.23it/s]
==> Finished Epoch 2.
==> Start Training Epoch 3, lr=0.010000 ...
loss=0.0126 (0.0134): : 100% 49/49 [00:02<00:00, 21.19it/s]
==> Finished Epoch 3.
==> Start Training Epoch 4, lr=0.010000 ...
loss=0.0071 (0.0076): : 100% 49/49 [00:02<00:00, 22.08it/s]
==> Finished Epoch 4.
==> Start Training Epoch 5, lr=0.010000 ...
loss=0.0040 (0.0059): : 100% 49/49 [00:02<00:00, 21.86it/s]
==> Finished Epoch 5.
==> Start Training Epoch 6, lr=0.010000 ...
loss=0.0045 (0.0046): : 100% 49/49 [00:02<00:00, 21.84it/s]
==> Finished Epoch 6.
==> Start Training Epoch 7, lr=0.010000 ...
loss=0.0035 (0.0048): : 100% 49/49 [00:02<00:00, 21.85it/s]
==> Finished Epoch 7.
==> Start Training Epoch 8, lr=0.010000 ...
loss=0.0034 (0.0042): : 100% 49/49 [00:02<00:00, 22.42it/s]
==> Finished Epoch 8.
==> Start Training Epoch 9, lr=0.010000 ...
loss=0.0051 (0.0032): : 100% 49/49 [00:02<00:00, 21.91it/s]
==> Finished Epoch 9.
==> Start Training Epoch 10, lr=0.010000 ...
loss=0.0017 (0.0029): : 100% 49/49 [00:02<00:00, 22.36it/s]
==> Finished Epoch 10.
++> Evaluate at epoch 10 ...
loss=0.0023 (0.0023): : 100% 1/1 [00:00<00:00,  1.18it/s]
PSNR = 22.404175
++> Evaluate epoch 10 Finished.
[INFO] New best result: None --> 0.0023169806227087975
==> Start Training Epoch 11, lr=0.010000 ...
loss=0.0022 (0.0027): : 100% 49/49 [00:02<00:00, 22.20it/s]

Yes, i download the latest version.
I try this on the v100 and 3090.
The inference part also generate those problem.

ok, i soved the problem, the vision of my torch is 1.08.

@ashawkey Hi, about this strange phenomenon, I've found a simple workaround.
This seems to be caused by not explicitly giving the return value in class _composite_rays(Function).

I think it is better to manually adding a return value (for example, return tuple()) to those functions not giving a return value, even if they aren't accessed at all. The same change should be applied to another function class _compact_rays(Function) similarly. I think this is because a lower pytorch exists this bug which is fixed in a higher version.

By the way, about another suggestion. I also notice that it'd be great if you could remove all the indexing='ij' in meshgrid function so that your codes are compatible with a lower pytorch version (notice that ij is already the default arguments and we could remove it).

@Karbo123 Thanks for the solution! I will add it soon. As for the reason of using indexing='ij', it says the default behaviour will be changed in later pytorch versions. I'll write a condition to better support lower versions.


Add asynchronous dns lookups

Perhaps using c-ares is the correct choice?  Its a fairly heavy dependency but it should build inline with the application.  c-ares appears to provide sockets that can be polled on for read/write events, this will work very well with the coroutine client connect() call as is.
The other option might be to use https://man7.org/linux/man-pages/man3/getaddrinfo_a.3.html but I'm not sure how well this will work with the coroutine setup.

It looks like c-ares will be the best solution for right now, the getaddrinfo_a doesn't work with file descriptors from what I can tell, so it won't cleanly integrate with the coroutine event loop scheduler.


Retheme Docs to follow Vargas Arts theme

https://github.com/eggheadio/egghead-next uses MDX for enhanced layouts. Definitely take a look


Add new build system for the website

Switches from the ad-hoc description in the README.md file to a Meson
based build. This build provides run targets for deploying and viewing
the v1 and v2 docs. It isn't as good as I want it to be, but I feel it
is a lot easier to use, which is good.
This also opens the door for adding the Javadocs for hse-java to the
site soon.

Went ahead and added a CI workflow as well. I think we should probably add checkoss if we are committed to the script (I am not :)).
Bhavesh, please make sure that you can successfully deploy the website. If you let me know what the commands are, I can document them/make another run_target().
After doing this work, I am not sure how much I like Material for MkDocs and Mike, especially Mike. Good enough for now, I guess.

This build makes use of a custom build for doxybook2 and inja. I submitted the inja build upstream to inja. We will see if upstream accepts it. If so, we can drop the custom build. In the case of doxybook2, I want to get this into the WrapDB, but it is pending the inja build. Not bad though.

I thought in a Teams message, you said to delete it. I'll add it back.

Why are we removing the CONTRIBUTING.md in this PR?


Add config set/get commands, similar to Heroku, to fnctl

See: https://github.com/iron-io/functions/pull/247#discussion_r87277460

With #287 it can be closed.


Update index.html

Nicely done. Thanks for your contribution from Acadamy Affair office of Northeastern University. The server edition will be updated soon!


Plugin does not work / compilation error

UPDATE (1/10/22): as of now, there does not appear to be a way to access the module responsible for changing the user status. Any help in this regard would be very much appreciated

due to a recent discord update that broke most of the plugins and messed up BD as well, AutoIdleOnAFK has been affected too.
The issue mainly is the module the plugin was using cannot be found now due it being changed/replaced to something else by discord. The fix would be to find the new module (again).
I'll be working on that when i get some free time.

traceback:
TypeError: Cannot destructure property 'StatusSetting' of 'Webpack.getModule(...)' as it is undefined.
    at plugin (betterdiscord://plugins/AutoIdleOnAFK.plugin.js:70:13)
    at eval (betterdiscord://plugins/AutoIdleOnAFK.plugin.js:297:13)
    at eval (betterdiscord://plugins/AutoIdleOnAFK.plugin.js:298:3)
    at Object.requireAddon (<anonymous>:4:43438)
    at Object.loadAddon (<anonymous>:4:9197)
    at Object.loadAddon (<anonymous>:4:41165)
    at Object.loadAllAddons (<anonymous>:4:11607)
    at Object.initialize (<anonymous>:4:6257)
    at Object.initialize (<anonymous>:4:40368)
    at Object.startup (<anonymous>:4:376262)

also use this opportunity to do the following:

[ ] move away from update url being release asset, to a main branch url
[ ] modify action to publish + build only on release
[ ] also publish sha256 hashes via actions cuz why not
[ ] move away from build scripts to the npm build module

There's an updated/forked version at https://github.com/haxurus/BetterDiscordPlugin-AutoIdleOnAFK

UPDATE (1/10/22): as of now, there does not appear to be a way to access the module responsible for changing the user status. Any help in this regard would be very much appreciated
due to a recent discord update that broke most of the plugins and messed up BD as well, AutoIdleOnAFK has been affected too.
The issue mainly is the module the plugin was using cannot be found now due it being changed/replaced to something else by discord. The fix would be to find the new module (again).
I'll be working on that when i get some free time.
traceback:
TypeError: Cannot destructure property 'StatusSetting' of 'Webpack.getModule(...)' as it is undefined.
    at plugin (betterdiscord://plugins/AutoIdleOnAFK.plugin.js:70:13)
    at eval (betterdiscord://plugins/AutoIdleOnAFK.plugin.js:297:13)
    at eval (betterdiscord://plugins/AutoIdleOnAFK.plugin.js:298:3)
    at Object.requireAddon (<anonymous>:4:43438)
    at Object.loadAddon (<anonymous>:4:9197)
    at Object.loadAddon (<anonymous>:4:41165)
    at Object.loadAllAddons (<anonymous>:4:11607)
    at Object.initialize (<anonymous>:4:6257)
    at Object.initialize (<anonymous>:4:40368)
    at Object.startup (<anonymous>:4:376262)

What about using the BDFDB lib? BDFDB.LibraryModules.SettingsUtils.StatusSetting.updateSetting

If the status module exposed via BDFDB works currently, then I'll have no problem using that in the plugin instead.

I'll look into that, thanks for letting me know!

Thank you for making this, sadly my JS skills are subpar or I'd give it a go myself.
The specific usage is shown here, https://github.com/mwittrien/BetterDiscordAddons/blob/241da8656dc90097924407c5a63fcd54c429dca5/Plugins/CustomStatusPresets/CustomStatusPresets.plugin.js#L393

Good news! thanks to the direction you've pointed me and looking through BDFDB's code, i have found the module used to change the status, and have successfully changed it via the console
I'll be working on a fix for the plugin now!
again, thank you @Violet-Vibes, im so happy the plugin wont have to stay dead anymore

Good news! thanks to the direction you've pointed me and looking through BDFDB's code, i have found the module used to change the status, and have successfully changed it via the console
I'll be working on a fix for the plugin now!
again, thank you @Violet-Vibes, im so happy the plugin wont have to stay dead anymore

I'm happy to hear that, thank you for fixing it.


Bug 1767004: defer provided api update in operator groups

This is the case of a bug where the CSV status wasn't updated due to an
invalid CSV. The cause for this was twofold:

The CSV and OperatorGroup reconcile loops were undoing each others
changes continously, giving no chance for the CSV to get synced beyond
provided api conflict detection.

Due to the way the provided APIs are produced differently for
operator groups (uses GVKSTringToProvidedAPISet) and OLM (uses
NewOperatorFromV1Alpha1CSV) when an invalid CSV is in use, the resolver
attempts to continue to instruct that an operator group annotation
update is needed beyond the point of that being true.

The changes are to ensure that OLM does not get stuck attempting to
update the operator group annotations when they are no longer needed
combined with ensuring that the operator group sync loop does not try to
prematurely dismiss a provided api before the CSV has a chance to check
the requirements and produce status.
In order to accomplish "premature detection", providedAPIsFromCSVs was
modified to not only return the APIs but also the CSV that provides a
given API.

Description of the change:
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

/retest

/retest
I think this PR is good to go, but CI seems to be broken right now.

/retest

/retest

/retest

In general, it seems that having any loops updating resources other than the one the loop was started for leads to bad contention. But operator groups and CSVs are coupled in a way that makes the problem extremely bad.

/lgtm


Fix bug incorrectly displaying the chance of rain

forecast.io gives us the percentage chance of rain from 0...1, not 0...100 (as
our code previously assumed)

Also renames the todaysConditions variable to currentConditions for clarity

Trello: https://trello.com/c/N3qcunas/35-reports-100-precipitation-as-sight-chance-of-rain
Pair: Adam Sharp

Oh no! Good catch. :+1:
Was wondering why I never saw good chance of rain while in NYC, hah

Nice fix! I actually noticed this behavior last night. I'm in Detroit and it was pouring rain for hours and Tropos was saying there was a slight chance of rain tonight. :+1:


fix #353: remove legacy endpoints support

Sorry @jayunit100 , I didn't see the "good first issue" :smiling_face_with_tear: I got too motivated by the cleanup

so motivated, leaving us newcomers amazed...

haha thank you sir !
/lgtm
/approve


Bitasset data

I have created a new bit asset data, but can't use

You'll need to be more precise if you want people to answer you request.
Why can't you use it?
Also, the official bitshares repo is no bitshares/bitshares-core

I use "create_asset" method and create market-issued asset.
When I use "issue_asset" method for issue new shares of an asset, method returns me
"
....
10 assert_exception: Assert Exception
!a.is_market_issued(): Cannot manually issue a market-issued asset.
....
"
Thanks for your feedback.
Ghazar.

@GhazarArm market-issued asset can only be issued by the market but not the owner. Firstly you need to set price feeds for the asset, then use the "borrow_asset" command to create new tokens.
This is by design. For more info (e.g. how to use and etc), please discuss / ask in the forum.

How to set price feeds ?

The asset id in quote in core_exchange_rate is wrong .. it should be 1.3.xxx not 1.4.xxx

I tried it
publish_asset_feed hayq MCORE {"settlement_price": {"quote": {"amount": 1000000, "asset_id": "1.3.0"}, "base": {"amount": 8563, "asset_id": "1.3.474"}}, "core_exchange_rate": {"quote": {"amount": 8563, "asset_id": "1.3.574"}, "base": {"amount": 1050000, "asset_id": "1.3.0"}}} true
10 assert_exception: Assert Exception
feed.settlement_price.base.asset_id == feed.core_exchange_rate.base.asset_id:
{}
th_a  asset_ops.cpp:141 validate
{"publishing_account":"hayq","symbol":"MCORE","feed":{"settlement_price":{"base":{"amount":8563,"asset_id":"1.3.474"},"quote":{"amount":1000000,"asset_id":"1.3.0"}},"maintenance_collateral_ratio":1750,"maximum_short_squeeze_ratio":1500,"core_exchange_rate":{"base":{"amount":1050000,"asset_id":"1.3.0"},"quote":{"amount":8563,"asset_id":"1.3.574"}}},"broadcast":true}
th_a  wallet.cpp:1246 publish_asset_feed

How to get price_feed in publish_asset_feed()?

Either your asset id is 1.3.574 or 1.3.474 not both

I put in
borrow_asset account_name 3000 asset_name 300 true
and get this error

...
exception: unspecified
10 assert_exception: Assert Exception
enable_black_swan: Black swan was detected during a margin update which is not allowed to trigger a blackswan
{}
th_a  db_update.cpp:250 check_for_blackswan
{}
th_a  db_market.cpp:547 check_call_orders

...
0 exception: unspecified
10 assert_exception: Assert Exception
enable_black_swan: Black swan was detected during a margin update which is not allowed to trigger a blackswan
{}
th_a  db_update.cpp:250 check_for_blackswan
{}
th_a  db_market.cpp:547 check_call_orders

{"o":{"fee":{"amount":2000000,"asset_id":"1.3.0"},"funding_account":"1.2.18","delta_collateral":{"amount":30000000,"asset_id":"1.3.0"},"delta_debt":{"amount":30000,"asset_id":"1.3.4"},"extensions":[]}}
th_a  market_evaluator.cpp:275 do_apply

{}
th_a  evaluator.cpp:51 start_evaluate

{}
th_a  db_block.cpp:662 apply_operation

...

you need more collateral to borrow that asset!


Fix GemHelper to allow reading Input/Output

What was the end-user problem that led to this PR?
Note: This PR is a super seed of #7005.
Currently, users that have enabled 2FA in RubyGems.org with both Web+API are not able to upload their gems via rake release, due to a bug in GemHelper#sh_with_status not handling input/output correctly.
What was your diagnosis of the problem?
See https://github.com/bundler/bundler/issues/6854
What is your fix for the problem, implemented in this PR?
See  https://github.com/bundler/bundler/issues/6854 as this PR is mostly the same.
The major difference here is that i've merged the change into #sh_with_status and that i'm using IO#getc instead of IO#gets. This is because commands could potentially be asking for input on the same line as output, resulting in output that still hasn't been printed.
Example using passwd
# Using IO#gets
➜ ruby -I lib -rbundler -e 'Bundler::GemHelper.new.send(:sh, "passwd")'
Changing password for colby.
<INPUT>

# Using IO#getc
$ ruby -I lib -rbundler -e 'Bundler::GemHelper.new.send(:sh, "passwd")'
Changing password for colby.
Old Password:<INPUT>

Closes #6854 #7005

I'm not particularly sure how to go about testing this, any ideas welcome.

I made one suggestion in the other PR.
In particular, have a look at the specs that call the bundle helper with a block
https://github.com/bundler/bundler/blob/48e767da514bb2280b7694a97797eab3904f0442/spec/commands/newgem_spec.rb#L829-L871

Superseded by #7199.


footer statistics

Adds statistics information in the footer

closes #196

add commit date to footer

review notes and added test.
@NothingEverHappens plz re-check


How to manage correctly 2 tests of the same type in the same product?

I have 1 product.
In this product, I have 1 engagement of type CI/CD.
In this engagement, I have 2 tests of type Sonatype (or whatever), one for the JavaScript client application located in a specific repository, another for the C# server application located in another repository.
When I import a scan result via the API, with the parameter close_old_findings set to true, it will close ALL the findings from the other test, which is not what I need.
If I enable the parameter Deduplication on engagement on my current engagement, I lose the capability to deduplicate accross other engagements, which is a requirement in order to follow the finding accross the environment. I have 1 engagement per environment, I cannot have findings duplicated X-times.
It would be great to be able to close old finding only for the current test, and not for all the tests of the same type... Even if I need to set a new flag like close_old_findings_same_test !

OK we have found a workaround .
When importing a scan test for the first time, set the parameter close_old_findings  to false, or it will close the other tests of the same type.
When re-importing a scan test, you can set the parameter close_old_findings to true, it will close only the current test, as expected.

Thanks for following up with your workaround!


Can I use bootstrap?

After several days of testing, I concludes that this code is very useful for testing heaan in python.
But when I tried to bootstrap, an "IndexError:map::at" error occured.
Can someone help me by checking the normal working of bootstrap?

Can you post the error?
Try to validate the code in c++.

Oh, it works fine now. It was caused by not adjusting logq in bootstrapAndEqual function.
But I still can't understand what logq or logp do in this algorithm. Can you recommend a paper?

You can refer to these issues from HEAAN 28, 51 or the paper https://eprint.iacr.org/2018/153.pdf.

Thanks a lot!!


Typo on log demo code example

This line should be *node->get_clock() right?
https://github.com/ros2/ros2_documentation/blob/b9413a79d1574cd1a1a637f7fef881d8f95f52c3/source/Tutorials/Demos/Logging-and-logger-configuration.rst?plain=1#L108

This line should be *node->get_clock() right?

Yes, absolutely.  Please feel free to open a pull request to fix it, and we'll be happy to review and merge it.

I've just sent a PR. It's ready for review.

Closing now.


MP4 Doesn't Load on Mobile

It seems there is a loading and preview issue with MP4s on mobile. I minted a MP4 today and the preview loads and then you are able to play it on desktop. However, on mobile (iOS, haven't tried android), there is no preview and when play button is pressed, nothing loads.

HTML5 Video tag not working in Safari , iPhone and iPad

I guess introducing own mp4 player might do the job.
Putting $$ if someone want's to hack on this.

Looks like some props fixed the issue


Issue with initial timer expiry started with uv_timer_start

I have a C application and in that i do have a uv_loop. At some time in my applcation, i start a timer as shown below.
assert(uv_timer_init(&link->loop, timer)==0);
assert(uv_timer_start(timer, modbus_ctl_periodic_monitor, 0, 1000)==0);
some times after starting the timer as shown above, it works correctly as in timer immediately starts and i get the callback and thereafter i get the callback periodically.
But there are times, when i do not get the initial callback for 20 to 40 seconds. When it starts thereafter, it periodically expires correctly.
During the time when it does not work (after calling uv_timer_start), i see the following in gdb
#0  syscall () at ../sysdeps/unix/sysv/linux/x86_64/syscall.S:38
#1  0x000000000041a78a in uv.epoll_wait ()
#2  0x0000000000418ddc in uv.io_poll ()
#3  0x0000000000410030 in uv_run ()
#4  0x00007ffff70b6802 in dslink_handshake_handle_ws () from /home/ghalwasi/work/iox-services/c-mlib/deps/libs/libsdk_dslink_c.so
But when it starts working, following is the traceback.
(gdb) bt
#0  0x00007ffff6ba3c53 in select () at ../sysdeps/unix/syscall-template.S:81
#1  0x00007ffff6e7daf5 in _connect (sockfd=29, addr=addr@entry=0x7ffff3754bd0, addrlen=addrlen@entry=16, ro_tv=ro_tv@entry=0x7fffdc0079e0) at modbus-tcp.c:284
#2  0x00007ffff6e7dbcc in _modbus_tcp_connect (ctx=0x7fffdc0079d0) at modbus-tcp.c:344
#3  0x000000000040ca63 in mb_master_connect (mb_ctl=0x7fffc800d190) at modbus_master.c:67
#4  0x000000000040b868 in modbus_read_snapshot (mb_ctl=0x7fffc800d190) at modbus_ctrl.c:523
#5  0x000000000040bac4 in modbus_ctl_periodic_monitor (timer=0x7fffc8010690) at modbus_ctrl.c:569
#6  0x00000000004170ae in uv.run_timers ()
#7  0x000000000040ff9a in uv_run ()
#8  0x00007ffff70b6802 in dslink_handshake_handle_ws () from /home/ghalwasi/work/iox-services/c-mlib/deps/libs/libsdk_dslink_c.so
Requesting you to kindly help me in finding out what is this issue and fix it eventually.

How are you running your libuv loop? I see there are other libraries involved, any chance you are running a blocking function in a libuv callback?

Thanks for responding.

How are you running your libuv loop?
uv_loop_init(&link->loop);
uv_poll_t *poll = dslink_calloc(1, sizeof(uv_poll_t));
{
uv_poll_init(&link->loop, poll, link->_socket->socket_ctx.fd);
poll->data = link;
uv_poll_start(poll, UV_READABLE, io_handler);
}
uv_timer_t *ping = dslink_calloc(1, sizeof(uv_poll_t));
{
uv_timer_init(&link->loop, ping);
ping->data = link;
ping->close_cb = ping_timer_on_close;
uv_timer_start(ping, ping_handler, 0, 30000);
}
uv_run(&link->loop, UV_RUN_DEFAULT);

This is the code for initialization. Later the same loop is used for initialzing the timer which i was talking about in the problem i described above.

I see there are other libraries involved, any chance you are running a blocking function in a libuv callback?
Not really. But some of the call backs may be doing few things for few milli seconds. Even if it is few milli seconds, how can it explain 30 to 40 seconds of delay.?

Later the same loop is used for initialzing the timer which i was talking about in the problem i described above.

How exactly? I see you run the loop with UV_RUN_DEFAULT so you can't call uv_run again, uless you stop it first, it's not recursive.

How exactly? I see you run the loop with UV_RUN_DEFAULT so you can't call uv_run again, uless you stop it first, it's not recursive.
As i have loop pointer (&link->loop), i dont call uv_run again. Basically at some later point when i want to create and start a new timer, i just initialize a new timer using uv_timer_init (specifying the loop pointer &link->loop) and start it with uv_timer_start. Thats when i see that it takes 30 to 40 seconds for it to get fired for the first time.

As i have loop pointer (&link->loop), i dont call uv_run again. Basically at some later point when i want to create and start a new timer, i just initialize a new timer using uv_timer_init (specifying the loop pointer &link->loop) and start it with uv_timer_start. Thats when i see that it takes 30 to 40 seconds for it to get fired for the first time.

Do you do this from another thread?

Do you do this from another thread?

Yes.

I'm afraid you voided the warranty.  Libuv is not thread-safe, so creating
a timer from another thread is not supported.  It may break in many ways,
or misbehave as you observed.
This is by design.
Saúl
On Nov 25, 2016 22:59, "gauravh278"<EMAIL_ADDRESS>wrote:

Do you do this from another thread?
Yes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/libuv/libuv/issues/1153#issuecomment-263026540, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AATYGOfNBqQfyRlMembGoTw6b3nBPiNvks5rB1o9gaJpZM4K8HvH
.

I'm closing the issue since it's not a libuv problem.  If you need help finding ways to make your program use libuv safely please use the mailing list and we'll hel you there. Thanks!

Thanks for pointing it out.

Just in case you can quickly share your thoughts.
So in case if i want to start a timer (for something) from a different thread, as per your explaination, it looks like i have 2 options:-
1.) Either create a new libuv loop and then start a timer using that loop.
or
2.) Use some other timer library/API to start a time.
Let me know if my understand is correct.

@gauravh278 That's correct.  You can also coordinate with the other thread using uv_async_send() but see the documentation for caveats.


applicationinsights: updating the swagger to match the breaking api change

As per this comment the API now returns a 201 when it previously used to return a 200 - as such both a 200 and 201 should be documented in the Swagger to match the API definition.
The breaking API change should also be reverted, but that's a separate problem.
FYI @JeffreyRichter @jhendrixMSFT

Backend has been fixed and returning 200. I don't think existing versions of the API should be changed.

@grlima since this is the second time this breaking change has been unintentionally rolled out to this  particular API, whilst I agree it should only highlight a 200, is there any harm leaving this in incase this surfaces again?

consider breaking change and backend reverted to returning 200 and original issue closed, I will close this PR for now. pls re-open if need


Use correct classes for order flexbox docs.

Saw this when I looked at #21627. .order-* classes don't exist, so we shouldn't mention them.

Ah, these should start with .order- instead of .flex given it should be .property-value. New PR opened to try to tackle that, though I'm unsure on the naming scheme for them (which is likely why I had flex in here to start.


Update MD file on new branch 2017-20-08_14-11-46

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

Line 16: [Error] Restore nuget package W:\t5f3-s.optemp\packages\opbuild.scripts.1.41.0-alpha-778-g9855cfb\tools\packages.config in folder:W:\t5f3-s.optemp\packages failed, possibly due to transient errors from nuget servers. Please retry building your content. If the issue still happens, open a ticket in http://SiteHelp. Detailed error: Unable to read package from path 'opbuild.filemappreprocessor.1.41.0-alpha-778-g9855cfb.nupkg'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update gulp-sass for compatibility with current versions of Node

Update gulp-sass to a newer version in order to bring
in a node-sass version which is compatible with current
stable Node releases (v4+)

This might actually be better solved just by updating to the current version of Sage


Adding celery integration in sentry and add False as default

Description
I added celery integration.
Also I set a default to False because we are using the same settings for development and also for testing where we don't have sentry and we don't need sentry there. Then I think it's better to have it as False by default and add the URL where we really need, like in production
Code formating
The code has to be well formatted following the formatting rules. For example in Python the code has to follow the PEP8.
Before create your PR, please make sure your code is well formatted and styled doing:
$ >> pre-commit run --all

Hi! I also fixed in a commit of my PR
https://github.com/GeeksCAT/anem-per-feina/pull/110/commits/84a5f860c201faf5749887df5e6b916da3362e78
But it's ok for me fix it in this PR.

@jbagot Sorry, I didn't realize that we need to add Celery in requirements.txt


I am getting an error with the links

Once I go to the About page or the tags page or a particular tag, I can't use any of the links in the header.
http://localhost:8000/tag
I get links like these which all go to the 404 page
http://localhost:8000/tag/tag
http://localhost:8000/tag/about
http://localhost:8000/tag/tag/angular

Hi @ahande123 and thank you for the issue because it can improve the project.
Since I wasn't able to replicate the issue neither on my local environment nor on production, I would ask you:

Are you able the replicate the issue on the production ?
Which browsers are you able to replicate the issue on?
Since you navigate from http://localhost:8000, are there errors in the browser console? If yes, can you share them?
Which OS are you using? Which version of Node.js, NPM or Yarn are you using?

Issue closed since there was any response by its author.

Issue closed since there was any response by its author.


Remove ssl.enable flag

Doesn't seem to be used anywhere.  Should clean up the setting.

There's one more place where this configuration is mentioned:
lib/perl/lib/Apache/TS/AdminClient.pm:602: proxy.config.ssl.enabled


No error raised from exceptions.py when trying to publish a workbook

Hello,
I'm having trouble figuring out what is my problem on this.
I used move_workbook_sites.py sample to try to make something different.
I wanted to take one workbook from a specific site/project and publish a copy of it on a different project.
So it's different than just changing the project_id like in move_workbook_projects.py sample.
My source_site is the same as the dest_site but it doesn't matter and I changed the code to add a dest_project.
The problem I have is a bit weird : when I try to publish the workbook (using "new_workbook = dest_server.workbooks.publish(new_workbook, workbook_path, mode=TSC.Server.PublishMode.Overwrite)" ) my code raises an error but the Tableau Server Client library doesn't give me the correct error and just post this final error on exceptions.py :
File "D:\Users\user_1\AppData\Local\Programs\Python\Python37-32\lib\site-packages\tableauserverclient\server\endpoint\exceptions.py", line 18, in from_response
error_response = cls(parsed_response.find('t:error', namespaces=ns).get('code', ''),
AttributeError: 'NoneType' object has no attribute 'get'
I've checked on the server, I have the rights to publish using Tableau Desktop with same user and password.
I've also managed to download the workbook to copy in a temp folder like in move_workbook_sites.py.
What could be the problem about exceptions.py and also what could cause my code to fail this way on its way to publish the report ?
Thanks in advance for your help about this issue and have a great day !
Thomas

I have the same problem when I try to use publish_workbook.py with the same workbook and without changing anything about the file.
Maybe there's something wrong about the way I access my Tableau Server but I could totally do other things :

change project_id of a workbook
download a workbook
sign in without any issue

Does someone have an idea of what could be wrong ?
Thanks a lot in advance.

May be similar to #358 ?  Can you publish the workbook manually?
Please try the latest version of TSC and let us know how it goes.

I'll go ahead and close this one due to age, but please reopen (or create a new issue) if needed.


whitespace at the start/end of mtext elements is lost on rendering

White space at the start or end of mtext elements (at least) is being lost when rendered.
For example, <mtext> and </mtext> is rendered as "and" rather than " and ".
Here's a live example: http://fiddle.jshell.net/hubfiddle/Evwhj/show/light/

Is there an option to disable this?
If not what part of the JS can I alter to stop this behavior. I am using wolfram's MathML output and wolfram dose not respect the MathML specification (see Section 2.1.7)

@abinash18, no there is no option to disable this, as that is what the specification requires.
If you are getting the MathML from Wolfram as a serialized MathML string, then you could use
mathml.replace(/<mtext>( +)/, (match, spaces) => '<mtext>' + spaces.replace(/ /g, '\u00A0'))
      .replace(/( +)<\/mtext>/, (match, spaces) => spaces.replace(/ /g, '\u00A0') + '</mtext>')

If you want to retain multiple internal spaces as well, you could just do
mathml.replace(/<mtext>(.+)<\/mtext>/, (match, text) => '<mtext>' + text.replace(/ /g, '\u00A0') + '</mtext>')

Otherwise, the code that does it is this line:
https://github.com/mathjax/MathJax-src/blob/41565a97529c8de57cb170e6a67baf311e61de13/ts/input/mathml/MathMLCompile.ts#L247
You could use a startup.ready() function in your configuration to override this method of the MathMLCompile object.


Game against Priest not recorded

I was just playing a game against a Priest and my game was not recorded. I looked into the debug logs and saw this curious line:
[2015-11-17 00:08:34.474][engine][log]: {:stats=>:win, :against=>nil, :with_deck=>"MY_DECK_NAME", :rank=>15}

This line seems weird because the :against field is usually filled in with the opponent's class's name, not nil. Sounds like the class name did not get recorded properly, despite my battle clearly starting with this line:
[2015-11-17 00:00:48.030][engine][log]: {:opponent=>"Anduin Wrynn", :hero_id=>"HERO_09"}

So, HSTracker knows I'm playing Anduin, but can't figure out that he is a Priest. I'm confused. Anyways, recording a win against a nil opponent class resulted in the win not being recorded in my deck's stats. Is HSTracker suddenly forgetting which class I'm playing against a known issue?
Another funny thing I noticed (no idea if it is related) is that this was the first game I played after midnight struck in a session that technically began the day before. So, the complete console logs for this game was being printed in the previous date's file, not the log file for the current date. Could that have caused the problem?

Same issue here.
I've noticed for a while that HSTracker doesn't always recognize the opponent's class (not having  the windows "locked" makes it easy to see, because the opponent's deck window normally changes its titlebar to match their class), and that my matches aren't always showing in HearthStats... but kudos to you for figuring-out that they're related.
There also doesn't seem to be any consistent pattern.  In my last session, 2/3 of my priest match-ups (and 2/5 of my Rogue match-ups) suffered from it... but certainly not all of them.
It's an odd one to be sure.

for the opponent name, this is something like   :type   => :engine,  :data   => {    :opponent     => "Nitroman4eg"}
that you have to find in the log

This should (maybe) be resolved in next release


Add new apis for test button feature

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Hi @thomasp98296 could you please take a look at the issues in ModelValidation?
@jianyexi could you please take a look at the issues in LintDiff? The model collision will always block the LintDiff in monitor

Hi @thomasp98296 could you please take a look at the issues in ModelValidation? @jianyexi could you please take a look at the issues in LintDiff? The model collision will always block the LintDiff in monitor

Hello, the failure in the model validation is an existing issue. You can see it is the same in the old version. Our API is a little different where it will return only the "notificationId", and caller used the notificationId to make different call (GET) to get the notification status. Thank you for your review.

Hi @thomasp98296 could you please take a look at the issues in ModelValidation? @jianyexi could you please take a look at the issues in LintDiff? The model collision will always block the LintDiff in monitor

Hello, the failure in the model validation is an existing issue. You can see it is the same in the old version. Our API is a little different where it will return only the "notificationId", and caller used the notificationId to make different call (GET) to get the notification status. Thank you for your review.

Hi, @ArcturusZhang ArcturusZhang, @jianyexi could you help? Thanks so much.

Hi @thomasp98296 any update on this?

Hi @thomasp98296 any update on this?

Thanks. Please advise on the next step. The breaking changes is due to the removal of the response body, part of the "location headers" changed that I have been asked to do. Thanks!


[Java] Optimize the copy performance of some thread-safe classes

Is your feature request related to a problem? Please describe.
Some thread-safe classes use a generic copyElements() method, for example: CopyOnWriteArrayList...
when adding elements, locking operations are performed, which is very time-consuming.
Describe the solution you'd like
Optimize the performance of these copy classes
Additional context

I will deal with this issue

I will deal with it. If there is anything to add, please let me know. @chaokunyang


keyPrefix does not apply to publish/subscribe

I was going to submit a pull request, but ioredis depends on redis-commands, and the fix is both in ioredis and redis-commands.
In redis-commands/commands.json,   subscribe/unsubscribe/psubscribe/punsubscribe should all be changed from:
    "keyStart": 0,
    "keyStop": 0,
    "step": 0

to
    "keyStart": 1,
    "keyStop": 1,
    "step": 1

This will cause the keyPrefix to be applied to the commands sent to redis.  The next step is in parser.js - removing the keyPrefix from the channel before emitting it.
My fixes, starting at line 105 of parser.js.  Use/apply as appropriate:
  var item, channel, count, message;
  if (this.condition.subscriber) {
    var replyType = Array.isArray(reply) ? reply[0].toString() : null;
    debug('receive reply "%s" in subscriber mode', replyType);
    switch (replyType) {
    case 'message':
      message = (this.options.keyPrefix) ? reply[1].toString().substr(this.options.keyPrefix.length) : reply[1].toString();
      if (this.listeners('message').length > 0) {
        this.emit('message', message, reply[2].toString());
      }
      if (this.listeners('messageBuffer').length > 0) {
        this.emit('messageBuffer', new Buffer(message), reply[2]);
      }
      break;
    case 'pmessage':
      var pattern = reply[1].toString();
      message = (this.options.keyPrefix) ? reply[2].toString().substr(this.options.keyPrefix.length) : reply[2].toString();
      if (this.listeners('pmessage').length > 0) {
        this.emit('pmessage', pattern, message, reply[3].toString());
      }
      if (this.listeners('pmessageBuffer').length > 0) {
        this.emit('pmessageBuffer', pattern, new Buffer(message), reply[3]);
      }
      break;
    case 'subscribe':
    case 'psubscribe':
      channel =  this.options.keyPrefix + reply[1].toString();
      this.condition.subscriber.add(replyType, channel);
      item = this.commandQueue.shift();
      if (!fillSubCommand(item.command, reply[2])) {
        this.commandQueue.unshift(item);
      }
      break;
    case 'unsubscribe':
    case 'punsubscribe':
      channel = reply[1] ? this.options.keyPrefix + reply[1].toString() : null;
      if (channel) {
        this.condition.subscriber.del(replyType, channel);
      }
      count = reply[2];
      if (count === 0) {
        this.condition.subscriber = false;
      }
      item = this.commandQueue.shift();
      if (!fillUnsubCommand(item.command, count)) {
        this.commandQueue.unshift(item);
      }
      break;

This is similar to the warning message in https://github.com/luin/ioredis#transparent-key-prefixing. Since the option is keyPrefix, the prefix will be only applied to keys, instead of patterns and channels.
Your fix seems good to me, however I'm not sure if we should use another option (maybe channelPrefix) to set the prefix for channels.

Makes sense... if I wanted to submit a PR, should I first to it on redis-commands?

redis-commands/commands.json is generated by the script https://github.com/NodeRedis/redis-commands/blob/master/tools/build.js. It's just the result of COMMAND command, which I think we shouldn't change.
To support channelPrefix, we can write a argument transformer for publish command (we also have to pass the options to the transformer so it can know what to prefix https://github.com/luin/ioredis/blob/master/lib/command.js#L67).
However I think channelPrefix option is not necessary since it's not hard to be implemented by the users.


Library management

Documents the current libraries needed with author and version, as well as deleted a redundant library

Implemented in previous PR