CommandBarFlyout content extends beyond the taskbar

Describe the bug

CommandBarFlyout secondary commands content extends below the taskbar, which cuts off the commands. It does seem to recognize the bounds of the window itself (and will adapt when the content would extend below the window), just not the taskbar. This occurs when the taskbar is in bottom and left positions, it does not occur when the taskbar is in top or right positions.
Steps to reproduce the bug

Steps to reproduce the behavior:

Open xaml controls gallery (with winui 2.6 preview), navigate to CommandBarFlyout page
Drag window towards bottom of display
Right click image
Content extends below taskbar

Expected behavior
The CommandBarFlyout should move if it is at the bounds of the window.
Screenshots
In Xaml controls gallery with Microsoft.UI.Xaml 2.6.0-prerelease.210405001:

In an actual app (Files) with Microsoft.UI.Xaml 2.6.0-prerelease.210405001:

Left taskbar position:

This doesn't seem to occur when the taskbar is set to right position:

Version Info

NuGet package version:
[Microsoft.UI.Xaml 2.6.0-prerelease.210405001]

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (19043.962)
Yes

October 2020 Update (19042)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

Wow, for some reason our Bot never tagged this with Needs-Triage so we haven't seen it... Thank you for reporting @winston-de .

I beleive this issue has been fixed. @ranjeshj and @llongley FYI

@winston-de does this still repro with latest bits ? We have made some fixes in this area in windows 11, so you might not see this fix in windows 10 unfortunately.

No, this doesn't repo in 22000.71. Thanks for the fix!

Looks like this issue still seems to occur (build 22000.184) with the left and right sides of the screen...

Was this resolved?

@winston-de, thank you for reporting this. Unfortunately we are currently unable to investigate WinUi2 issues (more on that here, but I tested this on WinUI3 Gallery and am not able to repro in the scenario of your screenshot #1:

the flyout is positioned above the taskbar as expected when invoked near the task bar area. I suspect a fix was made in WinUI2 as suggested in earlier comments (and has been ported to WinAppSDK).

Resolving in light of the above. Please comment here if you still repro the problem in WinAppSDK.


feat: run query on variable and header editor through ctrl+enter

Fixes #1140
Changes proposed in this pull request:

runQueryAtCursor shouldn't check if the query editor is on focus, as it will never be when used by Variable and Header editor

 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.

The tests seems to be broken in master. Some are fixable through yarn test -u but the other ones are really broken

@brunoluiz try master branch now

I agree, this is how GrahiQL works, and whenever I built a GraphiQL POC that works this way people love it. the new vanilla monaco demo will work this way for example.

@acao I was not sure if you were relating to the feature itself or to the tests. As the feature seems to not be in master, I suppose you mean the tests.
I pulled master but it seems yarn test is still broken. I've done yarn install on both root folder and graphql-playground-react. Perhaps I am missing something on the test/development set-up?
➜  graphql-playground git:(master) cd ./packages/graphql-playground-react
➜  graphql-playground-react git:(master) yarn test
yarn run v1.22.1
$ jest
 FAIL  src/index.test.tsx
  ● Test suite failed to run

    SecurityError: localStorage is not available for opaque origins

      at Window.get localStorage [as localStorage] (../../node_modules/jsdom/lib/jsdom/browser/Window.js:257:15)
          at Array.forEach (<anonymous>)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        0.485s
Ran all test suites.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
➜  graphql-playground-react git:(master)

Argh, I rebased with master but it seems it deletes the PR -- Will re-open in a second and reference again here

I'm talking about the behaviour introduced.
GraphiQL works this way, for example, where you can execute queries from any editor instance.
As far as tests, I don't know what's the situation is here, I just got access last week and the tests seem to be mostly snapshots. Used to having a lot more test coverage!

going to merge for the next patch release! thanks :)


Add test globals to main .jshintrc for Hound CI

@tldavies @yyscamper

@heckj since you got the other one.

+1

+1


Sanitize nicknames to limit length to 80 bytes

Fixes #71

Truncates names longer than 80 bytes (for the Discord API, see #71)
Does not check for names of length 0 (I can't envision a scenario where this happens)
Does not handle Encoder/Decoder errors (though I couldn't produce any that should effect anything)

I can't imagine this will have a major performance impact, and I believe it'll catch every scenario.

Awesome, thanks a lot!


Rename 'master' branch to 'main'

Closes #141

@hughsk I do not have permission to rename the main branch, could you update this for me?

bump @hughsk

@hughsk any chance we can get this landed?


Add toLazyM'

with the same caveat as toLazy and toLazyM: but this is sometimes useful and is currently a slightly odd gap in the API

Yeah, you're right.  This should be included for consistency.  Thanks! :)

Alright, this is up on Hackage as pipes-bytestring-2.1.5

thanks!


Fix events in tutorial.

I believe this was wrong. Thanks for @choldgraf for pointing this out on gitter.
See if you're happy with the changes.

LGTM

Current coverage is 87.38% (diff: 100%)

Merging #3662 into master will not change coverage

@@             master      #3662   diff @@
==========================================
  Files           343        343
  Lines         60780      60780
  Methods           0          0
  Messages          0          0
  Branches       9302       9302
==========================================
  Hits          53111      53111
  Misses         4912       4912
  Partials       2757       2757

Powered by Codecov. Last update c22b9ae...2d2ff6f

thx @jaeilepp

ah ok cool - thanks for clarifying. I was worried that I had been giving people an incorrect description of the events array all this time...


Vary http header should include 'Origin' not the value of the origin

The vary currently includes the url of the origin, but it should just include the name of the header the response is depending on, and not the actual value: see https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Vary

Implementation actually is correct


Create bayschoolsohio.txt

Adding local school district

I'm sorry for the delay. The school's domain seems to be www.bayvillageschools.com/ - please confirm that committed address also belongs to the school. Thanks!

The domain is the .com, however the current email server is a .org.
Thanks!
On Monday, April 27, 2015, Elena Ganenkova<EMAIL_ADDRESS>wrote:

I'm sorry for the delay. The school's domain seems to be
www.bayvillageschools.com/ - please confirm that committed address also
belongs to the school. Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/JetBrains/swot/pull/873#issuecomment-96709486.


Descending Keys and DDL record upgrades in mixed version clusters

Descending Keys originally reviewed in https://github.com/basho/riak_kv/pull/1401 but has been blocked because it requires DDL record upgrade/downgrade support in mixed version clusters.
RFC:
https://github.com/basho/rfc/pull/35
Dependent PRs:

https://github.com/basho/riak_ql/pull/117
https://github.com/basho/riak_test/pull/1055

The "descending keys" part of this PR was reviewed in https://github.com/basho/riak_kv/pull/1401

thumbot retry

Can't get the staff, triggering a retry while the remedial PR was unmerged, who hired this @gordonguthrie clown anyhoo?

thumbot retry

Third time lucky becuz I am the muppet of all muppets...

thumbot retry

The one where thumbot is not retrying...

+1

+1 b6630a3

@borshop merge


Authentication Section I can't hide the URL

Bug report
What happened
Authentication Section I can't hide the URL
even normal user can access  below URLs.
/user
/role
/permission
Is it a bug in the latest version of Backpack?
Yes
Backpack, Laravel, PHP, DB version
PHP VERSION:
PHP 8.1.10 (cli) (built: Aug 30 2022 18:05:49) (ZTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.1.10, Copyright (c) Zend Technologies
LARAVEL VERSION:
v9.42.2@607d7867c93706eae20e28e46679f8a66e2a23ec
BACKPACK VERSION:
5.4.10@050b288caf2f60c79313f2b0b7d4fefd25177f3f

Hey @mikdanjey thanks for the report.
I was not able to reproduce your issue:

And when I go to admin/role I get redirected to login page if I am not authenticated.

Check how they are setup inside PermissionManager package: https://github.com/Laravel-Backpack/PermissionManager/blob/main/src/routes/backpack/permissionmanager.php
I am going to close this, I don't think there is any bug here, rather some problem in the implementation. Please feel free to continue the conversation if needed and I will help you the best I can.
Wish you all the best 🎅

Hello, @pxpm,
You though wrong, I think you are not understand my bug report.

Haa sorry my bad @mikdanjey I misunderstood your question.
One thing that I always do is to only have the ListOperation in the roles/permissions cruds. That way the routes for CreateOperation update etc are not even registered as I don't plan to ever edit any of them via interface.
Denying access works, but some user with access can change a role name or permission or something and break your code that rely on those role names etc to work.
Sorry again if I didn't understood the question at first.
Cheers

Please check my below screenshot for list of users.
I want to hide the normal users, This Authentication should visible only on Super-Admin.
I have checked your complete documentation & your source code. it seems your side is not providing the hide the Authentication module.

admin/user

Please give me a fix here.

To customize the sidebar you should go ahead to resources/views/vendor/backpack/base/inc/sidebar_content.
You can use something like:
@if(backpack_user()->hasRole('super-admin')
.... show the menu here only if user is super-admin
@endif

Cheers

I did this complete code already in side panel. This was hiding only in side panel. but not able to remove /admin/user - while browsing with direct URL.
How to reproduce my steps: == Consider this code is applied in Side panel with proper roles & permissions.

Login with normal User
You can't see Authentication menu in side panel. == Which is Correct
Browse direct URL /admin/user == You can see the users, this is very big issue.

@if (backpack_user()->hasRole('super-admin'))

     Authentication

 Users

 Roles

 Permissions

@endif

Hey @mikdanjey
To avoid the urls beeing accessible you can use what you already described, in your UserCrudController, in the setup() method, you can add:
public function setup()
{
if(!backpack_user()->hasRole('super-admin')) {
$this->crud->denyAccess(['list', 'create', 'update', 'show']);
}
}

If you don't want the routes to be registered at all you need to conditional load the OperationTraits.


[iPad?] Infinite-scroll indicator positioning

From KP:
Infinite-scroll:  the loading indicator is front-and-center.  It should probably just be at the bottom.
Not sure if this applies to regular devices also

Currently all the loading indicators across the app are displayed more or less in the middle.


[Bug] Monitor keyboard key combination exceptions on Windows systems

On Windows systems, monitor keyboard key combination exceptions. When using a key combination, ctrl + any letter key, the letter key is not displayed properly. It is normal to use keys like alt, or shift in combination with letters.

This seems to be an issue for https://github.com/moses-palmer/pynput/issues.
You can report there to improve the representation.
Instead, we'd probably add a transformer to improve the display string.

BTW when the modifier key changes the following key's semantic, it will display the effective key.
Said if you press Shift and 3, you'd get: Key.shift+'#' instead of `Key.shift+'3'.

Closing ...
Not quite an issue of this repo. Perhaps pynput not the best but I don't find other library better.

Reported to pynput in https://github.com/moses-palmer/pynput/issues/591.


Rename vm/src/stdlib/time_module.rs to time.rs

It doesn't fit to our naming convention.

I fixed this issue in #2885


Errors when trying to install due to package not in package.json

Steps to replicate the error

First I switched to Node.js v8.0.0 (I assumed it was supported),and installed relevant dependencies using NPM

nvm use v8.0.0;
git clone https://github.com/paritytech/oo7; cd oo7; npm install; cd ..;
git clone https://github.com/paritytech/oo7-parity; cd oo7-parity; npm install; cd ..;
git clone https://github.com/paritytech/oo7-react; cd oo7-react; npm install; cd ..;
git clone https://github.com/paritytech/parity-reactive-ui; cd parity-reactive-ui; npm install; cd ..;

But during the installation of the dependencies using NPM for parity-reactive-ui, it gave me the following errors. We need to save @parity/parity.js to package.json.

ERROR in ./src/AddressBond.jsx
Module not found: Error: Can't resolve<EMAIL_ADDRESS>
ERROR in ./src/AccountLabel.jsx
Module not found: Error: Can't resolve<EMAIL_ADDRESS>
My system
npm - 5.1.0
node - 8.0.0
macOS - 10.12.6

Closing this issue because it's relevant to fixing issue 16 in parity-reactive-ui repo, upon which this repo is dependent


update rpi_ws281x and handle death of rpi_ws281x process

update rpi_ws281x submodule
compile pcm.c -> pcm.o and add to link of rpi_ws281x
handle unexpected death of rpi_ws281x process

Awesome, thanks for taking the time to make this contribution! I'm working on getting this tested and merged now.

Thanks again for taking the time to make this PR! 🎈 🚀


Subscriptions lost after reconnection

Steps to reproduce:

Start Redis
Start a ioredis client with no options given and subscribe to a channel. When we run 'PUBSUB CHANNEL' in redis-cli, the subscription should show
Kill the redis process and start it again. Now 'PUBSUB CHANNEL' shows nothing.

After I looked a little bit at the code it seemed that when the client reconnects it checks the property prevCondition.subscriber to see if the client was in subscribe mode and to renew all subscriptions. However the value of this property is 'false', so the subscriptions are not renewed.

This issue happens when reconnecting failed more than one times. Fixed in the master branch. Thanks for reporting.

Thank you for the quick reaction.


KeyError: 'I-PRT'

While using Conll2003/en/ner data for training.
KeyError: 'I-PRT'

Hi @MesiA
Thanks for valuable information.
Please show me stacktrace.

I met the same problem, it seems like the problem occurs due to 'I-PRT' tag didn't exist in the training set of conll03 chunking but in the valid set. There is a bug in handling unseen label in the evaluation process.


GH-276: Rip out ListenerInvoker

Resolves: https://github.com/spring-projects/spring-kafka/issues/276
Since the implementation if KIP-62, it is no longer necessary to run the listener
on a separate thread, enabling pause/resume on the consumer thread.
Remove all logic and properties related to handing off to a ListenerInvoker and remove all
"slow consumer" tests.

LGTM, but one question:
Is that true that we don't have one TaskExecutor any more?
I mean the recent doc about threads in the ListerContainer must be polished with this PR, too.

Sorry - yes; I meant to change that too - will push...

Merged as https://github.com/spring-projects/spring-kafka/commit/9d533988194ae0e50859b4772ad504ed9943817c.
Let me know if that should be cherry-picked to 1.2.x!


About Training Epochs

The Training Epochs is 240 in "Readme.md", but 24  in "joint.py". Could you tell me what setting is for the best resulting?Thanks!

The config is still using 240 epochs:

Since I'm using a RepeatDataset with times set to 10 (for saving time between epoch switches), it's still 24 x 10 = 240 epochs.

The config is still using 240 epochs:
Since I'm using a RepeatDataset with times set to 10 (for saving time between epoch switches), it's still 24 x 10 = 240 epochs.

Thank you!


added a numbers example and changed some of the sentences

Preview Changes at:
http://bw-apdocs-updates-to-accounts.s3-website-us-east-1.amazonaws.com/

Preview Changes at:
http://bw-apdocs-updates-to-accounts.s3-website-us-east-1.amazonaws.com/

Preview Changes at:
http://bw-apdocs-updates-to-accounts.s3-website-us-east-1.amazonaws.com/


How to use playwright light house for authenticated links

How to use playwright light house for authenticated links

@abhinaba-ghosh , Could you please help me to understand this. When I try to configure playwright-lighthouse, the page is redirecting to the login page always. Do I need to add any settings to work with this?

No action required


Add stuck queue check to health status

This PR adds a task stuck queue check for checking pending tasks that:

Have lasted a maximum threshold as pending
Have exceeded the maximum number of pending tasks allowed

The implementation shoould cause the health check page to error if there are N amount of unprocessed tasks or if there are unprocessed tasks older than X time.
The task is : https://app.clickup.com/t/14289126/FC-2768

PR looks good, but i think we need to make sure all task queues are cleaned before we deploy this, so retargeting to draft-pr-branch for now.

Lets rebase and adjust this for the changes in https://github.com/fullctl/fullctl/pull/226 once that's merged

something about the commit history is janky, will reopen a new pr


Add Emby Support

I'm expecting a bit of pushback, but I'm mostly happy with this as of right now.

Couldn't figure out a config, made the "isVideoPlayerPage()" logic quite a bit more complicated, the host regex is overly simple as is...
path doesn't work since emby uses the same path for everything
Might be able to just go through ApiClient Directly? But i already had the code written to hit the RESTApi directly.

examples of urls where we notice the video player and attempt to get subtitles.
http://ip:8096/web/index.html#!/videoosd/videoosd.html
https://emby.custom.url/web/index.html#!/videoosd/videoosd.html

Ah crap, can you run prettier on the changed files? Will merge it in after that.

 Will merge it in after that.

Done.

Thank you!


Support for building iOS app with Swift Package Manager

With Apple announcing support for Swift Package Manager for iOS in the next release of Xcode, I would like to see this support added here as well.

@w0mba7 What exactly are you looking for? App Center's SDKs being available through Swift Package Manager?

I was just actually looking to be able to build an iOS project using Swift Package Manage to pull in dependencies, the feature added in Xcode 11.  Support for being able to add the SDK using SPM would be nice as well.

@w0mba7, we're taking a look at adding this support for Build. Does that sound like what you are looking for?

yup, thanks.

Current error.

Has there been any work on this?

Hi @egfconnor, this hasn't been added in yet.

@w0mba7 are you seeing errors building an iOS app with swift packages using App Center? If so, could you show me the error? Would be the best to have a sample project that reproduces the issue. From our testing, it resolves the dependencies correctly and built successfully.
For all that follows along the Issue, I'm scoping this down to supporting iOS app with swift packages in Build service. If you are looking for support of adding App Center SDK through Swift Package Manager, they are tracked in #1168 and #1295

@DennisPan I'm facing the same issue trying to build an Xcode project on app center which relies on a local (same git repo) swift package. The error I'm getting is the following:
User defaults from command line:
    IDEArchivePathOverride = /Users/runner/runners/2.160.1/work/1/output/build/archive/my

Build settings from command line:
    CODE_SIGN_STYLE = Manual

note: Using new build system
note: Planning build
note: Constructing build description
Build system information
error: Missing package product 'MyLocalSwiftPackage'

** ARCHIVE FAILED **

##[error]Error: /usr/bin/xcodebuild failed with return code: 65```

What I tried is following Apples [SPM CI Article](https://developer.apple.com/documentation/swift_packages/building_apps_that_use_swift_packages_in_continuous_integration_workflows) resulting in the following post-clone script:

#!/usr/bin/env bash
git submodule update --init --recursive;
xcodebuild -resolvePackageDependencies -forceResolvedPackageVersions
``
which is not working unfortunately. Is there any way to pass xcbuild arguments directly via the configuration? If that is not the case what could be another approach to make that work?

I am trying to build the Swift-Package directly in the Appcenter Build pipeline.
Appcenter detects the package workspace (probably in the .swiftpm folder) but does not find the shared scheme that includes the test target.

Did you consider supporting SwiftPM directly (without any real Xcode project)? There is this hidden Xcode workspace in .swiftpm  though

@christianhuck Regarding the question

which is not working unfortunately. Is there any way to pass xcbuild arguments directly via the configuration?

That's not configurable at the moment.
@tapwork No immediate plan on supporting building Swift package at the moment. Feel free to submit a new feature request (I don't recall we have one tracking that)

@DennisPan that's a pity! The community is currently switching to Swift PM . Preferred over CocoaPods and Carthage.
I totally like the nice and clean App Center interface. But due to the lack of real a CLI and missing Swift package support it is not usable for real projects right now.

@tapwork To clarify, we do support build iOS app using Swift packages. Just don't support building the packages only.

I'm unable to build my app that has a Swift Package. The cocoapods packages get pulled in fine, but during the build I get
Resolve Package Graph
Fetching git@github.com:dmytro-anokhin/url-image.git

xcodebuild: error: Could not resolve package dependencies:
  The server SSH fingerprint failed to verify.

##[error]Error: /usr/bin/xcodebuild failed with return code: 74

@Bendetta could you see if the suggestion here works for you? https://stackoverflow.com/questions/58125659/github-actions-xcodebuild-fails-due-to-server-fingerprint

For anyone using an Azure DevOps Repo with temporary tokens instead of SSH, following this worked for me:
https://confluence.atlassian.com/fisheye/permanent-authentication-for-git-repositories-over-http-s-960155789.html

I'm not sure I understand this issue, the problem is that if the project uses SPM as dependency manager it will not pull dependencies, i.e. will not be buildable in app center?
From what I can see in the docs it only work CocoaPods, is this correct?

@DennisPan  Hi, i'm also trying to compile ios app using spm in app center, and the build failed (swift compile generation...)

@RiviElfNeura Sorry for late response. Could you reach out to support about the build failure?

I have a similar issue building my app in AC using PM to bring in the Facebook SDK - getting this error:
xcodebuild: error: Could not resolve package dependencies: because no versions of facebook-ios-sdk match the requirement 7.0.2..<8.0.0 and root depends on facebook-ios-sdk 7.0.2..<8.0.0, version solving failed.
It seems that this should be supported by AC, but not sure how to solve this problem.  Anything I should try?

I have a similar issue building my app in AC using PM to bring in the Facebook SDK - getting this error:
xcodebuild: error: Could not resolve package dependencies: because no versions of facebook-ios-sdk match the requirement 7.0.2..<8.0.0 and root depends on facebook-ios-sdk 7.0.2..<8.0.0, version solving failed.
It seems that this should be supported by AC, but not sure how to solve this problem. Anything I should try?

Have anyone found a solution? I have the same error

@DennisPan I have a similar issue building my app in appcenter using Swift Package Manager from private repository, getting this error:
xcodebuild: error: Could not resolve package dependencies:
  Authentication failed because no credentials were provided.
##[error]Error: /usr/bin/xcodebuild failed with return code: 74
Any suggestion! my CI process is stopped

We have the issue. We have a swift package in private Github repository A and a project-iOS app in a Github repository B that uses the Swift package at repository A.
When trying to build the project-iOS app at App Center we get an authentication error.
xcodebuild: error: Could not resolve package dependencies:   The server SSH fingerprint failed to verify.
We have contacted the App center's support without any answer yet.
Can you provide detailed steps to resolve this issue?

@AntonioCarrascoGBM @ptsiogas
I had the same issue but found a solution based on these posts:
https://discuss.bitrise.io/t/spm-gitlab-the-server-ssh-fingerprint-failed-to-verify/15592/3
https://github.com/microsoft/appcenter/issues/237#issuecomment-527991036
#237
Add the following to the appcenter-post-clone.sh file (this example is for Bitbucket but should work for GitHub as well):
# Add Bitbucket fingerprint to known_hosts
mkdir -p ~/.ssh

# Find Bitbucket IP's for adding to App Center's ssh known hosts: https://discuss.bitrise.io/t/xcode-11-resolving-packages-fails-with-ssh-fingerprint/10388/2
# To avoid this error:
#   xcodebuild: error: Could not resolve package dependencies:
#   The server SSH fingerprint failed to verify.
echo "Adding bitbucket to ssh known hosts"
for ip in $(dig @<IP_ADDRESS> bitbucket.org +short); do ssh-keyscan bitbucket.org,$ip; ssh-keyscan $ip; done 2>/dev/null >> ~/.ssh/known_hosts

# Print out ssh known hosts for debugging
echo "Known hosts:"
cat ~/.ssh/known_hosts

# Add Bitbucket ssh key
echo "Adding bitbucket SSH key:"
echo $SSH_KEY | base64 -D > ~/.ssh/bitbucket-ssh
chmod 600 ~/.ssh/bitbucket-ssh
ssh-add ~/.ssh/bitbucket-ssh

Generate a base64 formatted SSH key like this:
base64 -i ~/.ssh/my_private_key | pbcopy
Add the SSH key to App Center -> Build Settings -> Environment Variables:
SSH_KEY = {key}

@ptsiogas
we have same issue, do you found any solution ?
if not, can this one be re opened ?

@MaxenceMax no we did not found a workaround. We switched to private Cocoa pod libraries.

@ptsiogas
I've just found a solution. we have SP in a private gitlab repo, so I added a read-only token access in those repos and added them like that in SPM :
<EMAIL_ADDRESS>Of course our app repo is private so the token is safe.

@MaxenceMax
Where did you save such URL?
Does not seem Xcode Package Dependency GUI would allow you to set such URL.
Is there a way to use git@github.com:dmytro-anokhin/url-image.git type URLs instead of https://github.com/dmytro-anokhin/url-image.git in Xcode 13 ?

I am having the same issue; private repo in GitLab that contains our Swift Package.  I am getting no errors in the logs, just times out.   I don't see away to provide a url in the form of<EMAIL_ADDRESS>with in Xcode 13.
Has anyone found a solution?

I am facing the same issue here using Xcode 13.3

Same problem !

Anyone else having problems with AppCenter's Xcode 13.3.1+ and SPM?
If I use Xcode 13.2.1 the build works fine and all SPM dependencies are properly downloaded and installed
But if selecting newer Xcode versions, the build always gets stuck forever while downloading a random dependency, until it timeouts after 1h and fails  .

Any workarounds for this? AppCenter support basically said they cannot do anything because "SPM is not officially supported" 😓

I'm facing a problem with publickey using a private repository.
xcodebuild: error: Could not resolve package dependencies:
  Failed to clone repository git@github.com:Infoglobo/ui-globo-apps.git:
    Cloning into bare repository '/Users/runner/Library/Developer/Xcode/DerivedData/app-globo-mais-one-ios-fahhdoxaguehuoejpaevgcrhkaty/SourcePackages/repositories/ui-globo-apps-a8107c52'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
    Please make sure you have the correct access rights
    and the repository exists.

Skipping cache due to an error: Failed to clone repository git@github.com:Infoglobo/ui-globo-apps.git:
    Cloning into bare repository '/Users/runner/Library/Caches/org.swift.swiftpm/repositories/ui-globo-apps-a8107c52'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
    Please make sure you have the correct access rights
    and the repository exists.

How can I add a publickey to use it in appcenter?
Regards


Extra position in undo list after all? 'shifted' characters.

Software Versions

Python: 3.7.3
OS: Linux Fedora 30
Kivy: 2.1.0
Kivy installation method: rpm

Describe the bug
In TextInput after write letters using shift I get extra position in undo list with empty ('delsel', cIdx, '')
cIdx - is cursor index that same as letter write with shift. So after is ('insert', cIdx, cIdx+1)
''       - empty string
The effect: ctrl+z have to be press twice. Second just move cursor.
Expected behavior
Only one position in undo list like other letters.
To Reproduce
from kivy.app import runTouchApp
from kivy.uix.textinput import TextInput
koTxt=TextInput(text = 'sdfdsf dfg ds fdsf gffg fdg fgf dgdsfds\nasfds fdgf gfd gfd af sfds',size_hint=(1,1))
runTouchApp(koTxt)
quit()
Code and Logs and screenshots
{'undo_command': ('delsel', 13, ''), 'redo_command': (13, 13)}
{'undo_command': ('insert', 13, 14), 'redo_command': (13, '*')}

Can I take up this issue?

Hi.
Are you asking me?
Sorry I am new on Git Hub. Do whatever is necessary.
Radek


Uncaught Error: ENOENT: no such file or directory, lstat '~/AppData\Local\atom\app-1.33.1\CSS - M...

[Enter steps to reproduce:]

...
...

Atom: 1.33.1 x64
Electron: 2.0.16
OS: Microsoft Windows 10 Home Single Language
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT: no such file or directory, lstat 'C:\Users\Shantanu\AppData\Local\atom\app-1.33.1\CSS - My Site Images'
At fs.js:1661

Error: ENOENT: no such file or directory, lstat 'C:\Users\Shantanu\AppData\Local\atom\app-1.33.1\CSS - My Site Images'
    at Proxy.realpathSync (fs.js:1661:15)
    at Proxy.fs.realpathSync (ELECTRON_ASAR.js:336:29)
    at TreeView.t.exports.TreeView.moveEntry (~/AppData/Local/atom/app-1.33.1/resources/app/static/<embedded>:14:916493)
    at TreeView.t.exports.TreeView.onDrop (~/AppData/Local/atom/app-1.33.1/resources/app/static/<embedded>:14:922377)
    at HTMLDivElement.<anonymous> (~/AppData/Local/atom/app-1.33.1/resources/app/static/<embedded>:14:896158)

Commands
     -5:08.9.0 editor:copy-path (input.hidden-input)
  2x -2:33.7.0 tree-view:add-folder (span.name.icon.icon-file-directory)
     -2:05.5.0 core:confirm (input.hidden-input)
     -1:57.7.0 tree-view:collapse-directory (div.tool-panel.tree-view)

Non-Core Packages
atom-beautify 0.33.4

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/tree-view/issues/1282 so you can subscribe there for updates if you'd like.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Add layers, history, drawing of components, anchors…

This is a PR which we hope to merge soon, which represents the next major version of MFEKglif.
All development is happening right now on this layers branch. Users are free to test it, but note that the following still needs to be added to merge it to master:

add back saving and exporting
add layer renaming
QOL changes to layers dialog

@MatthewBlanchard would know if I missed anything from there.

@alerque FYI I merged b1e74e9 here instead of master as it's our current development branch.


Router. get_bgp_neighbors_detail() should cache results

Environment

Python version: 3.8
Peering Manager version:  1.3.2

Steps to Reproduce

Create multiple BGP Groups and/or multiple IX's on the same router
Poll the router using the poll_peering_sessions management script

Expected Behavior
I expect the code should only poll the router once and re-use the data for a short period (1-2 minutes?)

Observed Behavior
The script will request the all BGP details from the router for each BGP Group or IX associated with it.

1-2 minutes is probably not enough for a large number of devices. We have nearly 100 devices in Peering Manager, and it is taking just over 10 minutes just to poll all the routers once. So the cached data needs to be around at least that long to ensure it gets reused for the entire polling session. Some of this could be mitigated by parallelising the scan of the routers, but given the structure of the polling algorithm, this doesn't map nicely to a router centric caching strategy.
I had set the caching at 10 minutes and it started re-polling a bunch of routers. It was still an improvement over the 60 minutes or so that it was taking before. This number needs to be tunable per site as some people may want to poll their routers more frequently than 10-15 minutes and this will mask a bunch of the updates

PR #472 has a 15 minute default cache time, which is perhaps a bit long. So need to decide on a reasonable value. Alternatively, the cache time could be dynamically scaled based on the number of routers configured. Based on my tests, we need at least 8 seconds per router to ensure we make it through the poll before the cache expires. So using 10 seconds per router would work. However, this could be tuned down if polling can be made more parallel. I assume the caching is done in redis, so will the cache be shared across processes?


a question about resnet-18/12  kernel size

Hi  that's really a nice code!  However, i have a question about resnet.  In the resnet paper, resnet's first conv layer's kernel size is 7. But  in your code , I found the first conv kernel size is 3. I also found other codes which use resnet-12 or 18. They also set the kernel size is 3.
What's more, I read the TADAM's paper which design the resnet-12(maybe not). It just use blocks, didn't add conv layer before the blocks.
Could you tell me why you change conv kernel size ?  Thank you !

I print your resnet-18's  output size. Could you tell me why you didn't set the 128 channel?

We follow the ResNet-12 of (MetaOptNet)[https://github.com/kjunelee/MetaOptNet/blob/master/models/ResNet12_embedding.py] and ResNet-18 of SimpleShot (results with ResNet-18 are not reported in the paper).
The main reason is that the images used in FSL experiments are small images (84x84), so we should remove some downsampling operators.


use the term union instead of merge

Coverage remained the same at 95.544% when pulling edc5dbe60944e3f441aa31f57bd65033a0abd46f on rename-merge-to-union into f445dc9d00539f5a2407e7823e2829fa18bbc576 on master.


Add friction + coulomb + threshold parameters

What's new:

As per https://github.com/robotology/icub-firmware/issues/396 removes the old kbemf torque parameter in favor of the new torque control parameters:

viscousPos
viscousNeg
coulombPos
coulombNeg
verlocityThres

The new friction compensation law is the following one (see also the source code):
float PID_do_friction_comp(PID *o, float32_t vel_fbk, float32_t trq_ref)
{
    const float32_t MICRO { 1000000.0 }; // conversion from Nm into microNm
    const float32_t coulomb_pos_converted = o->coulomb_pos_val * MICRO;
    const float32_t coulomb_neg_converted = o->coulomb_neg_val * MICRO;

    const float32_t viscous_neg = o->viscous_neg_val * vel_fbk;
    const float32_t viscous_pos = o->viscous_pos_val * vel_fbk;

    if (vel_fbk <= -o->velocityThres_val)
    {
        return o->Ktau*(-coulomb_neg_converted + viscous_neg + o->Kff*trq_ref);
    }
    else if (vel_fbk > o->velocityThres_val)
    {
        return o->Ktau*(coulomb_pos_converted + viscous_pos + o->Kff*trq_ref);
    }
    else
    {
        return o->Ktau*(o->Kff*trq_ref);
    }
}

Note:

The following robots will be affected by this changes:

#
robot

1
ergoCubSN000

2
ergoCubSN001

3
iCubEdinburg01

4
iCubErzelli01

5
iCubErzelli02

6
iCubGenova02

7
iCubGenova07

8
iCubGenova08

9
iCubGenova10

10
iCubGenova11

11
iCubLisboa01

12
iCubNancy01

13
iCubPargue01

14
iCubShanghai01

15
iCubShenzhen01

16
iCubSingapore01

17
iCubTemplates

18
iCubValparaiso01

19
iCubWaterloo01

20
iCubZagreb01

cc @isorrentino @marcoaccame @Nicogene

Hi @sgiraz
To guarantee back-compatibility, we need to make sure that the values of viscousPos and viscousNeg must equate the values of kbemf1. This is true for most of the robots but NOT for those where kbemef <> 0.
Thus, there are robots requiring a fix.
Footnotes

See https://github.com/robotology/icub-firmware/pull/410/files. ↩

Hi @pattacini,
Changes applied! Thanks for your feedback.

/remind December 18, merge the PR.

The modifications seem okay. Obviously, the parameters provided are not the ones that make the joints work in torque, but with those values we are confident that nothing will happen if the torque control is activated.


Release google-auth-library-java v0.16.1

This pull request was generated using releasetool.
06-06-2019 13:10 PDT
Dependencies

Update dependency com.google.http-client:google-http-client to v1.30.1 (#265)

Internal / Testing Changes

Add credentials for autorelease (#267)
Add developers section to google-auth-library-bom pom.xml
Bump next snapshot (#264)

Release is at https://github.com/googleapis/google-auth-library-java/releases/tag/v0.16.1


Integrate datable to Kedro extras datasets

Description
Add datatable to Kedro's extras datasets
Context
datatable is derived from R's data.table. As of 20/09/2020 (Version 0.11.0), datatable is fully supported on Windows, Linux and MacOS and can be installed through pip. Offering datable in Kedro's extras dataset could offer many advantages in building pipelines:

It would make the transition for R-users wanting to use Kedro a lot easier.
Datable supports CSV, XLSX, and plain text as well as .tar, .gz, .zip, .gz2, and .tgz.
Certain performance advantages over pandas (especially for datasets larger than 50GB): benchmark
fread is really fast at reading large CSVs. It also automatically detects separators.
datatable's df.to_csv has an append parameter which appends the dataframe with column names if the file does not exist yet. If the file already exists, the column names are not added. Pandas to_csv offers "mode='a'" and "header=True" parameters which means choosing between rewriting the column names with each append or not having any column names. This is really practical for pipelines where you add new data from an IncrementalDatasets folder to a concatenated CSV.

Possible Implementation

Using the code from pandas.CSVDataSet, we could change the "pd.read_csv(fs_file, **self._load_args)" to "dt.fread(fs_file, **self._load_args)". Saving the file should be similar code albeit different parameter names.
For ExcelDataSets, we could also use code from pandas.ExcelDataSet. Datatable also uses the xlrd engine for xlsx. Replace "pd.read_excel(fs_file, **self._load_args)" with "dt.fread(fs_file, **self._load_args)". Datatable does not output xlsx. So either we do not implement ExcelDataSets for datatable or we convert the datatable to pd.DataFrame and write to excel using pd.DataFrame.to_excel.

@lucasjamar Thanks for the suggestion. I've logged the request for it internally, however no timeline/priority can be provided at this time. If this is something of high importance for you and you have an implementation in mind, please feel free to go ahead and contribute this feature to Kedro 🙂

I'm interested to try to implement it

@DmitriiDeriabinQB  thanks and sorry for not getting back earlier. I have already developed a version of the CSV dataset and Excel dataset that I use in my current kedro project as an extras dataset. I haven't had more time to spend to try implementing it because of some issues . @mlisovyi , would you be interested in my current code for these two datasets or would you like to start from scratch? this zip is the folder that I currently place in the folder src//extras/dataset. The code is 95% derived from the pandas equivalents. The current implementation of CSVdataset works fine for me but I still havent used the Exceldataset.
datatable.zip

Hi @lucasjamar . Thanks for the code! What were the challenges that you have encountered? Was there anything that would prevent you from pushing it "as is"?

Hi,
CSVdataset appears to work fine and I think it could go to straight to testing (let me know if you spot any issues). I just havent been able to set up the kedro environment on my Windows. I haven't been able to make Exceldataset work however (both xlsx and xls). I think its an issue with the engine but I havent found any excel engine options in fread:

https://datatable.readthedocs.io/en/latest/api/dt/fread.html
https://github.com/h2oai/datatable/blob/140a3abdae94e77badf608936b7bdbd5b091d6e5/src/core/read/py_fread.cc#L239-L286

@lucasjamar  Thanks for feedback. Yes, i had the same problem. At the end i decided to narrow down to CSV only. What you think?
In the meantime I've been thinking if one in general want to have the actual reader using datatable. If the purpose is to benefit from faster reading, then it's unavoidable. If the purpose is to make use of the R-user-friendly API for tables, then maybe it makes more sense to have wrappers around pandas dataset classes to transform pd.DataFrame into dataframe.Frame (if it is possible to do it efficiently) before/after writing/reading.

@mlisovyi . Yep I think its best to stick to just CSV in a first part. The main advantage in my opinion with datatable is that it is really fast at reading CSVs and it has the append parameter for writing. This is quite practical for data collection and concatenation tasks. Coming from R, I do not believe that someone familiar with data.table will have any more trouble with the pandas syntax vs. the datatable syntax. Plus, datatable is still in beta phase. In my opinion, it lacks a few key features to be a true equivalent to pandas. Most notably, it does not have a date-time type yet

Hi @lucasjamar, would you be interested in implementing this? We're looking at cleaning up open issues. The team currently doesn't have capacity to take this on, so we'll be deleting this in 2 weeks if there's no further interest. Thanks!

Hi @MerelTheisenQB , I am sorry but I already have other PR I need to close. Plus, this issue seems to be quite a difficult one. I would prefer tackling small issues first to get accustomed with contributing to Kedro before taking on a such a big one. Sorry. Thanks for everyone's hard effort nonetheless!

Hi @lucasjamar, no worries at all! I'll close this issue for now. But anyone who wants to take this up can re-open it again.


Adjusting GestureDetector hitbox

Hello,
I tried to use Transform to adjust GestureDetector's hitbox. I even tried to use two transforms, one with transformHitTests = true and one with transformHitTest = false in an attempt to move the widget and transforms down, then move the widget only up. That failed.
So I haven't found a good way to adjust a hitbox, something that is very easy in react-native using the hitSlop property.
Any insights?

follow-up to #14794

Wow, I opened up two of these? I must have been really tired last night. Sorry about that.

@lukepighetti thanks for the pointer
closing as dup of #18554


Allow instance updates with values

repo.exec(user.update(updated_at: Time.now))

Doesn't work now, as Model#update requires Changeset argument.

I think that it would be better to add argless update and delete shortcuts:
user.update == User.where(id: user.id!).update

The first example would be changed to:
repo.exec(user.update.set(updated_at: Time.now))


Upgrade the React Native generator to use React Native Web

Description
I would like to upgrade the React Native generator to use Next + Expo or Next + React Native Web.
The we could

use Next also for the native app generator
share most of the code between the web and the native Next generators, as we've done in the past for the react and the react native generator
remove the React generators that aren't using Next

Personally, I think it would be beneficial to keep the React generator.
Not everyone is using or want to use Next.js. It uses React Router, which is the foundation of Remix. Eventually it could share code with a Remix generator.


Fails to build under TypeScript due to missing async-lock definition

Describe the bug
When building a TypeScript project that uses @azure/amqp-common (in my case a dependency of @azure-event-hubs), I get the following error:
node_modules/@azure/amqp-common/typings/lib/util/utils.d.ts:1:23 - error TS7016: Could not find a declaration file for module 'async-lock'. '/mnt/c/Users/nebri/Projects/aquarium-control/server/node_modules/async-lock/index.js' implicitly has an 'any' type.
  Try `npm install @types/async-lock` if it exists or add a new declaration (.d.ts) file containing `declare module 'async-lock';`

1 import AsyncLock from "async-lock";

When I install the @types/async-lock module (which should not be necessary as an end-user), I then get the following error:
node_modules/@azure/amqp-common/typings/lib/util/utils.d.ts:1:8 - error TS1192: Module '"/mnt/c/Users/nebri/Projects/aquarium-control/server/node_modules/@types/async-lock/index"' has no default export.

1 import AsyncLock from "async-lock";

To Reproduce
Install "@azure/event-hubs": "^2.0.0" and "typescript": "^3.4.5". Use this tsconfig:
{
  "include": [
    "src/**/*.ts"
  ],
  "compilerOptions": {
    /* Basic Options */
    "target": "esnext",                          /* Specify ECMAScript target version: 'ES3' (default), 'ES5', 'ES2015', 'ES2016', 'ES2017', or 'ESNEXT'. */
    "module": "commonjs",                     /* Specify module code generation: 'none', commonjs', 'amd', 'system', 'umd', 'es2015', or 'ESNext'. */
    "sourceMap": true,                        /* Generates corresponding '.map' file. */
    "declaration": true,
    "declarationDir": "declarations",
    "outDir": "./dist",
    "rootDir": "./src",

    /* Strict Type-Checking Options */
    "strict": true,                           /* Enable all strict type-checking options. */
    "noImplicitAny": true,                    /* Raise error on expressions and declarations with an implied 'any' type. */
    "strictNullChecks": true,                 /* Enable strict null checks. */
    "noImplicitThis": true,                   /* Raise error on 'this' expressions with an implied 'any' type. */
    "alwaysStrict": true,                     /* Parse in strict mode and emit "use strict" for each source file. */

    /* Additional Checks */
    "noUnusedLocals": true,
    "noImplicitReturns": true
  }
}

Expected behavior
This should compile without errors
Package-version<EMAIL_ADDRESS>(why is the production version of<EMAIL_ADDRESS>installing a preview version of amqp-common?)
node.js version: 10.15.3
OS name and version: Windows 10 64-bit, via WSL with Ubuntu
Additional context
Add any other context about the problem here.

Two issues here:
The former error is because @types/async-lock is a dev dep but should be a normal dep. This needs to be fixed.
The latter error is likely due to your config lacking the TS esModuleInterop or allowSyntheticDefaultImports setting. This needs to be documented ala ServiceBus.

Thanks for the info @bterlson
FYI I cannot use the esModuleInterop flag as it breaks other packages I'm using. If it's broken for me, we probably shouldn't be recommending it to our users. That said, allowSyntheticDefaultImports worked, so perhaps an explainer of the dangers of using esModuleInterop?
Related: I'm still curious about why a preview version of amqp-common is installed with the production version of event-hubs. It may not be a big deal, but without context into the specifics it sounds disconcerting. Any insight?

Since the repository is now moved to https://github.com/Azure/azure-sdk-for-js/tree/master/sdk/core/amqp
This issue is now tracked at https://github.com/Azure/azure-sdk-for-js/issues/2706 instead.


Transfer progress

We need to check this integration test though: SitlTest.MissionDownloadCancellation

Works locally, going to merge it for now.


Implement shell with autocomplete feature

Try to implement a shell with tab auto completion feature.

I'll take a swing at this.


feat(combobox): add clearSearchTermOnSelect param to ComboboxRoot

I added the clearSearchTermOnSelect prop so that the search term will not automatically clear when Combobox selects triggers. Maybe users will want to select multiple values based on the same search fields
Original Issue: https://github.com/radix-vue/shadcn-vue/issues/709.
Video example:
https://github.com/user-attachments/assets/535318bd-c907-4949-a8d4-bab39460fe98
Docs updated too:

I managed to rename clearSearchTermOnSelect to resetSearchTermOnSelect as we already have resetSearchTermOnBlur which will be more intuitive 🙂
commit: c2733af (#1257)

Cool addition @romanhrynevych . Can you check the failing test? 😁

@hrynevychroman Great addition, will use this. Could you please finalise this PR by fixing the failing test? 💯


32565 Fix response validation error for reverse search address

Description
For some coordinates, the server returns an array of strings, while for others, it returns an empty array for the route_numbers field. According to the documentation, the field should contain an array of strings, not an array of integers:
"routeNumbers": {           "description": "The codes used to unambiguously identify the street",           "type": "array",           "items": {             "type": "string",             "readOnly": true           },           "readOnly": true         },
This has been tested for multiple coordinates, including all those listed in the issue.
Resolves: https://github.com/Azure/azure-sdk-for-python/issues/32565
Note: _models_py3.py is no longer generated using autorest, but it's still used, therefore, all changes should be applied manually.

As discussed in https://github.com/Azure/azure-sdk-for-python/pull/34987, this change needs to be updated in the spec since we'll lose this fix upon next generation.

As discussed in #34987, this change needs to be updated in the spec since we'll lose this fix upon next generation.

@kristapratico I will put everything in this comment:
Spec is already updated: https://github.com/Azure/azure-rest-api-specs/pull/17151
Upon trying to regenerate new code described in this draft pr: https://github.com/Azure/azure-sdk-for-python/pull/35029 I managed to resolve the issue, but there is a lot of work to be done before fixing tests/everything..

/check-enforcer override


Add 4XX errors body to exception

Usually errors 4XX have useful explanation about an occurred error, but the library does parse and return error body only if error code is < 400.
See ApiResource line 194.
I think it's better to do something like
  // Handle 4XX responses, 5XX responses will throw an HttpAdapterException
if ($statusCode < 500) {
    $body = json_decode($response->getBody()->getContents(), true);
    if ($statusCode < 400) {
        return $body;
    } elseif ($statusCode === 404) {
      throw new APIResponseException('The specified resource does not exist', 404);
    } else {
        throw new APIResponseException('Received bad response from ' . ucfirst($this->endpoint) . ' API: '. $statusCode.', body:' . print_r($body));
    }
}

I'm ready to prepare a PR.

+1
Getting back a message like this:
Received bad response from Transmissions API: 422
Is pretty useless as the status code covers many errors as illustrated here:
https://support.sparkpost.com/customer/en/portal/articles/2140916-extended-error-codes
Sparkpost's actual response body is really useful for debugging
{ "errors": [ { "message": "required field is missing", "description": "At least one of 'text' or 'html' needs to exist in 'content'", "code": "1400" } ] }

+1

@zaporylie Nice!!!

@TresErres You're welcome


MatchConditions headers are improperly applied.

Library name
Azure.Data.Appconfiguration
Describe the bug
The MatchConditions headers should be applied by calling this method. As it currently is, when the headers are applied (e.g. here), the quotes are always omitted from the ETag value. However, when options.IfMatch.Value does not equal ETag.All, the ETag values must be stored with the quotes included. Without the quotes, ETag values cannot be compared.
The client is generated using the following swagger:
Input File :  -  https://github.com/Azure/azure-rest-api-specs/blob/e01d8afe9be7633ed36db014af16d47fec01f737/specification/appconfiguration/data-plane/Microsoft.AppConfiguration/stable/1.0/appconfiguration.json
Expected behavior
ETag values are stored with quotes included.

Actual behavior
ETag values are stored without quotes included.

According to the spec, If-Match should be like:
If-Match = "If-Match" ":" ( "*" | 1#entity-tag )

And the Entity-Tag should be like:
entity-tag = [ weak ] opaque-tag
weak       = "W/"
opaque-tag = quoted-string

Note the quoted-string.
RFC7232 is about the HTTP conditional requests. In the spec, string values (excluding *) of If-Match should be quoted:
Examples:

     If-Match: "xyzzy"
     If-Match: "xyzzy", "r2d2xxxx", "c3piozzzz"
     If-Match: *

Since we already have all the conditions applied in ConditionalRequestOptionsExtensions.ApplyHeaders method, we should consider calling it for applying MatchConditions headers.

Thanks @JoshLove-msft for pointing out that we can use ETag.ToString("H") which provides better support for this case. I'm going to split the work into two parts:

Fix the serialization of ETag in https://github.com/Azure/autorest.csharp/blob/ee854e3cc58d2ba63d8e905b8b1ccd515bd9d0f1/src/assets/Generator.Shared/RequestHeaderExtensions.cs#L67-L95 As you can see, ETag value are not correctly serialized in those API. We have to fix them anyway.
In generator, update the logic to generate codes to set the ETag header values:

From request.Headers.Add("If-Match", ifMatch.Value.ToString());
To request.Headers.Add("If-Match", ifMatch.Value); which will invoke the overload version for ETag: https://github.com/Azure/autorest.csharp/blob/ee854e3cc58d2ba63d8e905b8b1ccd515bd9d0f1/src/assets/Generator.Shared/RequestHeaderExtensions.cs#L67

I filed https://github.com/Azure/azure-sdk-for-net/issues/30441 to track removing the shared source method that should not be used.

I made a mistake. We don't need to fix generator. The generated codes invoke the overloaded method:
request.Headers.Add("If-Match", ifMatch.Value);


CI: Use 2020 resolver for pip to ensure correct resolution

In PyPI checks, we see:
ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts.

We recommend you use --use-feature=2020-resolver to test your packages with the new resolver before it becomes the default.

This PR adds the suggested flag to ensure we won't get caught short in October.

@oesteban I'm not sure I understand the current

sdcflows<1.5,>=1.3.2
nitransforms<20.2,>=20.0.0rc3

Neither sdcflows 1.4 nor nitransforms 20.1 exist, so I'm not sure why we're preemptively supporting them.

Neither sdcflows 1.4 nor nitransforms 20.1 exist, so I'm not sure why we're preemptively supporting them.

For nitransforms it is just flexibility. We don't know yet what release plan (if there is one) there is for nitransforms, and its API should not really change.
For SDCFlows I just checked https://www.nipreps.org/devs/versions/ and it seems we did not contemplate a minor version bump with 20.2.x so this is my mistake. We should probably set sdcflows ~= 1.3.2 for the next release.


Optimize is no longer recommended

Since Solr 3.6, Solr has used the TieredMergePolicy which makes,
in most scenarios, optimization a harmful rather than beneficial
step.
We should leave the feature in the client, but not publicize it, IMO.

Thanks!


Pywren Runtime buckets not public

It seems that the new pywren-runtime buckets are not publically accessible:
aws s3 ls pywren-runtimes-public-us-west-1
An error occurred (AccessDenied) when calling the ListObjectsV2 operation: Access Denied

I have tried a few different regions / buckets but get the same reults.

Whoops we didn't enable the list permission (read access was allowed however)

Fixing now.

Thanks looks all good now :)


Extra Checkable background on press for API <21

This can be fixed by setting the backgrounds of the radio buttons and check boxes to @null on API 21+. I'll fix it in a bit.

I have already fixed it. Making pull request just now


alex.Condescending rule may need an exception

Check for existing issues

[X] Completed

Describe the feature
This could be considered a bug, or a feature request, depending on how you look at it.
Here's the scenario:
The alex.Condescending rule matches some tokens and flags their usage as an error without considering the context.
For example, given this input file:
A less laborious method is possible when the value we want to compare is not easily obtainable within the script,
but is either the result of a calculation or a value retrieved from an internal variable, a database operation, or a REST API call.

With this input, vale states that easily usage is condescending. I believe that this usage (with a negation) should not considered as such, as it is describing a condition, not suggesting that the reader can do something without effort.
If the above is a bug, is there a way of fixing it, other than replacing easily with readily as I did?
If it is not a bug, is there a way of modifying the alex.Condescending rule to ignore the token when it is negated?
OS: MacOS
Install method: Homebrew
Vale version: 3.8.0

My stance on the implementation of third-party styles is to match the intent of the specific tool or style guide. In this case, alex reports the same issue, so I consider this to be "working" as intended.
If you open an issue upstream that results in a change, I'll try to match the new behavior here.


Order column position and save settings into DB

Hello,
I love bootgrid. It is very common and understable and useful jquery plugin.
But.
For me and my usage I need to know If it is possible to change the order column position for change by user with drag and drop style.
The second question is If it is possible to save the select columns, select positions of the columns into Database table as user settings under the username for usage when the user is logged in.
Please, If it is possible, write me, where I can do it and how.
Thanks a lot
Honza H

No, it's not possible
also not possible


Download link in email

Seems like a href="data:text/calendar;charset=utf8....." download can not be used in the email, its there a way to use add to outlook/ican calendar in email?

@boqian999 not so sure I understand it correctly, cloud you please clarify more about it?
What does it mean "download can not be used in the email"?


Fpad 1769 - RPT - Create CloudFormation Template / ApiGateway for Generator lambda

Tickets solved
What tickets will be solved (marked as done) once this PR is merged?

FPAD-1769

Tickets created
What tickets have been created during this PR? Any bugs found? Out-of-scope features?
N/A
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. This is simply a reminder of what we are going to look for before merging your code.

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] I have tested my changes on AWS

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, links to any design documents, etc...

Reviewed my Iimaan for education / mentoring purposes.


Ontology curation error fixes

List of ontology curation errors that I discovered when preparing zooma data source as well as those identified by other components
Corrections based on https://github.com/HumanCellAtlas/dcp2/issues/13 https://docs.google.com/spreadsheets/d/1Wk7SGxEz00AkNokYv3YlFHJVF9U6THKcrvgHARsnau8/edit#gid=0

All projects with library_preparation.method.ontology set as EFO:0009310 and

[ ] library_preparation.end_bias 3 prime tag OR 3 prime bias should have the ontology updated to EFO:0009899 and ontology_label updated to label for that term
[ ] library_preparation.end_bias 5 prime tag or 5 prime bias should have ontology updated to EFO:0009900

[ ] All projects with library_preparation.method set as a 10x method should have 3 prime tag (not 3 prime end bias) set in the library_preparation.end_bias field

Project specific corrections:
Requires bulk update
Single cell RNAseq characterisation of cell types produced over time in an in vitro model of human inhibitory interneuron differentiation - 2043c65a-1cf8-4828-a656-9e247d4e64f1

[ ] All cell_suspension.timecourse.unit.ontology fields should be changed to UO:0000033, the correct ontology for 'day' where cell_suspension.timecourse.unit.text is 'day'
[ ] The model organ for the cell line 557606a0-96dd-40cc-bb72-94d5711bb8ec should be changed to UBERON:0000922 (I couldn't find this term in the ui, not sure why

BM_PC - a29952d9-925e-40f4-8a1c-274f118f1f51:

[ ] All donor_organism.development_stage.ontology should be changed to the correct term for human adult - HsapDv:0000087
[ ] All specimen_from_organism.organ ontologies should be changed to the correct term for bone UBERON:0002481 and ontology label bone tissue

Single cell transcriptome analysis of human pancreas - cddab57b-6868-4be4-806f-395ed9dd635a

[ ] all donor_organism.development_stage.ontology should be changed to the correct term for human adult - HsapDv:0000087

GSE67833_neural_stem_cells - e8808cc8-4ca0-4096-80f2-bba73600cba6

[ ] current curations for organ in specimen_from_organism entities should be in 'organ_part'
[ ] organ should be set to 'brain' - UBERON:0000955

Non-specific 'cortex' term (UBERON:0001851) in specimen_from_organism.organ_parts should be changed to organ specific cortex term, either kidney or brain, for the following projects:

[ ] scRNAseqSystemicComparison - cerebral cortex UBERON:0000956
[ ] Diabetic Nephropathy snRNA-seq - cortex of kidney UBERON:0001225
[ ] AllelicExpressionPatterns - cerebral cortex UBERON:0000956
[ ] 1M Neurons - cerebral cortex UBERON:0000956
[ ] HumanBrainSubstantiaNigra - cerebral cortex UBERON:0000956

Easy to do manual fixes:
Pitx2DevelopingHeart - 7027adc6-c9c9-46f3-84ee-9badc3a4f53b

[ ] 45b47d43-ab70-48d9-a205-4b0509bf7114 - donor_organism.organism_age_unit.ontology should be UO:0000033
[ ] There is onedonor_organism.genus_species set as Mus musculus castaneus , which I think should just be Mus musculus
[ ] donor_organism.organism_age_unit text says 'day' but ontology is 'week'

HumanIsletType2Diabetes - 7adede6a-0ab7-45e6-9b67-ffe7466bec1f

[ ] donor_organism.ethnicity where text is Hispanic should be changed to HANCESTRO:0014

Things to investigate further

[ ] Was it intentional to use 'liver primordium'  rather than 'liver' for 'AllelicExpressionPatterns' liver specimens? - curated by Rachel Schwartz

EpithelialDiversityHealthInflammation - c893cb57-5c9f-4f26-9312-21b85be84313

[ ] There is a mismatch between the 'text' field and 'ontology' field for some specimens where a non-inflamed sample was taken from a patient with uclerative colitis, i.e. the text says 'ulcerative colitis' but the ontology says 'normal' which I think is confusing but not sure which part needs to be corrected

HumanFirstTrimesterPlacentaDecidua - 1cd1f41f-f81a-486b-a05b-66ec60f81dcf

[ ] specimens with text 'decidua' and 'chorion membrane' have been curated as placenta

HeartSingleCellsAndNucleiSeq - ad98d3cd-26fb-4ee3-99c9-8a2ab085e737

[ ] donor_organism.disease text given as diabetes but ontology curated as hypertension (MONDO:0005015), probably a mistake in ordering on an array field

DevelopingMouseKidneyHumanKidneyOrganoids - 7b947aa2-43a7-4082-afff-222a3e3a4635

[ ] donor_organism.gestational_age_unit.text given as '12', should be 'week' or similar

General thoughts

[ ] We have two terms that we currently use to indicate 'skin' as an organ, 'zone of skin' and 'skin of body' we should probably pick one and try to be consistent

Just a thought, that maybe this ticket needs splitting in separate tickets.


Fix kubectl top for memory-demo pod

Fixed command for accessing metrics(top) for memory-demo pod.

I have signed the CLA. Thanks

/assign @zparnold

@nihanthd Thanks for the update. Could you please sign the CLA following the instructions listed at  https://github.com/kubernetes/community/blob/master/CLA.md

If you're having trouble signing the CLA, or you have signed it and it's not
working correctly, see https://github.com/cncf/servicedesk for more info. You
can also contact #sig-contribex on Slack. We cannot accept this contribution until the checks are working
correctly. Thanks!

@nihanthd Just a friendly reminder to sign the CLA when you are able.

@Rajakavitha1 @zparnold Completed signing the CLA.

/approve
/lgtm


Remove all uses of JS classes from OCaml

Depends on all "Move _ to JS issues"

closed by https://github.com/o1-labs/snarkyjs/pull/979


Unrequire passing tests

This does some whitespace cleanup, but more importantly, removes @requires from tests that pass now.

+1

Thanks for the review. Merging.


Port generalization changes from omeshp/itemleveldownloader repo so a…

…s to support upload as well

@bishal-pdMSFT,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


chore(release): v0.18.8

Version bump in package.json for release v0.18.8

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Relax][ONNX] Add Multiple ONNX Frontend Support for Clip / Equal / Shape / Not / Tanh

@zxybazh This looks good Xiyou, can you just rebase on top of TUZ-6?

Sure, doing rebase now.


How to work with Postgresql?

Hi I have a question like I can work with postgresql, I do not know the variables of the coloumnas, I know that I must replace the following commands:
<Entry key = 'database.driver'> org.postgresql.Driver </ entry>
<Entry key = 'database.url'> jdbc: postgresql: //<IP_ADDRESS>: 5432 / [DATABASE] </ entry>
<Entry key = 'database.user'> [USER] </ entry>
<Entry key = 'database.password'> [PASSWORD] </ entry>

thanks friends.

Sorry columns

Then the question doesn't make sense to me. What columns are you talking about?

Forgive, what kind of records or what should be captured in the postgresql table?

Which table?

Traccar will create multiple tables, not just one.

Then traccar automatically creates the tables

Yes, it does.

Thank you very much for your attention and very good contribution, good day, bye.


BreastSpecimenTypeVS SCT descriptions need to be revised

I get the intent of including a mapping to ICD-O-3 codes, but they're not part of the original SCT descriptions. There's a comment about including synonyms; maybe we include both SCT and ICD-O-3 codes in the VS?
What was the intent of moving to SCT here? The issue this change ties to was simply a question on why the codes were local, which stems for a terminology server issue HL7 needs to resolve. Moving to SCT does remove the need for a local code system, but ICD-O-3 is a pretty significant terminology in the cancer space. Do we want to ignore it altogether?

Despite @markkramerus preference to keep the ICD-O-3 corresponding, after discussing with @mlterryMitre we agreed to remove ICD-O-3 references, since the rationale to replace ICD-O-3 codes with SNOMED CT codes was to use a non-licensed terminology.
We might consider providing well-vetted mappings in the future.
This applies to BreastCancerSite as well.


fix layout

Rough

[x] header
[x] footer
[x] support gratipay notice
[x] content pages like about pages
[x] two-column things like charts and FAQ
[x] profiles
[x] community pages
[x] all communities listing /for/

Finish

[x] penny background
[x] signing out fails? #2940
[x] clean up logo + stats in header
[x] figure out column story
[x] Your Weekly Gift
[x] sub nav
[x] root out slug == none
[x] clean up sign-in/you-are area

Finish Finish

[x] min height (no empty space below footer)
[x] refactor SCSS
[x] harmonize markup/scss between homepage and inside pages
[x] clean up error pages
[x] clean up Giving
[x] clean up Receiving
[x] clean up History
[x] clean up Account
[x] clean up Events
[x] clean up elsewhere profiles
[x] clean up elsewhere teams
[x] fix that gosh darn "Sign Out" button in the #header
[x] notifications
[x] "upgrade browser" notice
[x] cross-browser review

Finish Finish Finish

[x] i18n review
[x] credit card & bank account forms
[x] make giving widget in sidebar AWESOME okay
[x] join/leave sidebar on community pages
[x] responsiveness
[ ] cross-browser review

Punchlist

[ ] "return" doesn't work from inside the your-tip <input>
[ ] padding is off in your-tip <input>, if I select with the mouse and go down then the text shifts

All of this looks great. What's left here?

I think we're waiting for @whit537 to realize that this is good enough to be merged.

I'll get there, I'll get there ...

[ ] fix subtle styling regressions on homepage

Old:

New:

[ ] remove space between homepage and about links in top nav at low res (the reason for the space at higher res is alignment with the main copy)

[ ] "About" content should start above the fold on mobile. This means reworking the nav somehow.

[ ] Explore is crowded on mobile

There's a few other things I want to flag, I'll be back ...

Is this explained by different Python minor versions?

Yes. I upgraded to 2.7.9 locally and now I have p.__dict__. We're using 2.7.3 in production atm so I guess we should upgrade before landing this PR.

@clone1018 points out that only 5% of our users are on mobile, but they're our most important users, people like @weallhave24hrs and @renaantosz. We've got a sampling bias, in other words. We need to look great on mobile.

http://bradfrost.com/blog/post/atomic-web-design/
http://alistapart.com/article/responsive-comping-obtaining-signoff-with-mockups

So far Atomic Web Design is what I resonate most with for organizing a design system. I just don't get the SMACSS categorization. But anything is better than what we have now.

[ ] invert media queries (mobile first, special-case the wider layouts)

[ ] reorganize scss

@whit537 This PR is already too big, please stop holding it over details, merge it and make another PR to reorganize the SCSS.

This PR is already too big, please don't hold it over details, merge it and make another PR to reorganize the SCSS.

Yeah, sorry. If this PR goes out as it is now, I'm going to be really embarrassed about how bad it is on mobile (I don't see the problems as just "details"), and I feel like we're reaching a point where making it not suck on mobile requires some refactoring. :-/

Over the 4 years we have slowly moved away from device specific breakpoints in favour of content specific breakpoints, i.e. adding a breakpoint when the content is no longer easy to consume.

http://responsivedesign.is/articles/why-you-dont-need-device-specific-breakpoints

Pick major breakpoints by starting small, then working up
Pick minor breakpoints when necessary
Optimize text for reading
Never completely hide content

https://developers.google.com/web/fundamentals/layouts/rwd-fundamentals/how-to-choose-breakpoints

Yeah, sorry. If this PR goes out as it is now, I'm going to be really embarrassed about how bad it is on mobile (I don't see the problems as just "details")

A few pixels out of place is definitely a detail, what I'm embarrassed about is the other 370 open issues we have. More specifically #469 is what I've been working on, and I can't fix it until this PR is merged. #2809 is also blocked, as is almost every other issue or PR that needs to modify the UI.

I feel like we're reaching a point where making it not suck on mobile requires some refactoring. :-/

There was no need to refactor anything in order to fix the Explore page.

Is there a list of pages to review? Can we generate complete sitemap without missing teeth?

A few pixels out of place is definitely a detail
There was no need to refactor anything in order to fix the Explore page.

We're looking at different things. I am seeing way more than a few pixels out of place on the Explore page.

More specifically #469 is what I've been working on, and I can't fix it until this PR is merged. #2809 is also blocked, as is almost every other issue or PR that needs to modify the UI.

Fair enough. After diving in a bit on a refactor I admit that it's too much to add to this PR. I've started a new PR with my initial refactoring commits from last night, and have force-pushed those commits off of this branch. Go ahead and merge this when you're ready and I'll pick up on #3063.

Is there a list of pages to review? Can we generate complete sitemap without missing teeth?

$ find www -name \*.spt

Did a final visual review and sneaked in a fix for a small bug unrelated to this PR. Merging.


Failed to load module created in test

Describe the bug
After upgrading Vitest to 2.0.2, I get the error when import a file created in test:
import { afterEach, expect, it } from "vitest";
import { pathToFileURL } from "url";
import { unlinkSync, writeFileSync } from "fs";

const filename = "bar.js";

afterEach(() => unlinkSync(filename));

it("should works", async () => {
	writeFileSync(filename, "export default 123");
	const mod = await import(pathToFileURL(filename).href);
	expect(mod.default).toBe(123);
});

Error: Failed to load url /D:/Coding/JavaScript/vite-svg-sfc/bar.js (resolved id: /D:/Coding/JavaScript/vite-svg-sfc/bar.js). Does the file exist?
    at loadAndTransform (file:///D:/Coding/JavaScript/vite-svg-sfc/node_modules/.pnpm/vite@5.3.3_@types+node@20.14.10/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:52400:17)
    at async ViteNodeServer._transformRequest (file:///D:/Coding/JavaScript/vite-svg-sfc/node_modules/.pnpm/vite-node@2.0.2_@types+node@20.14.10/node_modules/vite-node/dist/server.mjs:524:16)
    at async ViteNodeServer._fetchModule (file:///D:/Coding/JavaScript/vite-svg-sfc/node_modules/.pnpm/vite-node@2.0.2_@types+node@20.14.10/node_modules/vite-node/dist/server.mjs:486:17)
    at async Proxy.fetch (file:///D:/Coding/JavaScript/vite-svg-sfc/node_modules/.pnpm/vitest@2.0.2_@types+node@20.14.10/node_modules/vitest/dist/vendor/cli-api.FtQZKvQk.js:8093:22)
    at async EventEmitter.<anonymous> (file:///D:/Coding/JavaScript/vite-svg-sfc/node_modules/.pnpm/vitest@2.0.2_@types+node@20.14.10/node_modules/vitest/dist/vendor/index.BpSiYbpB.js:69:20)

It works in Vitest 1.6.0
Reproduction
See above
System Info
System:
    OS: Windows 11 10.0.22621
    CPU: (12) x64 AMD Ryzen 5 5625U with Radeon Graphics
    Memory: 6.20 GB / 15.31 GB
  Binaries:
    Node: 22.4.1 - D:\Program Files\NodeJS\node.EXE
    npm: 10.8.1 - D:\Program Files\NodeJS\npm.CMD
    pnpm: 9.5.0 - ~\AppData\Roaming\npm\pnpm.CMD
    bun: 1.1.18 - ~\.bun\bin\bun.EXE
  Browsers:
    Edge: Chromium (124.0.2478.67)
    Internet Explorer: 11.0.22621.3527

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Works fine for me:

Seems to be an error only on Windows.
https://github.com/Kaciras/vitest-issue-1/actions/runs/9886516528/job/27306432893

Same error in vuepress. Only failed on windows.
https://github.com/vuepress/core/actions/runs/9934091779/job/27437900959

Same error in vuepress. Only failed on windows.
https://github.com/vuepress/core/actions/runs/9934091779/job/27437900959

I think this is fixed by #6109. Can you wait for the release?


Tensorflow 2.5 already out for JetPack 4.6

There's a wheel for jetpack 4.6 of TF2.5 here https://elinux.org/Jetson_Zoo#ONNX_Runtime

Are you sure?
I found a TF2.5 for Python3.6, not Ubuntu's 3.8.
Also found some ONNX_Runtime wheels intended for Python 3.8.
However, with a clear statement of the TensorFlow team that TF2.5 needs CUDA 11, I really doubt if it will work.
Sure, with a cross-compile you can generate everything you like. Did you test it?

@Qengineering it seems that nvidiais able to do it anyway. You are able to install tensorflow 2.7 on Jetpack 4.6 with the following steps:
# for tensorflow
sudo apt-get install libhdf5-serial-dev hdf5-tools libhdf5-dev zlib1g-dev zip libjpeg8-dev liblapack-dev libblas-dev gfortran -y
sudo apt-get install python3-pip -y
sudo python3 -m pip install --upgrade pip
sudo -H pip3 install testresources setuptools
# numpy .5 -> core dumb
sudo -H pip3 install numpy==1.19.4 future mock keras_preprocessing keras_applications gast protobuf pybind11 cython pkgconfig packaging
# h5py needs some special magic
sudo -H H5PY_SETUP_REQUIRES=0 pip3 install -U --no-build-isolation h5py==3.1.0
sudo -H pip3 install --extra-index-url https://developer.download.nvidia.com/compute/redist/jp/v461 tensorflow==2.7.0+nv22.1

Would it be possible after the upgrade to Ubuntu 20?

@muellevin,
Are you sure the installed TensorFlow version uses CUDA?
It is not the regular ARM - python 3.6 TensorFlow you have?

On current JP Version 4.6.3 with Ubuntu 18.04 after the installation i typed python3 -c "import tensorflow as tf; print(tf.config.list_physical_devices('GPU')) which returns [PhysicalDevice(name='/physical_device:GPU:0', device_type='GPU')]
If using the deprecated tf.test.is_gpu_available() it says that it found the NVIDIA Tegra X1 as GPU
The Tensorflow Version is from Nvidia. I was following their guide guide to install it but needed the modification from aove.


pointcloud convert to range image, is it always 64x870?

Hi, thank you first for sharing this excellent work!
If my input is pointcloud, such as the  KITTI .BIN file,  is this conversion using projection (sphere or ring),  always conver to 64x870 size?
How/Where this 870 is calculated?
And can we change this size 870, I saw some other size such as 64x1030.
My concern is the original xyz of the poincloud from BIN is around ~120,000 points while the range image is only covering  55680 xyz points (55680=870*64).  It seems more than 50% points are removed/overwrite in this "pc2range" conversion process. Can you comment on this?
Thank you very much!

So, the way it is implemented now was tailored to Frank Moosman's data. These data are all 64x870, which seems to give a good tradeoff between speed and precision. The way I handle multiple points that fall into the same pixel is that only the furthest reading is taken per pixel, but all the points that fall into it get assigned the same label. It can create some minor artifacts, but seemed to work well in practice for me.
You can however, pick any resolution you want if you look into the ProjectionParams class by adding another function like this one with any resolution you want.
Be advised, that working with KITTI data directly might be a little imprecise as the HDL64 has two sets of beams that do not exactly match onto a single depth image.

i think KITTI should be set as:
params.SetSpan(SpanParams(-180_deg, 180_deg, 4000),
SpanParams::Direction::HORIZONTAL);

because Velodyne HDL-64E has 0.09 degree angular resolution.

@niosus

I also think HDL64 row step should be 4000 or 2048,  but will this increase runtime a lot ?
can you  explain further about "HDL64 has two sets of beams that do not exactly match onto a single depth image"
the AngleDiff::PreComputeAlphaVecs,  if the angle is add by step, then it's a constant value, why need a vector to store and obtain it by r,c indices ?

thanks for your work and reply!


Statistics

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

56 of 60 (93.33%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 99.502%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

godal.go
35
39
89.74%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2600

Relevant Lines:
2613

💛  - Coveralls

Hello @tbonfort  & @thomascoquet
Thanks both of you!
Here are my proposals:

Statistics setter
User can set his own stats through:
err := bnd.SetStatistics(min, max, mean, std)
This function is a wrapper around GDALSetRasterStatistics() and populate metadatas.

Statistics computation
User can ask for statsitics computation. Moreover user can ask for accurate or approximated stats. Note that, this function force computation even if statistics are presents in metadatas.

Accurate statistics
stats, err := band.ComputeStatistics()

Approximated statistics
stats, err = band.ComputeStatistics(godal.Approximate())

This function is a wrapper around GDALComputeRasterStatistics() and populate metatdatas.

Statistics getter
User can ask for pre-computed statistics if they are present. If not a boolean is return as false.
flag, stats, err := bnd.GetStatistics()
Since GDALGetRasterStatistics() is not a pure getter on statistics, GetStatistics is not wrapper around GDAL function.
GODAL function is just a parser of metaDatas.
This prevent from:
+ use of appoxOk & bForce flags;
+ compute pre-computed statistics;
From user pre-computed stats, workflow could be:
err := bnd.SetStatistics(min, max, mean, std)         // do some stuff   flag, stats, err = bnd.GetStatistics()
From approximated statistics, workflow could be:
 stats, err := band.ComputeStatistics(godal.Approximate())   // do some stuff  flag, stats, err := bnd.GetStatistics()  //stats.Approximate == true

Statistics cleaner
User can clear statistics through:
err := ds.ClearStatistics()
This function is a wrapper around GDALDatasetClearStatistics(). This function is new feature from GDAL 3.2.


FEATURE: Add Table Component

Version
Vben Admin V5
Description
Add Table Component
Proposed Solution
Add Table Component
Alternatives Considered
No response
Additional Context
No response
Validations

[x] Read the docs
[x] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[x] I have searched the existing issues and checked that my issue does not duplicate any existing issues.

请问预计什么时候出来table组件呢

非常期待这个组件，加油

同求table组件！！！

啊啊啊啊，这个最重要了，怎么没有BasicTable了， 大佬期待啊！

请问大概什么时候可以推出？

等待table组件出来，就可以上生产了，加油


fix: Category/Section Errors

Realised I was committing stuff into 188832, so just making a quick PR to merge in to it instead of doing a push for visibility.

Do we need the internal name?

Do we need the internal name?

I think so, it makes the mapper happy and begins to join up all the tables to add the category in (now failing because it doesn't have all its sections, W.I.P)


Not working or bad install ?

[Vue warn]: Failed to mount component: template or render function not defined.
found in
--->

my code
window.vuejsDatepicker = require("vuejs-datepicker");
Vue.use(vuejsDatepicker);
Vue.component("my-component", vuejsDatepicker);
Please for help

How should I please ?

[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.

[Vue warn]: Unknown custom element: - did you register the component correctly? For recursive components, make sure to provide the "name" option.

getting same issue


[CU-8686ctghu] Adds page and services to claim cosmetics

Video overview: https://t8459928.p.clickup-attachments.com/t8459928/6e7c5bbd-c862-4784-8d6a-806d826e8931/screen-recording-2023-11-10-18%3A13.webm?view=open

Task linked: CU-8686ctghu Claim a Badge Page


Push button

I rotated the push button to be put on a wall instead of a table. It didn't really workout since I think the speed towards the initial position is depending on world postion y or something like that?

The button in the example is done using position constraints on the Rigidbody, which are in worldspace. I've had success managing button position with a Configurable Joint with xyz rotation locked/x and z movement locked with y movement limited, a very low linear limit and a high spring force. This results in something similar to a spring joint but with local space constraints instead of world space. Use code similar to NVR Button to trigger a button press when the button moves too far. This approach works no matter what angle the button is at.

Actually, I lasted tested NVR's buttons on what is now a several weeks old version of the repo, @zite in #28 seems to imply that this might not be the case any more.

Yeah, we're using local constraints now. Should work find on a wall

Closing, see commit https://github.com/TomorrowTodayLabs/NewtonVR/commit/33351dafa940780e4f5c9ea3e49be3defec0a44e


Find strings that are not localized

https://github.com/OrchardCMS/Orchard/blob/3ce2fc5f9ee8ba696637ceb7bc208105df12be10/src/Orchard/Localization/Services/DefaultLocalizedStringManager.cs#L52
We would like to find all labels and strings for which we do not have a localization in a given culture in our tenants.
To do this, we would like to make a change to the DefaultLocalizedStringManager so that we can report/log all the times a translation isn't found.
This change would not alter the API or the functionality of the current class: we would just move most of the body of FormatForScope GetLocalizedString(IEnumerable<string> scopes, string text, string cultureName) to a protected method that returns null if no localization is found; the current public method would call that, and return its current default if needs be.
This would allow us to create a feature where we would simply override that method in a class inheriting from DefaultLocalizedStringManager to log out the scope and text of those messages, so that we can then build our po files.
I'll provide a PR in a while with that change to DefaultLocalizedStringManager. This will target dev right now.

Closing this because the logger thing from the PR was merged already


The pretrained model for ResNet-18 backbone is not available

The link https://drive.google.com/open?id=10h_axdnkjupEDYe-OiUzm5ALX8w5DX_5 returns 404 code

You may find it here.
https://github.com/kenshohara/3D-ResNets-PyTorch


Known issue reporter skipping update in most of the issues

It was reported in FR channel that know issues reporter is not running for all the issues, investigating

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

After some investigation the problem was an issue that had problems assigning old the values for a user that was no longer in the org, I created some follow up issues:

https://github.com/dotnet/dnceng/issues/964
https://github.com/dotnet/dnceng/issues/962

closing this issue


Awesomewm: LD_LIBRARY_PATH handed down to Subprocesses

Basic info

Kernel: Linux vbob 4.4.2 #1-NixOS SMP Wed Feb 17 20:31:25 UTC 2016 x86_64 GNU/Linux
System: 16.03.git.40c586b (Emu)
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: 16.09.git.40c586b

Describe your issue here
When subprocesses are being started by awesome-wm they will inherit the complete wrapped environment of the window manager.
$ cat /run/current-system/sw/bin/awesome
export LUA_CPATH="/nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/lib/lua/5.2/?.so"${LUA_CPATH:+;}$LUA_CPATH
export LUA_PATH="/nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/share/lua/5.2/?.lua;/nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/share/lua/5.2/lgi/?.lua"${LUA_PATH:+;}$LUA_PATH
export GI_TYPELIB_PATH=/nix/store/n131myci13cns1q7x9a74awxqfpgqw3p-pango-1.38.1/lib/girepository-1.0:/nix/store/rbqrkq5g4yd6s6s67plqlg8yi65cy63i-gdk-pixbuf-2.32.3/lib/girepository-1.0:/nix/store/n2m8mk36686iilfq96r0hqpcjyhfdi78-gobject-introspection-1.46.0/lib/girepository-1.0:/nix/store/n131myci13cns1q7x9a74awxqfpgqw3p-pango-1.38.1/lib/girepository-1.0${GI_TYPELIB_PATH:+:}$GI_TYPELIB_PATH
export LD_LIBRARY_PATH=/nix/store/m0cp2r2n81k8277nf4v34iyp767fizwm-cairo-1.14.4/lib:/nix/store/n131myci13cns1q7x9a74awxqfpgqw3p-pango-1.38.1/lib:/nix/store/n2m8mk36686iilfq96r0hqpcjyhfdi78-gobject-introspection-1.46.0/lib${LD_LIBRARY_PATH:+:}$LD_LIBRARY_PATH
export PATH=/nix/store/xsi1z8b9yzg5znhqb7d2azywc1g5dh16-compton-0.1_beta2/bin:/nix/store/k6i5lmhz6y0gbz7bs7xrbr1wrp7cqf7p-unclutter-8/bin:/nix/store/3scdfmawm938iss0cxkqh1hw9vq1qr2s-procps-3.3.11/bin:/nix/store/f2z68wfywh4h31g9jpz4qizh9yr9kzr3-iproute2-4.3.0/sbin:/nix/store/78rzgch6q4x60frjh88095spyqpb8n12-coreutils-8.24/bin:/nix/store/1m46ywl6ca7h8a2j8rzcfnpw63694661-curl-7.47.1/bin:/nix/store/fpbcbaz0d1ksr8jvsvjga2ahdnbnks1p-alsa-utils-1.1.0/bin:/nix/store/xgnmdv1mq5l0pwr1jsp7dcwk4imzg341-findutils-4.4.2/bin:/nix/store/r7p17q7sq46p75lrahx23kvxqxrc3p3z-xterm-320/bin${PATH:+:}$PATH
exec -a "$0" /nix/store/794qxrmq3jfhhafnls5wlqqsj78lsv2p-awesome-3.5.8/bin/.awesome-wrapped "${extraFlagsArray[@]}" "$@"

Expected result
When processes are started under awesome, the basic environment should be used. LD_LIBRARY_PATH should not contain explicit paths to /nix/store. My specific problem was  that i was unable to start pidgin after installing it via nix-env because of the error message pidgin: symbol lookup error: /nix/store/n131myci13cns1q7x9a74awxqfpgqw3p-pango-1.38.1/lib/libpangoft2-1.0.so.0: undefined symbol: hb_buffer_set_cluster_level
Actual result
When a new terminal (urxvt+zsh in my case) is started the new shell will contains this environment, LD_LIBRARY_PATH being the most critical here because when new tools are installed via nix-env the LD_LIBRARY_PATH may not necessarily match.
Steps to reproduce

Enable xserver with awesome enable:

  services.xserver = {
    enable = true;

    windowManager = {
      awesome.enable = true;
      awesome.luaModules = [ pkgs.luaPackages.vicious ];
      default = "awesome";
    };

    desktopManager.xterm.enable = false;
  };
   environment.systemPackages = [ pkgs.rxvt_unicode-with-plugins ];

start GUI, and open a new shell WIN+r urxvt

$ echo $LD_LIBRARY_PATH
/nix/store/m0cp2r2n81k8277nf4v34iyp767fizwm-cairo-1.14.4/lib:/nix/store/n131myci13cns1q7x9a74awxqfpgqw3p-pango-1.38.1/lib:/nix/store/n2m8mk36686iilfq96r0hqpcjyhfdi78-gobject-introspection-1.46.0/lib:/run/opengl-driver/lib:/run/opengl-driver-32/lib
# this actually may only contain /run/opengl-driver/lib:/run/opengl-driver-32/lib

/cc maintainer @lovek323.
I expect the worst can be avoided easily by using patchelf+RPATH instead of $LD_LIBRARY_PATH.

@vcunat , sorry for the late reply.
i tried to use RPATH instead of LD_LIBRARY_PATH but then awesome does not boot up anymore. Any other idea how to solve this?

Update: the problem persists unfortunately and it breaks software from old nixpkgs when you try to run it because awesomewm spills cairo,gobject,pango and sane, as well as /run/opengl-driver/lib into the library path

i tried to use RPATH instead of LD_LIBRARY_PATH but then awesome does not boot up anymore. Any other idea how to solve this?

Well, why not? Which library is not found? Why doesn't dlopen look in RPATH as it should?
For the list of environment variables given in the first comment (assuming the goal is to get rid of as many of them as possible):
export LUA_CPATH="/nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/lib/lua/5.2/?.so"${LUA_CPATH:+;}$LUA_CPATH
export LUA_PATH="/nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/share/lua/5.2/?.lua;/nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/share/lua/5.2/lgi/?.lua"${LUA_PATH:+;}$LUA_PATH

The above can be replaced with passing --search /nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/lib/lua/5.2/ --search /nix/store/pc0lrj04hp54syzqgfiba8gk7lm7byk2-lgi-0.7.2/share/lua/5.2/lgi/ to awesome (needs at least v4.1, I think).
export GI_TYPELIB_PATH=/nix/store/n131myci13cns1q7x9a74awxqfpgqw3p-pango-1.38.1/lib/girepository-1.0:/nix/store/rbqrkq5g4yd6s6s67plqlg8yi65cy63i-gdk-pixbuf-2.32.3/lib/girepository-1.0:/nix/store/n2m8mk36686iilfq96r0hqpcjyhfdi78-gobject-introspection-1.46.0/lib/girepository-1.0:/nix/store/n131myci13cns1q7x9a74awxqfpgqw3p-pango-1.38.1/lib/girepository-1.0${GI_TYPELIB_PATH:+:}$GI_TYPELIB_PATH

Sorry, I don't know much about gobject-introspection myself, but I guess this env var cannot be easily avoided.
For LD_LIBRARY_PATH:

cairo is loaded through dlopen() (through gobject-introspection) and man dlopen says that it should respect DT_RPATH tags.
Same for pango
Awesome itself does not use gobject-introspection. Assuming that lgi is properly built (=finds gobject-introspection on its own), I don't see why this entry is necessary at all.

That leaves $PATH for which I also don't think it is necessary, but I guess this was done for convenience so that "this random set of tools" is found via $PATH from the user's config.

@psychon thanks! i will test DT_RPATH and --search for awesomewm this evening

Yes, extending RPATH through patchelf does (generally) work for dlopen.

Okay, I took a closer look and installed NixOS in qemu. As a first try, without modifying anything, the following command worked to start awesome (yes, I'm running this as root; line breaks added for readability):
env -i \
GI_TYPELIB_PATH=$GI_TYPELIB_PATH \
LD_LIBRARY_PATH=/nix/store/5mx1*cairo-1.14.10/lib \
HOME=/root \
DISPLAY=:1 \
/nix/store/2gi6wif*awesome-4.2/bin/.awesome-wrapped \
--search /nix/store/72v*lgi-0.9.1/lib/lua/5.2/ \
--search /nix/store/72v*lgi-0.9.1/share/lua/5.2/

So getting rid of LUA_PATH and LUA_CPATH seems to work.
(And yes, most stuff is already in RPATH, apparently.)
I then used patchelf to add that LD_LIBRARY_PATH to awesome's rpath and... that did not work...
According to strace, only /nix/store/*glib-2.52.3/lib/libcairo-gobject.so.2, /nix/store/*pcre-8.41/lib/libcairo-gobject.so.2, and /nix/store/g1g31ah*-glibc-2.25-49/lib/libcairo-gobject.so.2 are checked, no other paths are searched?! Who pulled in pcre and why is the RPATH ignored?!? This is a completely different search path that is being applied.
Looking at gobject-introspection and glib, the code here does some weird dances, but in the end just calls dlopen("libcairo-gobject.so.2") which then fails, for whatever reason.
Out of a hunch, I checked DT_RPATH for /nix/store/1nv*gobject-introspection-1.52.1/lib/libgirepository-1.0.so and this at least contains a subset of the search path that is being used. No idea how DT_RPATH on binary and on shared library are combined, but apparently one overwrites the other?
@vcunat Thoughts?

Does patchelf --print-rpath show what you would expect?  There's an automatic patchelf run in fixup phase, so it's typically best to add postFixup where you --print-rpath and extend it by some more paths.

@vcunat I have not much clue about NixOS, so I did not built my own awesome package. Instead I literally did what I said above: I copied the binary built by NixOS to /root/awesome and used patchelf on that. Hence, I'm quite certain that the automatic patchelf run in fixup phase did not affect this at all.
Instead, it seems like dlopen() is sensitive to its caller: When called from the main binary, the RPATH (actually, RUNPATH) of the binary is used, while when called from a shared object, only that object's settings are used?! Google doesn't really find any documentation on this...
I just copied libgmodule-2.0.so.0 to /root/, too, and used patchelf to append the directory containing libcairo-gobject.so.2 to its RPATH (well, RUNPATH). Then I replaced the LD_LIBRARY_PATH from my previous comment with LD_LIBRARY_PATH=. (= make awesome use the patched libgmodule-2.0.so.0). With this I was able to run awesome without any "real" $LD_LIBRARY_PATH.
I don't know how to incorporate this into NixOS. I guess you do not want to make glib (which is where libgmodule-2.0.so.0 comes from) depend on cairo, so...
You could patch awesome to dlopen("libcairo-gobject.so.2") at the beginning of main(). This is ugly, but could work.
The above raises a question for me: Does libgmodule work on NixOS? From the above analysis, it should pretty much always fail to find libraries, unless they appear in $LD_LIBRARY_PATH?!

I'm not sure, but it seems very likely that for dlopen only R(UN)PATH of the original executable is considered.  (The dlopen call will hardly be trying to find which part of code called it.)

@vcunat Here's the line in dlopen that gets the instruction pointer in the caller:
https://sources.debian.org/src/glibc/2.23-1/dlfcn/dlopen.c/#L84
And here is the line of code that seems to translate that into a loaded library:
https://sources.debian.org/src/glibc/2.24-12/elf/dl-open.c/?hl=598#L222
The resulting handle is then passed to _dl_map_object:
https://sources.debian.org/src/glibc/2.24-12/elf/dl-open.c/?hl=598#L237
...which accesses things like ->l_info[DT_RUNPATH]:
https://sources.debian.org/src/glibc/2.24-12/elf/dl-load.c/?hl=867#L2271
So yes, I have not actually tried this in a self-contained example, but to me it looks like dlopen only checks the DT_RUNPATH of the object that called it. So, for loading libcairo-gobject.so.2, the RUNPATH in the awesome binary is ignored, because dlopen is called from libgmodule.
This also fits with my other experiment above: When adding the path to libcairo-gobject.so.2 to the binaries's DT_RUNPATH, the library is still not found. When adding the path to libgmodule-2.0.so.0's DT_RUNPATH, everything starts working.

i am really glad that this issue got solved! thanks to everyone involved

Thanks for digging into these mechanisms.