Modiy azuredeploySpringEnterprise.sh

Fix some small errors

move duplicate space
Fix error text from Workspace Resource ID to Workspace Resource Name
remove --enable-java-agent true

The public doc refer to these bash scripts and we want to fix some small errors in this doc:
https://learn.microsoft.com/en-us/azure/spring-apps/quickstart-deploy-infrastructure-vnet-azure-cli?tabs=azure-spring-apps-enterprise

@aarthiem @selvasingh Please help review this PR. Thansk!

@luisfeliz79 Please help merge this PR at your convenience. Thanks!

@luisfeliz79 @aarthiem @selvasingh Would you please help merge this PR? Thanks!


Comparing a tuple input to a variadic tuple no longer narrows it

Bug Report

🔎 Search Terms
Tuple, variadic tuple, narrowing, parameters, generics, const parameters, readonly
🕗 Version & Regression Information

This changed between versions 5.0 and 5.1

⏯ Playground Link
Playground link with relevant code
💻 Code
type narrow<def> = def extends string
    ? def
    : def extends [unknown, ...unknown[]]
    ? def
    : {
          [k in keyof def]: narrow<def[k]>
      }

declare const parse: <def>(def: narrow<def>) => def

const result = parse([{ a: "foo" }])
//    ^?
// <5.1: [{a: "foo"}]
//  5.1: [{a: string}]

Description
During the pre const generic parameter-era, patterns similar to this one became widely used for narrowing an array input to a tuple. This particular case has been simplified down from ArkType's parser, but as you can see it is very easy to reproduce and it seems possible it could break existing functionality in other libraries.
The workaround I found for this that supported my use case was to narrow the array input iteratively, like this (simplified):
export type validateTupleLiteral<
    def extends readonly unknown[],
    result extends unknown[] = []
> = def extends [infer head, ...infer tail]
    ? validateTupleLiteral<tail, $, [...result, head]>
    : result

However this would probably not be intuitive for other users who may be affected by the change, and const parameters are still new enough that most libraries cannot adopt them yet (though they do fix the problem). @Andarist mentioned he thought it could be related to this change:
https://github.com/microsoft/TypeScript/pull/53709/files

I can confidently confirm that I bisected this to https://github.com/microsoft/TypeScript/pull/53709/files

Unfortunately this got merged late, so I think we will just roll with this in 5.2 instead.


Aria2c

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.02.17. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.02.17

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here.  URI=https://r2---sn-hxb54vo-30
4z.googlevideo.com/videoplayback?mv=m&source=youtube&clen=21276920&sparams=clen%
2Cdur%2Cei%2Cgcr%2Cgir%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Clmt%2Cmime%2Cmm
%2Cmn%2Cms%2Cmv%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&ratebypas
s=yes&upn=iyd1s8gnfrk&beids=%5B9452307%5D&itag=18&key=yt6&lmt=1434942982676415&i
pbits=0&gcr=ru&id=o-ABD4uRs769QOOckVMlyjQ_I31nVxTugw2iBfyd_kmSIZ&mime=video%2Fmp
4&pl=23&mm=31&mn=sn-hxb54vo-304z&ms=au&mt=1487611774&gir=yes&ip=<IP_ADDRESS>&re
quiressl=yes&ei=ECirWIT0KpngYsnNMg&initcwndbps=646250&dur=310.706&expire=1487633
520&signature=A36A799D29468DA61B6132C753C211E4D843DF38.8DC89C9BB0FAEA58B4223D62F
BD0501AECD8B943
Exception: [AbstractCommand.cc:351] errorCode=1 URI=https://r2---sn-hxb54vo-304z
.googlevideo.com/videoplayback?mv=m&source=youtube&clen=21276920&sparams=clen%2C
dur%2Cei%2Cgcr%2Cgir%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Clmt%2Cmime%2Cmm%2
Cmn%2Cms%2Cmv%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&ratebypass=
yes&upn=iyd1s8gnfrk&beids=%5B9452307%5D&itag=18&key=yt6&lmt=1434942982676415&ipb
its=0&gcr=ru&id=o-ABD4uRs769QOOckVMlyjQ_I31nVxTugw2iBfyd_kmSIZ&mime=video%2Fmp4&
pl=23&mm=31&mn=sn-hxb54vo-304z&ms=au&mt=1487611774&gir=yes&ip=<IP_ADDRESS>&requ
iressl=yes&ei=ECirWIT0KpngYsnNMg&initcwndbps=646250&dur=310.706&expire=148763352
0&signature=A36A799D29468DA61B6132C753C211E4D843DF38.8DC89C9BB0FAEA58B4223D62FBD
0501AECD8B943
  -> [SocketCore.cc:1015] errorCode=1 SSL/TLS handshake failure: unable to get l
ocal issuer certificate

02/20 21:32:06 [NOTICE] Download GID#3b6a960cb08c83cc not complete: C:/Program F
iles/Total Commander/Plugins/exe/Ария - Точка невозврата (Official Video).mp4.pa
rt

Download Results:
gid   |stat|avg speed  |path/URI
======+====+===========+=======================================================
3b6a96|ERR |       0B/s|C:/Program Files/Total Commander/Plugins/exe/Ария - Точк
а невозврата (Official Video).mp4.part

Status Legend:
(ERR):error occurred.

aria2 will resume download if the transfer is restarted.
If there are any errors, then see the log file. See '-l' option in help/man page
 for details.

ERROR: aria2c exited with code 1
  File "__main__.py", line 19, in <module>
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\__init__.py", line 452, in main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\__init__.py", line 442, in _real_main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 1796, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 707, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 753, in process_ie_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 1440, in process_video_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 1704, in process_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 1646, in dl
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\downloader\common.py", line 353, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\downloader\external.py", line 44, in real_download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\downloader\common.py", line 162, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 570, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp9b6x_5wr\bu
ild\youtube_dl\YoutubeDL.py", line 532, in trouble

C:\Program Files\Total Commander\Plugins\exe>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
I'm try to download with aria2c (v.1.31.0) as external downloader, but without sucess. Found similar issue, but as i found out it doesn't work with hls, and i just try to download ordinary youtube.

That's an aria2c/SSL issue not a youtube-dl one.


Provide a LICENSE file so we can attribute your work properly and legally!

The title says it all --- I'm investigating supplementing Chromium's devtools with this project, and it would be very nice if we had a proper document to attribute your work. I think legally I can just make something up which includes your full name, but I would feel a lot better if you produce a LICENSE file yourself.

My pleasure.

Looking at https://github.com/ternjs/acorn/issues/414#issuecomment-219723060, if you find something it doesn't do, please let me know

Reading the code, it looks like the [NoLineTerminatorHere] handling (which seems to be performed by a RegExp) will fail completely if there is a comment between the async token and the remaining tokens, so I expect that will not work. For instance, it looks like it will treat async // if true, non-serial execution<linefeed> will be parsed as an async function, because the parser is not aware that the next token is on a different line.
I'm verifying this at the moment, will file an issue if it's a problem

That seems like a distinct possibility. It you can come up with a simple breaking case, I'll have a go at fixing it

You're right. I'll try and fix in the next few days. See https://github.com/MatAtBread/acorn-es7-plugin/issues/7


Fixed the bug which was causing vehicle with no details to be added.

Added a check to ensure that a vehicle can only be added if it has at least one character in each of its name, model, and make fields.
What it Does

Closes issue #24
It checks for the name, make, and model of a vehicle before adding it.

How I Tested

Run the application
Tap the 'Settings' button
Tap on 'Add Vehicle' Button
Do not fill any details and tap 'Add'.
'Add Button' will be grayed out until you fill some details in all the columns.
Fill some details.
Tap on 'Add Button'.
Vehicle will get added.

Notes

Added check for name, make, and model strings, whether they are empty or not.
Please close pull request #26, I am creating this pull request after making all the necessary and suggested changes.

Screens
https://github.com/mikaelacaron/Basic-Car-Maintenance/assets/63922782/b3729fdc-cd09-4b21-b86b-faab9ba39656

@mikaelacaron Can you provide feedback on my PR?

@mikaelacaron Can you provide feedback on my PR?

I am reviewing PRs tonight, please give me 2-3 days, in general, because of the amount of issues and PRs coming in at the moment (with it being the start of Hacktoberfest)


[Android] Implicit conversion of the package name when exporting for Android with using "$genname" for project name starting with numbers.

Godot version
3.5.1.stable, v4.0.beta.custom_build [0a87ead64]
System information
Ubuntu 22.04.1
Issue description
For example, if project has name "300spartans", we can't enter the "org.godotengine.300spartans" to "Unique Name" field. We get the error "Invalid package name: A digit cannot be the first character in a package segment". But , if we enter "org.godotengine.$genname", there will be no error message. When exporting, the name will be set as "org.godotengine.spartans" without "300" in beginning, but user will not be informed about it. I think, it is not very good. It may not be worth making changes to the source code, but mention it in the documentation.
Steps to reproduce
See issue description.
Minimal reproduction project
N/A

It should be easy to add a warning if using $genname but the generated name would be invalid.

@akien-mga theoretically, the project name can start with other invalid characters, not just numbers.


Fancontrol: add test etc from #70933

Motivation for this change
@Zopieux submitted #70933 which includes a test and made me review my previous work.
Things done
Merge #69398 and #70933 adding a test, clarification and improving the service.

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

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

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

cc @markuskowa since you merged the previous work.

@GrahamcOfBorg test fancontrol


Prototyping Keyboard shortcut to bring up Input Simulation Cheat Sheet

Overview
Rough draft for implementing an input shortcut to bring up a cheat sheet for input simulation controls/features. When first getting started with MRTK, its hard to figure out the input simulation controls. Adding a cheat sheet which is quickly accessible should ease the experience for first time users.
Changes

Fixes: #7675

Updated with more accessible interface and with placeholder tooltips etc. reworked

/azp run

/azp run

/azp run


Fix build for if_ruby

Patch is already produced https://groups.google.com/forum/#!msg/vim_dev/V44Im17o9jQ/UjUMH30IAgAJ


IpBlocks.GetItems should return IpBlock Slice, not pointer to IpBlock Slice

The function's comment states that the zero value of []IpBlock should be returned if Items is nil. Why can't we then return a slice instead of a slice pointer?
https://github.com/ionos-cloud/sdk-go/blob/207dec5b5aec6512e821aaa9b402dd755d5a1f90/model_ip_blocks.go#L149

Thanks, makes sense, I think this would be a breaking change. We will prioritize appropriately.

Fixed in https://github.com/ionos-cloud/sdk-go-bundle/ v2


Single server smart graphs

A new subsection "Testing Graphs on Single Server" within the "Graphs" is created. It describes the new feature of creating SmartGraphs (of different kinds: also Disjoint, Hybrid ans HybridDisjoint) and SatelliteGraphs on a single server instead of on a cluster, see https://github.com/arangodb/arangodb/pull/14821. The graphs can be created with all properties related to sharding, however the properties do not affect anything. Then the graphs can be dumped and restored in a cluster with all sharding properties, which become functional now.
This is  3.10 feature which doesn't appear in earlier versions.
Also in related documents (about smart graphs and satellite graphs) minor changes and some reformulations of the existing text are made.

verified with jenkins


[super_drag_and_drop] drag and drop plain text to flutter web failed

Hi. Thanks for the great package.
I am using super_drag_and_drop and I have encountered an issue when dragging text to a flutter web application (Chrome).
It is working in the macOS version.
The assertion in super_native_extensions/lib/src/web/drop.dart failed since the file is null.
@override
Future<VirtualFileReceiver?> createVirtualFileReceiver(
  DataReaderItemHandle handle, {
  required String format,
}) async {
  assert(file != null,
      'Should not have requested virtual file receiver without file');
  return _VirtualFileReceiver(format, file!);
}

I removed the assertion and changed it to return file == null ? null : _VirtualFileReceiver(format, file!); and it works
To reproduce:
Simply drag an drop text from chrome or excel to a flutter web app.
flutter version: 3.7.0
super_drag_and_drop: 0.2.0
chrome: 110.0.5481.77

Please reopen if you still have this problem with 0.2.1.


No icon in tab

Please add support to having no icon in tab. Now it is possible to use Colors.transparent for icon's color but there is a gap...

Will do it tomorrow and update you asap

Fixed! Thank you for your suggestion


https://www.google.com/webhp?hl=ar&sa=X&ved=0ahUKEwjcpdudl-rmAhXpsIsKHVWND6kQPAgH

VSCode Version:
OS Version:

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes/No

Yeah, this is sketchy. I would advise against visiting the link in the title.

@mohmmedmeshi thank you for the issue. Please open a new issue with detailed steps to reproduce the problem and with an explanation, why someone should follow a particular link.


Add before / after indicators for trivia in Oak view

I propose we add some indicators in the Oak view, showing whether the given trivia comes from ContentBefore or ContentAfter.
Something like this:

Alternatively, maybe just highlight the parent node on hover over trivia?

PS: do you know about graph mode?

Oh, wow! Sweet! I missed this. That's something that should not be hidden in the settings. A little toggle button maybe?

A little toggle button maybe?

No, I don't think that is really necessary. Overall, this is nice and useful but I rarely use it, to be honest.

I took this into account in https://github.com/fsprojects/fantomas-tools/pull/566


Questions about the loss used for optimizing the proxy model

@sangmichaelxie It seems that the loss used for optimizing the proxy model in the code is different from the one described in the paper.
loss = (pertoken_loss * curr_domain_weights.detach()).sum() / normalizer

In the code, you directly use the proxy model's own loss here to optimize. But in the paper, the loss seems to be the minimax loss which uses the excess loss. So which one should I conform? Or there is something wrong with my understanding. Thanks.

In the code, you directly use the proxy model's own loss here to optimize. But in the paper, the loss seems to be the minimax loss which uses the excess loss. So which one should I conform? Or there is something wrong with my understanding. Thanks.

The reference model loss is a constant with respect to the proxy model's parameters, so it doesn't affect the proxy model update and we omit it. The reference model loss does affect the domain weight update.

When training the main model, what is the difference between resampling the data from the new distribution or using new weights to re-weight loss? Will these two have a significant performance gap?

Check out the main paragraph in pg 9 and Table 3b in the paper.

I checked it, and it makes sense. Thank you!


Adding KEYS required for release

(cherry picked from commit b3885f73e4c518b45368ecbdb9d730826b2bad06)

as we discussed this morning -- KEYS should be in dist.apache.org -- not here in the repo:
https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS
https://dist.apache.org/repos/dist/release/incubator/trafficcontrol/KEYS


[Network]Support p2s vpn gateway and vpn server configuration

Fix Azure/azure-cli#13020
This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

add to S171


ModuleNotFoundError: No module named 'snakefmt.config'

Trying to setup snakefmt with neovim.  When I installed it using the vim-plug manager using:
Plug 'snakemake/snakefmt'

The command produces this message:
Error detected while processing function provider#python3#Call:
line   18:
Traceback (most recent call last):
  File "<string>", line 19, in <module>

Which suggests it didn't successfully install. When I launch vim, I get this error:
Error detected while processing function provider#python3#Call:
line   18:
Traceback (most recent call last):
  File "<string>", line 19, in <module>
ModuleNotFoundError: No module named 'snakefmt.config'

Running :checkhealth  provider from neovim indicates that Python 3.8.6 is being used and supporting neovim:
## Python 3 provider (optional)
  - INFO: Using: g:python3_host_prog = "python3"
  - INFO: Executable: /projects/ace_benchmarking/miniconda3/bin/python3
  - INFO: Python3 version: 3.8.6
  - INFO: python3-neovim version: 0.4.2
  - OK: Latest python3-neovim is installed: 0.3.1

I've also got snakefmt installed (via conda):
> which snakefmt
/projects/ace_benchmarking/miniconda3/bin/snakefmt

I must be doing something wrong here. The instructions don't explicitly say neovim (https://github.com/snakemake/snakefmt/blob/master/docs/editor_integration.md#Vim). Is the possible that it doesn't work with neovim?

I'm afraid I don't use neovim, so we would need to find advice from someone who does :thinking:

Hi @tinyheero , I now use neovim, and successfully installed and ran snakefmt inside neovim.
Does this error still occur for you?
If so, could you try installing snakefmt via pip not conda?


(linux) Ignore PrtScr in the autocomplete combobox

Capturing a screenshot doesn't work anyway but it at least doesn't change the state of the dropdown
Fixes #2570

Looks good!


changed the arrival state computation method in residual calculation segement of Atom

made modifications to atom.hpp file.

MOST IMPORTANT CHANGE - in the residual calculation portion, instead of computing the arrival state directly by passing the timeOfFlight as the argument to the sgp4 propagator, the timeofflight seconds were added to the departure epoch thus giving an arrival epoch. this arrival epoch was passed as an argument to the sgp4 propagator to compute the arrival state.

The return type of executeAtomSolver was modified. some 64 bit distros of linux are known to have the problem of not providing the utility.h library with the standard library distributuion and because of that std::pair type output was creating errors. Now the output type is a 6 element std::vector.

Some housekeeping variables were passed to the main code using the atom solver. To enable this, some new arguments were added to the executeAtomSolver function.

Hi Abhishek,
Can you split these off into different commits? Pull requests should ideally be single features, bug fixes etc. In this case, the residual computation bug is the important bit. The other changes we haven't discussed. For (2), there's another solution we should use. (3) is unclear to me and the code is not enlightening.


gitversion error

hi im not sure what the go is with this error Severity	Code	Description	Project	File	Line	Suppression State
Error		The command ""C:\Users\J.nuget\packages\gitversion.msbuild\5.11.1\tools\net48/gitversion.exe" "C:\Users\J\Downloads\obfuscar-master\Console"  -output file -outputfile obj\gitversion.json" exited with code 1.	Obfuscar.Console		gitversion work s though so i dont know

https://gitversion.net/docs/reference/requirements#repository
This repo uses GitVersion. So, if you want to compile the source code successfully you need to do a full clone, but just download the ZIP package from GItHub.

i tried both and still get same error

It is impossible to get "the same error" when you use Git clone (at least the file path and error message should change).
Without further information on what exactly happened after you switched, no assistance can be provided.


[EventHubs] Connection idle timeout implementation

Can one of the admins verify this patch?

/azp run python - eventhubs - tests

/azp run python - eventhubs - tests


Fixed small grammar errors in ISSUE_TEMPLATE.md

Description
I fixed one small grammar error in the md file
Motivation and Context
It allows the md file to have clear and concise grammar
How Has This Been Tested?
By rereading the file
Types of changes

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

Checklist:

[x ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ x] I have read the CONTRIBUTING document.
[x ] I have added tests to cover my changes.
[ x] All new and existing tests passed.

Nice.
Thanks @rocketbear27 👍

No Problem! 👍


fold_object ---- (Implement a non-blocking, streaming codec for TopoJSON)

fold_object is like map_object but allows you to accumulate some result that is then returned to you.

Hi @patricoferris, What do you think? The test fails sometimes.

I am guessing the test is failing because we are not encoding the transform object (yet?)

It was failing because of a bug! We weren't decoding/encoding bounding boxes correctly, see the fix (which has now been merged) in #42 -- if you rebase your branch on main it should be good to go

Done. But the test is still failing on my end 😥

Another bug ^^" very similar to the last one, thank you for finding them! Should be good to rebase again and try

Works perfectly now :))

And promote the new result of mapping because you modified the test case :))

💃🏻💃🏻💃🏻💃🏻


Allow edits of prepared and preparing tables

[ch2211]
What's needed on quasar for ch2795.

@djspiewak hold off on merge on this this, there might be reason to make it breaking, not 100% sure yet.

Ok we're good to go with feature. The exception I was getting in the SDBE test was because the test assumed the response had a body, which it didn't have anymore.


Strings.xml google_app_id conflict with google-services.json

Expected Behavior
App compiles, Push notifications work, angels sing, the crowds rejoice
Actual Behavior
I get an error with gradle.
Error: Duplicate resources
I've traced the issue to the Strings.xml google_app_id which conflicts with the values.xml that gets generated from the google-services.json
If I comment out the strings.xml line
<!--string name="google_app_id">1234567890123456</string-->
then all compiles.
Reproduce Scenario (including but not limited to)
Google is making developers move from the cloud services to firebase-messaging.
When setting this up, google generates a google-services.json file which gets put at the base of the project.
Steps to Reproduce

set up android push plugin
set up firebase messaging
import the google-services.json file
build

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
android
I'm compiling in Android Studio 2.1.3
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
N/A
Cordova CLI version and cordova platform version
cordova --version                                    6.3.0
cordova platform version android          android 5.2.1

Plugin version
cordova plugin version | grep phonegap-plugin-push:

phonegap-plugin-push 1.8.0 "PushPlugin"
Sample Push Data Payload
      {
        "pn_gcm": {
            "data": "Test message to android!"
        }
    }

Sample Code that illustrates the problem
Logs taken while reproducing problem
:processDebugGoogleServices
Parsing json file: /Users/jeffwalters/Documents/phonegap apps/pushnotification/platforms/android/google-services.json
:generateDebugResources
:mergeDebugResources
Error:Error: Duplicate resources
:mergeDebugResources FAILED
Error:Execution failed for task ':mergeDebugResources'.

[string/google_app_id] /Users/jeffwalters/Documents/phonegap apps/pushnotification/platforms/android/res/values/strings.xml   [string/google_app_id] /Users/jeffwalters/Documents/phonegap apps/pushnotification/platforms/android/build/generated/res/google-services/debug/values/values.xml: Error: Duplicate resources
Information:BUILD FAILED

@cybermill yeah, FCM is not support by this plugin. It will be part of the 2.0.0 release which you can check out right now in the v2.0.x branch. Try that version and let me know if the problem still exists.

@macdonst:  Ahhh. Thought it must be something like that. I was puzzled by the lack of chatter about it in the issues forum. I'll check out the 2.0 version after the holiday and provide some feedback.
Great job with the plug-in. I used it with the iOS app and it works brilliantly.
Thanks so much for all you do.

here is the latest.  The 2.0.x branch worked, but only after I commented out the strings.xml google_app_id as described in my bug report. If you do that, I have it working on my samsung device so hurray!
Thanks for your help on this macdonst.

Implementation notes:

write a hook that copies google-services.json to platforms/android
remove setting of google_app_id to strings.xml
add google services gradle plugin
switch push plugin logic to read sender ID from gcm_defaultSenderId

I've never had to install a branch version like this before. I tried googling and couldn't figure it out.
How do I get version 2.0.0 into my app?

@leetheguy install via the github url:
cordova plugin add https://github.com/phonegap/phonegap-plugin-push#v2.0.x
I should have a release candidate out in npm sometime this week.

add the next lines to "config.xml" file

then try to install
$ cordova plugin add phonegap-plugin-push

Hi, v2.0.0-rc2 fails for me with another exception during cordova build process:
cordova build --release android

FAILURE: Build failed with an exception.

* Where:
Script 'D:\customer\platforms\android\phonegap-plugin-push\customer-push.gradle' line: 12

* What went wrong:
A problem occurred evaluating root project 'android'.
> Failed to apply plugin [id 'com.google.gms.google-services']
   > For input string: "[9"

I have following plugins installed:
com.googlemaps.ios 2.1.1 "Google Maps SDK for iOS"
com.phonegap.plugins.nativesettingsopener 1.3 "Native settings"
cordova-custom-config 3.1.2 "cordova-custom-config"
cordova-plugin-appsee 2.3.11 "Appsee"
cordova-plugin-compat 1.1.0 "Compat"
cordova-plugin-console 1.0.5-dev "Console"
cordova-plugin-crosswalk-webview 2.1.0 "Crosswalk WebView Engine"
cordova-plugin-device 1.1.4-dev "Device"
cordova-plugin-dialogs 1.3.0 "Notification"
cordova-plugin-firebase 0.1.18 "Google Firebase Plugin"
cordova-plugin-geolocation 2.4.1-dev "Geolocation"
cordova-plugin-globalization 1.0.4 "Globalization"
cordova-plugin-google-analytics 1.7.2 "Google Universal Analytics Plugin"
cordova-plugin-googlemaps 1.4.0 "phonegap-googlemaps-plugin"
cordova-plugin-hotline 0.6.1 "Hotline plugin for Phonegap"
cordova-plugin-intercom 3.0.19 "Intercom"
cordova-plugin-mixpanel 2.2.1 "Mixpanel"
cordova-plugin-network-information 1.3.1-dev "Network Information"
cordova-plugin-request-location-accuracy 2.2.0 "Request Location Accuracy"
cordova-plugin-splashscreen 4.0.1 "Splashscreen"
cordova-plugin-statusbar 2.2.0 "StatusBar"
cordova-plugin-vibration 2.1.3 "Vibration"
cordova-plugin-whitelist 1.3.1-dev "Whitelist"
cordova.plugins.diagnostic 3.3.2 "Diagnostic"
ionic-plugin-deploy 0.6.5 "IonicDeploy"
ionic-plugin-keyboard 2.2.1 "Keyboard"
me.tonny.cordova.plugins.multidex 0.1.0 "Cordova Multi Dex"
phonegap-plugin-push 2.0.0-rc2 "PushPlugin"
urbanairship-cordova 6.3.0 "Urban Airship"

Any ideas?

i had a lot of issues with phonegap-plugin-push, nowadays i have been working with "onesignal native plugin" for push notifications are great and free!, it's easy to work. Tell me if you want more information.

Having same problem as @evereq:
* Where:
Script '/Users/kirill/Idea/NCApp/platforms/android/phonegap-plugin-push/cordova-push.gradle' line: 12

* What went wrong:
A problem occurred evaluating root project 'android'.
> Failed to apply plugin [id 'com.google.gms.google-services']
   > For input string: "+"

Javierfernandez93
Did the trick. BTW, i needed to upgrade my Cordova up to 6.4.0 to allow that plugin version be installed.

hi @kirillgroshkov, what did you do? did you manage to so solve the problem

Where:
Script '/Users/kirill/Idea/NCApp/platforms/android/phonegap-plugin-push/cordova-push.gradle' line: 12

What went wrong:
A problem occurred evaluating root project 'android'.

Failed to apply plugin [id 'com.google.gms.google-services']
For input string: "+"

Hi @kirillgroshkov and @macdonst ; we have the same problem:
Failed to apply plugin [id 'com.google.gms.google-services']
For input string: "+"
Is there already a fix or workaround for this now?
Thanks!

@rickgoud
You must to change the + string to stable version i actually works on 9.8.0 for all google services

I tried all the version of phonegap-plugin-push but still facing the issue:
`A problem occurred evaluating root project 'android'.

Failed to apply plugin [id 'com.google.gms.google-services']
For input string: "+"`

Ionic Info
Cordova CLI: 7.0.1  Ionic Framework Version: 2.0.0-rc.4 Ionic CLI Version: 2.2.1 Ionic App Lib Version: 2.2.0 Ionic App Scripts Version: 1.3.7 ios-deploy version: Not installed ios-sim version: Not installed OS: macOS Sierra Node Version: v8.0.0 Xcode version: Not installed
Phonegap plugins
phonegap-plugin-push 2.0.0-rc2 "PushPlugin"
I followed this link to install push notification
https://docs.ionic.io/services/push/#setup
Using this command: cordova plugin add phonegap-plugin-push --variable SENDER_ID=12341234 --save
I am facing duplicate app id issue:
https://github.com/phonegap/phonegap-plugin-push/issues/1660
Please help

For me, com.google.gms.google-services was already installed, so I could remove from /platforms/android/phonegap-push-plugin/[appname]-push.gradle:
cdvPluginPostBuildExtras << {
    apply plugin: 'com.google.gms.google-services'
}

Seems like the 'apply plugin' gradle command needs some check if 'plugin' doesn't already exist.

Fixed in the 2.0.0 release candidates.

Hi @macdonst , sorry to bother again.
I'm getting bit by this (duplicate resources for google_app_id and google_api_key) using version 2.1.3 of the plugin (cordova-android 6.4.0).
The only way I can build is deleting the lines under "res/values/strings.xml" key in android.json.
But then the plugin never registers (the registration handler is never called) and I am guessing it has to do with this, because it did before.
I am using the following plugins:
<plugin name="ionic-plugin-keyboard" spec="^2.2.1" />
    <plugin name="cordova-plugin-whitelist" spec="^1.3.3" />
    <plugin name="cordova-plugin-device" spec="^1.1.7" />
    <plugin name="cordova-plugin-splashscreen" spec="^4.1.0" />
    <plugin name="cordova-plugin-ionic-webview" spec="^1.1.16" />
    <plugin name="cordova-plugin-badge" spec="^0.8.7" />
    <plugin name="cordova-plugin-local-notification" spec="~0.9.0-beta.2" />
    <plugin name="phonegap-plugin-push" spec="2.1.3">
        <variable name="FCM_VERSION" value="11.6.2" />
        <variable name="SENDER_ID" value="XXXXXX" />
    </plugin>

Any suggestions?


Support for React 19

Hi,
I am using the latest React 19 release (19.0.0-rc-02c0e824-20241028) along with react-dom in my project. While integrating @studio-freight/react-lenis, I encountered a peer dependency conflict because the package currently specifies:
ERESOLVE unable to resolve dependency tree peer react-dom@"^17 || ^18" from @studio-freight/react-lenis
Would it be possible to add support for React 19 in the peer dependencies?
Thanks for the amazing library, and I’d be happy to provide more details if needed!

@studio-freight/react-lenis has been deprecated, please use lenis/react instead


⚠ Bofrost.de has degraded performance

In 7c4a80e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2061 ms

Resolved: Bofrost.de performance has improved in 1d1f27a after 1 hour, 18 minutes.


Unittests with incompatible pytest annotations

Pytest parametrization annotations are incompatible in unittests. From their documentation:

The following pytest features do not work, and probably never will due to different design philosophies:
• Fixtures (except for autouse fixtures, see below);
• Parametrization;
• Custom hooks;

One option might be to replace the pytest annotations with annotations from parameterized.

This error likely was my doing with the initial TES version parametrization effort, apologies! Happy to create a PR for this and can update this issue.

Just to confirm, @LakiG would you prefer tes-compliance-suite to use unittest or pytest for internal testing?

Closing this as test structure has been updated in latest release.


Remove support for upstream mysql java connector

The connector can be installed using the puppet/mysql_java_connector
module inside a stash profile.

two points:

could you extend the readme and mention the puppet/mysql_java_connector module and a link to it?
not sure if we already want to write the changelog now

Less depedencies and still easy to add if needed... Like this one ;) (+1 for link to mysql_java_connectore module in readme)

Thanks, updated readme.


Rosetta transaction integration tests

Goal of this PR
Fixes #46.

add transaction integration tests
move common code for integration tests to a separate file
minor tweaks to balance and block integration tests so that they're more uniform

Checklist

[x] Is on the right branch
[x] Documentation is up-to-date
[x] Tests are up-to-date

Nice 👍
Be honest, don't you think the validation-related stuff reads much more smoothly now?
validateTx: validateTransfer(op),

Isn't it super obvious / easy to follow?

Yes, it's much cleaner and concise :)
Reminds me of the notion that the shortest distance between two points is a straight line. Kind of like that but with naming.


dial tcp: i/o timeout

During hackfest in Boston, several users (but not all) experience the following issue. Any thoughts?
odl_user@Azure:~/kubernetes-hackfest$ helm upgrade --install data-api ~/kubernetes-hackfest/charts/data-api --namespace hackfest

Error: UPGRADE FAILED: Get https://aksodluser-aks-rg-odluser22-08dbf2-9cd3b3e6.hcp.eastus.azmk8s.io:443/api/v1/namespaces/kube-system/configmaps?labelSelector=NAME%!D(MISSING)data-api%!C(MISSING)OWNER%!D(MISSING)TILLER%!C(MISSING)STATUS%!D(MISSING)DEPLOYED: dial tcp: i/o timeout

This occurs for all charts.

Follow-up: I could not replicate the problem. About 1/4 of the event attendees had this issue. There was some effort to debug, but I could not let us get bogged down. After lunch, attendees who faced the issue partnered with people who successfully deployed the Service Tracker charts, which worked out well.
Ahead of the Friday event in New York, if anyone has debugging suggestions, please chime in.
Note: https://github.com/helm/helm/issues/4171 refers to "a bug in Azure proxy pod," but this is from June 2018. Still relevant, perhaps?

Apparently there were some intermittent issues related to Tiller and some clusters yesterday. I think it is something that can be ignored for now. But be sure your Helm client is up to date just to be safe.

I've hit this today, AKS deployed today and Helm v2.13.1 client and server.

Same here. Suddenly, all newly deployed pods with Tiller report this issue. All Tiller pods deployed previously, keep working as before.


TabBar labelStyle.color and unselectedLabelStyle.color does not take effect

Steps to Reproduce
1.(similar issue #67342 but on tab bar) Execute flutter run on the code sample
void main() => runApp(MyApp());

class MyApp extends StatelessWidget{

  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage>  with SingleTickerProviderStateMixin {

  late final TabController _controller;

  @override
  void initState() {
    super.initState();
    _controller = TabController(length: 2, vsync: this);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Column(
        children: [
          Container(
            decoration: BoxDecoration(
              color: AppColors.background,
              borderRadius: BorderRadius.circular(12),
            ),
            child: TabBar(
              controller: _controller,
              labelStyle:TextStyle(
                fontWeight: FontWeight.w700,
                fontSize: 14,
                color:  Colors.white
              ),
              unselectedLabelStyle:
              TextStyle(
                  fontWeight: FontWeight.w700,
                  fontSize: 14,
                  color:  Colors.black
              ),
              indicator: BoxDecoration(
                  color: Colors.teal,
                  borderRadius: BorderRadius.circular(8)),
              tabs: [
                Tab(text: 'Tab 1'),
                Tab(text: 'Tab 2'),
              ],
            ),
          ),
          Expanded(
            child: TabBarView(
              controller: _controller,
                children: [
              Text('tab 1'),
              Text('tab 2'),
            ]),
          )
        ],
      )

    );
  }
}

Expected results:  unselected tab text color should be black
Actual results:   unselected tab text color is same as selected tab

Logs

[√] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.19043.1889],)
    • Flutter version 3.0.5 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (4 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\Users\asus\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

tab 1
tab 2

Hi @dayveedaniel, Thanks for filing the issue. I am able to reproduce the issue on the latest stable and the master channel. Looks like labelStyle.color   and unselectedLabelStyle.color are not taking effect while other properties seem to work.
https://user-images.githubusercontent.com/31410839/184602159-5c6256d6-c568-486b-a77a-3c664c4102e9.mov

code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage>
    with SingleTickerProviderStateMixin {
  late final TabController _controller;

  @override
  void initState() {
    super.initState();
    _controller = TabController(length: 2, vsync: this);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(),
        body: Column(
          children: [
            Container(
              decoration: BoxDecoration(
                color: Colors.cyan,
                borderRadius: BorderRadius.circular(12),
              ),
              child: TabBar(
                controller: _controller,
                labelStyle: TextStyle(
                    fontWeight: FontWeight.w700,
                    fontSize: 14,
                    color: Colors.red),
                unselectedLabelStyle: TextStyle(
                    fontWeight: FontWeight.bold,
                    fontSize: 20,
                    color: Colors.red),
                indicator: BoxDecoration(
                    color: Colors.teal, borderRadius: BorderRadius.circular(8)),
                tabs: [
                  Tab(text: 'Tab 1'),
                  Tab(text: 'Tab 2'),
                ],
              ),
            ),
            Expanded(
              child: TabBarView(controller: _controller, children: [
                Text('tab 1'),
                Text('tab 2'),
              ]),
            )
          ],
        ));
  }
}

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.4 21F79 darwin-arm, locale en-IN)
    • Flutter version 3.0.5 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (5 days ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc4)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 33.0.0-rc4
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.67.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (3 available)
    • sdk gphone arm64 (mobile) • emulator-5554 • android-arm64  • Android 11 (API 30) (emulator)
    • macOS (desktop)           • macos         • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)              • chrome        • web-javascript • Google Chrome 103.0.5060.114

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.1.0-0.0.pre.2009, on macOS 12.4 21F79 darwin-arm, locale en-IN)
    • Flutter version 3.1.0-0.0.pre.2009 on channel master at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9d96c7b8be (3 days ago), 2022-08-01 00:39:06 -0400
    • Engine revision cf0db3e546
    • Dart version 2.19.0 (build 2.19.0-49.0.dev)
    • DevTools version 2.16.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc4)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 33.0.0-rc4
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.69.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.46.0

[✓] Connected device (3 available)
    • sdk gphone arm64 (mobile) • emulator-5554 • android-arm64  • Android 11 (API 30) (emulator)
    • macOS (desktop)           • macos         • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)              • chrome        • web-javascript • Google Chrome 104.0.5112.79

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

looks like this is a breaking change (ref: https://github.com/flutter/flutter/pull/109541#issuecomment-1294241417). we'd want to wait until there was strong demand for the feature before we can make the change.

Since @werainkhatri isn't actively working on this. I've un-assigned you.

@TahaTesser - is this issue still unresolved?

looks like this is a breaking change (ref: #109541 (comment)). we'd want to wait until there was strong demand for the feature before we can make the change.

@TahaTesser - is this issue still unresolved?

Yes. this seems to be a breaking change. When @werainkhatri tried to fix this it broke the internal tests per your comment .
I can work on a fix with a breaking change migration guide to fix this issue if you want.

A fix would be good thing to work on, yes.


Drop unused views and tests

Proposed Changes

Drop unused views and tests

Merge Checklist

[ ] Tests added/fixed
[ ] Update docs in /docs
[ ] Linting Complete
[ ] Any other necessary step

Only PR's with test cases included and passing lint and test pipelines will be reviewed
@ohcnetwork/care-backend-maintainers @ohcnetwork/care-backend-admins

End of an Era, Gone but not forgotten - ICD 11


kubectl get: when resource name is in form resource_type/name the error occurs

What happened?
When command kubectl get pods,deployments deployment.apps/coredns -n kube-system is executed it returns the error:
error: arguments in resource/name form may not have more than one slash

What did you expect to happen?
kubectl get should return deployment.apps/coredns resource.
How can we reproduce it (as minimally and precisely as possible)?
Create local cluster with kind/minikube/etc:
kind create cluster

By default it should have coredns deployment, try to run:
kubectl get pods,deployments deployment.apps/coredns -n kube-system

Anything else we need to know?
It's not necessary should be treated as a bug but maybe as a bad error description. Up to discussion.
Kubernetes version

$ kubectl version
Client Version: v1.29.5
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.2

Cloud provider

Local setup with Kind

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Container runtime (CRI) and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

@ardaguclu The error is error: arguments in resource/name form may not have more than one slash. Fixed original post. Thanks for noticing!

I'm closing this issue, if you don't mind. Thank you.
/close


Filter lookup results from interfaces in visitMemberExpr.

Fixes #1377
This change is intended to fix only #1377. It might not be general enough. For example, I used declRef.parent.as<InterfaceDecl>() to test if a lookup result is an abstract requirement. There might already be some helper functions out there for such tests. There might also be other cases that need filtering.

Our approach to resolving which declaration is being referred to in cases of overloading is starting to have a lot of ad hoc tweaks to deal with the combination of inherited and "direct" declarations in the candidate set. This kind of issue would only get worse if we start to introduce more general forms of inheritance and/or the ability to override inherited declarations.
At the same time, we also have some related issues when a method on a type wants to reference a global function with the same name as a member function of its type (such that overload resolution should be able to disambiguate things).
It seems that we are close to the point of needing a general-purpose system for representing the "priority" of a particular lookup result relative to the others, so that we can easily collapse an overloaded result by dismissing everything that doesn't have the "best" priority (to be used for things like direct member reference as is happening here). That priority could be attached to overload candidates at call sites and be used as a final disambiguation metric (after all the type-based resolution has occured).
The biggest missing thing here is a really concrete plan for what the priority should look like.


ASN1 Generation for OtherName parameter using UnstructuredName is incorrect

I generated a new certificate signing request with this config and command:
[req]
default_bits = 2048
prompt = no
default_md = sha256
distinguished_name = dn
req_extensions = req_ext

[ dn ]
CN=Blah

[ req_ext ]
subjectAltName = @altNames

[ altNames ]
otherName = 1.2.840.1135<IP_ADDRESS>;UTF8:blah

openssl req -new -out test.csr -config config.txt -nodes

1.2.840.1135<IP_ADDRESS> here is the UnstructuredName attribute defined in https://tools.ietf.org/html/rfc2985#section-5. This generated a CSR with this structure in the subjectAltNames field:
    0:d=0  hl=2 l=  21 cons: SEQUENCE
    2:d=1  hl=2 l=  19 cons:  cont [ 0 ]
    4:d=2  hl=2 l=   9 prim:   OBJECT            :unstructuredName
   15:d=2  hl=2 l=   6 cons:   cont [ 0 ]
   17:d=3  hl=2 l=   4 prim:    UTF8STRING        :blah

However, according to https://tools.ietf.org/html/rfc5280#section-<IP_ADDRESS>, the OtherName attribute should be a Sequence, like this structure generated by PKIjs:
    0:d=0  hl=2 l=  75 cons: SEQUENCE
    2:d=1  hl=2 l=  73 cons:  cont [ 0 ]
    4:d=2  hl=2 l=  71 cons:   SEQUENCE
    6:d=3  hl=2 l=   9 prim:    OBJECT            :unstructuredName
   17:d=3  hl=2 l=  58 cons:    cont [ 0 ]
   19:d=4  hl=2 l=  56 prim:     UTF8STRING        :blah

Unless there's some justification which I don't know about (which is entirely possible), this seems to go against spec. Am I missing something, or is this a bug?
The full asn1parse of the CSR is below:
    0:d=0  hl=4 l= 645 cons: SEQUENCE
    4:d=1  hl=4 l= 365 cons:  SEQUENCE
    8:d=2  hl=2 l=   1 prim:   INTEGER           :00
   11:d=2  hl=2 l=  15 cons:   SEQUENCE
   13:d=3  hl=2 l=  13 cons:    SET
   15:d=4  hl=2 l=  11 cons:     SEQUENCE
   17:d=5  hl=2 l=   3 prim:      OBJECT            :commonName
   22:d=5  hl=2 l=   4 prim:      UTF8STRING        :Blah
   28:d=2  hl=4 l= 290 cons:   SEQUENCE
   32:d=3  hl=2 l=  13 cons:    SEQUENCE
   34:d=4  hl=2 l=   9 prim:     OBJECT            :rsaEncryption
   45:d=4  hl=2 l=   0 prim:     NULL
   47:d=3  hl=4 l= 271 prim:    BIT STRING
  322:d=2  hl=2 l=  49 cons:   cont [ 0 ]
  324:d=3  hl=2 l=  47 cons:    SEQUENCE
  326:d=4  hl=2 l=   9 prim:     OBJECT            :Extension Request
  337:d=4  hl=2 l=  34 cons:     SET
  339:d=5  hl=2 l=  32 cons:      SEQUENCE
  341:d=6  hl=2 l=  30 cons:       SEQUENCE
  343:d=7  hl=2 l=   3 prim:        OBJECT            :X509v3 Subject Alternative Name
  348:d=7  hl=2 l=  23 prim:        OCTET STRING      [HEX DUMP]:3015A01306092A864886F70D010902A0060C04626C6168
  373:d=1  hl=2 l=  13 cons:  SEQUENCE
  375:d=2  hl=2 l=   9 prim:   OBJECT            :sha256WithRSAEncryption
  386:d=2  hl=2 l=   0 prim:   NULL
  388:d=1  hl=4 l= 257 prim:  BIT STRING

And I parsed the subjectAltName with:
openssl asn1parse -i -in test.csr -strparse 348

OpenSSL just puts in ASN1 what you've told it to do, you give it an OID and the data it needs to put there, and you told it to use an UTF8STRING.
What I see is:
OtherName ::= SEQUENCE {
        type-id    OBJECT IDENTIFIER,
        value      [0] EXPLICIT ANY DEFINED BY type-id }

And you say that the OID you used is for an unstructuredName. So the "ANY DEFINED BY" is seems to be:
unstructuredName ATTRIBUTE ::= {
           WITH SYNTAX PKCS9String {pkcs-9-ub-unstructuredName}
           EQUALITY MATCHING RULE pkcs9CaseIgnoreMatch
           ID pkcs-9-at-unstructuredName
   }

   PKCS9String { INTEGER : maxSize} ::= CHOICE {
           ia5String       IA5String (SIZE(1..maxSize)),
           directoryString DirectoryString {maxSize}
   }

I don't see anything about a sequence there.

OpenSSL just puts in ASN1 what you've told it to do, you give it an OID and the data it needs to put there, and you told it to use an UTF8STRING.
What I see is:
OtherName ::= SEQUENCE {
        type-id    OBJECT IDENTIFIER,
        value      [0] EXPLICIT ANY DEFINED BY type-id }

And you say that the OID you used is for an unstructuredName. So the "ANY DEFINED BY" is seems to be:
unstructuredName ATTRIBUTE ::= {
           WITH SYNTAX PKCS9String {pkcs-9-ub-unstructuredName}
           EQUALITY MATCHING RULE pkcs9CaseIgnoreMatch
           ID pkcs-9-at-unstructuredName
   }

   PKCS9String { INTEGER : maxSize} ::= CHOICE {
           ia5String       IA5String (SIZE(1..maxSize)),
           directoryString DirectoryString {maxSize}
   }

I don't see anything about a sequence there.

Of course I'm missing this part:
Attribute               ::= SEQUENCE {
      type             AttributeType,
      values    SET OF AttributeValue }
            -- at least one value is required

Of course I'm missing this part:
Attribute               ::= SEQUENCE {
      type             AttributeType,
      values    SET OF AttributeValue }
            -- at least one value is required

If I'm understanding this correctly, I believe there should be two sequences in SubjectAltName: one for GeneralNames and one for OtherName.
   SubjectAltName ::= GeneralNames

   GeneralNames ::= SEQUENCE SIZE (1..MAX) OF GeneralName

   GeneralName ::= CHOICE {
        otherName                       [0]     OtherName,
        rfc822Name                      [1]     IA5String,
        dNSName                         [2]     IA5String,
        x400Address                     [3]     ORAddress,
        directoryName                   [4]     Name,
        ediPartyName                    [5]     EDIPartyName,
        uniformResourceIdentifier       [6]     IA5String,
        iPAddress                       [7]     OCTET STRING,
        registeredID                    [8]     OBJECT IDENTIFIER }

   OtherName ::= SEQUENCE {
        type-id    OBJECT IDENTIFIER,
        value      [0] EXPLICIT ANY DEFINED BY type-id }

So, theoretically, the ASN1 should look like this:
    0:d=0  hl=2 l=  75 cons: SEQUENCE                               // SubjectAltName
    2:d=1  hl=2 l=  73 cons:  cont [ 0 ]
    4:d=2  hl=2 l=  71 cons:   SEQUENCE                             // OtherName
    6:d=3  hl=2 l=   9 prim:    OBJECT            :unstructuredName // type-id
   17:d=3  hl=2 l=  58 cons:    cont [ 0 ]                          // value
   19:d=4  hl=2 l=  56 prim:     UTF8STRING        :blah

Instead, it looks like this:
    0:d=0  hl=2 l=  21 cons: SEQUENCE
    2:d=1  hl=2 l=  19 cons:  cont [ 0 ]
    4:d=2  hl=2 l=   9 prim:   OBJECT            :unstructuredName
   15:d=2  hl=2 l=   6 cons:   cont [ 0 ]
   17:d=3  hl=2 l=   4 prim:    UTF8STRING        :blah

At least, that's my current understanding.

If I'm understanding this correctly, I believe there should be two sequences in SubjectAltName: one for GeneralNames and one for OtherName.
   SubjectAltName ::= GeneralNames

   GeneralNames ::= SEQUENCE SIZE (1..MAX) OF GeneralName

   GeneralName ::= CHOICE {
        otherName                       [0]     OtherName,
        rfc822Name                      [1]     IA5String,
        dNSName                         [2]     IA5String,
        x400Address                     [3]     ORAddress,
        directoryName                   [4]     Name,
        ediPartyName                    [5]     EDIPartyName,
        uniformResourceIdentifier       [6]     IA5String,
        iPAddress                       [7]     OCTET STRING,
        registeredID                    [8]     OBJECT IDENTIFIER }

   OtherName ::= SEQUENCE {
        type-id    OBJECT IDENTIFIER,
        value      [0] EXPLICIT ANY DEFINED BY type-id }

So, theoretically, the ASN1 should look like this:
    0:d=0  hl=2 l=  75 cons: SEQUENCE                               // SubjectAltName
    2:d=1  hl=2 l=  73 cons:  cont [ 0 ]
    4:d=2  hl=2 l=  71 cons:   SEQUENCE                             // OtherName
    6:d=3  hl=2 l=   9 prim:    OBJECT            :unstructuredName // type-id
   17:d=3  hl=2 l=  58 cons:    cont [ 0 ]                          // value
   19:d=4  hl=2 l=  56 prim:     UTF8STRING        :blah

Instead, it looks like this:
    0:d=0  hl=2 l=  21 cons: SEQUENCE
    2:d=1  hl=2 l=  19 cons:  cont [ 0 ]
    4:d=2  hl=2 l=   9 prim:   OBJECT            :unstructuredName
   15:d=2  hl=2 l=   6 cons:   cont [ 0 ]
   17:d=3  hl=2 l=   4 prim:    UTF8STRING        :blah

At least, that's my current understanding.

You need to tell OpenSSL what it needs to generate, see the ASN1_generate_nconf() manpage. I'm not sure what needs to be generated.

You need to tell OpenSSL what it needs to generate, see the ASN1_generate_nconf() manpage. I'm not sure what needs to be generated.

One thing that maybe wasn't clear is that this line:
 2:d=1  hl=2 l=  73 cons:  cont [ 0 ]

Is the sequence for OtherName that contains the OID and value, and you seem to want to add a new sequence in that sequence.

One thing that maybe wasn't clear is that this line:
 2:d=1  hl=2 l=  73 cons:  cont [ 0 ]

Is the sequence for OtherName that contains the OID and value, and you seem to want to add a new sequence in that sequence.

Thanks for the help! I'll start taking a look at telling OpenSSL how to generate the correct ASN1 in the config file. Out of curiosity, is it also possible to tell OpenSSL how to correctly parse OtherName with openssl req? Currently, when it tries to parse the OtherName parameter, its gives out a bunch of random junk for SubjectAltName:
Certificate Request:
    Data:
        Version: 0 (0x0)
        Subject: CN=Chris
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
                Public-Key: (4096 bit)
                Modulus:
                    <modulus>
                Exponent: 65537 (0x10001)
        Attributes:
        Requested Extensions:
            X509v3 Subject Key Identifier:
                67:FB:F5:5A:60:88:CA:B6:4E:15:7F:ED:55:0D:30:CF:AC:A8:58:00
            X509v3 Subject Alternative Name:
....:.8GGRS2PZN2PRTXLYHDBMWDR7GIPFSBNPEHFBLOYUVALHGZ4NMMRVA====
    Signature Algorithm: rsassaPss
         Hash Algorithm: sha256
         Mask Algorithm: mgf1 with sha256
         Salt Length: 0x20
         Trailer Field: 0xBC (default)

         <signature>

Thanks for the help! I'll start taking a look at telling OpenSSL how to generate the correct ASN1 in the config file. Out of curiosity, is it also possible to tell OpenSSL how to correctly parse OtherName with openssl req? Currently, when it tries to parse the OtherName parameter, its gives out a bunch of random junk for SubjectAltName:
Certificate Request:
    Data:
        Version: 0 (0x0)
        Subject: CN=Chris
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
                Public-Key: (4096 bit)
                Modulus:
                    <modulus>
                Exponent: 65537 (0x10001)
        Attributes:
        Requested Extensions:
            X509v3 Subject Key Identifier:
                67:FB:F5:5A:60:88:CA:B6:4E:15:7F:ED:55:0D:30:CF:AC:A8:58:00
            X509v3 Subject Alternative Name:
....:.8GGRS2PZN2PRTXLYHDBMWDR7GIPFSBNPEHFBLOYUVALHGZ4NMMRVA====
    Signature Algorithm: rsassaPss
         Hash Algorithm: sha256
         Mask Algorithm: mgf1 with sha256
         Salt Length: 0x20
         Trailer Field: 0xBC (default)

         <signature>

OpenSSL's encoding is correct. The SEQUENCE tag from OtherName is supposed to be omitted when wrapped in a GeneralName because the module defining GeneralName uses implicit tagging ("DEFINITIONS IMPLICIT TAGS ::="), and the GeneralName structure doesn't specify anything else.
This is, sadly, a rather confusing corner of ASN.1. Let's Encrypt has a nice overview of ASN.1 which discusses implicit vs explicit tagging and how you determine which is used where. (They're missing an even more confusing rule about CHOICE types, but OtherName is not a CHOICE type, so you don't have to worry about it here.)
https://letsencrypt.org/docs/a-warm-welcome-to-asn1-and-der/

OpenSSL's encoding is correct. The SEQUENCE tag from OtherName is supposed to be omitted when wrapped in a GeneralName because the module defining GeneralName uses implicit tagging ("DEFINITIONS IMPLICIT TAGS ::="), and the GeneralName structure doesn't specify anything else.
This is, sadly, a rather confusing corner of ASN.1. Let's Encrypt has a nice overview of ASN.1 which discusses implicit vs explicit tagging and how you determine which is used where. (They're missing an even more confusing rule about CHOICE types, but OtherName is not a CHOICE type, so you don't have to worry about it here.)
https://letsencrypt.org/docs/a-warm-welcome-to-asn1-and-der/

(I don't think what ASN.1 specified in the config file would affect this. That controls the value field of the OtherName, whereas the question here is about the OtherName structure itself. That part is something OpenSSL knows about and encodes on its own.)

(I don't think what ASN.1 specified in the config file would affect this. That controls the value field of the OtherName, whereas the question here is about the OtherName structure itself. That part is something OpenSSL knows about and encodes on its own.)

By the way, if PKIjs is including the implicit SEQUENCE tag, you may want to file a bug on their end. It's possible they've gotten the tagging wrong.

By the way, if PKIjs is including the implicit SEQUENCE tag, you may want to file a bug on their end. It's possible they've gotten the tagging wrong.

With that junk, I assume that it's not actually properly encoded,
and might be with the extra sequence in it. It should output:
"othername:". And least that's what it does with x509,
I assume req does the same.
I have a certificate that has:
X509v3 Subject Alternative Name:
email:EMAIL, othername:
Which asn1parse shows as:
0:d=0  hl=2 l=  42 cons: SEQUENCE
2:d=1  hl=2 l=  29 prim:  cont [ 1 ]
33:d=1  hl=2 l=   9 cons:  cont [ 0 ]
35:d=2  hl=2 l=   3 prim:   OBJECT            :description
40:d=2  hl=2 l=   2 cons:   cont [ 0 ]
42:d=3  hl=2 l=   0 prim:    PRINTABLESTRING   :

With that junk, I assume that it's not actually properly encoded,
and might be with the extra sequence in it. It should output:
"othername:". And least that's what it does with x509,
I assume req does the same.
I have a certificate that has:
X509v3 Subject Alternative Name:
email:EMAIL, othername:
Which asn1parse shows as:
0:d=0  hl=2 l=  42 cons: SEQUENCE
2:d=1  hl=2 l=  29 prim:  cont [ 1 ]
33:d=1  hl=2 l=   9 cons:  cont [ 0 ]
35:d=2  hl=2 l=   3 prim:   OBJECT            :description
40:d=2  hl=2 l=   2 cons:   cont [ 0 ]
42:d=3  hl=2 l=   0 prim:    PRINTABLESTRING   :

@kroeckx do you mean the output in https://github.com/openssl/openssl/issues/13881#issuecomment-763158654? I think that may be OpenSSL trying to parse an incorrectly-encoded GeneralName.otherName, from whatever other software (PKIjs?) that @cmdli is using. @cmdli, is that right?
When I run the command from the original report, openssl req manages to parse it back out just fine.
I think this bug has correctly identified that OpenSSL and the other software disagree on how to encode GeneralName.otherName. But I believe the error is in the other software, not OpenSSL.

@kroeckx do you mean the output in https://github.com/openssl/openssl/issues/13881#issuecomment-763158654? I think that may be OpenSSL trying to parse an incorrectly-encoded GeneralName.otherName, from whatever other software (PKIjs?) that @cmdli is using. @cmdli, is that right?
When I run the command from the original report, openssl req manages to parse it back out just fine.
I think this bug has correctly identified that OpenSSL and the other software disagree on how to encode GeneralName.otherName. But I believe the error is in the other software, not OpenSSL.

@davidben Yes, that garbled output is from a CSR generated by PKIjs. I'll be taking a closer look and maybe filing a bug with them.

@davidben Yes, that garbled output is from a CSR generated by PKIjs. I'll be taking a closer look and maybe filing a bug with them.

Yes, I think we agree that the other software is doing it wrong.
But it's also still unclear to me what is expected for the UnstructuredName. I think it's a PKCS9String, which is a CHOICE between ia5String and directoryString, where directoryString can be an UTF8String. So I think the "UTF8:blah" is wrong.

Yes, I think we agree that the other software is doing it wrong.
But it's also still unclear to me what is expected for the UnstructuredName. I think it's a PKCS9String, which is a CHOICE between ia5String and directoryString, where directoryString can be an UTF8String. So I think the "UTF8:blah" is wrong.

Oh, I see. I don't think that would affect the output in openssl req since OpenSSL doesn't parse OtherName values anyway. (It doesn't look like PKCS9String adds any tags to the ia5String and directoryString CHOICE arms. Rather it relies on the two arms having disjoint tag sets. So just emitting a UTF8String tag plain is right.)

Oh, I see. I don't think that would affect the output in openssl req since OpenSSL doesn't parse OtherName values anyway. (It doesn't look like PKCS9String adds any tags to the ia5String and directoryString CHOICE arms. Rather it relies on the two arms having disjoint tag sets. So just emitting a UTF8String tag plain is right.)

So in conclusion, there was no problem on the OpenSSL side, and
the config file he used was also correct.

So in conclusion, there was no problem on the OpenSSL side, and
the config file he used was also correct.

I am closing this issue since my personal questions/issue have been resolved. Thank you!

I am closing this issue since my personal questions/issue have been resolved. Thank you!


AttributeError: type object 'Tensor' has no attribute '_make_wrapper_subclass'

Describe the issue:
When I add the profiler as tutorial instructed below:
from nni.nas.strategy.middleware import Filter, Chain
from nni.nas.profiler.pytorch.flops import NumParamsProfiler
profiler = NumParamsProfiler(model_space, torch.randn(1, 1, 28, 28))
search_strategy = Chain(
    strategy.Random(),
    Filter(lambda sample: profiler.profile(sample) < args.max_num_params)
)

This error appeared:
  File "/usr/local/lib/python3.8/dist-packages/nni/nas/profiler/pytorch/utils/shape.py", line 301, in __new__
    r: ShapeTensor = torch.Tensor._make_wrapper_subclass(  # type: ignore
AttributeError: type object 'Tensor' has no attribute '_make_wrapper_subclass'

Environment:

NNI version: latest(Build from source and use Dockerfile in master branch)
Training service (local|remote|pai|aml|etc): local
Client OS: Unbuntu
Python version: 3.8
PyTorch/TensorFlow version: 1.10.2
Is conda/virtualenv/venv used?: No
Is running in Docker?: Yes

Configuration:

Experiment config (remember to remove secrets!): Same as Latest Version in Hello NAS
Search space: Random

Please upgrade your pytorch to some version between 1.11 and 1.13.

torch.randn(1, 1, 28, 28)

Thanks for the reply, when I upgrade to 1.13, it still have the same error.

Any noticeable warnings?

@ultmaster Here is the warning:
WARNING: Shape information is not explicitly propagated when executing aten.max_pool2d_with_indices.default, and and none of its parent modules have a shape inference formula. Shape inference is likely to fail. Module calling stack:
  - '' (type: __main__.MyModelSpace, NO shape formula)

Either register a formula for aten.max_pool2d_with_indices.default like what was done here:
https://github.com/microsoft/nni/blob/fea29c7cf5fea0bdfe11e908d50bb93337ef5d3f/nni/nas/profiler/pytorch/utils/shape_formula.py#L371
or use nn.MaxPool2d as it looks like it's already been registered:
https://github.com/microsoft/nni/blob/fea29c7cf5fea0bdfe11e908d50bb93337ef5d3f/nni/nas/profiler/pytorch/utils/shape_formula.py#L349

@matluster aAnother error appeared after I change to nn.MaxPool2d
[2023-05-11 18:30:08] Setting up...                                                                                [2/1871]
[2023-05-11 18:30:08] Web portal URLs: http://<IP_ADDRESS>:8080 http://<IP_ADDRESS>:8080 http://<IP_ADDRESS>:8080
[2023-05-11 18:30:08] Successfully update searchSpace.
[2023-05-11 18:30:08] Checkpoint saved to /root/nni-experiments/62m5slcr/checkpoint.
[2023-05-11 18:30:08] Experiment initialized successfully. Starting exploration strategy...
[2023-05-11 18:30:08] ERROR: Strategy failed to execute.
Traceback (most recent call last):
  File "/home/dzhang/Documents/ml-experimental/TinyNAS/nas_tutorial/random_search.py", line 173, in <module>
    exp.run(debug=True)
  File "/usr/local/lib/python3.10/dist-packages/nni/experiment/experiment.py", line 235, in run
    return self._run_impl(port, wait_completion, debug)                                                                                3" 18:00 11-May-23
  File "/usr/local/lib/python3.10/dist-packages/nni/experiment/experiment.py", line 204, in _run_impl
    self.start(port, debug)
  File "/usr/local/lib/python3.10/dist-packages/nni/nas/experiment/experiment.py", line 270, in start
    self._start_engine_and_strategy()
  File "/usr/local/lib/python3.10/dist-packages/nni/nas/experiment/experiment.py", line 230, in _start_engine_and_strategy
    self.strategy.run()
  File "/usr/local/lib/python3.10/dist-packages/nni/nas/strategy/middleware.py", line 89, in run
    return self._strategy.run()
  File "/usr/local/lib/python3.10/dist-packages/nni/nas/strategy/base.py", line 170, in run
    self._run()
  File "/usr/local/lib/python3.10/dist-packages/nni/nas/strategy/bruteforce.py", line 223, in _run
    self.engine.submit_models(model)
  File "/usr/local/lib/python3.10/dist-packages/nni/nas/strategy/middleware.py", line 292, in submit_models
    if self.filter_fn(model):
  File "/home/dzhang/Documents/ml-experimental/TinyNAS/nas_tutorial/random_search.py", line 163, in <lambda>
    Filter(lambda sample: profiler.profile(sample) < 1000*1000)
  File "/usr/local/lib/python3.10/dist-packages/nni/nas/profiler/profiler.py", line 44, in profile
    return self.expression.evaluate(sample)
  File "/usr/local/lib/python3.10/dist-packages/nni/mutable/symbol.py", line 175, in evaluate
    eval_args.append(arg.evaluate(values))
  File "/usr/local/lib/python3.10/dist-packages/nni/mutable/symbol.py", line 175, in evaluate
    eval_args.append(arg.evaluate(values))
  File "/usr/local/lib/python3.10/dist-packages/nni/mutable/symbol.py", line 175, in evaluate
    eval_args.append(arg.evaluate(values))
  [Previous line repeated 5 more times]
  File "/usr/local/lib/python3.10/dist-packages/nni/mutable/symbol.py", line 543, in evaluate
    value = next(values)
TypeError: 'SimplifiedModelSpace' object is not an iterator

This line here:
Filter(lambda sample: profiler.profile(sample) < 1000*1000)

It should be
Filter(lambda model: profiler.profile(model.sample) < 1000*1000)

@matluster The issue is solved. Thanks!


Joint Training Settings

Thank you for your great work!
I met a problem when I reproduced the result of your joint train. Do any weights of the backbone network need to be loaded during the joint training e.g. video-swin-b-kinetics400-22k. I observed that there is no option to load any weights in the published joint training script. Thanks!

Thanks for your interest.
Yes, you need to load the imagenet-pretrained backbone weights for joint training. Please download the swin/video-swin backbone weights from their official repos first, and then use the arg --backbone_pretrained to specify the weight path.
https://github.com/wjn922/ReferFormer/blob/9c8f237adc260c512a1c5ecfc7aee81b8282649a/opts.py#L33

Thank you for your reply!


[autolink-extract] Support (and ignore) LLVM IR files.

In Linux, the current implementation of swift-autolink-extract does not support LLVM IR files resulting from using LTO.
If one tries to build LLVM using LTO and then try to link one of the targets that use swiftc to link, but link against LLVM object files (like swift-plugin-server), swift-autolink-extract will fail saying that some object files are not valid.
To deal with LLVM IR files correctly, create and pass a llvm::LLVMContext around, which allows the APIs in llvm::object to read LLVM IR files. Additionally, handle the case of IRObjectFile when extracting, but perform no action.

@swift-ci please test

@swift-ci please test Linux platform

@drodriguez For bitcode linker input files, we already emit llvm.linker.options metadata and lld understands the given options, so we don't need swift-autolink-extract when performing LTO and the driver does not run swift-autolink-extract.
Also I'm curious what invokes swift-autolink-extract in your scenario.
https://github.com/apple/swift-driver/blob/49daa7ef0a0e662a95ec022869409d0ed9fb502f/Sources/SwiftDriver/Jobs/AutolinkExtractJob.swift#L20-L23

The setup is complicated. We always build Swift not standalone, but using LLVM_EXTERNAL_PROJECTS (unified). We want to enable LTO for LLVM. I think the same will happen if one builds standalone and tries to LTO the LLVM sources, generates .a files with embedded LLVM IR files, and then uses that build directory, but I haven't gone that path.
swift-plugin-server (https://github.com/apple/swift/blob/main/tools/swift-plugin-server/CMakeLists.txt) depends on something that brings some LLVM libraries into the linker step. We do not enable LTO for the Swift pieces, only for the LLVM pieces. Swift is being used as a linker driver only, I am afraid. Some objects are IR objects (the LLVM ones and C/C++ ones), but some are not (the Swift ones). I think the code you linked does not check for each of the files (and static archives) to be passed to the swift-autolink-extract step, but checks if the driver is doing LTO (which is not, we are just linking at that point).
The provided test jumps over the usage of the Swift driver, but more or less reproduces what's going on: some step compiles a C++ files with LTO, the Swift driver is invoked and given those bitcode objects (both in the form of single objects and also in the form of archives of bitcode objects), the driver passes all the paths with extensions .a and .o, and autolink-extract tries to read those objects.

@drodriguez Thank you for your elaboration! Given that situation, accepting and ignoring object files from foreign sources makes sense to me. I'd like to hear @compnerd opinion just in case.

@compnerd: added you as a reviewer. @kateinoigakukun above wanted to know your opinion.


Music break (Player just stop) 0.o

Aurora tends to stop playing, Resume or anything else doesn't work and you need to add in a new play command for it to play again.

 Music break (Player just stop) 0.o


Statement averaged scores

changes wrapped in with latest commit (pre-filter for blob, set no longer needed due to new orthogonal feature lexicons are each a set already).


File Format for editing in Mauro

Currently the format is DITA (/html) and needs to think about the future format.
GSC to org. James W, NO, Cath, for prep

HTML decided


refactor: invert test for early return

We now return early if the path is not a directory, which improves readability. There are no other changes in this commit.

This is a follow-up PR from https://github.com/nlnwa/hermetic/pull/16


Investigate LRU cache for constructed views

Right now, constructed components get "cached" in a dict for as long as the components.py module stays in memory. To prevent excess memory consumption for long-lived processes, it might make sense for that cache to be smarter and only keep the last... 50?... 100? classes. A simple LRU cache would probably be sufficient.
I have not seen this actually occur in production, but I think it could potentially happen.

Done in https://github.com/adamghill/django-unicorn/commit/c696b3ad146c467ac8f104a2e64083798428dc95.


Can't install since xCode 7.1.1 update

I were trying to install it / clone+build, doesn't work. Nothing changes.
Xcode Version 7.1.1 (7B1005)

I have same problem.

What version of Alcatraz do you currently have installed?

I have the same issue on Xcode 7.0.1 and Alcatraz 1.1.13

I reinstalled and it works. Maybe the first time I ignore this message.
https://gyazo.com/a19db2bbec47c04b0ecf6c34dbeaa168

I have the same issue.
I have uninstalled Alcatraz (using the instruction on the website) and reinstalled.  No luck

same problem here

I have the same problem on Xcode 7.1.1 & Alcatraz 1.1.13

I have the same issue.

I have the same problem on Xcode 7.1.1 and the Alcatraz new version

I have same problem.

uninstall + install helped me today)

+1

I have the same problem on Xcode 7.1.1

@ngothanhtai , I select load bounds  ...but still can't  install any plugin ..   why ?

I updated mac os capitan ,  xcode could installed plugin ...   if anyone like me , you may try it.

What worked for me was completely removing Alcatraz first. Then I cloned the repo and build the project myself from Xcode.
Next I installed the VVDocumenter plugin and it works fine now.

same same

Cloning the Repo and build the project brings back Package Manager option.

@rizwankce : sorry, it does not.

@fvvliet I have followed these steps.

Removed all files related to Alcatraz
Cloned Repo and build using xCode
Quit the xCode and reopened.
Clicked "Load Bundle" when the xCode asked

@rizwankce : Ok I'll try again, thanks

I have already solved the problem.
And I write a shell script to reload all of Xcode plug-ins.
Portal Here!
It works for me.

Thanks @ForkPanda! it works!

Thanks @ForkPanda!

thanks ForkPanda!

Thank you. @rizwankce
It works!

Thank you @ForkPanda, works like a charm :)

thank you @ForkPanda

The issue was probably caused by two things:

Alcatraz being added to the "skipped plugins" list of Xcode, which was fixed in #315
All plugins not including the new Xcode UUID just after its release, which is still a problem, but progress on this will be tracked in #442

Closing as this is becoming a "thank you" thread :). Please leave a comment if you feel this issue was not yet fixed and this should be left open. Thanks everyone!


CommandCompletion.CompleteInput finds a lot of results even with empty Runspace

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
When creating a Runspace from a completely empty InitialSessionState (InitialSessionState.Create()), the CommandCompletion should not find any completions, since the runspace does not contain any commands, aliases, types, variables, functions or even assemblies.
However, it still returns all the results it can find on my machine (C# console example with Nuget package Microsoft.PowerShell.SDK installed):
using System;
using System.Management.Automation;
using System.Management.Automation.Runspaces;

var emptySessionState = InitialSessionState.Create();

Runspace.DefaultRunspace = RunspaceFactory.CreateRunspace(emptySessionState);
Runspace.DefaultRunspace.Open();

var commandCompletion = CommandCompletion.CompleteInput(input: "get-a", cursorIndex: 5, default);
foreach (var completion in commandCompletion.CompletionMatches) {
    Console.WriteLine($"Completion match: {completion.CompletionText}");
}
Console.WriteLine($"Found {commandCompletion.CompletionMatches.Count} completions in total.");

More information: The CommandCompletion.CompleteInput creates a PowerShell instance using the thread static Runspace, that's why I set Runspace.DefaultRunspace above. The code will throw an exception otherwise. Since it uses the Runspace I just created from an empty session state, I expect the completions to take that into account as well.
Expected behavior
Found 0 completions in total.

Actual behavior
... # lots of lines above here
Completion match: Get-AzWebPubSubSku
Completion match: Get-AzWebPubSubUsage
Completion match: Az.Sql\Get-AzSqlDatabaseVulnerabilityAssessmentSetting
Completion match: Az.Sql\Get-AzSqlInstanceDatabaseVulnerabilityAssessmentSetting
Completion match: Az.Sql\Get-AzSqlServerAdvancedThreatProtectionSetting
Found 1108 completions in total.

Error details
No response
Environment data
Not really applicable to the C# code, but I tried SDK version `7.2.5` and `7.3.0-preview.5`, same issue. But, since I don't want the issue to be rejected on a technicality, here is the `$PSVersionTable` output of my system:

Name                           Value
----                           -----
PSVersion                      7.2.5
PSEdition                      Core
GitCommitId                    7.2.5
OS                             Linux 5.13.0-52-generic #59~20.04.1-Ubuntu SMP Thu Jun 16 21:21:28 UTC 2022
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Yes, that does indeed work!
Thank you very much!


Swift 4.1 conversion.

Thanks for your efforts. The insight of removal of the link attribute if there is one is cool and helped on what I'm working on.

I did this work in Xcode 9.3 for Swift 4.1. Not having as many lucky breaks in Xcode 9.2 in Swift 4. It crashes where I updated these coercions: ...    @objc optional func linkLabel(_ label: INDLinkLabel, attributesForURL URL: URL, originalAttributes: [NSAttributedStringKey: Any], proposedAttributes: [NSAttributedStringKey: Any]) -> [NSAttributedStringKey: Any]
Any insight welcome.


请问如何单独部署prober,并且与服务端相连?

What would you like to be added:
Why is this needed:

n9e-prober etc/identity.yml etc/address.yml etc/prober.yml 打个包放目标机器上跑就行了。当然配置文件中的ip改成实际环境的ip


Forgot password form submission crashes when notifier is disabled

Steps to reproduce the issue:

Ensure notifier is disabled.
If a user is logged in, log him off.
 (or just go to /forgot).
Enter a valid email address and click on "Reset Password!".

In the browser console you'll see this error:

And in the Terminal you'll see this:

The error is being thrown in this line, because data does not exists in that scope.

I've been working on this on the release-0.9.0 branch; I'll be shipping the fix today.


.sfw failing to enable word filtering

As seen here:
https://i.imgur.com/q0r2Jbp.png
The permissions are all in order and no roles are ignored, so I am not sure what's causing this issue, but it is very much happening.

Filtered words will not work on people with server owner or people with admin roles.

Ah, thank you. I was thoroughly confused for a while there.


TypeError: Cannot read property 'match' of undefined

Hi,
I just updated to 1.7.3 and now get the following error:
Mar 14 16:58:48 parenthoods-prod-new app/workerPushNotification.1:      at apnCertificateFromPem (/app/node_modules/apn/lib/credentials/parsePemCertificate.js:25:24) ```

Any ideas?

Thanks

Luke

When I log the certificate, the feedback service outputs all the required variables including "type" but when sending a push notification, only the body is shown.
If I remove the check in parsePemCertificate.js at line 25 then the push notifications reach the devices however I get the following error:
at Object.pki.certificateFromAsn1 (/app/node_modules/apn/node_modules/node-forge/js/x509.js:1258:17)
at apnCertificateFromPem (/app/node_modules/apn/lib/credentials/parsePemCertificate.js:30:36)
at parse (/app/node_modules/apn/lib/credentials/parse.js:9:24)
at /app/node_modules/apn/lib/connection.js:155:15 ```

Would you be able to email me a copy of the certificate? Something definitely isn't right about the file format and I'd like to figure out where the problem lies.
\A

On 14 Mar 2015, at 11:56, Luke Rhodes<EMAIL_ADDRESS>wrote:
When I log the certificate, the feedback service outputs all the required variables including "type" but when sending a push notification, only the body is shown.
If I remove the check in parsePemCertificate.js at line 25 then the push notifications reach the devices however I get the following error:
GMT apn Error: Cannot read X.509 certificate. ASN.1 object is not an X509v3 Certificate.
at Object.pki.certificateFromAsn1 (/app/node_modules/apn/node_modules/node-forge/js/x509.js:1258:17)
at apnCertificateFromPem (/app/node_modules/apn/lib/credentials/parsePemCertificate.js:30:36)
at parse (/app/node_modules/apn/lib/credentials/parse.js:9:24)
at /app/node_modules/apn/lib/connection.js:155:15

Reply to this email directly or view it on GitHub.

Also, which OS and version of node.js are you using?
I can't find any code-path which would cause the error you're seeing, unless Regular Expressions are somehow broken(!)

Closing due to inactivity.


No way to set max_connections > 100?

Hi,
I'm trying to create a timescale docker image that will override a few config defaults, run timescaledb-tune, and start up all in one action without needing to run timescale-tune, stop the server, make my additional config changes, and restart.
I took a page out of your docker file and decided to override the settings I care about in postgresql.conf.sample (remember, I want this tuning to happen and be applied on first startup of the container).
Here's the entirety of my docker file:
FROM timescale/timescaledb:1.3.2-pg11
RUN sed -r -i "s/[#]*\s*max_connections\s*=.*/max_connections = 1024/" /usr/local/share/postgresql/postgresql.conf.sample
RUN sed -r -i "s/[#]*\s*timezone\s*=.*/timezone = 'UTC'/" /usr/local/share/postgresql/postgresql.conf.sample
RUN sed -r -i "s/[#]*\s*synchronous_commit\s*=.*/synchronous_commit = off/" /usr/local/share/postgresql/postgresql.conf.sample

Two of my settings make it through (timezone and synchronous_commit), but max_connections is always reset to 100.
(I've verified my changes are indeed being applied properly in /usr/local/share/postgresql/postgresql.conf.sample, so it's timescaledb-tune that's nuking it)
Is there any way to selectively tune the configuration or provide manual overrides? I'm a bit stuck as to how I can deploy this to different hardware with a combination of automatic tuning and my custom settings.
Any guidance is much appreciated!

@RobAtticus mind taking a look?

Hey @rickbatka, you probably want to actually include a docker entrypoint script to do the change instead, since timescaledb-tune runs on container startup. Anything in the Dockerfile will get overwritten in that case.
We should probably expose a catch all TIMESCALEDB_TUNE_FLAGS var that can be set since it is possible to tell timescaledb-tune which number it should use for max_connections. Or perhaps the logic should change that if max_connections is >= to our recommendation, to just use that.
That said, 1024 seems a bit high and could negatively affect your query performance by reducing the amount of work mem for each.

you probably want to actually include a docker entrypoint script to do the change instead, since timescaledb-tune runs on container startup

I see what you mean. Unfortunately the timescale docker behavior is to run timescaledb-tune and then immediately start the database. Since that startup script overwrites that value, I would have to  write my own docker entrypoint, that did everything the current one does except also make this one change after timescale-tune, before starting the database. And now I've effectively forked away from the timescale docker image.
To your other points, I think it would be helpful if there were a way to affect the behavior of timescale-tune such that we could not tune certain settings that we want to set manually.

cc @bboule perhaps something for you to keep an eye on

I did almost the same thing as you... : sign0/postgis-timescale-h3/initdb-extensions.sh#L92
And, I modify my "work_mem" to 104kB as recommended here for 1000 connections : pgtune

I second that, it would be amazing to tell timescale via environment variables what to do (or not to do).

@rickbatka  you can now use TS_TUNE_MAX_CONNS


Are there metrics for testing on ogb ?

Unfortunately, no. But we have metrics for heterophilic datasets, long range graph benchmark, homophilic datasets and the TUDatasets.


Naming convention

How do you want to access the response?
First of all good news the code gets a lot easier. You could see a lot of places where it wasn't understood correctly.
For example the parsed method is doing nothing for reports, it's just passing on a string, or bytes. From now on only a better encoded string.
A. For the request.response object resolve duplicates choose one of:

self.original
self.response

B. Default:

use self.parsed no matter what, aka """Recieve a nice formatted response, this can be your default."""
dependent on text and xml. Since text we dont parse we just access it with request.original.text

C. For XML Data we provide more options:

a python dictionary, we can name it pydict , should it also get a parsed method?

additional:
headers object

#207 is cleaning up a lot of this by introducing MWSResponse to replace DictWrapper and DataWrapper, based partly on #75 changes that have finally been retooled and implemented. Please see docs related to that PR.