undefined method `include?' for nil:NilClass (NoMethodError)

I want to add In-APP-Purchase but I can`t find the app
my code
require 'spaceship'
require 'csv'
Spaceship::Tunes.login("xxx", "xxxx")
Spaceship::Tunes.select_team(team_id: "xxx")
app = Spaceship::Tunes::Application.find("xxxx")

output of console
/Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/client.rb:793:in `detect_most_common_errors_and_raise_exceptions': undefined method `include?' for nil:NilClass (NoMethodError)
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/client.rb:770:in `parse_response'
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/tunes/tunes_client.rb:258:in `applications'
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/tunes/application.rb:67:in `all'
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/tunes/application.rb:73:in `find'
	from iap_add.rb:7:in `<main>'

🚫 fastlane environment 🚫
Stack

Key
Value

OS
12.3

Ruby
2.6.8

Bundler?
true

Git
git version 2.32.1 (Apple Git-133)

Installation Source
/usr/local/bin/fastlane

Host
macOS 12.3 (21E230)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib

OpenSSL Version
LibreSSL 2.8.3

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
13.4

Swift Version
5.6.1

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

default_platform(:ios)

platform :ios do
  desc "Description of what the lane does"
  lane :custom_lane do
    # add actions here: https://docs.fastlane.tools/actions
  end
end

`./Appfile`
# app_identifier("[[APP_IDENTIFIER]]") # The bundle identifier of your app
# apple_id("[[APPLE_ID]]") # Your Apple email address

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.207.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

rake
13.0.6

rexml
3.2.5

CFPropertyList
3.0.5

public_suffix
4.0.7

addressable
2.8.0

artifactory
3.0.15

atomos
0.1.3

aws-eventstream
1.2.0

aws-partitions
1.603.0

aws-sigv4
1.5.0

jmespath
1.6.1

aws-sdk-core
3.131.2

aws-sdk-kms
1.57.0

aws-sdk-s3
1.114.0

babosa
1.0.4

bundler
2.3.4

claide
1.1.0

colored
1.2

colored2
3.1.2

highline
2.0.3

commander
4.6.0

declarative
0.0.20

digest-crc
0.6.4

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

dotenv
2.7.6

emoji_regex
3.2.3

excon
0.92.3

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

multipart-post
2.0.0

faraday-multipart
1.0.4

faraday-net_http
1.0.1

faraday-net_http_persistent
1.2.0

faraday-patron
1.0.0

faraday-rack
1.0.0

faraday-retry
1.0.3

ruby2_keywords
0.0.5

faraday
1.10.0

http-cookie
1.0.5

faraday-cookie_jar
0.0.7

faraday_middleware
1.2.0

fastimage
2.2.6

gh_inspector
1.1.3

jwt
2.4.1

memoist
0.16.2

multi_json
1.15.0

os
1.1.4

signet
0.17.0

googleauth
1.2.0

httpclient
2.8.3

mini_mime
1.1.2

trailblazer-option
0.1.2

uber
0.1.0

representable
3.2.0

retriable
3.1.2

webrick
1.7.0

google-apis-core
0.7.0

google-apis-androidpublisher_v3
0.24.0

google-apis-playcustomapp_v1
0.10.0

google-apis-iamcredentials_v1
0.13.0

google-apis-storage_v1
0.17.0

google-cloud-env
1.6.0

google-cloud-errors
1.2.0

google-cloud-core
1.6.0

google-cloud-storage
1.37.0

json
2.6.2

mini_magick
4.11.0

naturally
2.2.1

optparse
0.1.1

plist
3.6.0

rubyzip
2.3.2

security
0.1.3

simctl
1.6.8

terminal-notifier
2.0.0

unicode-display_width
1.8.0

terminal-table
1.8.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

word_wrap
1.0.0

nanaimo
0.3.0

xcodeproj
1.22.0

rouge
2.0.7

xcpretty
0.3.0

xcpretty-travis-formatter
1.0.1

generated on: 2022-07-07

We are experiencing the same issue.

Same here.

Same issue.
fastlane/2.207.0/libexec/gems/fastlane-2.207.0/spaceship/lib/spaceship/client.rb:793:in detect_most_common_errors_and_raise_exceptions': \e[31m[!] undefined method include?' for nil:NilClass (NoMethodError)

undefined method `include?' for nil:NilClass

Also hitting this issue with fastlane init.
/Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/client.rb:793:in `detect_most_common_errors_and_raise_exceptions': [!] undefined method `include?' for nil:NilClass (NoMethodError)
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/client.rb:770:in `parse_response'
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/tunes/tunes_client.rb:258:in `applications'
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/tunes/application.rb:67:in `all'
	from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.207.0/spaceship/lib/spaceship/tunes/application.rb:73:in `find'

Same issue.

Same issue

Same issue

Apparently this is causing MS App Center to fail too. MS just replied to me indicating that its this issue.

This is probably neither fastlanes nor MS App Centers fault, it seems like Apple changed their API, thus spaceship client is confused.

Same issue.
I'm wondering that whether it is temporary or permenent, I with it can be fixed soon.

Same here, I was given the link to this issue by Microsoft App Center support, does anybody know whether this was a bug in Fastlane or an API change by Apple?

Same here, I was given the link to this issue by Microsoft App Center support, does anybody know whether this was a bug in Fastlane or an API change by Apple?

It should be apple modified api

@wallisch : Thanks a lot for the clarification. Since you're a contributor, do you have any Idea in what timeframe we can expect Fastlane adapting to Apples API changes (jusst guessing that this is not the first time happening)? I don't want to stress anyone, but I have absolutley zero experience with directly manually releasing to iOS App Store and now need to assess whether to wait for a fix or if I should better learn how to build and release on my own to iOS App Store.

Same issue

@wallisch : Thanks a lot for the clarification. Since you're a contributor, do you have any Idea in what timeframe we can expect Fastlane adapting to Apples API changes (jusst guessing that this is not the first time happening)? I don't want to stress anyone, but I have absolutley zero experience with directly manually releasing to iOS App Store and now need to assess whether to wait for a fix or if I should better learn how to build and release on my own to iOS App Store.

You can upload your version manually. It's easy than upload by fastlane. You need set correct version and build number, archive app and upload to app store

thanks for the workaround but, with all due respect, a lot of people who are using App Center are doing this because they don't necessarily have the time to upload versions manually and they are under pressure (e.g. me). This is why we need at least a time frame. thanks.

Same issue.

Same issue

Same issue

I believe the error is on the Apple side.
Spaceship calls this endpoint to get the application data: https://appstoreconnect.apple.com/WebObjects/iTunesConnect.woa/ra/apps/manageyourapps/summary/v2
and it is returning something like this:
{"data":null,"messages":{"warn":null,"error":null,"info":null},"statusCode":"SUCCESS"}

Spaceship thinks it's an error message because it has a null value in the data field and tries to get the error message in an empty string.
I think apple used to return a response like this: https://github.com/fastlane/fastlane/blob/2.19.2/spaceship/spec/tunes/fixtures/app_summary.json
but I'm not sure about it.

Same issue on appcenter.
This is still working through AppStoreRelease@1 and ServiceConnection in ADO

Same issue on appcenter. We need a temporarily work around.
Has anyone submitted an Apple Developer bug report or via Apple Developer support?

+1

For people having issues with app center, you can download the ipa file from app center distribute option.
To avoid calling the endpoint: https://appstoreconnect.apple.com/WebObjects/iTunesConnect.woa/ra/apps/manageyourapps/summary/v2
I create a new Application manually with data from ConnectAPI.
It should not be suitable for all cases, but it has worked for me until now.
Old call to get application data
app = Spaceship::Tunes::Application.find(app_id)

New way to get data from ConnectAPI and create an application manually
conn_app = Spaceship::ConnectAPI.get_app(app_id: app_id).first

app = Spaceship::Tunes::Application.factory({})
app.apple_id = conn_app.id
app.name = conn_app.name
app.bundle_id = conn_app.bundle_id

Looks like AppCenter have rolled out a fix that worked for me, had to open the build config and save before rebuilding but it’s working for me

Hey, everyone 👋 Sorry for getting here so late!
Usage of Spaceship::Tunes
What @lucgrabowski mentions about Spaceship::Tunes is what I would have said too! It's using the legacy private API that can change or shut down without us knowing. Using Spaceship::ConnectAPI is the most recommend way as that uses either the new private Apple ID auth endpoints or the official App Store Connect API with API Key auth.
However, there are times where you still need to use Spaceship::Tunes with with IAP 😬
Good news
App Store Connect API 2.0 just dropped yesterday with official support for IAP. This is huge but it's not a easy implementation or migration. There are over 104 new endpoints that we will need to design into fastlane better.
So there will be super stable official support for IAP soon ™  I don't have an estimate date yet as I'm still going through everything from yesterday evening's release.

Hello everyone
Currently I fixed the problem with fastlane init locally:
Specifically in setup_ios.rb, find this method verify_app_exists_itc
Find this code:
app = Spaceship::Tunes::Application.find(self.app_identifier)
Replace with:
app = Spaceship::ConnectAPI::App.find(self.app_identifier)
then save the code
This will work fine

@shizu2014 To fix that you cloned the repository and you did these changes manually?

@joshdholtz No workaround for now to run fastlane init without failing?

Thx for it, I can login now but still another problem happened when i tried to upload the IAPs.
undefined method `in_app_purchases' for #<Spaceship::ConnectAPI::App:0x0000000####### @id="#######", @name="#####", @bundle_id="######"...

So there will be super stable official support for IAP soon ™ I don't have an estimate date yet as I'm still going through everything from yesterday evening's release.

@shizu2014 To fix that you cloned the repository and you did these changes manually?

Local fastlane code file

With Spaceship::Tunes.login I was able to access apps of other teams where I have been invited to.
Since this does not work anymore, I tried it with Spaceship::ConnectAPI
but this seams to require an API Access for each team, which does not work for me.
One task is e.g.: (similar to @aizliang )
clientTunes = Spaceship::Tunes.login('xxx', 'xxx')
clientTunes.team_id = xxx
clientTunes.applications.to_json

Will be there a workaround for this?

Thx for it, I can login now but still another problem happened when i tried to upload the IAPs.
undefined method `in_app_purchases' for #<Spaceship::ConnectAPI::App:0x0000000####### @id="#######", @name="#####", @bundle_id="######"...

So there will be super stable official support for IAP soon ™ I don't have an estimate date yet as I'm still going through everything from yesterday evening's release.

try it: https://github.com/fastlane/fastlane/issues/20444#issuecomment-1182140327

Hey sorry! I commented way to late in my evening and didn't fully understand the problem 🤦♂
I'll make that PR to fix the unit and then also look if we use this anywhere else and fix it there as well.
Will get that done today and new version released tonight some time!

Thx for it, I can login now but still another problem happened when i tried to upload the IAPs.
undefined method `in_app_purchases' for #<Spaceship::ConnectAPI::App:0x0000000####### @id="#######", @name="#####", @bundle_id="######"...

same problem . I didn't found iap module in ConnectAPI .

Will get that done today and new version released tonight some time!

any update?

Any update

Trying to distribute application to Test Flight via AppCenterReceived an email from Apple to use:
Currently, the workaround does not involve using App Center.
Here's what we recommend:

build app through App Center without distribution step
download the .ipa
run the following command (works only on Mac)
xcrun altool --upload-app -f .ipa -u  -p  -type
*In case of errors or conflicts, consult manual for altool (man altool)
--type "{macos | ios | appletvos}"

Looking for update? Would prefer not to use the command so as to not have to change our pipeline process in the future?

✨ Official Question
Can somebody who is still seeing AppCenter question help me out by telling me which flow is failing exactly?
Is it a failure with the AppCenter fastlane plugin? Or is it on AppCenter's CI where they upload to a signed built to the App Store / TestFlight for you?
Thanks!

@joshdholtz I'm not able to register a new app into App Center, it just doesn't show anything:
https://appcenter.ms/api/v0.1/apple/itunes/get_all_apps returns:
{"code":"internal_server_error","message":"{\"code\":\"internal_server_error\",\"message\":\"undefined method `include?' for nil:NilClass\"}"}

UI shows:

Here's the update I got from App Center support:

Hi there,
Thank you for contacting us.
The App Center engineering team is working with high priority to deploy a fix. We apologize for the inconvenience.
This incident is caused by an issue with FastLane:
https://github.com/fastlane/fastlane/issues/20444
We have also updated our status page so you can subscribe to it for automatic updates as we progress towards a resolution:
https://status.appcenter.ms/incidents/ts3n2ks33wt9
Best regards,
Ximei.

@joshdholtz I'm not able to register a new app into App Center, it just doesn't show anything:
https://appcenter.ms/api/v0.1/apple/itunes/get_all_apps returns:
{"code":"internal_server_error","message":"{\"code\":\"internal_server_error\",\"message\":\"undefined method `include?' for nil:NilClass\"}"}

UI shows:

Thank you for that! The fix I'm making here will not fix that since it seems like MS is using the deprecated Spaceship::Tunes and not the fastlane init for that call 😬
I've tweet them to see if they can reach out to me so I can work with them on a fix 🤷♂

Thanks @joshdholtz I've also contacted them again from my side, asking them to get in touch with you.

Thanks @joshdholtz I've also contacted them again from my side, asking them to get in touch with you.

@will-gluwa Thank you!! Hopefully I can get an official relationship with them after this to prevent downtime in the future
Sorry again about this, fam!

so there won't be a fix for Spaceship::Tunes ? 😢

Is it a failure with the AppCenter fastlane plugin? Or is it on AppCenter's CI where they upload to a signed built to the App Store / TestFlight for you?

@joshdholtz , we experience the latter. It's just the AppCenter CI stopped distributing the apps automatically for us

@joshdholtz Will the IAP uploading flow be covered under any of the latest fastlane APIs ?

Experiencing the same internal server issue 500 that other users have reported.
##[error]{"code":"internal_server_error","message":"internal error (Support ID: 0b13f7e4-0908-4583-92f4-6efba58a508c)"}
This happens when sending a successful / completed build from AppCenter -> Appstoreconnect
Since a 500 is general server and disclaimer without any further logs - I reached out to Apple.

@joshdholtz I understand it seems the most impactful TODO item here being AppCenter and all its users, but the originally reported issue is about IAP uploading flow. Do you mean that after getting past the app selection step, the existing APIs should be good to go about uploading in app purchases? Or are we out of luck on that? I'll also upgrade that part of the flow now & report back my findinds.

@rafaelnobrekz  I totally read "IAP" and "IPA" 🙄 My apologies on that... that was a really unfortunate misread on my part 🤦♂
The in-app purchase APIs I'm working on... It's a very slow process because its a huge change and there are a lot of new APIs 😬 I'll create a new issue (and pin it) to discuss that one (outside of this one because its slightly different).
You can follow https://github.com/fastlane/fastlane/issues/20477 for the IAP stuff

✨ Official Thought
I might be able to patch/hack this in an fastlane update that MS AppCenter can use but I won't love it 😬
Going to experiment by guessing the APIs they are using but I'd still like to talk to their team about an official and long term fix
Will update when I have more info!

✨ Official Update
I got a pull request finally up for some fixes here - https://github.com/fastlane/fastlane/pull/20480
Fixed fastlane init
fastlane init will no longer use Spaceship::Tunes::Application.find (which was broken due to an endpoint being no longer available) and is now using Spaceship::ConnectAPI::App.find
That was the easy one 😊
Temporary patch on Spaceship::Tunes::Application.find
I know that Spaceship::Tunes::Application.find still needs to be used for IAPs so I retrofitted this method to be powered by Spaceship::ConnectAPI::App.find. It's a terrible retrofit since there is no way to nicely get all of the data that but you'll at least get app id, app name, and available platforms.
This should also hopefully fix the issues that App Center is having with the App Store Distributions. I don't know for sure since I think they use Spaceship::Tunes::Application on their backend but I'm hoping this retrofit works for them if they are using it 🤷♂  I'm still trying to get in contact with them.
Adding IAP support to Spaceship::ConnectAPI
This is not part of this PR but I a working on it. I have an issue I'll post feedback on soon here - https://github.com/fastlane/fastlane/issues/20477
It will take some time because there are A LOT of new PRs but follow that issue for more info

That is all! So sorry about this taking so long. It ended up being a pretty back task to retrofit but I'm hoping it works out for some of you.
If any of you are able to test this PR / branch to make sure the retrofit is working for you that would be 🔥  I need to go to bed now so I'll look at any response in the morning 😊

fastlane 2.208.0 🚀
thanks! it work for me.

@sk-chanch Glad to hear!!
@SoftWyer No problem! I actually talked to some of the team this morning and I'm going to keep a formal relationship with them open so that I can communicate any changes that might affect them. I think this will be good for both fastlane and App Center 💪

@joshdholtz Great job! It also work for me!

Works for us as well. Great job @joshdholtz and thank you!

Thanks alot for the fix @joshdholtz ! I have been in contact with Appcenter support through this also and they indicate working on their own implementation talking directly to apple API:s but knowing how long they take to fix stuff and how many issues we have had with appcenter,  my guess is that will take very long and even longer now that this has been patched.


[Access Request]  page 1001 "Job Task Lines Subform" - PerTaskBillingFieldsVisible

Describe the request
Could you please change the access scope of the following variable to protected?
protected var
    PerTaskBillingFieldsVisible: Boolean;

Additional context
We would like to hide and show custom fields depending on this value. Therefore we need access to those variable.
Internal work item: AB#527180

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 18872.


Tweak to improve searching

I had to view source to find how to remove toasts. (via clear). I suggest adding the words remove and delete in docs to help with text searches for alternative terms.

Some changes:
toastr.clear() executes closing animations.
toastr.remove() does an immediate removal without animations.
Can you update your docs and re-submit against the develop branch?

Ah great! Remove is what I actually needed. Thanks!


AttributeError: type object 'SMPL' has no attribute 'SHAPE_SPACE_DIM'

Thank you for your perfect work. When I run the optimization on the demo video, I met the following problem:

I have downloaded the smplh model from the website. Could you please give me some advice?
Thx!

Hello. This might be because you are using an old version of the smplx library. Please make sure that you have version 0.1.28 installed as indicated in the requirements file.
If for some reason you really can't upgrade to the newest version you could replace SMP.SHAPE_SPACE_DIM in the code here to be hardcoded to 300. However, using an old version of smplx may affect other parts of the code in unexpected ways, so I would suggest avoiding this.

Hello. This might be because you are using an old version of the smplx library. Please make sure that you have version 0.1.28 installed as indicated in the requirements file.
If for some reason you really can't upgrade to the newest version you could replace SMP.SHAPE_SPACE_DIM in the code here to be hardcoded to 300. However, using an old version of smplx may affect other parts of the code in unexpected ways, so I would suggest avoiding this.

Thank you for your solution. I have solved this problem after I updated the smplx to version 0.1.28.


getBlockInfo call is not efficient

Alluxio Version:
2.5
Describe the bug
getBlockInfo call can take a very long time
To Reproduce
Steps to reproduce the behavior (as minimally and precisely as possible)
Expected behavior
A clear and concise description of what you expected to happen.
Urgency
Describe the impact and urgency of the bug.
Additional context
pingMetaService can be an expensive operation (take ~500ms in my test) to determine the primary master rpc address and in the connect() function in AbstractClient , we are basically calling it 3 times
in getAddress()
in getConfAddress()
in checkVersion  (It doesnt call pingMetaService explicitly but still call the versionService)

with #13881 closed, will we close this issue @yuzhu @ggezer ?

Can this be closed?


Ensure notebooks-cloud integration tests run on current pyvespa-version

Currently, since we pip3 install pyvespa inside the notebooks after pip install -e .[dev,notebooks] in the action workflow, the former will override pyvespa with latest version from pypi, rather than the checked out version.

Closed by #907


Fix nesting of Test.group

Pull Request Description

Closes #8430 by ensuring that specs within a nested group are re-attached to the parent group.

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[ ] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

With this PR, the following code
from Standard.Base import all

from Standard.Test import Test, Test_Suite
import Standard.Test.Extensions

main = Test_Suite.run_main spec

spec =
    Test.group "My group" <|
        Test.specify "test OK" <|
            2+2 . should_equal 4

        Test.group "nested group" <|
            Test.specify "nested test OK" <|
                2+2 . should_equal 4

            Test.specify "nested test bad" <|
                2+2 . should_equal 3

        Test.specify "other test OK" <|
            2+3 . should_equal 5

        Test.group "2nd nested" <|
            Test.group "double nested" <|
                Test.specify "nested test OK" <|
                    2+2 . should_equal 4

                Test.specify "nested test bad" <|
                    2+2 . should_equal 3

yields
My group:  [4/6, 154ms]
    - test OK [18ms]
    - nested group - nested test OK [0ms]
    - [FAILED] nested group - nested test bad [127ms]
        Reason: 4 did not equal 3 (at C:\NBO\repr\test.enso:18:17-36).
    - other test OK [0ms]
    - 2nd nested - double nested - nested test OK [0ms]
    - [FAILED] 2nd nested - double nested - nested test bad [6ms]
        Reason: 4 did not equal 3 (at C:\NBO\repr\test.enso:29:21-40).
4 tests succeeded.
2 tests failed.
0 tests skipped.

and correctly returns a non-zero exit code.
So now nested groups are handled correctly :)


Improve Color Contrast and Footer Spacing for Better Visual Appeal 🎨✨

Issue Description:
In the current design of the CabRental website, the orange background color is touching the white section directly below it (as seen in the attached screenshot). This reduces the overall visual appeal and makes the section feel cramped. By adding more space or modifying the color contrast, the design can be made more attractive and user-friendly. 🌟
Proposed Solution:

Add appropriate padding or margin between the orange section and the white section below. ➕
Ensure better color contrast and alignment for a more visually appealing layout. 🎨
Test the changes for responsiveness on both desktop and mobile views. 📱💻

Screenshot

@07sumit1002 I would be happy to take up this issue and make the necessary adjustments. Kindly assign this issue to me! 🙌
Thank you!

i would like to work on this issue


no "Settings" button to change fonts

Pretty sure this was a bug in a couple Atom versions. Unable to reproduce in v1.27.2:

Please open new issues if this still happens on current Atom versions.


License?

Could you add a license please?
If you're not sure which one to choose, this site might help.

I like syncing my notes (incl. .obsidian/) via git, but without a license it's not clear that I have the rights to use your plugin or distribute it as part of my own (non-public) repository.

+1
@delashum A license gives legal clarity both to users and contributors around what circumstances we're allowed use the plugin, if we're allowed to modify and/or redistribute the code, etc.
The website @kinbiko linked is a good start. Another good resource is the OSI. I usually stick to one of GPL, MPL, or Apache licenses personally :)

MIT license added


Missing Words

General Info

Client Info

Minecraft Version: 1.6.5

Forge Version: 36.1.1

Mod Version: current

Java Version: 8

Issue Description
Sphere and Projectile page listings to not list latin counterpart for casting

Reproduce Steps
Read the book

Additional Information

A fix has been uploaded to Curse :)


StyleGuidelines.md: Added [Parameter()] attribute to 'Good'-examples

Added [Parameter()] attribute to 'Good'-examples so that the StyleGuidelines follow the style guideline. :)
Also replaced PowerShell with powershell on code blocks which enables syntax highlighting in VS Code.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


[MODEL EVALUATION REQUEST] OLMo models

Describe the model
AllenAI has released a 1B and 7B model fully open source (dataset, models, scripts). It is specifically English-only, so should only be benchmarked on English. They're available in this Hugging Face collection.

[ ] allenai/OLMo-1B
[ ] allenai/OLMo-7B
[ ] allenai/OLMo-7B-Twin-2T

Depends on https://github.com/vllm-project/vllm/pull/2832


Condo Documentation

For now, I would like to get all of the shades supported by condo documented.
For each of the macros, I would like the following details:

name
description
supported operating systems
arguments
global arguments (those not prefixed with the name of the shade)
related shades (see also)

The documentation markdown file should be named after the shade (excluding the _), and placed in the docs/shades path within the gh-pages branch.
---
layout: docs
title: Update self
group: shades
---

Updates the build scripts located at the root of the project structure.

## Contents
* Will be replaced with the ToC, excluding the "Contents" header
{:toc}

## Supported Operating Systems

{% icon fa-apple fa-3x %} {% icon fa-windows fa-3x %} {% icon fa-linux fa-3x %}

## Arguments

This uses no arguments.

## Global Arguments

The following global arguments are used by bower:

            Name
            Type
            Default
            Description

            base_path
            string
            $PWD
            The base path in which condo was executed.

## Examples

Condo can update itself by adding the following call within any target:

{% highlight sh %}
    update-self
{% endhighlight %}

The default lifecycle includes a target aptly named `#update-self` that calls this shade.

## See Also

There are no related shades.

The following macros exist and need to be documented in the gh-pages branch:

[x] bower
[ ] bower-download
[ ] bower-install
[ ] brew
[ ] brew-download
[ ] brew-install
[ ] bundle
[ ] bundle-download
[ ] bundle-install
[ ] clean
[ ] copy
[ ] cover
[ ] cover-report
[ ] dnu
[ ] dnu-build
[ ] dnu-pack
[ ] dnu-restore
[ ] dnu-locate
[ ] dnx
[ ] dnx-run
[ ] dnx-test
[ ] dnx-watch
[ ] dnx-watch-download
[ ] exec
[ ] gem
[ ] gem-install
[ ] gem-locate
[ ] git
[ ] git-config
[ ] git-tag
[ ] gm-download
[ ] grunt
[ ] grunt-download
[ ] gulp
[ ] gulp-download
[ ] info
[ ] info-collect
[ ] log
[ ] mkdir
[ ] mocha
[ ] mocha-download
[ ] node
[ ] node-download
[ ] npm
[ ] npm-install
[ ] nuget
[ ] nuget-pack
[ ] nuget-push
[ ] nuget-restore
[ ] rimraf
[ ] rimfraf-download
[x] ruby
[x] run
[x] tsd
[ ] tsd-download
[x] update-self
[ ] verify-hash

I don't see _tsd_install.shade :person_frowning:

... I have no idea what you're talking about.

:dancers: :dancers: :dancers: :dancers: :dancers: :dancers: :dancers: :dancers: :dancers: :dancers:

Uh.....

ok i could have SWORN that you had tsd-install in the list above... as well as tsd-download. So I figured you removed it after my comment and then added a sarcastic "... I have no idea what you're talking about." comment. Either way, it's no longer there.

Hmm... It's magic. :)


Convert index.html to index.vue as SFC #8

Currently extracted the ,  and  component. Will close this issue to refine and add new components if necessary.


"invalid bounds arguments passed to std::clamp" when resizing window to zero height

Windows Terminal version
1.12.10733.0
Windows build number
10.0.22000
Other Software
No response
Steps to reproduce
Launch OpenConsole.exe compiled in debug mode, drag bottom of the window to the very top.
Expected Behavior
No assertion
Actual Behavior
Assertion from from STL "invalid bounds arguments passed to std::clamp" on the line
clampMe.Top = std::clamp(clampMe.Top, Top(), BottomInclusive());

in Viewport Viewport::Clamp

Call stack

 	ucrtbased.dll!00007ff9248a7fbc()	Unknown
 	ucrtbased.dll!00007ff9248a7ef4()	Unknown
 	ucrtbased.dll!00007ff9248a7d90()	Unknown
>	OpenConsole.exe!std::clamp>(const short & _Val, const short & _Min_val, const short & _Max_val, std::less _Pred) Line 10188	C++
 	OpenConsole.exe!std::clamp(const short & _Val, const short & _Min_val, const short & _Max_val) Line 10208	C++
 	OpenConsole.exe!Microsoft::Console::Types::Viewport::Clamp(const Microsoft::Console::Types::Viewport & other) Line 245	C++
 	OpenConsole.exe!Microsoft::Console::Render::Renderer::TriggerRedrawCursor(const _COORD * const pcoord) Line 284	C++
 	OpenConsole.exe!TextBuffer::TriggerRedrawCursor(const _COORD position) Line 992	C++
 	OpenConsole.exe!Cursor::_RedrawCursorAlways() Line 190	C++
 	OpenConsole.exe!Cursor::SetIsOn(const bool fIsOn) Line 103	C++
 	OpenConsole.exe!Microsoft::Console::CursorBlinker::TimerRoutine(SCREEN_INFORMATION & ScreenInfo) Line 153	C++
 	OpenConsole.exe!CursorTimerRoutineWrapper(_TP_CALLBACK_INSTANCE * __formal, void * __formal, _TP_TIMER * __formal) Line 24	C++
 	ntdll.dll!TppTimerpExecuteCallback()	Unknown
 	ntdll.dll!TppWorkerThread()	Unknown
 	kernel32.dll!00007ff9feee54e0()	Unknown
 	ntdll.dll!RtlUserThreadStart()	Unknown

Looks like it happens because SCREEN_INFORMATION::_InternalSetViewportSize increases viewport's Top in srNewViewport.Top -= DeltaY; (DeltaY is negative) line without any check to prevent Top becoming more than Bottom which sadly can happen. Clamp cannot work when Top is more than Bottom

For my own reference, which commit are you building off of? I just wanna know how far back we'll have to service this fix 😅

I'm building recent main (v1.10.1933.0-605-g41ef5554f) and haven't tried to figure out when exactly it appeared unfortunately.

uh, https://github.com/microsoft/terminal/releases/tag/v1.10.1933.0 is like, 10 months old... If this goes that far back, well, we're in some serious trouble

The more I think about, the more I think that call to view.Clamp is fundamentally wrong anyway. The intention was to clip the cursor rect to the viewport, so you'd only be drawing the portion of the cursor that was visible, but that's not what Viewport::Clamp does. If the cursor is completely off screen, it's basically going to pull it back into view, forcing the renderer to redraw an area of the screen that doesn't actually need refreshing. That's probably not the end of the world, but it's not right.
For the same reason, I don't think this bounds error is the end of the world either. In a release build, the clamp is just going to fail to do anything, but that's no worse than the off screen clamping. It's just invalidating an area of the screen that doesn't need redrawing.
So while we definitely should be fixing this, I don't think you need to panic about backporting the fix, unless I'm missing something.

@zadjii-msft I think we may want to reopen this issue. I was just testing PR #14661, which kind of reverts your fix for this, and I wanted to make sure I hadn't reintroduced the bug. What I discovered was that the issue was never fully resolved - while we don't get a clamp assertion in the cursor code, we now just get it somewhere else instead! For me it occurred here:
https://github.com/microsoft/terminal/blob/4d27a05318f4e86b0918497ba6f980968725a1d2/src/terminal/adapter/adaptDispatch.cpp#L219
And I double checked with the commit from when PR #13001 was merged (79904361b345b68f4612d41b390c4eb3f3d18fae), and it was already failing then. It may not be obvious in some shells, but in a WSL bash shell it crashes straight away. In a cmd shell I had to type some command (e.g. CLS) while the window had a zero size in order to trigger the crash.
It's not a major problem, since it only happens in a debug build, but if we think it needs fixing we should probably reopen this, or at least open another issue if you prefer that.
As for how we fix it, I would have liked it if we could put a minimum size on the viewport, but I suspect that's not possible in conhost for legacy reasons. So failing that, a quick fix would be to replace the clamp calls with a min/max combo which won't assert. But my concern is that the VT architecture is kind of dependent on a non-zero screen size, so there's a fair chance this issue might just surface elsewhere at some point in the future. Maybe we just have to accept that.

Thanks for the write up @j4james. I'm gonna just promote that to another thread to try and keep the history a bit cleaner.


sublime4-dev: init at 4079

Motivation for this change
Development version of the upcoming Sublime Text, not merging this since it did not yet reach general availability.

works with my 7 year old license so far
i686 is no longer supported
aarch64 support has been added
adds harware accelerated (needs libGL.so.1)
adds python 3.8 plugin host (needs libssl.so and libcrypto.so)

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"
[x] 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.

I am not sure how would steamenv help. Is not that just a chroot with tons of outdated dependencies?

I saw it used in some places and was hoping it would solve the plugin woes of vscode but didn't investigate closer yet.
I was under the impression that you can add your own dependencies, thus pretending to be a "normal distro"

Added in https://github.com/NixOS/nixpkgs/pull/124057


Entity Configuration custom IEqualityComparer

Hi,
I'm trying to add a custom eqaulityComparer in the EntityConfiguration, but I can't seem to get it working.
I get the error message "the expression must refer to a property"
`
public class DriverChangeDetector : EntityConfiguration
{
public DriverChangeDetector()
{
Add(x => x.AccountNumber, "Account Number");
Add(x => x.StartDate, "Start Date", new DateTimeEqualityComparer());
Add(x => x.EndDate, "End Date", Formatters.FormatDateTime, new DateTimeEqualityComparer());
    }
}

public class DateTimeEqualityComparer : IEqualityComparer<DateTime?>
{
    public bool Equals(DateTime? x, DateTime? y)
    {
        var xUtc = x?.ToUniversalTime();
        var yUtc = y?.ToUniversalTime();

        if (DateTime.Equals(x, y))
        {
            return true;
        }
        else { return false;}
    }

    public int GetHashCode(DateTime? obj)
    {
        throw new NotImplementedException();
    }
}`

Can you point me in the right direction?
Cheers

That error means the property you are configuring is:

Not a property (a field for example)
Not a direct property (x.Prop.Value)

I do have some tests testing nullable DateTimes.
https://github.com/jehugaleahsa/ChangeDetector/blob/master/ChangeDetector.Tests/ChangeDetectionTester.cs#L136
I played around with my unit tests to see if I could reproduce your issue
based on your example and didn't have any luck.
On Mon, Oct 3, 2016 at 7:20 AM, laitang2000<EMAIL_ADDRESS>wrote:

Hi,
I'm trying to add a custom eqaulityComparer in the EntityConfiguration,
but I can't seem to get it working.
I get the error message "the expression must refer to a property"
`public class DriverChangeDetector : EntityConfiguration
{
public DriverChangeDetector()
{
Add(x => x.AccountNumber, "Account Number");
Add(x => x.StartDate, "Start Date", new DateTimeEqualityComparer());
Add(x => x.EndDate, "End Date", Formatters.FormatDateTime, new
DateTimeEqualityComparer());
}

}
public class DateTimeEqualityComparer : IEqualityComparer<DateTime?>
{
public bool Equals(DateTime? x, DateTime? y)
{
var xUtc = x?.ToUniversalTime();
var yUtc = y?.ToUniversalTime();
    if (DateTime.Equals(x, y))
    {
        return true;
    }
    else { return false;}
}

public int GetHashCode(DateTime? obj)
{
    throw new NotImplementedException();
}

}`
Can you point me in the right direction?
Cheers
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jehugaleahsa/ChangeDetector/issues/1, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABTgPhpNLNBU---PLL1cwDgcjcqqusuSks5qwOT3gaJpZM4KMeXF
.

Awesome! my field was not a nullable date!
Thank you.
This is a awesome little library btw!

I modeled this library after Entity Framework. I've always wished they provided a more general-purpose change detector for entities. Even then, I wasn't always comparing database entities anyway. I made this in an application that did diffs for event logging, notified users how approvers modified their requests and tracked changes needed to update external systems. I've been able to use this library to take the Unit of Work pattern way beyond just database concerns. I'm glad your finding it useful. Feel free to share how your using.


Error downloading videos marked explicit

Checklist

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

Verbose log

$ youtube-dl -vx "https://www.youtube.com/watch?v=HRfkqBfiLuM"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-vx', 'https://www.youtube.com/watch?v=HRfkqBfiLuM']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.8.5 (CPython) - Linux-5.4.72-microsoft-standard-WSL2-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] HRfkqBfiLuM: Downloading webpage
ERROR: HRfkqBfiLuM: YouTube said: Unable to extract video data
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 797, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1788, in _real_extract
    raise ExtractorError(
youtube_dl.utils.ExtractorError: HRfkqBfiLuM: YouTube said: Unable to extract video data

Description

youtube-dl is encountering some glitch with explicit YouTube videos.

For what it's worth, what I noticed with my own youtube downloader, the bypass url that was working up until a little while ago is now returning 404 Not Found errors. I don't know it youtube-dl uses the same bypass approach but that might be the reason.

Works with the latest version 2021.05.16 if you pass cookies.

Works with the latest version 2021.05.16 if cookies passed.

How do you that?

How do I pass cookies to youtube-dl?


Replace minimp3 with dr_mp3

Last time minimp3 got updated was in November 2021 (almost 3 years ago).
Meanwhile, dr_mp3, a piece of work done on top of minimp3, has been updated less than a year ago.
Within Godot, the main differences are:

No need for patches
No more MSVC warnings (removed system header treatment)
Possibility of overriding memory allocations (using memalloc)

Template release binary sizes remain the same.
Since minimp3 is removed, the following are also changed in this PR:

minimp3 module renamed to mp3
minimp3_extra_formats build variable renamed to mp3_extra_formats

This should not break compatibility unless module/build option name changes count (does not affect existing projects™).

It seems like the change adds a 0.05s silence at the beginning, gonna check if there's a way to fix this


Issue with getting data from Mat

Unable to get the bytes from a Mat
Running on Windows 10
byte[] yuvBuffer = File.ReadAllBytes(@"F:\tulips_yuv420_inter_planar_qcif.yuv");
Mat src = new Mat(height, width, MatType.CV_8UC1, yuvBuffer);
Cv2.CvtColor(src, dest, ColorConversionCodes.YUV2RGBA_I420);
Cv2.ImWrite(@"F:\newImage.jpg", dest);
The above write a good image file. But I want to have a way to parse the data in dest.
int length = dest.Width * dest.Height * dest.Channels(); // or src.Height * src.Step;
byte[] rgbBuffer = new byte[length];
Marshal.Copy(rgbBuffer, 0, dest.Data, length);
I don't end up with the right bytes in rgbBuffer

Have you tried using ImDecode instead of Marshal.Copy?

I actually have a YUV frame and use the code below to convert the YUV frame to RGB:
byte[] yuvBuffer = File.ReadAllBytes(@"F:\test.yuv");
Mat src = new Mat(height, width, MatType.CV_8UC1, yuvBuffer);
        Mat dest = new Mat(new OpenCvSharp.Size(width, height), MatType.CV_8UC4);
        Cv2.CvtColor(src, dest, ColorConversionCodes.YUV2RGBA_I420);

//            Cv2.ImWrite(@"F:\projects\sensei-testprojects\newImage.jpg", dest);
So I was hoping to get to the raw bytes of the image

If what you need is an array of bytes, try ImEncode


can't run mentalist (windows)

when I run it, I do get this error

and I have python 3.8.5 and pyinstaller installed

I am having the same issue

I just tested it on windows 10 64 bit and can't confirm this. I assume you are using windows:

Download the last release version 1.0 "Mentalist-v1.0-Win.zip"
unpack the zip file
start the .exe file
it's working (even without python installed on host)


remove duplicated code in the engine/content provider.

before making the first release, remove all bad codes :-)
{"yes":true}

fixed in 1.1 many code duplications (mostly from the engine provider).


webpack asset size vs chunk size

I noticed that upon moving files around that we get very strange differences in size reported: #14888.
The webpack stats are available for the base at https://circleci.com/gh/mui-org/material-ui/78237#artifacts/containers/0 and for the PR at https://circleci.com/gh/mui-org/material-ui/78307#artifacts/containers/0
What was weird is that the asset size for Modal changed by a whole kb while the size of the only chunk that was used stayed the same (parsed and gzipped). The modules didn't change which makes sense if the chunk didn't change.
My question is what the difference between asset and chunk size is? It seems like chunk size is more stable. We use the asset size since that was used by our previous solution (size-limit) and is closer to bundlephobia.com. Is this just some bundler deoptimization?

I built a tool to help you analyze webpack bundles for size regressions, and report them directly to GitHub PRs. It's free for open source, so it might be worth checking out and helpful in this scenario.
https://packtracker.io

Closing as we got no relevant feedback in 18 months.


US-16647 - Proof Entitlement getChBEntitlment DPage no longer needs NINO parameter

Updated requirements for Proof of Entitlement service no longer require NINO to be passed from front end as parameter to the getEntitlement datapage, so removing the passing of this parameter from the getDPage api call.

Resolving US-16647 - high priority due to blocking of STG1


PayloadExtraction/useFetch persists data when it shouldn't

Environment

Operating System: Linux
Node Version:     v16.20.0
Nuxt Version:     3.6.5
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      devtools, experimental, nitro
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-4ym9i1?file=pages%2F[id]%2Findex.vue
Describe the bug
To reproduce:

yarn run build
node .output/server/index.mjs
Go to a product and select a size
Go back
Go to same product as in step 3

You'll then see the size is not the default size.
Additional context
No response
Logs
No response

Have the exact same problem.

@madsh93 to me, the general usecase in your reproduction seems very resonable. I can see myself and others trying something similar when working with paginated queries, for example.
But now that I think about it, while this issue is likely a bug, it might be highlighting a missing feature too.
This problem could also be solved by allowing dynamic keys like this:
useAsyncData(() => route.fullPath, () => $fetch(...))

Most usecases of the watch option would probably benefit from using the watched values as (additional) keys.

@madsh93 to me, the general usecase in your reproduction seems very resonable. I can see myself and others trying something similar when working with paginated queries, for example.
But now that I think about it, while this issue is likely a bug, it might be highlighting a missing feature too. This problem could also be solved by allowing dynamic keys like this:
useAsyncData(() => route.fullPath, () => $fetch(...))

Most usecases of the watch option would probably benefit from using the watched values as (additional) keys.

Something that I had in mind when opening #23000

Interesting issue. I've used
const { data: product } = await useFetch('/api/product', {
  key: route.fullPath,
  params
})

And locally it works (with build) - but not in stackblitz 🙈
Can you check if that's the case for you too?
weird-behavior-usefetch-payload.webm


Added documentation section resolving issue 883

Added documentation section, which links to the RTD page to the README file, which resolves issue #883

@cglewis I added myself to authors

Codecov Report

Merging #884 into master will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #884      +/-   ##
==========================================
+ Coverage   76.97%   77.05%   +0.07%
==========================================
  Files          52       52
  Lines        5543     5543
==========================================
+ Hits         4267     4271       +4
+ Misses       1276     1272       -4

Impacted Files
Coverage Δ

vent/api/plugins.py
82.63% <0%> (+0.73%)
:arrow_up:

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 8570b97...09ec38b. Read the comment docs.


How to discover device one by one?

This repository's startDiscovery() function looks like waiting until discovering finished.
I want to add device to state one by one when it has been discovered. Is this possible?
Below is react-native-ble-plx example.
            manager.startDeviceScan(null, null, async (error, device) => {
                  // found a bluetooth device
                  if (device) {
                    console.log(`${device.name} (${device.id})}`);
                    await setDeviceCount(Object.keys(newScannedDevices).length);
                    await setScannedDevices(scannedDevices);
                  }

It update scannedDevices when device is discovered.

You should be able to listen for the EventType.DEVICE_DISCOVERED event.  It looks like it's in the Android Native code, but I'm unsure if it's connected to the React JS side of things.
@Override
public void onDeviceDiscovered(NativeDevice device) {
      // This wasn't previously an event, but now we can send out and request them
      Log.d(TAG, String.format("Discovered device %s", device.getAddress()));
      sendEvent(EventType.DEVICE_DISCOVERED, device.map());
}

I suggest you take a look at the code and play around with it.  If it's not working, or you think there is a better way in which it should work, I'm open to suggestions.  Right now there are three things:
Events:
DEVICE_DISCOVERED - this only happens once per device found.  There was another issue where someone mentioned that Android could find a device multiple times (once without the name and once with) which I didn't account for.
Promise response - this occurs when the startDiscovery has completed (or is cancelled) after the appropriate Timeout.  Android docs say it's 20s.
You should be able to listen for DEVICE_DISCOVERED and then cancel (and ignore) the promise when you get the device you want.  But remember, at this point each device is only sent once, so if the comment above is true, this will not be sufficient at times.
Feel free to submit a pull request with any changes you think important.

@kenjdavidson Thanks for information. I red your answer and tried to use onDeviceDiscovered handler.
Just simply did

const RNBC = RNBluetoothClassic
useEffect(() => {
     RNBC.onDeviceDiscovered((device) => {
      console.log('device discovered.')
      console.log(device)
      // (I should find a way to cleanup listener)
    })
}, [])

And It's done! It already implemented like you said.
Now I can see discovered devices quickly. Some of them are duplicated, but I think I can distinguish them.
Thanks for your kindness, and the effort that managing this library. I cannot find supporting classic bluetooth module like hc-06 except this lib!
Happy New Year!

Glad to hear, stay safe!


bb - dockerfix

In this PR, we change the entry point for the dockerfile to enable deploys on render

https://briggsrr-s23-jpa02.onrender.com/

bc25107 uses hello-1.1.0.jar and works for vivian but for some reason I need to use the regular expression on my machine. No idea why.


Automated cherry pick of #3231: fix: 避免删除azure失败

Cherry pick of #3231 on release/2.10.0.
#3231: fix: 避免删除azure失败

/lgtm

/approve


Reorganize Link types (NavLink, LinkMail, @names, Embeds)

Now that we support JSX in the markdown parser, the link code can be handled much better.
The autoembed code should no longer be called from inside the markdown parser. Instead, some smart classes should be used instead.
NavLink and LinkMail should be merged. The SVG prefixing should become a part of both.
@names should have their own class. The class should be smart, and lookup if the user specified is a legal user (and do the avatar insert if so).
Embeds are a special case type of link, that falls back on the regular link code.

Status:

[X] LinkMail
[X] NavLink
[X] AutoEmbed
[ ] atname (uses NavLink)

how about emojis, should they have <Emoji>? How about image, code, codespan


nixpkgs-vet suggests moving multi-version library to by-name

See https://github.com/NixOS/nixpkgs/actions/runs/11827734660/job/32956454563?pr=355629
Arguably multi-version libraries should still be added to pkgs/development/libraries until a solution can be found that's better than hard-to-maintain bunch of individual <packagename>_<version>/package.nix files.

Have you considered this recommendation for new packages with multiple versions?

I didn't see that. It would be great if nixpkgs-vet could point to that; I didn't even realize that by-name had a README!


Update view.Flush() assert

Fixed another case of the view.Flush() assert issue. Went through most of the asserts in the code base, and it appears all the others are just conditional checks, not actual functions. Except this one.
Going to do one more pass, if I find any others I will send a follow up PR.

Good catch, thanks!


Handling password and token in Ballerina.toml

Description:
If we have to specify a password or token in the toml, How we are going to handle it?
Sample toml file:
org-name= "foo"
version= "0.1.0"

[platform]
target = "java8"

    [[platform.repositories]]
       id="wso2-releases"
       url="http://maven.wso2.org/nexus/content/repositories/releases/"
       username="foo"
       password="*****"

This is currently not part of our plans or a priority, so we are closing the issue for now. We will reopen it if we decide to pursue this in the future.


Adding steps for decagram

I would like to add steps here. Please assign this to me.

Go ahead
Deadline 22 May,2021


Jaeger as Datasource with oauth-proxy

Hi I'm having trouble adding Jaeger as a datasource.
I am using the Redhat version of the Jaeger operator and have a proxy on the Jaeger query. How do I allow Grafana access to Jaeger-query service?
I'm getting 502 Bad gateway when I try to add it as a datasource.
The jaeger ingress is setup like this:
ingress:
    openshift:
      sar: '{"namespace": "foo", "resource": "pods", "verb": "get"}'
    options: {}
    resources: {}
    security: oauth-proxy

if I disable the security I can add Jaeger as a datasource, but  i would like to keep it on.
The Grafana docs only give an example when you have a user/password setup, but I cant use that setup in my case.
DataSource:
spec:
  datasources:
    - access: proxy
      isDefault: true
      jsonData:
        timeInterval: 5s
        tlsAuthWithCACert: true
        tlsSkipVerify: false
      name: Jaeger
      secureJsonData:
        tlsCACert: >-
          -----BEGIN CERTIFICATE-----

          -----END CERTIFICATE-----
      type: jaeger
      url: 'https://jaeger-query.foo.svc.cluster.local:16686/'

@MikeJOBirch is there any logging on the Jaeger side that indicates if Grafana is making a request and if yes, why it fails?

Hi i am now getting Jaeger: Bad Request. 400. Unable to load TLS certificate.
The logs when i try to connect the data source.
Jaeger logs:
{"level":"warn","ts":1644929763.0030396,"caller":"channelz/logging.go:62","msg":"[core]grpc: Server.Serve failed to create ServerTransport: connection error: desc = \"transport: http2Server.HandleStreams received bogus greeting from client: \\\"\\\\x16\\\\x03\\\\x01\\\\x02\\\\x00\\\\x01\\\\x00\\\\x01\\\\xfc\\\\x03\\\\x03\\\\xc4a\\\\xecb\\\\xf4+\\\\xf6\\\\x19ƛd\\\\x159\\\"\"","system":"grpc","grpc_log":true}
oauth-proxy logs:
2022/02/15 12:47:51 provider.go:128: Defaulting client-id to system:serviceaccount:foo:jaeger-ui-proxy 2022/02/15 12:47:51 provider.go:133: Defaulting client-secret to service account token /var/run/secrets/kubernetes.io/serviceaccount/token 2022/02/15 12:47:51 oauthproxy.go:203: mapping path "/" => upstream "http://localhost:16686/" 2022/02/15 12:47:51 oauthproxy.go:230: OAuthProxy configured for  Client ID: system:serviceaccount:foo:jaeger-ui-proxy 2022/02/15 12:47:51 oauthproxy.go:240: Cookie settings: name:_oauth_proxy secure(https):true httponly:true expiry:168h0m0s domain:<default> samesite: refresh:disabled 2022/02/15 12:47:51 http.go:107: HTTPS: listening on [::]:8443 I0215 12:47:51.697627       1 dynamic_serving_content.go:130] Starting serving::/etc/tls/private/tls.crt::/etc/tls/private/tls.key 2022/02/15 12:47:51 http.go:61: HTTP: listening on <IP_ADDRESS>:4180 2022/02/15 12:51:42 server.go:3119: http: TLS handshake error from <IP_ADDRESS>:34540: read tcp <IP_ADDRESS>:8443-><IP_ADDRESS>:34540: read: connection reset by peer

Hi @pb82
Any update on this?


Conflict with  version of werkzeug required

it seems that webui has the werkzeug version set to 0.15.3, there is a bug in that version that gives this error: TypeError: required field "type_ignores" missing from Module to fix this all that is needed is an update to that version: Stackoverflow question. If you could please update the requirements allowing for the update so the issue is resolved

This module is a bit starved for maintainer attention right now, if you want to make a pull request that would be a great start.

This got fixed a while ago.


Where Can I Get PDB's?

According to this commit 'no more pdbs on release builds', it looks like PDB's are deliberately not shipped.
In particular, I'm looking for PDB files for e_sqlite3.dll on UWP with x86/x64/ARM64 architectures. Is it possible to get hold of them from somewhere?

At the moment, no, I don't have PDBs for the e_sqlite3 builds.
Another recent request for same prompted me to log #312

Closing in favour of https://github.com/ericsink/SQLitePCL.raw/issues/312


Attempting to unpickle unsafe module collections

Hi there,
I've got this exception, actually a lot of them (similar to #608 ) with graphite-web 0.9.15:
Wed Jan 20 23:33:19 2016 :: Exception Caught
Traceback (most recent call last):
  File "/opt/graphite/webapp/graphite/render/datalib.py", line 231, in fetchData

  File "/opt/graphite/webapp/graphite/render/datalib.py", line 140, in query
    return connection
  File "/opt/graphite/webapp/graphite/render/datalib.py", line 166, in send_request
    serialized_request = pickle.dumps(request, protocol=-1)
  File "/opt/graphite/webapp/graphite/render/datalib.py", line 181, in recv_response
    else:
  File "/opt/graphite/webapp/graphite/util.py", line 108, in loads

  File "/opt/graphite/webapp/graphite/util.py", line 97, in find_class
    # http://nadiana.com/python-pickle-insecure
UnpicklingError: Attempting to unpickle unsafe module collections

Any ideas?

As you can see in https://github.com/graphite-project/graphite-web/blob/fe04c9e64d53057877a25cbc78edd70dc8127759/webapp/graphite/util.py#L161-L167 collections class is included in PICKLE_SAFE variable, so, it should unpickle safely.
So, my idea is that you're not running 0.9.15
Could you please check your local webapp/graphite/util.py file - does it looks similar?

No it doesn't looks similar!
pip output is:
[root@graphitecarbon-172 graphite]# pip list | grep graphite-web
DEPRECATION: Python 2.6 is no longer supported by the Python core team, please upgrade your Python. A future version of pip will drop support for Python 2.6
graphite-web (0.9.15)

But file webapp/graphite/util.py is different:
  try:
    defaultProfile = Profile.objects.get(user=defaultUser)
  except Profile.DoesNotExist:
    log.info("Default profile does not exist, creating it...")
    defaultProfile = Profile(user=defaultUser)
    defaultProfile.save()

# This whole song & dance is due to pickle being insecure
# The SafeUnpickler classes were largely derived from
# http://nadiana.com/python-pickle-insecure
# This code also lives in carbon.util
if USING_CPICKLE:
  class SafeUnpickler(object):
    PICKLE_SAFE = {
      'copy_reg': set(['_reconstructor']),
      '__builtin__': set(['object', 'list']),
      'graphite.intervals': set(['Interval', 'IntervalSet']),
      'graphite.render.datalib': set(['TimeSeries']),
      'collections': set(['deque']),
    }

    @classmethod
    def find_class(cls, module, name):
      if not module in cls.PICKLE_SAFE:
        raise pickle.UnpicklingError('Attempting to unpickle unsafe module %s' % module)
      __import__(module)
      mod = sys.modules[module]
      if not name in cls.PICKLE_SAFE[module]:
        raise pickle.UnpicklingError('Attempting to unpickle unsafe class %s' % name)
      return getattr(mod, name)

Is there is any chance I still have old files during the upgrade (pip install upgrade graphite-web)?
How can I properly upgrade or fully remove it and install it again?

Please show output of
pip show -f graphite-web

https://gist.github.com/mrh666/0ea1eb1f195d94bac2ad

Location: /usr/lib/python2.6/site-packages
So, it's in under /usr/lib/python2.6/site-packages and not in /opt/graphite
Try to reinstall it in /opt/graphite:
pip uninstall graphite-web
pip install graphite-web --install-option="--prefix=/opt/graphite" --install-option="--install-lib=/opt/graphite/webapp"

Maybe you need to do the same for carbon and whisper too.

deniszh, thank you!
"pip uninstall graphite-web" <- sure doesn't delete files under  /opt/graphite/webapp, so I removed them manually.
Then "pip install graphite-web --install-option="--prefix=/opt/graphite" --install-option="--install-lib=/opt/graphite/webapp" did the clean install. All those exceptions are gone after "service httpd restart".

yw
Not sure why default install not going to /opt/graphite
pip is odd sometimes, and developing quite fast, though.


[9.x] Document some more return types (removes Symfony 6.3 deprecation notices)

In Symfony 6.3, we've added PHPdoc return types to all methods.
I've run the type patch utility on this repository, to make sure all return types are also defined by Laravel to avoid indirect deprecation notices. I'm not sure about your policy regarding adding return types, so some of these might be turned into a real PHP return type.

I'm not sure about your policy regarding adding return types, so some of these might be turned into a real PHP return type.

Generally, we only use native types into code that may live on user land or to "new" code that gets introduced on the framework.


Merge recent commits from master towards the vester_visual_testing branch

Pull request to trigger a GitHub Action that calls Vester to visually test the branch attempting to merge.
This master branch is synced with emilk's egui master branch and this is an attempt to merge it with the vester_visual_testing branch, which is updated to the commits up to Sep 24th 2024.
By creating this pull request a GitHub Action will start, that will request Vester to compile, run and visually analyze the "egui demo app" resulting from this up-to-date branch, and report if its additional commits, obtained from the recent sync, alter the demo app's visuals. In case they don't it will be safe to merge this master branch with the vester_visual_testing branch, since we will have certainty that the commits that the master branch has ahead of the vester_visual_testing branch do not cause visual alteration to its resulting application.

Vester's scan of the merging "master" branch's "egui demo app" visuals failed because some recent commits altered some elements of the "Misc demos" and "Window resize test" tabs.
These are the differences:

They seem not to be unintended, and do not cause any issue with the application from a GUI standpoint, therefore I've set them as the new expected visuals and re-run the GitHub Action to test the "egui demo app" visuals with these new settings.
The scan successfully passed.
I'm proceeding with the merge.


[BUG] Connect-PNPOnline -interactive fails when MFA is enabled on 365 tenant

Reporting an Issue or Missing Feature
Reporting an issue
Expected behavior
Attempting to connect with MFA enabled on the tenant - Expect to login successfully.
Actual behavior
When attempting to connect, initial Microsoft authentication page is displayed, however when it reaches the URL "https://login.microsoftonline.com/common/reprocess?..." after completing the MFA process, the screen is blank and does not continue beyond this
Steps to reproduce behavior
Connect-PnPOnline -Url https://hattrick.sharepoint.com -Interactive
What is the version of the Cmdlet module you are running?
1.10.0
Which operating system/environment are you running PnP PowerShell on?
Windows

@Luquos connecting like this in PnP PowerShell is used millions of times a day. I use it myself regularly as well. If this would be broken for some reason, I would expect many more issues to be reported, which is not the case. Can you check if it can be something on your side? Perhaps a proxy on your connection? Or some kind of weird policy? Try a different connection and/or machine and see if you get a different result. I.e. try Azure Cloud Shell to rule out your machine.

Interesting. Technically PnP PowerShell uses MSAL to authenticate, which is nothing different from most other modern modules and applications such as Outlook and the rest of the Office products. Are you able and technically capable to capture a Fiddler trace while connecting and examine that? It could provide more insight into why it gets stuck. If you're not able to analyze it yourself, feel free to share it with me privately (don't share it here). It will contain your login information though, so if you do so, please use a temporary test account to log on with and disable or change password after capturing the Fiddler trace.

Any progress on this one @Luquos from your side? I cannot reproduce this from my side, so I'm afraid there's nothing I can do to resolve this for you.

Closing this as no further feedback received.

I too have encountered this issue, and had to adjust my scripts to -UseWebLogin instead of -Interactive which does work.
After logging in I get the MFA challenge which I have completed both in the Authenticator App and using SMS but the auth box stays on the same page waiting to complete. I did try using Fiddler to check and it does appear that the result of the MFA is being returned to the client just not picked up


binutils conflicts with itself (due to .dev output)

Issue description
So it cannot be installed:
$ nix-env -iA nixos.binutils
installing ‘binutils-2.27’
building path(s) ‘/nix/store/g6iksgvyza9470w1rsvk2mhxkk747cp0-user-environment’
Wide character in die at /nix/store/64jc9gd2rkbgdb4yjx3nrgc91bpjj5ky-buildenv.pl line 79.
collision between ‘/nix/store/xank34knz9h7351m1km1x4bz4p8l1x46-binutils-2.27/bin/addr2line’ and ‘/nix/store/nqdk6dzwwhrq63gmiy4hm473z51yxcan-binutils-2.27-dev/bin/addr2line’; use ‘nix-env --set-flag priority NUMBER PKGNAME’ to change the priority of one of the conflicting packages
builder for ‘/nix/store/idd9rbr78x1i0kipk5qvf6ylq85f5jlk-user-environment.drv’ failed with exit code 255
error: build of ‘/nix/store/idd9rbr78x1i0kipk5qvf6ylq85f5jlk-user-environment.drv’ failed

Technical details

System: NixOS 16.09

Is that an old nix? I think the newer ones have the logic to install only relevant outputs. I don't know if that code is only on master though.

It's the default Nix on NixOS 16.09 (as of a couple of days old, c02b9b93f6bff1dcebdb7b7148e405aec6e5df66):
$ nix-env --version
nix-env (Nix) 1.11.4

Hm, damn. I guess the workaround in the meantime is to install binutils.out.

@dezgeg: Does that work for you? I get
$ nix-env -iA nixos.binutils.out
installing ‘binutils-2.27’
building path(s) ‘/nix/store/a08pjik14zzdf7qifc9n5vnzxgm1l113-user-environment’
Wide character in die at /nix/store/64jc9gd2rkbgdb4yjx3nrgc91bpjj5ky-buildenv.pl line 79.
collision between ‘/nix/store/nm9r5lymydnsrlxjn30ym2kix6mbyr19-binutils-2.27/bin/addr2line’ and ‘/nix/store/37hyhsfqz2wdd6gi9ns7c5h7f6gsnxbm-binutils-2.27-dev/bin/addr2line’; use ‘nix-env --set-flag priority NUMBER PKGNAME’ to change the priority of one of the conflicting packages
builder for ‘/nix/store/f5yxclwyfpdskpg7h6y8ismab5qqzdqn-user-environment.drv’ failed with exit code 255
error: build of ‘/nix/store/f5yxclwyfpdskpg7h6y8ismab5qqzdqn-user-environment.drv’ failed

D'oh, I forgot that it doesn't actually work...

Why does the dev output have binaries, those should only be in the out output.

It has a symlink ${binutils.dev}/bin -> ${binutils.out}/bin, probably that could be removed if anything that depends on is fixed.

A quick grep seems to indicate that the dev output is only used for include paths ${binutils.dev}/include

I'll just throw it in, in case it's not obvious for someone — a nasty workaround is to put some symlinks into your ~/.local/bin directory. On my machine, the following command does it:
cd ~/.local/bin
for x in /nix/store/dxmcix1wv08fad29nmwh07y3hjply7y3-binutils-2.27-dev/bin/*; do ln -s $x $(basename $x); done

I somehow missed this.  I believe the symlink is useless, especially now that we have binutils.out as the default.


Added 'disable_half_float' project setting to the class reference.

Just forgot to do this in the original PR.
The setting may end up being temporary anyway.. if it does fix the problem we will add some detection / hard coding for iOS. It may even end up being worth retaining afterwards anyway in some form, for debugging purposes, and to cover our bases in case of future hardware problems.

Thanks!


Issues with upload/download

Hello Dean, we have just made this sample repo.
https://github.com/Ansh-Rathod/icloud_sample_rpo
Please check Readme, we reported there test results which show issues with both upload/download.
Demo is simple, we fetch an image from web and then upload/download it to/from icloud.
We tested on emulator (ios 15.5) and real device (iphone pro 13 ios 16).
Can you please check whether this is a bug or our code?

Forgot to mention that the same device where upload stalls can upload files to icloud when using whatsapp backup or system apps

Other curious thing noticed. If we tap Upload button twice, the upload progress work on the second call (on first tap it always returns 100% instantly).
Download progress always show 100% instantly.
https://github.com/Ansh-Rathod/icloud_sample_rpo

@deansyd please look at new example (vid below).
It seems that upload issue is only with iOS16. This is why we got those differences between emulator and real phone. Emulator was ios15 and real phone was ios16.
You can check code here:
https://github.com/Ansh-Rathod/icloud_sample_rpo
With example, we tap buttons in following order:

fetch file from web
check that file is saved locally (it exists)
upload file to iCloud (it will upload and also delete file from device)
check that file is not saved locally (it doesnt exist)
download file from iCloud
check that file is saved locally (it exists)

It works with ios16, it doesnt with ios16 (upload stalls).
https://user-images.githubusercontent.com/83366073/212097260-b1d95669-9c28-492f-8bee-86756a9682c5.MP4

Made more tests, it works with:

ios 15.5

It doesnt work with:

ios 16.1 (both emulator and device)
ios 15.0
much older ios versions (ie. ios 14)

@iosephmagno Hi, why closed? Any new? Think you


docs: Invenio v3.0 related updates

Initial release of Invenio v3.0 related documentation
updates. (addresses #3555)

Signed-off-by: Tibor Simko<EMAIL_ADDRESS>

Looks good (just check https://github.com/inveniosoftware/invenio/pull/3561#discussion_r44278010). :shipit:

:shipit:


workflows/eval: Add the step summary as a comment

See it in action https://github.com/azuwis/nixpkgs/pull/7
Make https://github.com/NixOS/nixpkgs/pull/360339 more discoverable.

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/)
25.05 Release Notes (or backporting 24.11 and 25.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
[ ] (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.

I am not in favor of this change. It will make the GH search results even more useless, since searching for a package would then also find all dependency updates.

That's also a good point. Less close it.


[FEATURE]: Import brew from local file

Basic idea:
We save files to .txt, with a very specific syntax, it'd be nice if we could add some import function to the user page, to load into HB a brew using the .txt as base (after making sure no malware is in it, ofc)
For future reference, the .txt syntax looks like this:
```metadata
title: title of brew
description: >-
    Description text
tags: []
systems: []
renderer: V3
theme: 5ePHB

```

```css
.page {
	background:blue;
}
```
# Brew text

I guess what is needed is a function that takes the txt file, gives it a brew id and checks for missing metadata and fixes it, and stores it into the default db, then open that brew in edit page.
This of course would need to stop if the file will not render correctly, for example, if brew name is missing, or if some other syntax is wrong, and output an error message and not store it at all.

I believe last time this type of feature was discussed, the preferred outcome was populating /new with the result, rather than allowing direct entry to the DB. Then when the user hits SAVE, all of the usual checks take place.

that is an interesting idea, but would it replace the new brew content?


Cannot Add Effects and Ranges

Describe the bug
Foundry 0.7.9
After creating compendiums as instructed in the instructions, when it comes time to create Abilities, it is not possible to save the effects or ranges.
To Reproduce
Steps to reproduce the behavior:

Go to Items
Click on Create Item
Create a new ability
Click Add Effect
Click Add Range
Attempt to save

Expected behavior
The Range saves and brings you back to the Effect window, whereupon you can click Save there and be returned to the Ability.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows on an AWS hosted Ubuntu server
Browser Edge
Version 87.0.664.75

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

My apologies for the delay on this. You should now be able to add effects, and ranges. I've also added compatibility with So Nice Dice for visual dice rolling. Please let me know if you experience any other issues.

This seems to be working fine now.  Have added numerous effects, ranges, etc.


jacobian_warping computation of right-multiplication

Thanks for your great work. I have a question related to the jacobian_warping：
In models/algorithms.py, in the function
def compute_jacobian_warping(p_invdepth, K, px, py):
...
dx_dp = torch.cat((-xy,     1+x2, -y, invd, O, -invd*x), dim=2)
dy_dp = torch.cat((-1-y2, xy,     x, O, invd, -invd*y), dim=2)
...
The jacobian (dx_dp,dy_dp) here is the left-multiplication jacobian.
Are the left-multiplication and right-multiplication jacobian the same?
I am not familiar with right-multiplication, Could you explain it? Thanks a lot.

No, it won't be the same. The jacobian of the right-multiplication will requires an additional adjoint operation of the Lie-group, which you can refer to http://www.ethaneade.org/lie.pdf, section 2.3, 3.3, & 3.4.

Thank you very much for sharing and helping me to deepen my understanding of Lie-Group.
However, according to my derivation, the {Delta xi}^{-1} calculated according to formulas (28) and (11) in the paper should be multiplied to xi_k by left-multiplication. Is there any problem with me?

Not quite sure what is your derivation. But the original equation (5) does not have xi_k to be part of the chain rule. I think you may solve the jacobian from a different linearization.

Thank you zhaoyang, I've figured it out. I confused the optical flow method with the direct method before.


Introduce resolution strategy to deactivate global substitution rules

Gradle implicitly registers dependency substitution rules for all configurations in the whole build tree to find projects in other included builds. These rules are always active by default.
There are however cases, where a certain configuration should not apply these rules when resolving. For example, if a binary version of a module should be discovered that is also represented by a project in the same or another build.
One use case for this is to compare the previous published Jar of a module with the latest locally built Jar of the same module.
Context

In a project, we needed to resolve the local and the previously published versions of our libraries next to each other to determine if new versions need to be published.
We can use reflection to set the DefaultResolutionStrategy.globalDependencySubstitutionRules field (private final 😬) to DependencySubstitutionRules.NO_OP. This solved our use case.
This PR proposes to make exactly this behaviour available through public API.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

I am with Louis that it sounds like a detached resolver. However, I think it makes sense that the detached resolver builder can declare what it inherits from. For example, when building the resolver, you should be able to say "inherit substitution rules" or "inherit settings repositories", etc. Basically, if you say nothing you'd get something which is completely isolated, like the current internal API, but we could make it more configurable.

The current internal API also has the global substitutions on (otherwise we would have used that). And I guess it has to, because there would be no way to turn it back on (or "inherit" it).
This PR is adding a way to turn these substitutions on/off, which fits the current pattern of doing such things in the resolution strategy.
It won't hinder the development of a complete public "detached resolver" API in the future. I don't think these two topics need to be mixed.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Align project with new FINOS governance

See https://github.com/finos/community/tree/master/governance
Added CONTRIBUTING.md to root folder, to be more visible

Thanks Maurizio


About length of nonce

I find out some implementations of chacha20 have " XNonceSize = 24".
eg.
const (
// KeySize is the length of ChaCha20 keys, in bytes.
KeySize = 32
// NonceSize is the length of ChaCha20 nonces, in bytes.
NonceSize = 8
// XNonceSize is the length of XChaCha20 nonces, in bytes.
XNonceSize = 24
)
https://godoc.org/github.com/codahale/chacha20
There is not a 12B nonce .
My question is:
what is the XNonce?
And why we have a 12B nonce?

The original ChaCha20 has 64-bit nonce: http://cr.yp.to/chacha.html
The IETF version has 96-bit nonce.  You may refer to RFC 7539 section 2.3
https://tools.ietf.org/html/rfc7539#section-2.3


Remove support for Kafka 0.8, fixes #66

Codecov Report

Merging #69 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #69   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           8      4    -4
  Lines          22     11   -11
=====================================
- Hits           22     11   -11

Impacted Files
Coverage Δ

...det/spark/kafka010/writer/DStreamKafkaWriter.scala
100% <ø> (ø)

...det/spark/kafka010/writer/KafkaProducerCache.scala
100% <ø> (ø)

...nfradet/spark/kafka010/writer/RDDKafkaWriter.scala
100% <ø> (ø)

...thub/benfradet/spark/kafka010/writer/package.scala
100% <ø> (ø)

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 1c1a68c...798aba2. Read the comment docs.


kubernetes_sd: Allow attaching node labels for endpoint role

The Kubernetes service discovery can only add node labels to targets from the pod role.
This commit extends this functionality to the endpoints and endpointslice roles.
Follows up on: https://github.com/prometheus/prometheus/pull/10080
Related to: https://github.com/prometheus/prometheus/issues/9510

Conflict should be gone now. Yes, I tested this locally as well :)

Awesome! Amazing work! 💯


initiate transfer out implementation

As an end-user, I need to be able to initiate a transfer out from the review page.
Our existing FiatConnect partners do not require a waiting period between creating a transfer and sending the crypto for a transfer out. (Should confirm this explicitly with BitSSA and PayChant before beginning work on this.) So let's avoid over-optimizing for potential future partners that do require this, and add in the extra complexity if/when the need arises.
AC: Tapping "confirm" on the transfer review screen should

submit a transfer out request to the provider
send crypto to the provider
show a spinny wheel until the tx is confirmed
show a catch-all error screen with a retry button if the request to initiate a transfer fails, or if the tx fails for some reason
show a success screen once the tx is confirmed, similar to this one for transfers in:

ENG:

Verify using the mock provider in alfajores

I feel the actual initiating a tranfer out request to the provider should happen on the prev screen, it would be really weird if that request failed and we showed the user "Your cUSD is on the way"


Updates to IGC Helm template

Updates the IGC Helm template for VDC chart to fix #976:

Points to static ODPi GitHub repo location for obtaining IGC repository connector (for now)
Simplifies the IGC template (removes duplicate sections across conditional statements)
Updates ConfigMap, Values and Proxy Init job to use new parameters of the connector

Note: the updated template will not actually work until a new Egeria (OMAG Server Platform) Docker image has been built and released that uses the new packaging mechanism included in #964.

Have started a new build of images from master (official builds/id not quite ready yet).

I tested the contents of this PR locally against a local Docker image built against master, and the PR worked prior to submitting; but please do (/ happy to) re-test once we have the latest images publicly available on Docker Hub 👍

Have hit some issues with building (inc resource, travel etc) but FYI current block is
https://github.com/odpi/egeria/issues/1006
Will try and investigate when time allows, but I think we need to hold off merging this change until the base images are in place

Will try and investigate when time allows, but I think we need to hold off merging this change until the base images are in place

That's fine, just be aware that the IGC adapter code is already removed from this repo, so the base images will no longer have any IGC connectivity embedded once built & put into Docker Hub.  (So we should try to merge these very shortly thereafter for continuity of the IGC piece 😉)

I've now made the other nec essary changes so suggest we merge these all together

@planetf1 I'll let you "orchestrate" the merges, if that works? 😄

Yes, and I'll also deploy/test


CompletionItem needs information to position the cursor

I couldn't find a way to tell through the protocol that the client should move the cursor to a certain position after applying a proposal.
Simple use case, would by applying a function call, where the cursor should sit between the parentheses afterwards. E.g. foo(|)

I think this is covered by the snippet support in CompletionItem in version 3.0 of the protocol? Also see #236.
CompletionItem: {
  insertTextFormat: InsertTextFormat.Snippet
  textEdit: {
    range: ...
    newText: foo($1)
  }
}

@dbaeumer I think this can be closed?

Agree. Thanks


New version: ValueShapes v0.5.1

Registering package: ValueShapes
Repository: https://github.com/oschulz/ValueShapes.jl
Created by: @oschulz
Version: v0.5.1
Commit: 7b2001b1299f5801b56878bfa432aa4b37d6d16b
Reviewed by: @oschulz
Reference: https://github.com/oschulz/ValueShapes.jl/commit/7b2001b1299f5801b56878bfa432aa4b37d6d16b#commitcomment-35593672

Your new version pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request during the next cron job.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]


interceptBufferProtocol but fallback to default

Hey there!
I'm trying to intercept 'file' protocol so that a different index.html is served depending on some variables. I currently have something like this:
  var url = 'file://' + __dirname + '/index.html';

  // Replace index.html with custom one
  electron.protocol.interceptBufferProtocol('file', function (request, callback) {
    if (request.url === url) {
      callback({
        data: customHTML,
        mimeType: 'text/html'
      });
    } else {
      // ... Fall back to default??
    }
  }, function (err) {
    if (err) throw err;
  });

However, this breaks browser features like Image and other media (audio/video). I can add a custom fs.readFile and send the result to callback({ data, mimeType }), which works with image.onload, but errors are not handled as they should be by the browser and image.onerror won't get called.
Is there another approach to replacing the HTML dynamically from file:// protocol? Or a way to fall back to default protocol behaviour? Or some means of only targeting a specific URL?

Invoking callback with chromium net_err_codes does call onerror event handler. But if you are just serving local files why not use interceptFileProtocol ?

You can also use webRequest.onBeforeRequest API to redirect the request to your custom HTML file.

Thanks @zcbenz, it seems like the one issue with that approach is that it changes the "base directory" for resources like <img src="">, so that now the browsers loads them relative to the new path/to/custom/index.html. I will keep poking around, though.

You can override that behavior using the base tag.
<base href="file://whatever/">

or
<base href="http://whatever.com/">


Laravel 7 compatibility

seems to be working properly

@martinlindhe can you merge this? thanks @code2prog

@martinlindhe please merge 🙏

We need this! 😢

@martinlindhe +1 on could you please merge :)

We need this! 😢

you can make a fork of my repository and use it this way without waiting :)

Merged https://github.com/martinlindhe/laravel-vue-i18n-generator/pull/113 (same change)

Sorry for being slow on this.


Configurable cache path

Having the cache path created in the current directory may not be convenient. bashful could be invoked from a directory where the user does not have write permissions, and the execution would fail because the cache directory can't be created.

Would it make sense to also default the cache path to a writable folder such as /tmp?

the cache path cli override makes sense, but when running multiple projects with bashful it's nice to keep the cache close to the project dir instead of combined in one spot (like /tmp). Thanks for the addition!


[SPARK-6743][SQL] Fix empty projections of cached data

Merged build triggered.

Merged build started.

Test build #32759 has started for   PR 6165 at commit aad7eab.

Test build #32759 has finished for   PR 6165 at commit aad7eab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32759/
Test PASSed.

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33270/
Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #33272 has started for   PR 6165 at commit 4fad158.

Test build #33272 has finished for   PR 6165 at commit 4fad158.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33272/
Test FAILed.

Merged build finished. Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #33290 has started for   PR 6165 at commit 4fad158.

Test build #33290 has finished for   PR 6165 at commit 4fad158.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33290/
Test PASSed.


Added json response in benchmark

Benchmark code:
package zhttp.benchmarks

import org.openjdk.jmh.annotations._
import zhttp.http._

import java.util.concurrent.TimeUnit

@State(Scope.Thread)
@BenchmarkMode(Array(Mode.Throughput))
@OutputTimeUnit(TimeUnit.SECONDS)
class HttpRouteTextPerf {

  private val res          = Response.text("HELLO WORLD")
  private val app          = Http.fromHExit(HExit.succeed(res)).whenPathEq("/text") ++ Http
    .fromHExit(HExit.succeed(res))
    .whenPathEq("/plain")
  private val req: Request = Request(Method.GET, URL(!! / "text"))

  @Benchmark
  def benchmarkHttpProgram(): Unit = {
    val _ = app.execute(req)
    ()
  }

}

jmh benchmark results
[info] Benchmark                                Mode  Cnt        Score         Error  Units
[info] HttpRouteTextPerf.benchmarkHttpProgram  thrpt    3  4052918.714 ? 1315668.386  ops/s

Codecov Report

Merging #1091 (38710e4) into main (06c29d2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1091   +/-   ##
=======================================
  Coverage   57.49%   57.49%
=======================================
  Files          73       73
  Lines        2303     2303
  Branches       66       66
=======================================
  Hits         1324     1324
  Misses        979      979

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 06c29d2...38710e4. Read the comment docs.

Can we improve performance by using a special Combine operator that isn't built on fold?

Can we improve performance by using a special Combine operator that isn't built on fold?

After adding composeHttp:
[info] Benchmark                                Mode  Cnt        Score         Error  Units
[info] HttpRouteTextPerf.benchmarkHttpProgram  thrpt    3  7615488.786 ? 3227764.384  ops/s

Benchmark with HttpCombineEval(++):
[info] Benchmark                           Mode  Cnt      Score       Error  Units
[info] HttpCombineEval.benchmarkNotFound  thrpt    3  65230.423 ? 10185.351  ops/s
[info] HttpCombineEval.benchmarkOk        thrpt    3  42583.211 ?  7217.063  ops/s

Benchmark with HttpComposeHttpEval(composeHttp):
[info] Benchmark                           Mode  Cnt       Score       Error  Units
[info] HttpCombineEval.benchmarkNotFound  thrpt    3   77365.635 ? 56534.469  ops/s
[info] HttpCombineEval.benchmarkOk        thrpt    3  106470.738 ? 51292.490  ops/s


pynacl takes ages to build on termux

Is there a better way?

no longer fooling around with termux


Add support for hive-server

Hivemetastore is finished.

[X] hivemeastore
[ ] hiveserver

Offer
@fredrikhgrelland
I would like to suggest make coupled-implementation for hivemetastore and hiveserver.
It means that the module will contain 1 nomad job with two tasks retrospectively. Deploy two pods with integration between.
Why
Conditional rendering (of nomad jobs) will slow down delivery progress.
Next releases we can focus on how to segregate hive running modes into one nomad_job with conditional rendering.
Take a look at branch feature/add-server

This is fine for now. Future iteration might offer three posibilities in this module. server/metastore/combined

@zhenik Is this something we need to consider implementing in the near future?

@claesgill No. I even think we should actively NOT consider this. Hive metastore is a requisite as a catalog service for common "big data" management tools. hive is not and is deeply entrenched in Hadoop. Trying to implement a working hive server to process data will be time consuming and reap no rewards. I suggest closing this issue as won't-do

@claesgill No. I even think we should actively NOT consider this. Hive metastore is a requisite as a catalog service for common "big data" management tools. hive is not and is deeply entrenched in Hadoop. Trying to implement a working hive server to process data will be time consuming and reap no rewards. I suggest closing this issue as won't-do

Ok, thanks's for the update. Closing.


Create file that maps TLAREFs to NaPTANs

This will map the station codes to NaPTAN keys. These can then be used to look up more data about the station such as lat / long.

Created in 38649c2


Custom protocol not registering in OSX 10.10.4

I'm trying to add a custom protocol to my app and the protocol is being added to my plist. However, Safari doesn't associate the protocol with the app even if I open the built app. Is there something I've missed in my config to open the app when a user clicks a link in the browser?
electron-packager: 5.0.1
platform (to and from): OSX 10.10.4
        options: {
            name: 'MyApp',
            dir: 'app',
            out: 'dist',
            version: '0.25.3',
            platform: 'darwin',
            protocols: [{
                name: 'myapp',
                schemes: ['local']
            }],
            arch: 'x64',
            overwrite: true
        }

I fixed the issue. There was some confusion about how to define the protocols. The scheme defines what is recognized as the custom protocol, not the name.
    options: {
        name: 'MyApp',
        dir: 'app',
        out: 'dist',
        version: '0.25.3',
        platform: 'darwin',
        protocols: [{
            name: 'MyApp',
            schemes: ['myapp']
        }],
        arch: 'x64',
        overwrite: true
    }


Update documentation

Documentation is currently PMA focused. Some documentation files aren't even imported. Some modules don't have documentation.

[ ] Create Issues for missing docs sections
[ ] About section
[ ] Move submodules overview to docs
[ ] Add examples

Notes:

Need to create documentation about moving from ptpma (note: PMA prefix is now dropped) and add link to init.py for deprecated routes

Closing this, as we have a first pass of the docs. We can create new Issues for anything that needs doing