Commiting non staged changes

Type: Bug
Hello,
I've been looking all over internet to find a way to fix it. I unchecked "smart commit" option but still, it's commiting non staged changes.
I happened to me on this computer and in another mac computer.
VS Code version: Code 1.80.0 (660393deaaa6d1996740ff4880f1bad43768c814, 2023-07-04T13:39:48.236Z)
OS version: Linux x64 5.19.0-46-generic
Modes:

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 3968)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
15.30GB (8.13GB free)

Process Argv
--unity-launch --crash-reporter-id e810f301-1267-4e71-9dbc-7a720b5306f5

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
wayland

Extensions (6)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.4.2

githistory
don
0.6.20

prettier-vscode
esb
9.19.0

git-graph
mhu
1.30.0

svg-preview
Sim
2.8.3

json-color-token
yec
1.3.2

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
282f8724:30602487
89544117:30613380
showlangstatbar:30737416
vsctsb:30748421
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30771522
hideindicator:30785051
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30779714
pythonidxpt:30784022
pythonnoceb:30776495
h7j2d465:30786200
dsvsc013cf:30789518

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.80.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[UIFlow 2.0] [JP] Hat & Unit -ToF (additional)

Additional translation the "ToF" hat in UIFlow 2.0 under the Hat category into Japanese.
Additional translation the "ToF" unit in UIFlow 2.0 under the Unit category into Japanese.

Translation is finished.


Time condition in parser

Hi Olaf,
the Sysmon parser includes a time condition (i.e. ago). The best practice is to leave out the time condition from a parser and leave it to the parser user to add when using the parser.
~ Ofer

Hi @oshezaf, thanks for the heads up! It's much appreciated, i've now removed the time condition from the parser.


Update node to 14.19.3.

This is the latest version in the 14 LTS line. I've seen a lot of failures installing 14.19.0 recently and I'm hoping this will help.

Love me some new-er node!


Update to Venafi support page.

Please hold on merges pending vendor review
Thanks

Ok. The vendor approves. Happy to merge after review.


Can't seem to get 'theme' to work

Code excerpt:
        $select.selectize({
            theme: 'mytheme',
            valueField: 'id',
            labelField: 'name',
            searchField: 'name',
            dropdownParent: 'body',
            render: {
                option: function(data, escape) {
                    return '
' +
                        '' + escape(data.name) + '
';
                }
            },

In the DOM, nothing has 'mytheme' attached to the classes and thus my css isn't getting picked up.
Basing this off the github example.

This should be mentioned more clearly in the website/docs/examples. For example https://github.com/brianreavis/selectize.js/blob/master/examples/customization.html


[BrokenLinksH2] Fix path in link

Docs Build status updates of commit 5d986b6:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Implement -c COUNT on windows

Now you can specify that you want x screenshots and then want to exit if you are
on windows. This was implemented at the JS side and on *ix, but not on windows.

Thank you!

Run of -c 1:
Existing config file found ("ingress-ice.conf"). Starting ice...
Taking 1 screenshots...

     _____ )   ___      _____)
    (, /  (__/_____)  /
      /     /         )__
  ___/__   /        /
(__ /     (______) (_____)  v4.3.0

If you need help or want a new feature, visit https://github.com/nibogd/ingress-ice/issues
21.02.2016 17:52:15: Logging in...
21.02.2016 17:52:29: Validating login credentials...
21.02.2016 17:52:51: Will start screenshooting in 180 seconds...
21.02.2016 17:52:51: The first screenshot may not contain all portals, it is intended for you to check framing.

21.02.2016 17:52:51: Screen #1/1 captured
21.02.2016 17:52:53: Screen saved
21.02.2016 17:55:51: Finished sucessfully. Exiting...
Thanks for using ingress-ice!

What I don't understand is why there is first a Will start screenshooting in 180 seconds... but the screenshot is taken immediately, but then afterwards it takes 3min to shutdown. Looks like there is some kind of wrong sleep?
Actually It would be nice if there wouldn't be a sleep before the first screenshot and no sleep after the last...

Hm… I'll think about it. Thanks for report ;)

but the screenshot is taken immediately
It meant to be a testing screenshot to check framing


i3 icons wont change from qt5c and wont change manually

Describe the issue
hello i am new here, so sorry for the posible mistakes.
i do change my i3 bar with qt5c usually, but one day randomly the battery icon get bugged and now is a cloud. (the charge% is working properly)
after that all the icons did bug and now when i try to change the i3bar to the default icons i just cant.
what should i do ? btw what is the manjaro original icon pack in the i3bar ?
Was it working before?
yes, all was working normaly with diferent icons packs, i did change the bar icons a lot with qt5c
Screenshots
if you can see up, the battery icon now is a cloud and the icons are from adwaita and i am using other icon pack,  i did try to change the icons from diferent ways and nothing happens
https://imgur.com/gallery/qAdof0I
Environment:

WM: i3
Distro: manjaro
Output of polybar -vvv: i dont know

Additional context

i3bar and polybar are not the same. It's not totally clear from the screenshot whether you are using i3bar or polybar but since polybar does not support these qt5c icon packs and you don't seem to have the polybar command on your machine, I'm assuming you are actually using i3bar.
The i3 repository that contains i3bar is here, but I think such a question is better asked on their subreddit.
If I made a mistake and you are actually using polybar, please post your polybar config as well as the polybar output when you start your bar in the terminal and the output of the polybar -vvv command.


PMenuBar should support on mouseOver - on MouseOut

If you click on PMenuBar PItem, it expands a subMenu.
Then lose the focus on the subMenu and hover other main Pitems: the subMenu is still appearing, which is confusing
It would be nice that if mouse is out of the subMenu window, the subMenu disappear

ponysdk-1.13


Local Assets Path

After #13, things are feeling pretty clean, but the built app still reaches out to js.arcgis.com for assets:

... which is OK in most cases but may be an issue for some users. It would be nice to update the example to use the assetsPath property per https://developers.arcgis.com/javascript/latest/es-modules/#working-with-assets (maybe using ncp package, if necessary) to avoid any requests going to js.arcgis.com.

The CDN assets are our current recommended way of loading assets. Doing the local copy of assets will probably add to build times (dev and prod) and larger deployed sizes. Maybe you can add a readme note on how to do the local copy if needed, but I think you should leave the CDN assets as default.

Thank you very much for that clarification @odoe. Now that I am reading this more critically:

For most local builds, the API's assets are pulled from the ArcGIS CDN at runtime and there is no need for additional configuration. The assets include styles, images, fonts and localization files. This behavior is based on the default setting for config.assetsPath.
https://developers.arcgis.com/javascript/latest/es-modules/#working-with-assets

... I do see how it implies that the recommended approach is to allow the assets to load from the CDN

I'm now revisiting this topic as I'm faced with a project where we have to rely totally on locally managed assets. No CDN chatter allowed.
Firing up ncp and setting up the esriConig.assetsPath property all check out to me, but have either of you figured out where to copy ArcGIS ESM assets with Vite?
The Vite docs about static assets https://vitejs.dev/guide/assets.html provide both great explanations along with interesting caveats. Maybe the /public folder isn't quite the right spot? But if it is, I don't know how we'd have this line
esriConfig.assetsPath = "./assets";
point to the correct place if you're in dev mode or building for production (where it all ends up shifted to the /dist folder).
Perhaps we'd rely on env vars? https://vitejs.dev/guide/env-and-mode.html#env-variables

UPDATE: I think I've figured out the right combo and will report back with some notes once I just verify a few more things.

@jwasilgeo not sure if you can even see this, but if so, how did you end up using static assets in Vite?

@samMatenaer as I recall it is doable. I think when I wrote the earlier comments/investigation above something wasn't that clear to me but then it started to click eventually. (I regret not updating this thread afterwards though.) I think it came down to each 3rd party library involved (3rd party from the perspective of Vite) might need its own handholding depending on certain criteria. Vite itself is great though. There should be some examples floating around in EsriPS I think? @gavinr @dhatcher


Third party library: google-protobuf eval issue

I am trying to install google-protobuf, which has two components
a) npm package
b) generated js files
OS?

Mac OSX Sierra

Versions.

angular-cli: 1.0.0-beta.24
node: 6.8.1
os: darwin x64
@angular/common: 2.4.1
@angular/compiler: 2.4.1
@angular/core: 2.4.1
@angular/forms: 2.4.1
@angular/http: 2.4.1
@angular/platform-browser: 2.4.1
@angular/platform-browser-dynamic: 2.4.1
@angular/router: 3.4.1
@angular/compiler-cli: 2.4.1

Repro steps.

per doc:

npm install google-protobuf --save
copied the protoc generated js files in a local folder libraries/protoc
added in angular-cli.json:

      "scripts": [
        "../node_modules/google-protobuf/google-protobuf.js",
        "libraries/protoc/client_pb.js"
      ],

in src/typings.d.ts:

declare module 'google-protobuf';
declare module 'client_pb';

in the service

import * as googleProtobuf from 'google-protobuf';
import * as p from 'client_pb';

The log given by the failure.

after running ng build (no errors),
the scripts.bundle.js contains

/***/ 217:
/***/ function(module, exports) {
module.exports = "var COMPILED=!0,goog=...."...

and
/***/ 615:
/***/ function(module, exports) {
module.exports = "/**\n * @fileoverview\n * @enhanceable\n * @public\n */\n// GENERATED CODE..."...

(i.e. code is in quotes)
when running (ng serve), this creates errors like
VM739:321Uncaught ReferenceError: exports is not defined
    at eval (eval at module.exports (addScript.js:9), <anonymous>:321:471)
    at eval (<anonymous>)
    at module.exports (addScript.js:9)
    at Object.161 (google-protobuf.js?a64b:1)
    at __webpack_require__ (bootstrap 44c22f0…:52)
    at Object.720 (scripts.bundle.js:51)
    at __webpack_require__ (bootstrap 44c22f0…:52)
    at webpackJsonpCallback (bootstrap 44c22f0…:23)
    at scripts.bundle.js:1

looks to me that evaluating the code in the string causes these errors
Mention any other details that might be useful.
When using webpack 2.2 directly in a js project, it creates code like
/* 1 */
/***/ function(module, exports) {
var COMPILED=!0,goog=...

working fine
i.e. without bundling/eval the code in a string like
module.exports = "..."

Questions:

How do I include the js code directly in scripts.bundle.js  (assume that the eval is the cause of the error)?
I copied the generated js code (i.e. not npm packaged) in the project and referenced it in the script file (see above) _ Is this the correct/recommended approach?

I assume the case of the eval error is that the npm package google-protobuf
not just contains the "google-protobuf.js" file but that it dynamically loads additional .js files in the included directory "google/protobuf/compiler/.js".  These dependencies are handled correctly by webpack in a js project.. as without the string eval.
... so seems to me the solution might be to just hand it over to webpack to handle/include the dependencies ... and yes, it will then create a global variable.

Dupe of https://github.com/angular/angular-cli/issues/2141, fixed in https://github.com/angular/angular-cli/pull/3814. Should be in the next release.

@jorgjanke I encountered the same problem when import google-protobuf.js. How did you solve it?

... switched to json for the moment :-/


[connectors] chore(microsoft): Add full path for all synchronized root nodes

Description
Add full path in root node names for easier identification :

Risk
Deploy Plan
deploy connectors

This is quite a divergence from what we're seeing in other connectors.
What happens if I then pick a parent node to sync?
Will this remove the path from the name?
I see the motivation but I think we should fix it horizontality and in the product for all connectors vs hacking with the node name 🙏


Problems merging a translation

Usually I can fix things forking a translation and putting the new description in place of the old one; this time it seems there are more issues: any tip to finally have it approved as it was the case with the other I did for the same kata?
[The author modified the description after that, but I thought that I went around that with the last fork, if this may help]

I was able to push it through by adding a new line. There is definitely a bug here but unless it starts happening more often I'm going to consider it tolerable and close it.


[scss] link doesn't work with ~Tilde to search in node module or webpack alias

Using a **~**Tilde to go directly to a node_module path without having to add aliases to webpack this path doesn't get resolved by VS Code.
So with a node_module 'bootstrap' installed, we can import bootstrap sass-files using this construction:
@import '~bootstrap/scss/bootstrap-grid';
@import '~bootstrap/scss/utilities/sizing';

While this is working fine with Webpack and the sass-loader/node_sass, VS Code doesn't recognise this tilde-construction and searches on a follow-link in our source-folder\bootstrap\ instead of in node_modules\bootstrap\
When we use aliases in Javascript we solved his resolve problem by adding aliases to the jsconfig.json. But AFAIK there's not such a config file for Sass in VS Code.
I've been searching everywhere, including settings on sass, but couldn't find anything where we could make VS Code respect the node_modules path (and Webpack aliases) with a tilde~.
So unfortunately we seem to not being able to click to follow link on these sass imports.
versions

VSCode Version: 1.37.0
OS Version: Windows 10 Home Updated

Steps to Reproduce:

Have a node modulle, like node_modules/bootstrap, installed
Import a sassfile from this module, like @import '~bootstrap/scss/bootstrap-grid';
Ctrl+click on the import line to try to 'Follow Link'
Result: it shows an error-message popup saying 'Unable to open' with the wrong path

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

Some extra information:
I see there happened to be a pull request here on exactly this and it seems to be merged, but for some reason doesn't work?:
https://github.com/microsoft/vscode/pull/70693
Documentation webpack about imports with a tilde:
https://webpack.js.org/loaders/sass-loader/#imports

I had this working but was unaware that the distributed version of vscode has parts of the codes webpacked, including where I had written the resolve function, so webpack complained.
This was rewritten but the rewritten version doesn't work as far as I'm aware.
We need to either find a way to get this to work after being processed by webpack, or have the code sit in a non-webpacked part of vscode (if that's exists/is possible) and have the code called from there.
Unfortunately my knowledge of why and how vscode are using webpack for their distributable is beyond me and I have no idea where to search for help on this.

Perhaps someone from the vscode team can answer:

why is vscode using webpack for the production build? Is this so it can be used within a browser?
is the entirety of vscode being passed through webpack?
If not, is it possible to write helper functions in part of the vscode that is not being webpacked, then use them from within the code that is?

For the last point, in extensions/css-language-features/server/src/utils/documentContext.ts:resolvePathToModule I want to use node module resolution if vscode is running within a node environment. I can't use require.resolve directly so either I need to call out to a helper function that can or somehow recreate what require.resolve is doing manually (this would probably need file system access though so would likely have the same issue).

webpack is probably the best packager out there. For browsers and also for commonjs. It's a pity it's so restrictive with require as it I could really allow require.resolve(module, {path: [...]})

is it possible to write helper functions in part of the vscode that is not being webpacked

In the webpack.config.js you set a list of node modules that should not be bundled but just required.
Alternative is to reimplement the require lookup. I think that's the typescipt language server does:
https://github.com/microsoft/TypeScript/blob/01e1b1bb276f48eaa0612a34dd8f1eec5d86bd49/src/compiler/moduleNameResolver.ts

I just remembered this
https://github.com/Microsoft/vscode-languageserver-node/blob/dbfd37e35953ad0ee14c4eeced8cfbc41697b47e/server/src/resolve.ts#L17-L24
Which I think is what's calling require.resolve when you do jump to definition on a .js file.
I assume there's a way we can use require.resolve in the same way? Will look in to it a bit more

webpack is probably the best packager out there

I'm not questioning webpack, just why there's a need to package at all and not just run from modular, transpiled code

https://github.com/browserify/resolve may be an option

In the webpack.config.js you set a list of node modules that should not be bundled but just required.

@aeschli I think you mentioned this before but I couldn't find it. Do you mean in this file?
https://github.com/microsoft/vscode/blob/master/extensions/shared.webpack.config.js
Or this one?
https://github.com/microsoft/vscode/blob/master/extensions/css-language-features/server/extension.webpack.config.js
Or another?

It would go into extension.webpack.config.js.
Example:
externals: {
			'vscode': 'commonjs vscode', // ignored because it doesn't exist
		},

I've drafted something here https://github.com/microsoft/vscode/pull/79651 but not had a chance to test yet, will hopefully pick up later or tomorrow

I propose that we don't add one-liner dependencies to just bypass webpack.
Does doing it this way work for your cases?

Find filepath
Find workspace folder path
For each folder along the way, determine if node_modules/<module-name> exists
If so resolve into it

For example, if you have /foo/bar/baz.js and your workspace folder is /foo, and you are requiring ~bootstrap/something.css, it would try:

/foo/bar/node_modules/boostrap/something.css
/foo/node_modules/boostrap/something.css
Stop here, even though require would try /foo/node_modules/boostrap/something.css, we don't reach outside your workspace.

Stop here, even though require would try /foo/node_modules/boostrap/something.css, we don't reach outside your workspace.

I think you mean /node_modules/bootstrap/something.css in that example.

@octref for now, I think your suggestion should satisfy most use cases, however I am pretty sure it will stop working once npm tink is finalised:
https://blog.npmjs.org/post/178027064160/next-generation-package-management
https://www.npmjs.com/package/tink
https://medium.com/@Andrew_Mc/yarn-pnp-npm-tink-and-pnpm-oh-my-720d02221b4b
I think using node module resolution directly would allow easier solutions to things like this. I also  previously mentioned yarn pnp and yarn portal which could cause issues but I'm not sure how these could be easily solved!

I'll try find some time today to modify #79651 to work as @octref has suggested

I just realised @zcat has left npm and development on tink seems to have stopped, though it is on the roadmap for npm 8

@octref I have raised a new PR implementing as you describe above, see #81555

If any one try https://github.com/webpack/enhanced-resolve?

I don't know much about sass and not sure what I need to verify this. If I follow the steps, I get this error when clicking the link

That's in a .css file. In a .scss file, the import path is not linkified.

@roblourens Not sure who you target your message at and what you do or don't know about sass etc., but maybe this helps you?:
The path starting with ~ should point to the local node_modules\bootstrap folder inside the projectfolder normally while most developers install bootstrap locally in their project. In the message in your screenshot it doesn't look like a project folder to me, but maybe you have your project inside /private/tmp and called your projectfolder sass?
Open door, but just in case: Is the bootstrap package installed there and does the file /private/tmp/sass/node_modules/bootstrap/scss/bootstrap-grid.scss (could be _bootstrap-grid.scss (with underscore)) exist?
Also as I understand it you see this in a .css file? Not sure what you did there, but @import statements can only be in a scss-file (sass) as that's sass-specific. There's no such thing in css and it wouldn't make sense to import a sass file from a css file either. But probably I just don't get what you mean?
If the import path in the scss-file is not 'linkified' it sounds like something is not working as expected, as normally @import-paths are linkified in scss files. Like so:

Does this help?

I'm asking @aeschli what I need to set up to verify that it was fixed. And CSS imports do exist. But when I have a workspace with a sass file and an import like

and node_modules/bootstrap/scss/bootstrap-grid.scss does exist, but the import is not linkified.

Just checking this on my commit here https://github.com/vscode-langservers/vscode-css-languageserver/commit/981a10cba07346c8b799b7f2985c545894a1a9d5
style.css
@import '~bootstrap/scss/bootstrap-grid'; /* unable to open */
@import '~bootstrap/scss/bootstrap-grid.scss'; /* works */
@import '../node_modules/bootstrap/scss/bootstrap-grid'; /* unable to open */
@import '~bootstrap/scss/bootstrap-grid'; /* unable to open */
@import '../node_modules/bootstrap/scss/utilities/sizing'; /* unable to open */

style.scss
@import '~bootstrap/scss/bootstrap-grid'; // works
@import '~bootstrap/scss/bootstrap-grid.scss'; // works
@import '../node_modules/bootstrap/scss/bootstrap-grid'; // works
@import '~bootstrap/scss/bootstrap-grid'; // works
@import '../node_modules/bootstrap/scss/utilities/sizing'; // works

All of the above is intended behaviour as I wouldn't expect a css file to import an scss file using scss filename resolution (but if for some reason it directly linked to the file then it links through fine).
@JacksonKearl @roblourens what versions/commits are you testing against?

here's a test project (make sure to yarn install)
https://github.com/penx/vscode-css-link

I feel like I'm missing a setting or an extension or something. This is on today's insiders.

@roblourens and the files definitely exist on disk? i.e. you’ve done a yarn or npm install bootstrap?
I’ll try check the insiders build shortly but may be Monday

@roblourens that's odd... for me the relative links do properly linkify.

I pushed a change to my demo repo as I had the wrong relative path in the root scss files sorry.
Confirmed for me on Insiders (commit b1ef2bf05f90dd0e6438782411ab84756f97d8f1 ) that in the scss file, only the relative links work.
However, on the commit where I raised the PR this was working. Will see if I can figure out what changes.

@aeschli looks like you removed the code here? https://github.com/microsoft/vscode/commit/2e5b0824d1d3c1114a6f7fc36f9694723c86fc9c#diff-90971796685fd5e48e8f97b9daf95404L58-L79

ok I'm confused now, I wrote a test that should be failing but it's passing.
https://github.com/microsoft/vscode/pull/99487
@aeschli could you take a look? I'm happy to work on this but could do with some pointers
e.g.

is linking for scss files being handled elsewhere?
why is this test passing if it's not working on Insiders?
what do I need to do to test/ensure this is still working with the web/headless version? (which I think has been the previous issue)

@penx Yes, I moved the code into the service: https://github.com/microsoft/vscode-css-languageservice/blob/master/src/services/cssNavigation.ts#L273
I broke the node module resolving for scss as scss has some additional resolving resolving on.
Sorry, for that. I pushed a fix, added a test for scss.
If you have any additional any tests, you can add them

css: https://github.com/microsoft/vscode-css-languageservice/blob/61ed31a6083a6f61f28433d71c188f375d586f0b/src/test/css/navigation.test.ts#L362
scss: https://github.com/microsoft/vscode-css-languageservice/blob/61ed31a6083a6f61f28433d71c188f375d586f0b/src/test/scss/scssNavigation.test.ts#L205
I will eventually remove the (now duplicated) tests from
https://github.com/microsoft/vscode/blob/master/extensions/css-language-features/server/src/test/links.test.ts

And CSS imports do exist.

Yes I know. But it doesn't make sense to import sass from css.


Newline proposal

There's been some recent changes on how insta handles newlines. IIUC, we now just trim them from the start and end of every snapshot; this avoids showing incorrect diffs and inline snapshots failing to discriminate between newlines in the code and newlines in the snapshot.
I think there's an approach that would handle this well, but not confident and wanted to socialize it in case I'm missing something:

If an inline snapshot is sufficiently short[^1], it can go on one line; i.e.assert_snapshot!("foo", @"foo");

If an inline snapshot isn't sufficiently short, a newline gets added at the start and end which aren't part of the snapshot value; i.e.:assert_snapshot!(foo, @r#"
    multiple
    lines
#");

...has a snapshot value of two lines; excluding the first and last newlines within the string
Then, if there are newlines in the snapshot value, these can then be recognized because we know we've added exactly one newline at the start and at the end. Here's an example if the snapshot value had a leading newline:assert_snapshot!(foo, @r#"

    multiple
    lines
#");

This way, I don't think there can be any ambiguity.
[^1]: this generally means "is only one line", though very long single lines could start on the next line and become multiline

One issue I didn't consider above is that some autoformatters & editors attempt to ensure there's exactly one new line at the end of non-empty files (for example end-of-file-fixer). Some options:

Users should exclude .snap files from that check themselves
Finish with a YAML-like ---
Finish with some special character that isn't normally used, such as ␙

One option would also just be to make the start/end marker be a problem of the formatter and say that if this is relevant, we encourage/provide support to format blessed markers into the output format.

https://github.com/mitsuhiko/insta/pull/506 does the "Finish with a YAML-like ---" option.
We'd need to adjust the assertion code to check for newlines. And would also need a transition period where snapshots pass but warn they won't in the future.

One option would also just be to make the start/end marker be a problem of the formatter and say that if this is relevant, we encourage/provide support to format blessed markers into the snapshot.

Is the "formatter" here yaml / json / ron? I think that could be fine but newlines are often an issue with blocks of text, which are likely using assert_snapshot. And we control yaml now!


[QUESTION] passing additional information to import class

Versions
Laravel Excel 3.1 (latest)
PHP 7.4
Laravel 6
Question
Is it possible to pass additional params to import class?
class MarketDataCsvImport implements ToCollection, WithHeadingRow
I need to know the type of imported data. I want to pass the type not within rows array but somehow differently. Is it possible? can you give me an advice?
I talk about such classes.
class MarketDataCsvImport implements ToCollection, WithHeadingRow
{
    public function collection(Collection $rows)
    {

Can I have some buisinesslogics based not on the rows only?

Import classes are just PoPo's, so you can pass in anything you want via constructor or setters. I'd highly recommend reading the architecture documentation and specifically this part: https://docs.laravel-excel.com/3.1/architecture/objects.html#constructor


fuse mount cannot set option 'default_permissions'

Describe the bug
Cannot set option 'default_permissions' by mount command.
User permission not working on client side.
node2@node2:/home/weed$ ll
drwxr-xr-x   1 root  root    0 9月  30 16:27 ./
drwxr-xr-x 112 root  root 4096 9月  30 00:28 ../
-rw-r--r--   1 other  other     0 9月  30 16:27 test
node2@node2:/home/weed$ id
uid=1000(node2) gid=1000(node2) 组=1000(node2),4(adm),24(cdrom),27(sudo),30(dip),46(plugdev),113(lpadmin),128(sambashare)
node2@node2:/home/weed$ rm test
node2@node2:/home/weed$ ll
drwxr-xr-x   1 root  root    0 9月  30 16:27 ./
drwxr-xr-x 112 root  root 4096 9月  30 00:28 ../

node2@node2:~$ sudo mount -t weed fuse /home/weed -o "default_permissions,filer='...',filer.path='/'"
node2@node2:~$ sudo mount|grep weed
...:/ on /home/weed type fuse.seaweedfs (rw,relatime,user_id=0,group_id=0,allow_other)

System Setup

List the command line to start "weed master", "weed volume", "weed filer", "weed s3", "weed mount".

weed master -ip=... -mdir=...
weed volume -ip=... -port=... -dir=... -mserver="..." -max=2000
weed filer -ip=... -master="..."
sudo mount -t weed fuse /home/weed -o "default_permissions,filer='...',filer.path='/'"

OS version: master/volume/filer - truenas linux 5.10, client - Ubuntu 16.04 and 22.04
output of weed version: version 30GB 3.29 1ffb1e696e1f651c6294fc34c86b9abb6db1985d linux amd64
if using filer, show the content of filer.toml: Using default settings

Expected behavior
Add default_permission option to fuse mount option and get permissions work.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

where can i get the latest build to test, or i have to build it by myself.

@chrislusf after the 3.30 update, i can't mount seaweedfs by mount command, and i can't add extra options too.


backup an sql server instance with copyonly option

hi ,
I try to backup an sql server instance with copyonly option like this :
subclient.backup(backup_level=r'full', data_options=['CopyOnly'])
This is not working .
Can you tell me why ?
thank
sorin

Hi @sorin00 ,
You are passing an incorrect list item here. The correct item to pass is copy_only. You can get the acceptable list values from the method docstrings.
https://github.com/Commvault/cvpysdk/blob/master/cvpysdk/subclients/sqlsubclient.py#L201
Thanks

thanks
you can close the ticket
sorin00


docs(devs-start): about web components

What I did

Added / improved "About Web Components" section under Developers' Get Started page

Testing Instructions

Proofread content and ensure accuracy

Closes #1616

Few copy notes...
What are web components?

No need for semicolons in the third sentence (since there are currently no commas in the list). Should just be commas.

Web standard APIs

Change "build" to "built" in the first sentence.
I'd probably change the end of the last sentence to a parenthetical: "(e.g., Scoped Custom Element Registries)" instead of "for example, Scoped Custom Element Registries."
OR rework the whole sentence: "As the browser's native component model, Web Components continually benefit as the web platform adds new specifications and features, like Scoped Custom Element Registries."

Framework agnostic

Change "javascript" to "JavaScript" (or just "JS").
Add an "and" to the last sentence: "and reduce duplication of effort."
Reword the third sentence, since it's not clear who "their" refers to or who is switching the frameworks. Are you really saying something like this: "Engineers can then switch frameworks, if necessary, without throwing away their previous work"? Play Mad Libs with "Engineers," if I'm wrong on who's doing the framework switching, but the sentence should have a more-defined subject.

@markcaron Content here looks good to me.


[INLONG-552] TUBEMQ-421 rebase master branch

TUBEMQ-421 rebase master branch
JIRA link - [INLONG-552] created by stingpeng

fixed by https://github.com/apache/incubator-inlong/pull/413


[Improvement] Create Function for Whitelist Checking

The PR https://github.com/cybergis/cybergis-compute-core/pull/59 added the Core checking the Jupyterhost against a whitelist of hosts that we support. While this works, it uses copy/pasted code for many routes.
This issue requests that we turn that into a function and ensure that it's called to check for the whitelist on all routes that would require a jupyterhubtoken.

Closed by #68


Feature: Allow users to delete their own scripts

Description
People who are logged in should have the capability to delete scripts they own.

Resolved in #217


go to symbol regression in 1.73.1

Does this issue occur when all extensions are disabled?: yes

VS Code Version: 1.73.1
OS Version: macOS 12.1

If symbol has capitalization, go to symbol won't find matches, even if the query matches exactly.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.73.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Please do not provide just screen shot but always provide the source as text (copy-paste-friendly)

one.md
# hey there

asdf

# hi there

asdf

two.md
# THIS HEADER

asdf

# THAT HEADER

asdf


CI: switch to Github Actions?

Github Actions is an alternative to circleci. It looks worth exploring! Things like code coverage and uploading to codecov.io looks much simpler for instance (it's a bit like circleci's orbs but integrated into github; also the snippets are hosted here on this platform)
Discussion started here:
https://github.com/interchainio/tendermint-rs/pull/134#issuecomment-579710445

@ebuchman @xla what you are thoughts on this? If people are up for the change, I'd love to make a contribution to the project.

I’ve moved most of my projects over. The UX is much better, and actions-rs is great

@marbar3778 at least I don't see a reason not to open a PR. I think it will also clarify a few things (e.g. performance, configurability) and make the discussion less abstract.

@marbar3778 make sure to check out https://github.com/actions-rs if you haven't already seen it

I have a branch here: https://github.com/marbar3778/tendermint-rs/pull/1 just need to figure out the codecov part but everything else looks good. Once its fixed ill open the PR against this repo

Awesome! The codecov part looks seemingly easy with this: https://github.com/marketplace/actions/rust-grcov
although, as always the devil is (or might be) in the details: https://github.com/actions-rs/grcov/issues/3
The good thing about GA is that one can read through the code and understand what is going on in the background (in this case TypeScript)

grcov + codecov works fine

Awesome work Marko, thanks!


Implement NuGet redirection on renaming of packages

I'm not sure which GitHub repository is responsible for the problem. Thus I post the new issue here.
I see, that a lot of developers who starts with ASP.NET 5 with early betas or RC1 spend a lot of time in migration to the next version. One of the common problem is renaming of packages. One continue the renaming in RC2 (see many announcements already published). Many developers even have no idea about renaming because NuGet gives no information about it. In many cases the renaming holds the most names of the methods.
Let us we look on an example. EntityFramework.SqlServer is renamed to EntityFramework.MicrosoftSqlServer in RC1, but the NuGet repository https://www.nuget.org/packages/EntityFramework.SqlServer/ gives no information about such renaming. I suggest to implement some kind of automatic redirection to https://www.nuget.org/packages/EntityFramework.MicrosoftSqlServer/. On searching of updates to the latest version of EntityFramework.SqlServer (dnu restore or some other) one should get at least a warning about renaming of the package to EntityFramework.MicrosoftSqlServer in version 7.0.0-rc1-final instead of just showing that 7.0.0-beta8 is the latest version of EntityFramework.SqlServer and everything is "up to date".
It would be of cause more better to implement  automatic renaming (or renaming after the confirmation dialog) of the entries from project.json and implementation the Tooltips in Visual Studio to show (or include) the correct using statement. For example if I would type
services.AddIdentity<ApplicationUser, IdentityRole>();

I'll get tooltips like

and then another one, which allows me to include "Microsoft.AspNet.Identity": "3.0.0-rc1-final" and "Microsoft.AspNet.Identity.EntityFramework": "3.0.0-rc1-final" in project.json with additional including using Microsoft.AspNet.Identity.EntityFramework; in the C# code.
I think that the exact implementation can suggest the corresponding team of ASP.NET 5 project. The more comfort on renaming of packages the better, but the current state with no warning is really very bad.

@yishaigalatzer

Yes we are considering such an item (definitely not auto redirection) but a warning and suggestion for sure.
Currently package owners have the full capability of doing this by posting a new package with a readme or error various different ways to do (you can see win2d as an example)

@yishaigalatzer The package owner, can change "Title", "Description" and some other fields, which will be not used by  dnu restore for example. Placing information like "deprecated Win2D 1.0.0" and "DEPRECATED - please use Win2D.win81 or Win2D.uwp instead" is really absolute minimum which one could do, but such step don't solve the problem of any real solution, which has many projects with its dependencies. Event such simple step didn't made by ASP.NET 5 team till now.
I look at the problem from the position of developer, which uses many packages. One common design problem is that the name of the package is the same as the id, which one uses for references. It means that the renaming breaks all existing references.
The most renaming of packages made last time by the ASP.NET team have no changes in the functionality. One wanted mostly just to change "the label" only, not the ID. Thus I see very important to extend the NuGet model to allow the redirection. I made design analysing of package dependencies for one large German company and I know that the problem is really complex and have many aspects. I could identify that renaming is relatively common case, for example, if one software product will be sold to another company, or after the companies involved in the takeover, or if the company reorganizes the structure of projects (like in your case) and so on. Thus what I suggest is mostly two things:

consider to provide non-mutable package id (a GUID/UUID or any other) or NuGet packages; I understand that the problem is not simple if one take in consideration long life of the product (think about Package Code and Upgrade Code of the Windows Installer), but any constant ID is better as changeable name. One can at least use the ID for internal references in NuGet repository. It could be that the ID already exist.
providing redirection ways by addressing (referencing) of the package by name. I mean that the publisher of the package can declare the end of life of one name and to choose the form of redirection or just a warning message: either automatic redirection with small warning or just a warning message without any actions.

The publisher of the old package should be able to decide the strategy of redirection. The current state of the problem is really terrible. The publisher renamed EntityFramework.SqlServer after the version 7.0.0-beta8 to EntityFramework.MicrosoftSqlServer and published the next minor version as 7.0.0-rc1-final. The developers, who had complex solution with many projects, like many class libraries, which used EntityFramework.SqlServer name get no information about the renaming (which is just changing "the label" in my previous definition). Everything (7.0.0-*) will be successful resolved to the last old version (7.0.0-beta8). It's the reason of many errors. It's difficult to look through all announcements published on https://github.com/aspnet/Announcements/issues, through readmes and so on. One can take a look on the questions on the stackoverflow after publishing of every new beta release of ASP.NET. One sees a lot of project.json which uses packages by old names and just try to solve new problems after changing to the new version.
The example with splitting of one package Win2D into two new packages Win2D.win81 and Win2D.uwp is less common case. I think that the current state with the package is also not sutisfied. If I would try to add "Win2D" in the "dependencies" part of some the project.json I will see all published versions of the package and will just choose "1.0.0" as the latest one. If I would resolve the dependency in Visual Studio 2015 or with dnu restore I will see "Restore complete" in the output. The project which have
"dependencies": {
    "Win2D": "1.0.0"
}

will be successfully build and the developer get no advice that the project uses the package, which is deprecated since more as a half of year. Even if the developer would open "Updates" tab of NuGet Package Manager in Visual Studio Project, he/she will get no information that any update exist.
I understand, that it would be better to implement such feature already yesterday, but probably one could better implement some features tomorrow as never.

In the win2d case the user can take the extra step and actually publish a new version that adds a readme, throws an error etc.
The way we currently think about it is a deprecation feature rather than redirection. So as a user you get an error and a message.
We don't have a specific design yet, and once we get far enough to a point that we can picking this idea up we will pay Av design on our wiki.
Either way this is the wrong repo to track this as dnu has reached and end of life and we are moving to use nuget proper for restore.

Oh and we advise against using * notation for public packages. It is designed for CI build updates

@yishaigalatzer An introduction of deprecation could be the only option, where I can agree with you under some conditions, but sometime one want to build with specific old version to revert to old version if something is broken, for example.
The problem is that one need mostly to use the latest version of some package (like moving from Newtonsoft.Json 8.0.0 to 8.0.1 and then to 8.0.2). Because of that exist the syntax in packages like ^2.6.0 or ~2.6.0. There are exist tools like ncu, which one starts automatically and so on. GitHub allows fo example to transfer the repository to another location and the old URLs will be automatically redirected to the new location. Npm have at least deprecate command and I don't know the practice that one renames the old packages to another name. On the other side ASP.NET team makes many renaming of assemblies, which had no problem before the publishing the assembly as the package.
In any way I wanted just to give an advice on the problem, which I see, but the ASP.NET team (NuGet team) decide of cause yourself what should be implemented and what not. Thanks your your answer.

For anyone else finding this issue and wanting this as a feature, you can do a pseudo redirection by creating one last package version for the old package that only has a readme.txt (explaining the redirect), and that last old package should list the new package info as the only dependency.
Users can then be kept along your intended upgrade path and only have to remove the last old package at some point (but not remove its dependencies).


File watcher latency is higher than syncthing-inotify

For my use case, I need file changes to be detected and synched with the lowest latency as possible.
I did some tests with syncthing-inotify binary, and when changing a file, changes are propagated with an acceptable latency, around 1 or 2 seconds.
But when using SyncTrayzor with file watcher enabled, file changes are synched with a higher latency, from 4 to 10 seconds.
After reading sources, it seems that when changing a single file, the whole directory is synched, making the process slower than with syncthing-inotify. Also there are some features in syncthing-inotify, like -interval="500ms" to throttle calls to syncthing that are interesting and could be implemented in this project too.
I would like to contribute with a pull request, but when trying to build the project, I have the following errors, could you help me please ?
Severity	Code	Description	Project	File	Line	Suppression State
Error		The tag 'Interaction.Behaviors' does not exist in XML namespace 'http://schemas.microsoft.com/expression/2010/interactivity'. Line 25 Position 18.	SyncTrayzor	C:\devel\projects\SyncTrayzor\src\SyncTrayzor\NotifyIcon\TaskbarIconResources.xaml	25
Error		The tag 'Interaction.Behaviors' does not exist in XML namespace 'http://schemas.microsoft.com/expression/2010/interactivity'. Line 82 Position 22.	SyncTrayzor	C:\devel\projects\SyncTrayzor\src\SyncTrayzor\Pages\ConflictResolution\ConflictResolutionView.xaml	82
Error		The tag 'Interaction.Behaviors' does not exist in XML namespace 'http://schemas.microsoft.com/expression/2010/interactivity'. Line 31 Position 14.	SyncTrayzor	C:\devel\projects\SyncTrayzor\src\SyncTrayzor\Pages\ConsoleView.xaml	31
Error		The tag 'Interaction.Behaviors' does not exist in XML namespace 'http://schemas.microsoft.com/expression/2010/interactivity'. Line 19 Position 6.	SyncTrayzor	C:\devel\projects\SyncTrayzor\src\SyncTrayzor\Pages\Settings\SettingsView.xaml	19
Error		The tag 'Interaction.Behaviors' does not exist in XML namespace 'http://schemas.microsoft.com/expression/2010/interactivity'. Line 12 Position 6.	SyncTrayzor	C:\devel\projects\SyncTrayzor\src\SyncTrayzor\Pages\ShellView.xaml	12
Error		The tag 'EventTrigger' does not exist in XML namespace 'http://schemas.microsoft.com/expression/2010/interactivity'. Line 90 Position 50.	SyncTrayzor	C:\devel\projects\SyncTrayzor\src\SyncTrayzor\Pages\Tray\FileTransfersTrayView.xaml	90
Warning		The referenced component 'System.Windows.Interactivity' could not be found.	SyncTrayzor
Warning		Could not resolve this reference. Could not locate the assembly "System.Windows.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL". Check to make sure the assembly exists on disk. If this reference is required by your code, you may get compilation errors.	SyncTrayzor
Warning	IDE0006	Error encountered while loading the project. Some project features, such as full solution analysis for the failed project and projects that depend on it, have been disabled.	SyncTrayzor		1	Active

You'll need the blend SDK to build. I'll add it to the README.
Syncthing is gaining its own filesystem notification support though (it's in the RC currently as an advanced option) so syncthing-inotify is going to become obsolete and I'm going to remove my filesystem notification support. Therefore making any changes to it at this point is slorettu pointless I'm afraid.

Ho yes, good news, i'll try the RC then, thanks!


ipp-usb and CUPS?

Can you please clarify how ipp-usb and CUPS play together? CUPS has a USB backend, and speaks IPP, so does a person use ipp-usb  instead of CUPS?

Hi,
I hope Alex (the ipp-usb author) or you doesn't mind if I answer :) . ipp-usb registers a printer which is capable of IPP-over-USB standard and it is connected to your PC via USB as a DNS-SD service on localhost. Then CUPS can automatically find the service on localhost and create a temporary queue for it.
So ipp-usb is meant to provide DNS-SD service of USB printer, which CUPS can easily process. The biggest pros are you don't need a classic driver for your printer, because CUPS is able to generate 'driver' by itself based on IPP communication with device, and that you don't need to install a printer on your machine at all - the printer will appear when you are about to print (in application's print dialog) and it will disappear after successful printing, so it doesn't block your resources.
USB backend in CUPS is for devices which don't support IPP-over-USB standard and you need a classic driver for printer to work, if you use USB backend.
Alex, if I missed something, please correct me.
Thanks!

Hi guys,
sorry for delay with answer: I'm very busy with another project and feel very tired sometimes.
If ipp-usb is installed and printer supports IPP over USB protocol, most likely ipp-usb takes control, and CUPS works this way: CUPS->ipp-usb->printer. At this case, for CUPS the printer appears as a network printer, supporting IPP protocol and connected to the localhost, while ipp-usb translates network communication with CUPS into the USB-level communication with the device.
Usually, when ipp-usb takes control, it blocks all another USB-level drivers (printing, scanning, proprietary drivers) that may speak with the device in the same time, but there are some exceptions. Actually, it depends on device (how USB functions wired to the USB interfaces), but in most cases ipp-usb is not compatible with legacy USB drivers.
It is actually hard to tell, if the particular device supports IPP over USB protocol. For the convenience, ipp-usb check command prints a list of detected compatible devices:
# ipp-usb check
IPP over USB devices:
 Num  Device              Vndr:Prod  Model
   1. Bus 003 Device 003  0482:069d  "Kyocera ECOSYS M2040dn"

This is also possible to look to the avahi-browse output:
# avahi-browse -rt _ipp._tcp
+     lo IPv4 Kyocera ECOSYS M2040dn (USB)                  _ipp._tcp            local
=     lo IPv4 Kyocera ECOSYS M2040dn (USB)                  _ipp._tcp            local
   hostname = [tosha.local]
   address = [<IP_ADDRESS>]
   port = [60000]
   txt = ["air=none" "mopria-certified=1.3" "rp=ipp/print" "priority=50" "kind=document,envelope" "PaperMax=legal-A4" "URF=CP255,DM4,IFU0,IS19-20,OB1-10,PQ4,RS600,V1.4,W8" "UUID=4509a320-00a0-008f-00b6-002507510eca" "Color=F" "Duplex=T" "note=" "qtotal=1" "usb_MDL=ECOSYS M2040dn" "usb_MFG=Kyocera" "usb_CMD=PCLXL,PostScript Emulation,PCL5E,PJL" "ty=ECOSYS M2040dn" "product=(ECOSYS M2040dn)" "pdl=application/octet-stream,application/pdf,image/tiff,image/jpeg,image/urf,application/postscript,application/vnd.hp-PCL,application/vnd.hp-PCLXL,application/vnd.xpsdocument,image/pwg-raster" "txtvers=1" "adminurl=http://localhost:60000/airprint" "Fax=F" "Scan=T"]

If you, among other things, see a device at the <IP_ADDRESS>:60000 address, this is ipp-usb.

Thanks for all the details @alexpevzner and @zdohnal. Very clear. So many things to learn when CUPS doesn't work :)


Add ST_QA band for Landsat C2L2 products

When using the thermal bands from Landsat C2L2 products, the band ST_QA is a critical piece of information. This band contains the uncertainty of the Surface Temperature band (ST_B10), in K. It is used to ignore outlier values, resulting for instance from passing clouds.
There are currently, as far as I'm aware, no way to retrieve this band through EOReader so it needs to be loaded manually.
It would be interesting to be able to load this band directly. I see two ways of doing this :

Add the ST_QA band to the available bands for Landsat C2L2, which is probably the most straight-forward solution.

Return the ST_QA band alongside TIR_1 in the resulting XArray. The big downside of this approach is that it might break retro-compatibility, but I'd argue that the ST_QA band should pretty much always be used alongside the surface temperature.

This would also allow to pass a 'max_incertitude' kwarg to the load method, automatically replacing surface temperature pixels exceeding the given incertitude threshold by NaNs. Although this might go beyond the scope of this feature request.

I just realised the TIR_1 band from EOReader does not in fact match the B10 (Surface Temperature) band, making this issue pull double duty as B10 doesn't seem to be handled either.
However this band is only available for L8/9 so it would probably need a specialised subclass of LandsatProduct

I thanks for this issue !
I will first try to answer the second message.
B10 is the band read for TIR_1, so I don't understand what makes you think the contrary 😅 Could you precise what you mean by "matching" ? Are you converting the band to BT on your side by using the Landsat CCoefficients that can be found in the metadata?
Concerning the first message:
It could be a good thing to add the possibility to load masks in a not hidden way for every satellite. This would however take time. For now, you can just load this band by using the following workaround:
c2l2_prod._read_band(c2l2_prod._get_path("ST_QA"))

Thanks for the quick response and the workaround, this will work wonder
About the B10 issue :
After having a closer look at EOReader's code, I got mistaken between the _map_bands_tm() and _map_bands_etm() methods, so the band is indeed correct.
The values seem slightly off to me though, which is why I initially thought of retrieving the QA band.
I first opened it and tried to convert it to Celsius by first applying B10 * 0.00341802 + 149 to go from reflectance to Kelvin, as per the product's documentation (Page 12).
If I understand correctly, the conversion between Reflectance and Kelvins is done automatically when loading the band ? However the values don't seem to match when I do the same conversion manually.
This is the result of TIR_1 - 273 (to go to Celsius) with EOReader :

And this is the result of (B10 * 0.00341802 + 149) - 273 with Rasterio :

Note the different values in the legends

The only process I'm doing to the BT bands are the L1 conversion (see the _to_tb function, following this tutorial)
It should be OK for L1 data, but you are right, maybe it is wrong for L2 data (to be honest, I never really use L2 data, so this may not be tested well 😓)
If I understand correctly, for L2 level, the offset and gain I should use from the MTD are the one stored in LEVEL2_SURFACE_REFLECTANCE_PARAMETERS and LEVEL2_SURFACE_TEMPERATURE_PARAMETERS, is that right ?

There is an issue when loading the data : The metadata can't be read.
The issue comes from the band name, which is ST_10 internally whereas the expected metadata name is TEMPERATURE_MULT_BAND_ST_B10

Sorry my last comment wasn't really clear. There is a problem with the metadata being read, which isn't caught in the try/except block because it's a TypeError, and not a ValueError :
TypeError: float() argument must be a string or a real number, not 'NoneType'

Arf sorry, I did this too fast and I haven't noticed I wasn't testing thermal bands in my CI 🙄
This should be OK now, could you please try again?

Yes, all fixed. The two resulting rasters (rasterio and eoreader) are identical ! Thank you

Ok I'll close this issue as your demand for loading quality masks is openend in another issue:

#101


bench: 1.35.3 / Alonzo

So, we have the 1.35.3 leading to a massive CPU usage increase, so we need to re-benchmark that.

Closing this. If this is still relevant, please re-open it.


⚠ GreatPosterWall has degraded performance

In dfa281d, GreatPosterWall ($GPW) experienced degraded performance:

HTTP code: 200
Response time: 1887 ms

Resolved: GreatPosterWall performance has improved in 9f1a558.


Build error: missing ssl_api.hrl

Have Erlang/OTP 22 installed.  Cloned the plugin and ran make:
`

make
rebar3 compile
===> Verifying dependencies...
===> Running neotoma...
===> Compiling c_src/decoder.c
===> Compiling c_src/encoder.c
... more ...
===> Compiling emqx_reloader
===> Compiling esockd
===> Compiling _build/default/lib/esockd/src/esockd_transport.erl failed
_build/default/lib/esockd/src/esockd_transport.erl:20: can't find include lib "ssl/src/ssl_api.hrl"
`

Hi  @cogitaria-admin It is a BUG in rebar3.
Plz use the rebar3 in the emqx-rel repo

Thank you!
On Mon, Aug 17, 2020 at 1:49 AM JianBo He<EMAIL_ADDRESS>wrote:

Hi @cogitaria-admin https://github.com/cogitaria-admin It is a BUG in
rebar3.
Plz use the rebar3 in the emqx-rel repo
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/emqx/emqx-auth-http/issues/197#issuecomment-674750270,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHBJIAIAW2PFDLJZ67ZCBBLSBDVLNANCNFSM4QA7RHHA
.


[Bug] IndexError attempting to list builds

Describe the bug
Attempting to list the builds for a pipeline, I bump into an error.
To Reproduce
Azure Cli Version:
$ az --version | grep azure-cli
azure-cli (2.0.55)

Azure-Devops extension version:
$ az --version | grep devops
azure-devops (0.2.0)

Steps to reproduce the behavior:

login using az login
az pipelines build list --organization jaraco --project pip-run
See error

Expected behavior
Expected to see the list of builds for that pipeline or at least a better error message explaining what expectation was missed.
Debug logs
https://gist.github.com/ce354b016106a7d370f5e6d769aef7a9
Additional context
This is my first time attempting to use the az cli for Azure Devops, so I'm very new. I was using the vsts-cli earlier. The project is one I've just created today and is the only one in my org (which is just me).
I don't see any resources in the Azure portal. Is that expected?

I also saw this error on login:
pip-run master $ az login
Note, we have launched a browser for you to login. For old experience with device code, use "az login --use-device-code"
You have logged in. Now let us find all the subscriptions to which you have access...
No subscriptions were found for 'None'. If this is expected, use '--allow-no-subscriptions' to have tenant level accesses

Perhaps that's related.

You do not seem to have azure subscription associated with the account you used to login.
As the message says.. try az login --allow-no-subscriptions
and then try the devops commands.

@jaraco - did that work for you?

I tried logging in --allow-no-subscriptions, which suppresses the warning on login, but the az client still emits the same error as reported.
I was able to use the older vsts-cli, which was working... and of course I have pipelines set up (well just one).
I thought perhaps the problem was related to which account is associated with my pipelines. I have two Microsoft accounts associated with the same e-mail address (by necessity, not my choice), so I tried logging in with each. I think by default az login was using my "work" account (and has no subscriptions), so I tried a second time with my "personal" account. This time, az login showed two subscriptions:
$ az login
Note, we have launched a browser for you to login. For old experience with device code, use "az login --use-device-code"
You have logged in. Now let us find all the subscriptions to which you have access...
[
  {
    "cloudName": "AzureCloud",
    "id": "db9079df-1ed4-4475-a341-543afc87979d",
    "isDefault": true,
    "name": "Visual Studio Enterprise",
    "state": "Enabled",
    "tenantId": "39b48c3c-3b5d-42bd-bb10-7095bdc0749f",
    "user": {
      "name"<EMAIL_ADDRESS>      "type": "user"
    }
  },
  {
    "cloudName": "AzureCloud",
    "id": "0cd3e5fd-e192-4bed-9805-f9a744569d51",
    "isDefault": false,
    "name": "Free Trial",
    "state": "Disabled",
    "tenantId": "39b48c3c-3b5d-42bd-bb10-7095bdc0749f",
    "user": {
      "name"<EMAIL_ADDRESS>      "type": "user"
    }
  }
]

But still, az pipelines ... emits the IndexError.

Can you help me understand the relationship between DevOps pipelines and Azure subscriptions? Is a subscription not needed? Is there a way to tell which Microsoft account my DevOps pipelines are associated with?
Okay - I've confirmed that my Azure DevOps is associated with my "work" address:

And that's the account that doesn't have any subscriptions.

I see that in your trace the organization name is jarco. Could you give the fully qualified URL as organization and give it one more try (using the ID with which you were able to login, but it threw an error on az pipelines build list)?
az pipelines build list --organization https://dev.azure.com/jarco --project pip-run

Okay, I seem to be getting closer:
$ az login --allow-no-subscriptions
Note, we have launched a browser for you to login. For old experience with device code, use "az login --use-device-code"
You have logged in. Now let us find all the subscriptions to which you have access...
[
  {
    "cloudName": "AzureCloud",
    "id": "da4a5525-1805-4be8-913e-758c3b28915b",
    "isDefault": true,
    "name": "N/A(tenant level account)",
    "state": "Enabled",
    "tenantId": "da4a5525-1805-4be8-913e-758c3b28915b",
    "user": {
      "name"<EMAIL_ADDRESS>      "type": "user"
    }
  }
]
$ az pipelines build list --organization https://dev.azure.com/jaraco --project pip-run
Before you can run Azure DevOps commands, you need to run the login command(az login if using AAD/MSA identity else az devops login if using PAT token) to setup credentials.  Please see https://aka.ms/azure-devops-cli-auth for more information.

I'll follow up on the hint. I suspect I need to use a PAT token, as that's what I was using for vsts-cli.

Azure subscription is an azure concept and is used by Azure Pipelines when provisioning any resource on Azure. As per my understanding, if you are able to login and create a pipeline on Azure Pipelines using your work account which has no subscription, then you should be able to access the same pipelines via azlogin (using the --allow-no-subscriptions switch) and then running az pipelines build list. Let me know what happened after including the fully qualified URL for organization parameter in the list command.

Are you running this on an installed client or cloudshell (shell.azure.com)? If on client, could you retry with az logout and az login and give it another go? If on cloudshell, just do az login one more time and it should start working for you. Let me know if this is not the case.

After logging in with az devops login, the command succeeds (but only when the full URL is supplied):
$ az devops login
Token:
$ az pipelines build list --organization https://dev.azure.com/jaraco --project pip-run
[
  {
    "buildNumber": "20190207.3",
    "buildNumberRevision": 3,
...

I get the same IndexError if I don't supply the full URL.
I'm using an installed client (macOS Homebrew installed).
And I can confirm, after doing an az logout and az login --allow-no-subscriptions, the command is now succeeding:
~ $ az logout
~ $ az login --allow-no-subscriptions
Note, we have launched a browser for you to login. For old experience with device code, use "az login --use-device-code"
You have logged in. Now let us find all the subscriptions to which you have access...
[
  {
    "cloudName": "AzureCloud",
    "id": "da4a5525-1805-4be8-913e-758c3b28915b",
    "isDefault": true,
    "name": "N/A(tenant level account)",
    "state": "Enabled",
    "tenantId": "da4a5525-1805-4be8-913e-758c3b28915b",
    "user": {
      "name"<EMAIL_ADDRESS>      "type": "user"
    }
  }
]
~ $ az pipelines build list --organization https://dev.azure.com/jaraco --project pip-run
[
  {
    "buildNumber": "20190207.3",
...

And to be sure, I did an az devops logout to ensure it wasn't that credential that was allowing access, and I was still able to retrieve a result for build list.
To summarize, I think there are two issues at play here:

Because I have two Microsoft accounts under the same e-mail, it's easy to be confused as to which is the correct one to choose for a given service (I actually wonder if I should switch my organization over to the other ("personal") account, which has an MSDN subscription). Because of this, I was logging into the azure cli with the wrong account, one not associated with the DevOps org.
It wasn't clear from the error message that the --organization parameter was expecting the URL and not the name of the organization, leading to an obscure error message.

The latter issue is the main subject of my report. I still think there's something to be done here - specifically to provide a nicer error message if --organization doesn't look like a URL.
But based on the help, I believe I'm unblocked from further exploring use of the az cli as an interface for automating the enrollment of my projects with DevOps Pipelines. Thanks for the help.

Thanks for the update @jaraco. Glad to hear that you are unblocked. When I tried it out, I was getting an error indicating that the fully qualified url was missing. Will try and repro this at our end and fix any ambiguous error statements. Keeping this open till we complete an investigation from our end.

I am able to repro this issue. I was providing organization as myorg.visualstudio.com and it was throwing an error indicating it is an incorrect URL. However, on providing just myorg as organization value, it returns the same index out of range error. Need to improve the error message returned to indicate that full organization URL is expected.

This is the issue with how we parse URLs and not specific to build list. So it exists across all commands.

Thank you for reporting the issue.
Fixed in 0.11.0.
Release ETA: Next week.


Switch to Brotli from Gzip

Add a column for Brotli. I think it has wide-enough usage for static assets that it should be listed.
Use max compression level for Gzip. Static assets are "pay once to compress", so there's no downside to max compression level.

Thanks for the PR @JohnnyFangJr! Those are great contributions.
Happy to accept the Gzip max compression level change if you don't mind pulling that out into a separate PR.
I'm not sure about displaying both Gzip and Brotli because it doesn't fit the page width and it adds more complexity to reading the table. The data should also be less about comparing compression algorithms, and more about showing how compressible the minified output is so I think just showing one of them is enough.
I'm currently leaning towards replacing Gzip with Brotli because that's what the standard is today. Sadly major services like Fastly (used by GitHub for example) don't even support Brotli yet, so I'm worried this will make the data less relevant/practical.
Do you have any data on the adoption of Brotli vs Gzip? I couldn't find any from a quick Google, but that might make this decision easier.

Fastly doesn't "do the work for you" where you can upload the uncompressed asserts and they'll create the GZip version for you, but enable one to do it manually on their own. I'd
All modern browsers from past several years support Brotli, CanIUse puts it at 94% availability browser-side.
But I'm guessing your question is about bottom-line real world usage.
I googled and only found this source claiming 33% adoption by the top 10 million websites.
I agree about having more than one compression algorithm being bad - the whole table becomes noisy. Whereas the existing one is crisp clean and concise.
I don't really have a horse in this race. I thought it would be nice to have Brotli as it's slowly becoming the de-facto standard, but maybe it's still too early.
Up to you :)
I'll separate using max compression into a different PR.

Thanks for sharing that link! Very useful, and seems recent too.
It's been a few days now but let me know if you're still planning to separate out the Gzip max compression to a separate branch.
Afterwards, this branch can be updated to replace Gzip with Brotli and I can build it to keep a reference.

I updated this branch along with the title so you'll have it for future reference.
Using max compression submit separately.

Thanks so much @JohnnyFangJr ! 🙌

Use max compression level for Gzip. Static assets are "pay once to compress", so there's no downside to max compression level.

Both Cloudflare and Fastly don't provide information on which gzip compression level they are using.
Cloudflare uses level 5 Brotli compression (see https://blog.cloudflare.com/brotli-compression-using-a-reduced-dictionary/).
Terraform for Fastly VCL uses gzip level 3  as its default value (see https://registry.terraform.io/providers/fastly/fastly/latest/docs/resources/service_vcl)
Fastly also provide a Chrome extension chrome-extension-estimate-compression-level. If you test the extension on GitHub Pages (which is using Fastly), the estimate gzip level is 5.
So IMHO, we should set the compression level close to the real-world configuration (which should be 5).

at the very least i feel like the compression level should be stated
also the cloudflare and fastly compression level could probably be figured out by comparing local compression levels to whatever they're doing

If I can add grist to the mill, here are the levels choosed by deno:

Brotli: Quality level 6 is based on google's nginx default value for on-the-fly compression
Gzip: After level 1, it doesn't produce significant size difference. Probably the reason why nginx's default gzip compression level is 1.


Get transactions endpoint

go-spacemesh has one today, but the API does not. Right now the API just returns tx receipts, which include a txid. CC @avive

I think it should be part of the node service and return a transaction if the node knows about it. This should include transactions that the node submitted or got from the mempool and transactions which are on the mesh. If we put this in the mesh service then it doesn't make sense to return a transaction that the node knows about which is not yet on the mesh.

We have a transactions service. I think we should just modify TransactionsState() to include the transaction object in the TransactionsStateResponse. This makes sense as it will return to the caller the tx data (immutable) and its current state (mutable).

We might also want to add a flag to the param if the caller wants the full tx data (tx object) in the response or not because there will be many cases where it already has it and just want to know what the current state is.


fixes for some multiple resources related to compliance

Pull Request (PR) description
This PR resolves some issues with extracting information about multiple Compliance cases. Only one case hold policy and one case hold rule would be shown.
I also modified the code to extract accepted domains a bit.
The fix is mostly concatenating string differently and forcing the results of Get-* command to arrays to that it behaves as expected.
This Pull Request (PR) fixes the following issues
None.

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: vinkobedek sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #259 into Dev will increase coverage by 0.55%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              Dev     #259      +/-   ##
==========================================
+ Coverage   87.09%   87.64%   +0.55%
==========================================
  Files          62       62
  Lines        5787     5788       +1
  Branches       10       10
==========================================
+ Hits         5040     5073      +33
+ Misses        737      705      -32
  Partials       10       10

Impacted Files
Coverage Δ

...s/MSFT_SCCaseHoldPolicy/MSFT_SCCaseHoldPolicy.psm1
100% <100%> (+17.14%)
:arrow_up:

...urces/MSFT_SCCaseHoldRule/MSFT_SCCaseHoldRule.psm1
97.22% <100%> (+1.33%)
:arrow_up:

...MSFT_EXOAcceptedDomain/MSFT_EXOAcceptedDomain.psm1
87.8% <100%> (+15.18%)
:arrow_up:

...s/MSFT_SCComplianceCase/MSFT_SCComplianceCase.psm1
88.73% <100%> (+0.32%)
:arrow_up:

Continue to review full report at Codecov.

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


fix: opa success just continue

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

Comparison is base (021387c) 38.01% compared to head (0baee2c) 38.01%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #825   +/-   ##
=======================================
  Coverage   38.01%   38.01%
=======================================
  Files          61       61
  Lines       10428    10428
=======================================
  Hits         3964     3964
  Misses       6163     6163
  Partials      301      301


mk-session should attempt to load rules from qualified symbols

Right now (mk-session 'my.rule.ns) loads all rules from the namespace as expected. In some use cases, code may generate rules as a sequence in a particular symbol that users may want to load with (mk-session 'my.ns/my-rules), where my.ns/my-rules contains a production or sequence of productions. Today calling mk-session this way yields an exception since it assumes any symbol is a namespace.
To address this, we can check if mk-session is given a qualified symbol, and attempt to load rules from the content of that symbol.

Merged!


Distro-agnostic registry redirect

As a user of Zarf's Utility Cluster mode, I need a way to automatically mutate image registries specified in kubernetes container specs to point at my utility cluster, so that I can utilize the utility cluster without having to change all of my configuration files to point at a different registry
Notes:

The way this is talked about being done right now is to create a new Zarf example that deploys Kyverno with the replace-image-registry mutating webhook

Additional Background Context:
The reason Kyverno was suggested is because although containerd and cri-o offer registry mirroring functionality, they don't offer a consistent solution. A mutating webhook would result in a solution that's consistent regardless of the underlying implementation details (ex: cri-o vs containerd), and the mutating webhook is more push button to implement compared to the yak shaving exercise of figuring out a method of configuring cri-o / containerd's registry mirroring on every node.

Talked about this for a bit today, mostly in relation to the native-apply stuff that is being worked on. If we end up making a "Zarf Agent" that runs in the cluster we may not need this, because that would handle the need here already.
/cc @jeff-mccoy

Completed by #237


@inheritdoc does not work in Visual Studio Code (only @inheritDoc does)

VSCode intellisense does not seem to respect all lower-case @inheritdoc in tsdoc comments. It seems to only respect the camelCase @inheritDoc. Other tools (e.g. API-Extractor) do not seem to have any issue with the all lower-case form.
Note that the eslint plugin also does not seem to care about casing. If the all lower-case form is not considered valid per spec, then it would be nice if the eslint plugin would catch the incorrect casing.
That the linter and the VSCode behavior seem to disagree on casing requirements makes it very easy to introduce comments that are incompatible with VSCode intellisense.

Does @inheritDoc work for you today in vscode? I run into this quite frequently where it doesn't seem that inheritDoc is supported at all by vscode.

it doesn't work for me

typedoc does not identify inheritdoc so inheritDoc is the way. Either way a method description with @inheritDoc is not evaluated by vscode. Would be really nice if it did.


use error codes in stderr for processing errors

When minidump-stackwalk processes [ca472b14-67b6-4e00-a272-52f7c0211208])http://localhost:8000/report/index/ca472b14-67b6-4e00-a272-52f7c0211208?refresh=cache), it doesn't produce any JSON output, so there's no "status" field. Turns out Socorro's crash signature generation looks at that field for error codes and adds them to the signature. Fun times.
Instead of putting that data in the "status" field, minidump-stackwalk chose to spit it out to stderr. I agree that makes more sense.
However, now signatures have changed for error cases. I can adjust Socorro to look at stderr for phrases that indicate the various error situations, but the indicator text is an english sentence and someone could change that down the line breaking the Socorro processor code.
For example, minidump-stackwalk spits this out:
[ERROR] Error processing dump: The thread list stream was not found

I could look for "thread list stream was not found", but if anyone changes any of those words, the match fails and we'd never know about it.
I'd like to add error codes. So minidump-stackwalk spits out whatever, but also spits out ERROR_NO_THREAD_LIST. Maybe something like this:
[ERROR] Error processing dump: ERROR_NO_THREAD_LIST: The thread list stream was not found

I think it's sufficient to add error codes to this block here:
https://github.com/luser/rust-minidump/blob/6ccdead2bffb0c9db60037b1ea6923c69afba023/minidump-processor/src/processor.rs#L29-L38
Does that seem ok? Sometimes having error codes makes documentation easier. Would it help here?

More specifically, the signatures that changed are the ones that used to have ERROR_NO_THREAD_LIST and ERROR_NO_MINIDUMP_HEADER. I haven't seen other cases.
ERROR_NO_MINIDUMP_HEADER shows up when the minidump is 0-bytes. I added a check for that in the Socorro processor to shortcut the situation and not call minidump-stackwalk at all, so I'll handle that.
Thus we only really need to add ERROR_NO_THREAD_LIST for Socorro parity.

This is reasonable. I had also considered modifying mdsw to change the return code of main to reflect the error, but the string is probably less cryptic to integrate into socorro than "oh yeah 7 means ERROR_NO_THREAD_LIST".
I will implement your proposal now.

Thank you!


add "forgot password" feature

Please add a feature to reset the password of a user.

Finally merged, see #276. :smile: :tada:


Update mill-main to 0.10.1

Updates com.lihaoyi:mill-main from 0.10.0 to 0.10.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.10.0).
You might want to review and update them manually.
.github/workflows/build.yml
.github/workflows/publish_snapshot.yaml
.github/workflows/release.yml

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "mill-main" } ]

labels: library-update, early-semver-minor, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #185.


Profile: 500 internal server error

Discussed in https://github.com/EddieHubCommunity/LinkFree/discussions/4362

Originally posted by Collins-Omariba February  2, 2023
I get a 500 internal server error when I visit my profile, any ideas why?

There is a missing icon on this link ....
{
      "name": "portfolio website",

      "url": "https://collins-omariba.github.io"
    },

I think you found a bug as it should not break the whole page and use the fallback icon, thank you for raising 👍


not running on Pi with Ubuntu and pi-hole

I just did a fresh install of Ubuntu server 20.04.1 LTS on my raspberry pi and installed pi-hole.  I then installed Docker and your  stubby-docker.  Everything seemed to go fine, but when I tried running it, it said there's already something running on port 53 which I'm guessing is the pi-hole though when I run netstat -tulpn it shows the following:
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name tcp        0      0 <IP_ADDRESS>:46723         <IP_ADDRESS>:*               LISTEN      - tcp        0      0 <IP_ADDRESS>:4711          <IP_ADDRESS>:*               LISTEN      - tcp        0      0 <IP_ADDRESS>:80              <IP_ADDRESS>:*               LISTEN      - tcp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*               LISTEN      - tcp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*               LISTEN      - tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN      - tcp6       0      0 ::1:4711                :::*                    LISTEN      - tcp6       0      0 :::80                   :::*                    LISTEN      - tcp6       0      0 fe80::acc8:f287:7c15:53 :::*                    LISTEN      - tcp6       0      0 ::1:53                  :::*                    LISTEN      - tcp6       0      0 :::22                   :::*                    LISTEN      - udp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*                           - udp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*                           - udp        0      0 <IP_ADDRESS>:68              <IP_ADDRESS>:*                           - udp        0      0 <IP_ADDRESS>:68            <IP_ADDRESS>:*                           - udp6       0      0 fe80::acc8:f287:7c15:53 :::*                                - udp6       0      0 ::1:53                  :::*                                -
so I edited docker-compose.yml and changed the following:
    ports:      - "5335:5335/udp"      - "5335:5335/tcp"
then when I ran docker-compose up -d it appeared to work.  It now shows:
stubby-docker_stubby_1 is up-to-date stubby-docker_unbound_1 is up-to-date
though nothings showing running on port 5335
But when I go into my pi-hole's DNS settings and enter either <IP_ADDRESS>:5335 or <IP_ADDRESS>:5353 it tells me:
IP (<IP_ADDRESS>:5335) is invalid! No DNS server has been selected. The settings have been reset to their previous values
any ideas what I'm doing wrong?
thank you!

See if this info helps:
https://github.com/MatthewVance/unbound-docker/issues/35
Also, this may help: https://docs.docker.com/compose/networking/
On Sat, Dec 26, 2020, 5:50 AM thejim<EMAIL_ADDRESS>wrote:

I just did a fresh install of Ubuntu server 20.04.1 LTS on my raspberry pi
and installed pi-hole. I then installed Docker and your stubby-docker.
Everything seemed to go fine, but when I tried running it, it said there's
already something running on port 53 which I'm guessing is the pi-hole
though when I run netstat -tulpn it shows the following:
Proto Recv-Q Send-Q Local Address Foreign Address State PID/Program name
tcp 0 0 <IP_ADDRESS>:46723 <IP_ADDRESS>:* LISTEN - tcp 0 0 <IP_ADDRESS>:4711 <IP_ADDRESS>:*
LISTEN - tcp 0 0 <IP_ADDRESS>:80 <IP_ADDRESS>:* LISTEN - tcp 0 0 <IP_ADDRESS>:53
<IP_ADDRESS>:* LISTEN - tcp 0 0 <IP_ADDRESS>:53 <IP_ADDRESS>:* LISTEN - tcp 0 0
<IP_ADDRESS>:22 <IP_ADDRESS>:* LISTEN - tcp6 0 0 ::1:4711 :::* LISTEN - tcp6 0 0
:::80 :::* LISTEN - tcp6 0 0 fe80::acc8:f287:7c15:53 :::* LISTEN - tcp6 0 0
::1:53 :::* LISTEN - tcp6 0 0 :::22 :::* LISTEN - udp 0 0 <IP_ADDRESS>:53
<IP_ADDRESS>:* - udp 0 0 <IP_ADDRESS>:53 <IP_ADDRESS>:* - udp 0 0 <IP_ADDRESS>:68 <IP_ADDRESS>:*

udp 0 0 <IP_ADDRESS>:68 <IP_ADDRESS>:* - udp6 0 0 fe80::acc8:f287:7c15:53 :::*
udp6 0 0 ::1:53 :::* -

so I edited docker-compose.yml and changed the following:
ports: - "5335:5335/udp" - "5335:5335/tcp"
then when I ran docker-compose up -d it appeared to work. It now shows:
stubby-docker_stubby_1 is up-to-date stubby-docker_unbound_1 is up-to-date
though nothings showing running on port 5335
But when I go into my pi-hole's DNS settings and enter either
<IP_ADDRESS>:5335 or <IP_ADDRESS>:5353 it tells me:
IP (<IP_ADDRESS>:5335) is invalid! No DNS server has been selected. The
settings have been reset to their previous values
any ideas what I'm doing wrong?
thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MatthewVance/stubby-docker/issues/21, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUMHOVISVMRDEYPK7WWCQ3SWXEY3ANCNFSM4VJ3LLFA
.

Did the links I provided help?

Did the links I provided help?

Hi
I decided not to use stubby but rather just go with pihole and unbound
Thanks
Jim Kelley

On Jan 5, 2021, at 6:00 PM, Matthew Vance<EMAIL_ADDRESS>wrote:

Did the links I provided help?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi
I decided not to use stubby but rather just go with pihole and unbound
Thanks
Jim Kelley

On Jan 5, 2021, at 6:00 PM, Matthew Vance<EMAIL_ADDRESS>wrote:

Did the links I provided help?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


chore: bump and align package versions

needed to get web aligned on latest package versions

There should be an updated yarn.lock also

There should be an updated yarn.lock also

i've run yarn and it doesn't update the lock. it's because they're sub packages that don't have yarn locks, they're already in the root lockfile.


Inconsistency between sub and gsub

I'm not sure if this is a bug.
Passing an empty string as the third argument,  sub returns the replacement string.
λ> sub [re|^$|] "replacement" ""
"replacement"
λ> sub [re|.*|] "replacement" ""
"replacement"

But gsub returns an empty string for the same input.
λ> gsub [re|^$|] "replacement" ""
""
λ> gsub [re|.*|] "replacement" ""
""

That does look strange indeed… any idea why it could happen?

Please forgive my poor English...
The immediate cause is the gusb0.loop's first guard (offset >= l). offset and l are both 0.
However, without the guard gsub0 loops forever because any string has empty substring at its right end that matches the regex.
I think that the root of the problem is that gsub0 cannot determine whether the empty string is consumed or not from the value of offset.


Creates BoxPlot class and tests

Fixes #88

Sweet! Just checked the travis output and see that it skipped the matplotlib test. I'll see if I can sort that.

Latest commits now get the matplotlib tests passing on travis also. For the record I've had to use http://ryanvarley.uk/solutions/2014/05/14/how-to-solve-travis-ci-matplotlib-tclerror-no-display/ (writing this down so that when it breaks I can come back and see it: might throw it in an issue somewhere...).
On another note it looks like matplotlib is being installed every time a commit is pushed to travis. This seems to be the way things are done...


Fixes to the recording a bag tutorial.

This makes the tutorial work on Galactic.  In particular:

Switch to using a std_msgs/msg/String for the first example.
That's what 'demo_nodes_cpp::Talker' uses, so this should match.
Fix up the API calls in the generating your own data node
so it actually works.
Rename the data object to data_ to better match convention.

Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>@aprotyas FYI, this fixes the Record-a-Bag tutorials for me on Galactic.  This does not fix the problem for me on Rolling; further work will have to be done there.

Ah, and I found the magic to make this work on Rolling (it's one more change on top of this one).  So once this goes in, I'll port this to Rolling, then open one more PR to make these tutorials work there.

@mergifyio backport rolling


python3Packages.aiotractive: 0.5.2 -> 0.5.3

Motivation for this change
Update to latest upstream release 0.5.3
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.aiotractive
python39Packages.aiotractive


Updated the clean up application resource message

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2277125

@SanjalKatiyar , @GowthamShanmugam Please review

@GowthamShanmugam @SanjalKatiyar Please review

can we merge it @GowthamShanmugam @bipuladh ?

@TimothyAsirJeyasing What is the status we display on the dashboard when the list page shows cleanup pending?

I am approving this PR and merging this on 4.17, also this needs to be backported to 4.16. Please takecare of the dashboard changes on a separate PR with a different BZ.

/approve

/cherry-pick release-4.17

/cherry-pick release-4.17-compatibility


Segmentation fault (core dumped)

An error occurred when I try SIF on my Solidity codes
sif -a ~/Desktop/SIF/out546.ast -j ~/Desktop/SIF/out546.json -o out546.sol
Then I got
Segmentation fault (core dumped)
I don't have any clues about this error.

Could you provide the source code of the contract?

An error occurred when I try SIF on my Solidity codes
sif -a ~/Desktop/SIF/out546.ast -j ~/Desktop/SIF/out546.json -o out546.sol
Then I got
Segmentation fault (core dumped)
I don't have any clues about this error.

Hi there. I've fixed a tiny bug which causes seg faults and tested SIF again with a fairly big contract. Can you try the new version again?


WIP: Make Quantity an abstract base class

This is a draft PR just to make sure there are no big objections to moving forward with the idea.
The draft is there mostly to show how the new quantity and SupportFloatT is used, the _quantities module in particular has a lot of noise, so it shouldn't be reviewed in detail.
The code in this PR passes mypy and flake8 checks on src, but not in tests or benchmarks, as these are not updated yet.
The most important things to note are:

It uses SupportFloatT as a TypeVar bound to SupportsFloat to allow Sample and most other types working with either float or Quantities to be used.

This means for the generic types needing to actually use the underlaying value, we sometimes need an extra conversion to float when some operation is needed. This only happens in Sample3Phase to get the max/min, and the resampling function. It happens also implicitly when checking for NaN of infinity.

Added a value constructor to the resampler to be able to produce new samples. This has the nice side effect that if you need to resample something that you know is in, say, kW instead of W, you can use Power.from_kilowatts as the constructor.

There are quite a few (unfinished) fixes/improvements to Quantity and its sub types. The most important changes to Quantity are:

The trick to disable the default constructor was replaced by a more straightforward implementation in the __init__ and the utility _new method is added to create instances.
The .base_value was replaced by __float__, as this was only exposed to be able to work with a Quantity as if it was an unknown float.
QuantityT is not needed anymore and removed.
Quantity is now an ABC.
The _base_value is moved to the constructor, to have better rendering for the docs.
Removed isnan() and isinf() as now the math functions can be used directly instead (maybe we can keep them for convenience).
There are some random migrations to match instead of if, which needs to be cleaned up.

Of course once this have a rough approval, I'll split in smaller commits (and even PRs) and update tests and docs.
Fixes #821.

FYI, I cleaned up this PR a bit, it is still a WIP, but except from the commits with WIP in it, they should be much more readable and give a much better idea of what the intention of this PR is and how things are supposed to work.
That said when fixing the tests I bumped into an issue that might be tricky to solve.
If I understand correctly, all the FormulaEngine stuff is parametrized on a QuantityT mainly because of the high level formula engine, which allows to create formulas from formulas.
So far it seems like there is a lot of code and complexity to deal with QuantityT, which can be a Quantity or float, or even 3-phase stuff. For what I've seen these classes also don't differentiate very well between the input and output type, and these are usually mixed (one QuantityT TypeVar parameter for both things, which can resolve to Quantity and thus have a different type for the input and output, but the class is then specialized with Quantity, not the specific subclass).
As it is taking me a quite a bit of time  trying to understand all the formula engine stuff, in particular when thinking about input and output types, I will delay the work on this PR.
It is not very clear to me which stuff should work with float and which should use quantities. For example, the evaluation is all done with plain floats by the FormulaEvaluator, which also means that if we are mixing different types.
We need to make sure to use the base_value in a way that is compatible, in particular for multiplication (like current * voltage). In this sense I think we might need to have a more explicit definition that base_values are stored in the base SI unit or something like that, so we can operate on them safely as pure floats.
To progress with this PR I will probably add a second parameter for formula engine stuff that could have a different input and output type, so we have something like InputT and OutputT.

FYI, I cleaned up this PR a bit, it is still a WIP, but except from the commits with WIP in it, they should be much more readable and give a much better idea of what the intention of this PR is and how things are supposed to work.

This might have a new chance after this PR is merged:

https://github.com/frequenz-floss/frequenz-sdk-python/pull/1011


Update data-box-deploy-export-ordered.md

Fix typo for guidance on following steps to match the available options

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

emailed author

emailed author

Per Author:  Data Box work was transitioned to Aakash Dubey’s team. Can we reassign these PRs please? Thanks!

@akashdubey-ms - @alkohli indicated that this work has been reassigned to your team. Can you review this PR?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
@MicrosoftDocs/public-repo-pr-review-team

emailed assignee

Looks good.
#sign-off

#sign-off

#sign-off


Add Omen 15-en0010ca

Description of changes
The required modules not yet set to compile on Nixpkgs provided Kernels as of right now (See the relevant PR).
I've commented the relevant config out before the PR merges.
Things done

[x] Tested the changes in your own NixOS Configuration
[ ] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

I'll finish the README, then it'll be ready for review.

@mergify queue