sift image registration should work in linear space, not gamma compressed space

This impacts only the sRGB -> gray preprocessing where gamma decompression should be applied.
This creates artefacts in the gradient calculation, over-representing dark regions.
Beside this is would be interesting to have all processing in the range [0..1[ instead of [0..255] which is anyway in single precision float.

I am no more sur about this ...

leave it open for discussion if needed


update monitored ldiskfs to use lvm and ha

update monitored ldiskfs to use lvm and ha
Signed-off-by: johnsonw<EMAIL_ADDRESS>

This change is

and I was able to run ha-ldiskfs-lvm-fs-setup without the environment variable

This breaks the VBox fence agent, which cannot communicate without the var being set

Ah I see. Ok I'll make sure to run the command with that environment variable set.

Ah I see. Ok I'll make sure to run the command with that environment variable set.

Make sure it's obfuscated and doesn't appear in the source or test output.


Playground Ada still different from mockchain Ada

Forging 10 Ada in the Playground and then spending Ledger.Ada.adaValueOf 10 results in a NegativeValueError because the currency that was forged is different from the currency that adaValueOf creates.

I believe this is also the cause of #952


Add addYears and addMonths

I'm not sure if you left these out on purpose but I needed them and thought I would add them. I wasn't able to write them using addMilliseconds as obviously months and years can have a different amount of milliseconds in them.

I also seem to get failing tests locally on latest master (before my commit):
✓ sanity
✓ noon without input date
✓ noon with input date
✓ adding days
✓ adding hours
✓ adding minutes
✓ adding seconds
✓ adding milliseconds
⨯ differenceInDates
  not ok 28 should be equal
    ---
      operator: equal
      expected: 1
      actual:   0
      at: Test.<anonymous> (/Users/chris/Source/sheepsteak/instadate/test/main.js:79:5)
    ...
✓ differenceInDays
✓ differenceInWeekendDays
✓ differenceInHours
✓ differenceInMinutes
✓ differenceInSeconds
✓ isSameYear
✓ isSameMonth
✓ isSameDay
✓ dates
✓ equal
✓ dateString
✓ min
✓ max
⨯ firstDateInMonth
  not ok 82 should be equal
    ---
      operator: equal
      expected: |-
        'Fri Jan 01 2016 13:07:17 GMT+0200 (EET)'
      actual: |-
        'Fri Jan 01 2016 11:07:17 GMT+0000 (GMT)'
      at: Test.<anonymous> (/Users/chris/Source/sheepsteak/instadate/test/main.js:227:5)
    ...
⨯ lastDateInMonth
  not ok 83 should be equal
    ---
      operator: equal
      expected: |-
        'Sun Jan 31 2016 13:07:17 GMT+0200 (EET)'
      actual: |-
        'Sun Jan 31 2016 11:07:17 GMT+0000 (GMT)'
      at: Test.<anonymous> (/Users/chris/Source/sheepsteak/instadate/test/main.js:233:5)
    ...
✓ isWeekendDay
✓ isWeekendDate
✓ isWorkDay
✓ isWorkDate
✓ setWeekendDays
✓ daysInPeriod
✓ weekendDaysInPeriod
✓ workDaysInPeriod
✓ isAfter
✓ isBefore
✓ isYearAfter
✓ isYearBefore
✓ isMonthAfter
✓ isMonthBefore
⨯ isDayAfter
  not ok 171 (unnamed assert)
    ---
      operator: notOk
      expected: false
      actual:   true
      at: Test.<anonymous> (/Users/chris/Source/sheepsteak/instadate/test/main.js:417:5)
    ...
⨯ isDayBefore
  not ok 177 (unnamed assert)
    ---
      operator: ok
      expected: true
      actual:   false
      at: Test.<anonymous> (/Users/chris/Source/sheepsteak/instadate/test/main.js:429:5)
    ...
✓ isYearBetween
✓ isMonthBetween
✓ isDayBetween
✓ isoWeekDay
✓ isoDateString
# tests 205
# pass  200
⨯ fail  5

So I've found out what is causing the tests to fail but I don't understand why. I noticed you're doing everything in the EET timezone but my Mac is running in British English.
I'm getting some strange results like the following when using toString in Node 5.9.1 and Chrome Dev (51):
> new Date('Mon Jan 18 2016 13:07:17 GMT+0100 (BST)').toString()
'Mon Jan 18 2016 12:07:17 GMT+0000 (GMT)'
> new Date('Mon Jan 18 2016 14:07:17 GMT+0200 (EET)').toString()
'Mon Jan 18 2016 12:07:17 GMT+0000 (GMT)'
> new Date().toString()
'Fri Apr 01 2016 11:07:52 GMT+0100 (BST)'

It's forcing things into UTC unless I use the empty Date constructor.

Thanks for the PR! :hand:
Yeah, I left them out because the days in months and years are changing.
Could you add some more tests to see what happens when you add 11/12 months for instance. Does the year change, will the month stay the same when adding 12 months?
Any ideas why your tests might be failing? In which timezone are you?

Good find! I don't have time right now to fix the timezone toString bug but I will take a look at it later.

Yeah, I will add more tests.
I noticed you have export TZ=Europe/Helsinki in your travis.yml. I tried this locally and the tests pass. I'm in Europe/London so that is why toString was outputting in GMT+0000 for me by default (even though we're in GMT+0100 daylight savings right now!). So TZ needs setting for npm test or all tests should use UTC. I'll make a separate issue about this so you can decide what you want.

Thanks!

Hi @herkyl, I've added the extra tests you wanted.

Thanks! Published as 0.3.0


Issue connect with Cyberduck

Hi,
thanks for the great tutorial. I have setup virtual machine exactly the same according to you instructions .I managed to connect google storage from virtual machine but when I tried to connect from cyberduck its not connected. This is the error . Operation timed out. The connection attempt was rejected. The server may be down, or your network may not be properly configured   In the tutorial you have not explained any firewall related issue:-)
thanks in advance

Howdy my friend ... let's see if we can get a handle on the puzzle.  I'm assuming that you are running the sftp demon/code on a Compute Engine within GCP.  Where are you running your cyberduck client?  Is the puzzle only with the cyberduck client?  If you tried with an alternate SFTP client does it work?

thanks for the quick reply, Yes I am running demon/ code on a computer engine and cyber duck on the local machine. No it does not work with alternative SFTP client as well. I think port is the issue and I tried to create firewall rule on GCP but it still does not work. I like the way you did the implementation and it's perfect:-) When I run locally in Compute engine it works.If we can solve this access outside with username and password.This is great implementation

Okly ... I'm imagining you are running the demon on a Compute Engine on GCP.  I am imagining that the Compute Engine is running a Linux OS such as Debian.  I am imagining that the Compute Engine is attached to a GCP VPC network subdomain.  I am also imagining that the Compute Engine has a public IP address that you are using to reach it over the Internet.
Have you created any firewall rules for accessing the demon?  What TCP/IP port number is the demon listening upon?
My gut is saying that we will have to create an inbound (ingress) firewall rule that permits traffic to come into your Compute Engine from the Internet to the port on which the demon is listening.

yes, I follow the same instruction according to your tutorial.Yes,compute engine is attached and using external IP to access from cyber duck.
I have created the rule for firewall rules to the demon and demon listening port is 8022
Everything as you recommended:-) Here are the commonds:
sudo apt-get update
sudo apt-get install nodejs git --yes
curl -L http://www.npmjs.org/install.sh | sudo sh
npm install
git clone https://github.com/kolban-google/sftp-gcs.git
cd sftp-gcs
ls
npm install
node sftp-gcs --debug=debug --bucket gs://gubi_oddo --port=8022 --user=***** --password=*********
If you have a time and I can share my screen on skype or somewhere else?

Image with firewall rule

I think a screen share will be great.  Its 3:18pm on Thursday afternoon.  You can email me at<EMAIL_ADDRESS>or try and use Google Chat at the same address.  I have a client call 4pm-5pm.  I am out tomorrow ... if not now, then maybe we can synch up for next week?  Email me a couple of good times for you as needed.
Neil

@kolban-google  thanks and it works after opening the firewall:-) Thanks for the help

Hi there,
Thank you for the tutorial and elegant bridging solution you built @kolban-google.
I'm encountering a similar issue.
I followed all the steps laid out here https://www.youtube.com/watch?v=yREQRw1H23Q&t=389s
Trying to connect to the SFTP server from my mac computer via Cyberduck fails with the same error message you laid out @sham1111
Operation timed out. The connection attempt was rejected. The server may be down, or your network may not be properly configured
I didn't set any specific firewall rule.
Could you expand on this it works after opening the firewall ?
I tried using the native "SFTP" of my mac terminal and was unable to connect to the server as well..

After Filezilla I found that cyberduck also can't be used for sftp with this software (linux ftp command and winscp works). Is the problem same as with Filezilla with serial or something different


use secp256k1 library for signing

The Problem

The Solution

Checklist:

[ ] I am making this PR against staging, not master
[ ] My code follows the code style of this project.
[ ] I have described any backwards-incompatibility implications above.
[ ] I have highlighted which parts of the code should be reviewed most carefully.
[ ] If my change requires a change to the documentation, I have updated it accordingly.

This was breaking with ConnextBenchmarking and even simply loading the DaiCard, I'm unsure how this even passed all the tests. We should investigate the case where signing breaks but all the tests are passing.


add uploadAudioFile method

Hi,
Thank you very much for your work. This is help me a lot.
And,
I have add uploadAudioFile feature and example.
This method can upload audio file(NSData) to soundcloud.
If you have free time, please help me to review it. Thank you very much :-)

Nice! Thanks a lot for the contribution, the code looks good :)


Update the name of the 'Query' button to 'Close Query' in the toolbar after opening query builder

Storage Explorer Version: 1.17.0
Build Number: 20210120.2
Branch: main
Platform/OS: Windows 10/ CentOS 7.6.1810 (Core)/ MacOS Catalina
Architecture: ia32/x64
Regression From: Previous release (1.17.0)
Steps to Reproduce

Expand one storage account -> Tables.
Open one table -> Click the button 'Query' in the toolbar.
Check whether the button name is 'Close Query'.

Expected Experience
The button name is 'Close Query'.

Actual Experience
The button name is 'Query'.

Verified on build 20210210.2. This issue still reproduces and we will continue to verify it.

Cloud Explorer is updated, so fix should be available now.

This issue has been fixed on build main/20210218.1.

Hi @JasonYeMSFT Verified on the lasted build main/20210512.2. The name has been fixed. But the tooltip of the button 'Close Query' is incorrect.

Expect:

Verified on build 20210513.1. Fixed.


Feature/zoom

this is branched from the multiline label work, implementing basic d3 zoom logic as a starting point so that we can refine

@rlskoeser is this meant to be reviewed on mobile only?

I was envisioning that when you zoom on desktop it will zoom within the limits of the svg container – do we prefer this?

If we decide it will look like the above screenshot then I think we should hide the footer when the zoom is active.

I do expect something like the above screenshot happening on zoom on mobile

is this meant to be reviewed on mobile only?

Should there be any zoom on desktop, or is it only for mobile?

I was envisioning that when you zoom on desktop it will zoom within the limits of the svg container – do we prefer this?

No, sorry! I thought I added a style change to hide overflow, which should truncate at the boundary of the svg container, but maybe I didn't push that change.

is this meant to be reviewed on mobile only?

Should there be any zoom on desktop, or is it only for mobile?

@rlskoeser I think we'll need it on desktop, as I mentioned I don't think we should try to show everything at once on desktop.

I was envisioning that when you zoom on desktop it will zoom within the limits of the svg container – do we prefer this?

No, sorry! I thought I added a style change to hide overflow, which should truncate at the boundary of the svg container, but maybe I didn't push that change.

ah that makes sense, thanks

closing this PR since we're not ready to implement zoom


Undefined swiftlint?

To preface this, I work behind a corporate firewall/proxy, and unfortunately some GitHub repositories whose release artifacts are hosted on AWS CDN are completely blocked. SwiftLint is one of those repos.
All that is to say that the install process for this gem was not easy or fun (usually never is for me, regardless of the dependency, not your fault).
So this is basically what I did:
# clone SwiftLint at the version that we need (0.31.0)
git clone --branch 0.31.0 https://github.com/realm/SwiftLint.git
cd SwiftLint

# build and install SwiftLint
git submodule update --init --recursive; make install

# verify SwiftLint installed:
which swiftlint # should output: "/usr/local/bin/swiftlint"
swiftlint version # should output "0.31.0"

# tell the installer for this gem to skip installing SwiftLint because we did it ourselves already
export DANGER_SKIP_SWIFTLINT_INSTALL="YES"
gem install danger-swiftlint -v 0.21.1

# after installation, we need to symlink SwiftLint into this gem's bin path
cd ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-swiftlint-0.21.1/ext/swiftlint/
mkdir bin;cd bin
ln -s /usr/local/bin/swiftlint $PWD/swiftlint

I got ~/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-swiftlint-0.21.1/ext/swiftlint/bin/ as the install path that your Rakefile was attempting to install SwiftLint at.
Anyways, immediately after doing this (a couple days ago), everything worked, I was able to run danger pr locally and test lint a PR without issue. Even worked when running the Jenkins job against the PR. I don't believe anything has changed in the last couple days, but as of yesterday our PR's are now failing the linting stage with the following error:
[12:21:34]: Exit status of command 'bundle exec danger --verbose --danger_id=danger_linting --dangerfile=/Users/<jenkins-username>/jenkins-agent/workspace/<job-name-directory>/.build/Dangerfile --fail-on-errors=true' was 1 instead of 0.
You used `puts` in your Dangerfile. To print out text to GitHub use `message` instead
bundler: failed to load command: danger (/Users/<jenkins-username>/.rbenv/versions/2.6.3/bin/danger)
Danger::DSLError:
[!] Invalid `Dangerfile` file: undefined local variable or method `swiftlint' for #<Danger::Dangerfile:0x00007f92a910f728>
 #  from /Users/<jenkins-username>/jenkins-agent/workspace/<job-name-directory>/.build/Dangerfile:4
 #  -------------------------------------------
 #  if !(github.pr_labels.include? "skip-linting")
 >    swiftlint.config_file = '.build/swiftlint.yml'
 #    swiftlint.binary_path = '/usr/local/bin/swiftlint'
 #  -------------------------------------------

  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/dangerfile.rb:70:in `method_missing'
  /Users/<jenkins-username>/jenkins-agent/workspace/<job-name-directory>/.build/Dangerfile:4:in `eval_file'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/dangerfile.rb:297:in `eval'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/dangerfile.rb:297:in `eval_file'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/dangerfile.rb:200:in `block in parse'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/dangerfile.rb:197:in `instance_eval'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/dangerfile.rb:197:in `parse'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/dangerfile.rb:273:in `run'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/danger_core/executor.rb:29:in `run'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/lib/danger/commands/runner.rb:72:in `run'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/claide-1.0.3/lib/claide/command.rb:334:in `run'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/danger-6.1.0/bin/danger:5:in `<top (required)>'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/bin/danger:23:in `load'
  /Users/<jenkins-username>/.rbenv/versions/2.6.3/bin/danger:23:in `<top (required)>'

This is our Dangerfile:
if !(github.pr_labels.include? "skip-linting")
  swiftlint.config_file = '.build/swiftlint.yml'
  swiftlint.binary_path = '/usr/local/bin/swiftlint'
  swiftlint.lint_files fail_on_error: true
  puts "Completed Linting Stage!"
else
  warn "You skipped linting."
end

Any ideas what could be wrong? I'm kinda stumped at this point. I assumed maybe it had something to do with Ruby or Danger itself not knowing what "swiftlint" is, so I tried to add a symlink to it in ~/.rbenv/shims but that cleaned itself up and removed it the next time Terminal was opened (or the next time RBENV was initialized).

Hey! Thanks for opening the issue – sorry for the delay in responding (holidays).
It doesn't look like it's SwiftLint's installation that is the issue. Ruby is complaining that there is no swiftlint variable, but that should be loaded as a result of having the danger-ruby-swiftlint in your Gemfile. The problem is that Danger isn't finding the danger-ruby-swiftlint plugin, not that it can't find the SwiftLint executable.
I've read and re-read the console output you've provided and it all looks good. I've looked through the Danger issues and couldn't find anything relevant. You said it was working, and then stopped? Did you update Ruby or Danger versions in the interim?

I have not. Still using:

ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-darwin17]
danger (6.1.0)
danger-swiftlint (0.21.1)
fastlane 2.137.0
swiftlint 0.31.0

When I run danger --version, get some nice deprecation output before the version too 😝
danger --version
NOTE: Inheriting Faraday::Error::ClientError is deprecated; use Faraday::ClientError instead. It will be removed in or after version 1.0
Faraday::Error::ClientError.inherited called from /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/octokit-4.14.0/lib/octokit/middleware/follow_redirects.rb:14.
6.1.0

Okay thanks, and thanks for the info. Those all look good.
Taking a look again, it could be the way that the danger-swiftlint gem is installed. You have this line in the CI config:
gem install danger-swiftlint -v 0.21.1

And this installs danger-swiftlint in the global gems. But later, you invoke danger through Bundler:
bundle exec danger ...

So I'm wondering if that could be the source of the problem? I think Danger uses the Gemfile to load Danger plugins, so being installed as a global gem might not be enough to actually load the plugin (which would cause the error we're seeing). I took a look through Danger's plugin-loading code but didn't see any smoking guns.
Here's what I'd try next: you can either install Danger globally, or (what I'd recommend) is moving the danger-swiftlint installation out of gem install and into your Gemfile. Let me know what I can clarify.

Honestly, still a bit new to some of the CI stuff, so I genuinely didn't know there was a difference in how gems were installed (via gem install danger-swiftlint -v 0.21.1 vs. bundle install using a cloned repo's Gemfile). I'll uninstall everything and see if installing them all one specific way helps.
It's a tad bit confusing when I go to list gems and they're saying "LOCAL" even though you're right, I'm pretty sure I installed them globally on the system via that gem install command.
$ gem list -e danger -d

*** LOCAL GEMS ***

danger (6.1.0)
    Authors: Orta Therox, Juanito Fatas
    Homepage: https://github.com/danger/danger
    License: MIT
    Installed at: /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0

    Like Unit Tests, but for your Team Culture.

$ gem list -e danger-swiftlint -d

*** LOCAL GEMS ***

danger-swiftlint (0.21.1)
    Authors: Ash Furrow, David Grandinetti, Orta Therox, Thiago Felix,
    Giovanni Lodi
    Homepage: https://github.com/ashfurrow/danger-ruby-swiftlint
    License: MIT
    Installed at: /Users/<jenkins-username>/.rbenv/versions/2.6.3/lib/ruby/gems/2.6.0

    A Danger plugin for linting Swift with SwiftLint.

 Can't thank you enough for the assistance and I apologize for my rookie mistake.
$ gem list -a danger

*** LOCAL GEMS ***

danger (6.1.0)
danger-swiftlint (0.21.1)
$ gem uninstall -a danger-swiftlint
Remove executables:
	danger-swiftlint

in addition to the gem? [Yn]  Y
Removing danger-swiftlint
Successfully uninstalled danger-swiftlint-0.21.1
$ gem uninstall -a danger
Remove executables:
	danger

in addition to the gem? [Yn]  Y
Removing danger
Successfully uninstalled danger-6.1.0

$ bundle install
Warning: the running version of Bundler (1.17.2) is older than the version that created the lockfile (1.17.3). We suggest you upgrade to the latest version of Bundler by running `gem install bundler`.
Fetching gem metadata from https://rubygems.org/.............
Fetching gem metadata from https://rubygems.org/..
Resolving dependencies...
... removed irrelevant spam
Fetching danger 6.1.0
Installing danger 6.1.0
Fetching danger-swiftlint 0.21.1
Installing danger-swiftlint 0.21.1 with native extensions
... more irrelevant spam removal
Bundle complete! 12 Gemfile dependencies, 126 gems now installed.
Use `bundle info [gemname]` to see where a bundled gem is installed.

$ bundle exec danger pr <github-enterprise-url> --dangerfile=.build/Dangerfile --danger_id="danger_linting" --verbose --fail-on-errors=true
Running your Dangerfile against this PR - <github-enterprise-url>

You used `puts` in your Dangerfile. To print out text to GitHub use `message` instead
Completed Linting Stage!
Results:

Errors:
- [ ] Failed due to SwiftLint errors

Markdown:
### SwiftLint found issues

#### Warnings

File | Line | Reason |
| --- | ----- | ----- |
SomeRandomViewController.swift | 121 | Colons should be next to the identifier when specifying a type and next to the key in dictionary literals. (colon)

That was run locally, going to test it through Jenkins now, but got excited and wanted to post this small victory here first.

Reporting back in, successfully running against PR's using Jenkins. Thanks again so much for all of your help with this, you are the best! 👍

@gdoublev Awesome! So glad that worked 🎉 Absolutely no need to apologize, either. Could make a good blog post, if you're keen!


webcaster broken

Checklist

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

Verbose log

[debug] System config: ['--prefer-free-formats']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://start.webcaster.pro/iframe/feed/start/free_995f7f31248f3be39606a964fa3a216d_hd/33_2711111319/53c7af8beeada7307929eaef372de470/4765903776']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.01.08
[debug] Python version 3.8.6 (CPython) - Linux-5.10.6-100.fc32.x86_64-x86_64
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4
[debug] Proxy map: {}
[generic]<PHONE_NUMBER>: Requesting header
WARNING: Falling back on generic information extractor.
[generic]<PHONE_NUMBER>: Downloading webpage
[generic]<PHONE_NUMBER>: Extracting information
ERROR: Unsupported URL: https://start.webcaster.pro/iframe/feed/start/free_995f7f31248f3be39606a964fa3a216d_hd/33_2711111319/53c7af8beeada7307929eaef372de470/4765903776
Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/youtube_dl/YoutubeDL.py", line 802, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/lib/python3.8/site-packages/youtube_dl/YoutubeDL.py", line 823, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3.8/site-packages/youtube_dl/extractor/common.py", line 532, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3.8/site-packages/youtube_dl/extractor/generic.py", line 3467, in _real_extract
    raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://start.webcaster.pro/iframe/feed/start/free_995f7f31248f3be39606a964fa3a216d_hd/33_2711111319/53c7af8beeada7307929eaef372de470/4765903776

Description

URL is playable in browser but not detected by youtube-dl

Hi! Thanks for opening an issue. Looks like that particular format of URL isn't supported right now - are you able to get a bl.webcaster.pro link? That should work. If not I can look into it further for you.

Hi! Thanks for opening an issue. Looks like that particular format of URL isn't supported right now - are you able to get a bl.webcaster.pro link? That should work. If not I can look into it further for you.

This one works
https://bl.webcaster.pro/feed/start/free_995f7f31248f3be39606a964fa3a216d_hd/33_2711111319/53c7af8beeada7307929eaef372de470/4765903776

This one works
https://bl.webcaster.pro/feed/start/free_995f7f31248f3be39606a964fa3a216d_hd/33_2711111319/53c7af8beeada7307929eaef372de470/4765903776

Ah excellent! We can probably close this issue then I think.

Ah excellent! We can probably close this issue then I think.


Molecular Weight Calculator

Title and Issue number
Molecular Weight Calculator Added
Issue number #4676
Close #4676
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 @TusharKesarwani Sir kindly review the pr

@MohitGupta121 sir kindly review the pr


[Bug]: Document properties

Attach (recommended) or Link to PDF file
https://zenodo.org/records/14107216
Web browser and its version
Firefox Nightly 135.0a1 (2024-12-12) (64-bit)
Operating system and its version
Windows
PDF.js version
4.9.143
Is the bug present in the latest PDF.js version?
Yes
Is a browser extension
No
Steps to reproduce the problem
Visit the page and select the "Document Properties..." option in the embedded document.
What is the expected behavior?
A popup shows with document properties.
What went wrong?

Link to a viewer
No response
Additional context
No response

TL;DR Please contact that website and ask them to update their outdated PDF.js version.

Attach (recommended) or Link to PDF file
https://zenodo.org/records/14107216

Unfortunately that website uses a customized and extremely outdated version of the PDF.js library, hence it's not the Firefox built-in PDF Viewer you're seeing. According to the web console:
Warning: Setting up fake worker. [pdf.js:342:12](webpack://invenio-assets/node_modules/pdfjs-dist/build/pdf.js)
PDF 35e79f792768ac49b2210bfcab068d1d [1.7 - / Microsoft Word] (PDF.js: 1.10.100) [viewer.js:7347:14](webpack://invenio-assets/js/invenio_previewer/pdfjs/viewer.js)

Note that that version was released over six years ago, see https://github.com/mozilla/pdf.js/releases/tag/v1.10.100

PDF.js version
4.9.143
Is the bug present in the latest PDF.js version?
Yes

Please note that the issue template asks you to check the latest PDF.js version, using https://github.com/mozilla/pdf.js#online-demo

Steps to reproduce the problem
Visit the page and select the "Document Properties..." option in the embedded document.

WFM, when using https://mozilla.github.io/pdf.js/web/viewer.html


Release 0.18.0

Signed-off-by: Miroslav Kovar<EMAIL_ADDRESS>

Codecov Report

Merging #281 (02cf6ae) into master (b708ce4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #281   +/-   ##
=======================================
  Coverage   49.73%   49.73%
=======================================
  Files         149      149
  Lines       19640    19640
  Branches     6134     6132    -2
=======================================
  Hits         9768     9768
  Misses       5086     5086
  Partials     4786     4786

Flag
Coverage Δ

integration
22.76% <ø> (ø)

unittests
45.43% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Rule 3.7: declare one variable on one line

Each property or variable declaration should be declared on separate line. Bad example: val n1: Int; val n2: Int.
Duplicate of reccomendation 3.4?

Duplicate of rule 3.4.


3D outer interocular distance calculation

Hi!
Firstly, thank you for making the code available! I am trying to benchmark the method using the metric mentioned in the paper named NME. However, I am wondering how do you get the outer interocular distance on the mesh in 3D euclidean space? Do you use the fixed topology index in 3DMM and if so, would you mind sharing which index of the mesh correspond to the eye region? Thank you!

In the case of the 3DDFA datasets, the 3DMM mesh was indeed used. The vertex for normalisation was selected by finding the nearest neighbour to the landmark points (37 and 46) on many frontal images prior to computing any metrics (as in, this remained fixed from this point on).
On Florence and BU4DFE it is clearly more difficult because the meshes do not have fixed correspondence. If I remember correctly, I ran a landmark detect on the frontal texture images and transferred these to the 3D space using the texture coordinates for the 3D object. So in this case I just used the landmarks mentioned above.

Hi!
Thank you for your clarification! Yes! That's what I am doing for 3DDFA. I see. yes, Florence and BU4DFE does not have fixed topology. So would you mind sharing how do you handle the profile view in Florence and BU4DFE then? As the ray casting of 2D landmark to 3D is ambiguous, especially when half of the face is occluded. Thanks!

The 2D landmarks are not projected onto the 3D rendering. They are looked up via the texturing information. It's a bit annoying to implement but it works very well. Each face in the mesh has a texture coordinate index, and each texture coordinate has xy values between 0-1. So, you scale the texture coordinate back to the original texture size, and find each facial point in the texture, and work back from there to the mesh's face and then to one of the surrounding vertices. Use the xyz coordinate of the vertex you have chosen as your new facial point. The XY projection will be the same, but the Z component will be correct even for the self occluded landmarks.

I see. Got it. Thank you!! I will look into the texture image. Just to clarify, the texture image that you are talking about is the conformal mapping from facial mesh to the 2D plane right? Something like this

for the 3D mesh of

No, otherwise you would not be able to run a landmark detector on them. They are just high resolution photographs which have been mapped to the 3D object. An example from BU-4DFE below. I just had a look at the Florence images and I may have done it slightly differently because they do include frontal textures. If I remember or find the time to look up what I did, I'll get back to you.

I see! Sure! Thank you for helping in digging this up! I will look into more details of the other two dataset and see if I can get the 3D position of the two corners using the method described. Thanks!

I have the following in my code:
    d = sqrt(sum((G(2853,:) - G(13673,:)).^2,2));

This was applied BEFORE the vertex filter. I haven't checked them so obviously please confirm visually that this is correct. Otherwise, I might be looking at the wrong code.

Hi!
Thank you very much! Yes, that was before the vertex filter and it works! Just that for other people who are also interested in it, when using the python code base, the index needs to be minused by 1 since matlab uses 1-based indexing.
Thanks!

Hi!
Yes! I am using the open3D python binding for icp. Yes, I figured out a way to align them better. Also, thank you for the heads up on the Matlab function as well! In addition, may I know that how do you get the 3D NME for 3DDFA? since isn't 3DDFA's ground truth is generated by the 3DMM fitting? Thanks!

Hi!
Yes! I am using the open3D python binding for icp. Yes, I figured out a way to align them better. Also, thank you for the heads up on the Matlab function as well! In addition, may I know that how do you get the 3D NME for 3DDFA on the AFLW2000-3D dataset? since isn't AFLW2000-3D's ground truth is generated by the 3DDFA fitting itself? Thanks!

Hi @Jim61C , I had a question regarding computing NME. When the output 3D mesh of an algorithm has for example 43k vertices (for the case of PRNet, how can we compare this with the groundtruth mesh which has for example 53k vertices(for the case of AFLW2000 dataset).
Would you please share the NME computing code you used based on above discussion?
Thank you.


fix: removed README image from bundle

Saves 45KBytes of bloat ;-)

@ColinEberhardt Thanks!! However can we merge this to develop. I'm aware its just the readme, but we don't want the community of getting into the habit of sending PR's to master

However can we merge this to develop. I'm aware its just the readme, but we don't want the community of getting into the habit of sending PR's to master

If you want people to PR against develop then perhaps you should set this as the default branch?

The Readme/API on develop won't necessarily reflects the module released on NPM


Chrome Refresh UI Not working in M124.0.6367.218 - 40th Release

System Details

OS: Windows 11
Thorium Version M124.0.6367.218

Problem
Chrome Refresh UI Not working despite being enabled in chrome://flags and in the labs icon in the top right.
Screenshots
Additional Notes
This could also just be that the UI has changed to be much more compact and alike the beforehand; however, I don't think this is an intended feature.
Thanks!

System Details
OS: Windows 11
Thorium Version M124.0.6367.218
Side panel completely bugged and broken with red on top of the Chrome Refresh not working

System Details
OS: Windows 11 Thorium Version M124.0.6367.218
Side panel completely bugged and broken with red on top of the Chrome Refresh not working

Thank you for this addition. I noticed this as well and stupidly forgot to add it! You're awesome, thank you!

I have the same issue in Linux. Web Refresh is not working.

--disable-features=CustomizeChromeSidePanelNoChromeRefresh2023
You will need to specify it manually on the commandline, either from the terminal, or to make it permanent, by editing the shortcut (on Windows), or editing /usr/bin/thorium-browser (on Linux). Next release will fix this.

--disable-features=CustomizeChromeSidePanelNoChromeRefresh2023
You will need to specify it manually on the commandline, either from the terminal, or to make it permanent, by editing the shortcut (on Windows), or editing /usr/bin/thorium-browser (on Linux). Next release will fix this.

Can confirm this workaround works.

i got the same issuee

just asking when is it going to be fixed?

just asking, when is it going to be fixed?

atp who knows. he used to update it like biweekly at least to update the chromium version but it hasn't been updated in quite a bit. For now, It's probably better to switch to a different browser that has the latest chromium versions or at least updates it more often. When an update rolls out I"ll be coming back but for now I dislike all of the bugs present and the outdated chromium version (which effects extensions like 1password)


Trace Ids not sent in the response for Spring 1.5.17 and Spring Cloud Edgware.SR5

I've recently migrated my application from Spring Boot 1.4.3.RELEASE to Spring Boot 1.5.17.RELEASE
and Spring Cloud from Brixton.SR4 to Edgware.SR5.
After migration, trace and span ids are not being sent in the API response
(X-B3-Traceid and X-B3-Spanid)
I can see that they are generated in the logs.

Yes, that feature was removed in the meantime. You have to add those values yourself. One of the corresponding issues is https://github.com/spring-cloud/spring-cloud-sleuth/issues/633


No documentation for ajax operators?

RxJS version: 5.4.0
Code to reproduce: N/A
Expected behavior: Looking at the documentation available here I would expect to find a reference for the ajax operators.
Actual behavior: As far as I can tell there are no reference docs. There are some on the classes but not the operators like switchMap for example.

dupe of https://github.com/ReactiveX/rxjs/issues/2656


FoundationEssentialsTests: repair the test suite on Windows

We would use the Unix (non-portable) arc separator (/). On Windows, we need to ensure that we account for the proper arc separator (\) when checking the FSR of the string.

@swift-ci please test


All stories displayed on a single page (kitchen sink)

It would be nice if storybooks could run in a monorepo and generate a storybook for each package in the monorepo. It would also be nice if you had a kitchen sink that would show an example of all the components in the package in one spot rather than only seeing one at a time similar to how it's done in React Styleguide Generator Alt. This would in effect create a kitchen sink of all the components available in the mono repo. Here's a UI mock up of what this might look like:

I don't think you need anything special to make this run in a monorep, just need to use the right require.context.
I've set this up for my monorepo:

I think she's asking if there was a way to display all your stories in a single place, kind of like how bootstrap and foundation's kitchen sink looks like. Instead of clicking through to each component's story, you see everything together.

This asks for 2 things in 1 issue, and I think @novascreen answered the first:
Storybook has no concepts op repos, it looks at your filesystem, using webpack. So a monorepo can be traversed easily, all be it semi-manually. We currently have no autoloading of stories. But this is being discussed.
Secondly: The question of a "show me everything you've got" page. Thanks for the suggestion! I think this is something we should do some brainstorming on how to do a good UX for that.

I think this is something we should do some brainstorming on how to do a good UX for that.

Ideally this would be achievable via a plugin, although I don't think it is right now?
The tricky part is figuring out how to display stories side-by-side in a sensible fashion--does anyone have any ideas about how you could do that? I mean I guess you could literally just do:
<div>
  <Component1 {...story1}/>
  <Component1 {...story2}/>
</div>

But would that actually render properly in all cases?

If an addon could wrap storybook components... and thus add styling and or logic...
It could be conceivable that such an addon would completely replace the iframe component with something of their own..
If we provide a method for getting all the stories (in hierarchy) then such an addon should have no problem displaying everything on one page.

I think this can be achieved by using the addon very similar to chapters.
In this case API could look like this:
storiesOf('Buttons Package')
  .addStyleGuide('Default button', guide => guide
        .add('with text', fn(1))
        .add('with icons', fn(2))
  )

where '.add' could be "normal" story or .addWithInfo
Looks like we can even add such API to chapters if there'll be an interest for that!
In fact, it is now already possible to add custom TOC to "chapter"
Looks really promising to continue this feature and implement something like #602

#1394 In progress here


Async entry read causes exception when reading AES encrypted zip.

Describe the bug
When trying to read an encrypted archive's entry's input stream asynchronously, it reports an exception containing message: "Auth code missing from input stream".
Yet, if the same archive is read synchronously it works fine.
Reproduction Code
https://dotnetfiddle.net/RhJM7o
Steps to reproduce
It doesn't seem to happen for every input. But for inputs where it does happen, changing the async call to the synchronous version fixes the problem.
The zip input is in the dotnetfiddle.
The steps to create the zip archive from 7-Zip are:

Using the file provided (appendix A)
7-Zip -> Add to archive
Ensure params:

Archive Format: Zip
Compression level: 5 - Normal
Compression method: * Deflate
Encryption method: AES-256
Set password (I set mine to "test")

Run the decompress/decrypt steps outlined in the dotnetfiddle

Expected behavior
For the same input, I expected them both to work (the synchronous and the asynchronous).
Operating System
Windows
Framework Version
.NET 6
Tags
ZIP, Encryption, Async
Additional context
No response

Thanks for the report and for actually using the dotnet fiddle template!
I compacted it somewhat to emphasise that the only difference is the async/sync calls: https://dotnetfiddle.net/5alzjK
I have not found I cause yet, but I will look into this as soon I can.

Yeah, this is yet another time that the framework bypasses the public methods of CryptoStream for increased performance.
ZipAESStream overrides the Read and ReadAsync methods of CryptoStream, but as you can see from the

Stacktrace
   at ICSharpCode.SharpZipLib.Encryption.ZipAESTransform.TransformFinalBlock(Byte[] inputBuffer, Int32 inputOffset, Int32 inputCount) in /_/src/ICSharpCode.SharpZipLib/Encryption/ZipAESTransform.cs:line 141
   at System.Security.Cryptography.CryptoStream.ReadAsyncCore(Memory`1 buffer, CancellationToken cancellationToken, Boolean useAsync)
   at System.Security.Cryptography.CryptoStream.ReadAsyncInternal(Memory`1 buffer, CancellationToken cancellationToken)
   at System.IO.StreamReader.ReadBufferAsync(CancellationToken cancellationToken)
   at System.IO.StreamReader.ReadToEndAsyncInternal()
   at Program.<<Main>$>g__Repro|0_0(String type)
   at Program.<<Main>$>g__TestAsync|0_2[T](Func`2 a, T input)

, StreamReader just bypasses ReadAsync and calls ReadAsyncCore directly, which bypasses our read logic. That method is internal and cannot be override by us.

As for consumers (including @rilehudd), you can replace the StreamReader with:
var e2 = zip.GetEntry("abcdefgh.txt");
var entryStream = zip.GetInputStream(e2);
var entryBuffer = new byte[e2.Size];
using (var ms2 = new MemoryStream(entryBuffer))
{
	byte[] buffer = new byte[ushort.MaxValue];
  	int read;

	if (type == "async")
    {

		while ((read = await entryStream.ReadAsync(buffer, 0, buffer.Length)) > 0)
		{
			ms2.Write(buffer, 0, read);
		}

	}
    else
    {
		while ((read = entryStream.Read(buffer, 0, buffer.Length)) > 0)
		{
			ms2.Write(buffer, 0, read);
		}
	}
	var output = Encoding.UTF8.GetString(entryBuffer);

      // ... use output ...
}

It's not ideal, for sure, but that is the only async path that avoids the optimisations in .NET that bypasses the AES encryption logic.

Is there a way we can detect this "bypassed" state an compensate for this optimization?
For example detect that ZipAESTransform.TransformFinalBlock has been called from the non-expected path (to implement what CryptoStream is expecting). I'm may not be understanding the feasibility of attempting to compensate for the optimization. Or maybe it is possible but would result in unacceptable complexity. (I wonder what dotnet designers expected implementors to do with them bypassing public methods).
Otherwise, it seems like this might be an optimization that might likely affect many encryption stream implementations. Do you think it makes sense to submit this optimization bug upstream to dotnet?

Thanks for taking the time to take a look at this!

There is another solution to this. The compression method that is used for the entry is STORE, which means that it's uncompressed. This problem only happens to such entries because we directly return the ZipAESStream to the consumer. If there is some decompression to be done, the stream will be wrapped in the appropriate decompression stream, and the optimized path is not taken. We could add a StoreOutputStream that just passes the data through, but in doing so mitigates this problem.


Enable wild cards for scp file transfer?

Hi,
Would it be possible/easy to allow the use of wildcards for only uploading specific files from a folder and not only the full folder (and hence avoid user to create specific folders to import on the server).
Thanks
Best
Franck

The code will automatically download all valid formats from a given folder so no need for wildcards or specific files. Note that the instructions have the valid formats and ask to have a single folder for all the valid formats:


bug: use of keyword INVALID

In master, there seems to be an error in graphql-api-generator/utils/utils.py. On line 983 there is a reference to a keyword/variable INVALID that is not defined, which causes issues during the build process.
if arg.default_value != INVALID:
   # [...]

@keski I seem unable to replicate any errors related to this specific line. For me arguments without default values is set to INVALID as default and the comparison behaves as expected.
INVALID is an actual graphql type and should not cause any issues.
A type(INVALID) call gives, and should give <class 'graphql.error.invalid.InvalidType'>.
Is there any differences in the version/configuration of the graphql we are using?

I see, I'll try updating my GraphQL lib and report back.

It seems that INVALID has been deprecated in favor of Undefined graphql-core/issues/77. Apparently I had already bumped up my lib to 3.3.1, and INVALID is apparently not listed in the modules __all__ anymore and thus has to be imported explicitly.

As rcapshaw just released the new release. I suggest we update our requirements to be >= 3.1.1 and that I fix this properly for the newer versions then.

I corrected this in de8742a but kept the backwards compatibility with version of grapql-core <3.1.1.
If we want to enforce version 3.1.1, or discuss it, I suggest opening a new issue as I'm closing this one now.


0.58.0 release

Remove bbox filter field
Exclude changesets newer than 5 minutes from query results
Change position of status notification
Show untrusted teams and add string to distinct the trusted ones on the filters page

Exclude changesets newer than 5 minutes from query results

Why isn't the fix in changeset-map deployed to production instead?

@mmd-osm I'm having some issues with changeset-map, caused probably by some outdated dependencies, so I'm not succeeding on building it and the installation of newer versions on osmcha-frontend is failing. I asked @kepta to help me with it, but he is very busy this week. I hope next week we can have some news on it.

@mmd-osm good news! @batpad gave me a solution to that problem. I'll test it later and update changeset-map on this release

@mmd-osm I finally could push to production the updates on changeset-map regarding the overpass api. Thank you!


Hi @a-priestley I'm checking into this with my team.

I am facing same issue with one of account
We can't continue with the signup process for the following reason:
Your phone number did not pass security checks.
MS-CorrelationId: 00000000-98dd-4f28-b972-65eb19313921
Code: 40315
I tried with 4 different numbers but it's still showing the same error "Your phone number did not pass security checks".

Above is the error please check and provide the solution for that

i'm facing the same issue. any solution?????....why is it says that even if we use our personal phone number

Hi, sorry to hear you were having issues and hope they're resolved now! I'm closing this issue; if you still need assistance, contact<EMAIL_ADDRESS>As you can see, this repo isn't monitored regularly. We're working on that!


Refactor TextEditor updating

Description of the change
This PR refactors the codes that were responsible for updating different parts of TextEditor out of update function, and, then, it uses those refactored functions directly, which bypasses the loop and switch case inside update.
finishUpdate is also refactored that allows finishing updating of TextEditor from inside themselves without the need for creating objects outside and passing them in.
Benefits

This reduces the complexity of the update algorithm for those that are updated directly (O(n^2)->O(1)). This removes the overhead of checking for the given value against the keys in switch-case, and it also removes the overhead of the for-loop. Directly calling these functions is similar to the compile-time method dispatch used in other languages like C++. The previous method was a runtime dispatch algorithm that does not allow the V8 engine to optimize these calls and indirections.

Having these functions separate makes it easier to maintain, find, and reason about them.

Verification
I have moved the code inside update to a separate function as it is (exact copy-paste), and I have not changed "any part of the actual code". If you follow my commits one by one, you will see that the moved code is kept exactly identical.
All the tests must pass because this does not change the functionality.
How to review
Follow my commits one by one.
Release Notes

Refactor the codes that were responsible for updating different parts of TextEditor

This looks pretty neat. 👍

You're welcome!

This looks pretty neat. 👍

You're welcome!


Complex subscript expressions

As of now, we don't seem to support something like this: array[2 + 2].
Look for the number 230 in the code to find clues to implement this feature.

Hi! I'd like to contribute but looks like i don't understand the essense of the problem. I use the last build of Cesium and code like this works without expressions.
int main()
{
    int array[6];

    array[1 + 1] = 10;

    return array[1 + 1];
}

I may have been wrong in formulating the issue. See this code: https://github.com/ForNeVeR/Cesium/blob/677417723732865ecd848481a5a07eb142cf9461/Cesium.CodeGen/Ir/Expressions/SubscriptingExpression.cs#L35-L41
Looks like it expects the subscript base (the a in a[10]) to be simple, and not the index. I'll update the issue description.

As of now, we don't seem to support something like this:
typedef struct { int[10] x; } foo;
int main() {
  foo a;
  a.x[0] = 0;
}

Look for the number `230` in the code to find clues to implement this feature.

if (_expression is not IdentifierExpression identifier)
         throw new WipException(230, "Subscription supported only for IdentifierConstantExpression");

So in that case _expression would be PointerMemberAccessExpression and we need to get the _memberIdentifier of pointer?

Generally, the target expression should just emit something indexable (such as: return an IValue or, dunno, something of kind IIndexableValue, if we think it's required), and the SunscrptingExpression should perform an index operation (say, some address operation). It shouldn't depend on the target expression type.


add overflow hidden to dropdown menu

This fixes the issue that when a dropdown has no vertical margins ($dropdown-padding-y: 0), the hover state has no rounded corners.

Then anything with a tooltip or popover would be hidden. Sorry, not the right fix for this. I'd recommend adding the utilities or custom CSS to accomplish this on your end.


Added missing types into token handler

Description
Added missing types into token handler
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[x] Minor

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

/retest

/retest

/retest

/lgtm

/approve

/retest

/retest

/lgtm

/lgtm

/lgtm
/approve


Update auto release script

Feature Request
Is your feature request related to a problem? Please describe:
PD no longer accepts PRs that do not ref issues in the commit message.

Describe the feature you'd like:
We could modify the release script to include ref #4257 in the commit message. This makes it clear that what problem does the PR solve.
As an example, https://github.com/tikv/pd/pull/4253 shows that this kind of message works.

Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

Resolved by https://github.com/pingcap/tidb-dashboard/pull/1054, https://github.com/pingcap/tidb-dashboard/pull/1133.


fix: memberOf should be spec.memberOf for User Entities

When following the Manual admin group configuration with group entities (advanced alternative) guide, the yaml snippet for the example User Entity has memberOf at the root —it should be at spec.memberOf instead.

Tackled in #688


Source code for supplier leaflets in all languages

@hfukuchi do you have the source code for the Supplier Leaflet in Chinese Simp/Trad? I will also try to locate the current source code for JP/EN

OK, I got the source code, latest I have for Chinese S+T is October 3rd. Do you have anything newer?


Regression with optional properties in 5.2.0

Hey folks,
I've noticed a regression with regard to Optional properties as of runtypes 5.2.0. Specifically, this issue: https://github.com/pelotom/runtypes/issues/60
Before 5.2.0, we were able to specify a property with .Or(Undefined) and if the property was missing, it would still pass the type checks.
const myType = Record({
   foo: String,
   bar: String.Or(Undefined)
});

myType.check({ foo: 'abcd' }); // <-- This would pass.

However, after 5.2.0, the above code would throw a validation error.
I'm aware of the workaround (using .And(Partial({ bar: String }))).
If this regression is intentional, then I think the documentation needs to be updated: https://github.com/pelotom/runtypes#optional-values

This is intentional, see #206. We're sorry to surprise you with this breaking change. You're right, I'm going to append a caution about this to the readme.
Note that there exists one more handy workaround now: Optional(String).

Ah, I only looked at "open" issues so I missed that one. Thanks for the
quick answer!
On Wed, Apr 7, 2021, 19:19 Yu Shimura @.***> wrote:

This is intentional, see #206
https://github.com/pelotom/runtypes/issues/206. We're sorry to surprise
you with this breaking change. You're right, I'm going to append a caution
about this to the readme.
Note that there exists one more handy workaround now: Optional(String).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pelotom/runtypes/issues/219#issuecomment-815396740,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAANVRCPNJLOYEHWA7BJLPDTHUHDBANCNFSM42RHGE6Q
.

Description added in the section. Thanks!


avoid calls to bind() when creating texture

Better just the already existing unsafe methods (see #558)

It's a 2-lines change with the existing unsafe methods.


Release artifacts for M66.1

Note that we released Firestore 1.11.2 on top of release-6.19. Should release 6.20 include that?

Yep the Firebase.podspec should be updated to include 1.11.2

OK. I created https://github.com/firebase/firebase-ios-sdk/pull/5091 to merge that to master.

Thanks for doing that - I can cherry-pick now!


Style updates for tour page

A few css changes to better format the tour page

Great, thank you!


Add GraphQL Pagination Support

User Story
As Gary the GraphQL Power User…,
I want to get data from a GraphQL with pagination for larger queries...,
So that timeouts of the servers are not approached …
I will know this is done when GraphQL queries support pagination with the following showing up in the query support.
(limit: 10, offset: 0)

Database Changes (Optional)
None
External Dependencies (Optional)
Graphene update?

Food for thought:

https://graphql.org/learn/pagination/
https://github.com/instruct-br/graphene-django-pagination
https://github.com/eamigo86/graphene-django-extras

Merged into next.


fix _expanded to work

Closes #2057
Description
fix _expanded to work

@cidkumagai @108yen
Hi, thanks for the PR.
You removed aria-expanded and now you want to reinstate it?
Shouldn't we use data-expanded?

@hirotomoyamada
Yes, I agree. We should use data-expanded. I will make the necessary changes.
Thank you.

@hirotomoyamada
I have addressed the comments you provided and made the necessary corrections. Could you please review it again?


method to check valid cards from hand based on selected cards from table in client

closed in https://github.com/sopra-fs23-group-37/client/commit/830e1676b3c94abf9dc5ff3e946a5e4d36d5623c


nimbus: init at 23.2.0

Description of changes
Nimbus is a client for the Ethereum consensus layer that is lightweight, secure and easy to use.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I have tested this using the following config:
{
  services.geth = {
    mainnet = {
      enable = true;
      authrpc = {
        enable = true;
        port = 8551;
        vhosts = ["localhost"];
        jwtsecret = "/etc/geth/jwtsecret";
      };
    };
  };

  services.nimbus-beacon-node = {
    enable = true;
    web3Urls = ["http://localhost:8551"];
    jwtSecret = "/etc/geth/jwtsecret";
    rest.enable = true;
  };

  environment.etc."geth/jwtsecret" = {
    mode = "0444";
    text = "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa";
  };

  services.getty.autologinUser = "root";

  systemd.services.geth-mainnet.serviceConfig.StandardOutput = "journal+console";
  systemd.services.nimbus-beacon-node.serviceConfig.StandardOutput = "journal+console";
}

And ran it using:
nixos-generate -f vm-nogui -c $PWD/configuration.nix -I nixpkgs=$HOME/work/nixpkgs

Which worked without issues.

Thanks for the feedback @aanderse, I didn't know types.path checks for null and that types.enum exists, that's much cleaner than what I was doing.
I've addressed most point aside from the user creation, which I think is necessary since the user has to manage their validator files permissions and for that a static user is necessary, so we can't use the DynamicUser option like some services.

Sorry if you're not interested by I'm desperate for reviews, and you've worked on the Geth package and service.
I'd appreciate some feedback from @pennae @ncfavier @iceman-p @bachp @ryantm.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1512

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1910


ui: copy original content instead of formatted content for CopyLink

fix #798 in step 1

According to the comment in https://github.com/pingcap/tidb-dashboard/issues/798#issuecomment-731554967 , we can supply "Copy Original" and "Copy Formatted" buttons both.
The effects:

/merge


completely blank page

Hello, I'm trying to run with the following code and I only see a blank page when browsing to the url.

import swaggerJSDoc from 'swagger-jsdoc'
import swaggerUi from 'swagger-ui-restify'

function applySwagger (app) {
  const swaggerDefinition = {
    info: {
      title: 'Topix API',
      version: '0.0.1',
      description: 'This is the API documentation for the Topix API.',
    },
    host: 'localhost:3000',
    basePath: '/',
    securityDefitions: {
      bearerAuth: {
        type: 'apiKey',
        name: 'Authorization',
        scheme: 'bearer',
        in: 'header'
      }
    }
  }

  const options = {
    swaggerDefinition,
    apis: [
      './api/v1/auth.js',
      './api/v1/post.js',
      './api/v1/user.js',
    ],
    explorer: true,
    baseUrl: 'docs'
  }

  const swaggerSpec = swaggerJSDoc(options)
  app.get('/\/docs\/+.*/', ...swaggerUi.serve)
  app.get('/docs', swaggerUi.setup(swaggerSpec));
};

module.exports = applySwagger

There are no exceptions / logging printed.  Do you see anything wrong with my configuration here?

The path in app.get('/\/docs\/+.*/', ...swaggerUi.serve) isn't correct, but that's a documentation problem, not your problem. After going to http://yourhost/docs, the page tries to get the following files from the server:
<link rel="stylesheet" type="text/css" href="{BASEURL}/swagger-ui.css" >
<script src="{BASEURL}/swagger-ui-bundle.js"> </script>
<script src="{BASEURL}/swagger-ui-standalone-preset.js"> </script>
<script src="{BASEURL}/swagger-ui-init.js"> </script>

where {BASEURL} is ., which becomes your host name. The app.get('/\/docs\/+.*/', ...swaggerUi.serve) is meant to handle this, but the path is wrong. Instead, you can do
app.get('/swagger-ui-init.js', ...swaggerUi.serve);
app.get('/swagger-ui.css', ...swaggerUi.serve);
app.get('/swagger-ui-bundle.js', ...swaggerUi.serve);
app.get('/swagger-ui-standalone-preset.js', ...swaggerUi.serve);

or, if you don't mind all otherwise unmatched URLs being processed by swaggerUi.serve, you can do this:
app.get('/*', ...swaggerUi.serve);

Updated fix: https://github.com/bvanderlaan/swagger-ui-restify/issues/6#issuecomment-624268765


Add example of publishing PyPI package to Local Python repository

An example of how to make PyPI packages available from an offline Package Manager instance, until Package Manager supports air-gapped/offline PyPI repositories.
Adds an example bash script that will:

Download a specified package source tarball from PyPI and its associated binary wheels.
Upload those package files to a local Package Manager instance using twine

Looks reasonable to me one the issues Jacob brought up are addressed.

What is the likelihood of an air-gapped customer being able to use this? Would they have pypi.org blocked on their network, making them unable to pull down packages like this?

That's a good question, but technically you could run this from some machine in a DMZ that does have access to PyPI, but also has access to the internal PPM and can upload the packages to there, I guess.  You could also adapt the example to separately pull the packages you want then copy them manually and push them.  It's a starting point at least!


Set max line length to 100 characters

Previously we had been setting rustfmt and black to use an 80 character
line length. This was primarily because my personal dev workflow is
setup around 80 character-wide windows. However, as the project has a
growing number of contributors my personal archaic taste in dev workflow
should not be subjected upon everyone. This commit bumps the max line
length in rustfmt and black up to 100 which is a more common default
that will work better in most editors and not needlessly wrap lines
everywhere. Most of this diff is just the auto-formatting adjustments
now that there are an additional 20 characters available for them to
use.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

417 of 422 (98.82%)  changed or added relevant lines in 34 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.04%) to 98.413%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/generators.rs
46
47
97.87%

src/graph.rs
39
41
95.12%

src/isomorphism/vf2.rs
44
46
95.65%

Files with Coverage Reduction
New Missed Lines
%

src/graph.rs
1
98.51%

src/isomorphism/vf2.rs
1
96.26%

src/steiner_tree.rs
1
97.57%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
9672

Relevant Lines:
9828

💛  - Coveralls

I like this, I think it is reasonable to bump the line lenght to 100. Our Rust code is shorter and I don't think the readability is affected.
Also, we could also profit from this PR to update black to 21.11b1

Given that this will cause a merge conflict for everyone with an open PR, I'd like to merge more of our open reviews first. We can merge this right before we're ready to release 0.11.0 to minimize the pain this causes.


docs.rs iceoryx2 build fails since libacl.h is missing

Required information
See build log: https://docs.rs/crate/iceoryx2/0.0.4/builds/1061128
In the default environment in docs.rs the libacl.h is not available and therefore the build fails. Either we need to install it, provide a stub implementation of the interface just for docs.rs or we deactivate all acl abstractions in docs.rs.
I would favor a solution where we make libacl.h somehow available, even when it is just a stub.
The first affected crate is iceoryx2_bb_posix and then every crate that depends on it.

@elBoberido do you have any thoughts since you had to tackle a similar issue in iceoryx-rs?

I had similar problems when building iceoryx-sys since I fetched iceory via git. For the next release of the bindings I wanted to bundle the iceoryx source but did not make a release yet. I guess I would have the same problem.

I'll create a PR to add libacl1-dev to https://github.com/rust-lang/crates-build-env/blob/master/linux/packages.txt


Admin is not receiving mail for customer registration, after enabling the feature from Notifications in Configure Section - 8893

Issue Reference

#8893
Description

Registration emails are now being sent when the setting is enabled in the backend.
To send registration emails to customers, please enable the 'Send a confirmation e-mail after customer registration' option.
To send registration emails to the admin, please enable the 'Send a confirmation e-mail to admin after customer registration' option.

How To Test This?

Navigate to the Admin panel, then go to Configure, Notifications, and select the third option: "Send a confirmation email to admin after customer registration."
Enable this feature to send customer registration emails to the admin.
And select the second option: "Send a confirmation email after customer registration."
Enable this feature to send customer registration emails to the new customer.
Return to the shop, create a new customer account, and check email of new customer and admin for the confirmation mail.

Documentation

[ ] My pull request requires an update on the documentation repository.

@sunarcrajneesh It is working fine. Here

Are there any change requests on this PR and can you please let us know when we can expect these changes to be merged?

@shivendra-webkul I have checked the same issue and it is fixed now, please review the code. For reference I am adding a video for the same.
Video
https://github.com/bagisto/bagisto/assets/92721837/4346e68f-36d5-45e8-83a5-333d9243557a

@amit-webkul,
There is no need for local or currency information in the customer registration email, and it is also not utilized in the Bagisto shop, could you please explain the rationale behind using the $this->prepareMail function?


#printf docs missing

The API doc of both Kernel#printf and #sprintf refer to IO#printf for documentation. But the two overloads there are actually undocumented (first being empty, second saying ditto 😆).
(Maybe unrelated, but in the source I see that String::Formatter is used, but for some reason that class is hidden from the API doc. Maybe that could be a good place to put the documentation?)

#triage
This issue has languished, and it appears that attempts were made to resolve, but eventually got closed with no true resolution.
The methods remain undocumented:
https://crystal-lang.org/api/0.21.0/IO.html#printf(format_string%2Cargs%3AArray|Tuple)%3ANil-instance-method
https://github.com/crystal-lang/crystal/blob/c2c2276ec667171af5c2347490f49acdbdc4a3ca/src/io.cr#L352-L360
What can we do to push this forward? This discussion appears to point at the Ruby docs for sprintf.
Should/can we copy the style (tables, formatting) and get that in here? @asterite You seemed to have the stronger opinion here.

I would document the IO#printf method, not the private formatter class (it's an implementation detail). I would document what is actually supported as a simple list of codes and expected types, with a few examples. Something very basic at first. Maybe see the Ruby docs and the printf man page for reference.

What's the status on this? I would like to contribute and this seems like a good starting place if nobody is taking this on. Let me know if you all would like me to start working on it.

@WazzaSoft Still up for this?

@straight-shoota Yes, I am.

@WazzaSoft go right ahead, no need to ask before making a PR.

I did not see any PR regarding this issue so I have created one.
https://github.com/crystal-lang/crystal/pull/5762
It follows both @ysbaddaden and @asterite suggestions about how to document this method.
Please take a look when you have the time and let me know if there is anything else you would me to change.


@all-contributors please add @viewv for develop

@all-contributors please add @viewv for develop

@all-contributors please add @viewv for develop

@all-contributors please add @viewv for code

@all-contributors please add @viewv for code

@all-contributors please add @nekomiao123  for code

@all-contributors please add @nekomiao123  for code

@all-contributors please add @viewv for ideas

@all-contributors please add @viewv for ideas

@all-contributors please add @Singularity0909  for code

@all-contributors please add @Singularity0909  for code

@all-contributors please add @superzhaoyang for code


Verify that ranges/slices are still accurate

Before merging in mainline decaf, I recall seeing some off-by-one changes in transformed ranges.  I'm not sure whether before/after was correct, so I should check that, eh?

Do you mean that decaf is creating off-by-one ranges, or that our transform is messing things up?

Decaf.

#170 contains a few edge cases that'll have to be fixed.


JSDoc sees type as never when marking some properties as required

Hi,
I'm trying to use the library with JSDoc and it almost works :D.
See this screenshot (TS to the left, JS with JSDoc to the right):

Here's a gist with both files.
The definitions are very similar, the first ones don't specify required and the second ones do.
The inferred schema for the version without required seems to be equivalent when applied to a value. However, when the version that has required is used valueRequired2 in JS, the TypeScript extension in VSCode fails with the following error:
Type '{ input: { x: number; }; }' is not assignable to type 'never'.ts(2322)
const valueRequired2: never
@type — {import('json-schema-to-ts').FromSchema}

This is all with TypeScript 5.3.3, forcing VSCode to use the version of TS from node_modules like this in .vscode/settings.json:
{
  "typescript.tsdk": "node_modules/typescript/lib",
  "typescript.enablePromptUseWorkspaceTsdk": true
}

This is the current jsconfig.json:
{
  "compilerOptions": {
    "checkJs": true,
    "skipLibCheck": true,
    "strict": true,
    "noStrictGenericChecks": false,
    "moduleResolution": "node"
  },
  "exclude": ["node_modules"]
}

I tried toggling both strict and noStrictGenericChecks on and off as suggested in the readme but it makes no difference.
I understand this might not be a problem with the library itself but more so with the combination of JSDoc and everything else. Figured I'd ask anyway if you may have any clues as to what could be going on?
Thanks!

Hi @dariocravero and thanks for the issue, I never tried using FromSchema in JSDocs before.
My guess is that it comes from the as const statement missing. The @satisfies seems to partially narrow the type of your schemas, which is already great (so "object" is kept as "object" and not widened as string), but it doesn't do the trick for the required keyword which leads to FromSchema being unable to infer required properties.
Can you add some kind of /** @as const **/ ?

Awesome! Thanks!


python312Packages.tencentcloud-sdk-python: 3.0.1148 -> 3.0.1149

Diff: https://github.com/TencentCloud/tencentcloud-sdk-python/compare/refs/tags/3.0.1148...3.0.1149
Changelog: https://github.com/TencentCloud/tencentcloud-sdk-python/blob/3.0.1149/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 312384 run on x86_64-linux 1

4 packages built:

python311Packages.tencentcloud-sdk-python
python311Packages.tencentcloud-sdk-python.dist
python312Packages.tencentcloud-sdk-python
python312Packages.tencentcloud-sdk-python.dist


Tooltips misplaced in tables

Hi,
I was working on a personnal project and just noticed that tooltips seems to be misplaced for the last line of the tables. Here's a codepen with the issue :
https://codepen.io/Fankohr/pen/aYKJLa
Thanks and sorry for my english !

Fixed in c00c954f


Failed to check accuracy ssd-vgg16. Should use other model for testing SSD

https://aipg-rancher.intel.com/jenkins/algo-lr/job/ngraph-mxnet-run-accuracy-inference-models/153/console reported:
00:04:09.860366 nGraph: network:ssd-vgg16, type:inference, batch_size:1, accuracy:fail
I have coco data in /dataset/coco/

This is not urgent. However, I would like to sort it out this week.
Should I use ssd_300_vgg16_atrous_voc instead of ssd-vgg16 ?

Yes, ssd-vgg16 by itself doesn't have a pretrained version, so real data inference testing is out the question. ssd_300_vgg16_atrous_voc is a close replacement we can test with real data.


Old command for RG creation won't work.

I fixed the command for creating resource groups in Azure CLI. The previous commands didn't work, I got series of errors when I ran the code, fixing it so it helps someone else later.❤

@mumian
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Tonyette - thank you for your comments.  Both options work.  See https://learn.microsoft.com/cli/azure/group?view=azure-cli-latest#az-group-create.  What are the errors?

#please-close


⚠ Processors has degraded performance

In b8ad64c, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 60 ms

Resolved: Processors performance has improved in 8fc0769.


Migrate repository

@philippfrank I am not sure about the license which still says 2016 Propertybase GmbH. Otherwise this should be good to go. I manually deployed the storybook to https://react-lds.github.io/react-lds

@quadreex @PeterSiegl I added you to contributors in package.json. Please have a look and let me know if you object to that.

I amended the license as suggested here. Lacking a better idea, I added myself as holder. I'm happy to amend change that, let me know if you see an issue with the decision. I also added a note about original maintainership to the readme.


Obsoletion notice for children of sphingosine-1-phosphate signaling pathway

Hello,
The proposal has been made to obsolete

GO:0003377 regulation of apoptosis by sphingosine-1-phosphate signaling pathway
GO:0003378 regulation of inflammatory response by sphingosine-1-phosphate signaling pathway

The reason for obsoletion is that these terms represent GO-CAM models.
There are no annotations, no mappings to this term.
Any comments can be added to the issue: https://github.com/geneontology/go-ontology/issues/15995
We are opening a comment period for this proposed obsoletion. We’d like to proceed and obsolete this term on July 12, 2018. Unless objections are received by July 12, 2018, we will assume that you agree to this change.
Best regards, Pascale

Note_: this follows from the ticket #15822


Save game to file

It should be possible to save games in at least one file format - e.g. T1j like format

Done. Created pull request with this functionality.

Works really nice! Thanks for the implementation


Add staclint to validation docs

Added a blurb about staclint and removed the sentence about remote schemas since the validator will take care of this for the user.

Not sure about this PR, but the "Files changed" tab only shows me changes in the .gitignore?


Resource.readAsString is doing something wrong with encoding.

I suspect this may be a combination of the webserver doing something strange and Resource being strange, but I lost a couple hours tracking this down this afternoon. :/
package:http reads the json file no problem, package:resource encodes it wrong somehow.
This is likely the problem code:
https://github.com/dart-lang/resource/blob/master/lib/src/io_io.dart#L64
Repro:
import 'package:http/http.dart' as http;
import 'package:resource/resource.dart';

void main() async {
  String url =
      'http://ddragon.leagueoflegends.com/cdn/7.2.1/data/en_US/champion.json';
  String fromHttp = await http.read(url);
  String fromResource = await new Resource(url).readAsString();
  print('${fromHttp.length} ${fromResource.length}');
}

Output:
127543 255086

JSON.decode, etc. fails for package:resource, but succeeds in the package:http case.  Presumably package:resource should just use package:http instead of rolling its own?

Those numbers do look suspiciously like they differ by a factor of 2.
The simplest explanation is that the content is being sent as UTF-16, and the resource package doesn't recognize that. Another option is that the request isn't asking for the correct encoding.
Package resource is rolling its own HTTP here. Using package:http is an option. Currently the resource package is using the experimental configurable imports, and the http package isn't.


gpstracker database not updating with new info (iis 8/dotnet/ssms 2012)

My setup was working for couple of days but now the db is not updating with new gps data.  I checked the IIS logs and GPS information is being logged so i know the phone is reaching the server.
I'm also able to DELETE old routes via the main page (displaymap.aspx).  not sure what else to check, any help would be greatly appreciated.
Things i have tried......rebooted the server, restored the gpstracker database, replaced all the files in (except for web config) C:\inetpub\wwwroot\gpstracker folder in case I modified any on accident.

Resolved.  Make sure your phone is pointing to http://yourserver.com/gpstracker/UpdateLocation.aspx
and not to http://yourserver.com/gpstracker/displaymap.aspx


Podspec specifies minimum deployment target 8.0 but doc says 7.0

In the podspec file the minimum iOS deployment target is 8.0 but the documentation specifies that the minimum iOS deployment target required is 7.0.
Which one is correct and which one wrong?

Just realized that embedded frameworks (introduced in CocoaPods 0.36 for swift support) is only compatible with iOS 8+...


Fix: Avoid duplicate payment source creation

test: add test for shipping from primary
test: add key for ship from primary test
test: add check for shipping price and coupon
fix: avoid double payment intent creation
fix: use new component for stock transfer
test: add await for click evaluation on thank you page

Fixed thanks to a new version of react components lib


feat: screenshot whole page, hide certain elements, screenshot fixed size

does what it says on the tin

updated view


Allow using imageName as a selector

What type of PR is this:
/kind feature
/area devfile-spec
What does this PR do / why we need it:
This PR allows handling imageName field on Image components as a selector. This means that, for each Image component with a relative image name, odo will automatically build and push a fully qualified image (using a user-defined registry and a generated image tag), and will replace any matching image names found in any Container or Kubernetes/OpenShift components.
See https://github.com/devfile/api/issues/985#issuecomment-1404939472 for more context and details.
For this to work however, this PR exposes a new ImageRegistry preference, which users need to set first. And, for Kubernetes/OpenShift components, replacement is done for matching image names found in core Kubernetes resources (like Pods, Deployments, DaemonSets, ...).
Which issue(s) this PR fixes:
Fixes #6677
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[x] Documentation

https://deploy-preview-6768--odo-docusaurus-preview.netlify.app/docs/overview/configure#preference-key-table
https://deploy-preview-6768--odo-docusaurus-preview.netlify.app/docs/development/devfile#how-odo-handles-image-names

How to test changes / Special notes to the reviewer:

Set a global image registry in the preferences, e.g.: odo preference set ImageRegistry "quay.io/$USER"
At this point, odo will treat all relative image names in Devfile Image components as selectors, i.e.:

it will replace the value in all matching Container and resources in Kubernetes/OpenShift Components (for Kubernetes/OpenShift components, for now, this handles only images referenced in core Kubernetes resources: CronJob, Job, Pod, DaemonSet, Deployment, ReplicaSet, ReplicationController, StatefulSet)
Non-matching image names will not be updated.

See https://github.com/devfile/api/issues/985#issuecomment-1404939472 for examples of how image names are matched against selectors.
Note that this behavior applies to both odo dev  and odo deploy, and should work the same way on both Podman and cluster.

+ [FAILED] [21.463 seconds]
odo delete command tests [BeforeEach] when a component is bootstrapped using a devfile.yaml with URI-referenced Kubernetes components when the component is running in both DEV and DEPLOY mode and dev mode is killed when the component is deleted while having access to the devfile.yaml should delete the appropriate resources
  [BeforeEach] C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/integration/cmd_delete_test.go:25
  [It] C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/integration/cmd_delete_test.go:625

  Timeline >>
  Created dir: C:\Users\Administrator.ANSIBLE-TEST-VS\AppData\Local\Temp\1356430847
  Created dir: C:\Users\Administrator.ANSIBLE-TEST-VS\AppData\Local\Temp\4066833606
  Setting KUBECONFIG=C:\Users\Administrator.ANSIBLE-TEST-VS\AppData\Local\Temp\4066833606\config
  Creating a new project: cmd-delete-test625txw
  Running oc.exe with args [oc new-project cmd-delete-test625txw] and odo env: []
  [oc] Unable to connect to the server: dial tcp <IP_ADDRESS>:30329: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
  [FAILED] in [BeforeEach] - C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/helper/helper_cmd_wrapper.go:101 @ 05/22/23 06:06:25.182
  Setting current dir to:
  [FAILED] in [AfterEach] - C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/helper/helper_filesystem.go:90 @ 05/22/23 06:06:25.184
  << Timeline

...
  Running oc.exe with args [oc get pods --namespace cmd-dev-test2781rgp --selector=component=javaspringboot-vnnlcc -o jsonpath={.items[*].metadata.name}] and odo env: []
  [oc] javaspringboot-vnnlcc-app-6f985c85fb-rh4bfRunning oc.exe with args [oc exec javaspringboot-vnnlcc-app-6f985c85fb-rh4bf --namespace cmd-dev-test2781rgp -c runtime -- bash -c grep [s]pring-boot:run /proc/*/cmdline] and odo env: []
  [oc] Unable to connect to the server: dial tcp <IP_ADDRESS>:30329: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
  [FAILED] in [It] - C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/helper/helper_cmd_wrapper.go:101 @ 05/22/23 06:11:45.034
  [odo] Cleaning resources, please wait

...

+ [FAILED] [21.440 seconds]
odo remove binding command tests [BeforeEach] when the component with binding is bootstrapped (bindingName=my-nodejs-app-cluster-sample-ocp) when odo dev is running when binding is removed should have led odo dev to delete ServiceBinding from the cluster
  [BeforeEach] C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/integration/cmd_remove_binding_test.go:16
  [It] C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/integration/cmd_remove_binding_test.go:71

  Timeline >>
  Created dir: C:\Users\Administrator.ANSIBLE-TEST-VS\AppData\Local\Temp\2529226010
  Created dir: C:\Users\Administrator.ANSIBLE-TEST-VS\AppData\Local\Temp\2291510114
  Setting KUBECONFIG=C:\Users\Administrator.ANSIBLE-TEST-VS\AppData\Local\Temp\2291510114\config
  Creating a new project: cmd-remove-binding-test71fxi
  Running oc.exe with args [oc new-project cmd-remove-binding-test71fxi] and odo env: []
  [oc] Unable to connect to the server: dial tcp <IP_ADDRESS>:30329: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.

Network issues in 3 out of 4 failures.
The other one was about Persistent Volumes not being bound in a timely manner:

...
+ [FAILED] [549.472 seconds]
odo dev command tests when a component is bootstrapped when Automount volumes are present in the namespace when odo dev is executed [BeforeEach] should mount the volumes
  [BeforeEach] C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/integration/cmd_dev_test.go:667
  [It] C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/integration/cmd_dev_test.go:678
...
  [odo]  !  0/3 nodes are available: 3 pod has unbound immediate PersistentVolumeClaims. preemption: 0/3 nodes are available: 3 Preemption is not helpful for scheduling.
  [odo]  !  AttachVolume.Attach failed for volume "pvc-8625590e-378d-4060-8d87-b659cdcadfb2" : timed out waiting for external-attacher of vpc.block.csi.ibm.io CSI driver to attach volume r010-7331ef70-d3e1-43e7-b270-b44d32829245
  [odo]  !  AttachVolume.Attach failed for volume "pvc-3efa3caa-5d2a-43d2-93bf-4e673e3f0407" : timed out waiting for external-attacher of vpc.block.csi.ibm.io CSI driver to attach volume r010-9aa63bb0-81c2-4ac4-81cf-7672c5bd607f
  [odo]  !  Unable to attach or mount volumes: unmounted volumes=[auto-pvc-automount-readonly-pvc auto-pvc-automount-default-pvc], unattached volumes=[odo-shared-data auto-secret-automount-access-mode-subpath-secret auto-secret-automount-mount-path-secret auto-cm-automount-mount-path-configmap auto-pvc-automount-readonly-pvc kube-api-access-ghdgj auto-pvc-automount-mount-path-pvc auto-cm-automount-access-mode-configmap auto-cm-automount-access-mode-configmap-decimal auto-secret-automount-default-secret odo-projects auto-cm-automount-default-configmap auto-secret-automount-subpath-secret auto-pvc-automount-default-pvc auto-cm-automount-access-mode-subpath-configmap auto-cm-automount-subpath-configmap auto-secret-automount-access-mode-secret]: timed out waiting for the condition
  [odo]  !  AttachVolume.Attach failed for volume "pvc-3efa3caa-5d2a-43d2-93bf-4e673e3f0407" : timed out waiting for external-attacher of vpc.block.csi.ibm.io CSI driver to attach volume r010-9aa63bb0-81c2-4ac4-81cf-7672c5bd607f
  [odo]  !  AttachVolume.Attach failed for volume "pvc-8625590e-378d-4060-8d87-b659cdcadfb2" : timed out waiting for external-attacher of vpc.block.csi.ibm.io CSI driver to attach volume r010-7331ef70-d3e1-43e7-b270-b44d32829245
  [odo]  !  Unable to attach or mount volumes: unmounted volumes=[auto-pvc-automount-default-pvc auto-pvc-automount-readonly-pvc], unattached volumes=[odo-shared-data auto-secret-automount-subpath-secret auto-secret-automount-mount-path-secret auto-pvc-automount-mount-path-pvc auto-secret-automount-default-secret auto-secret-automount-access-mode-secret auto-cm-automount-default-configmap auto-secret-automount-access-mode-subpath-secret auto-pvc-automount-default-pvc auto-cm-automount-subpath-configmap auto-cm-automount-access-mode-subpath-configmap kube-api-access-ghdgj auto-cm-automount-access-mode-configmap auto-pvc-automount-readonly-pvc auto-cm-automount-mount-path-configmap auto-cm-automount-access-mode-configmap-decimal odo-projects]: timed out waiting for the condition
  [odo]  !  AttachVolume.Attach failed for volume "pvc-3efa3caa-5d2a-43d2-93bf-4e673e3f0407" : timed out waiting for external-attacher of vpc.block.csi.ibm.io CSI driver to attach volume r010-9aa63bb0-81c2-4ac4-81cf-7672c5bd607f
  [odo]  !  AttachVolume.Attach failed for volume "pvc-8625590e-378d-4060-8d87-b659cdcadfb2" : timed out waiting for external-attacher of vpc.block.csi.ibm.io CSI driver to attach volume r010-7331ef70-d3e1-43e7-b270-b44d32829245
  [odo]  !  Unable to attach or mount volumes: unmounted volumes=[auto-pvc-automount-readonly-pvc auto-pvc-automount-default-pvc], unattached volumes=[auto-secret-automount-default-secret auto-cm-automount-access-mode-subpath-configmap kube-api-access-ghdgj auto-cm-automount-default-configmap auto-pvc-automount-mount-path-pvc auto-cm-automount-mount-path-configmap odo-projects auto-pvc-automount-readonly-pvc auto-cm-automount-access-mode-configmap-decimal auto-secret-automount-mount-path-secret auto-secret-automount-subpath-secret auto-secret-automount-access-mode-secret auto-cm-automount-access-mode-configmap odo-shared-data auto-pvc-automount-default-pvc auto-secret-automount-access-mode-subpath-secret auto-cm-automount-subpath-configmap]: timed out waiting for the condition
  [FAILED] in [BeforeEach] - C:/Users/Administrator.ANSIBLE-TEST-VS/3981/tests/helper/helper_run.go:54 @ 05/22/23 06:24:34.433

Overriding because all those tests passed in the previous run (with the same changes).
/override windows-integration-test/Windows-test


"Download to wikitable" parallel to "Download to CSV" in sqllab

Discussed in https://github.com/apache/superset/discussions/24455

Originally posted by stuartyeates June 20, 2023
"Download to wikitable" parallel to "Download to CSV" in sqllab
Motivation
The Wikimedia Foundation (WMF) currently runs an in-house SQL explorer called Quarry https://quarry.wmcloud.org/ The plan is to move to superset, see https://phabricator.wikimedia.org/T169452. A very heavily used feature of Quarry is export as a wikitable (i.e. markup that can be included in a wikimedia wiki page) See for example https://quarry.wmcloud.org/query/74483
Since there are many tens of thousands of wikimedia wiki installs around the world, such as feature would be likely to be of use to others as well as ourselves.
Note: I'm a WMF volunteer, not a WMF staffer.  I do not / cannot speak for the WMF.
Proposed Change
Clone everything under /api/v1/sqllab/export/ to /api/v1/sqllab/exportwiki/ (or a similar name) and change the output. By keeping things separate impact on the existing code should be less significant. The core of the csv export is https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.to_csv.html but there is not directly equalivent for a wikitable, so I'll have to write my own based partly on the logic at https://github.com/pandas-dev/pandas/blob/main/pandas/io/formats/csvs.py
The documentation on the output format is at https://www.mediawiki.org/wiki/Help:Tables
https://quarry.wmcloud.org/ (which has dozens of users working in dozens of natural languages) would be the test server for this. I'll have my own DEV install.
I believe I understand how to do most of this work. I've already had one pull request merged to master (see https://github.com/apache/superset/pull/24440).
New or Changed Public Interfaces
Addition of /api/v1/sqllab/exportwiki/
Migration Plan and Compatibility
Migration: none.
Compatibility: By using the basic definition at https://www.mediawiki.org/wiki/Help:Tables rather than the richer definition at https://en.wikipedia.org/wiki/Help:Table the plan is to keep the output as portable as possible, both across wikis and going forward in time.
It is likely that binary fields will be dropped (or values swapped with placeholders), because there is no reliable way to represent them in wikitables. UTF-8 will have to be un-escaped (I believe); there are plenty of WMF users querying non-Roman-script wikis to test with.

@rchard2scout on Jun 21, 2023:

Just FYI, the wikitable formatting code in Quarry is available under an MIT license here. It's not very complicated.

@stuartyeates replying to @rchard2scout on Jun 21, 2023:

You are correct.
However the context is very different; Quarry has already mapped VARBINARY(M) and other types to UTF8, whereas Superset hasn't; binary fields aren't handled in that Quarry export as well as they are in Superset; etc.

Hi @fredster33 - sorry this slipped under the collective radar when you posted it. We're trying to get better about these things.
Are you still interested in contributing to this? I think we would welcome a SIP or a PR to discuss, if so. It does indeed seem useful... we just need to make sure that as we add more and more export formats, they're kept discrete and maintainable - maybe even pluggable!
In any case, normally, I'd close this as stale, and because it's not a bug, but meanwhile I'll move it to a GitHub Discussion as an Ideas thread in case you (or anyone) want to pick it up. Let us know!


Jsonld 1 1

I've fixed the issue we discussed around the implementation report not running all the test executables so this produces a full report now

There were a couple of fixes that I made in the first commit to include the http-link-header-cpp as a submodule and updated the README to reflect that.  Also had a couple of compile errors which I fixed in the first commit

I just noticed that I forgot to add the http-link-header-cpp as a submodule when I commited the changes to CMakeLists.txt. So sorry about that!


Remove signature form after signed or confirmed

Is your feature request related to a problem? Please describe.
I think that displaying the whole form for someone who is signing a petition is not very good ux.

Describe the solution you'd like
I simply want to remove the form and let the whole space for the message.

What is the purpose of the progress bar above the message ? Should the message disappear after a certain time ?

Someone already gave me this feedback : the form needs to disappear upon successful signing. No point in showing it again.
We should only show the green "success" message, I fully agree.
The bar is meant to always be there, it's the "progress bar" showing the % of signatures over the signature target (which is configurable).
Like the progress bar in crowd-funding pages.
The green message will disappear as soon as you leave or refresh the page, it's only temporary and it is showed once.
I don't want to keep the information that someone signed the petition.


copying xeol to /usr/local/bin would need root privileges

Hello, I noticed that the recommended one liner script gives permission denied error while installing at /usr/local/bin which is expected. I would suggest to add sudo at the beginning (sudo sh -s -- -b /usr/local/bin) to make sure the script has sufficient permission.

would prefer to leave sudo out in the default command especially since it's executing an install script from the internet. I would recommend installing to a specific location and then sudo moving to /usr/local/bin if only root has access to /usr/local/bin
curl -sSfL https://raw.githubusercontent.com/xeol-io/xeol/main/install.sh | sh -s -- -b /tmp && sudo mv /tmp/xeol /usr/local/bin/xeol


python3Packages.pysonos: 0.0.48 -> 0.0.49

Motivation for this change
Update to latest upstream release  0.0.49
Change log: https://github.com/amelchio/pysonos/releases/tag/v0.0.49

Related Home Assistant change: https://github.com/home-assistant/core/pull/50841
Target Home Assistant release: next release

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123641 at 1d826752 run on aarch64-linux 1

3 packages built successfully:

home-assistant
python38Packages.pysonos
python39Packages.pysonos