Make Frontdoor compatible with XSkipDetailedDiffForChanges

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

The Frontdoor resource needs tweaking to operate correctly with XSkipDetailedDiffForChanges.
Context: bridge is introducing a change to the diffing algorithm that is currently flagged under
XSkipDetailedDiffForChanges but is on the fast track to become turned on unconditionally as it
solves important issues for GCP and AWS provider. This change was introduced in:
https://github.com/pulumi/pulumi-terraform-bridge/pull/1502
When pre-testing this change through Azure tests, TestAccFrontoor started failing. It is expected
that after a successful pulumi up, the subsequent pulumi preview generates a no-changes plan.
With XSkipDetailedDiffForChanges it started to generate an Update plan instead, based on these
attributes:
     CHANGING FROM DIFF_NONE to DIFF_SOME
    ~ "backend_pool_settings.#" => {1 0 false false <nil> false false 0}
    ~ "backend_pool_settings.0.backend_pools_send_receive_timeout_seconds" => {0 60 false false <nil> false false 0}

Investigating this further, the root cause was discovered. The
backendPoolsSendReceiveTimeoutSeconds setting is advertised as defaulting to 60 in the docs, but
this default value is not exposed through normal metadata in the upstream provider but enforced at
runtime. The code references here are:

https://github.com/hashicorp/terraform-provider-azurerm/blob/master/internal/services/frontdoor/frontdoor_resource.go#L1034
https://github.com/hashicorp/terraform-provider-azurerm/blob/master/internal/services/frontdoor/frontdoor_resource.go#L272

It appears that this default value is applied on the read path. Since Pulumi differs from TF in not
automatically applying a Read upon every update, the initial pulumi up deployment fails to apply
the default, while the subsequent deployment does apply the default.
A possible mitigation here would be to lift the 60 default to the regular defaulting mechanism,
possibly by populating SchemaInfo.Default override for this property, which should make Pulumi apply
this default consistently and avoid unexpected plans.
Affected area/feature

Looks like this also happens in terraform with -refresh=false:

terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "3.102.0"
    }
  }
}

# Configure the Microsoft Azure Provider
provider "azurerm" {
  features {}
}

resource "azurerm_resource_group" "example" {
  name     = "resourceGroup"
  location = "West Europe"
}

resource "azurerm_frontdoor" "example" {
  name                = "example476"
  resource_group_name = azurerm_resource_group.example.name

  routing_rule {
    name               = "rule1"
    accepted_protocols = ["Http", "Https"]
    patterns_to_match  = ["/*"]
    frontend_endpoints = ["frontEndEndpoint1"]

    forwarding_configuration {
      forwarding_protocol = "MatchRequest"
      backend_pool_name   = "exampleBackendBing"
    }
  }

  backend_pool_load_balancing {
    name = "exampleLoadBalancingSettings1"
  }

  backend_pool_health_probe {
    name         = "exampleHealthProbeSetting1"
    protocol     = "Https"
    probe_method = "HEAD"
  }

  backend_pool {
    name = "exampleBackendBing"
    backend {
      host_header = "www.bing.com"
      address     = "www.bing.com"
      http_port   = 80
      https_port  = 443
    }

    load_balancing_name = "exampleLoadBalancingSettings1"
    health_probe_name   = "exampleHealthProbeSetting1"
  }

  frontend_endpoint {
    name      = "frontEndEndpoint1"
    host_name = "example476.azurefd.net"
  }
}

I wonder if setting the property as computed would fix this, as the provider is giving a value for a non-computed property here.

Opened https://github.com/hashicorp/terraform-provider-azurerm/issues/25911 upstream, will try to patch it on our side and ignore in the test if not.


A library is packaged in a way a service part of the injected module in a library is added after the module.

🐞 bug report
Is this a regression?
No it was fine in the past. It was fine with Angular 7 and 8. This is happening since updating to Angular 9.
Description
Getting: Cannot access 'someService' before initialization error when I the updated library.
I went and checked the build file and I am able to get it running with no issue when I moved the service anywhere before the module that is injected to.
I moved around the order of the services since it was only affecting the providers. Once I re-build the library it's another provider that gets added at the end of the package file.

Ivy Workspace:

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.901.9
@angular-devkit/core         9.1.9
@angular-devkit/schematics   9.1.9
@schematics/angular          9.1.9
@schematics/update           0.901.9
rxjs                         6.5.4

ng-packagr 9.1.5

Hi @smlee, without a reproduction it would be hard to identify the issue. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

I think I fixed the building issue by refactoring the services that were considered circular to have their own entry points.

Closing as it seems that the issue has been resolved by the user.


[Feature] Provide the ability to monitor all the components of vela

For now, all the components of the vela is deployed in ONE hub-cluster, so the hub-cluster is a must to provide more high SLA.

The apiserver and the core controller are needed to monitored with some metrics, e.g: cpu, memoery, netIO, diskIO and QPS and etc
The components of the hub-cluster are needed to monitored, e.g: etcd, k8s-apiserver, controller-manager, scheduler and kubelet
And more, the core (maybe all) call-trace among the velaux, the apiserver, the core-controller and the cluster-gw is needed to monitor

@Somefive is working on part of this issue.

all metrics addons are available now in the catalog, please refer to https://github.com/kubevela/kubevela.io/pull/882.
I'm going to close this issue, feel free to create a new one if there's new features request.


[Codec] Add new Interface Registry

NOTE: This specific implementation is a mock, although it passes tests it may be changed depending on feedback in #906
Description

Summary generated by Reviewpad on 17 Jul 23 12:37 UTC
This pull request adds a new interface registry with the following changes:

A new file interface_registry.go is added with 155 insertions.
A new file interface_registry_test.go is added with 166 insertions.
The existing file codec_test.proto is modified with 13 changes.

The interface_registry.go file contains the implementation of an interface registry that handles registration and marshaling/unmarshaling of interfaces. It provides methods for registering interfaces, registering implementations, marshaling interfaces into byte arrays, and unmarshaling byte arrays into interfaces.
The interface_registry_test.go file contains tests for the interface registry implementation. It tests the registration of interfaces, registration of implementations, marshaling of interfaces, and unmarshaling of interfaces.
The codec_test.proto file is modified by adding a new message TestProtoStructure2.

Issue
Fixes #906
Type of change
Please mark the relevant option(s):

[x] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Major breaking change
[ ] Documentation
[ ] Other

List of changes

Adds interface registry
Enables the registration of interfaces and their implementations
Enables the marshalling of interfaces
Enables the unmarshalling of marshalled interfaces back into the interface
Adds unit tests covering the above

Testing

[x] make develop_test; if any code changes were made
[x] make test_e2e on k8s LocalNet; if any code changes were made
[x] e2e-devnet-test passes tests on DevNet; if any code was changed
[x] Docker Compose LocalNet; if any major functionality was changed or introduced
[x] k8s LocalNet; if any infrastructure or configuration changes were made

Required Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added, or updated, godoc format comments on touched members (see: tip.golang.org/doc/comment)
[x] I have tested my changes using the available tooling
[ ] I have updated the corresponding CHANGELOG

If Applicable Checklist

[ ] I have updated the corresponding README(s); local and/or global
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added, or updated, mermaid.js diagrams in the corresponding README(s)
[ ] I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

Closing due to more appropriate alternative


Unable to cook - demo content has errors

I'm unable to cook my version of 4.26 with the BugSplat plugin without deleting the Content folder.
  LogAssetRegistry: Error: Package ../../../ShooterGame/Plugins/BugSplat/Content/UI/W_BugSplatButton.uasset has newer custom version of FortniteMain
  LogAssetRegistry: Error: Package ../../../ShooterGame/Plugins/BugSplat/Content/UI/W_BugSplatDemo.uasset has newer custom version of FortniteMain

I get the above error when using it. I can't just resave the assets either, they don't show up in the content browser at all - I can see the other plugin content, just not those widgets.

Hi @highlyunavailable unfortunately the sample for this project is only available in Unreal 5.0 and later. The plugin should work just fine in 4.26 however.

Ah, interesting, I'm sorry I misunderstood. I'll see what we can do...

Yeah... I'd suggest having a demo project and then leave all content out of the plugin. It doesn't seem like I need the plugin though as I can run sendpdbs manually, and it doesn't help with CI all that much (no commandlet to update the crash reporter URL, no auto update of crash reporter URL on postbuild that can see) so I'm probably not going to even use it. I also can't get the crash reporter URL to update from the editor (there's no ini file for packed builds), and it doesn't seem to update the global crash report URL either but that's a different issue.

Not taking any action here, UE4 developers can work around this by deleting the demo content.


Q2 Content performance updates

Fixes #21154
Internal review URL

headings - they should be in sentence case.

Do you know if there is a VS Code tool to automate that? There are a lot of headings in this file.

headings - they should be in sentence case.

Do you know if there is a VS Code tool to automate that? There are a lot of headings in this file.

I don't know of any personally. What I would do is this:

CTRL-F to open the find window
Change mode to regex
Enter ^##  to get all of the headings
Click once back in the doc to set focus, on an empty line so the caret isn't on a word.
CTRL+SHIFT+L to create a cursor on every line matching the find.
SHIFT+END to select the whole line of the header
F1and select Transform to lower case ENTER

Now you only have to go to each header and capitalize the first letter instead of every word.

@adegeo Thanks for the suggestion and @IEvangelist Thanks for the review!


Download fails for file "C'est là le moulin?.JPG"

Log is:
INFO:root:Downloading C'est_l+á_le_moulin?.JPG with width 99999
INFO:root:Requested width is bigger than source - downloading full size
Traceback (most recent call last):
  File "C:\Python27\Scripts\download_from_Wikimedia_Commons-script.py", line 9, in <module>
    load_entry_point('CommonsDownloader==0.2', 'console_scripts', 'download_from_Wikimedia_Commons')()
  File "build\bdist.win32\egg\commonsdownloader\commonsdownloader.py", line 85, in main
  File "build\bdist.win32\egg\commonsdownloader\commonsdownloader.py", line 27, in download_from_file_list
  File "build\bdist.win32\egg\commonsdownloader\thumbnaildownload.py", line 122, in download_file
IOError: [Errno 22] invalid mode ('wb') or filename: ".\\Pictures\\C'est_l\xc3\xa0_le_moulin?.jpg"

Running @symac's patched version with Python 2.7.8 in PowerShell in Windows 8.1 Pro N. Issue seems to be caused by NTFS / Windows not supporting "?" in filenames.

With the proper exception handling introduced in 7dad7e6 and 3e1c728, the severity of this is mitigated since 1. CommonsDownloader will no longer crash when such a problem occurs 2/ The incident will be logged and notified to the user.

Digging more, I have not unearthed any obvious way to filter Windows forbidden characters in Python. Sounds like such a thing should exist though.


FASTLANE_SESSION not working even though set it up

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

FASTLANE_SESSION has expired and has been renewed using fastlane spaceship -u <email>
and set it up to .env but it isn't working
Command executed

execute fastlane spaceship -u <email>
set it up to `.env``
execute fastlane run latest_testflight_build_number
occur error ----> The request could not be completed because:  Authentication error, you passed an invalid session using the environment variable FASTLANE_SESSION or SPACESHIP_SESSION

Complete output when running fastlane, including the stack trace and command used

 [REPLACE THIS WITH YOUR INFORMATION]

Environment

fastlane gems

Gem
Version
Update-Status

fastlane
2.204.3
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

error_highlight
0.3.0

did_you_mean
1.6.1

atomos
0.1.3

rexml
3.2.5

CFPropertyList
3.0.5

claide
1.1.0

colored2
3.1.2

nanaimo
0.3.0

xcodeproj
1.21.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.8.0

terminal-table
1.8.0

plist
3.6.0

public_suffix
4.0.6

addressable
2.8.0

multipart-post
2.0.0

word_wrap
1.0.0

optparse
0.1.1

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

artifactory
3.0.15

babosa
1.0.4

colored
1.2

highline
2.0.3

commander
4.6.0

excon
0.91.0

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

faraday-multipart
1.0.3

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.9.3

unf_ext
0.0.8

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.4

faraday-cookie_jar
0.0.7

faraday_middleware
1.2.0

fastimage
2.2.6

gh_inspector
1.1.3

json
2.6.1

mini_magick
4.11.0

naturally
2.2.1

rubyzip
2.3.2

security
0.1.3

xcpretty-travis-formatter
1.0.1

dotenv
2.7.6

bundler
2.3.7

simctl
1.6.8

jwt
2.3.0

uber
0.1.0

declarative
0.0.20

trailblazer-option
0.1.2

representable
3.1.1

retriable
3.1.2

mini_mime
1.1.2

memoist
0.16.2

multi_json
1.15.0

os
1.1.4

signet
0.16.0

googleauth
1.1.0

httpclient
2.8.3

webrick
1.7.0

google-apis-core
0.4.2

google-apis-playcustomapp_v1
0.7.0

google-apis-androidpublisher_v3
0.16.0

google-cloud-env
1.5.0

google-cloud-errors
1.2.0

google-cloud-core
1.6.0

google-apis-iamcredentials_v1
0.10.0

google-apis-storage_v1
0.11.0

rake
13.0.6

digest-crc
0.6.4

google-cloud-storage
1.36.1

emoji_regex
3.2.3

jmespath
1.5.0

aws-partitions
1.554.0

aws-eventstream
1.2.0

aws-sigv4
1.4.0

aws-sdk-core
3.126.0

aws-sdk-kms
1.54.0

aws-sdk-s3
1.112.0

set
1.0.2

forwardable
1.3.2

logger
1.5.0

pathname
0.2.0

shellwords
0.1.0

cgi
0.3.1

date
3.2.2

timeout
0.2.0

stringio
3.0.1

securerandom
0.1.1

uri
0.11.0

openssl
3.0.0

digest
3.1.0

io-nonblock
0.1.0

ipaddr
1.2.4

io-wait
0.2.1

zlib
2.1.1

resolv
0.2.1

time
0.2.0

open-uri
0.2.0

mutex_m
0.1.1

net-protocol
0.1.2

net-http
0.2.0

ostruct
0.5.2

english
0.7.1

erb
2.2.3

strscan
3.0.1

abbrev
0.1.0

io-console
0.5.11

tempfile
0.1.2

delegate
0.2.0

fileutils
1.6.0

tmpdir
0.1.2

base64
0.1.1

singleton
0.1.1

open3
0.1.1

nkf
0.1.1

prettyprint
0.1.1

pp
0.3.0

find
0.1.1

yaml
0.2.0

psych
4.0.3

 [REPLACE THIS WITH YOUR INFORMATION]

The request could not be completed because:
Authentication error, you passed an invalid session using the environment variable FASTLANE_SESSION or SPACESHIP_SESSION
same problem

same problem

same problem.

same problem

same problem here

I have same issue

Duplicate of #20071

same problem here

same here

The request could not be completed because: Authentication error, you passed an invalid session using the environment variable FASTLANE_SESSION or SPACESHIP_SESSION same problem
Back to 2.176.0, work for me!

help，how to install special version fastlane?

gem uninstall fastlane
sudo gem install fastlane --version {version}

me too same problem

thanks for you ,but I performed fastlane to submit App review prompt Build number does not exist, I cry...

The request could not be completed because: Authentication error, you passed an invalid session using the environment variable FASTLANE_SESSION or SPACESHIP_SESSION same problem
Back to 2.176.0, work for me!

This works, thanks!
Waiting for updates

same problem

downgrading to 2.176.0 worked for me

same problem

✨ Official Update
This appears to be a duplicate of #20071. Going to close this so we can keep everything together in one issue 🙃

same problem  here  . ;/

Same issue here 🙃🙃🙃🙃🙃

Same problem here.


electrum-xzc-3.3.8a7-portable

I cannot send coins because i cannot confirm my transaction at my nano s.
 how can i fix this issue?

Are these mining inputs? If yes, please try the steps here https://github.com/firoorg/electrum-xzc/issues/28#issuecomment-513573689

Are these mining inputs? If yes, please try the steps here #28 (comment)

That also not work. I also get an error in the begin at the nano s ( The derivation path is unusable )  Derivation path (1/8) if i click further then X Reject if you're not sure or Approve derivation path. If i choose Approve derivation path then i see my xzc address in the screen and can Approve.

You need to click approve on the 'the derivation path is unusable' messages, otherwise it will not work.
https://support.ledger.com/hc/en-us/articles/360015739499

You need to click approve on the 'the derivation path is unusable' messages, otherwise it will not work.
https://support.ledger.com/hc/en-us/articles/360015739499

Yes for sure i did that but still i am not able to send my coins because i not get an message to confirm it at my nano s?

I contact ledger and all is ok, their advice me to contact you guys.
I'm upgrade to electrum-xzc-3.3.8a9-portable and still facing the same problem i cannot confirm at mu ledger because nothing popup.
Please help me to solved this issue.

What is the version of Zcoin app on your ledger? How many inputs does this transaction have?

What is the version of Zcoin app on your ledger? How many inputs does this transaction have?

Thank you for your reply.
I run Zcoin 1.5.1 and try to send an low amount but all the time the same result.

In Electrum-xzc, select View -> Show coins. Right-click on only one entry and select Spend, then try sending this one out. Your Ledger should prompt you almost immediately.
If that is successful, try Ctrl+Left Click in the Coins tab to select a maximum of five entries. Then right-click and select Spend. You will now notice a bit of sluggishness while waiting for the Ledger to show the transaction approval prompt.
You can try with 10, 15 or 20 entries at once, but I would not recommend going past 20 since the Ledger will time out.

Jup i try that to same issue :S

Which operating system, what version? How much are you sending? Are you able to get the Ledger to prompt you when you use Tools -> Sign/Verify Message?

Hi,
My system windows 10 fully updated. I try to send one coin two three four five but no succes. I get the Sign/Verify Message screen:

At the Sign/Verify message window, try pasting an address from your Ledger, type in anything in the Message box and click on Sign. Does the Ledger prompt you for with a confirmation?
If the above works, try sending a very small amount (about 0.1 XZC).

Yes that works.

Did you have any solution for me?

Hi @oxzhor , the solution here is to select just few inputs to be spent. Please refer to this image https://user-images.githubusercontent.com/42809091/61594743-e2c57380-ac21-11e9-9aa0-b3733e8be333.png
and select just 2 UTXOs to be spent and try spending it. If successful, double the UTXO number and try spending 4. If successful, try spending 8, etc. Eventually, you'll end up with the number of UTXOs that your hardware can handle.

sadly that is not working like i said before :S

What is the error message?

I cannot confirm the transaction on my ledger.

Why you cannot? Please provide more details so that we can help you.
For now please ignore the "derivation path is unusual" message and approve the tx.

In Electrum-xzc, select View -> Show coins. Right-click on only one entry and select Spend, then try sending this one out. Your Ledger should prompt you almost immediately.
If that is successful, try Ctrl+Left Click in the Coins tab to select a maximum of five entries. Then right-click and select Spend. You will now notice a bit of sluggishness while waiting for the Ledger to show the transaction approval prompt.
You can try with 10, 15 or 20 entries at once, but I would not recommend going past 20 since the Ledger will time out.

I did this with 5 transactions for testing then i get an popup with https://user-images.githubusercontent.com/19476081/99242619-c6c6e780-27ff-11eb-9e60-f7f84c410583.png and no confirm screen on my ledger so i cannot confirm the transaction.

Please strictly follow these steps:

Open the Coins tab (No coins tab? Click View->Show coins)

Thank you for your reply.
Still same no confirm screen at my ledger

Thanks for providing the images, you were doing it ad suggested and it should have worked. For now I can only think of some hardware issue, or, maybe, there is some other application which interacts with you Ledger simultaneously.
We can exclude your Ledger device from the workflow. To do so, please recreate your wallet with your seed words, as described here: https://github.com/firoorg/electrum-xzc/issues/21#issuecomment-472604529

This go to slow, is their an way to speed it up? An couple of UTXOs per time take ages to send them all.

This go to slow, is their an way to speed it up? An couple of UTXOs per time take ages to send them all.

Unfortunately, it's the Ledger hardware limitation. The only way is to recreate a wallet without the device. If recreated without Ledger, more than 2 inputs can be spent.

Unfortunately, it's the Ledger hardware limitation. The only way is to recreate a wallet without the device. If recreated without Ledger, more than 2 inputs can be spent.

Thx i will do that because this take me ages :P

Thx i will do that because this take me ages :P

Credits to https://github.com/firoorg/electrum-firo/issues/21#issuecomment-472604529 perfect i send them all quickly. Topic can be closed.

Credits to https://github.com/firoorg/electrum-firo/issues/21#issuecomment-472604529 perfect i send them all quickly. Topic can be closed.


ktfmt-parent 0.20 is missing on Maven Central

I've been trying to bump the version of ktfmt used by ktfmt-gradle to 0.20 here: https://github.com/cortinico/ktfmt-gradle/pull/8
Seems like the ktfmt is declaring a dependency on a parent POM from ktfmt-parent (why?).
Looks like version 0.20 of ktfmt-parent is missing on Maven Central:
https://search.maven.org/artifact/com.facebook/ktfmt-parent
(Like to the broken CI run here)

Ugh... I ran mvn from the wrong directory, of course, so only ktfmt was uploaded.
Now, rerunning mvn from the correct directory fails because ktfmt cannot be updated.
Looking into this, worst case I'll publish 0.21.

Ok, I managed to trick mvn into uploading only ktfmt-parent.
It's now over at https://repo1.maven.org/maven2/com/facebook/ktfmt-parent/0.20/ .
It might take another few hours to sync to other repositories.

Thanks for reporting!!!


fix: remove aws and gcp detector from SDK

Which problem is this PR solving?
Short description of the changes
AWS and GCP detectors were moved to contrib.
NodeSDK should not depend on contrib resources.
Refs: #2392
Refs: https://github.com/open-telemetry/opentelemetry-js/pull/3018#discussion_r890296983
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Checklist:

[x] Followed the style guidelines of this project
[ ] Unit tests have been added
[ ] Documentation has been updated

Seems CI is broken, OTLPMetricExporter - node with TLS, without metadata - export - should export metrics fails consistent for some reason.

regenerating certificates for GRPC exporters make CI green again


Generic type parameters don't work with Union types

Environment data

Language Server version: v2022.6.30
OS and version: Windows 11: 10.0.22000 Build 22000
Python version (& distribution if applicable, e.g. Anaconda): 3.10.5

Code Snippet

from typing import Type, TypeVar, Union, cast

T = TypeVar("T")

Foo = Union[int, str]

"""
(type alias) Foo: Type[int] | Type[str]
Expression of type "Foo" cannot be assigned to parameter of type "Foo | Type[T@bar]"
  Type "object" cannot be assigned to type "Foo | Type[T@bar]"
    Type "object" cannot be assigned to type "Type[T@bar]"
    Type "object" cannot be assigned to type "Type[int]"
    Type "object" cannot be assigned to type "Type[str]"Pylance(reportGeneralTypeIssues)
                               vvv"""
def bar(return_type: Type[T] = Foo) -> T:
    return cast(T, "")

spam = bar(str)  # (variable) spam: Unknown

Expected behavior
No type error should be raised, and spam s type should be recognized as str, not Unknown.
Actual behavior
Error is raised, and spam's type is not inferred.
Logs

Experiment 'pythonaacf' is active
Experiment 'pythonTensorboardExperiment' is active
Experiment 'pythonSurveyNotification' is active
Experiment 'PythonPyTorchProfiler' is active
LSP Notebooks experiment is disabled -- not in treatment group
> conda info --json
Python interpreter path: /usr/bin/python
> ~/.pyenv/versions/3.9.13/bin/python ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/interpreterInfo.py
> /bin/python ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/interpreterInfo.py
> /sbin/python ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/interpreterInfo.py
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/interpreterInfo.py
> /usr/sbin/python ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.8.1/pythonFiles/interpreterInfo.py
Starting Pylance language server.

There are a couple of problems with your code here.
First, Foo is a type alias. At runtime, it is a typing.Union object. Normally unions are used only in type annotations because it is a special form that has no meaning in value expressions. But you are attempting to use this typing.Union object as a default argument value for parameter return_type. I presume that's not what you intend to do here. Perhaps you meant to use int as the default argument?
def bar(return_type: Type[T] = int) -> T: ...

Or perhaps your intent was to constrain T to be either int or str but then default to one of them?
T = TypeVar("T", int, str)
def bar(return_type: Type[T] = int) -> T: ...

Second, assigning default arguments for a parameter defined with a TypeVar is problematic from a type safety standpoint. Other type checkers like mypy don't allow this at all. Pyright (the type checker that pylance is built upon) allows it, but it is very restrictive in what it allows. The recommended approach here is to use an overload.
T = TypeVar("T", int, str)

@overload
def bar(return_type: type[T]) -> T: ...
@overload
def bar() -> int: ...
def bar(return_type: type = ...) -> object:
    return return_type

I think I understand your point. My intent is to create a function which functions something like a type guard. You pass in some value, you pass in a type, and you get assurance that the return value from the function is the type you wanted.
def ensure_type(val: Any, _type: Type[T]) -> T:
    ... # do type checks at runtime
    return val

But with this particular function, I don't want the user to have to specify _type every time when there is a particular one that is used nearly every time. It's a custom Union type.
I'm surprised that a custom Union type alias isn't treated as a first-class type here.
I do think the overloading workaround is suitable for my case:
@overload
def ensure_type(val: Any, _type: Type[T]) -> T: ...
@overload
def ensure_type(val: Any) -> CustomUnion: ...
def ensure_type(val: Any, _type: Type = CustomUnion) -> object: # is object significant here vs omitting the return type?
    # runtime typechecks
    return val


Support for Unit

Could you please add support for the Unit type ()? I tested using Option<()>, but even when I set the value to Some(()) and export it, it'll result in ~.

Can you elaborate your use case and write a test case?

My use case was to have an optional boolean in a config file. Either the value exists (meaning true) or it doesn't (= None) meaning false. The example code is the following:
extern crate serde;
#[macro_use]
extern crate serde_derive;
extern crate serde_yaml;

#[derive(Serialize)]
struct Foo {
    bar: Option<()>,
}

fn main() {
    let ser = serde_yaml::to_string(&Foo { bar: Some(()) }).unwrap();
    println!("{}", ser);
}

The use case was invalid when I learned about the field attribute #[serde(default = "default_function")]. With that I was able to rewrite the code to:
#[derive(Serialize)]
struct Foo {
    #[serde(default = "bar_default")]
    bar: bool,
}
fn bar_default() -> bool {
    false
}


Cache Audio from url, and play without internet

Hey there guys, I want to play audio files from the internet using jcplayer. I know it is possible with the function createFromURL() can anyone tell me how do I cache the song played from url and store it on the apps private cache, so that later I can retrieve it from the cache without internet connectivity.
This could be really useful to me to allow users to play songs without internet, once they have played/downloaded the song once when the are connected, later they can play without internet,

@jeancsanchez  I have been working on this the last few weeks to test this feature out. I was able to do it by following the steps below:
If a user wants to play an audio file from Url instead of Passing the Url of the server online, I passed the Url of the cached data in the createFromUrl(), I have edited the sample project to try this out.

Created a Class called Downloader that downloads the song to cache dir of the app.
In the MainActivity, I created an anonymous object of the Downloader class and pass the Cache Dir of the App
in the Downloader Class, I have a method that downloads the song to the cache
Once the song is in cache I createFromUrl by passing the cahce dir in the url, in the MainActivity.

and it works,
Limitations and Further enhancement that can be done:

Player waits until song is downloaded then Starts Playing the songs.
I Did not do a check on if File Exists in cache so right now it downloads the song then play it everytime the player is loaded: enhancement would be to check if file exists in cache then don't download file, just play it directly.
Note: In the MainActivity: the object is Created in a Thread, since i cannot make network connections in the Main UI: enhancement would be to have a better way if possible to cache the song outside the MainActivity.

CODE:
MainActivity.java
songurl="http://<IP_ADDRESS>/songs/";
filename="title.mp3";
//ADDED DOWNLOADS
try{
int SDK_INT = android.os.Build.VERSION.SDK_INT;
if (SDK_INT > 8)
{
StrictMode.ThreadPolicy policy = new StrictMode.ThreadPolicy.Builder()
.permitAll().build();
StrictMode.setThreadPolicy(policy);
new Downloader(getCacheDir(),songurl,filename);
}
jcAudios.add(JcAudio.createFromFilePath("in Cache",getCacheDir().toString()+"/"+filename));
}
catch(Exception e) { e.printStackTrace(); }
Downloader.java
import android.util.Log;
import java.io.File;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.net.HttpURLConnection;
import java.net.MalformedURLException;
import java.net.URL;
/**

Created by Peace on 27/03/2018.
*/

public class Downloader  {
public Downloader(File getDir, String songurl,String filename)
{
download(getDir,songurl,filename);
}
private void download(File getDir, String songurl,String filename) {
try {
//set the download URL, a url that points to a file on the internet
URL url = new URL(songurl+""+filename);
        //create the new connection
        HttpURLConnection urlConnection = (HttpURLConnection) url.openConnection();

        //set up some things on the connection
        urlConnection.setRequestMethod("GET");
        urlConnection.setDoOutput(true);

        //and connect!
        urlConnection.connect();

        //set the path where we want to save the file
        File SDCardRoot = getDir;
        //create a new file, specifying the path, and the filename
        //which we want to save the file as.
        File file = new File(SDCardRoot,filename);
       // Log.d("LOG","DIR Store "+SDCardRoot + file);

        //this will be used to write the downloaded data into the file we created
        FileOutputStream fileOutput = new FileOutputStream(file);

        //this will be used in reading the data from the internet
        InputStream inputStream = urlConnection.getInputStream();

        //this is the total size of the file
        int totalSize = urlConnection.getContentLength();
        //variable to store total downloaded bytes
        int downloadedSize = 0;

        //create a buffer...
        byte[] buffer = new byte[1024];
        int bufferLength = 0; //used to store a temporary size of the buffer

        //now, read through the input buffer and write the contents to the file
        while ( (bufferLength = inputStream.read(buffer)) > 0 ) {
            //add the data in the buffer to the file in the file output stream (the file on the sd card
            fileOutput.write(buffer, 0, bufferLength);
            //add up the size so we know how much is downloaded
            downloadedSize += bufferLength;
            //this is where you would do something to report the prgress, like this maybe
            updateProgress(downloadedSize, totalSize);

        }
        //close the output stream when done
        fileOutput.close();

//catch some possible errors...
} catch (MalformedURLException e) {
e.printStackTrace();
//Toast.makeText(this," E"+e,Toast.LENGTH_SHORT).show();
} catch (IOException e) {
e.printStackTrace();
//            Toast.makeText(this,"2 E"+e,Toast.LENGTH_SHORT).show();
}
catch(Exception e){
e.printStackTrace();
}
}
private void updateProgress(int downloadedSize, int totalSize) {
    Log.d("DOWNLOADER","done " +downloadedSize+ " Total "+totalSize);
}

}

Good job :D


npm Vulnerability in logger package

We're using this package as a dependency in our app, but it seems it's using a vulnerable version of package winston.
When updating this dependency, the packages' test command fails.
I'm assessing the work needed in fixing this for a possible PR, but is anyone currently working (or aware of) updating the dependencies of this package to pass tests using the latest, secured version of Winston?
Refs:
https://snyk.io/test/npm/hmpo-logger/4.1.3
Thank in advance!
Martin

I've been looking at this today, either upgrading the code to use Winston 3, or not using Winston as a logging backend at all

Great, Paul.
Is it something you're looking to update the package with as it's holding up a couple of stories for now, so thinking of options...
If you reckon it's something you can have fixed fairly soon (likely more easily than me as you're familiar with it!) then we can hold those stories for a bit...
Alternatively I'd be able to make a new work item to help take a look too, and try to get that in our current sprint to free those up. Depend show much work you reckon you'll need to do/how long. it could be - what would you suggest?
Thanks for picking this up so soon too 👍

major version 6.0.0 has been released that no longer used winston. This version should be drop in compatible, but the pretty-print console logging will look slightly different.


GeneratePackageOnBuild=True: Ability to also specify package folder

Would be great if I could specify a path as well... Or maybe if a path is specified packages are just built....

This issue was moved to NuGet/Home#4480


Optional parameter type

🔍 Search Terms
"optional", "parameter"
✅ Viability Checklist

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[X] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
[X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
I would like a type that describes an optional parameter - similar to the existing ?, but set as a type rather than as a parameter modifier. This would allow parameters to be conditionally optional.
JavaScript already allows for optional parameters (defaulting the value to undefined) so this wouldn't be a change to existing JavaScript code. The compiled JavaScript code wouldn't change in any way. Since the effect can be simulated (see Use Cases), this feature request is mainly TypeScript syntactic sugar.
📃 Motivating Example
When defining a class, it can be useful to make a parameter conditionally optional.
For example, consider the following class. it has a single method that takes 2 parameters: a string (the key defined in a map) and the type of data that map has defined.
class Foo<TMap = {}> {
    doSomething<K extends keyof TMap>(key: K, datum: TMap[K]) {
        console.log({ key, datum });
    }
}

At the moment, the doSomething method requires 2 parameters even if TMap[K] evaluates to undefined.
const foo = new Foo<{ one: string, two: undefined }>();
foo.doSomething("one", "a");        // valid
foo.doSomething("one");             // ts(2554) as expected
foo.doSomething("two", undefined);  // valid
foo.doSomething("two");             // ts(2554) sadly

A way of telling TypeScript that the parameter may be optional in some circumstances would be useful.
class Foo<TMap = {}> {
    doSomething<K extends keyof TMap>(
        key: K,
        datum: TMap[K] extends undefined ? omitted | undefined : TMap[K]
    ) {
        console.log({ key, datum });
    }
}

const foo = new Foo<{ one: string, two: undefined }>();
foo.doSomething("one", "a");        // valid
foo.doSomething("one");             // ts(2554) as expected
foo.doSomething("two", undefined);  // valid
foo.doSomething("two");             // valid

💻 Use Cases

What do you want to use this for?

I want to use this for an observer that I'm building. Sometimes I need to pass data to the event, but other times I don't. It would save me some typing if I didn't have to pass undefined manually, or add in workarounds to make the parameter conditionally optional.

What shortcomings exist with current approaches?

At the moment, the way to define an optional parameter is using the ? parameter modifier, but this makes the parameter optional even when it shouldn't be.
class Foo<TMap = {}> {
    doSomething<K extends keyof TMap>(key: K, datum?: TMap[K]) {
        console.log({ key, datum });
    }
}

const foo = new Foo<{ one: string, two: undefined }>();
foo.doSomething("one", "a");
foo.doSomething("one"); // no ts(2554)

What workarounds are you using in the meantime?

It's possible to simulate this behaviour using a rest parameter, but it requires the parameter to be defined as both an empty array and an array containing undefined. The variable itself also needs to be extracted before it can be used.
class Foo<TMap = {}> {
    doSomething<K extends keyof TMap>(
        key: K,
        ...data: TMap[K] extends undefined ? [] | [undefined] : [TMap[K]]
    ) {
        const [ datum ] = data;
        console.log({ key, datum });
    }
}

@RyanCavanaugh yeah, I think it is. I must have missed that one when I searched for issues - I'll close this ticket since uts just a duplicate


🐛 Avoid syncing sanity check

Codecov Report

Merging #56 (f1fab2c) into main (90c58e3) will decrease coverage by 1.18%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main      #56      +/-   ##
==========================================
- Coverage   92.56%   91.39%   -1.18%
==========================================
  Files           9        9
  Lines         148      151       +3
  Branches       13       14       +1
==========================================
+ Hits          137      138       +1
- Misses          6        7       +1
- Partials        5        6       +1

Files Changed
Coverage Δ

src/wandb_osh/lightning_hooks.py
84.61% <50.00%> (-15.39%)
:arrow_down:


should not download chromium by default

We had solo all puppet right now, and use wechat4u as default puppet. While when I npm install wechaty, it will download chromium default, maybe we should not download chromium at this period.
➜  test npm install wechaty

><EMAIL_ADDRESS>install /Users/jiaruili/git/test/node_modules/node-expat
> node-gyp rebuild

  CC(target) Release/obj.target/expat/deps/libexpat/lib/xmlparse.o
  CC(target) Release/obj.target/expat/deps/libexpat/lib/xmltok.o
  CC(target) Release/obj.target/expat/deps/libexpat/lib/xmlrole.o
  LIBTOOL-STATIC Release/libexpat.a
  CXX(target) Release/obj.target/node_expat/node-expat.o
  SOLINK_MODULE(target) Release/node_expat.node

><EMAIL_ADDRESS>install /Users/jiaruili/git/test/node_modules/leveldown
> prebuild-install || node-gyp rebuild

><EMAIL_ADDRESS>install /Users/jiaruili/git/test/node_modules/puppeteer
> node install.js

ERROR: Failed to download Chromium r564778! Set "PUPPETEER_SKIP_CHROMIUM_DOWNLOAD" env variable to skip download.
{ Error: read ETIMEDOUT
    at _errnoException (util.js:999:13)
    at TLSWrap.onread (net.js:629:25) errno: 'ETIMEDOUT', code: 'ETIMEDOUT', syscall: 'read' }
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/jiaruili/.npm/_logs/2018-07-07T07_13_49_473Z-debug.log

What's wechaty version had you installed?

package.json shows ^0.16.5
This is the default version when I use npm install wechaty without any command.
We should let this command using a wechaty without install chromium

I ask you is not for the answer because I had already known that when I ask.
I ask you this question is because I want you to think about that.
Close this issue because it's not a problem.


How to refresh the refresh_token itself?

If I read the api docs correctly refresh_token will expire after 14 days. So Yo need to get a new pair of access_token + with refresh_token on every 13th day. How do I achieve  with this api?

This package is built on top of the League's oauth2-client, which supports exchanging refresh tokens for new access tokens. You can read about it here: https://github.com/thephpleague/oauth2-client#refreshing-a-token
As far as performing this action on day 13, you will need to write some code or setup some tools to execute the code within this package to schedule those tasks.

Thanks.


sink(ticdc): clean up table checkpoint ts in buffer and MQ sink

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/4464
What is changed and how it works?

clean up table checkpoint ts in buffer sink
clean up table checkpoint ts in MQ sink

See more: https://github.com/pingcap/tiflow/issues/4464#issuecomment-1121989321
Check List
Tests

Manual test

Code changes
None
Side effects
None
Related changes

Need to cherry-pick to the release branch(I will do it manually.)

Release note
None

/run-all-tests

/merge
/hold
Waiting for test-infra test landed.

/unhold
test passed.
/run-verify
/run-leak-test


IP tracker and ad in wiki

Issue Type
[ ] Question
[ ] Documentation
Description and/or steps/code to reproduce the problem
I'd like to draw your attention to a matter I came across while browsing the project's wiki. I observed that the recent revisions include promotions for ChatGPT-Undetected.com and an IP tracker within the "Latest Releases" hyperlink. Are these elements intended to be part of the project's wiki?
This content appears somewhat dubious and out of place, prompting concerns about its appropriateness within the project's wiki and safety of the codebase.
IP tracker:

Latest Releases

ChatGPT-Undetected promotion at the bottom of the wiki:
## Using chatgpt to improve code quality ?

For developers seeking to enhance their coding prowess without being flagged by AI detection tools,
[ChatGPT-Undetected.com](https://www.chatgpt-undetected.com/) offers a specialized suite of solutions.
This platform provides strategies and insights to refine code,
ensuring it remains authentic and indistinguishable from human-written software.
It's a resource designed to maintain the integrity and originality of your programming work in
environments where AI contributions are monitored or restricted.

Environment
If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
6.1.2

Where exactly?
On Thu, Jan 25, 2024, 2:58 PM aia80212 @.***> wrote:

Issue Type
[ ] Question
[ ] Documentation
Description and/or steps/code to reproduce the problem
I'd like to draw your attention to a matter I came across while browsing
the project's wiki. I observed that the recent revisions include promotions
for ChatGPT-Undetected.com and an IP tracker within the "Latest Releases"
hyperlink. Are these elements intended to be part of the project's wiki?
This content appears somewhat dubious and out of place, prompting concerns
about its appropriateness within the project's wiki and safety of the
codebase.
IP tracker:
Latest Releases
ChatGPT-Undetected promotion at the bottom of the wiki:
Using chatgpt to improve code quality ?
For developers seeking to enhance their coding prowess without being flagged by AI detection tools,
ChatGPT-Undetected.com offers a specialized suite of solutions.
This platform provides strategies and insights to refine code,
ensuring it remains authentic and indistinguishable from human-written software.
It's a resource designed to maintain the integrity and originality of your programming work in
environments where AI contributions are monitored or restricted.
Environment If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
6.1.2
—
Reply to this email directly, view it on GitHub
https://github.com/osmdroid/osmdroid/issues/1976, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPCIGL3Y653YKW2ISK5W3DYQK2OFAVCNFSM6AAAAABCLCZ6SKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDCMBXG44DSNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

https://github.com/osmdroid/osmdroid/wiki
ChatGPT-Undetected promotion: https://github.com/osmdroid/osmdroid/wiki#using-chatgpt-to-improve-code-quality-
added in this revision: https://github.com/osmdroid/osmdroid/wiki/Home/56eae1be7735a9ee2c33dc26eb5a6aca9b4b75ab
IP tracker: https://github.com/osmdroid/osmdroid/wiki#osmdroid
@@ -5,6 +5,7 @@ Welcome to the osmdroid!
Chances are good that you want to get a map up and running in your app as soon as possible. If that's the case, consider reading the following guides (they are short!) in this order.

1. Include the binaries in your project:
  * [**Latest Releases**](https://tracker.iplocation.net/iogm/)
  * [Prerequisites](Prerequisites)

added in this revision: https://github.com/osmdroid/osmdroid/wiki/Home/a037e4d237e57b13bcd90998b14d888658305fd5

thank you, it's been removed and the user has been banned from the organization.


nixos/prowlarr: Add dataDir config option

Description of changes
Same like in Sonarr
https://github.com/NixOS/nixpkgs/blob/ef176dcf7e76c3639571d7c6051246c8fbadf12a/nixos/modules/services/misc/sonarr.nix#L13-L17
Things done

Built on platform(s)

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

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

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

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

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

[X] Fits CONTRIBUTING.md.

Please use a bind mount instead, as a separate dataDir will not work well with DynamicUser´and I'm not in favor of allocating a static uid/gid for sonarr.
Read up on the discussion in the PR where this module was created: #136039 (comment)

So the reason for me wanting a dataDir option is so that I can back up the stateful files with no issues. This application sadly has some database with all the settings so it's extremely hard to configure it with nix and I don't feel like setting it up again if I reset my server. So here is my specific use case, I just want to be sure the dataDir is where I expect it.
https://github.com/rhoriguchi/nixos-setup/blob/5e281ed38a4921295a5c354d915f3c29ea7ce328/configuration/devices/headless/server/rsnapshot.nix#L47
cc @jdreaver

An idea: we can probably use DynamicUser=, SupplementaryGroups=, BindPaths= and setgid bit and/or POSIX ACLs on the state directory. (And have a boundary directory like /var/lib/private, to prevent access by users not in the SupplementaryGroup=, or re-used DynamicUser= UIDs..)
Yes, that sounds more complicated, but now sharing is opt-in and managed by group membership, instead of having to run multiple different services under the same UID. Also, dataDir options that are absolute paths break when passed to systemd StateDirectory=, so I think we should come up with something that plays along with systemd.

Closing in favor of #215817


Overzealous require() substitutions

Hi, there is a problem with string substitutions being overzealously applied in nexe.
Example 1:
var fs = require('fs'); var http = require('http'); becomes var fs = require('myapp.js'); var http = require('myapp.js'); which is just wrong.
Example 2:
Also later on, using "fs" or "http" as string constants also become erroneously replaced, e.g. if (protocol_used = "http") becomes if (protocol_used = "myapp.js") . This is quite bad. The replacements should err on the side of missing replacements rather than blindly every single usage of the string.
This is causing any project that depends on node fs/http libraries to fail (and i assume the problem exists with other builtin libraries)

This should be marked as fixed.


.prettier configuration

Contains .prettier formatted code

@kalanamt
I'm fine for a .prettier config to added, but I don't think a PR that changes every single file at once is a great idea.
Can we set prettier to give warnings, and then incrementally improve?

Styling is a one time configuration and its required to format files in order to get this working , if we set it to give warnings means some may keep unformatted styles in their working environments .   I need to see how to give warnings for style formatting, ideally if a code base has incorrect formatting or styles we should not merge it to the main or staging branch . Open to further #discussion

Closing due to low priority and too many conflicts.


What changed in 1.16.100?

Hey guys! Looking to crowd-source information about 1.16.100, which will be turned into a blog-post for the News section.
what broke?
what was added?
format changes?
what moved from EXP into stable?
Do share :)

https://wiki.bedrock.dev/tutorials/spawning-items#all-purpose-method
This is broken on 1.16.100 as "drop_item_for" is now undocumented and needs extra parameters to function. We have found that the entity is gone before it gets time to drop. This is kind of patched by increasing movement speed, but not perfect. The best swap for now is "sneeze", but it leaves a nasty particle behind that I believe is redstone based so it can't be removed.

It is now easier than ever to spawn items: simply save an empty structure with an item inside (make sure to save entities) and load that structure with /structure load

https://github.com/DrAv0011/Bedrock-Template-Updates/compare/1.16.20...1.16.100

{
	"minecraft:behavior.drop_item_for": {
		"priority": 1,
		"seconds_before_pickup": 0,
		"cooldown": 0.25,
		"drop_item_chance": 0.7,
		"offering_distance": 5,
		"minimum_teleport_distance": 2,
		"max_head_look_at_height": 10,
		"target_range": [
			5,
			5,
			5
		],
		"teleport_offset": [
			0,
			1,
			0
		],
		"time_of_day_range": [
			0.74999,
			0.8
		],
		"speed_multiplier": 1,
		"search_range": 5,
		"search_height": 2,
		"search_count": 0,
		"goal_radius": 1,
		"entity_types": [
			{
				"filters": {
					"test": "is_family",
					"subject": "other",
					"value": "player"
				},
				"max_dist": 6
			}
		],
		"loot_table": "loot_tables/entities/cat_gift.json",
		"on_drop_attempt": {
			"event": "minecraft:cat_gifted_owner",
			"target": "self"
		}
	}
}

A new addition has been added for minecraft:behavior.drop_item_for, this include :

"seconds_before_pickup": value
"cooldown": value
"minimum_teleport_distance": value
"max_head_look_at_height": value
"target_range": [ value, value, value ]
"teleport_offset": [ value, value, value]

Diff viewer: https://github.com/DrAv0011/Bedrock-Template-Updates/compare/1.16.20...1.16.100

Actual changelog: https://feedback.minecraft.net/hc/en-us/articles/360052592091-Minecraft-1-16-100-Bedrock-

Description for drop_item_for
https://paste.stirante.com/ehawaroquc.hs

Description for drop_item_for
https://paste.stirante.com/ehawaroquc.hs

How did you get this? @stirante

New commands:

/camerashake
/clearspawnpoint
/event
/fog
/music
/playanimation
/ride
/structure
/schedule

New Block and Item components. Custom block geometry. These components are probably still behind experimental gameplay (can't confirm).

New Block and Item components. Custom block geometry. These components are probably still behind experimental gameplay (can't confirm).

Figuring out whats in EXP, whats in BETA, and whats in STABLE is a big thing I would like to disambiguate.

Here's the tested format for "run_command" in entity events in behaviors in this update 1.16.100.
Notes:
_1. Experimental needs to be used currently,

Commands do not use slashes,

Target is used to run the command off that target if no selector provided.

Uses [] list for multiple commands for the value of "command"_

  "prefix:event_name": {
    "run_command": {
      "command": "say true",
      "target":"self"
    }
  }

Spawn eggs have changed this update of 1.16.100 and they no longer randomly generate a damage value, instead they use this format for give commands and loot tables, trades from my testing the id of a creeper spawn egg is in NBT.
minecraft:creeper_spawn_egg

Custom material inheriting is no longer valid and causes content log errors the workaround Is to define the material fully custom with just the prefix and material name.
{
  "materials": {
    "version": "1.0.0",
    "prefix:window_glass:entity": {//now throws a content log error.
      "+states": [
        "Blending"
      ],
      "defines": [
        "ENABLE_FOG",
        "ENABLE_LIGHT",
        "USE_ONLY_EMISSIVE"
      ]
    },
    "prefix:window_glass:": {//corrects the content log error.
      "+states": [
        "Blending"
      ],
      "defines": [
        "ENABLE_FOG",
        "ENABLE_LIGHT",
        "USE_ONLY_EMISSIVE"
      ]
    }
  }
}


chore: improved docs seo

Proposed changes
improved docs seo
Types of changes

[x] 📚 Documentation Update (if none of the other choices apply)

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (b094895) 100.00% compared to head (35f4d58) 96.79%.
 Consider uploading reports for the commit 2851ab0 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              main     #893      +/-   ##
===========================================
- Coverage   100.00%   96.79%   -3.21%
===========================================
  Files          103       67      -36
  Lines          914      812     -102
  Branches       160      146      -14
===========================================
- Hits           914      786     -128
- Misses           0       16      +16
- Partials         0       10      +10

see 135 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


system/cu: disable error print if DEFAULT_SMALL

Summary
system/cu: disable error print if DEFAULT_SMALL
Test on sim/nsh (CONFIG_DEFAULT_SMALL=y):
   text	   data	    bss	    dec	    hex	filename
 397049	  26704	   4128	 427881	  68769	nuttx      /* before */
 396481	  26704	   4128	 427313	  68531	nuttx      /* after */
   -568

Impact
N/A
Testing
sim/nsh

CONFIG_DEFAULT_SMALL should not be used directly control features.   See full comment in #1788

CONFIG_DEFAULT_SMALL should not be used directly control features. See full comment in #1788

I made the corresponding changes according to your suggestion, please review again, thanks!


Getting laravel_through_key in hasManyThrough relation

Laravel Version: 5.8.*
PHP Version: 7.3.2

Description:
Getting laravel_through_key intead of foreign key on the "through" model  in hasManyThrough relation

This has been changed in Laravel 5.8 to prevent column overriding: https://github.com/laravel/framework/pull/25812
What exactly is your issue?

It has not been documented in the version release.

See https://github.com/laravel/docs/pull/4563.

@staudenmeir Thanks!

@staudenmeir Thanks!

look at for solution
https://github.com/laravel/framework/issues/745#issuecomment-15610315

I am trying to access a third table details from one but it is only displaying laravel_through_key

@ashwinsabu any success?


Add custom support for traceback location information available with Python 3.11

Currently, Friendly-traceback relies on executing to find the location of the code that causes an error. This currently occasionally fails with Python 3.11. While there is an executing issue to fix this problem, and which should work on all frames of the traceback, I should implement my own "fail safe" method for 3.11 to identify the "bad line".

After experimenting with this, it looks like I would have to reimplement the entire logic of the executing issue, which does not seem to be worthwhile at this point.


[AFF] Custom AJSON filter for INTF serialization

Ref #5658
Aims to emulate behavior described in AFF, $default and other annotations

https://github.com/sbcgua/ajson/pull/111 is required to set values without changing the path (camel case).
Actually, originalLanguage becomes originallanguage after setting a custom value via set method.

https://github.com/sbcgua/ajson/pull/111 merged.
P.S. potentially the behavior may change in future so that conversion is applied at parsing time. Then it will create a bug as the access path will be original_language not originalLanguage. Anyway for the moment the ajson is unblocked

https://github.com/abapGit/abapGit/pull/5682

@larshp can you update the ASJON inside abapGit? The ordering is as expected and has been merged, see https://github.com/sbcgua/ajson/pull/115

done

Let me add a short video
https://user-images.githubusercontent.com/38354196/180751465-a6244542-842e-4de6-8b1e-716cfbe62087.mp4

Few notes:

please add unit tests
we need to generalize/move some of the json parts to somewhere generic

we need to generalize/move some of the json parts to somewhere generic

Any suggestions, starting points?

lets get this PR merged first, then we can take a look
I'll merge it tomorrow morning(EU time) if there are no additional comments


Loadpoint: fix race condition accessing soc estimator (#16180)

Fix #16180
It might not be ideal since a now soon-to-not-exist socEstimator may be updated, but the alternative would have been to have mutexes.
Since the bug seems to be rather esoteric, the risk of showing "old" information for a short period of time seems to be acceptable. It should be better than a hard crash in any case.
With that patch applied I couldn't provoke a crash anymore but since I don't fully understand the code paths of evcc (yet)…

Klasse, auf den Fix bin ich gar nicht gekommen!

Indeed. Nice catch @jennes !


Plugin request: haskell-scope-highlighting.nvim

Plugin: haskell-scope-highlighting.nvim
Plugin repository: https://github.com/kiyoon/haskell-scope-highlighting.nvim
Is the plugin on nixpkgs: No
Extra information
Good suggestions are given in the customizations section. I'm new to nix and nixvim, so I apologize if there are difficulties with this plugin I'm overlooking. Thanks!

Waiting on #284070 :D


SCRUM-1356 Adding Pagination

Adding code to delivery pagination.  Functions are limited to First,  Last, Next, and Previous.  The first page is defined as page zero (to  make the maths easier) but displayed as page one.  Page is reset to zero  upon changing the size of pages or running another query.  Its possible that all the pagination could be moved into its own component as it get more complex... or now if we think thats necessary.
Needs the corresponding SCRUM-1356 pull from the agr_literature_service repo to function properly.

I've checked out the branch and the search page still shows the "query by exact xref" component at the top. Is this branch synced with main?

the new pagination component looks good, thanks @pjhale

I haven't pulled in main since I branched but no merge issues seem to exist.

@pjhale Probably best to merge main into your branch, see if it seems okay, and then update this PR with that


IATU: The ASSERT condition was violated for ITS templates

Hi,
I am moving objects to GIT repository using ZABAPGIT. While I commit HTML files of ITS templates, dump is raised in PROCESS_STAGE_LIST method at line 49369. ZABAPGIT has been last updated on 16 APR 2018.
How can I resolve this?
Thanks.

What is the error message?

The ASSERT condition was violated ----- as short text in dump

^sorry I read "16 APR 2019", not "16 APR 2018"
update to latest version

Will this new version supports ITS templates

Can you send me the link to update the latest version of ZABAPGIT

https://docs.abapgit.org/guide-upgrade.html

Hi,
I have coiped the latest version of ZABAPGIT. While committing HTML files it is not raising dump now but throws below error.

Thanks.

any special characters part of the filename?

are you using SAP GUI for Java?

any special characters part of the filename?

Filename is like
/src/itsserivename 99 htmlfile.iatu.html

are you using SAP GUI for Java?

We are not using SAP GUI for JAVA

do you have multiple services with the same service name?

No ITS service names are unique

unable to reproduce, see example in https://github.com/abapGit-tests/IARP_IASP_IATU/tree/master/src

How did you move IATU files to GIT, can you let me know the process

closing, unable to reproduce


Log out delays

Recently with Auctionator I have been getting log out delays, could there be something I am missing or not doing that has caused this to start happening? I have gone through the process of elimination with my addons and it only occurs once I select the auctionator addon.

Thanks for the report, not sure how I missed it... must have notifications set incorrectly.  :frowning:
If you got to the settings for Auctionator:
Interface > Addons > Auctionatory > Basic Options
And select the "Enable debug mode" option, you'll at least see what the activity is that Auctionator is doing at logout.  I suspect you have a ton of data that is getting saved, but it's hard to say without being able to reproduce your logout environment.  Be warned that debug mode is super spammy in your chat box (it basically logs every time Auctionator does anything interesting)... I'll have to think about how to make it so we can store the output after the game exits, but maybe you can try to screen cap as you log so I can see what is happening?

Closing, no response


Automated cherry pick of #1301: fix update zstack account password error

Cherry pick of #1301 on release/2.10.0.
#1301: fix update zstack account password error

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@d931a6a). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/2.10.0   #1302   +/-   ##
================================================
  Coverage                  ?   7.88%
================================================
  Files                     ?     369
  Lines                     ?   57668
  Branches                  ?       0
================================================
  Hits                      ?    4549
  Misses                    ?   52719
  Partials                  ?     400

Flag
Coverage Δ

#aFlag
7.88% <ø> (?)

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 d931a6a...d073188. Read the comment docs.

/approve
/lgtm


Merge Protein Visualizer into develop

Updates:

figure can display overlapping domains
improved code readability and structure
legend for variants added
readable protein feature domain labels
deterministic & reproducible coloring

@frehburg
The CI is reporting this error:
src/genophenocorr/view/_protein_visualizer.py:154: in <module>
    @dataclass(slots=True)
E   TypeError: dataclass() got an unexpected keyword argument 'slots'

This is strange because you showed me the code working fine today. Do you know what is going on? In general, we never accept a PR if the CI is broken! Thanks!

Dear Peter, oh I am sorry. The feature slots was only added in Python 3.10 if I remember correctly. I can revise the code while keeping it's memory efficiency tomorrow.
https://docs.python.org/3/library/dataclasses.html

If this is a useful feature, I do not see much point in supporting many backward versions of Python -- @ielis what about supporting Python 3.10 and upwards? We are at 3.12 now and could also add that to the CI?

I looked at the docs and I do not see any benefits of using slots=True at the cost of removing support for Python <3.10. @frehburg are there any strong arguments for using this option?
IMO, support of broader Python versions is more useful than anything else I can see at this point, because an older Python may be all that people have in some constrained environments.
So, unless there are strong arguments for using slots=True, we should probably avoid it.
@frehburg please let us know.

Dear @ielis,
as I said. I will rewrite that bit of code today. There are other options.
Regards,
Filip

@pnrobinson @ielis There is a way to manually define slots, which was already available in 3.8. It's a bit less neat, but preserves the compatibility with older Python versions.
I hope this works.


Array column should support hash function

Steps to reproduce the behavior (Required)
create table t0(c0 INT, c1 array<varchar(65533)>) duplicate key(c0) distributed by hash(c0) buckets 10 properties('replication_num'='1')
insert into t0 values(0, null),(1, ['x']),(1,['x']),(2, ['y']),(2, ['y'])
select count(*), count(c1), count(distinct c1) from t0

+-----------------------------------------------------------------------------------------+
| Explain String                                                                          |
+-----------------------------------------------------------------------------------------+
| WORK ON CBO OPTIMIZER                                                                   |
| PLAN FRAGMENT 0                                                                         |
|  OUTPUT EXPRS:3: count() | 4: count(2: c1) | 5: count(distinct 2: c1)                   |
|   PARTITION: UNPARTITIONED                                                              |
|                                                                                |
|   RESULT SINK                                                                           |
|                                                                                |
|   6:AGGREGATE (merge finalize)                                                          |
|   |  output: count(3: count()), count(4: count(2: c1)), count(5: count(distinct 2: c1)) |
|   |  group by:                                                                          |
|   |  use vectorized: true                                                               |
|   |                                                                                |
|   5:EXCHANGE                                                                            |
|      use vectorized: true                                                               |
|                                                                                |
| PLAN FRAGMENT 1                                                                         |
|  OUTPUT EXPRS:                                                                          |
|   PARTITION: HASH_PARTITIONED: 2: c1                                                    |
|                                                                                |
|   STREAM DATA SINK                                                                      |
|     EXCHANGE ID: 05                                                                     |
|     UNPARTITIONED                                                                       |
|                                                                                |
|   4:AGGREGATE (update serialize)                                                        |
|   |  output: count(3: count()), count(4: count(2: c1)), count(2: c1)                    |
|   |  group by:                                                                          |
|   |  use vectorized: true                                                               |
|   |                                                                                |
|   3:AGGREGATE (merge serialize)                                                         |
|   |  output: count(3: count()), count(4: count(2: c1))                                  |
|   |  group by: 2: c1                                                                    |
|   |  use vectorized: true                                                               |
|   |                                                                                |
|   2:EXCHANGE                                                                            |
|      use vectorized: true                                                               |
|                                                                                |
| PLAN FRAGMENT 2                                                                         |
|  OUTPUT EXPRS:                                                                          |
|   PARTITION: RANDOM                                                                     |
|                                                                                |
|   STREAM DATA SINK                                                                      |
|     EXCHANGE ID: 02                                                                     |
|     HASH_PARTITIONED: 2: c1                                                             |
|                                                                                |
|   1:AGGREGATE (update serialize)                                                        |
|   |  STREAMING                                                                          |
|   |  output: count(*), count(2: c1)                                                     |
|   |  group by: 2: c1                                                                    |
|   |  use vectorized: true                                                               |
|   |                                                                                |
|   0:OlapScanNode                                                                        |
|      TABLE: t0                                                                          |
|      PREAGGREGATION: ON                                                                 |
|      partitions=1/1                                                                     |
|      rollup: t0                                                                         |
|      tabletRatio=1/1                                                                    |
|      tabletList=169413                                                                  |
|      cardinality=5                                                                      |
|      avgRowSize=1.0                                                                     |
|      numNodes=0                                                                         |
|      use vectorized: true                                                               |
+-----------------------------------------------------------------------------------------+
61 rows in set (0.02 sec)

For the hash functions, we need to pay attention to the following points
the hash function of the data distribution and the hash function of the aggregation should not be the same, we need to reduce the conflict rate as much as possible
we need to consider compatibility issues and cross-platform issues. For example, whether the hash function is generic enough to maintain compatibility with other languages, such as java
for the data distribution of the hash function, crc is sufficient, in addition to the need for shuffle and aggregation of the hash function

Here was the performace between fnv32 and murmurhash32
but fnv32 quality may not good at murmurhash32
https://quick-bench.com/q/McXn9e7Eg9S7o7cnj8SeGTKfiJ4


Font Clipping

Any ideas on how to fix this font clipping issue?  It seems to clip on the g's and p's.  I'm trying out "Inconsolata-dz for Powerline dz".

That seems to be a line-height problem, have you tried to reproduce it with another font?

Duplicate of #628.


python312Packages.carbon: patch cf.readfp

Description of changes
Fix build (https://hydra.nixos.org/build/271885017)

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

4 packages built:

python311Packages.carbon
python311Packages.carbon.dist
python312Packages.carbon
python312Packages.carbon.dist


Add reference to Helm v2 architecture

Feedback from @henrynash:
Perhaps it might be an idea just to add a helm 2 or helm 3 moniker in each of the Lab0 READMEs - maybe near the diagram, and remind them of the other fork.

LGTM - thanks for adding the reference.

Thanks for the review @henrynash @spzala


Don't recurse into metadata

Codecov Report

Merging #532 (efada4b) into main (23ac0db) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #532   +/-   ##
=======================================
  Coverage   94.82%   94.82%
=======================================
  Files          93       93
  Lines       12305    12307    +2
  Branches       26       26
=======================================
+ Hits        11668    11670    +2
  Misses        629      629
  Partials        8        8

Impacted Files
Coverage Δ

src/build_context.rs
96.21% <100.00%> (+0.05%)
:arrow_up:

Continue to review full report in Codecov by Sentry.

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

LGTM


Strip down dependencies of rails

Hi!
Thanks for this gem, I really like it. I had to change the gemspec to fit my app which uses only some gems of rails and not the full pack.
This is a simple change to use only the required parts of rails that this gem needs, so it keeps the same behaviour but allows to use in apps that have only actionpack/support for example (thus not installing extra gems  without need)
I think it can be useful for more people in the same situation that I am.
What do you think?

Hi @thiago-sydow
Good idea. Thank you for PR!


[doctest] mpmath.calculus.optimization.steffensen fails on alpha in 1.4.0a1

On alpha processor:
================================== FAILURES ==================================
_____________ [doctest] mpmath.calculus.optimization.steffensen ______________
1034
1035     x* is a fixpoint of the iteration x_{k+1} = phi(x_k) if x* = phi(x*). For
1036     phi(x) = x**2 there are two fixpoints: 0 and 1.
1037
1038     Let's try Steffensen's method:
1039
1040     >>> f = lambda x: x**2
1041     >>> from mpmath.calculus.optimization import steffensen
1042     >>> F = steffensen(f)
1043     >>> for x in [0.5, 0.9, 2.0]:
Differences (unified diff with -expected +actual):
    @@ -5,5 +5,5 @@
              2.32831e-10          -2.84465e-27
              5.42101e-20           2.30189e-80
    -         2.93874e-39          -1.2197e-239
    +         2.93874e-39                     0
              8.63617e-78                     0
             7.45834e-155                     0

/var/tmp/portage/dev-python/mpmath-1.4.0_alpha1/work/mpmath-1.4.0a1/mpmath/calculus/optimization.py:1043: DocTestFailure

This was tested against 1.4.0a1. This has been tested by a volunteer, but I'm going to try getting access to the box, and testing on git.

I'm slightly surprised that it's only one doctest failure.
I can see several variants to address this.  First, we could cast x from the list to mp.mpf and then print with print(f'{fx:20g}  {Fx:20g}').  Second, we could try to use pytest-doctestplus with FLOAT_CMP flag.  Unfortunately, that option is far from being trivial: right now it casts all floating-point literals found - to Python floats.

See https://github.com/mpmath/mpmath/pull/839


Not working on Windows using Chrome

I can hotswap streams and it reflects the newly added stream locally but the receiving client still gets the old stream. Any ideas?

I'm assuming you're using an RTCPeerConnection to send the stream. Are you sure you're adding merger.result to the peer?

Closing since I can't reproduce this.


chore(v2): Add E2E test for yarn v2

Motivation
It's very easy for us to miss some dependencies and not caught by CI, because Yarn workspace is very loose: as long as you define a dependency in package A, you can access it in package B locally since Yarn workspace try to put as many dependencies into the root node_modules as possible. Therefore, we might frequently need bug fixes like #3007 for Yarn v2 when these problems are found after a new release.
This PR tries to solve the undetectable missing dependency problem by setup an E2E test for Yarn v2. Currently, it can only pass in pnp loose mode since some of our dependencies don't properly declare their dependencies. However, we can still see the warnings printed in the log so it can guide us to fix these problems upstream in the future.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
CI passes.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Not very familar with yarn 2 but LGTM

Btw, seems related but we should reenable the eslint import/no-extraneous-dependencies once missing deps fixed.
Also, I tried to install this tool that worked great on another monorepo, but my short attempt was not so great (unfortunately, it's not very configurable so you need to conform to all the checks apparently). If you are interested to give it another try: https://github.com/Thinkmill/manypkg

LGTM


Segmentation fault if collector is accessed without calling assignment.Add()

The following code segfaults when the last line is called. This is because solution.Add(myvar) wasn't called right after solver.Assignment().
from __future__ import print_function
from ortools.constraint_solver import pywrapcp

solver = pywrapcp.Solver("bug")
myvar = solver.IntVar(0, 10, "myvar")
solver.Add(myvar == 5)

solution = solver.Assignment()
collector = solver.FirstSolutionCollector(solution)
solver.Solve(solver.Phase([myvar], solver.INT_VAR_SIMPLE, solver.ASSIGN_MIN_VALUE), [collector])
num_solutions = collector.SolutionCount()
print("num_solutions: ", num_solutions)

print(collector.Value(0, myvar))

The desired behavior would be to not segfault and throw a reasonable error message first.

I agree.
Still, the solver is in maintenance mode. and the new solver (CP-SAT) does not exhibit this behavior.
And patching the SWIG code and the C++ code to fix this is a bit too much work.
So, I will not fix it :-(

Thanks for the response. Right now, a lot of the examples on the website and in the examples/ folder are based on the old CP solver. And it looks like simply replacing the solver does not suffice to migrate to the CP-SAT solver, because a lot of methods have changed (solver.IntVar becomes model.NewIntVar, solver.Max no longer exists etc.). Where can I figure out the python APIs for the CP-SAT solver?

https://github.com/google/or-tools/blob/master/ortools/sat/doc/reference.md
and more generally
https://github.com/google/or-tools/blob/master/ortools/sat/doc/index.md
All this is slowly migrated to developer.google.com/optimization, but it
takes time.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le mar. 6 nov. 2018 à 18:59, Lalith Suresh<EMAIL_ADDRESS>a
écrit :

Thanks for the response. Right now, a lot of the examples on the website
and in the examples/ folder are based on the old CP solver. And it looks
like simply replacing the solver does not suffice to migrate to the CP-SAT
solver, because a lot of methods have changed (solver.IntVar becomes
model.NewIntVar, solver.Max no longer exists etc.). Where can I figure out
the python APIs for the CP-SAT solver?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/872#issuecomment-436348255,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17URIFCYL-9-1QrTFPS_KBcFbwF4Yks5usc4IgaJpZM4XKdDZ
.


Slack alerts are not working.

On version 0.13.2 we are able to set up the slack alerts from UI for test-cases, test suites but we are not receiving the slack alerts on channel.

ref to the thread: https://openmetadata.slack.com/archives/C02B6955S4S/p1678357649317809

@mohityadav766 can you take a look

Turning this into doc task as the feature works as expected

Hi Sriharsha, the slack alerts does not work in OM version, 0.13.2.
Thanks.
On Wed, Apr 12, 2023 at 11:49 AM Sriharsha Chintalapani <
@.***> wrote:

Turning this into doc task as the feature works as expected
—
Reply to this email directly, view it on GitHub
https://github.com/open-metadata/OpenMetadata/issues/10492#issuecomment-1504725391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3CYENJGXDG4VXLDUEK4CHTXAZCQPANCNFSM6AAAAAAVU57424
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This message (including any attachment(s) hereto) is confidential and may
also be privileged. It is intended solely for the addressee. If you are not
the intended recipient you are hereby notified that any disclosure,
copying, distribution, or taking any action in reliance on the contents of
this information is strictly prohibited and may be unlawful. If you have
received this message in error you are requested to delete it from your
system. ADJ Utility Apps Private Limited is not liable for the improper
transmission of this message nor for any damage sustained as a result of
this message.

Hi, have the docs for the Slack alerts been updated?  We cannot get it to work on V0.13.2.  Do we need the Secret Key, if so how do I find that?  Or do we need to approve the connection somehow?
Thanks!

@mohityadav766, is this still an issue?

@mohityadav766, is this still an issue?

@sureshms I'll close this one , once #12189 is merged


Inconsistent Zoom on Album Cover

There is an inconsistent Zoom level on Album Cover. On startup it is over zoom. Some time it shows the correct zoom level.
Image 1 is over Zoomed
Image 2 is correct zoom

I already to noticed this issue. This issue also comes up on some photoCovers and the albums covers in the FileOperationsDialog.
But I couldn't figure out what causes this issue. I will look into this.


CLI prompts for "authentication token", should be "OpenID Connect ID token"

Please answer the following questions before submitting your issue. Thanks!
What version of DC/OS + DC/OS CLI are you using (dcos --version)?
If DC/OS CLI version < 0.4.5, DC/OS version can be found at http://dcos_url/dcos-metada/dcos-version.json
0.4.12
What operating system and version are you using?
Mac OSX Yosemite
What did you do?
I followed the instructions in the open-source documentation https://dcos.io/docs/1.8/administration/id-and-access-mgt/managing-authentication/  and received a prompt to enter my "authentication token"
What did you expect to see?
A prompt to enter my "OpenID Connect ID token"
What did you see instead?
As noted in "What did you do" :)
CC: @alberts @jgehrcke

@emanic the cli only supports oauth authentication for open clusters right now (we have an open jira to expand this). What prompt did you see?
The docs are correct, you should have seen

    https://<public-master-ip>/login?redirect_uri=urn:ietf:wg:oauth:2.0:oob

Enter authentication token:

@tamarrow Thanks so much. You are correct, the documentation matches what is in the prompt right now. However, it turns out that the prompt has the wrong term. Actually, there are two tokens and we need to differentiate them clearly as it is causing a lot of confusion in the field. The token that the CLI is requesting here is actually the "OpenID Connect ID token", not the DC/OS "authentication token." Let me know if you would like to discuss this further on Monday. :))) I know it is confusing.

@alberts you okay with changing the cli prompt to "OpenID Connect ID token"?

I'm okay with this. Let's see if @jgehrcke agrees and we can proceed.

Yes, sure, that's what it is :)
In the spec it is called ID Token (note the capitalization, but that's not too important).
For reference: http://openid.net/specs/openid-connect-core-1_0.html#IDToken

The primary extension that OpenID Connect makes to OAuth 2.0 to enable End-Users to be Authenticated is the ID Token data structure. The ID Token is a security token that contains Claims about the Authentication of an End-User  [...]


docs: add zeno visualization integration

Hi Ragas team! Thanks for this awesome library!
I'm part of the team working on Zeno, an AI evaluation tool. We've had some users working on RAG who mentioned the ragas library, I wrote up a short integration tutorial I thought would be a good addition.
Let me know if you need any clarification from our end!

thanks a lot for @cabreraalex for the PR. Pretty interested in the Zeno project ourselves. would love to talk sometime too to learn more, could you shoot a mail to<EMAIL_ADDRESS>

Thanks Jithin! Will do, excited to chat!

Hey @cabreraalex
Thank you so much for helping us improve ragas with your PR ❤
Now since it's Christmas and all we wanted to send you a postcard and a couple of stickers as our way of saying thank you for your contribution. If you are interested could you shoot an email at<EMAIL_ADDRESS>and I'll tell you more?
Cheers 🙂
Jithin


Change connection name to HIVE_CONN

So that it removes the mismatch with the next create schema statement.

This is good candidate for next kehrwoche.


fix: corrected include directory name for constants/float32/half-pi

Description

What is the purpose of this pull request?

This pull request:

renames the include directory name for constants/float32/half-pi

Related Issues

Does this pull request have any related issues?

None.
Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

☑ Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Sorry for the mistake, the folder had been set to float64 instead of float32.
This PR is a pre-requisite for #1950.
cc: @kgryte


EarlyStoryVisibleTimeoutPatch: check for game type being story mode

Instead of the patch stopping if we are in multiplayer, instead stop if we are in any game mode other than CampaignStoryMode.
I'm not absolutely certain, but basically because all the base game code checks in the this way (for CampaignStoryMode) before accessing story mode vars. Also hypothetically TW might add a new game mode in the future which is not multiplayer but is also not CampaignStoryMode and in that case the patch would incorrectly apply itself.

Examples of the base game checking for (game.GameType is CampaignStoryMode)
in StoryMode.StoryModeSubModule
protected override void OnGameStart(Game game, IGameStarter gameStarterObject) {
    if (!(game.GameType is CampaignStoryMode))
        return;

    // do stuff related to StoryMode
}

// and similar checks in OnGameEnd and OnGameInitializationFinished

in StoryMode.StoryMode
public static StoryMode Current {
    get {
        return !(Game.Current?.GameType is CampaignStoryMode gameType) ? null : gameType.StoryMode;
    }
}


Fix authz [rhel7-1.10.3]

Fix https://bugzilla.redhat.com/show_bug.cgi?id=1331003
@rhatdan @mrunalp PTAL and merge so Lokesh can take the fix and rebuild docker-latest

LGTM

LGTM

On May 4, 2016, at 3:46 AM, Daniel J Walsh<EMAIL_ADDRESS>wrote:
LGTM
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


Formatting in for hcl and tf code response does not show code blocks properly.

Bug Report
Description
Bug Summary:
Seems the "hcl" and "tf" items are not respected the same as ```html items in the UI. When requesting Terraform code, it does not properly format the code block in the UI.
Steps to Reproduce:

Load "dolphin-mixtral" or "codellama" in Ollama
Request some terraform code with a prompt like "Can you provide complete terraform code with variable in terraform.tfvars for an AWS VPC, 1 private subnet, 1 public subnet".
Output is not properly show in the code block

Expected Behavior:
The expectation is that the code is shown as a proper code block in the UI
Actual Behavior:
The hcl and tf code flags are not properly shown like ```html is as a code block.
Environment

Operating System: Linux/Docker
Browser (if applicable): : Chrome/Brave

Reproduction Details
Confirmation:

[x] I have read and followed all the instructions provided in the README.md.
[x] I have reviewed the troubleshooting.md document.

Logs and Screenshots
Browser Console Logs:
NA
Docker Container Logs:
NA
Screenshots (if applicable):

Installation Method
Installed using Docker
Additional Information
[Include any additional details that may help in understanding and reproducing the issue. This could include specific configurations, error messages, or anything else relevant to the bug.]
Note
If the bug report is incomplete or does not follow the provided instructions, it may not be addressed. Please ensure that you have followed the steps outlined in the README.md and troubleshooting.md documents, and provide all necessary information for us to reproduce and address the issue. Thank you!

Unable to reproduce the issue :/

I'll close this issue for now as I highly suspect that there was an issue with the model-end, if not you might want to try updating to the latest version and see if that fixes your issue! Keep us updated!


my contribution

@dkay9 please provide the direct url to the channel

@breakerh sorry this is one of my first open source contributions and I'm thinking i got everything right. Do you mean the url to Leon Noel's channel on youtube?

@dkay9 correct, the link you need is https://youtube.com/channel/UCGiRSHBdWuCgjgmPPz_13xw
Please make another PR with this link instead

Ok I'll do that thanks


Can't get extractnet to work -> regex._regex_core.error: bad escape \d at position 7

Hey,
I tried to make extractnet work on my Macbook Pro with M1 chip.
I tried several python versions (3.9.9 / 3.10.4) with the same result (extractnet installed through pip).
This is what I'm getting:

Any ideas how this can be solved?
Thanks

Sorry, just found a way to get it working:
pip uninstall regex -y
pip install regex==2022.3.2
See here: https://stackoverflow.com/questions/71498132/error-in-heroku-regex-regex-core-error-bad-escape-d-at-position-7-when-usin

This can be fixed by upgrading instead of downgrading now.  In requirements.txt set dateparser to 1.2.0


"Back" button doesnt work (From Bridge site to previous site)

Hi! an user detected a little bug related to our bridge site. He (and i) cant go back to pool/swap page from bridge site. I've tested it with desktop (firefox and chrome) and phone (safari and chrome).

When I am on Pool page (https://amarok.bridge.connext.network/pools) and then move to Bridge page (https://amarok.bridge.connext.network/USDC-from-arbitrum-to-ethereum)
I am not able to go back (to pool page)
<- - button doesnt work, it just updates my page but doesnt redirect me to previous page

PD: it also happens if i want to go back to SWAP site.

replicated tthis, hanks @MinistroDolar
passing to Coinhippo