new "helm wait" command

Hi,
Couldn't find a way to do this with current options.
For multiple reasons, I'd like to be able to split the helm install/upgrade --wait into helm install/upgrade and helm wait (or something similar).
I want the ability to run the checks that helm would do on that release. Is there any existing option to achieve something similar? If not, are there any design/vision issues with such an option?
If it seems useful, I can look into implementing it, just wanted to get some feedback before diving in.
Thanks!
Output of helm version:
version.BuildInfo{Version:"v3.6.1", GitCommit:"61d8e8c4a6f95540c15c6a65f36a6dd0a45e7a2f", GitTreeState:"dirty", GoVersion:"go1.16.5"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.4", GitCommit:"e87da0bd6e03ec3fea7933c4b5263d151aafd07c", GitTreeState:"clean", BuildDate:"2021-02-21T20:21:49Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"21+", GitVersion:"v1.21.2-eks-0389ca3", GitCommit:"8a4e27b9d88142bbdd21b997b532eb6d493df6d2", GitTreeState:"clean", BuildDate:"2021-07-31T01:34:46Z", GoVersion:"go1.16.5", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
EKS

This would be a good candidate for a plugin. See https://helm.sh/docs/topics/plugins/ for more details.


Refactor Change#visit method in injector module

This PR updates the method below:
Modification visit(NodeWithAnnotations<?> node, Range range)

To
<T extends NodeWithAnnotations<?> & NodeWithRange<?>> Modification visit(T node)

Since the value for range is retrievable from the class implementing NodeWithAnnotations , all calls to this method were as below:
visit(node, node.getRange)

With this update, all calls are changed to simply:
visit(node)

Fancy :-)

@msridhar Thank you very much for the review, landed it. 🙂.


Managing abstract dependencies of setup.py

You say pipenv is a marriage of Pipfile, Pip, & Virtualenv. But there's one depency related task that's left to be done manually, that's managing setup.py's install_requires.
As @dstufft wisely says in his article, abstract depencies and concrete depencies are used in different contexts. But when you do your code, you're managing them at the same time. If you change a depency, you'll have to modify both setup.py or the Pipfile. I believe, it'd be nice to make it possible to manage the abstract dependencies using pipenv.
From a pipenv user perspective, I mean something that would look like:
# in Pipfile → [dev-packages]
pipenv install --dev pytest mock delegator.py toml Sphinx
# in Pipfile → [packages]
pipenv install pew>=0.1.26
# in setup.py → install_requires
pipenv install --abstract virtualenv pip pew>=0.1.26

Maybe could we extend the syntax so it has three options:
# in Pipfile → [dev-packages]
pipenv install --dev pytest mock delegator.py toml Sphinx
# in Pipfile → [packages] (only)
pipenv install --concrete foobar
# in Pipfile → [packages] and setup.py → install_requires
pipenv install pew>=0.1.26
# in setup.py → install_requires (only)
pipenv install --abstract virtualenv pip

That way, pipenv is the entry point tool to manipulate everything related to dependency management and associated virtualenv. And from a trouple marriage, it'll be a marriage of four tools 😁.
Now, on the question on how to implement this, I'm not much in favour of automatic edition of a .py file, because there are just too many ways it can go wrong. But instead, store the abstract depencies externally in a new [abstract-packages] section of Pipfile if it's ok with @pypa, or in an abstract-requirements.txt file or any other smarter solution I haven't think of yet.
What do you think?

Hey @guyzmo, thanks for starting this discussion. This is definitely a decision that needs to be made by the Pipfile project itself. I'd suggest proposing this there and seeing what the response is from the pypa community. From a historical perspective, there's been pushback from Pipfile adopting any more *-packages sections at this time.
Feel free to reference this issue if you'd like.

well, the issue I'm posting here has 3 questions:

do you think pipenv would be the right tool to manage abstract dependencies?
what command line API would it be nice to expose?
how to implement the mutability of those abstract dependencies?

in Pipfile, by negociating with pypa?
in some file we parse with something looking like the following in setup.py?
with open('abstract-req') as r: install_requires=r.read().splitlines()
by modifying the setup.py file, like it's a config file (it wouldn't be hard to parse it using ast, find a given variable and mutate the RHS of that variable assignation).

So, to me the first two questions are the most important ones we should discuss here, and on which a decision shall be made. Only then, we should discuss the third question, which is, IMHO, a technical detail. If people of the Pipfile project decide it's nice to also manage abstract dependencies in Pipfile, it's great, if they don't, then we can figure out another way.
I did not say, but I'd be happy to write a patch to implement whatever gets decided in this discussion, and ask the Pipfile project for permission, if contributors of this project share my opinion that it'd be a great adition to pipenv.

I think Pipfile should leave setup.py alone completely, personally.

@kennethreitz and what do you think about pipenv managing the install_requires?

After doing some research, it seems the right place to manage that would be indeed neither in the Pipfile or the setup.py, but rather in the setup.cfg, which supports it since setuptools 30.3.0.

Thanks for investigating this @guyzmo! From my understanding of what you're asking, I would say setup.cfg is probably what you're looking for here. Let us know if you have any further thoughts here.

so if I understand this correctly, you're not interested to extend pipenv so it manages abstract depencies (using setup.cfg as backend) with the following syntax:
# in Pipfile → [dev-packages]
pipenv install --dev pytest mock delegator.py toml Sphinx
# in Pipfile → [packages] (only)
pipenv install --concrete foobar
# in Pipfile → [packages] and setup.cfg → install_requires
pipenv install pew>=0.1.26
# in setup.cfg → install_requires (only)
pipenv install --abstract virtualenv pip

and if I want to see this happen, you're not definitely not interested in such a patch, so I should either write yet another tool or hard fork pipenv (which is basically the same)?

@guyzmo i found myself making another tool to manage dependencies for setup.py. and actually i am just doing away with setup.py altogether and replacing it with a static file (json for now but will use toml later). its a pet project for now.
curious to see if it addresses your use-case at all or what you have learned about this stuff so far

@pcattori sounds promising, though are you adressing the issue of abstract vs concrete deps management? Also, have you considered using the standard files (setup.cfg, Pipfile…) instead of adding yet another one?

@guyzmo yes i am planning on handling abstract vs concrete dependencies. specifically, the add command will update the entry in the project's json config (which is doing the job that setup.py was doing) and though I haven't added all the functionality yet, it will allow users to specify abstract vs concrete dependencies as necessary.
I have briefly considered using existing standard files like Pipfile and the new pyproject.toml but not really sure what to use. It seems like Pipfile is the replacement for requirements.txt, not setup.py. I think pyproject.toml might be what I want, but for short-term prototyping I haven't devoted the research to answer this question and instead just using something I'm familiar with (json) and something I have complete control over (my own format for that json file). Definitely want to use the standard solution for this if one exists.


Missing expo config in Manifest Response Body

Starting problem
Assets like images fail to load or the app crashes after an update using the example server (./expo-updates-server) when using method Linking.createURL(path, namedParameters) of expo-linking
adb logcat shows the following error when assets fail to load:
ReactNativeJS: Error: Cannot make a deep link into a standalone app with no custom scheme defined
And when it crashes:
ReactNativeJS: Error: Cannot make a deep link into a standalone app with no custom scheme defined
--------- beginning of crash
AndroidRuntime: FATAL EXCEPTION: expo-updates-error-recovery
AndroidRuntime: Process: <redacted>, PID: 4427
AndroidRuntime: com.facebook.react.common.JavascriptException: Error: Cannot make a deep link into a standalone app with no custom scheme defined, stack:

Cause
expo-linking requires the field (app.json|app.config.json).expo.scheme to be set, but following the code from that error in expo-linking backtracking to line 120, calls the method collectManifestSchemes that makes multiple gets to Constants.expoConfig (lines 68, 69, 73, 76), someway this method fails to return the manifests and returns an empty array, causing the throw error
Making another standalone build, commenting the call to Linking.createURL and with console.log("Constants.expoConfig::", Constants.expoConfig); in App.tsx
If we launch the app, without a new OTA update, console.log returns the content from app.json|app.config.json (expo config going forward) that is bundled with the app
If we launch the app, WITH a new OTA update, console.log returns an object with the exact response from the manifest endpoint aka the Manifest Response Body, but Constants.expoConfig it is expected to have expo config as stated in constants documentation
Real problem
Documentation or a fix is missing for how we should add the expo config to the metadata response, as the current implementation anything that expects that Constants.expoConfig to be expo config, will receive a Manifest Response body instead, in my case breaking expo-linking and showing the version of the app as undefined when getting Constants.expoConfig?.version
EAS Updates implementation
As stated from this pull request to eas-cli, referencing the pull request /expo/universe/pull/7893 the EAS Updates server (private repo I guess? Returns 404) will support storing and serving the extra.expoClient field in the eas update manifest
So when using eas publish, it's adding to the metadata that will upload to EAS Updates the expo config in the extra.expoClient field
But this special extra key (expoClient) is not documented in the Manifest Response Body as it only says: extra: { [key: string]: any }; about the extra field
Hacks / Workarounds
These are two ways I got Constants.expoConfig to have the expected fields from expo config after an OTA Update
Append expo config to metadata response
Appending to ./expo-updates-server/pages/api/manifest.ts:47 manifest, the expo config, Constants.expoConfig will have a union of expo config & Manifest Response Body
This was the first hack I got working to make the app get an update and not break expo-linking & Constants.expoConfig?.version
EAS Updates current implementation?
I did all my investigations without checking a real world response from EAS Updates, but making a puzzle with all the source code available from expo & eas I think I understand how the EAS Update service works to get the expo config in an OTA update and have it work
Checking the getter for Constants.expoConfig
It calls getManifest that if all the checks are OK, just returns rawManifest
rawManifest in a nutshell will be: if we got an OTA update, it will be Manifest Response Body, if not, the bundled expo config (from ExponentConstants.manifest)
This is where all my problems come from, this rawManifest that can be Manifest Response Body or expo config, but by default in an OTA Update will be Manifest Response Body if we don't make the specific "dance" to get the expo config from the Manifest Response Body

One would think that by default, if no expo config is given in the OTA Update (something that is not documented, or I couldn't find after almost a week of having fun with this), it will fall back to the bundled one, but instead just sets Constants.expoConfig to Manifest Response Body, what a rabbit hole I had to go to find out why our app wasn't working because expo config wasn't there after an OTA Update

The dance
Checking the code from the EAS Update implementation, is when I found out that extra.expoClient IS expo config, so I could finish the puzzle from the expoConfig getter
So, to make the metadata endpoint return the expo config, and get Constants.expoConfig to return it, I had to:
Change the ./expo-updates-server/pages/api/manifest.ts:47 manifest object type with the following modifications:
import { ExpoConfig } from https://github.com/expo/expo/blob/main/packages/%40expo/config-types/src/ExpoConfig.ts
type Manifest = {
  id: string;
  createdAt: string;
  runtimeVersion: string;
  launchAsset: Asset;
  assets: Asset[];
  metadata: {
    [key: string]: string;
    _randomKey: "random Value";
  };
  extra: {
    [key: string]: any;
    expoClient: ExpoConfig;
  };
}

Constants.expoConfig getter will check if rawManifest has a metadata key (metadata._randomKey in the modified type to ensure we have metadata). If we have metadata, it will return rawManifest.extra?.expoClient ?? null
Because we set expoClient in the type example, we will finally get Constants.expoConfig with the expected value of expo config from the response of Manifest Response Body. Hurray 🎉

The Manifest Response body spec says "The metadata associated with an update. It is a string-valued dictionary. The server MAY send back anything it wishes to be used for filtering the updates. The metadata MUST pass the filter defined in the accompanying expo-manifest-filters header." but I didn't understand how it's supposed to be used, that's why I added the _randomKey

What I wish
This repository guide about how to make an update, in a nutshell the steps are:

Make modifications to app
expo export --experimental-bundle
Copy dist to expo-update-server/updates/<runtime version>

IMHO I think that the expo config should be as easy to update, as the exported bundle
expo export --experimental-bundle could also export the expo config in a file called app.config.json next to metadata.json in the dist directory
And then the example server in expo-updates-server implement the needed logic to add it to extra.expoClient & add an empty metadata key (or whatever should have)

@SrZorro - excellent write up!
So there's a few issues in here:

This repo (custom-expo-updates-server) incorrectly implements the expo-updates specification. Namely, the metadata field is required in the response (as you noticed). The fix is to add metadata: {} to the update response, since it is required for determining manifest type in expo-constants (JS) and expo-manifests (native code). I will put up a PR for this. I'm honestly shocked expo-manifests working without it so an investigation into why it works without it will be the first step.
We need to add information about the extra field in the expo-updates protocol. expo-updates itself shouldn't specify what fields are in the extra field, but it should specify where and how extra fields work. I'll put up a separate PR for this.
We need to add documentation to libraries that make use of expoConfig indicating which config fields they depend on. This is a much more difficult problem to address since there's so many libraries, but starting with expo-linking seems good. I'll put up a PR for this as well.

This example repo is designed to be a barebones implementation of the expo-updates protocol specification: https://docs.expo.dev/technical-specs/expo-updates-0/. But that doesn't mean that it shouldn't be easy to fork it and add additional expo libraries to it, so I think that is the end goal.

@SrZorro - I implemented the first two items I proposed. The last one is a bit tough to do thoroughly since so many other libraries require config.

What I wish
This repository guide about how to make an update, in a nutshell the steps are:

Make modifications to app
expo export --experimental-bundle
Copy dist to expo-update-server/updates/<runtime version>

IMHO I think that the expo config should be as easy to update, as the exported bundle
expo export --experimental-bundle could also export the expo config in a file called app.config.json next to metadata.json in the dist directory
And then the example server in expo-updates-server implement the needed logic to add it to extra.expoClient & add an empty metadata key (or whatever should have)

This is an interesting idea. I'm not sure I'm the right person to assess whether this should be part of the expo export command though versus just having it be an extra command. Maybe @byCedric or @EvanBacon can comment. Something like adding a flag: expo export --expo-config.
The same thing can be done by just using getConfig from @expo/config. If we think about how EAS does it, it just reads the config and uploads it to the server. So the equivalent for this repo is to just have another step in the yarn expo-publish script that exports the result of getConfig into the dist directory as well, and then have the server read it and serve it.


Maybe add conda as installation option instead of venv

Why not both?
What are the commands you use to activate conda?

both is good
conda create -n <env name>
conda activate <env name>


Test: Problems panel

Related to #59976

[ ] macOS: @sandy081
[ ] linux @sbatten
[ ] windows @alexr00

Complexity: 4
The Problems panel was heavily refactored to adopt the new ObjectTree. Please completely test the problems panel

Make sure problems are shown correctly without missing.
Make sure the badge shows correct number of problems
If auto reveal is enabled problems.autoReveal, Switching between files should reveal the problems for the current active file.
Test filtering by typing and also test files exclude filter (files.exclude)
Test Keyboard Navigation: Navigating problems using keyboard (up and down arrows) should select the problem and also reveals it in the file by opening it.
Test Mouse handling: Expanding and Collapsing, Selection, Double click, Opening file to the right (cmd+click).
Test scrolling with mouse and also with keyboard.
Test actions: Light bulb, Collapse All, context menu actions.
Test accessibility using Voice over / NVDA in Mac / Windows.

You can use following sample to generate problems when active editor is changed
const diagnosticsCollection = vscode.languages.createDiagnosticCollection('test');
vscode.window.onDidChangeActiveTextEditor(e => {
    if (e && e.document) {
        const diagnostics: vscode.Diagnostic[] = [];
        const diagnostic = new vscode.Diagnostic(new vscode.Range(new vscode.Position(10, 1), new vscode.Position(10, 10)), `This is an error`, vscode.DiagnosticSeverity.Error);
        diagnostic.source = 'test';
        diagnostic.code = 1234;
        diagnostics.push(diagnostic);
        diagnosticsCollection.set(e.document.uri, diagnostics);
    }
});

@joaomoreno, with problems.autoReveal should the problems pane show? I'm seeing the badge pop up on PROBLEMS when I switch files, but the problems pane isn't getting switched to.

@alexr00 With auto reveal works only when problems panel is open... just like auto reveal in explorer.


Connecting to specific catalog not working

In the README there is this example
db, err := sql.Open("databricks", "token:@.databricks.com:443/sql/1.0/endpoints/********")
And it also says the DNS format is:
token:[your token]@[Workspace hostname]:[Port number][Endpoint HTTP Path]?param=value
So I figured that if I want to add catalog like described here https://docs.databricks.com/integrations/jdbc-odbc-bi.html#default-catalog-and-schema
The URL should be
token:[your token]@[Workspace hostname]:[Port number][Endpoint HTTP Path]?ConnCatalog=main
However this doesn't seems to work:
Configuration ConnCatalog is not available.",....,"message":"databricks: failed to execute query: query SET `ConnCatalog` = `main`;"}

After I change ConnCatalog to ConnCatalog it seems to work

Ohh yea sorry, I just edited it

I will update the readme in the next release.  Are you ok with closing this issue?

Closed :)


Improve shape validation when jax_dynamic_shapes=True

Fixes #18937

Probably needs a test before we can merge.


dart run orm generate throws ProcessException: The system cannot find the file specified.

After upgrade to orm: 3.0.0 running dart run orm generate gives me following error:
> dart run orm generate

Unhandled exception:
ProcessException: The system cannot find the file specified.

Command: npm exec -- prisma version --json

which is strange because if I run npm exec -- prisma version --json it return I think correct output:
> npm exec -- prisma version --json
Environment variables loaded from .env
{
  "prisma": "4.11.0",
  "@prisma/client": "4.11.0",
  "current-platform": "windows",
  "query-engine-(node-api)": "libquery-engine 8fde8fef4033376662cad983758335009d522acb (at node_modules\\@prisma\\engines\\query_engine-windows.dll.node)",
  "migration-engine": "migration-engine-cli 8fde8fef4033376662cad983758335009d522acb (at node_modules\\@prisma\\engines\\migration-engine-windows.exe)",
  "format-wasm": "@prisma/prisma-fmt-wasm 4.11.0-57.8fde8fef4033376662cad983758335009d522acb",
  "default-engines-hash": "8fde8fef4033376662cad983758335009d522acb",
  "studio": "0.483.0"
}

also npx prisma generate does not generate prisma_client.dart. Just prints:
> npx prisma generate
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma

I have the same issue (also on windows), but neither flutter clean nor dart pub cache clean changed anything for me :(

I reopen it, this may be a long term concern.

PS: I don't have a Windows device, it might take more scope to check exactly why it's happening. But I confirm it's not a code problem.

Looks like we need to run 'npm.cmd' instead of 'npm' to retrieve the Prisma version and platform on Windows.
I tried to modify the process in bin/generator/prisma_info.dart:15 as a quick workaround and the PrismaInfo object was successfully initialized.
final result = Process.runSync(
      packageManager == 'npm' && Platform.isWindows ? 'npm.cmd' : packageManager,
      [
        'exec',
        if (packageManager == 'npm') '--',
        'prisma',
        'version',
        '--json',
      ],
      stdoutEncoding: convert.utf8,
    );

But even after solving that, the prisma_client.dart file is not generated as the process hangs on bin/orm.dart:45.
No events are received from the stdin stream in bin/generator/generator.dart:37, so the Completer<Generator>() never manages to complete.
@medz any hints or suggestions on this?

Looks like we need to run 'npm.cmd' instead of 'npm' to retrieve the Prisma version and platform on Windows. I tried to modify the process in bin/generator/prisma_info.dart:15 as a quick workaround and the PrismaInfo object was successfully initialized.
final result = Process.runSync(
      packageManager == 'npm' && Platform.isWindows ? 'npm.cmd' : packageManager,
      [
        'exec',
        if (packageManager == 'npm') '--',
        'prisma',
        'version',
        '--json',
      ],
      stdoutEncoding: convert.utf8,
    );

But even after solving that, the prisma_client.dart file is not generated as the process hangs on bin/orm.dart:45. No events are received from the stdin stream in bin/generator/generator.dart:37, so the Completer<Generator>() never manages to complete.
@medz any hints or suggestions on this?
UPDATE It seems that I was mistakenly running dart run orm generate instead of npx prisma generate. When I ran the latter the prisma_client.dart was successfully generated!

@nikosportolos Does yarn and pnpm have the same problem?

@medz Yes, I just installed and tried yarn and pnpm. They both needed the *.cmd file.
So I guess that line could change to:
Platform.isWindows ? '$packageManager.cmd' : packageManager,

@nikosportolos @jackstefansky Version 3.0.2 fixes it, is it valid now?

I tried running dart run orm on 3.0.2 ref and it fails with:
Unhandled exception:
ProcessException: %1 is not a valid Win32 application.

Command: "C:\Program Files\nodejs\npm" exec -- prisma version --json
#0      _ProcessImpl._runAndWait (dart:io-patch/process_patch.dart:487:7)
#1      _runNonInteractiveProcessSync (dart:io-patch/process_patch.dart:632:18)
#2      Process.runSync (dart:io-patch/process_patch.dart:68:12)
#3      new PrismaInfo.lookup (file:///C:/Users/stefp/AppData/Local/Pub/Cache/git/prisma-dart-28fe075d1f3a5c48115b70f5da4f85366d919015/bin/generator/prisma_info.dart:19:28)
#4      main (file:///C:/Users/stefp/AppData/Local/Pub/Cache/git/prisma-dart-28fe075d1f3a5c48115b70f5da4f85366d919015/bin/orm.dart:47:27)
#5      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:32)
#6      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192:12)

Ok, I see, it seems the fix is not working.
I'll find a Windows device and within a day, I'll fix it!
You can manually specify the file address of npm with the -e parameter, hopefully it works 👉 https://prisma.pub/docs/configuration.html#node-package-manager-executable

(@odroe @medz) perhaps this is out of scope for this project, but may I suggest GitHub Actions matrix testing?
The problems with this project seem to largely be around platform compatibility, and I feel that a quality CI server can help reduce the time that it takes to find, analyze, and correct these issues.
If you would like or require help in setting up CI matrix testing, you can feel free to assign me to an issue/PR :)

@gutenfries Thank you so much bro, I don't know anything about Windows. So I don't know where to go from here to handle it, plus I'm currently having trouble finding a Windows device to test on.
It would be great if you could handle that. Indeed this is a compatibility issue on Windows.

@medz Version 3.0.2 solved the package manager problem, but the prisma_client.dart is still not being generated.
I'll try to debug this a bit later in the day.

@medz Version 3.0.2 solved the package manager problem, but the prisma_client.dart is still not being generated. I'll try to debug this a bit later in the day.

Thx 💗

On Windows we need to escape whitespaces in paths when running commands.
So if you tried to run
dart run orm -e C:\Program Files\nodejs\npm.cmd

it would fail, because of the whitespace in 'Program Files' with the error message: Unable to find C:Program executable.
In order to solve this, it needs to change to:
dart run orm -e "C:\Program Files\nodejs\npm.cmd"

So in the schema.prisma file we need to do the same, but the problem is that we can't have nested double quotes.
generator client {
  provider = "dart run orm -e "C:\Program Files\nodejs\npm.cmd""
}

An alternative would be to use %PROGRAMFILES%\nodejs\npm.cmd instead, but this fails to be recognised as a file on bin/generator/prisma_info.dart:57.

UPDATE
@medz it seems that you've already fixed that, but I was using it the wrong way.  😅
We don't have to use the full path, but just the executable name instead:
generator client {
  provider = "dart run orm -e npm.cmd"
}

If anyone else can verify that this works, I think this issue can be marked as done.

UPDATE
@medz it seems that you've already fixed that, but I was using it the wrong way. 😅 We don't have to use the full path, but just the executable name instead:
generator client {
  provider = "dart run orm -e npm.cmd"
}

If anyone else can verify that this works, I think this issue can be marked as done.

I can confirm - this works :)

I don't think this is a good fix, until it works with npm without passing any arguments it should be successful.


incompatible with typescript node16 resolution with type: module

This is because .d.ts is considered as commonjs declaration by default

Workaround: export react as named export at the same time, so that the user can use import { react } from '@vitejs/plugin-react-swc'

The vite config is not running in node, so you should use the bundler resolution added in TS 5

The vite config is not running in node, so you should use the bundler resolution added in TS 5

It can be. Vite can be used as a server middleware.

yeah but the config will be bundled by Vite

yeah but the config will be bundled by Vite

It can be loaded by user manually and overrides the vite config accordingly.

This issue is about compatibility, I don't understand why you are requesting to change the tsconfig which is used by other tools at the same time.


Replace homegrown Natural and Positive with refinement types provided by refined

@wemrysi and @drostron Finally got around to rebasing this work on master and addressing some of @wemrysi's final review comments!! :smile:

Current coverage is 79.40%

Merging #1082 into master will decrease coverage by -0.12% as of daf36c5

@@            master   #1082   diff @@
======================================
  Files          167     164     -3
  Stmts         9684    9576   -108
  Branches       316     313     -3
  Methods          0       0
======================================
- Hit           7701    7604    -97
  Partial          0       0
+ Missed        1983    1972    -11

Review entire Coverage Diff as of daf36c5

Powered by Codecov. Updated on successful CI builds.

@wemrysi btw, fixed the issues with the CI build.

@jedesah I haven't forgotten about this, but mind if it waits until next week after 2.5?

@wemrysi No problem! I will have to resolve conflicts anyway now that we finally merged the other PR. Also, if scala 2.11.8 gets release in the meantime (probably not) that would allow me to get rid of many workarounds and comments.

@wemrysi I think this is finally ready to be merged!

:+1:


uki: drop INITRAMFS_IMAGE support

Enabling INITRAMFS_IMAGE and UKI results in the dependency loop. Initramfs depends on the kernel to be packaged, while the kernel's do_install gets dependency on the initramfs do_image_complete task.
Drop the whole INITRAMFS_IMAGE support from the uki.bbclass. One should use systemd-boot's initrd support natively.

cc @sbanerjee-quic

cc @sbanerjee-quic

@lumag we will review and feedback. We have a long weekend.
cc @quic-vkraleti

@quic-vkraleti I'm not sure, what is Type1 / Type2 here. A reference would be a appreciated. Anyway, currently built-in initrd support is broken, try building such config on your own.

@quic-vkraleti I'm not sure, what is Type1 / Type2 here. A reference would be a appreciated. Anyway, currently built-in initrd support is broken, try building such config on your own.

https://uapi-group.org/specifications/specs/boot_loader_specification/#the-boot-loader-specification describes details about Type#1 & Type#2 specifications. Goal is to support Type#2 and this mandates initrd to be part of UKI.
Regd broken built-in initrd support,  can you share more details on how to reproduce? As I see even kernel.bblcass is fetching initrd from INITRAMFS_DEPLOY_DIR_IMAGE, which is a soft assignment to DEPLOY_DIR_IMAGE. uki.bbclass is also using same method to get initrd.

For me it was simple, I enabled INITRAMFS_IMAGE = "initramfs-test-image" and got a errors regarding the task dependency loop.

From build POV, ramdisk image is also another image which by default it pulls in a dependency on do_packagedata task of kernel. Here uki.bbclass is trying to create kernel-uki package, that need ramdisk image to complete. This is the cause for circular dependency. To address this, need to stop creating kernel-uki package and place efi image in deploydir instead.

Closed in favour of #562  which provides a better, more complex solution


Rubocop housekeeping

Followup to https://github.com/capistrano/capistrano/pull/1594.

Updates TravisCI to use full Rubocop suite
Updates docs to reflect change

:ship:


Changelog for 0.6.2

This is missing from https://github.com/seanmonstar/reqwest/releases.

Thanks for reminding me: https://github.com/seanmonstar/reqwest/releases/tag/v0.6.2


Importing mime-types results in 154 KiB additional space in Webpack bundle (bloat)

I was investigating a bump in my Chrome extension bundle size, which goes from
                       Asset       Size  Chunks                    Chunk Names
                 ../icon.png   1.17 KiB          [emitted]
            ../manifest.json  412 bytes          [emitted]
../offline_verification.html  271 bytes          [emitted]
             ../options.html  208 bytes          [emitted]
               ../popup.html  239 bytes          [emitted]
               background.js    612 KiB       1  [emitted]  [big]  background
     offline_verification.js   9.79 KiB       2  [emitted]         offline_verification
                    popup.js    9.1 KiB       3  [emitted]         popup
                   vendor.js    866 KiB       0  [emitted]  [big]  vendor

to
                       Asset       Size  Chunks                    Chunk Names
                 ../icon.png   1.17 KiB          [emitted]
            ../manifest.json  412 bytes          [emitted]
../offline_verification.html  271 bytes          [emitted]
             ../options.html  208 bytes          [emitted]
               ../popup.html  239 bytes          [emitted]
               background.js    612 KiB       1  [emitted]  [big]  background
     offline_verification.js    9.9 KiB       2  [emitted]         offline_verification
                    popup.js    9.1 KiB       3  [emitted]         popup
                   vendor.js   1010 KiB       0  [emitted]  [big]  vendor

I have isolated the cause to be the use of mime-types library. Any idea of the reason why and how to debug?

I forgot to link my usage: https://github.com/inblockio/VerifyPage/blob/e1c7e80b234227011d8f444dc7d495c997fc676e/src/offline_verification.tsx#L11.

It's the enormous database of mime types and their file extensions :) it is a 181 kb file before minification, which I assume your webpack is doing. The database is how this module provides the functionality of the mime type look ups.

Ah, I was already looking at mime-db, but was somehow missing the "giant" db.json. Thank you!


Remove the https reference from s3 paths

This came during one of the feedback from Arun Gupta
"Change samples to use s3:// by default instead of https. This came up when he was changing regions, and asked if he had to change them for all https URLs."
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@cadedaniel  @RedbackThomson

In our documentation do we have references to changing S3 URLs with https://?

In our documentation do we have references to changing S3 URLs with https://?

I don't think so.


[Question] Can't get user information

Initial Check

[] Python version is 3.8.3;
[] Updated Twint with pip3 install --user --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint;
[] I have searched the issues and there are no duplicates of this issue/question/request.

Command Ran
import twint
c = twint.Config()
c.Username = "realDonaldTrump"
c.Custom["user"] = ["id", "username"]
c.Output = "username.csv"
c.Store_csv = True

twint.run.Lookup(c)

Description of Issue
I try to scrape specific user information and follow the code in user attribute in wiki. But I got the following errors:

ERROR:root:twint.run:Lookup:Unexpected exception occured.
Traceback (most recent call last):
File "c:\users\user\src\twint\twint\run.py", line 393, in Lookup
get_event_loop().run_until_complete(get.User(url, config, db.Conn(config.Database)))
TypeError: User() missing 2 required positional arguments: 'bearer_token' and 'guest_token'
Traceback (most recent call last):
File "C:/Users/user/Desktop/project/SN0-0.py", line 9, in
twint.run.Lookup(c)
File "c:\users\user\src\twint\twint\run.py", line 393, in Lookup
get_event_loop().run_until_complete(get.User(url, config, db.Conn(config.Database)))
TypeError: User() missing 2 required positional arguments: 'bearer_token' and 'guest_token'

Environment Details

Occurred in Terminal and PyCharm 2020.1.2

Command in Terminal:

C:\Users\user\AppData\Local\Programs\Python\Python38\Scripts>twint -u "realDonaldTrump" --user-full

Facing this same issue.

same with --user-full

+1 same issue

I solved the problem by adding bearer_token and guest_token arguments to line 393 of the run.py file but the problem will occur in other lines. The bearer_token should be defined in the Config variable c, but it was set to None by default. Unfortunately, there is no documentation that explains what does this variable means. There is only a global bearer variable in the run.py which can be used.
The second problem is the guest_token variable which is not defined anywhere in the codes. By analyzing the project, I found we should use the token.py file to get a guest token. So the test example should be like
c = twint.Config()
twint.token.Token(c).refresh()
c.Bearer_token= twint.run.bearer
c.Username = 'myusername'
c.Custom["user"] = ["id", "username"]
c.Output = "username.csv"
c.Store_csv = True
twint.run.Lookup(c)

By running this code you will get an error on 19 of the user.py file. Because the function is recursively called without necessary argument. I think this is not a recursive function and the user class in the _usr = User(), is a container for the retrieved user info, as the User class is defined in the 5th line of the file. So, I changed the function name to UserM and also edited the name of this function in the output.py file (lines 5 and 186).  Also, in line 15 of the user.py file you should change and operator to or.
After doing these changes, I ran the example and get:
KeyError: 'malformed json! cannot be parsed to get user data'

because the ur variable which passed to the UserM function is:
{'data': {}, 'errors': [{'message': 'Not found', 'path': ['user'], 'locations': [{'line': 126, 'column': 3}], 'source': 'Server', 'code': 50, 'kind': 'NonFatal', 'extensions': {'source': 'Server', 'code': 50, 'kind': 'NonFatal'}}]}

I think this is because of Twitter changes and project contributors should update the project...

Same issue :(

Hi! I am facing the same issue (kubuntu 20.4) - it works just fine on my mac though

Does anyone know what the guest token refers to?

Yeah,
Guest Token is required to make requests to Twitter.
It hasn't been implemented in Lookup yet,
So try to use Search instead for now.
Lookuo is broken right now.
I might fix it later. But I am a bit occupied at the moment.

Ok, but Seach can not be used for the same purpose I think.
How can I check account location for example?

@seonake
yes, you are right. Search cannot be used in stead of Lookup.
I had some free time, so I put up a fix for this issue.
Try installing from my master branch and let me know if your issue gets resolved.
Note: Some things might still be broken, or perhaps some new things might have broken. Bear with me a little, I'll try to fix things as soon as I can. :)

Following up on this one, now there is a new error indeed: KeyError: 'url'

@micaelamaria
Could you share the script you ran & the entire error that you got?

@himanshudabas  yes!! I paste it below
mik@prisma-ds:~$ twint -u MaraRoller2 --user-full
CRITICAL:root:twint.get:User:'url'
ERROR:root:twint.run:Twint:Lookup:Unexpected exception occurred.
Traceback (most recent call last):
File "/home/mik/.local/lib/python3.8/site-packages/twint/run.py", line 307, in Lookup
await get.User(self.config.Username, self.config, db.Conn(self.config.Database))
File "/home/mik/.local/lib/python3.8/site-packages/twint/get.py", line 228, in User
await Users(j_r, config, conn)
File "/home/mik/.local/lib/python3.8/site-packages/twint/output.py", line 177, in Users
user = User(u)
File "/home/mik/.local/lib/python3.8/site-packages/twint/user.py", line 31, in User
_usr.url = ur['data']['user']['legacy']['url']
KeyError: 'url'
Traceback (most recent call last):
File "/home/mik/.local/bin/twint", line 8, in
sys.exit(run_as_command())
File "/home/mik/.local/lib/python3.8/site-packages/twint/cli.py", line 339, in run_as_command
main()
File "/home/mik/.local/lib/python3.8/site-packages/twint/cli.py", line 326, in main
run.Lookup(c)
File "/home/mik/.local/lib/python3.8/site-packages/twint/run.py", line 386, in Lookup
run(config)
File "/home/mik/.local/lib/python3.8/site-packages/twint/run.py", line 329, in run
get_event_loop().run_until_complete(Twint(config).main(callback))
File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
return future.result()
File "/home/mik/.local/lib/python3.8/site-packages/twint/run.py", line 235, in main
await task
File "/home/mik/.local/lib/python3.8/site-packages/twint/run.py", line 270, in run
await self.Lookup()
File "/home/mik/.local/lib/python3.8/site-packages/twint/run.py", line 307, in Lookup
await get.User(self.config.Username, self.config, db.Conn(self.config.Database))
File "/home/mik/.local/lib/python3.8/site-packages/twint/get.py", line 228, in User
await Users(j_r, config, conn)
File "/home/mik/.local/lib/python3.8/site-packages/twint/output.py", line 177, in Users
user = User(u)
File "/home/mik/.local/lib/python3.8/site-packages/twint/user.py", line 31, in User
_usr.url = ur['data']['user']['legacy']['url']
KeyError: 'url'

@himanshudabas have you been able to look at this one?> Just asking in case I should openanother thread!


Update ClrTraceEventParser with new assembly loader events

Update ClrTraceEventParser to properly show new assembly loader events: https://github.com/dotnet/runtime/blob/5da85d0c4b6fc390047cb466a9e0ed674d9c121f/src/coreclr/src/vm/ClrEtwAll.man#L1673-L1808

@elinor-fung thanks for contributing this.  Do you have a trace file that contains examples of these new events?  I'm interested to see what real data from these events looks like.

Trace with a few assembly loads (not actually a .zip - just a .nettrace with the .zip added): trace-assemblyLoader.nettrace.zip
That was from a program that explicitly loads System.Xml by name and then another assembly (AssemblyToLoad) that relies on an event handler to be found.
Short snippet from the events view could be something like:

Event Name
Stage
Result
AssemblyName
ResultAssemblyPath
Rest

Microsoft-Windows-DotNETRuntime/AssemblyLoader/Start

System.Xml, Culture=neutral, PublicKeyToken=null

Microsoft-Windows-DotNETRuntime/AssemblyLoader/KnownPathProbed

0

FilePath="D:\dotnet-test\shared\Microsoft.NETCore.App\3.0.0\System.Xml.dll"

Microsoft-Windows-DotNETRuntime/AssemblyLoader/ResolutionAttempted
FindInLoadContext
AssemblyNotFound
System.Xml, Culture=neutral, PublicKeyToken=null

Microsoft-Windows-DotNETRuntime/AssemblyLoader/ResolutionAttempted
ApplicationAssemblies
Success
System.Xml, Culture=neutral, PublicKeyToken=null
D:\dotnet-test\shared\Microsoft.NETCore.App\3.0.0\System.Xml.dll

Microsoft-Windows-DotNETRuntime/AssemblyLoader/Stop

System.Xml, Culture=neutral, PublicKeyToken=null
D:\dotnet-test\shared\Microsoft.NETCore.App\3.0.0\System.Xml.dll

Thanks much!  LGTM.

cc @lpereira @vitek-karas


Release/0.6.0

[ ] Update README usage example
[ ] Update remaining headers to 2019

Hey @BenFradet! This is an initial version of refactored Iglu Client. I think likely it will have even more changes, but you already can have a look.


Fix leak in KMAQueryEncoder

CFURLCreateStringByAddingPercentEscapes follows the Create Rule so it's returning a +1 retained object, however the __bridge sends it to ARC as a +0.

Oops, should have seen this before I fixed it again in #27

This can be closed now 'cause it's fixed in #27 right?


Compute and display user contributions across all organization repos

As mentioned by @cgreene in his review of https://github.com/cognoma/cognoma/pull/28:

Also, it may be helpful to note in text that the number of contributors is far larger than this. If it's possible to track who has participated outside the sandbox and automatically or periodically update a list of names here (github linked to usernames) of all contributors that would be great.
I'd love to see a full list of contributors but maybe this is something that is hard to put together.

In short, we want a way to automatically compile a list of all cognoma contributors. It would also be nice to measure the contribution per user. We could then embed this information in the README alongside "Maintainers" or in a separate document.
Hopefully we can find some utilities to help extract contributions from git/GitHub. We may also want to consider continuous analysis to automate this task.

GitHub API
The GitHub API may be a good way to start. Here are some useful queries:

All Cognoma repos
https://api.github.com/orgs/cognoma/repos

List of contributors for a cognoma repo
https://api.github.com/repos/cognoma/sandbox/stats/contributors

Closing as we used the GitHub API to extract contribution info in https://github.com/cognoma/cognoma/pull/33, which uses commit counts to measure contribution. Happy to reopen this issue if someone is interested in other metrics of assessing contribution.


Infra: When and how to prepare the 2.10.x line?

Situation
We've decided to freeze 2.10.x and no new features will be added to this version (except security and severe bugfixes). As we are going towards 3.0.0, master will contain the new version soon.
However, we still haven't thought about how to deal/manage both versions, old and new, so they can live peacefully together.
Proposed Solution
Probably we should open a maintenance branch for the 2.10.x line. The open questions are:

When should we open the maintenance branch? Now or in the not so far future?
How should we deal with upcoming 2.10.x versions?

Here is my idea that I would like to discuss with you:

Make sure we have a clean, good state (should be already the case; but it doesn't hurt to look a bit deeper).
Branch off from current master and create a maint/2.10.0 branch. We could also use stable/2.10.0. Any better name? :wink:
Change version in master to 3.0.0+alpha1 (just an example).
Optionally indicate it in the README as "work in progress".
Now we can start implementing code for 3.0.0 only.

After these steps, we have the new version in master and the old version in a maintenance branch. If and how we want to create a new release for the 2.10.x line, it's a matter of taste. One way to do it could be like this:

Add the changes to the 2.10.0 maintenance branch.
Raise version to 2.10.x.
Tag the release.
Publish the 2.10.x release.

It could be helpful to protect the maintenance branch like we did with master.
Anything else we need to cover? Comments? :wink:

Update: Changed 2.9.x -> 2.10.0

I would let Python 2.7 die before
https://pythonclock.org/
2 months is not so long...

I think we could create this branch right now and start adding things for Py3.5+ only but it's only my opinion :)

@ppkt
Yes, I would have no problem to create a maintenance branch release for the 2.9.x series and adding new feature in the master branch (which would be the upcoming 3.0.0 release). I would love to start for the next major release. :wink:
Of course, there are other ways to do it, like creating a dedicated 3.0.0-pre release branch, adding stuff there and then merge it to master when it's "ready".
Personally, I would prefer the first method (create a 2.9.x maintenance branch, master will be the upcoming 3.0.0 release). I think, it makes it a little bit easier.

Currently, we have one problem and that is Travis. For some reason, it doesn't seem to finish it's task and blocks our pull requests. :disappointed: I'm not sure how to fix it...

@tomschr agree, I think separate branches for 2.9.x and 3.0.x would be great (so we could consider master as "stable" branch).

@ppkt Whatever works is fine for me. :wink: Let's see what @scls19fr has to say about that.
What would you prefer? Would you wait til January (as indicated in some other comment)? Or can we start right away? :wink:

Submitting PR is always opened and discussion can occur on these PR.
We will see in January for merging them to master.
Maybe we should have a tag WIP (and/or Release 3.x.y) to just remember not to merge them.
I don't think we have enough human ressources for managing several branches (even if here several means 2 :smile: )
In January we will create a 2.9.x branch before merging 3.x features to master.
So our 3.x.y development will be in master and we will have a 2.9.x branch if we need to go back (hoping that we won't need to do it)

Maybe we should have a tag WIP (and/or Release 3.x.y) to just remember not to merge them.

That would be one way. Another way would be to open PRs as "Draft". If you do that, merging is disabled unless you change "Draft" to "Ready for review".

I never used previously PR as Draft but it seems to be a good idea as it will be a place to have discussion about changes we expect to be merged for 3.x.y.

As we are preparing a 2.10.0 in #237 soon, this issue is related to it.
@python-semver/reviewers
Once 2.10.0 is released, I would suggest to work on the semver3 after. IMHO, that gives us the chance to remove all the old stuff of Python2 and focus on one code base.
Any concerns or objections about this idea? :wink:

I guess, this is solved by #246


when scatterPlotLayer zoom over 17, the point element will be deformed or even disappeared

https://codepen.io/anon/pen/jKpWNR?editors=1010

thank you,i will try it


Top class

Add top class to top most component when mouse hover
This:

Instead this:

Code:
.top:hover { position: sticky }

This seems simple enough for you to add on your own? I'm not sure a global class is needed

For beginner people, maybe harder to do this... I'm beginner too, i had to keep looking and testing this attribute on css file using VSCode intelliSense.
How much more easy, is better


[PERF] longValue(exactRequired = true) is very slow

Describe the bug
No problem with the returned value, but the cost of using exactRequired = true is order of magnitude more costly than using exactRequired = false.
To Reproduce
I was working on some load tests on our mathematical library, so a lot of calls were involved (BD creation, addition, multiplication, ...). In my case the BigDecimal was always rounded just before the call to longValue(exactRequired = true).
With the exactRequired = true this test was taking 7.5 minutes, when I just changed the boolean to false, it took 1.4 seconds.
Sorry that's maybe not clear to reproduce, but I believe any attempts should show a similarly high impact.
In my case, I'd prefer to use exactRequired = true as I'm actually expecting an exact value, but the performance wasn't explicit.
Platform

Reproduced on JVM (maybe other platforms impacted)

Hi @glureau thanks for reporting, I'll give it a look when I have time, 7.5 minutes vs 1.4 sounds drastic :)
There's probably room for improvement wherever you look in the library, and this might be one of the places that could be improved.


proxy.process.net.calls_to_* stats needs cleanup

ATS has 8 proxy.process.net.calls_to_* stats. But some of them are not working or meaningless.
List of proxy.process.net.calls_to_* stats
https://github.com/apache/trafficserver/blob/472075bd7b5755cc1d8bf9a4bbf750474c4da53c/iocore/net/Net.cc#L84-L91
1). net_calls_to_readfromnet_stat and net_calls_to_readfromnet_afterpoll_stat are always 0.
Both of them are never incremented.
2). net_calls_to_writetonet_stat and net_calls_to_writetonet_afterpoll_stat are exactly same
Both of them are incremented only in write_to_net.
https://github.com/apache/trafficserver/blob/master/iocore/net/UnixNetVConnection.cc#L347-L348

Duplicated with https://github.com/apache/trafficserver/issues/7214


Fix spacing issues in Hacktoberfest Page

1. Top & Bottom spacing of Guidelines needed to fix appropriately.

2. The bottom padding needs to be fixed.

3. The space between Read More button & SVG needs to be increased.

I want to work on this.

Yup👍
Assigning it to you.

Hey, Can you assign me if it is not yet fixed? I would like to work on it.

@Anshumansengar don't worry.
@anirudh-bukka is assigned now. If the issue remains after the pr of Anirudh. We will assign it to you.
Till then look into other repos and contribute 👍

Hey @Debajyoti14 , can you pls link the page that needs to be fixed? I'm unable to find the exact page.

https://www.clueless.tech/hacktoberfest
This is the link

Hey @Debajyoti14 can you assign it to anyone else?
Ambushed with assignments and mid term.

hey, can I take the issue??

I would like to work on this issue. Let me know if you want to assign me to this.

Ok,
@Garvit1809 assigning it to you.
@AbirSantra will assign it to you after @Garvit1809

Guidelines issue:-

Margin Bottom issue:-

Space between Read more and SVG

@Debajyoti14 can u see if this is good or not

increase bottom padding a little at 2

Now looks good ?? Added a bit of margin in image as well as the heading


How to make RealDllPath work? Using dxwrapper with Colin McRae Rally 1

I'm trying to use DxWrapper with Colin McRae Rally 1, and also DgVoodoo.
DxWrapper because it's the only way to have the game actually run smoothly. And DgVoodoo to run it in high resolution and without changing my desktop res to 800x600 (grrrr!).
The only 2 lines I edited in dxwrapper.ini are:
Dd7to9                     = 1
RealDllPath                = C:\Games\Misc\CMRally\D3D9.dll
The latter is a DgVoodoo dll. But I only got DxWrapper, not DgVoodoo.
What am I doing wrong here?

Oops, my ini needed to match the dll file name. It now adds DgVoodoo to the mix, but then crashes.

Well, the FullscreenWindowMode and DxWnd options outright crash. Is it literally impossible to run this damn game without changing my desktop res?

Dd7to9 = 1

This game uses Direct3D and the Dd7to9 option does not yet work with Direct3D games.

It now adds DgVoodoo to the mix, but then crashes.

If you are using the Dd7to9 option that could be why it is crashing.

Well, the FullscreenWindowMode and DxWnd options outright crash.

For this game, the FullscreenWindowMode option requires the Dd7to9 option which does not work with this game.
I added a quick hack for this game that will stretch the image to fill the screen without changing resolutions.  Try using this wrapper update:  dxwrapper.zip

Closing this as a duplicate of #32.  I do have progress on this game and it is playable with the latest dev build,


New version: Plotly v0.3.0

Registering package: Plotly
Repository: https://github.com/plotly/Plotly.jl
Created by: @sglyon
Version: v0.3.0
Commit: 0a16fb75584b58e45d3580a16365a554efd561d6
Reviewed by: @sglyon
Reference: https://github.com/plotly/Plotly.jl/commit/0a16fb75584b58e45d3580a16365a554efd561d6#commitcomment-36700297

@sglyon Next time, please add the compat entries. You may find CompatHelper helpful.


Example policy from documentation does not work: invalid resource

The example policy in https://docs.min.io/minio/baremetal/security/IAM/iam-policies.html returns an error when used in mc admin policy add myminio mypolicy policy.json. myminio is running locally in a docker container.
Expected Behavior
mc admin policy add myminio mypolicy policy.json should add a new policy
Current Behavior
The command returns an error:
mc: <ERROR> Unable to add new policy: invalid resource 'arn:minio:s3:::*'.

Possible Solution
It seems the resource should be arn:aws:s3:::* rather than arn:minio:s3:::*.
Steps to Reproduce (for bugs)

Start MinIO in a docker container like this: docker run -p <IP_ADDRESS>:9000:9000 minio/minio server /data
Run mc admin policy add myminio mypolicy policy.json where policy.json contains the example from the policy documentation page

Your Environment

mc version RELEASE.2021-06-13T17-48-22Z
minio version RELEASE.2021-06-17T00-10-46Z

I'm fixing this as part of https://github.com/minio/docs/pull/353

Will also resolve https://github.com/minio/minio/issues/12662 in #353


feat: support setting optimizerStatisticsPackage

optimizerStatisticsPackage can be set in QueryOptions when running Cloud Spanner queries.
Can be configured through the following mechanisms:

Through the SPANNER_OPTIMIZER_STATISTICS_PACKAGE environment variable.
At Database level using spanner.instance('instance-name').database('database-name', sessionPoolOptions, queryOptions).
At query level using ExecuteSqlRequest.queryOptions.

If the options are configured through multiple mechanisms then:

Options set at an environment variable level will override options configured at the Database level.
Options set at a query-level will override options set at either the Database or environment variable level.

If no options are set, the optimizer statistics package will default to:

The optimizer statistics package the database is pinned to in the backend.
If the database is not pinned to a specific statistics package, then the Cloud Spanner backend will use the "latest" version.

@olavloite thanks for making #846  so neat. It made it so much easier to add this new option :)


Some Chinese characters not displayed from PDF generated by Inkscape

You can get the file here:
https://www.dropbox.com/s/09eg9pj6pf4g0ir/hk-2020.pdf
There are some Chinese characters failed to display on the map:

In the legend box to the upper right, you can see holes that fill up the character "觀塘" next to the green line, and "灣" just next to the red line.
On the red line, the station names are missing the character "灣" for "荃灣" and "長沙灣".
On the green line, the station names are missing the characters "塘" for "九龍塘" (light blue/green lines interchange) and "油塘" (purple/green lines interchange), "灣" for "九龍灣" and "觀塘" is completely missing.
On the blue line, the station names are missing the character "灣" for "灣仔", "銅鑼灣", "西灣河", "筲箕灣" and "柴灣".
On the brown line, the station names are missing the character "灣" for "土瓜灣".

In general, the characters missing are "灣", "塘" and "觀塘". If you highlight the text you can see the bounding box of the characters. If you select them and right click, you can see the proper text shown up in the Search Google for item.
Testing in Nightly and latest pdf.js (0.8.55). This happens both on Windows 8 (with hardware acceleration on) and Arch Linux (with latest update installed). The PDF file is generated from Inkscape 0.48.4 on Windows from an SVG file, using File > Save a Copy. The PDF setting are:

Restrict to PDF version: 1.5
Rasterize filter effects: checked
Resolution for rasterization: 90 dpi
Other options are left unchecked

The Chinese font used in the SVG is Adobe Fan Heiti Std (Adobe繁黑體 in Chinese) and it came from an Adobe CS6 Suite install.
The Web Console does not display any useful information.
[12:21:32.214] "PDF 27d234c2487072caf713a2b425f7128 [1.5 cairo 1.11.2 (http://cairographics.org) / cairo 1.11.2 (http://cairographics.org)]"
[12:21:32.385] "Warning: Not enough parameters for rcurveline; actual: 0, expected: 8"
[12:21:32.385] "Warning: Not enough parameters for callsubr; actual: 0, expected: 1"
[12:21:32.409] "Warning: Not enough parameters for rcurveline; actual: 7, expected: 8"

@xavier114fch Can you post a new link? The above does not work for me anymore. Or if anyone else has the pdf still.

issue3117.pdf

Closing as fixed by PR #6724.


[MINOR][SQL] Fix minor formatting issue of SortAggregateExec.toString

What changes were proposed in this pull request?
This PR fixes a minor formatting issue (missing space after comma) of SorgAggregateExec.toString.
Before:
SortAggregate(key=[a#76,b#77], functions=[max(c#78),min(c#78)], output=[a#76,b#77,max(c)#89,min(c)#90])
+- *Sort [a#76 ASC, b#77 ASC], false, 0
   +- Exchange hashpartitioning(a#76, b#77, 200)
      +- SortAggregate(key=[a#76,b#77], functions=[partial_max(c#78),partial_min(c#78)], output=[a#76,b#77,max#99,min#100])
         +- *Sort [a#76 ASC, b#77 ASC], false, 0
            +- LocalTableScan <empty>, [a#76, b#77, c#78]

After:
SortAggregate(key=[a#76, b#77], functions=[max(c#78), min(c#78)], output=[a#76, b#77, max(c)#89, min(c)#90])
+- *Sort [a#76 ASC, b#77 ASC], false, 0
   +- Exchange hashpartitioning(a#76, b#77, 200)
      +- SortAggregate(key=[a#76, b#77], functions=[partial_max(c#78), partial_min(c#78)], output=[a#76, b#77, max#99, min#100])
         +- *Sort [a#76 ASC, b#77 ASC], false, 0
            +- LocalTableScan <empty>, [a#76, b#77, c#78]

How was this patch tested?
Manually tested.

Test build #63173 has started for PR 14480 at commit 43081c5.

Test build #63173 has finished for PR 14480 at commit 43081c5.

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

Merged build finished. Test PASSed.

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

LGTM

Thanks, merging to master.


feat(deps): bump esp-hal fork

Description
This bumps our esp-hal fork.
Mostly, this adds the extra GPIO support for esp32, needed for #467.
Also, changes the used branch name to not include a + anymore (see cargo bug).
Issues/PRs references

Open Questions

Change checklist

[x] I have cleaned up my commit history and squashed fixup commits.
[x] I have followed the Coding Conventions.
[x] I have performed a self-review of my own code.
[x] I have made corresponding changes to the documentation.

(this is not touching any currently used code, so if it compiles, it should work(tm)).


Consolidate Connections detector

Perfmon data from Bilal's event's detector
Jenny's systemstat event - TIME_WAIT counts
Aspaan's detector
Server wide connection
Per VM limit of 160 (SNAP Port exhaustion)

There are numerous support tickets opened for connection problems and most often the support engineer gets the SystemStat network print out and puts it directly in the email. Example:
July 14:
Total = 4055 Inbound = 26, Outbound = 4026, LoopBack = 3
Established : 4008
TimeWait : 47
==> Remote-Endpoint
==> <IP_ADDRESS>:443
Established : 3952
TimeWait : 1
==> <IP_ADDRESS>:11078
Established : 1
==> <IP_ADDRESS>:80
Established : 2
==> <IP_ADDRESS>:443
Source: https://mssolveweb.partners.extranet.microsoft.com/MSSolveWeb/srdetails.html?q=117071416044438&s=0

Didn't mean to close


using dbsClient for own publication script

Hello,
I like to use the dbsClient to publish some files which crab is unable to do so (crab-dev cannot help). I tried to follow the instructions in dbsClient.py but to me it seems as if they are either outdated or not working properly. Can you provide me with some minimal instructions of how to use the interface to get a file published under a name that I can choose?
So far, I tried to follow how crab is using it [1] and just modied the 'dataset' dictonary to another processed_ds_name. This resulted strangely in an error "duplicated lfn".
I started again using just the 'insertFiles' method with the minimal settings as stated in the dbsClient.py file (here is my request [2]). This results in an "internal server error". The primary dataset and the dataset are already in dbs. Do I explicitly need to create a block or something else (e.g. acquisition_era) first?
What am I doing wrong? What would be the easiest way of publishing a file using dbsClient.py? Are the instruction in the file up-to-date?
Beyond that, it might be good to get a better error description from the server.
I am at CERN, so meeting in person would be great...
Best regards,
Matthias
[1] https://github.com/dmwm/CRAB2/blob/master/python/crab_dbs3publish.py#L418
[2]
[{'check_sum': '4108645093', 'event_count': '61087', 'logical_file_name': '/store/user/mkomm/SingleMu/2014Nov17_stpol/0b660941bfe423b9433b072f93b65912/output_Skim_9_1_K3w.root', 'block': '/SingleMu/mkomm-2014Nov17_stpol_foo-0b660941bfe423b9433b072f93b65912/USER#842def0f-bfdd-4b25-8465-f8d3361900c0', 'dataset': '/SingleMu/mkomm-2014Nov17_stpol_foo-0b660941bfe423b9433b072f93b65912/USER'}]

@matt-komm
@belforte
@ericvaandering
You may find DBS client document at https://cms-http-group.web.cern.ch/cms-http-group/apidoc/dbs3-client/current/dbs.apis.html. If you find a particular API document is wrong, please let us know and we will update it.
Publishing data in DBS requires a lot of details to be taken care. What you did by taking crab3 as example is a right approach.   We don't suggest DBS clients to call individual API to in server a file. The best way to do is call  insertBulkBlock API to insert a completed block.
The error you got was expected, for example the "duplicated lfn". Lfns have to be unique across entire CMS data. If you only modified the dataset and try to insert the rest of data as they were, you would get this kind errors.

Thanks for your answer. So I tried to invalidate first the already partially and wrongly published files before publishing the same lfns under a new dataset name ... unfortunately, I still get an "duplicated lfn" error although the dbs query "dataset file=<one of my file's lfn>" returns empty now after invalidating. Can I not publish those files again even when their original dataset is invalidated?

Invalidate anything in DBS means marking it invalid and will not physically remove it from DBS. So the Lfns are still in DBS. So you cannot reuse them.

Well that is very very bad for me... Is there a particular reason for this design choice. Basically, users will have to rerun their jobs when just something with the publication did not work out (e.g. config mistake). Like in my case, crab by default prevents from publishing the outputs of two tasks under the same name because it misses proper checks for lumi overlaps (according to dev).

Matthias, we need to go back to your issue, make sure it is well understood and
then find the proper solution in crab context, even modifying Crab3 if needed.
Although if you change configuration the only good solution is a new
dataset.
In the end we will always prefer robust bookkeeping to convenience,
replacing a file with another with same name is too fragile.
On 01/20/2015 10:41 PM, Matthias Komm wrote:

Well that is very very bad for me... Is there a particular reason for this design choice. Basically,
users will have to rerun their jobs when just something with the publication did not work out (e.g.
config mistake). Like in my case, crab by default prevents from publishing the outputs of two tasks
under the same name because it misses proper checks for lumi overlaps (according to dev).
—
Reply to this email directly or view it on GitHub
https://github.com/dmwm/DBS/issues/439#issuecomment-70739619.

--
Stefano Belforte - I.N.F.N.     tel   : +39<PHONE_NUMBER> (fax: 375-6258)
Area di Ricerca - Padriciano 99 tel mobile: +39<PHONE_NUMBER>
34012 TRIESTE TS  - Italy       AIM: stefanobelforte

Ok after learning and reviewing all the information about crab & dbs I found a working solution suddenly. I just set the block publication size to 1 and skipped the check if some files where already published before under the original dataset name. This way, publication will fail only for those files already present in dbs but all missing files are getting published.


gulp problem

[21:53:44] Task never defined: default
[21:53:44] To list available tasks, try running: gulp --tasks
[lk@blackbird theme-machine]$ gulp watch
[21:54:44] Using gulpfile /lab/hackerthemes/theme-machine/gulpfile.js
[21:54:44] Task never defined: watch
[21:54:44] To list available tasks, try running: gulp --tasks
[lk@blackbird theme-machine]$ gedit gulpfile.js

This should be fixed by upgrading gulp to 4.0.1 or higher. I haven't had time to do push the changes unfortunately but it can be done using npm pretty easily.

thanks for the prompt response.


Bump @metamask/utils to 2.1.0

Supercedes #26 (see there for release notes). This update has no impact on this tool but we might as well bump it.

Will bump this later, now that 3.0.1 is out.


Request to add Metadata API environment variables

Would you consider adding support for explicit timeout & retry when accessing the AWS Metadata API, similar to AWS_METADATA_SERVICE_TIMEOUT & AWS_METADATA_SERVICE_NUM_ATTEMPTS in Boto3?
See:
https://boto3.amazonaws.com/v1/documentation/api/latest/guide/configuration.html
https://github.com/boto/botocore/blob/21ad34d727a301dbfc3bb47159a41cbd64e3bff2/botocore/configprovider.py#L59-L68
Reasoning:
I'm indirectly using Minio (I'm an Argo user: https://github.com/argoproj/argo) in an Amazon EKS cluster. There's a bad interaction with another tool (kube2iam), that may introduce errors when accessing the Metadata API. We've been able to circumvent the issue in our Python projects using Boto3 using these two environment variables, but Argo (using Minio) has a high number of failures in our environment.
Thanks!

@jhsmith Just to clarify, do you need to use environment variables similar to the ones you have mentioned above but for S3 in minio-go?
Also, minio implements only S3 API. Will check internally if we can support such an environment variable.

Correct. We'd like environment variables that control performing retries (and controlling the timeout) when accessing the following urls upon encountering an error: https://github.com/minio/minio-go/blob/e6694bcc4a6b0c1756797be85b8458635a65924e/pkg/credentials/iam_aws.go#L54
We believe this will help alleviate what is may be a race-condition or a design issue with kube2iam: https://github.com/jtblin/kube2iam/issues/136
Adding retries and increasing the timeout when accessing the AWS Metadata API in Boto3 allowed our Python projects to work as expected.

i would really like to be able to control the behavior of this http call, as it seems precarious to have any http call that could end your session if it doesn't get success on the first and only try.

We would love to support feel free to send a PR

@jhsmith I spoke about this internally and we have decided not to add environment variables. Application should handle environment variables and not SDKs. If let's say there are more than one application using the same SDK and one wants to use the environment variable then another application will get it by mistake.

Closing this issue as won't fix. Please feel free to reach out to us if you have further questions.

@jhsmith did you end up finding a workaround? just got bit by this one ourselves with KIAM.

@stefansedich we had to go around kiam/kube2iam and add explicit aws creds as env vars for our pods rather than relying on the metadata api. Rather disappointing.


Minor fix: Color mismatch

Two colors were accidentally swapped at some point in commit history, this fixes that. Also changes the grid color in dark mode to be less sharp of a contrast, making it easier on the eyes.
Summary by Sourcery
Fix color mismatches in the dark mode color scheme by correcting swapped colors and adjusting the grid color for better visual comfort.
Bug Fixes:

Correct the swapped colors for 'background' and 'bed' in the dark mode color scheme.

Enhancements:

Adjust the grid color in dark mode to a less contrasting shade for improved visual comfort.

Someone on windows should test just to verify this works properly


Revert breaking changes

revert breaking changes for now to release next minor.
revert:

#2216
#2181
#2180
#2090
#2048

We can either release 2.0 directly or the other idea was to have a deprecation workflow as proposed in https://github.com/woodpecker-ci/woodpecker/discussions/2383#discussioncomment-6965949.


Roll engine 5a730c60d3ce..bdc9708d235e (25 commits)

https://github.com/flutter/engine/compare/5a730c60d3ce...bdc9708d235e
git log 5a730c60d3ce..bdc9708d235e --first-parent --oneline
2019-12-28<EMAIL_ADDRESS>Revert "Use ELF for Dart AOT snapshots on Fuchsia. (#13896)" (flutter/engine#14823)
2019-12-27<EMAIL_ADDRESS>Roll src/third_party/skia 89bf1547f3aa..5a2f962313a5 (16 commits) (flutter/engine#14825)
2019-12-27<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from ikxBh... to sxapd... (flutter/engine#14821)
2019-12-27<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from IEPKx... to iImpF... (flutter/engine#14817)
2019-12-27<EMAIL_ADDRESS>Roll src/third_party/dart 94a4f6415e6c..c547f5d933e5 (9 commits) (flutter/engine#14812)
2019-12-27<EMAIL_ADDRESS>Roll src/third_party/skia f2d522a3f434..89bf1547f3aa (4 commits) (flutter/engine#14814)
2019-12-26<EMAIL_ADDRESS>Roll src/third_party/dart 232a171a0b1c..94a4f6415e6c (8 commits) (flutter/engine#14808)
2019-12-26<EMAIL_ADDRESS>Roll src/third_party/skia 07d744fb8a53..f2d522a3f434 (4 commits) (flutter/engine#14807)
2019-12-26<EMAIL_ADDRESS>Roll src/third_party/skia 05eb83be171c..07d744fb8a53 (3 commits) (flutter/engine#14806)
2019-12-26<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from v3R8A... to ikxBh... (flutter/engine#14802)
2019-12-26<EMAIL_ADDRESS>Roll src/third_party/dart 75fc15c7e186..232a171a0b1c (4 commits) (flutter/engine#14803)
2019-12-26<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from ZA31z... to IEPKx... (flutter/engine#14702)
2019-12-26<EMAIL_ADDRESS>Roll src/third_party/skia 3d3150c89c5f..05eb83be171c (3 commits) (flutter/engine#14800)
2019-12-24<EMAIL_ADDRESS>Roll src/third_party/skia a2d7225c0fc1..3d3150c89c5f (1 commits) (flutter/engine#14680)
2019-12-24<EMAIL_ADDRESS>Roll src/third_party/dart 8736fec094bb..75fc15c7e186 (6 commits) (flutter/engine#14679)
2019-12-24<EMAIL_ADDRESS>Run and collect benchmarks (flutter/engine#14556)
2019-12-23<EMAIL_ADDRESS>Roll src/third_party/dart 4a639e8df261..8736fec094bb (11 commits) (flutter/engine#14678)
2019-12-23<EMAIL_ADDRESS>Roll src/third_party/skia ac29f1342fd2..a2d7225c0fc1 (5 commits) (flutter/engine#14677)
2019-12-23<EMAIL_ADDRESS>fix selection issue (flutter/engine#14604)
2019-12-23<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from TUoVa... to v3R8A... (flutter/engine#14670)
2019-12-23<EMAIL_ADDRESS>Roll src/third_party/dart cfca5ea9f2e9..4a639e8df261 (7 commits) (flutter/engine#14671)
2019-12-23<EMAIL_ADDRESS>Roll src/third_party/skia 6ec826085b2a..ac29f1342fd2 (1 commits) (flutter/engine#14672)
2019-12-23<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from IZ278... to ZA31z... (flutter/engine#14675)
2019-12-21<EMAIL_ADDRESS>Roll src/third_party/dart 1db1a837f8c9..cfca5ea9f2e9 (2 commits) (flutter/engine#14667)
2019-12-21<EMAIL_ADDRESS>Roll src/third_party/skia 2723af6a6d43..6ec826085b2a (3 commits) (flutter/engine#14665)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


Recorded Future Identity Bugfixes and solution pack fixes

Required items, please complete
Change(s):

Removed hard-coded  parameters.
Fixed Solution naming

Reason for Change(s):

Bug fixes

Version Updated:

No

Testing Completed:

Yes

@v-prasadboke is this PR ok? It contains two fixes for not using hard-coded parameters and pick up the values from the installation. And fixes for the Solution where we used the wrong name and updated to paths in the solution.json


Russian Peasant Algorithm - C++

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#978

@atarax665 I have made the required changes


Does middleman regenerates all pages when using 'build' ?

Yes, unless you use the --glob flag

Here is the output in the console :
   create  build/generic-page/index.html
   identical  build/elements/index.html
   updated  build/generic-page/index.html
   remove  build/css/ie9

If I have 100 pages, it's not good if they are going to be generated every time I use middleman build unless I modified the layout (e.g. modifying generic.html.erb implicates that all pages that have this layout will be regenerated). From what I saw in the console I think it doesn't generate them all, just what I modified.

Everything will be generated on every build. If the output is the same as the current file, it will say so.


refactor(image_projection_based_fusion): read lidar models parameters from autoware_launch

Description
This PR provides migrating lidar detection model parameters from autoware_launch
Releated: https://github.com/autowarefoundation/autoware.universe/issues/4250

autoware_launch PR: https://github.com/autowarefoundation/autoware_launch/pull/450

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@yukke42 Could you review this PR?

@yukke42 I resolved conflicts :+1:

https://github.com/autowarefoundation/autoware.universe/actions/runs/5612516522/job/15206437184?pr=4278
CI fails due to the shotage of the disk space ...
https://github.com/autowarefoundation/autoware.universe/issues/4349

@ismetatabay Cloud you check to launch logging_simulator.launch in rosbag replay simulation?
I checked to run lidar_centerpoint and get output from perception in the main branch but not in this PR.

@yukke42 Hi, I checked and confirmed this PR and autoware_launch PR works as expected in my local environment, logging_simulator demo works without any problem. I also checked changing parameters in autoware_launch and there is no problem for me. How did you reproduce this situation? Can you give me some more info? Could there be a problem in the build phase? (maybe symbolically linked .param file have a issue)

@ismetatabay Sorry, I missed something. I built again and then confirmed that this PR works correctly!

@yukkysaito Hi, can you check it also?


Name of the labels 'Backend host' and 'Frontend host' are misplaced

Steps:
1.Visit https://apinf.io
2.Sign in
3.Select 'API Backends' from menu
4.Click on 'Add API Backend'
5.Enter API name
6.Select the backend protocol from drop down menu
7.Enter the details of servers: Host and Port
8.Enter Frontend host and Backend  host

Names of the labels 'Backend host' and 'Frontend host' are misplaced

Screenshot:

Confirmed. I stumbled upon this today. When adding API and when editing API, front and backend change order.

Possibly resolved with combining Add API backend and Edit API backend forms. To be checked.

Fix the following lines in the relevant file: https://github.com/apinf/api-umbrella-dashboard/blob/develop/client/views/api_backends/api_backend_form/api_backend_form.html#L18-L31

Verified this resolved issue on develop version.


Supporting informations page

CHANGES:
I added a table of supporting information, Supporting information is a collection of
Correspondence::Email, Correspondence::PhoneCall, Correspondence::Meeting, Test::Result and CorrectiveAction
At the bottom are the Generic attachments: attachments that are directly linked to a case but and therefore not linked to any of the supporting information
Things to watch out for, link to the view page for corrective actions will be removed in the table if the viewing user does not have access to view restricted details.

 (5) Table view for supporting information

looks good to me!


Document modifications to the release process

Summary
This commit updates the CONTRIBUTING.md file to more clearly document
the current release process. Previously for the release cycle
documentation we just had a mechanical list of the steps done by the
core team and what the automation does but it didn't really contain any
details on the aspects of how contributions with releasing. This commit
updates this section to better describe the different phases of a
release and the expectations and order of operations a contributor will
need to know around releasing.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.363%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
54490

Relevant Lines:
64590

💛  - Coveralls


Help with guestbook example

I have the kubernetes cluster up, just can't seem to make the guestbook example reachable from the outside in the final step. It's likely user error on my part, any advice would be awesome.

Simply uncomment "type:LoadBalancer", then run kubectl describe svc frontend and you will see NodePort (31265/TCP) in my case. This port is exposes your service on every single k8s node, so simply point your browser to this port on any node and you will see running guestbook.
Name:			frontend
Labels:			name=frontend
Selector:		name=frontend
Type:			LoadBalancer
IP:			<IP_ADDRESS>
Port:			<unnamed>	80/TCP
NodePort:		<unnamed>	31265/TCP
Endpoints:		<IP_ADDRESS>:80,<IP_ADDRESS>:80,<IP_ADDRESS>:80
Session Affinity:	None

The message reads:
                        An external load-balanced service was created.  On many platforms (e.g. Google Compute Engine),
                        you will also need to explicitly open a Firewall rule for the service port(s) (tcp:80,tcp:443) to serve traffic.

                        See https://github.com/GoogleCloudPlatform/kubernetes/tree/master/docs/services-firewalls.md for more details.

Based on that, we shouldn't have to know the port 31265. But it does seem that is required to actually see what is running outside of the nodes themselves.

This actually makes sense when you run on Cloud provider like GCE or AWS and have integration with external LB. But for our case it just simpler to know on which port it is published. Here is iptables rules on node:
Chain KUBE-NODEPORT-CONTAINER (1 references)
target     prot opt source               destination
REDIRECT   tcp  --  <IP_ADDRESS>/0            <IP_ADDRESS>/0            /* default/frontend: */ tcp dpt:31265 redir ports 51351

Chain KUBE-NODEPORT-HOST (1 references)
target     prot opt source               destination
DNAT       tcp  --  <IP_ADDRESS>/0            <IP_ADDRESS>/0            /* default/frontend: */ tcp dpt:31265 to:<IP_ADDRESS>:51351

I found the following items that help especially given external load balancer capabilities:
This provides the discussion and the need I was referring to:
https://github.com/GoogleCloudPlatform/kubernetes/issues/9995
The workaround for now:
https://github.com/GoogleCloudPlatform/kubernetes/tree/v1.0.1/contrib/for-demos/proxy-to-service

Thank you for sharing.
Well, at least for testing current setup is enough, besides that's the real scenario how external traffic passes to services in cloud provider where integration works. Maybe some time ago OpenStack will be in that list )
FYI there is also some interesting project for external connectivity kiwi

Yup, everything works!


Sound Volume doesn't work

Describe The Bug:
The button volume on iPhone doesn't modify the volume of the TV when the widget is enable
To Reproduce:
I have restart servers, and iPhone but it doesn't work.
Expected behavior:
When I press the button volume on the left of the iPhone, when the widget TV is enable
Logs:
The logs doesn't log when I press button volume
Plugin Config:
  "accessories": [         {             "accessory": "PhilipsAndroidTV",             "name": "Philips TV Oled",             "ip": "YYY",             "macAddress": "ZZZ",             "apiUser": "XXX",             "apiPass": "YYY",             "apps": [                 "Plex",                 "Netflix",                 "YouTube",                 "Guide TV"             ],             "channels": [                 "TF1",                 "France 2",                 "M6",                 "BFM TV",                 "LCI"             ]         }     ]
Environment:

Plugin Version: V2.0
Homebridge Version: 1.2.3
Node.js Version: v12.16.3
NPM Version: v6.14.5
Operating System: Raspbian

Thank's for this great plugin, the first for me which works fine with channels, apps AND widget.

Hi, thanks for reporting
actually I didn't get yet how to control volume from iPhone. :)
It works properly in Homebridge UI, but couldn't figure out where it supposed to be on iPhone.
Have you seen volume working in an other plugin?

Added missing attribute, and volume control works from iPhone Remote App.
        this.tvSpeaker.setCharacteristic(hap.Characteristic.VolumeControlType, hap.Characteristic.VolumeControlType.ABSOLUTE);
I will publish it today thanks.

Delivered as part of 0.2.2 drop

Thanks a lot, the volume is now functional !!
However the status of the device in the home application doesn't work....
This appear with the v0.2.2n, not initially with the 0.2.0.
Any idea ?

Hey,
Great to hear that 👍
Do you mean that device is offline, when its on?
Or that cant be connected/not responsive?
Compared to 0.2.0, I’ve added 10 sec interval to check status to keep those up to date ( on, off, which channel is set) - because it was losing synchronization when normal remote is used.

The status appears in uptdate, and after that the device is not connected / not responsive.
Maybe 10 sec of interval is to long...
Thanks

Tested v0.2.3, all works fine.
Great job 👍
Thanks a lot !


Displays a warning when starting

D:\AI\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-segment-anything-2\sam2\modeling\sam\transformer.py:20: UserWarning: Flash Attention is disabled as it requires a GPU with Ampere (8.0) CUDA capability.
OLD_GPU, USE_FLASH_ATTN, MATH_KERNEL_ON = get_sdpa_settings()

Hi, this is likely because you have an old GPU that doesn't support Flash Attention. If the image or video prediction works for you, you can ignore this warning.
If there is an error, we have recently added fallback to all available kernels (in https://github.com/facebookresearch/segment-anything-2/pull/155) as a workaround to this problem. You can pull the latest code via git pull, which should resolve the error.

Good evening.  I have a 3080ti which should allow this, but I am getting this same error:
[...]\ComfyUI-segment-anything-2\sam2\modeling\sam\transformer.py:20: UserWarning: Flash Attention is disabled as it requires a GPU with Ampere (8.0) CUDA capability.
  OLD_GPU, USE_FLASH_ATTN, MATH_KERNEL_ON = get_sdpa_settings()

im getting the same error:
[...]ComfyUI\custom_nodes\ComfyUI-segment-anything-2\sam2\modeling\sam\transformer.py:20: UserWarning: Flash Attention is disabled as it requires a GPU with Ampere (8.0) CUDA capability.
  OLD_GPU, USE_FLASH_ATTN, MATH_KERNEL_ON = get_sdpa_settings()

Heck, I have a 4090 with this. :(


Type narrowing on object properties lost in async IIFE

TypeScript Version:  3.4.0-dev.20190327

Search Terms:
narrowing iife control flow
Code
declare const x: string | undefined;
declare const y: { z: string | undefined };
function needsString(it: string) { return it; }

function a() {
  if (!x) {
    throw new Error("Missing x");
  }
  const res1 = (() => needsString(x))(); // all good, per #8849
  const res2 = (async () => needsString(x))();  // ditto

  if (!y.z) {
    throw new Error("Missing z.")
  }

  const res3 = (() => needsString(y.z))();  // still good
  const res4 = (async () => needsString(y.z))();  // now things blow up
}

Expected behavior:
The call to needsString in the expression for res4 succeeds, like the one in res2 and res3. Even though the function is async, the code in it -- at least that's before an await -- runs immediately, I believe, and so should be subject to the narrowing (esp. given that res2 works).
Actual behavior:
Error for needsString(y.z): Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
Playground Link: https://www.typescriptlang.org/play/index.html#src=declare const x%3A string | undefined%3B
declare const y%3A { z%3A string | undefined }%3B
function needsString(it%3A string) { return it%3B }

function a() {
  if (!x) {
    throw new Error("Missing x")%3B
  }
  const res1 %3D (() %3D> needsString(x))()%3B%20%2F%2F%20all%20good%2C%20per%20%238849%0A%20%20const%20res2%20%3D%20(async%20()%20%3D%3E%20needsString(x))()%3B%20%20%2F%2F%20ditto%0A%0A%20%20if%20(!y.z)%20%7B%0A%20%20%20%20throw%20new%20Error(%22Missing%20z.%22)%0A%20%20%7D%0A%0A%20%20const%20res3%20%3D%20(()%20%3D%3E%20needsString(y.z))()%3B%20%20%2F%2F%20still%20good%0A%20%20const%20res4%20%3D%20(async%20()%20%3D%3E%20needsString(y.z))()%3B%20%20%2F%2F%20now%20things%20blow%20up%20%20%0A%7D
Related Issues:
https://github.com/Microsoft/TypeScript/pull/8849, which I think was supposed to fix IIFE issues like this, but appears to have missed a case (or I'm missing something).

async and/or generator functions that are immediately invoked are intentionally not inlined in the control flow, i.e. they were not missed back then.
However, it doesn't really make sense to treat these cases different since yield and await don't reset narrowing in the current function scope (see #27900).
The only thing that comes to mind is calling an IIAFE inside a constructor. Inlining control flow would make the type checker think that a property is definitely assigned in the constructor while at runtime it may be assigned later:
class Foo {
  prop: number; // currently there's an error with strictPropertyInitialization

  constructor() {
    (async () => this.prop = await something)();
  }
}

async and/or generator functions that are immediately invoked are intentionally not inlined in the control flow

If that's true, then why does this assignment from my example work (even under strictNullChecks):
const res2 = (async () => needsString(x))();  // ditto


Post the news for QInfer making Julia interface.

Need one person to review. I will squash if necessary. Thanks.
Cheers,
Qi

@amitjamadagni Thank you for the review! I will prioritize some JuliaQuantum tasks this summer on my own schedule. Hopefully, we will work together soon.


Update Work Show for Consistent Headings and Collection Relationships

Grabbed some earlier code from Sufia before nested works was introduced. Replaces relationships_parent_rows with an unordered list that just grabs the collections and updates the header in the yml file. Updates text for heading change and adds comments.


Safari cannot open the page because the address is invalid

I'm getting the following error when trying to open version.url with
Linking.openURL(version.url)
Safari cannot open the page because the address is invalid
My version object looks like this:
{"bundleId": "com.dahlsjoo.shinner", "country": "us", "lastChecked": "2021-07-16T00:05:22.581Z", "needsUpdate": false, "notes": "Bug fixes", "platform": "ios", "released": "2021-07-01T18:19:45Z", "updateType": null, "url": "https://apps.apple.com/us/app/shinner-skate-spots-videos/id1476836516?uo=4", "version": "1.2.4"}

Hi! I don't think that's directly related to this library, as the payload the library returns seems to contain a valid URL.
Would you be able to share a bit more of your code so that I might be able to give you some pointers?


Move inventory around to allow for additional inventory files

This will all make more sense in a minute once I open another pull request :)
Basically, the purpose of this is to allow the creation of additional inventory files in the hosts/ directory, such as the data for a Jenkins slave. The .gitignore changes makes it so you can locally add your Jenkins slave inventory file, without it polluting the repository.


tslint.enable also deprecated?

I migrated from vscode-tslint and now the option tslint.enable reports as unknown. Does it mean that it is enabled by default for all .ts .tsx files? How do I configure it to not lint per workspace/filetype?

Use tslint.exclude or disable the extension per workspace


intake-stac setup and error on aws search example

Hello,
I've noticed that there is an error running the aws search notebook example. Am getting, at cell 11 loading the item to Dask da = item.B04(chunks=dict(band=1, y=2048, x=2048)).to_dask() the following error:
ValueError: No plugins loaded for this entry: image/tiff; application=geotiff; profile=cloud-optimized A listing of installable plugins can be found at https://intake.readthedocs.io/en/latest/plugin-directory.html .
I've ensured that the module is installed and the example on the intake-stac readme does run fine.
Initially was running this inside of the pangeo notebook docker image and thought that might be the issue but am getting this same error just installing the dependencies in a clean virtual environment.

Hi @dogrdon-cu, sorry about that, we're working on a fix over in https://github.com/intake/intake-stac/pull/61 and hope to have a new release ASAP that fixes this.
In the meantime a workaround is to pin intake-xarray=0.3.1 in your environment

thanks @scottyhq that worked.
FWIW, I noted that I also had to remove and reinstall intake-stac with
pip uninstall intake-stac
pip install git+https://github.com/intake/intake-stac
in the pangeo-notebook to get from the plugin error to the open_local error ref. in #61

@scottyhq Any workarounds for dealing with 'sentinel-s2-l1c' products? From the example above, intake-xarray=0.3.1 works for Sentinel 2 L2A dataset but not for Sentinel 2 L1C dataset returning the following error:
ValueError: No plugins loaded for this entry: image/jp2
A listing of installable plugins can be found at https://intake.readthedocs.io/en/latest/plugin-directory.html .
It seems there's a plugin issue due to the different format of the datasets from Sentinel 2 even though they're both raster files that should be handled by the existing plugins.

@hssiddiqui , thanks for pointing this out. this pending PR #61 sets the driver to 'rasterio' by default so even if the entry type is not recognized we'll attempt to read the data rather than going straight to the ValueError.
So for now you'll have to install from the branch in the PR above. Hopefully this week we'll get new versions released...
Note also the S2L1C imagery is in a requester-pays bucket, so you'll have to use the following syntax:
import rasterio
awsconfig = rasterio.session.AWSSession(requester_pays=True)
with rasterio.Env(session=awsconfig):
    da = item.B01.to_dask()

closed by #61


az role assignment list unexpected error

This is autogenerated. Please review and update as needed.
Describe the bug
When running the az role assignment list an error is thrown but conitnues.
Command Name
az role assignment list
Errors:
'id=/subscriptions/cdd8dcb4-1f9b-4d0a-af79-170583e49d3a/resourceGroups/rg-vm-operations-westus2-001; location=westus2; managedBy=; name=rg-vm-operations-westus2-001; properties=; tags=; type=Microsoft'
Traceback (most recent call last):
Temp\pip-unpacked-wheel-rpp6y7vh\knack\cli.py, ln 215, in invoke
cli\core\commands\__init__.py, ln 659, in execute
cli\core\commands\__init__.py, ln 722, in _run_jobs_serially
cli\core\commands\__init__.py, ln 715, in _run_job
...
Temp\pip-unpacked-wheel-qa9gi9_9\msrest\service_client.py, ln 151, in format_url
KeyError: 'id=/subscriptions/cdd8dcb4-1f9b-4d0a-af79-170583e49d3a/resourceGroups/rg-vm-operations-westus2-001; location=westus2; managedBy=; name=rg-vm-operations-westus2-001; properties=; tags=; type=Microsoft'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az cloud set
az login
az ad signed-in-user show
az account list
az role assignment list --include-inherited --include-groups --subscription {}

Expected Behavior
A list of roles is returned.
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.17.1

Extensions:
interactive 0.4.4

Additional Context

az role

az role

This issue is fixed. Please upgrade to latest version of azure cli 2.39.0